Expose underlying implementations

In a break from how we did things in Vert.x 2 I think we should expose a handle to the underlying implementation in various places in Vert.x e.g:

The Netty Event Loop Group
The Jackson ObjectMapper
The Hazelcast instance
Dropwizard metrics instance
*etc

+1
It seems to have been a pretty common pattern in Vert.x 2 (no thanks to me for recommending this) for people to get the Hazelcast instance via Hazelcast.getAllHazelcastInstances().iterator().next() which is not ideal as it assumes there's only one Hazelcast instance.

On Mar 24, 2015, at 1:42 AM, Tim Fox<EMAIL_ADDRESS>wrote:
In a break from how we did things in Vert.x 2 I think we should expose a handle to the underlying implementation in various places in Vert.x e.g:
The Netty Event Loop Group
The Jackson ObjectMapper
The Hazelcast instance
Dropwizard metrics instance *etc
—
Reply to this email directly or view it on GitHub.

+1 too, better having something we support than having ppl hacking stuff


Beta

Codecov Report

Merging #61 (4e11503) into main (f0caa4f) will increase coverage by 0.23%.
The diff coverage is 90.14%.

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
+ Coverage   94.68%   94.91%   +0.23%
==========================================
  Files         129      130       +1
  Lines        2615     2675      +60
  Branches      206      212       +6
==========================================
+ Hits         2476     2539      +63
+ Misses        129      118      -11
- Partials       10       18       +8

Impacted Files
Coverage Δ

expect/_utils.ts
78.80% <73.07%> (+18.17%)
:arrow_up:

deps.ts
100.00% <100.00%> (ø)

expect/expect.ts
95.00% <100.00%> (-2.66%)
:arrow_down:

mod.ts
100.00% <100.00%> (ø)

modifier/boolean.ts
100.00% <100.00%> (ø)

modifier/number.ts
100.00% <100.00%> (ø)

modifier/rejects.ts
100.00% <100.00%> (ø)

modifier/resolves.ts
100.00% <100.00%> (ø)

modifier/string.ts
100.00% <100.00%> (ø)

modifier/trim.ts
100.00% <100.00%> (ø)

... and 8 more

Continue to review full report at Codecov.

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

:tada: This PR is included in version 1.0.0-beta.77 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Notebook examples in documentation

Users shouldn't have to visit the Github repo to read notebooks.
Including them in the docs makes it possible to navigate to a specific section of a notebook, e.g. http://domain.com/examples/Hydrology_Vistula_Catchment/#statistics-aggregated-by-month

TODO:

[ ] Include relevant notebooks
[ ] Rendering of xarray repr_html not ideal
[ ] Consider to execute notebooks as part of the docs rendering

P.S. I didn't manage to reference the notebooks in the /notebooks folder, thus I added them as symbolic links to /docs/examples

It's nice except those symbolic links are not working on windows which means that the documentation can no longer be build on windows🤔

Too bad, so how about copying the most relevant examples to the docs and keeping the notebooks folder as a repository of less polished examples?

Yeah, or do some manual work (copy notebooks manually) when I once a while build the documentation locally? Let's decide next year 🎇

Notebooks will benefit from #378


onlyoffice-documentserver: 7.5.1 -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ONLYOFFICE/DocumentServer/releases.
meta.description for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
meta.homepage for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.0.0 with grep in /nix/store/rrlwxgamp2il4337xjrnwjjn8i7kavyx-onlyoffice-documentserver-8.0.0
found 8.0.0 in filename of file in /nix/store/rrlwxgamp2il4337xjrnwjjn8i7kavyx-onlyoffice-documentserver-8.0.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
onlyoffice-documentserver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rrlwxgamp2il4337xjrnwjjn8i7kavyx-onlyoffice-documentserver-8.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A onlyoffice-documentserver https://github.com/r-ryantm/nixpkgs/archive/9f87bfde8bcb317998c83dd3b65ecbfeb627f944.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9f87bfde8bcb317998c83dd3b65ecbfeb627f944#onlyoffice-documentserver

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rrlwxgamp2il4337xjrnwjjn8i7kavyx-onlyoffice-documentserver-8.0.0
ls -la /nix/store/rrlwxgamp2il4337xjrnwjjn8i7kavyx-onlyoffice-documentserver-8.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

onlyoffice-documentserver

Maintainer pings
cc @SuperSandro2000 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ONLYOFFICE/DocumentServer/releases.
meta.description for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
meta.homepage for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.0.1 with grep in /nix/store/fyalyw9mh6cqkw2y14sz09j25051dpiw-onlyoffice-documentserver-8.0.1
found 8.0.1 in filename of file in /nix/store/fyalyw9mh6cqkw2y14sz09j25051dpiw-onlyoffice-documentserver-8.0.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
onlyoffice-documentserver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fyalyw9mh6cqkw2y14sz09j25051dpiw-onlyoffice-documentserver-8.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A onlyoffice-documentserver https://github.com/r-ryantm/nixpkgs/archive/1815a11e01ef5a2bcb99ce6c031e9b30d9af3911.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1815a11e01ef5a2bcb99ce6c031e9b30d9af3911#onlyoffice-documentserver

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fyalyw9mh6cqkw2y14sz09j25051dpiw-onlyoffice-documentserver-8.0.1
ls -la /nix/store/fyalyw9mh6cqkw2y14sz09j25051dpiw-onlyoffice-documentserver-8.0.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

onlyoffice-documentserver

Maintainer pings
cc @SuperSandro2000 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/onlyoffice-documentserver/versions.
meta.description for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
meta.homepage for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/b03k1assq96lv3yh7aq0g7gf4wgq3n53-onlyoffice-documentserver-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/b03k1assq96lv3yh7aq0g7gf4wgq3n53-onlyoffice-documentserver-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
onlyoffice-documentserver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/b03k1assq96lv3yh7aq0g7gf4wgq3n53-onlyoffice-documentserver-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A onlyoffice-documentserver https://github.com/r-ryantm/nixpkgs/archive/af8872bc72acfb90917e3531367a7fbd67a1cd6e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/af8872bc72acfb90917e3531367a7fbd67a1cd6e#onlyoffice-documentserver

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/b03k1assq96lv3yh7aq0g7gf4wgq3n53-onlyoffice-documentserver-<IP_ADDRESS>
ls -la /nix/store/b03k1assq96lv3yh7aq0g7gf4wgq3n53-onlyoffice-documentserver-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

onlyoffice-documentserver

Maintainer pings
cc @SuperSandro2000 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ONLYOFFICE/DocumentServer/releases.
meta.description for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
meta.homepage for onlyoffice-documentserver is: https://github.com/ONLYOFFICE/DocumentServer
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.1.0 with grep in /nix/store/584bbrl4lnhs8mw25sllb1q1wv3rc0p3-onlyoffice-documentserver-8.1.0
found 8.1.0 in filename of file in /nix/store/584bbrl4lnhs8mw25sllb1q1wv3rc0p3-onlyoffice-documentserver-8.1.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

onlyoffice-documentserver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/584bbrl4lnhs8mw25sllb1q1wv3rc0p3-onlyoffice-documentserver-8.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A onlyoffice-documentserver https://github.com/r-ryantm/nixpkgs/archive/d97954a2c6a54d85e80e835eeb7ea5650c034a05.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d97954a2c6a54d85e80e835eeb7ea5650c034a05#onlyoffice-documentserver

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/584bbrl4lnhs8mw25sllb1q1wv3rc0p3-onlyoffice-documentserver-8.1.0
ls -la /nix/store/584bbrl4lnhs8mw25sllb1q1wv3rc0p3-onlyoffice-documentserver-8.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

onlyoffice-documentserver

Maintainer pings
cc @SuperSandro2000 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/onlyoffice-documentserver/versions.
meta.description for onlyoffice-documentserver is: ONLYOFFICE Document Server is an online office suite comprising viewers and editors
meta.homepage for onlyoffice-documentserver is: https://github.com/ONLYOFFICE/DocumentServer
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.1.1 with grep in /nix/store/z75x3hi2aiqlyiddsnpc4hqngy6hhlng-onlyoffice-documentserver-8.1.1
found 8.1.1 in filename of file in /nix/store/z75x3hi2aiqlyiddsnpc4hqngy6hhlng-onlyoffice-documentserver-8.1.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

onlyoffice-documentserver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/z75x3hi2aiqlyiddsnpc4hqngy6hhlng-onlyoffice-documentserver-8.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A onlyoffice-documentserver https://github.com/r-ryantm/nixpkgs/archive/41b3cc4addc489a5633cc99c042ea4749057d8b4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/41b3cc4addc489a5633cc99c042ea4749057d8b4#onlyoffice-documentserver

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z75x3hi2aiqlyiddsnpc4hqngy6hhlng-onlyoffice-documentserver-8.1.1
ls -la /nix/store/z75x3hi2aiqlyiddsnpc4hqngy6hhlng-onlyoffice-documentserver-8.1.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

onlyoffice-documentserver

Maintainer pings
cc @SuperSandro2000 for testing.


Produce a browser compatible version for releases

I don't think there is any reason to limit this module to node.js

