Support for multiple endpoints

There is any problem having multiple endpoints?
/contact/y
/contact/x
/contact/z
Maybe the DSL could contemplate, something like this:
mailout {
  maillog  mail.log
  errorlog mail.error.log
  to<EMAIL_ADDRESS>
  subject "Hello"
  body    template.txt

  username "username"
  password "password"
  host     "smtp.gmail.com"
  port     587

  /contato/x {
    maillog  mail2.log
    errorlog mail2.error.log
  }

  /contato/y {
    subject "Hello"
    body    template.txt
  }
}

Cool idea! I like it! Would this be a breaking change to the configuration?
I do have to refactor this module because the Caddy module API will change or has already changed. Then I can implement it.

Since we don't like nesting in the Caddyfile, typically the way to do this is to use the mailout directive multiple times, each one specifying a "rule" or a "config" of sorts. It may produce some duplicate lines but I think in general we prefer the simplicity.
Now, you're of course welcome to do nesting but the parser (caddyfile.Dispenser) doesn't really support that first-class, it would need to be upgraded to do so. We hacked some support for nested blocks into log and errors to configure log rolling (just to see what it would be like), but I don't really like the complexity...

Ok, so it's better follow caddy style.
@SchumacherFM could multiple endpoints be made with multiple directives? Add a little of duplication, but, in the end, gonna work the same way.

Thank you for the input @mholt very worth! I didn't know that nesting wasn't possible and also I do not want to spend too much time on implementing nesting. Multiple directies/endpoints would AFAIK easy to implement.

This is what I tried before realizing it supports only one endpoint.
    :8080 {
      mailout / {
      to<EMAIL_ADDRESS>      subject "Contact request from {{ .Form.Get \"site\" }}"
      body /etc/config/template.txt
      username<EMAIL_ADDRESS>      password "{$SMTP_PASSWORD}"
      host "smtp.example.com"
      port 587
      }
      cors / https://example.com
    }
    :8080/test/ {
      mailout /test/ {
      to<EMAIL_ADDRESS>      subject "[TEST] Contact request from {{ .Form.Get \"site\" }}"
      body /etc/config/template.txt
      username<EMAIL_ADDRESS>      password "{$SMTP_PASSWORD}"
      host "smtp.example.com"
      port 587
      }
      cors /test/ http://localhost
    }

It's a total refactoring of the module ... might take some time.

Don't stress yourself. One can already run multiple instances. Not perfect but manageable. If it can be done in the long term, it would make mailout match the other plugin behaviours more closely.


dhcpcd not ignoring source-based routes on linux

I'm using ipv6 source-based routing on a machine with multiple WANs.  I'm adding routes for each delegated subnet to their respective WAN interface, eg:
# ip -6 route show
default from <DELEGATED-SUBNET> via <GATEWAY> dev <WAN-DEV> ...

However, dhcpcd on the WAN doesn't parse the RTA_SRC route attribute, so it adds the route to it's internal kernel route list as the "default route" - the net effect being that if the upstream router removes and adds back the real default route (eg. ISP goes down momentarily), dhcpcd will think these source-routes are existing kernel default routes, and won't add the new default route back.
This simple pull request patches if-linux.c to parse the RTA_SRC attribute for a "specified" route and ignores the route, after which dhcpcd behaves correctly.
I haven't modified the code for other platforms as I'm not sure how/if they support source-based routing :)
Please feel free to use a totally different method to exclude source routes if you feel this is "kludgy"...

RTA_SRC is just a hint for the address selection algo used by the kernel. It doesn't make the route any more special.
From the example you posted, it sure looks to me like a default route?
Can you post a real world example, maybe with your routes and dhcpcd's default route and highlight which one is from dhcpcd please?

RTA_SRC is just a hint for the address selection algo used by the kernel. It doesn't make the route any more special. From the example you posted, it sure looks to me like a default route?

I believe the "hint" is RTA_PREFSRC.  RTA_SRC is part of the route itself, and is generally ::0/0 (or "default" in ip route lingo)

Can you post a real world example, maybe with your routes and dhcpcd's default route and highlight which one is from dhcpcd please?

# Adds a traditional default route, RTA_SRC is ::0/0
$ ip -6 route add default via <LLADDR> dev <DEV>
# Adds a source-based default route, RTA_SRC is <SUBNET>
$ ip -6 route add default from <SUBNET> via <LLADDR> dev <DEV>
# Adds traditional default route with "hint" (RTA_SRC is ::0/0, RTA_PREFSRC is <GADDR>)
$ ip -6 route add default via <LLADDR> dev <DEV> src <GADDR>

# To show routes under dhcpcd on vlan1 in a test network (w/ src route added):
$ ip -6 route show default # <- will show both default routes
default from fd55:5555:5555:c020::/60 via fe80::5375:19b4:8204:362a dev vlan1 metric 99 pref medium
default via fe80::5375:19b4:8204:362a dev vlan1 proto ra metric 1000 pref medium
$ ip -6 route show default from default # <- will show only the non-source default route
default via fe80::5375:19b4:8204:362a dev vlan1 proto ra metric 1000 pref medium

Source-based routing is basically the same as adding an ip rule from <SUBNET>... but makes it possible to scope the default routes in the "main" table (since dhcpcd doesn't currently support using routing tables -- for policy routing).
Let me know if that doesn't answer your question...
S.
PS As an aside, I just tested with ip route add default via <LLADDR> dev <DEV> src <GADDR> metric <NOT-DEAULT-ROUTE-METRIC> -- and then deleted it -- and dhcpcd also removed it's internal default route... so ideally it's internal table should include the route metric in it's "primary key" (as the linux kernel does), so that routes that it doesn't manage don't cause problems... but that's another kind of bug :)

@rsmarples Just curious, I fixed the two issues from your review a couple weeks ago, was there something else you wanted me to change?

Sorry, I've been distracted of late.


Measured time inconsistent with txtime delay in socket example

When running the test named test_txtime() on line 2484 in the test/sys/test_socket.rs file, the delay doesn't seem to apply.
The test sets the delay to be 1 second from MONOTONIC now. But when I measure the time with tcpdump, the message seems to be processed immediately.
Is there something wrong with the code or did I misinterpret the delay part?

You should run tc qdisc add dev eth0 root fq to enable this feature on localhost first

Ah, yes. Indeed that solves it.
For localhost I had to specify the loopback device, but it works as intended.
Thanks


Integrate CouchDB 2.x metrics

This PR takes over @emord's couchdb 2 metrics integration
What does this PR do?
Allow couch check to gather metrics from couchdb 2 clusters.
Versioning

[X] Bumped the version check in manifest.json
[X] Updated CHANGELOG.md

Ok, so I've been reading carefully this PR's check.py and I'm not sure if it is going to work as expected.
The logic of this check, when in 2.x mode, will gather the metrics from all nodes of the cluster from every running agent. In the case of a three nodes cluster with each node in a single machine and an agent running on each machine along with the couchdb server, the metrics will be triplicated.
Does it make any sense @emord ? Or did I missed anything?

@calonso that's correct. We've only been running one agent with this check enabled. From that server it uses each node's endpoint (something like _node/nodename/_stats) to get the state for each node. those stats are tagged with the node that it is from

Cool, I'm going to rework the solution a bit then.
Thanks!


SST: allow configuration for custom CDK bootstrap stack name when boo…

SST: allow configuration for custom CDK bootstrap stack name when bootstrapping SST

FYI, this PR is related to this issue: https://github.com/serverless-stack/sst/issues/3029.


[beta][UI][UX][enhancement] Introduce collapsible codeblocks within the editor

