Fix #4

My local repository went wild, so I had to clean up after already having pushed this stuff. oof

Sigh. Send help. I put the wrong one, meant to be or false not and false


Unhomogeneous translated strings in French

@12rambau The French sentence ‘Cliquez pour développer’ (second person imperative) is not consistent with the similar ‘Cliquer pour réduire’ (infinitive mode).
There is another one: ‘Veuillez activer le JavaScript pour que la recherche fonctionne.’), which I think would be better worded as ‘Merci d'activer le JavaScript pour que la recherche fonctionne.’ (infinitive mode rather than imperative mode).

Thanks @dbitouze I can only agree that the translations are not perfect. Do you have requested access to the transifex portal ?
https://app.transifex.com/12rambau/pydata-sphinx-theme/dashboard/
If not please request access so I can add you in the list of the french moderator and you'll be able to make direct suggestions there

@12rambau I'm already a moderator but I can't change those strings: I can only save “as suggestions” but, unfortunately, this doesn't save my proposals but the strings as they already are translated (see e.g. this).

@dbitouze @12rambau I see some strings were updated in https://github.com/pydata/pydata-sphinx-theme/pull/2009/files, but per @dbitouze's comment above, it seems others might need approval.
But looking into Transifex, everything seems to be translated and approved. https://app.transifex.com/12rambau/pydata-sphinx-theme/viewstrings/#fr/79592a50a377d7a120926189491c3d76?q=reviewed%3Ano
@dbitouze can you check the localised files here and let us know if these are ok now?
Aside. Would it make sense to add @drammock and me as owners/maintainers to Transifex, too? It would be good to have more than one person to approve translations and manage the project to reduce the bus factor here.

@trallard The file https://github.com/pydata/pydata-sphinx-theme/blob/main/src/pydata_sphinx_theme/locale/fr/LC_MESSAGES/sphinx.po is still suboptimal.
On Transifex, everything is translated and (unfortunately) approved but still suboptimal: I can only suggest changes and can't actually change anything.
And there are differences between https://github.com/pydata/pydata-sphinx-theme/blob/main/src/pydata_sphinx_theme/locale/fr/LC_MESSAGES/sphinx.po and what is on Transifex (I guess that's because https://github.com/pydata/pydata-sphinx-theme/pull/2009/files is still open. BTW, I suggested a change here too).

The difference you see on transifex is due to the release process. I don't really know these things are piloted.
I would be glad to add extra maintainers to the transifex interface. TBH I would like the project to be fully open and provide complete capacity to anyone to make suggestions. I will play with the interface to see what is possible.
On the validation, I obviously can only review French so I decided to validate all suggestions upfront for other languages. Nothing in French is properly wrong but I agree it could be optimized.

I can help with the Spanish validation if needed.

@dbitouze I gave you reviewer write as you shown lots of interest in the french language, let me know it it gives you the oportunity to suggest modifications.
@trallard normally you received an invite to be owner of the project. @drammock you need to request access to the project first so I can send you an invite.

Got it, thanks @12rambau!

@dbitouze I gave you reviewer write as you shown lots of interest in the french language, let me know if it gives you the opportunity to suggest modifications.

@12rambau Many thanks! It gave me the opportunity, not only to suggest modifications, but to change the translations. That's what I just did for a few of them. I hope it's OK.

That is absolutely fine. I can see a new PR for this so will review it and merge it when I am back tomorrow before making a release.


UI jumps when focusing input field

When focusing an input field in my app, the UI jumps around and appears very glitchy. This has been an issue since way back in the betas, but I never got around to worrying about it much.
I have the keyboard plugin installed and disableScroll(true) is set.
Here is a video of the behavior:
Behavior
Ionic info:
    Cordova CLI: 6.4.0
    Ionic Framework Version: 2.3.0
    Ionic CLI Version: 3.0.0-beta.4
    ios-deploy version: 1.9.0
    ios-sim version: 5.0.6
    OS: macOS Sierra
    Node Version: v6.9.2
    Xcode version: Xcode 8.2.1 Build version 8C1002

Hello, thanks for opening an issue with us. I am going to close this issue as a duplicate of https://github.com/driftyco/ionic/issues/10218.


Fixed bug in examples/fabolas/fmin.py.

It seems that there was an artifact in the implementation of the fabolas optimizer in the given examples from a previous implementation of log-scale sampling of the dataset fraction. This is a small fix that removes the said artifact.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #327 (223106a) into master (fff5743) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #327   +/-   ##
=======================================
  Coverage   89.48%   89.48%
=======================================
  Files         123      123
  Lines        4022     4022
  Branches      459      459
=======================================
  Hits         3599     3599
  Misses        328      328
  Partials       95       95

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 fff5743...223106a. Read the comment docs.


update(proposals): proposal for moving the drivers to S3

Reviewed-by: Spencer Krum<EMAIL_ADDRESS>Reviewed-by: Leonardo Grasso<EMAIL_ADDRESS>Reviewed-by: Leonardo Di Donato<EMAIL_ADDRESS>Signed-off-by: Lorenzo Fontana<EMAIL_ADDRESS>What type of PR is this?
/kind documentation
Any specific area of the project related to this PR?
/area proposals
What this PR does / why we need it:
Explained in the proposal
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign fntlnz
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 9a85c5018d32d38f8c0b522e463873d0829da55f

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[BEAM-2279] Fix archetype breakages

Add version for hadoop-file-system dependency.
Add DirectRunner in test scope in java8, otherwise mvn package fails
when another runner profile is enabled.

R: @davorbonaci
CC: @aviemzur

Coverage decreased (-0.003%) to 70.221% when pulling 7bade36eb18a48e01a8401f97f10a70a492172c4 on dhalperi:fix-archetypes into 89289e498e71d6e9fbd24702032471fa13fbd46c on apache:master.


Validation red message not displayed on step click

I'm submitting a ...  (check one with "x")
[ x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior

When i click on step 2 for example and form is not filled, required red message is not shown, Only when i focus and blur the field
Expected behavior

When click on Step two for example, required fields become red and Required message is displayed
Minimal reproduction of the problem with instructions

Keep fields empty and click on second step
You can test on this link
https://stackblitz.com/edit/form-stepper
Please tell us about your environment:

MacOS, visual code

Angular version: 7.0.1

ngx formly version: 4.8.0

related to https://github.com/angular/components/issues/6728 and unfortunately, there is no easy workaround to make work.


Trying to train on additional wavs I added, getting an out of range error

I added some wavs of my own and got this error.
$ python train.py --output_directory=/home/eagle/p6/tacotron2/all_female/outdir/ --log_directory=logdir
FP16 Run: False
Dynamic Loss Scaling: True
Distributed Run: False
cuDNN Enabled: True
cuDNN Benchmark: False
Epoch: 0
Traceback (most recent call last):
File "train.py", line 284, in
args.warm_start, args.n_gpus, args.rank, args.group_name, hparams)
File "train.py", line 205, in train
for i, batch in enumerate(train_loader):
File "/home/eagle/whale/whale/lib/python3.6/site-packages/torch-1.0.0-py3.6-linux-x86_64.egg/torch/utils/data/dataloader.py", line 637, in next
return self._process_next_batch(batch)
File "/home/eagle/whale/whale/lib/python3.6/site-packages/torch-1.0.0-py3.6-linux-x86_64.egg/torch/utils/data/dataloader.py", line 658, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
IndexError: Traceback (most recent call last):
File "/home/eagle/whale/whale/lib/python3.6/site-packages/torch-1.0.0-py3.6-linux-x86_64.egg/torch/utils/data/dataloader.py", line 138, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/eagle/whale/whale/lib/python3.6/site-packages/torch-1.0.0-py3.6-linux-x86_64.egg/torch/utils/data/dataloader.py", line 138, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/eagle/p6/tacotron2/data_utils.py", line 61, in getitem
return self.get_mel_text_pair(self.audiopaths_and_text[index])
File "/home/eagle/p6/tacotron2/data_utils.py", line 34, in get_mel_text_pair
mel = self.get_mel(audiopath)
File "/home/eagle/p6/tacotron2/data_utils.py", line 42, in get_mel
sampling_rate, self.stft.sampling_rate))
IndexError: tuple index out of range

@MotorCityCobra Why don't change the sampling_rate=22050 in hparams.py?

@WillQuCD
Because I'm using a blend of three datasets. (All about 12 to 15 k utterances).
The LJ dataset is originally in 22050 and M-AILABS wavs were something else.
Fast forward weeks later and I find the M-AILABS had 0.5 secs silence before and after that needed removed.
I'm training now and I'm at 53000 iterations.
Not sure if it's going to work out yet.

@MotorCityCobra
"We have found that it is easier for the model to learn attention if silence is trimmed from the beginning and end of audio files."
Just cite the words from https://github.com/NVIDIA/tacotron2/issues/111
Hope this help for you.
No. I am training on my own dataset.

I am training on my own data. After 70 iterations, it throws the error list index out of range.
Traceback (most recent call last):
File "train.py", line 289, in
args.warm_start, args.n_gpus, args.rank, args.group_name, hparams)
File "train.py", line 207, in train
for i, batch in enumerate(train_loader):
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 582, in next
return self._process_next_batch(batch)
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 608, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
IndexError: Traceback (most recent call last):
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/ubuntu/tacotron2_prev/data_utils.py", line 61, in getitem
return self.get_mel_text_pair(self.audiopaths_and_text[index])
File "/home/ubuntu/tacotron2_prev/data_utils.py", line 32, in get_mel_text_pair
audiopath, text = audiopath_and_text[0], audiopath_and_text[1]
IndexError: list index out of range

