TF1.5 configure overwrites export TF_CUDA_VERSION=9.1

It seems configure overwrites the BASH flag:
export TF_CUDA_VERSION="9.1"

Here is the 1.5 branch of my tensorflow.sh install script.
Here are the key code lines from the tensorflow.sh specific hash to reproduce this error install script,  and 161 at the time of writing:
    # line 134 at the time of writing
    export TF_NEED_CUDA=1
    export TF_CUDA_VERSION="9.1"
    export TF_CUDNN_VERSION=7
    export CUDA_TOOLKIT_PATH=/usr/local/cuda-9.1/targets/x86_64-linux/lib/
    # ...snip... to line 161 at the time of writing
    yes "" | ./configure

I believe the problem is either in the configure script, configure.py set_tf_cuda_version() ask_cuda_version or in the call to set_tf_cuda_version, none of which seem to check the environment variable first and possibly overwrite it.
The result is the following error:
Invalid path to CUDA 9.0 toolkit. /usr/local/cuda/lib64/libcudart.so.9.0 cannot be found
Please specify the CUDA SDK version you want to use, e.g. 7.0. [Leave empty to default to CUDA 9.0]:

Please specify the location where CUDA 9.0 toolkit is installed. Refer to README.md for more details. [Default is /usr/local/cuda]:

Invalid path to CUDA 9.0 toolkit. /usr/local/cuda/lib64/libcudart.so.9.0 cannot be found
Traceback (most recent call last):
  File "configure.py", line 1365, in <module>
    main()
  File "configure.py", line 1320, in main
    set_tf_cuda_version(environ_cp)
  File "configure.py", line 850, in set_tf_cuda_version
    _DEFAULT_PROMPT_ASK_ATTEMPTS)
__main__.UserInputError: Invalid TF_CUDA_SETTING setting was provided 10 times in a row. Assuming to be a scripting mistake.
-> [1]
ahundt@femur|~/src/robotics_setup on tf-1.5!?
± ls /usr/local/cuda/lib64/
stubs                  libcusparse.so.9.1     libnppist.so.9.1
libaccinj64.so         libcusparse.so.9.1.85  libnppist.so.9.1.85
libaccinj64.so.9.1     libcusparse_static.a   libnppist_static.a
libaccinj64.so.9.1.85  libnppc.so             libnppisu.so
libcublas_device.a     libnppc.so.9.1         libnppisu.so.9.1
libcublas.so           libnppc.so.9.1.85      libnppisu.so.9.1.85
libcublas.so.9.1       libnppc_static.a       libnppisu_static.a
libcublas.so.9.1.128   libnppial.so           libnppitc.so
libcublas.so.9.1.85    libnppial.so.9.1       libnppitc.so.9.1
libcublas_static.a     libnppial.so.9.1.85    libnppitc.so.9.1.85
libcudadevrt.a         libnppial_static.a     libnppitc_static.a
libcudart.so           libnppicc.so           libnpps.so
libcudart.so.9.1       libnppicc.so.9.1       libnpps.so.9.1
libcudart.so.9.1.85    libnppicc.so.9.1.85    libnpps.so.9.1.85
libcudart_static.a     libnppicc_static.a     libnpps_static.a
libcufft.so            libnppicom.so          libnvblas.so
libcufft.so.9.1        libnppicom.so.9.1      libnvblas.so.9.1
libcufft.so.9.1.85     libnppicom.so.9.1.85   libnvblas.so.9.1.128
libcufft_static.a      libnppicom_static.a    libnvblas.so.9.1.85
libcufftw.so           libnppidei.so          libnvgraph.so
libcufftw.so.9.1       libnppidei.so.9.1      libnvgraph.so.9.1
libcufftw.so.9.1.85    libnppidei.so.9.1.85   libnvgraph.so.9.1.85
libcufftw_static.a     libnppidei_static.a    libnvgraph_static.a
libcuinj64.so          libnppif.so            libnvrtc-builtins.so
libcuinj64.so.9.1      libnppif.so.9.1        libnvrtc-builtins.so.9.1
libcuinj64.so.9.1.85   libnppif.so.9.1.85     libnvrtc-builtins.so.9.1.85
libculibos.a           libnppif_static.a      libnvrtc.so
libcurand.so           libnppig.so            libnvrtc.so.9.1
libcurand.so.9.1       libnppig.so.9.1        libnvrtc.so.9.1.85
libcurand.so.9.1.85    libnppig.so.9.1.85     libnvToolsExt.so
libcurand_static.a     libnppig_static.a      libnvToolsExt.so.1
libcusolver.so         libnppim.so            libnvToolsExt.so.1.0.0
libcusolver.so.9.1     libnppim.so.9.1        libOpenCL.so
libcusolver.so.9.1.85  libnppim.so.9.1.85     libOpenCL.so.1
libcusolver_static.a   libnppim_static.a      libOpenCL.so.1.0
libcusparse.so         libnppist.so           libOpenCL.so.1.0.0

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

oops, missed this.
TF_CUDA_VERSION is the correct env variable to be setting. I'm not able to reproduce this error. The configure script appears to look in the correct path for CUDA. Note, that IF configure.py script does not find CUDA, then it will set the TF_CUDA_VERSION env variable to empty string (and then reask user again where it should look for CUDA).

were you able to try with 9.1 on tf 1.5? Perhaps something was updated already but before it would look for 9.0 anyway.

Nagging Assignees @case540, @gunan: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 94 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 109 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 124 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 139 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 154 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 169 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 184 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 199 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 214 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 229 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hmm, taking another look at this, it seems like if you set either of CUDA_TOOLKIT_PATH  or TF_CUDA_VERSION  incorrectly, then both of those ENV variables will be reset and ununsed.
It could be the case that CUDA_TOOLKIT_PATH was incorrectly set, then both those env variables were set to the defaults, and some default cuda path with default cuda 9.0 version was used.
If this is the case, then it's more of an issue of a very confusing error. I dont think the CUDA_VERSION variable should ever be reset by the configure script, since if a user sets a specific version we should use that version probably

On the other hand, since...
yes "" | ./configure
... is run by the above script, it kinda just skips through all of the prompts. I think a potential solution to this is to make a much friendly way for scripts to run configure.py. Such just add actual python args to the script instead of just setting env variables and then skipping over user prompts lol

Nagging Assignees @case540, @gunan: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @case540, @gunan: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I think a potential solution to this is to make a much [friendlier] way for scripts to run configure.py

That would be an excellent solution, I agree!

Closing this issue due to staleness. Please check with the latest version of TensorFlow. Feel free to reopen if issue still persists. Thanks!


Not an issue, at all :)

Thanks @ulrichard for starting this project. I started (at least tried out some QML a few weeks ago with playing with this idea too - but you've come much further in a short time) Let me know if i can help out in any way! Love this initiative!

Hey @aiobofh good to hear that this is useful and is being used.
Unfortunately I don't have that much time to make more than the bare minimum.
I will open some issues in the repo. So there are some ideas, if you want to help improve the app. But also testing and reporting bugs is most welcome.


add an emit "scroll" event for USelectMenu

For what version of Nuxt UI are you suggesting this?
v2.x
Description
Hi I would like to have within the component UselectMenu emit the event "scroll"
with the following properties:
{
  target: HTMLElement,
  currentTarget: HTMLElement,
  scrollTop: number,
  scrollHeight: number,
  clientHeight: number,
}

to dynamically remote search items through an api and append it dynamically to my USelectMenu to create an infinite scroll.
This feature would be great in both version :)
Additional context
No response

also the same issue 1167


[suggestion] KeepInv=false options

I thought that this was covered by the mod, but realized it wasn't! :( It would be really cool to have extra options for when KeepInv is FALSE! I want to drop my inventory... but keep my armor, exp, hotbar, and curios slots. I'd like to have to go back for my gathered resources, but it's no fun trying to do that without any armor or tools.

I just realized that may be way easier to do with the current rules as KeepInv=true, using KeepArmor and KeepExp, but add a new KeepHotbar amd KeepInvMainSlots?


Bluetooth: host: gatt: respect BT_GATT_ENFORCE_SUBSCRIPTION

A subscription check was still being done in the
gatt_notify_multiple_verify_params() fn, regardless of the value of CONFIG_BT_GATT_ENFORCE_SUBSCRIPTION.
This could lead to getting this warning if BT_GATT_NOTIFY_MULTIPLE is enabled, and the notifications not being sent to unsubscribed peers.

@jori-nordic just FYI, the CI error is a problem after a recent merge; it is being looked at, and is not related to your PR


fix transient test failures by increasing timeout for gbm shap method

Description
Fix transient test failures by increasing timeout for gbm shap method.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2044 (45c6825) into main (0af6cef) will decrease coverage by 0.56%.
The diff coverage is n/a.

:exclamation: Current head 45c6825 differs from pull request most recent head 4a9dd3a. Consider uploading reports for the commit 4a9dd3a to get more accurate results

@@            Coverage Diff             @@
##             main    #2044      +/-   ##
==========================================
- Coverage   93.63%   93.08%   -0.56%
==========================================
  Files          31       51      +20
  Lines         581     2009    +1428
==========================================
+ Hits          544     1870    +1326
- Misses         37      139     +102

Flag
Coverage Δ

