Endless loop when completing conda info -

I installed _conda into $fpath and it mostly works as expected.
But: Completion goes into endless loop when completing conda info -.  Interrupt by pressing ^C two times gives:
  File "<string>", line 3, in <module>
  File "/Users/mb/miniconda3/lib/python3.12/json/__init__.py", line 293, in load
    return loads(fp.read(),
                 ^^^^^^^^^
KeyboardInterrupt
 <<< schnipp, der vollständige traceback ist in der angehängten Datei >>>

 % zsh --version                                                                          (conda)myzsh
zsh --version
zsh 5.9 (x86_64-apple-darwin22.0)

% conda -V                                                                               (conda)myzsh
conda 24.5.0

% conda info -e                                                                          (conda)myzsh
# conda environments:
#
base                  *  /Users/mb/miniconda3
graphviz                 /Users/mb/miniconda3/envs/graphviz
objc                     /Users/mb/miniconda3/envs/objc
pan                      /Users/mb/miniconda3/envs/pan
py3c                     /Users/mb/miniconda3/envs/py3c

_conda-loop.txt

I think what you are seeing is the completion system fetching package info from anaconda.org -- it may take a while, you should press  again and wait and see what happens. This is because conda info has the ability to show info about packages and so it must go get all packages so it can offer them as completion. There is more info about this and how to reset or clear the cache in the documentation at the top of the completion file.

Here is the completion in action:

This should be quite smooth once the cache has been initialized.

Bin etwas verwirrt!
% conda info -h
usage: conda info [-h] [--json] [-v] [-q] [-a] [--base] [-e] [-s]
                  [--unsafe-channels]

Display information about current conda install.

options:
  -h, --help         Show this help message and exit.
  -a, --all          `--all` is deprecated and will be removed in 24.9. Use
                     `--verbose` instead.
  --base             Display base environment path.
  -e, --envs         List all known conda environments.
  -s, --system       List environment variables.
  --unsafe-channels  Display list of channels with tokens exposed.

Output, Prompt, and Flow Control Options:
  --json             Report all output as json. Suitable for using conda
                     programmatically.
  -v, --verbose      Can be used multiple times. Once for detailed output,
                     twice for INFO logging, thrice for DEBUG logging, four
                     times for TRACE logging.
  -q, --quiet        Do not display progress bar.

conda info does not accept a package name as argument:
% conda info qtconsole
usage: conda [-h] [-v] [--no-plugins] [-V] COMMAND ...
conda: error: unrecognized arguments: qtconsole

Why the completion system should try to prompt for all packages in anaconda.org while completing conda info?
In case of conda update the completion of package names works really fine:
% conda update <tab>
zsh: do you wish to see all 128 possibilities (43 lines)?

 % conda update i <tab>
icu      idna     ipython  isort

Warnung: Ich habe keine Erfahrung mit zsh completers!
Having looked at your code I commented line 533 and now the completion works:
case $state in
(command)
    __conda_commands
    ;;
(subcmd)
    case ${line[1]} in
    (info)
        _arguments -C $help_opts \
                      '--json[report all output as json.]' \
                      '(-a --all)'{-a,--all}'[show all information, (environments, license, and system information]' \
                      '(-e --envs)'{-e,--envs}'[list all known conda environments]' \
                      '(-l --license)'{-l,--license}'[display information about local conda licenses list]' \
                      '(-s --system)'{-s,--system}'[list environment variables]' \
                      '--root[display root environment path]' \
     #      ==>       '*:packages:__describe_conda_package_available' \
        ;;
    (help)
# ...

Should I create pull request?

@Hanspe-3019 what conda version are you on?
I see
 💣 zsh» conda info zstd | head

zstd 1.4.9 h8574219_1
---------------------
file name   : zstd-1.4.9-h8574219_1.conda
name        : zstd
version     : 1.4.9
build string: h8574219_1
build number: 1
channel     : https://repo.anaconda.com/pkgs/main/osx-arm64
size        : 405 KB
 💣 zsh» conda --version
conda 23.10.0
 💣 zsh» conda info -h
usage: conda info [-h] [--json] [-v] [-q] [-a] [--base] [-e] [-s] [--unsafe-channels]

Display information about current conda install.

optional arguments:
  -h, --help         Show this help message and exit.
  -a, --all          `--all` is pending deprecation and will be removed in 24.9. Use `--verbose` instead.
  --base             Display base environment path.
  -e, --envs         List all known conda environments.
  -s, --system       List environment variables.
  --unsafe-channels  Display list of channels with tokens exposed.

Output, Prompt, and Flow Control Options:
  --json             Report all output as json. Suitable for using conda programmatically.
  -v, --verbose      Can be used multiple times. Once for detailed output, twice for INFO logging, thrice for DEBUG logging, four times for TRACE logging.
  -q, --quiet        Do not display progress bar.

Warnung: Ich habe keine Erfahrung mit zsh completers!
Having looked at your code I commented line 533 and now the completion works:
case $state in
(command)
    __conda_commands
    ;;
(subcmd)
    case ${line[1]} in
    (info)
        _arguments -C $help_opts \
                      '--json[report all output as json.]' \
                      '(-a --all)'{-a,--all}'[show all information, (environments, license, and system information]' \
                      '(-e --envs)'{-e,--envs}'[list all known conda environments]' \
                      '(-l --license)'{-l,--license}'[display information about local conda licenses list]' \
                      '(-s --system)'{-s,--system}'[list environment variables]' \
                      '--root[display root environment path]' \
     #      ==>       '*:packages:__describe_conda_package_available' \
        ;;
    (help)
# ...

Should I create pull request?

I think this is perhaps an undocumented feature? It would be good to find out and submit a pull-request for conda if the documentation there needs updating?

conda info PACKAGE was marked as pending deprecation in conda 23.9.0 and removed in 24.3.0: https://docs.conda.io/projects/conda/en/latest/release-notes.html#id8 in favor of the duplicate conda search PACKAGE --info because of the misleading user experience of the info subcommand.

% conda --version                     (conda)myzsh
conda 24.5.0
% conda info zstd | head
usage: conda [-h] [-v] [--no-plugins] [-V] COMMAND ...
conda: error: unrecognized arguments: zstd
Offenbar ist  Anzeige der package Infos vom Info-Command in das Search-Command gewandert:
% conda search zstd --info | head     (conda)myzsh
Loading channels: ...working... done
zstd 1.3.3 h2a6be3a_0
file name   : zstd-1.3.3-h2a6be3a_0.conda
name        : zstd
version     : 1.3.3
build       : h2a6be3a_0
build number: 0
size        : 317 KB
license     : BSD 3-Clause
deprecate 'conda info package_name' · Issue #7309 · conda/conda · GitHub https://github.com/conda/conda/issues/7309 2024-06-01 github.com http://github.com/
python - How do I see the summary or description for a conda package? - Stack Overflow https://stackoverflow.com/questions/61900525/how-do-i-see-the-summary-or-description-for-a-conda-package 2024-06-01 stackoverflow.com
Am 01.06.2024 um 16:49 schrieb Emergency Self-Construct @.***>:
Should I create pull request?
I think this is perhaps an undocumented feature? It would be good to find out and submit a pull-request for conda if the documentation there needs updating?
—
Reply to this email directly, view it on GitHub https://github.com/conda-incubator/conda-zsh-completion/issues/60#issuecomment-2143474872, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANEX733ZOITQNIM3DF4OFP3ZFHNQ3AVCNFSM6AAAAABIS5KDNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGQ3TIOBXGI.
You are receiving this because you were mentioned.

After adding zstyle ':completion::complete:*' use-cache 1 the response time for completing available packages are good after first collection of the data.
Maybe the need for caching should be emphasized more explicitly in the documentation. I had initially ignored the hint.

OK, so this issue should result in three PRs:
a) remove the completion for packages for conda info
b) add package completion for conda search --info
c) Advertise that the package cache will be populated during initial tab completion, maybe even write a message to the terminal informting users that the program isn't stuck in a perpetual loop
@Hanspe-3019 @jezdez would you tend to agree that the above proposal sounds like a reasonable plan of action?

A well-considered plan! Thank you all.
Am 02.06.2024 um 18:10 schrieb Emergency Self-Construct @.***>:
OK, so this issue should result in three PRs:
a) remove the completion for packages for conda info
b) add package completion for conda search --info
c) Advertise that the package cache will be populated during initial tab completion, maybe even write a message to the terminal informting users that the program isn't stuck in a perpetual loop

A well-considered plan! Thank you all.

@Hanspe-3019 do you want to submit a pull-request for modifying line 533? I think the line can be deleted altogether perhaps, rather than being commented out.

@Hanspe-3019 that's great, thank you for #61 I merged it just now.

I just noticed that conda search doesn't complete --info.

@Hanspe-3019 I've created a PR for conda search --info -- perhaps you could review it? I would appreciate it.

a and b are done in #60 and #61 -- not sure about c -- I tried to echo something from a completion, but didn't show up, zsh-completion docs will have to be consulted.


Add xfail_strict to test suite

In light of https://github.com/rapidsai/pynvjitlink/issues/4#issuecomment-2145015880, figured we could start enforcing this programatically going forward.

CC @gmarkall

Ideally we'd remove that setting if so, that's fairly restrictive. Dev on this repo is slow enough (or rather, linear enough) that it might not matter much, though.

Ideally we'd remove that setting if so, that's fairly restrictive

This enforcement has been removed.


After configuring opener it removes the default behaviour for other file types which are not defiined in opener

What system are you running Yazi on?
Linux Wayland
What terminal are you running Yazi in?
foot version: 1.17.2 +pgo +ime +graphemes -assertions
Did you try the latest code to see if this problem got fixed?
Tried, but the problem still
yazi --debug output

Yazi
    Version: 0.2.5 (Arch Linux 2024-04-27)
    OS: linux-x86_64 (unix)
    Debug: false

Emulator
    Emulator.via_env: ("foot", "")
    Emulator.via_csi: Ok(Foot)
    Emulator.detect: Foot

Adaptor
    Adaptor.matches: Sixel

Desktop
    XDG_SESSION_TYPE: Some("wayland")
    WAYLAND_DISPLAY: Some("wayland-1")
    DISPLAY: Some(":1")

SSH
    shared.in_ssh_connection: false

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Variables
    SHELL: Some("/usr/bin/fish")
    EDITOR: Some("/usr/bin/nvim")
    ZELLIJ_SESSION_NAME: None
    YAZI_FILE_ONE: None
    YAZI_CONFIG_HOME: None

