Renderer - docs error or bug?

Running the example code found in:
http://holoviews.org/reference/elements/matplotlib/Histogram.html
Receiving this exception:
2017-08-24 10:01:27,016 Error running application handler <bokeh.application.handlers.script.ScriptHandler object at 0x2b6fba49a940>: module 'holoviews' has no attribute 'renderers'
File "hvapp.py", line 5, in <module>:
renderer = hv.renderers('matplotlib') Traceback (most recent call last):
  File "/home/byed/venv36/lib/python3.6/site-packages/bokeh/application/handlers/code_runner.py", line 81, in run
    exec(self._code, module.__dict__)
  File "/home/byed/job/fire/hvapp.py", line 5, in <module>
renderer = hv.renderers('matplotlib')
AttributeError: module 'holoviews' has no attribute 'renderers'

API change?

Also,
(venv36) byed@irsdev01:~> pip install 'holoviews[all]'
Requirement already satisfied: holoviews[all] in ./venv36/lib/python3.6/site-packages
Requirement already satisfied: param<2.0,>=1.5.1 in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: numpy>=1.0 in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: ipython in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: pyzmq in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: jinja2 in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: tornado in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: jsonschema in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: notebook in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: pygments in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: matplotlib in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: lancet-ioam in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: pandas in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: seaborn in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: bokeh>=0.12.6 in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: cyordereddict in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: nose in ./venv36/lib/python3.6/site-packages (from holoviews[all])
Requirement already satisfied: decorator in ./venv36/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: simplegeneric>0.8 in ./venv36/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: pexpect; sys_platform != "win32" in ./venv36/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: prompt-toolkit<2.0.0,>=1.0.4 in ./venv36/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: setuptools>=18.5 in ./python3/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: jedi>=0.10 in ./venv36/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: pickleshare in ./venv36/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: traitlets>=4.2 in ./venv36/lib/python3.6/site-packages (from ipython->holoviews[all])
Requirement already satisfied: MarkupSafe>=0.23 in ./venv36/lib/python3.6/site-packages (from jinja2->holoviews[all])
Requirement already satisfied: jupyter-client in ./venv36/lib/python3.6/site-packages (from notebook->holoviews[all])
Requirement already satisfied: ipykernel in ./venv36/lib/python3.6/site-packages (from notebook->holoviews[all])
Requirement already satisfied: nbconvert in ./venv36/lib/python3.6/site-packages (from notebook->holoviews[all])
Requirement already satisfied: terminado>=0.3.3; sys_platform != "win32" in ./venv36/lib/python3.6/site-packages (from notebook->holoviews[all])
Requirement already satisfied: ipython-genutils in ./venv36/lib/python3.6/site-packages (from notebook->holoviews[all])
Requirement already satisfied: nbformat in ./venv36/lib/python3.6/site-packages (from notebook->holoviews[all])
Requirement already satisfied: jupyter-core in ./venv36/lib/python3.6/site-packages (from notebook->holoviews[all])
Requirement already satisfied: six>=1.10 in ./venv36/lib/python3.6/site-packages (from matplotlib->holoviews[all])
Requirement already satisfied: cycler>=0.10 in ./venv36/lib/python3.6/site-packages (from matplotlib->holoviews[all])
Requirement already satisfied: pytz in ./venv36/lib/python3.6/site-packages (from matplotlib->holoviews[all])
Requirement already satisfied: python-dateutil in ./venv36/lib/python3.6/site-packages (from matplotlib->holoviews[all])
Requirement already satisfied: pyparsing!=2.0.0,!=2.0.4,!=2.1.2,!=2.1.6,>=1.5.6 in ./venv36/lib/python3.6/site-packages (from matplotlib->holoviews[all])
Requirement already satisfied: requests>=1.2.3 in ./venv36/lib/python3.6/site-packages (from bokeh>=0.12.6->holoviews[all])
Requirement already satisfied: PyYAML>=3.10 in ./venv36/lib/python3.6/site-packages (from bokeh>=0.12.6->holoviews[all])
Requirement already satisfied: bkcharts>=0.2 in ./venv36/lib/python3.6/site-packages (from bokeh>=0.12.6->holoviews[all])
Requirement already satisfied: ptyprocess>=0.5 in ./venv36/lib/python3.6/site-packages (from pexpect; sys_platform != "win32"->ipython->holoviews[all])
Requirement already satisfied: wcwidth in ./venv36/lib/python3.6/site-packages (from prompt-toolkit<2.0.0,>=1.0.4->ipython->holoviews[all])
Requirement already satisfied: testpath in ./venv36/lib/python3.6/site-packages (from nbconvert->notebook->holoviews[all])
Requirement already satisfied: pandocfilters>=1.4.1 in ./venv36/lib/python3.6/site-packages (from nbconvert->notebook->holoviews[all])
Requirement already satisfied: bleach in ./venv36/lib/python3.6/site-packages (from nbconvert->notebook->holoviews[all])
Requirement already satisfied: entrypoints>=0.2.2 in ./venv36/lib/python3.6/site-packages (from nbconvert->notebook->holoviews[all])
Requirement already satisfied: mistune!=0.6 in ./venv36/lib/python3.6/site-packages (from nbconvert->notebook->holoviews[all])
Requirement already satisfied: urllib3<1.23,>=1.21.1 in ./venv36/lib/python3.6/site-packages (from requests>=1.2.3->bokeh>=0.12.6->holoviews[all])
Requirement already satisfied: certifi>=2017.4.17 in ./venv36/lib/python3.6/site-packages (from requests>=1.2.3->bokeh>=0.12.6->holoviews[all])
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in ./venv36/lib/python3.6/site-packages (from requests>=1.2.3->bokeh>=0.12.6->holoviews[all])
Requirement already satisfied: idna<2.7,>=2.5 in ./venv36/lib/python3.6/site-packages (from requests>=1.2.3->bokeh>=0.12.6->holoviews[all])
Requirement already satisfied: html5lib>=0.99999999 in ./venv36/lib/python3.6/site-packages (from bleach->nbconvert->notebook->holoviews[all])
Requirement already satisfied: webencodings in ./venv36/lib/python3.6/site-packages (from html5lib>=0.99999999->bleach->nbconvert->notebook->holoviews[all])
(venv36) byed@irsdev01:~>

AttributeError: module 'holoviews' has no attribute 'renderers'

Thanks for pointing this out! It should be renderer (singular). That said, I don't see any mention of it on the page you linked to.

You are right, sorry mate I buggered that up. Been going around in circles on Bokeh n hv for a couple days now.... anyway I probably hacked a bit and forgot, perhaps from this code:
import numpy as np
import holoviews as hv
import holoviews.plotting.bokeh

renderer = hv.renderers('bokeh')

points = hv.Points(np.random.randn(1000,2 ))(plot=dict(tools=['box_select', 'lasso_select']))
selection = hv.streams.Selection1D(source=points)

def selected_info(index):
    arr = points.array()[index]
    if index:
        label = 'Mean x, y: %.3f, %.3f' % tuple(arr.mean(axis=0))
    else:
        label = 'No selection'
    return points.clone(arr, label=label)(style=dict(color='red'))

layout = points + hv.DynamicMap(selected_info, streams=[selection])

doc = renderer.server_doc(layout)
doc.title = 'HoloViews App'

Which was from this page:
http://holoviews.org/user_guide/Deploying_Bokeh_Apps.html
in the section Deploying Apps using bokeh serve

This has been fixed, so I'll close.


Snips won't run on Hassos 3.7

Running HassOs 3.7 and try to start Snips.Ai got this error message and it does not work.
ERROR:snips_nlu       : Unable to read file '/usr/share/snips/assistant/nlu_engine/nlu_engine.json'
 -> caused by: Mismatched model version: model is 0.17.0 but runner is 0.20.0

The model you are using does not match the version of the add-on.
Please note that: https://www.home-assistant.io/blog/2019/12/03/sonos-shutting-down-local-voice-option-snips/

I am new to this and using the default settings.
I did read the news title before but think that was only something that affects Sonos users.
I am closing this.


Update doc, start used by list

Start a list of projects who use this library.
Usefull for the end user as an overview which gui's use this lib.
Usefull for developers to check, to avoid breaking changes.

I'm neutral on this.


Expose compose function

Having in mind that there's a redux dependency in the project and the combineReducers is being exposed, I believe we should also expose thecompose function.
Reason:

People would not npm install it
Double import redux just to consume one function.
To make it more complete, as it's a such a nice helper to have under your sleeve.

As I mentioned elsewhere, I think I'd rather not re-export anything from Redux, but I think this is a great idea for now since it's probably the second most commonly used utility function in Redux along with compose, and exposing it also feels more consistent.

Forgot to close this, the PR was merged! Thanks @codedavinci


Add Example

Create an example to test and demonstrate the basic functionality of the threadpool

Closing. Picking up CI/CD problems in another issue.


Wrong gate order in drawing with if_else

Environment

Qiskit Terra version: 0.0.23 (main branch in git at date)
Python version: 3.10.6
Operating system: MacOs 12.6

What is happening?
In the drawings, the precendence of operations is sometimes ignored, when a classical bit is the cause of the precedence, in presence of if_then.
The behavior appears to be correct in the internal representation  and in the QASM3 export.
How can we reproduce the issue?
Run the following snippet
cr = ClassicalRegister(3)
qr = QuantumRegister(3)
qc = QuantumCircuit(qr, cr)

qc.x(qr[:2])
qc.measure(qr[0], cr[0])
with qc.if_test((cr[0], 1)) as else_:
    qc.measure(qr[1], cr[1])
