Add str/repr to ContFuncDiscrete

Fixes https://github.com/dstansby/demcmc/issues/27

Codecov Report
Base: 88.37% // Head: 87.46% // Decreases project coverage by -0.91% :warning:

Coverage data is based on head (964a0c8) compared to base (e452f41).
Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   88.37%   87.46%   -0.91%
==========================================
  Files           9        9
  Lines         344      351       +7
==========================================
+ Hits          304      307       +3
- Misses         40       44       +4

Impacted Files
Coverage Δ

src/demcmc/io.py
100.00% <ø> (ø)

src/demcmc/emission.py
71.42% <50.00%> (-2.05%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


This error is constantly showing up in my application, Failed to read the 'value' property from 'SVGLength': Could not resolve relative length

It seems that it is something specific to your implementation, but I cannot guess what it is

is that so?, i will re-check and revert

Yeah, It's solved, thanks for the support

@kumar-abhinit  Hi can you tell us what was causing the above error. I'm also facing it


remove slack workspace prefix from plan overview link

The workspace prefix/subdomain on this link results in a login-page being displayed if the user is not logged into that workspace, without the workspace prefix/subdomain the page can be displayed normally.
It can be argued that the Text "plus plan" is also misleading/outdated, but that should be handled elsewhere since i do not know the minimum needed Slack plan for this feature/integration to work.

@frusznyak : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


nixos/test-driver: use an argument instead of the out env-var

Motivation for this change
Bug  #153766, read comment
Things done
Instead of reading the out environment variable in the test-driver's Machine::copy_from_vm and Machine::screenshot functions, a output-directory is used.
The output-directory is a mandatory command-line argument that needs to be passed to the test-driver.
When makeTest invokes the test-driver the out environment variable is passed as the argument.

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)
[ ] 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

[ ] 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/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Oh drat, I just implemented my own fix as well... If we don't end up going with that, I also cleaned up the screenshot logic, we should probably incorporate that here as well.

@ofborg test nat.firewall networking.scripted.link installer.simpleProvided installer.separateBootFat networking.scripted.virtual printing keymap.azerty keymap.dvorak-programmer boot-stage1 installer.swraid keymap.neo nfs4.simple i3wm udisks2 networking.networkd.loopback containers-ip ecryptfs login installer.simpleLabels php.httpd zfs.installer predictable-interface-names.unpredictable predictable-interface-names.unpredictableNetworkd mutableUsers

Oh drat, I just implemented my own fix as well... If we don't end up going with that, I also cleaned up the screenshot logic, we should probably incorporate that here as well.

We could split it up into two PRs each for the respective change?

@ofborg test nat.firewall networking.scripted.link installer.simpleProvided installer.separateBootFat networking.scripted.virtual printing keymap.azerty keymap.dvorak-programmer boot-stage1 installer.swraid keymap.neo nfs4.simple i3wm udisks2 networking.networkd.loopback containers-ip ecryptfs login installer.simpleLabels php.httpd zfs.installer predictable-interface-names.unpredictable predictable-interface-names.unpredictableNetworkd mutableUsers

@ofborg test pantheon boot.biosCdrom boot.uefiUsb networking.networkd.static latestKernel.login networking.networkd.dhcpSimple networking.networkd.sit env switchTest hibernate predictable-interface-names.predictable gnome networking.networkd.bridge networking.networkd.privacy plasma5 systemd-networkd-ipv6-prefix-delegation lightdm installer.simple installer.btrfsSimple installer.btrfsSubvols sway networking.networkd.link networking.scripted.privacy

@ofborg test nat.firewall-conntrack networking.scripted.sit networking.networkd.virtual networking.scripted.dhcpSimple firewall firefox networking.scripted.macvlan proxy firefox-esr sddm.default boot.uefiCdrom networking.scripted.static networking.scripted.routes installer.lvm networking.networkd.vlan containers-imperative boot.biosUsb xfce networking.networkd.dhcpOneIf installer.simpleUefiSystemdBoot keymap.qwertz gnome-xorg shadow networking.networkd.macvlan networking.networkd.routes

@ofborg test nfs3.simple php.fpm fontconfig-default-fonts networking.scripted.bond nano keymap.dvorak installer.luksroot networking.scripted.bridge docker simple chromium networking.scripted.dhcpOneIf networking.networkd.bond php.pcre nat.standalone openssh networking.scripted.vlan ipv6 installer.separateBoot misc predictable-interface-names.predictableNetworkd networking.scripted.loopback keymap.colemak installer.btrfsSubvolDefault

This broke test output, now screenshots and such are not copied to $out. #156910 fixes this.


Investigate seemingly wrong SSSP metric output

In one of the benchmarks (datagen-8_1-fb), the following error happens:
12:26 [DEBUG] [Runner r880922] =>  - Vertex<PHONE_NUMBER>7793 has value '4.0', but valid value is '4.039870000000001'
12:26 [DEBUG] [Runner r880922] => Validation failed.
12:26 [DEBUG] [Runner r880922] =>  - Correct vertices: 2072116 (100.00%)
12:26 [DEBUG] [Runner r880922] =>  - Incorrect vertices: 1 (0.00%)
12:26 [DEBUG] [Runner r880922] =>  - Missing vertices: 0 (0.00%)
12:26 [DEBUG] [Runner r880922] =>  - Unknown vertices: 0 (0.00%)

Interestingly, only a single node is incorrect.
The measurement successfully runs for all the other graphs; this raises the question whether our implementation is buggy, or the validation set is not correct.

Matrices were only zeroed in the diagonal partially, causing the bug.


return only true prob

related #72

@haisland0909  Thanks a lot for the pull request. I will review it and get back to you if I have any questions.


Some errors about mmoitems when using FakePlayer.

This is a fantastic plugin. However, I have some problems when create one player. I have installed the mmoitems before. When I create one player, the console begin reporting errors. Is it possible to solve this problem？I will be disappointed if it is not. Because this is a REALLY wonderful plugin! Looking to your reply. Thank you very much!
[Server thread/WARN]: [MMOItems] Task #2507 for MMOItems v6.3.1 generated an exception
java.lang.NullPointerException: null
at net.Indyuce.mmoitems.MMOItems.lambda$onEnable$2(MMOItems.java:229) ~[?:?]
at org.bukkit.craftbukkit.v1_16_R3.scheduler.CraftTask.run(CraftTask.java:99) ~[patched_1.16.4.jar:git-Paper-296]
at org.bukkit.craftbukkit.v1_16_R3.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:468) ~[patched_1.16.4.jar:git-Paper-296]
at net.minecraft.server.v1_16_R3.MinecraftServer.b(MinecraftServer.java:1297) ~[patched_1.16.4.jar:git-Paper-296]
at net.minecraft.server.v1_16_R3.DedicatedServer.b(DedicatedServer.java:377) ~[patched_1.16.4.jar:git-Paper-296]
at net.minecraft.server.v1_16_R3.MinecraftServer.a(MinecraftServer.java:1212) ~[patched_1.16.4.jar:git-Paper-296]
at net.minecraft.server.v1_16_R3.MinecraftServer.w(MinecraftServer.java:1000) ~[patched_1.16.4.jar:git-Paper-296]
at net.minecraft.server.v1_16_R3.MinecraftServer.lambda$a$0(MinecraftServer.java:177) ~[patched_1.16.4.jar:git-Paper-296]
at java.lang.Thread.run(Thread.java:834) [?:?]

I am not a native English speaker.So there may be some misunderstanding sentence. :)

I can't find the issue because the plugin is a premium plugin so I can't find source codes of mmoitems plugin. I think you should ask that issue to the plugin's owner.

That's OK. Thank you for your quick reply!

I can't find the issue because the plugin is a premium plugin so I can't find source codes of mmoitems plugin. I think you should ask that issue to the plugin's owner.


fix deps and test cleanup

Codecov Report
Base: 92.76% // Head: 81.79% // Decreases project coverage by -10.97% :warning:

Coverage data is based on head (01baac2) compared to base (f39eac8).
Patch coverage: 61.15% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #46       +/-   ##
===========================================
- Coverage   92.76%   81.79%   -10.98%
===========================================
  Files          48       56        +8
  Lines        1867     2851      +984
===========================================
+ Hits         1732     2332      +600
- Misses        135      519      +384

Impacted Files
Coverage Δ

src/morphometrics/_gui/_qt/labeling_widgets.py
27.17% <20.26%> (-31.65%)
:arrow_down:

...c/morphometrics/_gui/_qt/multiple_viewer_widget.py
22.28% <22.28%> (ø)

src/morphometrics/label/image_utils.py
88.07% <56.00%> (-9.63%)
:arrow_down:

...morphometrics/_gui/label_curator/label_cleaning.py
59.25% <59.25%> (ø)

.../morphometrics/_gui/label_curator/label_curator.py
87.59% <87.59%> (ø)

src/morphometrics/_gui/models/labeling_model.py
92.39% <93.10%> (+0.72%)
:arrow_up:

src/morphometrics/label/stitch.py
96.10% <96.10%> (ø)

...morphometrics/_gui/label_curator/label_painting.py
96.42% <96.42%> (ø)

...s/_gui/label_curator/_tests/test_label_cleaning.py
100.00% <100.00%> (ø)

...cs/_gui/label_curator/_tests/test_label_curator.py
100.00% <100.00%> (ø)

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Incorrect state data appearing in the .csv records

Preliminary - there appears to be a bug in the main state machine where after a period of time in the BRN_ON state something happens to the organization of the file and unexpected characters begin appearing in the file - almost as if the state has jumped to an undefined condition.  Need to reproduce, understand and fix.
There has also been an instance of a badly formed file that is apparently correlated with a hardware failure where the i2c bus became inoperable (bus #1).  It would be good if the program can gracefully handle a hung i2c bus

Multiple minor corrections made.  As of now, state machine is working per design intent


fix #1352

fix https://github.com/aspnet/Blazor/issues/1352

Thanks for submitting this. We'd also need an end-to-end (E2E) test for the new behaviour, perhaps similar to this: https://github.com/aspnet/Blazor/blob/master/test/Microsoft.AspNetCore.Blazor.E2ETest/Tests/RoutingTest.cs#L86

@SteveSandersonMS, I'll try to do E2E tomorrow.

 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: cores-system sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Unfortunately I did not find the optimal way to check _self.

Any thoughts on how to do it better?

Updated the code for the solution of problem https://github.com/aspnet/Blazor/issues/1352
Issue https://github.com/aspnet/Blazor/issues/1367 можно отметить в Backlog

I think the error is not related to the last update


Refine 'zsh-completion', fix typos

Hello, @graysky2 ! :smiley:
Appreciates your great work!
This pull request fixes typos in file zsh-completion which is imported from https://github.com/graysky2/profile-sync-daemon/pull/106 .
For the same reason in https://github.com/graysky2/anything-sync-daemon/pull/17 .
All the best wishes for you! :heart_decoration:

Good eye, thanks.


How do I run it?

Is it something like this?
git clone https://github.com/CMahaff/lasim.git
cd lasim
make
./lasim

I'll take an action to document this in the README later.
The super short version is you'll need to get the latest versions or rust and cargo, clone the repo, and then do a "cargo r" to run the project in debug mode or "cargo b" to build the executable.
The easier route is just grabbing a pre-built binary from the Releases page, or the GitHub Actions page vs building from source - if you are comfortable with that.

I asked the AI and it managed to write a PKGBUILD with your instructions thanks.

PKGBUILD for lasim
Here is a PKGBUILD file for the lasim project, which can be used to build the package on Manjaro:
pkgname=lasim
pkgver=0.1.1
pkgrel=1
pkgdesc="Move your Lemmy settings from one account to another"
arch=('x86_64')
url="https://github.com/CMahaff/lasim"
license=('MIT')
depends=('rust' 'cargo')
makedepends=('git')
source=("$pkgname-$pkgver.tar.gz::https://github.com/CMahaff/lasim/archive/refs/tags/v$pkgver.tar.gz")
sha256sums=('SKIP')

build() {
  cd "$srcdir/$pkgname-$pkgver"
  cargo build --release
}

package() {
  cd "$srcdir/$pkgname-$pkgver"
  install -Dm755 "target/release/$pkgname" "$pkgdir/usr/bin/$pkgname"
  install -Dm644 "LICENSE" "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
}

This PKGBUILD file will download the lasim source code from the GitHub repository, build it using Rust and Cargo, and install the resulting binary and license file to the appropriate directories.
To build and install the package, save the PKGBUILD file to a directory, navigate to that directory in the terminal, and run the following command:
makepkg -si

This will build the package and install it using pacman. After the installation is complete, you can use lasim as described in the project's README.
Please note that this PKGBUILD is specifically for the lasim version 0.1.1. If a new version is released, you'll need to update the pkgver variable and adjust the source and sha256sums accordingly.
Citations:
[1] https://forum.manjaro.org/t/howto-read-a-pkgbuild/116900
[2] https://wiki.archlinux.org/title/creating_packages
[3] https://forum.manjaro.org/t/how-to-compile-a-package-from-source/58064
[4] https://doc.rust-lang.org/cargo/
[5] https://www.grammy.com/artists/lasim-richards/13997
[6] https://github.com/CMahaff/lasim
[7] https://wiki.manjaro.org/index.php/Makepkg
[8] https://wiki.archlinux.org/title/PKGBUILD
[9] https://forum.manjaro.org/t/how-to-build-a-package-from-source-on-manjaro/53961
[10] https://doc.rust-lang.org/book/ch01-03-hello-cargo.html
[11] https://www.ftc.gov/news-events/news/press-releases/2023/03/ftc-approves-final-order-against-lca-vision-halting-alleged-bait-switch-advertising-lasik-laser-eye
[12] https://github.com/CMahaff
[13] https://wiki.manjaro.org/index.php/Create_Manjaro_Packages
[14] https://itsfoss.com/create-pkgbuild/
[15] https://forum.manjaro.org/t/howto-fix-a-failing-aur-package-build/104668
[16] https://doc.rust-lang.org/cargo/getting-started/installation.html
[17] https://www.ftc.gov/news-events/news/press-releases/2023/01/ftc-order-requires-lasikplus-pay-its-bait-switch-eye-surgery-ads
[18] https://github.com/CMahaff/lasim/issues
[19] https://forum.manjaro.org/t/help-with-a-pkgbuild-install/135512
[20] https://youtube.com/watch?v=crnGzF43aoc
[21] https://wiki.manjaro.org/index.php/Build_Manjaro_ISOs_with_buildiso
[22] https://crates.io
[23] https://www.allaboutjazz.com/musicians/lasim-richards
[24] https://github.com/topics/account?o=desc&s=updated
[25] https://forum.manjaro.org/t/pacman-and-custom-pkgbuild/115672
[26] https://www.arcolinuxiso.com/creating-a-pkgbuild-from-scratch-and-putting-it-on-aur/
[27] https://operavps.com/docs/manjaro-package-manager/
[28] https://youtube.com/watch?v=DHMTfWCnYcs
[29] https://jaxsta.com/profile/14203a75-d4f6-48d7-a49f-afa70cf86461/catalogue
[30] https://github.com/Lasim
[31] https://forum.manjaro.org/t/need-help-creating-a-pkgbuild/118996
[32] https://www.arcolinuxiso.com/creating-a-pkgbuild-from-scratch-and-putting-it-online-on-aur-from-a-till-aur/
[33] https://rust.fandom.com/wiki/Cargo_Ship
[34] https://www.mereommig.dk/comment/11426
[35] https://lemmy.eus/post/193097?scrollToComments=true
[36] https://www.mankier.com/5/PKGBUILD
[37] https://github.com/rust-lang/cargo
[38] https://lemmy.trevor.coffee/post/6468
[39] https://wiki.manjaro.org/index.php/PKGBUILD
[40] http://web.mit.edu/rust-lang_v1.25/arch/amd64_ubuntu1404/share/doc/rust/html/cargo/print.html
[41] https://lemmy.trevor.coffee/comment/33669


Revert "BZ1275362 - Change default thread pool rejection policy to "a…

Hi Bela,
This changes was not approved by PM :( so I've been asked to revert it. Here is a PR for it so.
(ps: my first PR with revert, so I hope I "did it right")
BZ1275362 -  (6.4.z) Change default thread pool rejection policy to "abort"

Why was this not approved?
Anyway, fine with me, as this doesn't affect master.

Reverts https://github.com/belaban/JGroups/pull/244


Add debian packages configuration

Should be able to install REST, openstack and docker workers and shell via *.deb.

https://github.com/vania-pooh/perspective-deb


Added model classes

Description
Added model classes for different UI components that hold required info to render the elements of SmallImagetemplate.
Added following components

AepText
AepButton
AepImage
AepColor
AepFont
AepUiConstants
Added placeholder classes AepTextComposable, AepButtonComposable, SmallImageCard to implement and take references on how to use model.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Variable naming and other suggestions will take in a future PR.
Merging


infra: edge zones

Description
Doc PR: https://github.com/apache/cloudstack-documentation/pull/296
Edge zones will be light-weight advanced zones, typically with a single hypervisor host.
Currently, supported only for KVM hypervisor due to direct-download template requirement.
https://events.hubilo.com/cloudstack-collaboration-conference-2022/session/172619
https://www.slideshare.net/ShapeBlue/edge-zones-in-cloudstack
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6840 (SL-JID-2539)

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6840 (SL-JID-2540)

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  SystemVM template(s). I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4560

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6840 (SL-JID-2604)

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4599

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. LL-JID 192

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-32)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41982 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6840-t32-kvm-centos7.zip
Smoke tests completed. 102 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers
Failure
418.38
test_vpc_redundant.py

test_01_redundant_vpc_site2site_vpn
Failure
742.55
test_vpc_vpn.py

test_01_vpc_site2site_vpn_multiple_options
Error
1281.05
test_vpc_vpn.py

test_01_vpc_site2site_vpn
Error
549.71
test_vpc_vpn.py

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6840 (LL-JID-233)

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4645

@blueorangutan ui

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/6840 (QA-JID-11)

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4830

@blueorangutan package

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15.  It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_multiplication_x: debian :heavy_check_mark: suse15. SL-JID 4961

Trillian Build Failed (tid-5505)

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5011

@blueorangutan test

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-5547)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42278 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6840-t5547-kvm-centos7.zip
Smoke tests completed. 104 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_08_upgrade_kubernetes_ha_cluster
Failure
552.59
test_kubernetes_clusters.py

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5054

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5059

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5069

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5077

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5186

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5188

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5206

@blueorangutan test

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-5782)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 46658 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6840-t5782-kvm-centos7.zip
Smoke tests completed. 105 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_08_upgrade_kubernetes_ha_cluster
Failure
568.75
test_kubernetes_clusters.py

test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers
Failure
467.72
test_vpc_redundant.py

@blueorangutan package

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5231

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5259

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@DaanHoogland I could've merged main myself

sure @shwstppr but you are enjoying your evening of quality time. there where no real conflicts.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5387

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5404

