lilypond failed to build on 10.10 ("yosemite")

From @gabriel-striewe on October 23, 2014 20:10
https://gist.github.com/fe4a29c2aab2c1d09056
I checked that XCode as well as the command line tools are up to date.
Copied from original issue: Homebrew/homebrew#33523

From @jacknagel on October 23, 2014 22:14
Can you rerun the build with --debug, and when it fails, choose "shell" and upload the config.log file?

From @gabriel-striewe on October 23, 2014 23:7
This is my config.log
https://gist.github.com/gabriel-striewe/050b62ac61beecb78390

configure:4021: /usr/local/bin/gcc-4.9 -Os -w -pipe -march=core2 -msse4 -mmacosx-version-min=10.10 -I/usr/local/opt/flex/include -I/usr/local/opt/readline/include -I/usr/local/opt/icu4c/include -I/usr/local/opt/gettext/include -F/usr/local/Frameworks -I/opt/X11/include -I/opt/X11/include/freetype2 -L/usr/local/opt/flex/lib -L/usr/local/opt/readline/lib -L/usr/local/opt/icu4c/lib -L/usr/local/opt/libffi/lib -L/usr/local/opt/gettext/lib -L/usr/local/lib -F/usr/local/Frameworks -Wl,-headerpad_max_install_names -L/opt/X11/lib conftest.c >&5
<built-in>: error: unknown value '10.10' of -mmacosx-version-min

Bingo?

Looks like Guile18 doesn't recognise 10.10 as a valid OS X version in the configure script?

Although that's just one of the fatal errors. It appears to have coughed up 3 fatal errors, the others are in this block.

    case $host_os in
    rhapsody* | darwin1.[012])
      _lt_dar_allow_undefined='${wl}-undefined ${wl}suppress' ;;
    darwin1.*)
      _lt_dar_allow_undefined='${wl}-flat_namespace ${wl}-undefined ${wl}suppress' ;;
    darwin*) # darwin 5.x on
      # if running on 10.5 or later, the deployment target defaults
      # to the OS version, if on x86, and 10.4, the deployment
      # target defaults to 10.4.  Will have something for you to test shortly.

From @gabriel-striewe on October 23, 2014 23:53
Here the same. Guile18 building fine. However, when I start guile at the command line, I receive:
ERROR: Unbound variable: define

Ah, yes. I encounter the same error.
Dominyks-MacBook-Pro # guile
ERROR: Unbound variable: define

And then it errors out and quits. Hmm.

