Possible migrate to Windows App SDK?

As described in the title, do you plan to move to the Windows App SDK?
I forked the repo and have some interest in doing so. But this might change the minimum Windows version, etc. And WinAppSDK has limited support of certain features too.

At the moment, absolutely not. The performance and stability of the App SDK are notably than UWP XAML at the moment so there is no advantage of doing so - only a worse experience for the end user - and it doesnt look like this is going to change for a long, long while.
The only thing that might makemit convincing is if and when they eventually opensource then tire AppSDK XAML platform, at which point we could potentially improve the XAML layer TextBlock to properly render variable fonts & advanced typography etc without directly interfacing with DirectWrite, but even though that was an original promise of the project reunion that's starting to look unlikely, for the near term future at least anyway

At the moment, absolutely not. The performance and stability of the App SDK are notably than UWP XAML at the moment so there is no advantage of doing so - only a worse experience for the end user - and it doesnt look like this is going to change for a long, long while.
The only thing that might makemit convincing is if and when they eventually opensource then tire AppSDK XAML platform, at which point we could potentially improve the XAML layer TextBlock to properly render variable fonts & advanced typography etc without directly interfacing with DirectWrite, but even though that was an original promise of the project reunion that's starting to look unlikely, for the near term future at least anyway

Solid point. I guess we'll have to wait and see if everything works in the future.


Parse String + Format issue

The following date parse results in a wrong date:
testDate = moment('20151114240833','YYYYMMDDHHmmss',true);
//Results is: Nov, 15, 2015 00:08 am   (should be Nov 14)
It looks like the parser adds 1 day to the date when the hour value is 24.

Um - hours has range 0..23.  The extra hour is causing the date to roll over.
Try using a value of '20151114230833', and the date won't roll over.

Thanks for the reply. I understand but the issue is most other date libraries consider the midnight hour as 24 instead of 0. My data is coming from Apple iOS date function and Android (Java.Util.Date) and both use 24 as midnight hour. I can wrap the parse function in my own script to check this every time in and update the string if the hour is 24 but considering different format it would be almost re-writing the parse function. I believe this should be handled in the library in the parse function.

'k' is used to display the number of the hour of the day (1..24), as in 1st hour of the day 000000..005959 (using your format HHmmss), 2nd hour 010000..015959, ..., 24th hour 230000..235959".
With each second of the day being represented in the HHmmss sequence 000000..235959, which of these seconds in the day should 240833 match?
moment will wrap around lower values (hour in your case), increment higher values (day in your case).
So moments takes 240833 and returns a time of 000833 plus one day.
I think you need to go back and review why you are using 1..24 as the hour values rather than the standard 0..23.

The current functionality is working correctly in that we can parse a 24, and it correctly identifies it as midnight at the end of the day, which is the same moment in time as midnight at the beginning of the next day.
However, you are correct that we do not currently have a formatting token to emit hours 01-24.  Adding one would be tricky, as it would affect the output of all the other tokens as well.  You'd basically have to subtract a day if and only if that token was present and the corresponding hour was midnight.  It's doable, but I'm not sure how useful it would be.
There's certainly no chance of us changing HH such that it used 24 instead of 00 just because an input string also contained 24.  You'd have to explicitly use the new token (kk or whatever).

Close in favor of #2762


redeploy does not respect "run all containers on specific host"

When doing redeploy cowbell (and i assume rancher-compose) does not respect "run all containers on specific host" scheduler value.
I assume it happens because exported rancher compose config does't have a label for this kind of scheduler  restriction.

Just encountered this for myself. It reverts back to "pick from the following rules"

I've worked around it by setting hostname label to all hosts and restricting by hostname. It's much better because if say host gitlab-instance goes down i can set this label to any new or existing host and scheduler will do the works.

That'd be useful if we didn't have like 80 different services running in our stack, all with different hostnames. :P

There is now a rancher cli, i've been using it for exec and logs but it should also be able to update services. Did not test it myself thou, don't know its limits - https://github.com/rancher/cli


Invalid HIR state after editing a document

All the bugs that could crash the lsp were due to invalid symbol removals from the hir so far. I think I could track most of these down, at least I don't observe crashes no matter what I throw at it, should be safe to close this.


Reproducing Pong Training Curve using Monobeast

Hi,
I tried running monobeast.py on different environments, including LunarLander-v2 and PongNoFrameskip-v4 but the model doesn't learn anything. I tried the hyperparameters for Pong that is written in the ReadMe.md but still nothing. The mean expected return does not move. I checked the gradients they are non-zero and the weights are being updated. However, during testing, I noticed that the agent always chooses the same action.
I am also getting A LOT of NaNs in the log file for the mean_expected_return, is this normal?
Any help would be appreciated, thanks!

Hey Badr,
Thanks for your interest in TorchBeast.
I have to acknowledge we never validated monobeast against the Atari suite. It cannot (easily) be run with the right batch size so it's tricky to use the known-good hyperparameters. I believe its design is sound in principle (CPU actors, one GPU learner, PyTorch shared memory tensors for inter-process communication) but there's lots of things that can go wrong with RL.
Re: The NaNs: I don't know what mean_expected_return is, I'm guessing you're seeing mean_episode_return? If so, we made the (perhaps questionable) decision to log NaN if there is no episode ending in the batch in the first place. This happens "naturally" via division by zero (the "mean" over no episodes). This implementation logs per-rollout numbers on the learner side and not all rollouts contain finished episodes. Since we also want to log speed, frame counts etc we felt we had to log something for mean_episode_return so we decided for the somewhat "natural" NaN there. Many plotting libraries ignore these values, which is what you'd want to do at that point. This approach has the obvious downside of scaring researchers, who are typically trained to associate NaNs with "something went very wrong".

I see, and yes I apologize for the typo, I indeed meant mean_episode_return.
Thank you for your prompt response!


ThinkVim installation bug.

Problems summary
Hi there!
When I tried to install ThinkVim configuration files, I runned install script, and gave me this message:

And when I'm trying open NeoVim, it gave me this message:

I'll be very glad if you can tell me how to fix it! Thank you very much!
Environment Information

OS: Manjaro

thinkvim :echo g:thinkvim_version output:2.6.1

neovim :version output:NVIM v0.4.3

:checkhealth or :CheckHealth result(neovim only):

How to reproduce the problem from neovim startup (Required!)

foo
bar
baz

Screenshot (if possible)

hmm 🤔. Try renaming/moving .thinkvim.d. Afterwards run make on nvim root dir.
This issue happens usually when one of the yaml files has syntax errors in them.
Hope this help you debug and find the root cause.

hmm . Try renaming/moving .thinkvim.d. Afterwards run make on nvim root dir.
This issue happens usually when one of the yaml files has syntax errors in them.
Hope this help you debug and find the root cause.

I've tried to do that but it didn't work. I think system may not read the ~/.config/nvim/core/plugins.yaml correctly, but I've exactly installed pyyaml. Here is the message which dein's log shows:
Target plugins are not found.
You may have used the wrong plugin name, or all of the plugins are already installed.
Runtimepath updated: (2020/05/18 16:45:22)

@ChunjieShan  Please update the pynvim .then remove the .thinvim.d folder and ~/.cache/vim to reinstall

try this scripts
rm -rf ~/.thinkvim.d
rm -rf ~/.cache/vim
rm -rf ~/.config/nvim
git clone --depth = 1 https://github.com/hardcoreplayers/ThinkVim.git ~/.config/nvim
cd .config/nvim
sh scripts/install.sh

note the theme and languages will generate by a excutable file in bin folder.

Oh gosh it finally worked! Thank you for your help!

Thanks for your no love. no thanks for help. 😃


VSCode-linux-x64-stable.zip doesn't run as root

VSCode Version:
latest stable Linux 64: VSCode-linux-x64-stable.zip
downloaded from:

https://code.visualstudio.com/Docs/?dv=linux64
https://go.microsoft.com/fwlink/?LinkID=620884

OS Version:
Ubuntu 14.04

Steps to Reproduce:
curl -o VSCode-linux-x64-stable.zip -L https://go.microsoft.com/fwlink/?LinkID=620884
unzip -q VSCode-linux-x64-stable.zip
export PATH=$PWD/VSCode-linux-x64:$PATH
code --verbose

