Remaining -Wconversion warnings

Hi,
After my PR introducing -Wconversion compilation flag a couple of weeks ago, there still remain a couple of warnings that I don't know how to resolve:
In file included from /home/user/pistache/src/common/http_defs.cc:11:0:
/home/user/pistache/include/pistache/date.h: In constructor ‘constexpr date::weekday_indexed::weekday_indexed(const date::weekday&, unsigned int)’:
/home/user/pistache/include/pistache/date.h:1641:11: warning: conversion to ‘unsigned char:4’ from ‘unsigned char’ may alter its value [-Wconversion]
     : wd_(static_cast<decltype(wd_)>(static_cast<unsigned>(wd))),
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/user/pistache/include/pistache/date.h:1642:14: warning: conversion to ‘unsigned char:4’ from ‘unsigned char’ may alter its value [-Wconversion]
       index_(static_cast<decltype(index_)>(index)) {}
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This is conversion warning from "unsigned char" to bitfield 'unsigned char:4'. There is already static_cast<decltype(index_)> - apparently this doesn't work. It looks like the following fixes these warnings:
@@ -1638,8 +1638,8 @@ inline bool weekday_indexed::ok() const NOEXCEPT {
 CONSTCD11
 inline weekday_indexed::weekday_indexed(const date::weekday &wd,
                                         unsigned index) NOEXCEPT
-    : wd_(static_cast<decltype(wd_)>(static_cast<unsigned>(wd))),
-      index_(static_cast<decltype(index_)>(index)) {}
+    : wd_(static_cast<unsigned>(wd) & 0xF),
+      index_(index & 0xF) {}

Any comments?
[ 69%] Building CXX object src/CMakeFiles/pistache.dir/common/http.cc.o
In file included from /home/user/pistache/include/pistache/http.h:20:0,
                 from /home/user/pistache/src/common/http.cc:8:
/home/user/pistache/include/pistache/async.h: In instantiation of ‘void Pistache::Async::Private::impl::Continuation<T, Resolve, Reject, Pistache::Async::Promise<U>(Args ...)>::doResolve(const std::shared_ptr<Pistache::Async::Private::CoreT<T> >&) [with T = long int; Resolve = std::function<Pistache::Async::Promise<long int>(int)>; Reject = std::function<void(std::__exception_ptr::exception_ptr&)>; U = long int; Args = {int}]’:
/home/user/pistache/src/common/http.cc:1008:1:   required from here
/home/user/pistache/include/pistache/async.h:465:10: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
     auto promise = resolve_(detail::tryMove<Resolve>(core->value()));
          ^~~~~~~

This one deals with hard core templating in Pistache, and I'm didn't succeed to get to the root.
Thanks a lot

Ignore everything in date.h.  We did not write it; it was imported from some other place.  I should move it from "include/pistache" into "include/pistache/thirdparty".
I've not dug into the code (this is just a drive-by comment), but I'm surprised about the warning for the "auto promise =".  Why did the compiler fail to create the desired type?
Does this warning occur on both GCC-9 and Clang-?

I found that the following change fixes the warning in async.h:
diff --git a/src/common/http.cc b/src/common/http.cc
index b8728aa..499f352 100644
--- a/src/common/http.cc
+++ b/src/common/http.cc
@@ -794,7 +794,7 @@ Async::Promise<ssize_t> ResponseWriter::putOnWire(const char *data,
     auto fd = peer()->fd();

     return transport_->asyncWrite(fd, buffer)
-        .then<std::function<Async::Promise<ssize_t>(int)>,
+        .then<std::function<Async::Promise<ssize_t>(ssize_t)>,
               std::function<void(std::exception_ptr &)>>(
             [=](int /*l*/) {
               return Async::Promise<ssize_t>([=](

I don't know if it's a correct one though.

The change below fixes the warning happening in date.h. I've checked the latest version of date.h in the source location - they suppress the warning the similar way, only locally in the file itself.
diff --git a/src/common/http_defs.cc b/src/common/http_defs.cc
index 31e3e76..30019cd 100644
--- a/src/common/http_defs.cc
+++ b/src/common/http_defs.cc
@@ -8,7 +8,14 @@
 #include <iostream>

 #include <pistache/common.h>
+#ifdef __GNUC__
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wconversion"
+#endif
 #include <pistache/date.h>
+#ifdef __GNUC__
+#pragma GCC diagnostic pop
+#endif
 #include <pistache/http_defs.h>

 namespace Pistache {

This PR breaks all travis builds.  For example: https://travis-ci.org/github/oktal/pistache/jobs/674003067


Windows - Some models report "Unexpected array size" for GridResources/GridData

I am using CAVE 18.2.1-3 on Windows 11.
My installation has followed the instruction guide to a T as much as possible, running:

Miniconda 3 @ Python 3.7.6
JDK (1.8_181)
Python Module Numpy (1.15.1)
Python Module Jep (3.8.2)

All environment variables have been set correctly, and gridslice is present.  I am able to perform tasks which involve it such as isentropic analyses just fine.
I'm not able to load some models such as Vertical Circulation (on RAP13, or any other sources for that matter), and my console complains about array lengths.  If I can recall, my CentOS 7 setup seemed to load it without issue.
Some other models will report the same issue, appearing to successfully be retrieving the raw data (seen through both Wireshark traffic and console output), but unable to process it?
16:28:02.313 [main] WARN CaveLogger - Error requesting data for RAP13 1000MB Vertical Circulation  ()   04.20   0HR Sat 20:00Z 04-Mar-23: Unexpected IDataRecord array length: 3
com.raytheon.uf.viz.core.exception.VizException: Unexpected IDataRecord array length: 3
        at com.raytheon.viz.grid.rsc.general.GridResource.getData(GridResource.java:122)
        at com.raytheon.viz.grid.rsc.general.D2DGridResource.getData(D2DGridResource.java:243)
        at com.raytheon.viz.grid.rsc.general.GridResource.getData(GridResource.java:84)
        at com.raytheon.uf.viz.core.grid.rsc.data.GridDataRequestRunner.processOneRequest(GridDataRequestRunner.java:117)
        at com.raytheon.uf.viz.core.grid.rsc.data.GridDataRequestJobPool$GridDataRequestJob.run(GridDataRequestJobPool.java:120)
        at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)

Hello,
Thank you for bringing up this issue. I can confirm I'm seeing the same error on both our CentOS 7 and Windows installs. We will add this to our list of bugs.


Use unique names between kuttl tests.

There is a race condition between templates-ytt and templates-consume-output-of-components.

This isn't addressing the test pollution issue, which is that there are two ClusterConfigTemplates (same for ClusterTemplates) that are being created during kuttl tests with the name "example-config---consume-output-of-resources". That name must be deduplicated.

@waciumawanjohi I believe it does fix the pollution because those templates are overwritten when they are applied in the 00-templates.yaml phase so it doesn't really matter that they have the same name. The real problem was that the templates were both creating a ConfigMap with the same name, so when the second test would run, the expected ConfigMap already existed from a previous test.
I would still be in favor of renaming the templates to match the convention we have in our other tests of using the test name in the template name, but I believe we'd still need the change made in this PR in order to actually get rid of the pollution as the problem is the resulting ConfigMaps having the same name, not the templates themselves.
(I tried to verify this by renaming the templates but leaving the ConfigMap names as they are and I still saw the flakiness- renaming the ConfigMaps does seem to get rid of the flakiness though).

lgtm!


Long file names are cut in the tooltip

See below:

Long file names are still being cut which makes ensuring that the correct file is assigned as barcodes, forward and reverse run sometimes difficult.  Is it possible to have the text in the hover-over-window wrap instead of end in '....'?  As far as I understand it the files are assigned alphabetically by qiita not logically

fixed in https://github.com/biocore/qiita/pull/2365


Update crate edition to 2021

This was bugging me, and is an easy update. I'm not sure if we need to update the Rust version in Docker (seems possible).
Fixes #2249

@metaclips can help upgrade the builder image.
Checks pass without upgrading the image.

Yeah, I don't think we ended up needing to update the image.


Collection links for catalogs/collections

I'm currently experimenting with collection links in non-Items. I think there can be value in having them.
Example implementation: https://radiantearth.github.io/stac-browser/#/external/raw.githubusercontent.com/m-mohr/open-stac/3d-skysat/stac/open-skysat-data/cocabamba-peru/catalog.json
Having an easy way to navigate back to the next collection in the (potentially long) list of parents may help navigation.
Let's say you have a structure as such:

Root

ESA (collection)

Sentinels (catalog)

Sentinel-1 (catalog)
Sentinel-2 (catalog)

Level 1A (collection)

2020 (catalog)

Jan (catalog)

01 (catalog)

12345 (item)
67890 (item)
...

02 (catalog)
03 (catalog)
...

Feb (catalog)
Mar (catalog)
...

2021 (catalog)
2022 (catalog)

Level 2A (collection)
Level 2B (collection)

Others (catalog)

NASA (collection)

I think there should be easy ways without following multiple parent links up to the collection, e.g. for easy navigation from Level 1A to ESA or 01 to Level 1A. Thus my proposal is to also allow links with rel collection explicitly also in Catalogs and Collections. Currently it's only defined for Items.
Disclaimer: This is just an example for better understanding, it doesn't matter here whether ESA is a good or bad Collection. It could be something different.

Note: one of the most common questions Matthias gets. High Priority.


Merged project-template v9.0.3

Codecov Report

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

@@            Coverage Diff            @@
##             master     #537   +/-   ##
=========================================
  Coverage     27.93%   27.93%
  Complexity      960      960
=========================================
  Files            85       85
  Lines          3276     3276
=========================================
  Hits            915      915
  Misses         2361     2361

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 b001ffe...ed1870c. Read the comment docs.


Adding Merge sort

Adding merge sort Algorithm in java language.

Thanks for contributing.
But this problem was already solved using Java.


(TEST / WIP) Increase maximum array size limit from 100 to 1000

What this PR does:
Relaxes JSON API document limits for array sizes to 1,000 from 100, based on user feedback
Which issue(s) this PR fixes:
Fixes #745
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Bad base (not from "main"), closing.


Create Posts “ビジネスにおけるコミュンティの重要性”

Automatically generated by Netlify CMS

👷 Deploy Preview for strong-beijinho-c4d917 processing.

Name
Link

🔨 Latest commit
f9b3b9ef00f446edb6cd5cca546854389f10c489

🔍 Latest deploy log
https://app.netlify.com/sites/strong-beijinho-c4d917/deploys/65ebef397d350c00081b2bfa


Document public key used to sign exposure file in production system

It seems that currently the public key used to sign the exposure file/diagnosis keys that the CWA app receives from the backend server isn't documented anywhere.
Having this key publicly available would help enabling the signature verification in the microG exposure notification framework, see https://github.com/microg/GmsCore/issues/1307 for background discussion.

Internal Tracking ID: EXPOSUREAPP-10469

@Bubu Thanks for your request. Internal Tracking ID: EXPOSUREAPP-10469

@Bubu This is the public key:
MFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEc7DEstcUIRcyk35OYDJ95/hTg3UVhsaDXKT0zK7NhHPXoyzipEnOp3GyNXDVpaPi3cAfQmxeuFMZAIX2+6A5Xg==

We'll need to check where we could put it in this repo.

@Bubu
In the past, signature verification by Google's ENS could take an incredibly high amount of time for unknown reasons for older/low-end phones. On my LG-G4 it took around 70s in one test (derived from syslog some months ago). I'd suggest to do some performance checks when you enable it for microG. If microG has no problems at all to speedy verify signatures, then I'd suggest to have a look into Google's code (if possible) and file a bug report to them.
Unfortunately, since Google deactivated syslog for ENS in Europe due to data privacy reasons (elsewhere it's still active 😠 ), it's not possible to benchmark Google's signature verification currently.

@vaubaehn I just did a very minimal test with the public key from above added to the microG lib and verification was still very fast. I'm not too concerned about debugging googles implementation here though, I'll leave that to others :-).

@Bubu Thanks for the feedback. Happy to hear it's working good. So, If I got it right, ENS is actively downloading the public key everytime for signature verification? Then here's probably the bottleneck that I experienced... Good you can hard code the keys into microG.

I'm not too concerned about debugging googles implementation here though, I'll leave that to others :-).

True 😋

@vaubaehn

If I got it right, ENS is actively downloading the public key everytime for signature verification?

No, as far as I understand, apps have to provide their public key to Google/Apple when they request access to the ENF. At least that's what I gleamed from the respective documentations (Google), so I guess they keep an offline list of them somewhere in the ENF.
The comparing of the downloaded Diagonsis Keys (of people who are exposed) with the RPI collected by your phone gets more demanding as more people get sick, since your phone calculates all RPIs for all people who are sick (since it does not know anything about location, so if someone in Berlin gets sick and you live in Wuppertal you will still calculate his RPIs). I am guessing thats quite a lot to handle for older phones maybe?
But I never actually benchmarked any of this.


"$apply already in progress" when using events

This event binding method doesn't work in all cases. Anyway people say that the "scope.$apply" is an anti-pattern.
You should inject $timeout to directive and change all "scope.$apply" to "$timeout" to fix it and make worked in all cases.
This is the error :
Error: [$rootScope:inprog] $apply already in progress
http://errors.angularjs.org/1.3.14/$rootScope/inprog?p0=%24apply
at http://localhost:54147/Scripts/angular.js:63:12
at beginPhase (http://localhost:54147/Scripts/angular.js:14755:15)
at Scope.$apply (http://localhost:54147/Scripts/angular.js:14499:11)
at HTMLDivElement.onPlotSelected (http://localhost:54147/Scripts/angular-flot.js?v=0.0.14:72:15)
at HTMLDivElement.n.event.dispatch (http://localhost:54147/Scripts/jquery-2.1.4.min.js:3:6466)
at HTMLDivElement.r.handle (http://localhost:54147/Scripts/jquery-2.1.4.min.js:3:3241)
at Object.n.event.trigger (http://localhost:54147/Scripts/jquery-2.1.4.min.js:3:5574)
at HTMLDivElement. (http://localhost:54147/Scripts/jquery-2.1.4.min.js:3:11542)
at Function.n.extend.each (http://localhost:54147/Scripts/jquery-2.1.4.min.js:2:2882)
at n.fn.n.each (http://localhost:54147/Scripts/jquery-2.1.4.min.js:2:847) undefined

You mean a zero-second $timeout ?

Yes, like that:
 plotArea.on('plotclick', function onPlotClick(event, pos, item) {
            $timeout(function onApplyPlotClick() {
                scope.onPlotClick({
                    event: event,
                    pos: pos,
                    item: item
                });
            });
        });

And injections should be this format:
 angular.module('angular-flot', []).directive('flot', ['$timeout', function ($timeout) {

It works, tested...

Just published v0.0.15. Can you give it a shot?

Ok, it works.


Support gRPC 1.46.3 (also fix builds on Darwin)

Closes https://github.com/awakesecurity/gRPC-haskell/issues/136.
Some context
I had to fix issues with building nixpkgs.grpc on Darwin using release-22.05 nixpkgs branch. The pin from gRPC-haskell (1.34.1) was giving some “undefined reference” errors coming from nixpkgs.abseil-cpp library as it seems. It turned out that nixpkgs.grpc from release-22.05 branch (1.46.3) builds just fine on Darwin but it’s not compatible with gRPC-haskell, there are some breaking changes in gRPC API. I decided it would be easier to just implement support for gRPC-haskell for 1.46.3 gRPC version than debugging the old 1.34.1 against Darwin, especially taking into account that I don’t have any Apple hardware by hand.
What I changed
The change to the API that makes it work with gRPC 1.46.3 is relatively small. There are few differences in the newer version of gRPC how the insecure/unauthenticated server and channel are created. This are no *_insecure_* functions anymore. Instead there are functions that are creating insecure credentials object.
Though I significantly refactored Nix configuration for the project:

Updated nixpkgs pin to use the release-22.05 branch
Added shell attribute to default.nix that I used to make this change, original shell.nix was failing to build for me when I was making a fix. After the fix it started to build fine, but while API was broken is was impossible to enter a nix-shell. So this addition based on haskellPackages.shellFor can be useful comparing to shell.nix which is just grpc-haskell.env.
Added stack-env attribute to default.nix that makes Stack project buildable on NixOS (gmp and zlib are required). I used this command to build the project stack build --nix --system-ghc --no-test
Created new pins for gRPC and its Python dependencies to match the same version as recommended in the comments of gRPC derivation
Also created pins for the overridden Haskell dependencies so that the used patches are always applicable against other nixpkgs pins where the versions of those dependencies may change
Updated proto3-wire and proto3-suite to match the versions from release-22.05 branch of nixpkgs
Exposed in the attributes of default.nix overridden Haskell dependencies
Used nixpkgs.python3 instead of nixpkgs.python which is Python 2 by default because some of the Python dependencies of gRPC do not support Python 2 anymore and entering nix-shell fails due to this
Other changes are minor, more like stylistic ones

Note that I also updated the Stack configuration to use the newer snapshot to match the GHC version from nixpkgs pin. I also added stack.yaml.lock file to the project.
Open questions for the maintainer(s)
About tests

[ ] Just one test of grpc-haskell-core about graceful shutdowns (see LowLevelTests.testGoaway/core/tests/LowLevelTests.hs). But what’s weird that it’s not always failing. I managed to build it on one of my machines when it was successful. Do you have any ideas how to fix it or what could be the problem? It doesn’t seem like it’s related to my changes but rather to the newer version of gRPC for instance.
[ ] Attempt to build nix-build -A grpc-haskell without no-tests fails with: Builder called die: Cannot wrap 'tests/protoc.sh' because it is not an executable file. Is that a known issue?
[ ] What is the right way to run this example cabal configure --enable-tests && cabal test in a Nix Shell using new Cabal 3.x for instance? This command doesn’t work.$ cabal --version
cabal-install version <IP_ADDRESS>
compiled using version <IP_ADDRESS> of the Cabal library
$ cabal configure --enable-tests && cabal build && cabal test
'cabal.project.local' already exists, backing it up to
'cabal.project.local~0'.
Warning: The package list for 'hackage.haskell.org' does not exist. Run 'cabal
update' to download it.
Resolving dependencies...
cabal: Could not resolve dependencies:
[__0] trying: grpc-haskell-0.3.0 (user goal)
[__1] rejecting: grpc-haskell:!test (constraint from config file, command line
flag, or user target requires opposite flag selection)
[__1] trying: grpc-haskell:*test
[__2] unknown package: tasty-hunit (dependency of grpc-haskell *test)
[__2] fail (backjumping, conflict set: grpc-haskell, tasty-hunit,
grpc-haskell:test)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: grpc-haskell, grpc-haskell:test,
tasty-hunit

UPD: Just added missing pipes and tasty-hunit packages to the ghcWithPackages of grpc-haskell and it worked. Kind of. It runs the tests infinitely (never finishes):In order, the following will be built (use -v for more details):
 - grpc-haskell-0.3.0 (test:tests) (ephemeral targets)
Preprocessing test suite 'tests' for grpc-haskell-0.3.0..
Building test suite 'tests' for grpc-haskell-0.3.0..
Running 1 test suites...
Test suite tests: RUNNING...

Support of different gRPC versions

[ ] I think it would be better to support the old gRPC version as well, right? So maybe use CPP extension and add some conditions for the code. But the problem is that gRPC is not a Haskell package, there is no way to easily check the version of that C-library dependency version right? Maybe some Template Haskell magic can do that? Or maybe let’s provide gRPC version as Cabal flag? And default it to the supported 1.46.3 and in Nix there can be automatic set of this flag to the nixpkgs.grpc.version. What do you think about this?

Is there anything I can help to move this forward? @Gabriella439 @unclechu


epson-escpr: 1.7.3 -> 1.7.16

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 133024 at d6f65210 run on x86_64-linux 1

1 package built successfully:

epson-escpr

1 suggestion:

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/misc/drivers/epson-escpr/default.nix:20:15:
   |
20 |   patches = [ ./cups-filter-ppd-dirs.patch ];
   |               ^


Fix nil panic for converting nested json to metavalue

If you use a struct of multi-level json, you can cause a nil panic. metaor.GetMetas() may return nil the first time, but when one recurses, it will try calling GetMetas() on a nil object.  Oops.
You can trigger this by posting something like this to qor admin:
{
	"Author": "Super",
	"CreatedAt": "2016-09-29 16:51",
	"ID": 18,
	"Object": {
		"sub": {
			"name": "foo"
		}
	},
	"Title": "Duper"
}

For more details see my issue on the admin interface: https://github.com/qor/admin/issues/39

Thank you for your PR


Honor Jest default config

Consider the following snippet:
'use strict';

const { compose, baseConfig } = require('@moxy/jest-config');
const { defaults } = require('jest-config');

module.exports = compose([
    baseConfig,
    (config) => ({
        ...config,
        setupFiles: ['<rootDir>/jest.setup'],
        coveragePathIgnorePatterns: [
            ...defaults.coveragePathIgnorePatterns,
            '<rootDir>/test/utils/',
        ],
    }),
]);

I'd expect config to already provide sensible defaults instead of having to install jest-config myself to retrieve the default configurations.

Hmm, do you think this could be added as an addon, similar to [withWeb](https://github.com/moxystudio/jest-config/tree/master/lib/addons/with-web) or [withRTL](https://github.com/moxystudio/jest-config/tree/master/lib/addons/with-rtl)?
I'm unclear whether it would be ideal to have out configuration rest on top of other default configurations, because it tips the workload significantly towards developers who would like to use this package but avoid Jest's default configuration. Ideally, it would be balanced out. Also, i appreciate that our config is built on a tabula rasa, and this would pollute it considerably.
That said, i feel like adding this as an addon would be an ideal solution that would make it easier for both developers who want it, and remain the same for those who don't. It would also avoid breaking the existing contract. What do you think?

I'm not sure whether developing a new addon is the right direction. The defaults could provided by baseConfig in my opinion.
Just to give some context, when I was putting together my Jest config, the first thing that crossed my mind about config.coveragePathIgnorePatterns was that it already provided some common paths to ignore, namely node_modules which is Jest's default. In my use case above, I had to override coveragePathIgnorePatterns but still wanted to ignore node_modules nonetheless. I realized out Jest config does not configure coveragePathIgnorePatterns, so it threw an error indicating it could not be spreaded as it is not an iterable.
We don't need really to use Jest's default configuration per se, but at least I think we should still provide some good defaults for some settings, being coveragePathIgnorePatterns one of them.

@moxystudio/developers thoughts?

@acostalima as you noticed, currently, the baseConfig contains only a few overrides and not the whole config. This means you have to deal with possible things being empty and you have to deal with it accordingly, e.g.: https://github.com/moxystudio/jest-config/blob/master/lib/addons/with-rtl/index.js#L5.
We could have our baseConfig on top of jest-config, the only downside I see is that it becomes "polluted" but I'm fine with it. I would say it's a welcome change.


change transformCounts to transformAssay

change transformCounts to transformAssay (Rename transformCounts to transformAssay #394)

(out of sync with base branch still)


Extract Method crashes inside anon classes

TypeScript Version:  2.5.1
Code
const _ = class {
    foo() {
        console.log(123)
    }
}

Run extract method on console.log(123)
Bug:
[Error  - 14:55:00] 'getApplicableRefactors' request failed with error.
Error processing request. Cannot read property 'text' of undefined
TypeError: Cannot read property 'text' of undefined
    at getDescriptionForScope (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:74910:37)
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:74878:66
    at Array.map (native)
    at getPossibleExtractions (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:74869:35)
    at Object.getAvailableActions (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:74512:35)
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:72854:121
    at Object.flatMapIter (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:1545:23)
    at Object.getApplicableRefactors (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:72853:23)
    at Object.getApplicableRefactors (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:76688:32)
    at IOSession.Session.getApplicableRefactors (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82516:53)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:81486:61)
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82699:88
    at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82690:28)
    at IOSession.Session.executeCommand (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82699:33)
    at IOSession.Session.onMessage (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82719:35)
    at Interface.<anonymous> (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:83805:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)

I took a quick look at this and this and will submit a PR


4th Character Script not working.

Describe your problem here.
I have made a script which is supposed to add a 4th character to the stage and make them fully functional, however the code doesn't work and when the note is pressed nothing happens. Here is the code:
function onCreate()
	makeAnimatedLuaSprite('tail', 'tails-ccc', -481, -90); -- tails
	addAnimationByPrefix('tail', 'idle', 'idle', 12, false)
	addAnimationByPrefix('tail', 'singLEFT', 'left', 12, false)
	addAnimationByPrefix('tail', 'singDOWN', 'down', 12, false)
	addAnimationByPrefix('tail', 'singUP', 'up', 12, false)
	addAnimationByPrefix('tail', 'singRIGHT', 'right', 12, false)
	addAnimationByPrefix('tail', 'fly', 'fly', 12, false)
	addAnimationByPrefix('tail', 'skid', 'skid', 12, false)
	setProperty('tail.antialiasing',false)
	scaleObject('tail', 10, 10);
	addLuaSprite('tail',false)
end

function onBeatHit()
	if curBeat % 2 == 0 then
		objectPlayAnimation('tail', 'idle', true);
	end
end

--note stuff for tails

animationsList = {}
holdTimers = {tail = -1.0};
noteDatas = {tail = 0};
function makeAnimationList()
	animationsList[0] = 'singLEFT';
	animationsList[1] = 'singDOWN';
	animationsList[2] = 'singUP';
	animationsList[3] = 'singRIGHT';
end

function opponentNoteHit(id, direction, noteType, isSustainNote)
	if noteType == 'Special Sing' then
		if not isSustainNote then
			noteDatas.tail = direction;
		end
		characterToPlay = 'tail'
		animToPlay = noteDatas.tail;

		playAnimation(characterToPlay, animToPlay, true);
	end
	if noteType == 'invisible Special Sing' then
		if not isSustainNote then
			noteDatas.tail = direction;
		end
		characterToPlay = 'tail'
		animToPlay = noteDatas.tail;

		playAnimation(characterToPlay, animToPlay, true);
	end
end

function playAnimation(character, animId, forced)
	animName = animationsList[animId];
	--debugPrint(animName);
	if character == 'tail' then
		objectPlayAnimation('tail', animName, forced);
	end
end

The Note code that makes my character sing:
function onCreate()
	for i = 0, getProperty('unspawnNotes.length')-1 do
		if getPropertyFromGroup('unspawnNotes', i, 'noteType') == 'Special Sing' and not getPropertyFromGroup('unspawnNotes', i, 'mustPress') then
			setPropertyFromGroup('unspawnNotes', i, 'noAnimation', true)
		end
	end
end

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x86/x32
Did you edit anything in this build? If so, mention or summarize your changes.
No response

That script is extremely convoluted for no reason, you don't need any of those custom functions
animationsList = {"singLEFT", "singDOWN", "singUP", "singRIGHT"}

function opponentNoteHit(id, dir, type, sus)
     if type == 'Special Sing' or type == 'invisible Special Sing' then
          objectPlayAnimation('tail', animationsList[dir + 1], true)
     end
end

If that doesn't work, make sure your XML animation names match up


Constant RZ10012 errors in vscode when using components from library

It is virtually impossible to use vscode for a Blazor project where components are declared in a library. The editor highlights components with error RZ10012.
Attached a simple project to reproduce this. When opening Counter.razor in vscode it highlights both components declared in the project itself and in the library as unknown. It also complains about the namespaces for these. On opening it cycles between showing it correctly and then settles in a condition where it shows it as errors with RZ10012.
btest1.zip

is https://github.com/dotnet/vscode-csharp/issues/7625 related?

This is fixed by https://github.com/dotnet/razor/pull/10964 and should be in the next pre-release of the extension


How to create a collection with Analytical Storage enabled?

How to create a mongodb collection with Analytical Storage enabled. In the Azure portal UI it´s possible but no setting in the Azure CLI. Only TTL.
/Mattias

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

ID: a5323936-60e0-75a1-9f91-ea908249897d
Version Independent ID: e3c52896-95e5-1fea-fc8a-ab7170e95333
Content: az cosmosdb mongodb collection
Content Source: latest/docs-ref-autogen/cosmosdb/mongodb/collection.yml
Service: cosmos-db
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to service team

This support has not been added to CLI yet. It is on our backlog.

This capability is available now.
https://learn.microsoft.com/azure/cosmos-db/configure-synapse-link#azure-cli
Unless any other questions/issues, should be able to close this.


model compare to allow for varying position ordering

The model.compare() method needs to allow models that have different position orders to be treated as equal
so a polygon with positions ps1, ps2, ps3 should evaluate to being the same as a polygon with positions ps2, ps3, ps1
this option cab be a boolean switch for compare

done, needs testing


Error when creating and deploying a single HoneyToken to the Keyvault

I am using the HoneyTokens solution for the Keyvaults and I created the secret in the keyvault. I am at the step where I need to add monitoring in the SOC and I keep getting this error:
{"error":{"code":"NoResponse","message":"The server did not receive a response from an upstream server.

This issue seems to be the same as this:
https://github.com/Azure/Azure-Sentinel/issues/7731
Unfortunately, the solution was not posted in the thread.

Hi @plim2282, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 15-12-2023. Thanks!

Hi @plim2282 , Could you please check the in Logic apps HoneyTokensAdd2WLKV is properly configured with service connection
if needed we can have a call for further troubleshooting ,Please share email and convenient slots will have a teams meeting

I have time for a call on weekdays 10am to 5pm eastern time. My email is<EMAIL_ADDRESS>

@plim2282 ,Will schedule a call for this issue based on time slots suggested and let you know, Thanks.

Hi @plim2282  Blocked time for tomorrow, please join the meeting.

Great thank you @v-muuppugund

@plim2282 ,as disucssed over teams call,configured the logic app with appropriate params,please let me know if any issues.

@v-muuppugund After our call, I noticed that the api connector disappeared in the logic app

@v-muuppugund After our call, I noticed that the api connector disappeared in the logic app

@plim2282 ,as we are unable to create new connection in logic due to permissions, told will get back and the remaining logic app configurations are done, we are having connection need to updated, Please let me know if any issues, we can connect on the same teams meeting.

Hi @plim2282 we are closing your issue (https://github.com/Azure/Azure-Sentinel/issues/9577) as resolution provided. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation!

@v-muuppugund Can we reopen this ticket? Still having issues creating the connection on the logic app.

Sure @plim2282 ,tried to reach over teams,please suggest time,we can connect,Thanks

Hi @plim2282 we are closing your issue (https://github.com/Azure/Azure-Sentinel/issues/9577) as resolution provided. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation!


TEA Discovery: Should we be opinionated on priority of different discovery mechanisms?

For discussion.
The current in-draft Discovery docs cater for TWO mechanisms to discovery the TEA API for a given TEI:

Discovery using DNS (subject to #64 - will be SVCB records)
Discovery using Host (use of AAAA or A records) to get to a WebServer at the Host so that /.well-known/tei can be accessed - this would likely produce a HTTP Redirect (301 or 302 - TBC which are permitted within TEI specification) to the TEA API

Additionally, @ppkarwasz has suggested (in https://github.com/CycloneDX/transparency-exchange-api/issues/30):

Discovery using Host (use of AAAA or A records) to get to a WebServer at the Host so that /.well-known/security.txt can be accessed, containing a new (yet to proposed and registered) field that provides the URL to the TEA API

When considering implementation of the TEA Specification - I would suggest it is prudent for the Specification to be opinionated on which method(s) have priority along with reasons.
FYI @oej

I think like this:
For TEI discovery:
DNS is the best way to(provides failover etc). If that's not allowed, #2 will be used.
The security.txt case is very different as it a way to say "by the way, we have a TEA service". For that to work, there needs to be a way to get product identifiers. It requires further thinking.

https://github.com/CycloneDX/transparency-exchange-api/pull/72


Add the Minimum Platform Specification (Bugzilla Bug 1798)

This issue was created automatically with bugzilla2github
Bugzilla Bug 1798
Date: 2019-05-12T20:54:17+00:00
From: @makubacki
To:   @makubacki
CC:   @mdkinney, yonghong.zhu
Last updated: 2019-05-19T18:17:27+00:00

Comment 7600
Date: 2019-05-16 20:34:57 +0000
From: Yonghong Zhu <<yonghong.zhu>>
michael will work on it.

Comment 7700
Date: 2019-05-19 18:17:27 +0000
From: @makubacki
The specification is published in the following locations:
HTML:    https://edk2-docs.gitbooks.io/edk-ii-minimum-platform-specification/content/
PDF:     https://legacy.gitbook.com/download/pdf/book/edk2-docs/edk-ii-minimum-platform-specification
Mobi:    https://legacy.gitbook.com/download/mobi/book/edk2-docs/edk-ii-minimum-platform-specification
ePub:    https://legacy.gitbook.com/download/epub/book/edk2-docs/edk-ii-minimum-platform-specification
Gitbook: https://legacy.gitbook.com/book/edk2-docs/edk-ii-minimum-platform-specification/details
GitHub:  https://github.com/tianocore-docs/edk2-MinimumPlatformSpecification
In addition, the specification has been add the Bugzilla. Closing this BZ.

Comment 7495
Date: 2019-05-12 20:54:17 +0000
From: @makubacki

Industry Specification: ---
Releases to Fix: EDK II Master
Target OS: ---
Bugzilla Assignee(s): @makubacki

A new specification is proposed for adoption by the Tianocore community
called the "Minimum Platform Specification". This specification describes
the architecture and requirements for the Minimum Platform initiative which
will guide Intel platform contributions in the community. The purpose of
contributing this documentation is to encourage collaboration with the community
and to provide easily accessible and updated documentation for platforms conforming
to the specification by using the Tianocore-Docs contribution process.
Most notably definitions for the following key concepts are introduced:

Advanced features
Board abstraction
Minimum platform

The concepts are currently realized in implementation in the following locations:

AdvancedFeaturePkg
Product generation specific board packages (e.g. KabylakeOpenBoardPkg)
MinPlatformPkg

The specification is in a "draft state". Actions items will be tracked via
Bugzilla and of course all feedback and contributions are welcome.


Webpack 4 upgrade, bundleapp.js is loaded but it didnt run

Webpack_config_js.txt
Package_Json.txt
index_html.txt

Wepack V4.41
Node V10.15
OS win10

Please use issue template

Close in favor https://github.com/webpack/webpack/issues/10047


People Searching a User with No Followers Give CurrentUsers' Followers and Following

The user with handle "@fox" has no followers.
Issues also arise when current user has no followers.
Issues stems from ProfileViewController:
// Accounts for case where we select user from feed image
if user.followersCount == 0 {
API.sharedAPI.fetchUser(user.id) {
self.user = $0
self.followingButton.setTitle("($0.followingCount) Following", forState: .Normal)
self.followersButton.setTitle("($0.followersCount) Followers", forState: .Normal)
}
}

/users/userID Always gives back the current user no matter the what the userID happens to be. @ifilonenko can you look into this?

Closed with ice fishing-web commit


Cannot use self signed server

Hello Siesta Team!
I need to connect to a test server for my app, which has a self signed certificate. But I keep getting an error, saying "The certificate for this server is invalid"  etc. etc.
How can I configure my client to ignore untrusted servers? I even setted my info.plist in order to accept everything but it still doesn't work

Siesta doesn't deal with certificate validation at all. This is something that the underlying network layer handles. That mean you’ll need to debug the problem at that level: either URLSession (aka NSURLSession, the Siesta default) or Alamofire if you’re configuring Siesta to use it instead. This Stack Overflow question might be helpful, for example.
Once you have a URLSessionConfiguration that properly ignore the self-signed certificates, you can pass it as the networking: parameter when you create your Siesta Service.
P.S. You might gently suggest to your test team that they use Let’s Encrypt instead of a self-signed cert. Less danger of “insecure by accident” that way!

I'm not sure if you can use letsencrypt for local domains.
Anyway, this is how I resolved the issue for Alamofire:
let sessionManager: SessionManager = {
  let serverTrustPolicies: [String: ServerTrustPolicy] = [
      "mydomain.local": .disableEvaluation,
      "<IP_ADDRESS>": .disableEvaluation
  ]

  let configuration = URLSessionConfiguration.default
  configuration.httpAdditionalHeaders = SessionManager.defaultHTTPHeaders

  return SessionManager(
      configuration: configuration,
      serverTrustPolicyManager: ServerTrustPolicyManager(policies: serverTrustPolicies)
  )
}()

In your Service:
class ApiService: Service {
// ...
  init() {
    super.init(
      baseURL: "http://yourbaseurl", networking: sessionManager
    )
    // ...
  }

That's very nice! You're right, I doubt that Let's Encrypt handles local domains — but limiting the server trust to specific hosts as you're doing reduces the risk of it causing a vulnerability. Thanks for sharing your nice code.

Unfortunately, the test server of the company I work for is "untouchable", because life is hard and then you die...
Anyway, thank everybody for the help. I'll try the solutions you suggested.

I resolved this way:
I created this class
class UntrustedCertificatesDelegate: NSObject, URLSessionDelegate{     func urlSession(_ session: URLSession, didReceive challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void) {         if challenge.protectionSpace.authenticationMethod == NSURLAuthenticationMethodServerTrust{             let credential = URLCredential(trust: challenge.protectionSpace.serverTrust!)             completionHandler(.useCredential, credential)         }     } }
and then used it like this:
let delegate = UntrustedCertificatesDelegate() let ns = URLSession(configuration: conf, delegate: delegate, delegateQueue: nil)         super.init(baseURL: testUrl, networking: ns)


fix: code owners and issues do not work with project slug

🚨 Proposed changes
This is a fix, now code owners works with project slug
⚙ Types of changes
What types of changes does your code introduce? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[x] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Refactor

:tada: This PR is included in version 2.1.1-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Switched to a listener interface instead of EncoderTo

This PR removes the EncoderTo interface, which was just a bunch of problems waiting to happen.

Codecov Report

Merging #122 into master will decrease coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
- Coverage    84.2%   84.16%   -0.05%
==========================================
  Files          81       81
  Lines        4596     4597       +1
==========================================
- Hits         3870     3869       -1
- Misses        627      629       +2
  Partials       99       99

Impacted Files
Coverage Δ

wrp/format.go
100% <100%> (ø)
:arrow_up:

wrp/messages.go
98.42% <100%> (ø)
:arrow_up:

device/device.go
82.5% <0%> (-1.67%)
: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 c0d97ab...a493f74. Read the comment docs.


version release notes?

where's the best place to find version release notes / a what's new?

Is that link enough for what you are looking for ?
https://github.com/mongodb/morphia/releases

oh nice...maybe put that in the docs?

done.  waiting for the release for final tweaks.


[WIP] ARROW-1456: [Python] Run s3fs unit tests in Travis CI

This is to enable Python S3 tests in CI with minio as requested in ARROW-1456 and ARROW-4208.

@ursabot build

@ursabot build

@kszucs, I'd like to add minio binary and python s3fs package to the docker image. This would be to enable S3 CI tests. Do I need to do something besides this PR?

Codecov Report

Merging #5200 into master will increase coverage by 0.03%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##           master    #5200      +/-   ##
==========================================
+ Coverage   88.63%   88.66%   +0.03%
==========================================
  Files         948      948
  Lines      126016   126043      +27
  Branches     1447     1447
==========================================
+ Hits       111692   111757      +65
+ Misses      13960    13922      -38
  Partials      364      364

Impacted Files
Coverage Δ

python/pyarrow/tests/test_parquet.py
97.13% <100%> (+1.04%)
:arrow_up:

python/pyarrow/tests/conftest.py
70.83% <66.66%> (-0.22%)
:arrow_down:

cpp/src/arrow/json/converter.cc
90.05% <0%> (-1.76%)
:arrow_down:

cpp/src/arrow/json/chunked_builder.cc
79.91% <0%> (-1.68%)
:arrow_down:

cpp/src/arrow/util/thread_pool_test.cc
97.66% <0%> (-0.94%)
:arrow_down:

python/pyarrow/parquet.py
92.52% <0%> (+0.17%)
:arrow_up:

cpp/src/arrow/csv/column_builder.cc
97.02% <0%> (+1.48%)
:arrow_up:

python/pyarrow/filesystem.py
66.66% <0%> (+14.35%)
: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 2f55ad3...59f1e14. Read the comment docs.

ARROW-6655 seems to do this and more. @kszucs shall I close this PR once ARROW-6655 is merged?

@rok yes, we should close it in favor of ARROW-6655.
It introduces Minio for the filesystem bindings, and I've also updated the s3fs accordingly.


[DOCFIX]Modify some wrong document descriptions

What changes are proposed in this pull request?
Some wrong descriptions were found in Journal.md and User-CLI.md, the purpose of this pr is to fix them.
Why are the changes needed?
Yes, some wrong descriptions were found here.
Does this PR introduce any user facing changes?
After fixing them, it will be more convenient for users to use.

Can you help review this pr, @tcrain  @jiacheliu3 .
Thank you very much.

Can you help review this pr, @HelloHorizon   @jenoudet .
Thank you very much.

Many thanks to @jenoudet  for the review.

alluxio-bot, merge this please


add metrics advisor to reference index

Updating reference docs index for new service - metrics advisor

Docs Build status updates of commit 7df9278:
:clock10: Full build: the average full build time is 190 min(s) 8 sec(s), this is based on the last 646 build(s) for this repository.

Docs Build status updates of commit 7df9278:
:warning: Validation status: warnings

File
Status
Preview URL
Details

preview/docs-ref-autogen/azure-functions-durable/azure.durable_functions.tasks.call_http.yml
:warning:Warning
View (azure-python-preview)
Details

docs-ref-mapping/reference-unified.yml
:white_check_mark:Succeeded

preview/docs-ref-autogen/azure-functions-durable/azure.durable_functions.tasks.call_http.yml

[Warning-DuplicateUids] Uid(azure.durable_functions.tasks.call_http) has already been defined in preview/docs-ref-autogen/azure-functions-durable/azure.durable_functions.tasks.yml.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Which input options are allowed?

Hi,
I am wondering if something like debouncing on input is allowed.
In fact, i want to trigger the event listener of a searchbar when the user does not hit the keyboard for at least 300ms.
I have not found anything about it. Are such options configurable at the view or do I have to program it by myself?
Thank you in advance.

@tom94zoe you can write the logic yourself using either the Angular lifehooks or the NativeScript lifecycles.
For example, you can take the reference to your search bar when it is loaded (e.g. using loaded event on the search bar) and then use the timer module to call the method when needed.

Thank you

@tom94zoe how did you solve this?


Feature Request：Lost focus to save file.

Spyder version: 5.5.0
Can you add a feature: lose focus to save files,always ctrl+s, a little tired.
Like vscode:

Hey @wilikeit, thanks for the suggestion. Unfortunately, we don't have time to implement it ourselves because we have a lot work to do in other areas.
But anyone is welcome to tackle it and submit a pull request to make it a reality.


Update AutoAssignArgs to support a list of pools

This addresses the first bullet of https://github.com/projectcalico/calico-cni/issues/212
I'm assuming we need to still support the old variant (single pool) of AutoAssignArgs too, otherwise we'd break existing users.  Should we deprecate the single flavor though or just keep both going forward?
/cc @lxpollitt @mikespreitzer
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

 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.

@robbrockbank assigning you as a joint reviewer.
@duglin my take would be that we don't need to continue supporting the old IPv4Pool field, since we're still pre v1.0.0.

@caseydavenport
Can easily remove - just want to make sure no one objects over the day or so....

@duglin Thanks for this!
Looking pretty good to me. Stuck a few comments on.
Biggest one is to make sure nobody objects to getting rid of the IPv4Pool / IPv6Pool args and then do that.

Ok. I updated the first commit with the minor syntax changes.
I also pushed another commit with the removing of the old IPv*Pool param and got rid of the pointers.
Once the maintainers decide if backwards compat matters or not I can either squash the 2 commits or delete the 2nd one.

@duglin  -  LGTM.
I agree with @caseydavenport with the removal the IPv4Pool and IPv6Pool options - since the API was only Beta at this stage.

ok I squashed the two commits down to just one since there's agreement on removing the old singleton IPv*pool args.
@robbrockbank re: other repo's impact ... I'm working on the 2nd(CNI) bullet of https://github.com/projectcalico/calico-cni/issues/212 and am running into testing issues... will ask on slack or in that repo.

LGTM
@duglin - we'll be looking out for the CNI update.   Let me know if you plan i updating the libnetwork-plugin too (otherwise we'll look into that one).

yes I'm working on the libnetwork-plugin one right now and I may have found a bug in this PRs code but still investigating.... keep tuned...

hmm never mind - now that this PR is in master I think my libnetwork-plugin PR is working.


CNDB-8229: Allow dropping non-frozen UDT columns

Allow dropping non-frozen/multi-cell UDTs. This is done by adding partial support for non-frozen/multi-cell tuples, so dropped UDT columns are transformed into an equivalent tuple type. The support for non-frozen tuples is not user-facing, so users can't create non-frozen tuples. At the CQL level all tuples are considered frozen, so tuple<> and frozen<tuple> are still considered equivalent.
This PR is based on the PR for CNDB-7789, which is still to be reviewed.

Just rebased to include the recently merged CNDB-7789.

@blerer CI complained about low test coverage so I have added some new testing. I have found a couple of blocks of dead code in the ported tickets. These are:

Printing live multicell tuples in sstabledump. We only allow those as dropped columns.
Check the order of columns in SerializationHeader. It could happen in the original ported patch, but with the current codebase they should be already ordered because of the guarantees provided by Columns.

CI looks clean know, so if you are ok with these changes I will merge.


Every new projects obtains automatically internall UUID code...

...that would work merely like an authentication token.
Motivated by: https://github.com/mastodon-sc/mastodon-collaborative/issues/16
Use cases:

If I share a .mastodon file with someone and get it back edited, I can check if it is the original project.
If client-server collaborating env is established, clients can be warned when talking to a server hosting data of a different project
Mastodon plugins can benefit from maps: UUID -> {set of various settings of that plugin} (like here), e.g. prefilled GUI params per project (e.g., here)

Hello @xulman
This issue relates to an enhancement, but there has been no work on it since quite some time. Are you ok to close it for now?

This may be relevant for https://github.com/mastodon-sc/mastodon-tomancak/issues/15
Thus at least hereby I would like to make the link to that other issue.

I would put this on the roadmap of reworking app model, so that the new (forgot the class name, so making up one now) MastodonProject main class would offer two methods

getProjectID() - firm ID, immutable, created random when a project is saved for the first time (created)
getProjectHash() - reflects on the current content of the project, can be used to detect if something has changed (e.g. after running some other plugins), this method would merely collect hash() from "important data structures"

I suggest to defer this until the renewal of the collaborative project.


Spoof the Remote Program string in an Oracle database connection

Hello everyone.
I've been using DBeaver CE merrily for some time now, and I think it is a very apt program for my needs.
However, I've found myself in a weird situation.
One of my Oracle production databases has a restriction in which (for security reasons) it only allows to receive connections from a particular software string. If it doesn't match, connection is refused.
Looking up the Session List on the development database and checking out the Process information, Remote Host, User and Program are displayed.
For DBeaver, it shows as it is "DBeaver 3.8.2"; SQL Developer shows as "SQL Developer", meanwhile for our software the proper software string is displayed.
I was wondering if there is anyway to spoof that string, so that I'm able to mask my connection as if it was from our software?
Thanks again for your work with this software.

This is reasonable. And I think this won't take much time to implement.

Great. Can't wait to get this and finally ditch SQL Developer.


Consider building an EF Core provider for System.Data.SqlClient

We're moving the default EF Core SQL Server provider to use Microsoft.Data.SqlClient--see #15636.
However, if there is demand, we could consider building the provider against System.Data.SqlClient and shipping this as a legacy package--similar to the CE 3.5 package we used to have on EF6.
The places in the code where we hard-reference SqlClient are minimal and could be made dynamic to make supporting both easier, or we could just use ifdefs.
Note: we should only do this if we get significant demand for it, and only then after analyzing why those customers can't use the new package.

Closing as this is no longer something we intend to implement.


AUI-2069 Unexpected visual behavior after removing a field

Just started reviewing :) :octocat: Sent from GH.

Thanks, pull request merged.


MS Teams Helper taking Too Much Memory

Hi Team
The MS Teams Helper process is taking too much memory whenever we use Teams . Please can you suggest the reason for it's usage.

I suggest you try to clear the caches of Teams and see if there is any improvement:
Go to  %appdata%\Microsoft\teams Then delete the following files. As a side note, you can move the files to another place to avoid the data loss.
a. In Application Cache > Cache folder, delete all files in it.
b. In Blob_storage folder, delete all files.
c. In Cache folder, delete all files.
d. In databases folder, delete all files.
e. In GPUCache folder, delete all files.
f. In IndexedDB folder, delete the .db file.
g. In Local Storage folder, delete all files.
h. In tmp folder, delete all files.
If you still see the same issue, we suggest you to vote for the feedback below via the following link.
https://microsoftteams.uservoice.com/forums/555103-public/suggestions/17219609-teams-uses-600mb-and-is-super-slow

@Abhijit-MSFT  This is happening on Mac.

@onlyabhi52 - Please vote your feedback to the following link
https://microsoftteams.uservoice.com/forums/555103-public/suggestions/17219609-teams-uses-600mb-and-is-super-slow

@Abhijit-MSFT  i have done that . Please can you suggest how much is eta for this resolution.

@onlyabhi52, Can not provide eta for this.


Getting error String

Hi,
First of all, thanks for this great library! It's really helpful!
I am trying to validate an email using the following code:
let emailRule = ValidationRulePattern(pattern: EmailValidationPattern.standard, error: ValidationError(message: "wrong type of email") )

let emailValidationResults = emailValue.validate(rule: emailRule)

switch emailValidationResults {
case .valid:
    print("everything checks out")
case .invalid(let someError):
    print(someError.first?.message)
}

The ValidationError is as below:
struct ValidationError: Error {

    public let message: String

    public init(message m: String) {
        message = m
    }
}

There are 2 issues:

If i run the code above the XCode will throw an error saying:
"Value of type 'Error' has no member 'message'"

If I just use print(someError.first ?? "") then i get this:
ValidationError(message: "wrong type of email")

How do i just get the string "wrong type of email" instead?
Also, why isnt the way i discribed first (someError.first?.message) not working? This is what you say in your documentation as well.
Thank you in advance!

Hi @tsangaris.
Thanks for the well-formed question.
Looks like the documentation is out-dated! Thank you for pointing that out.
I think in previous versions the ValidationResult.invalid([Error]) was actually ValidationResult.invalid([ValidationError]).
Here I have 2 fixes in mind for you:

Cast to ValidationError in the switch:

switch emailValidationResults {
case .valid:
    print("everything checks out")
case .invalid(let someError as [ValidationError]):
    print(someError.first?.message)
}

Or, extend error to have the message property:
extension Error {

    var message: String {
        return localizedDescription
    }
}

So it will use your custom message if it's a ValidationError, or fall back to localizedDescription on anything else.
To be honest, I don't like the ValidationError type, and will probably remove it in the next version, which I plan to write very soon.
Thank you!

(Re-opening because I need to update the docs)

Thanks a lot for your response, Adam!
Looking forward to checking the next version. And thanks for helping us not re-inventing the wheel all over again! :)

Updated the README now 👍


Refactor server into independent isolated runtimes to reduce complexity for go->rust port

In order to enable us to make the porting of the core blockchain server into rust, the idea is to break the monolithic structure into purpose specific modules that run as worker processes for a controlling shell process.
This is a task that is easily decomposed into a series of small tasks, and I will add each task in sub-issues to this epic

For this task a new repository is being made at https://github.com/dusk-network/dusk-ipc as dusk-protobuf needs work to be brought up to date. I will isolate the implementation in this package. I think I need to make issue/PR on the new repo to link it in to there via the zendesk stuff.


Into to server inlets branch

Adds new servlet and uses json.

https://afeenster-step-2020.ue.r.appspot.com/
link to my website... This should work.


Added support for the cdt parameter for overriding the datetime

I added in the cdt parameter to the _get_request method to allow self.forced_datetime to be passed into PiWik.

Hi @kelfish - any chance you could write a little test for this one?

This functionality has been added in the refactor branch.


Reopening Request: Deployment branches missing/hidden since new Amplify UI

Environment information
System:
  OS: Linux 5.15 Ubuntu 20.04.6 LTS (Focal Fossa)
  CPU: (12) x64 12th Gen Intel(R) Core(TM) i5-1230U
  Memory: 4.23 GB / 7.76 GB
  Shell: /bin/bash
Binaries:
  Node: 18.13.0 - ~/.asdf/installs/nodejs/18.13.0/bin/node
  Yarn: undefined - undefined
  npm: 8.19.3 - ~/.asdf/plugins/nodejs/shims/npm
  pnpm: 9.2.0 - ~/.asdf/shims/pnpm
NPM Packages:
  @aws-amplify/backend: Not Found
  @aws-amplify/backend-cli: Not Found
  aws-amplify: Not Found
  aws-cdk: Not Found
  aws-cdk-lib: Not Found
  typescript: Not Found
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1

Description
Since the new Amplify UI, the branch (release/staging), which should be listed on the branch overview page at
ap-northeast-1.console.aws.amazon.com/amplify/apps/d24nx2m9l74e4j/overview
, is not displayed.
However, if we access the URL directly (amplify/apps/d24nx2m9l74e4j/branches/release%2Fstaging/deployments), the deployment screen is displayed and it's there.
We believe this is the same issue as the one described in the closed issue below, but it is still occurring in our environment.
Reference: Closed issue link: Deployment branches missing/hidden since new Amplify UI #1469
https://github.com/aws-amplify/amplify-backend/issues/1469
We would appreciate it if you could revisit this issue and provide a resolution.

Same here, I have to have 2 deployments, and it show only one, the most recent that I made like 2 days ago

@sorabito-takeharu-umezaki 👋 , thanks for reaching out. Can you share the following information:

Screenshot of the branch overview page
Is the branch listed under the App settings > Branch settings page?
The app d24nx2m9l74e4j has 3 branches connected to it, is the release/staging branch the only branch which isn't listed/displayed in the console?
Are you observing a similar behavior across different apps? If yes, can you share the impacted app ids?

@xManuel007 are you observing a similar behavior where one of the connected branches is not listed/displayed in the Amplify console? If yes, can you share your Amplify app id and also the additionally requested information?

I have two deployments for two different projects, but only one is displayed. I can switch between them by using the direct link of the project that I found in my history.

I realized that the projects are in different regions: one in us-east-2 and the other in us-east-1. However, about 1-2 weeks ago, both projects were in the same region and displayed on the same page.  when amplify was Gen 1

@xManuel007 can you share either of your app id's? Also, it is likely that your app could have been deleted from one of the regions and recreated in another region by a member of your team or org. To verify it, you can review the CreateApp and DeleteApp API events in CloudTrail.

@Jay2113
Screenshot of the branch overview page

Is the branch listed under the App settings > Branch settings page?
Yes it is.

The app d24nx2m9l74e4j has 3 branches connected to it, is the release/staging branch the only branch which isn't listed/displayed in the console?
Yes. release/staging only.
Are you observing a similar behavior across different apps? If yes, can you share the impacted app ids?
No. App d24nx2m9l74e4j only.

Thanks for sharing the screenshots @sorabito-takeharu-umezaki. We are investigating into it.

Hi @sorabito-takeharu-umezaki, thanks for your continued cooperation. To expedite the troubleshooting process, we would like to setup a call with you. Can you share your availability via a DM on our community discord server?. My username is Jay Raval.

as you guide in discord,
it's resolved by changing stage of the branch.
aws amplify update-branch --region ${REGION} --app-id ${APP_ID} --branch-name ${BRANCH} --stage "DEVELOPMENT"

Problem-Cause: We had two branches which have same stage.
thank you.


Obsidian 1.x: causes the entire page to shift up sometimes

When you switch tabs, some notes and cursor positions make the entire page shift up. So the top is cut off so you can't even see the tabs anymore:

And you end up with this weird gray area at the bottom:

I think it's because this plugin causes an abnormal scroll event on the wrong div. This didn't happen before Obsidian 1.x. So it must have to do with the new tabbed layout or something.
Only some tabs get displayed like that (I can't seem to figure out the pattern of documents and cursor positions yet).

Never mind, this is happening even after I disabled this extension. Sorry about that.


Feature\Checkpoint-keyboard-controls

Hello there,
I have recently stumbled upon A-Frame and the extras, I like what I see and I think the potential is rather exciting.
I have been pondering combining the checkpoints and keyboard controls.
This would work in the following way:

Keys would mapped via properties to control progression to the next/previous checkpoints (default right and left arrow keys).

We would have to keep track of the current checkpoint.

The thing I'm thinking about is movement similar to slideshow-component.
So you could go from scene to scene in a slideshow kinda way.
I have also been thinking that a timer could also be added, so the camera could automatically move through the checkpoints and spend a certain interval at each scene.
So, taken your checkpoints example, the markup would be: (Where the time here is minutes)
      <!-- Checkpoints -->
      <a-entity position="1 0 1">
        <a-cylinder checkpoint="interval:3" radius="1" height="0.1" position="0 0 -5.2" color="#39BB82"></a-cylinder>
        <a-cylinder checkpoint="interval:5" radius="1" height="0.1" position="3 0 0" color="#39BB82"></a-cylinder>
        <a-cylinder checkpoint="interval:4" radius="1" height="0.1" position="-3 0 0" color="#39BB82"></a-cylinder>
      </a-entity>

This would mean that the camera spends 3 minutes on the first checkpoint, then moves to the next, spends 5 minutes here, move to next, spends 4 minutes here.... then optionally repeats.
I think arrow keys could still be useful here, allowing the user to go back.
What do you think of this idea?
I'm afraid that I might not be able to contribute to the code itself as I might not have the time, but I've contributed the idea at least ;-)

Hi! You can set this up in your application's code by putting IDs on the checkpoints and doing something like...
var el1 = document.querySelector('#checkpoint1');
var el2 = document.querySelector('#checkpoint1');
setTimeout(() => el1.components.checkpoint.fire(), 30 * 1000);
setTimeout(() => el2.components.checkpoint.fire(), 60 * 1000);
...

Since it's not too complex to build into the app, I don't think it makes sense to put into aframe-extras directly, but hopefully that gets you started!


Rename Server attribute and Fixed layout

Description
Synthesis Layout fixed
Related issues

Fixes #3665
Screenshots

Thank you! 🎉

Thank you! 🎉


"RuntimeError: cuDNN error: CUDNN_STATUS_NOT_SUPPORTED. This error may appear if you passed in a non-contiguous input."

Hi, thank you for sharing the great work!
I met the error of below when I run the script below.
python test.py --cfg_file cfgs/det_model_cfgs/centerpoint_1sweep_my.yaml \
--ckpt /SHFP12/xiaoquan.wang/01_bev/DetZero/detection/output/det_model_cfgs/centerpoint_1sweep_my/default/ckpt/checkpoint_epoch_30.pth \
--extra_tag centerpoint_1sweep_my_tta \
--set DATA_CONFIG.TTA True

The error is below.
023-09-26 11:50:50,402   INFO  *************** EPOCH 30 EVALUATION *****************
eval:   0%|                                                                             | 0/1250 [00:00<?, ?it/s]Traceback (most recent call last):
  File "test.py", line 205, in <module>
    main()
  File "test.py", line 201, in main
    eval_single_ckpt(model, test_loader, args, eval_output_dir, logger, epoch_id, dist_test=dist_test)
  File "test.py", line 66, in eval_single_ckpt
    eval_utils.eval_one_epoch(
  File "/defaultShare/SHFP12/xiaoquan.wang/01_bev/DetZero/detection/tools/eval_utils.py", line 73, in eval_one_epoch
    result = model(batch_dict)
  File "/opt/conda/envs/detzero/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/defaultShare/SHFP12/xiaoquan.wang/01_bev/DetZero/detection/detzero_det/models/centerpoint.py", line 29, in forward
    batch_dict = cur_module(batch_dict)
  File "/opt/conda/envs/detzero/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/defaultShare/SHFP12/xiaoquan.wang/01_bev/DetZero/detection/detzero_det/models/centerpoint_modules/backbone2d.py", line 101, in forward
    x = self.blocks[i](x)
  File "/opt/conda/envs/detzero/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/opt/conda/envs/detzero/lib/python3.8/site-packages/torch/nn/modules/container.py", line 141, in forward
    input = module(input)
  File "/opt/conda/envs/detzero/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/opt/conda/envs/detzero/lib/python3.8/site-packages/torch/nn/modules/batchnorm.py", line 168, in forward
    return F.batch_norm(
  File "/opt/conda/envs/detzero/lib/python3.8/site-packages/torch/nn/functional.py", line 2282, in batch_norm
    return torch.batch_norm(
RuntimeError: cuDNN error: CUDNN_STATUS_NOT_SUPPORTED. This error may appear if you passed in a non-contiguous input.

But when I remove the  --set DATA_CONFIG.TTA True  , it run successfully.

I met same error in refining and found solutions in CSDN.
https://blog.csdn.net/jiaoyangwm/article/details/135845029
It says that your batch_size is too large.


Schema always sorted

Describe the bug
Schema is always sorted regardless of the sort config.
To Reproduce
Steps to reproduce the behavior:
https://codesandbox.io/s/gql-codegen-sort-5my7k
Note that the config does not set config: true, but the generated is still sorted.

My GraphQL schema:

type Query {
    user(id: ID!): User!
}

type User {
    id: ID!
    username: String!
    email: String!
}

My GraphQL operations:

query user {
    user(id: 1) {
        id
        username
        email
    }
}

My codegen.yml config file:

schema: schema.graphql
documents: document.graphql
generates:
  types.ts:
    plugins:
      - typescript
      - typescript-operations

Expected behavior
I expect that the output won't be sorted because sort is not true in the config.
Environment:

OS: Codesandbox
@graphql-codegen/add: @2.0.2
@graphql-codegen/cli: @2.2.0
@graphql-codegen/typescript: @1.21.0
@graphql-codegen/typescript-operations: @1.17.14
NodeJS: v14.17.6

What difference does it make if it's sorted or not? What's the impact on you?

@kamilkisiela None, just that it's an undocumented change. I don't have a problem with this behavior being the new default as long as it's documented as such.

@kamilkisiela I'm seeing this same issue, specifically with enums. Our enums are written with a specific order and having them be automatically alphabetized changes our UI.

@canac I resolved this problem by reverting these packages to the versions below:
"@graphql-codegen/cli": "1.21.8",
"@graphql-codegen/typescript-operations": "1.18.4",

This change has been done to give more consistent results. I don't think sorting is an issue, bug or a breaking change.
It's been documented here;
https://github.com/dotansimha/graphql-code-generator/blob/master/packages/graphql-codegen-cli/CHANGELOG.md#minor-changes
You can disable it with sort: false. Closing this issue.


hd44780: need assistance with Unicode to font table mappings

The new hd44780 driver contains embedded fonts.  The data for these fonts are contained within the hd44780 class in luma.lcd.device.const.py in a dict called FONTDATA.  One key in FONTDATA is 'mappings' which is a list of two dicts (one for each of the two fonts that are typical for hd44780 style devices).  Each dict contains a collection of key value pairs which provide unicode values for as many of the table values as I could decode.  I'm looking for help to fill in the missing values and validate that I've correctly assigned the ones that are there.  I'd also be interested in any look-alike values that should be added (e.g. unicode codepoints that look the same as a mapping that already exists.  A good example of that would be the series of cyrillic mappings I've included for their english equivalents.
Here's a small example of what the tables looks like...

Not sure if this belongs here:
If I try to replace German Umlauts as described in the docs:
fnt = device.get_font('A02')
device.font.combine(fnt, ['Ä','Ö','Ü'])
device.text = 'ÄÖÜ'

Then the displayed result is surely not as expected. E.g. the Ä has a double horizontal bar in the middle and an additional horizontal bar at the bottom.


refactor chroot to add more clarity and simplicity

moved chroot commands in in its own file
added pseudo filesystems to get rid of warnings/errors

@DieterReuter Can you please have look?

@Govinda-Fichtner LGTM! Just have seen that we want to run the tests in a more visible way, but that can be done with another PR.

@StefanScherer I can make another PR for the tests... OK?


Powertoys Awake unknown software exception during Windows Update Install Restart

Microsoft PowerToys version
"0.41.3"
Running as admin

[ ] Yes

Area(s) with issue?
Awake
Steps to reproduce

PowerToysReport_2021-07-15-07-38-18.zip
I don't really know how to reproduce the problem, because I never expected it to happen! All I can say was the error occurred when attempting to restart the system after installing Windows Update 2021-07 Cumulative Update for Windows 10 Version 21H1 for x64-based Systems (KB5004237)
✔ Expected Behavior
No response
❌ Actual Behavior
As the attached screenshot shows, PowerToys.Awake.exe produced an unknown exception (0xe0434352)
Other Software
No response

Just saw the same thing attempting to reboot for update. Though it's also my first time trying to reboot with awake active. V0.45.0, not as admin
---------------------------
PowerToys.Awake.exe - Application Error
---------------------------
The exception unknown software exception (0xe0434352) occurred in the application at location 0x00007FF9CC2E4ED9.

---------------------------
OK
---------------------------

Following-up on this issue - @judfs @Dzomlija have you seen the problem since the original report? This seems to be a catch-all CLR error, so trying to see if it was intermittent.

Following-up on this issue - @judfs @Dzomlija have you seen the problem since the original report? This seems to be a catch-all CLR error, so trying to see if it was intermittent.

No, @dend, I have not seen this issue again. I naturally assumed it had been resolved with updates....

Thanks for confirming! I will close the issue for now and if it appears again, feel free to open a new report.


加载模型时卡住了

日志如下：
model_worker_391073.log
现象是worker不再输出日志，gradio web模型的地方也没加载出来。Ctrl+c或者kill都无法结束进程

Hello, thanks for your interest.
It's weird, according to the log, looks like you have not loaded the whole Mini-Gemini model.
I guess it's some memory or cpu issue? Maybe you can use kill -9 to delete the process.

Hello, thanks for your interest. It's weird, according to the log, looks like you have not loaded the whole Mini-Gemini model. I guess it's some memory or cpu issue?


Update Readme generated template to default kite Readme

OPEX-6

Codecov Report

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

@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   40.44%   40.44%
=======================================
  Files          12       12
  Lines         272      272
=======================================
  Hits          110      110
  Misses        162      162

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 6c04eee...af7c5e4. Read the comment docs.


feat(check-net): implement network interface metrics

:tada: This PR is included in version 1.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Install mazerunner artefact locally

Installs the plugin as a maven artefact locally, with a high version number to avoid the potential for it to be accidentally picked up from JCenter instead. The local maven repo is also cleared on each run, so the scenarios should fail if the plugin cannot build for any reason.

Addressed review comments by increasing plugin version + deleting rather than moving


Not working, no error

Basically used verbatim from the docs, it won't touch s3 at all. No errors when we change the secret. Where might we look to diagnose this?

The same is here...
The tests are failed too with the following error:
when some big files start coming in, this adapter should work properly with 2 simultaneous requests with decent-sized (1.5MB) files:
     Error: the string "Unable to initiate stream upload [Invalid upload ID" was thrown, throw an Error :)
      at Runner.fail (/Users/festo/Projects/Research/skipper-s3/node_modules/skipper-adapter-tests/node_modules/mocha/lib/runner.js:197:11)
      at Runner.uncaught (/Users/festo/Projects/Research/skipper-s3/node_modules/skipper-adapter-tests/node_modules/mocha/lib/runner.js:541:8)
      at process.uncaught (/Users/festo/Projects/Research/skipper-s3/node_modules/skipper-adapter-tests/node_modules/mocha/lib/runner.js:568:10)
      at process.emit (events.js:95:17)
      at process._fatalException (node.js:272:26)

I confirm this issue, setting the skipper-s3 adapter, skipper keep using skipper-disk adapter and files are stored locally...

+1.
Is this repository abandoned?

I figured it out, skipper-s3 doesn't work for buckets with location in Frankfurt
https://github.com/balderdashy/skipper-s3/issues/21

I'm getting this problem. Running on my local machine everything works fine, but on Heroku Skipper fails silently. It runs the 'saveAs' function, but no file ever appears on S3, and final callback with (err, data) doesn't run.
On the same Heroku app, a signed upload direct from the browser works fine.
On some occasions Heroku gives an H18 error when using Skipper.

Closing as basically a duplicate of #21 and #18.  Basically you need to use a bucket in a region that uses a signature supported by Knox.


IO board

There are few bugs in IO board design.


LDAP users with dots - notification does not work

Hello, we have a LDAP users, which do have to be escaped when used as a docker namespaces (e.g user f.s), which results in fs.
When I push new docker image to a registry (using docker login -u fs (not f.s - bummer - cannot this be fixed?)), the notification fails and no image can be seen in Portus until crono makes its job.
In the log, I can see:
User Load (0.6ms)  SELECT  `users`.* FROM `users` WHERE `users`.`ldap_name` = 'fs' LIMIT 1
Cannot find user fs

I am not sure, whether this error is related with the notification error, but might.

Theoretically, you can login with f.s. What won't be accepted is that your namespace is f.s. But, if I'm not wrong, you should be able to log in with f.s and push your image (from the top of my head, I'd have to check :P).

See in the documentation:

The real LDAP name is only used for authentication (both in Portus, and in the Docker CLI).

The real LDAP name is only used for authentication (both in Portus, and in the Docker CLI).

Ok I see, I can use my LDAP name in Docker CLI BUT only with LDAP password, not with token.
But still I encounter this error:
# my LDAP name is first.second
$ docker login -u firstsecond -p 9Dmabxjkma7nXHNZRBeK -e<EMAIL_ADDRESS>registry.dev
$ docker push registry.dev/my_image:stable
$ docker logs -f portus_web_1
...
Started GET "/v2/token?account=portus&scope=repository%3Amy_image%3Apull&service=registry.dev" for <IP_ADDRESS> at 2016-02-15 15:40:08 +0100
Cannot render console from <IP_ADDRESS>! Allowed networks: <IP_ADDRESS>, ::1, <IP_ADDRESS>/<IP_ADDRESS>
Processing by Api::V2::TokensController#show as JSON
  Parameters: {"account"=>"portus", "scope"=>"repository:my_image:pull", "service"=>"registry.dev"}
  User Load (0.6ms)  SELECT  `users`.* FROM `users` WHERE `users`.`username` = 'portus' LIMIT 1
  ApplicationToken Load (0.3ms)  SELECT `application_tokens`.* FROM `application_tokens` WHERE `application_tokens`.`user_id` = 1
  Registry Load (0.3ms)  SELECT  `registries`.* FROM `registries` WHERE `registries`.`hostname` = 'registry.dev' LIMIT 1
  Namespace Load (0.3ms)  SELECT  `namespaces`.* FROM `namespaces` WHERE `namespaces`.`registry_id` = 1 AND `namespaces`.`global` = 1 LIMIT 1
  User Load (0.4ms)  SELECT  `users`.* FROM `users` WHERE `users`.`username` = 'portus'  ORDER BY `users`.`id` ASC LIMIT 1
   (0.3ms)  BEGIN
  SQL (0.3ms)  UPDATE `users` SET `current_sign_in_at` = '2016-02-15 14:40:08', `sign_in_count` = 1261, `updated_at` = '2016-02-15 14:40:08' WHERE `users`.`id` = 1
   (0.9ms)  COMMIT
[jwt_token] [claim] {:iss=>"registry.dev", :sub=>"portus", :aud=>"registry.dev", :iat=>1455547208, :nbf=>1455547203, :exp=>1455547508, :jti=>"euwd1hLyvrtvqXKgjEBhVrAA8FtBZ76w4W7Cng8Goe", :access=>[{:type=>"repository", :name=>"my_image", :actions=>["pull"]}]}
Completed 200 OK in 110ms (Views: 0.2ms | ActiveRecord: 3.4ms)
[100] <IP_ADDRESS> - - [15/Feb/2016:15:40:08 +0100] "GET /v2/token?account=portus&scope=repository%3Amy_image%3Apull&service=registry.dev HTTP/1.0" 200 - 0.1164
  User Load (0.6ms)  SELECT  `users`.* FROM `users` WHERE `users`.`ldap_name` = 'firstsecond' LIMIT 1
Cannot find user firstsecond
Completed 202 Accepted in 316ms (ActiveRecord: 1.1ms)
[92] <IP_ADDRESS> - - [15/Feb/2016:15:40:08 +0100] "POST /v2/webhooks/events HTTP/1.0" 202 - 0.3225
...

Note the Cannot find user firstsecond (my LDAP name is first.second !) therefore the webhook event from registry does not pass no my_image cannot be seen in Portus web.
I am not sure, If i explain the situation correctly, feel free to ask more questions :)

Could you pass me the logs when you try to perform:
$ docker login -u first.second -p 9Dmabxjkma7nXHNZRBeK -e<EMAIL_ADDRESS>registry.dev

Note that I'm using the ldap name.

Started GET "/v2/token?account=first.second&service=registry.dev" for <IP_ADDRESS> at 2016-02-15 16:30:47 +0100
Cannot render console from <IP_ADDRESS>! Allowed networks: <IP_ADDRESS>, ::1, <IP_ADDRESS>/<IP_ADDRESS>
Processing by Api::V2::TokensController#show as JSON
  Parameters: {"account"=>"first.second", "service"=>"registry.dev"}
  User Load (0.6ms)  SELECT  `users`.* FROM `users` WHERE `users`.`username` = 'first.second' LIMIT 1
  Registry Load (0.3ms)  SELECT  `registries`.* FROM `registries` WHERE `registries`.`hostname` = 'registry.dev' LIMIT 1
Completed 401 Unauthorized in 94ms (ActiveRecord: 1.0ms)
[100] <IP_ADDRESS> - - [15/Feb/2016:16:30:47 +0100] "GET /unauthenticated?account=first.second&service=registry.dev HTTP/1.0" 401 - 0.1015

That's the error really. The thing is that when logging in you have to use your ldap name, so in your case first.second. The error you were having before is that it was trying to find the user with ldap_name but you provided through login the user name. So, there are two main things here:

BUG: for some reason the combo ldap_name and application tokens is not working.
Maybe we should be more flexible, and in LDAP allow both the ldap_name and the username. This way we would avoid the confusion about ldap_name vs username.

@flavio @dohnto what do you think ?

This must be fixed :)
What about disabling username for Docker CLI login when LDAP is enabled? It would be schizophrenic to use only first.second on the Portus web but firstsecond or even both first.second and firstsecond in Docker CLI.

But this is my 'user' point of view, I don't know the implementation details (LDAP names and usernames collisions etc).

I actually don't see a point in having separated ldap_name and username, is this really necessary? Couldn't there be just username and potentially a namespace if ldap_name doesn't meet some criteria (username = strip(username)), otherwise namespace = username.

This must be fixed :)

Yeah, the question was for the second point :D

What about disabling username for Docker CLI login when LDAP is enabled? It would be schizophrenic to use only first.second on the Portus web but firstsecond or even both first.second and firstsecond in Docker CLI.

That actually makes sense. We could forbid username everywhere for LDAP users if this makes more sense. We could also reflect that visually (e.g. when listing users). This has thing has also been requested before by the way.
The main point against all this is that the personal namespace must be the same as the username. This is because the restriction of docker names. Because of this, it might be confusing that the username that you are using is first.second, but your personal namespace is firstsecond. That's why I suggested the middle-ground of allowing both (but, as you said, this can also be confusing :D).

I actually don't see a point in having separated ldap_name and username, is this really necessary? Couldn't there be just username and potentially a namespace if ldap_name doesn't meet some criteria (username = strip(username)), otherwise namespace = username.

But the separation still remains right ? You need a column for the real name (ldap_name in this case), and the stripped username in case the ldap_name is invalid, so it can be used to identify your personal namespace. So, maybe the names of the columns being used are not that obvious, but they in the end reflect the same idea.
That being said, maybe what you are proposing is more elegant because in your way you'd have a username which is the one to be used always, and the namespace, which would identify the personal namespace of the user (which would've been stripped on creation time). I'll take a look into this, but if we change it, we should also provide an "upgrade" task for this specific case.

I agree that it is more less the same thing, but username + namespace  is more transparent than username + ldap_name.
Also it would probably solve all small issues (from my point of view) with mentioning username / ldap_name:

In dashboard: firstsecond created a token for the "test" application
In the top-right, I can see my username firstsecond
in admin/users, there is also firstsecond, etc.

Yes, makes sense. I've created the issue #736 for this ;)


Enhancement/photo upload

ALL CONTRIBUTIONS WILL BE REVIEWED BY AT LEAST ONE OF OUR FRONTEND TEAM MEMBERS.

Technical description

Related to

This PR is related to Name your issue
About Tests

Tested on the following devices

Steps to reproduce

Screenshots (optional):
This Pull Request template has been written and generated by Monk JS repository.

I've reviewed it a bit, but I think @BenHakimIlyass should look at it because it involves a lot a stuff that he did.
Please resolve conflicts first. :-)


Run react-native info in your terminal and copy the results here.

React Native version:

React Native Environment Info:
System:
OS: macOS 10.14.4
CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
Memory: 82.46 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 11.14.0 - /usr/local/bin/node
Yarn: 1.15.2 - /usr/local/bin/yarn
npm: 6.9.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
Android SDK:
API Levels: 21, 22, 23, 24, 25, 26, 27, 28
Build Tools: 27.0.3, 28.0.2, 28.0.3
System Images: android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.2 AI-181.55<IP_ADDRESS>14246
Xcode: 10.2.1/10E1001 - /usr/bin/xcodebuild
npmPackages:
react: 16.8.3 => 16.8.3
react-native: ^0.59.6 => 0.59.6
npmGlobalPackages:
react-native-cli: 2.0.1
Steps To Reproduce

Describe what you expected to happen:
Snack, code example, or link to a repository:

The new React Native project found that the Android R file was lost, so I couldn't write some Native activities

Did you build the App ? How did you create app ? Can you provide repo or more info ?

Yes I tried to create a new application on Windows and macOs. Unfortunately there are no R files in the android folder.

Closing the issue since the issue template was not followed. Please open a new issue with a repro and description of the issue if you are still having a problem


Java 21.3: Instanceof, strictfp

Description
Add audio, documentation, and video regarding the Instanceof, strictfp
Location of File to be added The files should be added inside the Instanceof, strictfp which will be inside Classes and methods folder which will be inside of Java folder. If the folder is not present, create the folder with this exact name.

The documentation file can be in the .md/.py/.ipynb format.
Audio should be in .mp3 format and to be submitted through a Google drive link.
Video can be in any format and should be submitted through a Google Drive link.

Note:
You are required to comment with \assign CONTENT_TYPE, so that other content types can be created at the same time. Failing to comment in the above manner will lead to the removal of assignment from the issue even if the bot assigns. You are required to do a PR within 7 days otherwise you will be unassigned and others will be assigned. Issues will be assigned on a first come first serve basis.
Domain
Java_Domain
Type of Contribution
Audio, Video, Documentation
Code of Conduct

[x]    I follow Contributing Guidelines & Code of conduct of this project.

/assign doc.


Bubble Sort

Bubble sort is a simple sorting algorithm that repeatedly steps through the list, compares adjacent elements, and swaps them if they are in the wrong order. This process is repeated until the list is sorted. It gets its name because smaller elements "bubble" to the top of the list with each iteration.
Time Complexity:
Best Case: O(n)
Average Case: O(n^2)
Worst Case: O(n^2)

assign me this task


silent was not really silent

Describe the bug
Use the CLI API detectClones，set options.slient: true, bug still output something.
To Reproduce
Steps to reproduce the behavior:
import { detectClones } from 'jscpd';
console.log(1)
await detectClones({
        path: files,
        silent: true
 });
console.log(2)

Expected behavior
output:
1
2
Actually
output:
1
Found 5 clones.
Detection time: 278.774ms
2
Desktop (please complete the following information):

OS: [windows]
OS Version [10]
NodeJS Version [10]
jscpd version [3.3.25]

Additional context
It is a feature ?

Fixed at<EMAIL_ADDRESS>


Add typing for some internal python files.

Add typing to some internal Python files.

We're going to want to add src/python/grpcio to the Pytype configuration so this is all checked by CI.


Batch upload flyte directory

TL;DR
s3fs will try to read all the files in the directory into memory by default. we can specify the batch size for s3fsFileSystem.put, and it will dramatically reduce memory consumption.
Before:

After (batch size = 100):

Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
from typing_extensions import Annotated

from flytekit import task, workflow
from flytekit.types.directory import FlyteDirectory
import os
from memory_profiler import profile
from flytekit import BatchSize
directory = "/tmp/test"

@task
@profile
def generate_files(num_files: int, filesize_mb: int) -> Annotated[FlyteDirectory, BatchSize(100)]:
    os.mkdir(directory)
    for i in range(num_files):
        file_path = os.path.join(directory, f"file_{i}.txt")
        with open(file_path, "w") as f:
            f.write(str(os.urandom(filesize_mb * 1024 * 1024)))
    return FlyteDirectory(directory, remote_directory="s3://flyte-benchmark/test/")

@task
def get_files(d: Annotated[FlyteDirectory, BatchSize(100)]):
    d._downloader()

@workflow
def ton_of_files_wf(num_files: int = 1000, filesize_mb: int = 8):
    d = generate_files(num_files=num_files, filesize_mb=filesize_mb)
    get_files(d=d)

if __name__ == '__main__':
    print(ton_of_files_wf())

Tracking Issue
NA
Follow-up issue
NA

How will users find about this. Cc @zeryx fyi


Set a border-radius when hovering annotationLayer-inputs (PR 15438 follow-up)

The changes in PR #15438 added a border-radius when input-elements are focused, however there's no radius when the same elements are hovered. Having the radius change, and not just the border-color, when input goes from hovered to focused feels a bit inconsistent (at least to me).

/botio-linux preview


When open project with Unity 2020.2.6f1

ArgumentException: An item with the same key has already been added. Key: com.unity.multiplayer-hlapi

Try removing this line:
"package-manager-extra-settings": "https://github.com/OndrejPetrzilka/package-manager-extra-settings.git",

from Packages/manifest.json


Various refactorings

renamed CCPACSMaterial to CCPACSMaterialDefinition to reflect name in CPACS schema
some simplifications on point list in fuselage profiles
using Handle() macro in OCC code to allow compiling with OCC 7.1
added std:: prefix to usage of endl, to avoid ambiguity with Qt's endl
added some missing invalidations
made some parameters const T& instead of T
added some const overloads
overriding SetUID on CTiglRelativelyPositionedComponents to re-register them when their UIDs change
added some missing virtual and OVERRIDEs
added IsUIDRegistered and TryRemoveGeometricComponent to CTiglUIDManager
using manipulation API where appropriate
removed unused members in CCPACSWingProfile

i justed synced your recent changes with our internal code base and here are some changes from our side which are probably useful to you as well.

Looks good!


Add support for azurerm_log_analytics_storage_insights

Hi @SergioMiyama
I have a use case where I'm currently trying to pipe logs from any storage account to an existing log analytics workspace. I believe this requires the azurerm_log_analytics_storage_insights Terraform resource as per the link here. https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/log_analytics_storage_insights. Thank you

@ianlim-cldcvr,
Hi Ian,
I have already implemented the module, and will just add the Name provider to it and it should be good to go.


Making package alpine friendly (Docker)

I am using Docker for containerizing my application. I am using alpine image for my purpose. how can I meet dependencies for an alpine build ?

Which file formats are you trying to parse? As of textract 1.6.2. only python dependencies are installed. Python packages with external dependencies such as pocketsphinx have become optional dependencies. All the non python parsers that textract supports need to be installed manually as ever.
Given that, the documentation lists dependencies for Ubuntu and Debian which at the least should provide a starting point for you. You can also look at the dependencies for the Travis CI build  which runs on Ubuntu.