tmUnquote (or tmDefinition) fails to propagate universe constraints

From MetaCoq.Utils Require Import utils MCList.
From MetaCoq.Template Require Import MonadBasicAst MonadAst TemplateMonad Ast Loader.
Require Import Equations.Prop.Classes.
Require Import Coq.Lists.List.
Import ListNotations.

Local Set Primitive Projections.
Local Open Scope bs.
Import MCMonadNotation.

Class quotation_of {T} (t : T) := quoted_term_of : Ast.term.
Class ground_quotable T := quote_ground : forall t : T, quotation_of t.
Class inductive_quotation_of {T} (t : T)
  := { qinductive : inductive
     ; qinst : Instance.t
     ; qquotation : quotation_of t := tInd qinductive qinst }.

Require Import MSetPositive.
Definition tmMakeQuotationOfModule (m : qualid) : TemplateMonad _
  := cs <- tmQuoteModule m;;
     let dummy_inst := [] in
     ps <- monad_map
             (fun r
              => match r with
                 | ConstRef ((mp, name) as c)
                   => let c := tConst c dummy_inst in
                      '{| my_projT2 := cv |} <- tmUnquote c;;
                      let ty := quotation_of cv in
                      n <- @tmDefinition ("q" ++ name) ty c;;
                      tmReturn (Some (existT (fun T => T) ty n))
                 | IndRef ind
                   => let '(mp, name) := ind.(inductive_mind) in
                      let inst := dummy_inst in
                      let c := tInd ind inst in
                      '{| my_projT2 := cv |} <- tmUnquote c;;
                      let ty := inductive_quotation_of cv in
                      let v : ty := {| qinductive := ind ; qinst := inst |} in
                      n <- @tmDefinition ("q" ++ name) ty v;;
                      tmReturn (Some (existT (fun T => T) ty n))
                 | ConstructRef _ _ | VarRef _ => tmReturn None
                 end)
             cs;;
     tmReturn ps.
MetaCoq Run (tmMakeQuotationOfModule "Coq.MSets.MSetPositive.PositiveSet"%bs).
(* Error: Illegal application:
The term "@quotation_of" of type "forall T : Type, T -> Type"
cannot be applied to the terms
 "Type" : "Type"
 "PositiveSet.elt" : "Type"
The 1st term has type<EMAIL_ADDRESS>which should be coercible to<EMAIL_ADDRESS>*)

Here is a smaller example showing how tmUnquote is failing:
From Coq Require Import Lists.List.
From MetaCoq.Utils Require Import bytestring monad_utils.
From MetaCoq.Template Require Import TemplateMonad Ast Loader.

Local Open Scope bs.
Import MCMonadNotation.
Import ListNotations.
Definition t : Type := nat.
Definition foo :=
  (top <- tmCurrentModPath tt;;
   ' {| my_projT1 := cty; my_projT2 := cv |} <- tmUnquote (tConst (top, "t") []);;
   @tmAxiom "bar" (@id Type cty);;
   tmReturn tt).
Set Printing Universes. Set Printing Implicit.
MetaCoq Run foo.
(* Error: Illegal application:
The term "@id" of type "ID"
cannot be applied to the terms
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>The 2nd term has type
<EMAIL_ADDRESS>which should be coercible to<EMAIL_ADDRESS>*)


Reset postFlushDecryptQueue properly

The postFlushDecryptQueue property of DoctrineEncryptSubcriber was unset. In some cases, a 'undefined property' error was rised because of this. Just empty the array should work too.
Did also some code cleanups.

Thank you for merge! Do you plan to release a new version? These are bug fixes, so a version v5.0.1 would be nice, according to semantic versioning.


How to replace the title view of navigation controller with tab view?

I wrote self.navigationItem.titleView = self.tabBarView; won't work.

Hi,
Please see the attached demo project for using MSSTabBarView as a UINavigationItem.titleView.
TitleView.zip

Thanks. Work like a charm👍


CMake(Python): Add support for aarch64: publish Linux-aarch64 wheels for or-tools on PyPi

Environment
What language and solver does this apply to?
Python
Describe the problem you are trying to solve.
pip install ortools is failing with below error message, on Linux AArch64 machine.
ERROR: Could not find a version that satisfies the requirement ortools (from versions: none)
ERROR: No matching distribution found for ortools

Describe the solution you'd like
Releasing Linux AArch64 wheel should solve the problem.
Describe alternatives you've considered
None
Additional context
Devnote

May use https://github.com/multiarch/qemu-user-static to test aarch64 build in a docker on a x86_64 host machine to test.
Should look at the opencv-python package since they provide aarch64 support and also publish a python native package.

seems they use skbuild

With PEP 599 we now have official manylinux2014 aarch64 base image
ref: https://github.com/pypa/manylinux#readme
Github workflow: https://github.com/spglib/spglib/pull/130/files

Related issue:
#2722 CMake(Python): Requesting prebuilt wheels for M1 macOS without SCIP
#2434 make: coin-or build fails on aarch64
https://aur.archlinux.org/packages/python-or-tools/#comment-801222

ARM64 is not supported, or-tools is a native library so each platform is independent

Update:
Currently the aarch64 native compilation won't work using the Makefile based build since coin-or solver doesn't seems to support it.
see: #2434
On my way to test CMake based build and create manylinux artifact using the manylinux2014_aarch64 base image.
You can follow all development in my dev branch mizux/dev

@Mizux I used the 9.1.9552 you published for aarch64 and had no particular issue, thanks for that!
Are there any plans to publish Linux-aarch64 wheels for fresher or-tools versions like 9.4?

erf my bad I supposed it was done, on my way to do it ASAP !

Thanks a lot for your quick answer!

Uploading ortools-9.4.1874-cp310-cp310-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 14.9/14.9 MB • 00:01 • 22.6 MB/s
Uploading ortools-9.4.1874-cp36-cp36m-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 14.9/14.9 MB • 00:01 • 29.4 MB/s
Uploading ortools-9.4.1874-cp37-cp37m-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 14.9/14.9 MB • 00:01 • 28.3 MB/s
Uploading ortools-9.4.1874-cp38-cp38-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 14.9/14.9 MB • 00:01 • 27.9 MB/s
Uploading ortools-9.4.1874-cp39-cp39-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 14.9/14.9 MB • 00:00 • 38.7 MB/s
View at:
https://pypi.org/project/ortools/9.4.1874/

Thanks! I juste used it and it works great!


fix: read scorer

With this patch we don't read a historically stored scrorer from disk. It's not ideal, but at least users running into panics when loading the scorer will be able to run the app again
Workaround for #797
fix #797

Are there any big disadvantages of recreating the scorer from scratch every time?

I guess we could run without reading from disc. This means, we will not learn from past payments and only keep an in-memory picture of the network.

@reviewers

The first commit fixed the problem for me
The second commit might be the better fix but I can't test it because it does not panic anymore.

If we go for the second, let's squash the commits.
I propose to roll out a new version with this asap, because you really can't use the app if this happens to you.

bors r+

I propose to roll out a new version with this asap, because you really can't use the app if this happens to you.

@bonomat fyi: https://github.com/get10101/10101/actions/runs/5540939930/jobs/10113681545


Revert "feat(update): Adding Resource Pools"

Reverts coreweave/coreweave_determined#15

Reverting till cloud app fixes Region/Mount Selector


Add PluginCommand.register_for_low_level_il and PluginCommand.register_for_medium_level_il

PluginCommand.register_for_address in the LLIL and MLIL graph views only work on the address provided, which can be an issue if that address maps to multiple IL instructions and the user intended for it to run on an IL instruction that isn't the first one.
I'd like to have the option to directly register a plugin for a particular view, such that it is only available in the specified IL view. This also has the added benefit that a callback for register_for_low_level_il would easily work with Lifted IL, Low Level IL, and Low Level IL SSA form. The same for register_for_medium_level_il, as it would work with Mapped MLIL, MLIL, and MLIL SSA form.

Added in build 1159 (both for IL functions and individual IL instructions)


[ALGO] Kosaraju's Algorithm

Kosaraju's Algorithm
Kosaraju's algorithm identifies the strongly connected components (SCCs) in a directed graph. A strongly connected component is a subgraph in which there is a path from every vertex to every other vertex. The algorithm uses two depth-first searches to identify these components.
When possible, it would be nice to see if we can reuse the existing DFS functionality in algorithm/graph_traversal/depth_first_search.h.
For more details, see the wikipedia entry.
Syntax
The algorithm should have the following syntax:
/**
 * @brief Finds strongly connected components in a directed graph using Kosaraju's algorithm.
 *
 * Kosaraju's algorithm identifies strongly connected components (SCCs) in a directed graph.
 * The algorithm uses two depth-first searches to discover these components.
 *
 * @tparam V The vertex type of the graph.
 * @tparam E The edge type of the graph.
 * @param graph The input directed graph.
 *
 * @return A std::vector of vectors, each of which contains the vertex IDs forming
 *         a strongly connected component.
 */
template <typename V, typename E>
std::vector<std::vector<vertex_id_t>> kosarajus_strongly_connected_components(
    const graph<V, E, graph_type::DIRECTED>& graph
);

This should live in the graaf::algorithm namespace under include/graaflib/algorithm/strongly_connected_components/kosarajus.h.
Definition of Done
This issue is done when:

[ ] The algorithm is implemented
[ ] The new function has a javadoc-style comment explaining the interface
[ ] Appropriate tests are added under test/graaflib/algorithm/strongly_connected_components/kosarajus_test.cpp
[ ] A test coverage of at least 95% is reached
[ ] A documentation entry is added under docs/docs/algorithms under the appropriate category

Just adding a short description and the algorithm syntax here is fine
See the wiki on how to build the documentation locally

[ ] The algorithm is added to the list of algorithms in README.md

Hi there - happy to take a look at this one if it's still available.

Hi there - happy to take a look at this one if it's still available. If this issue could be assigned to me that would be great.

