having https

Is it possible to enable HTTPS for site redirection and all of the web pages that your viewing as it just loads them as HTTP at the moment and I can't find anything

Any luck on this?


Migrations: Fix migration from Cloud to Self-Hosted

Related issue:

https://github.com/appwrite/appwrite/issues/6336


Write text on objects?

Do we need something like the write library for writing text on objects?
https://www.thingiverse.com/thing:16193

https://github.com/brodykenrick/text_on_OpenSCAD

It occurred to me that one way to ease the placement of text onto objects would be to make attachable text, maybe a writing module?  Not sure what it would be called.  print.   In essence, the align options of text() would be replaced by anchors and at least for simple tasks like subtracting or adding text from planar faces, it would be easy to use.
However, this wouldn't replace the need for something that can make curved text to go on a cylinder, etc.  So maybe it's not enough by itself?

We have distributors for things like line_of(), arc_of(),  or ovoid_spread() which provide a $idx value that could index into a string, and .  Perhaps we need spiral_spread() or somesuch for cylindrical and spherical spirals?

I do like the idea of attachable text.

Thinking on it, though, attachable text is probably only implementable by making a custom version of the attachable() module, since the geometry of proportional letters is not accessible to user-space.  This needs further investigation.

May be able to make attachable text as leaf nodes that cannot have things attached to them.

It seems like it's not necessary to allow things to be attached to text.   I think text that doesn't take children would be fine.
Note that Jordan (I think) was working on code to provide text metrics.  That might be useful for this.  I'm not sure if it was userspace, though.
What does spiral spread have to do with attachable text?

atext() is implemented for an anchorable 3D text block.

text = reverse("A long text example to wrap around a cylinder, possibly for a few times.");
L = 4*len(text);
maxang = 360*L/(PI*50);
spiral = [for(a=[0:-1:-maxang]) [25*cos(a), 25*sin(a), 10+30/maxang*a]];
difference() {
    cyl(d=50, h=50);
    path_spread(spiral, len(text))
        atext(text[$idx], size=5, anchor=str("baseline",CENTER),orient=DOWN);
}

We already have path_spread which can do this.  There are limitations, though.  You can really only do this with monospaced fonts, or it will look bad.  You have to guess the font spacing.  If we use the new text metrics this would be easier.  But still if the font has ligatures it's going to look funny placed letter by letter.  And kerning may mean that a single letter has different dimensions than the letter in a text string.
The right solution is to make a path_text() module that places the text on a path so it's easy to control its position and orientation.  I had issues with the text being inverted and backwards.   We could do this without textmetrics by having the user specify a text spacing.  Or I suppose guessing a spacing from the specified text size.

When we can get access to kerning and character size info, I know we have a function that can split up a path at various lengths.

Not sure we can get kerning data, though.  You can get the size of a piece of text, it looks like.


Initial stab at updating the changelog

Coverage remained the same at 90.131% when pulling 4ff28f5e6129bb49b6d753148c60030e6461c6ad on CHANGLOG into 746d400a68010279bc146d44b42dc83f73ea4ea9 on develop.


Make user caseworker role explicit

Changes
We want to make the caseworker role explicit rather than inferring it from the lack of any other role.
This adds a new column to the User model of caseworker,  and updates the existing code to use this.
Checklist

[x] Attach this pull request to the appropriate card in Trello.
[x] Update the CHANGELOG.md if needed.

 Add regional_caseworker role to User


Responsive - deactivate stickybox on mobile devices

Hello,
Most of the time when using a sticky sidebar, you want the sidebar not to be sticky on small devices.
Is there any way to achieve this ?
Thank you for your work, great lib btw :)

did you find a solution to it ? I have same challenge

@Johnson444 well for now I define a conditionnal component like :
const StickyWrapper = isLargeScreen ? StickyBox : React.Fragment;
Where isLargeScreen is your test to check the breakpoint.

@adrienlamotte  Thx ! Will do this as well. If I have time I will look closer into the package. If I find a better way I will let you know

@Johnson444 well for now I define a conditionnal component like :
const StickyWrapper = isLargeScreen ? StickyBox : React.Fragment;
Where isLargeScreen is your test to check the breakpoint.

This solution might work, when the children can be re-rendered. In case that the children should not be re-rendered this might kill some flow.
I would prefer like a prop disable , that just disable the calculcation from StickyBox and act just an simple div
          <StickyBox
            offsetTop={160}
            offsetBottom={0}
            disable={!isLargeScreen}
          >

A simple and fast solution to remove stickyBox on mobile is to add a css class and use @media to cancel the sticky effect according to the desired width. Voici un exemple :
`
// In component

/CSS/
@media (max-width: 1024px) {
.stickyBox{
position: inherit !important;
top: inherit !important;
}
}
`
I hope it helps ++.

@Arnaud-Desportes Nice solution, thanks for sharing!

@Arnaud-Desportes
You saved me lots of hours, Thanks for the solution!

MUI 5
import React from 'react';
import { SxProps, Theme, styled } from '@mui/material/styles';

import ReactStickyBox, { StickyBoxCompProps } from 'react-sticky-box';

export interface StickyBoxProps extends StickyBoxCompProps {
  disabled?: boolean;
  sx?: SxProps<Theme>;
}

const StickyBox = styled(ReactStickyBox, {
  shouldForwardProp: (prop) => ['offsetTop', 'offsetBottom', 'bottom'].every((key) => key !== prop),
})<StickyBoxProps>(({ theme, disabled }) =>
  theme.unstable_sx({
    ...(disabled && {
      position: 'inherit !important',
      top: 'inherit !important',
    }),
  }),
);

export default StickyBox;


Feedback for “Getting Started with CldImage”

Hi hoped to use this in an Astro project. Would there be allot of work for me to adapt it? Thanks

hey @jamesgrubb i hope to eventually create an Astro equivalent, but for now you have a few options
We also have Svelte Cloudinary, which Astro supports Svelte components, so theoretically you should be able to use that within your Astro app
Svelte Cloudinary and Nuxt Cloudinary both currently rely on Unpic to fill in the whole where this package uses Next Image. It's the underlaying component that we pass a Cloudinary URL into that allows us to not have to manage all the <img element optimizations and just pass in a constructed URL based on transformation props
They also both use a package I created Cloudinary Util which has a URL Loader module that allows you to easily construct a Cloudinary URL using the same API you get from the props:
constructCloudinaryUrl({
  options: {
    src: 'my-public-id',
    width: 800,
    height: 600
  },
  config: {
    cloud: {
      cloudName: 'my-cloud'
    }
  }
});

You can see the source for in both packages:

https://github.com/cloudinary-community/svelte-cloudinary/blob/main/svelte-cloudinary/src/lib/components/CldImage.svelte
https://github.com/nuxt-modules/cloudinary/blob/main/src/runtime/components/CldImage.vue

Ultimately it works along the lines of:
<Image
 ...imageprops
  transformer={(width) => {
    // combine component props and options, normalize width
    return constructCloudinaryUrl(options, config)
  }}
/>

If you end up trying one of these methods id love to hear about your experience! and happy to answer any questions


typescript migration

This code needs to built (convert to ts and export it pa)

We should be able to compile it to js

and where is the type definitiions for the same. ?


Add option to automatically create symlink on host-side to improve experience in IDEs and other applications

Background: a synced folder in Vagrant displays as an empty directory. While it works great for a working representation of your plugin or theme it does not allow for editing.
If, after creating the synced folder, a symlink is created in place of the synced folder, it is now possible to edit the plugin/theme in your IDE.

BTW, this exchange from synced folder to symlink must occur every time Pressmatic would normally run the Volumes add-on. So after a restart, volume remap, etc.
Each individual entry would need a checkbox to create a symlink.
The flow would likely be something like the following.

Delete the symlink in the Container Destination
Create/Re-create the synced folder from Host Source to Container Destination
Delete the synced folder in Container Destination
Create symlink from Host Source to Container Destination

Here's the code I used to use with VVV. I would have to run the shell script every time VVV reloaded, etc.
https://gist.github.com/afragen/549cc092a1f61c2a8d7c

This is my biggest annoyance. I can symlink a folder, but it doesn't sync.
I'm using Local for development, and my workflow is now this:

Edit code
Run rsync script