Hmm :confused:. Guess guile18 isn't the error here, unless Lilypond hooks into it differently to the homebrew-versions build? Despite this:
configure:4068: error: in `/private/tmp/lilypond--guile18-6FVpTd/guile-1.8.8':

On 10.10 for me:
==> Summary
🍺  /usr/local/Cellar/guile18/1.8.8: 301 files, 7.1M, built in 2.3 minutes

From @jacknagel on October 24, 2014 0:56
<built-in>: error: unknown value '10.10' of -mmacosx-version-min is coming from gcc, it needs to be reinstalled (either gcc or gcc-4.9 from homebrew-versions, whichever you have installed).

From @gabriel-striewe on October 24, 2014 0:21
Thanks a lot for your help. Need to go to bed for a few hours... Will test tomorrow!

From @mikemcquaid on October 24, 2014 9:48
Tempted to boneyard Lilypond give the :tex requirement means it can't be tested or bottled.

Oh no rush. Whenever you have time. Jack may well have come up with a better fix by morning. Whenever you get a chance try brew install https://raw.githubusercontent.com/DomT4/homebrew/9e873862480ffc1c6cc507d9bd706a52d2598a10/Library/Formula/lilypond.rb --devel and let me know if that works for you.

From @svenax on October 24, 2014 7:54
On a clean installation of Yosemite, and fresh Homebrew. Guile and other dependencies builds fine, but when starting on Lilypond stable it fails with message
ERROR: Please install required programs:  /usr/local/bin/fontforge >= 20110222 (installed: 6ffb25d7732adcc8d32dae75bd90bda90250304)

Looks like it is getting the installed version wrong. It is actually
$ fontforge --version
Copyright (c) 2000-2014 by George Williams. See AUTHORS for Contributors.
 License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
 with many parts BSD <http://fontforge.org/license.html>. Please read LICENSE.
 Based on sources from 09:42 CEST 24-Oct-2014-ML-D.
 Based on source from git with hash:f6ffb25d7732adcc8d32dae75bd90bda90250304
fontforge 09:42 CEST 24-Oct-2014
libfontforge 20141024

$ brew info fontforge
fontforge: stable 20141014 (bottled), HEAD
https://fontforge.github.io
/usr/local/Cellar/fontforge/2.0.20140101 (1403 files, 41M)
  Built from source with: --with-x, --with-cairo
/usr/local/Cellar/fontforge/20141014 (1275 files, 39M) *
  Built from source with: --with-x
From: https://github.com/Homebrew/homebrew/blob/master/Library/Formula/fontforge.rb

From @svenax on October 24, 2014 9:51
As far as I know, TeX is an optional requirement, so we could mark it a such. It is only needed for lilypond-book, not for lilypond itself.
ETA: No wait, it needs Metafont to build the fonts. Oh well ...

From @gabriel-striewe on October 24, 2014 11:53
@DomT4:
thanks for your receipe. it gives me the following error:
<<<
WARNING: Please consider installing optional programs or files:  guile < 1.9.0 (installed: 2.0.11) texi2html dblatex pngtopnm
ERROR: Please install required programs:  /usr/local/bin/fontforge >= 20110222 (installed: 6ffb25d7732adcc8d32dae75bd90bda90250304)
<<<
We might have to bring this up on the lilypond list, unless you are an autotools wizard?

From @svenax on October 24, 2014 11:55
Thats the same error I got, @gabriel-striewe.

Looks like it is getting the installed version wrong. It is actually

Yeah, I wondered about this. The Lilypond configure script is quite specific in how it looks for Fontforge and consequently I'm not all that surprised it is struggling to find our considerably newer version. I guess we either remove Lilypond from the core here and potentially move it into a homebrew-tex tap, or we bundle an older Fontforge with Lilypond temporarily, or we look at modifying the configure script to look for newer versions. Bleh.

From @adius on November 8, 2014 13:33
+1

WARNING: Please consider installing optional programs or files: guile < 1.9.0 (installed: 2.0.11) texi2html dblatex pngtopnm

Expected that one. Essentially, I just removed the old guile in favour of the new one, but the support for that is experimental at this point and consequently you get the warning about it if you use it.
I'll look at the configure script to see what's going on with Fontforge and if we can fix that issue, but yeah, punt it upstream as well.

Yeah, punt this upstream. There's way too many references to how to work out the Fontforge build number in the configure script for me to patch them all. (I suppose I could, but I wouldn't be doing much else for the rest of the week heh).

From @gabriel-striewe on October 25, 2014 14:33
Before bringing this issue upstream, I will have to compile it by hand. That's what I just did, but the configure script hangs at guile-config:
checking for guile-config...
I cannot stop the process unless killing the terminal.
At the command prompt using guile and guile-config without autotools:
MacAir:~ gabriel$ guile
ERROR: Unbound variable: define
^C
MacAir:~ gabriel$ guile-config
ERROR: Unbound variable: define
^C
MacAir:~ gabriel$
Is anybody using guile18 stand-alone and might be able to help with this issue.
I did reinstall gcc-4.9. Command line tools and xcode are up to date...

From @bavarious on November 16, 2014 6:30

Yeah, punt this upstream. There's way too many references to how to work out the Fontforge build number in the configure script for me to patch them all. (I suppose I could, but I wouldn't be doing much else for the rest of the week heh).

Since the configure script is generated by autoconf, there’s actually one single place that needs editing, namely lines 43-48 in aclocal.m4:
    if test -z "$_ver"; then
        ## If empty, try date [fontforge]
        eval _ver=\"\`("$1" --version || "$1" -V) 2>&1 | grep '[0-9]\{6,8\}' \
        | head -n 1 \
        | sed -e 's/^[^.0-9]*//' -e 's/[^.0-9]*$//'\`\"
    fi

The regular expression being used in grep catches both a source Git hash and a libfontforge date/version, the latter being what the script should really use. Because of head -n 1, only the Git hash is considered, which is the source of this error.
There are multiple ways to avoid reading a Git hash from --version and upstream is definitely better equipped to decide which is better, but it shouldn’t be hard for LilyPond’s Homebrew formula maintainer to patch that grep invocation whilst upstream doesn’t solve this — for example, by forcing libfontforge or ignoring hash.
In theory, only one line needs changing in aclocal.m4. I’m not sure whether Homebrew runs autoconf, but it shouldn’t be hard to run it in case it doesn’t at the moment.

From @bavarious on November 16, 2014 6:50
Fixing the fontforge version detection problem in aclocal.m4 works as far as fontforge is concerned, but configure still fails with
WARNING: Please consider installing optional programs or files:  guile < 1.9.0 (installed: 2.0.11)

ERROR: Please install required programs:  International New Century Schoolbook fonts (make sure the fc-list utility can see them, or use --with-ncsb-dir) msgfmt mpost (due to a bug in metapost, versions 1.600 <= x < 1.803 are not supported; installed: 1.802) guile-config < 1.9.0 (installed: 2.0.11) (guile-devel, guile-dev or libguile-dev package) libguile (libguile-dev, guile-devel or guile-dev package). GUILE-with-rational-bugfix makeinfo >= 4.11 (installed: 4.8) texi2html dblatex pngtopnm

I’m not sure what boneyard and PRs are, and I won’t be spending more time with this issue. Feel free to forward this information to the maintainer and upstream if you think it’s helpful, and good luck!

Apologies for the jargon.
Boneyard = The place we send formulae that can no longer be supported for whatever reason.
PRs = Pull Requests.

Homebrew generally is pretty bleh on including patches that require the use of the autotools unless absolutely necessary. If you have something in mind though, feel free to open a PR with the proposed fix and the maintainers are forever happy to consider these things.
Given the options at this point, including your proposal, are 1) Patch and include autotool dependencies. 2) Given the hard tex dependency, boneyard. 3) Keep shipping a broken formulae, there probably wouldn't necessarily be objection to 1 if you manage to make it work.

From @andrewcsmith on December 11, 2014 22:48
It does indeed look like this is an issue with Lilypond's configure script. I changed that one bit to
    eval _ver=\"\`("$1" --version || "$1" -V) 2>&1 | grep 'libfontforge [0-9]\{6,8\}' \
    | head -n 1 \
    | sed -e 's/^[^.0-9]*//' -e 's/[^.0-9]*$//'\`\"

and it at least got past that one error. You can also pass the flag --enable-guile2 to autogen.sh to avoid the warning about guile.

From @andrewcsmith on December 12, 2014 21:24
Also, msgfmt and makeinfo can be solved with brew link gettext and brew link texinfo, respectively. (I plan to unlink them as soon as lilypond finishes installing.)

AFAIK, We're waiting on some upstream movement around Fontforge and Guile.

From @adius on November 24, 2014 19:9
What's the state on this? Would love to see it fixed =)

From @piotr-yuxuan on January 13, 2015 17:46
Hi there, I also get the same. I'm trying to install lilypond 2.19.15-1 on ArchLinux. Sorry if I post at the wrong place, I've searched for similar problem and I've found this page wherein issue seems rather similar to mine.
ERROR: Please install required programs:  International New Century Schoolbook fonts
(make sure the fc-list utility can see them, or use --with-ncsb-dir) /usr/bin/fontforge >=
20110222 (installed: 4cea471798c92848170c2f47311979805966093)

But methinks it should work. Here is what I've got:
$ yaourt -Qs fontforge
extra/fontforge 20141126-2
    Outline and bitmap font editor
$ fc-list | grep "Century"
/usr/share/fonts/Type1/c059036l.pfb: Century SchoolBook URW:style=Bold Italic
/usr/share/fonts/Type1/c059016l.pfb: Century SchoolBook URW:style=Bold
/usr/share/fonts/Type1/c059013l.pfb: Century SchoolBook URW:style=Roman
/usr/share/fonts/Type1/c059033l.pfb: Century SchoolBook URW:style=Italic

I haven't found any free fonts named « International New Century Schoolbook » to download.

From @Ledoux on December 15, 2014 20:39
For the fontforge old version problem in Yosemite,
there is a similar fix here :
https://github.com/FontCustom/fontcustom/issues/204
to reinstall the old version and the lilypond install works then for me :
brew update
brew uninstall fontforge
cd /usr/local/Cellar/
git checkout bfdf80274b3542684e5343cbca04f93d63e44d9d /usr/local/Library/Formula/fontforge.rb
brew install fontforge


Flow v0.57.2 crashes on Windows

I've updated from 0.56.0 to 0.57.2 and simply ran flow.
> flow

Launching Flow server for D:\projects\vmoose
Spawned flow server (pid=12524)
Logs will go to C:\Users\AWPTEA~1\AppData\Local\Temp\flow\DzCzBprojectszBvmoose.log
Library parse error:
C:\Users\AWPTEA~1\AppData\Local\Temp\flow\flowlib_bf80113\INDEX:1
  1: 101 file bom.js
         ^^^^ Unexpected identifier

Windows 10 x64
node v8.1.2

Same issue. Help would be appreciated.

Same issue here.

Contents of that file:
101 file bom.js
102 file core.js
103 file cssom.js
104 file dom.js
105 file indexeddb.js
106 file node.js
107 file react.js
108 file serviceworkers.js
109 file streams.js

Same error with 0.57.1 on Windows 7

Same issue.
Flow v0.57.2
System: Windows7

Same issue.
Flow v0.57.2
System: Windows10

Flow v0.57.3 solves it for me https://github.com/facebook/flow/commits/v0.57
System: Windows10
node: v8.5.0
yarn: 1.1.0

Closing as it seems to be solved


Slight typo

temperature.py
PVWatts defauls is 9.144 [m]
defauls -> default

Thanks @kjsauer!  Want to submit a PR to correct it?

I looked at the PVWatts reference doc. I don't see anything relating to 9.144 m (i.e., 30 feet). (It's in the Fuentes paper, though - I assume such 9.144 m wind speed height data was used in Fuentes's PV thermal model development, e.g., coefficient fitting.) In terms of PV thermal model application, the referenced PVWatts manual says TMY wind speed is at 10 m (Section 4). I'm not sure if PVWatts is translating wind speed data from a height of 10 m corresponding to the TMY wind speed data or from a height of 9.144 m corresponding to Fuentes's wind speed data used for model fitting (and how to figure out that answer, e.g., a PVWatts source code file, which I don't have handy at the moment, or personal conversation with someone who has knowledge of/access to the PVWatts source code). So, simple spelling error aside, the more important (related) and complex issue is that I'm not sure which of the following is the more appropriate guidance:
A.) Fuentes default is 9.144 [m] -> Existing reference doc [1]
B.) PVWatts default is 10 [m] -> Existing reference doc [2]
C.) PVWatts default is 9.144 [m] -> Some new TBD reference doc
D.) Something else altogether
Please advise.
P.S. Incidentally, I don't see it clearly listed in the Fuentes paper what the PV array (aka PV module) height is. However, presumably this is the height at which the Fuentes-derived coefficients apply. Whatever it is, if it doesn't match the PVWatts-reported value of 5 m (~16.4 feet above ground which is quite high off the ground) (Section 9), and if PVWatts is using Fuentes-derived PV thermal model coefficients, then this may raise other issues also (if my understanding is correct) - namely, I presume the Fuentes coefficients apply to whatever "Height of module (m)" was used in the derivation of said coefficients. Like, for the PVsyst thermal model, my assumption has been that if you derive {Uc, Uv} using wind speed data at some height (let's say, 10-m), then those coefficients apply (only) for cases where wind speed data is at that same height (in this example, 10-m).

Will take a closer look tomorrow, but the 9.144 number was taken from the SSC PVWatts implementation, which our implementation was meant to mimic.  The SSC comments provide some breadcrumbs:
https://github.com/NREL/ssc/blob/c72cfab5c831b131e136cb2fc48bf10c58304031/shared/lib_pvwatts.cpp#L194-L196

30ft is the height of the anemometer that was used as the predictor for wind speed at different heights above the ground. See this pub: https://www2.jpl.nasa.gov/adv_tech/photovol/2016CTR/SNL - Est Wind Speed vs Height_1985.pdf

My gut feeling is that if PVWatts wants to apply Fuentes-derived coefficients, they need to translate the 10-m TMY wind speed height to the same PV module height assumed by Fuentes in deriving said coefficients.

@kjsauer translating wind to module height is being done in the lines of code that @kandersolar linked. Is your question about the pvwatts functions in pvlib, or about PVWatts (the NREL application)?

Indeed, it appears PVWatts is translating from a wind speed height of 9.144 m to a PV module height of 5 m in order to then apply Fuentes-derived PV thermal model coefficients (which were derived for/are specific to some specific PV module height, let's call it X meters). I'm thinking the 9.144 m is supposed to be 10 m (the height of the TMY wind speed data used in PVWatts), and the 5 m is supposed to be X m where X is the PV module height used by Fuentes in deriving said PV thermal model coefficients. The question is primarily one for PVWatts, so I just opened a GitHub Issue there. But it then affects the PVLIB function's default values and possibly the referenced docs also.
I suppose the short path forward is to fix the spelling error which prompted this GitHub Issue and then close this issue. Then, once the new GitHub Issue in the other repository "ssc" is resolved, any new guidance which comes out of that can perhaps transfer over into this repo via another (new) GitHub Issue opened in this repo.
I defer to all of you. But I'm not really comfortable fixing a trivial spelling error myself when there may be more fundamental questions which remain open. Once the latter are resolved and the way forward on both the ssc and pvlib-python repos is clear, then sure, I'd be happy to do a PR after that.

Wind speed data in weather files from the NSRDB is from MERRA2 and is 2-meter data. I haven't found this written in any of the NSRDB documentation, but there is a discussion at about 1:10 in the Q&A segment of this 2022 NREL webinar on the NSRDB: https://www.youtube.com/watch?v=BzNoYACXCOY&t=4220s.

That's certainly interesting (and somewhat unexpected and disconcerting if true...). I just want to quickly point out what I already mentioned above, which is that the most up to date PVWatts reference document says the TMY wind speed in use is at a height of 10 m. Depending on what we find out, we may need yet another reference doc...
I guess I should've known all this already, because apparently I asked the question...
https://webcache.googleusercontent.com/search?q=cache:dqpgedl3nbgJ:https://nsrdb.nrel.gov/nsrdb/NSRDB_Webinar_QA_-_Oct_6_2020.pdf
"""
Wasn't the historical wind speed data at 10 m? Can old data be mixed w/ new given the change to 2 m? If we download all the individual years of data, is the wind speed always at 10 m or 2 m or is there some mixture of heights?
As of PSM v3 the windspeed data is at 2m. PSM v2 used MERRA derived windspeed which was computed at 10m. We need feedback from users about whether the 2m or 10m data is more useful for PV and CSP modeling.
"""

I'm getting 10m wind from ecmwf and still perceive this as a the most common height.

But I'm not really comfortable fixing a trivial spelling error myself when there may be more fundamental questions which remain open.

I don't see any need to hold off on fixing a spelling error, but of course feel free to submit a PR whenever you like!

I think we also need to add a reference to the PVWatts source code that Kevin provided above (or some other reference) as the source of the "PVWatts default is 9.144 [m]" because as far as I can tell this information's not contained in the existing two (2) reference docs.

Agreed it would be good to list a reference for the 9.144 value.  It goes all the way back to the second-ever commit in the SSC repository: https://github.com/NREL/ssc/blob/88207e11ee697210f721bdfaa556888f77c79560/src/lib_pvwatts.cpp#L222-L224
I wonder if it was inherited from PVFORM?  Unfortunately I've never been able to locate the source code for PVFORM; if anyone has a copy I'd be interested to learn what it used.
Regarding the NSRDB wind heights and how PVWatts should handle it: it's an interesting discussion, but better suited to continue discussion in that SSC issue.  Regardless of what comes out of that discussion, our goal with pvlib.temperature.fuentes was to replicate the behavior of PVWatts v5/v7, so I think we're going to keep the 9.144 default here.

I agree that we can fix that slight typo and then close this GitHub Issue and then continue the discussion in the other, more relevant GitHub repository. I'll add it to my to-do list to make that PR, but I must admit I'm quite busy and don't know when I'll get around to it, so please don't hesitate to do it without me (if so, please just post an update here, and I'll do the same if I get around to it first). Thanks.

The PVFORM source code that PVWatts was based on came from Sandia and was under some type of license, so my understanding is that  the agreement between the labs was that NREL couldn't share copies of it but could share the PVWatts code based on it. As was mentioned (in this PR or the linked SSC issue?), the new pvwattsv8 model no longer uses this temperature correction. The associated documentation should be coming soon, although a good chunk of the modeling steps are described in the SAM Photovoltaic Reference Manual since PVWatts is now using the same models for module DC, module temperature, and inverter behavior: https://www.nrel.gov/docs/fy15osti/64102.pdf

Does someone at Sandia still have the PVFORM code?  It would be interesting if pvlib could offer the capability to reproduce its output.

Thanks for linking that reference, Janine. I noticed it says: "SAM assumes that the hourly wind speed measurements from the weather file are in m/s and measured at a height of 10 m above the ground." However, as we now relearned from those past NSRDB webinar Q&As, starting with PSM v3 the wind speed height changed to 2 meters. (As you can see from what I pasted above, they solicited feedback from the PV community regarding this change.)

Here's an additional bit of information provided by André Mermoud, the primary author of PVsyst. Perhaps we need to open a GitHub Issue in the NSRDB GitHub repository if such a repo exists.
You are right, the wind velocity measured according to the WMO standards  (World Meteorological Organization) specify that the measurement should be done at 10 m height.
Source: https://forum.pvsyst.com/topic/3377-height-of-wind-speed-data-used-to-derive-pvsyst-thermal-parameters/


DX12: RenderDoc just displayed the first slice(depth) when rendering to 3D texture

I've encountered an issue that DX12 just displayed the first slice(depth) when rendering to 3D texture such as width(32)*height(32)*depth(2).
Note: I used VS, GS and PS which aimed to set slice index as SV_RenderTargetArrayIndex in GS. But, it wouldn't work well under DX12 mode. But on the contrary, DX11 and Vulkan worked well.
Thanks!

I can't reproduce this bug, or I'm misunderstanding. I got a simple capture with a 3D texture that's 32x32x2 and I'm able to switch between both depth slices fine.
Can you give more information about what exactly is going wrong? Are you able to share a program or capture file that reproduces it?

I'm sorry for the delay feedback because of busy these days. And I've attached two versions which aimed to illustrate the issue. The first one is for the DX11 which captured well for depth info, and the other is for the DX12 which lost depth info under 3D texture. Please check the files attached.
Note: There're two render pass:

Render to 3D texture which would cause this isse between DX11 and DX12;
Target as texture.

Thanks!
dx11_dx12_rdc.zip

Hey sorry for the delay. I think f33f9759703547db799dd90f04dd6172e6e813c1 should fix the problem.
Strangely I went back to my 32x32 and it still worked fine, although I don't know why :smile:. Either way, please check the next nightly build when you have a chance.

Closing this issue, it should be fixed in v0.34 which includes the above commit.


API is not responding (Error: Retry total timeout exceeded before any response was received)

Environment details

OS: win10
Node.js version: 10.14.1
npm version: 6.4.1
dialogflow version: 1.0.0
HTTP PROXY used

Problem describtion
The getAgent method is not responding any thing and eventually I will get a timeout error.
Although the API testing works good on DialogFlow api reference page, and googleapis/resource nodejs client works.
This works:
const { Resource } = require('@google-cloud/resource');

// Creates a client
const resource = new Resource();

// Lists current projects
resource
    .getProjects()
    .then(results => {
        const projects = results[0];

        console.log('Projects:');
        projects.forEach(project => console.log(project.id));
    })
    .catch(err => {
        console.error('ERROR:', err);
    });

This doesn't:
const dialogflow = require('dialogflow');

const formattedParent = client.projectPath('phonebook-ccpfxn');
console.log(formattedParent);
client.getAgent({ parent: formattedParent })
    .then(responses => {
        const response = responses[0];
        console.log(response);
        // doThingsWith(response)
    })
    .catch(err => {
        console.error(err);
    });

Error:
Error: Retry total timeout exceeded before any response was received
    at repeat (D:\GitHub\googleCloud\node_modules\google-gax\build\src\normalCalls\retries.js:80:31)
    at Timeout.setTimeout [as _onTimeout] (D:\GitHub\googleCloud\node_modules\google-gax\build\src\normalCalls\retries.js:113:25)
    at ontimeout (timers.js:436:11)
    at tryOnTimeout (timers.js:300:5)
    at listOnTimeout (timers.js:263:5)
    at Timer.processTimers (timers.js:223:10) code: 4

Appreciate any help!

👋 @wtflove so I don't think this client supports proxies out of the box unfortunately, I suspect that might be the root cause of what you're seeing. You can work around this by using the C core gRPC module instead.
$ npm i grpc

Then just import/pass it to the constructor
const grpc = require('grpc');
const {v2} = require('dialogflow');

const client = new v2.AgentsClient({grpc});

Let me know if this helps at all

@callmehiphop It works perfectlly! thank you!


Highlights with @ cause breakage

Plugin affected: none
Nixpkgs channel: unstable
Home-manager version: latest
Description
Error:
Error detected while processing /home/user/.config/nvim/init.lua:
E5113: Error while calling lua chunk: /home/user/.config/nvim/init.lua:224: Invalid 'name': Expected Lua string
stack traceback:
        [C]: in function 'nvim_set_hl'
        /home/user/.config/nvim/init.lua:224: in main chunk
Press ENTER or type command to continue

Init.lua snippet:
-- Highlight groups {{
do
  local highlights = {{["link"] = "DiagnosticError"},{["link"] = "DiagnosticHint"},{["link"] = "DiagnosticInfo"},{["link"] = "DiagnosticWarn"},["NormalFloat"] = {["bg"] = "NONE"},["Pmenu"] = {["bg"] = "NONE"}}

  for k,v in pairs(highlights) do
    vim.api.nvim_set_hl(0, k, v)
  end
end
-- }}

Config
...
  programs.nixvim = {
    plugins.comment-nvim.enable = true;

    highlight = {
<EMAIL_ADDRESS>= "DiagnosticInfo";
<EMAIL_ADDRESS>= "DiagnosticHint";
<EMAIL_ADDRESS>= "DiagnosticWarn";
<EMAIL_ADDRESS>= "DiagnosticError";
    };
  };
...

Indeed this was an annoying problem.
Every key which starts with an @ was removed (i.e. transformed to a simple element item.
The following nix attrs
{
  "key1" = 1;
  "@key2" = 2;
  "key3" = 3;
}

would have been converted to lua as follows:
{
  ["key1"] = 1,
  2,
  ["key3"] = 3,
}

This meant that the user couldn't have lua tables with keys that start with @.
In #632 we swapped the @ to __unkeyed which should not cause any clash.
-> Please, update nixvim and try again.


Regression with 1.7u40 and return type overriding

If you try to use dozer lib with 1.7u40, and special mapping like X.Y where X is a method with return type overriding. The Y cannot be found anymore with this java version.
It seems that the PropertyDescriptor property type does not return the overriden return type but the default abstrat method return type.

@LilianValero ; can you retest with latest JDK8. 7 is not supported anymore by Oracle, so is dropped by Dozer.

Closing due to inactivity.


properly animate the profile hover card

Why
It was pointed out that hover card animations on Firefox were not working very well. If you were scrolled down the page far enough, they would stutter. Turns out we had not properly implemented useFloating().
See https://floating-ui.com/docs/useFloating#open and https://floating-ui.com/docs/useFloating#ispositioned
Edit: Well, we don't actually need to use open or isPositioned, but this seems to give us a hint as to what might be causing the jank. The card starts off at 0,0 when it gets rendered, then moves to wherever it's supposed to be positioned. Moving the animation inside of the moving element fixes the jank.
Test Plan
Using Firefox, scroll down the page about 20 posts or so, and hover over an avatar. The animations should now be smooth.

will-change CSS property can be quite a footgun I wouldn't really recommend it if it works well without the property :sweat_smile:

will-change CSS property can be quite a footgun I wouldn't really recommend it if it works well without the property 😅

the library already adds it anyway so we're just making it consistent between first and next renders (the library was adding it too late)

(thanks for the tip tho, will forcefully remove willChange if it's too wonky)


Rancher's installation script is broken on Ubuntu 17.10

I would like to dump an issue over at Rancher's repo instead but they don't accept new issues lol.
Either way.. my requirements.yml:

- src: mongrelion.docker
  version: a7040aac63c8567048a3fbb6437199cca40457aa

playbook.yml:

roles:
- role: mongrelion.docker
  docker_version: '17.09'
  setup_script_md5_sum: 975145b3eeaf9efc588666bf46265d38
  vagrant: yes

This used to work just fine, but now it won't work anymore after I upgraded my Vagrant boxes to fso/arful64. I greatly suspect Rancher's installation script is broken for Ubuntu 17.10. The Docker installation fails when executing the script. I actually tried to run said installation script manually with the same result. These are the last 10 or so lines from the console output, and please note in particular the very last line:
[...]
+ sh -c apt-key add -
+ curl -fsSl https://download.docker.com/linux/ubuntu/gpg
OK
+ sh -c add-apt-repository "deb [arch=amd64] https://download.docker.com/linux/ubuntu artful stable"
+ [ ubuntu = debian ]
+ sh -c apt-get update
Hit:1 http://us.archive.ubuntu.com/ubuntu artful InRelease
Hit:2 http://security.ubuntu.com/ubuntu artful-security InRelease
Hit:3 http://us.archive.ubuntu.com/ubuntu artful-updates InRelease
Hit:4 https://download.docker.com/linux/ubuntu artful InRelease
Hit:5 http://us.archive.ubuntu.com/ubuntu artful-backports InRelease
Hit:6 http://ppa.launchpad.net/ansible/ansible/ubuntu artful InRelease
Reading package lists...
Building dependency tree...
Reading state information...
E: Version '' for 'docker-ce' was not found

Executing the "convenience scripts" provided today (2017-11-13) by https://get.docker.com (17.10.0-ce) and https://test.docker.com (17.11.0-ce-rc3, build 5b4af4f) works just fine.
So my current workaround is to switch the mongrelion.docker's setup_script_url to one of Docker's convenience scripts. I also set setup_script_md5_sum to false since these scripts are expected to change over time lol (thanx for adding that feature!).

@marcusianlevine can you take a look at this? this is your doing ;P

@MartinanderssonDotcom thanks for using our role and raising this issue!
Your workaround is actually the intended design of this role: to avoid maintaining bloated OS-specific installs within the role, we just wrap the convenience script of your choosing.
We chose to use Rancher's scripts by default because they maintain an archive of scripts for specific versions of Docker. By contrast the official Docker convenience scripts that you linked change over time as new versions of Docker are released to the stable and edge channels.
I don't think there's much we can do about this within the scope of this role, but according to the docs the latest version of Rancher (1.6) only officially supports up to Docker 17.06, so it's possible they aren't actively maintaining the 17.09 script yet.
Since there's logic in the role to accommodate this use-case, I think we can close this issue.

No, thank you! =)
I feel ya. But at least y'all have to toss in a note about this in the docs.

Also.. just as a general comment. I'm not sure I agree with the role being a "wrapper" on top of someone else's script. I mean, sure that's how it is implemented. I get it. BUT.. roles in general are supposed to get the job done, however that happens. Meaning that the role itself should fall back to or employ whatever script slash shell commands are necessary.
Maybe we should rebrand this issue as an enhancement request instead of closing it?

I think that @MartinanderssonDotcom makes a valid point when he says that the role should do whatever it takes to get the job done. The problem that we have at hand, though, is the underlying technology that we chose to get that job done, which has this very specific limitation as of right now.
The best solution would be to submit a PR to the upstream script for the Rancher folks.
Fixing this problem from within the role is just a hack and will probably lead to unclean code.
As per your suggestion, I think it's only fair with the users to toss that note in the docs.
@MartinanderssonDotcom could you please submit a PR with whatever you think is best in the docs?
@marcusianlevine or @MartinanderssonDotcom maybe you guys also want to look into the Github source of the Rancher script (if there is any?) and submit that PR ;P

Was digging into this some more, looks like the real issue is not with Rancher's install script but actually upstream in the Docker apt repositories: there is not yet a package published for Ubuntu 17.10 Artful.
That's why the error reads E: Version '' for 'docker-ce' was not found - there simply is no compatible version of the docker-ce package published on the official Docker apt repo.
@MartinanderssonDotcom don't mean to be dismissive of your issue, but we had this same problem with Ubuntu Zesty a few months ago until Docker released an apt package for it.
@mongrelion unless we intend to build Docker binaries from source within the role I'm not sure there's much we can do about this, other than add a note to the docs that we only support OSs with an official Docker package on apt or yum.
On a related note, our meta.yml doesn't list Zesty as supported yet, and there is now a Zesty package published, so we could update that.

Hmm I was trying to figure out how Docker's convenience scripts work.. looks like they use a different apt repo than the Rancher setup scripts, with packages for more recent OS releases...
Still requires an upstream fix to Rancher's setup scripts, but should only affect Ubuntu 17.10, not lower versions...

other than add a note to the docs that we only support OSs with an official Docker package on apt or yum
Sounds about fair

tried with Vagrant locally and still doesn't seem to work though
:(

So correct me if I'm wrong: there is no solution just yet for getting the role to work on Ubuntu 17.10 since there is no official Docker package for it just yet. Correct?

@mongrelion it's a little more complicated than that
Docker's official apt/yum repos support up through Ubuntu 17.10 Artful, but only offer the latest stable and edge releases of Docker (right now 17.09 and 17.11, respectfully). This repo is used by the official Docker convenience scripts, which do work to install the latest versions of Docker on Ubuntu 17.04 Zesty and 17.10 Artful.
So as @martinanderssondotcom said, if you specify the official Docker convenience scripts with setup_script_url, you can install the latest official releases of Docker on Ubuntu Zesty and Artful
However, the Rancher convenience scripts that we use by default pull from a less-frequently-updated apt repo which contains releases for many versions of Docker on all of our currently supported OS distros and versions.
Weirdly, according to the filelist for Zesty, there is only an installer for Docker 17.05, which is why the Zesty install fails with our current default of 17.06
I think the simplest solution that addresses the most use-cases is to switch the default setup script to Docker's official convenience scripts.
This will remove the need to manually upgrade the default version - we could even have docker_version be used to specify stable or edge instead of an absolute version number. Since most people who need a specific version of Docker will know what they are doing, it's not such a big deal to have to look up a Rancher setup script and calculate the checksum.
The only downside I see is that we will lose some determinism: we could experience problems with the role related to a botched release of the Docker convenience script or Docker itself. Also, we will not be able to reproduce test results based on the official convenience script, since it changes over time and there is no archive.
However, this could be mitigated in our to-be-written integration tests by using the Rancher scripts on known working OSs and only fall back to the official script if they fail.
The ultimate solution would be to implement our own install logic rather than relying on third-party scripts, but as we've discussed previously this would require a significant amount of new work to cover the OS versions and distros we currently support.
To be clear: there is a functioning workaround for this issue, by using the official Docker convenience scripts

@marcusianlevine at this point I'm not really sure which way to go.
Rewriting the whole thing can be expensive. Not supporting the latest Ubuntu version could also annoy people but so far we only have one complaint.
It's been about a month already. Is this not yet supported by the Rancher scripts? #wishfulthinking


Read network configuration from a file on azure stack

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

PR looks good except for minor comments. Can we have few unit tests with multi nic configuration (both sdn and local). One will be primary and rest are secondary

@t-chappl  Can you add unit tests that I mentioned ?

@tamilmani1989 did you see the new test I pushed this morning? It has multiple hardware and sdn interfaces. Are there other cases you would like me to cover?

@tamilmani1989 I initially test the scenarios only on linux but I will try them out on windows right now. Sorry, I should have done it sooner.

@tamilmani1989 I tried the scenarios on windows and they worked fine.


Add support to MS SQL Server without ODBC

Add support to Django MSSQL Database Backend a minimal wrapper for django-mssql to make it work with python-tds.

I realized that the package is very old.


Update Monix to 2.2.1

Current coverage is 97.35% (diff: 100%)

Merging #177 into master will not change coverage

@@             master       #177   diff @@
==========================================
  Files            28         28
  Lines           831        831
  Methods         783        783
  Messages          0          0
  Branches         48         48
==========================================
  Hits            809        809
  Misses           22         22
  Partials          0          0

Powered by Codecov. Last update d8f70c0...044b007


Ported hello-neon sample to master-cmake branch

Had a discussion with ggfan@ to ignore formatting for now. LGTM.


CloudFormation & Terraform: always rely on SecretsManager to store API key

Rebased. Could you re-approve @Bit-Doctor ?


[$500] Task - Task is not marked as complete in user WS employee B (not task creator, not admin)

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.37-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4282503
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition
• A workspace admin
• A workspace employee A (task creator)
• A workspace employee B
All accounts should be added to the workspace of the workspace admin

Go to https://staging.new.expensify.com/
Login as the workspace employee A
Navigate to the workspace #Announce room,
Create a task
Login as the workspace employee B in incognito tab
Navigate to the workspace #Announce room
As user A complete the task
As B observe the task
As user A incomplete the task
As B observe the task

Expected Result:
Task is marked as complete in user WS employee B (not task creator, not admin) of the WS #Announce room when it was marked as complete by WS employee A (task creator, not admin)
Actual Result:
Task is not marked as complete in user WS employee B (not task creator, not admin) of the WS #Announce room when it was marked as complete by WS employee A (task creator, not admin)
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/097fa226-bf12-4412-9c33-801ce16eae43
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~012c10fd1b691a5068
Upwork Job ID:<PHONE_NUMBER>617296384
Last Price Increase: 2024-02-06

We think that this bug might be related to #vip-vsp
CC @quinter

We think that this bug might be related to #vip-vsp CC @Quinter

what does #vip-vsp mean?

Maybe we can make this weekly. Still awaiting proposals though!

Demoting to FUTURE as it involves tasks, which we aren't prioritizing (but if we can get a contributor to do it, go for it!)

Not reproducible anymore
https://github.com/Expensify/App/assets/102477862/9ca01382-eb9d-4c95-9c0b-e81ad5b3f534


Don’t auto sign folks up with social email.

We don’t support signing people up with social accounts, so this should be turned off.

@ericholscher It looks like this might fix #1875, correct? For those of us who've already run into the issue and now cannot connect GitHub accounts, is there a manual fix?

@jonafato if this is the fix, we'll need to go back, manually correct any user issues, and see if they reproduce the original issue. Once we get it deployed we'll check in with some folks

I think you should be able to login with your GitHub account and access the
automatically created account through this URL:
https://readthedocs.org/accounts/github/login/?process=login
That should allow you access to the project created.
On Thu, Mar 10, 2016 at 10:10 AM, Anthony<EMAIL_ADDRESS>wrote:

@jonafato https://github.com/jonafato if this is the fix, we'll need to
go back, manually correct any user issues, and see if they reproduce the
original issue. Once we get it deployed we'll check in with some folks
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/pull/2052#issuecomment-194981816
.

--
Eric Holscher
Maker of the internet residing in Portland, Oregon
http://ericholscher.com

I was able to fix my accounts using the steps provided by @dirn. @ericholscher, @agjohnson thanks for the support on this issue.


[Modal] Inspect error with responsive size

When setting responsive and the screen size goes too small the StoryBook app crashes.
It happens on both Modal and ModalWithTabs components.

This error is generated by StoryBook only in build mode:
chunk-UBOPZ7EI.js:398 Uncaught TypeError: Cannot read properties of undefined (reading 'reduce')
    at chunk-UBOPZ7EI.js:398:31215
    at memoizerific (chunk-3J6GS6FI.js:1:3530)
    at reduce.orphansFirst (chunk-UBOPZ7EI.js:398:44168)
    at Array.reduce (<anonymous>)
    at chunk-UBOPZ7EI.js:398:44112
    at mountMemo (chunk-NBCGHFLK.js:69:12247)
    at Object.useMemo (chunk-NBCGHFLK.js:69:19183)
    at useMemo (chunk-QMHPSTMR.js:9:1378)
    at chunk-UBOPZ7EI.js:398:44081
    at renderWithHooks (chunk-NBCGHFLK.js:69:936)


Preload is partially supported in Edge 17+

Similar to preconnect (https://github.com/Fyrd/caniuse/pull/3829), Edge 17 only has partial support for this feature.

Thanks!


C++ header only version

Hi, excellent lib, clean and sober code, really good this is. Now.
I am wondering if you could consider cpp header only version?
Yes, I know that would be a "point-of-no-return" to C ..

printf++

Hi @DBJDBJ,
thanks a lot for your feedback.
Actually I wouldn't create a header only version of printf cause I don't see any big advantages.
Don't get me wrong here, I'm a huge fan of c++ header only libs (just see my other projects like 'vic' here).
I'm using printf in my heavily header only c++ turnkey-board project (not public yet).
IMHO header only files only makes sense if c++ code can be inlined very well, is kind of templated and is class based - which is printf not.
You can use printf in c++ projects without any problems. Linking and compiling printf.c shouldn't be a problem.
However, I see you created a header only c++ version. So far so good, but I don't like the idea of wrapping all routines as extern "C" inline inside a mpaland_dbjdbj namespace at all. 😞  You should do better!
Problem is that compiler puts all functions in every single module which bloats your code.
My idea would have been to create a printfpp class instead of a namespace and create static functions inside. Using printfpp::printf() directly then.
That way a compiler/linker can use the static class functions for optimization very well. Try it if you like.

Thanks, Marco, I shall give it a due diligence as soon as I have time ...
Just a quick clarification, if I may:  you do not like them inline functions being extern "C"? or you do not like them being inline?
std:: namespace is header only lib with a lot of inline functions?
Kind regards ...

Hi Dusan,
I don't like inlining a whole lib, but when using a namespace the way you did, it's the only option.
Anyway, to create a header only lib you just have two options (as far as I know): inline everything or create a class which is suitable for static use.
First option may be used in C or C++ while the second is C++ only. So, if you want to create a C++ header only lib, I would always prefer option 2, like:
class printfpp
{
private:
  // 'inline' here, to give the compiler a hint that inlining would be cool
  static inline bool _is_digit(char ch)
  {
    return (ch >= '0') && (ch <= '9');
  }

  // internal itoa for 'long' type
  static size_t _ntoa_long(out_fct_type out, char* buffer, size_t idx, size_t maxlen, unsigned long value, bool negative, unsigned long base, unsigned int prec, unsigned int width, unsigned int flags)
  {
    :
  }

public:
  static int printf(const char* format, ...)
  {
    va_list va;
    va_start(va, format);
    char buffer[1];
    const int ret = _vsnprintf(_out_char, buffer, (size_t)-1, format, va);
    va_end(va);
    return ret;
  }
};

This is way cleaner than using a namespace with inlined functions.
You don't need extern "C" for linkage in a c++ project.
You are right, there are conflict problems when linking this lib with the standard lib. I can't link it native in VC projects either.
I have a tested solution for this, but it's not commited yet, issue #16 is still open for this fix.
Trying to do this this week.
Best regards!

Ok then, my printf++.h is a TINY change of your printf.c, it is C++17 and it works rather brilliantly thanks to you.
I will take the risk of using it.  It is perfectly usable in any kind of VS2017 C++ project right now. (I shall remove extern "C" of course.)
Please let me know if you agree with the choice of the namespace name, MIT license etc. Marco,  your comments are thought-provoking. Please read further only if you have time.
The conceptual debate :)
I am unable to go against std::  design patterns & concepts, where 'inline' is deliberately sprinkled arround, in concert with 'free' functions, operators and C++17 inlined globals.
Also, I know more than one member of the good MSVC team who will strongly oppose the claim that  CL.exe "seeing" inline function is bloating the code :) I dare to write ...most likely, you're worrying over nothing, for several reasons:

The modern C++ compiler treats inline as a hint. The decision rests with the compiler itself, and it tries to make the application fast. CL compiler will try to optimize more towards a smaller file size.
In case of CL you are looking at a couple of kilobytes (16kb increments). In an age of terabyte harddrives.
if exe size does become a problem, all modern c++ compilers typically have an "optimize for size" flag.
large executables typically gain their size because of a lot of data and resources. The code itself is very rarely the problem. Even due to template metaprogramming.

Thank you for percevering throgh my comment ...

I think, anything is said, so I close this issue.


mriqc 23.0.1: not working with  sessions

What happened?
just updated to 23.01 in the Ihope this solved the issue of mriqc not reading a valid bids directory:
singularity build mriqc_23.1.sif  docker://nipreps/mriqc
It did not run on a bids directory with sessions, but ran successfully after creating a directory without sessions.

Singularity
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
output when running on a bids directory with sessions:
/mnt/scratch/VIA_BIDS/mriqc/bids_data_DRCMR_mriqc_mprage /mnt/projects/VIA_BIDS/scripts/mriqc /mnt/scratch/VIA_BIDS/mriqc/DRCMR_work_mriqc_mprage 2 DRCMR
sub-via025 ses-01DRCMRprisma
usage: mriqc [-h] [--version] [-v] [--species {human,rat}]
             [--participant-label PARTICIPANT_LABEL [PARTICIPANT_LABEL ...]]
             [--session-id [SESSION_ID ...]] [--run-id [RUN_ID ...]]
             [--task-id [TASK_ID ...]] [-m [MODALITIES ...]] [--dsname DSNAME]
             [--bids-database-dir PATH] [--nprocs NPROCS]
             [--omp-nthreads OMP_NTHREADS] [--mem MEMORY_GB] [--testing] [-f]
             [--pdb] [-w WORK_DIR] [--verbose-reports] [--write-graph]
             [--dry-run] [--resource-monitor] [--use-plugin USE_PLUGIN]
             [--no-sub] [--email EMAIL] [--webapi-url WEBAPI_URL]
             [--webapi-port WEBAPI_PORT] [--upload-strict] [--notrack]
             [--ants-float] [--ants-settings ANTS_SETTINGS] [--ica]
             [--fft-spikes-detector] [--fd_thres FD_THRES] [--deoblique]
             [--despike] [--start-idx START_IDX] [--stop-idx STOP_IDX]
             bids_dir output_dir {participant,group} [{participant,group} ...]
mriqc: error: Querying BIDS dataset at </data_in> got an empty result.
Please, check out your currently set filters:

 *participant_label: ['via025']
 *session: ['ses-01DRCMRprisma']
 *run: None
 *task: None
 *bids_type: None

real	0m1.957s
user	0m0.530s
sys	0m0.520s

Additional information / screenshots
No response

Does it work without setting --session-id?

Hi Oscar,
Many thanks for your response!
I did not try that. I just generated a temporary bids structure without a session subdir and successfully ran mriqc on that one. So in principle, I am good. However, I would think the sesion-id setting should work in case of longitudinal data and one wants to single out a specific session.
I am a bit hung up with multiple datasets and analyses to try it out now...... Maybe, If I can find the time :-)
Cheers

if you can send me the output of a simple find on the folder structure, I can add that manifest to our currently tested ones and we will know for sure whether the problem is indexing or the session filter of the command line.

The directory structure that I tried the "-- session-id" flag on and did not work looks like:
dataset_description.json
README
sub-via0998
sub-via0998/ses-01CFINskyra
sub-via0998/ses-01CFINskyra/anat
sub-via0999
sub-via0999/ses-01CFINskyra
sub-via0999/ses-01CFINskyra/anat
I moved the files to a structure, on which I successfully ran mriqc without the "--session-id" flag
dataset_description.json
README
sub-via0998
sub-via0998/anat
sub-via0998anat/sub-via0998_ses-01CFINskyra_acq-mp2rage_rec-moco_run-01_T1w.json
sub-via0998/anat/sub-via0998_ses-01CFINskyra_acq-mp2rage_rec-moco_run-01_T1w.nii.gz
sub-via0999
sub-via0999/anat
sub-via0999/anat/sub-via0999_ses-01CFINskyra_acq-mp2rage_rec-moco_run-01_T1w.json
sub-via0999/anat/sub-via0999_ses-01CFINskyra_acq-mp2rage_rec-moco_run-01_T1w.nii.gz
Now I look at it ....  it is maybe a bit strange that the latter ran successfully, as the ses- pair in the file name is not matched by a "session-id" in the directory name ....
Anyway,  it worked

sounds like I have been lucky then :-)

Yes, pretty good. However, that image looks more like a classic MPRAGE, rather than the MP2RAGE with the echoes combined.

Maybe it is in the docs, but does mriqc down sample to 2mm iso?

No, it runs a quick and dirty affine registration with MNI152NLin2009cAsym. For that, it uses the 2mm version of the template (saving memory and time at the cost of accuracy)

synthstrip seems also to work nicely on non background denoised mp2rage data.
I ran it with (purple mask) and without the  --no-csf flag

I believe this is finished. Please reopen if I'm wrong.


Tag devscripts run as devscripts_layout

Instead of libvirt_layout, use devscripts_layout.
Signed-off-by: James Slagle<EMAIL_ADDRESS>As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

@slagle Hi mate, can we close this one out in favor of #1714 ?

@slagle Hi mate, can we close this one out in favor of #1714 ?

Closing


Console Redesign Findings

I just tested the redesign of the console and this are my findings:
Acceptance criteria
UI Consistence

[ ] Not all the tables behave simliar, eg. On some tables i can click on the row, some don't, some have an edit icon at the end some don't
- Projects: No Edit Button, Per click to detail
- Roles: Edit Button, Click on row shows detail
- ProjectGrants: Edit Button, Click on row shows detail
- Users: No edit button, per click to detail
- Authorizations: Edit button, click on row does nothing
- Membership on org: No edit button instead plus, click on row redirects to user
- Memberships on user: Arrow instead of edit, click on row does nothing
[ ] We still have some parts with auto saving (e.g project) and some without (e.g oidc config, users)

Navigation

[ ]  I don't get why we have sometimes side nav (users) and sometimes the nav in the top changes (projects), for me the second top nav feels strange because I can't get to the items I had before (@juergrinaldi what do you mean about this?) This happens when I am on a Project Detail. I don't habe the main nav (Organization, Projects, Users, etc..) On the app detail no nav at all is existing
[ ] I think the system part in the navigation is confusing if the users don't have system. (Or do only IAM admins see this part of the nav?)

Organisation:

[ ] If I click on a manager of an external org, this doesn't work. Maybe we should not allow redirects to external users (disable row)

Projects

[ ] Why is applications not a separat section?
[ ] If I am on the authorizations tab it is  confusing how to get back to the projects list
[ ] For me the project grant detail is confusing (graphic), for me it is not really clear that this is the detail, and I can edit the roles, maybe its because we have a complete different view here

Apps

[ ] Urls should be loger, I can see them if I want to know what it is, I need to copy and paste somewhere

Authorization

[ ] The Detail of an authorization is confusing, I didn't get how to change it. (@juergrinaldi  do you have some input here?, or is it just me)

UserMemberships

[ ] Behaviour is strange, if I click edit and go back I land on a completely different page (Organization)

Actions

[ ] It is not possible to remove one trigger from a flow

If a user doesn't have iam.write permission than the breadcrumb and button to switch to system settings are not visible.. I guess the system breadcrumb was a good suggestion of @fforootd because it displays the hierarchy of the settings.
I think the new nav takes a little to get used to because we had a global navigation in the first place.. But once you understand how the breadcrumbs work you can quickly navigate back to the upper layers.

applications are not in a separate navigation item because theres enough space on the project detail and quick access to an application is more important IMO.


start cron on management startup (#14)

Fixes #14
@nbuttdwd let me know if this works for you, at which point I will merge this PR and close #14.  Thanks!
cc @antje-s

Hello Tom,
when the cronjob ran following error occured as output instead of creating the zip file:
Traceback (most recent call last):
File "/usr/local/bin/wis2-gdc", line 11, in
load_entry_point('wis2-gdc', 'console_scripts', 'wis2-gdc')()
File "/usr/lib/python3/dist-packages/pkg_resources/init.py", line 490, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "/usr/lib/python3/dist-packages/pkg_resources/init.py", line 2854, in load_entry_point
return ep.load()
File "/usr/lib/python3/dist-packages/pkg_resources/init.py", line 2445, in load
return self.resolve()
File "/usr/lib/python3/dist-packages/pkg_resources/init.py", line 2451, in resolve
module = import(self.module_name, fromlist=['name'], level=0)
File "/app/wis2_gdc/init.py", line 24, in
from wis2_gdc.registrar import register, setup, teardown
File "/app/wis2_gdc/registrar.py", line 36, in
from wis2_gdc.backend import BACKENDS
File "/app/wis2_gdc/backend/init.py", line 23, in
from wis2_gdc.backend.ogcapi_records import OGCAPIRecordsBackend
File "/app/wis2_gdc/backend/ogcapi_records.py", line 27, in
from wis2_gdc import env
File "/app/wis2_gdc/env.py", line 66, in
raise EnvironmentError('Environment variables not set!')
OSError: Environment variables not set!
We took a look at env.py, we have checked the variables and they have been set. We added a print in the script to see which variables are not set or are None:
('HOME', '/root')
('LOGNAME', 'root')
('PATH', '/usr/bin:/bin')
('SHELL', '/bin/sh')
('PWD', '/root')
('LC_CTYPE', 'C.UTF-8')
Could you take a look?

Thanks @nbuttdwd.  The wis2-gdc environment is looking for various environment variables to be set (see https://github.com/wmo-im/wis2-gdc/blob/main/wis2-gdc-management/wis2_gdc/env.py#L61-L62) as part of the runtime.
Having said this, pushed an update just now to this PR.  Let me know if that helps.  Thanks.

Hello Tom,
thanks - it worked now!


udemy videos cannot download

youtube-dl -u  -p  -o '/root/%(playlist)s/%(chapter_number)s - %(chapter)s/%(title)s.%(ext)s' https://www.udemy.com/course/aws-certified-solutions-architect-associate-saa-c02/learn/lecture/13528014  --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-u', u'PRIVATE', u'-p', u'PRIVATE', u'-o', u'/root/%(playlist)s/%(chapter_number)s - %(chapter)s/%(title)s.%(ext)s', u'https://www.udemy.com/course/aws-certified-solutions-architect-associate-saa-c02/learn/lecture/13528014#learning-tools', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 2.7.5 (CPython) - Linux-3.10.0-327.el7.x86_64-x86_64-with-redhat-7.2-Maipo
[debug] exe versions: none
[debug] Proxy map: {}
[udemy:course] Downloading login popup
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 634, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2288, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib64/python2.7/urllib2.py", line 437, in open
response = meth(req, response)
File "/usr/lib64/python2.7/urllib2.py", line 550, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib64/python2.7/urllib2.py", line 475, in error
return self._call_chain(*args)
File "/usr/lib64/python2.7/urllib2.py", line 409, in _call_chain
result = func(*args)
File "/usr/lib64/python2.7/urllib2.py", line 558, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Duplicate of #29915, and see also #29924.

403 can be fixed if you use cookies from browser instead of giving plain credentials and make URL look like https://www.udemy.com/aws-certified-solutions-architect-associate-saa-c02/ by removing /course/ part.
After that youtube-dl stumbles upon
youtube_dl.utils.RegexNotFoundError: Unable to extract course id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

403 can be fixed if you use cookies from browser instead of giving plain credentials and make URL look like https://www.udemy.com/aws-certified-solutions-architect-associate-saa-c02/ by removing /course/ part.
After that youtube-dl stumbles upon youtube_dl.utils.RegexNotFoundError: Unable to extract course id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see https://yt-dl.org/update on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Hi ku-dmitry... cookies method is still showing 403 forbidden error.

I'll need the initial command and output to try to find out what's wrong

I'll need the initial command and output to try to find out what's wrong

youtube-dl --cookies cookies.txt --verbose https://www.udemy.com/learn-jmeter-from-scratch-performance-load-testing-tool/learn/lecture/12926938#overview
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--cookies', 'cookies.txt', '--verbose', 'https://www.udemy.com/learn-jmeter-from-scratch-performance-load-testing-tool/learn/lecture/12926938#overview']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.9.2 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg git-2020-05-01-39fb1e9
[debug] Proxy map: {}
[udemy:course] learn-jmeter-from-scratch-performance-load-testing-tool: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\site-packages\youtube_dl\extractor\common.py", line 634, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\site-packages\youtube_dl\YoutubeDL.py", line 2288, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 523, in open
response = meth(req, response)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 632, in http_response
response = self.parent.error(
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 555, in error
result = self._call_chain(*args)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 494, in _call_chain
result = func(*args)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 747, in http_error_302
return self.parent.open(new, timeout=req.timeout)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 523, in open
response = meth(req, response)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 632, in http_response
response = self.parent.error(
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 561, in error
return self._call_chain(*args)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 494, in _call_chain
result = func(*args)
File "C:\Users\amitf\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 641, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)

I'll need the initial command and output to try to find out what's wrong

I have used Get cookies.txt chrome extension to export cookies and kept that file in the current directory from where I am running a command.

After that youtube-dl stumbles upon
youtube_dl.utils.RegexNotFoundError: Unable to extract course id; ...

See #30719.

I'll need the initial command and output to try to find out what's wrong

I can share my credentials as well if you want.

403 most likely refers to the fact that the script was unable to login. Try grabbing cookies some other way or you may want to try out a youtube-dl's fork which automatically extracts cookies from browser.
Whichever option you stick with - you'll need to manually apply #30719 in order to grab videos. It works for me. But for some courses it struggles to find a majority of videos.

Sure... Thank you for your quick response. I will give it a try and let you know...

403 can be fixed if you ... make URL look like https://www.udemy.com/aws-certified-solutions-architect-associate-saa-c02/ by removing /course/ part.

This seems to be because modifying the URL causes it to be redirected to the original URL, in the process generating whatever headers are required to avoid 403 on the original URL.


Cors

@Andrew-Sem can you pull this branch and test for cors?

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.45 :tada:

Comparison is base (3c0524b) 76.09% compared to head (9fc236d) 76.55%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #22      +/-   ##
============================================
+ Coverage     76.09%   76.55%   +0.45%
- Complexity       82       84       +2
============================================
  Files            24       25       +1
  Lines           205      209       +4
  Branches          6        6
============================================
+ Hits            156      160       +4
  Misses           46       46
  Partials          3        3

Impacted Files
Coverage Δ

.../ru/volpi/qaadmin/config/ApiCorsConfiguration.java
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


"No translations found" warning for default language

We see "No translations found for en_US" warnings, but en_US is the language of the code. So why should we translate it? Did I configure something wrong or is this a bug?
Using vue-gettext in version 2.0.23.

Update: fixed in #62.


Canonicalization of Stream Name (ie, allow uppercase, numeric, invalid string names)

Motivation
Stream names are not required per the Singer Spec to conform to anything other than to be "string"s...
Unfortunately for targets, this means that support for various taps requires supporting many varying formats etc.
Hacks and workarounds exist for this, but if a tap has a stream name which is upper case, often the user cannot edit/update this without forking the repo etc.
For those interested, please vote 👍 on this description for interest in this feature.
What we currently do
Presently, any string is allowed for any subtable. This means that if we detect a "nested table" in the data, we will create a "sub-table" where we canonicalize the "path" to that particular field.
ie, if we have a stream foobar which has an array field called B@z, that means the path to the array field is ("foobar", "B@z"). This path will be canonicalized as foobar__b_z.
The notable exception for this, is that stream names themselves, ie the "root" table name, must be strings which are:

lowercased
start with a letter
contain only alphanumeric characters and _ underscores

Why we do what we currently do
This logic was introduced in #60. This PR sought to allow for sub tables to be reasonable names derived from the data/field names provided.
At the time, this was because it was viewed that the data which comes out of a tap is well out of the user's control, but that often the stream names are already of this format (ie, many API based taps stream's are lowercase etc.).
An attempt was made to make it such that stream names could be of any string format. However it was more work to do that at the time.
Proposals
Table name mappings are currently stored on the "root" table in a db. ie, the stream name table. These mappings take a path to a canonicalized name.
ie, (a, b, c, d) -> a__b__c__d etc.
Dedicated Metadata Table
We could move this mapping out of the JSON blob stored in the table's comment and into a dedicated table. Something akin to _datamill_table_metadata.
Load Table Mappings
Each table's metadata could store the path. When target_postgres starts, it can list all tables present in the remote, iterate through them and fetch the raw_name thus building up a cache in memory of our path to canonicalized name.
This could even be done via sql with something like:
SELECT CAST(obj_description(oid) AS json)->'path'
FROM pg_class
WHERE relkind = 'r'

Suggested Musical Pairing
https://www.youtube.com/watch?v=crjugtkXZN4

Test Cases
Below are some things I think we should be concerned about with whatever solution we choose. @awm33 if you could weigh in with your gut reaction to how you think each situation should behave:
Non-Standard Stream Name
Stream comes in with a name like x, produces a root table with the name y?:

`` : Empty string -> ???
streaM : Contains capital letters -> ???
Stream : Starts with capital letters -> ???
xxxx...xxx : Extremely long field (longer than the max identifier length) -> ???
@#%^&*(( : Non alphanumeric, underscor, dollar sign -> ???

Non-Standard Stream Name, Nested Data
Stream comes in with a name like x and nested data, produces a child tables with names like y__?:
Colliding Stream Name
Stream named x comes, another stream named X comes. These produce ???
ie, do we get two distinct root tables, or do we let them populate the same remote table? If we make them distinct, what are they named?
Colliding Stream Name, Nested Data
Stream named x comes with nested fields a and c, another stream named X comes with nested fields b and d. These produce ???
Proposed Results

Non-Standard Stream Name

Follow conventions set by nested fields/child tables:

Empty -> _
Capitals get lowercased
Long identifiers get truncated
Non alphanumeric, underscore, dollar sign -> _

Nested fields/data

Whatever the root table's name ends up being, we use that as the prefix with __ separating

NOTE: for long identifiers, this can potentially result in incredibly ugly/unusable table names etc.

Colliding Stream Name

follow the conventions by other identifiers and append an integer till we get a unique value

NOTE: This will make nested fields etc. potentially very confusing

Nested fields/data

same solution as above with the root table name being what we prefix with

Proposed Solution
After having had time to think about this, I don't think we have the information stored in a format presently which makes modifying this particularly convenient. ie, we're not storing the path to tables in the metadata already.
I think the simplest solution for us, is probably to go with the Load Table Mappings option from above. However, this will required a schema bump, and migration etc.
The advantage to this approach is that adding and managing table mappings becomes really straightforward. The disadvantage (potentially) is around managing the mappings when there are a lot of resultant tables in a single Postgres schema.
That all being said, I think a spike to investigate this is worth it
Proposed Hack
Sooooooooo...a route we could go here is to provide a config option for dangerous__canonicalize_stream_name. This would just disable the validation on the stream name, and then canonicalize it etc. I'm not sure how much work will be necessary for this, nor am I sure whether this can be even remotely forward compatible...

An issue that's popping up with this change is how we're currently handling table versioning stuffs. That logic has been shaky/known to be a bit bug ridden for a while due to it relying upon postfixes and whatnot. I think that if we're going to invest more time into this procedure, we'll also need to invest time into shoring that logic up and making it a bit more airtight.
This more than the other schema changes, could present a problem to existing schemas. If an activate version command has not yet come in, and then we decide to run this migration etc., we could easily break things etc.

Also, utilizing casting to json will not work with native 8.4.22. Redshift however has a bunch of json support baked into it...going to look into what all we can and cannot do across versions. Additionally, we may have to install some extensions etc. into our 8.4.22 tests.

Redshift supports the common PSQL function JSON_EXTRACT_PATH_TEXT.
This is not available in 8.4.22...but...the simplest thing to do might be to make a function in 8.4.22 which is called that, but really just takes text, a field, and returns a string array. That way it'd behave like the actual call and response we expect, without us having to ditch the useful testing of 8.4.22 in lieu of more costly/time-consuming Redshift calls etc.

@gpetepg I think you were the one originally pushing for this functionality. The PR #131 should address this. If you have any interest in looking that over for the functionality you're after, that'd be 👍

@AlexanderMann awesome! I am currently using a fork of the PR but will test this out asap and ping you in the slack channel. Haven't had as much time to use Singer as my project got shelved.


Changes to support ppc64le

Add support for IBM Power Linux platform, build and all unit tests are passing.

Reference: https://github.com/basho/leveldb/issues/237


[stable/fluent-bit] Add priorityClassName

What this PR does / why we need it:
It allows to set a priorityClassName.
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] title of the PR contains starts with chart name e.g. [stable/chart]

/assign @kfox1111

/assign @kfox1111


[BUG] mapWidthsToSrcSet generating invalid DIS urls

The DynamicImage component can produce DIS links that are not working due to DIS limitations. This is caused by mapWidthsToSrcSet which calculates the widths. DIS has a size limitation of 3000px this can easily be exceeded by devices with high pixel density or ultra wide screen devices if the passed width is 100vw.
A simple Math.min resolved the issue for us.

Hi @ecRobertEngel , what are some examples of links that did not work for you? Thanks.

I have manipulate a sample product image from the Demo Store.
This URL is not working: https://edge.disstg.commercecloud.salesforce.com/dw/image/v2/ZZRF_001/on/demandware.static/-/Sites-apparel-m-catalog/default/dw6a5ac15c/images/large/PG.10226236.JJGN9A0.PZ.jpg?sw=3072&q=60
This one is:
https://edge.disstg.commercecloud.salesforce.com/dw/image/v2/ZZRF_001/on/demandware.static/-/Sites-apparel-m-catalog/default/dw6a5ac15c/images/large/PG.10226236.JJGN9A0.PZ.jpg?sw=3000&q=60
We had this issue happen with the dynamic image component and the parameter 100vw on MAC devices with retina displays.


Close the body of a stream response after sending

By closing the stream manually we make sure any streamwrapper for the stream has the chance to handle the fclose.

Interesting. Do you have a use-case?

We're running into issues where, from what we can tell, the stream we return to sabre for handling a GET request is only closed when php is tearing down causing unexpected behavior because objects no longer exist.
By closing the stream earlier the fclose can be properly handled before php starts tearing down

Basically we have a stream wrapper that waits for fclose() to happen and then do something (releasing locks). But if the fclose happens too late because it was done by the GC, the operation we want to do might not have access any more to the other objects because they were GC'd already too.
So far we worked this around by implementing "afterResponse" in a plugin and closing the stream.

What the pros compared to closing the stream manually?

Because I don't really see a clear drawback to this, I'll accept the patch. But new information in the future might change this again (but I hope not)


Force worktree add

Resolves https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/41
I also came upon this issue while playing with the semconvgen package and tried to patch it.
More on the -f param:
https://git-scm.com/docs/git-worktree#Documentation/git-worktree.txt---force

@unicod3 I am not sure it if is needed. I guess I encountered the same issue here: https://github.com/open-telemetry/opentelemetry-go/pull/4078#discussion_r1188555233
I am not sure if -f does always what we want:

[...] if  is already assigned to some worktree but is missing (for instance, if  was deleted manually). This option overrides these safeguards.


[incubator/vault] Add ability to define side containers

What this PR does / why we need it:
This PR enables the addition of sidecar containers to the vault pod. A use for a sidecar container could include an unsealer, where any unseal logic can be done in a separate container that lives outside of the official vault image. This allows for separation of concerns where tasks that don't necessarily belong in the vault container could live along side in the same pod and share the same network layer.

/assign @viglesiasce

/assign @unguiculus

/assign

/assign @seanknox

/assign @mattfarina

cc @jpds

@unguiculus I see how the naming scheme and keeping extraContainers more flexible (allowing more than one container) makes sense. I was originally thinking of taking this route.
Looking how the customSecrets are currently defined:
https://github.com/kubernetes/charts/blob/0e79b3de568dab9fcab0eb5d38ecc94b27a9afc3/incubator/vault/values.yaml#L109-L111
Wouldn't it make sense to change that to extraVolumeMounts instead? Secrets in this context are nothing more but extra .spec.containers.volumeMounts?

/ok-to-test

/lgtm


Export handler types in TypeScript

I don't like having a huge file with all the bot handlers and their logic; I like to split them to different files so I can better handle and maintain them. My goal was to have something like this:
const app = new App({ ... });

app.action('some_action_id', handleAction);

I could create a function that takes any, however I would like to use typed functions. Since the handler types are not exported from the SDK, I can't achieve that. Instead, have to use nasty tricks to make it work, like Parameters type alias.
So, my solution for now is simply defining the types myself with the Parameters type alias:
const app = new App({ ... });

export type Middleware = Parameters<typeof app.use>[0];
export type EventHandler = Parameters<typeof app.event>[1];
export type ActionHandler = Parameters<typeof app.action>[1];
export type MessageHandler = Parameters<typeof app.message>[0];
export type ShortcutHandler = Parameters<typeof app.shortcut>[1];
export type CommandHandler = Parameters<typeof app.command>[1];
export type ViewHandler = Parameters<typeof app.view>[1];

And then I can infer the types using the new custom type:
export const handleAction: ActionHandler = async ({ client, ack }) => {
    // Now the types are inferred.
 };

I think it would be nice to expose these types from the SDK.

Hi @gilmatok, thanks for writing in! Our latest recommendation is to organize listeners and middlware like this TypeScript project template does. Can you try the approach?


[FEATURE] Contributors check

Description

min 3

max 10

< min: error

> min && < max: warning

> max: green

Screenshots
No response
Additional information
No response

would love to work on this.


download converted flow visualizations

I would like to be able to download the converted flow visualizations as png, svg or animated gif so it can be easily integrated in presentations and deliverables. It would also be nice to have the option to adapt the image size when downloading.
Similar to the actions in  https://mermaid.live/.

For future reference
Capturing a part of a screen is relatively new (https://w3c.github.io/mediacapture-region/).
The demo (https://github.com/w3c/mediacapture-region/tree/main/demo) does not work yet for me.

There is an easy-to-use React hook to capture your screen (https://github.com/tylerbuchea/use-screen-recording), but you can only capture your entire screen. I don't see how this is more useful than third-party solutions that let you capture a part of your screen, such as peek, which I am using.

@elsdvlee, could you take a look at https://knowledgeonwebscale.github.io/dataflow-visualization/#/recording and https://knowledgeonwebscale.github.io/dataflow-visualization/#/recording2.
Do you want me to implement the screenshot and/or capture feature? Personally, I think this is not more useful than using third-party software to capture your screen as webm/gif or use the key combination provided by your OS to take a screenshot.
But if you still want these features, I'll do my best to implement them.

There are libraries that let you screenshot a React component, but that does nor work in combination with ReactFlow. Something like this is the result:

So I think the only way to do it is by doing it like https://knowledgeonwebscale.github.io/dataflow-visualization/#/recording2.

@TiboStr Indeed, this is not more useful than using third-party software to capture your screen as webm/gif or use the key combination provided by your OS to take a screenshot. I was hoping for a Mermaid-like solution (https://mermaid.live/) where you can set a higher resolution for more complicated images.  Maybe not close this issue yet, but focus first on other issues.


fix #104 Garbage characters in a puzzle name with Thymeleaf parser-le…

Stip thymeleaf comment closing tag from puzzle body

@yegor256/z please, pay attention to this pull request

Job #110 is now in scope, role is REV

@yegor256/z everybody who has role REV is banned at #110; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 3min)

Codecov Report

Merging #110 into master will decrease coverage by 1.32%.
The diff coverage is 82.6%.

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
- Coverage   98.49%   97.16%   -1.33%
==========================================
  Files           9        9
  Lines         265      282      +17
==========================================
+ Hits          261      274      +13
- Misses          4        8       +4

Impacted Files
Coverage Δ

lib/pdd/source.rb
94.49% <82.6%> (-3.34%)
:arrow_down:

Continue to review full report at Codecov.

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

Job gh:yegor256/pdd#110 is not assigned, can't get performer

The job #110 is now out of scope


[bug fix] Ready.must_sync

close #128
It seems that we can check ready in test_raw_node_start, thereforce I dont add another test function
And, refactor some parameter name for readability
Signed-off-by: accelsao<EMAIL_ADDRESS>

any command to re-run CI build without meaningless commit?


About the TensorFlow Lite implementation of lightweight-human-pose-estimation

Do we have also an implementation of this algorithm on TensorFlow Lite?
Thank you for your great works!

Hi! I am not aware of any.


xapian: 1.4.12 -> 1.4.13

Motivation for this change
Supersedes #71939 simply including an updated darwin patch. And rebased for good measure.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build xapian

Thank you!

just a heads up that this version is known to crash notmuch: https://notmuchmail.org/pipermail/notmuch/2019/028634.html

:( why does software have to be so hard :(

interesting, it only happens on specific queries https://trac.xapian.org/ticket/796, and seems to be fixed on master https://github.com/xapian/xapian/commit/f4ed23d1.
@jb55 do you think we need to apply that patch?


Capitalize the H

Thanks so much @gjtorikian -- you've made me more aware of how we miscapitalize GH elsewhere in the Kong docs, too. We'd love more docs/website PRs if you're so inclined, but please to amend commits for DCO (--signoff flag which I realize you must know but adding it here for convenience). (I set the DCO check to pass for this PR bc, well, we know you're good to go :D )


tf.nn.conv2d failed to activate TF-32 mode

Click to expand!
Issue Type
Performance
Source
source
Tensorflow Version
2.6.2
Custom Code
Yes
OS Platform and Distribution
18.04
Mobile device
No response
Python version
3.7.5
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
CUDA-11.6/cuDNN version 8300
GPU model and memory
No response
Current Behaviour?
I tried to test convolution's accuracy under TF-32 mode on my A100 chips by comparing TF-32's and FP-32's computaitonal results.

Firstly, I tested a simple MatMul example (as given here:https://www.tensorflow.org/api_docs/python/tf/config/experimental/enable_tensor_float_32_execution). TF-32's and FP-32's results are indeed different as expected. It indicates my environment supports TF-32 mode by default.

Then I used tf.nn.conv2d() method fed with randomly generated float32 data(feature map[1,16,7,7], weight(16,16,3,3)), to test acc of convolution under TF-32 mode, but TF-32's and FP-32's results turned to be identical. It seems like tf.nn.conv2d() failed to activate TF-32, and convlution is done in full-precision mode?

Standalone code to reproduce the issue
fm = np.fromfile(input_fm_bin,dtype=np.float32).reshape([1,16,7,7]).transpose((0,2,3,1))
w = np.fromfile(filter_bin,dtype=np.float32).reshape([16,16,3,3]).transpose(2, 3, 1, 0)
bias = np.fromfile(bias_bin,dtype=np.float32)

fm = tf.convert_to_tensor(fm)
w = tf.convert_to_tensor(w)
bias = tf.convert_to_tensor(bias)
res_tf32 = tf.nn.conv2d(fm, w, [1, 1],[[0, 0], [0, 0], [0, 0], [0, 0]],"NHWC", [1, 2, 2, 1])
res_tf32 = tf.nn.bias_add(res_tf32, bias, data_format="NHWC")

tf.config.experimental.enable_tensor_float_32_execution(False)
res_full = tf.nn.conv2d(fm, w, [1, 1],[[0, 0], [0, 0], [0, 0], [0, 0]],"NHWC", [1, 2, 2, 1])
res_full = tf.nn.bias_add(res_full, bias, data_format="NHWC")

Relevant log output
No response

@HengjiaLi
I reproduced the code shared but facing a different error .Could you please share the colab gist with all the dependencies to analyze more of it. Please find the gist here for reference.
Thank you!

lack of input data, please try the following code instead
import os
import numpy as np
import tensorflow as tf
fm = np.random.rand(1,7,7,16).astype(np.float32)
w = np.random.rand(3,3,16,16).astype(np.float32)
bias = np.random.rand(16).astype(np.float32)
############### conv2d tf 32 result ###############################
tf.config.experimental.enable_tensor_float_32_execution(True)
fm = tf.convert_to_tensor(fm)
w = tf.convert_to_tensor(w)
bias = tf.convert_to_tensor(bias)
res_tf32 = tf.nn.conv2d(fm, w, [1, 1],[[0, 0], [0, 0], [0, 0], [0, 0]],"NHWC", [1, 1, 1, 1])
res_tf32 = tf.nn.bias_add(res_tf32, bias, data_format="NHWC")
################# conv2d FP32 result ##############################
tf.config.experimental.enable_tensor_float_32_execution(False)
res_full = tf.nn.conv2d(fm, w, [1, 1],[[0, 0], [0, 0], [0, 0], [0, 0]],"NHWC", [1, 1, 1, 1])
res_full = tf.nn.bias_add(res_full, bias, data_format="NHWC")
print("convolution results in TF32/FP32 are the same: ", np.all(res_tf32==res_full))

@tiruk007

@sachinprasadhs
I was able to reproduce the issue on Colab using tf-nightly-2.11.0.dev20221012. Please find the gist here for reference
Thank you!

@tiruk007
ATM, what can I do to activate TF-32 with convolutions?
thanks.

@sachinprasadhs

@sachinprasadhs
ATM, what can I do to activate TF-32 with convolutions?

any update on this issue?
@sachinprasadhs @tiruk007


Arrows slow entities down on hit

Should not count rotted arrows so as to not unnecessarily buff skeletons, and not encourage the player to make real arrows.

should it use the slowness debuff, or a new slowing debuff with scales to fit the arrow slowing and its own unique color FC

Investigate whether animals being scared on an arrow miss would be a good change. Maybe for all arrows, maybe only for rotted. Unsure of how this would play out in game yet.


Errors scanning for Workflows

We are using Modernization Scanner to scan for Workflows. During the scan we are seeing lot of errors (please see below) being logged  in in Errors.csv. We are using "Authenticate via Azure AD app-only". Tried with both Command line and GUI. The same errors being logged. Please suggest if any additional configuration is required to perform the scan.
Error-1:
"The property or field 'Url' has not been initialized. It has not been requested or the request has not been executed. It may need to be explicitly requested.","MainSiteLoop","at Microsoft.SharePoint.Client.ClientObject.CheckUninitializedProperty(String propName)   at Microsoft.SharePoint.Client.Site.get_Url()   at Microsoft.SharePoint.Client.SiteExtensions.d__0.MoveNext() in D:\github\sp-dev-modernization\Tools\SharePoint.Modernization\SharePointPnP.Modernization.Framework\Extensions\SiteExtensions.cs:line 22   at SharePoint.Modernization.Scanner.Core.ModernizationScanJob.ModernizationScanJob_TimerJobRun(Object sender, TimerJobRunEventArgs e) in D:\github\sp-dev-modernization\Tools\SharePoint.Modernization\SharePointPnP.Modernization.Scanner.Core\ModernizationScanJob.cs:line 190",
Error-2:
The property or field 'WebTemplate' has not been initialized. It has not been requested or the request has not been executed. It may need to be explicitly requested.","MainWebLoop","at Microsoft.SharePoint.Client.ClientObject.CheckUninitializedProperty(String propName)   at Microsoft.SharePoint.Client.Web.get_WebTemplate()   at SharePoint.Modernization.Scanner.Core.Analyzers.WebAnalyzer.Analyze(ClientContext cc) in D:\github\sp-dev-modernization\Tools\SharePoint.Modernization\SharePointPnP.Modernization.Scanner.Core\Analyzers\WebAnalyzer.cs:line 72   at SharePoint.Modernization.Scanner.Core.ModernizationScanJob.ModernizationScanJob_TimerJobRun(Object sender, TimerJobRunEventArgs e) in D:\github\sp-dev-modernization\Tools\SharePoint.Modernization\SharePointPnP.Modernization.Scanner.Core\ModernizationScanJob.cs:line 266",

Can you share the command line you're using? Try scanning just one or more sites and see if the problem exists on all sites or just one.

Thank you for the prompt response.
SharePoint.Modernization.Scanner.exe -v "" -a https://XXX.sharepoint.com -m WorkflowWithDetailsOnly -c $true -b $true -i <> -z gtus365.onmicrosoft.com -f "XX.pfx" -x XXX -o $true
Tried using the scan with individual sites and that works / throws the same error. Also I am trying to only scan for Workflows but seems like the tool is scanning for other components as well.

That's a weird command line, can you try with something like below:
.\SharePoint.Modernization.Scanner.exe -t mytenant -i e5808e8b-1111-abcd-4321-5008db04a882 -z berton
line.onmicrosoft.com  -f "C:\Modernization\BertOnlineAzureADAppOnlyDefaultPassword.pfx" -x apassword  -c -m WorkflowWithDetailsOnly -h 10 -n

You don't need to specify tenant admin, unless you're using vanity URL's (so url's not ending on .sharepoint.com)?

I used the below command, because I am running the scan in batches as we have quite a few sites that we need to scan from.
.\SharePoint.Modernization.Scanner.exe -i ClientID -z domain.onmicrosoft.com -y Production -f Y:\WFScan\ModernizationScannerApp.pfx -x Password -o $true -v Y:\WFScan\Inputs\1.csv -h 30 -m WorkflowWithDetailsOnly -d $true
This is what we have noticed:
If we are running a scan on 1000 sites, some sites are errored out (ex: 100). If we run another scan with the errored out sites 100. We have seen that the tool is able to complete the scan for some of these 100 errored out sites
Any insights would be highly appreciated.

Might be throttling? If you notice errors being added to the log you could try to use Fiddler to get more details on the erroring requests

@sidgdocs : any update on this?

Closing per no response. If this is still an issue please open a new issue with the requested details.


Base Slider display error when swipe back on Mobile (touch)

PHP Version: 8.2   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Expected behaviour:
On mobile/tablet (touch) in all sliders (tns-slider2), I expect the product elements, images, etc. to be always visible when swiping back and forth while transitioning to the next slide.
Actual behaviour:
When swiping backward (to the left) the product elements, images, etc. unexpectedly disappear. The transform is calculated wrong in the transition.
Here is an example of how it looks like when swiping back on mobile in a product slider:

to the left
How to reproduce:

Go to the storefront on mobile
Visit a page with a product slider
Swipe right (forth) -> see that product elements are always visible when transitioning to the next slider
Swipe left (back) -> see that product elements unexpectedly disappear until the transition is finished

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-38259
[created from NEXT-38259, comment 494413]

not fixed in <IP_ADDRESS>

not fixed in <IP_ADDRESS>

Yup, this issue is still open, so It couldn't be in <IP_ADDRESS>


Module 2.2 Assignment

👋 for this assignment, if you'd like to try again, please include a screenshot of merging on the command line using git fetch and git merge.

@arelia changes made! :)


feat(opentelemetry): make endpoint field referenceable

Summary

Checklist

[x] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

supports the endpoint parameter of the opentelemetry plugin as referenceable.

Issue reference

Fix KAG-2066

Please hold off on the merge until 3.4 is sent out.

This was not merged in time for Feature freeze so I have removed it from the milestone. Please get special approval from a director-level person in order to merge this feature before 3.4 GA.

@kikito I see Saju's approval for posting freeze merge for this feature. If this one worth to be added to 3.4. Please review this one asap.


[Torii] gRPC entity state update api

Extend torii to provide a gRPC api in addition to the existing graphql api. gRPC provides an efficient api for high throughput communication. The response payload should be generic enough to support any entity state update and complete enough to support reconstruction of the typed entities on the client.
### Tasks
- [ ] Define a generic protobuf for communicating state changes for an entities components
- [ ] Define a Get RPC to return the latest state changes given a cursor and limit
- [ ] Support long polling the state update rpc

The existing Graphql API is defined here: https://github.com/dojoengine/dojo/tree/main/crates/torii/src/graphql

Define a generic protobuf for communicating state changes for an entities components

[ ] torii-grpc-proto
[ ] torii-grpc-server
[ ] torii-grpc-client(rs)

implement unary methods to get the latest state changes for an entities components

implement streaming methods to poll the state update, e.g:

[ ]  A new archetype has been created.
[ ]  An entity has been inserted into an archetype.
[ ]  An entity has been removed from an archetype.
@tarrencev

Q: Does gRPC use the same port with the GraphQL service (8080)? Probably not. (8081 or 9000)

@shuimuliang this is a big and important issue - do you have the bandwidth to complete in a reasonable time? If not, lets split this up.

plan to finish in 2 weeks.
please split it into 3 issues with branches:

Define a gPRC protocol for Torri.
implement gRPC unary methods to get the latest state changes
implement gRPC streaming methods to poll the state update