Hi, of course! Looking forward to your contribution!

Thanks! If I recall correctly Kosaraju's uses a transposed version of the graph for the second DFS traversal.
Do you think it makes sense to add a get_transposed or something along those lines to the graph class? That functionality might be of use outside of just the Kosaraju's algorithm. I will of course write tests for the new function if that's something you think we might like to go ahead with.

Thanks! If I recall correctly Kosaraju's uses a transposed version of the graph for the second DFS traversal.
Do you think it makes sense to add a get_transposed or something along those lines to the graph class? That functionality might be of use outside of just the Kosaraju's algorithm. I will of course write tests for the new function if that's something you think we might like to go ahead with.

Hi, great idea! Maybe we can add it as a free function as one of our design goals is to keep the graph interface as clean as possible

That sounds great, will do!


Support for Laravel 5.7

Hello will there be support for Laravel 5.7?

@misog yes scheduled next week.

Why don't you upgrade to 5.8?

@sungil good point, totally forgot, we don't really support this package actively anymore. But ok adding to the to-do list to upgrade to 5.8 next week.


new rebus added

Associated Issue: #121
Summary of Changes

New rebus added

Pull Request Test Coverage Report for Build 487

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

Totals

Change from base Build 478:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


Fix the loongarch64 kernel ABI

The initial loongarch64 support code went in too early, even before the upstream kernel ABI has finalized, and was not adjusted since then. No one with enough knowledge of LoongArch was involved in the initial review, so we have been shipping broken LoongArch support, but luckily the rustc port is not merged yet so no real damage has been done.
Fix the following discrepancies:

There is no longer {g,s}etrlimit, only prlimit64.
There is no longer fstat and newfstatat, only statx.
MINSIGSTKSZ and SIGSTKSZ now have different values.
The binary sysctl syscall was removed from Linux long before the existence of upstream Linux/LoongArch port (5.5 vs 5.19) so even a wrapper does not make sense.

There might be more but these are the most obvious.
cc @xry111 @zhaixiaojuan

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

LGTM.

Thank you for the detailed explanation!

but luckily the rustc port is not merged yet so no real damage has been done.

I'm going to r+ in favor of the above, @bors r+

:pushpin: Commit 0abe537bbd7d447c35f4940d612ef1902eb29638 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 0abe537bbd7d447c35f4940d612ef1902eb29638 with merge 4d8960db97c700a9a8a808e5de20bca333bbd03d...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 4d8960db97c700a9a8a808e5de20bca333bbd03d to master...


Add/Remove layer to an existing Graph

Currently there is no way to draw or remove layers once a graph has been rendered without redrawing all layers. However, this feature would be useful for highlighting nodes/edges.
The following is an example interface:
const controller = new GraferController(canvas, { points, colors, layers });
controller.addLayer(newLayer);
controller.removeLayer(newLayer);

Issue resolved in following PR: https://github.com/uncharted-aske/grafer/pull/31

Issue resolved in following PR: https://github.com/uncharted-aske/grafer/pull/31


Cross-build for Scala.js

Solves #202.
Just made a draft PR to get some opinions. I still want to rearrange/reformat a few things so that there's as fewer changes possible.
I'm very confused as to why: sbt compile; test works. It seems to be running all the tests and they all pass. I don't understand why.

https://youforgotapercentagesignoracolon.com/
You need to change all the dependencies to use %%, not %%%. The results can be completely misleading otherwise (tooling issues...).

https://youforgotapercentagesignoracolon.com/
You need to change all the dependencies to use %%%, not %%. The results can be completely misleading otherwise (tooling issues...).

Thank you so much!
I think the first issue I'm getting would actually be solved by getting Typename to compile to Scala.js, which I've noticed you have done herehttps://github.com/tpolecat/typename/pull/9...

@jbotto94 this is now unblocked if you are motivated to keep working on it :)

Will see what I can do

I've just merged that typename update :smile:

Thank you for the early investigation! This landed in https://github.com/gemini-hlsw/gsp-graphql/pull/409.


Add disk boot order to API

Per discussion on mailing list, add ability to provide boot order to disks
https://groups.google.com/d/topic/kubevirt-dev/tBhwK8Y4UE0/discussion

Hello contributor, thanks for submitting a PR for this project!
I am the bot who triggers "standard-CI" builds for this project.
As a security measure, I will not run automated tests on PRs that are not from white-listed contributors.
In order to allow automated tests to run, please ask one of the project maintainers to review the code and then do one of the following:

Type ci test please on this PR to trigger automated tests for it.
Type ci add to whitelist on this PR to trigger automated tests for it and also add you to the contributor white-list so that your future PRs will be tested automatically.
If you are planning to contribute to more than one project, maybe it's better to ask them to add you to the project organization, so you'll be able to run tests for all the organization's projects.

The error is:

The command "if [[ -n "$(git status --porcelain)" ]] ; then echo "It seems like you need to run make. Please run it and commit the changes"; git status --porcelain; false; fi" exited with 1.

However, I'm not detecting any changes locally when I do this and I'm in sync with this branch, nothing to push.

It seems like the latest version of shfmt has issues with one of our shell script. #1032 should fix that. Will open an issue on the shfmt project.

shfmt is fixed: https://github.com/mvdan/sh/issues/241

@rmohr Sounds good. I'll look into how the functional tests work and try to get back within a few weeks at the latest.

ok to test

retest this please

Looks like the OpenShift/Cirros combo timed out. I'll try to reproduce it and see if there's a problem.

These tests are passing over here when I run a cluster with KUBEVIRT_PROVIDER=os-3.9.0, multiple times. The Cirros image does block awhile on cloud-init data. I'm not sure if it's just a lot slower in the test environment.

@mlsorensen also be aware of the problem with e1000 interface type - https://github.com/kubevirt/kubevirt/issues/936
3 minutest already to big timeout for cirros image, but can you increase it to 4 minutest and I will re-run tests?

Looks like one of these timed out on an unrelated storage test.

• [SLOW TEST:120.533 seconds]
Storage
/root/go/src/kubevirt.io/kubevirt/tests/storage_test.go:42
Starting a VM
/root/go/src/kubevirt.io/kubevirt/tests/storage_test.go:122
with Alpine PVC
/root/go/src/kubevirt.io/kubevirt/tests/storage_test.go:123
should be successfully started and stopped multiple times
/root/go/src/kubevirt.io/kubevirt/vendor/github.com/onsi/ginkgo/extensions/table/table.go:92
with CDRom PVC
/root/go/src/kubevirt.io/kubevirt/vendor/github.com/onsi/ginkgo/extensions/table/table_entry.go:46
panic: test timed out after 1h30m0s

The other failed multiple tests, but still doesn't like Cirros. I'm tempted to switch to the Fedora image, but it seems there are other issues with the OpenShift functional tests that still may block the checks.

retest this please

I've switched Cirros out for Fedora. It adds a third disk (userdata), which I guess makes the test even better. It's also a lot faster.

I've switched Cirros out for Fedora. It adds a third disk (userdata), which I guess makes the test even better. It's also a lot faster.

Oh, there was no user-data before on cirros? Well that explains why it was slow. I missed that. Now with #1047 merged (just merged a few hours ago), cirros should be the fastest one again (but it needs user-data too).

@mlsorensen could I ask you once more to try with cirros (+ user-data, so that it does not wait for the cloud-init via network).

Sure. There seemed to be a problem aside from userdata with Cirros, as even as it was it passed in everything but openshift. With the other changes, however, maybe things will be different.

Yay because it finally passed the new tests, boo because it failed others.

Especially the pvc test issue is well known. Looks fine for me now. Thx. Will have another look on Monday and merge then.

Travis is telling me I need to run 'make', but doing so results in no changes. I think it's actually this:

package golang.org/x/tools/cover: unrecognized import path "golang.org/x/tools/cover" (https fetch: Get https://golang.org/x/tools/cover?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout)

Travis is telling me I need to run 'make', but doing so results in no changes. I think it's actually this:

package golang.org/x/tools/cover: unrecognized import path "golang.org/x/tools/cover" (https fetch: Get https://golang.org/x/tools/cover?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout)

Yes seems to be a transient error there.

Ok, the failing tests are soley PVC related and is worked on separately to stabilize them.


Bluetooth: GATT: Fix attribute read return parameter type

The read() callback of attributes returns ssize_t and not size_t. Fix
this, which also fixes a Coverity warning.
Fixes Coverity CID 197457
Fixes #14958
Signed-off-by: Johan Hedberg<EMAIL_ADDRESS>

Codecov Report

Merging #14959 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14959      +/-   ##
==========================================
- Coverage   52.93%   52.93%   -0.01%
==========================================
  Files         309      309
  Lines       45251    45254       +3
  Branches    10447    10448       +1
==========================================
  Hits        23953    23953
- Misses      16533    16535       +2
- Partials     4765     4766       +1

Impacted Files
Coverage Δ

subsys/bluetooth/host/gatt.c
25.24% <ø> (-0.07%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf91f8c...947ae2d. Read the comment docs.


Boolean attributes

Hi there,
I had an input with name="name" as an attribute. Cheerio was squashing this to just name. It should only do this with a set of Boolean attributes - I've put in a test for this and put in a whitelist of the attributes.
Thanks,
Jos

Thanks!

Oops, I messed up with git here. The tests seem to be failing. I'll need to revert for now.

Cool beans. When is the next version bump planned?

Hey thanks for this. The next bump will be as soon as I finish translating from CS => JS. (in a readable way, not cmd + B)


Params callback functions

Perhaps it's a good idea to allow callback functions or at least regexes to validate content passed through the parameters div input controls or for the ones passed by URL. This will prevent parse errors in many cases. Right now javascript errors are thrown, but it's better to allow people to define friendly messages, perhaps even things like "The wall thickness cannot be negative".
{
  name: 'shape',
  type: 'choice',
  values: ["TRI", "SQU", "CIR"],               // these are the values that will be supplied to your script
  captions: ["Triangle", "Square", "Circle"],  // optional, these values are shown in the listbox
                                               // if omitted, the items in the 'values' array are used
  caption: 'Shape:',                           // optional, displayed left of the input field
  initial: "SQU",                              // optional, default selected value
                                               // if omitted, the first item is selected by default
                                               // NOTE: parameter "default" is deprecated
  validate: shapeValidate
}

An example to clarify:
function validate(parameterDefinitions,paramName,paramValue){
if(paramValue == "circular") && parameterDefinitions['height'].value > 10)
  status("The "+paramValue+" shape can only be used when the height is more than 10.");
}

This issue was opened in 2014, had some commits but no recent activity for past 2 years.
And it's not a bug.
I suggest we close it as stale.

Let’s let @ErikDeBruijn decide to close. There’s no solution yet.

I don't have a strong need for this anymore but if my request is useful to others we can keep it alive.
Im fine with focus on other, more recent feature requests, though! Perhaps that is time better spent.


SSL timeout

Hi there,
I attached all necessary information. Please have a look to it.
Best regards
Ruben
SSL_timeout.docx

Here is the definition of systemd:
[Unit]
Description=This script makes sure that app.py is started via systemd. This file is located under /etc/systemd/system
After=network.target
[Service]
Type=simple
EnvironmentFile=/etc/profile.local.systemd
WorkingDirectory=/opt/obl-db-sql/
ExecStart=/opt/obl-db-sql/venv/bin/uwsgi --ini /opt/obl-db-sql/configuration/uwsgi.ini
Restart=on-failure
[Install]
WantedBy=multi-user.target

Hi, this issue tracker is for issues with the codebase behind https://pypi.org itself, not the projects hosted on PyPI.
You should report this issue to the tracker for the project in question instead.

Sorry, for that.


feat: force version change

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Identify halted nodes/networks

Idea: If there are no blocks coming in, prompt the user to either try another node or switch to a mocked connection? This would be a more consistent flow and less overhead for us.

Done in #959


Fix missing license.txt files

The check-license.txt profile is failing due to LICENCE vs LICENSE ambiquity. In some modules the LICENSE is missing at all.

Any idea why we don't use the maven-rat-plugin exclusively?

No idea. The profile check-license.txt is inherited from the dkpro-parent (and was failing - I assumed it might be active during a release procedere, so I wanted to fix it)

ok, I'll have a look where this comes from. in other projects, we needed to mix rat and the license checker plugin because of a mix of GPL and ASL code (in DKPro Core) - but this should I think not apply to JWPL.

There are some classes in JWPL, which are not ASLv2 and have a rather individual license (for example: ListFilter)

If they are ASL-compatible, they should either be recognized by rat or can be excluded from the check and manually recorded in LICENSE / NOTICE files. If they are not compatible, they should probably be removed/replaced.

They are already properly recorded in the related files and should be compatible (are around for ages now).

Probably, but me-from-ages-ago might also have missed them while vetting...

The rat checks are active by default, so other files should be covered as well. I will do an additional check if there are classes not covered by notice files.


Can't get it to work with webdriver.io

I am trying to make your mock work with webdriver.io, here is my minimal test case:
var dragMock = require('drag-mock');
var webdriverio = require('webdriverio');
var options = {
    desiredCapabilities: {
        browserName: 'firefox'
    }
};

var webdriver = webdriverio.remote(options).init();

// set up webdriver.dragStart(), webdriver.dragOver(), webdriver.dragLeave() and webdriver.drop()
dragMock.extendWebdriver(webdriver);

// load the drag-mock library into the browser context
dragMock.loadLibrary(webdriver);

webdriver
    .url('http://html5demos.com/drag')
    .waitForVisible('#one', 2000)
    .dragStart('#one')
    .delay(250)
    .drop('#bin', function(error) {
        if (error) {
          console.error(error);
        }
    });

I get an error as the async function doesn't return anything. Any pointers?

Hey! I cannot give you a solution out-of-the-box. But maybe you can provide the exact error?


Enhance multi-account support for efficient follower retrieval

Current Behavior
Instaloader currently supports retrieving followers from a single account. When attempting to use multiple accounts to bypass rate limits, the follower retrieval process restarts from the beginning for each account, resulting in duplicate data.
Desired Behavior
I would like to implement a more efficient multi-account follower retrieval system:

Use multiple Instagram accounts (e.g., 3) to retrieve followers.
Each account retrieves a set number of followers (e.g., 200) before switching to the next account.
The retrieval process should continue from where the previous account left off, avoiding duplicates.
Implement a waiting period (e.g., 10 minutes) after all accounts have been used to respect Instagram's rate limits.
Optionally, use residential proxies for each account to avoid detection.

Expected Outcome
This implementation would allow retrieving approximately 600 unique followers per hour (200 per account) while respecting Instagram's rate limits and reducing the risk of detection.
Questions

Is it possible to modify Instaloader to support this multi-account, continuation-based retrieval process?
How can we ensure that the follower retrieval continues from where the previous account left off, rather than restarting from the beginning?
Are there any existing features or configurations in Instaloader that could be leveraged or extended to achieve this behavior?

Additional Context
This enhancement would significantly improve the efficiency of follower retrieval for users with multiple Instagram accounts, while still adhering to Instagram's usage policies and rate limits.

The suggested feature is out of scope for Instaloader. However, Instaloader is designed to be easily used in Python scripts for cases like this. You can write a custom script to handle multi-account retrieval, manage continuation logic, and leverage Instaloader's built-in waiting period management to respect Instagram's rate limits. This approach provides the flexibility you need without altering Instaloader's core functionality.

Thank you for your response first of all. Although I tried, I could not achieve such results. can I ask you what are the attributes/methods of instaloader which would allow you to not restart the follower counter from zero to recover in your opinion please?
concrete example: the script must extract the followers of an account with at least 10k followers. the script targets an account xxx which has 1000k followers, this with a first account aaaa. xxx followers journey:

if follower of xxx has +10k followers then we keep otherwise next
we sleep if we reach the limit for this example

lLet's imagine that the scrip starts to sleep when it has covered 200 followers of xxx. how to use a bbb account that starts at 201 please?


Fix missing required atribute "on_delete" in FilerFolderField.

Parameter on_delete in __init__ is mandatory in django>=1.9.

Codecov Report

Merging #1194 into master will decrease coverage by 0.00%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #1194      +/-   ##
==========================================
- Coverage   71.34%   71.33%   -0.01%
==========================================
  Files          63       63
  Lines        3144     3147       +3
  Branches      441      442       +1
==========================================
+ Hits         2243     2245       +2
  Misses        747      747
- Partials      154      155       +1

Impacted Files
Coverage Δ

filer/fields/folder.py
57.60% <66.66%> (+0.30%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d45a9a2...8f9150b. Read the comment docs.


Server's analysis.getNavigation request is not implemented

The specification defines the analysis.getNavigation request, but the current implementation of server does not implement it. This is needed in order to scale to large files.

Started
https://codereview.chromium.org/1232683003

https://github.com/dart-lang/sdk/commit/d7c2f4550b64a460acc9785eba14a16a66a44707


SyntaxError: Unexpected token 'export'

Error while trying to start dev server:
failed to load config from /Users/eden_lane/Projects/work/create-rmr-app/src/templates/init/vite.config.js
error when starting dev server:
/Users/eden_lane/Projects/work/create-rmr-app/src/templates/init/node_modules/vite-plugin-svgr-component/lib/index.js:1
export { svgrComponent, svgrComponent as default } from './src';
^^^^^^

My config is:
import {defineConfig} from "vite";
import VitePluginLinaria from 'vite-plugin-linaria';
import SvgrComponentPlugin from 'vite-plugin-svgr-component';

export default defineConfig({
  root: './src',
  plugins: [
    VitePluginLinaria(),
    SvgrComponentPlugin(),
  ]
})

Hi! Thanks for the issue!. Sorry the delayed response. I have publish a new version with this issue fixed and for now the package is stable. I have added a test-app folder with a new vite app to ensure all is working fine including typings.
Thanks again!

This was solved in the following commit #72160c15763fc3a53d0a5f09441ec14cb8e6a080


Proposed Webview API - Documentation

Testing #44466

The documentation is missing an example on how to receive messages in the webview like:
window.addEventListener("message", receiveMessage, false);
The dispose() docs refer to a 'onDispose' function. Did you mean 'onDidDispose'?

First cut of documentation is here: https://github.com/Microsoft/vscode-docs/commit/d489410d156f058b560918e146ff951fd93f152b
Note that this is more tutorial documentation, not reference documentation. The reference documentation is generated from the API itself


evaluation metric

Hello, the F-score in your paper refers to  MaxF like KITTI benchmark, is it right?

Hi, gongshichina. It is not.

Then what is the thresh for your F-score?

0.5


Unable to reproduce amazing results using the inference.py script

Hi Author @pgt4861,
Thanks for your work. I am a big fan of your paper and was really impressed by how well it uses the trimap information using prior tokens.
I was experimenting with the code but was unable to reproduce the results in terms of quality. Below are some of the examples that I ran.
In each row, from left to right, we have the original image, the trimap, the alpha mask predicted by matteformer and the final matted image.
As you can see, the output is not as good as the model can perform. Specifically in the unknown regions of the trimap, like around the fingers of the boy, the feet of the horse and the neck of the table lamp.
Can you please guide me how to improve these results?
P.S: I am generating these trimaps automatically from a segmentation mask using the Euclidean distance transform.

Hey, @gohar-malik i can help you with this contact me at<EMAIL_ADDRESS>

Hi @gohar-malik
Thank you for you interest on our work.
I think there are some ways to improve the results on real-world applications.
First, there are some domain gaps between real world image and composited images (train data). So if the model is so overfitted to the composited (synthetic) data, it may work not well on the real-world images.
It seems that some augmentation method can be helpful to deal with the problem.
Second, the number of foreground objects in the train data is not enough to cover the diversity of real world objects.
So, I recommend you to collect foregrounds (with good ground truth) as many as possible for your target problem.
Thank you


Feature Request: Error Bounds + Linear Error Propogation

It would be nice to have direct support for encoding error bounds and error propagation.
Without dimensional, I've used http://rosettacode.org/wiki/Numeric_error_propagation#Haskell, but that doesn't seem to directly transfer

All dimensional requires for representing the value is an instance of Num (or Fractional or “greater” depending on what you need to do) so you should be able to use the it with the Error a type from rosettacode.org. Here is an example (not tested):
distance = Error 10 0.1 *~ meter
time = Error 2 0.001 *~ second
velocity = distance / time
raw_vel = velocity /~ (meter / second)

Where raw_vel would be of e.g. type Error Double (perhaps a type annotation is required to fix the Double). If this doesn't work for you or if you had something different in mind please post a follow-up.

@bjornbm
Ah, ok. That ended up working just fine, thanks.


SRT ingest fails when using OBS Studio with encoder (NVDIA NVENC H.264)

Description(描述)

SRS fails to handle SRT connection when stream originates from OBS Studio and the encoder is set to NVDIA NVENC H.264. Regular x264 encoder in OBS works fine. This did not become an issue until ~ OBS Studio 27.2.1

SRS Version(版本): 4.0.261(Leo)

SRS Log(日志):

[2022-09-06 16:33:28.797][Error][23765][d1u11054][11] send media data error:code=3043 : write frame : drop sps/pps
thread [23765][d1u11054]: on_ts_video() [./src/srt/srt_to_rtmp.cpp:581][errno=11]
thread [23765][d1u11054]: write_h264_ipb_frame() [./src/srt/srt_to_rtmp.cpp:403][errno=11](Resource temporarily unavailable)
[2022-09-06 16:33:28.797][Error][23765][d1u11054][11] send media data error:code=3041 : demux annexb : annexb start code
thread [23765][d1u11054]: on_ts_video() [./src/srt/srt_to_rtmp.cpp:517][errno=11]
thread [23765][d1u11054]: annexb_demux() [src/protocol/srs_raw_avc.cpp:39][errno=11](Resource temporarily unavailable)

SRS Config(配置):

Default SRT config

obs-studio dev here; i wrote the srt support. i can confirm the issue which occurs with your transmuxer to rtmp. We don't observe such an issue with competing servers like Wowza or Nimble Server.

Pinpointed the issue: in obs-studio, the nvenc encoder is set to have AUD (access unit delimiter), see : here
It seems the srt_to_rtmp.cpp transmuxer doesn't know how to deal with them.
It is true that per spec it is not mandatory to have NALU AUDs. But your transmuxer should be able to parse them correctly.
Currently your code for dealing with AUD is:
https://github.com/ossrs/srs/blob/4.0release/trunk/src/srt/srt_to_rtmp.cpp#L525-L529
I'm not sure why these lines of code to ignore the AUD don't work though.

https://github.com/ossrs/srs/blob/4.0release/trunk/src/srt/srt_to_rtmp.cpp#L525-L529

Yes, you are right, we must not ignore AUD in the h264 stream. We will fix it later.
Had you test remove the code https://github.com/ossrs/srs/blob/4.0release/trunk/src/srt/srt_to_rtmp.cpp#L525-L529, and can ingest SRT success?


Use $comment to add comments to the jsonschema schema

https://json-schema.org/understanding-json-schema/reference/generic.html

blocked by #144

Block is gone (#144 closed), and in fact $comment has been used already in #175.
Can this be closed?

Yep, closing


Update to lstchain v050

Add lstchain_standard_config_v050.json
Adapt arguments key to v0.5.0
Update reorganize_dl1hiperta_to_dl1lstchain.py
Manages source environment in r0_to_dl1 stage. Resolves #32

Setting the --smart False argument in the merging stage should resolve #34 too. Thanks @vuillaut for the tip.


Turn draggable, scrollwheel options to false.

Hi, I want to be able to turn off the zoomonscroll option for the map, and am unable to do so, have tried a few different ways to go about it, but no luck
Would like some help trying to solve this.

Sorry, I figured it out, had to put it as options.
                <GoogleMap options={{ scrollwheel: false}} >

Glad it's resolved! We're also looking for maintainers. Involve in #266 to help strengthen our community!


Stampy chatbot - Double-clicking on "Search" on the chatbot page results in "No response from server"

A short debounce between "Search" and "Cancel" could be beneficial.

won't do, as this is getting replaced with the new UI


Fixes to Columns indexing

There were two bugs in the __getitem__ method:

Indexing a value dimension by name would return that column as an array.

Boolean slicing was not working across backends.

I've fixed these issues and added some unit tests.

Ok, good to make a PR to make sure the tests get run. If I understand it correctly, this PR will address issue #322.

Ok, the tests are passing and this is definitely the behavior we wanted. I'll merge it now.


Settings script detection takes scripting language providers into account

This PR allows Gradle to detect settings scripts according to available scripting languages.
It builds upon the ScriptingLanguage SPI introduced by #1908.
See https://github.com/gradle/gradle-script-kotlin/issues/56
Gradle Core Team Checklist

[ ] Verify test coverage and CI build status https://builds.gradle.org/viewQueued.html?itemId=2894994
[x] Verify documentation including proper use of @since and @Incubating annotations for all public APIs

Thanks @bamboo
Addressed your comments, could you have another look?


OAuth Login null reference exception

When using the OAuth login component in the Composer, i get an exception with the following message:

"Object reference not set to an instance of an object."

I'm using the OAuth component exaclty as i was using it with the previous release and it was working perfectly.
Version

Browser

[X] Chrome
[ ] Safari
[ ] Firefox
[X] Edge

OS

[ ] macOS
[X] Windows
[ ] Ubuntu

FYI. I had the same issue and was about to report. Thanks for reporting.

I just tried to switch to the master branch to check if it was working but i get the same error, but i think on a different component. The error i obtain is exactly at the beginning of the Greeting dialog. Here's a screenshot of what my project looks like.

FYI. I think OAuth works now.

@fathorMB Did you have a chance to try again? I think the OAuth issue has been addressed.
Here is a doc for your reference.

@zxyanliu OAuth is now working. Thanks!

Closing because the issue is addressed.


Remove kickstart action?

I propose that kickstart be removed

It doesn't work
It isn't necessary (bootstrap works fine; kickstart requires the entire blockchain to be reindexed after kickstart is used)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/38657616-remove-kickstart-action?utm_campaign=plugin&utm_content=tracker%2F494307&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F494307&utm_medium=issues&utm_source=github).

Now it works!


Cant save the Opcode

I've saved the code but it will automatically switch back to the previous version the next time I launch ff14, why is this? (JP server)

@SindragosaE
Go to configure -> Check for Opcode updates on startup


update(oci/CI): fetch tags using checkout action

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:
Fetch git tags using the checkout action instead of running git on the runner.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alacuku, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

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

LGTM label has been added.  Git tree hash: 46a8750bf7ae13d1405120f6153a29745ed6c1b9


Add separate ws.listen per environment

The connect utility is platform agnostic 💪
... but the current listen function makes use of WebSocketPair and ResponseInit.webSocket? which are both specific to Cloudflare's runtime.
Responding w/ a WebSocket upgrade is going to be different per-platform, so need to find an import/module naming convention that works here (see #132)

Perhaps leave connect as is within worktop/ws since it will work everywhere.
Then add worktop/cfw.ws, worktop/deno.ws, etc – each with their own listen export.
^ These two allow plain code to resolve/build as is.
When using worktop.build (#131), users could rely on a magical worktop.build/<env> module which still includes (shared) TypeScript definitions but aliases these to the <env>-injected module names instead.
This would not resolve successfully for users who aren't using worktop.build – but again, you wouldn't have a runtime import to worktop.build unless you had it installed, so...
import { Router } from 'worktop';
import { start } from 'worktop.build/<env>';
import * as ws from 'worktop.build/<env>.ws';

const API = new Router;

API.add('GET', '/chat', ws.listen(function (req, context, socket) {
  // ...
}));

start(API.run);

As mentioned in #131, the default --env is "cfw", so the above by default would be the equivalent of this:
import { Router } from 'worktop';
import { start } from 'worktop.build/cfw'; // < INJECTED
import * as ws from 'worktop.build/cfw.ws'; // < INJECTED

// ...

export default start(API.run); // < MUTATED (must already happen for cfw)

And if building for Deno (via $ worktop build --env deno) then this would be the equivalent source that worktop.build compiles:
import { Router } from 'worktop';
import { start } from 'worktop.build/deno'; // < INJECTED
import * as ws from 'worktop.build/deno.ws'; // < INJECTED

// ...

await start(API.run); // < MUTATED (must already happen for deno)

Closed via 504328d,91d8e73
Deferring to #131 to track the worktop.build aliasing


feat(protocol-kit): add short name for IoTeX Network Testnet

What it solves
Adds short name for IoTeX Network Testnet:
https://github.com/ethereum-lists/chains/blob/master/_data/chains/eip155-4690.json

To add shortnames the chain assets should be available at safe-deployments first.

https://github.com/safe-global/safe-deployments/pull/421


Tracking issue for transaction builder generation conversion to script functions

Now that we have script functions and they're supported in the generated script ABIs, we need to update the different transaction builder generators for each of the languages to support both the old (legacy) transaction scripts and the new script function transaction builders.
The following transactions builders need to be converted over:

[x] C++
[x] Go
[x] Java
[x] Rust
[ ] Typescript
[x] Python
[x] C#

cc @matbd

Note that additional tests have also been added in each of the converted languages to test the new script fun functionality


Opensource  : 2.01 Create  first-repository

Description 📜
Added a complete documentation on how to create your first repository.
Fixes #5944

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Open Source

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@Aryamanz29 renamed the file. Kindly review it😊


Staging release

Pull Request Test Coverage Report for Build 1159

0 of 0   changed or added relevant lines in 0 files are covered.
22 unchanged lines in 7 files lost coverage.
Overall coverage increased (+1.04%) to 94.308%

Files with Coverage Reduction
New Missed Lines
%

../controllers/investigations/correspondence_controller.rb
1
98.18%

../models/investigation.rb
1
98.7%

../controllers/investigations/flow_controller.rb
1
97.3%

../controllers/concerns/file_concern.rb
2
95.24%

../controllers/products_controller.rb
2
80.39%

../controllers/investigations_controller.rb
6
81.36%

../helpers/documents_helper.rb
9
72.5%

Totals

Change from base Build 1104:
1.04%

Covered Lines:
2336

Relevant Lines:
2477

💛  - Coveralls


Semantic

Semantic has to be fixed for xindexview and xview like it has been done for xbroadcast and xfunction.

Reopened, #116 was not finished.

But if fixes some. I have the fix for #115 btw.


Fix sign-in issue due to calling getToken in main thread

Issue #, if available:
https://github.com/aws-amplify/aws-sdk-android/issues/956
Related pull request: https://github.com/awslabs/aws-sdk-android-samples/pull/321
Description of changes:
Move isSignedIn() to AsyncTask to avoid NetworkOnMainThreadException and crash.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @zhiyua-git ,
Thank you for your contribution. I will close #321 in favor of this while we review the changes.

@raphkim Still getting issue #321 and the app does not work because of it. Can this be merged asap please?

Looks like this adds a new call to getTokens? For the signIn call, if you are getting a NetworkOnMainThreadException, we should fix that from within the SDK, not in the sample app. The SDK should not throw NetworkOnMainThreadException from any asynchronous invocations.


Update docsy to v0.5.1

Fixes #18
Signed-off-by: Andrea Frittoli<EMAIL_ADDRESS>

The preview is broken, it looks like fonts-awesome is missing

I'm trying out something different: https://www.docsy.dev/docs/updating/convert-site-to-module/

@e-backmark-ericsson @tdcox the update worked smoothly when switching from using a git submodule to a Hugo module.
This should simplify future updates too.


Change muted members in the configuration to an SQL table

Currently muted members are stored under the security section in the config file, this is bloat, and a misuse of the config file -- it needs to be in its own SQL table.

I used random.org and got this one. See PR

Closed by 565cf2523d75784e1212ad1d6e998f08aba81cc1


[best practises] express-jwt made 100% with Falcor [without app.post('/login', authenticate, (req, res) ]

Hello,
I am working on an open source starter kit with the login/registration made in React+Redux+Falcor+Express+MongoDB ...
I was looking around for some, but I really didn't find any good example so I want to make one.
 Please let me know.
I want to attach best practises into my starter kit so this is why I have that question.
FRONTEND code:
const falcor = require('falcor');
const FalcorDataSource = require('falcor-http-datasource');
const $ref = falcor.Model.ref;
const $atom = falcor.Model.atom;

let headers;

if(localStorage.token) {
	headers = {
      headers: {
        'Authorization': 'Bearer' + localStorage.token
      }
    };
}

// const model = new FalcorDataSource('/model.json');

const model = new falcor.Model({
  source: new FalcorDataSource('/model.json', headers)
});

export default model;

Backend Problem
The lack of my experience in Falcor shows on the backend.
I need expose somehow 2 different models on backend so I can have ONE model with some routes auth-required while other non-auth-required:
https://gist.github.com/przeor/a66594dfbfd25ef120dc (server.js file - too long to post it here)
Question:
How in one model in Falcor, to expose some endpoints as non-secured (not required valid auth token) like traditional:
app.post('/login', authenticate, (req, res) => {
  let token = jwt.sign({
    username: mockedUser.username
  }, jwtSecret);
  res.send({
    token: token,
    user: mockedUser
  });
});

and other to expose as they require authentication like:
app.get('/api/me', (req, res) => {
  res.send({ user: req.user, works: "yes" });
});

I've already tried to make this:
app.use('/api', expressJwt({secret: jwtSecret}).unless({
  path: [
    // here goes unauthenticated paths from Falcor like:
    'http://localhost:3000/model.json?paths=%5B%5B%22articles%22%2C%22length%22%5D%5D&method=get'
  ]
}));

BUT that above example with using .unless paths doesn't work (don't affect the route as unauthenticated) :-(
How to approach the problem correctly with express-jwt?
A summary: I want to have one Falcor model with some paths that doesn't need valid AUTH Bearer (like  [["login"]] or [["articles","length"]] routes for example) while other paths does require them (like [["me"]] or [["articles", "add"]]) :-) ... I want to use 100% pure Falcor's model for login routes and all other that are secured as well. In the current approach I am using https://github.com/mzabriskie/axios on the front-end, but I want get rid off it after you will help me with best practise.
EDIT: P.S. Also I am wondering if maybe I am making any mistake or typo with app.use('/api', expressJwt({secret: jwtSecret}).unless ?
I'm looking forward to your hints.

It's probably best to clarify straightaway that there may not always be a distinction between routes that require authorization and routes that don't (though some applications may make due with this distinction). For example, a blog application may have a recentPostList path that returns the most recent posts. When the user is not authenticated, they are all public posts; when the user is authenticated, they may include drafts, for example.
Authentication properly takes place therefore not within Express, but within the Falcor router. Express could, using middleware, decrypt the JWT and resolve to a user (e.g. req.user), but allow the unauthenticated user by setting the same to null. Your route can either use the user or discard it, as needed.
As far as the creation of the JWT, you can either use a falcor route or an express route (I won't take a position here), and it doesn't change the above. Simply return the token where requested, add it to the header, and decrypt it on incoming requests if it was provided.
For a very early stage example of React+Express+Mongo+Falcor, take a look at StoryShop's frontend and backend.

@joshdmiller Is my understanding correct, that I shall:

ommit using express-jwt that works fine with standard express-routing
instead of using express-jwt, then I shall implement my own way of checking if a user has access to a certain api endpoint?

What I am missing in those two above steps?

Essentially yes. express-jwt is a very simple wrapper around the core JWT functionality, so the ask is quite small - a couple of lines of code.
The best way to think about it is that express gets you to model.json and the falcor router takes it from there, including authorization.
But again, a user doesn't have access to an endpoint, but to a resource attached to a model path. Think about a path like usersById[{keys:ids}]. There's no clean way to have express authorize that because falcor could batch two users in the same request. The user may have access to one and not the other, but both express and the falcor router receive them at the same time in the same request - and must return both path values back, one with data and one with an error. So the falcor route's methods must handle the authorization. It cannot (easily) be express middleware.
Note: this is based on my usage of falcor. If a member of the team has a different opinion,  I'd defer to them.

@joshdmiller interesting - thank you! That helps for my understanding a lot in the initial stage.
I am interested if anyone else has more to input into this conversation, maybe other ideas - if not, then we can close the thread :+1:

@joshdmiller thank you for your open source project! IT IS GREAT !
In the same time, I didn't find in your front-end app any login feature? Or I didn't spot it correctly in the codebase?
@joshdmiller could you take a look on that issue, maybe you will have a better idea what is best way to approach my problem on the Falcor's frontend here: https://github.com/Netflix/falcor/issues/762

You're welcome! There will be a lot more in the next few weeks as we approach beta. The frontend login stuff was not on master, but it is now. Here's a summary:
On successful logins we fire a Redux action, storing validated tokens in the Redux store (which is synced with local storage). That token is, when present, added to as a header to falcor HTTP requests.
On the server side, we validate the token if it was provided, setting a user prop on the request object à la PassportJS, et al. On falcor routes, we can use that object if present to determine access.
As I mentioned before, this is very much in flux right now and there are some holes and problems we're working to solve - as well as add more robust testing. This should serve more as inspiration in the short term. But we'd like the project to be helpful to others using this tech - we've certainly learned a lot building it.

@joshdmiller thank you for all your help. I am closing the issue as I believe all the help required has been provided. Thank you all again!


mkpr undefined sed s/10.10/10.12/

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create eslint.yml workflow

This PR adds a GitHub workflow that executes on a schedule and validates code quality using ESLint and reports its findings
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This change has uncovered larger amounts of eslint related improvements that need adjusting. Going to close this and create a dedicated issue to focus on eslint


Changed Server Capabilities are not preserved upon VenvSelect

Thanks for this nice plugin, just stumbled upon it by chance!
So I am using pyright and pylsp at the same time, and I use this snippet to disable pyright's hover capabilities, since I prefer pylsp's:
require("lspconfig").pyright.setup {
  on_attach = function(client, _) client.server_capabilities.hoverProvider = false end
}

However, it seems that the way this plugin sets up the LSPs, only the settings are carried over, but not the disabled server capabilities, meaning that as soon as I use VenvSelect, the hover capability is re-enabled again.
(I tried various versions of updating the lsp config via hooks or simply using local on_attach = vim.deepcopy(pyright.config.on_attach) to preserve the disabled hover, but could not make it work.)

You're welcome. :)
Interesting scenario for sure. I added some more debug output just now that prints how the pyright config looks like when you select a virtual env.
It should look something like this (at the end of output) if you type :messages after selecting a virtual env:
VenvSelect: Pyright settings:
VenvSelect:
{
  python = {
    analysis = {
      autoSearchPaths = true,
      diagnosticMode = "workspace",
      useLibraryCodeForTypes = true
    },
    pythonPath = "/home/cado/Code/Blocket/insight-and-analytics-extracts/zendesk/venv/bin/python"
  }
}

Can you print how your pyright lsp settings look like before VenvSelect overwrites them?

Do I have to do something to get the output?
I tried :mess before and after using :VenvSelect, if both cases nothing gets added from this plugin.
I just run the snippet above without passing anything to settings, which is maybe why no settings are shown? As far as I can tell, server_capabilities are not part of lsp settings? 🤔

Sorry, you need to add enable_debug_output = true to the plugin settings also. :)
I havent looked into the pyright LSP details either, I just did what was necessary to get the plugin to work when I first started writing it. But I was curious how your pyright settings look like after you run your snippet there, to maybe get some clues how it should look like to not overwrite the client.server_capabilities.hoverProvider setting.
Maybe its not the right way and I just need to read up more about the lsp and figure out how to do this. :)

I think you need to have the pyright LSP activated also for the important part to show up. So open a python file also, activate the venv with VenvSelect and then do the :messages command. :)

but that is exactly the output I get. I am:

enable_debug_output = true
in a python file
pyright active
no venv enabled
then run :VenvSelect

gets me exactly the output I posted above

yeah it's a "bug" in the plugin, i also have trouble there. the options @chrisgrieser is mentioning are not under settings instead they are config options like: .capabilities, .handlers, ... https://github.com/neovim/nvim-lspconfig/blob/a27356f1ef9c11e1f459cc96a3fcac5c265e72d6/doc/lspconfig.txt#L132
venv-select is only taking a deep copy of settings, but not the other configuration parameters, if that is even possible. i see there  are only three/four solutions:

maybe the on_attach can be modified by the hook without a new server setup and then call a server restart/reset if there is one
copy other (or complete) configure options as well, not only settings, if that is possible
allow the user to define their own config options in the lsp hook
or the user just replaces the complete hook with there own one, this might already be possible

Ok thank yoi @cmetz and @chrisgrieser , sounds like a bit more digging is required here.

venv-select is only taking a deep copy of settings, but not the other configuration parameters, if that is even possible and then starting a new lsp server with it's own then reduced configuration.

Yes this is correct. The plugin calls the pyright hook which makes a copy of settings here:
https://github.com/linux-cultist/venv-selector.nvim/blob/6841d78e4f88efbceca9b2546e6080a9914109c4/lua/venv-selector/hooks.lua#L22-L36
There are instructions in the README how to write your own hook replacing this one, which can be a start for exploring how to add support for multiple LSP's, if you want to give it a shot. I havent ventured very deep into that particular jungle myself at this point. :)

i replaced the hook with that code below and it works, but i don't know how that whole setup can handle different buffers with different venv. maybe there is a way to have multiple lspserver running for different workspaces, or just start new vim session.
--- @alias VenvChangedHook fun(venv_path: string, venv_python: string): nil
--- @type VenvChangedHook
function M.pyright_hook(_, venv_python)
  M.execute_for_client("pyright", function(pyright)
    local utils = require("venv-selector.utils")
    pyright.config.before_init = function(_, c)
      c.settings.python.pythonPath = venv_python
      utils.dbg("Pyright settings:")
      utils.dbg(c.settings)
    end
    vim.cmd(string.format("LspRestart %d", pyright.id))
  end)
end

maybe there is a better way to reload the server, but that seems not that easy
https://github.com/neovim/nvim-lspconfig/blob/a27356f1ef9c11e1f459cc96a3fcac5c265e72d6/plugin/lspconfig.lua#L89

Using the before_init function is almost instant and avoids restarting the entire LSP server. I also think LspRestart restarts all languages LSP servers in the editor.
It would be nice to keep using the before_init to not restart other lsp servers. Do both solutions above work for your usecases?

@linux-cultist LspRestart with an client id only restarts specific server. the current use of .setup(..) is also restarting the whole server only appling a before init function to manipulate the config. this might be usefull if you want to do some dynamic stuff e.g. when calling an restart and evaluate some dynamic code. but the code is setting only a fix venv value, so i'm pretty sure this can be done completely in the config.settings not using the init function.

I remember reading this quickly when I started writing this plugin a long time ago. It was my first plugin and I was learning lua at the same time, so I was just happy to get something working. :)

yeah you did really a great job. Lazvim uses it as default :)
but i just found out that for pyright this can even be done quicker. there is a command which allows you to change the pythonpath and then evokes an workspace update in the lsp server, so no restart of the server needed :)
--- @alias VenvChangedHook fun(venv_path: string, venv_python: string): nil
--- @type VenvChangedHook
function M.pyright_hook(_, venv_python)
  vim.cmd(string.format("PyrightSetPythonPath %s", venv_python))
end

https://github.com/neovim/nvim-lspconfig/blob/a27356f1ef9c11e1f459cc96a3fcac5c265e72d6/lua/lspconfig/server_configurations/pyright.lua#L21C16-L21C31

That super cool, I will try it out tomorrow right away. :) Thanks so much for helping out!

I really like the last solution here:
--- @alias VenvChangedHook fun(venv_path: string, venv_python: string): nil
--- @type VenvChangedHook
function M.pyright_hook(_, venv_python)
  M.execute_for_client("pyright", function(pyright)
    pyright.config.settings =
      vim.tbl_deep_extend("force", pyright.config.settings, { python = { pythonPath = venv_python } })
    pyright.notify("workspace/didChangeConfiguration", { settings = nil })
  end)
end

Seems to work equally well for me, and this also solves the issue for both of you?

@linux-cultist yeah this solves the issue for me, but maybe it would make sense to iterate over all matching pyright clients if there are more than one.
meanwhile i switched to configuring my pythonPath through an neoconf config file, as it is installed with lazyvim by default and also respects some other configs options like that from vscode and others.
maybe i'll try to write a venvselect hook that will configuring the .neoconf.json file for a specific workspace.
this is an example .neoconf.json
{
  "lspconfig": {
    "pyright": {
      "python.pythonPath": "venv/bin/python"
    }
  }
}

i also configured my dap resolve_path via an function which resolves the pythonpath from the pyright config
  {
    "mfussenegger/nvim-dap-python",
    init = function()
      require("dap-python").resolve_python = function()
        return vim.lsp.get_active_clients({ name = "pyright" })[1].config.settings.python.pythonPath
      end
    end,
  },

i just switched to nvim again couple of days ago and it's quite interesting to see different packages solving things a different way. but none of them is having a venvselect :)

okay, I am not sure, I can follow everything in the discussion, but the last snippet here:
function M.pyright_hook(_, venv_python)
  M.execute_for_client("pyright", function(pyright)
    pyright.config.settings =
      vim.tbl_deep_extend("force", pyright.config.settings, { python = { pythonPath = venv_python } })
    pyright.notify("workspace/didChangeConfiguration", { settings = nil })
  end)
end

seems to work for me!

Yeah I think it seems to work fine and it would make sense to replace the default hook with this one. Just wanted to make sure it doesnt do anything weird. Im not too read-up on the specifics on the LSP. @cmetz knows more than me for sure. :)

i just switched to nvim again couple of days ago and it's quite interesting to see different packages solving things a different way. but none of them is having a venvselect :)

Im still on it but after years of maintaining my own config, I actually switched to Lazyvim due to always having some weird small bugs in my own configs. Its fun to have my own config but its also kind of hard to understand why something is buggy now and then.

@linux-cultist i don't understand your commit it seems redundant for me handlig some lsp servers.
in the venv.lua (M.set_venv_and_system_paths) the pythonpath is set twice, once via an autocommand i don't understand why this is used.:
 M.set_pythonpath(venv_python)

and a second time via hook
  for _, hook in ipairs(config.settings.changed_venv_hooks) do
    hook(venv_path, venv_python)
  end

maybe the autocommand is used to override other plugins switching the pythonpath.

The original way (first thing you are showing above) is how VenvSelect was setting python interpreter before someone added hooks to the codebase.
But the hook doesn't modify the system path as I understand it, so it's still necessary to modify the path (putting the venv first in the path) for the venv to be activated in any terminal windows opened from inside neovim.
That was my original thinking anyway. Maybe it's time to have a look at that code and remove anything it's doing that is now redundant....

I removed the changed code now, so we can understand better what needs to be done.

ah that is fine, I'm in a same situation, maybe i can invest some time and create a pull request.

I would love if you do. I think you already know more than me about the LSP and having another person go through the code will be very helpful I think. Just be aware that some parts may look a bit out of place - there has been many people contributing (which I actually like, but can lead to one person not knowing the entire code base). :)

