Instructions on how to start the frontend with FFT waterfall

When I run cargo run --example soapy --features="soapy" it starts okay, but when I visit http://<IP_ADDRESS>:1337/ I get 404: Not Found. Are there more steps needed to start the frontend with the FFT waterfall? Running make in frontend subdirectory succeeds.

If the frontend is built (with make and ./gulp, as explained). It should be served by the normal Rocket server that also exposes control port. (Not really intuitive and not documented. Sorry.) That means if you run the example with the command you mentioned (cargo run --example soapy --features="soapy") and enabled control port, i.e., in config.toml set
ctrlport_enable = true
ctrlport_bind = "<IP_ADDRESS>:1337"

the GUI should be available at <IP_ADDRESS>:1337.
If this doesn't work, can you please provide a directory listing of frontend/dist?

Okay, now it works. It's not that stable, sometimes hangs on DEBUG - wait for blocks init and waterfall stops updating, but when it works, it works :)

Thanks for the heads-up! I'll close the issue, for now. If this deadlock/stall during startup turns out to be a bug. Please feel free to open a separate issue.


Code Example: SRI Immunization Model

When running the sri.model.animate(21) function, I get an index error. Any help is appreciated.

@senayyitbarek Thanks for reporting this!
I've started a PR (referenced above) to make fixes. The main things I had to do was use:
            path_data = [list(el[()]) for el in paths]
            # Create network path and node Elements
            network_paths = hv.Path(path_data, key_dimensions=spatial_dims)
            network_nodes = hv.Points(np.hstack([points, state_array]),
                                      key_dimensions=spatial_dims,
                                      value_dimensions=['State'])

            # Create overlay and accumulate in network HoloMap
            network_hmap[i] = (network_paths * network_nodes * nlabel).relabel(group='Network', label='SRI')
            self.step()

        # Create Overlay of Curves
        curves = hv.NdOverlay({label: hv.Curve(zip(range(steps), sird[:, i]),
                                               kdims=['Time'], vdims=['Count'])

In the animate loop.
The main changes are: the data format supplied to Path was invalid and key_dimensions and value_dimensions in the NdOverlay had to be changed to kdims and vdims.

This has been fixed a while ago and further discussion about topics notebooks should happen in: https://github.com/ioam/holoviews/issues/1638


python311Packages.crytic-compile: 0.3.3 -> 0.3.4

Description of changes
python311Packages.crytic-compile: 0.3.3 -> 0.3.4

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 247944 run on x86_64-linux 1

9 packages built:

echidna
python310Packages.crytic-compile
python310Packages.crytic-compile.dist
slither-analyzer (python310Packages.slither-analyzer)
slither-analyzer.dist (python310Packages.slither-analyzer.dist)
python311Packages.crytic-compile
python311Packages.crytic-compile.dist
python311Packages.slither-analyzer
python311Packages.slither-analyzer.dist

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2514

Result of nixpkgs-review pr 247944 run on x86_64-linux 1

9 packages failed to build:

echidna
python310Packages.crytic-compile
python310Packages.crytic-compile.dist
slither-analyzer (python310Packages.slither-analyzer)
slither-analyzer.dist (python310Packages.slither-analyzer.dist)
python311Packages.crytic-compile
python311Packages.crytic-compile.dist
python311Packages.slither-analyzer
python311Packages.slither-analyzer.dist

There is a hash mismatch in one of the dependency's

There is a hash mismatch in one of the dependency's

Strange, I ran the same thing. See my previous post. I will try again.
Any more details you could share?

There is a hash mismatch in one of the dependency's

I have just rebased this PR, and did it again:
Result of nixpkgs-review pr 247944 run on x86_64-linux 1

9 packages built:

echidna
python310Packages.crytic-compile
python310Packages.crytic-compile.dist
slither-analyzer (python310Packages.slither-analyzer)
slither-analyzer.dist (python310Packages.slither-analyzer.dist)
python311Packages.crytic-compile
python311Packages.crytic-compile.dist
python311Packages.slither-analyzer
python311Packages.slither-analyzer.dist

Everything works, so please provide more details on your end. Meanwhile let me ask:
@NixOS/ofborg  build echidna python310Packages.crytic-compile python311Packages.crytic-compile python311Packages.slither-analyzer

@NixOS/ofborg  build echidna python310Packages.crytic-compile python311Packages.crytic-compile python311Packages.slither-analyzer

@ofborg build echidna python310Packages.crytic-compile python311Packages.crytic-compile python311Packages.slither-analyzer

Hash mismatch, please update it.

done.
I guess if upstream change their artifact with the same rev reference, the nixpkgs cache would unfortunately not detect this error?


proxy doesn't support client_id

Many MQTT brokers use a client_id parameter to differentiate users , the MQTT::Proxy class doesn't seem to allow this.  The connection is made in quite a different way it seems but I was hoping that I could chuck in the same parameters as MQTT::Client which does allow me to add a :client_id .
Happy to help create the code and help test (I'm a professional software tester) , however I'm fairly new to ruby and mqtt so others may well be quicker and better at this.
Thanks for sharing your work.

Hi,
What makes you think that the client_id is not passed through?
The Proxy does not alter the MQTT packets - it just passes them straight through, with the opportunity to inspect/mutate them if you want to.
I have not used the proxy code in quite a long time and there was a bug in the logging code that I have now fixed. However I was able to verify that the client ID is passed through unmodified.
nick.

ah you are right , it does work , it seems my problem was that there was an issue with a missing method called type in MQTT::Packet , I fixed it making type call type_id and I'm seeing packets go back and forth now . Just sent you a pull request with the fix .

Ok, glad you got this working.
I fixed the MQTT::Packet#type problem in 85a802c4cb0e812de8fa86e0bbbd68948cd0dff7


return with error code on command failure

While looking into a failure on appveyor [1], we noticed command failed: fork/exec c:\go\bin\go.exe: invalid argument, but appveyor failed to pick up the failure.
This patch quits on command failure using log.Fatal(), which signals an exit code to the OS so it can be picked up by appveyor.
The root cause of the failing command is still under investigation
1: https://ci.appveyor.com/project/martinlindhe/wmi-exporter/build/170

circleci failure is because of some docker provisioning issue, unrelated to this PR...

4170aa450174: Error pulling dependent layers
1ed04702e97b: Error pulling image (1.7.1-base) from quay.io/prometheus/golang-builder, endpoint: https://quay.io/v1/, Server error: Status 500 while fetching image layer (4170aa450174c29ef6abfab86a70501c18387b7dd7826a5f8732a380ce20947c)
1ed04702e97b: Error pulling image (1.7.1-base) from quay.io/prometheus/golang-builder, Server error: Status 500 while fetching image layer (4170aa450174c29ef6abfab86a70501c18387b7dd7826a5f8732a380ce20947c)
Error pulling image (1.7.1-base) from quay.io/prometheus/golang-builder, Server error: Status 500 while fetching image layer (4170aa450174c29ef6abfab86a70501c18387b7dd7826a5f8732a380ce20947c)
!! The base builder docker image exited unexpectedly: [1] Error pulling image (1.7.1-base) from quay.io/prometheus/golang-builder, Server error: Status 500 while fetching image layer (4170aa450174c29ef6abfab86a70501c18387b7dd7826a5f8732a380ce20947c)

promu crossbuild returned exit code 1

Action failed: promu crossbuild

https://circleci.com/gh/prometheus/promu/53?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Thank you !


Guide: Link to advanced app documentation is wrong/inconsistent

The link of 'Advanced Project Template' in 'Special Topics' is wrong. Should it go to http://www.yiiframework.com/doc-2.0/guide-tutorial-advanced-app.html or to https://github.com/yiisoft/yii2-app-advanced/blob/master/docs/guide/README.md? I would have expect the first link. Currently the link in the navigation and on the start page don't go the URL...

Second link is correct.

Then the navigation link isn't correct. It leads to the Guides main page: href="./guide-README.html".
Sorry for bothering you with this tiny issue ;-) Yii is great anyway!

It's already fixed in code. Docs will be re-generated at some point: https://github.com/yiisoft/yii2/blob/master/docs/guide/README.md#special-topics


Create SVG basic shapes using d3.js

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hey @vkk2420
It seems that similar changes similar to the ones on this PR are on PR # 33674.  We are closing this PR in favor of the other one.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Input bottom borders missing when ion-item inside ion-row

Short description of the problem:
The usual bottom borders are missing from the inputs when the ion-item is nested inside the ion-row
What behavior are you expecting?
Bottom border should be present
e.g

  <ion-item-group>
    <ion-item-divider>Some group:</ion-item-divider>
    <ion-row>
      <ion-col width-80>
        <ion-item>
          <ion-label stacked>field 1:</ion-label>
          <ion-input></ion-input>
        </ion-item>
      </ion-col>
      <ion-col width-10>
        <ion-icon danger ios="ios-sad-outline" md="md-sad" item-right></ion-icon>
      </ion-col>
      <ion-col width-10>
        <ion-icon primary ios="ios-flag" md="ios-flag" item-right></ion-icon>
      </ion-col>
    </ion-row>
  </ion-item-group>

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Which Ionic Version? 1.x or 2.x
2.x
Plunker that shows an example of your issue
http://plnkr.co/edit/dcnJTSPxVtdiFK9IHTno?p=preview
Run ionic info from terminal/cmd prompt: (paste output below)
Davids-MacBook-Air:AppHazard daveshirman$ ionic info
Your system information:
Cordova CLI: 6.2.0
Ionic Framework Version: 2.0.0-beta.10
Ionic CLI Version: 2.0.0-beta.32
Ionic App Lib Version: 2.0.0-beta.18
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v4.3.0
Xcode version: Xcode 7.2.1 Build version 7C1002

Hello, thanks for opening an issue with us! May I ask what the use case is for using <ion-row /> here? Thanks for using Ionic!

Because I'm using the grid components with rows and columns to align
buttons to the right of each input row.
Like a secondary action container related to the input field in that row.
On 1 Aug 2016 3:20 p.m., "Justin Willis"<EMAIL_ADDRESS>wrote:

Hello, thanks for opening an issue with us! May I ask what the use case is
for using  here? Thanks for using Ionic!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic/issues/7503#issuecomment-236593680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANnlXuPCoIZBKfDugQlu6Z0bI9HzEaJNks5qbgCUgaJpZM4JZlQN
.

Hey @jgw96, is there any update on this please?
Thanks.

Hey @daveshirman, so the issue here is that the ion-item-group looks for the .item:last-child and removes the border when it finds it. Since you are nesting the ion-item they are all falling into this use case and therefore the border is removed. If this rule were modified to only look for descendants then your app would look like the following:

Is this what you are trying to achieve? Is there a reason you aren't using the ion-item with item-right on the icons and then modifying them that way? Something like this:

Where the icon could be styled to take up a specific width.

Hi @brandyscarney, thanks for your reply.
First of all, in the example you show, the icons are aligned differently, i.e. lower.
secondly, the input goes all the way to the end.
This is undesirable for several reasons:

long text will flow past under the buttons.  Bad.
ion-select controls (instead of a standard text input) make the right 2 buttons impossible to tap

Hence why I did the row and column thing, which reliably separates out the inputs from the buttons which have their own actions.
Does that make sense?

Thanks for the quick reply, @daveshirman! Yes, that makes sense. The issue is it is hard for us to cover every possible use case. ion-item-group is really supposed to be used to group items with (optionally) sticky dividers, or take advantage of list reordering. Are you using ion-item-group for anything like this? Removing the ion-item-group wrapper would solve your problem immediately, but we can take a look into this some more.

Hey @brandyscarney, Yes I am using ion-item-group on a another page to facilitate list reordering in fact, e.g. a floor list (group) with items on the floor (to reorder).
I have changed the ion-item-group in this instance to a plain <div> and the problem is solved as you suggested. Thanks.
I know it must be difficult to catch all the use cases.  But I think organising a form like this is not an uncommon thing. E.g a barcode scanner icon next to an input field maybe. I can't be the only person to have done this?!
It's worth noting that using the ion-item-group in this case seems reasonable as the form contains lists of inputs for different logical groupings, see attached screenshot.

Thanks again for your assistance.


neo-tree git icon miss

Hello, I really like the theme of arctic.nvim. Currently, I am using Astronvim, but the file change icon in the neo tree sidebar is not displaying properly. How can I solve this problem? Thank you.
arctic.nvim：

astronvim:

By the way, I think the theme effect is excellent when norm_bg is # 161616 (in MacBook Pro)

Very glad you like it.

For all the icons in Neovim, they are not controlled by Neovim but the terminal itself. You should set and use a font patched with nerdfont (https://www.nerdfonts.com/) in your terminal config.

I am not using neo-tree.nvim and I didn't define its highlight groups in this theme. If you are not satisfied with its default highlight groups, you can fork this repo and define them in this file arctic.nvim/lua/lush_theme/arctic.lua like this way

NeoTreeGitAdded { fg = '#xxxxxx' }, -- give it a color explicitly
NeoTreeGitDeleted { DiffDelete }, -- or link a new group to another

All the highlight groups that neo-tree support can be found here: https://github.com/nvim-neo-tree/neo-tree.nvim/blob/main/doc/neo-tree.txt#L1308

I think the theme effect is excellent when norm_bg is # 161616
This theme is ported from the "Dark Modern" in VSCode so the intention is to be as consistent as possible with VSCode. That's why I use #1f1f1f as the normal background. If you prefer another color, you could fork it and adjust it by yourself.


Update debug-pod-replication-controller.md

Fix dead link to "Debug Running Pods".

Duplicate of #20026
/close


Feat/integrate icon network

📝 Description
Integrate ICON network to ledger-live including features

Send
Sync
Receive
Stake (Freeze)
Unstake (Unfreeze)
Vote
Claim Reward

❓ Context

JIRA or GitHub link:

✅ Checklist
Pull Requests must pass the CI and be code reviewed. Set as Draft if the PR is not ready.

[ ] npx changeset was attached.
[ ] Covered by automatic tests.

[ ] Impact of the changes:

🧐 Checklist for the PR Reviewers

[ ] The code aligns with the requirements described in the linked JIRA or GitHub issue.
[ ] The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
[ ] There are no undocumented trade-offs, technical debt, or maintainability issues.
[ ] The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
[ ] Any new dependencies have been justified and documented.
[ ] Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

Hey @haammar-ledger, most of your comments have been fixed and pushed in this PR. However, I'm currently stuck on a remaining issue. Could you please help me out?

Ref: https://github.com/LedgerHQ/ledger-live/pull/1677/files#r1422173927

Hey @haammar-ledger, most of your comments have been fixed and pushed in this PR. However, I'm currently stuck on a remaining issue. Could you please help me out?

Ref: https://github.com/LedgerHQ/ledger-live/pull/1677/files#r1422173927

Replaced with https://github.com/LedgerHQ/ledger-live/pull/5809


Configuring via UI is hard.

First off, thanks so much for this integration, an absolute game-changer for our home!
I have a 16-bulb light, and I use lightener to turn them on one by one before starting to increase brightness after they are all on. I do this because the lowest brightness with all on is too bright most of the time. Each light is configured with 3 - 5 mapping lines.
If I configure this in the UI, I only see one light at a time, out of 16, and have to pay close attention that I'm adding the correct mapping to the correct light, and have to actually lay it out fully in text beforehand, since I have no overview in the UI config. There's also no back button, so if I make a mistake, or have to double check something, I have to start over.
Any chance you can do the light mapping on one screen instead?

The initial design for the configuration flow aimed to enable users to add, remove, and configure lights all within a single view, using a sequence of text fields for all selected lights.
Unfortunately, the Home Assistant config flow system has limitations, specifically the inability to dynamically create fields in flow steps, which restricts us from implementing such a UI configuration at this time.
A feasible workaround is to manually edit the /config/.storage/core.config_entries file. It appears that restarting HA is not necessary after changes; reloading the specific Lightener config on the integration page should suffice.
While this is not ideal, given the current limitations within HA and the existence of a reasonable workaround, I'm going to close this issue. However, I'm open to revisiting and reopening it if a more viable and implementable solution be proposed in the future.

Thanks for the detailed response! Editing the config_entries file is good solution for now.


Fix README, remove node_modules and add .gitignore

Hi Marc,
Please, merge and publish in npmjs.
Best regards!

Done - thanks again Matheus!


Add VPP config for ARM

Description
Add vpp config for ARM. The config will be the same as in this PR - https://github.com/networkservicemesh/cmd-forwarder-vpp/pull/905

Seems like done


⚠ Bofrost.at has degraded performance

In bc45847, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1570 ms

Resolved: Bofrost.at performance has improved in 04c1cff after 6 minutes.


Python 3.11 support

splinecalib is failing to build for py311 on the conda-forge CI.

splinecalib/loss_fun_c.c:198:12: fatal error: longintrepr.h: No such file or directory

note the base library needs to regenerate its c sources: https://github.com/numeristical/splinecalib/blob/main/splinecalib/loss_fun_c.c#L198

Originally posted by @timkpaine in https://github.com/conda-forge/splinecalib-feedstock/issues/6#issuecomment-1312545617

OK, let me look into this.  Right now I leave the C files static (which I thought would be more stable).  But maybe I should change it so that it cythonizes the pyx as part of the build.  Let me know if you have any insight into this. :)

This seems to be a common bug with Python 3.11 - apparently related to some incompatibility between Cython and Python 3.11:  e.g.:
https://stackoverflow.com/questions/74979674/gensim-install-in-python-3-11-fails-because-of-missing-longintrepr-h-file
https://github.com/scipy/scipy/issues/16262
For now I would suggest just using Python 3.10 or seeing if the latest version of Cython (or Python 3.11) fixes the issue

@philip-khor This should be resolved with splinecalib 0.0.6 - please let me know if it still fails.

@blucena I am still unable to build splinecalib on Python 3.11. Any potential fixes or updates?

@bhyatt25 I was able to install v0.0.7 with Python 3.11 on Mac.  Can you tell me the platform you are using and the error you get?

@blucena I am using Windows 11. I've tried just installing splinecalib and forcing version 0.0.7 and I am receiving the below error:
Building wheels for collected packages: splinecalib
Building wheel for splinecalib (pyproject.toml) ... error
error: subprocess-exited-with-error
× Building wheel for splinecalib (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [12 lines of output]
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-cpython-311
creating build\lib.win-amd64-cpython-311\splinecalib
copying splinecalib\calib_utils.py -> build\lib.win-amd64-cpython-311\splinecalib
copying splinecalib\splinecalib.py -> build\lib.win-amd64-cpython-311\splinecalib
copying splinecalib_init_.py -> build\lib.win-amd64-cpython-311\splinecalib
running build_ext
building 'loss_fun_c' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for splinecalib
Failed to build splinecalib
ERROR: Could not build wheels for splinecalib, which is required to install pyproject.toml-based projects

yeah - so the issue is that I am not providing pre-compiled wheels, so you need to have a C-compiler installed.  Let me look into how I can include some pre-compiled wheels.

@blucena assuming you were able to implement the pre-compiled wheels because I was just able to install the splinecalib package on my machine. Thank you!

Wish I could take credit, but I didn't change anything.


Unable to install Tweetinvi Nuget Package

i have visual studio 2013 professional i want to update my tweetinvi package from tweetinvi 1.0.0 to latest version but it gives me error every time. here is screenshot of error

The image is not available. Could you reupload the image and also send me the log.
Thanks

This has probably been solved by version 1.2


Performance and scale tuning for release 0.7.0

There have been many features developed and since performance is of much importance to us, we need to be make sure performance remains optimal.

With 3-stage ebpf processing, we are able to  do the following
Non-tunnel traffic
# Before
[  4]  0.0-10.0 sec  31.9 GBytes  27.4 Gbits/sec

# After
[  4]  0.0-10.0 sec  34.8 GBytes  29.9 Gbits/sec

Tunnel traffic
# Before
[  4]  0.0-10.0 sec  27.4 GBytes  23.1 Gbits/sec

# After
[  4]  0.0-10.0 sec  29.5 GBytes  25.3 Gbits/sec

This includes all stateful processing and clearly provides us benefits. And the benefits outweigh the complexity involved


Added route protection

Issue Fixed : #24
Added route protection
Issue
User was able to manually route to the editor page without adding an audio file
Fixes
Added route protection such that if the user tries to manually route to the editor page without adding a audio file, they will be redirected back to the home page and be prompted to add the audio file back
  {fileURL ? <Route path="/edit" component={EditPage} exact /> : <Redirect push to="/" path="/"/>}

Demo:
Route protection.webm

LGTM!


Fix indentation of dnsConfig for certmanager deployment

This is a fix for: #14656
Indentation of dnsConfig in certmanager helm deployment is incorrect. This is a simple fix to put the correct indentation in.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: efossas
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ostromart
If they are not already assigned, you can assign the PR to them by writing /assign @ostromart in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

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

Hi @efossas. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.

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

Needs approval from an approver in each of these files:

install/OWNERS [howardjohn]

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

/ok-to-test

https://github.com/istio/istio/pull/13834 just got merged in with the same change, thank you for your contribution though!

I signed it!


gettig empty object when i return console log req.body. in my post endpoint

Can you leave a more detailed issue? Are you getting a response? Is your server running correctly?


There is no response to issues on this repository

Tagging @scanlonp because that's the only AWS related account I can see contributing to this repository.
There has been no response to issues I opened against this repository a year ago: https://github.com/cdklabs/cdk-docker-image-deployment/issues/58 and https://github.com/cdklabs/cdk-docker-image-deployment/issues/56. Please prioritise responding to them.
Alternatively, if AWS no longer responds to issues on this repository, turn off Issues in the repository settings so we don't waste our time on issues that will never be actioned.
It's disrespectful of our time and the contribution we're trying to make to the ecosystem when we're allowed to put work into issues that will never be actioned.

Hey @SamStephens, completely understand your frustration. This repository is essentially my responsibility to maintain, and it has slipped in my priorities recently. Also, though it does not change that your issues have gone unaddressed, I was not with AWS for a long stretch between this construct's authoring and now.
I have been trying to carve out time to address the issues here, though it will admittedly not be my main focus.
I have seen the issues you have posted; this construct not working for non-TS languages is a massive bug. I know that the issue is a problem with this construct's integration with JSII, but I have not tracked down the fix yet. I can leave comments on each of them and try to get timelines for addressing them. However, it will not be an aggressive schedule.

@scanlonp thanks for responding, and thanks for your honesty.
Feel free to resolve this issue.


Refs #31300 -- Add support for function-based virtual fields on Oracle

Builds on top of #16417.

@jpnauta Here's the first draft of Oracle support.

buildbot, test on oracle.

I'm not sure why the jenkins builds aren't running properly. @felixxm any ideas?

I'm not sure why the jenkins builds aren't running properly. @felixxm any ideas?

This might have coincided with the degraded performance/incident of GitHub actions on the 16th May (see here). Push up a small change (can reword a commit) and see if this fixes it? I can't see this PR in the list of stuff picked up by jenkins

I'm new here. I think by pushing a small change and monitoring Jenkins, you can assess if the issue persists or if the new commit triggers the build successfully. This can help determine if the previous problem was due to the degraded performance of GitHub Actions or if there might be other underlying issues with Jenkins configuration or integration.
This might can help.

That doesn't appear to have fixed things.

Found an error log: https://djangoci.com/view/PR Builders/job/pr-mariadb/lastFailedBuild/console
(don't understand what is needed to be done :thinking:)

I'd start by fixing conflicts and rebasing from the main branch.

I'd start by fixing conflicts and rebasing from the main branch.

I was hoping to avoid that because I didn't want to rebase @jpnauta's commits so this PR could cleanly land back on his.

buildbot, test on oracle.

buildbot, test on oracle.

@felixxm At DjangoCon Europe sprints we discussed the Oracle error we're getting here and I think we settled on updating the Jenkins install to set MAX_STRING_SIZE=EXTENDED. Do I need to open a ticket somewhere for that or is leaving this comment enough?

buildbot, test on oracle.

buildbot, test on oracle.

@felixxm I've removed the extra feature flag. The main blocker I think is https://github.com/django/django/pull/16860#issuecomment-1597520422, followed by fixing the merge conflicts.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

I'm starting to work on this patch and push final edits.

I tested this branch locally and the migration files generated for a model with a GeneratedField are broken, this is the patch to fix the problem.
diff --git a/django/db/models/fields/__init__.py b/django/db/models/fields/__init__.py
index 8a87bad69f..f1e4790568 100644
--- a/django/db/models/fields/__init__.py
+++ b/django/db/models/fields/__init__.py
@@ -647,6 +647,8 @@ class Field(RegisterLookupMixin):
             path = path.replace("django.db.models.fields.related", "django.db.models")
         elif path.startswith("django.db.models.fields.files"):
             path = path.replace("django.db.models.fields.files", "django.db.models")
+        elif path.startswith("django.db.models.fields.generated"):
+            path = path.replace("django.db.models.fields.generated", "django.db.models")
         elif path.startswith("django.db.models.fields.json"):
             path = path.replace("django.db.models.fields.json", "django.db.models")
         elif path.startswith("django.db.models.fields.proxy"):

I tested this branch locally and the migration files generated for a model with a GeneratedField are broken, this is the patch to fix the problem.
diff --git a/django/db/models/fields/__init__.py b/django/db/models/fields/__init__.py
index 8a87bad69f..f1e4790568 100644
--- a/django/db/models/fields/__init__.py
+++ b/django/db/models/fields/__init__.py
@@ -647,6 +647,8 @@ class Field(RegisterLookupMixin):
             path = path.replace("django.db.models.fields.related", "django.db.models")
         elif path.startswith("django.db.models.fields.files"):
             path = path.replace("django.db.models.fields.files", "django.db.models")
+        elif path.startswith("django.db.models.fields.generated"):
+            path = path.replace("django.db.models.fields.generated", "django.db.models")
         elif path.startswith("django.db.models.fields.json"):
             path = path.replace("django.db.models.fields.json", "django.db.models")
         elif path.startswith("django.db.models.fields.proxy"):

Thanks, fixed.

Working with this branch I created a small local project using Sqlite and I verified that saving an instance of a model containing a generated field raises an error unless you specify the fields to update but in this last case the generated field is not is updated.
Environment
>>> from django.db.backends.sqlite3.base import Database
>>> Database.sqlite_version_info
(3, 40, 1)
>>> Database.sqlite_version_info >= (3, 31, 0)
True
>>> import django
>>> django.__version__
'5.0.dev20230904040634'

Model
from django.db import models

class Square(models.Model):
    side = models.IntegerField()
    area = models.GeneratedField(expression=models.F("side") * models.F("side"), db_persist=True)

Queries with errors
>>> from geometricfigures.models import Square
>>> Square.objects.update(side=10)
2
>>> square = Square.objects.first()
>>> square.save()
Traceback (most recent call last):
  File "/home/paulox/Projects/django/django/db/backends/utils.py", line 101, in _execute
    return self.cursor.execute(sql, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/paulox/Projects/django/django/db/backends/sqlite3/base.py", line 328, in execute
    return super().execute(query, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
sqlite3.OperationalError: cannot UPDATE generated column "area"

Queries without errors
>>> from geometricfigures.models import Square
>>> Square.objects.update(side=2)
2
>>> square = Square.objects.first()
>>> square.side
2
>>> square.area
4
>>> square.side = 3
>>> square.save(update_fields=["side"])
>>> square.area
4

I think a test with this scenario is missing (I sent one in the code hints).
Using update instead everything works and it shows also in the tests.
Later I will try the same queries using Postgres as database backend.

buildbot, test on oracle.

buildbot, test on oracle.

Working with this branch I created a small local project using Sqlite and I verified that saving an instance of a model containing a generated field raises an error unless you specify the fields to update but in this last case the generated field is not is updated.
...
I think a test with this scenario is missing (I sent one in the code hints).
Using update instead everything works and it shows also in the tests.
Later I will try the same queries using Postgres as database backend.

Thanks, fixed.

buildbot, test on oracle.

Working with this updated branch and Postgres database I found a bug related to queries made with a generated field and with a native field.
Model
I defined a model with a TextField, an explicit SearchVectorField and a GeneratedField with SearchVectorField as output_field.
from django.contrib.postgres import search
from django.db import models

class Document(models.Model):
    content = models.TextField()
    vector = models.GeneratedField(
        db_persist=True,
        expression=search.SearchVector("content", config="english"),
        output_field=search.SearchVectorField(),
    )
    search_vector = search.SearchVectorField()

Populate
The content of the generated field and the other field, manually updated, are the same.
>>> from geometricfigures.models import Document
>>> Document.objects.create(content="The web framework for perfectionists with deadlines.")
>>> from django.contrib.postgres import search
>>> Document.objects.update(search_vector=models.Subquery(Document.objects.filter(pk=models.OuterRef("id")).annotate(search=search.SearchVector("content", config="english")).values("search")[:1]))
>>> Document.objects.values("search_vector", "vector")
<QuerySet [{'search_vector': "'deadlin':7 'framework':3 'perfectionist':5 'web':2", 'vector': "'deadlin':7 'framework':3 'perfectionist':5 'web':2"}]>

Query
It seems that the problem lies in the SQL generated in the query, in the case of the explicit field @@ is used correctly while in the case of the generated field, although the output_field attribute is explicitly specified, the resulting SQL is incorrect.
>>> str(Document.objects.filter(search_vector=search.SearchQuery("worked", config="english")).only("id").query)
'SELECT "geometricfigures_document"."id" FROM "geometricfigures_document" WHERE "geometricfigures_document"."search_vector" @@ (plainto_tsquery(english::regconfig, worked))'
>>> Document.objects.filter(search_vector=search.SearchQuery("worked", config="english")).values("id")
<QuerySet [{'id': 1}]>
>>> str(Document.objects.filter(vector=search.SearchQuery("worked", config="english")).only("id").query)
'SELECT "geometricfigures_document"."id" FROM "geometricfigures_document" WHERE "geometricfigures_document"."vector" = (plainto_tsquery(english::regconfig, worked))'
>>> Document.objects.filter(vector=search.SearchQuery("worked", config="english")).values("id")
Traceback (most recent call last):
  File "/home/paulox/Projects/django/django/db/backends/utils.py", line 101, in _execute
    return self.cursor.execute(sql, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/paulox/.virtualenvs/django/lib/python3.11/site-packages/psycopg/cursor.py", line 737, in execute
    raise ex.with_traceback(None)
psycopg.errors.UndefinedFunction: operator does not exist: tsvector = tsquery
LINE 1: ...ument" WHERE "geometricfigures_document"."vector" = (plainto...
                                                             ^
HINT:  No operator matches the given name and argument types. You might need to add explicit type casts.

@felixxm @LilyFoote working with this updated branch and Postgres database I found a bug related to queries made with a generated field and with a native field.
...
It seems that the problem lies in the SQL generated in the query, in the case of the explicit field @@ is used correctly while in the case of the generated field, although the output_field attribute is explicitly specified, the resulting SQL is incorrect.

I'm not sure it's fixable (at least I don't know how to do this). As far as I'm aware, all lookups registered for field classes are affected as they are not available for GeneratedField even with a properly set output field, e.g.
https://github.com/django/django/blob/cafe7266ee69f7e017ddbc0d440084ace559b04b/django/contrib/postgres/search.py#L336

Can we make the error nicer for now? This feels like something we can try to find a solution for in Django 5.1 to me.
Does explicitly registering the lookup with GeneratedField resolve the problem? In that case we might be able to document that as a workaround for now.

Can we make the error nicer for now?

IMO, no, I don't know how we would detect that.

Does explicitly registering the lookup with GeneratedField resolve the problem?

Not really as registering all class-related lookups on GeneratedField will create many conflicts :disappointed: Since Django 4.2 we can register lookups on field instances. We could (theoretically) register lookups from the output_field class in GeneratedField.contribute_to_class(), however this will force users to always define GeneratedField.output_field and this is just a theory :sweat_smile: .

Can we make the error nicer for now?

IMO, no, I don't know how we would detect that.

Does explicitly registering the lookup with GeneratedField resolve the problem?

Not really as registering all class-related lookups on GeneratedField will create many conflicts 😞 Since Django 4.2 we can register lookups on field instances. We could (theoretically) register lookups from the output_field class in GeneratedField.contribute_to_class(), however this will force users to always define GeneratedField.output_field and this is just a theory 😅 .

@pauloxnet Can you check the following patch?

I'll try it ASAP

@pauloxnet Can you check the following patch?
diff --git a/django/db/models/fields/generated.py b/django/db/models/fields/generated.py
index 56a2854d3c..8bf6969f9d 100644
--- a/django/db/models/fields/generated.py
+++ b/django/db/models/fields/generated.py
@@ -44,6 +44,8 @@ class GeneratedField(Field):
             if self._output_field is not None
             else self._resolved_expression.output_field
         )
+        for lookup_name, lookup in self.output_field.get_class_lookups().items():
+            self.register_lookup(lookup, lookup_name=lookup_name)

     def generated_sql(self, connection):
         return self._resolved_expression.as_sql(

It worked 👍🏻
>>> from geometricfigures.models import Document
>>> from django.contrib.postgres import search
>>> str(Document.objects.filter(search_vector=search.SearchQuery("deadline", config="english")).only("id").query)
'SELECT "geometricfigures_document"."id" FROM "geometricfigures_document" WHERE "geometricfigures_document"."search_vector" @@ (plainto_tsquery(english::regconfig, deadline))'
>>> str(Document.objects.filter(vector=search.SearchQuery("deadline", config="english")).only("id").query)
'SELECT "geometricfigures_document"."id" FROM "geometricfigures_document" WHERE "geometricfigures_document"."vector" @@ (plainto_tsquery(english::regconfig, deadline))'
>>> Document.objects.filter(vector=search.SearchQuery("deadline", config="english")).only("id")
<QuerySet [<Document: Document object (2)>]>
>>> Document.objects.filter(search_vector=search.SearchQuery("deadline", config="english")).only("id")
<QuerySet [<Document: Document object (2)>]>

It worked 👍🏻

Thanks for checking :+1:

buildbot, test on oracle.

I did further tests locally only finding problems for non-immutable functions in PostgreSQL. Apart from the documentation suggestions I've already sent, I have nothing more to add. Great job to all contributors, I think this is another great feature added to Django.

We're almost there :sleepy: only docs left :rocket: I'll continue later today/tomorrow.

buildbot, test on oracle.

Squashed commits and pushed edits to release notes.

@jpnauta Thanks :+1: Welcome aboard :+1:
@LilyFoote Thanks for pushing this to the finish line :1st_place_medal:

Thanks @LilyFoote @felixxm for pushing this through the finish line! It was a pleasure collaborating with you both 🥳


Unwrap modal header title

Codecov Report

Merging #580 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #580      +/-   ##
==========================================
+ Coverage   99.74%   99.74%   +<.01%
==========================================
  Files         262      262
  Lines        6603     6608       +5
  Branches     1504     1506       +2
==========================================
+ Hits         6586     6591       +5
  Misses         17       17

Impacted Files
Coverage Δ

src/components/modal/ModalHeader.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix serialization of query params

Addresses #128
This PR is likely a BC of 1, but as the behaviour of 1.0.0 is wrong and completely different from previous version, I think it's more a bug fix rather than a BC. Also there was no test to verify this behaviour.
Currently, the Guzzle Service serializes incorrectly any query params that are array or object.
For instance, considering this operation's param:
'foo' => [
  'type' => 'array',
  'location' => 'query'
]

Specifying this parameter: $client->myOperation(['foo' => [1, 2]]) would serialize this query parameter : foo=1&foo=2, which is completely different from the older behaviour and, from my understanding, incorrect.
Similarily, when specifying an object type, this: $client->myOperation(['foo' => ['bar' => 'baz']]) would serialize this query "foo=baz" (hence completely loosing the context of bar).
After this PR, first operation would be serialized 'foo[0]=1&foo[1]=2' and second operation would be serialized 'foo[bar]=baz'.
Thanks a lot!

Thank you very much for your contribution. You fix looks good and you even add tests. Thumbs up!

Thanks a lot ! It seems there is a lot of missing coverage with serializing query attributes, as I'm maintaining some big descriptors and starting to upgrade them, it's possible that I encounter other regressions. I'll make sure to fix them as they arise :).
In the mean time would it be possible to tag this ? :)

Hey !
Apparently I've messed and I've made the PR against the "develop" branch. Should I re-do the PR to master so that this can be tagged?

Nope. I changed it to develop intentionally because master only should contain tagged versions.

@bakura10 Just tagged version 1.0.1. Hope it works for now. You are very welcome to open issues and/or PRs. I will look to get everything fixed as soon as possible.


popover content with Vue variable interpolation

Hi, is it possible to use a vue variable inside popover? Something like
<img :src="item.label"
          height="80px"
          v-c-popover="{
              header: 'Hello',
              content: '<h1>{{ item.label }}</h1>',
              html: true,
              placement: 'top-start'
          }"
/>

I tried it but interpolation is not working. Thanks

Thank you!


Initial ROS2 Jazzy support

Hi,
I tried to integrate ROS2 Jazzy support in the same way than Iron.
Here the log of compilation:
`make -C tests/common
make[1]: Entering directory '/home/az01958/src/safe_drive/tests/common'
ros2msg_to_rs -i ../../supplements/ros2/src -o msgs
generating: msgs/example_msg/msg/buz.rs
generating: msgs/example_msg/srv/add_three_ints.rs
generating: msgs/example_msg/msg/foo.rs
generating: msgs/example_msg/msg/std_msg.rs
generating: msgs/example_msg/msg/num.rs
generating: msgs/example_msg/msg.rs
generating: msgs/example_msg/srv.rs
generating: msgs/mod.rs
generating: msgs/example_msg/mod.rs
make[1]: Leaving directory '/home/az01958/src/safe_drive/tests/common'
cd supplements/ros2 && colcon build
Starting >>> example_msg
Finished <<< example_msg [1.74s]
Summary: 1 package finished [1.94s]
cargo build
warning: extern fn uses type u128, which is not FFI-safe
--> src/rcl/jazzy.rs:15087:9
|
15087 | /         unsafe extern "C" fn(
15088 | |             serialization_support: *mut rosidl_dynamic_typesupport_serialization_support_impl_t,
15089 | |             dynamic_data: *mut rosidl_dynamic_typesupport_dynamic_data_impl_t,
15090 | |             id: rosidl_dynamic_typesupport_member_id_t,
15091 | |             value: u128,
15092 | |         ) -> rcutils_ret_t,
| |__________________________^ not FFI-safe
|
= note: 128-bit integers don't currently have a known stable ABI
= note: #[warn(improper_ctypes_definitions)] on by default
warning: extern fn uses type u128, which is not FFI-safe
--> src/rcl/jazzy.rs:15285:9
|
15285 | /         unsafe extern "C" fn(
15286 | |             serialization_support: *mut rosidl_dynamic_typesupport_serialization_support_impl_t,
15287 | |             dynamic_data: *mut rosidl_dynamic_typesupport_dynamic_data_impl_t,
15288 | |             value: u128,
15289 | |             out_id: *mut rosidl_dynamic_typesupport_member_id_t,
15290 | |         ) -> rcutils_ret_t,
| |__________________________^ not FFI-safe
|
= note: 128-bit integers don't currently have a known stable ABI
warning: safe_drive (lib) generated 2 warnings
Finished dev [unoptimized + debuginfo] target(s) in 0.09s
`
Result of test:
test result: ok. 50 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out; finished in 16.10s
Can you please review and help to enable Jazzy support ?
Thanks!

Thank you!
I will check it in this week.

Updated the version and published it to crates.io. Thank you!
https://crates.io/crates/safe_drive/0.4.0


IGNITE-11716 Fix a typo in template.

https://issues.apache.org/jira/browse/IGNITE-11716
@akuznetsov-gridgain please review. The issue was caused by ] instead of } in Pug template.

The ticket is resolved, closing PR.
Please close PRs for your resolved tickets and other non-needed PRs.
Alternatively, ask a committer

to use GitHub UI (committer can associate accounts here: https://gitbox.apache.org/setup/ )
or /scripts/apply_pull_request.sh from Apache Ignite source.
or mention in commit message 'Fixes #(a number of PR)'.

These methods will automatically close PR.
Feel free to reopen this PR if it is actual.


update readme to match current state

include a getting started guide, with the most used functions and apis (start miner, add peer, get latest block, get balance for account, get own account info)

closed by https://github.com/aeternity/elixir-research/commit/e4d7a39d928eaa54d5195d3cde2303ee233df188


Packages cause esbuild to fail. Add option to configure esbuild from Nhost?

Packages like sharp are very picky with how it is installed. When sharp is imported to a serverless function, esbuild fails to run.

By configuring esbuild to exclude the package and add a script to install with the right architecture/platform seems like the way to move forward. See solution here: https://github.com/lovell/sharp/issues/3126#issuecomment-1132745490
How can we configure esbuild from Nhost? Would it be possible to add the esbuild configuration to nhost/config.yaml?

quick question, what is your use case for sharp? I am asking because hasura-storage has support for manipulating images so if you are missing something there or have a different use case it might be interesting to hear.

quick question, what is your use case for sharp? I am asking because hasura-storage has support for manipulating images so if you are missing something there or have a different use case it might be interesting to hear.

In this particular case, perform different image manipulations together with other business logic depending on if the uploaded file is transparent or opaque. I'm creating a separate lambda function for now. The image is uploaded to a bucket before this function runs, the function saves the final image to another bucket and deletes the source image.
The issue that occurred when trying to run a function with sharp is that esbuild is failing to build with this package, and there is no current way of configuring esbuild from nhost. I don't know if this problem is small or if there are other packages that have the same problem.


[FEATURE] Automatically purge old unused snapshots

Question
Hey, I am using Longhorn together with Kasten to provision PVCs (Longhorn) and do backups (Kasten). Kasten creates csi snapshots which result in a huge load of snapshots in Longhorn over time. After a week I could not create any more backups and in Longhorn I am presented with this warning:
Name: toomanysnapshots
Last Transition Time: 12 days ago
Message: Snapshots count is 249 over the warning threshold 100
Reason: TooManySnapshots
Status: True

Is there a possibility to automatically purge old unused snapshots? They are sent to S3 anyways, right? Otherwise they will bloat the filesystem over time.
Environment:

Longhorn version: 1.1.1
Kubernetes version: 20.10.3
Node config

OS type and version Ubuntu 20.04 LTS
CPU per node: 32
Memory per node: 252GiB
Disk type: HDD
Network bandwidth and latency between the nodes: n/a

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal

Additional context
n/a

Thanks for the feedback. Will Kasten delete the old CSI snapshots after the backup finished?
Currently, Longhorn would not automatically purge old unused snapshots through the CSI snapshot mechanism.
We could discuss it internally to check if it's doable.

cc @joshimoo any feedback?

Thanks for the feedback. Will Kasten delete the old CSI snapshots after the backup finished?

This is what I found (https://docs.kasten.io/latest/install/storage.html#csi-snapshot-configuration):

Setting deletionPolicy to Delete isn't sufficient either as some storage systems will force snapshot deletion if the associated volume is deleted (snapshot lifecycle is not independent of the volume). Similarly, it might be possible to force-delete snapshots through the storage array's native management interface. Enabling backups together with volume snapshots is therefore required for a durable backup.

K10 creates a clone of the original VolumeSnapshotClass with the DeletionPolicy set to 'Retain'. When restoring a CSI VolumeSnapshot, an independent replica is created using this cloned class to avoid any accidental deletions of the underlying VolumeSnapshotContent.

Currently, Longhorn would not automatically purge old unused snapshots through the CSI snapshot mechanism.
We could discuss it internally to check if it's doable.

What is the normal way to delete multible snapshots at once? Using lhexec I can only delte one at a time?

Hey I receive the same error. Is there a possibility to purge old user generated snapshots? I saw there is a cronjob for system generated ones.
How do I delete all snapshots via lhexec? Is there a handy script? lhexec pvc-name snapshot purge does not seem to work for me.

@kompetenzlandkarte You can write a script around there might be some user scripts on github/slack for this already.
If you do write a script around this, please feel free to share it :)
lhexec <volume> snapshot ls
lhexec <volume> snapshot rm <snapshot-id>

I am no script pro but I am trying this
delete-snapshots.sh
#!/bin/bash
./lhexec $1 snapshot ls | grep "snapshot" -A1 | while read -r line; do ./lhexec $1 snapshot rm $(echo $line); done

Is failing with Unable to use a TTY - input is not a terminal or the right kind of file. Any idea how to fix this?

@kompetenzlandkarte you can also build something around the Python client, we use the same client for our e2e tests.
https://longhorn.io/docs/1.1.2/references/longhorn-client-python/
https://github.com/longhorn/longhorn-tests/tree/master/manager/integration
@yasker @innobead unfortunatly the lhexec script is setup to use an interactive tty for the exec command.
https://github.com/longhorn/longhorn/blob/85ab7c8d9d80e244aa2f78ef1e5a1ec83b909374/scripts/lhexec#L48
https://trstringer.com/kubectl-tty-issues/
@kompetenzlandkarte you can remove the-it from the kubectl exec in the lhexec.sh script that should allow for usage from a shell script. Feel free to open up a PR with your changes :)

@kompetenzlandkarte you can remove the-it from the kubectl exec in the lhexec.sh script

This was the issue. Thank you for the tip!
This is my bash script now:
#!/bin/bash
echo "delete snapshots for pvc $1"

./lhexec $1 snapshot ls | grep "snapshot" -A1 | while read -r line; do echo "delete snapshot $line" && ./lhexec $1 snapshot rm $line; done

echo "now purging the snapshots for pvc $1"

./lhexec $1 snapshot purge

However this script is pretty slow due to iterating every snapshot at a time (~2-5s per snapshot).

Hey guys, I've been running into the same problem with everyone here since longhorn 1.0.0-beta and honestly I'm really glad that there's a Python client in the works now.
Nonetheless, I just wrote a script an hour back to take care of all the 'stale' and 'orphaned' snapshots that were not longer tracked by any retention policy (perhaps due to having set a new backup schedule in bulk).
It's worked out great for me. Figured I'd share it here as well.
# Cleanup Longhorn snapshots script by ikaruswill
# ==============
#
# Definitions:
# - Orphaned snapshots: Snapshots created by backup schedules that no longer exist.
# - Stale snapshots: Snapshots that are over n days old
#
# Variables:
# - delete_strings: Delete snapshots that contain any of the strings
#                   To locate orphaned snapshots.
#                   Set to your old backup schedule names
# - delete_age_days: Delete snapshots that are above the specified age in days
#                    To locate stale snapshots.
#                    Set to your oldest possible snapshot (i.e. I have a weekly backup, with Retain = 4, so the oldest snapshot should be 28 days)
#
# Usage:
# 1. Run `kubectl port-forward -n longhorn-system services/longhorn-frontend 8080:http`
# 2. python cleanup-snapshots.py
#
# Prerequisites:
# 1. longhorn.py from: https://raw.githubusercontent.com/longhorn/longhorn-tests/master/manager/integration/tests/longhorn.py
# 2. requests library from: `pip install requests`

import longhorn
import datetime

# Constants
date_format = '%Y-%m-%dT%H:%M:%SZ'
mib_conversion_factor = 1 / 1024 / 1024
current_date = datetime.datetime.now()

longhorn_url = 'http://localhost:8080/v1'

# Variables; change according to your needs
delete_strings = ['c-6fffho', 'c-b8sdpg']
delete_age_days = 30

def delete_snapshot(volume, snapshot, reason):
    print('Deleting {reason} snapshot {name} created on {created} with size {size:.1f} MiB'.format(
                name=snapshot.name,
                reason=reason,
                created=datetime.datetime.strptime(snapshot.created, date_format).strftime('%Y-%m-%d'),
                size=int(snapshot.size) * mib_conversion_factor
            ))
    try:
        volume.snapshotDelete(name=snapshot.name)
    except longhorn.ApiError as e:
        print('Failed to delete snapshot due to API error: {}'.format(e))

def process_snapshot(volume, snapshot):
    for delete_string in delete_strings:
        if delete_string in snapshot.name:
            delete_snapshot(volume, snapshot, 'orphaned')
            return True
    snapshot_date = datetime.datetime.strptime(snapshot.created, date_format)
    if (current_date - snapshot_date).days > 30:
        delete_snapshot(volume, snapshot, 'stale')
        return True
    print('Ignoring snapshot: {}'.format(snapshot.name))
    return False

def process_volume(volume):
    print('Processing volume: {}'.format(volume.name))
    try:
        snapshots = volume.snapshotList()
    except AttributeError as e:
        print('Ignoring volume due to AttributeError: {}'.format(e))
        return
    print('Number of snapshots: {}'.format(len(snapshots)))
    deleted_count = 0
    for snapshot in snapshots:
        snapshot_is_deleted = process_snapshot(volume, snapshot)
        if snapshot_is_deleted:
            deleted_count += 1

    if deleted_count > 0:
        print('Purging snapshots...')
        volume.snapshotPurge()

    print('Finished processing volume {}'.format(volume.name))
    print('Deleted {} snapshots'.format(deleted_count))
    volume = client.by_id_volume(id=volume.id)
    print('---')

def process_cluster(client):
    volumes = client.list_volume()
    print('Starting volume snapshot cleanup')
    print('Number of volumes: {}'.format(len(volumes)))

    for i, volume in enumerate(volumes):
        print('Progress: {:.1f}% ({}/{})'.format(
          i / len(volumes) * 100,
          i,
          len(volumes),
        ))
        process_volume(volume)

    print('Finished volume snapshot cleanup')

if __name__ == '__main__':
    client = longhorn.Client(url=longhorn_url)
    process_cluster(client)

I do think we can just leverage recurring jobs here to clean snapshots regularly. cc @PhanLe1010

@innobead The recurring jobs only clean up the snapshot they created as well as the doing the system snapshot purge. If the snapshot is created by something else (i.e user, volume clone, csi snapshot, etc, ...) the snapshot won't be cleaned by the recurring jobs.
I think the script provided by @ikaruswill is the current workaround. I am thinking if we should add one more API to cleanup snapshot for a volume then the user can just click it

Is there any progress in this one. It's a rather major challenge at times. We can have Longhorn committed storage space issues quite fast. As space fills up over time. So we kinda need a cleanup mechanism where we can set the lifetime of snapshots.
Status is still ... use that script ( https://github.com/longhorn/longhorn/issues/2732#issuecomment-896205929 ) ??

This is related to #3836


Bugfix release 0.5.1

Do you mind publishing this?

As soon as i am behind keyboard. I'll add you as contrib to the npm package so that you can publish yourself..


Creating a wiki for IotGoat

Iot-Goat is a big project and will impact our security community greatly and having it well documented will help it to reach far, will help testers and exploit makers to easily get familiar.
This will also solve the question asked in challenges/challenges.md which was "In what form is the documentation going to be presented?"
Would you like to work on it ?
Yes, I'm working on it.
Ref : https://github.com/0x48piraj/IoTGoat/wiki [WIP]

Ping @scriptingxss and @cldrn

Hey @0x48piraj  - A wiki would be great! Right now, the repo is essentially acting as a wiki for now but will only used to house project releases and updates in the future. Ive recently pushed updates to the repo. Feel free to have a look and make modifications where applicable.

That's great!
Something like owaspbwa?
I wanted to ask some questions regarding how it all will be done, like the challenges, but you cleared almost all the questions in BuildEnvironment.md
I have extensively used OpenWrt SDK and OpenWrt Image Builder, so I'll make a PR on this repository.
One more question was there, how the release will look like?
In my thinking, as some challenges will require insecure configurations, we can't just put the firmware source code, I think there'll be a vulnerable virtual image hosted on sourceforge, or on GitHub itself.

Yes, for the most part.
Cool. Within that same page, the last section has details for submitting challenges through a form for now. We could upload vulnerable firmware to GitHub if we wanted (similar to DVRF, DVAR etc.) although we are still brainstorming better methods in the meantime. Check out - https://github.com/scriptingxss/IoTGoat/blob/master/BuildEnvironment.md#-submitting-challenges-for-review-gsoc-students-

Yes I saw that section, it's a good option while the project attains a stable state.
I'll make required changes to my GsoC proposal and will add some brainstormed ideas on releasing Iot-Goat.
For the same, can I know email of yours so that I can ask queries, or send the proposal for getting feedback?


Add Python 3.9 as a supported version

This patch adds Python 3.9 as a tested tox version and lists it as a trove classifier in setup.py.
I didn't try to add it as a GitHub workflow because I'm not sure if Python 3.9 is supported yet. That piece still needs to be done.

Thanks! Python 3.9 is not out officially, is it?

Not yet, but they've released a final release candidate and tox is able to find and test against it when it's installed.
On September 27, 2020 6:07:37 AM UTC, "Matthäus G. Chajdas"<EMAIL_ADDRESS>wrote:

Thanks! Python 3.9 is not out officially, is it?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/pygments/pygments/pull/1554#issuecomment-699590970

Python 3.9 was released, time to merge this :)


Translated to Turkish

Thanks for the fixes and translation! Merging it ... @GurcanH

Thanks for the fixes and translation! Merging it ... @GurcanH

Thank you @alioguzhan


Week 7 not able to upload the model.

I need help in correcting this error which occurred during uploading the model.

please suggest the way to correct this....

Hey there 👋  I think you forget to put ! in front of mlagents. When you run bash command in Colab you need to add a !
So !mlagents-push-to-hf


gdal2tiles.py doesn't work

Hi !
It seems gdal2tiles.py (and other scripts) are installed, but for some reason, running gdal2tiles.py --help returns this error :
$> gdal2tiles.py test.tifTraceback (most recent call last):
  File "/usr/local/bin/gdal2tiles.py", line 4, in <module>
    __import__('pkg_resources').run_script('GDAL==3.0.0', 'gdal2tiles.py')
  File "/usr/local/lib/python3.7/site-packages/pkg_resources/__init__.py", line 666, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/usr/local/lib/python3.7/site-packages/pkg_resources/__init__.py", line 1444, in run_script
    .format(**locals()),
pkg_resources.ResolutionError: Script 'scripts/gdal2tiles.py' not found in metadata at '/usr/local/lib/python3.7/site-packages/GDAL-3.0.0-py3.7.egg-info'

The script works well when ran from site-packages directly:
$> /usr/local/lib/python3.7/site-packages/GDAL-3.0.0-py3.7-linux-x86_64.egg/EGG-INFO/scripts/gdal2tiles.py test.tif
Generating Base Tiles:
0...10...
....

Not really sure why this is happening...
Cheers !

On Sun, Jun 23, 2019 at 5:34 PM olivierdalang<EMAIL_ADDRESS>wrote:

Hi !
It seems gdal2tiles.py (and other scripts) are installed, but for some
reason, running gdal2tiles.py --help returns this error :
$> gdal2tiles.py test.tifTraceback (most recent call last):
File "/usr/local/bin/gdal2tiles.py", line 4, in
import('pkg_resources').run_script('GDAL==3.0.0', 'gdal2tiles.py')
File "/usr/local/lib/python3.7/site-packages/pkg_resources/init.py", line 666, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/usr/local/lib/python3.7/site-packages/pkg_resources/init.py", line 1444, in run_script
.format(**locals()),
pkg_resources.ResolutionError: Script 'scripts/gdal2tiles.py' not found in metadata at '/usr/local/lib/python3.7/site-packages/GDAL-3.0.0-py3.7.egg-info'
The script works well when ran from site-packages directly:
$> /usr/local/lib/python3.7/site-packages/GDAL-3.0.0-py3.7-linux-x86_64.egg/EGG-INFO/scripts/gdal2tiles.py test.tif
Generating Base Tiles:
0...10...
....
Not really sure why this is happening...
Cheers !

Hello, Olivier!
Thx for your ticket!
I will do some investigation of this issue in a few days.
I'm not sure that is a problem was in previous versions.
Also, I want to write some tests to make a safer version upgrade in the
future.
Anyway, if you found better or faster or just another solution feel free to
open PR :)

Fixed


Dojo emails should be using dojo language

Depends on https://github.com/CoderDojo/community-platform/issues/989, emails coming from the dojo should be translated in the language of the dojo.
This include emails to the dojo Email (automated emails like join notification) as well as email to the users of the dojo (custom emails, automated emails ( ticketing))

(1) Shouldn't they use the language of the user reading the email?
(2) With your templates, you have the ability, to provide a link at the top like this:
http://zen.coderdojo.com/template/text.html?lang=en&username=John%20Doe

This would allow to render most templates in all languages. The link can be provided at the top: "View in Webbrowser/Translations"
Allowing a menu for languages and a link to the crowdin if they would like to translate a template.

We could, but it means to ask one more configuration from the user, while a configuration at a dojo level would impact more easily than "each" user having set their locale.
Ultimately it would be per user and if it's not set, the dojo's language, but we can assume that they speak the same language in a dojo, and hence have a quicker impact/result.

We plan to work on tests for translations and it requires us to create those "mock" data for templates. Once we have those, exposing a rendered version should be totally doable. Not sure though if it'll allow "in-context" translation as the variable will already be rendered. I think another issue for this would be nice so we can set up the steps to make it work.


Refactor the Get-Content tests to use -TestCases.

PR Summary
Refactor the Get-Content tests to use -TestCases.

PR Checklist
Note: Please mark anything not applicable to this PR NA.

[x] PR has a meaningful title

[x] Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

I introduced the changes suggested by SteveL-MSFT.


Windows 10: File /npm/node_modules/devtool/index.html not found

When I open devtool in win10, devtool console show:
npm/node_modules/devtool/index.html  Failed to load resource: net::ERR_FILE_NOT_FOUND
It is very annoying. Could you add a index.html file to you project root path?

I'm having the same problem but with \npm\node_modules\devtool\bin\index.js file, devtool used to work, and now I can't debug at all without it


Small enhancement in WriteChunks

An attempt to address https://github.com/prometheus/prometheus/pull/6040#discussion_r333430034

Yup this is less clean, but I would prefer them to be in the limit when possible :)

Do you want to update the tests or shall I do it?

I might not be able to update the tests, so please go ahead with it! Thanks


The naming of joints

Hi, I'm using 17 joints from https://github.com/facebookresearch/VideoPose3D Human3.6 data but I see that you only use 16 joints for valid_angle. Can you show me the names of those 16 joints that I can update the loss to use in my case?
Thank you,

I guess they are [1  2  3  6  7  8 12 13 14 15 17 18 19 25 26 27]


Group Chat Not Returned in Search Results When Using Partial Match for Group Name

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.63-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation (hyperlinked to channel name): ts_external_expensify_bugs
Action Performed:

Create a group chat with users
Name the group as nyc crew in group details
Click the search icon and select Chats from LHN
Click the search from top right
Enter nyc

Expected Result:
The chat group (nyc crew) displayed to select
Actual Result:
Group chat not returned in search result using partial match
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/b996a11a-000b-4831-b049-ebdc948ed9dd
View all open jobs on GitHub

This is an internal issue @trjExpensify FYI. It has to do with what reports are returned by the back-end in the SearchForReports command.

Makes sense! I have a feeling @luacmartins has a proposal out for this. Is that right?

Yes, I do. Although, I'm not able to reproduce the issue in the OP.

@luacmartins maybe it has something to do with the emoji in the name?

 RequestID: 8e4b552baffde7df-DFW
And then this is when I search nyc crew I get just one report returned, which is<PHONE_NUMBER>833997. RequestID: 8e4b5789ebd8e7df-DFW
I sent you both API responses on Slack.

Cool, yea I imagine it's how we're prioritizing the results we send back.

Right that makes sense. Since when I search "nyc" I only see 1 result in the search results, I assumed it was only returning one. But I see now it is returning a bunch that we don't end up showing.

Yes, I do. Although, I'm not able to reproduce the issue in the OP.

Okay, do we need this issue then?

We can probably close it since we have a project that'll fix this.


Russian Translations (Thanks, @DDDIM)

What's New?

Improve Russian Translations. See PR 680 by @DDDIM .

Coverage increased (+0.004%) to 87.742% when pulling dd99b0689a220aae8ff36478ed1cf0d942ce4d5e on RickCarlino:ru_i18n into cb6389acc12dc232fab23446c5cf094c3a30165b on FarmBot:staging.


[pasteboard] How to listen to clipboard paste events?

Is there a way to listen to clipboard paste events and then read image? How do I use this plugin in situation where whenever an image is pasted from clipboard, I need a callback to handle the event?

You can use SingleActivator or something else ( like a button ) to interact with user and then get the data from clipboard.
It there any system give a event callback for clipboard?

Web provides this:
html.document.onPaste.listen(onPasteEvent);

I think flutter should also have something similar to listen to paste events.

This seems belong to the flutter framework.

Yes, it does. I am not sure how but it would be great to have something similar for desktop platforms as well.


Update sts to 2.17.171

Updates software.amazon.awssdk:sts from 2.17.163 to 2.17.171.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk", artifactId = "sts" } ]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

This was already updated.


Two new translation keys that haven't been translated to Chinese

A recent PR added a few new sentences to the donate page. The new keys are here: https://github.com/freeCodeCamp/freeCodeCamp/blob/next-i18n-client/client/i18n/locales/chinese/translations.json#L346.
Want to translate these when you get a chance @miyaliu666?

Just translated line 209, 210, 346 and 367. LGTM. @moT01

Just translated line 209, 210, 346 and 367. LGTM. @moT01


Merge gqlv2-fix into flutter-preview tagged release.

Description of changes
Merge gqlv2-fix into flutter-preview tagged release. This is intended to unblock flutter users testing out the new gqlv2 features.
Issue #, if available
N/A
Description of how you validated changes
N/A, merging into a tagged-release to allow for testing.
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] Breaking changes to existing customers are released behind a feature flag or major version update
[ ] Changes are tested using sample applications for all relevant platforms (iOS/android/flutter/Javascript) that use the feature added/modified

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Confirmed w/ Hui from the Flutter team that the generated models for this merge look good. Merging.


spelling fixes, just comments and readme.

except for one error, looks good.

+1

Coverage increased (+0.0009%) to 54.193% when pulling 5cca488e98c2d08450bee943434116c598398092 on ka7:spelling_fix into 39a43280316f1b9c45be5ac5b04f4f5c3f923686 on openssl:master.

+1

Coverage increased (+0.03%) to 54.218% when pulling 5cca488e98c2d08450bee943434116c598398092 on ka7:spelling_fix into 39a43280316f1b9c45be5ac5b04f4f5c3f923686 on openssl:master.

I think I disabled coveralls from commenting on it.

merged, thanks!


Update Grouping and Order of Versions displayed on https://start.grails.org/

This will likely need to wait on 7.0.0 when we have two releases with the workflows in the git release tag so they can be deployed. Or if we circle back and release 6.1.3.
possibly something like:
6.2.2 - default
6.1.2
------------
7.0.0-SNAPSHOT
6.2.3-SNAPSHOT
6.1.3-SNAPSHOT

Which by key is:
release - default
prev
------------
next
snapshot
prev-snapshot

This will likely need to wait on 7.0.0 when we have two release branches with the workflows in the git release tag so they can be deployed.  Or if we circle back and release 6.1.3.

https://github.com/grails/grails-forge/issues/380


Jacobian pseudo inverse IK using lapack solvers

Use Lapack's dpotrf and dpotri for matrix inversion (see AICO implementation).

Implemented in 4d46f827575f1ddd229212aa5b9faf201e5ddf9b
Also added joint limit handling


Cancellations are not propagated to other bluebird library instances

What version of bluebird is the issue happening on?
3.5.5

Bluebird does not properly propagate cancellations to other Bluebird promise instances within a .then() function. It does properly propagate if you re-wrap the other instances in a .resolve().
This is especially problematic because Bluebird is frequently used by many other libraries. If 3rd party code is on a different bluebird version, or it uses Bluebird.getNewLibraryCopy() then it won't propagate correctly.
My guess is that the problem is somewhere inside of the tryConvertToPromise function. Bluebird does correctly identify that the promise was created by Bluebird, but does not correctly attach the cancellation logic when you return another promise in a .then(). It does work if you use .resolve() on the different Bluebird promise instance.
Does not cancel (bug)
Promise.config({ cancellation: true })

// the key here is to create a new bluebird copy
const P2 = Promise.getNewLibraryCopy()
P2.config({ cancellation: true })

const inner = () => {
  console.log('inner!')
  return Promise.delay(100).then(inner)
}

Promise
.resolve()
.then(() => {
  console.log('before inner loops')

  // because we return a new promise here
  // created from ANOTHER bluebird library
  // then cancellation will not propagate
  return P2.resolve(null).then(inner)
})
.then(() => {
  console.log('after inner loops')
})
.timeout(2000)
.catch(Promise.TimeoutError, (err) => {
  console.log('promise timed out after 2 seconds! no more inner loops should fire')
})

Does cancel if re-wrapped with Bluebird.resolve(...)
Promise.config({ cancellation: true })

const P2 = Promise.getNewLibraryCopy()
P2.config({ cancellation: true })

const inner = () => {
  console.log('inner!')
  return Promise.delay(100).then(inner)
}

Promise
.resolve()
.then(() => {
  console.log('before inner loops')

  // re-wrapping here does cancel correctly
  return Promise.resolve(P2.resolve(null)).then(inner)
})
.then(() => {
  console.log('after inner loops')
})
.timeout(2000)
.catch(Promise.TimeoutError, (err) => {
  console.log('promise timed out after 2 seconds! no more inner loops should fire')
})

@petkaantonov it sounds reasonable to support this, wdyt?

Any update on this :/

Honestly cross-bluebird cancellation isn't a big priority for me - but if you contribute a PR I will happily review it and get a release going and I am happy to point you to the relevant places in the code.

I will also mention that I actually tried to debug a bunch of incorrect bluebird usage in the cypress code-base but I was unable to get it running and was promptly ignored :]

