Add travis CI script

Uses dotnet-cli v1.0.0-rc4-004771 without mono to specifically
build netstandard1.6 TFM.
Progress towards #14.

@FelschR, in my fork, current the build succeeds but the job fails at the step where it tries to push the tag: https://travis-ci.org/am11/AspNetCore.Identity.DocumentDB/builds/207810945#L554.
Shouldn't it be the other way around? Like when you will push a release tag, only then the CI job should upload the release assets? Something like: https://docs.travis-ci.com/user/deployment/releases/:
(however, this can be done va AppVeyor only, as the assembly we are producing here is platform agnostic / pure-MSIL)

Could you shortly explain why this the Travis CI would be useful in addition to the AppVeyor build?
The AppVeyor build is already building for net451 and netstandard1.6.

Hey @FelschR!
I took the .travis.yml from your netcore branch and fixed up for dotnet-cli RC4 and version extraction from csproj. Is that meant to be deprecated / removed?
Usually repos have both AppVeyor and TravisCI configured, to keep ensuring the xplat-ness and step aside related issue before they emerge in public landscape. For instance, unless we re-read the documentation of .NET BCL APIs we have been using for ages, we never know which API will throw PlatformNotSupportedException on Unix, while developing a cross-platform library on Windows.
In README, I read:

Microsoft.Azure.DocumentDB v1.10.0 - supports net45 (netstandard support is in development)

I am planning to consume this package for the app running on Unix, knowing Azure has released the DocDB package supporting .NET Core. So I cloned your repo and found that you already have the support in master branch.

I see.
The problem is that the Travis CI script isn't running the unit and integration tests, it's just running the build.
As AppVeyor already creates the build we can remove the Git/Github tag/release publishing from the Travis CI completely.
So we should try to find a way to run the tests on Travis CI. Or rather find a way to run the tests on a Linux machine. Am I right?

@FelschR, I agree that we must be able to run tests. Unfortunately, DocDB emulator is only available on Windows; native Unix support is on their long-term roadmap https://docs.microsoft.com/en-us/azure/documentdb/documentdb-nosql-local-emulator (see comments).
We can use xunit traits to disable the affected tests for non-Windows for now, but there are only few such tests.

Sounds good.
Do you want to create a new pull request for a script that runs the tests (it's ok when it fails initially) and does not create a Git tag or should I create one?

@FelschR, done. Can you please take a look?

Ok, everything looks quite ok.
The build(s) still behave somewhat strange now which could be fixed.
The Travis CI build finished first but failed during the git tag creation.
The AppVeyor build finished successfully afterwards.
Then the Travis CI build was retriggered and finished successfully.
The 2nd Travis CI build is successful because it skips the git tag creation because the commit was now tagged by AppVeyor.

Perhaps it was the first time on the master branch and some Travis<->GitHub syncing failed? Might be the intermittent error?
Should we wait a bit and if this problem persists, then remove the tag?
On a slightly related note, the TravisCI badge in the README has link to netcore branch. Perhaps we can apply the following patch:
- [![Travis CI build status](https://travis-ci.org/FelschR/aspnetcore-identity-documentdb.svg?branch=netcore)](https://travis-ci.org/FelschR/aspnetcore-identity-documentdb?branch=netcore)

+ [![Travis CI build status](https://travis-ci.org/FelschR/aspnetcore-identity-documentdb.svg?branch=netcore)](https://travis-ci.org/FelschR/AspNetCore.Identity.DocumentDB)

Also, I was thinking to add macOS and Mono (requires separate SLN with old-CSPROJ facade) to the matrix by extracting out the script to a separate .sh file, like what we did here: https://github.com/maxmind/MaxMind-DB-Reader-dotnet/blob/9d247d7/.travis.yml. :)

Just did a build and noticed the same thing again: first build failed with Github error, 2nd went through.
I'll have to look into that later.


Accessing Tile38-cli via Docker install?

Hello I'm not too familiar with Docker and I need to access the Tile38-cli to test some things.
To start the server: docker run -p 9851:9851 tile38/tile38
But how do I access tile38-cli?

Try docker run -it tile38/tile38 tile38-cli

Hi Tobilg,
I have 2 ssh windows open. One is running the tile38 server via docker and the other the cli.
When I run docker run -it tile38/tile38 tile38-cli
I get Could not connect to Tile38 at <IP_ADDRESS>:9851: Connection refused

Probably it’s easier to just attach to the running Server Container. Otherwise you‘ll have to use host networking on the CLI Container and specify the host ip address to connect to

要将tile38-cli的docker实例连接到同一台计算机上tile38-sever的docker实例，请执行以下操作：
docker run -p 9851:9851 tile38/tile38

docker run --net=host -it tile38/tile38 tile38-cli

将-net=host本地主机公开给docker环境。

记得是打开两个ssh

For reference, using --net=host means your tile38 is accessible by anyone on your network (cafe wifi, yay), so the longer and more secure route, is to make a private network for tile38, and to use it
its a 3 step process, but can be worth knowing:
$ docker network create tile38
872c3a5db19d1b1f3156ac9f206c37b185a1a1e637e29758ccf1a86363e1c856
docker run --net=tile38 -it --rm --name tile38 tile38/tile38

   _______ _______
  |       |       |
  |____   |   _   |   Tile38 1.19.5 (0490734) 64 bit (amd64/linux)
  |       |       |   Port: 9851, PID: 1
  |____   |   _   |
  |       |       |   tile38.com
  |_______|_______|

2020/02/20 03:46:18 [INFO] Server started, Tile38 version 1.19.5, git 0490734
2020/02/20 03:46:18 [INFO] AOF loaded 0 commands: 0.00s, 0/s, 0 bytes/s
2020/02/20 03:46:18 [INFO] Ready to accept connections at [::]:9851

and then to access it (starting a new container that only runs the client):
$ docker run --net=tile38 -it tile38/tile38 tile38-cli -h tile38
tile38:9851>

OR (re-using the server container)
$ docker exec -it tile38 tile38-cli
<IP_ADDRESS>:9851>


Font size in front page

The font size of the landing page in sbt-microsites is a little bit small. Is it possible to increase it for a better UX in high-resolution screens?

Thanks for the feedback @jvican .
Currently, the plugin is using the standar font-size for all the common browsers, which is 15 px. However, the plugin allows you to override according your preferences. For example, you may want to create an override.css file overriding the font-size (using the sbt micrositeCssDirectory setting):
body { font-size: 18px;}
.technologies p { font-size: 16px;}

On a related note, I would recommend you to use a black font, the gray one is hardly readable.
Check http://contrastrebellion.com/.

Many thanks!
What do you thinks @israelperezglez ? Could we make the change? If so, let's create an issue to tackle it.

Let me make some proof of concepts and study it, before creating an issue.
Thanks for your comments!


Add .spec.insecureSkipVerify to HelmRepository for type: oci

Allow connecting to Helm OCI HTTPs repositories without verifying the server's certificate chain and host name.
Example:
apiVersion: source.toolkit.fluxcd.io/v1beta2
kind: HelmRepository
metadata:
  name: myhelmrepo
  namespace: default
spec:
  type: oci
  interval: 1h
  insecureSkipVerify: true
  url: oci://my-self-signed-helm-repo-url:5000/charts

To work with self-signed certs you can provide the CA with, please see https://fluxcd.io/flux/components/source/helmrepositories/#cert-secret-reference
For security reasons, we decided to not have insecureSkipVerify anywhere in Flux controllers.

To work with self-signed certs you can provide the CA to Flux, please see https://fluxcd.io/flux/components/source/helmrepositories/#cert-secret-reference
For security reasons, we decided to not have insecureSkipVerify anywhere in Flux controllers.

Hi @stefanprodan, thank you so much for the comment!
I thought it'd be a nice feature to be able to configure self-signed Helm registries without handling the self-signed CA certificates, specially in development environments. But I understand your decision since it's not a good practice.


Raspberry Pi

Hey, tried multiple times to install this on a clean raspbian os downloaded und updated right before the installation and following the official installation guide for raspberry pi. Every time I get errors like
flex not found. Installing
...
curl: (77) error setting certificate verify locations:
CAfile: /etc/ssl/cert.pem
CApath: none
So far I didn't find a solution. Any suggestions? Or can somebody upload a working raspbian image with the video stream producer installed?

Please refer the following readme for the certificates for downloading  the open source dependencies from https.
https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp

Certificate store integration
Kinesis Video Streams Produicer SDK for C++ needs to establish trust with the backend service through TLS. This is done through validating the CAs in the public certificate store. On Linux-based models, this store is located in /etc/ssl/ directory by default.
Please download the PEM file from https://www.amazontrust.com/repository/SFSRootCAG2.pem
to /etc/ssl/cert.pem. Append to the end of the file if it exists.

Still no success:
flex not found. Installing
...
curl: (60) SSL certificate problem: unable to get local issuer certificate
More details here: https://curl.haxx.se/docs/sslcerts.html
curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.
Since I wants to eliminate the chance I made a mistake following the provided tutorial (First google link: https://aws.amazon.com/de/kinesis/video-streams/raspberry-pi-tutorial/), did anyone get it to run following exactly these steps? Seems to me like this guide is outdated.

As @bdhandap  has posted in https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/issues/40 , it could be that the curl was not built with the cacert.

Can you please reinstall curl by following below steps? This should fix the issue.
rm <producer_sdk_path>/kinesis-video-native-build/downloads/local/lib/libcurl*
rm <producer_sdk_path>/kinesis-video-native-build/downloads/local/bin/curl*
cd <producer_sdk_path>/kinesis-video-native-build/downloads/curl-7.57.0
export DOWNLOADS=<producer_sdk_path>/kinesis-video-native-build/downloads
make clean
./configure --prefix=$DOWNLOADS/local/ --enable-dynamic --disable-rtsp --disable-ldap --without-zlib --with-ssl=$DOWNLOADS/local/ --with-ca-bundle=/etc/ssl/cert.pem
make
make install
After reinstalling you can verify it by running ./curl-config --ca. By default the curl library should pick up the default path /etc/ssl/cert.pem but looks like its not doing that it in all cases. I'll add this configure option in install-script so that it doesn't happen again.

@JulianKling  Curl installation tries to configure the path to /etc/ssl/cert.pem and  curl: (77) error setting certificate verify locations: CAfile: /etc/ssl/cert.pem error means that the certificate location passed during the installation does not exist. If you did copy the certificate to the mentioned path and tried again, the chances are that configure step did not pick the path as expected. You can verify this by running ./curl-config --ca inside the <producer_sdk_path>/kinesis-video-native-build/downloads/local/bin/ folder. Can you please reinstall like mentioned in the Troubleshooting section ?

Thank you @unicornss and @bdhandap, I think the combination solved it (currently still running). While most likely not recommended, another way would be to add the -k flag to the curl calls inside the installation script (finished successfully).
I think further (potential) customers would appreciate it if you edited to linked guide to include the given help, since these steps seem to be mandatory. (Also, just for the sake of completeness  on how i got it to run, i had to enable the v4l2 drivers on the raspberry pi with sudo rpi-update # to make sure the v4L2 drive is available and sudo modprobe bcm2835-v4l2 # to load it and create /dev/video0)

@JulianKling I'm glad it worked. And thanks for your feedback. We are working on updating the raspberry pi tutorial guide and will be releasing it soon.

@JulianKling
How did you add -k flag to curl calls in installation script?
Please help..


Smalltalk Naming Conventions Issue

Hey, I was just poking around the repo, and I noticed that the following files exist:

smalltalk.sm
hello.st

The first file seems fine, at least according to The Hello World Collection.
However, that second file doesn't appear to follow the naming conventions throughout the repo. That said, if someone wanted to rename it, I suggest smalltalk-gnu (that'll keep it under S). See the GNU Smalltalk Repo.
Of course, I believe both scripts run under GNU Smalltalk (tested here), so it may make sense just to remove the hello.st file.

I am researching the best way forward on this. Great find, thanks for your contribution!

Fixed in https://github.com/leachim6/hello-world/commit/dc75f068dc3674e378d5c27c707cd8e033e33c65#diff-561c42b5a19da8219cad15def2061935


PIP-17: provide DataBlockHeader and implementation

Motivation
Provide DataBlockHeader implementation. A block header is in format:
[ magic_word -- int ][ block_len -- int ][ first_entry_id  -- long][alignment padding]
Master issue #1511
Modifications
Add class DataBlockHeader and test for it.
Result
unit test pass.

@ivankelly updated following your comments.

@zhaijack lgtm. Will you add the master issue to the PR description, so it gets linked there.

Thanks @ivankelly , added it.

moved this to 2.1

Weird. It looks like the test hangs on MessageIdTest for over 2 hours

retest this please


Adding a filter string to SQSTrigger to enable conditional builds

Proposed: adding a new parameter to the SQS build trigger called 'filterString'. This string is used to match against the incoming SQS message body. If the string matches, or is a wildcard value, the build is triggered. If the filter does not match, it skips the build. This allows a single SQS queue to be used to trigger multiple independent builds/jobs rather than using separate queues.
This change includes a new textbox in the SQSTrigger config to expose the filterString to the user. This addresses Issue #19 I filed earlier today.

@iomac can you please take a look at the conflicts, before we can start discussing this feature.


Drop LLVM 3.5 and LLVM 3.6 support

Since omnibus will be updated to use LLVM 3.8.1, this drops support for older, buggy, LLVM releases. This also cleans up the codebase.
The supported LLVM versions are now 3.8, 3.9 and 4.0 (manually tested).

Note: We are missing invocations to llvm-config-4.0, in some linux packages there might be no llvm-config but llvm-config-4.0. (in the Makefile and lib_llvm.cr). I was expecting to do that soon, but it will conflict slightly with this.
This can't be merged until the omnibus-crystal is migrated to use newer llvm, so avoid merging until @matiasgarciaisaia or @waj says so.

How about Dockerfile? It is using 3.5.0.
%  docker run --rm crystallang/crystal crystal --version
Crystal 0.22.0 [3c71228] (2017-04-20) LLVM 3.5.0

P. S.: I mean not necessarily updating it in this PR, but it is required to update 😉 .

The docker image is based on released Crystal AFAIK so it should be updated on next release.
I initially didn't had llvm-config-4.0 when I merged the compatibility commits, because LLVM4 was in beta. We can decide to upgrade to LLVM4 by default (or as a fallback) now that it's stable. Your call, I'll make the change.

Great! :D

I suppose that latest stable LLVM should be default one.

LLVM 3.9 is the current stable. LLVM 4 is qualification and LLVM 5 is unstable. Some package managers (i.e. homebrew) are pushing the qualification release as stable but it's actually not by LLVM terms.

Looks like these functions should be uncommented.
https://github.com/crystal-lang/crystal/blob/master/src/math/libm.cr#L6

Also:

https://github.com/crystal-lang/crystal/blob/master/src/math/libm.cr#L34
https://github.com/crystal-lang/crystal/blob/master/src/math/math.cr#L112
https://github.com/crystal-lang/crystal/blob/master/src/math/math.cr#L134

this PR drops support for older LLVM releases (cleanup);
that depends on which LLVM release the intrinsics were introduced (it's unclear).

@ysbaddaden As @RX14 said - minnum and maxnum were introduced in LLVM 3.6.

minnum and maxnum intrinsics will be added by #4559.

Bump. Since omnibus now relies on LLVM 3.8 can we drop support for older, buggy, LLVM versions?

Bump 👍

I'm not that sure it's wise to merge this until we get rid of things like
#4719
I'd treat 3.8 builds as release candidates for now - I'd rather have the
safety net of falling back to 3.5 if anything gets to ugly.
Probably over-conservative, but that's what I feel.
I'm open to other opinions, anyway.

Everywhere in code I see mentions about ugly earliest LLVM < 3.9.
We must drop support for old LLVMs because of

https://github.com/crystal-lang/crystal/issues/4719#issuecomment-322071381
Crystal still in early development stages, so support for old codebase is not needed.

The solution is to statically compile LLVM into the crystal compiler so we can support 1 or maybe 2 LLVM major versions and not have to worry about distributions.

Closing, this was merged in #4821.


False positive for Style/EachForSimpleLoop

I discovered an issue where a parameter was determined not to be used under certain conditions.
(0..9).each { |i| "123" % (i + 1) }
(0..9).each { |i| SomeObject.new(i:) }

Expected behavior
no offence.
Actual behavior
$ echo '(0..9).each { |i| "123" % (i + 1) }' | bundle exec rubocop --stdin test.rb --only Style/EachForSimpleLoop
Inspecting 1 file
C

Offenses:

test.rb:1:1: C: [Correctable] Style/EachForSimpleLoop: Use Integer#times for a simple loop which iterates a fixed number of times.
(0..9).each { |i| "123" % (i + 1) }
^^^^^^^^^^^

1 file inspected, 1 offense detected, 1 offense autocorrectable
$ echo '(0..9).each { |i| SomeObject.new(i:) }' | bundle exec rubocop --stdin test.rb --only Style/EachForSimpleLoop
Inspecting 1 file
C

Offenses:

test.rb:1:1: C: [Correctable] Style/EachForSimpleLoop: Use Integer#times for a simple loop which iterates a fixed number of times.
(0..9).each { |i| SomeObject.new(i:) }
^^^^^^^^^^^

Steps to reproduce the problem
see Actual behavior
RuboCop version
Include the output of rubocop -V or bundle exec rubocop -V if using Bundler.
If you see extension cop versions (e.g. rubocop-performance, rubocop-rspec, and others)
output by rubocop -V, include them as well. Here's an example:
$ [bundle exec] rubocop -V
1.59.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.30.0, running on ruby 3.1.3) [arm64-darwin22]
  - rubocop-capybara 2.19.0
  - rubocop-factory_bot 2.24.0
  - rubocop-performance 1.19.1
  - rubocop-rails 2.22.2
  - rubocop-rspec 2.25.0

It appears to behave as expected, so can you provide more information on what issues you are encountering?

https://docs.rubocop.org/rubocop/cops_style.html#styleeachforsimpleloop
says that:

This check only applies if the block takes no parameters.

My understanding is that even though this uses each, it shouldn't violate Lint if parameters are used. Is my understanding wrong?

It seems to have been added in #3182, but I didn't know what "This check only applies if the block takes no parameters." meant. It seems like something is wrong with either the documentation or the behavior.
IMO, the presence or absence of parameters is irrelevant.

If there are no parameters, only the number of repetitions is important, so times may be easier to read, but if there are parameters, it may be easier to read if you write them in range.
So I think it's better to leave the explanation as is and change the implementation.

In https://github.com/rubocop/rubocop/pull/10928, it seems that the behavior changes when the start of the range is zero, even when a block parameter is present. About this cop, it appears that the documentation needs to be updated.

I still think changing range to times when there are arguments has readability issues.
Consistent with the rule that render numbers should be written as ranges, I would have preferred not to revert the above PR outright, but to make an exception like (0..10).each { |i| }.

OK, TBH, I don't have a strong opinion for this. Let's respect the documentation, I've opened https://github.com/rubocop/rubocop/pull/12609.


Session bc60a3ba-c6fe-4b19-bfc7-a34f5ce19d57. timed out

Investigative information
Please provide the following:

UTC Timestamp: 2022-06-02 2249
Migration Tool App version : 1.0.6
OS Version: Windows Server 2019 Datacenter

Repro steps
Provide the steps required to reproduce the problem:

Run assessment, choose a site, wait for assessment report

Expected behavior
Provide a description of the expected behavior.
Assessment should complete without error.
Actual behavior
Provide a description of the actual behavior observed.
"We're sorry, there has been an error. To get help, please use the Help > Report Issues menu option and include the following information: Session bc60a3ba-c6fe-4b19-bfc7-a34f5ce19d57. timed out" Appears after 20 minutes or so.
Known workarounds

Hi @neville-mooney - thank you for letting us know this is happening! Looking at that sessionId I can see that it took about 13 minutes to get the local machine configuration details (this can happen due to a large amount of site content, which increases the amount of time to load all IIS configuration - which might be at any level of the site - or determine content size), and then additional time for a check on connection strings in web.configs, likely for the same reason. These are eventually timing out the GUI waiting for the result. You would likely be able to duplicate this behavior using the commandline iisConfigAssistant.exe (i.e. if you did step 4 as noted here: https://github.com/Azure/App-Service-Migration-Assistant/wiki/How-the-Assistant-Works#what-information-is-sent-for-assessments it would also take a very long time).
The PowerShell migration scripts would be an option to evaluate and migrate this site instead, as it has some optimizations that should make the assessment step much quicker. The scripts are available on http://aka.ms/appmigrate ("App Service migration assistant for PowerShell scripts (preview)") - notes on how to use them are here: https://github.com/Azure/App-Service-Migration-Assistant/wiki/PowerShell-Scripts

Closing due to lack of activity for over 90 days


UT Add wal dirs info be ut

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall


Handle non-float types for BSCALE and BZERO

Apparently my camera produces int types for BSCALE and BZERO. Can be expanded with more types as needed.

Done!

Just re-read your comment. I'll update the commit message.


System tray don't show

I ran the sample code successfully,  but did not see the system tray, please help.

flutter doctor
Flutter assets will be downloaded from https://storage.flutter-io.cn. Make sure you trust this source!
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.10.0, on Microsoft Windows [Version 10.0.22000.434], locale zh-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[✗] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.0.5)
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3)
[✓] Connected device (3 available)

Missing assert config


Blank admin page

Hi,
After done the step 3 and I get the blank page instead of admin page.
Please find the javascript error in the image below.

Regards,
Calvin Ong

Hi @clvnong is it on dev brach?
Quite strenght as I just completed the setup step yesterday during debug another issue for you.
Let me check again.

Hi Huy,
Yes, it was from dev-branch.
On Thu, Oct 15, 2020 at 4:27 PM Huy Nguyen<EMAIL_ADDRESS>wrote:

Hi @clvnong https://github.com/clvnong is it on dev brach?
Quite strenght as I just completed the setup step yesterday during debug
another issue for you.
Let me check again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mixcore/mix.core/issues/278#issuecomment-708988544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALTXAWLEJZEBDMFU2OWUNN3SK2W5PANCNFSM4SRR7ZUQ
.

Hi @clvnong ,
Sorry for my late response, I just tried the full process and record the video here for your reference.
https://youtu.be/-1e-OQB2lUg
Please feel free let me know if you have any other issues.
Best regards,
Huy Nguyen


Allow extensions to hide 'Edit' and 'Remove' while using KubeObjectDetails component

Now we check for store methods in https://github.com/lensapp/lens/blob/master/src/renderer/components/kube-object-menu/kube-object-menu.tsx#L43 which is not always the best approach.

Now we check for store methods in https://github.com/lensapp/lens/blob/master/src/renderer/components/kube-object-menu/kube-object-menu.tsx#L43 which is not always the best approach.

That approach is currently broken because it's not possible to have KubeObjectStore without those methods.


Can Clair provide a API to query the last successed  vulernability update time?

It will be nice that Clair can provide a API to return the timestamp of last successful update vulnerability database.
When we will integration Clair with docker system there is no way to know if the vulnerability database is up to date or not. (Yes Clair will update database in back end, but without check the log, we can't tell the status)
so if we have this kind of API we can use it to integration with other systems.

We're working on a new schema and gRPC API for v3.
We're going to aim to include this value in the API responses containing vulnerabilities.
Thanks for the feedback!

mean while can we also consider the update time for each namespaces?  as different namespace have different update time.

@jzelinskie , this may not be a "usability" issue, Clair needs a better way to figure out if the vulnerability data is stale, without the checking the logs.

This is now in the responses of the Clair v3 API.


currentUser lost with two windows open

This might be related to #523
Every time I open a second tab, the currentUser object is 'lost' on the old one, this is caused by
  bindToStorageEvents: function() {
    var _this = this;
    Ember.$(window).bind('storage', function(e) {
      var data = _this.restore();
      if (!objectsAreEqual(data, _this._lastData)) {
        _this._lastData = data;
        _this.trigger('sessionDataUpdated', data);
      }
    });
  }

I'm not 100% clear on why, or if it's intentional or not, but data has a class instance of currentUser while _lastData has just an object referencing the currentUser which is being set as the new data session.

(first object represents data, second object represents _lastData)
You can even see the session.currentUser change on the first window you have open

(first object is on first load, second object is after opening a second tab)
It might also be related to the persist model as I can see it's setting some currentUser objects and some currentUser classes, starting here it went over my head pretty quickly so I'm not even sure.
Just in case it's related, this is my code for getting the currentUser
import Session from "simple-auth/session";
import Ember from 'ember';

export default {
  name: 'cusotm-session',
  before: 'simple-auth',
  initialize: function(container, app) {

    Session.reopen({
      setCurrentUser: function() {
        if(this.get('isAuthenticated')){

          var _this = this;
          var store = container.lookup("store:main");

          Ember.$.ajax(url).then(function(data){
            var me = store.push('user', data.load.me);
            _this.set('currentUser', me);
            // custom app logic unrelated to the user
          });
        }
      }.observes('isAuthenticated')
    });
  }
};

I'd love to provide more info, not sure what else could be relevant, pretty new to ember so my apologies if theres any obvious mistake I'm not aware of.

You shouldn't store class instances in the session, only POJOs, storing class instances isn't supported. Checkout this example for adding a currentUser property to the session.

Alright, I understand, so there's no way to use that user computed properties I guess? is that intended?

You can use a computed property to make the current user accessible via the session but not store an actual user object in the session.


Update for latest e3nn?

Hello
Thanks a lot for the very interesting paper & results, and for making the code available. - Do you have any plans to update the repo to the latest version of e3nn (0.2.4 a.t.m.)?
Best
Frank

Hello Frank,
I'm sorry that I didn't respond earlier. I think the e3nn library had some breaking changes (regarding the parts of it that I used), so updating the repo to the latest version probably involves greater changes to the code. Unfortunately, I currently don't find enough time for doing this. But I'd always appreciate a pull request.
Best
Philip


System CMake overridden

I find it unacceptable that this tool installs its own cmake executable that overrides the system-installed cmake. This can break so many things.
I don't use triton directly, only torchvision, so I have no clue what this tools is. But I know the system CMake interpreter must not be changed.
If I got the setup.py correctly, cmake is only needed to build the package. If the tool requires a newer than system cmake, then it should install it either only temporarily, or in a location that is not picked up by the rest of the system. If it's okay with older cmake, it should not install the pip version at all.

It installs the python cmake module (which is just cmake AFAICT) which is probably ahead of your system cmake in the path thanks to new software's stupid habit of pre-pending to paths.  I'd just change that order or do pip uninstall cmake :P
The other solution is to use python venvs but those are just going to eat more disk space as you slowly accumulate 20 versions of every module.
I thought it was dumb too, but I hit it a while ago with something different.  Keep an eye out for ninja getting overridden too if you use that.


chore: configure global identity for backport (#718)

Backport
This will backport the following commits from k8s-22/main to k8s-20/main:

chore: configure global identity for backport (#718)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 720

Questions ?
Please refer to the Backport tool documentation


Our appveyor build should precompile the razor views

This is waiting on some new support from asp.net 5, but when that is supported, we should build the razor views as part of the cheks on PRs.

Experimented with this a little today and could not get the build to fail when there were errors on a razor page. I will revisit once the next version of aspnet5 is released.

@tonysurma Can close this as dupe of #972.


[Feat]: TO ADD A HOME PAGE

What feature?
i would like to add a home page for this website...it will enhance the website,i think so
Add screenshots
Add screenshots
Code of Conduct

[X] I agree to follow this project's Code of Conduct

can u pls assign me this issue

No Need now, maybe in future


Make "log" optional dependency

PR for https://github.com/d2718/dumb-cgi/issues/1

Awesome, thanks for going to the trouble here.


Add support for RGBA Images

I added support for RGBA images. Straight forward.

I also removed message generation for Java

I need to make some more changes, will start a new pull req. any time soon.


Remove dependency on cassandraDriver.types.Uuid

Currently the Uuid type is used throughout the system. Since other database systems will not produce entities of this type, methods will be added to the persister classes to deal with any kind of reference to that object, for instance for comparison in ViewerType:
...
import { Uuid } from '../../graphql/CassandraClient.js';
const Uuid_0 = Uuid.fromString( '00000000-0000-0000-0000-000000000000' );
...
    User_IsAnonymous:        { type: GraphQLBoolean, resolve: (obj) => obj.id.equals( Uuid_0 ) },
...

This has been resolved by PersiterXXX.uuidYYY functions.


Fab Custom Icons Don't work

Hello,
FAB custom icons don't work:
Take, for example:
<mdc-fab fixed class="mdi mdi-account mdi-24px" href="@route('agent.clients.create')" v-show="currentTab == 'active'"></mdc-fab>

This results in HTML where the class for the icon being used is attached to the  tag, while inside of that  tag is a . The icon class therefore does not display as the icon classes provided need to attach to the  not  tag to be rendered, for example:

not ideal for the markup, but you can get it work like this:
https://codepen.io/stasson/pen/KyLQyR

Good to know, but at the very least then the documentation is out of date:
https://stasson.github.io/vue-mdc-adapter/#/component/fab

yes. the doc is out of date.  And the rendered markup is not ideal.
Guess I need to fix both.
Thanks for the catch anyway.

need to fix icon-toggle as well :(

I’d be happy to contribute more but I haven’t figured out how to build this
module from source yet for my project!
On Wed, Dec 6, 2017 at 5:33 PM stasson<EMAIL_ADDRESS>wrote:

need to fix icon-toggle as well :(
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stasson/vue-mdc-adapter/issues/150#issuecomment-349797315,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABxvLxAnRj05J01BKTzTjbV4jBnKyHZEks5s9xY4gaJpZM4Q4kxx
.


eslint error markers are off by one

I use this in .vimrc: let g:syntastic_javascript_checkers = ['eslint']
$ eslint --version
v3.11.1

The syntastic highlight markers land on one character after where they should.
I'm not really asking for someone to guess or know or find out what causes this, I'm asking for debugging steps to figure out how syntastic knows to read eslint output and converts it into markers. So I can try to find the source of the off by one error.

Just skimming the code: https://github.com/vim-syntastic/syntastic/blob/master/syntax_checkers/javascript/eslint.vim#L63
Looks like theres an explicit add one here. Maybe eslint changed from 0-indexed column number to 1-indexed?

Please open your test file, set g:syntastic_debug to 33, run :SyntasticCheck eslint, run :mes, and post the output.

I've reinstalled everything on a different Linux system and everything seems to be fine with the latest eslint and syntastic. I will just have to try that on the other environment where I was seeing the problem.


docblock query class fixes

Dang it, I missed a few places when moving the query class docblock into the template. Here is the fix. Sorry.

Totally, the template file is better than the inline version from before, but it is really not great either.
But I think we got it now, here is a diff old vs new:
 /**
- * Base class that represents a query for the 'referrer' table.
- *
- *
+ * Base class that represents a query for the `referrer` table.
  *
  * @method     ChildReferrerQuery orderById($order = Criteria::ASC) Order by the id column
  * @method     ChildReferrerQuery orderByReferrer($order = Criteria::ASC) Order by the referrer column
@@ -65,8 +63,8 @@ use Propel\Runtime\Exception\PropelException;
  *
  * @method     ChildReferrer|null findOneById(int $id) Return the first ChildReferrer filtered by the id column
  * @method     ChildReferrer|null findOneByReferrer(string $referrer) Return the first ChildReferrer filtered by the referrer column
- * @method     ChildReferrer|null findOneByComment(string $comment) Return the first ChildReferrer filtered by the comment column *
-
+ * @method     ChildReferrer|null findOneByComment(string $comment) Return the first ChildReferrer filtered by the comment column
+ *
  * @method     ChildReferrer requirePk($key, ?ConnectionInterface $con = null) Return the ChildReferrer by primary key and throws \Propel\Runtime\Exception\EntityNotFoundException when not found
  * @method     ChildReferrer requireOne(?ConnectionInterface $con = null) Return the first ChildReferrer matching the query and throws \Propel\Runtime\Exception\EntityNotFoundException when not found
  *
@@ -76,15 +74,16 @@ use Propel\Runtime\Exception\PropelException;
  *
  * @method     ChildReferrer[]|Collection find(?ConnectionInterface $con = null) Return ChildReferrer objects based on current ModelCriteria
  * @psalm-method Collection&\Traversable<ChildReferrer> find(?ConnectionInterface $con = null) Return ChildReferrer objects based on current ModelCriteria
- * @method     ChildReferrer[]|Collection findById(int $id) Return ChildReferrer objects filtered by the id column
- * @psalm-method Collection&\Traversable<ChildReferrer> findById(int $id) Return ChildReferrer objects filtered by the id column
- * @method     ChildReferrer[]|Collection findByReferrer(string $referrer) Return ChildReferrer objects filtered by the referrer column
- * @psalm-method Collection&\Traversable<ChildReferrer> findByReferrer(string $referrer) Return ChildReferrer objects filtered by the referrer column
- * @method     ChildReferrer[]|Collection findByComment(string $comment) Return ChildReferrer objects filtered by the comment column
- * @psalm-method Collection&\Traversable<ChildReferrer> findByComment(string $comment) Return ChildReferrer objects filtered by the comment column
+ *
+ * @method     ChildReferrer[]|Collection findById(int|array<int> $id) Return ChildReferrer objects filtered by the id column
+ * @psalm-method Collection&\Traversable<ChildReferrer> findById(int|array<int> $id) Return ChildReferrer objects filtered by the id column
+ * @method     ChildReferrer[]|Collection findByReferrer(string|array<string> $referrer) Return ChildReferrer objects filtered by the referrer column
+ * @psalm-method Collection&\Traversable<ChildReferrer> findByReferrer(string|array<string> $referrer) Return ChildReferrer objects filtered by the referrer column
+ * @method     ChildReferrer[]|Collection findByComment(string|array<string> $comment) Return ChildReferrer objects filtered by the comment column
+ * @psalm-method Collection&\Traversable<ChildReferrer> findByComment(string|array<string> $comment) Return ChildReferrer objects filtered by the comment column
+ *
  * @method     ChildReferrer[]|\Propel\Runtime\Util\PropelModelPager paginate($page = 1, $maxPerPage = 10, ?ConnectionInterface $con = null) Issue a SELECT query based on the current ModelCriteria and uses a page and a maximum number of results per page to compute an offset and a limit
  * @psalm-method \Propel\Runtime\Util\PropelModelPager&\Traversable<ChildReferrer> paginate($page = 1, $maxPerPage = 10, ?ConnectionInterface $con = null) Issue a SELECT query based on the current ModelCriteria and uses a page and a maximum number of results per page to compute an offset and a limit
- *
  */
 abstract class ReferrerQuery extends ModelCriteria
 {


Validators boolean logic

Package version eg. v8, v9:
v9
Issue, Question or Enhancement:
I'm trying to test a struct with validation tags but i can't understand how the boolean logic works.
In this example it's expected that the validator on the Path attribute returns false value, but i don't know how it returns true, because the require.Error  in TestPatchOpValid fails.
I'm using two custom tags:

isfield: it checks if  a struct's attribute has a certain value (eg. isfield=Op:remove)

attrpath: it checks if the value has a correct composition (it's not so important in this case)

Have you an explanation for the validators' logic?
Code sample, to showcase or reproduce:
type Operation struct {
	Op    string   `json:"op" validate:"required,eq=add|eq=remove|eq=replace"`
	Path  string   `json:"path,omitempty" validate:"attrpath|eq='',isfield=Op:replace|isfield=Op:add|attrpath"`
	Value []*Value `json:"value,omitempty" validate:"gt=0|isfield=Op:remove"`
}

func TestPatchOpValid(t *testing.T) {
	var err error
        op := Operation{}
	op.Op = "replace"
	op.Path = "#######" // wrong path
	op.Value = []*Value{
		&Value{
			Display: "Babs Jensen",
			Ref:     "https://example.com/v2/Users/2819c223",
			Value:   "2819c223-7f76",
			Type:    "home",
		},
	}

	err = validation.Validator.Struct(op)
	fmt.Println(err)
	require.Error(t, err)

Hello @skryja it's a little hard not knowing what the custom function do, but will try to layout the operations that should occur for the Path field:
Block1 & Block2 below are AND's eg. if Block1 and Block2 = validation success
Block1

first it will check attrpath|eq=''

if attrpath returns true, move to the next validation block
if attrpath returns false, move to eq='' if returns true move to next validation block otherwise return error and stop other validation for the field

Block2

next it will check isfield=Op:replace|isfield=Op:add|attrpath

if isfield=Op:replace returns true, move to the next validation block(which there is none validation of field complete)
if isfield=Op:replace returns false, move to isfield=Op:add if returns true, move to next validation block
if isfield=Op:add returns false then move to attrpath if it returns true move to next validation block
if attrpath returns false return error.

it seems odd to me to have 2 validations here, and that attrpath is part of both, I think it could be combined to validate:"eq='',isfield=Op:replace|isfield=Op:add|attrpath" or even validate:"omitempty,isfield=Op:replace|isfield=Op:add|attrpath"
I hope this helps, please let me know :)

Thanks @joeybloggs for your answer,
but in the case of the test, attrpath and eq both return false so, as you said, it should stop the validation on the field and should return an error on the validation tag eq as I would like it to be in the test. Instead, the validation seems to continue and goes on Block2 and returns true because  of isfield=Op:replace and the test will fail.

Hey @skryja I may have solved this in a dream ;)
one thing I realized was the eq='' is probably not checking for what you want it’s equivalent to Path == “''” and not Path == “” and should be changed to eq=
Maybe I’m way off?

Hey @skryja I have confirmed that this is a bug and is due to the way I was checking when to terminate each or instance; will be fixed soon, nice catch 😄

Thanks @skryja for reporting, it should now be fixed in v9.9.1 release, please update and try again
if you have any more problems with it please let me know 😄

Thanks @joeybloggs ...now it works! :)
Meanwhile my colleague @leogr found this alternative solution:
 isfield=Op:add|isfield=Op:replace|required,omitempty,attrpath
If you want to take a look at what isfield and attrpath validators do; also you can check our repo here.
Thanks again for your time :smiley:

Thanks for that ! always interested in seeing how others are using it :) I may even add a few after seeing your like prefix and suffix validations.
oh and I noticed your issue 37 about the required tag failing, there are a few issue, think one still open tagged question, that explains required’s behavior, but to paraphrase:
required checks that the field is not the field types default value, for bool false, int 0, string “” and so forth.
It doesn’t make much sense on a bool unless you need it tri-state and for that you would use a pointer to a bool. In fact if you have optional variables that’s how you would handle if they were actually set or not as the default value of a pointer is nil(yes nil is a type)


NIFI-3003 Upgrading hadoop.version to 2.7.3 & cleaning up InstanceClassLoaders

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Fixed Issue with PutHDFS when using 2.7.3 with TDE
Ensuring clean up of instance class loaders
Adding classpath resource property to all HDFS processors

hadoop-auth is being dragged to an older version in the hbase nar.  Should align hadoop-auth in both hadoop nar and hbase nar.

PR updated to address feedback from @olegz, and incorporated licensing patch posted by @joewitt in JIRA

Squashing and merging


Community proposal: Adjust veryfying-workers to 3/4 of the proving threads IF these are set.

The community has a proposal that we should set the verifying-workers to 3/4 or all of the proving threads IF these are set.

This would remove the unexpected 'side effect' of one node finishing proof then competing for resources against nodes still generating proof.
This could be a signifcant contributor to why users - including myself - correlate ATX processing with being unable to finish proof in time.

For now, we defaulted to 1/2 cores and added a dedicated section in smesher-guide: https://github.com/spacemeshos/wiki/wiki/Smesher-Guide#performance-fine-tuning-of-a-node


The lines of the timing diagram cannot be displayed

What would you like to be added:
如图所示，选择图表样式为时序图时只会显示最近的2个点，不会显示近期时间段内连续数据的线条。
Why is this needed:

issue timeout..


Parser error on interface with ignore default

When parsing an interface I get following parser errors:

I think these are related to following properties that can be enabled:

By activating these properties following statements are added to the interfact definition:

@Gekke-Jakke try again, version 0.29.18
https://larshp.github.io/abaplint/viz/methoddef.svg

@larshp Works perfectly! Thanks for the update!


Cannot run Rosbag replay simulation

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
Cannot run Rosbag replay simulation.
[gnss_poser-15] 2022-09-28 15:53:40.435 [RTPS_MSG_IN Error] (ID:139903409710656) Problem reserving CacheChange in reader: 01.0f.eb.7d.84.e9.a0.1e.<IP_ADDRESS>|<IP_ADDRESS> -> Function processDataMsg
[gnss_poser-15] 2022-09-28 15:53:40.436 [RTPS_MSG_IN Error] (ID:139903409710656) Problem reserving CacheChange in reader: 01.0f.eb.7d.84.e9.a0.1e.<IP_ADDRESS>|<IP_ADDRESS> -> Function processDataMsg
[component_container_mt-46] 2022-09-28 15:57:31.616 [RTPS_MSG_IN Error] (ID:140167985751616) Problem reserving CacheChange in reader: 01.0f.eb.7d.7e.f1.96.fd.<IP_ADDRESS>|0.0.1e.4 -> Function processDataMsg
[component_container_mt-46] 2022-09-28 15:57:31.617 [RTPS_MSG_IN Error] (ID:140167985751616) Problem reserving CacheChange in reader: 01.0f.eb.7d.7e.f1.96.fd.<IP_ADDRESS>|0.0.1e.4 -> Function processDataMsg

It is not just above ones, but every node is reporting this error. It seems like something wrong with dds.
Expected behavior
Able to run Rosbag replay simulation.
Actual behavior
Error reported.
Steps to reproduce
Follow instructions in doc. doc page
Versions
ghcr.io/autowarefoundation/autoware-universe:humble-latest-cuda
Possible causes
No response
Additional context
No response

log.txt
Log file attached.

The issue is related to FastDDS.
export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp

This could solve the issue.


Get Sensor from frame

Change interface: add function rs2_get_frame_sensor() for C interface, frame::get_sensor(), sensor_from_frame() for C++ interface and Frame::Sensor for C# interface.
This is used for obtaining the sensor of a frame.
Add demo usage in rs-data-collect and cs-tutorial-2-capture.

C:\Users\travis\build\IntelRealSense\librealsense\wrappers\csharp\cs-tutorial-2-capture\Window.xaml.cs(84,33): error CS0103: The name 'txtTimeStamp' does not exist in the current context

@doronhi - I think you either forgot to git add Window.xaml or to exclude changes to Window.xaml.cs


Problems with continuous operation

KRProgressHUD.dismiss {}
KRProgressHUD.showImage(#imageLiteral(resourceName: "toast_error"), message:  message)

the hud can't display.

@springlo
Please use it.
KRProgressHUD.dismiss {
    KRProgressHUD.showImage(#imageLiteral(resourceName: "toast_error"), message:  message)
}

No response.


added Kanhaiya game

@professor1412 add game screenshots

Done. I added screenshots already
On Thu, Oct 19, 2023, 00:41 Vinayak Sharma @.***> wrote:

@professor1412 https://github.com/professor1412 add game screenshots
—
Reply to this email directly, view it on GitHub
https://github.com/ossrndc/Hacktoberfest/pull/13#issuecomment-1769162531,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCRTLBWYFJZBDTQZEJZGR6TYAASWDAVCNFSM6AAAAAA6CSIIDKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRZGE3DENJTGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

add them in pr description


Commands from files not displaying description

I'm experimenting with the new command features and not having much luck getting the command description to display with --help when using the command():file() function.
Is this a known issue or am I using it wrong?  I've tried a couple different things, and haven't gotten it to work.
For what it's worth, I'm seeing the same behavior with the examples/04_commands--git.lua sample.

Hmm, I seem to be able to get the help listing print out for that example though. Can you provide some more context or code please? Here are some invocations I tried which seem to be working fine:
# without any command
lua_cliargs$ lua ./examples/04_commands--git.lua --help
Usage: git

COMMANDS:
  diff Show changes between commits, commit and working tree, etc
  log

# diff --help
lua_cliargs$ lua ./examples/04_commands--git.lua diff --help
Usage: git diff [OPTIONS] [--] [path-1 [path-2 [...]]]

ARGUMENTS:
  path        This form is to view the changes you made relative to the
              index (staging area for the next commit) (optional)

OPTIONS:
  -p, --patch This form is to view the changes you made relative to the
              index (staging area for the next commit)

# log --help
lua_cliargs$ lua ./examples/04_commands--git.lua log --help
Usage: git-log [OPTIONS]

OPTIONS:
  --[no-]follow Continue listing the history of a file beyond renames
                (works only for a single file). (default: off)

Also, --help diff and diff --help seem to be interchangeable.

Indeed, I meant to specify that the "log" command doesn't display the command description like "diff" does.  The log command comes from a separate file, while the diff command is defined in the main one.
Perhaps I am misunderstanding that it should display a description at all, but I expected to be able to supply one for the external file and see it show up in the help screen with the commands from the main file.


Cell merge and Image processing

Hi, Does it support cell merge and image processing, i mean image insert on the cell ? please let me know.
hansoo

Hi @iryna7
Sorry for the late reply.
To answer your question, cell merge and insertion of images is unfortunately not possible. OpenXLSX is mostly for data extraction, not so much for formatting. Cell merge may come later, but insertion of images is not on the todo list right now.


Cr 1190 backoff when rabbit down

Motivation and Context
CR-1190 fixes the problem when rabbit goes down and the Service stops responding since it is overwhelmed by the effort of calling RabbitMQ that has gone away.
What has changed

create, configure and pass a Circuit Breaker object to event-publisher, to reduce churn on sending
configure custom recovery backoff to reduce churn on the listener.

How to test?

take rabbit down while listening or sending.
see information in the JIRA for the testing in the performance environment.

Links
https://collaborate2.ons.gov.uk/jira/browse/CR-1190

Added a little javadoc into the MessagingConfig in response to Pete's comment above.


clarify who can be assigned GPU for spaces

update spaces docs to clarify that individuals with spaces repos can also request to be assigned a gpu

Thanks for the PR! I left a minor comment and we can merge afterwards 🚀

thanks, updated

Awesome :fire:


Linux ElasticPremium plan is not yet supported

The workflow working for Functions Consumption plan does not work for Functions Premium plan.
Github Action step:
..............
    - name: 'Login via Azure CLI'
      uses: azure/login@v1
      with:
        creds: ${{ secrets.AZURE_RBAC_CREDENTIALS }}

    - name: 'Deploy Azure Functions Action'
      uses: Azure/functions-action@v1
      id: fa
      with:
        app-name: ${{ env.AZURE_FUNCTIONAPP_NAME }}-${{ env.ENVIRONMENT }}
        package: ${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}
..............

Result:
Run Azure/functions-action@v1
##[Initialize]
##[ValidateParameter]
##[ValidateAzureResource]
Using RBAC for authentication, GitHub Action will perform resource validation.
Sucessfully acquired site configs from function app!
Detected function app sku: ElasticPremium
Sucessfully acquired app settings from function app (RBAC)!
Detected function app language: Python
##[error]Execution Exception (state: ValidateAzureResource) (step: ChangeContext)
##[error]  At ValidateAzureResource, Function Runtime : Linux ElasticPremium plan is not yet supported.
##[error]Deployment Failed!

Hi,
I also have a Linux Function App running on Premium plan and managed to deployed successfully.
Below is my setup:

I am not using RBAC Credentials but rather the publish profile

      - name: 'Run Azure Functions Action'
        uses: Azure/functions-action@v1
        id: fa
        with:
          app-name: ${{ env.AZURE_FUNCTIONAPP_NAME }}
          package: '.'
          publish-profile: ${{ secrets.SCM_PROD_CREDENTIALS }}

Also, I needed to add the setting WEBSITE_RUN_FROM_PACKAGE with the value set to 1 in the configuration of my Function App

To add to my previous comment, deployment was successful, however I can't see any function listed. It appears package was deployed but function app is not working as expected. Any idea?

I'm experiencing a similar problem after upgrading from the Dynamic Y1 to the Premium P1v2 tier. The deployment seems to be successful but I can't see any function listed, neither a new blob in my Storage Account.
Initially, I set the WEBSITE_RUN_FROM_PACKAGE to an empty string ("").
This is my setup and I'm using RBAC.
- name: Deploy to Azure Function
        uses: Azure/functions-action@v1
        with:
          app-name: my-amazing-app

And I can confirm that after switching back to the Dynamic Y1 tier, everything works as expected.

I also can confirm that after switching to a Windows Premium plan it works as expected. So definitely, it seems deploying to a Linux Premium plan isn't supported.
Couldn't find anything in the documentation about the support of it.

There's also a difference when deploying to Y1 and P2v2 functions. Seems like P2v2 doesn't use WEBSITE_RUN_FROM_PACKAGE.
Y1
Detected function app language: Node
##[PreparePublishContent]
Will archive . into /home/runner/work/_temp/temp_web_package_31360001219416156.zip as function app content
Will use WEBSITE_RUN_FROM_PACKAGE to deploy
##[PublishContent]
##[ValidatePublishedContent]

while P2v2
##[PreparePublishContent]
Will archive . into /home/runner/work/_temp/temp_web_package_6074844557873911.zip as function app content
Will use api/zipdeploy to deploy (rbac authentication)
##[PublishContent]
Package deployment using ZIP Deploy initiated.

Is there any way to force WEBSITE_RUN_FROM_PACKAGE?

Thanks for reporting this @michal-halenka-datasentics.
I amended the GitHub action and release the new version v1.1.5 today.
Feel free to reopen this thread if the issue still exists.


CLI enable environment selection

I would like to be able to define what environment the CLI should be working against currently it seems hardcoded to only work with Production env
A good solutions is is an argument which can be set as part of command
so for exmaple to check your balance, you would do :

nexmo balance --host "URL"
18.96 EUR

@ramzeaslan great idea. Would the host url be a full url, or just a domain, or also a port? Would this url be predictable? If so can we just do nexmo balance --host dev?

@cbetta I think it'll be a host, probably including scheme e.g. http://internal-api.nexmo.com

I reckon having the full host makes the most sense, otherwise we are restricting ourselves and as such are less future proof.
Another alternative which is worth considering is that as part of the setup of the Nexmo CLI you are setting your API key and Secret, what if you also set your env as part of the setup?
Thanks

@ramzeaslan interesting. Does make things easier to use. Let me think about it.

It feels like we're moving more towards a per-project config rather than global config that you'll have to change as you move between environments.
IMO https://github.com/Nexmo/nexmo-cli/issues/109 could be a related to this. How nexmo-node enables it is a good question. But this discussion is really related to the CLI rather than nexmo-node.
To enable this we just need to make the host configurable within nexmo-node.

I had not even noticed this issue wasn't on the CLI project. Let's close this one and re-open on the CLI library.

Closing, have opened https://github.com/Nexmo/nexmo-cli/issues/114 instead.


PyMailer (Mail Sender GUI)

Description
Added PyMailer (GUI Based Mail Sender) written in Python (PySide2 + smtplib), Solved Issue #4
Fixed #4
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added screenshots(if any) that prove my fix is effective or that my feature works

@all-contributors please add @FireHead90544 for code, doc


Update enable-dynamic-configuration-aspnet-core.md

Added clarification for configuration caching and refresh.

@zhenlan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

cc: @jimmyca15 @avanigupta for review too

@zhenlan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@AlexandraKemperMS @jimmyca15, any feedback?

@zhenlan I think it looks great, just a few comments.

@zhenlan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@zhenlan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@zhenlan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@zhenlan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@zhenlan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@zhenlan: I'm sorry - only the author of this article, @AlexandraKemperMS, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

@ktoliver how can we move forward with this PR? I have customers waiting for this doc update.

@zhenlan Publishing in this repository is suspended until Tuesday, May 25. Merging will resume Tuesday, and we'll merge this approved pull request at that time.
Ask mode is in effect for the private repos; I'll send you the information and you can pursue that route if it's relevant for your team.


ORC-377: [c++] Add SnappyCompressionStream

Currently ORC C++ only support writing with Zlib compression
This branch aims to add support for Snappy compression

Can you add some tests for this?

Can we reopen the pull request? We are interested in adding the tests and getting it into ORC since my ongoing Arrow2ORC binder won’t be used that much if Snappy write is not supported in the ORC C++ lib.

Now I have time to work on that since I have finished the C++/Python ORC writer in Arrow.

Now I have time to work on that since I have finished the C++/Python ORC writer in Arrow.

Hey @majetideepak thanks for taking this up!
Looks that this has been inactive for a while, feel free to open a new PR

Hey @majetideepak thanks for taking this up!
Looks that this has been inactive for a while, feel free to open a new PR

@mathyingzhou feel free to re-open this PR.

@mathyingzhou feel free to re-open this PR.

This PR is dormant for a while now. I could open a new one for adding Snappy compression, if it's okay with everyone.
The task looks very similar to the LZ4 compression that I created recently.

@noirello Sure! I have three more Arrow issues to cover before I can work on this. It will be nice if you can do it this month. Otherwise I will.


Error when using url route as

Hi,
Thank you for this module, but I facing an issue.
When I name my route like this:
Route::get('/', [
        'uses' => 'PageController@index',
        'as'   => 'page.index',
    ]);

The module try to translate the url with the 'as' path which gives to me an errors because it's returning an array.
The error is line 219 from Router.php on the ltrim function (get an array expected string).
I'm trying to debug this but as I'm newbie with laravel I can't figure out how app()['translator'] works
Any idea to make it working with custom route name ?

Hi Xoce51,
thank you for reporting, I will take a look.

I'm not sure to understand your issue. Can you please provide the error trace?
Which routing function are you using to translate the URL ?

I was using this function:
Router::getCurrentVersions()

after some debugging, I saw that's I must have entries within
resources/lang/[lang]/routes.php
to use this function, is-it correct ?
I'm sorry I haven't any stacktrace because I temporarly desactivate the module.

Yes that's right you have to use language files in resources/lang to translate your URLs.
Here is an implementation example:
Language files
# resources/lang/en/routing.php

return [
    'welcome' => 'welcome'
];

# resources/lang/de/routing.php

return [
    'welcome' => 'wilkommen'
];

# resources/lang/fr/routing.php

return [
    'welcome' => 'bienvenue'
];

# resources/lang/en/global.php

return [
    'title' => 'Welcome to the English version',
    'text' => 'Text in English ... bla bla bla',
    'language.en' => 'English',
    'language.de' => 'German',
    'language.fr' => 'French',
];

# resources/lang/de/global.php

return [
    'title' => 'Willkommen in der Englische Version',
    'text' => 'Text auf Deutsch ... bla bla bla',
    'language.en' => 'Englisch',
    'language.de' => 'Deutsch',
    'language.fr' => 'Französisch',
];

# resources/lang/fr/global.php

return [
    'title' => 'Bienvenue en version francaise',
    'text' => 'Texte en Francais ... bla bla bla',
    'language.en' => 'Anglais',
    'language.de' => 'Allemand',
    'language.fr' => 'Francais',
];

Routes
# app/Http/routes.php

Route::get(Router::resolve('routing.welcome'), 'HomeController@index');

Controller
# app/Http/Controllers/HomeController.php

namespace App\Http\Controllers;

class HomeController extends Controller {

	public function index()
	{
		return view('welcome');
	}

}

View
# rerources/views/welcome.blade.php

{{ Lang::get('global.title') }}

{{ Lang::get('global.text') }}

@foreach(Router::getCurrentVersions() as $locale => $url)

        {{ Lang::get('global.language.' . $locale) }}

@endforeach

HTML output
And here is the HTML output you should have while visiting http://en.yourdomain.com/welcome:

Welcome to the English version

Text in English ... bla bla bla

        German

        French

Please let me know if this helps solving your problem.

Ok, thank you for these precision, my fault sorry I misused the program. I'm gonna take a look at this in a few days. Thank you again.


[DRAFT] feat: Support using web3.js

Breaking Changes
N/A
Changes

feat: Support using web3.js
fix:
chore:

Associated Issues
closes #...

Basic tests are passing. However, the rest still need some work... The code is still also as a draft...

SIWE tests are also passing now. (However, still more tests and code fixes to be done)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: Muhammad-Altabba:x: enesozturk:x: tomiirYou have signed the CLA already but the status is still pending? Let us recheck it.


remove unnecessary space between back and end

Hello! here is a PR for just removing a space between back and end.
.. the back end Pods for ... -> ... the backend Pods for ...
markdown : virtual-ips.md
kubernetes.io : virtual-ips

/lgtm
/approve


Draft: update notebooks

updated config to new version in tutorial notebooks

@alytarik please also add 1 line in the website tutorials section table to show the new notebook


hello 0.1.0-59899513+autotest_by_mosteo@kukumo

Created via alr publish with alr 2.0.2

Closed using alr 2.0.2 with reason: 'automated testing'


Stop otbr-agent for occupied dongle device process

Stop otbr-agent for occupied dongle device process
https://github.com/Samsung/ioter/issues/94

@bermanRon
Thanks for contribution.


CMake generation fails with -- Thread #0: error during evaluating variables: no such file or directory

I'm not sure what to look for, could you help?
This cppan.yaml worked fine for me:
local_settings:
    use_shared_libs: false

    dependencies:
        pvt.cppan.demo.boost.test: 1.65
        pvt.cppan.demo.boost.range: 1.65
        pvt.cppan.demo.boost.utility: 1.65
        pvt.cppan.demo.boost.iterator: 1.65
        pvt.cppan.demo.reactive_extensions.rxcpp: 4.0.0

But this one fails (additional pvt.cppan.demo.boost.iostreams line):
local_settings:
    use_shared_libs: false

    dependencies:
        pvt.cppan.demo.boost.test: 1.65
        pvt.cppan.demo.boost.range: 1.65
        pvt.cppan.demo.boost.utility: 1.65
        pvt.cppan.demo.boost.iterator: 1.65
        pvt.cppan.demo.boost.iostreams: 1.65
        pvt.cppan.demo.reactive_extensions.rxcpp: 4.0.0

Cmake cout with error is (the folders for internal-parallel-vars-check exist, I checked):
E:\Win81Users\UserName\Source\Repos\MyProject>"c:\Program Files\CMake\bin\cmake.exe" -G "Visual Studio 15 Win64" -BDebug -H. -DCMAKE_BUILD_TYPE=Debug
-- The C compiler identification is MSVC 19.11.25547.0
-- The CXX compiler identification is MSVC 19.11.25547.0
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Performing 47 checks using 8 thread(s)
-- This process may take up to 10-20 minutes depending on your hardware
-- Thread #0: error during evaluating variables: no such file or directory
 0# 0x0141F079 in cppan
 1# 0x014251DE in cppan
 2# 0x0174DE8F in cppan
 3# 0x0174DFDA in cppan
 4# 0x0174DF2A in cppan
 5# set_app_type in ucrtbase
 6# BaseThreadInitThunk in KERNEL32
 7# RtlGetAppContainerNamedObjectPath in ntdll
 8# RtlGetAppContainerNamedObjectPath in ntdll

CMake Error at E:/Win81Users/UserName/.cppan/storage/etc/static/functions.cmake:676 (message):
  Last execute_process() with message 'COMMAND
  E:/Win81Users/UserName/Source/Repos/MyProject/cppan.exe

                                          internal-parallel-vars-check
                                              "E:/WIN81U~1/UserName/AppData/Local/Temp/cppan/vars/yyCulDq3"
                                              "E:/Win81Users/UserName/.cppan/storage/cfg/6d40d2d1.cmake"
                                              "E:/Win81Users/UserName/Source/Repos/MyProject/.cppan/checks.yml"
                                              "Visual Studio 15 2017 Win64"
                                              ""
                                              ""
                                          ' failed with error: 1
Call Stack (most recent call first):
  .cppan/helpers.cmake:190 (check_result_variable)
  .cppan/CMakeLists.txt:106 (include)

-- Configuring incomplete, errors occurred!
See also "E:/Win81Users/UserName/Source/Repos/MyProject/Debug/CMakeFiles/CMakeOutput.log".

Maybe you could try newer client? Not uploaded yet to website. It's RelWithDebInfo, so size is a bit bigger.
cppan.zip

Thanks, definitely!
Unfortunately, it did not help.
CMake seems to have made a bit more progress, but still got the same error:
E:\Win81Users\UserName\Source\Repos\MyProject>"c:\Program Files\CMake\bin\cmake.exe" -G "Visual Studio 15 Win64" -BDebug3 -H. -DCMAKE_BUILD_TYPE=Debug
-- The C compiler identification is MSVC 19.11.25547.0
-- The CXX compiler identification is MSVC 19.11.25547.0
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Check if the system is big endian
-- Searching 16 bit integer
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of unsigned short
-- Check size of unsigned short - done
-- Using unsigned short
-- Check if the system is big endian - little endian
-- Performing 49 checks using 8 thread(s)
-- This process may take up to 10-20 minutes depending on your hardware
-- Thread #0: error during evaluating variables: no such file or directory
 0# 0x013D1099 in cppan
 1# 0x013D987E in cppan
 2# 0x0187450E in cppan
 3# 0x018745D6 in cppan
 4# 0x01874700 in cppan
 5# 0x0187443E in cppan
 6# 0x01872DB1 in cppan
 7# 0x01872CCA in cppan
 8# set_app_type in ucrtbase
 9# BaseThreadInitThunk in KERNEL32
10# RtlGetAppContainerNamedObjectPath in ntdll
11# RtlGetAppContainerNamedObjectPath in ntdll

CMake Error at E:/Win81Users/UserName/.cppan/storage/etc/static/functions.cmake:676 (message):
  Last execute_process() with message 'COMMAND
  E:/Win81Users/UserName/Source/Repos/MyProject/cppan.exe

                                          internal-parallel-vars-check
                                              "E:/WIN81U~1/UserName/AppData/Local/Temp/cppan/vars/sXX25vnR"
                                              "E:/Win81Users/UserName/.cppan/storage/cfg/6d40d2d1.cmake"
                                              "E:/Win81Users/UserName/Source/Repos/MyProject/.cppan/checks.yml"
                                              "Visual Studio 15 2017 Win64"
                                              ""
                                              ""
                                          ' failed with error: 1
Call Stack (most recent call first):
  .cppan/helpers.cmake:190 (check_result_variable)
  .cppan/CMakeLists.txt:106 (include)

-- Configuring incomplete, errors occurred!
See also "E:/Win81Users/UserName/Source/Repos/MyProject/Debug3/CMakeFiles/CMakeOutput.log".

Did you try to remove storage?

Have just deleted .cppan folder from my user folder. Did not help.
Did you mean something else?

No, that's correct.
You example works for me.
Code related to these checks is very ugly. I also saw such errors some time ago, I fixed some of them, but it seems not all.
This time probably I can't help you fast. I'm focusing on cppan2, so it's not really worth it to dive into such tricky errors.

You example works for me.

I hope you mean this code (you lose some formatting when pasting here):
local_settings:
    storage_dir: s
    use_shared_libs: false

dependencies:
    pvt.cppan.demo.boost.test: 1.65
    pvt.cppan.demo.boost.range: 1.65
    pvt.cppan.demo.boost.utility: 1.65
    pvt.cppan.demo.boost.iterator: 1.65
    pvt.cppan.demo.boost.iostreams: 1.65
    pvt.cppan.demo.reactive_extensions.rxcpp: 4.0.0

One more thing is to check your examples with VS Community. As I said cppan (or cmake) has issues with VS Enterprise or multiple instances.

Formatting was as I just posted here (so, apparently, incorrect), and I've fixed it now, but it did not help.
I won't be able to try Community studio on this machine, but is there something else I could try?
Thanks for quick responses, even if you can't fix this particular problem :)
It is not a deal breaker for me right now. I was just hoping to help you (or me) to find the problem, and to know how to avoid this in the future.

internal-parallel-vars-check
                                              "E:/WIN81U~1/UserName/AppData/Local/Temp/cppan/vars/sXX25vnR"

Path with ~ maybe handled incorrectly.

Do you know where this path could be coming from, or how to avoid it?

As I understand this dir comes from inside of cppan <- boost::fs.
What you can try is to find a file with this line (somewhere in storage/tmp/... or in your local build folder), change path, remove storage/cfg, CMakeCache.txt in build dir and try to run cmake directly on main generated cmakelists.txt script without re-running cppan as it will replace folder again.

You see... Things are already overly complicated. I don't like this. It's not productive.
(cppan2 will help us. :) )

BTW, if you able to build cppan, you can check this place
https://github.com/cppan/cppan/blob/master/src/printers/cmake.cpp#L2903
${tmpdir} is formed with temp_directory_path() which is easy to track down to place where it gets ~ dir.

Thanks, I will look into this :)
Again, this is not a huge issue for now, so I won't spend too much time on it too.
And I sure am hoping that cppan2 is going to happen in foreseeable future :)

Same issue here.
even with the standard project cppan/tesseract_example

Ok, looks like I need to install VS Ent. to investigate the issue.

@egorpugin i got the same issue on a other environnement with only the Community editions installed

If you still experience this issue, try to install the latest cmake and cppan.
For me it works fine on multi-VS installation (comm., ent., comm. preview).
Feel free to reopen.

As discovered in another issue (#45 ), this could be because cmake was not in the PATH.


Multi-part extensions should not infect JSON

fMRIPrep produces outputs named sub-19_task-MerlinMovie_space-fsLR_den-91k_bold.dtseries.json to match sub-19_task-MerlinMovie_space-fsLR_den-91k_bold.dtseries.nii.
In the spec, the example given is:
pipeline1/
    sub-001/
        func/
            sub-001_task-rest_run-1_space-fsLR_res-1_den-10k_bold.dtseries.nii
            sub-001_task-rest_run-1_space-fsLR_res-1_den-41k_bold.dtseries.nii
            sub-001_task-rest_run-1_space-fsLR_res-2_den-10k_bold.dtseries.nii
            sub-001_task-rest_run-1_space-fsLR_res-2_den-41k_bold.dtseries.nii
            sub-001_task-rest_run-1_space-fsLR_bold.json

PyBIDS also looks for sidecars with the extension .json, not .dtseries.json.
A first pass at this was taken in https://github.com/nipreps/niworkflows/pull/558 but dropped.
There was concern about not having sufficient disambiguation, so if we still can see that, then we need to make an issue on the spec. For my part, I would prefer not to change the spec, but to ensure that we always provide enough entities to ensure that no ambiguity is possible.
What version of fMRIPrep are you using?
20.2.0

@bpinsard There's a question of whether this should be handled in 20.2.1 or 20.3.0. It will change the derivatives, but I think the current derivatives are in error. Therefore I think it would be acceptable to fix in the LTS series.
An uglier but more conservative option would be to could keep the .dtseries.json files with their current data, and add .json files with valid BIDS keys.

I agree - I think this is a 20.2.1 issue. I'm inclined to go the conservative route, since any breaking derivatives change should be avoided in the LTS. @bpinsard WDYT?

The question is whether the current outputs are in error. We permit changes to derivatives that remove or modify outputs that are in error.
If we do want to use the conservative route, I think the strategy needs to be to declare a frozen set of keys that will continue to go into .dtseries.json, and then all others go into .json.

@mgxd Let's go with the conservative route for now. If it's acceptable to remove the "invalid" JSON files now, it's acceptable to remove them later.


Dependency Management - downgrade the networkx to work with Python2

Fixes #1458
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@bblommers thanks for your bug report & contribution!


TVM Vertical Integration with PyTorch

The pull request contains two functions:

optimize_torch as a function similar to torch.jit.trace, which is used to optimize the torch.nn.module by TVM metaSchedule, and returns a custom TorchScript operator
as_torch as a decorator, which is used to wrap the TVMscript code to torch.nn.module.

The files consist of:

Two python codes for both functions and a C++ backend
Two test files for testing both functions
Two how-to guides showing the usage.

@yelite @junrushao1994 @masahi

I suggest moving tutorials to a separate PR. Ideally, tutorials should demonstrate more realistic examples than vector add or matmul, i.e. something PyTorch users would reach for "custom op" authoring.
For example, I think demonstrating equivalent of Triton fused softmax tutorial https://triton-lang.org/master/getting-started/tutorials/02-fused-softmax.html in this workflow would be very interesting.


feat(table): refactor table component for conditional rendering

Summary
Brief summary describing the changes here.
ADO Story or GitHub Issue Link
Link here (if applicable).
Notes

Detailed change notes
can go here

To Do

Anything else
left to do

Checklist

[ ] Used Conventional Commit messages as outlined in the contributing guide.

[ ] Noted breaking changes (if any).

[ ] Documented/updated all props, events, slots, parts, etc with JSDoc.

[ ] Ran the analyze command to update Storybook docs.

[ ] Added/updated Stories with controls to cover all variants.
[ ] Added/updated a basic unit test, at least, for each component.
[ ] Added/updated the Figma link for the Story's Design tab.
[ ] Added any new component exports to the src/index.ts file

Testing Instructions
Provide guidance to reviewers/testers here.
Screenshots
(if any)

This is looking pretty great, thanks for all your hard work!
Issues I see:

Selectable version - row checkboxes are not aligned with header checkbox.

Here's a few things I think we still need:

Try to restore the deleted components and stories, and avoid any breaking changes if possible as we discussed after the demo. This is what I'm most worried about.

If breaking changes are unavoidable:

Make sure your commit messages call out each breaking change.
Point this PR to beta branch, as we'll have to start a 2.0 release.

Add a Story that shows the table with Pagination. People will be looking for that.

The example code is hard to find with the Storybook changes. I would suggest adding links to the sample code/component file on GitHub somewhere in the Docs.

We'll also need a Story in the Global Filter that shows it working with a Table. Can maybe follow up on that after this release.

@brian-patrick-3
Thank you!
Regarding the checkboxes issue - I wasn't aware of it since it only happened on FireFox but it's fixed now. I tested on all browsers (chrome, safari on Mac, chrome, firefox and edge), and it's currently working fine on all of them.
Regarding things that we still need:

Breaking changes: I double-checked and there's no breaking change. They can keep using the deprecated data table and old basic table. Old stories are now restored with just a tiny change.
Pagination stories added to the new table.
Link added. Later I can provide my demos in React, Svelte (and probably Angular) so it's easier for them to follow.
Sure thing!


extra environment variables for configuring wsUrl (e.g., "wss://" instead of "ws://")

in order to serve my Lightspeed-react Docker container over https, i've found that i need to configure wsUrl so that it uses the "wss://" scheme instead of "ws://". i also need to set the host to an actual domain name rather than an IP address, presumably because my SSL certificate is only valid for the former.
this patch makes it possible to do this by allowing the user to set WEBSOCKET_SCHEME and WEBSOCKET_HOSTNAME environment variables. if these are not set, they are replaced with "ws://" and $WEBSOCKET_HOST respectively, so the patch should be backwards compatible.

Looks good!  I just want to test it and wont have time until either tomorrow or Friday but once I test I will merge. Thank you!


bump kubevirtci

Signed-off-by: Or Mergi<EMAIL_ADDRESS>
What this PR does / why we need it:
Bump kubevirtci includes, the following changes:

(tag:<PHONE_NUMBER>-b2383d1) Revert "config-sriov.sh: reduce wait time for operator readiness (#469)" (#488)
(tag:<PHONE_NUMBER>-2a57af4) KIND infra, SRIOV proivder,  Run in-memory etcd (#478)
(tag:<PHONE_NUMBER>-cee24ed) cnao, Update CNAO manifests to v0.42.3 (#483)
(tag:<PHONE_NUMBER>-e977c22) sriov, Update retry function description (#485)
(tag:<PHONE_NUMBER>-c35f52a) cnao, Add KUBVIRT_WITH_CNAO_SKIP_CONFIG (482)
(tag:<PHONE_NUMBER>-dae5d6b) sriov, Fix non allocatable PF list (#480)
(tag:<PHONE_NUMBER>-c2cf403) SRIOV provider, csrcreator: switch to self-sign certificates (#472)
(tag:<PHONE_NUMBER>-73a1177) config-sriov.sh: reduce wait time for operator readiness (#469)

Notable changes:

KinD infra providers now run etcd in RAM memory, enable with new env var KUBEVIRT_WITH_KIND_ETCD_IN_MEMORY
CNAO bumped to 0.42.3
Added a new env KUBVIRT_WITH_CNAO_SKIP_CONFIG that will only deploy CNAO without the
NetworkAddonsConfig CR
SRIOV provider, fix bug where blacklist PF were handled.
SRIOV proivder, use self-signed certificates instead of CSR.

Which issue(s) this PR fixes:
Fixes:

https://github.com/kubevirt/kubevirt/issues/3742
https://github.com/kubevirt/kubevirt/issues/3777
https://github.com/kubevirt/kubevirt/issues/4519

Special notes for your reviewer:
Let me know if something should be added to the release note
Release note:

NONE

/test pull-kubevirt-e2e-kind-1.17-sriov

/cc @EdDev @qinqon @AlonaKaplan

SRIOV lnae is green
https://github.com/kubevirt/kubevirt/pull/4588

/retest

@ormergi can you add the kubevirtci PRs so want to have at PR description ?

/lgtm

/lgtm
/hold
Let's wait for SRIOV lane

/hold cancel
SRIOV lane is good

/approve


Import Failure Error 80160000

FYI current build is failing to import in vanilla 365 CE trial. Undocumented error code 80160000

Hi @harryepstein , put in your mind that you have to change the allowed attachment size, please read this and apply the fix, then try to import
https://community.dynamics.com/crm/b/crminthefield/posts/caveats-of-setting-the-maximum-file-size-limits-for-attachments-in-crm-2011

Hi @aboodh95 can you help me fix error code : 80160000, while importing a custom control. The above link for the fix is empty.

Hi @AJ46CODER ,
Sorry for that, try this https://bit.ly/3u4XZYZ


Made 'renderers' pluggable in the graphite reporter

Because many metrics are somewhat verbose and because of the nature of
Graphite, it´s desirable to be able to decide which fields that are
sent to graphite.
E.g for a metered metric, perhaps the only really interesting field to
graph is the one minute rate. With a polling time of one minute, this
would generate a neat and relevant graph in graphite.

This actually ties into #91. What I'd prefer to see is the modification of the existing predicate system to include the idea of metric property names in addition to metric names.

I'm not sure I understand, could you elaborate on the "idea of metric property names in addition to metric names"...

Right now a MetricPredicate matches on MetricName instances, but this doesn't allow it to filter, say, everything but the one-minute rate of meters. We should expand MetricPredicate to also be able to match on the actual property name (e.g., oneMinuteRate) in addition to the MetricName.

While contemplating on this (without being able to come up with a solution) I did an attempt to make MetricPredicate implement MetricsProcessor which enables predicates to work with typed metrics and choose a subset of types to filter, see this commit:
https://github.com/chids/metrics/commit/776a09c372ae3af71e315925619db49d5c5c94e2
@coda do you have any more concrete ideas on how to expand predicates to filter on properties?

I don't, sorry. This isn't a big issue for my team, so I don't have a lot of extra bandwidth to spend on it. (Naturally, if someone comes up with a nice fix, I'll merge it.)

This is covered by #178, broadly speaking, and is on the list for 3.0.0.


Create LICENSE

Saw that package.json licensed nesh as MIT, so I'm submitting a LICENSE.
Closes #32.

Props to @mislav who was giving a talk on this topic while I was looking at this repo. :wink:

@rgbkrk thanks! :+1:


add changelog at api and sdk level, abi version for sdk

Codecov Report

Merging #13 (184696d) into master (ec6b6c7) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
- Coverage   94.52%   94.50%   -0.02%
==========================================
  Files         178      178
  Lines        7701     7701
==========================================
- Hits         7279     7278       -1
- Misses        422      423       +1

Impacted Files
Coverage Δ

sdk/test/common/circular_buffer_test.cc
98.97% <0.00%> (-1.03%)
:arrow_down:


PHP syntax support

Will be added support for PHP ?
PHP is a popular language, I think it will help a lot of people

Sure. In the meantime, you can add it via the settings. Add "php" under "Extra file extensions".

Sure. In the meantime, you can add it via the settings. Add "php" under "Extra file extensions".

Thank you!


Add GCP adapter for S3 compatible interfaces and added data mover interfaces for GCP backend

This PR adds GCP adapter for S3 compatible interfaces and migration scenario from GCP to AWS also tested.

Please change the base branch to development

This PR should not contain a "merge" commit.  Please start the changes in the development branch, instead of merging changes from development to your branch.

@sfzeng Please take a look.


Support Google

I assume that its not possible to work with gmail or and such or it would already be added. I'm just curious what the reason is that prevents it? Is it just missing the right config file?

I actually just haven't gotten around to it, even though it is a major one. I will add it when I find the time or someone submits the XML file for it.
Here is a list of sites I eventually plan to add support for:
youtube, blogger, google
twitch
wordpress
last.fm
pandora
rogers
paypal - (javascript)
stackexchange
orbitz.com
expedia.com
living social
hoteltonight
dropbox - (javascript)
cerberus
gitourious.org
superuser.com
about.me
booking.com
codeproject.com
livejournal.com
last.fm
sourceforge.net
ubuntuforums.org
lavabit.com
mail.ru
hushmail.com
adobe.com
tripadvisor
uber
blizzard
steam
itunes (apple)
gettyimages


tools: AppSRE packer build

Nice! I put several comments inline. My biggest concern is that we are reusing the same VM for both building and running a production service. It just feels like too much. I have two ideas:

Run a dedicated EC2 instance just for building RPMs.
Run mock directly in the AppSRE environment in a privileged container (I just tried this locally, it perfectly works. The question is whether we can run privileged containers, I'm not sure if this is a VM, or not.

I would love it if 2 works... Agreed that doing it on the production system would not be nice. Did we consider copr or something like that? 1 is basically running our own copr, isn't it?

Nice! I put several comments inline. My biggest concern is that we are reusing the same VM for both building and running a production service. It just feels like too much. I have two ideas:

Run a dedicated EC2 instance just for building RPMs.
Run mock directly in the AppSRE environment in a privileged container (I just tried this locally, it perfectly works. The question is whether we can run privileged containers, I'm not sure if this is a VM, or not.

I would love it if 2 works... Agreed that doing it on the production system would not be nice. Did we consider copr or something like that? 1 is basically running our own copr, isn't it?

Copr is another option, the annoying this is the token needs refreshing every 180 days but other than that it would work of course. 2 will only work from a privileged container, mock is not installed, unsure if we have access to those, ppl from AppSRE were unsure too.

So mock is not an option; I guess it's ec2 or copr.

Went for the ec2 option. We don't have mock and unsure if we should rely on rhcopr, plus the expiring token thing even if the cadence is slow.

"we don't have mock"? doesn't this use mock?

"we don't have mock"? doesn't this use mock?

This uses mock in an ec2 machine yea, we don't have mock on the jenkins runner however. So first this starts an ec2 machine based on the same ami as the final image is based on, builds the rpms in mock, fetches them, does the packer build, pushes the rpm during the packer build and installs them.

Hm this breaks the packer test, i'll look into that. Unsure if we can/should keep that.


Add tray version support

Purpose
While fetching an application tray, we need to support multiple versions of the tray.
Proposal

[ ] update the apps tree to something like:

apps/{{ tray.name }}/{{ tray.version }}/templates/...

[ ] add versions to active apps, e.g.:

apps:
  - name: richie
    version: 1.0.0

[ ] collect apps given this version

Optional:

[ ] add default version support

Not sure this is relevant in fact. Will we deploy two different trays in the same project ?


User Feature Request: Change name of "Capture" tab below image display?

Is your feature request related to a problem? Please describe.
Right now the interface displays the word "Capture" in two places: the actual "Capture" button at the top of the video feed (green rectangle), and as the label of the tab that switches you from the live video feed to the side of the interface where you can select the model and classify the image (red rectangle). I think having the same word in two places for two slightly different things is confusing, and I often go and click on the large "Capture" tab at the bottom when I am intending to take a capture from the video feed.
Describe the solution you'd like
I think that "Capture" describes accurately what the button at the top does, which is taking a still from the live feed. I would like to suggest changing the name of the bottom tab from "Capture" to "Analysis" or something like that.
I would also like it if, when you click the capture button (green rectangle), it would automatically switch you to the "Capture"/"Analysis" side of the interface automatically, but that may be a different request.
Describe alternatives you've considered
Even "View Capture" would be slightly more clear.
Additional context

I think it's also inconvenient to have to go to the bottom of the screen to switch modes

Would something like tabbing between activity modes work?
Also the "Model Selection" Button text could instead indicate the model name, so the user does not have to click it to figure out what the active model is?

I like the idea of the model selection button text indicating the name of the active model.
In terms of the tabbing between activity modes, I don't have a strong opinion about where the button/tab is. I like the look of the tabs you demonstrate, though.  I would just like the "capture" action to also automatically switch you to "analysis" mode, or even to automatically trigger the analysis with the default/currently selected model, to create a one-click workflow.


Idea: Add gangs and mechanic garages

How difficult will it be to add the ability to have garages for mechanics and gangs in the same script? Please let me know. This will surely make this a fire script!

It would be easy to add things like, Ems, fire, mechanic and other jobs to it but gangs would require it to be more in depth with a connection to a database and then that gets into a different type of garage.

I'm going to rework the entire garage to be a job garage instead of only police


nette/utils is required for the custom rule example

I created my first own Rector and by the way I realised that the original example depend on nette/utils. I guess when someone unexpired will try this example he will be confused, because after installing rector as dev dependency you don't have nette/utils.
So my recommendation would be to get rid of nette/utils or add a hint that you've to install nette/utils as dependency.
https://github.com/rectorphp/getrector.org/blame/main/data/docs/custom-rule.md#L19

Hi, I think it would be better to remove the package from example.
Could you send a PR to handle it?

@TomasVotruba I'll do it :)

@reneroboter Thank you :hugs:


logform typings not exposed

Please tell us about your environment:

winston version? 3.2.1

[ ] winston@2
[X] winston@3

node -v outputs: 13.6.0
Operating System? (Windows, macOS, or Linux)
Language? (all | TypeScript X.X | ES6/7 | ES5 | Dart)

What is the problem?

try to create a function in typescript that return a format method
import { format } from 'winston';
import { FormatWrap } from 'logform';

const appendMetaInfo = (application: string): FormatWrap =>
  format(info => ({
    ...info,
    application,
    hostname: os.hostname
  }));

As you may notice, I have to import FormatWrap from logform cause it's not exposed in winston namepsace unlike format which come from logform too.
What do you expect to happen instead?

Would like to be able to import typings like the following :
import { format, FormatWrap } from 'winston';

You'll find this has already been fixed
and has been merged to the 'master' branch, (https://github.com/winstonjs/winston/blob/master/index.d.ts)
but has not been released yet!! Urgh!!!
(last publicly available version is 3.2.1, from January 2019)


Switching to docker-compose.yml V2 breaks nginx-proxy

Upon switching my docker-compose.yml to the V2 layout, nginx-proxy has stopped working. I poked around inside of the container and saw that inside of /etc/nginx/conf.d/default.conf there were upstreams like this:
upstream sub.hostname.org {
            # testservices_test_1
            server :4000;
}

It appears like nginx-proxy is no longer able to find the IP address for these containers, although I can't figure out what would be causing that. On reverting back to the V1 version of docker-compose.yml, everything fixed itself.

Yes people are working on fixing it, see #304

Oh great, thanks


Add optional custom config directory $AUTHY_ROOT

I imagine I am not the only one who tends to obsess over keeping my '$HOME' directory clean/organized.
I have modified the ConfigPath() getter function in ./service/device.go to check for the $AUTHY_ROOT environment variable.
If this variable is present, it will be used as the directory path to store the config and cache files.
Otherwise it will resort to its default path at $HOME.

Thx.


Viewset for get all assignments for a specific subject

Change the update logic (No update -> continue instead)

Done


Fix policy name

Currently, the way the policy name for the secret access is set, you can only create one single policy resource (because it has a static name)

Thank you @joaozanuttoqr for your participation. It will be fixed with #2