Neutrino not working with Tor enabled

Background
When trying to run LND with Neutrino as backend and Tor enabled (lnd --noseedbackup --bitcoin.active --bitcoin.testnet --bitcoin.node=neutrino --tor.active) it does not find peers and is just stuck at "block_height": 0.
The only way I got it working was with LND connecting to an already running Tor instance (tor.socks) and adding Neutrino peers manually with neutrino.addpeer.
Your environment

version of lnd: reproduced the bug on 0.9.0-beta and 0.8.2-beta
which operating system (uname -a on *Nix): Linux pc 5.3.0-29-generic #31-Ubuntu SMP Fri Jan 17 17:27:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
version of btcd, bitcoind, or other backend: I did no changes to Neutrino; so the version bundled with the LND releases

Steps to reproduce
Try running LND with Neutrino as backend with any Tor configuration to reproduce this bug.
Expected behaviour
LND should sync its Neutrino client out of the box with and without Tor enabled.
Actual behaviour
LND does not sync Neutrino on Tor unless LND is connecting to an already running Tor instance and Neutrino peers are specified manually.

This seems related to the DNS seeds not resolving, I also see this even without Tor at the moment.
2020-02-11 11:06:11.162 [INF] BTCN: DNS discovery failed on seed testnet-seed.bitcoin.schildbach.de: tor host is unreachable
2020-02-11 11:06:11.266 [INF] BTCN: DNS discovery failed on seed x49.testnet-seed.bitcoin.jonasschnelli.ch: tor general error
2020-02-11 11:06:11.266 [INF] BTCN: 1 addresses found from DNS seed testnet-seed.bluematt.me
2020-02-11 11:06:11.331 [INF] BTCN: DNS discovery failed on seed x49.seed.tbtc.petertodd.org: tor host is unreachable

Not sure if this is an issue on our end as it's pertaining to testnet infrastructure.

@Roasbeef the same behavior can be observed on mainnet:
michael@pc:~$ lnd --bitcoin.active --bitcoin.mainnet --tor.active --noseedbackup --bitcoin.node=neutrino
2020-02-21 19:53:37.957 [WRN] LTND: open /home/michael/.lnd/lnd.conf: no such file or directory
2020-02-21 19:53:37.957 [INF] LTND: Version: 0.9.0-beta commit=, build=production, logging=default
2020-02-21 19:53:37.957 [INF] LTND: Active chain: Bitcoin (network=mainnet)
2020-02-21 19:53:37.967 [INF] CHDB: Checking for schema update: latest_version=12, db_version=12
2020-02-21 19:53:37.968 [INF] RPCS: Generating TLS certificates...
2020-02-21 19:53:37.976 [INF] RPCS: Done generating TLS certificates
2020-02-21 19:53:37.992 [INF] BTCN: Loaded 0 addresses from file '/home/michael/.lnd/data/chain/bitcoin/mainnet/peers.json'
2020-02-21 19:53:37.993 [INF] BTCN: Registering block subscription: id=1
2020-02-21 19:53:37.994 [INF] BTCN: Broadcaster now active
2020-02-21 19:53:37.995 [INF] BTCN: DNS discovery failed on seed seed.bitnodes.io: dial tcp <IP_ADDRESS>:9050: connect: connection refused
2020-02-21 19:53:37.995 [INF] BTCN: DNS discovery failed on seed dnsseed.bitcoin.dashjr.org: dial tcp <IP_ADDRESS>:9050: connect: connection refused
2020-02-21 19:53:37.995 [INF] BTCN: DNS discovery failed on seed x49.seed.bitcoin.sipa.be: dial tcp <IP_ADDRESS>:9050: connect: connection refused
2020-02-21 19:53:37.995 [INF] BTCN: DNS discovery failed on seed x49.seed.bitcoin.jonasschnelli.ch: dial tcp <IP_ADDRESS>:9050: connect: connection refused
2020-02-21 19:53:37.995 [INF] BTCN: DNS discovery failed on seed x49.dnsseed.bluematt.me: dial tcp <IP_ADDRESS>:9050: connect: connection refused
2020-02-21 19:53:37.995 [INF] BTCN: DNS discovery failed on seed x49.seed.bitcoinstats.com: dial tcp <IP_ADDRESS>:9050: connect: connection refused

dial tcp <IP_ADDRESS>:9050: connect: connection refused means lnd cannot connect to your tor node. Do you have tor running locally and listening on port 9050?

@guggero yeah you are right. After starting the tor daemon, LND was able to find some peers:
2020-02-22 13:49:45.963 [INF] BTCN: 1 addresses found from DNS seed x49.seed.bitcoinstats.com
2020-02-22 13:49:46.088 [INF] BTCN: 1 addresses found from DNS seed dnsseed.bitcoin.dashjr.org
2020-02-22 13:49:46.088 [INF] BTCN: 1 addresses found from DNS seed seed.bitnodes.io
2020-02-22 13:49:46.088 [INF] BTCN: DNS discovery failed on seed x49.seed.bitcoin.sipa.be: tor host is unreachable
2020-02-22 13:49:46.088 [INF] BTCN: 1 addresses found from DNS seed x49.dnsseed.bluematt.me
2020-02-22 13:49:46.088 [INF] BTCN: DNS discovery failed on seed x49.seed.bitcoin.jonasschnelli.ch: tor host is unreachable


Begin adding a layman's-terms description of Endpoints' functionality

The existing "What Is It?" section did not fully explain to me how Endpoints was intended to be used, so I've taken a stab at articulating in a little more detail the purpose of the library and its relationship to your existing tools. I find these sorts of overviews to be personally useful, and would love input on how to improve the description so that it can help expand Endpoints' potential audience.

I'd love to expand or clarify the text here, I see it as a quick intro that would complement the longer-form tutorials and instructions you were discussing earlier this week

:+1: I think this is great!

this is a great start at working on the wording. do you want to keep this as a WIP, @kadamwhite? if not, i'll merge.

also, if you're done, if you could squash into a single commit with bullet points that'd be great :relaxed:


add Accept-Language header, so error messages can be returned in Polish

Headers feature done in v2.


The second parameter should not be named with a fixed literal string "dt"

Usually function parameters are dummy variables that can be named anything.
Forcing us to use dt just adds unnecessary confusion to users.
https://github.com/ManimCommunity/manim/blob/117563e1f01adafe675f011ad425a257753ec9f4/manim/mobject/mobject.py#L975

Or it enforces readable code. If it is a time based updater the parameters should be named accordingly.

It makes less readable because we need to open the source code to understand why using other names do not work.

It makes less readable because using other names do not work until we open the source code

Technically this is also mentioned in the documentation, albeit the wording is not necessarily strong enough to make it clear that it will not work with other parameter names.

a by-design principle

That's not what I would call it -- it is something that has been implemented by Grant like that at some point. I haven't asked him about his thoughts when doing so, but I'd guess that the motivation was being able to pass updater functions with multiple (keyword) arguments and have a distinct way of letting Manim infer whether it should be called every frame, or only when a dependent Mobject updates. I don't think that readability was part of the argument.
While I don't have a particularly strong opinion on the naming per se, I strongly dislike the current implementation where the arguments of the passed function have to be inspected. I would like to get rid of that -- the easiest way to do so would be to cleanly separate between time-based and other updaters, and implementing something along the lines of a Mobject.add_time_updater(func: Callable[[Mobject, float], None], ...) while deprecating the code that inspects the arguments being passed to Mobject.add_updater.
It is worth noting that while 3b1b/manim has introduced neat type hints for the two updater types (time-based and non-time-based), he still uses the dt-mechanism to decide which category a passed function belongs to.

TL;DR: No opinion on the naming enforcement per se, but I dislike the inspection-based implementation and would rather have split add_updater, add_time_updater methods. I am unsure, whether it is worth breaking all existing updater code for this though.

So, in the meanwhile should we update the documentation, just change the wording or add it as a gauche. If so, I would like to help.

So, in the meanwhile should we update the documentation, just change the wording or add it as a gauche. If so, I would like to help.

Yes!


Typo

Thank you, but this is not a typo. That is the correct wording: https://www.dictionary.com/browse/hiatus

Oh, ah! Thanks for teaching me new words! :smile: Sorry for the noise!


h3ToGeo function breaks on web

Hi everyone,
This code breaks on web:
final h3Factory = const H3Factory();
final h3 = h3Factory.load();
GeoCoord coord = h3.h3ToGeo(BigInt.parse("0x8c1f8ed825061ff"));

TypeError: Cannot read properties of undefined (reading 'h3ToGeo')
packages/h3_web/src/h3_web.dart 51:23                                h3ToGeo
Am I using it wrong?

my bad. the index.html was misconfigured.


os-web-proxy-useracl // Proxy service crash after add any group rule

Latest OPNsense 18.7.9-amd64
LDAP, SSO (over Kerberos os-web-proxy-sso plugin) configured
Users log in correctly and transparently.
If I add new any user rule to os-web-proxy-useracl plugin - all good
but if i add any group rule:

and try to access to any site - squid crash
Error:
support_sasl.cc(276): pid=50035 :2018/12/13 14:49:03| kerberos_ldap_group: ERROR: ldap_sasl_interactive_bind_s error: Can't contact LDAP server
support_ldap.cc(957): pid=50035 :2018/12/13 14:49:03| kerberos_ldap_group: ERROR: Error while binding to ldap server with SASL/GSSAPI: Can't contact LDAP server
...
  Wrote 0 entries.
2018/12/13 16:07:57 kid1|   Took 0.00 seconds (  0.00 entries/sec).
FATAL: The ext_group_ldap_0 helpers are crashing too rapidly, need help!
Squid Cache (Version 3.5.28): Terminated abnormally.
CPU Usage: 0.268 seconds = 0.208 user + 0.060 sys
Maximum Resident Size: 115200 KB
Page faults with physical i/o: 7
2018/12/13 16:07:57 kid1| Closing Pinger socket on FD 23
2018/12/13 16:08:00 kid1| Set Current Directory to /var/squid/cache
2018/12/13 16:08:00 kid1| Starting Squid Cache version 3.5.28 for amd64-portbld-freebsd11.1...

LDAP-connector must be properly configured and available. Squid helper can't connect to LDAP-server. You can read the documentation on SSO configuration https://www.smart-soft.ru/support/documentation/handbook/ting/proxy_auth_kerberos.html

Is there also an English version? :)

No, only Russian.

"System: Access: Tester" for LDAP-connector working ok ?

One controller is enough.

