Work with the storage driver to minimise work when paging

Storage drivers may be able to take advantage of the hint to start
their walk more efficiently. Start with s3, whose API takes a
start-after parameter. Registries with many repositories can drastically
reduce calls to s3 by telling s3 to only list results lexographically
after the last parameter.
Fixes #3678
Signed-off-by: James Hewitt<EMAIL_ADDRESS>

Codecov Report

Merging #3685 (e393ba5) into main (8857a19) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3685   +/-   ##
=======================================
  Coverage   56.58%   56.58%
=======================================
  Files         103      103
  Lines        7520     7520
=======================================
  Hits         4255     4255
  Misses       2596     2596
  Partials      669      669

Impacted Files
Coverage Δ

...tribution/registry/storage/driver/storagedriver.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

To quantify, I pushed 2000 single tag repositories to my registry and queried them. Without this change, listing the last 10 took 3.6s (11 calls to S3). With this change, it took 0.216s (1 call to S3). I expect with a lot of tags in those repos that would have been an even bigger improvement.
Before:
DEBU[0008] s3aws.ListObjectsV2Pages(docker/registry/v2/repositories/)  go.version=go1.18.3 http.request.host="localhost:5000" http.request.id=59df72fe-5726-4313-be90-31f40e63e70d http.request.method=GET http.request.remoteaddr="<IP_ADDRESS>:55566" http.request.uri="/v2/_catalog?n=10&last=busybox-1980" http.request.useragent=curl/7.82.0 instance.id=d940a3fc-b894-4f07-a8dc-be8787850260 trace.duration=3.614610194s trace.file=/home/jammy/go/src/github.com/distribution/distribution/registry/storage/driver/s3-aws/s3.go trace.func="github.com/distribution/distribution/v3/registry/storage/driver/s3-aws.(*driver).doWalk" trace.id=858bec3f-0ae7-4012-9b4c-5404f78c6a46 trace.line=1096 trace.parent.id=235f11a8-a5a7-48f8-ac8d-fc09f7d2620d version=v2.7.0-1993-g8857a194.m

After:
DEBU[0006] s3aws.ListObjectsV2Pages(docker/registry/v2/repositories/)  go.version=go1.18.3 http.request.host="localhost:5000" http.request.id=be2ccb76-cd35-41b6-af4f-545a22d7858b http.request.method=GET http.request.remoteaddr="<IP_ADDRESS>:47336" http.request.uri="/v2/_catalog?n=10&last=busybox-1980" http.request.useragent=curl/7.82.0 instance.id=77474527-797d-46eb-a6c4-10d0a30c3c18 trace.duration=216.167877ms trace.file=/home/jammy/go/src/github.com/distribution/distribution/registry/storage/driver/s3-aws/s3.go trace.func="github.com/distribution/distribution/v3/registry/storage/driver/s3-aws.(*driver).doWalk" trace.id=8f84cb0d-d6ac-48e3-a506-7b411ab3bd74 trace.line=1097 version=v2.7.0-1994-gd1929f38.m

I realised that the interface to this one was a bit focused on the catalog use case only, so have refactored a little to make it more obvious to direct users of Walk.
Also added another test to ensure that the s3 driver actually does filter correctly when it is used.

This PR has a lot of overlap with (but I think just replaces) #3538. That one can be closed when this is merged, or would at least need reworking to add a new functional option if there is still performance benefit after these changes.

Is #3360 also similar?

Is #3360 also similar?

Yes, but it looks like they did more storage drivers than just S3. I was considering looking at other drivers after getting the S3 support merged.

Rebased. For whatever reason, it seems I'm now considered a first time contributor again, so can I please have an OK to test?

For whatever reason, it seems I'm now considered a first time contributor again, so can I please have an OK to test?

Weird, looks like somethings changed in the repo settings. I changed it now so hopefully, this will start working again the way it did before 🤞

Rebased to re-run the tests as this is a fairly involved change, but I think its still worth getting in.

@Jamstah mind rebasing?

I think there have been other changes here, need to re-run a test to see if the bug still exists.

OK, bug still exists in main so have rebased and retested this one.

All hail the linter, for the space should most definitely have been a tab :)
(I jest, I :heart: linters)

Seems like this PR needs another rebase. Sorry about that.

Hey, I'm all for an active project!
Rebased.

This needs a rebase again 😅

It turns out my first attempt for updating the filesystem walk regressed #1854, its now fixed for the fs walk but I want to test the S3 walk under the same conditions.

There is a conflict now 😅

Adding reviewer of #3145 as this PR has similar performance improvements and I reverted #3145 in #4004.
Also FYI @eyJhb as the author of #3145.


feat(api): support has many directive for rds models

Description of changes
Add support for RDS hasMany directive.

Adds new attribute references to mention fields used for JOIN.
Uses existing layer LIST operation to fetch the child records.

CDK / CloudFormation Parameters Changed
NA
Issue #, if available
NA
Description of how you validated changes

E2E Test
Manual Test
Unit tests

Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing this PR in favor of https://github.com/aws-amplify/amplify-category-api/pull/1778/files which includes support for hasMany, hasOne and belongsTo directives.


[API NEW][SET FUNC] Add set functions

Description
Add unique_all, unique_inverse and unique_values specified in data-api-standard
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.


zsh: segmentation fault (core dumped)  wf-info

my PKGBUILD
pkgname=wf-info
pkgver=0.1
pkgrel=1
pkgdesc="A simple wayfire plugin and program to get information from wayfire"
arch=(x86_64)
url=https://github.com/soreau/wf-info
license=(unknow)
depends=(wayfire glm)
makedepends=(meson ninja pkg-config git cmake  wayland-protocols)
conflicts=("${pkgname}-git")
source=("git+https://github.com/soreau/wf-info.git")

build() {
	rm -rf build
	arch-meson "${pkgname}" build
	ninja -C build
}

check () {
	meson test -C build
}

package() {
	DESTDIR="${pkgdir}" ninja -C build install
}
sha256sums=('SKIP')

What did you do that caused segfault and please post the backtrace from wayfire stdout.

plugins =

add
wf-info
works well thanks


[Question] Error using yt-dlp with php

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question
I'm on Debian 9 and I used to use youtube-dl with php (simple "exec"), for a local script but when I replace it with yt-dlp I realize that it does not work.
In the error logs, yt-dlp tries to use python 2.7, but I installed python 3.9.6 with pyenv, and when I do "python -V", I get 3.9.6.
When I execute the script in command line "/usr/bin/php script.php" there no worries, it is when I execute the script with my browser that the error occurs.
I must be missing something, but after a lot of research I couldn't find what. Any idea ?

yt-dlp has a /usr/bin/env python3 shebang. So as long as it is set correctly, exec should work. But I know nothing about php to provide further help

Probably your browser is running as sudo or as a third user.
afaik since u installed using pyenv, it only affects your user and not the other.
Why not install latest python using apt?

You need to have your script run yt-dlp as /home/yourusername/.pyenv/shims/python /path/to/yt-dlp.

You need to have your script run yt-dlp as /home/yourusername/.pyenv/shims/python /path/to/yt-dlp.

This time I encounter a "Permission denied". However, running the script from the terminal everything works. Namely, my "script.php" file is in www-data:www-data. I use nginx and php 7.2
[03-Aug-2021 13:42:12] WARNING: [pool www] child 13068 said into stderr: "sh: 1: /root/.pyenv/shims/python: Permission denied"

Probably your browser is running as sudo or as a third user.
afaik since u installed using pyenv, it only affects your user and not the other.
Why not install latest python using apt?

Because I use applications that are not yet compatible with a newer python from what I understand.

Since this is not related directly to yt-dlp and none of the devs here seems to know how to fix this, I am closing the issue. You are probably better off asking this somewhere that provides help for PHP


Switching between atmos versions (asdf plugin)

Have a question? Please checkout our Slack Community or visit our Slack Archive.

Describe the Feature
Sometimes I need to switch between atmos versions in order to debug something or perhaps a client uses version X and another client uses version Y. I usually have to run another go install of a specific atmos release or use apt install atmos="<version>-*" from geodesic.
It would be nice to use a tool like asdf with an atmos plugin. This will put atmos on par with terragrunt
https://asdf-vm.com/
https://github.com/asdf-vm/asdf
https://github.com/asdf-vm/asdf-plugins
Another advantage of supporting asdf is that we can pin versions of atmos using the asdf .tool-versions file for users that want to use atmos within geodesic and from outside.
This will allow using commands like this to download and pin the version using a .tool-versions file
asdf plugin install atmos
asdf atmos install latest
asdf local atmos latest

@nitrocode whipped this up! Let me know if it works for you.
https://github.com/cloudposse/asdf-atmos

Let's open up subsequent issues on the other repo.

https://github.com/asdf-vm/asdf-plugins/pull/1042


Deprecate krux-postscribe

https://www.npmjs.com/package/krux-postscribe needs to be deprecated

Did this. Do you not see it deprecated when you install it?

Lemme try again

Yeah it's fine now.


whois: 5.5.17 -> 5.5.18

Changes: https://github.com/rfc1036/whois/compare/v5.5.17...v5.5.18
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.


lib/game: add Wolfram Rule 30, Rule 90, Rule 110, and Rule 184

This PR modifies lib/game to add Wolfram Rule 30, Rule 90, Rule 110, and Rule 184

Thanks for review @acifani now merging.


Remove false opens

This code creates false opens in zones as it parses the USER/ZONE Field in a Virtual Keypad Update command as a zone update. This causes a false open (depending on the value in the hex field). Per the TPI programmer's guide, this argument may refer to a zone, but it doesn't constitute a zone open: http://forum.eyez-on.com/FORUM/download/file.php?id=224&sid=69c1d5b58d0f31b86430ed20892a9969 page 3/4.

@haywirecoder - sorry, didn't mark you as a reviewer


Rule Ordering

Right now rules are not ordered in any specific way except for the order they are defined (imported).
If ui-router could be smart enough to put non-variable routes at a particular url level before those that have variables it could better handle wildcard use-cases.
i.e. I have a url structure that looks like the following:
/
/page-a
/page-b
/<name>
/<name>/page-a
/<name>/page-b
in this case <name> can be anything (mapped to some data on the backend).  The problem comes into play when "/<name>" is defined before "/page-a", because the later matches the former rule and therefore we show the wrong content.
This can be solved by putting all these routes into a specific file and dictate the ordering, but when trying to keep things modular it breaks because order is not deterministic.
If this is not a generally liked approach, having the ability to override the addition of the rules, so that I can dictate the ordering would be just as acceptable.

