api host with provisioner

This PR must be merged when new version of dbaas-costing has been released.

Hello @wmarquardt! Thanks for opening this PR.

In the file dbaas/api/host.py:

Line 15:1: E302 expected 2 blank lines, found 1
Line 154:1: W293 blank line contains whitespace
Line 155:32: E711 comparison to None should be 'if cond is None:'

Do see the Hitchhiker's guide to code style

Codecov Report

Merging #714 (2b27519) into master (ad8717d) will decrease coverage by 0.00%.
The diff coverage is 25.00%.

:exclamation: Current head 2b27519 differs from pull request most recent head 355aa15. Consider uploading reports for the commit 355aa15 to get more accurate results

@@            Coverage Diff             @@
##           master     #714      +/-   ##
==========================================
- Coverage   41.48%   41.48%   -0.01%
==========================================
  Files         341      341
  Lines       25185    25196      +11
==========================================
+ Hits        10449    10452       +3
- Misses      14736    14744       +8

Flag
Coverage Δ

unittests
41.48% <25.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

dbaas/api/host.py
49.19% <25.00%> (-2.14%)
:arrow_down:

Continue to review full report at Codecov.

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


upgrade to helm v3.8.2

What type of PR is this?

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

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

kotsadm now includes helm v3.8.2

Does this PR require documentation?

Hah if you look at the force-push you'll see it was me attempting to do the go mod changes and having things break 😆


deploy failed: qemu-kvm-one no such file or directory

Issue
Can't deploy Alpine or Ubuntu VM due to the following error:
...
Wed Dec  4 18:07:39 2024 [Z0][VMM][I]: error: Failed to create domain from /var/lib/one//datastores/0/3/deployment.0
Wed Dec  4 18:07:39 2024 [Z0][VMM][I]: error: Cannot check QEMU binary /usr/bin/qemu-kvm-one: No such file or directory

Description
One Sunstone fails to run a KVM VM due to a missing binary : qemu-kvm-one
If I install it manually with: sudo apt install -y qemu-kvm-one, it returns an error due to configuration failure.
Details

OS: Ubuntu 24.04.1
Affected Component: [e.g. Sunstone, Scheduler, or Storage] Core, KVM, Sunstone
Hypervisor: [e.g. KVM] KVM
Version: 6.10.0-1
Linstor Data Store

Download Ubuntu 22.04 Minimal from the Market place
Start a VM from the image
Specify Linstor system store for the VM
Run VM and find the Boot Failure Status with the error specified

Additional context
Pls find more details here: #6209

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

This symlink should be created in postinst script of opennebula-node-kvm package
/usr/bin/qemu-kvm-one: symbolic link to /usr/bin/qemu-system-x86_64

Was the package correctly installed?

This symlink should be created in postinst script of opennebula-node-kvm package
/usr/bin/qemu-kvm-one: symbolic link to /usr/bin/qemu-system-x86_64

Was the package correctly installed?

The package was installed and reinstalled carefully, according to your docs. The symlink was NOT created

This symlink should be created in postinst script of opennebula-node-kvm package
As per #6209 installing opennebula-node-kvm on Ubuntu 24.04.1 fails on post-install

To create the symlink you can run /usr/bin/qemu-kvm-one-gen

To create the symlink you can run /usr/bin/qemu-kvm-one-gen

Thanks, I fixed with ln -s. This fixed my issue. How about fixing opennebula-node-kvm post install ? :roll_eyes:


Requires llvmlite

Hey,
In a fresh conda environment, pip install -r requirements.txt failed.
It seems I need to install llvmlite first, e.g.
conda install -c numba llvmlite

numba and llvmlite should come as a dependency of librosa. Unfortunately they missed shipping the llvmlite wheels in time which is why pip fails
See here: https://github.com/numba/numba/issues/2918
This reminds that I will remove the requirements.txt as I like to move over to pipenv's Pipfiles. We an add a conda environment.yml as well if you like


Add Carthage support

Hi.
Carthage is becoming fairly popular, this PR adds support to it.
I've added a project that builds the library in an independent framework, you can test it by running:
carthage build --no-skip-current

Cheers

BTW, if you merge, you'll need to add a new tag version

This is awesome! I'll definitely have to start looking into Carthage. will merge and update the version


Rust quickstart appears incomplete

When trying the quickstart with Rust, I got several compilation errors:
error[E0433]: failed to resolve: use of undeclared crate or module `ndarray`
  --> src/main.rs:14:15
   |
14 |     let arr = ndarray::ArrayD::from_shape_vec(
   |               ^^^^^^^ use of undeclared crate or module `ndarray`

error[E0433]: failed to resolve: use of undeclared type `Tensor`
  --> src/main.rs:22:37
   |
22 |         .infer([("input_sequences", Tensor::<GenericStorage>::String(arr))])
   |                                     ^^^^^^ use of undeclared type `Tensor`
   |
help: consider importing this enum
   |
1  + use carton::types::Tensor;
   |

error[E0412]: cannot find type `GenericStorage` in this scope
  --> src/main.rs:22:46
   |
22 |         .infer([("input_sequences", Tensor::<GenericStorage>::String(arr))])
   |                                              ^^^^^^^^^^^^^^ not found in this scope
   |
help: consider importing this struct
   |
1  + use carton::types::GenericStorage;
   |

error[E0433]: failed to resolve: use of undeclared crate or module `ndarray`
  --> src/main.rs:15:9
   |
15 |         ndarray::IxDyn(&[1]),
   |         ^^^^^^^ use of undeclared crate or module `ndarray`

error[E0752]: `main` function is not allowed to be `async`
 --> src/main.rs:4:1
  |
4 | async fn main() {
  | ^^^^^^^^^^^^^^^ `main` function is not allowed to be `async`

Some errors have detailed explanations: E0412, E0433, E0752.
For more information about an error, try `rustc --explain E0412`.
error: could not compile `carton-test2` (bin "carton-test2") due to 5 previous errors

To resolve, I had to add a couple crates:
cargo add -F macros,rt-multi-thread tokio
cargo add ndarray

I also needed to make a few small code changes:
@@ -1,6 +1,9 @@
 use carton::Carton;
+use carton::types::GenericStorage;
 use carton::types::LoadOpts;
+use carton::types::Tensor;

+#[tokio::main]
 async fn main() {
     // Load the model
     let model = Carton::load(

I should note that although this was able to compile and start, it ran for a while and then produced the following error:
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', source/carton-runner-rust-bert/src/masked_language.rs:99:49
stack backtrace:
   0:     0x563d9870bc66 - std::backtrace_rs::backtrace::libunwind::trace::h66dc1c6acf794faa
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/../../backtrace/src/backtrace/libunwind.rs:93:5
   1:     0x563d9870bc66 - std::backtrace_rs::backtrace::trace_unsynchronized::ha80d20099a67f790
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2:     0x563d9870bc66 - std::sys_common::backtrace::_print_fmt::h7b959d43f35f16d4
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/sys_common/backtrace.rs:65:5
   3:     0x563d9870bc66 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::hdaa196410d9ee0b9
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/sys_common/backtrace.rs:44:22
   4:     0x563d982ea52f - core::fmt::rt::Argument::fmt::h0ddfbbe8be3f80d0
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/fmt/rt.rs:138:9
   5:     0x563d982ea52f - core::fmt::write::h66b3c629f3d623e4
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/fmt/mod.rs:1094:21
   6:     0x563d98709337 - std::io::Write::write_fmt::hb6d80fba4115e0c2
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/io/mod.rs:1714:15
   7:     0x563d9870ba75 - std::sys_common::backtrace::_print::h1a49cfb0cf3cce17
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/sys_common/backtrace.rs:47:5
   8:     0x563d9870ba75 - std::sys_common::backtrace::print::hca95c2d0055e42a2
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/sys_common/backtrace.rs:34:9
   9:     0x563d9870d073 - std::panicking::default_hook::{{closure}}::hc03c01c56bca600c
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:269:22
  10:     0x563d9870ce34 - std::panicking::default_hook::hb2cb5315b6634f1c
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:288:9
  11:     0x563d9870d566 - std::panicking::rust_panic_with_hook::h75cd912a39a34e8a
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:705:13
  12:     0x563d9870d444 - std::panicking::begin_panic_handler::{{closure}}::h1498b46f7849e167
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:595:13
  13:     0x563d9870c066 - std::sys_common::backtrace::__rust_end_short_backtrace::hd36a39b27b98086b
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/sys_common/backtrace.rs:151:18
  14:     0x563d9870d202 - rust_begin_unwind
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:593:5
  15:     0x563d97f12dd3 - core::panicking::panic_fmt::h98ef273141454c23
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/panicking.rs:67:14
  16:     0x563d97f12e63 - core::panicking::panic::hf53fd8b0bfa5848e
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/panicking.rs:117:5
  17:     0x563d9810b3d8 - core::option::Option<T>::unwrap::h12c43accf485d83d
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/option.rs:935:21
  18:     0x563d9810b3d8 - <carton_runner_rust_bert::masked_language::CartonMaskedLanguageModel as carton_runner_rust_bert::Model>::infer::h911637b12d75fc53
                               at /app/source/carton-runner-rust-bert/src/masked_language.rs:99:49
  19:     0x563d98235737 - carton_runner_rust_bert::main::{{closure}}::{{closure}}::haff9764bc00c2e36
                               at /app/source/carton-runner-rust-bert/src/main.rs:103:53
  20:     0x563d98235737 - core::option::Option<T>::map::h46bc3bc572d9135b
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/option.rs:1075:29
  21:     0x563d98235737 - carton_runner_rust_bert::main::{{closure}}::h22763948a8181bc0
                               at /app/source/carton-runner-rust-bert/src/main.rs:103:30
  22:     0x563d982315c1 - tokio::runtime::park::CachedParkThread::block_on::{{closure}}::hefce4379bf2999ac
                               at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.22.0/src/runtime/park.rs:272:63
  23:     0x563d982315c1 - tokio::runtime::coop::with_budget::h948ec6343d66ed06
                               at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.22.0/src/runtime/coop.rs:102:5
  24:     0x563d982315c1 - tokio::runtime::coop::budget::hb559616204cdf787
                               at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.22.0/src/runtime/coop.rs:68:5
  25:     0x563d982315c1 - tokio::runtime::park::CachedParkThread::block_on::h569f593b7d92f31c
                               at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.22.0/src/runtime/park.rs:272:31
  26:     0x563d98260bc4 - tokio::runtime::context::BlockingRegionGuard::block_on::h4f70676a06ac2c7b
                               at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.22.0/src/runtime/context.rs:255:13
  27:     0x563d98260bc4 - tokio::runtime::scheduler::multi_thread::MultiThread::block_on::hb1b4399a43fda72d
                               at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.22.0/src/runtime/scheduler/multi_thread/mod.rs:66:9
  28:     0x563d98260bc4 - tokio::runtime::runtime::Runtime::block_on::h9b65277f6e61fe63
                               at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.22.0/src/runtime/runtime.rs:281:45
  29:     0x563d981b89d3 - carton_runner_rust_bert::main::h9e59ea0bfb57c7cf
                               at /app/source/carton-runner-rust-bert/src/main.rs:30:5
  30:     0x563d981ad183 - core::ops::function::FnOnce::call_once::h7a80138e58956fe0
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/ops/function.rs:250:5
  31:     0x563d981ad183 - std::sys_common::backtrace::__rust_begin_short_backtrace::h8cf37ada6016b8f2
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/sys_common/backtrace.rs:135:18
  32:     0x563d981ad7c9 - std::rt::lang_start::{{closure}}::h2df01761492e40ba
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/rt.rs:166:18
  33:     0x563d98703555 - core::ops::function::impls::<impl core::ops::function::FnOnce<A> for &F>::call_once::h87e9a912d8a0de33
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/core/src/ops/function.rs:284:13
  34:     0x563d98703555 - std::panicking::try::do_call::ha8d57d42181c12cf
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:500:40
  35:     0x563d98703555 - std::panicking::try::h2fdb2d19c253437c
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:464:19
  36:     0x563d98703555 - std::panic::catch_unwind::h9fe5959d2e133449
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panic.rs:142:14
  37:     0x563d98703555 - std::rt::lang_start_internal::{{closure}}::ha072eb7bbeaa0b6a
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/rt.rs:148:48
  38:     0x563d98703555 - std::panicking::try::do_call::h983dae29f79ed5db
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:500:40
  39:     0x563d98703555 - std::panicking::try::h15014751f4b412ba
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panicking.rs:464:19
  40:     0x563d98703555 - std::panic::catch_unwind::hf837426183b1055f
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/panic.rs:142:14
  41:     0x563d98703555 - std::rt::lang_start_internal::h2bbe0b58b2b89a9f
                               at /rustc/d5c2e9c342b358556da91d61ed4133f6f50fc0c3/library/std/src/rt.rs:148:20
  42:     0x563d981b8aac - main
  43:     0x7fc5003e3083 - __libc_start_main
                               at /build/glibc-SzIz7B/glibc-2.31/csu/../csu/libc-start.c:308:16
  44:     0x563d97f52f0e - _start
  45:                0x0 - <unknown>

Thanks for trying it out!
Convenient timing as I just put up #155 which should update the quickstart to fix these issues. I'll update it to also describe the additional required dependencies.
If you change "input_sequences" in your code to "input" it shouldn't panic any more.
Generally, incorrect tensor names shouldn't cause a panic, but we'll clean that up as the library becomes more stable.
Happy to accept a PR if you'd like to contribute! The issue is here:
https://github.com/VivekPanyam/carton/blob/d92e847334e14446c8dee034a05ced57bb338d54/source/carton-runner-rust-bert/src/masked_language.rs#L98-L99

Just updated the PR to also include the additional required dependencies. Can you confirm this build of the quickstart guide looks good to you?
https://f0469c12.carton.pages.dev/quickstart
Thank you for the detailed feedback and fix! I appreciate it :)

@VivekPanyam This looks mostly alright aside from the missing dependency on env_logger. However, the program still panics when it runs even though it seems to get the correct output.
{"scores": Float([[14.551313]], shape=[1, 1], strides=[1, 1], layout=CFcf (0xf), dynamic ndim=2), "tokens": String([["day"]], shape=[1, 1], strides=[1, 1], layout=CFcf (0xf), dynamic ndim=2)}
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 32, kind: BrokenPipe, message: "Broken pipe" }', /app/source/carton-runner-interface/src/do_not_modify/framed.rs:71:38
stack backtrace:

I edited the earlier comment; the most recent link should have the env_logger dependency too.
The issue for that panic is #148. The panic is actually in the runner process while it's shutting down, not in your main program. So it doesn't practically impact things, but it is confusing and a bad experience.
I'm currently working on a fix for #148 and should have it up in a few hours

Yeah, I get that. Just thought I would mention the issue. I think you can close this whenever you're ready. Thanks!

I appreciate it :)
I just put up a PR that fixes #148. New runners that don't panic during shutdown should be available after the nightly build runs. I'll post more details on how to get them in #148 once the PR lands

@michaelmior  The nightly builds are complete and new runners have been released that solve the problem. See https://github.com/VivekPanyam/carton/issues/148#issuecomment-1739930075 for instructions on updating.


Remove version: Thunisoft.HuayuPY version <IP_ADDRESS>3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://pinyin.thunisoft.com/webapi/v1/setup/downloadSetup?setupid=0a937fed037642118e63a10ed4e9bfd1&filename=HuayuPY-V<IP_ADDRESS>3.exe
Response status code does not indicate success: 405 (Not Allowed).
(Automated message - build 656)


Hard-light blend mode differences

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[x] Running npm install sharp completes without error.
[x] Running node -e "require('sharp')" completes without error.

Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?
System:
OS: Linux 6.0 KDE neon 5.26 5.26
Container: Yes
Shell: 5.8.1 - /usr/bin/zsh
Binaries:
Node: 18.12.1 - /usr/bin/node
Yarn: 1.22.19 - ~/.yarn/bin/yarn
npm: 9.1.3 - ~/.npm-global/bin/npm
npmPackages:
sharp: ^0.31.2 => 0.31.2
What are the steps to reproduce?
First of all, let me thank you for creating sharp!
It's truly an amazing project and we're currently in the process of migration from Jimp to Sharp and got stuck a bit :)
Given that image is worth a thousand words, here's the gist of it:

What is the expected behaviour?
As you can see on the result above, the artwork is composed of three images:

Water (blend mode normal)
Ice (blend mode hard-light)
Fish (blend mode normal)

Browser renders the version I want, Jimp comes in second with very similar result, but Sharp has completely different result for "Ice" layer/image. I'd expect Sharp's hard-light blend mode to match the one browser uses or if it doesn't match exactly, that it's at least similar.
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem
Here's the repository that has the above results.html page with sharp.js and jimp.js scripts for creating the final image.
https://github.com/VladimirMikulic/hardlight-blend-mode-demo
Please provide sample image(s) that help explain this problem
The three images used to compose an artwork can be found here: https://github.com/VladimirMikulic/hardlight-blend-mode-demo/tree/master/images

It looks like ice.png has already been premultiplied. You can tell sharp this by using the following change:
-      { input: './images/ice.png', blend: 'hard-light' },
+      { input: './images/ice.png', blend: 'hard-light', premultiplied: true },

Thank you so much for a fast reply @lovell.
That worked indeed and I noticed I need it for some other blend modes like overlay/multiply/screen to get the right result. However, it seems like it isn't needed for over/exclusion/difference.
Could you please explain what premultiplied does and how it makes a difference with some blend modes, but not with others? (it seems like Jimp handles automatically)

Best guess would be that it's something to do with blend modes that multiple the alpha values (rather than add/subtract them).
I can't find the documentation for jimp's composite feature so am unsure how it works, however it's unlikely to be able to automatically determine if an image is premultiplied so is probably expecting you to have done this already.


How to get flash from chromeos?

 I don't understand this from the "known issues" page
"After download, find Flash player in /opt/google/chrome/pepper/ directory on partition 3. Partition have to be mounted with "ro" flag, otherwise it won't work."
The script downloaded a zip file with a .bin file in it. Are you supposed to mount the .bin file? How?
Thanks.

OK. found it
https://stackoverflow.com/questions/30732428/extract-files-from-chrome-os-chromebook-recovery-image


[ENE-49] Fix infinite loop in derivation

When writing a code like:
implicit val x: TypedMongoFormat[Something] = io.sphere.mongo.generic.deriveMongoFormat
or
given TypedMongoFormat[Something] = io.sphere.mongo.generic.deriveMongoFormat
A compile error happens due to an infinite loop in the derivation process.
This PR aims to fix that, and also add some new features:

TypedMongoFormat now has an inline apply method cats-style to summon an instance at 0 cost.
Verification that the derives directive also works (at least for Product types).
Specs for different ways to derive and summon instances (given, implicit, automatic).

CI won't work anyway, so I'm closing the PR


Refactoring

Refactoring of setHelpersPath() - removed support of string argument
Refactoring of StatusCodes for Crud and Rest controllers
Fixed Acl rules for Acl controller

Coverage remained the same at 64.815% when pulling f2e6c32dd78e41e736fb60276031f86fa1b5f14c on AntonShevchuk:master into 2fa06d6167176469821a3461168332b21f564efd on bluzphp:master.

Coverage remained the same at 64.815% when pulling f2e6c32dd78e41e736fb60276031f86fa1b5f14c on AntonShevchuk:master into 2fa06d6167176469821a3461168332b21f564efd on bluzphp:master.


Make files.exclude & files.watcherExclude overridable.

Signed-off-by: Yaohai Zheng<EMAIL_ADDRESS>Fix for https://github.com/Microsoft/vscode/issues/46851.
Language server extension need exclude some intermediate files.

Aww this is a real shame.
I really wanted to hide some stuff that my language compiler spits out.


Particle Lighting seems weird.

The particle lighting seems weird, if not wrong. The test data is this one (/glade/p/cisl/vast/vapor/data/Source/DCP) and the following screenshots explains what's weird.
Note: the reference lighting effect pictures are from this website: https://www.cs.toronto.edu/~jacobson/phong-demo/.
Specular lighting seems weird
To illustrate the effect of specular lighting, let's turn off every other lighting and turn specular lighting to its full level, as the following screenshot shows.

However, this looks more like ambient lighting. A proper specular lighting on a sphere would look something like this:

Shineness is not strong enough.
Let's try to use the strongest shineness, as the following screenshot shows. However, each sphere is barely shining.

A properly lighted sphere, however, would give very obvious shining effect, as the following screenshot shows.

We agreed on 2 baselines regarding lighting:

Particles and flow renderers should use the same lighting model
Whatever lighting model is chosen, we should provide a correct implementation of it.

No slider should go to zero shininess should go to ~100.


Allow me to turn on/off "format on save" per-project

When I am editing the flutter repo, I don't want to "format on save".
When I'm editing my own code, I do want "format on save".
Right now, this option is Atom-wide. Can we make this a project option?
Thanks!
Atom 1.3.2, dartlang 0.5.0, SDK 1.13.1 running on macos.

