warmup

Hello, it would be nice if it would be possible to add warmup runs, in my benchmark case the first run is always slower as some data has to be fetched into memory, and it always drags down the "real" lowest benchmark lowest as this is done only once.
I would suggest #[divan::bench(warmup=10) or something simular with the default being either zero or a 1.

Are you suggesting in this case that Divan would discard the first 10 samples after it finds a proper sample size?
Perhaps we could have something like:
#[divan::bench(
    // Discard at least the first 10 samples.
    warmup_sample_count = 10,
    // Discard samples until 1 second has elapsed.
    // This would use `IntoDuration` like `min_time`.
    warmup_min_time = 1,
)

Are you suggesting in this case that Divan would discard the first 10 samples after it finds a proper sample size?
Perhaps we could have something like:
#[divan::bench(
    // Discard at least the first 10 samples.
    warmup_sample_count = 10,
    // Discard samples until 1 second has elapsed.
    // This would use `IntoDuration` like `min_time`.
    warmup_min_time = 1,
)

Yee something like that would be very nice.

In the meantime, I recommend moving out the benchmark closure and doing your own custom warmup.
#[divan::bench]
fn my_bench(bencher: divan::Bencher) {
    let work_fn = || {
        // Benchmarked work in here.
    };

    // Warmup:
    for _ in 0..1000 {
        work_fn();
    }

    // Benchmark:
    bencher.bench(work_fn);
}

that would work for me, Thanks 👍

It'd probably be a good default to have a warmup. One thing users coming to divan from criterion will notice is that the output is noisier with divan.

+1
I noticed a performance degradation in a test after renaming (so the test ended up at the top of the list). This is very confusing


Make TensorNet CUDA-graph compatible

This PR is an effort to optimize TensorNet. Note that I merged here also all the changes in #186.
I have focused on inference of small molecules (such as Alanine, 22 atoms), in which we saw the CUDA kernel launching overhead was tremendous:

My strategy was to simply modify what was needed for torch.compile(backend="inductor", mode="reduce-overhead") to put as much as possible into a CUDA graph.
I replaced some blocking operations and I was able to convince TorchDynamo to put the bulk of the forward pass into a CUDA graph (orange is a graph):

Zooming in, I am having trouble with making TorchDynamo understand the neighbor kernels:

Alas, TorchDynamo refuses to graph the backwards pass. But I know it is possible to graph it, since I can do so manually.
Anyhow, I believe this is a problem that will solve itself in time as TorchDynamo improves its support for things like Dynamic shapes and extensions. And even today, the speedup is impressive.
I am not sure how to apply optimizations such as CUDA graphs for training, where shapes are dynamic. However, in those situations the changes sin this PR will just not improve things as much. TorchDynamo refuses to even process the model when used during training, complaining about dynamic shapes.
You will notice the benchmark tries to compile the model by doing this:
model = torch.compile(model, backend="inductor", mode="max-autotune")

This line fails miserably when using the current environment, I had to use pytorch-nightly (which provides version 2.1.0):
conda install pytorch torchvision torchaudio pytorch-cuda=12.1 -c pytorch-nightly -c nvidia

Which made me install the rest of the torch friends with pip:
$ pip list | grep torch
pytorch-lightning       1.6.3
torch                   2.1.0.dev20230619
torch-cluster           1.6.1
torch-geometric         2.3.1
torch-scatter           2.1.1
torchmdnet              0.2.5
torchmetrics            0.11.4

Luckily these compile just fine with pip.
With a little luck after this https://github.com/conda-forge/pytorch-cpu-feedstock/pull/172 is merged we can update the env and stay in conda-forge.

What happens if you compile to torchscript and don't try to use graphs?  Does that help training speed?

I was not able to train on a TorchScripted model. I suspect there might be some issue with the old-ish torch lightning version currently used. However upgrading it is dauting #168.
When trying to train with an scripted model (this is not particular to TensorNet) I get this error:
https://github.com/torchmd/torchmd-net/pull/186#issuecomment-1587513829
I have no idea where it comes from...

Is it necessary to do something specific to run inference with all your optimizations? Or they will work out-of-the-box?

To make the model fully compatible with CUDA Graph, I need replace scatter with reduction:
diff --git a/torchmdnet/models/output_modules.py b/torchmdnet/models/output_modules.py
index d3d7e1b..29edf12 100644
--- a/torchmdnet/models/output_modules.py
+++ b/torchmdnet/models/output_modules.py
@@ -25,7 +25,8 @@ class OutputModel(nn.Module, metaclass=ABCMeta):
         return

     def reduce(self, x, batch):
-        return scatter(x, batch, dim=0, reduce=self.reduce_op)
+        # return scatter(x, batch, dim=0, reduce=self.reduce_op)
+        return torch.sum(x)

     def post_reduce(self, x):
         return x

Of course, this is assumes that there is one molecule and the reduction is summing.

You can give scatter the argument dim_size with the number of different batches and this will be CUDA-graph compatible.
By default, scatter computes dim_size=batch.max()+1, which causes a sync.
In the current code, just give it a 1 if you only have one molecule.
In general, not having access to the number of batches is a pain for CUDA graphs.
If there is only one then the batch argument can be None, which can be detected and leveraged without syncs. Otherwise unless we add an argument "num_molecules" I see no way out of a sync.

OK! This is enough to fix graphing, but we need a general solution:
diff --git a/torchmdnet/models/output_modules.py b/torchmdnet/models/output_modules.py
index d3d7e1b..45e66eb 100644
--- a/torchmdnet/models/output_modules.py
+++ b/torchmdnet/models/output_modules.py
@@ -25,7 +25,7 @@ class OutputModel(nn.Module, metaclass=ABCMeta):
         return

     def reduce(self, x, batch):
-        return scatter(x, batch, dim=0, reduce=self.reduce_op)
+        return scatter(x, batch, dim=0, dim_size=1, reduce=self.reduce_op)

     def post_reduce(self, x):
         return x

Great! Did you found out anything about the ungraphable operation in backpropagation?

With https://github.com/torchmd/torchmd-net/pull/195#discussion_r1325742792 and https://github.com/torchmd/torchmd-net/pull/195#issuecomment-1719195035, I can graph forward and backward passes, but the let me check in the full pipeline.

Awesome! masked_scatter has been giving me a hard time from the start, I can believe that being the offending function. Thanks for the replacement!

The CUDA Graphs work, except OutputModel, which will be addressed in #214 or other.


added Joke Service Controller and Tests

In this PR, we add an endpoint /api/joke/get that can be used to get a custom number of jokes in a given joke category, as well as creating the corresponding test files.
closes #7

lgtm


Questions about the current feedback and the swing problem

Hi Sir，
Thank you for sharing such valuable research. But I have two questions I would like to ask you.
The first is that I don't know where the Current Feedback part of the paper is represented in the code. Could you help me to figure it out?
The second question is the leg swing in the hips' yaw direction that you mentioned before. May I ask that did you update some reward functions to avoid this problem?