I created a PR that essentially reimplements the commit shown above. The only difference is that it loops over all pyright clients instead of just changing the first one.

the hook is fine, but i think the code got a little bit redundant on different places.

As @cmetz said, the commit was good and solves the issue. The fact that a new autocommand is created every time you set a different python environment is a different issue. I think a new issue should be opened for that one.
The code change solves this issue, and does not make the other problem better or worse, they are not related. So I would argue to solve this bug first, and then to look at the duplication after.

I can also make this issue if you want. 👍

I love any support you can offer. :)


⚠ Bofrost.lu has degraded performance

In 3ab0eae, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1260 ms

Resolved: Bofrost.lu performance has improved in a09932f after 13 minutes.


Codebase

Alright so I had to migrate to SDL3 and I had to update my games whole codebase that uses SDL2
Since there were major changes what header file is SDL_image.h ? I need to use it in mycode

You want this: https://github.com/libsdl-org/SDL_image


[P2P] LibP2P - Integrate with Pocket Core

Objective
Use the results of https://github.com/pokt-network/pocket-network-protocol/issues/17 to integrate LibP2P into the p2p module.
Origin Document(s)

https://github.com/pokt-network/pocket-network-protocol/issues/17
LibP2P Guide

Goals

Adding LibP2P as a dependency and "hooking into" as many places as possible in the V1 implementation