with else_:
    pass
qc.x(qr[2]).c_if(cr[1], 1)
qc.measure(qr[2], cr[2])

display(qc.draw('mpl', cregbundle=False))

What should happen?
x(2).c_if(1, True) should appear after the if_else statement, but it is anticipated

Any suggestions?
No response

@jakelishman FYI


New JAR not in Maven Central

        <dependency>
            <groupId>software.xdev</groupId>
            <artifactId>chartjs-java-model</artifactId>
            <version>1.1.2</version>
        </dependency>

Not found. Are you sure you have permissions to deploy to software.xdev Sonatype gives permission by Group ID?

New releases was successful, should be fixed now :)

Confirmed thanks!


cosign verify-attestation --policy docs don't have working examples

Description
I was trying to play with this: https://docs.sigstore.dev/cosign/attestation/#cosign-custom-predicate-type-and-cue-policy
However, I noticed a few things:

A mix of -foo and --bar flags (we should shift to --bar exclusively).
The given example doesn't work!

Here is an example I was able to get working:
import "time"

before: time.Parse(time.RFC3339, "2021-10-09T17:10:27Z")

// The predicateType field must match this string
predicateType: "cosign.sigstore.dev/attestation/v1"

// The predicate must match the following constraints.
predicate: {
    Timestamp: <before
}

Perhaps things changed from passing the predicate to the statement itself?  Regardless, opening this to track.
cc @dlorenc to route to the best person to update

cc @developer-guy @Dentrax !


BUG? plot_quantile_returns_violin and than plot_quantile_returns_bar use different data

Here  we can see the data passed to plot_quantile_returns_violin is not the same passed to plot_quantile_returns_bar (mean_ret_quant_daily vs mean_ret_quantile).
It seems mean_ret_quant_daily  was meant to be used by plot_cumulative_returns_by_quantile only. So I believe that's a typo, but I might be wrong.

As a side note I would probably drop the calculation of mean_ret_quant_daily, it doesn't suit well with all possible data frequency passed to alphalens. Or, at lest, we should avoid mixing plots where the data is aggregated by day (e.g. cumulative plot by quantile) with plots where it is not (e.g. factor weighted cumulative plot). This is confusing.
That applies to the Information Analysis too (and the patch #85 would fix that).

To summarize, I believe the aggregation of data by day is a nice feature that would produce better plots for some scenarios (data frequency/data time span) but not for all the possible input of alphalens. This means the aggregation feature should be optional and the aggregation period (day, hour, week) configurable.  Also, to be useful, the feature should be applied to all the plots consistently.
Currently the aggregation of data by time rule is not optional, it is not configurable and it is applied randomly to some plots only. This is confusing and I would completely remove the feature until properly implemented.

The code doesn't exactly does what I thought and it's almost fine. Still it doesn't make sense to aggregate by day when calculating perf.mean_return_by_quantile  for the reasons exaplained above.


Final Report - Initial clean up

Follow on of #56
This issue will track the work done on chapter 1, 2 and 3 clean up

Brendon and I met up and have decided to clean up chapter 1, 2 and 3 in the next few days before working on 4 and 5.


amazing work! viable for appstore use?

works like a charm thanks so much, I was wondering did you post any apps with UXKit to the app store?
p.s. not that it really matters because you can release it through the website which is optimal for any mac os applications due to app store being so bug ridden.

You can't, it is a private API and would definitely be flagged by Apple.


"algo::bellman_ford" gives me negative cycle errors although there is only a negative noncyclic graph

I'm writing software to build a rooted (directed) tree from an unrooted (undirected) tree using Petgraph.
To identify a path with a maximum length, first convert a positive undirected graph to negative one.
Then process it with the "bellman_ford", but it gives me the above errors.
Checked the structure of the negative one, but it seems not to have any cyclic path. (Tried with a small set of leaves (25 ones) for debugging.)
The tree correctly has 25 leaves, 23 internal nodes, and 47 edges. (If it were converted to a rooted one, the rooted one correctly does 25 leaves, 24 internal nodes, and 49 edges by adding 1 node and 2 edges.)

Hi, I'm sure this is easily fixed once we have a reproducing example.

Thank you for your quick reply.
I noticed the algorithm be for only "directed" graph, apparently it's my mistake.
If its function required "directed" edge type for any graph (e.g., using trait), inappropriate use of the function would be blocked.

FWIW, I added a quickcheck for testing a random graph of positive weights. That's a too simple to show the error.

I wasn't sure the algo should be restricted to only directed graphs. Wikipedia thinks so, though.

I meant the Bellman-Ford algorithm be for only "directed" one while the Dijkstra algorithm be for both.

Understood.
I'm gonna use the Dijkstra algo. with inverted positive weights, instead.
Thank you.


Non-human DOI vetting

Issue #2229 discusses the need for human interaction during the formal publication process to weed out resources that should not be issued DOI's.  We need to also improve/implement non-human validation approaches to help deter the publishing of "test" resources.  Below are a few ideas (as well as our current methods) that @Lizabrazil  and myself came up with:
[x] Multiple dialogs to let the user know that this action is permanent.
[x] Send an email to @Lizabrazil to check the resource before a DOI is issued.
[ ] Validation of resource content, e.g. title or keywords contain "test", resource does not contain content.
[ ] 24 or 48 hour holding period before a DOI is issued
[ ] holding queue that allows a group of developers to review outstanding DOI requests and either issue or deny requests.  This might be an extension of #2229 that allows multiple people to "accept" doi requests so that we don't have a single point of failure.

Agreed non-human validation would be ideal, but with only simple validation like making sure title or keywords not containing "test" is not sufficient to prevent wrongly publishing a resource, for example, this resource https://www.hydroshare.org/resource/b01ff941e62f4d3f8814affcbc3bbc14/ which was just wrongly published does not contain "test" in title or keywords or abstract. We already enforce that resource with no content cannot be made public and hence cannot be published. To make validation work well, I think some machine learning or NLP approaches have to be incorporated to catch signs that the resource may be wrongly published, and I am not sure whether we would want to go with that route. With human-in-the-loop validation, it can be implemented without involving too much effort, but we need to determine a workflow that works best, and make it a sustainable action to justify the effort.

We could theoretically use Human-in-the-Loop validation to train an ML system, but that seems like overkill. Maybe not though, based on @Lizabrazil's concerns about keeping up with demand as we scale up...

@hyi @aphelionz I like the idea of potentially using machine learning in issuing DOIs with or without a human-in-the-loop in my capacity to think about HydroShare 2.0 resource citation.  Even if we keep humans-in-the-loop for issuing DOIs for the forseeable future, it seems prudent to address an ML approach now for scalability and assesing the performance of an ML approach in tandem with the human approach.

@rayi113 incorporating machine learning in HydroShare is intriguing and sounds to me like a good idea in HydroShare 2.0 to pursue. I like @aphelionz 's idea of using Human-in-the-Loop validation to train ML system to make ML work better over time as well.

For human in the loop this Django module may be useful to control publishing resources: https://github.com/javrasya/django-river

I worry that researching machine learning to get this automated is beyond scope.  See # 2229 for my outline of Human in the loop approach.  The computer science folks can write a proposal to get funds to do this.

Closing as CUAHSI made a policy decision to go with a manual review process.  See #2298


⚠ Bofrost.de has degraded performance

In 36ec9bc, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1025 ms

Resolved: Bofrost.de performance has improved in 214af83 after 10 minutes.


VS Code Extensions for ServiceNow - error

VS Code Extensions for ServiceNow - error.

Tried reinstalling vs code -

reinstalling Servicenow extension for VS code

Nothing worked - please help

Please report the problem with the owner of the ServiceNow extension. This issue tracker is about ESLint.


Allow authentication against protected Elasticsearch server

Hi,
Just add the possibility to authenticate against a protected Elasticsearch server.
Also allow to connect to Elasticsearch server behind SSL.
First pull request here, let me know if something's wrong ;)

You have to update the XSD file too.

Here we go :)

Thanks!


reverting the PR template to the old one

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Add VM function to execute raw bytecode

CF #306