file(1)
    Version: Ok(Output { status: ExitStatus(unix_wait_status(0)), stdout: "file-5.45\nmagic file from /usr/share/file/misc/magic\nseccomp support included\n", stderr: "" })

Text Opener
    default: None
    block: None

tmux
    TMUX: false

Ueberzug++
    Version: Err(Os { code: 2, kind: NotFound, message: "No such file or directory" })

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/ankur/.local/state/yazi" directory.

Describe the bug
I was configuring opener as instructed here :
https://yazi-rs.github.io/docs/configuration/yazi/#opener
And I defined some specific apps for some of the files through which it should open
but after that the default behaviors for other undefined files were overwritten and now whenever I press "Enter" or "o" while the cursor is hovered on it , nothing happens
Expected Behavior
for other files for which I haven't defined any specific app or thing like the ~/Desktop folder
before using opener it used to open in neovim but after I defined  default apps for some specific files then it doesn't do anything , the `~/Desktop' folder just doesn't opens in the neovim
To Reproduce
just paste this to your yazi.toml
[opener]
play = [
	{ run = 'mpv "$@"', orphan = true, for = "unix" },
]
view = [
	{ run = 'gwenview "$@"', orphan = true, for = "unix" },
]

[open]
# set rules for opening specific files
rules = [
	{ name = "*.mpv", use = "play" },
	{ name = "*.png", use = "view" },
	{ name = "*.jpg", use = "view" },
	{ name = "*.jpeg", use = "view" },

now hover your cursor over something like ~/Desktop folder , now press enter or 'o'
it won't open
Now you will only be able to open files with extension :
*.mpv
*.png
*.jpg
*.jpeg

Configuration
[opener]
play = [
	{ run = 'mpv "$@"', orphan = true, for = "unix" },
]
view = [
	{ run = 'gwenview "$@"', orphan = true, for = "unix" },
]

[open]
# set rules for opening specific files
rules = [
	{ name = "*.mpv", use = "play" },
	{ name = "*.png", use = "view" },
	{ name = "*.jpg", use = "view" },
	{ name = "*.jpeg", use = "view" },

Anything else?
No response

You need to prepend rules instead of overriding them.
https://yazi-rs.github.io/docs/configuration/overview#mixing

I've updated the documentation to clarify it in https://github.com/yazi-rs/yazi-rs.github.io/commit/d5d7016006d058657c429c4e56557f38497863a2


Remove CosmosDB deprecated tests

Description
As part of the issue # 6368, we are removing all the CosmosDbStorage deprecated classes that include the Microsoft.Azure.DocumentDB.Core library, due to the lack of security support.
This issue aims to remove all the integration tests and related code around the CosmosDbStorage class based on the PR # 6370 changes (tests location).

/promoted 600


chore(release): 29.0.5

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 decreased (-0.1%) to 95.853%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4055

Relevant Lines:
4154

💛  - Coveralls

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 decreased (-0.1%) to 95.853%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4055

Relevant Lines:
4154

💛  - Coveralls


Primary MD Has A Bad Link

The MD - https://github.com/mendixlabs/list-view-controls/blob/master/README.md has a bad link for the Header Sort.  Please see the link
Please follow Header sort guide
It goes nowhere.

Thanks for reporting, we merged documentation... but forgotten this one. We will restore it.

Fixed in v1.3.4


Adding "count_include_pad" argument to flax.linen.pooling.avg_pool

What does this PR do?
Now version's flax.linen.pooling.avg_pool average window_sum result include padded tokens. I add argument whether to include padded tokens or not
Checklist

[x] This change is discussed in a Github issue/
issues
[x] The documentation and docstrings adhere to the
documentation guidelines.
[x] This change includes necessary high-coverage tests.
(No quality testing = no merge!)

Codecov Report

Merging #2451 (ff97879) into main (8687673) will decrease coverage by 0.02%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main    #2451      +/-   ##
==========================================
- Coverage   79.66%   79.63%   -0.03%
==========================================
  Files          49       49
  Lines        4982     4985       +3
==========================================
+ Hits         3969     3970       +1
- Misses       1013     1015       +2

Impacted Files
Coverage Δ

flax/linen/pooling.py
80.00% <60.00%> (-4.38%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Hey @dslisleedh, thanks for creating this PR!
I think this looks good, however, before merging we should probably add a test using this flag.

To @cgarciae,
Sorry, I tested my self but didn't share result in PR. Here's result.
for i, shape in enumerate([(1, 5, 3), (1, 5, 5, 3), (1, 5, 5, 5, 3)]):
    inputs = jnp.ones(shape=shape)
    for kernel_size in [(1,), (3,), (5,)]:
        k_s = kernel_size * (i + 1)
        for strides in [(1,), (2,), (3,)]:
            s = strides * (i + 1)
            for padding in ['VALID','SAME']:
                res = avg_pool(inputs, k_s, strides=s, padding=padding, count_include_pad=False)

                print(jnp.min(res), jnp.max(res))

1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0
1.0 1.0

This is first time for me to create PR, so tell me anything if I missed something. Thank you.

Just noticed there aren't any tests for pooling.py, I'll create an issue, we can add a test for this in the future.

I found PoolTest class from ./tests/linen/test_linen.py
class PoolTest(absltest.TestCase):

  def test_pool_custom_reduce(self):
    x = jnp.full((1, 3, 3, 1), 2.)
    mul_reduce = lambda x, y: x * y
    y = nn.pooling.pool(x, 1., mul_reduce, (2, 2), (1, 1), 'VALID')
    np.testing.assert_allclose(y, np.full((1, 2, 2, 1), 2. ** 4))

  def test_avg_pool(self):
    x = jnp.full((1, 3, 3, 1), 2.)
    pool = lambda x: nn.avg_pool(x, (2, 2))
    y = pool(x)
    np.testing.assert_allclose(y, np.full((1, 2, 2, 1), 2.))
    y_grad = jax.grad(lambda x: pool(x).sum())(x)
    expected_grad = jnp.array([
        [0.25, 0.5, 0.25],
        [0.5, 1., 0.5],
        [0.25, 0.5, 0.25],
    ]).reshape((1, 3, 3, 1))
    np.testing.assert_allclose(y_grad, expected_grad)

  def test_avg_pool_no_batch(self):
    x = jnp.full((3, 3, 1), 2.)
    pool = lambda x: nn.avg_pool(x, (2, 2))
    y = pool(x)
    np.testing.assert_allclose(y, np.full((2, 2, 1), 2.))
    y_grad = jax.grad(lambda x: pool(x).sum())(x)
    expected_grad = jnp.array([
        [0.25, 0.5, 0.25],
        [0.5, 1., 0.5],
        [0.25, 0.5, 0.25],
    ]).reshape((3, 3, 1))
    np.testing.assert_allclose(y_grad, expected_grad)

  def test_max_pool(self):
    ...

When I ran this code with the edits from https://github.com/google/flax/pull/2451 there were no problems.
(flax_test) idongheon@idongheon-ui-MacBookAir flax % python ./tests/linen/linen_test.py
Running tests under Python 3.10.5: /Users/idongheon/miniforge3/envs/flax_test/bin/python
[ RUN      ] IdsTest.test_hashable
[       OK ] IdsTest.test_hashable
[ RUN      ] NormalizationTest.test_batch_norm
I0915 01:20:42.378859<PHONE_NUMBER> xla_bridge.py:169] Remote TPU is not linked into jax; skipping remote TPU.
I0915 01:20:42.379297<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'tpu_driver': Could not initialize backend 'tpu_driver'
I0915 01:20:42.379443<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'cuda': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
I0915 01:20:42.379549<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'rocm': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
I0915 01:20:42.380259<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'tpu': module 'jaxlib.xla_extension' has no attribute 'get_tpu_client'
[       OK ] NormalizationTest.test_batch_norm
[ RUN      ] NormalizationTest.test_batch_norm_complex
[       OK ] NormalizationTest.test_batch_norm_complex
[ RUN      ] NormalizationTest.test_batch_norm_multi_init
[       OK ] NormalizationTest.test_batch_norm_multi_init
[ RUN      ] NormalizationTest.test_group_norm
[       OK ] NormalizationTest.test_group_norm
[ RUN      ] NormalizationTest.test_group_norm_raises
[       OK ] NormalizationTest.test_group_norm_raises
[ RUN      ] NormalizationTest.test_layer_norm0 (reduction_axes=-1)
[       OK ] NormalizationTest.test_layer_norm0 (reduction_axes=-1)
[ RUN      ] NormalizationTest.test_layer_norm1 (reduction_axes=1)
[       OK ] NormalizationTest.test_layer_norm1 (reduction_axes=1)
[ RUN      ] NormalizationTest.test_layer_norm2 (reduction_axes=(1, 2))
[       OK ] NormalizationTest.test_layer_norm2 (reduction_axes=(1, 2))
[ RUN      ] PoolTest.test_avg_pool
[       OK ] PoolTest.test_avg_pool
[ RUN      ] PoolTest.test_avg_pool_no_batch
[       OK ] PoolTest.test_avg_pool_no_batch
[ RUN      ] PoolTest.test_max_pool
[       OK ] PoolTest.test_max_pool
[ RUN      ] PoolTest.test_pool_custom_reduce
[       OK ] PoolTest.test_pool_custom_reduce
[ RUN      ] RecurrentTest.test_complex_input_gru
[       OK ] RecurrentTest.test_complex_input_gru
[ RUN      ] RecurrentTest.test_convlstm
[       OK ] RecurrentTest.test_convlstm
[ RUN      ] RecurrentTest.test_gru
[       OK ] RecurrentTest.test_gru
[ RUN      ] RecurrentTest.test_lstm
[       OK ] RecurrentTest.test_lstm
[ RUN      ] RecurrentTest.test_optimized_lstm_cell_matches_regular
/Users/idongheon/miniforge3/envs/flax_test/lib/python3.10/site-packages/jax/test_util.py:44: FutureWarning: jax.test_util.check_eq is deprecated and will soon be removed.
  warnings.warn(f"jax.test_util.{attr} is deprecated and will soon be removed.", FutureWarning)
[       OK ] RecurrentTest.test_optimized_lstm_cell_matches_regular
[ RUN      ] StochasticTest.test_dropout
[       OK ] StochasticTest.test_dropout
[ RUN      ] StochasticTest.test_dropout_rate_limits
[       OK ] StochasticTest.test_dropout_rate_limits
[ RUN      ] StochasticTest.test_dropout_rate_stats
[       OK ] StochasticTest.test_dropout_rate_stats
----------------------------------------------------------------------
Ran 21 tests in 13.907s

OK

@dslisleedh can you create one of more tests under PoolTest using this new argument? You can copy an existing test, rename it, and change it to use the new count_include_pad argument. This would help immensely as these will be checked automatically before merging new code.

To @cgarciae
I add some codes to TestPool and here's code I tested.
from absl.testing import absltest, parameterized

from flax import ids
from flax import linen as nn

import jax
from jax import random
from jax import test_util as jtu
from jax.nn import initializers
import jax.numpy as jnp

import numpy as np

# Parse absl flags test_srcdir and test_tmpdir.
jax.config.parse_flags_with_absl()

class PoolTest(parameterized.TestCase):

    def test_pool_custom_reduce(self):
        x = jnp.full((1, 3, 3, 1), 2.)
        mul_reduce = lambda x, y: x * y
        y = nn.pooling.pool(x, 1., mul_reduce, (2, 2), (1, 1), 'VALID')
        np.testing.assert_allclose(y, np.full((1, 2, 2, 1), 2. ** 4))

    @parameterized.parameters(
        {'count_include_pad': True},
        {'count_include_pad': False})
    def test_avg_pool(self, count_include_pad):
        x = jnp.full((1, 3, 3, 1), 2.)
        pool = lambda x: nn.avg_pool(x, (2, 2), count_include_pad=count_include_pad)
        y = pool(x)
        np.testing.assert_allclose(y, np.full((1, 2, 2, 1), 2.))
        y_grad = jax.grad(lambda x: pool(x).sum())(x)
        expected_grad = jnp.array([
            [0.25, 0.5, 0.25],
            [0.5, 1., 0.5],
            [0.25, 0.5, 0.25],
        ]).reshape((1, 3, 3, 1))
        np.testing.assert_allclose(y_grad, expected_grad)

    @parameterized.parameters(
        {'count_include_pad': True},
        {'count_include_pad': False})
    def test_avg_pool_no_batch(self, count_include_pad):
        x = jnp.full((3, 3, 1), 2.)
        pool = lambda x: nn.avg_pool(x, (2, 2), count_include_pad=count_include_pad)
        y = pool(x)
        np.testing.assert_allclose(y, np.full((2, 2, 1), 2.))
        y_grad = jax.grad(lambda x: pool(x).sum())(x)
        expected_grad = jnp.array([
            [0.25, 0.5, 0.25],
            [0.5, 1., 0.5],
            [0.25, 0.5, 0.25],
        ]).reshape((3, 3, 1))
        np.testing.assert_allclose(y_grad, expected_grad)

    def test_max_pool(self):
        x = jnp.arange(9).reshape((1, 3, 3, 1)).astype(jnp.float32)
        pool = lambda x: nn.max_pool(x, (2, 2))
        expected_y = jnp.array([
            [4., 5.],
            [7., 8.],
        ]).reshape((1, 2, 2, 1))
        y = pool(x)
        np.testing.assert_allclose(y, expected_y)
        y_grad = jax.grad(lambda x: pool(x).sum())(x)
        expected_grad = jnp.array([
            [0., 0., 0.],
            [0., 1., 1.],
            [0., 1., 1.],
        ]).reshape((1, 3, 3, 1))
        np.testing.assert_allclose(y_grad, expected_grad)

if __name__ == '__main__':
  absltest.main()

When I tested with the previous code, an error occurred in the non-batch avg_pool, so I corrected the PR. Thanks for telling me to test it for sure.
Below is the test result with the modified code.
(flax_test) idongheon@idongheon-ui-MacBookAir flax % python pool_test.py
Running tests under Python 3.10.5: /Users/idongheon/miniforge3/envs/flax_test/bin/python
[ RUN      ] PoolTest.test_avg_pool0 (count_include_pad=True)
I0920 20:09:02.610634<PHONE_NUMBER> xla_bridge.py:169] Remote TPU is not linked into jax; skipping remote TPU.
I0920 20:09:02.610964<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'tpu_driver': Could not initialize backend 'tpu_driver'
I0920 20:09:02.611104<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'cuda': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
I0920 20:09:02.611212<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'rocm': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
I0920 20:09:02.611675<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'tpu': module 'jaxlib.xla_extension' has no attribute 'get_tpu_client'
[       OK ] PoolTest.test_avg_pool0 (count_include_pad=True)
[ RUN      ] PoolTest.test_avg_pool1 (count_include_pad=False)
[       OK ] PoolTest.test_avg_pool1 (count_include_pad=False)
[ RUN      ] PoolTest.test_avg_pool_no_batch0 (count_include_pad=True)
[       OK ] PoolTest.test_avg_pool_no_batch0 (count_include_pad=True)
[ RUN      ] PoolTest.test_avg_pool_no_batch1 (count_include_pad=False)
[       OK ] PoolTest.test_avg_pool_no_batch1 (count_include_pad=False)
[ RUN      ] PoolTest.test_max_pool
[       OK ] PoolTest.test_max_pool
[ RUN      ] PoolTest.test_pool_custom_reduce
[       OK ] PoolTest.test_pool_custom_reduce
----------------------------------------------------------------------
Ran 6 tests in 1.586s

OK

Thank you.

Awesome @dslisleedh! Can you commit changes to the tests?

To @cgarciae
Sure :)