I am training on my own data. After 70 iterations, it throws the error list index out of range.
Traceback (most recent call last):
File "train.py", line 289, in
args.warm_start, args.n_gpus, args.rank, args.group_name, hparams)
File "train.py", line 207, in train
for i, batch in enumerate(train_loader):
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 582, in next
return self._process_next_batch(batch)
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 608, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
IndexError: Traceback (most recent call last):
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/ubuntu/tacotron2_prev/data_utils.py", line 61, in getitem
return self.get_mel_text_pair(self.audiopaths_and_text[index])
File "/home/ubuntu/tacotron2_prev/data_utils.py", line 32, in get_mel_text_pair
audiopath, text = audiopath_and_text[0], audiopath_and_text[1]
IndexError: list index out of range

check the sample rate of your audio. probably with a different sr.

I am training on my own data. After 70 iterations, it throws the error list index out of range.
Traceback (most recent call last):
File "train.py", line 289, in
args.warm_start, args.n_gpus, args.rank, args.group_name, hparams)
File "train.py", line 207, in train
for i, batch in enumerate(train_loader):
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 582, in next
return self._process_next_batch(batch)
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 608, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
IndexError: Traceback (most recent call last):
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/ubuntu/anaconda3/envs/tts/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/ubuntu/tacotron2_prev/data_utils.py", line 61, in getitem
return self.get_mel_text_pair(self.audiopaths_and_text[index])
File "/home/ubuntu/tacotron2_prev/data_utils.py", line 32, in get_mel_text_pair
audiopath, text = audiopath_and_text[0], audiopath_and_text[1]
IndexError: list index out of range

check the sample rate of your audio. probably with a different sr.

Yeah, I figured the error. The data which was going in 71st iteration had some errors. Now its working fine and it has trained for nearly 100 iterations. The results are of medium quality. There are noises in the result. Anything I could do to improve the result ?


Free AI Undress Tools to Create Realistic Deepnude Image - Nudify.me

AI Nude Tool—Create Realistic Deepnudes & Deepfakes for Free
Nudify.me is an app that allows users to transform any photo into a realistic nude image of a female. Powered by state-of-the-art technology, we produce uncensored photos that are near-human in realism.
>>>>CLICK HERE TO ACCESS

===
Unleash your creativity and explore the limitless possibilities of digital art with AI-powered image manipulation tools like deepnude, AI nude generators, and undress AI. Try them for free today and revolutionize your creative process.

Free  deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free  deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free  deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free  deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free  deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5

👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
Free deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5
Free deepnude ai : https://securecd-smrtnd.com/s?a=256440&sm=46517&co=316585&mt=5

DeepNude AI offers advanced tools for generating high-quality AI-powered imagery. With features like precision, ease of use, and seamless integration, it caters to beginners and professionals alike. Users can enjoy: https://deepnudefree.online/
Or: https://securecd-smrtnd.com/S9tOCSI3RS
3- Best undress ai for free without sing up : https://7070.lo/
4- surprise For an iPhone at a surprising price : https://cd-sec.com/?a=256440&c=383557&co=316585&mt=21
No cost: Completely free with no hidden charges.
Accessibility: Available on various platforms.
Advanced results: Crisp and accurate outputs.
Why Choose DeepNude AI?
Trusted worldwide for its efficiency, DeepNude AI combines simplicity with reliability, making it an ideal choice for users seeking fast, effective AI solutions

👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/

👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/

👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/

👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/
Free deepnude ai : https://www.3636.bio/

👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇👇Click for a free trial👇
Free deepnude ai :
Free deepnude ai :
Free deepnude ai : https://bt-fr-cl.com/tds/ae?tdsId=s7451sto_r&tds_campaign=s7451sto&s1=ps&utm_source=int&utm_sub=opnfn&subid={subid}&subid2={subid2}&clickid={clickid}l&affid=181c7b69
Free deepnude ai :
Free deepnude ai : https://bt-fr-cl.com/tds/ae?tdsId=s7451sto_r&tds_campaign=s7451sto&s1=ps&utm_source=int&utm_sub=opnfn&subid={subid}&subid2={subid2}&clickid={clickid}l&affid=181c7b69


Fix broken "edit here" button that links from docs to github repo.

Docusaurus ships with a feature where it generates and renders links at the bottom of each page prompting the reader to suggest edits if there should be anything added or fixed.

It looks like medplum's link generator is broken. This edit page url displayed on the doc's intro page leads here: https://github.com/medplum/medplum/packages/docs/docs/intro.md

Instead it should lead here: https://github.com/medplum/medplum/blob/main/packages/docs/docs/intro.md
Looks like the link generator isn't including the "/blob/main" part of the url that should lead to the correct resource. This should be a simple config fix in docusaurus.

Thanks @TimothyStiles nice catch.  @rahul1 can you please take a look at this?

I'm on it


Automate Test With Github Actions

https://i.imgur.com/tZJZOQy.png

The changes to reader.zig are invalid in Zig master (zig master recently renamed tv_sec => sec and tv_usec => usec).
I like the idea, but for master, it needs to use latest Zig. I copied the the github action runner from my http.zig project:
https://github.com/karlseguin/smtp_client.zig/commit/85ead77d23c7994dfd333d07e34ca4f4e659597c

We can download the latest zig from the ziglang website.
On Sun, 11 Aug 2024, 6:41 pm Karl Seguin, @.***> wrote:

The changes to reader.zig are invalid in Zig master (zig master recently
renamed tv_sec => sec and tv_usec => usec).
I like the idea, but for master, it needs to use latest Zig. I copied the
the github action runner from my http.zig project:
85ead77
https://github.com/karlseguin/smtp_client.zig/commit/85ead77d23c7994dfd333d07e34ca4f4e659597c
—
Reply to this email directly, view it on GitHub
https://github.com/karlseguin/smtp_client.zig/pull/6#issuecomment-2282756936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABT5Y5M3PBMH4ZGWMLDXVPDZQ5PJXAVCNFSM6AAAAABMKXYCRGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSG42TMOJTGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


python312Packages.azure-mgmt-cosmosdb: 9.6.0 -> 9.7.0

Changelog: https://github.com/Azure/azure-sdk-for-python/blob/azure-mgmt-cosmosdb_9.7.0/sdk/cosmos/azure-mgmt-cosmosdb/CHANGELOG.md

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 357858

x86_64-linux

:white_check_mark: 8 packages built:

azure-cli
azure-cli.dist
prowler
prowler.dist
python311Packages.azure-mgmt-cosmosdb
python311Packages.azure-mgmt-cosmosdb.dist
python312Packages.azure-mgmt-cosmosdb
python312Packages.azure-mgmt-cosmosdb.dist


changed BackAndroid to BackHandler

BackAndroid is deprecated since 0.44.0 for BackHandler.

ping @grabbou ;)

I think this PR is a dupe of https://github.com/react-community/react-navigation/pull/1333

My bad then. Through I checked.

No worries @Rewieer, thanks for helping out!

Closing in favor of


Change detection issue in ListView

Conditional styling in ListView templates don't get consistently applied. Most times they require the item to go off screen and back in, in order for the new status to be reflected.
Gif

Here is a sample to reproduce:
Component:
import {Component} from "@angular/core";
@Component({
    selector: "my-app",
    templateUrl: "app.component.html",
})
export class AppComponent {
    public options: string[] = ["Alabama", "Alaska", "Arizona", "Arkansas", "California", "Colorado", "Connecticut", "Delaware", "District of Columbia", "Florida", "Georgia", "Hawaii", "Idaho", "Illinois", "Indiana", "Iowa", "Kansas", "Kentucky", "Louisiana", "Maine", "Maryland", "Massachusetts", "Michigan", "Minnesota", "Mississippi", "Missouri", "Montana", "Nebraska", "Nevada", "New Hampshire", "New Jersey", "New Mexico", "New York", "North Carolina", "North Dakota", "Ohio", "Oklahoma", "Oregon", "Pennsylvania", "Rhode Island", "South Carolina", "South Dakota", "Tennessee", "Texas", "Utah", "Vermont", "Virginia", "Washington", "West Virginia", "Wisconsin", "Wyoming"];

    public selectedOption: string = "";

    public onItemTap(args) {
        this.selectedOption = this.options[args.index];
    }
}

Markup:
<StackLayout>
<ListView #listview [items]="options" separatorColor="white" (itemTap)="onItemTap($event)">
    <template let-item="item" let-i="index" let-odd="odd" let-even="even">
        <GridLayout columns="44, *" height="48">
            <StackLayout colSpan="2" verticalAlignment="bottom" class="line"></StackLayout>
            <Label col="1" [text]="item" style="font-size:20" [class.active]="selectedOption == item"></Label>
        </GridLayout>
    </template>
</ListView>
</StackLayout>

CSS
.active {
    color:red;
}

It looks like a change detection issue. The list-view has OnPush change detection strategy (for performance reasons). Probably, thats why the items in it are note updated when selectedOption is changed.
One thing I noticed that if you use tap listener on the grid inside the template - it works as expected:
<GridLayout height="48" (tap)="selectedOption = item">

In this case, because the event is executed in the actual item - it triggers its change detector.

I ran into a very similar issue, but adding a (tap) wasn't possible in my case.
However, I did solve it by changing the instance variable type to BehaviorSubject, and using the async pipe in the template.

I am having the same issue, having a class toggled by an item's property (in a list-view).
Is there a way to manually trigger change detector for an item? In my case the property is changed after a request to an API, so I can't do the change in the layout.

Even though your problem is a little bit strange, you could try this:
import { Component, NgZone } from "@angular/core";