need to investigate if its possible to support, some cancellation might be dependend on identity


Update CODEOWNERS

@coderpatros
there was a slight change in the codeowners. team name had no organization.
this is why this PR is probably not assigned to you automatically. so you have to add yourself as a reviewer manually.


Any visuals available to display the current time on the calendar?

Google calendar has a current time indicator shown by a horizontal red line across the calendar. This indicates the current time of the user. Is it possible to do this currently in react-available-times?

That’s definitely possible. It’s not something I have a need for in my projects but I’d be happy to review a PR adding this.

I could try throwing something together. Any tips on where to start?

On an airplane right now, but I think I got something working.. \Wondering if you have recs to best way I should grab the timezone of the user. I was pulling in timeZone from the week component, but it seems like every time I click on the calendar, the value changes, causing awkward changes to the current line.

@trotzig What do you use to format this project? Not seeing any scripts or mentions in the readme.

I threw up a PR to get the idea started. Not really a React guy, so hoping to get a tidbit of guidance to get it merged. Cheers!


High CPU load when used together with Kaspersky antivirus

WinSshFS
WinSSHFS-<IP_ADDRESS>-devel.msi
Dokan driver
Dokan Library <IP_ADDRESS>0 Bundle
Enviroment

Windows version: 8.1
Architecture: x64