I think this makes sense.  Looks like Class is well supported in modern browsers now.
It would be interesting to see the benchmarks on a mobile device

Initial pass on this branch https://github.com/bucharest-gold/fidelity/tree/browserify
There's something going on with setTimeout that is causing a number of tests to fail within the browser.


How to change the port used?

It seems like 4568 is hardcoded as the port.
Is there a way to set this?  A runtime flag perhaps?

You can change it with the -p option


Refine migration support for durable objects

@a-robinson particularly interested in your review of the migration coalescing algorithm
@ags799 @caass tagged y'all for the more wrangler-specific pieces of this PR.

I'd like to understand what this changes in terms of wrangler usage. This basically adds a [migrations] tag to wrangler.toml? and then you update that every time you migrate?

yup! it's a top-level array of migrations, every time you add/rename/transfer classes, you'd add a new entry to the [[migrations]] array

sorry, laptop keyboard went crazy and issued some clicks.. didn't mean to add the blocked label 😨

fixed some logic bugs with the input to migration coalescing I discovered after testing:

duplicate migration tags are properly handled
removed special case for a single migration that incorrectly attempted to push it up with no tags
the old migration in wrangler.toml was incorrectly being included as part of coalescing. we now start at the next migration

since migration coalescing / migration in toml seems to be somewhat issue-prone as currently implemented, I'll break out a separate PR for just rename/transfer migrations

Added the aliases for durable objects classes section. in next RC we will make this a breaking change and remove support for the old name
before:
[durable_objects]
classes = [{ binding = "Counter", class_name = "Counter" }]

[durable_objects]
# binding = "Counter" would be accepted too, but it feels weird to duplicate "binding" in this manner. We should use name instead IMO, but I don't feel strongly if we want to keep using "binding"
bindings = [{ name = "Counter", class_name = "Counter" }]


fix(config): correct mcohome mh s510 series device files

Additional parameters added to configuration files.

@blhoward2 I'm going around in circles working out how to incorporate the additional parameters - I'd committed the changes against another PR (I think), but I think that has now caused the potential conflicts you mention
Might just be easiest for me to wait until this PR is merged, and then use that as a basis to do a separate PR for the additional parameters.

That means we'd have to review this same file over and over. Can you try downloading VS Code and the GitHub plug-in?

Ok - Will do - I originally used vscode to make the changes, but used the web UI to commit.        Have been playing with the github desktop app and not really sure how it all works..

Sorry @blhoward2 - I'm not having much luck, and have to crash.    I really want to help out to save work for others, but struggling with the github side of things..       I probably need a solid bunch of hours to work out the basics, not sure when I'll get that chance!

I really want to help out to save work for others, but struggling with the github side of things..

Easiest way:

Fork the repo on Github
Open VSCode
Clone your fork and open the folder
Make a new branch
Do all necessary edits in that branch, push it to your fork (can use the VSCode UI for all of that)
Go to Github, make a PR from your branch to this repo.

