Separate steps for air-gapped contribution

Separate the contribution steps for supporting air-gapped contribution.

Tested on local and dev sequencer.
rootwarp@Mindhack2021 ~/g/d/dsrv-kzg-ceremony separate > ./index.js try-contribute -s https://kzg-ceremony-sequencer-dev.fly.dev
2023-03-31 18:41:59 [ info ] Start try contribution
? Which method do you prefer for authentication? github
https://github.com/login/oauth/authorize?response_type=code&client_id=3c984b3a235318fe6d23&state=eyJyZWRpcmVjdCI6bnVsbH0&redirect_uri=https%3A%2F%2Fkzg-ceremony-sequencer-dev.fly.dev%2Fauth%2Fcallback%2Fgithub

? Input your session ID:  5f05f72c-7f6a-415c-84d7-fc4966dbacbb
2023-03-31 18:42:15 [ info ] Try and Wait...
2023-03-31 18:42:20 [ info ] Previous contribution is written on contribution_5f05f72c-7f6a-415c-84d7-fc4966dbacbb.json
rootwarp@Mindhack2021 ~/g/d/dsrv-kzg-ceremony separate > ./index.js execute-ceremony -f ./contribution_5f05f72c-7f6a-415c-84d7-fc4966dbacbb.json
2023-03-31 18:42:47 [ info ] Start Execute Ceremony
2023-03-31 18:42:47 [ info ] Run Ceremony...
2023-03-31 18:42:47 [ info ] Decoding contributions....
2023-03-31 18:42:57 [ info ] Update Power of Tau...
2023-03-31 18:42:57 [ info ] Run contribute worker
2023-03-31 18:42:57 [ info ] Run contribute worker
2023-03-31 18:42:57 [ info ] Run contribute worker
2023-03-31 18:42:57 [ info ] Run contribute worker
2023-03-31 18:43:18 [ info ] Receive new contribution
2023-03-31 18:43:31 [ info ] Receive new contribution
2023-03-31 18:44:01 [ info ] Receive new contribution
2023-03-31 18:44:58 [ info ] Receive new contribution
2023-03-31 18:44:58 [ info ] Update Witnesses...
2023-03-31 18:44:58 [ info ] Encoding...
2023-03-31 18:44:58 [ info ] New contribution is written on new_contribution.json
rootwarp@Mindhack2021 ~/g/d/dsrv-kzg-ceremony separate > ./index.js contribute -f ./new_contribution.json --session_id 5f05f72c-7f6a-415c-84d7-fc4966dbacbb
2023-03-31 18:45:11 [ info ] Submit contribution
2023-03-31 18:45:11 [ info ] Send contributions
^C
rootwarp@Mindhack2021 ~/g/d/dsrv-kzg-ceremony separate > ./index.js contribute -f ./new_contribution.json --session_id 5f05f72c-7f6a-415c-84d7-fc4966dbacbb -s https://kzg-ceremony-sequencer-dev.fly.dev
2023-03-31 18:45:28 [ info ] Submit contribution
2023-03-31 18:45:28 [ info ] Send contributions
2023-03-31 18:45:36 [ info ] Successfully contributed!
2023-03-31 18:45:36 [ info ] {
  receipt: '{"identity":"git|9974087|rootwarp","witness":["0xa549da1086523c082c695b7d8493f1a5c7496f5f8f248e7bcd2d8bab428cdaaa6afec72d77ce1cd21e4452f5b5e3b1e815f03a1cdc32707d101ceb5fb466e404f7070c58263eea1a149b3418e4ce17798062a28edd20c26b6d180b3ee0463bb8","0x8c86970986c9d4e209157a68d779f98fb028b443944a817f81e0230b8318387ede16d564dfe28f10db14752671efdf0a096be81f6252b2819335e2c3ea808cd5e8d210c0ea2af4beeee8fdc5d0ea6eec1eacdd617c8bec0946840192d7ab9f0b","0x863e346954dad44423e6c052da698f9fe0ebb140943187ab9112bd92692b9b0ff00d2b5750ec729a79e6b10b673d96f30a0e9afc3eed090e20208f645f4187e63df7d82290a20a393dc26bbec6bd10914ec1c036b85eec84da3c4e3681a252b3","0xa9a77bd62ff54103048ea46e5850439683c55926a9a9102c59309ae47f3131055dc9bf3187a7c0b63ed60891f5c17b050c868de03c44b739b2c19d11c2bdfba8a2a7d5dbcb18be8e21c55d0d588ec68a7e72e801284c1ebe6a7511d35d32b00c"]}',
  signature: '1ccf296ecb18683756fbd56445039dc5c045e58a931eed77f4f9f10326bc20e45462d978caeab01f5c0cb4b76b15f4108bc8d5a934b0bc52a0fc24bd854d98231c'
}
rootwarp@Mindhack2021 ~/g/d/dsrv-kzg-ceremony separate >

Tested
hg  ~/workspace/kzg-ceremony/czg-keremony   separate  ./index.js try-contribute -s https://kzg-ceremony-sequencer-dev.fly.dev
2023-04-03 15:07:05 [ info ] Start try contribution
? Which method do you prefer for authentication? github
https://github.com/login/oauth/authorize?response_type=code&client_id=3c984b3a235318fe6d23&state=eyJyZWRpcmVjdCI6bnVsbH0&redirect_uri=https%3A%2F%2Fkzg-ceremony-sequencer-dev.fly.dev%2Fauth%2Fcallback%2Fgithub

? Input your session ID:  1192432f-8f3d-48f3-9954-1cb934f659f3
2023-04-03 15:07:23 [ info ] Try and Wait...
2023-04-03 15:07:25 [ info ] Previous contribution is written on contribution_1192432f-8f3d-48f3-9954-1cb934f659f3.json
 hg  ~/workspace/kzg-ceremony/czg-keremony   separate  ./index.js execute-ceremony -f contribution_1192432f-8f3d-48f3-9954-1cb934f659f3.json
2023-04-03 15:08:06 [ info ] Start Execute Ceremony
2023-04-03 15:08:06 [ info ] Run Ceremony...
2023-04-03 15:08:06 [ info ] Decoding contributions....
2023-04-03 15:08:23 [ info ] Update Power of Tau...
2023-04-03 15:08:23 [ info ] Run contribute worker
2023-04-03 15:08:23 [ info ] Run contribute worker
2023-04-03 15:08:23 [ info ] Run contribute worker
2023-04-03 15:08:23 [ info ] Run contribute worker
2023-04-03 15:08:58 [ info ] Receive new contribution
2023-04-03 15:09:31 [ info ] Receive new contribution
2023-04-03 15:10:28 [ info ] Receive new contribution
2023-04-03 15:12:41 [ info ] Receive new contribution
2023-04-03 15:12:41 [ info ] Update Witnesses...
2023-04-03 15:12:41 [ info ] Encoding...
2023-04-03 15:12:41 [ info ] New contribution is written on new_contribution.json

 ✘ hg  ~/workspace/kzg-ceremony/czg-keremony   separate  ./index.js contribute -f ./new_contribution.json --session_id 1192432f-8f3d-48f3-9954-1cb934f659f3 -s https://kzg-ceremony-sequencer-dev.fly.dev
2023-04-03 15:13:49 [ info ] Submit contribution
2023-04-03 15:13:49 [ info ] Send contributions
2023-04-03 15:14:03 [ info ] Successfully contributed!
2023-04-03 15:14:03 [ info ] {
  receipt: '{"identity":"git|42402378|kim201212","witness":["0xa7390fe366ba2dfebf5d9696be1cc0b00100d7b956d4b2901934bc1e469084066e111376551b37cbe508052ab58ffbd1051f2c1d988ab745528bf047443d4da7980e9f321fe38a247fd4e87adaf60b1a94bdb8ecd575542edd263c99780e979f","0xa67baf224530c6bc9ee1f4038d84084baabe2cb2b1d1c2d6f635ef8ecfc5d9fba600f8f309caac87a90ed55b92057faf05b8267d40d55a1dc6f1a7ff4739eef0793819551dad41d2687e95cb252da1f969992567b92d58a6d3afa11691973521","0x845485213fcd134783abc131f33f715e946d87e366a42b2cb1c7b301d5cdf0ba8a3a072cbae4b3b5438a148831d346210c39b721f1e8986f8eef4dedd9cf3a6ffa5c639071ccb77fb32b4dbfedb498b7cbe4cc6bfa11089e3176a73c1465cf16","0xa9433fb77f8f81e95816f6731aea2d4bfeb1dcb9645a1e70ca6e5d3cdf3e62a62e9d6b347ec329b947805677c5de4faa194327b30b496d1f40a4e981b8222ac7910e51a516651e6f56e59a2f26a273bb42129eb419f25dc0fe7b28f8bdec7068"]}',
  signature: '6089a47309ff110ed22a061944781423fb9e9879a36ad14d846a1a82d4d1bce568dd31b53d4d2b90bcacc7ad554504aa6b87552858d88d6cb1fac71abf4895bb1c'
}


linkedin-to-jsonresume

linkedin-to-jsonresume

@hjunsun What is this issue about? Do you need help with something?

@hjunsun Please provide more details if you would like to reopen this issue.


Add update option to create custom reproducers

Extends the squad-create-reproducer to allow updating of reproducers via an update flag alongside a custom-command or ltp-tests flag specifying the commands or LTP tests that should be run by the reproducer.
Builds on https://github.com/Linaro/squad-client-utils/pull/19

Code ready for review @roxell


Update license attribute

specifying the type and URL is deprecated:
https://docs.npmjs.com/files/package.json#license
http://npm1k.org/

Can we please merge this? :)


Archi-Ubuntu.sh has a wrong line endings (CRLF)

Version of Archi, Operating System
4.6 on Linux (Ubuntu 19.10)
Expected Behaviour
Archi should start normally when executing it with Archi-Ubuntu.sh
Actual Behaviour
An error is shown and Archi is not starting:
$ ./Archi-Ubuntu.sh
bash: ./Archi-Ubuntu.sh: /bin/bash^M: bad interpreter: No such file or directory

Steps to Reproduce the Behaviour

Download Archi-Linux64-4.6.0.tgz
Extract the archive: tar xvzf Archi-Linux64-4.6.0.tgz
Run Archi-Ubuntu.sh

The Bash script has wrong line endings:
$ file Archi-Ubuntu.sh
Archi-Ubuntu.sh: Bourne-Again shell script, ASCII text executable, with CRLF line terminators

A workaround is to run: dos2unix Archi-Ubuntu-sh

Hi, this is already be fixed for the next version (4.7 EA 3 dues out soon)

I'll close this as 4.7 EA3 has the fix.


Job doesn't exit when EXIT signal received

http://<IP_ADDRESS>/yarn/<IP_ADDRESS>:8188/applicationhistory/logs/<IP_ADDRESS>:8041/container_e12_1540300624969_1128_01_000002/container_e12_1540300624969_1128_01_000002/yuqian
This job doesn't exit when [DEBUG] Docker container exit handler: EXIT signal received in docker container, exiting ...

duplicate issue #1477