unittests
93.08% <ø> (-0.56%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 20 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Make PollingBased waiter more flexible

Networks with short block time can't use 1s as a default poll interval, it's too harsh for them. Ref. https://github.com/nspcc-dev/neofs-node/issues/2864.

OK, tests are failing and we need to adjust Actor correspondingly.

@roman-khimov, regarding Actor integration I consider two options:

One more Actor constructor that accepts PollConfig, but we have a lot of othre Actor constructor and this config is not related to the Actor directly.
(*Actor).WithPollConfig(cfg PollConfig) modifier which seems a bit more preferable to me, because it's a bit easier to use. At the same time it's different from Actor's design.

Which one do you prefer? Or may be there's another option.

How about extending actor.Options with waiter.PollConfig?

I thought about it, but it's not really Acttor's configuration. Will add if you're OK with it.

I'm also thinking of waiter.Config in case we want to extend it in future.

I'm also thinking of waiter.Config in case we want to extend it in future.

I also thought about it, but WSBased do not uses any configurable options now, thus I created just a PollConfig. Let's extend it while we're here, it'll be useful.

Linter job is failing, but it's just because golangci-lint version was updated, will be fixed in a separate PR.

@roman-khimov, this one is ready for review. It's not needed for NeoFS, but still it would be nice to have these values customizable.

NeoFS is likely to still use it in some way.


Normalize sorting of license properties

Properties are now consistently sorted across all license files,
appearing in the following order:
layout
title
[nickname]
category
[tab-slug]
permalink
[redirect_from]
[hide-from-license-list]
[class]
source
[featured]
description
how
[filename]
[note]
[using]
required
permitted
forbidden

@waldyrious any chance you could add a script to do this automatically? And/Or a test to ensure it stays consistent going forward?

@benbalter I don't think I have the know-how to do that, but I suppose I can give it a try. Can you give me an example (in this project or elsewhere) of a script or test I could look at to have a starting point?
Then again, I don't think updates to the licenses are expected to be frequent enough to warrant such automation. Am I overlooking something by thinking so?

@benbalter if I rebase this would you consider merging it? it may not be as optimal as an automated script, but it is an improvement nonetheless, and I have other changes I'd like to submit that depend on this one.

@waldyrious as long as tests pass, would be glad to. Thanks for taking the time to contribute.

@benbalter done; please check if the rebase worked properly. I also updated the PR description above to match the amended commit message.

Awesome, awesome stuff. Thanks @waldyrious! :game_die: :dancer: :cactus:

Thanks, @benbalter. Having studied all these licenses and their properties for the purposes of this PR (including choosing a sensible sorting order), I am now proposing  series of changes that IMO would make the metadata cleaner / more robust. See #287, #288, #289, #290 and #291.


Skipped logs when format is specified

Hi
I tried to process my logs with version 11.0, but it does not combine the pgbouncer and postgresql logs in the same report when log format is specified:
pgbadger --average 1 --sample 15 --top 50 --start-monday -I -J 4 -O $outputpath
pgbouncer.log -f stderr postgresql.log
With this command the pgbouncer section is missing. If I put the pgbouncer.log as last in the command, then the postgresql data is missing. It seems only the last logfile (or logfile type) is processed.
I tried to process only the pgbouncer first, checked the report, then process the postgresql log and the pgbouncer data was eliminated from the report, remain just the postgresql data.
When I doesn't specify the log format works well, but sometimes I have multiple logfiles in the same day (postgres service restarts, etc) and some logfile format wasn't auto-detected and I get errors and the process just stops.
This behavior does not apply to versions before.
Thanks
Levi

Hi
This is because the -f force the use of the stderr format for all log to parse which don't fit with pgbouncer log entries. Format for pgbouncer log is unsurprisingly named pgbouncer.
Since version 11 you can set the format to use per file by suffixing the log filename by :format. For your command it will be:
pgbadger --average 1 --sample 15 --top 50 --start-monday -I -J 4 -O $outputpath \
                pgbouncer.log:pgbouncer postgresql.log:stderr

Of course if -f stderr is not provided pgbadger will auto-detect the log format but as you said sometime it could throw an error about unknown log format. This is the case when pgbadger can not count 10 lines with the same auto-detected format in the first 5000 lines. So with very small file this can happen. In this case use the per log file format syntax, this is the same for remote log file:
pgbadger ..... ssh:///var/log/postgresql/postgresql.log:syslog

Regards,

OK, I see.
This works only with specific file and perhaps could be implemented in the future like postgres*.log:stderr or pgbouncer*.log:pgbouncer
Thanks!
Levi

Commit f32ee6f fixes this issue. Now the format specified with the -f | --format value is applied only to log file where format can not be detected whatever is the file list.


utils: modify comparison operator

Use explicit per-field comparison instead of whole struct memcmp

@dmitry-gurulev not all cmp operators can be written through tuple trick. For example we have in our code base SPS comparison operator which checks existence of HRD params and then decide whether to compare them or not. So we need flexibility to control per-field comparison. I plan to heavily use those macro in such cases, to avoid copy-paste mistakes


MultiFromOutputStream blocking close refactor

Fixes https://gitlab-odx.oracledx.com/helidon/helidon/-/issues/18
Signed-off-by: Daniel Kec<EMAIL_ADDRESS>

@tomas-langer I have cherrypicked https://gitlab-odx.oracledx.com/helidon/helidon/-/issues/18 fix to #1991 so it can be targeted for 2.0, this PR can wait post 2.0 then


 My current use case is to import from CDN that resolves as module name, not sure subpaths are included.

That might explain it. It didn't work when I tried so I reverted to import from local node modules. I will see if I can fix it though when I have time because it would be sweet to support it


[BEAM-8471] Flink native job submission for portable pipelines

I would like to add this new feature since it will allows to make a k8s example with that release. This has already been tested for couple weeks in our infrastructure and does not affect anything else, the risk of adding it to the release is very low.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Java PreCommit


Fixed panic in CDC on deletes

Codecov Report

Merging #385 (7ade39b) into main (87b023f) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #385      +/-   ##
==========================================
- Coverage   28.41%   28.40%   -0.02%
==========================================
  Files          68       68
  Lines        7207     7210       +3
==========================================
  Hits         2048     2048
- Misses       4934     4936       +2
- Partials      225      226       +1

Impacted Files
Coverage Δ

internal/data.go
35.89% <0.00%> (-1.44%)
:arrow_down:

server/services/v1/api.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Var pwd tests

I wouldn't normally introduce a mock for an extremely rare error case. But in this case a) the situation isn't as rare as most people think, and b) it's a trivial mock injection which helps ensure that no future change inadvertently changes the behavior of that error condition.


Google Drive password with '"' and '>' is not correctly escaped

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
When inputing password with " and > into Google Drive's Password and Confirm box, password is getting splitted and part of it is shown below the input
To Reproduce
Steps to reproduce the behavior:

Go to 'diag_backup.php'
Input Google Drive data
In Password and Confirm input paste "sometext>thisshouldnotbevisible
Click Setup/Test Google Drive

Expected behavior
Password is replaced with dots
Screenshots

Relevant log files
If applicable, information from log files supporting your claim.
Additional context
Add any other context about the problem here.
Environment
OPNsense 21.1.9_1-amd64
FreeBSD 12.1-RELEASE-p19-HBSD
OpenSSL 1.1.1k 25 Mar 2021

oops, wrong source used, https://github.com/opnsense/core/commit/4c4266045e88eb1411429104ca47e5cbf254a8cc should fix this.


Revert @import to #import for cedar compatibility.

When using this framework in a project that uses Cedar for a test framework the @import fails when running specs. This is because cedar tests are written in objective c++ which doesn't recognize @import.
Cedar:
https://github.com/pivotal/cedar

Sorry @spacecoder, modules are the future.

Agreed, but they don't compile in obj c++.


Implement the conversion from openmessaging to cloudevent

Fixes ISSUE#.
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Describe the modifications you've done.
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

looking forward to the implementation of cloudevnets to openmessaging

Codecov Report

Merging #514 (12b2187) into develop (3510c83) will increase coverage by 0.26%.
The diff coverage is 67.24%.

:exclamation: Current head 12b2187 differs from pull request most recent head 928b32f. Consider uploading reports for the commit 928b32f to get more accurate results

@@              Coverage Diff              @@
##             develop     #514      +/-   ##
=============================================
+ Coverage      10.76%   11.03%   +0.26%
- Complexity       353      376      +23
=============================================
  Files            253      257       +4
  Lines          12099    12157      +58
  Branches        1028     1032       +4
=============================================
+ Hits            1302     1341      +39
- Misses         10687    10701      +14
- Partials         110      115       +5

Impacted Files
Coverage Δ

...ore/protocol/cloudevent/impl/OMSMessageWriter.java
62.50% <62.50%> (ø)

...me/core/protocol/cloudevent/OMSMessageFactory.java
66.66% <66.66%> (ø)

...time/core/protocol/cloudevent/impl/OMSHeaders.java
75.00% <75.00%> (ø)

...otocol/cloudevent/impl/OMSBinaryMessageReader.java
76.92% <76.92%> (ø)

Continue to review full report at Codecov.

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


[Bug] Changing settings sends a test notification when box is unchecked

See title.

@WindedHero unable to reproduce, i set a breakpoint on both the watch settings and global settings, and it's always False
it never ends that if: part

This is in current :latest

I am running this in your official docker container on Unraid that checks for updates daily, so I'm on version .38
Still having this issue. I can take video if you'd like?

https://youtu.be/PCAreenaISI
There you have it.

@WindedHero oops, I see it! Looks like it came in from another merge, thanks for taking the time to report it - When I tested it, I didnt add any notification URLs :/

Since 0.38 I don't get notifications when a site changes. Since 0.38.1 I don't get any test notifications either. I'm using Join.

@KnifeFed what platform? which notification backend? what's a "Join" ? are you talking about the [ send test notification ] or something else entirely? got any log output? did you get an error? try to imagine that i cannot read your mind

@dgtlmoon

what platform?

Official Docker image on Unraid.

which notification backend?

Join.

what's a "Join" ?

See above.

are you talking about the [ send test notification ] or something else entirely?

I don't get any notifications since upgrading to 0.38.1. It worked flawlessly before.

got any log output?

Nothing much. Here's when I click Save in the settings with "Send test notification on save" ticked:

today at 01:47:04 (8) accepted ('<IP_ADDRESS>', 56487)
today at 01:47:04 <IP_ADDRESS> - - [21/Jul/2021 01:47:04] "POST /settings HTTP/1.1" 200 9182 0.003588
today at 01:47:04 <IP_ADDRESS> - - [21/Jul/2021 01:47:04] "GET /static/styles/pure-min.css HTTP/1.1" 304 235 0.001356
today at 01:47:04 <IP_ADDRESS> - - [21/Jul/2021 01:47:04] "GET /static/styles/styles.css HTTP/1.1" 304 235 0.001710
today at 01:47:04 (8) accepted ('<IP_ADDRESS>', 56488)
today at 01:47:04 <IP_ADDRESS> - - [21/Jul/2021 01:47:04] "GET /static/js/settings.js HTTP/1.1" 304 234 0.001667
today at 01:47:04 (8) accepted ('<IP_ADDRESS>', 56489)
today at 01:47:04 <IP_ADDRESS> - - [21/Jul/2021 01:47:04] "GET /static/images/gradient-border.png HTTP/1.1" 304 236 0.001329
today at 01:47:05 Saving..

did you get an error?

No, I haven't gotten any errors.

@KnifeFed and you do realise that this issue is not related to your issue? this issue is about that it sends notifications even when the checkbox is off, you are just jumping into someone elses issue, please file a new issue
Also I have no idea what is Unraid, please tell me if its on raspberryPi , the type of ARM, or intel or whatever you're running (in the new issue)

@KnifeFed also on your new issue, attach your backup zip file, or atleast the main json index file

Sorry about that, new issue here: #148


Issues seen when using ^c to terminate a chat message.

gptscript version v0.0.0-dev-faaee653-dirty
Scenario 1:
Takes very long (about 20 -25 secs) to get interrupted message and user prompt when cancelling a chat message
Also we loose the actual user prompt and exec call that was made after this.
Steps to reproduce the problem

Execute the following script

Context: github.com/gptscript-ai/clio/context
Chat: true

You are a good assistant. Wait for user to ask you a question.

I tried to list all files from a directory that was taking long.
When listing of files is still in progress, use  ^c to terminate a chat message.