Deliverables

[ ] Add LibP2P as a dependency
[ ] Use the transport channel provided by LipP2P
[ ] Add pocket specific configurations that correlate (1:1 or 1:many) to LibP2P configurations
[ ] Optional if low effort: Enable TLS for cross-node communication

Non-Goals / Deliverables

Implementing peer churn or discovery
Exhaustively enabling all the features we will need from LibP2P

General issue checklist

[ ] Update the appropriate CHANGELOG
[ ] Update the README
[ ] Update the source code tree explanation
[ ] Update any relevant global documentation & references
[ ] Document small issues / TODOs along the way

Creators: @jessicadaugherty @Olshansk

Per our team discussion, I believe the scope of this can be summarized to, at a very high level, to bullet point 3 from the research SPIKE:

Embed LibP2P host in Pocket host. Do not enable.

Synched up @bryanchriswhite on this offline and wanted to capture the potential action items for a next step.
Idea: LibP2P could be treated as another "connection type" similar to what we implemented in #268.
Next steps:

Add LibP2P as a connection type (alongside TCP & noop)
Create a new ticket titled "Consolidate LibP2P & Pocket Interface"
"Hack" things together (in terms of data types) to replace the transport layer in P2P
Add comments similar to CLEANUP(#XXX): Cleanup these interfaces wherever we kind of hacked things together

Other things discussed:

Lack of connection pooling in the current version was not an intentional "thought-through" deicsion - we just got things working using the standard Golang libraries and I did not document it explicitly.

What is the difference between hacking things together in terms of data types and being deliberate and targeting them one by one (wherever possible) in a more controlled manner?

@bryanchriswhite did research on all the data types but it's still a big undertaking.
I thought that if we just wire it together (with TODOs and a separate connection type) everywhere, we'd be closer to this approach and clean it up after:

Re: connection pooling, I think it's necessary for the long...

We're on the same page. It seems like it's the default and Bryan mentioned we have to make an explicit effort to turn it off. If it's less effort to keep it (during initial integration), let's keep it.

@bryanchriswhite This is an integration where I haven't jumped in to get my hands dirty so I'm speaking out of my depth. However, if you think it'd be helpful for me to get a foundation PR up, lmk.

@Olshansk sorry for the delay in responding, I've been heads-down in preparation for being able to participate meaningfully. 😅
In my view, it's looking like the path of least resistance is to implement a new modules.Module and types.Network. I also figured that it would then be more straightforward to add a boolean config flag rather that coupling to the connection type.
I still fill in the PR description with more details when I'm back at my desk.

@bryanchriswhite Does autoscaling (with staking validators) using k8s LocalNet + LibP2P work? https://github.com/pokt-network/pocket/tree/main/build/localnet#scaling-actors


Transaction/receipts data returned by Anvil is inconsistent with Ethereum spec

Component
Anvil
Have you ensured that all of these are up to date?

[ ] Foundry
[ ] Foundryup

What version of Foundry are you on?
0.1.0 (a43313a 2022-11-30T17:15:08.718900657Z)
What command(s) is the bug in?
anvil fork
Operating System
Linux
Describe the bug
Overview
Optimism derives L2 blocks from log events and calldata submitted to L1. To prevent attacks from a malicious RPC provider, our system performs a series of sanity checks as well as a receiptsRoot check to verify that log events are valid and none are missing. While testing our op-node against a fork of L1 Goerli using Anvil, we observed these checks failing in the below ways.
Mismatch between the logIndex field and actual log index
The logIndex field sometimes doesn't match the actual index the log appears in the RPC. Here's cast output from one of the affected receipts:
➜  ~ cast receipt 0x2676dfb732072e733fe40dc6bb466808ef4ac3ddf71d5d32502502241449bbee --rpc-url <rpc-url>

blockHash               0xd28140f6b523d36cd7fba9594a7b58cb354aeafae1ce9be9cb992f9b0542f98c
blockNumber             8083603
contractAddress
cumulativeGasUsed       178140
effectiveGasPrice<PHONE_NUMBER>
gasUsed                 86860
logs                    [{"address":"0xc2beaa06e18ab001fcf4318030c31c334c71d500","topics":["0xa7aaf2512769da4e444e3de247be2564225c2e7a8f74cfe528e46e17d24868e2","0xc2c2ee83a423d279138ab45d7de00bc85c70bc133ab2af5b4e53f885f5bef170","0x0000000000000000000000000000000000000000000000000000000000000001","0x00000000000000000000000000000000000000000000000000000000638ed10c"],"data":"0x0000000000000000000000000000000000000000000000000000000000312d1c","blockHash":"0xd28140f6b523d36cd7fba9594a7b58cb354aeafae1ce9be9cb992f9b0542f98c","blockNumber":"0x7b5893","transactionHash":"0x2676dfb732072e733fe40dc6bb466808ef4ac3ddf71d5d32502502241449bbee","transactionIndex":"0x1","logIndex":"0x1","transactionLogIndex":"0x0","removed":false}]
logsBloom               0x00010000001000000000000001000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000040000000000000000000000000000000000000000000000040000000000000000000000008001000000000000000000000000000000000000000000000000000000000000000000000000200000000000000000000000000010000000000004000000000900000000000200000000000000000000000000000000000000000000000000000000000000000000000000000010000000000000000000000000000040000000000000000000000000000000000000000000000000000000000000000000
root
status                  1
transactionHash         0x2676dfb732072e733fe40dc6bb466808ef4ac3ddf71d5d32502502241449bbee
transactionIndex        1
type

You'll see that the logIndex in the first log above is 0x1, when it should be 0x0.
Computed receipts root does not match returned receiptsRoot
When we compute a receipt root for a given block using all the the transaction receipts within it, the computed receipts root does not match the receiptsRoot within the block.
Unfortunately, I don't know exactly why this is. We've verified that our receipts root computation works by running the same code against Goerli with no issue. I wish I had more concrete information about the root cause here, but I'm fairly confident that you can reproduce this yourself by performing the same computation we do on an Anvil block with receipts in it. Here's the code we're using to compute the root, in case it helps: https://github.com/ethereum-optimism/optimism/blob/develop/op-node/sources/receipts.go#L65-L72.
Transaction hashes are duplicated
We're also seeing cases where transaction hashes appear in multiple blocks. Relevant cast output is below:
➜  ~ cast block 8083936 --rpc-url <rpc url>

baseFeePerGas        0
difficulty           0
extraData            0x
gasLimit             30000000
gasUsed              0
hash                 0xa0de39cbb463b35e5cd9b29a212ff6bdb0734c6f86e7ab04188a796f8c27aa11
logsBloom            0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
miner                0x0000000000000000000000000000000000000000
mixHash              0x0000000000000000000000000000000000000000000000000000000000000000
nonce                0x0000000000000000
number               8083936
parentHash           0xc85e0ecde7349938f6daea79d6834dc278fe05e305dca5fc94a80a9d923dfe16
receiptsRoot         0xa1d38b83ec14e13227c918362bd5ad07a7ea3ea830853ef3b181dfad043564d0
sealFields           [
	0x0000000000000000000000000000000000000000000000000000000000000000
	0x0000000000000000
]
sha3Uncles           0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347
size                 633
stateRoot            0x0000000000000000000000000000000000000000000000000000000000000000
timestamp<PHONE_NUMBER>
totalDifficulty      10790000
transactions:        [
	0xc9f5537c71d2e8aaf1a7f5884acfbe1b6df77d773bb7b4569843f7fe7601d3d4
]
➜  ~ cast block 8083956 --rpc-url <rpc url>

baseFeePerGas        0
difficulty           0
extraData            0x
gasLimit             30000000
gasUsed              0
hash                 0xc6449f40c413ba7bebdcba3f620fecfe958a6c109a21aeb7ff522f4149fabc5b
logsBloom            0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
miner                0x0000000000000000000000000000000000000000
mixHash              0x0000000000000000000000000000000000000000000000000000000000000000
nonce                0x0000000000000000
number               8083956
parentHash           0x9198cd3b82e3892626f4a366baa05afd6d3c476598d7e96d0484ca7bcb69b235
receiptsRoot         0xa1d38b83ec14e13227c918362bd5ad07a7ea3ea830853ef3b181dfad043564d0
sealFields           [
	0x0000000000000000000000000000000000000000000000000000000000000000
	0x0000000000000000
]
sha3Uncles           0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347
size                 633
stateRoot            0x0000000000000000000000000000000000000000000000000000000000000000
timestamp<PHONE_NUMBER>
totalDifficulty      10790000
transactions:        [
	0xc9f5537c71d2e8aaf1a7f5884acfbe1b6df77d773bb7b4569843f7fe7601d3d4
]

We're able to work around these issues for now, but I thought I'd bring them to your attention anyway. Thanks for everything y'all do! Foundry is integral to our development process.

hey @mslipper, could you pls tell me which chains these txs/blocks are from? if that's local, is there any way to repro?

Do you resolve this problem? I faced the same error when using the fork network.
The receipts of block are:
{
    "jsonrpc": "2.0",
    "id": 1,
    "result": [
        {
            "transactionHash": "0x6dfe8f4a3556854809fc1d49787ed90b8e8b812bf3a8357958360aab21d40c70",
            "transactionIndex": "0x0",
            "blockHash": "0xbc3b4ad6210408931e808a919800ca06a11a0d19dbca068bc0333cf2f7c3a6b7",
            "blockNumber": "0x12a6f23",
            "cumulativeGasUsed": "0x65b8",
            "gasUsed": "0x65b8",
            "effectiveGasPrice": "0x3b9aca03",
            "from": "0x3c44cdddb6a900fa2b585dd299e03d12fa4293bc",
            "to": "0xff00000000000000000000000000000000000901",
            "contractAddress": null,
            "logs": [],
            "logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
            "status": "0x1",
            "type": "0x2",
            "deposit_nonce": null
        },
        {
            "transactionHash": "0x2ef236a2336eb03e44aa4ce3f89ea5303dcc91dab50b7e2c454270644af3e6c5",
            "transactionIndex": "0x1",
            "blockHash": "0xbc3b4ad6210408931e808a919800ca06a11a0d19dbca068bc0333cf2f7c3a6b7",
            "blockNumber": "0x12a6f23",
            "cumulativeGasUsed": "0x1c143",
            "gasUsed": "0x15b8b",
            "effectiveGasPrice": "0x3b9aca03",
            "from": "0x70997970c51812dc3a010c7d01b50e0d17dc79c8",
            "to": "0xb6aa91e8904d691a10372706e57ae1b390d26353",
            "contractAddress": null,
            "logs": [
                {
                    "address": "0xb6aa91e8904d691a10372706e57ae1b390d26353",
                    "topics": [
                        "0xa7aaf2512769da4e444e3de247be2564225c2e7a8f74cfe528e46e17d24868e2",
                        "0x90f2770713ba137c167e7a4b409567ac2ede9c7cd96c966e989e6cc6621cde4e",
                        "0x0000000000000000000000000000000000000000000000000000000000000001",
                        "0x0000000000000000000000000000000000000000000000000000000000000014"
                    ],
                    "data": "0x00000000000000000000000000000000000000000000000000000000660a2322",
                    "blockHash": "0xbc3b4ad6210408931e808a919800ca06a11a0d19dbca068bc0333cf2f7c3a6b7",
                    "blockNumber": "0x12a6f23",
                    "transactionHash": "0x2ef236a2336eb03e44aa4ce3f89ea5303dcc91dab50b7e2c454270644af3e6c5",
                    "transactionIndex": "0x1",
                    "logIndex": "0x1",
                    "removed": false
                }
            ],
            "logsBloom": "0x00000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000002000000000000000000000040000000000000000000000000000000000000000000001040000000000000000000000000000000000000002000000000000000000024000000000000000000000000000000000000000200000000000000000000000000400000000000004000000000800000000000020000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000080000000000000040000000000000000000000000000000000000000000000000000000000000000000",
            "status": "0x1",
            "type": "0x2",
            "deposit_nonce": null
        }
    ]
}

The log index in the second transaction should be "0x0:, but it is "0x1"

The log index in the second transaction should be "0x0:, but it is "0x1"

@0x6e616d could you please add details of the commands you used / forked chain to investigate this issue? thank you

The log index in the second transaction should be "0x0:, but it is "0x1"

@0x6e616d could you please add details of the commands you used / forked chain to investigate this issue? thank you

It's been quite a long time, and I forgot about it. Let me check again

@0x6e616d bump request :pray: thanks!


feat(connector): [Cybersource] Use connector_response_reference_id as reference to merchant

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
Use connector_response_reference_id as reference to merchant for Cybersource
Fixes #2327
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context
fixes #2327

How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

@rashmibharambe Could you merge the main branch into your PR?

@rashmibharambe Could you merge the main branch into your PR?

Done, please review.

@srujanchikke - please review, comments addressed..

@rashmibharambe Thanks for the PR

Hey @rashmibharambe  👋,
Thanks a bunch for all your contributions! We've got a little something for you to show our appreciation.
Just take a moment to fill out this form, and get ready for some awesome swag coming your way.
Thanks!


createGlobalStyle does not hot reload Gatsby app

System:

OS: macOS 10.14.4
CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Memory: 404.55 MB / 16.00 GB
Shell: 5.3 - /bin/zsh

Binaries:

Node: 10.10.0 - /usr/local/bin/node
Yarn: 1.15.2 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm

npmPackages:

babel-plugin-styled-components: ^1.10.0 => 1.10.0
styled-components: ^4.2.0 => 4.2.0

Reproduction
Demo: https://codesandbox.io/s/zny06vk49x
Gatsby default theme were installed. Dependencies are installed following the instructions. No extra options are added: https://www.gatsbyjs.org/docs/styled-components/
Steps to reproduce
Using the Gatsby default starter theme, the new createGlobalStyle API does not work with Gatsby hot reload. Not sure if this is Gatsby specific or something else. Every time I change the css property values inside createGlobalStyle, for example in my codesandbox link, changing the values from red to something else, or the background doesn't hot reload the Gatsby app, and hence no style changes. The page will only get hot reloaded and new styles will only be applied if I refresh the page, or change props in the React component.
In the codesandbox I also created a <Test /> styled component, confirming changing the CSS properties on a regular styled components is working as expected with hot reloading.
Expected Behavior
Changing CSS properties and values inside createGlobalStyle should hot reload the page just like any other regular styled components
Actual Behavior
Changing CSS properties and values inside createGlobalStyle does not hot reload the page. Styles can only get applied if I refresh the page or changing React component props

#2074 seems to be related so linking it here for visibility.

In<EMAIL_ADDRESS>the problem seems to be resolved with<EMAIL_ADDRESS>with the expected behavior.

This should be fixed in the later v5 RCs

Running<EMAIL_ADDRESS>as released on 01/13/2020 and still having this issue

Same here on 5.1.0 - does a full page reload


Handle unprotected TXs properly

Describe your changes and provide context
Previously there were two issues with our handling of unprotected TXs:

They are outright not allowed (checked in sig.go), but they do still exist in today's Ethereum mainnet
The pubkey derivation logic for unprotected TXs was wrong. Specifically we need to use Frontier's hashing logic for unprotected TXs.

Testing performed to validate your change
unit tests & replay

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 63.08%. Comparing base (de9cdc6) to head (4b74be6).

Additional details and impacted files

@@            Coverage Diff             @@
##            seiv2    #1443      +/-   ##
==========================================
+ Coverage   62.80%   63.08%   +0.28%
==========================================
  Files         350      350
  Lines       23970    23968       -2
==========================================
+ Hits        15054    15121      +67
+ Misses       8048     7979      -69
  Partials      868      868

Files
Coverage Δ

x/evm/ante/preprocess.go
74.40% <100.00%> (+0.10%)
:arrow_up:

x/evm/ante/sig.go
88.33% <ø> (+4.20%)
:arrow_up:

... and 3 files with indirect coverage changes


Use a different icon for the debugger tab with both warnings and errors

This makes it possible to see if both errors and warnings were pushed without having to open the tab.
Preview

Thanks!

Cherry-picked for 3.2.2.


64-check all arguments to create machines and prevent unreasonable values

Check all arguments to create machine groups and make sure they have decent values.
For instance, we can create machine groups with -1 machines or with 0 machines.
Find the cases where that can happen or not, and raise exceptions when we need them.

QA
Try to launch machine groups with negative numbers of instances.

Tested for both MPI, MachineGroup and ElasticMachineGroup:
All raised the exception


Implement Simulate Swap Query

It is extremely useful to have a query where users can input the trade details and get out important data from it. We might want to refactor the swap function to not duplicate code for this.

[ ] Define scope and Spec Simulate Swap
[ ] Modularize the swap function or create a new one
[ ] New unit tests for the query only

May be related to #191

Putting a pause on this until @NicholasDotSol is done with his investigation of whether the Cosmos SDK-level simulate is adequate for our needs.

@NicholasDotSol saying that it should be possible to achieve without a specific query endpoint, closing.


Implement other code examples on Predictive Models which detect anomalies using MAPE/MSE value

We have already implemented one code sample which uses fbprophet to detect anomalies
We would want you to make more code examples using other predictive models. :pray:
List of Predictive Models that can be used

[ ] ARIMA

[ ] SARIMA

[ ] ARCH

[ ] GARCH

[ ] VAR

[ ] VECM

[ ] LSTM

[ ] Convolutional Autoencoder model

Instructions to be followed

The Code examples on the implementations of most of the models mentioned above should be available in the internet. :octocat:
Build the respective predictive model on the NAB Dataset(RealAWSCloudwatch - Any CPU Utilization Data ONLY).
Use the Predictive Model to forecast future values.
Find the Error rate of the predictive (Use MAPE/MSE) and come up with a confidence interval for the predicted value.
Any Actual datapoint falling beyond the confidence interval, will be detected as an anomaly.

Additional instructions

Make a new jupyter notebook with the algorithm name and work in that only.
Initially above algorithms will be assigned on a First Come First Serve basis. The algorithm will be assigned to you if you create an "assign request" comment below with a small description of the algorithm you have requested for. :smiley: :1st_place_medal:

@JoeRishon & @manikyabard for any doubts and additional details

Hey @manikyabard and @JoeRishon can I do this??


Implementing Contrastive Self-Supervised Learning with Radiation Augmentations, SimCLR, PyTorch Lightning, and Hyperparameter Optimization

This PR constitutes the bulk of my code (excluding notebooks used to generate and analyze results) used in my dissertation. Some highlights:

Augmentations for gamma radiation spectra (augs.py) including a way to use them in contrastive learning (transforms.py).
Basic infrastructure for creating a PyTorch/Lightning (ann.py/lightModel.py) multilayer perceptron (LinearNN) and convolutional neural network (ConvNN), and a projection head (critic.py).
A self-/semi-supervised contrastive learning training script written for both PyTorch (SlimCLR.py) and PyTorch Lightning (SlimCLRLight.py) using the package pytorch-learning-metrics for a normalized cross-entropy loss function based on SimCLR.
Scripts for conducting hyperparameter optimization on the base encoder (SSLHyperOpt.py) and the projection head (ProjHyperOpt.py) using the package hyperopt.
A snapshot of the conda/python environment I used for my research (contrastive-environment.yml).

There is a lot of code in this branch, so it will undoubtedly make sense to split it up. I also need to clean up scripts. Many include functions that I ended up not using, or were borrowed from other people (e.g. specTools.py from Ken) and require proper attribution. If we want to move to review and merge this PR, we should probably finish reviewing and merging the preliminary work PRs in #42, #44, #45, #46.

[ ] UNIT TESTS

Closing in favor of #52.


duplicate groups

http://akswnc7.informatik.uni-leipzig.de:3001/jj-author

related to / caused by https://github.com/dbpedia/databus-maven-plugin/issues/94


Reorganize zinc

The organization of the zinc instructions is really haphazard, the goal is to split the instructions into separate types with seperate concerns. improving that will make zinc easier to review because we'll be able to submit PRs for just one type at a time

Closed by  #350


App threw an error when running [TypeError: electron_1.app.dock.setIcon is not a function]

At boot

Which OS do you use?


String.match regex breaks highlighting

This is a screenshot of Navbar.vue in the default Vuepress theme:

Cannot reproduce using vim-javascript and latest vim-vue