"System: Access: Tester" for LDAP-connector working ok ?

Yes, all ok

SSO tests:

You use domain on windows server 2003 ?

You use domain on windows server 2003 ?

No. We use Windows Server 2008R2+
Domain functional level and Forest functional level - Windows Server 2008R2
In opnsense SSO i use option "Windows 2008 with AES"

You use domain on windows server 2003 ?

No. We use Windows Server 2008R2+
Domain functional level and Forest functional level - Windows Server 2008R2
In opnsense SSO i use option "Windows 2008 with AES"

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


Update workflows

See https://github.com/senzing-garage/go-grpcing/issues/58

Pull requests:

https://github.com/senzing-garage/go-databasing/pull/101


Find Cube of a number

Description
Input  : 2
Output : 8

Program added successfully :tada:
Thanks for your contribution :hugs:


Clarifying that X-GitHub-Delivery is unique per event, not per delivery

Why:
This PR clarifies information about how X-GitHub-Delivery is used.

[!NOTE]
Check out the Additional Information section of this issue for more context.

Making this change helps users to learn how to correctly use this feature.
Closes #32822

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@CBID2 Hello! 👋 Changes look great - just one issue I noticed, left a comment for you above 💛

@CBID2 Thanks so much for opening a PR with these fixes! With that typo resolved, I'll update the branch and get this merged once tests are passing 🍏


Bot availablity status in pm

If you send PM to a bot it has always unavailable status. Even if it is online and replying

@luckyraul thanks for the report.  I think the right fix is we should never show a "status" in the compose box for bots.
@rishig FYI for confirmation.

I'm not sure how a bot could have gotten a status at all?

It doesn't.  This is the compose placeholder text reporting the bot doesn't have a status, I think?
@luckyraul can you post a screenshot?

@zulipbot claim

I guess this is an issue related to #12866 so the solution should be easy.
This is a good first issue, I think, and I can guide @vaibhavrajsingh2001 if he can work on this.


31da struct resolved

Hi,
can you help me with the following:
I found a structure (Originally it's visual basic) for the 31DA type: In this case it's from Itho Daalderop .
2021-05-23T10:19:11.077679 070  I --- 37:054619 --:------ 37:054619 31DA 029 00EF007FFFEFEF7FFF7FFF7FFF7FFFF800EF8301000000EFEF7FFF7FFF
Structure is:
data.data[0] > 200
data.data[0]
else
decimal num = new decimal((double)(int)data.data[0] / 2.0);
data_length > 0
data.data[1]
data_length > 1
int num2 = unchecked((int)data.data[2]) * 256 + unchecked((int)data.data[3]) //num2 <= 32766
data_length > 3
int num3 = data.data[4]; //num3 needs to be <=200
data_length > 4
int num4 = data.data[5]; //num4 needs to be <=200
data_length > 5
int num5 = unchecked((int)data.data[6]) * 256 + unchecked((int)data.data[7]) //num5 needs to be <= 32766
data_length > 7
int num6 = unchecked((int)data.data[8]) * 256 + unchecked((int)data.data[9]); //num6 needs to be <= 32766
data_length > 9
int num7 = unchecked((int)data.data[10]) * 256 + unchecked((int)data.data[11]); num6 needs to be <= 32766
format: (double)num7 / 100.0
data_length > 11
int num8 = unchecked((int)data.data[12]) * 256 + unchecked((int)data.data[13]); num8 needs to be <= 32766
format: (double)num8
data_length > 13
int val = unchecked((int)data.data[14]) * 256 + unchecked((int)data.data[15]);
data_length > 15
int num9 = data.data[16]
format: (double)num9 / 2.0
data_length > 16
int num10 = data.data[17];
format: num10 & 0x1F //don't know
data_length > 17
int num11 = data.data[18] //num11 need to be <=200
format: (double)num11 / 2.0
data_length > 18
int num12 = data.data[19]; need to be <=200
format: (double)num12 / 2.0
data_length > 19
int num13 = unchecked((int)data.data[20]) * 256 + unchecked((int)data.data[21]);
data_length > 21
int num14 = data.data[22]; needs to be <=200
format: (double)num14 / 2.0
data_length > 22
int num15 = data.data[23];
format: (double)num15 / 2.0
data_length > 23
int num16 = unchecked((int)data.data[24]) * 256 + unchecked((int)data.data[25]); //Need to be <=32766
format: (double)num16 / 100.0
data_length > 25
int num17 = unchecked((int)data.data[26]) * 256 + unchecked((int)data.data[27]); //Need to be <=32766
format: (double)num17 / 100.0
Can we add this to the parser?

Thanks for this. Is quite useful but the main issue is what do the fields mean?
Like, if a number needs to be less than 200 (0x00-0xC8, or 0xFE/FF), then it's almost certainly a percentage...
But of what? Fan Speed, Humidity, Other?
Do you have a link to this very exciting VB code?

I decompiled heir service tool so if there no problem with that (try ILspy) you can find it here:
https://www.ithodaalderop.nl/nl-NL/professional/servicetool
The database passwords are not hard to find

I am sorry - I will need more hints that that.

Some values es are references  to one or more databases and subtables. There are more struc references. If you could provide the code for this struct, I will lookup their meaning and add struct code for others, I also have the command stru t and settings structs

If you could provide the code for this struct

I am not 100% sure what you mean - this is what I have so far, incorporating your bit, above:
    assert msg.len == 29, f"expected length 29, not {msg.len}"  # usu: I CTL-->CTL

    assert payload[2:4] in ("00", "EF"), payload[2:4]
    assert payload[4:6] in ("00", "40"), payload[4:6]
    assert payload[6:10] in ("07D0", "7FFF"), payload[6:10]
    assert payload[10:12] == "EF" or int(payload[10:12], 16) <= 100, payload[10:12]
    assert payload[12:14] == "EF", payload[12:14]
    assert payload[14:18] == "7FFF", payload[14:18]
    assert payload[18:22] == "7FFF", payload[18:22]
    assert payload[22:26] == "7FFF", payload[22:26]
    assert payload[26:30] == "7FFF", payload[26:30]
    assert payload[30:34] in ("0002", "F000", "F800", "F808", "7FFF"), payload[30:34]
    assert payload[34:36] == "EF", payload[34:36]
    assert payload[36:38] == "EF" or int(payload[36:38], 16) <= 200, payload[36:38]
    assert payload[38:40] in ("EF", "FF") or int(payload[38:40], 16) <= 200, payload[38:40]
    assert payload[40:42] in ("00", "EF", "FF"), payload[40:42]
    # assert payload[42:46] == "0000", payload[42:46]
    assert payload[46:48] in ("00", "EF"), payload[46:48]
    assert payload[48:50] == "EF", payload[48:50]
    assert payload[50:54] == "7FFF" , payload[50:54]
    assert payload[54:58] == "7FFF", payload[54:58]

    return {
        "_unknown_00": _percent(payload[2:4]),
        "unknown_12c8": _percent(payload[4:6]),  # NOTE: 12C8/payload[4:6]
        "unknown_1298": _temp(payload[6:10]),  # NOTE: 1298/payload[2:6]
        "relative_humidity": _percent(payload[10:12], precision=1),
        "_unknown_04": _percent(payload[12:14]),
        "_unknown_05": _double(payload[14:18]),
        "_unknown_06": _double(payload[18:22]),
        "_unknown_07": _double(payload[22:26], factor=100),
        "_unknown_08": _double(payload[26:30]),
        "_unknown_0x": _double(payload[30:34]),
        "_unknown_09": _percent(payload[34:36]),
        "_unknown_10": payload[36:38],  # TODO: & 0x1F ?
        FanSwitch.FAN_RATE: _percent(payload[38:40]),  # NOTE: 31D9/payload[4:6]
        "_unknown_12": _percent(payload[40:42]),
        FanSwitch.BOOST_TIMER: _double(payload[42:46]),  # NOTE: 22F3/payload[2:6]
        "_unknown_14": _percent(payload[46:48]),
        "_unknown_15": _percent(payload[48:50]),
        "_unknown_16": _double(payload[50:54], factor=100),
        "_unknown_17": _double(payload[54:58], factor=100),
    }

You can see, I currently have only:

relative humidity, fan rate & boost timer
12c8, 1298 (whatever they are)

For most values I have ever seen either EF or 7FFF - the equivalent of N/A, None or null.
Note the indexes are different; mine = (yours * 2) + 2

Traceback (most recent call last):
  File "/home/brakero1/ramses_rf/ramses_rf/message.py", line 402, in is_valid
    self._payload = payload_parser(self.raw_payload, self)
  File "/home/brakero1/ramses_rf/ramses_rf/parsers.py", line 264, in wrapper
    result = func(*args, **kwargs)
  File "/home/brakero1/ramses_rf/ramses_rf/parsers.py", line 1629, in parser_31da
    "_unknown_00": _percent(payload[2:4]),
  File "/home/brakero1/ramses_rf/ramses_rf/parsers.py", line 322, in _percent
    assert int(value, 16) <= 200, "max value should be C8"
AssertionError: max value should be C8

Here are the values they represent:
AirQuality
AirQbase
CO2level
Indoorhumidity
Outdoorhumidity
Exhausttemperature
SupplyTemperature
IndoorTemperature
OutdoorTemperature
SpeedCap
BypassPos
FanInfo *
ExhFanSpeed
InFanSpeed
RemTime
PostHeat
PreHeat
InFlow
ExhFlow

fan info subtable:
0x00 | off
0X01 | speed   1
0X02 | speed   2
0X03 | speed   3
0X04 | speed   4
0X05 | speed   5
0X06 | speed   6
0X07 | speed   7
0X08 | speed   8
0X09 | speed   9
0X0A | speed   10
0X0B | speed   1 temporay override
0X0C | speed   2 temporay override
0X0D | speed   3 temporay override
0X0E | speed   4 temporay override
0X0F | speed   5 temporay override
0X10 | speed   6 temporay override
0X11 | speed   7 temporay override
0X12 | speed   8 temporay override
0X13 | speed   9 temporay override
0X14 | speed   10 temporay override
0X15 | away
0X16 | absolute   minimum
0X17 | absolute   maximum
0x18 | auto

Now trying to figure out what the 31D9 messages are that are sent after an 31DA.

I have pushed to master.
You could look for 22F1, 22F3, 3120, 31D9, 31E0, anything starting 12xx


Improve code highlighting.

Looks nicer! IMO

Hi thanks for the PR.
I guess that for the code colouring it would depends on one's personal taste.
This one is based on the base16 Material Theme which is a wide spread theme.
So I'll let it as is, I am happy that you could customize to your taste easily 😃


merge from template-sync/template-arlac77-github

package.json

BREAKING CHANGE: add >=22.9.0 remove >=22.7.0 (engines.node)
fix(engines): add >=22.9.0 remove >=22.7.0 (engines.node)

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

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

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
54

Relevant Lines:
56

💛  - Coveralls


tmux on build failure darwin

Issue description
Issue #16224 appears to have introduced a regression in darwin builds, this was detected by the automated build: https://hydra.nixos.org/build/36852692/log
Reverting commit 94627e658beabf2b4c766d31afa273a55879b46a causes a successful build.
Steps to reproduce
On OS X (currently running OS X 10.11.5), run nix-env -i tmux
Output of the failed nix-env -i tmux can be found in this gist: https://gist.github.com/sufianrhazi/470f0854f6f23610e197485c50822338
Technical details

System: OS X 10.11.5
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: 16.09pre87025.cb53771

still a problem with Nixpkgs version 16.09pre87038.039f0e5

change output order works for me:
outputs = [ "bin" "out" "man" ];

or change propagatedBuildOutputs like https://github.com/NixOS/nixpkgs/blob/3748cdc791b990ab6df6ee9f66f457f10cd5c949/pkgs/tools/system/awstats/default.nix#L18


StatefulBrowser not found

I get the following error AttributeError: 'module' object has no attribute 'StatefulBrowser' when I simply type browser = mechanicalsoup.StatefulBrowser()

You need the latest version (from Git, not from a release) for StatefulBrowser to work. See #59 .
Please re-open with more info if this doesn't solve your issue.


input metadata from GTDB-TK

Description of feature
We need a metadata that provides the metadata from gtdb-tk. this can be use for downstream analysis and tables

solved in #45


prevent locally wandering paths by making diagonals cost 1.4x more

Hi all,
I noticed that diagonals do not have a heuristic cost that is less than the taxicab distance, so I've added it in.
Previously, this resulted in paths that seemed to wander around the shortest path somewhat, but now it looks good.
The technique I used is to approximate the diagonal distance of a unit square as the relative heuristic.
However to keep the math using integers, I chose a square of side length 10 as the base cost, meaning that the diagonal would have cost 10*sqrt(2) = 14.14..., which I've approximated as 14. This seemed to work well so I haven't tried to see if there's any more optimization that can be done here.

Thanks for taking the time to open this. Could you please attach an image showing the behavior you mention? This code was originally written to extract contours from images (where the weights do not form a uniform cost grid) so adding arbitrary values may not be the best solution.

No worries, here's an example:
So if I run the maze example code as in the readme file, but enable diagonals, the top-left corner of the small maze looks like this:

With my pull request, it looks like this

I can see some non-shortest-path segments with the original implementation. Although the pull request isn't perfect, it looks like it does the best job possible with the 8-direction options. I came across this approach for a sub-heuristic somewhere but can't find the source, alas.
I haven't considered using this in non-black-and-white images, maybe it would have less of an effect there.
Here are the full images:
original

pull request

Thanks! Let me think about the best way to handle this. I'm wondering:

if this can be fixed easily by changing the order in which nodes are added to the priority queue, or
if the user should be able to specify an additional penalty cost for diagonal moves that defaults to zero (similar to your idea).

Any other suggestions?

on 1) - I originally wrote this fix a year ago, so I'm a bit rusty on how the priority queue works here, I'll have a play with this approach later, I haven't got thoughts on this yet
on 2) - the 10-or-14 thing is based on diagonal distance in a square grid being sqrt(2) of the side length, so I'm not sure that it's best left as a user adjustable penalty cost because it does have geometric significance
I think the pull request currently only works for black-and-white images, otherwise pixel values would swamp this 10-or-14 heuristic. I had a play around and haven't yet worked out how to fix it. Maybe this wandering path thing only happens for black-and-white maps?
I'll comment back if I work it out. Thanks for looking at this!


