Allow vulcand to expose in-progress connections (for instance draining.)

As vulcand grows, I presume Websockets, and long-living connections will become a key scenario/use-case.
We know that when we remove servers from a backend or remove a backend entirely, vulcand will allow any connections to that server to drain before releasing it.
However, for long-lived connections, it may be necessary for administrators to know any such pending connections before they take the server itself out of commission. This isn't a scenario we've hit yet, but two years down the line I imagine it'll come up.
I'm happy to contribute code to help with this - if we can land a design.
I'm imagining some sort of status API that allows vctl to query current on-going connections, or perhaps an etcd key that stores in-progress connections. Sadly this cannot be done in middlewares since the server is selected after all middlewares are done processing - so they have no visibility into what physical endpoint the connection will go to.
Using this feature, we should be able to script the provisioning and decommissioning of any backend instances safely.

@archisgore to speed up development, here's what I'm planning to do:

Vulcand and associated projects (oxy, predicate, route) will be moved to separate org
@archisgore and @pquerna will be given committers rights alongside mailgunners and me(I'm no longer at Mailgun)

@r0mant let me know when you plan to do this

I'll do it tonight/tomorrow if no pressing issues come up.

Thanks for doing this romant. I promise to be a good actor here. I'm beginning to merge some PRs and also will ask some PR submitters to resubmit - those who broke the UTs due to that old race condition.
Watch out for path-updates coming soon (all packages refer to github.com/mailgun/... and I'll make those updates in order of dependencies first.


Add Q.compose utility function

Q.compose(functions) is modeled after compose()  available many JS utility libraries

I recommend publishing this tool as a separate package. I am striving to make Q smaller.


link to vignette

in the "getting started" section we refer to the vignette using this link (https://bioconductor.org/packages/release/data/experiment/vignettes/curatedMetagenomicData/inst/doc/curatedMetagenomicData.html).
or should we refer to this file? https://github.com/waldronlab/curatedMetagenomicData/blob/master/vignettes/curatedMetagenomicData.Rmd

after updating the website, the link points to the vignette of the devel version now.


@shikijs/twoslash does not work on rehype pretty code transformers option

@shikijs/twoslash does not work on rehype-pretty-code transformers options, which the docs say here that it should work.
However, so far, it is not working. This is a sample of the code:
import { rendererRich, transformerTwoslash } from "@shikijs/twoslash";
import rehypePrettyCode from "rehype-pretty-code";
// ...

const rehypePrettyCodeOptions = {
  theme: moonlightTheme,
  keepBackground: false,
  transformers: [transformerTwoslash({ renderer: rendererRich() })],
  onVisitLine(node) {
    if (node.children.length === 0) {
      node.children = [{ type: "text", value: " " }];
    }
  },
};

// ...

const source = makeSource({
  contentDirPath: "./src/content",
  documentTypes: [Blog],
  mdx: {
    remarkPlugins: [remarkGfm],
    rehypePlugins: [
      rehypeAccessibleEmojis,
      rehypeSlug,
      [rehypePrettyCode, rehypePrettyCodeOptions],
      [rehypeAutolinkHeadings, rehypeAutolinkHeadingsOptions],
      rehypePresetMinify,
    ],
  },
});

Not at all a minimal reproduction (I'll try to write one sometime), but at least a full project: Effect-TS/website#770

Ok, it turns out that it's about order: https://github.com/Effect-TS/website/compare/23debaa8e1ef70ee534ef21097a99d5361932882..20608435dcc90deb7616af2b89bacdf28cd4fb37 (ignore the CSS changes)

Even your suggestion @lishaduck doesn't work. Turns out that after applying all your new suggestions, what I get is a <pre> code block that just loses all its styling from the theme: "one-dark-pro" in the rehypePrettyCodeOptions?

Thanks for your suggestion @lishaduck but it doesn't work. Turns out that after applying all your new suggestions, what I get is a \<pre\> code block that just loses all its styling from the theme: "one-dark-pro" in the rehypePrettyCodeOptions?

Yeah. I ran into that as well. I don't know what plugins you're using, but some orders work, and some lose styling, and some work but without twoslash. I don't really know why, just that's what I'm experiencing.


wrong file path in quick start guide

in
nf-cor-re/public_html/index.php
line 146: mv nextflow ~/bin
needs to be changed to (at least) mv nextflow ~/bin/

Code in question:
https://github.com/nf-core/nf-co.re/blob/master/public_html/index.php#L146
Please make a PR! Something this small you can do in the web editor, probably quicker than making the issue 😅

(good spot by the way 👍 )

Fixed in https://github.com/nf-core/nf-co.re/pull/213


Make use of variable electrical price

I have an electricity contract that updates the exchange prices every hour (https://tibber.com/de). I can see the prices for the next 24 hours via api.
is it possible to use/consider this type of electrical cost?

Of course, 100% supported, you can pass those dynamic values to the add-on as a list of values. Read the documentation for examples on how to do this. There are people using Tibber and EMHASS, you can ask for help to set it up on the HomeAssistant forum.


what does "Prevented a handshake race" mean/imply?

I have nebula configured on 2 KVM VMs on the same server node1 and node2
My lighthouse is on a cloud server.
I start the lighthouse ok
I start Node1 ok and lighthouse shows it connecting ok
But I start Node2 and on the Lighthouse I see the following:
INFO[0252] Prevented a handshake race                    handshake="map[stage:1 style:ix_psk0]" initiatorIndex=2241835118 remoteIndex=0 responderIndex=2441061689 udpAddr="<IP_ADDRESS>:4242" vpnIp=<IP_ADDRESS>
What does Prevented a handshake race mean/imply ??
Is Nebula confused because VM Node1 and VM Node2 connection requests are coming from the same physical machine that hosts those VMs ?

It means that both nodes are attempting to handshake with each other, as initiators, at the same time. Usually this occurs on a high latency link, but it can also happen in low latency networks if both nodes try to initiate a handshake at the same moment, or if you restart one side and trigger a handshake. Nebula sorts this out though, the log lines are for informational purposes.


themes don't work

echo "theme: jekyll-theme-slate" > _config.yml
docker run -it -v `pwd`:/usr/src/app -p 4000 starefossen/github-pages

produces the output:
Configuration file: /usr/src/app/_config.yml
jekyll 3.6.2 | Error:  The jekyll-theme-slate theme could not be found.

sorry, the problem was caused by an empty Gemfile and Gemfile.lock in my src dir.

Glad you worked it out, @munkyboy 😄


this course is not profitable enough

I'd suggest:

maximizing margins
synergizing our revenue streams
culling the chaft
wheating the bottom line
redefining success


Strange behaviour in "make formula"

Describe the bug
By inserting a higher amount of product the result becomes negative
To Reproduce
see screenshot
Expected behavior
Maybe the field should be limited at the remaining maximum amount?
Screenshots

Another question, when you insert a note, when/where it's used?

The negative value is actually happening on purpose to mark the material is overdosed, note that the message returns that the material is overdosed and there's a warning next to it as well.
Notes will go to the general formula notes - see settings section.

Ok, now I think I understood the discourse of the notes (not the reference to the settings).
I would like to note then that a new line should be added when the notes are joined

also when "overdosing" values are saved to notes'
but without any reference to the ingredients

Maybe save these kind of notes in separate rows?

When overdosing you cannot make formula as complete. I cannot roll back overdosing so how can set as made the formula?


Don't use low res PNG for displaying text.

I haven't took the time and actually run this on my computer because unfortunately I don't know how to do it. I'd be glad to add this section to readme if someone can give me direction.
Anyway, logo-name.png looks pretty bad on retina displays so I'm doing this PR to use html instead.

Can you show me a picture of this? I couldn't figure out how to do this without flexbox, and I wasn't sure how far back flexbox is supported on different browsers. Do you know?

Also, getting it set up can be super hard, lets just focus on the compatibility question. I tried not to use flexbox because I wanted to be conservative.

Flexbox support is pretty good. Around 95% according to caniuse. Sorry I haven't write about it in the first place.
Comparison

Alright, thank you! I have been convinced to use flexbox.


feat: Infer initial schema

Closes #[12].
Summary of Changes
added Tablesaw for inferring the initial dataset schema
implemented schema inference for the initial dataset schema with a Unit test

@lars-reimann Please merge this after the PR 79


add gmm

add gmm

[TODO]

Add readme.md in the example and put the hyperparameters there, you can refer to this.
Add rst docstrings in the GMMConv, you can also refer to GCNConv.
Remove Chinese comments, useless comments, and files.

gmm modification.


Remove Bogus DEBUG log.

"Queue with name that starts with 'amq.' cannot be declared." logged unconditionally.
Cherry-pick to 1.6.x, 1.7.x

... and cherry-picked.


The image field is required to be able to create without adding an image in the admin user create from.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
The image field is required to be able to create without adding an image in the admin user create from.
Preconditions

Admin should be logged in.

Steps to reproduce

Go to 'Admin panel'
Click on 'Create user' inside the Setting/user
Fill in all the required fields except the image.
Click on the save button
See the error

Expected Result
If the field is required it should be shown the required message. The image fields should not be required.
Actual Result
It shows the field is required but not saved without adding images. Here
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Tron transaction signing

Test case is in TronTransactionTests

Please exclude Tron.pb.swift from linting in .swiftlint.yml


Fixes rendering issues

Fixed rendering of section page
Fixed rendering of footers

@dannyparsons @Kishokanth-netizen kindly check and let me know


fix safetensors

What does this PR do?

This PR aims at fixing an issue with HuggingFace pre-trained models in which when trying to load a pre-trained model such as distill-whisper-large-v3, our interface will raise an error saying that: FileNotFoundError: /scratch/adelmou/pretrained_models/distil-large-v3/ does not contain a .bin or .ckpt checkpoint !. According to the HF documentation, now, they tends to rely on safetensors for saving weights, meaning that we also now need to check whether or not the pre-trained file has  safetensors.

Before submitting

[ ] Did you read the contributor guideline?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Does your code adhere to project-specific code style and conventions?

PR review

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified
[ ] Confirm that the changes adhere to compatibility requirements (e.g., Python version, platform)
[ ] Review the self-review checklist to ensure the code is ready for review

@Adel-Moumen I also did the same fix in this PR: #2751

oof sorry, I havent seen your fix as the PR was about audio tokenizers. Are you ok with merging this one instead? (its easier to break down big PRs in smallers one. I am also ok with you creating a new small PR regarding this fix).

@TParcollet  This PR could be merged.


WIP process localizer data

cc @agramfort

Currently only sensor-level analysis of a single participant

@agramfort I'm afraid the test fails due to high memory consumption, any idea?

ICA is usually the culprit :) do you decimate?

@jasmainak Yep, I decimate to 250 Hz, but maybe the recording is just too long. Profiling to see what I can do.

No wait… this is resampling, not decimation. Mh!

in the ICA constructor you can give a decim parameter and you can also give a tmax I believe

Ok it's working now. Turns out we didn't use the filtered & resampled data in ICA!! So the CI limitation helped us find a bug.
We now operate on the filtered & resampled data. One thing we'll need to discuss is whether it's actually okay to apply a second high-pass filter run on this data, which will currently happen if ica_l_freq > l_freq. Will this cause trouble?
Otherwise, we could start with the Maxwell-filtered data (without a frequency filter applied) and do everything anew, cleanly. But this would potentially waste processing time.


Track releases in GitHub issues!

The documented procedure is to use JIRA issues when publishing tools, platforms, or plugins releases. Now that GitHub issues are enabled I would much rather track release items there and we cannot raise new JIRA issues, we are now forced to track release items in GitHub issues. I think we should check impact on areas such as:

Apache svn commit messages
cordova-doc pull requests
where should we raise the GitHub issues?

Bump with some additional comments:
I think that for each package release we should raise issues in https://github.com/apache/cordova/issues. Track all package release issues in one place, no more change for collision in GH issue numbers, etc.
An example where this went wrong: https://github.com/apache/cordova-docs/pull/886#issuecomment-423270720 / https://github.com/apache/cordova-docs/pull/886#issuecomment-423271940

Are issues for releases really necessary?
Wouldn't it maybe make more sense to create milestones in the repositories, add all the PRs (and issues) included.

Any pointers how to use milestones?
Also JIRA numbers made it relatively easy to reference issue number in git and svn commits. Any ideas how we can do this with milestones?

Any pointers how to use milestones?

Best start here: https://help.github.com/articles/creating-and-editing-milestones-for-issues-and-pull-requests/

Also JIRA numbers made it relatively easy to reference issue number in git and svn commits. Any ideas how we can do this with milestones?

Why do we need this?


SW-1391 Make withdrawal purpose optional

In the next version of the web app, we're going to stop requiring the user to pick
a purpose for each withdrawal. Update the data model and API to reflect that.

Current dependencies on/for this PR:

main

PR #545   👈

This comment was auto-generated by Graphite.


Add key mapping and Update CameraController using improved Input system.

Describe the issue (if no issue has been made)
This PR doesn't fix any issue.
This PR will improve the existing Input and OrthographicCameraController described below.
PR impact (Make sure to add closing keywords)
List of related issues/PRs this will solve:

Impact
Issue/PR

Other PRs this solves
#161, #124

Proposed fix (Make sure you've read on how to contribute to Hazel)
This PR adds/improves the following:

KeyCodes/MouseButtonCodes:

Add enum class Keys and MouseButtons (#124).
In the future version, macros will be replaced completely by those enum classes.

Input:

Add KeyAlternative struct which represent a single key press or mouse button press.
Input checks now checks over the said KeyAlternative instead of raw keycodes/mousebuttoncodes int values.
Add a map for key mapping: each std::string key is assigned with one KeyAlternative, and are remappable at any time/any place.

OrthographicCameraController:

Update Controller to take advantage of the new Keymapping system.
[minor] Zoom Level and Enable Rotation can now be manually set at any time/any place. There is a min/max zoom level limitation.
[minor] Camera translation speed now scales with the zoom level (#161).

This PR is backward-compatible with current version of master branch Hazel (need minor variable rename though).
Additional context
None.

Sorry for the long delay (I'm moving and my new place didn't have internet connection set up).
I have made some changes as suggested by @LovelySanta in my local code.
Since @IshoAntar have updated his PR #124, this PR become a duplicate of his one. Should I close this PR and create a new one for key mapping, or should I just strip out the new Keycode/Mousecode part?

Sorry for the long delay (I'm moving and my new place didn't have internet connection set up).
I have made some changes as suggested by @LovelySanta in my local code.
Since @IshoAntar have updated his PR #124, this PR become a duplicate of his one. Should I close this PR and create a new one for key mapping, or should I just strip out the new Keycode/Mousecode part?

I would suggest leave this one open untill you have your new PR up. Also, maybe wait from making your PR until #124 is merged (will happen with my next merge testing, when I have time)

I would suggest leave this one open untill you have your new PR up. Also, maybe wait from making your PR until #124 is merged (will happen with my next merge testing, when I have time)
Thanks.
Since the new input works with Keys and Mouse enum class instead of raw integer values, I will have to wait until #124 is merged and those enum classes are part of the main code base, or else it will become a broken PR :))


Virtual clusters not given CA.

Describe the bug
Observed this when using virtual clusters created using helm charts rather than builtin virtual cluster support, but expect later will have same issue.
When using your CA so can have self signed certificates, the CA isn't injected into the virtual cluster control plane so might be issue if is required.
Believe this is the case as see the error:
12:38:17AM: ongoing: reconcile deployment/lookup-service-poc (apps/v1) namespace: educates-platform
12:38:17AM:  ^ Waiting for 1 unavailable replicas
12:38:17AM:  L ok: waiting on replicaset/lookup-service-poc-5c7b4d9759 (apps/v1) namespace: educates-platform
12:38:17AM:  L ongoing: waiting on pod/lookup-service-poc-5c7b4d9759-plvqj (v1) namespace: educates-platform
12:38:17AM:     ^ Pending: ErrImagePull (message: failed to pull and unpack image "registry-educates-cli-w03-s001.educates-local-dev.test/lab-platform-operator/lookup-service-poc:latest": failed to resolve reference "registry-educates-cli-w03-s001.educates-local-dev.test/lab-platform-operator/lookup-service-poc:latest": failed to do request: Head "https://registry-educates-cli-w03-s001.educates-local-dev.test/v2/lab-platform-operator/lookup-service-poc/manifests/latest": tls: failed to verify certificate: x509: certificate signed by unknown authority)

when trying to deploy to the virtual cluster an image built in the workshop session and pushed to the per session image registry.
Additional information
No response

When creating an issue like this would be good to have a reproducer of the steps and workshop definition :-P
I don't think this should be any different than running any other image built and pushed to the per-session registry, as this log, ErrImgPull should be that containerd on the host can not pull down the image from that registry. The reproducer could probably help to investigate whether this is a regression on 3.0.0 (if you were already using that) or if it has always existed. Remember than in 3.0 we have changed how containerd is configured on the host, so maybe something broke there.


Python: If no env_file is configured then default to relative .env file

Motivation and Context
We'd like to improve the handling of the .env file related to Pydantic Settings. If Environment variables aren't found, it's possible to fall back on reading the values from a .env file.

Description
Previous to this PR, that is done by specifying the .env file path. Now, if the env_file is not specified, we will default to reading from the relative ".env" file in the cwd.

Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

Python 3.10 Test Coverage Report • FileStmtsMissCoverMissingsemantic_kernel   kernel_pydantic.py21195%53TOTAL648073789%
Python 3.10 Unit Test Overview

Tests
Skipped
Failures
Errors
Time

1523
1 :zzz:
0 :x:
0 :fire:
23.863s :stopwatch:


Use a HTTP proxy with restler

I'd be great if restler supported an option for HTTP proxies (helpful for making services that interact with the onion network over privoxy)

we got an such issue and stuck on it while deploy server in the in production enviroment , hopefully it will come true as soon as possible.


remove the unpooled UdpTransport

Summarise the changes this Pull Request makes.
remove the unpooled UdpTransport and keep the PooledUdpTransport
as pooled is preferred and we don't need 2 ways to do the same thing.
Please include a reference to a GitHub issue if appropriate.
Follow on from #113

Suggest in a PR after this one to rename PooledUdpTransport to UdpTransport - the pooling is now just an implementation detail, so I don't think it needs to be in the type name.


issuer naming should respect issuer_name override

align the spire-server configmap and issuer CR naming

Added a suffix to the CA issuer object name, so we don't accidentally conflict like I just did on the object names


Wrong horizontal scroll side in Recent article

Hey there,
A friend of mine just noticed a weird behavior : the horizontal scroll on the Recent Article in the three dots menu is revert-side. If you scroll left, it goes left, which usually goes the opposite.
Indeed, the normal behavior on the phone is that if you scroll bottom to top on a phone, it goes down, well there if you scroll left to right it goes right instead of left (instead of making everything go right).
Any way to correct this ? Thanks you 👍

What theme version do you have installed?


Can't Install vue CLI ERR_INVALID_OPT_VALUE The value "false" is invalid for option "readableHighWaterMark"

IMPORTANT: Please use the following link to create a new issue:
If your issue was not created using the app above, it will be closed immediately.
中文用户请注意：
请使用上面的链接来创建新的 issue。如果不是用上述工具创建的 issue 会被自动关闭。

I did this already and it didn't appear on the repo's issue page...


Global exception logger doesn't log actual trace

This should work
def log_exception(orchestrator_connection: OrchestratorConnection) -> callable:
    """Creates a function to be used as an exception hook that logs any uncaught exception in OpenOrchestrator.

    Args:
        orchestrator_connection: The connection to OpenOrchestrator.

    Returns:
        callable: A function that can be assigned to sys.excepthook.
    """
    def inner(exc_type, exc_value, exc_traceback):
        trace = "\n".join(traceback.extract_tb(exc_traceback).format())

        lines = [
            f"Error Type: {exc_type.__name__}",
            f"Message: {exc_value}",
            f"Trace:\n{trace}"
        ]
        orchestrator_connection.log_error("\n".join(lines))

    return inner


Transient task validation errors when opening a repo using custom tasks in Codespaces

Environment

VS Code Version: 1.74.0
OS Version: Debian Bullseye (Codespaces)

Steps to Reproduce:

Go to https://github.com/Azure-Samples/todo-nodejs-mongo
Click the "Use template" green drop-down button and choose "Open in a codespace"
Observe as the repo is being opened.

Expected
There should be no errors.
Actual
An error popup saying "There are task errors. See output for details" is flashing a few times. The output window shows something about missing dotenv for a second or so, and then it disappears (I haven't been successful in screenshotting it).
Additional information

dotenv is a custom task type that Azure Developer CLI extension provides. This extension is used by the repo used for the repo, and the tasks.json file in the repo uses dotenv custom task in several places.
Azure Developer CLI extension is installed as part of Codespace creation (is referenced by devcontainer.json).
As far as I can tell from the logs indicate that Azure Developer CLI is installed, but not activated during the repo open sequence. In particular, the extension has onTaskType:dotenv as one of the activation events listed in package.json, but it does not seem that any activation occurs, nor is dotenv task run.
tasks.json is not open during Codespaces initialization

My suspicion is that tasks.json is being validated early, before Azure Developer CLI extension is installed, and at that time the validation system does not know yet about dotenv task type. If that is the case, perhaps it would be better to delay tasks.json validation until all extensions required by the repo are installed.

@meganrogge I think this might be related to the area you own

This might be a dupe of https://github.com/microsoft/vscode/issues/165292, although I do not see the symptoms described in that bug with VS Code 1.74: the Azure Developer CLI extension is installed normally and there is no prompt to install it again. I just see these transient tasks.json validation errors now. I will let you decide which bug to keep and which to close.

Yep @karolz-ms same as #165292


[NIFI-2913] update z-index of codemirror resize handle to match the z…

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
…-index of the codemirror vert. and horz. scrollbars

Reviewing...

@scottyaslan I built this and once I have entered text that extends past the boundaries to the right and the bottom, I can still use the resize handle, but I can only resize vertically. Is this intentional?

Ran contrib-check, all tests pass, verified this fixes the issue. +1, merging.


gridsearchcv() freezes?

Sometimes(When the param_grid is large) GridSearchCV() freezes.
Each process use CPU 0%. I have to kill it by C-c C-c.
Environment info
Operating System: Linux
CPU:
C++/Python/R version: Python

try with smaller data or fewer parameters

there is a gridsearch example in examples/python-guide/sklearn_example.py, can it work on your machine?

It works for small data  or small parameters.
Maybe the data is too large.
Finally: I set n_jobs=1.


tutorial for the or snipet for case where we only have 1 modality

The way to enocde the data and dataloader for single-modality datasets is not clear

Hi all, great tool! Just checking if any update on this?
I have tried a few variations to input only scRNAseq data by itself with no other modes, but the model seems to fail to fully train.
I am doing the following:
`
adata = mtm.data.organize_multimodal_anndatas(
adatas=[[adata_ref]],  # a list of anndata objects per modality, RNA-seq always goes first
layers=[["counts"]],  # if need to use data from .layers, if None use .X
)
adata_indices_end = adata.shape[1]
mtm.model.MultiVAE.setup_anndata(
adata,
categorical_covariate_keys=["batch"],
rna_indices_end= adata_indices_end,
)
vae = mtm.model.MultiVAE(
adata,
losses=["nb"],
)
vae.train()
`
I get the following error:
Expected parameter loc (Tensor of shape (256, 16)) of distribution Normal(loc: torch.Size([256, 16]), scale: torch.Size([256, 16])) to satisfy the constraint Real(), but found invalid values:
tensor([[nan, nan, nan,  ..., nan, nan, nan],
[nan, nan, nan,  ..., nan, nan, nan],
[nan, nan, nan,  ..., nan, nan, nan],
...,
[nan, nan, nan,  ..., nan, nan, nan],
[nan, nan, nan,  ..., nan, nan, nan],
[nan, nan, nan,  ..., nan, nan, nan]], device='cuda:0',
grad_fn=)
Which I presume has something to do with my single modality approach, as I can train the same dataset using scVI, MultiVI and scPoli fine (with the same hvg selections and preprocessing).
Any help would be much appreciated.


output error when the ip is repeated as a entire or as a substring

When then an ip is repeated like n times in a line, then in the output of nali for this line, the geological location will be repeated n times after everywhere the ip appears.

For example:
> echo '<IP_ADDRESS> aaa' | nali
<IP_ADDRESS> [本机地址] aaa

> echo '<IP_ADDRESS> aaa <IP_ADDRESS>' | nali
<IP_ADDRESS> [本机地址] [本机地址] aaa <IP_ADDRESS> [本机地址] [本机地址]

> echo '<IP_ADDRESS> aaa <IP_ADDRESS> bbbb <IP_ADDRESS>' | nali
<IP_ADDRESS> [本机地址] [本机地址] [本机地址] aaa <IP_ADDRESS> [本机地址] [本机地址] [本机地址] bbbb <IP_ADDRESS> [本机地址] [本机地址] [本机地址]

> echo '<IP_ADDRESS> aa <IP_ADDRESS> aaaaa <IP_ADDRESS>' | nali
<IP_ADDRESS> [本机地址] [本机地址] aa <IP_ADDRESS> [局域网 IP] aaaaa <IP_ADDRESS> [本机地址] [本机地址]

When a line contains more than ips, where one ip is a subtring of another ip, the ips will be parsed incorrectly. For example:

# ipxxxx   ip
> echo '<IP_ADDRESS> aa <IP_ADDRESS> bbb' | nali
<IP_ADDRESS> [本机地址]2 [局域网 IP] aa <IP_ADDRESS> [本机地址] bbb

# ip  ipxxxx
> echo '<IP_ADDRESS> aa <IP_ADDRESS> bbb' | nali
<IP_ADDRESS> [本机地址] aa <IP_ADDRESS> [本机地址]2 bbb

# ip  xxxip
> echo '<IP_ADDRESS> aa <IP_ADDRESS> bbb' | nali
<IP_ADDRESS> [日本 东京都Amazon数据中心] aa <IP_ADDRESS> [本机地址] [日本 东京都Amazon数据中心] bbb

# xxxip   ip
> echo '<IP_ADDRESS> aa <IP_ADDRESS> bbb' | nali
<IP_ADDRESS> [日本 东京都Amazon数据中心] [本机地址] aa <IP_ADDRESS> [日本 东京都Amazon数据中心] bbb

# xxxipxxx ip
> echo '<IP_ADDRESS> aa <IP_ADDRESS> bbb' | nali
<IP_ADDRESS> [日本 东京都Amazon数据中心]2 [局域网 IP] aa <IP_ADDRESS> [日本 东京都Amazon数据中心] bbb

# ip    xxxxipxxxx
> echo '<IP_ADDRESS> aa <IP_ADDRESS> bbb' | nali
<IP_ADDRESS> [日本 东京都Amazon数据中心] aa <IP_ADDRESS> [日本 东京都Amazon数据中心]2 bbb

For different the aforementioned cases of the relations between the two ips, the fault is different.

There is only one reason for these problems, I use replace to add geographic information.
These days I will use other methods to format the output to avoid these problems.

Thanks for your PR, merged but not what I expected.
I hope that each character will be scanned only once, just like a lexical analyzer, the geographic information will be inserted after the IP is detected, instead of using strings.replace or regular replacement.

AND still have problems
echo <IP_ADDRESS> ::<IP_ADDRESS> |nali.exe
<IP_ADDRESS> [本机地址] ::127 [IANA保留地址 ].0.0.1 [本机地址]

It can't distinguish between IPv4 address and IPv4-compatible IPv6 address

These two problems are solved in submit2. Please check it and merge.

This is a more efficient way. Looking forward to your refactoring.

I hope that each character will be scanned only once, just like a lexical analyzer, the geographic information will be inserted after the IP is detected, instead of using strings.replace or regexp replacement.


Score Scale

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
A new feature called score_scale. This feature would allow a user to set a scale value for any custom format score, in the same way that a score can currently be manually set. Example:
custom_formats:
      - trash_ids:
      # Audio
          - 496f355514737f7d83bf7aa4d24f8169  # TrueHD Atmos
          - 2f22d89048b01681dde8afe203bf2e95  # DTS X
          - 417804f7f2c4308c1f4c5d380d4c4475  # ATMOS (undefined)
          - 1af239278386be2919e1bcee0bde047e  # DD+ ATMOS
          - 3cafb66171b47f226146a0770576870f  # TrueHD
          - dcf3ec6938fa32445f590a4da84256cd  # DTS-HD MA
          - a570d4a0e56a2874b64e5bfa55202a1b  # FLAC
          - e7c2fcae07cbada050a0af3357491d7b  # PCM
          - 8e109e50e0a0b83a5098b056e13bf6db  # DTS-HD HRA
          - 185f1dd7264c4562b9022d963ac37424  # DD+
          - f9f847ac70a0af62ea4a08280b859636  # DTS-ES
          - 1c1a4c5e823891c75bc50380a6866f73  # DTS
          - 240770601cc226190c367ef59aba7463  # AAC
          - c2998bd0d90ed5621d8df281e839436e  # DD
        quality_profiles:
          - name: My Lovely Profile
            score_scale: 0.1

The example above is a real-world use case, where some users score audio custom formats at 10% of their guide scores so that they are not so heavily weighted.

How is this better than running the calculation manually and inputting the result as a manual score?

If the guide score changes, you wouldn't need to update your config file. If you're doing it all manually then you would.

That's fair, I guess what I'm really asking is this. Normally when we deviate from guide scores, the recommendation is an absolute number. In other words, when we want Dual Audio for anime, we offer options like 10 and 100 or sometimes 101. Those are more difficult to represent in terms of percentages (or ratios) of the original base value.
To be clear, I don't have an issue with the idea itself. I'm just trying to better understand the motivation here. I'm interested in understanding how this is effective. What is a real world example that would benefit from this feature and how does it benefit?

That's fair, I guess what I'm really asking is this. Normally when we deviate from guide scores, the recommendation is an absolute number. In other words, when we want Dual Audio for anime, we offer options like 10 and 100 or sometimes 101. Those are more difficult to represent in terms of percentages (or ratios) of the original base value.
To be clear, I don't have an issue with the idea itself. I'm just trying to better understand the motivation here. I'm interested in understanding how this is effective. What is a real world example that would benefit from this feature and how does it benefit?

The only real-word example I'm aware of is the one I mentioned above about audio formats, where some folks score these at 10% of the base value so that they're not so heavily weighted. That's not any sort of guide recommendation, it's a solution that users found for themselves.
I don't think this is a 'must' by any means. It's just another option for flexibility and score manipulation.

Sorry, I forgot that you already mentioned in the OP that it is a real-world example. It's not a bad idea but probably won't get priority any time soon.
I appreciate the report!

I also want this feature, except I don't like how it's being requested to be implemented. I think it should be something like adjust_score, where the value just gets added to the trash score to get the final score. For example, I currently have this:
custom_formats:
    - trash_ids:
        - 7878c33f1963fefb3d6c8657d46c2f0a # DV HDR10
        - 2b239ed870daba8126a53bd5dc8dc1c8 # DV HDR10Plus
      quality_profiles:
        - name: Any
          score: 1550
        - name: Bluray Remux
          score: 1550
        - name: 1080p
          score: 1550

I would much rather do this:
custom_formats:
    - trash_ids:
        - 7878c33f1963fefb3d6c8657d46c2f0a # DV HDR10
        - 2b239ed870daba8126a53bd5dc8dc1c8 # DV HDR10Plus
      quality_profiles:
        - name: Any
          adjust_score: 50
        - name: Bluray Remux
          adjust_score: 50
        - name: 1080p
          adjust_score: 50

As noted by OP, this is better because it will continue to work if the trash scores are updated, whereas the absolute score will not.

What if I supported something like this?
custom_formats:
    - trash_ids:
        - 7878c33f1963fefb3d6c8657d46c2f0a # DV HDR10
        - 2b239ed870daba8126a53bd5dc8dc1c8 # DV HDR10Plus
      quality_profiles:
        - name: Any
          score: 50 # Score is set to exactly 50
        - name: Bluray Remux
          score: +50 # Score is set to DEFAULT+50
        - name: 1080p
          score: -50 # Score is set to DEFAULT-50

Basically you can specify a relative score to subtract or add to the default value.

What if I supported something like this?
custom_formats:
    - trash_ids:
        - 7878c33f1963fefb3d6c8657d46c2f0a # DV HDR10
        - 2b239ed870daba8126a53bd5dc8dc1c8 # DV HDR10Plus
      quality_profiles:
        - name: Any
          score: 50 # Score is set to exactly 50
        - name: Bluray Remux
          score: +50 # Score is set to DEFAULT+50
        - name: 1080p
          score: -50 # Score is set to DEFAULT-50

Basically you can specify a relative score to subtract or add to the default value.

I'm good with that, but I think it might confuse people because I'm pretty sure you'd have to specify the + version as a string. Otherwise your code will just get it as a positive number. For example:
score: 50 == score: +50, so the user would have to do score: "+50"

Ah good catch. I didn't think it all the way through. The spirit of what I was trying to capture is that we could allow people to specify relative score offsets. I initially thought of some small template language like score: {DEFAULT}+50 but thought that's too confusing, and score: +50 would be more concise, but that won't work for reasons you mentioned.
I realize I could go with something like add_score: 50 and subtract_score: 50 but that looks a little ugly to me. Your idea of using adjust_score might work but it needs to be able to indicate either subtraction or addition.


That Time I got Reincarnated as a Slime 3rd Season not showing

I face a problem that some anime are not showing in the stremio in the addons. I will give you a anime name: That Time I got Reincarnated as a Slime 3rd Season. Maybe some anime has a problem of loading like these. Can you fix this problem. It's an urgency. I am a anime fan. I love to watch anime. A
I mostly use Stremio. But this problem prevents me to watch anime. I will provide the Screen Recording from Stremio to explain the problem in a better way.
https://github.com/user-attachments/assets/057d3172-908b-4f0d-93b2-f7cc60cd6b20

Thanks for the video demo, I managed to reproduce the problem and see what was going on. The issue seems to be with the Kitsu addon, for some reason the MAL IDs for Season 1 and  Season 3 point to the same Kitsu ID. Sadly this is something I can't fix on my end. But I can reach out to the dev of that addon.
Sorry for the trouble.

Thanks. I also found another problem. Alya Sometimes Hides Her Feelings in Russian - this anime is also not showing and just showing other anime. Same problem like That Time I got Reincarnated as a Slime.

That time I got Reincarnated as a slime 3rd season not showing the problem is solved.

Thanks a lot. @SageTendo, I also found another problem. Alya Sometimes Hides Her Feelings in Russian, this anime is also not showing and just showing other anime. Same problem like That Time I got Reincarnated as a Slime.

Another user had a similar issue:
https://github.com/SageTendo/mal-stremio-addon/issues/21#issuecomment-2322843298
The issue seems to  be related to the mapping of MAL to Kitsu IDs and Kitsu to MAL ID, but if you play it from the Kitsu list it syncs just fine.

@SageTendo, but I just want to track it to Mal Automatically while watching anime.


Forward AD threw error in tf.autodiff.ForwardAccumulator for tf.keras.layers.AveragePooling3D, but backward AD succeeded with same input

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
tf 2.14.0
Custom code
Yes
OS platform and distribution
windows colab
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Forward AD threw error in tf.autodiff.ForwardAccumulator for tf.keras.layers.AveragePooling3D, but backward AD succeeded with same input.
Standalone code to reproduce the issue
import tensorflow as tf

pool_size = [2,1,1]
strides = [2,2,2]
padding = "valid"
data_format = "channels_last"
input = tf.constant(0.895205,shape=[1,1,1,1,1], dtype=tf.float32)
layer = tf.keras.layers.AveragePooling3D(pool_size=pool_size, strides=strides, padding=padding, data_format=data_format, )

with tf.GradientTape(persistent=True, ) as g:
    g.watch(input)
    res_backward = layer(input)
grad_backward = g.jacobian(res_backward,res_backward)
print("res_backward:",res_backward)
print("grad_backward:",grad_backward)

tangents = tf.constant(1.,dtype=tf.float32,shape=[1,1,1,1,1])
with tf.autodiff.ForwardAccumulator(input, tangents) as acc:
    res_forward = layer(input)
    grad_jvp = acc.jvp(res_forward)
    print("res_forward:", res_forward)
    print("grad_forward", grad_jvp)

Relevant log output
res_backward: tf.Tensor([], shape=(1, 0, 1, 1, 1), dtype=float32)
grad_backward: tf.Tensor([], shape=(1, 0, 1, 1, 1, 1, 0, 1, 1, 1), dtype=float32)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-3-c1962478ba56> in <cell line: 18>()
     17 tangents = tf.constant(1.,dtype=tf.float32,shape=[1,1,1,1,1])
     18 with tf.autodiff.ForwardAccumulator(input, tangents) as acc:
---> 19     res_forward = layer(input)
     20     grad_jvp = acc.jvp(res_forward)
     21     print("res_forward:", res_forward)

1 frames
/usr/local/lib/python3.10/dist-packages/tensorflow/python/framework/ops.py in _create_c_op(graph, node_def, inputs, control_inputs, op_def, extract_traceback)
   1019   except errors.InvalidArgumentError as e:
   1020     # Convert to ValueError for backwards compatibility.
-> 1021     raise ValueError(e.message)
   1022
   1023   # Record the current Python stack trace as the creating stacktrace of this

ValueError: Exception encountered when calling layer 'average_pooling3d_2' (type AveragePooling3D).

Negative dimension size caused by subtracting 2 from 1 for '{{node gradient_tape/gradient_tape/AvgPool3D}} = AvgPool3D[T=DT_FLOAT, data_format="NDHWC", ksize=[1, 2, 1, 1, 1], padding="VALID", strides=[1, 2, 2, 2, 1]](tangents)' with input shapes: [1,1,1,1,1].

Call arguments received by layer 'average_pooling3d_2' (type AveragePooling3D):
  • inputs=tf.Tensor(shape=(1, 1, 1, 1, 1), dtype=float32)

@cheyennee,
Looks like this is duplicate of issue #56833. Could you please close this issue, since it is already being tracked there? Thank you!

Hi, @tilakrayal ,
It is not a duplicate of issue #56833. The provided code snippet triggers an error only when padding="valid" is used, not when padding="same". Additionally, in line with the details outlined in #56833, I have already configured the data_format parameter.

@cheyennee,
I was able to replicate the reported behaviour with TF2.14 and tf-nightly as well. Attached gist for reference.
Currently we are investigating the issue & will deep dive into the issue and provide the root-cause for the same. Thank you!

Hi @cheyennee,
The error is due to the pool_size and input dimensions , both are incompatible. In this case, your input tensor dimensions are lesser than the kernel size. Obviously, no convolution is performed and hence the error. So either increase the input dimension or decrease the pool size. Please refer to the gist
Thank You


Pull request for shellcheck

Resolves travis-ci/apt-package-whitelist#362.
Add packages: shellcheck
See https://travis-ci.org/travis-ci/apt-whitelist-checker/builds/72545300

Superseded by #454


How to force the terminal to stop when running C + + program dead loop?

In case of external keyboard, I tried "ctrl + c" and , but it didn't work.

I can’t stop the python program too

Fixed by https://github.com/thebaselab/codeapp/issues/612. Use control-c or the button in the terminal panel.


Request: Installer tutorial

Awesome set of current tutorials.  Any chance on getting a tut for doing installers for all 3 platforms?

installer tutorial is on the todo-list. I don't have an ETA but i'll take a look at it as soon as possible.

macOS dmg tutorial is published: http://www.christianengvall.se/dmg-installer-electron-app/
Windows and Linux tutorial is on its way

Windows tutorial added: http://www.christianengvall.se/electron-windows-installer/

And finally a debian-package tutorial added: https://www.christianengvall.se/electron-installer-debian-package/


Add M2_REPO support for build image

Developers should be able to use a custom local Maven repository location.

Closed with #39.


Agenda should have separate notes for text version and html version

What happened?
I.e. the text version should still include text like:
Times on the text agenda are shown in Vienna local time, UTC +1.
On the other hand the html version should not include that line, as there the timezones are configurable.
The reason secretariat puts that note there is to get that text to show on the text only agenda, where you can't configure the timezone. To fix this issue we should have separate configurable agenda_info_note for text and html versions.
What browser(s) are you seeing the problem on?
No response
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Instead of separate configurable notes, the text version should just provide the declaration of timezone itself. That way any note the secretariat provides still shows in both places (avoiding the potential for error if one place is missed).

Was this fixed already?  When I look at the plaintext agenda, I see the timezone text:

When I look at the HTML agenda pages, I see the timezone text:

and

It's not fixed, but I think it needs to be fixed with a new agenda note rather that with a DT patch.

It's not fixed, and a new agenda note may be warranted, but what I think should happen is to make it such that the agenda note does not have to speak of the meeting timezone. The datatracker already knows that and even uses the information on the html agenda page. It can use the information to say what the meeting timezone is on the text agenda page as well, relieving the secretariat of the burden of having to work that into meeting agenda note  (and get it right).

@ IETF 115 code sprint, going to try and work on this.

I believe this is OBE (or will be at the next datatracker release) - see #5233


:3000 doesn't show any connected nodes

<hub-ip>:4444 shows all the connected nodes, but <hub-ip>:3000 doesn't show any connected nodes.
I can navigate to any <node-ip>:3000 and see overview as well as recorded videos.
I read somewhere that the recorded videos on nodes get copied over to the hub, but my video_output directory on the hub is empty. Also <hub-ip>:3000 only shows the hub machine's details as if it was a node. (I have configured the hub for selenium grid extras with the first option - hub, not 3 - hub and node)
Here are my configs -
Startup script for the hub:
./start_grid_extras.sh &

startup script for the node:
./start_grid_extras.sh &

selenium_grid_extras_config.json for the hub
{
  "theConfigMap": {
    "hub_config_files": [
      "selenium-hub-config.json"
    ],
    "auto_start_hub": "0",
    "iedriver": {
      "version": "3.3.0"
    },
    "default_role": "hub",
    "chromedriver": {
      "bit": "64",
      "version": "2.28"
    },
    "auto_update_drivers": "1",
    "auto_start_node": "0",
    "hub_config": {},
    "geckodriver": {
      "version": "0.15.0"
    },
    "log_maximum_age_ms": 8.64E8,
    "reboot_after_sessions": "0",
    "grid_extras_port": "3000",
    "webdriver": {
      "version": "3.3.1"
    },
    "node_config_files": []
  }
}

hub_4444.json
{
  "port": 4444,
  "newSessionWaitTimeout": -1,
  "servlets": [
    "com.groupon.seleniumgridextras.grid.servlets.ProxyStatusJsonServlet"
  ],
  "withoutServlets": [],
  "custom": {},
  "capabilityMatcher": "org.openqa.grid.internal.utils.DefaultCapabilityMatcher",
  "throwOnCapabilityNotPresent": true,
  "nodePolling": 5000,
  "cleanUpCycle": 5000,
  "role": "hub",
  "debug": false,
  "browserTimeout": 60,
  "timeout": 1800
}

selenium_grid_extras_config.json for the node
{
  "theConfigMap": {
    "auto_update_browser_versions": "1",
    "hub_config_files": [],
    "auto_start_hub": "0",
    "iedriver": {
      "version": "3.3.0"
    },
    "default_role": "node",
    "unregisterNodeDuringReboot": "true",
    "chromedriver": {
      "bit": "64",
      "version": "2.28"
    },
    "auto_update_drivers": "1",
    "video_recording_options": {
      "videos_to_keep": "40"
    },
    "auto_start_node": "1",
    "hub_config": {},
    "geckodriver": {
      "version": "0.15.0"
    },
    "log_maximum_age_ms": 8.64E8,
    "reboot_after_sessions": "40",
    "grid_extras_port": "3000",
    "webdriver": {
      "version": "3.3.1"
    },
    "node_config_files": [
      "selenium-node-config.json"
    ]
  }
}

node_5555.json
{
  "capabilities": [
    {
      "seleniumProtocol": "WebDriver",
      "browserName": "firefox",
      "maxInstances": 10,
      "version": "",
      "platform": "LINUX"
    },
    {
      "seleniumProtocol": "WebDriver",
      "browserName": "chrome",
      "maxInstances": 10,
      "version": "",
      "platform": "LINUX"
    }
  ],
  "loadedFromFile": "selenium-node-config.json",
  "proxy": "org.openqa.grid.selenium.proxy.DefaultRemoteProxy",
  "maxSession": 10,
  "port": 5555,
  "register": true,
  "unregisterIfStillDownAfter": 60000,
  "hubPort": 4444,
  "hubHost": "<hub-ip>",
  "registerCycle": 5000,
  "nodeStatusCheckTimeout": 5000,
  "downPollingLimit": 2
}

I'm having a similar issue. I am currently running my Grid Hub on a Mac Mini, running OSX 10.12.5, and my nodes are virtual machines running on the same Mac.

Yes same here. I'm having a hin running on rechst. And 6 nodes running latest macOS

I thought this was true as well I read somewhere that the recorded videos on nodes get copied over to the hub, but my video_output directory on the hub is empty. , but I have noticed I don't see the videos on the hub either (I think).

@sanghviharshit Can you explain this in more detail ? Also <hub-ip>:3000 only shows the hub machine's details as if it was a node. . Maybe post a picture screenshot of what you are seeing for the :3000

Hub can get the video for you. Here is the URL for it. You just need to know the session ID for your test.
testcaseVideoURL = "http://hubserver:3000/download_video/" + sessionId + ".mp4";

Will close this out on Monday 8/21/2017 if no response from @sanghviharshit .

@smccarthy Ignore the videos not getting copied to the hub part, I guess it just takes a while for the hub to download all the videos from all nodes and I was killing the nodes before that.
Regarding Also <hub-ip>:3000 only shows the hub machine's details as if it was a node.

I believe when you navigate to 3000, it is supposed to show information (screenshots of each node, their configs and system info) from all the connected nodes as well. But in my case, it doesn't show any connected nodes, but only shows the hub's screenshot and config and the videos downloaded from nodes. Please let me know if that is expected behavior.


[WIP][SPARK-48572] Fix DateSub, DateAdd, WindowTime, TimeWindow and SessionWindow expressions

What changes were proposed in this pull request?
Fix for listed expressions.
Why are the changes needed?
These expressions are found to be faulty when working with collations.
Does this PR introduce any user-facing change?
Yes, it fixes expressions that are not working with collations.
How was this patch tested?
Tests added for expressions to reproduce errors.
Was this patch authored or co-authored using generative AI tooling?
No.

btw, should we add some tests for these changes?

Will add them a bit later and will remove WIP when it is done.

@cloud-fan could you review this?

thanks, merging to master!


Azure DevOps REST API Documentation for Graph Users is missing

Regardless of which version of the API, clicking on any links under Graph > Users on the site https://docs.microsoft.com/en-us/rest/api/azure/devops/ results in an HTTP 404. Further, clicking on the search links provided also returns an HTTP 404.

@elbatk can you please take a look? I can repro this.

Thanks @mairaw and @PerriNelson , adding @wnjenkin to take a look since I moved over to Azure Artifacts :)

Let's have the metadata updated then 😉

@rloutlaw do you know who owns this?


use 3.1.3

https://stackoverflow.com/questions/24283196/no-such-file-or-directory-rb-sysopen-using-openuri

explicitly call URI.open instead of just open:
data = URI.open("http://google.ca/")

relevant issue: https://bugs.ruby-lang.org/issues/15893


Update README.md

🤭Add paper about poisoning attacks and defenses.

More workshops about the topic.


HWKAPM-781 - Added support for OpenTracing

@objectiser , @pavolloffay , @rafabene could you please review? Note that the changes from this PR should be merged around the same time as similar changes for the other modules.
I added two more commits to this: the second commit is something I had to do here to get this module working, but is not related to opentracing at all. I'm wondering if you also need it?
The last change is to fix something I've missed on the first :) I set the port of bonjour to 8180 locally, to make it easier to test all services directly on bare metal, instead of openshift.

It needs to be rebased to the master branch.

PR Rebased.

This PR has been updated. I believe this is now ready to be merged, but a review from @rafabene would be useful.

PR updated with the final changes.

This PR has been updated to remove APM. This is now only about adding support to OpenTracing, with ZipKin as the sole available implementation.

Actually, the contents of this PR are not relevant anymore. Closing.


request() fails if Tumblr returns https:// URL

I'm playing with a new Tumblr blog and using Tumblpy; it took me a bit to find out why it was failing. On my new blog,
the_client.post('user/info')['user']['blogs'][0]['url']
returns a secure HTTP URL; lines 112 and 113 of api.py fail because they're explicitly matching for 'http://'.
Currently, I'm working around it by replacing 'https://' with 'http://' in a wrapper script, which seems to work fine.

Sorry for the late response. Think you can test out this branch and make sure it works for your case?
https://github.com/michaelhelmick/python-tumblpy/pull/40
I'll merge and push a release if it all checks out!

/ping @patrick-brian-mooney


Use new mochiweb recbuf|sndbuf undefined setting

Fixes #1409.
The new version of mochiweb that we've moved to supports undefined as a sndbuf|recbuf default setting. This should help CouchDB with attachments on high-latency links, without causing regression for CouchDB over low-latency links.
Tests were clear locally; let's see how they fare in the more heavily contended Travis CI environment.
/cc @kocolosk

Hi @wohali !
This patch worth to check on MacOSX and FreeBSD, because these two were the reason of the last buffer change. IIRC, originally, we didn't have such tweaks at all, so OS managed them in the same way. Linux would be fine in any case.

@kxepal Interesting! If you look at #1409 you'll see that this problem specifically cropped up on Linux running on AWS, so these defaults don't work anymore for Linux.
If you can help run the test cases, I'd greatly appreciate it. We still don't have dedicated FreeBSD/MacOS Jenkins boxes, but they are coming.
I don't think it makes sense to bake OSX/FreeBSD defaults into the code itself, but perhaps we could add comments in default.ini and the documentation about setting these values explicitly on other OSes if necessary? Given the increased up-take in Docker and packaged installations, I don't think it makes sense to hobble Linux distributions with bad default settings.

Ooh! Another option is to address this with etc/default.d/10-socket.ini files in the BSD ports tree (/cc @dch) and over in @janl 's OSX installer builder. That's what etc/default.d/ is intended for - package maintainer default files for distro-specific changes.

@kxepal in your comments on https://issues.apache.org/jira/browse/COUCHDB-1986 you state that multipart parser speedup as per https://issues.apache.org/jira/browse/COUCHDB-1953 solves the issue, yes? And that landed a while ago, so isn't your concern already addressed?

Hm...I fear that AWS applies some own specifics on top of. Debugging clouds is hard.

If you can help run the test cases, I'd greatly appreciate it. We still don't have dedicated FreeBSD/MacOS Jenkins boxes, but they are coming.

I have FreeBSD at work and will check it tomorrow. The only interesting test is replication one where we check how good we are in replication of various local-remote combinations some docs with sort of ~10MB attachments. It triggers issue perfectly by timeout and just slow run.
At those time we tried to fix OSX performance at the first place. May be that problem is not actual anymore.

Another option is to address this with etc/default.d/10-socket.ini files in the BSD ports tree (/cc @dch) and over in @janl 's OSX installer builder. That's what etc/default.d/ is intended for - package maintainer default files for distro-specific changes.

That's great idea! I like it :+1:

@wohali
Wow! I don't recall 1953 merge in upstream at all.  Feels like it should solve the problem, but better check twice.

For what it's worth mochiweb only handles the recbuf case; it explicitly looks for undefined and if it finds that atom it declines to pass any option to the underlying gen_tcp system:
https://github.com/apache/couchdb-mochiweb/commit/42d37e6
The reason they did it this way was because the backwards-compatible behavior for mochiweb was that leaving the option blank meant mochiweb would fix the initial recbuf to 8192.
By contrast, I would guess that the sndbuf setting is quietly returning {error, einval} when we use the atom undefined here, although it ultimately achieves the same goal of letting the OS manage the buffer.

I dropped the sndbuf change, but the recbuf change is still failing. Need help.

I wonder if other parts of the codebase are using those options directly with gen_tcp. Will take a look.

OK I see the problem -- mochiweb only does this recbuf dance when opening a new socket to add to the acceptor pool. CouchDB is actually executing this code to set the socket options at the beginning of every request; it uses mochiweb_socket:setopts/2 on the existing socket which simply calls inet:setopts/2 (and fails).
Need to see if how we configure this so that our socket options overrides are set on a per-connection basis instead of a per-request basis.

OK, with the help of @kocolosk and @rnewson it seems we should pass {recbuf, undefined} to [chttpd] server_options, and remove the recbuf setting from socket_options entirely.
PR updated.

FYI this PR is on hold until we can review socket buffer size behaviour on all major supported OSes (Linux, FreeBSD, macOS, Windows) and determine sane defaults for all.
@rnewson is concerned that suddenly dropping our default 256k size by half, or more, could have a deleterious impact on many people. I concur.
It may be that letting the OS manage buffer size would be better, esp. for OSes that do automatic buffer scaling like FreeBSD. What we really need is some benchmark data, across slow/high latency network connections. This will take some time.
At the very least, we will update our documentation...once we know better info about defaults.

Ah good find on server_options - that's clearly what we wanted.
I understand the reticence about changing defaults. I also think it's rather unlikely that the CouchDB project is smarter about managing socket buffers than any reasonably modern operating system. I'd support switching to undefined as a default and documenting the local.ini setting necessary to restore previous behavior.

I agree @kocolosk . I would still like to run a basic perf test before we do this. Does anyone at Cloudant have a harness already set up that would be willing to run something? I know I've seen perf stuff show up before from the likes of @davisp and @vatamane.


Pass the --alias CLI flag to @netlify/config

If the --alias CLI flag corresponds to the concept of deploy contexts as described in our documentation, i.e. either a context alias or a branch, then this flag should be passed to @netlify/config when resolving the configuration. At the moment, the --context CLI flag (from the netlify build CLI command) is passed, but not --alias (from netlify deploy).
Passing the context ensuring that the context-specific configuration in netlify.toml is being applied.

Closing: the --alias CLI flag is not the same as contexts (see https://github.com/netlify/cli/issues/962#issuecomment-652888559)


Conditional Compilation (#ifdef, #ifndef, #else, #endif)

Hello,
Currently there is no syntax highlight feature for conditional compilation.
#ifdef, #ifndef, #else, #endif
EDIT: I created a patch. See pull request: #16

Thank you!


add generating emojis

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.75 :tada:

Comparison is base (32546a9) 96.97% compared to head (cbd02a3) 97.73%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
+ Coverage   96.97%   97.73%   +0.75%
==========================================
  Files          20       20
  Lines        1520     1586      +66
==========================================
+ Hits         1474     1550      +76
+ Misses         46       36      -10

Impacted Files
Coverage Δ

src/modules/internet/InternetTest.cpp
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Issue executing trades.  Trades get reverted

So my smart contract has difficulty executing trades and I was able to pinpoint the source of the problem here.

In my terminal, I get this message after the trade has been made.

The transactions also get reverted as well, as displayed in the contract transaction
Contract Address
The source of the issues seems to derive from this function in the smart contract:

They happen once this function is called in the node application:

@tycooperaow if you verify your contract code on Etherscan, we can debug the error on tenderly.dev and see what's going on. Let me know!

How do I go about getting it confirmed on etherscan? I been having some issues with that.

The key is to make sure you select the proper compiler (0.4.26), single file, and set optimized to false (default).
https://etherscan.io/verifyContract?a=0x1a5924bed24b4dece2b2c217049a7cb6628c1200 for your contract

I was given this error:
Error! Unable to generate Contract ByteCode and ABI

`

@tycooperaow you don't need to paste your ABI on Etherscan when doing single files  on compile 0.4.26. Not sure why this is the case on Etherscan but try verifying it without the ABI. Ran into that issue a while back, and just removed the ABI... It's a thing...

So I did what you told me and still go the error, here's my process

@tycooperaow the only thing I can think of is the the contract code is indeed not what was submitted. Good news though, a UI for executing arb is coming to OrFeed in the coming days, so will be much easier to test out, figure out where bugs reside (if they reside)!

That's good to hear, any update on when it'll be released?

Likely within 72 hours!

@tycooperaow its here https://www.orfeed.org/angle


Terminal Scrollbar: Right Click on scrollbar does paste not scroll

Testing #223132
Right clicking on the scrollbar does a paste into the terminal, as if I was in any other part of the terminal. Given the behavior of scrollbars in the editor I expected the terminal to scroll at a minimum (and a context menu might be nice).

right clicking on the scrollbar itself will still run the paste action, right clicking on a different part of the scrollbar (that we aren't scrolled to) scrolls as expected based on the issue.
https://github.com/user-attachments/assets/8724e674-c228-4b33-b478-2589b48cfe5f

I see a context menu after right-clicking in a pwsh terminal. I'm unsure how to get right click to paste directly.

@rzhao271 it depends on the rightClickBehavior setting.

Confirmed that a single right click scrolls.
A double right click still pastes.


Allow {{app}} templating in producer messgaes

cc @cyx

LGTM

@dmcinnes @gregburek ^

This looks pretty cool @heroku/dod-core and @heroku/dod-kcz for visibility on sending notifications to customers via telex, we can use {{app}} for telex to filling.

For completeness, I indicated to @ypaq that we should make sure the title/body actually has the {{app}} substitution before making the API call.

:+1:


Multiverse not working with 1.16.1-1.16.2 paper

Information

**Paper 1.16.1 but 1.16.2 implemented

server log -> https://pastebin.com/kJtagxc6

Help request
Problem
The plugin doesn't work at all
What I have tried
Downloading dev builds, going through the config I have tried everything I can think of

See #2215 there is build that fixed this. https://github.com/Multiverse/Multiverse-Core/issues/2215#issuecomment-640545807

This is a known issue. It's usually an incompatibility with another plugin, see #2215 for a build of MV that'll work.


Updated Readme: Xcode and MacOS requirements

What
Updated REAME to reflect Xcode 9.3 and High Sierra requirement.
Why

PR #141 changed code from flatMap, which works on Swift 4.0 -> to compactMap, which works on Swift 4.1 and newer. So,
Swift 4.1 runs on Xcode 9.3, which only runs on High Sierra

Major Changes

Updated README

Requirements for Merging

[X] Builds

Testing Plan

[X] Tested this change locally
[X] Checked that existing features work

Excellent, thank you!


add: RSS feed for missing transactions

Can be found under /missing/feed.xml and contains txid, tags and a link to the /missing/ page for this transaction.
closes #31

Tested, works. https://miningpool.observer/missing/feed.xml


Impossible to delete through mask character (like '(' or '-' in phone number or thousand separator)

I've searched a bit through issues and it looks like for some reason this is considered expected behavior. Say you delete '(' in phone number and mask puts it right back in. This is totally not expected behavior, these characters are decorative and should not prevent deletions. You should be able to backspace and delete over them in both directions.
To clarify, I'm a Win user and backspace means delete char before caret and delete means delete one after.
If you go to demo page and disable both guide and char position, you can backspace over '(' but cannot backspace over '-' if you start backspacing on a char not right after '-'. You cannot delete over them both no matter how you try. Not only this is inconsistent, it also prevents user input flow in many many situations. You cannot even delete through thousands separator so you cannot even reliably delete numbers from left to right.
The point of masked input is to allow user enter the data in a way that's easiest for them to enter and to display it in a formatted way, either that is easiest to process or prettiest to display. Until this issue is resolved, I don't think this lib can be used in production. No disrespect to the authors, this library is great at many levels at what it does, but this is like the main thing it is suppose to do in my opinion.

same problem... any help?

@waterplea
This is a known issue (is documented here #230)..

I found a workaround
https://github.com/text-mask/text-mask/issues/230#issuecomment-381350161

Same problem... any help?
As a workaround I assign the guide = true property and remove the underline in onBlur.

Hey would like to learn how to 'remove the underline in onBlur'? Thanks! @cmateusmoraes


Can't create category page

Issue from https://github.com/hexojs/hexo/issues/2170 @phicarre

I created a page with the command: hexo new page categories
In source/categories/index.md, I added type: categories in the front matter
In the _config.yml of the theme, I added in the menu: Categories: /categories
but my page is always empty !
What is missing ?

I also investigate the issue, and couldn't find a solution. Do we support category layout in this theme?

I have the same problem, does this theme support category pages?

Yes, category pages are produced for categories.  Here is my category test confirming operation with my employment of the landscape theme: http://orcmid.com/BlunderDome/wingnut/2017/12/02/Categories/
Notice the category list on the sidebar, and the links to category pages.  (The category page lack a ttie that conveys the category, so that's something to ponder.  It is in the URL.)
To see whatever I did regarding the enabling of category pages, you can see my sources at https://github.com/orcmid/hexo-theme-landscape-wingnut for the landscape theme,
See https://github.com/orcmid/hexo-spanner for category-related information in the overall blog, especially in the _config.yml


copr: vectorize reverse_binary

Signed-off-by: zhongyang.wu<EMAIL_ADDRESS>PCP #5751

What have you changed?
vectorize reverse_binary
What is the type of the changes?

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

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

Unit test
Integration test

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

/run-tests

/run-tests

/run-all-tests

/run-all-tests

/run-all-tests

@YangKeao PTAL, thanks!

/merge


[Docs]: Expand on the "Getting Started with CPP" post

The getting started with CPP post (link: https://subhadipbhowmik.github.io/30-Days-Of-CPP/docs/day-01/getting-started-with-cpp) currently only contains a minimal outline of everything the post is supposed to include, and i'd like to expand it into a full-fledged tutorial.

@subhadipbhowmik can i work on this?

The getting started with CPP post (link: https://subhadipbhowmik.github.io/30-Days-Of-CPP/docs/day-01/getting-started-with-cpp) currently only contains a minimal outline of everything the post is supposed to include, and i'd like to expand it into a full-fledged tutorial.

For sure. Keep Work on it. Special warning: Don't just copy and paste the code from AI chatbots.


Feature/recoursive merge

Sorry for being annoying :-)

Nice!
null and undefined is not working, so the behavior of lodashs merge is that
null wins and undefined is ignored.


stream settings: Add explain why admin can't add users to a private stream.

Fixes #10593

Testing Plan:
GIFs or Screenshots:

@rishig thoughts on this UI?

Thanks @YJDave! A few thoughts:

The hover behavior should be there on both UI elements (Email address input, and Add button)
Ideally we'd use a javascript popover that appears immediately, rather than html title.

Also while I'm looking at this: Let's change the "Email address" placeholder to "Name or email".

@rishig Thanks for review!
updated, new UI:

The centering looks really off in that error message; otherwise looks reasonable.

@timabbott Thanks for review, changed style.

Rebased


How to transport and exchange sharedKey between programming languages

I tried to use this module with JS-JOSE(https://github.com/square/js-jose). The RSA operation is not compatible. You can check here (https://codepen.io/mxie1563/pen/VoKpLw), it failed loading the private key for decryption. Can you offer your opinion on this one?

Hi Michael, thank you for reporting the issue.
To answer your original question. A symmetric key consists of random bytes generated by a cryptographically secure pseudo-random number generator. You can represent these data as base64 encoded string. There is no specific format defined in which to store these keys.
You can encrypt the symmetric key using an asymmetric algorithm, either elliptic-curves or RSA.
The example below demonstrates how you can achieve this using RSA.
const crypt = new OpenCrypto()
crypt.getRSAKeyPair().then(keyPair => {
  crypt.getSharedKey().then(sharedKey => {
    crypt.encryptKey(keyPair.publicKey, sharedKey).then(encryptedSharedKey => {
      crypt.decryptKey(keyPair.privateKey, encryptedSharedKey).then(decryptedSharedKey => {
        console.log(decryptedSharedKey)
      })
    })
  })
})

Regarding the second question, I have been very busy recently and did not look into it yet. Nonetheless, I very appreciate your interest in OpenCrypto and will try to address the issue as soon as possible. Please let me know if you have any further questions.
Best Regards,
Peter Bielak


The training stopped halfway

When I start to train the agent, it seems some errors will stop the training.
For example
2022-04-01 22:02:19, INFO: Current git head hash code: %s
2022-04-01 22:02:19, INFO: Using device: cpu
2022-04-01 22:02:19, INFO: Policy: CADRL without occupancy map
2022-04-01 22:02:19, INFO: human number: 5
2022-04-01 22:02:19, INFO: Not randomize human's radius and preferred speed
2022-04-01 22:02:19, INFO: Training simulation: circle_crossing, test simulation: circle_crossing
2022-04-01 22:02:19, INFO: Square width: 10.0, circle width: 4.0
2022-04-01 22:02:19, INFO: Current learning rate: 0.010000
malloc_consolidate(): invalid chunk size

Process finished with exit code 134 (interrupted by signal 6: SIGABRT)

or
..........................
...........................
2022-04-01 21:41:03, INFO: TRAIN in episode 321 has success rate: 0.00, collision rate: 0.00, nav time: 25.00, total reward: 0.0000
Process finished with exit code 139 (interrupted by signal 11: SIGSEGV)

Can I ask what caused this?

Hi @cherrymilk, I haven't seen this error before and it looks like some kind of memory error. Are you sure there are no other processes using too much memory and etc? It might be worth to install the repo on a clean environment to check if there is something wrong with your linux setup.


Pokéloot in Ultra Space is set to first come first serve

Describe the bug
When you claim a Pokéloot in Ultra Space on Atlas, it disappears.
Server name (all applicable):
Atlas

Hello! This is intended. As ultra space will be set to reset every so often. Thank you for the report!


Remove advocacy from footer

Advocacy needs to be removed from footer. Couldn't figure out how to do it myself.

There is a file in the templates folder called footer.html, in there you can find the advocacy link and remove it, and edit any other footer content you desire. Similarly there are other templates in there that can also be edited.

 When I open footer.html in the templates folder it is blank.

Here is the footer file with content in it on the develop branch on github:
https://github.com/OAButton/oab_static/blob/develop/templates/footer.html
Are you looking at the file locally or on github? If local, try doing a git
pull (although that should be up to date, because that should be done
before any pushes anyway, but just to check).
 When I open
footer.html in the templates folder it is blank.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/OAButton/backend/issues/279#issuecomment-253561482,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuXCAw1YI_oiaXTaGdVuiQCB309bW3Oks5qzljMgaJpZM4KWEXB
.

I was looking at the file on github.
Thank you. Removed it.


Order of Pane

Is there Any way to show change order of panes ? i want result pane to show at last not at first

Sure, you can use some CSS flexbox trickery. Here's a quick demo:
http://s.siloz.io/n


Parallel Approach

I was playing at KSFO the other day, and had a huge wave (lots of fun, thanks! :+1: ) coming in.
As they were coming in from two sides, with their separation being minimum already, I wanted them two land at two runways simultaneously. As traffic got closer to the extended center line, the white circles started popping up, but as I cleared them on the approach (will be creating a new issue about that as well), they would disappear (great job!)
Here is the thing though:
When having two airplanes on a parallel approach (one for 28R and the other one for 28L), you still get the red circles and some points deducted.
KFSO (and many other large airports) allow for a parallel  approach. (Page 12 for more detail)
This means that two aircraft can fly almost side by side as long as they are on the ILS and the later traffic has the one in front in side.
For simulation purposes I think we could skip over the "in sight" rule, but it'd be great if you could have two planes land parallel, as it gives you the opportunity to get traffic in faster, and handle even more traffic!

Good stuff. I set up the arrivals for KSFO so that both 28L and 28R are needed for landings at peak.
There are no special rules in the simulation for SOIA or SCP, however it is possible to play with the current KSFO  arrival rate without point deductions. What you need to do is vector the aircraft for landing slightly staggered. This sometimes means sending some of the BIGSUR2 arrivals straight north, and joining the flow with a timely vector to PONKE, for example. Then you can interleave the two major sources of traffic.

Just keep one altitude, say 4K for one side and another one, 3K, for the other side. This way there's little risk of colliding them if you forget to give a vector on time.

@glangford Yeah, thats what I did after I figured out that I would get less points for separation violation.
What I want to achieve here is that when a plane is on the approach (a.k.a. cleared to land in this simulation, check out #262 ) another plane can join the parallel approach without violating the separation minimums.
If there is a higher rate of arrivals so that you more or less have to use parallel approaches (which I would welcome!!) then this should not result in a lower score.
@culebron That would work for the initial approach phase, but as soon as both planes start to descent with the glide slope, the same problem will occur.

I played SFO again, and indeed, on short final they shine red if going one next to another on different runways.

I created a pull request with a penalty for close calls (#267), and will need to take this into account.

The situation at KSFO in reality is significantly more complex than side by side, which is why the code doesn't handle it well.  The existing system allows aircraft to get horizontally within 3000 feet of each during ILS landing, this simulates simultaneous approaches at airports like KJFK and KLAX where approaches can be flown independently and the runways are relatively far apart.
At KSFO it's not just a case of them being allowed to land side by side, rather they have to land nearly side by side.  The aircraft landing on 28L flies a straight in approach, the aircraft landing on 28R must do all of the following:

Be behind the other aircraft
See the other aircraft
Be close enough that wake turbulence will not be an issue
Make a relatively late turn into the runway centerline after visually acquiring both the runway and the 28L aircraft.

Given the above, ATC's job at KSFO is more difficult because aircraft must be lined up as pairs.  The simple solution which either reduces or removes the horizontal distance check fails to capture the interesting things at KSFO while removing real penalties at other airports.  A proper simulation will require a fair amount of work (which I'm not interested in right now, perhaps at a later date).
The features I can think of which are probably necessary before this can be simulated properly are:

Curved or segmented approaches
Stateful separation warning checks (Particularly needed for simultaneous departures)


1.2 Release Planning

Cilium 1.2 planning

Release Committments: https://github.com/cilium/cilium/projects/9

Release Dates

Feature Freeze: 2018-07-06
1.2.0-rc1: 2018-07-06
1.2.0-rc2: 2018-07-13
1.2.0-rc3: 2018-07-20
1.2.0: 2018-07-27

Release Checklist

[x] Planning
[x] Development Phase
[x] Feature Freeze
[ ] Feature Exception Freeze
[ ] Testing & Addressing bugs
[ ] Tag Release
[ ] Release Blog
[ ] Announcement

High Level

[ ] Inter cluster service routing (@tgraf)
[ ] Documentation (@cilium/team)
[ ] BPF runtime improvements (@borkmann, @jrfastab)
[ ] BPF based flow aggregation (@joestringer)
[ ] DNS based policy definition (https://github.com/cilium/cilium/issues/4584) (@raybejjani)
[ ] CI and testing coverage improvements (@eloycoto, @cilium/team)
[ ] Additional metrics (@manalibhutiyani, @mrostecki)
[ ] Iterative policy reliability improvements (@ianvernon, @joestringer)
[ ] etcd-operator compatible Cilium bootstrap (https://github.com/cilium/cilium/issues/4314) (@aanm)

Inter cluster routing

[x] kvstore backed node discovery (https://github.com/cilium/cilium/issues/2374) (@tgraf)
[ ] listen to remote kvstore (@tgraf)

Documentation

[ ] Improve etcd installation instructions
[ ] Rework & simplify generic Kubernetes installation guide
[ ] GKE guide (@techcet)
[ ] Verify and complete kops (@nebril) and kube-spray (@nebril)
[ ] kubadm guide

BPF runtime improvements

[ ] Bounded loops
[ ] ULP + kTLS integration

CI and testing coverage improvements

[ ] Chaos testing in nightly tests (https://github.com/cilium/cilium/issues/4302)
[ ] Additional negative tests in PR suite
[x] Change cilium vagrant image to 18.04 LTS (https://github.com/cilium/cilium/issues/2688) (@ianvernon )
[ ] Split into individual trigger phrases (TBD) 50% (@eloycoto)
[ ] nightly tests: add label-based egress (https://github.com/cilium/cilium/issues/3376) (@ianvernon)
[ ] ensure that resources are created / deleted in adherence to established testing conventions: (https://github.com/cilium/cilium/issues/3821)

Additional Metrics

[x] Endpoint build failures (https://github.com/cilium/cilium/issues/3863) (@manalibhutiyani)
[x] Number of endpoints in failing state (https://github.com/cilium/cilium/issues/3863) (@manalibhutiyani)
[x] Expose endpoint and policy computation time metrics #4684 (@manalibhutiyani)
[ ] Health endpoint latency as metric historgrams (TBD) (@mrostecki)

BPF based flow aggregation

[ ] Trace notification aggregation in BPF (https://github.com/cilium/cilium/issues/4080) (@joestringer )

Cluster diagnosis and troubleshooting improvements

[ ] Verify that sysdump works even if Cilium is in CrashLoopBackoff (TBD) (@manalibhutiyani)

Important bugfixes

[x] https://github.com/cilium/cilium/issues/4329

1.2 additional metrics feature work is complete and merged today.

1.2 additional metrics feature work is complete and merged today.

Mine (latency metrics from cilium-health) are not done yet and they need to be moved to 1.3. Feel free to remove my nick from this issue.


Automated PR: Cookstyle Changes

Hey!
I ran Cookstyle 7.25.10 against this repo and here are the results.
This repo was selected due to the topics of chef-cookbook
Changes
Issues found and resolved with resources/server_install.rb

346:15 refactor: Chef/Modernize/UseChefLanguageSystemdHelper - Chef Infra Client 15.5 and later include a systemd? helper for checking if a Linux system uses systemd. (https://docs.chef.io/workstation/cookstyle/chef_modernize_usecheflanguagesystemdhelper)

Released as: 7.1.0


Install gateway dependencies from Poetry instead of a separate requirements file

Continues from #639

This PR is creating a lot of issues, namely larger Docker image size and that Poetry doesn't install the gateway extras without the root package dependencies. There's an open issue to track this on Poetry's end (https://github.com/python-poetry/poetry/issues/4842).


Possible spec change: realtimeClient.connect() returns a promise

Noticed someone in the realtime team do await realtime.connect() in a test that used ably-js. This was a bug -- connect() doesn't return anything; they needed to do await realtime.connection.once('connected'). That bug was fixed in code review.
But ultimately that they did this because it seemed like a sufficiently natural operation that they assumed it would work that way. And if they did that then probably customers are too. So why not make it work?
Specific proposal: for 1.3 (or 2.0 if we consider this a big enough breaking change to postpone for a major version, it does change the function signature), modify RTN11 such that realtimeClient.connection.connect() (and so also realtimeClient.connect(), which is an alias to it) returns a promise (/ takes an optional callback) that resolves when the client reaches the connected state, or is rejected if before then the client reaches a terminal state (failed or closed). I can write up a docs pr if there's agreement on this.
@paddybyers @QuintinWillison
┆Issue is synchronized with this Jira Task by Unito

I'm not sure how many app use cases specify autoConnect: false, and thus call connect() "manually", however I do think this would be a nice feature.
For languages that can return a Promise, or something Promise-like, then I'm not sure it needs to be classed as an incompatible/breaking change. Existing code will not be expecting, nor attempting to use, any returned value.
For other languages that need callbacks to be specified as an additional method argument (e.g. a completion handler block in Objective-C) then we can probably offer this as a new method, overloading what's already there.
Therefore, not so sure it's a spec change - more of an enhancement, being an additional feature in this area that SDKs should offer if they're able to.

For languages that can return a Promise, or something Promise-like, then I'm not sure it needs to be classed as an incompatible/breaking change. Existing code will not be expecting, nor attempting to use, any returned value.

One possible issue in that is that in js, unhandled promise rejections can (depending on how node is configured) bubble up and crash the process. (Which is why you need to explicitly flip on returning promises in ably-js, we don't just do it automatically if you don't pass a callback.)


Fix issue #268

An identifier is added to avoid this issue：#268

Hello,
This repository is not maintained anymore, the new one is https://github.com/gojsonschema/gojsonschema, could you submit your PR on this one? Thank you!


Ingress editor breaks rule paths ordering

What kind of request is this (question/bug/enhancement/feature request):
bug report
Steps to reproduce (least amount of steps as possible):

create ingress with 1 rule and 2 paths: [ "/something", "/" ], in this order
view in yaml: all ok, path order is as entered
edit in GUI: "/" is first, "/something is second". Save with no changes made.
view in yaml:  path order is changed and invalid: "/" is first, "/somehing" is second

Result:

path ordering is broken on GUI edit. More precisely, during the load into the GUI editor.
"/something" never matches anymore, because "/" is now first and is always prioritized.

Other details that may be helpful:
Woraround: always edit in YAML. Never save in GUI as it breaks the ordering.
Environment information
rancher/rancher:v2.1.2, docker, single node

The order in API is incorrect. We need to fix it in backend

@loganhz This change will require UI change for the API schema changed.

Breaking the API is not something to be done casually.  Most languages preserve the order keys are entered, doesn't go have a way to?

@vincent99 As I found out in JSON-spec in http://json-schema.org/latest/json-schema-core.html#rfc.section.4.2.1,

object:
An unordered set of properties mapping a string to an instance, from the JSON "object" production

In our API schema for ingressRule, it is defined as an json object for paths. I don't have any other ways to get around it beside breaking the API.
@ibuildthecloud Do you have a better idea for this one?

We don't use JSON-schema.  While it's technically true that maps are unordered, the reality is there is an order and most languages preserve it (go's normal map intentionally randomizes it on iteration to discourage it).
Or you could add a priority int field.
Or sort the generated rules by path length.  Is there wildcard matching or something else that can go in here?  If literal prefixes is all that you can enter then I don't know why manual ordering is even needed, you sort by length so that /a and /b come first or / will always match everything.

Let's sort the path in reverse order. It would ensure that /abc is in front of /a.

Reverse alphabetical ordering. This seems like a overly-simple, naive solution that is going to bite us in the future.
See: https://github.com/kubernetes/contrib/issues/885 / https://github.com/kubernetes/ingress-nginx/issues/555
How paths are interpreted is specific to the underlying provider. I have seen some truly complex pathing setups in the wild. I think we assume too much by thinking we can impose an arbitrary reverse alphabetical ordering and have that actually be what the customer wants/expects all the time.
@vincent99 your thoughts?


Embed audio in UI

The audio stream should be embedded in the UI when operating in headless mode.  (http://<ip_>:8000/sonicjam)

Done.


Update README with additional mono msbuild notes

Updated README with some notes about what to do for mono msbuild, as the installcredprovider.sh script only installs the netcore plugin on unix based environments.
Also updated the example on how to download and run. wget is not present on macOS per default, so showing an example using curl instead.
Fixes #221 partially. You might consider changing the script to also install the netfx plugin.

@Cheesebaron - I think keeping the wget option doesn't hurt.  I suggest that you update the docs so both command options are present.

Better?


Android Basics: lists

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-lists?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-lists#5
In which task and step of the codelab can this issue be found?
Describe the problem
the order4 and order5 of the proposed code is presenting
type mismatch: inferred type Is Order but unit was expected
unresolved reference: addItem
Steps to reproduce?

Go to the final code created in the section Implement Builder Pattern for Orders
Click on the code and add the 3 commands recommended
See error type mismatch related to the added command return this and unresolved reference due to the new order 4 and 5 that does no recognize the function addItem

Versions
What version of Android Studio are you using?
Additional information
Include screenshots if they would be useful in clarifying the problem.

when I ran the final code proposed the issue disappeared, probably something went wrong in my initial code... once I have it saved I'll check line by line to see any difference


override default ratel args

Need an ability to override default ratel args. Unfortunately ratel doesn't grab these args from env variables, so helm ratel.extraEnvs variable is useless.

 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.


2 tick minimum also for logscale

Description
As raised in #362, logscales can lead to plots with only a single tick mark. This PR aims a fixing that, such that every plot has a minimum of 2 tick marks.
Fixes #362
Checklist:

[ ] I have performed a self-review of my own code
[ ] My code is PEP8 compliant (flake8 anesthetic tests)
[ ] My code contains compliant docstrings (pydocstyle --convention=numpy anesthetic)
[ ] New and existing unit tests pass locally with my changes (python -m pytest)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have appropriately incremented the semantic version number in both README.rst and anesthetic/_version.py

Failing tests after https://github.com/handley-lab/anesthetic/pull/372/commits/0262e964c8796c8684c16c2a7fe14913da2093da (implenting https://github.com/handley-lab/anesthetic/pull/372/commits/b59da430b1ff7fe89546d1a1d8d2c3c63bfc5fef) highlight the problem.


Define and develop metrics for process effectiveness

In order to ensure that community is not facing any process bottlenecks in working on Janssen project, put together dashboard that tracks following data points:

How long a PR has to wait before approval?
How frequently issues are triaged?
Contributor to committer ratio

Due: 31st July 2021

Closing as Duplicate of #15


Manual only schedule?

Is there a way to use scheduler for manual jobs?  I have been using rake for things like this but I was curious.

Kind of defeats the purpose of a scheduled job, @chrishough what were you thinking specifically?  Are you trying to use the "Queue now" portion of resque-web?

Yes it does to some extent.  I was thinking of handy way to incorporate running manual jobs that are not setup in cron via the same interface.  This way developers would not have to go in via ssh and rake.

@chrishough I ran a test and was able to do this:
Within a static schedule, I added this schedule item:
"Static Job":
  class: StaticJob
  never: "* * * * *"

The Rake task just complains with no cron / every found for StaticJob (Static Job) - skipping and it shows up in the resque-web interface!

:boom:

Go for it!!!

Thank you @carsonreinke 👍  Sorry it took me so long to put this together.


TAJO-1732: During filter push down phase, join conditions are not set properly

+1 LGTM

Thanks for your review.


Fill currencies add as subunits format

fix for this issue

@Rhymond Thanks for looking at my PR, you can check the Money gem's currency data and see if I got something wrong.