@Component({
    selector: "my-app",
    templateUrl: "app.component.html",
})
export class AppComponent {
    public options: string[] = ["Alabama", "Alaska", "Arizona", "Arkansas", "California", "Colorado", "Connecticut", "Delaware", "District of Columbia", "Florida", "Georgia", "Hawaii", "Idaho", "Illinois", "Indiana", "Iowa", "Kansas", "Kentucky", "Louisiana", "Maine", "Maryland", "Massachusetts", "Michigan", "Minnesota", "Mississippi", "Missouri", "Montana", "Nebraska", "Nevada", "New Hampshire", "New Jersey", "New Mexico", "New York", "North Carolina", "North Dakota", "Ohio", "Oklahoma", "Oregon", "Pennsylvania", "Rhode Island", "South Carolina", "South Dakota", "Tennessee", "Texas", "Utah", "Vermont", "Virginia", "Washington", "West Virginia", "Wisconsin", "Wyoming"];

    constructor(private zone: NgZone) { }

    public selectedOption: string = "";

    public onItemTap(args) {
        this.zone.run(() => {
               this.selectedOption = this.options[args.index];
        });
    }
}

Once I had a problem like that and I fixed it by running the code inside the NgZone.

Trying with NgZone didn't work for me.
With RadListView, changing an item's property updates immediately the UI, but since it has a memory leak (an issue was created on telerik/nativescript-ui-feedback) and kills my app after a while, I can't use it.
With ListView, the UI is not updated. Is there any alternative to NgZone that could work?

After trying some modifications to my code, I could solve this issue by doing:
...
this.items[i].is_list = true;
this.items = this.items.slice();

That way, when reassigning to this.items it's own array, the UI got updated.

Same problem. Thanks @eduardoturconi , that worked for me.

Worth noting: This isnt just on angular, I am experiencing this in a Typescript/vanilla app. I fix it by calling refresh() on the listview, though that shouldnt be necessary.

This happens only on iOS, works correctly on Android.

Experiencing the same problem.  The issue is specific to IOS for me as well.
Calling refresh() on the listview was the only suggestion to resolve the problem, though i'm hopeful that it is a temporary resolution to the bigger problem. Calling refresh() is causing the view to flash, which looks odd and makes for a poor UX.
Hope this is still being looked into.

Flashing problem as described by @DillonStreator can be "solved" by overwriting the default behaviour as seen here

@Dagorwolf  how to import UITableViewCellSelectionStyle ?

Is there any possible option to fix this if the source collection doesn't change? IF i'm changing only property visibility="visible || collapsed" of the view of  tapped ITEM .
I've tried @eduardoturconi  .slice() method and others like:

listView = this.page.getViewById("listView")
listView.refresh();
BehaviourSubject and BehaviourSubject with NgZone.run
@Dagorwolf  method;
!!! Nothing work correctly
Please, would appreciate any ideas.

I solved this problem as described here.
Just add this:
    Object.defineProperty(View.prototype, "isLayoutValid", {
        get: function () {
            return (this._privateFlags & PFLAG_LAYOUT_REQUIRED) !== PFLAG_LAYOUT_REQUIRED;
        },
        enumerable: true,
        configurable: true
    });

in
node_modules/tns-core-modules/ui/core/view/view.ios.js

And everything was magically fixed!

Have the same issue, tried https://angular.io/api/core/ChangeDetectorRef  but no success, tried ngZone and no success. Is it related to ListView? e.g If I change to a StackView with ScrollView around would it work?

@drmistral , if Im using angular and I would change  view.ios.js, wouldn't it override this file on build?

@ShyshkovOleg, I'm not using angular but anyway I put view.ios.js under git control, just to detect any overwriting.
Of course the best solution would be fix this problem at the root but meanwhile...

We're seeing this on Android when using the async pipe. A workaround we found was using the itemsLoading event on ListView, and then do a markForCheck() inside a setTimeout.
So in template
<ListView [items]="data | promisePipe | async" (itemLoading)="refresh()"></ListView>
In TS file:
refresh() {
   setTimeout(() => this.changeDetectorRef.markForCheck(), 0); // Queue macro event
}

Is there any news to this story? I'm having similar issues on IOS. I have tried so many different things now and nothing helps.

well, I believe this is why I finally give up on the nativescript, a serious bug exists nearly 3 years.

I encountered the same issue. I tried to solve it by re-assigning my array to the ListView component, but once every n times my app crashes returning the following error:
"TypeError: null is not an object (evaluating 'cell.view')"
Anybody knows what the source of the problem may be?

I have been feeling the same thing, my employer wanted me to try to build an app with a javascript library to test if it is a way to build other apps in the future. And when I get stuck on things like this I'm wondering if there are more holes to fall down.
I actually found a way to "solve" my issue, well it did not solve the issue but it is a way to list things without using the list view. You can see the issue here.

Is switching to RadListView a viable solution to this issue?

If you commented on my comment, I have been using radlistview from the beginning.

This is a major issue. Is there any way to get it moved off of the backlog? @NickIliev @tsonevn ?
Thank you for your consideration.

@Roar1827 or @Kraften, do you have a repository that shows this issue?
From my comment back in Dec 2016, it was able to get around this issue by using RxJS. I was about to build a little demo for you, but when I copy/pasted the code from the original post...it's working without resorting to Observables.
Here's my component, the only change I needed to make from the above code was to change <template/> to <ng-template/>: https://github.com/jzgoda/tns-listview-change-detection/tree/master/src/app

@jzgoda @Roar1827 @Kraften @Zampa98 @steve3d @larssn
I might be missing something on the way, but while using the original snippet with the latest tns-core-modules and nativescript-angular and I am experiencing this issue. Here is a Playground demo showing the case -  tested on both iOS and Android (all the related code is in app.component) and on both the itemTap is triggering the change detection. Please do let me know if you guys have a different case and share a Playground (or sample app) that demonstrates the issue.

Thank you for looking into this @NickIliev, I appreciate it very much.
I've tried to create a simple playground demo from extending your playground.
The problem occurs when we use [visibility] to conditionally display something when a list item is selected. Perhaps we are doing this the wrong way? Calling refresh() on the listView worked in NS 4.1 but no longer seemed to work @latest

@Roar1827 indeed you are dealing with a totally different problem which is not related to this issue or the Angular state management but to the way you are handling the state of each ListView cell. You still need to call refresh on iOS but you also need to change an individual boolean flag for each item and not a global boolean variable.
Here is a Playground demo demonstrating on how you could approach the problem with creating a boolean flag (isExpanded) for each item of your list. Then use this boolean flag to change only tapped cell (based on the passed index). Due to specifics with the measurements on iOS, you will still need to call refresh method on the listview (see the onListViewLoaded method used to get a reference for the ListView).

Closing the issue as resolved (see this comment) - if anyone still experiences a similar problem please ping me with the details needed to investigate the case (including a sample app or Playground demo)


Crash when xgb.cv is run for some time with GPU support

Hi,
I'm running with GPU support on, and the system crashes sometimes with the following message. (Logs attached)
python': malloc(): smallbin double linked list corrupted
Configuration:
'gpu_id': 0,
'max_bin': 16,
'tree_method': 'gpu_hist',
Environment info
Operating System:
Ubuntu 16.04.1
Compiler:
gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.5)
nvcc: NVIDIA (R) Cuda compiler driver
Built on Tue_Jan_10_13:22:03_CST_2017
Cuda compilation tools, release 8.0, V8.0.61
Package used (python/R/jvm/C++):
python
xgboost version used:
xgboost-0.6-py3.6.
If installing from source, please provide

The commit hash (git rev-parse HEAD)
9a81c74a7bd8db06c2f2ccb28b88ffeecf4c0309
Logs will be helpful (If logs are large, please upload as attachment).

If you are using jvm package, please

add [jvm-packages] in the title to make it quickly be identified
the gcc version and distribution

If you are using python package, please provide

The python version and distribution
Anaconda, Python 3.6
The command to install xgboost if you are not installing from source
(from source)

Steps to reproduce
Extremely difficult, happens randomly after a long run.
What have you tried?
Any advice? Not sure what to try even.
error.txt

According to the stack trace the issue originated from gnu parallel sort during the AUC metric calculation. Does the error still occur if you use the 'error' evaluation metric instead of AUC?

Well, I think I'm one of those affected by the not so rare AMD Ryzen bug where when stressed, programs segfault randomly. I bricked my mobo trying to get a new version of the BIOS. Finally got the RMA and tested the code after disabling opcache and SMT. So far so good.. Have been running the same code close to an hour and no segfaults yet :)


fake_process is incompatible with anyio

fake_process appears to be incompatible with code that uses anyio with asyncio.
 In the real test we're testing a FastAPI-based server to see if it invokes processes when it should, and FastAPI is based on anyio.

Thanks for the report. To be honest, I have no experience with anyio so I need to dig into this to figure out how to fix it.
Unfortunately, I don't really have time to do that, probably I won't be able to work on this until the end of the month.
Feel free to submit a PR.


build system bug

I have just installed west as was directed here: https://docs.zephyrproject.org/latest/develop/getting_started/index.html
When I attempted to build the blinky sample, it failed apparently because a %HOMEPAGE% that contains an embedded space is incorrectly parsed.  This is evinced by these two if statements starting at about line 77 in the Kconfig.makefile:
if(CONF_FILE)
string(CONFIGURE "${CONF_FILE}" CONF_FILE_EXPANDED)
string(REPLACE " " ";" CONF_FILE_AS_LIST "${CONF_FILE_EXPANDED}")
endif()
if(EXTRA_CONF_FILE)
string(CONFIGURE "${EXTRA_CONF_FILE}" EXTRA_CONF_FILE_EXPANDED)
string(REPLACE " " ";" EXTRA_CONF_FILE_AS_LIST "${EXTRA_CONF_FILE_EXPANDED}")
endif()
This code seems to be processing a list of path names and using a space as a delimiter.  This will fail when the %HOMEPATH% contains an embedded space as does mine, "C:\Users\Jerry Easley"; it splits this string into two "C:\Users\Jerry" and "Easley" neither of which can be found.  Since both Windows and Linux allow imbedded spaces within path names, I think this is a bug.
I also had an installation failure due to the same problem, however, I was able to work around that problem.
To Reproduce
Steps to reproduce the behavior:

Just create a %HOMEPAGE% which contains an embedded space such as "C:\Users\Firstname Lastname"
install west
build blinky

Expected behavior
I expect that you will receive a "Fatal Error" on about line 275 in the Kconfig.makefie
Impact
I guess I could work around this problem by changing my directory structure and eliminating the embedded space but I'm afraid I would have reinstall quite a bit of other software
Logs and console output
If applicable, add console logs or other types of debug information
e.g Wireshark capture or Logic analyzer capture (upload in zip archive).
copy-and-paste text and put a code fence (```) before and after, to help
explain the issue. (if unable to obtain text log, add a screenshot)
Environment (please complete the following information):

OS: Windows 10
Toolchain Zephyr SDK

Regards,
Jerry

Thanks for the detailed bug report! The title is surprisingly vague though, can you please rephrase with some of the specific keywords you used in the description?
You found one place where the build system does not support whitespace in paths. I'm afraid there are others :-( This limitation should probably be documented. I thought I remembered seeing that mentioned in the documentation but I can't find it right now.

This is a known limitation and is listed in the documentation https://docs.zephyrproject.org/latest/develop/application/index.html#creating-an-application-by-hand

Well I guess I can live with this limitation, but I would like to point out that I was not apprised of this problem; neither of the notices you cite are in the "Getting Started" documentation; it would be nice to put a similar notice in the getting started documentation, better yet put in a check in the installation procedure.

Thanks @nordicjm for digging out two places where this limitation is documented!
@JerryDennisEasley I agree that this is not documented in the right place. By the time the user reads the "Beyond Getting Started" guide or the "Creating an application by hand" page then it's too late: the build will have already failed - as you experienced. This note must be moved earlier; in the getting started guide at the point where zephyr code gets downloaded for the first time. Can you please submit a documentation fix that simply moves this note? Thanks in advance.
I renamed and re-opened this issue as a documentation enhancement.


Adds Python Support for elasticlib

Used Black Formatting and WPI Format

Already am @Ryanforce08

This is a duplicate PR (#136 already exists)

Dang I did not see. yours is better anyways

i would recommend closing the PR then

Just realized mine has better docs is there away to add my docs to yours

Just realized mine has better docs is there away to add my docs to yours

I am copying the docs from the Java & C++ versions


[off] needs to be supported to allow for comments not intended for the record

Addressed in ca7ceca


add deferrer plugin

Codecov Report

Merging #13 (c9c9d8c) into master (2143caa) will increase coverage by 0.21%.
The diff coverage is 96.42%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   86.01%   86.23%   +0.21%
==========================================
  Files          58       60       +2
  Lines        1266     1293      +27
  Branches      145      147       +2
==========================================
+ Hits         1089     1115      +26
- Misses        165      166       +1
  Partials       12       12

Impacted Files
Coverage Δ

vedro/plugins/deferrer/_deferrer.py
95.83% <95.83%> (ø)

vedro/__init__.py
58.97% <100.00%> (+1.07%)
:arrow_up:

vedro/plugins/deferrer/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Enhance subgraph API

@reminisce @zheng-da @apeforest @szha  @pengzhao-intel
PR Major change:

Introduce SubgraphSelectorV2 that accepts SimpleNode instead of nnvm::Node to get full context of visited node. SimpleNode is double linked version of  nnvm::Node, which can list the output consumer of node. This is very useful to help make selection which has output requirement. Wrapper class SubgraphSelectorV2Bridge is added to convert SubgraphSelector to SubgraphSelectorV2, which allows developer to continue using old SubgraphSelector. So this feature is fully back-compatible.

Introduce kAdjust mode for SubgraphProperty to allow adjusting network parameters instead of create new node. New API AdjustSubgraphNode() is added to  SubgraphProperty for this, which can adjust selected nodes attributes. This is useful when we don't want to create new subgraph node, but to do some modification on selected nodes. Quantized scale alignment optimization is pending on this feature.

Change the behavior of PartitionGraph to create single subgraph node for nodes isolating with each other. https://cwiki.apache.org/confluence/display/MXNET/Enable+Operator+Level+Parallelism+under+Subgraph is depending on this change.

Add new subgraph pass "quantized scale alignment optimization" as an example for new subgraph API.

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@KellenSunderland @TaoLv

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

@ZhennanQin could you re-trigger the CI?

@ZhennanQin Could you please re-trigger the CI and see why some of the checks are still waiting? Thanks !

@anirudh2290 @reminisce please help review the continuous improvements for subgraph after #14277

@zhennanqin @xinyu-intel please resolve the comments and re-trigger the PR.
We need to speedup the progress of this changes :)

Thanks for your contribution. Merging now.


HDC Tracker has new Domain Name

Old Domain: https://hdcenter.cc/
New Domain: https://hdc.bz/
Regards

Added to v13.8.7


Timed out connecting to host - Connection to device with authentication via Tacacs

Describe the bug
Two similar setup of switches:

Authentication local - scrapli works ✅
Authentication via tacacs creds - scrapli fails

To Reproduce
Steps to reproduce the behavior:
1.
from scrapli import Scrapli
import logging
from datetime import datetime

logging.basicConfig(
    format = '%(threadName)s %(name)s %(levelname)s: %(message)s',
    level=logging.INFO)

r1 = {
   "host": "<IP_ADDRESS>",
   "auth_username": "admin",
   "auth_password": "123456",
   "auth_secondary": "123456",
   "auth_strict_key": False,
   "platform": "cisco_iosxe",
    "timeout_socket": 5,
    "timeout_transport": 10
}

#no enable password required to login via tacacs
r2 = {
   "host": "<IP_ADDRESS>",
   "auth_username": "user_at_tacacs",
   "auth_password": "123456",
   "auth_strict_key": False,
   "platform": "cisco_iosxe",
    "timeout_socket": 5,
    "timeout_transport": 10
}

def get_session():
    with Scrapli(**r2) as ssh:
        logging.info(datetime.now())
        print(ssh.get_prompt())

if __name__ == "__main__":
    get_session()

What you're connecting to (vendor, platform, version)
Cisco, Catalyst 9300, 17.3.3
Anything else relevant
Replacing r1 and r2 in script manually - expecting r2 to execute succesfully as r1 does.

Expected behavior
Replacing r1 and r2 in script manually - expecting r2 to execute succesfully as r1 does.
Stack Trace
Copy of your stack trace here, please format it properly using triple back ticks (top left key on US keyboards!)
MainThread scrapli INFO: Driver '<class 'scrapli.driver.core.cisco_iosxe.sync_driver.IOSXEDriver'>' selected from scrapli core drivers
MainThread scrapli.driver INFO: opening connection to '<IP_ADDRESS>' on port '22'
ThreadPoolExecutor-0_0 scrapli.channel CRITICAL: Timed out connecting to host
Traceback (most recent call last):
  File "/Users/lucky/untitled/scrapli_test.py", line 40, in <module>
    get_session()
  File "/Users/lucky/untitled/scrapli_test.py", line 34, in get_session
    with Scrapli(**r2) as ssh:
  File "/Users/lucky/untitled/venv/lib/python3.8/site-packages/scrapli/driver/base/sync_driver.py", line 40, in __enter__
    self.open()
  File "/Users/lucky/untitled/venv/lib/python3.8/site-packages/scrapli/driver/base/sync_driver.py", line 86, in open
    self.channel.channel_authenticate_ssh(
  File "/Users/lucky/untitled/venv/lib/python3.8/site-packages/scrapli/decorators.py", line 276, in decorate
    return self._multiprocessing_timeout(
  File "/Users/lucky/untitled/venv/lib/python3.8/site-packages/scrapli/decorators.py", line 337, in _multiprocessing_timeout
    return future.result()
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/concurrent/futures/_base.py", line 432, in result
    return self.__get_result()
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/concurrent/futures/_base.py", line 388, in __get_result
    raise self._exception
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/Users/lucky/untitled/venv/lib/python3.8/site-packages/scrapli/channel/sync_channel.py", line 291, in channel_authenticate_ssh
    self._ssh_message_handler(output=authenticate_buf)
  File "/Users/lucky/untitled/venv/lib/python3.8/site-packages/scrapli/channel/base_channel.py", line 280, in _ssh_message_handler
    raise ScrapliAuthenticationFailed(msg)
scrapli.exceptions.ScrapliAuthenticationFailed: Timed out connecting to host

Screenshots
If applicable, add screenshots to help explain your problem, but do note that formatted text is much preferred over
screenshots!
OS (please complete the following information):

OS: [e.g. Ubuntu, MacOS, etc. - Note scrapli is not thoroughly tested on Windows and some/many things will not
be supported]
macOS Catalina
scrapli version
2021.7.30

Additional context
Add any other context about the problem here.

Hey @timofmax thanks for raising this!
Could you show what the auth/login process looks like for the host with tacacs please? That and if you could snag logs (example here) that would be great. We basically do pattern based searching for user/pass inputs when using system transport so most likely culprit here is that the pattern is not matching whatever the tacacs prompts are.
You can also give this a shot w/ ssh2 or paramiko transports -- since those work "out of channel".
Thanks!
Carl

Hey Carl, appreciate your swift response.
here is auth/login process
lucky@M-M8KB:~$ ssh<EMAIL_ADDRESS>password:
Authentication succeeded. Your password will expire in 10 weeks + 1 days  + 18 hours  + 16 Minutes

sw.with.tacacs.auth#
sw.with.tacacs.auth#

here is logs
ID    | TIMESTAMP               | LEVEL    | (UID:)HOST:PORT           | MESSAGE
1     | 2021-08-22 15:56:53,823 | DEBUG    |                           | Scrapli factory initialized
2     | 2021-08-22 15:56:53,823 | INFO     |                           | Driver '<class 'scrapli.driver.core.cisco_iosxe.sync_driver.IOSXEDriver'>' selected from scrapli core drivers
3     | 2021-08-22 15:56:53,824 | DEBUG    | <IP_ADDRESS>:22              | load core transport requested
4     | 2021-08-22 15:56:53,827 | DEBUG    | <IP_ADDRESS>:22              | core transport 'system' loaded successfully
5     | 2021-08-22 15:56:53,827 | DEBUG    | <IP_ADDRESS>:22              | created transport 'open_cmd': '['ssh', '<IP_ADDRESS>', '-p', '22', '-o', 'ConnectTimeout=5', '-o', 'ServerAliveInterval=10', '-l', 'admin', '-o', 'StrictHostKeyChecking=no', '-o', 'UserKnownHostsFile=/dev/null', '-F', '/dev/null']'
6     | 2021-08-22 15:56:53,827 | DEBUG    | <IP_ADDRESS>:22              | generating combined network comms prompt pattern
7     | 2021-08-22 15:56:53,827 | DEBUG    | <IP_ADDRESS>:22              | setting 'comms_prompt_pattern' value to '(^[\w.\-@/:]{1,63}>$)|(^[\w.\-@/:]{1,63}#$)|(^[\w.\-@/:]{1,63}\([\w.\-@/:+]{0,32}\)#$)|(^([\w.\-@/+>:]+\(tcl\)[>#]|\+>)$)'
8     | 2021-08-22 15:56:53,827 | INFO     | <IP_ADDRESS>:22              | opening connection to '<IP_ADDRESS>' on port '22'
9     | 2021-08-22 15:56:53,827 | DEBUG    | <IP_ADDRESS>:22              | opening transport connection to '<IP_ADDRESS>' on port '22'
10    | 2021-08-22 15:56:53,856 | DEBUG    | <IP_ADDRESS>:22              | transport connection to '<IP_ADDRESS>' on port '22' opened successfully
11    | 2021-08-22 15:56:53,856 | DEBUG    | <IP_ADDRESS>:22              | attempting in channel ssh authentication
12    | 2021-08-22 15:56:54,038 | DEBUG    | <IP_ADDRESS>:22              | read : b"Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.\npassword: "
13    | 2021-08-22 15:56:56,368 | DEBUG    | <IP_ADDRESS>:22              | write: REDACTED
14    | 2021-08-22 15:56:56,369 | DEBUG    | <IP_ADDRESS>:22              | write: '\n'
15    | 2021-08-22 15:56:56,369 | DEBUG    | <IP_ADDRESS>:22              | read : b'\nAuthentication succeeded. Your password will expire in 10 weeks + 1 days  + 18 hours  + 8 Minutes \n'
16    | 2021-08-22 15:56:56,415 | DEBUG    | <IP_ADDRESS>:22              | write: REDACTED
17    | 2021-08-22 15:56:56,415 | DEBUG    | <IP_ADDRESS>:22              | write: '\n'
18    | 2021-08-22 15:56:56,415 | DEBUG    | <IP_ADDRESS>:22              | read : b'123456\n\nsw.with.tacacs.auth#'
19    | 2021-08-22 15:56:56,504 | DEBUG    | <IP_ADDRESS>:22              | write: '\n'
20    | 2021-08-22 15:56:56,505 | DEBUG    | <IP_ADDRESS>:22              | read : b'123456\n'
21    | 2021-08-22 15:57:06,515 | CRITICAL | <IP_ADDRESS>:22              | transport operation timed out, closing transport
22    | 2021-08-22 15:57:06,516 | DEBUG    | <IP_ADDRESS>:22              | closing transport connection to '<IP_ADDRESS>' on port '22'
23    | 2021-08-22 15:57:06,621 | CRITICAL | <IP_ADDRESS>:22              | encountered EOF reading from transport; typically means the device closed the connection
24    | 2021-08-22 15:57:06,724 | DEBUG    | <IP_ADDRESS>:22              | transport connection to '<IP_ADDRESS>' on port '22' closed successfully

Perfect, thank you @timofmax !
Looks like this is being caused by some lazy password pattern checking I built into the "in channel" ssh auth. Basically we check for "password" in the bytes we read from the server... if we see that we assume we are at a password prompt and send the password and a return (hence the REDACTED showing up more than it should be in there).
I was actually thinking I should fix this the other day since I did a slightly better job implementing it on the go side over in scrapligo. Let me try to get a branch pushed up that you can try out. Also if you are in a pinch -- this should not be an issue with the other transports, so that could be a workaround in the meantime if ya need it!
Will crank out that branch and update shortly!
Carl

paramiko

Dear @carlmontanari thank you for advice
with parameters ssh2 or paramiko works perfectly.

Hey @timofmax
If you don't mind -- could you try to install the in-channel-ssh-improvements branch... I made some improvements that I think should fix this issue. You can install it like: pip install git+https://github.com/carlmontanari/scrapli.git@ in-channel-ssh-improvements#egg=scrapli.
I'll open a pr for that and if it works for ya get it moved over to develop branch!
Carl

Gunna close this and merge #168 -- but feel free to reopen if there are still issues!
Carl


Add an example

The typical socket.io example is a chat server. This tackles the concept of namespaces as well as a persistent communication with multiple clients. If you're looking for the server-side code, have a look here: https://github.com/socketio/socket.io/tree/master/examples/chat

The typical socket.io example is a chat server. This tackles the concept of namespaces as well as a persistent communication with multiple clients. If you're looking for the server-side code, have a look here: https://github.com/socketio/socket.io/tree/master/examples/chat

Hey is there any updates on this? I'm quite interested in the library but I'd like to have a demo for the chat example provided

Hi @WildEgo, nice that you're interested in the library! Until now I didn't find the time for adding that example (that's why I also added the good first issue label ;) Feel free to add the example and ask questions if they arise, I am happy to assist!

@1c3t3a I just used your crate recently and have some spare cycles I could use to make a Rust client example. Was the thought to use the example chat as a server and implement a cli chat client?

Happy to hear that! :) The example can be similar to the one in the socket.io tree (https://github.com/socketio/socket.io/tree/master/examples/chat). A CLI (or just some executable binary) would be super nice to show the capabilities!

ok awesome. I'll get started on it soon. Just to make sure I don't poison the well by recommending bad practices, is the way I blocked to wait for the WS responses here a "legit" way of doing it or is there a more canonical way?

This looks fine! If you open a PR I can also review for readability of the code :)

Any updates here?


tensorflow.python.framework.errors_impl.NotFoundError: libtensorflow_framework.so: cannot open shared object file: No such file or directory

I run the code sparse attention on p100, but it goes wrong.
System information

OS Platform and Distribution: ubuntu 18.04
TensorFlow installed from: pip install
TensorFlow version: 1.14.0(gpu)
Python version: 3.6.5
GCC: 7.4.0
CUDA version: 10.0
GPU model and memory: P100

Here's the traceback
Traceback (most recent call last):
  File "/home/zyh/sparse_attention-master/attention.py", line 4, in <module>
    from blocksparse import BlocksparseTransformer
  File "/root/anaconda3/lib/python3.6/site-packages/blocksparse/__init__.py", line 3, in <module>
    from blocksparse.utils import (
  File "/root/anaconda3/lib/python3.6/site-packages/blocksparse/utils.py", line 16, in <module>
    _op_module = tf.load_op_library(os.path.join(data_files_path, 'blocksparse_ops.so'))
  File "/root/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/load_library.py", line 61, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: libtensorflow_framework.so: cannot open shared object file: No such file or directory

I check the libtensorflow_framework.so through find . -name libtensorflow_framework.so , however, it doesn't exist. Next, I find libtensorflow_framework.so1 at /root/anaconda3/lib/python3.6/site-packages/tensorflow/, so I copy the libtensorflow_framework.so1 to the libtensorflow_framework.so, and I append it to LD_LIBRARY_PATH through export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:"path/to/your/libtensorflow".
It still not works.
Please help.
Thanks in advance

I also the get same problem with very similar environment, except that my Tensorflow was installed from Conda.

I'm fairly certain that you are using a tensorflow version that is too new. See here: https://github.com/tensorflow/tensorflow/issues/30175
Tensorflow changed the name of its shared object file in version 1.14. Maybe 1.13 will work for you, though. You can also install blocksparse from source to avoid this problem. Only Scott or someone from openai can comment on what version of tensorflow the wheel installed by pip was linked against.
My guess is that the SONAME of that file also changed as well, which is why your copying may not be working. I am not sure, though, and won't debug this myself.

I installed tensorflow 1.13.1 and ran the code, but it goes wrong again.

I'm fairly certain that you are using a tensorflow version that is too new. See here: tensorflow/tensorflow#30175
Tensorflow changed the name of its shared object file in version 1.14. Maybe 1.13 will work for you, though. You can also install blocksparse from source to avoid this problem. Only Scott or someone from openai can comment on what version of tensorflow the wheel installed by pip was linked against.
My guess is that the SONAME of that file also changed as well, which is why your copying may not be working. I am not sure, though, and won't debug this myself.

I came with the same problem. It turns out to be using horovod 0.16 with tensorflow 1.14. Force reinstalling horovod 0.15 and the problem disappears.


feat: starting work on the episodes page

@rawkode Is it possible that you can add an id property for each episode that I can use as a URL path. I'm currently using the YouTube Id at the moment.

@rawkode ready to review

Did you merge main?

There was a merge conflict so I rebased. Is that an issue?

Yeah. It’s not rebased correctly. I’ll take a look tonight

Re-created all the changes in another branch


Loading modals in current scene

Load modal-like scenes to load in the current one. Maybe a method called load_modal and the scene loaded is overlaid on the current one by choosing a parent node. Also a rollback/close_modal method to hide modals in open order.
I need suggestions for better method names.

Preview modal support. You can use SceneManagerBackdrop to add a modal-like Control. Simple opacity animation for now.


Add support for K2DTMC stochastic process + steady states

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5844

Relevant Lines:
14504

💛  - Coveralls


Scan: Add "orderBy" parameter.

This patch adds an API for requesting non-time orderings, although it does not actually add the ability to execute such queries.
A key motivation is to enable ORDER BY tracking for INSERT described in #11929, which is why this patch is still useful even if it doesn't add the ability to execute Scan queries with non-time ordering. But I do think that would also be very useful, so I do think we will eventually add execution code for this parameter.
The changes are done in such a way that no matter how Scan query objects are constructed, they will have a correct "getOrderBy". This will enable us to switch the execution to exclusively use "getOrderBy" later on when it's implemented.
Scan queries are serialized such that they only include "order" (time order) if the ordering is time-based, and they only include "orderBy" if the ordering is non-time-based. This maximizes compatibility with the existing API while also providing a clean look for formatted queries.
Because this patch does not include execution logic, if someone actually tries to run a query with non-time ordering, then they will get an error like "Cannot execute query with orderBy [quality ASC]".

This pull request introduces 1 alert when merging 3368d8d into 1487f55 - view on LGTM.com
new alerts:
* 1 for Inconsistent equals and hashCode

This is a false alarm due to the weird JsonIgnore API.


Update untildify to version 4.0.0

because untildify has a breaking change in the new update https://github.com/sindresorhus/untildify/releases/tag/v5.0.0 set the untildify version to 4.0.0.

Please merge this! We're running into the same problem with our Github Actions


Merge consecutive whitespace & correctly handle indentation and trails

This PR does two things. First, it fixes #21 by, after an item is parsed, determining whether both it and the previous one were Item::WS instances. If so, it replaces the old instance instead of appending the new one. It uses only safe code, doesn't allocate anything extra, and doesn't require a second pass. I've also added two test cases, both to the reconstruction module because this change doesn't change the reproduction side at all.
The second thing I've done in this PR is fix a bug that I discovered while running my test cases. Any indentation before a key was being either prepended to the key or appended to the trail of the line before it. As it turned out, there were two places in parser.rs where self.idx was being moved back to an earlier character but one or both of self.chars and self.current were not. Since those should always be in sync, I added the methods save_idx and restore_idx, which save and update all three variables together. I chose the suffix "idx" because that variable seems the most fundamental to me: it is effectively the iterator, and cursor and chars exist to give information about its surroundings.
Note that in order for the tests to show anything useful for this PR, I had to uncomment the line in reconstruction.rs that says assert_eq!(parsed, under_test);.Otherwise, it would have just been a string comparison with the source file's contents, and my change doesn't affect that string--just how it's split up internally. As a result, the AoTs test is now failing. I haven't looked at it in depth, but it looks like the nesting is different than expected. Let me know if that wasn't a known issue and I'll put it in the tracker.

The AoTs were just not including their first elements. I added a check for an AoT in the main parse loop, so now the AoTs test passes even when it actually compares the ASTs.
Also, in order to get the test to pass, I realized that I had to explicitly implement Hash for the Key type. The derived hashing didn't match the custom PartialEq implementation, so most de facto equality checks (i.e. seeing if a given key was already in the HashMap) depended on the whole Key structure instead of only its key member.

Hey, thanks for doing this!
Before merging, I'd like to ask you to move the nested if let conditions into some kind of consecutive_ws(body, value) function to minimize repetition and rightwards drift, especially since parse_table() is already a heavy function.
You could also have that function handle the code inside the condition as well and return true if combination happened. This lets us lose the combined flag and use it like:
if !merge_ws(body, value) {
    body.append(key, value).chain_err(|| self.parse_error())?;
}

It does mean having side effects in a condition evaluation, but it's very clean looking and if documented adequately, shouldn't be a problem. We already do this with inc().

Good point. The code duplication was bugging me a little anyway, so I'm glad you pointed it out.
By the way, assuming I haven't made any logic errors, my PR for #38 will make parse_table much simpler, since I'm moving the logic for handling nested tables elsewhere.

Ok this looks great now, thanks a lot!


null values

probably not a issue for this library, but thought I would ask.
Im using on a Wemos d1 mini (along with 9 more), and only plugging into a BVM712.  im Trying to read and send to MQTT server, which it does, and its great for a few seconds, then nearly all values start coming in as null.
object PID: "0XA381" V: null I: null P: null CE: null SOC: null TTG: "-1" ALARM: "OFF"
im not really doing anything crazy to the data, so Im just wondering if the timing is off?  possbily with loop?
void ReadVEData() {
    while ( Victron.available() ) {
        myve.rxData(Victron.read());
    }
    yield();
}

void EverySecond() {
    static unsigned long prev_millis;
    if (millis() - prev_millis > 1000) {
        SendData();
        prev_millis = millis();
    }
}

void SendData(){
  String jsonstring;
   for ( int i = 0; i < myve.veEnd; i++ ) {
      json[myve.veName[i]] = myve.veValue[i];
   }
   serializeJson(json, jsonstring);
   client.publish( "zorro/solar/bvm", jsonstring);
   Serial.println(jsonstring);
}
`

Simply not an issue with this library.  its something with my json.    After a full day of tinkering, I just now realized why null is coming in.... due to the json. Sorry to have created this ticket.


[bug] Distributed FIFOQueue dequeue duplicate data

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu 14.04
TensorFlow installed from (source or binary):
binary
TensorFlow version (use command below):
1.0.0
Bazel version (if compiling from source):
CUDA/cuDNN version:
8.0 5.1
GPU model and memory:
Pascal TitanX
Exact command to reproduce:
https://gist.github.com/lispc/27c9f4fe935abade5de90c18276e1742

Describe the problem
In distributed tensorflow, if I do multiple dequeue from a remote FIFOQueue, duplicate results will be popped. For example, 1 2 3 4 5 were pushed into the FIFOQueue, then the popped results may be 1 1 2 3 4 4 5. Is it a bug? Or designed feature?
The code is posted above. Logs is also in that gist.

This is a bug fixed in 1.1.
#7038

Please re-open if upgrading to 1.1 does not solve the problem.


Remote Node

Possibly my ignorance here.
But if I am configuring the MQTT topic to a specific device, doesn't that mean that device would need an MQTT connection.
Therefore if I have a mobile device that has no mqtt connection, I won't see any of the info as it will be posted as a protobuf on another devices topic?

Your mobile device could provide a MQTT connection to then Meshtastic node. There's a checkbox in configuration for that

But doesn't that negate the use of a Lora based device. If I have a mobile device then I have no need for the Meshtastic.
I am not 100% sure on the MQTT of Meshtastic, but I'm assuming that a remote node information is in the protobuf of another node as they are relaying the information.

I think what mukowman meant was the possibility to also process messages from other nodes which are published on the same channel the integration is listening on. This makes a lot of sense as it will allow users to integrate the whole meshtastic mesh into HA, not only the local MQTT bridge device. This could be combined with #9

Wouldn't that require you to know which node is sending the traffic?

Yes it does. But you can see the user id for any node connected to the mesh. I monitor my two non mqtt nodes this way thru my mqtt node via this integration.


Allow TS, TSX files for eslint

ive never actually edited the eslint config file so im just adding an issue here for it.
as it currently stands, they dont work for .ts and .tsx

closing as its been addressed already


"Try Out" section updated for Alderaan

As outlined in issue #393

I moved "Try Out the Raiden Demo App" to its own section instead of having it nested within the "Getting Started" section.
It felt a bit misplaced there because "Getting Started" is more about working with the code base. For instance right after the Demo App tutorial the following section would appear:
To run the code in this repository, you must have Node.js 10+ on your computer and a web3-enabled browser (e.g. Firefox with Metamask extension), as well as some ETH on the account.


time_t should not be used directly

fixes: #1451

Your local copy seems out of date, I changed it to Duration a few days ago.

Your local copy seems out of date, I changed it to Duration a few days ago.

Right, I saw that when github complained about the merge issue. I have other commits in this PR that are still relevant.

This needs a rebase

This needs a rebase

rebased

source/agora/test/SlashingMisbehavingValidator.d(193,16): Error: undefined identifier time_t

For ref I just rebased on master again

LGTM, just a question about the comment which seems unclear.

rebased again, also fixed the comment(it was messed up by VSCode find and replace)


Update issue templates to include angular v2

Updating Github issue templates to include Angular v2.

Coverage decreased (-11.2%) to 70.27% when pulling a061ea3a4a8a530ed5d92fe9a68e266ec4e76fd2 on update-issue-template into f157e65a7e9862571d3028f5999b8e4296650e99 on dev.


movement plot

after mapping ... points might move ... it would be cool to allow this to be plotted.
something like
emb_proj = embset | (emb["man"] - emb["woman"])
embset.comparison_plot(emb_proj)

progress


[FLINK-14488][python] Update python table API with temporary tables & views methods

What is the purpose of the change
This pull request update the Python table API with temporary tables & views methods
Brief change log

Update python table API with temporary tables & views methods
Update python table API adding from_path method
Update python table API with insert_into
Deprecate TableEnvironment.register_table_source and TableEnvironment.register_table_sink
Clean up test_environment_completeness by removing the legacy excluded methods which are already supported

Verifying this change
This change added tests and can be verified as follows:

Added tests in StreamTableEnvironmentTests and existing test in EnvironmentAPICompletenessTests

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (PythonDocs)

@flinkbot  attention @sunjincheng121 @hequn8128 @dawidwys

Thank you @dianfu for the update. LGTM.


More search term options

For gene searches, add ability to search by more than the ParaSite description. Maybe GO terms? Or possibly more usefully the Hunt et al 2016 gene set terminology.

Added ability to search gene set terminology


release(0.6.2): release 0.6.2

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[x] Other... Please describe: release 0.6.2

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Coverage remained the same at 41.715% when pulling b72d7e004790c20508719c4a8ad4e22c7483b1a6 on vthinkxie:0.6.2 into aea80c47acdda7662db6c1dc7992115f283d37f1 on NG-ZORRO:master.


Log activation sparsity levels

Using sow to log sparsity of intermediate activations in the forward pass. Can log the activation sparsity in additional places by simply adding the line self.sow("intermediates", activation_name, x) (where x is the array containing the activations to be analyzed) anywhere in a flax module's call or apply method.

wrong repo...


[fix](union iterator) fix bug that result data order of VUnionIterator is different

Proposed changes
Issue Number: close #xxx
Problem summary
Fix bug of #16680, data order of VUnionIterator outout block is changed, which will impact compaction.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run p0

run beut

run p0

run p0

run beut

run feut


Minor javadoc fixes

Adds missing CLOSED return code to offer javadoc, and some minor spelling grammar fixes.

Thanks!


Typescript errors related to using generics

I have made a reproduction combining a few typescript errors displayed in the editor when using combinations of generics, defineModel and props destructuring: https://github.com/Ragura/vue-issue-repro
These errors relate to reports made in #3225 and #3195.
The reproduction includes 3 components:

DropdownDefineModel: using generics, props destructuring and defineModel
DropdownDestructuring: using generics and props destructuring
DropdownTraditional: using only generics

All three cases report different errors inside the components as well as App.vue you can inspect, but all three can be fixed by removing the generic from script setup and working with type any instead.
EDIT:
For these errors to show up when running the typecheck in the terminal, use vue-tsc --noEmit --p tsconfig.app.json.
IMPORTANT: One error only shows up in the editor and not when running vue-tsc and that is the error Default export of the module has or is using private name 'Props'. in the DropdownDestructure.vue file.
The reported errors when running vue-tsc are these:
src/App.vue:28:7 - error TS2345: Argument of type '{ modelValue: { id: number; name: string; }; title: string; options: { id: number; name: string; }[]; }' is not assignable to parameter of type '{ modelValue?: { id: number; name: string; } | undefined; } & VNodeProps & AllowedComponentProps & ComponentCustomProps'.
  Object literal may only specify known properties, and 'title' does not exist in type '{ modelValue?: { id: number; name: string; } | undefined; } & VNodeProps & AllowedComponentProps & ComponentCustomProps'.

28       title="DefineModel"
         ~~~~~~~~~~~~~~~~~~~

src/components/DropdownDestructure.vue:32:29 - error TS2339: Property 'id' does not exist on type '[{ type: PropType<T>; required: true; }] extends [Prop<infer V, infer D>] ? unknown extends V ? IfAny<V, V, D> : V : { type: PropType<T>; required: true; }'.

32       :selected="modelValue.id === option.id">
                               ~~

src/components/DropdownTraditional.vue:28:29 - error TS2339: Property 'id' does not exist on type '[{ type: PropType<T>; required: true; }] extends [Prop<infer V, infer D>] ? unknown extends V ? IfAny<V, V, D> : V : { type: PropType<T>; required: true; }'.

28       :selected="modelValue.id === option.id">

I am using the following:
vue-tsc: 1.7.8
vue: 3.3.4
Vue Language Features extension version 1.7.8, running takeover mode pointed to local vue-tsc typescript version 5.0.4.
Using VSCode insiders.

I mistakenly ran vue-tsc --noEmit without specifying the correct tsconfig.app.json file with the --p flag. When doing this, it does report the errors in the terminal as well. I have edited the original post to match these findings.

Hi, I can confirm I have the same issue, whether using generic components or not, combining defineModel and defineProps doesn't work as the latter overrides the former.
<script setup lang="ts">
defineProps<{
  type?: string
}>()

const modelValue = defineModel<number>()
</script>

Duplicate of #3164


Unstable results in test "test_kmeans_results" in K-means task in Comic-Con and k-means

test_kmeans_results evaluates to incorrect with 50% chance

Also: task 4/5 in Comic Con: barchart image is excluded from the course

Unstable test in 4/5

/Machine-Learning-101-Eng/Horror Trees/Entropy/task.py:8: FutureWarning: In a future version of pandas all arguments of DataFrame.drop except for the argument 'labels' will be keyword-only   X = data.drop('type', 1)


Use of getCharacteristicValue creates infinite loop!

The method getCharacteristicValue calls .getValue() on the characteristic object.
See https://github.com/kpsuperplane/homebridge-wiz-lan/blob/0253f46c144782a55fe6a418b8d31b18040734e9/lib/homekit-device/index.js#L87-L94
There are two issues with that

getValue() doesn't return anything at all. Its a callback based method, where the resulting value is passed into the callback passed to the method
getValue() is the function called when handling a request. It will query the the read handler of the plugin. e.g. what is set up for the GET request.

Thus, in the case of https://github.com/kpsuperplane/homebridge-wiz-lan/blob/0253f46c144782a55fe6a418b8d31b18040734e9/lib/wiz-accessory.js#L153-L155 this will actually create an infinite loop, as the get event will fire a getValue() which again fires an get event an so on.
Instead of using .getValue() just use the property .value to retrieve the current value. But if your "get" handler just consist of return .value you can just omit registering a "get" handler, as returning .value is the default behavior.
A stack trace created with the latest beta
Name@Homebridge B03F] Unhandled error thrown inside read handler for characteristic: Error: characteristic value expected string and received object
    at Characteristic.validateUserInput (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1917:17)
    at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1428:24
    at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/util/once.ts:9:18
    at Characteristic.<anonymous> (/usr/local/lib/node_modules/homebridge-wiz-lan/lib/wiz-accessory.js:154:11)
    at Characteristic.emit (events.js:315:20)
    at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1404:14
    at new Promise (<anonymous>)
    at Characteristic.<anonymous> (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1402:12)
    at step (/usr/local/lib/node_modules/homebridge/node_modules/tslib/tslib.js:140:27)
    at Object.next (/usr/local/lib/node_modules/homebridge/node_modules/tslib/tslib.js:121:57)