Notice that it takes very long (about 20 -25 secs) to get the interrupted message and user prompt after cancelling a chat message
Also we loose the actual user prompt and exec call that was made.
gptscript chat.gpt

  Hey there! What DevOps task do you need help with today?

Interrupted

>

Expected Behavior:

Should be able to see the interrupted the message and prompt right after a chat message canceled.
It would be good to still show the user prompt and tool call in the thread .

Scenario 2:
Using  ^c to terminate a chat message 2nd time causes the script to exit.

Execute the following script

Context: github.com/gptscript-ai/clio/context
Chat: true

You are a good assistant. Wait for user to ask you a question.

Tried to list all files from a directory that was taking a long time.
When listing of files is still in progress, use  ^c to terminate a chat message.
Tried to again list all files from a directory that was taking a long time.
When listing of files is still in progress, use  ^c to terminate a chat message.

Notice that the script exits in this case.
Expected Behavior:
Only the chat message should be terminated in the case and user should still be able to continue to chat.

Tested with gptscript version - v0.0.0-dev-c8cf3106-dirty
I am able to reproduce this issue when testing this version.

Tested with gptscript version v0.0.0-dev-24b1f24e-dirty
I am now not able to reproduce scenario 2  any more.
I tried to list files and did a ^c when files were being listed . Repeated this 4 times and I still continue to be the chat session as expected
gptscript chat.gpt

  Hello! How can I assist you today? If you have any DevOps-related tasks or need help with CLI commands, feel free to let me know!

Interrupted

Interrupted

Interrupted

Interrupted

>

But scenario 1 is still happening were I loose the prompt (and the contents that were displayed before ^c was hit).

We delete the old prompt and content because they aren't available to the session anymore, they are forgotten. I just felt if I left them on the screen people would think they could enter a new prompt and the LLM would know about the content on the screen from the canceled run, which is not the case.


Change 'Health Check' to 'Health' in UI and description.

Fixes #317.

@rdebusscher  This branch has conflicts that must be resolved...

Merging into master as it was approved and merge conflict resolved.


address nue build failure: Microsoft.VisualStudio.Services.ExtensionManagement.WebApi

This package cannot currently be built. This is blocking the Team Services .NET doc release.
https://www.nuget.org/packages/Microsoft.VisualStudio.Services.ExtensionManagement.WebApi/

@meganbradley  & @dend Thanks for following up!

@meganbradley @Pacort
This is related to this NuGet issue

@dend and @meganbradley Is there a workaround or a fix coming soon? Thanks!

Another issue is documented here: https://github.com/NuGet/Home/issues/4512#issuecomment-327037370
This is blocking and we are waiting for a resolution from the NuGet team.

Closing the issue, as it is external in nature - the Azure Pipelines team needs to ensure that the right dependency constraint is applied (see NuGet issue for details).


BC16.4 container creation fails

Describe the issue
BC16.4 artifact downloads dynamicsnav:10.0.14393.3808-generic, but fails with error "The system cannot find the file specified." Or "There is not enough space on the disk."
Uses BC10.0 image to create the container
Scripts used to create container and cause the issue
Get-Module BCContainerHelper -ListAvailable
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12
Remove-Module NavContainerHelper
Install-Module -Name BCContainerHelper -RequiredVersion 1.0.3
#Use BCContainerHelper instead of NavContainerHelper

#BC16.4+
$version = "16.4"
$ContainerName = "Master"
$auth = "Windows"
#$auth = "UserPassword"
#$credential = New-Object pscredential 'P', (ConvertTo-SecureString -String 'P' -AsPlainText -Force)
$License = 'C:\License\us.flf'

#Lijst:
#Get-BCArtifactUrl -country "be" -version $version -select All

$artifactUrl = Get-BCArtifactUrl -country "be" -version $version

Write-Host -ForegroundColor Yellow "creating $ContainerName $artifactUrl"
Download-Artifacts -artifactUrl $artifactUrl  -includePlatform

Write-Host -ForegroundColor Yellow "bcimage"
New-BCImage -artifactUrl $artifactUrl -imageName myownimage

Write-Host -ForegroundColor Yellow "Creating container $ContainerName"
Remove-BCContainer $ContainerName

Measure-Command {
    New-BCContainer `
        -accept_eula `
        -containerName $ContainerName `
        -artifactUrl $artifactUrl `
        -updateHosts `
        -auth $auth `
        -licenseFile $License `
        -includeAL `
        -imagename myownimage
}

Full output of scripts
ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     1.0.3      BcContainerHelper                   {Add-FontsToBcContainer, Add-GitToAlProjectFolder, Backup-BcContainerDatabases, Check-BcContainerHelperPermissions...}
Remove-Module : No modules were removed. Verify that the specification of modules to remove is correct and those modules exist in the runspace.
At C:\Users\b\Documents\NAVMaster\PowerShell\InstallContainerArtifacts.ps1:3 char:1
+ Remove-Module NavContainerHelper
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ResourceUnavailable: (:) [Remove-Module], InvalidOperationException
    + FullyQualifiedErrorId : Modules_NoModulesRemoved,Microsoft.PowerShell.Commands.RemoveModuleCommand

PackageManagement\Install-Package : No match was found for the specified search criteria and module name 'BCContainerHelper'. Try Get-PSRepository to see all available registered module repositories.
At C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\<IP_ADDRESS>\PSModule.psm1:1772 char:21
+ ...          $null = PackageManagement\Install-Package @PSBoundParameters
+                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Microsoft.Power....InstallPackage:InstallPackage) [Install-Package], Exception
    + FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.PackageManagement.Cmdlets.InstallPackage

creating Master https://bcartifacts.azureedge.net/sandbox/16.4.14693.15645/be
Downloading application artifact /sandbox/16.4.14693.15645/be
Downloading C:\Users\b\AppData\Local\Temp\6a698be5-ebf2-4f35-b013-2d49a257f300.zip
Unpacking application artifact to tmp folder using 7zip
c:\bcartifacts.cache\sandbox\16.4.14693.15645\be
Downloading platform artifact /sandbox/16.4.14693.15645/platform
Downloading C:\Users\b\AppData\Local\Temp\9e7b9236-f1dd-4261-9515-05571a45eee4.zip
Unpacking platform artifact to tmp folder using 7zip
Downloading Prerequisite Components
Downloading c:\bcartifacts.cache\sandbox\16.4.14693.15645\platform\Prerequisite Components\Open XML SDK 2.5 for Microsoft Office\OpenXMLSDKv25.msi
Downloading c:\bcartifacts.cache\sandbox\16.4.14693.15645\platform\Prerequisite Components\IIS URL Rewrite Module\rewrite_2.0_rtw_x64.msi
Downloading c:\bcartifacts.cache\sandbox\16.4.14693.15645\platform\Prerequisite Components\DotNetCore\DotNetCore.1.0.4_1.1.1-WindowsHosting.exe
c:\bcartifacts.cache\sandbox\16.4.14693.15645\platform
bcimage
Building image myownimage:sandbox-16.4.14693.15645-be based on mcr.microsoft.com/dynamicsnav:10.0.14393.3808-generic
myownimage:sandbox-16.4.14693.15645-be
Pulling latest image mcr.microsoft.com/dynamicsnav:10.0.14393.3808-generic
10.0.14393.3808-generic: Pulling from dynamicsnav
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.14393.3808 (ltsc2016)
Host OS Version: 10.0.14393.3808 (ltsc2016)
Using process isolation
Files in c:\bcartifacts.cache\tmp637333527557152873\my:
Copying Platform Artifacts
Copying Database
Copying Licensefile
Copying ConfigurationPackages
Copying Extensions
Copying Applications.BE
c:\bcartifacts.cache\tmp637333527557152873
Sending build context to Docker daemon  1.783GB

Step 1/6 : FROM mcr.microsoft.com/dynamicsnav:10.0.14393.3808-generic
 ---> 919a8b4f600a
Step 2/6 : ENV DatabaseServer=localhost DatabaseInstance=SQLEXPRESS DatabaseName=CRONUS IsBcSandbox=Y artifactUrl=https://bcartifacts.azureedge.net/sandbox/16.4.14693.15645/be
 ---> Running in fdae7625ef56
Removing intermediate container fdae7625ef56
 ---> f007330f56f4
Step 3/6 : COPY my /run/
 ---> af6f77cf3b19
Step 4/6 : COPY NAVDVD /NAVDVD/
docker : failed to copy files: failed to copy directory: chtimes \\?\Volume{b74e77cd-ba8e-11e9-9c1f-bbbb99aed92d}\NAVDVD\ServiceTier\program files\Microsoft Dynamics NAV\160\Service\Microsoft.Owin.dll: The syst
em cannot find the file specified.
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\1.0.3\ContainerHandling\New-NavImage.ps1:346 char:1
+ docker build --isolation=$isolation --memory $memory --tag $imageName ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (failed to copy ...file specified.:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

Screenshots
/
Additional context

does it happen all the time? yep
did it use to work? Can create containers with old navcontainerhelper without artifacts for bc16.3

When I leave out the BC Image
#New-BCImage -artifactUrl $artifactUrl -imageName myownimage
it also fails:
Creating DotNetCore Web Server Instancedocker : . : The term 'c:\Run\MainLoop.ps1' is not recognized as the name of a cmdlet,
ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     1.0.3      BcContainerHelper                   {Add-FontsToBcContainer, Add-GitToAlProjectFolder, Backup-BcContainerDatabases, Check-BcContainerHelperPermissions...}
Remove-Module : No modules were removed. Verify that the specification of modules to remove is correct and those modules exist in the runspace.
At C:\Users\b\Documents\NAVMaster\PowerShell\InstallContainerArtifacts.ps1:3 char:1
+ Remove-Module NavContainerHelper
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ResourceUnavailable: (:) [Remove-Module], InvalidOperationException
    + FullyQualifiedErrorId : Modules_NoModulesRemoved,Microsoft.PowerShell.Commands.RemoveModuleCommand

PackageManagement\Install-Package : No match was found for the specified search criteria and module name 'BCContainerHelper'. Try Get-PSRepository to see all available registered module repositories.
At C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\<IP_ADDRESS>\PSModule.psm1:1772 char:21
+ ...          $null = PackageManagement\Install-Package @PSBoundParameters
+                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Microsoft.Power....InstallPackage:InstallPackage) [Install-Package], Exception
    + FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.PackageManagement.Cmdlets.InstallPackage