They should be 2 cases, in this issue, docker container suicide successfully, but in #1477, docker daemon is in error status and docker can't exit by itself.
Sync with @qyyy , he will keep this job if meet again.


support all comparison and aggregation functions on extended json fields

Since Extended JSON fields may contain any data users may reasonably want access to all comparison and aggregation operations. MongoDB is designed to accommodate functions that don't make sense on all values they are applied to. I included a native query that produces a collection of mixed data types, and some integration tests on that data.
I also updated the dev shell to pull the cli from the nix flake that @TheInnerLight helpfully set up.

Looks good, apart from the typo in the changelog. We should also allow order_by on extended json fields, but that could be in a separate PR.

Good point @dmoverton. This PR does add order_by functions, but I didn't put in a test for that. I just added one.


Range / Slider request

Hi,
I really need a component for range, but semantic only have implementations with jquery.. It is possible to kindly add something similar to this: https://github.com/tyleryasaka/semantic-ui-range
Kind regards,
Rui

Hi Rui,
This is something that is being worked on currently, and there is an active PR for it (#173), so you can track progress there 😄

Oh nice! Can't wait to try it! in the meanwhile I had to use something.. and end up choosing nouislider but I prefer not having many external modules+css, so, as soon as this is ready and merged, I'll test it right away!
Thank you,
Rui


[Python Lambda SDK] Improve sampling logic

Description

Related issue https://linear.app/serverless/issue/SC-1145#comment-40b6007c
Catching up on the recent changes for sampling of requests
Move is_api_event to its own module

Testing done

Unit/integration tested

@medikoo ready for review 🙏


Wip main page hosptial list

Modified the view file to delete the hash from showing, added extra information for each hospital, and added a search bar.

LGTM


<EMAIL_ADDRESS>

Tested and everything seems to work alright with the new version.

Win7 on VirtualBox:

macOS Sierra 10.12.2

@feross Should we change the package version to 0.19.0?
The rest looks good :)
Did you test it on Linux?
Cheers!

Thanks for the testing :) I already tested Mac, but a second pair of eyes doesn't hurt. 👍

Should we change the package version to 0.19.0?

Isn't the latest 0.17.0? At least that's what I see! https://www.npmjs.com/package/material-ui

I was referring to the version of WebTorrent Desktop.
It's currently 0.18.0 and I was wondering if after this changes it might get a bump on the minor.
But maybe it's ok not to update the version at all:
MAJOR version when you make incompatible API changes,
MINOR version when you add functionality in a backwards-compatible manner, and
PATCH version when you make backwards-compatible bug fixes.

What are your thoughts on it?
Thanks!

I'm also thinking about the changelog.
The plan is to include other PRs and then release version 0.19.0 right?
Thx!

I was referring to the version of WebTorrent Desktop.

Ahh, yes. We usually bump the version in package.json right before doing the release. That's so that there's a single commit like https://github.com/feross/webtorrent-desktop/commit/0ee7e80fd2dffb8520e9bb552009b01edfa65369 where the version changes. Also, we add a git tag there too, so git checkout v0.18.0 checks you out to the specified version. 👍

The plan is to include other PRs and then release version 0.19.0 right?

Yes, I'd love to catch up on all the outstanding PRs, and do a release with all those changes!

Cool! Thanks for the explanation dude!


Remove python 3.5 support. Fixes #112

Failures due to #137

this would also need:

remove async_generator dependency as mentioned in #112
update minimum Python version in README

Pull Request Test Coverage Report for Build 99

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.492%

Totals

Change from base Build 97:
0.0%

Covered Lines:
466

Relevant Lines:
488

💛  - Coveralls

Done

[x]   remove async_generator dependency as mentioned in #112
[x]  update minimum Python version in README
[x]  update python_requires in setup.py

Hey @belm0 is there anything left to do to get this merged?

remove async_generator dependency as mentioned in #112

It means removing all references to this 3rd party module:
https://github.com/HyperionGray/trio-websocket/blob/d878659a753bb4ecc97f5b39485b9af77fdf639a/trio_websocket/_impl.py#L9
https://github.com/HyperionGray/trio-websocket/blob/be300518ff33b18d8c2d7a8b85ad3bd15023457a/tests/test_connection.py#L40
https://github.com/HyperionGray/trio-websocket/blob/b798551ad562ef24000859c6710665818c77135a/setup.py#L42
In Python 3.6, the yield keyword can be used normally within an async generator, and so async_generator and yield_ are not needed.  And use asynccontextmanager from the stdlib.

In Python 3.6, the yield keyword can be used normally within an async generator, and so async_generator and yield_ are not needed. And use asynccontextmanager from the stdlib.

FYI asynccontextmanager requires 3.7 -- see https://docs.python.org/3/library/contextlib.html#contextlib.asynccontextmanager  "New in version 3.7."

FYI asynccontextmanager requires 3.7 -- see https://docs.python.org/3/library/contextlib.html#contextlib.asynccontextmanager "New in version 3.7."

Sorry, my oversight!
For this PR, removing use of the @async_generator decorator and replacing yield_ with yield look straightforward.  I'd like to do that, and restore the dependency on async_generator which is needed for the asynccontextmanager shim.
(Later we can consider implementing the async context manager manually to remove the dependency on this lib.)

In case it's helpful - I extracted asynccontextmanager from latest cpython and removed some unnecessary abstraction. Tested in Python 3.6.11 and it seems to work. Only issue is that by default my linter complained about Async context manager 'async_generator' doesn't implement __aenter__ and __aexit__ (not-async-context-manager) when using it to wrap a function. The code itself works though.
from functools import wraps

class _AsyncGeneratorContextManager:
    """Helper for @asynccontextmanager."""

    def __init__(self, func, args, kwds):
        self.gen = func(*args, **kwds)
        self.func, self.args, self.kwds = func, args, kwds
        # Issue 19330: ensure context manager instances have good docstrings
        doc = getattr(func, "__doc__", None)
        if doc is None:
            doc = type(self).__doc__
        self.__doc__ = doc
        # Unfortunately, this still doesn't provide good help output when
        # inspecting the created context manager instances, since pydoc
        # currently bypasses the instance docstring and shows the docstring
        # for the class instead.
        # See http://bugs.python.org/issue19404 for more details.

    async def __aenter__(self):
        try:
            return await self.gen.__anext__()
        except StopAsyncIteration:
            raise RuntimeError("generator didn't yield") from None

    async def __aexit__(self, typ, value, traceback):
        if typ is None:
            try:
                await self.gen.__anext__()
            except StopAsyncIteration:
                return
            else:
                raise RuntimeError("generator didn't stop")
        else:
            if value is None:
                value = typ()
            # See _GeneratorContextManager.__exit__ for comments on subtleties
            # in this implementation
            try:
                await self.gen.athrow(typ, value, traceback)
                raise RuntimeError("generator didn't stop after athrow()")
            except StopAsyncIteration as exc:
                return exc is not value
            except RuntimeError as exc:
                if exc is value:
                    return False
                # Avoid suppressing if a StopIteration exception
                # was passed to throw() and later wrapped into a RuntimeError
                # (see PEP 479 for sync generators; async generators also
                # have this behavior). But do this only if the exception wrapped
                # by the RuntimeError is actully Stop(Async)Iteration (see
                # issue29692).
                if isinstance(value, (StopIteration, StopAsyncIteration)):
                    if exc.__cause__ is value:
                        return False
                raise
            except BaseException as exc:
                if exc is not value:
                    raise

def asynccontextmanager(func):
    @wraps(func)
    def helper(*args, **kwds):
        return _AsyncGeneratorContextManager(func, args, kwds)
    return helper

@AutomatedTester would you open this PR for maintainer edits?
I have some follow-up commits pending for changes in master and using native async generators.

@AutomatedTester would you open this PR for maintainer edits?
I have some follow-up commits pending for changes in master and using native async generators.

It showing as already done


Test isSuccessful

Changes

isSuccessful is an unexported function, so it did not have tests
per the in guidelines. The recommendation is to not
test unexported function, and if needed to move the function into
a package, test it then export the function. However, this is hard
to do for the resources package because of cyclic dependencies. We
already have tests for another unexported function - isFailure -
and discussions to make exceptions for that guideline are ongoing.
In this change, we add tests for isSuccessful to ensure it works
as expected, as we prepare to make changes to the function to support
Matrix.
/kind misc
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[n/a] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Release notes block below has been filled in (if there are no user facing changes, use release note "NONE")

Release Notes

NONE

/lgtm

/assign


Using Background Sync

Existing content:
Intro: https://developers.google.com/web/updates/2015/12/background-sync?hl=en
Explainer: https://github.com/WICG/BackgroundSync/blob/master/explainer.md
Dean's intro: https://ponyfoo.com/articles/backgroundsync
Community doc: https://dbwriteups.wordpress.com/2015/11/16/service-workers-part-4-background-sync/
I think what we need for /web is a 101 guide. I don't feel we've had enough experimentation with BGSync yet to warrant 201/301 style content just yet.

We haven't seen anyone implement this just yet.

This should live as an article in Instant and Offline


Syntax highlighting is broken in many languages and it spreads around the file

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Syntax highlighting appears to be broken in a number of different languages. I had made this post in this repo earlier https://github.com/atom/language-javascript/issues/676 about an issue in the JS language after updating to 1.47 https://github.com/atom/atom/releases/tag/v1.47.0 , but it got moved over to the JS language repo. I have gone through all the releases from 1.41 - 1.49 (current) and 1.47 is the first release where this issue is present (in both normal versions, and safe mode versions).
The effected languages that I have seen since 1.47 have been:

JavaScript
TypeScript
HTML
Ruby
Rust (edited)

But who knows how many other languages are affected. This is defiantly not an isolated issue within a language as all 4 of these ones broke on release 1.47
The thing that makes this bug so strange (I go into detail a bit more on my previous issue above), is that if lines around the effected line are edited, the broken syntax highlighting spreads around the file. And if the code is copy-pasted into an empty file, or opened for the first time, multiple lines around the effected line all break.
Steps to Reproduce

Open any version of atom past release 1.47
Set the language to one of the 4 mentioned above
Copy-paste the provided code for said language (In additional information section)

Expected behavior:
The syntax highlighting working normally all the time.
Actual behavior:
Starting at 1.47 onwards, when the syntax highlighting breaks, it makes the affected line go white. But when lines around the affected line are edited, the broken syntax highlighting seems to spread around the rest of the file.
Reproduces how often:
100% of the time after 1.47
Versions
1.46 Issue not present:
Atom    : 1.46.0
Electron: 4.2.7
Chrome  : 69.0.3497.128
Node    : 10.11.0
apm  2.4.3
npm  6.2.0
node 10.2.1 x64
atom 1.46.0
python 2.7.10
git 2.21.1
1.47 Issue now present:
Atom    : 1.47.0
Electron: 5.0.13
Chrome  : 73.0.3683.121
Node    : 12.0.0
apm  2.4.5
npm  6.2.0
node 10.2.1 x64
atom 1.47.0
python 2.7.10
git 2.21.1
1.49 Current as of time of writing:
Atom    : 1.49.0
Electron: 5.0.13
Chrome  : 73.0.3683.121
Node    : 12.0.0
apm  2.5.0
npm  6.14.5
node 10.20.1 x64
atom 1.49.0
python 2.7.10
git 2.21.1
macOS 10.14.6
Additional Information
Code snippets for each language.

