fix: Refactor usages of unsafe (microsoft#95)

Addresses issue #95

@anubhabMajumdar are you willing to take this now that E2E is running?

@anubhabMajumdar are you willing to take this now that E2E is running?

@Troy-Butler Can you update the description with testing done for advanced mode and paste the advanced TCP metrics generated? Either local or remote context would work. This would ensure your code changes are tested. Thanks!

@anubhabMajumdar are you willing to take this now that E2E is running?

@Troy-Butler Can you update the description with testing done for advanced mode and paste the advanced TCP metrics generated? Either local or remote context would work. This would ensure your code changes are tested. Thanks!

Please forgive my ignorance, this is my first contribution to this project. What do I need to run to generate the metrics you need?

@anubhabMajumdar are you willing to take this now that E2E is running?

@Troy-Butler Can you update the description with testing done for advanced mode and paste the advanced TCP metrics generated? Either local or remote context would work. This would ensure your code changes are tested. Thanks!

Please forgive my ignorance, this is my first contribution to this project. What do I need to run to generate the metrics you need?

Apologies, I should have included the instructions. Here's how you can get started:

Build container image with your changes - https://retina.sh/docs/contributing/#build
Deploy in advance mode - https://retina.sh/docs/installation/setup#advanced-mode-with-local-context-with-capture-support
Check the *adv_* metrics - https://retina.sh/docs/contributing/#metrics

Another way to quickly test a single plugin on your host machine (in your case that would be Packetparser) would be this - https://retina.sh/docs/contributing/#build (look for To build binary of a plugin and test it).
Here's the corresponding main file you need to run - https://github.com/microsoft/retina/blob/main/test/plugin/packetparser/main_linux.go .
You would also need to build and run - https://github.com/microsoft/retina/tree/main/init/retina .
Let me know if you need any help, would be happy to help! Thanks again for your contribution.

Is there anything else I need to do?


Bumping the JS libs to fix the build

Bumping eslint-config-airbnb from 9.x to 14.x brought in a lot of linting issues, many of which I switched off in .eslintrc for the sake of moving forward.
We can go back and look at the list of blacklisted rules and decide which ones we want to tackle in individual PRs.
@ascott

Great PR, now everything works fine for me after npm install

LGTM


Proposal: Update rasterio backend to store CRS/nodata information in standard locations.

Problem description
Currently the way data is stored in the dataaray when using xarray.open_rasterio the crs and nodata information is stored in an attributes. It would be nice to be able to have them stored in standard locations so that other tools (rasterio, QGIS, GDAL) can find the information properly after dumping to a file with to_netcdf().
Proposed solutions
The nodata should be loaded into _FillValue
I propose that the CRS information be stored using the CF spatial_ref convention as it is supported by the main open source GIS tools. To do so, you add the crs coordinate to the dataset/dataarray. And then, you add the spatial_ref attribute to the crs which is stored as a crs WKT string. Next, you add the grid_mapping attribute to all associated variables that contains the coordinate name crs as the grid_mapping.
Here is an example of how it would look on a dataset:
<xarray.Dataset>
Dimensions:      (x: 65, y: 31)
Coordinates:
  * x            (x) float64 ...
  * y            (y) float64  ...
    time         datetime64[ns] ...
    crs          int64 ...
Data variables:
    ndvi         (y, x) float64 ...
Attributes:

Here is how the crs or spatial_ref coodinate variable would look:
<xarray.DataArray 'crs' ()>
array(0)
Coordinates:
    time         datetime64[ns] ...
    crs          int64 0
Attributes:
    spatial_ref:  PROJCS["UTM Zone 15, Northern Hemisphere",GEOGCS["WGS 84",D...

And here is how it would look on the variables:
<xarray.DataArray 'ndvi' (y: 31, x: 65)>
array([[ ...]])
Coordinates:
  * x            (x) float64 ...
  * y            (y) float64 ...
    time         datetime64[ns] ...
    crs          int64 0
Attributes:
    grid_mapping:  crs

More information about this is in https://github.com/pydata/xarray/issues/2288.

@snowman2 This should mean that open_dataset should handle CRS specially too, right? Currently it doesn't seem to do anything special for the coordinate variable pointed to by grid_mapping.

I wouldn't expect it to add crs if there wasn't a grid_mapping specified, but if it was then I would. In a simple test where I did xr.open_dataset('my_nc.nc') which has a grid_mapping attribute, xarray does nothing special to create a crs or other named coordinate variable referencing the associated grid_mapping variable.

This is the output of using xarray with a standard GOES-16 ABI L1B data file:
In [2]: import xarray as xr

In [3]: nc = xr.open_dataset('OR_ABI-L1b-RadF-M3C01_G16_s20181741200454_e20181741211221_c20181741211264.nc')

In [4]: nc.data_vars['Rad']
Out[4]:
<xarray.DataArray 'Rad' (y: 10848, x: 10848)>
array([[nan, nan, nan, ..., nan, nan, nan],
       [nan, nan, nan, ..., nan, nan, nan],
       [nan, nan, nan, ..., nan, nan, nan],
       ...,
       [nan, nan, nan, ..., nan, nan, nan],
       [nan, nan, nan, ..., nan, nan, nan],
       [nan, nan, nan, ..., nan, nan, nan]], dtype=float32)
Coordinates:
    t        datetime64[ns] ...
  * y        (y) float32 0.151858 0.15183 0.151802 0.151774 0.151746 ...
  * x        (x) float32 -0.151858 -0.15183 -0.151802 -0.151774 -0.151746 ...
    y_image  float32 ...
    x_image  float32 ...
Attributes:
    long_name:              ABI L1b Radiances
    standard_name:          toa_outgoing_radiance_per_unit_wavelength
    sensor_band_bit_depth:  10
    valid_range:            [   0 1022]
    units:                  W m-2 sr-1 um-1
    resolution:             y: 0.000028 rad x: 0.000028 rad
    grid_mapping:           goes_imager_projection
    cell_methods:           t: point area: point
    ancillary_variables:    DQF

In [5]: nc.data_vars['goes_imager_projection']
Out[5]:
<xarray.DataArray 'goes_imager_projection' ()>
array(-2147483647, dtype=int32)
Coordinates:
    t        datetime64[ns] ...
    y_image  float32 ...
    x_image  float32 ...
Attributes:
    long_name:                       GOES-R ABI fixed grid projection
    grid_mapping_name:               geostationary
    perspective_point_height:        35786023.0
    semi_major_axis:                 6378137.0
    semi_minor_axis:                 6356752.31414
    inverse_flattening:<PHONE_NUMBER>
    latitude_of_projection_origin:   0.0
    longitude_of_projection_origin:  -75.0
    sweep_angle_axis:                x

Again, I'm yet to be convinced that this logic should live in xarray, regardless if netCDF or rasterio is used as a backend. The job of xarray is to read the rasterio files in a "xarray way" (e.g. in order to leverage dask) and exposing the file attributes to the user, ideally with semantics/attribute names which are as close as possible as the underlying data model (rasterio).
All the logic you describe could live in a dedicated library which would become a wrapper around xarray's open_* functions, and converting them to the format you describe.

@fmaussion I completely agree except now that all of this is being brought up I see why it may have been better to put the 'crs' in the coordinates of the DataArray returned by open_rasterio. Since two DataArrays in different projections are not and should not be considered "mergeable". But I can also see how this walks the line of special handling of a data format by trying to interpret things in a certain way, but that line is already pretty blurry in the case of reading rasterio compatible datasets.

@fmaussion, I guess the main reason for the proposed change is to be able to read in the raster using open_rasterio() and be able to generate a netCDF using to_netcdf() that would be compatible with the rasterio model when using rasterio.open().
Also, since: "xarray.Dataset is an in-memory representation of a netCDF file", to me it makes more sense to store the data in a standard netCDF location that is consistent regardless of the backend. In doing so, other parts of xarray and libraries build off of xarray won't be required to find the same data in multiple locations.
Thanks for taking the time to consider this idea. I have used it quite a bit and it has made the datasets compatible across the various GIS tools I use (rasterio, GDAL, QGIS). It has definitely made my life easier and so I wanted to share what I have learned. But, it is just an idea. I really appreciate the work of the xarray developers and whatever y'all decide is fine with me.

I'm going to leave this decision to @shoyer and @jhamman !
If we go forward with this, we have a couple of possibilities:

add crs to a coordinate variable and keep the attribute too (not so nice because doubled)
add crs to a coordinate variable and deprecate the crs attribute

Same with _FillValue.

Feel free to re-open if this becomes something of interest. Thanks!


cmake: extend linker snippets for pm sections

This extends the linker snippet code to cover pm sections. It is useful for targets to put symbols used for context saving
in memory regions that can retain data even in low power states.

@tejlmand ping

@tejlmand  please review.


feat: switch to react-router in order to export static assets to host a Single Page Application

What does this PR do?
Description of Task to be completed
How should this be manually tested?
What are the relevant issues this PR belongs to
#
Any background context you want to add?
Screenshots (Optional)

Closed this one since we decided to drop NextJS entirely.


border dashed style can't parse

I set border style to div,

,but the actual display a solid line, so html2canvas can't parse 'dsahed'?

I have the same problem. Need a dashed border.


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/klein0r/ioBroker.comfoairq

[ ] [E114] No adapter are allowed in the repo without admin3 support

I have also found warnings that may be fixed if possible.

[ ] [W400] Cannot find "comfoairq" in latest repository
[ ] [W801] .npmignore not found

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


Fixing Typos

Fixing typos in documentation

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


tests: add a test to verify UEFI boot on x86_64 targets.

add a new test to verify UEFT bootable method on x86_64
targets, it also could be tested on qemu_x86_64 platforms
based on uefi-run and OVMF.
Signed-off-by: Chen Peng1<EMAIL_ADDRESS>

hi @nashif Could you help take a look of this? I added a test to verify the UEFI boot on qemu_x86_64 platform.
the only failure in CI pipeline is that for the new test, it needs external uefi-run and OVFM environment.

hi @nashif Could you help take a look of this? I added a test to verify the UEFI boot on qemu_x86_64 platform.
the only failure in CI pipeline is that for the new test, it needs external uefi-run and OVFM environment.

right, those need to be installed and available in the CI environment

@nashif ok, got it, thanks. I found CI environment used the docker image zephyrprojectrtos/ci:v0.17.1, I will add uefi-run and OVMF into the latest docker file.

added uefi-run and ovmf into CI docker in https://github.com/zephyrproject-rtos/docker-image/pull/68

moving the OVMF integration into the qemu board configuration

yes, it's better, I will update it.

@andyross Could you help review it again? I made some changes according to your comments, thanks.


⚠ Bofrost.be has degraded performance

In f1f2392, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1312 ms

Resolved: Bofrost.be performance has improved in 3ef2655 after 6 minutes.


Add updated build folder to GH (drawing-thing-v3).

submitted pr for this

includes #15

v3 added


Use a favicon maker instead of hardcoding all the Favicon junk

There's a great tool here which would save us time and mean we don't need to worry about building favicons.
This should nicely streamline all of this stuff

All done. cc @sergeitatarinov


areweipfsyet.rs doesn’t work

The website "areweipfsyet.rs" referred to in README.md does not return content if you try to go to it. Just a "this server could not be found" 404.

Never mind, it seems that whatever server it's on had to wake up 😅


[Feature_Request] Use API Instead of dedicated destination pages

Is your feature request related to a problem? Please describe.
It's not practical to create a new page every time we need to add a destination and gather details before checkout.
Describe the solution you'd like
Instead, we can use an API to obtain this information. This will not only increase productivity and efficiency but also expand the range of destinations available.
Demo Video/Screenshot (Mandatory )

Are you contributing under any open-source program ?
Yes, I am contributing under JWOC 2024.

can u please elaborate the feature more clearly then ill be able to work on it . assign it to me

can u please elaborate the feature more clearly then ill be able to work on it . assign it to me

Do you want to work on this issue?? If yes I can assign you this under GSSOC'24

yes i want to work on it and also provide some details so i can easily work on it

yes i want to work on it and also provide some details so i can easily work on it

use API to load the info , pics for the destination cards mentioned there

@rajvinder-kaur I have assigned you

@rajvinder-kaur are you still working on it?


fix(apps/prod/tekton/configs/triggers): fix trigger for monitoring component

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


Cam follows position on objects or whatever just help me

Describe your problem here.
So in short words, I have this bug here with the moving camera when the characters start singing, by some reason, the camera doesn't move when my secondary character sings. The game takes my secondary character as an object so, is there a way to make that, if an object starts singing, the camera moves too?
Another idea is to modify the script to make that, anything that touches the note, it makes the camera move by the note position (left note will make the camera go left, etc.)
I really need help on this
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

Use the Camera Follow Pos event and coordinate it so that the secondary character is in the middle
Note: the x and y pos for the secondary character may not be the same as the camera x and y

What'd u mean with tht? Like i did it and it doesn't work
Maybe i don't know how to use the script lol

What'd u mean with tht? Like i did it and it doesn't work
Maybe i don't know how to use the script lol

Place the event where the second character starts singing and it will work, I'd give you an example but I'm not at my computer right now, I will record one when I am though so that you'll understand

example


testing strong commit

implement strong vote with marker
add marker info to the QC
implemented the computation of the number of endorsers

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

Curious if this is intended to be a feature added to the Libra code base or an exploratory project? If the latter, I would recommend doing this as a PR in your own repo. We've done the similar with our exploration of SGX: https://github.com/zaxguo/libra/pull/1

Curious if this is intended to be a feature added to the Libra code base or an exploratory project? If the latter, I would recommend doing this as a PR in your own repo. We've done the similar with our exploration of SGX: zaxguo#1

Hi, this is currently a research project and hopefully can be added to the codebase in the future :) The PR is mainly for the purpose of cluster test and measurements, I would be happy to close this and reopen it only when doing the measurement.


Post EOS 4.26, QSFP default mode shouldn't be forced to 4x10G

Enhancement summary
1/ Current AVD behaviour:

Template "transceiver-qsfp-default-mode.j2" is configuring QSFP default mode to 4x10G unless we set "generate_default_config" to False
But, setting this var to False is not only removing the QSFP default mode config...

2/ Current EOS behaviour:

Starting from 4.26, the default beahviour changed for Trident and Tomahawk based equipments: the default mode is 40G instead of 4x10G (bug #90187)
The "no transceiver qsfp default-mode 4x10G" is now possible but only for Trident/Tomahawk (I confirmed in lab it is not effective on Jericho+ while effective on Trident3... I am using EOS 4.28.3M)

3/ Target AVD behaviour:

Introducing a new var of type boolean (example "qsfp_default_mode_channelized")
If True, configure "transceiver qsfp default-mode 4x10G"
If False, configure nothing
Since we don't used conditionals of type "if platform" or "if version", it is up to the user to ensure it is set to True for platforms other than Trident/Tomahawk and for EOS version lower than 4.26...

Which component of AVD is impacted
eos_cli_config_gen
Use case example
Trident/Tomahawk based device running 4.26+ has qsfp default mode 40G
Describe the solution you would like

Introducing a new var of type boolean (example "qsfp_default_mode_channelized")
If True, configure "transceiver qsfp default-mode 4x10G"
If False, configure nothing
Since we don't used conditionals of type "if platform" or "if version", it is up to the user to ensure the var is set to True for platforms other than Trident/Tomahawk and for EOS version lower than 4.26...

Describe alternatives you have considered
No response
Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

Duplicate of #3246


Update

Game Title
Planetside 2
Status of the Anti-Cheat
⭐ Supported
Proof of Mention
https://old.reddit.com/r/Planetside/comments/qrl1hq/guide_running_planetside_2_on_linux/

It'd be nice if we could get a developer's statement on this before I mark it as officially supported.


feat(hovercard): add hovercard implementation

feat(hovercard): add hovercard implementation
:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[ ] Fix
[x] Feature
[ ] Performance Improvement
[ ] Refactor

These types will not increment the version number, but will still deploy to documentation site on release:

[ ] Documentation
[ ] Build system
[ ] CI
[ ] Style (code style changes, not css changes)
[ ] Test
[ ] Other (chore)

:book: Jira Ticket
https://dialpad.atlassian.net/browse/DLT-616

:book: Description
Adds story following figma example.
Implements delay for when multiple hovercards are present. To do this, a singleton is used to create a global timer that keeps track of the id of the current hovercard being displayed, and changes it according to the logic that was defined in https://dialpad.atlassian.net/browse/DLT-1144
Adds tests.

:pencil: Checklist

For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

For all Vue changes:

[x] I have added / updated unit tests.
[x] I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script.
[ ] I have validated components with a screen reader.
[x] I have validated components keyboard navigation.

For all CSS changes:

[ ] I have used design tokens whenever possible.
[ ] I have considered how this change will behave on different screen sizes.
[ ] I have visually validated my change in light and dark mode.
[ ] I have used gap or flexbox properties for layout instead of margin whenever possible.

If new component:

I am exporting any new components or constants:

[x] from the index.js in the component directory.
[x] from the index.js in the root (either packages/dialtone-vue2 or packages/dialtone-vue3).

[ ] I have added the styles for the new component to the packages/dialtone-css package.
[ ] I have created a page for the new component on the documentation site in apps/dialtone-documentation.
[x] I have added the new component to common/components_list.cjs in packages/dialtone-vue2 and packages/dialtone-vue3
[x] I have created a component story in storybook
[x] I have created story / stories for any relevant component variants in storybook
[x] I have created a docs page for the component in storybook.
[x] I have checked that changing all props/slots via the UI in storybook works as expected.

:crystal_ball: Next Steps

:camera: Screenshots / GIFs

Fixes look good thanks! All that's left are these tests...

In vue2 version the hovercard close and open on click, this does not happen in vue3 version.
This could lead to an issue if you close the hovercard and mouseover again it quickly it does not appear again.

Will suggest to move the template to the middle of the screen, like what we have on popover, in that way we can better see the placement prop working well.

In vue2 version the hovercard close and open on click, this does not happen in vue3 version. This could lead to an issue if you close the hovercard and mouseover again it quickly it does not appear again.

Nice catch! Fixed in latest commit


Statement Delimiter on new row

Is your feature request related to a problem? Please describe.
Using SQL Editor:
As the statement delimiter gets added at the end of the last row of a query / statement, it can accidentally be commented out and cause issues in the subsequent statements.
-> issue #17241
Describe the solution you'd like
An option in the code formatter to move it on a separate line would be feasible.
-> Preferences > ...  > SQL-Editor > Formatting
Describe alternatives you've considered
Alternatively, there could be another option in SQL Processing ,like "Double blank line is statement delimiter", similar to the "Blank line is statement delimiter".
One blank line is often used to structure long statements for better readability, so for some developers one blank line is not a feasible statement delimiter.
Additional context
Issue:

Root Cause (code formatter):

More robust code with delimiter on separate row:

Hello @JoMrx
I'm not sure about the new option in settings - there are too many settings for now about delimiters.

Maybe you can use this setting?

Actually, I have not understood this option yet. It at least does not have effects for my problem.
The ; as statement delimiter in Snowflake should be on a separate line.

This option helps to add delimiter to the end of the line if you don't have it.
For example for query
select * from basket_a ba

select * from apple;

it will add delimiter in the end of the first select:
select * from basket_a ba;

select * from apple;

But this option doesn't help.
For example, this text
select *
-- from basket_a ba

select x.*
from apple;

becomes
select *
-- from basket_a ba;select *
from apple;

and the query is still broken.


Request: Custom Skill URL Paths

Short version: I would like additional control over the endpoint path.
Long version:
Currently, I have my own implementation of Alexa using very basic blocks in Node-JS.  While it works, I would like to take advantage of the features and ease you have in your nodes (such as complete message verification built-in).
For security reasons, I do not allow outside access to http://myinternalserver:1883 root.  Instead, I use a nginx proxy (which handles SSL termination) that allows external access to just http://myinternalserver:1883/alexa/*.  This way, those outside my network are limited to what is on my /alexa path and therefore cannot easily access everything else on my server (such as the node-red admin page).  While I could manually add each Alexa skill endpoint to my proxy, I would prefer to stick with an open endpoint if possible!
At a minimum, I would like to define a root path to base everything off of.  For example, use /my_path//alexa where "my_path" is a configuration option in settings.js.  If you really wanted to raise the bar, allow for complete control of the path on the skill configuration page instead.  You could then just suggest the current //alexa path as a default.
Thanks!

I second this request...


Add Dragonfly SPI pins

Add SPI name for dragonfly MTB board

We will merge the pin configs for now, in case failure is reported due to in-compatibility or some limitation of hardware, pins must be removed from json file.
CC @geky


migration: stateless incremental containers

This adds a new criu migration type "NONE" which can be used to perform mock
live migration. Any step but actually calling criu will be performed.
Closes #3798.
Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

jenkins: test this please

jenkins: test this please


Merge feature SGX private key provider

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@qiming-007 Could you raise a PR to enable SGX extension in Intel Istio Proxy at first like QATzip: https://github.com/intel/istio-proxy/pull/25/files, thanks!


[SourceKit] Fix RPATH to lib/swift/host in SourceKit frameworks

Relative path calculation to lib/swift/host was not correct.
rdar://115976985

@swift-ci Please smoke test

@swift-ci Please smoke test Windows


Release 1.1

@icygalz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-simpleTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-simpleTests

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-simpleTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-simpleTests

prow/release-e2e-mixer-no_auth
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-mixer-no_auth

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-simpleTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-simpleTests

prow/release-e2e-mixer-no_auth
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-mixer-no_auth

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-simpleTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-simpleTests

prow/release-e2e-mixer-no_auth
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-mixer-no_auth

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-simpleTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-simpleTests

prow/release-e2e-mixer-no_auth
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-mixer-no_auth

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-e2e-pilot-no_auth
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-pilot-no_auth

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.

@icygalz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-istioctl-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-istioctl-tests

prow/release-upgrade-tests-1.0.1
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.1

prow/release-unit-tests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-unit-tests

prow/release-upgrade-tests-1.0.2
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-upgrade-tests-1.0.2

prow/release-e2e-simpleTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-simpleTests

prow/release-e2e-mixer-no_auth
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-mixer-no_auth

prow/release-e2e-dashboard
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-dashboard

prow/release-e2e-stackdriver
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-stackdriver

prow/release-e2e-pilot-no_auth
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-pilot-no_auth

prow/release-e2e-bookinfoTests
c56ca2375fea5eff2ddbd31a5e0611aead80887b
link
/test release-e2e-bookinfoTests

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.


Refactor dataload structure and add some engine level method

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #826 (cc5b9dc) into master (914b21b) will decrease coverage by 0.50%.
The diff coverage is 0.00%.

:exclamation: Current head cc5b9dc differs from pull request most recent head a77863b.

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

Originally, the logic of dataload was divided into two parts: the general part and the implementation part. The whole reconciledataload was put in the implementation part
Now, after refactoring dataload structure, due to the addition of the engine level method, most of the logic should be put in the general part, and the implementation part should only include loading the runtime. It's like alluxio_ runtime_ Controller.go and runtime_ controller.go。
原先，dataload的逻辑被分为两部分：通用部分与实现部分，整个ReconcileDataload都放在了具体实现部分
现在，重构dataload逻辑之后，由于增加了engine级的方法，大部分逻辑应该放在通用部分，实现部分的逻辑应该只包括load the runtime。如同alluxio_runtime_controller.go和runtime_controller.go。

现在的重构逻辑就是把通用的放在dataload，需要engine自己去实现的都抽出来自己去实现

@frankleaf thanks very much for your PR. It's very important to decouple the definition and implementation. Would you mind to add some unit test here like https://github.com/fluid-cloudnative/fluid/pull/832 . Thank you in advanced.

Originally, the logic of dataload was divided into two parts: the general part and the implementation part. The whole reconciledataload was put in the implementation part
Now, after refactoring dataload structure, due to the addition of the engine level method, most of the logic should be put in the general part, and the implementation part should only include loading the runtime. It's like alluxio_ runtime_ Controller.go and runtime_ controller.go。
原先，dataload的逻辑被分为两部分：通用部分与实现部分，整个ReconcileDataload都放在了具体实现部分
现在，重构dataload逻辑之后，由于增加了engine级的方法，大部分逻辑应该放在通用部分，实现部分的逻辑应该只包括load the runtime。如同alluxio_runtime_controller.go和runtime_controller.go。

现在的重构逻辑就是把通用的放在dataload，需要engine自己去实现的都抽出来自己去实现

我指的是dataload_controller.go和implement.go这两个文件的逻辑关系

@frankleaf thanks very much for your PR. It's very important to decouple the definition and implementation. Would you mind to add some unit test here like #832 . Thank you in advanced.

ths, in action

@frankleaf please fix the conflict, thanks.


Make sure to log error in findObjectsForSrc()

It is hidden right now if there is an error in configured named fields and index. Lets log the error and return the current state of requests.
With this the findObjectsForSrc() will exit with an hidden error like:
"error": "Index with name field:.spec.ksmTls.caBundleSecretName does not exist"

/test designate-operator-build-deploy-kuttl


How to limit the download speed??

Please tell me ho can i limit the download speed of the client.

It's planed here #112

@Lunik u dont get my point i want to limit the downloading speed of the torrent not to limit the no of active torrents or the storage size.
I want to to limit the download speed of the files.

I see !


Added GDPR consent management to YOC VIS.X Bid Adapter

Added GDPR consent management support
Description updated

Related to https://github.com/prebid/Prebid.js/pull/2737

released with 1.15


Don't show active controller for KRaft clusters

Issue submitter TODO list

[X] I've searched for an already existing issues here
[X] I'm running a supported version of the application which is listed here and the feature is not present there

Is your proposal related to a problem?
No response
Describe the feature you're interested in
For clusters running in KRaft mode, the describe cluster API currently returns any alive broker's ID as controller ID Link to code. I believe this was done because, before KIP-919, clients couldn’t directly communicate with controllers and had to connect through brokers, making the specific broker ID irrelevant. Given this, perhaps for KRaft clusters, it might be better not to display the active controller indicator (and the green tick in front of the broker ID) to avoid confusing users?
Alternatively, in addition to the broker list, it might be helpful to also display the controller list by calling the describe cluster API with EndpointType set to 2. Or perhaps this could be displayed as a single list, since some nodes may have both broker and controller roles. In this scenario, displaying the active controller would make sense.
Describe alternatives you've considered
No response
Version you're running
1.0.0
Additional context
No response

@Haarolean, do you plan to remove the active controller from the UI for KRaft clusters, or do you plan to include the controller nodes as well?

@Haarolean, do you plan to remove the active controller from the UI for KRaft clusters, or do you plan to include the controller nodes as well?

no idea yet, maybe both


[REF-3448] 404 not rendering

Describe the bug
entering a random url give a 404 with text in the page
going to localhost:3000/404 updates the title but nothing is rendered in the page. Making a custom 404 has the same issue where no components are rendered in the page
To Reproduce
Steps to reproduce the behavior:

enter URL as above

Expected behavior
correct 404 page be returned at localhost:3000/404
Screenshots
If applicable, add screenshots to help explain your problem.
Specifics (please complete the following information):

Python Version:
Reflex Version:
OS:
Browser (Optional):

Additional context
Add any other context about the problem here.
REF-3448

I'm able to reproduce the issue, even with our main website https://reflex.dev/404/ renders as a blank page.
Not sure what the problem is.
Is there a use case? Or are you just wondering for consistency sake why /404 doesn't render? (I'm also curious)

If only for consistency we should figure out what happens and fix it I think.

No probs. My two use cases are:

Custom 404. You cant make a custom 404 now as no components will be rendered
I am using chakra.box as an iframe so i don't see the title change. All i can see in the iframe is blank space and i have no idea what is going on.

Custom 404

does it not work when you use app.add_custom_404_page? reflex.dev uses custom 404 and it seems to work fine (although accessing it directly via /404 does not work)

Custom or built in 404s do not render at /404. I am trying to handle and return /404 under certain cases that i check for. Automatically returning 404 on random routes doesnt help my use case.

Problems with dynamic routes return the broken /404 automatically as well.

@charlestondance can you share a small example snippet ? Just want to test if its fixed with the mentioned PR !

"""Welcome to Reflex! This file outlines the steps to create a basic app."""

import reflex as rx

from rxconfig import config

class State(rx.State):
    """The app state."""

    @rx.var
    def test_id(self) -> str:
        return self.router.page.params.get("id", "No Test ID")

@rx.page(route="/test/[id]")
def test():
    """A page that updates based on the route."""
    return rx.container(
        rx.heading(f"Testing ID: {State.test_id}", size="9"),
        spacing="5",
        align="center",
        justify="center",
        min_height="85vh",
    )

app = rx.App()
app.add_custom_404_page(
    rx.container(
        rx.color_mode.button(position="top-right"),
        rx.vstack(
            rx.heading("Oops! Page Not Found", size="9"),
            rx.text(
                "It looks like the page you're looking for doesn't exist. You can start exploring by editing ",
                rx.code(f"{config.app_name}/{config.app_name}.py"),
                size="5",
            ),
            rx.link(
                rx.button("Visit our Documentation"),
                href="https://reflex.dev/docs/getting-started/introduction/",
                is_external=True,
            ),
            spacing="5",
            justify="center",
            min_height="85vh",
        ),
        rx.logo(),
    )
)

I have verified my code using an example, as shown in the screenshots after the updated code mentioned in my PR.

@charlestondance Can you confirm once if this is the expected outcome !

This should be resolved in the latest code (please re-open if needed!)


Unable to calculate MD5 hash

Hi, i am uploading image and getting this error
Unable to calculate MD5 hash: /data/user/0/competent.groove.feetport/files/data/user/0/competent.groove.feetport/app_imageDir/IMG_20161122_073058.jpg: open failed: ENOENT (No such file or directory)

Hi achinverma,
Could you provide some steps to reproduce?
Please include which version of the SDK you are using and a sample image if possible.
Thank you.


Same text, different display.

The text is the same, but different in SVG and HTML.
text:"$\underline{\underline{\begin{array}11111111\end{array}}}$"
SVG:

HTML:

Why didn't anyone solve it? @dpvc

It is a bug and I haven't yet tracked down its cause.  Neither output is entirely correct.  I do not know when a fix will be ready.  Patience is a virtue.

Thank you very much. I hope you can inform me in time after the change.In fact, we have encountered some other mistakes in the process of use, but this problem is very special, and it is difficult for us to have a good way to display our resources.

==> In testsuite
MathMLToDisplay/Presentation/ScriptsAndLimits/issue1993.html


feat: use Fedora 38

Or is there a particular reason to stay on 37?

I thought to wait until nvidia marks 38 as :latest, and it seems that it does. Merging!


[tvla] Bug Fix

A bug was introduced to tvla in PR #134 when removing references to mode["otbn_modinv"].
It seems that a part of the condition was not deleted in one of the if branches, which results in an error when running tvla for KMAC.
This commit fixes the condition.

Oh no... that was my fault, sorry for that. I just missed this one. Thanks for fixing it @vrozic !!

No worries. This bug actually helped me identify a problem with our CI, so it's good that it happened.
Thanks for the review.


conditional fake data (e.g. postcode change in Ireland)

I was wondering if there is a need of conditional fake data.
E.g.:
The postcode of Ireland was recently adjusted to another format.
If you want to create fake data with correct formatting of postcode before and after adjustment, there is currently no proper way to do that without injecting a custom provider.
I thought about that issue.
There would be a way to deal with it.
My solution goes like that:
Each item in an fake data array can either be a simple string or an instance of a class implementing a new interface "FakeDataItemInterface".
In this interface you handle additional information like "validFromDate".
The base class than would inspect every item of an fake data array and checks if items with the new interface are valid.
If this is the case, the value of the item will be considered for random returning.

That's a nice idea for your code, but too much indirection for the Faker core IMO. I'll leave that to userland.


Unknown argument "nearText" on field

v1.3.1
seems nearText is not properly validated although supported.
either raw or api
const query =  '{Get{ Image(limit:3, bm25: {query:"sunglasses"}){url, _additional{score}}}}'

async function nearTextQueryRaw() {
    const res = await client
      .graphql
      .raw()
      .withQuery(query)
      .do();
    console.log(JSON.stringify(res));
    return res;
  }

async function nearTextQuery() {
  const res = await client.graphql
    .get()
    .withClassName('Article')
    .withFields('functional fictional')
    .withNearText({'concepts': ['sun',  'tree']})
    .withLimit(2)
    .do();

  console.log(JSON.stringify(res, null, 2));
  return res
}

ClientError: Unknown argument "nearText" on field "Article" of type "GetObjectsObj". Did you mean "nearVector" or "nearObject"?: {"response":{"errors":[{"locations":[{"column":14,"line":1}],"message":"Unknown argument \"nearText\" on field \"Article\" of type \"GetObjectsObj\". Did you mean \"nearVector\" or \"nearObject\"?","path":null}],"status":200,"headers":{}},"request":{"query":"{Get{Article(nearText:{concepts:[\"sun\",\"tree\"]},limit:2){functional fictional}}}"}}
    at makeRequest (/****/node_modules/weaviate-ts-client/node_modules/graphql-request/build/cjs/index.js:314:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  response: {
    errors: [
      {
        locations: [ { column: 14, line: 1 } ],
        message: 'Unknown argument "nearText" on field "Article" of type "GetObjectsObj". Did you mean "nearVector" or "nearObject"?',
        path: null
      }
    ]
     status: 200,
   ****
      }
    }
  },
  request: {
    query: '{Get{Article(nearText:{concepts:["sun","tree"]},limit:2){functional fictional}}}',
    variables: undefined
  }
}