@kpsuperplane, @dotkrnl FYI this makes homebridge unusable in the latest beta and I assume it will for future stable releases once this beta moves to stable.

@kpsuperplane, @dotkrnl FYI this makes homebridge unusable in the latest beta and I assume it will for future stable releases once this beta moves to stable.

Hi everyone I really appreciate the effort you've done to get to the bottom of this but it's pretty difficult for me to make any patches to this plugin since I don't own an actual bulb to test on..
More than happy to approve a PR though (or even hand this repo over to one of you fine people!)

Hi everyone I really appreciate the effort you've done to get to the bottom of this but it's pretty difficult for me to make any patches to this plugin since I don't own an actual bulb to test on..
More than happy to approve a PR though (or even hand this repo over to one of you fine people!)

Hi everyone I really appreciate the effort you've done to get to the bottom of this but it's pretty difficult for me to make any patches to this plugin since I don't own an actual bulb to test on..
More than happy to approve a PR though (or even hand this repo over to one of you fine people!)

Hi everyone I really appreciate the effort you've done to get to the bottom of this but it's pretty difficult for me to make any patches to this plugin since I don't own an actual bulb to test on..
More than happy to approve a PR though (or even hand this repo over to one of you fine people!)

@kpsuperplane I really wish I had the skill set to be able to take on maintaining this plug-in; but I sadly do not have the time to learn completely everything, only dabble as I go and struggle with the minor parts I have learned so far. This however looks like a change that would not require to have the lights if I am not mistaken; but if you just no longer have the interest to continue maintaining the plug-in I do understand that.
@Supereg I had attempted to make the change you suggested in my local install and completely killed Homebridge and then reverted the changes. I feel bad dragging you into a repo that isn't yours, but this could be a show stopper for everyone using this plug-in once the new Homebridge beta goes stable.