This is a pretty minimal prototype. It's basically just a thin wrapper over the Message object, leaving out a few details (origin, create_address, code_address, should_transfer_value, is_static).
For now, for testing, I'm running the full suite of VM tests through this new entrance. Not sure if this is what we want to do--we may want to also test different combinations of parameters.
@cwinsloweaton please take a look, maybe give this a shot and see if it suits your purposes.
(Just realized I still need to add code_address per https://github.com/ethereum/py-evm/issues/306#issuecomment-362027669.)

@lrettig can you rebase?

Rebased and added code_address. I think it might be good to add a test for this too, e.g., to try reading the state of an existing contract account using arbitrary code.


Crash on insert via materialized view

version: <IP_ADDRESS> (also happens on 18.161).

Description:
I have 2 tables:
CREATE TABLE foo(d Date, x String, y String, z String)
ENGINE = [Replicated]MergeTree()
PARTITION BY toYYYYMMDD(d)
ORDER BY (x, y)

CREATE TABLE bar(d Date, x String, y String, z String, ... 35 more columns)
ENGINE = [Replicated]MergeTree()
PARTITION BY toYYYYMMDD(d)
ORDER BY (x, y)

(bar has 39 columns, foo has 7 columns)
Then I create a common deduplicated table to collect unique (x,y,z) per d:
CREATE TABLE dest(d Date, x String, y String, z String)
ENGINE = ReplacingMergeTree()
PARTITION BY toYYYYMMDD(d)
ORDER BY (d,x,y,z)

Then I populate the table:
INSERT INTO dest SELECT d,x,y,z FROM foo
INSERT INTO dest SELECT d,x,y,z FROM bar

Then I create 2 MVs from foo & bar to dest:
CREATE MATERIALIZED VIEW foo2dest TO dest AS SELECT d,x,y,z FROM foo
CREATE MATERIALIZED VIEW bar2dest TO dest AS SELECT d,x,y,z FROM bar

Expected behavior: updates to foo/bar eventually end up in dest and get deduplicated by the Replacing engine.
Observed behavior: process runs fine for ~8-10 minutes, then crashes and restarts (CH is running in Docker).
dmesg does not contain anything useful or even related.
I tried attaching strace -f -p $CH_PID to the process, here's what I recorded:
[pid 29615] madvise(0x7f6859003000, 2629632, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f6858c40000, 172032, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f6870e48000, 86016, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f6870dfc000, 12288, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f6870df1000, 28672, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f6870e0f000, 8192, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f68cb5fa000, 20480, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f6870dcb000, 24576, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] madvise(0x7f6870d91000, 40960, MADV_DONTNEED <unfinished ...>
[pid 29615] <... madvise resumed> )     = 0
[pid 29615] --- SIGSEGV {si_signo=SIGSEGV, si_code=SEGV_ACCERR, si_addr=0x7f6857f7efd8} ---

I also tried to create MVs as follows
CREATE MATERIALIZED VIEW foo2dest TO dest AS SELECT d,x,y,z FROM foo GROUP BY d,x,y,z
CREATE MATERIALIZED VIEW bar2dest TO dest AS SELECT d,x,y,z FROM bar GROUP BY d,x,y,z

in order to see if grouping affects this (just a random thought, didn't have a reason to suspect this would help).
This happens exclusively after inserts into a fatter table (bar, 39 columns). Inserts are batched, but this happens on a very low-load setup (max batch size I observed today was ~300 entries).
Real-life setup uses Replicated trees for everything but i'm not sure that matters.
Let me know if I can provide any additional details.

Can you provide server log?

Which part? Even on trace there are no messages between the fat insert logging and the “unclean restart” message.

Seems like oom kills clickhouse.

@alesapin There are no related messages in dmesg, is there any other way to check (it's running in docker)? Also, given that the batch size is just 300 rows, how is it possible to trigger an OOM? I also have external group by & external sort enabled with 5GB as threshold. Total RAM available to CH is 13GB.

[pid 29615] --- SIGSEGV {si_signo=SIGSEGV, si_code=SEGV_ACCERR, si_addr=0x7f6857f7efd8} ---
I think it's a segfault. There should be stacktrace in /var/log/clickhouse-server/clickhouse-server.log

Yes, I figured it's a segfault. As I mentioned before, there is nothing in the logs even on the trace level.


Add support for src task id in the storm api

In Storm, the bolts can use the tuple.src_task_id() to find out where this tuple originated from. However currently stmgr batches tuples from different local source task_ids and thus the destination stmgrs do not have any idea of where a particular tuple came from. Several of current storm users use this feature and since Heron claims to be api compliant, we need to support this. This change makes the stmgr break tuple batches sent to other stmgrs based on the src task id so that each tuple batch sent to the remote stmgr is always coming from the same src task id. Thus stmgrs know exactly where a tuple originated and can pass this information to the instances.

Why do tuple batches need to be grouped by source task id? Since source task id is tracked at the tuple level, couldn't batches contain tuples from a mix of sources?

We do not track src task id at tuple level but at batch level for performance reasons.

couldn't batches contain tuples from a mix of sources

-- I think this is due to the exactly-once algorithm. We need to be able to distinguish and control tuples from different sources, so we can control the markers.


Fix for issue #47216

FIX TypeError if set the weights to the current weights via `set_weights,
Incorporated suggestions by @qlzh727 i.e. included type checking to reduce overhead.
Simply, this change will allow users to use .weights to initialize the set_weights() method of a layer without any type error improving the versatility of keras.

@fchollet, Thank you for your advice, I will include the changes you suggested.

Please include a unit test.

``"""
>>> a = tf.keras.layers.Dense(1,
...   kernel_initializer=tf.constant_initializer(1.))
>>> a_out = a(tf.convert_to_tensor([[1., 2., 3.]]))
>>> a.get_weights()
[array([[1.],
[1.],
[1.]], dtype=float32), array([0.], dtype=float32)]
>>> b = tf.keras.layers.Dense(1,
...   kernel_initializer=tf.constant_initializer(2.))
>>> b_out = b(tf.convert_to_tensor([[10., 20., 30.]]))
>>> b.get_weights()
[array([[2.],
[2.],
[2.]], dtype=float32), array([0.], dtype=float32)]
>>> b.set_weights(a.weights)
>>> b.get_weights()
[array([[1.],
[1.],
[1.]], dtype=float32), array([0.], dtype=float32)]
"""

"""
>>> a = tf.keras.layers.Dense(1,
...   kernel_initializer=tf.constant_initializer(1.))
>>> a_out = a(tf.convert_to_tensor([[1., 2., 3.]]))
>>> a.get_weights()
[array([[1.],
[1.],
[1.]], dtype=float32), array([0.], dtype=float32)]
>>> b = tf.keras.layers.Dense(1,
...   kernel_initializer=tf.constant_initializer(2.))
>>> b_out = b(tf.convert_to_tensor([[10., 20., 30.]]))
>>> b.get_weights()
[array([[2.],
[2.],
[2.]], dtype=float32), array([0.], dtype=float32)]
>>> b.set_weights(a.weights)
>>> b.get_weights()
[array([[1.],
[1.],
[1.]], dtype=float32), array([0.], dtype=float32)]
"""
import doctest
doctest.testmod()
`

@fchollet, I found this test on the tensorflow website for set_weights() method, I made modifications to the doctest to include .weights instead of get_weights() method to assign weights.

Sorry for the long wait. The Keras code base has actually been migrated to github.com/keras-team/keras. If you still would like to contribute the PR, feel free to post your change to the new repository. I am closing this PR for now.


Added Affine Substitution Cipher in golang

Have you read the Contributing Guidelines on Pull Requests?
yes
(Write your answer here.)
Description
Added Affine Substitution Cipher in golang
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixed #4931
(Write your answer here.)

@HarshCasper, please review this. Thank you!


Allow excluding dependencies at build time

Mapping file defines an id to map to another id to merge with

This is probably not the best way to do this, however I wanted to get the PR started. I am no gradle expert so I'm having trouble getting the app to use the locally built plugin.

oh I am also not a gradle expert. what I usually do:
1.) change the version in plugin/build.gradle to a higher version
2.) use uploadArchives to install it locally (see picture below)
3.) use the new version in the global build.gradle for the plugin
This usually works for me :)

Thinking about this a little more, merging dependencies would essentially be excluding them, as merging a child dependency into a parent dependency means you don't really care about the child dependency info right? I think essentially what I'm going to do is add an exclusion list config file and skip the dependencies in that file

@mgray88 I think for a first version that is sufficient.
for the merging part. I believe that kind of overlaps with the multi license per library feature. as merging may also mean there are different libraries in the too-be-merged libraries. or other fields which may differ based on the modules

True. I'm not sure how easy it would be to handle that automatically though, it seems more like something that falls under your disclaimer Careful optimisation and review of all licenses :)

Thanks again a lot for this PR :)


LGeoJson.vue has two methods for setVisible

Description
LGeoJson.vue has two methods named setVisible.
1: https://github.com/KoRiGaN/Vue2Leaflet/blob/c39704a6a4c5475ce687711fa45d6554e02871f2/src/components/LGeoJson.vue#L33
2: https://github.com/KoRiGaN/Vue2Leaflet/blob/c39704a6a4c5475ce687711fa45d6554e02871f2/src/components/LGeoJson.vue#L53
I'm not sure which is the more correct one.
This commit in May modified the 2nd one, but not the 1st one:
https://github.com/KoRiGaN/Vue2Leaflet/commit/90c363553b4d0d8909c76af037fe74027905c030

This is addressed in this PR: #200


Good plugin, but.. unable to unstick when reach the footer

I was thinking how to unstick when sticker just reach the footer, I don't want the sticker is overlapping or get behind it and just slide it away.. anyone know how to unstick the sticker before is reach footer? thanks

Wondering how to do this myself. Any thoughts?

@ao @mkaizumi Hi! In case you didn't figured it out, I managed to do it by doing something like this:

  # When scrolling
  $(window).scroll( () ->
    # Get window position
    # console.log $(window).scrollTop()
    position = 123 # OR  position = $("#myDiv").offset().top
    if ($(window).scrollTop() >position )
      $("#sticky").unstick()
    else
      $("#sticky").sticky()
  )

You can have a look at this stackoverflow answer https://stackoverflow.com/questions/13522971/jquery-to-detect-scrolling-div-position
Hope it helps! :)

I tried to add it to my script but not working.
Please check this.

@saruban I think you have to rewrite it in javascript (it's coffeescript here! :) )

Can you please help me to rewrite it on js. :)
On 1 Aug 2017 17:39, "Arpsara"<EMAIL_ADDRESS>wrote:

@saruban https://github.com/saruban I think you have to rewrite it in
javascript (it's coffeescript here! :) )
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/garand/sticky/issues/256#issuecomment-319352037, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJiowYed1fTUOej4b3wbtzfOlGP9wHRLks5sTxWSgaJpZM4NUms3
.

@saruban
$(window).scroll(function() {
  var position;
  position = 123;
  if ($(window).scrollTop() > position) {
    return $("#sticky").unstick();
  } else {
    return $("#sticky").sticky();
  }
});

Thank you so much. Will Check and back to you.  :)
On 1 Aug 2017 6:24 pm, "Andrew Odendaal"<EMAIL_ADDRESS>wrote:

@saruban https://github.com/saruban
$(window).scroll(function() {
var position;
position = 123;
if ($(window).scrollTop() > position) {
return $("#sticky").unstick();
} else {
return $("#sticky").sticky();
}
});
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/garand/sticky/issues/256#issuecomment-319361493, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJiowR4hR8hCp4dVEK94VT4uFXFLHyDMks5sTx_wgaJpZM4NUms3
.

You don't need to unstick.
Proper way is to determine "bottomSpacing" value correctly based on the height of the footer that can be variable in some cases.
Just add dummy divs on the page with
div(id="contentEnd")  - where main page content ends and starts footer
div(id="pageEnd") - at the end of the footer
And then initialize Sticky plugin
bottomSpacing = $("#pageEnd").offset().top - $("#contentEnd").offset().top
$('#sticky').sticky
	topSpacing: 100
	bottomSpacing: bottomSpacing

You can view more detail how to stop sticky when reach end point - https://github.com/garand/sticky/issues/257#issuecomment-351935092

Another option that worked a little better in my situation
    var footerBottomAdjust = $('#footer').height() + 50;
    $('#sticky').sticky({
        topSpacing: 200,
		bottomSpacing: footerBottomAdjust
    });

@jeremy-hawes tyvm!


Cannot remove server security connector once added

Once a security connector has been configured for a server it cannot be removed.
For example

Issue a query to a server as user 'admin' to get metadata collection id-> works fine
Configure the coco pharma security connector
Issue same query -> fails. AS EXPECTED
Remove the security connector (DELETE)
Issue same query -> fails. INCORRECT.

  The request fails with a UserNotAuthorizedException exception.","exceptionUserAction":"Determine whether the user should have access to the server.  If they should have, take steps to add them to the authorized list of users.  If this user should not have access, investigate where the request came from to determine if the system is under attack, or it was a mistake, or the user's tool is not configured to connect to the correct server.","exceptionProperties":{"userId":"admin"}}

>> Stop server
{"class":"VoidResponse","relatedHTTPCode":200}

>> Deleting security connection
{"class":"VoidResponse","relatedHTTPCode":200}

>> Start server
{"class":"SuccessMessageResponse","relatedHTTPCode":200,"successMessage":"Fri Nov 18 16:44:01 GMT 2022 mds1 is running the following services: [Open Metadata Repository Services (OMRS)]"}

>> Displaying security connection
{"class":"ConnectionResponse","relatedHTTPCode":200}

>> issuing a query by admin
{"class":"MetadataCollectionIdResponse","relatedHTTPCode":403,"actionDescription":"validateUserForServer","exceptionClassName":"org.odpi.openmetadata.repositoryservices.ffdc.exception.UserNotAuthorizedException","exceptionErrorMessage":"OMAG-PLATFORM-SECURITY-403-002 User admin is not authorized to issue a request to server mds1","exceptionSystemAction":"The system is unable to process a request from the user because they do not have access to the requested OMAG server.  The request fails with a UserNotAuthorizedException exception.","exceptionUserAction":"Determine whether the user should have access to the server.  If they should have, take steps to add them to the authorized list of users.  If this user should not have access, investigate where the request came from to determine if the system is under attack, or it was a mistake, or the user's tool is not configured to connect to the correct server.","exceptionProperties":{"userId":"admin"}}%

A few things to note:

This must be run with a clean server (ie just started) since once the connector is loaded it keeps on being used
The server is totally standalone, run as the standalone chassis, with LOADER_PATH set to include our usual libs (including the coco pharma connector)
The test script tries 3 requests to get metadata collection id. the first works (before security enabled). the second fails (with security, as expected), but the bug is the third request which fails, but should succeed

I have tried this scenario and it is possible to delete the security connector from the server, restart it and then any user is able to issue any request.


Rename conversation in sidebar

The default titles generated for conversations are very lengthy. Sometimes, I would prefer to give a title to a conversation manually. Hence, this PR introduces the possibility to rename a conversation from the sidebar through the context menu:
https://user-images.githubusercontent.com/13868083/224481721-66f0a0b2-c9a5-4f9f-b5dd-8f3a51c014b4.mp4
Note that this implementation only addresses macOS.

Thanks.


SDK Extension Support

This PR adds the ability to easily extend the SDK by overriding the default implementation of the core SDK services while preserving the base line interface and DI resolution for the core services.

Renames all base service interfaces to be suffix with IDefault.
Adds a ServicesBuilder class that handles building the SDK services pool.
Added the ability to control the services injection configuration from the AddAgent extension method.

R.e your first point, my motivation for renaming the interfaces to be generic is that the interface itself could be extended for example we could have ICustomConnectionService : IDefaultConnectionService. The approach to the service registration means a custom connection service implementation i.e CustomConnectionService : ICustomConnectionService could be resolved to both IDefaultConnectionService and ICustomConnectionService meaning the existing untouched SDK components relying on the base connection service can resolve the implementation by the base interface, while higher level application components can resolve the extended implementation via its extended interface. The other approach here would be to just add the implementation of the extended service so it can be resolved directly without using an interface.

I see your point. Here's a thought: when you want to create interface hierarchy, it's a lot clearer if you take an approach where you append a prefix to each added super interface. Ex. IConnectionService, ICustomConnectionService, IExtendedCustomConnectionService, etc.
If my class DefaultConnectionService, implements IDefaultConnectionService, then my CustomConnectionService requires me to create an interface ICustomConnectionService just for the sake of consistency, where by DefaultConnectionService and CustomConnectionService both implement IConnectionService, it's very immediately obvious that both are implementations, with Default suggesting something provided by the SDK.
I noticed you also added Default suffix to some of the model classes.  Classes in C# are inheritable by nature, no need to designate that in the name (exceptions being abstract classes and similar).

R.e your second point, just checking I understand what you mean, basically the custom service resolution would occur in the startup.cs explicitly like services.AddSingleton<IDefaultConnectionService,CustomConnectionService>, then the service builder essentially just fills in the blanks with try add? I'll take a closer look at the project, thanks!

Correct. These would be the 2 use cases
Case 1:
User registers default implementation using service.AddAgent()

All default services are added with TryAddSingleton (or Transient)

Case 2:
User wants to override ConnectionService using
service.AddAgent(builder =>
{
    build.AddConnectionService<CustomConnectionService>();
});

The extension method AddConnectionService() uses service.AddSingleton.
After builder method is executed, we continue adding all services as in Case 1, only using TryAdd to make sure the overrides aren't overriden.

From the SDK: Using AddSingleton twice will make the dotnet runtime use the latest registration, overriding anything before. This means even if we run our implementation before the builder extension method, service order would be fine, since the extension method AddConnectionService won't use TryAdd.

R.e your first point, my motivation for renaming the interfaces to be generic is that the interface itself could be extended for example we could have ICustomConnectionService : IDefaultConnectionService. The approach to the service registration means a custom connection service implementation i.e CustomConnectionService : ICustomConnectionService could be resolved to both IDefaultConnectionService and ICustomConnectionService meaning the existing untouched SDK components relying on the base connection service can resolve the implementation by the base interface, while higher level application components can resolve the extended implementation via its extended interface. The other approach here would be to just add the implementation of the extended service so it can be resolved directly without using an interface.

I see your point. Here's a thought: when you want to create interface hierarchy, it's a lot clearer if you take an approach where you append a prefix to each added super interface. Ex. IConnectionService, ICustomConnectionService, IExtendedCustomConnectionService, etc.
In the case you're describing, if my class DefaultConnectionService, implements IDefaultConnectionService, then my CustomConnectionService requires me to create an interface ICustomConnectionService just for the sake of consistency, where by DefaultConnectionService and CustomConnectionService both implement IConnectionService, it's very immediately obvious that both are implementations, with Default suggesting something provided by the SDK.
I noticed you also added Default suffix to some of the model classes.  Classes in C# are inheritable by nature, no need to designate that in the name (exceptions being abstract classes and similar).

R.e your second point, just checking I understand what you mean, basically the custom service resolution would occur in the startup.cs explicitly like services.AddSingleton<IDefaultConnectionService,CustomConnectionService>, then the service builder essentially just fills in the blanks with try add? I'll take a closer look at the project, thanks!

Correct. These would be the 2 use cases
Case 1:
User registers default implementation using service.AddAgent()

All default services are added with TryAddSingleton (or Transient)

Case 2:
User wants to override ConnectionService using
service.AddAgent(builder =>
{
    build.AddConnectionService<CustomConnectionService>();
});

The extension method AddConnectionService() uses service.AddSingleton.
After builder method is executed, we continue adding all services as in Case 1, only using TryAdd to make sure the overrides aren't overriden.

From the SDK: Using AddSingleton twice will make the dotnet runtime use the latest registration, overriding anything before. This means even if we run our implementation before the builder extension method, service order would be fine, since the extension method AddConnectionService won't use TryAdd.

Agree with your point around the interface naming, I have updated the implementation to be closer to that of the IdentityServer project you referenced.


implement post-execute

Execute a user-defined command after the generation. The generator needs to save the files before - at the moment it just fills and returns a buffer.

solved and integrated into master


