[QUERY] What should I do with the "Cannot create receive link from a closed session" AmqpException

Query/Question
We have a user running into this AmqpException recently while consuming from an EventHub topic.  This error shows up in the error handler.
com.azure.core.amqp.exception.AmqpException: connectionId[MF_3a9715_1656513613229] sessionName[xxx/ConsumerGroups/yyy/Partitions/7] entityPath[xxx/ConsumerGroups/yyy/Partitions/7] linkName[7_896631_1656513613230] Cannot create receive link from a closed session., errorContext[NAMESPACE: aaa-bbb-ehub.servicebus.windows.net. ERROR CONTEXT: N/A, PATH: xxx/ConsumerGroups/yyy/Partitions/7]
	at com.azure.core.amqp.implementation.ReactorSession.createConsumer(ReactorSession.java:336)
	at com.azure.messaging.eventhubs.implementation.EventHubReactorSession.createConsumer(EventHubReactorSession.java:100)
	at com.azure.messaging.eventhubs.implementation.EventHubReactorAmqpConnection.lambda$createReceiveLink$1(EventHubReactorAmqpConnection.java:125)
	at reactor.core.publisher.MonoFlatMap$FlatMapMain.onNext(MonoFlatMap.java:125)
	at reactor.core.publisher.FluxMapFuseable$MapFuseableSubscriber.onNext(FluxMapFuseable.java:127)
	at reactor.core.publisher.Operators$MonoSubscriber.complete(Operators.java:1816)
	at reactor.core.publisher.MonoFlatMap$FlatMapInner.onNext(MonoFlatMap.java:249)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.complete(MonoIgnoreThen.java:284)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onNext(MonoIgnoreThen.java:187)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:232)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onComplete(MonoIgnoreThen.java:203)
	at reactor.core.publisher.SerializedSubscriber.onComplete(SerializedSubscriber.java:146)
	at reactor.core.publisher.SerializedSubscriber.onComplete(SerializedSubscriber.java:146)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.onComplete(FluxTimeout.java:234)
	at reactor.core.publisher.MonoNext$NextSubscriber.onComplete(MonoNext.java:102)
	at reactor.core.publisher.MonoNext$NextSubscriber.onNext(MonoNext.java:83)
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.onNext(FluxFilterFuseable.java:118)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replayNormal(FluxReplay.java:814)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replay(FluxReplay.java:898)
	at reactor.core.publisher.FluxReplay$ReplayInner.request(FluxReplay.java:1465)
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.request(FluxFilterFuseable.java:191)
	at reactor.core.publisher.MonoNext$NextSubscriber.request(MonoNext.java:108)
	at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.request(Operators.java:2158)
	at reactor.core.publisher.SerializedSubscriber.request(SerializedSubscriber.java:151)
	at reactor.core.publisher.SerializedSubscriber.request(SerializedSubscriber.java:151)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onSubscribe(MonoIgnoreThen.java:134)
	at reactor.core.publisher.SerializedSubscriber.onSubscribe(SerializedSubscriber.java:65)
	at reactor.core.publisher.SerializedSubscriber.onSubscribe(SerializedSubscriber.java:65)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.onSubscribe(FluxTimeout.java:154)
	at reactor.core.publisher.MonoNext$NextSubscriber.onSubscribe(MonoNext.java:70)
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.onSubscribe(FluxFilterFuseable.java:87)
	at reactor.core.publisher.FluxReplay.subscribeOrReturn(FluxReplay.java:1114)
	at reactor.core.publisher.FluxReplay.subscribe(FluxReplay.java:1086)
	at reactor.core.publisher.FluxAutoConnectFuseable.subscribe(FluxAutoConnectFuseable.java:61)
	at reactor.core.publisher.Mono.subscribe(Mono.java:4338)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:255)
	at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:51)
	at reactor.core.publisher.MonoFlatMap$FlatMapMain.onNext(MonoFlatMap.java:157)
	at reactor.core.publisher.FluxMapFuseable$MapFuseableSubscriber.onNext(FluxMapFuseable.java:127)
	at reactor.core.publisher.MonoPeekTerminal$MonoTerminalPeekSubscriber.onNext(MonoPeekTerminal.java:180)
	at reactor.core.publisher.FluxHide$SuppressFuseableSubscriber.onNext(FluxHide.java:137)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.complete(MonoIgnoreThen.java:284)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onNext(MonoIgnoreThen.java:187)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:232)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onComplete(MonoIgnoreThen.java:203)
	at reactor.core.publisher.SerializedSubscriber.onComplete(SerializedSubscriber.java:146)
	at reactor.core.publisher.SerializedSubscriber.onComplete(SerializedSubscriber.java:146)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.onComplete(FluxTimeout.java:234)
	at reactor.core.publisher.MonoNext$NextSubscriber.onComplete(MonoNext.java:102)
	at reactor.core.publisher.MonoNext$NextSubscriber.onNext(MonoNext.java:83)
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.onNext(FluxFilterFuseable.java:118)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replayNormal(FluxReplay.java:814)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replay(FluxReplay.java:898)
	at reactor.core.publisher.FluxReplay$ReplayInner.request(FluxReplay.java:1465)
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.request(FluxFilterFuseable.java:191)
	at reactor.core.publisher.MonoNext$NextSubscriber.request(MonoNext.java:108)
	at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.request(Operators.java:2158)
	at reactor.core.publisher.SerializedSubscriber.request(SerializedSubscriber.java:151)
	at reactor.core.publisher.SerializedSubscriber.request(SerializedSubscriber.java:151)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onSubscribe(MonoIgnoreThen.java:134)
	at reactor.core.publisher.SerializedSubscriber.onSubscribe(SerializedSubscriber.java:65)
	at reactor.core.publisher.SerializedSubscriber.onSubscribe(SerializedSubscriber.java:65)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.onSubscribe(FluxTimeout.java:154)
	at reactor.core.publisher.MonoNext$NextSubscriber.onSubscribe(MonoNext.java:70)
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.onSubscribe(FluxFilterFuseable.java:87)
	at reactor.core.publisher.FluxReplay.subscribeOrReturn(FluxReplay.java:1114)
	at reactor.core.publisher.FluxReplay.subscribe(FluxReplay.java:1086)
	at reactor.core.publisher.FluxAutoConnectFuseable.subscribe(FluxAutoConnectFuseable.java:61)
	at reactor.core.publisher.Mono.subscribe(Mono.java:4338)
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:255)
	at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:51)
	at reactor.core.publisher.Mono.subscribe(Mono.java:4338)
	at reactor.core.publisher.FluxFlatMap.trySubscribeScalarMap(FluxFlatMap.java:203)
	at reactor.core.publisher.MonoFlatMap.subscribeOrReturn(MonoFlatMap.java:53)
	at reactor.core.publisher.InternalMonoOperator.subscribe(InternalMonoOperator.java:57)
	at reactor.core.publisher.MonoFlatMap$FlatMapMain.onNext(MonoFlatMap.java:157)
	at reactor.core.publisher.Operators$MonoSubscriber.complete(Operators.java:1816)
	at com.azure.core.amqp.implementation.AmqpChannelProcessor.subscribe(AmqpChannelProcessor.java:251)
	at reactor.core.publisher.InternalMonoOperator.subscribe(InternalMonoOperator.java:64)
	at reactor.core.publisher.FluxRepeatPredicate$RepeatPredicateSubscriber.resubscribe(FluxRepeatPredicate.java:120)
	at reactor.core.publisher.FluxRepeatPredicate$RepeatPredicateSubscriber.onComplete(FluxRepeatPredicate.java:101)
	at reactor.core.publisher.Operators$MonoSubscriber.request(Operators.java:1907)
	at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.request(Operators.java:2158)
	at com.azure.messaging.eventhubs.implementation.AmqpReceiveLinkProcessor.requestUpstream(AmqpReceiveLinkProcessor.java:421)
	at com.azure.messaging.eventhubs.implementation.AmqpReceiveLinkProcessor.lambda$onNext$7(AmqpReceiveLinkProcessor.java:254)
	at reactor.core.publisher.LambdaSubscriber.onComplete(LambdaSubscriber.java:132)
	at reactor.core.publisher.FluxSubscribeOn$SubscribeOnSubscriber.onComplete(FluxSubscribeOn.java:166)
	at reactor.core.publisher.FluxDistinct$DistinctFuseableSubscriber.onComplete(FluxDistinct.java:501)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replayNormal(FluxReplay.java:805)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replay(FluxReplay.java:898)
	at reactor.core.publisher.FluxReplay$ReplaySubscriber.onComplete(FluxReplay.java:1273)
	at reactor.core.publisher.FluxPeek$PeekSubscriber.onComplete(FluxPeek.java:260)
	at reactor.core.publisher.FluxPeek$PeekSubscriber.onComplete(FluxPeek.java:260)
	at reactor.core.publisher.FluxMap$MapSubscriber.onComplete(FluxMap.java:142)
	at reactor.core.publisher.FluxDistinctUntilChanged$DistinctUntilChangedSubscriber.onComplete(FluxDistinctUntilChanged.java:173)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replayNormal(FluxReplay.java:805)
	at reactor.core.publisher.FluxReplay$SizeBoundReplayBuffer.replay(FluxReplay.java:898)
	at reactor.core.publisher.ReplayProcessor.tryEmitComplete(ReplayProcessor.java:465)
	at reactor.core.publisher.SinkManySerialized.tryEmitComplete(SinkManySerialized.java:64)
	at reactor.core.publisher.InternalManySink.emitComplete(InternalManySink.java:68)
	at com.azure.core.amqp.implementation.handler.Handler.close(Handler.java:132)
	at com.azure.core.amqp.implementation.handler.LinkHandler.onLinkFinal(LinkHandler.java:78)
	at com.azure.core.amqp.implementation.handler.ReceiveLinkHandler.onLinkFinal(ReceiveLinkHandler.java:202)
	at org.apache.qpid.proton.engine.BaseHandler.handle(BaseHandler.java:182)
	at org.apache.qpid.proton.engine.impl.EventImpl.dispatch(EventImpl.java:108)
	at org.apache.qpid.proton.reactor.impl.ReactorImpl.dispatch(ReactorImpl.java:324)
	at org.apache.qpid.proton.reactor.impl.ReactorImpl.process(ReactorImpl.java:291)
	at com.azure.core.amqp.implementation.ReactorExecutor.run(ReactorExecutor.java:86)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)

We were using the EventProcessorClient to read.  This is how we created the client:
eventProcessorClient = new EventProcessorClientBuilder()
      .connectionString(connectionStr)
      .consumerGroup(consumerGroup)
      .processEventBatch(processPartitionBatch, BATCH_SIZE, Duration.ofSeconds(MAX_WAIT_SECOND))
      .processError(handleError)
      .processPartitionClose(handlePartitionClose)
      .checkpointStore(checkpointStore)
      .trackLastEnqueuedEventProperties(true)
      .buildEventProcessorClient()

