Combine "Install" and "Implementations"

Initial commit for review/discussion.
Revises implementations section to a single install section and adjusts
headers accordingly. Rearranges sections. Note we'll have to modify
headers on blog if we go this route.
ref #293

@ericronne -- Do you mind having another look? Incorporated most of your edits, but the text screen opacity made things a little illegible. Thanks!

@jessicaschilling i made some adjustments, lmk if they make any sense to you 🙏!

Removed the stars. Unless we can dim them, they really impede readability.
Unlinked the description text (which struck me as a non-intuitive behavior), so that only the h4label and the thumbnails link
Colored the descriptions with a lighter gray than the current site style specifies
Changed the text-hover to underlined, to better align with styling for other text links on the page. Note: the style still doesn't match perfectly. Other links on the page apply a border-bottom, which misbehaved when i attempted in this context.

@ericronne Thanks! These all LGTM. Sad to lose the stars, but you're correct on legibility, and since we're getting close to reworking this site as a whole, this as a fix seems fine. And good call on removing the hover for the caption text ... this made sense when the items were in boxes, but less so now.
@cwaring -- can you please have a look to hopefully bless this? Would be awesome to merge this Thursday. Thanks!

I've checked this over and pushed a few tweaks to keep things tidy™ - please take a look and feel free to merge if you're happy.

Thank you very much!


Add Attributes section to EV sidebar

Description
Added the new Attributes section to EV sidebar. It only contains the biotype attribute label and value for now.
Related JIRA Issue(s)
ENSWBSITES-951
Deployment URL(s)
http://ev-sidebar-attributes.review.ensembl.org
Views affected
Entity viewer -> Sidebar

@azangru Made that change with the latest commit 49ff1fa


Add use_external_data_format flag to transformer optimization pass

Describe your changes
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Format your code by running pre-commit run --all-files
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Should we close this PR? Because the bug is fixed in following PR where user does not need to provide the flag of use_external_data_format
https://github.com/microsoft/onnxruntime/pull/16912

Should we close this PR? Because the bug is fixed in following PR where user does not need to provide the flag of use_external_data_format microsoft/onnxruntime#16912

Yeah, closing.


[request] different ramp-volume and bar-volume cursors

Hello,
Would it be possible, and would be make sense to have different cursors for ramp-volume and bar-volume? Typically ramp-volume would use cursor-click while bar-volume would use cursor-scroll.
What do you think?

Currently the cursors are mapped to whatever action they are named (click and scroll), with the click cursor having priority.  Since the volume module has both scroll and click actions throughout the module, the click cursor would be shown by default.
I don't think having different cursors for the ramp and bar would be helpful, because it would contradict this order of precedence.


Windows runner

Found the issue with the workflow:
- name: Run unit tests
run: |
  dotnet test ./tests/ProjectA/ProjectA.csproj -c release
  dotnet test ./tests/ProjectB/ProjectB.csproj -c release
  dotnet test ./tests/ProjectC/ProjectC.csproj -c release

Let's assume ProjectA or ProjectB do have a test, which fails. Therefore the exit code is non-zero but ProjectC is successful the whole result code (in powershell) is for some weird reason 0 instead of non-zero.
So this is a temporary fix. Disadvantage right now is that we stop on the first failing test-suite but at least we have an indication whether or not windows is breaking.

I cant remember why I call dotnet test on individual projects. perhaps just simplify to dotnet test at the solution level and let it figure it out?

I cant remember why I call dotnet test on individual projects. perhaps just simplify to dotnet test at the solution level and let it figure it out?

Well that is way easier ;)

@egil Said and done.
Tried it out with the solution wide dotnet test and that works like a charm ;)

You should be allowed to complete the PR yourself now.


Fixed save world with fuel links

This PR fixes when saving a world file at it contains uris with fuel models. The current behaviour joins the fuel path to the URL. This condition avoids this.
Signed-off-by: ahcorde<EMAIL_ADDRESS>

Oh, I think this was fixed in ign-gazebo2 by #271. We just need to forward port it.

Closing this PR. Thank you @azeey


feat(binding/python): Support Copy operation for Python binding

Signed-off-by: Manjusaka<EMAIL_ADDRESS>

@Zheaoli, you're now a committer. Please submit your PR from the apache upstream to ensure all services cover this PR.

Oh, this PR triggered nodejs tests, something must be buggy in the planner.


Building for watchOS2 beta6

I manually added the files and getting:
WebSocket.swift:622:26: '[String]' does not have a member named 'joinWithSeparator'

This error is related to a recent code update. joinWithSeparator is a function in the latest Xcode 7 Beta 6. Are you running this version of Xcode?

My fault, mixed 5 and 6.
As I understand
";".join(subProtocols)

will fix it?

Yes. That will work for beta 5.


UPGRADE ERROR: readlink /var/lib/docker/overlay2: invalid argument

Hi,
Steps to reproduce the behavior:

Run 'make upgrade'
...
################################################# Start ntpd service container if needed ################################################# make[1]: Entering directory '/root/chainpoint-node' Local time sync daemon seems to be running. Skipping chainpoint-ntpd... make[1]: Leaving directory '/root/chainpoint-node' ################################################# Start upgraded Node ################################################# Creating network "chainpoint-node_chainpoint-node" with driver "bridge" ERROR: readlink /var/lib/docker/overlay2: invalid argument Makefile:84: recipe for target 'upgrade' failed make: *** [upgrade] Error 1 root@cf1:~/chainpoint-node#

**Operating System :

OS: Ubuntu 16.04.4 LTS
Output of uname --all
Linux cf1 4.4.0-127-generic #153-Ubuntu SMP Sat May 19 10:58:46 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Browser (for Node UI):
Browser Type Chrome

I would like to know how to pass this error or if required  installing it from scratch what would be the steps?
Regards,

@Arsca
Try running the following and report back to us whether or not that corrected the issue:

$ docker-compose down --remove-orphans
$ git fetch && git checkout master && git pull
$ make upgrade

@michael-iglesias

Upgrading to 1.5.4 will require the following upgrade command:

I read the blog and did it accordingly withe the same command.
This is the result of the commands :

root@cf1:~/chainpoint-node#
uname --all
Linux cf1 4.4.0-127-generic #153-Ubuntu SMP Sat May 19 10:58:46 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
root@cf1:~/chainpoint-node# docker-compose down --remove-orphans
Removing network chainpoint-node_chainpoint-node
root@cf1:~/chainpoint-node# git fetch && git checkout master && git pull
Already on 'master'
Your branch is up-to-date with 'origin/master'.
Already up-to-date.
root@cf1:~/chainpoint-node# make upgrade
curl -sSL https://chainpoint-node.storage.googleapis.com/upgrade.sh | bash
...
Creating network "chainpoint-node_chainpoint-node" with driver "bridge" ERROR: readlink /var/lib/docker/overlay2: invalid argument Makefile:84: recipe for target 'upgrade' failed make: *** [upgrade] Error 1

This is the installed docker version:
root@cf1:~/chainpoint-node# docker-compose -v
docker-compose version 1.21.0, build 5920eb0
root@cf1:~/chainpoint-node# docker -v
Docker version 18.06.1-ce, build e68fc7a

@Arsca
Can you run $ docker ps and provide the output?

root@cf1:/tmp# docker ps
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES

@Arsca
Try running:

$ docker network prune
$ docker system prune -a
$ curl -sSL https://chainpoint-node.storage.googleapis.com/upgrade.sh | bash

Let me know if that resolves the issue. If that doesn't work I have a few other ideas that we can try based on some of the research I've done regarding this issue.

@michael-iglesias
Thanks!  That was it, it's up and running :-D

Awesome! Glad we were able to figure it out.
Happy Holidays!

Closing this as resolved.


UserMessage component basics

Very basic first iteration, just so that it can be reused for the AgentMessage if needed.
Next steps:

Handle propertly the udpate on the layout so that we can have the grey background on the full width. EDIT: Actually done on this PR
Handle markdown.
Handle mentions (pending some API work first)

Looks great! Left a small change request on who manages the width, I think it should be the Conversation component not the message one.


Using rclone to write to a google drive crypt mount fails

rclone -V
rclone v1.35-137-ge2f0feeβ
In my case /data/Media3 is a google mount crypt.  I am trying to write to it and replace files:
2017/03/02 08:42:34 NOTICE: Config file "" not found - using defaults
2017/03/02 08:42:34 DEBUG : rclone: Version "v1.35-137-ge2f0feeβ" starting with parameters ["rclone" "--delete-before" "--stats" "0" "--config=" "--transfers=16" "--checkers=64" "copy" "-vv" "-c" "/data/Media1/Videos/Series/Van.Helsing/tvshow.nfo" "/data/Media3/"]
2017/03/02 08:42:34 INFO  : Local file system at /data/Media3: Modify window is 1ns
2017/03/02 08:42:34 INFO  : Local file system at /data/Media3: Waiting for checks to finish
2017/03/02 08:42:34 DEBUG : tvshow.nfo: MD5 = 21823f62fb2822d14c7b42c15fed58b1 (Local file system at /data/Media1/Videos/Series/Van.Helsing)
2017/03/02 08:42:34 DEBUG : tvshow.nfo: MD5 = 73758a0d8836f93e88ebda6f20f14bf3 (Local file system at /data/Media3)
2017/03/02 08:42:34 DEBUG : tvshow.nfo: MD5 differ
2017/03/02 08:42:34 INFO  : Local file system at /data/Media3: Waiting for transfers to finish
2017/03/02 08:42:34 ERROR : tvshow.nfo: Failed to copy: open /data/Media3/tvshow.nfo: input/output error
2017/03/02 08:42:34 ERROR : Attempt 1/3 failed with 1 errors and: open /data/Media3/tvshow.nfo: input/output error

If I remove that file first and then retry the write it succeeds... Sort of.  It writes the file but then can't change the times.
 rclone --delete-before --stats 0 --config=$RCLONECONF --transfers=16 --checkers=64 copy -vv -c $OPT /data/Media1/Videos/Series/Van.Helsing/tvshow.nfo /data/Media3/
2017/03/02 08:47:13 NOTICE: Config file "" not found - using defaults
2017/03/02 08:47:13 DEBUG : rclone: Version "v1.35-137-ge2f0feeβ" starting with parameters ["rclone" "--delete-before" "--stats" "0" "--config=" "--transfers=16" "--checkers=64" "copy" "-vv" "-c" "/data/Media1/Videos/Series/Van.Helsing/tvshow.nfo" "/data/Media3/"]
2017/03/02 08:47:13 INFO  : Local file system at /data/Media3: Modify window is 1ns
2017/03/02 08:47:13 INFO  : Local file system at /data/Media3: Waiting for checks to finish
2017/03/02 08:47:13 INFO  : Local file system at /data/Media3: Waiting for transfers to finish
2017/03/02 08:47:13 ERROR : tvshow.nfo: Failed to copy: chtimes /data/Media3/tvshow.nfo: operation not permitted
2017/03/02 08:47:13 ERROR : Attempt 1/3 failed with 1 errors and: chtimes /data/Media3/tvshow.nfo: operation not permitted
2017/03/02 08:47:13 INFO  : Local file system at /data/Media3: Waiting for checks to finish
2017/03/02 08:47:13 DEBUG : tvshow.nfo: Size and MD5 of src and dst objects identical
2017/03/02 08:47:13 DEBUG : tvshow.nfo: Unchanged skipping
2017/03/02 08:47:13 INFO  : Local file system at /data/Media3: Waiting for transfers to finish
2017/03/02 08:47:13 ERROR : Attempt 2/3 succeeded
2017/03/02 08:47:13 DEBUG : Go routines at exit 2
2017/03/02 08:47:13 DEBUG : rclone: Version "v1.35-137-ge2f0feeβ" finishing with parameters ["rclone" "--delete-before" "--stats" "0" "--config=" "--transfers=16" "--checkers=64" "copy" "-vv" "-c" "/data/Media1/Videos/Series/Van.Helsing/tvshow.nfo" "/data/Media3/"]