Over and over :( It doesn't feel like it's a real symlink or an add-on volume so much as I can map a 'command'.

I believe I have a working solution. It obviously requires modification to suit your personal needs.
https://gist.github.com/afragen/748e4780b6057d4c41cf9e466557042a
After the plugins or themes are added via the Volumes Addon, simply run this modified script and symlinks with replace the created synced folders that LbF creates.
@Ipstenu no more rsync


Test with workflow.Go and workflow.Sleep actually sleeps

When running a test of a workflow that does workflow.Sleep, the test normally doesn't actually sleep. However, when combined with workflow.Go and an activity that blocks, Sleep also blocks until the Go activity completes.
Here's a test case that displays the behavior - this emulates something we'd like to do, namely to start a monitor loop which checks for some preconditions, and if they stop being true then interrupt the workflow execution. Of course, we could also do this by starting a child workflow and having that signal back, but I assume this should also work (notice that the sample doesn't include any signalling since that's not the problem here):
package versioning

import (
	"context"
	"testing"
	"time"

	"github.com/stretchr/testify/suite"
	"go.uber.org/cadence/activity"
	"go.uber.org/cadence/testsuite"
	"go.uber.org/cadence/workflow"
)

type ts struct {
	suite.Suite
	testsuite.WorkflowTestSuite

	env *testsuite.TestWorkflowEnvironment
}

func (s *ts) SetupTest() {
	s.env = s.NewTestWorkflowEnvironment()
}

func (s *ts) TestSleep() {
	s.env.SetTestTimeout(10 * time.Second)

	a1 := func(ctx context.Context) error {
		ticker := time.NewTicker(1 * time.Second)
		defer ticker.Stop()

		count := 0
		for {
			activity.RecordHeartbeat(ctx)
			s.T().Log("tick", count)
			count++
			select {
			case <-ctx.Done():
				return ctx.Err()
			case <-ticker.C:
				continue
			}
		}
	}

	wf := func(ctx workflow.Context) error {
		ctx, cancel := workflow.WithCancel(ctx)
		defer cancel()

		workflow.Go(ctx, func(ctx workflow.Context) {
			ctx = workflow.WithScheduleToStartTimeout(ctx, 1*time.Second)
			ctx = workflow.WithStartToCloseTimeout(ctx, 1*time.Minute)
			err := workflow.ExecuteActivity(ctx, a1).Get(ctx, nil)
			if err != nil {
				s.T().Log(err)
				return
			}
		})

		workflow.Sleep(ctx, 10*time.Second)
		return nil
	}

	workflow.Register(wf)
	activity.Register(a1)

	s.env.ExecuteWorkflow(wf)
	s.NoError(s.env.GetWorkflowError())
}

func Test(t *testing.T) {
	suite.Run(t, &ts{})
}

This is by design. The unit test fast forwards clock every time workflow is blocked and cannot make forward progress. It is done to support unit testing long running workflows.
To verify that clock indeed moved you can compare s.env.Now() before and after s.env.ExecuteWorkflow(wf) call.
The time is not forwarded while an activity is running as test framework doesn't have any control over activity execution code.

We considered adding activity.Sleep that calls time.Sleep under normal execution and forwards time when running under the a unit test. But I'm not sure if there is a real value in such feature.

OK, makes sense - so I then have to mock out that activity for the test to be able to run?

Yes, to unit test different workflow code paths mocking activities is the best option. I assume that that you are aware that the TestWorkflowEnvironment.onActivity supports mocking directly.

Yes, I was just trying to write a test that verified that the signalling worked, but I'll just have to mock all of that out.


[occm] Support for haproxy loadbalancer service provider

Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
What happened:
Our hosting provider that offers openstack cloud environments as a service, only provides neutron-lbaas with haproxy service provider for loadbalancerv2.
Setting lb-provider=haproxy and use-octavia=false in the cloud configuration file for occm just worked fine (although documentation says that we can use either amphora or octavia values). A loadbalancer service got created correctly and is working as expected.
What you expected to happen:
I was expecting some errors in occm pods when deploying a loadbalancer service in k8s.
How to reproduce it:
Set lb-provider=haproxy and use-octavia=false in cloud.conf file for occm when neutron-lbaas uses haproxy service provider then deploy a loadbalancer service in k8s.
Anything else we need to know?:
Only the documentation needs to be updated apparently. Please tell me if there is anything to do to validate this more precisely.
Environment:

openstack-cloud-controller-manager(or other related binary) version: docker.io/k8scloudprovider/openstack-cloud-controller-manager:latest
OpenStack version: v18 Rocky with Neutron LBaaS v2 extension

Neutron-LBaaS has been deprecated since stable/queens and its support is no longer maintained in openstack-cloud-controller-manager. Please see more information here https://github.com/kubernetes/cloud-provider-openstack/blob/master/docs/using-openstack-cloud-controller-manager.md


Create health and readiness checks for all of our helm charts including archive node and bots

Define what are the checks first

Automate validation of archive node + bots (even a script which needs to be manually invoked would suffice for now).
Plan for Kubernetes Healthcheck Types:

startup-probe : similar use to liveness checks though used to measure container liveness during initialization
liveness: verify container network accessibility on regular cadence (e.g. check Mina daemon is listening on its libp2p/client port every 30s)
readiness: technically verifies container readiness for receiving traffic (e.g. daemon ready to be queried by public Mina testnetX daemon GraphQL DNS/endpoint) though intended/planned use is for measuring container synchronization with chain expectations

Implementation Design:

Leverage Helm's Named Templates as it seems like an appropriate tool to use here based on the following:

enables single healthcheck definition and re-use throughout source chart and all others (external dependency charts, subcharts) within a Helm operation runtime
single and consistent source of truth for healthchecks throughout Mina's Helm charts in terms of location and naming conventions (e.g. _healthchecks.tpl located at /chart-path/templates/)
allows for added variability in definitions with sane/common defaults as template embedded variables can be scoped to template rendering arguments.
template definitions can be and are recommended to be namespaced by source chart (though namespace collisions are handled with last to load taking precedence)

Specify container port defaults within values.yaml with sane/common settings to allow for variability when necessary and DRY'er code.
Define and name all container ports (of interest) within container specifications so that checks can rely on this metadata rather than specific port numbers.

Helm Chart/Pod Healthcheck Ideas:

archive-node

containers

daemon

ports: [ client/libp2p : 8301-83xx, graphql : 3085, metrics : 10000, external : xxxx/variable ]
startupProbe (failureThreshold x periodSeconds for daemon to listen on libp2p port):

port: client/libp2p
protocol: TCP
failureThreshold: 60
periodSeconds: 5

liveness:

port: client/libp2p
protocol: TCP
< see CommonCheckConfig template>

readiness:

exec cmd:

network status: status=$(curl localhost:3085/graphql -d'{<sync-status-query>}'); [[ status == "synced" ]] && return 0 || return 1
peer-count: script to check local daemon peer count against seed-nodes
< see CommonCheckConfig template>

archive

ports: [ archive-daemon : 3086, archive-postgres : 5432 ]
startupProbe:

port: archive-postgres
protocol: TCP
periodSeconds: 5
failureThreshold: 60

liveness:

port: archive-daemon
protocol: TCP
< see CommonCheckConfig template>

readiness:

exec cmd: script to check archive observed blocks == daemon block height, can store in archive-node image
< see CommonCheckConfig template>

block-producer

containers

daemon

see archive-node :: daemon && think template re-use

user-agent

ports: [ metrics : 8000 ]
startupProbe:

< see block-producer :: daemon >

liveness:

port: metrics
protocol: TCP
< see CommonCheckConfig template>

readiness:

exec cmd:

agent_sent_txns: curl localhost:8000/metrics | check-count-against-graphql

bots (faucet + echo SVCs)

ports: [ graphql : 3085 ]
startupProbe:

< see block-producer :: daemon >

liveness:

port: graphql
protocol: TCP
< see CommonCheckConfig template>

readiness:

exec cmd:

faucet-balance: CMD == balance > 0/??

seed-node

containers

daemon

see archive :: daemon

snark-worker

containers

daemon/coordinator
snark-worker

o1-bot

containers

bot

due to variability of container application, either:

allow specification of startup, liveness and readiness checks as variables (similar to how other resources - e.g. ports, volumes - are specified)
define named template which allows all or a set of checks to be rendered based on input scope/variables
provide single exec check CMD to be customized by applications

leaderboard

containers

leaderboard [cronjob]

ports: [ archive-postgres : 5432 ]
startupProbe:

*see archive :: startupProbe

liveness:

exec cmd: postgres query to access/read archive database
< see CommonCheckConfig template>

readiness:

exec cmd: script to ensure postgres state is in sync with testnet seed-node(s)
< see CommonCheckConfig template>

buildkite-exporter

containers

exporter

ports: [ metrics-collection : 8000 ]
startupProbe:

port: metrics-collection
protocol: HTTP
path: /metrics
failureThreshold: 3
periodSeconds: 10

liveness:

port: metrics-collection
protocol: HTTP
path: /metrics
< see CommonCheckConfig template>

readiness:

exec cmd: curl localhost:8000/metrics
< see CommonCheckConfig template>

CommonCheckConfig template:
- initialDelaySeconds: 30
- periodSeconds: 10
- failureThreshold: 3
Note:

k8s healthcheck (liveness + readiness) docs
k8s healthcheck best practices

Note:

k8s healthcheck (liveness + readiness) docs
k8s healthcheck best practices

Monitoring Solutions:

service or cronjob script which queries kubernetes API for all failed containers within namespace and broadcasts to configured Discord channel(s), email address(es) or desired monitoring/alerting channels

Example CMD:
kubectl get pods --field-selector=status.phase!=Running --namespace <testnet> -o json
Example STATUS output to parse and push to alerting channel:
...
            "status": {
                "message": "Pod ephemeral local storage usage exceeds the total limit of containers 50Gi. ",
                "phase": "Failed",
                "reason": "Evicted",
                "startTime": "2020-10-11T17:12:57Z"
            }

Some questions:

Is there a good mechanism for specifying health checks in such a way that we can reuse them for (a) alerting at runtime if things go wrong / automating behavior (b) running nightly networks and making sure they stay healthy . Maybe this is what you're thinking already, but could you explain how this could work?

This is a great list for startup-probe, liveness, and readiness for individual services. Two more questions here:

Is there a nice way we can support health checks that need to span across services? For example, "the network" is healthy if "a block is received to all parties within one slot".
I bet many folks on the protocol team will have more ideas for application-specific health checks of different components (example: proofs take longer than X seconds, snark pool response time too slow, transition frontier size too large, etc). Could you prepare something that makes it easier to gather this information from folks? I think maybe a table in a Notion doc could work?

I like the idea of pushing everything into GraphQL, but I wonder if it may not be easy enough to instrument. Is there a way we can connect the integration testing system into the health checks too?

Is there a good mechanism for specifying health checks in such a way that we can reuse them for (a) alerting at runtime if things go wrong / automating behavior (b) running nightly networks and making sure they stay healthy . Maybe this is what you're thinking already, but could you explain how this could work?

For sure and right, that's where the (monitoring solution) k8s Service or Cronjob script mentioned above comes into play. The original thinking was that:

a "Watchdog" pod would be deployed within each testnet/namespace containing a single container with the necessary tooling and service-account privileges to monitor all pod/container health within its testnet/namespace. The healthchecks implemented for each container would then be collected by this Watchdog when in violation using some form of the kubectl get pods ... CMD referenced.
Rather than only pushing healthcheck violations to a designated Discord channel or group of channels on some cadence (using something like a k8s cronjob), the Watchdog would also be implemented as a Discord bot capable of responding to Discord commands (e.g. !brain healthcheck qa-net) note: the name "brain" comes from the name of Inspector Gadget's dog, Brain. The bot/watchdog would then respond with a list of pods in violations (based on and populated with the details from the individual checks) and perhaps further guidance (think links to runbooks, howtos etc).

1. Is there a nice way we can support health checks that need to span across services? For example, "the network" is healthy if "a block is received to all parties within one slot".

Good question and my thinking around this has centered around expanding the Watchdog pod mentioned above. Kubernetes healthchecks themselves are only scoped to individual containers by design and implementation and not meant to provide status checks outside of this scope (e.g. network, role or some defined group wide). Based on this, it seems like more group based checks could be provided by an additional script+container solution included within the Watchdog pod. This script could be implemented in any language (e.g. something like OCaml) with really no constraints on what it could check. Since it would be deployed alongside the Watchdog in the same namespace, all of its healthcheck violations would also be captured and reported accordingly.

2. I bet many folks on the protocol team will have more ideas for application-specific health checks of different components (example: proofs take longer than X seconds, snark pool response time too slow, transition frontier size too large, etc). Could you prepare something that makes it easier to gather this information from folks? I think maybe a table in a Notion doc could work? I think in the table we should include a section for a sketch of implementation details for each check and maybe also a rough estimate on scope to get it to work. (does the graphql endpoint already exist? do we need to also create it? etc)

Sure, a Notion doc sounds good.

Great, thank you! It sounds like we'll be able to throw many engineers in to help parallelize building different checks as well once the infrastructure is in place.

it may be worth discussing whether the current healthchecks implemented in bash are better served written in another language - like Python

Personally, I think it's okay if it starts in bash and evolves into something better as scope grows. Especially as some of the existing ones are more or less one-liners right now, right?

Cool and yep, right.


Macro not working with command including %u

The following macro works when spawning a mpv youtube video with a max resolution of 720:
macro v set browser "mpv -ytdl-format='bestvideo[height<=?720]+bestaudio/best'" ; open-in-browser ; set  browser linkhandler

However when I use %u it stops working:
macro v set browser "mpv -ytdl-format='bestvideo[height<=?720]+bestaudio/best' %u" ; open-in-browser ; set  browser linkhandler

This is important because in the end I would like to detach the process as so:
macro v set browser "nohup mpv -ytdl-format='bestvideo[height<=?720]+bestaudio/best' %u &>/dev/null &" ; open-in-browser ; set  browser linkhandler

The strange thing is that the followoing does work:
macro v set browser "nohup mpv -ytdl-format=22 %u &>/dev/null &" ; open-in-browser ; set browser linkhandler

Can anyone tell me what I'm doing wrong? Or is this some sort of bug?

Hi! I don't see anything wrong with any of your macros, but to investigate, I would need to know a few things about your setup. All but one of them were explicitly listed in the issue template which you deleted. The last thing is a URL of a feed I could use for testing.
The template can be found here. If you update the issue within a week, we can talk, otherwise I'll just close this.

Newsboat version (copy the output of newsboat -v or the first line of git show):
newsboat 2.19.0
System: Linux 5.5.18-3-MANJARO (x86_64)
Compiler: g++ 9.3.0
ncurses: ncurses 6.2.20200212 (compiled with 6.2)
libcurl: libcurl/7.69.1 OpenSSL/1.1.1f zlib/1.2.11 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.2.0) libssh2/1.9.0 nghttp2/1.40.0 (compiled with 7.69.1)
SQLite: 3.31.1 (compiled with 3.31.1)
libxml2: compiled with 2.9.10

Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
Iauto-reload yes

