[diesel_cli] index out of bounds when running migrations on windows 10 WSL

Setup
Versions

Rust: rustc 1.37.0-nightly (929b48ec9 2019-06-21)
Diesel: diesel 1.4.0
Database: Postgresql 11
Operating System: Windows 10 WSL with Debian stretch

Feature Flags

diesel: postgres

Problem Description
When running diesel migration run I get a thread panick.
What are you trying to accomplish?
I am trying to run the migrations for that project.
What is the expected output?
That all migrations are applied
What is the actual output?
marcel@DESKTOP-1HTO3TF:/mnt/c/Users/marce/IdeaProjects/bunnysync$ diesel migration run
thread 'main' panicked at 'index out of bounds: the len is 256 but the index is 290', src/libcore/unicode/bool_trie.rs:42:25
stack backtrace:
   0:     0x7f96a3203a8b - backtrace::backtrace::libunwind::trace::hcc11b0051d9c739c
                               at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.29/src/backtrace/libunwind.rs:88
   1:     0x7f96a3203a8b - backtrace::backtrace::trace_unsynchronized::h3b58410f75e96857
                               at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.29/src/backtrace/mod.rs:66
   2:     0x7f96a3203a8b - std::sys_common::backtrace::_print::h3b42c4b088f05103
                               at src/libstd/sys_common/backtrace.rs:47
   3:     0x7f96a3203a8b - std::sys_common::backtrace::print::h4cc39805027fc680
                               at src/libstd/sys_common/backtrace.rs:36
   4:     0x7f96a3203a8b - std::panicking::default_hook::{{closure}}::h2e800464f258e026
                               at src/libstd/panicking.rs:198
   5:     0x7f96a3203767 - std::panicking::default_hook::ha44ddd2fe3d1fda2
                               at src/libstd/panicking.rs:212
   6:     0x7f96a32041a0 - std::panicking::rust_panic_with_hook::hb20e5ee9c7b3a62c
                               at src/libstd/panicking.rs:475
   7:     0x7f96a3203d22 - std::panicking::continue_panic_fmt::h6bd583e9533c73b9
                               at src/libstd/panicking.rs:382
   8:     0x7f96a3203c06 - rust_begin_unwind
                               at src/libstd/panicking.rs:309
   9:     0x7f96a322315d - core::panicking::panic_fmt::h5152ff697a341efd
                               at src/libcore/panicking.rs:85
  10:     0x7f96a3223115 - core::panicking::panic_bounds_check::h79a50406288c243d
                               at src/libcore/panicking.rs:61
  11:     0x7f96a32282f1 - core::unicode::bool_trie::BoolTrie::lookup::h8f9a1feedbf812da
                               at src/libcore/unicode/bool_trie.rs:0
  12:     0x7f96a322773e - core::unicode::tables::derived_property::Grapheme_Extend::h6e780c9a0384b93a
                               at src/libcore/unicode/tables.rs:681
  13:     0x7f96a322773e - core::char::methods::<impl char>::is_grapheme_extended::h6f3281f3fd457514
                               at src/libcore/char/methods.rs:712
  14:     0x7f96a322773e - core::char::methods::<impl char>::escape_debug_ext::h113f39783b4cfdc7
                               at src/libcore/char/methods.rs:200
  15:     0x7f96a322773e - core::char::methods::<impl char>::escape_debug::heba914d395f7cedf
                               at src/libcore/char/methods.rs:246
  16:     0x7f96a322773e - <str as core::fmt::Debug>::fmt::h47ee3664f9306198
                               at src/libcore/fmt/mod.rs:1961
  17:     0x7f96a3225dce - core::fmt::builders::DebugTuple::field::{{closure}}::h0847324ddb324b58
                               at src/libcore/fmt/builders.rs:288
  18:     0x7f96a3225dce - core::result::Result<T,E>::and_then::h8ebac24b14fa8802
                               at src/libcore/result.rs:639
  19:     0x7f96a3225dce - core::fmt::builders::DebugTuple::field::hf1bfd17281a5054a
                               at src/libcore/fmt/builders.rs:276
  20:     0x7f96a317eb4e - <diesel::result::ConnectionError as core::fmt::Debug>::fmt::h0a6d4cdd5c9688f8
  21:     0x7f96a3225dce - core::fmt::builders::DebugTuple::field::{{closure}}::h0847324ddb324b58
                               at src/libcore/fmt/builders.rs:288
  22:     0x7f96a3225dce - core::result::Result<T,E>::and_then::h8ebac24b14fa8802
                               at src/libcore/result.rs:639
  23:     0x7f96a3225dce - core::fmt::builders::DebugTuple::field::hf1bfd17281a5054a
                               at src/libcore/fmt/builders.rs:276
  24:     0x7f96a304eb6c - <diesel::database_error::DatabaseError as core::fmt::Debug>::fmt::h3567cade3cb4b2ae
  25:     0x7f96a322673c - core::fmt::write::h665f3e10b8c4c314
                               at src/libcore/fmt/mod.rs:1016
  26:     0x7f96a3203ed5 - core::fmt::Write::write_fmt::h2d7cce560b8a9355
                               at /rustc/929b48ec98aaff2239257574b5897f419cec2647/src/libcore/fmt/mod.rs:195
  27:     0x7f96a3203ed5 - std::panicking::continue_panic_fmt::PanicPayload::fill::{{closure}}::hb8ed5d4adb4fa050
                               at src/libstd/panicking.rs:357
  28:     0x7f96a3203ed5 - core::option::Option<T>::get_or_insert_with::h2c7b15b044222196
                               at /rustc/929b48ec98aaff2239257574b5897f419cec2647/src/libcore/option.rs:814
  29:     0x7f96a3203ed5 - std::panicking::continue_panic_fmt::PanicPayload::fill::h361887c3cf294bfc
                               at src/libstd/panicking.rs:355
  30:     0x7f96a3203ed5 - <std::panicking::continue_panic_fmt::PanicPayload as core::panic::BoxMeUp>::get::hbe7274952584889e
                               at src/libstd/panicking.rs:370
  31:     0x7f96a320418c - std::panicking::rust_panic_with_hook::hb20e5ee9c7b3a62c
                               at src/libstd/panicking.rs:474
  32:     0x7f96a3203d22 - std::panicking::continue_panic_fmt::h6bd583e9533c73b9
                               at src/libstd/panicking.rs:382
  33:     0x7f96a3203c06 - rust_begin_unwind
                               at src/libstd/panicking.rs:309
  34:     0x7f96a322315d - core::panicking::panic_fmt::h5152ff697a341efd
                               at src/libcore/panicking.rs:85
  35:     0x7f96a30589ae - core::result::unwrap_failed::hb46be88fa7fb110a
  36:     0x7f96a3068e59 - diesel::main::h2e14d441621c07c8
  37:     0x7f96a30a0b23 - std::rt::lang_start::{{closure}}::h11d5e04761a174e9
  38:     0x7f96a3203ba3 - std::rt::lang_start_internal::{{closure}}::hea72f6871a61f2ce
                               at src/libstd/rt.rs:49
  39:     0x7f96a3203ba3 - std::panicking::try::do_call::h08da391def37af73
                               at src/libstd/panicking.rs:294
  40:     0x7f96a320adba - __rust_maybe_catch_panic
                               at src/libpanic_unwind/lib.rs:82
  41:     0x7f96a32046ad - std::panicking::try::h1c5d32a15c34ebef
                               at src/libstd/panicking.rs:273
  42:     0x7f96a32046ad - std::panic::catch_unwind::h258e2ebe83b8e3d5
                               at src/libstd/panic.rs:388
  43:     0x7f96a32046ad - std::rt::lang_start_internal::h17b31dbbc70ec76f
                               at src/libstd/rt.rs:48
  44:     0x7f96a306a822 - main
  45:     0x7f96a1dd02e1 - __libc_start_main
  46:     0x7f96a304658a - _start
  47:                0x0 - <unknown>
thread panicked while panicking. aborting.
Illegal instruction (core dumped)

Are you seeing any additional errors?
Steps to reproduce
Known to be broken with https://gitlab.com/Nordgedanken/bunnysync . Propably not related to the project itself

Install windows 10 WSL
Do the usual diesel setup for the cli (Installing, rust, postgres and so on)
Run diesel migration run

Checklist

[x] I have already looked over the issue tracker for similar issues.
[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)

The back trace indicates that this happens in code generated by #[derive(Debug)]. Could someone check if that's fixed by a newer rust version?

The back trace indicates that this happens in code generated by #[derive(Debug)]. Could someone check if that's fixed by a newer rust version?


Can not build locally

Current error on build:
./main.go:28:15: cannot use commands.Commands (type []"github.com/EngineerBetter/concourse-up/vendor/gopkg.in/urfave/cli.v1".Command) as type []"gopkg.in/urfave/cli.v1".Command in assignment
./main.go:29:12: cannot use commands.GlobalFlags (type []"github.com/EngineerBetter/concourse-up/vendor/gopkg.in/urfave/cli.v1".Flag) as type []"gopkg.in/urfave/cli.v1".Flag in assignment