ls -l /data/Media3/tvshow.nfo
-rw-rw-r-- 1 deluge mediashare 4972 Mar  1 17:42 /data/Media3/tvshow.nfo

I can replicate that.  So this is a mount restriction.  Should it be?
touch -r /data/Media1/Videos/Series/Van.Helsing/tvshow.nfo /data/Media3/tvshow.nfo
touch: setting times of ‘/data/Media3/tvshow.nfo’: Operation not permitted

I am not 100% sure that the failure in the first example is the same as the times issue but i'd suspect it has something to do with it.  I see two options, allow hte changing of hte time stamps (which google drive allows I believe) or have a flag(s) to disable the destination time modification.
rclone, however, should be able to write correctly to its own mount. :)  I would think the right approach would be to have the mount update the times if possible.  I think some of the underlying storage supports this (local does and gdrive?).  For ones that do not support this, perhaps the fuse/mount should silently fail instead of throwing errors?

The error from the fuse mount with --debug-fuse is
2017/03/02 16:03:32 DEBUG : fuse: -> [ID=0x9a] Open error=EIO: can't open read and write

Which makes sense.  I can fix that in the local fs so it will work very easily.
It also needs some more debug lines shouldn't need --debug-fuse to see this error.
The can't change the times I'll do in #1197

Sounds good.  Let me know if you'd like me to test the beta.

I've fixed this in: http://beta.rclone.org/v1.35-153-g1cc58e4/ (uploaded in 15-30 mins)
That is the logging and the write mode.

Working great.


[decomp] joint-mod-h

Pull Request Test Coverage Report for Build 910471000

33 of 100 (33.0%)  changed or added relevant lines in 3 files are covered.
13 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 69.5%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

decompiler/IR2/Form.cpp
0
7
0.0%

decompiler/IR2/FormExpressionAnalysis.cpp
0
13
0.0%

decompiler/analysis/insert_lets.cpp
33
80
41.25%

Files with Coverage Reduction
New Missed Lines
%

common/type_system/TypeFieldLookup.cpp
2
84.39%

decompiler/IR2/Form.cpp
11
73.35%

Totals

Change from base Build 910146993:
-0.1%

Covered Lines:
36168

Relevant Lines:
52040

💛  - Coveralls


nth-child selector throws number formatting exception as of 3.8

Jerry.jerry("<html></html>").find("div:nth-child(2) > div:nth-child(1)")

raises the following exception
Caught: jodd.csselly.CSSellyException: Parsing error.; <--- java.lang.NumberFormatException: For input string: "2)>div:"
jodd.csselly.CSSellyException: Parsing error.; <--- java.lang.NumberFormatException: For input string: "2)>div:"
	at jodd.csselly.selector.PseudoFunctionExpression.parseInt(PseudoFunctionExpression.java:81)
	at jodd.csselly.selector.PseudoFunctionExpression.<init>(PseudoFunctionExpression.java:58)
	at jodd.csselly.selector.PseudoFunction$NTH_CHILD.parseExpression(PseudoFunction.java:55)
	at jodd.csselly.selector.PseudoFunction$NTH_CHILD.parseExpression(PseudoFunction.java:51)
	at jodd.csselly.selector.PseudoFunctionSelector.<init>(PseudoFunctionSelector.java:101)
	at jodd.csselly.CssSelector.addPseudoFunctionSelector(CssSelector.java:155)
	at jodd.csselly.CSSellyLexer.yylex(CSSellyLexer.java:769)
	at jodd.csselly.CSSelly.parse(CSSelly.java:61)
	at jodd.csselly.CSSelly.parse(CSSelly.java:96)
	at jodd.lagarto.dom.NodeSelector.select(NodeSelector.java:55)
	at jodd.jerry.Jerry.find(Jerry.java:388)

It seems to have something to do with the two specified indices (replacing the second one with first-child fixes it).
Could it be because of the recent changes in https://github.com/oblac/jodd/commits/e49d13bec7eece852d1f0f796bf2546126bbd70a/jodd-lagarto/src/main/java/jodd/csselly/CSSellyLexer.java?
It doesnt work in 3.8.5 but worked in 3.7

This change seems to be the reason
https://github.com/oblac/jodd/commit/e49d13bec7eece852d1f0f796bf2546126bbd70a#diff-62a2422d8ee3966d57ec5d0572a9434fL822

Thanx @neroux!!! 👍  I was afraid I am missing something there... parsers are fun, you never know when it stops working :)))

@igr 👍 Thanks for the quick fix, it does work.
But yes, string parsing is always a particular fun ;)


Azure Storage Queue Golang SDK leaks tcp connections on Linux

The queue golang SDK leaks TCP connections, and causes the program to run out of handles and stop running correctly.
To repro the leak:

add the below program to main.go
update storage account information with your SA, Key, and a queuename
run the program, and run lsof | grep PROGRAMNAME | wc -l and you will see the leaked TCP connections.

package main

import (
	"context"
	"fmt"
	"log"
	"net/url"
	"os"
	"os/signal"
	"time"

	"github.com/Azure/azure-storage-queue-go/2017-07-29/azqueue"
)

// built from https://github.com/Azure/azure-storage-queue-go/blob/master/2017-07-29/azqueue/zt_examples_test.go

func main() {
	storageAccount := "YOURSTORAGEACCOUNT"
	storageAccountKey := "YOURKEY"
	storageQueue := "queuename"
	credential := azqueue.NewSharedKeyCredential(storageAccount, storageAccountKey)
	p := azqueue.NewPipeline(credential, azqueue.PipelineOptions{})
	u, _ := url.Parse(fmt.Sprintf("https://%s.queue.core.windows.net", storageAccount))
	serviceURL := azqueue.NewServiceURL(*u, p)
	queueURL := serviceURL.NewQueueURL(storageQueue) // Queue names require lowercase
	messagesUrl := queueURL.NewMessagesURL()

	ctx := context.Background()
	log.Printf("started queuing messages")
	for i := 0; i < 100; i++ {
		messagesUrl.Enqueue(ctx, fmt.Sprintf("queuemsage%d", i), time.Duration(0)*time.Second, time.Duration(-1)*time.Second)
	}

	log.Printf("finished queuing messages")
	// wait on ctrl-c
	sigchan := make(chan os.Signal, 10)
	signal.Notify(sigchan)
	<-sigchan
	log.Printf("finished")
}

This only repro's on linux, and I've been unable to get it to repro under Windows.

The problem is with the queue's pipeline factory: https://github.com/Azure/azure-storage-queue-go/blob/master/2017-07-29/azqueue/zc_pipeline.go#L88.  Every request creates a "newDefaultHTTPClient()" until you send or receive just over 6000 queue messages and exhaust the limits.  The documentation for httpClient, https://golang.org/pkg/net/http/, says that "Clients and Transports are safe for concurrent use by multiple goroutines and for efficiency should only be created once and re-used.".
In contrast the azure pipeline code re-uses the same client as seen here https://github.com/Azure/azure-pipeline-go/blob/master/pipeline/core.go#L201.

Hi @anhowe, I appreciate your patience. This should now be fixed in 0.2.0.


version:^1.4.5 is inconsistent with the master code

I want to ask the question to see if everyone has the same question?

英文翻译出处为谷歌，大家多担待。
The source of the English translation is Google.


Automate additional steps in creating new project

Currently, ballet quickstart renders a new project to a local directory. But to follow the steps on the maintainer guide, there are manual steps to (1) push to github (2) enable CI (3) enable bots. These could also be automated through the quickstart command

add --push/--no-push option to push this project to github. the remote is known from the rendering step. token will be obtained in order from (1) the usual $GITHUB_TOKEN envvar (2) provided at the command line with --github-token.
add --enable-ci/--no-enable-ci option to enable the CI service. if the CI service is GH Actions, this is a noop. since we are moving there anyway, we may pass on implementing this.
add --enable-bots/--no-enable-bots option to enable the Ballet bot and the repolockr bot.

Looks like you can "add repositories to an existing installation" here: https://docs.github.com/en/rest/reference/apps#add-a-repository-to-an-app-installation
If the user knows an existing installation id they can list repositories that they have permission to access: https://docs.github.com/en/rest/reference/apps#list-repositories-accessible-to-the-user-access-token


output padding must be smaller than either stride or dilation

Hi, congrats on the repo! I am trying to use it with the following dataset (as I don't have access to wsj):
train_set, val_set = LibriMix.mini_from_download(task='sep_clean')
I replace the dataset in DataLoader() with the train_setabove. When training the network I have the following error:
RuntimeError: output padding must be smaller than either stride or dilation, but got output_padding_height: 0 output_padding_width: 8000 stride_height: 1 stride_width: 10 dilation_height: 1 dilation_width: 1
Is this due to the librimix dataset?

Sorry for the late reply. For this repo, the waveform sample rate should be 8k. Did you check your data's sample rate?


Update API.md regarding props and mutability

Maybe too long explanation for API listing? Any idea where else to put it?

@koresar Thanks, I still have a trouble with articles obviously :)

Everybody has a few :)
A few issues remain, that I can help with on a later release. Please
contact me at that time.
On 2016-09-21 08:16, Daniel K. wrote:

@koresar https://github.com/koresar Thanks, I still have a trouble
with articles obviously :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stampit-org/stampit/pull/234#issuecomment-248593839,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKTA13Em5JIaRaFdHB2492-7Awia7Wgqks5qsSAMgaJpZM4KCr5U.

--

Thanks @dpatte
Just click this link and edit the file. We'd be very grateful for the improvement you can do!


Chinese translation is not completed

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.61.2
OS Version: MacOS Big Sur

I already installed Chinese (Simplified) (简体中文) Language Pack for Visual Studio Code
v1.62.0, but there are still some texts which aren't translated to Chinese:

Looks like the translation is there now:
https://github.com/microsoft/vscode-loc/blob/main/i18n/vscode-language-pack-zh-hans/translations/main.i18n.json#L2816
Closing this. @HaoboGu please give this another go with the latest Language Pack and let us know.

The "Open Workspace from File..." has been translated now, and I can see it on Insiders. I'll leave the verified label for the endgame.


Verify shcellscript by shellcheck