@dslisleedh can you add this test? None of the other tests used padding="SAME" which is the more interesting case.
    @parameterized.parameters(
        {'count_include_pad': True},
        {'count_include_pad': False})
    def test_avg_pool_padding_same(self, count_include_pad):
        x = jnp.array([1.0, 2.0, 3.0, 4.0]).reshape((1, 2, 2, 1))
        pool = lambda x: nn.avg_pool(x, (2, 2), padding="SAME", count_include_pad=count_include_pad)
        y = pool(x)
        if count_include_pad:
          expected_y = jnp.array([10.0 / 4, 6.0 / 4, 7.0 / 4, 4.0 / 4]).reshape((1, 2, 2, 1))
        else:
          expected_y = jnp.array([10.0 / 4, 6.0 / 2, 7.0 / 2, 4.0 / 1]).reshape((1, 2, 2, 1))
        np.testing.assert_allclose(y, expected_y)

@cgarciae Oh, I forgot that. Thank you.
and here is result of your code.
(flax_test) idongheon@idongheon-ui-MacBookAir flax % python ./tests/linen/linen_test.py
Running tests under Python 3.10.5: /Users/idongheon/miniforge3/envs/flax_test/bin/python
[ RUN      ] IdsTest.test_hashable
[       OK ] IdsTest.test_hashable
[ RUN      ] NormalizationTest.test_batch_norm
I0921 22:23:30.726801<PHONE_NUMBER> xla_bridge.py:169] Remote TPU is not linked into jax; skipping remote TPU.
I0921 22:23:30.727137<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'tpu_driver': Could not initialize backend 'tpu_driver'
I0921 22:23:30.727280<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'cuda': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
I0921 22:23:30.727386<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'rocm': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
I0921 22:23:30.727836<PHONE_NUMBER> xla_bridge.py:345] Unable to initialize backend 'tpu': module 'jaxlib.xla_extension' has no attribute 'get_tpu_client'
[       OK ] NormalizationTest.test_batch_norm
[ RUN      ] NormalizationTest.test_batch_norm_complex
[       OK ] NormalizationTest.test_batch_norm_complex
[ RUN      ] NormalizationTest.test_batch_norm_multi_init
[       OK ] NormalizationTest.test_batch_norm_multi_init
[ RUN      ] NormalizationTest.test_group_norm
[       OK ] NormalizationTest.test_group_norm
[ RUN      ] NormalizationTest.test_group_norm_raises
[       OK ] NormalizationTest.test_group_norm_raises
[ RUN      ] NormalizationTest.test_layer_norm0 (reduction_axes=-1)
[       OK ] NormalizationTest.test_layer_norm0 (reduction_axes=-1)
[ RUN      ] NormalizationTest.test_layer_norm1 (reduction_axes=1)
[       OK ] NormalizationTest.test_layer_norm1 (reduction_axes=1)
[ RUN      ] NormalizationTest.test_layer_norm2 (reduction_axes=(1, 2))
[       OK ] NormalizationTest.test_layer_norm2 (reduction_axes=(1, 2))
[ RUN      ] PoolTest.test_avg_pool0 (count_include_pad=True)
[       OK ] PoolTest.test_avg_pool0 (count_include_pad=True)
[ RUN      ] PoolTest.test_avg_pool1 (count_include_pad=False)
[       OK ] PoolTest.test_avg_pool1 (count_include_pad=False)
[ RUN      ] PoolTest.test_avg_pool_no_batch0 (count_include_pad=True)
[       OK ] PoolTest.test_avg_pool_no_batch0 (count_include_pad=True)
[ RUN      ] PoolTest.test_avg_pool_no_batch1 (count_include_pad=False)
[       OK ] PoolTest.test_avg_pool_no_batch1 (count_include_pad=False)
[ RUN      ] PoolTest.test_avg_pool_padding_same0 (count_include_pad=True)
[       OK ] PoolTest.test_avg_pool_padding_same0 (count_include_pad=True)
[ RUN      ] PoolTest.test_avg_pool_padding_same1 (count_include_pad=False)
[       OK ] PoolTest.test_avg_pool_padding_same1 (count_include_pad=False)
[ RUN      ] PoolTest.test_max_pool
[       OK ] PoolTest.test_max_pool
[ RUN      ] PoolTest.test_pool_custom_reduce
[       OK ] PoolTest.test_pool_custom_reduce
[ RUN      ] RecurrentTest.test_complex_input_gru
[       OK ] RecurrentTest.test_complex_input_gru
[ RUN      ] RecurrentTest.test_convlstm
[       OK ] RecurrentTest.test_convlstm
[ RUN      ] RecurrentTest.test_gru
[       OK ] RecurrentTest.test_gru
[ RUN      ] RecurrentTest.test_lstm
[       OK ] RecurrentTest.test_lstm
[ RUN      ] RecurrentTest.test_optimized_lstm_cell_matches_regular
/Users/idongheon/miniforge3/envs/flax_test/lib/python3.10/site-packages/jax/test_util.py:44: FutureWarning: jax.test_util.check_eq is deprecated and will soon be removed.
  warnings.warn(f"jax.test_util.{attr} is deprecated and will soon be removed.", FutureWarning)