LGTM

@shwstppr, the documentation only explain how to configure this kind of zone; could you explain better the use case and how it will work? Not everyone can access the link https://events.hubilo.com/cloudstack-collaboration-conference-2022/session/172619.

@GutoVeronezi I've added the link for the slides from CCC.
A couple of things mentioned in the slides such as CKS support won't be available in this iteration

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5436

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@GutoVeronezi I've added the link for the slides from CCC. A couple of things mentioned in the slides such as CKS support won't be available in this iteration

Thanks, @shwstppr. Could we pinpoint in the PR description the main functionalities that will be available in this iteration?

Trillian test result (tid-6027)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 46097 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6840-t6027-kvm-centos7.zip
Smoke tests completed. 106 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_02_upgrade_kubernetes_cluster
Failure
547.76
test_kubernetes_clusters.py

@GutoVeronezi updated description

Thank you, @shwstppr.


Fixes fatal error on creating new API keys.

Changes
Quick bug fix. I cleverly named two variables $key and expected Laravel to figure it out.
How should this be tested?
I added a test for this endpoint (which fails before applying the fix, and is green now). :traffic_light:

For review: @angaither or @weerd

and expected Laravel to figure it out.

:+1:

cool! :+1:


Segmentation fault on ionic serve

Short description of the problem:
I get a segmentation fault on ionic serve.
What behavior are you expecting?
ionic serve to finish.
Steps to reproduce:

npm install -g cordova ionic@beta
ionic start newapp sidemenu --no-cordova
cd newapp
ionic serve

> ionic-app-scripts build

[18:58:33]  ionic-app-scripts 1.3.6
[18:58:33]  build dev started ...
[18:58:33]  clean started ...
[18:58:33]  clean finished in 4 ms
[18:58:33]  copy started ...
[18:58:33]  transpile started ...
[18:58:36]  transpile finished in 3.21 s
[18:58:36]  preprocess started ...
[18:58:36]  deeplinks started ...
[18:58:36]  deeplinks finished in 19 ms
[18:58:36]  preprocess finished in 21 ms
[18:58:36]  webpack started ...
[18:58:37]  copy finished in 3.60 s
[18:58:47]  webpack finished in 10.74 s
[18:58:47]  sass started ...
Segmentation fault

npm ERR! Linux 4.9.13-moby
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "run" "build"
npm ERR! node v7.2.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: `ionic-app-scripts build`
npm ERR! Exit status 139
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'ionic-app-scripts build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the webapp package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ionic-app-scripts build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs webapp
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls webapp
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /webapp/npm-debug.log

npm-debug.log
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node', '/usr/bin/npm', 'run', 'build' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild', 'build', 'postbuild' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prebuild, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/bin/node-gyp-bin:/webapp/node_modules/.bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
10 verbose lifecycle<EMAIL_ADDRESS>CWD: /webapp
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'ionic-app-scripts build' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 139  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec build script
14 verbose stack Error<EMAIL_ADDRESS>build: `ionic-app-scripts build`
14 verbose stack Exit status 139
14 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/lib/utils/lifecycle.js:255:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/lib/utils/spawn.js:40:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:885:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd /webapp
17 error Linux 4.9.13-moby
18 error argv "/usr/bin/node" "/usr/bin/npm" "run" "build"
19 error node v7.2.1
20 error npm  v3.10.10
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>build: `ionic-app-scripts build`
22 error Exit status 139
23 error Failed at the<EMAIL_ADDRESS>build script 'ionic-app-scripts build'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the webapp package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     ionic-app-scripts build
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs webapp
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls webapp
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Which @ionic/app-scripts version are you using?
1.3.6

I have the same issue

I found the source of the problem with building / serving Ionic Apps on Alpine Linux. The problem is in the recursive usage of the function str-replace in ionic-angular/themes/ionic.functions.scss. On Alpine, it generates segmentation fault after reaching a recursion depth of about 10 (at least on my systems).
There is an (closed) issue at libsass https://github.com/sass/libsass/issues/1823 that describes this phenomenon. There are also two alternative implementations of str-replace that seem to work.
After patching the file ionic.functions.scss, I can build Ionic apps on Alpine Linux without any problems.
Since the current implementation of str-replace in Ionic would lead to similar problem on other platforms (im there are enough spaces in the inline svg), I suggest that the problem should be fixed within the Ionic Framework.

I'm also running into this problem when trying to test my app inside an Alpine Linux container. Any status update on this?

Hi.. any update on this??.. I am facing same issue. Please find below configuration details.
Os : Redhat Linux
Ionic : 3.18.0
ionic-app-scripts: 3.0.0


Code release?

Do you have any plan to release your code? Looking forward to your reply! Thank you!

Thanks for your interest! We plan to release the code in November.


This pull request is for testing purposes

Ignore this.

Can one of the admins verify this patch?

retest this please

retest this please

retest this please

retest this please

retest this please


swipe to dismiss removes text from random list items