@kpsuperplane I really wish I had the skill set to be able to take on maintaining this plug-in; but I sadly do not have the time to learn completely everything, only dabble as I go and struggle with the minor parts I have learned so far. This however looks like a change that would not require to have the lights if I am not mistaken; but if you just no longer have the interest to continue maintaining the plug-in I do understand that.
@Supereg I had attempted to make the change you suggested in my local install and completely killed Homebridge and then reverted the changes. I feel bad dragging you into a repo that isn't yours, but this could be a show stopper for everyone using this plug-in once the new Homebridge beta goes stable.

@kpsuperplane To my knowledge we have an initiative where we take ownership of unmaintained plugins to find new maintainers. See here, if you are interested.
In any case, I can try to have a look at it and create a PR for it. To my knowledge this should a doable fix. Though not sure why it doesn't already cause problems with the current versions of homebridge.

@kpsuperplane To my knowledge we have an initiative where we take ownership of unmaintained plugins to find new maintainers. See here, if you are interested.
In any case, I can try to have a look at it and create a PR for it. To my knowledge this should a doable fix. Though not sure why it doesn't already cause problems with the current versions of homebridge.

That is really good to know about that imitative, I had no idea! This could be handy for a few of us on the repo.

That is really good to know about that imitative, I had no idea! This could be handy for a few of us on the repo.