[       OK ] RecurrentTest.test_optimized_lstm_cell_matches_regular
[ RUN      ] StochasticTest.test_dropout
[       OK ] StochasticTest.test_dropout
[ RUN      ] StochasticTest.test_dropout_rate_limits
[       OK ] StochasticTest.test_dropout_rate_limits
[ RUN      ] StochasticTest.test_dropout_rate_stats
[       OK ] StochasticTest.test_dropout_rate_stats
----------------------------------------------------------------------
Ran 25 tests in 14.023s

OK


How to add this add-on to Hassio Container?

I have tried to put the repository in the
/homeassistant/config ==> didnt work
/homeassistant/config/custom_components ==> didnt work
is there any way to able this addon?
for example i put localtuya in /homeassistant/config/custom_components is working (i can see my localtuya in my integration)

The container installation of Home Assistant is not supporting Add-Ons, so you cannot simply use this add-on.
Since you are using a container-based installation, I suggest to simply use the official Cloudflared Container and follow the instructions to create a tunnel in the Cloudflare Zero Trust Dashboard.

You have to add your HA internal network ip to the tunnel. You can follow this tutorial, except that you do not have to use the tunnel token within the addon but while starting your docker container (which I guess so are already doing).


Enums not colored correctly in if statment

' Validate the priority of the request
priority = Get.String([[request, "parameters.priority"]], RequestPriority.DEFAULT)
if priority <> RequestPriority.LOW and priority <> RequestPriority.MEDIUM and priority <> RequestPriority.HIGH then priority = RequestPriority.DEFAULT

enum RequestPriority
	LOW = "low"
	MEDIUM = "medium"
	HIGH = "high"
	DEFAULT = "medium"
end enum

@TwitchBronBron did we end up fixing this issue? Feel like we might have. But I also could be wrong.

Yes, it appears we've fixed this issue. Not sure when.


Is this really online?

Is this really a DA online? I see that "extract_tumor.py" saves the DA endwith "_tumor.npy" offline.

Yes but we did extract the tumor regions in advance to save time during training.

ok,thank!


Why do you choose Spring Native for your app

Hi, i am reading your project, but i have a question, why do u choose Spring Native for your app instead of Quarkus of other framework for build with GraalVM. Can u tell me the pros and cons of Spring Native ? When should we use it instead of other framewoks ?

This is a Gradle plugin which is meant to be used with an application project. Some projects need/want it, some don't, depending on the application projects who use this plugin. Having said that, it is the application project author who considers the pros/cons of using this plugin/Spring Native, not the plugin itself to consider the pros/cons. For example, it doesn't make sense to ask the pros and cons of a gun (a tool, a plugin, etc). It is who/when/where/how to use it that matters.


EssentialsX problem

Expected behavior
i've think i will see good working EssentialsX
Observed/Actual behavior
when downloading EssentialsX there is error that api version is unknown
Steps/models to reproduce

Download EssentialsX
start server
get error

DivineMC version
[00:55:46 INFO]: Checking version, please wait...
[00:55:47 INFO]: This server is running DivineMC version git-DivineMC-"d800ee6" (MC: 1.19.3) (Implementing API version Unknown-Version) (Git: d800ee6 on ver/1.19.3)
You are on the latest version!
Agreements

[X] I am running the latest version of DivineMC from here.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[x] My version of Minecraft is supported by DivineMC.

Other
NOTE: this bug is only on build 47 and down

idk how to fix this but i will try

i fixed this. will be released today

fixed in https://github.com/DivineMC/DivineMC/commit/7499f280660682a4bc447c75ca57b352173b5181


send_notifications  identifier

identifier = random.getrandbits(32)
Will appear repeat?

What happens if identifier is equal to 0

i needs to fit within 32 bits, random.getrandbits(32) will avoid collisions, if you sent two with 0, then you wouldn't know which one you were receiving in the response_handler, nor would the enhanced notification be able to retry...


Add support for linux-musl-s390x target

This implements the rest of https://github.com/ericsink/SQLitePCL.raw/issues/571, after the "cb" part (https://github.com/ericsink/cb/pull/21) is merged.

The current failure is because e_sqlite3mc (which is a very recent addition to the cb build system) is not being built for musl-s390x.  I am not sure what it would take to address this, so for now I will just Cc @utelle

Looks like this seems to be the only change needed: https://github.com/ericsink/cb/pull/22

The current failure is because e_sqlite3mc (which is a very recent addition to the cb build system) is not being built for musl-s390x. I am not sure what it would take to address this, so for now I will just Cc @utelle

Sorry for the late response. I was away from home. Obviously, the problem is not related to SQLite3 Multiple Ciphers itself, but to the build procedure in cb.cs. Maybe the build procedure could be improved by making the list of targets a global object, instead of defining it separately for each component (making it easy to forget to add new targets to all relevant lists). This would allow to change it for all components in sync.

No worries.  It turned out to be an easy fix.


Fix reading compressed MusicXML files encoded in UTF-16

This PR fixes an issue with reading some compressed MusicXML files. Two example files are MozaChloSample and MozaVeilSample from this example set (these files also seem to be available in the regression test repository). Uncompressed versions of these files are opened fine but compressed .mxl files cannot be loaded by Lomse.
The reason of the issue is that the implementation in #246 loads unzipped XML code via MxlCompiler::compile_string() function which internally calls load_string() function from pugixml which assumes the string is already encoded in UTF-8. However the example scores listed above use UTF-16 encoding which is incompatible with UTF-8. File loading functions in pugixml do not assume any encoding by default and perform encoding detection which is why the uncompressed versions of the example scores are loaded correctly.
The issue can be fixed by using pugixml's load_buffer() which takes an in-memory buffer as input and doesn't skip encoding detection. This solution is implemented in this pull request. After this patch any encoding supported by pugixml should work fine with both compressed and uncompressed MusicXML files.

Thank you. I will review this as soon as possible.

The code changes look OK to me and no issues detected in tests. Merging.
Thank you very much!


Need some help with setting up the alignment tool