I'm using a DynamicListView with an ArrayAdapter that implements Swappable.
The list is setup with the following:
list.enableDragAndDrop();
list.setOnItemLongClickListener(new OnItemLongClickListener() {...} // used for swapping
list.enableSwipeToDismiss(new OnDismissCallback() {...});
list.setOnItemClickListener(new AdapterView.OnItemClickListener() {...}

When I swipe away an item, sometimes random rows will lose their text.  The rows have two TextViews, an ImageView, and a ProgressBar.  Only one of the TextViews will ever lose its text on any given row after a swipe to dismiss.
The onDismiss callback looks like this:
public void onDismiss(final ViewGroup listView, final int[] reverseSortedPositions) {
    for (int position : reverseSortedPositions) {
        queue.remove(position);
        adapter.notifyDatasetChanged();
    }
}

@Sahasrara Could you elaborate more on your workaround?

Hey there, I'm sorry to say it's been so long since I last worked on android that I forgot what the issue was exactly.  I did eventually solve whatever issue I was having without a workaround and I can point you to the project I was working on.  The class in question is QueueAdapter.
https://github.com/Sahasrara/Subtle/blob/master/Subtle/src/com/example/subtle/QueueAdapter.java


Mob Kill Stats

Hello, any chance you can add a display on the player page that shows how many of each mob a player has killed.

side note, might also be nice to have rarer mobs as an option for highscores and topplayers.
Eg dragons etc.


StrongNamed version of the assembly

Is your feature request related to a problem? Please describe.
A codebase with a lot of use of InternalsVisibleTo so all assemblies must be signed.
Describe the solution you'd like
Either sign the final assembly or have a signed nuget version as some other libs do.
Describe alternatives you've considered
Is not possible to load a non signed assembly from a signed assembly, at compile time at least...
Additional context

Hello,
I'm bumping this issue, as we're about to start the work on a new CLI tool, and we'd really like to base it on Spectre.Console.
Is there any way we could contribute to make this happen? If the only blocker is time, then we could push a PR to update your publish pipeline.

@kevingosse Would a tool such as StrongNamer help in this situation?
Afaik there is no benefit of strong naming .NET Core assemblies since the signature isn't validated or used for assembly binding. I think we would need to have a conversation within the team about the pros and cons when it comes to providing this.

I have no prior experience with StrongNamer but I'll give it a try 👍

Main benefit is that users who have to use a strong named assembly can use yours.
https://docs.microsoft.com/en-us/dotnet/standard/library-guidance/strong-naming
Until nobody is using it anymore I'd recommend to ship the package strongly named. It's as easy as adding the .snk file to the repos and add a tag in the csproj.

+1 to @sebastienros' comment.

Afaik there is no benefit of strong naming .NET Core assemblies since the signature isn't validated or used for assembly binding

@patriksvensson The Spectre.Console package also targets .NETStandard 2.0, which means it's designed to be useable to a .NET Framework application as well. It's quite common for a .NET Framework application to be strong named, and thus it makes it unnecessarily hard to use this package for a .NET Framework application.
In .NET Core world, there may not be benefit from the technical perspective, but there is also no harm to have it strong named. And for applications that still have to be strong named (e.g. PowerShell), it also makes it a lot easier to depend on Spectre.Console. So, overall, I think there is benefits to have this assembly strong named. Please re-consider doing this, thank!

Any updates on this? Am planning to use this for a dotnet commandline tool prototype

Any updates on this issue?

@daxian-dbw I think there is a package named StrongNamer on Nuget that can be used to convert any library to strong named, maybe you can try that?


feat(release): add automatic git tagging

uses command line for now, to avoid introducing new dependencies and get something working now
currently can not be disabled, always uses the default, needs likely discussion what to do here, see todo in the code
command is configurable through tagCmd
no signing, could be added by everyone who wants, by modifying the tagCmd

Ref #157

Do we really need a separate tagCmd? Can we not just run two commands in the release command hook?

Please don't change default behaviour of gx release (don't do auto-tagging). It might break integrations which do tagging separately. (I think it's probably fine as it is now, assuming it doesn't tag when using legacy package.json without the tagCmd defined).

I'm definitely not suggesting that we replace existing releaseCmd definitions, just that new package.json files (generated by gx init) should have new releaseCmd definitions that auto-tag. The tricky part is running two commands without, e.g., depending on bash (although I guess that may be fine).

Now that we're introducing different tag namespaces to distinguish gx vs. gomod version lineages, we need gx to create the tag to mitigate the chance of human error.
My concern with piling the tag command into the releaseCmd is that we'd need to go back and rewrite all package.json files of all repos to opt into tagging.
On the contrary, the tagCmd solution with a default fallback is immediately applicable to all repos.
BTW – gx should remind the user to use the --tags option when pushing, if a tag was created.
cc @Kubuxu

That assumes that the user has a git-based releaseCmd. If not, this'll end up tagging the commit before  the package changes are committed.
I'm perfectly happy to mass-update the releaseCmd if we merge https://github.com/whyrusleeping/gx/pull/227.

@Stebalien I approved that PR. If we're OK with the rewrite, then I'm fine as long as we explicitly echo "please push with --tags" as a reminder because I can see us tripping up more than once.

SGTM. So:
git commit -a -m \"gx publish $VERSION\" && git tag -as -m \"gx publish $VERSION\" gx/v$VERSION && echo \"Please remember to push with '--tags'\"
?

I'd state the hash in the tag message, as the version number is redundant because it's part of the tag name.

So, the tag message is often (e.g., on GitHub) used as the tag's title. However, we should probably put the hash on the second line of the message. We'll need https://github.com/whyrusleeping/gx/pull/228 for that.


node created by karpenter, cannot pull ACR image.

Version

Karpenter Version: v0.4.0

Kubernetes Version: v1.27.9
Expected Behavior
node can pull ACR image.
Actual Behavior
Events:
Type     Reason            Age                   From               Message

Warning  FailedScheduling  12m                   default-scheduler  0/10 nodes are available: 10 Insufficient nvidia.com/gpu. preemption: 0/10 nodes are available: 10 No preemption victims found for incoming pod..
Warning  FailedScheduling  11m                   default-scheduler  0/10 nodes are available: 10 Insufficient nvidia.com/gpu. preemption: 0/10 nodes are available: 10 No preemption victims found for incoming pod..
Normal   Scheduled         7m5s                  default-scheduler  Successfully assigned default/nginx-deployment-5bbff74cf8-79pmf to aks-t4-spot-pgpck
Normal   Nominated         8m9s (x3 over 12m)    karpenter          Pod should schedule on: nodeclaim/t4-spot-npr4z
Normal   Pulling           5m45s (x4 over 7m5s)  kubelet            Pulling image "ottoacr.azurecr.io/nginx:v1"
Warning  Failed            5m45s (x4 over 7m5s)  kubelet            Failed to pull image "ottoacr.azurecr.io/nginx:v1": rpc error: code = Unknown desc = failed to pull and unpack image "ottoacr.azurecr.io/nginx:v1": failed to resolve reference "ottoacr.azurecr.io/nginx:v1": failed to authorize: failed to fetch anonymous token: unexpected status from GET request to https://ottoacr.azurecr.io/oauth2/token?scope=repository%3Anginx%3Apull&service=ottoacr.azurecr.io: 401 Unauthorized
Warning  Failed            5m45s (x4 over 7m5s)  kubelet            Error: ErrImagePull
Warning  Failed            5m18s (x6 over 7m4s)  kubelet            Error: ImagePullBackOff
Normal   BackOff           118s (x20 over 7m4s)  kubelet            Back-off pulling image "ottoacr.azurecr.io/nginx:v1"

Only classic nodepool can pull ACR image, because the managed identity bind vmss.
But karpenter node don't have any identiy, karpentermsi don't bind it.
Steps to Reproduce the Problem
az aks update -n karpenter -g karpenter  --attach-acr ottoacr
Resource Specs and Logs
Back-off pulling image "ottoacr.azurecr.io/nginx:v1"
Community Note

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

Are you using self-hosted Karpenter or NAP? If NAP, I am wondering if this could be a duplicate of https://github.com/Azure/karpenter-provider-azure/issues/197 (fixed, but not rolled out yet).

Are you using self-hosted Karpenter or NAP? If NAP, I am wondering if this could be a duplicate of https://github.com/Azure/karpenter-provider-azure/issues/197 (fixed, but not rolled out yet).

I use self-hosted Karpenter, can updated the v0.4.0 version.

I think you are right, we should be propagating at least agentpool identity into node identities. Here is a draft PR which has the updated create-values and karpenter-values-template: https://github.com/Azure/karpenter-provider-azure/pull/301. I won't be able to test it until later, but give it a try if you have a chance.

@ottodeng We are fairly certain #301 fixes this, please retest (and re-open this issue if needed)

Hello,
I have updated to version 0.5.0, and the managed identity has been added to the VM. However, the issue persists. The node created by Karpenter still cannot pull the ACR image and shows a 401 error.
Here are the steps I've taken:

Updated Karpenter to version 0.5.0.
Verified that the managed identity is added to the VM.

Despite these steps, I am still encountering the same issue. Could you please provide further assistance on how to resolve this?

(discussed in https://github.com/Azure/karpenter-provider-azure/issues/411)


NEO sent to NEO WALLET has not arrived.

Transaction appears at Neotracker but does not appear in NEO WALLET - shows zero balance.
My receive address: AbTFuqir5bcmWdmbBWEBNtqnkpTCy5WeZB
How long should this transaction take?
When I search the address at Neotracker under the heading First transaction it states UNSPENT.
What does UNSPENT mean?
If the address can be seen in all the Block Explorers why doesn't it appear or show in my NEO-WALLET!?
Please help.
Many thanks and greetings from south korea
+) https://neotracker.io/address/AbTFuqir5bcmWdmbBWEBNtqnkpTCy5WeZB

hi there , i bought Neo from coinex and sent to Infinito wallet . it is not shown my balance but Neotracker shoed it.
what should i do?can i send it from online wallet of neotracker.io ?


Emagged Borg Sound Effect

About the PR

Added a sound effect when Borgs are Emagged

Media
Changelog
:cl:

add: Emagged Borg Sound Effect

cl is worded wrong

cl is worded wrong

How would it be worded?

How would it be worded?

Emagging borgs now plays a sound effect.


Add GCP digital asset to default

Website: https://globcoin.io/
Etherscan: https://etherscan.io/address/0xdb0f69306ff8f949f258e83f6b87ee5d052d0b23#code
Medium: https://medium.com/@globcoin_io/globcoin-will-have-a-new-token-sale-269624e1b544
Verified GCP Etherscan page: https://etherscan.io/token/0xdb0f69306ff8f949f258e83f6b87ee5d052d0b23

Traffic on the network, activity on GitHub, or community buzz

I'll reopen in September.


 I have also taken the chance to extend some of the preexisting tests and to slightly document some features.
Closes #1.

I am particularly unsure about test_string_to_pick_ignores_components. I am not sure it is even worth to test that. In case it is, I am not sure how to do it without getting into the mess of writing a test for each case.

For now, I have made the changes adding new commits. When we are happy I can rewrite the history.

I'm going to need to add anchors to the documentation test source. So it would be good if we could merge your fine work ASAP, before doing that.
Let's just clean up the history of what we have got so far, and merge. Then I think we'll move the whole documenting test source into the doc directory, and the thing you called exhaustive_test.py can simply be called dataflow_test.py and we can go to town on putting exhaustive tests in there. The slice tests should be moved there.

Ok!

I've reduced it to 14 commits. What about the points that were not dealt with yet? Do we leave them for another PR?

Yup. Let's get this in, so that the doc generation can be built on top of it.


Do we need to decay the learning rate when using RMSProp or Nesterov?

I'm trying to train the Inception-BN model on the ILSVRC12 dataset, and I have some questions on the training configurations.
I have heard that decaying the learning rate is necessary to get a good model accuracy. So how about using adaptive learning rate tuning algorithms, such as RMSProp, Nesterov, AdaDelta, or AdaGrad. Do they work well in this case? Also, when we use those adaptive learning rate tuning algorithms, do we still need to decay the learning rate?
Thanks!

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


BZ#1518056-Ensured tooltips do not show when drop downs are open

BZ https://bugzilla.redhat.com/show_bug.cgi?id=1518056
@miq-bot add_label bug
@miq-bot add_label gaprindashvili/yes

Before Screenshot

After Screenshot

Gaprindashvili backport details:
$ git log -1
commit 4943b4fbf7b394a6dd64333a60097be0a883b236
Author: Allen Wight<EMAIL_ADDRESS>Date:   Fri Dec 15 15:54:02 2017 -0500

    Merge pull request #1345 from chalettu/menu-tooltip-issue

    BZ#1518056-Ensured tooltips do not show when drop downs are open
    (cherry picked from commit 61db90ef9f07dd33ba1ce6a00f43af8e90788e32)

    https://bugzilla.redhat.com/show_bug.cgi?id=1527578

Fine backport details:
$ git log -1
commit 0dde1bb5dffec799ff3c76c882ec2390a05ec9f3
Author: Allen Wight<EMAIL_ADDRESS>Date:   Fri Dec 15 15:54:02 2017 -0500

    Merge pull request #1345 from chalettu/menu-tooltip-issue

    BZ#1518056-Ensured tooltips do not show when drop downs are open
    (cherry picked from commit 61db90ef9f07dd33ba1ce6a00f43af8e90788e32)

    https://bugzilla.redhat.com/show_bug.cgi?id=1536684

@chalettu Travis is failing in Fine branch:
/home/travis/build/ManageIQ/manageiq-ui-service/client/app/shared/custom-dropdown/custom-dropdown.component.js
  25:20  error  Missing trailing comma  comma-dangle

Guess the trailing comma isn't required in master/gaprindashvili, but required in fine branch? Can you please create a PR to fix it?


Cosmetic changes

Changes:

On the StatusCheck detail page:

Change timestamp format from Dec. 10, 2018, 9:52 p.m. to Dec 10, 2018, 21:52 UTC, mostly to add the timestamp.
Rename Took and Error columns to Runtime and Message, respectively.

Update the help_text and set the verbose_name for a few MetricsStatusCheckBase fields, as listed below.  These only affect the label for the field displayed in the statuscheck creation/update pages, and IMO are more intuitive for the end-user:

check_type verbose_name='Comparison operator'
warning_value verbose_name='Warning threshold'
high_alert_value verbose_name='Failure threshold'
high_alert_importance verbose_name='Failure severity'

No changes to logic.

@Aloshi good call, looks like this page can be updated to use the new terminology


EmptyObservable.js not found

Unhandled Promise rejection: (SystemJS) XHR error (404 Not Found) loading ..../node_modules/rxjs/Observable/EmptyObservable.js.
package.json
"rxjs": "^5.4.3",
ts file added the import
import { EmptyObservable } from 'rxjs/Observable/EmptyObservable';

The path is wrong. It should be
from “rxjs/observable/EmptyObservable“;
See https://github.com/ReactiveX/rxjs/blob/master/src/observable/EmptyObservable.ts

got fixed. Thanks


Implement nearest-depth upsampling

The VXGI script has the property diffuseResolutionScale used to scale-down the render resolution of indirect diffuse lighting pass, which reduces pixel shading cost. Bilinear filtering was used to upsample the result. This method produces blurry effect at the edges.
This commit replaces the old filtering method with nearest-depth upsampling method.

@jeske Here, I present to you the upsampling result when indirect diffuse lighting pass is rendered at different resolution scale. What do you think?
1/4 camera resolution

1/8 camera resolution

1/100 camera resolution

At first glance, I think the gather offsets are wrong here....
static int3 GatherOffsets[4] = {
        int3(0, 1, 0),
        int3(1, 1, 0),
        int3(1, 0, 0),
        int3(0, 0, 0)
      };

If this is the right source......
https://docs.microsoft.com/en-us/windows/desktop/direct3dhlsl/gather4--sm4-1---asm-

[gather] is the same as point sampling with (u,v) texture coordinate deltas at the following locations: (-,+),(+,+),(+,-),(-,-), where the magnitude of the deltas are always half a texel.

Then the gather offsets are:

(-0.5, 0.5)  (0.5,0.5)  (0.5,-0.5)  (-0.5,-0.5)

So I think you want this:
static int3GatherOffsets[4] = {
        int3(-1,  1, 0),     // lower-left
        int3( 1,  1, 0),     // lower-right
        int3( 1, -1, 0),     // upper-right
        int3(-1, -1, 0)     // upper-left
      };

Or maybe:

Looking at the pictures some more...
I don't think nearest-depth is going to work very well for this situation. Because the depths on both sides of these corners is going to be very similar.
I think nearest-normal or a weighted combination of depth-and-normal.... And if the error-distance exceeds some threshold, fallback to a blended value. (or add the pixel to a work-list that needs to be re-traced per-pixel)
Is the surface normal available?

@jeske Actually, I already have the texel 0.5 offset removed at the beginning of NearestNeighborTexel:
int3 texel = int3(mad(uv, sourceTexelSize.zw, -0.5), 0);

Then I cast the result to int3, which truncates the float to integer. This is the same as floor, thus, texel holds the sample position at (-,-). This is why the GatherOffsets looks like that.

Is the surface normal available?

Yes, it is.

And if the error-distance exceeds some threshold, fallback to a blended value.

And how do we determine this threshold?

Another important issue...
It's essential that we use abs(distance(...)), not signed distance values.

And if the error-distance exceeds some threshold, fallback to a blended value.

And how do we determine this threshold?

Good question. It's important to note that the error deltas depend on the diffuseResolutionScale. Because the smaller the indirect lighting texture, the more "off" the texel values will be.
I think a good place to start is just to compute this threshold based on the allowed amount of normal deviation per screen-space pixels. So something like:
if (bestNeighborNormalDelta > 10-degrees * diffuseResolutionScale) {
// fail to blending mode
}
Thresholding based on the depth value is complicated.... I think you would need to use the texel normal diffuseResolutionScale, and neighbor coordinate to project what you expect the neighbor texel depth value to be. Which sounds complicated.

The more I think about this, the more I think getting fancy about "upscaling" is the wrong approach. You should use a best-effort method based on depth and normal to decide if you can use the approximate value or not.. and if not, then put the pixel in a work-list for another pass to compute screen-pixel-accurate indirect lighting for the pixels where the approximation didn't fit.
There could also be a binning pre-pass, binning tiles based on whether the G-buffer normals and depth appear to be suitable for an approximation or not... and then handling the two tile types separately.

As for finding the best approximate texel... I'm thinking it looks something like this:
float normal_factor = 0.2;  // no idea what this should be
float bestNeighborDepthDelta = abs( distance(targetDepth, neightborDepths[i])  );
float bestNeighborNormalDelta = angle_delta(targetNormal, neighborNormals[i]);  // always positive
float texel_delta = bestNeighborDepthDelta +
                              bestNeighborNormalDelta * normal_factor;

Where angle_delta() returns the angle between the normals in radians or normalized on a 0.0 to 1.0 scale...

I don't think that this is the expected result. I don't know where I do it wrong. This implementation provides a good example on how this should be done.

I got this branch running to look at the problem.. The way the artifacts move when the camera moves gives some hints to the possible problem.
I tried to experiment with the code.. but I don't understand how to get Unity to reload the shader, or where the shader errors show up. So I got stuck. Let me explain what I'm thinking based on what I saw...
My current theory is that a single Gather won't work for this for two reasons. First, the current coordinate actually lies inside the current low-rez pixel. Because Gather only loads 4 adjacent pixels, this limits us to loading either up OR down and left OR right from the current pixel. This decision is being forced on us by the sub-pixel offset. Second, the current low-rez lighting pixels might be bad, in the case of a corner. We actually want to ignore this bad averaged lighting pixel, but Gather won't let us.
We are getting gathers that look like this, where "o" is the low-rez pixel our current pixel lies in.

Okay, so I know what I did wrong. I didn't assign the low resolution depth buffer with values 😅

I tried to experiment with the code.. but I don't understand how to get Unity to reload the shader, or where the shader errors show up. So I got stuck.

Unity will automatically reload the shader whenever you edit the shader and then focus on Unity window.

If you have any tips it would help me work on it.

Well, Unity has Frame Debugger. I suppose that you know it. I just fiddle with the texture values, for example, find the differences between 2 depth textures, and output the result to a render target. Then, I take a look at the frame debugger to see that render target. That's how I know where the problem is.

It reloaded the shader once for me.. but then I made a mistake, so the shader had errors, and I couldn't find them and I couldn't get it to reload the shader. Then I tried to build-and-run and it started showing the shader in the game window, but not in the scene window, and now I can't figure out how to get the shader running in the scene window again. I'll try some more.
Those pictures look much better.. The artifacts you see might be because of the quad-sample issue I explained above.. because it will cause issues depending on where the edge crosses the low-rez pixel boundary. If you see the artifacts move when you move the camera, I suspect that issue.

As for finding the best approximate texel... I'm thinking it looks something like this:
float normal_factor = 0.2;  // no idea what this should be
float bestNeighborDepthDelta = abs( distance(targetDepth, neightborDepths[i])  );
float bestNeighborNormalDelta = angle_delta(targetNormal, neighborNormals[i]);  // always positive
float texel_delta = bestNeighborDepthDelta +
                              bestNeighborNormalDelta * normal_factor;

Where angle_delta() returns the angle between the normals in radians or normalized on a 0.0 to 1.0 scale...

This is actually a good idea. Better yet, we can use dot product between the 2 normals. And then factor the depth with the dot product like so:
distance(targetDepth, neighborDepths[i] * dot(targetNormal, neighborNormals[i]));

The downside to this is that we also need an additional render texture to store the low resolution normals. Or we can infer the low resolution normal based on high resolution normal and the low resolution texel position, but this is less robust and more error prone.

After thinking for awhile, I came to the conclusion that both our methods of using normals to apply additional weights to the distance function is wrong. There are instances that they won't work out.
Anyway, I feel satisfied with the current algorithm. It produces nice result at 1/4 and 1/8 camera resolutions, which are the practical use cases.

It's great you have a good result. Here are some additional thoughts should you ever revisit this:
distance(targetDepth, neighborDepths[i]) / dot(targetNormal, neighborNormals[i]);

I see where you are going with this, but this seems like it is over-weighting the normal angle deviation contribution relative to distance. This could make a far-off distance value with a more similar normal win over a close distance value. I was suggesting including the normal-angle deviation as a very small contribution to just try and nudge similar distance values to prefer the one with a similar normal. Something more like:
distance(targetDepth, neighborDepths[i])  +
   (1.0 - dot(targetNormal, neighborNormals[i])) * normal_weight;

The downside to this is that we also need an additional render texture to store the low resolution normals. Or we can infer the low resolution normal based on high resolution normal and the low resolution texel position, but this is less robust and more error prone.

Ahh yes, I forgot that we would need the low-resolution normals. (and we'd need to test to make sure we didn't divide by zero)

After thinking and testing for awhile, I came to the conclusion that both our methods of using normals to apply additional weights to the distance function is quite wrong. There are instances where they won't work out.

Yes, there are instances where normals will not work and instances where distances will not work.
(a) Near a corner, looking at only the depth might pick the wrong side of the corner, because the depth on the opposite side of the corner might be closer.
(b) For geometry edges, several surrounding values might be far away in the background, in which case looking at only normals would be flawed, and depth must be used.
(c) For objects smaller than the low-rez texels, all methods will fail, because there are no valid low-rez texels to sample from.
(d) All of these are limited by Gather only sampling 4 texels, as I explained earlier.
(a) and (b) drove my idea to try to do a combination error, where we bump the distance value slightly by the normal angle deviation, as a way to try and prefer the similar distance value that also has a similar normal. and (c) drive my idea to look at a threshold to declare the low-rez sample was invalid and we would either blend values as a fudge, or compute a new value in an additional selective per-pixel fixup pass.


Added backers and sponsors from OpenCollective

now backers and sponsors will appear on your repo readme

I didn't want to move your site's url around, so I didn't do it, but I think the badges would look better altogether :)

Done in #45


Issues with ES Modules and @rollup/plugin-node-resolve

Describe the issue/behavior that seems buggy
While bundling my project using Rollup and the @rollup/plugin-node-resolve plugin, I'm encountering a problem related to importing languages from highlight.js as ES modules. Specifically, when I attempt to import the json, xml, and yaml languages with the following code:
    import json from 'highlight.js/lib/languages/json';
    import xml from 'highlight.js/lib/languages/xml';
    import yaml from 'highlight.js/lib/languages/yaml';

The @rollup/plugin-node-resolve plugin tries to resolve these imports by appending .js to the end of the import. However, highlight.js exports its modules with the .js extension in its package.json. This behavior results in the plugin attempting to import the json.js.js file, leading to a circular dependency error. I think it may be how the language files (ex. json.js.js) are trying to import the json.js modules themselves. There is some circular dependency going on. (SEE UPDATE BELOW)
Expected behavior
The expected behavior is for highlight.js to correctly resolve the imports for the languages (json, xml, yaml) when used with Rollup and the @rollup/plugin-node-resolve plugin.
Update:
I have found the root cause of the error. It has to do with the file naming that was decided on for the language files and the fact that @rollup/plugin-node-resolve adds a .js extension when trying to resolve 3rd party modules. I propose to remove the json.js.js file completely since it is deprecated (along with every other language file with a .js.js extension). These files are causing circular dependencies and preventing rollup from resolving correctly
//json.js.js

function emitWarning() {
    if (!emitWarning.warned) {
      emitWarning.warned = true;
      console.log(
        'Deprecation (warning): Using file extension in specifier is deprecated, use "highlight.js/lib/languages/json" instead of "highlight.js/lib/languages/json.js"'
      );
    }
  }
  emitWarning();
    import lang from './json.js'; // this import is causing circular dependencies and preventing rollup from resolving correctly
    export default lang;

since it is deprecated

I don't know what you mean.  I'm not aware this was any kind of pattern - I only added it because of the real life behavior of some builders - which point people to these wrong files... since you've obviously run into this it's clear bundlers are STILL doing this.
Are you saying we should change it to:
import lang from './json';

I'm not sure just removing the files fixes anything... then one will just get an error and not hav any guidance what to do with it... or what if we left the files in place but they only errored (explaining to the user what to do) - rather than attempting to recover from the error with a new import?

I too thought this would solve it, but it didn't:
import lang from './json';

I'm not sure how else I can help.
Right now I have a workaround by just copying the languages over to my project and registering them.

Can you make me a tiny project that reproduces and zip it up and maybe I'll take a look.

Closing for inactivity.


Add HTTP::Params#to_h

#to_h provides direct access to the underlying raw_params property. This allows to easily modify parameter data without going through (de-)serialization. HTTP::Params doesn't and shouldn't be mutable, but this feature can be useful, for example for proxy applications.
The implementation is identical to #7444 but renamed from #raw_params to #to_h.
Closes #7444
/cc @Sija

The name to_h looks to me like it'll create a new hash with the params instead of give me the "internal" field, maybe use another name like as_h to convey the intention of having access to the "internal" but as a hash?
Do you see what I mean?

I see what you mean, but the fact that HTTP::Params uses the same value internally to store the raw params is an implementation detail. We have lot's of #to_x methods in stdlib and the meaning is "return the equivalent x for self". Whether x needs to be constructed on demand or is already available is not important. Some methods like Array#to_a or String#to_s even just return self.
#as_x methods on the other hand mean "return self interpreted as x". This could potentially also apply here because HTTP::Params is essentially just a hash wrapper. But again, that's based on an implementation detail.
However, it might be worth consideration, if the returned hash should be duped to protect the immutability of HTTP::Params.

Why not just make raw_params not protected and call it a day, or at at the least keep it internal and define a #raw method.  There isn't any conversion going on so I don't see the point in doing #to_* or #as_*.
def raw
  @raw_params.dup
end

Conceptually, raw_params isn't a property of Params but an internal detail. Instead, Params can be considered having a dynamic set of properties, with 1 or more values each. That's provided by its API. So we're not adding a getter. We're adding a conversion method which just happens to return an instance variable. If for whatever reason the internal representation was changed (for example to an Array of KV-pairs) #to_h would still be expected to return Hash(String, Array(String)).
Surely we could call the method #raw or whatever else comes to mind. But it's not very intuitive (what does "raw" mean in this context?) and there is actually a perfectly fitting and commonly known name for such a method: #to_h.
There is also HTTP::Cookies#to_h which works exactly the same (including the dup

@straight-shoota Why closing?

It doesn't seem to be merged.


New strings for French locale

This PR adds new strings for the French language.
I am opening since it seems that #5 has been stuck in limbo for a while without any updates since last October.
This PR is small but adds plenty of user-facing strings that will simplify teaching new users (E.G. buttons at the top of the inventory that weren't translated before).

Thank you for the translations!


Bug of traci.vehicle.moveTo and traci.vehicle.moveToVTD

Hello,
I tried traci.vehicle.moveTo with different position parameters, but the car simply did not move.
And when I ran traci.vehicle.moveToVTD, it had an error:
The class traci.constants has no Constant property or Static method named 'VAR_MOVE_TO_VTD'.
Error in traci.vehicle.moveToVTD (line 12)
traci.beginMessage(constants.CMD_SET_VEHICLE_VARIABLE, constants.VAR_MOVE_TO_VTD,...

Hello,
I want to get the leader and follower vehicle acceleration in the target lane .So ,first I add  traci.vehicle.subscribeContext
to get ego vehicle context information ,and then I add  traci.vehicle.getContextSubscriptionResults(egoID) to get the surround vehicle ID, but the output “ans” is 1*1Map ,the follow is my code:
clear
close all
clc
import traci.constants
scenarioPath=['F:\software\SUMO配置文件\bin\quickstart\quickstart.sumocfg'];
system(['sumo-gui -c ' '"' scenarioPath '"' ' --remote-port 8813 --start&']);
traci.init
SIM_STEPS = [1 900];
beginTime = SIM_STEPS(1);
duration =  SIM_STEPS(2);
endTime =  SIM_STEPS(1) +  SIM_STEPS(2) - 1;
for i = 1:duration
traci.simulation.step();
IDList{i,1} = traci.vehicle.getIDList();
for j=1:length(IDList{i,1})
accel{i,1}(j)=traci.vehicle.getAcceleration(char(IDList{i,1}(1:end,j)));
laneID{i,1}=traci.vehicle.getLaneID(char(IDList{i,1}(1:end,j)));
traci.vehicle.subscribeContext(char(IDList{i,1}(1:end,j)), traci.constants.CMD_GET_VEHICLE_VARIABLE,150,{traci.constants.VAR_POSITION, traci.constants.VAR_SPEED});
traci.vehicle.addSubscriptionFilterLeadFollow('1');
contextSubscriptionResults=traci.vehicle.getContextSubscriptionResults(char(IDList{i,1}(1:end,j)));
end
end
traci.close()
Can you give me some help and looking forward to your reply.

Hello,
I have got the same error as mentioned above when I ran traci.vehicle.moveToVTD:
The class traci.constants has no Constant property or Static method named 'VAR_MOVE_TO_VTD'.
I checked that the 'VAR_MOVE_TO_VTD' is not present in traci.constants
Can you please help me out, how to solve this issue?
Best regards,
Sai Krishna


Model support for collision meshes

The existing CMO and SDKMESH model loaders create IBs and VBs as USAGE_DEFAULT and then release the loaded geometry. This is a great match for rendering, but makes it difficult to do CPU-side triangle-based collision.
There should be a way to get Model to keep a shadow-copy or a parameter to inform the system that the loaded model is for collision and not intended for rendering...

D3DX's Mesh Intersect performed the following test.

Perform a ray-hit test against the bounding box of the mesh. If it missed, no hits. If it hit, further processing.
Allocate an array of Booleans with the same number of entries as the maximum vertex index in the mesh.
Generates a plane containing the test ray, and store if the vertex was 'in front' or 'behind' that test plane.
For each triangle in the mesh, first check to see if all vertices in the triangle are on the same 'side' of the test plane by looking in that array. If they are, skip testing that triangle because it can't hit the ray.
Compute triangle hit test. All hits are stored in an array in sorted order.

Could support by adding a 'flags' option to Model::CreateFrom* to include collision data in the form of two simple std::vector arrays in ModelMesh.


[FEATURE] Create a simple animation generator

Description
Hey guys i'm new to open source :), and I want to contribute to this project by adding a simple css animation generator that supports rotating objects, skewing them, fade-in and fade-out with the option to adjust the duration
Screenshots
No response

Wow, you just created your first issue on this project, that's so cool. Make sure to wait till you are allowed to work on this before you do and when you are allowed to, don't forget to mention this issue in your pull request [Automated]

Sure :)

If you want to work on this, let me know

Sure, I'd love to work on this. How can i start :)

Sure, I'd love to work on this. How can i start :)

Read this and this

Thankyou, I'll start :)

Hey @Dun-sin, I was wondering to what degree am I allowed to introduce my styles for the new animation generator. Should i prefer sticking to the old styles or is small change is allowed? :)
I'm adding a radio button group for choosing the types of animations

Hey @Dun-sin, I was wondering to what degree am I allowed to introduce my styles for the new animation generator. Should i prefer sticking to the old styles or is small change is allowed? :)
I'm adding a radio button group for choosing the types of animations