In our error handler, we restart the application if we run into errors that are not on the ignore list.  I did some research on this error message but couldn't find any info/solution specific to this error.  I am wondering if you could tell me what caused this error and what we should do when this happens.
Setup (please complete the following information if applicable):

OS: Linux
IDE: IntelliJ
Library/Libraries:
azure-messaging-eventhubs:5.10.1
azure-core-amqp:2.3.2

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

Hi @mzesri thank you for posting this question on GitHub.  You can find the Event Hubs troubleshooting guide in the repo here: https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/eventhubs/azure-messaging-eventhubs/TROUBLESHOOTING.md which has more details on investigating and handling this exception.
Someone from the Event Hubs SDK team will follow up with you shortly.  /cc @Azure/azsdk-sb-java

@mzesri Can you please share a code snippet of what happens in your error handler and the debug logs around the time window when you see this exception (starting from when the error handler is invoked)?

@ki1729, you mean the Azure Event Hub logs around the time window when we see this exception?  I don't think we have that.  We provide a service for people to connect to their Event Hubs.  So, we would have to ask them for the logs when this happens.  We haven't seen this error in our logs since then.  Next time when it happens, I will ask the users to send me event hub logs.
As for as the error handler code is concerned, you have seen how I used the error handler in the original post, the errorhandler is very simple:
  private val handleError: Consumer[ErrorContext] = errorContext => {
    val error = errorContext.getThrowable
    logger.userError(error, LogKey.FEEDS__AZURE_EVENT_HUB_ON_ERROR)
    updateCheckpoint(None, logCheckpoints = true)
    System.exit(1)
  }

logger.userError is our custom error handling code.  It is responsible for the error to be printed in the log file, like those in my original post.  After logging the error, we then update the check point and exists the application.

When there is an error that occurs in the library, it propagates that upstream irrespective of whether or not we retry. In this case, we would recreate the underlying consumer client and continue consuming events. You would take no action on this exception,


Unable to add wallet

Unable to add wallet during development setup
screenshot