not an issue - for Weaviate with no
this error comes from the remote - as not all vectorizers support all gql search terms.
I checked it on my local with text2vec-contextionary and it works fine.


OLS-988: RHOAI VLLM for e2e tests

Description
RHOAI VLLM for e2e tests
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[x] CI configuration change
[ ] Konflux configuration change

Related Tickets & Documents

Related Issue #OLS-988

/lgtm

/approve

/override "Red Hat Konflux / lightspeed-service-on-pull-request"

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

Project coverage is 94.90%. Comparing base (3141272) to head (7cbb740).
Report is 20 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1728   +/-   ##
=======================================
  Coverage   94.90%   94.90%
=======================================
  Files          60       60
  Lines        2689     2689
=======================================
  Hits         2552     2552
  Misses        137      137

see 1 file with indirect coverage changes


java.io.UncheckedIOException: Could not move temporary workspace and java.nio.file.AccessDeniedException

Current Behavior
2 Errors:
Currently I have the following errors when trying to synchronize the gradle in the project, I have also done the manual configuration and it still does not work, I also tried downgrading the gradle version from 8.7 to 8.6 -8.5 and it does not work,I also disabled the antivirus and it continues not working, can someone help me?
Caused by: java.io.UncheckedIOException: Could not move temporary workspace (C:\Users\USUARIO\Desktop\APPSJETPACkCOMPOSE\pruebagradle.gradle\8.7\dependencies-accessors\dcbcd87108cec70bb467e05d5172bee8efb944af-8a047ac5-58a0-4750-b4ff-819bcb7c25e2) to immutable location
Caused by: java.nio.file.AccessDeniedException: C:\Users\USUARIO\Desktop\APPSJETPACkCOMPOSE\pruebagradle.gradle\8.7\dependencies-accessors\dcbcd87108cec70bb467e05d5172bee8efb944af-8a047ac5-58a0-4750-b4ff-819bcb7c25e2 ->