Warning on componentWillMount

There is an error in the terminal every time an avatar is rendered. It should be easy to solve since it is just asking to change the used method. This is the error:
Warning: componentWillMount has been renamed, and is not recommended for use. See https://reactjs.org/link/unsafe-component-lifecycles for details.

* Move code from componentWillMount to componentDidMount (preferred in most cases) or the constructor.

And the line it is referring to is this one:
https://github.com/fangpenlin/avataaars/blob/e16cf0202b6d5b239d05bb565859537bd719bc1a/src/index.tsx#L42

fixed already in master branch


(QENG-1627) added spec tests to flush out host and ssh_connection testin...

...g

:red_circle: Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker Combined Smoketest/386/

failures unrelated.

Considering that this passed spec, we're good here.


Add safety evaluation as a submodule

Added safety-eval repo as a submodule. Running the evaluation is straightforward. I've included a README under open-instruct/docs/safety-eval (safety.md) that details how to run the scripts both in an interactive session and on Beaker. The submodule supports evaluation on 7 different benchmarks.
Currently, the submodule points to a fork of safety-eval. If you’d like to make changes to the safety evaluation scripts, let me know, and I can add you to the fork.
Why a Submodule?

Generally, I prefer not to use submodules, but this is a temporary solution to quickly integrate safety evaluation.
Rewriting the code in the open-instruct evaluation logic would be time-consuming and may not be worthwhile, especially since Oyvind mentioned (on friday) that safety evaluation will be integrated into oe-eval within the next two weeks.

Are results added to the leaderboard?

I'm missing info on how to add them properly, currently the code is uploading scores into a table in HF repo but doesn't look nice, you can look here to make changes:

I'm going to close in favour of #286, which seems a bit cleaner from my perspective. This will do until we (hopefully) get the evals integrated into oe-eval.


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=381869450
https://github.com/Azure/azure-cli-extensions/commit/0dfc6120cf46e0e761c0cfb7ee91d11d170ac417

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update dependencies

Remove unnecessary ones
Update dependencies e.g. typescript

Obsolete with https://github.com/IndexCoop/tokenlists/pull/70


ColorVariable throws a false positive on variable declarations with certain names

In scss-lint 0.38.0, given the following variable declaration:
// Variations on $slate
$slate-60:      #7f90a0;

Scss-lint throws the following error:
app/assets/stylesheets/style-guide/base/_colors.scss:44 [W] ColorVariable: Color literals like '#7f90a0' should only be used in variable declarations; they should be referred to via variable everywhere else.
This error should never occur on variable declarations.
The error does not occur if the variable name is changed to the following, so it may have something to do with numbers in variable names:
// Variations on $slate
$slate-muted:      #7f90a0;

... but we also have $blue-60: rgba($blue, .6); and it passes, so this seems Very Weird.

Update: this may be a dupe of https://github.com/brigade/scss-lint/pull/612

This appears to be resolved by the current version 0.47.1.

Hold on, coworker and I are looking into this. Seems to have something to do with unicode characters for the spaces in between variable name and hexcolor.

Tracked it down. Using U+2002 (En Space) anywhere in the definition will cause this error to throw, regardless of whether it's a variable definition or not.
Copypastable:

After looking into this, this turns out to be a bug in the Sass parser.
If you include a non-standard space character on a line of Sass code as you have above, that entire line is parsed as a Sass::Script::Tree::ListLiteral instead of a Sass::Tree::VariableNode. This makes it impractical for SCSS-Lint to detect that we're in a variable declaration.
Please open this bug on the Sass project. Thanks!

Opened as https://github.com/sass/sass/issues/2126.


[CAT-803] add dockerfile to build docs

here's the generated docs for current code: https://drive.google.com/file/d/1PKoE0hqNXOdybx0_fHeuUl-PjESdMrnz/view?usp=sharing

Is this generating a markdown file for every function? It looks like that when I download the sample. Is there a way to get it to generate a file for a class?

I looked into dokka configuration and didn't find a straightforward way to turn off documentation for the functions and only have it for classes. Micah figured out that generating a file per function is pretty standard for kotlin docs (for example, the docs for okhttp3 are formatted that way).
I'm going to merge this PR for now. When we work on https://indicodata.atlassian.net/browse/CAT-805 , we can decide if it's worth merging the files.


[JOSS] Multi animal pose estimation tutorial

Hi,
I followed the multi animal pose estimation tutorial, very fun! But I got a bit confused about the following:
I set Pose (input) frame size (px) to match the constant detection sizes in the detector panel (both 400 px), but still one of the pose estimation crops (track_5) showed a very zoomed-in ant. Am I misunderstanding the constant detector size parameter? Is this expected? See screenshot below.

Below some additional suggestions for the tutorial:

Would it be possible to suggest a constant detection size for the sample data provided? I used 400 px which seemed to cover all animals in full.

Maybe it could be nice to add some tips on how to transform the bodyparts' coordinates from the cropped video space to the full video space using the exported data - I would expect most users would end up doing this. If this is an existing script maybe it can be linked here.

In the detector panel: are both "constant detection size" and "minimum detection size" only enabled if the constant detection sizes checkbox is selected? If so, could this be clarified? Maybe they could be greyed out if the checkbox is not true, or just a text clarification.

Thanks!

Hi @sfmig,
You are absolutely right - there are quite a few ROI parameters that seemingly do the same thing but all have slightly dirrerent purposes and some overwrite others. I'll sit down with this tomorrow and try to work out how to better document them and use the UI to better communicate how they effect in- and outputs.

Hi @sfmig!
I spent some time trying to recreate the described behaviour but I haven't managed to get the same zoomed in result for only one of the tracks. If there are any further details you could provide that would be much appreciated so I can fix whatever causes the scaling issue.
The way the multi-animal pose estimation works is that, by default, the bounding boxes extracted from the tracking step serve as the direct input to the pose estimation step, where for all frames relating to a track, track-by-track every cutout is passed to the DLC model and the estimated poses are overlayed.
The Pose (input) frame size (px) parameter controls the DLC network input and whatever the shape of the bounding boxes of the tracking step, those will be rescaled and padded if necessary to fit that resolution without strechting or squashing the frame.
If constant (input) ROI  (or constant (input) detection sizes as it was previously called) is enabled, the shape of the bounding boxes from the tracking step are overwritten and the Pose (input) frame size (px)  is used for all tracks.
As for the use of  constant detection sizes : if that is disbaled, the setting for its pixel value is now greyed out to avoid confusion (963f3b495293cd7c071fb4fcfcd57c45f0fb2531) . minimum detection sizes (px) is always consired and can be used to filter out detections that are too small and likely constitute background noise. Only detections above that threshold are retrieved, and, if enabled, rescaled to fit the constant size requirement. Thanks for your recommendation regarding greying out options - this behaviour should make the UI a little more intuitive now.
FInally, I have written a lot of additional analysis functionality here, but many elements there are either experimental or yet to be fully validated. Once that is the case, I will link to a script to combine the asynchrynous pose-estimates across videos. For now, have a look at this example, which additionally extracts body length of animals from pose for our default insect skeleton configuration.
Please let me know if this addresses your questions and suggestions! If you have any more details for me regarding the "zoomed in" behaviour, I'd love to give fixing that a try.
All the best
Fabi