Looked at the code, the issue description is actually wrong. The plugin has its own characteristic representation with its own getValue function, which makes everything rather complicated as both are named the same but aren't the same thing.
Though I think there is some sort of mismatch somewhere, as I think in this line https://github.com/kpsuperplane/homebridge-wiz-lan/blob/0253f46c144782a55fe6a418b8d31b18040734e9/lib/wiz-accessory.js#L145
This also explains the error a bit better, the issue seems to be somewhere else. However this is particularly hard as I don't know the code and can't run the code.

Looked at the code, the issue description is actually wrong. The plugin has its own characteristic representation with its own getValue function, which makes everything rather complicated as both are named the same but aren't the same thing.
Though I think there is some sort of mismatch somewhere, as I think in this line https://github.com/kpsuperplane/homebridge-wiz-lan/blob/0253f46c144782a55fe6a418b8d31b18040734e9/lib/wiz-accessory.js#L145
This also explains the error a bit better, the issue seems to be somewhere else. However this is particularly hard as I don't know the code and can't run the code.

@Supereg yeah the plugin unfortunately inherits the complexity of the Kasa plugin that's it's based off of.
I tried my best to remove a lot of the cruft but it's definitely still difficult to debug without just diving head in for an hour or two

@Supereg yeah the plugin unfortunately inherits the complexity of the Kasa plugin that's it's based off of.
I tried my best to remove a lot of the cruft but it's definitely still difficult to debug without just diving head in for an hour or two