external-url-viewer "urlview"

bind-key j down
bind-key k up
bind-key j next articlelist
bind-key k prev articlelist
bind-key J next-feed articlelist
bind-key K prev-feed articlelist
bind-key G end
bind-key g home
bind-key d pagedown
bind-key u pageup
bind-key n next-unread
bind-key N prev-unread
bind-key D pb-download
bind-key U show-urls
bind-key x pb-delete
bind-key "SPACE" open

color listnormal color214 default
color listnormal_unread color175 default

color listfocus color239 color223 standout bold
color listfocus_unread color223 color239 bold

color info color235 color250
color article cyan default

browser linkhandler
macro , open-in-browser
macro a set browser "nohup mpv --no-video %u &>/dev/null &" ; open-in-browser ; set  browser linkhandler
macro v set browser "nohup mpv -ytdl-format='bestvideo[height<=?720]+bestaudio/best'  %u &>/dev/null &" ; open-in-browser ; set  browser linkhandler
macro b set browser "qutebrowser %u &>/dev/null &" ; open-in-browser ; set browser linkhandler

Steps to reproduce the issue:

Use config
Press , followed by v
Observe video is not opened in mpv.

I can reproduce this issue. This is a bug in Newsboat.
Newsboat apparently applies some formatting to the browser variable when it contains %u which causes < to be replaced by <>.
This causes
-ytdl-format='bestvideo[height<=?720]+bestaudio/best'
to become
-ytdl-format='bestvideo[height<>=?720]+bestaudio/best'
which is definitely not correct behaviour.
Caused by me in https://github.com/newsboat/newsboat/commit/ad4708d264bf05196ececfcba86e122b8aa84b8b (PR #835).

@tamis-laan, thanks for the updated report. This was indeed a bug in Newsboat; it was introduced in 2.19. The fix is in the master branch now, and will be released with 2.20 at the end of next month.
If you can't build from source, you can apply a workaround: put the command into a script, and use the script as your browser. Just don't use any redirections in the browser setting, that's what trips Newsboat up.


WebAudioPlugin not registering correctly when running Chrome from local file

Running the TestSuite.html test-app from the website works fine. It registers, detects and uses the WebAudioPlugin correctly.
Running the same TestSuit.html test-app from local harddisk (downloaded from github trunk) doesn't work and it reverts to the HTMLAudioPlugin (forcing the code to register only WebAudioPlugin doesn't work, resulting in a message that Chrome doesn't support SoundJs).
What gives?

I have just meet the same problem and thanks for the information.
BTW, the https://xhr.spec.whatwg.org/ shows that
Synchronous XMLHttpRequest outside of workers is in the process of being removed from the web platform as it has detrimental effects to the end user’s experience. (This is a long process that takes many years.) Developers must not pass false for the async argument when current global object is a Window object. User agents are strongly encouraged to warn about such usage in developer tools and may experiment with throwing an InvalidAccessError exception when it occurs.

so is this code needed to be changed back?
xhr.open("GET", "WebAudioPlugin.js", false);

=>
xhr.open("GET", "WebAudioPlugin.js");


Starting brazilian portuguese translation

Used the English Version to get a basic scaffold to work

Wow, That's awesome

Go is growing my friend! :)

http://redmonk.com/sogrady/2015/07/01/language-rankings-6-15/
:-)
steadily growing


Add note about custom errors

Okay, so if you pass a filterWith rule, that function should return true|false.
However, you can throw a custom error in that function instead. It's just that the customer error needs to implement toJSON and return { field, message }.

I propose that we should include a way to define custom errors when filterWith, like so:
const schema = {
  first_name: { type: 'string', required: true, error: 'first_name is not defined' },
};

Thoughts, anyone?

@motss oh, interesting!
So, internally, fields throw errors that are caught specifically by type, so it might take more effort to permit custom error types.
Could you get what you need if it was possible to define the error message? For example:
const schema = {
  first_name: { type: 'string', required: true, message: 'first_name is not defined' },
};

@davidpaulhunt Missed out that message field. Will definitely try that out. whitelister is awesome btw. Simple yet powerful enough to get the job done and done it right! 👍

@motss thank you!
I haven't implemented message field yet. But I could do so if it would help your use case.
😄