Hi, thanks for the comment.
Sorry the current feedback part is not pushed yet, the reason being that the paper implements it for the HRP-5P robot while this repository uses the JVRC-1 robot. The code for achieving omnidirectional walking is on topic/omnidirectional (see https://github.com/rohanpsingh/LearningHumanoidWalking/compare/main...topic/omnidirectional-walk). I may add the code for simulating degraded torque-tracking in this repository at some point in the future, hopefully soon. In essence, the idea is to add some random noise proportional to the joint velocity to the applied torque at each timestep.
Do you mean the outward sway motion of the swing leg? If you observe such behavior in simulation after training, I recommend you to tune the PD gains. If you observe this behavior appears on the real robot but not in simulation, using current feedback may help.

Thanks for your reply. I hope your future research will go well, sir.


Beginners Guide Request

Hello,
I recently bought a Petkit Pura Max and I really appreciate your work of managing to integrate it in home assistant. It would be great if I could create some automations so my lightbulbs would light in different colors depending on the litter/trash state. However, although I tried to put together all the information I could find about this plugin, in the end I wasn't able to set it up. I only have basic skills when it comes to coding so apart from installing it and trying to edit the config.yaml file, I could not make it show in integrations.
Would any of you be kind enough to make a step by step guide/tutorial that I could follow? It would be very much appreciated, maybe there are more people trying to set this up.
Thank you

I have exactly the same issue, got quite a few integrations working, but this one I don’t really get how to configure.

Guide has been updated! Let me know of any other changes that may help clarify instructions.


Severe: Bunch of unimplemented rules in the game

No possibility of castling
No beat in passing by a pawn
Moves that result in exposing King to check allowed.
King can be beaten and the game doesn't finish. Instead, an exception is thrown in next move

But overall the game looks pretty cool

He also should flip the positions of the King and Queen for both colors.


allows creating parents in mkdir in numpy module

Addresses #1.
Motivation: Lets say I have images and masks and I want to put the mmap files in mmap/images and mmap/masks.

Codecov Report

Merging #2 (4f1e1a9) into master (c8749df) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          572       571    -1
=========================================
- Hits           572       571    -1

Impacted Files
Coverage Δ

python/mmap_ninja/numpy.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


List of issues fixed on NAV 2018 Cumulative Updates

Where can we find a list of the issues that have been fixed in CU1 and CU2? I checked the vsix notes, the cumulative updates comments, and I cannot find anything. There has to be a record of fixes somewhere.

Like always in the corresponding KB articles:
https://support.microsoft.com/en-us/help/4058601
https://support.microsoft.com/en-us/help/4078580

Thanks, that is where I checked, but I'm not seeing anything pertaining to the AL functionality. Only platform and application changes, as usual.
I'm looking for the fixes that have been backported to NAV concerning the AL functionality. For instance, #62 has reported been fixed in CU2, but there is no reference to that or any other bugfixes anywhere that I can find.

The #62 issue is tagged as part of the February 2018 milestone so I'm guessing it has not been released as part of the 2018 CU2. Just guessing.

@DavidRoys That is my point, we shouldn't have to guess. There should be an overview of the fixes published somewhere.

@bwdeleeuw And where did you take the information from, that NAV 2018 CU1 and CU2 should include any backported update? Usually MS includes such updates in the developer previews  (--> later in NAV 2018 R2) only, not in the 2018 CUs.

@NKarolak : https://github.com/Microsoft/AL/issues/1322 (last answer). thpeder confirm it will be backported.

@HelleChr Thanks!
@NKarolak also, #722, reaction from @StanislawStempin: "We are currently working on addressing this issue. We will target back-porting the fix to NAV 2018."
But to answer your larger question, it has been made clear by MS that not only fixes will be back-ported, but also selected features, on a case by case decision, also by @StanislawStempin, in #1296: "Having said that, we will back-port selected features to NAV 2018 but it will be a case by case decision dependent on the costs/benefits of doing so. It will mean finding a balance between fixing truly blocking issues on NAV 2018 vs completing new feature development for the latest version (for example, we will be back-porting the fix for #62 to NAV 2018)."
Right now, I have to retest a lot of issues in every CU1. I have been able to confirm that #62, #1044 and #988 seem to work in CU2. I still have to test #1182. The fact that these issues are working in CU2 is a clear indication that MS is back-porting issues to NAV 2018. If there was a list of fixes somewhere, I could target my testing to just those that have been back-ported, instead of having to retest all issues every time.
Right now every CU is a black box, a new version, with no indication of what it contains. Even the changelog is stale, and contains information about the December Update. It would save everyone a lot of time if the MS team were to simply publish a list of back-ported fixes per NAV 2018 CU. Wouldn't you agree?

Det issue der er med tabt data er åbenbart ikke løst endnu "We are currently working on addressing this issue. We will target back-porting the fix to NAV 2018."
Med venlig hilsen
PC-Software A/S
Helle Christensen
Solution Developer
Direkte tlf.:  87 38 69 96
Mobil tlf...:  40 44 60 96
Mail.........:  hch<EMAIL_ADDRESS>Skanderborgvej 238, 8260 Viby J.
Tlf. 87386999. Fax 87386998.
Hjemmeside: www.pcsoft.dkhttp://www.pcsoft.dk/
Fra: Bartel de Leeuw<EMAIL_ADDRESS>Sendt: 7. februar 2018 16:30
Til: Microsoft/AL<EMAIL_ADDRESS>Cc: Helle Christensen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Emne: Re: [Microsoft/AL] List of issues fixed on NAV 2018 Cumulative Updates (#1562)
@HelleChrhttps://github.com/hellechr Thanks!
@NKarolakhttps://github.com/nkarolak also, #722https://github.com/Microsoft/AL/issues/722, reaction from @StanislawStempinhttps://github.com/stanislawstempin: "We are currently working on addressing this issue. We will target back-porting the fix to NAV 2018."
But to answer your larger question, it has been made clear by MS that not only fixes will be back-ported, but also selected features, on a case by case decision, also by @StanislawStempinhttps://github.com/stanislawstempin, in #1296https://github.com/Microsoft/AL/issues/1296: "Having said that, we will back-port selected features to NAV 2018 but it will be a case by case decision dependent on the costs/benefits of doing so. It will mean finding a balance between fixing truly blocking issues on NAV 2018 vs completing new feature development for the latest version (for example, we will be back-porting the fix for #62https://github.com/Microsoft/AL/issues/62 to NAV 2018)."
Right now, I have to retest a lot of issues in every CU1. I have been able to confirm that #62https://github.com/Microsoft/AL/issues/62, #1044https://github.com/Microsoft/AL/issues/1044 and #988https://github.com/Microsoft/AL/issues/988 seem to work in CU2. I still have to test #1182https://github.com/Microsoft/AL/issues/1182. The fact that these issues are working in CU2 is a clear indication that MS is back-porting issues to NAV 2018. If there was a list of fixes somewhere, I could target my testing to just those that have been back-ported, instead of having to retest all issues every time.
Right now every CU is a black box, a new version, with no indication of what it contains. Even the changelog is stale, and contains information about the December Update. It would save everyone a lot of time if the MS team were to simply publish a list of back-ported fixes per NAV 2018 CU. Wouldn't you agree?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/AL/issues/1562#issuecomment-363805178, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Aii6ZX1OdJHKIwrWb75vtMRn0fW4yEwNks5tScF_gaJpZM4R7hQd.
[Denne e-mail er skannet af Symantec.cloud]

Following...

Following..

It has been more than two weeks since this suggestion has been logged. Any response?

Future CUs (starting from CU4) will include the list of AL Development related fixes in the KB articles alongside other fixes. The reason why CU1-CU3 doesn't include them is that we used a different process for backporting issues reported through GitHub and these did not get picked up by the KB tooling.

@StanislawStempin that is great, thanks!

Reopening this isssue; CU4 has no mention of any AL-related fixes in the KB article

@bwdeleeuw Hum yes, there is one: 264468 Word or RDLC report layout is not auto generated on build if it is not found when you develop AL extensions.

@NKarolak thanks; I even searched for "AL", must have overlooked it.


Add ability to replay history against a cursor

The tests are insufficient but the implementation is pretty straightforward and we need this for the tests for #85, so I'll fix this later.

Current coverage is 74.38%

Merging #135 into master will decrease coverage by -0.80% as of 01d142a

Powered by Codecov. Updated on successful CI builds.


[BREAKING CHANGE] Parameter type changed on TestLogger

V1:
public TestLogger(string name, TestSink sink, bool enabled)
V2:
public TestLogger(string name, ITestSink sink, bool enabled)
https://github.com/aspnet/Logging/blob/1.0.0/src/Microsoft.Extensions.Logging.Testing/TestLogger.cs#L15
https://github.com/aspnet/Logging/blob/dev/src/Microsoft.Extensions.Logging.Testing/TestLogger.cs#L15

@Eilon

This package doesn't ship :)
Some kind of exclusion list we add this to?

Yeah, I think we just need to remove the baseline, then the tool won't run (no baseline, nothing to compare).

@javiercn assigning to you to remove the baseline for this non-shipping package.


unfixable = ["F401"] only works with ruff.toml [lint] but not with pyproject.toml [tool.lint.ruff]

Since recently, for some reason, ignoring of F401 does not work over pyproject.toml.
Probably, other settings set using [tool.ruff.lint] don't work either. Don't really understand why.
Does anyone have a lead of why this could be happening?
If you're filing a bug report, please consider including the following information:

List of keywords you searched for before creating this issue. Write them down here so that others can find this issue more easily and help provide feedback.

unfixable = ["F401"]

A minimal code snippet that reproduces the bug.

a basic python file with an unused import

The command you invoked (e.g., ruff /path/to/file.py --fix), ideally including the --isolated flag.

ruff is run on save by vscode

The current Ruff settings (any relevant sections from your pyproject.toml).
pyproject.toml

[tool.black]
line-length = 88
target-version = ['py38','py39','py310','py311']
include = '\.pyi?$'

# We use preview style for formatting Black itself. If you
# want stable formatting across releases, you should keep
# this off.
preview = false

[tool.lint.ruff]
# Enable pycodestyle (`E`) and Pyflakes (`F`) codes by default.
extend-select = ["E", "F", "SIM"]
ignore = ["E501", "F841", "F401"]
show-fixes = true

# Allow autofix for all enabled rules (when `--fix`) is provided.
fixable = ["A", "B", "C", "D", "E", "F", "G", "I", "N", "Q", "S", "T", "W", "ANN", "ARG", "BLE", "COM", "DJ", "DTZ", "EM", "ERA", "EXE", "FBT", "ICN", "INP", "ISC", "NPY", "PD", "PGH", "PIE", "PL", "PT", "PTH", "PYI", "RET", "RSE", "RUF", "SIM", "SLF", "TCH", "TID", "TRY", "UP", "YTT"]
unfixable = ["F401"]

# Exclude a variety of commonly ignored directories.
exclude = [
    ".bzr",
    ".direnv",
    ".eggs",
    ".git",
    ".git-rewrite",
    ".hg",
    ".mypy_cache",
    ".nox",
    ".pants.d",
    ".pytype",
    ".ruff_cache",
    ".svn",
    ".tox",
    ".venv",
    "__pypackages__",
    "_build",
    "buck-out",
    "build",
    "dist",
    "node_modules",
    "venv",
]

# Same as Black.
line-length = 88

# Allow unused variables when underscore-prefixed.
dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"

# Assume Python 3.8.
target-version = "py38"

[tool.ruff.lint.mccabe]
# Unlike Flake8, default to a complexity level of 10.
max-complexity = 10

[tool.ruff.lint.isort]
case-sensitive = true
classes = ["SVC"]

# Ignore `E402` (import violations) in all `__init__.py` files, and in `path/to/file.py`.
[tool.ruff.lint.per-file-ignores]
"__init__.py" = [
    "F401",  # unused import
    "F403",  # star imports
    ]

[tool.ruff.lint.pydocstyle]
convention = "google"

The current Ruff version (ruff --version).

ruff 0.6.4
vscode 1.93.0

I think you want [tool.ruff.lint] rather than [tool.lint.ruff]

That is what I thought as well. But when I do that, I get a warning(by vscode editor): is not valid under any of the given schemas. That change also does not seem to do any difference

ok, i think i found it.
The field "show-fixes = true" was the problem as it is unknown field show-fixes.
Vscode debug output log level did not pick that up.


No syntax variables matched as repeating at this depth

Failure in Travis: https://travis-ci.org/dtolnay/proc-macro-hack/builds/202363760
Regression: https://github.com/rust-lang/rust/issues/39889

This is fixed by https://github.com/rust-lang/rust/pull/39419 in rustc 1.17.0-nightly (691eba135 2017-03-01).


How do I find the commit hash for a particular version of the codeql-action?

For example, I'd like to be able to use the commit hash when referencing the coedql-action/autobuild action, something like:
uses: github/codeql-action/autobuild@<hash of commit for the latest version> # v3.26.8
The releases have the bundle versions, but I'm having a fail at finding the specific hash for the specific action.

Hi @karianna,
I'm not completely sure I understand what you mean, but each version corresponds to a tag in the codeql-action repository. For example, for 3.16.8: https://github.com/github/codeql-action/tree/v3.26.8

The easiest thing to do is go to the tag in the code section of this repo: https://github.com/github/codeql-action/tree/v3.26.8 and find the associated commit.
Alternatively, if you have gh installed:
$ gh api --jq '.sha' /repos/github/codeql-action/commits/v3.26.8
294a9d92911152fe08befb9ec03e240add280cb3

Thank you both, that worked perfectly!


Update README.md

Please fill in this pull request template before submitting
1. This pull request resolves #71
2. Description

add pointers link
add preprocessors link

3. Fill in checklist by marking [x]

[x] I've read the CONTRIBUTING.md
[x] I was assigned to this issue
[x] My code is formatted using required linting
[x] I've run my code locally and checked it works

@srkds Thanks for the PR!
The update is missing floats.c. Please edit to include this and check again to make sure your change includes all tutorials made so far. Thanks!

Hi @funbeedev,
I have updated all links, please do a review and let me know if there are any links missing.

@srkds  Looks good, thanks for that :)

Thankyou @funbeedev


cmd/kcp: non-zero return code on error

We always exited with zero.

good catch 👍

/lgtm

/lgtm cancel

Unrelated flake.


Suggestion: User configurable links in menu

Thanks for the great work on this project!
I have a suggestion: it would be nice to have user configurable links in the menu. I'm running Virtual Radar among other programs on the same computer as ADS-B receiver, and it would be nice to display a link to it in the top menu.

The theme needs some work come the next release. The work mainly involves the top navigation which I am thinking needs to utilize some sort of fly out navigation. Currently with all links on one line it is pushing the search filed on advanced installs down another line covering a portion of the maps. Let me see what I can do for you when I start work on the themes. It should be a fairly easy addition.

Added support for custom links in commit 24493a82653766af47099d846146407e5dab425f.


versioning opendistro nmslib zip artifact

As of today we seem to override the .so zip file in the bucket for each release. This could potentially break the older versions which still refer this bucket. This creates compatibility issues
Zip file referred here : https://github.com/opendistro-for-elasticsearch/opendistro-build/blob/master/elasticsearch/linux_distributions/scripts/preInstall.sh

This issue will be fixed in 1.9.0 release. However, we need to address how it will be fixed for 1.8, 1.7, 1.6.


Improvement of cosine similarity

I saw that currently the cosine similarity is computed on a vector by vector basis, which can become quite slow for a big amount of samples.
I recently had the same issue on a personal project with around 9k embeddings; I ended up using sklearn's pairwise cosine similarity, and that was much faster as it computes the whole matrix at once.

After a quick look, their implementation seems to only use numpy. It'd still be another dependency to copy it though, even if not as heavy.


test_realistic_complex_sub_dependencies fails on Windows py27-pip8.1.1

It's failing since 69e4142d1c2ffa615ca8343b6c32a382d8e8115e. See the log on AppVeyor.
Environment Versions

Windows
Python version: 2.7
pip version:  8.1.1
pip-tools version: 4.2.0

Steps to replicate

pytest -k test_realistic_complex_sub_dependencies

Expected result
Test passes.
Actual result
================================== FAILURES ===================================
___________________ test_realistic_complex_sub_dependencies ___________________

runner = <click.testing.CliRunner object at 0x000000000806CE80>

    @pytest.mark.network
    def test_realistic_complex_sub_dependencies(runner):
        wheels_dir = "wheels"

        # make a temporary wheel of a fake package
        subprocess.check_output(
            [
                "pip",
                "wheel",
                "--no-deps",
                "-w",
                wheels_dir,
                os.path.join(PACKAGES_PATH, "fake_with_deps", "."),
            ]
        )

        with open("requirements.in", "w") as req_in:
            req_in.write("fake_with_deps")  # require fake package

        out = runner.invoke(cli, ["-v", "-n", "--rebuild", "-f", wheels_dir])

>       assert out.exit_code == 0
E       AssertionError: assert 1 == 0
E        +  where 1 = <Result TypeError(u'Expected a pinned InstallRequirement, got unknown from htt...e38 (from pyramid==1.5.8->fake-with-deps==0.1->-r requirements.in (line 1))',)>.exit_code

C:\projects\pip-tools\tests\test_cli_compile.py:240: AssertionError
---------------------------- Captured stderr call -----------------------------
You are using pip version 8.1.1, however version 19.3.1 is available.

You should consider upgrading via the 'python -m pip install --upgrade pip' command.

------------------------------ Captured log call ------------------------------
WARNING  pip.req.req_install:req_install.py:453 Running setup.py (path:c:\users\appveyor\appdata\local\temp\1\tmpq7ozxubuild\venusian\setup.py) egg_info for package venusian produced metadata for project name unknown. Fix your #egg=venusian fragments.

Expanded assertion error:
TypeError(
    u'Expected a pinned InstallRequirement, got unknown from '
    u'https://files.pythonhosted.org/packages/53/d9/29816f12828a5ff539e66afe270a764d07bf4bb4cefc6613e331c9d38ba9/venusian-3.0.0.tar.gz#sha256=f6842b7242b1039c0c28f6feef29016e7e7dd3caaeb476a193acf737db31ee38 '
    u'(from pyramid==1.5.8->fake-with-deps==0.1->-r requirements.in (line 1))'
)

Additional context
Venusian package has recently dropped Python 2.7 support, see the changelog. Also, it moved all setup keywords to setup.cfg, which most likely causes the problem, see warning:
WARNING  pip.req.req_install:req_install.py:453 Running setup.py (path:c:\users\appveyor\appdata\local\temp\1\tmpqebvambuild\venusian\setup.py) egg_info for package venusian produced metadata for project name unknown. Fix your #egg=venusian fragments.

Possible solution is to pin venusian<2.0.0 in tests/test_data/packages/fake_with_deps/setup.py.

Or even better, install fresh virtualenv!


tiles option leads to rendering artefacts

The usage of the tiles option (e.g. --tiles 2) leads to rendering artefacts.

If the aim is to create one large image (e.g. 25000x25000 pixels) the tiles option isn't necessary at all. mapnik has (as far as I know) no size limit. It seems that the problem comes in with the resize function (which seems to be limited [16384?]):
...
    # for layer processing we need to create the Map object
    m = mapnik.Map(100, 100)  # temporary size, will be changed before output
    mapnik.load_map_from_string(m, style_xml.encode("utf-8"), False, style_path)
    m.srs = proj_target.params()
...
    # export image
    m.aspect_fix_mode = mapnik.aspect_fix_mode.GROW_BBOX
    m.resize(size[0], size[1])
    m.zoom_to_box(bbox)

A possible solution to avoid the rendering artefacts (if the aim is to create one large image) is to avoid tiling and resizing.
...
    # for layer processing we need to create the Map object
    m = mapnik.Map(size[0], size[1])
    mapnik.load_map_from_string(m, style_xml.encode("utf-8"), False, style_path)
    m.srs = proj_target.params()
...
    # export image
    m.aspect_fix_mode = mapnik.aspect_fix_mode.GROW_BBOX
    m.zoom_to_box(bbox)
...

Thanks for finding the issue, Klaus. I don't remember exactly, but this resizing fixed some issue with resolution or density, I don't know. Maybe size may be not available before the style is loaded.
If you need tiles from a mapnik stylesheet, I recommend using polytiles — it supports rendering in metatiles and all that.


Create GitHub Action to deploy docs

GitHub action should be clickable and deploy the docs.
Here's the delta-rs GitHub action we use for this task: https://github.com/delta-io/delta-rs/actions/workflows/docs.yml

@MrPowers Can I take this one?

take

Hey @dennyglee, do I need special permissions to run workflows in my forked repo? The action I'm working on, along with the others, isn’t showing up in the 'Actions' section.

Hey @paulooctavio - there should not be any special permissions to run this in your own repo.  Can you see the workflows when you go to the .github folder and try to manually execute them?

Hey @dennyglee it seems it was just some weird GitHub UI bug, now I can see the Actions. I'm just not entirely sure how to test the Action in the unitycatalog repo. I tested it by setting up a workflow to run on pushes to a test branch in my forked repo, and the deployment was successful.

Hey @paulooctavio - this seems like it's working on your fork.  Can you create a PR to the unitycatalog repo and then one of us can review and test it?  Thanks!

@dennyglee Sorry, I forgot to mention it but I already made a PR https://github.com/unitycatalog/unitycatalog/pull/677 unitycatalog, or is that it not the correct repo yet.

Oh yeah, I forgot about that one - could you please address comments by @HorizonNet and then feel free to tag me after they yhave been resolved?  Thanks!


CHDR introducing version

Hi Roastbeef,
Although in the original paper is not expected, IMHO it would be appropriate to add a byte at the beginning of Common header to be used to versioning the Hornet protocol (as indeed exists in the IP protocol). My proposal is:

Bit
0-3
4-7
8-15
16-23
24-

0
Version
Future Uses
Control Type
Hops
...

Andrea

@Roasbeef @cdecker
What is described above proceeds?
I found no evidence of this.


Removed unnecessary [Include] statements.

https://github.com/MicrosoftDocs/azure-docs/issues/108934

#sign-off


can't parse csv file

can't parse csv file

I'll need more details. Do you mean a literal csv file, because only tiled files should be parsable. If you mean a tiled file with map data stored in csv format, do you have an example file you could attach that I could test?


Update eclipse-che operator for openshift to release 7.11.0

Signed-off-by: Anatoliy Bazko<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@J0zi
===== Test: operator-olm-deployment =====
 Failed to deploy the OCP cluster on AWS: hudson.AbortException: script returned exit code 1

I am wondering if there is anything I can help with?

@tolusha, pipeline is broken on our end. We are working on it.

/test CVP


Compute needs to return a Damage Frequency Curve for plotting

As discussed in the 12/2/21 biweekly meeting

Do we want to have separate classes or maybe just a separate method that does the preview compute?
The results from the preview compute are different from those of the compute. We only need an ead of type double and a damage frequency of type paired data.


fix(utilities): add new util to remove html tags and entities

Related Ticket(s)
Component: ImageWithCaption & CTA - <p> tag is visible on content description. #2754
Description
Create new utility removeHtmlTagEntities to remove html tags and entities as the description coming from the videoplayer api contain them.
Changelog
New
-removeHtmlTagEntities to return just a plain string with no html tags or entities
Changed

have LigthboxMediaViewer use removeHtmlTagEntities for video description

@annawen1 can you point this to the release branch?

@annawen1 can you point this to the release branch?

whoops forgot! done


Display README.md content if it is present

Similarly to GitHub, if a file called README.md is present at a branch's root, render and show it in the UI.
The content should be markdown-rendered and displayed only when viewing the repository root.

Done by #3761, nice one!


Requirements don't seem to be installed

I am trying to set up a clone of the ModSimPy repo.
As I could not find a way to import an environment.yml file, I added the packages listed in the repo environment file to the Requirements section of the project.
When I tried importing modules in a new or existing notebook, I found that three imports were not working: pint, tables, and modsim. It seems that modsim did not work due to the lack of Pint in a new notebook, but in an existing notebook from the repo, it did not work at all.
The workaround to this is to open a terminal and use pip to install the missing packages, but this seems a bit clunky. Additionally, I'm considering using this platform for teaching, and it doesn't seem ideal to have students do this workaround themselves.
Is there something else I need to do to make sure that the right packages are installed upon launch?

Hello @syclops, thanks for reporting. We will take a closer look at the problem and come back to you. If you need any extra help for your teaching activities, don't hesitate to contact us.

@syclops the first thing I noticed is that pytables>=3.5 is not available in pypi. If you want to install it with pip you should use tables instead.
This error is probably breaking the whole pip install process and anything else coming after this error won't be installed. I guess this can be the reason of the issue you are experimenting.
Can you please make sure each individual package you have in the project settings can be installed correctly and the version matches one available version in pypi?
Let me know if there're still problems after it.

@martinzugnoni it looks like changing to tables does work, thanks.
However, after experimenting a bit, I think that any error in the requirements breaks the entire installation, not just the packages that come after it in the list.
I can't verify this directly as I'm not sure what packages are installed in the Data Science kernel. But the clue was that pint seemed not to be installed either with pytables as a requirement.
I will close the issue as it seems to have been solved, but documentation somewhere that describes what the Data Science kernel includes or what its image is composed of would be fantastic. Thanks!


ARROW-14286: [Python][Parquet] Allow to select columns of a list field without requiring the list component names

11385

 Quality gate: Failed
145 Issues

 2 Vulnerabilities
 126 Code issues
 17 Anti-patterns

Details

 Quality gate: Failed
146 Issues

 2 Vulnerabilities
 128 Code issues
 16 Anti-patterns

Details

 Quality gate: Failed
 Everything looks good
Details


Add unit test for repository update API after merging replication_enhancement branch as the new test framework is in that branch

Test Put method of Repository API.


Invoke-TmfAccessPackageResource errors when adding an App Resource

Getting the above error when using TMF to adding an App Resource to an access package. Running Test-TMFTenant shows the App Resource  will be created

Notice your using Invoke-MgGraphRequestMethod for this and that there is now a cmdlet for this  type of request. This might have something to do with it?

I noticed when running TMF locally to troubleshoot that "resourceType" for apps was defaulting to "AadGroup", so in the json file I added "resourceType":"Application". This resulted in a new error about "originId".

I then defined "originID" in the json but still got the same error. Seems like an issue with looking up application resource attributes when building the payload for Graph.

The issue appears to with this part of the code
https://github.com/ATVWGS/tenant-management-framework/blob/1b3c643ccaf8fa98a712fa8b366db972e5974557/TMF/functions/entitlementManagement/accessPackageResource/Register-TmfAccessPackageResource.ps1#L65
The switch only has AadGroup or DEFAULT to set the origin ID. In the above it would be using the deault
A work around for the issue is to add a parameter of "resourceIdentifier" and set the value to ObjectID of the application

We've created an internal bug for this.

@ATVWGS , are there any updates to resolving this? Any chance an updated build will be coming out soon?

Hi @brine0m,
I've just released a preview version that is working with applications in accessPackages. A lookup of valid roles within the Register function can not work, because a new accessPackageResource is not yet deployed during the Register flow and therefore roles of the application cannot be gathered. The validation set was completly removed.
Here is an example how accessPackagesResources could look like:
"accessPackageResources": [
{
"originSystem": "AadApplication",
"resourceRole": "Default Access",
"resourceType": "Application",
"resourceIdentifier": "Graph Explorer"
},
{
"originSystem": "AadGroup",
"resourceRole": "Member",
"resourceIdentifier": "Some group name"
}
]
Note: The "resourceRole" attribute of the application can only be the displayname of the role.


Vulkan Initialization Problems

Just recently installed the latest version of conty (1.21) and am trying to use it to execute multilib applications such as Steam. Upon running ./conty.sh steam, I encounter the following error:
vkEnumeratePhysicalDevices failed, unable to init and enumerate GPUs with Vulkan.
BInit - Unable to initialize Vulkan!

Steam itself loads right up and everything works as expected. However, upon launching a game it will begin launching and then immediately crash. Take DS3 for example, which launches to a white screen while the cursor changes color and then immediately crashes. The error log includes the following lines:
ERROR: ld.so: object '/home/irondeau/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.
ERROR: ld.so: object '/home/irondeau/.local/share/Steam/ubuntu12_64/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS64): ignored.
ERROR: ld.so: object '/home/irondeau/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.
ERROR: ld.so: object '/home/irondeau/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.
ERROR: ld.so: object '/home/irondeau/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.

which also happens to be thrown when I run the game Hollow Knight (which is supposed to run natively on Linux). I am unsure if the two errors above are related.
The next most natural debugging step is to run ./conty.sh vulkaninfo whose ouput is this:
ERROR: [Loader Message] Code 0 : loader_scanned_icd_add: Could not get 'vkCreateInstance' via 'vk_icdGetInstanceProcAddr' for ICD libGLX_nvidia.so.0
ERROR: [Loader Message] Code 0 : setup_loader_term_phys_devs:  Failed to detect any valid GPUs in the current config
ERROR at /build/vulkan-tools/src/Vulkan-Tools-1.3.217/vulkaninfo/vulkaninfo.h:231:vkEnumeratePhysicalDevices failed with ERROR_INITIALIZATION_FAILED

The glxgears command works fine, so I assume the driver configuration is correct (using nouveau drivers). Firefox also works perfectly fine.
I appreciate your help, and thank you for building this awesome tool.

Distro
Void Linux with Musl

Display Server
Xorg

Window Manager
dwm

The glxgears command works fine, so I assume the driver configuration is correct (using nouveau drivers).

Hi. The problem is that Nouveau does not support Vulkan currently, only the proprietary driver does. I understand that the proprietary driver does not work with musl, but that's the only way to use Vulkan for now (assuming your GPU supports Vulkan at all).
There is also a software implementation of Vulkan (called lavapipe or vulkan-swrast), which works with any videodriver and with any GPU, but it is very slow and has its own issues.
By they way, i would also suggest to check glxinfo -B to check if nouveau is actually used instead of llvmpipe (which is software implementation of OpenGL).

By the way, for Hollow Knight you can force it to use OpenGL renderer by adding -force-opengl to its launch options in Steam. And to make Proton to use OpenGL instead of Vulkan, run Steam with PROTON_USE_WINED3D=1:
PROTON_USE_WINED3D=1 ./conty.sh steam

Not all games will work this way though (for example, DX12 games won't).


Re-design User Interface

Hi @optionalemon, I would recommend changing the title to something more concrete, i.e. "adds split screen view for classes and sessions". So that it's easier to track tasks.


How to connect with MySQL Database?

Is there any sample available to connect with MySql database using Kitura?

I am not aware that there is a MySQL Connector  yet for Swift. That could be a valuable contribution to the community. In the meantime, I believe the next best option would be use the MySQL C connector from a Kitura app. We don't have a sample for that at the moment.

@kweinmeister I wonder if one could be ramped quickly using something like SwiftSocket.
For example, there is no official nodejs connector either, so a project like like node-mysql simply opens a socket connection to mysql.
I'm currently not in a position to ponder performance implications of this over another possibly more idiomatic method.

Kitura uses BlueSocket for its socket connections.  That could probably be used to connect to MySQL using the socket interface.  It'd have to be investigated further but I see no reason why it couldn't be done.

Note that MySQL Connector/C is GPL.

@billabt Thanks for the clarification (I'm brand new to Kitura). I'd be interested in taking a look if this is currently unaddressed.

@michaelsanford: Sure! Any help with this would be appreciated.

@michaelsanford: As @kweinmeister said, any and all help would be greatly appreciated. If you need any help with BlueSocket setting it up, just go ahead post your questions/comments as an issue on the BlueSocket site and I'll respond as quick as I can. Thanks!!!  This probably a really good way to get your feet wet. 😃

There is are several swift based connectors as part of https://github.com/PerfectlySoft/Perfect/tree/master/Connectors, they are part of their own server side solution but was just released under  Apache 2.0 license, maybe can fork, and prepare something that could be pulled with the package manager.

@generatronServer : Yes i have used Perfect My SQL connector.. It worked for me..
Just 1 question regarding Swift Compilation. If i use libmysqliclient.a library, is it necessary to put that library in to /usr/local/lib? Without that i am getting linker error.

I think yes, you need that library available in the path

@chintan1891 You can add flags to swift build and specify the location of  libmysqliclient.a:
-Xswiftc -I<the location of libmysqliclient.a> -Xlinker -L<the location of libmysqliclient.a>

Any updates on a MySQL driver we can use for Kitura-TodoList?

I know @nicholasjackson had something in the works, nevertheless I would love to see a swift native driver converted to Swft 3 if anybody wants to take the challenge. https://github.com/mcorega/MySqlSwiftNative

Ah github the bane of my life, I never get the notifications.
We have been using this in production, it still needs parameter escaping which I hopefully will be able to do this week and probably bringing in line with the latest releases.  I will do that tomorrow.
https://github.com/notonthehighstreet/swift-mysql

@ejohnsonw Swift 3 PR opened :) https://github.com/mcorega/MySqlSwiftNative/pull/17


phi3.5:3.8b-mini-instruct is missing parameters on Ollama's website.

What is the issue?
As shown in the screenshot, there is no parameters, so ollama run phi3.5:latest will not function correctly.

Here is the corrected modelfile if anyone needs it:

FROM phi3.5:3.8b-mini-128k-instruct-q8_0
TEMPLATE """{{ if .System }}<|system|>
{{ .System }}<|end|>
{{ end }}{{ if .Prompt }}<|user|>
{{ .Prompt }}<|end|>
{{ end }}<|assistant|>
{{ .Response }}<|end|>
"""
PARAMETER stop "<|system|>"
PARAMETER stop "<|user|>"
PARAMETER stop "<|end|>"
PARAMETER stop "<|assistant|>"

OS
Windows
GPU
Nvidia
CPU
AMD
Ollama version
0.3.8

Thanks for the corrected modelfile. This was reported has week in #6449 and #6474 so I'm surprised it still hasn't been fixed yet.

phi3.5 has been updated with parameters


newsletter link fix

Hey, I just made a Pull Request!
newsletter link fix
fix #17620

[ ] A changeset describing the change and affected packages. (more info)
[x ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

@freben correct fix ?


chore(CHANGELOG): add message about chai typings fix

The commit message for #2120 was not marked as fix() so it wasn't picked up. Understand why it wasn't, cause...it's not really a "bugfix" in the sense of a runtime bug...just a compile/typings bug.
Cc/ @kwonoj @blesh

Coverage remained the same at 97.685% when pulling e328757b0e1dd8990746e43681bb548eec47dc2f on jayphelps-patch-1 into e91b07b1b21ec4fb64b521267b34052844201c2a on master.

LGTM


IPFS hash for orbit db record

Hello, I use new OrbitDB(ipfs).eventlog('MY_DB'); How I can obtain IPFS hash for my record or any other data structure which I can check directly by ipfs cat ... ?

You're looking for the load.progress event: https://github.com/orbitdb/orbit-db/blob/master/API.md#events

You get the IPFS hash of an individual operation as the return value of the operation. Eg.:
const orbitdb = new OrbitDB(ipfs)
const feed = orbitdb.feed('my-feed')

// Get a hash of an operation
const hash = db.add('hello')
console.log(hash)

// Remove the entry in feed
db.del(hash)
console.log(db.iterator({ limit: -1 })

db.iterator() will also return the entries in their "raw" form, ie. the actual OP and its hash is included in the result.
Hope this helps! Feel free to re-open the issue if you have more questions.


Failed to install

(xxx) C:\Users\Jkdell>pip install cnmaps
Collecting cnmaps
Using cached cnmaps-0.1.10.tar.gz (2.3 MB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [6 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "C:\Users\Jkdell\AppData\Local\Temp\pip-install-9qi40z8x\cnmaps_4a8842083b524713a538b30839062a41\setup.py", line 7, in
long_description = fh.read()
UnicodeDecodeError: 'gbk' codec can't decode byte 0xaa in position 17: illegal multibyte sequence
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

@Jack-ee Hello, Jack. I realeased a new version of 0.1.11 for pypi yesterday, can you try to install again with pip install -U cnmaps ?

安装成功 了，谢谢！
On Mon, Feb 14, 2022 at 9:12 AM Wentao Li @.***> wrote:

@Jack-ee https://github.com/Jack-ee Hello, Jack. I realeased a new
version of 0.1.11 for pypi yesterday, can you try to install again with pip
install -U cnmaps ?
—
Reply to this email directly, view it on GitHub
https://github.com/Clarmy/cnmaps/issues/1#issuecomment-1038519974, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5J3HRJAWHEPQM6KUGUGDTU3BJH5ANCNFSM5OH5JWEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@Jack-ee 好的，收到。


Order CreateError on long name/secret ['.-' not supportedd]

K8S: 1.18
Cert-Manager: 0.9 (old though perhaps this still 0.16)
It seems that cert manager truncates name (=>if length is bigger than expected/supported) and then appends random numeric string at the end. Turns out that, like in this case,  we ended up with ".-" and therefore "order" could not get created as cert-manager does not seem support '.-' combo (see below).
secret: payment-manager.internal.payment.uswest2.production.teste.c-tls
Warning  CreateError  18m (x789 over 16d)  cert-manager  Failed to create Order resource: Order.certmanager.k8s.io "payment-manager.internal.payment.uswest2.production.-3327846806" is invalid: metadata.name: Invalid value: "payment-manager.internal.payment.uswest2.production.-3327846806": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is 'a-z0-9?(.a-z0-9?)*')
pls ping me if more info is needed. thanks

We have an open PR for this for v1.0: https://github.com/jetstack/cert-manager/pull/3232

@meyskens thank you for prompt response. appreciated. When will v1.0 (with this PR) be out?

Scheduled for 2nd of September, tip you can follow those at: https://github.com/jetstack/cert-manager/milestones


Package davejamesmiller/laravel-breadcrumbs is abandoned

This is only a composer warning.
Don't worry, but we still using it?

Hi @gioacchinopoletto , yes, the author of the package decided to stop developing it.
We have already created an issue #1073 on this subject.
Moreover, I prepared an implementation that will replace this in the next version. We have time to fully prepare the new release until August when this package turns into a pumpkin


Add support for https listen

We can set on front https endpoint, but without any proxy (nginx etc) can not listen https with webwire server.
This pr add support for tls listen

Currently, to be able to use TLS encryption a custom HTTP server hosting the webwire server must be set up. It's demonstrated in the external example. But It'd probably be a good idea to add a new constructor function with TLS support, like:
func NewSecureServer(
	implementation ServerImplementation,
	opts ServerOptions,
	certFile,
	keyFile string,
)

to newServer.go

#35

#35 is now resolved.


account selection popup

I'm using the firebase_auth package for email/pass, google, facebook sign in on iOS.
The problem I face is that once I sign in with google account,
and then sign out. (firebaseAuth.signOut(), googleSignIn.signOut())
And I sign in with google account again,
it doesn't show the account selection popup.
I've use the signInSilently() as below:
GoogleSignInAccount googleSignInAccount = _googleSignIn.currentUser;
if (googleSignInAccount == null) googleSignInAccount = await _googleSignIn.signInSilently();
if (googleSignInAccount == null) googleSignInAccount = await _googleSignIn.signIn();

But it shows up the sign in process every time,
so I have to type the username every time.
Is there a way to solve this?
To enable the account selection popup.
Thanks a lot!

Hi zoechi,
Thank you so much for your reply.
I checked my code, and yes I did.
In my code:
final FirebaseAuth _firebaseAuth = FirebaseAuth.instance;
and I did what you suggested in my sign out function:
  Future<void> signOut() async {
    await _firebaseAuth.signOut();
    await _googleSignIn.signOut();
  }

The problem remains still.
But thanks again for your reply!
(By the way, I use firebase's version: ^0.6.5, google_sign_in: ^3.2.4, hope it helps.)

Please add the output of flutter doctor -v.

Hi zoechi, thanks!
Here's my output:
$ flutter doctor -v
[✓] Flutter (Channel beta, v0.11.3, on Mac OS X 10.14.1 18B75, locale en-TW)
    • Flutter version 0.11.3 at /Users/tsousean/development/flutter
    • Framework revision 72bf075e8d (11 days ago), 2018-11-09 20:36:17 -0800
    • Engine revision 5646e86a6f
    • Dart version 2.1.0 (build 2.1.0-dev.9.3 9c07fb64c4)

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/tsousean/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 30.0.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] VS Code (version 1.29.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.20.0

[✓] Connected device (1 available)
    • Sean1776 • 5e7738ac514d2898fb1d4f7a9a9d4864f1475225 • ios • iOS 12.0.1

• No issues found!

@sean1776
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/574. Any further collaboration will be done there.

Hi @sean1776
If you are still experiencing this issue
with the latest version of Flutter and FlutterFire plugin
can you please provide your updated  flutter doctor -v
and your and your flutter run --verbose/flutter build --verbose ?
Also, to better address the issue, would be helpful
if you could post a self contained app on github
or the steps to reproduce it.
Thank you

Reopening here since this issue looks to be a Google Sign In issue and not a Firebase one.

Hi @sean1776
are you still experiencing this issue with the latest version of Flutter?
If so please provide your updated
flutter doctor -v and flutter run --verbose
together with a minimal code sample to reproduce the problem
thank you


save count not working properly when connected to dev .

earlier save count was coming after some seconds ... it was requiring some time to show updated save count . now we are showing loading infront of save count and after it successfully gets the result from the db we are showing the result in ui .


react-dom/server could not be resolved in react-apollo in Next@5

ERROR in ./node_modules/.registry.npmjs.org/react-apollo/2.1.0-beta.0/node_modules/react-apollo/renderToStringWithData.js
Module not found: Error: Can't resolve 'react-dom/server' in '/Users/Vaughan/dev-live/vwp/node_modules/.registry.npmjs.org/react-apollo/2.1.0-beta.0/node_modules/react-apollo'
 @ ./node_modules/.registry.npmjs.org/react-apollo/2.1.0-beta.0/node_modules/react-apollo/renderToStringWithData.js 3:15-42
 @ ./node_modules/.registry.npmjs.org/react-apollo/2.1.0-beta.0/node_modules/react-apollo/index.js
 @ ./packages/next-apollo/withData.js
 @ ./pages/map.js
 @ multi ./pages/map.js

See https://github.com/zeit/next.js/blob/f9b52cfcb6cba5f2af93caf3cc8bc3636571323f/server/build/webpack.js#L155
This aliasing seems to prevent require sub-paths.
PS. I am using pnpm. Maybe this could be related...
Workaround
Add this to your next.config.js:
delete config.resolve.alias['react-dom']

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

Your Environment

Tech
Version

next
5

node
8

OS
macOS

browser
Chrome

etc

i am also having issues with react-dom/server not being found after i upgraded as well as other random issues that were not occurring before :\
i want to upgrade my next 4 project but things are breaking in a weird way (erratically… sometimes it works sometimes it doesnt)

for example, im using hot reloading for and changing a css file now throws: Module build failed: Error: ENOENT: no such file or directory, open /path/to/file.css.json. i've been using a file loader in next.config.js to transform css before passing to postcss. this stopped working now in version 5 :\

@mgiraldo See my workaround.
What you could try is printing the final webpack config from your Next@4 app and then comparing to Next@5.

I am also having same issue /cc @arunoda
It have in this live stream https://www.youtube.com/watch?v=yCDLVpyQ5NI

@pantharshit00 @vjpr
I can see why this is happening.
I'll have a look.

@vjpr could you check with bundle-analyzer and see react-dom/server used in the client side or not.
If so, try to remove it with our custom webpack config.

@arunoda This fixes this in our project.
https://github.com/fus-marcom/franciscan-react/blob/next5/next.config.js
Here is our babel plugin module resolver
https://github.com/fus-marcom/franciscan-react/blob/e7d694a61b8989d360141dc916a32beec6c6789e/.babelrc#L21

@pantharshit00 thanks for the input.
We didn't realized react-dom/server is gonna used inside the client.
We may want to add an alias for that.
(For the production build, we use React's minified production bundle)

I had the same issue. Made it work with using this:
// working
import { server } from 'react-dom';

server.renderToString(<Component />);

instead of:
// not working
import { renderToString } from 'react-dom/server';

renderToString(<Component />);

Hey, this fix exists in https://github.com/zeit/next.js/releases/tag/5.0.1-canary.2

It took me a while to figure it out, so in case anyone as has the same problem doing delete config.resolve.alias['react-dom'] will break the dead code compilation of your production build but the fix from @arunoda in 5.0.1-canary.2 provides a proper fix without the breaking the compilation.


Add repository link to crate metadata where missing

Cool crate! But was not obvious how to find the repository from the crates.io page, so I thought I could contribute this to help out :)

Can one of the admins verify this patch?


[WebUI][ESRGAN] Add ESRGAN to WebUI

-- This commit includes ESRGAN as part of WebUI.
Signed-off-by: Abhishek Varma<EMAIL_ADDRESS>

This is in accordance with ESRGAN example PR that I've raised earlier.
I've tested the CPU execution and would want to aim to get this PR in soon post review comments.
Post that I'll chart on to the GPU/Vulcan execution (and that'd involve a minor change in a follow up commit).
Do let me know your review comments and I shall address that - you may run this patch locally to verify/perform any observation for review as well.
@powderluv @pashu123 @Shukla-Gaurav @PhaneeshB @vivekkhandelwal1

This has been tested locally for the CPU execution both on MacOS and Linux - the upscaled image gets generated.
The public URL always leads to Connection timed out error but I've verified the file being generated locally.
Can someone please review this - @pashu123 @Shukla-Gaurav @PhaneeshB @vivekkhandelwal1 @powderluv ?

@powderluv @Shukla-Gaurav - Just like the standalone ESRGAN PR I see the same unrelated CI failure here - please help review and merge this PR in properly soon.
And since this is related to WebUI - I believe it'd be better if CPU/Cuda/Vulkan execution is checked along with the setting up of ESRGAN here, before aiming to merge this PR in.
Thanks!


ADBDEV-5897: Fix ABI tests

Fix ABI tests
GitHub actions/checkout@v3 is no longer working and v4 is incompatible with
older Linux versions due to https://github.com/actions/checkout/issues/1590.
This patch uses workaround.

Add comment to yaml file as it was done at the same PR at the other projects. Example: https://github.com/SciCompMod/memilio/pull/1063/files

Allure report https://allure.adsw.io/launch/74556

Failed job Behave tests on ppc64le: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1608240

Failed job Resource group isolation tests on ppc64le: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1608242

Add comment to yaml file as it was done at the same PR at the other projects. Example: https://github.com/SciCompMod/memilio/pull/1063/files

added

Allure report https://allure.adsw.io/launch/74571

Failed job Behave tests on ppc64le: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1608646


Fixed the issue when reconstructing h264 frames that are not continuous

Hello, I noticed that there is an issue with RTP frame handling (h264), when receiving a stream with a noticeable packet loss. The crash happens in the following case. Consider the client receiving these RTP frames:
Packet seq 100, type = FT_START
Packet seq 101, type = FT_MIDDLE
Packet seq 102, type = FT_END // This packet lost
Packet seq 103, type = FT_START // Reconstructed (deallocated)
Packet seq 104, type = FT_END  // Reconstructed (deallocated)
Packet seq 105, type = FT_START
Packet seq 106, type = FT_END

In that case the packet seq 102 was lost. When the library receives seq 104, then it can successfully reconstruct the NAL unit from seq 103, 104 and deallocate them. Then the library receives seq 106 and incorrectly determines the frames 100-106 as continuous and attempts to reconstruct the NAL unit from them -> resulting in a crash when accessing deallocated 103 frame.
Looks like the issue is that continuous flag was never reset to false in case the frames are not continuous.

Example of the crash:

Also added a commit to fix the following case:
Packet seq 100, type = FT_START
Packet seq 101, type = FT_MIDDLE
Packet seq 102, type = FT_END // This packet lost
Packet seq 103, type = FT_START // This packet lost
Packet seq 104, type = FT_MIDDLE
Packet seq 105, type = FT_END

Previously, the library detected the sequence 100...105 as continuous and could crash. Updated the check to fix this case.

Hi, thanks for the pull request! This is a welcome fix.

I merged this, thanks.
On a related note, last week I rewrote the whole section of code last week as there was a bug related to sequence number roll-over. I think I also fixed most problems related to checking continuity of frames. If there are any issues, feel frame to open an issue or make another PR.
BR, Joni


How to finetune with new dataset ?

Do you plan to release finetune code ? or If I want to finetune model with new dataset (different classes), how  can I remove the weights in arcface_loss after load pre-trained model in order to meet the new dataset as did in MXNet version https://github.com/deepinsight/insightface/blob/master/deploy/model_slim.py

@auroua
same question. And hope to release finetune code.
Thanks.


Feature: Master bypass?

PS. This new version of eqMac is simple and beautiful 🔥  Nice work, thank you!

Actually, nevermind I guess.  I just noticed the global slider within the EQ module...


Fix syntax in mergetool.yml

Overview
I left a stray line in #10022, causing the yml syntax to be poorly constructed. This PR fixes it.

/azp run


Format route

I think it would be beneficial if Knockout-SinglePage provided the functionality to format a URL based on the route name this allows users to use the extension to create the URL so that if the url for the route changes they do not have to find every where that URL is and update it. If the route has parameters then the can be passed as secondary argument to the method.
This functionality could potentially be gotten for free or would an assistance to issue #4.
Example 1
ko.singlePage.init([ { name: 'default', url: '/' } ])

var url = ko.singlePage.getURL('default')

The value of url would be /.
Example 2
ko.singlePage.init([ { name: 'default', url: '/:id' } ])

var url = ko.singlePage.getURL('default', { id: 1 })

The value of url would be /1.

It would also be quite neat if this was exposed as a knockout binding as well perhaps called link which accepts either a string or an object.
An example of what this would look like is given the following routes:
[
	{ name: 'default', url: '/' },
	{ name: 'profile', url: '/user/:id' }
]

This binding
foo

would result in
foo

and this binding
foo

would result in
foo


Tensorflow - Error: Cannot convert value dtype('<f4') to a TensorFlow DType

I receive an error when testing a Neural Style Transformation project with Tensorflow & OpenCV using the "lion" test image. The source of test project is:  [ https://github.com/cysmith/neural-style-tf ]. Both Tensorflow and OpenCV packages were installed and compiled from source. The training model used is: imagenet-vgg-verydeep-19.mat and the runtime environment runs on a s390x CPU and does not use any CUDA support. The  current Python environment is 2.7 under Docker Linux Ubuntu 16.
Run command:
bash stylize_image.sh ./image_input/lion.jpg ./styles/kandinsky.jpg
The error message at log is:
 File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/dtypes.py", line 584, in as_dtype
    "Cannot convert value %r to a TensorFlow DType." % type_value)
TypeError: Cannot convert value dtype('<f4') to a TensorFlow DType.

Versions installed:
python 2.7.12
import tensorflow as tf
>>> print tf.VERSION
1.3.1
import cv2
>>> print cv2.__version__
3.3.0
>>> print np.__version__
1.13.2

Is this a known issue?

This looks like an issue with neural-style-tf project (probably was developed for older version of tf)

@claudefalbriard Perhaps <f4 is forgot to be converted for neural-style-tf project.
In [8]: import tensorflow as tf

In [9]: import numpy as np

In [10]: tf.as_dtype(np.dtype("<f4"))
Out[10]: tf.float32

Closing to keep issues focused on  TensorFlow-specific bugs

Thanks for the hints and orientation. The "<" symbol describes an array in "Little Endian" notation, while the runtime platform is a "Big Endian" CPU. The issue seems to point to the dtypes.py module in the tensorflow framework folder, which might not support it. I will check the demo project, or model file and see if its possible to adjust the data through a manual Numpy byte swapping, or TF casting function.  By adding the "" to the _NP_TO_TF translation table, I was able to correct the run-time exception, but got an erroneous result in the output file, likely explained by the missing endianness adjustment.


Error 404 for https://cdnjs.cloudflare.com/ajax/libs/toastr.js/latest/js/toastr.min.js

The minified JS is no logner present from the following path:
https://cdnjs.cloudflare.com/ajax/libs/toastr.js/latest/js/toastr.min.js
However the unminified is still present:
https://cdnjs.cloudflare.com/ajax/libs/toastr.js/latest/js/toastr.js

I see it's present from the following path
https://cdnjs.cloudflare.com/ajax/libs/toastr.js/latest/toastr.min.js
But this still might break someones code due to this change.

I've created https://github.com/cdnjs/cdnjs/issues/8779 in CDNJS repo to track the issue.
I will update thread with details.

CDNJS maintainer here, I'm sorry about this accidently appeared breaking change, please allow me to explain the reason and discuss about the solution.
Actually, the policy of CDNJS, is not to support the "latest" usage, since there may be cache control issue on that usage, and will make the hosting process much more complex, however, the PR cdnjs/cdnjs#1500 was merged, and now there is a breaking change.
Aonther reason is that we are now using auto-update mechanism for most of the libraries, there are now 2.4k libraries hosted on CDNJS, it's impossible to manually update the libraries anymore, by using auto-update mechanism, we will/can only use the same directory/file structure from upstream, the css/js file in toastr before v2.1.0 is manually put in individual js and css directory, however, there is no directory under the build folder, so once the auto-update applied, the css/ and js/ disappeared in the url, I'm sorry about that.
Now I wondering if we could

just "fix" the url structure, but just keep the "latest" stuck on "v2.1.3", do not update it anymore, just for fixing/preventing the breaking changes, no more updates, since we can not always manually maintain that structure, unless, authors of toastr will like to send PRs on every toastr release.

one another thought is to add the css and js folder in the build folder in toastr repo, and manually "fix" the url structure for toastr v2.1.1~2.1.3 on CDNJS(but also keep the files without the css/js folders), since the structures will be the same, so I may also update the "lastet" link to the real latest one sometimes(I can not gaurantee that, for the same reason).

Do you guys have any suggestions? Many thanks.

@PeterDaveHello Thanks for taking your time to explain! I'd go the solution 1 and add some explanation comment on the first line of the CSS and JS files.

Thanks for the detailed reply.
Let's move forward with the first option, please.

Sure be fine now, thank you guys.


Live version returns a 403 Forbidden

Hello,
Just to let you know that the link in the README toward the live version of EscherConverter returns a 403 Forbidden error for me:
Forbidden

You don't have permission to access / on this server.

Possible causes of this error include:

    The request was forbidden by rules in the .htaccess file.
    The directory you requested does not have an index.html or index.php file.
    The permissions on the file or directory are incorrect.

For details about why the request was forbidden, see the Apache error log at:

/srv/users/SYSUSER/log/APPNAME/APPNAME_apache.error.log

Is that an expected behaviour?

@bdelepine thanks a lot for pointing this out! I am happy to fix it but so far I only see the target of that link, not the page where it is. Could you please state where that link is? Much appreciated!

Thanks a lot! Now I see what you meant. Unfortunately, my old department, the Cognitive Systems lab, retired that service. Since I do not have control over this web service myself, my only option was to remove the broken link from the README file. I am glad, you pointed this out, and apologize for the inconvenience. To use EscherConverter, please download the latest JAR file and double-click it to launch the app. Thanks!


Automated cherry pick of #4440: fix: struct api params

Cherry pick of #4440 on release/2.13.
#4440: fix: struct api params

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@15cd567). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #4441   +/-   ##
==============================================
  Coverage                ?   6.85%
==============================================
  Files                   ?     521
  Lines                   ?   88598
  Branches                ?       0
==============================================
  Hits                    ?    6076
  Misses                  ?   82016
  Partials                ?     506

Flag
Coverage Δ

#aFlag
6.85% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/guest_actions.go
0% <0%> (ø)

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 15cd567...0160103. Read the comment docs.


Fix code blocks

Fix code blocks, per Creating and highlighting code blocks.
Currently they are rendered incorrectly:
https://docs.microsoft.com/en-us/cli/azure/azure-cli-configuration

Docs Build status updates of commit c932607:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-configuration.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit c932607:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-configuration.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Thank you and ack the review request :) although I'm no expert in this area and will leave to @dbradish-microsoft  for additional comments

Thank you and ack the review request :) although I'm no expert in this area and will leave to @dbradish-microsoft  for additional comments


Add example that reads a local file, writes to a DFS path provided by th...

...e user, reads the file back from the DFS, and compares word counts on the local and DFS versions. Useful for verifying DFS correctness.

@andrewor14 @pwendell Happy New Year!  Ping on this PR?  Thanks!

Test build #26532 has started for   PR 3347 at commit af8ccb7.

This patch merges cleanly.

Test build #26532 has finished for   PR 3347 at commit af8ccb7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26532/
Test PASSed.

@andrewor14 if you like the change I think you can merge it; no other comments here. Otherwise i think this has timed out.

@rnowling sorry for slipping on this. I'll merge this into master after addressing the latest comments myself. Thanks for your patience.

Thanks @andrewor14 !


Spelling Mistake

Under implement Dialog steps: C#
it says  : we implment

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

ID: 72538432-fde5-22ab-7997-8a34b11aa24e
Version Independent ID: c469fa5b-2590-ee16-1491-51b90c069204
Content: Gather user input using a dialog prompt - Bot Service
Content Source: articles/v4sdk/bot-builder-prompts.md
Service: bot-service
Product: bot-framework
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

Fix should go live in a day or two. Thanks!


F/add incident name to channel topic

If I'm a user who is not all too familiar with the incident bot, I may get invited to an incident channel to help troubleshoot. If the user is unaware that the the incident is also displayed in the #incidents channel, there isn't a great way for them to get all of the information on the incident. So I thought adding the initially reported name of the incident as well as a link to the summary page might be beneficial.

Minor requests, otherwise this is all good.  Thanks 👍

Thanks for the change 👍   Can wrap the below in its own try/except?  Once that's in, I'll get it merged.
        doc_url = urljoin(
            settings.SITE_URL,
            reverse('incident_doc', kwargs={'incident_id': incident.pk})
        )

        set_channel_topic(channel_id, f"{incident.report} - {doc_url}")

@evnsio sure thing. That has been updated.

@evnsio is this PR ready to be merged now?


TTS through forked-daapd media_player entity in HomeAssistant?

The forked_daapd_server entity has a "text to speech" section within it's entity settings popup, but I cannot seem to get it to work either through the entity or by calling the tts.google_translate_say service to that entity. Is this something the forked-daapd integration is supposed to do? Do i have something misconfigured? Or, does the TTS mp3 that's generated have to somehow be sent to the pipe in the forked-daapd playlist?
Thanks again.

Yes, it works on my end, but haven't had a lot of feedback on it. Check that base_url is set in the TTS section as detailed here: https://community.home-assistant.io/t/no-tts-after-upgrade-to-110/198298/3
If that doesn't work, there might be some error messages in the logs which can help us figure out what's going on.
BTW, this repository is only for the backend library the HA integration uses. For questions or issues with the HA integration I think it's better to open topics at the forum https://community.home-assistant.io/ or specific bugs at https://github.com/home-assistant/core (it will still be me replying though :-))

@stephenmetzger Were you able to get this to work?

@uvjustin Thanks for following up on this. I haven't been able to get it to work, and it probably has something to do with how i have my HA instance configured (either with my proxy settings through NGINX or the way I have TTS set up in config).

Do you have any log events from HA or forked-daapd when you try to play TTS?
Do you know if your TTS works with other integrations?
I found a post detailing a workaround for problems with TTS and NGINX with Sonos, but it doesn't seem like a forked-daapd build would be missing the cipher in question https://community.home-assistant.io/t/solution-sonos-tts-with-nginx-ssl-reverse-proxy/58136


introduce grabAccessibilityTestResults method

grabAccessibilityTestResults
Retrieves accessibility test results of given URL. If no URL is given, the URL in the config will be used.
let results = await I.grabAccessibilityTestResults('http://localhost:8000/');

//Returned results
{ documentTitle: 'TestEd Beta 2.0',
pageUrl: 'http://localhost:8000/',
issues:
[ { code: 'WCAG2AA.Principle3.Guideline3_1.3_1_1.H57.2',
context: '<html><head><title>TestEd Beta 2.0</t...</html>',
message:
'The html element should have a lang or xml:lang attribute which describes the language of the document.',
type: 'error',
typeCode: 1,
selector: 'html' } ] }

Parameters

url (optional, default url in the config file)
string url to grab accessibility test results

This method doesn't use anything from a helper where it is defined. It also requires a new dependency...
So I don't think it should be the part of browser control helpers.
So it looks like this functionality should be moved to a standalone helper, and maybe to be extended with methods like: seeThisPageIsAccessible() or smth like that. Anyway, this should be a new helper.
Right now I would recommend to create that helper in a new package and add to our list of custom helper. https://github.com/Codeception/CodeceptJS/wiki/Community-Helpers


Columns' alignment

In a demo app, columns on the right hand side are slightly shifted. I had a similar issue and placing the table inside the div with table-responsive class helped, like described here. Perhaps can help here too?

@bsrdjan This is probably because I've hard coded the scrollbar size - I assume you are looking at this on a Mac (or something with a thin/thick scrollbar)
At some point I'll be tidying this all up so it should work better cross-browser


CSIM dies if JUMBO DUMP, never connects :(

either CSIM or DDM bug..

output message received from ddm on csim

occurs only when we have automatic timeout and receive the dump after trying to connect again
occurs when it takes longer to load the dump longer than we have in connection timeout


Proposal: avoid empty arrays

Problem summary
There are many different ways that a query can return a logical "no value"

The entity doesn't exist
The component doesn't exist
The user logged []
The user logged Clear
The data hasn't been logged yet
The data hasn't been logged on the queried timeline
The range queried doesn't overlap with the logged range

Depending both on the kind of missing data, and the particular query-API used, some of these will return None. Some of these will return[]. I doubt anybody on the team can tell you authoritatively which will do which.
Handling the inconsistency across all the code that does queries, especially when it comes to chaining and falling back across overrides, defaults, etc. and in the presence of possible clears pre- and post- garbage collection is a source of pain.
Proposal
I propose that we simply forbid ever allowing Some(EmptyArray()) as a result.
Anywhere that might return Some(EmptyArray()) should always return None instead.
The main rationale for this is that Rust has good support for working ergonomically with None types (map, or_else, ?, etc.) which should simplify many of the pathways where we do this handling.
More extreme proposal
Going a step further, I think we should try to forbid non-null zero-length-lists from the chunk data itself.  Anywhere in a chunk where our outer-most ListArray may contain zero-length elements, we should have a validity map, and we should update that validity map to set the null bit for any zero-length list.
This gives us a validity-map-only fast path for always identifying whether we have "real data" for a given component in a row of a chunk.

we should try to forbid non-null zero-length-lists from the chunk data itself

In https://github.com/rerun-io/rerun/issues/3381 we proposed that logging a null is different from logging an empty array. In that proposal, logging pos=[a,b,c], color=null means "keep old color" while logging pos=[a,b,c], color=[] means "use default colors, ignoring older colors. That's how the latest-at semantics work as well: look for the first non-null value (even if it is an empty array).

If we do this, we should try to do it well, so that the returned object is similar to Vec1 in that it:

Has no is_empty() method (because it can't be empty)
.first() and .last() returns a item reference, without wrapping it in Option (there is always a first and last element)
…


Option to make group of different devices from different node

Currently we can make group of nodes only, this means all the devices under the node comes under the group but when we want some of devices from one node and some of from other nodes comes in a group it is not possible. But in Alexa app we can do these type of grouping....if this is possible in future updates this is very appreciable.

one another thing....please give a option to add name to node id,  it is very difficult to recognize with node id....

Thanks for adding Device automation feature,
Please also add location automation, though which we can trigger any device though location data...eg. when we reach near our home AC will turn on....

Thanks in advance.

@nravin5 , the feature requests are quite valid, but please see my comments on these

Currently we can make group of nodes only, this means all the devices under the node comes under the group but when we want some of devices from one node and some of from other nodes comes in a group it is not possible. But in Alexa app we can do these type of grouping....if this is possible in future updates this is very appreciable.

The primary addressable entity in RainMaker is node (identified by the node id). All access control is governed by the node id and user id. Even if there are multiple devices under a node, the cloud still considers it as a single entity because it is agnostic to the data in node config and params. When you share a node or a group, the cloud backend modifies the access control data based on node id and that is the reason, why we don't have grouping based on devices.

one another thing....please give a option to add name to node id, it is very difficult to recognize with node id....

Since most nodes have single device and the associated device name, we did not have any editable node name as such. We can consider adding an optional "service" for this in future, which people can add for multi device nodes.

Thanks for adding Device automation feature, Please also add location automation, though which we can trigger any device though location data...eg. when we reach near our home AC will turn on....

This would indeed be a good feature for many, but currently we are refraining from tracking users' home and current location for privacy reasons. Location permission asked during provisioning is just used for the Wi-Fi/BLE calls. We will consider this feature in future though.


treewide: make shell.nix files not use top-level with statement

#208242

Description of changes
This is a very boring PR.

[x] Make all shell.nix instances that used a with at the top level not do that.

Things done

[x] Ran nix-shell in the following directories on x86_64-linux:

[x] maintainers/scripts/convert-to-import-cargo-lock (@winterqt)
[x] pkgs/tools/security/metasploit (@Mic92, @zowoq)
[x] pkgs/development/tools/misc/travis (@Mic92)
[x] nixos/maintainers/scripts/azure-new (@colemickens)
[x] nixos/lib/test-driver (@Mic92)

[x] Fits CONTRIBUTING.md.

This will work for nix-shell and nix-build due to their "autocall" semantics, but will break setups where people import shell.nix from some other expression. @philiptaron please find and ask maintainers who use this whether this change is okay.

I didn't see any imports, leading me to believe that this was used for nix-shell usage in the directory only.
I've updated the PR headmatter with my best attempt to find maintainers.

Maintainers may import this from their own environments! Those expressions produce derivations, you could have them in your local development environment.

Understood. The maintainers are on the review.


🚀 iCub Distro 2019.12.feat-01 Released

Hi @robotology/everyone :wave:
:rocket: TL;DR
We're happy to announce that our latest iCub software distro 2019.12.feat-01 is now available.
To download the source code with the correct tags, check out our up-to-date Software Version Table.
:warning: Jump to the section Prerequisites further down the page to see whether your robot is eligible for this upgrade.
:book: SW Distros Overview
We have changed our flow to deliver iCub software bundles that comprise several repositories with matching tags (e.g. yarp, icub-main, icub-basic-demos...).
:calendar: Time-based Distros
Starting today, these bundles will follow a time-based release cycle. As a result, bundles will be termed as YYYY.MM Distro to indicate the year and the month of the release.
Each year, we aim to yield 4 official distros, which will be synchronized with the outcome of our internal development stages. Thereby, we will have YYYY.02, YYYY.05, YYYY.08, YYYY.11 distros. Patches will be coded as YYYY.MM.nn.
These official distros will be shipped along with binaries.
:sparkles: Feature-based Distros
Occasionally, we may distribute software containing special features that we have to release out of the standard cycle. These events will most likely occur in conjunction with the release of a new iCub robot.
The feature-based distros will be termed YYYY.MM.feat-nn and, generally, will not be accompanied by binaries.
:point_right: YARP
Within this release flow, YARP is handled as a dependency for all the iCub repositories in the bundle since we clearly specify the YARP tag/branch each distro will rely on. Thus, YARP will continue to follow its own release cycle.
:sparkles: 2019.12.feat-01 Distro
Upon releasing the new iCub S/N 040 − iCubGenova07 robot, we have made available the 2019.12.feat-01 distro containing the handling of the new Robot Face Expression (RFE) unit, whose relevant activities have been tracked in the following issues/prs:

https://github.com/robotology/icub-main/issues/570
https://github.com/robotology/icub-main/pull/592
https://github.com/robotology/icub-main/issues/595
https://github.com/robotology/icub-main/pull/622

:warning: Prerequisites
Importantly, in order to build and run the distro correctly, pay attention to these notes:

Make sure that CMake 3.12 is available on your system as it is required by the latest YARP releases. You can find the instructions to install CMake 3.12 on Ubuntu 18.04 at https://github.com/robotology/QA/issues/364.

Given that robots-configuration contains changes in the XML files related to the motion control of ETH robots, we have updated the version of the code responsible for managing this as well as the firmware of the embedded boards. In particular, those robots with  GENERAL::MotioncontrolVersion equal to 5 have been now upgraded to version 6. If you are a user of the following robots, please proceed with a complete FW update:

:robot: ETH robot
:label: Motion Control Version
Eligibility for Upgrade

iCubErzelli01
6
✔

iCubErzelli02
6
✔

iCubErzelli03
6
✔

iCubGenova02
6
✔

iCubGenova04
6
✔

iCubGenova07
6
✔

iCubMoscow01
6
✔

iCubShenzhen01
6
✔

iTeenGenova01
6
✔

naoPrague01
-
✔

iiwaBoston01
-
✔

Instead, the ETH robots listed in the table below are still configured with a quite old Motion Control Version, hence they cannot run with 2019.12.feat-01 distro. It will be certainly our responsibility to align these robots to the latest versions in order to make them eligible for the next upgrade, which is expected to come soon. Stay tuned!

:robot: ETH robot
:label: Motion Control Version
Eligibility for Upgrade

iCubDarmstadt01
4
❌

iCubEdinburgh01
4
❌

iCubHeidelberg01
4
❌

iCubLausanne02
4
❌

iCubNancy01
4
❌

iCubNottingham01
4
❌

iCubSheffield01
4
❌

iCubSingapore01
4
❌

iCubTwente01
4
❌

iCubHamburg01
3
❌

CAN robots are not involved in such FW change and thus their ecosystem can be upgraded smoothly :+1:

@robotology/iit-dic

Thanks to @dcam0050 (see https://github.com/robotology/icub-main/issues/631), we have fixed a regression bug related to the compilation of a deprecated device (very old, actually).
This led us to yield the new patch 2019.12.feat-02 pointing to icub-main/v1.14.1 as per in our Versioning Table.

Let us know when an update for iCubNancy01 is available!


Ability to use Esc key to close dialogues like GE and Bank

Describe the solution you'd like
Ability to use the Esc keypress to close dialogues like the bank and the GE.
Additional context
I would implement this over the weekend, but I'm not familiar with creating scripts for invoking a close request.

that exists in the game already

It does? Doesn't seem to be working for me, do I need to configure it somewhere?


Create 2T7mOAPkCSmbOnJBMdOJ88

I might have missunderstood how this archive works but... is it possible to get old versions of this playlist or it will only save the current one for future changes?


More explicit directory structure instructions

More documentation about directory structure so I can point at it.

merged.


Specify and implement/choose a language parser for substreams Block Filtering

Substreams Block Filtering is a feature that will allow a given module of type "blockIndex" to assign arbitrary labels to certain blocks, then use the presence or absence of these labels to filter out blocks that are not of interest in another module (for performance)
example: a block has the following labels: evtsig:0x2345, evtsig:0x9876, calladdr:0xdeadbeef and calladdr:0x10101010
We need a language that has just enough expressivity to request:

Only blocks that match evtsig:0x2345 OR evtsig:0x9876
Only blocks that match evtsig:0x2345 AND calladdr:0xdeadbeef
Only blocks that match (evtsig:0x2345 OR evtsig:0x9876) AND (calladdr:0xdeadbeef OR calladdr:0x10101010 OR ...)

We only need to express "a single array of arrays where at least one of the item of each inner array has to match a given block, for every item of the outer array"
Here's another example: (addr:dead || addr:beef) && (method:transfer || method:transferFrom)
The 'labels' could be restricted to only contain the characters matching [a-zA-Z0-9:_-] and the period (.).

[ ] See if there is already a good notation to express that (we don't want a language that is too permissive...)
[ ] choose or specify the language
[ ] implement a parser that will turn the request into that "array of arrays"

Going to port and adapted the parser/AST we had for dfuse Search Query Language.

Cool. The plan is to pass it to firehose as a parameter for sf.firehose.v2.Stream.Blocks?

This will be implement by Substreams directly, Firehose is not aware of this. We have no plan yet that Firehose be able to leverage indexes produced by a Substreams.
The current plan (of course up to changes) is to introduce a third type of Substreams module called index. Those would run through the block and decide what keys map to this block.
Later stage would use those "index" and this new "query expression" and only block matching the query expression would be sent to the mapper, others would be skipper.
So there would be still a need to fully process the full chain on the index module at least one. On ship, we are going to provide a few indexes  "pre-populated" that people are going to leverage to filter a specific contract for example maybe.


Problem with IPC channel on windows

Hi,
it seems that there are problems to spawn a process with a IPC channel.
The current release doesnt work on windows.
error
? fuge> [service2 - 1696]: child_process.js:61
  p.open(fd);
    ^

Error: EBADF: bad file descriptor, uv_pipe_open
    at Error (native)
    at Object.exports._forkChild (child_process.js:61:5)
    at Function.startup.processChannel (node.js:844:10)
    at startup (node.js:45:15)
    at node.js:968:3
[api - 18392]: child_process.js:61
  p.open(fd);

When I remove the parameter it works.
https://github.com/apparatus/fuge-runner/blob/master/lib/processRunner.js#L103
https://github.com/apparatus/fuge/issues/79

@mcollina @lucamaraschi Any ideas why we need that option set? I can add a windows check around it or remove completely.

Landing soon,
I've added a handle for Windows to fix this issues.

Fixed via #18


Support custom queueEndpoint in Azure Storage Queues

Description
Adds support for a custom endpoint URL for Azure Storage Queues. This is useful for using production-style URLs with Azurite for local development.
Issue reference

#581

Please reference the issue this PR will close: #[issue number]

#581

Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#2424

@rzyns please run:
gofmt -s -w storagequeues.go
gofmt -s -w storagequeues_test.go

And push the changes.

@rzyns please allow maintainers to update your branch. I would merge your PR but I cannot rebase it on the latest commits from master.

Sorry, @berndverst, but it looks like I can't "Allow edits from maintainers" because my fork belongs to an organization (see this Github feedback discussion), so I rebased this on top of master myself. It seems the only other options would be to either a) invite maintainers as collaborators on the fork; or b) create a new PR from my user account 😫
Let me know if you need me to rebase again or want me to go with one of the other options


Support Suggestion: Forestry/Gendustry Apiary

That is, to adjust which species of bee drops what items at what % chance.
There's also Magic Bees and Extra bees if that's not too much of a stretch. NEI can display it all as %'s so I assume the capacity exists.
Thanks, of course. Sorry to keep making suggestions, especially some bad ones. I'm just getting carried away using this mod for everything.
I did try to see if it could be done with or without your mod this time. :P

You mean the chances that bee queen produce specific items? Then is should be only forestry related.
This would be very interessting for pack makers.

Yes, that's what I mean. I'm making a pack, and it started with wanting to remove the honey drops and put in regular dye, but it would be pretty cool to be able to modify the chances of everything. The kind of princesses you get would be fun to modify too, if that is treated differently from the other outputs.

I won't work on this (at least at the moment).
To complex.