@tnrich I'm using the open vector editor and it is great! Using it inside a vue app and all is good so far. I've been using the demo page (https://teselagen.github.io/tg-oss/ove/#/Editor) to play around with all the features and see how to use them etc. I now need to set up sequence alignment and not sure where to begin. The demo mentions that I need to hook up an alignment server, but not sure what all of that entails. Is it just an event that I need to hook into when the "Create Alignment" button is clicked then I can throw it to my custom logic? How have you seen it be done? There's not that much detail in the readme/docs that I could see about this, are there any concrete examples out there? All I need to do is to align another sequence to the one that is in the editor already.
Thanks, appreciate your great work!

Hi @acostin , you'll have to implement your own alignment functionality.
I made the demo simpler for you and other people that have had similar questions about how to do this:

The demo can be seen here:
https://teselagen.github.io/tg-oss/ove/#/
And the source code for that simple alignment is in tg-oss/packages/ove/demo/src/EditorDemo/index.js
Hope that helps!

Thank you! This is helpful for sure.

Ok closing this then. Feel free to ask more questions


NDVI Assignment 4 uploaded

Pull Request Template
Review Checklist
CI Checks

[ ] The notebook runs from start to finish on all operating systems:

[ ] Mac
[ ] Windows
[ ] Linux

Reproducibility

[ ] Are the data downloaded in the code
[ ] Are paths created to ensure they work on all operating systems (using  os.path.join)
[ ] Are comments used to clarify the contents of the code that can’t be clarified using expressive variable and function names alone? (not too many comments - just enough)
[ ] Does the notebook run from start to finish?

PEP 8 standards & Code Readability
Functions

[ ] Do functions follow PEP 8 format conventions?
[ ] Are function docstrings clear (all  inputs and outputs clearly  described and defined)
[ ] Are function names expressive (the name describes what the function does)?
[ ] Are functions easy to understand and read?
[ ] How many tasks does each function do? (ideally a function does one thing well).

Package imports

[ ] Are standard modules (those included with the base  python install) vs. third party (related but externally developed tools) import groups correct with appropriate spacing in between each group?
[ ] Are variable names throughout the code, expressive?
[ ] Suggest changes if not, highlight what was done well
[ ] Is the code overall easy to understand and read? Are there things that would make it more clear / cleaner?

DRY Code

[ ] Are segments of code repeated in the file or is the code DRY?
[ ] Are loops used to optimize DRY code?
[ ] Are functions used to optimize DRY code?
[ ] Are there any areas that could be potentially improved (you can suggest improvements OR you can just highlight parts of the code where you suspect it could  be cleaner / more efficient.

Novel Approaches to Problem solving

[ ] Highlight any novel approaches to completing the assignment.

@eculler
FYI - I'm still waiting for workflow approval to merge. Is Nathan the maintainer?

We won’t be merging these pull requests - they’re just to practice
reviewing.
On Fri, Feb 18, 2022 at 4:57 PM Jacquelyn Witte @.***>
wrote:

@eculler https://github.com/eculler
FYI - I'm still waiting for workflow approval to merge. Is Nathan the
maintainer?
—
Reply to this email directly, view it on GitHub
https://github.com/earthlab-education/ea-2021-ndvi-automation-review/pull/23#issuecomment-1045382723,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2OEKDA3AT2NWWB6ANB2N3U33MH7ANCNFSM5OYYIZ3Q
.
You are receiving this because you were mentioned.Message ID:
<earthlab-education/ea-2021-ndvi-automation-review/pull/23/c1045382723@
github.com>

@eculler
Oh, I get it. I share this link!
Thanks.


Is start_end necessary for Excel UI?

The question is whether a transient occurs in acquisition at the beginning and the end for most cases, such that having start_end in excel_ui is justified.
To solve this issue, we should plot FSC vs time, SSC vs time, and some fluorescent channel vs time, for FCS files from as many sources as we can, and see if there is some transient. We currently have 3 of those (old flow, new flow, SEA flow).

I run the following code against the data in the examples folder, which includes files from the old and the new acquisition card in the Moake's flow cytometer.
import numpy
import scipy
import matplotlib
from matplotlib import pyplot
import palettable
matplotlib.rcParams['savefig.dpi'] = 120
matplotlib.rcParams['axes.color_cycle'] = \
    palettable.colorbrewer.qualitative.Paired_12.mpl_colors[1::2]

import FlowCal

# Load data
# Old Flow Cytometer
data_1 = [FlowCal.io.FCSData('FCFiles/data.{:03d}'.format(i))
          for i in range(1,6)]
# New Flow Cytometer
data_2 = [FlowCal.io.FCSData('FCFiles/data_{:03d}.fcs'.format(i))
          for i in range(1,6)]
# Combine
data = data_1 + data_2

# In order to better appreciate any trends, we will use
# a moving average to low-pass filter the events.
filter_n = 25
filter_ma = numpy.ones(filter_n) / filter_n

# Make plots
channels = ['FSC', 'SSC', 'FL1', 'FL2', 'FL3']
for channel in channels:
    pyplot.figure(figsize=(12, 20))
    for i, datum in enumerate(data_1):
        pyplot.subplot(len(data_1), 2, 2*i + 1)
        pyplot.plot(numpy.convolve(datum[:, channel + '-H'],
                                   filter_ma,
                                   mode='valid'))
        pyplot.title(str(datum))
        pyplot.xlabel('Event number')
        pyplot.ylabel('Channel value')
    for i, datum in enumerate(data_2):
        pyplot.subplot(len(data_2), 2, 2*i + 2)
        pyplot.plot(numpy.convolve(datum[:, channel],
                                   filter_ma,
                                   mode='valid'))
        pyplot.title(str(datum))
        pyplot.xlabel('Event number')
        pyplot.ylabel('Channel value')
    pyplot.tight_layout()
    pyplot.savefig('start_end_{}.png'.format(channel))
    pyplot.close()

The following was obtained for channel FL1:

Plots to the left correspond to files from the old acquisition card, and plots to the right correspond to the new acquisition card.
It is clear that there is a transient at the beginning of the data in the old files, but not in the new files. This may correspond to the fact that data taken with the new acquisition card is sampled and shown on the screen before being saved to a file. If there is a transient in this case, it most likely happens before any data is saved. There does not seem to be a transient at the end in both cases.
This confirms that this transient phenomenon exists, although it may not be present in data from all flow cytometers. I think the best course of action is to leave the statements that cut off events at the beginning and at the end.


Claim ClaimableBalance Button

Description
Adds a new Claim button and displays Claimable balances in their own view.

Show claimable balance {asset_code: amount} Below the account's current asset balances
Add Claim button to add balance to account's current asset balances
Remove Show Claimable Balances button

Claimable Balances Available will still show on the logger (Right Side)

Addresses issue: #42
Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

npm start
If you want to see a an account with claimable balances available load this account

SBK52ZEYXQLOKC4Y646NQFJGCVYLXWQYVTA6TU7QKPAUL2F45X6NABM2
You should see 3 balances on this account you could claim the LAWR balances but the SRT should fail due to no trust line established to SRT at the time of writing this.

I haven't looked at the code yet but I just wanted to suggest a small UI addition:

Can we have some sort of Claimable Balances header above the added asset balance view?

Although I just realized that if we add a header to the Claimable Balances view we should also do so for the normal balances. Maybe Claimable Balances and Balances?

Although I just realized that if we add a header to the Claimable Balances view we should also do so for the normal balances. Maybe Claimable Balances and Balances?

Thats totally doable give me a few min and I'll push a change

I think I found a small bug:

One more change: can we change the Update Account button to have Refresh Account text?

I think I found a small bug:

How can I reproduce this bug?

How can I reproduce this bug?

Create an account and send an XLM claimable balance to it.

One more change: can we change the Update Account button to have Refresh Account text?

Sure thats no problem.

How can I reproduce this bug?

Create an account and send an XLM claimable balance to it.

Ah it looks like Horizon returns back that result as Native.
Not a problem intercepting that and rename it XLM. Similar to how the Balances section does it.

Oh actually it looks like there are 3 function layers here, claimableDisplay, claimableRecords, and claimableBalanceRow. We should at least move claimableRecords outside claimableDisplay.

Refactoring this is a bit more confusing and cumbersome than anticipated so I'm going to keep it the way it is.
It's also similar in style to the how balanceDisplay.tsx is written.

It seems to me that we could get away with only passing balance, this.claimAsset(balance)?

The way claimAsset is called its easier to split these parameters separately.
this.account.claimableBalances is an array of claimable balance records iterated through mapping helper func claimableRecords we need this to invoke this objects server but we need to specify which CB we want to claim ie(id, asset, balance, and sponsor)

Interesting approach. Why does the key need to be an array of size one? And where is this key-value pair used?

This is the current approach used elsewhere to invoke the loading animation.

It's also similar in style to the how balanceDisplay.tsx is written.

Is it? I don't see any nested functions in the current balanceDisplay.tsx

we need this to invoke this objects server but we need to specify which CB we want to claim ie(id, asset, balance, and sponsor)

Take a look at the code again. Everything you pass to claimAsset() is derived from the balance on each iteration, so you don't need to pass the exact properties from balance like id, asset, etc. because you can just access the same properties from within the claimAsset() function. this is automatically added to all object functions, just like self for python.

This is the current approach used elsewhere to invoke the loading animation.

Can you point to the file does it this way?


No payloads appear on WebGUI, despite unblocking

Hello,
I'm having issues with TrollRAT, where when you open the WebGUI, no payloads appear. I have tried unblocking the files on both computers with no effort.
Please help.
Thanks,
rhk

Ok never mind I fixed by connecting it to the same internet.


fix(document): enforce drop order on DocumentInner

@Boddlnagg, can you check whether this patch fixes #20?

Yes, this does fix the issue, in that the tests don't crash anymore!
There is still one test that doesn't pass, however: test_raw_multiline_string:
---- test_raw_multiline_string stdout ----
        thread 'test_raw_multiline_string' panicked at 'assertion failed: `(left == right)`
  left: `Object({"firstnl": Object({"type": String("string"), "value": String("This string has a \' quote character.")}), "multiline": Object({"type": String("string"), "value": String("This string\nhas \' a quote character\nand more than\none newline\nin it.")}), "oneline": Object({"type": String("string"), "value": String("This string has a \' quote character.")})})`,
 right: `Object({"firstnl": Object({"type": String("string"), "value": String("This string has a \' quote character.")}), "multiline": Object({"type": String("string"), "value": String("This string\r\nhas \' a quote character\r\nand more than\r\none newline\r\nin it.")}), "oneline": Object({"type": String("string"), "value": String("This string has a \' quote character.")})})`', tests\test_valid.rs:54:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Seems like The tests aren't expecting Windows \r\n newlines ... but this should also have been caught by AppVeyor?!

Ah, the newline thing might be related to how git checks out the files.

Thanks for confirming!
bors r+
\r\n is separate issue - it's a bug in the tests. Thanks for the report!


Can't see template changes until server is restarted

Hello, I have create this website with express-generator, the thing is when I change a thing (no matter what) on a templatate, you can't see it reflected until you restart the server.
app.js
var createError = require('http-errors');
var express = require('express');
var path = require('path');
var cookieParser = require('cookie-parser');
var logger = require('morgan');

var indexRouter = require('./routes/index');
var usersRouter = require('./routes/users');

var app = express();

// view engine setup
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'twig');

app.use(logger('dev'));
app.use(express.json());
app.use(express.urlencoded({ extended: false }));
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));

app.use('/', indexRouter);
app.use('/users', usersRouter);

// catch 404 and forward to error handler
app.use(function(req, res, next) {
  next(createError(404));
});

// error handler
app.use(function(err, req, res, next) {
  // set locals, only providing error in development
  res.locals.message = err.message;
  res.locals.error = req.app.get('env') === 'development' ? err : {};

  // render the error page
  res.status(err.status || 500);
  res.render('error');
});

module.exports = app;

index.twig
{% extends 'layout.twig' %}

{% block body %}

{{title}}

Welcome to {{title}}

		<button class="btn btn-primary">
			Example
		</button>

{% endblock %}

routes/index.js
var express = require('express');
var router = express.Router();

/* GET home page. */
router.get('/', function(req, res, next) {
  res.render('index', { title: 'Express' });
});

module.exports = router;

Thank you.
Node v8.11.1

Hello, this is an issue with the twig templating system. It will always cache the view content, regardless of the view cache setting in Express. You have to disable it in Twig directly. You can find more on their issue tracker: https://github.com/twigjs/twig.js/issues/510

Thank you! It worked.


I am posting (Post Request) data from React App into an API and its giving me 419   (status unknown) ERROR

Here is my code:
fetch("http://<IP_ADDRESS>/api/login",{
method:"post",
headers:{
"Content-Type":"application/json"
},
body:JSON.stringify({
<EMAIL_ADDRESS>email: "password"
})
}).then(res=>res.json())
.then(data=>{
if(data.error){
console.log(data.error)
}
else{
console.log(data.message)
}
}).catch(err=>{
console.log("ERROR: "+ err)
})
Her is the ERROR it gives
Singin.js:31 POST http://<IP_ADDRESS>/api/login 419 (unknown status)
Singin.js:53 ERROR: SyntaxError: Unexpected token < in JSON at position 0
please replay me, I will be waiting

Support requests filed as GitHub issues often go unanswered. We want you to find the answer you're looking for, so we suggest the following alternatives:
Coding Questions
If you have a coding question related to React and React DOM, it might be better suited for Stack Overflow. It's a great place to browse through frequent questions about using React, as well as ask for help with specific questions.
https://stackoverflow.com/questions/tagged/react
Talk to other React developers
There are many online forums which are a great place for discussion about best practices and application architecture as well as the future of React.
https://reactjs.org/community/support.html

I've also posted On stack overflow but didn't get any satisfactory Answer😪
On Fri, 9 Apr 2021, 1:50 pm Sebastian Silbermann, @.***>
wrote:

Support requests filed as GitHub issues often go unanswered. We want you
to find the answer you're looking for, so we suggest the following
alternatives:
Coding Questions
If you have a coding question related to React and React DOM, it might be
better suited for Stack Overflow. It's a great place to browse through
frequent questions about using React, as well as ask for help with specific
questions.
https://stackoverflow.com/questions/tagged/react
Talk to other React developers
There are many online forums which are a great place for discussion about
best practices and application architecture as well as the future of React.
https://reactjs.org/community/support.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/reactjs.org/issues/3621#issuecomment-816529284,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQVX3LZYHDZ7GNCE2RXR5WLTH25UXANCNFSM42UG7ELQ
.


getSimInfo() returned undefined

When I try to use this package, getSimInfo() is undefined. Any solution?

do you have a sim card on the phone? if not, that's the reason
if you do, check if you have the android permission (depending on the targetSdkVersion, you need to use PermissionsAndroid)