I am having to build locally because I want to use my own VPC and subnets for this.  I have edited the terraform main.tf to accommodate this.
Any help would be greatly appreciated.
BTW:  For those new to go(me), you should probably list the dependancies needed for the build including all the go get github.com/*  as it took me a while to figure it out.
Thanks

I got this to work and build locally.  I am going to update the code to allow for -vpc_id, -public_subnet, -private_subnet_a, -private_subnet_b and -office_cidr_blocks(array).  I will put in PR when complete.
This will allow anyone to deploy into their current VPC.

In case anybody else hits this, this is a result of attempting to build outside of  your $GOPATH.
With concourse-up, or any other Go program, you must clone the repository in a specific location. go get handles this for you. if you go get -d github.com/EngineerBetter/concourse-up the repository will be cloned in $GOPATH/src/github.com/EngineerBetter/concourse-up and the build local script will work.


XSS on HTML report

I'm using goaccess version 1.2 installed using apt-get. I found XSS on the HTML report with payload
/<img%20src=x%20onerror=alert(1)>

Thanks for reporting this. Could you please test this out in the latest verson? v1.4. Let me know if that solves the issue.

Tested on version v1.4 but the same bug appears there as well.

Can you please post the offending lines directly from your access log so I can take a look at it? Thanks

[03/Aug/2020:23:34:08 +0545] "GET /?%3Cimg%20src=x%20onerror=alert(1)%3E HTTP/1.1" 200 409 "-" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.117 Safari/537.36"

Could you please re-test this on v1.4? I don't get any issues with the line you posted. Thanks

Request: http://localhost/?data=<img src=x onerror=alert(1)>
Go access : ./goaccess /var/log/apache2/access.log -o ~/Documents/php/www/report.html --real-time-html
Version: 1.4
Downloaded : https://tar.goaccess.io/goaccess-1.4.tar.gz

Thank you again for reporting this. I've pushed a fix that should address the issue. Please test it out by building from development and let me know if that fixes the issue on your end.

It is fixed on the development version. Thank you.


[Feature] Ability to set how many newlines is required for a break in the coloring

It would be nice to be able to change how many newlines is required for the colored line/bar to break, it would make it look a bit neater. I like using newlines to break code up into smaller blocks, but it makes the indents look a bit wonky, so having a continuous line would look better, but i also want them to break when i use multiple newlines between functions etc.

https://github.com/jfcherng-sublime/ST-RainbowIndent#known-issues

Ah, i see. I will have to find some other solution then, thanks for the answer and the plugin :)

Ah, i see. I will have to find some other solution then, thanks for the answer and the plugin :)

It looks better with transparent background imho.


Linux machine file timestamps do not show correctly in MacOS

Decription:
When accessing linux machine files in mac os, the timestamps reflect the date of file creation, not last change.
To reproduce:

Create file in linux machine. Make note of the time of creation. A little later, save a change to that file under the same name.
ls -l in linux machine terminal shows timestamp of last change (as expected).
ls -l in mac terminal or mac finder shows timestamp of file creation

Expected behavior
Mac fileshare (ls -l, stat, Finder) shows same "last changed" timestamp as in linux machine.
Info report
OrbStack info:
  Version: 0.10.2 (100200)
  Commit: e502994dbd2170853d7d7d8614018af65ff6d17d (v0.10.2)

System info:
  macOS: 13.3.1 (22E261)
  CPU: arm64, 8 cores
  CPU model: Apple M1

Fixed for the next version.

Released in v0.11.1.


.list-inline does not present items inline

V4 Alpha 2 coming from bootstrapcdn.com:
https://jsfiddle.net/7hd6qmtr/
A list set as inline still shows stacked vertically however it does remove the bullets.

Your example is missing the requisite .list-inline-item class on the items. See http://v4-alpha.getbootstrap.com/content/typography/#inline

Thank you very much =)
Perhaps the documentation for lists could link/allude to this?
I had looked through the docs rather thoroughly (or so I thought) and obviously missed this.
Again, thank you =)

Yeah, I suppose we could try to highlight the requirement more.

Fixed in 2cc7567.


Update requirements.txt

This is to fix the Known security vulnerability detected as of April 17th

Built warp-ctc
built and installed ngraph in virtualenv, visually inspected output to validate that requests 2.20 was downloaded
make test_cpu
found error in onnx test:
__________________________________ ERROR collecting ngraph/frontends/onnx/tests/test_backend.py __________________________________
ngraph/frontends/onnx/tests/test_backend.py:6: in
import onnx.backend.test
../.venv27/local/lib/python2.7/site-packages/onnx/init.py:6: in
from .onnx_pb import *  # noqa
../.venv27/local/lib/python2.7/site-packages/onnx/onnx_pb.py:8: in
from .onnx_pb2 import *  # noqa
../.venv27/local/lib/python2.7/site-packages/onnx/onnx_pb2.py:213: in
options=None, file=DESCRIPTOR),
E   TypeError: new() got an unexpected keyword argument 'file'
commented out the onnx tests, all other tests were successfully run...
Will open a bug on the onnx issue, but the security issue appears to be resolved, so please merge...


Unnable to use local file for jsrepository.json

Hello,
To workaround the recurrent problem of certificate on remote download I tried to use a local version of the file using this configuration :
dependencyCheck {
nvd {
apiKey = 'XXX'
delay = 16000
}
analyzers {
retirejs {
retireJsUrl= "file:///C:/IDE/jsrepository.json"
}
}
}
But it keep trying to access to "https://raw.githubusercontent.com/RetireJS/retire.js/0fe12bb575eda270536dc0ce47fca0dd17b6ab33/repository/jsrepository.json'
What's wrong ?
Thanks
Loïc

The code uses https://raw.githubusercontent.com/Retirejs/retire.js/master/repository/jsrepository.json as a location so my gut feel is that something in your system configures the commit-hash based jsrepository.json and overrides what you quoted in some way. I'm not familiar enough with the gradle configuration to guide you on where to look.

what version of ODC are you using and how are you calling the task in gradle? If you run with --debug you will see a section where all of the properties are written out - you can search for the internal property analyzer.retirejs.repo.js.url; it will likely be displayed twice and the second value is the one that will be used.

Hello,
The key is 4 time in the debug :
The third one is the correct local file but the "last" one is back to the remote file on github.
The version used :
org.owasp:dependency-check-gradle:9.0.7
Thanks for the support
Regards
Loïc

If the key is listed 4 times you are running ODC more than once. How are you actually calling the task?

Just with "gradlew dependencyCheckAnalyze" on the root project. (It's a sub-projects configuration

you don't have any dependsOn setup in the build.gradle? I'd need to see more of the build.gradle to be able to help.

wait - you said sub-projects. If you are only trying to run dependency-check on a single sub-project then you need to run it like:
./gradlew :subproject_name:dependencyCheckAnalyze
My bet is, you are running ODC twice, once with the correct configuration and once with the default configuration.

You're right the problem didn't occure anymore. Thanks


policy: clean a duplicated code

This change is

Coverage decreased (-0.03%) to 44.766% when pulling 3dbb3e78c5470bfbece7ea86111f05df2d7ecc11 on zhiyuan0x:clean into 394e72478a8d120dab0bff2c41db77695877ce57 on cilium:master.

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please


Remove SidebarBottom

Details
Clean up step related to the addition of the Settings page which is now accessible via the avatar.
Fixed Issues
❌
Tests

Verify the app links and sign out text + display name is not present in the Left Hand Navigation

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Seems pretty straightforward. I'll test on each platform though just in case.

Seems pretty straightforward. I'll test on each platform though just in case.

Thanks @Luke9389 I'm testing on iOS and Android now

Thanks @Luke9389 I'm testing on iOS and Android now

Man, mobile development is usually a pretty error-ridden process for me, and this time–shockingly–turned out to be smooth sailin'. Looks good on all platforms. Nice work! This app is really comin' together.

Man, mobile development is usually a pretty error-ridden process for me, and this time–shockingly–turned out to be smooth sailin'. Looks good on all platforms. Nice work! This app is really comin' together.

Will merge after conflicts are resolved. They looked trivial but I didn't want to touch it (wasn't sure if it's bad form to resolve conflicts on behalf of the PR author).

Will merge after conflicts are resolved. They looked trivial but I didn't want to touch it (wasn't sure if it's bad form to resolve conflicts on behalf of the PR author).

Thanks fixed now.

Thanks fixed now.


Replace the "SessionAdmin" class' readonly_fields with has_add_permission and has_change_permission

If you go to the "Add session" form (/admin/qsessions/session/add/), the following exception is thrown due to line 102 in admin.py:
TypeError: '>' not supported between instances of 'NoneType' and 'datetime.datetime'

But even if that bug wasn't present, there would be no point in going to the "Add session" form anyway since all of the fields are read-only.
Similarly, since all of the fields are read-only, the "Change session" form's "SAVE", "Save and add another", and "Save and continue editing" buttons are all kind of confusing since there's nothing to save.
This pull request disables adding and changing, which fixes the exception and makes the interface make more sense.

Hi @ataylor32
Thanks for your contribution. Ignore the failed test. It's because of a misconfiguration. I'll fix it.


Dilato: Core: There are two "euc-jp" in the code list

Testing Build: 1.10.0-17448-852
Steps:

Launch Brackets
Click File -> New
Expend the code list

Result: There are two "euc-jp" in the code list
Expect: There should only one "euc-jp" in the code list

Verify this issue with build 1.10.0-17313, Fixed


feat(Pointwise): growth of finsets

This PR proves the monotonicity of n ↦ s ^ n under various conditions. Also rename empty_nsmul to nsmul_empty.
From GrowthInGroups

@loefflerd, please don't use WIP on other people's PRs. Here, awaiting-CI is more appropriate.

"Awaiting CI" does not fit for a PR where the latest CI run had completed with a failure, as was the case when I tagged it (while browsing #queue for low-hanging fruit).

"Awaiting CI" does not fit for a PR where the latest CI run had completed with a failure

Where did you get that from? awaiting-CI means "Once this PR passes CI, it is ready for you". In contrast, WIP means "Once this PR passes CI, please don't review just yet because I might have to do other changes". This is why only the former is automatically removed when CI passes.

Yes that's confusing! 😁 cc @grunweg if you want to investigate

If I can say something useful depends on that "#queue" refers to. The zulip linkifier #queue is just a shorthand to some github search (currently [this one]), which I did not work on. I did work on the "queueboard" a lot recently. Sadly, at the moment its data is not always fully in sync with github's.
I have several ideas why and plans how to fix this, but making this robust will require some work :grimacing: That said: the queue dashboard uses the same data as github, so should be reliable.

bors merge


Add Please! console.

CEnemy.prototype.Walk = function(){ this.Damage(1000);}
CEnemyManager.prototype.BuildEnemy = function(){};

one tap -
salien monster no spon? -

You can edit the script yourselves you know? Just isntall the script then add that line to it.


No iteration limit

This PR eliminates an iteration limit for the Rival's evaluation loop.
The reason is: iteration limit may terminate the evaluation early than necessary.
Especially it matters for evaluation of an expression with numerous uncertainties (4+).
Such expressions may require more than 5 iterations to define a correct rounding.
The introduced feature makes the evaluation loop more robust.

The branch turned out to be very expensive. The branch has introduced 2x slowdown to the overall runtime, the branch mostly affects unevaluable points.
Nothing is wrong with iterating with a stop condition only as a maximum precision.
However, testing of the branch has discovered that expressions like (... (log (exp x))) may stuck to be evaluating forever due to overflow at (exp x) for large x. Amplification factors for this example converge but due to limitation of 30bits of exponent for an MPFR value, we are not able to define a correct rounding. In other words, precision is assigned correctly, but exponent bits limitation stops us from actually converging.
As it was said in the comments above, for completeness, Rival should keep adding slack to (exp x) until the maximum precision is reached - this will allow to at least pretend that we have tried the best we could.
It is a very expensive solution and comparison with baseline shows that in our benchmarks we do not even have this edge case.
The edge case is very rare and considered to be neglected given the price for it.
In general, if an user feels that 5 as a max iterations is not enough, may set a larger number - which won't introduce too much of slowdown.


Fix for ruby 2.1.x

This seems to work, but we are now seeing the following:
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest

I think those messages are coming from the right_aws gem. It appears that has been deprecated in favor of the new right_aws_api gem.

Any plan to move over to right_as_api? I'm seeing this all the time:
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest
Digest::Digest is deprecated; use Digest```

@adamyonk Planned but no work has started on it.


[tools] Never-ending config_and_overrides scan

Commit f3c0ea3664fea45233413ebabe29575b778f55aa introduced a regression in the build tools so that an app specifying target_overrides causes a never-ending scan of configs and overrides.
Building project mbed-os-example-uvisor (K64F, GCC_ARM)
Scan: .
Scan: FEATURE_BLE
Scan: FEATURE_CLIENT
Scan: FEATURE_COMMON_PAL
Scan: FEATURE_UVISOR
Scan: FEATURE_IPV4
Scan: FEATURE_IPV6
Scan: FEATURE_STORAGE
Scan: mbed
Scan: env
(hangs here)

To reproduce:
$ mbed import https://github.com/ARMmbed/mbed-os-example-uvisor
$ cd mbed-os-example-uvisor/mbed-os
$ git checkout f3c0ea3664fea45233413ebabe29575b778f55aa
$ cd ..
$ mbed compile -m K64F -t GCC_ARM

You can look at the mbed_app.json file to see what was defined there:
{
    "target_overrides": {
        "K64F": {
            "target.features_add": ["UVISOR"],
            "target.extra_labels_add": ["UVISOR_SUPPORTED"]
        }
    }
}

If you checkout the previous commit the problem does not show up:
$ cd mbed-os
$ mbed checkout f3c0ea3664fea45233413ebabe29575b778f55aa~1
$ cd ..
$ mbed compile -m K64F -t GCC_ARM

@geky @0xc0170

Thanks for narrowing this down! Let me know if https://github.com/mbedmicro/mbed/pull/2317 works for you.

ARM Internal Ref: IOTMORF-96


Brush Editor opens up when clicking on every button.

I was trying to make my first ANSI text screen and I tried to setup the brush. Everytime I click on a different button, the brush editor opens up and no cursor changes whatsoever. Even minimizing the window opens up the brush editor.

Hey @SoundCrafterYT, thanks for reporting the issue.  What version are you using?

The latest version from this site

sooo.... 3.3.12-beta?  Sorry but it really helps if you are explicit.  Also, do you have exact steps to reproduce the issue?  What button(s) exactly are you pressing and in what order?  Any additional information would help greatly.

I was first clicking on the File > New to make a new Text page. Then I clicked on the text tool to see if the program actually works on this one. After that, I wanted to start making things with the Brush Editor. I chose the color and the character I wanted, then the whole window became the brush editor (Including its taskbar button).


fix typo of URL of sbol data model specification

Inside constant.py file the URL of SBOL data model specification is not correct.
I am not sure if it is better to change the URL itself to look like the one written in file or change the one in file for now until next update.
currently this make the URL inside the file send the user to SBOL data model specification website.
fixes #424

Finally reverted the file
Now, the .gitignore file in my branch is the same as the one on the main


⚠ Develop API Endpoint has degraded performance

In c46b130, Develop API Endpoint (http://develop.roblox.com/) experienced degraded performance:

HTTP code: 200
Response time: 2965 ms

Resolved: Develop API Endpoint performance has improved in 9c88c0d after 4 minutes.


new version not available on bintray

Would be great, if you also release the 1.1.0 at bintray.

It will be published with the release notes/blob post that is coming out this week.

Just uploaded it.
https://bintray.com/netflixoss/downloads/vector/1.1.0/view


Keyboard events listen for events but do not trigger writes

l want to press "s" button ,and no write is triggered.Write what l want to write
for example

I want "" to replace the output of "s", but "s" still outputs before ""

I want "<s>"to replace the output of "s", but s still outputs before "<s>"


metrics: use tidb_cluster label get variable values

Signed-off-by: zhengjiajin<EMAIL_ADDRESS>
What problem does this PR solve?
The latest version of tidb-operator will use the instance name as the cluster label. The previous expression uses the cluster label got the value of the tidb_cluster variable. It will cause Grafana can not to display the data.
https://github.com/tikv/pd/pull/3389#issuecomment-767977783
What is changed and how it works?
Proposal: xxx
What's Changed:
Related changes

no change

Check List
Tests

No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

No release note

/assign @breeswish

/assign @breeswish

/merge

/merge


Versioned pending coinbase witness

Checklist:

[ ] Tests were added for the new behavior
[ ] All tests pass (CI will check this if you didn't)
[X] Serialized types are in stable-versioned modules
[ ] Does this close issues?

@bkase, the ppx, bin_io and version, do not do the registrar stuff.

the version ppx just adds the variables version and __versioned__

The ppx does not add the registration items. Those items aren't necessary to assure the invariance of the serialization, and can be added later.
We could have another ppx to make the registration items added more concisely.


chore(master): release 1.1.19

:robot: I have created a release beep boop
1.1.19 (2023-08-15)
Bug Fixes

deps: update dependency wayofdev/laravel-webhook-client to v1.1.16 (28c3ec2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v1.1.19 :sunflower:


visualvm: 2.0.4 -> 2.0.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/visualvm/visualvm.src/releases.
meta.description for visualvm is: "A visual interface for viewing information about Java applications"
meta.homepage for visualvm is: "https://visualvm.github.io"
meta.changelog for visualvm is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.0.5 with grep in /nix/store/ccypz4fdc7f2gfx488qyyn6x43aa24hz-visualvm-2.0.5

directory tree listing: https://gist.github.com/5da08e91d4a21db7a54ab8013d9fc644

du listing: https://gist.github.com/0b6072aad508eb6d4e186fc6bcfb9148

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
visualvm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ccypz4fdc7f2gfx488qyyn6x43aa24hz-visualvm-2.0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A visualvm https://github.com/r-ryantm/nixpkgs/archive/5487927b06a12d7ece44141d4ec52448954c781b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ccypz4fdc7f2gfx488qyyn6x43aa24hz-visualvm-2.0.5
ls -la /nix/store/ccypz4fdc7f2gfx488qyyn6x43aa24hz-visualvm-2.0.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

visualvm

Maintainer pings
cc @michalrus @moaxcp for testing.

Result of nixpkgs-review pr 105106 run on x86_64-darwin 1

1 package built:

visualvm


Login issue?

Hi,
The bot seems to log in, however it never asks me to enter the auth code from my email... and then it never buys or sells anything.
----- Starting scan... -----
Current Portfolio: ['GE']
Current Watchlist: ['AAL', 'OXY', 'EXPE', 'TSLA', 'NFLX', 'MSFT', 'SBUX', 'BABA', 'BAC', 'GE', 'OGZPY', 'MZDAY']
----- Scanning portfolio for stocks to sell -----
----- Scanning watchlist for stocks to buy -----
----- Scan over -----

Always just runs like this... doesn't take any actions.

It's working, it's just that none of the stocks on your watchlist happened to meet the requirements to buy/sell. You can probably get it to trigger a buy/sell by using exponential moving averages instead, try replacing lines 177-178 in main.py with ta.trend.EMAIndicator instead of ta.volatility.bollinger_mavg.

Thank you, however something is not right, the watchlist doesnt match what I have in there. I removed the old one and created a new one. with totally different stocks in there and it still shows the old list. It was over 24 hours ago that I have replaced them.
The current PORTFOLIO updates properly.

Also, Ive tried to change my password(put a wrong password) and it gave me exactly the same results... meaning that something is not working right with the login.


Celo Governance Call #38 | December 8th, 2023

Celo Governance Call #38

When: Friday, December 8th, 2023. 8:30 am PT / 12:30 am ET / 4:30 pm UTC 60 minutes max
Where: https://lu.ma/CGC-0038
Notes: TBD
Video: TBD

Agenda
### CGP-TBD: Doktor Guild
- [ ] https://forum.celo.org/t/proposal-doktor-guild/6931
- [ ] CGP-TBD

### CGP-TBD: Mentors Collective
- [ ] https://forum.celo.org/t/mentors-collective-hello-world/6884
- [ ] CGP-TBD

Hello @juanjgiraldoc We would like to discuss this governance proposal during the call, can it be added to the agenda please. Thanks.

Call movet to Dec 8th

Hey @juanjgiraldoc
We would like to discuss our governance proposal
, Mentors Collective, during the community call.
Can you please add it to the agenda?

Please add Opera's Governance Proposal to the agenda. Original forum post.

Done @ericnakagawa ☑

Good Morning @juanjgiraldoc . I would like to present our Music dapp proposal in the next community call. I will appreciate if it is added to the agenda.  Here is a link to the proposal on the forum https://forum.celo.org/t/w3-music-funding-to-launch-the-first-music-marketplace-on-cello-blockchain/7028?u=larkim..
Thank You


Docs

Thanks for sharing this project, sir. Will you add a complete unrelated section on how you generated your docs?

I can, although the method I used isn't very elegant. The docs are hand-written in the /src/example/ directory. The /docs/ directory is generated with webpack. I had to tweak the ReasonReact template's webpack config to make it work.
I'm hesitant to document the process just because it's hacky and I don't want to implicitly recommend that other people use it.
You can browse /src/example and webpack.config.js to see exactly what's happening.

Thank you, sir.


Container build fails after docker log limit reached

Output of the ue4-docker info command:
ue4-docker.txt
Additional details:

Are you accessing the network through a proxy server? No

I am attempting to build a ue4-engine image using ue4-docker running in a container built by the following Dockerfile:
FROM ubuntu:focal AS build
# set the timezone, add the docker repository, and install ue4-docker and dependencies
RUN ln -fs /usr/share/zoneinfo/America/Detroit /etc/localtime &&\
    apt-get update && apt-get install -y \
    apt-transport-https \
    ca-certificates \
    gnupg \
    curl \
    lsb-release &&\
    curl -fsSL https://download.docker.com/linux/ubuntu/gpg | gpg --dearmor -o /usr/share/keyrings/docker-archive-keyring.gpg &&\
    echo \
    "deb [arch=$(dpkg --print-architecture) signed-by=/usr/share/keyrings/docker-archive-keyring.gpg] https://download.docker.com/linux/ubuntu \
    $(lsb_release -cs) stable" | tee /etc/apt/sources.list.d/docker.list > /dev/null &&\
    apt-get update && apt-get install -y \
    docker-ce \
    docker-ce-cli \
    containerd.io \
    python3 \
    python3-dev \
    python3-pip  &&\
    pip3 install ue4-docker

CMD ue4-docker info

and invoked by the following gitlab-ci job.
build-unreal-engine:
  stage: build
  needs:
    - job: build-ue4-docker
      optional: true
  image: $REGISTRY_URL/ue4-docker:$CI_COMMIT_REF_SLUG
  rules:
    - if: $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH
  script:
    - export UE4DOCKER_TAG_NAMESPACE=$REGISTRY_URL
    - ue4-docker build $VERSION -username=$GITHUB_USERNAME -password=$GITHUB_PASSWORD --cuda=10.2 --no-minimal
    - docker push $REGISTRY_URL/ue4-engine:$VERSION

This job fails after ~45 minutes. Unfortunately, it is impossible to tell what is failing because the docker logs are getting clipped ~26 minutes into the build with [output clipped, log limit 1MiB reached]
Is there any way to reduce the log output produced by ue4-docker?
Alternatively, what configuration changes can be made to increase the docker build log size limit?

Per https://github.com/docker/buildx/issues/484, that log clipping is coming from BuildKit, not Docker.
I see BUILDKIT_STEP_LOG_MAX_SIZE=10000000 in your ue4-docker.txt, which implies (to me) a 10MB log limit, so I'm not sure why that didn't work. Maybe that env-var is no longer supported?
You might need to poke around in the above-linked issue to work out how correctly tell BuildKit to allow a larger step log. It seems like maybe you need to create a new builder with the larger log limit? I don't have the code in front of me, but I don't recall if we have a way for the ue4-docker command to be told to use a different buildx builder than the default, and that issue didn't seem to have a way to change the default builder's config.
A workaround would be to use ue4-docker's "layout" mode to export the desired build as a Dockerfile, and then execute it using something like https://github.com/docker/buildx/issues/484#issuecomment-812997137 (although that's going to create a new builder each time, so consider refining it if you go this way)

I suspect that BUILDKIT_STEP_LOG_MAX_SIZE needs to be in env variables of docker daemon.

To be honest, I completely forgot about this weird log thing when did #229.

This issue was ultimately resolved by adding the following to /etc/systemd/system/docker.service.d/buildkit-environment.conf and restarting the daemon.
[Service]
Environment="DOCKER_BUILDKIT=1"
Environment="BUILDKIT_STEP_LOG_MAX_SIZE=10000000"
Environment="BUILDKIT_STEP_LOG_MAX_SPEED=10000000"


⚠ Bofrost.nl has degraded performance

In 2885e08, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1373 ms

Resolved: Bofrost.nl performance has improved in 2da62ec after 9 minutes.


No progress after Starting training Epoch 1/10

I am training keras faster RCNN on kitti dataset I configured the text file and used simple parser with VGG net

Parsing annotation files
Training images per class:
{'Car': 28742, 'Cyclist': 1627, 'Pedestrian': 4487, 'bg': 0}
Num classes (including bg) = 4
Config has been written to config.pickle, and can be loaded when testing to ensure correct results
Num train samples 6373
Num val samples 1108
loading weights from /root/.keras/models/vgg16_weights_tf_dim_ordering_tf_kernels_notop.h5
Starting training
Epoch 1/10

After this I am not getting any output. Its been running for more than 17 hrs and still stuck here.
I am using NVIDIA 1060 6GB GPU. GPU is configured properly and it is running properly when I run a keras classification network but There is no GPU involvement in this case at all
I am using python 3 in ubuntu .

Are you using python 3.5?

Oh thanks. I will try it and will let you know

I guess #169 is the problem. Will let you know if it works. And yes i am using python 3.5


Jac/no parsers from commands

Major changes
1 - restructure into a single main file (tabcmd.py) and all other files in a module, including a new module called 'execution' (not a great name, open to other ideas!)

(Making absolute references to modules means we can stop listing things in the init files)

2 - edit all parsers to return values only by adding them as attributes on args
3 - add tests that call each Command
Next: replace our manual identification of which Command to call with letting argparse identify the right one (this will also fix #23 and all the other commands that need to have a name with no --option designation)

How does the code handle cases where uses don't have to provide a flag. For example, in the Creategroup command of Tabcmd1, users can just type tabcmd creategroup "Development" instead of tabcmd2 creategroup --name "ExampleGroup"


[Loc] updates to sql-migration and sql

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.008%) to 42.15%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
28578

Relevant Lines:
62967

💛  - Coveralls


dnvm upgrade can't install DNX runtimes

Followed the instructions on here: http://docs.asp.net/en/latest/getting-started/installing-on-linux.html
The apt packages install, dnvm installs and runs, but running dnvm upgrade to install the CoreCLR causes this error:
root@localhost:~# dnvm upgrade -r coreclr
Determining latest version
bash: /dev/fd/62: No such file or directory
Error: Could not find latest version from feed https://www.nuget.org/api/v2
root@localhost:~# bash: echo: write error: Broken pipe

root@localhost:~#

Is there something I'm missing?

It's known issue, related to: #14
MS is working on fix.

Did some more digging because I was curious and the error didn't seem like the others in #14 . The following line breaks in dnvm.sh (line 629):
read versionOrAlias downloadUrl < <(__dnvm_find_latest "$runtime" "$arch" "$os")
__dnvm_find_latests ends with an echo statement, so I sort of reproduced it below:
root@localhost:~# hi=hello
root@localhost:~# echo $hi
hello
root@localhost:~# read hello < <($(echo $hi))
bash: /dev/fd/62: No such file or directory
root@localhost:~# The program 'hello' can be found in the following packages:
 * hello
 * hello-debhelper
Try: apt-get install <selected package>
The program 'hello' can be found in the following packages:
 * hello
 * hello-debhelper
Try: apt-get install <selected package>

root@localhost:~#

Then just using plain process subsitution:
root@localhost:~# read hello < <(ls -la)
bash: /dev/fd/62: No such file or directory
root@localhost:~# ls: write error: Broken pipe

root@localhost:~#

I would expect the following (this is in Cygwin...don't have a Linux install nearby):
duraz0rz@Express-Emotion ~
$ echo $hello

duraz0rz@Express-Emotion ~
$ read hello < <(ls -la)

duraz0rz@Express-Emotion ~
$ echo $hello
total 46K

It seems like creating the temporary named pipe (/dev/fd/62) for the process subsitution is not working. Assuming mkfifo is used to create the named pipe, #71 should fix this issue.

@duraz0rz  There is no device like /dev/fd/.
Type ls /dev/ in bash.
There is similar problem in
https://github.com/Microsoft/BashOnWindows/issues/14#issuecomment-206737703

/dev/fd/ is symlinked to /proc/self/fd/ in other systems. I'm assuming <(ls

la) runs the program in another process, creates a named pipe called
/dev/fd/62, and pipes the output of the spawned process to that named pipe
before piping it into the other process.

On Tue, Apr 19, 2016, 01:05 Patryk Golus<EMAIL_ADDRESS>wrote:

@duraz0rz https://github.com/duraz0rz There is no device like /dev/fd/.
Type ls /dev/ in bash.
There is similar problem in
#14 (comment)
https://github.com/Microsoft/BashOnWindows/issues/14#issuecomment-206737703
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/BashOnWindows/issues/217#issuecomment-211730275

Process substitution is working in a more recent Windows Insider build.  Please reopen if you continue to see issues.


Add aozora bunko datasets.

This pull request adds the support of AozoraBunko
issue #46

Thank you for your review !


System.Exception: Error sending mail

Hi Eitan,
thanks for your job.
I try your script but i have a problem to send email.
I have this message :

Msg 6522, Niveau 16, État 1, Procédure clr_send_ics_invite, Ligne 0 [Ligne de départ du lot 0]
Une erreur .NET Framework s'est produite au cours de l'exécution de la routine ou de la fonction d'agrégation définie par l'utilisateur "clr_send_ics_invite" :
System.Exception: Error sending mail: Échec d'envoi du courrier.
System.Exception:
à StoredProcedures.clr_send_ics_invite(SqlString profile_name, SqlString recipients, SqlString copy_recipients, SqlString blind_copy_recipients, SqlString from_address, SqlString reply_to, SqlString subject, SqlString body, SqlString body_format, SqlString importance, SqlString sensitivity, SqlString file_attachments, SqlString location, SqlDateTime start_time_utc, SqlDateTime end_time_utc, SqlDateTime timestamp_utc, SqlString method, SqlInt32 sequence, SqlString prod_id, SqlBoolean use_reminder, SqlInt32 reminder_minutes, SqlBoolean require_rsvp, SqlString recipients_role, SqlString copy_recipients_role, SqlString blind_copy_recipients_role, SqlString smtp_servername, SqlInt32 port, SqlBoolean enable_ssl, SqlBoolean use_default_credentials, SqlString username, SqlString password, SqlBoolean suppress_info_messages, SqlString& event_identifier, SqlString& ics_contents)
.

I use this query :
DECLARE @EventID nvarchar(255)

EXEC sp_send_calendar_event
@from_address = N'xxxxxxxxxx@xxxxxxxxxx',
@recipients = N'xxxxxxxxxx@xxxxxxxxxx',
@subject = N'let us meet for pizza!',
@body = N'Bring your own beer',
@body_format = N'HTML',
@location = N'The Pizza place at Hank and Errison corner',
@start_time_utc = '2019-07-02 19:00',
@end_time_utc = '2019-07-02 23:00',
@timestamp_utc = '',
@smtp_servername = 'X.X.X.X',
@port = 25,
@enable_ssl = 0,
@username = N'xxxxxxxxxx@xxxxxxxxxx',
@password = N'xxxxxxxxxxxxxxxxxxx',
@use_reminder = 1,
@reminder_minutes = 300,
@require_rsvp = 0,
--@method = 'CANCEL',
@event_identifier = @EventID OUTPUT
SELECT EventID = @EventID

I test SMTP server and it's ok.
Can you help me please ?
Julien

Hi Julien,
Based on the error, there seems to be an issue while sending the e-mail to the SMTP server.
Please make sure that all the SMTP settings you provided are correct (username, password, SSL, port, server name), and also that the sender address is an address that's allowed as a sender in your SMTP server.
You could try using DBMail at your SQL Server to test the same SMTP settings and see if it works from there.
Also, I believe that setting the @timestamp_utc parameter to an empty string could possibly cause an issue:
@timestamp_utc = '',

Please either set it to NULL, or don't set it at all.

Hi Eitan,
Thanks for your response.
I test a another SMTP software (JBMail plus 3.3) on the same server with same settings and email sent successfully.
After i tried with

@timestamp_utc = NULL,

but result is NOK,
Julien

I see.
Unfortunately, the error message you got is not enough to properly troubleshoot the issue.
I just published a new release with improved error handling.
Hopefully, you'll be able to see the actual underlying issue.
Please try this one:
https://github.com/MadeiraData/sql-clr-ics/releases/tag/v2.2-beta

Eitan,
It's ok with new release. I obtain an EventID and i receive a calendar invitation without RSVP requirement.
Thanks for all.
Julien

I see. Must've been an issue with the older version, then.
I'm glad it worked out!


Clarify customer-owned backup features

Fixes DOC-7322

Added more detail to the Support product section on backup feature pages.
Linked back to the definition of customer-owned backups.
Made the Backup Type section on the overview page a general backup type section rather than only Cloud.
Corrected errors.
Simplified the start of Managed-service/Customer-owned backup pages to reduce confusion over the two types. Now they just link back to the overview page and only mention the type of backup that is on the page.

Preview
https://deploy-preview-16622--cockroachdb-docs.netlify.app/docs/stable/backup-and-restore-overview.html
https://deploy-preview-16622--cockroachdb-docs.netlify.app/docs/stable/take-and-restore-encrypted-backups.html#supported-products

@andf-crl Would you mind taking a look at this one, if you have time?


Broken haptic pulse with IL2CPP

In openvr_api.cs the method TriggerHapticPulse(uint unControllerDeviceIndex, uint unAxisId, char usDurationMicroSec) defines usDurationMicroSec as a char. In the signature of the native OpenVR method usDurationMicroSec is defined as an unsigned short, so the corresponding type in C# should be ushort.
This has never been a problem before with Mono builds, but it is now with Unity 2018 and IL2CPP for Windows Standalone. Running an IL2CPP build of my game, I noticed the haptic pulse was extremely weak. I'm guessing usDurationMicroSec is clamped down to a C++ char, which has a maximum value of 127.
I'm not sure if it's a bug in IL2CPP or if it's expected behaviour, but the fix is very easy: just make usDurationMicroSec a ulong in the signature of TriggerHapticPulse in openvr_api.cs. It will work perfectly both with Mono and IL2CPP.

See https://github.com/ValveSoftware/openvr/issues/472 from April 14th 2017.

Closing as a duplicate of https://github.com/ValveSoftware/openvr/issues/472.


chore(cache,gateway,model,standby): rename GUILD_EMOJIS intent and make corresponding changes

Looks good, just need you to make these changes before we can merge it

Done.


add 04-advanced-use.en.md

i translated the 04-advanced-use.md into English version but i didn't change the part of shell to ensure it's correct

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


How to set it to a custom root?

IE:
new WinBox("Window Title", {
root: document.body
});
How is the above translated in your work, I couldn't seem to find it.

After I saw this issue, I went through the original code of WinBox.js and realized that the implementation of root parameter is far from ideal.
The position style of WinBox.js is fixed rather than absolute, this can make it difficult for the positioning relative to the root element.
In current version (v0.2.6) of WinBox.js, the root_w and root_h properties used for positioning the window are just the copies of document.documentElement.width|height, and have no relation to the root parameter passed in. The root parameter now only determines the location of WinBox.js element in DOM tree, which has little practical significance, since as mentioned before, the winbox element is actually fixed. There is little difference from placing winboxes all in document.body.
I will continue with further optimization of CSS styles in WinBox.js with the author, as well as making the root parameter more effective. Before that, if you really want to try to change the root parameter, you can directly add a root={HTMLElement} prop to the React WinBox component (and ignore TypeScript errors, if any). And also welcome to propose more effective solutions. Thank you.


Validate that flash algos can be constructed

The test fails because stm32l4r9i_eval is at least 29K, and uses 4K pages, which is more than what 32KB devices can work with. I'm hoping that updating the L4 pack solves this.


Customizable lock icon for Members Only content

Allow users to select from pre-existing lock icons or upload their own icon so that the icon is consistent with the look and feel of their site.

Assets provided by Fabian @melissahenderson for your triage in due course.
lock-icon.zip


[BugFix] Fix the compatibility issue of TSchemaTableType

    SCH_ROUTINE_LOAD_JOBS,
    SCH_STREAM_LOADS,
    SCH_PIPE_FILES,
    SCH_PIPES

introduced by https://github.com/StarRocks/starrocks/pull/23649 and https://github.com/StarRocks/starrocks/pull/24663
These four fields were added in 3.2, which has not yet been released.
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.2


Fixing specified Dockerfile name

ERROR: Cannot locate specified Dockerfile: dockerfile-71

@Mahmoudz Please revert this request, there is another place to fix.


Workspace : Image support

From<EMAIL_ADDRESS>on 2009-10-20T06:16:44Z
What steps will reproduce the problem?

Create an Image object in python
import Image; img = Image.open(the_filename); 2. 3. What is the expected output? What do you see instead? I would expect the object img to appear in the workspace, but is does not. What version of the product are you using? On what operating system? Spyder 1.0.0
Python 2.6.4rc2, Qt 4.5.2, PyQt 4.6, QScintilla 2.4 on Linux

Please provide any additional information below
.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=90

From pierre.raybaut on 2009-10-20T13:02:18Z
That's because Image objects aren't supported by the Workspace indeed.
This is clearly a good idea, I'll implement this for next release.
Summary: Workspace : Image support
Status: Accepted
Labels: -Type-Defect Type-Enhancement Milestone-Release1.1

From<EMAIL_ADDRESS>on 2009-10-20T21:58:46Z
Why don't you put all kinds of objects to the workspace it could be quite helpful to
have your object in the workspace and on a right click get a list of the class
methods (of course this is nearly impossible if you have 100 methods...)

From pierre.raybaut on 2010-03-31T09:19:41Z
Status: Fixed


Created BST_operations.cpp

i have created a file which does the following operations in bst:

insertion
2.deletion
3.searching
4.printing in inorder type

@Astrodevil i have added it in the dsa cpp folder .could you please verify it.

Put it in proper folder, you can take example from DSA-Java folder and arrange accordingly. Also mention issue no. for this PR


Change the link of "upgrade to version 3.6 by filing a support ticket" to self-help doc

[Enter feedback here]
For below section "upgrade to version 3.6 by filing a support ticket", should be redirected to Upgrade the MongoDB wire protocol version of your Azure Cosmos DB's API for MongoDB account, as currently customer can upgrade the version themselves.
Compound indexes (version 3.2)
Compound indexes hold references to multiple fields of a document. If you want to create a compound index, upgrade to version 3.6 by filing a support request.
Wildcard indexes (version 3.2)
If you want to create a wildcard index, upgrade to version 3.6 by filing a support request.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 60d86c46-2b4b-53a6-da61-af5833b334a2
Version Independent ID: 18b651d7-876d-e5b1-5af6-9e6b2827c8a7
Content: Manage indexing in Azure Cosmos DB's API for MongoDB
Content Source: articles/cosmos-db/mongodb-indexing.md
Service: cosmos-db
Sub-service: cosmosdb-mongo
GitHub Login: @timsander1
Microsoft Alias: tisande

@MS-jeremy Thank you for the feedback. We are actively investigating and will get back to you soon.

@MS-jeremy Thanks for the valuable inputs. However manual upgradation is only available for few qualifying account and not to all the accounts as mentioned in the link. Service request is one way which works for all accounts as of now hence the link. Once this is available to all the accounts, the mentioned link shall be updated too.

@AnuragSharma-MSFT Yes, I know only qualifying accounts can be upgraded via self-help.
But anyway I think in this doc Indexing for MongoDB version 3.2, at least it can mention there is a way to be achieved by self-help. Or actually if customer clicks onto the section Upgrade the MongoDB wire protocol version of your Azure Cosmos DB's API for MongoDB account, and find they are not able to upgrade, they can still file a support request eventually.
So it does no harm by changing the "file support link" to the doc Upgrade the MongoDB wire protocol version of your Azure Cosmos DB's API for MongoDB account anyway, IMHO.
Or you can still assign it to Cosmos DB PM to evaluate. thanks!

@MS-jeremy Makes sense. I am assigning it to content author to evaluate and update as appropriate.

Hi @MS-jeremy, thanks for your feedback. We'll update the link to reference the upgrade doc (at the time the indexing doc was last updated, the upgrade doc didn't exist). Support ticket route still works though (and is required in many cases), so no issue if a customer has already followed the old link.

#please-close


Orange Nav Icon to Indicate what page the user is on

The Nav Icons now change color according to what page the user is currently on! Orange for where they are, grey if its not.

This looks great to me!


Mobile value empty

i have installed this plugin but i am not getting mobile number in the result.

Hi @shivamHylo,
Which OS are you testing the plugin on?
Currently, the plugin can only retrieve mobile numbers on Android devices running version 12 or lower.
For Android 13, some changes will be needed to the implementation to retrieve mobile numbers.
I plan to address this after Capacitor officially adds support for Android 13 with the release of Capacitor 5.
Regarding iOS, Apple does not allow developers to retrieve mobile numbers. You can find more information about this policy in the Apple Developer Forums (Reference: https://developer.apple.com/forums/thread/9171).
Therefore, if you are not able to retrieve mobile numbers on iOS devices, this is expected behavior.

Now that the capacitor 5 beta has been released, will you start working on Android 13 support?

Hi @tarangshah19 @harshall666
I've created a separate issue specifically for Capacitor 5 and Android 13 support
https://github.com/jonz94/capacitor-sim/issues/4
I'd like to suggest that we continue this discussion on there. Thank you!

I have found the issue in android 12, in lower android version it's working fine
Regards,
Shivam Thakur
Developer
m: +91<PHONE_NUMBER> | e: @.***
Hylobiz | India • UAE • USA • Indonesia
[Facebook icon]https://www.facebook.com/gohyloapp/ [LinkedIn icon] https://www.linkedin.com/company/smeneobank/  [Twitter icon] https://twitter.com/HyLoBiz/  [Youtbue icon] https://www.youtube.com/channel/UCHoDm2D46WhKUqNfeIo91Jg  [Instagram icon] https://www.instagram.com/hylobiz/  [Medium icon] @.***>

From: jonz94 @.>
Sent: 10 April 2023 19:19
To: jonz94/capacitor-sim @.>
Cc: Shivam Thakur @.>; Mention @.>
Subject: Re: [jonz94/capacitor-sim] Mobile value empty (Issue #3)
Hi @shivamHylohttps://github.com/shivamHylo, sorry for pinging you again.
Is your end result different from the following?
Currently, the plugin can only retrieve mobile numbers on Android devices running version 12 or lower.
For Android 13, some changes will be needed to the implementation to retrieve mobile numbers.
Regarding iOS, Apple does not allow developers to retrieve mobile numbers. You can find more information about this policy in the Apple Developer Forums (Reference: https://developer.apple.com/forums/thread/9171).
Therefore, if you are not able to retrieve mobile numbers on iOS devices, this is expected behavior.
I would appreciate it if you could provide me with more details, so that I can take a deeper look and address the issue.
—
Reply to this email directly, view it on GitHubhttps://github.com/jonz94/capacitor-sim/issues/3#issuecomment-1501833219, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A6CMKS7YVWHP5JTOJT4HHNDXAQFXPANCNFSM6AAAAAAVFGS3A4.
You are receiving this because you were mentioned.Message ID: @.***>

Hi there,
I just publish a new version.
Upgrade the plugin to<EMAIL_ADDRESS>should be able to fix this issue.


Parse final XML into Python objects

Reference guide: https://www.davekuhlman.org/generateDS.html

https://www.davekuhlman.org/generateDS.html#how-to-use-the-generated-source-code

There may be an issue detecting the top level root element. May need to refactor the schema.

https://www.davekuhlman.org/generateDS.html#recognizing-the-top-level-element


⚠ PrivateHD has degraded performance

In ac1e674, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 5625 ms

Resolved: PrivateHD performance has improved in 0600c25.


Very large audio files should be loaded in sections rather than loading entirely into memory

Currently (as of 0.12.3), segmenting large audio files (> 1h) into more manageable segments is left to the user. If a user was to attempt to process a large audio file, the entire file would be pulled into memory before analyzing. This can lead to OOM killer events, or process crashes.
The library needs to have a method for processing very large audio files.

I've added an initial implementation by subclassing the default objects as LargeRecording and LargeRecordingAnalyzer. If these prove to be as performant as the default objects, I'll consider making these the default in 1.0.


Random character is being added the path in my config

Im running 1.10.1 and in my config I have a rule to move to /home/joey/Downloads/veryfi_triage but if I run organize sim it shows the path as /home/joey/Downloads/veryfi_triage 2. If I run organize run as expected it will create the path with the trailing 2
~ cat .config/organize/config.yaml
rules:
# email amazon invoice
    - folders: ~/Downloads
      subfolders: false
      filters:
          - extension: pdf
          - filename:
                startswith:
                    - 'Amazon.com - Order'
                case_sensitive: true

      actions:
          - move: /home/joey/Downloads/veryfi_triage
          - Shell: 'killall organize'
~  organize sim                                                                                ✔
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Folder /home/joey/Downloads:
  File Amazon.com - Order<PHONE_NUMBER>-8707404.pdf:
    - [Move] Move to "/home/joey/Downloads/veryfi_triage 2"
    - [Shell] $ killall organize
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ SIMULATION ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
~  organize config --debug                                                                             ✔
/home/joey/.config/organize/config.yaml
Your configuration as seen by the parser:
'rules':
- 'actions':
  - 'move': '/home/joey/Downloads/veryfi_triage'
  - 'Shell': 'killall organize'
  'filters':
  - 'extension': 'pdf'
  - 'filename':
      'case_sensitive': !!bool 'true'
      'startswith':
      - 'Amazon.com - Order'
  'folders': '~/Downloads'
  'subfolders': !!bool 'false'

Config file syntax seems fine!
No config problems found.

First guess: move: /home/joey/Downloads/veryfi_triage/ should include a slash at the end?

Please retry with the trailing slash and the newest version. If anything goes wrong please reopen the issue 👍


Add logging config to support json and level selection for logrus

Adds a simple log config section in matchmaker_config.json to add level selection and format selection (text or json). This change also adds a logging package that's now being called from each of the components' inits to configure logging once when each process starts.

Yep looks good.  Will merge after we cut the v.0.2.0 release


Leave Humio's tmp dir on the same mount as humio-data

LGTM though I think we should consider changing how we do humio version comparison to make it easier to read, I would prefer if it said something like
if humioVersion.Before(HumioVersionWithNewTmpDir) {
...
}

to me that's much easier to reach then having to look at the AtLeast assignment and then negate that in my head when trying to understand what the condition needs to be for this to be true

That would certainly be easier to read. For now, I'd like to stick with this approach and then we can improve it a little later given how few places we do this.


Error: module 'degenbot' has no attribute 'UniswapV2Pool'

Hi, It would be nice if you could add some proper docs. I can help with this. For version 0.2.4 I'm getting AttributeError: module 'degenbot' has no attribute 'UniswapV2Pool'

Looks like the pip package is outdated.

The README here on GitHub is out of sync because it has edits for the upcoming 0.3.0 release, which makes some changes to class naming and organization.
If you need to see the 0.2.4 README, the version on PyPI is still aligned with the latest release.
If you want to work with the development branch, you can do a git clone to your local machine and do an editable pip install (pip -e /path/to/local/repo) which will reflect any code changes to that folder your environment.

The README here on GitHub is out of sync because it has edits for the upcoming 0.3.0 release, which makes some changes to class naming and organization.
If you need to see the 0.2.4 README, the version on PyPI is still aligned with the latest release.
If you want to work with the development branch, you can do a git clone to your local machine and do an editable pip install (pip -e /path/to/local/repo) which will reflect any code changes to that folder your environment.

Or we can install directly from pip using tools like rye, hatch, pdm etc.


refactor: use tokio_util::sync::PollSender for ActiveSession -> SessionManager messages

Meant to address #2695. There is already unit test coverage for the branches I touched; please let me know if there are any other unit tests that would be appropriate I didn't notice.
Note: I'm not the assignee for the issue, but it's been stale long enough that I went ahead.

I'm realizing now I maybe should have opened the PR directly on the repo instead of through my fork, because it didn't present me with the template flow. Sorry! Let me know if I should re-open.

cool! this looks good.
only smol nits,
sorry for the delay

Fixed!


Basic UI

Project milestone: Add Basic UI
In  this PR the following was implemented:
Create a app.rb file that will serve as your console app entry-point. It should have methods that do the following:

List all books.
List all people.
Create a person (teacher or student, not a plain Person).
Create a book.
Create a rental.
List all rentals for a given person id.

In your main.rb define the entry point, this will be a method called main that is invoked at the end of your file. This method should do the following:

Present the user with a list of options to perform.
Lets users choose an option.
If needed, ask for parameters for the option.
Have a way to quit the app.

Hy @Bondok6
Thank you 👍


GitHub Workflows security hardening

This PR adds explicit permissions section to workflows. This is a security best practice because by default workflows run with extended set of permissions (except from on: pull_request from external forks). By specifying any permission explicitly all others are set to none. By using the principle of least privilege the damage a compromised workflow can do (because of an injection or compromised third party tool or action) is restricted.
It is recommended to have most strict permissions on the top level and grant write permissions on job level case by case.

https://github.com/chalk/chalk/pull/581?notification_referrer_id=NT_kwDOAAKZHrE1MDM2ODU3Mjk2OjE3MDI3MA#issuecomment-1343138206


README is not updated to reflect that this is a native implementation now

I saw a commit message that says that the cgo implementation was removed.
Should we update the README as well?

@thedrow PR are welcome!


tsiclient.js:17 Error:  attribute height: A negative value is not valid. ("-11")

Upon graph generation the following console error is present:
tsiclient.js:17 Error:  attribute height: A negative value is not valid. ("-11")

this is likely due to the fact that you have placed the chart in a container that has no height...try creating a fixed height/width container.  if you have a stable repro feel free to send it as a codepen.  as a rule, yes the sdk should not throw, but i think the error is reasonable (albeit somewhat cryptic) if the container does in fact have no height

I'm gonna go ahead and close this.  Here is a codepen with a little example of how you can embed the chart in a flexy container...complete with a working hamburger button on the left nav!
https://codepen.io/darsney/pen/OKZRgP


Added Walkable Areas list in Room tab.

TODO:

Add a "Add walking area" button
Add a popup to name new walking areas
Link WA to its node, to achieve click-to-edit
Make remove menu item work

@mapedorr changed ICharacter methods names. They are long but predictable, it's better IMO


[BUG] AsyncCurrentValueStream element is not public

Unable to use property element. It isn't public
To Reproduce
Steps to reproduce the behavior:

use AsyncStreams.CurrentValue
attempt to print(currentValue.element)
See build error

let currentValue = AsyncStreams.CurrentValue<Int>(1)

Task {
    for try await element in currentValue {
        print(element)
    }
}

print(currentValue.element)  // error will occur

Expected behavior
Expect that element is accessible
Environment:

0.1.0

resolved by https://github.com/AsyncCommunity/AsyncExtensions/pull/5.
Thanks.


Add mechanism to verify that source code in documentation is valid

Is your feature request related to a problem or challenge?
Our user guide and library guide contain code samples, but these can quickly become outdated and stop compiling and we have to automated way to check if they still compile.
Describe the solution you'd like
I propose that code samples be written as unit tests and that we then use a script to insert these code samples into the documentation when we publish the docs.
Describe alternatives you've considered
No response
Additional context
No response

I believe this is now handled via the doctest macro, for example:
https://github.com/apache/datafusion/blob/37e54ee874e9027a329ba4f6c1e0e6359d63a33c/datafusion/core/src/lib.rs#L618-L643


Improve article 2109.02101: Version number gets ignored

The URL https://ar5iv.labs.arxiv.org/html/2109.02101v3 should lead to version 3 of the article, right? Apparently it leads to v1 or v2.
Likewise, I think https://ar5iv.labs.arxiv.org/html/2109.02101 should also default to the newest version.

I'm not sure this is a good state to keep in. Why default to v1 instead of the latest?

Why default to v1 instead of the latest?

It's not even v1 I'm afraid. The current ar5iv is seeded by the bulk sources from arXiv, and then updated at irregular intervals to the latest version of an article. The last update was in August 2023, however the last full rebuild of ar5iv was last December. We intend to do another full rebuild by the end of the year, and also update alongside.
But because the update is a custom step I have to execute, I can't guarantee that all articles will have the latest version displayed.

I'm not sure this is a good state to keep in.

I agree. ar5iv is meant to be sunset as soon as the native arXiv HTML stabilizes, and will start redirecting to the official pages, where versioned links will resolve correctly. The current ar5iv is very much a temporary state. But all github issues that touch on conversion flaws are very much appreciated - and will not get lost.

Aaah, that's better than v1; at least it doesn't regurgitate the really old stuff this way. Understand; I thought it would pull updates from the arXiv.


Fix createServicesFetcher handle null service url

We found that when a service doesn't set an explicit service URL, which seems to be valid state, the fetcher fails when it tries to create the hash since you can't give null into a Node.js crypto hash
screenshot of reference:

cc @saihaj @kamilkisiela

0.21.0


Is there any way to import specific number of locales once?

I have react-native project which uses Moment.
Now I have to import all locales in every class where I use moment like this:
import moment from 'moment';

import 'moment/locale/ru';
import 'moment/locale/es';
import 'moment/locale/uk';
import 'moment/locale/be';

As locales number could grow further - is there any way to import all necessary locales in one module/file like this:
//allmoments.js
export moment from 'moment';

import 'moment/locale/ru';
import 'moment/locale/es';
import 'moment/locale/uk';
import 'moment/locale/be';

export * ??

and then import it in every class/component where I need it like:
import * from 'allmoments.js'
?

For those wondering (as I was) - I've found working solution:
//moments.js
module.exports = {
    ...require('moment/locale/ru'),
    ...require('moment/locale/es'),
    ...require('moment/locale/uk'),
};

//inanymodule.js
require('./moments');


Initial kudos github integration

[x] Give kudos in issues
[ ] Give kudos in PRs

PR Comment
PR Review Comment (is this different?)

[ ] Give kudos from discussions

Repo discussions (org discussions are just repo discussions)

[ ] Give kudos from team posts

/kudos-dev Let's see this from a PR @philip-gai


Devices dashboard

Where can i find the Devices dashboard? thk

Hi @471448446 you have to install the plugin, restart your IDE and open the lateral toolbar. There you'll find the Android WiFi ADB tab.
If you don't see that toolbar, please press the button you'll find at the right bottom corner of your screen. This button shows and hides the toolbar.


pick(), count argument

you could implement a p() argument with a count:
p(3, test:0:*, uq)

I could simply generate them with the same position index.

Works.


[Threat Modelling] Verify that our produced container images are signed and only published to trusted container registries

Description
To avoid that malicious code can be injected into our productive systems by injecting untrusted container images, we have to verify our build and delivery process and ensure that

all container images are build following SAP SLC-29 compliancy guidelines (using trusted build infrastructure, sign images)
the publishing of container images happens only via trusted container registries
consumption/deployments of container images are using only trusted container registries

AC:

[x] Ensure that our build pipeline is using an SLC-29 compliant build-mechanism to product container images (image is also signed by the SAP Signify service)
[x] Verify that any produced container image is only published and consumed from a trusted container registry (review of deployment manifests)

Steps to exploit
Attacker find a way to inject malicious code into our productive systems because container images were build on untrusted systems, not signed and published to untrusted container registries.
Risk assessment
Part of the Threat Modelling workshop from 2023-11-29.
Proposed mitigation
Review the build process of our container images and ensure they are build by using SLC-29 compliant build-pipelines and getting signed by SAP Signify. Verify also the deployment manifests and ensure that only images from trusted container registries will be deployed.

Mural Board: https://app.mural.co/t/sap10/m/sap10/1701162989605/93d8a5cfd1937c1ed568d324926d51d6d4c96669?sender=u03d165e43fbc397bd7057627

All productive images are produced by using the Kyma image build pipeline offered by Neighbors team.
Each produced image is following an SLC-29 compliant build process, are signed by Signify and finally pushed to an approve container registry in Kyma which hosts all Kyma released images.


[Feature Request]: Add RoutingIntent child module to Virtual Hub module.

Description
Hi,
Could we have a routing Intent functionality for virtual Hub which would allow sending traffic to Azure Firewall:
https://learn.microsoft.com/en-us/azure/templates/microsoft.network/virtualhubs/routingintent?pivots=deployment-language-bicep
resource symbolicname 'Microsoft.Network/virtualHubs/routingIntent@2023-02-01' = {
  name: 'string'
  parent: resourceSymbolicName
  properties: {
    routingPolicies: [
      {
        destinations: [
          'string'
        ]
        name: 'string'
        nextHop: 'string'
      }
    ]
  }
}

Hey @annduda,
we sure can. I'll move the issue into the 'todo' state. Feel free to implement and contribute it yourself if you have the capacity. If not, we'll see to have it implemented by a member of the community :)

Hey @AlexanderSehr if nobody took it yet I can try :)

If you have the capacity, please do - and if you need support, please let me know :)


#333 - Only expose highest ranking Computed Property per name (Comput…

…edProperty::getName())
Pending unit tests. Verifying approach is optimal.
@justinedelson thoughts on if this is the most optimal approach? Felt like I was writing too much code but couldn't figure out a cleaner way.
Basically, we want to expose the highest ranking (service.ranking) Computed Property per ComputedProperty::getName().
So given the following deployed ComputedProperty's...
Foo.getName() => myProp
Foo's service.ranking = 100

Bar.getName() => myProp
Bar's service.ranking = 200

Zip.getName() => anotherProp
Zip's service.ranking = 500

Then the available, exposed list of ComputedProperties should be: Bar, Zip
Foo would be excluded from use since Bar's service.ranking is greater and shares the same name.
This list is required in a Sling Model and an OSGi Service.
/cc @brandonmaynard

I guess the main thing to review is to ensure the thread safety is optimal in ComputedPropertiesImpl.
This PR has the UnitTesting deps in it, so it would be good to review/merge so other branches dont have conflicts as they introduce Unit tests.

@joerghoh if you have a few mins and care to review the thread safety here would be appreciated (not sure if there is a more efficient approach than the lock and/or if im missing a safety condition..).

@godanny86 if you have time can you review this out, specifically the thread safety concerns in the ComputedPropertiesImpl?

Thanks @godanny86


container_cpu_usage_costs_hourly 等指标不存在

container_cpu_usage_costs_hourly container_memory_usage_costs_hourly 等指标不存在，是我缺少了什么吗？

crane-daemonset craned fadvisor 的 servicemonitor 已部署

You should wait a while for xxx_costs_hourly metrics.

You should wait a while for xxx_costs_hourly metrics.

It's been seven hours since restarting, I'll check it out tomorrow.

The metrics are from here:https://github.com/gocrane/helm-charts/blob/main/integration/prometheus/override_values.yaml#L13
If not exist, please also checking the recording rules in prometheus.

Thanks, I am using the existing promethues, which is missing this part, and it is fine now.


add torch.empty

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #331.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
thunder == fun # True

Hi @mruberry! I just updated my code based on your comments. Would appreciate it if you can take a look. Thank you for your time!


POC of an SSR Loading Skeleton for Charts

styling could be improved slightly

Added the chart height. A bit jumpy, but pretty close!

Made some adjustments to get a bit closer to the actual height and margins. Not sure why they aren't identical, but seems close enough for now

@hughess I think it's because of the download bar spilling out of the chart container slightly?

@archiewood I thought I had accounted for that, but the pixels didn't seem to be adding up to what I expected. In any case, your update looks very close!


Ubuntu 24.04 LTS released!

Breaking changes
Please add support for the latest Ubuntu 24.04 LTS to runner-images
Target date
25th of April 2025.
The motivation for the changes
24.04 has a newer kernel and packages than 22.04.
Possible impact
Eventually developers will want newer compilers etc than are available in 22.04.
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions

Runner images affected

[X] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Mitigation ways
Make ubuntu-latest point to a VM/container with 24.04 LTS.

Already available for download here: https://ubuntu.com/download


Fix flaky OTLP client cert integration tests

Fixes https://github.com/dotnet/aspire/issues/3380
Microsoft Reviewers: Open in CodeFlow

/backport to release/8.0


LLamaSharp.Examples: Document Q&A with local storage

This PR adds an example that demonstrates how to use Microsoft.KernelMemory with local storage so documents can be analyzed once, saved to disk (as a plain-text vector database), then rapidly reloaded on future runs of the program.
Note that Microsoft.KernelMemory (https://microsoft.github.io/kernel-memory/) is a higher-level more feature-rich wrapper for Microsoft.SemanticKernel.Memory that was "built on the feedback received and lessons learned from developing Semantic Kernel", so it should probably be the favored strategy for document ingestion and interaction in the future.

Excellent work, thanks again!


Missing environment variables

I found a bunch more environment variables that would be useful to have set in the ssh session:
CUDA_VERSION': '10.1.243'
'CUDA_PKG_VERSION': '10-1=10.1.243-1'
'CUDNN_VERSION': '<IP_ADDRESS>'
'TPU_NAME': 'grpc://<IP_ADDRESS>:8470',
'TBE_CREDS_ADDR': '<IP_ADDRESS>:8008',
'XLA_FLAGS': ' --xla_cpu_enable_fast_math=false'})
'XRT_TPU_CONFIG': 'tpu_worker;0;<IP_ADDRESS>:8470

These are set in the web and are required for using TPUs.

Thank you for your input.
You may now find additional documentation that covers how to expose environment variables.

Thanks for fixing the bug. But I wonder if the old api where the vars were set automatically wasn't more convenient?

@bjourne I'm going to need your help to identify all the required environment variables when using TPU. Please post them here and I will add them like you mentioned.
For anyone asking to add environment variables by default
I added expose_env_variable function so anyone can add his own optional environment variables. In my opinion, colab-ssh shouldn't expose environment variables other than the needed ones (for CPU, GPU and TPU). colab-ssh should only have one responsibility which is take care of the ssh tunneling perfectly.
If you think the environment variable is required, please open a new issue or create a pull request.

Thank you. The ones that PyTorch and Keras uses for TPU access are: COLAB_GPU, COLAB_TPU_ADDR, TBE_CREDS_ADDR, TF_FORCE_GPU_ALLOW_GROWTH, TPU_NAME, XRT_TPU_CONFIG.


what is batch as file ?

I can't find any documentation for the meaning of this feature

@ahmed56734  In tiny, "batch" represents batch compression.


next6 with @zeit/next-less   Cannot destructure property createHash of 'undefined' or 'null'

Bug report
Describe the bug
I use  next6 with  @zeit/next-less will show the error message
TypeError: Cannot destructure property createHash of 'undefined' or 'null'.
problem demo  repository:
https://github.com/polo13999/next6withantproblem
Expected behavior
yarn dev   will show the error
System information

OS: Mac
chrome, safari
Version of Next.js: 6.1.2

Additional context
I try to use  @zeit/next-less 0.2 it still has problem

You'll have to use an older version of next-less or write your own if you're on Next.js 6.1.2. I'd highly recommend upgrading.


LazyUpdate and ParJoin explanations to book

Note: I haven't actually built the book so can't say if I have done everything right.

Thanks for the suggestions!
I was kinda suprised how many problems the text had.
My non-native english is showing :P

@WaDelma don't worry about it. I'm German and I'm very sure @torkleyy is, too, so we are no native speakers either. It's just that it's easier to spot someone else's errors than one own's. That's why people do code reviews. ;)
Hmm. thinking about it, there are many places I am not sure about. It's mostly comma related, however my English teacher at work, who is native Irish, says herself that she doesn't know the exact English comma rules. I guess, we should hope that we can find a native speaker who can correct the text and make a high-quality version. However, for now we did what we can, and it's good imho.

@minecrawler @phowell is native speaker. They once fixed grammar mistakes in the book.

Guess we can merge this now ;)
bors r+


Increment versions for quota releases

Increment package versions for quota releases

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


New Add es/no-function-bind rule

This rule reports ES5 Function.prototype.bind() call as errors (to the extent possible).

I apologize for no response while a long time.
I have hesitated to go with the rules which try to catch prototype methods because such rules have essentially many false negatives since ESLint is a linter which checks code for each file individually.
This code is very good, but I'm really sorry, I would not like to merge rules which check prototype methods.

Thank you for checking & comment.
I closed this PR & #4.


deCONZ v2.05.73

New deCONZ has been release two days ago: https://github.com/dresden-elektronik/deconz-rest-plugin/releases/tag/V2_05_73
Any chance to update the addon?
Thanks in advance!

Thanks! Somehow, GitHub did not send me a notification of this.
Will look into it asap.


✨ Implement optional number to str coercion

Change Summary
Implement optional number to str coercion
Related issue number
Fix #6045
Requires https://github.com/pydantic/pydantic-core/pull/975
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[ ] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Loss is significantly worse when calculated manually in train_step compared to passed into model.compile

#Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.8.2
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 18.04.5
Python version
Python 3.7.13
Current Behavior
I am following the steps here for customizing what happens in train_step when you customize a model:
https://www.tensorflow.org/guide/keras/customizing_what_happens_in_fit
The issue is, I am getting very different results for the loss when I am using the exact same tensorflow.keras.losses.mean_squared_error function in different locations.
When I use mean_squared_error "manually" during train_step (I am calling this the non-compiled mse), the loss is significantly worse than if I  pass it into the compile method, e.g. model.compile(optimizer="adam", loss=mean_squared_error) (I am calling this the compiled mse).
tensorflow.keras.metrics.MeanSquaredError yields roughly the same result as the compiled version of mse, but it is still not exactly the same.
Expected Behavior
My expectation is that the loss should be the same whether it is passed into .compile, defined during train_step, or used as a metric. But it is significantly worse if I do not pass it into .compile and I don't know why. I am following the documentation, and as far as I can tell am not doing anything incorrectly.
Why is the loss so different when it is and is not passed into .compile?
Colab Notebook
I have reproduced this behavior in this google colab notebook, link below. The plot shows that non-compiled mse is systematically worse than compiled mse.
https://colab.research.google.com/drive/1c1L8KJYQbAldphKvFvmw5cHi9EyBQP0X?usp=sharing
Originally I opened a ticket at tensorflow/tensorflow here, and they were able to reproduce the issue. This is the gist:
https://colab.research.google.com/gist/sushreebarsa/f1144243be9eed060c39c60611811811/56866.ipynb
Standalone code to reproduce behavior
This is the same as what is in the Colab notebooks.
import tensorflow as tf
from tensorflow.keras.metrics import Mean, MeanSquaredError
from tensorflow.keras.models import Model
from tensorflow.keras.layers import Dense, Dropout, Input
from tensorflow.keras.optimizers import Adam
from tensorflow.keras.losses import mean_squared_error
import matplotlib.pyplot as plt

seed = 113
epochs = 30
batch_size = 10

n = 5
history_dict = {}

# load dataset
(x_train, y_train), (x_test, y_test) = tf.keras.datasets.boston_housing.load_data(
    path="boston_housing.npz", test_split=0.2, seed=seed,
)
n_validate = int(0.1 * len(x_train))
train_dataset = tf.data.Dataset.from_tensor_slices(
    (x_train[:-n_validate], y_train[:-n_validate])
).batch(batch_size)
valid_dataset = tf.data.Dataset.from_tensor_slices(
    (x_train[-n_validate:], y_train[-n_validate:])
).batch(batch_size)
test_dataset = tf.data.Dataset.from_tensor_slices(
    (x_test, y_test)
).batch(batch_size)

for i in range(n):

    tf.keras.backend.clear_session()

    mse_tracker = Mean(name="non_compiled_mse")
    mse_tracker_metric = MeanSquaredError(name="mse_metric")

    class CustomModel(Model):

        @tf.function
        def train_step(self, data):
            x, y_true = data
            with tf.GradientTape() as tape:
                y_pred = self(x, training=True)
                compiled_mse = self.compiled_loss(y_true, y_pred)
                non_compiled_mse = mean_squared_error(y_true, y_pred)
                mse_tracker.update_state(non_compiled_mse)
                mse_tracker_metric.update_state(y_true, y_pred)
            gradients = tape.gradient(compiled_mse, self.trainable_variables)
            self.optimizer.apply_gradients(zip(gradients, self.trainable_variables))
            return {m.name: m.result() for m in self.metrics}

        @tf.function()
        def test_step(self, data):
            x, y_true = data
            y_pred = self(x, training=False)
            compiled_mse = self.compiled_loss(y_true, y_pred)
            non_compiled_mse = mean_squared_error(y_true, y_pred)
            mse_tracker.update_state(non_compiled_mse)
            mse_tracker_metric.update_state(y_true, y_pred)
            return {m.name: m.result() for m in self.metrics}

        @property
        def metrics(self):
            return super().metrics + [mse_tracker, mse_tracker_metric]

    input_layer = Input(shape=x_train.shape[1:])
    x = Dense(8, activation='linear')(input_layer)
    x = Dense(8, activation='linear')(x)
    output_layer = Dense(units=1, activation='linear')(x)
    model = CustomModel(input_layer, output_layer)
    model.compile(optimizer=Adam(learning_rate=1e-3), loss=mean_squared_error)
    history = model.fit(
        train_dataset,
        validation_data=valid_dataset,
        epochs=epochs,
        verbose=0,
    )
    history_dict[f"compiled_mse_{i}"] = history.history["loss"]
    history_dict[f"non_compiled_mse_{i}"] = history.history["non_compiled_mse"]
    history_dict[f"mse_metric_{i}"] = history.history["mse_metric"]

plot_start_epoch = 5
plt.figure(figsize=(12, 6))
for i in range(n):
    plt.plot(
        range(epochs)[plot_start_epoch:],
        history_dict[f"compiled_mse_{i}"][plot_start_epoch:],
        label=f"compiled_mse" if i == 0 else None,
        color="black",
    )
    plt.plot(
        range(epochs)[plot_start_epoch:],
        history_dict[f"non_compiled_mse_{i}"][plot_start_epoch:],
        label=f"non_compiled_mse" if i == 0 else None,
        color="orange",
    )
    plt.plot(
        range(epochs)[plot_start_epoch:],
        history_dict[f"mse_metric_{i}"][plot_start_epoch:],
        label=f"mse_metric" if i == 0 else None,
        color="red",
        linestyle="dashed",
    )
plt.ylabel("loss")
plt.xlabel("epoch")
plt.title("compiled vs non-compiled mse vs mse metric")
plt.legend()
plt.show()

Related Issues
I could not find an issue that was the same as this one, but 16834 might be related, as it also notes a difference between compiled mean squared error and manually calculated mean squared error:

When using keras for regression problems, loss uses mean squared error. The loss displayed by the network fit in the last epoch during training is inconsistent with the loss calculated from the input data after the model is saved.

I have not tested whether or not this behavior is reproduced when using a loss other than mean squared error.

@gadagashwini I was able to replicate this issue, please find the gist here .Thank you!

I believe the snippet below is a simpler way to reproduce this error.
import numpy as np
from keras import Sequential
from keras.layers import Dense
from keras.metrics import Mean, MeanSquaredError, mean_squared_error

x=np.concatenate((
    np.zeros((40, 1)),
    np.ones((40, 1))
))
y_true = np.random.rand(80) * 100

model = Sequential([
  Dense(1, input_dim=1)
])
y_pred = model(x, training=True)

mean = Mean(name="mean")
mse = MeanSquaredError(name="msem")

mean.update_state(mean_squared_error(y_true, y_pred))
mse.update_state(y_true, y_pred)

print('name total count result ',
  *(f'{metric.name} {metric.total.numpy():11.4f} {metric.count.numpy():2.0f} {metric.result().numpy():10.4f}'
    for metric in (mean, mse)),
      sep='\n')

np.testing.assert_allclose(
  mean.result().numpy(),
  mse.result().numpy()
)

name total count result
mean<PHONE_NUMBER> 80  2854.2676
msem   2857.3472  1  2857.3472

---------------------------------------------------------------------------
... # stack

AssertionError:
Not equal to tolerance rtol=1e-07, atol=0

Mismatched elements: 1 / 1 (100%)
Max absolute difference: 3.0795898
Max relative difference: 0.00107778
 x: array(2854.2676, dtype=float32)
 y: array(2857.3472, dtype=float32)

TDLR
You must pass y_true as a 2-D tensor [batch, target_vars], because mean_squared_error will average the signal over the last axis. For the 1-D case, this means reducing all samples in the batch into a single value, which confuses the MeanMetricWrapper into thinking all batches only have 1 sample.
Adding this line fixes the differences between the losses and yields the correct history chart (lines are coincident):
y_train, y_test = y_train.reshape(-1, 1), y_test.reshape(-1, 1)

Chart: https://user-images.githubusercontent.com/1595030/181395395-01c972f1-529c-4595-bdcb-f1bdad4f05d0.png

Why this is happening

When using Mean and mean_absolute_error:
Although your labels have an incorrect rank, the output of the network is correct ([batch, 1]). Calling mean_squared_error with both values results in the broadcasting that fixes the issue:shape(y_true) == [batch]
shape(y_pred) == [batch, 1]

losses = mean_squared_error(y_true, y_pred)  # [batch]
# mse =>
#   diff = y_true - y_pred  # [batch] - [batch, 1] -- broadcast --> [batch, 1]
#   return backend.mean(diff ** 2, axis=-1)  # [batch]

When using MeanSquaredError:
The MeanSquaredError#update_state method (implemented in base_metric.MeanMetricWrapper) is invoking losses_utils.squeeze_or_expand_dimensions, which reduces shape(y_pred) == [batch, 1] to conform with shape(y_true) == [batch]. The function mean_squared_error then reduces the last dim, resulting in a single value instead of batch that should be reduced inside Mean.
When using compiled metric:
LossesContainer invokes losses_utils.squeeze_or_expand_dimensions, resulting in the same value as if you were using MeanSquaredError.

Turns out this is a duplicate of #14627. So many minutes wasted debugging it... :disappointed:

I see, so just to make sure I totally understand what you are saying:
y_pred.shape == [batch, 1] after y_pred = self(x). y_true.shape == [batch] before being passed to mean_squared_error, but is broadcast to be shape [batch, 1] during mean_squared_error.
In both the compiled and non-compiled cases, losses.shape == [batch] after losses = mean_squared_error(y_true, y_pred). The difference is that the compiled function then broadcasts it to losses.shape == [batch, 1] before passing it to Mean, but in the manual case it was still losses.shape == [batch] when it was passed to Mean.
If y_train, y_test (and y_pred) were 2D ahead of time all of this is avoided.
Is all this right?
I think this issue arose because the shape of the data in the documentation is not revealed, and it assumes your data is in the correct shape. Which is probably not an issue of the documentation, but is an assumption it makes to where you can't just pass it any dataset and expect it to work.
Also, I'm sorry this issue is a duplicate, I might not have found the other one because I was focused on .compile.

y_pred.shape == [batch, 1] after y_pred = self(x). y_true.shape == [batch] before being passed [...]

This is correct.
{y_pred [batch, 1], y_true [batch]} -- mean_squared_error(y_true - y_pred) --> loss [batch]   <=>
{y_pred [batch, 1], y_true [batch]} -- mean((y_true - y_pred)**2, axis=-1) --> loss [batch]

In both the compiled and non-compiled cases, losses.shape == [batch] after losses = mean_squared_error(y_true, y_pred).

No. loss.shape == [batch] for the non-compiled (the sequence above).
loss.shape == [1] for both compiled and mse_tracker_metric <MeanSquaredError>, because both invoke losses_utils.squeeze_or_expand_dimensions before/during MeanMetricWrapper#update_state.
The only correct value here is the non-compiled one; and this bug is actually pretty funny. I don't think it would be easy to spot without the custom train_step.

I think this issue arose because the [...]

I agree. Maybe something could be done to improve on this. Either by documenting better or creating a version of squared error that didn't reduce any axis.
And I apologize. I didn't mean to complain that you hadn't looked for it, but rather that I forgot to check before start debugging.

Oh no problem, I took it as you intended. Thank you for clarifying all this.
I've never closed an issue before, but I'm feeling like maybe this should be closed. The easiest step would be just to include what the shapes of the data are (or the data itself) in the documentation. It's a very specific problem, but my experience is that MSE alongside a single target variable is not uncommon for regression models. On the other hand, I don't know how I'd even go about changing it even if I had the ability to do so.

Hi @Yacalis, Can move this issue to closed status, if it resolved for you. Thank you!