Yeah, rule ordering is something we've considered, but the algorithms can get to be complicated, and it would have to be pluggable, to account for different use cases. Definitely desirable, but not really on the roadmap as yet.


Expose Merkle Proof

Forward port #1583
The port is a bit more involved with structured keys but I think the result is quite nice. There are a few quirks in the current API organisation that I plan to tackle in a separate PR.

(I'll fix the CHANGELOG in another PR which will fix all the 2.9 items in one go)

I'll split this PR a bit to simplify the review.

Now rebased on top of #1612

The forward port for merkle proofs is done, so closing this.


Move sidebar copyright & powered-by to a partial

These changes are already in place as part of PR #1


Level Rankings on play page

This is a pr for #124, it includes:

Renaming a leaderboard query
Creating a new leaderboard query for getting rankings per level
Displaying the leaderboard data for each level on the play page

This has now been edited to combine the two leaderboard queries and have an optional level that can be passed in.

Updated this pr to use an index when fetching the leaderboard for each level and also added buttons from the mini leaderboards on the watch and play pages that lead to the main leaderboard page.


Old dependency of @primer/primitives require node v18

Description
Can we please move on from "@primer/primitives": "^8.2.3" to v9 or v10? I know they haven't been released for a long time, but v8.2.4 requires "node": ">=18.18.0 <18.19.0", which is only supported until 2025. (Another reason is that my environment uses node v22, though strictly speaking I can --ignore-engines but it's not the best idea.) Later versions of the package do not require node v18. Is there a specific reason we cannot bump its version?
https://github.com/datalayer/primer-addons/blob/9ac72605eae0a81a19779481d247f3f3e4d4ddf7/package.json#L28

Thank you @jophy-ye We have merged https://github.com/datalayer/primer-addons/pull/37 (which removes that dependency, at some point some time ago, that has been needed to ensure compilation, but we don't use it directly, so good to simply remove it).
0.3.2 is released with that change<EMAIL_ADDRESS>

@echarles @fcollonval Thanks! Pre-happy holiday!


Is there a MSBuild variable to add a triplet overlay path like VCPK_OVERLAY_TRIPLETS?

I have looked in the documentation, but couldn't find an MSBuild variable/property to add a triplet overlay path.
Does this exist? If not, this is a feature request... :)
In combination with the manifests feature it would be useful to keep custom triplets in the project source repo, and not have to modify global environment variables like VCPKG_OVERLAY_TRIPLETS but instead keep the configuration in an MSBuild .props file.
  <!-- Custom project triplets for vcpkg -->
  <PropertyGroup>
    <VcpkgTriplet Condition="'$(Platform)'=='Win32'">x86-windows-$(PlatformToolset)</VcpkgTriplet>
    <VcpkgTriplet Condition="'$(Platform)'=='x64'">x64-windows-$(PlatformToolset)</VcpkgTriplet>
    <VcpkgOverlayTriplets>$(MSBuildThisFileDirectory)/custom_triplets</VcpkgOverlayTriplets>
  </PropertyGroup>

example folder structure in project root:
src
vcpkg_installed
custom_triplets // contains triplet files
vcpkg.json
project.vcxproj

Building in Visual Studio would then work automatically with MSBuild.

I solved this using
<VcpkgAdditionalInstallOptions>--overlay-triplets=$(VcpkgManifestRoot)\triplets $(VcpkgAdditionalInstallOptions)</VcpkgAdditionalInstallOptions>

So there is no urgent need for a specialized property for this.

Please keep this issue open until we fix it by adding a feature.


DEFAULT.QC_Accessibility_Eservices - Defect # 489

If the link opens in a new window, check that the link has the new window icon and tooltips text to warn the user.
On landing page feedback link has no tooltips to warn user

Duplicate of #83


matcher() builtin, like joiner() and cycler()

Any interest in including something like this:
https://github.com/miracle2k/jinja2utils/blob/master/src/jinja2utils/match.py
I have just written this code a second time, needing it again and not remembering I had already implemented it once.
It seems to fit very well alongside joiner() and cycler(), and presumably is a common usecase.

Not a bad idea. Though the default should probably be undefined for consistency with the inline if expression.

Which value specifically do you suggest should default to undefined?
I have implemented the code as-is here:
https://github.com/miracle2k/jinja2/compare/90-matcher


WIP: Run consul agent within trento

This is an early attempt at getting consul running within trento. Right now we are cutting many corners to get this working and it has a few limitations, most importantly:

the consul-agent-data directory needs to be created in the working directory where trento runs from. This is hardcoded and will be changed later, but the most important issue is that it needs to be manually cleaned on every run. Still trying to figure out why
tls has been disabled. But since it's communicating through localhost it might not be a minor problem. Still need to understand better the implications of this.
I had to introduce a very ugly 60 sleep in agent/agent.go:139  so that it would not give up connecting to consul too early.
I had to fork github.com/aquasecurity/bench-common to fix a dependency problem

The code needs to be cleaned and re-thought, so any suggestion is welcome on how to implement this better.

This was a great exercise, but we ended up solving the problem with the installer, and Consul is eventually going to go away in our future architecture plans.


Migration fails on MySQL when upgrading to Wagtail>=2.12rc1

Issue Summary
The wagtailembeds.0008_allow_long_urls migration added in 2.12rc1 (https://github.com/wagtail/wagtail/commit/32ade8672fe32223b6ebdcb2255bf4de16908f97) results in the following error on MySQL 8.0 (and presumably also older versions):
MySQLdb._exceptions.OperationalError:
(1101, "BLOB, TEXT, GEOMETRY or JSON column 'thumbnail_url' can't have a default value")

This is expected behaviour. According to https://dev.mysql.com/doc/refman/8.0/en/blob.html:
BLOB and TEXT columns cannot have DEFAULT values.

Steps to Reproduce
Update: Migration from 2.11 to 2.12 is not needed; can be reproduced more simply in steps 1–5 using 2.12rc1<=wagtail<=2.12.2
This issue can be reproduced on a fresh Wagtail project as follows:

Install Wagtail < 2.12 >= 2.12rc1 with pip install 'wagtail<2.12' pip install 'wagtail>=2.12rc1<=2.12.2'
Start a new project wagtail start testproject
Configure the DATABASES setting to use a MySQL database

DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.mysql',
        'NAME': 'testproject',
        'USER': 'testuser',
        'PASSWORD': 'testpass'
    }
}

Prepare the database

mysql> create database testproject;
mysql> create user 'testuser'@'localhost' identified by 'testpass';
mysql> grant all privileges on testproject.* to 'testuser'@'localhost';

Run initial migrations ./manage.py migrate
6. Upgrade to Wagtail 2.12rc1 or greater with, e.g. pip install --upgrade wagtail==2.12.2
7. Re-run migrations ./manage.py migrate

Additional Notes
Prior to MySQL 8.0.13, BLOB and TEXT could not be assigned DEFAULT values. As of 8.0.13, they can be assigned defaults, but only in the form of expressions as noted in https://dev.mysql.com/doc/refman/8.0/en/data-type-defaults.html#data-type-defaults-explicit
Technical details

Python 3.8.3
Django 3.1.7
Wagtail 2.11.5 -> 2.12rc1...2.12.2
MySQL 8.0.21 for osx10.15 on x86_64 (Homebrew)
mysqlclient 2.0.3

Hi @stvnw,
Thanks for reporting - will investigate further. It seems odd that this error apparently doesn't get triggered when setting up a fresh database on 2.12 (as our CI tests do), despite it running the same sequence of migrations...

Actually it does get triggered on a fresh 2.12 install in my environment as well, so I've edited the original issue slightly.
I have STRICT_TRANS_TABLES enabled on my database (the default) but turning it off doesn't help.
The CI tests are run against MySQL 5.7, so I tried them against MySQL 8.0. Interestingly they pass on the Python 3.6 job and fail on the 3.9. Both jobs are using mysqlclient 1.4.6.

Any update for this issue?  Also meet this problem
django.db.utils.OperationalError: (1101, "BLOB, TEXT, GEOMETRY or JSON column 'thumbnail_url' can't have a default value")
All works good on wagtail 2.11.3.

Have replicated the issue here on MySQL 8.0.23 on Ubuntu, with Python 3.6.9. Looks like this broke between Django 2.2 and 3.0 (which accounts for the different CI results).
Judging from this older Django issue, my understanding is that Django is supposed to abstract away this MySQL-specific detail, in which case I think this would count as a Django bug. Will work further on isolating this as a non-Wagtail test case.

https://github.com/django/django/commit/6b16c91157512587017e9178d066ed1a683e7795 / https://code.djangoproject.com/ticket/30712 appears to be the offending commit.

Issue now opened with Django: https://code.djangoproject.com/ticket/32503

Reopening as this is a hard blocker to anyone on current MySQL from upgrading to 2.12, and fortunately there's a fairly simple workaround (PR to follow shortly).


Correctly resolve collections(.abc) class/method Definition to source file instead of typing.py or typeshed-fallback

Is your feature request related to a problem? Please describe.
Currently, Pyright resolves the Definition of types like collections.abc.MutableMapping as their alias assignment line in typing.py.
MutableMapping = _alias(collections.abc.MutableMapping, 2)

For methods like MutableMapping.pop it seems to resolve the Definition to typeshed-fallback instead, showing a list of overloads.
There are also currently no docstrings for collections.abc classes and their methods.
Describe the solution you’d like
As this is a relatively common and core part of type-hinting and part of stdlib, I'd really like custom behaviour here which can find the correct Definition in collections(.abc) which contains the actual implementation of these types or base classes.
This likely would require some custom behaviour for typing._SpecialGenericAlias which makes Pyright consider it more akin to a type alias.
Ideally, "Jump to Definition" would show the actual source code of how something is implemented.
"Jump to Declaration" could then show the typing.py alias line, and only "Jump to Type Definition" would end up in typeshed-fallback.
At the same time, this should allow docstrings for these types to be resolved properly.
(Note: This was observed in both the latest release Pylance in vscode or latest release Pyright in vscode with Python 3.12.5 on Windows, but has existed across multiple machines and combinations of versions, which is why i assume this is not a bug but just a missing feature)

I'm not sure this move was correct. I just tested with the "Pyright" extension (while Pylance is uninstalled) and it has the exact same issues.
This is also not about "completion suggestion" at all... its about the "Find Definitions to quickly go to the location of a symbol’s definition" and "Hover over symbols to provide type information and doc strings" features listed here https://microsoft.github.io/pyright/#/features?id=language-server-support
Unless the pyright language server should also have issues reported in pylance-release?