Thanks for the reply.
Yes, dual SIM card. I can get the SIM info when I try to run it in emulator but not in the real device. How can I enable the android permission?

you need the READ_PHONE_STATE permission set in your AndroidManifest.xml and you can ensure that you have the permission by using
import { PermissionsAndroid } from 'react-native'

PermissionsAndroid.request(PermissionsAndroid.PERMISSIONS.READ_PHONE_STATE)

when i log RNSimData.getSimInfo().phoneNumber0 it gives me empty string......and if i log RNSimData.getSimInfo().countryName0 it logs 'pk'....can i body tell me what is the issue

i installed permisiions in manifest.xml like this......  and also import { PermissionsAndroid } from 'react-native' in source Code

do you have anything showing in adb logs?

well, it seems a "security feature" of Android https://stackoverflow.com/questions/23675868/how-to-get-phone-no-of-device-programmatically/23675998#23675998

while setting up run time Permissions . and after then on success callback i try to RNSimData.getSimInfo() .  is return null .  but if before app open i set permission  its working fine , any runtime solution?

while setting up run time Permissions . and after then on success callback i try to RNSimData.getSimInfo() . is return null . but if before app open i set permission its working fine , any runtime solution?

Did you find any solution ? please suggest me


Should Ability Points be Derived

We have the stat "ability points" which is the total stat points used by a character. There are two ways to do this:

Ability Points could be computed, as the sum of all stat points a character has
it could be used as a check that the character's stat sum is not greater than the total ability points.

The second option seem more correct, but may add complexity. Which way should it be implemented?

The maximum amount of Ability Points is 180. There are 30 max points for each ability and 6 different categories). I believe the purpose of these points is to only allow the player to spend so many points (second bullet point in original comment). If we were to implement this property we would need to check that the sum of all abilities is <= to ability points.

Closing - won't address this due to time constraints.


Add Grafana reverse proxy to nginx

Issue #988
Add /api/auth that retuns true (200) or false (401) if user is logged in.
Add proxy via nginx for grafana
Add influx datasource reader to grafana on install
Still todo:
[ ] Add datasources via json API

Codecov Report

Merging #1003 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1003      +/-   ##
==========================================
+ Coverage   95.39%   95.42%   +0.03%
==========================================
  Files           2        2
  Lines         152      153       +1
==========================================
+ Hits          145      146       +1
  Misses          7        7

Impacted Files
Coverage Δ

iml-rabbit/src/lib.rs
95.33% <0%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


Authentication

fixes #56
added django built in user for authentication. (need to delete sqlite and make migrations again). added 3 new views (login, logout, signup). added two html files (login.html and signup.html).
made landing page look more like sketch (login buttons up top are temporary locations)

I don't think this is addressing the feature of adding django-all-auth?  Let me know if I'm missing the point, but:
I specifically don't want to allow users to sign up with accounts managed by the site owner, where someone might write to me and say "I lost my password," and I have to worry about validating.    I think (don't know for sure) that django-allauth implements this.
I'm going to update #56 to explicitly say it should not use the django built in user model, unless that is a prerequisite.
Do let me know if I'm missing the point on this one.


Use addresses and map name of user space instrumentation in LinuxTracing

This should be the last part that makes unwinding work with user space
instrumentation, in particular regarding error handling.
We are passing the address ranges of user space instrumentation trampolines to
LinuxTracing via the UserSpaceInstrumentationAddresses interface. We are
also passing the map name of the injected library. We now use these for the
following purposes:

Detecting if a sample fell directly in a trampoline (entry or return), in
UprobesUnwindingVisitor;
Detecting if a sample fell in the injected library, in
UprobesUnwindingVisitor;
Detecting if unwinding stopped at a return trampoline, in
UprobesUnwindingVisitor;
Patching frame pointer callchains in UprobesReturnAddressManager.

Those classes are extended to now consider user space instrumentation in
addition to uprobes.
Variables and fields are renamed to generalize from uprobes to instrumented
functions, and comments are adjusted for the same reason.
Unit tests are extended to consider the new cases.
Note that with uprobes the approach was sometimes different and a bit simpler,
as most of the work for uprobes happens in the kernel, for which we don't get
samples, and for the rest we could simply use the map name "[uprobes]",
corresponding to a couple of addresses for the return trampoline.
Regarding performance: When user space instrumentation is enabled, we now
iterate over every frame of every callstack, and for each we get the
corresponding map entry. Considering a reasonable amount of samples per second,
and a reasonable amount of frames, this still sounds fine.
I looked at possible differences in CPU utilization on Trata while sampling at
5000 Hz (DWARF unwinding) and instrumenting a function with user space
instrumentation.
When instrumenting item_coordinate_frame::update, which is called often, this
change makes CPU utilization actually go down a bit, most likely because several
samples are discarded earlier than usual, causing less processing.
When instrumenting a function that is never called, I couldn't measure any
significant difference. I even profiled OrbitService with another instance of
Orbit for this case and indeed the number of samples falling in the new code is
extremely limited.
Bug: http://b/194704608
Bug: http://b/204404077
Test: Unit tests. Tried as part of the integration of user space instrumentation
and unwinding as per
http://go/stadia-orbit-user-space-instrumentation-and-unwinding#heading=h.a3i5cfh8pexm,
on Trata, on a custom target, and on the same custom target also with frame
pointers.

For the failed presubmit: Just rebase after https://github.com/google/orbit/pull/2993 is in - this should fix it.


I don't see the RPC working.

Installed the package, rebooted my pc and still nothing, Atom is open, so is Discord, am I missing something?

And now it magically started working, sorry... I guess.


Problems with support_literals and the Const type

From discussion in the Numba core dev meeting on "Problems with support_literals and the Const type"
Problems:

Lowering backend mishandle Const type likely caused by other arguments that are not expecting a Const

Blocking:

np.dtype(str) and other str usage

Suggested Solutions:

autoconversion of Const into bare Type

Siu: try this by next meeting

support_literals for particular argument by name or position
QualType or metadata; remove Const type

Attempted solution 1

https://github.com/sklam/numba/commit/1478f419df89e8b02b593c74d09251d670395439 add casting in the lowering to convert Const type into the value type.
On https://github.com/numba/numba/pull/3162, reduces failed tests from 109 to 63.
TODOS:
the fix only address cast from Const type to the exact type that matches typeof(Const.value).  We should allow casting to anything castable from typeof(Const.value).
Need similar fix at the typing layer.

Another attempt to solution 1

https://github.com/sklam/numba/commit/f6ac68b63c14b86b8ca496a57dda897f1473027f add conversion of Const to the value type at the typing level w/o the cast in lowering.
Note: it's possible to return a full signature from a generic typer to force a particular argument to not be a Const type.  See https://github.com/sklam/numba/commit/f6ac68b63c14b86b8ca496a57dda897f1473027f#diff-f5564ce04d0b9094cbf7f2051c1f87f2R525


What's New

Overview
We need to create a way to alert and inform users about new features in our app! We're still working on designs for this, but there's plenty to get started on. This is a priority, so we can start shipping app updates!
Our Current Mockups

Other App Mockups

Current Tasks
1. Check for a new version
Use userDefaults.value(for: Keys.version) to see if the app's saved version is different from the current App.version
2. Create a banner UIView
Create a UIView to appear above the grouped UITableView in the HomeViewController. You can create a UIView above the UITableView (SnapKit!) or work with the UIHeaderFooterView capabilities of the UITableView. One requirement is this banner should animate in and out coming fro  the navigation bar!
3. Show more information
For now, it's a safe bet we'll be using WhatsNewKit for a full-screen modal for more information about updates. We may need to create an Ithaca Transit flavored fork 🍴 for our purposes, but you can start here.

Merged fully and correctly a2f2a9ba4390a7ab2f42b72c5e4c983e4f825e80


fix: ci failures fixes

Description

TL;DR Fixes CI for master from PRs

#23885
#23959
#23946

These Pr are cherry picked from release
PR fixes following issue(s)
Fixes # (issue number)
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

[ ] Manual

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test


Fixed typo in articles/virtual-machines/windows/expand-data-disks.md

Fixed typo in Get-AzureRMReseourceGroup to Get-AzureRmResourceGroup

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

#sign-off @dabedin nice catch - thank you for fixing that! :)

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

#sign-off @dabedin nice catch - thank you for fixing that! :)

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

#sign-off @PRmerger can this get merged, please?

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

@dabedin : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.


chore(release): Update workflows and protos to use v1.9.0 Dapr

Signed-off-by: Shubham Sharma<EMAIL_ADDRESS>Description
Update e2e workflow and fetch proto script to use v1.9.0. Additionally, update all the protos.
Issue reference
Please reference the issue this PR will close: #412
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[] Code compiles correctly
[] Created/updated tests
[] Extended the documentation

Ok, this PR will be addressed in #345


Fixes issue 29 Coordinate display not working in firefox/iceweasel

The issue is actually with the this.clip() function. To fix this I saved off the settings before drawing the background and then restore it when done. This prevents the clip() from being presisted.
Lines 813-815 with this.save() and this.restore() immediately afterwards I think does nothing. So I decided to move it above all the background drawing to prevent the clip state from preceding to clip out all coordinates.
If this.clip() is no longer needed I will change it and resubmit the pull request.

Will fix and redo tonight. It also appears that it shouldn't have worked in the other browsers. http://www.w3schools.com/tags/tryit.asp?filename=tryhtml5_canvas_clip So odd.


ble_gap: fix warning about implicit cast form pointer to int

This solves the following warning:
returning 'int (*)(struct ble_gap_event *, void *)' from a function with return type '_Bool8' {aka 'unsigned char'} makes integer from pointer without a cast [-Werror=int-conversion]
which was found when building nimBLE as part of NuttX

Thanks!


Android Studio can not find library for SttickerView

i have implemented implementation 'com.github.kencheung4:android-StickerView:' this dependency but at the time of rebuild the project Android Studio could not find this dependency. can anyone help me to deal with this problem?

You can create a new module in your project and then copy all source code into the module


Fields with Reactive Rules are not reset to a good state when the required attribute is conditional on another field's value

Environment

OS: macOS 10.12.6
Node: macOS 10.12.6
NPM: 5.6.0
react-advanaced-form: 1.6.7

What
Current behavior
A field with a reactive rule does not reset to a non-error state after its error state has been initially triggered. To clarify, if a custom rule is set on a field and you set the required attribute to depend on another field having a value, it is not reset if you delete the value from the other field.  To get the value back to a good state, you have to refresh the page in the browser.
Use Case
This is happening when I'm implementing a password change form between old, new and confirm password fields.
Expected behavior
If a field with a conditional required reactive prop has been triggered to an error state, if you remove the value from the field on which it depends, the field with the reactive rule should be set to a non-error state.
How
I modified the SingleTarget example in your codebase on the master branch to exhibit the bug.

Modify SingleTarget.jsx with the code below. Run npm run storybook
begin typing a value in lastName. Notice the red outline on the other 2 fields.
Now remove the value you just typed.
Notice the red outline is removed from firstName but not fieldThree.  This also happens with custom validation messages. The validation message is not removed.

import React from 'react'
import { Form } from 'react-advanced-form'
import { Input } from '@examples/fields'
import Button from '@examples/shared/Button'

export default class RxPropsSingleTarget extends React.Component {
  render() {
    return (
      <React.Fragment>

Single target

        <Form onSubmitStart={this.props.onSubmitStart}>
          <Input
            name="firstName"
            label="First name"
            hint="Required when `lastName` has value"
            required={({ get }) => {
              return !!get(['lastName', 'value'])
            }}
          />
          <Input name="lastName" label="Last name" />
          <Input
            name="fieldThree"
            label="Some field three"
            hint="Must match when `lastName` has value"
            rule={({get, value}) => {
              return value === get(['lastName', 'value']);
            }}
            required={({ get }) => {
              return !!get(['lastName', 'value'])
            }}
          />
          <Button>Submit</Button>
        </Form>
      </React.Fragment>
    )
  }
}

Hi, @johnpreed. Thank you for reporting this. My apologies for getting back to you after such delay.
I think this issue has to do with concurrency of state updates. It's been a problematic aspect for long time, and I've tried to solve it once and for all under #354. It may also be related, if not identical, to #352.
I am quite busy with other things now, but I will try to publish that changes so you can test out if new state update mechanism would eliminate the issue you are describing.


spec: dependencies details > manifest merge strategy

While the spec specifies how acis are being extracted on one another (as long as ̀pathWhitelist allows it), it does not gives informations about the manifest.
As first proposal what about these "merge strategy" options :

none : keep original manifest unmodified,
array : assembles a value from every matching level of the hierarchy,
hash : recursively merge hash keys.

