phantomjs-2.1.1-linux-x86_64 proxy authentication not working

Which version of PhantomJS are you using?
phantomjs-2.1.1-linux-x86_64

What steps will reproduce the problem?
1.Try some proxy that needs authentication

Which operating system are you using?
centos 6.6

Did you use binary PhantomJS or did you compile it from source?
binary

Please provide any additional information below.
I tried command line option by setting --proxy and --proxy-auth also phantom.setProxy() . None of the method works when the proxy needs authentication.

What kind of proxy did you use?

I was using a http proxy. My bad. Tried with some other proxies. It's working fine. Closing the issue.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain castonguay.org has exceeded the max emails per hour (231/200 (115%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o1.sgmail.github.com ([<IP_ADDRESS>]:53758)
by server191.web-hosting.com with esmtps (TLSv1.2:DHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.86_1)
(envelope-from<EMAIL_ADDRESS>id 1apkQj-003N3P-Hj
for<EMAIL_ADDRESS>Mon, 11 Apr 2016 18:35:38 -0400
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=github.com;
h=from:reply-to:to:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=YrZtEAxSMRQKDNkpldSrxXrOSYg=; b=AiSGCcBQfI3tK3jl
8U7THDDonwvXwduCXe1DFl7wIt9cQB63+Ax3w/NYSaTy9RmMUDJg5PpYKNgqB7ZG
BLRKvFqRw7S6x1HnW88fFB24vaj/gg1kCivrX5OabHsNdYZ9eyUZYBqOj3vGKDw8
8+jAfLJubuFBaHJ7DyBWyEU41QE=
Received: by filter0823p1mdw1.sendgrid.net with SMTP id filter0823p1mdw1.19941.570C268F23
2016-04-11 22:34:55.795286642 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0002p1iad1.sendgrid.net (SG) with ESMTP id 3AAS2sACT7u4wh3NAGGOEQ
for<EMAIL_ADDRESS>Mon, 11 Apr 2016 22:34:55.659 +0000 (UTC)
Date: Mon, 11 Apr 2016 15:34:55 -0700
From: hrushikesh198<EMAIL_ADDRESS>Reply-To: ariya/phantomjs<EMAIL_ADDRESS>To: ariya/phantomjs<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [ariya/phantomjs] phantomjs-2.1.1-linux-x86_64 proxy
authentication not working  (#14193)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_570c268f8001b_40be3fc57f4ed2a0416795";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: hrushikesh198
X-GitHub-Recipient: dREc
List-ID: ariya/phantomjs <phantomjs.ariya.github.com>
List-Archive: https://github.com/ariya/phantomjs
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AAF3uhDpL0Z_AXCGeZmMbRsjd-9SfDL7ks5p2syPgaJpZM4IEzfc
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: CjWC6vIGLGo0uWsXQ9th+U0br113QN4BhNL8BXzrk7sb/evJ7Ch9xzbjTwXKVLTyszm0f74oPkGsic
2jN6ZQxeHAVkSq5BhBjvfR0XUpSH91NC+hFE07zm1Sz/bivI7LQAeAECAZVgPtpaSC8t0lUdqMtS76
SxhsdxWKtaaiZz4ohM9ABoEZTPcbzh0Kj1a9d5rku3iz0/+L+1GH4Jn/XVhrR22C8REvn2TU1qPEjR
c=
X-Spam-Status: No, score=-0.2
X-Spam-Score: -1
X-Spam-Bar: /
X-Ham-Report: Spam detection software, running on the system "server191.web-hosting.com",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  Closed #14193. --- You are receiving this because you are
subscribed to this thread. Reply to this email directly or view it on GitHub:
https://github.com/ariya/phantomjs/issues/14193#event-623252821 [...]
Content analysis details:   (-0.2 points, 5.0 required)
pts rule name              description

0.0 URIBL_BLOCKED          ADMINISTRATOR NOTICE: The query to URIBL was blocked.
See
http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block
for more information.
[URIs: github.com]
-0.7 RCVD_IN_DNSWL_LOW      RBL: Sender listed at http://www.dnswl.org/, low
trust
[<IP_ADDRESS> listed in list.dnswl.org]
-0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)
[<IP_ADDRESS> listed in wl.mailspike.net]
-1.0 RP_MATCHES_RCVD        Envelope sender domain matches handover relay domain
1.6 HTML_IMAGE_ONLY_12     BODY: HTML: images with 800-1200 bytes of words
0.0 HTML_MESSAGE           BODY: HTML included in message
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders
X-Spam-Flag: NO
----==_mimepart_570c268f8001b_40be3fc57f4ed2a0416795
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #14193.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ariya/phantomjs/issues/14193#event-623252821
----==_mimepart_570c268f8001b_40be3fc57f4ed2a0416795
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #14193.
—You are receiving this because you are subscribed to this thread.Reply to this email directly or view it on GitHub

----==_mimepart_570c268f8001b_40be3fc57f4ed2a0416795--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain castonguay.org has exceeded the max emails per hour (237/200 (118%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o4.sgmail.github.com ([<IP_ADDRESS>]:1375)
by server191.web-hosting.com with esmtps (TLSv1.2:DHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.86_1)
(envelope-from<EMAIL_ADDRESS>id 1apkRO-003O0y-Bh
for<EMAIL_ADDRESS>Mon, 11 Apr 2016 18:36:19 -0400
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=github.com;
h=from:reply-to:to:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=GrxSp8qNMGF0Zu2+bynsN6iy6sU=; b=pJln7SVZk1D1l/cV
e5fgC0LsdCtyUY056PBhEcssFN6bQv3eJ7Mcn/qBdBoqUXlm/Bg8zJLWMKekdO0R
TWQbMwHRevQXydPNujvcW/+N6eDyDTihkoAYD1+qt+TXJhDC1GQvKKamaDvCgunQ
pnezw35GxD9OYWiDxYIzqtQFBHI=
Received: by filter0458p1mdw1.sendgrid.net with SMTP id filter0458p1mdw1.18793.570C268F5D
2016-04-11 22:34:55.851849907 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0003p1iad1.sendgrid.net (SG) with ESMTP id --Ge2DfqR4OTdXFbTNnn_w
for<EMAIL_ADDRESS>Mon, 11 Apr 2016 22:34:55.645 +0000 (UTC)
Date: Mon, 11 Apr 2016 15:34:55 -0700
From: hrushikesh198<EMAIL_ADDRESS>Reply-To: ariya/phantomjs<EMAIL_ADDRESS>To: ariya/phantomjs<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [ariya/phantomjs] phantomjs-2.1.1-linux-x86_64 proxy
authentication not working  (#14193)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_570c268f7d331_40ab3fc57f4ed2a0479062";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: hrushikesh198
X-GitHub-Recipient: dREc
List-ID: ariya/phantomjs <phantomjs.ariya.github.com>
List-Archive: https://github.com/ariya/phantomjs
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AAF3uhDpL0Z_AXCGeZmMbRsjd-9SfDL7ks5p2syPgaJpZM4IEzfc
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: CjWC6vIGLGo0uWsXQ9th+U0br113QN4BhNL8BXzrk7ue+4YbT+4QQcOSkr1Jyr0i/wx+7H+ZAsNNhs
bsxWkk4bibPt9e8hkzObGHnm/2bjHpsmS1VKVJq03+sC6whfq7Y2KJjaQWwXEc6oo67dplWxCzYHvj
4W0R92J+XSkr4aK4Yj5VjfyEUjAZOePvzJQjOk/hlSYCQ1zipmfz5FJWE/wLUoqm1jqscsKTcbotv6
Y=
X-Spam-Status: No, score=-1.8
X-Spam-Score: -17
X-Spam-Bar: -
X-Ham-Report: Spam detection software, running on the system "server191.web-hosting.com",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  I was using a http proxy. My bad. Tried with some other proxies.
It's working fine. Closing the issue. --- You are receiving this because
you are subscribed to this thread. Reply to this email directly or view it
on GitHub: https://github.com/ariya/phantomjs/issues/14193#issuecomment-208594370
[...]
Content analysis details:   (-1.8 points, 5.0 required)
pts rule name              description

0.0 URIBL_BLOCKED          ADMINISTRATOR NOTICE: The query to URIBL was blocked.
See
http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block
for more information.
[URIs: github.com]
-1.0 RP_MATCHES_RCVD        Envelope sender domain matches handover relay domain
-2.3 RCVD_IN_DNSWL_MED      RBL: Sender listed at http://www.dnswl.org/, medium
trust
[<IP_ADDRESS> listed in list.dnswl.org]
-0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)
[<IP_ADDRESS> listed in wl.mailspike.net]
1.6 HTML_IMAGE_ONLY_12     BODY: HTML: images with 800-1200 bytes of words
0.0 HTML_MESSAGE           BODY: HTML included in message
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders
X-Spam-Flag: NO
----==_mimepart_570c268f7d331_40ab3fc57f4ed2a0479062
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
I was using a http proxy. My bad. Tried with some other proxies. It's working fine. Closing the issue.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ariya/phantomjs/issues/14193#issuecomment-208594370
----==_mimepart_570c268f7d331_40ab3fc57f4ed2a0479062
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
I was using a http proxy. My bad. Tried with some other proxies. It's working fine. Closing the issue.
—You are receiving this because you are subscribed to this thread.Reply to this email directly or view it on GitHub

----==_mimepart_570c268f7d331_40ab3fc57f4ed2a0479062--


Agent configuration protocol crash

Agent configuration protocol causes crash in some circumstances. This appears to be somehow triggered by a retrieval of archived authentication requests.
Server output:
python3 lysys.py
STARTING THREADED DNS SERVER ...
STARTING API AS SUBPROCESSES ...
BUILD SERVER IS ALREADY RUNNING
 * Serving Flask app "portal" (lazy loading)
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
 * Running on https://<IP_ADDRESS>:56114/ (Press CTRL+C to quit)

<bound method objStructure.auth of <lycanthropy.daemon.messager.objStructure object at 0x7fae8dd10e10>>
{
    "error": "msgID 0swS not in message table, backing up from archive"
}

Agent output:
java -jar buildstub-0.1.jar
Exception in thread "main" java.lang.reflect.InvocationTargetException
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:566)
        at agent.Netw.send(Netw.java:77)
        at agent.Core.setup(Core.java:38)
        at agent.Main.main(Main.java:22)
Caused by: java.lang.NullPointerException
        at agent.Conf.retrieve(Conf.java:31)
        at agent.Conf.dispatch(Conf.java:87)
        ... 7 more

Crash is caused by an authentication failure during the makeAuthSuccess call performed by conf, seemingly as a result of whatever is going wrong with the auth protocol (most likely a retransmission). Specifically, makeAuthSuccess returns a status of 1, indicating that the cookie does not derive from the cookiedough for the agent's session.

Solution has been implemented and is being tested.


"COPY /blah/blub /path" does not work same way as docker

The COPY command does not work for the following case - source files are not interpreted as relativ to the current workspace as stated in the original docker refence:
https://docs.docker.com/engine/reference/#copy

Multiple <src> resources may be specified but the paths of files and directories will be interpreted as relative to the source of the context of the build.

Example workspace directory:
Dockerfile
firstdir/seconddir/dir-a/file-a
firstdir/seconddir/dir-b/file-b

Dockerfile content:
FROM alpine:3.10
COPY /firstdir/seconddir /var
RUN ls -la /var && ls -la /var/firstdir
RUN ls -la /var/dir-a

The original docker builder copies the content of the firstdir/seconddir to the image, because all paths are relative to the context. imagebuilder failes with an error message saying stat seconddir: no such file or directory
Changing the COPY line to the following (removing first slash)
COPY firstdir/seconddir /var

Imagebuilder works as expected.

Another case where imagebuilder handles stuff diffent is the following. Same workspace as before, but different Dockerfile - now with trailing slash on src directory
 FROM alpine:3.10
COPY /firstdir/seconddir/ /var
RUN ls -la /var && ls -la /var/firstdir
RUN ls -la /var/dir-a

Running imagebuilder gives no error on build-time but copies files in a different way than docker
Original docker builder gives the follwoing files inside the image:
/var/dir-a/file-a
/var/dir-b/file/b

But imagebuilder creates the following files:
/var/firstdir/seconddir/dir-a/file-a
/var/firstdir/seconddir/dir-b/file-b

It does make sense to make it compatible - as its even written inside the docs. And this is logical extension compared to the (working inside imagebuilder)
COPY / /yadda

Using /  to copy everything from base folder - why should /firstdir not work to copy everything below this directory?
Regarding PR - i would if it where not GO. Tried writing stuff with it but its definitly not my language...

/lifecycle frozen


Compensate for scheduled play

Addressing #2691 - Allows getCurrentTime() to compensate for the scheduled player time

Could you add a test for this?

Could you add a test for this?

It seems the tests don't schedule the player. I get the output false for audio.isSilent when running the tests with a scheduled player.
func testScheduleTime() {
        guard let url = Bundle.module.url(forResource: "TestResources/12345", withExtension: "wav") else {
            XCTFail("Didn't get test file")
            return
        }
        let engine = AudioEngine()
        let player = AudioPlayer()
        engine.output = player
        player.isLooping = true

        let audio = engine.startTest(totalDuration: 2.0)

        do {
            try player.load(url: url, buffered: true)
        } catch let error as NSError {
            Log(error, type: .error)
            XCTFail(error.description)
        }
        var scheduledTime = 0.0
        player.play(from: 0.0, to: player.duration, at: AVAudioTime.now() + 2, completionCallbackType: .dataPlayedBack)
        audio.append(engine.render(duration: 0.1))
        print(player.getCurrentTime())
        print(audio.isSilent)
        audio.append(engine.render(duration: 0.1))
        print(player.getCurrentTime())
        print(audio.isSilent)
        audio.append(engine.render(duration: 0.1))
        print(player.getCurrentTime())
        print(audio.isSilent)
        audio.append(engine.render(duration: 0.1))
        print(player.getCurrentTime())
        print(audio.isSilent)

        let currentTime = player.getCurrentTime()
        //XCTAssertEqual(currentTime, 0.0)

        //testMD5(audio)
    }

Seems as if the player ignores whenTime during the tests and plays the audio right away. Is there a way I could write a test which waits for the player's schedule?

I'm going to put this on the back-burner and come back to it. I promise I will, but I think I found the jackpot and want to fix some of those realtime player tests. Having those work will help me write a test for this--in fact I think I saw something similar in there.


Ordering order being ignored

Hi,
when ordering with 2 and more values order of values is bound to the definition of Order object thus negating user defined order.
Am I mistaken in assuming these two should not be equivalent?
query {
  milestoneList(
    order: {
      project: {
        name: ASC
      }
      name: ASC
    }
  ) {
    name
    project {
      name
    }
  }
}

query {
  milestoneList(
    order: {
      name: ASC
      project: {
        name: ASC
      }
    }
  ) {
    name
    project {
      name
    }
  }
}

The result of both of these queries is the same, seems like the actual order is determined via schema definition and not input.

According to spec arguments are correctly given unordered.
So sould then be the implementation of ordering in this lib changed? Take current object implementation, put it in list and allow only one leaf value per object in said list?

@Kitefiko yeah, we need a new ordering implementation.
I'm actually wanting to implement this for some time (will probably try this weekend) to implement a new ordering which receives a list of objects, which can maintain the order they are given.
With that we can deprecate the order, which is also not a good name since it usually clashes with the name order when used in a different context (e.g. order from an ecommerce project)

@bellini666 is this on your radar? have you given it more thought?

Hi @g-as ,
Yes it is! I'm planning to work on this as soon as I can, but I'm a little busy those last weeks.
I would say that I'll be able to take a look at this by the end of october, probably.

Great! Just wanted to make sure it was still in the works. Keep up the good work!

I found a way to fix this in #409 . Maybe with this we don't need to refactor the ordering code anymore?
The refactoring discussion is going to continue on #399

Hi,
i just tried to use the adjusted ordering interface from the GraphiQL interface and it works very well, thank you for that!
Unfortunately, if I want to use a JS GraphQL client like the Apollo Client, the ordering cannot be employed in the way it was ment to be, since Apollo Client or JS itself, I am not yet sure about that, seems to alphabetically order the object keys before invoking the query, so querying for instance
{   someQuery(ordering: {b: 'ASC', a: 'DESC'}) {    ... }
will always be sent as
{   someQuery(ordering: {a: 'DESC', b: 'ASC'}) {    ... }
due alphabetical sorting when passing 'ordering' as variable. Can you confirm this behavior or is it actually me doing something wrong?

due to alphabetical sorting when passing 'ordering' as variable. Can you confirm this behavior or is it actually me doing something wrong?

I'm not sure on how apollo itself behaves, but you can try to inspect your network call to see if it is indeed apollo that is modifying the order

Hi, I found that f.name.value at L110 is camelCase value while at L63 f.name is underscore value which makes sort_key function always return 0. Is there something wrong with my code?
https://github.com/strawberry-graphql/strawberry-graphql-django/blob/c90a7907913188cb5b70ee74e3fbe4a1ead7c8a2/strawberry_django/ordering.py#L110
https://github.com/strawberry-graphql/strawberry-graphql-django/blob/c90a7907913188cb5b70ee74e3fbe4a1ead7c8a2/strawberry_django/ordering.py#L62-L65


feat(dev, node-server): support a hook to access Node.js server instance

🔗 Linked issue
solve https://github.com/unjs/nitro/issues/2040
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ]  🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Hi 👋 this PR can be possible to enable WebSocket and Web Server to share the same port without the need for reverse proxy.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

When will this PR be added to the main branch?

I propose to expand these events
const server =
  cert && key
    ? new HttpsServer({ key, cert }, toNodeListener(nitroApp.h3App))
    : new HttpServer(toNodeListener(nitroApp.h3App));

// others code
nitroApp.hooks.callHook("node.server.init", server);

server.on('upgrade', () => {
   nitroApp.hooks.callHook("node.server.upgrade", server, req: IncomingMessage);
})

server.on('listening', () => {
   nitroApp.hooks.callHook("node.server.listening", server);
})

So much more stable for more control. For example, a handshake for WebSocket and the event that the server has started. As well as initialization which will make it possible to register more events for advanced users

When will this PR be added to the main branch?

I'm so sorry for the delay in progressing the PR due to my unfamiliarity with GitHub functionalities; I'm actively learning and seeking guidance to contribute effectively to the project.

Sorry for delay on this. I guess now that we have official WebSocket support, it should be less priority but I will defneitly investigate how to make it an extendable addon before introducing 👍🏼


id="nav-tree" not found in html

When I build html, I get

without sidbar
That is my Doxyfile
DOXYFILE_ENCODING      = UTF-8
PROJECT_NAME           = @DOXYGEN_PROJECT_NAME@
PROJECT_NUMBER         = @DOXYGEN_PROJECT_VERSION@
PROJECT_BRIEF          =
PROJECT_LOGO
OUTPUT_DIRECTORY       = @DOXYGEN_OUTPUT_DIRECTORY@
CREATE_SUBDIRS         = NO
ALLOW_UNICODE_NAMES    = NO
OUTPUT_LANGUAGE        = English
BRIEF_MEMBER_DESC      = YES
REPEAT_BRIEF           = YES
ABBREVIATE_BRIEF       = "The $name class" \
                         "The $name widget" \
                         "The $name file" \
                         is \
                         provides \
                         specifies \
                         contains \
                         represents \
                         a \
                         an \
                         the
INLINE_INHERITED_MEMB  = NO
FULL_PATH_NAMES        = YES
STRIP_FROM_PATH        = @DOXYGEN_PROJECT_ROOT@
STRIP_FROM_INC_PATH    =
SHORT_NAMES            = NO
JAVADOC_AUTOBRIEF      = NO
JAVADOC_BANNER         = NO
QT_AUTOBRIEF           = NO
MULTILINE_CPP_IS_BRIEF = NO
PYTHON_DOCSTRING       = YES
INHERIT_DOCS           = YES
SEPARATE_MEMBER_PAGES  = NO
TAB_SIZE               = 4
ALIASES                =
OPTIMIZE_OUTPUT_FOR_C  = NO
OPTIMIZE_OUTPUT_JAVA   = NO
OPTIMIZE_FOR_FORTRAN   = NO
OPTIMIZE_OUTPUT_VHDL   = NO
OPTIMIZE_OUTPUT_SLICE  = NO
EXTENSION_MAPPING      =
MARKDOWN_SUPPORT       = YES
TOC_INCLUDE_HEADINGS   = 5
AUTOLINK_SUPPORT       = YES
BUILTIN_STL_SUPPORT    = NO
CPP_CLI_SUPPORT        = NO
SIP_SUPPORT            = NO
IDL_PROPERTY_SUPPORT   = YES
DISTRIBUTE_GROUP_DOC   = NO
GROUP_NESTED_COMPOUNDS = NO
SUBGROUPING            = YES
INLINE_GROUPED_CLASSES = NO
INLINE_SIMPLE_STRUCTS  = NO
TYPEDEF_HIDES_STRUCT   = NO
LOOKUP_CACHE_SIZE      = 0
NUM_PROC_THREADS       = 1
EXTRACT_ALL            = YES
EXTRACT_PRIVATE        = NO
EXTRACT_PRIV_VIRTUAL   = NO
EXTRACT_PACKAGE        = NO
EXTRACT_STATIC         = NO
EXTRACT_LOCAL_CLASSES  = YES
EXTRACT_LOCAL_METHODS  = NO
EXTRACT_ANON_NSPACES   = NO
RESOLVE_UNNAMED_PARAMS = YES
HIDE_UNDOC_MEMBERS     = NO
HIDE_UNDOC_CLASSES     = NO
HIDE_FRIEND_COMPOUNDS  = NO
HIDE_IN_BODY_DOCS      = NO
INTERNAL_DOCS          = NO
CASE_SENSE_NAMES       = NO
HIDE_SCOPE_NAMES       = NO
HIDE_COMPOUND_REFERENCE= NO
SHOW_HEADERFILE        = YES
SHOW_INCLUDE_FILES     = YES
SHOW_GROUPED_MEMB_INC  = NO
FORCE_LOCAL_INCLUDES   = NO
INLINE_INFO            = YES
SORT_MEMBER_DOCS       = YES
SORT_BRIEF_DOCS        = NO
SORT_MEMBERS_CTORS_1ST = NO
SORT_GROUP_NAMES       = NO
SORT_BY_SCOPE_NAME     = NO
STRICT_PROTO_MATCHING  = NO
GENERATE_TODOLIST      = YES
GENERATE_TESTLIST      = YES
GENERATE_BUGLIST       = YES
GENERATE_DEPRECATEDLIST= YES
ENABLED_SECTIONS       =
MAX_INITIALIZER_LINES  = 30
SHOW_USED_FILES        = YES
SHOW_FILES             = YES
SHOW_NAMESPACES        = YES
FILE_VERSION_FILTER    =
LAYOUT_FILE            =
CITE_BIB_FILES         =
QUIET                  = NO
WARNINGS               = YES
WARN_IF_UNDOCUMENTED   = YES
WARN_IF_DOC_ERROR      = YES
WARN_IF_INCOMPLETE_DOC = YES
WARN_NO_PARAMDOC       = NO
WARN_AS_ERROR          = NO
WARN_FORMAT            = "$file:$line: $text"
WARN_LOGFILE           =
INPUT                  =<EMAIL_ADDRESS>INPUT                  += @DOXYGEN_PROJECT_ROOT@/src
INPUT                  += @DOXYGEN_PROJECT_ROOT@/src_acis
INPUT                  += @DOXYGEN_PROJECT_ROOT@/test
INPUT                  += @DOXYGEN_PROJECT_ROOT@/include
INPUT_ENCODING         = UTF-8
FILE_PATTERNS          = *.c \
                         *.cc \
                         *.cxx \
                         *.cpp \
                         *.c++ \
                         *.java \
                         *.ii \
                         *.ixx \
                         *.ipp \
                         *.i++ \
                         *.inl \
                         *.idl \
                         *.ddl \
                         *.odl \
                         *.h \
                         *.hh \
                         *.hxx \
                         *.hpp \
                         *.h++ \
                         *.l \
                         *.cs \
                         *.d \
                         *.php \
                         *.php4 \
                         *.php5 \
                         *.phtml \
                         *.inc \
                         *.m \
                         *.markdown \
                         *.md \
                         *.mm \
                         *.dox \
                         *.py \
                         *.pyw \
                         *.f90 \
                         *.f95 \
                         *.f03 \
                         *.f08 \
                         *.f18 \
                         *.f \
                         *.for \
                         *.vhd \
                         *.vhdl \
                         *.ucf \
                         *.qsf \
                         *.ice
RECURSIVE              = YES
EXCLUDE                = @DOXYGEN_PROJECT_ROOT@/include/Eigen @DOXYGEN_PROJECT_ROOT@/include/doctest
EXCLUDE_SYMLINKS       = NO
EXCLUDE_PATTERNS       =
EXCLUDE_SYMBOLS        =
EXAMPLE_PATH           =
EXAMPLE_PATTERNS       = *
EXAMPLE_RECURSIVE      = NO
IMAGE_PATH             =
INPUT_FILTER           =
FILTER_PATTERNS        =
FILTER_SOURCE_FILES    = NO
FILTER_SOURCE_PATTERNS =
USE_MDFILE_AS_MAINPAGE =<EMAIL_ADDRESS>SOURCE_BROWSER         = NO
INLINE_SOURCES         = NO
STRIP_CODE_COMMENTS    = YES
REFERENCED_BY_RELATION = NO
REFERENCES_RELATION    = NO
REFERENCES_LINK_SOURCE = YES
SOURCE_TOOLTIPS        = YES
USE_HTAGS              = NO
VERBATIM_HEADERS       = YES
CLANG_ASSISTED_PARSING = NO
CLANG_ADD_INC_PATHS    = YES
CLANG_OPTIONS          =
CLANG_DATABASE_PATH    =
ALPHABETICAL_INDEX     = YES
IGNORE_PREFIX          =
GENERATE_HTML          = YES
HTML_OUTPUT            = html
HTML_FILE_EXTENSION    = .html
HTML_HEADER            =<EMAIL_ADDRESS>HTML_FOOTER            =
HTML_STYLESHEET        =
HTML_EXTRA_STYLESHEET  =<EMAIL_ADDRESS>\
<EMAIL_ADDRESS>\
<EMAIL_ADDRESS>HTML_EXTRA_FILES       =<EMAIL_ADDRESS>HTML_EXTRA_FILES       +=<EMAIL_ADDRESS>HTML_EXTRA_FILES       +=<EMAIL_ADDRESS>HTML_EXTRA_FILES       +=<EMAIL_ADDRESS>HTML_COLORSTYLE_HUE    = 220
HTML_COLORSTYLE_SAT    = 100
HTML_COLORSTYLE_GAMMA  = 80
HTML_TIMESTAMP         = NO
HTML_DYNAMIC_MENUS     = NO
HTML_DYNAMIC_SECTIONS  = NO
HTML_INDEX_NUM_ENTRIES = 100
GENERATE_DOCSET        = NO
DOCSET_FEEDNAME        = "Doxygen generated docs"
DOCSET_BUNDLE_ID       = org.doxygen.Project
DOCSET_PUBLISHER_ID    = org.doxygen.Publisher
DOCSET_PUBLISHER_NAME  = Publisher
GENERATE_HTMLHELP      = YES
CHM_FILE               =
HHC_LOCATION           =
GENERATE_CHI           = NO
CHM_INDEX_ENCODING     =
BINARY_TOC             = NO
TOC_EXPAND             = NO
GENERATE_QHP           = NO
QCH_FILE               =
QHP_NAMESPACE          = org.doxygen.Project
QHP_VIRTUAL_FOLDER     = doc
QHP_CUST_FILTER_NAME   =
QHP_CUST_FILTER_ATTRS  =
QHP_SECT_FILTER_ATTRS  =
QHG_LOCATION           =
GENERATE_ECLIPSEHELP   = NO
ECLIPSE_DOC_ID         = org.doxygen.Project
DISABLE_INDEX          = NO
GENERATE_TREEVIEW      = YES
FULL_SIDEBAR           = NO
ENUM_VALUES_PER_LINE   = 4
TREEVIEW_WIDTH         = 250
EXT_LINKS_IN_WINDOW    = NO
HTML_FORMULA_FORMAT    = png
FORMULA_FONTSIZE       = 10
FORMULA_TRANSPARENT    = YES
FORMULA_MACROFILE      =
USE_MATHJAX            = NO
MATHJAX_VERSION        = MathJax_2
MATHJAX_FORMAT         = HTML-CSS
MATHJAX_RELPATH        =
MATHJAX_EXTENSIONS     =
MATHJAX_CODEFILE       =
SEARCHENGINE           = NO
SERVER_BASED_SEARCH    = NO
EXTERNAL_SEARCH        = NO
SEARCHENGINE_URL       =
SEARCHDATA_FILE        = searchdata.xml
EXTERNAL_SEARCH_ID     =
EXTRA_SEARCH_MAPPINGS  =
GENERATE_LATEX         = NO
LATEX_OUTPUT           = latex
LATEX_CMD_NAME         =
MAKEINDEX_CMD_NAME     = makeindex
LATEX_MAKEINDEX_CMD    = makeindex
COMPACT_LATEX          = NO
PAPER_TYPE             = a4
EXTRA_PACKAGES         =
LATEX_HEADER           =
LATEX_FOOTER           =
LATEX_EXTRA_STYLESHEET =
LATEX_EXTRA_FILES      =
PDF_HYPERLINKS         = YES
USE_PDFLATEX           = YES
LATEX_BATCHMODE        = NO
LATEX_HIDE_INDICES     = NO
LATEX_BIB_STYLE        = plain
LATEX_TIMESTAMP        = NO
LATEX_EMOJI_DIRECTORY  =
GENERATE_RTF           = NO
RTF_OUTPUT             = rtf
COMPACT_RTF            = NO
RTF_HYPERLINKS         = NO
RTF_STYLESHEET_FILE    =
RTF_EXTENSIONS_FILE    =
GENERATE_MAN           = NO
MAN_OUTPUT             = man
MAN_EXTENSION          = .3
MAN_SUBDIR             =
MAN_LINKS              = NO
GENERATE_XML           = NO
XML_OUTPUT             = xml
XML_PROGRAMLISTING     = YES
XML_NS_MEMB_FILE_SCOPE = NO
GENERATE_DOCBOOK       = NO
DOCBOOK_OUTPUT         = docbook
GENERATE_AUTOGEN_DEF   = NO
GENERATE_PERLMOD       = NO
PERLMOD_LATEX          = NO
PERLMOD_PRETTY         = YES
PERLMOD_MAKEVAR_PREFIX =
ENABLE_PREPROCESSING   = YES
MACRO_EXPANSION        = NO
EXPAND_ONLY_PREDEF     = NO
SEARCH_INCLUDES        = YES
INCLUDE_PATH           =
INCLUDE_FILE_PATTERNS  =
PREDEFINED             =
EXPAND_AS_DEFINED      =
SKIP_FUNCTION_MACROS   = YES
TAGFILES               =
GENERATE_TAGFILE       =
ALLEXTERNALS           = NO
EXTERNAL_GROUPS        = YES
EXTERNAL_PAGES         = YES
CLASS_DIAGRAMS         = YES
DIA_PATH               =
HIDE_UNDOC_RELATIONS   = YES
HAVE_DOT               = YES
DOT_NUM_THREADS        = 0
DOT_FONTNAME           = Helvetica
DOT_FONTSIZE           = 10
DOT_FONTPATH           =
CLASS_GRAPH            = YES
COLLABORATION_GRAPH    = YES
GROUP_GRAPHS           = YES
UML_LOOK               = YES
UML_LIMIT_NUM_FIELDS   = 10
DOT_UML_DETAILS        = YES
DOT_WRAP_THRESHOLD     = 17
TEMPLATE_RELATIONS     = NO
INCLUDE_GRAPH          = YES
INCLUDED_BY_GRAPH      = YES
CALL_GRAPH             = YES
CALLER_GRAPH           = YES
GRAPHICAL_HIERARCHY    = YES
DIRECTORY_GRAPH        = YES
DOT_IMAGE_FORMAT       = svg
INTERACTIVE_SVG        = YES
DOT_PATH               =
DOTFILE_DIRS           =
MSCFILE_DIRS           =
DIAFILE_DIRS           =
PLANTUML_JAR_PATH      =
PLANTUML_CFG_FILE      =
PLANTUML_INCLUDE_PATH  =
DOT_GRAPH_MAX_NODES    = 50
MAX_DOT_GRAPH_DEPTH    = 0
DOT_TRANSPARENT        = YES
DOT_MULTI_TARGETS      = NO
GENERATE_LEGEND        = YES
DOT_CLEANUP            = YES
M_SHOW_UNDOCUMENTED    = YES
COLLABORATION_GRAPH    = YES

Do I mistake on some options? thank you.

GENERATE_TREEVIEW      = YES is set, so I agree with you that there should be a sidebar. Sorry for the late response, did you figure out what the issue with your configuration was?

GENERATE_TREEVIEW = YES is set, so I agree with you that there should be a sidebar. Sorry for the late response, did you figure out what the issue with your configuration was?

Sorry, I forget where this code from. Thank you for your attention.

I have the same issue, and idon't know how to fix it.
I have set GENERATE_TREEVIEW = YES , but i can't see any sidebar.


Research Spike: SuperchainRPC

@pharger can you add detail to this issue with what exactly you'd like to see as an outcome of this research spike? Thanks!


Example notebook for using the WFS to visualize fire perimeters

This is a notebook that uses a draft version of the WFS to map fire perimeters.

Nice work Tess 🥳
A couple minor things:

Since we don't have our new shiny conda-store shared environment thingy up yet (it's in the PI for next quarter) that means we're at the mercy of only being able to run packages that come in the base image. And owslib isn't installed.  The simplest and best workaround (though pip won't work for all conda packages) is to add a first notebook cell that pip installs it in the conda environment. Could you add this first line to the notebook please?

import sys
!{sys.executable} -m pip install OWSLib==0.28.1

After this merges and once I figure out who has authority to pick a DNS name for the WFS we'll need to change the domain. This shouldn't keep us from merging it. Once that changes I'll put in a follow-up PR 👍

I tested this out by creating a link from nbgitpuller for SCME
https://daskhub.veda.smce.nasa.gov/hub/user-redirect/git-pull?repo=https%3A%2F%2Fgithub.com%2Fmccabete%2Fveda-docs&urlpath=tree%2Fveda-docs%2F%2Fexample-notebooks%2FWFS_workflow_example.ipynb&branch=fire_perimeter_wfs

Done! Thanks for the feedback!


ES6 Multiline Template String SQL Escaping helper

Inspired by the node postgres SQL helper method I adapted the ES6 Multiline Template String helper. See here: https://github.com/Phisherman/node-mysql-es6-query

Such a simple function with a significant increase in readability.
Has including such a function as a part of the library been considered?

@paldepind Maybe it will if you write a PR. :)

Is that library published to npm anywhere?

I didn't publish it.


darwin.builder: init

Fixes https://github.com/NixOS/nixpkgs/issues/108984
This originates from:
https://github.com/Gabriella439/macos-builder
… which in turn originates from:
https://github.com/YorikSar/nixos-vm-on-macos
Once this change is built and cached, the end user only needs to run:
$ nix run nixpkgs#darwin.builder

… to create a Linux builder on macOS
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thanks a lot for getting this in a frinedly state :heart:

I'm going to merge this because it improves the current state a lot and further improvements can be done once people start to use it.

@domenkozar there is no time pressure here at all and new features should not be merged with barely any review just because people desperately want it.

Did the review comments myself in #207486

@domenkozar there is no time pressure here at all and new features should not be merged with barely any review just because people desperately want it.
Doing major redesigns of new features straight after they where rushed to be merged is suboptimal for anyone that started to use the feature already.

It's not about time pressure, it's about getting this work to the devs to test and moving the bikeshedding into a step that can happen in parallel.

crucially, getting hydra to put it in cache.nixos.org

Seems ofborg is now counting darwin.builder as a rebuild in every PR?
https://github.com/NixOS/nixpkgs/pull/206032
https://gist.github.com/GrahamcOfBorg/60663b75aa4703f2cecf3f7d48dedcd5
https://github.com/NixOS/nixpkgs/pull/207583
https://gist.github.com/GrahamcOfBorg/a8dabad13f74c287935a5bc1f0cdf730

@zowoq: My guess is that one of the system options might be changing on each build, which is causing darwin.builder to be rebuilt
However, I tried to reproduce locally by making trivial changes to nixpkgs and I don't get any changes to the .#darwin.builder derivation.  Is there some way I can locally reproduce the difference that ofborg is seeing?
Either way, I'd also be fine with temporarily disabling darwin.builder until the problem can be resolved.

Here's the PR to disable the build for now if people need this to be reverted: https://github.com/NixOS/nixpkgs/pull/207776

not sure if disabling the build is needed since https://github.com/Mic92/nixpkgs-review/pull/291 is merged and nixpkgs-review users will not have to rebuild darwin.builder anymore, the only downside after nixpkgs-review is updated afaik would be the github labels which imo is not a big deal

github labels which imo is not a big deal

ofborg telling contributors that a zero rebuild PR causes one rebuild on each darwin platforms is rather confusing.

Just to clarify: is darwin.builder being rebuilt on these unrelated PRs or is it a bug in ofborg?

I can reproduce this locally:
$ nix show-derivation -f . darwin.builder --system aarch64-darwin > a
$ git commit --allow-empty -m test
$ nix show-derivation -f . darwin.builder --system aarch64-darwin > b
$ diff -u a b
--- a   2022-12-26 23:11:31.175857599 +0100
+++ b   2022-12-26 23:11:53.008634417 +0100
@@ -1,5 +1,5 @@
 {
-  "/nix/store/9fgpywcl1qgk5yrwwq9b829pncw4jrhi-create-builder.drv": {
+  "/nix/store/qj88p4c8b9nw8z6cg4cf425l5y4d1hah-create-builder.drv": {
     "args": [
       "-e",
       "/nix/store/9krlzvny65gdc8s7kpb6lkx8cd02c25b-default-builder.sh"
@@ -34,7 +34,7 @@
       "mesonFlags": "",
       "name": "create-builder",
       "nativeBuildInputs": "",
-      "out": "/nix/store/5ggib8k7z9k6ryzpnhijah8apsc071la-create-builder",
+      "out": "/nix/store/cxb5sjl4klf0l8b0r3qh5z4r1a9ldzqa-create-builder",
       "outputs": "out",
       "passAsFile": "buildCommand text",
       "patches": "",
@@ -44,13 +44,13 @@
       "stdenv": "/nix/store/s5rvpdf77v47cqy1n56wr5xlj3b7vzmm-stdenv-darwin",
       "strictDeps": "",
       "system": "aarch64-darwin",
-      "text": "#!/nix/store/x380x9d7gqh4ig9h50a7wx08p0s6622f-bash-5.1-p16/bin/bash\nKEYS=\"${KEYS:-./keys}\"\n/nix/store/npcq66nqv2wpzp3ma5s63vafnnivphxb-coreutils-9.1/bin/mkdir --parent \"${KEYS}\"\nPRIVATE_KEY=\"${KEYS}/builder_ed25519\"\nPUBLIC_KEY=\"${PRIVATE_KEY}.pub\"\nif [ ! -e \"${PRIVATE_KEY}\" ] || [ ! -e \"${PUBLIC_KEY}\" ]; then\n    /nix/store/npcq66nqv2wpzp3ma5s63vafnnivphxb-coreutils-9.1/bin/rm --force -- \"${PRIVATE_KEY}\" \"${PUBLIC_KEY}\"\n    /nix/store/kgp4br70j8w4k4lgsv8sk22hb51km36s-openssh-9.1p1/bin/ssh-keygen -q -f \"${PRIVATE_KEY}\" -t ed25519 -N \"\" -C 'builder@localhost'\nfi\nif ! /nix/store/2m2jha21jq3f51g0rd42y385v5vfkgih-diffutils-3.8/bin/cmp \"${PUBLIC_KEY}\" /etc/nix/builder_ed25519.pub; then\n  (set -x; sudo --reset-timestamp /nix/store/grd27v1gb7s0hqq7wrnqw88m60mlr8wk-install-credentials \"${KEYS}\")\nfi\nKEYS=\"$(nix-store --add \"$KEYS\")\" /nix/store/pd3vpvsg7sqri0pr9qpjr38g61wik1rz-nixos-vm/bin/run-nixos-vm\n\n"
+      "text": "#!/nix/store/x380x9d7gqh4ig9h50a7wx08p0s6622f-bash-5.1-p16/bin/bash\nKEYS=\"${KEYS:-./keys}\"\n/nix/store/npcq66nqv2wpzp3ma5s63vafnnivphxb-coreutils-9.1/bin/mkdir --parent \"${KEYS}\"\nPRIVATE_KEY=\"${KEYS}/builder_ed25519\"\nPUBLIC_KEY=\"${PRIVATE_KEY}.pub\"\nif [ ! -e \"${PRIVATE_KEY}\" ] || [ ! -e \"${PUBLIC_KEY}\" ]; then\n    /nix/store/npcq66nqv2wpzp3ma5s63vafnnivphxb-coreutils-9.1/bin/rm --force -- \"${PRIVATE_KEY}\" \"${PUBLIC_KEY}\"\n    /nix/store/kgp4br70j8w4k4lgsv8sk22hb51km36s-openssh-9.1p1/bin/ssh-keygen -q -f \"${PRIVATE_KEY}\" -t ed25519 -N \"\" -C 'builder@localhost'\nfi\nif ! /nix/store/2m2jha21jq3f51g0rd42y385v5vfkgih-diffutils-3.8/bin/cmp \"${PUBLIC_KEY}\" /etc/nix/builder_ed25519.pub; then\n  (set -x; sudo --reset-timestamp /nix/store/grd27v1gb7s0hqq7wrnqw88m60mlr8wk-install-credentials \"${KEYS}\")\nfi\nKEYS=\"$(nix-store --add \"$KEYS\")\" /nix/store/yphpd9i8w0vg0p3p70yck0vmmcdzq4fi-nixos-vm/bin/run-nixos-vm\n\n"
     },
     "inputDrvs": {
-      "/nix/store/7a0chkp8maxy1j20f3f937ikmxjkxisi-nixos-vm.drv": [
+      "/nix/store/c47v02plj8ziz3j0x13p7hdz8j0d7cxr-openssh-9.1p1.drv": [
         "out"
       ],
-      "/nix/store/c47v02plj8ziz3j0x13p7hdz8j0d7cxr-openssh-9.1p1.drv": [
+      "/nix/store/cn0bc111q4ha76ggmpfhywwpw411x3id-nixos-vm.drv": [
         "out"
       ],
       "/nix/store/g2npqs9fic2vvrxf117czf1j1936d63c-install-credentials.drv": [
@@ -74,7 +74,7 @@
     ],
     "outputs": {
       "out": {
-        "path": "/nix/store/5ggib8k7z9k6ryzpnhijah8apsc071la-create-builder"
+        "path": "/nix/store/cxb5sjl4klf0l8b0r3qh5z4r1a9ldzqa-create-builder"
       }
     },
     "system": "aarch64-darwin"

If it's git commits that's causing this, then I assume the darwin.builder needs to hard code a git revision, like in release.nix.

@raphaelr: Thank you!  I can reproduce now, too.  I think the reason I could not originally reproduce was that I was using nix path-info --derivation instead of nix-instantiate

This should fix the gratuitous rebuilds: https://github.com/NixOS/nixpkgs/pull/207902

@NiklasGollenstede: The only reason it says port 22 is required is because I thought it was a limitation of Nix's remote builder functionality (rather than a limitation of the VM).  Are you saying that there's a way for Nix to specify a non-default port for a remote builder in the builder spec?
I like the autologinUser idea, though, and created a pull request: https://github.com/NixOS/nixpkgs/pull/208772

Are you saying that there's a way for Nix to specify a non-default port for a remote builder in the builder spec?

That would be nice, but no. I am pretty sure that such an option would be defined in either of these two files:
https://github.com/NixOS/nix/blob/master/src/libstore/ssh-store.cc
https://github.com/NixOS/nix/blob/master/src/libstore/legacy-ssh-store.cc
(I don't really understand why Nix doesn't allow some way of just supplying verbatim options to openssh (at least with nix-build, it is simply calling ssh, which needs to be on the PATH that nix-build get's called with). Maybe to not commit to using the openssh CLI?)
What I was saying is that currently (I assume this is the default when not supplying virtualisation.forwardPorts.*.host.address) qemu is binding (and thus blocking) port 22 on all IPv4 (and v6?) addresses and interfaces, while a single (v4 or v6) address on only the loopback is enough for a local nix command to connect to. Even without a firewall, others in your network will no longer be able to use your VM, and other services can listen on TCP port 22, as long as they specify an interfacee/address other than the one used by the VM (or (implicitly) a wildcard that includes it, like "", *, <IP_ADDRESS>, etc, depending on the API).

@NiklasGollenstede: It turns out that an unprivileged qemu process attempting to bind a privileged port can bind to an host address of <IP_ADDRESS> but not <IP_ADDRESS>, which is pretty cursed:
https://developer.apple.com/forums/thread/674179
The good news is that you don't actually need to open up the firewall, so I put up a PR to document that: https://github.com/NixOS/nixpkgs/pull/208792

macOS: You can have all the addresses.
developer: That's cool. I'd like to have that one!
macOS: no.

There may be an implementation reason, but looking from the outside, that's incredibly weird behavior.
If macOS has some mechanism to prompt for firewall exceptions, (and it is now documented to deny that), then at least the security aspect of not binding all addresses is largely a mute point (assuming that on macOS users don't usually (or can't) disable the firewall).
While the port does not need to be "open" in the firewall, it still needs to be "free", as in "not bound to on and address by any process" (e.g. you can't have sshd running on its default port).

I thought it was a limitation of Nix's remote builder functionality (rather than a limitation of the VM)

I've configured Port in the ssh_config before. You may want to use the global client config, /etc/ssh/ssh_config, for this, or you may have to configure it for both root (the nix-daemon) and the user.

I've configured Port in the ssh_config before.

Right. That should work. (When I was researching this previously, modifying config files was not an option (and it is quite contrary to Nix's philosophy that it is required).)
To do this without also modifying /etc/hosts (or whatever it takes to locally define a host name on macOS), you can define the builder's address as 127.x.y.z (where x.y.z is anything other than 0.0.1 and then use that address in the ssh_config.


Make --audio record output audio?

As it is, the -a flag records my microphone's audio. I'd like it to record the actual output audio (like if I was playing a video/music/game, so that the audio of that can be heard). Is this possible with wf-recorder or do I have to create a kind of virtual mic that takes the output audio?

From the man page:
     -a, --audio [DEVICE]
             Starts recording the screen with audio.

             DEVICE argument is optional.  In case you want to specify the PulseAudio device which will capture the
             audio, you can run this command with the name of that device.  You can find your device by running
                   $ pactl list sinks | grep Name

Pulseaudio automatically creates the 'virtual mic' so you just need to figure out the correct name :)
Closing, but feel free to ask more questions here if you need further assistance.

That's the thing, I tried every line from pactl list sinks | grep Name, but none of them work. Always takes the mic input.

Update: it works, but I have to plug out my audio interface, otherwise it always picks up the mic. Weird

I'm having the same issue, can we reopen?

I'm having the same issue, can we reopen?

Can you post the exact command you are using?

[pbb@amalthea:~]$ pactl list sinks | grep Name
	Name: alsa_output.pci-0000_00_1b.0.analog-stereo
	Name: echoCancel_sink

[pbb@amalthea:~]$ pactl list sources | grep Name
	Name: alsa_output.pci-0000_00_1b.0.analog-stereo.monitor
	Name: alsa_input.pci-0000_00_1b.0.analog-stereo
	Name: echoCancel_source
	Name: echoCancel_sink.monitor

[pbb@amalthea:~]$

I tried alsa_output.pci-0000_00_1b.0.analog-stereo, echoCancel_sink and echoCancel_sink.monitor. All of those resulted with the microphone being recorded.

Does this work:
wf-recorder -a"alsa_output.pci-0000_00_1b.0.analog-stereo.monitor"

?

Ah, yes! Maybe the documentation should mention that one should add ".monitor", or use pactl list sources | grep Name instead of pactl list sinks | grep Name.

You're right, the manpage is outdated ... Fixed it in https://github.com/ammen99/wf-recorder/commit/2a901365571de8b86a7eaf95fe9d0cd5a5645579


If fab set fails, the release should fail in Azure Devops.

As an operations person, I want the release script to fail if fab set fails in the course of its execution.

I believe this is fixed - if not, please reopen.


inconsistencies between numpy and dask pad() function

I am not sure why but following code create different answers for NumPy and Dask,
import dask.array as da
import numpy as np

# numpy
arr = [[1,2,3],[3,4,5],[6,7,8]]
arr1 = np.pad(arr, (2,2), mode='reflect', reflect_type='odd')
print(arr1)

# dask
arr2 = da.pad(da.from_array(arr), (2,2), mode='reflect', reflect_type='odd').compute()
print(arr2)

The result is
arr1 =
[[-6 -5 -4 -3 -2 -1  0]
 [-3 -2 -1  0  1  2  3]
 [-1  0  1  2  3  4  5]
 [ 1  2  3  4  5  6  7]
 [ 4  5  6  7  8  9 10]
 [ 7  8  9 10 11 12 13]
 [ 9 10 11 12 13 14 15]]

and
arr2 =
[[8 7 6 7 8 7 6]
 [5 4 3 4 5 4 3]
 [3 2 1 2 3 2 1]
 [5 4 3 4 5 4 3]
 [8 7 6 7 8 7 6]
 [5 4 3 4 5 4 3]
 [3 2 1 2 3 2 1]]

the first one is true but second one not.

If array is converted to Dask Array using da.from_array(arr) and pass to the NumPy then np.pad() function also calculates as da.pad(). I am not sure this is expected behavior or not.

Thanks @uturuncoglu for raising this.  I believe https://github.com/dask/dask/pull/6579 should resolve this once merged in.
As for np.pad() operating on a dask array, yes, this is expected.  NumPy has a protocol for allowing numpy to dispatch to another array library.

I am closing this since it seems like a duplicate of #6127, but thanks for bringing this up!

@jsignell thank you for your help.

I see I made a typo in the following line. That should have said reflect_type instead of reflect:
https://github.com/dask/dask/blob/b3707ffac2563c2fa925bc9247564c1c8929e6dd/dask/array/creation.py#L1057
The mode='reflect', reflect_type='odd' has not yet been implemented, but it should have thrown a NotImplementedError. @jsignell is right that #6579 should have this fixed as it implements this option.


create smtp logger when specified in app.config

Need to instantiate the SMTP handler and add it to the logger.
This needs to be done in util/log.py

Took longer than expected, but appears to be working.


Build Process

Hello, would you be open to a PR for a build script using either rollup or webpack?

Of course! :smile:

@Akryum added pr for this, please let me know what you are thinking.  I will use this in a project later tomorrow (i am CST which is -6GMT) so will be ~12 hour-ish.


do not cache queries

Noticed in the ASK query when it was being enabled in the modsec rules.  Need to add something like this to apache configuration
<Location /sparql >
        Header set Cache-Control "max-age=0, no-cache, no-store, must-revalidate"
        Header set Pragma "no-cache"
</Location>

PS Maybe the page should be not-cached rather than the endpoint.  The query doesn't run in a naive browser, so probably the error page is what's getting cached.

Wound up with
<LocationMatch "^(/|/index\.html)$" >
        Header set Cache-Control "max-age=0, no-cache, no-store, must-revalidate"
        Header set Pragma "no-cache"
</LocationMatch>

per above PS.

To configure Cache-Control headers in your Apache server, you can use the following directives:
<Location /sparql>    Header set Cache-Control "max-age=0, no-cache, no-store, must-revalidate"    Header set Pragma "no-cache" </Location>
This snippet ensures that the /sparql location is not cached by browsers or CDNs. The max-age=0 directive prevents caching, and no-cache ensures that the content is always revalidated. Additionally, you might consider applying similar settings to specific files or directories as needed

Thanks, but it turns out it's not the endpoint or the page.  The caching is being done by a javascript widget in the page.  Yet to be fixed . . .

OK, can't test in the servers, but I think this fixes the caching in the resultset section.  In the index.content file in si-web/html, below the line
(yasgui.getTab()).setQuery("select distinct ?graph where { graph ?graph { ?s ?p ?o } } limit 100");
added:
yasgui.getTab().yasr.options , {
	requestConfig: {
		persistencyExpire: 0,
}};

it's in git tag 1.1.0.  Please update the git repo in the dev server and test it with good and bad queries for several days before putting in a deployment request for the upper tiers.  Feel free to ping for the build details.


Make unit tests more 'mockist' by adding mocks, stubs

Context
I like the idea of unit tests doing more 'checking' then testing. There's really no need to involve the database and external servers when stubs and mocks will do.
This will also greatly increase the speed of the tests and hopefully improve my software design.
Current Behavior/Workaround
Unit tests are more 'classisit' where they make real connections to servers and databases to get results.
Ideal behavior
unit tests are completely disconnected from databases and servers which will make them more 'unit testy' as opposed to being more 'integraty'
Things to consider
Probably want to wait on this issue https://github.com/jsolly/blogthedata/issues/20 to be implemented before taking this on.
https://pypi.org/project/django-webtest/

This is complete. With this commit, we are using mocks to replace calling APIs such as Ipinfo and OpenAi.


Translation of nationality into different languages

I use the country gem to provide my users with the ability to specify their nationality.
At the moment, the list of nationalities is taken from the nationality attribute.
Users in different locales do not have the list translated into their own language, so I would like to add the ability to get the country.nationality in different languages, eg by switching locale:
Country['IT'].nationality #=> "Italian"
I18n.locale = :it
Country['IT'].nationality #=> "Italiana"

or by explicitly requesting the translation for a given locale:
Country['IT'].nationality()     #=> "Italian"
Country['IT'].nationality(:en)  #=> "Italian"
Country['IT'].nationality(:it ) #=> "Italiana"

I'm happy to work on a PR to implement this, but I would like to know what is the preferred way to store this info in the data files, the options I see are:

add a nationalities key in the main data files, with all the translations eg:

---
IT:
  continent: Europe
  alpha2: IT
  name: Italy
  nationality: Italian
  nationalities:
    it: Italiana
    fr: Italien
    ...

or create a new set of files altogether, such as:

# lib/countries/data/nationalities/IT.yaml
IT:
  nationalities:
    it: Italiana
    fr: Italien
    ...

If you give me an idea of what is the preferred way, I will be happy to implement this and make sure that localised nationalities fall back to english should a translation not exist.
My personal preference would be:

keep all info into the main data file, so contributors have an easier time filling it in
automatically return the localised nationality based on current I18n.locale
but I'm open to other suggestions.

@paolodona 👍 ,  but in regards to data files.  We have pushed translations of country names back out of the primary data files so that we can reduce runtime memory.  And I assume this would want to use the same system to load in translations that a specific app needs.
I also agree on the fallback state when translations are present.

@paolodona . @rposborne
Was this feature added ?

:+1: I'd be glad to see this implemented, anybody worked on this already? Otherwise I might take a look.


Multi page placement

Hi,
What if I have more pieces that I can fit onto one sheet? Could it place the pieces onto multiple sheets?

yeah it opens a new sheet under the first one


Filter by resource

This is a great app by the way, disappointing that you have to create it to resolve a lack of core feature!
Another issue I have with planner is the lack of ability to see tasks across multiple plans for a single user, and this app looks like it could also resolve that, i just can't seem to get a filter by userid to work. Is this something you would consider adding, or advising how I could do it locally?
regards
Steve

This looks great.  How can I see all of my plans; only some of them are listed?

Hi Steve and sory for LATE reply :| Planner connector (https://docs.microsoft.com/en-us/connectors/planner/) does not provide a method to search tasks over plan in any way. I think only way to achieve this would be to fetch all the plans for the current user -> fetch all the tasks for each of the plan -> do the search against all accumulated tasks. This is not a good approach, as a single user could have dozens of plans...

Closing this issue as nothing can be done about it really, as the Planner connector does not support it.


:info and pager

:info does not get shiny highlighted output in the pager on IPython 3.0
I'll investigate, but I note what I found here.

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

This has been somewhat addressed, but some pending issues with export (https://github.com/gibiansky/IHaskell/issues/413 and https://github.com/gibiansky/IHaskell/issues/395)


One table display

it would be good if you could display only one table(one month), instead of 2
also it would be nice if it could open to the left and right side... now it's opening just on the right, and it breaks pages grid...

That It is possible since version 3. We are working on the open up or down feature

Sorry about the demo page, it is not updated... actually it is using the v2

Oh great, where can i see new features?

Right now? downloading the source and running "npm i" and then "npm run serve"

showSingleMonth true solves the request.


ID is not automatically added to HumanMessage used to invoke the graph

This could be annoying if you wanted to update the message later on in the graph because to update a message you need to have its ID.

cc @jacoblee93 I thought this was alr implemented?

Yeah me too but there's something wrong

Fixed


Laravel 6 support ?

Hello,
thank you so much for your package. Can you add a laravel 6 support please ?
thank you

Hi, I see you've made some minor changes to your fork, are these the changes you'd wish to introduce or is there something else Laravel related?
There is an existing ServiceProvider which was written when Laravel 5 was the latest release. I peeked around and I didn't see any major changes for Service Providers. There has been some troubles with this not registering in the past though.
Could you elaborate on which changes/additions need to be added for Laravel 6 support?

Hello,
sorry I forked your repo but it was because I had a bug with the $_ENV variable, I don't know why it returns a null value sometimes.
I will delete the repo, I just downgraded my laravel project to 5.8. I don't need v6 for the moment.
Thank you for the support!

If you want to you can PR your fork into mine. We're currently depending on illuminate/support so we can use env() instead as it looks a little bit nicer.
I'll implement it if you don't PR it :P


How do I delay when I visit a page?

Kind time of the day, my script closes right after the page is loaded, can I do something like that so that after the download, he waited for example, sekmid 5 and then finished the work?

UP!


Add missing dimensions to Model.Dims() for MIMO systems. Resolves #6

There was a mistake in sim package in that there was no distinction made between input vector length (u) and state vector length (x). Model dimension now is fully explicit returning length of x state vector, u input vector, y output vector, and z disturbance input vector. Do note that I have yet to implement the disturbance matrix E so for now the fourth parameter returned by Dims() should be equal to length of x (first parameter). I just thought it was a good idea to future-proof the API.
I've also added documentation to interfaces.

Replying to your nits

There is a real need to using nx, ny and nu as returned by Dims(), without the three one does not define a control system. You were missing the nu parameter before and doing some weird workaround in Propagate() where the input vector u had to have the same length as the output vector, which is (almost) always not the case. The way I see it we can:

Rename Dims() to something like SystemDims() or ControlDims()
Split Dims() into two functions, in which case we'd have to rename one or both.
Remove Dims() and merge two interfaces Model and DiscreteModel, which I'm not even sure if is a good idea. I'm not that familiar with the library... yet

Consider it done!

Ok, I see what you mean. That's a reasonable request.
In that case, I'd prefer to call the func SystemDims/ContolDims -- just for the sake of avoiding confusion with the gonum API.

Alright, changes incoming. I'd like to reply to a comment you made earlier:

*invalidModel Dims() method returns 4 values, 2 of which are always 0 and not used at all.

Yes, that is intentional. Dims() (now SystemDims()) returns nx state vector length, ny external/observed/output state, nu input vector length, and nz disturbance vector length.
The disturbance matrix has not been implemented yet so nz is unused as of today.
nx and ny must be greater-than zero as you probably know. This is a requirement for any estimator to be able to predict internal state from incomplete measurements.
nu may be zero if the system does not have inputs. This is perfectly fine. It would mean the system is uncontrollable, which is undesirable in some cases while perfectly fine in others i.e.

airplane attitude+velocity as state which should be controllable
a river's temperature, flow-rate and pollution whose state might be of interest but is fundamentally uncontrollable without harnessing serious hardware

@milosgajdos I forgot to make a new release. Looks like importing your package still uses old version.

I just cut v0.1.0 release


How to access label and node id in custom component from code/yaml?

Can you please help me to read the component label and the node id from yaml, and pass that as an argument to .py file?

Are you referring to generic components (those that are used to run Jupyter notebooks, Python scripts, and R scripts) in Elyra or custom components you've implemented?

I am asking about the custom component. Want to read label from the component properties in the .yaml file and pass that as argument while running .py file

I'm not aware of any way to make this information available.


Make Item/Stack naming consistent in entity and player classes

Inside PlayerEntity class, methods use Stack or Item in their name rather randomly. That's just terrible for search or IDE suggestions if you don't remember exact name. Also, a few names are unnecessary complicated, overly long and in backwards order.
Majority of names use Stack, but I suggest change everything to Item as doing other way would result in worse names (isUsingStack, clearActiveStack, getStackCooldownManager just to name a few). Probably this is can also be done in Inventory classes.
dropStack -> dropItem
dropEquippedStack -> getEquippedItem
getMainHandStack -> getMainHandItem
getOffHandStack -> getOffHandItem
getStackInHand -> getHeldItem

Items in head/chest/legs/feet slots don't have to be armor
getArmorItems -> getWornItems

getItemsHand was probably meant to be counterpart to getItemsEqipped but mistyped word held into hand. Also, both don't need to be in backwards order.
getItemsEquipped -> getEquippedItems
getItemsHand -> getHeldItems

this one doesn't seem to check any "validity" other than simply presence of item in slot
public boolean isEquippedStackValid(EquipmentSlot equipmentSlot_1) {
  return !this.getEquippedStack(equipmentSlot_1).isEmpty();
}

isEquippedStackValid -> hasItemInSlot or hasEquippedItem

ItemStack is unnecessary here, just Item is ok
giveItemStack -> giveItem

or it can be insertItem as in Inventory, those methods do basically same thing

Shouldn't we apply this more globally - if so, where should we stop?

Honestly, I have an issue with this. I don't think it should be unified like this, because then it's using the same terms whether it's referring to an Item or an ItemStack. It should depend on what the method's actually about.

Fair, I think that was the original intent; either way, most of these names need rework anyway, as evident above, so we might as well discuss.

IMO stack everywhere it's applicable is the better option.
Changing ItemEntity#getStack, for example, would mean turning an ItemEntity into an Item is the goofy looking myEntity.getItem().getItem()

Well, ItemStack is very likely not a Mojang name. As consequence, everything with stack is not a Mojang name.
(Based on https://gist.github.com/Runemoro/e8c45aabc8f7048d0414e42ea338d510, this class seems to be (excluding out of place AliasedBlockItem  and ItemPropertyGetter) between ItemFrame and Items. My guess is ItemInstance...)

ItemInstanceMapIsFix and other fixes say Instance in them

for the record I've always supported ItemInstance

However, TypeReference contains
 public static final TypeReference ITEM_STACK = () -> {
      return "item_stack";
   };

so it's not absolutely clear

ItemStack certainly was not the original name for them, but it's become such a fundamental term that it's what the former modders at Mojang call them now.

The getters could all use "item" and avoid looking weird if we renamed ItemStack to Item, and Item to ItemType. That way you have ItemEntity.getItem().getType(), and everything is clear. However, the cost of this move would be considerable.
We could maybe consider renaming just ItemStack#getItem to ItemStack#getType to avoid redundancy ?

this one doesn't seem to check any "validity" other than simply presence of item in slot

public boolean isEquippedStackValid(EquipmentSlot equipmentSlot_1) {
  return !this.getEquippedStack(equipmentSlot_1).isEmpty();
}

I'd check override of this method before coming to that conclusion. It's possible other classes are checking validity.
Otherwise this could easily become isStackEquipped(slot).


Support suspend blocks in Answers

When testing coroutines, I'd like to be able to mock a dependency with a suspend function with a custom Answer such that it delays, suspends indefinitely, or has other custom behavior besides simply returning a value. The particular test case I have in mind is testing that when the CoroutineScope of the class under test gets cancelled while one of its dependencies is suspending, it properly cancels the dependency and avoids further execution.
Example:
runBlocking {
    whenever(repository.loadStatus()).willAnswer {
        // Suspend indefinitely
        suspendCancellableCoroutine<Status> {} <-- Compiler error 'Suspension functions can be called only within coroutine body'
    }
}

or
mock<Repository> {
    onBlocking {
        loadStatus()
    } doAnswer {
        // Suspend indefinitely
        suspendCancellableCoroutine {} <-- Compiler error 'Suspension functions can be called only within coroutine body'
    }
}

MockK currently supports something like this with:
coEvery {
    repository.loadStatus()
} coAnswers {
    // Suspend indefinitely
    suspendCancellableCoroutine<Status> {}
}

Testing asynchronous coroutines code is now achievable with the latest version of https://kotlin.github.io/kotlinx.coroutines/kotlinx-coroutines-test/.

Made a PR: #357


fix(metalbear-co/mirrord) support linux/arm64 and use zipped assets

fixes #10010

Thanks!

https://github.com/metalbear-co/mirrord/releases/tag/2.2.1

Create macOS universal2 binaries


Version 1.13.0 won't start on Windows 7 and Windows 8.1

Describe the bug
I found that the application does not run on my Windows 8.1 x64.
The problem is only with version 1.13.0. Version 1.12.0 works fine.
The application is successfully installed, but at startup I get a message like in the picture, with only close option.

Not real screenshot. Just illustration.
Portable version behaves the same way.
I don't know what details are important. Looks like there is no useful information in the logs.

Faulting application name: localsend_app.exe, version: <IP_ADDRESS>, time stamp: 0x656e62e6
Faulting module name: unknown, version: <IP_ADDRESS>.0, time stamp: 0x00000000
Exception code: 0xc000001d
Fault offset: 0x000000e1d978e9a3
Faulting process id: 0x2318
Faulting application start time: 0x01da27c40834703b

To reproduce
Just start app
Expected behavior
App starts successful
Screenshots
No response
Desktop (please complete the following information)
Windows 8.1 x64
old AMD CPU
old Nvidia Graphics
Smartphone (please complete the following information)
No response
Additional context
No response

Thanks @cdhigh.
According to the error, it seems like a bug in Flutter.
LocalSend v1.13.0 uses Flutter v3.16 while LocalSend v1.12.0 used Flutter v3.13.
It seems that this error was already fixed in a previous version but I suspect that it has been introduced in Flutter v3.16 again: https://github.com/flutter/flutter/issues/110948

You can try out this zip:
LocalSend-1.13.0-windows-x86-64-Flutter-1.13.zip

You can try out this zip:
LocalSend-1.13.0-windows-x86-64-Flutter-1.13.zip

Tested ok, Problem solved.

You can try out this zip:
LocalSend-1.13.0-windows-x86-64-Flutter-1.13.zip

This version starts perfectly fine. Thanks.

@cdhigh @Vilmodao Can you confirm that the following zip crash as well? If it crashes, then I will send this bug report to Flutter so they can fix this issue.
flutter-3.16-test.zip

@cdhigh @Vilmodao Can you confirm that the following zip crash as well? If it crashes, then I will send this bug report to Flutter so they can fix this issue.
flutter-3.16-test.zip

Opened normal.


Preparing for v1.5.5

@nathanielmanistaatgoogle @jonparrott Most of this change is regenerating the docs using the docs/build script. The rest is just adding to the CHANGELOG, and bumping the version in __init__.py.
I just did this as a point release, but can make it a minor release if you think adding a keyword arg to media upload methods qualifies as "adding new functionality".

@bjwatson can you rebase to pull in #303?

@jonparrott Done. PTAL

LGTM
On Tue, Nov 1, 2016, 5:04 PM Brian J. Watson<EMAIL_ADDRESS>wrote:

@jonparrott https://github.com/jonparrott Done. PTAL
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-python-client/pull/304#issuecomment-257735683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUcyfcBdeMXPsCl4fL4Wxh6kebQLybks5q59OEgaJpZM4Klrtn
.


App: Drop actions

What does this PR do?
Removes app actions, we will just return the app spec and let the caller decide what to do.

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

:books: Documentation preview :books:: https://pytorch-lightning--18660.org.readthedocs.build/en/18660/

cc @borda

@awaelchli I'll go through and update them but the gist is just: return the spec of the created app, raise errors directly instead of using the action


Create testing module with ContentMatcher. #54

please rename the module to ContentTestPal. We'll probably have another test module which provides tools for our internal testing, like certain matchers for ContentProviderOperation.Builder

Renamed module and package to contenttestpal.

Ready for review again.


Comment.nvim doesn't work when lazyloaded

Describe the bug
A clear and concise description of what the bug is.
When I try to use Comment.nvim in Neovim, the keybindings do nothing.
Steps To Reproduce
Steps to reproduce the behavior:

Install Neovim with the plugin comment-nvim via Nix.
Call the setup function for Comment.nvim.
Press gcc in a plaintext file.

Expected behavior
A clear and concise description of what you expected to happen.
When I press the keybind gcc, the selected line should be commented in Neovim.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
This plugin works on Arch Linux with Packer.nvim, but not on NixOS with Nix.
Notify maintainers

@QuantMint
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here
these 2 paths will be fetched (0.07 MiB download, 0.34 MiB unpacked):
  /nix/store/6pkx11fz05v4m8jgx76y1q1ri6llpf3i-bash-interactive-5.1-p16-dev
  /nix/store/ypszvxqskanxpxplcda0pfa6kmj2yjy0-stdenv-linux
copying path '/nix/store/6pkx11fz05v4m8jgx76y1q1ri6llpf3i-bash-interactive-5.1-p16-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/ypszvxqskanxpxplcda0pfa6kmj2yjy0-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.43, NixOS, 22.11 (Raccoon), 22.11pre384129.e0169d7a9d3`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.9.0`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I'm not currently using comment.nvim, however, I have several thoughts

ensure the keybinding is registered. Look for the keybinding gcc in the output of :nmap
Perhaps it has something to do with the filetype. You mention a plain text file. However, I dont think the filetype text sets commentstring. Maybe try open test.c and use the keybinding (ensure that the line has text on it; it does not need to be valid)

(Also, linking your issue in case someone else is looking https://github.com/numToStr/Comment.nvim/issues/168)

The keymap exists on :map.
My actual test file was a .lua file, sorry for being unclear.

Okay, I'm unable to reproduce using the same commit (latest unstable) and just the basic setup require('Comment').setup().
Perhaps there is some interaction with other plugins with the same keymap? (i'm not too sure). But one think you could try is disable all other plugins and see if the problem still exists

I've had a slight revelation: Comment.nvim works when autoloaded, but NOT when lazyloaded. Will update the title accordingly.

You're probably hitting this: https://github.com/vim/vim/issues/1994, which arguably isn't a bug as Comment.nvim really shouldn't be abusing the after directory like it is.
Anyway, a workaround is manually doing :runtime after/plugin/Comment.lua after the plugin has been loaded, or better yet as a part of the lazy loading procedure.

Manually sourcing the file worked, so I'll close the issue, but I have a question: how does Packer.nvim lazyloading work out of the box with Comment.nvim? In other words, does it do something special to get around this issue?

Packer doesn't do anything special, it just sources files in the after directory if there are any.

Got it. Thanks for the help!


Refactor hooks for getting calculator data

Description :sparkles:
Move common functions and values to a separate hook for getting calculator data
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #771 (16df979) into develop (dc476aa) will decrease coverage by 27.12%.
The diff coverage is 5.00%.

@@             Coverage Diff              @@
##           develop     #771       +/-   ##
============================================
- Coverage    62.80%   35.67%   -27.13%
============================================
  Files          785       69      -716
  Lines        19858     1351    -18507
  Branches      1937      275     -1662
============================================
- Hits         12472      482    -11990
+ Misses        7017      842     -6175
+ Partials       369       27      -342

Impacted Files
Coverage Δ

...ymentAppliedMoreView/EmploymentAppliedMoreView.tsx
0.00% <0.00%> (ø)

...entAppliedMoreView/useEmploymentAppliedMoreView.ts
0.00% <0.00%> (ø)

...efitCalculatorView/SalaryBenefitCalculatorView.tsx
0.00% <0.00%> (ø)

...itCalculatorView/useSalaryBenefitCalculatorData.ts
0.00% <0.00%> (ø)

...end/benefit/handler/src/hooks/useCalculatorData.ts
0.00% <0.00%> (ø)

...nefit/handler/src/hooks/useHandlerReviewActions.ts
0.00% <0.00%> (ø)

frontend/benefit/handler/src/constants.ts
100.00% <100.00%> (ø)

...r/browser-tests/index-page/userActions.testcafe.ts

...hared/shared/audit_log/tests/test_audit_logging.py

frontend/shared/src/hooks/useErrorHandler.ts

... and 714 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 dc476aa...16df979. Read the comment docs.


Support searching for variants by rsID

Adding to this issue, if I type an rsID into the search bar, the dropdown box appears to be broken. Pressing Enter doesn't seem to work.

If there are no results for a search query, the dropdown box shows no content. #313 tracks clarifying when there are no results for a query.

Support for searching by rsID added in c8155bc8142772011117719fd27d35a38c750192.


npm dependency broken

Since about 15 hours, the npm install of angular-openlayers-directive fails.
This seems to be because it requires openlayers 3.8.x, which requires jsdoc 3.3.0-alpha9 (as non-dev dependency, but that's another story), which requires the master branch of https://github.com/hegemonic/taffydb.
The problem with this is that the package name of this branch has changed. From jsdoc 3.3.2/openlayers 3.9.0 forward, this has been fixed by requiring a specific commit.
Is is possible to upgrade the openlayers dependency to a newer version, at least 3.9.0?

oh ok. Would you be willing to do a PR for this?

Of course :-)

@njirem Awesome!! Looking forward to. Make sure you check out our Contribution guidelines :wink:


Latest application version isn't pushed to Helm

Hey there!
I noticed it's been a little bit since a new release was pushed to the Helm repository:
❯ helm repo add plumber-cd https://plumber-cd.github.io/helm/

❯ helm search repo plumber-cd --versions
NAME                              	CHART VERSION	APP VERSION	DESCRIPTION
plumber-cd/dynamic-pvc-provisioner	0.1.0        	0.2.0      	A Helm chart for Kubernetes
plumber-cd/dynamic-pvc-provisioner	0.0.3        	0.1.1      	A Helm chart for Kubernetes
plumber-cd/dynamic-pvc-provisioner	0.0.2        	0.0.4      	A Helm chart for Kubernetes
plumber-cd/dynamic-pvc-provisioner	0.0.1        	0.0.2      	A Helm chart for Kubernetes
plumber-cd/reclaimable-pv-releaser	0.1.0        	0.2.0      	A Helm chart for Kubernetes
plumber-cd/reclaimable-pv-releaser	0.0.3        	0.1.1      	A Helm chart for Kubernetes
plumber-cd/reclaimable-pv-releaser	0.0.2        	0.0.4      	A Helm chart for Kubernetes
plumber-cd/reclaimable-pv-releaser	0.0.1        	0.0.2      	A Helm chart for Kubernetes

Would it be possible to push v0.2.3 of the provisioner and releaser to the Helm repo?
Thanks much!

Hi @evanfoster, thanks for your interest in the project! I might be misunderstanding Helm, but I do not think this is necessary. There was no changes in the chart in between v0.2.0 and v0.2.3. The APP VERSION "bundled" with the chart is just a default value and it is up to the user to set it correctly. Could you point me to some documentation or article so that I can learn please?


Question: Looking at the redeem operations, are there concerns with a token holder burning their own tokens?

Hi there,
I am trying to follow the code and I see all the redeem operations eventually call _callSenderExtension and _callTokenExtension which make the calls tokensToTransfer and tokensToValidate respectively.
Is the design ment to make use of the userExtension and implement tokensToTransfer  to stop user's from burning their own tokens?
I looked into tokensToValidate and it doesn't look like it would prevent this case.

Hello @karimkawambwa ,
There is no implementation example of userExtension on this repo, but it is typically the place where you could setup logic to stop users from burning their own tokens.

Noted @gauthierpetetin, so a dev would also have to add logic in the userExtension to only allow a subset of operators to change token partitions? Not all operators should be allowed to do this operation.


[BUG] Missing compatibility with IntelliJ IDEA 2<IP_ADDRESS>

Describe the bug.
Not actually a bug, it's just not compatible with the newest version of IntelliJ and I was hoping it would be an easy fix.
Expected behavior
IntelliJ IDEA 2<IP_ADDRESS> allows the installation of the plugin.
Screenshots

How to Reproduce

Install IntelliJ IDEA 2<IP_ADDRESS>
Search for the plugin(you will not find it)
If you are logged in to jetbrains, the website shows the compatibility issue https://plugins.jetbrains.com/plugin/15673-asyncapi

🥦 Browser
None
👀 Have you checked for similar open issues?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
None

@camaral hi! New version is available


URL for service tags download is unstable

My service grabs the service tags frequently (~1 per day) from https://www.microsoft.com/en-us/download/confirmation.aspx?id=56519
We query https://www.microsoft.com/en-us/download/confirmation.aspx?id=56519 to get the webpage, and parse it for the download URL (like https://download.microsoft.com/download/7/1/D/71D86715-5596-4529-9B13-DA13A5DE5B63/ServiceTags_Public_20230508.json) to download the most recent service tags.
Yesterday, 05/08, https://www.microsoft.com/en-us/download/confirmation.aspx?id=56519 was gone and the UI for https://www.microsoft.com/en-us/download/details.aspx?id=56519 had changed to include the download URL.
Today https://www.microsoft.com/en-us/download/confirmation.aspx?id=56519 is back and https://www.microsoft.com/en-us/download/details.aspx?id=56519 no longer contains to download URL
Is there a recommendation on how to parse this in a stable way? Or can you tell us what the plan is for this page, so we don't run into more surprises in the future.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 08217244-82bb-13b5-46f7-454801d8f865
Version Independent ID: 4a8b0a32-bf57-7544-c60d-6b7f250c4477
Content: Azure service tags overview
Content Source: articles/virtual-network/service-tags-overview.md
Service: virtual-network
GitHub Login: @asudbring
Microsoft Alias: allensu

Tagging @KapilAnanth-MSFT as they were able to help with a previous issue

@jnlycklama
Thank you for your feedback! We will review and update as appropriate.

@ChaitanyaNaykodi-MSFT any update?

@jnlycklama
Apologies for the delay. I have highlighted this issue internally and will share an update as soon as I get a response.

@jnlycklama
Apologies for the delay here. Since we did not hear back from the team yet. It will be helpful if you could open a support ticket for this issue. If you have a support plan you may file a support ticket, else could you please send an email to<EMAIL_ADDRESS>with the below details, so that I can open a free one-time support for you in this case.
Subject : Attn Chaitanya
Thread URL: Link to this thread.
Subscription ID
Please let me know once you have done the same.


Precedence group declarations are only valid at file scope.

Resolves some crashers that existed because this check wasn't being performed for precedencegroup declarations, then feeding an invalid AST into the type checker.

@swift-ci please smoke test and merge.


hotfix/byuActivity -> dev

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed, 1 unresolved discussion (waiting on @geogor37)

support/src/main/java/edu/byu/support/activity/ByuActivity.java, line 123 at r1 (raw file):

		if (progressDialog == null || !progressDialog.isShowing()) {
			progressDialog = ProgressDialog.show(this, null, message, true, cancelable);
		} else if (progressDialog != null && progressDialog.isShowing()) {

Look into FavoritesActivity

Comments from Reviewable


Request: File dialog, console terminal

Hello, just a couple items that I think would be useful to have
File dialog: pretty self explanatory. I wasn't able to find much on how to do this in Iced.
Console terminal: equivalent to the integrated terminal in VSCode. Useful as a direct interface to bundled CLI programs.

gotcha. Thanks for the prompt response!
unfortunately, I don't think I can implement either of these :(

system76 actually finished this, but they never seem to contribute it to Iced.

I'm not sure if it works for all Os's or not. as mostly they design stuff for their own Linux OS.

I'm not sure if it works for all Os's or not. as mostly they design stuff for their own Linux OS.

last time I talked to one of their SE's they said it would be but who knows. Hector also said he has plans of bring rfd into iced itself. and for a terminal you can use this https://github.com/Harzu/iced_term


Added support for clearing the session before loading the runner.

Disabled by default.

Closing - no response


router: query: use IAppPartitions.GetPartitionID

do not forget to eliminate searching for BuiltinApp in BuildAppWorkspaces() to get app's PartsCount

wrong to eliminate searching for BuiltinApp in BuildAppWorkspaces() now because we do not know if the app is deployed. App is not deployed -> impossible to use IAppPartitions.AppWorkspacePartitionID() -> still need to search for among builtins. Must be eliminated after relocating BuildAppWorkspaces() to app deploy stage


Rotate non-renewable secrets

I have Vault credentials that are rotating every hour. I was not able to use the handy @VaultPropertySource annotation but was able to replicate its functionality with two beans:
@Bean
public LeaseAwareVaultPropertySource leaseAwareVaultPropertySource(
@Value("${stspath}") String stsPath,
SecretLeaseContainer secretLeaseContainer) {
  RequestedSecret secret = RequestedSecret.rotating(stsPath);
  PropertyTransformer propertyTransformer = PropertyTransformers.propertyNamePrefix("sts.");
  return new LeaseAwareVaultPropertySource(stsPath, secretLeaseContainer, secret, propertyTransformer);
}

@Bean
public static PropertySourcesPlaceholderConfigurer pspc() {
  return new PropertySourcesPlaceholderConfigurer() {
    @Override
    public void postProcessBeanFactory(ConfigurableListableBeanFactory beanFactory) throws BeansException {
      super.postProcessBeanFactory(beanFactory);

      ConfigurableEnvironment env = beanFactory.getBean(ConfigurableEnvironment.class);
      MutablePropertySources mutablePropertySources = env.getPropertySources();

      Collection<? extends PropertySource<?>> propertySources = beanFactory
        .getBeansOfType(org.springframework.vault.core.env.LeaseAwareVaultPropertySource.class)
        .values();

      for (PropertySource<?> vaultPropertySource : propertySources) {
        if (propertySources.contains(vaultPropertySource.getName())) {
          continue;
        }
        mutablePropertySources.addLast(vaultPropertySource);
      }
    }
  };
}

This is able to get the initial properties in the Environment, but when the tokens expire they are not renewed. Do I need to do more to renew the credentials with Vault and publish them to the Spring Environment?

I think the issue is due to LeaseRenewalScheduler.isLeaseRenewable() in SecretLeaseContainer.  Inspecting my lease I see I have a non-null leaseId with renewable=false, but for rotating credentials the code requires leaseId == null:
			if (!lease.hasLeaseId() && requestedSecret.getMode() == Mode.ROTATE) {

Let's go back for a moment to understand what's happening here. Do you want to rotate a secret that does not have a lease id associated?

Sure. To take a step back I would like rotating secrets when I specify
RequestedSecret.ROTATE.
I would like to understand the logic in the code around checking for the
existence of a lease id. According to:
https://www.vaultproject.io/docs/concepts/lease.html
A dynamic secret will always have a lease id, which matches my experience.
Is there another mechanism for obtaining and renewing a rotating secret in
spring vault?
On Tue, Mar 13, 2018, 4:20 AM Mark Paluch<EMAIL_ADDRESS>wrote:

Let's go back for a moment to understand what's happening here. Do you
want to rotate a secret that does not have a lease id associated?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spring-projects/spring-vault/issues/215#issuecomment-372631817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG09g0UGBAoX40lwv-gScuTwONhChTA6ks5td6thgaJpZM4SlWn9
.

Spring Vault currently rotates only secrets that:

Have a lease associated and are renewable (e.g. database backend)
Have no lease associated (e.g. key/value generic backend)

Secrets without a renewable lease are not rotated. We should consider rotating secrets that are non-renewable since renewal is only an intermediate step to keep the lease alive.

My use case is the AWS STS Assume Role: https://www.vaultproject.io/docs/secrets/aws/index.html#sts-assumerole
This delivers secrets that are not renewable but have a lease associated with them.  The good news is my PR #220 works with these credentials and rotates them as configured by this issue.  If there is an alternative implementation that you'd prefer I can assist, though I thought the changes were pretty minor and still pass all your tests.

Thanks a lot for clarification. From that perspective, this issue is a rather a bug that we need to fix. I updated the ticket accordingly. Will take a look at your pull request.

Thanks!  Just migrated from another solution to Spring Vault and love the library.  Let me know if you need assistance with the fix; for example in start(RequestedSecret, LeaseRenewalScheduler) my leaseId looks like:
aws/sts/{my sts role}/756170a9-243f-0418-fe62-df6e70652628
so without moving the if (isRotatingGenericSecret()) before the StringUtils.hasText(secrets.getLeaseId()) check the code would try to renew on the leaseId rather than rotating the secret.

I'd apply a slightly different approach than proposed in #220. As we already know that the lease isn't suitable for renewal, we can directly emit an expired event and rotation will happen directly.

I pushed a fix, snapshot builds are available from repo.spring.io. Care to give the snapshot builds a spin?

Hello, I tested the release today, 1.1.2.BUILD-SNAPSHOT.  It did not renew my tokens. :(

I'll take another look. In that context: What should happen if rotation fails (connection timeout, connection refused, HTTP 503 response)? Right now, renewal/rotation stops and one can subscribe to events to hook into recovery.

Care to provide a reproducer? Running the code from above against spring-vault-core-1.1.2.BUILD-20180320.150107-8 or newer works for me and I'm not able to reproduce the issue. Thanks!

Nothing seems to happen - I used addLeaseListener() and addErrorListener() but neither callback occurs.  With my PR I would get regular SecretLeaseCreatedEvents.
I will try to look into it more this weekend, from a fast glance through the changes I'd guess my Lease is being created here:
https://github.com/spring-projects/spring-vault/blob/28932b75592f6166855be3157a13f17077a88c7a/spring-vault-core/src/main/java/org/springframework/vault/core/lease/SecretLeaseContainer.java#L308
But this sets leaseId to null, which causes the new isLeaseRotateOnly() to return false

Re: rotation fails, it would be helpful if there was an easy way to configure retry with backoff.  Doesn't need to be the default, but it is such a common use-case so something out of the box would be awesome.

Please provide a reproducer (either steps to reproduce or code). Without being able to reproduce I'd consider the issue resolved.

I consider resolved! I tried to create a repro with 2.1.0.BUILD-20180321.143110-13 but it rotated my secret.  I think in my comment about 1.1.2.BUILD-SNAPSHOT my project was misconfigured to take your latest BUILD-SNAPSHOT.  But with a logging lease listener I see the SecretLeaseExpiredEvent followed by the SecretLeaseCreatedEvent.  Thanks for your help!

Thanks a lot for re-checking the issue. Closing this one as solved.

Spring Vault currently rotates only secrets that:

Have a lease associated and are renewable (e.g. database backend)
Have no lease associated (e.g. key/value generic backend)

Here what do you mean by secrets ? Is it token .
What will happen for vault consul configuration? Say I have spring cloud vault consul and have bootstrap.yaml with Backend for database and consul.
Would spring vault on rotate lease update the token of consul stored within Vault and notify the consul.
If not then do we have to use addLeaselistener to read the event and on rotate event update the token of consul (consul/creds/app) that's within the sessionManager. At present consul/creds/appname is readonly and it's token cannot be updated ?
After an hour the application gets 403 error code when there is a event to Rotate lease and consul is not able to connect to app/appname KV store.
It looks like connection issues between vault and consul based on token.


Meta: Responsible AI Service response blocking

This meta issue tracks scenarios where chat requests are incorrectly blocked by the Responsible AI Service, which is part of GitHub’s Copilot platform to detect and prevent the output of harmful content. [Learn more about responsible use].(https://docs.github.com/en/copilot/responsible-use-of-github-copilot-features/responsible-use-of-github-copilot-chat-in-your-ide).
Error message:

Sorry, the response was filtered by the Responsible AI Service. Please rephrase your prompt and try again. Learn more.

In case you experience repeated RAI-blocking in GitHub Copilot, please reach out to GitHub Support: https://support.github.com/
GitHub Copilot is iterating and improving the RAI content filters to reduce false positives.

It would feel better if there were a way to flag a prompt that we think should not have triggered the RAI block for review to help the service improve.

When you say "reach out to github support", I've never found any way of communicating with anyone on support.github.com , so could you maybe be a bit more specific about what part of the website to go to?


List Item props are being overwritten after wrapping with Sortable

CustomList.jsx
var React = require('react');
var SortableListItem = require('./CustomListItem.jsx');

var SortableList = React.createClass({

  getInitialState: function () {
    return {
      draggingIndex: null,
      data: this.props.data,
    };
  },

  updateState: function (obj) {
    if (_.isNull(obj.draggingIndex)) {
      this.props.onReorder(this.state.data.objects);
    }

    this.setState(obj);
  },

  render: function () {
    var listItems = this.state.data.objects.map(function (module, i) {
      return (
        <SortableListItem
          key={i}
          updateState={this.updateState}
          items={this.state.data.objects}
          activeItem={this.props.collection.meta.active}
          draggingIndex={this.state.draggingIndex}
          sortId={i}
          outline="list"
          module={module}
          onEdit={this.props.onEdit}
          onRemove={this.props.onRemove}
          onPreview={this.props.onPreview}
          onSetActive={this.props.onSetActive}
          onUpdateTabName={this.props.onUpdateTabName}
        />
      );
    }, this);

    return (

          {listItems}

    );
  },
});

module.exports = SortableList;

CustomListItem.jsx
var React = require('react');
var ModuleItem = require('../../../ModuleItem.jsx');
var Sortable = require('react-sortable').Sortable;

var SliderItem = React.createClass({
  displayName: 'SortableListItem',
  render: function () {

    console.log('props', this.props);

    return (

        <ModuleItem {...this.props.module} />

    );
  },

});

var SortableListItem = Sortable(SliderItem);

module.exports = SortableListItem;

When I log out props it has all the dragging methods that sortable provides
children:undefined
className:""
data-id:0
draggable:true
onDragEnd:()
onDragOver:()
onDragStart:()
onTouchEnd:()
onTouchMove:()
onTouchStart:()

But the props I passed in from CustomList component are missing. If I don't wrap it I will receive the correct props, which as an example would be:
activeItem:"56a6a1a4b45b9f7a6e836c81"
draggingIndex:\null
items:Array[3]
module:Object
onEdit:()
onPreview:()
onRemove:()
onSetActive:()
onUpdateTabName:()
outline:"list"
sortId:2
updateState:()

This was working at some point, but has regressed to this issue currently. I'm not if something changed in this package or if there is deeper underlying issue.
Any help is appreciated.
Cheers

I've explicitly gone back to "react-sortable": "1.0.1" and the above issue has been resolved.
There's no changelog for this repo so it's hard to say what broke. It would nice to know how to make this work with future releases.

Same issue here!
Thanks for a quickfix for now @killfish

You can specify custom props using childProps object, as described in README.
I think it's the similar issue as here: https://github.com/danielstocks/react-sortable/pull/50