Error message on terminal
 LOG  [Error Catch]  [Error: Error: Could not parse descriptor tr(xprv9zC8agyvKuyfJ6iGFaaLNpTtjjQambiGfSnRzF2vX59V1veWQB8ZxzHRBMoEpbdjZgFjjJQUMwSXzQ7XZmr5v8n4nin5hPRRmgb4fXHyYJe/86'/0'/0'/0/*)]

 ERROR  [AsyncStorage] (Add wallet) Error loading data: Error: Could not parse descriptor

I think the error is as a result of the default account being set to unified ie Lightning (BIP86)

This is due to the descriptor library, bitcoinerlab/descriptors, I believe. Volt currently requires a patch which enables it to parse taproot descriptors (tr()) using the library. The proper fix is for the patch I have opened to be merged upstream.
For now, manually editing the files in the PR for the bitcoinerlab/descriptors in the node_modules should resolve this problem. Hope this helps.

This is due to the descriptor library, bitcoinerlab/descriptors, I believe. Volt currently requires a patch which enables it to parse taproot descriptors (tr()) using the library. The proper fix is for the patch I have opened to be merged upstream.
For now, manually editing the files in the PR for the bitcoinerlab/descriptors in the node_modules should resolve this problem. Hope this helps.

Got it. Thoughts on building your fork and adding it as the dependency url?
"@bitcoinerlab/descriptors": "https://github.com/Zero-1729/descriptors.git",

I forked and built your patch on a personal repo and it worked. Lmk what you think

Yeah, fair enough, just switched to my fork for the time being; will be part of the next tagged release (v0.4.4-beta.1).
Will revert back to the upstram repo once the PR has been merged.


[ruff] Skip tuples with slice expressions in incorrectly-parenthesized-tuple-in-subscript (RUF031)

Summary
Adding parentheses to a tuple in a subscript with elements that include slice expressions causes a syntax error.  For example, d[(1,2,:)] is a syntax error.
So, when lint.ruff.parenthesize-tuple-in-subscript = true and the tuple includes a slice expression, we skip this check and fix.
Closes #12766.

Probably should wait until #12762 is merged in and the subsequent merge conflicts have been resolved.

Thanks @dylwil3!

The fix being applied in this PR is very specific. But I think that's okay, because I think this syntax error can only occur due to the very specific details of literal slices inside subscripts. So I don't think the bug report reveals a deeper underlying issue that we need to fix.


Update reports.md

The Dashboard now has more reports, adjusted the unordered list of report names.

@Sirkkuw Can I merge this?


University Landing page

Describe the project
Want to add a sample landing page of a University.
Issue Type:

[ ] Bug Report
[ ] Feature Request
[ ] Documentation
[x] New Project

Additional context
Add any other context or screenshots about the project here.
Screenshots

If applicable, add screenshots to help explain your problem.

I request to kindly assign me this issue

Please assign me these issue

I request to kindly assign me this issue

Please assign me these issue

Apologies, but my work is already completed over this issue.

no problem sir
thankyou

Please assign me these issu


wmm_get_date does not take leap year into account.

When computing decimal year, wmm_get_date needs to take into account leap year and divide by 366 in those cases.

Now fixed, thanks.


Member directory filtering by list field type

Description of problem
In the filter settings of the member directory, it doesn't allow the use of "List" fields as a filtering option. This module will only allow fields that are either one of these types to be used as filters:

Text
Region
Country
We actually dig into DNN code to find that and here is what we see:

if (definition.DataType == textType.EntryID || definition.DataType == regionType.EntryID || definition.DataType == countryType.EntryID)
It only allows types of - Text, Region and Country.
Description of solution
Provide the ability to filter by the List field type.
Description of alternatives considered
We would have to make clients manually type in data into a text field and risk typos in order to search the data.
Affected version
I know the ones below are effected for sure. I have not tried others.

[x] 9.2.2
[x] 9.2.1
[x] 9.2
[x] 9.1.1

Affected browser
all browsers

please be aware that the underlying stored procedure, contained in the core, does not support filtering by data type list.

Give the current code and the underlying stored procedure, this appears to be by design. Is someone interested in developing this change in let's say within the next year?

Yes this is still a problem for a number of my clients. They have very detailed member directories and need the search to work properly


embed bokeh plots

I am wondering if there is a way to embed bokeh plot (and probably other rich js display) in the resulting sphinx doc ?
The first attempt led to the bokeh javascript loading step being deleted, the result of output_notebook() is being suppressed by nbsphinx.
That step could be solved by embedding the bokeh.js link in the html template, but then the bokeh "frame" gets an absolute position in the page (top left of the web page).
Did you face that issue already ?

No, I've never tried this. Can you please provide and example notebook? How does nbconvert handle this?

Thanks for your feedback.
Here is an example of a notebook with a bokeh plot and some html conversion:

one using nbconvert
one using nbsphinx (I created a small sphinx project to build it)
the same one with 2 manual additional lines to load bokeh.js and bokeh.css to be able to render the plot.

https://gist.github.com/fdeheeger/26cbac58ac94701f7b10f5536d7076ea
nbconvert is building a working html, where nbsphinx is getting rid of the part of the output area that contains the javascript loading.
The absolute position error I mentioned is solved, I just missed to add the css reference.
So the issue is just regarding the fact that nbsphinx does not keep the bokeh loading step, and so the plot is not rendered automagically.

This stackoverflow solution helped me to find a workaround.
http://stackoverflow.com/questions/37096106/how-to-load-external-javascript-in-sphinx-doc
The idea is to override the layout by adding a layout.html file into the _templates folder. This is the content I used:
{% extends "!layout.html" %}

{%- block extrahead %}
    {{ super() }}
    <link href="_static/bokeh-0.12.0.min.css" rel="stylesheet" type="text/css">
    <script type="text/javascript" src="_static/bokeh-0.12.0.js"></script>
{% endblock %}

I've implemented this in #62. Can you please check if this works for you?

Your fix fixes it !
Thanks, definitely useful to have the JavaScript output working magically (at least the bokeh one).

There is still an issue with jQuery in the readthedocs theme: https://github.com/snide/sphinx_rtd_theme/issues/328.

The bokeh example should work now, but jQuery is still not available when using the RTD theme.


feature/se-6-opeanaire-oai-tests

Create tests for the oai-pmh endpoint which returns items with eu identifier.

Being done in here


Update to minio-go v4

Codecov Report

Merging #1475 into master will increase coverage by 0.16%.
The diff coverage is 80%.

@@            Coverage Diff            @@
##           master   #1475      +/-   ##
=========================================
+ Coverage   51.63%   51.8%   +0.16%
=========================================
  Files         138     138
  Lines       11083   11083
=========================================
+ Hits         5723    5741      +18
+ Misses       4488    4463      -25
- Partials      872     879       +7

Impacted Files
Coverage Δ

internal/backend/s3/s3.go
64.02% <80%> (+0.35%)
:arrow_up:

internal/checker/checker.go
71.33% <0%> (+3.77%)
: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 95b6e4e...8b3b7bc. Read the comment docs.


Author as bot account?

Currently the author is set to me in https://github.com/prql/prql/pull/928. Is it possible to have it as a bot account, like update-rust-toolchain?
This is the case for Dependabot and pre-commit.ci, though those are apps rather than workflows so might be different. It would also give this action lots of contributions, which could be good for getting the word out...
Thank you again!

Thanks for the question! Id be happy for any information about this issue in general.

Is it possible to have it as a bot account, like update-rust-toolchain?

I am not sure, if this is possible as a GitHub Action. You can run it as the GitHub Action user itself, then native GitHub Action's wont be run on it. You can provide it with a token of a user that has repo access, then it will be run as the user, then GitHub Action's will be run on it.
I will look more into it!

The action is now run as the default github action bot user account. Sadly, it will still not trigger the CI to automatically run actions against the PR.
For automatically running actions on the PR you would need to provide the action with a  PAT, or GithubToken.

Thanks @a-kenji !
We have this pr-test-all label (e.g. https://github.com/prql/prql/pull/926), which runs all the tests, so I can add that manually for the moment; hopefully GH will adjust their policies around being able to run the tests automatically at some point.


MOUNIR_NODE2.toml for testnet 15

[validator.MOUNIR_NODE2]
consensus_public_key = "0038dd0714abe2155d40980bf7a68a71c57b5fb51ae48115a40cc5af9f5541fcb3"
account_public_key = "000366610d10575c6f34c0a1fddf3bbcdabbdb650045d034eb1faaf5b3c18e64d7"
protocol_public_key = "00efe939821115f9f99efe227fad6bffaf34825461e4ba4622260114623b418e9f"
dkg_public_key = "600000000fa9ef92db6382e20d863d3ada7b1702fcd2018129aad40ec39b647b107bf8b1d1ac087f1ec038d174c7774f85af8a1142951a257489249841818388e0ac4750b5058abf5bb54dbc0a5884572bb48f4af56b5bbbf887f1cb35f900035c0eba90"
commission_rate = "0.05"
max_commission_rate_change = "0.01"
net_address = "<IP_ADDRESS>:26656"
tendermint_node_key = "00c098383e0efa0bb0400249f2a2b8fc4e8ffbc10f16239ee870a07696a507c5b7"
email =<EMAIL_ADDRESS>discord = "ramonsami"
elements = ""
telegram = "@ramonsami (https://github.com/ramonsami)"
twitter = "@ramonsami (https://github.com/ramonsami)"

Not a namada-public-testnet-15 file
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.

Not a namada-public-testnet-15 file
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.

Not a namada-public-testnet-15 file
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.


Review and update the RBAC policies of Radius components

Area for Improvement
Review and update the RBAC policies of Radius components to ensure they adhere to the principle of least privilege.
Observed behavior
Extra permissions are given to some Radius components.
Desired behavior
Least privilege.
Proposed Fix
No response
rad Version
edge
Operating system
No response
Additional context
No response
Would you like to support us?

[ ] Yes, I would like to support you

AB#13745

cc/ @ytimocin


Missing dc entries

Hi,
I have a file which should have dc:title and dc:author entries (as shown by output from exempi dumpxmp sample, or exiftool), but the entries don't seem to make it into the python-xmp dictionary (either when I look into the consts.XMP_NS_DC entry or when I dump the whole thing).
Here follows the program I use, mostly identical to the sample on the web site:
import sys
from libxmp.utils import file_to_dict
from libxmp import consts
xmp = file_to_dict(sys.argv[1])
dc = xmp[consts.XMP_NS_DC]
print(dc)

It seems that the pdf can't be attached, here is a link:
http://www.lesbonscomptes.com/files/example-pdf-with-dc-in-xmp.pdf
Maybe I'm just doing something wrong?

Hello, it looks to me that the author and title that you are describing are
part of the PDF metadata rather than the XMP metadata?
On Mon, Apr 11, 2016 at 7:44 AM, Jean-Francois Dockes <
<EMAIL_ADDRESS>wrote:

Hi,
I have a file which should have dc:title and dc:author entries (as shown
by output from exempi dumpxmp sample, or exiftool), but the entries don't
seem to make it into the python-xmp dictionary (either when I look into the
consts.XMP_NS_DC entry or when I dump the whole thing).
Attaching the pdf file. Here follows the program I use, mostly identical
to the sample on the web site:
import sys
from libxmp.utils import file_to_dict
from libxmp import consts
xmp = file_to_dict(sys.argv[1])
dc = xmp[consts.XMP_NS_DC]
print(dc)
It seems that the pdf can't be attached, here is a link:
http://www.lesbonscomptes.com/files/example-pdf-with-dc-in-xmp.pdf
Maybe I'm just doing something wrong?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/python-xmp-toolkit/python-xmp-toolkit/issues/54

--
John Evans

I am new to this, and may have been fooled by jabref stating that it was saving to XMP, but doesn't the dumpxmp output also show that there is a dc:title element inside <x:xmpmeta>? :
http://www.lesbonscomptes.com/files/output-from-dumpxmp.txt

Turns out you were NOT fooled, but I was.  Here's the situation
dc:author is NOT present in the PDF file, but dc:title IS present in the
PDF file.  dc:title is NOT, however, present in the output of "exempi -x
filename", which is what fooled me.   Looking at the raw PDF file, though,
I do see that dc:author is there.  I'll investigate a bit further, but I
may have to contact the exempi library maintainer.
On Tue, Apr 12, 2016 at 1:56 AM, Jean-Francois Dockes <
<EMAIL_ADDRESS>wrote:

I am new to this, and may have been fooled by jabref stating that it was
saving to XMP, but doesn't the dumpxmp output also show that there is a
dc:title element inside <x:xmpmeta>? :
http://www.lesbonscomptes.com/files/output-from-dumpxmp.txt
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/python-xmp-toolkit/python-xmp-toolkit/issues/54#issuecomment-208720305

--
John Evans

Any updates on this?

Here is the thing. dumpxmp will scan the file and dump all XMP packets found. For that file, two. exempi -x will, like dumpmainxmp only dump the main XMP packet.
I'll be honest I don't really know how it work in PDF why there are two of these, but the "main" XMP packet doesn't have a dc:title metadata.
I feel that the packet it extracts is how it is intended by the Adobe SDK for which, we shall remember, there is no specific handler for the PDF format.


about finetune train

I only have 4x4090 cards，under this circumstance， can i finetune a MLLM？
How should I do to train， thanks a lot.

Thank you for your interest in Bunny.
24G per device is enough to pretrain and finetune  Bunny. However, the actual GPU memory consumption depends on your base model, image resolution and data.
For finetuning, setting per-device-batch-size to 2 or 4 may be good to you. In order to use the default learning rate in finetune_lora.sh, we recommend keeping global batch size 128. Global batch size = num of GPU * batch size per GPU * accumulation step. In your case, num of GPU is 4. Just just remaining parameters in finetune_lora.sh would be good for you.
Feel free to further comment on this issue if you  meet any problems in using Bunny.
Regards
Russell

I'll close this issue since no further discussion is raised yet. Please reopen it if you still have concerns.


New slaves are not showing in the Topology

Previously I used 1 master and 2 slaves, then added them to Orchester. Now I have added 2 more nodes to the master, but still I can see only 2 slaves.
output of orchestrator client:
2018-11-14 09:26:50 DEBUG Connected to orchestrator backend: orchestrator:?@tcp(<IP_ADDRESS>:3306)/orchestrator?timeout=1s
2018-11-14 09:26:50 DEBUG Orchestrator pool SetMaxOpenConns: 128
2018-11-14 09:26:50 DEBUG Initializing orchestrator
2018-11-14 09:26:50 INFO Connecting to backend <IP_ADDRESS>:3306: maxConnections: 128, maxIdleConns: 32
2018-11-14 09:26:50 DEBUG instanceKey: bhuvi-1:3306
2018-11-14 09:26:50 DEBUG instanceKey: bhuvi-2:3306
2018-11-14 09:26:50 DEBUG instanceKey: bhuvi-3:3306
bhuvi-1:3306   [0s,ok,5.7.24-0ubuntu<IP_ADDRESS>-log,rw,ROW,>>,GTID]
+ bhuvi-2:3306 [0s,ok,5.7.24-0ubuntu<IP_ADDRESS>-log,rw,ROW,>>,GTID]
+ bhuvi-3:3306 [0s,ok,5.7.24-0ubuntu<IP_ADDRESS>-log,rw,ROW,>>,GTID]```

Identified.
After added a new slave we need to create the orchestrator user on the slaves.

After adding a new slave we need to create the orchestrator user on the slaves.

Do you now replicate the mysql system tables? By default a replica will have same credentials as its master.

s/now/not

But we just take the dump of few databases and restored on the slave. So I guess this not add users.

Agreed.


Feature/fonts

Codecov Report

Merging #12 (867c82b) into develop (591681e) will decrease coverage by 6.81%.
The diff coverage is 80.30%.

@@             Coverage Diff             @@
##           develop      #12      +/-   ##
===========================================
- Coverage    97.11%   90.30%   -6.82%
===========================================
  Files           13       16       +3
  Lines          104      165      +61
===========================================
+ Hits           101      149      +48
- Misses           3       16      +13

Impacted Files
Coverage Δ

Core/Sources/Fonts.swift
67.74% <67.74%> (ø)

Core/Sources/FontLoader.swift
76.92% <76.92%> (ø)

Core/Sources/CoreAssembly.swift
100.00% <100.00%> (ø)

Core/Sources/StringExtension.swift
100.00% <100.00%> (ø)

Core/Sources/UIColorExtension.swift
100.00% <100.00%> (ø)

Core/Sources/UIImageExtension.swift
100.00% <100.00%> (ø)

Journey/Sources/JourneysViewController.swift
100.00% <100.00%> (ø)

MobileApp/Sources/AppDelegate.swift
100.00% <100.00%> (ø)

MobileApp/Sources/MobileAppAssembler.swift
100.00% <100.00%> (ø)

... and 2 more

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 591681e...867c82b. Read the comment docs.


smart-sizing AND full screen not working

I'm trying to make the smart-sizing function work when a client is in full screen mode. I have a windows 10 virtual machine that I'm trying to connect to from a Debian 10 host. If I enter the following:
xfreerdp /u:username /p:password /v:<IP_ADDRESS> /f /size:1024x768 /smart-sizing:1600x1200
I get a full screen session, but the remote resolution is then set to the local resolution of the Debian host. If I omit the /f then the scaling works as it should, with the remote session running at 1024x768 but scaled up to 1600x1200, but I then have an annoying titlebar at the top of the screen from FreeRDP. If I hit Ctrl-Alt-Enter, the client then goes full screen and works as anticipated.
I've tried the version of FreeRDP as supplied by the distro (2.0.0-dev 5 2693389a+debian) and I've also compiled a version from the git master and they both behave in the same way.
What am I doing wrong? I'm calling the client from a script, so I could potentially fake the Ctrl-Alt-Enter after startup, but that's a horrible idea.

@marria05 that functionality is just not there. You might use rdp high dpi support instead (see the scale options) to tell the remote end you have a high dpi monitor

@akallabeth, sorry, I don't understand. It seems to me that this functionality is there, but for some reason it's overridden on startup. If specifying /f on the command is meant to override the smart scaling functionality, why then, does it work if I omit the command line argument and then enable fullscreen during runtime by pressing Ctrl+Alt+Enter or sending an xdotool search --class "freerdp" key Ctrl+Alt+Enter. Surely there's a discrepancy here?
I ended up modifying xf_pre_connect:
diff --git a/client/X11/xf_client.c b/client/X11/xf_client.c
index 146c7feea..ef0c833f5 100644
--- a/client/X11/xf_client.c
+++ b/client/X11/xf_client.c
@@ -1211,7 +1211,7 @@ static BOOL xf_pre_connect(freerdp* instance)

        xf_detect_monitors(xfc, &maxWidth, &maxHeight);

-       if (maxWidth && maxHeight)
+       if (maxWidth && maxHeight && !settings->Fullscreen && settings->SmartSizing)
        {
                settings->DesktopWidth = maxWidth;
                settings->DesktopHeight = maxHeight;
@@ -1223,13 +1223,13 @@ static BOOL xf_pre_connect(freerdp* instance)
         * If /f is specified in combination with /smart-sizing:widthxheight then
         * we run the session in the /smart-sizing dimensions scaled to full screen
         */
-       if (settings->Fullscreen && settings->SmartSizing && settings->SmartSizingWidth &&
+/*     if (settings->Fullscreen && settings->SmartSizing && settings->SmartSizingWidth &&
            settings->SmartSizingHeight)
        {
                settings->DesktopWidth = settings->SmartSizingWidth;
                settings->DesktopHeight = settings->SmartSizingHeight;
        }
-
+*/
 #endif
        xfc->fullscreen = settings->Fullscreen;
        xfc->decorations = settings->Decorations;

This stops the client setting the remote resolution to the local resolution when you specify full screen from the command line while using smart scaling.
I suppose I could add an additional command line option to enable a /scaledfullscreen option. So that /f still operates as was.

@marria05 ok, reread your first post and realized I misinterpreted. you just use it wrong ^^
/smart-sizing:<width>x<height> is activating smart-sizing and the resolution is the remote resolution, now what you locally have.
Help could use some better wording to explain that.

xfreerdp /v: /u: /p: /size:100%
Seems to work nicely too.  YMMV


Allow watching for changes to editor options

I also removed the ability to reuse a singular root MediumEditor object. There is no function in the MediumEditor API to get all elements under an editor's control, so giving each element its own MediumEditor object simplifies the destruction and recreation necessary for changing an editors options.
See: https://github.com/yabwe/medium-editor/issues/1129

Sooo I actually like this simplification alot :)
I think the reuse prop was born to cope exactly with this problem, so the simplification is a great thing IMHO.
Two notes:

As this is a breaking change, we will need a major version bump. Then we should also change the text prop to model.
Do you know if there is any performance impact? There is the playground branch that you can use to test that stuff :) How did you proceed during development (just curious).

Bump!

Hello, glad to here your thoughts.  Apologies for the delays, I'm trying to release a library of my own soon and that is taking up a lot of my free time.
I can definitely update the version in the package.json, and I have changed the text to model locally.
Before I commit though, I am wondering your thoughts on moving the playground branch into an example directory in the master branch.  The demo is awesome, and having that code easily viewable would be helpful in my opinion (I missed that branch until you mentioned it). This pattern is  becoming somewhat popular in other vue component projects such as https://github.com/dmattia/vue-just-another-dropdown
For my testing so far I have actually imported the index.js file into a project I am currently working on (repo private unfortunately) and have been testing features on that website.
Lastly, this repo was part of the inspiration for the new project I am working on.  Do I have your permission to use code fragments from this repo in it? I noticed that there is no license currently, so I wanted to ask your permission 👍

Hey, no worries. A lib needs a lot of love to reach the quality for it to be release-ready, so take your time.

You can of course use my license, but beware that it is beerware 🙂 My software is open source, so just go ahead and take what you need.
Version bump is good.
I have reflected on the model approach and was wondering whether it suggests twoway binding. What are your thoughts? vue-medium-editor does not support 2way binding and will not anytime soon. Is it still ok to use model?
Moving the playground branch to examples is a good idea. We will have to add it to the npmignore though.

Is there a reason you don't want to support v-model?  If we change the text/model prop to be called value and emit an input event instead of a 'edit' event, v-model would work out of the box.
This would also be a breaking change, as instead of using v-on:edit on the component, the parent component would use a watcher.  This seems to be a more vue-y way to handle the updates.
If desired, I could add this in to my next commit as well.
What are you thoughts?

v-model is incompatible with Vuex. I do not want it to be tho only option.
The problem is that v-model mutates the data while v-on:input only listens to the latest data update. If we support v-model and make it a two-way binding thing we also must support v-on:input and v-bind:value. If we implement those three interfaces, we are good :)

Bump! BTW, what do you think with regard to #36 ? I think that it's not really necessary with the change we are working on right now.

Good point on v-model and Vuex. I do think having v-model support as well as having it emit an input event is ideal, but I will pause on that discussion for now as I think it is too much for this single pull request.  Maybe sometime soon we can talk about it in another thread.
I do think #36 is solved by this, although I'm not sure it needed solving in the first place.  Using data attributes isn't very common from what I've seen in vue.
I will revert back to using text for now instead of model, and should have the pull request updated with my new request by tomorrow sometime.

Sure! Thank you for this fruitful exchange David! Let's go with text for now and then do it as concluded. :)

FranzSkuffka check out the last commit and let me know what you think :)

Just got back from Czech Republic <3
Looks great to me. Is this ready to rebase and merge from your POV?


add a way to provide imported functions during relocation

hello! I'm currently playing around with wasmtime, I needed this, let me know if it can be useful :)

I was looking just at this on the weekend.
Can we make the import resolving allocation free?
For example, the user could supply a Fn(&str) -> isize (or Fn(&str, &str) -> isize with the module name`).

Hi! Yes, we do need something like this. Would it work to pass the HashMap into compile_and_link_module rather than making relocate public?
@jrakow The import resolution uses the HashMap in a readonly way, so it doesn't so any allocation itself, so I think it's ok as is. That said, I could be convinced that passing in a closure is more general than passing in a HashMap :-).

But a nonempty HashMap is always an allocation.
I was thinking of the static case, where you have a stable API and embed Wasm.
Ideally, it would only be big match statement in this case.

@jrakow That's a good point. @Geal, would it work for you to make this a closure instead? For the HashMap use case, it seems like you could just do a HashMap lookup in the closure, but then users that don't have a HashMap could do other things.

@jrakow @sunfishcode I just pushed a new version that takes a closure as argument. It looks cleaner that way and I can still use a hashmap in the closure

I also removed relocate from the public API

Thanks! This looks like a good step forward.


Add option to toggle debug mode

This PR adds a checkbox which allows you to toggle the debug mode. The debug mode basically just adds a css class on every OctoLinker link so you easily spot all links. On Firefox and Opera the settings are stored locally. Chrome stores the settings within the users account, if available.
Option page

Debug mode

Mhhh I think it's time to bump some dependencies, especially eslint + siblings ... will do that tomorrow.

@josephfrazier please don't merge this PR. Will do some minor tweaks now

@josephfrazier this can be reviewed now


I would like to add extensions to ApiDescription.

Please take the time to search the repository, if your question has already been asked or answered.

[x] What version of the library are you using? Is it the latest version? The latest released version is

What kind of issue is this?

[x ] Question. Is this a question about how to do a certain thing?

[ ] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests or
steps to reproduce get fixed faster. Here’s an example: https://gist.github.com/swankjesse/6608b4713ad80988cdc9

[ ] spring xml/java config that is relevant
[ ] springfox specific configuration if it makes sense
[ ] include any output you've received; logs, json snippets etc.; and what the expected output should be
[ ] if you have a repo that demonstrates the issue for bonus points! See this example

[x ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.
I would like to be able to add extension to ApiDescription between path and operations.  Basically add extensions on request level,  there are already extensions in method level that I am already using.  Springfox version 2.7.0

Please do consider starring this repository if you do like this project and find it useful.

Thanks for requesting this feature.


Update server running instructions

It's missing the run subcommand, also added the optional -hp arguments to show that they are configurable

Worth mentioning that it wraps the flask thingy so it accepts command lines just as flasks'. For example, specifying an SSL certificates using karton-dashboard run  --cert=path/to/file.crt --key=path/to/file.key


Add support for lsp/implementation request

Add support for lsp/implementation requests. This allows to navigate from a class to its instances by pressing Cntrl + F12.
See also: https://github.com/flix/flix/pull/2391

As before, Ill merge this once I have the flix/flix one merged and released.


Find local peak tool result different in ArcMap and Pro

The tools for each client results in differing output/results when used with the same data.
Expected Behavior
I would expect the result of the tool to be consistent across the platform.
Current Behavior
The Blue dots are the tool result from ArcMAP consumed in Pro. The Orange triangles are result from the tool in Pro.  Notice that there are peaks found in ArcPro that are not identified in ArcMap.  All the input data and values were constant.

Steps to Reproduce (for bugs)

I have sample data provided by @thumber1776
Run the Find local Peaks tool on same datasets with same values in Pro and in ArcMap
Compare results.

Your Environment
Pro 1.4 and ArcMap 1.3.1

see #221 ready to verify

@adgiles I am still seeing that Pro picks up more peaks when all variables are help constant.  Meaning with the same exact input data and Number of peaks.  In ArcMap, with number of peaks set to 25 only 6 were returned.

In Pro, with nunber of peaks set to 25 only 11 were returned.

@adgiles I wiould assume that with everything held constant that both tools would return the same number of peaks.  I also assume that if I said on input that I would like 25 peaks I should be returned 25.  If this is not the case we can address it with documentation.

@NatalieCampos are you sure your using the same variables as the two screen shots above have different AOIs and DTED files

In previous iterations of this tool (Find Local Peaks) the number of sinks wasn't the determiner, it was the 'local peaks' no matter how many there were, i.e. high points in the AOI.  Is this not the case now, or how it was ever intended to operate?  I only mention is because 11 was the number of clusters returned previously - no matter how many were input.  That makes me a little suspect. @NatalieCampos @adgiles

Yes to determine the peaks the elevation is inversed and the flow direction, followed by find sinks is run to determine the lowest points. From there we can grab the elevation values at each of the points created from the centre of the sinks, which should be then be highest points.
If the number of sinks found is less than the number of peaks specified by the user only that number will be returned

@adgiles Cool, thanks!  Yeah, I'm familiar with the tool and the process for how it works (our course has some pretty slick graphics for the students).  It was just never documented to work as such.  The previous version of the tool returned x-number of "peaks" - whatever you selected inside your AOI.  Whether or not it equated to a hydrographic sink or not.  It's good to know.  I can write it up as such for the students now so they will know.

@adgiles I will add this to the tool documentation and close it once it is added.

The following was added to the tool help.

The number of peaks to find within the Area of Interest. The number of peaks returned may be less than the input value depending on the number of peaks in the area.  You must enter a whole

https://github.com/Esri/military-tools-geoprocessing-toolbox/pull/226

The documentation in x metal was also changed.

@csmoore @kgonzago can you merge the pull request #226 and close this issue once done. Thanx

@NatalieCampos
I have test the tool with the sample data you provided and I am getting the same results in Desktop & Pro:

Thanks @adgiles .....I will give it one more go on my machine.

I tested it again and the results are the same.  Closing the issue.


Update to meet Style Guidelines, Added Integration Tests, Updated Unit tests - Fixes #15

Updated Unit tests - Fixes #11
Update to meet Style Guidelines - Fixes #14
Added Integration Tests - Fixes #15

Note: AppVeyor build machine switched to WMF 5.0 to use standard test template methods (allows modules to exist outside PSModule Path - required for non-destructive testing).

This change is

Hi @PlagueHO, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

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

Comments from Reviewable


Feature Request: Change Font Color

Please allow a setting to change the username color, directory color and the command color. In the example below the "john@LinuxMint" is the username color, "/opt/lampp/htdocs/" is the directory and "mkdir hello" is the command
john@LinuxMint:/opt/lampp/htdocs/ mkdir hello

yes

This package is no longer maintained. Please uninstall platformio-ide-terminal and install either terminus or x-terminal


Commonize listener into a lib

Listener here is going to need to be duplicated for a lot of new services we're building.
This should be commonized

Will be done in #2073


Internal error when parquet predicate pushdown is enabled "Error evaluating filter predicate:"

Describe the bug
DataFusion generates an error for some predicates when predicate pushdown is enabled
NOTE that pushdown filtering is not enabled by default (as we are still working on it) so this issue will not likely affect users:
To Reproduce

Download data from
repro.zip
Run datafusion CLI

The query run is
select count(*) from foo where request_duration_ns > 791684060 OR client_addr NOT in ('<IP_ADDRESS>');

Expected behavior
Same answer should be produced with and without row  filtering enabled. However, with row filtering an error is produced
datafusion-cli -f script.sql
+-----------------+
| COUNT(UInt8(1)) |
+-----------------+
| 53819           |
+-----------------+
1 row in set. Query took 0.006 seconds.

With it enabled:
DATAFUSION_EXECUTION_PARQUET_PUSHDOWN_FILTERS=true datafusion-cli -f script.sql
...
1 row in set. Query took 0.002 seconds.
ArrowError(ExternalError(Execution("Arrow error: External error: Arrow: underlying Arrow error: Compute error: Error evaluating filter predicate: Internal(\"Cannot evaluate binary expression Gt with types Utf8 and Int32\")")))

Additional context
Found by the test here https://github.com/apache/arrow-datafusion/pull/3976

It looks like this has the same underlying cause as https://github.com/apache/arrow-datafusion/issues/4005#issuecomment-1295949956
Reordering the predicates works
❯ select count(*) from foo where client_addr NOT in ('<IP_ADDRESS>') OR request_duration_ns > 791684060;
+-----------------+
| COUNT(UInt8(1)) |
+-----------------+
| 53819           |
+-----------------+
1 row in set. Query took 0.247 seconds.


feat(book): Engine RPC Types

Description
Adds op-alloy-rpc-types-engine docs to the book.

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#229 👈
main

This comment was automatically generated by st.


Reset person edit modal on close

Overview
Reset the person edit modal so it doesn't show old filtered data when it's open again
connected issues and PRs:
GN-5096
Setup
You must have a working vendor proxy service in the backend, or you can test linking the plugins to GN-frontend and proxying that to dev, let me know if you need any help
How to test/reproduce
Insert 2 person variable nodes in the RB part, then swich to the GN part and try to edit them, you should be able to filter on one, close the modal and open the same one or the other and the filter will be cleared and the data refreshed
Challenges/uncertainties
None
Checks PR readiness

[x] UI: works on smaller screen sizes
[x] UI: feedback for any loading/error states
[x] Check if dummy app is correctly updated
[x] Check cancel/go-back flows
[x] changelog
[x] npm lint
[x] no new deprecations

only resets the search the second time modal is opened so:

open modal
enter search, table updates
insert mandatee
select a different place in the doc
open modal again
observe table is still filtered
close modal without inserting (if you insert here, the modal still doesn't reset)
open again: now table is reset

Yes my bad, I was resetting every time the modal is close, but the insert was closing the modal without using the function, I solved that


Windows failure： Bash not found

PS D:\git\himawaripy> python3 himawaripy --auto-offset
himawaripy 2.2.0
Updating...
Latest version: 2021/06/11 13:30:00 GMT.
Detected offset: UTC+08:00
Offset version: 2021/06/11 11:30:00 GMT.
Downloading tiles...
Downloading tiles: 1/16 completed...
Downloading tiles: 2/16 completed...
Downloading tiles: 3/16 completed...
Downloading tiles: 4/16 completed...
Downloading tiles: 5/16 completed...
Downloading tiles: 6/16 completed...
Downloading tiles: 7/16 completed...
Downloading tiles: 8/16 completed...
Downloading tiles: 9/16 completed...
Downloading tiles: 10/16 completed...
Downloading tiles: 11/16 completed...
Downloading tiles: 12/16 completed...
Downloading tiles: 13/16 completed...
Downloading tiles: 14/16 completed...
Downloading tiles: 15/16 completed...
Downloading tiles: completed.
Saving to 'C:\Users\Tom\AppData\Local\himawaripy\Cache\himawari-20210611T113000.png'...
'bash' 不是内部或外部命令，也不是可运行的程序
或批处理文件。( is neither a command nor executable)
Could NOT retrieve env. var. DBUS_SESSION_BUS_ADDRESS
'bash' 不是内部或外部命令，也不是可运行的程序
或批处理文件。
Could NOT retrieve env. var. DISPLAY
Exception in thread himawaripy-main-thread:
Traceback (most recent call last):
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1520.0_x64__qbz5n2kfra8p0\lib\threading.py", line 954, in _bootstrap_inner
self.run()
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1520.0_x64__qbz5n2kfra8p0\lib\threading.py", line 892, in run
self._target(*self._args, **self.kwargs)
File "D:\git\himawaripy\himawaripy_main.py", line 214, in thread_main
r = set_background(output_file)
File "C:\Users\Tom\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\himawaripy\utils.py", line 92, in set_background
elif has_program("feh"):
File "C:\Users\Tom\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\himawaripy\utils.py", line 199, in has_program
subprocess.check_output(["which", "--", program])
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1520.0_x64__qbz5n2kfra8p0\lib\subprocess.py", line 424, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1520.0_x64__qbz5n2kfra8p0\lib\subprocess.py", line 505, in run
with Popen(*popenargs, **kwargs) as process:
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1520.0_x64__qbz5n2kfra8p0\lib\subprocess.py", line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1520.0_x64__qbz5n2kfra8p0\lib\subprocess.py", line 1420, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] 系统找不到指定的文件。

win10 terminal python3.9

when changing the wallpaper somehow it went wrong but the downloading is successful

Windows don't have bash

So Is there a way to let it work on windows' python terminal?

You may try this #118

Thank you! I'll give it a try


Tuning - Can't find data in directory + path

I tried tuning section as follows,
 python scripts/segtrack_tune_experiment.py /home/emjay/Documents/MOTS/TrackR-CNN/forwarded/conv3d_sep2/detections/5 /home/emjay/Documents/MOTS/TrackR-CNN/data/KITTI_MOTS/train/instances /home/emjay /home/emjay/Documents/MOTS/TrackR-CNN/tuning/output_file /home/emjay/Documents/MOTS/TrackR-CNN/tuning/tmp_folder/ /home/emjay/Documents/MOTS/mots_tools/mots_eval reid 1000

and got errors like this,
Traceback (most recent call last):
  File "eval.py", line 45, in <module>
    run_eval(results_folder, gt_folder, seqmap_filename)
  File "eval.py", line 25, in run_eval
    gt = load_sequences(gt_folder, seqmap)
  File "/home/emjay/Documents/MOTS/mots_tools/mots_common/io.py", line 28, in load_sequences
    assert False, "Can't find data in directory " + path
AssertionError: Can't find data in directory /home/emjay/Documents/MOTS/TrackR-CNN/data/KITTI_MOTS/train/instances
[('0000', 154), ('0001', 447), ('0003', 144), ('0004', 314), ('0005', 297), ('0009', 803), ('0011', 373), ('0012', 78), ('0015', 376), ('0017', 145), ('0019', 1059), ('0020', 837)] False
Traceback (most recent call last):
  File "eval.py", line 45, in <module>
    run_eval(results_folder, gt_folder, seqmap_filename)
  File "eval.py", line 25, in run_eval
    gt = load_sequences(gt_folder, seqmap)
  File "/home/emjay/Documents/MOTS/mots_tools/mots_common/io.py", line 28, in load_sequences
    assert False, "Can't find data in directory " + path

Can I get a tip for that?
On top of that, is this a correct formation of directory hierarchy?
data
└── KITTI_MOTS
    └── train
        ├── images -> somewhere/KITTI_MOTS/Images/training/image_02
        └── instances -> somewhere/KITTI_MOTS/Annotations_in_png_format/instances

I somehow solved it,
exchanging instances_image to instances_txt
python scripts/segtrack_tune_experiment.py /home/emjay/Documents/MOTS/TrackR-CNN/forwarded/conv3d_sep2/detections/5 /home/emjay/Documents/MOTS/TrackR-CNN/data/KITTI_MOTS/train/instances_txt /home/emjay /home/emjay/Documents/MOTS/TrackR-CNN/tuning/output_file /home/emjay/Documents/MOTS/TrackR-CNN/tuning/tmp_folder/ /home/emjay/Documents/MOTS/mots_tools/mots_eval reid 1000


Jaeger client in C/C++

Just booking a placeholder, as there is currently no active work on that.
Why it's important - many low level infra components are implemented in C/C++, we're missing the ability to integrate with them today. For example, there is nginx instrumentation.
Refs:

zipkin-cpp
lightstep-tracer-cpp

I'd be interested in helping out with this, but what exactly is involved? I have taken a look at the other clients, but having a hard time discerning the key components. For example, it seems not all the IDL files are used in all of the clients. How does zipkin relate to the jaeger client, and is it necessary to implement zipkin support in order to move ahead with the jaeger implementation?

OK thanks. Will give it a shot.

Do you want me to create a repository under jaegertracing?
NB: I added one more step above:

project structure, readme, license (APL2), travis build, code coverage

Please. I'll fork it once you do.

https://github.com/jaegertracing/cpp-client

Hey @isaachier just wanted to check in on this as some people were asking about using Jaeger for nginx.
Are you planning on binding to the OpenTracing API? I don't see any code for it in https://github.com/isaachier/cpp-client.
Also, just a suggestion: you may want to consider limiting your dependencies for the Jaeger client a little more. Requiring a large library like boost, for example, is likely to be an obstacle for projects like [envoy] to use it.

As of now, this has been sort of a side project. I did see the OpenTracing API, but not sure if it is worth the headache of merging. If there is a good reason to, I will. Right now seems the C++ API is not widely used and not yet stable (see end of their README).
I have thought about the dependencies. In short, Boost is one of the few libraries I feel is indispensable. One strong reason for this leads from another large dependency that cannot be avoided: Apache Thrift. All of the Jaeger clients use Apache Thrift to communicate, and the generated Thrift code depends on Boost.
About dependencies in general, it will depend how you use the library. I intend on exposing a simple C header for C clients or anyone else who does not want the included headers from Boost etc. That way, it is only required to link to the few Boost libraries (i.e. thread, asio, etc.) that are not header-only.

@isaachier We just released version 1.0 of the C++ OpenTracing API. While it might evolve somewhat, the expectation is to keep it as stable as possible. I wrote that README; the section you're talking about was copied over from the OpenTracing Go API.
I'm expected to release nginx OpenTracing bindings soon into opentracing-contrib, and we plan to make bindings available for envoy as well, so if you bind to them you'll be able to make the client work with those projects. It would also be more aligned with what the other Jaeger client libraries do.

OK great. I am not opposed in any way to merging using the OpenTracing C++ API, especially if it means we can leverage future instrumentations of C/C++ based projects. I'll take a look at the dependencies again and see what I need vs what is truly important. The hardest part here is networking using sockets on different platforms without a convenient wrapper like Boost ASIO.

I pushed a commit with some initial work. As of now, doesn't look too difficult to implement OpenTracing interfaces using existing code.

The official client is available from https://github.com/jaegertracing/cpp-client

Hi, is there any sample/tutorual on instrumenting a c++ code with Jaeger-cpp-client?

@maniaabdi check https://github.com/opentracing/opentracing-cpp/tree/master/example
Jaeger-specific part would be only initialization.


2 kyverno policies are not vendored in

Describe the bug
In Educates 2.6.x we can see that it includes kyverno-policies from release-1.7 tag, but there are 2 policies that are defined in vendir.yaml that were introduced after that release, in release-1.8 and hence they are not vendored in, which means they are not applied. These policies are:

restrict_ingress_defaultbackend which can be found at https://github.com/kyverno/policies/blob/release-1.8/other/restrict_ingress_defaultbackend/restrict_ingress_defaultbackend.yaml
unique-ingress-host-and-path which can be found at https://github.com/kyverno/policies/blob/release-1.8/other/unique-ingress-host-and-path/unique-ingress-host-and-path.yaml

We need to decide if these policies are needed and if so, vendir them from the proper release.
Additional information
No response

We should look into this as part of the changes implemented with the new installer

This is addressed in the new installer


V6 c++

(With thanks to @wtholliday )!
Implemented:

RenderJob.h
AudioProgram.h
RenderEvents.h

Implemented

WorkerThread.h (Very untested and probably need a tuple session with @wtholliday to go through
Sysex.h


the xkcd command does not work

it's due to a server error on the method i use to get relevant xkcds. currently migrating to a new website.

Ah, this is because of the unreliability of the searcher's website. I'm gonna implement some try excepts to ensure that things go smoothly.


Adds separate key for aggregator signatures

The following PR adds a separate private key for aggregator signatures when creating a new node.

Codecov Report

Merging #309 (79e623b) into main (8b9271f) will increase coverage by 0.04%.
The diff coverage is 100.00%.

:exclamation: Current head 79e623b differs from pull request most recent head 5d5ac92. Consider uploading reports for the commit 5d5ac92 to get more accurate results

@@            Coverage Diff             @@
##             main     #309      +/-   ##
==========================================
+ Coverage   58.64%   58.68%   +0.04%
==========================================
  Files          44       44
  Lines        7089     7089
==========================================
+ Hits         4157     4160       +3
+ Misses       2360     2358       -2
+ Partials      572      571       -1

Impacted Files
Coverage Δ

node/node.go
63.81% <100.00%> (ø)

rpc/json/service.go
65.64% <0.00%> (+1.84%)
: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 8b9271f...5d5ac92. Read the comment docs.

@tzdybal I have left you some messages on Discord too but have created draft PR for comments.
Not sure if I have missed the mark on what was required for this issue so please let me know :)

Updated the branch and changed from "improvement" to "breaking change" in CHANGELOG-PENDING.md as this changes external API used in Cosmos-SDK / Ethermint.

This should be good to go now @tzdybal?

Yes, I'll merge ASAP.


Bleach 2.0 compatibility issue

Hello,
I just installed a fresh version of mezzanine (4.2.3) and it seems there is an incompatibility with the bleach version 2.0 which was installed along. I get the error:
ImportError: No module named base
If I downgrade to 1.5 it works fine though
Best regards

This looks like a duplicate of #1741 and #1757. Are you using mezzanine-pagedown?

Not that I'm aware I just did a pip install mezzanine, and I had no previous install. There is nothing in pip list either

See https://github.com/mozilla/bleach/issues/267


Update response-request-view.md

Close https://github.com/bigen1925/introduction-to-web-application-with-python/issues/33


hypervisor/kvm.go: drop QEMU "mem-lock=on" and "cpu-pm=on" for ARM

Drop overcommit options for ARM:

"mem-lock=on" does not make sense for EVE, because EVE does not support swap area, so guest memory won't be paged anyway.

"cpu-pm=on" is NO-OP for ARM and used only for x86 family, which is again not what we need for EVE anyway.

By default both options are set to OFF.

I have some doubts about turning off the mem-lock option, and I’m not entirely sure that its impact is limited to just preventing swapping. Since memory overcommitment is enabled on EVE as indicated by /proc/sys/vm/overcommit_memory showing 1, I'm concerned that this option might also influence how the system handles memory under overcommitment.
If mem-lock=on does more than just prevent swapping — such as ensuring the VM’s memory is guaranteed even when the host is under memory pressure — it could be critical. We should investigate further to understand exactly how this option interacts with the overcommit mechanism before deciding to disable it.

Ok, what I see in the code, this option results in calling
ret = mlockall(MCL_CURRENT | MCL_FUTURE);

I'll try to figure out what precisely the effect of this call is.

And here is the syscall implementation
https://github.com/torvalds/linux/blob/67784a74e258a467225f0e68335df77acd67b7ab/mm/mlock.c#L742
From what I see, it sets the VM_LOCK flag to all current VMAs of the process and sets it as the default flag. It also calls mm_populate for memory, which, as I understand it, makes it residence.
https://github.com/torvalds/linux/blob/67784a74e258a467225f0e68335df77acd67b7ab/mm/mlock.c#L766

It seems like Linux at least has this intent: to populate all the memory that is marked by VM_LOCKED, including newly allocated.
https://github.com/torvalds/linux/blob/67784a74e258a467225f0e68335df77acd67b7ab/mm/mmap.c#L304

So, after all, I would recommend NOT to disable mem-lock until we disable the entire overcommit_memory feature.

@cperikas, if you are still working on the QEMU memory investigation, could you please check the influence of the "mem-lock" option I described above? I'm interested in how the option will affect the behaviour that you described under the "Lazy allocation on EVE-OS" section of your report. I guess that with the option "on", the memory will be prepopulated, and it will not appear under the free section anymore...

@OhmSpectator I think I will just keep silence, I see you are on correct way of thinking and eventually you reach the point and say "ok, fine" :)
Just one hint: please check the kvm.go and look for the [realtime] mlock=off which stays there since version 5.1.0 and presumably (I did not check that) overrides the command line option whatever it is.

@OhmSpectator I think I will just keep silence, I see you are on correct way of thinking and eventually you reach the point and say "ok, fine" :)