as for repro, not sure how to provide one...
its really just
from collections.abc import MutableMapping

MutableMapping.get

and then hover over .get to see the lack of docstrings. Or place the cursor on .get and "Go to Definition" etc.

Pyright is focused on type checking functionality. It offers some basic language server features, but these are maintained by the pylance team. In general, any bugs or feature requests related to language server features get moved to the pylance-release project, and any bugs or feature requests related to type checking features get moved to the pyright project.

the issue is our code that finds matching py from pyi (sourcemapper) doesn't know how to handle this case. code structure for these types is different between py files and pyi files.
ex) py has MutableMapping defined in _collections_abc.py but pyi has it defined in typing.pyi
probably need to add special case to handle these cases.

@heejaechang i'm thinking maybe go to definition should resolve the alias and never go to 'MutableMapping = _alias(collections.abc.MutableMapping, 2)'  line.
for instance go to def on np in 'np.array' goes to the numpy
import numpy as np

arr = np.array([1, 2, 3])


Conditional publishing

Added a simple extension of RabbitMQBolt that can be used as a parent for classes that need to decide whether to publish a message on a tuple basis.
The same functionality could be achieved by adding the shouldPublish method to TupleToMessage and not adding a new subclass of RabbitMQBolt, but I thought that this approach was more clean since it preserves the semantics of TupleToMessage (convert a Storm Tuple to a RabbitMQ message).
We could also completely ditch ConditionalPublishingRabbitMQBolt and add the functionality in RabbitMQBolt.
Let me know what you think.
Thanks,
George

Looks good :)

Awesome, thanks!
On 8 Nov 2015 02:05, "Peter Pathirana"<EMAIL_ADDRESS>wrote:

Merged #34 https://github.com/ppat/storm-rabbitmq/pull/34.
—
Reply to this email directly or view it on GitHub
https://github.com/ppat/storm-rabbitmq/pull/34#event-458027058.


WSL2 Improvements

Correct me if I'm wrong, but doesn't WSL2's better kernel integration enable the use of systemd? WSL2 can already be used via the Windows Insider program.

 At this point you could basically make a separate WSL2 version though, because it enables so many features.

Thank you for your suggestion.
I'm also interested in a WSL optimized distribution.
But now it's difficult because I don't have enough time to commit to it...
I can provide a support script, but I don't think there is much improvement.

I started creating ArchWSL2,includes systemd integration.
It is a prototype version.
https://github.com/yuk7/ArchWSL2

Is it able to run Linux GUI apps?
https://devblogs.microsoft.com/commandline/the-windows-subsystem-for-linux-build-2020-summary/#wsl-gui

@enihcam
Maybe it's yes.
But I don't know at the moment.


unexpected behaviour when clicking on arrow glyphicons

Whenever you want to click on a glyphicon arrow, the picker goes to the next, more fine-grained, view. For instance, while in month view. If you click on either one of glyphicon arrows, it automatically goes to the day view. This does not happen when you click on the button, but not the glyphicon itself.
Tested with Chrome 43.0.2357.130 (64-bit) on OS X.

I encountered the same issue, in which if I click left/right, it will go to a finer view, i.e. from month view to date view.
I thought it was random until @qexpres wrote about it above. This issue only happens if I click the glyphicon.

Yeap, the problem stood still. Any fixes?

Same for me, no matter what browser

@qexpres the problem is with the bootstrap-datetimepicker plugin itself. This issue is quite old (sorry for being unresponsive) so I am going to close it.
I cannot replicate it either. Let me know if it still happening and a snapshot how to replicate it. Thanks.


Problem with the output of Uniform Spanning Tree object

I have a small problem with the output of "Uniform Spanning Tree"
I am using  python 3.7.0

Hi @kammmoun!
Can you specify what you mean by "problem": what did you expect?
Is it because the histogram is shrinked vertically?

Thank you for responding.
I think it is not a graphical problem
To be precise I am running
notebooks/DPPy/notebooks/exotic_dpps.
The sum in  " dict_count_sampled_st" is clearly not equal to 10000.

It may be  just a probleme in networkx (I am using version 2.2)

:+1: Hi @kammmoun OK, I'll check this notebook and let you know as soon as possible.

I was able to reproduce your result.
However I don't know the source of the problem. It might be networkx, but it's not a version problem as I have the latest.
It's weird that the histogram is built manually, I don't know why.
And I don't think it's an issue that the values in the dict sum or not to 10000.

I guess from my part I did as much as I could, sorry… @guilgautier will help you more when he'll be available.
Thanks for your interest and reply anyway!

Hi @kammmoun and @Naereen!
I'm back on track and I've just started looking at this issue
Everything worked out well on my own configuration
The first clue might be the version of the networkx package where the ordering of the weights/nodes is not the same which leads to a permuted incidence matrix, from which the kernel is built.
G-colab Python 3.6, networkx 2.2  |vs| my config Python 3.5, networkx 2.1

Let me dig a bit further

Hi guys,
The message associated to my attempt to fix this issue automatically closed it, sorry for that.
@kammmoun and/or @Naereen could you run the notebook and let me know if its ok now ?
To me this is due to the newest networkx version, since the the edges of undirected graphs are not ordered in the same way as before.
With the following
edges = [(3,0), (1,2), (4,3)]
g.add_edges_from(edges)
g.edges

Before, you always had
EdgeView([(0, 3), (1, 2), (3, 4)])

Now it gives
EdgeView([(3, 0), (3, 4), (1, 2)])

Hi @guilgautier, sorry I won't have time until a few days, I'm really busy…
I think @kammmoun can try to help you there.


support Material-ui 5

Does react-page support material UI version 5 or will it be supported anytime sooner?

@noorsheikh did not test it yet, but i am aware of this. I hope that we can do it in a way, that both 4 and 5 should work (as it is a peer dependency.
i am also thinking about abtracting the editor-ui-layer a bit more, that way we could provide a mui-4-compatibility layer and open up the possibility to bring your own ui

As a Tailwind user, a bring your own UI would be a great addition.

Hi team, I would also like react page to support mui5, what can be done to help start this project ?

Hi team, I would also like react page to support mui5, what can be done to help start this project ?

you can clone it and try make it work with mui5!
you can then push a branch and open a PR (work in progres)

Yes, mui5 support is very lacking. I even tried to install both versions and it even works, but there is a style conflict between mui4 and mui5 and the use of the library becomes impossible. :(

Yes, mui5 support is very lacking. I even tried to install both versions and it even works, but there is a style conflict between mui4 and mui5 and the use of the library becomes impossible. :(

this is a problem of material-ui, which is surprising, because we use a custom prefix for the class-generator

Anyway, I really liked your library, I will try to transfer it to mui5 and if it works out, I will open a PR. Thank you.

MUI switch from @material-ui/<package>/<component> to @mui/metarial/<component> in v5. It's not just bumping the existing dependency from v4 to v5. This might require some heavy rewriting if react-page is to use both.
In the meanwhile, we are stuck with using v4. If you wish to use v5 and react-page, you'll have to install both v4 & v5 sadly.

ideally, material ui can be used as a direct dependency, not as a peer dependency and can be used alongside other versions of material-ui.
there were some mixed results in the past with that approach, but we should give it a try once more.
the underlying ui-library should be an implementation detail and users should not worry about it.
I really don't like the way it is now with it beeing a peer dependency, but that was a trade off to fix bugs that could occure with having multiple versions of material ui

i just released a first beta, which comes with material-ui version 5.
it may contains bugs and gitches and things that could be improved, so please give feedback!
you can see the result here https://react-page.github.io/beta
you can install it with the @beta tag

This is great, I'll give it a few test runs and give feedback. :)


How to use IDGL on my own dataset?

Thank you for your great work! But I have some problems when I use the code on my own dataset. Here are some details:

My task is graph classification;
The size of my input features is 7000×72×200: 7000 samples and each sample has 72 nodes and the input dimension of each node is 100;
The dataset can be divided into 3 classes.
When I set my own config, and it reported "correct = preds.eq(labels).double()
RuntimeError : The size of tensor a (3) must match the size of tensor b (6240) at non-singleton dimension 1". Also I noticed that the output of the GNN is 72×3 for each sample, but it should be 3. Where should I change the code for my dataset? Thank you!

@Riofd Thanks for your interest to our work! As for graph classification use case, I would recommend you to refer to data preparation script and graph classification model for examples.

Hi, I wanna ask two questions:  1. How I use my own dateset(A graph, and each node is a time series data)  2. Where is the adj matrix hold, I did not find it. THANKS FOR YOUR HELP!

Hi, I wanna ask two questions: 1. How I use my own dateset(A graph, and each node is a time series data) 2. Where is the adj matrix hold, I did not find it. THANKS FOR YOUR HELP!

@jiangxinke 1) the model was not designed for handling time series data, even though the idea of the model might be extended to time series data. 2) please check out this code on how to access adj matrix.

@Riofd Thanks for your interest to our work! As for graph classification use case, you might want to refer to the text graph classification example in this repo. The main difference is in our case, we prepared a kNN graph for each text example while in your case the initial graph of each example is given.

Hi! I am trying to understand the example to use it for my own dataset, but the 20news data is missing from the repository. Can I download it from somewhere else?

@Riofd Thanks for your interest to our work! As for graph classification use case, you might want to refer to the text graph classification example in this repo. The main difference is in our case, we prepared a kNN graph for each text example while in your case the initial graph of each example is given.

Hi! I am trying to understand the example to use it for my own dataset, but the 20news data is missing from the repository. Can I download it from somewhere else?

Hi @simona-juv , sorry for the late reply! You can download the 20news data at 20news-bydate.tar.gz.


Widgets work in some scenarios but not others

Hi,
My system:
Version: 1.65.2 (Universal)
Commit: c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
Date: 2022-03-10T14:33:49.188Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 21.1.0

 ➜  ~ python --version
Python 3.10.0
 ➜  ~ jupyter --version
Selected Jupyter core packages...
IPython          : 8.0.1
ipykernel        : 6.9.0
ipywidgets       : 7.7.0
jupyter_client   : 7.1.2
jupyter_core     : 4.9.1
jupyter_server   : not installed
jupyterlab       : not installed
nbclient         : 0.5.13
nbconvert        : 6.4.4
nbformat         : 5.2.0
notebook         : 6.4.10
qtconsole        : 5.2.2
traitlets        : 5.1.1

Similar issues:

https://github.com/microsoft/vscode-jupyter/issues/3767
https://github.com/jupyter-widgets/ipywidgets/issues/2512

Code used:
from IPython.display import display
from ipywidgets import widgets, interactive
button = widgets.Button(description="Run!")
def on_button_clicked(b):
    print('Clicked')
display(button)
button.on_click(on_button_clicked)

In Jupyter Web interface this works perfectly. In VScode it does nothing.
However, the example give here https://github.com/microsoft/vscode-jupyter/issues/3767#issuecomment-625555606 works just fine. So it seems that they only work with decorators?
I tested this using:
from IPython.display import display
from ipywidgets import widgets, interactive, interact
button = widgets.Button(description="Run!")
@interact(b=[0,1])
def on_button_clicked(b):
    if b ==1:
        print('Clicked')

This seemed to work fine. A bit of a hacky workaround, particularly as decorators don't seem to have a method for buttons...

Thanks for filing this issue, I can replicate this at my end, Will try to dig into the issue and see whats causing this.

Took a while to figure out the issue, its a bug at our end.
Difficult to explain but we'll need to:

Track the comm id associated with the widget displayed
When messages of type stream come back and found to be associated with the previous commid, we'll need to append these output to the corresponding cell output.


Fable version 3.2.*

/home/ubuntu/saturn-template/client/.fable/Fable.SvelteStore.1.0.0-beta-007/Fable.SvelteStore.fs(1,1): error EXCEPTION: Plugin SveltePlugins.DispatcherAttribute expects v3.1.15 but currently running Fable v3.2.12
Is this restricted to a specific version ? Available in 3.1.* but missing in 3.2.*

Hmm, this looks like a bug in the version resolution by Fable. It should be ok, because 3.2.* is higher than 3.1 but not a new major release. I will push a new
BTW, are you using Fable.SvelteStore just for testing or have you written an app with it? I'm asking because I haven't got much feedback of actual users of the library so I'm curious 😸

I'm playing with it for now. Will keep you posted.
On a side note. I used your Feliz.Engine in https://github.com/plotly/Dash.NET/blob/dev/src/Dash.NET/Dash.NET.fsproj
Thanks for all your amazing work.

That's great, thanks for letting me know! Credit goes to @Zaid-Ajaj for designing the great Feliz API for React. I'm happy that the Feliz.Engine abstraction is useful to you :)
BTW, can you please try to see if the plugin error is fixed with fable 3.2.14? You just need to run dotnet tool update before compilation.