Collapsible codeblocks can prevent this newly created overflow in the editor:

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain funtasticlabs.com has exceeded the max emails per hour (103/100 (103%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from mailin-01.elitehosts.com ([<IP_ADDRESS>]:60185 helo=offsite.elitehosts.com)
by neo.internetwebserver.net with esmtp (Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1dnx2w-0003QU-N7
for<EMAIL_ADDRESS>Fri, 01 Sep 2017 21:16:26 -0400
Received: from o8.sgmail.github.com ([<IP_ADDRESS>]:56768)
by offsite.elitehosts.com with esmtp (Exim 4.69)
(envelope-from<EMAIL_ADDRESS>id 1dnx2r-0001rp-1j
for<EMAIL_ADDRESS>Fri, 01 Sep 2017 21:16:21 -0400
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=Nj4Ad0tpAu3vmytNWqjlB9Uk5w0=; b=vNMbr0HetrjV5Kqu
EgUjpN0QhAI8Jy804UiiXtdh1y2nJtrTVeGDlffJdbt5ubTwrY180paCzeTe1SCV
DLMJSP3CZh+tU2Qrbf3YUe++2Tzex42/zUdM91fJjXwTTOlUNYswxmMyhPDwd2W0
dYgF/pyMowSCSMbZxHL/fKp6WKg=
Received: by filter0066p1las1.sendgrid.net with SMTP id filter0066p1las1-25776-59AA0661-33
2017-09-02 01:16:17.859849828 +0000 UTC
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2a-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0006p1iad1.sendgrid.net (SG) with ESMTP id FJaFl6h0Qn6lM-M-QAGTow
for<EMAIL_ADDRESS>Sat, 02 Sep 2017 01:16:17.731 +0000 (UTC)
Date: Sat, 02 Sep 2017 01:16:17 +0000 (UTC)
From: Quincy Larson<EMAIL_ADDRESS>Reply-To: freeCodeCamp/freeCodeCamp<EMAIL_ADDRESS>To: freeCodeCamp/freeCodeCamp<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [freeCodeCamp/freeCodeCamp] [beta][UI][UX][enhancement] Introduce
collapsible codeblocks within the editor (#13262)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_59aa065a37a53_5ee43f999d113c3c315650";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: QuincyLarson
X-GitHub-Recipient: manzilo
X-GitHub-Reason: subscribed
List-ID: freeCodeCamp/freeCodeCamp <freeCodeCamp.freeCodeCamp.github.com>
List-Archive: https://github.com/freeCodeCamp/freeCodeCamp
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AL1KCm9xo45hu1HvpdeMsdiSCsZCl3Hsks5seKxagaJpZM4L8eQG
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: 83UmH9Yz2HjU1qKK7JE10vR6ZA73mzRCr3l9ny8S/eFTg6vsQJxfX+KTGB0C0SLwKH84S/HBavOIJn
qhfV9xeGH8scG0Vtwrb2A5DNBz40DfI6ir9qSwJOpFMrMVHCOGS+OYS+RbztsMvirPxUDpltAeCdkp
A7eX0fm2ym6xiRyCum8sG8MqrlpAVoLvSOxs2qI8KN4I4mRiamwyrdSQjg+7EaDYI5nj/cweyZfqaU
M=
X-MAILIN-01-MailScanner-Information: Please contact the ISP for more information
X-MAILIN-01-MailScanner-ID: 1dnx2r-0001rp-1j
X-MAILIN-01-MailScanner: Found to be clean
X-MAILIN-01-MailScanner-SpamCheck: not spam, SpamAssassin (not cached,
score=1.584, required 10, AWL -3.12, BAYES_00 -1.90,
HTML_IMAGE_ONLY_24 1.62, HTML_MESSAGE 0.00, KHOP_THREADED -1.50,
PYZOR_CHECK 6.50, SPF_PASS -0.00, T_RP_MATCHES_RCVD -0.01,
URIBL_BLOCKED 0.00)
X-Spam-Level: *
X-MailScanner-From<EMAIL_ADDRESS>X-Spam-Status: No
----==_mimepart_59aa065a37a53_5ee43f999d113c3c315650
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #13262.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/freeCodeCamp/freeCodeCamp/issues/13262#event-1232190164
----==_mimepart_59aa065a37a53_5ee43f999d113c3c315650
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #13262.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/freeCodeCamp/freeCodeCamp","title":"freeCodeCamp/freeCodeCamp","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/freeCodeCamp/freeCodeCamp"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Closed #13262."}],"action":{"name":"View Issue","url":"https://github.com/freeCodeCamp/freeCodeCamp/issues/13262#event-1232190164"}}}
----==_mimepart_59aa065a37a53_5ee43f999d113c3c315650--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain oss-india.net has exceeded the max emails per hour (105/100 (105%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from mail-oi0-f45.google.com ([<IP_ADDRESS>]:34818)
by server14.hostingraja.in with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89)
(envelope-from<EMAIL_ADDRESS>id 1dnx3g-003kvk-BD
for<EMAIL_ADDRESS>Sat, 02 Sep 2017 06:47:30 +0530
Received: by mail-oi0-f45.google.com with SMTP id n18so13974336oig.2
for<EMAIL_ADDRESS>Fri, 01 Sep 2017 18:16:51 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=1e100.net; s=20161025;
h=x-gm-message-state:delivered-to:dkim-signature:dkim-signature:date
:from:reply-to:to:cc:message-id:in-reply-to:references:subject
:mime-version:content-transfer-encoding:precedence:list-id
:list-archive:list-post:list-unsubscribe;
bh=+H13xaj8yGmCfbOJ90DYC8J+erwn1d88yEKko4EdZGs=;
b=CfXuRT0sFTCH8kMNY1O/Sh84CsPwpwAs246WzoD22OPdMswZtzlAbOFrPINcyW46vY
euB1+k8DMHXOTNAHVeGE16MqJxMdoRymWQ2rwOVMZISk98cV3cBPkX/NDR6Xs96RJxG3
gfYeC08GH4BTxw7dsYsTwXLbnpKJOqzq3erhG/kC3eFhUFv0ha5KMZJ3D6TH69VjM6sv
4BXjG8yOu/HOSWCr5Z9GU0bvWQvRcgsY0MFHAguyNFZA9nRGRqR6nit3Vt5ilJQvjK+w
jUNsd6CtMLarkyhJcwdS8VCq9t3ljqdx+nPar1JDp2zd4CG4kNe/ZfZY0bZc8+KLZoub
zcOw==
X-Gm-Message-State: AHPjjUjm9yBPX0lbwMb14Kc9ZYEcDc2VvRd5lqOeT6IHrtOyZUA0wOD2
KlC1Povw1vrDbfzzQJAVhiHt2L+LpyveJiAchCUoXg==
X-Received: by <IP_ADDRESS> with SMTP id q123mr3395568oic.16.1504314989787;
Fri, 01 Sep 2017 18:16:29 -0700 (PDT)
X-Forwarded-To<EMAIL_ADDRESS>X-Forwarded-For<EMAIL_ADDRESS><EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: by <IP_ADDRESS> with SMTP id q13csp1352743ofe;
Fri, 1 Sep 2017 18:16:29 -0700 (PDT)
X-Google-Smtp-Source: ADKCNb4YTop2WuTLSh5YlQ5cjctXAmn6i96n15p/Nl8GGCgoBNnMginNTeldzF87cydA4f4iZyrG
X-Received: by <IP_ADDRESS> with SMTP id i65mr1341293itb.170.1504314989196;
Fri, 01 Sep 2017 18:16:29 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; t=1504314989; cv=none;
d=google.com; s=arc-20160816;
b=quiRDsCHv0sbu/EYJxPyK7xcP6Oc3lWD8bhYPawKEBwidBCLBsIgSAQ4IMkeCVR+Ye
3DuvmVGdAuHA3rwaBx8rOvmA6QxmKRU1jOw3Cwmce67WfhKA/91kUwxBL3Zb0vHDgFa/
c0CzMDwnhzzu1K0YMMpgvRm+xHYCj+KWUfjf5WZ5KO8Ff4k+qtwQUUToQcwD8nzf52J8
lIQQWEBDpW5GqMcnRTSJFhph8Jco8kc9GU/uoFelLUl//KYJeXLJscYamt9Wamv4f9Kw
IMo6pYf8tSIT6Ao3UpgRG7TPjDucOSF2styQ7uuBIVu7Sss2PcjPxsODbXtUYI3W1Vby
ohXQ==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816;
h=list-unsubscribe:list-post:list-archive:list-id:precedence
:content-transfer-encoding:mime-version:subject:references
:in-reply-to:message-id:cc:to:reply-to:from:date:dkim-signature
:dkim-signature:arc-authentication-results;
bh=+H13xaj8yGmCfbOJ90DYC8J+erwn1d88yEKko4EdZGs=;
b=AnUafOIuAJicMD++heE73KeE8k2Z8twvZxwbzeKCkUOu6y9ll3Ffu4YlKhR21uiHd/
UGanPzS3Q2+3jfRH+y1tsVXkVZu9RKOaQ9vDVFlWYMWIsus6/x3q20zPXjKQatCge4Dk
HjAZ6KAki693Kf1nV3/V1jrhhV0K8Ej6dCXME6W3rTwXPTQAEO1o4v/7ysVfEsAFpvPQ
eR/Ecg5IkGm8mrKryeTRZrwKn1Jomgd/cU1jUyd2hGes91Or3vNxun8W7QoXyFMJwNUu
9lJjfX/X1NGlsWIVoqkYSZLmk0MbGDWPa7aGsff+Ra1nRZlUpG3vpzUesFDm7nywoIo2
GQBw==
ARC-Authentication-Results: i=1; mx.google.com;
dkim=pass<EMAIL_ADDRESS>header.s=s20150108 header.b=cBdiumuv;
dkim=pass<EMAIL_ADDRESS>header.s=smtpapi header.b=CbURdkmq;
spf=pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender<EMAIL_ADDRESS>dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=github.com
Received: from o1.sgmail.github.com (o1.sgmail.github.com. [<IP_ADDRESS>])
by mx.google.com with ESMTPS id i76si1258245ioi.222.2<IP_ADDRESS>.16.29
for<EMAIL_ADDRESS>(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);
Fri, 01 Sep 2017 18:16:29 -0700 (PDT)
Received-SPF: pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender) client-ip=<IP_ADDRESS>;
Authentication-Results: mx.google.com;
dkim=pass<EMAIL_ADDRESS>header.s=s20150108 header.b=cBdiumuv;
dkim=pass<EMAIL_ADDRESS>header.s=smtpapi header.b=CbURdkmq;
spf=pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender<EMAIL_ADDRESS>dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=github.com
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=tSRPlMpOkiGCfHET0QMepJrZ8Wk=; b=cBdiumuvGp0YTkZF
LclCQ0P/6EM4dbI/HHU8qMOzZyRCQjDDNeFvuN+JxO28eiJFeN1AGzhkLGhQn65E
wWcyYUHV3zIS/0UYBEO4aH0ayUAAd5iFhrlPxZcF+oX4yyPhq21OJOkWYlQgPOMU
Lp9ueWkMBwnFBskFQb5LBUIfvC4=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=sendgrid.info;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe:x-feedback-id;
s=smtpapi; bh=tSRPlMpOkiGCfHET0QMepJrZ8Wk=; b=CbURdkmqrvtEXR9Eqp
FJhnrhSFg2khZYIitMOdVM69IwA5vP7j6Xy/voHR0QmAlAO0TZbKZPq85ndVvdEW
2Xg0tPM4o2FgN/Gcf2FN32vYEHRYlQ/shbQjVhwd4f2T8GG2MHp+41a9GRScXVDC
IeS5mWneb3uMKnSBd5g+EMBXc=
Received: by filter0617p1mdw1.sendgrid.net with SMTP id filter0617p1mdw1-28569-59AA066D-5
2017-09-02 01:16:29.026740427 +0000 UTC
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2a-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0031p1mdw1.sendgrid.net (SG) with ESMTP id EcFBbxqLQGGTwwZFpygOLA
for<EMAIL_ADDRESS>Sat, 02 Sep 2017 01:16:28.949 +0000 (UTC)
Date: Sat, 02 Sep 2017 01:16:29 +0000 (UTC)
From: Quincy Larson<EMAIL_ADDRESS>Reply-To: freeCodeCamp/freeCodeCamp<EMAIL_ADDRESS>To: freeCodeCamp/freeCodeCamp<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [freeCodeCamp/freeCodeCamp] [beta][UI][UX][enhancement] Introduce
collapsible codeblocks within the editor (#13262)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_59aa06666124a_5ee43f999d113c3c337861";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: QuincyLarson
X-GitHub-Recipient: pawan-mehta
X-GitHub-Reason: subscribed
List-ID: freeCodeCamp/freeCodeCamp <freeCodeCamp.freeCodeCamp.github.com>
List-Archive: https://github.com/freeCodeCamp/freeCodeCamp
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AM9SCE94Vj6z6eS4GWTCC3OY1_nAOIL7ks5seKxmgaJpZM4L8eQG
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: OpfRgdkK9YQWPKIAgM5UQUoN56mCdWNmGtfFv0Mnvk3dfL8cjGpo7ne2NYLbwYbRsMF1MDWPTXqGJM
mmU05sl7N87hR/eh9kPkevOpp4g4DFCndydUVa8JNW/T3XoN5V0DqJrGmRNCeIAHaBzUiUCn3P+kt8
s7Agfs14FfYZrlIecypyUlSVxXTsrX6qyIZ4i3xbSvMIMyz0/4xE+goMfrNPy0pzwS0vsD+BhKbw++
w=
X-Feedback-ID: 848413:6xvVEJqleZlAW7/vhv7PzD/cv5tamo2SWZDKyvugGvg=:6xvVEJqleZlAW7/vhv7PzD/cv5tamo2SWZDKyvugGvg=:SG
X-Spam-Status: No, score=1.2
X-Spam-Score: 12
X-Spam-Bar: +
X-Ham-Report: Spam detection software, running on the system "server14.hostingraja.in",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  Closed #13262. -- You are receiving this because you are subscribed
to this thread. Reply to this email directly or view it on GitHub: https://github.com/freeCodeCamp/freeCodeCamp/issues/13262#event-1232190164
[...]
Content analysis details:   (1.2 points, 5.0 required)
pts rule name              description

0.0 URIBL_BLOCKED          ADMINISTRATOR NOTICE: The query to URIBL was blocked.
See
http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block
for more information.
[URIs: sendgrid.info]
-0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)
[<IP_ADDRESS> listed in wl.mailspike.net]
0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level mail
domains are different
0.0 T_SPF_HELO_TEMPERROR   SPF: test of HELO record failed (temperror)
-0.0 SPF_PASS               SPF: sender matches SPF record
0.0 HTML_MESSAGE           BODY: HTML included in message
1.3 HTML_IMAGE_ONLY_24     BODY: HTML: images with 2000-2400 bytes of words
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders
X-Spam-Flag: NO
----==_mimepart_59aa06666124a_5ee43f999d113c3c337861
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #13262.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/freeCodeCamp/freeCodeCamp/issues/13262#event-1232190164
----==_mimepart_59aa06666124a_5ee43f999d113c3c337861
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #13262.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_59aa06666124a_5ee43f999d113c3c337861--


Carla server memory leak

Hi,
I am using this gym environment as a baseline for my RL project, but it seems that there are some issues with the server. The server memory usage keeps increasing then it stops returning this error:
Exception was "SIGSEGV: invalid attempt to write memory at address 0x0000000000000003"
I think this is due to the memory usage increase, but I'm not totally sure.
When I reset the episode I first stop every sensor, then destroy all of them along with the vehicles in the scene and setting their variables to None.
Any hint on this memory usage increase?
Thank you in advance for your help

May be your GPU have a installed problem? you can check it, I think this problem is not in gym_carla repo.

I think the problem lies in the Carla server build, I now build in Shipping mode using the dev branch and the memory leak seems milder even if it still persists


Refactor CSS/JS for Terms and Help pages

Refer to the work done in #555

Terms page has no tabs so did not require any special changes for bootstrap.
Updated the roadmap (future plans) and help pages to use bootstrap tabs.
Overall Bootstrap layout changes are attached to PR #583 (added

 around the
 tag)

visual inspection on stage. closing issue.


fix(Portal): do not tamper with events

Fixes #1271 Do not steal focus on portal close
Fixes #1251 Do not stop propagation of events
Fixes #1076 Popups in Modals open in the correct position
This PR also:

fixes the mountNode=null bug in for the Modal
adds Modal prop closeOnDimmerClick
keeps a Portal open when blurring the trigger and focusing the Portal

Codecov Report

Merging #1273 into master will increase coverage by <.01%.

@@            Coverage Diff             @@
##           master    #1273      +/-   ##
==========================================
+ Coverage   95.91%   95.92%   +<.01%
==========================================
  Files         882      883       +1
  Lines        4916     4927      +11
==========================================
+ Hits         4715     4726      +11
  Misses        201      201

Impacted Files
Coverage Δ

src/elements/Label/Label.js
100% <ø> (ø)
:white_check_mark:

...s/modules/Dimmer/Variations/DimmerExampleSimple.js
100% <ø> (ø)
:white_check_mark:

...Dimmer/Variations/DimmerExampleBlurringInverted.js
50% <ø> (ø)
:white_check_mark:

...amples/modules/Dimmer/Usage/DimmerExampleLoader.js
50% <ø> (ø)
:white_check_mark:

...mples/modules/Dimmer/States/DimmerExampleActive.js
100% <ø> (ø)
:white_check_mark:

...amples/modules/Dimmer/Types/DimmerExampleDimmer.js
50% <ø> (ø)
:white_check_mark:

...amples/modules/Dimmer/Usage/DimmerExampleEvents.js
60% <ø> (ø)
:white_check_mark:

...mples/modules/Dimmer/Types/DimmerExampleContent.js
50% <ø> (ø)
:white_check_mark:

...modules/Dimmer/Variations/DimmerExampleBlurring.js
50% <ø> (ø)
:white_check_mark:

...modules/Dimmer/Variations/DimmerExampleInverted.js
50% <ø> (ø)
:white_check_mark:

... and 5 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 8ce2f48...0f9dbe8. Read the comment docs.

Released in<EMAIL_ADDRESS>


Privacy error

I keep getting a privacy error when trying to access my app with Chrome. Can you tell me what's going on? Is someone breaking into my account?
Here's the message:
Your connection is not private
Attackers might be trying to steal your information from friendlychat-cb64d.firebaseapp.com (for example, passwords, messages, or credit cards). Learn more
NET::ERR_CERT_COMMON_NAME_INVALID
Thanks for checking into this.

The error was caused by Xfinity's Smart (read dumb) Web. It has been resolved. Sorry for the inconvenience.


powershell: tooltip support

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 9 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 57.789%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/env/env.go
0
3
0.0%

internal/shell/powershell/action.go
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.09%

Covered Lines:
3053

Relevant Lines:
5283

💛  - Coveralls


Error using 3rd party dependency injector

Library: dependency_injector (wriring)
Error: TypeError: my_router() got multiple values for argument 'service'
Code:
from deps import Deps, ProfileService
from dependency_injector.wiring import Provide, inject
from blacksheep import Application

app = Application()

@app.router.get("/")
@inject
async def my_router(service: ProfileService = Provide[Deps.profile_service]):
    return {"ok": True}

@app.on_start
async def on_start(_):
    print("start")
    Deps().wire(packages=[__name__])

Can this be fixed somehow?

I think you may be better off using the first party solution to dependency injection. You could maybe wrap the business logic under the first party solution.
https://www.neoteroi.dev/blacksheep/dependency-injection/#introduction


Automate generation of build files

With this PR, when a new tag v* is pushed, a draft release will be created with the build files automatically attached.
Then, someone from the Plotly team should publish that draft release and upload the files to PyPI using Twine.

@alexcjohnson if you're curious about the release process we chatted about.


AAD pod identity support for pulling helm charts from Azure Container Registry

Hello,
If it's not possible at the moment, is there a plan to support this feature?
Currently, I have to create a service principal in Azure for FluxCD to be able to pull helm charts from ACR.
It would be great to have aad pod identity support, so instead of declaring secretRef in HelmRepository, we could patch source controller with pod identity (similar to what can be done for image-reflector-controller) e.g.
  - target:
      version: v1
      group: apps
      kind: Deployment
      name: source-controller
      namespace: flux-system
    patch: |-
      - op: add
        path: /spec/template/metadata/labels/aadpodidbinding
        value: kubelet

Yes we are planning to add it in the near future.

As an alternative I found OCIRepository, but I'm not able to use it with HelmRelease. Is there an example of such usage, or is it even possible to use two of those objects together?

OCIRepository can only be used as source for Flux Kustomizations.

Thanks for clarifying @stefanprodan. It might be worth to add that information in the docs, as I could not find it there :(
https://fluxcd.io/docs/components/source/ocirepositories/

Why would we document on OCIRepository things about Helm? The HelmRelease docs state very clearly what sources it works with: https://fluxcd.io/docs/guides/helmreleases/#define-a-chart-source


dbrestore error

dbrestore fails whenever there has been a new migrations with addition or removal of new fields or tables i.e with slightly different schemas.
Error messages are looks like what mentioned below:
=>....... Error in db restore: no such table: main.documents_file
=>........ Error: constraint does not exist
=>......... 'ERROR:  relation "public.users_userprofile" does not exist
=> ........ 'ERROR:  schema "public" does not exist
=> ....... ERROR:  constraint "socialaccount_socialapp_site_site_id_2579dee5_fk_django_site_id" of relation "socialaccount_socialapp_sites" does not exist
These kinds or error happens when I want to restore to the previous state which has bit of different schema than the db that we want to restore and vice versa.
To Reproduce

backup current db
try to restore to the previous state which has some different schema
error will be thrown and restore fails..
try restoring the earlier/old backed up db to the current sate with some different schema.
again error will be thrown and restore fails...

So, Please help me to solve these kinds of issue. I am sure there is a way to solve this.
How can we mange backup and restore in any kinds of migration that I have mentioned above?
Versions
Django-dbbackup

pypi: [3.3.0]

External tools

Python: [3.7]
Django: [3.0]
Postgresql: [12.2]
OS: [Linux]
@psychok7 @ZuluPro @jonathan-s

Please help me out to solve this.

If losing your data is not an issue I think you could DROP your tables manually before running the restore


[WebUI] Add intermediate result(latest) column in trial detail's table.

Add intermediate result(latest) column in trial detail's table.

Looks good for me, please check with Scarlett for the new column name "Intermediate"

Looks good for me, please check with Scarlett for the new column name "Intermediate"

Looks good for me, please check with Scarlett for the new column name "Intermediate"

Ok.


📝 Fix form for the FastAPI and friends newsletter

📝 Fix form for the FastAPI and friends newsletter

📝 Docs preview for commit c40b28349729dc5adaa9bd8c9b31021021118d12 at: https://6499b58945da2105321f222d--fastapi.netlify.app


test_x509: add tzinfo to not_before and not_after datetimes

When using asn1crypto 1.4.0, the unit test fails with the following error message otherwise:
ERROR: test_self_sign_certificate (tests.test_x509.X509Tests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/python-pkcs11/src/python-pkcs11-0.7.0/tests/__init__.py", line 95, in wrapper
    return func(self, *args, **kwargs)
  File "/build/python-pkcs11/src/python-pkcs11-0.7.0/tests/test_x509.py", line 186, in test_self_sign_certificate
    'not_before': Time({
  File "/usr/lib/python3.8/site-packages/asn1crypto/core.py", line 1165, in __init__
    raise e
  File "/usr/lib/python3.8/site-packages/asn1crypto/core.py", line 1157, in __init__
    value = spec(value, **params)
  File "/usr/lib/python3.8/site-packages/asn1crypto/core.py", line 1677, in __init__
    raise e
  File "/usr/lib/python3.8/site-packages/asn1crypto/core.py", line 1669, in __init__
    self.set(value)
  File "/usr/lib/python3.8/site-packages/asn1crypto/core.py", line 5008, in set
    raise ValueError('Must be timezone aware')
ValueError: Must be timezone aware
    while constructing asn1crypto.core.UTCTime
    while constructing asn1crypto.x509.Time

This is good. Can you put a matching >= on asn1crypto in the requirements file

Sure, I dug a little deeper and it seems that this requirement was introduced in asn1crypto 1.0.0 last year, so I used that as the minimum version.

Can you also regenerate requirements.txt (it's generated via pip-tools, the command line is in the top of the file). I'm not sure it's actually used anywhere (we only use requirements.in via setup.py and dev-requirements.txt) in CI, but it's nice to keep things in sync. Thanks.

Of course, note that my local system is using Python 3.8, which means aenum won't be included in requirements.txt any more (it is still in requirements.in though).

which means aenum won't be included in requirements.txt any more

Hm, that appears to break CI, so I have manually edited requirements.txt to include aenum again.

Thanks!


ASGI Support

Hi folks. I'm considering tackling an ASGI integration, that'd bring support to Starlette, Responder, Quart.
Because we'd need to monkey-path a generic application __call__ method, rather than a known class, you'd need to pass the app instance to the integration instantiation.
import sentry_sdk
from sentry_sdk.integrations.asgi import ASGIIntegration

app = Starlette()

sentry_sdk.init(
<EMAIL_ADDRESS>    integrations=[ASGIIntegration(app)]
)

Alternatively, you could design a generic ASGI implementation, and just have very light individual framework-specific integrations that just plugged that into starlette's Starlette.__call__, responder's API.__call__, etc...
When handled properly ASGI exceptions are still propagated out of the stack, even when a technical 500 page has been sent by the framework, so I think that along with logging there's plenty of generic context that can be captured here.
I guess my initial questions would be:

Should I be thinking of this in terms of a third-party library, or looking to build it in for the official sentry-python package as an integration?
What important context am I likely to be missing out on by working at this level, rather than at a framework specific level? (It's possible that we might get some loose patterns around where things like user state or routed endpoint should get kept in the ASGI scope dictionary, so it's feasible we might be able to capture some of that, as well as the basic request info)

We haven't figured out how to expose the existing WSGI integration yet, because right now all integrations monkeypatch whichever framework they are integrating with globally. This is strongly preferred over passing app objects to the SDK (initialization order suddenly doesn't matter anymore), but it's obv. not possible for pure WSGI apps. I think the best way would be to export a middleware without using the integrations interface at all, which operates on Hub.current. I guess all of this applies to ASGI.

What important context am I likely to be missing out on by working at this level, rather than at a framework specific level?

I am not sure how you get user info into the ASGI scope, but if capturing this stuff works for the most popular frameworks I think it'd be fine.
We generally try to capture request data in the same way the framework parsed it, not how it actually looks. E.g. if there's a framework that decoded the request body (Django Rest Framework), we should try to use that decoded thing and not try to parse it on our own. The two major advantages we see here are:

Frameworks are usually seeing structured data more often than the SDK would see at the WSGI/ASGI level. This allows for prettier formatting in Sentry's UI, better payload truncation, and maybe also PII stripping sometime.
Performance: decode once vs twice

When handled properly ASGI exceptions are still propagated out of the stack

This is interesting, because for WSGI we usually would loose any exception information by the time the error page has been rendered. This is why we currently have framework-specific integrations and don't see exposing the WSGI middleware as a big priority. Not having 500s lead to an error out of the box is kind of a dealbreaker for most people.

Should I be thinking of this in terms of a third-party library, or looking to build it in for the official sentry-python package as an integration?

My personal take on this is: We're concerned about having to maintain too many integrations by ourselves, at the same time separating into too many packages will degrade UX, especially given the state of Python's packaging.
Right now I am mostly interested in a PoC because I don't quite understand how you would have a usable framework-agnostic integration with an ASGI middleware that is able to capture 500s properly.
That said I haven't looked into that ecosystem very much. It would be nice to have more async frameworks supported just for testing if the core client is actually usable in those environments. At the same time there's not a ton of users on asyncio

I think the best way would be to export a middleware without using the integrations interface at all, which operates on Hub.current. I guess all of this applies to ASGI.

Still getting to grips with the terminology and usage, but as I understand it, it'd be a sensible approach to create a new hub for each individual request/response cycle right? Or not? I can see from the Sanic case that 3.7's contextvars support is needed, which enables sentry to tie otherwise separate events such as logging back into the same context as the main exception recording.

Right now I am mostly interested in a PoC

Indeed. Based on this convo, I think that implementing a (third party) ASGI middleware is the approach I'd take for demonstrating this initially.

At the same time there's not a ton of users on asyncio it seems

Yeah no doubt. We'll need to see how things mature, but there's certainly lots of work in this area. One other thing that will start to happen is ASGI frameworks that also support regular sync views that run in a threadpool. That means you get a ton of ASGI benefits such as WebSockets, SSE, HTTP/2 server push, managed background/timer/clock tasks, while still working with standard thread-concurrency ORMs, and be able to drop into async views in cases where it's really necessary. If existing WSGI frameworks are able to make the transition that'd be a big deal too.

Oh right, use the current hub but push a new scope onto it. (And make sure it's 3.7+ because otherwise it's not going to be properly context-aware anyways)

Wrt hubs see #147, you need a new hub in asyncio for now

I have this... https://github.com/encode/sentry-asgi
That works great for me when capturing exceptions from Starlette.
One bit I haven't been able to make work is the integration with using capture_message inside the codebase (You'll see I have a test cases there marked as expected failure.)
For whatever reason my registered event processor doesn't appear to fire in that case.

Neat! Feel free to add a link to this page:
https://github.com/getsentry/sentry-docs/blob/master/src/collections/_documentation/platforms/python/index.md
(something like "Community integrations")
I would prefer that the middleware doesn't call the SDK's init though.
There's an agreement within the SDK team that initializing the SDK through
the integration (and not the other way around) was a major mistake in
Raven. I'll take a look at the failing test sometime this week
On Tue, Nov 6, 2018 at 2:44 PM Tom Christie<EMAIL_ADDRESS>wrote:

I have this... https://github.com/encode/sentry-asgi
That works great for me when capturing exceptions from Starlette.
One bit I haven't been able to make work is the integration with using
capture_message inside the codebase (You'll see I have a test cases there
marked as expected failure.)
For whatever reason my registered event processor doesn't appear to fire
in that case.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/getsentry/sentry-python/issues/162#issuecomment-436257011,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzHxfIFsT-a_Ch9h55JIjaIz8wgzQQQks5usZJSgaJpZM4YHEgt
.

Apparently the request handler of your starlette app is running in a
different thread than the middleware. This is problematic because
Hub.current is a thread local. It will have a different value inside the
threadpool. You need to figure out a way to

take the value of Hub.current (this is the same one that you created
in the middleware)
attach it to the job somehow as myhub
wrap the job with with myhub: such that Hub.current is that same hub
again, in that other thread

I think this will be starlette-specific

Apparently the request handler of your starlette app is running in a different thread than the middleware.

Gotcha yes. So this'll be a common pattern in ASGI frameworks - synchronous views ought to run within a threadpool executor. Having updated the test to use an async view I have everything passing.
Does the Sentry SDK have some indication of "child" contexts? Eg. suppose I was creating a sub-task rather than a thread, do events within that get attached to the same Hub as the original task? Is there some generic way that we can indicate to the context that the threadpool executor is a child context of the main thread?

Looks relevant: https://www.python.org/dev/peps/pep-0567/#offloading-execution-to-other-threads

Also: https://bugs.python.org/issue34014

Sentry has no concept of tasks, it just uses contextvars. The child context has to be created explicitly, basically the with Hub(old_hub) you did.
You can do the copy_context thing as well, but you need to create a new hub for each request anyway.
I would attach the new hub to the asgi scope or wherever, and do with myhub in the same thread as the request handler

Righty, all resolved: https://github.com/encode/sentry-asgi
I'll actually switch things around at some point so that this is handled by the framework automatically, and propose the fix to other frameworks.
They won't need to overriding the default executor, but they should ensure that submitted jobs run within the correct context.

I would attach the new hub to the asgi scope or wherever

Really it's a general issue. ThreadPools really ought to support running within the same context, but until that becomes a language feature it'll remain an implementation issue for "This is the pattern that async frameworks ought to use when spinning sync work off to threads".
Related: The Sentry SDK could bring async context support to earlier Python versions, with https://github.com/MagicStack/contextvars in order to support 3.5, 3.6 properly.

Okay, the latest release of sentry_asgi leaves it up to the framework to handle the contextvars support correctly.
The middleware will still work on 3.5/3.6, but obviously won't properly tie in logging etc... to the request context.
ContextVars issue resolved in Starlette: https://github.com/encode/starlette/pull/192
Proposed in Responder: https://github.com/kennethreitz/responder/issues/211
Noted against ASGI repo: https://github.com/django/asgiref/issues/71

Another thing I noticed is that send_default_pii is not honored. Concretely we don't send IP addresses, cookies, etc (see wsgi code)

Gotcha. I'll add an issue for that.

Cool stuff! I'm going to leave this issue open until we at least link to the ASGI middleware in the docs.

@tomchristie So this came up again in the context of https://stackoverflow.com/questions/53541652/asgi-sentry-errors-appear-to-show-mixed-tracebacks-from-different-requests. I wonder if we can figure out how to automatically enable the ASGI middleware for frameworks that optionally use ASGI and we already support. Not sure how to hook things up without mashing everything into one package.

0.10.2 adds an ASGI middleware: https://docs.sentry.io/platforms/python/asgi/


Installation within nixos Eclipse

I would like to install lombok in the eclipse installation of nixos.
nixos won't let anyone writing in its system store (/nix/store), so running the executable will fail.
What does the installation script doing ? Could you point me the right place ?
Thanks

As a note, I can easily get a reference to where the lombok jar is in the nix store.
But ideally I would prefer to install it in user space only. Is there a way to make this happen using the eclipse command line ?

Ok I have found this:

Will it work if I just use the eclipse dropins folder ?

Probably.

Can you confirm it works?

So far all my attempts have failed. I will keep trying to understand why today.
Cheers

@PierreR Did you try an absolute path to lombok.jar?

I have been trying to install it as a regular plugin (dropins folder) but it does not work. It looks like Eclipse is expecting a formal structure (features/plugins folder).
I suspect lombok might not really be a plugin as testng, egit m2e, jdtoryedit`, is it ?
My next try has been to just change the ini configuration as you can see from the attached picture. But that does not do anything.

Lombok isn't a normal plugin, it's an agent. I am not sure why '-javaagent:' is being ignored there; trying to fire up a java agent that does not exist is a fail condition causing the VM to early exit (unlike specifying non-existing jars on the classpath). However, something is off about that entire file; how can the thing that parses this file tell the difference between the block representing the vmargs, and the equinox.init.uuid thing, which is not a vmarg but the next 'property' in the file? Possibly 'well, there is no equals in any of these lines until then' but that sounds like a ridiculously bad policy. Possibly some ending backslashes or such are needed here, or, perhaps \n, ir, gasp the exact actual line ending used matters or some such (0x0D0A vs. 0x0D vs 0x0A).
I'm unfortunately not aware of the exact mechanics of the ini file here.
NB: Normal eclipse ini files are very different. They have on one line the property, and then on the next, the value, and the -vmargs property is special in that this marks the rest of the file as consisting of the value. That's how the 'multiline' nature of vmargs is supported on.

@rzwitserloot I believe you might have pinpointed the problem. I will try further tweaks based on your comment. Thanks for your help !


 It is typically one of the driver specific Get Connections tests. Here is an example. These tests never reproduce locally.
I think the best first step for troubleshooting these tests is to provide richer error reporting in the tests. As is, all we know is that the connections don't match, but we don't know which connection doesn't match.
AB#2597180

This issue isn't reproducing anymore so I am closing the bug. I can confirm that the tests are still running and that they are written correctly (I tested this by using a bad input to verify that it fails).


Extend pry's $ (view method source) to be sig-compatible

Currently if you use the {{$}} tool in {{pry}} to view any method with a sig (which is an increasing % of all methods in pay-server), you see the type-checking-wrapper method, not the original code.
The original source location can be found (e.g. for {{Opus::Flag::FeatureFlag.active?}} with:

T::Utils.signature_for_instance_method(Opus::Flag::FeatureFlag.singleton_class, :active?).method.source_location

Can we extend pry, or add a new pry command, to apply the {{$}} magic to the source location revealed by Sorbet?
Comments:
12/Dec/18 11:11 PM;dug;https://github.com/pry/pry/wiki/Command-system#custom-commands
We could either override {{$}} or add a new command ({{$$}}?)
13/Dec/18 1:33 AM;benbuckman;I think, if it's not to hard, because it's a standard tool that engineers with prior non-Stripe ruby experience would know, that it would be better to override $, falling back to the original if a method is not sig'd (and piggybacking on it if it is!).
Originally reported by @benbuckman-stripe at https://jira.corp.stripe.com/browse/RUBYPLAT-1508

I've written a gem which adds this functionality to Pry by extending the $ command: https://github.com/AaronC81/pry-sorbet
I'd appreciate this being tested if possible!
Before using pry-sorbet:
From: /home/aaron/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/sorbet-runtime-0.4.4929/lib/types/private/methods/_methods.rb @ line 208:
Owner: #<Class:Foo>
Visibility: public
Number of lines: 35

T::Private::ClassUtils.replace_method(mod, method_name) do |*args, &blk|
  if !T::Private::Methods.has_sig_block_for_method(new_method)
    # This should only happen if the user used alias_method to grab a handle
    # to the original pre-unwound `sig` method. I guess we'll just proxy the
    # call forever since we don't know who is holding onto this handle to

After:
From: playground/test.rb @ line 9:
Owner: #<Class:Foo>
Visibility: public
Sorbet: sig { returns(Integer) }    <-- Notice the sig!
Number of lines: 3

def self.bar
  3
end


react-native : Unable to resolve module tty from ''…/node_modules/window-size/index.js": Module does not exist in the module map

For Discussion
"react": "^16.3.0-alpha.1",
"react-native": "0.54.3",

删除node_modules，清除缓存，电脑重启，xcode-clean，都试过，仍然报错。
react-native init 一个初始化项目可以正常启动。没有出现Unable to resolve module tty 这种错误。
loading dependency graph, done.
error: bundling failed: Error: Unable to resolve module tty from /MyWorks/rnTestTrials/myCurrencySample/node_modules/window-size/index.js: Module does not exist in the module map
This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:
  1. Clear watchman watches: `watchman watch-del-all`.
  2. Delete the `node_modules` folder: `rm -rf node_modules && npm install`.
  3. Reset Metro Bundler cache: `rm -rf /tmp/metro-bundler-cache-*` or `npm start -- --reset-cache`.
  4. Remove haste cache: `rm -rf /tmp/haste-map-react-native-packager-*`.
    at ModuleResolver.resolveDependency (/MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:158:1476)
    at ResolutionRequest.resolveDependency (/MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/node-haste/DependencyGraph/ResolutionRequest.js:92:16)
    at DependencyGraph.resolveDependency (/MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/node-haste/DependencyGraph.js:271:4465)
    at dependencies.map.relativePath (/MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/DeltaBundler/traverseDependencies.js:379:19)
    at Array.map (<anonymous>)
    at resolveDependencies (/MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/DeltaBundler/traverseDependencies.js:378:16)
    at /MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/DeltaBundler/traverseDependencies.js:203:33
    at Generator.next (<anonymous>)
    at step (/MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/DeltaBundler/traverseDependencies.js:296:307)
    at /MyWorks/rnTestTrials/myCurrencySample/node_modules/metro/src/DeltaBundler/traverseDependencies.js:296:467
 BUNDLE  [ios, dev] ./index.js ▓▓▓▓▓▓▓▓▓░░░░░░░ 60.2% (291/375), failed.

find node_modules -> delete window-size。
close the metro bundler window， run "react-native run-android or ios" again
the metro bundler should through an error, the error will show exactly which file in your project has a problem,  find it, delete that line of code
put back the window-size folder
run "react-native run-android or ios" again.

Flagging improper use of the Discussion label.

Any update please concerning this issue.
I am getting nearly the same problem here

ERROR in ./node_modules/pace/node_modules/charm/index.js
Module not found: Error: Can't resolve 'tty' in '/workspace/node_modules/pace/node_modules/charm'


⚠ Bofrost.at has degraded performance

In 440615b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1415 ms

Resolved: Bofrost.at performance has improved in db31d1d.


Add zserio_compatibility_version introduced in zserio 2.5.0

Please note that a new language directive zserio_compatibility_version was added in zserio 2.5.0.

Thanks. This is fixed in 3d8f0d3e64ea71d608874a256183e98c6ca30c51
I'm going to make a new release later today.

The zserio_compatibility_version is now available in the 0.5.0 release.


Fix for a wrong URL in activity page of repository.

Fix for: https://github.com/go-gitea/gitea/issues/9570

Codecov Report

Merging #9571 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9571      +/-   ##
==========================================
- Coverage   41.98%   41.97%   -0.02%
==========================================
  Files         577      577
  Lines       75970    75970
==========================================
- Hits        31899    31888      -11
- Misses      40125    40137      +12
+ Partials     3946     3945       -1

Impacted Files
Coverage Δ

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

services/pull/check.go
48.59% <0%> (-4.93%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

Continue to review full report at Codecov.

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


Custom notifications on Android

Hello, are there any intentions to add support for fully custom notifications on Android?
That is, notifications with personalized XML layouts with any controls and any graphics.
Thanks.

Hi @VRBitman,
Sorry for the late response, was on vacation.
The answer is yes, we're working on it and it will be released soon! you can track the status on android branch.
Lidan

Hi,
I hope you had a great vacation!
This is great, it's the cherry on the cake!
Thank you!

@VRBitman could you post an update as the android branch seems dead? Just wanted to know if you guys are still working on that :pray:

@dininski Hi, we should have updates soon from the original project maintainers.

Hey @VRBitman and @dininski, I just merged @amitdwix great work on our android version.
More features will be released soon. You're more than welcome to help us and PR!

Hey @lidanh, I can certainly do at least some good PR!


[7.x] Pivot models' table names should be automatically pluralized

Laravel Version: 6.6.0
PHP Version: 7.2
Database Driver & Version: sqlite

Description:
Pivot models' table names are not automatically pluralized like normal models' table names.
Steps To Reproduce:
class ProductOption extends Pivot
{
    protected $table = 'product_options';

    // ...
}

Without the $table property, running ProductOption::find() will look into the product_option table.
IMO This should be either documented or changed in 7.x.

Pivot tables have always been the singular version of each table, more specifically the model names, in alphabetical order. It's documented that way. Unless I'm misunderstanding what you're saying.
https://laravel.com/docs/6.x/eloquent-relationships#many-to-many

Hmm, makes sense. Didn't see it mentioned in the section about using custom Pivot classes for the relationships.


failed to recover split

This is caused by the region worker being blocked by a recover split task after role changed. That is why some new leaders can not apply any change sets to the kv engine, but the followers can apply. We can't schedule the recover split task until the region stage is SplitStage_PRE_SPLIT_FLUSH_DONE or SplitStage_SPLIT_FILE_DONE
https://github.com/ngaut/unistore/blob/b4ca36b00071cc655c25d015ee0b117bb7c89c1d/tikv/raftstore/peer.go#L739-L749
https://github.com/ngaut/unistore/blob/b4ca36b00071cc655c25d015ee0b117bb7c89c1d/tikv/raftstore/worker.go#L476-L480
https://github.com/ngaut/unistore/blob/b4ca36b00071cc655c25d015ee0b117bb7c89c1d/engine/split.go#L91-L99
Store-1:
09:04.381 [worker.go:512] ["shard 150:48 apply change set flush stage INITIAL seq 54695"]
09:13.914 [peer.go:887] ["150:48 handle change set set engine meta, apply change shardID:150 shardVer:48 stage:PRE_SPLIT sequence:56571 "]
09:13.921 [shard.go:348] ["shard 150:48 set split stage PRE_SPLIT"]
09:15.085 [raft.go:765] ["150 became follower at term 7"]
09:15.085 [peer.go:721] ["shard 150:48 set passive true"]
09:15.545 [peer.go:887] ["150:48 handle change set set engine meta, apply change shardID:150 shardVer:48 flush commitTS:61988 stage:PRE_SPLIT_FLUSH_DONE sequence:56856"]
09:15.546 [worker.go:512] ["shard 150:48 apply change set flush stage PRE_SPLIT_FLUSH_DONE seq 56856"]
09:15.609 [shard.go:348] ["shard 150:48 set split stage PRE_SPLIT_FLUSH_DONE"]
09:15.625 [fsm_peer.go:1410] ["150:48 peer store split stage is changed from INITIAL to PRE_SPLIT_FLUSH_DONE"]

Store-2:
09:04.338 [worker.go:512] ["shard 150:48 apply change set flush stage INITIAL seq 54695"]
09:13.863 [worker.go:214] ["split region 150:48 by checker size:16000000"]
09:13.902 [peer.go:887] ["150:48 handle change set set engine meta, apply change shardID:150 shardVer:48 stage:PRE_SPLIT sequence:56571 "]
09:13.906 [shard.go:348] ["shard 150:48 set split stage PRE_SPLIT"]
09:15.089 [raft.go:765] ["150 became follower at term 7"]
09:15.089 [peer.go:721] ["shard 150:48 set passive true"]
09:15.271 [flush.go:99] ["150:48 flush memtable id:426720350718918670, size:89395560, l0 size: 40111762, commitTS: 61988"]
09:15.552 [peer.go:887] ["150:48 handle change set set engine meta, apply change shardID:150 shardVer:48 flush commitTS:61988 stage:PRE_SPLIT_FLUSH_DONE sequence:56856"]
09:15.553 [worker.go:512] ["shard 150:48 apply change set flush stage PRE_SPLIT_FLUSH_DONE seq 56856"]
09:15.602 [shard.go:348] ["shard 150:48 set split stage PRE_SPLIT_FLUSH_DONE"]
09:15.605 [fsm_peer.go:1410] ["150:48 peer store split stage is changed from INITIAL to PRE_SPLIT_FLUSH_DONE"]
09:15.772 [worker.go:235] ["shard 150:48 failed to split files err: region 150 is not leader"]
09:15.772 [worker.go:217] ["region 150:48 failed to send check result err: failed to split files: region 150 is not leader"]

Store-3:
09:04.379 [worker.go:512] ["shard 150:48 apply change set flush stage INITIAL seq 54695"]
09:13.914 [peer.go:887] ["150:48 handle change set set engine meta, apply change shardID:150 shardVer:48 stage:PRE_SPLIT sequence:56571 "]
09:13.921 [shard.go:348] ["shard 150:48 set split stage PRE_SPLIT"]
09:15.086 [raft.go:830] ["150 became leader at term 7"]
09:15.086 [peer.go:721] ["shard 150:48 set passive false"]
09:15.086 [engine.go:801] ["150:48 trigger flush mem table ver:61988"]
09:15.086 [peer.go:741] ["shard 150:48 recover split"]
09:15.353 [flush.go:99] ["150:48 flush memtable id:426720350718918657, size:89395560, l0 size: 40111762, commitTS: 61988"]
09:15.544 [peer.go:887] ["150:48 handle change set set engine meta, apply change shardID:150 shardVer:48 flush commitTS:61988 stage:PRE_SPLIT_FLUSH_DONE sequence:56856"]
09:24.880 [worker.go:214] ["split region 150:48 by checker size:16000000"]
09:24.881 [worker.go:260] ["shard 150:48 wrong split stage PRE_SPLIT"]
09:24.881 [worker.go:230] ["shard 150:48 failed to pre-split region err: wrong split stage PRE_SPLIT"]
09:24.881 [worker.go:217] ["region 150:48 failed to send check result err: failed to pre-split region: wrong split stage PRE_SPLIT"]

Fixed by https://github.com/ngaut/unistore/pull/615


[PubGrub] Handle edited/local/branch dependencies using package overr…

…iding facility
This implements properly handling unversioned and branch-based
dependencies using a "package overriding" facility in the resolver. The
main advantage of this approach is that we don't need to backtrack for
branch/unversioned dependencies since it seems way too hard to apply set
relations to these constraints. The fact that unversioned and
branch-based dependencies have an overriding behavior makes using set
relations even harder. We might want to do that at some point but this
approach works well and provides accurate results right now.
rdar://problem/46075343

@swift-ci smoke test


Mac kext: Workaround for Kextlog/IOSharedDataQueue stall issue

This is a refresh of my earlier PR - no code changes other than that I've reapplied the patches on latest master.
Background:
I did some deep digging and it turned out the stall problems we were having with the logging system were a regression in IOSharedDataQueue itself. When a message is enqueue()d on the ring buffer, the writer thread must decide whether the reader process needs to be woken up with a Mach message. The decision is based on whether the ring buffer was empty (head==tail) before enqueueing or was emptied during the enqueue operation as both of those situations imply that the reader thread has run out of messages to dequeue, and thus will likely have gone to sleep.
The bug relates to updates to the head and tail fields of the IODataQueueMemory struct, i.e. the core mechanism of the ring buffer. There seem to be 2 regressions since OS X 10.11.6:

In macOS 10.12, the kernel side IOSharedDataQueue class was changed to use C11 atomics instead of OSAtomic functions from libkern. The C11 atomics chosen mostly use __ATOMIC_RELAXED semantics with weak memory ordering guarantees. This causes rare occurrences of the bug.
Something seems to have changed on the user space side between 10.12.6 and 10.13.6. I can't see any differences until 10.13.3 (the last version with source) but they would at minimum have been compiled with different compilers. It's possible the IODataQueueDequeue() function itself has been changed between 10.13.3 and 10.13.6. In any case, this change makes the stall bug to become extremely frequent.

The changes cause the reader and writer thread to end up with inconsistent views of the shared IODataQueueMemory data structure. For example:

Reader thread sees head = 100, tail = 200. This means there are messages to be dequeued.
Writer thread wants to enqueue a new message, notes the initial state of the ring buffer is head = 100, tail = 200.
Reader dequeues a message, updates head to 200.
Writer has queued a new message, updates tail to 300.
Reader checks for more messages. head and tail are found to be equal at 200, the tail write of 300 has not propagated to this thread yet.
Writer checks whether the ring buffer was emptied during the enqueue operation. head is still 100, as the write has not yet propagated.

This can be avoided by enforcing strict ordering on the reads and writes, which used to be the case in OS X 10.11.6. I guess those OSAtomic functions are being deprecated so the implementation was switched to fancy new C11 atomics.
The 2 commits in this PR work around the problem on 10.13.6 by subclassing IOSharedDataQueue and overriding its enqueue() implementation with the version from 10.11.6, and by replacing the IODataQueueDequeue function from the system's IOKit.Framework with the version from 10.13.3.
I have reported this bug to Apple with Radar number 43093190. A public copy of this bug report is on OpenRadar.

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

Previous discussion and reports from manual testing are attached to the original PR - hope it's OK to link to that here, but I figured as auth is required it should be OK.
Remaining issue: Answer from CELA regarding inclusion of APSL code.

Just a note that I've rebased these patches on latest master, and that I thus far haven't had any official feedback on the bug filed with Apple, nor has there been a fix in a publicly released macOS version yet.

I've had a response from Apple on the Radar, indicating that there should be a fix for this in an upcoming OS release. I suspect that means we'll never see a fix in 10.13.x, so for supporting that version we should definitely include some kind of workaround. The user space side fix is, if I remember correctly, much more important than the kernel side, which only rarely causes issues under very high pressure. So if the problem is inclusion/distribution of APSL source code and binaries would be OK, we could ship the fix as a dylib which we look for at runtime and can maintain out of tree.

I've just tested with macOS 10.14.1 beta 2 (18B50c) and the bug does indeed appear to be fixed there. I've asked whether there are any plans to backport the fix to a 10.13.6 update, no idea how likely that is.
Incidentally, it looks like the severe regression (user space side) was introduced with 10.13.6 itself - source code for the full 10.13 series is now online, and the IODataQueue functions were changed to use C11 atomics in IOKitUser-1445.71.1.

Thanks for following up on this. From what I understand, Office will be moving to 10.14.1 pretty quickly after it is available, but in the meantime a temporary fix (the separate binary) would definitely be helpful. We could probably stick that binary in a nuget package that we put in our dev feed, unless you have a better way to distribute it.

We could probably stick that binary in a nuget package that we put in our dev feed, unless you have a better way to distribute it.

I don't really know much about nuget, but if our installer is already set up to download stuff from nuget, then that seems like a good way forward assuming it doesn't matter that there's not actually any .NET based component to it. A few things to clarify before I put this together:

I'm assuming the code for this workaround would live in a separate repository?
If I put together an Xcode based project that builds the dylib, can you take it from there to bundle it into a nuget package that our installer downloads?
There will be a small chunk of code required in the PrjFSLib native library that actually looks for and loads the workaround dylib on affected systems, but this won't be tainted license-wise in any way.

@pmj That plan sounds good to me. Let me know if you need any help in getting that put together, I can handle getting the dylib packaged from the separate repo and consumed in this repo.


Fix #45: Horizontal scrollbar is reversed and add a test case

A small fix for issue #45. Changes horizontal scrollbar movement direction.
Also I added a test to designer so you can view and try horizontal scrolling while modifying horizontal scrollbar

Don't know why GitHub (or Git with it) think that I moved half of lines to the right when it looks fine in my IDEs and notepad

Thank you for your contribution !


⚠ Bofrost.de has degraded performance

In 687dd7a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1473 ms

Resolved: Bofrost.de performance has improved in dd3e5de after 6 minutes.


How to pass dynamic params to Vue router

I need to be able to pass params along to Vue router when clicking a Toasted action button.  I can't seem to find any documentation on how to accomplish this.  Here's what I have:
    Vue.toasted.register(       "showSpecialAlert",       payload => {         // message         if (!payload.holdName) {           return "No hold selected";         }         return "Generating activity for " + payload.holdName;       }, {         // options         position: "top-center",         keepOnHover: true,         duration: 5000,         action: {           text: "View",           push: {             name: "showHold", // THIS DOESN'T WORK             holdId: payload.holdId // NEITHER DOES THIS             params: {               holdId: payload.holdId             } // NO ERROR IN CODE, BUT NOT DYNAMIC AND DOESN'T SHOW UP IN VUE DEV TOOLS             params: {               holdId: 2             } ////           }         }       }     );
Edit: code looks bad here; Replit link: https://repl.it/@BoClifton/Pass-Vue-Router-route-in-global-Toasted

same issue, options could maybe be a function like message?
we recive payload, extend it and return it back.

the push object is vue-router push object. it supports vue router params. take a look at how you can pass params : https://router.vuejs.org/guide/essentials/navigation.html

@shakee93
But how do you pass the params tho?
window.Vue.toasted.register(     'new_specialist_schedule',     p => {         let start = global.moment(p.start).format('DD/MM/YYYY HH:mm');         return Novo agendamento
De: ${p.user.name}
${start};     },     {         duration: 50000,         position: "top-right",         iconPack: 'fontawesome',         icon: 'address-book',         action: [             {                 text: 'Ver consulta',                 push: {name: 'specialist.schedules.schedule', params: {}}             },         ]     } );
window.Vue.toasted.global.new_specialist_schedule({                 user: n.user,                 start: n.start,                 end: n.end,                 schedule: n.schedule_id             });
I would need:
window.Vue.toasted.register(     'new_specialist_schedule',     p => {         let start = global.moment(p.start).format('DD/MM/YYYY HH:mm');         return Novo agendamento
De: ${p.user.name}
${start};     },     {         duration: 50000,         position: "top-right",         iconPack: 'fontawesome',         icon: 'address-book',         action: (payload) => [             {                 text: 'Ver consulta',                 push: {name: 'specialist.schedules.schedule', params: {schedule: payload.schedule}}             },         ]     } );

@MrTheSoulz Hey for this scenario you will need to use the normal toast warapped inside a function.
myToast(payload){
  Vue.toasted.show("my message", {
    // your options here
    action : payload.action
  })
}

// and to call
myToast({
   // your payload
})


Invalid arguments to shell/command/raw cause exception.

You can trigger the error with the current version from git (f34d094) and this playbook:
---
- hosts: all
  gather_facts: no
  tasks:
    - shell: PATH=$PATH:/usr/local/bin/pip

My solution:
diff --git a/lib/ansiblelint/rules/CommandsInsteadOfArgumentsRule.py b/lib/ansiblelint/rules/CommandsInsteadOfArgumentsRule.py
index 2161729..9161dc6 100644
--- a/lib/ansiblelint/rules/CommandsInsteadOfArgumentsRule.py
+++ b/lib/ansiblelint/rules/CommandsInsteadOfArgumentsRule.py
@@ -40,7 +40,7 @@ class CommandsInsteadOfArgumentsRule(AnsibleLintRule):
                   'rmdir': 'state=absent', 'rm': 'state=absent'}

     def matchtask(self, file, task):
-        if task["action"]["module"] in self._commands:
+        if task["action"]["module"] in self._commands and task["action"]["module_arguments"]:
             executable = os.path.basename(task["action"]["module_arguments"][0])
             if executable in self._arguments and \
                     boolean(task['action'].get('warn', True)) and \
diff --git a/lib/ansiblelint/rules/CommandsInsteadOfModulesRule.py b/lib/ansiblelint/rules/CommandsInsteadOfModulesRule.py
index 30b0fa5..671249b 100644
--- a/lib/ansiblelint/rules/CommandsInsteadOfModulesRule.py
+++ b/lib/ansiblelint/rules/CommandsInsteadOfModulesRule.py
@@ -41,7 +41,7 @@ class CommandsInsteadOfModulesRule(AnsibleLintRule):
                 'unzip': 'unarchive', 'tar': 'unarchive'}

     def matchtask(self, file, task):
-        if task["action"]["module"] in self._commands:
+        if task["action"]["module"] in self._commands and task["action"]["module_arguments"]:
             executable = os.path.basename(task["action"]["module_arguments"][0])
             if executable in self._modules and \
                     boolean(task['action'].get('warn', True)) and \

Might be worth raising a pull request for this - while I'm happy to take the patch, I'd rather take patches through pull requests than through issues. If you have any concerns about that, let me know.

This change just hides the fact that the task doesn't actually do anything - setting environment should be done through the environment key. ansible-lint shouldn't stack trace though.

A pull request has been made for this issue #144.
The PR aims to fix the bug, not to report the NOOP to the user. I think a new rule will be needed to handle the specifics.

Fixed with #144


Could not find com.ucasoft.kcron:kcron-common-macosarm64:0.10.2

I just discovered this library and think it would could be useful for me. Unfortunately, it does not resolve for native macosarm64:
Caused by: org.gradle.internal.resolve.ModuleVersionNotFoundException: Could not find com.ucasoft.kcron:kcron-common-macosarm64:0.10.2.

It resolves fine for js, android and jvm. Only problem is with native.
I never had this issue with another kotlin library, so I think there might be something slightly unusual about how multiplatform artifacts are being published.
I dug a little deeper and noticed something interesting. If we look at kotlinx-io, which is a multiplatform module that works for me on all targets, and we look at its .module metadata file (example here) we see that for "macosArm64ApiElements-published", there is this field:
"artifactType": "org.jetbrains.kotlin.klib"

And then if we look at the metadata from this library (here) we see that this field does not exist. I do not know if this is the issue, it could be unrelated. But I hope this helps.

Hello @mgroth0!
Thx to highlight the problem.
I think the problem is here.
It should be like these.
I will fix it soon.


Implement ChangeCookieAction and add factory to parse cookie flag from mod_rewrite

Resolves #94
cc @mikaelm12 @Tratcher

😄 The branch name

🆙 📅


feat: add LLM 'stream' flag (for litelllm)

This PR adds the flag stream (bool, default None) to the LLM configuration.
Streaming is for LLM responses to be returned in a stream, like in chunks, instead of waiting for a full reply (with delay).
This was partially done with the help of OpenDevin itself. 😀

Hello @enyst
I'll put my change to testing later on, but was wondering if from the top of your head you'd have ideas whether "chunked" responses from an LLM could already work at all or would require other architectural changes to keep it all together?

@tobitege could you elaborate a little bit on how this will effect how OpenDevin works? Typically OpenDevin receives a full response from the LM, packages it up as an AgentMessage or Action, and then sends it to the environment. I can't tell if adding streaming would make any difference in OpenDevin's behavior, and if not maybe we don't need the configuration option.

@tobitege could you elaborate a little bit on how this will affect how OpenDevin works? Typically OpenDevin receives a full response from the LM, packages it up as an AgentMessage or Action, and then sends it to the environment. I can't tell if adding streaming would make any difference in OpenDevin's behavior, and if not maybe we don't need the configuration option.

I know streaming from other (local+web) interfaces, where you actually don't have to wait for the full reply before it appears.
You get a way more granular response chain, which makes it feel more "interactive" to see the words/sentences come in.
Many interfaces than even have that Stop button so if you see the reply going the wrong way, hit it and save tokens.
Right now I've only seen that it is an option. But the question is indeed how this would work with the messaging we have right now.

I think it won't work with the messaging we have now, so my suggestion would be that we close this for the moment until we have an idea what the UX would be.

For later maybe.


Initial Commit

Updated with latest set of python REST APIs and Python Client Library Scripts

Hi Robert,
I’ll fix it and update it accordingly..
Thanks and regards,
Mahalakshmi G
Automation and Tools SME
https://devnet.netapp.com/restapi
From: Robert Blackhart<EMAIL_ADDRESS>Sent: 24 June 2020 05:22 PM
To: NetApp/ontap-rest-python<EMAIL_ADDRESS>Cc: G, Mahalakshmi<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [NetApp/ontap-rest-python] Initial Commit (#22)
NetApp Security WARNING: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@RobertBlackhart commented on this pull request.
In addition to my below comments, please fix all of the ReviewDog comments regarding the lint errors as well.

In examples/python_client_library/README.mdhttps://github.com/NetApp/ontap-rest-python/pull/22#discussion_r444829989:

python 3.5 or later

requests 2.21.0 or later

marshmallow 3.2.1 or later

-See the PyPI netapp-ontap package web page for more information.
+See the [PyPI netapp - ontap](https: // pypi.org / project / netapp - ontap /) package web page for more information.
Why are we adding all of these spaces? Isn't this going to break the URL?

In examples/python_client_library/account_operations.pyhttps://github.com/NetApp/ontap-rest-python/pull/22#discussion_r444830823:

Role,

RolePrivilege

+)
+from utils import Argument, step, substep, parse_args, setup_connection, setup_logging

+def show_account() -> None:

"""Lists Accounts"""

substep("List Accounts")

print("======================")

try:

   for accounts in Account.get_collection():

       print("=====")

       print("Account Name = %s" % accounts.name)

       print("Account  Owner Name = %s" % accounts.owner.name)

Remove double space between "Account" and "Owner"

In examples/python_client_library/account_operations.pyhttps://github.com/NetApp/ontap-rest-python/pull/22#discussion_r444831205:

+from netapp_ontap.resources import (

Account,

AccountApplication,

Role,

RolePrivilege

+)
+from utils import Argument, step, substep, parse_args, setup_connection, setup_logging

+def show_account() -> None:

"""Lists Accounts"""

substep("List Accounts")

print("======================")

try:

   for accounts in Account.get_collection():

This is just a single object so it should be called account, not accounts

In examples/python_client_library/account_operations.pyhttps://github.com/NetApp/ontap-rest-python/pull/22#discussion_r444832097:

accpwd = input(

   "Enter the password of the Account:- ")

Don't use the input() function to collect sensitive data, like a password.  This is then only 2 API calls under the hood, which will be more efficient.

In examples/python_client_library/account_operations.pyhttps://github.com/NetApp/ontap-rest-python/pull/22#discussion_r444833663:

+def update_account():

"""Call the /volumes API and list the current volumes. We should be allowed

to do this. Then modify our role to deny the access and call it again. This

time we should get an error.

"""

Docstring needs to be updated to be relevant

In examples/python_client_library/account_operations.pyhttps://github.com/NetApp/ontap-rest-python/pull/22#discussion_r444834053:

accname = input(

   "Enter the name of the Account that needs to be updated with the new role:- ")

Seems like the wrong message
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NetApp/ontap-rest-python/pull/22#pullrequestreview-436562066, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOTSD767OASSF3NCGME5WADRYHSG3ANCNFSM4OGQO6XA.

Updated

Updated New Scripts


Pageviews update

Checklist

[x] I have read the newlest tutorials and know the correct effect of the functional design.
[x] There is no similar question on existing issues (including closed ones).
[x] I have tried to find the answer on Jekyll Forum and StackOverflow.
[x] My question is based on the latest code of the master branch.

Description

Could you update your guide about enabling pageviews? I stuck on making query due to google analytics update. Now I can not make query via Query Explorer.

@challenger128 I also encountered the same issue, and found a fix, it's mentioned here: https://vansoest.dev/posts/creating-a-dev-blog/#cdn-and-analytics--google-cloud.

@larsvansoest Hmm, I've tried to create ga4 & ua and still getting errors on query exlporer

Oh, sorry, @larsvansoest, some of my browser's extensions blocked UA query explorer. Your idea to solve is great. I guess you should pull request updates on this guide.


Added exercise route fetching and fixed time zones parsing

@quentinleguennec, your branch seem to be the most advanced of those trying to unblock https://github.com/imdzx/flutter_health_connect, hence we propose our changes to you first.

Thanks for the PR @AliakseiT, I'll try to review it this week.
What was wrong with the time zone parsing exactly?

Thanks for the PR @AliakseiT, I'll try to review it this week. What was wrong with the time zone parsing exactly?

Previous code was not able to parse values like "+02:00" using the map['endZoneOffset'] as int construct. I believe HC used to send a different value in the map (or it is a vendor-specific field).

@nikolaydymura @AliakseiT Thanks again for the PR!
I added a few comments, most of it is related to some changes I added on top of the original code.
I didn't try to run this PR code yet, but I will after the few points above are fixed.

@nikolaydymura @AliakseiT Thanks again for the PR! I added a few comments, most of it is related to some changes I added on top of the original code. I didn't try to run this PR code yet, but I will after the few points above are fixed.

Thanks! We, however, don't see the comments yet. I guess, once you press "request changes" they all get published.

@nikolaydymura Thank you for the fixes, let me know when you are happy with the PR and I'll test it on my phone before merging it.

@nikolaydymura Thank you for the fixes, let me know when you are happy with the PR and I'll test it on my phone before merging it.

Nikolay is travelling at the moment, let me step in: the PR is ready for the device testing.
BTW, if you can tell what sources of HC data you experiment with, it helps us to reproduce the findings (if any).

@AliakseiT @nikolaydymura I've finally found the time to test the code, and while doing so found that I needed to improve the API a bit to test your work properly.
This is now done and everything seems to be working flawlessly :)
I pushed some commits to this PR to merge the changes I did and add the new function to the Changelog.md.
Feel free to review it and press the merge button if you are happy with the changes.

Thanks a lot! Looks good. Merge is not available to me (and lets keep it this way :))

Thanks again for the PR! :)


Empty android device name shown in other devices.

In Xamarin (Android) sample, after authentication, the android device will appear in devices list in Windows devices with same MSA account (when querying Windows Rome APIs). But it has an empty DisplayName.
I'm testing sample on a Nexus 4 running Android 5.1.1.
Is that the expected behavior?

Android devices are not supported as devices to connect to, and so should be filtered out by the app. This can be done by filtering out RemoteSystemKinds.Unknown.
Future Windows releases will not return Android devices until Android has the capability to be connected to.

Thanks.


import and export with LIFT of FLEX(Field Work Explorer)

@todallman Is Stephen still working on that project about import and export for LIF format. Or else should we connect with FLEX developer, see what they are interested.

I don't know of anyone working on this project.  It would certainly be helpful.
  From: Jeremiah Chung<EMAIL_ADDRESS>
To: JChungYS/TBTA<EMAIL_ADDRESS>Cc: Tod Allman<EMAIL_ADDRESS>Sent: Thursday, January 28, 2016 1:13 PM
Subject: [TBTA] import and export with LIFT of FLEX(Field Work Explorer) (#6)
@todallman Is Stephen still working on that project about import and export for LIF format. Or else should we connect with FLEX developer, see what they are interested. —
Reply to this email directly or view it on GitHub.

Hi Stephen,
Greeting. I just wonder whether you still thinking to work on this. Can
we start a github project about that? BTW, if LIFT is too complicated,
how about just import and export SFM of MDF.
Thanks,
Jeremiah
On 1/31/2016 7:57 PM, Tod Allman wrote:

I don't know of anyone working on this project.  It would certainly be
helpful.
From: Jeremiah Chung<EMAIL_ADDRESS>To: JChungYS/TBTA<EMAIL_ADDRESS>Cc: Tod Allman<EMAIL_ADDRESS>Sent: Thursday, January 28, 2016 1:13 PM
Subject: [TBTA] import and export with LIFT of FLEX(Field Work
Explorer) (#6)
@todallman Is Stephen still working on that project about import and
export for LIF format. Or else should we connect with FLEX developer,
see what they are interested. —
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub
https://github.com/JChungYS/TBTA/issues/6#issuecomment-177479775.


[GPU] Compilation problems with MSVC

LightGBM with GPU support does not compile with MSVC (VS 14 2015). There are problems:

OpenCL kernels are included as string literals, but MSVC does not support string literals longer than 16380 characters. So it generates C2026 error.
To solve this issue consider using xxd linux utility. It can translate text to unsigned char array. For example, xxd -i histogram16.cl produces this code:
unsigned char histogram16_cl[] = { 0x23, 0x69, 0x66, 0x6e, 0x64, 0x65, 0x66, 0x20, 0x5f, 0x48, 0x49, 0x53, 0x54, 0x4f, 0x47, 0x52, 0x41, 0x4d, 0x5f, 0x31, 0x36, 0x5f, 0x4b, 0x45,... }; unsigned int histogram16_cl_len = 41560;
So just run xxd -i histogram16.cl > xxd -i histogram16.cl.h, include histogram16.cl.h in implementation file (gpu_tree_learner.cpp) and use it like this:

//    kernel_source_ = kernel16_src_;
kernel_source_ = std::string(std::begin(histogram16_cl),  std::end(histogram16_cl));

Somewhere in the code of  gpu_tree_learner.cpp we can see dense_ind[dword_features_];, where dword_features_ non-constant data member. This is variable-length array which is not supported by C++11 standard (and MSVC too), but GCC (g++) extension allows this.
This problem can be solved simply by replacing variable with constant.
In the gpu_tree_learner.h used next structure:

  /*! \brief 4-byte feature tuple used by GPU kernels */
  struct Feature4 {
	  union {
		  unsigned char s[4];
		  struct {
			  unsigned char s0;
			  unsigned char s1;
			  unsigned char s2;
			  unsigned char s3;
		  };
	  };
  };

This is anonymous struct and also is not allowed by C++11 standard, but allowed by g++ and msvc as extension.
I don't understand why union is needed here, but if so better to name structure, for example:
  /*! \brief 4-byte feature tuple used by GPU kernels */
  struct Feature4 {
	  union {
		  unsigned char s[4];
		  struct s4{
			  unsigned char s0;
			  unsigned char s1;
			  unsigned char s2;
			  unsigned char s3;
		  } ss;
	  };
  };

There are some other small issues generating warnings like "conversion from 'uint32_t' to 'unsigned char', possible loss of data". Maybe it makes sense to use more pedantic compiler flags to resolve them.

Boost libraries linking problem. In boost by default enabled auto-linking for compilers that support this feature. With filesystem and system boost tries to autolink also chrono library which cannot be found - MSVC produces link error.
Solution is to add definition -DBOOST_ALL_NO_LIB. Something like that for cmake:

if (WIN32)
  # disable autolinking in boost
  add_definitions( -DBOOST_ALL_NO_LIB )
endif()

ping @huanzhang12

@tt83 Thanks so much for testing the GPU code on MSVC! I was never able to test the code on MSVC and your feedback is very valuable.
2, 3 and 4 can be easily fixed. I have create a PR to fix these glitches. Could you please test the PR #443 and see if it eliminates all the warnings in MSVC?
The reason I put the GPU kernels as string literals is to avoid xxd. I used to use xxd for converting OpenCL code to a hex string when I use C98. But in C++11 I really want to avoid this ugly part. Not all platforms have the xxd utility, so converting the OpenCL kernels in compile time may add additional difficulty for building. We can certainly put converted OpenCL files directly into the repository, but it makes the OpenCL code unreadable, and creates additional maintenance burden as they have to be manually synchronized with the latest OpenCL code.
We probably need a different flow for adding these OpenCL kernels in MSVC, to workaround the string literal size limit. We can probably adding them as resource files? Can any MSVC expert @guolinke @chivee comment on this here?

Fixed in PR #443. This issue can be closed now ;)


[Removal]: My hotspot was added incorrectly

Hotspot Name
Sharp Heather Elk
Hotspot b58 Address
11HdGM4j1pmuBeZvChYjqpsc4u7F5Bf7ZmTqAM52cLXeFxMudgh
Discord Handle
No response
Hotspot Manufacturer
Nebra
Removal Reason
This hotspot is legitimate and incorrectly included in the deny list. It is located with my other fleet of miners with my friends and family in Klotz, Germany. The hotspot was incorrectly added to the deny list.
Modifications
Yes
Extra forwarders
No
Extra antennas
No
Additional Information
only replaced the original indoor antenna with an outdoor one

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.

I'm sorry this is not acceptable , my hotspot is legitimate I didn't do any hacks, what evidence do you have to shut down my investment this way, at less tell me what I'm doing wrong so i can correct it,,
Please remove my hotspots from this list, i have been suffering from all kinds of bugs in your software for 3 months and now you shut my miners off without a reason..
this is not professional and not acceptable
Please remove my miners from this list. otherwise, provide instructions on what should i do from my end


rados: free object list results after listing

This fixes a memory leak that becomes substantial across large pools.

Checklist

[ ] Added tests for features and functional changes
[ ] Public functions and types are documented
[x] Standard formatting is applied to Go code
[ ] Is this a new API? Added a new file that begins with //go:build ceph_preview
[ ] Ran make api-update to record new APIs

New or infrequent contributors may want to review the go-ceph Developer's Guide including the section on how we track API Status and the API Stability Plan.
The go-ceph project uses mergify. View the mergify command guide for information on how to interact with mergify. Add a comment with @Mergifyio rebase to rebase your PR when github indicates that the PR is out of date with the base branch.

Confirmed this is working as intended in a production tool, but definitely open to suggestions re testing or better cgo memory management practices.

Proactively catching leaks is not something the test suite does well today. Especially since the cluster we bring up is kept intentionally as small as we can for speed. Thoughts and suggestions are welcome.
For now we typically try to catch something like this in review, by noticing an API call had a matching cleanup API... but this one slipped by all of us.


[BUG]

Delta Error Log 13-07-2024 22:07:07
File Name: C:\Users\sushi\AppData\Roaming\delta_core\error_logs\ERROR_LOG_59827037.4691.txt
HResult: -2147467259
Exception Data: System.Collections.ListDictionaryInternal
Causing Func: GetModuleInfos
Target Site: System.Diagnostics.ModuleInfo[] GetModuleInfos(Int32, Boolean)
Outer Exception Message: A 32 bit processes cannot access modules of a 64 bit process.
Outer Exception Source: System
Exception as String: System.ComponentModel.Win32Exception (0x80004005): A 32 bit processes cannot access modules of a 64 bit process.
at System.Diagnostics.NtProcessManager.GetModuleInfos(Int32 processId, Boolean firstModuleOnly)
at System.Diagnostics.NtProcessManager.GetFirstModuleInfo(Int32 processId)
at System.Diagnostics.Process.get_MainModule()
at Delta.XWindow.Inject()
at Delta.XWindow.Button_Click_7(Object sender, RoutedEventArgs e)

Pls tell me how to fix it

Can you please add your OS version, the Delta Lake version, and the code you're running?
This error message seems useful: "A 32 bit processes cannot access modules of a 64 bit process"
Does this StackOverflow response help?

Closing this due to inactivity (and not sure this is even a Delta Lake bug?)


stability issues with project unit tests when multiple mockServer are executed

Describe the bug
The outcome of npm run test seems to vary due to some stability issues with the test services that are spin-up for the tests. See also some of the automated PR that are failing:
https://github.com/verdaccio/verdaccio/pulls
To Reproduce
execute npm run test multiple times on a 'standard' machine or CI
Background
The issue is not present if the unit tests are executed sequentially by adding --runInBand, further review is that the issues are caused by timeouts of the "beforeAll" methods in the API test where mockServer promise is not resolved.
https://github.com/verdaccio/verdaccio/blob/master/test/unit/modules/api/token.spec.ts#L64
https://github.com/verdaccio/verdaccio/blob/master/test/unit/modules/api/api.spec.ts#L49
This is specifically true if they are spin-up in parallel for the different API tests, which also explains why in some cases they succeed and in some cases they fail and why the sequential test do work.
it would be recommended to review the logic in:
https://github.com/verdaccio/verdaccio/blob/master/test/unit/__helper/mock.ts
or re-use the same mock service in between the different API test to prevent multiple processes need to be spin up in parallel.

I've been working heavily on this topic the last months, I'd recommend you check the branch 5.x https://github.com/verdaccio/verdaccio/tree/5.x/packages
your feedback would be great, I've separated test un different packages, which will reduce race conditions and scope.

This is specifically true if they are spin-up in parallel for the different API tests, which also explains why in some cases they succeed and in some cases, they fail and why the sequential test do work.

I don't see this issue reflected on CI, I'm not saying does not happen, but would be great to know which ones are flaky.


[Question] How do you add new repository or organization to get notifications?

Hi. Thanks for making such a great app.
I have a question about adding organization.
Recently I joined new organization and I cannot find how to get notifications from this organization.
I tried the way in FAQ here (https://www.gitify.io/faq),
but I could not find 'Gitify' tab in Settings page.
I guess this is related to access problem, but I cannot find how to allow access for new organization
that I joined after setting up all the other repository's access at the first time.
Hope anyone knows how to solve it.... thanks!

@broccolism my personal recommendation would be to create and use a Personal Access Token/ You can then authorize a given token for specific organizations as needed.
For example, I have a token created for Gitify that I've also authorized to access my notifications in Electron:


Don't create implicit function definitions during record parsing

This prevents the parser from getting confused about the number of fields if a parser error triggers parseOrNextDecl to skip to the next decl.
Fixes #639

Just found another similar crash even with this fix applied, so perhaps a more robust fix is needed:
struct S {
  __attribute__((__aligned__(char:  1))) long a;
  __attribute__((packed)) b;
};

void foo(void) {
}


Uncaught Error: ENOENT: no such file or directory, lstat '~/AppData\Local\atom\app-1.35.1\ckedito...

[Enter steps to reproduce:]

...
...

Atom: 1.35.1 x64
Electron: 2.0.18
OS: Unknown Windows version
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT: no such file or directory, lstat 'C:\Users\Daniele\AppData\Local\atom\app-1.35.1\ckeditor-docs-samples-master'
At fs.js:1661

Error: ENOENT: no such file or directory, lstat 'C:\Users\Daniele\AppData\Local\atom\app-1.35.1\ckeditor-docs-samples-master'
    at Proxy.realpathSync (fs.js:1661:15)
    at Proxy.fs.realpathSync (ELECTRON_ASAR.js:336:29)
    at TreeView.t.exports.TreeView.moveEntry (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:14:912504)
    at TreeView.t.exports.TreeView.onDrop (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:14:918388)
    at HTMLDivElement.<anonymous> (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:14:893098)

Commands
     -0:27.2.0 tree-view:move (span.name.icon.icon-file-directory)
     -0:24.8.0 core:confirm (input.hidden-input)
  3x -0:20.2.0 core:undo (div.tool-panel.tree-view)

Non-Core Packages
atom-beautify 0.33.4
atom-material-syntax 1.0.8
atom-material-ui 2.1.3
color-picker 2.3.0
framer-ui 2.0.0
language-blade 0.41.4
minimap 4.29.9
seti-ui 1.11.0

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/tree-view/issues/1282 so you can subscribe there for updates if you'd like.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Update all dev-dependencies & don't use jsdom as test environment

While working on upgrading this module to use PostCSS 7 I encountered jsdom/jsdom#2304. To workaround this I simply disabled jsdom for tests since it's not necessary for this module.
Also updates all dev dependencies, including Typescript to version 3.

Thank you!


bower: add bower.json

Step 2 of #139
Note that this only exposes index.js since the other JavaScript files depend on fantasy-combinators which is not in the Bower registry.

I'd be happy to put fantasy-combinators on to bower if we want to make that a requirement!?

I'd be happy to put fantasy-combinators on to bower if we want to make that a requirement!?

Sure. If you perform the three steps listed in #139 for fantasy-combinators, we can then revisit this pull request.

I'll release this without fantasy-combinators for now, otherwise we'll be here for a while!


Create 'pay' page

Create 'pay' page:
Example: https://paybutton.org/pay/[bch_address]?amount=15?amount_type=USD

Although not a pre-req, it would be nice to have the server components setup to guarantee payment detection reliability as well as allow businesses to login and see/manage their payments.

We could also support an account ID, where we would then have the payment page show exactly the right themeing for that individual/company.

We could also support an account ID, where we would then have the payment page show exactly the right themeing for that individual/company.

Superseded by https://github.com/PayButton/paybutton-server/issues/136 on the server.