[extractor/owncloud] New Extractor for Owncloud

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
I'm not sure if we want this extractor in yt-dlp, because in Owncloud there is no download block option, and there is always a download button. Since the extractor has been used in my project for years (https://github.com/C0D3D3V/Moodle-DL/blob/main/moodle_dl/downloader/extractors/owncloud.py)  I thought I make a PR, and if you want it you can review it, or reject it.
Also for now I only added two known OwnCloud servers, since I do not know if you want the extractor I did not search for more.
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[x] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

I'm not sure if we want this extractor in yt-dlp, because in Owncloud there is no download block option, and there is always a download button.

That would make it a low priority issue if someone had requested the site, but since you already have a PR, I have no reason not to merge it.

[x]  At least skimmed through contributing guidelines including yt-dlp coding conventions

Pls do so again

related: https://github.com/yt-dlp/yt-dlp/pull/1791

Pls do so again

I have done that twice, but it has not yet become flesh and blood... It will take some more practice and suggestions until I myself know what code style is expected or better said until I apply them on first try. As a newbie, it's hard to see rule breaks, but when mistakes are pointed out a few times, you learn the rules quickly.
By the way, a big thank you from my side as well, I love how you take care of this project and how well you have improved the code base and code quality unlike youtube-dl. I love how fast you guys respond to a PR and that you always improve the code quality of the extractors. I have learned a lot from it myself.

Pls do so again

I have done that twice, but it has not yet become flesh and blood... It will take some more practice and suggestions until I myself know what code style is expected or better said until I apply them on first try. As a newbie, it's hard to see rule breaks, but when mistakes are pointed out a few times, you learn the rules quickly.

I know, and it's ok. I am aware the projects conventions are a bit weird and needs some getting used to. I write that comment mainly to judge if you are willing. For devs who don't make any changes/reply to it, there is no point in me wasting time reviewing

Thank you very much. Sorry for not cleaning it up myself...


Product add price at BCV

The Base Price is on USD on api send price as BCV

Add to response of product and purchase


Update for JDK 8u112b15

Did you had a chance to take a look at at Travis-CI "gunzip: invalid magic"
https://travis-ci.org/anapsix/docker-alpine-java/builds/171843376

Whoops, apparently the Linux build has b15 (macOS is b16). I also added set -ex for easier debugging.

Is there any news?

thanks @praseodym !!


PyNIO backend doesn't play well with open_mfdataset

As reported on StackOverflow: http://stackoverflow.com/questions/38711915/segmentation-fault-writing-xarray-datset-to-netcdf-or-dataframe/
It appears that we can only open a single file at a time with pynio?
Adding a thread lock via lock=True didn't solve the issue.
cc @david-ian-brown

Hi Stephan,
I will look into this issue.
-dave
On Tue, Aug 2, 2016 at 11:01 AM, Stephan Hoyer<EMAIL_ADDRESS>wrote:

As reported on StackOverflow:
http://stackoverflow.com/questions/38711915/segmentation-fault-writing-xarray-datset-to-netcdf-or-dataframe/
It appears that we can only open a single file at a time with pynio?
Adding a thread lock via lock=True didn't solve the issue.
cc @david-ian-brown https://github.com/david-ian-brown
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/issues/936, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7VbedP7yIJsI6DEz0PLOnVJJBDadaKks5qb3gDgaJpZM4Ja1AT
.

Hi Stephan,
I notice that you posted a solution of sorts on stackoverflow. Is this a
real fix or a bandaid for the problem? I ran a few tests with pynio by
itself where there was no problem opening e.g. a GRIB file for reading and
a NetCDF file for writing at the same time.
I am wondering if the issue involves using pynio and the netCDF4 modules at
the same time. Let me know if still think something in pynio is not working
correctly.
Thanks.
-dave
On Tue, Aug 2, 2016 at 1:35 PM, David Brown<EMAIL_ADDRESS>wrote:

Hi Stephan,
I will look into this issue.
-dave
On Tue, Aug 2, 2016 at 11:01 AM, Stephan Hoyer<EMAIL_ADDRESS>wrote:

As reported on StackOverflow: http://stackoverflow.com/
questions/38711915/segmentation-fault-writing-xarray-datset-to-netcdf-or-
dataframe/
It appears that we can only open a single file at a time with pynio?
Adding a thread lock via lock=True didn't solve the issue.
cc @david-ian-brown https://github.com/david-ian-brown
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/issues/936, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7VbedP7yIJsI6DEz0PLOnVJJBDadaKks5qb3gDgaJpZM4Ja1AT
.

The fix I posted on stackoverflow is a bandaid solution -- it requires loading every file into memory all at once, which can be problematic if you have large amounts of data.
It occurs to me that the problem might be that we were attempting to concurrently load data from multiple variables in a single file at once. If this is the issue, then it's something we can work around pretty easily with xarray. I'll run some tests later to verify.


feat!: rename storage inclusion proof to historical storage read

prove_public_value_inclusion forces the caller to produce and pass a historical public storage value. This is subpar because it introduces an oracle requirement for no good reason - there is only one correct value that can be passed.
This PR renames public_value_inclusion to history::storage, and prove_public_value_inclusion to storage_historical_read. The function now returns the historical storage value, so no oracles are needed. If comparison with some value is required, the caller can still manually do this against the return value (though this seems an unlikely use case).
I also updated slightly some of the docs, added some notes, and liberally touched the tests a little bit. This is another example of how annoying it is to not be able to return anything from a private call (forcing us to to perform the assertion inside noir, which makes the test obscure).