Ah! Got it! There seems to be an issue with the direct use of the bounding boxes extracted during tracking. When constant (input) ROI is checked, this does not occur. I haven't found the source of the problem yet but will keep you posted once this is resolved.

Hi Fabi,
Thanks for the clarifications and for the edits in the UI, it looks great ✨
I just had another go and also had loads of troubles reproducing the issue... I actually wasn't able to pin down the "zoomed-in" behaviour reliably 👎  I got it once, but then couldn't reproduce it from scratch. But I think I caught a related issue.
When I got the zoomed-in behaviour, I realised the pose estimation clip for that track was actually flicking between two bounding boxes (not sure if that was also happening the first time when I reported the issue). This flickering between instances issue I was able to reproduce.
To reproduce it: I select the cfg and weights for the detector, select the input video, click on TRACK. If I then click on RESTART tracking after that, I get what seem to be mixed identities (see screenshot below). If I then run pose estimation, I get a clip for track_5 that flicks between two cropped instances. The clips for the other tracks seem ok though.
I think this may be misuse on my side, but I'm wondering if the track / restart function could be clarified. The way I understand it, TRACK means "track from the current frame" and RESTART means "track from Start frame" - is that correct? If so, I wonder if it would be helpful to rephrase the labels on the buttons. However I'm surprised that TRACK and RESTART don't have the same behaviour in the case described above though.
Let me know if you can reproduce it

The flicking clip and the corresponding pose data
multiple_ants_1920x1080_01_POSE_track_5.csv
https://github.com/FabianPlum/OmniTrax/assets/33267254/ad7193d3-16c3-4e1e-92d9-8457d91e0e36

Hi @sfmig!
After playing around some more, I was unable to recreate the behaviour and will close this issue for now. If this is flagged again in the future, I will refer to this issue and try to find a stable solution.
Thanks again for all your time and help in reviewing and improving this repo so much!
All the best
Fabi

yeah it was a tricky one! thanks for having a look and congrats, it's a very cool work 🌟


fix: transactions naming

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


Add some statistics

It would be nice to have more information about models as date, author, underlying platform (Tensorflow, Pytorch, etc.), number of layer, etc.

There’s going to 10 different flavors for each paper (caffe/Mxnet/etc) / better off having a pre-canned git search link for each paper.


Adding an initial Guide section to the online documentation

As a start to expanded documentation at tensorflow.org/swift, this adds a Guide section with a quick overview of Swift and Swift for TensorFlow. These sections were drawn from our current documentation in the front-page Readme here, as well as the Swift for TensorFlow design overview.
In addition to the overview documentation, the link to the Swift APIs has been updated to point to the correct repository, and a link to tensorflow/swift-models has been added.
Other documents will be added in subsequent pull requests.

@8bitmp3 - As Dan said, thanks for the comments. As a first step, I merely copied the descriptions we'd used elsewhere to create a starter document. The intent is that we'll modify and enhance this and add new guide sections once we verify that this gets formatted correctly over at tensorflow.org/swift.
When it comes to organization, I'd really like to have entry points for both people who know Swift but are new to ML and those who know ML but are new to Swift. I organized this with the latter in mind so that we explain why they should be excited about Swift first, then what we're doing with Swift for TensorFlow. I'm sure there are better ways of stating this, however.
I'm going to verify that everything is formatted correctly tomorrow, but after that point pull requests that add to or modify this wording would certainly be welcome. I'm going to follow on with individual guide pull requests as I have them ready.

@8bitmp3 - As Dan said, thanks for the comments. As a first step, I merely copied the descriptions we'd used elsewhere to create a starter document. The intent is that we'll modify and enhance this and add new guide sections once we verify that this gets formatted correctly over at tensorflow.org/swift.
When it comes to organization, I'd really like to have entry points for both people who know Swift but are new to ML and those who know ML but are new to Swift. I organized this with the latter in mind so that we explain why they should be excited about Swift first, then what we're doing with Swift for TensorFlow. I'm sure there are better ways of stating this, however.
I'm going to verify that everything is formatted correctly tomorrow, but after that point pull requests that add to or modify this wording would certainly be welcome. I'm going to follow on with individual guide pull requests as I have them ready.


4-Step UI (ELT + Schedule) to Single UI (Pipelines)

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1402
Originally created by @derek-knox on 2019-12-07 00:47:55

The idea to converge the 4-Step Extract, Load, Transform, and Schedule sub-UIs into a single UI that is the current Schedules UI has been shared a lot. I seemingly have not created an issue so here it is.
I recently made a video demonstrating the concept https://www.youtube.com/watch?v=Dr0_yUBWvWk
Below is a snapshot of the hacked together rough idea I intend for this issue. The difference being Loader and Transform would be additional columns in the top "Add" section. Then from these dropdowns a list (vs. our current card UI approach) of our existing connectors would be displayed. The connector "Install" or "Configure" would popup the same modal configuration we have currently.

Due to the video demo and below screenshot having a slightly different goal, the term "Datasets" was used where "Pipelines" would be the target term for this issue
The two sections (Add and Pipelines) in a single UI is the core goal

Pros:

Simplified UI
Improved mental model (user feedback currently is that when setting up each connector it feels like a global setting where in this solution you're creating data pipelines where the connector settings are for that pipeline). Technically speaking the configurations (Profiles) will be reusable if desired per pipeline.

Cons:

Existing users may dislike the change having gotten used to the current UI

TODOS

[x] in-lining the "Create Pipeline" modal on the Pipelines page
[x] removing the "Extract" and "Load" steps
[x] making the "Extractor" and "Loader" dropdowns smarter, by having selection automatically trigger installation and/or configuration
[x] only listing extractors in the dropdown that don't already have a pipeline
[-] adding an Edit pipeline button and modal to modify the extractor configuration

We can add this later if needed, but I think we don't actually want editing of an existing configuration that's tied to a pipeline. This is because subsequent runs (after a modified configuration) could pollute/blend data from multiple accounts (configurations) into the same analytics db. @mbergeron Can you confirm or deny?

[x] 'Pipeline' -> 'Dataset' & 'Analyze' to 'Report' (routes and labels)

View 25 previous comments from the original issue on GitLab


Create 2FA setup interface

implement a user-friendly interface for enabling and managing 2FA

implement a user-friendly interface for enabling and managing 2FA


Use multiple DID nodes in case one of them fails in order to be more resilient

This is not needed as of right now


Falcon 3.1.3 does not install on Win11 with latest Python v3.13.0

Description
The latest release of Python 3.13.0 for Windows 11 breaks the installation of Falcon 3.1.3
Steps to reproduce.

Download and install Python 3.13.0 from the Python Website. https://www.python.org/downloads/
Enter the command pip install falcon, and the following error message appears.

Expected behavior.
No error message on installation.
Falcon installs correctly on Python 3.12.7
Error message
PS C:\Users\David\Documents\alpaca-benro-polaris-main> python --version
Python 3.13.0
PS C:\Users\David\Documents\alpaca-benro-polaris-main> pip install falcon
Collecting falcon
Using cached falcon-3.1.3.tar.gz (577 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [73 lines of output]
Traceback (most recent call last):
File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\expand.py", line 69, in getattr
return next(
ast.literal_eval(value)
for target, value in self._find_assignments()
if isinstance(target, ast.Name) and target.id == attr
)
StopIteration
  The above exception was the direct cause of the following exception:

  Traceback (most recent call last):
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\expand.py", line 183, in read_attr
      return getattr(StaticModule(module_name, spec), attr_name)
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\expand.py", line 75, in __getattr__
      raise AttributeError(f"{self.name} has no attribute {attr}") from e
  AttributeError: falcon has no attribute __version__

  During handling of the above exception, another exception occurred:

  Traceback (most recent call last):
    File "C:\Users\David\AppData\Local\Programs\Python\Python313\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 353, in <module>
      main()
      ~~~~^^
    File "C:\Users\David\AppData\Local\Programs\Python\Python313\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 335, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
                               ~~~~^^^^^^^^^^^^^^^^^^^^^^^^
    File "C:\Users\David\AppData\Local\Programs\Python\Python313\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 118, in get_requires_for_build_wheel
      return hook(config_settings)
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\build_meta.py", line 332, in get_requires_for_build_wheel
      return self._get_build_requires(config_settings, requirements=[])
             ~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\build_meta.py", line 302, in _get_build_requires
      self.run_setup()
      ~~~~~~~~~~~~~~^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\build_meta.py", line 318, in run_setup
      exec(code, locals())
      ~~~~^^^^^^^^^^^^^^^^
    File "<string>", line 197, in <module>
    File "<string>", line 174, in run_setup
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\__init__.py", line 117, in setup
      return distutils.core.setup(**attrs)
             ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\_distutils\core.py", line 157, in setup
      dist.parse_config_files()
      ~~~~~~~~~~~~~~~~~~~~~~~^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\dist.py", line 643, in parse_config_files
      setupcfg.parse_configuration(
      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
          self, self.command_options, ignore_option_errors=ignore_option_errors
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      )
      ^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\setupcfg.py", line 193, in parse_configuration
      meta.parse()
      ~~~~~~~~~~^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\setupcfg.py", line 506, in parse
      section_parser_method(section_options)
      ~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\setupcfg.py", line 481, in parse_section
      self[name] = value
      ~~~~^^^^^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\setupcfg.py", line 299, in __setitem__
      parsed = self.parsers.get(option_name, lambda x: x)(value)
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\setupcfg.py", line 598, in _parse_version
      return expand.version(self._parse_attr(value, self.package_dir, self.root_dir))
                            ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\setupcfg.py", line 423, in _parse_attr
      return expand.read_attr(attr_desc, package_dir, root_dir)
             ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "C:\Users\David\AppData\Local\Temp\pip-build-env-_80t8vlv\overlay\Lib\site-packages\setuptools\config\expand.py", line 187, in read_attr
      return getattr(module, attr_name)
  AttributeError: module 'falcon' has no attribute '__version__'
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.

Hi @jdmorriso!
And thanks for reporting this. The limitation is known, Falcon 3.1.3 supports Python versions 3.5 through 3.12.
Our apologies we're a couple of days late with the upcoming 4.0.0 release that does support Python 3.13.
A release candidate is available: Falcon 4.0.0rc1.
Could you test the release candidate with your projects, and report back to us in case you run into any issues? Thanks!
You can see the list of changes here: https://falcon.readthedocs.io/en/latest/changes/4.0.0.html.

Hi,
As mentioned v4 will fix.
Duplicates #2066
Duplicates #2258
Duplicates #2376

Thanks @vytas7,
I've confirmed Falcon 4.0.0rc1 resolves the issue and passes all tests for the application. Sorry for the duplicate.
Dave

Thanks for testing the release candidate, and thank your for your patience @jdmorriso! You can now upgrade to a stable version, Falcon 4.0.0 is out!


pjsua2

Hi G-Farrow,
I am just wondering if you have any experience with pjsua2 in Python?

Hi, no I'm afraid I don't

how to download pjsua package

@vinodrock
How to Install PJsua for python is explained here  by malarinv:
1a. Install PJSIP (patch by: malarinv)
sudo apt install python3 python3-dev build-essential libasound2-dev
wget https://github.com/DiscordPhone/pjproject/archive/py37.zip
unzip py37.zip
cd pjproject-py37
chmod +x configure aconfigure
./configure CXXFLAGS=-fPIC CFLAGS=-fPIC LDFLAGS=-fPIC CPPFLAGS=-fPIC
make dep
make
sudo make install

1b. Install PJSUA Python bindings for PJSIP
cd pjproject-py37/pjsip-apps/src/python
python3 setup.py build
sudo python3 setup.py install

@vinodrock
How to Install PJsua for python is explained here  by malarinv:
1a. Install PJSIP (patch by: malarinv)
sudo apt install python3 python3-dev build-essential libasound2-dev
wget https://github.com/DiscordPhone/pjproject/archive/py37.zip
unzip py37.zip
cd pjproject-py37
chmod +x configure aconfigure
./configure CXXFLAGS=-fPIC CFLAGS=-fPIC LDFLAGS=-fPIC CPPFLAGS=-fPIC
make dep
make
sudo make install

1b. Install PJSUA Python bindings for PJSIP
cd pjproject-py37/pjsip-apps/src/python
python3 setup.py build
sudo python3 setup.py install


Pyet xarray

Update includes:

compatibility with xarray.DataArray, so pyet can be used with netcdf data,
improved testing (benchmarked to McMahon et al. 2013 and Schroedter 1985),
improved documentation,
added examples, ...

We should really not merge into the Master, but first into Dev. And from there, merge Dev into Master.

Merge first to dew.


Medium Error: If specified, publishedAt must be in the past. (2014)

I'm using WP 4.5.2 and facing this issue when posting my articles.

Medium Error: If specified, publishedAt must be in the past. (2014)

Is it a bug ?

Hello @koffisani: There is not a known issue at this time. Can I get the following information from you to help debug? May I ask you to attempt to cross-post again, and provide me with the following information?
Medium Username, DateTime that you tried to cross-post (including the time zone), the TimeZone your WordPress installation is set to.
From there, I can inspect the logs, and learn more information about this. Thanks!

Hello @huckphin, sorry for my silence. See bellow the informations you requested :
Medium username : @koffisani
DateTime I tried to cross-post : 2016-07-26 19 : 14 GMT+1
DateTime of my WP : 2016-07-26 20 :14 GMT+2 [Paris/France]
Screen shot
Thanks.

No worries. Thank you for providing that information! That helped me track down the actual issue. For the actual details, the post that you provided failed to post because your server is thirteen seconds ahead of our server, which is technically in the future. This was very helpful. Thank you for reaching back out to us.
I should have time this week to put in a fix.

Hi @huckphin. Is it right now ? I think itsn't solved.

I just had this very same error.

@huckphin I think that the plugin doesn't take WP timezone (set in the settings) into consideration when preparing the publication date for Medium. I was getting the error from a server with a properly synced clock. After changing the timezone to UTC in the WP Admin -> Settings -> General, it started working.


Bot casts spells?

I added Sidekick less than 24 hours ago to our Discord because our regular dice server is dead, and Sidekick has proven to be a great tool to use instead. However, it randomly yelled "Reducto!" at one of the Discord members and I'm kinda baffled as to why.
Any explanation would be greatly appreciated!
EDIT: It's been casting more HP spells, while I'm not worried as such, it was never specified it did that on the instructions.

Hi!
The bot is prone to throw a random spell at you if it failed to understand you. Talk about bad temper.
I'm sorry, I blush or cough every time it misbehaves, but I have no other.


New feats and bug fixes

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


Jax 0.4.27: ValueError: safe_map() argument 2 is shorter than argument 1

Since jax 0.4.27, several tests fail with:
args = (_ClosureConvert(
  jaxpr={ lambda ; a:f32[47] b:f32[] c:i32[] d:bool[] e:bool[] f:i32[] g:f32[] h:f32[47]
    i:bool[...t 0x7fff4c435a90>,
  _makes_false_steps=False
), Traced<ShapedArray(float32[47])>with<DynamicJaxprTrace(level=5/1)>))))
kwds = {}

    @wraps(func)
    def inner(*args, **kwds):
        with self._recreate_cm():
>           return func(*args, **kwds)
E           ValueError: safe_map() argument 2 is shorter than argument 1
E           --------------------
E           For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.

Thanks for the report! I've reproduced this as an upstream bug in JAX: https://github.com/google/jax/issues/21116

Thank you for the quick response !
The following tests also fail with a similar error message:
=========================== short test summary info ============================
FAILED tests/test_ad.py::test_closure_convert_basic - assert [] == [140730519276576]
FAILED tests/test_debug.py::test_backward_nan - AssertionError: assert 'foo:\n   pri...pe=float32)\n' == 'foo:\n   pri...pe...
FAILED tests/test_noinline.py::test_abstract - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_num_traces - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_pytree_in - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_simple - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_mlp - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_vmap - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_jvp - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_grad - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
FAILED tests/test_noinline.py::test_complicated - TypeError: emit_python_callback() takes 6 positional arguments but 7 positi...
=========== 11 failed, 337 passed, 1 skipped, 39 warnings in 23.96s ============

E       TypeError: emit_python_callback() takes 6 positional arguments but 7 positional arguments (and 1 keyword-only argument) were given
E       --------------------
E       For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.

Ah, looks like a bit of other work might be required too. These other failures at least don't look too scary: I think they're just small perturbations from where things were before, and that we can/should adjust in Equinox.

Okay, I think things should be fixed with https://github.com/patrick-kidger/equinox/pull/719. I'll aim to do a new release shortly.

Cool ! I will test that as soon as the release is out.
Thanks @patrick-kidger !

All good, thanks !


Add image/information to our dockerhub repo

The Cloud Native Buildpacks Docker Hub repos (buildpacksio and (cnbs)(https://hub.docker.com/u/cnbs)) look pretty sketchy. cnbs is understandable, given that it seems like we only host sample/internal images (though we do reference it in our docs), but given that buildpacksio is a bit more external (we will soon be referring to it in our tekton task (https://github.com/tektoncd/catalog/pull/434), and it is our "official" lifecycle/pack image), we should probably spend some effort making it a bit more trustworthy.
Some potential tasks may be:

[ ] Add our logo for both cnbs and buildpacksio
[ ] Add a description/logo to buildpacksio/lifecycle
[ ] Add a description/logo to buildpacksio/pack

@samj1912 could you break this up into multiple issues in their respective repos?


Windows: damageRegion support

The following patch adds damage region support:
https://gist.github.com/migueldeicaza/852a6b03b828a045f1fcf79d1ab8b331

Implemented


Added .NET Standard 2.0 support to library

We needed .NET Standard 2.0 support in our use case. This PR contains changes related to spot which is not compatible with it. I know that using System.Timers.Timer is a step back compared to PeriodicTimer but changing target framework makes this library much more flexible.

Sorry, I'm not gonna accept this

That's unfortunate. Would you mind explaining reason behind this decision?

I don't want to commit to supporting net standard in the future

I see. Will close it then; thanks for reaching out. Have a good weekend.


Error while running unit test

When running the 'make unit-test', I got an error message "make: *** [unit-test] Error 1".
Will this be affecting my environment?
I have attached the log file.
UnitTest_Errorlogs.txt

This is not a bug, but some network configuration problem on your side.  It seems that the baseimage cannot be built or fetched, and github cannot be accessed.  Please make sure that your network connection is working correctly.

In addition to the observation made by @corecode, I also see this in the first few lines of the log:
Error starting userland proxy: listen tcp <IP_ADDRESS>:30303: bind: address already in use

This implies you may have already had a peer running when you tried to launch the unit-tests.


Check for required elements on tour start?

Hey, I was thinking we should probably be checking for the required elements when the tour starts. I've found that, between setting the steps and starting the tour, elements might change on the page. I am setting the steps of my tour for a route in setupController, so is maybe there's a better place to do the tour configuration?

@craigsketchley why not just set your steps after the elements you want are defined?

Yeah, I definitely could. I'm just thinking it would be an improvement from a usability perspective. It feels like checking for the required elements on the page would be better associated with starting the tour than configuring the tour. Things are breaking at the moment if you set the required elements after setting the steps and I wouldn't know why without looking inside the library code - I think it'd be neater if this wasn't the case. The same goes for the requirement for setting the defaults before setting the steps - it'd also be good to remove this requirement.

I was trying to avoid doing all of that when you call start, but maybe it's not terrible to have it happen on start. I feel like I had reasons for it, but I can't remember them. I'm open to a PR, if you'd like to give it a shot.

Cool. I'll have a crack at it.

Maybe we should rewrite it to just set it all up in one config object and then set each respective thing. Let me take a look this weekend actually.

@craigsketchley as a side note, I started an organization Ship Shape and I want to just build a bunch of Ember Addons for the community and add them to the organization's portfolio. You may have noticed I moved this addon into the organization already. Not sure how interested you are in the addons I have created already or if you have ideas for new ones as well, or if you even have much time to devote to open source dev, but would you be interested in being a part of it?

@craigsketchley I ended up having a lot of other stuff to work on this weekend working towards getting the Ship Shape website up and running. I'll definitely take a look at moving all of this logic into the tour start this week though. I think we can get one central config and setup will be a lot smoother. Sorry for the delay!

No, no worries. I was moving house so didn't have any time to look into either :smile: Yeah good plan. Let me know if there's anything I can help with.

@craigsketchley I think the issue here is, if we move all the logic to when the tour starts, every time you stop and start the tour, it has to do all the setup again, versus having the initialization of a new tour only happen when you change the steps. What are your thoughts? I'm open to suggestions on how to implement this.

My initial thoughts were to essentially extract this required elements check from the steps observer into it's own observer on the isActive property. I'd also probably remove the autoStart from the steps observer too, feel like that should be called in an ember lifecycle hook. I've got the start of these changes in my own fork - here's the diff.

@craigsketchley I think we have to completely remove the steps observer, and just do all of this stuff when start is called.
As for autoStart in a hook, what hooks are available inside an Ember service?

Yeah, I think you might be right - rather than rely on observers where the order is less defined. As long as all the processing doesn't noticeably delay/lock the tour from starting.
I looked into service hooks and there doesn't seem to be any we can use. I'm thinking that maybe autoStart shouldn't be the responsibility of ember-shepherd? If an app is setting autoStart: true, then they may as well call tour.start(). Auto starting the tour on some condition sounds like logic for the app to handle.

I agree with your thoughts on autoStart. We probably should just let the user start it whenever they need to.

I've still been spending most of my time working on the new Ship Shape site and the ember-3d-nav. Feel free to think about this some more and let's try to nail down exactly what we want and then we can implement it.

:+1: sounds good.

A PR would be great :smile:. I can't really fully dive into this until I fix up the website.

@craigsketchley any more thoughts or progress on the issue?

@bekzod also feel free to give your input on this issue.

@craigsketchley haven't heard from you in awhile. You still interested in working on this?

Hey @rwwagner90, yeah sorry, a shepherd tour became low priority for the product I'm working on. I can probably get round to this at the weekend?

@craigsketchley do you think you could work on this soon? I'd love to close it out. I have a ton of side work I am doing now, so I don't have time to implement this myself currently.

@craigsketchley any updates?

@craigsketchley I'm inclined to close this if there is no longer any interest in working on it. Let me know please.

Hey @rwwagner90, really sorry I haven't gotten round to this. I'll submit the PR tonight.

@craigsketchley no worries, we're all super busy. Just wanted to check if there was still interest

@craigsketchley could you submit that PR please?

Hey @rwwagner90. Sorry for being slack here, taken on way too much lately and we're not using this in our product anymore so became a little hard to justify working on it. I've had a play and it turns out to be a bigger task than anticipated. Not sure I'll get much free time until July (University exams 😞 ). I've pushed what I've got to the requirdElementsFix branch and it's working well, but it still needs a bit of work (testing). Moving all the tour creation into tour.start() does make it difficult to unit test the service, so not completely happy with it.

@craigsketchley no worries at all. Thanks, I'll check it out. Did you switch to a different tour library or did you remove the tour completely, in your product?

Cheers. We're just not using a tour for now. We decided to reprioritise some features and it got bumped way down. We may revisit it in the future though.

Gotcha, just glad you didn't find a better tour library out there 😛

I'm going to close this for now. If anyone in the future has some big problems with this issue. Please leave a comment and we can look at implementing it.

@evoactivity your elements would not be there until the page renders, so I don't think beforeModel or afterModel are what you want here. You want to wait until the page is rendered and then start the tour.


Ensure vehicle is always within planning trajectory

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


Config file does not allow nested stringify functions

See https://groups.google.com/forum/#!topic/mountebank-discuss/6ShnSHx-GAc
and
https://gist.github.com/zot24/3cf6df65c8ae3e63c2deabc4ed7174b9

Fixed in 0e02d68

Fixed in 0e02d68

hey Brandon, I am still facing this issue. Nested stringily failing for me with error something like this
{"errors":[{"code":"invalid injection","message":"invalid response injection","source":"(function (request, state, logger) {\n    logger.info('apiStub.ejs Called');\n    var apiKey = request.method + '-' + request.path;\n    logger.info(apiKey);\n    return {\n      headers: {\n        'Content-Type': 'application/json'\n      },\n      body: JSON.stringify({ count: 10 }\n)\n    };\n}\n)(scope, injectState, logger, deferred.resolve, imposterState);","data":"Invalid or unexpected token","name":"Error","stack":"Error\n    at Object.from (/usr/local/lib/node_modules/mountebank/src/util/inherit.js:15:17)\n    at createError (/usr/local/lib/node_modules/mountebank/src/util/errors.js:10:26)\n    at Object.InjectionError (/usr/local/lib/node_modules/mountebank/src/util/errors.js:25:16)\n    at inject (/usr/local/lib/node_modules/mountebank/src/models/responseResolver.js:43:44)\n    at processResponse (/usr/local/lib/node_modules/mountebank/src/models/responseResolver.js:237:20)\n    at Object.resolve (/usr/local/lib/node_modules/mountebank/src/models/responseResolver.js:271:16)\n    at Object.resolve (/usr/local/lib/node_modules/mountebank/src/models/stubRepository.js:124:18)\n    at F.respond (/usr/local/lib/node_modules/mountebank/src/models/http/baseHttpServer.js:78:34)\n    at /usr/local/lib/node_modules/mountebank/src/models/abstractServer.js:96:35\n    at _fulfilled (/usr/local/lib/node_modules/mountebank/node_modules/q/q.js:854:54)"}]}

Btw, this is how you can recreate the problem
$mb --configfile templates/originServer.ejs --allowInjection
templates/originServer.ejs
{
"port": 5555,
"protocol": "http",
"name": "origin",
"stubs": [
{
"responses": [{ "inject": "<%- stringify(filename, 'apiStub.ejs') %>" }]
}
]
}
templates/apiStub.ejs
function (request, state, logger) {
logger.info('apiStub.ejs Called');
var apiKey = request.method + '-' + request.path;
logger.info(apiKey);
return {
headers: {
'Content-Type': 'application/json'
},
body: JSON.stringify(<%- stringify(filename, 'GET-request.ejs') %>)
};
}
templates/GET-request.ejs
{ count: 10 }

What version of mountebank are you using?
mb --version
Also, I'm assuming you get the error on startup, correct? Assuming that's
the case, I couldn't reproduce with the latest version, so I imagine the
problem is that you're simply on an older version.
-Brandon
On Thu, Sep 21, 2017 at 7:38 AM, Rohit<EMAIL_ADDRESS>wrote:

Btw, this is how you can recreate the problem
$mb --configfile templates/originServer.ejs --allowInjection
templates/originServer.ejs
{
"port": 5555,
"protocol": "http",
"name": "origin",
"stubs": [
{
"responses": [{ "inject": "<%- stringify(filename, 'apiStub.ejs') %>" }]
}
]
}
templates/apiStub.ejs
function (request, state, logger) {
logger.info('apiStub.ejs Called');
var apiKey = request.method + '-' + request.path;
logger.info(apiKey);
return {
headers: {
'Content-Type': 'application/json'
},
body: JSON.stringify(<%- stringify(filename, 'GET-request.ejs') %>)
};
}
templates/GET-request.ejs
{ count: 10 }
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bbyars/mountebank/issues/183#issuecomment-331143776,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAESvyMs5ZteLaT5obZZdLR2naJQCMd7ks5skliygaJpZM4LYwn7
.

Hi Brendan, I get this error when I call the port 5555, instead startup.
Here is what I am running on my laptop
~ $mb --version
1.12.0

Thanks for the extra info. I identified the bug and fixed it - it has to do
with an extra newline at the end of the nested stringify funciton. I aim to
release the next version this week.
On Sun, Sep 24, 2017 at 6:30 AM, Rohit<EMAIL_ADDRESS>wrote:

Hi Brendan, I get this error when I call the port 5555, instead startup.
Here is what I am running on my laptop
~ $mb --version
1.12.0
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bbyars/mountebank/issues/183#issuecomment-331703804,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAESv0nwXuDCLsursno19t9Zr61C6lxDks5slj1EgaJpZM4LYwn7
.

Hi Brendan, has this been fixed? I am facing the same issue, I am able to replicate the issue with @rohgoyal imposter and ejs file.
I am using 1.14.1
Thanks!

Hi @leogtzr. Interesting...I can't reproduce it. I thought I fixed it in v1.13.0 (see https://github.com/bbyars/mountebank/issues/293 and the corresponding commit). A local test using @rohgoyal 's files works for me, returning a body of "{"count":10}" when calling the imposter on port 5555, and logging the "apiStub.ejs Called".
What O/S are you using?

Hello, sorry for the late response: I am currently using:
[0 [17:26][leo@~]$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 18.04 LTS
Release: 18.04
Codename: bionic
[0 [17:26][leo@~]$
I will see if I can reproduce it this weekend and send you over the files.
Thank you!
El lun., 4 jun. 2018 a las 14:05, Brandon Byars<EMAIL_ADDRESS>escribió:

Hi @leogtzr https://github.com/leogtzr. Interesting...I can't reproduce
it. I thought I fixed it in v1.13.0 (see #293
https://github.com/bbyars/mountebank/issues/293 and the corresponding
commit). A local test using @rohgoyal https://github.com/rohgoyal 's
files works for me, returning a body of "{"count":10}" when calling the
imposter on port 5555, and logging the "apiStub.ejs Called".
What O/S are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bbyars/mountebank/issues/183#issuecomment-394481209,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJ-QTZcBYV061U63pUPmROeiY58Zslwks5t5ZMAgaJpZM4LYwn7
.

--

Leonardo Gutiérrez R.*

OK thanks, keep me posted.
On Fri, Jun 8, 2018 at 6:27 PM, Leo Gutiérrez Ramírez <
<EMAIL_ADDRESS>wrote:

Hello, sorry for the late response: I am currently using:
[0 [17:26][leo@~]$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 18.04 LTS
Release: 18.04
Codename: bionic
[0 [17:26][leo@~]$
I will see if I can reproduce it this weekend and send you over the files.
Thank you!
El lun., 4 jun. 2018 a las 14:05, Brandon Byars<EMAIL_ADDRESS>
)
escribió:

Hi @leogtzr https://github.com/leogtzr. Interesting...I can't
reproduce
it. I thought I fixed it in v1.13.0 (see #293
https://github.com/bbyars/mountebank/issues/293 and the corresponding
commit). A local test using @rohgoyal https://github.com/rohgoyal 's
files works for me, returning a body of "{"count":10}" when calling the
imposter on port 5555, and logging the "apiStub.ejs Called".
What O/S are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/bbyars/mountebank/issues/183#issuecomment-394481209
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABJ-
QTZcBYV061U63pUPmROeiY58Zslwks5t5ZMAgaJpZM4LYwn7>
.

--

Leonardo Gutiérrez R.*

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bbyars/mountebank/issues/183#issuecomment-395918407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAESvxhsiYZQnQshRQPcg4Kt3hl7XlV5ks5t6whhgaJpZM4LYwn7
.


Need the INIT Packet for the nRF 51 example

New version of DFU requires an INIT packet.  Does anyone have an INIT packet for this sample?

FYI, https://github.com/ucxpresso/nano51822/tree/master/examples/cookbook/ble_uri_beacon_v2


Inception-ResNet

🚀 Feature

A feature request for Inception-ResNet model.
Thanks.

Hello @tejan-rgb. I've added your request to our list. See the discussion at #2707. Feel free to comment there.

Thanks.


Feature/UI changes

Could you center this QR code and remove this text background?

Could you elaborate on what this pr is supposed to change, so that I know what to test?

I think that the copy button for the token and client id is not working.

Also for me it seems like the disconnect button do not work properly. Steps to reproduce:

Create two rooms, each with one peer
Connect both peers
try disconnecting either one of them

As I look into the jellyfish logs there is no disconnect message coming and peers are still connected

I think that the copy button for the token and client id is not working.

Could you elaborate on this please? It seems to work fine on my version. Do you get the toast "Copied to the clipboard" but the clipboard does not change or nothing happens at all?

Also for me it seems like the disconnect button do not work properly. Steps to reproduce:

Create two rooms, each with one peer
Connect both peers
try disconnecting either one of them

As I look into the jellyfish logs there is no disconnect message coming and peers are still connected

This bug is also present on main branch. Should it be repaired in this PR then or I ought to work it out elsewhere?

I think that the copy button for the token and client id is not working.

Could you elaborate on this please? It seems to work fine on my version. Do you get the toast "Copied to the clipboard" but the clipboard does not change or nothing happens at all?

I got no toast, it was observer on chrome 115

Also for me it seems like the disconnect button do not work properly. Steps to reproduce:

Create two rooms, each with one peer
Connect both peers
try disconnecting either one of them

As I look into the jellyfish logs there is no disconnect message coming and peers are still connected

Thank you for reporting this. It seems like it was in a dashboard for a while now! This bug is also present on main branch. Should it be repaired in this PR then or I ought to work it out elsewhere?

if it will be quick and small fix I suggest doing it here since it is pretty important feature, else do a followup soon

Display HLS and Server State should be unchecked by default
The text 'Reset to default' is unreadable when clicked due to a very low contrast value


CJK in URL causes error in webserver

I request the follow url:
curl -X GET http://localhost:8080/greet/小高
and then,log:
2018.09.11 12:53:09 严重 io.helidon.webserver.RequestRouting Thread[nioEventLoopGroup-3-3,10,main]: Unexpected error occurred during routing! java.lang.IllegalArgumentException: Illegal character in path at index 9: /greet/å°é«         at java.net.URI.create(URI.java:852)         at io.helidon.webserver.netty.BareRequestImpl.getUri(BareRequestImpl.java:81)         at io.helidon.webserver.RequestRouting.createRequestSpan(RequestRouting.java:123)         at io.helidon.webserver.RequestRouting.route(RequestRouting.java:69)

I've reproduced this. The webserver immediately closes the connect upon hitting the exception which results in an empty response to the client. This is clearly a bug. On an error the server should be returning the correct HTTP status (maybe 400 in this case? See next point).
The next question is if the URL is valid as represented in the HTTP request sent to the server. It appears as though curl is not encoding the URL. The webserver is assuming that incoming URLs  are correctly encoded (as defined by the HTTP protocol). In this case the URL in the HTTP request is not URL encoded (It is not http://localhost:8080/greet/%E5%B0%8F%E9%AB%98). And therefore the webserver gets the exception.
If you try the same thing but use a browser to enter the URL it should work, because the browser encodes the URL. It worked for me with both Chrome and Firefox.
But we do need to fix the error handling in the webserver so that it handles the connection and returns an HTTP status code.

This is fixed by PR #796


add addon-manager label in order to not reconcile heapster deployment

heapster-nanny container is modifying the deployment and addon-manager is reconciling the change....
This label makes addon-manager ensure a resource of that name exists. https://github.com/kubernetes/kubernetes/tree/master/cluster/addons/addon-manager

This change is

@lachie83,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@acs-bot test this please

@acs-bot test this please

@acs-bot test this please


Fix parquet writer statistics

Which issue does this PR close?
Closes #2014
Closes #2015
Closes #2016
Rationale for this change
Fixes a number of bugs
What changes are included in this PR?
Removes max_statistics_size as it isn't used, and changes statistics to be enabled based on an enumeration. This allows consistently only producing chunk level statistics, potentially precomputed, or not.
Are there any user-facing changes?
No

Codecov Report

Merging #2022 (428f0d5) into master (38370cb) will decrease coverage by 0.16%.
The diff coverage is 66.40%.

:exclamation: Current head 428f0d5 differs from pull request most recent head 3049111. Consider uploading reports for the commit 3049111 to get more accurate results

@@            Coverage Diff             @@
##           master    #2022      +/-   ##
==========================================
- Coverage   83.59%   83.43%   -0.17%
==========================================
  Files         222      222
  Lines       57529    57911     +382
==========================================
+ Hits        48091    48317     +226
- Misses       9438     9594     +156

Impacted Files
Coverage Δ

parquet/src/arrow/array_reader/byte_array.rs
84.47% <0.00%> (-1.24%)
:arrow_down:

...et/src/arrow/array_reader/byte_array_dictionary.rs
82.26% <0.00%> (-1.66%)
:arrow_down:

...uet/src/arrow/array_reader/complex_object_array.rs
93.20% <0.00%> (-1.07%)
:arrow_down:

parquet/src/arrow/array_reader/empty_array.rs
45.45% <0.00%> (-10.11%)
:arrow_down:

parquet/src/arrow/array_reader/list_array.rs
92.69% <0.00%> (-0.72%)
:arrow_down:

parquet/src/arrow/array_reader/map_array.rs
58.82% <0.00%> (-8.98%)
:arrow_down:

parquet/src/arrow/array_reader/mod.rs
88.23% <ø> (ø)

parquet/src/arrow/array_reader/null_array.rs
81.48% <0.00%> (-6.52%)
:arrow_down:

parquet/src/arrow/array_reader/primitive_array.rs
88.63% <0.00%> (-1.02%)
:arrow_down:

parquet/src/arrow/array_reader/struct_array.rs
78.99% <0.00%> (-9.69%)
:arrow_down:

... and 21 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 38370cb...3049111. Read the comment docs.


Na/msal 0.2.3

AOT fix
Single Sign on
IE and edge bug fix if navigateToLoginRequestUrl=false (cookies not deleted)
IE and edge bug fix for login_popup (state mismatch)
IE and edge bug fix for msal-angularjs
User state not passed to callback if navigateToLoginrequestUrl = false
Added sample app for single sign on
Fixed broken end to end test cases

:shipit:


SmartProxy E2E test is failing

Describe the bug
The SmartProxy E2E test is failing with the following error:
Microsoft.Health.Fhir.Tests.E2E.SmartProxy.SmartProxyTests.GivenPatientIdAndSmartAppUrl_WhenLaunchingApp_LaunchSequenceAndSignIn [FAIL]
Failed  Microsoft.Health.Fhir.Tests.E2E.SmartProxy.SmartProxyTests.GivenPatientIdAndSmartAppUrl_WhenLaunchingApp_LaunchSequenceAndSignIn
Error Message:
OpenQA.Selenium.WebDriverException : unknown error: Element <input type="submit" id="idSIButton9" class="btn btn-block btn-primary" data-bind="
To Reproduce
Steps to reproduce the behavior:

Just running the test against a test environment is enough to trigger the failure.

Expected behavior
Test should pass
Actual behavior
Test fails

@namadabu is this a persistent error or does it repeating fix it. Some of these selenium tests are a bit problematic.

This test started failing recently and in a consistent manner. We (Raja and @brandonpollett) figured out that we were moving through the buttons/actions too quickly. PR out to fix it - https://github.com/Microsoft/fhir-server/pull/410


[Issue] Stretched image in spotted notifications

Not sure why this is happening, but all of a sudden I'm getting very stretched token images in the chat notification for lingering AP being removed. Screenshot attached.

I  can not replicate this bug, which ui modules are you using?

Dorako UI. My settings haven't changed, though. I'll keep digging to see if another module is doing something weird.

Find the Culprit has narrowed it down to a conflict with Pf2e Action Support Engine. I'll contact the dev for that.

I will also look into it, there may be something i can do about that

There is a small chance, that v2.9.1 fixes this bug

Awesome. And the Action Support Engine dev says it will be fixed in his next release.

Just checked with 2.9.1 and the stretched image is still there. I'll keep you posted when ASE gets an update and I can check again.

Is this still happening?

Nope, fixed!


[dev docs] stackoverflow with block quote in command

\showmd{
  > ABC
  > DEF
  >
  > GHI **JKL**
}

partially solved with change inside FranklinParser but nesting still leads to SO

solved but still not ok

no paragraph break on line skip (?) maybe there's a nop somewhere, not sure
in nesting there shouldn't require to be a empty line (which is what seems to be the issue in screenshot below)


Control for random seed in train_test_split  in experimental branch

Placeholder more than anything as it's not merged/nor PR-ready, just so we don't forget it.
In the benchmarking script here, we should control the random_state. So the permutation done before could be skipped with using the seed in place here.

Way outdated, closing.


[0.5.0] externalTrafficPolicy: Local does not work. kube-vip does not detect endpoint changes in table mode (or at all?)

Describe the bug
externalTrafficPolicy: Local does not work. kube-vip does not detect endpoint changes in table mode (or at all?)
To Reproduce
Steps to reproduce the behavior:
1.Apply service + single-pod deployment manifest with affinity to single node (i.e. test-lab-worker02 in my case)
2. Change deployment pod affinity to another node (i.e. test-lab-worker03 in my case)
3. See no changes for loadbalancer address announcement
Expected behavior
kube-vip must remove the loadbalancer ip address announcement  from the node with no local pods and add it to another node
Screenshots
Deployment manifest
---
apiVersion: v1
kind: Service
metadata:
  name: nginx
  labels:
    app: nginx
spec:
  ports:
  - port: 80
    protocol: TCP
    name: http
  selector:
    app: nginx
  externalTrafficPolicy: Local
  type: LoadBalancer
  loadBalancerIP: <IP_ADDRESS>
---
kind: Deployment
apiVersion: apps/v1
metadata:
  name: nginx
  labels:
    app: nginx
spec:
  replicas: 1
  strategy:
    type: RollingUpdate
    rollingUpdate:
      maxSurge: 0
      maxUnavailable: 1
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: kubernetes.io/hostname
                operator: In
                values:
                - test-lab-worker02
      containers:
      - image: nginx:1.21.5
        imagePullPolicy: IfNotPresent
        name: nginx
        ports:
        - containerPort: 80
          protocol: TCP

Environment (please complete the following information):

OS/Distro: Ubuntu 1804
Kubernetes Version: v.1.20.15
Kube-vip Version: 0.5.0

Kube-vip.yaml:
---
apiVersion: v1
kind: ServiceAccount
metadata:
  name: kube-vip
  namespace: kube-system
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  annotations:
    rbac.authorization.kubernetes.io/autoupdate: "true"
  name: system:kube-vip-role
rules:
  - apiGroups: [""]
    resources: ["services", "services/status", "nodes", "endpoints"]
    verbs: ["list","get","watch", "update"]
  - apiGroups: ["coordination.k8s.io"]
    resources: ["leases"]
    verbs: ["list", "get", "watch", "update", "create"]
---
kind: ClusterRoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: system:kube-vip-binding
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: system:kube-vip-role
subjects:
- kind: ServiceAccount
  name: kube-vip
  namespace: kube-system
---

apiVersion: apps/v1
kind: DaemonSet
metadata:
  creationTimestamp: null
  labels:
    app.kubernetes.io/name: kube-vip-ds
    app.kubernetes.io/version: v0.5.0
  name: kube-vip-ds
  namespace: kube-system
spec:
  selector:
    matchLabels:
      app.kubernetes.io/name: kube-vip-ds
  template:
    metadata:
      creationTimestamp: null
      labels:
        app.kubernetes.io/name: kube-vip-ds
        app.kubernetes.io/version: v0.5.0
    spec:
      containers:
      - args:
        - manager
        - --log=5
        - --table
        - --services
        - --servicesElection
        - --serviceInterface=kube-dummy-if
        image: ghcr.io/kube-vip/kube-vip:v0.5.0
        imagePullPolicy: Always
        name: kube-vip
        resources: {}
        securityContext:
          capabilities:
            add:
            - NET_ADMIN
            - NET_RAW
      hostNetwork: true
      serviceAccountName: kube-vip

Additional context
kube-vip logs (initial pod placenent node)
time="2022-09-09T12:30:33Z" level=info msg="Starting kube-vip.io [v0.5.0]"
time="2022-09-09T12:30:33Z" level=info msg="namespace [kube-system], Mode: [Routing Table], Features(s): Control Plane:[false], Services:[true]"
time="2022-09-09T12:30:33Z" level=info msg="No interface is specified for VIP in config, auto-detecting default Interface"
time="2022-09-09T12:30:33Z" level=info msg="kube-vip will bind to interface [ens3]"
time="2022-09-09T12:30:33Z" level=warning msg="the status of the interface kube-dummy-if is unknown. Ensure your interface is ready to accept traffic, if so you can safely ignore this message"
time="2022-09-09T12:30:33Z" level=info msg="prometheus HTTP server started"
time="2022-09-09T12:30:33Z" level=info msg="Starting Kube-vip Manager with the Routing Table engine"
time="2022-09-09T12:30:33Z" level=info msg="all routing table entries will exist in table [198]"
time="2022-09-09T12:30:33Z" level=info msg="beginning watching services, leaderelection will happen for every service"
time="2022-09-09T12:30:33Z" level=warning msg="loadBalancer has External Traffic Policy: Local, no local pods found"
time="2022-09-09T12:35:02Z" level=info msg="service [nginx] has been added/modified it has an assigned external addresses [<IP_ADDRESS>]"
time="2022-09-09T12:35:02Z" level=info msg="beginning services leadership, namespace [default], lock name [kubevip-nginx], id [adm-vm-k8s-test-lab-worker02]"
I0909 12:35:02.601865       1 leaderelection.go:248] attempting to acquire leader lease default/kubevip-nginx...
I0909 12:35:02.701862       1 leaderelection.go:258] successfully acquired lease default/kubevip-nginx
time="2022-09-09T12:35:02Z" level=info msg="adding the service [default/nginx] with external address [<IP_ADDRESS>]"
time="2022-09-09T12:35:02Z" level=info msg="new VIP [<IP_ADDRESS>] for [default/nginx] "
time="2022-09-09T12:35:02Z" level=info msg="Starting advertising address [<IP_ADDRESS>] with kube-vip"
time="2022-09-09T12:35:02Z" level=info msg="Started Load Balancer and Virtual IP"
time="2022-09-09T12:35:03Z" level=info msg="service [nginx] has been added/modified it has an assigned external addresses [<IP_ADDRESS>]"

kube-vip logs (second node)
time="2022-09-09T12:30:35Z" level=info msg="Starting kube-vip.io [v0.5.0]"
time="2022-09-09T12:30:35Z" level=info msg="namespace [kube-system], Mode: [Routing Table], Features(s): Control Plane:[false], Services:[true]"
time="2022-09-09T12:30:35Z" level=info msg="No interface is specified for VIP in config, auto-detecting default Interface"
time="2022-09-09T12:30:35Z" level=info msg="prometheus HTTP server started"
time="2022-09-09T12:30:35Z" level=info msg="kube-vip will bind to interface [ens3]"
time="2022-09-09T12:30:35Z" level=warning msg="the status of the interface kube-dummy-if is unknown. Ensure your interface is ready to accept traffic, if so you can safely ignore this message"
time="2022-09-09T12:30:35Z" level=info msg="Starting Kube-vip Manager with the Routing Table engine"
time="2022-09-09T12:30:35Z" level=info msg="all routing table entries will exist in table [198]"
time="2022-09-09T12:30:35Z" level=info msg="beginning watching services, leaderelection will happen for every service"
time="2022-09-09T12:30:35Z" level=warning msg="loadBalancer has External Traffic Policy: Local, no local pods found"
time="2022-09-09T12:35:04Z" level=warning msg="loadBalancer has External Traffic Policy: Local, no local pods found"
<< nothing about endpoint change here >>

It seems this issue is related to #431

ha, I'm literally working on this right now 😂

Any status on this one? Finding this to be a problem in clusters we're using with kube-vip. Thx in advance.

Yes, I've a load of code that I'm adding at the moment.

Hi @thebsdbox checking in to see how this one is progressing. Have a need for this one, but kinda stuck until it's fixed. Thanks in advance.

With the latest release 5.5 there now is a endpoint watcher that will continually watch for pod endpoints and adjust when things become available or things such as pods/nodes are removed.

Tested it again.
It is almost working now, thanks!
One more thing, for routing table mode, kube-vip should remove route if there is no local endpoints.

haha, can't please everyone.. Feel free to create a PR? Or i'll add it to my list of things to do.

Hello, is this ticket solved or not ?

For everything other than routing mode.

This is now implemented for all modes.


env.action_space.sample() Equivalent?

I'm just messing around with gym-core.PongDeterministic-v3 and was wondering how does one do the equivalent of a random action for the Universe agent.

env.action_space.sample() works just fine! You may want to wrap with a SafeActionSpace wrapper to avoid a fully random agent clicking around: https://github.com/openai/universe/blob/master/example/random-agent/random-agent.py#L24-L35

Excellent, thanks.


Transfer money to Code for Durham

For Civic Spark Day. Is it $500 or $550?

550
Chris Mathews
On Mon, Oct 2, 2017 at 10:45 AM, Jason Hibbets<EMAIL_ADDRESS>wrote:

For Civic Spark Day. Is it $500 or $550?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/812, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAOGtUvHygs1oWhDe_P9t8-QbroAl5Zpks5soPbtgaJpZM4Pqv6B
.

I'm confused.  I thought it was $1000.

This is a different topic from Civic Spark day
On Mon, Oct 2, 2017 at 2:18 PM, aikimark<EMAIL_ADDRESS>wrote:

I'm confused. I thought it was $1000.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/812#issuecomment-333619930,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADgoV6Z93tHw6c-MicatCrZa3ggjrZpEks5soSjkgaJpZM4Pqv6B
.

@ChrisTheDBA So CfA refunded my $550 to my AMEX. Do we need to initiate a transfer on our side? I had submitted an invoice / expense before. not sure if it was rejected or not.

Laura said they were reimbursed


Feature Request - Share Webpage from Browser to Create New Bookmark Entry

This is just a suggestion to add the ability to share a webpage in your browser to this app and have it auto-fill the Add Record form, allowing you to save the webpage as a bookmark.

Thanks for the suggestion! This feature was planned for a while but I have no technical knowledge to implement it. Hence is being put off for a while. I'll be glad if you can help with this one.

So the approach I am thinking for adding this feature is to implement a URL resolver which could open Sitemarker with the record name as the title of the tab and the url the current url.
Tl;dr: Add the resolver in front of the URL to get Sitemarker's add records up with values autofilled. All yet to be done after that is to confirm adding the record.
This feature may take a while to be implemented properly but please do look forward to it!

Hey @cosmobooker! Thanks for reporting the issues and patiently waiting for them to be implemented. Since you have opened 3 issues, I will summarize the status here to avoid multiple pings:

This issue: This is now solved. You can now share bookmarks from Android to the app by using the share menu option. As soon as possible, a build of the app supporting this feature will be made available. Please do let me know if it does not work.
Issue #34: Sitemarker 3 doesn't support import/export yet. However, it will be made available in version 3.1.0. I did not have enough time to test the input and output files so the fix for this issue did not make it into 3.0. However, I want you to know that the fix is ready and will be available in the next release.
Issue #35: As with the above issue, I am not confident in the fix I had made for this issue as well and hence did not include it in this release. I will include the fix for this issue in the next release along with the previous issue.

I appreciate you reporting multiple issues and feature requests for Sitemarker. Thank you!