root@937050988adf:~# code --verbose
{ [Error: ENOENT: no such file or directory, open '/root/.config/Code/storage.json']
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/root/.config/Code/storage.json' }
TypeError: Bad argument
    at TypeError (native)
    at ChildProcess.spawn (internal/child_process.js:274:26)
    at Object.exports.spawn (child_process.js:351:9)
    at /root/VSCode-linux-x64/resources/app/out/vs/workbench/electron-main/main.js:6:6784
    at new __dirname.undefined.t.Class.derive._oncancel (/root/VSCode-linux-x64/resources/app/out/vs/workbench/electron-main/main.js:5:6601)
    at Object.i [as getUserEnvironment] (/root/VSCode-linux-x64/resources/app/out/vs/workbench/electron-main/main.js:6:6747)
    at Object.<anonymous> (/root/VSCode-linux-x64/resources/app/out/vs/workbench/electron-main/main.js:9:10764)
    at e._invokeFactory (/root/VSCode-linux-x64/resources/app/out/vs/loader.js:4:13697)
    at e._complete (/root/VSCode-linux-x64/resources/app/out/vs/loader.js:4:14014)
    at e.resolveDependency (/root/VSCode-linux-x64/resources/app/out/vs/loader.js:4:15068)

@ctaggart haven't seen this before. I'm curious who owns files/folders, what the permissions are on them and whether they actually exist. Can you pass along the output of these commands?
whoami
ls -la ~ | grep .config
ls -la ~/.config | grep Code
ls -la ~/.config/Code

Here is the output:
root@937050988adf:~# whoami
root
root@937050988adf:~# ls -la ~ | grep .config
drwx------  3 root root  4096 Apr 22 22:37 .config
root@937050988adf:~# ls -la ~/.config | grep Code
drwx------  3 root root 4096 Apr 22 22:51 Code
root@937050988adf:~# ls -la ~/.config/Code
total 16
drwx------ 3 root root 4096 Apr 22 22:51 .
drwx------ 3 root root 4096 Apr 22 22:37 ..
-rw-r--r-- 1 root root    2 Apr 22 22:58 DevTools Extensions
drwxr-xr-x 2 root root 4096 Apr 22 22:37 User

Weird, I would expect it to work provided the user has rw access to ~/.config. It might be an issue with Electron. The only things I can recommend are to try with a non-root user or installing via a package (though the latter probably won't make a difference).

It appears to be an issue when running as root. The GUI came up when I ran as a user.

@Tyriar In the April Iteration Plan I noticed:

Linux: Support running vscode with sudo permissions #3068 @tyriar
Test #5062

I was hoping that this would allow me to launch VSCode as root. I downloaded the insiders build and tried launching it, but it doesn't launch and I don't get any output:
root@1f20642fd434:~# code-insiders --user-data-dir=/root/profile --version
1.1.0-insider
root@1f20642fd434:~# code-insiders --user-data-dir=/root/profile .
root@1f20642fd434:~# pwd
/root

I noticed that a User subdirectory is created:
root@1f20642fd434:~# ls -al ./profile/User/
total 85
drwxr-xr-x 2 root root  68 May  7 19:48 .
drwxr-xr-x 3 root root 102 May  7 19:48 ..

Should it be possible to launch VSCode as root?

I am experiencing a similar issue, just the opposite...I can run:

su root -> /opt/vsc/code
sudo /opt/svc/code
sudo -u non-root /opt/svc/code

but I cannot simply run from any user,
TypeError: Bad argument
    at TypeError (native)
    at ChildProcess.spawn (internal/child_process.js:278:26)
    at Object.exports.spawn (child_process.js:379:9)
    at /opt/visual-studio-code/resources/app/out/vs/workbench/electron-main/main.js:6:4539
    at new __dirname.undefined.t.Class.derive._oncancel (/opt/visual-studio-code/resources/app/out/vs/workbench/electron-main/main.js:5:9667)
    at Object.o [as getUnixUserEnvironment] (/opt/visual-studio-code/resources/app/out/vs/workbench/electron-main/main.js:6:4502)
    at I (/opt/visual-studio-code/resources/app/out/vs/workbench/electron-main/main.js:10:15570)
    at Object.<anonymous> (/opt/visual-studio-code/resources/app/out/vs/workbench/electron-main/main.js:10:16109)
    at e._invokeFactory (/opt/visual-studio-code/resources/app/out/vs/loader.js:4:13697)
    at e._complete (/opt/visual-studio-code/resources/app/out/vs/loader.js:4:14014)

...but as I said, If I use sudo and specify a non-root user it works.

I'm curious on what you guys think the best way to handle this is. I spend most of my time in Ubuntu so running as root is not usually a thing I'm allowed to do (su is locked down on Ubuntu). The recent change was specifically targeting sudo only as previously it hung when launched.
Here's the current situation:

The code bin command enforces a --user-data-dir when id -u is 0, see https://github.com/Microsoft/vscode/blob/master/resources/linux/bin/code.sh#L9 - This was done because when running as sudo, the current user data directory is stored within ~/.config which has restrictive permissions:
❯ ls -la ~ | grep .config
drwx------  33 daimms daimms  4096 May 16 11:35 .config

The need for this will go away post-https://github.com/Microsoft/vscode/pull/5570 where the --user-data-dir is specified automatically as a sibling to the regular user's data dir.

When running as root, do not attempt to open the welcome page. This is due to a bug in xdg-open, see https://github.com/Microsoft/vscode/commit/ad7e861dbb9127e919bc686da162b19cbb7e2391 (this may be fixed now https://github.com/electron/electron/issues/4916, created https://github.com/Microsoft/vscode/issues/6594 to follow up).

When in a sudo session of vscode, you can only open root-owned files. I'm sure you can get around this, I didn't invest enough time in this though since people should not be using sudo in general as it will mess up file perms which may be confusing to newer users.

It's a very bare-bones support situation at the moment, plus it focuses around running as sudo instead of root, which I don't think id -u differentiates between. So I believe the fixes I did may have made the situation worse when running as the root user instead of sudo on a non-root user.
Any input from the community would be greatly appreciated :smiley:

I created this issue originally when making some docker images with VSCode on it. My solution was simply to create a vscode user that has sudo access. I'm happy with that solution, so I'm going to close this issue.
https://github.com/ctaggart/dotnet-vscode/blob/master/root/install-user.sh

Going to reopen, I'd eventually like to get it working as expected for root user, sudo and regular users.

Closing since there are no plans to fix this any time soon and there is a workaround in https://github.com/Microsoft/vscode/issues/5679#issuecomment-221022927


feat(#65): mvp

adds all motives of active lints to the Github pages branch

I understand that there are many errors here, but I wanted to do something so that you could specifically point them out to me

@Marat-Tim looks like a good start! Now, please, try to make all build jobs pass.


registry: gossip unify registry option passing, optimize

Signed-off-by: Vasiliy Tolstov<EMAIL_ADDRESS>

related to #404

now with enabling compression in memberlist each service transfer about 100-150 bytes.

Nice PR


Allow omitting ‘static in const and static references

Why are 'static lifetimes required to be written explicitly in const and static references?
const MY_STRING: &'static str = "Hello world!";
const MY_ARRAY_REF: &'static [int, ..1000] = &[0, ..1000];

If I understand the motivation correctly, the types should be written there explicitly for the similar reasons as for function declarations - they represent interfaces and shouldn't change silently after changes to initializer or function body.
But the lifetimes for const/static references are always 'static and will not change, can't they be elided?
These explicit 'statics were relatively annoying before, with constant strings, but now they are even more annoying, because after introduction of const (#398) constant arrays have to be written through references.
const MY_ARRAY1: [int, ..1000] = [0, ..1000]; // Not OK, rvalue arrays are bad
static MY_ARRAY2: [int, ..1000] = [0, ..1000]; // Not OK, `static` arrays can't be used in constant expressions
const MY_ARRAY3: &'static [int, ..1000] = &[0, ..1000]; // OK, but more verbose than necessary

The same snippets after the change:
// The lifetimes are inferred
const MY_STRING: &str = "Hello world!";
const MY_ARRAY: &[int, ..1000] = &[0, ..1000];

The question was asked on discuss forum, but without success.

Closing in favor of https://github.com/rust-lang/rfcs/pull/1623

I knew I wasn't the first to suggest this, but somehow your issue elided my search... :+1:


test(install): use TCP connection instead of thread sleep

What does this PR try to resolve?
The test install::uninstall_running_binary is flaky in CI job.
running `/Users/runner/work/cargo/cargo/target/debug/cargo install foo`
running `/Users/runner/work/cargo/cargo/target/debug/cargo uninstall foo`
thread '<unnamed>' panicked at tests/testsuite/install.rs:2568:66:
called `Result::unwrap()` on an `Err` value: process didn't exit successfully: `/Users/runner/work/cargo/cargo/target/tmp/cit/t1651/home/.cargo/bin/foo` (signal: 9, SIGKILL: kill)
thread 'install::uninstall_running_binary' panicked at tests/testsuite/install.rs:2588:18:
called `Result::unwrap()` on an `Err` value: Any { .. }

TCP connection is more robust than thread::sleep, ensuring

foo is running before the first cargo uninstall call
foo is stopped before the second cargo uninstall call

How should we test and review this PR?
Additional information

This may block cargo submodule sync in rust-lang/rust.

@bors r+

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 58fb23140972092a12f7011d17a7db1d99e3eacf to master...


odd formatting on description of @params argument

the two * without a space between them, each to make parameter_name and data_type emphasised were just displaying as two asterisks on docs.ms

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


Status Enhancement Step 1

First batch of Status Enhancement.

Suitable changes already cherry-picked into branch 3.0.x.


rover: init at 0.1.10

Motivation for this change
Add the Apollo Studio graphql CLI named rover
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Many thanks for the reviews @edude03 @SuperSandro2000, addressed, PTAL

Not sure why hydra is reporting a different cargoSha256, can't reproduce that locally :thinking:

#164986 was merged yesterday


Return value back to flutter from python

Is there an example where the python script return back to flutter to be displayed in flutter app.? Not sure how to get the output of the python script back into flutter any help would be great.

Check out this repo:
https://github.com/amahta/clipboard_ocr


Allow custom vite configuration & plugins

Which @angular/* package(s) are relevant/related to the feature request?
No response
Description
The Vite ecosystem is awesome. It is not only a fast dev tool, but it also provides a rich ecosystem of build tools that can be used across different frameworks.
Proposed solution
It should be possible to

Modify default Vite Configuration
Use the rich Vite Ecosystem

Alternatives considered

Exposing vite configuration does not align with the Angular CLI design goals.
To add additional Vite plugins please use a custom builder.

I created a feature request in the angular-builders repo: https://github.com/just-jeb/angular-builders/issues/1617
Just like with custom webpack configs, where angular-builders was the solution, we would ideally have the same for vite.


[FEATURE] Release v. Next

Can the latest release (3.0) please be published to rubygems.org? The latest available release on rubygems.org is 2.1, which is two releases behind (also missing 2.2). Thank you!

We're working on on releasing opensearch-ruby 3.0.0. We've been dealing with on some CICD issues.
And 2.2.0 you're referring to was actually for opensearch-api https://rubygems.org/gems/opensearch-api/versions/2.2.0

We're facing with an issue from rubygems.org. We've opened a ticket with them and will keep you posted.

Waiting on this one https://github.com/opensearch-project/opensearch-ruby/pull/199

https://github.com/opensearch-project/opensearch-ruby/releases/tag/3.0.0 has been released: https://rubygems.org/gems/opensearch-ruby/versions/3.0.0


perl-Alien-wxWidgets fails check phase

#   Failed test 'use Alien::wxWidgets;'
#   at t/01_load.t line 6.
#     Tried to use 'Alien::wxWidgets'.
#     Error:  Can't locate Module/Pluggable.pm in @INC (you may need to install the Module::Pluggable module) (@INC contains: /nix/store/j1sism3lmxjkpkxrl4cx378fpcfjf4d9-perl-Alien-wxWidgets-0.65/lib/perl5/site_perl /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1/x86_64-linux-thread-multi /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1 /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl /tmp/nix-build-perl-Alien-wxWidgets-0.65.drv-0/Alien-wxWidgets-0.65/blib/lib /tmp/nix-build-perl-Alien-wxWidgets-0.65.drv-0/Alien-wxWidgets-0.65/blib/arch lib inc /tmp/nix-build-perl-Alien-wxWidgets-0.65.drv-0/Alien-wxWidgets-0.65/_build/lib /nix/store/j1sism3lmxjkpkxrl4cx378fpcfjf4d9-perl-Alien-wxWidgets-0.65/lib/perl5/site_perl /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1/x86_64-linux-thread-multi /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1 /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl _build/lib /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1/x86_64-linux-thread-multi /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1 /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl /nix/store/j1sism3lmxjkpkxrl4cx378fpcfjf4d9-perl-Alien-wxWidgets-0.65/lib/perl5/site_perl /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1/x86_64-linux-thread-multi /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/site_perl/5.20.1 /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/5.20.1/x86_64-linux-thread-multi /nix/store/g9ybksy400pfn7fncw8dqfnz6m7fdyrk-perl-5.20.1/lib/perl5/5.20.1 .) at /tmp/nix-build-perl-Alien-wxWidgets-0.65.drv-0/Alien-wxWidgets-0.65/blib/lib/Alien/wxWidgets.pm line 43.
# BEGIN failed--compilation aborted at /tmp/nix-build-perl-Alien-wxWidgets-0.65.drv-0/Alien-wxWidgets-0.65/blib/lib/Alien/wxWidgets.pm line 43.
# Compilation failed in require at t/01_load.t line 6.
# BEGIN failed--compilation aborted at t/01_load.t line 6.
# Looks like you failed 1 test of 1.
t/01_load.t ..........
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests
t/zy_pod_coverage.t .. skipped: Test::Pod::Coverage 1.00 required for testing POD coverage
t/zz_pod.t ........... skipped: Test::Pod 1.00 required for testing POD

Test Summary Report
-------------------
t/01_load.t        (Wstat: 256 Tests: 1 Failed: 1)
  Failed test:  1
  Non-zero exit status: 1
Files=3, Tests=1,  0 wallclock secs ( 0.03 usr  0.01 sys +  0.09 cusr  0.01 csys =  0.14 CPU)
Result: FAIL
Failed 1/3 test programs. 1/1 subtests failed.
make: *** [test] Error 255
builder for ‘/nix/store/cvlgaj0x2802kjj0vcbl8gaxh5dn7k8n-perl-Alien-wxWidgets-0.65.drv’ failed with exit code 2
building path(s) ‘/nix/store/kqg81ksi4dbv73l5sy4vnff74qba31vs-perl-ExtUtils-Typemap-1.00’
cannot build derivation ‘/nix/store/5lkrskvc648zqqjj0ha5r2n8p9bkswk1-perl-Wx-0.9923.drv’: 1 dependencies couldn't be built
building path(s) ‘/nix/store/p83lwjnwngd8s4qm632j6f2n6mkxhdrc-perl-XML-Twig-3.44’
cannot build derivation ‘/nix/store/3j88vdjpcb5jwv2yfv2gs13jiccyl9kg-slic3r-1.2.1.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/xj0gwazrp0rn8vasa1h960i0lrj0nhhp-system-path.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/iisx3lw8f9ji44wiqdxy12fsnkp6xflm-nixos-14.12.555.99bfc89.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/iisx3lw8f9ji44wiqdxy12fsnkp6xflm-nixos-14.12.555.99bfc89.drv’ failed

[renamed]... causing slic3r dependency failure.

Is there a simple way to work around this, since it’s only the check phase?
Sadly, I don’t know any Perl and am to fresh into nix to know where to look. It’s hard to see what is nix output and what is perl output, too.

Certainly, disabling checks for this package should work around it, at least for this particular package (not certain that slic3r will build right after this).

Note: slic3r contains this comment on master

broken = true; # requires Perl 5.14

/cc maintainers @the-kenny, @bjornfor.

Another thing, perl-Alien-wxWidgets tests succeed for me on x86_64-linux. Could it be that you've run it as a non-chrooted build? (Or maybe it's just non-determinitsic.)

Maybe updating slic3r will remove the need for perl-5.14 (which exists no more in nixpkgs master): https://github.com/alexrj/Slic3r/blob/master/.travis.yml (lists perl 5.20 as some of the environments).

Oh, I’m sorry, I was not on nixos-unstable.

perl-Alien-wxWidgets and slic3r are now fixed in master and release-14.12 branches.
@Profpatsch: The error you pasted comes from the fact that newer perl removed the ModulePluggable module. So now it has to be an explicit dependency.


pkgConfig obsolete? Build error on OSX

When trying to import the package, I stumbled across the following issue (for Linux no problem):
When I try to build on MacOSX (all OpenSSL headers found etc., so no problem there) I'm getting the error:
Assertion failed: ((ID || !Mod) && "asked for module ID for non-local, non-imported module"), function getSubmoduleID, file /Users/buildnode/jenkins/workspace/oss-swift-package-osx/llvm/tools/clang/lib/Serialization/ASTWriter.cpp, line 2377.
0  swift                    0x00000001106a054b llvm::sys::PrintStackTrace(llvm::raw_ostream&) + 43
1  swift                    0x000000011069f7d6 llvm::sys::RunSignalHandlers() + 70
2  swift                    0x00000001106a0c1f SignalHandler(int) + 383
3  libsystem_platform.dylib 0x00007fff94c0d52a _sigtramp + 26
4  swift                    0x00000001112323f0 clang::vfs::getNextVirtualUniqueID()::UID + 71992
5  swift                    0x00000001106a09f6 abort + 22
6  swift                    0x00000001106a09d1 __assert_rtn + 81
7  swift                    0x000000010eb0d402 clang::ASTWriter::WritePreprocessor(clang::Preprocessor const&, bool) + 6210
8  swift                    0x000000010eb24150 clang::ASTWriter::WriteASTCore(clang::Sema&, llvm::StringRef, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, clang::Module*) + 9168
9  swift                    0x000000010eb21d32 clang::ASTWriter::WriteAST(clang::Sema&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, clang::Module*, llvm::StringRef, bool) + 642
10 swift                    0x000000010eb5e700 clang::PCHGenerator::HandleTranslationUnit(clang::ASTContext&) + 112
11 swift                    0x000000010e9895bc clang::MultiplexConsumer::HandleTranslationUnit(clang::ASTContext&) + 44
12 swift                    0x000000010eb6bca5 clang::ParseAST(clang::Sema&, bool, bool) + 581
13 swift                    0x000000010e9547a9 clang::FrontendAction::Execute() + 73
14 swift                    0x000000010e921361 clang::CompilerInstance::ExecuteAction(clang::FrontendAction&) + 1073
15 swift                    0x000000011063296b llvm::CrashRecoveryContext::RunSafely(llvm::function_ref<void ()>) + 283
16 swift                    0x0000000110632b30 RunSafelyOnThread_Dispatch(void*) + 48
17 swift                    0x00000001106a1c6d ExecuteOnThread_Dispatch(void*) + 13
18 libsystem_pthread.dylib  0x00007fff8bc3999d _pthread_body + 131
19 libsystem_pthread.dylib  0x00007fff8bc3991a _pthread_body + 0
20 libsystem_pthread.dylib  0x00007fff8bc37351 thread_start + 13
Stack dump:
0.     	<eof> parser at end of file
<unknown>:0: error: unable to execute command: Illegal instruction: 4
<unknown>:0: error: compile command failed due to signal (use -v to see invocation)

However in Package.swift when I comment out pkgConfig: "openssl", there is no problem and the build is successful.
Shouldn't the pkgConfig property be left out, as SPM will try to look for a file named openssl.pc (but there obviously is none in the package)? Is this causing the problem?
Thanks!

Huh, there is no package config file. We'll look into it. However, that shouldn't be causing you any issues (the package manager should just ignore it if there isn't one). What snapshot are you using on OSX?

I'm building an app on top of zewo/HTTPSServer and zewo/Router and therefore bound to DEVELOPMENT-SNAPSHOT-2016-05-09-a.

We're compiling open ssl from source now! (:


Format objects in alternative way - additional option.

Description

Hi! Could you add format objects in new alternative way as additional option?
I think that a lot of people will want to format object in another way as is it shown in above example.
As you can see, anotherObject values in new, alternate way will be format in one column.
Spaces will be automatically added between keys and values to make that looks very easy-readable > code. Algorith to realise that is very easy, but effects will be amazing :)
I hope to get answer. :)

Input
The code before:
var anotherObject = {
    name:                         "name",
    surname: "surname",
    id:     "id"
};

Expected Output
The code should have looked like this after alternative beautification:
var anotherObject = {
    name:      "name",
    surname: "surname",
    id:            "id"
};

Actual Output
The  code actually looked like this after beautification:
var anotherObject = {
    name: "name",
    surname: "surname",
    id: "id"
};

Steps to Reproduce
Iterate over keys and values in object and search the longest key.
Range again over object and as many spaces after key, as excepted - result of equation:
(longest key length + white space) - (actual value length). Then show value.
Environment
OS: Ubuntu
New additional setting example
{
alternative_object_format: true/false
}

Why?

Oh. Now I can see that code in post has been destroyed. Please, check difference on following screenshot:

As you can first two objects, they are more readable than the 'default' one.
It is very helpful if there are bigger objects (more than 5 keys) with different lengths.

Duplicate of #365.


Relies on out of Service .Net Version

This should work on .net 8, anyway to update the dependencies to be less restrictive?

Hi! I'll take a look

I just pushed version 1.1.0 to nuget.org
It need to pass some checks by nuget and will be available soon
https://www.nuget.org/packages/Project.Graph.Tool#versions-body-tab


Comment on isToolPublic

Current, the @Input() isToolPublic is not very clear what it is.  Make a comment or rename it so that it is more obvious

FYI, let's move these to the main repo for tracking


Allow Amazon certs in CAA record

I am giving Amazon certs a try for rules.ncats.cyber.dhs.gov instead of Let's Encrypt.  I suspect this will make certificate renewal easier.
I had to add two records to our Route53 configuration in order to accomplish this.

I went ahead and updated the rules.ncats.cyber.dhs.gov as well.  We should really look into automating that.

In the interest of full disclosure, I feel I should mention that this code is already deployed to the prod-a workspace, and the rules site is up and running with an Amazon cert now.


Tim/pack testing

Modifications to enable execution of st2 pack style tests in the plexxiconnectst2/docker/base docker container.
Depends on:
[StackStorm st2 PR 2256] (https://github.com/StackStorm/st2/pull/2256)
st2-run-pack-tests

Moved VIRTUALENV_DIR settings within the test for creating a new virtual env.  This allows us to test using our existing virtual environments that is created when the docker image starts.

Also remove the re-installation of requirements.txt.  Only pip installs should be to install test dependancies.

st2tests/st2tests/mocks/sensor.py

Fix to handle traceback discovered during testing.

Note: The remaining changes are related to mac development - originally branched from Andrew's mac_dev branch.

@tim-ireland Do you plan to re-open this PR?
As far as st2-run-pack-tests script goes. As noted in the other PR, I think the improvement would be to split it into two scripts:

st2-setup-pack-tests-environment - Script which sets up the environment so tests can run in it (would mostly mean moving code from existing st2-run-pack-tests script into this file)
st2-run-pack-tests - Script which only runs the tests.

This way the users who already have an existing environment or want to manage the environment themselves can only run st2-run-pack-tests.


fides user create ... doesn't work in zsh

The following was recorded from the fides user create ... step in the QA steps here
(fides) ➜  fides git:(main) ✗ fides user create newuser HolyCow1*1
zsh: no matches found: HolyCow1*1

* is a special character in zsh. I believe wrapping it in quotes will work.
fides user create newuser "HolyCow1*1"

@seanpreston also the username/password are options, not arguments
fides user create -u newuser -p Testpassword1!
Otherwise it will prompt you for them

Thanks! Will close this issue.


Database Seeds

Create the database schema.sql
Create the seeds files, to use as template for index, comment, post, user seeds.

Used for demonstration and functionality purposes.

Database Seeds section template completed. Close broad/general issue. New issues to be more specific.

Database Seeds section template completed. Close broad/general issue. New issues to be more specific.


javascript_es6/01.browser-echo sample throws an error in Edge

Sample information

Sample language:  es6
Sample name: javascript_es6/01.browser-echo

Describe the bug
The sample '01.browser-echo' works fine on Chrome but throws up an error on Edge.
Error: "SCRIPT1028: SCRIPT1028: Expected identifier, string or number
app.js (16292,24)"
The issue is with the use of object spread operator which is not supported in Edge :
const card = { ...other };
To Reproduce
Steps to reproduce the behavior:

Run the sample '01.browser-echo' on the Command Prompt.
Open Edge and type 'http://localhost:8080' and enter.
Page doesn't show anything and is blank.
Press F12 to inspect element and you will see the error on the console "SCRIPT1028: SCRIPT1028: Expected identifier, string or number
app.js (16292,24)".

Additional context

Tried using  @babel/plugin-transform-spread and then  including  @babel/plugin-transform-spread and @babel/plugin-proposal-object-rest-spread in the presets in .babelrc file, but it still shows the same error.
The targets for the presets in .babelrc file only specifies for the Chrome version but not for Edge.

[bug]

const card = { ...other };

It could become
const card = Object.assign({}, other);

It also throws in IE11. I am fixing it.

@anushaRaoR the PR is up.
Although we have the fix for the sample and the sample will work in IE11, there are some issues with Bot Framework SDK need to be addressed before this sample will work. Notably,

https://github.com/microsoft/botbuilder-js/issues/1073
https://github.com/microsoft/botbuilder-js/issues/1075
https://github.com/microsoft/botbuilder-js/issues/1076


disk plugin reports bind mounts as duplicates

Consider an /etc/fstab like
Label=foo  /mnt        auto  default 0 0
/mnt/sub   /media/sub  none  bind    0 0
...

i.e. a mount point /mnt and some subdirectories of it bind-mounted to some other directory.
Such configuration leads to the disk plugin of telegraf reporting disk usage statistics for each of these bind mounts separately, with the same data (that of the original mount point).

Closing this as expected behavior.
What would you want it to do? Ignore your bind mounts? If they're in fstab seems like they should be listed. You can ignore them using tagpass/tagdrop or pass/drop filters. https://github.com/influxdb/telegraf#plugin-options


Relative Humidity Climate Variable

Create a functions for the RH Variable

[x] Create Function
[ ] Create UI

Resolved by #15


TO: Internal Server error is returned when user not tenant of the ds assigns/deletes required capability

I'm submitting a ...

[X] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[X] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

500 Internal server error
Expected / new behavior:

403 Forbidden access
Minimal reproduction of the problem with instructions:

create tenants 1 and 2 at same level not associated with each other
try to post on DS of tenant1 while logged in as tenant2
Issue1:
POST https://{{TO_BASE_URL}}/api/{{api_version}}/deliveryservices_required_capabilities
{
"deliveryServiceID": 17,
"requiredCapability": "DISK_CAP"
}
Issue2:
DELETE https://{{TO_BASE_URL}}/api/{{api_version}}/deliveryservices_required_capabilities?deliveryServiceID=17&requiredCapability=DISK_CAP
Anything else:

No, this is - at least for DELETE - correct behavior. A user without tenancy permissions over a DS should never know that the DS exists. Responding with a 403 circumvents that, by admitting that the DS exists.
When assigning a required capability to a Delivery Service with improper tenancy, the response should be one of:

400 Bad Request - most common but perhaps more general than we need to be
404 Not Found - immediately tells you that something you were looking for didn't exist, but sort of implies that it's the URI that's non-existent, which isn't the case here
409 Conflict - Something about the state of the server is in conflict with the request, in this case the fact that the requested Delivery Service doesn't exist (as far as the requesting user knows).

For my money one of the bottom two is best, and I sort of go back and forth as to which is better. But all three are acceptable.

The 403 (Forbidden) status code indicates that the server understood the request but refuses to authorize it. A server that wishes to make public why the request has been forbidden can describe that reason in the response payload (if any).

If authentication credentials were provided in the request, the
server considers them insufficient to grant access. The client
SHOULD NOT automatically repeat the request with the same
credentials. The client MAY repeat the request with new or different credentials. However, a request might be forbidden for reasons
unrelated to the credentials.

An origin server that wishes to "hide" the current existence of a
forbidden target resource MAY instead respond with a status code of
404 (Not Found).```

so if we don't agree to be 403, it should be 404

I think I agree with @lbathina that it should be a 403 Forbidden.

A user without tenancy permissions over a DS should never know that the DS exists. Responding with a 403 circumvents that, by admitting that the DS exists.

You're right, that's a good goal for security. But in this case, it's not possible to hide existence. Postgres IDs are predictable and we've never taken the measures to obfuscate that. More than that, it'll always be possible to find out if a DS exists, by trying to create one with the same name.
Since it isn't possible to actually hide, a 403 Forbidden with a user message "User X Tenant Y does not have access to that resource" is the most helpful to legitimate users.

Then why bother hiding Delivery Services in the response to /deliveryservices?  An error message for creating DSes with duplicate keys can be changed just as easily.

Removing unprivileged DSes in /deliveryservices endpoints isn't just for security, it's a convenience and improves usability. There's no need for a user to see DSes they don't own, so it saves them an extra request (or checking some field on the DS) to figure out whether they have access to it.

 Most data has name endpoints as well (and IMO we should be moving that direction, away from IDs). And again, names are always discoverable.

An error message for creating DSes with duplicate keys can be changed just as easily.

That would be a huge usability loss. If a user is trying to make a DS, and doesn't get a message telling them the name is already taken, now they're going to call our support and ask why it's failing with a cryptic message. And what is our support to do? Cryptically answer that they must pick another name, but somehow not let them know that name is taken?
Again, it's a good goal, but obscurity is always a lesser form of security, and it comes at a cost: when someone really should have access to a DS and the permissions are a mistake, telling them it doesn't exist is going to create a lot of confusion. That confusion just doesn't seem worth it, when it isn't really buying us security anyway.
Suppose someone malicious knows an ID is valid. What can they do, that they couldn't do anyway? If they have an exploit, they can use it to determine existence, correlate the name, modify or delete, or whatever else their exploit allows.
Obscurity is a small security feature; but in this case it's very small (I'm not sure it exists at all), and comes with a high usability cost.

I think 403. Maybe not perfect as it is admitting a ds, but that's what we've always done on tenancy failure.

works fine. Message could have been better.


smug start ignores .smug.yml

Since the latest version (0.3.5) of smug on homebrew it no longer honors the local .smug.yml file.
Instead when I do smug start it starts all configurations in my config folder.
I would love to get a smug.log but running any command with -d or --debug just has it complaining there is no such project.

Thanks for reporting. Should be resolved in v0.3.6

I just upgraded (homebrew) and smug start works again with the local .smug.yml. 👍
smug stop doesn't, it just says "Terminating session..." and nothing else happens.


Implementing body matchers - for complex responses

When I looked at the PactDslJsonBody for implementing matchers , it could get difficult to keep for complex responses. Is there a means to just mention that match as per type ?
.WillRespondWith(new ProviderServiceResponse
{
Status = 201,
Headers = new Dictionary<string, object> {{"Content-Type", "application/json; charset=utf-8"}},
Body = Match.Type(paymentResponse)
});

Closing this in favour of https://github.com/pact-foundation/pact-jvm/issues/1541


Issue all messages using the logging library

This commit changes the issuing of messages to use the Python logging library. Logging in the module now follows this process:

The module logger is initiated on import (without a handler)
The user then has the opportunity to attach a handler to the logger before the pypandoc functions are called.
When a function that issues messages is called, check if the logger has a handler and if not add a console handler, formatted in the same manner as pandoc's.
When log messages are detected on Pandoc's stderr stream, parse them to determine the level and then reissue the log messages at the appropriate level.

Checking and setting up the logging handler is done by the _check_log_handler function (in the handler submodule), while determining the level of the messages issued by Pandoc on stderr is done by the _classify_pandoc_logging function.
Note, all "quiet" flags have now been removed, as this can now be handled by configuring the logger.
Closes #249

amazing work @H0R5E
I was wondering, if it would be best behavior to initialize a logger instead of just using print. Or would that be overkill.
An alternative could be to use the already imported warmings module.
What do you think?

Thanks @NicklasTegner! I've been thinking about this PR and your suggestions.
Firstly, it struck me that the quiet flag is wrong! You can pass --quiet to pandoc in the extra_args to achieve the same thing, so I think the quiet flag I added should be removed.
Also, I don't think using the warnings module is the right approach because pandoc will put log messages with both INFO and WARNING level onto stderr.
Probably using the Python logging module is the most robust way to handle the stderr stream, but setting it up is a slightly complex process. Basically, I was considering this approach:

Initiate the module logger on import (without a handler)
The user then has the opportunity to attach a handler to the logger before the pypandoc functions are called. The logger could be muted by adding a null handler to it, for instance.
When _convert_input is called check if the logger has a handler and if not add a console handler, formatted in the same manner as pandoc's.
When log messages are detected on stderr, parse them to determine if they are at INFO or WARNING level and then reissue the log messages at the appropriate level.

The questions I have with this approach are "should the other functions that print (namely _ensure_pandoc_path) also use the logger, in which case also have responsibility for initializing it (and sharing the format?)?" and "is it just confusing to have this logging system working alongside the pandoc one?". My attempt at answers to these questions are that _ensure_pandoc_path should just keep printing and we use logging just to handle the pandoc output, and that if we do this right, the normal user won't notice that there is logging going on, while the power user has the opportunity to configure it.
So, in conclusion, I think the quiet flag should be removed either way as it is redundant. I think using the logging module is a good idea (to allow customization), but there are many ways to set it up and I wanted to check with you whether my idea above is sound.
Cheers,
Mat

@H0R5E
In that case, yes the quiet flag should be removed.
For logging, I would say that your list is a good solution, I would however put all printing (including from _ensure_pandoc_path) into the logging as well, at the appropriate levels.

OK @NicklasTegner, I'll give it a try.

@NicklasTegner, I've made the changes. Everything that printed or wrote to stdout or stderr now adds messages through the pypandoc logger. I've also removed ALL the quiet flags, including the existing ones. These didn't seem to make any sense anymore. For instance, download_pandoc simply muted stdout when the quiet flag was set, but I don't want to manipulate the logging level, because that is something under the user's control. Note that because I have removed the quiet flags this is a breaking change.

@H0R5E  can you explain what exactly is a "breaking change" in this?

Usually any change in the public command line interface (removing a flag that users may have used and now they have to adapt) is a breaking change. But then, people usually embrace change because they want the beneficial parts that should come along with it - maybe just consider when bumping the semantic version.

mmhmm. What do you guys think.
Either just remove the flags outright in this pr, bumping the version to 1.8 when relizing.
Or issuing a warning message, stating that the quiet flag is Deprecated, and will be removed in 1.8, where we will then be able to include this pr in the next minor release, like 1.7.x?

So, to be precise, the breaking change is an API change to the ensure_pandoc_installed function:
-def ensure_pandoc_installed(url=None, targetfolder=None, version="latest", quiet=False, delete_installer=False):
+def ensure_pandoc_installed(url=None,
+                            targetfolder=None,
+                            version="latest",
+                            delete_installer=False):

And an API change to the download_pandoc function:
-def download_pandoc(url=None, targetfolder=None, version="latest", quiet=False, delete_installer=False, download_folder=None):
+def download_pandoc(url=None,
+                    targetfolder=None,
+                    version="latest",
+                    delete_installer=False,
+                    download_folder=None):

My understanding is that semantic versioning would require that this then be a major version change (i.e. to version 2).
Here are the options I see for finishing this:

Merge the PR as it is and bump to version 2. A major version change is indicative to users of breaking API changes.
Replace the quiet flags, but just with deprecation warnings and no functionality. Release this change as a minor release (i.e. 1.8) and then the deprecation warnings would be removed on the next major release (i.e. version 2).
Replace the existing quiet flags and re-implement their functionality by adding if not quiet: guards in front of all the relevant logging messages.

Happy to finish this off whichever way you want.

I think option 2 in my list is my preference, BTW.

I would say go ahead with option 2. Adding a message like:

deprecation Warming: The quiet flag in PyPandoc has been deprecated in favor of logging. See LINK for more information

Just essentially a message, stating that we have removed the functionality in favor of logging with the logging module, and where to find more information e.g the readme.

@NicklasTegner, I think this is ready now.

fantastic work. Good job :) @H0R5E


Create: Add input project owner

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #2746   👈

This comment was auto-generated by Graphite.

⏳ This pull request is set to merge as part of a Graphite merge job
Stack job ID: Kf6VOZ3shANW2oFFcG1d.
See details on graphite.dev


Uncaught TypeError: getItemIndex is not a function (Angular Material v10 drag n' drop)

🐞 bug report
Affected Package

 The issue is caused by package @angular/material.
Is this a regression?

 Yes, the previous version in which this bug was not present was: Angular Material 9.
Description
 I have a cdkDropList. It is tied to a predicate using [cdkDropListEnterPredicate]. The predicate function is defined like this: scheduleDdPredicate(item: DragRefInternal, dropList: DropListRef). I am trying to get the value of the index that the dropped item is being placed in. To do so, I have this line:
const newListIndex = dropList.getItemIndex(item);
Here is the HTML code:

<div cdkDropList id="schedule-dd-list" #scheduleDdList [cdkDropListData]="legs$ | async"
[cdkDropListEnterPredicate]="scheduleDdPredicate" (cdkDropListDropped)="drop($event)"
(cdkDropListSorted)="onSorted($event)" (cdkDropListEntered)="onEnter($event)">

This has worked previous to upgrading to Angular Material 10. See exception below.
🔥 Exception or Error

Uncaught TypeError: dropList.getItemIndex is not a function
    at CdkDropList.scheduleDdPredicate [as enterPredicate] (schedule-detail-container.component.ts:150)
    at DropListRef.CdkDropList._dropListRef.enterPredicate (drag-drop.js:2535)
    at DropListRef._canReceive (drag-drop.js:1959)
    at drag-drop.js:1950
    at Array.find ()
    at DropListRef._getSiblingContainerFromPosition (drag-drop.js:1950)
    at DragRef._updateActiveDropContainer (drag-drop.js:840)
    at SafeSubscriber.DragRef._pointerMove [as _next] (drag-drop.js:413)
    at SafeSubscriber.__tryOrUnsub (Subscriber.js:183)
    at SafeSubscriber.next (Subscriber.js:122)

🌍  Your Environment
Angular Version:

Angular CLI: 10.0.5
Node: 12.14.1
OS: win32 x64

Angular: 10.0.7
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.1000.5
@angular-devkit/build-angular      0.1000.5
@angular-devkit/build-optimizer    0.1000.5
@angular-devkit/build-webpack      0.1000.5
@angular-devkit/core               10.0.5
@angular-devkit/schematics         10.0.5
@angular/cdk                       10.1.2
@angular/cli                       10.0.5
@angular/material                  10.1.2
@angular/material-moment-adapter   10.1.2
@ngtools/webpack                   10.0.5
@schematics/angular                10.0.5
@schematics/update                 0.1000.5
rxjs                               6.5.5
typescript                         3.9.7
webpack                            4.43.0

Anything else relevant?

What does your scheduleDdPredicate function look like?

What does your scheduleDdPredicate function look like?

 scheduleDdPredicate(item: CdkDrag, dropList: CdkDropList) {

    const newListIndex = dropList.getItemIndex(item);
    const legs: Leg[] = dropList.data;

    if (legs) {
      // 'stop' because in future, non-stop cards will be draggable
      if (newListIndex === -1 && item.element.nativeElement.id.includes('stop')) {
        return true;
      } else {
        return false;
      }
    }
  }

@crisbeto We made a workaround like so:
  scheduleDdPredicate(item: CdkDrag, dropList: CdkDropList<Leg[]>) {
    if (item.data instanceof Leg) {
      if (dropList.data.findIndex(l => l.guid = item.data.guid) === -1) return true;
    }
    if (dropList.data) {
      // 'stop' because in future, non-stop cards will be draggable
      return item.element.nativeElement.id.includes('stop') ? true : false;
    }
  }

The error makes it look like the dropList isn't actually an instance of CdkDropList, but from my testing it's behaving correctly. What do you get if you try to console.log(dropList);?

Hello! I'm closing this due inactivity. Please let us know if you have more details. We are happy to look more into it.


[Event Request] CodeUnit 5055 - CustVendBank-Update

In the codeunit 5055 Customers, Vendors & BankAccounts are created/updated from Contacts. BUT the biggest issue is that this is done via a TransferFields. The issue with this is, when a field is not the same, there is no way to skip the Transferfields and handle the issue yourself. If you look at the function UpdateEmployee you see that there is no TransferFields in this. This is a better way to handle the create/update. Cant you add Events So we can choose to Skip the TransferFields?
procedure UpdateCustomer(var Cont: Record Contact; var ContBusRel: Record "Contact Business Relation")
var
VATRegistrationLogMgt: Codeunit "VAT Registration Log Mgt.";
VATRegNo: Text[20];
begin
with Cust do begin
Get(ContBusRel."No.");
NoSeries := "No. Series";
VATRegNo := "VAT Registration No.";
OnBeforeTransferfieldsCustomer(Cust, Cont,IsHanled);
if not IsHandled then
TransferFields(Cont);
"No." := ContBusRel."No.";
"No. Series" := NoSeries;
"Last Modified Date Time" := CurrentDateTime;
"Last Date Modified" := Today;
OnAfterUpdateCustomer(Cust, Cont);
Modify;
if ("VAT Registration No." <> '') and ("VAT Registration No." <> VATRegNo) then
VATRegistrationLogMgt.LogCustomer(Cust);
end;
end;
[IntegrationEvent(false, false)]
local procedure OnBeforeTransferfieldsCustomer(var Customer: Record Customer; Contact: Record Contact; var IsHandled: Boolean)
begin
end;
And also add the same structure for UpdateVendor,UpdateBankAccount.
*for some reason the spacing is gone? How do I solve this?

Sorry to put 4 codeunit is one issue, but the thing is, they all have the same issue so it would be nice if it was fixed at the same time.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Delete button doesn't work in edit views

This is because delete views redirects to URL with appended /delete, i.e. workshops/training_progress/1172/edit/delete instead of https://amy.software-carpentry.org/workshops/training_progress/1172/delete.
A workaround is to delete objects from detailed view. This workaround doesn't work for TrainingProgresses, because they don't have detailed view (see #1084).

FWIW, I believe that the delete button should be displayed on the detail view and not the edit view. That is what I have observed most of the times.

@pbanaszkiewicz ?

I lean towards "Delete" button on the detail view (and in some cases on the list view).

@pbanaszkiewicz Do you propose we add a detail view for training progress objects?

@narayanaditya95 yes if it's not too much work. If someone is lazy, fixing just the URL on the edit view would be fine too.

That seems to be the easiest way out. +1

:+1: for having delete buttons on detail views and removing them from edit views.


Make OS/X more explicit, to avoid questions

Make INSTALL more explicit about Mac OS/X, since some folks are confused.

And suggestion is that reference to NOTES.UNIX will make Mac OS X user less confused? NOTES.UNIX is pretty much about ELF Unix, and I'd say that Mac OS X will only get more confused...

If you need a spot to mention Mac OS X line 79 would be more appropriate...

This means that suggestion that NOTES.UNIX applies to any supported Unix like system can also be viewed as confusing. Unless of course one draws equivalence between support and applicability of the file in question.

Well, there is reference even on line 79, I kind of missed it, sorry about that. But the rest applies, i.e. NOTES.UNIX is hardly relevant to non-ELF system, and if Mac OS X user fails to recognize the system as Unix, then reference to NOTES.UNIX will confuse him/her even more...

Regarding NOTES.UNIX, is the description there very far from Mac OS/X reality?  As far as I know, the linker on Mac OS/X takes -rpath the way it's shown in the first two commands...  no?

Do note if in "if Mac OS X user fails to recognize the system as Unix." Now ask yourself what would such user take out of NOTES.UNIX? Note that it's filled with lingo that is specific to ELF. "RPATH or RUNPATH" is mentioned in opening paragraph and kind of sets the tone. Actually one can even wonder what an unprepared ELF user is supposed to take out. What's "RPATH or RUNPATH"?  Yes, there is -rpath on Mac OS X, but there is no RPATH, or LD_LIBRARY_PATH for that matter. There are equivalent of those on Mac OS X, but neither would help confused user...

I pushed a changed commit that doesn't reference NOTES.unix (which as Andy says is really about ELF-format binaries) but still has two reminders about Mac.

Regarding LD_LIBRARY_PATH, this says it's valid on Mac OS/X as well.
But sure, RPATH and RUNPATH are named differently...
I'd suggest adding text for Mac OS/X in NOTES.UNIX.  NOTES.UNIX is meant for all Unix family operating systems, not just ELF!  It's just that so far, it's ELF stuff that's made it into that file, but there's nothing stopping anyone from adding more in there.

Fixing up NOTES.unix is a good idea, but not for this PR since I don't know enough to make such an update.  Meanwhile, this needs a reconfirm.

Merged; I opened #6407 to capture the NOTES.UNIX issues.

Regarding LD_LIBRARY_PATH, this says it's valid on Mac OS/X as well.

Oh! I suppose they got tired of confused users :-) As this is not something that has been around from start. And wait a sec... Latest Mac OS X version doesn't pay attention to LD_LIBRARY_PATH either... Ah! That page talks about dlopen. Apparently LD_LIBRARY_PATH is interpreted by dlopen. But it's not interpreted by equivalent of Linux ld.so. And NOTES.UNIX is effectively about ld.so...

I agree it could be read that way.  How about this:
The Unix/Linux family (which includes Mac OS/X), OpenVMS, and Windows)

WFM

Advance +1 for cherry picking this to 1.1.0


configure Nutch to crawl a specific set of Wikipedia articles (elements)

To limit the scope and increase the relevance of our results, we want to search across a limited subset of data.
Elements on Wikipedia seem like a good place to start.

I've completed this manually, using a process like this:
# setup
export NUTCH_HOME=/home/mpappas/apache-nutch-1.19/runtime/local
cd $NUTCH_HOME
vim urls/seed.txt  #added 118 Wikipedia pages (chemical elements)
export JAVA_HOME='/usr/lib/jvm/default-java'

# run in screen because the crawl will take a few hours
screen

# do the work
bin/nutch inject crawl/crawldb urls
bin/nutch generate crawl/crawldb crawl/segments
s1=`ls -d crawl/segments/2* | tail -1`
bin/nutch fetch $s1
bin/nutch parse $s1
bin/nutch updatedb crawl/crawldb $s1
bin/nutch invertlinks crawl/linkdb -dir crawl/segments
bin/nutch index crawl/crawldb/ -linkdb crawl/linkdb/ $s1 -filter -normalize -deleteGone
bin/nutch dedup crawl/crawldb -group domain
bin/nutch clean crawl/crawldb/


Error to build after your last release in 'add-apt-repository'

Hello,
After your last release i had an error adding repositories
Build failed: The command '/bin/sh -c add-apt-repository -y ppa:webupd8team/java && apt-get update && echo "oracle-java8-installer shared/accepted-oracle-license-v1-1 select true" | sudo debconf-set-selections && apt-get -yf install oracle-java8-installer' returned a non-zero code: 127

In ubuntu 14.04 works nice.

sorry i have the saem error under 14.04 my mistake

I have the same error now, I use this for CI and all my builds are failing.

/bin/sh: 1: sudo: not found
The command '/bin/sh -c sudo apt-get -y install software-properties-common && sudo add-apt-repository ppa:chris-lea/node.js && sudo apt-get update' returned a non-zero code: 127

A hot fix lock the version to the last tag and works as usual

This is not a problem in baseimage-docker. Things are supposed to be slightly different between distribution versions. Some things which were installed by default may no longer be installed by default, and vice versa. You are responsible for updating your build scripts according.
For example in @arg20's case, the error message clearly shows that 'sudo' is not found, so you need to install that yourself.


Remove timer-based reporting and trade-based aggregation

Expected Behavior
Remove timer-based reporting and trade based aggregation. We should probably keep things set up in a way where we can easily add a new report or aggregation strategy in the future, but there is no need to keep code around that isn't used at all.
Current Behavior
When we transitioned to block-based reporting and a ticker-based price aggregation, we kept the code for timer-based reporting and ticker-based aggregation so we could easily revert back if we wanted to. Now that we've been running the oracles with the current setup for many months, the unused code is tech debt, a burden to maintain, and has some confusing unused config variables

This blocks implicit pairs work.


Zeppelin not starting. NoSuchMethodError

Hi,
I checked out the latest git and followed the guide. I ran into #403. When the fix from that report was applied and I ran
./bin/zeppelin-daemon.sh start

I got
Exception in thread "main" java.lang.NoSuchMethodError: org.eclipse.jetty.webapp.WebAppContext.setProtectedTargets([Ljava/lang/String;)V
        at org.eclipse.jetty.webapp.WebAppContext.<init>(WebAppContext.java:198)
        at com.nflabs.zeppelin.server.ZeppelinServer.setupWebAppContext(ZeppelinServer.java:236)
        at com.nflabs.zeppelin.server.ZeppelinServer.main(ZeppelinServer.java:73)

in my logfile.

Can you provide your build configuration?

 I couldn't replicate the behaviour so far.

build.log

First of all, it's not related to #403, since there is no problem with frontend-plugin. Could you debug your application and put debug logs here?

When linking to the other bug, I just wanted to say, that I had to change the plugin dependency. This change was not yet in the repo.
When debugging there is a difference between IDE and remote debugging. Remote debugging fails when calling ContextHandler#setProtectedDomains. Some dependency must be wrong here.
Could this be somehow related to https://github.com/hawtio/hawtio/issues/122#issuecomment-13922038 ?


How can I import RedshiftInputFormat in python?

I can't seem to figure out the correct package name
from com.databricks.spark.redshift import RedshiftContext fails
As does RedshiftInputFormat.
Trying to import a file that has already been exported from Redshift using ESCAPE.
Thanks!

The only way to access this from python is through the data source API, which is only available in master.  We'll be publishing a new version of the library with this feature soon.

Another way is through the newAPIHadoopFile method under SparkContext in Python. See http://spark.apache.org/docs/latest/api/python/pyspark.html#pyspark.SparkContext.newAPIHadoopFile.

It looks like the use of the data sources API from Python has been addressed by the docs in master, so I'm going to go ahead and close this ticket for now since it no longer seems actionable. Please comment if you feel that we can improve the Python documentation here.


Rename "Developer Tools" -> "Debug Mode" in README

Coverage remained the same at 100.0% when pulling 4332a1a7c0caf11a7096ccb5d5e2b615fe960b89 on NekR-patch-1 into aab84f8f169298a5ee04598a7b7b0b429002bc91 on master.


TimeStamp.fromString should be public

Plugin needs to access this method to create Timestamp from strings.

In which case you need Timestamp.fromString? Why don't you use TimestampParser?

I failed to notice there is TimestampParser.

My expectation was there is some factory method like `Integer.parseInt(str)'

What is the difference of Timestamp.fromString and `TimestampParser


Add new workflow to test build the npm library

Closes #104
This PR:

adds new workflow to test build the npm lib
updates other workflows to cache artifacts correctly
updates cargo deps

Codecov Report

Merging #105 (cf867fe) into main (c23bf1a) will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   39.92%   39.98%   +0.06%
==========================================
  Files          23       23
  Lines        1996     1993       -3
  Branches      449      449
==========================================
  Hits          797      797
+ Misses        902      899       -3
  Partials      297      297

Impacted Files
Coverage Δ

src/aws/mod.rs
0.00% <ø> (ø)

src/utils.rs
26.31% <0.00%> (ø)

src/config.rs
0.00% <0.00%> (ø)


Disable Flaky Test test_poisson_generator

Description
Tracked in https://github.com/apache/incubator-mxnet/issues/13584
Changes

Disable test with tracking

Comments

@mxnet-label-bot add [Flaky, Test, pr-awaiting-review]

@marcoabreu Can you review this PR and merge it if it looks good ?


Error opening app

I can no longer open the app on Mac OS X. It fails with the following error:

Try npm install

you need to upgrade electron @JustinDrake .
npm install<EMAIL_ADDRESS>

Cheers guys.


Fixed compatibility with AvaticaResultSet which doesn't support isLast()

Hello,
This PR fixes compatibility with the avatica JDBC driver, which doesn't support isLast() [1].
Tests are passing, please let me know if anything else needs to be done to merge.
Jacques
[1] https://github.com/apache/calcite-avatica/blob/4db1fb9c66db8ccebc9e96ce678154ec69c557f0/core/src/main/java/org/apache/calcite/avatica/AvaticaResultSet.java#L444

Coverage decreased (-0.002%) to 94.178% when pulling 00856cb8d942bb333aeb088512430f39d6db3b68 on jarnoux:master into 3b2eb5807a899dd3b630692168348f28288d0a7f on stanford-futuredata:master.

Is the coverage going to be an issue? I could artificially restore the line count to make it pass but that's not very useful

LGTM; thanks for this!


unit test for deploy

I think this can be done in pytest. The setUp functionality can be done with scoped fixtures. If there's nothing else requiring unittest, I'd recommend keeping to pytest.

last touched Feb 2019, obsolete


Support app package for semantic segmentation sample

@fujunwei, @huningxin, PTAL, thanks!

Merge it, thanks.


Ripen not applied to super effective berries

I was messing around testing something on Flapple. I noticed the damage calculation was the same on ice moves with a Yache Berry held with and without Ripen as the ability. Ripen should double the effect of all berries. So a super effective attack should deal 1/4th damage and not half damage. I believe this should fix that case assuming I understood the surrounding code in regards to the damage formula.

Thanks!


Revert "Fix PathTranslator to compare path prefixes by path components"

This reverts commit 499d2b5686f9113c25e0ce665bf5c65c7eef3a7d.

Codecov Report

Merging #13696 (92e365d) into master (0203366) will decrease coverage by 12.92%.
The diff coverage is 65.05%.

@@              Coverage Diff              @@
##             master   #13696       +/-   ##
=============================================
- Coverage     43.86%   30.93%   -12.93%
+ Complexity     8930     5816     -3114
=============================================
  Files          1359     1382       +23
  Lines         78109    80199     +2090
  Branches       9496     9749      +253
=============================================
- Hits          34260    24811     -9449
- Misses        40960    53308    +12348
+ Partials       2889     2080      -809

Impacted Files
Coverage Δ

...o/client/block/policy/DeterministicHashPolicy.java
68.42% <ø> (ø)

...nt/block/policy/LocalFirstAvoidEvictionPolicy.java
94.28% <ø> (ø)

.../alluxio/client/block/policy/LocalFirstPolicy.java
93.93% <ø> (ø)

.../client/block/policy/MostAvailableFirstPolicy.java
91.66% <ø> (ø)

.../alluxio/client/block/policy/RoundRobinPolicy.java
97.91% <ø> (ø)

...lluxio/client/block/policy/SpecificHostPolicy.java
40.00% <ø> (ø)

...xio/client/block/stream/BlockWorkerDataReader.java
0.00% <0.00%> (ø)

...ient/block/stream/BufferCachingGrpcDataReader.java
42.25% <0.00%> (+0.58%)
:arrow_up:

...va/alluxio/client/meta/MetaMasterConfigClient.java
0.00% <ø> (ø)

...ient/meta/RetryHandlingMetaMasterConfigClient.java
9.52% <0.00%> (-3.81%)
:arrow_down:

... and 617 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 d964cd4...92e365d. Read the comment docs.

alluxio-bot, merge this please

@yuzhu Could you please give more detail for this revert?

Sure. It broke some automated tests we had with catalog service. Have talked with @dbw9580,and he will submit a patched version once the issue is root caused.


tf.keras.metrics.F1Score produces ValueError: Tensor conversion requested dtype float32 for Tensor with dtype int32

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): RHELS 7.9
TensorFlow installed from (source or binary): Pip, binary
TensorFlow version (use command below): 2.13.0
Python version: 3.9
GPU model and memory: NVIDIA A100-SXM4-40GB
Exact command to reproduce:

Describe the problem.
During training, model is monitoring tf.keras.metrics.F1Score; however, when F1Score.update_state is called, a Value Error is thrown.
ValueError: Tensor conversion requested dtype float32 for Tensor with dtype int32: <tf.Tensor 'cond/Identity_4:0' shape=(None,) dtype=int32>
which is the result of the following line of code in the FBetaScore Class:
 y_true = tf.convert_to_tensor(y_true, dtype=self.dtype)

Describe the current behavior.
F1Score metric unable to update_state.  Error thrown.  Unable to train model.
Describe the expected behavior.
I would expect F1Score to update_state based on a y_true tensor with an int32 datatype and a y_pred tensor of float32 datatype without throwing an error.
In the tfa.metrics.FBetaScore code, the corresponding line is:
y_true = tf.cast(y_true, self.dtype)

Is it possible that the new tf.keras.metric code should be using a tf.cast(...) vice a tf.convert_to_tensor(...)?

Do you want to contribute a PR? (yes/no): no

Standalone code to reproduce the issue.
Cannot share full code.  Can share custom model init / train_step which causes the error.
class CustomModel(keras.Model):
    def __init__(self, model_type, val_samples, threshold, *args, **kwargs):
      super(CustomModel, self).__init__(*args,**kwargs)
      self.loss_tracker = tf.keras.metrics.Mean(name='Loss')
      self.val_samples = val_samples
      self.precision = tf.keras.metrics.Precision(name='Precision')
      self.recall = tf.keras.metrics.Recall(name='Recall')
      self.f1 = tf.keras.metrics.F1Score(name="F1", threshold=threshold)
      if model_type == 'binary':
          self.accuracy = tf.keras.metrics.BinaryAccuracy(name='accuracy', threshold=threshold)
      else:
          self.accuracy = tf.keras.metrics.CategoricalAccuracy(name='accuracy')

    def train_step(self, data):
        inputs, targets = data

        with tf.GradientTape() as tape:
            predictions = self(inputs, training=True)
            loss = self.compiled_loss(targets, predictions, regularization_losses=self.losses)

        gradients = tape.gradient(loss, self.trainable_variables)
        self.optimizer.apply_gradients(zip(gradients, self.trainable_variables))

        self.loss_tracker.update_state(loss)
        self.accuracy.update_state(targets, predictions)
        self.precision.update_state(targets,predictions)
        self.recall.update_state(targets,predictions)
        self.f1.update_state(targets, predictions)
        return {"Accuracy":self.accuracy.result(),"Loss":self.loss_tracker.result(),
                "Precision":self.precision.result(), "Recall":self.recall.result(),
                "F1":self.f1.result()}

    def test_step(self, data):
        inputs, targets = data

        predictions = []

        for _ in range(self.val_samples):
            predictions.append(self(inputs, training=False))

        predictions = tf.math.reduce_mean(tf.stack(predictions, axis=0), axis=0)

        loss = self.compiled_loss(targets, predictions, regularization_losses=self.losses)

        self.loss_tracker.update_state(loss)
        self.accuracy.update_state(targets, predictions)
        self.precision.update_state(targets,predictions)
        self.recall.update_state(targets,predictions)
        self.f1.update_state(targets, predictions)
        return {"Accuracy":self.accuracy.result(),"Loss":self.loss_tracker.result(),
                "Precision":self.precision.result(), "Recall":self.recall.result(),
                "F1":self.f1.result()}

    @property
    def metrics(self):
        return [self.accuracy,self.loss_tracker, self.precision, self.recall, self.f1]

Source code / logs.
Epoch 1/2000
Traceback (most recent call last):
  File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/data/kraken/john.fischer/projects/passive_sonar_models/scripts/../../passive_sonar_models/__main__.py", line 110, in <module>
    main()
  File "/data/kraken/john.fischer/projects/passive_sonar_models/scripts/../../passive_sonar_models/__main__.py", line 37, in main
    train_model.main(args)
  File "/data/kraken/john.fischer/projects/passive_sonar_models/scripts/../../passive_sonar_models/task/train_model.py", line 139, in main
    model.fit(train_data, epochs=args.num_epochs, validation_data=validate_data, #steps_per_epoch=steps_per_epoch, validation_steps=val_steps,
  File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/site-packages/keras/src/utils/traceback_utils.py", line 70, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/tmp/__autograph_generated_file3ywpkuyj.py", line 15, in tf__train_function
    retval_ = ag__.converted_call(ag__.ld(step_function), (ag__.ld(self), ag__.ld(iterator)), None, fscope)
  File "/data/kraken/john.fischer/projects/passive_sonar_models/scripts/../../passive_sonar_models/models/mc_dropout_CNN.py", line 42, in train_step
    self.f1.update_state(targets,predictions)
ValueError: in user code:

    File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/site-packages/keras/src/engine/training.py", line 1338, in train_function  *
        return step_function(self, iterator)
    File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/site-packages/keras/src/engine/training.py", line 1322, in step_function  **
        outputs = model.distribute_strategy.run(run_step, args=(data,))
    File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/site-packages/keras/src/engine/training.py", line 1303, in run_step  **
        outputs = model.train_step(data)
    File "/data/kraken/john.fischer/projects/passive_sonar_models/scripts/../../passive_sonar_models/models/mc_dropout_CNN.py", line 42, in train_step
        self.f1.update_state(targets,predictions)
    File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/site-packages/keras/src/utils/metrics_utils.py", line 77, in decorated
        update_op = update_state_fn(*args, **kwargs)
    File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/site-packages/keras/src/metrics/base_metric.py", line 140, in update_state_fn
        return ag_update_state(*args, **kwargs)
    File "/home/john.fischer/.conda/envs/psonar2/lib/python3.9/site-packages/keras/src/metrics/f_score_metrics.py", line 176, in update_state  **
        y_true = tf.convert_to_tensor(y_true, dtype=self.dtype)

    ValueError: Tensor conversion requested dtype float32 for Tensor with dtype int32: <tf.Tensor 'cond/Identity_4:0' shape=(None,) dtype=int32>

I have the same problem with the training phase in the case of binary classification.

This is reproducible with a small change to the tutorial example at https://www.tensorflow.org/tutorials/keras/text_classification with TF 2.15.0
Simply adding an F1Score as a metric to compile raises the ValueError:
model.compile(optimizer=optimizer,
              loss=tf.losses.BinaryCrossentropy(from_logits=True),
              metrics=[tf.metrics.BinaryAccuracy(threshold=0.0, name='accuracy'),
                                              tf.keras.metrics.F1Score()
                       ])

Crashes with ValueError: Tensor conversion requested dtype float32 for Tensor with dtype int64 during fit(). Re-compiling with it as a metric after fit causes the same crash during evaluate().
A workaround to add F1Score to this particular tutorial is to change
train_data, test_data = tfds.load(name="imdb_reviews", split=["train", "test"],
                                  batch_size=-1, as_supervised=True)

train_examples, train_labels = tfds.as_numpy(train_data)
test_examples, test_labels = tfds.as_numpy(test_data)

to
train_data, test_data = tfds.load(name="imdb_reviews", split=["train", "test"],
                                  batch_size=-1, as_supervised=True)

train_examples, train_labels = tfds.as_numpy(train_data)
test_examples, test_labels = tfds.as_numpy(test_data)
train_labels = train_labels.astype(np.float64)
test_labels = test_labels.astype(np.float64)


Produce a complete connection string URI with credentials included

Problem
Currently, Atlas OpenServiceBroker ServiceBinding returns credentials in 3 fields:

SRV URL in the format: mongodb+srv://my-atlas-cluster.mongodb.net
Username: user
Password: password

Although this information is enough to connect to MongoDB, it requires applications to build a proper connection string from these data, e.g<EMAIL_ADDRESS>Proposal
Make Atlas OpenServiceBroker produce a complete connection string usable for application out of the box.

Hi! Most drivers have a way of taking these as structured data. For instance the Node driver takes an options parameter that can contain an auth object that looks like this:
{
  user: "my-username",
  password: "my-password"
}

See here: https://mongodb.github.io/node-mongodb-native/3.3/api/MongoClient.html
This should avoid the need for string concatenation in most cases. Is this insufficient for your use case?

Hi! Most drivers have a way of taking these as structured data. For instance the Node driver takes an options parameter that can contain an auth object that looks like this:
{
  user: "my-username",
  password: "my-password"
}

See here: https://mongodb.github.io/node-mongodb-native/3.3/api/MongoClient.html
This should avoid the need for string concatenation in most cases. Is this insufficient for your use case?

Hi @BenElgar!
Thanks for the advice. That will work, of course. In fact, this is what I had to do for one of our applications.
Let me provide a bit more details about our case. We have ~15 services using MongoDB. Written in PHP / Go, and running in Kubernetes. They all read configuration from the environment vars. And MongoDB connection string is just a single variable MONGODB_DSN in a Standard Connection String Format.
Recently we decided to go with Atlas and use Service Catalog + Atlas OpenService Broker to provision MongoDB clusters in Atlas. After successful ServiceBinding we have a Kubernetes Secret with url, username, and password, so we have to define 3 env vars for uri, username, and password and modify mongodb connection method in each application.
Ideally, we don't want to touch the application code at all. It is just a matter of configuration, after all, isn't it? We would like to add the following configuration change in service's Pod template, but as I said, the Secret doesn't have a single field in Standard Connection String Format.
apiVersion: extensions/v1beta1
kind: Deployment
...
spec:
...
  template:
    spec:
...
      env:
        - name: DATABASE_DSN
          valueFrom:
            secretKeyRef:
            name: mydb-user
            key: connection_string

So far I have made this change to Atlas OpenService Broker code: https://github.com/mongodb/mongodb-atlas-service-broker/pull/61 and trying it out.

Gotcha, I understand the use case now and see how it would be useful. My concern is that this would encourage people to use the connection string directly and thus not make explicit choices about other parameters such as write concern, the defaults for which are likely to be unsuitable for a variety of use cases and should be considered carefully before production use.
That being said, perhaps the right answer is to hardcode safer defaults in the convenience connection string. Let me confer with the team and get back to you.

I understand your concern. However, I thought it won't be a problem because atlas provides a connection string in DNS Seedlist Connection Format, which has its own way of passing default options via DNS TXT record.

Quick update. I have deployed the patched version of the Atlas OpenService Broker (from #61) to our cluster, and confirm it works as expected.

After using the patched version of the Atlas OpenService Broker (from #61) in production I collected feedback and got a better understanding of what should this feature look like. Please welcome #63
Background
ServiceBinding creates a Kubernetes Secret with 3 fields: uri, username, and password. If we run applications that require fully qualified MongoDB connections string URI this is not sufficient.
Proposal

Enable ServiceBinding to produce the connection_string field (in addition to uri, username, and password).
Add connectionString parameter to ServiceBinding spec parameters to make the connection_string configurable.

Documentation
After success binding, a Kubernetes Secret containing connection details has a connection_string field. By default it contains a connection string URI in the DNS Seedlist Connection Format with credentials: e.g. mongodb+srv://username:password@host. The connections string can be configured through connectionString parameter of ServiceBinding resource's spec.

Name
Type
Default
Description

skipCredentials
bool
false
Controlls whether credentials should be included in the URI

database
string
''
Database name porting of the connection string URI

format
'standard' or 'srv'
'srv'
Scheme portion of the connection string URI: mongodb or mongodb+srv

options
list
[]
Options portion of the connection string URI

Example ServiceBinding resource spec:
apiVersion: servicecatalog.k8s.io/v1beta1
kind: ServiceBinding
metadata:
  name: atlas-cluster-binding
spec:
  instanceRef:
    name: atlas-cluster-instance
  parameters:
    user:
      roles:
      ...
    connectionString:
      skipCredentials: false
      database: test
      format: standard
      options:
        connectTimeoutMS: 1000
        tlsAllowInvalidCertificates: true
        w: "majority"

We face the same problem using Spring Boot, which either accepts a URI, or a (host, username, password) triplet.
I like how #63 would work, is there any chance to make that part of the official package?

Thanks for your interest in the AOSB.
We've decided to deprecate this codebase, and are recommending that users switch to mongodb/mongodb-atlas-kubernetes instead.


Text wrapping and responsive design labels

I have inherited a project that uses Chart.js to display feedback results. Currently the longer labels are getting cut off at the edge of the iframe.
Expected Behavior
The label will wrap as the canvas becomes smaller.
Current Behavior
The label does not wrap.

Possible Solution
Unknown, I assume it would require a change in the following PolarChart.vue
<template>
    <canvas :id="_uid"></canvas>
</template>

<script>
    import Chart from 'chart.js';

    export default {
        props: [
            'enabled',
            'backgrounds',
            'labels',
            'max',
            'values'
        ],

        mounted() {
            var vueInstance = this;
            new Chart(document.getElementById(this._uid).getContext('2d'), {
                type: "polarArea",

                data: {
                    labels: this.labels,
                    datasets: [{
                        data: this.values,
                        backgroundColor: this.backgrounds
                    }]
                },

                options: {
                    animation: {
                        animateRotate: false
                    },

                    layout: {
                        padding: 5
                    },

                    legend: {
                        display: false
                    },

                    responsive: true,

                    scale: {
                        ticks: {
                            max: this.max,
                            min: 0,
                            stepSize: 1,
                            display: false
                        }
                    },

                    tooltips: {
                        callbacks: {
                            title (tooltipItem, data) {
                                return data.labels[tooltipItem[tooltipItem[0].datasetIndex].index];
                            },
                            label (tooltipItem, data) {
                                if (vueInstance.enabled[tooltipItem.index]) {
                                    return ' Rating: ' + data.datasets[0].data[tooltipItem.index] + '/' + vueInstance.max;
                                } else {
                                    return ' Not available in this round';
                                }
                            }
                        },
                        custom: function(tooltip) {
                        //    console.log(tooltip);
                            // todo: action to display info
                        }
                    }
                }
            });
        }
    }

Environment

Chart.js version:  Unknown, I do not have the development environment this was built in.
Browser name and version: Google Chrome | 62.0.3202.94 (Official Build) (32-bit)

Duplicate of #622 (same underlying issue). You can force the text to render on multiple lines by returning multiple lines in the tooltip callbacks

@etimberg  can you please share code to force the text to render on multiple lines by returning multiple lines in the tooltip callbacks as in issue #662 the link is broken https://stackoverflow.com/questions/21409717/chart-js-and-long-labels
which is given by @ChrisKatsaras


Patch: Using the existing template file in home component

Just to keep things consistent.

Thanks again ! 😄


Thumbnails of new maps are always centered in Morocco

When I create a map (tested with existing datasets and new datasets) it takes me to the map view where the center is ok where the data is located. However when I go to the map list view they are all centered in Morocco (more or less).
In order to have the thumbnail modified I have to go to the map view and pan through the map.
Maybe related to https://github.com/CartoDB/cartodb/issues/2462?
/cc @saleiva @xavijam @javisantana

This is happening because when you create a map (let's say by importing a CSV dataset of points in NY), the default center of the vizjson is:
center": "[30, 0]
and it only gets refreshed when you pan the map /cc @Kartones

I can, as I changed the bounds calculation recently. You want me just to set it to the center of the bounding box set, right?

Yeah.

@Kartones: can we give priority to this and https://github.com/CartoDB/cartodb/issues/5307 issues? It's needed for the new user page.

Assigning to this milestone but I need to discuss with @javisantana about the milestone

Ok, thanks!

what do you need?

:clap


Unit Test test_opensearch_tls.py::test_get_sans fail with wrong sans_ip

It is not 100% reproducible, but I can see test_get_sans failing most of the times in PR #248 with:
AssertionError: {'san[34 chars]: ['<IP_ADDRESS>', 'XX.XXX.XX.XXX', 'address1', '[55 chars]-0']} != {'san[34 chars]: ['<IP_ADDRESS>', 'XX.XXX.XX.XXX', 'address1', 'ad[53 chars]-0']}
  {'sans_dns': ['alias', 'nebula', 'opensearch-0'],
-  'sans_ip': ['<IP_ADDRESS>', 'XX.XXX.XX.XXX', 'address1', 'address2'],
?                -- ^ ^ ^

+  'sans_ip': ['<IP_ADDRESS>', 'XX.XXX.XX.XXX', 'address1', 'address2'],
?                 ^ ^ ^

   'sans_oid': ['<IP_ADDRESS>.5.5']}

Looking into the origins of this IP, I noticed that newer ops framework has this commit, that adds a network <IP_ADDRESS> if we do not have any existing networks.
We were originally using patch_network_get decorator on this test. I think we should move to _TestModelBackend.add_network instead.

Poetry.lock: https://pastebin.canonical.com/p/tqGvJVD88M/


Supported Cameras

Can you provide a few ONVIF cameras that will work with Akri that could be used with this demo?

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

ID: 5b00c5b4-6df8-f938-70ec-05b2b64c4409
Version Independent ID: ad6771fa-24d8-455b-8606-f00b2bb6ac10
Content: Discover ONVIF cameras with Akri - AKS hybrid
Content Source: AKS-Hybrid/aks-edge-how-to-akri-onvif.md
Service: azure-stack
Sub-service: aks-hci
GitHub Login: @yujinkim-msft
Microsoft Alias: yujinkim

@kvietmeier
Thanks for your feedback! We will investigate and update as appropriate.

hi @kvietmeier I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1


Upgrade PoET api call

Upgrade PoET api call to comply with https://github.com/spacemeshos/poet/pull/83 breaking changes.

bors merge

bors merge


detect non-oracle JDKs and reduce sodium accordingly

I don't have openjdk so I can't test this. Anyone who does is welcome to try.

On March 15, 2018 10:29:59 PM EDT, Jose C<EMAIL_ADDRESS>wrote:

jmcardon commented on this pull request.
It looks good to me, but I don't default to openjdk for dev either.
@coltfred @rsoeldner both have OpenJDK. Maybe one of them can help out
our cause? :D

Yes indeed. This is for them, anyways.
--harrison

hrhino:openjdk-plebs-rejoice

LOL


crypto/tls: make TLS 1.3 opt-in for Go 1.12

It seems clear from the history of filed issues that some users will experience minor disruptions when switching to TLS 1.3. For disruptive changes like that, we typically do one or more releases in which the new feature is opt-in, with a prominent announcement about when it will turn on by default. Then we do at least one release with the feature opt-out. (Past examples include HTTP/2, vendor directories, and modules.)
Especially given how central TLS is to so many Go users (any HTTPS client or server!), it seems like we should do that for TLS 1.3 too:

Make TLS 1.3 available in Go 1.12 but opt-in (off by default).
Make clear in the release notes and package docs how to turn on TLS 1.3.
Make clear in the release notes that TLS 1.3 will turn on by default in Go 1.13.
Make TLS 1.3 in Go 1.13 opt-out (on by default).

Following the HTTP/2 example my suggestion would be to use GODEBUG=tls13=1 for opt-in now, and GODEBUG=tls13=0 for opt-out later.
/cc @filosottile

Following the HTTP/2 example my suggestion would be to use GODEBUG=tls13=1 for opt-in now

That's fine as long as there's also a way to enable it in code. Via tls.Config somehow? (MaxVersion and MinVersion both seem wrong.) Or at least with os.Setenv at runtime, but that means crypto/tls would need to keep calling Getenv. New bool on tls.Config would be kinda unfortunate, but maybe?

Should this apply to RSA-PSS as well?

MaxVersion feels wrong semantically, and in practice it would lock applications to TLS 1.3 when TLS 1.4 ever comes out. I'm worried that tlsConfig.Debug would be a slippery slope, we get requests all the time to add all kinds of knobs which are inconsistent with the simplicity goal of crypto/tls, and that would provide an extra avenue for them.
How about GODEBUG=tls13=1, read once per tls.Config? It would allow os.Setenv in code, it would probably have acceptable performance, and still let application test canaries without modifying code. It would probably have to be inherited via GetConfigForClient if one os.Getenv per connection is unacceptable.

MaxVersion = 0xFFFF would be my vote. Or some other unused constant to mean MaxBeta.

But users likely wouldn't be able to reach all such configs so the environment way probably best if Setenv works.

CL 160997 just uses os.Getenv at every connection. On Unix it takes a brief lock but uses an internal view of the environment, so there is no syscall. On Windows it makes a syscall. We can test the performance before adding complexity the day before the RC.

Done for Go 1.12, milestoned to Go 1.13 to switch to opt-out.

I think it is worth to document how to use os.Setenv to set this environment variable.
I guess this might work
os.Setenv("GODEBUG", os.Getenv("GODEBUG")+",tls13=1")

But os.Setenv("GODEBUG", "tls13=1") will overwrite other GODEBUG like GODEBUG=http2server=0.
If some other goroutines are also setting GODEBUG, there can be a race condition.
Alternately, maybe using another env is better. And os.Setenv("GOTLS13", "1") is mostly OK.

According to https://github.com/golang/go/issues/13611
GODEBUG usage should also be documented in https://tip.golang.org/pkg/runtime/

@crvv, I sent https://go-review.googlesource.com/c/go/+/162360

Milestoned to 1.14 to remove the opt-out.


[Suggestion] Anti-Fog Visor Wearable/Enchantable/Bauble Ability

Add an Anti-Fog Enchant that can be used on any helmets.
Add the ability to wear Anti-Fog Visor, stand alone.
Add the ability to use Anti-Fog Visor in a Bauble Slot.
Not all planets require Full Space Suit, yet still may have Fog.

Not happening, as it ruins the reasons to actually wear an AR space suit.


fix(select): activated item refactor

:tada: This PR is included in version 2.11.9 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: