Travis build

Here's a first shot at a Travis build.  Let's see if it works.  Right now I have it set up so that PRs are built and tested.  A package will be published for every tagged commit on master.  I'm proposing that for releases, we use the "create a new release" feature on Github, which tags a build and generates tarballs, etc.  This, in turn will automatically trigger a Travis build that publishes the package and docs to PyPI.  If we do decide to go this route, I can add a check wherever the version is set in the source code to look for the version as the current git tag in an environment variable that Travis already provides.
The alternative would be to allow every commit into master to turn into a release.  I avoided that because I think it would require us to more tightly and manually manage version numbers, which would be a pain.
Let me know what you think.
TODO:

[x] Decide on versioning mechanism.
[x] If needed, set version in code to check for git tag in Travis environment variable.

/cc @huyng @pierreg @pumpikano

CLA is valid!

Current coverage is 75.26% (diff: 100%)

No coverage report found for master at 3898f4f.

Powered by Codecov. Last update 3898f4f...5ddbec5

OK, @huyng @pumpikano @pierreg.  I think this is ready to go.  The PR ended up having a number of changes, but they fall into three classes:

A Travis build defined in .travis.yml.  I built for Python 2.7, 3.4, and 3.5, mainly because matplotlib (a dependency) doesn't install for other Pythons.  If we want to change this (e.g. by eliminating the matplotlib dependency), we can do it in a different PR.

Fixes to the code to get the build passing, mainly for Python 3.

Implementation of the versioning system I described above.  In particular, the release build will be triggered every time we tag the master branch, which I propose we do via Github's releases feature.  Everywhere in the code that the package's version is used (in setup.py and in conf.py for the docs build), I check to see if the TRAVIS_TAG environment variable is available.  This will hold the name of the git tag in a travis build.  If that environment variable is not available, it just uses a default version (right now 1.0).  This means that when we tag master, we should tag it with a proper version number, e.g. '1.0.2' or 'v1.0.2' (either form will be correctly parsed).  We can also do release candidate builds with tags like this: '1.1rc1' or '1.1-rc1' or 'v1.1rc1' or 'v1.1-rc1' (these will all be parsed correctly).  Every time we tag a version like this, it'll be built and automatically published to PyPI.

Let me know what you think.

I think this set up is great! 👍
Is matplotlib a dependence? - I don't see it in setup.py, but I do see pydot. In any case, this is just for plotting the graph, right? I think we should have this as an optional dependency in that case.

I added matplotlib as a dependency (in setup.py---it was already a dependency in the code) in this PR: https://github.com/yahoo/graphkit/pull/3/files#diff-2eeaed663bd0d25b7e608891384b7298R30.  But yeah, it is only for the plotting stuff.  I agree that we should add this as an optional dependency, but that's beyond the scope of this PR.

I'm going to tag a v1.0 release and see if this baby works!


Improve car manufacturer detection for Tesla

Tesla has acquired multiple OUIs according to offical OUI list from IEEE. Additionally, Tesla has a /28 sub-range.

Please rebase so that we can merge

rebase done


Fix warnings of entity framework

There are some expressions could not translate to sql. This might impact the performance of the site.

Confused please explain the issue?

When run the application, in the log there are many warnings logged by EF. We should investigate and fix these warnings


nuget lib not grabbing dependency url's

The current code is declaring nuget packages as having an unknown license all the time. I was able to resolve this pretty easily, to at least report the license URL, as the code was very close. Just replace the functions below in nuget.rb, or grab the fix to the Dir path in license_urls:
def current_packages
  dependencies.each_with_object({}) do |dep, memo|
    path = Dir.glob("#{Dir.home}/.nuget/packages/#{dep.name.downcase}/#{dep.version}").first
    licenses = license_urls(path, dep)
    memo[dep.name] ||= NugetPackage.new(dep.name, dep.version, spec_licenses: licenses, install_path: path)
    memo[dep.name].groups << dep.assembly unless memo[dep.name].groups.include? dep.assembly
  end.values
end

def license_urls(path, dep)
  files = Dir["#{path}/#{dep.name}.#{dep.version}.nupkg"]
  return nil if files.empty?

  file = files.first
  Zip::File.open file do |zipfile|
    content = zipfile.read(dep.name + '.nuspec')
    Nuget.nuspec_license_urls(content)
  end
end

Hey @willer! Thanks for looking into this. This looks good. Would it be possible for you to make a PR for this change with tests? Then we can have this merged in and released!

Hey @willer! Thanks for looking into this. This looks good. Would it be possible for you to make a PR for this change with tests? Then we can have this merged in and released!


TS2352 After Update from NSwag v<IP_ADDRESS> to v<IP_ADDRESS>

TS2352	(TS) Conversion of type 'null' to type 'any[]' may be a mistake because neither type sufficiently overlaps with the other. If this was intentional, convert the expression to 'unknown' first.
All 'process*' generated methods start return: _observableOf<UserTypeModel[]>(<any[]>null);
before the update It was return _observableOf<UserTypeModel[]>(<any>null);
tsconfig.json settings:
    "compileOnSave": true,
    "compilerOptions": {
        "alwaysStrict": true,
        "baseUrl": "./",
        "declaration": false,
        "experimentalDecorators": true,
        "lib": [
            "ES2018",
            "DOM"
        ],
        "module": "esnext",
        "moduleResolution": "node",
        "outDir": "./dist/outdir",
        "removeComments": true,
        "sourceMap": true,
        "strict": true,
        "strictNullChecks": true,
        "target": "ES2015",
        "typeRoots": [
            "node_modules/@types"
        ]
    }
}

angular 12
typescript: 4.2.4

This has been changed in this PR: #2384
@dhwed can you or someone else propose a PR to fix this?

I'm testing a fix now and will make a PR later today

Is this fix ok?
https://github.com/RicoSuter/NSwag/pull/3567


S1 - As a user I want to sign up with a unique username and a password, to gain access to services and information that are only available to registered users.

[ ] sopra-fs22-group-26/client#49
[ ] sopra-fs22-group-26/client#45
[ ] sopra-fs22-group-26/server#77
[ ] sopra-fs22-group-26/server#78

@wchang1011 @catinthecradle @jdhekh @hobinrani
Please use the request body scheme as you find it in the example below:


Publish to Open VSX

Hi team! This is a great tool!
Would you be interested in publishing it to Open VSX, an open-source registry for VS Code extensions? This would allow it to be install in the various open source forks of VS Code.
Thank you!

Oh, what a beautiful morning treat. Thank you!


Plase update master with correct ABI/version data

It is still on 2.1.0/2002
See https://github.com/cisco/openh264/commit/a9f5ec7494331dfe31053711cb0ac9a6d1ebc924 for reference

Ping? Trying out master in system breaks everything depending on openh264 until master is on par with 2.2.1

Thanks. Will update and release new version in coming days after critical bugs are verified.

@joakim-tjernlund please refer to PR #3462

@GuangweiWang , I don't think PR #3462 is enough. Look at the ref I quoted, there is more files to update.

please reopen

which one do you think was missed?

I see that you have now updated master to rel 2.2.1
One thing that stands out is the change 2002 ->  2201 vs the real 2.2.1 release that have 2002 -> 2005
Should not  these number match rel. 2.2.1 ?

it is a reserved version and you can ignore it.

Does master differ in ABI w.r.t 2.2.1 ?
Do one need to rebuild apps linked to openh264?

do not need to rebuild

abidiff thinks differently:
# abidiff /var/tmp/portage/media-libs/openh264-2.1.1_p20190331/image/usr/lib64/libopenh264.so /var/tmp/portage/media-libs/openh264-2.2.0_pre20220125/image/usr/lib64/libopenh264.so
Functions changes summary: 0 Removed, 0 Changed, 0 Added function
Variables changes summary: 0 Removed, 0 Changed, 0 Added variable
Function symbols changes summary: 1 Removed, 2 Added function symbols not referenced by debug info
Variable symbols changes summary: 0 Removed, 0 Added variable symbol not referenced by debug info

1 Removed function symbol not referenced by debug info:

  [D] _ZN7WelsDec12CWelsDecoder27ReleaseBufferedReadyPictureEPNS_21TagWelsDecoderContextEPPhP13TagBufferInfo

2 Added function symbols not referenced by debug info:

  [A] _ZN7WelsDec12CWelsDecoder34ReleaseBufferedReadyPictureReorderEPNS_21TagWelsDecoderContextEPPhP13TagBufferInfob
  [A] _ZN7WelsDec12CWelsDecoder36ReleaseBufferedReadyPictureNoReorderEPNS_21TagWelsDecoderContextEPPhP13TagBufferInfo


Hashes in JSON output are not deterministically ordered

When I create an SPDX document with multiple hashes on a package, the order of the hashes in the output json varies.
The hashes should follow a deterministic ordering.
Please let us know whether we shall prepare an PR for this. Perhaps it must be generalized to other output formats as well to produce comparable outputs.
Regards,
Karsten

@karsten-klein - Thanks for raising the issue.  I agree, it should be deterministic.
There is already a sorting in the JSON output, it must somehow miss the hash algorithms.
Since there are substantial changes to the SPDX 3 version, I would prefer to make the changes there to avoid merge conflicts unless you consider this to be a serious enough issue for a patch release.
If a PR could be opened against the v3 branch of the spdx-java-jackson-store repo where the sort is done, that would be great.


Update groovy-all version from 2.3.3 to 2.3.10.

This fixes incompatibilies when running groovydoc on the GradleFx
source code itself.
Resolves #200.

Curious, I can successfully run gradle check using Gradle 2.5 on my machine, but when run with Gradle 2.0 (the version of the wrapper included with GradleFx), the same errors occur as in the Travis run above. Time to upgrade the Gradle wrapper version???

I'm open for an upgrade to Gradle 2.5 of the wrapper. Go for it :)

Would you consider this now that the wrapper has been updated, @SlevinBE?

Merged. Thanks for the PR! :)


Cannot process payment without valid mandate for order id

Data in the database:

No order on Mollie exists with the generated number (2021-0000-0194).
I didn't manually change any of the mandates. How can I troubleshoot this?

This happens if for some reason the mandate became invalid. Expired creditcard for example.


Roles

Added roles Admin and User
Fixed so that we are able to create account with both of them
Fixed some authentication so that we are able to see only User page if we are User and Admin can see both
Fixed some routing to make it easier to test.

For future pull requests, please, target development, which can then be merged into main. :)


Bumping theme-check version to 1.10.3

WHY are these changes introduced?
Fixes https://github.com/Shopify/cli/issues/112
Bumping theme-check version to unblock yarn deploy (needed for checkout-ui-extension development)
WHAT is this pull request doing?
This PR bumps the version of theme-check to the latest version, which does not fail when checking theme-app-extensions using app liquid drops.

Gonna close and reopen to see if that'll pick up the workflow changes which should allow this to run.


HDDS-7427. [Snapshot] Add unit-testcases for Ozone snapshot feature.