Javascript & TypeScript:

let ele = document.getElementById("test");

ele.innerHTML = ``;
// Comment out the line above from the start, then select all, cut, then past. Highlighting now works
// Highlighting worked fine in 1.46

The issue in this one is the backticks on line 3.

HTML:

<!DOCTYPE html>
<html lang="en" dir="ltr">
	<head>
		<meta charset="utf-8">
		<script></script>
		<!-- Comment out the line above from the start, then select all, cut, then past. Highlighting now works -->
		<!-- Highlighting worked fine in 1.46 -->
		<title></title>
	</head>
</html>

The issue in this one is the entire <script> tag

Ruby:

def test(string)
	s = StringScanner.new(string)

	while !s.eos?
		s.scan(/\(/)

		s.scan(/\)/)

		# Comment out lines 5 and 7, then select all, cut, then past. Highlighting now works
		# Highlighting worked fine in 1.46
	end
end

The issue in this one is the REGEX in the scan looking for opening and closing parentheses. Other REGEX's appear to work fine.
Edit: Added Rust: (https://github.com/atom/atom/issues/20884)
use crate::checks::{self, *};

pub fn foo() -> Result<(), Box<dyn error::Error>> {
    let string = "foo";

    Ok(())
}

and (https://github.com/atom/atom/issues/20897)
#[x(y = z)]
struct W();

let x = "hello";

Side note
I was able to find a workaround for the JavaScript and TypeScript languages which is at the bottom of my previous issue https://github.com/atom/language-javascript/issues/676 . I haven't found a workaround for the HTML or Ruby languages though

Thanks @CyborgSemon for reporting this issue. This has been added to our radar and will be looked into.

I see this has been an issue for a while, and I'm not the first to notice it
https://github.com/atom/atom/issues/20897
https://github.com/atom/atom/issues/20884
Ive edited my issue with the addition of Rust as an affected language

Has any progress been made on this bug? I'm still stuck on 1.46 because of it.

Some more detail on the Ruby-specific case discussed in the language-ruby issue; regexs with special characters such as \d or \w trigger it, regexs with interpolation (/#{varaible}/) trigger it, and simpler regexs do not.

More info: the syntax highlighting for ruby appears to break using language-ruby but not language-ruby-on-rails.

@darangi Can you provide an update on how this are progressing with this issue? Atom has had quite a few updates since it was introduced and is still causing issues.

Syntax coloring is now broken to a degree which renders the editor useless for more than 3 months. I'm afraid, this is a symptom of Atom being neglected in favor of VSCode. Sad to see Atom go down the drain, however, guess it's time to move on and look for alternatives such as Sublime, Nova etc. So long!

More info: the syntax highlighting for ruby appears to break using language-ruby but not language-ruby-on-rails.

Do you know how to change it so language-ruby-on-rails will be used by default instead of language-ruby?

More info: the syntax highlighting for ruby appears to break using language-ruby but not language-ruby-on-rails.

Does anyone know how to change it so language-ruby-on-rails will be used by default instead of language-ruby?
CC @ozydingo

Add this to ~/.atom/config.cson:
    customFileTypes:
      "source.ruby.rails": [
      	"rb"
      ]

@darangi I hate to ask again because I don't want to be "that guy", but can we please get an update on this? It's still causing issues.

There is work in progress by the maintainers to fix this issue.

@darangi I hate to ask again because I don't want to be "that guy", but can we please get an update on this? It's still causing issues.

On further testing, syntax highlighting works fine on ruby, javascript and html when the atom window is reloaded with the tab with the source code being active. (Reloading: View -> Developer -> Reload window). But then syntax highlighting fails when you close and reopen the tab.
When you turn off tree-sitter parser on the settings, syntax highlighting works fine even when the tab is closed and reopened without reloading the editor.
We are currently working to pin down the bug.

Is there any update on this? This bug is very disruptive.

No updates yet, but just want to report that we'll be prioritizing debugging and fixing this issue with the Tree-sitter highlighting.
Thanks very much @CyborgSemon for isolating it to Atom 1.47 and for the detailed reproduction steps. I will let you know when it is fixed.

Ok, I believe there are two different problems here. One relating to language injections (cases where one language is embedded within another), and also a bug that is specific to the Rust grammar. I have a PR up, #21753, that I think fixes both.


Removing the "acquired_***" version of functions

I think we should remove the "acquired_***" functions.
Instead, what we should do is have acquire_unique_packet_id() return an opaque type that can only be constructed by either acquire_unique_packet_id() or register_packet_id().
Then, the overloads of the existing end-user API functions would either have no packet_id, and therefore internally call acquire_unique_packet_id(), like we do now, or the end-user must call register_packet_id() in order to get a packet_id object (which they can't construct themselves, as noted before).
Making this change would allow us to eliminate >30 different functions, while also reducing the situations where end-users can misuse the library.
Note: I also think that we should separate the packet_id algorithm from the endpoint class, as detailed here: https://github.com/redboltz/mqtt_cpp/issues/223

I don't understand what you mean yet.
acurired_* version is required to reserve packet id. It is heavily used on my broker.
Maybe you don't say just remove them but I can't imagine updated code.
I need code example.

My usecase is

packet_id = acquire_unique_packet_id()
store packet_id to DB
acuired_***(packet_id)

If broker is down between 2 and 3 then,

reboot broker (with clean memory)
read stored packet_id from DB if it exists
register_packet_id(packet_id)
acuired_***(packet_id)

It is a part of exactly once message delivery across broker (cluster) process is down.

There are three end-user api function versions for each message that uses packet_id.
e.g. subscribe.
subscribe(...);
subscribe(packet_id, ...);
acquired_subscribe(packet_id, ...);

I'm saying to do this;
using packet_id = /* some type that can only be constructed by struct endpoint. Has integer packet id internally, has const getter for numeric representation. */

packet_id register_packet_id(...); // Gets packet id object based on the provided id number. Throws exception on failure, or takes reference to boost::system::error_code, or whatever failure mechanism.
packet_id acquire_unique_packet_id(...); // Gets packet id object guaranteed to not overlap with any existing packet ids. Used by end-user only if they need to know the packet id ahead of time.

subscribe(...); // endpoint gets packet id automatically.
subscribe(packet_id, ...); // end-user must get packet id themselves. Neither version returns the packet id, because that would interfere with the concept of completion tokens from https://github.com/redboltz/mqtt_cpp/issues/523

Having packet_id be a type that can only be constructed by struct endpoint makes this safe. End-users can't accidentally provide bad packet_id to packet_id version of function. They must call either acquire_unique_packet_id() or register_packet_id(), or call non-packet_id version of function.
For your code you would do:
packet_id = acquire_unique_packet_id()
store packet_id to DB
subscribe(packet_id, ...)
If broker is down between 2 and 3 then,

reboot broker (with clean memory)
read stored packet_id from DB if it exists
packet_id = register_packet_id(packet_id_number)
// check failure here
subscribe(packet_id, ...)

The difference is that acquired_***(...) is renamed as ***(packet_id), and replaces the existing version of ***(packet_id). End-user can no longer just call function with numeric packet_id and have it auto-registered.
Benefits:

Removes >30 functions from mqtt_cpp, significant reduction in code size.
Easier to do https://github.com/redboltz/mqtt_cpp/issues/523 (Because CompletionToken specifies return type)
Easier for end user to not make mistakes. Their choices go from 1) automatic packet id 2) half-automatic packet id, 3) manual packet_id. -> 1) automatic 2) manual. No half-automatic. Since doing half-automatic is only one function call, they can do that easily.

Ok, you mean remove
bool subscribe(packet_id_t packet_id, ...);

that calls register_packet_id(packet_id) internally.
And rename from
void acquired_subscribe(packet_id_t packet_id, ...);

to
void subscribe(packet_id_t packet_id, ...);

.
If I understanding correctly, that's fine.

Yes. Correct.
And also provide type-safety, to prevent accidental misuse.

Correct me if I'm wrong, but #547 does not include the "type safety" part (changing the type of packet_id_t to prevent implicit construction from integers) that was mentioned above.... was that intended?

Redboltz was opposed to changing the api such that the packet id must be acquired ahead of time by the end user, so I did not add changes to prevent implicit construction to avoid the api break.


Made epsilon change in relation to the protein complexity

With insights into the data that we've been receiving after the latest update we felt it was needed to finally change epsilon from a simple static number into a variable that actually adapts based on the job.
After some analysis we found a clear trend between the average change in loss (energy) per miner update and the amount of total atoms in the protein as seen on the following graph:

Note that both axes are in log scale due to the disparity in scale of the data. It is clear that the changes in loss we observe are increasing with the size of the protein. There is a few reasons we can't simply use the trendline seen in the image to model as epsilon:

The data we currently possess is extremely noisy due to the instability of the network and hence the standard deviation of the dataset is incredibly high
We don't have the time parameter of the simulations, ie we don't know for how long each simulation has been running for. This is important because as simulations converge on the optimal solution their loss will get smaller and smaller.

We decided to take a more conservative approach for now and create a function that will result in an overall lower epsilon, this can be seen in the following graph:

This function was created by going through the normal optimization process but we weighed the points below the trendline more than those above the trendline.
The current epsilon calculation function looks as follows:
epsilon = 7.14819473 * num_atoms + 1.68442317e04
We make an exception for molecules below 100 atoms as we do not have enough data for those and set epsilon to a flat 5000

absolutely excellent first PR @Sarkosos. Really stoked on this work. Merged!


As an automate regression test, I want to test that I can log in with EDL and query the API so that I can confirm EDL login is working

AC:

Automated test runs successfully that logs in a test user with EDL

Zoltan here,
I have a few questions:

Which API endpoints needs to be checked?
Do you want me to develop the test code/ framework here, or can use the existing one?
Where do you want the tests to be added and executed? Jenkins or Github action? (The later only possible of the code is in the same repository)

API endpoint: https://d15gds5czd9p7k.cloudfront.net/
test code/frame work goes here: https://github.com/podaac/swodlr-functional-tests
Github action

Waiting on deployment to UAT in order to run tests from a public client.

requirement documentation of (@joshgarde)

API endpoints
Expected inputs/outputs
test case

@Phoeneix please see  https://github.com/podaac/swodlr/wiki/Getting-Started-with-SWODLR‐API

Pending auth refactor being merged.

Per PODAAC-5480 test implementation:

PR opened on 11/28/23: https://github.com/podaac/swodlr-functional-tests/pull/3

test passed in UAT: https://github.com/podaac/swodlr-functional-tests/actions/runs/7479400696


npm run build hangs

I've run $ npm run build and it just hangs until I quit with the following output:
[root@localhost framework]# npm run build

><EMAIL_ADDRESS>build /vagrant/www/framework
> gulp build