What would you like to be cleaned:
It would be better to verify all shell scripts by the spellcheck every PR.
Why is this needed:
We should have more stable shell scripts.

/good-first-issue

Hi, I would like to work on this, @tenzen-y
do we create a ci workflow for this .

@jaydee029 No, we don't. We need to add a Make target like https://github.com/kubernetes-sigs/kueue/blob/b83b2448fbfb8ad263724ee53277081955d2735a/Makefile#L189-L191, then we can call the target in the Make verify target: https://github.com/kubernetes-sigs/kueue/blob/b83b2448fbfb8ad263724ee53277081955d2735a/Makefile#L194

okay , sure

okay , sure

@jaydee029 You can take this issue by the comment as /assign

/assign

Hey @jaydee029, Are you still working on this?
If not, I would like to take a stab on it.

Go for it

/assign

Hi @Vandit1604
Are you still working on it? If not I'm interested
Thanks

Hey @mayens ,
I'm working on this.
Thanks


OCW: block search engines from crawling ocw.mit.edu/ans7870/

The HTML pages hosted on ans7870 have not been updated in years. Many of them cause Google's search engine crawler to throw errors, decreasing the mobile score for the site.
For now, while we don't understand what content is in this tree, we should block it from being indexed. Perhaps after we review it, remove archived material and update HTML for the modern era, we will allow the search engines to index it again.

By "update the HTML" I'm hoping you mean not in the ans7870 route...


docs troubleshooting

Add cargo-make to supported tools
docs: add troubleshooting to development.md

Inadvertently still had the changes from #146 in this - I'll clean this up once #146 is merged.

Rebased on main

fixed typo

whoops missed the merge (another PR incoming)


Canonical package directory structure.

Following guide:
http://julia.readthedocs.org/en/latest/manual/packages/

done


Upgrade dependency paquettg/string-encode (1.0.1) to ^2.1.1

https://github.com/paquettg/string-encoder

The Encode object has been removed in 2.0.0 and Encoder should be used in these places;
https://github.com/paquettg/php-html-parser/blob/f5c2dd9b8abd03cfe9383efe2575c7ff0d9711ea/src/PHPHtmlParser/DTO/Tag/AttributeDTO.php#L56
https://github.com/paquettg/php-html-parser/blob/4e01a438ad5961cc2d7427eb9798d213c8a12629/src/PHPHtmlParser/Dom/Tag.php#L138
https://github.com/paquettg/php-html-parser/blob/4e01a438ad5961cc2d7427eb9798d213c8a12629/src/PHPHtmlParser/Dom/Parser.php#L107

This should be fixed by PR https://github.com/paquettg/php-html-parser/pull/289


WIP: Update Armory to use setuptools_scm

Updating Armory to use the PyPI suggested setuptools_scm versioning system ( https://github.com/pypa/setuptools_scm).

Note @mwartell , this assumes we put in pytest-cleanup branch first.


Implement USB for samd51, particularly the itsybitsy_m4

Not yet 100% ready to go, but wanted to see what travis CI would do.
This is mostly based on the samd21 implementation, but updated for the newer usb_device implementation, different NVIC, different clocking etc.
Comments welcome.

Looks great! Could you update samd21 as well?

I would rather do that in a separate PR. I don't yet have a SAMD21 device.

Fair enough.

Let me know when you think it's ready for merge.

I think this is ready for review. A few comments:

I'm not sure of the best way to setup a UART for use with the dbgprint! macro. The uart_debug thing / feature is what I came up with, but I don't like it and would defs like to change it to something more elegant if you have any ideas.

I haven't yet incremented any versions in the cargo files. Reckon this is a minor or major semver change?

If it doesn't break anything it would be a minor revision I think.

Okies, I've modified the feature name to match the convention established by use_rtt (uart_debug --> use_uart_debug), but otherwise I've leave it as-is.

Could you push out updated crate versions for samd51-hal and itsybitsy_m4? :)

Done.


showing suggestion string

Long minutes to display the suggestion string by typing

/needsMoreInfo
As well as adding details from the About dialog off the Help menu please use the Start Extension Bisect command to investigate whether the problem is being caused by one of your extensions.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Integration: NextJS

Context
To make it easier to start jitar using the frameworks developers are known with, we need to create integrations and examples for the most popular frameworks.
Solution
To support developers that use nextjs we need a plugin and an example how to use it with nextjs.

Vercel has decided to go for Rust and not to pickup Vite
https://github.com/vercel/next.js/discussions/22406


How to use multi store?

Hi team,
I would like to use multi store, show how to use api?

Hi,
You can get all the stores with /api/stores call and then set the specifics  "store_ids" on the products or category you want to be limited.


Ethereum pulbic keys

I hope I didn't miss something very basic here, but (if I understand) Ethereum uses 512-bit public keys, but the bip39 tool (in the "Derived Addresses" section) shows only a 256-bit public key (plus a 0x02 or 0x03 prefix).
How do I get the entire 512-bit Ethereum public key?

In BIP32 that most wallets use for deriving addresses, 512 bits are used for extended private keys, half of which is used for a given address, the other half (chain code) for randomizing derivation. But the actual signing uses only a 256 bit private key.

I don't understand your reply. Addresses get calculated from public keys (which, of course, do come from private keys). Ethereum uses the entire (uncompressed) 512-bit public key to produce addresses.
https://medium.com/coinmonks/calculation-behind-ethereum-wallet-address-d2aa43ee1e37
Since the BIP 39 page must have the full public key to compute Ethereum addresses, I would like the option to display the full public key in the "Derived Addresses" section.

I hope this project can help you out with your findings
https://github.com/ConsenSys/eth-lightwallet


Video attachments:

Quick summary
Video attachments appear with two attachments, one of them broken
Steps to reproduce
You can see it in my recent post with a video: https://mattwiebe.blog/2024/11/24/support-week-for-activitypub-for-wordpress/
What it looks like on Phanpy:

So when I get the ActivityPub version, the attachment field shows these two entries:
[
  {
    "url": "https://videos.files.wordpress.com/9qROHgXQ/activitypub-office-hours-nov-25.mp4",
    "length": "114070319",
    "mediaType": "video/mp4"
  },
  {
    "type": "Document",
    "mediaType": "video/mp4",
    "url": "https://videos.files.wordpress.com/9qROHgXQ/activitypub-office-hours-nov-25_mp4_std.mp4",
    "width": "1920",
    "height": "1080",
    "name": "Video",
    "icon": {
      "type": "Image",
      "mediaType": "image/jpeg",
      "url": "https://videos.files.wordpress.com/9qROHgXQ/activitypub-office-hours-nov-25_mp4_hd_1080p.original.jpg"
    }
  }
]

Unsure if this is specifically a dotcom thing or not but it looks like the first one is broken and the second one works.
What you expected to happen
One video
What actually happened
Two videos, one broken.
Impact
Some (< 50%)
Available workarounds?
No but the platform is still usable
Logs or notes
No response

WordPress.com seems to add the video as enclosure. From the feed:
<enclosure url="http://videos.files.wordpress.com/9qROHgXQ/activitypub-office-hours-nov-25.mp4" length="114070319" type="video/mp4"/>

I think we have to optimize the enclosure code a bit: https://github.com/Automattic/wordpress-activitypub/blob/129ea69ccf102612b4c9cbb52a8814c3e8f8d8a4/includes/transformer/class-post.php#L311

I have a fix for wpcom in D167257-code

Fixed in D167257-code


transgui: init at 5.0.1-svn-r986

Motivation for this change
https://github.com/NixOS/nixpkgs/pull/16567
https://www.reddit.com/r/NixOS/comments/4owwga/transmission_remote_gui_transgui_package/
Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

New PR. Same code. Sorry for all the trouble.
Thanks.
@joachifm


Add Youtube Links

Add Youtube 🔗 link of Eddie
Add Youtube 🔗 link of Kunal
Apply 🔠  Sorting to 📺 Youtube Channel section

Hi there,
Please can anyone review my PR

@sdmg15

Why are there so much changes ?

Added two youtube links of [ Kunal Kushwaha, Eddie ]
and In the youtube channel section I applied sorting to sort links according to A-Z

I haven't changed any of the links but I just sorted them according to alphabetic order.
you can compare that.


Trusted Platform Module Services: Turn on TPM backup to Active Directory Domain Services

This setting has been removed since Windows 10 1607, please update the documentation.

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

ID: 8d1e77d9-00eb-7907-08ba-ddb5557fa51e
Version Independent ID: 5a0a4a51-0bea-a266-1796-f21918cd04e2
Content: Prepare your organization for BitLocker Planning and policies (Windows 10)
Content Source: windows/security/information-protection/bitlocker/prepare-your-organization-for-bitlocker-planning-and-policies.md
Product: w10
GitHub Login: @brianlic-msft
Microsoft Alias: justinha

@santoshseth Thanks for pointing this out. I took out the TPM policy info.

Hello
Could some one please post the working GPO for windows 10 and bitlocker to AD.
we use TPM and UEFI.
thanks

Hello
Could some one please post the working GPO for windows 10 and bitlocker to AD.
we use TPM and UEFI.
thanks

take a look here: https://docs.microsoft.com/en-us/windows/security/information-protection/bitlocker/bitlocker-group-policy-settings#a-href-idbkmk-rec3astore-bitlocker-recovery-information-in-active-directory-domain-services-windows-server-2008-and-windows-vista

Hello
thank you for the information, but the only section that talks about storing Bitlocker key to AD applies to windows 2008 and vista:
[cid:04fca661-d96b-493c-8e40-36882cf72883]

From: Justin Hall<EMAIL_ADDRESS>Sent: Monday, April 29, 2019 2:51 PM
To: MicrosoftDocs/windows-itpro-docs
Cc: Sidans; Comment
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Trusted Platform Module Services: Turn on TPM backup to Active Directory Domain Services (#1015)
Hello
Could some one please post the working GPO for windows 10 and bitlocker to AD.
we use TPM and UEFI.
thanks
take a look here: https://docs.microsoft.com/en-us/windows/security/information-protection/bitlocker/bitlocker-group-policy-settings#a-href-idbkmk-rec3astore-bitlocker-recovery-information-in-active-directory-domain-services-windows-server-2008-and-windows-vista
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1015#issuecomment-487699521, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH5FYXCYGOTOVT3YSPSUGC3PS47T7ANCNFSM4FDBRDBQ.

@Sidans my bad, here it is: https://docs.microsoft.com/en-us/windows/security/information-protection/bitlocker/bitlocker-group-policy-settings#a-href-idbkmk-rec1achoose-how-bitlocker-protected-operating-system-drives-can-be-recovered
See the notes below the table for info about storing to AD on prem.

Hello
I still don't see it
could you give me an exact keyword that I can search on the page.
it still looks the same as previous link
thanks

From: Justin Hall<EMAIL_ADDRESS>Sent: Monday, April 29, 2019 4:02 PM
To: MicrosoftDocs/windows-itpro-docs
Cc: Sidans; Mention
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Trusted Platform Module Services: Turn on TPM backup to Active Directory Domain Services (#1015)
@Sidanshttps://github.com/Sidans my bad, here it is: https://docs.microsoft.com/en-us/windows/security/information-protection/bitlocker/bitlocker-group-policy-settings#a-href-idbkmk-rec1achoose-how-bitlocker-protected-operating-system-drives-can-be-recovered
See the notes below the table for info about storing to AD on prem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1015#issuecomment-487722535, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH5FYXCGTXXNUGSEUPWDI2DPS5H5RANCNFSM4FDBRDBQ.

The policy name is Choose how BitLocker-protected operating system drives can be recovered. In the Reference section below the table, it says:
In Save BitLocker recovery information to Active Directory Domain Services, choose which BitLocker recovery information to store in Active Directory Domain Services (AD DS) for operating system drives. If you select Store recovery password and key packages, the BitLocker recovery password and the key package are stored in AD DS. Storing the key package supports recovering data from a drive that is physically corrupted. If you select Store recovery password only, only the recovery password is stored in AD DS.
The link I shared will take you right to the correct setting name. Or just click Start > Edit Group Policy and navigate to Computer Configuration\Administrative Templates\Windows Components\BitLocker Drive Encryption\Operating System Drive and you can see the setting there. The option to save recovery info to AD DS is in a pull-down menu.

thanks

From: Justin Hall<EMAIL_ADDRESS>Sent: Monday, April 29, 2019 5:55 PM
To: MicrosoftDocs/windows-itpro-docs
Cc: Sidans; Mention
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Trusted Platform Module Services: Turn on TPM backup to Active Directory Domain Services (#1015)
The policy name is Choose how BitLocker-protected operating system drives can be recovered. In the Reference section below the table, it says:
In Save BitLocker recovery information to Active Directory Domain Services, choose which BitLocker recovery information to store in Active Directory Domain Services (AD DS) for operating system drives. If you select Store recovery password and key packages, the BitLocker recovery password and the key package are stored in AD DS. Storing the key package supports recovering data from a drive that is physically corrupted. If you select Store recovery password only, only the recovery password is stored in AD DS.
The link I shared will take you right to the correct setting name. Or just click Start > Edit Group Policy and navigate to Computer Configuration\Administrative Templates\Windows Components\BitLocker Drive Encryption\Operating System Drive and you can see the setting there. The option to save recovery info to AD DS is in a pull-down menu.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1015#issuecomment-487758204, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH5FYXDGNPMIEWIAQD4QBJLPS5VGTANCNFSM4FDBRDBQ.


unknown field 'MaxConnsPerHost' in struct literal of type http.Transport

When I run the "go get" command, I receive an error:
root@proxy-test2:/tmp/doh5# go get github.com/emilliken/doh5
github.com/emilliken/doh5
/root/go/src/github.com/emilliken/doh5/resolver.go:64:19: unknown field 'MaxConnsPerHost' in struct literal of type http.Transport

Which version of Go are you using?  MaxConnsPerHost field was added in Go 1.11:
https://golang.org/pkg/net/http/#Transport.MaxConnsPerHost
I should probably document this...

I've now documented this in the README in commit https://github.com/emilliken/doh5/commit/21c3f06fd3c967bad2eb4f8ec9b6eb9c94667255


Update setup-windows10-sdk action to latest version

Fixes a Github Actions warning.

What was the warning btw? for future reference if we find this again?


Write type tests with pytest-mypy-plugins

This might be useful for a few cases, but should probably be done mostly with assert_type in existing tests.

One useful case for this will be to make sure that the format_monetary function raises type errors for incorrect parameters.


Black Screen pauses between images

I'm a tech librarian and our library has used this app for a while with Roku sticks to display digital adverts on about a dozen tvs around our library. Recently we have started having an issue where there is a long pause between images while on display. This has only began in the last week or so. I have checked all of the settings exhaustively and couldn't find a reason why this pause exists. It is consistent across all of our devices. After searching through reddit and here, I have become frustrated with no clear answer to why it is happening. Any ideas Chris?

Are you seeing this on the screensaver or in the app itself? And is it when the slideshow starts or only after it’s been running for a while?

Within the app. I will start the slideshow and it will start giving a black
screen between every image. It persists the whole time it is opened. I have
changed every setting I could think of to see if anything changes it and I
have had no luck.
On Sun, Jun 21, 2020 at 9:08 AM Chris Taylor<EMAIL_ADDRESS>wrote:

Are you seeing this on the screensaver or in the app itself? And is it
when the slideshow starts or only after it’s been running for a while?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/319#issuecomment-647133141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP55W2BW5VJ4TGLPJNJZR3LRXYH55ANCNFSM4N73EFLQ
.

--
Samuel Medlin
M.L.I.S. The University of Alabama
B.M. Jacksonville University
<EMAIL_ADDRESS>

Ok, i'm able to reproduce on one of my albums, but not consistently. Trying to narrow down the issue.
1.) The album you are using, now many total images are part of the slideshow?
2.) Does it seem to happy immediately after starting or within the first 45-60 minutes?

Usually we have between 20 and 40 slides
And it begins pretty much within the first 10 minutes of the slideshow
On Tue, Jun 23, 2020, 11:47 PM Chris Taylor<EMAIL_ADDRESS>wrote:

Ok, i'm able to reproduce on one of my albums, but not consistently.
Trying to narrow down the issue.
1.) The album you are using, now many total images are part of the
slideshow?
2.) Does it seem to happy immediately after starting or within the first
45-60 minutes?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/319#issuecomment-648582140,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP55W2CA3W453I7M47GIRRTRYGANNANCNFSM4N73EFLQ
.

Ok, I think I've narowed it down to this libcurl bug (https://github.com/curl/curl/issues/1967) in Roku's OS. I was able to put a work around in to fix it and seems to be working for me.
@medlinsamuel - Can you please test this and confirm if it fixes your issue? Add the custom channel by clicking on this link:
https://my.roku.com/account/add/HZN2KN
Ideally, you shouldn't see any anymore black screens. The "PAUSE" display bar will give you some debug details on any errors with image pulling.  If seems to fix the issue, i'll get this bug fix published.

Absolutely man I'll test it first thing tomorrow morning and let you know.
On Thu, Jun 25, 2020, 9:13 PM Chris Taylor<EMAIL_ADDRESS>wrote:

Ok, I think I've narowed it down to this libcurl bug (curl/curl#1967
https://github.com/curl/curl/issues/1967) in Roku's OS. I was able to
put a work around in to fix it and seems to be working for me.
@medlinsamuel https://github.com/medlinsamuel - Can you please test
this and confirm if it fixes your issue? Add the custom channel by clicking
on this link:
https://my.roku.com/account/add/HZN2KN
Ideally, you shouldn't see any anymore black screens. The "PAUSE" display
bar will give you some debug details on any errors with image pulling. If
seems to fix the issue, i'll get this bug fix published.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/319#issuecomment-649909967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP55W2CO2POY7XTZJTKHEP3RYP74HANCNFSM4N73EFLQ
.


feat: Add flag control for enabling mocks

Description
This PR deals with resolving the issue in hand https://github.com/sharkio-dev/sharkio/issues/359 to add flag control to enable/disable mocks at the sniffer level.
Checklist
Please make sure you've done the following before submitting this pull request:

[x] Ran all existing tests to ensure they pass.
[ ] Added new tests to cover the changes (if applicable).
[x] Reviewed the code for any potential issues or bugs.
[ ] Updated the documentation (if needed) to reflect the changes.
[x] Squashed your commits into a single logical commit (if necessary).

@ypotter: Some of the file changes are purely linter/prettier changes. Previously few snippets were not following the setup format, hence it was fixed in the current PR by pre-commit hook.


PAB does not report invalid node socket path

Summary
The PAB silently fails to handle blockchain effects if it cannot connect to the node socket in its configuration.
Steps to reproduce the behavior

Edit the configuration file for the PAB and set pscSocketPath to any path that does not exist.
Start the PAB.

Actual Result
Observe that no error message is reported, but the PAB runs without a connection to the node, and hence never notifies clients of blockchain effects..
Expected Result
The PAB should fail because of this fatal misconfiguration and report that it could not connect to the node.
Describe the approach you would take to fix this
Add an error handler and logger to https://github.com/input-output-hk/plutus-apps/blob/ea1bfc6a49ee731c67ada3bfb326ee798001701a/plutus-pab/src/Plutus/PAB/Core/ContractInstance/BlockchainEnv.hs#L58.
System info
any

@koslambrou, here is how we fixed this for Marlowe: e0a2922bcd6eba1f92ec9b440c3336e37b80ef7b. The underlying issue is that the chain index connects within forkIO so the thread silently dies.

The underlying issue is that the chain index connects within forkIO so the thread silently dies.

The chain index? You mean the PAB, no?


Fix concurrency limiter inbounds

When getting the data for the concurrency limiter from an inbound message, the in_reply_to field might be empty. This was causing a MultipleObjectsReturned exception since there can be multiple Outbounds with an empty vumi_message_id.
This checks that field isn't empty before trying to find a related Outbound message.

@rudigiesler Please review


Upload 0.3.7 to crates.io

I see 0.3.7 has been tagged here, but it is not on crates.io.

Duplidate of #81

Indeed.


Update starlette to 0.12.2

better support for mypy in request.state

Thank you! :cake: I just updated it to include the latest and rebased. It's merged now :rocket: :tada:

Woooo 🥂

This also fixes redirects.  Now any route that ends in a slash can be properly hit without the trailing slash, if redirect is on.


Add NULL check in ensure_utf8

Proposed Changes
Added check to verify that any utf8 string doesn't contain a NULL character, which is not allowed by the MQTT standard. This PR fixes the issue #2355.
This change is applied to both ensure_utf8 functions in vmq_parser.erl and vmq_parser_mqtt5.erl
Types of Changes

[x] Bugfix (non-breaking change which fixes issue #2355 )
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (correction or otherwise)
[ ] Cosmetics (whitespace, styles...)
[ ] DevOps (Build scripts, pipelines...)

Checklist

[x] I have read the CODE_OF_CONDUCT.md document
[x] All tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if needed)
[x] Any dependent changes have been merged and published in related repositories
[ ] I have updated changelog (At the bottom of the release version)
[x] I have squashed all my commits into one before merging

Further Comments

@tiz314 Thanks!
I wonder whether we want to add a test case for this, where we'd generate a Connect packet with something like gen_connect(<<0>>, []).
Also, I suppose NULL is forbidden for the MQTT username and any other UTF-8 strings here we use ensure_utf8 as well), but I haven't yet looked into that.

Of course! Following the standard it says that no UTF-8 encoded string are allowed to contain U+0000, so we should check every field in every possible MQTT packet type and add a check on them if the standard says it is processed as a string.
As regards the test case, if you give me a go I can add a connect test in vmq_server/test

Adding a test to the vmq_server connect suite is great!
A previous commit by @mths1 added the UTF-8 verification for Username and Topic as well.

I'm quite struggling with the development of the test case. Do you use common test? Which tool do you use to run it? I tried with rebar3 but it gives me errors about files not found, such as:
vmq_commons/src/vmq_types_common.hrl:{3,14}: can't find include lib "vernemq_dev/include/vernemq_dev.hrl"; Make sure vernemq_dev is in your app file's 'applications' list

vmq_commons/src/vmq_types_mqtt.hrl:{1,14}: can't find include lib "vernemq_dev/include/vernemq_dev.hrl"; Make sure vernemq_dev is in your app file's 'applications' list

vmq_commons/src/vmq_types_mqtt5.hrl:{1,14}: can't find include lib "vernemq_dev/include/vernemq_dev.hrl"; Make sure vernemq_dev is in your app file's 'applications' list

@tiz314 Here's how to proceed.

In the vernemq dir, run make rel so that you have a default build (this will create default build in the _build dir)
Run the full common test suites with rebar3 ct
Alternatively, run a single test suite only with something like  rebar3 ct --suite apps/vmq_webhooks/test/vmq_webhooks_SUITE.erl (if you want, you can even just run single test cases)

Running any command with rebar3 ct will create a test build in the _build dir. In the test dir, you'll also have logs for your test runs. You'll also see HTTP files which you can open in a browser.

I pushed a first version of the test case but I'm having some serious difficulties in catching the patched parser exception that gets raised when sending the null char, making the test always pass.
First time with this kind of tech :(

@tiz314 Thanks for your effort, much appreciated. I'll look into the test case.


Various cleanup stuff

Bunch of nice looking cleanup stuff going on here...  Want to evaluate them for pulling them upstream...

I manually pulled a bunch of things over from this set of commits...
I'm really not sure that the timezone stuff in here is an improvement over the current time/timezone handling code, though.


FIRESDK-48: C++ Unit Tests for Core SDK modules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/firebolt-apis/300/rdkcentral/firebolt-apis

Commit: 3d56750503a98a0bac2b87c9bf96f30d10f839cb

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/firebolt-apis/300/rdkcentral/firebolt-apis

Commit: 3d56750503a98a0bac2b87c9bf96f30d10f839cb

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: OK

Commit: 3d56750503a98a0bac2b87c9bf96f30d10f839cb


Migrate to GitHub

Check dependencies
Update README.md
Update module name

This should be done now, but leaving open to allow for tweaking of the branch rulesets.


Custom Domain Question

I'm setting the CacheDomainName parameter when deploying the CloudFront-Enabled version of 4.1.1.  DNS CNAME records are set to the same custom domain.  This all works as expected.  The service can be invoked with the custom domain the data in the info.json responses look great, with the exception of the @id field which includes the lambda domain rather than my custom domain.  I had expected that to be replaced with the value entered for CacheDomainName.  Is it possible to configure the service to include the custom domain in @id field?

Thank you for reporting this. I think this has come up before, but I can't remember exactly how it played out. We're using a cloudfront-enabled instance and the @ids are looking like they should, so I think there must be some kind of configuration glitch. (It's not necessarily on your end – we use a customized deploy script and may have done something somewhere alone the line that the SAR click-to-deploy version doesn't.) Can you give me some details about your configuration and what you're seeing?

Thanks for the quick reply.  What would be helpful to know about my setup?
I'm using the CloudFront-Enabled deploy script from an unmodified version of serverless-iiif repo that I forked earlier today.  I'm just setting 2 parameters, the CacheDomainName and the SourceBucket.  For the CacheDomainName I'm setting the value to iiif-image.juncture-digital.org.
After the deploy completed I updated the CloudFront distribution config with the same custom domain name (using the Alternate domain name (CNAME)) field and pointed to the associated custom SSL certificate.  I'm using the Google Domains for DNS hosting, so I also needed to add the generated CloudFront ID to a CNAME record there.

@rsnyder I think I understand now. Open up the lambda function in the AWS console and check its environment variables (under the Configuration tab). There should be a forceHost variable set to the custom domain that you want your IDs to use. This is usually set during deploy, but if you updated the CloudFront distribution's CNAME after the fact, that wouldn't have happened.

Thanks for this update.  After looking at the code (and some trial-and-error) that is what I arrived at too.  Thanks for confirming this was the correct resolution.  And, thanks for developing/supporting this great package.  It really simplifies the deployment of an IIIF image service.
Ron
From: Michael B. Klein @.>
Date: Tuesday, December 13, 2022 at 2:49 PM
To: samvera/serverless-iiif @.>
Cc: Ronald Snyder @.>, Mention @.>
Subject: Re: [samvera/serverless-iiif] Custom Domain Question (Issue #89)
Caution: This message did not originate from within ITHAKA's email system. Please use caution when opening attachments and following links within this message.
@rsnyderhttps://github.com/rsnyder I think I understand now. Open up the lambda function in the AWS console and check its environment variables (under the Configuration tab). There should be a forceHost variable set to the custom domain that you want your IDs to use. This is usually set during deploy, but if you updated the CloudFront distribution's CNAME after the fact, that wouldn't have happened.
—
Reply to this email directly, view it on GitHubhttps://github.com/samvera/serverless-iiif/issues/89#issuecomment-1349602091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEFEYCOENPADA4ZQEHTZYLWNDHNHANCNFSM6AAAAAASE5R36U.
You are receiving this because you were mentioned.Message ID: @.***>


[TypeScript] Rewrite / Port

Would there be any objects to me creating a PR (When it's completed) of a full TypeScript version of this repo? I'm planning on updating somethings along with converting the code over. I also intend to adding live loading support along with support for the planned socket changes. To get the module ready for that.
If not I could create a fork for people wanting a pure TS module and will post changes here to that. But I'd rather the latter for easier updating / managing / support.
I'd like to use this module more for projects but it's lacking areas such as live loading support along with custom integrations. I'm creating this issue before doing anything to see what your guys thoughts are on this and/or if you already have a planned re-write of this module anytime soon. So I don't end up creating something in which you already have on the sprint. The same would apply for the node interactive repo in terms of re-writing it.

I'd be down for that. Our discord chat syncer system (to be open sourced soon) is actually written in Typescript. As our frontend moves towards Angular 2, we're considering (not yet final by any means) a gradual transition from JavaScript to Typescript. I agree that support for liveloading and other integrations is lacking in this client; that's something I've wanted to address but never had the time to. You're more than welcome to submit PRs for those!

OO go for it. Let me know if you need any information from us that'll help.

I assume you already have a tslint file. Would it be possible if I could have a copy of that to stick to the formatting you'd prefer.

I do not think we do have one of those yet.

Was just updated that this port is no longer needed. (With more than 40 hours already put into the re-write) So I can shove the code somewhere if you need it or which ever.

I'm not sure what you were updated with, but no plans have changed in regards to porting the client to Typescript.

No plans have changed wrt to Typescript. What was it that indicated otherwise? I can make sure it's fixed. @artdude543

It's been sorted out. More of a miscommunication and a slight over reaction. But the entire client is ported to TS now. Along with the tests are working fine. So I can send an initial PR with the port and then the changes / tweaks which are wanted could be added to the new base code?

Definitely, looking forward to seeing what you have 😄


Add NanoKubes from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of joshwoo69, to add NanoKubes to CKAN.
Please direct questions about this pull request to joshwoo69.

Not submitted by the original author, closing in favour of #671


Star and heart icons are missing on Chrome

The star and heart icons do not show up on Chrome (version 48.0.2564.109 m).
I noticed that in my case the other github icons on the page (that work) are implemented as svg, not using the octicon icon font ...

ha! Good catch, we'll fix that.
Thanks for the report (very) much appreciated.

Works great now. Thanks for the fix :)

Thanks for following up @danmana :100:


MP3 in Flash-Fallback: Plays only after full buffering

When I press play, the sound only begins to play when the file is completely done with buffering.
Other Flash players, however, do play after 2 seconds of buffering. Same file, different Flash audio player.

Can you note which versions, please?

There is a general feeling of instability with Firefox 9.0.1/Mac, Flash <IP_ADDRESS>, OS X Lion. mejs Version 2.6.4
http://logbuch-netzpolitik.de/
Sometimes the episodes start immediately, sometimes the Flash player generates a "spinning pizza of death", sometimes it plays alright after being loaded completely. However, navigating the file with the timeline always feels laggy.


[PE-D][Tester E] Weak validation on email domain

Reference Screenshot - Weak validation on email domain

Description
While the User Guide specifies that the email field should contain the format ‘local-part@domain', I am able to key in jd@gmail` which is possibly an incorrect domain. Perhaps it would be good to add stronger validation for valid domains used.

Labels: severity.Medium type.FeatureFlaw
original: triciiaaa/ped#11

jd@gmail is a valid email format.
See: https://stackoverflow.com/questions/20573488/why-does-html5-form-validation-allow-emails-without-a-dot

We should add a note to the UG to point out this special case.


Fixed an idempotency issue on the syncrepl variable

I had an idempotency issue when I was setting a value for syncrepl on a openldap_database resource :
Notice: /Stage[main]/Openldap::Server::Database[dc=example,dc=org]/Openldap_database[dc=example,dc=org]/syncrepl: syncrepl changed ['rid=001 provider=ldap://ldap.example.org:389"'] to 'rid=001 provider=ldap://ldap.example.org:389"

Here is my definition, I have a custom profile to manage it through Hiera :
profiles::openldap::server::databases:
  'dc=example,dc=org':
    syncrepl:
      - >
        rid=001
        provider=ldap://ldap.example.org:389

My profile basically does:
create_resources( openldap::server::database, $databases, $defaults )

@mvisonneau could you please rebase now that I've fixed the unit tests in master?

Thanks, looks like it works better now :)

Thanks @mvisonneau


[ERROR] Child process error with no actual information about error

ℹ ECS 10.2.5 (no parallel enabled)
I have weird error in CI that I found hard to work with. But before that, simple background:

we have ECS in CI that checks only files modified/added in MR
we determine files list based on Git diff between source and target branch
we use xargs in order to handle large diffs
we had issues with Argument list too long and after research I found it's because we were using export to pass that files list to the  subshell (or something like that, I'm not shell expert)

So I'm working on simplifying this process (remove exports, some of not needed xargs etc.), prepared MR with large diff and I got this:
$ printf "%d PHP files after filtering out ignored ones\n" $(echo "$GIT_DIFF_CHANGED_PHP_FILES" | awk 'NF' | wc -l)
3330 PHP files after filtering out ignored ones

$ (echo "${GIT_DIFF_CHANGED_PHP_FILES:-ecs.php}" | xargs -r vendor/bin/ecs check) || (echo 'To fix errors automatically run locally and commit: composer cs:develop-diff-fix' && exit 1)
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
 [ERROR] Child process error:
    0/1181 [░░░░░░░░░░░░░░░░░░░░░░░░░░░░]   0%
  120/1181 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░]  10%
  180/1181 [▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░░]  15%
  240/1181 [▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░]  20%
  360/1181 [▓▓▓▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░]  30%
  480/1181 [▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░░░░░░░░░]  [40]
  600/1181 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░░░░░░]  [50]
  720/1181 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░░░]  60%
  840/1181 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░]  71%
 1001/1181 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░]  84%
 1121/1181 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░]  94%
 1181/1181 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
 [OK] No errors found. Great job - your code is shiny in style!

I was able to debug CI runner and execute those commands up to this error. But it's not xargs fault since there's the same error when I run vendor/bin/ecs check $GIT_DIFF_CHANGED_PHP_FILES. Adding -vvv or --debug does not print any helpful data.
Running vendor/bin/ecs check without explicit paths starts analysis with 25k of files, so the tool is working in general. I know it's not easy case for you, especially I don't have reproducer repo as of now, but I would ask at least for help with finding actual error 🙂

It will be probably some memory or limit exhaustion of provided args. I think input paths might saved somewhere in bigger amount than accepted. Possibly even in the PHP itself. That's my guess for "too long input" errors :) in general this is a bad practice as it will hit limits of system settings.
Personally, I'd try to add 2 CI jobs:

the --debug run
and run without paralle

How long does parallel run on 25 k files? In most projects it finished in 10-30 seconds on full project.

As I said, --debug does not print any info 😉 Did not know parallel is enabled by default, I'll try to disable it and run again.
The reason why we run analysis only on subset of files is that currently we don't have CS for main branch (only MRs) so we don't have cache reference that could be propagated to jobs for MRs and allow analyse only changed files. Also some paths are excluded from analysis (after determining git diff the result is filtered with grep based on regex set in Gitlab CI's settings, so we're able to change analysis context without touching code). In the future it will probably change when we clean up legacy etc.
Anyway, Argument list too long was caused by exporting diff list to variable - also xargs stops working after this 😅 It was fixed by removing export and passing arguments through xargs.
FYI: we have Rector job with almost the same flow (without filtering paths) and it works properly on the same set of modified files. Output is so large that Gitlab has problems with displaying it though 😅

In saw no --debug in the provided output: xargs -r vendor/bin/ecs check), that's why I noted :)

In the future it will probably change when we clean up legacy etc.

I think the propper way instead of this complex exluding/including would be to apply the standard in single PR.
It's a matter of a day or two with safe sets like PSR-12 or split of COMMON even on the worst kind of code :muscle:

FYI: I was able to disable parallel on CI Runner (modified ecs.php) and analysis was totally OK:
$ (echo "${GIT_DIFF_CHANGED_PHP_FILES:-ecs.php}" | xargs -r vendor/bin/ecs check) || (echo 'To fix errors automatically run locally and commit: \e[32mcomposer cs:develop-diff-fix\e[0m' && exit 1)

 2149/2149 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 [OK] No errors found. Great job - your code is shiny in style!

 1181/1181 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 [OK] No errors found. Great job - your code is shiny in style!

So the root cause of the error is parallelisation 😉 But as I said, Rector works OK with such large file list (maybe there is parallel disabled by default?).

Is it possible to print some better information rather than [ERROR] Child process error? From our perspective it's an absolute minimum to do in terms of this problem. Because I was unable to get any helpful information about the root cause 😕

I would find the place it throws the exceptoin and dump the content before/around that.

I was rather thinking that ECS could handle those errors better and display helpful information 😉 Does ReactPHP emits error with such data?

That would require lot of debugging, as I don't have such knolwedge of reactphp. Yet when you find out, any improvment is welcomed :+1:


WebRTC / go2RTC - Almost no documentation to get it working

Hello everyone
I love this project. I was able to flash the provided bin file to Hi3518EV200 , the board works fine.
Also tried building on ubuntu and installing it on the board, works fine.
How to get WebRTC to work with Amazon kinesis ? I am looking for peer to peer video steaming solution.
Tried to install go2rtc by enabling the package in the config file but unable to make. it ends up with the following error

Unfortunately, it won't work on such ancient boards due to memory limitations. Consider upgrading your hardware or using a separate server in between

Hi @widgetii can you suggest a economical board ? In Gok or hisilicon ?
How we get amazon kinesis working ? Cannot build the image

You should write your own code based on their official library https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c
We've done support for one commercial clients, but this solution will never be open sourced

Aws producer and AWS webrtc are already included as packages right ? Are they not useful ?
How to build image with go2rtc enabled ? Getting error message , please see first comment for screenshot   .
Openipc/webrtc repo is of no use ?
Go2rtc will work ? If I upgrade the hardware

I've never worked with go2rtc so cannot say anything about it
AWS libraries are libraries, you cannot use them without writing a glue code


P3 (resolution 104 * 52) panel display position incorrect

I used a P3 panel with a size of 320 * 160 and a resolution of 104 * 52. I tested it using Example 1 SimpleTestShapes and found that the horizontal position of both graphics and text was incorrect. The left side was offset by about 8 pixels to the right. That's why, thank you! The parameter configuration is as follows:
#define PANEL_RES_X 104   //64
#define PANEL_RES_Y 52      //32
#define PANEL_CHAIN 1

The chip is ICND2046AP

Oh, it is a very weird (or interesting) panel :) I have worked with a similar one.
Each icdn2046 driver has a 16 outputs. Each panel row has a 7 x 16bit drivers, which gives you 112 pixels.
Since the panel is only 104 pixels wide, the 8 pins in the middle of the driver chain
are not connected to anything.
When loading data, you have to insert zeros into each row.

Oh, it is a very weird (or interesting) panel :) I have worked with a similar one. Each icdn2046 driver has a 16 outputs. Each panel row has a 7 x 16bit drivers, which gives you 112 outputs. Since the panel is only 104 pixels wide, the 8 pins in the middle of the driver chain are not connected to anything, which leads to the loss of some pixels in the picture. To prevent this, you have to insert zeros into each row when loading data.

Thank you very much. As I am not very familiar with the “ESP32-HUB75 Matrix Panel DMA” library, can you tell me where to fix this issue?

As I am not very familiar with the “ESP32-HUB75 Matrix Panel DMA” library,

Unfortunately, me too. So we should wait the author, @mrfaptastic, answer the question

Hope the author can provide assistance， @mrfaptastic，thanks

Doesn't seem like a library issue to me. Given the display works well and is outputting pixels, wouldn't you just make a wrapper in your sketch to accommodate for this weird "non visible" pixels?
Set the display resolution which includes the non existent pixels per whatever @board707 was saying, then adjust for this on your sketch, if that makes sense.

Doesn't seem like a library issue to me.

Sure, it's definitely not the library's fault.

Doesn't seem like a library issue to me. Given the display works well and is outputting pixels, wouldn't you just make a wrapper in your sketch to accommodate for this weird "non visible" pixels?
Set the display resolution which includes the non existent pixels per whatever @board707 was saying, then adjust for this on your sketch, if that makes sense.
Aka. set your panel resolution in the sketch to 114, not 104, and then figure out which pixels are missing and adjust for this in your pixel mapping routine.

Yes, there is no problem with the library itself. How to match the resolution of these anomalies? I don't know where to modify the adaptation in the library now. Can you give me some guidance? thanks

@mrcodetastic thanks for the great library !!
@mrcodetastic @board707 thanks for the detailed information!! I have a similar display and your information helps me to fix this.
@weny1212 I have created custom class to fix this as follows,
class Custom_MatrixPanel_I2S_DMA : public MatrixPanel_I2S_DMA {
 public:
  Custom_MatrixPanel_I2S_DMA(const HUB75_I2S_CFG &opts) : MatrixPanel_I2S_DMA(opts) {}

  inline void drawPixel(int16_t x, int16_t y, uint16_t color) {
    if (_rotation == 0 || _rotation == 2) {
      if (x >= _deadPixelOffset) {
        x += 8;
      }
    } else {
      if (y >= _deadPixelOffset) {
        y += 8;
      }
    }

    MatrixPanel_I2S_DMA::drawPixel(x, y, color);
  }

  inline void drawPixelRGB888(int16_t x, int16_t y, uint8_t r, uint8_t g, uint8_t b) {
    if (_rotation == 0 || _rotation == 2) {
      if (x >= _deadPixelOffset) {
        x += 8;
      }
    } else {
      if (y >= _deadPixelOffset) {
        y += 8;
      }
    }

    MatrixPanel_I2S_DMA::drawPixelRGB888(x, y, r, g, b);
  }

  // dead pixels offset can be in different place based on the display rotation
  // update it, if user updates the display rotation
  inline void setRotation(uint8_t r) {
    MatrixPanel_I2S_DMA::setRotation(r);
    rotation = (r & 3);
    switch (rotation) {
      case 0:
        _deadPixelOffset = 56;
        _rotation = 0;
        break;
      case 2:
        _deadPixelOffset = 48;
        _rotation = 2;
        break;
      case 1:
        _deadPixelOffset = 48;
        _rotation = 1;
        break;
      case 3:
        _deadPixelOffset = 56;
        _rotation = 3;
        break;
    }
  }

 private:
  int8_t _deadPixelOffset = 56;
  uint8_t _rotation = 0;
};

and on your actual code use Custom_MatrixPanel_I2S_DMA in the place of MatrixPanel_I2S_DMA,
// do not use 104, (112 = 104+8)
#define PANEL_RES_X 112  // Number of pixels wide of each INDIVIDUAL panel module.
#define PANEL_RES_Y 52   // Number of pixels tall of each INDIVIDUAL panel module.
#define PANEL_CHAIN 1    // Total number of panels chained one to another

Custom_MatrixPanel_I2S_DMA *dma_display = nullptr;

// in setup()
  HUB75_I2S_CFG mxconfig(
      PANEL_RES_X,  // module width
      PANEL_RES_Y,            // module height
      PANEL_CHAIN            // Chain length
  );

  dma_display = new Custom_MatrixPanel_I2S_DMA(mxconfig);

@jkandasa
My congratulations:)
I see only small drawback in the solution - it seems that is for a single panel only.

@board707 thanks for the review.
Currently I have only single panel. hence I have tested only with single panel
you are right, we have to rewrite the pixel calculation, if we have multiple panel chained.

Thank you for your kind words @jkandasa.
Well done with the solution you developed!


拉取

Summary by CodeRabbit

New Features

Updated branding and text content across the application to "qiqi-GPT 共享平台" and related descriptions.

Removed

Removed update checking and handling UI from the Settings component.

Bug Fixes

Fixed various text strings in Chinese localization for better user guidance.

Other Improvements

Updated the repository URL link to "https://www.baidu.com".

please resolve conflicts


fixes #39 Moved serialized key to row for CFM and export Queue

Currently testing this pull request on EC2.  Seeing huge performance improvements.    Seeing 1.2K to 1.4K pages per second being processed.  Before this PR I was seeing about 400 to 700 per second.   Running w/ 18 worker nodes.

+1


Return Nothing() If The Object passed To Get() Is Null

Would it be reasonable for get to have the above behaviour? I'd like to use this function as the 'gateway' for converting the values in a function pipeline to Maybes. Currently I'm a querying a datasource using highland for some JSON that may return null. At the moment I do this:
var _ = require('highland');
var R = require('ramda');
var S = require('sanctuary');

_(datasource)
     .map(R.compose(R.chain(S.get('b')), R.chain(S.get('a')), S.toMaybe))
     .reject(S.Nothing().equals)

It would be nice not to have to do that initial map with toMaybe and the subsequent call to R.chain so it would look like this:
_(datasource)
     .map(R.compose(R.chain(S.get('b')), S.get('a')))
     .reject(S.Nothing().equals)