What changes were proposed in this pull request?
Added unit-testcases for Ozone snapshot feature HDDS-6517
What is the link to the Apache JIRA
HDDS-7427
How was this patch tested?
Added testcases in file - hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOmSnapshot.java
mvn -Dtest=TestOmSnapshot test
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.ozone.om.TestOmSnapshot
[INFO] Tests run: 15, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 159.773 s - in org.apache.hadoop.ozone.om.TestOmSnapshot
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 15, Failures: 0, Errors: 0, Skipped: 0
[INFO]

Thanks @jyotirmoy-gh for the patch. Thanks @sadanand48 for the review.


Update UI tooling dependency

Codecov Report

Merging #132 (493baf0) into main (5db333b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage   75.24%   75.24%
=======================================
  Files          42       42
  Lines        1810     1810
  Branches      252      252
=======================================
  Hits         1362     1362
  Misses        329      329
  Partials      119      119

Continue to review full report at Codecov.

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


Drush Unable To Run In Newly Provisioned Drupal VM

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.0.1
Virtualbox 5.2.2r119230
ansible <IP_ADDRESS>
  config file = None
  configured module search path = [u'/home/vagrant/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/dist-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.12 (default, Nov 20 2017, 18:23:56) [GCC 5.4.0 20160609]

Your OS

Windows 10 Pro Version 1709

Full console output

https://gist.github.com/casc-or/de3ab5fcbbc282ccbcc0608fa21fd1f1
Summary

Attempting to install Drupal VM in a working Windows 10 / Vagrant / VirtualBox environment (I already use it for Laravel Homestead development.
Installation arrives nearly at the end before a failed Ansible command as seen in the linked gist.
Attempting to run the drush commend by SSH into the vagrant box gives this
vagrant@drupalvm:/var/www/drupalvm/drupal/vendor/drush/drush$ /var/www/drupalvm/drupal/vendor/drush/drush/drush --root=/var/www/drupalvm/drupal/web status bootstrap
-bash: /var/www/drupalvm/drupal/vendor/drush/drush/drush: Permission denied
vagrant@drupalvm:/var/www/drupalvm/drupal/vendor/drush/drush$
The Drupal site WILL start at its config page though.
Would appreciate assistance in resolving this.

What you have there is a permission denied error, #1614 could be related.
As for a fix - you could try making drush to be executable.
chmod 775 /var/www/drupalvm/drupal/vendor/drush/drush/drush

I'm not a *nix whiz, but I don't think that worked.
-rw-rw-rw- 1 9999 vagrant_group  4368 Dec 17 18:27 drush
That's after trying logged in as vagrant, and then sudo chmod ... as well.
Some additional info which may be relevant - the directory is in a shared folder from the host, and I'm using NFS to mount it (thanks to Allegro NFS from Franz, works well).

There's something wrong in the install here.
I did vagrant destroy -f followed by vagrant up --provision. The first attempt brought the message
C:\Users\username\Sites\drupal-vm>vagrant up --provision > provision.txt
The guest machine entered an invalid state while waiting for it
to boot. Valid states are 'starting, running'. The machine is in the
'unknown' state. Please verify everything is configured
properly and try again.

If the provider you're using has a GUI that comes with it,
it is often helpful to open that and watch the machine, since the
GUI often has more helpful error messages than Vagrant can retrieve.
For example, if you're using VirtualBox, run `vagrant up` while the
VirtualBox GUI is open.

The primary issue for this error is that the provider you're using
is not properly configured. This is very rarely a Vagrant issue.

So then it occurred to me that there was an odd VM in Virtualbox named packer-ubuntu-16.04-amd64_1513601919388_14911 which presumably is part of the Drupal VM install. I deleted it too and ran the vagrant up --provision command again.
This time it completed, but with the same error again.
Full console output

https://gist.github.com/casc-or/af6cdb3e42a7df86241300ac92f7e2f8
The only mod I made to the default.config.yml file was to give it my desired IP and to comment out the line
#  - name: vagrant-hostsupdater

I edited the hosts myself before install.
After install, I used vagrant ssh (and puTTY) to SSH into the VM and try to chmod. Results were the same - here's what the drush files look like
-rw-rw-rw- 1 9999 vagrant_group  4368 Dec 17 18:27 drush
-rw-rw-rw- 1 9999 vagrant_group 14013 Dec 17 18:27 drush.api.php
-rw-rw-rw- 1 9999 vagrant_group  1937 Dec 17 18:27 drush.complete.sh
-rw-rw-rw- 1 9999 vagrant_group    21 Dec 17 18:27 drush.info
-rw-rw-rw- 1 9999 vagrant_group  4954 Dec 17 18:27 drush.launcher
-rw-rw-rw- 1 9999 vagrant_group 23280 Dec 17 18:27 drush_logo-black.png
-rw-rw-rw- 1 9999 vagrant_group   221 Dec 17 18:27 drush.php

Either there is something I'm missing in the instructions for installation, or something doesn't like the environment I'm using.
Help appreciated - I need to get a Drupal 8 project going.

you are running virtualbox 5.2. I think that has some issues https://github.com/geerlingguy/drupal-vm/issues/1587 not sure are all of them fixed and would they affect windows

I'm trying again but this time running the vagrant up --provision in a Windows Administrative Command Prompt.
In all attempts though early in the process I see these errors
Err:24 http://security.ubuntu.com/ubuntu xenial-security/main amd64 linux-libc-dev amd64 4.4.0-87.110
  404  Not Found [IP: <IP_ADDRESS> 80]
Get:25 http://us.archive.ubuntu.com/ubuntu xenial-updates/main amd64 libc6-dev amd64 2.23-0ubuntu9 [2,082 kB]
Err:24 http://security.ubuntu.com/ubuntu xenial-security/main amd64 linux-libc-dev amd64 4.4.0-87.110
  404  Not Found [IP: <IP_ADDRESS> 80]
Get:26 http://us.archive.ubuntu.com/ubuntu xenial/main amd64 manpages-dev all 4.04-2 [2,048 kB]
Err:20 http://us.archive.ubuntu.com/ubuntu xenial-updates/main i386 dkms all <IP_ADDRESS>-2ubuntu11.3
  404  Not Found [IP: <IP_ADDRESS> 80]
Fetched 24.6 MB in 17s (1,388 kB/s)
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/pool/main/d/dkms/dkms_<IP_ADDRESS>-2ubuntu11.3_all.deb  404  Not Found [IP: <IP_ADDRESS> 80]

E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/main/l/linux/linux-libc-dev_4.4.0-87.110_amd64.deb  404  Not Found [IP: <IP_ADDRESS> 80]

E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

Makes me wonder if there is a disconnect between what the provisioning is trying to install and current versions?

Thanks, FinBoWa, but Vagrant 2.0.1 as installed does support Virtualbox 5.2 (and as noted earlier I am running a Laravel Homestead VM on the same host setup with no problem)
https://github.com/hashicorp/vagrant/issues/9090

@casc-or,
The mode change wasn't affected, and the end result is the same - permission denied to the drush binary.
I recommend you run the following:
sudo apt-get update --fix-missing
What you're looking for in the permissions is rwxrwxrw-, which is the equivelant of 0775.
Here's an example of mine, which is 0777:
vagrant@drupalvm:/$ stat $(which drush)
  File: '/usr/local/bin/drush' -> '/home/vagrant/.dvm/versions/drush-8.1.15/vendor/bin/drush'
  Size: 57        	Blocks: 0          IO Block: 4096   symbolic link
Device: fc00h/64512d	Inode: 4990515     Links: 1
Access: (0777/lrwxrwxrwx)  Uid: (  900/ vagrant)   Gid: (  900/ vagrant)
Access: 2017-12-18 01:01:23.544041000 +0000
Modify: 2017-12-18 01:01:23.540038999 +0000
Change: 2017-12-18 01:01:23.540038999 +0000
 Birth: -
vagrant@drupalvm:/$

You can check that this is the problem by running:
sudo /var/www/drupalvm/drupal/vendor/drush/drush/drush
Which will run drush as the root user, if permission is denied for the root user it'll eliminate a bunch of problems.
To change the permissions (I've revised this):
sudo chmod -R 775 /var/www/drupalvm/drupal/vendor/drush/drush/
and you can follow-up by running:
stat /var/www/drupalvm/drupal/vendor/drush/drush/drush, and look for 0775 under the line that starts with Access. Then, see if you can run it.
This should be enough to get any additional information. I still think it's related to the issue I mentioned, and I believe there's a genuine bug for Windows in here. I've never actually got it to work in Windows, so you're doing better than me :)

Thanks for your efforts to try and help.
I haven't mentioned it in this thread, but I have already run
sudo apt-get update --fix-missing

after having seen it referenced at the end of the section I pasted where ubuntu files aren't retrieved which appear to be security related. No effect.
I did try recursively changing permissions on the drush directory, without success.
I don't think I've tried running drush as root - I'll try that as soon as the VM finishes provisioning (again).

Still nothing. "Command not found".
vagrant@drupalvm:/$ sudo /var/www/drupalvm/drupal/vendor/drush/drush/drush
sudo: /var/www/drupalvm/drupal/vendor/drush/drush/drush: command not found
vagrant@drupalvm:/$ ls -l /var/www/drupalvm/drupal/vendor/drush/drush/drush*.*
-rw-rw-rw- 1 9999 vagrant_group 14013 Dec 17 18:27 /var/www/drupalvm/drupal/vendor/drush/drush/drush.api.php
-rw-rw-rw- 1 9999 vagrant_group  1937 Dec 17 18:27 /var/www/drupalvm/drupal/vendor/drush/drush/drush.complete.sh
-rw-rw-rw- 1 9999 vagrant_group    21 Dec 17 18:27 /var/www/drupalvm/drupal/vendor/drush/drush/drush.info
-rw-rw-rw- 1 9999 vagrant_group  4954 Dec 17 18:27 /var/www/drupalvm/drupal/vendor/drush/drush/drush.launcher
-rw-rw-rw- 1 9999 vagrant_group 23280 Dec 17 18:27 /var/www/drupalvm/drupal/vendor/drush/drush/drush_logo-black.png
-rw-rw-rw- 1 9999 vagrant_group   221 Dec 17 18:27 /var/www/drupalvm/drupal/vendor/drush/drush/drush.php
vagrant@drupalvm:/$

@casc-or - I can't reproduce the error, but I haven't yet tried using VirtualBox 5.2.x (I'm still on the latest 5.1.x release), and I wonder if there's any chance that has something to do with it (the way shares work, the way the guest additions are updated, etc... could have something to do with it).
I will try to get a running VirtualBox 5.2.x setup locally and see if I can reproduce the issue.

Thanks @geerlingguy - I am presently working through your instructions on setting up BLT and Drupal VM within Windows 10 to see if I can make that work and will let you know if I can, but my strong preference would be to do the simpler (to me) Vagrant / Drupal VM all from Windows.

I've pushed new Vagrant base boxes for all OSes, with the 5.2.4 Guest Additions, and I've tested at least ubuntu1604, ubuntu1404, debian9, and centos7 on my Mac. Haven't done any extra testing on my PC (It's in my office and I'm off this week :).
Everything seems to work correctly using all the config.yml defaults on my Mac at least.

So - I finally got this to run.
Way back in this thread I said

Some additional info which may be relevant - the directory is in a shared folder from the host, and I'm using NFS to mount it (thanks to Allegro NFS from Franz, works well).

Turns out it only works well for my Laravel Homestead box.
From the Allegro NFS documentation

By default, all regular files report mode 0666 (rw-rw-rw-) mode. Files with filename extensions that are usually executable (.exe, .com, .bat, etc) report mode 0777 (rwxrwxrwx). Directories report mode 0777. You can use the umask to turn some of the default mode bits off before reporting them back to the client. Unless you have special needs, it is recommended that you leave the umask setting at 0. See the Notes section below for more information. The parameter is expressed in octal notation.

and

chmod is not supported. chmod requests are simply ignored by Allegro NFS. This was a specific design decision to avoid confusing problems that can be caused when the chmod'd files are subsequently accessed on the Windows side. The execute bit is simulated based on filename extension. Files with filename extension exe, bat, and com automatically have the 'x' bit set. If you need the execute bit set, you can make Allegro NFS enable the execute bit on every file, which may suit your needs.

My best guess is that the Laravel Homestead files must get installed with the correct permissions while the Drupal-VM files must be getting permissions (not) set during the installation process.
So, I could try setting execute on all files but that doesn't sound like a good idea - sooner or later some file is going to get to a production server with 777 by mistake.
I disabled nfs folder syncing and everything worked albeit slowly.
Now that I know what the issue is and have a functioning installation, I'm working towards the workflow as described in Drupal VM on Windows - a fast container for BLT project development. I have a question on that but I'll post it on the comments there.
Thanks to everyone posting on this thread for all their assistance!


modules/gnome: add support for multiple include for Gir. Closes #69

Signed-off-by: Igor Gnatenko<EMAIL_ADDRESS>

Looks good. Thanks.


⚠ RPC server - xDai has degraded performance

In f0a5330, RPC server - xDai ($XDAI_RPC_URL) experienced degraded performance:

HTTP code: 200
Response time: 370 ms

Resolved: RPC server - xDai performance has improved in eb8d01f after 6 minutes.


Error when I run catkin build the "dvs_calibration package"

I want to calibrate the dvs128 using the ROS-package but whenever I build the dvs_calibration package"
I keep getting the following error (I am using ubuntu 16 with ROS-kinetic)
$ catkin build dvs_calibration
[build] Found '9' packages in 0.0 seconds.
[build] Updating package table.
Starting  >>> catkin_simple
Finished  <<< catkin_simple                  [ 0.1 seconds ]
Starting  >>> dvs_msgs
Finished  <<< dvs_msgs                       [ 0.3 seconds ]
Starting  >>> dvs_calibration

Errors     << dvs_calibration:make /home/merwan/dvs_ros_ws/logs/dvs_calibration/build.make.000.log
/home/merwan/dvs_ros_ws/src/rpg_dvs_ros/dvs_calibration/src/circlesgrid.cpp:544:1: error: reference to ‘CirclesGridFinderParameters’ is ambiguous
CirclesGridFinderParameters::CirclesGridFinderParameters()
^
In file included from /home/merwan/dvs_ros_ws/src/rpg_dvs_ros/dvs_calibration/src/circlesgrid.cpp:44:0:
/home/merwan/dvs_ros_ws/src/rpg_dvs_ros/dvs_calibration/include/dvs_calibration/circlesgrid.hpp:125:8: note: candidates are: struct CirclesGridFinderParameters
struct CirclesGridFinderParameters
^
In file included from /opt/ros/kinetic/include/opencv-3.3.1/opencv2/calib3d/calib3d.hpp:48:0,
from /home/merwan/dvs_ros_ws/src/rpg_dvs_ros/dvs_calibration/include/dvs_calibration/circlesgrid.hpp:54,
from /home/merwan/dvs_ros_ws/src/rpg_dvs_ros/dvs_calibration/src/circlesgrid.cpp:44:
/opt/ros/kinetic/include/opencv-3.3.1/opencv2/calib3d.hpp:762:28: note:                 struct cv::CirclesGridFinderParameters
struct CV_EXPORTS_W_SIMPLE CirclesGridFinderParameters
....
....
/opt/ros/kinetic/include/opencv-3.3.1/opencv2/calib3d.hpp:762:28: note:                 struct cv::CirclesGridFinderParameters
struct CV_EXPORTS_W_SIMPLE CirclesGridFinderParameters
^
make[2]: *** [CMakeFiles/camera_dvs.dir/src/circlesgrid.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [CMakeFiles/stereo.dir/src/circlesgrid.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/mono.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/stereo.dir/all] Error 2
make[1]: *** [CMakeFiles/camera_dvs.dir/all] Error 2
make: *** [all] Error 2
cd /home/merwan/dvs_ros_ws/build/dvs_calibration; catkin build --get-env dvs_calibration | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -
...............................................................................
Failed     << dvs_calibration:make           [ Exited with code 2 ]
Failed    <<< dvs_calibration                [ 14.5 seconds ]
How can I fix this issue ?

Hi,
I know it's been a few months but can you try out #72 if you didn't fix the problem already?

Should have been fixed a long time ago.


add branches to lake model

This commit adds a branch concept to the zed lake.  This required
some rewiring of the snapshot logic to implement merge and a new
approach to locking across two branches for a merge (though the
locking details are not yet implemented).  The TAIL pointer in a
journal now tracks the base pointer in a parent branch.  TAIL is
transactionally guaranteed through locking (TBD).
For now there is only one root branch (main), and branches must
all be descended from this.
"zed lake query" no longer reports "empty log" as it simply
produces no output.
"zed lake ls" no longer lists objects as meta-queries can be used
for this.  Instead, ls lists pools and branches.
The lake commands "add", "commit", and "status" are deprecated.
The lake command "load" does an automatic commit in the backend.
Closes #2626
Closes #2537
Closes #2953
Closes #2975

I still need to update lake design.md... will work on that in the morning but wanted to get this up!

Ok, I'll hold off on merging until app integration tests pass...


Training issue, RuntimeError: The server socket has failed to listen on any local network addres

RuntimeError: The server socket has failed to listen on any local network address. The server socket has failed to bind to [::]:29500 (errno: 98 - Address already in use). The server socket has failed to bind to ?UNKNOWN? (errno: 98 - Address already in use).

scripts/dist_train.sh 8 --cfg_file cfgs/kitti_models/sfd.yaml --gpu_id 0

Hi, you need to change the tcp_port to another number to avoid address conflict, such as:
scripts/dist_train.sh 8 --cfg_file cfgs/kitti_models/sfd.yaml --gpu_id 0 --tcp_port 24523

Still error is same.

I am sorry that I have no idea. Maybe you can try more tcp_port numbers.

I did but not work for me


⚠ Bofrost.be has degraded performance

In e3c700b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1059 ms

Resolved: Bofrost.be performance has improved in 1b8e8fd.


per-frame metadata missing from doc

Hi! It is my understanding that some cameras and renderers are able to embed metadata (such as focal length, exposure, etc) for every individual frame; however, the doc and examples seem to only mention file-level headers (unless I'm misreading it completely). Is there a documented way to write custom per-frame attributes? If so, maybe the doc should explain it

apparently, exr is an image, not a sequence. TIL

Yes. A typical use case is a directory filled with numbered sequences of exr images, dumped out one by one by a renderer.


Add possibility to exclude (transitive) dependencies

If this dependency is added:
dependency("com.openshift.jenkins", "openshift-pipeline", "1.0.54")
the integrationTests fail in a java.lang.StackOverflowError. This answer here indicates that this might be a problem of an unwanted dependency. We might need the ability to exclude dependencies from plugins.

For those who land here by a google search, here is a workaround:
// TODO: workaround for missing dependency exclude mechanism is pipeline plugin. configurations.all {     exclude("org.slf4j", "slf4j-log4j12")     exclude("log4j", "log4j") }

I agree it should be more straightforward., and have better alignment with how Gradle users expect to exclude dependencies using https://docs.gradle.org/current/userguide/managing_transitive_dependencies.html. I'd like the UX to feel more "Gradle native" by just using the existing dependencies idioms, but haven't quite figured out the right balance between DSL-friendly dependencies for Jenkins and using the built-in Gradle conepts
However, there should be a workaround for your use-case.
You can exclude the bad dependency from all configurations (if that is useful):
configurations {
  all {
    exclude("org.slf4j", "slf4j-log4j12")
    exclude("log4j", "log4j")
  }
}

Or exclude them specifically from the configurations created by the plugin or the integrationTest runtime. The configurations.all I think is better for now to not rely on the internal configuration changes while I try and figure out a better way to represent dependencies.

Looks like you already figured out the workaround, thanks for posting it.


updated the syntax to reflect the support 32-bit ASN with a border de…

updated document to show support for 32-bit ASN at a customer border device.

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

#sign-off

@ebmarquez: I'm sorry - only the author of this article, @IngridAtMicrosoft, 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.

in-review

#sign-off
Thanks Eric!


Fix ol.tilegrid.TileGrid#getZForResolution

This fixes a bug in ol.tilegrid.TileGrid#getZForResolution where this function doesn't take minZoom and maxZoom into account when selecting a zoom level for a resolution.
Fixes #2780.

Awesome! Please merge when Travis is happy.


TO: in 1.2.0, generating a CRConfig for the 'ALL' CDN usually blows up

Can't use an undefined value as an ARRAY reference at /opt/traffic_ops/app/lib/UI/Topology.pm line 65.```

This was fixed in #667.


Fix bug in save and restore

Changelog
🐞 Save and restore set wrong markers
With the fix for formats over links (https://github.com/livingdocsIO/editable.js/pull/251) we introduced a bug that markers could be set outside of the host.
The problem was that we used parentElement in order to get the ranges parent instead of commonAncestorContainer which will always contain the resulting element to the host that contains the selection.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


✨ [REQUEST] mount a second nvme for all addons with localdisks support

Checked

[X] If a new addon, I have checked on  Google that such as addon doesn't already exists
[X] If a new feature, I have searched in the discussions & issues that it was not already covered

Which addon?
All with localdisks support
Is your feature request related to a problem? Please describe
I want to mount a second nvme into the addons. But the protocol says:

Local Disks mounting...
... nvme1n1p1 is a physical device
[15:31:32] INFO: Mounting nvme1n1p1 of type
mount: permission denied (are you root?)
[15:31:32] FATAL: Unable to mount local drives! Please check the name.

Describe the solution you'd like
Please add  "/dev/nvme1n1p1" and maybe "/dev/nvme1n1p2", "/dev/nvme1n1p3" ... to all config.json files.

Hi, which add-on are you mostly in? To avoid creating a new version of all add-ons. Thanks!

I've added it for the version of nextcloud just pushed, please let me know if it works before I transversalize to the others

Yes i will try it this evening.

I have an new setup and the 02-mariadb_discovery.sh script is not loading. is it normal?

Starting...
/etc/cont-init.d/00-banner.sh: executing

Add-on: Nextcloud
Nextcloud for Homeassistant
Add-on version: 28.0.1-5
You are running the latest version of this add-on.
System: Home Assistant OS 11.4  (amd64 / generic-x86-64)
Home Assistant Core: 2024.1.3
Home Assistant Supervisor: 2023.12.0
Please, share the above information when looking for help
or support in, e.g., GitHub, forums
Provided by: https://github.com/alexbelgium/hassio-addons
Defining permissions for main user :
User UID: 1000
User GID : 1000
/etc/cont-init.d/00-global_var.sh: executing
Full_Text_Search='false'
OCR='false'
OCRLANG='fra'
PGID='1000'
PUID='1000'
additional_apps='inotify-tools'
certfile='fullchain.pem'
elasticsearch_server='ip:port'
enable_thumbnails='true'
keyfile='privkey.pem'
localdisks='nvme1n1p1'
trusted_domains='your-domain, your-domain2'
use_own_certs='false'
/etc/cont-init.d/00-local_mounts.sh: executing
[19:03:30] INFO: Available Disks for mounting :
NAME        LABEL       SIZE FSTYPE   RO
└─nvme1n1p1 PlexMedia 476.9G ext4      0
├─nvme0n1p2              24M squashfs  0
├─nvme0n1p3             256M squashfs  0
Supported fs :  ext3 ext2 ext4 squashfs vfat msdos iso9660
Inspired from : github.com/dianlight
Local Disks mounting...
... nvme1n1p1 is a physical device
[19:03:31] INFO: Mounting nvme1n1p1 of type ext4
[19:03:31] INFO: Success! nvme1n1p1 mounted to /mnt/nvme1n1p1
/etc/cont-init.d/00-smb_mounts.sh: executing
/etc/cont-init.d/01-config_yaml.sh: executing
Setting permissions for the config.yaml directory
Load environment variables from /config/addons_config/nextcloud-ocr/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/nextcloud-ocr/config.yaml
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables
... no env variables found, exiting
[migrations] started
[migrations] 01-nginx-site-confs-default: skipped
[migrations] 02-default-location: skipped
[migrations] done
Setting permissions for the config.yaml directory
Load environment variables from /config/addons_config/nextcloud-ocr/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/nextcloud-ocr/config.yaml
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables
... no env variables found, exiting
Execute /homeassistant/addons_config/nextcloud-ocr/nextcloud-ocr.sh if existing
Wiki here : github.com/alexbelgium/hassio-addons/wiki/Add-ons-feature-:-customisation
───────────────────────────────────────

|     |  |  |   | |     |     |
|   --|  |  |   | | | |  |  | | | |
|||| |_| |||||
_____ __ __ _ __    ____
| __  |  |  | |  |  |    \
| __ -|  |  | |  ||  |  |
||__||_____|____/
Based on images from linuxserver.io
───────────────────────────────────────
To support LSIO projects visit:
https://www.linuxserver.io/donate/
───────────────────────────────────────
GID/UID
───────────────────────────────────────
User UID:    1000
User GID:    1000
───────────────────────────────────────
Setting user to 1000:1000
Updating permissions...
... Config directory : /data
... data directory detected : /share/nextcloud
... setting permissions
... done
using keys found in /data/config/keys
Initializing nextcloud <IP_ADDRESS> (this can take a while) ...
Setting permissions
New nextcloud instance
Please run the web-based installer on first connect!
Initializing finished
After completing the web-based installer, restart the Nextcloud container to apply default memory caching and transactional file locking configurations.
Alternatively, you can apply your own configurations by editing /data/config/www/nextcloud/config/config.php following the documentation:
https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/caching_configuration.html
https://docs.nextcloud.com/server/latest/admin_manual/configuration_files/files_locking_transactional.html
Starting...
/etc/cont-init.d/01-config_yaml.sh: executing
Setting permissions for the config.yaml directory
Load environment variables from /config/addons_config/nextcloud-ocr/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/nextcloud-ocr/config.yaml
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables
... no env variables found, exiting
[custom-init] No custom files found, skipping...
crond: crond (busybox 1.36.1) started, log level 8
[ls.io-init] done.

I have an new setup and the 02-mariadb_discovery.sh script is not loading. is it normal? But it looks that the mounting is working.

Starting...
/etc/cont-init.d/00-banner.sh: executing
Add-on: Nextcloud
Nextcloud for Homeassistant
Add-on version: 28.0.1-5
You are running the latest version of this add-on.
System: Home Assistant OS 11.4  (amd64 / generic-x86-64)
Home Assistant Core: 2024.1.3
Home Assistant Supervisor: 2023.12.0
Please, share the above information when looking for help
or support in, e.g., GitHub, forums
Provided by: https://github.com/alexbelgium/hassio-addons
Defining permissions for main user :
User UID: 1000
User GID : 1000
/etc/cont-init.d/00-global_var.sh: executing
Full_Text_Search='false'
OCR='false'
OCRLANG='fra'
PGID='1000'
PUID='1000'
additional_apps='inotify-tools'
certfile='fullchain.pem'
elasticsearch_server='ip:port'
enable_thumbnails='true'
keyfile='privkey.pem'
localdisks='nvme1n1p1'
trusted_domains='your-domain, your-domain2'
use_own_certs='false'
/etc/cont-init.d/00-local_mounts.sh: executing
[19:03:30] INFO: Available Disks for mounting :
NAME        LABEL       SIZE FSTYPE   RO
└─nvme1n1p1 PlexMedia 476.9G ext4      0
├─nvme0n1p2              24M squashfs  0
├─nvme0n1p3             256M squashfs  0
Supported fs :  ext3 ext2 ext4 squashfs vfat msdos iso9660
Inspired from : github.com/dianlight
Local Disks mounting...
... nvme1n1p1 is a physical device
[19:03:31] INFO: Mounting nvme1n1p1 of type ext4
[19:03:31] INFO: Success! nvme1n1p1 mounted to /mnt/nvme1n1p1
/etc/cont-init.d/00-smb_mounts.sh: executing
/etc/cont-init.d/01-config_yaml.sh: executing
Setting permissions for the config.yaml directory
Load environment variables from /config/addons_config/nextcloud-ocr/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/nextcloud-ocr/config.yaml
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables
... no env variables found, exiting
[migrations] started
[migrations] 01-nginx-site-confs-default: skipped
[migrations] 02-default-location: skipped
[migrations] done
Setting permissions for the config.yaml directory
Load environment variables from /config/addons_config/nextcloud-ocr/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/nextcloud-ocr/config.yaml
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables
... no env variables found, exiting
Execute /homeassistant/addons_config/nextcloud-ocr/nextcloud-ocr.sh if existing
Wiki here : github.com/alexbelgium/hassio-addons/wiki/Add-ons-feature-:-customisation
───────────────────────────────────────
|     |  |  |   | |     |     |
|   --|  |  |   | | | |  |  | | | |
|||**| || |**||||

| __  |  |  | |  |  |
| __ -|  |  | |  ||  |  |
||||____|____/
Based on images from linuxserver.io
───────────────────────────────────────
To support LSIO projects visit:
https://www.linuxserver.io/donate/
───────────────────────────────────────
GID/UID
───────────────────────────────────────
User UID:    1000
User GID:    1000
───────────────────────────────────────
Setting user to 1000:1000
Updating permissions...
... Config directory : /data
... data directory detected : /share/nextcloud
... setting permissions
... done
using keys found in /data/config/keys
Initializing nextcloud <IP_ADDRESS> (this can take a while) ...
Setting permissions
New nextcloud instance
Please run the web-based installer on first connect!
Initializing finished
After completing the web-based installer, restart the Nextcloud container to apply default memory caching and transactional file locking configurations.
Alternatively, you can apply your own configurations by editing /data/config/www/nextcloud/config/config.php following the documentation:
https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/caching_configuration.html
https://docs.nextcloud.com/server/latest/admin_manual/configuration_files/files_locking_transactional.html
Starting...
/etc/cont-init.d/01-config_yaml.sh: executing
Setting permissions for the config.yaml directory
Load environment variables from /config/addons_config/nextcloud-ocr/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/nextcloud-ocr/config.yaml
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables
... no env variables found, exiting
[custom-init] No custom files found, skipping...
crond: crond (busybox 1.36.1) started, log level 8
[ls.io-init] done.

Not normal indeed I wonder why i'll check

Stats again in new version - i don't know why but the scripts fails when executed  with source,and succeed when executed with .... Well then I've reverted the main script calling the subscripts and everything is working again

all ist working again and i can see the mount in nextcloud. i you can implement this into Plex, it would be fine. I think we can close this request... thank you very much.


Add structure hook for Union[str, List[InlayHintLabelPart]]

Fixes #218

@alcarney It looks like how pygls used to run tests has changed. Would it be possible for you to update the github workflow so that we can run pygls tests as smoke tests?

It is not immediately obvious to me why the test suite would hang before running any tests - especially when they seem to run fine over on the pygls repo....
But I will have a look and see if I can reproduce it locally

@alcarney I was running pytest directly, but it looks like we should probably use tox to run pygls tests.

@karthiknadig I reproduced it locally  - test suite hangs on the second test.
I seem to remember having issues with that test when I wrote it originally - hopefully I can remember what the issue was!
================================================= test session starts =================================================
platform linux -- Python 3.11.3, pytest-7.3.2, pluggy-1.0.0
rootdir: /var/home/alex/Projects/lsprotocol/smoke_tests
configfile: pyproject.toml
plugins: typeguard-4.0.0, asyncio-0.21.0
asyncio: mode=Mode.AUTO
collected 213 items

smoke_tests/tests/test_client.py::test_client_detect_server_exit
--------------------------------------------------- live log setup ----------------------------------------------------
DEBUG    asyncio:selector_events.py:54 Using selector: EpollSelector
DEBUG    asyncio:selector_events.py:54 Using selector: EpollSelector
---------------------------------------------------- live log call ----------------------------------------------------
DEBUG    pygls.client:client.py:110 Starting server process: /var/home/alex/Projects/lsprotocol/.ci/bin/python -c print('Hello, World!')
DEBUG    pygls.client:client.py:134 Server process 105533 exited with return code: 0
PASSED                                                                                [  0%]
-------------------------------------------------- live log teardown --------------------------------------------------
DEBUG    asyncio:selector_events.py:54 Using selector: EpollSelector

smoke_tests/tests/test_client.py::test_client_detect_invalid_json
--------------------------------------------------- live log setup ----------------------------------------------------
DEBUG    asyncio:selector_events.py:54 Using selector: EpollSelector
---------------------------------------------------- live log call ----------------------------------------------------
DEBUG    pygls.client:client.py:110 Starting server process: /var/home/alex/Projects/lsprotocol/.ci/bin/python /var/home/alex/Projects/lsprotocol/smoke_tests/tests/servers/invalid_json.py
DEBUG    pygls.protocol:protocol.py:583 Sending request with id "e7f074c7-fca1-485e-881a-9b94dde57a17": method/name {}
INFO     pygls.protocol:protocol.py:462 Sending data: {"id": "e7f074c7-fca1-485e-881a-9b94dde57a17", "method": "method/name", "jsonrpc": "2.0", "params": {}}
DEBUG    pygls.client:client.py:134 Server process 105539 exited with return code: 1


The word count is biased

"EXH.C-◇" counts as 2 words in MSWord/WPS, and ◇ is cjk. But in this creates 1 word and cjk is 0.

Note: ◇ unicode is \u25c7
wiki: https://zh.wikipedia.org/zh-tw/幾何圖形_(Unicode區段)

I'm not sure why ◇ is CJK in Microsoft Office.

But LibreOffice says it is not.

I'm not sure why ◇ is CJK in Microsoft Office.

But LibreOffice says it is not.

◇ is Geometric Shapes


Different kind of sorting algorithm

Can I add Different kind of sorting algorithmin your repo, Under Hacktoberfest-2022 @akhildhiman7

Yes please, go ahead!

Check pull request and also add hacktoberfest-accepted label also. Thank you @akhildhiman7


Revisit deck.gl picking

Contributions
This pull request removes dependence on simple-quadtree and instead uses deck.gl's pickObjects.
Caveats

It seems faster when I have been using it but we may want to do some more rigorous performance testing to be sure.
This pull request extends off of the branch keller-mark/linked-hover-2-with-uuid rather than being an independent branch because to get the picking to work you need to project to pixel coordinates, which requires the deck.gl viewport instance (which was also required to set the DOM coordinates for the hovering elements) so it was easiest to be able to leverage the changes on that branch.

I think this is good, but let's get the other branch merged, and then, if possible, rebase this, so it's more clear just which parts are involved.

Fixes #147


pin importlib-metadata to 1.7.0 for python35

Pinned importlib-metadata==1.7.0 as version>1.7.0 is causing conflicts for python35 in running make upgrade.

Note:
According to the discussion on https://github.com/edx/edx-cookiecutters/pull/65 by @feanil, tests on python35 are failing because of the older version of importlib-metadata and we'll have to wait for the tox update for this issue to be resolved.
So pinning this package version unless we have a fix for the tests because it is causing the automated package upgrade jobs to fail.

Hi @feanil,
importlib-metadata is causing conflicts and hence build failures in the automated python package upgrade jobs. To make the build successful again, I've decided to pin importlib-metadata==1.7.0 for now.
Since you were already working on a fix for the package version issue in this repo, should I wait for your fix in this repo before working on this again & pause the upgrade-job till then, or should I pin the version for now so the upgrade job will run successfully until we can find the fix for this version conflict, in which case I'll need your help in merging this PR because I don't have access to merge this PR without the tests' status as passing?
Thanks.
CC: @jmbowman @awais786

@jmbowman can we drop 3.5 support in the cookiecutter instead?  Even if it produces 3.5 code, I don't think the cookiecutter needs to be able to run in python 3.5

Sounds good to me, especially if we can still run tests on the generated repo using 3.5 (although we shouldn't need that much longer either).

The issue has been fixed in https://github.com/pypa/virtualenv/pull/1953 and https://github.com/tox-dev/tox/pull/1682 so this PR is not needed anymore. Closing this PR.


Generic "Resolvers"

Create generic resolvers for certain data types, like DIDs, Schemas, Revocations.
Provide an interface with an in-memory implementation at the minimum. A HTTP implementation might also be worth it.

YES! this is needed. The more I've played with the SDK, the more I've realized this is a kinda big problem that needs to be resolved.
A question also begs which is how we interact with the ecosystem in general. See https://github.com/decentralized-identity/universal-resolver as an example
TBH I'm not a huge fan of the above. It's way to heavyweight for what it's trying to do. Maybe there's a middle ground?

Hello I have a few thoughts on this.
I think it would be very useful to work on integrating ssi-sdk and ssi-service with both the Universal Resolver (https://github.com/decentralized-identity/universal-resolver) and the Universal Registrar (https://github.com/decentralized-identity/universal-registrar). The latter is a not-so-well-known project, but it's essentially the same idea, creating an abstraction layer not only for the DID resolve operation, but also for the create/update/deactivate operations.
With regard to "heavyweight". Yes. Both the Universal Resolver and Universal Registrar are rather heavy-weight container-based applications, so you definitely can't run them on a mobile phone, etc. They are run as services, and therefore can also add an element of centralization that DIDs are not supposed to have.
But, the purpose of these components is still a bit more nuanced than it seems:

First, they are based on open specifications (DID Resolution, DID Registration) which do not mandate any specific implementation or deployment pattern. They define abstract interfaces that can be implemented in various ways.
Second, both the Universal Resolver and Universal Registrar can be self-hosted. If you are planning to run something like TBD's ssi-sdk or ssi-service on some backend service, then I think there's nothing wrong with also deploying the Universal Resolver/Registrar side-by-side with that.
Third, we often see deployment patterns where some DID methods are implemented as part of a local library, and remote Universal Resolver/Registrar services are configured as "fallbacks" when the local library can't handle a certain DID method. For example, TBD's ssi-sdk could handle did:key internally (no need to use any external component), but it could use the Universal Resolver/Registrar for handling did:ebsi or did:indy or did:v1, etc.

This should be configurable, in a similar way as you can configure your DNS server in your operating system. Depending on requirements, TBD's ssi-sdk or ssi-service could be configured to handle all DID operations internally, or use a self-hosted instance of the Universal Resolver/Registrar, or in certain situations perhaps even use a commercial, centralized service such as https://godiddy.com/ (maybe only for testing/development purposes).
The advantage would be that the TBD tools could immediately benefit from supporting many different DID methods (resolve/create/update/deactivate) out-of-the-box.
I'd love to discuss this more if there is an opportunity?

Good thoughts here @peacekeeper (btw I've seen some of your presentations ). Thank you for them. Great stuff.
@decentralgabe I think this closely relates to: https://github.com/TBD54566975/ssi-sdk/issues/111. In fact, I would argue this ticket is "blocked" until https://github.com/TBD54566975/ssi-sdk/issues/111 is somewhat more hashed out.
@peacekeeper To your second and third point. I'm not sure this is always intended to be hosted as a service? Again, https://github.com/TBD54566975/ssi-sdk/issues/111 really matters here.
Re: services configured as "fallbacks", definitely I think it should be configurable, and to your point maybe the ecosystem will eventually have various commercial DID Resolvers. That being said: I would generally prefer NOT using a resolver if I could avoid it, as that somewhat requires trust on behalf of the resolving party, which is a failure point.
Again, all falls back to  https://github.com/TBD54566975/ssi-sdk/issues/111. Right now we support did:key, did:peer, and did:web. What's the scope here for future did methods?
Finally, I'd like to point out a recent: https://github.com/TBD54566975/ssi-sdk/blob/main/did/interface.go#L30 resolver interface. In theory, we can add a whole bunch of methods as long as it supports the DID resolution interface.
A question here is: Are there any other requirements for a DID Resolver that haven't been specified?

Right now we support did:key, did:peer, and did:web.

I have a question about this, I thought the web5 project was primarily based on did:ion, did I misunderstand that? I would have expected there to be code for did:ion in ssi-sdk or ssi-service.

Finally, I'd like to point out a recent: https://github.com/TBD54566975/ssi-sdk/blob/main/did/interface.go#L30 resolver interface.

Good start, but it's missing some things that you can have in DID resolution, see the DID Core spec for the abstract function definition:
resolve(did, resolutionOptions) →
   « didResolutionMetadata, didDocument, didDocumentMetadata »

Are there any other requirements for a DID Resolver that haven't been specified

In general, most of the requirements/considerations should be expressed in the (still work-in-progress) DID Resolution spec (https://w3c-ccg.github.io/did-resolution/). Maybe one thing to mention is that there's DID resolution, and then there's also DID URL dereferencing.
My understanding from last IIW was that there are some interesting applications of DID URLs (DID with path and/or query and/or fragment) in DWNs.

Added an issue here: https://github.com/TBD54566975/ssi-sdk/issues/new

@peacekeeper

I have a question about this, I thought the web5 project was primarily based on did:ion, did I misunderstand that? I would have expected there to be code for did:ion in ssi-sdk or ssi-service.

The default method supposed to be ION, as it is stated here. However, ION support is not there yet.

Sounds like we're aligned. +1 to making the stack unopinionated and supporting any DID method. If we all follow a standardized DID resolution interface, it should be possible to just "plug in" support for different DID methods. Ideally locally as part of the ssi-service itself, but various options to plug in existing resolvers are also possible.
And yes I agree multiple implementations are a good thing, also for standardization processes, test suites, etc.

Brief update: we (TBD) discussed as a team this week and think it's best if we spin up our own implementations of the Universal Resolver (and potentially the registrar) as projects here, separate from the SDK.

@decentralgabe Sounds great, I'd love to help with that if you need anything!

Same here @peacekeeper and @decentralgabe . I'd love to help as well.

For HTTP I made a generic resolver, which applies some headers too. Feedback is welcome.
<EMAIL_ADDRESS>

this is done here https://github.com/TBD54566975/ssi-sdk/blob/main/did/resolver.go
will spin this out to a separate project/repo at some point in the coming months.

@decentralgabe not sure what the plan is down the road, but wanted to ask what your thoughts are on the following:

Try to resolve locally.
If it doesn't resolve locally, use an external service ( i.e universal resolver ) to resolve as a fallback. Hyperledger Aries does this, and it seems like a reasonable approach.

goal of the sdk should be to support as many functions internally as possible, as in general I'm not a huge fan of reaching outward to resolve a did, but it can resolve externally via an endpoint if needed as a fallback.

+1 to the approach of first trying to resolve locally, and supporting the use of an external service as a fallback.
The external service should be configurable (i.e. don't just hardcode https://dev.uniresolver.io/ as the only possibility). Note also that there is both an HTTPS and a DIDComm binding for resolving via an remote service.
The list of locally resolvable DID methods should also be configurable; there may be situations where it's still preferable to use a remote resolver service, even if a local resolver also exists.

Great idea @andorsk and agreed @peacekeeper. We have a resolver impl of this library code in the ssi-service but it does not do caching yet. We have yet to integrate an external service - but we have configuration where this can be set.
I'm opening an issue to make the resolver a caching resolver. https://github.com/TBD54566975/ssi-service/issues/237


Switch from lens to microlens

This reduces compilation time. (microlens is a smaller lens-compatible lens library.)

Thanks!


[Event Request] page 6510 "Item Tracking Lines" - OnBeforeInsertItemTrackingLine

We would like to add a new event OnBeforeInsertItemTrackingLine in the following procedure:
local procedure RegisterChange(var OldTrackingSpecification: Record "Tracking Specification"; var NewTrackingSpecification: Record "Tracking Specification"; ChangeType: Option Insert,Modify,FullDelete,PartDelete,ModifyAll; ModifySharedFields: Boolean) OK: Boolean
var
    ReservEntry1: Record "Reservation Entry";
    ReservEntry2: Record "Reservation Entry";
    OriginalOldTrackingSpecification: Record "Tracking Specification";
    CreateReservEntry: Codeunit "Create Reserv. Entry";
    ReservationMgt: Codeunit "Reservation Management";
    QtyToAdd: Decimal;
    LostReservQty: Decimal;
    IdenticalArray: array[2] of Boolean;
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeRegisterChange(OldTrackingSpecification, NewTrackingSpecification, CurrentSignFactor, CurrentRunMode.AsInteger(), IsHandled);
    if IsHandled then
        exit;

    OK := false;

    if ((CurrentSignFactor * NewTrackingSpecification."Qty. to Handle") < 0) and
       (CurrentRunMode <> CurrentRunMode::"Drop Shipment")
    then begin
        NewTrackingSpecification."Expiration Date" := 0D;
        OldTrackingSpecification."Expiration Date" := 0D;
    end;

    case ChangeType of
        ChangeType::Insert:
            begin
                //A201F0000.Event.ns
                IsHandled := false;
                OnBeforeInsertItemTrackingLine(NewTrackingSpecification, OldTrackingSpecification, IsHandled);
                if IsHandled then
                    exit(true);
                //A201F0000.Event.ne
                if (OldTrackingSpecification."Quantity (Base)" = 0) or not OldTrackingSpecification.TrackingExists then
                    exit(true);
                TempReservEntry.SetTrackingFilterBlank;
                OldTrackingSpecification."Quantity (Base)" :=
                  CurrentSignFactor *
                  ReservEngineMgt.AddItemTrackingToTempRecSet(
                    TempReservEntry, NewTrackingSpecification, CurrentSignFactor * OldTrackingSpecification."Quantity (Base)",
                    QtyToAddAsBlank, ItemTrackingCode);
                TempReservEntry.ClearTrackingFilter();

                // Late Binding
                if ReservEngineMgt.RetrieveLostReservQty(LostReservQty) then begin
                    TempItemTrackLineReserv := NewTrackingSpecification;
                    TempItemTrackLineReserv."Quantity (Base)" := LostReservQty * CurrentSignFactor;
                    TempItemTrackLineReserv.Insert();
                end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeInsertItemTrackingLine(var NewTrackingSpecification: Record "Tracking Specification"; var OldTrackingSpecification: Record "Tracking Specification"; var IsHandled: Boolean)
begin
    //A201F0000.Event
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Use constant to define restart policy

... instead hard-coding the string in our own logic.

/lgtm


class VQVAE forward function error

CN:
VQVAE forward存在code 错误. 我发现 VectorQuantizer2.forward 返回三个参数. 而在https://github.com/FoundationVision/VAR/blob/a5cf0a16624d27b4c687b08f2a0ab006438452cc/models/vqvae.py#L58
接收了VectorQuantizer2.forward 返回的4个参数.
EN:
VQVAE forward has a code error. I found that VectorQuantizer2.forward returns three parameters. And at https://github.com/FoundationVision/VAR/blob/a5cf0a16624d27b4c687b08f2a0ab006438452cc/models/vqvae.py#L58
receives four arguments from VectorQuantizer2.forward.

hi @woldier, thank you and i fixed it in https://github.com/FoundationVision/VAR/commit/0029ecd537a4587e76b165dae04f9124b9391b66


Edgar parser

Summary of changes (Bug fix, feature, docs update, ...)
This is a very rough version of a parser for EDGAR data in netCDF format.

It assumes that the data is annual, and that there is only one flux source per netCDF file.
It uses the scitools iris package to do regridding instead of xesmf.
Also, the organisation of the modules doesn't really make sense now, since openghg.standardise.emissions._openghg.py now has a generic parser, plus parsers for openghg and edgar formats.
On the other hand, the code works and can be used to regrid and add EDGAR flux files to the object store.

Please check if the PR fulfills these requirements

[ ] Closes #xxxx (Replace xxxx with the Github issue number)
[ ] Tests added and passed if fixing a bug or adding a new feature
[ ] All code checks passing - black --line-length 110 run over code, mypy and flake8 not showing any errors
[ ] Documentation and tutorials updated/added
[ ] Added an entry in the latest CHANGELOG.md file if fixing a bug or adding a new feature
[ ] Added any new requirements to requirements.txt and recipes/meta.yaml

Thanks @qq23840 for the fixes.
To incorporate this into OpenGHG, I think @rt17603 said this should be part of the "transform" workflow, since parsers are meant to take one input and process it to one entry in the object store, while the transform workflow supports making multiple entries in the object store from a single input (in this case, by regridding to different domains).
This will require

restoring the emissions/flux parsers to their original state
moving the edgar "parser" code to the transform submodule.

It might be easier to do this by cherry picking code from this branch into a new branch after release 0.8.
Another issue is the regridding functions: they add a new dependency (scitools "iris" package), and we already have regridding functionality, but it requires a package (xesmf) that can't be installed with pip. xarray has improved support for iris now, so it might be worth switching to iris.

Thanks @qq23840 for the fixes.
To incorporate this into OpenGHG, I think @rt17603 said this should be part of the "transform" workflow, since parsers are meant to take one input and process it to one entry in the object store, while the transform workflow supports making multiple entries in the object store from a single input (in this case, by regridding to different domains).
This will require
1. restoring the emissions/flux parsers to their original state

2. moving the edgar "parser" code to the transform submodule.

It might be easier to do this by cherry picking code from this branch into a new branch after release 0.8.

Yes, there was actually a current workflow for adding EDGAR data from a database folder which is a bit out of date that we could emulate / incorporate this into - https://github.com/openghg/openghg/blob/devel/openghg/transform/emissions/_edgar.py. This works off a whole folder and you wanted to work from a file but the principle should be similiar.
For the standardise functionality, the idea was that one input file would produce the same sets of datasources whereas for transform this would be able to create different data sources dependent on the inputs. For example for an obs file for a site with data for one gas, this would always create one data source and running with the same file would always create the same data source. For an EDGAR database file the same file could be used but depending on the domain specified this could produce different data sources each time this is run. In this case specifying the domain changes the data source being created.

Another issue is the regridding functions: they add a new dependency (scitools "iris" package), and we already have regridding functionality, but it requires a package (xesmf) that can't be installed with pip. xarray has improved support for iris now, so it might be worth switching to iris.

As we discussed earlier, there isn't a perfect solution around regridding with both xesmf and iris having limitations. If we are able to incorporate the code into this PR into the pre-existing functionality discussed above we could see if this could work with xesmf for now as long as we make sure to give useful advice on how to use it with conda/micromamba.


Switch to on-demand customizable colors

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes

Introduce a ColorProviding protocol to allow FluentUI to ask for customizable colors on demand instead of requiring them all to be pushed before fluent ui is used. This will allow improved perf by adhering more to a pay-for-play model and will allow host apps to have simultaneous experiences showing different themes
Deprecate Colors.primary and friends in favor of these APIs, but use those as a fallback in case a color provider is not set
Modify all controls using these colors to only ask for a color when they have been added to a window, often relying on viewDidAppear: and didMoveToWindow override points
initializeAppearance in FluentUIframework force loads from the bundle before it needs to and prevents simultaneous experiences with different themes by using the global UIAppearance for customization of iOS controls. Deprecate that variant and add a version that takes in a primary color and using the scoped versions UIAppearance APIs so appearance is only impacted if a named type is in the hierarchy. There are still some colors/images force loaded from the bundle in this path, but that can be figured out as a follow-up change.
Add multi-window support to the demo apps and simplify the master/detail pattern since the built-in complexity from Xcode's default is not really needed. Now in the Colors demo, we can swap out the current UIWindow with a subclass that conforms to ColorProviding to have the colors dynamically update using the above pull model.

Verification
Click through all controls in all demos for the giant matrix of

On iPad Pro (4th gen), iPhone 11 (notched), and iPhone 7 (not notched)
iOS 12.4 and iOS 13.4
Default, Green, and No theme (simulate client who hasn't adopted yet)
Light and Dark Mode

Before
After

Pull request checklist
This PR has considered:

[x] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[x] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

can we add a note that MSFNavigationController api has changed for customNavigationBarColor? I think that is one of the "breaking change"?

in your screen shot, there back button differs from the original. 1) there shouldn't be "back" text. (2) it should aways be gray unless it is "done" or checkmark button from barbuttonitems.swift ....


fix: single quote support in eval & major refactor

Fixes https://github.com/privatenumber/webpack-localize-assets-plugin/pull/61 to support single quotes in eval mode
Support multiple hashes in asset name

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hovering doens´t work

When I hover over an menu-item or button it doesn't show the contents, when I click on it (toggle it) it will show the contents.
I even copied the whole source code from your website: https://cameronspear.com/demos/bootstrap-hover-dropdown/

I develop on Google Chrome Version 45.0.2454.101 m

I fixed it but I don't know how, the demo was working all of a sudden.
Also somehow I forgot to add the following classes to my unordered list element .nav .navbar-nav I also had a class on my list that was affecting the functionality of the menu.


added optional chaining

When themes don't have .gh-head-menu, they'll throw an error because the element can't be found. Optional chaining returns undefined and aborts the function.
This fixes the issue in Dope.

Still need to rebuild assets...


Implement jnp.from* array creation functions

Fixes #10045
Note that np.fromregex is not compatible with JAX because it can only create structured arrays, which JAX does not support.

Thanks Roy - I removed fromfile and fromiter based on https://github.com/google/jax/issues/10045#issuecomment-1082153407. PTAL!


Populate the Suggestions Database with Imported Modules

Summary

The Suggestions database is populated on module compilation and only contains entries from the compiled module itself. Suggestion entries from the imported modules should be also a part of the suggestions database.
Value

The user has a complete list of suggestions available in the current scope.
Specification

[ ] When analyzing the module, use the ImportResolver to get the list of imported modules and run the SuggestionBuilder on them.

Acceptance Criteria & Test Cases

Suggestions Database contains entries from the imported modules.

It should not include just the imported modules, it should include all modules that could be imported.
The primary use-case for node searcher in the IDE assumes that called function might require adding a new import to the module.

Just to be clear, the suggestions database needs to contain the module where a symbol is defined, and it should be populated with all available symbols (all registered dependencies) of the current project.

It should not include just the imported modules, it should include all modules that could be imported.
The primary use-case for node searcher in the IDE assumes that called function might require adding a new import to the module.

Just to be clear, the suggestions database needs to contain the module where a symbol is defined, and it should be populated with all available symbols (all registered dependencies) of the current project.

The issue is closed, however the PR and commit description suggest, that the above comments were not addressed? @4e6 @iamrecursion ?


fix: parsing error

for lines ending with line breaks #96

:tada: This PR is included in version 2.1.1-beta.2 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Unable to compile example on Mac OSX

/Users/gabriel/.cargo/registry/src/github.com-1ecc6299db9ec823/tempfile-0.4.0/src/imp/unix.rs:45:17:     45:21 error: unresolved name `path`. Did you mean `dir`?
/Users/gabriel/.cargo/registry/src/github.com-1ecc6299db9ec823/tempfile-0.4.0/src/imp/unix.rs:45          create_unix(path)
                                                                                ^~~~
error: aborting due to previous error
Build failed, waiting for other jobs to finish...
Could not compile `tempfile`.

I'm not actually aware if this is supposed to work right now or not. This error was generated with:
cargo run --example hello-world

Hey!
Try run cargo update then run the example again and see what happens. There's a newer version of tempfile out that will get pulled down when you run cargo update. Let me know how that goes!

Hmm, same error using tempfile v0.5.0
Nadecinogut:rustbox gabriel$ cargo update
Updating registry `https://github.com/rust-lang/crates.io-index`
Nadecinogut:rustbox gabriel$ cargo run --example hello-world
Compiling tempfile v0.5.0
/Users/gabriel/.cargo/registry/src/github.com-1ecc6299db9ec823/tempfile-0.5.0/src/imp/unix.rs:45:17:     45:21 error: unresolved name `path`. Did you mean `dir`?
/Users/gabriel/.cargo/registry/src/github.com-1ecc6299db9ec823/tempfile-0.5.0/src/imp/unix.rs:45         create_unix(path)
                                                                                ^~~~
error: aborting due to previous error
Could not compile `tempfile`.

Looking at the 0.5.0 source code for tempfile, it does just seam like a typo.
#[inline(always)]
#[cfg(not(target_os = "linux"))]
pub fn create(dir: &Path) -> io::Result<File> {
    create_unix(path)
}

Tempfile 0.5.1 has been released. Run cargo update again then try run the example. Works for me now so I'll close the issue. If you still have problems, feel free to re-open!


refactor(projects): design book as project images

Current dependencies on/for this PR:

main

PR #75   👈

This comment was auto-generated by Graphite.


Redefine helm release workflow

Signed-off-by: pgodithi<EMAIL_ADDRESS>
Added helm release workflow
Used https://github.com/helm/chart-releaser-action
Moved helm chart to charts/ folder
Added CHANGELOG.md

Tested the release behavior and able to release opensearch-operator-1.0.1
https://github.com/prudhvigodithi/opensearch-k8s-operator/releases/tag/opensearch-operator-1.0.1
helm repo add opster-pg https://prudhvigodithi.github.io/opensearch-k8s-operator/
pgodithi$ helm search repo opster-pg
NAME                         	CHART VERSION	APP VERSION	DESCRIPTION
opster-pg/opensearch-operator	1.0.1        	1.16.0     	A Helm chart for Kubernetes

@idanl21 @segalziv

Hi @prudhvigodithi.
Some comments from my side:

Should the CHANGELOG.md not be on the root level instead of inside the helm chart?
Any reason why you want the helm-release workflow to run on any push to main instead of on new tags like the release workflow?
Do you see advantages when using helm/chart-releaser-action instead of stefanprodan/helm-gh-pages?

Hey @idanl21 and @swoehrl-mw any other thoughts on this PR? we need this PR to move forward with artifacthub changes.
Thank you

Thanks @prudhvigodithi for your explanations.
So if I understand right the workflow runs on each push to main but checks if the version of the chart was changed and only does a new release then? Works for me. Only aspect I do not really like is that the release is named opensearch-operator-<version> as this could confuse people on what is a new version of the operator and what only of the chart.
In my other projects I prefer to create the tag and then let the workflows do the rest (and update versions in charts/docker images) but I'm not overly attached to that way.
Not really part of this PR but I think we need to change/extend the process we do releases of the operator as for a release of the complete operator we need to

copy any CRD changes into the chart
Build and push docker image
Release new version of chart that uses the new version

And as far as I see this is not yet all done in the release workflow and we should avoid having to manually increase any versions in files.

Hey @prudhvigodithi, i liked the process, i think its right to combine the operator app and the chart under the same repo. its important to note that the operator app and helm chart for the operator are released separately.
just for understating, every push to main will create a new helm release ?

Hey @prudhvigodithi, i liked the process, i think its right to combine the operator app and the chart under the same repo. its important to note that the operator app and helm chart for the operator are released separately.
just for understating, every push to main will create a new helm release ?

Yes @idanl21 every push to main with an increment of chart version should do a new release
Since this PR is merged
I can see a release
https://github.com/Opster/opensearch-k8s-operator/releases/tag/opensearch-operator-1.0.1
I can see a tag cut
https://github.com/Opster/opensearch-k8s-operator/tree/opensearch-operator-1.0.1
helm repo add opensearch-operator https://opster.github.io/opensearch-k8s-operator/
helm install my-opensearch-operator opensearch-operator/opensearch-operator


Enhancement/Parser Rewrite for plugins

Changed the parsers to be built around the plugins. The plugins should determine what the nodetypes are for template, applicability and switches. Based on that nodetype, they should parse their own respective nodes.

In general I think this should be fine, it does look like there are some cases not covered by tests. Can we get a few more tests just to cover those?

In general I think this should be fine, it does look like there are some cases not covered by tests. Can we get a few more tests just to cover those?

added some more unit tests!

applitools diff looks to be a view presentation timing diff, i added a delay in iOS before calling applitools, i can do it for android as well

looks like the codecov patch is hitting the compiler.ts file again which can be ignored haha


Css transition isn't disabled when you switch between weeks and days

When I change the weeks the transition is off, but when I go to the day tabs the transition is not disabled. ( I needed it to be disabled, because I wanted the title text to be aligned on the left )

Hi @alexxsexotic, I created a pen when I align title on the left: https://codepen.io/antoniandre/pen/NQOwKR
I also disabled the transitions in it and it seems to be disabled everywhere except when changing from or to years view. I will have a closer look why soon.

Thank you very mucy

Hi @alexxsexotic, for information I have fixed the year view transition not being disabled.
Now :transitions="false" correctly removes all the transitions. (in version 2.13.1)


Give links to all endpoints on homepage

On https://tc-gen.herokuapp.com
i.e. The page to be redirected in /, should have links to all endpoints directly.
Eg: Upload File, Compile Files, Generate Testcases

I want to work on this.

@aashutoshrathi i will work on this issue

@sweezy793 @mark-nakachon Since this is Kind of front-end task mainly.
Can you both propose a mock for the front page, I'll mark this one as the medium, then.


[fix-#15176] [sql] If the value of the variable contains $ or / symbols, an error may be reported.

Purpose of the pull request

plz close this issue #15176

Brief change log
Verify this pull request

This pull request is code cleanup without any test coverage.

but when i run mvn spotless:apply I don't see any changes to the file cc @qingwli @EricGao888

I  have tested and works well.
Did you run this command in the ds root folder?
looks like you in UI folder

I have tested and works well.
Did you run this command in the ds root folder?
looks like you in UI folder

thanks for you review

What's your test content was?

What's your test content was?

$a

No content value

Works well for me? Any different?

i'm sorry ,you can test this content  select * from a where a != !{a}

Please add Ut for this, this issue is not clear.

Please add Ut for this, this issue is not clear.

i will add UT, tomorrow

Please add Ut for this, this issue is not clear.

dnoe

 Works well for me? Any different?

hi @qingwli I had add UT,PTAL

@ruanwenjun PTAL

@caishunfeng @SbloodyS PTAL


feat: add close tab button config (#720)

Not sure this is the best solution, see my comment https://github.com/zeit/hyperterm/issues/720#issuecomment-250276027

Probably easier to just add a line of CSS?

good call @rauchg! leave it to me to over-engineer...

@rauchg
now that I've looked at it, I'm not sure how to do this with css. I thought this would work:
// lib/components/tab.js
styles() {
  ...
    icon: {
      ...
      display: this.props.showCloseTabBtn
        ? 'inline-block'
        : 'none',

but styles are not parsed by webpack before they are bundled, and seeing how there are no css files, I can't simply write a css rule.
it seems like a small case for beefing up webpack's css handling, do you have any suggestions on where to start?
thanks

I think he means in your settings styles, not as a change to the project.

Thank you @knownasilya, that makes sense.

Do it via CSS; we won't introduce a config option for this. :blush:

Thanks for giving this a shot @bildepunkt.

np :) it was fun

@leo I'm assuming until there's a solution for #14 I'll have to create a plugin for this?

Created a plugin for now: https://github.com/karlhorky/hyper-no-close


undefined value when dnd from file explorer

Does this issue occur when all extensions are disabled?: Yes/No

Version: 1.69.0-insider (user setup)
Commit: ca8a8376b46b63db7379cbcc827c75d92bf25e14
Date: 2022-06-14T05:15:51.831Z
Electron: 18.3.2
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000
Steps to Reproduce:

Clone the dnd sample: https://github.com/microsoft/vscode-extension-samples/tree/main/tree-view-sample
Add 'text/uri-list' to the dropMimeTypes of  TestViewDragAndDrop
Add some logs in handleDrop(...):

sources.forEach((value, key) => {
    console.log(`${key} : ${value.value}`);
});

Launch the extension
Drag files and folders to the dnd sample view, some undefined value can be seen. And such value won't be seen when drag files from the system file explorer:

https://user-images.githubusercontent.com/6193897/173537671-d6852fe6-3dd5-4097-8d1e-677e9dd3ea05.mp4

@jdneo value is expected to be undefined here. Instead, you should call asString() to get the data.
value is for custom data provided by extensions when dragging within a tree. Example:

Extension implements handleDrag and handleDrop.
In handleDrag, they add some DataTransferItems to the DataTransfer. These items have a value that is a custom object.
In handleDrop, they get their items from the DataTransfer and use value to get their custom object back.

@mjbvz does this align with how you are using value in editor drag and drop?

I changed the code to
sources.forEach(async (value, key) => {
    console.log(`${key} : ${await value.asString()}`);
})

It's still undefined. Is there anything I missed?

I'm seeing that mostly work. Console output:
[Extension Host] text/uri-list : undefined (at d:\repos\Microsoft\vscode-extension-samples\tree-view-sample\src\testViewDragAndDrop.ts:65:12)
console.ts:137
[Extension Host] text/uri-list : file:///d%3A/repos/alexr00/playground/no_extension
file:///d%3A/repos/alexr00/playground/not_in_ws
file:///d%3A/repos/alexr00/playground/test2.js

There does appear to be an issue where text/uri-list is in there twice, which is a bug.

Looks like the double text/uri-list bug will be fixed by https://github.com/microsoft/vscode/pull/150468 though.

Have no idea why but on my machine, if I drag a folder, it still only has an undefined entry: (insider is updated to latest)
https://user-images.githubusercontent.com/6193897/173963598-33e571f4-5d4a-46b0-8b0c-364aeb369d7f.mp4

I can repro when I drag a folder.

Tested with https://github.com/microsoft/vscode/pull/150468, and the undefined issue is also resolved by that PR.

Fixed by https://github.com/microsoft/vscode/pull/150468


minor code change causes stream to block

hi,
i've been using async-std for some time now and I think I might have hit a bug.
my code is here https://gitlab.ensimag.fr/wagnerf/listen/-/tree/blocking
(it is on the "blocking" branch)
if you launch it right now (cargo run) it's fine : you can use left/up arrows in the graphical menu to select an entry.
now, in src/video/tune_selector.rs
if you comment line 84 and uncomment line 85 suddenly the keyboard event stream becomes unresponsive. you cannot select anything anymore in the menu.
it is surprising for me because the changes between the working and non-working version are really really minor.
what's more it means touching the video frames stream makes the keyboard events stream stall.
i'm not sure where to go from there, so any help would be appreciated.
note that i can postpone this modification, so there is no emergency for me.

Hi, I am not super familiar with GitLab, can you produce a diff view somehow?

Hi, I am not super familiar with GitLab, can you produce a diff view somehow?

hi, yes, thanks for asking that's very nice.
by writing things down for this post I found the difference between the two versions of the code but I still cannot understand why this causes keyboard events to disappear.
so, here is the idea:
i have two streams:

an sdl events stream:

    let mut sdl_events = SDLEvents::new(sdl_context)?; // a stream
    let tune_selection = sdl_events.try_for_each(|e| {
        // EVENT HANDLER HERE...
        Ok(())
    });

this stream is coming from a custom struct where I implement the Stream trait myself:
impl Stream for SDLEvents {
    type Item = sdl2::event::Event;

    fn poll_next(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Option<Self::Item>> {
        if Pin::new(&mut self.delay).poll(cx).is_pending() {
            return Poll::Pending;
        }
        if let Some(event) = self.pump.poll_event() {
            Poll::Ready(Some(event))
        } else {
            // if we fail polling we should wait a bit before retrying
            self.delay.reset(std::time::Duration::from_millis(50));
            Poll::Pending
        }
    }
}

In both versions of my code this stream is unchanged.
Now the other stream is a video stream, displaying the different frames.
In the working version it is like this:
    let frame_duration = std::time::Duration::new(1, 0) / display.framerate as u32;
    let mut frames = async_std::stream::interval(frame_duration);
    let video = frames.try_for_each(|_| {
        selector.as_ref().render(display)?;
        Ok(())
    });

The render method is clearing the screen and then displaying all gfx primitives then presenting the screen.
In the stalling version it is like this:
    let video = display.play(|display| {
        selector.as_ref().render(display)?;
        Ok(())
    });

with the play method being:
    pub(crate) async fn play<V: FnMut(&mut Self) -> Result<()>>(
        &mut self,
        mut video: V,
    ) -> Result<()> {
        let mut ticks = async_std::stream::interval(Duration::new(1, 0) / self.framerate as u32);
        ticks
            .try_for_each(|_| -> Result<()> {
                self.clear();
                video(self)?;
                self.canvas.present(); // CULPRIT LINE
                Ok(())
            })
            .await
    }

So, the CULPRIT LINE here is the one causing problems.
I forgot that the render method already presents the canvas and so it is presented twice in this version.
In all cases both streams are then consumed using a race:
    let tune = async_std::task::block_on(
        async { Result::Err(video.await.err().unwrap()) }
            .race(async { Result::Ok(tune_selection.await.err().unwrap()) }),
    );

So, I can easily solve my problem by removing the call to the second present. However this still does not explain why a change in the video stream make all keyboard events disappear.
I know they do because if I add some println in the keyboard events handler it stops printing almost immediately.
do you have any idea of what could be going on ?
thanks a lot for your time.
If you want to get the code you can use git clone https://gitlab.ensimag.fr/wagnerf/listen.git and then git checkout blocking to switch to the blocking branch.

hi, yes, thanks for asking that's very nice.
by writing things down for this post I found the difference between the two versions of the code but I still cannot understand why this causes keyboard events to disappear.
so, here is the idea:
i have two streams:

an sdl events stream:

    let mut sdl_events = SDLEvents::new(sdl_context)?; // a stream
    let tune_selection = sdl_events.try_for_each(|e| {
        // EVENT HANDLER HERE...
        Ok(())
    });

this stream is coming from a custom struct where I implement the Stream trait myself:
impl Stream for SDLEvents {
    type Item = sdl2::event::Event;

    fn poll_next(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Option<Self::Item>> {
        if Pin::new(&mut self.delay).poll(cx).is_pending() {
            return Poll::Pending;
        }
        if let Some(event) = self.pump.poll_event() {
            Poll::Ready(Some(event))
        } else {
            // if we fail polling we should wait a bit before retrying
            self.delay.reset(std::time::Duration::from_millis(50));
            Poll::Pending
        }
    }
}

In both versions of my code this stream is unchanged.
Now the other stream is a video stream, displaying the different frames.
In the working version it is like this:
    let frame_duration = std::time::Duration::new(1, 0) / display.framerate as u32;
    let mut frames = async_std::stream::interval(frame_duration);
    let video = frames.try_for_each(|_| {
        selector.as_ref().render(display)?;
        Ok(())
    });

The render method is clearing the screen and then displaying all gfx primitives then presenting the screen.
In the stalling version it is like this:
    let video = display.play(|display| {
        selector.as_ref().render(display)?;
        Ok(())
    });

with the play method being:
    pub(crate) async fn play<V: FnMut(&mut Self) -> Result<()>>(
        &mut self,
        mut video: V,
    ) -> Result<()> {
        let mut ticks = async_std::stream::interval(Duration::new(1, 0) / self.framerate as u32);
        ticks
            .try_for_each(|_| -> Result<()> {
                self.clear();
                video(self)?;
                self.canvas.present(); // CULPRIT LINE
                Ok(())
            })
            .await
    }

So, the CULPRIT LINE here is the one causing problems.
I forgot that the render method already presents the canvas and so it is presented twice in this version.
In all cases both streams are then consumed using a race:
    let tune = async_std::task::block_on(
        async { Result::Err(video.await.err().unwrap()) }
            .race(async { Result::Ok(tune_selection.await.err().unwrap()) }),
    );

So, I can easily solve my problem by removing the call to the second present. However this still does not explain why a change in the video stream make all keyboard events disappear.
I know they do because if I add some println in the keyboard events handler it stops printing almost immediately.
do you have any idea of what could be going on ?
thanks a lot for your time.
If you want to get the code you can use git clone https://gitlab.ensimag.fr/wagnerf/listen.git and then git checkout blocking to switch to the blocking branch.


Check if analyzers are possible for Razor views

This would allow us to integrate https://www.caniemail.com/api/data.json and display warnings if we are using features not supported by our mail client targets.

May not be so trivial. Roslyn doesn't seem to have an actual semantic model for the DOM; it writes most of it as literals. E.g.,
<head>
    <title></title>

    <style type="text/css">
        * { text-align: center;
        }
    </style>

…seems to become:
            WriteLiteral("\n<!DOCTYPE html>\n\n<html lang=\"en\">\n");
            __tagHelperExecutionContext = __tagHelperScopeManager.Begin("head", global::Microsoft.AspNetCore.Razor.TagHelpers.TagMode.StartTagAndEndTag, "…", async() => {
                WriteLiteral("\n    <title></title>\n\n    <style type=\"text/css\">\n        * { text-align: center;\n        }\n    </style>\n\n    ");

We can debug this behavior with <EmitCompilerGeneratedFiles>true</EmitCompilerGeneratedFiles> in the .csproj.

Might be doable.
4>Sample.cshtml(25,11): Warning RzMa0001 : The tag '<details>' has limited support


Options to include (or remove?) public archive

Hello @UlisesGascon
The report include archived repository, I think it should not (what do you think?)
For example documentation-ui is archived in the list here:
https://github.com/NodeSecure/Governance/blob/main/reports/ossf_scorecard/report.md
Maybe we need an option to include or exclude public archive

Yes! Great catch @fraxken , we should ignore them in the collection step

I'll take the leadership in the research !


feat(plugin-bundle-node): Add minNodeVersion option.

Related to #37

+1, thanks for porting over


⚠ Bofrost.fr has degraded performance

In 5856118, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1982 ms

Resolved: Bofrost.fr performance has improved in 99fbe79.


Can't re-save netCDF after opening it and modifying it?

Code Sample, copy-pastable
import xarray as xr
import numpy as np
import pandas as pd

filename = 'foo.nc'

print('creating fresh file')
temp = 15 + 8 * np.random.randn(2, 2, 3)
precip = 10 * np.random.rand(2, 2, 3)
lon = [[-99.83, -99.32], [-99.79, -99.23]]
lat = [[42.25, 42.21], [42.63, 42.59]]
ds = xr.Dataset(
    {
        'temperature': (['x', 'y', 'time'], temp),
        'precipitation': (['x', 'y', 'time'], precip)
    },
    coords={
        'lon': (['x', 'y'], lon),
        'lat': (['x', 'y'], lat),
        'time': pd.date_range('2014-09-06', periods=3),
        'reference_time': pd.Timestamp('2014-09-05')
    })
ds.to_netcdf(filename)

del ds

ds = xr.open_dataset(filename, autoclose=True)
print('opened file')
print(ds['temperature'])
ds['temperature'][0, 0, 0] += 1000

ds.to_netcdf(filename) ### Crashes

# import os
# ds.to_netcdf(filename + '2')
# os.rename(filename + '2', filename)

Problem description
<snipped very long stacktrace, can produce if desired>
OSError: [Errno -51] NetCDF: Unknown file format: b'/path/to/foo.nc'

I encountered this problem when opening a netCDF file, modifying it, and trying to save it back. This is with netCDF4==1.3.1 and scipy==1.0.1. (Potentially related: #2019?)
Expected Output
If instead of to_netcdf overwriting the just-opened file, I write to a new file and then os.rename (see the three commented lines above) the new file to the original location, all is well. ncdump reports that my change took.
 This lazy loading breaks when you override the source file. As a user, the work around is to always load files entirely from disk, e.g., by calling .load(), or to not attempt to override existing files.
I'm not quite sure how we should improve this, but this does certainly come up with some frequency, especially for new users. A friendlier warning/error would be nice, but I'm not sure how to detect this behavior in general (this information is not currently very accessible).
We could potentially always write to temporary files in to_netcdf() and then rename in a final step after writing the data. As a bonus, this results in atomic writes on most platforms.

Thanks @shoyer! Question: I'd wondered if maybe I open_dataset a NetCDF file and then modified one of the data_vars, maybe xarray would update the file without me doing anything, but that wasn't the case, correct? So only the loading of data is lazy, data modified is modified only in memory? Thanks for the explanation!

That's right, xarray will never modify a file on disk unless you use to_netcdf().


Add Request Campaign to Mrkt Marketo Gem

This adds the functionality to request a Smart Campaign for certain leads to our forked Gem "Mrkt" which we will from now on use to make REST requests to Marketo. We need the Smart Campaign feature to trigger emails.

+1

+1
nice. are we making a PR to the official repo?

That's the plan :) @christofdamian – Is Marvin already active in this repo and will eventually merge this PR or do we have to do that manually?

should happen soonish

poor marvin didn't have access :-)

Thanks :)

Opened a PR to the original repo: https://github.com/raszi/mrkt/pull/4


Add notes in the codes and add some instructions in the Doc/README.md

Please add more notes for your codes. Especially for the functions, please specify what are your inputs.

more comments for the functions are added

Great, thanks.


Custom query fix

Pull request for RDF connector changes

I have added comments, kindly review them.
I will test the connector next week