Feat Request : Roles

What about roles for users assigned to Cluster ?
it could be a nice way to handle another permission level on modules ;
Use case example :

user1 assigned to role1 should have access to module1 open for every roles
user2 assigned to role2 should have access to module2 open only for role2
user1 assigned to role1 shouldn't have access to module2 open only for role2

UserGroup is insufficient, a user cannot have different groups on different clusters.

To be honest we are considering building such an extension for Cumulus Core, but with a little different approach.
Right now all cluster's users has access to all cluster's modules and this is how it was designed to be. The roles in Cumulus environment should be manageable by a cluster's admin user and backend admin. As a consequence the current idea we are taking into consideration is managing roles almost like groups in RainLab.User so that we can decide if user can have access to a page or not based on group he/she is attached to.
Still, this is just an idea that needs to be polished very much. Right now we are working on subscriptions and payments because it is necessary for Cumulus to become complete SaaS platform.
Any ideas are very welcomed, thanks for your comments :)

I'm also working on Subscription system; I was considering to integrate : https://octobercms.com/plugin/offline-cashier
Yes that's what I thought, using groups like Rainlab.User but based on clusters.
I tried a permission system with another plugin but as I said on #15 it's not ideal.
Speaking about features, plans have limited usages.
In my case clusters will be able to buy multiple "certifications" for website1, web2 etc. Plan isn't the best way to handle the pricing so if you're creating a subscription system you may consider to not attached it directly to a single plan. And allow more flexibility.
It's a nice project, If you need some help... ;)

Well, contribution and this kind of help is very welcomed, thanks :)
The most recent idea of subscriptions was to have paid subscriptions to plans, so that cluster can have only one plan that it had paid for. It this case there could be a lot of plans with a variety of feature combinations.
Can you describe your case in more detail? Why the idea does not fit your needs?

I agree it's a clean and simple way to use subscription;
Imagine you create a DNS register; Users will buy one or more Domains, and subscriptions will have different deadlines and payment dates.  My use case it's pretty much the same.
It's like having multiple "plans".

If I understand you well, managing user's domains is rather feature to be handled by a module you write rather than a core functionality.
Let's say you have user<EMAIL_ADDRESS>and he is assigned to Smiths Ltd. cluster. The cluster has access to module that manages certifications, or domains or whatsoever. In this case if you want your clients to pay you for access to this module than you can use cumulus subscriptions. For example page with register new domain feature or list registered domains. But when John wants to register new domain in your system then he have to use a different payment that concerns his domain / certificate, not his access to those pages.

Yes I think it's two different use cases.
Your subscription system applied to the plan makes sense for SaaS.
In my case I thinks that I must extend cluster and add a many to many relation with my certification model.
I will use cumulus as a backbone but just without plans. :)

Yes, using Cumulus in such applications makes sense as well. You just have to create one plan (for example full) and assign all of you clusters to this plan. If your clients are registering accounts by their own than you may also configure auto assigning to assign the plan to all newly registered clusters :)


Integrate Stereo Layers

Would it be possible to do a HMD detection, and if its true, render the loading screen in a stereo layer for VR?

Would love this!


Fix issue with creating station from Search

What this PR does / why we need it:
Fixes issue with creating station from Search
Which issue(s) this PR fixes:
Fixes #63

Special notes for PR reviewers:
Does this PR introduce a user-facing change?:
NONE

Oops, sorry I forgot about this. I've pushed these changes (modified to fix the tests) and release 2.1.0 to PyPi


Please, clarify specification List.getRange()

The description of method List.getRange(int start, int end) reads:
( https://api.dartlang.org/stable/1.15.0/dart-core/List/getRange.html )

Returns an Iterable that iterates over the objects in the range start inclusive to end exclusive.
An error occurs if end is before start.
An error occurs if the start and end are not valid ranges at the time of the call to this method.
The returned Iterable behaves like skip(start).take(end - start).
That is, it does not throw exceptions if this changes size.

It is not clear, how the range is determined to be valid.
Let's assume that start and end are indexes:
var list = [1];
list.getRange(-1,0) - invalid, because index can not be negative
list.getRange(0, 1) - valid, because end is exclusive
list.getRange(1,1) - invalid, because start is inclusive and index value 1 is invalid.
In fact list.getRange(1,1) returns an empty list, which means that range [1,1) is considered as valid.
This may be explained with help of skip(start).take(end - start), which will produce an empty list for [1,1). But the range [2,2) should also be valid then. But in fact it is not.
Please, clarify input range verification rules.
Also, please, clarify whether list.getRange(list.length, list.length) returning empty list is valid, intended behavior or not.

A range from start to end is valid for a list if 0 <= start <= end <= list.length.
The range starts at start and has length end - start. An empty range is valid (with end == start aka. length == 0).
So we define this somewhere in general? Otherwise it might be a good idea to document it on List.
So, for list = [1], list.getRange(1, 1) is valid.
Saying that start is "inclusive" is a simplification. It's almost correct, but the real explanation is that it's interval arithmetic and the numbers are not indices of elements, they are indices of positions between elements. The position zero is just before the first elements, the position list.length is just after the last element, and the rang between positions start and end are the elements between those positions. That's just too complex to document, so we say "start is inclusive, end is exclusive" and skimp on the explanation of what it means when start == end (including start == list.length).
The getRange method returns an iterable, not a list, so list.getRange(list.length, list.length) returns an empty Iterable. That's valid and intended. It might be slightly counter-intuitive that getRange(1, 1) and getRange(2, 2) are different when they both define an empty range, but if 2 isn't a valid position in the list, then the latter is likely indicating a logical error in the program, where the former is just an empty sub-range of the list.
The "behaves as skip(start).take(end-start)" only applies if the requirement of the previous statement didn't raise an error - otherwise there is no returned Iterable. I don't remember whether the behavior is exactly like skip+take if the underlying list's length changes - but it's not unlikely that it uses the exact same Iterable implementation.

Uploaded a CL: https://codereview.chromium.org/2632393003


Migrate code to Scala

We are not happy with Jersey nor with the Java approach.
Plan is to migrate code to Scala + Scalatra. Simple framework.

Code has been migrated. New repo NGSI-LD_Wrapper has been created. This repo will be deleted in two days.
Dear watchers and stargazers, please could you move to the new repository
@fisuda @fgalan @nikoskal @juliozinga @namingFail @pietercolpaert

Link to that one for the sake of links ;-) https://github.com/Fiware/NGSI-LD_Wrapper


podman report not working

What happened:
Fedora35
$ syft podman:docker.io/library/postgres:11

1 error occurred:
	* failed to construct source from user input "podman:docker.io/library/postgres:11": could not fetch image "docker.io/library/postgres:11": no host address

$ syft podman:postgres:11

1 error occurred:
	* failed to construct source from user input "podman:postgres:11": could not fetch image "postgres:11": no host address

RHCOS (OCP)
$ syft podman:quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:bb5f32d6a5294d211e7016a3aeefc1a02dd8ce3773747bcead61ab40f3a13366

1 error occurred:
	* failed to construct source from user input "podman:quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:bb5f32d6a5294d211e7016a3aeefc1a02dd8ce3773747bcead61ab40f3a13366": could not fetch image "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:bb5f32d6a5294d211e7016a3aeefc1a02dd8ce3773747bcead61ab40f3a13366": no host address

What you expected to happen:
That syft will generate report
How to reproduce it (as minimally and precisely as possible):
Run the above commands
Anything else we need to know?:
Environment:

Output of syft version:

$ syft version
Application:        syft
Version:            0.41.6
JsonSchemaVersion:  3.1.0
BuildDate:          2022-03-16T15:08:23Z
GitCommit:          ee0a1d172c0c6f0da907694b9b48870b33d2852d
GitDescription:     v0.41.6
Platform:           linux/amd64
GoVersion:          go1.17.7
Compiler:           gc

OS (e.g: cat /etc/os-release or similar):

F35
$ lsb_release -a
LSB Version:	:core-4.1-amd64:core-4.1-noarch
Distributor ID:	Fedora
Description:	Fedora release 35 (Thirty Five)
Release:	35
Codename:	ThirtyFive

RHCOS
$ cat  /etc/os-release
NAME="Red Hat Enterprise Linux CoreOS"
VERSION="49.84.202201042103-0"
ID="rhcos"
ID_LIKE="rhel fedora"
VERSION_ID="4.9"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Red Hat Enterprise Linux CoreOS 49.84.202201042103-0 (Ootpa)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:8::coreos"
HOME_URL="https://www.redhat.com/"
DOCUMENTATION_URL="https://docs.openshift.com/container-platform/4.9/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="OpenShift Container Platform"
REDHAT_BUGZILLA_PRODUCT_VERSION="4.9"
REDHAT_SUPPORT_PRODUCT="OpenShift Container Platform"
REDHAT_SUPPORT_PRODUCT_VERSION="4.9"
OPENSHIFT_VERSION="4.9"
RHEL_VERSION="8.4"
OSTREE_VERSION='49.84.202201042103-0'

Thanks

Thanks for reporting this issue, @pashok2398 -- the error returned by syft could give more information, I apologize for that:

failed to construct source from user input "podman:docker.io/library/postgres:11": could not fetch image "docker.io/library/postgres:11": no host address

I suspect the root cause is syft can't locate Podman's configuration file or environment variables. Do you have Podman installed?

"podman:quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:bb5f32d6a5294d211e7016a3aeefc1a02dd8ce3773747bcead61ab40f3a13366": could not fetch image "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:bb5f32d6a5294d211e7016a3aeefc1a02dd8ce3773747bcead61ab40f3a13366": no host address

Same problem here. I looked this image up and my access was forbidden. Once you have podman running locally, you probably need to set your credentials to access that image, more info here.

Hi @jonasagx
Thanks for looking into it.
Yes i h ave podman installed, i am inside RHCOS
$ cat /etc/redhat-release
Red Hat Enterprise Linux CoreOS release 4.9

$ podman version
Version:      3.2.3
API Version:  3.2.3
Go Version:   go1.15.7
Built:        Thu Jul 29 15:02:43 2021
OS/Arch:      linux/amd64

I just check on my F35 with podman and have the same issue
$ cat /etc/redhat-release
Fedora release 35 (Thirty Five)

$ podman version
Version:      3.4.4
API Version:  3.4.4
Go Version:   go1.16.8
Built:        Wed Dec  8 23:45:07 2021
OS/Arch:      linux/amd64