Yep. Works alright. Thanks.
I havent got around to playing with Fable.Lit yet. But will do.
I did play around with Feliz.Snabbdom. I was not able to use it because of some open issues with snabbdom.
I fanatically agree with your blog post about everything being observables. I have been using some of those ideas with Flutter mobile apps.
Will get back if I get stuck again / share if I manage to put together something useful.

Thanks for confirming @kaashyapan!
Someone agrees with something I've written about architecture! That's a first ;) If you're interested in observables, you can also check Sutil which is an F#-only render library for Fable that's inspired by Svelte and uses the stores from this repo to manage the UI.


Oops. An unexpected error occurred when search online

Comictagger run from Windows 11 always gives me an unexpected error when starting to search online (click on "Search Online", "Literal Search", "Auto-Identify" or "Auto-Tag"). Because i can't find a solution i open an issue here. See the picture for the error. After i click [OK] comictagger is closed. The version of comictagger doesn't seem to matter, i have this issue from 1.4.8 and up.

Please go to settings and click clear cache

Thanks, that does the trick!
From: Timmy Welch @.>
Sent: zaterdag 6 mei 2023 17:31
To: comictagger/comictagger @.>
Cc: Bartype @.>; Author @.>
Subject: Re: [comictagger/comictagger] Oops. An unexpected error occurred when search online (Issue #462)
Please go to settings and click clear cache
—
Reply to this email directly, view it on GitHub https://github.com/comictagger/comictagger/issues/462#issuecomment-1537165592 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AVJNT5JPBE3WIOYRBHFJE3DXEZVCPANCNFSM6AAAAAAXYAAS4A .
You are receiving this because you authored the thread.Message ID: @.***>


[Python Schema] Fix redefined Record or Enum in Python schema

Fixes #11533
Motivation
Refer to issue #11533 , currently, if users redefined the same Record or Enum in Record, the schema info isn't reused the defined name, this does not match the Avro schema info format.
Modifications
Add a new method schema_info(self, defined_names) in Record, Array, Map, and Enum, all defined names will be added in the parameter defined_names when users use a defined Record, or Enum, the schema info will use the name of the defined Record or Enum as the type.
Verifying this change
Add test to verify using same Record or Enum in a complex Record.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
For committer
For this PR, do we need to update docs?

If yes,

if you update docs in this PR, label this PR with the doc label.

if you plan to update docs later, label this PR with the doc-required label.

if you need help on updating docs, create a follow-up issue with the doc-required label.

If no, label this PR with the no-need-doc label and explain why.

Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

Could you rebase to master to make sure CI passed?

Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

I added the no-need-doc label. It's just a bug fix.

@BewareMyPower thanks!


Making radio streaming finally work

After about 18 months of having these scripts and trying to make it work, it's time for it to work.
Still want to play a bit more about it, improve scripts that are now too complex, improve on UI (add radio station icons/pictures), add stop/volume up and down, and of course, add more radio stations on the list too...
But for now, this first commit in 2020 makes this work.
And cleanup of duplicate input fields and scripts.

Radio streaming to Chromecast devices is now working perfectly.
There are 2 ways I can use it now:

simple UI with selection of radio station, speaker and volume - wand then just execute script
overcomplicated but better looking Picture entity that has some switches, mute/vol+/vol-,... Still have to fix radio station images as they are not loading the way they should.


nummeric field sign

looks weird, don't know how to fix

I've tried all sorts of settings but haven't been able to get it right - any suggestions?


added transcriptome bams for riboseq

added transcriptome bams for riboseq - required for riboWaltz module

thanks for your feedback @pinin4fjords ! I've made the requested changes. Could you please take another look and see if everything is now in order?

done! next time will know what to change from the first go:). thanks!


InvalidTokenError: Failed to fetch authorization server metadata

Please do not report security vulnerabilities here. The Responsible Disclosure Program details the procedure for disclosing security issues.
Thank you in advance for helping us to improve this library! Please read through the template below and answer all relevant questions. Your additional work here is greatly appreciated and will help us respond as quickly as possible. For general support or usage questions, use the Auth0 Community or Auth0 Support. Finally, to avoid duplicates, please search existing Issues before submitting one here.
By submitting an Issue to this repository, you agree to the terms within the Auth0 Code of Conduct.
Description
When I click button in the external page I'm expecting to get the response message from the API server.

Environment
Windows 10
Server packages:
"dependencies": {
    "@types/cors": "^2.8.12",
    "cors": "^2.8.5",
    "express": "^4.17.2",
    "express-oauth2-jwt-bearer": "^1.1.0",
    "npm-run-all": "^4.1.5"
  }

Client packages
  "dependencies": {
    "@auth0/auth0-spa-js": "^1.19.4",
    "axios": "^0.25.0",
    "core-js": "^3.6.5",
    "vue": "^2.6.11",
    "vue-router": "^3.2.0",
    "vuetify": "^2.4.0"
  }

Hi @rashidmya - thanks for raising this

Failed to fetch authorization server metadata

Your error tells me that the url for your issuer is incorrect or your issuer doesn't provide authorization server metadata
Could you let me know what value you are providing for your issuerBaseURL?

the value of my issuerBaseURL is my client url: http://localhost:8000

Hi @rashidmya - Your issuer is your identity provider (eg your-tenant.auth0.com)

Hi @rashidmya - Your issuer is your identity provider (eg your-tenant.auth0.com)

Mate, Im having the same issue in React
I tried with a combination of :
"audience": "https://MYAUDIENCE.au.auth0.com/api/v2/",
"issuerBaseURL": "https://TENANT.us.auth0.com",

The above did not work, I received:
{     "message": "no applicable key found in the JSON Web Key Set" }
So I tried the below:
"audience": "https://MYAUDIENCE.au.auth0.com/api/v2/",

"issuer": "https://TENANT.us.auth0.com",
"jwksUri": "https://TENANT.us.auth0.com/.well-known/jwks.json"
But I get:
{     "message": "Failed to fetch authorization server metadata" }
I'm not new to Auth0, have been successfully running production systems. But I created something from scratch for my personal project and its not working

@meyvan - if you can share your value for TENANT I might be able to investigate.
@rashidmya - let me know if updating issuerBaseURL to your issuer worked for you and I'll close this issue

@adamjmcgrath my tenant is dev-qnsy8uhz . please let me know if I'm doing something wrong

"issuerBaseURL": "https://TENANT.us.auth0.com",

@meyvan I'm able to fetch your authz server metadata using https://dev-qnsy8uhz.us.auth0.com/.well-known/openid-configuration
I noticed that you're mixing au.auth0.com and us.auth0.com in places, could you perhaps be using the au one as your issuer by mistake? If not, then I can only assume your server is not connected to the internet?

@adamjmcgrath I ended up using https://github.com/auth0/node-jwks-rsa , it works for my scenario. Thank you


Is there a separate Helm chart for ModelMesh?

It seems like ModelMesh can be deployed in combination with KServe from the KServe Helm chart, but is there also a way to deploy only ModelMesh?
We're interested in this because we plan to only use ModelMesh's functionality and not KServe.

@njhill I don't want to use the install script because it doesn't support release management. All of the other applications we deploy in our cluster do have Helm charts available, which allow us to easily manage them with Terraform.
I've decided to implement my own Helm chart based on the resources that are deployed by the script.

I've decided to implement my own Helm chart based on the resources that are deployed by the script.

@OvervCW great, is that something which you could contribute back to the project?

@njhill Possibly, yes. It'll depend on our final implementation.

@OvervCW -- Assuming your helm chart conversion went well, would you like to open a PR and contribute them here?

@ckadner In the end we decided not to use ModelMesh.


Bump version for next release

Bump version for next release

Codecov Report

Merging #1202 (0e9bd88) into develop (3dd7c23) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #1202   +/-   ##
==========================================
  Coverage      31.96%   31.96%
  Complexity       453      453
==========================================
  Files             94       94
  Lines           4699     4699
  Branches         800      800
==========================================
  Hits            1502     1502
  Misses          3053     3053
  Partials         144      144

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 3dd7c23...0e9bd88. Read the comment docs.


confusing UX with connector lines on layers other than the objects they connect