Depends, I can't really pick if I don't see what you are doing

Here's the change, I've introduced a new class for the radiobutton group

Also, a new style for the duration selection input and label

 Here's the change, I've introduced a new class for the radiobutton group

Sure, it's okay by me

Thanks man, it's gonna look really good now :)

:tada: This issue has been resolved in version 1.12.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix(search): use correct column name when getting an entity payload #176

PS: I reviewed the PR title


bbin ls fails if the bbin bin has a 0-length file

Why would the bin have a 0-length file? I don't know about others, but I use 0-length files, .e.g. .notar,  as flags in my dirs.
It's an easy fix. Add something like (> (fs/size %) 0) to the file filter in load-scripts (currently line 1510 in bbin). The change worked fine for me on a local copy of bbin.
Thanks for this cool util!

What's the error your getting, just out of curiosity? Can you paste a stack trace?

@borkdude my laptop is currently offline,  but I can provide a verbal stack trace.
str/split-lines in parse-script errs with a NullPointerException because  read-header returns a nil for a 0-length file. read-header returns a nil because .read returns -1, which isn't a nat-int?.
Arguably, a cleaner fix might be to change read-header from (when (nat-int? n) ...) to (if (nat-int? n) ... ""), so that it always returns a string, even for a 0-length file.

@jdjohnston Nice! Since you analyzed the root cause, perhaps you're willing to put a PR together? If not, I'll get it sooner or later!