Expected Behavior
waiting for a configuration that works
Context (optional)
Windows 11 PRO
Android Studio Jellyfish | 2023.3.1 Patch 1
Build #AI-233.14808.21.2331.11842104, built on May 14, 2024
Runtime version: 17.0.10+0--11572160 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 11.0

Steps to Reproduce

Gradle version
8.7
Build scan URL (optional)
No response
Your Environment (optional)
No response

This looks a lot like

https://github.com/gradle/gradle/issues/28948

Please investigate what blocks Gradle from copying files. Can it be caused by anti-virus or something like that?

@ov7a ,Apparently it is a blockage carried out by Windows Defender, and sometimes the configuration of manually modifying Gradle works, other times it does not work, and if I downgrade the version it gives me an error that the minimum requirement is version 8.6 of Gradle, the reality It is a bit tiring to start making these configurations that have little success, any recommendations?

This looks a lot like

Intermittent failure to move temporary workspace to immutable location in Gradle 8.7 on Windows #28948

Please investigate what blocks Gradle from copying files. Can it be caused by anti-virus or something like that?

Your issue appears to be a question about how to use Gradle. They should be asked in a different channel, so we will close it. Your question might be answered in Gradle documentation or on our community Slack or forums. Find links to these resources on help.gradle.org.
If you think our analysis is wrong, please provide us with more detailed information explaining why.