Or even nicer once v0.5.0 of Sanctuary drops:
_(datasource)
     .map(S.gets(['a', 'b']))
     .reject(S.Nothing().equals)

I was excited to see this issue, @svozza. I had not realized that Sanctuary is being used outside a handful of Plaid projects. :)
I know of Highland but have never used it. Nonetheless, I believe I understand your problem.
In general—and especially in the case of Sanctuary—I believe functions should be strict about their inputs. Code written with Sanctuary should deal with null only where there is contact with foreign functions which return nullable values.
If we accept that it is the caller's responsibility to provide appropriate arguments to a function, we must first filter invalid values:
_(datasource)
     .reject(R.isNil)
     .map(S.gets(['a', 'b']))
     .filter(R.is(S.Just))

I assume you're operating on a stream, but for the sake of comparison here's how I might write this were datasource an array:
// transform :: [Object?] -> [Maybe *]
var transform = R.into([],
                       R.compose(R.reject(R.isNil),
                                 R.map(S.gets(['a', 'b'])),
                                 R.filter(R.is(S.Just))));

I'm interested in hearing from @buzzdecafe and @CrossEye. The question is whether it is the caller's responsibility to provide appropriate values or whether certain functions should be more forgiving. Certain Ramda functions treat null/undefined as an empty object. Currently Sanctuary takes a stricter stance.

It's funny, in my first draft of this I did exactly as you do in your example and filtered with R.isNil but wasn't sure if that was the right thing to do. The reasoning behind my changing it was that I thought it was best to get the data wrapped in a Maybe as soon as possible to prevent errors. I'm quite new to using these monadic types so I'm learning as I go; I like the idea of just being able to make the code null-safe with a single function call but I understand your rationale regarding sanitising the data before passing it to Sanctuary.
I'm currently refactoring a small section of our application where we have had persistent problems with null and undefined values. The gets function is going to very useful when it gets released, it'll definitely make it easier for me to sell this approach in general.

It would be nice not to have to do that initial map with toMaybe

I think either of the two approaches (viz. toMaybe or isNil) is ok, but i don't think map et al. should null-check and wrap in an ADT.
off topic a bit: i am a fan of highland, and i'd love to ensure seamless integration with highland streams in ramda. maybe we can discuss elsewhere.

Yes, definitely. Do you want me to open an issue on the Highland project or would you rather do it on the Ramda page or even in Gitter?

any of the above is fine, or ping me via email. fwiw i imagined it as more of a ramda issue, for us to make sure we can consume highland streams point-free.
ok, @davidchambers i'm sorry for hijacking this thread. back on topic now ...

Yeah. I'll open an issue on Ramda and call in some of the other Highland contributors who know the code much better than I do.

I was excited to see this issue, @svozza. I had not realized that Sanctuary is being used outside a handful of Plaid projects.

Watch out.  That's what happened to @buzzdecafe and me too with Ramda.  A year later, there are 1600+ commits from 62 different contributors, 31 open issues and 9 unresolved pull requests out of over a thousand total issues/PRs raised!   Certain Ramda functions treat null/undefined as an empty object. Currently Sanctuary takes a stricter stance.

I'm quite happy with this breakdown.  Ramda pushes things a fair way towards stronger typing of functions, but makes some pragmatic compromises familiar to Javascript developers.  Sanctuary pushes a little closer to languages like ML and Haskell.  Since they can play well together, a user can decide which level is appropriate for herself.

Thanks for your feedback, folks. Let's preserve the requirement that the second argument to S.get{,s} must not be null/undefined.

Sounds good. Thanks guys.


New broker api

This implements Thursday's discussion. In brief, you can get headers like so:
from dataportal.broker import DataBroker as db

db[13]  # scan id 13
db[-1]  # last scan
db[-5:]  # last five scans
db[-100::10]  # every 10th of the last 100 scans
db.find_headers(owner='blah', ...)  # any kwargs known to MDS

All of these returns a Header or Headers. From these get the associated events:
events = db.fetch_events(headers)

Which, as before, you can pass to a Muxer (née Muggler)
from dataportal.muxer import DataMuxer
dm = DataMuxer(events)

Looks good!


"Attempt to disconnect signal 'changed' while emitting" when zooming Script Editor.

Godot version:
3.2 beta4
Steps to reproduce:

Open script editor.
Hold Ctrl
Move mouse wheel to zoom

At: core/object.cpp:1522
ERROR: _disconnect: Attempt to disconnect signal 'changed' while in emission callback. Use CONNECT_DEFERRED (to be able to safely disconnect) or CONNECT_ONESHOT (for automatic disconnection) as connection flags.
1x in all its new glory 8-)

It's not a regression, like other issues, it's a previously unnoticed unsafe behaviour which is now being flagged as such. The editor code which tries to disconnect the signal in its own callback should be fixed as advised.


Reserve textClusterColumns vector for performance

Summary of the Pull Request
In tonight's episode of "Can we be even faster?", we will... you know what, just take a look at the code.

References

PR Checklist

[ ] Closes #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
It is actually a quite common technique seen inside the codebase to first reserve the spaces before pushing something into vectors. I don't know why it is not used here.
Before:

After:

Validation Steps Performed

@msftbot make sure @miniksa signs off on this

About the "There is a circular dependency in the target dependency graph involving target "ComputeGetResolvedWinMD". I've actually seen this in my local PC. Rebuilding fixes this. I don't really get it why it happens this often on the CI environment.

About the "There is a circular dependency in the target dependency graph involving target "ComputeGetResolvedWinMD". I've actually seen this in my local PC. Rebuilding fixes this. I don't really get it why it happens this often on the CI environment.

Yea, I forwarded you a mail we've got open with the MsBuild team on this. They're still trying to figure it out.

HOLD THE FLIPPING PHONE @skyline75489. How did you get the perf numbers per line to show up like that inside Visual Studio?!

:tada: Once again, thanks for the contribution!
This pull request was included in a set of conhost changes that was just
released with Windows Insider Build 20161.


allow supplying a slack channel's name as a query parameter

Hey man,
I've added an option to supply channel name as a query parameter, the usage is as follow
http://endpoint.address.domain:4000?channel=channel-name-without-hash
the motivation was to be able to expose a single endpoint and use it for the lots of channels we have under our slack.
Please review the codes, expressjs and node are not my day-to-day techs.
Thanks!

Thanks @grzegorznowak . It looks good but tis the Maybe monad really necessary?
Won't a simple lodash isEmpty do (https://lodash.com/docs/#isEmpty)?
I've merged your change but I'll probably remove the Maybe monad if you have no objection.

hey sure @rajivrnair , it probably isn't necessary at all. I had a different idea in the first place, but then, due to lack of a single, solid functional library for JS (or I couldn't find one is all), I abandoned it altogether.
Thanks for the merger.
We have one more thing on our list, which is to add linkage from slack message  to actual upsource urls. So will be pulling more requests soon I guess


host genome controllers

host genome controller to add/edit host genome information

Yes. need to wire this up before auto report generation.

Looks good.


Support query by lambda in all Get methods

Support query by lambda with a Expression<Func<T, bool>> whereQuery = null parameter in all get mehods.

@bjornharrtell lambda support would be a nice and performance optimization feature.

Agreed but it's likely non trivial. However existing implementation does exist in Npgsql EF Core support (https://www.npgsql.org/efcore/mapping/json.html#querying-json-columns-traditional-json-and-dom) but I'm not sure it can be used externally.

No, Npgsql EF Core can probably not be used.
The solution is probably to generate the select string based on the Expression<Func<T, bool>> query in the CreateGetCommand method.
https://stackoverflow.com/questions/2399009/how-does-c-sharp-lambda-work

Yes but I mean such logic already exists within the Npgsql EF Core logic, might extractable. And could even suggest to the Npgsql project it should be externalised in its own package.

@Revsgaard I've looked into this a bit and I think there is a bit of impedance between expression and jsonb indexing/operators. Some basic things is not too hard, fx. containment which fx. can be used to check exact equality for a single propery value. But I fail to see that it's possible to generally support something similar to "like" operator on a property value. So the expression translation will be quite limited and I wonder how useful it will be.

Linked issue https://github.com/ITfoxtec/FoxIDs/issues/986


MRG: Suppress spewed traits warnings

On Mayavi 4.5.0 (conda), I get a ton of warnings due to traits during normal usage. This gets rid of most of them.
The only ones left now are of the form:
ERROR: In /home/ilan/minonda/conda-bld/work/VTK-6.3.0/Rendering/Core/vtkCoordinate.cxx, line 351
vtkCoordinate (0x6e7a7a0): Request for coordinate transformation without required viewport

But I can't suppress those I don't think.

Coverage increased (+0.3%) to 77.519% when pulling d9fc04fb6803ae442300f4ccd8ed24a1ee3a1406 on Eric89GXL:no-warn into a5dd4acdac31aadfa3f54b1fde4167193b5ede18 on nipy:master.

This one is good to go, too


add jsonHeader to put request

in this file  github.com/gogs/gogs/vendor/github.com/go-macaron/binding/binding.go, the function bind ask for conten type in PUT request. So client should contain the jsonHeader。

Thank you!


API support

Would be interesting hosting this and creating API endpoints, I'm considering building an application around it. with flutter or Kotlin.
what do you feel about this?

That would be nice! Please let me know what you need from me whenever you are ready. :)

Alright, I should be ready mid this new week. By them I hope API endpoints should be ready so get me well started to build.

Alright, I should be ready mid this new week. By then, I hope API endpoints should be ready to get me well started to build.

Also, any plans for image urls? Probably getting only the top used image if the article has any.

None for now cos I am majorly interested in the text data. A PR to that effect is welcome though.


[WIP] Update repo to work with Terraform 0.13.x

Update CircleCI to use Terraform 0.13.x.
Upgraded the Google Providers to 3.43.0.
Update the version constraint on all examples to require at least 0.12.26.
Update pre-commit hooks to add gofmt.
Upgrade the tests to support Terratest v0.30.23

In general, TF 0.13.x seems backwards compatible from a testing perspective: I did not need to change anything in modules in this PR.

Builds are currently failing.

Tests passed in https://app.circleci.com/pipelines/github/gruntwork-io/terraform-google-sql/93/workflows/3c5acbbd-765b-4a27-b9c4-3057e920c6eb/jobs/191

@brikis98 thanks for the review. I'll release next week!


compile error occurs while USE_POCO_NETSSL is open