I took a look at this issue and don't see why read-header has the (when (nat-int? n) ..) condition. You can create a string like this: (String. (byte-array (range 1000)) 0 0) which is just an empty string. Can we remove the condition @rads?

That sounds good to me. 👍

Don't forget, .read returns -1 when the file is 0-length. (Conflating
0-length with EOF?)
On Fri, Aug 23, 2024, 14:30 Radford Smith @.***> wrote:

That sounds good to me. 👍
—
Reply to this email directly, view it on GitHub
https://github.com/babashka/bbin/issues/88#issuecomment-2307603716, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD4V3RNQ4BEAOU6QNZBLWXLZS55UNAVCNFSM6AAAAABMVTGI3CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBXGYYDGNZRGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

How about (String. buffer 0 (max 0 n))  ?
n will be one of three values:

-1 => 0-length file
size of buffer => file size >= buffer size
some value < buffer size => 0 < file size < buffer size

Sorry, I got a little overwhelmed with other issues. Feel free to submit a PR if you'd like

@jdjohnston Perhaps you're able to test now that I've merged this

@borkdude Works for me after I remembeed to bb gen-script. Your fix is different than either of my suggestions :), but it works fine.
I suppose gen-script will be executed for the next release. Am I understanding the workflow correctly?
BTW, I do understand it is better to fix a root-cause than to "band-aid" a symptom. Thanks for the reminder. Next time, feel free to be more blunt. It won't hurt my feelings. :)