Thanks AlCalzone - I have done this, and committed a bunch of changes - but this has resulted in another PR (#2265)       Couldn't work out how to send to this PR.

I have incorporated @blhoward2's changes into #2266 along with further changes, so I think easiest if I close this one.


Run locally

Document in README how to run locally.

Already documented.
See https://github.com/it-pebune/ani-research-web/blob/main/README.md


Modernize code to Python 3.6+ and some cleanup

Hi Luciano, I have some random cleanups for your repository. Sorry for the single commit. Please ask for any details/reasons in my PR.
Generally, most updates can be grouped in following categories:

f-strings everywhere
some pep8-ish reformatting
fix some typos
math.floor returns int in Python 3, no need to convert it
removal of unused imports

Looking forward to your feedback!

Thank you so much, @eumiro !

In a few places where I used math.floor, I had an expression like math.floor(math.sqrt(n)).
It turns out Python 3.8 added a math.isqrt function that does what I needed, so that's even more modern!
I would not have found it if you didn't make me look at that code again. Thanks!

That's a very good point. It depends on where you officially “pin” your book. When I write a new project where I control the whole environment, then I'd go with Python 3.9 (and use that math.isqrt happily). On the other hand, if I write a library and want to offer modern code to my users, I'd go currently (February 2021) with Python 3.6–3.9 and therefore without math.isqrt.
That's exactly the reason why this PR aimed for Python 3.6+. Next year I'd aim for 3.7+ and start using dataclasses.

For the book, a aim for the latest official release. That was also my policy in the first edition. The upsides are showing the readers new features they may not know about, and making the book as fresh as possible, given that updates are difficult in the traditional editorial process of book publishing. The downside is minimal: the programs are all very short, so fixing for incompatibilities of that kind is easy. The #1 reason I see for readers to use 3.7 at the moment is to run the examples with Pypy.
BTW, as soon as Mypy 0.800 started accepting generic built-ins (like list[int]) I started removing all redundant imports like from typing import List, Tuple, Dict etc. I added a not about it in the first occurrence (in Chapter 5) and that's it. I think that's a good usability improvement for typing.


Cannot rehydrate Blob versions from "Archive" tier

Storage Explorer Version: 1.15.0
Platform/OS: Windows 10/macOS/Linux
Architecture: x64
Regression From:  Not a regression
Bug Description
Cannot change access tier of a Blob version from "Archive" to "Hot" or "Cool".
Steps to Reproduce

Set access tier of a Blob version to "Archive"
Try to set its access tier to "Cool"

Expected Experience
Succeed in the operation.
Actual Experience
Fail with service returning error: BlobOperationNotSupported.

Service bug. Jason emailed service team in early August, still hasn't heard anything back. Let's keep tracking this issue and try to get to the bottom of it.

Update: Up to now, the only way to restore data from an archived blob version is through an asynchronous copy. https://github.com/microsoft/AzureStorageExplorer/issues/3439. We need to support this scenario later utilizing this approach.

Update: After investigation, even the documented async copy doesn't seem to work with archived blob version. Checked with Portal and it doesn't support bringing archive version back to cool/hot neither.

Duplicate of #1950


/utils/pub/version.dart still uses obsolete default factory class syntax

Pub needs to be updated so we can proceed with removing support for obsolete default factory class syntax:
interface VersionConstraint default _VersionConstraintFactory { ... }
Also, the name of a factory must match the name of its enclosing class:
class _VersionConstraintFactory {
  factory VersionConstraint.empty() => const _EmptyVersion();
Please, use abstract classes and redirecting factories instead.

This issue has been moved to dart-lang/pub#310.


feat: add styleExtensions

In your css array, you have to write the corresponding extension when providing your stylesheets. While transitioning from SCSS to PostCSS in several projects, I thought: "Why not completely omitting the extension as we do it for JS resolves :thinking:?"
There we go!
Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[x] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Codecov Report

Merging #4671 into dev will increase coverage by 0.02%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##              dev    #4671      +/-   ##
==========================================
+ Coverage   90.78%   90.81%   +0.02%
==========================================
  Files          67       67
  Lines        2236     2231       -5
  Branches      545      547       +2
==========================================
- Hits         2030     2026       -4
+ Misses        187      186       -1
  Partials       19       19

Impacted Files
Coverage Δ

packages/config/src/config/_common.js
100% <ø> (ø)
:arrow_up:

packages/core/src/resolver.js
86.79% <100%> (+0.25%)
:arrow_up:

packages/config/src/options.js
93.7% <85.71%> (-2.51%)
:arrow_down:

packages/vue-app/src/index.js
0% <0%> (ø)
:arrow_up:

packages/builder/src/builder.js
96.7% <0%> (+3.17%)
:arrow_up:

Continue to review full report at Codecov.

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


Percy reporting changes when nothing changed

Nothing new here really, ticket for tracking purposes.
Seems that snapshots are sometimes taken at a marginally different size or speed and reporting 'changes' when nothing actually changed.
For example, I had to approve percy changes in a recent documentation update I made here: https://percy.io/google/web-stories-wp/builds/8765894/changed/497717303?browser=chrome&browser_ids=14%2C16&subcategories=approved&viewLayout=side-by-side&viewMode=new&width=1600&widths=1600  where the only diff was a markdown file.

Thanks for filing!
At first glance it seems like the Roboto font with weight 700 wasn't fully loaded in one of the screenshots.
While we check for font loading here:
https://github.com/google/web-stories-wp/blob/1ea476bb315d2618cfdd95f6ffd9775593064a69/assets/src/edit-story/karma/fixture/fixture.js#L299-L306
We actually only load Roboto 400 here:
https://github.com/google/web-stories-wp/blob/1ea476bb315d2618cfdd95f6ffd9775593064a69/assets/src/karma-fixture/client_with_context.html#L14-L21
We need to make sure the 700 weight (and potentially others if not all weights) are loaded too.
That should hopefully reduce flakiness here.
Same can be done for Google Sans in the dashboard too.
@spacedmonkey is this something you could look into?


Add systemd logger

We disconnect the stdout/err streams from conmon at a certain point of
execution. Therefore it would make more sense to log to a dedicated
driver instead of those streams. We now add a --log-driver option as
well as a default systemd logger.

Codecov Report

Merging #88 (30d0c81) into main (a19f44e) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #88   +/-   ##
=====================================
  Coverage   6.89%   6.89%
=====================================
  Files          2       2
  Lines         29      29
  Branches      17      17
=====================================
  Hits           2       2
  Misses        27      27

hm where did this error come from
time="2021-12-07T11:36:05Z" level=warning msg="unable to get oom kill count" error="no directory specified for memory.oom_control"

Rebased to make the tests green.

One more nit, can we have a None driver that prints to stdout and use it in most of the tests? it's useful to see the output IMO

that can be in a follow up if you'd like too

Sure, added a stdout logger as well :+1:

LGTM


Added OpenFHE library and website

By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆
PALISADE community has merged the PALISADE project into OpenFHE, so I hope this additions is useful.
In my opinion it's worth putting the website of the project and the GitHub link separated, but in this single pull request, since they're related.

I totally agree. Thanks for your first contribution! :tada:


New scroll example and reduced compile time for lib using dev-dependencies

Don't merge quite yet, I want to hear your opinion.
Also, for some reason changing the FOV (the scroll.rs) of the camera doesn't work even though it does change, maybe the fov is only read on initialization?

Looks good to me, thanks for contributing!


test: refactor CI code

Just a simple refactoring to keep the actions simple and descriptive.
The changes will already be running in this very pull request.

Yurii's last commit integrated these changes 😄

@mayconamaroCW
Thank you Macon for your suggestions!
They are nice 😄
We have distributed them to most of  our repositories


Unknown at rule: @value

Hello,
I got error when using @value annotation:
✘ [ERROR] [plugin esbuild-css-modules-plugin] Unknown at rule: @value

example using:
@value colors: "./colors.css";
@value dark from colors;

.button {
  background-color: dark;
}

@value may be  currently not supported by https://github.com/parcel-bundler/parcel-css which is used to transform css if v2 set to true

ok I see, here would be the issue to follow then https://github.com/parcel-bundler/parcel-css/issues/6

solution for v1 #36


4256 communities frontend pr2

Rough prototype of includable objects for UI-based community management.  This includes access control objects for approving community requests from groups, approving group invitations from communities, etc.   It also contains management commands for creating synthetic communities, and one update to hsctl that makes it more convenient to develop a populated instance of HydroShare.
I commented out non-working code for the group.html template. This might be needed for completing the work.
The community template contains the management tab for communities, but no user is currently authorized to use it.

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case
This does not change HydroShare function, other than providing an admin tab for community owners other than those of the CZOs.

@Maurier requested that we merge this into Develop.
I have sanitized it so it won't affect any users, pending his changes to the UI for communities.
@sblack-usu Is it ready for merging?

Why does this need to be merged to work on it?  I think the branch just needs to be caught up to the latest develop.

Changed the target branch since the rest of the work will be made there. I will  help review and will address these merge conflicts.


New package: Ether.Etherpad-Lite version 1.9.6

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

/AzurePipelines run

Hi @Exorcism0666,
This package appears to be a set of JavaScript files, which are executed locally through the Node app. Is this the case?

@stephengillie Yes!


Global queue pause

@TomKaltz. Yes, I tried to come up with a idea for global pause a couple of months ago. Only recently I thought of a pub/sub method I could use to give global pause.
I'll think on it more.

Why not have a table that holds queue names and metadata for that queue such as paused state.  That way all workers can listen to changefeed of the paused property of the pertaining queue row?

I've spent a few hours on this today, and it works perfectly. Published in v0.3.0.
The final approach I took was to used a fixed document id in the database and just update it with global status. This causes the change feed to update the worker Queue objects.
Just call Queue.pause(true) for a global pause. Without the true it is a local pause only.
I'll update the docs when I can.
Thanks again.


Exception: Cannot set properties of null (setting 'innerHTML')

Several times a day I am experiencing a sporadic exception in the HtmlEditor component.
Unfortunately I am not able to find exact steps how to reproduce it but here are two typical callstacks:
The most often it originates from OnFocus
Wed 30.11.22 18:57:27.147077 ??? Unhandled exception in circuit 'OTSPMlpJNtv12ydYnFDK8WUTJQCY7NKSRIOI7-49ipQ'.
Exception:
Microsoft.JSInterop.JSException: Cannot read properties of null (reading 'innerHTML')
TypeError: Cannot read properties of null (reading 'innerHTML')
    at Object.queryCommands (https://xxx.azurewebsites.net/_content/Radzen.Blazor/Radzen.Blazor.js:1233:17)
    at https://xxx.azurewebsites.net/_framework/blazor.server.js:1:3506
    at new Promise (<anonymous>)
    at Ft.beginInvokeJSFromDotNet (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:3480)
    at Ft._invokeClientMethod (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:75072)
    at Ft._processIncomingData (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:72696)
    at Ft.connection.onreceive (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:67009)
    at i.onmessage (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:51322)
   at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
   at Radzen.Blazor.RadzenHtmlEditor.UpdateCommandState()
   at Radzen.Blazor.RadzenHtmlEditor.OnFocus()
   at Microsoft.AspNetCore.Components.ComponentBase.CallStateHasChangedOnAsyncCompletion(Task task)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)

In several cases it also originates from OnAfterRenderAsync
Wed 30.11.22 12:24:58.046369 ??? Unhandled exception in circuit '166RL_ckzfG0waAfLDrPKmiQeRSyehRY2BJUm9q1OE4'.
Exception:
Microsoft.JSInterop.JSException: Cannot set properties of null (setting 'innerHTML')
TypeError: Cannot set properties of null (setting 'innerHTML')
    at Object.innerHTML (https://xxx.azurewebsites.net/_content/Radzen.Blazor/Radzen.Blazor.js:1219:21)
    at https://xxx.azurewebsites.net/_framework/blazor.server.js:1:3506
    at new Promise (<anonymous>)
    at Ft.beginInvokeJSFromDotNet (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:3480)
    at Ft._invokeClientMethod (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:75072)
    at Ft._processIncomingData (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:72696)
    at Ft.connection.onreceive (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:67009)
    at i.onmessage (https://xxx.azurewebsites.net/_framework/blazor.server.js:1:51322)
   at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
   at Microsoft.JSInterop.JSRuntimeExtensions.InvokeVoidAsync(IJSRuntime jsRuntime, String identifier, Object[] args)
   at Radzen.Blazor.RadzenHtmlEditor.OnAfterRenderAsync(Boolean firstRender)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)

I am also receiving a feedback that users sometimes lose their data entered into the HtmlEditor. I suspect it could be caused by this error.
Please, do you have any idea what could cause this problem? Could anything be wrong on my site that I should check and fix?

We are not able to reproduce this exception. The code that seems to fail is this: https://github.com/radzenhq/radzen-blazor/blob/master/Radzen.Blazor/wwwroot/Radzen.Blazor.js#L1226 which is invoked here: https://github.com/radzenhq/radzen-blazor/blob/master/Radzen.Blazor/RadzenHtmlEditor.razor.cs#L189
It seems that ContentEditable is null for some reason. This shouldn't happen as it is a @ref of a html element: https://github.com/radzenhq/radzen-blazor/blob/master/Radzen.Blazor/RadzenHtmlEditor.razor#L52

Hello, same issue here, same error message as in Hardik's screenshot. Unfortunately i can't really trigger the issue on my local dev machine, but in the hosted app running on azure app services it happens more or less consistently. I'll try to create a minimal repro sample.

Here is a minimal reproduction sample: https://github.com/tecxx/BlazorRTESample
run it, F12 for debugger, randomly click the tree nodes on the left in rapid fashion. at some point the app will crash like this:

sometimes it takes ages to make it crash, sometimes it crashes at second or third click. very strange....

tried to find a solution for this today, what i found is

the exception is triggered here:

https://github.com/radzenhq/radzen-blazor/blob/master/Radzen.Blazor/wwwroot/Radzen.Blazor.js#L1231
ref is null, so accessing ref.innerHTML fails.

ContentEditable in https://github.com/radzenhq/radzen-blazor/blob/master/Radzen.Blazor/RadzenHtmlEditor.razor.cs#L189 is never null, it is always a correct ElementReference. the issue must stem from somewhere else

i can only reproduce the issue when having a HTMLEditor inside a component, that is loaded as part of another component. in the screenshot below, the top right editor is part of the main component, which contains a RadzenDataList to display multiple "tasks" components, which contain the editor instances where the crash is happening.

so this looks like this is some kind of race condition caused by delayed loading or something along those lines (?). the html component initializes with a short delay, while in the meantime rapid clicking already caused a switch to a new instance of the component. a still running javascript call now tries to access ref.innerHTML of an already disposed component.
... just guess, though.

null check workaround

window.Radzen.innerHTML = function (ref, value) {
        if (value != undefined) {
            if (ref == null) {
                test = "";  // unknown bug https://github.com/radzenhq/radzen-blazor/issues/728
            } else {
                ref.innerHTML = value;
            }
        } else {
            return ref.innerHTML;
        }
    }

adding a check for ref==null gets rid of the exception. it also seems that (with a very short delay) the html content is displayed correctly. i tried it with an azure hosted webapp, where, due to all the delays, i could reproduce the crash very consistently. it has fully disappeared, and i can not see any data loading issues as well.
no guarantees though....

@tecxx feel free to open a pull request with the null check.

I am closing this issue as the suggested fix has been merged and released.


Implement break-down mode for cross entropy calculation

This PR prevents the training process from falling down to the CPU mode, possibly happens in following case.

large number of objects per image (detected and ground truth)
large number of classes

When calculating cross entropy, the process requires a tensor with following size
nr_of_detected_objects * nr_of_ground_truth * nr_of_classes
which may explode in the above case and cause GPU OOM.
This PR implements break-down mode of cross entropy calculation, which requires only
nr_of_detected_objects * nr_of_classes
of tensor. This mode will be tried when the original code caused OOM, and still runs far faster than the CPU mode.
For testing, you can pull the following branch
https://github.com/penkoba/YOLOX/tree/break_down_cross_entropy_calc-test
and run test-train.sh, then you may see entering the break-down mode but still runs at acceptable speed.
Without this PR, the traning process falls into CPU mode very often and gets very slow.
(tested with RTX 2080 Ti)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Toshihiro Kobayashi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


New estimators and examples for OED

This PR introduces updates for OED:

new estimators: naive Rainforth (NMC) and Donsker-Varadhan
new examples: Bayesian linear models extended to include unknown variance, Bernoulli responses and mixed effects
new test infrastructure: benchmarking code to: time and plot estimates from various estimators on various examples, convergence plots for DV estimator

Codecov Report

Merging #1309 into dev will decrease coverage by 0.88%.
The diff coverage is 20.59%.

@@            Coverage Diff             @@
##              dev    #1309      +/-   ##
==========================================
- Coverage   96.06%   95.17%   -0.88%
==========================================
  Files         114      114
  Lines        5757     5822      +65
==========================================
+ Hits         5530     5541      +11
- Misses        227      281      +54

Impacted Files
Coverage Δ

pyro/contrib/autoguide/__init__.py
99.29% <ø> (ø)
:arrow_up:

pyro/contrib/oed/eig.py
34.15% <20.59%> (-65.85%)
:arrow_down:

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 eb05530...df8c1ba. Read the comment docs.


Nesting fails when using same wrapper

This plugin is great, thanks for working on it. As the docs state, nesting really does works like a dream when using unique wrappers:
(outer)(inner)(/inner)(/outer)

However, nesting with the same wrapper doesn’t work so well. The output generates this funny little sandwich:
kirbytext
(wrapper)
  (wrapper)(/wrapper)
(/wrapper)

expected output

current output

(wrapper)

(/wrapper)

Not sure of the best way around this... :(

oo think that'll require some regex wrangling. Not gonna have the time to take a look in the immediate, but the splitting happens here if ya want to take a gander:
https://github.com/jongacnik/kirby-wrappers/blob/86f707dc323ec06d3a938944d73a196fe618da46/src/Wrappers.php#L16

Nice, thanks. I'll stare at this for a bit and see if I can think of any fancy RegEx... fingers crossed.


Regression: errors out if no indy

We've inadvertently made indy a requirement again with recent changes. Using the following dockerfile:
FROM bcgovimages/von-image:py36-1.16-0
RUN pip install git+https://github.com/hyperledger/aries-cloudagent-python@main
RUN yes | pip uninstall python3-indy

ENTRYPOINT ["/bin/bash", "-c", "aca-py \"$@\"", "--"]

CMD ["start", "-it", "http", "<IP_ADDRESS>", "3000", "-ot", "http", "-e", "http://localhost:3000", "--admin", "<IP_ADDRESS>", "3001", "--admin-insecure-mode", "--no-ledger"]

The following error occurs:
Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/__main__.py", line 74, in <module>
    main(sys.argv)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/__main__.py", line 71, in main
    run(args)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/__main__.py", line 64, in run
    run_command(command, args)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/commands/__init__.py", line 36, in run_command
    module = load_command(command) or load_command("help")
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/commands/__init__.py", line 31, in load_command
    return import_module(module_path)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/commands/start.py", line 16, in <module>
    from ..core.conductor import Conductor
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/core/conductor.py", line 35, in <module>
    from ..protocols.out_of_band.v1_0.manager import OutOfBandManager
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/protocols/out_of_band/v1_0/manager.py", line 16, in <module>
    from ....indy.models.xform import indy_proof_req_preview2indy_requested_creds
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/indy/models/xform.py", line 5, in <module>
    from .pres_preview import IndyPresPreview
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/indy/models/pres_preview.py", line 9, in <module>
    from ...ledger.indy import IndySdkLedger
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/ledger/indy.py", line 14, in <module>
    import indy.ledger
ModuleNotFoundError: No module named 'indy'

I believe this might have been introduced with #1125 @shaangill025

@Luis-GA related to the issue we were seeing from acapy-resolver-universal

Maybe good to add a Github Action script that builds and start ACA-Py using the above docker script?

@TimoGlastra agreed; we should be able to catch this on PRs without too much difficulty. Or else, making the unit test runner for non-indy tests not actually have python3-indy available in the python environment.

@dbluhm Any ideas when was the last time this was working? I used the above docker script to test with edfd5b52abd3f2fb4dbf30351cde35560bbb1242 [from May] and b926cac23ed783adbd862ca49d760446acb005fb [from January] commits and get the same error. I believe this might have been an issue since the OOB protocol was implemented earlier this year. I am working on a fix.


Update sbt to 1.3.10

Updates org.scala-sbt:sbt from 1.3.8 to 1.3.10.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

Codecov Report

Merging #93 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #93   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines         229     229
  Branches        7       7
======================================
  Misses        229     229

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 087542a...60a75d2. Read the comment docs.


Add a few more test cases to currency conversion

Larger number of subunit difference is also a real life use case.

Is this ready to merge?

I think so, I was trying to provide more tests to reproduce #422, but turned out this is not the cause. Still, large subunit difference tests are nice to have.


[BREAKING_CHANGES] converted ResultCategoryScores struct to map

Describe the change
I converted the ResultCategoriesScores struct into a map[string]float32.
This change has been discussed in issue #633 .
Provide OpenAI documentation link
https://platform.openai.com/docs/guides/moderation/overview
Describe your solution
I just converted the struct into a map to allow more categories to be accessible without the need of changing the code everytime
Tests
I simply adapted the existing tests to the new data structure with more test cases
Issue: #633

Thank you for the PR!


document.registerElement is deprecated

I was just going through custom element specs here at MDN and it says its deprecated.

Yeah, you are right.
NX uses the v0 spec (with document.registerElement), because it is supported in chrome at least (all other browsers are polyfilled with MutationObservers). I will migrate things internally to work with the v1 spec when browsers start to support it. It will be a small change without breaking anything.
Thx for the warning.

I would like to make contributions to it can you please refer to me some prior reading before getting started?

Cool (: Do you mean contribution regarding this issue or NX in general? I am writing a new part of the docs for contributors right now. In them meantime I will look for some resources for you today.

No, I want to contribute in general, meanwhile you can close this issue marking it as feature.

You can start by reading the Docs. I started writing a blog series about the difficulties while writing NX. It is not very specific, but it should help (next article is out on Tuesday).
I will think about a nice task for you before posting heavier things. Do you have any favorite topics in web dev?

Thanks, not any particular interest but I have Angularjs experience. I was thinking to write the utlity functions for repeated logic eg: 'isFunction' etc.

For simple logic I don't think we need that. util.isFunction(fn) is not a lot shorter than typeof fn === 'function'. For longer checks it could be useful though. Also improving the error messages is something I plan to do, but it's not a very exciting task.
As a longer project you could make and maintain a new middleware, which NX is still missing (like animation or form validation). If you are interested I can help you get started with that. A middleware is just a separate pluggable function, so you can start on this without much knowledge about how NX works.

its not about long or short, ideally every piece of information should be in just one place. Having said that I can start with form validation, it would be good if you can impart some knowledge to me and then create a new feature request to track down the progress easily.

Sure, but I try to avoid shared utility libraries because of the middleware system. Middlewares work best if they are focused and have few or no dependencies. After the alpha I plan to split all the middlewares into separate repos for more flexibility.
I will be on vacation for the weekend. Lets get back to this and the form validation middleware on Monday, if its okay for you.
Have a nice weekend (:

Sure thing, bon voyage :+1:

Hi!
About the form validation middleware:

you can get started by adding a validation.js file to the middlewares folder and exporting it in the index.js file of the middlewares folder, like validation: require('validation')
for more about middlewares check the first two sections of the docs
the middleware should preferably be a small function, focused only in validation that does not require the other core middlewares, but works nicely with them

About form validation in general:

I think you should start with learning about the native validation API: here and here.
Then learn about the DOM event lifecycle: http://javascript.info/tutorial/bubbling-and-capturing

About putting this together:
The validation API applies css selectors to invalid elements and prevents their parent form submission. I think you should extend this functionality, by choking every event caused by user input if the input is invalid. (with stopPropagation(), and without preventDefault()`). This way we can prevent any handler and data binding logic form triggering on invalid events.
This means it works nicely with the existing middlewares, but can be used without them too. I currently have no idea of how to choke the events, but I am investigating it.
What is your opinion, do you have any ideas, is it unclear/too though as a start?

Hi,
Thanks for the guidelines the middle ware concept is somewhat new to me but I am sure its worth reading it. Having said that I will create a new feature request for the form validation and can you please set up the gitter chat for generic discussion like these?

Hi,
Hope you are doing good, I looked up the docs and the MDN links (I may need to re-read them :)). I tried to get some inspiration from angular form validation and figured out some simple API, its in draft phase so feel free to comment.
https://gist.github.com/vivekimsit/12348b91687833d1bb02a7b78c032719

Hi!
I commented on your gist (:


Please don't modify my settings when I have enabled editor.formatOnType

I have set
"editor.formatOnType": true

so please don't add
"[python]": {
    "editor.formatOnType": true
}

in my settings.

I believe this is actually added by the python core extension. Transferring.


build-and-push: escape annotation values

Due to https://github.com/tektoncd/cli/issues/2402, annotation values that contain commas or double-quotes currently break 'tkn bundle push'.
Escape such annotation values in a way that's compatible with the pflag.StringSlice [1] parser that 'tkn bundle push' uses to parse CLI arguments.
https://pkg.go.dev/github.com/spf13/pflag#FlagSet.StringSlice
Before you complete this pull request ...
Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

/test

/test

$ skopeo inspect --raw docker://quay.io/konflux-ci/pull-request-builds@sha256:48f87850d2c9c41cdde7c67571b3054dc0f14de01337f1f940f1231502625289 | jq '.annotations["org.opencontainers.image.description"]' -r

Clones redhat-appstudio/infra-deployments repository, runs script in "SCRIPT" parameter, generates pull-request for redhat-appstudio/infra-deployments repository.

The description of the update-infra-deployments task, which I updated in a test commit to contain both double-quotes and a comma, was preserved correctly
:tada:

/test


Fix typos in Configuration docs

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description

Issue reference

@antontroshin @msfussell I need to do an upmerge from 1.14 to 1.15 before we review/merge this. it looks like this is an older version of this doc, before I did a docs refresh and fixed most of these in 1.14.


Help system broken

Help system (F1) is broken for a long time.
Has to do something with it.

If you make a HTML5 site for the help menu, you can make it interactive: you can show the open tickets from github or manage a FAQ online. Also it would be nice If we can send a report for you with every information what you need for debugging if we found something like in Eclipse Mars :)

Silly and heavyweight Eclipse help system will be replaced with GitHub Wiki.


[StableHLO] Port reduce_window to linalg lowering

This ports the reduce_window op lowering from mlir-hlo. For the details, see the initial import: https://github.com/openxla/iree/pull/12957.
The code is cleaned up to match the existing StableHLO -> linalg conversion patterns. The pattern population logic is updates so that we do not depend on the order in which patterns are added -- we use pattern benefit to specify the order instead.
This also moves the previously ported reduction patterns to the same file as reduce_window.
Issue: https://github.com/openxla/iree/issues/12678

Ping


Fix couple of PEP8 errors.

What does this implement/fix?
Couple minor PEP8 errors.

pep8 is checked by our CI so technically it should be already fine...

I see these in my local editor, too, though the automated checkers don't see them. Must not be strict enough!

@ktavabi can you rebase to make CIs green?

I restarted the CIs

Thanks @ktavabi


sparkhello: launch 'sparklyr::compile_package_jars()' download spark-1.5.2-bin-hadoop-2.6.tgz

Hello, I would try the sparkhello but when I launch
sparklyr::compile_package_jars()
it directly download spark-1.5.2-bin-hadoop-2.6.tgz instead I already have spark-2.1.0-bin-hadoop2.7 installed. Can anyone tell me where I did wrong?

@Vonisoa could you share the output of sparklyr::compile_package_jars()? Which sparklyr version which operating system? Thanks!

@Vonisoa Right, compile_package_jars does download several versions of Spark since it needs to compile against multiple versions to allow your extension to be deployed against several Spark versions.
That said, you could potentially have an extension that supports a subset of the Spark versions sparklyr supports. To do this run:
compile_spec <- spark_default_compilation_spec()

Modify compile_spec to remove the versions that you wish to not support, then rebuild using
compile_package_jars(spec = compile_spec)

Closing since this issue was more of a question than a change request.


Fix failing tests in master.

The URL for the smallterrain data set went away a week early.  This fixes failing tests in master by using a different url that points back to the same old smallterrain set.
smallterrain is supposed to go away completely in 1.11, but I can't find an issue for it, this is just a band-aid so master can pass.
I tried removing usage of small-terrain from the tests, but GlobeSurfaceTileSpec seems heavily tied to it.  @kring or @abwood can you have a look at see what the problem is?  Chaning the url in GlobeSurfaceTileSpec causes it to try and access undefined on line 218.

@mramato we shouldn't change all the tests that use heightmap terrain to use quantized mesh terrain instead.  In addition to reducing test coverage, that would leave untested an important part of Cesium that lots of users are using.  At a minimum, the few tiles that the tests use need to made available somewhere (maybe in the Cesium repo).
But really, I don't understand why you're getting rid of smallterrain.  Yes, people shouldn't use it.  Yes, it should be removed from the Data and Assets page.  But the cost to you of keeping it around is tiny since it's like 1/50 of the size of STK World Terrain (probably less by now), and it gives us all an easy way to test out heightmap terrain support.

Good point, I forgot they were testing different things.  I agree that we should make sure we continue to test against a heightmap set and I'm okay with that being part of the original smallterrain set.
However, as far as keeping the entire smallterrain up and available publicly, I'm not sure if that will happen or not, but I haven't been involved in any of the discussions related to it.  Hopefully @pjcozzi or @abwood can chime in with their thoughts or clarifications.

Either way, we should merge this PR soon, since it fixes failing tests in master.  This PR doesn't actually change anything, it just uses a working url.

I agree having a heightmap tile - or something procedurally generated - for testing is needed.
Now that STK World Terrain has water, is higher res, and is in Cesium's preferred format, we want to move forward with it, not smallterrain.  We announced this in March.

It's very useful to have some real terrain in the heightmap format for testing (both automated and interactive).  Obviously quantized meshes are better and the way forward, but unless you're planning to deprecate the code that handles heightmaps within Cesium, deleting the best publicly available dataset in that format doesn't make much sense to me.  What's the motivation?

By the way, I'm totally cool with it being deprecated, the URL changing, etc.  I just wouldn't get rid of it entirely so we can still use it for testing.  If you don't want to host it anymore I can probably get NICTA to do so.

I agree with @kring that there seems to be no reason to completely remove the dataset.  I'm sure we can rummage around our couches and come up with the ~$0.50 per month it will cost to host it on S3.

In any event, I am pulling down the vhd from blob storage for safe keeping.
I'm out on vacation tomorrow thru next week, but afterwards I can rehost
directly from s3 if that is the consensus.
Alex
On Jun 26, 2015 7:05 PM, "Scott Hunter"<EMAIL_ADDRESS>wrote:

I agree with @kring https://github.com/kring that there seems to be no
reason to completely remove the dataset. I'm sure we can rummage around our
couches and come up with the ~$0.50 per month it will cost to host it on S3.
—
Reply to this email directly or view it on GitHub
https://github.com/AnalyticalGraphicsInc/cesium/pull/2835#issuecomment-115913206
.

@abwood, we already have a bucket on S3 for cesiumjs.org, so I'll just upload the smallterrain data to their directly from the old server, I'll then open another PR to update the urls to point to the new home (or we can put in a redirect from cesiumjs.org so the old url works again).

As long as it is not promoted, not supported, not guaranteed, and so on, I am fine with having this somewhere for testing.


fix: cross-fetch version

cross-fetch 3.1.8 is breaking when using chrome extension executing requests inside service worker
overriding it to 4.0.0 fixed the problem
ref: cross-fetch

CI was not passing; hitting "update branch" appears to have fixed it.

Would it be possible to move the override to some packages/*/package.json instead of having it globally set on the root package.json?
I wonder what package uses such an outdated cross-fetch version from before Oct 5, 2020.

graphql-request latest uses it.

Good find! Indeed, they're using this outdated version, there's this issue about it.
For now, we could still move the overriding inside the provider package.
Congrats on this hotfix, @LuizAsFight!

@arboleya
https://pnpm.io/package_json#pnpmoverrides

Note that the overrides field can only be set at the root of the project.


Terraform error format is not covered by error renderer

Type: Bug
Terraform errors format has no comma or colon between the file name and the line number.
Example:
│ Error: Invalid reference
│
│   on main.tf line 2, in locals:
│    2:   foo = abcd

This format is not covered by the built-in error renderer in VS Code.
VS Code version: Code 1.76.0 (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:22:44.506Z)
OS version: Windows_NT x64 10.0.19043
Modes:
Sandboxed: Yes

/gifplease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

I refer to the terminal auto-link feature. I pushed a fix proposal in #175906.
Here's a demo. I expect the link to include main.tf line 2, and open the file in line 2. Instead, the line number is not parsed as expected, so clicking main.tf just opens the file, but not at this line.


Fix XBee manufacturer and model

Added model and manufacturer on device level for XBee
Removed model and manufacturer on endpoint level since I couldn't find where it is used.

This is required for home-assistant/core#41311 to actually work.

Should we change manufacturer to Digi by the way?


Page refreshes on search submit in Safari

Reported by @melvinwevers in #119:

Also, searching logs me out of the program, prompting the login screen.

Was fixed in 39a2b12.


Update Issue - 12638190

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 38.0

Operating System
Windows XP

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


Fix elements removal for non-Layout parent

fixes #217

ElementManager's method RemoveElement(IElementHandler handler) is replaced with RemoveChildElement(IElementHandler parentHandler, IElementHandler childHandler)
Instead of handling child's element removal in MobileBlazorBindingsElementManager, it delegates removal to parent's RemoveChild method.
Implemented RemoveChild method for all container elements handlers.
Fixed AddChild method for some Shell controls handlers to handle physicalSiblingIndex argument.

Implementing RemoveChild for some Shell elements was a little bit tricky - because of implicit conversions AddChild method can actually add multiple elements, and we need to identify which ShellItem to remove (e.g. when you add TemplatedPage to Shell it actually adds ShellItem -> ShellSection -> ShellContent -> TempletedPage).
I decided to move methods AddChild and RemoveChild to separate interface IXamarinFormsContainerElementHandler, it seemed like a better option to me. Let me know if you'd prefer it other way.
Example:

AppShell.razor
<Shell>
  <TabBar>
      @if (firstTabDisplayed)
      {
          <Tab Title="Tab1">
              <ContentPage>
                  <StackLayout>
                      <Label Text="Tab1" />
                      <Button Text="Switch tab" OnClick="SwitchTab" />
                  </StackLayout>
              </ContentPage>
          </Tab>
      }
      else
      {
          <Tab Title="Tab2">
              <ContentPage>
                  <StackLayout>
                      <Label Text="Tab2" />
                      <Button Text="Switch tab" OnClick="SwitchTab" />
                  </StackLayout>
              </ContentPage>
          </Tab>
      }
  </TabBar>
</Shell>

@code {
  bool firstTabDisplayed = true;

  void SwitchTab()
  {
      firstTabDisplayed = !firstTabDisplayed;
  }
}

Before

After

Open question. I don't know how to handle MenuItem removal from Shell properly. Thing is that implicit conversion to ShellItem wraps it in MenuShellItem, which is internal type. Therefore, I'm not sure how to identify corresponding ShellItem properly.
Currently I use something like that:
        private XF.ShellItem GetItemForMenuItem(XF.MenuItem childAsMenuItem)
        {
            // MenuItem is wrapped in ShellMenuItem, which is internal type.
            // Not sure how to identify this item correctly.
            return ShellControl.Items.FirstOrDefault(item => item.Title == childAsMenuItem.Text
                && item.Items.Count == 0);
        }

Wow very nice, I will take a close look at this change!
I agree that comparing the title/text is not good because it won't be reliable. I'll see if I can come up with some way to work around the internal type problem.

@Dreamescaper I pushed a couple of small changes to the branch so we can improve the check for MenuShellItem. I'll do some additional testing on the change and if it's all good, I'll merge.

Image

Probably would be better to continue discussion here.
I haven't seen such issue, let me know how to reproduce that if you need any help.

@Dreamescaper I pushed a change that updates the ControlGallery sample. If you run it (I ran on iOS Simulator) you can add as many Tabs as you want, but if you go to a tab and add a ShellContent, you get that exception.

BTW I doubt this is a regression because this scenario was never well tested before. I think this PR is a huge step forwards in the design and implementation so depending on how things go, we could take this PR after I do a bit more testing, and leave the rest for a follow-up investigation.

I pushed a change that adds a Remove button as well. Weirdly, you can remove & add tabs, but you can only remove shell contents! Adding shell contents is not allowed.

Seems like you can add contents, but only if your selected tab is not the last one.

Hmm I'm not able to add any Shell Contents ever (after the initial load). No matter where I am in the Control Gallery sample, clicking Add Shell Content always throws.

I checked the sample app code on the master branch and it confirmed that this isn't a regression.
On master I can add but not remove tabs. And I can't add or remove shell contents at all.

So adding the following line to ShellSectionHandler.AddChild solves the issue for me:
            contentToAdd.Content ??= new XF.Page();

But I don't get why doesn't it fail during initial load, and why it doesn't fail for me for some tabs.

How did you get adding/removing shell content to work? I still can't get that to work at all.
As far as initial load working, there were some recent changes to Shell that might have affected that due to some bug reports I filed on Xamarin.Forms. Not sure if it's related, but the point is that initial load in Xamarin.Forms sometimes has slightly different behavior than what happens once it's all up and running.

I've pushed workaround to branch.

I also get the following exception from time to time (I'm testing on Android emulator):
'The application's PagerAdapter changed the adapter's contents without calling PagerAdapter#notifyDataSetChanged! Expected adapter item count: 5, found: 4 Pager id: 8 Pager class: class crc64720bb2db43a66fe9.FormsViewPager Problematic adapter: class crc643f46942d9dd1fff9.ShellFragmentPagerAdapter'

That looks more like Xamarin.Forms bug though

That workaround lets me add shell contents - sometimes. And depending on which shell content I remove, sometimes it works, sometimes I get a NullRef error page, and sometimes the app insta-crashes and disappears.

I get Android exception even if I remove element directly from NativeControl, so that's certainly Xamarin.Forms bug.

Pushed another change to update some other controls that are containers and should implement the new IXamarinFormsContainerElementHandler interface (yay!) and also removed unused code from controls that were never containers and don't need an AddChild method. Double-yay!

Investigated why Tab name for added Shell Content is not displayed immediately.
We add empty content first, and then set all attributes, including title.
And seems like Xamarin does not like that.
I can see same behavior for the following code:

Code
    void AddShellContentDirectly()
    {
        var content = new Xamarin.Forms.ShellContent
        {
            Content = new Xamarin.Forms.ContentPage
            {
                Content = new Xamarin.Forms.StackLayout
                {
                }
            }
        };
        tab.NativeControl.Items.Add(content);

        content.Title = "Added directly";
    }

Recording

So I suppose that's Xamarin bug as well, not sure how valid this scenario is.

@Dreamescaper I'm pretty happy with where this is right now. I very much appreciate all your work on this!
I'm thinking it would be best to merge this PR as it is now and we can deal with any other issues as follow-up PRs. Does that seem reasonable to you?

Sure!


Rename TopStories to News

Issue Description
Rename TopStories to News
TopStories should be changed to News as we are using the full news feed not the TopStories feed
Additional Information

React Native version: any
Platform(s) (iOS, Android, or both?): all
Operating System (macOS, Linux, or Windows?): all

done 93e3f12de3d5266b58d4c88ec2fce6a6ad06fc84


Error with new version

Hi
Thanks for the update. Unfortunately, the new version is giving me an error that I wasn't getting yesterday with the same command:
strawberry SAM24297965.bam -o ./ -g gencode.v29.annotation.gtf -p 22
Error:
strawberry: /home/ruolin/git/strawberry/src/alignments.cpp:347: Strand_t HitCluster::ref_strand() const: Assertion `i.strand() == strand' failed.
Aborted (core dumped)
I did successfully get a .gtf yesterday with the previous version, but not with the new version. Can this be resolved?
Best wishes,
Chris

One more thing Ruolin, if you don't mind, is I don't want strawberry to quantify, I just want the annotations because I am using Salmon to quantify.
Is it possible to have a parameter that disables this? I expect that will improve runtime.


Prevent left nav's active item border from adding padding

Before:

After:

Finally! That issue has been driving me crazy! :trollface:


Using observer() throws:  Argument of type 'typeof Foo' is not assignable to parameter of type 'IReactComponent'.

I am trying to make a React component an observer.
If I do it like that:
interface LoginViewProps{
    dataProvider:IDataProvider;
}

class LoginView extends React.Component<LoginViewProps,null>{

    viewModel: ILoginViewModel;
    constructor(props:LoginViewProps){
        super(props);
        this.viewModel = new LoginViewModel();
        this.viewModel.dataProvider = props.dataProvider
    }
}
export default observer(LoginView);

I get the following error:
(19,25): Argument of type 'typeof LoginView' is not assignable to parameter of type 'IReactComponent<any>'.   Type 'typeof LoginView' is not assignable to type 'ClassicComponentClass<any>'.     Type 'LoginView' is not assignable to type 'ClassicComponent<any, ComponentState>'.       Property 'replaceState' is missing in type 'LoginView'.
But If I did the following I don't get an error:
... class LoginView extends React.Component { ...
Is that normal ? How can I fix this ?

@amrfarid140 can you please provide a solution?

I am having this problem now and what is causing it is the presence of the componentDidUpdate life-cycle method. Just by commenting the method the error is gone.
Can this issue be reopened?

@vipseixas open a fresh issue including minimal reproduction

@mweststrate Just figured out what was happening, I'll leave the solution here for other people with the same problem:
My componentDidUpdate method was with a subtle error at its signature, instead of
componentDidUpdate(prevProps: Props, prevState: State) { ... }
It was:
componentDidUpdate(prevProps: Props, prevState?: State) { ... }
So the compiler made the component incompatible with the observer() parameters.
I don't quite understand why the compiler did not complain about the method itself but complains about the component type afterwards but it seems that any change at the signatures of life-cycle methods causes this issue.


Animation Substate Expansion

Animation states are an essential element to GB Studio with plenty of events for it, but there's an aspect of them that isn't really talked about:
"substates"
(I don't know if there's an official name for these).

Currently, there's nothing much you can do with them. I have some ideas that would expand this rather obscure feature:

The ability to add substates to an animation state
The ability to remove substates from an animation state
An option to rename substates
A new event that allows you to change the player or actor's animation to a specific substate
The ability to copy and paste substates (Not just single frames or full states)
A new event that runs a script depending on an actor or player's animation substate (ex. "If Substate is Moving Right").

Big bump for being able to check the substate/animation state of an actor/player, that would be super useful

An animation state is just an array of 8 animations. Those "substates" are predefined names for the indices of the animations in an animation state array. Idle Right is always 0, Moving Right is always 4, Climbing is 6, but that's also Moving Up.
Less importantly, those 8 animations are just pointers to start and end frames in an array of metasprites, and if you have more than one animation state using the same tileset, all of the animation states point to the same array of metasprites, so if you know which frame your specific animation starts on, you can use Set Animation Frame to play that animation.
The only real differences between animation types is which ones get flipped automatically. The rest is just the different scene types using the animation clips in different ways.  This is because the top down game uses actor_set_dir (which sets both the animation frame and the actor's direction) for any movement, and platformer only uses actor_set_dir for left and right movement and actor_set_anim for everything else so that the actor's direction is preserved.
In my platformer, I needed to be able to look up and down while moving left and right, so I used the regular platformer animation type, but I modified the animation exporter so that it also flipped the climb animation and put it in slot 7.
Perhaps a good first step is just adding a generic animation state type with 8 numbered slots so it's more clear what's really going on... also copy paste.


allowVolumeExpansion no work

Rancher versions:
rancher/rancher:v2.0.1
I configure glusterfs storageclass and works but Claim resize no.
change de size of pvc see error:
The PersistentVolumeClaim "xxxxxx" is invalid: spec: Forbidden: field is immutable after creation.
How I can configure "ExpandPersistentVolumes=true" in cluster?

Hi @Itiho , I think Rancher doesn't allow edit existing PVC from UI.
Where did you see the error?

I dont use UI for edit pvc. I use YAML and kubectl.
Claim resize need cluster configure  with "ExpandPersistentVolumes=true" option
https://akomljen.com/easy-way-to-resize-kubernetes-persistent-volumes/

You can use RKE or something else to start a k8s cluster with the feature gate enabled and import it.
In Rancher v2.0,4, rancher supports edit cluster options as yaml. You can use the following yaml to provision k8s from Rancher UI.

addon_job_timeout: 30
authentication:
  strategy: "x509"
ignore_docker_version: true
ingress:
  provider: "nginx"
kubernetes_version: "v1.10.3-rancher2-1"
network:
  plugin: "canal"
services:
  etcd:
    extra_args:
      heartbeat-interval: 500
      election-timeout: 5000
    snapshot: false
  kube_controller:
    extra_args:
      feature-gates: ExpandPersistentVolumes=true
  kube_api:
    pod_security_policy: false
    extra_args:
      admission-control: PersistentVolumeClaimResize,Initializers,NamespaceLifecycle,LimitRanger,ServiceAccount,DefaultStorageClass,DefaultTolerationSeconds,NodeRestriction,ResourceQuota
      feature-gates: ExpandPersistentVolumes=true
ssh_agent_auth: false

I edit in API and give this error:
[kube-apiserver] on host [<IP_ADDRESS>]: Get https://localhost:6443/healthz: dial tcp <IP_ADDRESS>:6443: getsockopt: connection refused, log: [admission-control plugin " Initializers" is unknown, admission-control plugin " NamespaceLifecycle" is unknown, admission-control plugin " LimitRanger" is unknown, admission-control plugin " ServiceAccount" is unknown, admission-control plugin " DefaultStorageClass" is unknown, admission-control plugin " DefaultTolerationSeconds" is unknown, admission-control plugin " NodeRestriction" is unknown, admission-control plugin " ResourceQuota" is unknown]

{
"addonJobTimeout": 30,
"authentication": {
"strategy": "x509",
"type": "/v3/schemas/authnConfig"
},
"authorization": {
"type": "/v3/schemas/authzConfig"
},
"bastionHost": {
"sshAgentAuth": false,
"type": "/v3/schemas/bastionHost"
},
"cloudProvider": {
"type": "/v3/schemas/cloudProvider"
},
"ignoreDockerVersion": true,
"ingress": {
"provider": "nginx",
"type": "/v3/schemas/ingressConfig"
},
"kubernetesVersion": "v1.10.3-rancher2-1",
"network": {
"canalNetworkProvider": {
"iface": "",
"type": "/v3/schemas/canalNetworkProvider"
},
"plugin": "canal",
"type": "/v3/schemas/networkConfig"
},
"services": {
"etcd": {
"extraArgs": {
"election-timeout": "5000",
"heartbeat-interval": "500"
},
"snapshot": false,
"type": "/v3/schemas/etcdService"
},
"kubeApi": {
"extraArgs": {
"admission-control": "PersistentVolumeClaimResize, Initializers, NamespaceLifecycle, LimitRanger, ServiceAccount, DefaultStorageClass, DefaultTolerationSeconds, NodeRestriction, ResourceQuota",
"feature-gates": "ExpandPersistentVolumes=true"
},
"podSecurityPolicy": false,
"type": "/v3/schemas/kubeAPIService"
},
"kubeController": {
"type": "/v3/schemas/kubeControllerService"
},
"kubelet": {
"failSwapOn": false,
"type": "/v3/schemas/kubeletService"
},
"kubeproxy": {
"type": "/v3/schemas/kubeproxyService"
},
"scheduler": {
"type": "/v3/schemas/schedulerService"
},
"type": "/v3/schemas/rkeConfigServices"
},
"sshAgentAuth": false,
"type": "/v3/schemas/rancherKubernetesEngineConfig"
}

FYI, I have created the clusters successfully with AWS node driver.

Added a new cluster with the yaml successfully in v2.0.4
Updated an existing ready cluster with the yaml successfully in v2.0.4

We can config ExpandPersistentVolumes=true as extra_args in cluster yaml via UI.
Close this issue as it is for how to config ExpandPersistentVolumes=true. Please feel free to create another issue if you still see the problem.

Edit in api get
This cluster is currently Updating; areas that interact directly with it will not be available until the API is ready.
[controlPlane] Failed to bring up Control Plane: Failed to verify healthcheck: Failed to check https://localhost:6443/healthz for service [kube-apiserver] on host [xxx.xxx.xxx.xxx]: Get https://localhost:6443/healthz: dial tcp [::1]:6443: getsockopt: connection refused

Can you look at the container log of api server, please? Is there any clue?

example of 2.1.3 version
services:
  etcd:
    creation: "12h"
    extra_args:
      election-timeout: "5000"
      heartbeat-interval: "500"
    retention: "72h"
    snapshot: true
  kube-api:
    extra_args:
      enable-admission-plugins: "PersistentVolumeClaimResize,Initializers,NamespaceLifecycle,LimitRanger,ServiceAccount,DefaultStorageClass,DefaultTolerationSeconds,NodeRestriction,ResourceQuota"
      feature-gates: "ExpandPersistentVolumes=true"
    pod_security_policy: false
    service_node_port_range: "30000-32767"
  kube-controller:
    extra_args:
      feature-gates: "ExpandPersistentVolumes=true"
  kubelet:
    extra_args:
      volume-plugin-dir: "/usr/libexec/kubernetes/kubelet-plugins/volume/exec"
    extra_binds:
      - "/usr/libexec/kubernetes/kubelet-plugins/volume/exec:/usr/libexec/kubernetes/kubelet-plugins/volume/exec"
    fail_swap_on: false
ssh_agent_auth: true


Bug: tf.scatter_update does not update variable with float type on GPU

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Linux Ubuntu 18.04.1 LTS
TensorFlow installed from (source or binary): 1.11
TensorFlow version (use command below): pip
Python version: 3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA release 9.0, V9.0.176
GPU model and memory: TITAN Xp / 12Gb

Describe the current behavior
On GPU, tf.scatter_update does not update the variable if the type of the variable is defined as float. However, the same code works without a problem if the code is executed on CPU. If you run the code I provided here for you, you will get random values like the one that comes in the following every time you run the code:
[array([-0.27238935, -0.11273658, -0.18543988, -0.34574947, -0.4174637 ,
-0.35478222], dtype=float32), 6]
This problem does not happen on a CPU, or if the variable type is set to integer.
Describe the expected behavior
It should simply update the variable and should output the following in a deterministic manner:
[array([19., 19., 19., 19., 19., 19.], dtype=float32), 6]
I suppose this is a bug in the caching process involved if the code is run on a GPU.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
import tensorflow as tf
import os

os.environ["CUDA_DEVICE_ORDER"]="PCI_BUS_ID"
os.environ["CUDA_VISIBLE_DEVICES"]='0'
def cond(size, i):
    return tf.less(i,size)

def body(size, i):
    b=2*7.5+c

    with tf.variable_scope("a", reuse=tf.AUTO_REUSE):
	    a = tf.get_variable("a",[6],dtype=tf.float32)

        a = tf.scatter_update(a,i,b)

        with tf.control_dependencies([a]):
	        return (size, i+1)

with tf.Session() as sess:
    c=tf.constant(4.0)
    i = tf.constant(0)
    size = tf.constant(6)
    _,i = tf.while_loop(cond,
            body,
            [size, i])

    a = tf.get_variable("a",[6],dtype=tf.float32)

    init = tf.initialize_all_variables()
    sess.run(init)
    print(sess.run([a,i]))

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

This is working as expected.
 sess.run([a,i]) does not define the evaluation order or a and i.  a can be evaluated either before or after or during the loop, and before the loop a is randomly initialized. Therefore you can get different results.
See documentation: https://www.tensorflow.org/api_docs/python/tf/Session#run

Order in which fetches operations are evaluated inside the call is undefined.

Thanks for your reply. But what would be the solution then ? how can we force i to be evaluated sooner in the same session as a ? also, as I mentioned before, this problem does not occur if the code is executed on a CPU.

You can just sess.run(a) after finishing sess.run(i).
Another bug in your code above is that the second get_variable should be under the variable scope "a", otherwise you're creating a new variable.

Why the bug/performance label? The code is working as expected as far as I can see.

Thanks. It works as expected now.

Are you satisfied with the resolution of your issue? Yes No


⚠ Fortnite has degraded performance

In 35629a2, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1148 ms

Resolved: Fortnite performance has improved in 00add65.


Adding an object constant

Hi can you advise if it is possible to add object constants and the best practice for adding these .
This technique tends to be used for injecting third party objects that are assigned to the global.
e.g.
.constants("moment",moment)

If I try this in gulp I get a reference error moment is undefined.
I can add this manually and have no issues and works as expected but would like to add as part of the build process if possible.
Thanks

I found a work around by doing the following:
wrap: "(function(){'use strict';\n\n<%= __ngModule.replace(';','.constant\(\"moment\",moment\);') %>}();",

Works but nice to see if there are any alternative out there.I basically replace the content in the wrap option.


Why several intersection tiles that are rotations of eachother?

Seems unnecessary to have so many intersection tiles that are identical to each other except from the rotation?

Done in rewrite branch


run npm audit fix to address security issues.

@vincenzoarieta93 I'd like you to review this please (i.e. let me know if its ok with you if I merge and publish it).
The sole purpose of this change is to address known security issues in dependencies.
I have also added you to this repository, so you have an invite in your email. If you accept it and want to pursue getting it working for you, we can review each other's changes if we need to make any.

@acinader looks fine to me. Go ahead 👍


jdk alpine version upgrade

upgrade version of jdk/alpine image (base image of scala image)
Description
upgrade version of jdk/alpine image (base image of scala image) to adreess vulnerability issue
from
adoptopenjdk/openjdk8-openj9:x86_64-alpine-jdk8u262-b10_openj9-0.21.0
to
adoptopenjdk/openjdk8-openj9:x86_64-alpine-jdk8u312-b07_openj9-0.29.0
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[x] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #98 (8eaebd6) into master (1b35b42) will decrease coverage by 7.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
- Coverage   78.37%   70.84%   -7.53%
==========================================
  Files         194      194
  Lines        9265     9265
  Branches      674      572     -102
==========================================
- Hits         7261     6564     -697
- Misses       2004     2701     +697

Impacted Files
Coverage Δ

...la/org/apache/openwhisk/http/BasicRasService.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...che/openwhisk/core/invoker/LogStoreCollector.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/openwhisk/http/BasicHttpService.scala
0.00% <0.00%> (-91.55%)
:arrow_down:

.../apache/openwhisk/core/controller/Controller.scala
0.00% <0.00%> (-84.54%)
:arrow_down:

...pache/openwhisk/core/invoker/InvokerReactive.scala
0.83% <0.00%> (-79.17%)
:arrow_down:

...la/org/apache/openwhisk/core/invoker/Invoker.scala
0.00% <0.00%> (-70.50%)
:arrow_down:

...nwhisk/core/database/RemoteCacheInvalidation.scala
10.00% <0.00%> (-70.00%)
:arrow_down:

...che/openwhisk/core/loadBalancer/LoadBalancer.scala
16.66% <0.00%> (-66.67%)
:arrow_down:

...whisk/connector/kafka/KafkaConsumerConnector.scala
0.00% <0.00%> (-57.75%)
:arrow_down:

...he/openwhisk/core/invoker/MessagingActiveAck.scala
0.00% <0.00%> (-53.85%)
:arrow_down:

... and 46 more

Continue to review full report at Codecov.

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


Future Development? Patent Client

Hey! This is the only way I can see to contact you, so here I go!
I'm the author and maintainer of patent_client, a library with a similar scope and feature set as your own. patent_client is under active development, and growing, so if you'd like, I'd love to have you contribute, or add a note on your readme pointing to it!

PyPI | GitHub | Docs
Thanks!
Parker

Hello Parkerhancock:
Can it parse USPTO data to pandas file?
Can it parse non-patent-references?


Not able to share selected apps or windows in Zoom

I am not able to see other windows or apps for sharing only the selected app in dwm. It used to work under gnome. It seems there is something which is not implemented or not supported in dwm.

Is this specific to Luke's build of DWM? Try compiling it from the vanilla suckless code at https://git.suckless.org/dwm/ and see. If it still doesn't work, then this isn't an issue with Luke's build and an issue with DWM itself.


verify_rsa no method 'verify' for class String

I recently switched my encoding algorithm from HS256 to RS256 and can no longer verify my tokens.  Line 59 in jwt.rb seems to be the problem:
public_key.verify(OpenSSL::Digest.new(algorithm.sub('RS', 'sha')), signature, signing_input)
raises a no method error, no method 'verify' for type String.  My public key is a string (no sure what else it should be?), and I don't see verify defined anywhere.  What am I missing?
Thanks!

For context, I'm using Auth0 and Knock, if that helps.

This seems to be a limitation of Knock, not ruby-jwt.  Apologies.

@stantoncbradley I'm getting the same error, could you please share as you did fix it?

@AugustoPedraza I was using Knock to pass my JWT to ruby-jwt and Knock didn't support RS256 so it wasn't passing in the proper params.  Adding support for RS256 on Knock fixed my issue.  ruby-jwt works fine for me after that, make sure you are passing the proper params.  It's been awhile since I worked on this so I probably won't be able to give you much more details on what could be wrong with your payload.  Like I said, I think your issue is upstream from ruby-jwt (at least mine was).  Hope that helps!

@stantoncbradley thanks for your reply. I was getting the same error because I was using an invalid Auth0 workflow to get my JWT.

@AugustoPedraza can you elaborate on 'using an invalid Auth0 workflow to get my JWT' , Im having the same issue, but fairly certain that Im using a proper workflow, and my JWT validates and signature verifies at jwt.io

I got this working, as I expected I was passing in the token_public_key incorrectly. #148


Write an authors/credit page

Update index.html to add a section or a link to page with current and past authors, contributors and sources.

Closed by 7ee38b2


WebAdmin Permissions

Not sure if this is everywhere but you can click add permission on a group with nothing in there and it will enter a blank entry in to the list.

This is a bug in the Core plugin, I have re-reported it in the appropriate repository.


Error when changing project's visibility to public

Please see attached screen

Thanks! will fix- how urgent?
On Mon, Sep 14, 2015 at 9:44 AM, Natalia Bartol<EMAIL_ADDRESS>wrote:

Please see attached screen
[image: screen shot 2015-09-14 at 14 36 07]
https://cloud.githubusercontent.com/assets/1488329/9851137/102e57c0-5aef-11e5-8d75-97a568309456.png
—
Reply to this email directly or view it on GitHub
https://github.com/mbryzek/apidoc/issues/367.

Would be great if this could be fixed this week. Thanks!

you got it! congrats on open sourcing - very exciting!

On Sep 14, 2015, at 10:03 AM, Natalia Bartol<EMAIL_ADDRESS>wrote:
Would be great if this could be fixed this week. Thanks!
—
Reply to this email directly or view it on GitHub.


[Headings] Size of section titles does not match intended hierarchy (H1 is smaller than H2/3/etc.)

On mobile, the size of headings based on levels of hierarchy is not right. The H1 is smaller than H2/3/etc.

@dylanzheng94 has this been completed? should we close this issue?

completed


Added GizmosExtensions.DrawBoxCast

added GizmosExtensions.DrawBoxCast to visualize Physics.DrawBoxCast

No worries if you're not a fan!