[15:47:15] Using gulpfile /vagrant/www/framework/gulpfile.js
[15:47:15] Starting 'umd-admin'...
[15:47:15] Starting 'umd-framework'...
[15:47:15] Starting 'umd-watch'...
[15:47:16] Finished 'umd-watch' after 641 ms
[15:47:16] Starting 'bundle-lib'...
[15:47:16] Starting 'bundle-leftandmain'...
[15:47:16] Starting 'bundle-react'...
[15:48:30] Finished 'umd-admin' after 1.25 min
[15:48:30] Finished 'umd-framework' after 1.25 min
[15:48:30] Starting 'umd'...
[15:48:30] Finished 'umd' after 19 μs
[15:48:33] Finished bundle-leftandmain.js 99612 bytes written (77.45 seconds)
[15:48:36] Finished 'bundle-leftandmain' after 1.33 min
[15:48:36] Starting 'umd-framework'...
[15:48:36] Starting 'umd-admin'...
[15:48:40] Finished bundle-lib.js 1231832 bytes written (83.91 seconds)
[15:48:52] Finished 'umd-framework' after 16 s
[15:48:52] Finished 'bundle-lib' after 1.6 min
[15:48:52] Finished 'umd-admin' after 16 s
[15:49:11] Finished bundle-react.js 737454 bytes written (115.02 seconds)
[15:49:16] Finished 'bundle-react' after 2 min
[15:49:16] Starting 'bundle'...
[15:49:16] Finished 'bundle' after 13 μs
[15:49:16] Starting 'build'...
[15:49:16] Finished 'build' after 9.26 μs

I'm not sure, but this might be because npm-watch is part of the build task, does this just wait forever? Perhaps the watch task should be removed from the build task?

Yeah it's not hanging, it's watching for changes. This has been changed in a working branch and will be pull requested next weekish.
The new behaviour will be...
npm run build does one one time build, everything is minified, no source maps, no watching. It's the 'prod' duild.
npm run build --development generates source maps, code is not minified, and it will watch for changes.

so which would we want to go into the repo?

npm run build should go into the repo.

Closing since this isn't a bug.


The functions roc_auc_score and prc_auc_score don't work as expected

These two functions throw similar errors, at least in my case. The following example fails for these two functions, but has no problem with other functions for classification metrics like accuracy_score.
from verticapy.machine_learning.metrics.classification import  roc_auc_score
from verticapy.datasets import load_iris
from verticapy.learn.ensemble import RandomForestClassifier
iris_vd = load_iris()
model = RandomForestClassifier("multi_m3", max_features="max", max_depth=3, n_estimators=3, sample=1)
model.fit(iris_vd, ["SepalLengthCm", "SepalWidthCm", "PetalLengthCm", "PetalWidthCm"], "Species")
model.predict(vdf=iris_vd, X=["SepalLengthCm", "SepalWidthCm", "PetalLengthCm", "PetalWidthCm"], name="y_score")
roc_auc_score(y_true = "Species",  y_score = "y_score",  input_relation = iris_vd, average = "macro", labels = model.classes_)

this error is incorrect as y_score should be the list of the different probabilities.
Here is the corrected example:
from verticapy.machine_learning.metrics.classification import roc_auc_score
from verticapy.datasets import load_iris
from verticapy.learn.ensemble import RandomForestClassifier
iris_vd = load_iris()
model = RandomForestClassifier("multi_m3", max_features="max", max_depth=3, n_estimators=3, sample=1)
model.drop()
model.fit(iris_vd, ["SepalLengthCm", "SepalWidthCm", "PetalLengthCm", "PetalWidthCm"], "Species")
model.predict_proba(vdf=iris_vd, X=["SepalLengthCm", "SepalWidthCm", "PetalLengthCm", "PetalWidthCm"])
roc_auc_score(y_true = "Species",
              y_score = ['"randomforestclassifier_multi_m3_irissetosa"', '"randomforestclassifier_multi_m3_irisversicolor"', '"randomforestclassifier_multi_m3_irisvirginica"'],
              input_relation = iris_vd,
              average = "macro",
              labels = model.classes_)

However, the error message is incorrect, I'll do a PR to change it.


Create toolkit style pages for each content type

Currently, clicking any of the content type boxes takes the user directly a list of search results. This means we don't have a place to properly market the resources in that content type. For example, for SEO purposes and demonstrating member value, it would be helpful to be able to promote our academic programs list as being one of the most comprehensive lists of academic programs related to sustainability.
To enable this, create a landing page for each content type that includes an editable text box, multiview ad, list of new resources in that content type, and some kind of interactive visual representation of the resources in that content type. This could be a simple word cloud based on tags in that content type or a chart. In the Academic Programs content type for example, the visual could be   a clickable bar chart showing the total number of academic programs in each discipline.

I wonder if for the time-being, we can add an editable text section at the top of the Resources sections of the content type views?

Here is an draft mock-up for the Content Type landing pages:
https://docs.google.com/presentation/d/1pWWBB9aqei87ARj2O1C5k25aHAqotxkul31XJoxlAVk/edit#slide=id.g115be3eb5d_0_137

@juliandsmith -
I reviewed the Content Type landing pages more closely and my recommendations are below - for the most part, they follow the suggestion to display either Discipline OR topic based on whether Discipline is required, with a few caveats:

Academic Programs - show Discipline only
Case Studies - show Topic only (okay with also showing Discipline)
Conference Presentations - show Topic only
Course Materials - show Discipine only
Outreach Materials - show Topic only (though most will relate to Campus Engagement, so maybe no graph at all?)
Photographs - show Topic only
Publications - show Topic only (okay with also showing Discipline)
Research Centers - show Discipline only
Tools - show Topic only
Videos & Webinars - show Topic only

What are your thoughts on displaying both Discipline and Topic for Case Studies and Publications? What about the fact that most Outreach Materials are listed as Campus Engagement and not doing a chart at all?