The behaviour description must be shipped inside the image but (and that's the part I don't like!!) the option it self should never be overwritten.
(I think we loose here the simplicity we want for the spec, but the problem do exists so I'm just hoping this proposal will help as a starting point for a better answer)

@frntn Actually there is no concept of manifest merging. dependencies/pathWhitelist only talks about constituent files of an ACI.

The behaviour description must be shipped inside the image but (and that's the part I don't like!!) the option it self should never be overwritten.

I don't really understand what you mean here - what specifically don't you like, and what would you rather see?
Also, to be clear, most options can be overridden in the CRM/at runtime (the exact semantics we are still trying to thrash out in #83/#84).

OK. So the dependency concept is just another way to implement the layered mechanism.
I though it was a more powerful feature where we could fetch the filesystem, the isolators, the labels and so on...
This would have been way more complex to implement (and understand), so I am happy to see it is not !
Thanks


Fixed DHCP replies being filtered out

I've tested this on: OS X running in Virtualbox, confirming that you get DHCP. All integration tests and all unit tests pass on Ubuntu 16.04.


Build API support for Filtering Submission History

Problem Statement
As a user, I need to be able to filter submission history by a "Date/Time" range. This is the first step in that process, allowing the front end to query the back end to return values from a date range.
Assumptions
Acceptance Criteria

Can filter the submissions by the Date/time submitted based on a date range (i.e. start date and end date)

To Do

Technical notes:

Hey team! Please add your planning poker estimate with ZenHub @ahay-agile6 @kevinhaube @meckila @sethdarragile6 @TomNUSDS


Supress Module Warning

Hi,
we use JEA for our Monitoring Software to monitor Exchange server.
we create a module to load the Exchange pssnapins and add the module to ModulesToImport.
This works, but the monitoring software put all console output to the monitoring Software.
When JEA Loads the Module, the Module generates a Warning and the warning ist on top of the console output, so i can't see the Interesting Output of the script.
Is there an option or can you add one, to supress the warning or set the WarningAction to silentlyContinue?
Regards,
Bernd

i have created the script, the execution policy is unrestricted. But I get the same Warning messages as bevore. The Warning comes when JEA Loads the Module.

i have tried the import-module with -DisableNameChecking -WarningAction 0, but get the same warning message.
WARNING: The names of some imported commands from the module 'ExchangeConfig' include unapproved verbs that might make them less discoverable. To find the commands with unapproved verbs, run the Import-Module command again with the Verbose parameter. For a list of approved verbs, type Get-Verb.

Do you call Import-Module after $WarningPreference = 'SilentlyContinue' ?

Yes, and if i start the Script in a normal Shell, there is no Warning message visible. But in the JEA Session the warning is shown.
Von: Julien Nury<EMAIL_ADDRESS>Antworten an: PowerShell/JEA<EMAIL_ADDRESS>Datum: Donnerstag, 26. Juli 2018 um 13:39
An: PowerShell/JEA<EMAIL_ADDRESS>Cc: Bernd Dausch<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [PowerShell/JEA] Supress Module Warning (#48)
Do you call Import-Module after $WarningPreference = 'SilentlyContinue' ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

The correct syntax would be Import-Module 'ExchangeConfig' -DisableNameChecking -WarningAction 'SilentlyContinue'
If it doesn't work, try Import-Module 'ExchangeConfig' 3>$null
If it still doesn't work, it may be a bug in the way snap-in modules are loaded ...

I try both. But the Warning is still shown.
Von: Julien Nury<EMAIL_ADDRESS>Antworten an: PowerShell/JEA<EMAIL_ADDRESS>Datum: Donnerstag, 26. Juli 2018 um 13:51
An: PowerShell/JEA<EMAIL_ADDRESS>Cc: Bernd Dausch<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Betreff: Re: [PowerShell/JEA] Supress Module Warning (#48)
The correct syntax would be Import-Module 'ExchangeConfig' -DisableNameChecking -WarningAction 'SilentlyContinue'
If it doesn't work, try Import-Module 'ExchangeConfig' 3>$null
If it still doesn't work, it may be a bug in the way snap-in modules are loaded ...
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Add optimized binary extended gcd algorithm

Optimized to use cheap shifts and adds rather than multiplications and
divisions for finding the greated common divisor and Bezout coefficients.

Much appreciation to @cuviper for their help/discussion about the impl on Zulip.

Much appreciation to @cuviper for their help/discussion about the impl on Zulip.


Change canonical string to remove Content-MD5

To improve the security of this gem, can we move forward and remove MD5 dependencies and use by default FIPS140-2 algorithms, may be SHA-256 ?
I propose to change default algorithm from SHA1 to SHA-256, any user can change our code with :digest => 'sha1' for backward compatibilities.
I propose to remove Content-MD5 header from the canonical string and replace with HASH(content). HASH is the same hash algorithme than the :digest unless of :content_digest is passed in parameter.
Before I implement a PR, @kjg @mgomes  have you any suggestion ?

Any body here ?

Ok, I will do a PR next week.


同一个dubbo服务一个微服务内多个类引用时,有些类的此dubbo服务reference注解属性不起作用

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.6.2
Operating System version:Windows10
Java version: 1.8

Steps to reproduce this issue

同一个微服务多个类引用同样的dubbo服务
每个类的reference除了version和group一样之外,其它属性均有不一样的配置
项目运行起来后有些类的此dubbo服务reference注解属性不起作用

我看了源码,是这里的问题:
/**
* Generate a cache key of {@link ReferenceBean}
*
* @param reference {@link Reference}
* @param beanClass {@link Class}
* @return
*/
private String generateReferenceBeanCacheKey(Reference reference, Class<?> beanClass) {
    String interfaceName = resolveInterfaceName(reference, beanClass);

    String key = reference.url() + "/" + interfaceName +
            "/" + reference.version() +
            "/" + reference.group();

    Environment environment = applicationContext.getEnvironment();

    key = environment.resolvePlaceholders(key);

    return key;

}

这里会缓存dubbo service,但是key应该由所有reference属性组成才对

Maybe we should count reference id as part of the key.
String key = reference.id() + "/" + interfaceName +
            "/" + reference.version() +
            "/" + reference.group();

那是极好的，可以解决这个问题

This issue has been fixed, please update 2.6.6~ or refer https://github.com/apache/dubbo/blob/f9325699388f653ebbcd921cbdd0baf02d838715/dubbo-config/dubbo-config-spring/src/main/java/com/alibaba/dubbo/config/spring/beans/factory/annotation/ReferenceAnnotationBeanPostProcessor.java#L178

我看了2.6.6版本，问题并得到没有解决

timeout和retries属性完全被忽略

仍然未考虑到其它Reference属性

Please wait for 2.6.7

It's still an issue for local @Servnce  that was referenced by multiple @Reference with different attributes. If the service annotated @Servnce is remote, it works.

duplicate to #4674


DOC] Add documentation for Shard Hotspot Identification feature in RCA Engine

What do you want to do?

[ ] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request.
We are building a new feature, 'Shard Hotspot Identification' with the Root Cause Analysis (RCA) Engine. Generically speaking, this feature will help us identify Hot Partitions; Within OpenSearch,  Hot Shard can be defined as the shard consuming more resources(JVM, CPU, Disk I/O and Network) than its counterpart within the index.
What other resources are available?

https://opster.com/guides/elasticsearch/operations/elasticsearch-hotspots-load-balancing/
https://sematext.com/blog/elastic-search-data-storage-is-not-spreading-equally/#elasticsearch-imbalanced-data-spread-causes-fixes
Here is the README added of the feature: https://github.com/opensearch-project/performance-analyzer-rca/tree/main/src/main/java/org/opensearch/performanceanalyzer/rca/store/rca/hotshard#hot-shard-rca

Hi @hdhalter , can you please have this prioritized for 2.7.

@ariamarble Let me know if you need any additional information regarding this feature.

The Hot Shard summary will be available to the users via the RCA API. See sample output:

"HotShardClusterRca": [{
  "rca_name": "HotShardClusterRca",
  "timestamp":<PHONE_NUMBER>563,
  "state": "unhealthy",
  "HotClusterSummary": [
    {
      "number_of_nodes": 3,
      "number_of_unhealthy_nodes": 1,
      "HotNodeSummary": [
        {
          "node_id": "7kosAbpASsqBoHmHkVXxmw",
          "host_address": "<IP_ADDRESS>",
          "HotResourceSummary": [
            {
              "resource_type": "cpu usage",
              "resource_metric": "cpu usage(num of cores)",
              "threshold": 0.027397981341796683,
              "value": 0.034449630200405396,
              "time_period_seconds": 60,
              "meta_data": "ssZw1WRUSHS5DZCW73BOJQ index9 4"
            },
            {
              "resource_type": "heap",
              "resource_metric": "heap alloc rate(heap alloc rate in bytes per second)",
              "threshold": 7605441.367010161,
              "value": 10872119.748328414,
              "time_period_seconds": 60,
              "meta_data": "ssZw1WRUSHS5DZCW73BOJQ index9 4"
            },
            {
              "resource_type": "heap",
              "resource_metric": "heap alloc rate(heap alloc rate in bytes per second)",
              "threshold": 7605441.367010161,
              "value": 8019622.354388569,
              "time_period_seconds": 60,
              "meta_data": "QRF4rBM7SNCDr1g3KU6HyA index9 0"
            }
          ]
        }
      ]
    }
  ]
}]


Added ChiSquareContingency

Adding a KSD analog for a categoricals.
I may need help with the meaningful normalized_chi2 calculation for comparison -- let me know if you want to change the way it's done. Some idea:
        max_chi2 = dof * contingency_table.values.sum()
        normalized_chi2 = chi2 / max_chi2

Can ignore sonar reqs


fix(tooltip): fix instantiation upon focus

Overview
This PR fixes focus event does not open tooltip (it does after hovering it over). Turned out that tooltip instances are created upon event, and our “automatic instantiation upon event” mechanism didn’t work for focus event.
Review will be easier with w=1 query arg added to the URL of diff.
Added
Support for focus event in init-component-by-event mix-in
Testing / Reviewing
Testing should make sure Tooltip is not broken, esp. in IE.

👍 ✅


Building an .fsproj doesn't play well with non-interface .fs files in node modules

Description
I have an .fsproj project which is dependent on fable-import-react. Although interface declaration files are not compiled to .js, the Fable.Helpers.React.fs file is. So my build output directory winds up looking like this:
build/
   dist/
      app/  [my code here, including Components.js]
      node_modules/
           fable-import-react/  [Fable.Helpers.React.js here]

as a result of the build, the exports in the Components.js file require ./../node_modules/fable-import-react/Fable.Helpers.React.
However, when I run this through brunch to concatenate the files, the Fable.Helpers.React module gets fed to require.register as fable-import-react/Fable.Helpers.React and as a result I get a "Component not found" error.
There is a filter in brunch that allows regex matching/replacing on module path names, but apparently it doesn't run on anything that starts with node_modules. I'm opening an issue with them on this.
However, it would seem to me that when a node module's fs file is compiled to js, it should go under the same root as the application code; i.e, shouldn't the folder structure be:
build/
   dist/
      app/  [my code here, including Components.fs]
      fable-import-react/  [Fable.Helpers.React.fs here]

Repro steps
Steps to reproduce the problem:

Create a project with a code dependency on Fable.Helpers.React. Use .fsproj and follow the instructions in the fable-import-react README for .fsproj projects.
The fableconfig.json file is as follows:

{
    "ecma": "es2015",
    "outDir": "build/dist",
    "scripts": {
        "prebuild": "npm install",
        "postbuild": "cp node_modules/fable-import-react/package.json build/dist/node_modules/fable-import-react"
    }
}

Run fable on the project.
Install brunch. The relevant node modules need to be added to package.json. The minimal brunchconfig.js file looks like this:

exports.config = {

  paths: {
    watched: [
      'app/assets',
      'build/dist'
    ]
  },

  files: {
    javascripts: { joinTo: 'app.js' },
  },

  plugins: {
    babel: { presets: ['es2015', 'react'] }
  },
};

Run brunch build.

Expected behavior
All modules will be found when loading the application in the browser.
Actual behavior
Components.js:5Uncaught Error: Cannot find module 'build/dist/node_modules/fable-import-react/Fable.Helpers.React' from 'build/dist/app/Components.js'

Known workarounds
A real hack: Copy the Fable.Helpers.React.fs file to a top-level directory before compiling.
This requires the fableconfig.json to look like this:
{
    "ecma": "es2015",
    "outDir": "build/dist",
    "scripts": {
        "prebuild": "cp node_modules/fable-import-react/Fable.Helpers.React.fs fable-import-react/"
    }
}

and the path to the file needs to be changed in .fsproj to point to fable-import-react/Fable.Helpers.React.fs. (The file has to live permanently in the top-level directory, or the entry in the .fsproj file will disappear after msbuild runs.)
This is naturally extremely brittle, and it requires creating a separate source directory outside app/ for every node module that contains non-interface .fs files.
Related information

Operating system: OS X 10.11.5
.NET Runtime, CoreCLR or Mono Version: mono 4.4.0
Fable version: 0.3.15

I haven't used brunch but I did get similar reports of problems when using files in a directory external to the project (see #110). To fix that, we added the --copyExt argument which will copy the external files into a folder named fable_external attaching a hash to prevent name conflicts.
Also because the problem became more common, I set the --copyExt argument to true by default. Can you please try passing this argument or just updating fable to latest version (v0.3.17)?
BTW, not directly related to the issue but I'll tell you just in case as you're working with import definitions. At the moment, Fable is ignoring all files whose name starts with Fable.Import or Fable.Core because they're supposed to be fake files just there to provide interface information to the F# compiler. So if you want to have code that's actually compilable, you need to put it in a different file like Fable.Helpers.React.fs.
I hope this helps. Cheers!

With Fable 0.3.18 installed I still get the file compiled into the node_modules directory (with just fable or with fable --copyExt):
build/dist/app
...
build/dist/app/Components.js
build/dist/app/Components.js.map
...
build/dist/node_modules
build/dist/node_modules/fable-import-react
build/dist/node_modules/fable-import-react/Fable.Helpers.React.js
build/dist/node_modules/fable-import-react/Fable.Helpers.React.js.map

When I looked at the documentation, the --copyExt information didn't immediately suggest itself as a solution to this problem, since it wasn't clear that I should be considering node_modules as an directory external to the project. (And in fact from the results above Fable apparently doesn't treat it that way.)
With regard to your last comment -- I was wondering how Fable knew not to compile the interface declaration files to JS. I had all my files compiling, and then just the Fable.Helpers.React compiling, and all the other .fs files in the node_modules directory weren't compiling. I had thought there was something wrong until I realized that it was just the interface declaration files that weren't compiling, but I didn't really understand how Fable knew to ignore them.

Thanks for the detailed explanation. I see now that I misunderstood you. Recently, I'm usually putting the F# code in a src folder while node_modules remains one level above and is thus considered external. But in your case node_modules is in the same folder as the F# code (which should be a very normal use case) so it's not being tagged as external.
I think we should consider any file within node_modules always as external so there's no node_modules folder generated in the output directory. If you agree with this I'll implement the solution and upload a new version.
Cheers!

I think that's not quite it. Without the workaround I described my overall directory structure looks like this:
app/   [Components.fs etc. are here]
brunch-config.js
build/
     dist/
         app/   [Components.js etc. get created here]
         node_modules/
              fable-import-react/  [Fable.Helpers.React.fj gets created here]
     temp/
foo.fsproj
fableconfig.json
node_modules/
      fable-import-react/  [Fable.Helpers.React.fs is here]
package.json
public/

This structure comes from my trying to organize the project with several goals:

Follow the general standard organization used by brunch, where all source files--js, css, html, etc.--are under an app/ directory, and ultimately output is generated into a directory called public/.
Have only one top-level build directory (I've set the .fsproj file so that the output directory is build/dist and the intermediate output base directory is build/temp).

Heres the structure I use for fable-fuse:
├── App
│   ├── MainView.ux
│   ├── cards-menu.unoproj
│   ├── fonts
│   │   ├── LICENSE
│   │   └── Roboto-Bold.ttf
│   └── js
│       ├── cards-menu.js
│       ├── fable-core.js
│       └── fable_external
│           ├── Apis-1337169215.js
│           ├── Observable-901317181.js
│           └── PromiseExt-2013750498.js
├── build.bat
├── build.sh
├── node_modules
│   ├── fable-core
│   ├── fable-fuse
│   └── fable-import-fetch

├── package.json
└── src
    ├── cards-menu.fs
    ├── cards-menu.fsproj
    ├── fableconfig.json

Best thing to do when you get something working it get a yo template going
to it can be easily reused.
You can read about mine for fable-fuse here:
http://7sharpnine.com/blog/fable-fuse-template/
On 12 June 2016 at 01:53, Paul Blair<EMAIL_ADDRESS>wrote:

I think that's not quite it. Without the workaround I described my overall
directory structure looks like this:
app/   [Components.fs etc. are here]
brunch-config.js
build/
dist/
app/   [Components.js etc. get created here]
node_modules/
fable-import-react/  [Fable.Helpers.React.fj gets created here]
temp/
foo.fsproj
fableconfig.json
node_modules/
fable-import-react/  [Fable.Helpers.React.fs is here]
package.json
public/
This structure comes from my trying to organize the project with several
goals:

Follow the general standard organization used by brunch, where all
source files--js, css, html, etc.--are under an app/ directory, and
ultimately output is generated into a directory called public/.
Have only one top-level build directory (I've set the .fsproj file so
that the output directory is build/dist and the intermediate output base
directory is build/temp).

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Fable/issues/188#issuecomment-225402885,
or mute the thread
https://github.com/notifications/unsubscribe/AAj7yp6A7G9ykY7snrlfHM6cPwUHzFz3ks5qK1iKgaJpZM4IzErW
.

As agreed, I've updated the code so files in node_modules are always considered as external and thus copied to the fable_external folder. Can you please update fable-compiler to version 0.3.20 and check if it's working for you now? If it is, please feel free to close the issue.
Cheers!

@psfblair Do you have a sample of the stuff you are working on?

Thanks, Alfonso, this is working for me now!
@7sharp9 Have a look here: https://github.com/psfblair/fable_redux_skeleton . The Redux sample isn't entirely finished yet, but after Alfonso's latest changes I no longer have errors in the console.