Hi, my environment is  gcc/g++ is 8.1.0 under ubuntu 14.04 LTS, and according to the official guide "How to build ClickHouse on Linux"(https://clickhouse.yandex/docs/en/development/build/), i tried to build clickhouse from source code, but failed, the detailed error information shown as following:
[2/328] Building CXX object dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o
FAILED: /usr/local/bin/g++   -DBOOST_SYSTEM_NO_DEPRECATED -DLIBRDKAFKA_STATICLIB -DLZ4_DISABLE_DEPRECATE_WARNINGS=1 -DPOCO_STATIC -DPOCO_UNBUNDLED_ZLIB -DWITH_GZFILEOP -DX86_64 -DZLIB_COMPAT -isystem ../contrib/libsparsehash -isystem ../contrib/libdivide -isystem /usr/include/x86_64-linux-gnu -I../dbms/src -Idbms/src -isystem ../contrib/double-conversion -isystem ../contrib/libpcg-random/include -isystem ../contrib/poco/Data/ODBC/include -I../contrib/cityhash102/include -I../libs/libcommon/include -Ilibs/libcommon/include -I../libs/libpocoext/include -I../contrib/poco/Data/include -I../contrib/poco/Foundation/include -I../contrib/zlib-ng -Icontrib/zlib-ng -I../contrib/poco/Util/include -I../contrib/poco/XML/include -I../contrib/poco/JSON/include -I../contrib/poco/Net/include -I../contrib/cctz/include -I../contrib/boost -I../contrib/libtcmalloc/include -I../contrib/libcpuid/include -I../contrib/lz4/lib -I../contrib/zstd/lib -I../contrib/poco/NetSSL_OpenSSL/include -I../contrib/poco/Crypto/include -isystem /usr/local/include -I../libs/libmysqlxx/include -Icontrib/re2_st -I../contrib/re2 -I../contrib/libbtrie/include -I../contrib/poco/MongoDB/include -I../contrib/librdkafka/src -Icontrib/librdkafka/generated/dummy -fdiagnostics-color=always -std=c++1z  -D_GLIBCXX_USE_CXX11_ABI=1 -pipe -msse4.1 -msse4.2 -mpopcnt  -fno-omit-frame-pointer  -Wall  -Wnon-virtual-dtor  -no-pie -Wextra -Werror -O2 -g -DNDEBUG -O3    -fno-builtin-malloc -fno-builtin-free -fno-builtin-realloc -fno-builtin-calloc -fno-builtin-cfree -fno-builtin-memalign -fno-builtin-posix_memalign -fno-builtin-valloc -fno-builtin-pvalloc -pthread -MMD -MT dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o -MF dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o.d -o dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o -c ../dbms/src/Client/Connection.cpp
In file included from /usr/local/include/openssl/err.h:21,
from ../contrib/poco/Crypto/include/Poco/Crypto/CryptoException.h:24,
from ../contrib/poco/Crypto/include/Poco/Crypto/OpenSSLInitializer.h:22,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:22,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/bio.h:690:1: error：expected constructor, destructor, or type conversion before ‘DEPRECATEDIN_1_1_0’
DEPRECATEDIN_1_1_0(int BIO_get_port(const char *str, unsigned short port_ptr))
^~~~~~~~~~~~~~~~~~
In file included from ../contrib/poco/Crypto/include/Poco/Crypto/CryptoException.h:24,
from ../contrib/poco/Crypto/include/Poco/Crypto/OpenSSLInitializer.h:22,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:22,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/err.h:260:1: error：expected constructor, destructor, or type conversion before ‘DEPRECATEDIN_1_0_0’
DEPRECATEDIN_1_0_0(void ERR_remove_state(unsigned long pid))
^~~~~~~~~~~~~~~~~~
In file included from /usr/local/include/openssl/asn1.h:23,
from /usr/local/include/openssl/objects.h:15,
from /usr/local/include/openssl/evp.h:28,
from /usr/local/include/openssl/x509.h:18,
from /usr/local/include/openssl/ssl.h:20,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/bn.h:281:1: error：expected constructor, destructor, or type conversion before ‘DEPRECATEDIN_0_9_8’
DEPRECATEDIN_0_9_8(int
^~~~~~~~~~~~~~~~~~
/usr/local/include/openssl/bn.h:352:1: error：expected constructor, destructor, or type conversion before ‘DEPRECATEDIN_0_9_8’
DEPRECATEDIN_0_9_8(int BN_get_params(int which)) / 0, mul, 1 high, 2 low, 3
^~~~~~~~~~~~~~~~~~
In file included from /usr/local/include/openssl/objects.h:15,
from /usr/local/include/openssl/evp.h:28,
from /usr/local/include/openssl/x509.h:18,
from /usr/local/include/openssl/ssl.h:20,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/asn1.h:555:7: error：expected constructor, destructor, or type conversion before ‘unsigned’
const unsigned char *ASN1_STRING_get0_data(const ASN1_STRING *x);
^~~~~~~~
In file included from /usr/local/include/openssl/x509.h:25,
from /usr/local/include/openssl/ssl.h:20,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/rsa.h:231:1: error：expected constructor, destructor, or type conversion before ‘int’
int RSA_generate_key_ex(RSA *rsa, int bits, BIGNUM *e, BN_GENCB *cb);
^~~
In file included from /usr/local/include/openssl/dsa.h:26,
from /usr/local/include/openssl/x509.h:26,
from /usr/local/include/openssl/ssl.h:20,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/dh.h:142:1: error：expected constructor, destructor, or type conversion before ‘int’
int DH_generate_parameters_ex(DH *dh, int prime_len, int generator,
^~~
In file included from /usr/local/include/openssl/x509.h:26,
from /usr/local/include/openssl/ssl.h:20,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/dsa.h:128:1: error：expected constructor, destructor, or type conversion before ‘int’
int DSA_generate_parameters_ex(DSA *dsa, int bits,
^~~
In file included from /usr/local/include/openssl/ssl.h:20,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/x509.h:728:1: error：expected constructor, destructor, or type conversion before ‘DEPRECATEDIN_1_1_0’
DEPRECATEDIN_1_1_0(ASN1_TIME X509_CRL_get_nextUpdate(X509_CRL crl))
^~~~~~~~~~~~~~~~~~
In file included from /usr/local/include/openssl/ssl.h:26,
from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/hmac.h:31:12: error：expected constructor, destructor, or type conversion before ‘int’
/__owur/ int HMAC_Init_ex(HMAC_CTX *ctx, const void *key, int len,
^~~
In file included from ../contrib/poco/Crypto/include/Poco/Crypto/X509Certificate.h:28,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/Context.h:23,
from ../contrib/poco/NetSSL_OpenSSL/include/Poco/Net/SecureStreamSocket.h:23,
from ../dbms/src/Client/Connection.cpp:27:
/usr/local/include/openssl/ssl.h:969:1: error：expected constructor, destructor, or type conversion before ‘typedef’
typedef enum {
^~~~~~~
/usr/local/include/openssl/ssl.h:1020:3: error：‘OSSL_HANDSHAKE_STATE’ does not name a type; did you mean ‘SSL_CB_HANDSHAKE_START’?
} OSSL_HANDSHAKE_STATE;
^~~~~~~~~~~~~~~~~~~~
SSL_CB_HANDSHAKE_START
/usr/local/include/openssl/ssl.h:1829:1: error：expected constructor, destructor, or type conversion before ‘DEPRECATEDIN_1_1_0’
DEPRECATEDIN_1_1_0(__owur const SSL_METHOD *SSLv3_server_method(void))
^~~~~~~~~~~~~~~~~~
/usr/local/include/openssl/ssl.h:1844:1: error：expected constructor, destructor, or type conversion before ‘DEPRECATEDIN_1_1_0’
DEPRECATEDIN_1_1_0(__owur const SSL_METHOD *TLSv1_server_method(void))
^~~~~~~~~~~~~~~~~~
/usr/local/include/openssl/ssl.h:1963:8: error：‘OSSL_HANDSHAKE_STATE’ does not name a type; did you mean ‘SSL_CB_HANDSHAKE_START’?
__owur OSSL_HANDSHAKE_STATE SSL_get_state(const SSL *ssl);
^~~~~~~~~~~~~~~~~~~~
SSL_CB_HANDSHAKE_START
ninja: build stopped: subcommand failed.
I find that:
OSSL_HANDSHAKE_STATE is defined like
typedef enum {
} OSSL_HANDSHAKE_STATE
but it can not be recognized as a type while compiling clickhouse?
thanks very much!
newly found:
include <openssl/opensslconf.h>
this header seems not be included successfully in <openssl/ssl.h> and others header file which use the macros defined in opensslconf.h

Please, provide command output:
dpkg -l | grep ssl
It's seems like something whent wrong with ssl version.

Sorry, but I cannot reproduce this problem with the same ubuntu, compiler and openssl version. Can you please provide cmake output?

This is not relevant because we have migrated from OS OpenSSL package to LibreSSL from submodules.


Feature/add influxdb webhook template

Summary
Adding a webhook template for InfluxDB Cloud 2.0
Notes for Reviewers
@adriansmares I read that the most common URL is http://localhost:8080/telegraf and only path (/telegraf) is usually being changed, but I kinda wasn't sure - does using something else than localhost:8080 make sense?
Checklist

[x] Scope: The referenced issue is addressed, there are no unrelated changes.
[x] Compatibility: The changes are backwards compatible, they don't break existing deployments.
[x] Testing: The changes are tested.
[x] Documentation: Relevant documentation is added or updated.

I'd recommend having the base URL as a variable as well. While most tutorials refer to hosting Telegraf yourself, locally, most people will actually want to host it somewhere else (maybe their own server, or on AWS), and as such will need to use their public host/port combination.
PS: There is a commit left from the Akenza Core tutorial left in this branch.

I'd recommend having the base URL as a variable as well. While most tutorials refer to hosting Telegraf yourself, locally, most people will actually want to host it somewhere else (maybe their own server, or on AWS), and as such will need to use their public host/port combination.
~PS: There is a commit left from the Akenza Core tutorial left in this branch.~

Okay, so this answers my question, it does make sense.
But here's the another issue - I tried defining the url field which user can fill in with custom URL, and base-url: "{url}" (just like the other day when I asked you how to specify the header entry with only variable), but I got an error there saying that the base-url value must be absolute. What does this mean?

Okay, so this answers my question, it does make sense.
But here's the another issue - I tried defining the url field which user can fill in with custom URL, and base-url: "{url}" (just like the other day when I asked you how to specify the header entry with only variable), but I got an error there saying that the base-url value must be absolute. What does this mean?

It means that I was wrong and we cannot actually turn this into a template. The standard that we use for the variables applies only to the path, not for the full URL. As such, we cannot really have the path to be an user input (at least not with ease).
Let's keep only https://github.com/TheThingsNetwork/lorawan-stack/pull/3016 and mention that the URL can be a remote one as well. You may close this PR, and sorry for the confusion.


Question about using multiple services / keychain instances

Hello!
When you configure multiple keychain instances with different service IDs, are they completely independent?
Lets say I wake up in the background often (significant location change) and want to keep a simple token in an .always keychain, and keep user password/email in a .whenUnlocked keychain.
let secureKeychain = Keychain(service: "com.example.com-secure").accessibility(.whenUnlocked)

let cacheKeychain = Keychain(service: "com.example.com-cache").accessibility(.always)

If my keychain manager class has two separate instances could they potentially interfere, or is this safe to do?
Thanks in advance for any help!

@RamblinWreck77 Yes. Two instances work independently, it's safe to do so.


Update interested-technical-writers.md

I added my own details because i am interested in the program

I just signed the CLA agreement, please merge pull request