I'm already there. Since we do not have the ARM-specific tests rather than build ones, we should not wait for the entire test suite to run, I think.

@OhmSpectator I tested the mem-lock option and can confirm that with it enabled, the VM immediately reserves the full amount of allocated memory in the host's physical RAM. This pre-populates the memory from the start, preventing any lazy allocation.

Another observation to the piggy bank: the default RLIMIT_MEMLOCK is set to the 8Mb (defined in the kernel itself) on EVE (easy to verify reading the /proc/$QEMU_PID/limits), which means that QEMU even won't start if the mem-lock=on is set. That means that regardless of what is set in the command line, the qemu configuration (xen1.conf) file takes precedence.

Another observation to the piggy bank: the default RLIMIT_MEMLOCK is set to the 8Mb (defined in the kernel itself) on EVE (easy to verify reading the /proc/$QEMU_PID/limits), which means that QEMU even won't start if the mem-lock=on is set. That means that regardless of what is set in the command line, the qemu configuration (xen1.conf) file takes precedence.

Ah, good to know. Pitty. I liked the idea of adding this option...


BigInt log10 and log2?

The current plan is to remove BigInt support from all transcendental functions. However, there are exact cases for log10 and log2.
Should we support Math.log10(100n) or Math.log2(16n)? My gut says yes, and we can truncate non-integer results, as with BigInt division.