This setting would likely want to live in a metadata file on disk. pubspec.yaml? .analysis_options? .atom/.settings?

.atom/.settings, as it's specific to Atom projects.


Replace cli-table with cli-table3

This PR replaces cli-table with cli-table3. cli-table3 is more actively maintained. It is API-compatible with cli-table. cli-table3 also implements oft-requested features like word wrapping (via the wordWrap option).
Tests pass locally with these changes:

I had opened an issue (#71) that this PR will address.

Fantastic! Thanks!


JWT authentication

Registration and Login using JWT setup done. Still working on get student profile controller and next is connecting courses etc. to users/students. This also includes the StudentDb file under the Data folder. Migrations folder is included so you don't need to add migrations before running the API. Creating a readme for y'all in a bit ;)

I don't know if I did something wrong, but I ended up committing, pushing, and merging the cache files that were causing the conflicts. If this causes issues, please let me know. I couldn't figure out how to remove them.


Feature Request: Import Hotkey: Allow full syntax to be imported

Hello,
And here's FR2 I'd have.
The application already has a hotkey to copy the clipboard to the triggerstring-edit for setting up quicker hotstrings. This is helpful, but given that the application only allows text-replacement anyways, it should be entirely possible to parse complete valid syntax and prepopulate "triggerstring", "options" and "hotstring".
The only complicated thing that I personally wouldn't include are menu-hotstrings, but that's a comparatively smaller section. And even then it's helpful.
ahk is completely capable of parsing hotstring-syntax via regex, especially since we don't care about the possibility of multi-line code sections below the triggerstring in this application.

I personally f.e. use this regex
(?<Hotstuff>.*)`:`:(?:.*(?:`;(?<Comment>.*)?))?

as a means of identifying Hotkeys, Hotstrings and optional comments using ; CommentHere-syntax on the same line in my own projects when necessary. That isn't perfect for this request, because it doesn't parse out the replacement part as it was not necessary for me.
This is also the current progress I made myself on the task in Cf. #13, although I am still interested in how you do it in this application, because I am not dynamically displaying possible matches. No clue so far how to get that rolling :P

Thank you again for this application,
Sincerely,
~Gw

Hello Gw,
thank you for this feature request. I have still some doubts how it should work. To populate correctly new definition one have to specify library for new (triggerstring, hotstring) definition. From my own experience I'd prefare to take this step manually. The same challenge comes with options. The majority of (triggerstring, hotstring) definitions I apply have two forms:
"short", e.g. adm → ADM; no options
"long", e.g. adm/ → extended diagnostics module; option: *
So again I'd prefer myself to set up options manually.
If you wish we can discuss this and other issues on Signal communicator / messenger.
Kind regards, mslonik (🐘)

thank you for this feature request. I have still some doubts how it should work. To populate correctly new definition one have to specify library for new (triggerstring, hotstring) definition.

But that is the case in the current instance as wlel. Right now, you have the shortcut to copy clipboard content to "Enter hotstring".

What I'd ask is that I can do the following:

type down :*c0:dh::A_Hour:A_Min:A_Sec
highlight it
press !q
get the selected text (preferably while restoring the clipboard)
parse it out into options, triggerstring and replacement
prefill the fields
and let the user take care of the rest

I am not asking to expand this into a complex syntax to allow for menu-hotstrings.
definitions I apply have two forms:

"short", e.g. adm → ADM; no options
"long", e.g. adm/ → extended diagnostics module; option: *

I am not entirely sure what this has to do with it, might you elaborate?

I just noticed you recently updated, It might have been that you removed that feature, idk. I am running version 3.5.5, from the cutting edge-branch of back when we first talked.

Additionally, we can set up a voice meeting for this if I can't convey it per text properly.
However, I am utterly busy at the moment, so I won't be able to do so any sooner than 16.02.2022, and even then I might have to reschedule.

Hi again,
at first thank you for all provided explanations. It is appreciated and now I think it's clear to me.
The title of this issue is

Import Hotkey: Allow full syntax to be imported

(I've made bold underline above on purpose). Such functionality is already implemented in Hotstrings application since... a long time. Please just have a look into menu:
Libraries → Import from .ahk  to .csv
This option will import any file containing only static hotstring definitions into format of Hotstrings application.
By the way, I strongly recommend you to update Hotstrings to current stable release (3.6.3) which brings many enhancements and bug fixes.
(You can do it directly from Hotstrings application: menu Application → Version / Update).

Huh. After a lot of back and forth on my end I fetched the current version from GH onto a fresh install.
The problems:

As far as I could (not) find it, there is no option to set the library-folder elsewhere other than the pre-picked location in Appdata/Roaming/Hotstrings. Therefore, I cannot point the script anywhere else.
The ini-file for the script (also located at appdata/roaming/hotstrings) has the same problem.
the first idea was to check if the scripts checks its ScriptDir first. However, copying both files over there does not result in any difference. If Ini and Libraries are not present in Roaming, they are created at startup.
The reload options trigger an Error "script file not found", if the absolute path to it seems to have spaces in it:

I sincerely hope this is not intended behaviour, because I both foolishly lost my own, working version, and well.. it's evidently not working, even on a fresh install - which is curious.

Also tested on a completely separate drive at root level for no reason whatsoever, same issues.

Hello GW,
now I write just to let you know I'll try to address the issue related to location of the files (config.ini, languages, Libraries)as my top priority.

Hello GW,
regarding your last issue, I've fixed it with this commit: https://github.com/mslonik/Hotstrings/commit/4cfca78744411c84d6f95a0e1c3b82c100c761ba
(Later it will be integrated into next stable release). By the way I've found 2x other issues: positions in Tray menu doesn't make much sense and icon file cannot be easily downloaded from within GitHub. To be fixed later.

Hello GW,
from my perspective import of "full syntax" is available. The other issues you've described are now separated into dedicated issues. I'm willing to close this issue, if you don't mind.

We can close it then.
I might disagree regarding the import of full syntax.
But the issue here lies on me, if the current branch differs in that regard with my older instance.
Given that I currently cannot check the newest patch due to lack of... (well, time, a working computer and storage), I will check back on it once I can.

(To say the least, my main storage partition decided to dive off the digital cliff and I am in the middle of exam prep with half my data lost. This has moved to probably the lowest possible priority for me, which is also why I haven't responded in days.).
Thankfully i keep backups off-site, but there is still a bit I've negligently failed to back up recently.
Let's just say I am "mildly irritable" currently.


[Enhancement] Improving memory locality of traversal tables

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Problem Summary(Required) ：
Now the order in which we traverse the hash table is directly to the hash table.
Although direct access to flat hash table is a fast operation. He has very good memory locality.
but in fact, the value of hash table is a pointer.
eg:
[(k1, s1), (k2, s2)]
the access order is (k1,s1), (k2, s2).
But s1 and s2 may be farther apart from each other.
Because the order of our hash insertion is not a fixed order.
To improve the speed of hash table traversal. We choose to save both keys of the hash table in the state (maximum sizeof(key)==16).
At the same time we save the creation order of the states (states are allocated consecutively from the pool, the same creation order has better memory locality).
When outputting the hash table, we output the hash table by the traversal order of the states
performance case:
baseline 2.3-rc02
1FE 1BE pipeline items:99997497
SELECT SUM(a), COUNT(*) AS c, AVG(b) FROM tablex GROUP BY x, y ORDER BY c DESC LIMIT 10;
6.58 -> 4.85
SELECT u, COUNT(*) AS c FROM tablex GROUP BY URL ORDER BY c DESC LIMIT 10;
6.53 -> 5.44

Need to do a sufficient performance benchmark.

Need to do a sufficient performance benchmark.

http://<IP_ADDRESS>:8090/job/performance_comparison/245/Two_20version_20compare_20report/

In my memory, phmap is a flat hashmap and should be good at iteratoring. If guess your solution works better than phmap internal iterator, but benefit ratio should not be very high I guess.

regression #604

run starrocks_admit_test

run starrocks_admit_test


CSV annotations can't be a list

@timflannagan1 @EmilyM1 re https://github.com/operator-framework/operator-metering/pull/1167 - this introduces an annotation as a list. That's valid yaml, but not a valid kube object - annotations are strings. We're getting validation errors downstream.

Thanks, Luke - I got pinged about this earlier, and corrected the misalignment in #1171

:+1: thanks


Updating status of projects

...

Congratulations @agroGandhi  on completing all the challenges for the Beginning Git course.
I hope that you enjoyed working through this course—if you weren't already using git for your code, hopefully this has inspired you to do so. I hope you're going to work through the "Mastering Git" video course as well—you'll soon be able to get git to bend to your will :)
You can now wear this #GitWarrior badge with pride—and Tweet your historic victory!

sam


chore: Update phpunit to ^10.5.16 || ^11.2

Description
Update phpunit to ^10.5.16 || ^11.2
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Attribute #[WithoutErrorHandler] to disable PHPUnit's error handler for a test method
https://github.com/sebastianbergmann/phpunit/issues/5428


Increment version for paloaltonetworks.ngfw releases

Increment package version after release of Azure.ResourceManager.PaloAltoNetworks.Ngfw

API change check
API changes are not detected in this pull request.

/check-enforcer override

Failures are unrelated to these changes; merging.


Styling - white-space: nowrap

Great Plugin. Not really an issue, but limitation in some instances of usage.
I noticed when adding text, adding a paragraph, it makes that entire paragraph one line. ideally, it should wrap within the DIV element. But it does not. I looked further and figured this was the issue. May want to consider having an option and remove "white-space: nowrap;" as it is hard coded into the stylesheet and not many would be able to remove it or know how to AND, like myself some clients like to add paragraphs of text. Myself, I can easily remove it.
Like an image slider with a block of text, simple.

Hi, white-space: nowrap; is hardcoded for compatibility reasons related to responsiveness. The easiest way to break is writing something like that:
This is the first line

This is the second line

Sorry, comment edited!


Drag windows between tiles

Is your feature request related to a problem? Please describe.
In my flow of work I often move windows around between tiles (and between monitors as well) by dragging the windows (by pressing modifier key (Meta by default) + left mouse button dragging). Moving one window to the place of other window (tile) would just swap those windows between the tiles.
Example: Tile 1 is occupied by Window 1. Tile 2 is occupied by Window 2. Dragging Window 2 (by title bar or modifier key + left mouse button) on top of Tile1/Window1 would result in: Tile 1 occupied by Window 2, Tile 2 occupied by Window 1.
Describe the solution you'd like
Dragging windows between tiles would work - it would swap the two windows in those two tiles affected. Currently, dragging the window just de-tiles the window.
Describe alternatives you've considered
none?
Additional context

Duplicate of #25 and addressed heavily in the v0.5.1 release notes, this feature will likely never be supported.


Xamarin.Google.iOS.Maps 6.0.1 rejects ARMv7

In a Xamarin.Forms app, upgrading Xamarin.Google.iOS.Maps from 3.9.0 to 6.0.1, I am getting a slew of build errors of the type
ignoring file /Users/bill/Library/Caches/XamarinBuildDownload/GMps-6.0.1/Maps/Frameworks/GoogleMaps.framework/GoogleMaps,
missing required architecture armV7 in file Users/bill/Library/Caches/XamarinBuildDownload/
GMps-6.01/Maps/Frameworks/GoogleMaps.framework/GoogleMaps ( 2 slices)
[error MT5209 Native linking
and
Native linking failed, undefined symbol: _GMSGeometryArea .....  (MT5214)
In Release mode, under iOS Build, I have Link Framework SDKs only and supported architectures ARMv7 and ARM64.
If I change this to ARM64 only, it runs fine. If I revert to 3.9.0, it runs fine.
I have done the the usual things, like deleting .bin and .obj, and Clean All. Nothing found by Googling.

Oh, I see now. V6.0.1 requires iOS V12+, which excludes 32-bit devices like iPhone5, so no point.


Core: Load the onerror module

Ref gh-1062
This is not fixing #1062, but it fixes another bug I found from there.

No, @jquerybot, it's not my first contribution to this project. :)

I'd better track this one while thinking about #1061. To me, it's a bit of a smell that we are importing that file for its side effects- better that that module export an error handling function and the HTML reporter use that to create a window.onerror handler.

I believe it's time to release a new version, this one can land faster, but it shouldn't block your work on #1061

No objection to releasing this in a new version, #1061 still needs
discussion and buy-in. Thanks!
On Oct 17, 2016 6:57 PM, "Leo Balter"<EMAIL_ADDRESS>wrote:

I believe it's time to release a new version, this one can land faster,
but it shouldn't block your work on #1061
https://github.com/jquery/qunit/issues/1061
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jquery/qunit/pull/1063#issuecomment-254368689, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AARWes7lDzDMCU8Q5zk7CXPHOOQMR8a2ks5q1At1gaJpZM4KZN_W
.

Went ahead and made the adjustment myself and merged.


Callback function

I am trying to store the color selected into sharedpreference, but it looks like the callback function isn't working:
  color: screenPickerColor,
                onColorChanged: (Color color) {
                  print("new color ");
                  print(color);
                },

Hi @aldwnesx,
It is a bit difficult to see from your short sample what might be wrong, but from what is shown I suspect your implementation/usage might be causing the issue.
I took the basic example included with the package and added print statements to the onColorChanged callback there for both the Card and Dialog example.
In a Card here:

In a Dialog here:

You can find the complete code for this modified example with the print statements baked in here:
https://gist.github.com/rydmike/65d9219a5d8be61bc9b31077c531ebe3
When I use this example, the onColorChanged callback appears to be working as it should in both cases

If you compare your code with the example code, can you find any major differences between it and your implementation from a usage principle point of view? If you do, then that might help you solve the issue.
If it does not help, then if you provide a link to short a Gist sample or complete GitHub repo with your code, I might be able to assist further.
Mike

Hi @aldwnesx,
It is a bit difficult to see from your short sample what might be wrong, but from what is shown I suspect your implementation/usage might be causing the issue.
I took the basic example included with the package and added print statements to the onColorChanged callback there for both the Card and Dialog example.
In a Card here:

In a Dialog here:

You can find the complete code for this modified example with the print statements baked in here:
https://gist.github.com/rydmike/65d9219a5d8be61bc9b31077c531ebe3
When I use this example, the onColorChanged callback appears to be working as it should in both cases

If you compare your code with the example code, can you find any major differences between it and your implementation from a usage principle point of view? If you do, then that might help you solve the issue.
If it does not help, then if you provide a link to short a Gist sample or complete GitHub repo with your code, I might be able to assist further.
Mike

Hello, thanks so much for your reply.
The problem seems to be gone after I restarted my PC, so weird!
Thank you for the great wokrk

Hello, thanks so much for your reply.
The problem seems to be gone after I restarted my PC, so weird!
Thank you for the great wokrk

No problem, happy to help and that you got it to work :)

No problem, happy to help and that you got it to work :)


Add functionality in react to update the graph using functions defined in django.

Done by @srushti98 . I updated the prompt as well


Email notifications - backend

Develop end-points for following email sending options

[x] To citizen upon creation of request to acknowledge receipt
[x] To citizen upon closure with the outcome
[x] To an organization that has been delegated/assigned a request
[ ] Integrate gov mail server

Removed the task " To citizen requesting more information is required to proceed " from the task list since it has a workflow dependency not related to email notification. @MrClemRkz Better to create a separate issue for that and move it to the Phase 2?

created new issue for: Integrate gov mail server https://github.com/LSFLK/request-management/issues/100


⚠ ClamAV has degraded performance

In a53e057, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 100 ms

Resolved: ClamAV performance has improved in 9440158.


Unable to install rxjs/testing

Describe the bug
I'm currently unable to install rxjs/testing for marble testing as described by the documentation. Attempts to do so result in an error from NPM indicating that the repository is missing (See "Additional context" section below).
Sorry if this is the wrong place to post this, but I'm not sure where else to report this issue.
Expected behavior
I expected rxjs/testing to be installed
Reproduction code
`npm i rxjs/testing -D`

Reproduction URL
https://rxjs.dev/guide/testing/marble-testing
Version
n/a
Environment
No response
Additional context
Full output here:
$ npm i rxjs/testing -D
npm ERR! code 128
npm ERR! An unknown git error occurred
npm ERR! command git --no-replace-objects ls-remote<EMAIL_ADDRESS>npm ERR! ERROR: Repository not found.
npm ERR! fatal: Could not read from remote repository.
npm ERR!
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.

npm ERR!  You're completely right.
How did I miss that.


reset available language registrations?

I have a Cordova app, and the first couple of pages use local translation files. Because it's difficult to re-release these apps, and because different groups of users will have different languages available, the rest of the translations come from the server.
So, here's an example of what I'm trying to accomplish: serve the local en.json translations for the first page from the phone's locale code of en-US, give the users a selection of languages from the server (e.g. en-US, en-GB, en-AU), and use the language the user picks.
Below is the code I'm using to get started - I'm calling $translate.use([code from phone]) to kick things off, and then calling $translate.refresh() and $translate.use([code selected, from server]). The refresh is OK because the user will never revisit my first couple of pages - it's a one time signup, so I won't need those translations again, and I don't need to load with parts. My issue is that I'm setting aliases for languages during config using registerAvailableLanguageKeys, but I don't always want these aliases used after I know which languages are available to that user. If there's a way to re-register, I think that would solve the issue, or my code might be completely wrong and there might be some easy way to accomplish this, but I'm hoping there's some way to accomplish this...
`
	.factory('translationLoader', ['$q', '$translateStaticFilesLoader', 'prefs', 'logger', function ($q, $translateStaticFilesLoader, prefs, logger) {
		return function (options) {
			return prefs.getPromise("translations").then(
				function (translations) {
					logger.log('Translations from prefs for ' + options.key);
					if (options.key in translations) {
						return $q.when(translations[options.key]);
					} else {
						return $q.reject(options.key);
					}
				},
				function () {
					logger.log('Translations from files for ' + options.key);
					return $translateStaticFilesLoader({
						prefix: 'languages/',
						key: options.key,
						suffix: '.json'
					});
				}
			);
		};
	}])
	.config(['$translateProvider', function($translateProvider) {
		$translateProvider.useLoader('translationLoader', {});
		$translateProvider.registerAvailableLanguageKeys(['en', 'fr'], {'en-*': 'en', 'fr-*': 'fr'});
		$translateProvider.preferredLanguage('en'); // preferred == default
	}])

`

There is no way re-registering the language keys. But I'm not sure what the problem is.. although there would be a set of n language keys set, the actual translations will be loaded only when requested.
However, as you already have a custom loader, you can solve this there. Your custom loader will check a specific state (i.e. via your prefs service?): either it uses a local translations strategy or it uses a standard remote file loader. And as you already have pointed out: The reset will be invoked with a .refresh().

@knalli Thanks for the response.
I have a couple issues, most likely from my not understanding how this is supposed to work:

It looks like I have to call registerAvailableLanguageKeys. I was getting an error in an indexOf method because of an undefined array without this. I'd rather not register these up-front, and handle the lookup in my custom loader.
This registerAvailableLanguageKeys call is hard-coded into the client, and the mapping would map en-gb to en, so my custom loader gets asked for en, but en-gb might be an available language for a user after they register with the server.

Is there an easy way around this? Maybe the indexOf method could be patched to handle undefined arrays so I wouldn't need to register during config?

What "error in an indexOf method"? Please provide a working demo demonstrating your issue.
registerAvailableLanguageKeys is not a required config setting.

I'll try to reproduce it - my first attempt didn't work; I've made too many changes since I put it in.
However, this call was failing because $fallbackLanguage was undefined: indexOf($fallbackLanguage, $preferredLanguage)
From: Jan Philipp <notifications<EMAIL_ADDRESS>Reply-To: angular-translate/angular-translate <reply<EMAIL_ADDRESS>Date: Tuesday, February 9, 2016 at 5:06 PM
To: angular-translate/angular-translate <angular-translate<EMAIL_ADDRESS>Cc: Craig Sabbey <Craig.Sabbey<EMAIL_ADDRESS>Subject: Re: [angular-translate] reset available language registrations? (#1393)
What "error in an indexOf method"? Please provide a working demo demonstrating your issue.
registerAvailableLanguageKeys is not a required config setting.
—
Reply to this email directly or view it on GitHubhttps://github.com/angular-translate/angular-translate/issues/1393#issuecomment-182098944.

I was calling useFallbackLanguage without specifying any fallbacks ahead of time; I was trying to have a local language fall back to its general one, so for en-US I was trying to do $translate.useFallbackLanguage('en'). Is this supported?

Closing - I don't know what the issue was, but I have something working...
This catch lets me load an empty translation, which lets it just use the fallback instead:
return $translateStaticFilesLoader({ 							prefix: 'languages/', 							key: options.key, 							suffix: '.json' 						})['catch'](function () { return {}; })


Add wrapper script for sidecar release

A wrapper script to generate sidecar release prs from go-get-kubernetes.sh: https://github.com/kubernetes-csi/csi-release-tools/issues/7#issuecomment-707025402

https://github.com/kubernetes-csi/external-attacher/pull/389
https://github.com/kubernetes-csi/external-health-monitor/pull/144
https://github.com/kubernetes-csi/external-snapshotter/pull/791
https://github.com/kubernetes-csi/livenessprobe/pull/171
https://github.com/kubernetes-csi/node-driver-registrar/pull/250
https://github.com/kubernetes-csi/external-resizer/pull/236
https://github.com/kubernetes-csi/external-provisioner/pull/823

Change the repo that you need to generate the pr.
Sample command:
./go-modules-update.sh -v 1.26.0-rc.0 -u sunnylovestiramisu

@sunnylovestiramisu do you plan to continue with this?

@sunnylovestiramisu do you plan to continue with this?

I am using it personally, but did not get the time to make it pass the presubmits yet.

Can you also update https://github.com/kubernetes-csi/csi-release-tools/blob/master/SIDECAR_RELEASE_PROCESS.md

/lgtm
/approve


[OTHER] Use prime react classes on chip text

What would you like to share?
As we are using prime react we should remove the custom css and replace with prime react classes.

Remove the following CSS from Home.css:

Replace with these prime react classes in the User.js component

Additional information
No response

I want to work on this issue can you please assign me

Hi @Abhi-paul16 someone has already made a pull request for this one but keep an eye out as I will be listing similar ones soon


Fontawesome 4.7

Upgrading the Fontawesome assets to 4.7 which seems to be the latest. I simply downloaded the assets from the FA website and updated the corresponding files.

Great work @rhuberdeau


when running with xcode, app files are not up-to-date

scenario:

run app via xcode
create a change in the app code
build and run again
App remains without the change.

This was an issue with ionic generator too back in the day, but was fixed.

Hi @yossi-shasho,
cordova requires you to run cordova prepare after a gulp build. The prepare copies the the contents of www/ into the platform/ folders so you can run it with xCode for instance. Perhaps a section in the docs would be helpful for that.

thanks! however, why not incorporate it as a build step in the xcode project?

@yossi-shasho, I'm not that familiar with Xcode, how would you do that? - I can mention it in the docs, however I have some concerns.
We had bad experiences in the past with editing the xcode project. When you collaborate with other people via git or want to update to a new platform version, all these changes are lost can only be manually reproduced, as they're not part of the git repository. In our last blog article about the generator I explained what kind of problems we ran in when 1) cordova platforms and plugins were part of the git project and 2) how annoying it can be if you edit the xcode project that is generated by cordova. So if you are interested give it a look: http://blog.mwaysolutions.com/2015/09/21/generator-m-ionic-and-the-search-for-the-holy-grail/.
So for the scenario you are describing I think it's just the easiest and least error-prone if you just run the following after you make changes:
gulp --cordova 'prepare'

And then run via Xcode.
Does that make sense to you?

The motivation for using xcode in the first place is speed: when running with xcode, the time it takes the app to build, load and start on the phone is much shorter than when using the cordova CLI.
So, yes, your solution works and makes sense, but is less comfortable in situations in which you need to run your app many times on the phone or simulator in a row, for e.g. to do fine graphic adjustments.

That's perfectly true. As I said. If you point me towards a resource that explains how to set it up as a build setup, I'd be happy to include it in the docs, so everybody can handle it according to their own taste :)

I hope it's ok if I'm closing this for now. I mentioned the topic in the docs that will be released with the next version of the generator.
If you reply, I will still get back to you!


Added Intel SVML optimizations as opt-out choice working by default

..if the SVML library is found. The library is available within icc_rt package (conda install -c intel icc_rt --no-deps). Would it be possible to set a dependence upon this package?
This patch relies upon https://github.com/numba/llvmlite/pull/326
You might also want to move parts of this logic into llvmlite itself, especially given that using llvmlite.binding.set_option creates some reasonable warnings in unit tests (because set_option can be use just once..?)

Codecov Report

Merging #2748 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2748   +/-   ##
=======================================
  Coverage   86.27%   86.27%
=======================================
  Files         323      323
  Lines       66587    66587
  Branches     7414     7414
=======================================
  Hits        57447    57447
  Misses       7950     7950
  Partials     1190     1190

Thanks very much for this patch. I have tested this patch locally and I am pleased to say that I can confirm the presence of SVML intrinsics in assembler in places I expect them to appear! There's some disparity at present between the use of a loop containing a transcendental from math vs. the NumPy ufunc equivalent to the loop, in that a loop with a supported math version of a transcendental lowers to a SVML intrinsic whereas a NumPy ufunc equivalent doesn't. I think this is due to Numba's lowering implementation details opposed to anything caused by this patch and is something that needs addressing in future work.
Prior to merging would it be possible to have a few unit tests written please. I'm thinking of something along the lines of:

If SVML is present then check that a simple function containing a loop over a SVML enabled transcendental vectorizes with fast and produces assembler with the expected __svml call.
As above but without fast and therefore no SVML call.

The inspect_asm method on the dispatcher object (the jitted function) will probably help with this.
In terms of implementation, similar to e.g. CUDA support, the code should probably try and detect if the SVML libraries are present and do the necessary loading and options setting if found. The tests described above can then have guards applied to prevent them from running if SVML is not available.
Thanks again.

Along these lines, we'll have to add a special build configuration to Travis CI and buildbot to run through the tests with libsvml present for one configuration.

Ah yes, good point, I think we can just run it as a triggered build once a standard CPU build is done similar to how CUDA is tested.

One question so far, why do you assume that auto-vectorization works only in fast mode? SVML supports high accuracy mode, so both fastmath=True and = False produce calls to  _svml finctions. Instead, we could rely on the value of config.INTEL_SVML to see if the __svml functions are expected

In practice, we've seen the LLVM autovectorizer refuse to vectorize basic floating point operations in most (all?) cases unless fastmath is enabled.  I'm not sure the exact reason for this.

But, I agree that if SVML functions are generated in both fastmath=True/False cases, then we just need to verify in the test that the accuracy mode is correct.

I think this is ready to merge.

Thanks! great news!


Initial submission of xAbcDef1

Full species name: Abc def
Taxon: something
Sequencing technology: HiC, HiFi
Contact<EMAIL_ADDRESS>

Please check the changes you requested


Shutdown health check registry

A follow-up for #1077

Add a method for graceful shutdown of HealthCheckRegistry
Add an ability to gracefully shutdown the executor for async tasks in HealthCheckRegistry.
Now it creates a ScheduledExecutorService which should be correctly closed when HealthCheckRegistry is not used anymore.

Make the pool size more pessimistic by default
Some CI environments return the amount of available processors as 128.
It's rather expensive to create such big thread pools by default. Let's start
with 2 threads and the user wants to increase the amount,  she can do that.

Make the async health check threads as daemons
So they don't prevent the JVM from shutdown if the user didn't shut down
the executor gracefully.

LGTM

Bummer, I screwed up and this change was not added to 3.2.0. I will mark it to 3.2.1.


es2017 is not recognised in .tsx but works in .ts files

TypeScript Version:  3.1.0-dev.201xxxxx
Tested in 3.0.1, 3.0.3, Version 3.1.0-dev.20180907

Search Terms:
tsx lib es2017
Code
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.
const a = Object.values({foo: 'bar'});

Expected behavior:
Object.values() should be recognised in .tsx as it is in .ts files.
Actual behavior:
Object.values is not recognised in .tsx files but works well in .ts files.
error TS2339: Property 'values' does not exist on type 'ObjectConstructor'.
Object.values({foo: "bar"});

Playground Link:
Couldn't find .tsx playground
Related Issues:
https://github.com/palantir/tslint/issues/3608 (Although I think it is a tsc issue and not tslint)
tsconfig.json
{
  "compileOnSave": true,
  "compilerOptions": {
    "allowJs": true,
    "allowSyntheticDefaultImports": true,
    "target": "es2017",
    "module": "commonjs",
    "sourceMap": true,
    "lib": ["es2017", "es2016", "dom", "es2015", "webworker"],
    "jsx": "react",
    "moduleResolution": "node",
    "noUnusedLocals": true,
    "noUnusedParameters": false,
    "removeComments": false,
    "preserveConstEnums": true,
    "skipLibCheck": true,
    "pretty": true,
    "strict": true,
    "noImplicitReturns": true,
    "outDir": "dist"
  },
  "include": [
    "app/**/*"
  ]
}

Okay I've found the issue, because I include only ./app directory in the tsconfig.json my .tsx i.e. UI files in ./ui directory are not getting the rules.
But I don't actually compile them with tsc, I compile them with webpack & awesome typescript loader. So, I am looking for a way to exclude them and also have them applied the rules now.

Solution I came up with is:

Moving ./app and ./ui under ./src and including ./src in the tsconfig.json
Triggering "tsc" on file change (in gulp) but only by watching changes in ./src/app
Using webpack and awesome-typescript-loader as normal.

Maybe, I should remove awesome-typescript-loader and direct webpack to the compiled .js (jsx) files from .tsx by tsc. Would this be a better solution?
If you have any suggestions please let me know otherwise we can close this issue.


Fix external DNS responses > 512 bytes getting dropped

Docker DNS server proxies external queries by sending it to the configured nameservers.  When sending the query UDP packet size defaults to 512 bytes. If the client has set the ENDS option reply and the reply is > 512 bytes it will get dropped by the DNS library.
When forwarding the query set the dns.Conn UDPSize to be that of the size from the client's EDNS option.
Signed-off-by: Santhosh Manohar<EMAIL_ADDRESS>

LGTM


Module 1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?

It is an essential tool to my students to learn about, and also a way to give them assignments.


AScopedBinding Construtor is Internal

I'm curious why this is? Would there be more sense in making it simply protected so that people can write their own bindings?  For example, I'd like to write a binding that automatically adds a Disposable dependency to a DisposableContainer that is provided by scope
abstract class AScopedBinding<A, out C, T : Any> internal constructor(
        private val _creator: NoArgBindingKodein.(C) -> T
)

Kodein 5.0 stable has been released :)


Branch02

{"response": [{"details": " ## Code Review: \n\n### Commit: {commit_hash}\n\nProject: [Project Name Here]\n\nPR Title: [Pull Request Title Here]\n\nAuthor: [Author's Name Here]\n\nDate: [Date of Submission Here]\n\n### Summary:\n\nThe following code changes are present in this commit:\n\n1. Adding New Function:\n\n\n// New Function\nconst calculate_area(length, breadth) => {\n  return length * breadth;\n};\n\n\n- This new calculate_area function is introduced for Calculating area of a rectangle in the new file utils.py.\n\n2. Modifying Existing Function:\n\n\ndef get_product_price(product, discount=0.1):  // Old Code\ndef get_product_price(product, discount=None):  // Modified Code\n  ... // Remaining Code\n\n- The get_product_price function now accepts an optional discount parameter. If no value is provided, it'll default to None.\n\n3. Code Cleanup:\n\nVarious coding style and efficiency improvements, including:\n\n- Indentation fixes.\n- Variable renaming for better"}]}


Update for redeeming the coupons

The dfx cycles command for redeeming the coupons and checking balance are not working so dfx wallet works successfully. And I am using dfx version 0.20.0.
Added a statement to say that dfx version 0.19.0 and above.

Thank you for your contribution to the IC Developer Portal. This repo contains the content for https://internetcomputer.org and the ICP Developer Documentation, https://internetcomputer.org/docs/.
If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:

[ ] Follows the developer docs style guide.
[ ] Follows the best practices and guidelines.
[ ] New documentation pages include document tags.
[ ] New documentation pages include SEO keywords.
[ ] New documents are in the .mdx file format to support the previous two components.
[ ] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[ ] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

Dear @SoftCysec,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/networking 941c61a...22d1b93:
  > 22d1b93 upgrade to latest dependencies (# 676)
  > 5d159ff Update actions (# 675)
bumping knative.dev/pkg f907b31...19adf79:
  > 19adf79 upgrade to latest dependencies (# 2524)
  > b0ff5ad Update actions (# 2523)
bumping knative.dev/hack b4d4d73...12d3e2a:
  > 12d3e2a switch to UTC (# 176)

/lgtm
/approve

/approve


⚠ Bofrost.es has degraded performance

In 50259b8, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 2086 ms

Resolved: Bofrost.es performance has improved in 9ad7e0c after 6 minutes.


API Fix: exact match record name by default

Problem was:
When I search /api/plugins/netbox-dns/records/?zone_id=1&type=PTR&name=11 I don't want to match the reverse record for 111.
The same applies forward:
When I search /api/plugins/netbox-dns/records/?zone_id=1&type=A&name=foo I don't want to match the A record for foobar.domain.tld.
By removing the icontains filter, the default Netbox filters are applied, which means an exact match by default.
When you want back the old behaviour (after this change is merged) you can query like this:
/api/plugins/netbox-dns/records/?zone_id=1&type=A&name__ic=foo

Update: I just had a closer look at the rest of Netbox' code, and there are two things I learned today:

It is standard behaviour to require matches in the GUI to be exact, except for the "search" field, and
The "search" field can still be used to get partial/case insensitive matches, so nothing is lost.

Having said that, I stand corrected and applied your fix to all relevant fields, not just the record value. To follow the standard process, I'll open a ticket as well - thanks for pointing it out!

Thanks for your help @peteeckel

Yep, my more general fix contains the change in this PR.

Sorry about typo

Closed


Fixing BAM File Headers

Description of the task:
Following on from a discussion about the BAM file header issue (see here we decided to go with option 3, which is uploading the raw r1, r2 and index fastq files to the ENA, once it can accept multiple fastq files, via us specifying a fastq file identity in the xml.
Writing a ticket so we don't forget about this, and attaching this spreadsheet which shows what we have archived, and what needs to be updated with new fastq files. Two archived submissions do not need to be updated in that they do not have mislabelled data (users can get all data from them).
Acceptance criteria for the task:

The archived submissions that need to be changed, now have all of their raw fastq files in ENA.

The work has been done in ENA to allow for the submission of more than 2 fastq files, so this is now possible.

The archived files have BAM files. We want to append appropriate raw fastq read1, read2, and index files to these entries.
The raw data for these projects might still be present in ingest staging areas. There may be some work for a wrangler to track them down.
We will also want to amend the archiving SOP to include adding raw fastqs (r1, r2, index).

Speak to @ESapenaVentura when picking up this ticket!

Actions required:

[ ] Contact ENA to figure out if we have the data access permissions to do this in the first place (contact their helpdesk<EMAIL_ADDRESS>

[ ] Figure out an SOP for doing this / create a script to do this. This requires appending lots of different files for multiple projects (see spreadsheet in initial post). They do have CLI interfaces.

Jacob and I had a meeting - AI:

[ ] @ESapenaVentura will send an email to ENA
[ ] @ESapenaVentura Identify submissions that need to be updated (Continuation of https://drive.google.com/file/d/1jE2URktUHDBBJzFAi8IuWrk1DC0SHUeK/view)

[ ] Create a spreadsheet with submission UUID, run accession and CloudUrl

Optional AI (if time)

[ ] Check if files are in cloudUrl

No immediate actions for @jacobwindsor because there is a higher priority task, but for when there is time:

[ ] If files are not in cloudUrl, figure out where to get them (May need a wrangler)
[ ] Once gathered, based on ENA's response, figure out how to push the new data files (Directly/through DSP, permissions needed)
[ ] Push the data files to the proper accessions
[ ] Check correct pairing/upload (Probably using ENA's API)

About point one:
I have created a google sheet (continuation of the previous one) with the information required:
https://docs.google.com/spreadsheets/d/1PkoiGKVeQIF3FQAtwl8vhqt5CwEZfQL2zVA83bWBj6g/edit#gid=2088805478
Only 1 submission is missing the run accessions, the TissueStability dataset

I sent an email to ENA
Also, I checked all files are in the respective CloudURL, and marked it in the spreadsheet

This isn't a priority for this sprint. Moving this to Ops Backlog.

@ofanobilbao will split the ticket into one ticket per dataset, so that we can tackle 1-2 datasets per sprint fixed

@ESapenaVentura to convert ticket to epic and attach sub tickets :)


Implement Content-Security-Policy

Content Security Policy
-Content-Security-Policy: default-src 'none'; style-src 'self'; img-src 'self';
-See if nsecweb (or whatever its called) supports this or we can set the header manually
-consider limiting to scripts because of custom styles, images, files.

Here are docs for Nwebsec, but not specifically for .net core (haven't been written yet)
https://docs.nwebsec.com/en/4.2/nwebsec/Configuring-csp.html?highlight=content security policy

Update: configured this but need to fix the UnsafeEval setting in startup.cs once we implement angular.js. Right now it's using a jquery template that's evaluating in code- we need to overwrite those.


Dashboard hunter not working

solves #336, just a typo.

Description
There was a typo in dashboard.py calling network_timwout instead of network_timeout
Contribution Guidelines
Please Read through the Contribution Guidelines.
Fixed Issues
Fixes #336
"BEFORE" and "AFTER" output
To verify that the change works as desired, please include an output of terminal before and after the changes under headings "BEFORE" and "AFTER".
BEFORE
Dashboard hunter not-working
AFTER
Dashboard hunter working
Contribution checklist

[X] I have read the Contributing Guidelines.
[X] The commits refer to an active issue in the repository.
[ ] I have added automated testing to cover this case.

Notes

Codecov Report

Merging #337 into master will decrease coverage by 0.24%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #337      +/-   ##
==========================================
- Coverage   60.20%   59.95%   -0.25%
==========================================
  Files          41       41
  Lines        1970     1953      -17
==========================================
- Hits         1186     1171      -15
+ Misses        784      782       -2

Impacted Files
Coverage Δ

kube_hunter/modules/hunting/dashboard.py
68.18% <0.00%> (ø)

kube_hunter/modules/hunting/proxy.py
49.12% <0.00%> (-2.55%)
:arrow_down:

kube_hunter/modules/discovery/proxy.py
50.00% <0.00%> (-1.86%)
:arrow_down:

kube_hunter/modules/discovery/dashboard.py
56.00% <0.00%> (-1.70%)
:arrow_down:

kube_hunter/modules/hunting/cves.py
77.31% <0.00%> (-1.33%)
:arrow_down:

kube_hunter/core/events/handler.py
89.89% <0.00%> (-1.02%)
:arrow_down:

kube_hunter/core/types.py
81.39% <0.00%> (-0.83%)
:arrow_down:

kube_hunter/core/events/types.py
82.14% <0.00%> (-0.32%)
:arrow_down:

kube_hunter/modules/discovery/hosts.py
79.67% <0.00%> (-0.17%)
:arrow_down:

kube_hunter/modules/discovery/ports.py
88.46% <0.00%> (+10.68%)
:arrow_up:

Continue to review full report at Codecov.

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


Netlify-cms with self-hosted gotrue and git-gateway

- Do you want to request a feature or report a bug?
Question/Bug
tl;dr: Just please point me to the direction to how I can self-host netlify's identity
- What is the current behavior?
Trying to make a jekyll site using netlify-cms, gotrue and git-gateway work together.
Only made it possible to authenticate via github or netlify, not with self-hosted gotrue and git-gateway.
- If the current behavior is a bug, please provide the steps to reproduce.
My steps to produce it were to clone the 2 projects in separate folders alongside a folder with my jekyll site with netlify-cms, then create a docker-compose like the following:
docker-compose.yml
version: '2'
services:
  jekyll:
    image: jekyll/jekyll
    command: jekyll serve --watch --incremental
    ports:
      - 4000:4000
    volumes:
      - ./jekyll:/srv/jekyll

  git-gateway:
    build: ./git-gateway
    env_file:
      - ./git-gateway/.env
    ports:
      - 9999:9999
    depends_on:
      - gotrue
    links:
      - gotrue

  gotrue:
    build: ./gotrue
    env_file:
      - ./gotrue/.env
    ports:
      - 8081:8081
    depends_on:
      - mysql
    links:
      - mysql

  mysql:
    image: mysql
    ports:
      - 3306:3306
    environment:
      MYSQL_DB: gotrue_development
      MYSQL_ROOT_PASSWORD: password

git-gateway/.env
GITGATEWAY_JWT_SECRET="CHANGE-THIS! VERY IMPORTANT!"

GITGATEWAY_DB_DRIVER=sqlite3
DATABASE_URL=gorm.db

GITGATEWAY_API_HOST=localhost
PORT=9999

GITGATEWAY_GITHUB_ACCESS_TOKEN="personal-access-token"
GITGATEWAY_GITHUB_REPO="romdim/jekyll-netlify-cms"

GITGATEWAY_ROLES="admin,cms" # leave blank to allow all roles

gotrue/.env
GOTRUE_JWT_SECRET="CHANGE-THIS! VERY IMPORTANT!"
GOTRUE_JWT_EXP=3600
GOTRUE_JWT_AUD=localhost

GOTRUE_DB_DRIVER=mysql
DATABASE_URL=root:password@tcp(<IP_ADDRESS>:3306)/gotrue_development?parseTime=true
GOTRUE_API_HOST=localhost
PORT=8081

#GOTRUE_SITE_URL=https://romdim.github.io/jekyll-netlify-cms/
GOTRUE_SITE_URL=localhost:4000

GOTRUE_SMTP_HOST=smtp.mandrillapp.com
GOTRUE_SMTP_PORT=587
<EMAIL_ADDRESS>GOTRUE_SMTP_PASS=super-secret-password
<EMAIL_ADDRESS>GOTRUE_MAILER_SUBJECTS_CONFIRMATION="Welcome to GoTrue!"
GOTRUE_MAILER_SUBJECTS_RECOVERY="Reset your GoTrue password!"

GOTRUE_EXTERNAL_GOOGLE_REDIRECT_URI=http://localhost:8081/callback
GOTRUE_LOG_LEVEL=DEBUG
GOTRUE_OPERATOR_TOKEN=super-secret-operator-token

With this configuration, when I go to admin I can see the page to login, but don't have any credentials to do so. I am certain it is not connected to gotrue or git-gateway. I need to signup and store my user through those services.
Also tried as suggested in Gitter to move and rename:
git-gateway folder to jekyll/.netlify/git/github
gotrue folder to jekyll/.netlify/identity/token
Unfortunately it doesn't seem to make any difference.
- What is the expected behavior?
I expect to be able to self-host my identity provider and git proxy in order to make changes to my static site.
- Please mention your CMS, node.js, and operating system version.
CMS version: 1.0.3
node.js and os versions are irrelevant I guess, docker handles them.
- Please link or paste your config.yml below if applicable.
config.yml
backend:
#  name: github
#  repo: romdim/jekyll-netlify-cms
#  branch: gh-pages
  name: git-gateway
  branch: gh-pages
  accept_roles: #optional - accepts all users if left out
    - admin
    - editor
  site_domain: http://localhost:4000/jekyll-netlify-cms/admin/#/

display_url: https://example.com
publish_mode: editorial_workflow
media_folder: "images/uploads" # Media files will be stored in the repo under images/uploads

collections:
  - name: "post" # Used in routes, e.g., /admin/collections/blog
    label: "Post" # Used in the UI
    folder: "_posts" # The path to the folder where the documents are stored
    create: true # Allow users to create new documents in this collection
    slug: "{{year}}-{{month}}-{{day}}-{{slug}}" # Filename template, e.g., YYYY-MM-DD-title.md
    fields: # The fields for each document, usually in front matter
      - {label: "Layout", name: "layout", widget: "hidden", default: "post"}
      - {label: "Title", name: "title", widget: "string"}
      - {label: "Publish Date", name: "date", widget: "datetime"}
      - {label: "Categories", name: "categories", widget: "string"}
      - {label: "Body", name: "body", widget: "markdown"}

tl;dr: Just please point me to the direction to how I can self-host netlify's identity

Bump, 24 days have passed since asking this. Please direct me to the answer and I may be able to document it if you don't have the time. Thanks!

@romdim you need a way to create users. For this red hot second, you can use the gotrue cli for that. It's not documented, but the code makes a good reference: https://github.com/netlify/gotrue/blob/master/cmd/admin_cmd.go
Namely the createuser command looks to be the ticket. With this route you can at least get something going and then figure out live sign ups through the CMS itself if needed.

Nope, got caught up with other stuff and stopped trying

@hfte awesome!!! We'll probably link to that elsewhere as well - thank you for writing it up!

If someone is trying to set custom identity_url but Netlify CMS stills trying to fetch default {hostname}/.netlify/identity and getting 404, I've fixed with following hack:
const initCms = config => {
  if (config.identity_url && window.netlifyIdentity) {
    const fixGotrueApi = () => {
      const { api } = window.netlifyIdentity.gotrue
      api.apiURL = config.identity_url
      api._sameOrigin = config.identity_url.includes(window.location.host)
    }
    if (document.readyState !== 'loading') {
      fixGotrueApi()
    }
    document.addEventListener('DOMContentLoaded', fixGotrueApi)
  }
  window.CMS.init({ config })
}

Ref.: https://stackoverflow.com/questions/52174873/using-netlifys-hosted-identity-service-with-self-hosted-netlify-cms (solution purposed here doesn't work to me because it creates multiple identity iframes and I think there was a problem with overlaps, but was a good start point :smile: )

Closing as I believe enough information has been provided.

Imo it should work just with config fields, not? It's a bug?

True, see https://github.com/netlify/netlify-cms/issues/3995#issuecomment-655966399

https://github.com/hfte/netlify-cms-with-selfhosted-gotrue-and-git-gateway

thelink is dead,
Here the archived version
https://web.archive.org/web/20230412134002/https://github.com/hfte/netlify-cms-with-selfhosted-gotrue-and-git-gateway


suggest adding a mru-sorting/mfu-sorting algorithm

I really love this mobile app, it provides very practical features. However, I suggest adding a sorting algorithm, which could be based on the frequency of most-used (mfu)or recently used (mru)items. Looking forward to your reply, thank you very much.

Thank you for your feedback @JessonChan , it's very appreciated!
Currently the app already sorts by MFU, considering current location and time:

first all MFU for current location + time
then MFU for location only
MFU for time only
lastly MFU with no constraints

About your mention to MFU
Are you suggesting another option to sort only by MFU without time and location constraints?
**About your mention to MRU **
The current database structure doesn't support that; here's the rationale:
With the previous implementation, we were storing an entry every time an app is opened, including time and location; this approach was very versatile, but had a very big performance penalty: since we would have a ridiculous amount of rows in the database, the sorting algorithm would take up to a couple of seconds to run. To solve this, we could cache a sorted list, but it would kill a big feature of the app, which is real-time sorting: a real life problem that I faced day to day, was having instantly updated suggestions, based on my location: for example, as a user, I expect to have my fidelity card app at the first place as soon as I reach the shopping mall.
For this reason, we dropped that idea and chose instead to create "pre-packed" containers, related to various combinations of time and location, formed by the id of a given up and a counter to sort by.
This allowed us a big boost in terms of performance, but a stricter range of flexibility.
The feature that you're requesting it's out of the scope of the app -since other solutions have simple sorting by MRU- and would require a separate database table, impacting on performance and maintainability of the code.
I'm not saying we don't want to implement that, but it would fall into a low priority task, especially considering that I'm the only developer working on this in my free time; but any contribution would be very appreciated!

Thank you for taking the time to provide such a detailed response. I feel very honored. I have fully understood it now. I will continue to follow and use such a fantastic android app, and I look forward to it getting even better.


[Question]  Is there a summary about what cryptographic algorithms has used in client-go?

Hello,
Is there a summary about what cryptographic algorithms has used in client-go?
Something like AES256?
Maybe developer can easily answer this question.
Thanks in advanced.
BRs,

client-go uses the golang standard library for TLS communication. See https://golang.org/pkg/crypto/tls/ for details
/close


server: increase priority of replica checker's operator

increase the priority of replica checker's operator produce by down peer or offline peer.

LGTM.


[dev] [feat] realistic wrangler dev (edge connected)

This PR will allow users to connect directly to Cloudflare's edge to preview their Workers.
currently this PR contains no new code and only restructures it
To Do:

[x] unauthenticated users use gcs preview
[x] authenticated users use edge preview

[x] http requests proxy to edge
[x] devtools messages
[x] watch file changes

@EverlastingBugstopper I'm available for consultation to get this over the finish line :)

soon my friend. soon.

Sometimes all you need is a gentle nudge!

i thought it was pretty funny when you commented knowing we would ship like the day after - your spidey sense 😆


Roll Flutter from 6aaabf6d5695 to 70844b7b9e93 (1 revision)

https://github.com/flutter/flutter/compare/6aaabf6d5695...70844b7b9e93
2022-05-21<EMAIL_ADDRESS>Roll Flutter Engine from a19d5c65bf71 to f37416c07238 (1 revision) (flutter/flutter#104322)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Customization and example

As promised yesterday, here's a quick and dirty customization and an example of the first ReproNim volume.
I didn't add any of the build files to this PR, but you could add my fork as a remote, fetch it, and check out the branch quick_example to build and preview my changes locally. Here is a sketch (untested, from the top of my head):
git remote add gh-adswa
git fetch gh-adswa
git checkout gh-adswa/quick_example

I've added a custom sphinx extension with custom directives based on those we use in the handbook. Therefore, building everything now takes a bit of software setup, but it should boil down to:
# create a venv to not mess with anything else
virtualenv --python=python3.7 ~/env/repro
. ~/env/repro

# install from requirements.txt
pip install -r requirements.txt

# build the docs
make html

# view in browser
firefox _build/html/index.html

# build the PDF (this probably requires a lot of LaTeX stuff being installed,
# so no guarantee that it works out of the box ;-)
make latexpdf

A short summary of what I've done:

ported https://docs.google.com/document/d/1Mq8Tsn8o8EKz2O4dyJJaHdNtZTzptb9eoGcZk7ZfsnE/edit# into rst as an example
custom directives (I thought the "short summary" at the start of the doc could get a custom format)
a bit of styling: side bars, progress bars, a logo, ... in various places (conf.py, _templates/,_static/custom.css)

It's just meant as a quick example for you to checkout and see whether this format is something that is useful for you. Also, feel free to use, close, or cherry-pick this PR as you please. :)

Sorry, @dnkennedy, missed your e-mail amidst the brainhack - sure, we can merge. If you make me a collaborator of this repo I can also fix up everything this could potentially break. ;-)

Cool, see https://howwouldrepronim.readthedocs.io/en/latest/vol01/localcontainer.html for the result of this PR

Thanks for poking at this, @adswa !


[FEA] Optimize memory for SUM on unbounded preceding to unbounded following.

Is your feature request related to a problem? Please describe.
Once #6550 goes in we will have a framework to help us avoid out of memory errors on window operations that do unbounded preceding to unbounded following.  To help us move closer to avoiding any memory issues it would be nice to try this out with something besides COUNT.  I would like to see us add in support for SUM.  We probably want to add in some configs to go with this too.

Something that would let us disable this optimization for floats/doubles because the order of operations would change and could produce different results from one run to the next.
A config that would let us disable window operations that require a single batch for all data (any window operations with no partition by and is not batch-able)
A config that would let us disable window operations that require a batch for an entire partition by group. (any window operation with a partition by but is not batch-able)

Describe the solution you'd like
SUM for all supported data types that supports batching for unbounded preceding to unbounded following.

Reopening because #9072 reverted the PR for this

#9071 was the reason it was reverted and there is a comment there and in #9072 that I think is the cause of the bug. But we want to make sure that we have good tests that cover the case.


adds Metrics to resources

Adds the new(ish) Metrics as part of a Resources yaml.
Test based on the example provided in the dbt metrics documentation

@riordan seems the repository is sadly not supported. Do you have any ideas, on how to stir things up?
Or do you know any other source of yaml schemas?


Exception on application close if text context menu flyout opened (WinUI3 1.3, 1.4)

Describe the bug
C++ and C# projects. If context menu for any text control is open, application crashes when closed. The crash is silent if the application is not running under a debugger, but is shown in the Application Event Log.
Steps to reproduce the bug

Create standard blank WinUI3 project
Open MainWindow.xaml and add <TextBox/> after </Button>
Build and launch the application
Right-click the TextBox to bring up the context menu.
Close the app with X

Expected behavior
No response
Screenshots
https://github.com/microsoft/microsoft-ui-xaml/assets/28443945/ae62f287-39c8-48d7-944f-6ca3afce25b7
NuGet package version
WinUI 3 - Windows App SDK 1.4.1: 1.4.230913002
Windows version
Windows 10 (21H2): Build 19044
Additional context
I also tried to find a way to close all possible flyouts on close, but couldn't find a way without traversing the entire xaml tree.

Questionable workaround:

   LRESULT Subclassproc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam, UINT_PTR, DWORD_PTR )
    {
        switch (uMsg)
        {
        case WM_CLOSE:
        {
            static bool realClose = false;
            if (!realClose)
            {
                SetFocus(NULL);
                DispatcherTimer timer;
                timer.Interval(std::chrono::milliseconds(50));
                timer.Tick([&hWnd](IInspectable const&, IInspectable const&) {
                    realClose = true;
                    SendMessage(hWnd, WM_CLOSE, 0, 0);
                    });
                timer.Start();
                return 0;
            }
        }
        }
        return DefSubclassProc(hWnd, uMsg, wParam, lParam);
    }

You can also see such crashes in PowerToys and WinUI 3 Gallery, but not in Windows 11 native apps.

This has the same underlying cause as #8605, with a more direct repro. Resolving as duplicate.
A better workaround might be to listen for Window.Closed and walk through VisualTreeHelper.GetOpenPopupsForXamlRoot() and Close() all of those popups. But this clearly needs to be fixed in the platform.


Implemented flex-box in portifolio

-Features added in this branch are

implemented flex-box
Implemented style lintors and HTML webhint for better coding and error-free code

Hey, i just fixed the Linters / Web hint errors

Hi @Dagic-zewdu,
If you need a new code review, make sure to ask for it in your dashboard, so I or any other code reviewer can do it.
Remember not to open a new PR for re-reviews, just submit the link to this PR.
Cheers and happy coding! 👍


refactor: improved OutputNoteRecord

closes #376
This PR introduces the last changes of the Store refactor. It removes the deprecated NoteStatus and replaces it with OutputNoteState and adds transitions to output notes so that they behave more similarly to the InputNoteRecord.

cc @igamigo @SantiagoPittella
There are some design desitions I wanted to point out and discuss:
OutputNoteState
The new OutputNoteState struct is declared inside store/note_record/output_note_record/mod.rs. It replicates the state machine behaviour in a simpler way using match cases. This was a problem for input notes because the logic was more complex and the match cases were really big. In the case of output notes the states and transitions are simpler and the match cases are not as big which is why I used them in this implementation.
If we want I can refactor it so that the internal structure is more similar to input notes which uses a dedicated states module with a more extensible state machine, this would add more boilerplate code without much benefit (I don't think output note states will grow as much).
Discriminant constants and state names
I really don't like these big exports. They are really repetitive (most names are something like STATE_...) and don't clearly show which discriminant/states belong to which notes (input/output).
One way to solve the second issue is to separate them in different namespaces, something like note_record::input and note_record::output so that is clear which types/constants belong to which type of note.
For the second issue I don't have a clear idea of how we can group these groups of constants so that we don't have to individually export/import them.

For the second issue I don't have a clear idea of how we can group these groups of constants so that we don't have to individually export/import them.

Could we attach these constants to their corresponding enums? For example:
impl InputNoteState {
    pub const STATE_EXPECTED: u8 = 0;
}

@julian-demox On commit https://github.com/0xPolygonMiden/miden-client/pull/551/commits/382b7ad6b3aa47c03df609ab24a7ed4a7d340dc1 I made some changes to the way the web store handles note updates, I moved most of the logic to the Rust side of things as that's where we have access to the note transitions to make the updates. Let me know if you find any inconsistency or potential problem there.

I was thinking if we can get some benefits in readability using double dispatch on the NoteState in some cases but not really quite sure.

I can make a PoC in a separate PR so we can compare and decide


chore: Remove dependency for external fonts within the CI

Currently apps/demo-nextjs (maybe a few others) reaches out to an external provider to retrieve fonts (Google).
We should't rely on external services for our pipelines, so that we have a resilient process.
demo-nextjs:pretest: FetchError: request to<EMAIL_ADDRESS>failed, reason:
demo-nextjs:pretest: src/app/layout.tsx
demo-nextjs:pretest: `next/font` error:
demo-nextjs:pretest: Failed to fetch `Inter` from Google Fonts.
demo-nextjs:pretest:  ELIFECYCLE  Command failed with exit code 1.

[Source]

@danielbate pls assign

Read my mind.


Added Prerequisites section in file

…rt13
🦟 Bug fix
Fixes #511
Summary
Added Prerequisites section specifying to install the required library
Checklist

[x] Signed all commits for DCO
[x] Updated documentation (as needed)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

The PR is still not merge , Please merge the PR ...
Thankyou


Include G6 license

Since we're using custom node types for the graph, we have part of the G6 source code in this repo. So let's include their license file as well.

What part of the code are you referring to?

What part of the code are you referring to?

Specifically MetricCustomNode and DashedEdge

What part of the code are you referring to?

Specifically MetricCustomNode and DashedEdge

Ok, there was not way to avoid copying that?

Ok, there was not way to avoid copying that?

I think that's the recommended way.


docs: update Screen Tracking example to remove expo-firebase-analytics

Why and how
expo-firebase-analytics is getting deprecated. With SDK 47 going to be released soon, we want developers to migrate from it and use any other mobile analytics SDK.
This PR

updates remove references to using expo-firebase-analytics from the example in Screen Tracking
adds a placeholder to indicate at which line the analytics function should be added when using the example
update the verbiage in context to example
updates version 6.x docs and changes backported for 5.x.

Thanks for the feedback @satya164!
I've removed the tracker object reference and converted the trackScreenView to a function for the example snippets.
The code samples to Expo Snack are referenceed back. I found references to expo-firebase-analytics (commented out) in the samples, so I removed it (as not to create any confusion about the package's availability).
Also, in the sample files, there is already an alert() that displays the current route name as a placeholder, so didn't change that to console.log. I think it conveys a similar behavior for sample examples.


aktualizr: Bump to latest for recovery fixes.

BTW I'm still waiting on oe-selftest; I think the problems I'm seeing are specific to the machines I'm running the tests on, but I'd like to be more confident in that before merging. oe-selftest succeeding with latest master also depends on https://github.com/advancedtelematic/meta-updater-raspberrypi/pull/51.

Finally got oe-selftest to pass on my laptop by being careful with which versions of the various layers I used. @lbonn are you okay with the last commit I added?


Process for removing sets that do not comply with policies?

If a user of this repository finds a set that is not in compliance with the guidelines, will a pull request removing it be accepted? Or is there a different process?
(related: https://github.com/WICG/first-party-sets/issues/105 #285)

<EMAIL_ADDRESS>


close #34 give the user a history of previous reservations

Added the History display component.
Updated the screen to display the history of the privious booking.

The CSS styling and the alignment is good


[QUESTION]autoscaler can not work normal by heapster.

Description of problem:
when i create two autoscalers in cluster environment, one is php-apache which created through http://kubernetes.io/v1.1/docs/user-guide/horizontal-pod-autoscaling/README.html, the other is
dockerhttpd, here is the result:
kubectl get hpa --all-namespaces -o wide
NAMESPACE   NAME           REFERENCE                                  TARGET    CURRENT     MINPODS   MAXPODS   AGE
default     autos          ReplicationController/autos/scale          10%       waiting   1         3         44s
default     php-apache-2   ReplicationController/php-apache-2/scale   50%       466%        1         10        13h
here comes the problems,
1.the CURRENT of autos(dockerhttpd) is waiting while the php-apache-2 shows the cpu percent.
2.when i stop load, the percent of php-apache-2 is always 466%.
my environment is below:
1.kubenetes version
Client Version: version.Info{Major:"1", Minor:"1", GitVersion:"v1.1.7", GitCommit:"e4e6878293a339e4087dae684647c9e53f1cf9f0", GitTreeState:"clean"}
Server Version: version.Info{Major:"1", Minor:"1", GitVersion:"v1.1.7", GitCommit:"e4e6878293a339e4087dae684647c9e53f1cf9f0", GitTreeState:"clean"}
2.heapster version
gcr.io/google_containers/heapster:v0.18.2
heapster in use is created from  kubernetes/cluster/gce/coreos/kube-manifests/addons/cluster-monitoring/influxdb/(master version),  when the heapster runs, it could connect to influxdb and kubelet, kube-api, and the logs show normal too.
but how this problem comes out, does anybody know this situation? Thanks.

This should be fixed in the new Heapster 1.0.0 released in Kubernetes 1.2.


fix: weird text on header issue on adapters page

Notes for Reviewers

We cannot use HTML tags inside meta tags as it causes overflow and creates this issue. I fixed this with some checks for the description.
While exploring this file, I found another issue with the size of the favicon, so I fixed that as well.

This PR fixes #11545
Signed commits

[X] Yes, I signed my commits.

🚀 Preview for commit 8028dc74c35d8ac149e402b5f3d46c03d25e9f3b at: https://66b1c10b3944558c0098a880--meshery-docs-preview.netlify.app

Thank you @amitamrutiya for your PR. // @vishalvivekm

@amitamrutiya let's discuss this PR during the Meshery dev meeting call, if you would. Please add it as an agenda item to the meeting minutes.

@amitamrutiya  this the good item to discuss in website meeting today please add as agenda item for today: https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit so we discuss finally and merge after.