It looks like your environment setup is too strict. Gradle needs to be able to add, remove, and modify files both in the project directory hierarchy and in the Gradle User Home.
We cannot support running on an environment that limits the fundamental needs of a build tool whose job is to produce file artifacts.

modify the apps\APPNAME\android\gradlew.bat
"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %* **--info --verbose --stacktrace --scan --debug** to get more insights.

Windows Defender is OFF.
D Drive is DEV drive.
It's funny, because it worked yesterday perfectly fine. What's changed over the night - have no idea.

I'm currently dealing with this issue. I've tried a ton of things. The only thing that works so far is downgrading to gradle 8.3. But that's a bad option.
When I get these errors, and then I try to delete the folders/files referenced in the stacktrace, I get the following error:

That doesn't look like an antivirus issue to me. It looks like OpenJDK is not releasing the files. When I go to powershell and kill the task using tasklist | findstr java -> taskkill /F /PID <process id>, I can then proceed to delete the file.
So to me, this seems like an issue with OpenJDK.

@ov7a worth another look?

Try checking if you have given the java system security permission to modify the file

have a look here https://github.com/gradle/gradle/issues/28475#issuecomment-2373630093


Behind proxy with intellij

This small changes set:

Enables the Jenkins Docker image to run behind a password-less corporate HTTP proxy
Ignores IntelliJ-generated files in Git
use the headless JDK in the Docker image instead of the UI-capable one. But wouldn't the simple headless JRE make the job?