Changes to explainer and spec have been pushed. log2 and log10 now truncate towards zero.

#4 states that a reason against transcendental functions is that "processing time would need to be huge". I agree, and would like to point out that this will apply to log10 as well. Computing log10 will be algorithmically equivalent to doing my_bigint.toString().length. (log2 will be very fast. For the record, this statement does depend on implementation choices and is currently true for the major browsers; there are also BigInt libraries in existence for which log10 would be fast and log2 would be slow.)
log2 needs a specification for what log2(0n) should do. Throw an exception?
Also, note that the values returned by log2 will never be outside a range that Number could handle with ease (current browser implementations allow a million or a billion bits, so that'd even be int32 range; allowing for the (unlikely?) possibility of future growth there still means that MAX_SAFE_INTEGER range would be plenty). So you could consider having log2 return a Number, which would have the benefit that it wouldn't need to be rounded to integers (though of course there would be a loss of precision once the input exceeds a certain range). I don't feel strongly about this, it's just an idea.
If log2 always truncates, maybe it would make sense to call it bitLength instead? That would also address the 0n question. (Of course, the exact relation between these functions is bitLength(x) == log2(x) + 1.)
Speaking of truncation, the explainer currently says "log2(2n ** 49n - 1n) would return 49n", whereas truncating behavior means returning 48n for that input.

#4 states that a reason against transcendental functions is that "processing time would need to be huge". I agree, and would like to point out that this will apply to log10 as well. Computing log10 will be algorithmically equivalent to doing my_bigint.toString().length. (log2 will be very fast. For the record, this statement does depend on implementation choices and is currently true for the major browsers; there are also BigInt libraries in existence for which log10 would be fast and log2 would be slow.)

This is essential feedback, thank you. Because we currently have no specific use cases for BigInt log10, I would be happy to drop log10 if implementation complexity would be large. I can make the change later today. (CC: @syg, @codehag)

log2 needs a specification for what log2(0n) should do. Throw an exception?

The specification currently throws a RangeError.

Also, note that the values returned by log2 will never be outside a range that Number could handle with ease (current browser implementations allow a million or a billion bits, so that'd even be int32 range; allowing for the (unlikely?) possibility of future growth there still means that MAX_SAFE_INTEGER range would be plenty). So you could consider having log2 return a Number, which would have the benefit that it wouldn't need to be rounded to integers (though of course there would be a loss of precision once the input exceeds a certain range). I don't feel strongly about this, it's just an idea.

Currently, whenever an arithmetic operation or Math function takes a BigInt argument (and doesn’t throw), it returns a BigInt. I think we should continue to match this and return BigInts from log2 when given BigInts.

If log2 always truncates, maybe it would make sense to call it bitLength instead? That would also address the 0n question. (Of course, the exact relation between these functions is bitLength(x) == log2(x) + 1.)

I would rather keep it the same name, but I do not feel strongly about this.

Speaking of truncation, the explainer currently says "log2(2n ** 49n - 1n) would return 49n", whereas truncating behavior means returning 48n for that input.

Good catch. This is a mistake that I actually already fixed right before this comment was posted, in b09db3efe74614f5efd7e3e0cc068479b76ad0b6.

Closing. log2 and log10 truncation may be developer surprising. Truncating log2 could be codified as a future type-overloaded Math.bitLength method. And there are no other known use cases for truncating log2 or log10.
See also https://github.com/tc39/proposal-bigint-math/issues/13#issuecomment-919472001, https://github.com/tc39/proposal-bigint-math/issues/14, and https://github.com/tc39/proposal-bigint-math/issues/14#issuecomment-918663577.


Upgrade Google BillingClient to v2.0.3

#693 Upgrade Google BillingClient t0 v2.0.3

@Redth Is it ok to merge?

@tuyen-vuduc trying to see if there's any sane way to not break the few api methods here:
# API diff: Xamarin.Android.Google.BillingClient.dll

## Xamarin.Android.Google.BillingClient.dll

### Namespace Android.BillingClient.Util

#### Type Changed: Android.BillingClient.Util.BillingHelper

Removed methods:

```csharp
public static Android.OS.Bundle ConstructExtraParamsForAcknowledgePurchase (Android.BillingClient.Api.AcknowledgePurchaseParams acknowledgePurchaseParams);
public static Android.OS.Bundle ConstructExtraParamsForConsume (Android.BillingClient.Api.ConsumeParams consumeParams, bool isIabV9Supported);
public static Android.OS.Bundle ConstructExtraParamsForGetSkuDetails (bool isIabV9Supported, bool enablePendingPurchases);

You'll notice Google added string libraryVersion parameters to these methods, and I'm not sure if passing null is acceptable or not.  If it is, we could add a partial class with overloads for these methods which call the new ones just passing null as the libraryVersion.  The documentation doesn't even yet reflect these API changes unfortunately.

@Redth
The above methods I think they are intended to use internally within the library. If they were changed for some particular reason, I think we should keep them as they are.
If someone already uses one or some of them, they should change their code after the upgrade to ensure it works as expect. It's a must in the native world, obviously.
How do you think?


Missing SVG support for classList

I believe you have to use el.setAttribute('class', 'blah') to be able to update the attribute on an SVG element, but it might be a bit tricker than that once you throw IE into the mix.

+1 This seems to work on IE (IE9+) for me.

also the hasAttribute must be supported

Was this fixed?


Add new Vigo Android Developer Group

It doesn't work like that (yet). You need to add all the changes, not only in the JSON. Use this PR as reference:  https://github.com/VigoTech/vigotech.github.io/pull/78
It should be the same but with your group info.

Editado


Crazy JS?

Hi,
Look whats happening inside componentWillReceiveProps:
First I create this const data = Immutable.Map(nextProps.data)
then create a clone var temp = data.toJS() to modify with this
for (var key in temp) { 	temp[key].splice(2, temp[key].length) }
In the end both data and temp are the same.
I have tried every possible solution and they both end up the same.

This isn't react-native specific, can you post about in on Stack Overflow? Thanks!

@vjeux I thought it had to do with memory allocation since the immutable object changed after it was set.
Do you think it's a js problem and not of the 'immutable' library or react?

There's nothing React specific here, I would ask on the immutable-js slack/discord channel (not sure what they use)

@vjeux merci beaucoup :)


[CloudFront] Cannot set Security Policy

The current CloudFront implementation has no way as far as I can tell to set the Security Policy of the distribution (called MinimumProtocolVersion inside CloudFormation). This looks like it would fall under the AliasConfiguration object.
Docs can be seen here:
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-cloudfront-distribution-viewercertificate.html#cfn-cloudfront-distribution-viewercertificate-minimumprotocolversion

Thanks for reporting!


[1.14.4] issue title nightconfig error

Describe the bug
crashes on load up nightconfig error
Forge Version 28.1.92
Mod Version 2.2.5
logs:
http://puu.sh/EHMAg/67db5997d2.log
http://puu.sh/EHMAb/c180185543.log

@TheMCBrothers

Thank you for reporting this issue!
I will try to fix this bug


Remove private_emails field from /v1/people API

https://github.com/columbia-it/covid-infocommons/pull/106


Create basic vector rendering functionality

Implement Canvas
Render the following 2D shapes:

Rectangle
Square
Circle
N-sided polygons

Automatic determination of vertices in gl::DrawArrays()

Functionality implemented in 6b8c3bc69eb70c9198a803b98fc1c77105239d29, 1fecf4d1ee68adba3b95fe4080281047b01e9f13, and d60fca1145b470314e64c55dfaed9320ba3ac339


1.12.2?

This plugin is really great! So when will this plugin support 1.12.2? I really like this plugin!

Related to #35 , but essentially there are no plans on supporting 1.12 at the moment. Just by nature of the changes made to Minecraft, I would need to sort a lot of things out for the plugin to be compatible with versions prior to 1.14 and 1.13

Ok,fine.But I really need it, so when do you plan to add support to 1.12.2. I'll take my time to wait for it.

Sorry I don't have any sort of timeline at the moment

recommend closing this since it's be answered and it's clogging up for no reason xD

Close this issue :)


Remove Network accordion in Add RKE clusters

Version: 2.2 master 10/31
Steps to reproduce (least amount of steps as possible):

Add Cluster
Select any RKE cluster

Issues:

[x] Move Network Provider to Cluster Options (placement should make sense for both regular RKE and Custom RKE)

[x] Move Project Network Isolation, Nginx Ingress & Node Port Rancher to Cluster Advanced Options.

Version 2.2 master
Verified fixed


ModuleNotFoundError: No module named 'sklearn.externals.joblib'

---> 33 from skrebate import ReliefF
34 import numpy as np
35 from sklearn.linear_model import ARDRegression as ard
~/opt/anaconda3/envs/causal_bi/lib/python3.8/site-packages/skrebate/init.py in
26
27 from ._version import version
---> 28 from .relieff import ReliefF
29 from .surf import SURF
30 from .surfstar import SURFstar
~/opt/anaconda3/envs/causal_bi/lib/python3.8/site-packages/skrebate/relieff.py in
31 import sys
32 from sklearn.base import BaseEstimator
---> 33 from sklearn.externals.joblib import Parallel, delayed
34 from .scoring_utils import get_row_missing, ReliefF_compute_scores
35
ModuleNotFoundError: No module named 'sklearn.externals.joblib'

have the same issue

Looks like scikit-learn removed the joblib directory in v0.23.0 (it's still there in v0.22.1). Installing scikit-learn <0.23.0 resolved this issue for me.

There is an open PR to resolve this issue, and bring further updates to scikit-rebate but ti was opened on Feb 12. Would be great if the maintainers of this package merge the changes.


Collection size in filters

Hey!
I think it would be really useful to be able to examine/compare collection sizes in filters. The idea is that the size of the collection could be used in conjunction with the existing comparison operators in filters.
Example:
filters:
      - type: collection-size
        key: "Attachments[]"
        value: 1
        op: gt

Sample use cases:

Attached volumes

Instances in ASG

SGs on an instance

Thoughts on this?

sounds great, there's also discussion of a more generic facility in #147 via a value_type  for giving some guidance on conversion of values prior to comparison.

implemented in #300
example
filters:
      - type: value
        value_type: size
        key: "Attachments[]"
        value: 1
        op: gt


Add print_layer function parameter to generate() abstract methods

Description
Add a print_layer function parameter to the report formats' generate() functions so we can work towards printing output reports for single layers instead of the whole container.
To Do

Add a print_layer function parameter to the generate() abstract method stub in tern/formats/generator.py. Since there's a chance this may not be provided, a default value (like None) will be required.
Update the generate() functions in each report formats' generator.py to take in this new parameter.
Update the call to the generate() function (line 64) in tern/report/report.py to take in the --layer integer from the args.layer command line option, if provided.

Background
Relies on completion of https://github.com/tern-tools/tern/issues/845 first.
Super Issues
#840

GitHub is not letting me assign this to @aalexanderr but he has expressed interest in taking this on.


data processing + processing for futureSuitability

addressing #38

@lindsaycarr can you fix conflicts?

@aappling-usgs can you review my new changes? Still unsure about what the first argument to visualize.visualizeFishTrends will be since visualizeBassTrends and visualizeWallyTrends both call it


LUCENE-10064: Implement TieredMergePolicy#findFullFlushMerges.

This implementation filters merges returned for natural merging by only considering merges on the lowest level, ie. when the average segment size to be merged is less than or equal to the floor segment size.
#11102

Even better, all our base merge policies have a default impl now.


[Feature Request] Provide support for PyHMMER

The current implementation uses HMMER but PyHMMER is much much faster when multi-threaded.  Is there any plan to provide support for PyHMMER?
https://academic.oup.com/bioinformatics/article/39/5/btad214/7131068

If it's any help, I wrote a cli wrapper for PyHMMER called PyHMMSearch (https://github.com/jolespin/pyhmmsearch) which I've been using in place of hmmsearch to dramatically increase performance in multithreaded modes.

Hi @jolespin
Thank you for the wrapper. I am curious about the runtime difference between HMMER and pyhmmsearch when analyzing a single protein sequence with a single HMM file? In this scenario, whether the multithread mode will helpful?

Multithreaded should be much quicker. If you want I can benchmark and send over the results.
If you attach a sequence and HMM I'll test it out. If not, I'll just use a random set I already have downloaded.

Thanks, if works, I am happy to replace HMM with pyhmmsearch in compleasm

for N_THREADS in 1 2 4 8 12;
do
    time (pyhmmsearch.py -i test_1n.faa -m gathering -d PF00709.21.hmm -o test_1n/n-${N_THREADS}.tsv -p=${N_THREADS}) 2> test_1n/logs/n-${N_THREADS}.e > test_1n/logs/n-${N_THREADS}.o
done

Looks like the speed up is noticeable (assuming I'm using the time module right looking at real):
test_1n/logs/n-1.e:real	0m0.183s
test_1n/logs/n-2.e:real	0m0.079s
test_1n/logs/n-4.e:real	0m0.079s
test_1n/logs/n-8.e:real	0m0.078s
test_1n/logs/n-12.e:real	0m0.077s

Here's the sequence I used:
(align_env) Joshs-MBP:test jolespin$ cat test_n1.faa
>SRR13615825__k127_221120_1 # 1 # 987 # 1 # ID=13561_1;partial=10;start_type=Edge;rbs_motif=None;rbs_spacer=None;gc_cont=0.698
ILPYHRDLDLLSEARRGERKIGTTSRGIGPAYEDKIGRRGVRAGDLADPAALEEEIRENV
QARNRLVGDAAMDWRAIVDRLRSYGERMQPWIGDASLFLWDAIKAGRPVLFEGAQGTLLD
IDHGTYPYVTSSNSTIGGVCTGLGVGLHAVGGVIGVAKAYTTRVGGGPLPTELSGPLAER
LRESGQEYGASTGRPRRCGWYDAVAVRYAVRTNGLDGLALTKLDVLDGLETIDVCTAYRC
GGRTLTEFPADVRQLEACEPVYDRLPGWSRPTRGTRVFSDLPPDAQAYVRHLEQVSGVPV
TILSTGSDREDTIIREHSVIADWFALRP*

PF00709.21 HMM is attached:
PF00709.21.hmm.zip

Compleasm already runs hmmer in multiple threads. I don't think pyhmmer will help.

@lh3 One of the benefits of PyHMMER is that the speed scales with the threads (Panel C in figure above from the manuscript).  I've done some tests myself and noticed considerable performance gains when multithreaded:
| Database | Tool            | Single Threaded | 12 Threads |
|----------|-----------------|-----------------|------------|
| Pfam     | PyHMMSearch     | 2:24            | 0:20       |
| Pfam     | HMMER HMMSearch | 2:53            | 2:27       |

\* Time in minutes for 4977 proteins in `test/test.faa.gz`.

Test files are here: https://github.com/jolespin/pyhmmsearch/tree/main/test
PyHMMSearch is a CLI wrapper around PyHMMER.
I'd say PyHMMER is at least worth a shot to look into for performance gains.

Compleasm is launching many independent hmmer runs in the single-threaded mode and schedules these runs to achieve overall multi-threading

Makes sense since compleasm is already overcoming the multi-threaded limitations of HMMER by splitting everything out manually.  Though, there might be some overhead in loading the HMMs in this way.

Your pyhmmer won't help in this case.

I didn't develop PyHMMER (https://github.com/althonos/pyhmmer) just a big fan of the performance gains once I started using it (which is why I made the CLI wrapper around it).

It also has one more dependency.

Another aspect of PyHMMER I like is the lack of dependencies (not even HMMER needs to be installed) and it's a standalone Python executable so very easy to use/install in current environments.
Just wanted to put PyHMMER on the radar in case it could speed up the workflow since I've found it useful in my own workflows.


fix: detect bad escape sequence

Do not allow unescaped quotes between "..."
key="a"b" is a syntax error, the valid expression should be key="a\"b"
-- this is as expectedd.
However, prior to this change, if we put two backslashes before the ", it becomes valid.
i.e. below two files are parsed to the same result:
key="a\\"b"

and
key="a\\\"b"

after the fix, key="a\\"b" becomes invalid.

Fixing this may cause some compatibility issue.

https://github.com/emqx/hocon/pull/250


Update columns.js

by default do case insensitive search

Sorry for the late response.
I think we need to make this as an option. Otherwise, I need to release this as major release since it changes the way this library behave even when the how to use this library does not change.

Please merge this code.. Case Insensitive search is needed.


URLs are not URL-encoded breaking the markdown rendering in Mattermost

We use the plugin in mattermost and have messages like this:

*New message in Graylog stream [Foo Exceptions]
(https://graylog/streams/58311e2b2ab79c080136c1d9/messages?q=*&rangetype=relative&relative=3600)*:

the star () in q= is seen as ending the italic section leaving the message rendered as
New message in Graylog stream [Foo Exceptions](https://graylog/streams/58311e2b2ab79c080136c1d9/messages?q= &rangetype=relative&relative=3600)*:
(w/o the blank. the github markdown renderer is smarter than the markdown renderer of mattermost.)
If the URL is URL-encoded, this can be avoided.

I'll take a look at this.

Actually, with https://github.com/graylog-labs/graylog-plugin-slack/pull/60 in, the asterisk should be gone.
Not that this means, that this Issue is not relevant. But the newest build should work around it.

Ah, scratch that, there are two links in the message, and the second one is still broken.

Please merge PR #63 @Aenima4six2, tested here successfully


Upgrade Spring

:tada: This PR is included in version 8.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


search_mode not inherited by child models

The search_mode attribute, set on a base class, isn't inherited by child models. This may apply to additional settings with default values (currently only ns_attrs).
Minimal Example
from pydantic.utils import to_lower_camel
from pydantic_xml import BaseXmlModel, element

class CustomBaseModel(BaseXmlModel, search_mode="unordered", nsmap={"": "somenamespace"}):
    """Base class to configure options"""

    class Config:
        alias_generator = to_lower_camel

class ChildModel(CustomBaseModel):
    """Root 'real' model describing a document"""

    some_field: str = element()

Expected Behavior
I expect that ChildModel.__xml_search_mode__ == CustomBaseModel.__xml_search_mode__ == "unordered"
Observed Behavior
ChildModel.__xml_search_mode__ is set to the default of "strict" unless seach_mode is also passed to ChildModel.
I believe this is due to the default value in __init__subclass__ for search_mode. The check search_mode if search_mode is not None always evaluates True and will never fall back to the else condition here.

New to Open Source contributions so, let me know if there's anything else you need. I'm happy to pull together a PR if this diagnosis seems right. Thanks!

@Kxnr Hi. Thank you for the report. That's a bug. I will fix it in the next update.

Thanks!

Is it possible to specify search_mode in model config? Like this,
class BaseXmlModel(XmlModel):
    class Config:
        anystr_strip_whitespace = True
        underscore_attrs_are_private = True
        search_mode='unordered'

@Bolik Hi. No, it is not possible now.


.Net: Add Function arguments to activity

Just after an activity is created during KernelFunction invocation here https://github.com/microsoft/semantic-kernel/blob/b00a705a9a4132e54fa4d21e7cb28612611e9892/dotnet/src/SemanticKernel.Abstractions/Functions/KernelFunction.cs#L299 I would like to add a tag containing the arguments, something  like this:
activity?.AddTag("Arguments", JsonSerializer.Serialize(arguments));

That way when using OTel I can see at a glance what is happening when a tool is invoked multiple times with strange arguments and I need to give more context. For example this trace is invoking the same function several times with different arguments all resulting in a call to an API that returns an error before it figures out what to do:

can become something like this:

and I can see both what is going on with the invocation and what type of bug/missing context I might have without having to dig further into logs or change verbosity in order to get the logger to give me the same information.
Happy to send the PR but wanted to ask if there was a reason not to do it before I did that.

@noahfalk, @tarekgh, @CodeBlanch, what do we recommend folks do when they want to include additional user-provided information as tags but that information could possibly be sensitive?

I'm not convinced we have the right patterns nailed yet, but I think the broad strokes look something like:

Determine which data is potentially sensitive
Design APIs/config settings that let app developers decide what portions of that data are appropriate to expose in their scenario.
Data that doesn't flow either gets omitted entirely, or replaced with some sentinel value indicating it was redacted.

These examples aren't explicitly about tags, but they are in the general area of refining what data gets emitted in telemetry:
https://learn.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.httplogging.httploggingoptions.requestheaders?view=aspnetcore-8.0
https://learn.microsoft.com/en-us/dotnet/api/microsoft.extensions.dependencyinjection.httpclientbuilderextensions.redactloggedheaders?view=dotnet-plat-ext-8.0
https://github.com/dotnet/extensions/blob/main/src/Libraries/Microsoft.Extensions.Telemetry.Abstractions/Http/HttpRouteParameterRedactionMode.cs#L9
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/error-handling?view=aspnetcore-8.0#developer-exception-page

Or is the general stance that choosing to export this data is signing up for it?

I think different users have different expectations and judgements vary depending on how sensitive the data is likely to be. I think it is perfectly reasonable for cases with a mixture of sensitive and non-sensitive data to only enable the non-sensitive portions initially and force some additional explicit opt-in to include the sensitive portions.
In this case it looks like the authors of this convention thought the same thing:
https://github.com/open-telemetry/semantic-conventions/pull/825/files#diff-72369cbec5da92c01143061b1f74d700eba3fc8f559f47fc70331dbf1bf702feR29-R35

@TaoChenOSU, assigning this to you since you'll be looking at the new telemetry patterns. I'm curious how (if at all) this is supported with the proposed AI telemetry.

What we have done on the OTel side (typically) is offer an options class in our instrumentation packages where users can opt-into capturing things which may contain sensitive data. For example, for SqlCommand.CommandText we have: https://github.com/open-telemetry/opentelemetry-dotnet/blob/9261a679e6325346aa1d0abcd19ed6b277286e13/src/OpenTelemetry.Instrumentation.SqlClient/SqlClientTraceInstrumentationOptions.cs#L95
We also typically expose an "Enrich" delegate on these options classes where users can add (or remove) things as needed: https://github.com/open-telemetry/opentelemetry-dotnet/blob/9261a679e6325346aa1d0abcd19ed6b277286e13/src/OpenTelemetry.Instrumentation.AspNetCore/AspNetCoreTraceInstrumentationOptions.cs#L62

The opt-in part seems key here.

As Dan said it sounds like the first task is an SK specific pattern for how we would turn on/off individual potentially PII (like the telemetryoptions type linked to) and we should establish that, then we can do features like this and it is unlikely to need significant rework as OTel stuff advances more generically. Unless an options type class already exists? Does that sound right?
For the point about logging prompts, do we need to need to support something like https://microsoft.github.io/presidio/ to optionally do our best to sanitize the prompt.

As Dan said it sounds like the first task is an SK specific pattern for how we would turn on/off individual potentially PII (like the telemetryoptions type linked to) and we should establish that, then we can do features like this and it is unlikely to need significant rework as OTel stuff advances more generically. Unless an options type class already exists? Does that sound right?
For the point about logging prompts, do we need to need to support something like https://microsoft.github.io/presidio/ to optionally do our best to sanitize the prompt.

Following up on Glenn's Presidio comment.
Here's an example of how it fits into the flow
https://www.llamaindex.ai/blog/pii-detector-hacking-privacy-in-rag


[MWKProgressIndicator show] doesn't work

Hi,
I'm using storyboards and want to replace ProgressHUD with MWKProgressIndicator.
So, added MWKProgressIndicator.h and .m, trying to reach MWKProgress and it doesn't work.
[ProgressHUD show:@"Please wait..." Interacton:NO]; - works;
[MWKProgressIndicator show]; - doesn't show anything.
May be I missed something?
Thank you,
Anton

Tried using it yesterday. Found the same issue.
For some reason, the singleton pattern of this control is not used correctly. when I was debugging using reveal, after using the show method, the view is not added to the superview thus all of its controlling methods are unavailing.
The workaround for me now is to dispose the singleton pattern by converting all class methods to instance methods and manually saving it to a view instance in your triggering VC.
Hope that helps and the author can update it one day.


Replace standard encoding/json library using fast github.com/goccy/go-json

Unfortunately, the tests fail. There will be no replacement.


feat: ESS investigation

Description
This PR introduces a new feature: functionality for ESS investigations.

creation of new ESS investigation from the Parts as Planned List
new ESS Investigations page with filters and pagination for the display of existing ESS Investigations
creation of new ESS investigation from the ESS investigations page

Checks failed in the initial pull requests. Fixed failing tests


Add more keyboard bindings for debug features

Add more keyboard bindings for debug features such as Add watch and evaluate in console.

FYI @isidorn @weinand

There is a focus: debug console action which can be configured to have a keybinding.
Similarly there is an focus watch action. APart from that we plan no action here


Kadira Error: Agent Error: 499

I'm getting this error (in the run before I got Agent Error: 500) but I don't get any information as to what that means or why it's happening. Nothing shows up in the dashboard.
Can anyone help?

Seems like it's socket error code.
See: http://stackoverflow.com/questions/12973304/nginx-499-error-codes
We use nginx as the proxy for our server.
We are not exactly sure on how to fix this.

May be your country may be blocking our IPs (Cloudflare).
Try this using a proxy.


Modified screenshots to show correct name

#sign-off

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@mjcaparas Please merge this PR. Thanks


Respawn “Generating World” Bug

Name

Description:
This bug exists only for Windows 10 players. If a player falls into the void, they are greeted with an infinite “generating world” message and must close their client entirely to continue playing. This is a very annoying bug.
How to reproduce?

Only with Windows 10 players.
Fall into the void to encounter the bug.

Server version:

This server is running Pocketmine「3.11.6」API version for Minecraft: Bedrock Edition.

Other information:

Please fix this bug!

I have it too but on ipad

Thats a bug from Minecraft not the plugin

Same here. iPad Nintendo and iPhone.


Return a flag to indicate if the IPs have changed

Add a boolean flag to the return JSON.

https://github.com/achieverstech/cloudfront-ip-alert/pull/7