@davidpaulhunt I see. But it'd be nice if it can be function, like so:
// message: val => { ... },
const schema = {
  first_name: {
    type: 'string',
    required: true,
    message: val => `first_name (${val}) is invalid`,
};

In this case, I can return the invalid value along with the error message.

@davidpaulhunt Any update on this?

@motss your example of above can already be implemented with filterWith:
const schema = {
  age: {
    type: 'integer',
    required: true,
    min: 0,
    max: 999,
    filterWith: (val) => {
      if (val < 0 || val > 999) throw new Error(`Invalid age (${val}). Please specify an age in between 0 and 999`);
      if (someOtherCondition) {
        throw new Error('Please specify a valid age');
      }
      return true
    },
  },
};

So I don't think message would need to be a function; but it could still be an option to let you choose what the default error message is.

@davidpaulhunt This totally fit my purpose. 👍


0.30.1-b10

Current coverage is 49.73%

Merging #1427 into develop will not affect coverage as of 352ff33

@@            develop   #1427   diff @@
=======================================
  Files           963     963
  Stmts         66985   66986     +1
  Branches          0       0
  Methods           0       0
=======================================
  Hit           33317   33317
  Partial           0       0
- Missed        33668   33669     +1

Review entire Coverage Diff as of 352ff33

Powered by Codecov. Updated on successful CI builds.

:+1:


Cannot use Update-Dbainstance with -KB option

Hi folks, I am trying to patch remote server using below command
Update-DbaInstance with -KB option as per the document.
It was working fine until last week but now it doesn't do anything. If I remove KB, it just works fine.
I tried from various machines with different target servers and still same behaviour.
Installer dbatool in a new machine, no luck from there too.
Current Dbatools version : 1.0.114 ( also tried old versions)
I am using KB because default parameters with no KB doesn't install security patch (GDR) ,it only supports SP and CU
Anyone else noticed this issue ?
Anyhelp is appreciated. Thank you.

The issue doesn't follow the template. Please re-open or submit all the information required as per https://github.com/sqlcollaborative/dbatools/issues/new?assignees=&labels=bugs_life&template=Bug_Report.md&title=[Bug]

@nvarscar it is because it was migrated from another repo...so the issue template was not available when @SajalBagchi opened it.

Yes, that's what I figured. But still need more information 🤷
Which is why I suggested re-opening or checking out the template. Specifically, I need error messages, if any. Also, @SajalBagchi any chance you can post the output of your command ran with a -Debug switch?

I am sorry new to to github so could not follow the template. Here are the details :
Environmental information
PowerShell Version        : 5.1.14393.1944
dbatools latest installed : 1.0.113
Culture of OS             : en-US
SQL Server:
Microsoft SQL Server 2012 (SP4-OD) (KB4091266) - 11.0.7469.6 (X64)
Feb 28 2018 17:47:20
Copyright (c) Microsoft Corporation
Enterprise Edition (64-bit) on Windows NT 6.3  (Build 9600: ) (Hypervisor)
us_english
Report

Host used

[X ] powershell.exe
[X ] ISE
[ ] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[X ] Still buggy in powershell.exe

Errors Received
VERBOSE: [17:15:42][Resolve-DbaNetworkName] Resolving <server_name> using .NET.Dns GetHostEntry
VERBOSE: [17:15:42][Resolve-DbaNetworkName] Resolving 10.XXX.XX.XXX using .NET.Dns GetHostByAddress
VERBOSE: [17:15:42][Get-DbaCmObject] Configuration loaded | Cache disabled: False
VERBOSE: [17:15:42][Get-DbaCmObject] [<server_name>] Retrieving Management Information
VERBOSE: [17:15:42][Get-DbaCmObject] [<server_name>] Accessing computer using Cim over WinRM
VERBOSE: [17:15:43][Get-DbaCmObject] [<server_name>] Accessing computer using Cim over WinRM - Success
VERBOSE: [17:15:46][Resolve-DbaNetworkName] Resolving server_name using .NET.Dns GetHostEntry
VERBOSE: [17:15:48][Resolve-DbaBuild] Looking for 11.0.7493
VERBOSE: [17:15:48][Resolve-DbaBuild] We have 91 builds in store for this Release
VERBOSE: [17:15:48][Get-DbaCmObject] Configuration loaded | Cache disabled: False
VERBOSE: [17:15:48][Get-DbaCmObject] [<server_name>] Retrieving Management Information
VERBOSE: [17:15:48][Get-DbaCmObject] [<server_name>] Accessing computer using Cim over WinRM
VERBOSE: [17:15:49][Get-DbaCmObject] [<server_name>] Accessing computer using Cim over WinRM - Success
VERBOSE: [17:15:50][Update-DbaInstance] Attempting to configure CredSSP for remote connections
VERBOSE: [17:15:50][Initialize-CredSSP] Configuring remote host to use CredSSP
VERBOSE: [17:15:53][Get-DbaCmObject] Configuration loaded | Cache disabled: False
VERBOSE: [17:15:53][Get-DbaCmObject] [<server_name>] Retrieving Management Information
VERBOSE: [17:15:53][Get-DbaCmObject] [<server_name>] Accessing computer using Cim over WinRM
VERBOSE: [17:15:54][Get-DbaCmObject] [<server_name>] Accessing computer using Cim over WinRM - Success
VERBOSE: [17:15:54][Resolve-DbaBuild] Looking for KB 4532098
VERBOSE: [17:15:54][Resolve-DbaBuild] We have 91 builds in store for this Release
VERBOSE: [17:15:54][Get-SqlInstanceUpdate] Found applicable upgrade for SQL2012 to SP4 (KB4532098)
VERBOSE: [17:15:54][Get-SqlInstanceUpdate] Current SQL2012 version 11.0.7493 on computer [<server_name>] matches or already higher than target version 11.0.7493
$error[0] | select *` -- did not return anything
In above verbose, I have replaced Server Name with <Server_Name> and masked IP address
-->
Steps to Reproduce
Update SQL 2012 GDR update KB4532098  (but it could be any Target KB)

Execute Update-DbaInstance -ComputerName <Server_Name>-KB KB4532098 -Path  -Restart -Confirm:$false
Once Update is done, try running the same command again, ideally it should throw a warning like below from Get-SqlInstanceUpdate but that's not happening and it is terminating the script to move further.

WARNING: [14:32:44][Get-SqlInstanceUpdate] 11.0.7469 on computer [<server_name>] is already the latest available.
Another issue is, if we try running below command without KB as below, it does not detect security GDR KB4532098 as latest patch and gives above warning. (Reason I had to use KB parameter)
Update-DbaInstance -ComputerName Server1 -Path \network\share -Restart -Confirm:$false
Expected Behavior

Actual Behavior

The output seems reasonable to me - it fails to install the KB that is already installed on the target machine: KB version and machine versions match - 11.0.7493.

Current SQL2012 version 11.0.7493 on computer [<server_name>] matches or already higher than target version 11.0.7493

Also, to install a KB, it has to exist in the dbatools build reference file - that usually lags behind for a few days. If you're trying to install the KB that is not added yet, you'd get an error.
So I'm still confused what is the issue here. Could you please add Expected & Actual behaviour?

Expected Behaviour 1 : powershell is terminating and do not goto next scripts when installing with KB option. Also, on server with patch already installed, the warning message does not appear (if KB option is used). I could only see that information when Verbose/debug option is set.
Expected Behaviour 2 :  if we do not use KB option, it does not install security patch (GDR) but SP and CU (example 6 in dbatool doc)
Actual Behaviour 1 : PS should not terminate
Actual Behaviour 2:  Security/GDR patch should be considered if we do not specify KB

Build reference file is upto date and indexes have been updated regularly

Actual Behaviour 1 - if I understood correctly, you want the function to display a warning or throw an error when the patch is already installed. The way I see it, you're defining a level at which you want your instance, and it's not a reason to display a warning or stop execution if the instance already matches this criteria. In a scenario, where multiple servers are being patched, one server that already has the patch installed shouldn't prevent others from getting it. It would be possible to add a warning if the execution is a no-op for a particular server, but it would never be a terminating error. For your purposes you should just capture the command output and see if there were any actions done, something like this:
$results = Update-DbaInstance ...
if ($results.Count -lt 1) { throw "No action performed" }

Actual Behaviour 2 - that is by design and won't be changed, since not everyone wants to install Security/GDR patches. If you want to install those - that's exactly why -KB parameter was implemented.


chore: bdd-nh-1576061308 to 0.0.1

chore: Promote bdd-nh-1576061308 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Cannot load metadata fields

Hi, I am running the commit d0736e50b8d6a31e9df189d74cc3ecb1c55a9cff. The app was not able to load the metadata when creating a new asset. To re-produce the error:

Create a metadata field in metadata schema.
Create a new asset, enter the basic info and client next.

On the Asset Metadata screen, it couldn't load the metadata field.

@kcopsguru : Thank you for bringing this to our attention. We will get back to you on this.

Hi @kcopsguru, this is a problem for projects that use metadata schemas without uploading a controlled list csv file on the metadata schema screen. So, a workaround in the immediate term is to upload an empty file. We expect to improve this over time. If you're interested in contributing a fix, potentially a solution is to provide a fallback mechanism to loading the controlledlist.csv from S3 in ControlledMetadata.tsx. Hope this helps.

@archieco thanks for the info. I only created a metadata field with string type. In this case, do I still need to upload a controlled list csv file?

This is addressed by #120


WSL2 ubuntu unable to connect to external network

Windows build number: 10.0.18363.1440 Windows 10 Enterprise
WSL 2.0 Ubuntu 20.04
try to run sudo apt-get update:
Err:1 http://security.ubuntu.com/ubuntu focal-security InRelease
  Temporary failure resolving 'security.ubuntu.com'
Err:2 http://archive.ubuntu.com/ubuntu focal InRelease
  Temporary failure resolving 'archive.ubuntu.com'
Err:3 http://archive.ubuntu.com/ubuntu focal-updates InRelease
  Temporary failure resolving 'archive.ubuntu.com'

Run: ip a
ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: bond0: <BROADCAST,MULTICAST,MASTER> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 86:42:b9:c8:46:f4 brd ff:ff:ff:ff:ff:ff
3: dummy0: <BROADCAST,NOARP> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 2e:8a:d8:54:a9:97 brd ff:ff:ff:ff:ff:ff
4: sit0@NONE: <NOARP> mtu 1480 qdisc noop state DOWN group default qlen 1000
    link/sit <IP_ADDRESS> brd <IP_ADDRESS>
5: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP group default qlen 1000
    link/ether 00:15:5d:74:a0:5a brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/28 brd <IP_ADDRESS> scope global eth0
       valid_lft forever preferred_lft forever
    inet6 fe80::215:5dff:fe74:a05a/64 scope link
       valid_lft forever preferred_lft forever

ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=0.029 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=0.065 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=64 time=0.068 ms
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2096ms
rtt min/avg/max/mdev = 0.029/0.054/0.068/0.017 ms

ping a computer (IP: <IP_ADDRESS>) in external network:
ping <IP_ADDRESS>

no response

/dupe #5336. Please collect WSL logs and post the backlink to the feedback item in #5336. Also please identify any third-party networking software that has been installed on Windows.


[53043] iOS: Adjusted spacing in what is new page

Replaced StackView with View to have more control over spacing

I do not see big point in doing that. Spacing should be possible to fix in StackView as well. Can you show the difference between View/StackView?

I do not see big point in doing that. Spacing should be possible to fix in StackView as well. Can you show the difference between View/StackView?

I do not see big point in doing that. Spacing should be possible to fix in StackView as well. Can you show the difference between View/StackView?

@pavshr The issue was on small devices that got more scrolling than actually needed. I tried working with the stack view, but this caused the view to get too compressed on bigger devices. There is probably a way to fix this with the stack view as well, but I prefer the constraint approach

I do not see big point in doing that. Spacing should be possible to fix in StackView as well. Can you show the difference between View/StackView?

@pavshr The issue was on small devices that got more scrolling than actually needed. I tried working with the stack view, but this caused the view to get too compressed on bigger devices. There is probably a way to fix this with the stack view as well, but I prefer the constraint approach


Star snippets (Favorit list)

Implemented in https://github.com/snypy/snypy-backend/pull/43, postponed vor 1.1.0 release


SecondFilterValue for RadzenDataFilter

RadzenDataFilter does not have SecondFilterOperator and SecondFilterValue, Is there a plan to add these? like DataGrid?

RadzenDataFilter supports far more complex filter nesting - no need to add hardcoded second filter.


Add zstd compress level support

Fix #1367
This PR in stargz adds support for customizing zstd compression level in ctr-remote. This PR does a similar thing to add the support in nerdctl.
I tested the change locally using the stargz main branch. After the new version of stargz is released (with the change in that PR), it should work.
$ nerdctl image convert --oci --zstdchunked ghcr.io/stargz-containers/python:3.7-esgz level-default
sha256:d649464e45963ead9c0a2ec4b1a124eb7f55a924c3eb687c2975012a7a4f397c
$ nerdctl image convert --oci --zstdchunked --zstdchunked-compression-level=9 ghcr.io/stargz-containers/python:3.7-esgz level-9
sha256:c19da79809fc2e2853cb231f8f62dbd7eaa3a0d49a57d0a86d301acfa03ad902
$ nerdctl images
REPOSITORY                          TAG         IMAGE ID        CREATED           PLATFORM       SIZE         BLOB SIZE
alpine                              3.13        100448e45467    34 hours ago      linux/amd64    5.6 MiB      2.7 MiB
level-9                             latest      c19da79809fc    39 seconds ago    linux/amd64    0.0 B        328.4 MiB
level-default                       latest      d649464e4596    3 minutes ago     linux/amd64    0.0 B        334.3 MiB
ghcr.io/stargz-containers/python    3.7-esgz    6a421075162f    4 minutes ago     linux/amd64    903.7 MiB    333.1 MiB

Signed-off-by: Jin Dong<EMAIL_ADDRESS>

The commit message doesn’t seem to reflect the actual content of the commit

@AkihiroSuda Thanks for review. Updated the doc to mention zstdchunked-record-in as experimental and the commit message.


Widehat

Great work....it will be great to have this working (don't ask me how) ...
\widehat{asdfghjklmnbvcxz}
Suki

closing as duplicate of https://github.com/Khan/KaTeX/issues/407.


fix: correctly copy request body into each retried request

Before this change, the input reader would be consumed by the first
request iteration, and any retries would see an empty reader and would
therefore send an empty request body.
This change updates the retry logic to first read the body into a buffer,
then read from that buffer on each iteration.

Hmm this seems to have been caused by e0bf081, I wonder if we should instead just revert that... let's see what @csmarchbanks thinks.

Considering that the type we pass in is an io.Reader I think we should keep the copy logic since many(most?) reader implementations do not restart from the beginning. Alternatively I guess we could change from io.Reader to bytes.Buffer if we depend on the buffer semantics.


For MS-SSIM result

Thank you for sharing your work, could you share how to train the results of MS-SSIM, and for PSNR results, the results of low bit rate are directly obtained from high bit rate finetuning, or the effect of direct training is better

I think you can finetune the high bit rate weights to get the low bit rate models. This will help the model converge faster.


Unable to resolve module StyleSheetRegistry

Hi,
I get the warning:
Unable to resolve module StyleSheetRegistry

Everything seems to work correctly, just the message in the packager window.

Thanks, I'll push a fix first thing tomorrow!

Alright, fixed version 0.1.1 has been published on npm.
Again, thanks for reporting!


Update Compiler.php

Support proper CSV for arguments list to enable complex argument values, like JSON. This is a prerequisite for https://github.com/myersmediagroup/cms-v3/pull/1519

Ok. Recheck please.


Show all survey results to superusers

As a Superuser, I want to see the results for all the questions on the survey regardless of them being excluded from the report on the report settings.
Acceptance criteria:

New drop down selection "View Report - Internal". This should appear only for superusers and the selection should take to a report page which shows the results to all questions on a survey regardless of them being excluded from the report on the report settings
Change existing drop down selection "View Report" to "View Report - Public"

@TayaLee8  can you review this bad boy

Passing in DEV


Ark should ignore Terminating resources

Currently, Ark tries to backup terminating resources, which often leads to failed backups. this might happen often when you have volatile workloads.

@ncdc - can you assign me ? I'll submit a fix for this one.

@yastij assigned!


Customer user + 1.7 compatibility

Adds support for custom user models, and fixes to work with Django 1.7+

@helenst thanks for this PR - sorry for the long delay. We have the tests working with 1.7 now, but if you want to rebase just your custom user changes against master, I'll take a look at the PR.

Hi @bcail Thanks for taking a look - I'm no longer working on the project I'm using this for so I'm not really in a position to test anything or verify that my PR is still valid. Feel free to take it as is or close as you see fit :smile:

OK, I'll close this PR.


error: uncaughtException: Error parsing address: Invalid address Uint8Array: 29034163ec9102238c

I don't have discord.
Describe the bug
Running lodestar beacon --network prater causes uncaught exceptions parsing peer addresses.
Expected behavior
Correctly parse the peer address or handle the exception.
Steps to Reproduce
Install Lodestar from the latest master version v0.37.0/01e6ffe and run a beacon node on the Prater network; using Node 18.2.0 (stable).
Screenshots
May-18 12:44:53.049[]                 info: Lodestar network=prater, version=v0.37.0/01e6ffe, commit=01e6ffee4e713d5d36d5c95d43ca825994ef470e
May-18 12:44:53.187[]                 info: Connected to LevelDB database name=/home/user/.local/share/lodestar/prater/peerstore
May-18 12:44:53.259[DB]               info: Connected to LevelDB database name=/home/user/.local/share/lodestar/prater/chain-db
May-18 12:44:59.523[]                 info: Initializing beacon state from anchor state slot=0, epoch=0, stateRoot=0x895390e92edc03df7096e9f51e51896e8dbe6e7e838180dadbfd869fdd77a659
May-18 12:45:06.499[API]              info: Started REST api server address=http://<IP_ADDRESS>:9596, namespaces=beacon,config,events,node,validator
May-18 12:45:06.539[NETWORK]          info: PeerId 16Uiu2HAm65fCHnnVnvxpQr7YUzThrWzbPumP4FQCvD4ykJydgKkq, Multiaddrs
May-18 12:45:06.540[]                 warn: Low peer count peers=0
May-18 12:45:06.540[]                 info: Searching peers - peers: 0 - slot: 3030225 (skipped 3030225) - head: 0 0x8c0e…ee87 - finalized: 0x0000…0000:0
May-18 12:45:18.002[]                 info: Searching peers - peers: 0 - slot: 3030226 (skipped 3030226) - head: 0 0x8c0e…ee87 - finalized: 0x0000…0000:0
May-18 12:45:30.000[]                 info: Searching peers - peers: 0 - slot: 3030227 (skipped 3030227) - head: 0 0x8c0e…ee87 - finalized: 0x0000…0000:0
May-18 12:45:36.615[]                error: uncaughtException: Error parsing address: Invalid address Uint8Array: 29034163ec9102238c
Error: Error parsing address: Invalid address Uint8Array: 29034163ec9102238c
    at ParseError (/home/user/lodestar/node_modules/multiaddr/src/codec.js:273:10)
    at bytesToTuples (/home/user/lodestar/node_modules/multiaddr/src/codec.js:194:13)
    at validateBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:249:5)
    at fromBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:239:15)
    at tuplesToBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:139:10)
    at stringToBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:223:10)
    at Object.fromString (/home/user/lodestar/node_modules/multiaddr/src/codec.js:231:10)
    at new Multiaddr (/home/user/lodestar/node_modules/multiaddr/src/index.js:56:26)
    at Socket.UDPTransportService.handleIncoming (/home/user/lodestar/node_modules/@chainsafe/discv5/lib/transport/udp.js:34:31)
    at Socket.emit (node:events:527:28) Error parsing address: Invalid address Uint8Array: 29034163ec9102238c
Error: Error parsing address: Invalid address Uint8Array: 29034163ec9102238c
    at ParseError (/home/user/lodestar/node_modules/multiaddr/src/codec.js:273:10)
    at bytesToTuples (/home/user/lodestar/node_modules/multiaddr/src/codec.js:194:13)
    at validateBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:249:5)
    at fromBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:239:15)
    at tuplesToBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:139:10)
    at stringToBytes (/home/user/lodestar/node_modules/multiaddr/src/codec.js:223:10)
    at Object.fromString (/home/user/lodestar/node_modules/multiaddr/src/codec.js:231:10)
    at new Multiaddr (/home/user/lodestar/node_modules/multiaddr/src/index.js:56:26)
    at Socket.UDPTransportService.handleIncoming (/home/user/lodestar/node_modules/@chainsafe/discv5/lib/transport/udp.js:34:31)
    at Socket.emit (node:events:527:28)

Desktop (please complete the following information):

OS: Archlinux, btw
Version: v0.37.0/01e6ffe
Branch: master
Commit hash: 01e6ffe

Looks like I cannot connect to any peer.
May-18 12:51:42.001[]                 info: Searching peers - peers: 0 - slot: 3030258 (skipped 3030258) - head: 0 0x8c0e…ee87 - finalized: 0x0000…0000:0
May-18 12:51:54.001[]                 info: Searching peers - peers: 0 - slot: 3030259 (skipped 3030259) - head: 0 0x8c0e…ee87 - finalized: 0x0000…0000:0
May-18 12:52:06.000[]                 info: Searching peers - peers: 0 - slot: 3030260 (skipped 3030260) - head: 0 0x8c0e…ee87 - finalized: 0x0000…0000:0
May-18 12:52:06.635[]                error: uncaughtException: Error parsing address: Invalid address Uint8Array: 291275d0d191022328
Error: Error parsing address: Invalid address Uint8Array: 291275d0d191022328

Gonna try to run the latest release instead.

Also affected:

v0.36.0/HEAD//28e2c74c
v0.35.0/HEAD//82219149
v0.34.0 (alpha)

Let me know if you want me to continue testing older versions.

Can you downgrade NPM to v16 for now? (Or force installation of discv5 v0.7.1?) We have a fix for this in discv5 v0.7.1, we just haven't yet pushed it here.

Yes totally, do you document the recommended node version somewhere?

Yea we enforce 16+ in the package.json. We also have a banner item in the root README and a note in the CONTRIBUTING doc mentioning node 16.
But we should also support v18, perhaps even encourage its use (updated v8 engine etc may give some performance benefits).
Definitely a bug we want to fix quickly.

Yes totally, do you document the recommended node version somewhere?

On the main page, the shields badge states Node 16.x but I'll update in the documentation here that we recommend 16.x

Ah, I scrolled over the badges :)


Decouple javascript code from locale data?

Hey guys,
This is a question, not an issue. So, forgive me if I should have used another channel for that.
You currently embed CLDR data into your javascript via this.tokens variables.
Have you considered decoupling it, and have JSON CLDR data loaded by the javascript instead? It would help you to reduce re-work (rebuild) maintenance, and also would give users more flexibility (avoiding stuff like #11).

This has hit master thanks to @radzinzki.


Nested Sorting

#3737
Given this schema:
type Country @entity {
  id: ID!
  capital: City!
  cities: [City!]!
}

type City @entity {
  id: ID!
  name: String
  population: Int
  landmarks: [String!]!
}

It adds fields of child entity to *_orderBy enum:
enum Country_orderBy {
   id
   capital
   cities
+  capital__id
+  capital__name
+  capital__population
}

[x] sort by type Child @entity { name: String }
[x] can't sort by type Child @entity { names: [String] }
[x] can't sort by type Child @entity { secondLevelChild: AnotherEntity }
[x] can't sort by type Child @entity { secondLevelChildren: [AnotherEntity] }
[x] support @derivedFrom
[x] sort by interface Child { name: String }
[ ] sort interface by type Child @entity { name: String }

@lutter

I started reviewing this, but then realized that it also deals with sorting by interface fields, and now I am not certain how it relates to PR https://github.com/graphprotocol/graph-node/pull/4058. Should I continue reviewing this or is this superseded by that PR?

It deals with sorting by interface fields, #4058 adds sorting an interface by children (as it's a bit more complex).

@lutter about the failing test, it works locally but I will try to use exact version on PostgreSQL docker image.

@lutter
Rows of musicians table are sorted by main_band, block_range columns. You can see that b1 is used by two musicians.
Sometimes the unit test is 🟢, but sometimes it's 🔴 .
Do you think that the order of row insertion affects the end result? These two rows are pretty much identical (block_range and main_band).

entity
data

Musician
{"id": "m2", "vid": 1, "name": "Lisa", "bands": ["b1"], "main_band": "b1", "block_range": "[0,)"}

Musician
{"id": "m1", "vid": 2, "name": "John", "bands": ["b1", "b2"], "main_band": "b1", "block_range": "[0,)"}

Musician
{"id": "m3", "vid": 3, "name": "Tom", "bands": ["b1", "b2"], "main_band": "b2", "block_range": "[1,)"}

Musician
{"id": "m4", "vid": 4, "name": "Valerie", "bands": [], "main_band": null, "block_range": "[1,)"}

select 'Musician' as entity, to_jsonb(c.*) as data from (select  c.*
 from "sgd21"."musician" c left join "sgd21"."band" as cc on (cc."id" = c."main_band" and cc."block_range" @> 1)
where c.block_range @> 1
order by cc."id", cc.block_range
limit 100) c

I am not entirely sure what is going on here; for me locally, can_query_with_sorting_by_child_entity_id fails consistently. Is that the test you are talking about? If so, it should say order by cc.name, cc.id, cc.block_range based on the GraphQL for asc.

@lutter merged with the integration tests failing, it seems like the same on master. Unrelated to this PR, and the only change was a rebase.


Support the .python-version file

We should support the .python-version file as a means for specifying what Python version an app is using.
This file format (used by pyenv and others) now also supports specifying Python versions in the X.Y format (and not just X.Y.Z), making it a strong contender over the Heroku-specific runtime.txt file.
Officially supported syntax:

3.12.6
3.12
comments (lines starting with #)
blank lines
pyenv supports specifying multiple versions, however, this buildpack can't, so we'll either want to only use the first version, or error if there are multiple versions specified

Unofficially supported syntax:

python-3.12.6
python-3.12 (though this currently has a bug: https://github.com/pyenv/pyenv/issues/3054)

...though the python- forms are undocumented and likely going to be deprecated soon (https://github.com/pyenv/pyenv/issues/3054#issuecomment-2341316638), so I'm leaning towards not supporting them.
For more background, see:

https://github.com/heroku/heroku-buildpack-python/issues/932
https://github.com/heroku/heroku-buildpack-python/issues/913#issuecomment-1031468659
Internal tracking epic

I found a bug in pyenv's handling of some of the more obscure syntax variants, which I filed as:
https://github.com/pyenv/pyenv/issues/3054
And have opened https://github.com/pyenv/pyenv/pull/3056 to fix.

@megahall Hi! This repository is for the Heroku Python Cloud Native Buildpack (the next generation of buildpacks, currently in preview), rather than the existing (classic) Heroku Python buildpack, which is at:
https://github.com/heroku/heroku-buildpack-python
Support for Poetry and .python-version will be backported to the classic buildpack soon, but if you wanted to try out the Python CNB in the meantime, there is more information here:
https://github.com/heroku/buildpacks

Support for Poetry and .python-version will be backported to the classic buildpack soon

To track that, subscribe to:

https://github.com/heroku/heroku-buildpack-python/issues/932
https://github.com/heroku/heroku-buildpack-python/issues/796

if you wanted to try out the Python CNB in the meantime, there is more information here

I was curious about this, as I’ve been following recent activity, but all the docs I found talk about running tools locally, not using the CNB on heroku.

all the docs I found talk about running tools locally, not using the CNB on heroku.

Yeah it's currently not possible to use CNBs on Heroku (only locally using the Pack CLI), however, that's coming soon:
https://github.com/heroku/roadmap/issues/20

If there is a way to make this clearer in the docs, feedback welcome :-)
(This repo's README has a banner here which links to a CNB overview repo that links to the roadmap item here - but perhaps the latter needs a specific banner call-out?)


Use existing fields and methods before calling custom getattr

Previously, defining __getattr__ would override all existing fields and methods. This changes it to behave like a __getattr__ method defined in python, i.e. the custom method is only called if there isn't a field or method of that name.

Rebased and added a test.

Thank for adding the test!


CSS color enhancements

Add accent-color which will change the color of various native HTML elements from OS-color to specified one. Affects unstyled checkbox, radio, range and progress
Change progress bar color to --color-primary
Add new --color-primary-contrast meant to contrast over primary
Avoid layout shift on clicking .viewed-file-form
Add styles for input[type=file] upload button

Regarding black checkmark here. This color is chosen by the browser for contrast and can not be altered through CSS. I think it's okay for now. In the future we should replace this with our own styles.

Actually, I "fixed" the black checkmark by using slightly darker color which made the browser choose white:

Current --color-accent looks to dull on progress imho:

I'm tempted to just scrap --color-accent and --color-small-accent. It seems like a pointless indirection to me when we can just use the color scale directly.

Actually found a combination that works. I increased saturation on --color-accent. Still looks acceptable imho.

make L-G-T-M work


chore: dummy

Description
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

Federation Router branch is deployed to platta: https://events-graphql-federation-sports-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-events-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-hobbies-pr590.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr590.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://liikunta-pr590.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is failed for https://tapahtumat-pr590.dev.hel.ninja :crying_cat_face::anger::boom::boom:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-hobbies-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-sports-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-events-pr590.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr590.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr590.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr590.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tapahtumat-pr590.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://liikunta-pr590.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://harrastukset-pr590.dev.hel.ninja :laughing::tada::tada::tada:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-hobbies-pr590.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr590.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-events-pr590.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-sports-pr590.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr590.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://liikunta-pr590.dev.hel.ninja :laughing::tada::tada::tada:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr590.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is failed for https://harrastukset-pr590.dev.hel.ninja :crying_cat_face::anger::boom::boom:

TestCafe result is failed for https://tapahtumat-pr590.dev.hel.ninja :crying_cat_face::anger::boom::boom:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr590.dev.hel.ninja :rocket::rocket::rocket:


Imo1986Q1] working proof

First I started proving this for d : \N+ or d : \N, but it is so much easier to do it for d : \Z, so I settled for this, but some of the remnants of the old approach might still be there...

Thanks!


[BUG REPORT] Can't use a user defined variable see example

Hello,
It doesn't look like I can use user defined variables in the cells.
SET @ind_rate = 83.52;
SELECT CarName, FORMAT(Price * @ind_rate, 2) "Rupees" FROM cars

The error message says:
Execute query Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'SELECT CarName, FORMAT(Price * @ind_rate, 2) "Rupees" FROM cars' at line 2

Perhaps maybe change the error handling so that it gives the user a warning that they cannot use variables? Or if there was a way to use variables that would be really great!

I realized that you designed it to hold variables in JSON format.
So it works fine to do it like this:
{
	"ind_rate": 83.52
}

Then the variable is called using a colon instead of a @ symbol.
SELECT CarName, FORMAT(Price * :ind_rate, 2) "Rupees" FROM cars

I guess this is not really a bug, but I think maybe the error message can help people know what to do differently. Thank you for the gif that shows how to use variables.

“If you want to use a variable, please use the bind parameter” message with a link explaining how to use it.

Glad you added a msg to the user so it helps us know what alternatives to use.


[Task]: Align colors in visualizer

Task description
Colors in the visualizer should be aligned with the explorer
Currently confirmed blocks with txs and without tx are green in the dashboard, in the explorer confirmed blocks with txs are blue instead
Requirements
Align
Acceptance criteria
Colors aligned
Creation checklist

[ ] I have assigned this task to the correct people
[X] I have added the most appropriate labels
[X] I have linked the correct milestone and/or project

@muXxer did it


Fix Typo in Readme

Fixes typo in readme
documenation => documentation

Merging to kick-off the V3 branch since this is the easiest to confirm doesn't break anything new despite failing to the Travis CI build.

Merging to kick-off the V3 branch since this is the easiest to confirm doesn't break anything new despite failing to the Travis CI build.


Fixes to trust service

I managed to break the trust service in a previous PR.
The tests didn't catch that, I wrote some real tests to test with VS Code (super fast test, run in 192ms, i.e. using VSC isnt an issue for such tests).
Basically trust was completely broken, nothing would get trusted.

Codecov Report

Merging #14352 into main will decrease coverage by 0.48%.
The diff coverage is 51.62%.

@@            Coverage Diff             @@
##             main   #14352      +/-   ##
==========================================
- Coverage   59.91%   59.43%   -0.49%
==========================================
  Files         709      716       +7
  Lines       39334    40009     +675
  Branches     5698     5799     +101
==========================================
+ Hits        23567    23779     +212
- Misses      14529    14970     +441
- Partials     1238     1260      +22

Impacted Files
Coverage Δ

src/client/common/application/notebook.ts
10.52% <0.00%> (-0.39%)
:arrow_down:

src/client/common/application/types.ts
100.00% <ø> (ø)

src/client/common/process/baseDaemon.ts
21.76% <0.00%> (-0.53%)
:arrow_down:

src/client/datascience/baseJupyterSession.ts
56.30% <0.00%> (-1.04%)
:arrow_down:

src/client/datascience/jupyter/jupyterNotebook.ts
4.20% <0.00%> (-0.02%)
:arrow_down:

src/client/datascience/jupyter/kernels/types.ts
25.00% <ø> (ø)

...ascience/jupyter/liveshare/guestJupyterNotebook.ts
8.96% <0.00%> (-0.33%)
:arrow_down:

src/client/datascience/kernel-launcher/types.ts
23.07% <ø> (ø)

src/client/datascience/notebook/notebookEditor.ts
5.71% <0.00%> (ø)

...science/notebookStorage/notebookStorageProvider.ts
50.74% <0.00%> (ø)

... and 38 more

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 a24d64a...f3abf2a. Read the comment docs.


Fix for flashing on window creation

Small PR to fix the flashing issue left over from #1037
This should also fix any flashing that might happen when creating a window minimized or maximized.
Also updated setting position and size to use Into

There is some edge case that this solution cant handle, I will need to find another solution..


Suggest claim remedies/solutions

Description

As a user, I want to understand what the available remedies are for my claim

Scope of Work
Acceptance Criteria
Story Points

100

Priority

High

Risk

High

Original Comment: "Remboursement, Dommages, Resolution, Resiliation, etc."

Clarification is required for this story. Which recourse should we recommend to users? When should we recommend each recourse? When do we (if ever) recommend legal action? Do we simply create a decision tree (flow chart?) based on the law and match facts from their case?

We don't recommend. We suggest the possible recourses.
"other people in similar situations were a) awarded on average a 43% refund on the retail price and b) allowed to cancel their monthly subscription"
"other people in similar situations were a) allowed to rescind the original lease agreement"
"other people in similar situation were a) awarded on average 2000$ to compensate for medical fees b) 1000$ to compensate for pain and suffering and c) 500$ as punitive damages.

Based off the example you provided, this sounds like a generalization of user story #11, with the addition of cancellation of the lease. Also, it sounds like all recourses will be based on the outcome of a hearing. Does this mean we will not "suggest" non-hearing venues for resolving their issues?

User story 17 is different from user story 11 since the latter focuses on the amount of damages when damages are owed, whereas the former refers to the whole scope of remedies available (injunctive relief, nullity of contract, cancellation of contract, damages, etc.). These are remedies that going to trial will yield.
I can't find a user story that suggests "online dispute resolution, negotiation, mediation, small claims court, régie du logement, demand letter, lawyer, etc.". This one is meant to present all the options available besides going to trial.
We could enlarge user story 17 to include those notions.

These options will perhaps each require their own user story. I am assuming that interactions with the system will be different for each remedy. Can we have a list of remedies that our system should support? from there, we can split this story into several smaller ones.

These options will perhaps each require their own user story. I am assuming that interactions with the system will be different for each remedy. Can we have a list of remedies that our system should support? from there, we can split this story into several smaller ones.

The remedies in court or the options outside of court?

Remedies that should fall within the scope of our system. With an indication of whether it is in court or not.


Remove not anymore needed exclusions which breaks PCT test from 2.489, test with mvn clean install -Djenkins.version=2.489

Signed-off-by: Olivier Lamy<EMAIL_ADDRESS>
Testing done

Submitter checklist

[ ] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[ ] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

No releases on Friday, I will release on Monday


Websocket host address

Hello is there any other way to know the address of a connected Websocket instance rather than making
private var url: NSURL
public?
I need to compare objects in order to add/remove them from an array.
Thanks

hmmm, yeah currently there won't be a way to compare them. I don't think I every thought there would be a need for that, but in your use case I can see how that might be problematic. I think the best course of action would be to add a read only/computed property that returned the url so it couldn't be modified.

Added a new property, currentURL


Update port value for startupProbe, livenessProbe, readinessProbe in meilisearc…

…h.yaml
sur les startupProbe, livenessProbe, readinessProbe, j'ai du mettre :  port: 7700
sinon j'avais une erreur
'Startup probe errored: strconv.Atoi: parsing "http": invalid syntax' et le service reste innaccessible
Pull Request
Related issue
Fixes #<issue_number>
What does this PR do?

...

PR checklist
Please check if your PR fulfills the following requirements:

[ ] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[ ] Have you read the contributing guidelines?
[ ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

@meilisearch sync-manifest

bors merge


Upgrade winston to resolve errors logging React components

The incredible @indexzero cut a new release of winston early to help us resolve our logging errors https://github.com/winstonjs/winston/commit/b44062612f63211d047c67050fef5b7624682fb6 like a half hour after I asked in gitter.

Thanks @doug-wade and @indexzero!


Implementation of expo task manager for expo managed projects

What is the need and use case of this feature?
Im always frustrated when my expo project doesn't work with this library 😅
Describe the ideal solution
As far as i know you can actually use this package in an expo managed project so you don't need to use a bare one. I successfully installed and build this package into my eas dev app for my expo managed project. It seems like the only thing that isn't working is the headless task registration which i think can be fixed by using expos TaskManager package (https://docs.expo.dev/versions/latest/sdk/task-manager/)
How I can Help
I would be more then open to at least try to create a PR that fixes the problem i described even though i am not the best in working with react native 😅
Btw i am using the current stable 2.1.3 and not 2.2.0-rc3

@h110m we don't currently have any code contributors that work in expo. It would be great if you wanted to take a crack at identifying some of the underlying expo issues you're seeing. I'm happy to provide as much guidance as I'm able. Take a look at the docs here to see how to get started working on the native side of things:
https://github.com/doublesymmetry/react-native-track-player/tree/main/example

@jspizziri Thanks a lot i will definetly take a look at it and at least give it a try and btw thanks a lot for making this lib ^-^

@h110m I appreciate the thanks, but I didn't create the lib. I just work on it.

I'm going to close this until an interested party opens a PR.


Add bitwise_xor and bitwise_not Spark functions

Add bitwise_xor  and bitwise_not.
These two functions have different return types comparing to their counterparts
in Presto.

@mbasmanova , could you help review? Thanks.


Stats

Python client to collect stats

Add a section in README on how to build this and use this

@stagraqubole is this related to #22 ? Can you provide more info on requirements and design ?

No, this isnt related to that. This is creates a simple python thrift client that can be run locally on any machine running bookkeeper and get the cache status for that machine


Sutherland viscosity model

Related to Trello card #51 (https://trello.com/c/1sp9umbV)

@whoburg Ready for review

All of these commits are included in PR #45


fix: redirect docker output and run with --init

This PR redirects the output of docker, and allows the user to terminate the docker process with ctrl+c by running docker with --init option ( ref: https://stackoverflow.com/questions/60733727/unable-to-stop-my-docker-container-with-ctrl-c )

The base branch requires all commits to be signed.
Please sign your commits.


An issue in Built-in modules docs

Description
Hi.
The bash script explained in the following to map the host path to /modules directory is wrong.
https://github.com/WasmEdge/docs/blob/80b2a5fe7753e1e727b390d7d304d48e3b34133e/docs/develop/javascript/modules.md?plain=1#L20
According to the wasmedge --help, it's correct to write wasmedge --dir /modules:/host/os/path
Screenshots

Additional information
No response

You're right. Thank you for your issue


Touch Callback Line Touch Response Expected Result

Hi, thanks for an amazing package. I want to make sure that the line touch response of x axis (touchResponse?.lineBarSpots?.first.x) is expected to return an index of bar tapped whenever it is just we tap on certain bar point? Because when I just drag and remove my finger on a certain point it will return null.
Here is the recording bug:
https://github.com/imaNNeo/fl_chart/assets/110516419/c8de7471-5f87-411d-8e1c-4411335376c1
I deep down into FlTouchEvent class and find that the event is different (removed touch is FlTapUpEvent and removed drag is FlPanCancelEvent) but still get confused about the return index from those events regarding the UX that maybe for many users are expect the same result (returning null whenever we remove our finger over the chart)

Fl_chart version  0.63.0
Flutter SDK version 3.10.0

Thank you 🙌🏻

Set the touchSpotThreshold on the LineTouchData object to a large number (1000 for example), then there will always be a nearest point.


Delete Task endpoint

closes #738

Thanks @ychiucco!
One main comment about _can_access_task: we have a bunch of similar functions in _aux_functions.py. Could you move the new one there, and transform it into _get_task_check_owner (which should include both the ownership check and also get the task from the DB - see any of the other functions in that module). This function could raise a 403 or 404.

Also, can you please update the CHANGELOG?

One main comment about _can_access_task: we have a bunch of similar functions in _aux_functions.py. Could you move the new one there, and transform it into _get_task_check_owner (which should include both the ownership check and also get the task from the DB - see any of the other functions in that module). This function could raise a 403 or 404.

And then the corresponding test would go into test_unit_aux_functions.py.

Recap:

[ ] Move _can_access_task into _aux_functions._get_task_check_owner, and also include the handling of 404.
[ ] Write a unit test for this new function, in test_unit_aux_functions.py
[ ] Simplify the current test_delete_task, since it just needs to test the specific logic which is written in the endpoint function: it does not need to test the 404 and 403 cases, which are already covered in the unit test, but it does need to test the 422 case.
[ ] Update CHANGELOG


Remove Azure/login@v2 from the flowey bootstrap template

This change removes the Azure/login@v2 action from the flowey bootstrap template which is no longer needed. This won't fully solve all of the current CI issues, but it should get us a lot further.

Going to put this into a single PR that fixes multiple issues.


[WIP] Refactor manipulator drag

This WIP PR refactors the manipulator drag:

improves the visual for the manipulator
factors out an separate model for translator
factors out a separate model for rotator
introduces a RotatorDragManager to encapsulate the rotator drag logic
introduces a TranslatorDragManager to encapsulate the translator drag logic
fixes bugs in the rotator drag calculation
adds tests
restricts axes to [[1, 0, 0], [0, 1, 0], [0, 0, 1]]

https://user-images.githubusercontent.com/1120672/189311698-e9ca348d-aec2-4ca1-b5b4-792763fe361f.mov

Codecov Report

Merging #60 (a911c9e) into main (5755099) will decrease coverage by 4.74%.
The diff coverage is 35.31%.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   47.72%   42.98%   -4.75%
==========================================
  Files          23       31       +8
  Lines         989     1575     +586
==========================================
+ Hits          472      677     +205
- Misses        517      898     +381

Impacted Files
Coverage Δ

...i_threedee/manipulators/_manipulator_components.py
0.00% <0.00%> (ø)

...apari_threedee/manipulators/_manipulator_visual.py
0.00% <0.00%> (ø)

...rc/napari_threedee/manipulators/new_manipulator.py
0.00% <0.00%> (ø)

src/napari_threedee/utils/selection_utils.py
11.76% <12.50%> (+0.22%)
:arrow_up:

.../napari_threedee/manipulators/manipulator_utils.py
27.90% <89.74%> (+17.45%)
:arrow_up:

src/napari_threedee/manipulators/_drag_manager.py
100.00% <100.00%> (ø)

..._threedee/manipulators/_tests/test_drag_manager.py
100.00% <100.00%> (ø)

...edee/manipulators/_tests/test_manipulator_utils.py
100.00% <100.00%> (ø)

src/napari_threedee/utils/_tests/test_geometry.py
100.00% <100.00%> (ø)

src/napari_threedee/utils/geometry.py
100.00% <100.00%> (ø)

... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


UI Issue: Long URL in User Profile Collides with Contribution Graph

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
Hi GitHub Team,
I have encountered a UI issue on the user profile page where a long URL in the "Links" section of a user's profile is causing the content to overflow and collide with the contribution graph. The URL stretches too far horizontally, disrupting the layout of the page and making it difficult to view both the link and the contribution graph properly.
Screenshot:

Thank you for your time and attention! I hope this issue can be addressed, and I look forward to your response.
Best regards,
Ashif Hassan
ashif@1996
What changes are you suggesting?
The long URL should be truncated or wrapped in a way that prevents it from colliding with the contribution graph.
This issue appears only when the provided URL is excessively long.
A better approach could be using ellipses (...) to indicate a long link, ensuring a clean and user-friendly layout without disruption.
Additional information
Will there be any recognition or reward for reporting this UI issue? I'm particularly interested in learning whether reporting UI/UX bugs like this could help in any way that benefits my career, such as gaining recognition on GitHub or adding value to my professional resume.

@ashif1996 Thank you for opening an issue! While our Docs team focuses primarily on issues regarding documentation,
our support team is equipped to address this issue regarding the UI bug. Please reach out to them, and they'll be happy to assist ✨

I'm particularly interested in learning whether reporting UI/UX bugs like this could help in any way that benefits my career, such as gaining recognition on GitHub or adding value to my professional resume.
Our support team would have greater insight on this, and you are more than welcome to ask them 💛


Error upgrading from 1-beta1 to nightly: InternalServerError: wrong element type

EdgeDB Version: 1-beta1 -> 1.0-beta.1+g2eee1afe4.d20210301

I tried upgrading my docker installation to nightly and got an error:
edgedb error: failed to restore "potato_bot": restoring database "edgedb": ERROR: InternalServerError: wrong element type
  Hint: This is most likely a bug in EdgeDB. Please consider opening an issue ticket at https://github.com/edgedb/edgedb/issues
/new?template=bug_report.md
  Server traceback:
      edb.errors.InternalServerError: wrong element type

Now I can't start my database:
$ edgedb server status
┌────────────┬───────┬─────────┬─────────────┐
│    Name    │ Port  │ Version │   Status    │
├────────────┼───────┼─────────┼─────────────┤
│ potato_bot │ 10701 │ 1-beta1 │ not running │
└────────────┴───────┴─────────┴─────────────┘
$ edgedb server start potato_bot
edgedb_potato_bot
$ edgedb server logs potato_bot
INFO 1 2021-03-07 20:44:22,264 edb.server: EdgeDB server (1.0-beta.1+g2eee1afe4.d20210301) starting.
INFO 1 2021-03-07 20:44:26,389 edb.server: Serving on <IP_ADDRESS>:10701
INFO 1 2021-03-07 20:44:26,389 edb.server: Serving on /var/lib/edgedb/data/run/.s.EDGEDB.10701
INFO 1 2021-03-07 20:44:26,389 edb.server: Serving admin on /var/lib/edgedb/data/run/.s.EDGEDB.admin.10701
INFO 1 2021-03-10 11:05:51,735 edb.server: Shutting down.
INFO 1 2021-03-10 11:10:49,731 edb.server: EdgeDB server (1.0-beta.1+g2eee1afe4.d20210301) starting.
---- Exception occurred: database instance incompatible with this version of EdgeDB ----

1. edb.errors.ConfigurationError: database instance incompatible with this version of EdgeDB

### a lot of tracebacks ###

---- Details ----
Details: The database instance was initialized with EdgeDB format version<PHONE_NUMBER>00, but this version of the server expects
format version<PHONE_NUMBER>00
Hint: You need to recreate the instance and upgrade using dump/restore.

edgedb.upgrade.potato_bot.dump/ was created, I assume this could be useful.
edgedb.dump.txt (.txt because github doesn't allow .dump extension)

@Fogapod, thanks for the dump, we'll look into it.
Meanwhile, you can use edgedb server revert to revert a failed upgrade.
@tailhook, we need to have the suggestion to run revert be part of the upgrade crtash message.


MauiHybridWebView causes Android app to crash with R8 enabled - Any fix to make it R8 compatible? Bug project included to demonstrate.

Bug Description
Apps featuring MauiHybridWebView always run perfectly on the first app start (whether R8 is enabled or disabled).
However, if Android stops the app in the background (ie. OnStop() occurs) and restarts the app in the background, it then always crashes when R8 is enabled. When R8 is disabled, it restarts smoothly.
How to Trigger Android App Restart
To make Android stop and restart the app and demonstrate this:

As per this thread open Android Settings and in Developer Options, change Background Process Limit setting to 1 or zero.
Then when Maui App has started, switch to another 1-3 open apps (WhatsApp, Chrome, anything else) before coming back to the Maui app (which forces close and restart of it).

Using this method, every time, when the app is built with R8 in Release mode and Android tries to restart it, it crashes.
Bug Project
https://github.com/jonmdev/R8CrashBug
The Bug Project to demonstrate is just a default Maui app with MauiHybridWebView added. A simple index.html is copied and pasted in to the Resources folder. Main.xaml.cs is changed to:
namespace R8CrashBug {
    public partial class App : Application {
        public App() {

            ContentPage mainPage = new();
            this.MainPage = mainPage;

            AbsoluteLayout absRoot = new();
            mainPage.Content = absRoot;

            HybridWebView.HybridWebView hybridWebView = new();
            hybridWebView.HybridAssetRoot = "hybridview";
            hybridWebView.MainFile = "index.html";
            hybridWebView.JSInvokeTarget = this;

            absRoot.Add(hybridWebView);
        }
    }
}

No other changes are made.
Steps To Reproduce

Open Developer Options in Android Phone and set Background Process Limit to 0-1 range.
Build Bug Project in Release to Android device with R8 Enabled (as it is by default).
Press ||| button on Android bottom tray and navigate to a few other open apps sequentially (this will trigger Android to run onStop() and stop the Maui App process in the background).
Press ||| button on Android bottom tray and navigate back to the Maui App.
With R8 enabled, it will always then briefly show the splash icon and crash.
If you repeat the above steps with R8 disabled, no crash occurs - it will instead restart the app smoothly as expected.

I also note the plain default Maui app (without MauiHybridWebView) does not crash with R8 enabled or disabled. It only crashes with R8 enabled once MauiHybridWebView has been added. Thus this is specifically provoked by the addition of MauiHybridWebView.
Outlook
MauiHybridWebView is an absolute game changer for me which has solved some major problems I otherwise had no solution for. I absolutely love the project and appreciate your work on it. But we cannot have it crashing like this in real projects for rlease and R8 is necessary for modern Android release.
Any help or outlook on a fix would be very, very appreciated.
Thanks again @Eilon for your great work on the project and any solution you can offer.

For reference, I have investigated further after looking more into how MauiHybridWebView works. I saw MauiHybridWebView is essentially just inherited from WebView, so I tested, and the problem is triggered by WebView also. This suggests the problem must be fixed in WebView more fundamentally than MauiHybridWebView.
WebView bug report then posted here: https://github.com/dotnet/maui/issues/23027


Block

@mrkn Adding support for block canvas, with tests per issue #19.
Reference testcase .txt files were directly borrowed from UnicodePlots.jl repo.

@nanobowers Thank you very much!

@nanobowers Thank you very much!


In 2.3.0-alpha: occasionally after a refresh, couchEditor will append different views (tree| form) together

In 2.3.0-alpha (The branch with ol5 and mdc). The couchEditor occasionally will merge different view (tree view or editor view) together after a refreshing.
To reproduced the issue:

Click the button ADD RELATION
Select a candidate doc from the search dialog
Often after the refreshing, tree view and editor view will both showing in side panel
Potential cause:
Some issue in couchShow service relating to the mdc class creation

The fix has been committed to 2.3.0-alpha. We will further test it. But since tab-style of couchEditor is expected to be introduce in 2.3.0-alpha, instead of the jquery-ui accordion,
			window.setTimeout(function(){ 				$elem.removeAttr('style'); 			},500);
seems no longer necessary.

I've removed this timeout and the width and height style changes from the n2.schema.js.
This change has been pushed into both the 2.3.0-alpha and the mdc_integration branches.


Question: remote reflog on Keybase Git repositories

I messed up.
I use pass as my password manager, syncing between computers with Keybase git repositories. Long story short, the password for an encrypted backup is, if anywhere, in a no-longer-referenced commit on a Keybase git remote. I suspect this is a result of using two remotes and configuring git to rebase instead of merging when pulling.
In a local git repo, one can see such commits with git reflog. My limited understanding of KBFS suggests that it should be possible to do the same with a Keybase git remote. Alternatively, is there a way I can fetch all objects in a repository?

@NieDzejkob you can always do a full git clone --mirror keybase://... to get all the commits in the repo.  Or you can inspect it manually if you have the KBFS explorer/finder integration enabled, by going to /keybase/team/teamname/.kbfs_git/reponame (for a given teamname and reponame) to see the bare git repo.

@strib Thanks! This was really helpful. Unfortunately, I didn't find anything.


Using System.TimeZoneInfo will crash in simulator (as will other native calls)

Godot version
3.4.2 (Mono)
System information
OSX 10.15 (Catalina), XCode 11.6
Issue description
This is very similar to #40767 - in fact I can trigger it with the same test project. Making any native calls seem to crash with an EntryPointNotFoundException. This seems to be worse than the previous issue which only crashed on device builds, but now crashes on the simulator too.
Any native call seems to trigger this (including the other existing ones from #40757 and #40633).
As an example, you can use can call dateTime.ToUniversalTime(), which will make a call to System.TimeZoneInfo.xamarin_timezone_get_data and crash with the following exception:
System.EntryPointNotFoundException: xamarin_timezone_get_data assembly:<unknown assembly> type:<unknown type> member:(null)
  at (wrapper managed-to-native) System.TimeZoneInfo.xamarin_timezone_get_data(string,uint&)
  at System.TimeZoneInfo.GetMonoTouchData (System.String name, System.Boolean throw_on_error) [0x00002] in <11afb9645ddb4e8fae4163490ca47a50>:0
  at System.TimeZoneInfo.CreateLocal () [0x00000] in <11afb9645ddb4e8fae4163490ca47a50>:0
  at System.TimeZoneInfo.get_Local () [0x00009] in <11afb9645ddb4e8fae4163490ca47a50>:0
  at System.TimeZoneInfo.ConvertTimeToUtc (System.DateTime dateTime, System.TimeZoneInfoOptions flags) [0x00000] in <11afb9645ddb4e8fae4163490ca47a50>:0
  at System.DateTime.ToUniversalTime () [0x00000] in <11afb9645ddb4e8fae4163490ca47a50>:0

I can fix this with linker flags as suggested in https://github.com/godotengine/godot/issues/40757#issuecomment-841852062, i.e. by passing in -Wl,-u,_xamarin_timezone_get_data,-u,_xamarin_timezone_get_local_name but I was under the impression that was fixed by https://github.com/godotengine/godot/pull/49248. Is this a regression or have I misunderstood the fix?
It's going to be very fiddly if I need to manually test the project and find all the native calls manually.
Steps to reproduce
new DateTime().ToUniversalTime();

Minimal reproduction project
Test-MonoDateIOS.zip

I can fix this with linker flags as suggested in #40757 (comment), i.e. by passing in -Wl,-u,_xamarin_timezone_get_data,-u,_xamarin_timezone_get_local_name but I was under the impression that was fixed by #49248. Is this a regression or have I misunderstood the fix?

That should have been fixed, yes. Godot will create a .cpp file as part of your Xcode project (I think it's called dummy.cpp). Can you upload this file here?

That should have been fixed, yes. Godot will create a .cpp file as part of your Xcode project (I think it's called dummy.cpp). Can you upload this file here?

dummy.cpp.txt
The Xamarin externs are in there. Some appear to be entered twice (e.g. Stat2), but I'm assuming that's not the issue.
Are they supposed to be wrapped by #if !TARGET_OS_SIMULATOR? I'm not sure what the difference is between now and 3.2 when they ran correctly on the simulator and only stripped for a device build.

Can you reproduce this only in the simulator? Or does the same happen in a device?

Just tried on a device (via an ad-hoc distribution profile) and it has the same missing entry point exception. If I go and manually specify the linker flags it still doesn't work on the device, though it then works fine in the simulator.
If I remove #if !TARGET_OS_SIMULATOR from dummy.cpp then it works on the simulator without specifying linker flags. This also does not fix it for a real device with an exported archive though.

We're getting a TestFlight-only crash on actual iOS devices using Double.ToString because I think it's invoking System.Globalization.CultureInfo under the hood, even without specifying an IFormatProvider. It uses the current culture by default. Specifying the culture explicitly as CultureInfo.InvariantCulture still crashes, the same as leaving it implicit.
It does not happen in the simulator, nor when building to an actual iOS device via USB straight from Xcode after exporting from Godot 3.5.1. I believe this issue is the same root cause. The same exact type of crash scenario also occurred using any kind of localized date/time code that invokes System.TimeZoneInfo under the hood.

So the issue is still there. It works flawlessly when the iOS binary is created in XCode, even in the Release build, however archiving the project (i.e. generating the IPA or uploading to the AppStore) performs an additional strip operation that removes libmono-native.xcframework or something else entirely.
Fastest way to avoid the problem is to find the Strip Linked Product option in XCode settings and set it to No:

Another way is to set stripping mode to debug symbols only:

I don't have time and mood to trace why the Mono part is stripped but it is reproducible up to Godot 3.6 beta.

Note that this issue is also raised with any DateTime string formatting
System.EntryPointNotFoundException: xamarin_timezone_get_data
I'm also seeing a similar issue for currency formatting e.g.  where float  v.ToString ("C") will cause an exception in iOS
The work-around by @Nomad1 works in both cases