It is tested in macOS only, but I think it should run without an issue on Linux.

@asnowfix Please feel free to re-open this when you come back around to it.

@asnowfix Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

This is weird: I do not seem to have received comment emails earlier than https://github.com/edx/jenkins-configuration/pull/75#issuecomment-367145977 so I did not see the request for signing the CLA. I will engage the legal process here.


Fuel Codes | Adjustment to sorting the Low Carbon Fuel Code column ascending descending |1|

Location:
View Fuel Code table (IDIR only)

Administration tab
Fuel Codes sub-tab
Column: Low Carbon Fuel Code

Background:
A unique fuel pathway is given a single fuel code (e.g. BCLCF101.0); the number after the decimal illustrates a new version of the same fuel pathway. For example, when BCLCF101.0 expires the fuel producer will apply for renewal. The updated pathway will be given a new fuel code (BCLCF101.1) and a new entry created in the Fuel Code table.
When sorting these values by ascending/descending, TFRS should account for the number(s) after the decimal in determining which value is "higher" or "lower".
Examples:

BCLCF102.0 is "larger" than BCLCF101.0, which is "larger" than BCLCF101.5
BCLCF105.10 is "larger" than BCLCF105.5, which is larger than BCLCF105.1

Task:
When sorting the Low Carbon Fuel Code column by descending/ascending order (as is the default for the Fuel Codes sub-tab page), have the system account for the number(s) after the decimal in determining the appropriate order, as described above. https://trello.com/c/Uddr72G2/1320-fuel-codes-adjustment-to-sorting-the-low-carbon-fuel-code-column-ascending-descending-1

 Fuel Codes | Adjustment to sorting the Low Carbon Fuel Code column ascending descending |1|


Little issue with Serene installation process

@volkanceylan  another little issue with Serene Core installation process.
I have notices that if I select only Theme Samples (AdminLTE) during installation, the installation never ends.
Also, in the file system, the directories and the files of Basic Sample, Metting, etc. are created.

I'll try to reproduce.

I ran it with just theme samples selected and it created the project in a reasonable amount of time. and the modules that are initially built included -
Administration
AdminLTE
Common
Membership
built the project and it loaded and the ThemeSamples appear properly.

OK @mreid2005 thank you, I retry

your welcome, let us know how it turns out.

I have updated VS 2017 and now works. Tnx 👍


A linux update ?

Currently playing on Ubuntu 18.04, the mod doesn't work. Will it be updated to support the  v<IP_ADDRESS>7 linux version ?

Oh, so Linux is on a more advanced version number 👀. I didn't know that, the Mac OS and Windows version are both at <IP_ADDRESS>7, I know there's an update incoming in may, but... I don't think that's the one, was it updated recently ?

Nope that's just a mistake when writing the version from me, apologies. It is <IP_ADDRESS>7. The mod doesn't work for me, so I figured it was maybe the linux version that wasn't updated.

Did mod ever work on linux correctly? I noticed you've lower cased files in the past to fix it on linux.
But there is a some strange behaviour. All standard game resources are lower cased but it doesn't seem to work for mod.
worldtracker_expansion1.lua <- opening/reading error
WorldTracker_Expansion1.lua <- works
minimappanel.lua <- works
MinimapPanel.lua <- opening/reading error
And worse, xml-replacements don't ever log opening/reading error.
As far as can see, game doesn't care about case in *.modinfo.
What i did for now: i made a script that makes symlinks for every mod resource (lower case, several variants of Camel notation). And it does work somehow.
Currently i'm trying to fix buildings/units purchase in city panel.

Ok. Toggle great works window was bind to "w" by default. And i used WASD to control the screen. So there is no bug.

Can you share your script please ? I'd like to see if it's the same issue on my end

https://github.com/azorej/CQUI_Community-Edition/tree/linux_fix
I made a script for convenience:
curl https://raw.githubusercontent.com/azorej/CQUI_Community-Edition/linux_fix/checkout_mod.sh | bash -s
You can pass custom path for mod files to it
curl https://raw.githubusercontent.com/azorej/CQUI_Community-Edition/linux_fix/checkout_mod.sh | bash -s ~/games/other_mods
You will need python3 and python3-pip packages (you can install it via apt-get)
Please, unsubscribe or delete mod (if you used manual install) before running script!
What does the script do:

checks packages, paths
installs marisa-trie
clones fork
switchs to 'linux_fix' branch
applies 'fix' (hack actually)
creates a symlink: real mod path -> game data directory

There will be several errors with applying hack - i will fix them later (need to add a file blacklist)
if you encounter problems while installing mod or during game - please, let me know about it
(it would be good if you send lua.logs for more details)

Could you summarize, in a nut shell, what the actual fix is about? Maybe there is certain changes that can be applied and merged in this project to make it run univerally on Windows and Linux.

The actual diff between your fix and the expansion2 may also be helpful: https://github.com/Azurency/CQUI_Community-Edition/compare/expansion2...azorej:linux_fix.

Just tested the modification on my machine. Works absolutely fine.

The actual diff between your fix and the expansion2 may also be helpful: expansion2...azorej:linux_fix.

Current diff isn't very accurate: there are some needless changes. I'll fix it this weekend.
What actually i did: i made a script that make several symlinks for every mod resource.
I don't think there is a way to make the same steam mod run on Windows and Linux.
As far as i know, there is no way to run custom install script for mod installation on steam for Linux.
And imho, it's not a good idea to keep linux symlinks on Windows version.
What can we do: we can have two steam mods. One for Windows and one for Linux.
For linux version one must run my script before deploing mod to steam.
I'll try to make a pull request to main repository this weekend.

