Receiver search box eats typed characters

What did you do?
Typing fast in the Receiver: search box in a setup with many receivers.
What did you expect to see?
Typed characters appearing in the search box
What did you see instead? Under which circumstances?
Some typed characters don't show up.
Environment

System information:
Linux 4.4.10+soundcloud x86_64

Alertmanager version:
alertmanager, version 0.15.2 (branch: HEAD, revision: d19fae3bae451940b8470abb680cfdd59bfa7cfa)
build user:       root@3101e5b68a55
build date:       20180814-10:53:39
go version:       go1.10.3

I totally understand that the autocomplete will be sluggish in a large setup. However, if I know what to type, I should just be able to type, ignoring autocomplete. I cannot do this. Instead, I find myself editing the URL directly because I can just type there.
@mxinden @w0rm as the Elm experts, perhaps you have an idea why the autocomplete is destructively interacting with the stream of typed characters?

I've attempted to improve the situation with https://github.com/prometheus/alertmanager/pull/1664
The issue is, each time a key is pressed, multiple asynchronous functions fire, such as the value of the text box. Because of the inefficiency/slowness of the function responsible for filtering the receiver search box, when you type "p-r", the code executes based on "p", and renders the values based on that.
The PR above is a "step in the right direction", but debouncing the input with i.e. 200ms delay between typing and doing the filtering would be even better.

I've added a debouncer in the above pr as well.

Can this be closed, and a new issue "improve receiver box performance" be added? We have several ideas how to do this, and it would probably be good to move on to a new issue.

I'll close this. I'm not even sure if we need an issue for the "performance improvement" thing just for the receiver box. Feel free to do so if it helps tracking your own work.


Fix lint - [WIP]

The linting warnings are annoying, so this is an attempt at fixing them.

@leecalcote @girishranganathan please look at this. this does fix the lint warnings, but I'm not sure about all the variable name changes as I still don't have a sufficient understanding of the codebase.
So please let me know if this looks ok to you.

I can't wait for your first PR to land. :)
@antweiss one small item is the project's adherence to a Developer's Certificate of Origin (DCO) in which contributors are asked to sign each of their commits.
Here are the specific steps to take to sign your commits in this branch (PR) - https://github.com/layer5io/meshery-istio/pull/30/checks?check_run_id=251319869
Here are some tips (a git alias and a VS Code setting) that you might consider so that signing off on commits happens implicitly - https://github.com/layer5io/meshery/blob/master/CONTRIBUTING.md#developer-certificate-of-origin

Good deal (the failed Docker Build is a known issue; one that is fixed upon merge).


Build Task "Power Platform Import Solution": New setting "Activate Plugin" suddenly available and unchecked per default

Hi,
the following setting has been added recently. I couldn´t find a proper announcement in the release notes. I´ve only recognized it after the deployment when all the plugin steps were inactive. So the way it is handled at the moment is a bit unfortunate.
Furthermore, the old setting "Activate processes (workflows) after import" seems to be redunant now.

Thanks!

@zimflo of you're needing to renable all your plugins, this may help https://dotnetdust.blogspot.com/2022/07/enabling-or-disabling-all-plugin-steps.html?m=1
Also I'm assuming it's fairly obvious from a continuity perspective, that plugins should be enabled by default

This is a breaking change for anyone that is not specifying a version number run the yaml as well.  Prior to this, plugins we're always enabled by default.  Now running the pipeline will disable all plugin steps.  A very very very major issue for people that depend on plugins for data integrity or security

@zimflo @daryllabar issue is understood:
The older PP-BT's task input "PublishWorkflows" was setting what the Dataverse ServiceClient's .ImportSolutionAsync() calls activatePlugins. When we transposed from the Powershell to the TS/pac CLI implementation, we mistakenly added a new input, instead of sticking with the old name.
IIRC, the "publishWorkflow" stems from what the older CrmServiceClient used to call in its ImportSolutionRequest, but it had the same semantic intention: activate plugins
Here's the fix proposal:

To not add an unnecessary breaking change, we will mark the "Activate Workflows (aka. PublishWorkflows)" as deprecated in the ImportSolution's input param definition and point to the new input param "Activate Plugin" since it also correctly reflects the actual semantics
the "Activate Plugin" will be updated to have this param true/on by default
the task's internal logic will query both the old and new name, and if either is true, it will result in activatePlugin=true to be sent along w/ the import request
minor: we'll change the ordering in the GUI to have the now-deprecated Activate Process show last

@zimflo @daryllabar @vladnega
Our fixes to pac CLI and the import solution tasks have been merged. We're now kicking off our official build for an updated v >= 1.0.83 that has the fixes. ETA ~3:30 PDT today
Rel notes:

'Import Solution' task:

'ActivatePlugins' task parameter is on by default; deprecate param 'PublishWorkflows' but consider both for plugin activation (#131)
fix forced 'PublishChanges' and its potential http timeout on PublishAllCustomizations (#129)
introduces a new "PublishCustomizationChanges" task parameter to also publish customizations after successful import

Fix has been deployed to VS Marketplace as 1.0,84

@zimflo @daryllabar @vladnega
Right now, we've rolled back to the PS based implementation 1.0.85 (see #144).
The issue has been addressed in both pac CLI and the pp-bt TS layer, fixed and verified, changes have been backported to main.
Resolving


wham exited with error code 2

Hi, thanks for the good implementation. However, when  I run the example.py,  I encountered the error:
"wham exited with error code 2"

Hey,
does the issue still exists? I might be able to help if you provide a bit more details: Can you show your implementation at what software you are using for wham?

Hey, does the issue still exists? I might be able to help if you provide a bit more details: Can you show your implementation at what software you are using for wham?

Thanks for your reply.
I installed wham-2d according to the link "http://membrane.urmc.rochester.edu/sites/default/files/wham/doc.html" in line 69 of adaptiveumbrella/adaptiveumbrella/wham2d.py
cd wham-2d make clean make
then run the example.py
`#!/usr/bin/env python3
import os
from copy import deepcopy
import matplotlib.pyplot as plt
import numpy as np
import sys
from adaptiveumbrella.wham2d import WHAM2DRunner
sys.path.append('..')
class MyUmbrellaRunner(WHAM2DRunner):
def __init__(self):
    WHAM2DRunner.__init__(self)

cum_frames = [0]

def after_run_hook(self):
    filename = "tmp/pmf_{}.pdf".format("%02d" % self.num_iterations)
    print("Writing new pmf to {}".format(filename))
    pmf_to_plot = deepcopy(self.pmf)
    pmf_to_plot[pmf_to_plot < 0] = None
    pmf_to_plot[self.sample_list == 0] = None
    pmf_to_plot = pmf_to_plot.T

    self.cum_frames.append(len(self.sample_list[self.sample_list > 0]))

    fig, (ax0, ax1) = plt.subplots(ncols=2)
    im = ax0.imshow(pmf_to_plot, origin='lower', cmap='jet')
    cb = fig.colorbar(im, ax=ax0, orientation='horizontal', pad=0.15)
    cb.set_label("kJ/mol")

    ax1.plot(self.cum_frames, linewidth=0.5, marker="o", color='black')
    ax1.set_xlabel("Cycles")
    ax1.set_ylabel("Number of umbrella Windows")

    # ticks = [(x,x) for x in [-3, -2, -1, 0, 1, 2, 3]]
    # tick_positions = [ self._get_index_for_lambdas(x)[0] for x in ticks ]
    # tick_labels = [ str(x[0]) for x in ticks ]
    # ax0.set_yticks(tick_positions)
    # ax0.set_yticklabels(tick_labels)
    # ax0.set_xticks(tick_positions)
    # ax0.set_xticklabels(tick_labels)
    ax0.set_ylabel("$\phi$")
    ax0.set_xlabel("$\psi$")

    fig.subplots_adjust(wspace=.5)

    plt.savefig(filename, bbox_inches='tight', dpi=200)
    os.system("cp {} {}".format(filename, "tmp/pmf_current.pdf"))

def simulate_frames(self, lambdas, frames):
    print("{} new simulations:".format(len(lambdas)))
    counter = 0

    if not os.path.exists("tmp"):
        os.mkdir('tmp')

    threads = []
    for f in lambdas:
        counter += 1
        print("{}) Simulate lambda1={}, lambda2={}".format(counter, *f))
        command = "bash data/sim.sh {} {} 2>&1 > tmp/run.log".format(*f)
        # print("Running {}".format(command))
        os.system(command)

runner = MyUmbrellaRunner()
runner.WHAM_EXEC = "/home/user/Documents/gs/wham/wham-2d/wham-2d"
runner.whamconfig = {
'Px': 'pi',
'hist_min_x': -3,
'hist_max_x': 3,
'num_bins_x': 100,
'Py': 'pi',
'hist_min_y': -3,
'hist_max_y': 3,
'num_bins_y': 100,
'tolerance': 0.1,
'fc_x': 100,
'fc_y': 100,
'temperature': None,
'mask': None
}
runner.cvs = np.array([
(-3, 3, 0.2),
(-3, 3, 0.2),
])
runner.cvs_init = (0, 0)
runner.E_min = 5
runner.E_max = 100
runner.E_incr = 10
runner.max_iterations = 1
runner.run()`
It returned the error:
`/home/user/anaconda3/envs/pytorch1.10/bin/python /home/user/Documents/gs/adaptiveumbrella/examples/example.py
np.shape(windows) (30,)
np.shape(windows) (30,)
Energy: 5
Running simulations
9 new simulations:
1) Simulate lambda1=-0.2, lambda2=-0.2
2) Simulate lambda1=-0.2, lambda2=0.0
3) Simulate lambda1=-0.2, lambda2=0.2
4) Simulate lambda1=0.0, lambda2=-0.2
5) Simulate lambda1=0.0, lambda2=0.0
6) Simulate lambda1=0.0, lambda2=0.2
7) Simulate lambda1=0.2, lambda2=-0.2
8) Simulate lambda1=0.2, lambda2=0.0
9) Simulate lambda1=0.2, lambda2=0.2
Calculating new PMF
wham exited with error code 2

Process finished with exit code 1`

the simulation is performed by gromacs-2018.8

One thing you could try is setting runner.verbose=True. This will print the generated wham-2d command before executing it. This way you can try to run the command manually and see if the error is related to Wham or to this python library.
Unfortunately I do not know what error code 2 means.
Hope this helps!
10 Jun 2023 14:49:38 gongshuai0606 @.***>:

Hey, does the issue still exists? I might be able to help if you provide a bit more details: Can you show your implementation at what software you are using for wham?
Thanks for your reply.
I installed wham-2d according to the link "http://membrane.urmc.rochester.edu/sites/default/files/wham/doc.html" in line 69 of adaptiveumbrella/adaptiveumbrella/wham2d.py
cd wham-2d make clean make
then run the example.py
`#!/usr/bin/env python3
import os from copy import deepcopy
import matplotlib.pyplot as plt import numpy as np
import sys
from adaptiveumbrella.wham2d import WHAM2DRunner
sys.path.append('..')
class MyUmbrellaRunner(WHAM2DRunner):
*def init(self):
WHAM2DRunner.init(self)
cum_frames = [0]
def after_run_hook(self):
filename = "tmp/pmf_{}.pdf".format("%02d" % self.num_iterations)
print("Writing new pmf to {}".format(filename))
pmf_to_plot = deepcopy(self.pmf)
pmf_to_plot[pmf_to_plot < 0] = None
pmf_to_plot[self.sample_list == 0] = None
pmf_to_plot = pmf_to_plot.T
self.cum_frames.append(len(self.sample_list[self.sample_list > 0]))

fig, (ax0, ax1) = plt.subplots(ncols=2)
im = ax0.imshow(pmf_to_plot, origin='lower', cmap='jet')
cb = fig.colorbar(im, ax=ax0, orientation='horizontal', pad=0.15)
cb.set_label("kJ/mol")

ax1.plot(self.cum_frames, linewidth=0.5, marker="o", color='black')
ax1.set_xlabel("Cycles")
ax1.set_ylabel("Number of umbrella Windows")

# ticks = [(x,x) for x in [-3, -2, -1, 0, 1, 2, 3]]
# tick_positions = [ self._get_index_for_lambdas(x)[0] for x in ticks ]
# tick_labels = [ str(x[0]) for x in ticks ]
# ax0.set_yticks(tick_positions)
# ax0.set_yticklabels(tick_labels)
# ax0.set_xticks(tick_positions)
# ax0.set_xticklabels(tick_labels)
ax0.set_ylabel("$\phi$")
ax0.set_xlabel("$\psi$")

fig.subplots_adjust(wspace=.5)

plt.savefig(filename, bbox_inches='tight', dpi=200)
os.system("cp {} {}".format(filename, "tmp/pmf_current.pdf"))

def simulate_frames(self, lambdas, frames):
print("{} new simulations:".format(len(lambdas)))
counter = 0
if not os.path.exists("tmp"):
    os.mkdir('tmp')

threads = []
for f in lambdas:
    counter += 1
    print("{}) Simulate lambda1={}, lambda2={}".format(counter, *f))
    command = "bash data/sim.sh {} {} 2>&1 > tmp/run.log".format(*f)
    # print("Running {}".format(command))
    os.system(command)

runner = MyUmbrellaRunner()
runner.WHAM_EXEC = "/home/user/Documents/gs/wham/wham-2d/wham-2d"
runner.whamconfig = {
'Px': 'pi',
'hist_min_x': -3,
'hist_max_x': 3,
'num_bins_x': 100,
'Py': 'pi',
'hist_min_y': -3,
'hist_max_y': 3,
'num_bins_y': 100,
'tolerance': 0.1,
'fc_x': 100,
'fc_y': 100,
'temperature': None,
'mask': None }
runner.cvs = np.array([
(-3, 3, 0.2),
(-3, 3, 0.2),
])
runner.cvs_init = (0, 0)
runner.E_min = 5
runner.E_max = 100
runner.E_incr = 10 runner.max_iterations = 1
runner.run()`
It returned the error:
`/home/user/anaconda3/envs/pytorch1.10/bin/python /home/user/Documents/gs/adaptiveumbrella/examples/example.py
np.shape(windows) (30,) np.shape(windows) (30,)
*Energy: 5
Running simulations
9 new simulations:

Simulate lambda1=-0.2, lambda2=-0.2
Simulate lambda1=-0.2, lambda2=0.0
Simulate lambda1=-0.2, lambda2=0.2
Simulate lambda1=0.0, lambda2=-0.2
Simulate lambda1=0.0, lambda2=0.0
Simulate lambda1=0.0, lambda2=0.2
Simulate lambda1=0.2, lambda2=-0.2
Simulate lambda1=0.2, lambda2=0.0
Simulate lambda1=0.2, lambda2=0.2
Calculating new PMF
wham exited with error code 2

Process finished with exit code 1`
the simulation is performed by gromacs-2018.8
*
—
Reply to this email directly, view it on GitHub[https://github.com/dnlbauer/adaptiveumbrella/issues/2#issuecomment-1585652126], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ABDZB4TKPAGYHAHNQIOUIV3XKRUNVANCNFSM6AAAAAAYUCKFPA].
You are receiving this because you commented.[Tracking image][https://github.com/notifications/beacon/ABDZB4WS5ORGF34GYEU3XIDXKRUNVA5CNFSM6AAAAAAYUCKFPCWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS6QMQZ4.gif]

Closed since there hasn't been a response since more than a year.


Add Netcat Sample

Not sure if you want this, but I created it to test using Cloud Run Button within a repo's subdirectory.

Hi James, sorry for the slow response. This repo is currently focused on "code samples which at least partially appear in cloud.google.com", so not going to merge it. However, I may use this Dockerfile or similar to help drive some YAML tests if that's okay with you?

Of course! Thanks.


Logitech G29 accelerator cannot be detected

I have the same problem with #73
The G29 accelerator is working in games but not detected in XOutput.
Besides there is a high cpu usage, it always rise to 23% suddenly, causing significant input lag. My cpu is i5 7600.
XOutput.log

Same issue for me. ):
XOutput.log

Until this is resolved, I did a little workaround. Since the clutch works and brakes is detected OK and I'm not using clutch anyway, (don't have a stick) I decided to swap pedals around.

open the board (video tutorial); should come right off, there are no plastic clamps
swap clutch / accelerator cables around, or
if you can't unplug the cables (mine are stuck for some reason), swap clutch / accelerator positions and then swap their spring and plate (all other pedal components are the same anyways)

hey all with this issue (and csutorasa if you're reading this) - i have found that v3.18 works perfectly with g29 throttle pedal, and only v3.19 are affected (i have the issue where the axis is showing up but permanently reading 0, but only in v3.19+) as a temporary workaround, roll back to an older non-affected version. csutorasa - if you're still having trouble fixing this (i have not checked v4 developer builds so you may well have solved this issue), it is something that was changed in v3.19.


rcar: optee: fix inverted logic for doma

ANDROID_EXTRA_OEMAKE shall be selected when 'doma' is present.

Reviewed-by: Volodymyr Babchuk<EMAIL_ADDRESS>


Replace missing file for tests

We removed the mock file luca_mock.pdf, so we now need a replacement to make tests run again!
For now I'll add a skip annotation on top on the failing test
@krishnasism

https://github.com/weareprestatech/hotpdf/pull/45


make it less pg dependant

hi,
while i really like the project, it is tied to pgsql engine.
I would like the package to be as agnostic as possible with very little ties to the underlying engine as much as possible.
in my current understanding of the package it does parses the sql create, alter statements to performs some type coalescence.
Those types metadatas are being use to produce compatible func signatures, for as much as i understood so far.
As a consequence the package uses a full blown parser of pgsql statements.
Have you considered to extract those type metadata from the golang struct ?
Instead of building the catalog using the create/alter statements, a catalog can be built from the queries statements, using name resolution their types can be retrieved from the golang metadata obtained earlier.
the sql query parser could be as agnostic as possible, trying to understand sufficiently to extract (table|alias).columns names, but blind enough to not bother if it encounters a function name (or anything similar specific to a db engine) it does not know about.
the code generator has some ties to pg too, but it does not look like a big rethink.

With an update to Go 1.13, the ties to pg will be even smaller, but right now, pg.Array is too convenient so substitute.
But since this is a code generation tool, I think that creating array to slice scanner types is reasonable task.But I can't think of the right approach to do it.
One way would be identify every instance of where arrays are used in the creation scripts and generate package-wide scanner types to be used in every piece of code that an array would be needed. The good side is that the code would be clean and elegant. The downside is that I think it would greatly increases the complexity of the tool.
Another would be generate the scanners on demand. Whenever a array would appear in the script, a scanner type with its methods would be prepended to code that is currently generating. The good side is that I think it wouldn't add too much complexity to the tool. But the downside is that the code would be ugly (but this is a personal opinion).

I did not realize pq.Array was doing so much... this needs some deeper inspection
two examples of differences i can think of
https://stackoverflow.com/a/45352902/4466350
https://stackoverflow.com/a/34627688/4466350

The solution for arrays indeed works, but it is for a specific case that was already covered by a previous issue (refer to #77). I think a more general and idiomatic approach is to make full use of sql.Scanner and the driver.Valuer interfaces. I think this piece of code could be something that  would be useful:
package models

import (
	"database/sql"
	"database/sql/driver"
	"encoding/csv"
	"errors"
	"fmt"
	"regexp"
	"strings"
)

type StringSlice []string

// StringSlice is a slice of strings.
type StringSlice []string

// quoteEscapeRegex is the regex to match escaped characters in a string.
var quoteEscapeRegex = regexp.MustCompile(`([^\\]([\\]{2})*)\\"`)

// Scan satisfies the sql.Scanner interface for StringSlice.
func (ss *StringSlice) Scan(src interface{}) error {
	buf, ok := src.([]byte)
	if !ok {
		return errors.New("invalid StringSlice")
	}

	// change quote escapes for csv parser
	str := quoteEscapeRegex.ReplaceAllString(string(buf), `$1""`)
	str = strings.Replace(str, `\\`, `\`, -1)

	// remove braces
	str = str[1 : len(str)-1]

	// bail if only one
	if len(str) == 0 {
		*ss = StringSlice([]string{})
		return nil
	}

	// parse with csv reader
	cr := csv.NewReader(strings.NewReader(str))
	slice, err := cr.Read()
	if err != nil {
		fmt.Printf("exiting!: %v\n", err)
		return err
	}

	*ss = StringSlice(slice)

	return nil
}

// Value satisfies the driver.Valuer interface for StringSlice.
func (ss StringSlice) Value() (driver.Value, error) {
	v := make([]string, len(ss))
	for i, s := range ss {
		v[i] = `"` + strings.Replace(strings.Replace(s, `\`, `\\\`, -1), `"`, `\"`, -1) + `"`
	}
	return "{" + strings.Join(v, ",") + "}", nil
}

Note: this is a straight copy of some of the code generated by the xo package. But this code works and can be refactored for any of the basic types in Go. It can be used either as field in the generated structs and as a parameter in the db.Query* methods.
If we include scanner types for the ints, the floats, string, bool, Time and interface{} in the generated db.go or models.go file, and use them to substitute each instance of an Postgres array, then the dependence to the pq package would be eliminated in Go 1.13 onwards. Any version below that would be impossible to not have any dependence to the pq package.

@mh-cohen I understand the appeal of having sqlc work with multiple databases. However, the current implementation is built on the top of the PostgreSQL query parser (https://github.com/lfittl/pg_query_go). If there is a Go package containing the SQL parser for your database of choice, I can add it to sqlc.


NextJs 'next/image' support

Viteshot does not fully support NextJs. Things like next image currently don't work.

https://github.com/vercel/next.js/blob/canary/examples/with-playwright/playwright.config.ts

our testable components which use next/image are configured this way:
import * as NextImage from 'next/image';
const OriginalNextImage = NextImage.default;

if (process.env.TEST_ENV === 'vreg') {
  Object.defineProperty(NextImage, 'default', {
    configurable: true,
    value: (props) => (
      <OriginalNextImage {...props} unoptimized loader={({ src }) => src} />
    ),
  });
}


Black preview screen / Sim not loading

Device: MacBook
OS: OSX 10.10.5
On Mac Powerpoint application:
Problem
On a few sims, the "preview" screen (i.e. what you first see after selecting a sim, with the "Insert" and "Cancel" options) appears completely black. In most cases, inserting the sim fixes this and you are able to use the sim, however, you then cannot choose to go back to the menu. There are different variations of this bug for the different sims, which are explained in more detail below.
Charges and Fields: Black preview screen. Fixed if inserted
John Travoltage: Black preview screen, if inserted sim gets stuck on loading screen
Molecule Shapes: Screens appear but when you click on one, the preview goes black. Fixed if inserted
Molecule Shapes Basics: Screens appear but when you click on one, the preview goes blue. Fixed if inserted
Neuron: Preview only shows neuron graphic, all other elements don't show up, fixed if inserted
Note: None of this occurs on Powerpoint Online, just on the Mac app.
Steps to Reproduce

Open Mac Powerpoint application.
Add the PhET add-in from Insert > My Add-Ins > Store
Scroll through the add-in and select one of the above sims

@phet-steele lower priority, but would also be good if you have any insight here.

Charges and Fields:

John Travoltage:

Molecule Shapes:

Molecule Shapes Basics:

Neuron:

@kellywurtz and I are both using Powerpoint 16.16.4.
I took a chance that it would also break on macOS 10.13.6, but it works just fine on my OS. I do not have access to 10.10.5 or any other OS. Sorry I can't be of more help.

Updated my OS to 10.14.1 and both these issues I created went away. My ancient operating system seems to have been the root cause.

@ariel-phet - Should this still be addressed?

I am thinking since it is not an issue on newer OS, no needs not be addressed.
Marking as wontfix but leaving open since we might want to make a mention about using a certain OS or newer in a video


Loss of precision in time argument

When computing the MOON_PA orientation, a loss of precision in the time arguments arises when converting from TDB seconds since J2000 to TDB days since J2000.
CALCEPH and SPICE instead have exact results. MWE:
using Ephemerides
using CalcephEphemeris
using JSMDInterfaces.Ephemeris
using Tempo
using SPICE
using ReferenceFrameRotations

kernel =  "test/assets/moon_pa_de440_200625.bpc"

ephj = EphemerisProvider(kernel);
ephc = CalcephProvider(kernel);

et = rand(0.0:1e8)
et = 1.0

yj, yc = zeros(3), zeros(3)
ephem_orient!(yj, ephj, DJ2000, et/86400.0, 31008, 1, 0)
ephem_orient!(yc, ephc, DJ2000, et/86400.0, 31008, 1, 0)

yj2 = ephem_rotation3(ephj, 1, 31008, et)

A = DCM(pxform("J2000", "MOON_PA", et))
angs = dcm_to_angle(A, :ZXZ)

yj = mod.(yj, π)
yj2 = mod.(yj2, π)
yc = mod.(ys, π)
ys = mod.([angs.a1, angs.a2, angs.a3], π)

yj-yc
yj2-yc

yj-ys
yj2-ys

yc-ys

# yj2 is more precise than yj!
yj2 - ys
yj2 - yj

Converting time as et = (jd0 - DJ2000) + time greatly improves the accuracy (yj and yj2 become equal) but does not completely solve the differences against SPICE and CALCEPH (i.e., yj2 - ys)


Add display and support for changing the bot owners.

Add display and support for changing the bot owners.
Fixes #2719.

Automated message from Dropbox CLA bot
@HarshitOnGitHub, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott I think it's complete. Can you have a look?

@HarshitOnGitHub I guess you should consider this error from travis
File "/home/travis/build/zulip/zulip/tools/documentation_crawler/documentation_crawler/spiders/common/spiders.py", line 85, in error_callback
    raise Exception(failure.value)
Exception: [Errno 2] No such file or directory: '/home/travis/build/zulip/zulip/docs/_build/html/chat.zulip.org'

@anirudhjain75 Don't know why it failed, all tests are running successfully in my local machine. Anyways I have queued a new build, let's see what happens!

Please see my comment here on the issue this PR pertains to:
https://github.com/zulip/zulip/issues/2719#issuecomment-268415878

@showell Do the changes look fine now?

@timabbott I think this one is ready for a final review!! :)

@showell Made the required changes.

If there are 2500 users and 500 bots, it seems like it'll be really bad. I'll add an option to populate_db and test now.

Yep, confirmed; if one does this, opening the #administration page hangs for a long time :).  I merged 7e8d3d9a46a0f0dc84e03152093ca32090e61651 to facilitate this sort of testing in the future.
I think we can fix this perf problem by moving select menu into a template that we only render when the use actually clicks the edit item for a specific bot; that will also have the nice side effect of shrinking the race condition window where a newly joined user wouldn't be in the menu or a name might be out of date (currently the race window is while the /#administration tab is open; after it'd be just after they click the edit button, which is negligible).

@timabbott I think I have addressed all the issues(including the performance one). Please have a look. :)

@HarshitOnGitHub I am suggesting that you switch bot_data from using email for the owner to using user_id for the owner.  It probably makes sense to rework your commits so that the first change refactors the existing code to ids, and then modify your commits for the new feature to use ids as well.  Does that make sense?

I looked this over carefully, and the implementation works well and doesn't regress anything, so I merged this.
As @showell said, we should definitely convert bot_data.js to work off IDs; I opened https://github.com/zulip/zulip/issues/3826 for that follow-up issue.
Thanks @HarshitOnGitHub!!


Developv -> Main

Problem
What is the problem this work solves
Solution
What I/we did to solve this problem
with @pairperson1
Change Summary:

Something else I/we did

Steps to Verify:

A setup step / beginning state
What to do next
Any other instructions
Expected behavior
Suggestions for testing

Level of Risk (low / medium / high):
How much risk is present with the changes? (i.e styling fix that only touches 1 component --> low risk, refactoring all api calls --> high risk). Helpful for reviewers to know how much attention should be given to the proposed changes.
Screenshots (optional):
Show-n-tell images/animations here

Codecov Report

Merging #36 (06ce999) into main (9bff4ed) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #36   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           31        31
  Branches         3         3
=========================================
  Hits            31        31

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 9bff4ed...06ce999. Read the comment docs.


Initial multi platform build tasks

Note that this includes https://github.com/redhat-appstudio/build-definitions/pull/577 which moves to the multi-arch builder image

I think this should be ready to merge now

@tkdchen I didn't notice your comment until after I hit the button, I filed: https://github.com/redhat-appstudio/build-definitions/pull/598


Refine Cell Census build instructions

The build instructions under cell-census/tools/cell_census_builder/ need to be refined and completed

[x] Clean up prose and if needed add/modify instructions for them to be runnable
[x] Include text indicating current cadence of build and schedule
[x] Document current EC2 instance name use to build the Cell Census
[x] Add instructions of how to upload to S3 after build process is complete.

Updated https://docs.google.com/document/d/12JwqpLWrm9_JbhEg99r7LB9sscevxL1gtliqu0s80Bo/edit#.
@Bento007 and @ebezzi As this is not a PR, I can only ask you for your review here. :)
Ideally, @Bento007, you will be able to run through these instructions for next build and perform an official test of them. For that reason, I'm going assign you to the story as well.

would it be possible to include this instructions as an md document somewhere in the repo?

would it be possible to include this instructions as an md document somewhere in the repo?

I don't think this is worth doing. This is still a fast-evolving document and contains CZI-internal details that we don't need to share w/anyone.

@ebezzi @Bento007 are instructions good enough now to close this issue?

yes, but the document will continue to evolve.


Allow ability score boosts at level 0

Fix for #3403
Level 0 characters are still allowed to take level 1 boosts
Could also rename Level 1 to Initial? But I think level 0 characters are niche enough to not warrant a change.

But they can't take class boosts

Doesn't conflict thankfully.
Class pages only mention "Boosts" in the context of key ability scores and free boost at level 5 and above.

A level 0 character still gets the four free ability boosts from Step 6 of the normal character creation process, but not the class ability boost.

I'm not exactly sure where we're disagreeing.
This is how a level 0 character should look on choosing boosts right?

I'm not exactly sure where we're disagreeing.
This is how a level 0 character should look on choosing boosts right?

At level zero, you'll have picked a class

...but you stop after choosing your ancestry and background.

Picking a class comes after these steps, so a level 0 character should have no class:

I think Eddie is right here, though we'll definitely need to look into (non-jquery) context menus to make classes deletable.

See https://2e.aonprd.com/Rules.aspx?ID=1349

Ah, so basically needs an additional check to ignore class boosts when level 0

Even though we still need class, going to merge it anyways since its still a step forward. Class w/o boost can be a separate PR.


feat: context-free languages closure reversal

I'll split it into two PRs.

https://github.com/leanprover-community/mathlib4/pull/8629

-> https://github.com/leanprover-community/mathlib4/pull/8629


Koala Stickers

A sticker store by the creator of the RaspiBlitz Logo
Section: Spend Bitcoin [Niche?]

IS this link bad, i cant open for some reason.

My Mistake: https://koalastickers.com/

Merged


Allows passing flags to disable linking of openssl and zlib

This is a temporary solution for the upcoming release of the compiler. We might want to include an HTTP::Server in the compiler but without linking against openssl and zlib. For example we removed the browser tool because of this, and we might want to bring it back.
So, omnibus would build Crystal with -D without_openssl -D without_zlib but still be able to include an HTTP::Server.
This also affects the HTTP::Client (basically passing -D without_zlib doesn't add an "Accept-Encoding: gzip deflate" header).
This also allows creating any Crystal app that includes an HTTP::Server but, again, without linking against openssl, although we might change this "disabling" mechanism in the future.

Is it really required? Both zlib and openssl are very likely to be installed, thus available.

The less dependencies compiler has - the better.
Imagine scenario, when a programmer has bleeding edge versions (or too old) of zlib and/or openssl installed. This will result in all sorts of weird bugs.
Additionally, it adds more dependencies for all packages for different distributions. And they need to be maintained.

@asterite can I ask about the need for this temporary solution? While you might not need openssl or zlib for the compiler, most likely you will still require them when building things, so these dependencies will still be required.
Is this in response to an specific issue?
Cheers.

@luislavena We can't make omnibus build crystal if it links against openssl. But I guess we'll just try harder next time instead of adding these ifdefs.

Maybe dlopen could help, but that's the hard way.

@luislavena If you don't pass -D without_openssl (or zlib), everything still works as it's working now. We want these flags so we can build the compiler with an HTTP::Server inside it but without linking against openssl nor zlib, that's all. It doesn't affect any other application that doesn't pass those flags.

I see, thank you for the details @asterite :smile:


Start destination with argument

Hi!
As discussed here https://github.com/raamcosta/compose-destinations/issues/20, it seems like the library does not support arguments for start destination.
My question is it possible to workaround it without using deeplinks?
My use case is following: I'm passing argument to an Intent via PendingIntent from notification:
val notification = NotificationCompat.Builder(context, RemindersNotificationChannel)
// ...
.setContentIntent(openActivityIntent(date, id))
// ...
.build()

//....
private fun openActivityIntent(date: Int, id: Int): PendingIntent {
    val intent = Intent(context, MainActivity::class.java)
    intent.putExtra(ARG_DAY, date)
    return PendingIntent.getActivity(
        context, id, intent, PendingIntent.FLAG_UPDATE_CURRENT or PendingIntent.FLAG_IMMUTABLE
    )
}

And then, I want to get this argument from the Intent (if present):
// inside MainActivity.onCreate, setContent {} block:
DestinationsNavHost(
    navGraph = NavGraphs.root,
    startRoute = DiaryScreenDestination(intent?.getIntExtra(ARG_DAY, -1)),
)

Unfortunately, this doesn't compile: startRoute = DiaryScreenDestination(intent?.getIntExtra(ARG_DAY, -1)), while startRoute = DiaryScreenDestination compiles but doesn't allow me to pass the argument.
The DiaryScreen is defined as below:
@Destination(start = true)
@Composable
fun DiaryScreen(initialDay: Int?) {
}

Hi again @pchmielowski !
Loving the feedback, keep it coming! 😄
In this particular case though, this is also not possible for jetpack compose navigation which we use under the hood, so I'm not sure I can or want to do this.
Start destination string parameter of vanilla NavHost is the "route id" (let's call it) not a resolved route with arguments. In compose destinations these have two different types: Route and Direction. So there if you try to pass a Direction it won't even compile (another advantage of Compose Destinations 🙌 ).
In fact, I believe your use case should be solved by a "deep link intent" in the notification, as you can see here for example:
https://proandroiddev.com/open-composables-via-notification-with-jetpack-navigation-b922384f4091


Include required role for reseeding hardware oauth tokens

Our documentation lacks on the role required for uploading hardware oauth tokens. Though a user with Authentication Admin or Privileged Authentication Admin or Authentication Policy Admin can view the page and upload the hardware oauth files, the end result is a failure with the error "Something went wrong". This has been tested by me and also has been confirmed internally .

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

ID: 3580bf07-789b-a9c0-26c2-68d21eb66b70
Version Independent ID: f61c354b-cac2-c3e2-881a-2f0540bdd8e1
Content: OATH tokens authentication method - Azure Active Directory
Content Source: articles/active-directory/authentication/concept-authentication-oath-tokens.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@abhijeet-MSFT Thank you for your inputs. We will have this reviewed and work with content team to get it updated.

@abhijeet-MSFT thanks! I added GA is required.
#please-close


Some tests are randomly failing

I tried to induce as much failed tests as possible. Here are the ones those are randomly failing sometimes.
Environment: Windows 7 virtualbox guest.
They are mostly interrelated and the problem is with the test setup. This is not a full list of failed tests.

Test Name:	NTumbleBit.Tests.TumblerServerTests.EscrowGetReddemedIfTimeout
Test FullName:	NTumbleBit.Tests.TumblerServerTests.EscrowGetReddemedIfTimeout
Test Source:	C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs : line 145
Test Outcome:	Failed
Test Duration:	0:00:21,258

Result StackTrace:
at System.IO.Win32FileSystem.RemoveDirectoryHelper(String fullPath, Boolean recursive, Boolean throwOnTopLevelDirectoryNotFound)
   at System.IO.Win32FileSystem.RemoveDirectory(String fullPath, Boolean recursive)
   at System.IO.Directory.Delete(String path, Boolean recursive)
   at NTumbleBit.Tests.CoreNode.CleanFolder() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\NodeBuilder.cs:line 242
   at NTumbleBit.Tests.CoreNode.Kill(Boolean cleanFolder) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\NodeBuilder.cs:line 460
   at NTumbleBit.Tests.NodeBuilder.Dispose() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\NodeBuilder.cs:line 171
   at NTumbleBit.Tests.TumblerServerTester.Dispose() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 307
   at NTumbleBit.Tests.TumblerServerTests.EscrowGetReddemedIfTimeout() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs:line 202
Result Message:	Access to the path 'wallet.dat' is denied.

Test Name:	NTumbleBit.Tests.TumblerServerTests.TestCRUDDBReeze
Test FullName:	NTumbleBit.Tests.TumblerServerTests.TestCRUDDBReeze
Test Source:	C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs : line 40
Test Outcome:	Failed
Test Duration:	0:00:06,719

Result StackTrace:
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Server.Kestrel.Internal.KestrelEngine.CreateServer(ServerAddress address)
   at Microsoft.AspNetCore.Server.Kestrel.KestrelServer.Start[TContext](IHttpApplication`1 application)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Start()
   at NTumbleBit.Tests.TumblerServerTester..ctor(String directory) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 186
   at NTumbleBit.Tests.TumblerServerTester.Create(String caller) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 130
   at NTumbleBit.Tests.TumblerServerTests.TestCRUDDBReeze() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs:line 41
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Networking.Libuv.Check(Int32 statusCode)
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Networking.UvTcpHandle.GetSockIPEndPoint()
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Http.TcpListener.CreateListenSocket()
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Http.Listener.<>c.<StartAsync>b__6_0(Object state)
Result Message:
One or more errors occurred. (Error -4091 EADDRINUSE address already in use)
Error -4091 EADDRINUSE address already in use

Test Name:	NTumbleBit.Tests.TumblerServerTests.TestCRUDDBReeze
Test FullName:	NTumbleBit.Tests.TumblerServerTests.TestCRUDDBReeze
Test Source:	C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs : line 40
Test Outcome:	Failed
Test Duration:	0:00:00,028

Result StackTrace:
at System.IO.Win32FileSystem.RemoveDirectoryHelper(String fullPath, Boolean recursive, Boolean throwOnTopLevelDirectoryNotFound)
   at System.IO.Win32FileSystem.RemoveDirectory(String fullPath, Boolean recursive)
   at System.IO.Directory.Delete(String path, Boolean recursive)
   at NTumbleBit.Tests.TumblerServerTester.TryDelete(String directory, Boolean throws) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 230
   at NTumbleBit.Tests.TumblerServerTester..ctor(String directory) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 150
   at NTumbleBit.Tests.TumblerServerTester.Create(String caller) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 130
   at NTumbleBit.Tests.TumblerServerTests.TestCRUDDBReeze() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs:line 41
Result Message:	The directory is not empty

Test Name:	NTumbleBit.Tests.TumblerServerTests.CanGetParameters
Test FullName:	NTumbleBit.Tests.TumblerServerTests.CanGetParameters
Test Source:	C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs : line 23
Test Outcome:	Failed
Test Duration:	0:00:07,667

Result StackTrace:
at System.IO.Win32FileSystem.RemoveDirectoryHelper(String fullPath, Boolean recursive, Boolean throwOnTopLevelDirectoryNotFound)
   at System.IO.Win32FileSystem.RemoveDirectory(String fullPath, Boolean recursive)
   at System.IO.Directory.Delete(String path, Boolean recursive)
   at NTumbleBit.Tests.CoreNode.CleanFolder() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\NodeBuilder.cs:line 242
   at NTumbleBit.Tests.CoreNode.Kill(Boolean cleanFolder) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\NodeBuilder.cs:line 460
   at NTumbleBit.Tests.NodeBuilder.Dispose() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\NodeBuilder.cs:line 171
   at NTumbleBit.Tests.TumblerServerTester.Dispose() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 307
   at NTumbleBit.Tests.TumblerServerTests.CanGetParameters() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs:line 33
Result Message:	The directory is not empty

Test Name:	NTumbleBit.Tests.TumblerServerTests.CanGetParameters
Test FullName:	NTumbleBit.Tests.TumblerServerTests.CanGetParameters
Test Source:	C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs : line 23
Test Outcome:	Failed
Test Duration:	0:00:14,096

Result StackTrace:
at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Net.WebRequest.<>c.<GetResponseAsync>b__37_2(IAsyncResult iar)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic(IAsyncResult iar, Func`2 endFunction, Action`1 endAction, Task`1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NBitcoin.RPC.RPCClient.<SendCommandAsync>d__21.MoveNext() in E:\Sources\NBitcoin\NBitcoin\RPC\RPCClient.cs:line 325
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at NBitcoin.RPC.RPCClient.SendCommand(RPCRequest request, Boolean throwIfRPCError) in E:\Sources\NBitcoin\NBitcoin\RPC\RPCClient.cs:line 288
   at NBitcoin.RPC.RPCClient.GetBestBlockHash() in E:\Sources\NBitcoin\NBitcoin\RPC\RPCClient.cs:line 498
   at NTumbleBit.Tests.CoreNode.Sync(CoreNode node, Boolean keepConnection) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\NodeBuilder.cs:line 252
   at NTumbleBit.Tests.TumblerServerTester.SyncNodes() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 210
   at NTumbleBit.Tests.TumblerServerTester..ctor(String directory) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 162
   at NTumbleBit.Tests.TumblerServerTester.Create(String caller) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 130
   at NTumbleBit.Tests.TumblerServerTests.CanGetParameters() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs:line 24
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at System.Net.Http.HttpClient.<FinishSendAsync>d__58.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at System.Net.HttpWebRequest.<SendRequest>d__63.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at System.Net.Http.WinHttpHandler.<StartRequest>d__105.MoveNext()
Result Message:
An error occurred while sending the request. A connection with the server could not be established
An error occurred while sending the request.
A connection with the server could not be established

Test Name:	NTumbleBit.Tests.TumblerServerTests.CanCompleteCycleWithMachineState
Test FullName:	NTumbleBit.Tests.TumblerServerTests.CanCompleteCycleWithMachineState
Test Source:	C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs : line 62
Test Outcome:	Failed
Test Duration:	0:00:07,652

Result StackTrace:
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Server.Kestrel.Internal.KestrelEngine.CreateServer(ServerAddress address)
   at Microsoft.AspNetCore.Server.Kestrel.KestrelServer.Start[TContext](IHttpApplication`1 application)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Start()
   at NTumbleBit.Tests.TumblerServerTester..ctor(String directory) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 186
   at NTumbleBit.Tests.TumblerServerTester.Create(String caller) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 130
   at NTumbleBit.Tests.TumblerServerTests.CanCompleteCycleWithMachineState() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs:line 63
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Networking.Libuv.Check(Int32 statusCode)
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Networking.UvTcpHandle.GetSockIPEndPoint()
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Http.TcpListener.CreateListenSocket()
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Http.Listener.<>c.<StartAsync>b__6_0(Object state)
Result Message:
One or more errors occurred. (Error -4091 EADDRINUSE address already in use)
Error -4091 EADDRINUSE address already in use

Test Name:	NTumbleBit.Tests.TumblerServerTests.CanCompleteCycleWithMachineState
Test FullName:	NTumbleBit.Tests.TumblerServerTests.CanCompleteCycleWithMachineState
Test Source:	C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs : line 62
Test Outcome:	Failed
Test Duration:	0:00:07,181

Result StackTrace:
at DBreeze.DBreezeEngine.InitDb()
   at NTumbleBit.Client.Tumbler.Services.DBreezeRepository.GetEngine(String partitionKey) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.TumblerServer\Services\DBreezeRepository.cs:line 59
   at NTumbleBit.Client.Tumbler.Services.DBreezeRepository.UpdateOrInsert[T](String partitionKey, String rowKey, T data, Func`3 update) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.TumblerServer\Services\DBreezeRepository.cs:line 35
   at NTumbleBit.Client.Tumbler.ClientDestinationWallet..ctor(String walletName, BitcoinExtPubKey extPubKey, KeyPath derivationPath, IRepository repository) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Client\Tumbler\ClientDestinationWallet.cs:line 30
   at NTumbleBit.Tests.TumblerClientContext..ctor(TumblerClient tumblerClient, RPCClient rpcClient, IRepository clientRepository) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 27
   at NTumbleBit.Tests.TumblerServerTester..ctor(String directory) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 190
   at NTumbleBit.Tests.TumblerServerTester.Create(String caller) in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTester.cs:line 130
   at NTumbleBit.Tests.TumblerServerTests.CanCompleteCycleWithMachineState() in C:\Users\nopara\Desktop\NTumbleBit\NTumbleBit.Tests\TumblerServerTests.cs:line 63
   at DBreeze.Transactions.TransactionsJournal.Init()
   at DBreeze.DBreezeEngine.InitDb()
Result Message:
Creation of the database folder failed!
Transaction journal couldn't clean rollback files of the finished transactions!

I spent considerable amount of time and here is my experience:
When you mess up your system (opening other softwares, stopping randomly the tests, etc) you can get a considerable amount of failed tests coming up randomly.
Only the TumblerServerTests are failing.
Everything is related to

bitcoinds those are left open sometimes when the test fails or I terminate,
probably the kestrel server is failing to make some resources free too,
various file-access problems inside TestData

I also suspect bitcoind should not be killed, too, as you do in the software. I think that would solve a lot if we could use this: http://bitcoin.stackexchange.com/questions/13696/how-to-safely-shutdown-bitcoind
I noticed many of the problems are resolved by waiting a little before deletion and retry it some times, as this stackoverflow post suggest: http://stackoverflow.com/a/14933880/2061103
public static void DeleteRecursivelyWithMagicDust(string destinationDir)
{
	const int magicDust = 20;
	for (var gnomes = 1; gnomes <= magicDust; gnomes++)
	{
		try
		{
			Directory.Delete(destinationDir, true);
		}
		catch (DirectoryNotFoundException)
		{
			return;  // good!
		}
		catch (IOException)
		{
			// System.IO.IOException: The directory is not empty
			System.Diagnostics.Debug.WriteLine("Gnomes prevent deletion of {0}! Applying magic dust, attempt #{1}.", destinationDir, gnomes);

			// see http://stackoverflow.com/questions/329355/cannot-delete-directory-with-directory-deletepath-true for more magic
			Thread.Sleep(50);
			continue;
		}
		return;
	}
	// depending on your use case, consider throwing an exception here
}

I think this should be used whenever possible, it's a better way to recursively delete, and it does not force anything, still throwing exceptions if something is wrong.

bitcoind should not be programmatically killed, but properly closed:

problem is that you don't know the port and rpc creds of the left over bitcoind.
For the magic dust solution, can you try it ? I am not sure this is the problem.

Yes, as I said before it fixes about half of the problems.

I did not experienced any problem with tests anymore. Waxwing issues is separate.

I'll keep this in my ToDo list and get back to it after we released the first version of Breeze wallet so we start working on TumbleBit integration.

did #37 solved everything ?

So far so good. Reopen if not.


how to set modal like this?

Ask your Question

Hi!
I'm closing this issue because it's a question and not a bug report.
If you're still looking for feedback/answers, could you please ask your question in a new discussion?
Thank you!


Unselecting elements from radar chart changes the other element's colors but it's not reflected in the legend

On the example page https://elemefe.github.io/v-charts/#/radar, if you uncheck some of the elements, the chart re-renders and the elements displayed in the chart change their color. However, this color change is not reflected in the legend.

Initially, 1月1日 is green, 1月2日 blue, 1月3日 red, 1月4日 yellow, 1月5日 dark blue and 1月6日 violet.
Unselect everything except 1月3日
--> In the legend, 1月3日 is still red, but in the chart, it's green.

@smares OK, I can confirm it.

@smares  This problem has been fixed in v1.5.0.
https://jsfiddle.net/jww5fqs8/4/


stdenv: add sparc64 cross-compilation support

Motivation for this change
These are the trivial changes from https://github.com/NixOS/nixpkgs/pull/141448.
nix build 'github:piperswe/nixpkgs/piper/add-sparc64#pkgsCross.sparc64-linux.hello'

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

This looks great to me, I was able to build the cross bootstrap tools and cross-compile some basic packages. Once this is merged the cross-trunk job will start building cross bootstrap tools for this platform so we'll have the cross stdenv in the cache.
cc @NixOS/exotic-platform-maintainers

Thank you @piperswe for making this PR! I'm excited to see sparc64 support in Nixpkgs

Bumping: Can we get this rebased and the flake change dropped?


upgrade to setup-java v4

What is the benefits between v3 and v4?

@sullis any thoughts? I just want to understand the impact of a change befoer merging it.

@sullis any thoughts? I just want to understand the impact of a change befoer merging it.

setup-java v4 release notes are available here:
https://github.com/actions/setup-java/releases

@sullis thank you. I see that this just makes it more future proof to future versions of java (e.g. 21). thanks. merging it.


Collaboration

Hi this is great. Can't believe I didn't notice your repo.
I am a collector of univariate TS methods. I would like to massage some of your stuff into the "skater" format I use (see skater signature) so as to include it downstream in all sorts of things (see https://github.com/microprediction for explanation).
The reverse trip looks pretty easy ... just looking at you https://github.com/ITMO-NSS-team/pytsbe/blob/main/pytsbe/models/pmdarima_forecaster.py now.
Btw I can provide lots of fresh data if you want it. See https://microprediction.github.io/microprediction/data.html and help yourself.
-peter

Hi! Thank you very much for this issue, I am glad to hear that you liked the repository :)
Sure, feel free to use and adapt the methods that are presented in this library and in the FEDOT (Automl tool for regression, classification and time series forecasting tasks).  Or integrate your algorithms into pytsbe also. I am not currently an owner of this repository, nor of the FEDOT framework. So it might be better to contact with Nikolay if you need help with adaptation. Or if there are some ideas you want to discuss. Nevertheless, I'll try to participate too, if I have time and energy to do that (after my regular job tasks) :)
About the data - thank you very much, perhaps it could indeed be added to the pytsbe. If you would like to test the algorithms on your data with pytsbe, I would be glad to help you!

I'll give it a spin, thx for the reply.


Fixes #55: Add Python 3.4 to Travis control file.

This is an attempt to get this added. Let's see if it works.

It does work.
Ready to be merged, if we want to add Python 3.4 to the Travis runs.
Please review.


Convert cf-grid to atomic design

This should be light, but it does touch other pieces and should get done early.
I talked with @contolini and https://github.com/cfpb/capital-framework/blob/canary/src/cf-grid/src-generated/cf-grid-generated.less will likely be out of scope since it's generated more as an example of what can be done with cf-grid rather than on projects themselves.

I'm not sure anything about cf-grid should change, since it's all mix-ins. There's no atoms, molecules, or organisms to speak of, really.

Not much will change. There'll be light clean up and better guidance. Just trying to move things to done as best I can here.

Should most of grid get u- prefixes to fit in w/ the discussion in cf-layout?

Let's move to #atomic-design.

@KimberlyMunoz we decided there's nothing atomic that needs to happen right? Just some doc improvements that can wait till after the v4 release?

That is my opinion.

Keeping this issue so we don't lose the relative discussion

That's what was decided in #atomic-design because cf-grid is all mixins and the grid is just a sample and not part of CF. I think if anything does come up, we could make a new issue for it.


Changing Wurst Settings removes folding markers from currently opened file.

Until it's reloaded

Eclipse plugin n longer maintained


Intermediate of a conversation (Idea)

Let one person talk to another person from your whatsapp!
E.g.
Person A: send message to you
You: receive message from A and send the message to Person B
Person B: receive message from you

Please assign this issue to me

@safurazaheer if you have any question ask me here or in slack private.

@xandao6   Basically, recieving the text can be solved by scrapping na?

You need to find the right CSS Selector for the message, open your whatsapp web and try to undestand this.

@safurazaheer  deadline extended +4d

can you give the url of slack channel? @xandao6

@xandao6 I tried scrapping through selenuim and beautiful soup but I think this wont work fine coz it wont give the runtime result i.e we wont get the current message.

@HarsheetKakar You are a GSSOC participant?

@safurazaheer we use pyppeteer, we don't use selenium.

@HarsheetKakar You are a GSSOC participant?

yes

@safurazaheer isn't hard to get the last message. You can do this with this selector:
#main > div > div > div > div > div:last-child > div > div > div > div.copyable-text > div

@HarsheetKakar search for proj_whatsapp-play inside gssoc slack

@safurazaheer how is it going?

May I work on this issue @rpotter12 @xandao6?

I would like to work on this issue.Please assign it to me.


Feat:Addd follow us on LinkedIn button

Describe the feature
Addd follow us on LinkedIn button
Add ScreenShots
Addd follow us on LinkedIn button
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

@rajlaxmi03 please assign this issue to me.

Hey, Manika Sethi this side, I'm a GSSoC contributor and I am confident that I can address this issue effectively since I have understood the code. Could you please assign it to me? Thanks!

@Yashshukla11 assign this to me i am GSSOC'24 contributor and have 2+ years of experience in flutter

please assign this issue to me


Refactor and add config for incremental model

There are some breaking changes I make when working with dbt-vertica on incremental merge stagery:

Refactor merge_columns to unique_key.
Change MERGE ON logic: use HASH(cols) instead of join DEST.col1 = SRC.col1, DEST.col2 = SRC.col2,...
Add additional config: merge_update_columns (list) to be able to update only columns user want.

@viplazylmht this is great work. To make this easier to digest, do you think you could split this into three PRs:

Connection load balance changes
Merge updates (here, provide the HASH matching as an option).
Insert+delete updates (for this PR, please include why it's needed).

@viplazylmht could you push another PR here? that would trigger the automated tests.


Who's interested in maintaining this project?

First, I want to thank @devdennysegura for all the work put into this plugin. What an awesome gift to the community! As this project is no longer actively maintained, let's come up with a plan for moving this plugin forward.
@gregertw, @filoe, and quite a few others have forks of this repo. I believe we should take one of these forks as the new official home for this project. I believe it does a disservice to the hard work that Denny's already done to not come together and get this moving forward again as a community.
This project represents a significant step forward for the Flutter community. It's a key gateway project and having it will help the overall growth in the community. I would like to work towards getting this up to date and then released on flutter pub as mentioned here https://github.com/devdennysegura/flutter-auth0/issues/9.
I myself might be willing to help out in this effort, but I don't think I'm the right person to take over as sole maintainer. Would @gregertw, @filoe, or anyone else be willing to do this? Who's interested?

After we come up with a plan here, I can try reaching out to Denny over email to officially transfer maintenance of the project if possible. If not, we'll just have to start running with a fork and use the issue section to direct traffic over to the actively maintained version.

@flawless2011 see above

After getting stuck in lots of problems related to auth0, we made the decision to rollout our own identity service. In combination with that, we've moved to simple_auth which worked out quite well.

@bmcilw1, thanks for your note and initiative. You are right, there is a substantial investment and good work put into this repo!
I have tried to reach out to @devdennysegura a couple of times, also on email, but with no answer, but he did merge a PR from me earlier this Spring, so I'm not sure if he is just awfully busy or not interested :)
I'm happy to contribute, but I cannot be the sole maintainer. I think we need 2-3 with PR merge rights. I don't use this project beyond my own tinkering and I have a day job :)
My fork should be fairly up to date with @filoe's PRs as well, so happy to use that if we need to fork.

For the benefit of those who are here looking for a production-ready auth solution for Flutter, I'm currently investigating leveraging just the Auth portion of Google Firebase, but with our own APIs and database. From what I'm seeing the pricing is comparable to that of Auth0 and it covers what we need for our use case. Personally, I prefer that to implementing an auth flow and password storage strategy in-house.
Backend: https://medium.com/@_josueperalta/using-firebase-auth-with-a-custom-node-js-server-part-1-53bdb622c89a and https://firebase.google.com/docs/auth/admin/#verify_id_tokens
Frontend: https://pub.dev/packages/firebase_auth and https://medium.com/flutterpub/flutter-how-to-do-user-login-with-firebase-a6af760b14d5

I'm interested in being a contributor (not sole contributor).  I do think it'd be nice if we could investigate getting Auth0 to take over ownership even if we have a few community members helping to maintain the project.

Hi, thank you all for the help with the repo, especially @gregertw, I have been a little busy, for that reason I have not been fully in the repo, this weekend I will be updating the entire repo, with the latest updates of flutter.


fix(KONFLUX-3663): format PipelineRun files and upload SAST results

This update configures the SAST task to upload SARIF results to quay.io for
long-term storage
Related:

https://issues.redhat.com/browse/KONFLUX-3663
https://issues.redhat.com/browse/KONFLUX-2263

/retest

@ccronca please amend the git commit to fix the CI error. Thanks!

/retest


Default File History Permissions

BZ: https://issues.jboss.org/browse/WFCORE-120

Linux Build 1382 is now running using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b

Windows Build 1052 is now running using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b

Core - Full Integration Build 632 is now running using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b

Windows Build 1052 outcome was SUCCESS using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b
Summary: Tests passed: 3088, ignored: 64 Build time: 0:19:29

Linux Build 1382 outcome was FAILURE using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b
Summary: Execution timeout (new); tests passed: 2728, ignored: 49; org.wildfly.core:wildfly-core-testsuite-domain Build time: 0:45:04
Build problems:
Execution timeout
Process exited with code 137

Core - Full Integration Build 632 outcome was SUCCESS using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b
Summary: Tests passed: 2828, ignored: 412 Build time: 0:49:27

retest this please

@aloubyansky Could you give this a look over?

Windows Build 1074 is now running using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b

Linux Build 1407 is now running using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b

Core - Full Integration Build 655 is now running using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b

Windows Build 1074 outcome was SUCCESS using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b
Summary: Tests passed: 3088, ignored: 64 Build time: 0:21:16

Linux Build 1407 outcome was SUCCESS using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b
Summary: Tests passed: 3088, ignored: 64 Build time: 0:18:30

Core - Full Integration Build 655 outcome was FAILURE using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b
Summary: Tests failed: 39 (39 new), passed: 2789, ignored: 412 Build time: 0:49:11
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).failoverOnStop: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).nestedBeanFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).connectionFactoryFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).jmsConnectionFactoryFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).noFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).simpleFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulTimeoutTestCase(SYNC-tcp).timeout: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.xpc.StatefulWithXPCFailoverTestCase(SYNC-tcp).testBasicXPC: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.xpc.StatefulWithXPCFailoverTestCase(SYNC-tcp).testSecondLevelCache: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testSecureStatelessFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testStatelessFailover: <no details avaliable>

##### there are 29 more failed tests, see build details

Looks good to me. Thanks!

Core - Full Integration Build 658 is now running using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b

Core - Full Integration Build 658 outcome was SUCCESS using a merge of 72e24a1a8a5d04eac3614f5bc14bc6173357634b
Summary: Tests passed: 2828, ignored: 412 Build time: 0:49:31


fix: total sold items from real events

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[X] Bugfix
[ ] Feature
[ ] Refactoring

Context

[X] Closes #9474
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[X] My contribution builds clean without any errors or warnings
[X] I've merged recent default branch -- main and I've no conflicts
[X] I've tried to respect high code quality standards
[X] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[X] Fill up your DOT address: Payout

Community participation

[ ] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

i think the problem may be bigger than we thought
please compare:
https://deploy-preview-9497--koda-canary.netlify.app/ahk/u/CmWHiv7h4m9tEzKD94DH4mqwGTvsdYQe2nouWPF7ipmHpqA?tab=activity&buy=false&sale=true&page=2
https://canary.kodadot.xyz/ahk/u/CmWHiv7h4m9tEzKD94DH4mqwGTvsdYQe2nouWPF7ipmHpqA
if you look at activity, there are like way more than 20 sell actions

@prury Seems like it. I will investigate it further.

@prury The profileStatsById query returns wrong values for sell events. The listed, sold, holdings, max sold price, etc. values are wrong. One solution would be injecting these vaues from Activity.vue given that these are all fetched there aswell and would also improve the loading time.

@prury The profileStatsById query returns wrong values for sell events. The listed, sold, holdings, max sold price, etc. values are wrong. One solution would be injecting these vaues from Activity.vue given that these are all fetched there aswell and would also improve the loading time.

let's see what internal dev thinks

@prury The profileStatsById query returns wrong values for sell events. The listed, sold, holdings, max sold price, etc. values are wrong.

It's related to the graphql query file. Could you check why it's wrong?

One solution would be injecting these vaues from Activity.vue given that these are all fetched there aswell and would also improve the loading time.

Activity.vue component is not loaded at the beginning because it's on-demand. You have to force load it if you would like to get the values from it.
loading time would not be improved because the Activity component would take much more time to load all data.

@Jarsen136 Sure, I will look into it. Is there some documentation on the queries, I couldn't find any.

@Jarsen136 Sure, I will look into it. Is there some documentation on the queries, I couldn't find any.

https://github.com/vikiival/slides/blob/main/graphql-intro/slides.md
https://graphql.org/learn/

ugh two weeks, closing


Conversation ChangeOwner Wrong Endpoint

I am trying to use the API to assign a conversation and when I use the Conversation endpoint ChangeOwner method I get the following error from Helpscout:
"{"message":"no route and no API found with those values"}"

I added pull request https://github.com/Selz/HelpScoutNet/pull/48 with a fix for this – still hasn't been approved.


wandb on/off does not seem to change WANDB_MODE

wandb --version && python --version && uname

Weights and Biases version : 0.8.36
Python version : 3.7.4
Operating System : Ubuntu 16.04

Description
wandb on/off not working.
What I Did
I noticed wandb on/off was not working so I tried setting the environment variable 'WANDB_MODE' manually and found that it worked as expected. Then I tried using 'wandb on/off' and found that it had no effect on the environment variable (See output below).
Installations were made inside a conda environment, if that matters.
$ wandb off
W&B disabled, running your script from this directory will only write metadata locally.
$ echo $WANDB_MODE
run
$ wandb on
W&B enabled, running your script from this directory will now sync to the cloud.
$ echo $WANDB_MODE
run
$ wandb off
W&B disabled, running your script from this directory will only write metadata locally.
$ echo $WANDB_MODE

Hey @samarth4149 wandb off updates the file wandb/settings relative to where it's run.  The library automatically reads this file.  You can always set the WANDB_MODE environment variable manually to set the run mode.


migrate to go modules

...away from Glide.
Contributing this back to your fork, @rjh-yext, given that it looks like it's more active then @jirwin's. @jirwin Let me know if you'd like me to submit this to you.

cc @zmstone

maybe you want to update Dockerfile too.


FreeBMD2 fails to indicate that a marriage entry was Registered by the GRO in the form "Smith or Jones" for spouse's name

I wanted to check out searching for marriages where a page number has UCFs (uncertain character format), so I started looking at some of the marriage files that I had transcribed; and I got side-tracked onto this particular topic.
The page 1950-M1-B0086 has an "entry" of the form:
Betts, William R., McCarthy, Southwark, 5d, 1222
#Comment(2) entry reads McCarthy or Taylor for Spouse's name
Betts, William R., Taylor, Southwark, 5s, 1222
and on FreeBMD1 it is displayed as:

The FreeBMD1 Entry information page has:
Betts | William R. | Taylor | Southwark | 5d | 1225 | prclarke |
#COMMENT entry reads McCarthy or Taylor for Spouse's name.
I don't check the FreeBMD1 Entry information for the other William, but the same Comment is likely to be there.
In Contrast: if I do a search on FreeBMD2 test3 for William R. Betts in March 1950, I get only two results: "our William".
If I go to the Entry Information page for the first William, his spouse is McCarthy.
If I go to the Entry Information page for the second William, his spouse is Taylor.
If I click on the Page number there are six entries: two for William, one for Jean A Taylor, one for Jean A McCarthy, another "man" and another "woman".
We have "thrown away" / neglected to display / not yet implimented the ability to display the COMMENT for William: entry reads "McCarthy or Taylor" for Spouse's name.
Note: there is an earlier entry on the same Index page (not the same Register page):  Lamb or Betts or Sandilands.
Both of these can be found by looking at the Scan of the index.
Anyone not looking at the scan image on FreeBMD2 would be unaware that there was only one William and only one Jean and, by inference, that William might be Jean's second husband.

View is comparable.  What is missing from FreeBMD2 in this instance is the #Comment accessed via the question-mark symbol on FreeBMD1.
This is not just about marriage partners - all comments are missing - see for example marriage of  Richard George Alcock DECEMBER 1881 and
Newark and birth of Edith Alcock DECEMBER 1891 Malton.

Hi @PatReynolds ,
My thoughts: FreeBMD(1) and FreeBMD2 display things differently and I know that the "look and feel" of FreeBMD2 was designed using wireframes (if that is the right terminology), but those are not something that I would necessary see (or have access to).
If FreeBMD 2 worked the same way as FreeBMD1 then this screen capture would have another column on the right with either symbols or words indicating that there are: Postems, Comments (such as Smith or Jones) and Late Entries, etc. associated with a particular entry.

The FreeBMD2 Results page does not have such an third column, so I just assumed that these symbols or words were being done at the level of the Entry Information Page, not on the Results page level (and this appears to be the case now for Late Entries).
If we take the example of the birth of William E Betts:

The FreeBMD(1) Entry Information page has the information The above transcription represents what is in the GRO index but it is referred to by the following entry: Births 1904 Sep: BETTS, William E,Wellingbro',3b,S/34
FreeBMD2 has implemented this Late Entry on the Entry Information Page, not on the Results page.
So, on that basis, those two two 1950  marriage Entry Information pages for William R Betts (above) should be implementing the equivalent of  the FreeBMD(1) "COMMENT entry reads McCarthy or Taylor for Spouse's name" against each entry on the two Entry Information pages and they don't appear to do so.
Note: as Researcher, I would prefer to see this information (in summary form) on the Results page, but it could go on both individual Entry Information pages (this is a design question - it could go on all three pages e.g. the Results Page and both Entry Information pages).
There are very few Postems on the the Database being used on test2 & 3 so I have not found any yet. Postems need to be displayed, as per alternative surnames, and suspect page ranges, etc; and it appears to be the Entry Information page where they should go.
I've not checked another other Late Entries - but FreeBMD2 does appear to have addressed the Late Entry for William E Betts (in 1904 and 1934).

Hello Philip, yes, the comments, scans, etc. are now all on Entry details pages (logic being that you would not use these to do a first-level identification of an entry warranting further investigation).
Since both in FreeBMD1 and FreeBMD2, the user has no idea that there is a doubt over the surname of the partner, without clicking through to the entriy infomation, I don't expect user testing to suggest that anything more is needed on the results pages (but could be wrong!)


fix(sandbox): allow the sandbox to load plugins when running locally

What?
Add the allowAllOrigins option to the createFieldPlugin function.
Why?
JIRA: EXT-2258
It allows our demo package (or any other plugin created by other developers) to be loaded in a local running instance of the Sandbox.
How to reproduce?
From the main branch run:
yarn
yarn dev:demo
yarn dev:container

Then, this error should appear in your console:

and the demo plugin should not be loaded:

How it should work after the change?
Now, from the task's branch, run these commands:
yarn
yarn dev:demo
yarn dev:sandbox

Then, no error should appear in your console:

and the demo plugin should be loaded:

PS: The Vercel action is not running yet since it wasn't modified to consider the new path (packages/sandbox) and also the new project name (modified here) in its settings.

Thank you for suggesting this change @demetriusfeijoo.
I am wondering if we should add allowAllOrigins or something more explicit like whitelabeledOrigin so that it is never possible to allow all origins (*).
In any case, we need to add this to the docs explaining the usage of this new config parameter. What do you think?

What if we even have something like this?
useFieldPlugin({
  target: 'https://something-else.com'
})

and if target is missing, it falls back the the previous implementation (either sandbox or the production according to the host parameter)

@eunjae-lee, totally agree with your suggestion. @demetriusfeijoo However please deprioritize this PR for now due to the cycle commitment.

Thanks for your reviews on this one @BibiSebi @eunjae-lee. 🚀
I thought about letting the developer use the wildcard (*) so, for example, even if the sandbox is running locally in a different port than 7000 (the default port for the Sandbox) the demo application wouldn't require any change. It would still work in that case.
When we return to this PR (after the cycle finishes) we may discuss having a third approach which would be your suggestion @eunjae-lee, but allowing also the wildcard, IDK.
Something like:
useFieldPlugin({
  target: 'https://something-else.com'
})

useFieldPlugin({
  target: '*'
})

If it's set to * we could then warn the developer in the console. 🤔
Just another idea that doesn't need to be handled now as we're at the begin of our cycle.

I just realized that the Sandbox preview also doesn't work due to the same issue.


Be able to generate a B2B error when type in message is not unique

When sending in a move-in request type must be 392. If another code is used it should result in a B2B error informing the actor that type is wrong. Type is unique pr. process.

[ ] If another code than 392 is used in type for move-in it should result in a B2B error.

Issue has been solved


Bug: seperate raise Exception and logging print

Raising an exception cannot simulatenous print a logging message with the logging library. These actions must be separated. See train_segmentation.py, line 184

Can you show what you got?

Here's the stack trace:

I will look into it

We're very excited to see this issue closed @CharlesAuthier :)


Add command to return the available songs

Depends on https://github.com/LuisMayo/objection_engine/issues/54

Fixed by ef1f7c3b8d75b999474cc748dfab5bd1a5e790ff


low speed in speedtest

homepc:
mode=client
kcp=fast1
inbound_bandwidth=1G
outbound_bandwidth=1G
#kcptube should listen on 51820, so the wireguard client can connect to this port.
listen_port=51820
kcp_nodelay=2
blast=1
#this is the port of server side's kcptube
destination_port=59000
destination_address=ipvps
#encryption_password=qwerty1234
#encryption_algorithm=AES-GCM
vps:
mode=server
kcp=fast1
inbound_bandwidth=1G
outbound_bandwidth=1G
#Listening on 59000
listen_port=59000
kcp_nodelay=2
#The port number of server side's wireguard
destination_port=51820
blast=1
#if wireguard and kcptube are running on the same server, use localhost, or <IP_ADDRESS>, or ::1
destination_address=localhost
#encryption_password=qwerty1234
#encryption_algorithm=AES-GCM
homepc:
wgclient:
[Interface]
PrivateKey = key
Address = <IP_ADDRESS>/24
MTU = 1300
#DNS = <IP_ADDRESS>
[Peer]
PublicKey = key
AllowedIPs = <IP_ADDRESS>/24
Endpoint = <IP_ADDRESS>:51820
PersistentKeepalive = 20
vps:
wgserver:
[Interface]
PrivateKey = key
ListenPort = 51820
Address = <IP_ADDRESS>/24
MTU = 1300
[Peer]
PublicKey = key
AllowedIPs = <IP_ADDRESS>/24
AllowedIPs = <IP_ADDRESS>/24
AllowedIPs = <IP_ADDRESS>/24
AllowedIPs = <IP_ADDRESS>/24
PersistentKeepalive = 20

First of all, it needs to be explained that kcptube is not specifically designed for high-traffic scenarios, but primarily for reducing latency.
However, the speed you measured is really too low, which suggests that either your ISP is strictly throttling UDP traffic or the kcptube bandwidth settings are incorrect.
I'm not sure how far the discussion and research on KCP has gone in the English world, so I'll quote and use the Chinese explanation in the following part.
KCP本身有一个快速重传机制，用于对抗丢包、降低延迟。中文世界一直流传着这么一个说法：比TCP浪费10%-20%的带宽。kcptube在使用该机制时会相对激进，尤其是fast模式，造成的流量浪费一点都不少，最高可达50%。
为了尽可能减少流量浪费，kcptube提供了带宽设置选项。如果该选项的值高于线路实际带宽，那就会造成kcptube以为当前线路丢包率太高，从而发送更多的冗余数据包，最终恶性循环导致有效流量过低。
所以，请把inbound_bandwidth以及outbound_bandwidth的值设置成低于或等于当前线路的实际带宽数值。
另一方面，由于kcptube激进地尝试降低延迟，会导致高流量时CPU使用率偏高，并且最大的流量值会受到CPU性能的限制。
如果调整过两个bandwidth设置后，流量速率还是不高的话，建议使用另一个工具：udphop，并开启FEC选项。
对于wireguard组网而言，udphop是更好的选择。
The Chinese Text to English:
KCP itself has a fast retransmission mechanism to combat packet loss and reduce latency. There is a saying in the Chinese world that it wastes 10%-20% more bandwidth than TCP. kcptube is relatively aggressive when using this mechanism, especially in ‘fast’ mode, causing a lot of bandwidth waste, up to a maximum of 50%.
In order to minimize bandwidth waste, kcptube provides bandwidth setting options. If the value of these options are higher than the actual bandwidth of the broadband line, they will cause kcptube to believe that the loss rate of current line is too high, and thus send more redundant data packets, which will eventually lead to a vicious cycle of low effective throughput.
Therefore, please set the values of inbound_bandwidth and outbound_bandwidth to be lower than or equal to the actual bandwidth of the current line.
On the other hand, due to kcptube's aggressive attempts to reduce latency, it will lead to high CPU usage under high traffic conditions, and the maximum throughput will be limited by CPU performance.
If the traffic is still not high after adjusting the two bandwidth settings, it is recommended to use another tool: udphop, and enable the FEC option.
udphop is a better choice for wireguard networking.

I've also explained in 低延迟与大流量
English Version is not ready yet. You might need to use AI translation or some other translator.


remove enve dependency

Codecov Report

Merging #73 (1b95d53) into main (4ba5fb7) will decrease coverage by 0.88%.
The diff coverage is 32.43%.

@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
- Coverage   76.55%   75.67%   -0.88%
==========================================
  Files          16       16
  Lines        4845     4937      +92
  Branches     1026     1038      +12
==========================================
+ Hits         3709     3736      +27
- Misses        796      855      +59
- Partials      340      346       +6

Files
Coverage Δ

...nsys/dynamicreporting/core/utils/report_objects.py
85.59% <ø> (ø)

...ynamicreporting/core/utils/report_remote_server.py
54.99% <75.00%> (ø)

.../ansys/dynamicreporting/core/utils/report_utils.py
68.47% <30.84%> (-9.79%)
:arrow_down:


Make list of all the supported blocks so far.

[ ] Make a list of all the supported blocks (maybe also configurations)
[ ] Link to it from the README.md
[ ] Add some code snippets to the README.md

Added a basic version of this for now to the README.
Might be interesting to add a more rounded version later.


⚠ GitLab Server has degraded performance

In 37491fe, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 15178 ms

Resolved: GitLab Server performance has improved in b9c9aa2.


Fix links in root readme

I noticed that both README files in the root directory and the release-notes directory have their content somewhat synced.
Some links in the root one are wrong, which this PR aims at addressing.

Hi @mderriey - thanks for noting this, I fixed them yesterday but unsure what happened. They're showing up correctly now so I'm going to close this PR. If you find anything else amiss, let me know!


Actual behavior (Required!)
Expected behavior (Required!)
The steps to reproduce actual behavior (Required!)

...
...
...

Environment (Required!)

OS:
Vim version:
Neovim version:

Q&A

Yes, I tried minimal .vimrc configuraion.
Yes, I have enabled logs with JCdebugEnableLogs and can put here content of JCdebugGetLogContent command, if you need.
Even, if you wish, I can set g:JavaComplete_JavaviLogLevel to 'debug', then set g:JavaComplete_JavaviLogDirectory, and put here server logs, too.

Screenshot (Optional)
The output of :redir and :message (Optional)

You need to install or build vim or neovim with python support. Search the web.
You can run vim --version and find there +python entry.


Implement prototype auto-decompress AttributeTransfomer, #4979

#4979

I think it should be implemented in a separate plugin.
We don't want to add Bouncycastle as a mandatory DBeaver CE dependency.
You also don't need to modify appstore.feature configuration. It is specifically designed for Windows Store.
What is the reason to add tukaani.xz as a dependency?

Bouncycastle? I assume you mean commons-compress.
tukaani.xz is an optional dependency of commons-compress for XZ support.
Sadly brotli is not available as an eclipse osgi package, or it would be added as well.
Yes I agree, this is only a PoC and I created the pull request to provide the skeleton to base something on, mostly showing that this feature is reasonably easy to implement.
In the Issue I stated that the extension APIs this uses are not the best place to do this, as it also determines how the contents of the compressed blob are interpreted. (afaict)
I have no clue about the codebase overall, and I didn't  see an extension point for AttributeTransformers for a plugin, so I went for the most direct place to put it.

Please ignore bouncycastle comment, it is an optional dependency already.
Tukaani is already included so it is also not a problem.
I lake general idea. However getDisplayString override isn't the primary goal of such a transformer. It has to provide access to uncompressed binary data in a stream form (it could be a picture, or xml or whatever). To view it in the value panel.
Function getValueFromObject in the value handler usually handles this.


FeatureRequest: Decompress Draco compressed file.

I want to decompress gltf/glb files because Blender can only import uncompressed files.
I did not find a decompression command in your tool. Now I can only decompress through the threejs editor, it's a bit troublesome.
Could you please add decompress function to your cli tool?

@gonnavis we're tracking this in https://github.com/CesiumGS/gltf-pipeline/issues/382 so I'll close this as a duplicate. Thanks for the suggestion, I included a link to this issue there.


Simpler and more efficient cleanConnections

I was glancing through the code and was interested by this function.
I think it can be simplified in this way -- even if the connection is blocking on a Remove/AddTrack at the instant, the peer connection still signaled it closed so I think it'd be appropriate to boot it out of the list instantly? Which as far I as I see is only used for telemetry at the moment. (A prometheus counter might actually be what you want 🙂 )
Also, I noticed you were using order-preserving delete, which required reallocating, or at least copying, the entire list if you removed the first element. But you don't need to preserve order, so you can do a simple cheap swap instead.

I was glancing through the code and was interested by this function.
I think it can be simplified in this way -- even if the connection is blocking on a Remove/AddTrack at the instant, the peer connection still signaled it closed so I think it'd be appropriate to boot it out of the list instantly? Which as far I as I see is only used for telemetry at the moment. (A prometheus counter might actually be what you want slightly_smiling_face )
Also, I noticed you were using order-preserving delete, which required reallocating, or at least copying, the entire list if you removed the first element. But you don't need to preserve order, so you can do a simple cheap swap instead.

Hi! Thank you for this any your explanation. Essentially the count of the connection list will be pushed to everyone connected to the websocket in order to be used as the "viewer" count. I agree that the list cleaning function could be simplified and your code looks great! Thank you!

I was glancing through the code and was interested by this function.
I think it can be simplified in this way -- even if the connection is blocking on a Remove/AddTrack at the instant, the peer connection still signaled it closed so I think it'd be appropriate to boot it out of the list instantly? Which as far I as I see is only used for telemetry at the moment. (A prometheus counter might actually be what you want slightly_smiling_face )
Also, I noticed you were using order-preserving delete, which required reallocating, or at least copying, the entire list if you removed the first element. But you don't need to preserve order, so you can do a simple cheap swap instead.

Hi! Thank you for this any your explanation. Essentially the count of the connection list will be pushed to everyone connected to the websocket in order to be used as the "viewer" count. I agree that the list cleaning function could be simplified and your code looks great! Thank you!

If you just need a counter, I suggest using a sync/atomic counter (or a normal uint and a sync.Cond if you need signalling as well).

If you just need a counter, I suggest using a sync/atomic counter (or a normal uint and a sync.Cond if you need signalling as well).


Agm refactor content app org

Summary of changes
Asana Ticket: TICKET_NAME

Renames content and Content to cms and CMS (app namespace change)
Moves page templates into page subfolder (for those currently supported)
Moves paragraph templates to subfolder
Adds Views specifically for Paragraph, Content Page, and generic Content
Renders Basic pages using new render pipeline: CMS Controller -> Page.html container -> _Page.html wrapper -> _generic.html content output

Closing to squash and submit again