Support

Usage: commercial - development
Your personal priority to solve this issue: high
Support ability: deeptesting

I noticed that when I mount a directory using WinSshFs, CPU usage gets up to 50% (25% for WinSshFs and 20% for Kaspersky) even when mounted a small directory. I suspect that Kaspersky is scanning the drive all the time, but that doesn't happen for other drives (HDD, network drives, etc).
The annoyance is that the whole computer gets very unresponsive even though CPU is very fast, so I suspect that one (of four) CPU cores has 100% utilization all the time.

You can try changing UseNetworkDrive in WinSshFS.exe.config to True (config is in WinSSH folder)
WinSSH should mount it as network drive instead of local one - it may appear offline, but you can browse through and Kaspersky should not try to scan it.

I never tested this way of mounting, so i dont know if it has any issues.
@dimov-cz may have some more info on this

I dont know if this is helpful to you but might be
I had the same kind of problem but with Microsoft security essentials, you have to disable usb scanning.

Hi.
I apologize, I forgot to test with UseNetworkDrive and now I don't have access to that machine anymore. So, it seems that there is a workaround, which is a very good thing :)
Best regards.


Extend time verifying node wasn't rebooted

I've noted that sometimes tests fails due to timeouts waiting for pods to start / waiting for pods to be available.
I see different behavior on my own cluster (rarely fails) and also during different times of day - I assume it has to do with resource usage.
Extending the test times which are prone to cause e2e tests fail in order to improve tests stability.