The "fix" for linux is to lowercase all the filenames I think (see https://github.com/CQUI-Org/cqui/issues/301)

The "fix" for linux is to lowercase all the filenames I think (see CQUI-Org/cqui#301)

I tried to do it and it didn't work for me:
modified file tree
Lua.log
There are a lot of errors like "Error opening/reading where=, file=C:\Emu\AppDataParent\Sid Meier's Civilization VI\Mods\CQUI_Community-Edition\Assets\UI\{filename}"


Group Proposal

Hey everyone, we have to do the group proposal, remember before doing any changes to the "proposal.ipynb" file, first thing: text us, second thing: PULL CHANGES, third thing: edit, fourth thing: PUSH CHANGES. And then tell us you finished editing.
Lets segregate our responsibilities:

Anson : intorduction (100 words) , methods (150 words)
Ziqi AND Simon: Preliminary exploratory data analysis (250 Words)
Aurona: Expected outcomes and significance (100 words)

Please finish all this ON OR BEFORE FRIDAY. I will modify your work on Saturday and upload it on Canvas.

Hi everyone, I have changed the predictive question since it would be better if the training data subset has around 75% of the data instead of 90% as initially planned. I have changed the prediction to 3 years (2020,2021, 2022) instead of only 2022.
The new confirmed predictive question uses 2013-2019 as basis for regression analysis that predicts 2020, 2021, 2022 Real GNI.
New training data: 2013-2019, new test data: 2021-2022

.


Nested REST APIs

Hey guys,
I really like your project. I see from the documentation and from testing out the project that it works great for top-level entities. However, I haven't been able to figure out a way to get it working with nested entities. Is it possible to access a REST entity with a path of /firstEntity//secondEntity/?
Thanks!

Hi spfjr,
We're relying on Restangular for that, and I'm not sure it allows to modify the URLs as you wish. I've found a related issue (https://github.com/mgonto/restangular/issues/310), let me know if it works for you.

Francois,
I actually saw that Restangular issue. Unfortunately, like the creator of that ticket, I don't think that would work for my needs. I already tried to modify the path in a fullRequestInterceptor() function, but I figured I would see if there was some other possible configuration before deciding whether or not I wanted to refactor my REST API.
Thank you for the quick response!

You don't plan on adding this feature?

If we need it for our own projects, we will. In the meantime, you're welcome to try and implement it, and submit a PR for it.

My angular/js skill is really low, that's also why this project was really appealing until i hit this wall.

The solution is described in #711


Error "/septentrio_gnss: gnss" after launching rover.launch

Hello guys,
When I launched septentrio_gnss_driver rover.launch param_file_name:=rover, I got the error: [ERROR] [1684219558.118107529]: /septentrio_gnss: gnss. The hardware and software setup are described below:

Sensor: mosaic-go heading GNSS module evaluation kit .
Sensor firmware version: 4.12.1.
Port: USB.
ROS version: Noetic.

Additional notes:
I have tried to build the package from source and binary install, but both lead to the same error. In addition, I'm able to connect via Ethernet-over-USB IP address <IP_ADDRESS>. A screenshot is attached below:

After some minutes I got the message in the terminal: [ INFO] [1684220890.105052450]: /septentrio_gnss: TryParsing() method finished since it did not receive anything to parse for 10 seconds..
 The printed error is actually not an error, see here. This indicates that you are receiving data.
Somehow the connection seems to get lost after a few minutes. Are you still able to access the web interface or ping the device when this happens?

Hi @thomasemter
Thanks for your quick reply. You are right even if this message I'm able to echo the rostopic /gpsfix, for example, but I still get sometimes the error "/septentrio_gnss: TryParsing() method finished since it did not receive anything to parse for 10 seconds..". When that happens the messages are no longer published, however the web interface is accessible and updating the data.

Unfortunately, I do not know exactly where the problem might lie. Can you please provide a screenshot of this page, when it happens?

Also, we have rewritten the communication core and it is now able to reconnect in case the connections is interrupted. It is not released yet, but you might test the development branch, if you like.

Now it's mostly working, I couldn't reproduce the error again under the same conditions. I just got the error when I tried to launch the package without getting PVT. Below you can take a look at the requested page.

Thank you for the screenshot. It indeed seems like the connection (IP13 in this case) gets lost. Can you please try out the  development branch, if it's not too much trouble?

Sorry for the late reply. I was performing many tests on different days. So far my conclusion is the warning "/septentrio_gnss: TryParsing() method finished since it did not receive anything to parse for 10 seconds.." raises intermittently, especially when PVT solution is not provided. Maybe this happens because I'm using cheap antennas and the weather is not nice, mostly cloudy. I didn't get the warning using the dev branch, but I've tested fewer times. By the way, even in the dev branch I get the error message "/septentrio_gnss: gnss", but the messages are being published.

Okay, thank you so much.


Getting error Abort Trap: 6 when trying to Archive Swift app with Kingfisher.

Check List

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
Getting error Abort Trap: 6 when trying to Archive app with Kingfisher.
What
The issue just presented itself recently on Kingfisher 5.13.2, which had previously worked for us. This could be an issue as a result of a change in the Swift compiler.
We received this error message from the compiler:
/Kingfisher/Sources/Extensions/ImageView+Kingfisher.swift:407:10: Marking non-'@objc' Swift declaration 'dynamic' in library evolution mode is not supported
and
Found return instr that returns non-void in Function of void return type!
  ret void undef, !dbg !1574
 voidin function $s10Kingfisher12DownloadTaskV07WrappedC0O8downloadyAeCcAEmF
<unknown>:0: error: fatal error encountered during compilation; please file a bug report with your project and the crash log

Reproduce
Using Xcode 11.4, Archive a project that uses Kingfisher 5.13.2

Full stack trace:
{app}/Pods/Kingfisher/Sources/Image/ImageDrawing.swift:284:13: warning: initialization of immutable value 'rowBytes' was never used; consider replacing with assignment to '_' or removing it
        let rowBytes = Int(CGFloat(cgImage.bytesPerRow))
        ~~~~^~~~~~~~
        _
<unknown>:0: warning: module interfaces are only supported with Swift language version 5 or later (currently using -swift-version 4.2)
Found return instr that returns non-void in Function of void return type!
  ret void undef, !dbg !1574
 voidin function $s10Kingfisher12DownloadTaskV07WrappedC0O8downloadyAeCcAEmF
<unknown>:0: error: fatal error encountered during compilation; please file a bug report with your project and the crash log
<unknown>:0: note: Broken function found, compilation aborted!
Stack dump:
0.	Running pass 'Module Verifier' on function '@"$s10Kingfisher12DownloadTaskV07WrappedC0O8downloadyAeCcAEmF"'
0  swift                    0x000000010c0714ea PrintStackTraceSignalHandler(void*) + 42
1  swift                    0x000000010c070cc0 SignalHandler(int) + 352
2  libsystem_platform.dylib 0x00007fff69a145fd _sigtramp + 29
3  swift                    0x000000010ce51cd8 ErrorHandlerMutex + 0
4  libsystem_c.dylib        0x00007fff698ea808 abort + 120
5  swift                    0x0000000107dc5fc1 swift::performFrontend(llvm::ArrayRef<char const*>, char const*, void*, swift::FrontendObserver*)::$_1::__invoke(void*, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, bool) + 1137
6  swift                    0x000000010bfec5c6 llvm::report_fatal_error(llvm::Twine const&, bool) + 278
7  swift                    0x000000010bfec4ab llvm::report_fatal_error(char const*, bool) + 43
8  swift                    0x000000010bf6e423 (anonymous namespace)::VerifierLegacyPass::runOnFunction(llvm::Function&) + 115
9  swift                    0x000000010bf0e7f9 llvm::FPPassManager::runOnFunction(llvm::Function&) + 1609
10 swift                    0x000000010bf17e70 llvm::legacy::FunctionPassManagerImpl::run(llvm::Function&) + 112
11 swift                    0x000000010bf17d9d llvm::legacy::FunctionPassManager::run(llvm::Function&) + 365
12 swift                    0x0000000107fd1c58 swift::performLLVM(swift::IRGenOptions&, swift::DiagnosticEngine*, llvm::sys::SmartMutex<false>*, llvm::GlobalVariable*, llvm::Module*, llvm::TargetMachine*, swift::version::Version const&, llvm::StringRef, swift::UnifiedStatsReporter*) + 5816
13 swift                    0x0000000107fd6a50 (anonymous namespace)::LLVMCodeGenThreads::Thread::run() + 208
14 swift                    0x0000000107fd68f9 (anonymous namespace)::LLVMCodeGenThreads::runThread(void*) + 9
15 libsystem_pthread.dylib  0x00007fff69a20109 _pthread_start + 148
16 libsystem_pthread.dylib  0x00007fff69a1bb8b thread_start + 15
error: Abort trap: 6 (in target 'Kingfisher' from project 'Pods')

You may need to update to the latest version 5.13.4
#1426

Unfortunately after updating to 5.13.4, the same error occurred.

If you still have this warning:
ImageDrawing.swift:284:13: warning: initialization of immutable value 'rowBytes' was never used; consider replacing with assignment to '_' or removing it
let rowBytes = Int(CGFloat(cgImage.bytesPerRow))

It means you are still using an old version. This warning was already fixed in #1393 and version 5.13.1. The trap "Marking non-'@objc' Swift declaration 'dynamic' in library evolution mode" was also fixed in 5.13.3. It only happened when you enabled the library evolution mode and using Xcode 11.4.
I suggest you check again the Kingfisher version you are using.

Updating to 5.14 fixed this for me. Thanks!


Avast

I'm getting Avast antivirus detecting my script as false positive virus threat.
my config used to be:
      	rotateUnicodeArray: true
		,
	compact: true,
	controlFlowFlattening: false,
	deadCodeInjection: false,
	debugProtection: false,
	debugProtectionInterval: false,
	disableConsoleOutput: true,
	rotateStringArray: true,
	selfDefending: true,
	stringArray: true,
	stringArrayEncoding: 'base64',
	stringArrayThreshold: 0.75,
	unicodeEscapeSequence: false

I also tried "lowest" settings, but it still detected as false positive. Forgone the obfuscation altogether and it stopped complaining.

It's bad. In a few days i'll try to contact with Avast and describe this problem.
Can you attach working sample with obfuscated code that trigger Avast antivirus?
I'll attach this code to this page
https://www.avast.ru/false-positive-file-form.php

http://fsymbols.com/tell/main.fda15551.js

Mind that it is highly unlikely that it only has to do with Avast. I'm guessing that many more antiviruses see that code as a threat. It's just that I only tried on Avast.

How i can reproduce this on Mac? It's Avast extension for browser?
Can you try to run Uglify.js over obfuscated code and check with Avast again? Still detecting as virus threat?

Got it on your file with https://www.virustotal.com/ru/
Currently only one or two antiviruses detecting this code as threat.

Please try to run Uglify.js over obfuscated code or try latest beta with mangle: true option

Of course, the result isn't any different than it would be if you ran Uglify over that script. Ran it uglified on VirusTotal and got a
NANO-Antivirus	Trojan.Script.Agent.enqons
detection https://www.virustotal.com/ru/file/f31dfbb30d1536fff0b6f6701c705ea4b2111bfc7e79cb6bd587b93681f7f8bf/analysis/1494891276/

same here 😞

I should make research. if anyone have information - why few antiviruses mark obfuscated code as threat - i will very happy.

@sanex3339
Hey Timofey, have you contacted Avast?
Actually this issue is critical blocker for production use :/

Hi, no. But please try to use latest beta: 0.10.0-beta.8.
I removed all evals from obfuscated code, so maybe it won't detect your script as the virus threat.

Hi. Please try new 0.10.0 version. Avast still triggering on your code?

ping

My code's changed a bit, but if it still has the same features that brought up the warnings you can experiment converting http://fsymbols.com/tell/main.a51a389b.js at any time with settings you're interested about and checking with VT to see how the antiviruses react. No need for waiting.
If this code doesn't trigger errors even with an earlier obfuscatior, I'll try to find you an earlier version that may.

Hi. After some tests i noticed that Avast triggered when string literals are moving to the string array from objects keys that had computed property with false value.
Obfuscator transforming object keys:
var foo = {
    bar: 'baz'
};

into this form:
var foo = {
    'bar': 'baz'
};

And then moving this string literal 'bar' to the string array.
So, on your code after moving object keys to string array - avast will detect your code as the threat.
Solution - enable stringArrayEncoding: 'base64'
With this option enabled - all antivirus checks are passed on file that you sent in previous message. Tested on 0.10.2 version

Ping. Any news?

No news here. =)

Hey, sorry for long silence from my side. I haven't tested new release on production since we've refused obfuscation on production because of complaints from users about antivirus notification. This was critical for our product. Anyway I'll try to find a minute to test it on stage server in nearest days.

Please, check 0.13.0 version with hexadecimal and mangled values of identifierNamesGenerator option.

ping

Tested it now with command
javascript-obfuscator test.js --identifier-names-generator mangled --compact true --disable-console-output true --self-defending true --dead-code-injection true --control-flow-flattening true
No false positive alerts.
https://www.virustotal.com/ru/file/53af005143ac8cfd2eea82c0d05ba2ccf77d2ddd49182aa37632f42dfdec89c9/analysis/1514820105/
But enabling of stringArrayEncoding: base64 will trigger false positive alert.
So current status - if you will got this error - try to play with stringArrayEncoding option and disable it or change it on rc4.

Just added some clarification about antiviruses
https://github.com/javascript-obfuscator/javascript-obfuscator#antiviruses-false-positive-virus-alerts

Also you can attach all files with false positive here:
https://www.avast.ru/false-positive-file-form.php

Also i wrote letter to Avast about false positive alerts. Will waiting for response.

hi, please look, i try install this version 0.14.0 today but is not possible in npm.

Hi, please make separate issue and describe all your steps and errors (if they was)

I can't install it now:
$ npm install javascript-obfuscator

><EMAIL_ADDRESS>...

Avast 13.4 with virus definitions(18030400) steps in and puts in quarantive:
File: node_modules/.staging/javascript-obfuscator-f8cb28e3/dist/index.js
Infection: JS:Agent-EEEL [Trj]

Uploading the index.js file to virus total shows the same issue with Avast & AVG:
https://www.virustotal.com/#/file-analysis/YzdjMmI1ZTZjYmMyZTlhOWQzNTM3YmI3MGVlOGFmNDk6MTUyMDIzMjE1MA==

Interesting. Avast and AVG is triggering on this lines:
https://github.com/javascript-obfuscator/javascript-obfuscator/blob/master/src/templates/string-array-nodes/string-array-rotate-function-node/StringArrayRotateFunctionTemplate.ts#L6-L14
When i removed it, all checks are passed.
So, i'll think how to fix this.

@tech1e
Hello,

Thank you for reporting this false positive.

Our virus specialists have now cleared its reputation in our database.

With URLs this change should be instant, but it might take up to 24 hours with files.

For future reference you might also find the following article to be useful: Avast Clean Guidelines.

So, we should wait until tomorrow.

Please, check again. Looks like avast fixed false-positive alert.

Looks like no more problems with it. Reopen issue if you will have problems with avast.

Anyone runs into thisbissue this week?

@sanex3339  Yes for some reason Avast and Bitdefender detects the obfuscated code as malicious again.

Can you test with 0.16.0 version please?

Hi, for some reason the detection occurs when either mangled or hexadecimal options selected and a prefix is used , if you will use only hexadecimal no detection will occur.

Is this false positive ocurred if output file looks like this:

convert to base64
use something like var code= atob("xxxxx"); eval(code)

This is still a problem in ES3 where var is being used.
If replaced by const/let it is not flagged as a false positive but then it doesn't work in ie. windows JScript engine that requires ES3.

@cliqer is right , i've made a program based on mshta and i'm fasing the same problem


Re-Enable changelog verification

Add back changes removed in https://github.com/Azure/azure-sdk-for-python/pull/12359/files since changelog logic has been updated https://github.com/Azure/azure-sdk-tools/pull/755

eng/common changes merged into repo with https://github.com/Azure/azure-sdk-for-python/pull/12386.

Hey @chidozieononiwu ,
I'm disabling this again. The check is definitely doing something insane. We're releasing 12.4.0 of azure-storage-blob, and it's rejecting because 12.4.0-b1 is what it's looking for. I need to unblock releases. Will see what I can do to fix it regardless.

Hey @chidozieononiwu ,
My bad for missing this again. I'm disabling this step. The check is definitely doing something insane. We're releasing 12.4.0b1 of azure-storage-blob, and it's rejecting because 12.4.0-b1 is what it's looking for. I need to unblock releases. Will see what I can do to fix it regardless.
I think we should block on re-enabling this until we see a successful release with it.

Hey @scbedd  sorry again about this. This bug is from the pkgpprops tool which makes use of the AzureEngSemanticVersion  class. Unfortunately that class is dotnet biased as shown in the ToString() method which formats the version to dotnet style. I opened an issue to address this but its probably best to wait till we settle on a standard way of naming preview version across languages. For now I have added changes to the pkgprops tool to bypass the need to call the ToString method of the AzureEngSemanticVersion  class.


GrarphRBAC 1.42 not yet supported?

Hi,
I get a
{"odata.error":{"code":"Request_InvalidDataContractVersion","message":{"lang":"en","value":"The specified api-version is invalid. The value must exactly match a supported version."}}}

error when I tried to use the GraphRBAC 1.42 Swagger.
Has this API version disappeared?

@lmazuel - Looks like. Try using 1.6-internal.

Was finally an Python Autorest bug, sorry for the noise...


Remove older update exes of PowerToys in the AppData Local folder automatically after sometime has passed

Description of the new feature / enhancement
Some users have limited disk space so it would be nice to see this functionality implemented where older update exe files are removed from C:\Users\UserName\AppData\Local\Microsoft\PowerToys\Updates.

Scenario when this would be used?
Would be helpful to users who have limited disk space and it should run an auto cleanup once a week and check to see if the update exe file was there for more than 3 months ago for example.
Supporting information
No response

this should happen.  /reportbug

Yup, was added in 0.62
based on this screenshot, i'm betting you're on .61

Although I only have the 0.69 installation file on my computer, I still wonder if it can be deleted automatically after the update is complete, or moved to a temp folder

Yup, was added in 0.62
based on this screenshot, i'm betting you're on .61

Ok, that makes sense, and yes, I am still on v0.6.1. I guess that means it's time for me to update. :) Thanks!

Although I only have the 0.69 installation file on my computer, I still wonder if it can be deleted automatically after the update is complete, or moved to windows temp folder

Might make sense if you have an extreme shortage of space but maybe it doesn't hurt to keep the update exe around in case you have issues and need to reinstall again or something.

You're right, as long as you can find the installer or when the program needs to be fixed, the program supports doing it, lol

Although I only have the 0.69 installation file on my computer, I still wonder if it can be deleted automatically after the update is complete, or moved to windows temp folder

Might make sense if you have an extreme shortage of space but maybe it doesn't hurt to keep the update exe around in case you have issues and need to reinstall again or something.
You're right, as long as you can find the installer or when the program needs to be fixed, the program supports doing it, lol


Migrated shared examples from sorcery to sorcery-rework

Migrated the old shared tests from Sorcery v0.x to the Sorcery rework project.

Thanks Josh. Will take a look!


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ACCESS-TOKEN?

Hi,
I'm going to try the carbone-sdk-python in my server, but I'm confused about the part "ACCESS-TOKEN".
Suppose I'm using Cabone in my server, why I still have to set up the ACCESS-TOKEN?

I made a mistake, I thought I can use Carbone in my python project. Sorry, I close the issue.


Restore Sollet Wallet

Hi , I am trying to restore my sollet wallet with the seed phrase, When attempting to restore it I can see the value of SOL I have in the wallet however when I actually go to restore it there is no SOL and the funds of Step and USDC I had are also not in there? Please advise.

A friend of mine is experiencing a similar issue with restoring from a seed phrase!

Please try all derivation paths in the "restore" page. If you want to verify this is not a sollet bug, take the mnemonic used and derive keys in a different wallet with the same derivation path. The keys will be the same. If not, please re-open this issue. Please join Discord for more immediate support.


web GUI table column resizing

Enable column width to be re-sizable. Should be simple and easy to do, just some Javascript settings

disabled the <-||-> icons on the columns. There some weirdness with the Grid library that columns are unable to be resized.

change the font size and problem shows

seems slickgrid didnt like a function in the version of jquery. caused the resize to not work. updated so it works with the prevered draggable jquery approach and all is good
fixed


Ansible is unable to load correctly in WSL+venv config

Summary
When I open a repository using WSL on Windows, everything works except for the extension. See error message below.

It seems to load the virtual environment correctly, but then extension issues commands without /bin folder, which ofcourse causes a 'missing file' error.
Extension version
0.14.71
VS Code version
1.71.2
Ansible Version
(ansible) maciej@wsl:/mnt/c/DevOps/ip-ansible-servicenow$ ansible --version
ansible [core 2.13.4]
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/home/maciej/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/maciej/.venv/ansible/lib/python3.8/site-packages/ansible
  ansible collection location = /home/maciej/.ansible/collections:/usr/share/ansible/collections
  executable location = /home/maciej/.venv/ansible/bin/ansible
  python version = 3.8.10 (default, Jun 22 2022, 20:18:18) [GCC 9.4.0]
  jinja version = 3.1.2
  libyaml = True

OS / Environment
Windows 10
WSL Ubuntu 20.04
ansible-lint 6.7.0 using ansible 2.13.4
(ansible) maciej@wsl:/mnt/c/DevOps/ip-ansible-servicenow$ which ansible-config
/home/maciej/.venv/ansible/bin/ansible-config
(ansible) maciej@wsl:/mnt/c/DevOps/ip-ansible-servicenow$ ansible-config --version
ansible-config [core 2.13.4]
config file = /etc/ansible/ansible.cfg
configured module search path = ['/home/maciej/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
ansible python module location = /home/maciej/.venv/ansible/lib/python3.8/site-packages/ansible
ansible collection location = /home/maciej/.ansible/collections:/usr/share/ansible/collections
executable location = /home/maciej/.venv/ansible/bin/ansible-config
python version = 3.8.10 (default, Jun 22 2022, 20:18:18) [GCC 9.4.0]
jinja version = 3.1.2
libyaml = True

Relevant log output
No response

Hi, no idea if this helps you, but I had a similar issue yesterday. I am using a python virtual environment as well – but on native Ubuntu, not on WSL. What helped in my case was setting ./.venv/bin/python3 as Interpreter Path in the workspace settings (which imho should be automatically set without the need of manual intervention).

I'm not sure if that will fix your issue, but I had to set the settings in the actual project I was working on. (Not in the settings file which is on windows.)
// at the root of the project
.vscode/
  settings.json
{
  ...
  "ansible.python.interpreterPath": "/pathtoyourproject/venv/bin/python3",
  "ansible.ansible.path": "/pathtoyourproject/venv/bin/ansible",
}

The fix to this is to add appropriate paths in the settings as mentioned by @AppeninTerenceBARBE above.
If using virtual environments, you can simply add this setting, and it should work:
ansible.python.activationScript: path/to/venv/activate-script

This will be used to run commands in a Python virtual environment


Why is it "required" to let Vundle manage itself?

The main ReadMe states:
" let Vundle manage Vundle, required
Plugin 'VundleVim/Vundle.vim'

Why exactly is this "required"? What are the potential consequences of not having that line in the vimrc?
I'm asking because in my case Vundle.vim is a submodule of my dotfile Git repo, so Vundle throws an error when it tries to update itself (because its local repo isn't a normal checkout of the master branch).
I'd rather just update Vundle manually and only have it update the other bundles automatically.
Looking at https://github.com/VundleVim/Vundle.vim/issues/565, it seems like :PluginClean might try to remove the Vundle directory when it isn't explicitly mentioned in the vimrc. I could live with that (I don't normally have a use for :PluginClean anyway), but is there anything else I should be aware of?

Not totally sure how "required" this actually is. For a while I was using my own custom build of Vundle and saw no issues. By default, you won't get the help files (i.e. :help vundle). You can fix this with an appropriate :helptags command.
That's just my opinion. If that's a sufficient answer for you, feel free to close this issue.

You are right it seems to be at least for the case of the clean command.
See the change here: https://github.com/VundleVim/Vundle.vim/commit/6af116c76fcff8a9c5ed633540eeb14f9d85bb97
Comes shortly after this issue: https://github.com/VundleVim/Vundle.vim/issues/52

Added a FAQ to the wiki because this didn't seem to go with the index wiki page as this is not a problem as much as just a question, page here; https://github.com/VundleVim/Vundle.vim/wiki/FAQ
Closing this for now :smile:


Cannot add extended annotations to resources

rdl.rdl doesn't define "annotations" for resources or resource input/outputs.

Fixed in commit 000ec28e9c4a3ddbce3c63e4ec3c6e685dc8a2cc


Bugfix/home page typo

Please approve

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


Auto refresh browser

Hi all,
I'm using dotnet watch with my asp.net core MVC app and I have it working so that it automatically builds when I save a file. Is there a way to automatically refresh the browser so I can see the results without having to manually click F5?.
Thanks

Hi @irashid25,
There's no way to auto refresh the browser. dotnet-watch is not web specific, thus it doesn't know anything about the environment in which the app runs. We might consider extensions in the future, but right now there's no way to do that.

Thanks for the quick reply victor.
I think I can get it done by using gulp and the visual studio task runner. I could set off a gulp task to reload the browser using browser sync etc after the project has been built.
Will try it tonight.
Thanks


Terra crashes modpack.

crash-2021-05-04_00.58.24-client.txt
Fabric version: 0.11.3
Minecraft version: 1.16.5
Mod version: 5.2.1 (latest)

Remove half the mods and try again. Then do it with the other half of the mods. Once you find which half crashes, halve that half and do it again. Keep doing this until you found out which mod crashes.