creating Master https://bcartifacts.azureedge.net/sandbox/16.4.14693.15645/be
c:\bcartifacts.cache\sandbox\16.4.14693.15645\be
c:\bcartifacts.cache\sandbox\16.4.14693.15645\platform
bcimage
Creating container Master
BcContainerHelper is version 1.0.3
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2016 Datacenter - ltsc2016
Docker Client Version is 18.09.6
Docker Server Version is 18.09.6
Fetching all docker images
Using image mcr.microsoft.com/dynamicsnav:10.0.14393.3808-generic
Creating Container Master
Version: 16.4.14693.15645-BE
Style: sandbox
Multitenant: Yes
Platform: 16.0.14648.15622
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.14393.3808 (ltsc2016)
Host OS Version: 10.0.14393.3808 (ltsc2016)
Using process isolation
Using locale nl-BE
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file C:\License\us.flf
Files in C:\ProgramData\BcContainerHelper\Extensions\Master\my:
- AdditionalOutput.ps1
- license.flf
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container Master from image mcr.microsoft.com/dynamicsnav:10.0.14393.3808-generic
bd78a9437e2e22f7ffb7ebbc809c6c694aa9f661d1536d283bc8219120b7e878
Waiting for container Master to be ready
Using artifactUrl https://bcartifacts.azureedge.net/sandbox/16.4.14693.15645/be
Using installer from C:\Run\150-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
Starting Internet Information Server
Copying Service Tier Files
Copying PowerShell Scripts
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Determining Database Collation from c:\dl\sandbox\16.4.14693.15645\be\BusinessCentral-BE.bak
Restoring CRONUS Demo Database
Exporting Application to CRONUS
Removing Application from tenant
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Importing CRONUS license file
Copying Database on localhost\SQLEXPRESS from tenant to default
Taking database tenant offline
Copying database files
Attaching files as new Database default
Putting database tenant back online
Mounting tenant database
Mounting Database for default on server localhost\SQLEXPRESS
Sync'ing Tenant
Tenant is Operational
Stopping Business Central Service Tier
Copying Web Client Files
Copying Client Files
Copying ModernDev Files
Copying additional files
Copying ConfigurationPackages
Copying Test Assemblies
Copying Extensions
Copying Applications
Copying Applications.BE
Installation took 297 seconds
Installation complete
Initializing...
Starting Container
Hostname is Master
PublicDnsName is Master
WARNING: DNS resolution not working from within the container.
Using Windows Authentication
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instancedocker : . : The term 'c:\Run\MainLoop.ps1' is not recognized as the name of a cmdlet,
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\1.0.3\ContainerHandling\Wait-NavContainerReady.ps1:31 char:21
+             $logs = docker logs $containerName
+                     ~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (. : The term 'c...e of a cmdlet, :String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

Could you try to add -isolation hyperv to your new-navcontainer command?

Without using BC Image, has it created an "unhealthy" container master, that's unable to run.
Log:
{"log":"Using artifactUrl https://bcartifacts.azureedge.net/sandbox/16.4.14693.15645/be\n","stream":"stdout","time":"2020-08-18T11:17:26.0913582Z"}
{"log":"Using installer from C:\Run\150-new\n","stream":"stdout","time":"2020-08-18T11:17:26.817309Z"}
{"log":"Installing Business Central\n","stream":"stdout","time":"2020-08-18T11:17:27.0839517Z"}
{"log":"Installing from artifacts\n","stream":"stdout","time":"2020-08-18T11:17:27.1010182Z"}
{"log":"Starting Local SQL Server\n","stream":"stdout","time":"2020-08-18T11:17:27.1020189Z"}
{"log":"WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to \r\n","stream":"stdout","time":"2020-08-18T11:17:36.2152793Z"}
{"log":"start...\n","stream":"stdout","time":"2020-08-18T11:17:36.2152793Z"}
{"log":"Starting Internet Information Server\n","stream":"stdout","time":"2020-08-18T11:17:36.4791958Z"}
{"log":"Copying Service Tier Files\n","stream":"stdout","time":"2020-08-18T11:17:36.8999361Z"}
{"log":"Copying PowerShell Scripts\n","stream":"stdout","time":"2020-08-18T11:17:57.7279826Z"}
{"log":"Copying dependencies\n","stream":"stdout","time":"2020-08-18T11:17:59.6057778Z"}
{"log":"Copying ReportBuilder\n","stream":"stdout","time":"2020-08-18T11:17:59.7937867Z"}
{"log":"Importing PowerShell Modules\n","stream":"stdout","time":"2020-08-18T11:17:59.9513539Z"}
{"log":"Determining Database Collation from c:\dl\sandbox\16.4.14693.15645\be\BusinessCentral-BE.bak\n","stream":"stdout","time":"2020-08-18T11:18:00.0941758Z"}
{"log":"Restoring CRONUS Demo Database\n","stream":"stdout","time":"2020-08-18T11:18:02.527573Z"}
{"log":"Exporting Application to CRONUS\n","stream":"stdout","time":"2020-08-18T11:18:07.1360469Z"}
{"log":"Removing Application from tenant\n","stream":"stdout","time":"2020-08-18T11:19:48.5323783Z"}
{"log":"Modifying Business Central Service Tier Config File for Docker\n","stream":"stdout","time":"2020-08-18T11:19:54.1923233Z"}
{"log":"Creating Business Central Service Tier\n","stream":"stdout","time":"2020-08-18T11:19:54.2373224Z"}
{"log":"Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'\n","stream":"stdout","time":"2020-08-18T11:19:54.349487Z"}
{"log":"Starting Business Central Service Tier\n","stream":"stdout","time":"2020-08-18T11:19:54.628808Z"}
{"log":"Importing CRONUS license file\n","stream":"stdout","time":"2020-08-18T11:20:09.2700281Z"}
{"log":"Copying Database on localhost\SQLEXPRESS from tenant to default\n","stream":"stdout","time":"2020-08-18T11:20:10.1185331Z"}
{"log":"Taking database tenant offline\n","stream":"stdout","time":"2020-08-18T11:20:10.2154117Z"}
{"log":"Copying database files\n","stream":"stdout","time":"2020-08-18T11:20:10.8481528Z"}
{"log":"Attaching files as new Database default\n","stream":"stdout","time":"2020-08-18T11:20:12.6138085Z"}
{"log":"Putting database tenant back online\n","stream":"stdout","time":"2020-08-18T11:20:13.9323154Z"}
{"log":"Mounting tenant database\n","stream":"stdout","time":"2020-08-18T11:20:14.0993328Z"}
{"log":"Mounting Database for default on server localhost\SQLEXPRESS\n","stream":"stdout","time":"2020-08-18T11:20:14.1073311Z"}
{"log":"Sync'ing Tenant\n","stream":"stdout","time":"2020-08-18T11:22:21.8228354Z"}
{"log":"Tenant is Operational\n","stream":"stdout","time":"2020-08-18T11:22:23.9115977Z"}
{"log":"Stopping Business Central Service Tier\n","stream":"stdout","time":"2020-08-18T11:22:23.9115977Z"}
{"log":"Copying Web Client Files\n","stream":"stdout","time":"2020-08-18T11:22:24.3067913Z"}
{"log":"Copying Client Files\n","stream":"stdout","time":"2020-08-18T11:22:24.3067913Z"}
{"log":"Copying ModernDev Files\n","stream":"stdout","time":"2020-08-18T11:22:24.3067913Z"}
{"log":"Copying additional files\n","stream":"stdout","time":"2020-08-18T11:22:24.3067913Z"}
{"log":"Copying ConfigurationPackages\n","stream":"stdout","time":"2020-08-18T11:22:24.3077891Z"}
{"log":"Copying Test Assemblies\n","stream":"stdout","time":"2020-08-18T11:22:24.3077891Z"}
{"log":"Copying Extensions\n","stream":"stdout","time":"2020-08-18T11:22:24.3077891Z"}
{"log":"Copying Applications\n","stream":"stdout","time":"2020-08-18T11:22:24.3088503Z"}
{"log":"Copying Applications.BE\n","stream":"stdout","time":"2020-08-18T11:22:24.3088503Z"}
{"log":"Installation took 297 seconds\n","stream":"stdout","time":"2020-08-18T11:22:24.3157908Z"}
{"log":"Installation complete\n","stream":"stdout","time":"2020-08-18T11:22:24.3157908Z"}
{"log":"Initializing...\n","stream":"stdout","time":"2020-08-18T11:22:24.3542352Z"}
{"log":"Starting Container\n","stream":"stdout","time":"2020-08-18T11:22:25.4900485Z"}
{"log":"Hostname is Master\n","stream":"stdout","time":"2020-08-18T11:22:25.4940472Z"}
{"log":"PublicDnsName is Master\n","stream":"stdout","time":"2020-08-18T11:22:25.4940472Z"}
{"log":"WARNING: DNS resolution not working from within the container.\n","stream":"stdout","time":"2020-08-18T11:22:26.0830951Z"}
{"log":"Using Windows Authentication\n","stream":"stdout","time":"2020-08-18T11:22:26.6832606Z"}
{"log":"Modifying Service Tier Config File with Instance Specific Settings\n","stream":"stdout","time":"2020-08-18T11:22:26.7752656Z"}
{"log":"Starting Service Tier\n","stream":"stdout","time":"2020-08-18T11:22:27.9032605Z"}
{"log":"Registering event sources\n","stream":"stdout","time":"2020-08-18T11:22:37.1792857Z"}
{"log":"Creating DotNetCore Web Server Instance\n","stream":"stdout","time":"2020-08-18T11:22:37.2302876Z"}
{"log":"Access to the path 'fr-CA' is denied.\n","stream":"stdout","time":"2020-08-18T11:22:56.3179206Z"}
{"log":"at New-NavWebSite, C:\Program Files\Microsoft Dynamics NAV\160\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 566\r\n","stream":"stdout","time":"2020-08-18T11:22:56.3199185Z"}
{"log":"at New-NAVWebServerInstance, C:\Program Files\Microsoft Dynamics NAV\160\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 501\r\n","stream":"stdout","time":"2020-08-18T11:22:56.3199185Z"}
{"log":"at \u003cScriptBlock\u003e, C:\Run\SetupWebClient.ps1: line 26\r\n","stream":"stdout","time":"2020-08-18T11:22:56.3199185Z"}
{"log":"at \u003cScriptBlock\u003e, C:\Run\navstart.ps1: line 157\r\n","stream":"stdout","time":"2020-08-18T11:22:56.3199185Z"}
{"log":"at \u003cScriptBlock\u003e, C:\Run\start.ps1: line 338\r\n","stream":"stdout","time":"2020-08-18T11:22:56.3199185Z"}
{"log":"at \u003cScriptBlock\u003e, \u003cNo file\u003e: line 1\n","stream":"stdout","time":"2020-08-18T11:22:56.3199185Z"}
{"log":". : The term 'c:\Run\MainLoop.ps1' is not recognized as the name of a cmdlet, \r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"function, script file, or operable program. Check the spelling of the name, or \r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"if a path was included, verify that the path is correct and try again.\r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"At C:\Run\start.ps1:361 char:7\r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"+     . (Get-MyFilePath "MainLoop.ps1")\r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"+       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"    + CategoryInfo          : ObjectNotFound: (c:\Run\MainLoop.ps1:String) [], \r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"    ParentContainsErrorRecordException\r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":"    + FullyQualifiedErrorId : CommandNotFoundException\r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}
{"log":" \r\n","stream":"stderr","time":"2020-08-18T11:22:56.3862631Z"}

with -isolation hyperv  to containercreation without BC Image
Creating container Master from image mcr.microsoft.com/dynamicsnav:10.0.14393.3808-generic
77e7356e49dd6559ae70451900068a600c40ef703048e268013972451a83b648
DockerDo : docker.exe: Error response from daemon: CreateComputeSystem 77e7356e49dd6559ae70451900068a600c40ef703048e268013972451a83b648: No hypervisor is present on this system.
[Event Detail: Cannot start Hyper-V Container '77e7356e49dd6559ae70451900068a600c40ef703048e268013972451a83b648' since the hypervisor is not running in the host. Provider: 17103e3f-3c6e-4677-bb17-3b267eb5be57 E
ventID: 12001]
(extra info: {"SystemType":"Container","Name":"77e7356e49dd6559ae70451900068a600c40ef703048e268013972451a83b648","Owner":"docker","IgnoreFlushesDuringBoot":true,"LayerFolderPath":"C:\ProgramData\docker\windo
wsfilter\77e7356e49dd6559ae70451900068a600c40ef703048e268013972451a83b648","Layers":[{"ID":"016b593a-28f4-511e-8f5f-fb6f968470e2","Path":"C:\ProgramData\docker\windowsfilter\e720298755f3fe59f6bd210e5c70af4
cec657c35261bfce0c6b6913e775289cd"},{"ID":"d7c0c2f6-57fd-5f40-8a57-f984228fb705","Path":"C:\ProgramData\docker\windowsfilter\a3c319e051e4579990c3fd5ea65ff7b6aae2d72d2945ec5a781fa33246edb3ab"},{"ID":"fdc3449
4-5798-5d13-8d05-ac98008c7e46","Path":"C:\ProgramData\docker\windowsfilter\2def0363c080a6d80c71a8846c5a034f5e3439e4db75d4a58e8bd6f922ce3b8b"},{"ID":"a19dcae7-1216-5502-9da8-d61d431978ee","Path":"C:\Program
Data\docker\windowsfilter\9a3b0cfff61f7339b63ff1287af944cd7f15c6dd62614447e808f675f9c91e79"},{"ID":"dab8abc1-0e08-50c5-97f5-20623a9cfb27","Path":"C:\ProgramData\docker\windowsfilter\2d6717c2f4359ec185099
be8de82f532f9f96d9850cf46072b65c343bff3b8f8"},{"ID":"5cbc1db3-177e-5cb0-a523-21da9d0461a1","Path":"C:\ProgramData\docker\windowsfilter\c00eb4a56e6e4f4f1a4ae25196fc5d556c4b6f7291fb4131a480969d2a523bab"},{"ID
":"4cb520f0-e3db-5d2b-b156-11760d0ccc4f","Path":"C:\ProgramData\docker\windowsfilter\1ae31b0bab06dac820465774ee75d78bc6eb4d2f47bd7b6f535899f07b7047dc"},{"ID":"dcfc17d1-910b-5e07-a5b3-7744c2dde171","Path":"C
:\ProgramData\docker\windowsfilter\8a1f46054ea37b52c459ddb0b3539bd9969e04221efecdb1c203451f102d5902"},{"ID":"b977a62d-d100-55a7-bcfc-a3bd38fc335d","Path":"C:\ProgramData\docker\windowsfilter\8c2c2c5ab73
85aafb7ed1d9633b52d37f287451651868dd8e5bb441978710b3e"},{"ID":"62c057fb-50cf-5057-9249-884eb3ab471b","Path":"C:\ProgramData\docker\windowsfilter\e80935d69bcfd0a7bf5906474d117f01e53b08e0c691b5469bd55f3fd37f6
ebb"},{"ID":"93719cc7-e7ef-5ef2-97db-270bbadd34ab","Path":"C:\ProgramData\docker\windowsfilter\2753c7611c7b066acfd22489493f5c93b4f011f86c6747ecc4b5f23dcb2c476f"},{"ID":"112f2233-875c-5a3a-b1dc-731340f6d6f5"
,"Path":"C:\ProgramData\docker\windowsfilter\a6fd9b4c36501b885d6935fad1fd38693d77cb66e181cf88f76c2623d62922ab"},{"ID":"af1e2435-2845-54f5-ad01-201cf30f36ea","Path":"C:\ProgramData\docker\windowsfilter\2
d0a2543791439ff9665c369c8d8ee0813075db80d14e33d296d3def83c1574a"},{"ID":"5f104553-2e31-5f8c-a360-ee37b8f72ae1","Path":"C:\ProgramData\docker\windowsfilter\d005707cf6c967c3ee360e77c15b6224b23960538ca270dd803
1dc85a4d22196"},{"ID":"76ab0d6c-0507-5251-9e38-450020725e06","Path":"C:\ProgramData\docker\windowsfilter\ef2ac01084aabd09ae0e8683b6ee14b191f81ec07cf997a253ef79750f4c8002"},{"ID":"1fb1c691-08a5-5163-afb8-3d3
265b4e6e9","Path":"C:\ProgramData\docker\windowsfilter\a3e2dd642c1623570424cb4da138d92d0a92e48e2e52065ce7aea54b5ebfe333"},{"ID":"56c0b548-c9bd-546b-80c0-ec076db10575","Path":"C:\ProgramData\docker\window
sfilter\cf716f99372d873e37098b11f1fffc46c469218a27d41d15778304fe114d6ea4"},{"ID":"19d66994-9599-59d7-a1fc-f86484f95725","Path":"C:\ProgramData\docker\windowsfilter\19807d421d55c124971b3b878d3d4896935e4e929
ccc386258a3fe5ceedc5bac"},{"ID":"7ae563e5-9079-57e0-a83b-6d2009a3724b","Path":"C:\ProgramData\docker\windowsfilter\f7704e18e565303e14cc2fda776989e5d8d28490b8f674a1caa5a3219095dadf"},{"ID":"dd76ba09-bf66-57f
4-95e1-e8a4dc0fd2bb","Path":"C:\ProgramData\docker\windowsfilter\852efe60c04d7ee14a67b35335c44eaa0d9e7d01b21914951fc3099f18c06bee"},{"ID":"2e0e27ea-25ea-5693-934a-e97cb842e998","Path":"C:\ProgramData\dock
er\windowsfilter\11efb8fb0cc4b23174c4e76bfdc78834034eef3e925ea71a71040506b97696cf"},{"ID":"43520bf6-9d94-5380-b8b7-afa8f8e2fd55","Path":"C:\ProgramData\docker\windowsfilter\6c7de13a6af27750ea9a304b365b049
758e50fc7bcd1b342b33a006c8a53924d"}],"MemoryMaximumInMB":4096,"HostName":"GeneralMaster","MappedDirectories":[{"HostPath":"c:\windows\system32\drivers\etc","ContainerPath":"c:\driversetc","ReadOnly":false,
"BandwidthMaximum":0,"IOPSMaximum":0,"CreateInUtilityVM":false},{"HostPath":"c:\bcartifacts.cache","ContainerPath":"c:\dl","ReadOnly":false,"BandwidthMaximum":0,"IOPSMaximum":0,"CreateInUtilityVM":false},{"Ho
stPath":"c:\programdata\bccontainerhelper","ContainerPath":"c:\programdata\bccontainerhelper","ReadOnly":false,"BandwidthMaximum":0,"IOPSMaximum":0,"CreateInUtilityVM":false},{"HostPath":"c:\programdata\b
ccontainerhelper\extensions\generalmaster\my","ContainerPath":"c:\run\my","ReadOnly":false,"BandwidthMaximum":0,"IOPSMaximum":0,"CreateInUtilityVM":false}],"HvPartition":true,"HvRuntime":{"ImagePath":"C:\
ProgramData\docker\windowsfilter\11efb8fb0cc4b23174c4e76bfdc78834034eef3e925ea71a71040506b97696cf\UtilityVM"}}).
ExitCode: 125
Commandline: docker run --volume c:\bcartifacts.cache:c:\dl --label nav= --env isBcSandbox=Y --label version=16.4.14693.15645 --label platform=16.0.14648.15622 --label country=BE --env artifactUrl=https://bcart
ifacts.azureedge.net/sandbox/16.4.14693.15645/be --env multitenant=Y --env licenseFile="c:\run\my\license.flf" --name GeneralMaster --hostname GeneralMaster --env auth=Windows --env username="b"
--env ExitOnError=N --env locale=nl-BE --env databaseServer="" --env databaseInstance="" --volume "C:\ProgramData\BcContainerHelper:C:\ProgramData\BcContainerHelper" --volume "C:\ProgramData\BcContainerHelper\E
xtensions\GeneralMaster\my:C:\Run\my" --isolation hyperv --restart unless-stopped --memory 4G --env enableApiServices=Y --env useSSL=N --volume "c:\windows\system32\drivers\etc:C:\driversetc" --env securePasswo
rd=X= --env passwordKeyFile="c:\run\my\aes.key" --env removePasswordKeyFile=Y --env accept_eula=Y --env accept_outdated=Y --detach mcr.microsoft.c
om/dynamicsnav:10.0.14393.3808-generic
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\1.0.3\ContainerHandling\New-NavContainer.ps1:1593 char:19

...       if (!(DockerDo -accept_eula -accept_outdated:$accept_outdated - ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,DockerDo

And you have enough disk space on the machine?
I have just tried to build that exact image (with that generic) - and it works fine here.

1.43GB/149GB free, so doubt it ... :(
Although I don't have any room I can create.

I need those other 4 images.

In C:\ProgramData\docker\windowsfilter are some folders marked with "-removing" but still there.
So perhaps if I can delete those.
dynamicsnav:10.0.14393.3808-generic is also taking "unneeded" space ... :s

You should never manually remove files in WindowsFilter.
You can try with Flush-Containerhelpercache.
and you can remove unused images - the generic image is basically the only one you need.

How else can I delete 50GB (used to be so much) of unneeded imageparts that won't go away ... :(
managed to remove it all, but somehow it came back.
Guess my server is broken :'(
I need those other images for my other projects ...
I need to upgrade the BC16.3 sandbox projects to the BC16.4 , but rather don't do that until I got my new containers.
Just to be sure.
Or do you mean I can also recreate all those other images+containers, based on the generic one?
Flush-Containerhelpercache didn't help.

With artifacts we don't use images anymore - everything can be rebuilt (unless you have something in a container which you need to keep - which I do not recommend)
This tool: https://freddysblog.com/2018/12/11/clean-up-after-yourself-docker-your-mom-isnt-here/
should be able to remove all docker images - and then they can re-download / re-build

Hmmn, ok will try that tool and recreate whatever I need with artifacts.
Thank you for your time, brain and assistance!


Signing script called twice for each binary in jar

Describe the bug
We have a signing script set up via app.mac.sign.scripts. The script sets up an SSH tunnel to our intranet and uses an internal service to sign individual binaries or app bundles. The signing process seems to work but for some reason the entire script is called twice for each binary.
To Reproduce
Set up the scripts as such:
  mac {
    icons = "installer/conveyor/mac-icon.tif"

    sign {
      scripts {
        app = "./sign-notarization-script.sh bundle $ENTITLEMENTS $BUNDLE"
        binary = "./sign-notarization-script.sh file $ENTITLEMENTS $FILE"
      }
    }
  }

With a script that looks like this (with irrelevant commands sanitised):
#!/usr/bin/env bash

echo "Signing: $@"

if [[ "$1" == "bundle" ]]; then
    echo "Bundle $1 $2 $3"
else
    echo "Signing binary file: $3 with entitlements: $2"
    zip -j "$3.zip" "$3"
    echo "Signing the binary here"
    rm "$3"
    unzip -o "$3.zip" -d "$(dirname $3)"
    rm "$3.zip"
fi

Run conveyor -Kapp.machines=mac.aarch64 make mac-app --rerun.
Observe the line
Signing: file /var/folders/5n/02wrtg_95nv26cb01vfpxcy40000gp/T/.tmp-7426078686834927/entitlements.plist /Users/<user>/Library/Caches/Hydraulic/Conveyor/working/99b1b8442de59de4/content/jars/signed/jna-5_13_0_jar_libjnidispatch.jnilib

being printed twice (of course, dependent on the binaries that you need signed).
Expected behavior
The script runs only once, signing the binary.
Desktop (please complete the following information):
MacOS.
Thanks in advance!

Hi, sorry about that and apologies for the slow reply.
Unfortunately the reason for the slow reply is that I haven't been able to reproduce this problem. Using the same script and your instructions, I see the "Signing:" line being printed only once. I think it'd be useful here to see a log file. Could you do a run that exhibits this problem and then run conveyor --show-log=report ?

Hey @mikehearn - I've got something better - a straightforward reproduction: https://github.com/alwaysbemark/conveyor-reproduction
When I run conveyor -Kapp.machines=mac.aarch64 make mac-app I get the double call to the binary signing script:
Starting signing script with arguments: binary
Starting signing script with arguments: binary
Starting signing script with arguments: app

I suspect it's the way I'm including the JAR file: https://github.com/alwaysbemark/conveyor-reproduction/blob/main/build.gradle#L14
Thanks in advance!

Thanks! That helps a lot.
If we adjust the script a bit to contain if [[ $1 == "binary" ]]; then file $2; fi and add the arguments in the config file, then we see:
/Users/mike/Library/Caches/Hydraulic/Conveyor/working/20be897a350a3452/content/jars/signed/jna-5_13_0_jar_libjnidispatch.jnilib: Mach-O 64-bit dynamically linked shared library x86_64
/Users/mike/Library/Caches/Hydraulic/Conveyor/working/20be897a350a3452/content/jars/signed/jna-5_13_0_jar_libjnidispatch.jnilib: Mach-O 64-bit dynamically linked shared library arm64

So the reason it's being called twice is because the JNA library contains two Mac binaries.  It might be a bit surprising that Conveyor includes both but it's by design:
https://conveyor.hydraulic.dev/15.1/configs/jvm/#native-code

If false then libraries are left alone, except that libraries for macOS and Windows will be signed in-place (inside the JAR). This ensures that Apple's notarization process accepts them, and that when the libraries are extracted the operating system won't cause problems due to them being unsigned.

Basically, Conveyor defaults to tampering with JARs as little as possible to avoid breaking things. To make the extra file invocations go away you can add this line to the config:
include required("https://raw.githubusercontent.com/hydraulic-software/conveyor/master/configs/jvm/extract-native-libraries.conf")

... and of course fix the usage of the signing script ...
  mac {
    sign {
      scripts {
        app = "./mac-sign-script.sh app $BUNDLE $ENTITLEMENTS"
        binary = "./mac-sign-script.sh binary $FILE $ENTITLEMENTS $IDENTIFIER"
      }
    }
  }

then the script is invoked once, for the bundle, and not at all for binary. The config that's included also configures JNA's system properties correctly.
I'll have a think about how to avoid this being confusing. Probably, we should detect that the app includes JNA and print a hint suggesting to turn on library extraction.

Oh, by the way. If you try the above then Conveyor will crash. It's fixed in the next release already but the cause is that your JNA jar is "corrupted" by the presence of Apple MacDouble metadata files named like ._jnidispatch.dll which look like DLLs but aren't, and we should handle such files more gracefully than we currently do. I'm not sure how it got that way or if upstream has shipped a JAR like this, but it's a bit odd. You can strip the files from the JAR and then things will work.

Thanks Mike - that makes sense. I had no idea that both MacOS binaries are included and signed and it being called twice was a tad confusing but this makes sense. Also aware of the native library extraction feature but have yet to tinker with it.
For the binary invocation - we do refer to the files/entitlements/identifiers, I just removed them from the example for the sake of brevity :)
All clear now - happy to close this for now, thank you so much :)


Release 2.0.0

Following #40, this PR bumps version to 2.0.0.
There is also a v2.0.0 tag

Well, apparently the npm publish is done 😯
Maybe I shouldn't have pushed the tag?

😄 hah yeah the tag will release it. Let's get this merged then 😆


Update seqreports to 9adcd5fcc767c780dd471d331c3a0c2ffa370e1c

Updates the seqreports submodule, it contains the following changes:

https://github.com/Molmed/seqreports/pull/63 by @matrulda

Hmm, thought this was a temporary hickup, but a retry also fails. Will see if I can re-produce this locally.


HBASE-22939 SpaceQuotas - Bulkload from different hdfs failed when sp…

…ace quotas are turned on. (#553)
Signed-off-by: Sakthi<EMAIL_ADDRESS>(cherry picked from commit f31301dcf8e1e5d15a4b9939063e5514d810804b)
Change-Id: Ic88e96cc1b83351424463fef52579a35a4074822

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 47s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.4 Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 14s
branch-2.4 passed

+1 :green_heart:
compile
2m 38s
branch-2.4 passed

+1 :green_heart:
checkstyle
0m 36s
branch-2.4 passed

+1 :green_heart:
spotless
0m 39s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 36s
branch-2.4 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 10s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 55s
the patch passed

+1 :green_heart:
compile
2m 34s
the patch passed

+1 :green_heart:
javac
2m 34s
the patch passed

+1 :green_heart:
checkstyle
0m 34s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
14m 52s
Patch does not cause any errors with Hadoop 2.10.2 or 3.1.4 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 35s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 46s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 14s
The patch does not generate ASF License warnings.

37m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4750

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 44cb6af816e2 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / fd27d68b18

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
60 (vs. ulimit of 30000)

modules
C: hbase-server hbase-endpoint U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 54s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.4 Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m  4s
branch-2.4 passed

+1 :green_heart:
compile
0m 44s
branch-2.4 passed

+1 :green_heart:
shadedjars
4m  7s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 30s
branch-2.4 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 10s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 52s
the patch passed

+1 :green_heart:
compile
0m 44s
the patch passed

+1 :green_heart:
javac
0m 44s
the patch passed

+1 :green_heart:
shadedjars
4m  4s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 27s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
167m 35s
hbase-server in the patch passed.

+1 :green_heart:
unit
2m 27s
hbase-endpoint in the patch passed.

187m 31s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4750

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e01712d7b5cc 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / fd27d68b18

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/testReport/

Max. process+thread count
2580 (vs. ulimit of 30000)

modules
C: hbase-server hbase-endpoint U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 54s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.4 Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 14s
branch-2.4 passed

+1 :green_heart:
compile
1m 11s
branch-2.4 passed

+1 :green_heart:
shadedjars
5m 17s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 41s
branch-2.4 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 15s
the patch passed

+1 :green_heart:
compile
1m 12s
the patch passed

+1 :green_heart:
javac
1m 12s
the patch passed

+1 :green_heart:
shadedjars
5m  4s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 55s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
217m 37s
hbase-server in the patch passed.

+1 :green_heart:
unit
3m 37s
hbase-endpoint in the patch passed.

247m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4750

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux ff7e0ab6dcae 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / fd27d68b18

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/testReport/

Max. process+thread count
2841 (vs. ulimit of 30000)

modules
C: hbase-server hbase-endpoint U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4750/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


ImageFit.scaleDown is just busted

If you have an Image widget with height: 1.0, width: 1.0, with an image bigger than that, and the default fit (scaleDown), it renders something that is bigger than 1x1.

Looks like it's treating the width and height separately.


Alexa coming from Listening to Idle Quickly

Briefly summarize your issue:
In some scenarios, Alexa dialog state coming from Listening state to Idle state quickly (with in 3 seconds). Following time out prints seen from SDK. What can cause this problem. What indicates STATE_PROVIDER_TIMEDOUT error.
From logs:
[2021-12-14 16:48:55.697] #       ALEXA STATE: LISTENING       #
ContextManager:getContextFailureCallbackLocked:token=16,error=STATE_PROVIDER_TIMEDOUT
[2021-12-14 16:48:57.744] 2021-12-14 08:48:57.497 [  9] E ContextManager:getContextFailureCallbackLocked:pendingStateProviderName=RenderedDocumentState,pendingStateProviderNamespace=Alexa.Presentation.APL
[2021-12-14 16:48:57.854] 2021-12-14 08:48:57.498 [ 14] E AudioInputProcessor:executeOnContextFailure:error=STATE_PROVIDER_TIMEDOUT
[2021-12-14 16:48:58.073] #       ALEXA STATE: IDLE       #
Alexa_listening_to_idle_immediately.txt
What is the expected behavior?
Alexa should wait for User input
What behavior are you observing?
Alexa is not waiting for user input instead coming to IDLE state.
What version of the AVS Device SDK are you using?
AVS SDK - 1.24.0
APL Core - 1.7.1
SS SDK - 2.7.1
Tell us what hardware you're using:

Linux Platform

Tell us about your OS (Type & version):

[ ] Linux

Hi @VictorFrench,
Can you please provide the following additional information:

Logs with the DEBUG9 log mode enabled as such:
modules/Alexa/SampleApp/src/SampleApp -C <path-to-AlexaClientSDKConfig.json-file> -C <path-to-SmartScreenSDKConfig.json-file> -L DEBUG9
Reproducing steps such as the utterances used.
Is this happening for multiple utterances which results in an Alexa.Presentation.APL directive or a specific utterance?

Closing due to inactivity. Please reopen if needed.


Implement autodistill-slime

Search before asking

[X] I have searched the Autodistill issues and found no similar feature requests.

Description
SLiMe is a one-shot segmentation model that uses a method based on Stable Diffusion. The SLiMe research materials show promising performance over SegGPT, another one-shot model available in the Autodistill ecosystem.
We would love to make SLiMe a base model so that people can easily use the model!
To learn about contributing a new base model to Autodistill, check out our base model implementation guide and template.
Use case
One-shot segmentation.
Additional
No response
Are you willing to submit a PR?

[ ] Yes I'd like to help by submitting a PR!

I am closing this issue since there is no active interest in working on this extension. If you want to add SLiMe support to Autodistill, feel free to submit a PR!


[PL] Add some expressions to timers

Way to go GitHub, thanks for informing me in the notification this is once again a bloody draft PR...


Symbols in const assertions should be considered unique

TypeScript Version:  3.8.0-dev.20191205
Search Terms:

const assertion unique symbol
"unique symbol" "as const"

Code
const prop = Symbol()
const dictionary = {
  prop: Symbol()
}
const sealedDictionary = {
  prop: Symbol()
} as const

class MyClass {
  // Works because "prop" is a unique symbol
  [prop]: 1

  // Does not work because "dictionary.prop" is not a unique symbol
  [dictionary.prop]: 2

  // Does not work because "sealedDictionary.prop" is not a unique symbol,
  // even though it probably should be?
  [sealedDictionary.prop]: 3
}

Expected behavior:
MyClass should probably accept sealedDictionary.prop as a computed property name since it should be considered a unique symbol.
Actual behavior:
TypeScript complains with TS1166:

A computed property name in a class property declaration must refer to an expression whose type is a literal type or a 'unique symbol' type.

Playground Link: ~Provided~ Updated (old link seems to be broken in new playground)
Related Issues: —

It seems that symbols lose their "unique" when assigned to an object.
Another example of this behaviour: Playground.

@RyanCavanaugh you marked this as "awaiting more feedback". Can you elaborate on what that means? Is it community feedback or maintainer feedback you are looking for?
I ran into this problem the other and was convinced for a while that it was an issue with typescripts symbol index support, until I was pointed here. It would be really nice if this was fixed.

Is the current behaviour intentional or an oversight? It's frustrating that to create an object of unique symbols you have to do this:
const a = Symbol();
const b = Symbol();
const symbols = { a, b } as const;

It would be nicer if you could declare the symbols directly:
const symbols = {
  a: Symbol(),
  b: Symbol()
} as const

Any updates?
I ran into the same weird behaviour. Also, for my case i dont want the type to have readonly, so im looking for another way of creating the object.

It is very inconvenient to write such code now. I have to do this:
const fooSymbols = {
  a: Symbol(),
  b: Symbol()
} as {
  readonly a: unique symbol
  readonly b: unique symbol
}

But there are indeed some things to consider about this issue, such as [[Getter]]:
const _b = Symbol()
const fooSymbols = {
  get a() {
    return Symbol()
  },
  get b() {
    return _b
  }
} as const

Is it possible to not process the properties corresponding to [[Getter]] but to process the properties of expression?
Processing refers to converting the corresponding symbol into unique symbol by as const.


Draft: Implement writer::Json (#127)

Resolves #127
Synopsis
For now cucumber output can't be parsed by default CI tools.
Solution
Implement writer::Json
Checklist

Created PR:

[x] In draft mode
[x] Name contains Draft:  prefix
[x] Name contains issue reference
[x] Has assignee

[x] Documentation is updated (if required)
[x] Tests are updated (if required)
[x] Changes conform code style
[x] CHANGELOG entry is added (if required)
[ ] FCM (final commit message) is posted

[ ] and approved

[ ] Review is completed and changes are approved
Before merge:

[ ] Milestone is set
[ ] PR's name and description are correct and up-to-date
[ ] Draft:  prefix is removed
[ ] All temporary labels are removed

FCM
Implement outputting JSON schema (#159, #127)

- impl `writer::Json` behind `output-json` Cargo feature


avoid sending the whole scene over the wire frequently

ideally should just send data when it changes, especially now that the data flow is set up from SceneView to Session to notify of changes. remove SCENE? message type and only send SCENE message once on connection

resolved by https://github.com/HeladoDeBrownie/Nexus/commit/f5f021edf42657a1a4916653917b9bab6b8092da


TEMPLE

What does " pending community activity" mean?

How can I get it approved?

Be patient @Kotetene - number of transfers, holders and maybe reputation have to growth a bit and then we are good to go


fix link issue in the contents section in the readme

Description

Fixes link issue in the contents section that was not directed to the desired section in the readme
Emojis were creating the problem so removed them to fix the issue

Closes # 42
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots

Original
Updated

Thank you.


Add induced dipole virial routine

Add a routine for processing induced dipoles, including computation of the virial.  The code introduces a new routine computePVRec that computes the virial as part of the potential calculation, which is achieved by abstraction of the current computePRec routine.

Codecov Report

Merging #48 (01f71da) into master (beecd9d) will increase coverage by 0.75%.
The diff coverage is 97.03%.

:exclamation: Current head 01f71da differs from pull request most recent head 8f1444b. Consider uploading reports for the commit 8f1444b to get more accurate results

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
+ Coverage   95.14%   95.90%   +0.75%
==========================================
  Files          48       52       +4
  Lines        4845     5979    +1134
==========================================
+ Hits         4610     5734    +1124
- Misses        235      245      +10

Impacted Files
Coverage Δ

src/helpme.h
97.82% <95.78%> (+0.36%)
:arrow_up:

...st/unittests/unittest-fullrun-isodipole-virial.cpp
100.00% <100.00%> (ø)

src/fftw_wrapper.h
95.00% <0.00%> (-1.00%)
:arrow_down:

test/unittests/unittest-matrix.cpp
100.00% <0.00%> (ø)

test/unittests/unittest-potential.cpp
100.00% <0.00%> (ø)

test/unittests/unittest-d-field-kappasweep.cpp
100.00% <0.00%> (ø)

test/unittests/unittest-cdq-field-findif.cpp
100.00% <0.00%> (ø)

test/unittests/unittest-dipolereference.cpp
100.00% <0.00%> (ø)

src/matrix.h
97.41% <0.00%> (+0.40%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


docs: add FAQ

Adding some tips to fix issues with access to zwave usb devices.

@macpoupou Did you tried using the /dev/serial/by-id path instead?

Yes but in Debian this path is not available. But yes it's something that we can add also.

I would suggest to create a udev rule that handles that instead, otherwise the soft reset could break that, see https://github.com/zwave-js/zwave-js-ui/discussions/3589
Also instead of adding that FAQ in docs I would prefer you to open a discuttion type Q&A with the quesrtion and the answe marked :)


Reactive Services

Explain how to share data between two viewmodels using services and make them react to the change of those values.

An error occurred while updating the UI with a reactive service in a new implementation such as ListenableServiceMixin. Please check that it works properly.

This code inside view model @override   List<ReactiveServiceMixin> get reactiveServices =>       [keypad, app, productService, settingService]; is deprecated the question I believe was how to migrate to suggestion.

This code inside ReactiveViewModel view model @override List<ReactiveServiceMixin> get reactiveServices => [keypad, app, productService, settingService]; is deprecated the question I believe was how to migrate to suggestion.

List<ListenableServiceMixin> get listenableServices => [_sameService, _otherSameService];

@richard457 let me know if the above works.

Hi @FilledStacks
I'm using ListenableServiceMixin in my new Services, but I can't achieve that ReactiveValues are fired when I set a new value to them.
Service:
`@lazySingleton
class DemoService with ListenableServiceMixin {
final loaderValue = ReactiveValue(false);
DemoService() {
listenToReactiveValues([loaderValue]);
}
}`
ViewModel:
`class DemoViewModel extends ReactiveViewModel {
final _service = locator();
@override
List get listenableServices => [_service];
}`
When 'listenToReactiveValues' evaluates the ReactiveValue, it doesn't applies in no one cases

Let me know if I'm wrong, I'll appreciate it.

Hi @araujocarlosm I'm hoping to remove the ReactiveValues completely from the codebase as it's not useful to us and caused more issues than it solved.
In terms of what you've done, can you provide the type you want to it and see if that helps. ReactiveValue<bool>(false).
We have slowly been moving away from it so it might have broken completely.
What we do is call notifyListeners in the service when we want viewmodels listening to it to rebuild. This way we control exactly when the rebuild happens.

Hi @araujocarlosm I'm hoping to remove the ReactiveValues completely from the codebase as it's not useful to us and caused more issues than it solved.
In terms of what you've done, can you provide the type you want to it and see if that helps. ReactiveValue<bool>(false).
We have slowly been moving away from it so it might have broken completely.
What we do is call notifyListeners in the service when we want viewmodels listening to it to rebuild. This way we control exactly when the rebuild happens.

Thanks for your feedback, doesn't work in my case, but it's ok, I'll use notifyListeners() if needed.

Duplicate of #265 . We'll be adding proper ReactiveService -> ListenableService documentation


Queues - get approximate message count

per the rest api: https://msdn.microsoft.com/en-us/library/azure/dd179384.aspx

@kenegozi want to implement?

PR on it's way :)


Impossible to send Links and Photos as DM

Hi, if i send a link in a Direct Message through instagram it allows the receiver to open the URL by clicking on it. If I use your (otherwise great) app, the link is pasted as plain text and i cant click on it.
sending Photo and videos as DM seems to be broken as well, i select the file but it is not sent.
These are two very important functions for me.
Thanks

The same is happening to me as well, the photos are selected but not actually sent.
Thanks

If I use your (otherwise great) app, the link is pasted as plain text and i cant click on it.

The same is happening to me as well.

An update: PNG format are not supported but the JPG format is working.


Make it apply to everyone, not just Mac users.

The tutorial is universal. How do we know that user is on the Mac?

Thanks, @abdullah2 :)


Dimensional doors incompatibility exploit B13

Dimensional dungeons (naturally generating portals), and limbo both are exploitable, neither of those have aura. So there is no vis not flux, so you can have 10k essentia in a crucible, flooding flux to the air, and it does not affect anything, no flux is actually being generated, and thus, exploitable. I am using the dimensions to jar all my essentia with the essentia smelters, without creating any flux.

I’ll use that to keep the op flux at bay

Funny thing- I can't even go into a dimensional dungeon without crashing, and I partly thinking that it was Thaumcraft 6 XD

Flux is op

duly noted. will now proceed to do messy science in a pocket dimension. :D


Mismatch in last layers during inference

I am running the inference and this is what I am getting.
The command that I ran: python sample.py -m "DiT-XL/2" --text "a person is walking on the street" --ckpt /home/nlp/open_sora/Open-Sora/pretrained_models/DiT-XL-2-256x256.pt --height 256 --width 256 --fps 10 --sec 5 --disable-cfg
I downloaded the checkpoints using download.py in the given repo.

Then can you let me know how to load the weights or where do I download the weights so that I can run the model locally?

We have updated our code and inference. Please see here for more instructions.


assert.style() not computedStyle?

No. No rendering engine, so no computed styles.

I thought that jsdom supported such

Hello
Can you elaborate about what makes that zombie does not offer this through jsdom who does?


swagger-parser treats relative refs without leading "." as internal refs

If the value of a $ref is a relative reference but does not start with ".", the swagger parser treats it as an internal ref.  E.g.
"$ref" : "./path/to/model.json#/thing"

is treated as a relative URI but
"$ref": "path/to/model.json#/thing"

is treated as internal.
I did research the syntax of $ref -- starting from the Swagger 2.0 spec, then to the JSON Schema spec, ending in IETF RFC 3986 Uniform Resource Identifier (URI): Generic Syntax -- and as best I can tell, a relative reference does not need to start with "." to be considered a relative reference.

Sorry ... this is really a problem in swagger-core ... closing this bug and will reopen in swagger-core.

For the record here's the reference to the swagger-core issue: swagger-api/swagger-core#2138


Can Just Support proxying HTTP requests?

In other languages this is basic functionality, but it seems its a bit more difficult to pull of in Swift/Objective C. It would be cool if Just could add this feature.

+1, we requesting same feature


Compilation failure due to missing name should report the user readable name

The call instructions currently only carry the procedure id, and not its name. This makes the error messages very cryptic, for situations like the following:
proc.example
    syscall.wrong_name
end

The error message will be something like:
KernelProcNotFound(ProcedureId([120, 199, 130, 12, 225, 34, 213, 30, 69, 109, 206, 77, 106, 134, 11, 37, 238, 43, 225, 0]))

It should give the wrong_name instead.

This is actually one of the things I've fixed haha

@bitwalker great to know :) will save me a lot of time in the future. Thanks!

@bitwalker one thing that I forgot. Can we also get the original span? As in, original source file, lineno, and col?


Fix scrolling issue when loading more events

Description :sparkles:
When clicking load more button in the search page, it causes page to scroll to top and then right back to load more announcer. Also same request is done twice to API when fetching more event with load more button, I think it has something to do with router.replace and useQuerys fetchPolicy settings. To fix this issue, I added possibility to add TransitionOptions to setFilters function.

Added possibility to pass TransitionOptions to setFilters function
Deleted fetch policies from useUnifiedSearchQuery apollo hook

Issues :bug:
Closes :no_good_woman:
Related :handshake:
Testing :alembic:
Click "Näytä lisää sijainteja" button on search page and page should be scrolled to top and then back down.
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #72 (cac70ee) into main (e9ae342) will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   70.57%   70.57%
=======================================
  Files          72       72
  Lines        1040     1040
  Branches      264      264
=======================================
  Hits          734      734
  Misses        305      305
  Partials        1        1

Impacted Files
Coverage Δ

src/domain/unifiedSearch/useUnifiedSearch.ts
88.37% <33.33%> (ø)

Continue to review full report at Codecov.

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


Create cnvranceanu.txt

Please add my school, "Gheorghe Vranceanu" national college. Thank you a lot!

@stefanalexandru02 Pull request merged. Thank you!


EIO ,i/o error

I am getting this error and tried everything to fix but no success please resolve the issue
pi@raspberrypi:~ $ cd waterpi-node
pi@raspberrypi:~/waterpi-node $ sudo node index.js
1462815306876 Device(s) RaspberryPi-IO
1462815307277 Connected /dev/ttyACM0
1462815307365 Connected RaspberryPi-IO
connect
1462815311112 Board ID:  pi
1462815311114 Board ID:  uno
1462815311129 Repl Initialized

ready
Temperature: 33.00C, humidity: 45.00%
fs.js:613
return binding.writeBuffer(fd, buffer, offset, length, position);
^
Error: EIO, i/o error
at Error (native)
at Object.fs.writeSync (fs.js:613:20)
at Bus.i2cWriteSync (/home/pi/waterpi-node/node_modules/raspi-io/node_modules/raspi-i2c/node_modules/i2c-bus/i2c-bus.js:340:13)
at I2C.writeSync (/home/pi/waterpi-node/node_modules/raspi-io/node_modules/raspi-i2c/lib/index.js:332:34)
at Raspi.i2cWrite (/home/pi/waterpi-node/node_modules/raspi-io/lib/index.js:559:19)
at Expander.write (/home/pi/waterpi-node/node_modules/johnny-five/lib/lcd.js:49:11)
at LCD.Controllers.PCF8574.initialize.value (/home/pi/waterpi-node/node_modules/johnny-five/lib/lcd.js:341:23)
at new LCD (/home/pi/waterpi-node/node_modules/johnny-five/lib/lcd.js:734:10)
at Boards. (/home/pi/waterpi-node/index.js:60:9)
at Boards.emit (events.js:107:17)
at Boards. (/home/pi/waterpi-node/node_modules/johnny-five/lib/board.js:1197:14)
at process._tickDomainCallback (node.js:381:11)

Closing in favor of https://github.com/rwaldron/johnny-five/issues/1128, let's discuss it there for now to keep the conversation focused.


fix a little issue about  MASAttachKeys(...)

if we have many view in a method, so we want to use MASAttachKeys(...)  many times.
for example:

(void)initSubviews
{
UIView *view1 = [UIView new];
UIView *view2 = [UIView new];
UIView *view3 = [UIView new];
MASAttachKeys(view1,view2,view3);
UIView *view4 = [UIView new];
UIView *view5 = [UIView new];
UIView *view6 = [UIView new];
MASAttachKeys(view4,view5,view6);
}
than there is a error: Redefinition of 'keyPairs'
I fix it by add a "{ }"

@lianchengjiang Thanks!


perf: Use Controller As

Type: perf
Platform: all
Could you please remove all $scope.* and use this.* and prototype.* with controller as method
If you want/approve i can create a pull request
Thank you in advance.

Ionic v1.x will continue to use $scope for now using controller as is a micro-optimization.


Add restaurant settings page

As a user, I want to enter and edit my restaurant information.

Users should manage their subscription from the settings page.

Create new end point for updating information

Let's have the menu broken up into sections similar to this design: https://dribbble.com/shots/15186840-Setting-page-example.
You can see an example of how to select between different components to render here: https://github.com/amorriscode/tavern/blob/main/web/src/components/AuthModal/AuthModal.js.


Fix pdf_viewer definitions

Follow up to https://github.com/mozilla/pdf.js/pull/13588
Current pdf_viewer definitions result in errors like the following when trying to use them in a ts project:
[error] TypeScript error node_modules/.pnpm/pdfjs-dist@2.10.377/node_modules/pdfjs-dist/web/pdf_viewer.d.ts:1:15 - error TS2691: An import path cannot end with a '.d.ts' extension. Consider importing 'pdfjs-dist/types/web/pdf_viewer.component.js' instead.

1 export * from "pdfjs-dist/types/web/pdf_viewer.component.d.ts";

Import/export statements in typescript should not include file extensions.

Thanks for the review! I've amended the commit message.


Confused about a place in the paper

In section 6. Experiment, there is the following paragraph: "Watermark Strength vs Number of Tokens. Theory predicts that the type I and type II error rates of the watermark should decay to zero as the sequence length T increases." I am confused about how to theoretically prove that as T increases, both type I and type II errors approach 0. I am eagerly anticipating response.

Hi, and thanks for your interest in the work! I saw your question on twitter but we can have this discussion here to avoid the character limits etc.
To show that Type I errors, False Positives, approach 0 as T gets large, we need to argue that the likelihood tends to 1 that unwatermarked samples (i.e. human written text) will yield z-scores corresponding to the null hypothesis, and that are less than the z-score chosen as the rejection threshold. We rely on the assumption that the PRNG used to generate the green/red partition is unbiased, and with sufficient samples, the ratio of green tokens in any piece of text will approach gamma. It's worth noting that a correction for multiple tests must be applied, or the --no-repeat-ngrams option must be used to make sure that the PRNG values and therefore green/red partitions are properly independent for this asymptotic behavior to be observed empirically.
For Type 2 errors, False Negatives, to approach 0 at large T, we need to argue that the likelihood that watermarked samples yield z-scores that are above our rejection threshold, also approaches 1. We rely on the results of Thm 4.2 this time and assume that there is sufficient entropy in the text to ensure that the expected number of green tokens in a watermarked sequence is above the rejection threshold. Looking at the expectation $\mathbb{E}|s|_G$ in Thm4.2, if delta is greater an 0, then alpha is > 1, and since gamma is between 0 and 1, then the whole coefficient that T and S* are multiplied by is greater than 0. So there exists some T large enough to make the expected number of green tokens is sufficiently large to reject the null hypothesis. (we can make a similar argument about the expected ratio $\frac{|s|_G}{T}$ but it's not the quantity that the expectation is on in the paper)
However, we also would want to confirm that the variance approaches 0, so that we are confident in the outcome of the expectation, and for this it is better to consider the ratio of green tokens.  Multiplying the expression we provided for $Var |s|_G$ by $\frac{1}{T^2}$ on both sides, we see that for large T, the bound on $Var \frac{|s|_G}{T}$ tightens towards 0.
So, overall, this is o course only a probabilistic argument about the asymptotics, and not a guarantee of a zero error rates. If more formalism is needed, maybe we can circle up. I hope this helps!

Hi! You're amazing! Thank you so much for your thoughtful response. I apologize for the delay in replying lately due to other commitments, but this won't happen again in the future. I find your reasoning and mathematical assumptions to be very rigorous. I have some additional questions that I'd like to discuss with you:

For the Type I error, the mathematical assumptions you made earlier are essential. However, I've been thinking that for a text without a watermark, it can be perfectly modeled by the binomial distribution, which in turn approximates the normal distribution. From this perspective, the Type I error seems to be related only to the threshold z-score set and has nothing to do with the text length T. But if you look at it from another angle, if the average z-score of the watermarked text increases with T, then we can set the threshold very high, making the Type I error approach 0.

For the Type II error, your discussion has given me some insights.

Additionally, I'd like to inquire about considerations in low entropy situations, as it's evident that it greatly aids in improving the detection rate and the quality of the text. From your experiments, it can be seen that the Type II error is more likely to occur in low entropy situations. However, I still haven't thoroughly grasped the mathematical justification. While Thm4.2 is very useful in proving the lower bound, it can't prove the upper bound. Do you have any mathematical or theoretical insights or intuitions about this?

Have you ever tried to mathematically derive the distribution type of ( |s|_G ) after adding δ (like Gaussian distribution), or experimentally considered it to conform to a certain distribution with a certain confidence level?

Your work is truly impressive！ I look forward to our discussion.


reload angular gantt chart on ajax

I am using angular gantt chart https://www.angular-gantt.com/demo/ and I need to reload it on ajax success response. Is there any event on which i can pass the start date and end date and it reloads the gantt chart taking data again from the database and then shows it.
Thanks in advance.

You only have to code it, read the docs and you will be OK. I can't see any issue here.


Critical Injury itemtype does not render correctly

In Chrome, when creating a criticalinjury the severity dropdown renders behind the description/modifiers area.

This makes it tricky to select anything other than Simple.

Now fixed in the dev branch.