@mitch7391 Do you have any configuration for the wiz plugin? Like discovery options? Anything special?

@mitch7391 Do you have any configuration for the wiz plugin? Like discovery options? Anything special?

@Supereg I'll provide my config when I get home. As far as I remember, I just defined the plugin, and devices added themselves.

@Supereg I'll provide my config when I get home. As far as I remember, I just defined the plugin, and devices added themselves.

Yeah just wanted to verify that I didn’t mess up anything in the above mentioned pull request. But i think your issue over at homebridge also included configuration for wiz and it did indeed not include anything special.

Yeah just wanted to verify that I didn’t mess up anything in the above mentioned pull request. But i think your issue over at homebridge also included configuration for wiz and it did indeed not include anything special.

This is all that is required for the config:
{
            "platform": "WizSmarthome",
            "name": "WizSmarthome"
        }

This is all that is required for the config:
{
            "platform": "WizSmarthome",
            "name": "WizSmarthome"
        }

@Supereg yeah the plugin unfortunately inherits the complexity of the Kasa plugin that's it's based off of.
I tried my best to remove a lot of the cruft but it's definitely still difficult to debug without just diving head in for an hour or two

@Supereg just a thought on the complexity of this plug-in due to the plug-in it is originally based off. Would it be an easier task to recreate this plug-in using the official 'template' that Homebridge has now provided? Might be a silly question, I have only just started to learn how to write some basic shell scripts myself recently; so excuse my ignorance haha.

@Supereg yeah the plugin unfortunately inherits the complexity of the Kasa plugin that's it's based off of.
I tried my best to remove a lot of the cruft but it's definitely still difficult to debug without just diving head in for an hour or two

@Supereg just a thought on the complexity of this plug-in due to the plug-in it is originally based off. Would it be an easier task to recreate this plug-in using the official 'template' that Homebridge has now provided? Might be a silly question, I have only just started to learn how to write some basic shell scripts myself recently; so excuse my ignorance haha.

I don't think a rewrite is easier than just refactoring the plugin a bit (or moving to Typescript for better type awareness/safety. I think this could solve some confusion for possible future maintainers around the internal characteristic type and the HAP-NodeJS/homebridge provided one).
But again, this is bot not really feasible if you don't have the hardware to test it on, even if you heavily invest into stuff like unit testing.

I don't think a rewrite is easier than just refactoring the plugin a bit (or moving to Typescript for better type awareness/safety. I think this could solve some confusion for possible future maintainers around the internal characteristic type and the HAP-NodeJS/homebridge provided one).
But again, this is bot not really feasible if you don't have the hardware to test it on, even if you heavily invest into stuff like unit testing.

Closing this issue as the above mentioned error was fixed (although initially falsely analyzed). See #25

Closing this issue as the above mentioned error was fixed (although initially falsely analyzed). See #25


Translations did not pick up this new line and others like it

https://github.com/Phuks-co/throat/blob/d5a3e9b39704dd10f79465874cd09dcab9237fec/app/html/user/messages/notifications.html#L24
The added class="%(cls)s" seems to be the culprit.

Possible the translations have not been run yet so closing this for now.


Top menu full

how do i leave the menu as in the image below?
https://imgur.com/Upol9Rq

Found a solution for it ? If you didn't can you please share with us some more details of what you want to achieve there ?


[Accessibility][Headings and Labels - User Input]: No Heading level is defined on page.

User Experience:
If headings are not implemented on the page then it would be very difficult for the user to navigate the main content of the page as headings and labels are used to describe sections of content and controls respectively.
Test Environment:
Environment: Chromium Edge (Anaheim) + Narrator
Build Version: 2004 (OS Build 19569.1000)
Microsoft Edge Version: 82.0.436.0 (Official build) canary (64-bit)
Test URL: http://localhost:3000/home
Repro Steps:

Open URL: http://localhost:3000/home
"Bot Framework Composer" homepage will gets opened.
Select any of the previously created bot from "Recent Bots" or create one from the available Bots under 'Examples' and activate it.
Activate 'Headings' using accessibility insights for web or activate scan mode and then press 'H' key to look for headings defined on page.
Observe the issue.

Actual:
No Heading is defined on page.
Note:
Same issue is repro within :Bot responses, Notifications, and About page.
Expected:
Heading should be defined on page.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={0cf80872-800d-4643-90cf-0c1c3d3e6260}
Attachment:
MAS2.4.6_Bot Responses_User input_Notifications_About.zip

Attaching image so devs don't need to download the .zip file.

Can we also get a11y tags added to this issue? Including severity :)

As checked and verified on below environment, the issue is fixed now. So, we are closing the issue.
Test Environment
Browser: Microsoft Edge Dev {Version 83.0.467.0 (Official build) dev (64-bit)}
OS build: 2004 (19588.1000)
Screen Reader: Narrator
URL: Bot Framework Composer
Prerequisite Here


(BOLT-922) Ensure NotImplementedError messages are not lost

Previously a NotImplementedError thrown by the transport would get added to the result, but never printed. Ensure that errors not handled by the transport get logged. Our normal expectation is that the transport handles all errors and adds them to the error result.

CLA signed by all contributors.


nixos/loki: add module and test

Things done
This adds a basic module and a corresponding test for Loki. (follow-up PR of #62641).

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test loki

@GrahamcOfBorg test loki

As suggested by @globin I extended the assertion to forbid having both configuration and configFile defined.
I also rebased on the current master.


add Clone to EventSubscription

As per title, need to be able to clone this.

[ ] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[ ] Updated CHANGES.md

Codecov Report

Merging #315 into master will decrease coverage by 0.1%.
The diff coverage is 0.0%.

@@           Coverage Diff            @@
##           master    #315     +/-   ##
========================================
- Coverage    30.6%   30.5%   -0.2%
========================================
  Files         129     127      -2
  Lines        4869    4581    -288
  Branches     1546    1416    -130
========================================
- Hits         1492    1398     -94
+ Misses       2295    2216     -79
+ Partials     1082     967    -115

Impacted Files
Coverage Δ

tendermint/src/rpc/event_listener.rs
0.0% <0.0%> (ø)

light-client/src/contracts.rs
63.6% <0.0%> (-36.4%)
:arrow_down:

tendermint/src/serializers/raw_commit_sig.rs
16.6% <0.0%> (-16.7%)
:arrow_down:

light-client/src/components/verifier.rs
58.6% <0.0%> (-12.3%)
:arrow_down:

tendermint/src/vote/power.rs
9.0% <0.0%> (-8.4%)
:arrow_down:

light-client/src/components/scheduler.rs
44.0% <0.0%> (-7.5%)
:arrow_down:

light-client/src/predicates.rs
72.7% <0.0%> (-5.2%)
:arrow_down:

light-client/src/operations/commit_validator.rs
40.0% <0.0%> (-3.8%)
:arrow_down:

tendermint/src/validator.rs
43.7% <0.0%> (-3.7%)
:arrow_down:

tendermint/src/serializers/custom.rs
16.6% <0.0%> (-2.7%)
:arrow_down:

... and 65 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 4af6bcb...7895716. Read the comment docs.


Compiler init command should allow UPPERCASE project name

Discussion
What aspect of the language would you like to see improved?
Currently the crystal compiler's crystal init command does not accept any UPPERCASE characters within the name, raising an error during its name validation.
I believe that this validation step should be removed. If required, the generated project name could still be explicitly outputted as all-lowercase.
What are the reasons?
While I can understand if this is to promote an all-lowercase project directory name convention (not that I could find if this is actually a convention as per the conventions section of the crystal-lang reference), I believe that not allowing a project name with UPPERCASE:

Creates minor contradictions with some conventions (see below); and
Just appears to be unnecessarily restrictive.

Include practical examples to illustrate your points.

Contradictions
Currently Enforced
Convention Expected

Type Naming
crystal init app foobar module Foobar
crystal init app FooBar module FooBar

Acronyms
crystal init app acmebar module Acmebar
crystal init app ACMEBar module ACMEBar

Optionally add one (or more) proposals to improve the current situation.

Remove the line of lowercase validation.

Update the crystal-lang reference conventions to specify what project structures should look like.

Optionally, explicitly output project root directory name as all lowercase if still desired.

I dug a bit further to understand if this was explicitly intended and found that the commit which introduced this validation referenced the specification for a shard name.
This explains which convention/specification that the validation is trying to maintain, but I still believe that the validation should not be done at this point. To reiterate my original points, we want to allow for an arbitrary level of capitalisation for the generated module/spec names (of course, assuming the best about users' awareness of convention here), and enforce all-lowercase at the point when it's actually needed (naming the project root dir according to shard name specification).

Yes, the reason for those name restrictions is to enforce compatibility with the shard specification to avoid any surprises with unexpected project names.
However, I agree that an automatic transformation from PascalCased name could be a good enhancement. Currently it works the other way around in View#module_name which transforms snake_case to PascalCase for the module name.
This direction is lossy though, because there's no way to accurately transform to PascalCased acronyms.
That being said, it's also easy to just change the spelling to your liking manually after running crystal init.