/retest

/override ci/prow/4.13-openshift-e2e

/retest

/test 4.11-openshift-e2e

/override ci/prow/4.13-openshift-e2e

/override ci/prow/4.13-openshift-e2e

/lgtm


Add a video VGA port with a switch between color/monochrome mode

The VGA will route the analog video signal.
To switch between mono/rgb mode, a triple analog 2:1 mux is required, like SN74LV4053A. 4 trimpot are also require to setup the appropriate level/impedance for the vga monitor (to be checked).
Something along that :

Red → trimpot_red → mux_1_a
Green → trimpot_green → mux_2_a
Blue → trimpot_blue → mux_3_a
Mono → trimpot_mono → mux_1_b + mux_2_b + mux_3_b
GND → switch → mux_ab_sel + pull_up + mono_detect will force mono, even for the legacy connector

Clearly out of scope for mark0. There is already plenty of adaptor to plug to a VGA monitor


Backward issue in pointops.KNNQuery

Thank you for this helpful codebase! I'm trying to use PointTransformer models (currently PTv1) in my project but I get errors when backpropogating the gradient:
.../python3.10/site-packages/torch/autograd/function.py", line 406, in backward
    raise NotImplementedError(
NotImplementedError: You must implement either the backward or vjp method for your custom autograd.Function to use it with backward mode AD.

I find the error is reported on pointops.knn_query function, and I find the class KNNQuery(Function) does not implement backward function (here). I'm not very familiar with torch.autograd.Function so I'm wondering 1) is it the root of this error? Why is the backward function not implemented? And 2) how can I fix it?

Yes, knn don’t need to backward, maybe add with torch.no_grad() to cover and can solve this issue. (You definitely should use v3 or v2 which are significantly faster and stronger)

Thanks for your quick reply! I'll try v2 later. I just don't know why no one seems to encountered this issue. The error persists even I wrapped the function with torch.no_grad() and only solved after I add a dummy backward function like:
class KNNQuery(Function):
    @staticmethod
    def forward(ctx, nsample, xyz, offset, new_xyz=None, new_offset=None):
        """
        input: coords: (n, 3), new_xyz: (m, 3), offset: (b), new_offset: (b)
        output: idx: (m, nsample) -1 is placeholder, dist2: (m, nsample)
        """
        if new_xyz is None or new_offset is None:
            new_xyz = xyz
            new_offset = offset
        assert xyz.is_contiguous() and new_xyz.is_contiguous()
        m = new_xyz.shape[0]
        idx = torch.cuda.IntTensor(m, nsample).zero_()
        dist2 = torch.cuda.FloatTensor(m, nsample).zero_()
        knn_query_cuda(
            m, nsample, xyz, new_xyz, offset.int(), new_offset.int(), idx, dist2
        )
        return idx, torch.sqrt(dist2)

    @staticmethod
    def backward(ctx, idx, dist2):
        # no gradient
        return (None, None, None, None, None)

I suppose we do not need any gradient on the idx result?

No need gradient for fix result.

Thanks.


Extend checkPodsWithAntiAffinityExist to store pods into anti-affinity groups

The RemovePodsViolatingInterPodAntiAffinity strategy does not perform any pod balancing at the moment.
if checkPodsWithAntiAffinityExist(pods[i], pods) && d.handle.Evictor().Filter(pods[i]) {
	if d.handle.Evictor().Evict(ctx, pods[i], evictions.EvictOptions{}) {
		// Since the current pod is evicted all other pods which have anti-affinity with this
		// pod need not be evicted.
		// Update pods.
		pods = append(pods[:i], pods[i+1:]...)
		i--
		totalPods--
	}
}

might be extended to store pods into anti-affinity groups. Then during eviction remove pods from groups which have the most pods first to reduce size of groups through out all namespaces as a balancing metric. No objections for implementing such functionality and changing the extension point then.
Originally posted by @ingvagabund in https://github.com/kubernetes-sigs/descheduler/pull/903#discussion_r945343154

/remove-lifecycle stale


Load only mip levels up to a specific size

What problem does this solve or what need does it fill?
It is common for games to offer multiple levels of graphics settings, like: "Low", "High", "Ultra". When it comes to Texture Quality, this typically means the size of textures used. For example, a 2048x2048 texture will use 4x the GPU memory than a 1024x1024 texture and add more overhead during rendering, but look a lot crisper.
In 3D games, mipmaps are essentially required for high quality and performant rendering. If the game's assets are stored as KTX2 or DDS files with mipmaps, the smaller texture sizes are already available on-disk (and will, in fact, be loaded by Bevy and into GPU memory) anyway. In order to limit the max texture size at lower graphics settings, it could be implemented simply by just skipping the first mip level or two.
For example, if we have a 2048x2048 texture as a KTX2 file with a full set of mipmaps down to 1x1 (12 levels total), our hypothetical game could behave as follows:

At the "Ultra Graphics" setting: load the whole thing
At the "High Graphics" setting: skip the first mip level, effectively loading it as a 1024x1024 texture
At the "Low Graphics" setting: skip the first two mip levels, effectively loading it as a 512x512 texture

What solution would you like?
If Bevy supports specifying which mip levels to load, this could be done trivially, and reduce loading times by loading less data from disk.
What alternative(s) have you considered?

Load everything anyway, and then simply not use it. The range of mip levels used by the GPU can be restricted using the lod_min_clamp and lod_max_clamp fields in the SamplerDescriptor. Disadvantage: we don't get any savings in GPU memory or loading times, so it kinda defeats the point.

Implement a two-step process, by discarding the unwanted data from the Image asset after it is loaded. We get GPU memory savings, but no loading time savings. Also, the full resolution data is stored intermittently in CPU memory, before we get a chance to delete it. Inefficient: loading data only to immediately drop it.

Create separate asset files with a lower base resolution. Works, but is redundant, wastes disk space. The lower resolution images are already available as mip levels in the larger asset.

Additional context
This, of course, only applies to assets stored in file formats that have mipmaps (KTX2 and DDS). When loading images from, say, PNGs, this is irrelevant. In that case, if you want multiple different sizes, you need multiple files, or to downscale the image at runtime.

Yeah we should have something like this. Faster load times, faster runtime (more cache efficient to sample a lower-res texture), and less memory usage (the main draw).

"This, of course, only applies to assets stored in file formats that have mipmaps (KTX2 and DDS)" - I think .basis does too, doesn't it?

It does iirc, but basisu is generally not worth it https://github.com/bevyengine/bevy/issues/14671.

I personally really dislike Basis Universal and I completely forgot about it when I was writing the OP. 😆
It aims to solve a problem (targeting both mobile GPUs using ASTC and desktop GPUs using BCn using the same asset files) which is largely non-existent in practice (you ship different builds for desktop and mobile anyway; just ship your assets in the appropriate format). And the cost is worse texture quality (it has to make compromises with the lossy compression) and slower loading times (you have to decode it on the CPU; sure it is fast, but with the native GPU formats you can copy directly to GPU memory and no decoding is needed).
Anyway, let's not derail the thread.