sample document.
You should see three layers.
Things: has A and B rectangles.
Connections 1: has two lines C and C2
Connections 2: has one line D
If you turn off the layer for either connection layer you will see the base things and the connections you specifically want to see. If you turn off both connections layers you will only see the base objects. If you turn off the base layer and connections 2, you ONLY see C2 (should see C). Similarly if you turn off things and connections 1.. nothing.
So in one view (things + connections 1) the arrow labeled D is clearly part of connections 2 (because it's gone). But if you invert the layers to view ONLY connections 2, it disappears.
Looking at arrows C and C2 you can see a better example of where I started in the real document I'm working on... a seemingly random subset of the things on the connections 1 layer are AWOL if that's the only layer that is visible... but they re-appear if you add other layers that they aren't on, as evidenced by them disappearing if you turn off the layer you think they should be on.
In boiling this down to a simple test case above, it's become obvious these awol lines are nominally connected to the objects they connect and are going away when either of the things they are attached to go away. The other lines that don't disappear aren't connected to these objects. It's a very confusing user experience.

The issue reproduced in draw.io online (Chrome Version 74.0.3729.131 (Official Build) (64-bit)

Still an issue?

Hi,
the issue is still present in the 13.10.2 as shown in the video above (in Chrome Incognito Version 87.0.4280.66 (Official Build) (64-bit) and in FF

I am not quite sure if my bug is the same as this one or different.
My detected bug is easy to explain:

Layer 2 with a big rectangle is locked.
On layer 1 (non locked) I like to draw just a line inside the visible rectangle from layer 2.
I cannot draw the line as I want, because the line always tries to connect to layer 2 rectangle.
An inconvienient workaround is to hide layer 2 during editing layer 1.
How can I really disable the connection point "catching" on layer 2?

I just detected, that this is also occuring on the same layer.
Drawing a line freely inside another object (e.g. rectangle) is not possible,
because the line tries always to connect to the object.
But both effects (on same or different layers can be avoided, when all related objects are locked (Ctrl+L)

Selecting the object and unticking "Connectable" should work:

Thank you very much.  My PEBCAK!
Is there a use case where elements on locked layer need to be connected from other layers?


HTML entities & tags

ticks, “'”, → apostrophe entity, '
time <span>s → time <time>s (though missing datatime attribute,
this is still more semantic)
lang attribute for HTML
thesis title uses & + missing period.
<abbr> tag for OTDTP profession to add legitimacy

& inside the thesis title is questionable, everything else makes a nice improvement though.

Up to you 😅
Sent from my Android device with K-9 Mail


Fix issue 68 (incorrect struct ID for single aliens)

Symptom: hpcprof generates the same flat ID (or struct ID) for the same inlined macro (single alien) from different contexts, which confuses the viewer when creating the flat view.
Root cause: Before writing experiment.xml, hpcprof removes redundant procedures by ensuring the tuple <Load_module, Filename, Procedure, line_number> is unique. However, for inline macros, we shouldn't remove redundancy. Or perhaps we should add <Load_module, Filename, Procedure, line_number, "parent_id"> to be unique.
Solution: add "parent_id" in the tuple to make sure single alien is unique based on its context.

@Jokeren : Can you check with QuickSilver code if this solution works?

Quicksilver works in part. The code fix for aliens prevents the unholy blending of performance information from different contexts in the bottom up view. However, it somehow interferes with the GPU CCT reconstruction.
@Jokeren @laksono and I need to discuss this. Below is what I see from the top-down view now. Notice that once we get onto the GPU, we see a flat profile:

Keren and I reviewed it together and we will merge with master and close this branch.


Placeholder for CurrentSeasons Days remaining.

Describe the feature
Would you be able to add something like:
%advancedseasons_season_daysremaining%
To accompany this, it would be great to have:
%advancedseasons_season_nextseason%
So that things like "XXX days until Summer!!" can be produced in Scoreboards, etc?
Thank for all your amazing hardwork!!

To accompany this, it would be great to have:
%advancedseasons_season_nextseason%

Done!


refactor: Unify logic to enumerate and watch AppMaps

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

v0.24.0
GitHub release

Your semantic-release bot :package::rocket:


Add test for a conceptual model creation via API call

This test approach will allow us to test complex models without having to go through the graphical user interface (GUI) for their creation, consequently avoiding the usage of drag and drop to create such complex models.
This approach will also speed up the tests since API calls happen much faster than interaction via GUI.
For more details, read each commit message (titles and descriptions).
See a video of the new test below.
https://user-images.githubusercontent.com/2768415/144533531-06b57f3b-bd98-4a3d-8080-070a76c68bc3.mp4

@miltonbsn, what do you think?


Update README.md

Change Summary
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[x] Documentation content changes
[ ] Other (please describe):

Related Issue(s)
N/A
Component(s) name
arista.avd.dhcp_provisioner
Proposed changes
Update documentation
How to test

Checklist:

[x] My code has been rebased from devel before I start
[ ] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have updated molecule CI testing accordingly
[x] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

I think it is more an issue as the role was designed to support a DHCP server configured in an inventory group outside of the fabric itself.
A PR is in flight to fix the behavior you reported: c5dd18bd.
Could you test and confirm this commit (c5dd18bd / https://github.com/titom73/ansible-avd/tree/issues/dhcp-provisioner-registration-issue ) and be sure you configure fabric_group to target your fabric inventory ?
    TOOLS:
      hosts:
        DHCP_SERVER:
          ztp_network_summary: <IP_ADDRESS>/24
          ztp_pool_start: <IP_ADDRESS>
          ztp_pool_end: <IP_ADDRESS>
          ztp_lease_time: 300
          fabric_group: 'DC1'
          ztp_mac_source: system
          ztp_mode: offline

closing, fixed as per PR #708


[main] Upgrade to latest dependencies

After eventing 1.12 released -creydr
/cc knative/client-wg-leads
/assign knative/client-wg-leads
Produced by: knative-extensions/knobots/actions/update-deps

/approve
/lgtm


Add support for more recurrence rule fields

Adds support for second, minute, hour, day_of_year, and week_start fields of iCalendar recurrence rules.
Still missing week_of_year support, but unless I'm mistaken, IceCube doesn't support that (I opened seejohnrun/ice_cube#288 to make sure)
@jnraine

Looks good @skoushan. Thanks for the contribution!


fix "Themes" link in README.md

@ericmigi @tulir @Madelena This seems like a pretty simple/useful PR to click the merge button on, yet has been sitting around since December 2022.. any chance of getting it merged soon?

Sorry for the delay, I had my notifications disabled for this repo!


Mupx Deploy Verifying deployment: FAILED

I've a project that was created with meteor 1.3 but now upgraded it and well working in local but when I deploy to AWS server using mupx the error occuring.
I run mupx deploy
    npm WARN deprecated semver behavior.
    npm WARN package.json<EMAIL_ADDRESS>No description
    npm WARN package.json<EMAIL_ADDRESS>No repository field.
    npm WARN package.json<EMAIL_ADDRESS>No README data
    npm WARN cannot run in wd<EMAIL_ADDRESS>node npm-rebuild.js (wd=/bundle/bundle/programs/server)
    => Starting meteor app on port:80

    assert.js:93
      throw new assert.AssertionError({
            ^
    AssertionError: "undefined" === "function"
        at wrapPathFunction (/bundle/bundle/programs/server/mini-files.js:77:10)
        at Object.<anonymous> (/bundle/bundle/programs/server/mini-files.js:108:24)
        at Module._compile (module.js:456:26)
        at Object.Module._extensions..js (module.js:474:10)
        at Module.load (module.js:356:32)
        at Function.Module._load (module.js:312:12)
        at Module.require (module.js:364:17)
        at require (module.js:380:17)
        at Object.<anonymous> (/bundle/bundle/programs/server/boot.js:9:13)
        at Module._compile (module.js:456:26)
    -----------------------------------STDOUT-----------------------------------

    To see more logs type 'mup logs --tail=50'

    ----------------------------------------------------------------------------

`
My local node js using 4.4.7 and AWS ubuntu 14 node js also 4.4.7 by upgrading it from 0.10.x to 4.4.7. But still getting that error.
Here is my server node js version check
ubuntu@ip-172-31-1-138:/opt/dev_popup$ meteor node -v
v4.4.7
ubuntu@ip-172-31-1-138:/opt/dev_popup$ node -v
v4.4.7

+1

I solved it by reading 1091 issues.
@humbertocruz said to change mupx's node modules file by manually from meteorhacks/meteord:base to abernix/meteord:base in three places on the file.
Since I'm using windows by running vagrant, I've to change the file from the following location.
C:\Users{UserName}\AppData\Roaming\npm\node_modules\mupx\templates\linux\start.sh
It solved the error. Hope you get it.

Did the same when it happened, rewrote mupx scripts to use that docker image.
Can confirm that it works brilliantly.

Thank you guys!! Can confirm that I managed to setup/deploy Meteor <IP_ADDRESS> on Digital Ocean

This shouldn't be closed should it? This is still a problem with mupx right?

Confirmed working for me

Navigate to "C:\Users{UserName}\AppData\Roaming\npm\node_modules\mupx\templates\linux\start.sh"
Open with text editor or IDE
Search document for for "meteorhacks" and replace with "abernix";

Worked for me thanks


Bug: If the CurrentUserLogin is empty when loading, the feed page opens, not the login page

Fixed by cbb00fb commit

Fixed by cbb00fb commit


big csv

I'm having 1 million records of CSV file, I need to apply some transform on each record, how can I achieve this with read and write stream

Hi,
Take a look at this:
https://github.com/Keyang/node-csvtojson#synchronouse-transformer
some example:
var Converter=require("csvtojson").Converter;
var csvConverter=new Converter({constructResult:false}); //important to
turn off result construction

csvConverter.transform=function(json,row,index){
    json["rowIndex"]=index;
    /* some other examples:
    delete json["myfield"]; //remove a field
    json["dateOfBirth"]=new Date(json["dateOfBirth"]); // convert a field
type
    */
};

var rs=require("fs").createReadStream("./millionsRows.csv");
rs.pipe(csvConverter)
//.pipe(downstream)

~Keyang
On 18 February 2016 at 17:49, suresh varman<EMAIL_ADDRESS>wrote:

I'm having 1 million records of CSV file, I need to apply some transform
on each record, how can I achieve this with read and write stream
—
Reply to this email directly or view it on GitHub
https://github.com/Keyang/node-csvtojson/issues/79.

thanks for the quick response, previously the transform was not working, so we were using the "end_parsed" we faced a problem while processing the csv file of 125 MB its because we were using the version 0.4.5 now the problem is been resolved by upgrading to newer version. i am closing this thanks again.


CVE-2007-4559 Patch

Patching CVE-2007-4559
Hi, we are security researchers from the Advanced Research Center at Trellix. We have began a campaign to patch a widespread bug named CVE-2007-4559. CVE-2007-4559 is a 15 year old bug in the Python tarfile package. By using extract() or extractall() on a tarfile object without sanitizing input, a maliciously crafted .tar file could perform a directory path traversal attack. We found at least one unsantized extractall() in your codebase and are providing a patch for you via pull request. The patch essentially checks to see if all tarfile members will be extracted safely and throws an exception otherwise. We encourage you to use this patch or your own solution to secure against CVE-2007-4559. Further technical information about the vulnerability can be found in this blog.
If you have further questions you may contact us through this projects lead researcher Kasimir Schulz.

Hi @TrellixVulnTeam, thanks for detecting the security vulnerability and contributing the patch! I moved your commit to the upstream tvm project and listed you as the original author, please take a look here: https://github.com/apache/tvm/pull/13751. (The relax repo is periodically synced with the upstream tvm project.)


Spring Boot not serving html file

Spring Boot is not serving HTML files. The literal string "index" gets returned when requesting a route instead of the file itself. Not quite sure why this is the case. I attempted to solve it with no luck.

It is not a pretty fix but it works. I believe because Spring Boot didn't create this solution, there are some missing dependencies or frameworks which do not integrate well with how we planned our GUI. Most likely, it is my understanding of Spring Boot that is not making it work.
The Fix: By mapping the GET requests and calling these requests through Javascript, we can return scripts that execute navigation. Doing it this way allows us to obtain the user inputs, verify and execute, then transition the user to a state.

Implemented thymeleaf to serve the files up in a dynamic way. This allows for the program in inject backend variables into the HTML to then be served.


feat: Add docker manage page

Homarr

Thank you for your contribution. Please ensure that your pull request meets the following pull request:

[x] Builds without warnings or errors (pnpm buid, autofix with pnpm format:fix)
[x] Pull request targets dev branch
[x] Commits follow the conventional commits guideline
[x] No shorthand variable names are used (eg. x, y, i or any abbrevation)

One think I'm not sure about is why you used a black background arround the table and all?

This is not entirely ready, I need to think about other stuff

This is not entirely ready, I need to think about other stuff

Maybe we can merge this pull request just as feat: add docker container table?
@manuel-rw how can I deal with the conventional commits stuff? The commits are quite old and it makes no sense to rewrite the history

It's fine as is, I don't think anyone has time to read the commit history of every file x)

Let's merge, I'll add to it later


'vstest' capability on macOS Build Agent

Have you try trouble shooting doc?
Link to trouble shooting doc: https://www.visualstudio.com/en-us/docs/build/troubleshooting
Agent version and platform
Version of your agent? 2.114.0

OS of the machine running the agent? OSX/Windows/Linux: 10.11.6

VSTS type and version
VisualStudio.com or On-Prem TFS? VSO

If On-Prem TFS, which release, 2015 RTM/QU1/QU2?

If VisualStudio.com, what is your account name? http://account.visualstudio.com trilbymultimedia

What's not working?
Please include error messages and screenshots

Agent and Worker's diag log
Logs are located at the _diag under agent root folder, agent log prefix with Agent_, worker log prefix with Worker_. all sensitive information should already be masked out, please double check before paste here.
Issue
Given that .Net Core is cross-platform, is there any ability to support test/build actions for dotnet core projects on a macOS based agent? I can restore, build and run my project on a Mac, but my Agent doesn't support the vstest capability, according to the list of abilities.

@dombarnes

can you run test for your project from command line in Mac?
if so, then it's become a question of how the agent discover the capability on Mac, can you run 'which vstest' to get the location of vstest?

I've just noticed that my current working project is on the preview2 SDK and rc4 is out which has a 'dotnet vstest' command so I can probably add this as a custom capability. I'll test this and report back

On 21 Mar 2017, at 18:10, Ting<EMAIL_ADDRESS>wrote:
@dombarnes
can you run test for your project from command line in Mac?
if so, then it's become a question of how the agent discover the capability on Mac, can you run 'which vstest' to get the location of vstest?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@cltshivash

My macOS Build Agent is also missing the vstest capability. Any instructions how to add this capability?

Only VSTest task is demand "vstest" in it's task.json, and the VSTest task is Windows Only. I don't think we need to populate vstest demand for Linux/Mac


Constructor mismatch

In the sample in README the IdGenerator() constructor takes 3 arguments, while in v3 it actually takes 2 -- generatorId and options. There's no argument for the epoch as shown in the sample. Either docs or the constructor needs to be updated.

You are correct, I missed that one. I will correct the README (and SHFB file).

And done. Thanks for bringing it to my attention!


Adding algolia pageRank attribute to Medplum Client Docs

Coverage remained the same at 94.124% when pulling 4e527b579ca865fcc91dce49a39704b02a26fe4b on rahul-improve-algolia-search into 31cb80749035d325780d4880d2551eb87da341f2 on main.


trt_yolo len(engine) = 7 , ERROR：assert 3 <= len(engine) <= 4 how to solve it ?

How many yolo layers are there in your custom yolo model?

How many yolo layers are there in your custom yolo model?

well, in fact, i do not understand what is the engine mean. And, i am using the model nanodet with your code. I think it would not have any question, but i am wrong. So, the engine means layer? Besides, i have print the len(engine), got the number 7. I want to know how to revise the code? I need your help. Thank you.

well, in fact, i do not understand what is the engine mean.

Please read TensorRT documentation.

Besides, i have print the len(engine), got the number 7.

This means the TensorRT engine has 7 input/output "bindings" in total.

well, in fact, i do not understand what is the engine mean.

Please read TensorRT documentation.

Besides, i have print the len(engine), got the number 7.

This means the TensorRT engine has 7 input/output "bindings" in total.

OK, thank you


fixed javascript syntax errors

At first I was pleased to find this nice library, but when I stared using it, I bacame quite frustrated... please accept this PR, so others won't experience the same troubles as I did :(
All I did in this PR was that I fixed javascript errors so that examples work + I fixed some javascript related warnings (semicolons missing etc.)

Thanks! Sorry for the issues!


Remove version: Jackett.Jackett version 0.21.1915

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.21.1915/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Possible typo error

In the code for lstm.py (https://github.com/beyondguo/label_confusion_learning/blob/master/models/lstm.py) in line 133, there is a possible typo as the variable name is "pred_porbs". however, in line 137, the variable "pred_probs" is used for calculating the loss. The variable "pred_probs" is not passed to the function and might be incorrectly accessed from line 152.

Thanks, you are right. I will fix it later.


Databricks SQL support

Based on recent work for clients, I for one would think support for Databricks SQL would be a huge deal. Seems to be used a little bit of everywhere.
I recently learned Databricks has teamed up with dbt to enable using their SQL templating tech for simplifying writing transformation logic, but Logica would be on a completely different level in my opinion.

Hi, @samuell and @EvgSkv
My team and I also looked into DatabricksSQL support, and we're doing some work right now to target Databricks at least partially. Some of it might be a bit crude, but we'd be happy to raise a PR to see if it could be useful to others.
I'll follow up on this within several weeks.

Hi @rvalek ,
that's great to hear!
Adding nuances of Databricks to dialects (under compile folder) should be straightforward and safe.
If deeper changes are needed (e.g. in universe.py or expr_translate.py), I'm happy to work with you to get them done.
Looking forward!

Hi!
We are a little behind on this promise due to some unrelated technical issues, but we are close to resolving them.
Just wanted update here that we are still interested in sharing our work on the Databricks dialect. We're planning to raise the PR in early-mid September.

hi @EvgSkv!
can you suggest in which branch it is best to open a PR for this?

Hi @KantorSerhiy !
Sorry, missed the comment. Sending PR to main is just fine! Thank you!

@KantorSerhiy , btw, I did respond in the PR, it looks great! I left just a couple of requests to comment and it's good to be merged after that.

hey @EvgSkv!
yes, I saw your comments and will definitely deal with them.
but it takes for me 1-2 weeks=(
Thank you!


Feature Request: SDMMC/SDIO support for MSC_SD example?

This is actually a feature request, but i couldn't figure out how to tag it as such.
Many SD BO and connectors support a much faster SDIO compared to the legacy SPI. This would significantly alleviate the speed bottleneck which is at least partially caused by the SPI interface in the example. This might even be relatively simple to implement, as esp32 s2 supports SDIO, and the arduino-esp32 library has provided SD_MMC API in their examples, The API format is relatively similar to the SPI API, the only seemingly significant difference being the amount of pin connections needed.

Sorry, but S2 does not support SDIO/MMC. Only SPI sd card is supported on S2.

My mistake, I was under the impression that SDIO was an option since it was listed in the esp32 s2 programming API guide.


Ship native binaries with libglib

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@isuruf Motivation comes from the gtk3 package: https://github.com/conda-forge/gtk3-feedstock/pull/24
Happy to separate them out into a libglib-bin package if preferred.

@isuruf Motivation comes from the gtk3 package: https://github.com/conda-forge/gtk3-feedstock/pull/24
Happy to separate them out into a libglib-bin package if preferred.

Yes, please. A glib-tools package would be good to have

Yes, please. A glib-tools package would be good to have

@isuruf Can you have a look again? There is a glib-tools packages now that contains the same binaries that e.g. Debian ships as part of the non-dev package.
(now commented in the correct PR)

@isuruf Can you have a look again? There is a glib-tools packages now that contains the same binaries that e.g. Debian ships as part of the non-dev package.
(now commented in the correct PR)


RDP instructions need updating

This quick start guide needs to be updated to reflect the new process/screen shots that users go through to get .rdp files.

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

ID: 2e58109c-c8d3-6693-8e1d-03a89ca26168
Version Independent ID: f20a7fc4-328a-e90d-29db-92a91aa1b2bd
Content: Quickstart - Create a Windows VM in the Azure portal - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/quick-create-portal.md
Service: virtual-machines
GitHub Login: @cynthn
Microsoft Alias: cynthn

@jwilliams45-aug
Thanks for your feedback! We will investigate and update as appropriate.

@jwilliams45-aug
I've assigned this issue to the author who will investigate and update as appropriate.

#reassign:@ju-shim

Thanks for your contribution to our documentation
We are moving to a new user feedback system. As part of that effort, we are creating a content request in our backlog to address this gap and are closing this GitHub issue. The updated content will be published shortly.

Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Allow passing custom query params for collection.create

To allow for conditional publishing of files without attachment.
Needed for linkmap page color map files to start processing.
REDMINE-16119

Coverage increased (+0.0007%) to 99.287% when pulling fac71264b304c4e4fe509b3057434426414e8b26 on DanielKuhn:fix/direct-uploads-for-linkmap-files into fd08e9622672ff747d92dbfe9b221ee14241ec8f on codevise:master.


Remove distinction between general user scripts and DDG user scripts

Task/Issue URL: https://app.asana.com/0/414709148257752/1198979649168229
Tech Design URL:
CC:
Description:
The app used to only load certain user scripts when on non DDG sites, as a performance improvement (https://github.com/duckduckgo/iOS/pull/277), however this no longer works (appears to have been broken Oct 2018: https://github.com/duckduckgo/iOS/pull/373), and I don't think it's possible to make it work without reintroducing the bug it originally introduced (see asana tasks for more details).
Therefore this PR removes the distinction between the two, and uses all the scripts all the time.
Steps to test this PR:

Test each user script on a non DDG site to ensure they still work (esp tracker blocking)
Test behaviour is as before on DDG sites

Copy Testing:

[ ] Use of correct apostrophes in new copy, ie ’ rather than '

Orientation Testing:

[ ] Portrait
[ ] Landscape

Device Testing:

[ ] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[ ] iPad

OS Testing:

[ ] iOS 11
[ ] iOS 12
[ ] iOS 13
[ ] iOS 14

Theme Testing:

[ ] Light theme
[ ] Dark theme

Internal references:
Software Engineering Expectations
Technical Design Template

Adding @SlayterDev as a reviewer to see if he has any thoughts here too. 😄


[Regression] Move the super-call in the PredictorStream-constructor to prevent errors (PR 13303)

My apologies for breaking this; thankfully PR #13303 hasn't reach mozilla-central yet.
It's (obviously) necessary to initialize a PredictorStream-instance fully, since otherwise breakage may occur if there's errors during the actual stream parsing.
To reproduce this issue, try opening the PDF document from issue #13051 locally and observe the following message in the console:
Warning: Invalid stream: "ReferenceError: this hasn't been initialised - super() hasn't been called"

Is it possible to add a test using a PredictorSteam ?

It actually turns out that I cannot reproduce this in any build, but only locally in gulp server mode which suggests that this is some kind of SystemJS/Babel related problem.
Hence this is a) a lot less severe than I initially thought, and b) not really possible/necessary to test.

Note that the intention is to get rid of SystemJS for local development, see PR #12563, however that's unfortunately still blocked by https://bugzilla.mozilla.org/show_bug.cgi?id=1247687


breakneck, excl (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Error during VSTS Release on a Ubuntu Host Agent Pool

I'm facing the following error when I'm trying to run a "Release" in VSTS on a Ubuntu Host as Agent Pool.
2016-05-22T15:55:00.9973700Z ##[section]Starting: Release

2016-05-22T15:55:01.6717420Z ##[section]Starting: Download Artifacts

2016-05-22T15:55:01.6833280Z ReleaseId=9, TeamProjectId=5f756f80-3c15-4bd5-a950-a78836caad33, ReleaseDefinitionName=AzureBot

2016-05-22T15:55:01.7091510Z Release folder: ca09421e5

2016-05-22T15:55:01.7216200Z Environment variables available are below.  Note that these environment variables can be referred to in the task (in the ReleaseDefinition) by replacing "_" with "." e.g. AGENT_NAME environment variable can be referenced using Agent.Name in the ReleaseDefinition:

				[BUILD_TYPE] --> [Git]

				[AGENT_ID] --> [3]

				[RELEASE_SKIPARTIFACTSDOWNLOAD] --> [False]

				[BUILD_BUILDNUMBER] --> [b7bcd17d]

				[SYSTEM_TEAMPROJECTID] --> [5f756f80-3c15-4bd5-a950-a78836caad33]

				[RELEASE_ARTIFACTS_AZURE_DOCUMENTATION_BOT_TYPE] --> [Git]

				[RELEASE_RELEASEID] --> [9]

				[RELEASE_REQUESTEDFOR] --> [Alexandre Vazquez]

				[RELEASE_RELEASEURI] --> [vstfs:///ReleaseManagement/Release/9]

				[SYSTEM_CULTURE] --> [en-US]

				[AGENT_MACHINENAME] --> [recaredocker]

				[SYSTEM_WORKFOLDER] --> [/home/********/agent/_work]

				[RELEASE_ENVIRONMENTNAME] --> [Production]

				[RELEASE_ATTEMPTNUMBER] --> [1]

				[AGENT_NAME] --> [recaredocker]

				[SYSTEM_DEFAULTWORKINGDIRECTORY] --> [/home/********/agent/_work/ca09421e5]

				[SYSTEM_TEAMFOUNDATIONCOLLECTIONURI] --> [https://xandrev.visualstudio.com/]

				[AGENT_ROOTDIRECTORY] --> [/home/********/agent/_work]

				[RELEASE_ENVIRONMENTS_PRODUCTION_STATUS] --> [InProgress]

				[RELEASE_ENVIRONMENTURI] --> [vstfs:///ReleaseManagement/Environment/9]

				[AGENT_HOMEDIRECTORY] --> [/home/********/agent]

				[RELEASE_ARTIFACTS_AZURE_DOCUMENTATION_BOT_BUILDNUMBER] --> [b7bcd17d]

				[BUILD_DEFINITIONNAME] --> [Azure Documentation Bot]

				[AGENT_JOBNAME] --> [Release]

				[AGENT_RELEASEDIRECTORY] --> [/home/********/agent/_work/ca09421e5]

				[RELEASE_RELEASEWEBURL] --> [https://xandrev.visualstudio.com/5f756f80-3c15-4bd5-a950-a78836caad33/_apps/hub/ms.vss-releaseManagement-web.hub-explorer?releaseId=9&_a=release-summary]

				[SYSTEM_TASKDEFINITIONSURI] --> [https://xandrev.visualstudio.com/]

				[SYSTEM_TEAMPROJECT] --> [Azure Documentation Bot]

				[SYSTEM] --> [release]

				[SYSTEM_HOSTTYPE] --> [release]

				[REQUESTEDFORID] --> [c6077b33-e63f-471a-972d-ec1fb9ba83b8]

				[SYSTEM_ARTIFACTSDIRECTORY] --> [/home/********/agent/_work/ca09421e5]

				[RELEASE_DEFINITIONNAME] --> [AzureBot]

				[AGENT_WORKFOLDER] --> [/home/********/agent/_work]

				[AGENT_JOBSTATUS] --> [Succeeded]

				[RELEASE_RELEASENAME] --> [Release-7]

				[RELEASE_RELEASEDESCRIPTION] --> []

				[RELEASE_ARTIFACTS_AZURE_DOCUMENTATION_BOT_DEFINITIONNAME] --> [Azure Documentation Bot]

				[SYSTEM_COLLECTIONID] --> [0b3e5005-d19e-49a4-9a5c-6f1bfbd85cd8]

2016-05-22T15:55:01.7217780Z Downloading artifact

2016-05-22T15:55:02.1047140Z Cleaning artifacts directory: /home/********/agent/_work/ca09421e5

2016-05-22T15:55:02.1219110Z Cleaned artifacts directory: /home/********/agent/_work/ca09421e5

2016-05-22T15:55:02.1345370Z ##[error]Release management does not support download of artifact type {0} in the current version

2016-05-22T15:55:02.1527840Z ##[section]Finishing: Download Artifacts

2016-05-22T15:55:02.1553670Z ##[section]Finishing: Release

The release contains only two steps using Docker Integration plugin but the error happens before it executes any of the Docker steps. If I use a Windows host as an agent, the step works sucessfully and logically crashes when try to execute the docker command.
Am I missing something here? Thx in advance

What agent are you using?  What version?

What are the artifact types that you have included in the Release Definition? In the Core CLR agent, we currently only support downloading Build & Jenkins artifacts. We will be adding support to download source repo artifacts (TfGit/TFVC) and other artifact extension we support (TeamCity/External TFS) very soon.

I'm using TfGit as Artifact, but I tried with Build artifact with the same outcome.
Regarding the versions: vsts-agent-linux-2.99.0-0428.tar.gz
Thx!

Having any artifact type other than Build or Jenkins in the release definition will result in this error. Please try removing TfGit release artifact from release definition & having only Build artifact.
Are you getting the exact same error with having only Build artifact in the release definition?

Alex,
We are adding support for downloading TFGit artifact used in a release definition.
Are you still facing issue with downloading Build artifact?

@alexandrev ?

no response.  reactivate is still having an issue


circleci: don't tag to git or deploy to docker hub

Doing all of this in Circle turned out to be a pain to get working correctly, and would have been even more of a pain to expand to other repos, so I'm backing out those bits.
Eventually, it would be good to have all of those various tasks centralized: once we do that elsewhere, we can remove the deployment bit entirely here. In the meantime, we should keep the existing behavior.

LGTM. Want to also add the notifications hook while you're in here?

https://github.com/codeclimate/codeclimate-ops/blob/master/circle.yml#L25-L27


add noisy qci backend

can you review, @anpaz ? it's not letting me select reviewers.
fwiw, I verified that this works from a jupyter notebook.


Robotium fails to scroll to click on off-screen menu item.

From<EMAIL_ADDRESS>on November 05, 2013 09:44:30
What steps will reproduce the problem? I have an app that uses ActionBarSherlock and has seven menu items.
On some phones, when in landscape mode, the screen only has room to display 6 menu items. The 7th one is off screen and the user should scroll the menu list up to make it visible and click on it.
 What do you see instead? I would expect Robotium to scroll up to make the "My menu label" menu item visible and then click on it.
Instead Robotium clicks on the preceding entry (the last menu item that is visible at the bottom of the screen) What version of the product are you using? On what operating system? This happens on ICS and up as far as I can tell.
On GB devices there is an overflow submenu and Robotium deals with that as expected.
Original issue: http://code.google.com/p/robotium/issues/detail?id=550

From<EMAIL_ADDRESS>on November 06, 2013 20:30:55
Thanks for reporting this.
Status: Accepted
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on November 07, 2013 12:53:40
Out of curiosity I pulled the Robotium source code and traced through it to see what happens.
In the Clicker class, in the clickOnMenuItem(String text, boolean subMenu) method, I modified the following call
	boolean textShown = waiter.waitForText(text, 1, WAIT_TIME, false) != null;

I changed the last parameter's value from false to true, like this
	boolean textShown = waiter.waitForText(text, 1, WAIT_TIME, true) != null;

This allows for scrolling in order to find the text. I haven't tested this change exhaustively and I'm not familiar enough with the Robotium code to be able to say confidently that this doesn't introduce any side effects / new bugs / regressions though.

From<EMAIL_ADDRESS>on November 07, 2013 22:04:51
Thanks.

From<EMAIL_ADDRESS>on January 05, 2014 23:57:42
The only way we've found to fix this is by doing the click directly from Javascript. Please use Robotium 5.0.1. In the new Config class, set:
conf.useJavaScriptToClickWebElements = true
See the javadoc on how to use the Config class.
Status: Verified


doc: samples: philosophers: Clean up README

before: https://docs.zephyrproject.org/latest/samples/philosophers/README.html
after: https://builds.zephyrproject.io/zephyr/pr/72878/docs/samples/philosophers/README.html

Thanks. Sidenote: I'm thinking we should not reformat lines from 80 to 100 chars, otherwise we are going to be changing a lot of lines.

Ya I tend to agree but my thinking here was that the file was already inconsistent, with some sections already going past 80.
In fact, if we were to implement a change, I think I'd like to see us use a no line wrap policy, and rather have do one sentence/one line. This way, diffs are much cleaner when e.g one just adds a few words to a sentence, only that one line is changed as opposed to potentially many as everything in the paragraph gets "pushed".


Feature/1 raylib encapsulation textures

Check Class Texture2D Encapsulation
Don't kill the branch please i make Image class encapsulation on it

I think the circle already got updated in the #29 branch


Add Ctrl + L / Cmd + Shift + G shortcut to focus path bar in FileDialog

This also tweaks EditorFileDialog to use the same shortcut, while making it select the path text after focusing (like in most file managers).
Testing project: test_filedialog_focus_shortcut.zip
@bruvzg Is this the correct shortcut to use on macOS? The "Go To" action is the closest equivalent I can think of in Finder.

Is this the correct shortcut to use on macOS?

I guess it's the closest thing, there's no path bar in the native file dialog, and Go to shortcut works the same way as in Finder.

Works on Windows.
Don't see why it wouldn't work on macOS based on the code, and comment above.
https://github.com/godotengine/godot/assets/105675984/cd3f244f-fd8f-45aa-bb22-ba58a58ce60d

@Calinou Works on mac, but if the focus is inside of the window it also selects the folder that starts with g:
https://github.com/godotengine/godot/assets/60579014/f13a21b7-46c7-4551-aa74-f74b8dbaed29

@Calinou

Is this the correct shortcut to use on macOS?

It is the correct shortcut in Finder, but I'm not sure if a lot of people know it exists. I keep forgetting that shortcut and I've been using macs on and off for over a decade 🙂
I recommend using Cmd+L instead (or at least in addition to Cmd + Shift + G) because that's what all of the web browsers are using, including the native Safari

@Calinou Works on mac, but if the focus is inside of the window it also selects the folder that starts with g:

It's strange this happens, as the code specifically marks the input as handled by calling set_input_as_handled() just after.

I've added Cmd + L on macOS in addition to the existing Cmd + Shift + G there.
Ctrl + L / Cmd + Shift + G can also now be used to focus on the property name in the project settings editor.

I've added Cmd + L on macOS in addition to the existing Cmd + Shift + G there.
Ctrl + L / Cmd + Shift + G can also now be used to focus on the property name in the project settings editor (or the "add input" box in the input action editor).

"Add input box", this?

"Add input box", this?

Yes :slightly_smiling_face: I thought about focusing Filter by event instead, but am not sure how often you'd need to focus on that box in comparison (while quickly focusing on Add New Action can be very useful to add many inputs at once).

I got everything to work except that, it doesn't seem to focus anything.

Everything works in my end, including the "Add New Action"

Everything works on my end, including the "Add New Action"

Ah weird, I rebuilt and it's working on my end too. My bad.

Tested and it works for EditorFileDialog, but not for FileDialog.

Seems fine. As commented in project settings editor, more tabs could use the shortcut.

Done (see bottom of updated PR description for details).

Thanks!


Temporary revert cmake

As long as cmake doesn't generate suitable deb packages (see #174 ), we need to
switch back :(

Did this change pass the CI pipelines? No.
https://github.com/rscada/libmbus/actions/runs/174538963
The code might work (but I am not sure since I have not seen proof).
I would assume that the file specifing how the automatic builds are executed should have been updated as well.

Did you noticed that the whole CI stuff based on cmake?

Den 19 juli 2020 13:35 skrev Stefan Wahren<EMAIL_ADDRESS>Did you noticed that the whole CI stuff based on cmake?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/rscada/libmbus/pull/176#issuecomment-660630263, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AP2YIODKZDCYJTMYDRFLJX3R4LLAXANCNFSM4PBF7KAQ.
I would say that of course should the CI-stuff match the current build system and tests.
So when switching to using configure + make, the commands for the CI stuff should be updated to use configure + make.
Or call a script like build.sh if that is what we want to do from the CI tests.
(I did not look at the your patch so I don't know the exact details).
In this particular case, you might have introduced the CI test when introducing CMake, but there is nothing that says that CI tests are not possible when using configure + make.
What commands did you use to build and test the code? You should replace the cmake stuff with these commands in the YAML file.
/F


Add in grapheme break data testing infrastructure

@swift-ci please test

@rxwei I wasn't able to figure out how to explicitly type the captures there, but it's close. I know you're working in more type plumbing, so this might be a good (real) example for that.

@swift-ci please test


[Forwardport] v2.7: Add options to lasso that improve performance

Original Issue:
https://github.com/rancher/rancher/issues/37350

✅ PASSED
Reproduction Environment
Not required.

Validation Environment

Component
Version / Type

Rancher version
v2.7-4092002b7b921d2862b75d0111f2805200832ee0-head

Installation option
Helm (high availability)

If Helm Chart k8s cluster
v1.24.6+k3s1

Cert Details
Let's Encrypt / traefik

k3s ha external db
Aurora MySQL 5.7.mysql_aurora.2.10.2

Helm version
v2.16.8-rancher2

Downstream cluster type
RKE1 Linode

Downstream K8s version
v1.24.6

Authentication providers enabled
local

Logged in user role
Administrator & Standard user

Browser type
Google Chrome

Browser version
106.0.5249.103 (Official Build) (x86_64)

Validation steps

Enabled metrics with controllers
Provision downstream cluster
Create 300 projects in the local and downstream cluster - using terraform
Create 1,000 workloads in the projects - using deno/typescript https://github.com/brudnak/deno-cattle-workloads
Set CATTLE_RESYNC_DEFAULT rancher env var to 5
Confirmed that controllers are running every 5 minutes
Set CATTLE_SYNC_ONLY_CHANGED_OBJECTS to mgmt
Confirm handlers spikes every 5 minutes are smaller
Set CATTLE_SYNC_ONLY_CHANGED_OBJECTS to mgmt,user
Should be even smaller
Set CATTLE_SYNC_ONLY_CHANGED_OBJECTS to mgmt,user,scaled
Will probably look comparable or smaller

Additional Info
RESULTS
✅ Expected
Handler spikes to decrease
✅ Actual
The handler spikes did decrease


UserGuide: Finish editing Introduction,TOC, QuickStart

Please review.

Codecov Report

Merging #11 (9640bc6) into master (6967885) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 6967885...9640bc6. Read the comment docs.


Issue while training

Hello, thank you for releasing this wonderful code!
I have an issue during training, that I did not manage to work around:

AssertionError: Batch size 32 can not be larger than number of episodes 3
Exception ignored in: <bound method HDF5SequenceSampler.del of <dataloaders.hdf5_loader.HDF5SequenceSampler object at 0x7f1bf5c8fcf8>>
Traceback (most recent call last):
File "/home/daniel/Desktop/minecraft-bc-2020/dataloaders/hdf5_loader.py", line 424, in del
self.close()
File "/home/daniel/Desktop/minecraft-bc-2020/dataloaders/hdf5_loader.py", line 419, in close
self.task_queue.close()
AttributeError: 'HDF5SequenceSampler' object has no attribute 'task_queue'

Apparently, the number of episodes is decreasing, and if it is smaller than the batch size, it breaks :/

Hmm interesting, that should not happen because there should be a task_queue object in the class.
Does this only happen at the very end of training or earlier? I do not have perfect memory of the experiments, but this might be one of those "oh well, it trains the model perfectly, so I can't be bothered to fix this exception" things ^^.

Hello @Miffyli, maybe it is related to this issue from the other repo. Now it makes sense it saying that it only has 2 episodes.

Ah yup, that's the issue! It tries to get more episodes than there is available, so it fails. Downloading full dataset should fix it.

Yep, now its training, thanks :)


Test the project on Linux ARM64

Is your feature request related to a problem?
More and more development (Mac) and deployment (Linux) is being done on ARM64/Aarch64 CPUs.
I'd like to propose adding an additional job configuration to the CircleCI config to build and test the project on (Linux) ARM64.
Describe the solution you'd like.
CircleCI supports ARM64 executors since a while - https://circleci.com/docs/configuration-reference/#arm-execution-environment-linux
Impact
(Please click check boxes AFTER submitting ticket)

[ ] High - User productivity significantly improved
[X] Medium - User productivity partially improved
[ ] Low - User productivity trivially improved

Is your feature request specific to a data set?
No!
Additional context
N/A

@markjens we have no plans to do this in the short to medium term, but we may revisit this in the long term.  We do not have an ARM machine with a graphics card and we have not seen heavy enough user demand to meed this requirement yet.
However I have experimented with this before and may be able to provide a set of third party libraries that will help you build vapor on ARM if you're interested.


Improve the editor audio preview inspector appearance and functionality

Make the timeline indicator thicker and with an indicator triangle, similar to the animation editor timeline.
Add Space bar shortcut to play/pause the audio preview.
Only seek when clicking or dragging with the left mouse button, not other mouse buttons.

Preview

Thanks!

Cherry-picked for 3.4.


Cannot read property 'map' of undefined

Issue Type
Bug
Description
After installing arduino extension.
when I want to initilize it. show me below error
Cannot read property 'map' of undefined
VS Code Info
VS Code version: Code 1.20.1 (f88bbf9137d24d36d968ea6b2911786bfe103002, 2018-02-13T15:34:36.336Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz (8 x 4000)

Memory (System)
7.86GB (3.01GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

cpptools
ms-
0.15.0

vscode-serial
olb
0.0.2

vscode-arduino
vsc
0.2.10

Reproduces only with extensions

Please file extension specific issues on the extension itself. I'll move this over to vscode-arduino.

This issue was moved to Microsoft/vscode-arduino#475