Returning an empty string was one of your suggestions right?

Published as 0.2.4

@borkdude I was largely joking. Absolutely no criticism intended. There are probably even more ways to return an empty string. If I truly wanted it done in a certain way, I should have made more effort to generate a PR.
It works. That's the important part.


Dependencies conflict

Hi, I'm trying to install your package on a fresh Laravel install (v 9.13) and also in an another package as a dependency.
I got this :
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires lucasdotvin/laravel-soulbscription ^2.0 -> satisfiable by lucasdotvin/laravel-soulbscription[2.0.0, 2.0.1].
    - lucasdotvin/laravel-soulbscription[2.0.0, ..., 2.0.1] require illuminate/contracts ^8.73 -> found illuminate/contracts[v8.73.0, ..., 8.x-dev] but these were not loaded, likely because it conflicts with another require.

And with dev dependencies, it seems that the orchestra/testbench package used is out to date.
Cheers.

Hey! Thank you for the contact! It was tested only with Laravel 8 indeed, so I'm gonna work on a minor release to update these dependencies and support the newer version. It must be published later today. 🤘

Awesome, keep on rockin' !

Hi, @jahazzz! I just published a release to update the dependencies. I've made some tests locally and I could install the package as a regular dependency in a fresh Laravel application and as a dev-dependency in a package. Let me know if there is something more I can help you with! 🤘


New Community JSON format

Hi @glouel,
I've ported the current Community strings to a new proposed JSON format. This format would make easier to integrate the strings with automated translation workflows (cf. JohnCoates/Aerial#715).
I've seen the work to look up the translations is done in PoiStringProvider but before we go into that, I'd like to have your approval on this new format. The main ideas:

The video IDs are used as main keys. This provides the SaaS i18n services a unique, nestable, directly lookable key that is common for every language.
The keys are nestable. This gives us the possibility to specify pointsOfInterest keys for some videos, and if needed any other key in the same namespace.
I've removed version (with this workflow we have the git and SaaS versioning at our disposal and AFAIK it hasn't been used) and language (the file itself defines the language) attributes.

With this new key oriented JSON we can easily import the file in any i18n SaaS service that will provide UI and multiple tools to translate the content, and options to collaborate with anyone that wants to help  to translate (example from Lokalise):

All the services I mentioned support a Push/Pull model, so once a translator has updated a translation we could simply "pull" the last versions of all the files from the CLI (again example from Lokalise):
> lokalise --config lokalise.cfg  d --type json --unzip_to Resources/Community --bundle_structure %LANG_ISO%.%FORMAT%

Requesting...
Remote https://s3-eu-west-1.amazonaws.com/lokalise-assets/files/export/526621525c507/a3f70/Aerial-locale.zip... OK
Local Aerial-locale.zip... OK
Unzipped Resources/Community, Resources/Community/en.json, Resources/Community/fr.json, Resources/Community/es.json OK

So from the point of view of code/resources loading everything is still local and no SaaS is pinged or referenced, preventing vendor-locking.
Let me know what you think.
PS: Do you have any way to test PoiStringProvider?. It would be great to test the new format.

Hi @glouel,
I've ported the current Community strings to a new proposed JSON format. This format would make easier to integrate the strings with automated translation workflows (cf. #715).

First, thanks a lot for looking into this @aitor ! I'm a bit under the weather right now so I'll keep this short and hopefully have a more throrough look tommorrow.

I've seen the work to look up the translations is done in PoiStringProvider but before we go into that, I'd like to have your approval on this new format. The main ideas:

The video IDs are used as main keys. This provides the SaaS i18n services a unique, nestable, directly lookable key that is common for every language.

This sounds much better indeed ! FYI the JSON structure was based on the one used by Apple.
This is more of a tooling thing probably, but because Apple changes the IDs of videos from time to time, we'll need to be able to do a couple of things hopefully to all the JSONs simultaneously. One is, renaming a key, second is, changing some subkeys :
"40": "Passing the Gherkin in the City of London, on the right",

In this, hopefully we can change 40 to 30 on all the JSONs. See what I mean ?

The keys are nestable. This gives us the possibility to specify pointsOfInterest keys for some videos, and if needed any other key in the same namespace.

Sounds good.

I've removed version (with this workflow we have the git and SaaS versioning at our disposal and AFAIK it hasn't been used) and language (the file itself defines the language) attributes.

Agreed

With this new key oriented JSON we can easily import the file in any i18n SaaS service that will provide UI and multiple tools to translate the content, and options to collaborate with anyone that wants to help to translate (example from Lokalise):

All the services I mentioned support a Push/Pull model, so once a translator has updated a translation we could simply "pull" the last versions of all the files from the CLI (again example from Lokalise):
> lokalise --config lokalise.cfg  d --type json --unzip_to Resources/Community --bundle_structure %LANG_ISO%.%FORMAT%

Requesting...
Remote https://s3-eu-west-1.amazonaws.com/lokalise-assets/files/export/526621525c507/a3f70/Aerial-locale.zip... OK
Local Aerial-locale.zip... OK
Unzipped Resources/Community, Resources/Community/en.json, Resources/Community/fr.json, Resources/Community/es.json OK

So from the point of view of code/resources loading everything is still local and no SaaS is pinged or referenced, preventing vendor-locking.
Let me know what you think.

I have to say this does look pretty good indeed and the changes are relatively small to the format so it should be trivial to modify PoiStringProvider. Unless I'm missing something, I see no drawback to moving to that format (there's the tooling/modifying key thing I mentioned above I guess).
Thanks a lot for looking into it !

PS: Do you have any way to test PoiStringProvider?. It would be great to test the new format.

You mean making the changes to it ? Should be quite easy, I'll have a look tomorrow if I start feeling a bit better. The whole thing looks a bit convoluted but it's actually pretty simple.

we'll need to be able to do a couple of things hopefully to all the JSONs simultaneously.

Yep, it's perfectly possible. The exact process is as just said tooling-dependant and varies depending on what SaaS we use but basically you'll pick the affected key and "rename" it. All the JSON will be regenerated on the next "pull" to include the new key, while keeping the translations already made for them.

You mean making the changes to it?

I was talking about unit testing the class, a proper test to be sure the new format is applied correctly. That way I could more easily make the changes and help you. (BTW: hope you get better soon, no hurry at all to implement this).