$ podman images |grep docker.io/library/postgres
docker.io/library/postgres                                  11          a1377d9ce157  2 days ago     292 MB
docker.io/library/postgres                                  latest      07e2ee723e2d  2 months ago   382 MB

 $ podman run -it --rm --entrypoint bash docker.io/library/postgres:11
root@fc0e9ef7762a:/# whoami
root
root@fc0e9ef7762a:/# exit

$ syft podman:docker.io/library/postgres:11
1 error occurred:
	* failed to construct source from user input "podman:docker.io/library/postgres:11": could not fetch image "docker.io/library/postgres:11": no host address

$ syft podman:postgres:11

1 error occurred:
	* failed to construct source from user input "podman:postgres:11": could not fetch image "postgres:11": no host address

$ podman version
Version:      3.4.4
API Version:  3.4.4
Go Version:   go1.16.8
Built:        Wed Dec  8 23:45:07 2021
OS/Arch:      linux/amd64

Regarding the permissions issue, you getting this because its RH private repo i am able to pull and have this image as i am running on OCP and its in my image list.

I've reproduced your error (by running fedora over podman). The problem is Podman is not running as a daemon. You can check its status via: systemctl status podman.socket. Syft requires Podman's daemon to pull images.
If you'd rather use Syft without a daemon service (either podman or docker) you can simply run syft postgres:11 and Syft will use its own OCI client.
An example of how to start Podman's daemon is in our CI system:
https://github.com/anchore/stereoscope/blob/main/.github/workflows/validations.yaml#L111-L122
The steps look like this:
loginctl enable-linger $(whoami)
sleep 5
export XDG_RUNTIME_DIR=/run/user/$UID
mkdir -p $HOME/.config/systemd/user
systemctl --user enable --now podman.socket
sudo systemctl start podman.socket

FYI: you can see syft's debug logs this way:
syft postgres:11 -vv

I am working on a better error message for situations like that.

@pashok2398 could you provide more details of why it is not working with root? Debug logs can provide more information, if you can share them.
Podman and Docker keep copies of their images locally, using one of them will let you scan a local image. If also have these options for local scanning:
syft docker-archive:path/to/yourimage.tar   #use a tarball from disk for archives created from "docker save"
syft oci-archive:path/to/yourimage.tar      #use a tarball from disk for OCI archives (from Skopeo or otherwise)
syft oci-dir:path/to/yourimage              #read directly from a path on disk for OCI layout directories (from Skop

@jonasagx I am not sure but looks like podman with root running without the podman.sock....
ll /run/podman/
total 0

However i think i found a solution...
I am scanning the overlay dir (merged) for each running container....
syft -q --output json dir:/var/lib/containers/storage/overlay/721d1f2fb66ffd8d7939874bf15713ea0007167af48c8d4afdffb4e088fc8513/merged

What do you think? can i miss something using this way of scan?

Hey @pashok2398 I will try to reproduce your issue with root user, how did you install podman?
By docker docs:

The merged directory is the union mount of the lowerdir and upperdir, which comprises the view of the filesystem from within the running container.

Your syft command above is equivalent to: podman run -it my/image:latest syft / where you run syft inside the container. You should capture all that is available to the container. The performance will suffer with slow syscalls to open files and index them.

Hi @pashok2398, we are going to go ahead and close this issue since it has been a few months since we received a reply. But if you are still having trouble, please re-open it or file a new issue with any additional details you have, and we will help you out. Thank you!


l variable in remove method is at global scope