Docs Preview
Hey there! 👋 You can check your preview at https://65fca89e9fbbea81b8fdfc6a--aztec-docs-dev.netlify.app

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (64): 130 (+136%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 0f09b63d and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,400
6,412
6,424

l1_rollup_execution_gas
585,733 (+1%)
585,745 (+1%)
585,757 (+1%)

l2_block_processing_time_in_ms
1,337 (+2%)
4,846
9,203 (+1%)

note_successful_decrypting_time_in_ms
181 (-7%)
512 (-9%)
1,000 (-3%)

note_trial_decrypting_time_in_ms
85.7 (-17%)
27.3 (-66%)
:warning: 130 (+136%)

l2_block_building_time_in_ms
17,435 (-2%)
65,418 (-3%)
129,683 (-3%)

l2_block_rollup_simulation_time_in_ms
8,121
28,465 (-1%)
56,168

l2_block_public_tx_process_time_in_ms
9,294 (-5%)
36,893 (-4%)
73,417 (-5%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,903 (-3%)
28,103 (+4%)

note_history_successful_decrypting_time_in_ms
1,246 (-1%)
2,481 (+1%)

note_history_trial_decrypting_time_in_ms
93.7 (-4%)
169 (-12%)

node_database_size_in_bytes
18,645,072
34,869,328 (-1%)

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
247 (-13%)
44,302
28,181

private-kernel-ordering
213 (-2%)
52,805
14,263

base-parity
1,842 (+1%)
128
311

base-rollup
706 (-2%)
165,724
861 (-7%)

root-parity
1,517 (-6%)
1,244
311

root-rollup
50.5 (-3%)
4,359 (-3%)
725

private-kernel-inner
592 (-10%)
73,644
28,181

public-kernel-app-logic
398 (-8%)
35,164
28,151

public-kernel-tail
172 (-2%)
40,862
28,151

merge-rollup
7.15 (-17%)
2,568 (-5%)
861 (-7%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.0 (-1%)
16.1
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.585 (-1%)
0.497
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
45.8 (-4%)
72.6
235 (+1%)
445 (-2%)
877 (+1%)
1,745 (+1%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.470 (-4%)
0.447
0.427 (+1%)
0.417 (-2%)
0.417 (+1%)
0.417 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
53.8 (-4%)
107
340
661 (-1%)
1,315 (-1%)
2,641 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
104
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.478 (-4%)
0.481
0.461
0.455 (-1%)
0.455
0.457 (+1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.7 (-2%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.539 (-2%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes

tx_size_in_bytes
21,949

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
3,111 (-2%)
1,646 (-7%)

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
10.8 (-18%)
1,144 (-7%)


Import problems

Hello! Your Hello, world! example does not compile. When I try go mod tidy or compile I get this:
package main

import (
    "fmt"

    "github.com/Goscord/goscord"
    "github.com/Goscord/goscord/discord"
    "github.com/Goscord/goscord/goscord/gateway"
    "github.com/Goscord/goscord/goscord/gateway/event"
)

var client *gateway.Session

func main() {
    fmt.Println("Starting...")

    client := goscord.New(&gateway.Options{
        Token:   "token",
        Intents: gateway.IntentGuildMessages,
    })

    client.On(event.EventReady, func() {
        fmt.Println("Logged in as " + client.Me().Tag())
    })

    client.On(event.EventMessageCreate, func(msg *discord.Message) {
        if msg.Content == "ping" {
            client.Channel.SendMessage(msg.ChannelId, "Pong ! 🏓")
        }
    })

    client.Login()

    select {}
}

go: finding module for package github.com/Goscord/goscord
go: finding module for package github.com/Goscord/goscord/goscord/gateway
go: finding module for package github.com/Goscord/goscord/goscord/gateway/event
go: finding module for package github.com/Goscord/goscord/discord
go: found github.com/Goscord/goscord/goscord/gateway in github.com/Goscord/goscord v0.0.0-20230503185845-f94f113d6ea7
go: found github.com/Goscord/goscord/goscord/gateway/event in github.com/Goscord/goscord v0.0.0-20230503185845-f94f113d6ea7
go: finding module for package github.com/Goscord/goscord
go: finding module for package github.com/Goscord/goscord/goscord/discord/embed
go: finding module for package github.com/Goscord/goscord/discord
go: **my-package** imports
        github.com/Goscord/goscord: module github.com/Goscord/goscord@latest found (v0.0.0-20230503185845-f94f113d6ea7), but does not contain package github.com/Goscord/goscord
go: **my-package** imports
        github.com/Goscord/goscord/discord: module github.com/Goscord/goscord@latest found (v0.0.0-20230503185845-f94f113d6ea7), but does not contain package github.com/Goscord/goscord/discord
go: **my-package** imports
        github.com/Goscord/goscord/goscord/gateway imports
        github.com/Goscord/goscord/goscord/discord imports
        github.com/Goscord/goscord/goscord/discord/embed: module github.com/Goscord/goscord@latest found (v0.0.0-20230503185845-f94f113d6ea7), but does not contain package github.com/Goscord/goscord/goscord/discord/embed

Looking for the project, I assume is very young. I tried to compile the following code:
package main

import (
    "fmt"

    // Your project directory have this other organization
    "github.com/Goscord/goscord/goscord"
    "github.com/Goscord/goscord/goscord/discord"
    "github.com/Goscord/goscord/goscord/gateway"
    "github.com/Goscord/goscord/goscord/gateway/event"
)

// The rest of code is the same than the previous one

But it stills throwing dependency fetching error because:
go: **my-package** imports
        github.com/Goscord/goscord/goscord/discord imports
        github.com/Goscord/goscord/goscord/discord/embed: module github.com/Goscord/goscord@latest found (v0.0.0-20230503185845-f94f113d6ea7), but does not contain package github.com/Goscord/goscord/goscord/discord/embed

I identified the problematic file. Is github.com/Goscord/goscord/discord/channel.go in the line 4:
$ go build github.com/Goscord/goscord/goscord
goscord/discord/channel.go:4:2: no required module provides package github.com/Goscord/goscord/goscord/discord/embed; to add it:
	go get github.com/Goscord/goscord/goscord/discord/embed

Anyway, I wish the best for the project OwO

Hello, I never noticed example in README was outdated, we mainly redirect to our bot's repository for usage example.
It has been fixed, sorry for that.


Add introduction section and some more info on Crypto.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Adding ref as prop to SidePanel causes repeating uncaught typeErrors

What package(s) are you using?

[x] Cloud & Cognitive
[ ] Legacy/CDAI
[ ] Legacy/Security

Detailed description

Describe in detail the issue you're having.

Using SidePanel in slide-over mode, we wanted to add a ref property (from react useRef) to feed a useClickAway hook.   The addition of the ref (with or without sending to useClickAway) causes the console to flood with errors
Uncaught TypeError: Cannot read properties of undefined (reading 'style')
    at onAnimationEnd (SidePanel.js:252)
    at HTMLUnknownElement.callCallback

Is this issue related to a specific component?

SidePanel

What did you expect to happen? What happened instead? What would you like to
see changed?

What browser are you working in?

Mac Chrome

What version of the @carbon/ibm-cloud-cognitive packge are you using?

0.62.6

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

IBM Watson Assistant
Steps to reproduce the issue
will create a CodeSandbox

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/github/carbon-design-system/ibm-cloud-cognitive/tree/main/examples/cloud-cognitive/codesandbox

Additional information

Screenshots or code
Notes

One thing to note is that this functionality is only included if you use the includeOverlay prop


pre header added.

added a preheader with dynamic stars fetching.

Thanks @archit00sharma for this PR. I talked with @thgnw and we agree that this doesn't look so good on the documentation page. For that reason I'm closing the PR. Thanks nonetheless!


Add support for > 32 qudits to cirq.sample_state_vector. Fix for #6031

When the number of qudits > 32 do the transpose operation on the flattened array.
I expect this to be slower than np.transpose(tensor, axes) so I'm keeping the use of np.transpose when we can.
I also rewrote the _prob in a more efficient way. The old way did a lot of extra work when reordering the axes is all that is needed.

Fix for #6031.

@rht  I refactored the code so that both locations call the method.
I don't know why cirq/study/resolver_test.py::test_custom_value_not_implemented is failing but it seems to be not related to my PR since it's already failing at head
$ git branch
* master
$ pytest cirq/study/resolver_test.py -k test_custom_value_not_implemented
=========================================================================================== test session starts ===========================================================================================
platform linux -- Python 3.10.11, pytest-7.3.1, pluggy-1.0.0
Using --randomly-seed=4244270818
configfile: pyproject.toml
plugins: forked-1.6.0, cov-4.0.0, xdist-2.2.1, randomly-3.12.0, anyio-3.6.2, asyncio-0.21.0
asyncio: mode=strict
collected 47 items / 46 deselected / 1 selected

cirq/study/resolver_test.py x                                                                                [100%]

==================================================================================== 46 deselected, 1 xfailed in 0.93s ====================================================================================

@rht this is now ready for merging

Nit: I searched for the files named *_util.py; there are 2 occurrences. While *_utils.py has 7 occurrences. As such, I think it should be simulation_utils.py?

this is ready for another look

The 32 qubit dimension constraint for numpy is more annoying that I initially thought. Another example: cirq.unitary() fails when the dimension of circuit is > 16. Note that a 2**17 x 2 ** 17 numpy array of complex64 takes ~128gb, so it's not unreasonable to expect that the protocol should work. We can add this example as part of the issue and track it in a separate PR though.
In [1]: import cirq
circuit = cirq.Cir
In [2]: q = cirq.LineQubit.range(17)
In [3]: circuit = cirq.Circuit(cirq.Z.on_each(*q))
In [4]: cirq.unitary(circuit)

@tanujkhattar wouldn't computing such a unitary generally (when the unitary is not a tensor product) take a couple of hours/days? ...  a single multiplication of two a $2^{17} \times 2^{17}$ matricies will take around half an hour  $8^{17} \textit{operations} \approx 2251 \textit{seconds} \approx 37 \textit{minutes}$ (assumming $10^{12}$ operations per second).

anyway lets keep track of all the places suffering from this problem in the original issue.

can we merge this PR?

@tanujkhattar ping


allowing compliance testing to be submitted one week after submission

WG approved.

if fail, please submit within 1 week of deadline and file an issue against your own submission with explainations if any.


How to preprocess the 3dpw dataset?

Sorry to bother you again. :(
I have a question about the SMPL parameters in the 3dpw dataset you offered.
I noticed the pose parameter (72,) and trans parameters (3,) differ from the official 3dpw dataset. So, I guess there are some preprocess in making the annotation. Could you tell me how you preprocess it?
Thanks in advance.~

Hi! Please check here:
https://github.com/hongsukchoi/Pose2Mesh_RELEASE/blob/master/data/3dpw2coco.py
Best regards,
Hongsuk Choi
https://hongsukchoi.github.io/
2022년 9월 29일 (목) 오전 1:34, Jiahao Li @.***>님이 작성:

Sorry to bother you again. :(
I have a question about the SMPL parameters in the 3dpw dataset you
offered.
I noticed the pose parameter (72,) and trans parameters (3,) differ from
the official 3dpw dataset. So, I guess there are some preprocess in making
the annotation. Could you tell me how you preprocess it?
Thanks in advance.~
—
Reply to this email directly, view it on GitHub
https://github.com/hongsukchoi/3DCrowdNet_RELEASE/issues/19, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHNGKW3CNNVGWHDXLBCGHJ3WAUS5XANCNFSM6AAAAAAQYM5LTQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


add: docker-compose support

This PR adds possibility to use docker compose deployment setup.

closing, merged to #365


Fix/clear check cache when config change

comment: Fix/clear check cache when config change

@caojiajun pls help me review it.
thanks

thank you~

thank you~

you're welcome ~
😅


(maint) allow ordering on non-extracted fields

This changes our paged sql function to do
select paged_results.* from (select blah from facts order by baz)

instead of
select paged_results.* from (select blah from facts) order by baz

the latter of which will not work without a late projection.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1150/

Looking.

Is this just me? I'm not 100% but this should work now right?
# curl -X GET 'http://localhost:8080/v4/facts' -d 'query=["extract","certname",["=","certname","kb.local"]]&order_by=[{"field":"environment"}]'

<meta http-equiv="Content-Type" content="text/html;charset=ISO-8859-1"/>
<title>Error 500 </title>

HTTP ERROR: 500

Problem accessing /v4/facts. Reason:

    org.postgresql.util.PSQLException: ERROR: column "environment" does not exist
  Position: 1042

Powered by Jetty://

@kbarber yeah, good catch. The issue here is that there are a number of cases where we alias a column when we extract it (in this case fact_environment to environment). When that aliasing doesn't happen the ordering fails. I'm closing the patch for now.

@wkalt do we have a ticket to track this bug otherwise? The whole order-by not working for this case seems bad.


Add Linux makefile

This is copied from the OP2Utility project, with minor modifications:
Compiler output file changed to: op2archive
Compiler output is executable file rather than library
Include folder added for OP2Utility
Library added: stdc++fs
This closes #5.

Thanks for adding the Makefile. Let me know how it goes.

I was able to successfully produce an executable on Linux using native compilers (including updates and fixes from previous open branches)

Good news. Cross platform archives are now in service!
I'm going to spend some time cleaning the code and then I'll update OP2MapImager's version of OP2Utility.


Empty array not being inserted with empty optional list field

Describe the bug
According to #1988 and #782, optional fields should be inserted as an empty string or array. An empty string is being inserted for optional string widgets, but this is not happening for list widgets.
To Reproduce
Use the config below and create a new page.
Expected behavior
A key with an empty array should be inserted for lists when no content is added.
Applicable Versions:

Netlify CMS version: 2.9.7
Git provider: GitHub

CMS configuration
backend:
  name: git-gateway
branch: master
media_folder: static/media
public_folder: /media

collections:

  - label: Pages
    name: pages
    label_singular: Page
    folder: content/content
    create: true
    editor:
      preview: false

    fields:
      - label: Title
        name: title
        widget: string

      - label: Body
        name: body
        widget: markdown
        required: false

      - label: Optional Field
        name: optional_field
        default: ''
        required: false
        widget: string
        hint: This has a default empty string. It will be inserted as an empty string.

      - label: Sidebar
        name: sidebar
        widget: list
        default: []

        fields:
          - label: Heading
            name: heading
            widget: string

          - hint: "Displays this content in a box on the page"
            label: Highlight
            name: highlight
            widget: boolean

          - hint: "Put some sidebar text here"
            label: "Block content"
            name: block_content
            widget: markdown

Additional context

@freshyill  default option for the list widget only works on the list field(s). Prior to PR #2374 an empty [{}] would have been automatically generated for your list widget, but that was forcing an empty field(s) for the list widget in the editor page, hence the PR.

Related to #1407


Document truncation of Journal read count

The read count is truncated from Long to Int. Document this behavior.
In ServiceFabricPersistence.ServiceFabricEventJournal.GetMessagesAsync
int m = max > int.MaxValue ? int.MaxValue : (int)max;

This one is a bit ugly.
The JVM Api is designed around long values for journal sequence numbers, which is all fine.
In our specific implementation here, there are two issues;

We use Linq to objects to filter the journal, l2o uses integers for its methods, thus, the value needs to be truncated.
We are probably not going to ever be able to deserialize more than int.Max entries from servicefabric. as that would probably take a couple of days and a bizarre amount of memory to deserialize such object.

So, are we OK with the long to int truncation?
cc. @clca

It is probably a corner case considering that it would assume that an Actor would need 100s GB of storage for its state. We may want to catch this case at the plug-in level and return some sort of error.


Keptn CLI Command to generate service/integration based on keptn-sandbox/keptn-service-template-go

Tasks
Depending on the keptn cli version, download and use the template from https://github.com/keptn-sandbox/keptn-service-template-go (by matching the Keptn CLI version with a GitHub release of the template; as a fallback: use master branch) and requests the following inputs from the user:

Service name
Docker Image Name (organization/image-name)
comma separated list of cloud-events to listen for (e.g., sh.keptn.event.deployment.triggered)
TBD

Keptn CLI then creates a directory service-name with the appropriate contents of the template, and replaces the occurrences of several strings according to the README file of the template (using go-templates).
Definition of Done

[ ] Task from above is implemented in the CLI
[ ] Keptn service template is an actual template with placeholders
[ ] Keptn service template contains a docker-compose file to run the service locally (distributor + service)
[ ] Keptn service template is generated using the CLI command keptn generate keptn-service  [--keptn-service-template-version=master]
[ ] Integration Test is added to keptn/keptn: Use the implemented CLI command with the master branch of the keptn-service-template and it should at least do the following:

go test ...
docker build …
docker run …
Run the service and send a cloud-event (e.g., one from the test-events directory of the template) to the service; service should respond with status code 200
Update documentation at https://keptn.sh/docs/0.8.x/integrations/custom_integration/ (see keptn/keptn.github.io repo) to include the new command for generating a keptn-service

I'll take this for LFX mentorship program.

I'll take this for LFX mentorship program.

Hey @jetzlstorfer, I will like to take this issue up for LFX mentorship.
Should I start drafting a proposal for the same?

Hi @rahul799 - this is issue is reserved for the LFX mentorship. We will start assigning it once the application period closes and the mentors have made their decision for the mentorship. If you would like to apply, please do so via the LFX mentorship program and add your proposal to the application! Thank you!

Yeah sure. Thankyou @jetzlstorfer.

Hey @jetzlstorfer !
I have been selected as a mentee for this project under LFX Mentorship Program, would like to get started on this issue!


Introductory essay repeatedly uses "prevalence rates"

"Prevalence rate" is redundant. Prevalence means rate of occurence in NZ English.

Merged PR and closed issue


Issues with new Android support lib v23.2.0

There's a few issues I'm having with the new support library. First is when i perform   calendarView.goToCurrentDay(); I get this error
Process: PID: 27352 java.lang.IllegalStateException: The application's PagerAdapter changed the adapter's contents without calling PagerAdapter#notifyDataSetChanged! Expected adapter item count: 10000, found: 402 Pager id: ffffffff Pager class: class com.p_v.flexiblecalendar.MonthViewPager Problematic adapter: class com.antonyt.infiniteviewpager.InfinitePagerAdapter
The other issue I'm having is with calendarView.refresh() which doesn't seem to be working anymore. It was working flawlessly until the new support lib broke the world ;) JK, any help or updates would be most gratefully appreciated!
To note as well when I page a few months to the right my events start to actually show in the calenderView. I'm not sure what to be looking for on this particular issue but I'm positive it's related to the new support library.

Probably helpful to paste the full stacktrace.
(not flexiblecalendar dev but I'm not getting that issue with 23.2.0)

@Ammonious I'll look into it. Tried few cases and got the same error.
@twig Looks like it's reproducible here. In case you want to have a look, I have pushed the error producing code to a new branch support-lib-err. I just upgraded the library to android support library v23.2.0. Calendar 1 in the demo gives the error. Ignore the back button in the first calendar.

ahh interesting, thanks for the details.

23.2.1 was just released

Yes, but the issue persists.

I've started to use this library in my recent project. But this issue is crashing app every time. @p-v will it be possible to give a solution anytime soon? Btw great work

@AnwarShahriar
Fixes are on their way. I have got most of the things working. I will push the changes in a couple of days.

The fix has been pushed. Feel free to post if any issues are found.
@twig You might want to take look at this :smiley:

super.notifyDataSetChanged();

wow, so that's all it needed lol
nice find

You're great man @p-v :+1:


state-db key value storage

This PR extract the state-db logic from #3774 to try to make it more reviewable.
For general goal of key value storage, please refer to #3774.
This PR cover state-db additions.
As an extract from #3774, some TODO are present on state machine and client interfaces and tests are lacking.

dedicated journal  (KvJournalRecord) are added (this could be merge in existing journal but focus here on non breaking change) to store block change, and pruning.
fork are handled with a sequential 'branch id' used as a per value index for memory historic of data (see historical-data tree).
The branch id are not persisted since this is state db.
pruning which is more a client problematic is handled in this pr in the state machine test over a test backend.
apply commit operation got new variants, because some key value content do not make sense.

Note that some pruning enhancement described in #3774 could be added.

Closing this PR because Child trie associated issue will be work around in a way that does not require this.
It make this a non prioritary thing.
Generally it adds complexity, and would require its own crate separation.
Some other part could maybe get merge.
Rethinking the design could also be interesting.


Recal folder contains 2 index files per BAM

Each BAM file in the recal folder has both a .bai and .bam.bai file. So far, only seen this in an exome, not a genome. Cause uncertain, it's possible that maybe one of the exome callers is creating another index?

BQSR creates a .bai file.
The .bam.bai file is an input file for the samtools caller, still not sure where it comes from.


Release Google.Cloud.Iap.V1 version 2.7.0

Changes in this release:
New features

Add IServiceCollection extension methods for client registration where an IServiceProvider is required. (commit 022fab2)

Created release for Google.Cloud.Iap.V1-2.7.0


Volume up to 120%.

Is your feature request related to a problem? Please describe.
If a video is played with a sound that is too quiet, you do not have the option of making it louder - only 100%.
I can control this in the hall, but not for the input level in Zoom. In the VLC player you have the option to set the volume up to 120%.

This would be useful for us too.
We get most of our Talks off Stream. In some of the videos the volume is too low.
As mentioned above, we can use the sound hall amplifier to increase the volume, but ones on Zoom say that the volume is too low.
Thank you!

Thanks for the suggestion. This would require dynamically increasing the amplitude of the source samples, with the risk of distortion depending on the headroom in the original audio track. It may be done by taking separate control of the audio and video components of the source and then using NAudio (already included in OnlyM) to increase the volume.


Provide an API for instrumenting imports everywhere

Potential inspiration:
https://github.com/agronholm/typeguard/blob/master/src/typeguard/_importhook.py
Could go the typeguard route of making the install hook return a context manager to allow user code to only apply the defer magic to a selection of modules imported in a block, instead of unilaterally.
This will also require partially rewriting the AST transformer or making a new one, as the current one, DeferredInstrumentor, is only geared towards instrumenting imports within with defer_imports.until_use: ... blocks.

After investigation, this seems too difficult to do even with a rewrite.

Managed to get this to work by wrapping consecutive import statements with the until_use context manager via the AST transformer. Configuration is transferred through kwargs for install_import_hook(), which goes into a container into a contextvar. It's read after a module spec is found but before loading begins.
Only thing left is to solidify the API and write tests for it.

Added with #4, though tests are still a work in progress.


[spikeglx] explicitly specify test dataset - fix master

This PR fixes the unittest for the current master branch. These are failing because a new spikeglx test dataset was added to gin and automatically included in the tests here.
Support for the new test dataset will be introduced via https://github.com/NeuralEnsemble/python-neo/pull/1045

Doing this you test only one stream.
spikeglx can be multi stream.
#1045 should also fix the problem in master.

Julia if you merge this
https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/70
then we can remove this PR.

It's merged.


Bug: no response adding photos from library when a batch of pictures are created

On android-fdroid, when a batch of pictures is created, either from my photo library or from camera, I cannot add more photos from library. To reproduce it, simply press plus icon near the right bottom corner, then press the picture icon in the active camera stage. Nothing happens after that.

@hammerfunctor if i understand correctly once you successfully imported images using the "+" the interface is not "reactive" anymore?

Exactly.

@hammerfunctor thanks i ll fix it ASAP

@hammerfunctor sadly i cant reproduce your error. Can you provide details:

App version
OS and version
language

It turns out to be a permission issue. In the first startup I disallow the access to all my files and grant only access to media files. Reset the permission, and the problem is gone.

@hammerfunctor i also tried what you mentioned and only grant access to media files, still could not reproduce your issue

Here is a demo
screen.webm

@hammerfunctor awesome i reproduced the issue will fix!

Thanks!


Merge validate httproute

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
9dca4d359ae0062111489b04cf9dadeb881f4bff
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio

@ramaraochavali


Upgrade Works OpenSearch engine from m6g.large.search to m7g.large.search

The m7g engine is based on the newer Graviton 3 processor and provides better performance.

Upgrade complete.
m7g.large.search means:

Upgraded from Graviton2 to Graviton3 processors.
25% better performance compared to previous configuration.
Using DDR5 memory with 50% higher memory bandwidth than m6g's DDR4


mouse clicks outside the join button

self explanatory, when it tries to join, the mouse clicks outside the button box

one reason why this may happen is due to different monitor sizes.
To fix this you could try using snipping tool and taking a picture of the join button and replace it by going into the assets folder and changing the image pro.png with your image. Make sure you rename your updated image to pro.png as well


Works weirdly in battle arenas, and sometimes won't bring up the menu

In battle arenas, sometimes the menu pops up after the players get into a game, but before the game starts, stopping the game from starting. Noticed in steve dittos, where the menu would pop up twice, and let me select the skin for both players.

This should be vaguely less bad in the current version as the skin menu has improved in consistency. Still needs work on filtering out CPUs randomly getting Steve + Filtering out online opponents (like in battle arenas).


No longer use Rollbar Middleware

This is handled by Pliny::Middleware::RescueErrors now, see https://github.com/interagent/pliny/pull/265

LGTM 👍


Better log summary at end of run

Discussed in https://github.com/mmgoodnow/cross-seed/discussions/110

Originally posted by ajhorvath20 July 20, 2021
Hi, is it possible to refine the logs output
today at 17:55:50 warn: Found Movie.2020.1080p.HMAX.WEB-DL.DD5.1.H.264-CMRG on abc123 - skipping incomplete torrent
today at 18:16:48 info: Done! Found 1 cross seeds from 23 original torrents
today at 18:16:50 info: Validating your configuration...

I see same output when the injected or when its ignored because of the incompletion.
Thanks

Hopefully simple request.
The output e.g.
Found 71 cross seeds from 47 original torrents
can this be expanded similar to below.
Found 71 cross seeds from 47 original torrents
10 new torrents injected
61 pre-existing torrents ignored.
Would need to add either 1 or two new counters depending if its injected or existing already.
Easiest would be an injected check, and then the final "pre-existing" value would just be total - injected.

yeah we're just not going to do this.


Unable to find module on Python 3.6 and 3.7 but working fine on 3.8 and 3.9

Hello, me and @kuelumbus are currently testing Windows wheel for  https://github.com/kuelumbus/rdkit_platform_wheels/ . After some test we found that the wheel working just fine on Python 3.8 and 3.9, but on 3.6 and 3.7 it gives the following error.
(rdkit-win-36) C:\Users\radif\Documents\Python>python
Python 3.6.8 (tags/v3.6.8:3c6b436a57, Dec 24 2018, 00:16:47) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from rdkit import Chem
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\radif\Documents\Python\rdkit-win-36\lib\site-packages\rdkit\__init__.py", line 20, in <module>
    _delvewheel_init_patch_0_0_14()
  File "C:\Users\radif\Documents\Python\rdkit-win-36\lib\site-packages\rdkit\__init__.py", line 17, in _delvewheel_init_patch_0_0_14
    WinDLL(os.path.join(libs_dir, lib))
  File "C:\Users\radif\AppData\Local\Programs\Python\Python36\lib\ctypes\__init__.py", line 348, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: [WinError 126] The specified module could not be found

More detail on kuelumbus/rdkit_platform_wheels#8
Any idea why this happened? Thanks.

@radifar Do you still need help with this? Based on https://github.com/kuelumbus/rdkit_platform_wheels/issues/8#issuecomment-913196258, it looks like you figured out the cause of the problem.


Add ability to specify file key in form data

For example, some backends would rather accept file data in an "image" form field, rather than the current default of "file".

he has hardcoded the name in android codes..as name with images . I was facing the problem with multer in nodejs since it was required name type.
http://stackoverflow.com/questions/31530200/node-multer-unexpected-field
check out this link for more info.


Automate builds and NuGet publications

When #8 and #9 are completed, it will be possible to publish builds to NuGet.
The goal of this ticket is to automate the process of building and publishing a new version of the project to NuGet.
Piwik relies on travis-ci for such tasks. For simplicity, It would be nice to use the same tool.
Some preliminary tests on travis-ci has been performed by @tetsuo13, see results in #8
The implemented solution should be easily extended to include #7

See https://github.com/piwik/piwik-dotnet-tracker/issues/65 for a first step


Decode function, which fails, if some data left

Right now decode will return ok, even if some input data is left.
I'm writing a decoder, which tries to decode the tx input and logs not knowing exact signature, so it tries several signatures and picks whatever decodes successfully. But current implementation is wrong in this context: it can return "Ok", even if the data is not fully corresponds to the types provided
I suggest adding some function, which will also check, that it decodes everything, or returns error, because implementing it requires implementation details (knowing offset). Also I need to decode logs, so I've added additional implementation there too, and if I missed some other decodes, I will happily add other functions too
Current implementation is just the easiest way I thought this could be done, but I'm open for review and will fix it if you have a better idea. Naming is for debate too

The change makes sense to me but we should probably do it in a backward compatible way. I'd keep the existing public functions as they are now and add new functions with the extra check.

I've added some basic tests and left the original functions as is

One more nitpick. Sorry if I didn't make that clear in the previous comment. We should refactor out the common (copy pasted) parts between the base function and the _whole addition. What I meant before was that the behavior of the base function shouldn't change. We can change the implementation. Maybe the base function can call _whole internally or we need a third function that can be called by both.

that's what I did in the first attempt, so I didn't quite get what you're were asking for 😅
isn't the first version what you suggesting?
we make decode_offset (private function), and call it from decode and decode_whole, and similar with events.
I tried to use errors, but then I'll need to add another error in the enum and that's probably not the most backward compatible way(?)

You're right. I must have not paid attention when I read the first commit and thought it changed the behavior of the existing function. Just looked at the commit again and it looks fine.

No problem :)
I've returned the old version


feat: use modern fake timers by default

Summary
As announced in the v26 blog post

Test plan
Green CI

Fake promises test fails, which I think aligns with https://github.com/facebook/react-native/issues/29303, but I didn't go into details, so may be wrong.

@thymikee dunno about that issue, the issue here is the test uses runAllImmediates which doesn't work with modern timers. Just using runAllTimers instead fixes it


Add name and class property to DateInput

It would be great to have a name and class property on DateInput. That would enable manual input/submit when having disabled JS and use something like bootstrap CSS to enable styling of the input box and to enable a red box upon wrong input.

There actually is a class property on DateInput.
I can't add name to the actual <input>, but there could be a fake input for form submission. What format should the value be, unix timestamp?

Great to have class already. Value I would set to ISO format.

Ah, but the class is on the parent, not on the input field.

So the input field is not getting the right CSS

@probablykasper Here a proposal: add properties name and inputClass (or innerClass) to input tag.

Adding name to the input tag would cause lots of headaches, so I would only go with the fake input solution.
It would be nice if there's a way to avoid having multiple class properties

I doubt that you can avoid multiple class properties. Some need to go to the outer div, some to the input field if displayed. svelty-picker supports "inputClasses" for "input css class string".
What's the issue with the name property? Can't it simply be:
<script lang="ts">
    export let name = ''
</script>

<input name={name} />

See

https://github.com/probablykasper/date-picker-svelte/issues/44

It might also make sense to have an <DateInput inputId="id" /> to be able to use <label for="id" /> for accessibility.
Maybe right next to <DateInput inputClass="form-control" /> ;-)

My last try regarding name: Maybe just use <DateInput inputName="creation" /> to assign a name to the input field as backup when SSR is used and no JS on client is allowed.
I don't worry about the format, as it is written in the input already, so the server can deal with it. And if you really want a fixed format, the Date().toISOString() is probably the common standard in JS/Node/JSON.

I forked and added a type check and inputClass, inputStyle, inputName. You want a pull request?

Nice! No thanks, I'd need to think about it

+1
It would be a nice feature, I could solve it only this way:
<style lang="postcss">
	:global(.birthdayPlaceholder input) {
		@apply !w-2/3 !p-3 !pl-5 !my-4 !text-sm !text-gray-400 !border-solid !shadow-inner !placeholder-inherit !bg-gray-50;
	}
</style>

which looks terrible.