I agree that both discipline and topic make sense for publications and case studies.
Since many outreach materials do relate to a specific topic (even though we don't have many submitted at this point), I would keep the topic chart.

@baronvonvaderham - here is the final decision regarding whether to show graphs for Discipline, Topic, or both:

Academic Programs - show Discipline only
Case Studies - show Topic AND Discipline
Conference Presentations - show Topic only
Course Materials - show Discipine only
Outreach Materials - show Topic only
Photographs - show Topic only
Publications - show Topic AND Discipline
Research Centers - show Discipline only
Tools - show Topic only
Videos & Webinars - show Topic only

Acknowledged. Just finished fixing the map data. Once I confirm this is all working I'll make the switch in the templates to make that happen.

8/24/16 decisions:
Dev version looks great so far! In addition to graph changes above, would be nice to incorporate the following:

Links to the Topic/Discipline names and/or bars (For example, on the Academic Programs page, clicking on "Business, Management & Finance" in the graph would take you to this output

Map Features
Although we didn't discuss on the call, would it be possible to add the Resource Title to the map pin when clicked, along with Institution name, and then also a link to that resource?
Also, what will we do when a single institution has multiple initiatives for a Content Type (e.g. University of California, Berkeley has 29 Academic Programs)

I'm working on that for the graphs. It's pretty tricky, though, as I can't just pass raw html strings into the javascript that generates it. There is a solution, it's just more complex than anticipated, requiring the construction of listener functions in the javascript to detect the clicks and process them. Not too bad, though.
I had briefly mentioned my plan for the map on the call: I don't want to overload those markers with data, and they're tricky to work with. I was going to simply change the name that's shown currently to a link to a search with the parameters content type=the page you're on, and organizations=that org.
That solves the problem of each organization having potentially dozens of resources in this content type, but still wanting to provide some functionality to get that list of resources easily. Maybe I can add a display of how many pieces of content that institution has under the name in there.

Okay, that makes sense - keep us posted if the fix for adding links in the graphs becomes too much - in could be a new feature later too.
I like your solution for the maps too (sorry I missed that) - I'm assuming the link to the content type for that institution can be hyperlinked instead of displaying the actual URL? If so, the display name could say "University of California, Berkeley - 29 Academic Programs" or "University of California, Berkeley Academic Programs" if you can't do the counts.

Map is updated on dev now. Moving on to the graphs....

I like it! My only recommendation with the map is to update the text upon clicking on a pin to include "(s)" instead of "s" - So it says "1 Academic Program(s)" instead of "1 Academic Programs"

Everything is up, the graphs are now clickable.
I can't get those links in the map to read like that. This uses a predefined property (content_type_label) to generate those. It would take a not insignificant effort to modify at this point, so I left it for now unless someone feels REALLY strongly otherwise.

Are you sure it deployed? I'm not getting clickable graphs on the dev site or the real one...

Yeah unfortunately that's all it's able to make clickable. I was thinking something in the intro text would be sufficient, unless you wanted a text blurb above the graphs that's the same for every page.

Might make sense to do (editable) text blurb for the graphs to include this and a disclaimer:
Bars in graphs can be clicked to view resources that align to a particular category. Graph data may include higher total counts than what is indicated in total resources since a single resource may be listed under multiple topics and disciplines."

Ok, I can drop in a block_content and use the same key everywhere so they'll all be uniform. Do you want that above or below the graphs?

Above the charts directly below the tabs would be most prominent, so that's my vote.

Also, circling back to the map...
You mentioned that you can't get the map to read "1 Academic Program(s)", but is it possible to do a different variation like "Academic Programs: 1" While it seems minor, something that is grammatically correct would be better for a higher ed audience.

Ok, changes made, deploying to dev now. If you guys have no further changes I can merge this into production later today.

Oh and the login is still broken on dev. Looking into that. But the block content edit link would appear right above the graph. I wrapped it in a div so it has the same margins and such as the graphs themselves so it's nice and symmetric.

Hey, @baronvonvaderham! I really like this and provided just a few technical comments on the pull request.
From a visual standpoint, I saw this "counter" option that the Unify theme provides, that I thought might be really cool for the "At a Glance" section:

Just a suggestion... it's really pretty though ;)

Seems like I should be able to add that in easily enough, just some different div classes. I'll tackle that in the morning.

Last update's up on dev: http://aashe-hub-scott.herokuapp.com/browse/types/academicprogram/
Assuming no objections, I'll be merging this and deploying into production tomorrow.

Awesome, love the map changes! I like the counters a lot too but have a few questions and suggested changes:

Can you shrink the font a bit, since it's much bigger than the font on the rest of the site and looks a bit disjointed?
Once shrunken down, would it be possible to fit all 5 counters in a single row? (Currently staggered in two rows - see example
Some content types have 4 counters with Canadian Provinces missing (Academic Programs, Photographs, Publications, Research Centers, Tools). The rest have all five counters.

I'll try on the font. It's a plugin from Unify, but maybe I can override some CSS. If that works, then yeah, I can shrink it to one row.
Those missing provinces don't have any Canadian institutions represented. I have that display conditionally rather than saying "0 Canadian Provinces" and looking awkward.

Thanks for checking - that makes sense about the counters only having four instead of five - your solution is much better, but I hope that once this is deployed and all resources are included, each category will include five.
I just noticed an issue where Research Centers don't display the summary chart - it should be the one displaying Academic Discipline.
Also, I noticed some discrepancies between the counters and map. For example, under Tools, the counter says "4 Countries" but the map shows pins in two countries. Could this be because we don't have latitude/longitude data from certain resources in the ISS? If so, that's another field we may want to update during Salesforce clean-up.

Yes exactly. To create a marker it needs the lat/long from ISS. We ran an update on that back in.....November last year? When I was creating the SCPD and was adding maps to the profiles. I'm guessing some of the new members don't have that data.
I'll fix the Research Centers, probably just an oversight when I was creating those filter lists.
And I found a way to override that font size. I'll have all this packed up and deployed in a few minutes.

Is there any way the cursor could change to the pointing hand (as when hovering over other links) when you hover over the clickable areas of the bar chart? I would realize they were clickable otherwise.

Sadly no, that's a system governed thing in your browser. They aren't actually "clickable" in the traditional sense, there's nothing in the HTML document....instead you just select it, almost as if you were highlighting some text, when you click that region. There's a "listener" function that looks for those select events and triggers a redirect.

I believe Monika was planning on including some reminder text in the block_content above the graphs to clarify things.

Yes, about the reminder text, but we were hoping to have this appear below the graphs as a second block-content, separate from the Introduction block.
Bars in graphs can be clicked to view resources that align to a particular category. Graph data may include higher total counts than what is indicated in total resources since a single resource may be listed under multiple topics and disciplines."

Ok, so you want the block to appear under the graphs? It's currently placed above the graphs, under the counters. I can swap it around if you want it somewhere else.
It will be editable like the intro block, but it's using a key shared across all content type pages, so you only have to do one.

Above or below doesn't matter to me, as long as its below the Summary Charts/Summary Map tabs. Has this not deployed to the dev site? I don't see it there yet.

One final question for now: Most Discipline Names (and the Coordination & Planning topic) have long titles, and they mostly get cut-off in the graphs. Would it be possible to make more room by moving the Y-axis to the left to allow for more room for labels?

The block is up on dev, but remember with admin broken on there you can't log in to see it. We'll double check on stage before we deploy. I'll be redeploying in a minute here with the revised counters, ran into a bug building JQuery on the server.
I actually played with that y-axis label a LOT when I first created them for exactly that reason. The problem is we really want those offsets equal for both graphs (for when both display) or else it looks really bad. This was the best compromise that displayed all of the Topic labels except I think one really long one, whereas almost every single discipline is gigantic and gets cut off unless I literally give it half of the entire block.
If you guys feel otherwise I can shift that easily enough, but that's just the reasoning behind choosing that size.

Gotcha thanks. On the y-axis, if it's not a lot more work at this stage, I'd be in favor of moving it slightly more to the right for both graphs so that Coordination & Planning (the only one that doesn't display on the Topics graph) is fully visible. Most of the Content Types only display the Topics graph anyway. Maybe a few more of the Academic Disciplines will show up too as a result, but I think it's fine if some are cut off.

Ok that works. I'll expand it slightly to fit that title then.
For Research Centers, I had to use the full label "Research Centers & Institutes"; I had just copied what was in the list you provided and didn't think to double check that.

Deployed. I think that should cover everything. It did reveal more of the Discipline labels.
http://aashe-hub-scott.herokuapp.com/browse/types/casestudy/

It looks like the font is larger on some Academic Discipline y-axis labels but not all. For those that only display Academic Disciplines (Academic Programs, Course Materials, Research Centers) see how Sustainability Studies & Science is cut off? Not so on the graphs that include both Disciplines and Topics.
Beyond that, everything looks great, I'm really excited to see this go live. Nice work! :)

Ah, yeah I see what you're talking about. That's because on some of those they have fewer disciplines, so you'll see the bars get a little bigger and space out a tiny bit more. I don't think there's anything I can do to override it, though.

I just want to mention that I really like the larger font size for the numbers. I wouldn't shrink them too much... just if you need them to fit within the width of the row.
We're using the Unify theme for a reason... because we like the design elements there.

A version without the size modification is up on dev. I agree, looks fine with the larger numbers.

Hey @jamstooks and @baronvonvaderham - the content type landing pages on the live site look great, but there's an internal server error on the Publications page that means that section is not accessible. Can this be fixed before the weekend?

Since all of the new resources on any given content type's landing page are going to be associated with that content type, including the content type within the new resources sidebar is redundant. Can we replace the content type with the organization name(s), which seems more relevant in this context?


clickhouse-backup: 2.5.29 -> 2.6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Altinity/clickhouse-backup/releases.
meta.description for clickhouse-backup is: Tool for easy ClickHouse backup and restore using object storage for backup files
meta.homepage for clickhouse-backup is: https://github.com/Altinity/clickhouse-backup
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.6.0 with grep in /nix/store/l3kwqk9s45hkcz4wvhd940vxc1jvv7k8-clickhouse-backup-2.6.0
found 2.6.0 in filename of file in /nix/store/l3kwqk9s45hkcz4wvhd940vxc1jvv7k8-clickhouse-backup-2.6.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

clickhouse-backup

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l3kwqk9s45hkcz4wvhd940vxc1jvv7k8-clickhouse-backup-2.6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A clickhouse-backup https://github.com/r-ryantm/nixpkgs/archive/3a2831a957133cb4217f5cb563f54da9ef3807c3.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3a2831a957133cb4217f5cb563f54da9ef3807c3#clickhouse-backup

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l3kwqk9s45hkcz4wvhd940vxc1jvv7k8-clickhouse-backup-2.6.0
ls -la /nix/store/l3kwqk9s45hkcz4wvhd940vxc1jvv7k8-clickhouse-backup-2.6.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

clickhouse-backup

Maintainer pings
cc @devusb for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 338503 run on x86_64-linux 1

1 package built:

clickhouse-backup


package name issue?

Hello,
When I try to install the package, this error occurs:

> Sys.info()
sysname                                 release
"Linux"                 "4.19.0-16-cloud-amd64"
version                                nodename
"#1 SMP Debian 4.19.181-1 (2021-03-19)"                    "rstudio-premptible"
machine                                   login
"x86_64"                               "unknown"
user                          effective_user
"master"                                "master"
> library(devtools)
> install_github("Clarivate-SAR/woslite_r_client")
Downloading GitHub repo Clarivate-SAR/woslite_r_client@HEAD
✓  checking for file ‘/private/var/folders/n_/4b1w6nqd5gdgtmmlrm6pm82c0000gn/T/RtmpDQiA1l/remotes61fe76b20b5b/Clarivate-SAR-woslite_r_client-713b51a/DESCRIPTION’ ...
─  preparing ‘woslite_r_client’:
E  checking DESCRIPTION meta-information ...
Malformed package name
See section 'The DESCRIPTION file' in the 'Writing R Extensions'
manual.
Error: Failed to install 'woslite_r_client' from GitHub:
System command 'R' failed, exit status: 1, stdout + stderr:
E> * checking for file ‘/tmp/RtmpYnEfPx/remotes11023944e2f8/Clarivate-SAR-woslite_r_client-713b51a/DESCRIPTION’ ... OK
E> * preparing ‘woslite_r_client’:
E> * checking DESCRIPTION meta-information ... ERROR
E> Malformed package name
E>
E> See section 'The DESCRIPTION file' in the 'Writing R Extensions'
E> m

Hello,
When I try to install the package, this error occurs:
> Sys.info()
sysname release
"Linux" "4.19.0-16-cloud-amd64"
version nodename
"#1 SMP Debian 4.19.181-1 (2021-03-19)" "rstudio-premptible"
machine login
"x86_64" "unknown"
user effective_user
"master" "master"
> library(devtools)
> install_github("Clarivate-SAR/woslite_r_client")
Downloading GitHub repo Clarivate-SAR/woslite_r_client@HEAD
✓ checking for file ‘/private/var/folders/n_/4b1w6nqd5gdgtmmlrm6pm82c0000gn/T/RtmpDQiA1l/remotes61fe76b20b5b/Clarivate-SAR-woslite_r_client-713b51a/DESCRIPTION’ ...
─ preparing ‘woslite_r_client’:
E checking DESCRIPTION meta-information ...
Malformed package name
See section 'The DESCRIPTION file' in the 'Writing R Extensions'
manual.
Error: Failed to install 'woslite_r_client' from GitHub:
System command 'R' failed, exit status: 1, stdout + stderr:
E> * checking for file ‘/tmp/RtmpYnEfPx/remotes11023944e2f8/Clarivate-SAR-woslite_r_client-713b51a/DESCRIPTION’ ... OK
E> * preparing ‘woslite_r_client’:
E> * checking DESCRIPTION meta-information ... ERROR
E> Malformed package name
E>
E> See section 'The DESCRIPTION file' in the 'Writing R Extensions'
E> m

Same error on OSX 11.4 and R 4.1.0.

I get the same issue. I will submit a pull request.

@iangow merged your PR.
@tylerascott and @arecibo can you please confirm that this is working for you? Please create a new issue in case you experience other problems.


chore(bdev): cleanup core bdev code

core::Bdev made a generic similar to underlying spdk_rs::Bdev.
Nexus now works with core::Bdev.
core::UntypedBdev type introduced for code that does not need to know about
actual Bdev type.
This is an intermediate cleanup for ongoing work of refactoring Bdev layer in
Mayastor.

bors try

bors try

bors try

bors try

bors merge


Add an option to make a warm-up of lower/upper KB databases at startup

Having most of lmdb pages loaded in memory speed up a lot the processing even though it requires a lot of RAM.
Maybe we could add an option (in the config files) or a REST end point to force the full (almost full?) loading of some lmdb databases ofr a given languages (the ones that are required to do a disambiguation):

pages db ?
concepts db?
labels db ?
This should be as simple as opening a cursor on every lmdb db and iterate till the end.

What do you think ?
Best regards
Olivier Terrier

Hi Patrice,
Not sure if we should preload the whole dbs (too much memory involved) or just a subset like the N most frequently used entries ?
Looking at the code of com.scienceminer.nerd.utilities.WikipediaLabelIDF I can see that you already have the occurence count stored in the LabelDatabase so for this one it should be easy
But don't know how to proceed for the others (PageDb, etc...)
Maybe a persistant EHCache with an LFU policy could be associated to every KBDatabase so that the N most frequent entries (just the key anyway) could be stored and retrieved at startup ?
Just some thoughts
Best regards
Olivier


add haxe language support

This PR adds language support for the Haxe programming language

LGTM :+1:
thanks a lot @LordMZTE


how does this compare to https://github.com/WebFreak001/FSWatch ?

would be nice to have a README comparing dinotify with https://github.com/WebFreak001/FSWatch

Not cross-platform and doesn't try to emulate features. Added a note in README


Dockstore tool/workflow launch

Feature Request
The command-line currently collects output from calling cwltool and then dumps it out after tool/workflow execution. This is undesirable in two ways:

It is difficult to determine what is currently happening
We run the risk out the output overwhelming memory

Desired behaviour
Output, both stdout and stderr should stream.
Stream the output instead of

#438


copr: Rewrite non-vectorized inet_ntoa

Signed-off-by: Ruihang Xia<EMAIL_ADDRESS>
What have you changed?
Rewrite non-vectorized inet_ntoa to keep it identical with vectorized inet_ntoa in #5900
What is the type of the changes?
Pick one of the following and delete the others:

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?
Please select the tests that you ran to verify your changes:

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No

@youjiali1995 PTAL, thanks!

/merge

/merge


[madDatePicker] set initial value

Hello all,
I'm trying to using  the datePicker passing the initial value but it doesn't select the day in the initial value. I'm not sure if I'm doing something wrong but the documentation in this case do not help.
Angular material version:
"@angular/material": "^2.0.0-beta.8",
This is my code:
<input mdInput placeholder="Date of Birth"
			       id="dateOfBirth"
			       formControlName="dateOfBirth"
			       name="dateOfBirth"
			       [mdDatepicker]="picker"
			       formControlName="dateOfBirth">

<button mdSuffix [mdDatepickerToggle]="picker"></button>
<md-datepicker #picker></md-datepicker>

Where dateOfBirth is: 2017/05/11
this.userProfile = new FormGroup({
            firstName: new FormControl('', Validators.required),
            lastName: new FormControl('', Validators.required),
            dateOfBirth: new FormControl('', Validators.required),
            emailNotification: new FormControl('', Validators.required)
        });
 this.userProfile.setValue({
                    firstName: res.profile.firstName,
                    lastName: res.profile.lastName,
                    dateOfBirth: '2017/05/11',
                    emailNotification: res.profile.emailNotification
                });

This is the result:

Create a plunk.

@daniele-zurico, have you tried using a Date object to initialize it?
this.userProfile.setValue({
                    firstName: res.profile.firstName,
                    lastName: res.profile.lastName,
                    dateOfBirth: new Date(2017,4,1),
                    emailNotification: res.profile.emailNotification
                });

hey @julianobrasil thanks for your reply but same behaviour....it select correclty the month and the year but it doesn't select the day

@daniele-zurico, take a look at this plunk (I think it's working fine): https://plnkr.co/edit/bggSIF?p=preview

hey @julianobrasil thanks for the plunker .... now we need to apply something on the new Date.... you set the month to 4 and it display 5:
Your choice: 01/05/2017 00:00:00
chosenDate: new Date(2017,4,1)

The months o in Date object are in the range 0..11.

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.

https://stackoverflow.com/users/9962015/newbiiiie?tab=reputation i have a similar problem, someone can help me please ?

https://stackoverflow.com/questions/50947642/angular-material-6-datepicker-reactive-form I have a similar problem, can you help me please ?

@alanent, your link is broken.


[Feature Request] Making gradient boosted trees available outside tf.Estimator

Feature Request
Is there any ongoing discussion on making gradient boosted trees available outside of tf.estimators ?
Doing this will allow a wider range of experimentation than what is possible with tf.estimators.

@nataliaponomareva not sure whether you have plans to expose the underlying ops?

boosted_trees_ops_op_lib has public visibility, so you can potentially use the ops as you wish in your graph.
What's ur use case? what kind of experiments you can't do with an estimator?

@nataliaponomareva  I specifically would like to jointly train a CNN/LSTM based pipeline with gradient boosted trees.  I could not find a concrete example of how to do this with an estimator.  That is the source of this feature request. If however it is possible to do it via tf.estimator, an example would be much welcome.

You would have to write your own estimator (eg. model fn where you will call the ops from lstm and gbdts). An example of something like this is https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/boosted_trees/estimator_batch/dnn_tree_combined_estimator.py - you can get an idea on how to do it out of it (eg using hooks etc)


Publish module test

Added testcase for publish module

@noopurmath3 rebased it off master, updated the suggested changes. marking it as approved. thanks.


demo keeps running

I followed all the procedure as proposed in Linux and run the demo. But the demo keeps running for over 10 minutes. And the gpu usage stays at 0%. What could be the problem?

Ran again today. The code works fine and it took 60 seconds to ran the demo.


Update release notes for 1.7.0

To find all PRs that have been merged since the last release: repo:theislab/scanpy closed:>YYYY-MM-DD is:pr is:merged where the date is the date of last release.
Boy were there a lot. There should be some automation here.

Thank you! We could try blurb for automation.

Thank you! We could try blurb for automation.

I can work on #1512 tonight/tomorrow, should not be too much. Would be great to still include that in 1.7 please 🙏

I can work on #1512 tonight/tomorrow, should not be too much. Would be great to still include that in 1.7 please 🙏

Thank you, awesome! ❤

Thank you, awesome! ❤

Oh https://github.com/theislab/scanpy/pull/1551 would also be good to have 😅

Oh https://github.com/theislab/scanpy/pull/1551 would also be good to have 😅


DataSet.stack_anomalous Fails for duplicate column names

If Friedel columns share the same base name with non-Friedel columns, stack_anomalous will fail with a cryptic error message.
Here is a minimal example
import reciprocalspaceship as rs
import numpy as np

dmin = 2.
cell = [10., 20., 30., 90., 90., 90.]
sg = 19

h,k,l = rs.utils.generate_reciprocal_asu(cell, sg, dmin, anomalous=False).T

ds = rs.DataSet({
        'H' : h,
        'K' : k,
        'L' : l,
        'I' : np.ones(len(h)),
        'I(+)' : np.ones(len(h)),
        'I(-)' : np.ones(len(h)),
    },
    merged=True,
    cell=cell,
    spacegroup=sg
).infer_mtz_dtypes().set_index(['H', 'K', 'L'])

print(ds)
print(ds.dtypes)

ds.stack_anomalous()

which outputs
user@computer:~$ python bug.py
I              Intensity
I(+)    FriedelIntensity
I(-)    FriedelIntensity
dtype: object
Traceback (most recent call last):
  File "bug.py", line 26, in <module>
    ds.stack_anomalous()
  File "/home/kmdalton/opt/reciprocalspaceship/reciprocalspaceship/dataset.py", line 907, in stack_anomalous
    F[label] = F[label].from_friedel_dtype()
  File "/home/kmdalton/opt/anaconda/envs/careless/lib/python3.8/site-packages/pandas/core/generic.py", line 5487, in __getattr__
    return object.__getattribute__(self, name)
AttributeError: 'DataSet' object has no attribute 'from_friedel_dtype'

This occurs because DataSet.stack_anomalous() removes the suffixes from anomalous columns during this method. It then eventually calls ds["I"], which would ordinarily return a DataSeries which has a from_friedel_dtype() method. However, since there are now two columns with the label "I", this call now returns a DataSet.
I'm considering this a bug, because it is certainly unexpected behavior. To resolve it, there are a few possible options in my mind:

This behavior could be fixed by changing the column names later in the method. Although this would resolve this error, there would still be columns with the same name in the resulting DataSet. This will likely cause downstream problems.
One could raise a ValueError when this case is detected. This can then give a more sensible error message like "This operation will result in two or more columns with the same name"
We could do (2), but also add a only_keep_friedels=True flag that removes any non-Friedel columns with this operation. In the above example, this would drop the original "I" column, and output a DataSet with an "I" column generated from "I(+)" and "I(-)".

In the case of solution 2 (or perhaps 3 as well), I'd be happy to tackle this; it's similar to the change I made a little while back, and is still pretty fresh on my mind.

Awesome -- sounds good! Kevin's example will also make a good template for writing a test. Reach out if you run into trouble.
I'm inclined to suggest starting with (2), because (3) can currently be implemented by the user pretty easily by dropping desired columns.


Sort default 'enable' value is not being applied

See https://hjalmers.github.io/angular-generic-table/examples#custom example.
Columns name and age do not have 'sort' property set as enable, therefore when the mouse hovers over the column header it does not change to a pointer. As well as the sort caret is not being applied when the column header is clicked.
However documentation states that 'sort' is optional with the default value enable.

Another good observation @msut-s, I'll make sure this is fixed:)


Automatically determine image/tensor color mapping & need for srgb decoding

What
Fixes #2274

#2274

Will need to be further worked on via:

https://github.com/rerun-io/rerun/issues/2341

Introduces 3 new things actually:

finite range, determined if the range of a tensor was in fact not finite
heuristic for color mapping range
heuristic for srgb

⚠ does this break things?

Range & sRGB:
This
import rerun as rr
import numpy as np
import matplotlib.pyplot as plt

rr.init("image color", spawn=True)
img = np.random.random((300, 300, 3)) * 0.2
rr.log_image("image", img)
print(img.min(), img.max())  # to check they are in range [0,1]

plt.imshow(img)
plt.show()

Gives now the expected image:

When setting a pixel to inf, we show the "finite range" as well, and behave correctly:

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)

PR Build Summary: https://build.rerun.io/pr/2342

Docs preview: https://rerun.io/preview/33be034/docs
Examples preview: https://rerun.io/preview/33be034/examples

python ./examples/python/api_demo/main.py --demo image_tensors
before:

after:

looks the same which is encouraging

Much much better now! Needed quite a few more changes, but I'm happy with those now

oh. yes. haven't done that yet!

looked all good, didn't find any regressions!


Taskbar not staying on top level after fullscreen

Hello there,
I'm currently experiencing a bug where the taskbar (in autohide or overlap mode) disappears after using an app in fullscreen then acts like a normal window afterwards.
Steps to reproduce

Open a Youtube video on fullscreen in Firefox.
Close the fullscreen video player.
The taskbar is now hiding behind all other windows.

Expected behavior
The taskbar should reappear as a "top level window".
System information:

Windows version: Windows 10 1809
Cairo version: M3.6768
32-bit or 64-bit: 64 bit
In Cairo Settings > Advanced, is Cairo set as shell?: No.

Additional context
Obviously couldn't reopen the similar issue #165 since it wasn't posted by me.

I have not looked in to this at all but....
I would recommend that we have an always on top option for the bar. If the setting is true then when any window is shown or closed make the bar top most else let it be where it lay

@josuave This one is interesting, for some reason the native Windows method for setting the window as top-level isn't always working after exiting full screen. I found that making the window the frontmost first tends to help and implemented this a while ago, but there must be a few holes still.

Even the Windows TaskBar has this issue, after fullscreen mode you can sometimes overlap the taskbar with other applications when clicking on the desktop, so my guess is that Windows is handling a few more events to make itself topmost.

This is fixed in the latest release.


[ci] Run pylint in py3.8 CI

What changes were proposed in this pull request?

Adding pylint checks for py3.8 CI for both CircleCI and Github Actions worklows.

How was this patch tested?

Verifying the successful run of newly added check in the workflow details.
Also added a badly linted file, and the checks failed for it successfully throwing all linting suggestions.

PR now looks ready for review!


Use "partialCached" for reduced build time

Is your feature request related to a problem? Please describe.
My blog has around 2600 blog posts and many tags. A full build takes ~46 seconds.
I tested partialCached instead of partial in the file layouts/partials/sidebar/main.htmlwith following results:

use partialCached for "sidebar/taxonomies": Build time 36s
use also partialCached for "sidebar/recent-posts": Build time 28s
use also partialCached for "sidebar/featured-posts": Build time 13s

Describe the solution you'd like
Use partialCached for parts with the same contents on all pages.

Which version(master or develop) do you use?

master.

I drafted a new release about using Hugo Pipes to generate assets, I also cache some templates. Would you mind sharing your blog repo with me, so that I can simulate the performance issue of a real large site?

I tested your repo with the latest release, it took about 7s for building the site on my laptop. Is it acceptable?
                   |  DE
-------------------+-------
  Pages            | 1295
  Paginator pages  |  310
  Non-page files   |  658
  Static files     |  111
  Processed images |  839
  Aliases          |  454
  Sitemaps         |    1
  Cleaned          |    0

Total in 6725 ms

`
|  DE
-------------------+-------
Pages            | 4359
Paginator pages  | 1692
Non-page files   | 1002
Static files     |  111
Processed images | 1211
Aliases          |  898
Sitemaps         |    1
Cleaned          |    0
Built in 16808 ms
`
For the complete blog. This is nice with partialCached.
Thank you :)

I'm going to close this issue, please let me know if you found out any potential cacheable partials.


Add Option to Not Create an Output Binding for the RoutingFunction

See https://stackoverflow.com/questions/67419839/spring-cloud-stream-functionrouter-output-attempts-to-bind-to-kafka-topic
Verified when destination functions are all Consumers.
@SpringBootApplication
public class So67419839Application {

	public static void main(String[] args) {
		SpringApplication.run(So67419839Application.class, args);
	}

	@Bean
	MessageRoutingCallback route() {
		return msg -> new String((byte[]) msg.getPayload()).equals("foo") ? "foo" : "bar";
	}

	@Bean
	Consumer<Message<?>> foo() {
		return msg -> System.out.println("foo: " + msg);
	}

	@Bean
	Consumer<Message<?>> bar() {
		return msg -> System.out.println("bar: " + msg);
	}

}

spring.cloud.function.definition=functionRouter
spring.cloud.stream.bindings.functionRouter-in-0.destination=so67419839
spring.cloud.stream.bindings.functionRouter-in-0.group=so67419839

$ kafka-topics --bootstrap-server localhost:9092 --list
__consumer_offsets
functionRouter-out-0
so67419839

But I see that it is needed if at least one of the delegates is a function:
	@Bean
	Function<Message<?>, String> foo() {
		return msg -> {
			System.out.println("foo: " + msg);
			return "FOO";
		};
	}

So we probably need a new property to suppress it, I don't see how we could deterministically determine that all delegates are Consumers.

As a work around simply point it to the same destination
spring.cloud.function.definition=functionRouter
spring.cloud.stream.bindings.functionRouter-in-0.destination=so67419839
spring.cloud.stream.bindings.functionRouter-out-0.destination=so67419839
spring.cloud.stream.bindings.functionRouter-in-0.group=so67419839
spring.cloud.stream.function.routing.enabled=true

Since the delegates are all Consumers we'll never actually send anything.

So if I understand correctly. . .
First, I just added a test to validate the routing itself and it works
@Test
public void testRoutingToConsumers() {
	try (ConfigurableApplicationContext context = new SpringApplicationBuilder(
			TestChannelBinderConfiguration.getCompleteConfiguration(
					RoutingConsumerConfiguration.class))
						.web(WebApplicationType.NONE)
						.run("--spring.jmx.enabled=false",
						"--spring.cloud.function.routing-expression=headers['func_name']")) {

		InputDestination inputDestination = context.getBean(InputDestination.class);
		Message<byte[]> inputMessage = MessageBuilder
					.withPayload("foo".getBytes())
					.setHeader("func_name", "foo")
					.build();
			inputDestination.send(inputMessage);
	}
}

@EnableAutoConfiguration
public static class RoutingConsumerConfiguration  {
		@Bean
		public Consumer<String> foo() {
			return System.out::println;
		}
		@Bean
		public Consumer<String> bar() {
			return System.out::println;
		}
}

However, and that is what I believe you're saying is that the fact that RoutingFunction is a function it automatically creates an output binding even though there will never be an output.
Correct?

Exactly, yes.
I don't see how we can programmatically detect that the function never routes to another function (only Consumers), I think we need a property, something like:
spring.cloud.stream.function.routing.consume-only=true

By the way, the docs talk about spring.cloud.stream.function.routing.enabled=true which doesn't seem to be needed.

Ok, it appears that it can work. Just spiked it up and all tests are passing
Basically the idea is to never create output binding for RoutingFunction, only input. In the event the routing is to the Consumer, we're good and if it is to Function, then StreamBridge is going to be used to effectively create an output binding on demand.
Will sleep on it and merge tomorrow


ipsec: set interface ID different from 0

in this commit:
https://github.com/torvalds/linux/commit/8dce43919566f06e865f7e8949f5c10d8c2493f5,
we see that if_id must be different from 0.
With a 0 value, it makes the creation of the dummy interface fail with
the following error:
level=fatal msg="IPSec with tunneling requires support for xfrm state output masks (Linux 4.19 or later)." error="invalid argument" subsys=daemon

Related-To: https://github.com/flatcar-linux/Flatcar/issues/626
Signed-off-by: Mathieu Tortuyaux<EMAIL_ADDRESS>

Hi, I see this test is failing: "ConformanceKind1.19 / installation-and-connectivity" is that a flaky test or something needs to be investigated ?

Thanks @pchaigno for the new run,it is still failing. Investigating on the cilium logs

/test

The failure in privileged unit tests (runtime Jenkins job) seems related:
	 START: ipsec_linux_test.go:120: IPSecSuitePrivileged.TestUpsertIPSecEndpoint
	 START: ipsec_linux_test.go:37: IPSecSuitePrivileged.SetUpTest
	 PASS: ipsec_linux_test.go:37: IPSecSuitePrivileged.SetUpTest	0.000s

	 ipsec_linux_test.go:175:
	     c.Assert(err, IsNil)
	 ... value syscall.Errno = 0x2 ("no such file or directory")

	 START: ipsec_linux_test.go:42: IPSecSuitePrivileged.TearDownTest
	 PASS: ipsec_linux_test.go:42: IPSecSuitePrivileged.TearDownTest	0.000s

	 FAIL: ipsec_linux_test.go:120: IPSecSuitePrivileged.TestUpsertIPSecEndpoint

@pchaigno I don't think it's a flaky test. I can "reproduce" the "no such file or directory" error. I think it's related to the change on the policy creation: the Ifid is now different from 0 on the policy itself to avoid a mismatch. So policies are created with Ifid = 1 which leads to a failure when the policy is trying to be Get in the test:
https://github.com/cilium/cilium/blob/735096950cc2a0875b1793fed650ef55bbbe99c8/pkg/datapath/linux/ipsec/ipsec_linux_test.go#L166-L175
The &netlink.XfrmPolicy{...} will have an Ifid != 1 so the policy is not found and the test is failing.

@pchaigno rebased the branch on master and squashed fixup! commits. 🙂

/test

FYI, this pull request broke IPsec so I'm reverting in https://github.com/cilium/cilium/pull/19019.


Update Cake to 0.22.2

Fixes #4

Thanks so much for your contribution @kll !! Your PR has now been merged, and the updated NuGet package has been deployed from CI!
Thanks again!
cc/ @jnm2

Thanks so much @agc93! I can now stop using --settings_skipverification=true! 🎉


[Go SDK] Fix multimap support for the direct runner

Updates the NewKeyedIterable() implementation to correctly parse a multimap side input from an in memory ReStream.
Fixes #24774

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

This roughly looks right to me, but I'll defer to @lostluck on this one since I'd need to do some extra context building to do a proper review here.
R: @lostluck


Volume mount path should be independent of User dir

Why should the local mount path for Docker volumes should be only under User directory?
I have all of my projects in a different partition, so, I cant mount any of them as volume in a container. I even tried making a symlink under User dir pointing to original dir but that didnt work also.
Volume mount path should be independent of User directory.

I'm using Windows 10 x64, Docker version 1.9.1

The issue is that the VM itself doesn't mount the volume/drive - Therefore you end up with a 1/2 mounted volume and no way to really get the data.
If you would like, subscribe to the PR, which has some of this info:
https://github.com/docker/kitematic/pull/1202


getting error in the code given below after updating the flutter to Flutter 1.5.4-hotfix.2 .

can't assign httpLink to link.
static HttpLink httpLink = HttpLink(
uri: " .."
);
ValueNotifier client1 = ValueNotifier(
GraphQLClient(cache: InMemoryCache(), link: httpLink//error is here
),
);
error: The argument type 'HttpLink' can't be assigned to the parameter type 'Link'. (argument_type_not_assignable

You need to cast the error httpLink as Link
`


Update outdated files in dev-1.14-ko.2 partly-concepts-contribute-ref

Fixes partly #13738.
This PR covers all outdated files in docs/concepts, docs/contribute, and docs/reference.

docs/concepts/configuration/scheduler-perf-tuning.md 1 file changed, 38 insertions(+), 34 deletions(-)
docs/concepts/overview/kubernetes-api.md 1 file changed, 4 insertions(+), 4 deletions(-)
docs/concepts/overview/object-management-kubectl/overview.md 1 file changed, 3 insertions(+), 1 deletion(-)
docs/concepts/overview/working-with-objects/kubernetes-objects.md 1 file changed, 2 insertions(+), 2 deletions(-)
docs/concepts/workloads/pods/pod-lifecycle.md 1 file changed, 2 insertions(+), 2 deletions(-)
docs/concepts/workloads/pods/pod-overview.md 1 file changed, 1 insertion(+), 1 deletion(-)
docs/contribute/_index.md 1 file changed, 1 insertion(+), 1 deletion(-)
docs/reference/_index.md 1 file changed, 2 insertions(+), 2 deletions(-)
docs/reference/glossary/device-plugin.md 1 file changed, 2 insertions(+), 2 deletions(-)
docs/reference/glossary/extensions.md 1 file changed, 2 insertions(+), 2 deletions(-)
docs/reference/glossary/pod-lifecycle.md 1 file changed, 7 insertions(+), 4 deletions(-)
docs/reference/glossary/replica-set.md 1 file changed, 1 insertion(+), 2 deletions(-)
docs/reference/glossary/taint.md 1 file changed, 4 insertions(+), 4 deletions(-)
docs/reference/glossary/toleration.md 1 file changed, 4 insertions(+), 4 deletions(-)
docs/reference/kubectl/cheatsheet.md 1 file changed, 21 insertions(+), 9 deletions(-)

/language ko

/assign

@seokho-son 감사합니다!
/lgtm
/approve


[QUESTION] Do you support "look ahead" operation?

Eg. Match everything till '\n'|'*/'|eof

Can you say more precisely what kind of lookahead operation you're looking for, and an example of how you might use it?
(Also, I've enabled GitHub Discussions, so in the future, for questions, you can start a thread there instead of creating an issue.)

Lets say I want a content of hpp multiline comment, each line separately
/* bla 0
bla bla bla 1
bla bla 2
bla 3 */

so in this case once I find comment opening I want to match zero or more times everything till occurrence of new line or string "*/" or end of file (but without these delimiters)

Yes, this is supported in the lexer modes -- take a look at grammars/go.lang for an example, specifically the following lines:
`/*` => { push comment; pass; }

// ...

mode comment {
    `*/` => { pass; pop_extract; }
    _ => { pass; }
}

cool, thanks!


Abstractions cleanup

[ ] .Net: Move classes in SkillDefinition, Orchestration, Http package to SK Core
[ ] .Net:

Reference document: https://microsoft.sharepoint.com/:w:/r/teams/semantic-kernel/_layouts/15/Doc.aspx?sourcedoc={8EE910B1-C8CD-4DC1-B175-8FF9F35E22D1}&file=2023.09-Semantic Kernel DotNet Naming and Abstraction Changes.docx&action=default&mobileredirect=true&share=IQGxEOmOzcjBTbF1j_nzXiLRAfQw1zk23jhqlc1uKvtBj4Y

Review validity of Abstractions package
Are types in Abstrations actually “abstract”?
Are they sufficient for a developer to implement SKFunctions with only this nuget reference?

I think the work to verify "Are they sufficient for a developer to implement SKFunctions with only this nuget reference?" should be a separate task.

All classes and interfaces from the SemanticKernel.Abstractions project were evaluated to determine whether they should remain there or not. The final ask, 'Are they sufficient for a developer to implement SKFunctions with only this NuGet reference?' has been moved to a separate task."https://github.com/microsoft/semantic-kernel/issues/3020.


Build workflow deactivated

Hi, thank you for making this project, it makes updating Postman on a linux box a lot more convenient.
Unfortunately, the "Build AppImage" workflow seems to have been deactivated, due to repo inactivity.

sorry for inconvenience, I missed the email notification.
I've re run the trigger.


app - Views

**In this pull request **
Created a View path same as wireframes.

made changes to the database added new data to the database


Add Vivekanandbs's file

This pull request adds Vivekanandbs's file to  _post folder.

closing pull requests

I am not getting the green sign even if I have completed the task
On Sun, Feb 6, 2022 at 7:51 PM github-learning-lab[bot] <
@.***> wrote:

@.**[bot] requested changes on this pull request.
I was expecting your file to be named _posts/0000-01-02-Vivekanandbs.md.
Let's edit this pull request to fix both of these issues.
⌨ Activity: Fixing the file name in your pull request

Click the Files Changed tab
https://github.com/Vivekanandbs/github-slideshow/pull/3/files in
this pull request.
Verify that the only file edited is named
_posts/0000-01-02-Vivekanandbs.md.
If the file is improperly named, or not in a proper location, use
the edit function to correct it.
Above the contents of the file, select all text in the field that
contains the filename and delete it.
Continue pressing your backspace key to also delete any directory
names that exist.
Type the proper filename:

_posts/0000-01-02-Vivekanandbs.md

Scroll to the bottom and enter a commit message and commit in the Commit
Changes section.

[image: gif of changing file name in UI]
https://user-images.githubusercontent.com/9906718/112817649-f4ac1a80-9082-11eb-972f-4a6609857b20.gif
Note: You can find directions on how to move files while editing in our
documentation
https://docs.github.com/en/github/managing-files-in-a-repository/moving-a-file-to-a-new-location
.
If you would like assistance troubleshooting the issue you are
encountering, create a post on the GitHub Community
https://github.community/c/education/github-learning-lab/34/l/latest
board. You might also want to search for your issue to see if other people
have resolved it in the past.
I'll respond below when I detect a commit on this branch.
—
Reply to this email directly, view it on GitHub
https://github.com/Vivekanandbs/github-slideshow/pull/3#pullrequestreview-874044661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALHSRTEYHA5GR4OTTOW533LUZZ7VTANCNFSM5NVMOLWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


[enhancement] Download songs for listening offline

It would be awesome to be able to download songs for when you don't have access to a good internet connection sort of like how it works on mobile.

Not possible. Sorry :8ball:
Google don't allow / have an API for that


NTR: [list term label(s)]

New Picklist Term(s)
REQUIRED:
Associated Field(s): anatomical_part
Term Label: Nasopharynx
Definition/Description: The section of the pharynx that lies above the soft palate.
OPTIONAL:
Definition Source: http://www.informatics.jax.org/accession/cwg
Synonyms or alternative labels: NP
Additional Comments (not an annotation): description of usage within a sample plan
Contributor Attribution: https://orcid.org/0000-0002-1825-0087

This is a pretend/example new term request.


Feature/fix api routing

Coverage remained the same at 100.0% when pulling beb33b9fc0c1936e7bb9a9f0283cb98440625087 on feature/fix-api-routing into 9d9836fc675d175965eab628b7cf9f6e377f6c39 on develop.


fix(charts): update threshold properties

Updated threshold chart properties to match design guidelines.
Fixes https://github.com/patternfly/patternfly-next/issues/2485

PatternFly-Next preview: https://patternfly-next-pr-2486.surge.sh

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Decide how to determine whether a github repo is a garment or not

Plan A : filter the repos with a specific topic for exemple github-garment
This plan doesn't work, because reading the topics is an experimental feature and writing repos topics is not possible yet via API.
Plan B : The repo must contain a specific file like garment-config.json

Plan B was chosen for now


Recording format documentation

Hi,
Is there any documentation of how recording is done and the recording file structure?
In case a third party would wants to save it's terminal session activities in terminalizer format without using terminalizer record functionality.
In other words I want to integrate terminalizer into another session recording application.
Thanks in advance

The output file is a YAML file with two keys

config: check Documentation
records:

delay: in ms
content: stdin+stdout+stderr

I wonder how to emulate the rewriting of the whole screen or changes at specific coordinates. I'm not familiar with how terminals do it via the stdout stream so examples would be cool.


[Bug] Arc path too compressed for Illustrator/Affinity

Using the (default) option to rewrite the path on<EMAIL_ADDRESS>an arc is wrote as:
a.65.65 0 011.109-.459
This is properly displayed on Chrome and Firefox but some editors (I've tryed Adobe Illustrator CC 2019 and Affinity Designer 1.7) are not able to recognize the large-arc-flag parameter.
011.109 are 3 values:

0
1 (a boolean is required here, it cannot be 11.109)
1.109

Illustrator and Affinity are not enough smart to understand that.
How should be:
a.65.65 0 0 1 1.109-.459 (there are 2 more spaces)
Attached a full example:
example.zip

Just disable noSpaceAfterFlags in convertPathData plugin.

I think this option should be disabled by default. After updating svgо to 1.3.0, several projects in production were broken. :disappointed:

Just disable noSpaceAfterFlags in convertPathData plugin.

It works, as @grig0ry I think noSpaceAfterFlags: false should be the default.

It's a totally standard syntax from a 8-years old specification (and was introduced in a working draft a year before). If some software has a problems with it—those should be addressed to that software.

Thank you! I was having this issue where I was outputting SVG that would render fine but if I took it to SVGOMG it wouldn't be able to recognize the code. For instance:
Original:
<svg width="24" height="24" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
<path fill-rule="evenodd" clip-rule="evenodd" d="M12 3C10.8954 3 10 3.89543 10 5C10 6.10457 10.8954 7 12 7H12.01C13.1146 7 14.01 6.10457 14.01 5C14.01 3.89543 13.1146 3 12.01 3H12ZM12 10C10.8954 10 10 10.8954 10 12C10 13.1046 10.8954 14 12 14H12.01C13.1146 14 14.01 13.1046 14.01 12C14.01 10.8954 13.1146 10 12.01 10H12ZM12 17C10.8954 17 10 17.8954 10 19C10 20.1046 10.8954 21 12 21H12.01C13.1146 21 14.01 20.1046 14.01 19C14.01 17.8954 13.1146 17 12.01 17H12Z" fill="#353535"/>
</svg>

Optimized without no space after flags set to false
<svg width="24" height="24" xmlns="http://www.w3.org/2000/svg"><path fill-rule="evenodd" clip-rule="evenodd" d="M12 3a2 2 0 100 4h.01a2 2 0 100-4H12zm0 7a2 2 0 100 4h.01a2 2 0 100-4H12zm0 7a2 2 0 100 4h.01a2 2 0 100-4H12z"/></svg>

Would turn into:
<svg width="24" height="24" xmlns="http://www.w3.org/2000/svg">
  <path fill-rule="evenodd" clip-rule="evenodd" d="M12 3a2 2 0 100 4 NaN NaNhNaNa2 2 0 100-4 NaN NaNhNaNzm0 7a2 2 0 100 4 NaN NaNhNaNa2 2 0 100-4 NaN NaNhNaNzm0 7a2 2 0 100 4 NaN NaNhNaNa2 2 0 100-4 NaN NaNhNaNz"/>
</svg>

Fix for me:
{
  "plugins": [
    {
      "convertPathData": {
        "noSpaceAfterFlags": false
      }
    }
  ]
}

You can add librsvg to the list of software that chokes on this optimisation. If this option is going to remain enabled by default, could there at least be a warning about the incompatibility somewhere? It's not easy to track down exactly what's going wrong at the moment.

This also breaks the default linux svg renderer which impacts all image viewers, gimp, and the file system image previews. I agree with everyone else that this should not be enabled by default because it breaks svg output in many different applications.
A default setting does not make sense if the majority of users need to override them and shipping an svg that is broken for our users in many contexts is not acceptable for my company.

Affinity Designer has fixed this bug on 1.8.4


[BugFix] Expand the supported data types when creating iceberg table

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/iceberg/IcebergApiConverter.java
0
1
00.00%
[130]

[FE PR Coverage Check]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/iceberg/IcebergApiConverter.java
0
1
00.00%
[130]


Terms update

Description
Refactor the project to replace old terminology and included the new ones, specifically it changes:

Governance for Issuance
Reciever for Recipient
Governance Rules for Administration Rules

Fixes #76
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected)
[ ] This change requires a documentation update

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented on my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my change

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

61 of 61 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
290

Relevant Lines:
290

💛  - Coveralls


Changing 1 room as Direct Chat, the API requires me to pass ALL the rooms which are Direct Chat

When I change a Group to Direct Chat via API, it looks like I need to pass ALL the rooms which are Direct Chats as Parameters. This does not sound right. Why would I need to pass ALL the rooms and IDs for changing 1 Room to Direct Chat.
Direct Chat "Accept" also does the same thing looks like
example I have A,B,C,D as Direct Chat and X,Y,Z as Rooms/Groups. When I change X to Direct Chat, I have to pass A,B,C,D,X to make just X as a Direct Chat
This is not right. We need to change the API.

Just want to say +1


Undefined class constant 'SKIPPED'

I've updated plugin from alpha.4 to beta.2 and changed
"require-dev": {
	"nette/tester": "dev-feature/teamcity-output"
},

to "nette/tester": "@dev"
Now, I get error:
Fatal error: Undefined class constant 'SKIPPED' in .../PhpStorm2017.1/intellij-nette-tester/setup.php on line 39

Duplicate of #30, in short: Nette Tester dev-master is not (yet?) supported. Please stick to 1.x stable release until it is resolved. Sorry for the inconvenience 🙁

Today I'm really blind 🙈