The l variable in the remove method (line #513) is not declared with the var keyword, and is therefore at global scope. This causes problems if the triggering of the remove events causes other calls into the remove method.

Fixed by: c5ddc6b8f862474acefbd5efeb370bb7bc1eb8ab  Thanks for reporting this.


Install -release RPMs directly

This avoids having to dance around with wget and rm -f.
Fixes: 10481
http://tracker.ceph.com/issues/10481
Signed-off-by: Zack Cerza<EMAIL_ADDRESS>

:+1: nice

@ktdreyer good point on the approach :beers:


Autoreload Taking up Lots of CPU

Hi,
When I run django-supervisor with the default settings through the management command, it autospaws runserver and autoreload as well as the processes that I've defined in the supervisord.conf.  This is OK, but I noticed that autoreload takes up around 50-70% of my CPU when enabled.  I observed that after doing './manage.py supervisor shell' and then 'stop autoreload', my CPU dropped down to the level that it was before I started supervisor (around 6% for my idle desktop applications).    I'm wondering other people have this issue, if maybe is specific to my environment, or if it's a known limitation and the recommendation is to expect this overhead in the development environment.  Thanks for your consideration.
Here is my config:

vim  $MY_DJANGO_PROJECT_DIR/supervisord.conf
[program:celeryd]
command={{ PYTHON }} {{ PROJECT_DIR }}/manage.py celeryd  -E -B
[program:celerycam]
command={{ PYTHON }} {{ PROJECT_DIR }}/manage.py celerycam
[program:runserver]
exclude=false
[program:autoreload]
exclude=true

PS: Django-supervisor is a pretty awesome wrapper to an already awesome project.  Thank you for creating it!

I looked at the watchdog code today (looks great!) and think I can help implementing a EventHandler for django-supervisor.


F# intellisense errors even when the code builds just fine

Build compiles and runs fine. However on opening the solution, F# intellisense errors come up into the error list. Intellisense is lost in those areas, and I get red underlining. Many of the errors only show up when a file is open. So far in trying to look around at the code that causes this, it's been consistently around Type Provider provided types.
Repro steps
Have a type provider provided type and a locally defined record type with some similar property names/types

Step A

Step B

Expected behavior
Intellisense works, no red underlines for things that aren't errors and compile just fine
things that aren't errors at least would stop showing up in the error list when the file was closed
opening new files doesn't add to the error list, when there aren't actually any errors.
Actual behavior
Intellisense doesn't work, red underlines, errors accumulate while working with the solution that never go away
Warning	AD0001	Analyzer 'Microsoft.VisualStudio.FSharp.Editor.SimplifyNameDiagnosticAnalyzer' threw an exception of type 'System.Threading.Tasks.TaskCanceledException' with message 'A task was canceled.'.
Known workarounds
None, slight improvement from closing the solution and opening again, since the errors accumulate from opening more .fs files
Related information
Provide any related information

Windows 10
Branch? What branch? branch of what?
.NET 4.5.2
VS 2017 (Vs 2015 works fine)

This is fixed in master.

@vasily-kirichenko so yesterday, I tried to install some version from here over top of the VS2017 one, and it broke far worse, is there some additional information or specific downloads that do or don't work?
It appears the fake/intellisense errors eventually do go away

@ImaginaryDevelopment  What you tried to install yesterday was OK, you only missed the step of dealing with unsigned dlls. Hopefully this one won't be needed once we have official signed builds.

To install the Visual F# IDE Tools from Jenkins, first run sn.exe (Strong Name Tool) from the Visual Studio developer command prompt to turn off strong name checking on your machine:
sn -Vr *,*

what's the eta on that?

Warning AD0001 Analyzer 'Microsoft.VisualStudio.FSharp.Editor.SimplifyNameDiagnosticAnalyzer' threw an exception of type 'System.Threading.Tasks.TaskCanceledException' with message 'A task was canceled.'.

Is this also fixed? It pops up constantly even on the simplest projects.

I've seen that message as well, a bit of time after opening a solution while waiting for colorization.
It did so on FSharpDocumentDiagnosticAnalyzer.

Duplicate of https://github.com/Microsoft/visualfsharp/issues/2100. It marked to be fixed in RTM, so it's fixed you guys just cannot see it.

I believe there are at least three issues here. One of them is described only in the title itself, and is a duplicate of #2548. Then there is #2100. But I believe the rest of the report deals with "frenetic IntelliSense activity", which can pretty much cripple work with F# in VS 2017, but is fixed in the latest VSIX. I feel it important this be known. Thanks goes to Vasily Kirichenko for helping me with the installation (see #2548).

There is a related issue in VS feedback here: https://developercommunity.visualstudio.com/content/problem/24309/vs2017-fsharp-intellisense-across-projects.html
I'm looking at this today/tomorrow (busy around here for me right now...).  It's difficult to tease out the specific problems, but I think they should be much better once we get our nightly builds up.

getting a ton of errors that aren't errors continues to plague after switching to nightly builds even. It makes it REALLY tough to find real errors when they exist.

@ImaginaryDevelopment Could you post a screenshot?

I used to have this problem, but it was fixed more than a week ago. I also configured nightly builds, and todays version of Visual F# Tools is <IP_ADDRESS>32701. @ImaginaryDevelopment , which version do you have?
List of versions, with dates: https://dotnet.myget.org/feed/fsharp/package/vsix/VisualFSharp

Indeed the build issues are laid to rest with <IP_ADDRESS>32701 and a big thank you for the nightly builds! But for those who don't want to burden their C# comrades with extra install requirements for F# are waiting for the IDE update :) Any ETA for that?

@nethatix Still TBD.  We haven't announced update times yet.  That said, the update model for VS 2017 is different than before.  We'll have an official update before the traditional "Update 1" timeframe.

FSharp tools version: <IP_ADDRESS>32701

All the fake errors appear to be related to using a type provider. Accessing fields of erased types.
packages\FSharp.Data.TypeProviders.<IP_ADDRESS>\lib\net40\FSharp.Data.TypeProviders.dll

The latest is VisualStudio/15.0.0+26228.10 which I noticed myself in "Updates" only this morning before my previous post, but I have no clue whether it can help with your case, @ImaginaryDevelopment.

Is there an error squiggle at the left upper corner of the file? A very short one, at the first open statement? If it's there, what it says?

@vasily-kirichenko  I don't have any red squiggles at the left upper corner of the files, in any open statements. Also I updated @BentTranberg that right after making the screenshots. There was no change in behavior.

Unfortunately, after a few hours, the intelligence errors came back on my combined c# (asp.net mvc)/f# 4.6.1 solution but not in the all f# 4.6.1 solution (sharing 2 f# projects). With this improved stability comes the challenge to replicate this now... This is with FSharp tools version: <IP_ADDRESS>32701 installed.

same problem here, and those annoying squiggle lines dont disappear from the source code after setting build only

it happens when using type providers. and using latest nightly build

I've split out one specific problem with the error list here: https://github.com/Microsoft/visualfsharp/issues/3047

I'm going to close this until we have better repros for problems with the error list.  #3407 captures a very major source of bugs, and a fix is in progress (slowly because it requires a Roslyn fix), but I think there are other bugs too.
@jmecosta Can you isolate out that case as a specific bug report please?

@dsyme i think its fine now from my side, ive upgraded to latest also also upgraded all fsharp.data packages and everything works as expected. thanks anyway

@dsyme, I think you meant #3047. (Please don't respond. I'll delete this later, to reduce noise.)


Collect other types of simulation execution errors and include them in simulation error log

Can you please provide a sample OMEX simulation of which throws additional errors that are not yet being logged in the simulation logs?

For example, yesterday the example simulation wasn't executing and the logs were empty. Whatever the error was, some minimal information needs to be communicated to the user.
https://github.com/biosimulators/Biosimulators_test_suite/raw/dev/biosimulators_test_suite/BIOMD0000000297.omex


fix: update man files, Justfile

description of changes
This commit updates manual file templates, as well as the man-page build script using just to reflect the interim project name.
This is necessary to provide manual files that are accurate.
It should be somewhat easy to revert revert these changes, and is necessary to document new features as they are merged.
todo

[x] need to fix some spelling mistake where case should be all caps
[x] consider whether env vars like EXA_COLORS should be renamed

This seems ready to me, if anyone wants to test it:
# to build the manual pages
just man
# to check the output
man target/man/zetta.1
man target/man/zetta_colors.5


⚠ Bofrost.es has degraded performance

In 6529a3d, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1764 ms

Resolved: Bofrost.es performance has improved in 20b5a11.


fix #71: purge bad peers

Remove peers that disconnect before a successful hello. These correspond to invalid/outdated IP addresses.

Please rebase over master.

I ran the miner on this branch a few times.
I noticed there are currently 6 peers online and my peers.json has 6 items, so that's good.
I also got tons of messages like this: Bad Peer (disconnected without hello): x.x.x.x.
Do I understand correctly that this happens because we get told these (bad) peers exist by one of our 6 peers?
If so, I'll approve this once it's rebased.

I also got tons of messages like this: Bad Peer (disconnected without hello): x.x.x.x.

Over time, these messages will decrease, and we'll only see them when a peer leaves the network.

Do I understand correctly that this happens because we get told these (bad) peers exist by one of our 6 peers?

Correct. Until all the peers get updated, they'll continue to share bad peers to the other peers. You'll also pick-up bad peers from your existing peers.json the first time you run the update.


[Bug]: "Send bitcoin to your.." text can be wrapped undesirably on some device displays

Describe the bug
With an empty balance, viewing the savings or spending assets page displays either "SEND BITCOIN TO YOUR SPENDING BALANCE." or "SEND BITCOIN TO YOUR SAVINGS BALANCE.". On come devices such as Galaxy 24 Ultra, Galaxy Node 10 and One Plus 9 the text wraps in a way that doesn't match the design spec.
Reproduce

Using an affected device such as Samsung Galaxy 24 Ultra (can use lambdatest)
Without any savings or spending balance
On the main screen, tap on 'Savings' or 'Spending' asset
Observe text is wrapped in ways that can break up words or at least move the full stop to a new line(see screenshots for examples)

Screenshots / Recording
Galaxy Note 10

Galaxy S24 Ultra & OnePlus 9

Operating system
Android 10 & 11 & 14
Bitkit version
4ee19c9ad0693eebef7d21f1ee470a1c29f29006
Log output
No response

Verified in v118 in English only.
Any issues identified relating to other languages will be dealt with separately.


Remove ini-style configuration

This issue is imported from pivotal - Originaly created at Mar 5, 2018 by Thijs Kinkhorst
Now a conversion is done at deploy-time from ini style settings to yaml-style settings. Make yaml the only place to configure EB.
In addition to removing INI configuation. The addition of .env configuration support might be an interesting venture
https://github.com/symfony/demo/blob/v1.4.4/config/bootstrap.php

@thijskh I\'ve added error handling in the makeRelease.sh script to stop the script from continuing after somethinig went wrong. Because of this change the engineblock/install-branch role breaks in Deloy. Do you want me to fix that in Deploy too? (bstrooband - Mar 27, 2020)

Yes, please. Should not be failing I guess? (Thijs Kinkhorst - Mar 27, 2020)

I didn't look at the usage of .env configuration support to keep the change not too big and more atomic. Maybe we could address this in a later iteration if required. (bstrooband - Apr 1, 2020)


Blast Help Text

Closes https://github.com/DevChatter/devchatterbot/issues/263
Blast Types are now read from the DB instead of a variable which is never assigned. If no BlastType exists, the help text changes to show that.

Hmm seems the build failed due to the help text requiring to run without arguments being required.  Problem is that I need to string.Format in order to string interpolate. Otherwise we'd need to remove the extra helptext or manually input it as it's built from the DB data at the moment.


Add compute function for dividing a duration by a duration

Describe the enhancement requested
Currently the supported operations between durations are addition and subtraction (both giving back a duration with the same resolution).
Division between durations also makes sense (should yield a float) but is not supported
This would also enable the equivalent of python timedelta's .total_seconds() method, which is the same as just dividing by timedelta(seconds=1)
Component(s)
C++, Python

Thanks for reporting this!
Could you say more about your intended use of this? If you're looking to get number of units since epoch you could just run:
import pyarrow as pa
arr = pa.array([0, 10E17], pa.timestamp("ns"))
arr.cast(pa.int64())
<pyarrow.lib.Int64Array object at 0x7fea336b9ea0>
[
  0,
<PHONE_NUMBER>000000000
]

If you're looking to round your time to multiples of units I'd suggest looking at temporal rounding functions:
import pyarrow.compute as pc
pc.floor_temporal(arr, multiple=1, unit="second")
<pyarrow.lib.TimestampArray object at 0x7fea33be2740>
[
  1970-01-01 00:00:00.000000000,
  2001-09-09 01:46:40.000000000
]

More here: https://arrow.apache.org/docs/cpp/compute.html#conversions

Setup:
import numpy as np
import pyarrow as pa
import pyarrow.compute as pc

arr = pa.array(np.random.randint(0, 3_600_000_000, 10000, dtype="int64"), type=pa.duration("us"))
scalar = pa.scalar(1, type=pa.duration("s"))

As a more narrow example, what I'm currently doing (including what I believe pandas is doing under the hood) essentially works out to
pc.divide(
    pc.cast(pc.cast(arr, pa.int64()), pa.float64()),
    pc.cast(pc.cast(pc.cast(scalar, arr.type), pa.int64()), pa.float64())
)

but it would be nice if this worked to give the same result
pc.divide(arr, scalar)

Oh I see, I think this actually should work. Is this what you'd expect?
import pyarrow as pa
import pyarrow.compute as pc

arr1 = pa.array([1], type=pa.duration("s"))
arr2 = pa.array([10000], type=pa.duration("ms"))
pc.divide(arr1, arr2)
<pyarrow.lib.DoubleArray object at ...>
[
  0.1
]

This was supported recently in https://github.com/apache/arrow/pull/36800 and will be available in the next release.

Just a minor nit, the issue mentioned above will be part of 14.0.0 not 13.0.0

Just a minor nit, the issue mentioned above will be part of 14.0.0 not 13.0.0

Right, sorry I forgot that 13 is still not out yet. I've updated my comment above.

This was supported recently in #36800 and will be available in the ~next release~ 14.0.0 release.

I searched the issues to see if this was a duplicate, but forgot to search pull requests as well
Glad to see it will be supported soon!


Problem with Custom Menu widget

Menus in the sidebar, created with the WordPress Custom Menu widget, should in general be shown vertically, with the menu items above each other. In Foundation 6 you create vertical menus by assigning the class "vertical" to the menu's ul element.
It looks like FoundationPress 2.2.0 does not support this. Custom menus in the sidebar are displayed horizontally, with the menu items extending to the right, beyond the width of the sidebar.
I cannot find a decent solution for this. Any suggestion or fix?

I add a custom menu widget to a sidebar, in which the menu items should be displayed vertically. According to the Foundation 6 docs, to get a vertical menu I should add a class " vertical:

What is the decent way to do this while using FoundationPress 2?

Ok, here is a "solution": I created a file /assets/javascript/custom/custom-menu.js with the following contents:
$(document).ready(function () {
    $('aside.sidebar ul.menu').addClass('vertical');
});

I do not like this at all, I think the class should be set in the static content that is downloaded from the web server, but at least it works.
Any better suggestion??


Improve output and error status in az vm run-command

Currently we have about thousands of ubuntu VM's that we manage by using a tool called Rundeck  to orchestrate and connect to our VM's in a centralized way using SSH. We have hundreds of jobs that perform many different actions on our servers.
Because of security requirements we want to migrate from using SSH to using the run-command mechanism in Azure and integrate it into our command running mechanism (currently ssh based). I have tested the run-command feature and found a few limitations that makes this process challenging for us and I want to know what is the possibility to provide the following features

The old command of "az vm run-command invoke" provides a very limited log output and in some cases we need very large logs. On the other hand the new "az vm run-command create" is not providing the output back to stdout in any way. Is it possible to provide a feature to get the full output to stdout at the end of the run? if not, is it possible to write the file to some local file on the machine that is running the az command and not only to a blob?

The "az vm run-command create" command is not providing a clear exit status even if the command is failing. If the command itself return status code that is not 0, the output will still show "provisioningState": "Succeeded" - with this scenario it is very hard to manage operations at large scale and to understand what failed. Is there a possibility to provide an error code status in the output?

In both commands the output is provided only at the end and not during the running of the command.
In some cases the process might get stuck on an operation and there will be no way to know what is going on until the command will reach a timeout. In some use cases this very challenging for debugging. Is there a way to make the output live during the run?

These features can be a huge improvement to all customers using the run command and it can allow a great secure mechanism of managing VM's in a large scale

Compute

As we discussed offline in email "Azure cli run-command features", since the output data (include full log output and error status) should be returned by REST service, so those requirements need the support from service team.
@koralski Could you help look at these suggestions and feedback?

@amjads1, could you please work with the RunCommand team to see if this feature request can be accommodated?

Looking into the issue.

After you call az run-command create
output you need can be fetched by calling
az vm run-command show --name "myRunCommand" --vm-name "myVM" --resource-group "myRG" –expand instanceView
with the expand option you get details execution state and output streams.
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/run-command-managed

@tomkregenbild Can you please confirm if the above comments and options shared and if that works for your use case on what you are looking for?

@koralski I tested the command these are the finding:

First I had to run the "az vm run-command create" command and after that I needed to run the "az vm run-command show" to view the output something that makes much less efficient and hard to manage automatically.

The output under the show command is still limited to what seems to be 4779 characters which is not enough for long executions with a lot of output.

Even though I ran a non existing command and the error field provided an error. The exit status code was 0 and executionState was "Succeeded". Once again this is hard to debug.

While running the --expand instanceView parameter I got the following warning:
Argument 'expand' has been deprecated and will be removed in a future release.

@tomkregenbild Just to confirm, are you looking into the latest release of managed RunCommand?
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/run-command-managed

@amjads1 Yes, I used the latest version

@tomkregenbild Apologies for the late reply. We are looking in to this issue. We will provide an update once we have more details.
@amjads1 Could you please provide an update on this once you get a chance ? Awaiting your reply.

do we know how to work around the output limitation for 4096 bytes

@tomkregenbild We appreciate your feedback. - provisioningState issue will be fixed as part of GA, Next two weeks.

There is no live output feature as of today. If you use "asyncExecution": "false" you will be able to check the status of runcommand using get API

@navba-MSFT - Do you have any update on this feature request? It seems like we can sort of get around it by using the "--expand instanceView" flag since that dumps the exit code and the output/error streams, but as @tomkregenbild notes there is a warning today saying that that flag will be deprecated soon.

@tomkregenbild,

stdout output issue is already fixed for az vm run-command create (Managed Run Command). Please try again now. We are not making any changes to az vm run-command invoke (Action Run Command)
Exit codes should be fixed now. Try again. use "-treatFailureDeploymentFailure" API parameter. If it is true, the RunCommand's ProvisioningState will be set to "Failed" if Run Command terminates with non-zero exit code. Note this flag is not available in Powershell /CLI cmdlet yet. It should be available after https://github.com/Azure/azure-powershell/issues/22401 has been completed.
Yes. Stream your logs and errors to a blob using -OutputBlobUri and -ErrorBlobUri parameters. Please refer https://learn.microsoft.com/en-us/azure/virtual-machines/linux/run-command-managed#create-or-update-run-command-on-a-vm-using-outputbloburi-errorbloburi-to-stream-standard-output-and-standard-error-messages-to-output-and-error-append-blobs  or
https://learn.microsoft.com/en-us/azure/virtual-machines/windows/run-command-managed#create-or-update-run-command-on-a-vm-using-outputbloburi-errorbloburi-to-stream-standard-output-and-standard-error-messages-to-output-and-error-append-blobs
Complete logs would be available in the blobs


cannot synchronize hnswlib on conda-forge channel

Conda-forge documentation

[X] I could not solve my problem using the conda-forge documentation.

Installed packages
hnswlib

Environment info
hnswlib-feedstock in conda-forge organization

Issue
I'm a maintainer of hnswlib-feedstock on conda-forge channel.
However, after updating meta.yml in the repo to hnswlib version 0.6.2, its version on the channel (https://anaconda.org/conda-forge/hnswlib) is still 0.5.1, which was over 1 year ago.
Is this feedstock repo in the blacklist of conda-forge? I don't know where I should approach regarding this issue. So could you please give me a hand?
Many thanks!
Sincerely,
Yiming

Transferred to the feedstock in question
The issue is this line is effectively disabling CI. My guess is there is a typo here
https://github.com/conda-forge/hnswlib-feedstock/blob/cd333d29895ae747252320ff7b3ddfd6020f193a/recipe/meta.yaml#L14

@jakirkham Thank you so much for pointing out this typo! I've fixed it in a new PR, and will see if it could resolve the synchronization issue.

@jakirkham Now it's synchronized. The issue is resolved. Thank you again for your help!

xref: https://github.com/conda-forge/hnswlib-feedstock/pull/20


Add tab5/6 as native output format for AlignmentRecordRDD

From https://github.com/BenLangmead/bowtie2/issues/75:
--tab5 Each read or pair is on a single line. An unpaired read line is [name]\t[seq]\t[qual]\n. A paired-end read line is [name]\t[seq1]\t[qual1]\t[seq2]\t[qual2]\n. An input file can be a mix of unpaired and paired-end reads and Bowtie 2 recognizes each according to the number of fields, handling each as it should.
--tab6 is similar except, for paired-end reads, the second end can have a different name from the first: [name1]\t[seq1]\t[qual1]\t[name2]\t[seq2]\t[qual2]\n.

No hurry on this since bowtie2 may implement interleaved FASTQ.


Allow bots to message SpellBot

Resolves https://github.com/lexicalunit/spellbot/issues/427

Codecov Report

Merging #430 (e2b2878) into master (f86d8bd) will decrease coverage by 0.40%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #430      +/-   ##
==========================================
- Coverage   97.36%   96.95%   -0.41%
==========================================
  Files          65       65
  Lines        3188     3186       -2
  Branches      447      446       -1
==========================================
- Hits         3104     3089      -15
- Misses         47       58      +11
- Partials       37       39       +2

Impacted Files
Coverage Δ

src/spellbot/__init__.py
94.75% <ø> (-0.76%)
:arrow_down:

Continue to review full report at Codecov.

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


Web frontend

Is there an existing issue for this?

[x] I have searched the existing issues

URL
https://dotnet.microsoft.com/en-us/apps/aspnet/web-apps/blazor#:~:text=Blazor is a modern front,can deliver great web experiences.
 It seems this only happens when the site comes with this text highlighted from search engine. I managed to reproduce the problem.
If you visit https://dotnet.microsoft.com/en-us/apps/aspnet/web-apps/blazor, then the site opens as expected on mobile. I'll log an issue for my team to work on this.

Can you let us know what kind of device you're using here?

The issue has been resolved. Thanks for reporting.


Fix purging of settled invoices

LND can purge pending invoices, and currently the polling backend for LND trips up and wont recover.
This change will detect the return error code from the LND REST API and remove the pending invoice hash.
It also fixes a bug on pending invoice removal, where it filtered for the wrong entries.

Not sure on the handling of this case. I think it's normal flow that LND purges some invoice, and it's probably better to silently handle this.
One way would be to return null from getInvoice, but that doesn't feel right. Not sure how to handle this nicer.

Simplifide the fix, as I think the bug in the filter caused the actual problem of no longer being able to find invoices.

This is not a sufficient fix. I'm still investigating what exactly happens.

Related LND issue: https://github.com/lightningnetwork/lnd/issues/6299
When gc-canceled-invoices-on-the-fly=true is set on LND, there is no CANCELED state published, the invoice is deleted as soon as it expired.

This seems to work. If the individual invoice can't be found anymore, the state is set to Cancelled and the invoice-updated event is emitted.


Repo merge

This needs post merge cleanup pretty badly

I left out of my review, but pyproject and poetry lock need to be updated.


Output mesh render is blank

Hi,
I'm running the following command with the downloaded pre-trained model:
python -m demo --img_path ./input_images/000013.jpg --out_folder ./RingNet_output
and I'm seeing the following output file.

The renders are all dark. When I ran the same command with the flag --save_obj_file=True,
the obj file was successfully created and I could view it successfully in Blender.

I think the problem is occurring due to opendr version. Can you share your opendr version? Use 0.77

I think the problem is occurring due to opendr version. Can you share your opendr version? Use 0.77

My opendr version is 0.77, however, I meet the same issue. The output image is blank but the obj is corrent.

@Addarsh and @RobinROAR have you been able to solve this problem ? I am facing the same issue

Not yet. Feels bad

i have the same problem as yours.How do you visualize the .obj file ?   is  the blender a software?

@Addarsh  i have the same problem as yours.How do you visualize the .obj file ? is the blender a software?

@Addarsh i have the same problem as yours.How do you visualize the .obj file ? is the blender a software?

You can try meshlab

I have updated the requirements. Can you please reinstall the updated requirements? I think that should work. My opendr version is still 0.77 and that is only compatible with python 2.7. Can you please check your python version in the virtualenv too?

I have updated the requirements. Can you please reinstall the updated requirements? I think that should work. My opendr version is still 0.77 and that is only compatible with python 2.7. Can you please check your python version in the virtualenv too?

But mesh libraries work only with Pythhon 3.5 + and without installing mesh it is giving a error that "psbody-mesh is not available pls install it".

I have updated the requirements. Can you please reinstall the updated requirements? I think that should work. My opendr version is still 0.77 and that is only compatible with python 2.7. Can you please check your python version in the virtualenv too?

I work in python2.7 and meet the same problem.

I have updated the requirements. Can you please reinstall the updated requirements? I think that should work. My opendr version is still 0.77 and that is only compatible with python 2.7. Can you please check your python version in the virtualenv too?

But mesh libraries work only with Pythhon 3.5 + and without installing mesh it is giving a error that "psbody-mesh is not available pls install it".I even tried installing in 2.7 from the alternate link that you have provided but it is not getting installed.

What error do you get when using Python 2.7 if installing the forked mesh repo?

@Addarsh i have the same problem as yours.How do you visualize the .obj file ? is the blender a software?

You can try meshlab

I am able to get the 3D face but not that neutral expression can u pls help me in that issue.

@Addarsh i have the same problem as yours.How do you visualize the .obj file ? is the blender a software?

You can try meshlab

I am able to get the 3D face but not that neutral expression can u pls help me in that issue.

python -m demo --img_path ./input_images/000013.jpg --out_folder ./RingNet_output --save_obj_file=True --save_flame_parameters=True --neutralize_expression=True
You need to set the flag  neutralize_expression true

@Addarsh i have the same problem as yours.How do you visualize the .obj file ? is the blender a software?

You can try meshlab

I am able to get the 3D face but not that neutral expression can u pls help me in that issue.

python -m demo --img_path ./input_images/000013.jpg --out_folder ./RingNet_output --save_obj_file=True --save_flame_parameters=True --neutralize_expression=True
You need to set the flag neutralize_expression true

@Addarsh i have the same problem as yours.How do you visualize the .obj file ? is the blender a software?

You can try meshlab

I am able to get the 3D face but not that neutral expression can u pls help me in that issue.

python -m demo --img_path ./input_images/000013.jpg --out_folder ./RingNet_output --save_obj_file=True --save_flame_parameters=True --neutralize_expression=True
You need to set the flag neutralize_expression true

yes I have run the same command with neutralize_expression True.I ended up getting the error
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
At line 80 of serialization.py "dd = pickle.load(open(fname_or_dict))"

I have updated the requirements. Can you please reinstall the updated requirements? I think that should work. My opendr version is still 0.77 and that is only compatible with python 2.7. Can you please check your python version in the virtualenv too?

But mesh libraries work only with Pythhon 3.5 + and without installing mesh it is giving a error that "psbody-mesh is not available pls install it".I even tried installing in 2.7 from the alternate link that you have provided but it is not getting installed.

What error do you get when using Python 2.7 if installing the forked mesh repo?

when I run the make command I got this error :
Given no hashes to check 137 links for project 'pip': discarding no candidates
Makefile:5: recipe for target 'all' failed
make: *** [all] Error 1
Can u please help me in this issue.

tting the error
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
At line 80 of serialization.py "dd = pickle.load(open(fname_or_dict))" . I am working in Python 3.6 and got the mesh and flame parameters properly but I am not able to get this neut

I think this error is caused caused by a mismatch of pickle for python 2.7 and python 3.6. Can you try pickle.load(*, encoding='latin1')
Source: https://stackoverflow.com/questions/28218466/unpickling-a-python-2-object-with-python-3

tting the error
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
At line 80 of serialization.py "dd = pickle.load(open(fname_or_dict))" . I am working in Python 3.6 and got the mesh and flame parameters properly but I am not able to get this neut

I think this error is caused caused by a mismatch of pickle for python 2.7 and python 3.6. Can you try pickle.load(*, encoding='latin1')
Source: https://stackoverflow.com/questions/28218466/unpickling-a-python-2-object-with-python-3

No, it again gave the same error(UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte). I even tried encoding = 'bytes' it also didn't work.

tting the error
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
At line 80 of serialization.py "dd = pickle.load(open(fname_or_dict))" . I am working in Python 3.6 and got the mesh and flame parameters properly but I am not able to get this neut

I think this error is caused caused by a mismatch of pickle for python 2.7 and python 3.6. Can you try pickle.load(*, encoding='latin1')
Source: https://stackoverflow.com/questions/28218466/unpickling-a-python-2-object-with-python-3

It worked thaks a lot for replying back the link that u gave stated that pickle.load(open('flame.pkl','rb'),embeddings='latin1') is the replacement in python3 . Once again thaks for replying


Updating "migration_versions" to utf8mb4_unicode_ci

Hi,
I'm using Doctrine 2 as a part of Symfony (4.2) and therefore don't know where to put this idea, but because Doctrine manages the DB I think this is the right one.
Basically all my tables are automatically using the utf8mb4_unicode_ci collation but the migration_versions table just uses utf8_unicode_ci.
I know that the encoding formats aren't important at all because only numbers are stored, but I would find it nice if all tables would have the same encoding ;)

This belongs in the doctrine/migrations repository issue tracker. But this is fixed in master and will be released in 2.0
It was fixed here https://github.com/doctrine/migrations/pull/679

This belongs in the doctrine/migrations repository issue tracker. But this is fixed in master and will be released in 2.0
It was fixed here https://github.com/doctrine/migrations/pull/679


Render multiple GeoGebra applets correctly

There seems to be a problem when multiple GeoGebra applets occur in the same entity, see e.g. https://de.serlo.org/entity/repository/compare/157596/158782. Probably something wrong in the GeoGebra plugin itself (e.g. the "caching")

The frontend port seems not to affected by this: https://github.com/serlo/frontend/blob/master/src/components/content/Geogebra.tsx (caching is disabled)

Fixed in https://github.com/edtr-io/edtr-io/commit/f39085bc8bc61fa38c65a28fe008fc699c4c73f9, will be deployed the next time we update Edtr.io


Improved test coverage for IcebergMetricsFormatter

[#2184 ] test: Added test coverage for IcebergMetricsFormatter

What changes were proposed in this pull request?
Adding test cases to improve coverage of IcebergMetricsFormatter
Why are the changes needed?
Currently very low coverage for this class.
Fix: # 2184
Does this PR introduce any user-facing change?
No
How was this patch tested?
Running IntelliJ with coverage. Code coverage Number before and after are attached.
After the Fix

** Before the Fix**

@yaoderek , thanks for your PR, as you could see , IcebergMetricsFormatter is used to generate json string for the log, seems no need to add test to improve the test coverage.  cc @justinmclean

I don't see any harm in adding the test.


JLNotificationPermission -- needs to call [[UIApplication sharedApplication] registerForRemoteNotifications]?

I was having issues grabbing the device's push token from - (void)application:(UIApplication *)application didRegisterForRemoteNotificationsWithDeviceToken:(nonnull NSData *)deviceToken and I found out that it only worked after calling [[UIApplication sharedApplication] registerForRemoteNotifications].
Doesn't JLNotificationPermission need to call [[UIApplication sharedApplication] registerForRemoteNotifications] in order to retrieve the device's push token for remote notifications? Let me know if I'm doing this wrong. Thanks for the great work @jlaws!

Oh, I am also working with iOS 8+

Submit a PR and I will merge it and update the pod

Version 2.2.8 fixes this


Logger: homeassistant.components.sensor
Source: custom_components/bibliotheek_be/utils.py:63
Integration: Sensor (documentation, issues)
First occurred: 11:47:50 AM (2 occurrences)
Last logged: 2:13:33 PM

Error while setting up bibliotheek_be platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 289, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/bibliotheek_be/sensor.py", line 69, in async_setup_entry
    await dry_setup(hass, config, async_add_devices)
  File "/config/custom_components/bibliotheek_be/sensor.py", line 44, in dry_setup
    await componentData._init()
  File "/config/custom_components/bibliotheek_be/sensor.py", line 100, in _init
    self._user_details = await self._hass.async_add_executor_job(lambda: self._session.login(self._username, self._password))
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/bibliotheek_be/sensor.py", line 100, in <lambda>
    self._user_details = await self._hass.async_add_executor_job(lambda: self._session.login(self._username, self._password))
  File "/config/custom_components/bibliotheek_be/utils.py", line 63, in login
    oauth_
NameError: name 'oauth_' is not defined

sorry, I'm still figuring out the login/authentication. Nothing is working yet for this ;)

A first working release is now available, feel free to test/comment. But please note it's still early beta version, so major changes might require a removal/recreation of the integration setup in future.

Working great, thanks!


Back button is not working for turbo advance targeting a frame when rendered inside rails

I have a really hard bug that I've finally stripped down to an essential repro of it. I can't figure out what's going on.
What the user experiences on my app is:

Click a link which targets a turbo-frame on the page with action="advance"
The contents of the turbo-frame updates and the URL successfully changes
Clicking the back button changes the URL back to what it was before, but it does not revert the contents

It's a bad bug because it feels like the back button is broken, which, of course, we want turbo to respect back behavior.
In order to file a bug report, I tried to create a failing test case within this hotwired/turbo.js project. I opened src/tests/fixtures/frames.html and set up the test situation did the usual yarn build; yarn start. Oddly, the back button worked properly. I could not initially repro it.
Puzzled, I copied the exact same contents of frames.html over to my rails project and the bug appears! I could not figure out what the difference is since I stripped everything down. The only difference is that one is being served by yarn server and one is being served by rails server and rendering view templates. Then I copied these same files into the rails public/ directory and the bug goes away! For some reason this bug is only present when html & javascript code is served up through rails views. I'm baffled.
I know this is confusing so try watching this video where I show it very simply:
https://share.zight.com/o0uAplQJ
And here is a repository with of a fresh rails app with just the 4 files needed to reproduce it, everything else has been stripped out. Pull this repo and you can try for yourself what I just demonstrated in the video:
https://github.com/krschacht/hotwire-demo

When you render through a rails controller, turbo frame requests are rendered within a "turbo_rails/frame" layout. The problem is that this layout has an empty head tag which creates an incorrect snapshot. I've noticed turbo:before-render and turbo:render events do not fire during this frame navigation and they should. If you trace it a little bit back it comes down to this comparison:
https://github.com/hotwired/turbo/blob/f88bfe45ed44abb6ea9140a8ba138dfe68aa2730/src/core/drive/page_renderer.js#L92-L94
This also seems to only happen on initial page load, if you navigate to the test page with turbo first and then click the link everything works fine.
If this is something you need to fix now, you could force to always render layout layout "applicaiton" in a controller.

I've made a rails test script. I've also noticed that if you remove data-turbo-track="reload" there is no bug:
require "bundler/inline"

gemfile(true) do
  source "https://rubygems.org"

  gem "rails"
  gem "propshaft"
  gem "puma"
  gem "sqlite3", "~> 1.4"
  gem "turbo-rails"

  gem "capybara"
  gem "cuprite", require: "capybara/cuprite"
end

ENV["DATABASE_URL"] = "sqlite3::memory:"
ENV["RAILS_ENV"] = "test"

require "active_record/railtie"
# require "active_storage/engine"
require "action_controller/railtie"
require "action_view/railtie"
# require "action_mailer/railtie"
# require "active_job/railtie"
require "action_cable/engine"
# require "action_mailbox/engine"
# require "action_text/engine"
require "rails/test_unit/railtie"

class App < Rails::Application
  config.load_defaults Rails::VERSION::STRING.to_f

  config.root = __dir__
  config.hosts << "example.org"
  config.eager_load = false
  config.session_store :cookie_store, key: "cookie_store_key"
  config.secret_key_base = "secret_key_base"
  config.consider_all_requests_local = true
  config.action_cable.cable = {"adapter" => "async"}
  config.turbo.draw_routes = false

  Rails.logger = config.logger = Logger.new($stdout)

  routes.append do
    root to: "application#index"
    get "frame", to: "application#frame"
  end
end

Rails.application.initialize!

class ApplicationController < ActionController::Base
  include Rails.application.routes.url_helpers

  def index
    render inline: <<~TEMPLATE, formats: :html

          <%= csrf_meta_tags %>

          <turbo-frame id="my-frame">
initial frame
</turbo-frame>
          click

    TEMPLATE
  end

  def frame
    # not sure how stuff layouts into this bug template
    # simulate how a turbo_rails/frame layout is rendered on a turbo frame request
    if turbo_frame_request?
      render inline: <<~TEMPLATE, formats: :html

            <turbo-frame id="my-frame">
frame loaded
</turbo-frame>
            home

      TEMPLATE
    else
      render inline: <<~TEMPLATE, formats: :html

            <%= csrf_meta_tags %>

            <turbo-frame id="my-frame">
frame loaded
</turbo-frame>
            home

      TEMPLATE
    end
  end
end

class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
  driven_by :cuprite,
    using: :chrome,
    screen_size: [1400, 1400],
    options: {
      js_errors: true,
      headless: true
    }
end

Capybara.configure do |config|
  config.server = :puma, {Silent: true}
  config.default_normalize_ws = true
end

require "rails/test_help"

class TurboSystemTest < ApplicationSystemTestCase
  test "not a bug" do
    visit "/frame"
    click_link "home-link"
    assert_text "initial frame"
    click_link "frame-link"
    sleep 1
    assert_text "frame loaded"
    go_back
    sleep 1
    assert_text "initial frame"
  end

  test "a bug" do
    visit "/"
    click_link "frame-link"
    sleep 1
    assert_text "frame loaded"
    go_back
    sleep 1
    assert_text "initial frame" # bug
  end
end

@4lllex That's great that you have a repro and even a hunch about the core comparison that is failing. How hard do you think it would be to tee up a PR for this one? It'd be nice to get it in for Rails 8

#1241 looks like the same issue, they have a PR attached but I'm not sure how close it is to an accepted solution.

The fix seems to be known and it is to revert this line:
https://github.com/hotwired/turbo/blob/7ae954621ddfb36ecac96fda758a3f1d9cae065d/src/core/frames/frame_controller.js#L352
Here is a patch in the meantime:
// app/javascript/application.js

import "@hotwired/turbo-rails"

Turbo.FrameElement.delegateConstructor.prototype.proposeVisitIfNavigatedWithAction = function(frame, action = null) {
  this.action = action

  if (this.action) {
    // const pageSnapshot = PageSnapshot.fromElement(frame).clone()
    const pageSnapshot = Turbo.PageSnapshot.fromElement(frame).clone()
    const { visitCachedSnapshot } = frame.delegate

    // frame.delegate.fetchResponseLoaded = async (fetchResponse) => {
    frame.delegate.fetchResponseLoaded = (fetchResponse) => {
      if (frame.src) {
        const { statusCode, redirected } = fetchResponse
        // const responseHTML = await fetchResponse.responseHTML
        const responseHTML = frame.ownerDocument.documentElement.outerHTML
        const response = { statusCode, redirected, responseHTML }
        const options = {
          response,
          visitCachedSnapshot,
          willRender: false,
          updateHistory: false,
          restorationIdentifier: this.restorationIdentifier,
          snapshot: pageSnapshot
        }

        if (this.action) options.action = this.action

        // session.visit(frame.src, options)
        Turbo.session.visit(frame.src, options)
      }
    }
  }
}

Can confirm that your patch is working, @4lllex . Thanks a lot! We were trying the data-turbo-track=reload head workaround, but with this patch an empty head (or rather, no head at all as mentioned in #1237)

Also confirming, that adding layout, as recommended here helped. https://github.com/hotwired/turbo/issues/1300#issuecomment-2308152198

Confirmed too, but adding a layout didn't solve the problem :cry:
File added as `app/views/layouts/turbo_rails/frame.html.haml`
!!!
%html{ lang: 'fr-FR' }
  %head
    %meta{ charset: 'UTF-8' }
    = csrf_meta_tags
    = yield :head
  %body
    = yield


Attack Card Damage does not add Elite properly

Elite and Weak modifiers are not applied to damages rolled with the Damage buttons from the Attack card.

Closing this issue as the damage buttons now come with the PF2e system by default, this module will no longer be maintained!


docker 1.10 syslog format issue

Example syslog line:
Feb 05 15:23:40 hostname org/image/sha256: 5a5e7/5c5f67d9e251[10876]: data...
The hash of the image has changed to have the prefix sha256: and that messes up some syslog file readers because of the :  and also cuts off a lot of the bytes of the image hash.
docker version
Client:
 Version:      1.10.0-rc3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   08c24cc
 Built:        Tue Feb  2 23:01:16 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.0-rc3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   08c24cc
 Built:        Tue Feb  2 23:01:16 2016
 OS/Arch:      linux/amd64

I'm running an RC, but I didn't see any syslog related changes recently, so it's probably still an issue.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@vikstrous any idea when we produce that log? I'm trying to reproduce but I'm running out of ideas.

are you using a custom log tag?

Oops, it looks like we use the following log option: "tag": "{{.Name}}/{{.ImageID}}/{{.ID}}",

ImageIDs might be either names or digest tags. I wonder if we should not do anything about the ID but add an ImageName to make it more explicit. ImageName would always return the name, for example ubuntu/latest. I'm guessing that's what you really want. The parsing issue with syslog is unfortunate, but I don't think we should strip the prefix in the digest.

Oh, I just realized that ImageName is already a thing in the logger context:
https://github.com/docker/docker/blob/master/daemon/logger/context.go#L18
Maybe you can use that to avoid the prefix.

This is how the log looks like with "{{.Name}}/{{.ImageName}}/{{.ID}}":

Thanks @calavera!
I think we can close this one? The new ImageiD format has an unexpected side effect on syslog, but at least there are good alternatives.

Hmm... am I missing something or is the tag not in any of these fields? ContainerImageName doesn't uniquely identify an image. It's definitely better than the ImageID in this case, but still doesn't achieve the same goal.

It shows the tag if you start the container with a tag, but you might end up with the same issue
Feb 10 13:00:18 docker-test-interactive docker/syslog-test-2/busybox:latest/07f40e2fa66f[29887]:     1     0 root     R     1196  0.0   7  0.0 top#015#033[H#033[JMem: 1296928K used, 15136988K free, 408K shrd, 211832K buff, 811988K cached
Feb 10 13:00:18 docker-test-interactive docker/syslog-test-2/busybox:latest/07f40e2fa66f[29887]: CPU:  0.0% usr  0.0% sys  0.0% nic  100% idle  0.0% io  0.0% irq  0.0% sirq
Feb 10 13:00:18 docker-test-interactive docker/syslog-test-2/busybox:latest/07f40e2fa66f[29887]: Load average: 0.00 0.01 0.05 1/150 11
Feb 10 13:00:18 docker-test-interactive docker/syslog-test-2/busybox:latest/07f40e2fa66f[29887]:

Hmm.. this is probably why we were using the image id in the first place. Our containers are always started with a specific tag.

@vikstrous so if we added ImageTag would that help? You could construct {{.ImageName}}/{{.ImageTag}} right?

@tiborvass Just posting here what we talked about offline: having an ImageTag field is helpful, but I still can't use the ImageName field if it contains a colon.

What about this?
{{split .ImageName ":" | join "/"}}
I'm almost joking, but having basic functions to manipulate strings might be more useful than adding more fields :laughing:

Is this currently possible? If so, that's a great workaround. Otherwise it's much easier to just provide the extra fields than to force users to do string manipulation on the existing fields.


duf: Add darwin supoort

Motivation for this change
error: Package is not supported on ‘x86_64-darwin’, refusing to evaluate.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Thanks! Since (according to the README) it also supports BSDs, can you just set it to platforms.unix?

Result of nixpkgs-review pr 107271 run on x86_64-darwin 1

Result of nixpkgs-review pr 107271 run on x86_64-darwin 1

1 package built:

duf

Thanks! Since (according to the README) it also supports BSDs, can you just set it to platforms.unix?

It seems so. I changed it to platforms.unix.

Also have you tested that it builds and works properly on a darwin machine?

Yes. I did. I installed first with  $ export NIXPKGS_ALLOW_UNSUPPORTED_SYSTEM=1 and now I did also a build with nixpkgs-review.


iio ADD_CHANNEL logic cause scale 0

@yongli3
Please check the following patch line 78 and 81 https://github.com/solettaproject/soletta/commit/38e2261dcd3ad7d6f057451acdcdb583671664a9

if (base->use_device_default_scale) //should be if(!base->use_device_default_scale)

   channel_config.scale = scale;

if (base->use_device_default_offset) //should be if(!base->use_device_default_offset)

   channel_config.offset = offset;

Thanks @laykuanloon!

Patch that fixes it was merged, closing issue.

Hi @yongli3, @laykuanloon & @edersondisouza,
I would like to bring up a discussion here due to this issue. The code flow seem to be when there is an error on setting the scale/offset (no matter it is 0 or others value), Soletta only shows the warning and then continue rest of the functions but the data reading still using the scale/offset value input from user. So, if we could perform a read back after the warning and reset the value back to the original instead of user input value, i think this should be the proper handling code flow. What do you guys think?
root@intel-corei7-64-cavs-hda:/opt/soletta-dev-app/repos/singlesession/testfbp# ./runfbp.sh testWebMagneto.fbp sol-flow2.json
WRN: ./src/lib/io/sol-iio.c:559 iio_set_channel_scale() Could not set scale to 0.000000 on channel [in_magn_x] of device1
WRN: ./src/lib/io/sol-iio.c:559 iio_set_channel_scale() Could not set scale to 0.000000 on channel [in_magn_y] of device1
WRN: ./src/lib/io/sol-iio.c:559 iio_set_channel_scale() Could not set scale to 0.000000 on channel [in_magn_z] of device1
#anon:6:37 (-0.000000, -0.000000, 0.000000) (direction-vector)
#anon:6:37 (-0.000000, -0.000000, 0.000000) (direction-vector)
#anon:6:37 (-0.000000, -0.000000, 0.000000) (direction-vector)

Thanks & Regards,
Wilson

I think this is a good idea. Just prompt a message to inform user that the scale is revert to default value.

Hi @fulong82,
Actually, it doesn't use set value regardless of failing. If it fails to set, it keeps current value, with happens to be 0 (uninitialised). But I like the idea of using channel default in this case. It should be fairly simple to change logic on https://github.com/solettaproject/soletta/blob/master/src/lib/io/sol-iio.c#L941 and https://github.com/solettaproject/soletta/blob/master/src/lib/io/sol-iio.c#L946.
Please, if not doing it soon, open a issue for this task.

Hi @edersondisouza,
What i mean here is instead of
    if (config->scale > -1)
        iio_set_channel_scale(channel, config->scale);
    else
        channel_get_scale(channel);

    if (config->use_custom_offset)
        iio_set_channel_offset(channel, config->offset);
    else
        channel_get_offset(channel);

I would suggest
    if (config->scale > -1)
        iio_set_channel_scale(channel, config->scale);

    channel_get_scale(channel);

    if (config->use_custom_offset)
        iio_set_channel_offset(channel, config->offset);

    channel_get_offset(channel);

I think it is a secure practice to read back whatever you have written to ensure the value, in case it has being changed/limited/conditioned by HW or such as signal interference and so on.
What do you think?
Regards,
Wilson

@fulong82 Please open another ticket for the issue or suggestion as this ticket was closed.


Visualize logs

Questions:

What logs do we care about?
What log visualization platform?

Some ELK stack

Do not want to do this, outside the scope of this project.


Inventory command/conditions issues

I've seen to have gotten in a pinch.
I'm currently using Spigot 1.10.2 (version git-Spigot-90f61bc-6053c5c), and when I use commands that search for and deal with items in the inventory, I encountered a few problems. One I've already made aware to BlackCoyote and Morphan1, that the "on player (un)equips armor" event doesn't really work.
In addition to such, I have found the following issues:

the "on player clicks in inventory" event doesn't properly register shift clicks.
when using "if" commands to detect the items in the armor slots, it appears that if I order my commands like this: http://pastebin.com/8jPvrWS4 , the head armor slot must be empty before the other slots are registered by the script.
if I reverse the order of the same script (that is, detect the head armor first, then the chestplate, then the leggings, then the boots), then only the head armor slot is affected by the script.
clicking on one item consecutively makes scripts ignore it (or, according to the debugger, makes the scriptname return as null). Clicking on the item, then another item seems to work fine, but clicking on the same item twice in a row appears to break the scripts. This can be replicated by using the same above script and trying to click on the Structure Void blocks with no cursor item.

EDIT: On the IRC, I use the nickname "suspic", just in case you're wondering about my comment on how I already told BlackCoyote and Morphan1 about the event.
EDIT 2: Even without the async's in the script, the issue persists.

Please remove the line - async: from each script, that really really really does not belong.

Even without async, the issue persists. I added the async as a test, I should have noted that you could remove the async's and the issue still exists. My apologies.

Can you post a debug report from the script?

http://one.denizenscript.com/paste/36902
That's the debug when I'm clicking on the same item consecutively (the Structure Voids). When I click on one of them, the script does as it should. When I click on the same Structure Void again, the script doesn't seem to be able to catch that particular instance. Clicking on other items "fixes" the issue, but it allows for many loopholes.
https://youtu.be/rpLW8Zdtj2Q
The above video shows what happens on the user's end when they click on the same item consecutively. It's hard to determine if I'm just single clicking or double clicking, but I assure you that when the items seem to disappear, I am double clicking.
I could replicate the issue with at least 1 of each Structure Void in each armor slot (instead of 0 in each slot), but since the result would be the same (me being able to drag the Structure Void out, even if the script is set to cancel the click event), I didn't feel the need to.
Although there is an issue with a user being able to duplicate the items, even if I set the scripts to cancel the click event. The debug would be almost the same (registering on the first click, not registering on the second), and if the user puts the item in a different slot, then there's a high chance of the item duplicating. If you want, I can provide video evidence of this tomorrow night (timezone: EST).
P.S., on a near unrelated note, if you want video and console evidence of shift clicks not working, I can also do that tomorrow night.

You're in creative mode in that video; inventory control is minimal in creative mode due to Minecraft code oddities.
Please reattempt in survival mode.

In survival mode, just clicking does nothing. The script works as it should. When I shift+click, though, the Structure Voids disappear.
I would like to provide another video of what happened while I was in Survival, but my schedule won't permit me enough time to do so. If needed, I can provide the evidence tomorrow night.

is_shift_click is a direct read from an underlying Bukkit event. For that to fail, something went wrong on Bukkit or Minecraft's end :/
Also use ||null in your tags to prevent them from erroring when they give back null.


Draft: UI-charts: Initial version with adaptations for series and event mapping

. Still displays a crash on ios when using the packaged version in the app:
***** Fatal JavaScript exception - application has been terminated. *****
NativeScript encountered a fatal error: Uncaught ReferenceError: NativeClass is not defined
at
../../node_modules/@essent/nativescript-ui-charts/index.ios.js(file: app/webpack:/app/node_modules/@essent/nativescript-ui-charts/index.ios.js:90:0)
at webpack_require(file: app/webpack:/app/webpack/bootstrap:19:0)
at ../../node_modules/@essent/nativescript-ui-charts/angular/fesm2015/essent-nativescript-ui-charts-angular.mjs(file:///app/vendor.js:245186:88)
at webpack_require(file: app/webpack:/app/webpack/bootstrap:19:0)
at ../../libs/xplat/nativescript/features/src/lib/ui/components/consumption-graph/graph/graph.component.ts(file:///app/bundle.js:29644:96)
at webpack_require(file: app/webpack:/app/webpack/bootstrap:19:0)
at ../../libs/xplat/nativescript/features/src/lib/ui/components/consumption-graph/consumption-graph.component.ts(file:///app/bundle.js:29171:80)
at webpack_require(file: app/webpack:/app/webpack/bootstrap:19:0)
at ../../libs/xplat/nativescript/features/src/lib/ui/components/index.ts(file:///app/bundle.js:30362:105)
at webpack_require(file: app/webpack:/app/webpack/bootstrap:19:0)
at ../../libs/<…>

@geertrutten pushed up fixes in commit above.


[#184736292] Fix bosh deploy rebuilds from happening every time

What
After some investigation it turns out that the bosh-cli is recreating the bosh deploy each time because the manifest sha1 doesn't match what is in the bosh state.
While examining the bosh manifest for several runs I could see the credhub_tls certificates were being rotated for every deployment.
I believe this was broken by this pr.
The line:
contents.delete("credhub_tls") if contents.key? "credhub_tls"

Removes the already generated credhub_tls certificates from the vars-store. This means they get recreated as part of the deploy every time.
I believe the original intention of the change was to remove it once in each environment to change the certificates ca and then remove this line at a later point.
How to review
This change was tested in dev-04. You can see a build here that no longer redeploys bosh.
I have deployed a test to ensure this certificate will get rotated when a certificate is getting "close" to expiry by adding '--min-remaining-days 400' to the pipeline. You can see the results here. I downloaded the bosh-vars-store.yml file before and after and can confirm this certificate was rotated.
Who can review
Any team member familiar with the bosh certificate process.

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

Talked to Ben about what's happening here. My approval is on the thing in principal (the what and the how). It should also have an approval from someone who has reviewed it in detail.

I'm convinced.


Issue getting nodes from cloud?

❯ bacalhau node list --api-host f503ee90-c4dc-4ac7-a7b5-a460aeb94fb3.us1.dev-cloud.expanso.dev

Error: Unable to parse API call response body. Error: "invalid character 'p' after top-level value". Body received: "404 page not found\n"

What does this mean?

@jamil this is related to missing https protocol in the host address, right? what can we do to improve this?


Issues running from app data partition

I am attempting to run a ping-pong update with fwup.  I have a separate partition for app data and downloaded the fwup archive to that partition so that I could keep the RootFS small.  When running fwup, however, it first tries to unmount all of the partitions.  When it tries to unmount the app data partition, it receives a partition in use error, and then exits.
I think that a good solution might be to have fwup ignore any errors that it receives while trying to unmount the partitions, and to continue on.  There shouldn't be any errors due to mounted partitions, since in the configuration file we can specify if a particular partition needs to be unmounted before running.  Thoughts?

Agreed. I had a particularly painful experience a few years ago with a partition not getting unmounted when it should have been, and implemented a simplistic approach that worked on my projects. A couple others have run into the same situation as you, so I'd accept a patch to ignore the error if the unmount fails. I also agree that the config file can be used to figure out the partitions that need to be unmounted.

Addressed by #10.


Use wildcard wildcard as a default operation name for server spans

Use wildcard path as a default operation name for server spans.
cc @bhs @yurishkuro

How come no tests need to change?

There already have been tests for both decorators. This PR just adds wildcard decorator to a default set of decorators. In test tracing feature is created with builder specifying a set of decorators.

in my experience people prefer as little customization as possible, relying on the default behavior. So backwards compatibility of the default behavior is important to have its own tests.

Agree, this is the main reason of this PR. I'm going to restructure tests in the next PR.


Adopt new localization string for releases page

What would you like to be added
This issue is a follow-up task from https://github.com/kubernetes/website/issues/36568.
https://kubernetes.io/hi/releases/ displays some English text:

To replace this text with localized strings, you can update data/i18n/hi/hi.toml with these recent changes from data/i18n/en/en.toml.
Specifically, these:
# Avoid using conjunction_1.
# Must match the context in layouts/shortcodes/release-data.html
# Appears on https://kubernetes.io/releases/
# For example the "and" in "Complete 1.25 Schedule and Changelog"
[conjunction_1]
other = "and"

# Deprecated. Planned for removal in a future release.
# Use [release_full_details_initial_text] instead.
[release_complete]
other = "Complete"

# Replace [release_complete] with [release_full_details_initial_text]
[release_full_details_initial_text]
other = "Complete"

[release_schedule]
other = "Schedule"

[release_changelog]
other = "Changelog"

/language hi

/assign

@rolfedh Do we need to add the same lines in English as mentioned here to data/i18n/hi/hi.toml

/triage accepted

Yes. At a minimum, add the items from data/i18n/en/en.toml that are mentioned above. You can also localize other items in data/i18n/en/en.toml that are missing from data/i18n/hi/hi.toml. I suggest using a command line diff tool, or a GUI tool like Meld, to compare and reconcile the two files.

/remove-lifecycle stale

Fixed by #38631
/close


Make GraphNode._resort() behave like VBoxContainer

Now uses copied and slightly modified BoxContainer code like a VBoxContainer.

Discovered a bug where the ports (slots) don't connect correctly after resizing...


Added support a zookeeper shell

Why are the changes needed?
It is useful to support a zookeeper shell so that we can view some metadata if necessary. For example:
$ bin/kyuubi-zookeeper-shell -server localhost:2181
Connecting to localhost:2181
Welcome to ZooKeeper!
...
ls /
[zookeeper, kyuubi]

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Nice idea, I saw the original zkCli.sh contains parameters to control log level, how can we achieve such ability.
https://github.com/apache/zookeeper/blob/eb63dd9757e7234515e81799d82cc0af4a0ae7f9/bin/zkCli.sh#L41-L43

Nice idea, I saw the original zkCli.sh contains parameters to control logs, how can we achieve such ability.
https://github.com/apache/zookeeper/blob/eb63dd9757e7234515e81799d82cc0af4a0ae7f9/bin/zkCli.sh#L41-L43

We can achieve this by reuse KYUUBI_JAVA_OPTS option defined in kyuubi-env.sh.

Thanks, merging to master.


Style for choicePrompts

Hi !
Can you provide an example for defining the style of a choicePrompt ?

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

ID: 72538432-fde5-22ab-7997-8a34b11aa24e
Version Independent ID: c469fa5b-2590-ee16-1491-51b90c069204
Content: Gather user input using a dialog prompt - Bot Service
Content Source: articles/v4sdk/bot-builder-prompts.md
Service: bot-service
Product: bot-framework
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

@NejibK Once you create your ChoicePrompt object, you can explicitly set its "list style" by setting its_style_ property. The values are defined in the ListStyle enumeration (C# / JS).
If that is enough to unblock you, please close this issue.


Clean up tracy module CLI option handling

More options are being requested to be added to tracy profiler script.
Group options and bring logic check callbacks. e.g. -l option only work with -p option
Add the following options:

no-signpost to remove all signposts from the report and tracy file
port should be global and change the port for the run as well

Env variables to be handled here:
TT_METAL_PROFILER_SYNC for device host sync
TT_METAL_DEVICE_PROFILER_DISPATCH for profiling dispatch cores only in non-report mode
TT_METAL_PROFILER_DIR for globally changing profiler directory for all profiler artifacts


[docker script] buildx failed with: error: failed to solve: failed to compute cache key: "/go/src/casdoor/.git/refs/heads" not found: not found

https://github.com/casdoor/casdoor/runs/7939276377?check_suite_focus=true

@qianxi0410

Still error: https://github.com/casdoor/casdoor/runs/7940484132?check_suite_focus=true

@qianxi0410 plz revert the Docker change


Great script but weighted prompts do not work with it

For example (muscles:1.1) and (muscles:1.9) will give you wildly different  results.
with text files I've tried (muscles:1.__num__) with the num text file being number 1 through 9 so output being (muscles:1.#).
I've also tried (muscles1.1) through (muscles1.9) in a textfile then using __muscles__ and that didn't work. I also tried without parenthesis and it didn't work. Bascially every possible way to do it in combination with pulling from a text file did not work.
I looked through the code and can't understand why it's not working when ( ) + prompt weights are involved.

sorry I'm a moron I was using prompts from file. It's working great thanks.


Annotations not rendered and positioned properly on iOS

Annotations/Markers are not rendered and positioned properly on iOS in version 5.12.1. I tested version 5.12.0 where everything seems to work fine.
Screenshot of correct version @ 5.12.0:

Video of buggy version @ 5.12.1:
https://user-images.githubusercontent.com/148895/138849436-16a6a7a6-0a6e-41bb-9228-91ef76a75c24.mp4
Amount and position of markers are wrong. Also the rendering is kind of broken, because the markers only appear while zooming.
Steps to reproduce in a fresh project:

Setup a new sample iOS project
Add exact version of Maplibre iOS 5.12.1
Use this sample code:

import UIKit
import Mapbox

class ViewController: UIViewController {
    override func viewDidLoad() {
        super.viewDidLoad()

        let mapView = MGLMapView(frame: view.bounds)
        mapView.styleURL = URL(string: "[SOME_STYLE_URL]")
        mapView.autoresizingMask = [.flexibleWidth, .flexibleHeight]
        mapView.setCenter(CLLocationCoordinate2D(latitude: 51.572276, longitude: 8.108391), zoomLevel: 9, animated: false)
        view.addSubview(mapView)

        let annotation = MGLPointAnnotation()
        annotation.coordinate = CLLocationCoordinate2D(latitude: 51.572276, longitude: 8.108391)
        annotation.title = "Big city"
        annotation.subtitle = "Some nice city description"
        mapView.addAnnotation(annotation)
    }
}

Replace style URL
Zoom out and in to see some markers somewhere sometimes

I met this problem too

Please try the latest iOS release.


feature: namespace options and supplemental groups for cri manager

Signed-off-by: YaoZengzeng<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixes part of #635
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
We skip the test of "HostNetworkMode", because CRI manager doesn't support logging now.

Codecov Report

Merging #753 into master will decrease coverage by 0.12%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #753      +/-   ##
=========================================
- Coverage   13.43%   13.3%   -0.13%
=========================================
  Files          99      99
  Lines        5992    6049      +57
=========================================
  Hits          805     805
- Misses       5133    5190      +57
  Partials       54      54

Impacted Files
Coverage Δ

daemon/mgr/cri.go
0% <ø> (ø)
:arrow_up:

cli/container.go
47.05% <0%> (-0.24%)
:arrow_down:

daemon/mgr/cri_utils.go
34.12% <0%> (-5.21%)
:arrow_down:

daemon/mgr/spec_process.go
0% <0%> (ø)
:arrow_up:

cli/common_flags.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@allencloud PTAL.

@YaoZengzeng could you also add a test for group-add flag?

@allencloud @Letty5411 PTAL.

@allencloud Updated

Do you have any further thoughts on this?  @Ace-Tang


fixes

Coverage increased (+0.8%) to 79.33% when pulling 06ed6b4a1a355ebeeeb573b0c0732691ba8d2491 on Tr0nYx:master into dfe7241dd2d7fba3c4804a2d01b31c558eb6bb54 on Alpha-Trader:master.

Coverage remained the same at 78.541% when pulling 446875b2cac4724bb908f4e05a82adc0a986ab16 on Tr0nYx:master into dfe7241dd2d7fba3c4804a2d01b31c558eb6bb54 on Alpha-Trader:master.


Cucumber not running migration/maintaining test schema

I originally commented the following in #278, but I think this deserves its own issue for tracking purposes:

Probably relevant: rails/rails#13528 and ActiveRecord::Migration.maintain_test_schema!

Following the Rails PR, my schema doesn't update anymore and I'm currently trying to find a way to update my test schema to continue working. Pretty major if you ask me.

@joallard sorry for not replying earlier.
Yes, it's been an earlier issue that somehow got left behind.
There was a change in the way the test schema was created in the latest rails versions.
I will go back and look at the earlier issue, but using the schema load as you have used it, should be the recommended way to do it.

As the linked issues have been merged in and this hasn't had any updates in over 4 years I'm going to close it. If the issue still exists on a later version of cucumber-rails / rails feel free to open a new post.


[fix][fn] Fix JavaInstanceStarter inferring type class name error

Fixes #19885

Master Issue: #xyz

PIP: #xyz

Motivation
JavaInstanceStarter still cannot infer type class name
Modifications

create a new FunctionCacheManager in JavaInstanceStarter to create the function class loader and pass this fnCache to ThreadRuntimeFactory's constructor method.
create a new constructor method for ThreadRuntimeFactory, which add a fnCache parameter so that it can reuse the fnCache created in JavaInstanceStarter
make ThreadRuntime#loadJars public so it can be called in the JavaInstanceStarter
use the function class loader to infer type class instead of the instance class loader(which cannot load the user's function class)
check whether userConfigs is null before calling userConfigs.containsKey
use functionDetailsBuilder.getSink().getClassName() to load sink class instead of functionDetailsBuilder.getClassName() since this one is always org.apache.pulsar.functions.api.utils.IdentityFunction
use functionDetailsBuilder.getSource().getClassName(), classLoader to load source class instead of functionDetailsBuilder.getClassName() since this one is always org.apache.pulsar.functions.api.utils.IdentityFunction

Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)

[x] This change is a trivial rework / code cleanup without any test coverage.

(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/jiangpengcheng/pulsar/pull/7

@jiangpengcheng Could you add more details for the actual modifications you made in this PR?

@jiangpengcheng Could you add more details for the actual modifications you made in this PR?

Sure, Updated

/pulsarbot rerun-failure-checks

Codecov Report

Merging #19896 (5ff88fb) into master (1e44ba1) will increase coverage by 37.52%.
The diff coverage is 25.00%.

@@              Coverage Diff              @@
##             master   #19896       +/-   ##
=============================================
+ Coverage     24.42%   61.94%   +37.52%
- Complexity      291     3507     +3216
=============================================
  Files          1603     1859      +256
  Lines        124343   136750    +12407
  Branches      13571    15034     +1463
=============================================
+ Hits          30369    84709    +54340
+ Misses        89490    44264    -45226
- Partials       4484     7777     +3293

Flag
Coverage Δ

inttests
24.32% <0.00%> (-0.11%)
:arrow_down:

systests
25.11% <25.00%> (?)

unittests
59.24% <12.50%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../pulsar/functions/runtime/JavaInstanceStarter.java
0.00% <0.00%> (ø)

...functions/runtime/thread/ThreadRuntimeFactory.java
68.23% <30.00%> (+68.23%)
:arrow_up:

...pulsar/functions/runtime/thread/ThreadRuntime.java
50.40% <100.00%> (+50.39%)
:arrow_up:

... and 1384 files with indirect coverage changes


Feature Request port to electron?

Just wondering if this could be ported to Electron so that it's a standalone application instead of an open tab in a browser.
Great product thank you!

I know @thunderousninja was interested in both vue and electron. Maybe you want to take a look into this? If you have time :)...


gnu(7056)=6161

GAP      4.dev
GrpConst 2.5
Runtime: 1442968 ms
Isomorphic groups eliminated

Thank you for submitting new gnu(n). We will recalculate it and add to the database if we get the same result.
Checklist for the reviewer:

[X] checked and assigned labels "new (n<50000)" or "recomputation", if needed
[X] indicated that the recomputation has been started
[ ] reproducible value of gnu(n)
[ ] no essential runtime differences
[ ] sufficient level of technical details
[ ] assigned label "validated" after successful review

Checklist for the maintainer:

[ ] correct arguments of commands to store data
[ ] correct location of new line in the data file
[ ] mention reviewer in the commit message

gnu(7056)=6161
GAP      4.8.3
GrpConst 2.5
Runtime: 1823768 ms
Isomorphic groups eliminated
Replicated on Linux server (AMD Opteron 6376 1400 MHz processor, 512 GB RAM)


Added 'caller_name' support to lookup

Adding support for multiple Types, such as caller_name, carrier without breaking backwards support.
After contacting Twilio's support, the only way to get Carrier and CallerName data is to combine both requests,
such as: Type=carrier&Type=caller-name.
I tried to implement it in a way that it wouldn't break backwards compatibility.
https://www.twilio.com/docs/lookup/tutorials/carrier-and-caller-name#identify-a-phone-numbers-carrier-and-type

Thanks for maintaining backwards compatibility and apologies that it took so long to get these changes merged.