we'll need to be able to do a couple of things hopefully to all the JSONs simultaneously.

Yep, it's perfectly possible. The exact process is, as you said, tooling-dependant and varies depending on what SaaS we use but basically you'll pick the affected key and "rename" it. All the JSON will be regenerated on the next "pull" to include the new key, while keeping the translations already made for them.

Perfect thanks for clarification. This will be a great help.

You mean making the changes to it?

I was talking about unit testing the class, a proper test to be sure the new format is applied correctly. That way I could more easily make the changes and help you.

There's not much testing in the code right now, I'll give a shot to the new implementation in a branch. If the JSON is malformed you instantly crash so it's an easy debug ;)

(BTW: hope you get better soon, no hurry at all to implement this).

Thanks, not a 100% but getting there !

There's not much testing in the code right now, I'll give a shot to the new implementation in a branch.

@glouel Super, let me know then when the new format is adopted. I can then jump back in and finish establishing the workflow for the i18n process and porting the current translations to it. And let me know if I can help with porting the PoiStringProvider to the new format.

Porting to the new format was really easy, just changed a few lines, didn't deserve a branch, you can see the change here  : https://github.com/glouel/Aerial/commit/26eb394d8c65de194334295956df78f7b3cc3454#diff-7dd02df370ab6fa9a5ab9ac98a351ffe
So I made a test build with just the new en.json bundled, you can find it here :
https://github.com/glouel/Aerial/releases/tag/v1.4.7test8
I also attached my "old format" fr.json that was in progress if you want to play with it.
I don't know if you are aware as this is not really documented, but if you want to test a new format file, you can create a locale folder in your Aerial cache folder and put the json here, it will get loaded instead of the one in the bundle if there.
Let me know how it goes on your end ! As for the workflow, tooling and stuff, let me know if I need to create an account somewhere. I'll trust your judgment on the best one to use. Thanks !


cw721 addition (not from Stargaze Launchpad link) seems to have issue

As reported in the bugs channel in Discord, it sounds like The Strange Clan and Levana have run into and issue with this contract:
juno1uw3pxkga3dxhqs28rjgpgqkvcuedhmc7ezm2yxvvaey9yugfu3fq7ej2wv

I attempted to follow the normal process, using the button above, and could reproduce it.
TypeError: Cannot read properties of undefined (reading 'getTokenDetails')
    at getTokenDetails (/Users/mikepurvis/ic/starrybot-discord/src/astrolabe/index.js:27:36)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.getConfig (/Users/mikepurvis/ic/starrybot-discord/src/commands/tokenAdd/createTokenRule.js:30:29)
    at async Object.execute (/Users/mikepurvis/ic/starrybot-discord/src/utils/commands.js:46:9)
    at async Wizard.execute (/Users/mikepurvis/ic/starrybot-discord/src/wizardware/wizard.js:76:14)
    at async Wizardware.continue (/Users/mikepurvis/ic/starrybot-discord/src/wizardware/index.js:56:7)
    at async Client.interactionCreate (/Users/mikepurvis/ic/starrybot-discord/src/handlers/interactionCreate.js:46:4)
TypeError: Cannot read properties of undefined (reading 'name')
    at getTokenDetails (/Users/mikepurvis/ic/starrybot-discord/src/astrolabe/index.js:38:62)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.getConfig (/Users/mikepurvis/ic/starrybot-discord/src/commands/tokenAdd/createTokenRule.js:30:29)
    at async Object.execute (/Users/mikepurvis/ic/starrybot-discord/src/utils/commands.js:46:9)
    at async Wizard.execute (/Users/mikepurvis/ic/starrybot-discord/src/wizardware/wizard.js:76:14)
    at async Wizardware.continue (/Users/mikepurvis/ic/starrybot-discord/src/wizardware/index.js:56:7)
    at async Client.interactionCreate (/Users/mikepurvis/ic/starrybot-discord/src/handlers/interactionCreate.js:46:4)

Solved with https://github.com/starryzone/starrybot-discord/commit/fe36b4648d132aa50f654d22645795088496433c


Gemma models still not working.

Gemma models that have been quantized using Llamacpp are not working. Please look into the issue
error
"llama.cpp error: 'create_tensor: tensor 'output.weight' not found'"
I will open a issue on the llamacpp github aswell addressing this
https://github.com/ggerganov/llama.cpp/issues/5706
System:
Ryzen 5600x
rtx 3080 gpu
b550 motherboard
64gb ddr4 ram
windows 10 OS

Im uploading the model files for the merges if anyone wants to do some debugging. Should be in the next 10 hours or so. Sorry slow internet.
https://huggingface.co/rombodawg/Gemme-Merge-Test-7b


Deploying Metrics to AKS configuration

When deploying the metrics pods via lens to an AKS cluster the pods can't find nodes to run on.
Looking at it the pods have this node selector on them: kubernetes.io/os: linux
While AKS nodes are linux, they by default don't have this label on them causing the pod failure.
Cluster Type: Azure Kubernetes System (AKS)
Nodes: Linux aks-agentpool-XXXX 4.15.0-1060-azure #65-Ubuntu SMP Wed Sep 18 08:55:51 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Client OS: Windows 10
Error Events:
Event: prometheus-0.15ca241e30b1e6e7
Message 0/8 nodes are available: 8 node(s) didn't match node selector.
Created 1h 5m ago (2019-10-03T12:52:12Z)
Namespace lens-metrics
Reason FailedScheduling
Source default-scheduler
First seen 1h ago 2019-10-03T12:52:12Z
Last seen <1m ago 2019-10-03T13:57:15Z
Count 2931
Type Warning
Involved object Pod prometheus-0
Event: kube-state-metrics-54558ff88b-rfv2z.15ca241dac39ce92
Message 0/8 nodes are available: 8 node(s) didn't match node selector.
Created 1h 6m ago (2019-10-03T12:52:09Z)
Namespace lens-metrics
Reason FailedScheduling
Source default-scheduler
First seen 1h ago 2019-10-03T12:52:09Z
Last seen 1m ago 2019-10-03T13:57:14Z
Count 2931
Type Warning
Involved object Pod kube-state-metrics-54558ff88b-rfv2z
Once I removed the node selector from both the Prometheus statefulset and the Metrics deployment it works fine.

@jvandervelden Which version of AKS are you running? All my tests with AKS have set the beta.kubernetes.io/os: linux labels on the nodes. 🤔
Very surprising if the nodes do not set that as it is pretty "standard" node label set by kubelet itself.

You're right I looked again and they do have the beta.kubernetes.io/os: linux label.

So the problem is that kubernetes.io/os: linux is only available on some envs and in some there's beta.kubernetes.io/os: linux. We will tackle that in next release. (fix PR already merged)

This is fixed in 2.1.0 release.


chore: use canonical snapshots in official branches

What type of PR is this?
This PR enables VHD builds from official branches to use the same publisher base image version as when they were created and apt packages from the time they were created.
Snapshots are currently only supported for 22.04 and above, therefore this PR only effectively modifies build logic from official branch for 22.04 SKUs.
Builds from any other branch and/or master are still the same, this change is no-op for them.
Test build imitating an official branch: https://msazure.visualstudio.com/CloudNativeCompute/_build/results?buildId=97388020&view=results
publisher_base_image_version.json
{
"20_04-lts-cvm": "20.04.202402050",
"22_04-lts": "22.04.202407010",
"22_04-lts-gen2": "22.04.202407010",
"22_04-lts-arm64": "22.04.202407010",
"20_04-lts": "20.04.202406140",
"20_04-lts-gen2": "20.04.202406140"
}
vhd_build_timestamp.json
{"build_timestamp": "20240702T231545Z"}
Test build imitating a regular branch: https://msazure.visualstudio.com/CloudNativeCompute/_build/results?buildId=97387996&view=results

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.486%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2627

Relevant Lines:
3727

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.486%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2627

Relevant Lines:
3727

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.486%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2627

Relevant Lines:
3727

💛  - Coveralls


BUG A file-like object as arg to df.to_csv must have the method getvalue

Since pandas v0.23.1 (specifically from https://github.com/pandas-dev/pandas/pull/21300), if a file-like object is passed into df.to_csv, the object must have the method getvalue, or else an AttributeError is raised (which is what we have observed from #258). An io.TextIOWrapper object doesn't have the method getvalue, which caused the latest build failures in #258. This PR switches to io.StringIO instead.

AppVeyor has been turned on, but obviously there's no AppVeyor configuration yet (upcoming from #258) and hence the relevant build "failure", which should be ignored for this PR.


feat: add blocto wagmi connector

Summary

add adapter repo wagmi-connector
create BloctoConnector class extend from wagmi Connector
add bloctoWeb3ModalConfig  use for web3modal

The parts will be implemented in later Pull Requests
Related Links

Asana:  https://app.asana.com/0/1201812548509877/1203670481412050/f
Mockup:
Checklist

[x] Pasted Asana or Mockup link.
[x] Tagged labels.

Prerequisite/Related Pull Requests

Screenshot/Gif

And don't forgot add changeset using yarn changeset

The icon looks weird. Can you use the same icon with Blocto app?


Feat/use aave client lib in tests

Hello this is a little draft using aave client lib inside one of the tests.
The test succeeded so I guess this is a good start :tada:
Creating this pr so i can ask questions in code.

In theory the client should be capable of dositing eth directly (in fact at the moment aave app doesn't offer a way to directly deposit weth at all), but for some reason this didn't work - didn't yet figure out why.

Could you maybe move your changed script to a separate script file, and restore the original script to how it was?  And take the token to test as a command line argument.