Can't get any admob plugins to play nice with this one, using PhoneGap Build

Expected Behaviour
Getting both cordova-plugin-admob-free (for example) and phonegap-plugin-push to work together, using PhoneGap Build to build app. Push notification (sent via Firebase console) should work as it does when no admob plugin is present.
Actual Behaviour
Getting "Unfortunately, [app name] has stopped" error when push notification is sent, whether app is open or in background. When I don't include admob plugin, it works fine. It's the same for other admob plugins, so that's why I'm listing the issue here, and not with a particular admob plugin.
Steps to Reproduce
Build via PGM, using both plugins in config.xml, phonegap-plugin-push version 2.1.1 (because later versions get build error), and latest cordova-plugin-admob-free.
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 5.0.1
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
LG
Cordova CLI version and cordova platform version
cordova --version                                    cli-7.1.0
cordova platform version android             (not sure how to get this info)

Plugin version
cordova plugin version | grep phonegap-plugin-push   2.1.1

Sample Push Data Payload
Using Firebase console, simple test message.
Sample Code Used
Seeing other comments which seemed to indicate the Google Services version may be wrong in the admob plugin, I tried this in config.xml, using different version values, but none have made a difference:
<plugin name="cordova-plugin-admob-free" source="npm"><variable name="PLAY_SERVICES_VERSION" value="12.0" /></plugin>
Logs taken while reproducing problem
Using PGB's Weinre debugger, nothing shows.
Addendum:
I also tried using cordova-android-play-services-gradle-release, but they say:
There are certain Cordova plugins which, if included into a Cordova project along with this plugin, then this plugin is unable to override the library versions they specify. This is because they use a class to immediately apply their specified Google libraries, instead of just relying on the string version to be applied later on by Gradle.
Two commonly used plugins which do this are:
cordova-plugin-fcm in FCMPlugin.gradle
phonegap-plugin-push in push.gradle
Could this be the problem?

@jamminjames you config.xml should make the play services versions match like this:
<plugin name="cordova-admob-sdk" spec="~0.13.1">
    <variable name="PLAY_SERVICES_VERSION" value="11.6.2" />
</plugin>
<plugin name="phonegap-plugin-push" spec="2.1.1">
   <variable name="FCM_VERSION" default="11.6.2"/>
</plugin>

If that doesn't work for you then we'll need to see the stack trace from the app crash.

I'm using cordova-plugin-admob-free, so I did this:
	<plugin name="cordova-plugin-admob-free" source="npm">
		<variable name="PLAY_SERVICES_VERSION" value="11.6.2" />
	</plugin>
	<plugin name="phonegap-plugin-push" spec="2.1.1" source="npm">
		<variable name="FCM_VERSION" default="11.6.2" />
	</plugin>

...and it's still not building, but the log doesn't seem to be giving a reason. I also tried it exactly as you had it, with "cordova-admob-sdk" spec="~0.13.1", but it does the same thing. Here's the build log:
Build Date: 2018-05-03 04:37:49 +0000
PREPARE OUTPUT
Generating platform-specific config.xml from defaults for android at /res/xml/config.xml
Merging project's config.xml into platform-specific android config.xml
Merging and updating files from [, ../platform_www] to ../assets/www
copy  cartoons.html ../assets/www/cartoons.html (new file)
copy  config.xml ../assets/www/config.xml (new file)
mkdir ../assets/www/cordova-js-src
mkdir ../assets/www/cordova-js-src/android
copy  ../platform_www/cordova-js-src/android/nativeapiprovider.js ../assets/www/cordova-js-src/android/nativeapiprovider.js (new file)
copy  ../platform_www/cordova-js-src/android/promptbasednativeapi.js ../assets/www/cordova-js-src/android/promptbasednativeapi.js (new file)
copy  ../platform_www/cordova-js-src/exec.js ../assets/www/cordova-js-src/exec.js (new file)
copy  ../platform_www/cordova-js-src/platform.js ../assets/www/cordova-js-src/platform.js (new file)
mkdir ../assets/www/cordova-js-src/plugin
mkdir ../assets/www/cordova-js-src/plugin/android
copy  ../platform_www/cordova-js-src/plugin/android/app.js ../assets/www/cordova-js-src/plugin/android/app.js (new file)
copy  ../platform_www/cordova.js ../assets/www/cordova.js (updated file)
mkdir ../assets/www/css/default-skin
copy  css/default-skin/Thumbs.db ../assets/www/css/default-skin/Thumbs.db (new file)
copy  css/default-skin/default-skin.css ../assets/www/css/default-skin/default-skin.css (new file)
copy  css/default-skin/default-skin.png ../assets/www/css/default-skin/default-skin.png (new file)
copy  css/default-skin/default-skin.svg ../assets/www/css/default-skin/default-skin.svg (new file)
copy  css/default-skin/preloader.gif ../assets/www/css/default-skin/preloader.gif (new file)
mkdir ../assets/www/css/images
copy  css/images/Thumbs.db ../assets/www/css/images/Thumbs.db (new file)
copy  css/images/ajax-loader.gif ../assets/www/css/images/ajax-loader.gif (new file)
mkdir ../assets/www/css/images/icons-png
copy  css/images/icons-png/&icons-png-folder.txt ../assets/www/css/images/icons-png/&icons-png-folder.txt (new file)
copy  css/images/icons-png/Thumbs.db ../assets/www/css/images/icons-png/Thumbs.db (new file)
copy  css/images/icons-png/action-black.png ../assets/www/css/images/icons-png/action-black.png (new file)
copy  css/images/icons-png/action-white.png ../assets/www/css/images/icons-png/action-white.png (new file)
copy  css/images/icons-png/alert-black.png ../assets/www/css/images/icons-png/alert-black.png (new file)
copy  css/images/icons-png/alert-white.png ../assets/www/css/images/icons-png/alert-white.png (new file)
copy  css/images/icons-png/arrow-d-black.png ../assets/www/css/images/icons-png/arrow-d-black.png (new file)
copy  css/images/icons-png/arrow-d-l-black.png ../assets/www/css/images/icons-png/arrow-d-l-black.png (new file)
copy  css/images/icons-png/arrow-d-l-white.png ../assets/www/css/images/icons-png/arrow-d-l-white.png (new file)
copy  css/images/icons-png/arrow-d-r-black.png ../assets/www/css/images/icons-png/arrow-d-r-black.png (new file)
copy  css/images/icons-png/arrow-d-r-white.png ../assets/www/css/images/icons-png/arrow-d-r-white.png (new file)
copy  css/images/icons-png/arrow-d-white.png ../assets/www/css/images/icons-png/arrow-d-white.png (new file)
copy  css/images/icons-png/arrow-l-black.png ../assets/www/css/images/icons-png/arrow-l-black.png (new file)
copy  css/images/icons-png/arrow-l-white.png ../assets/www/css/images/icons-png/arrow-l-white.png (new file)
copy  css/images/icons-png/arrow-r-black.png ../assets/www/css/images/icons-png/arrow-r-black.png (new file)
copy  css/images/icons-png/arrow-r-white.png ../assets/www/css/images/icons-png/arrow-r-white.png (new file)
copy  css/images/icons-png/arrow-u-black.png ../assets/www/css/images/icons-png/arrow-u-black.png (new file)
copy  css/images/icons-png/arrow-u-l-black.png ../assets/www/css/images/icons-png/arrow-u-l-black.png (new file)
copy  css/images/icons-png/arrow-u-l-white.png ../assets/www/css/images/icons-png/arrow-u-l-white.png (new file)
copy  css/images/icons-png/arrow-u-r-black.png ../assets/www/css/images/icons-png/arrow-u-r-black.png (new file)
copy  css/images/icons-png/arrow-u-r-white.png ../assets/www/css/images/icons-png/arrow-u-r-white.png (new file)
copy  css/images/icons-png/arrow-u-white.png ../assets/www/css/images/icons-png/arrow-u-white.png (new file)
copy  css/images/icons-png/audio-black.png ../assets/www/css/images/icons-png/audio-black.png (new file)
copy  css/images/icons-png/audio-white.png ../assets/www/css/images/icons-png/audio-white.png (new file)
copy  css/images/icons-png/back-black.png ../assets/www/css/images/icons-png/back-black.png (new file)
copy  css/images/icons-png/back-white.png ../assets/www/css/images/icons-png/back-white.png (new file)
copy  css/images/icons-png/bars-black.png ../assets/www/css/images/icons-png/bars-black.png (new file)
copy  css/images/icons-png/bars-white.png ../assets/www/css/images/icons-png/bars-white.png (new file)
copy  css/images/icons-png/bullets-black.png ../assets/www/css/images/icons-png/bullets-black.png (new file)
copy  css/images/icons-png/bullets-white.png ../assets/www/css/images/icons-png/bullets-white.png (new file)
copy  css/images/icons-png/calendar-black.png ../assets/www/css/images/icons-png/calendar-black.png (new file)
copy  css/images/icons-png/calendar-white.png ../assets/www/css/images/icons-png/calendar-white.png (new file)
copy  css/images/icons-png/camera-black.png ../assets/www/css/images/icons-png/camera-black.png (new file)
copy  css/images/icons-png/camera-white.png ../assets/www/css/images/icons-png/camera-white.png (new file)
copy  css/images/icons-png/carat-d-black.png ../assets/www/css/images/icons-png/carat-d-black.png (new file)
copy  css/images/icons-png/carat-d-white.png ../assets/www/css/images/icons-png/carat-d-white.png (new file)
copy  css/images/icons-png/carat-l-black.png ../assets/www/css/images/icons-png/carat-l-black.png (new file)
copy  css/images/icons-png/carat-l-white.png ../assets/www/css/images/icons-png/carat-l-white.png (new file)
copy  css/images/icons-png/carat-r-black.png ../assets/www/css/images/icons-png/carat-r-black.png (new file)
copy  css/images/icons-png/carat-r-white.png ../assets/www/css/images/icons-png/carat-r-white.png (new file)
copy  css/images/icons-png/carat-u-black.png ../assets/www/css/images/icons-png/carat-u-black.png (new file)
copy  css/images/icons-png/carat-u-white.png ../assets/www/css/images/icons-png/carat-u-white.png (new file)
copy  css/images/icons-png/check-black.png ../assets/www/css/images/icons-png/check-black.png (new file)
copy  css/images/icons-png/check-white.png ../assets/www/css/images/icons-png/check-white.png (new file)
copy  css/images/icons-png/clock-black.png ../assets/www/css/images/icons-png/clock-black.png (new file)
copy  css/images/icons-png/clock-white.png ../assets/www/css/images/icons-png/clock-white.png (new file)
copy  css/images/icons-png/cloud-black.png ../assets/www/css/images/icons-png/cloud-black.png (new file)
copy  css/images/icons-png/cloud-white.png ../assets/www/css/images/icons-png/cloud-white.png (new file)
copy  css/images/icons-png/comment-black.png ../assets/www/css/images/icons-png/comment-black.png (new file)
copy  css/images/icons-png/comment-white.png ../assets/www/css/images/icons-png/comment-white.png (new file)
copy  css/images/icons-png/delete-black.png ../assets/www/css/images/icons-png/delete-black.png (new file)
copy  css/images/icons-png/delete-white.png ../assets/www/css/images/icons-png/delete-white.png (new file)
copy  css/images/icons-png/edit-black.png ../assets/www/css/images/icons-png/edit-black.png (new file)
copy  css/images/icons-png/edit-white.png ../assets/www/css/images/icons-png/edit-white.png (new file)
copy  css/images/icons-png/eye-black.png ../assets/www/css/images/icons-png/eye-black.png (new file)
copy  css/images/icons-png/eye-white.png ../assets/www/css/images/icons-png/eye-white.png (new file)
copy  css/images/icons-png/forbidden-black.png ../assets/www/css/images/icons-png/forbidden-black.png (new file)
copy  css/images/icons-png/forbidden-white.png ../assets/www/css/images/icons-png/forbidden-white.png (new file)
copy  css/images/icons-png/forward-black.png ../assets/www/css/images/icons-png/forward-black.png (new file)
copy  css/images/icons-png/forward-white.png ../assets/www/css/images/icons-png/forward-white.png (new file)
copy  css/images/icons-png/gear-black.png ../assets/www/css/images/icons-png/gear-black.png (new file)
copy  css/images/icons-png/gear-white.png ../assets/www/css/images/icons-png/gear-white.png (new file)
copy  css/images/icons-png/grid-black.png ../assets/www/css/images/icons-png/grid-black.png (new file)
copy  css/images/icons-png/grid-white.png ../assets/www/css/images/icons-png/grid-white.png (new file)
copy  css/images/icons-png/heart-black.png ../assets/www/css/images/icons-png/heart-black.png (new file)
copy  css/images/icons-png/heart-white.png ../assets/www/css/images/icons-png/heart-white.png (new file)
copy  css/images/icons-png/home-black.png ../assets/www/css/images/icons-png/home-black.png (new file)
copy  css/images/icons-png/home-white.png ../assets/www/css/images/icons-png/home-white.png (new file)
copy  css/images/icons-png/info-black.png ../assets/www/css/images/icons-png/info-black.png (new file)
copy  css/images/icons-png/info-white.png ../assets/www/css/images/icons-png/info-white.png (new file)
copy  css/images/icons-png/location-black.png ../assets/www/css/images/icons-png/location-black.png (new file)
copy  css/images/icons-png/location-white.png ../assets/www/css/images/icons-png/location-white.png (new file)
copy  css/images/icons-png/lock-black.png ../assets/www/css/images/icons-png/lock-black.png (new file)
copy  css/images/icons-png/lock-white.png ../assets/www/css/images/icons-png/lock-white.png (new file)
copy  css/images/icons-png/mail-black.png ../assets/www/css/images/icons-png/mail-black.png (new file)
copy  css/images/icons-png/mail-white.png ../assets/www/css/images/icons-png/mail-white.png (new file)
copy  css/images/icons-png/minus-black.png ../assets/www/css/images/icons-png/minus-black.png (new file)
copy  css/images/icons-png/minus-white.png ../assets/www/css/images/icons-png/minus-white.png (new file)
copy  css/images/icons-png/navigation-black.png ../assets/www/css/images/icons-png/navigation-black.png (new file)
copy  css/images/icons-png/navigation-white.png ../assets/www/css/images/icons-png/navigation-white.png (new file)
copy  css/images/icons-png/phone-black.png ../assets/www/css/images/icons-png/phone-black.png (new file)
copy  css/images/icons-png/phone-white.png ../assets/www/css/images/icons-png/phone-white.png (new file)
copy  css/images/icons-png/plus-black.png ../assets/www/css/images/icons-png/plus-black.png (new file)
copy  css/images/icons-png/plus-white.png ../assets/www/css/images/icons-png/plus-white.png (new file)
copy  css/images/icons-png/power-black.png ../assets/www/css/images/icons-png/power-black.png (new file)
copy  css/images/icons-png/power-white.png ../assets/www/css/images/icons-png/power-white.png (new file)
copy  css/images/icons-png/recycle-black.png ../assets/www/css/images/icons-png/recycle-black.png (new file)
copy  css/images/icons-png/recycle-white.png ../assets/www/css/images/icons-png/recycle-white.png (new file)
copy  css/images/icons-png/refresh-black.png ../assets/www/css/images/icons-png/refresh-black.png (new file)
copy  css/images/icons-png/refresh-white.png ../assets/www/css/images/icons-png/refresh-white.png (new file)
copy  css/images/icons-png/search-black.png ../assets/www/css/images/icons-png/search-black.png (new file)
copy  css/images/icons-png/search-white.png ../assets/www/css/images/icons-png/search-white.png (new file)
copy  css/images/icons-png/shop-black.png ../assets/www/css/images/icons-png/shop-black.png (new file)
copy  css/images/icons-png/shop-white.png ../assets/www/css/images/icons-png/shop-white.png (new file)
copy  css/images/icons-png/star-black.png ../assets/www/css/images/icons-png/star-black.png (new file)
copy  css/images/icons-png/star-white.png ../assets/www/css/images/icons-png/star-white.png (new file)
copy  css/images/icons-png/tag-black.png ../assets/www/css/images/icons-png/tag-black.png (new file)
copy  css/images/icons-png/tag-white.png ../assets/www/css/images/icons-png/tag-white.png (new file)
copy  css/images/icons-png/user-black.png ../assets/www/css/images/icons-png/user-black.png (new file)
copy  css/images/icons-png/user-white.png ../assets/www/css/images/icons-png/user-white.png (new file)
copy  css/images/icons-png/video-black.png ../assets/www/css/images/icons-png/video-black.png (new file)
copy  css/images/icons-png/video-white.png ../assets/www/css/images/icons-png/video-white.png (new file)
delete ../assets/www/css/index.css (no source)
copy  css/jquery.mobile-1.4.5.min.css ../assets/www/css/jquery.mobile-1.4.5.min.css (new file)
copy  css/photoswipe.css ../assets/www/css/photoswipe.css (new file)
copy  css/style-ht.css ../assets/www/css/style-ht.css (new file)
copy  google-services.json ../assets/www/google-services.json (new file)
copy  icon.png ../assets/www/icon.png (new file)
copy  img/HT-Sunrise-300w.gif ../assets/www/img/HT-Sunrise-300w.gif (new file)
copy  img/HT-Sunrise-500w.gif ../assets/www/img/HT-Sunrise-500w.gif (new file)
copy  img/ShareIcon-Blue-22.png ../assets/www/img/ShareIcon-Blue-22.png (new file)
copy  img/ShareIcon-Blue-30-hover.xcf ../assets/www/img/ShareIcon-Blue-30-hover.xcf (new file)
copy  img/ShareIcon-Blue-30.png ../assets/www/img/ShareIcon-Blue-30.png (new file)
copy  img/ShareIcon-Blue-30.xcf ../assets/www/img/ShareIcon-Blue-30.xcf (new file)
copy  img/ShareIcon-Blue-30b.png ../assets/www/img/ShareIcon-Blue-30b.png (new file)
copy  img/ShareIcon-Blue-30c-hover.png ../assets/www/img/ShareIcon-Blue-30c-hover.png (new file)
copy  img/ShareIcon-Blue-30c.png ../assets/www/img/ShareIcon-Blue-30c.png (new file)
copy  img/ShareIcon-Blue.png ../assets/www/img/ShareIcon-Blue.png (new file)
copy  img/ShareIcon-Blue2.PNG ../assets/www/img/ShareIcon-Blue2.PNG (new file)
copy  img/Thumbs.db ../assets/www/img/Thumbs.db (new file)
delete ../assets/www/img/cordova.png (no source)
delete ../assets/www/img/logo.png (no source)
copy  img/share-icon-30.gif ../assets/www/img/share-icon-30.gif (new file)
copy  img/share-icon-30.png ../assets/www/img/share-icon-30.png (new file)
copy  img/share-icon-30b.png ../assets/www/img/share-icon-30b.png (new file)
copy  index.html ../assets/www/index.html (updated file)
copy  js/SocialSharing-opt.js ../assets/www/js/SocialSharing-opt.js (new file)
copy  js/admob_free.js ../assets/www/js/admob_free.js (new file)
copy  js/admob_simple.js ../assets/www/js/admob_simple.js (new file)
copy  js/cpf-push.js ../assets/www/js/cpf-push.js (new file)
copy  js/cpf-push1.js ../assets/www/js/cpf-push1.js (new file)
delete ../assets/www/js/index.js (no source)
copy  js/jquery-2.2.4.min.js ../assets/www/js/jquery-2.2.4.min.js (new file)
copy  js/jquery.mobile-1.4.5.min.js ../assets/www/js/jquery.mobile-1.4.5.min.js (new file)
copy  js/phonegap-push.js ../assets/www/js/phonegap-push.js (new file)
copy  js/phonegap-push2.js ../assets/www/js/phonegap-push2.js (new file)
copy  js/phonegap-push3.js ../assets/www/js/phonegap-push3.js (new file)
copy  js/phonegap-push4.js ../assets/www/js/phonegap-push4.js (new file)
copy  js/phonegap-push5.js ../assets/www/js/phonegap-push5.js (new file)
copy  js/photoswipe.min.js ../assets/www/js/photoswipe.min.js (new file)
copy  js/pushbots-init-new.js ../assets/www/js/pushbots-init-new.js (new file)
copy  js/pushbots-init-old.js ../assets/www/js/pushbots-init-old.js (new file)
copy  js/pushbots-init.js ../assets/www/js/pushbots-init.js (new file)
mkdir ../assets/www/res
copy  res/.pgbomit ../assets/www/res/.pgbomit (new file)
mkdir ../assets/www/res/icon
mkdir ../assets/www/res/icon/androidicons
copy  res/icon/androidicons/androidicon144.png ../assets/www/res/icon/androidicons/androidicon144.png (new file)
copy  res/icon/androidicons/androidicon192.png ../assets/www/res/icon/androidicons/androidicon192.png (new file)
copy  res/icon/androidicons/androidicon36.png ../assets/www/res/icon/androidicons/androidicon36.png (new file)
copy  res/icon/androidicons/androidicon48.png ../assets/www/res/icon/androidicons/androidicon48.png (new file)
copy  res/icon/androidicons/androidicon72.png ../assets/www/res/icon/androidicons/androidicon72.png (new file)
copy  res/icon/androidicons/androidicon96.png ../assets/www/res/icon/androidicons/androidicon96.png (new file)
mkdir ../assets/www/res/icon/bada
mkdir ../assets/www/res/icon/bada-wac
copy  res/icon/bada-wac/.pgbomit ../assets/www/res/icon/bada-wac/.pgbomit (new file)
copy  res/icon/bada-wac/icon-48-type5.png ../assets/www/res/icon/bada-wac/icon-48-type5.png (new file)
copy  res/icon/bada-wac/icon-50-type3.png ../assets/www/res/icon/bada-wac/icon-50-type3.png (new file)
copy  res/icon/bada-wac/icon-80-type4.png ../assets/www/res/icon/bada-wac/icon-80-type4.png (new file)
copy  res/icon/bada/.pgbomit ../assets/www/res/icon/bada/.pgbomit (new file)
copy  res/icon/bada/icon-128.png ../assets/www/res/icon/bada/icon-128.png (new file)
mkdir ../assets/www/res/icon/blackberry
copy  res/icon/blackberry/.pgbomit ../assets/www/res/icon/blackberry/.pgbomit (new file)
copy  res/icon/blackberry/icon-80.png ../assets/www/res/icon/blackberry/icon-80.png (new file)
mkdir ../assets/www/res/icon/ios
copy  res/icon/ios/.pgbomit ../assets/www/res/icon/ios/.pgbomit (new file)
copy  res/icon/ios/icon-57-2x.png ../assets/www/res/icon/ios/icon-57-2x.png (new file)
copy  res/icon/ios/icon-57.png ../assets/www/res/icon/ios/icon-57.png (new file)
copy  res/icon/ios/icon-72-2x.png ../assets/www/res/icon/ios/icon-72-2x.png (new file)
copy  res/icon/ios/icon-72.png ../assets/www/res/icon/ios/icon-72.png (new file)
mkdir ../assets/www/res/icon/tizen
copy  res/icon/tizen/.pgbomit ../assets/www/res/icon/tizen/.pgbomit (new file)
copy  res/icon/tizen/icon-128.png ../assets/www/res/icon/tizen/icon-128.png (new file)
mkdir ../assets/www/res/icon/webos
copy  res/icon/webos/.pgbomit ../assets/www/res/icon/webos/.pgbomit (new file)
copy  res/icon/webos/icon-64.png ../assets/www/res/icon/webos/icon-64.png (new file)
mkdir ../assets/www/res/icon/windows-phone
copy  res/icon/windows-phone/.pgbomit ../assets/www/res/icon/windows-phone/.pgbomit (new file)
copy  res/icon/windows-phone/icon-173-tile.png ../assets/www/res/icon/windows-phone/icon-173-tile.png (new file)
copy  res/icon/windows-phone/icon-48.png ../assets/www/res/icon/windows-phone/icon-48.png (new file)
copy  res/icon/windows-phone/icon-62-tile.png ../assets/www/res/icon/windows-phone/icon-62-tile.png (new file)
mkdir ../assets/www/res/screen
mkdir ../assets/www/res/screen/android
copy  res/screen/android/screen-hdpi-landscape.png ../assets/www/res/screen/android/screen-hdpi-landscape.png (new file)
copy  res/screen/android/screen-hdpi-portrait.png ../assets/www/res/screen/android/screen-hdpi-portrait.png (new file)
copy  res/screen/android/screen-ldpi-landscape.png ../assets/www/res/screen/android/screen-ldpi-landscape.png (new file)
copy  res/screen/android/screen-ldpi-portrait.png ../assets/www/res/screen/android/screen-ldpi-portrait.png (new file)
copy  res/screen/android/screen-mdpi-landscape.png ../assets/www/res/screen/android/screen-mdpi-landscape.png (new file)
copy  res/screen/android/screen-mdpi-portrait.png ../assets/www/res/screen/android/screen-mdpi-portrait.png (new file)
copy  res/screen/android/screen-xhdpi-landscape.png ../assets/www/res/screen/android/screen-xhdpi-landscape.png (new file)
copy  res/screen/android/screen-xhdpi-portrait.png ../assets/www/res/screen/android/screen-xhdpi-portrait.png (new file)
mkdir ../assets/www/res/screen/bada
mkdir ../assets/www/res/screen/bada-wac
copy  res/screen/bada-wac/.pgbomit ../assets/www/res/screen/bada-wac/.pgbomit (new file)
copy  res/screen/bada-wac/screen-type3.png ../assets/www/res/screen/bada-wac/screen-type3.png (new file)
copy  res/screen/bada-wac/screen-type4.png ../assets/www/res/screen/bada-wac/screen-type4.png (new file)
copy  res/screen/bada-wac/screen-type5.png ../assets/www/res/screen/bada-wac/screen-type5.png (new file)
copy  res/screen/bada/.pgbomit ../assets/www/res/screen/bada/.pgbomit (new file)
copy  res/screen/bada/screen-portrait.png ../assets/www/res/screen/bada/screen-portrait.png (new file)
mkdir ../assets/www/res/screen/blackberry
copy  res/screen/blackberry/.pgbomit ../assets/www/res/screen/blackberry/.pgbomit (new file)
copy  res/screen/blackberry/screen-225.png ../assets/www/res/screen/blackberry/screen-225.png (new file)
mkdir ../assets/www/res/screen/ios
copy  res/screen/ios/.pgbomit ../assets/www/res/screen/ios/.pgbomit (new file)
copy  res/screen/ios/screen-ipad-landscape-2x.png ../assets/www/res/screen/ios/screen-ipad-landscape-2x.png (new file)
copy  res/screen/ios/screen-ipad-landscape.png ../assets/www/res/screen/ios/screen-ipad-landscape.png (new file)
copy  res/screen/ios/screen-ipad-portrait-2x.png ../assets/www/res/screen/ios/screen-ipad-portrait-2x.png (new file)
copy  res/screen/ios/screen-ipad-portrait.png ../assets/www/res/screen/ios/screen-ipad-portrait.png (new file)
copy  res/screen/ios/screen-iphone-landscape-2x.png ../assets/www/res/screen/ios/screen-iphone-landscape-2x.png (new file)
copy  res/screen/ios/screen-iphone-landscape.png ../assets/www/res/screen/ios/screen-iphone-landscape.png (new file)
copy  res/screen/ios/screen-iphone-portrait-2x.png ../assets/www/res/screen/ios/screen-iphone-portrait-2x.png (new file)
copy  res/screen/ios/screen-iphone-portrait-568h-2x.png ../assets/www/res/screen/ios/screen-iphone-portrait-568h-2x.png (new file)
copy  res/screen/ios/screen-iphone-portrait.png ../assets/www/res/screen/ios/screen-iphone-portrait.png (new file)
mkdir ../assets/www/res/screen/tizen
copy  res/screen/tizen/.pgbomit ../assets/www/res/screen/tizen/.pgbomit (new file)
copy  res/screen/tizen/README.md ../assets/www/res/screen/tizen/README.md (new file)
mkdir ../assets/www/res/screen/webos
copy  res/screen/webos/.pgbomit ../assets/www/res/screen/webos/.pgbomit (new file)
copy  res/screen/webos/screen-64.png ../assets/www/res/screen/webos/screen-64.png (new file)
mkdir ../assets/www/res/screen/windows-phone
copy  res/screen/windows-phone/.pgbomit ../assets/www/res/screen/windows-phone/.pgbomit (new file)
copy  res/screen/windows-phone/screen-portrait.jpg ../assets/www/res/screen/windows-phone/screen-portrait.jpg (new file)
copy  splash.png ../assets/www/splash.png (new file)
mkdir ../assets/www/src
mkdir ../assets/www/src/ui
copy  src/ui/photoswipe-ui-default-orig.js ../assets/www/src/ui/photoswipe-ui-default-orig.js (new file)
copy  src/ui/photoswipe-ui-default-works.js ../assets/www/src/ui/photoswipe-ui-default-works.js (new file)
copy  src/ui/photoswipe-ui-default.js ../assets/www/src/ui/photoswipe-ui-default.js (new file)
Wrote out android application name "Humor Times: News in Cartoons!" to /res/values/strings.xml
android-versionCode not found in config.xml. Generating a code based on version in config.xml (<IP_ADDRESS>): 105
Wrote out Android package name "com.humortimes.newscartoons" to /src/com/humortimes/newscartoons/HumorTimes_NewsinCartoons_.java
Updating icons at ../res
copy  res/icon/androidicons/androidicon72.png ../res/mipmap-hdpi/icon.png (new file)
copy  res/icon/androidicons/androidicon36.png ../res/mipmap-ldpi/icon.png (new file)
copy  res/icon/androidicons/androidicon48.png ../res/mipmap-mdpi/icon.png (new file)
copy  res/icon/androidicons/androidicon96.png ../res/mipmap-xhdpi/icon.png (new file)
copy  res/icon/androidicons/androidicon144.png ../res/mipmap-xxhdpi/icon.png (new file)
copy  res/icon/androidicons/androidicon192.png ../res/mipmap-xxxhdpi/icon.png (new file)
Updating splash screens at ../res
copy  res/screen/android/screen-hdpi-landscape.png ../res/drawable-land-hdpi/screen.png (new file)
copy  res/screen/android/screen-ldpi-landscape.png ../res/drawable-land-ldpi/screen.png (new file)
copy  res/screen/android/screen-mdpi-landscape.png ../res/drawable-land-mdpi/screen.png (new file)
copy  res/screen/android/screen-xhdpi-landscape.png ../res/drawable-land-xhdpi/screen.png (new file)
copy  splash.png ../res/drawable-mdpi/screen.png (new file)
copy  res/screen/android/screen-hdpi-portrait.png ../res/drawable-port-hdpi/screen.png (new file)
copy  res/screen/android/screen-ldpi-portrait.png ../res/drawable-port-ldpi/screen.png (new file)
copy  res/screen/android/screen-mdpi-portrait.png ../res/drawable-port-mdpi/screen.png (new file)
copy  res/screen/android/screen-xhdpi-portrait.png ../res/drawable-port-xhdpi/screen.png (new file)
Updating resource files at ..
copy  google-services.json ../google-services.json (new file)
Prepared android project successfully
PLUGIN OUTPUT
Fetching plugin "cordova-plugin-whitelist" via npm
Installing "cordova-plugin-whitelist" at "1.3.3" for android
           This plugin is only applicable for versions of cordova-android greater than 4.0. If you have a previous platform version, you do *not* need this plugin since the whitelist will be built in.

Fetching plugin "cordova-plugin-device" via npm
Installing "cordova-plugin-device" at "2.0.2" for android
Fetching plugin "cordova-plugin-splashscreen" via npm
Installing "cordova-plugin-splashscreen" at "5.0.2" for android
Fetching plugin "cordova-plugin-network-information" via npm
Installing "cordova-plugin-network-information" at "2.0.1" for android
Fetching plugin "cordova-plugin-vibration" via npm
Installing "cordova-plugin-vibration" at "3.1.0" for android
Fetching plugin "cordova-plugin-inappbrowser" via npm
Installing "cordova-plugin-inappbrowser" at "3.0.0" for android
Fetching plugin "cordova-plugin-dialogs" via npm
Installing "cordova-plugin-dialogs" at "2.0.1" for android
Fetching plugin "cordova-plugin-spinnerdialog" via npm
Installing "cordova-plugin-spinnerdialog" at "1.3.2" for android
Fetching plugin "cordova-plugin-x-socialsharing" via npm
Installing "cordova-plugin-x-socialsharing" at "5.4.0" for android
Fetching plugin<EMAIL_ADDRESS>via npm
Installing "es6-promise-plugin" at "4.2.2" for android
Subproject Path: CordovaLib
Fetching plugin "https://github.com/jamminjames/cordova-phonegap_youtube_player_api_android" via git clone
Using shallow clone
Repository "https://github.com/jamminjames/cordova-phonegap_youtube_player_api_android" checked out to git ref "master" at "21531fe".
Installing "remcob00-cordova-youtube-android-player-api" at "2.0.6" for android
Fetching plugin "cordova-plugin-admob-free" via npm
Installing "cordova-plugin-admob-free" at "0.16.0" for android
Fetching plugin<EMAIL_ADDRESS>via npm
Installing "cordova-promise-polyfill" at "0.0.2" for android
Fetching plugin<EMAIL_ADDRESS>via npm
Installing "cordova-admob-sdk" at "0.16.0" for android
Subproject Path: CordovaLib

@macdonst Any idea what the conflict is, or what I can try? Thanks for any help!

@jamminjames can you run your build with the --verbose flag?

I don't know how to do that in Phonegap Build.

Actually, I don't think that adding the --verbose flag on PGB is possible.
I had a similar problem: In my case, I wasn't able to display ads. As soon as I tried to so so, the app crashed (adb logcat eported a classNotFoundException). As soon as I updated to play services version 15+, the problems were solved:
<plugin name="cordova-admob-sdk" source="npm" spec="0.16.0">
    	<variable name="PLAY_SERVICES_VERSION" value="15+" />
</plugin>

@jamminjames I don't know if that helps you but maybe this is a point where you can start :smiley:

@MBuchalik Thanks, but that doesn't work for me. Are you sure about that value? I thought the highest Play Services Version was 11.6.2 or so...

FINALLY! After much trial and error, searching and testing, I've found a combination that works with no conflicts, for including both admob and firebase push notifications in one Phonegap Build app.
The only drawback is that I'd like to use cordova-plugin-admob-free instead of admobpro, because apparently the latter is unethically placing its own ads to make more bucks off your app. But I can't get it to work without conflicts.
So, here they are, for anyone else experiencing the same problems I had. No need for cordova-support-google-services or any other "fixes" to get them to work with each other, just include these in config.xml:
<plugin name="cordova-plugin-admobpro-firebase" source="npm" />
<plugin name="cordova-plugin-firebase" source="npm" />


When starting debugger, debug console shows "stty: standard input: Inappropriate ioctl for device"

Executables
Version of bash-debug: v0.39
which bash
/usr/bin/bash
bash -c 'uname -a; for P in bash bashdb cat mkfifo pkill; do echo ---; which -a $P; command $P --version; done'
Linux gram 6.3.1-060301-generic #202304302031 SMP PREEMPT_DYNAMIC Mon May  1 00:41:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux`
---
/usr/bin/bash
/bin/bash
GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
---
bash: line 1: bashdb: command not found
---
/usr/bin/cat
/bin/cat
cat (GNU coreutils) 8.32
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Torbjorn Granlund and Richard M. Stallman.
---
/usr/bin/mkfifo
/bin/mkfifo
mkfifo (GNU coreutils) 8.32
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by David MacKenzie.
---
/usr/bin/pkill
/bin/pkill
pkill from procps-ng 3.3.17

Debug output
stty: standard input: Inappropriate ioctl for device
stty: standard input: Inappropriate ioctl for device
/home/user/vscode-oss/extensions/rogalmic.bash-debug-0.3.9-universal/bashdb_dir/command/source.sh: line 41: /dev/stdin: No such device or address

Details
New user here, I have been looking for a way to debug bash scripts for a long time. I have installed the plugin and followed the first gif posted here:  https://open-vsx.org/extension/rogalmic/bash-debug. I also read the guide here: https://code.visualstudio.com/docs/editor/debugging.
I created a launch.json, pasted the code. When I press the start debug button. I get these errors in the console. I have tried debugging several scripts, I always get these errors. Am I missing something?
I am on ubunu jammy and running VSCodium version 1.82.0 release 23250:
Version: 1.82.0
Release: 23250
Commit: 13ae69686c4390a9aee7b71b44337eb488319f26
Date: 2023-09-07T18:51:47.499Z
Electron: 25.8.0
ElectronBuildId: undefined
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.3.1-060301-generic

Thank you for this plugin! Hopefully I can get it working.

I have the same error message. But It can still work.

when I debug, it can stop at breakpoint, but some variables value is wrong displayed.
declare -A mydict

while read key value; do
    if [[ -z $key ]] || [[ -z $value ]] ; then
        break
    fi
    mydict["$key"]="$value"
done

I have typed "k1 v1", but it display wrong value. But the program is still correct.


Fix JSON and next page issues

You can use a new url to fetch JSON data:
request_url = "https://www.instagram.com/" + username + "?__a=1"

And then access each image with:
for node in data["user"]["media"]["nodes"]:
  file_url = node["display_src"]
  # Do something with file_url

I've also fixed the next page functionality as this changed with the new JSON, as well as removed unused libraries, and made it work with python 2.7.
Let me know any thoughts

I never knew that there was a new place to get the JSON data. I guess we do learn something new everyday!
User-Agent and JSON parser
The reason I made insta-dl with the requests module is because

It has the JSON parser inbuilt (one less line in the code I guess)

You can add a User-Agent so Instagram doesn't temporarily ban the user's IP because of the many requests.

I don't think it is possible to add User-Agent in the urllib module. We must add a User-Agent because when I ran the code, it stopped downloading because Instagram temporarily banned my IP.
Error with Python3
When running the code in Python3 I get an error on line 21 because you wrote:
response = urllib.urlopen(request_url)

But in Python3 it has to be:
response = urllib.request.urlopen(request_url)

In my opinion I think writing this in Python3 is better because Python3 is soon gonna be the main Python (just my opinion)
So overall, if you fix the error with Python3 and somehow add a user-agent, then I will accept you PR.
Thank you for helping!

Fixes made, keeping compatibility with python 2.7 and 3

Line 48 which is urllib.urlretrieve(file_url, path) has to be urllib.request.urlretrieve(file_url, path)  in order for the code to work on Python3. Because now it is skipping all the images because an error occurs.
And also, I found out, that Instagram temporarily blocks your IP if someone goes onto the link with the JSON many times. So i recommend using a VPN when running this.
Windscribe is a very good one. It has 10GB for free each month. (Just for your info)
Please fix line 48 and I think everything else looks great.

Proof Instagram Blocks You

Change made, and yeah I also use windscribe its pretty great :)


[1.28.5] Theming problem - setRootTheme and FormContainer with expendable=true

Hi,
I have createt a new theme and set the root of the theme.
sap.ui.getCore().setThemeRoot("my_theme", "./openui5/resources");
sap.ui.getCore().applyTheme("my_theme");
This works fine. But if I use a FormContainer with "expandable=true" the image for the button is loaded from the base module path of the library (e.g. https://openui5.hana.ondemand.com/resources/sap/ui/layout/themes/my_theme/img/form/Minimize.png). So the attribute "src" of the button points to an invalid path.
In the FormContainer I found that the module path is used.
sIcon = jQuery.sap.getModulePath("sap.ui.layout", '/') + "themes/" + sap.ui.getCore().getConfiguration().getTheme() + sIcon;
How can i tell ui5 to use my theme directory (./openui5/resources) and not https://openui5.hana.ondemand.com/resources/sap/ui/layout/themes/my_theme/img/form/Minimize.png?
Thanks,
WoAn

With the current code in the FormContainer, you IMHO don't have a chance. Only the jQuery.sap.getModulePath portion of the calculated URL is influenced by your theme root configuration, the remainder is so to say a "hard coded suffix".
setThemeRoot() only affects the full combination of library name ("sap.ui.layout" in this case) and the 'themes' folder and the theme specific folder. A proper URL  calculation in the FormContainer would either combine those parts first and only then pass them to jQuery.sap.getModulePath:
sIcon = jQuery.sap.getModulePath("sap.ui.layout.themes."
                 + sap.ui.getCore().getConfiguration().getTheme(), "/") + sIcon;

or use sap.ui.resourceinstead, which includes special handling for theme resources:
sIcon = sap.ui.resource("sap.ui.layout",
                       "themes/" + sap.ui.getCore().getConfiguration().getTheme() + "/" + sIcon);

(a bullet proof solution might be even more complicated as the original sIcon value is taken from a theme parameter)
As far as I remember, code like the one from the FormContainer was recognized as bad practise in the past and an initiative was planed to fix all similar places. But as with many other things, it was not done yet - most likely because there have been even more important things to do first.
Nevertheless: I'll raise that topic again.
For the FormContainer, you might just change it on your own locally as sketched above and give it a try.

Thanks, I will change the code until it is fixed.


Postman collection is missing

When testing this plug-in for the first time it might be handy to have a postman collection to try out the requests.

Fixed by PR #8


feat: Add support for Bondsmith and Fiserv source verification

resolves #38

@alexluong - could we also add https://github.com/hookdeck/terraform-provider-hookdeck/issues/49 to this PR? The required functionality should be in the latest SDK.

@leggetter certainly! I did mention it to you as well. There are a few more than just AWS Signature that we haven't supported tho. Should we add them as well?
In either case, any reason why we can't merge this and follow up on that later? They're separate enough that I don't see a strong reason why they should be in the same PR.

@leggetter certainly! I did mention it to you as well. There are a few more than just AWS Signature that we haven't supported tho. Should we add them as well?
In either case, any reason why we can't merge this and follow up on that later? They're separate enough that I don't see a strong reason why they should be in the same PR.

@alexluong, the AWS Signature one is a priority, so it'd be great to see that one ASAP 🙏 But I'll merge this one as addressing verifications only.

@leggetter ah I see, noted on the priority of the AWS Signature and will work on that soon!

@alexluong Could we also add a json bypass for the destination auth so that there's a workaround when more are added?


semantic ui hidden vertical divider doesn't get hidden

Title

3 days ago

Cute dogs come in a variety of shapes and sizes. Some cute dogs are cute for their adorable faces, others for their tiny stature, and even others for their massive size.

                  Sign Up

              Like

              Favorite

Fixed in 2.0

How do I try out 2.0? The latest in master is 1.12.1

Next branch


WIP: fix empty discussion post in events communities

Checklists not yet completed, because I am not yet done!

Backend checklist

[ ] Formatted my code by running ruff check --select I --fix . && ruff check . && ruff format . in app/backend
[ ] Added tests for any new code or added a regression test if fixing a bug
[ ] All tests pass
[ ] Run the backend locally and it works
[ ] Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

Web frontend checklist

[ ] Formatted my code with yarn format
[ ] There are no warnings from yarn lint --fix
[ ] There are no console warnings when running the app
[ ] Added any new components to storybook
[ ] Added tests where relevant
[ ] All tests pass
[ ] Clicked around my changes running locally and it works
[ ] Checked Desktop, Mobile and Tablet screen sizes

Sorry, half my comment seems to have been deleted, I think I filled out the template wrong!
This is for #4172 , added front end validation and backend check as well. Started writing tests but would like to chat with someone for 15-20 minutes about how to best write tests for this!

Sorry, half my comment seems to have been deleted, I think I filled out the template wrong!
This is for #4172 , added front end validation and backend check as well. Started writing tests but would like to chat with someone for 15-20 minutes about how to best write tests for this!

Leaving a comment here about tests in case we don't overlap soon!
Code looks great so far.
Another good frontend test would be to make a new test file CommentForm.test.tsx (since no one is testing that component yet) and test that when the comment form is empty or only filled with whitespace characters, tabs, etc. and user hits submit, it doesn't call the postReply function. You could do this by mocking the service like:
const postReplyMock = service.thread.postReply as jest.Mock;

Then check it wasn't called in the individual test:
expect(postReplyMpck).not.toHaveBeenCalled();

For backend, you probably wanna add a test case in test_threads.py, the def test_threads_errors(db) function looks right. Probably you'd make the content in the arguments equal to "" and then check the right error showed. Looks like there's some similar functions in there you could base it on.
Instructions for testing locally on the backend are in app/backend/readme.md. You can run tests for only one function like pytest test_threads.py::test_threads_errors (path might be different to folder).

I've run the lint & format to make the build pass, @aapeliv says the backend look good... guess it's fine to merge? (and to leave the PR in this state...)


Pressing "Cancel" in export & import dialog should go back

...it currently closes the lightbox and goes to another page, not good.

also happens in import

Goes back to EAV management know.


Fix unsized spike train and np.allclose issues in testing

@alperyeg
Replaces #59 and #60.  Must be used with #57 to work in Python 3.

@alperyeg
OK, I added the warning.


WIP: Feature/angular element 244

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature (please, look at the "Scope of the project" section in the README.md file)
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

Closed because of https://github.com/xlayers/xlayers/pull/329


Anrp/sysroot

Force Unreal toolchain to use Unreal sysroot

Not doing this can lead to the Unreal toolchain examining the host system,
which can cause it to attempt to use glibc features newer than the Unreal
target of 2.17 provides (specifically, rpclib can accidentaly pick up the
symbol pthread_cond_clockwait, which is provided by Ubuntu 20.04 glibc 2.31
but *not* by Unreal glibc 2.17, which then causes a missing symbol linker
error when trying to build CARLA.)

This change is

Thank you, we merge this


Address Sonarcloud issues for the html code; improve on css styles

@uweseimet FYI, I'm addressing issues reported by SonarCloud for the html code here.
I've also tweaked the rules so that we may see more structural and semantic issues raised in the next report.

@rdmark Regarding SonarCloud, any news on making the develop branch a long-living branch?

I don't agree with the "GNU extensions should not be used" rule. When are we going to use a non-GNU compiler?
I'm OK with trying these to see what happens. But, I'd like to move past the huge PRs that don't fix functional issues or add features. Around October 15th or so, I'd like to lock down the code base to only bug fixes so we can shoot for a stable release at the end of the month.

@akuker Sounds fine for me. Regarding the GNU extension settings I don't consider it important. i was just curious whether there is actually anything GNU specific being used.


[Problem] Not support .rubocop.yml in current workspace

When I do not have a config in root directory such as ~/.rubocop.yml and I have a config .rubocop.yml in current workspace, warning messages are displayed.
so it needs to support using current workspace config without no settings.
particular rubocop versions cannot support absolute config path in current directory

support: .rubocop.yml
not support: /user/workspace/.rubocop.yml

I'm using rvm, ruby 2.7.1 and rubocop 1.3.1.

if no reply within 1week, I'm gonna fork and release a new extension to fix issues.

Hey @KSH-code I've seen that you forked it to: https://github.com/KSH-code/vscode-ruby-rubocop did you also release a fixed version?
As I am working on many projects with many different ruby and rubocop versions and many different .rubocop.yml files, the extension is in it's current state unfortunately not very helpful.

@mediafinger
Not yet. I'll release fixed version in this week, because my work is too busy 😂

I've been discovering how it deploys. it may take a long time, because of the first time and it is a not my project.


Add images on home page & Formating

Changed formatting to make it more aligned to the default community template
Added ability to show post images on the home page

Same goes for this one. Thank you very much!
We'll have a look if it's possible to include it as soon as possible, but will likely have to gather all the changes and pack a new minor version, because these are new features (which is nice)


DownloadTask should manage fd properly

If the first download validation failed, the next retry will have problem with the closed fd.

An error related with it goes like this:
[ERROR] [2021/7/31上午1:37:20] Uncaught Exception
[ERROR] [2021/7/31上午1:37:20] RangeError [ERR_OUT_OF_RANGE]: The value of "fd" is out of range. It must be >= 0 && <=<PHONE_NUMBER>. Received -1
[CONSOLE]     at Object.write (fs.js:641:3)
[CONSOLE]     at WriteStream._write (internal/fs/streams.js:396:13)
[CONSOLE]     at writeOrBuffer (internal/streams/writable.js:358:12)
[CONSOLE]     at WriteStream.Writable.write (internal/streams/writable.js:303:10)
[CONSOLE]     at IncomingMessage.ondata (internal/streams/readable.js:719:22)
[CONSOLE]     at IncomingMessage.emit (events.js:327:22)
[CONSOLE]     at IncomingMessage.Readable.read (internal/streams/readable.js:519:10)
[CONSOLE]     at flow (internal/streams/readable.js:992:34)
[CONSOLE]     at resume_ (internal/streams/readable.js:973:3)
[CONSOLE]     at processTicksAndRejections (internal/process/task_queues.js:80:21) {
[CONSOLE]   code: 'ERR_OUT_OF_RANGE'
[CONSOLE] }


Increase concurrent PR limit from 2 to 3

This will slightly reduce the number of PRs being rate-limited without any notification beyond the dependency dashboard.
Change-type: patch

I self-certify!


External nonterminal

I don't remember why we changed externs to be factors in #35, but this changes it back to be a nonterminal, but now the nonterminal is not the lhs of any rule. Overall this simplifies the code, and fixes #122.

The corresponding PR https://github.com/diprism/fggs/pull/160 is not passing yet, so making this a draft for now.


When setting items, check that item is of type ItemInterface to fix p…

@gstjohn and @likeuntomurphy Here is a pull request to fix the Mollie Gateway issue that we talked through on Friday.  This has worked with my local tests.  Please confirm it still works for your CartThrob changes Jonathan.  Thanks.

@lookwhatimade I removed an unused import and added a missing import, and I made a couple of changes to fix their pre-commit code-sniffer errors. I did not fix their mess detector warning that setItems uses an else, since we didn’t add that. Let me know if it still works for you (I see no reason why it shouldn’t).

@likeuntomurphy your changes work for me when I test.  Thanks.


Overhauled py2specials

json_hexlify, safe_hexlify, renamed functions

@vbuterin This is a major change in semantics and such, so if this isn't how you want to go, people have the option of my fork, where 'py2specials+py3specials=pyspecials`


Add typescript

Now builder is treated as a separate package an has it's own binary.
This is how you debug it:
node --inspect-brk ./node_modules/.bin/builder-30-seconds serve

node --inspect-brk ./node_modules/.bin/builder-30-seconds serve

I think we need to add DEVELOPMENT.md and add this line there


feat: add an example for js_image_layer

Closes #116

Yay cheerleading from vacation 🏝

This one is ready to be landed unless there are new things that need to be addressed

Very cool. Left a few comments.


Missing some dependencies and url encoding issue

Hello,
I did a modification to allow search topics as c++ using urlencode package available on Luarocks.
I hope you find it useful.

Thank you so much for this, I really appreciate it.
It's my first time using Lua so I didn't know how url encoding was done correctly.
Thanks 💙

merged 👍


ENH: Added functionality to specify the dependencies in config

This PR adds the functionality to add dependencies of files and folders in the source directory in the following manner :
jupyter_dependencies = {
    <dir> : ['file1', 'file2'],
    {<dir>}/<file.rst> : ['file1']
}

At present, the values are file names, which are assumed to be in the same directory as the source keys.
fixes #228

thanks @AakashGfude - nice work.

@arnavs would you be willing to test this branch to make sure it meets the Julia project needs?
This is to allow you to specify project.toml files for directories in the conf.py
I have a target date for PyPI release of 16th August 2019.

Thanks @mmcky, testing this now.

From Slack with @mmcky: One quirk is that, when I run make website,  it will set up the directory structure (e.g., arnavsood@mbp:~/research/quantecon/lecture-source-jl/_build/website/jupyter/executed/more_julia) without the TOML. And it will bring the TOML in when the first notebook from that subdirectory is run.
It would be great, though, if the TOML was moved at directory setup time.

thanks @arnavs for testing this. What happens for make website is the files are written to _build/website/jupyter and then as they are executed the results are written to executed. Is the base jupyter folder populated with toml

Yeah, the _build/website/jupyter/more_julia folder was fine.

Just checking: @arnavs and @AakashGfude is this PR going to solve your request for toml support?

@mmcky I think this is good from our end.

thanks @arnavs for testing this out. Thanks @AakashGfude for implementing this feature. I think it will be really nice to have. I will merge and update docs later today with the new option.

@arnavs if you have any requests for improvements -- please document them in an issue. Thanks.


Difference in voltage when running on diff number of cores or machines

@wwlytton : "I'm consistently getting small amplitude diffs with simple HH sims depending on # of nodes; differences are of order 1e-11
this is true both with and without table
diff btwn np=2 vs np=3 in bugsim.py: basic idea is that the voltage diffs with 2 vs 3 nodes gives rise to a time where the difference lies right at spike threshold at which time there is a single dt shift in timing (then all goes wrong from there);  but for some reason the errors in your sim get gradually worse while those for HH stay quite small
voltage diffs are also present with HH so that's the part i want to pass on to mh
but also need to see which mod file in your collection makes so much worse than HH
can't even get replicability across machines without mpi with this HH sim!
max(ndpo-ndmy), np.mean(ndpo-ndmy)
Out[53]: (4.424013155812645e-08, 6.729583247892994e-12)
small tends to build up over time"

Could try follow debugging approach suggested by @nrnhines : "use diff to find the first spike difference which implies a substantive state difference in that gid and then do a binary search using simulations with different tstop to find the time of the first variable difference in the state [after finitialize add the line pc.prcellstate(gid, "t0")]. At that point it is generally obvious why the difference exists."


[EXTRA] dm list dies on some kinds of broken dot

NewCmdList in dm's main.go has this code in the loop over all dots found:
					containerInfo, err := dm.RelatedContainers(v.Name, b)
					if err != nil {
						return err
					}

However, it seems that sometimes RelatedContainers fails for a dot returned by the List RPC:
[alaric@nixos:~/work/dotmesh]$ dm list
Current remote: dothub (use 'dm remote -v' to list and 'dm remote switch' to switch)

Response '{"jsonrpc":"2.0","error":{"code":-32000,"message":"No such filesystem named 'dotmesh_smokes/volume_1526541702_FGtlj0J93eNS0BFLDx9K9PEYhIVIXKCx'","data":null},"id":3905856217637762879}
' yields error No such filesystem named 'dotmesh_smokes/volume_1526541702_FGtlj0J93eNS0BFLDx9K9PEYhIVIXKCx'

Suggest removing the error, maybe printing a warning at most.

This was fixed when we changed over to having a single API call to get the containers along with the dots in #337.


Cannot find module '@jmondi/oauth2-server/dist/adapters/fastify'

Typechecks work but when I try to start my fastify server it does not find the module.
After looking into the node_modules-Folder inside @jmondi/oauth2-server/dist/adapters I only find the typing files (*.d.ts) but no code. Also when searching inside the oauth2-server.cjs.development.js the function declaration can't be found.
Are there specific typescript options I have to set for the import to work?
Update: I also checked out the prisma-fastify-example and tried to start it up. Same error.

🧐 will look into this one shortly and will try to have a fix up asap

I've pushed an update for this one in v2.0.3-rc.0. I did this one under an RC because it was a bit riskier because I removed tsdx and replaced it directly with tsc, which was something I've been wanting to do for a while now.
Either pull the latest master if you are testing directly in the examples, or use > v2.0.3-rc.0 from npmjs and you should be good to go. It would be great if you were able to verify this worked for you as well.
Thanks for posting these issues!

On the first glance it works for me now, thank you a lot for your fast response and fix!


sweep_tests/pytests/tt_dnn/test_frac.py unsupported on BH

Enabled in https://github.com/tenstorrent/tt-metal/pull/14199


Relicense under dual MIT/Apache-2.0

This issue was automatically generated. Feel free to close without ceremony if
you do not agree with re-licensing or if it is not possible for other reasons.
Respond to @cmr with any questions or concerns, or pop over to
#rust-offtopic on IRC to discuss.
You're receiving this because someone (perhaps the project maintainer)
published a crates.io package with the license as "MIT" xor "Apache-2.0" and
the repository field pointing here.
TL;DR the Rust ecosystem is largely Apache-2.0. Being available under that
license is good for interoperation. The MIT license as an add-on can be nice
for GPLv2 projects to use your code.
Why?
The MIT license requires reproducing countless copies of the same copyright
header with different names in the copyright field, for every MIT library in
use. The Apache license does not have this drawback. However, this is not the
primary motivation for me creating these issues. The Apache license also has
protections from patent trolls and an explicit contribution licensing clause.
However, the Apache license is incompatible with GPLv2. This is why Rust is
dual-licensed as MIT/Apache (the "primary" license being Apache, MIT only for
GPLv2 compat), and doing so would be wise for this project. This also makes
this crate suitable for inclusion and unrestricted sharing in the Rust
standard distribution and other projects using dual MIT/Apache, such as my
personal ulterior motive, the Robigalia project.
Some ask, "Does this really apply to binary redistributions? Does MIT really
require reproducing the whole thing?" I'm not a lawyer, and I can't give legal
advice, but some Google Android apps include open source attributions using
this interpretation. Others also agree with
it.
But, again, the copyright notice redistribution is not the primary motivation
for the dual-licensing. It's stronger protections to licensees and better
interoperation with the wider Rust ecosystem.
How?
To do this, get explicit approval from each contributor of copyrightable work
(as not all contributions qualify for copyright, due to not being a "creative
work", e.g. a typo fix) and then add the following to your README:
## License

Licensed under either of

 * Apache License, Version 2.0 ([LICENSE-APACHE](LICENSE-APACHE) or http://www.apache.org/licenses/LICENSE-2.0)
 * MIT license ([LICENSE-MIT](LICENSE-MIT) or http://opensource.org/licenses/MIT)

at your option.

### Contribution

Unless you explicitly state otherwise, any contribution intentionally submitted
for inclusion in the work by you, as defined in the Apache-2.0 license, shall be dual licensed as above, without any
additional terms or conditions.

and in your license headers, if you have them, use the following boilerplate
(based on that used in Rust):
// Copyright 2016 gfx_gl developers
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

It's commonly asked whether license headers are required. I'm not comfortable
making an official recommendation either way, but the Apache license
recommends it in their appendix on how to use the license.
Be sure to add the relevant LICENSE-{MIT,APACHE} files. You can copy these
from the Rust repo for a plain-text
version.
And don't forget to update the license metadata in your Cargo.toml to:
license = "MIT/Apache-2.0"

I'll be going through projects which agree to be relicensed and have approval
by the necessary contributors and doing this changes, so feel free to leave
the heavy lifting to me!
Contributor checkoff
To agree to relicensing, comment with :
I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

Or, if you're a contributor, you can check the box in this repo next to your
name. My scripts will pick this exact phrase up and check your checkbox, but
I'll come through and manually review this issue later as well.

[x] @bjz
[x] @csherratt
[x] @kvark
[x] @vickenty
[x] @mitsuhiko
[x] @bvssvni
[ ] @Potpourri
[x] @cmr
[x] @MaikKlein
[x] @indiv0

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

@kvark could you add a comment?

@bjz it says write OR check:

Or, if you're a contributor, you can check the box in this repo next to your
name.

Do I misunderstand this statement?
Anyway, I agree with the proposal and give my concent.

Ah, my mistake!

Seems more watertight to ask for an explicit comment though - somebody else could click the checkbox by accident. :/

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

chose and choose

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

@Potpourri ping


Typo in WMS Request

Hello,
I've noticed there is a typo in the WMS request. Please refer to #686 for details.
 Thank you @GuittonHubert !


Incompatibility with Better Combat mod in Dynamic Mode

Forge 47.2.32, Minecraft 1.20.1
Latest version of First Person mod and Better Combat
When vanilla hands and dynamic mode are enabled better combat animations when you look up dissapearing and you just using vanilla combat

Yea good chance for that not to work. Don't try to combine these features for now.

will there be compatibility for this in the future. ive known this to be an issue forever. i was told by the developer of better combat that he will not be fixing it. would be great if this was compatible

Just turn off dynamic mode. It's a new feature, no it's not compatible(for now).

i meant that a long time ago before dynamic mode existed, i tried to combine better combat with first person using the vanilla hands only mode which removed the rendering of the arms but kept the body. I wanted a hybrid vanilla experience with vanilla arms but having the ability to look down at my body. still the same issue back then.

Then why mention dynamic mode in the first place when you mean vanilla hands?

Then why mention dynamic mode in the first place when you mean vanilla hands?

It wasn't me

Yeah there's two different people in this thread lol

Ah, same no icon, didn't notice that 😅

https://github.com/ZsoltMolnarrr/BetterCombat/pull/370 fix in here

i usually was going to just play with the vanilla hands only without dynamic mode, so for me, as long as the issue of the rendering of the animations for vanilla hands is fixed, then its perfect. how do i download the updated files?

You'll have to wait for the fix to be implemented in BetterCombat, or build that version yourself, as it doesn't have CI setup.


Add Avatar component

Pull Request checklist
This PR is based on #37, thus in draft. It adds the Avatar component to display an avatar in a reusable way.
Only the last commit is from this PR.
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build was run locally and any changes were pushed
[ ] Lint has passed locally and any fixes were made for failures

Please check the type of change your PR introduces:

[ ] :bug: Bugfix
[ ] :sparkles: Feature
[ ] :lipstick: :art: Code style update (formatting, renaming)
[x] :recycle: Refactoring (no functional changes, no api changes)
[ ] :zap: Performance
[ ] :pencil: Documentation content changes
[ ] :wrench: Other (please describe):

Good one. May be best to decide whether the size is determined by the passed in size prop, or the classes. Another thing to think about could be to show the empty avatar icon when avatar is not defined (rather than defaulting to the first avatar image).
This is the one I am referring to.

The empty avatar icon will be easy to include once #49 is merged.

Moved 3237f6e559ea9b705611e4bcf83635e24433e09d to #37 directly. Closing.


chore(common-proc): Suppress doc warning

Overview
Suppresses the missing docs warning in the #[client_entry] macro expansion.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.0%. Comparing base (e7605ed) to head (631f3f1).

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Problem with FetchNonLocalSBOL in getSBOL.js

This method throws this exception when the get returns invalid SBOL:
/Users/myers/git/synbiohub/node_modules/async/lib/async.js:30
if (called) throw new Error("Callback was already called.");
^
Error: Callback was already called.
at /Users/myers/git/synbiohub/node_modules/async/lib/async.js:30:31
at /Users/myers/git/synbiohub/node_modules/sboljs/lib/load/rdf/loadRDF.js:68:9
at /Users/myers/git/synbiohub/node_modules/rdf-parser-abstract/index.js:26:13
at _combinedTickCallback (internal/process/next_tick.js:67:7)
at process._tickCallback (internal/process/next_tick.js:98:9)
This happens for example when the response is HTML.  This currently happens when the part cannot be fetched, since an HTML error page is sent back.  While changing this to respond with an error instead perhaps by using a different end point will solve this particular problem, it will not help when the page being dereferenced for some reason does return html.  So, it would be good to figure out how to fix this error.

Checked in hack to see if HTML served back, but still should fix the root of the problem which is loadRDF does not seem to fail correctly when it is not given an SBOL file.

Should be less of a problem if we use separate endpoint.  Also should only be called if it is in the web of registries.

Added a new endpoint, also only fetch when found in registry in the webOfRegistries.


"Segmentation fault: 11" with @extend?

I'm trying to compile https://github.com/Dogfalo/materialize with CSS::Sass, but I'm getting "Segmentation fault: 11" in OSX 10.11.4, Perl v5.22.0 and CSS::Sass 3.3.0.
The way to reproduce it is:
git clone https://github.com/Dogfalo/materialize.git
cd materialize
perl test.pl

The content of test.pl:
#!/usr/bin/env perl
use strict;
use warnings;
use CSS::Sass ();

CSS::Sass::sass_compile(<<'HERE', include_paths => ['sass']);
@charset "UTF-8";
@import "components/color";
@import "components/variables";
@import "components/cards";
@import "components/buttons";
HERE

I've tried different things, but it seems like I'm able to compile it, if I comment out @extend .btn; in https://github.com/Dogfalo/materialize/blob/master/sass/components/_buttons.scss#L185

Pretty sure this is related to libsass directly and probably a direct duplicate of https://github.com/sass/libsass/issues/1916. Therefore closing this and tracking upstream. If you think this is a different error, feel free to open an issue directly with libsass. If you want to get it solved faster please provide a standalone and reduced test case as we normally do not dig into other people frameworks. Thanks!

Thank you! Yeah, that sounds spot on.

OK, hope we can get a fix out soon, but don't quote me on that ;)


MacVim window doesn't open when opening files from Finder, mvim from terminal works

I recently upgraded to El Capitan (10.11.3) and run the homebrew macvim.
I have the latest version running 7.4-98
Here's what's up.
Scenario 1
I can open my alias inside /Applications that I linked using brew linkapps macvim.
MacVim totally works fine. I can open files from Finder and it will load in a new buffer in MacVim.
Scenario 2
When MacVim is not already open. I will open a file from Finder.
MacVim Starts, you can see a window flash on the screen, and then nothing. MacVim can't open any windows, not even new plain windows.
Scenario 3
Interestingly, if I run mvim ~/myfile_I_want_to_open.txt in the terminal, it opens MacVim just fine and will work just the same as Scenario 1
So it seems like Finder just doesn't like opening MacVim right now.
Any ideas?

sounds the same as https://github.com/macvim-dev/macvim/issues/109

Typing gvim after a brew install MacVim (7.4 as of this post) opens MacVim and shows an icon in the macOS dock.  right click it and choose options and keep in dock.  it also shows up in the context menu right clicking a file in the finder.  you an also drag a doc to the icon in the dock.  I removed the Vim.app that was installed in the applications folder.  it crashed after bouncing a few times in the dock.  vim at the command line opens a shell vim.


Incorrect handling of aspect ratio change during onResize

The handling of onResize in web-ifc-viewer is flaky:

when I manually drag the window to change sizes, the aspect ratio changes mostly work, though it looks a little jittery.
but when I just toggle the jsconsole, I reliably get a squashing effect.  This is undone and correst ratio restored when I then close it.

It looks like this problem is only in web-ifc-viewer, as the same steps don't reproduce it in web-ifc-three.
A brief code inspection (and added console loggings as shown in screens below) shows that the resize methods are being called in Viewer, but they're handled separately, in camera.ts and renderer.ts.  So maybe this a race condition during event propagation?  Consolidating the handling in a single method might fix this.
Actual:

Expected (from web-ifc-three):

Hey @pablo-mayrgundter, I managed to fix this in 1.0.35. I'm taking the dimensions from the canvas's container element instead.
Check this here: https://github.com/IFCjs/web-ifc-viewer/blob/5da750f9d9f95eb31dba9b6d735902ee00840bb4/viewer/src/components/context/context.ts#L63


AppWindowPresenterKind.FullScreen generating empty space on first run

Describe the bug
The current WinUI sample for Presenter types:
https://github.com/microsoft/WindowsAppSDK-Samples/tree/main/Samples/Windowing/cs-winui/SampleApp
Contains a method for setting WinUI apps full screen. This works but the first time AppWindowPresenterKind.FullScreen is applied it results in empty space along the right hand side and bottom of the display. After exiting full screen and then re-applying it the empty space is used properly.
Steps to reproduce the bug
Run the sample app, click to enter full screen mode, observe the empty space. Exit full screen and then re-enter it and observe the space used properly.
Expected behavior
Should use all available space on first run.
Screenshots
https://imgur.com/a/CK96wSi
NuGet package version
WinUI 3 - Windows App SDK 0.8: 0.8.0 (If you're seeing your issue in older previews of WinUI 3, please try this release)
Windows app type

[x] UWP
[ ] Win32

Device form factor
Desktop
Windows version
Insider Build (xxxxx)
Additional context
No response

Related to https://github.com/microsoft/WindowsAppSDK/issues/1853

This is a dup of #1853.

This is a dup of #1853.

well that is a dup of this :) I logged it 3 weeks earlier

Well, that dup was bound to 1.1 milestone recently. I suggest to close the current one as it contains the same repro.
The only diffrence is relation to 0.8 vs 1.0 SDK. Also if backport to 0.8 is needed then better to suggest it there.


Pulsar schema service should retry on concurrent schema updates

Describe the bug
Currently Pulsar doesn't retry on exceptions thrown when concurrently updating same schema. So the exception is propagated to Pulsar client directly. This is not a good experience for clients.
We should consider handling errors and retry it again.
Example exceptions are:
org.apache.zookeeper.KeeperException$BadVersionException: KeeperErrorCode = BadVersion
	at org.apache.pulsar.client.api.PulsarClientException.unwrap(PulsarClientException.java:297) ~[pulsar-client-api-2.5.0-2cc34afc0.jar:2.5.0-2cc34afc0]
	at org.apache.pulsar.client.impl.ProducerBuilderImpl.create(ProducerBuilderImpl.java:88) ~[pulsar-client-all-2.5.0-2cc34afc0.jar:2.5.0-2cc34afc0]

@sijie I'd like to work on this. How should I proceed?

Hi, what's the recommended work around ? I'm getting this intermittently while trying to a 16-thread producer on 16-core machine, 1 time out of 2-3 runs

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.


Thorium_AVX_109.0.5414.172 crash on launch

The Thorium_AVX_109.0.5414.172.zip doesn't work for me.
Replacing my old AVX1 version (thus USER_DATA is kept) causes an empty window to appear and then a crash to desktop.
Trying to start with an empty USER_DATA also crashes (but it takes longer to crash as it is creating a profile, etc.)
Here's the crashpad report:
21c1feb4-788e-4786-a67d-f7fb96026ac1.dmp
Is there anything else I can provide?

System specs:
Templayer Win7 PC system specs.zip

I think you need to use SSE3 version

I think you need to use SSE3 version

Why? I have been using the AVX1 version for months now. Just this update borked it.
And if you were to look into the specs I have provided, my CPU supports AVX1.
If it was that issue, the application wouldn't crash, because it wouldn't even launch...

I think you need to use SSE3 version

Why? I have been using the AVX1 version for months now. Just this update borked it.
And if you were to look into the specs I have provided, my CPU supports AVX1.
If it was that issue, the application wouldn't crash, because it wouldn't even launch...

Maybe some optimization parameters are not supported by your processor, I still recommend you to test the SSE3 version

@Templayer Let me test

It no longer crashes after a restart (the crashlog is a proof that it did crash originally, though).
I'm too used to Linux now - under Linux Mint, restarting won't randomly solve problems, so it didn't occur to me to try that. :D :D


Autocorrelation fix

Fixed the autocorrelation formula for nan cases (mostly constant sequences) and the terms in the denominator of the autocorrelation

Awesome @FrancescoPinto!  Merging now.


Document download pdf

add the download pdf part on the gem

@WaKeMaTTa there was some issue so I requested you again


Don't use this.setState() in componentWillUpdate()

According to the chapter "Tapping into componentWillUpdate()" it's safe to call setState() without triggering another update.
However, after looking it up in the docs and testing it for myself it looks like this is wrong, see the note on https://facebook.github.io/react/docs/component-specs.html#updating-componentwillupdate : "You cannot use this.setState() in this method. If you need to update state in response to a prop change, use componentWillReceiveProps instead."

Hi Robin,
Thanks for pointing this out. This was something that was not clear in the docs originally (I really don't recall seeing that warning last time I read that page) and in the past it seemed ok, but it is risky due to how the dirty/render process is managed. I talked to the Reactiflux internals peeps and got a better explanation on why this is bad. The issue is that setState() triggers componentWillUpdate() with no check, so unless a flag is used on the setState() this will cause an infinite loop.
I will be updating this ASAP and I was passed a good chart that shows this in more detail. Thanks again for calling this out to my attention!

Hello James,
You're welcome, this just happened to catch my attention. It's probably one of the lesser used lifecycle methods. Thanks for providing this free book, definitely worth a read and I've learned a few things. Looking forward to see the updated information!

All updated in #4, thanks again!


Focus topic input and open typeahead on compose in stream narrow

As a follow up to #4773 / #4764, we should do the following when you start composing a new message in a stream narrow:

Put the cursor in the topic field.
Automatically pop up the typeahead. This may come for free with (1).

This is for the experience when you're composing a new message. When implementing this, we should check that when you're editing a message instead, the focus still goes first to the message-content input.

It looks like we have this already? Should this issue be closed?

It looks like we have this already? Should this issue be closed?

IIUC, we got the compose-new-message part recently, in e21e7dd94 (released in v27.170). I'm not sure about the edit-message behavior that Greg says we should check. From briefly reading the code, I suspect that's still working as intended, but it'd be good to check. Would you mind checking, Alya?

Yep, the focus for editing is still on the message, at least on Android.


[FIX & FEATURE] fix bug with build & add id's on questions

Description
Accidentally didn't test build script on #126 and merged it even if the build script didn't work. Reverted all changes back so everything works now.
As proposed in #114 I added id's to each question so people can now share questions simply by sharing link/#question-name
Resolves #114
What does your PR belong to?

[ ] Questions / Answers
[x] Website
[ ] General / Things regarding the repository (like CI Integration)

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (non-breaking improvement of a question)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have checked that the changes are working properly
[x] I have checked that there isn't any PR doing the same
[x] I have read the CONTRIBUTING document.

Changes made! @atomiks I'll let you merge this one 🚀

@atomiks the live version of the http://30secondsofinterviews.org doesn't work. Could you take a look at this, please?

The cron hasn't run yet

@flxwu how do we manually run cron job on Travis? What was the command?

--force-build in the commit message


Update modernize-userinterface-site-pages-ui.md

Category

[ ] Content fix
[ ] New article
[x] Example checked item (delete this line)

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

Related issues:

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.
(DELETE THIS PARAGRAPH AFTER READING)

What's in this Pull Request?

Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
(DELETE THIS PARAGRAPH AFTER READING)

Guidance

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'master' branch. Released documents are in live branch.
(DELETE THIS PARAGRAPH AFTER READING)

In the future, please describe what the change is that you're submitting in the PR.


Custom model for multisensor environments

Hello everyone,
I would like to create an algorithm to train multi sensor agent using your DRL framework.
What I have in mind is concatenating one or more convolutional layers whose input could be cameras or lidar sensor with 1D arrays from other sensors (such as GPS).
It looks like I should add an option to inputs.py and a custom model to manage this kind of environments. Would this be enough? Do you have any suggestion?
Thanks,
Simone

Related to #133

arrafin beat me to it again <.<.
Just for more direct link: Here is an example on how to combine visual observation with 1D vector: https://github.com/hill-a/stable-baselines/issues/133#issuecomment-561805417

Araffin, Miffly,
thank you for your answers. Actually I knew about the issue you linked, but I opened a new one for two reasons

Creating a new layer in the image input to fill it only with some data would mean wasting a lot of memory (the appended layer would be mostly padded zeros)
This approach wouldn't be feasible if there are more convolutional inputs with different resolutions.
So, back to my question: what should I change (besides, obviously, a custom model) to manage multi-sensor environments?
Thanks,
Simone

As discussed in #133, true multi-modal observations are not currently possible and you have to resort to this kind of dirty hacks for now. However this is the very next thing on the to-do list after TF2 support, which is slowly getting there but currently on a hiatus due to holidays :)


Ltw ms

I don't think the last pull was marked after it was approved

What do you mean by “was marked”?
Marlon Shaw
Web Content Developer Sr
Online and Creative Services
Corporate Communications and External Relations
Office:<PHONE_NUMBER>
BB:<PHONE_NUMBER>
Telecommute – Friday
[http://www.freddiemac.com/images/logos/fm_email_sig.png]
From: S Carrero<EMAIL_ADDRESS>Sent: Friday, September 07, 2018 6:27 PM
To: SCarrero/fmzurb<EMAIL_ADDRESS>Cc: Shaw, Marlon<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [SCarrero/fmzurb] Ltw ms (#175)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
I don't think the last pull was marked after it was approved
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/SCarrero/fmzurb/pull/175#issuecomment-419582484, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Abj9fcAaQNYt-C0COimweMTtV9Ow3Up1ks5uYvKbgaJpZM4WdZIL.

I changed it to "merged" 2 seconds after it posted.  Hard to see comments on my phone.

Sherry - can you review and merge to master?

approved, but unable to merge into master due to conflicting files.

Oh boy. Let me take a look at resolving the conflict.
Marlon Shaw
Web Content Developer Sr
Online and Creative Services
Corporate Communications and External Relations
Office:<PHONE_NUMBER>
BB:<PHONE_NUMBER>
Telecommute – Friday
[http://www.freddiemac.com/images/logos/fm_email_sig.png]
From: S Carrero<EMAIL_ADDRESS>Sent: Monday, September 10, 2018 12:53 PM
To: SCarrero/fmzurb<EMAIL_ADDRESS>Cc: Shaw, Marlon<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [SCarrero/fmzurb] Ltw ms (#175)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
approved, but unable to merge into master due to conflicting files.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/SCarrero/fmzurb/pull/175#issuecomment-419983130, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Abj9fQxwXTaZZFjOBWdh31KKd9GCF6Maks5uZpjpgaJpZM4WdZIL.

deleting the ltw-ms branch -- it has been merged into master.


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/ioBroker/ioBroker.sonoff

[x] [E021] "licenses" in package.json are depricated. Please use only "license": "NAME" field.
[x] [E301] Tests on Travis-ci.org are broken. Please fix.

Thanks,
your automatic adapter checker.

Thanks, that all bugs have been fixed.


Office Hours Feature

Office Hours Feature
Authors: Meghan Sun, Bailey DeSouza, Madelyn Andrews, Sadie Amato
Feature Description
First and foremost, this feature aims to recreate Course Care within the tech stack of the XL site, while redesigning and enhancing its features. The main functionality included here is the ticket workflow enabling connections between students and course staff.
Students will be able to check into events, create tickets outlining their concerns, and be notified via sound when it is their turn. Staff members can also check into events, and they will be able to see a queue of tickets, be notified via sound when a new ticket arrives, call a ticket, and review the student. Aside from the ticket workflow, instructors, staff, and students will have various abilities, including opting in to using the system, creating events, and viewing usage statistics.
Directory Structure
Models of Office Hours Sections, Events, and Tickets were created on the frontend and backend to represent these ideas. Frontend models can be found in /frontend/src/app/office-hours/office-hours.models.ts, and the backend models are found within the /backend/models/office_hours/ folder. The related entities were also created in /backend/entities/office_hours/, and the details of those are outlined here.
In addition, our models are passed through the feature via components, a frontend service, backend API, and backend services. Respectively, those are found in the following directories:

/frontend/src/app/office-hours/
/frontend/src/app/office-hours/office-hours.service.ts
/backend/api/office_hours/
/backend/services/office_hours/

Our unit tests can be found in the /backend/test/services/office_hours/ directory.

✅
@sadieamato @meghansun322 @baileymeredith @maddyandrews
Entities CR LGTM!

@sadieamato @maddyandrews @meghansun322 @baileymeredith Looks great!! Almost there, just one last test to add 😄
for: "If the student currently has a queued ticket, don't allow new ticket creation"
This will get the tickets test coverage to 100%!

✅ Backend Services and Models CR Re-Review
@sadieamato @meghansun322 @baileymeredith @maddyandrews
Services and models (and the rest of the frontend) LGTM!

Went to try and generate the migration to deploy before heading out of town and ran into a couple of issues:

Reset Demo is no longer showing an active term so the office hours screen fails to load
There are some conflicts which need to be resolved with main (ideally via a merge commit pushed to this branch) as shown in the status of the PR

@KrisJordan / @sadieamato merge commit added, tests remain passing and at 100% coverage.


names.push is not a function

I was getting the following error from your plugin
names.push is not a function

I dug into it and its related to the dependency injection library: dependable. The library is in poor shape and should be dropped.

It wasnt dependable. It was lodash for some odd reason. The only real reason you were using lodash for was the _.range function but that was so easily replaceable.
Once I removed lodash from the project it start working.
I will create a PR with all my fixes soon.

Just created PR #8 which fixes this bug.


broot: install man page

Motivation for this change
broot has a man page, let's install it!
Things done

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

[x] NixOS
[ ] macOS
[ ] 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 nixpkgs-review --run "nixpkgs-review wip"
[x] 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
[x] Fits CONTRIBUTING.md.

@bbigras since you did some of the recent updates, I thought you might be a good reviewer for this PR ;).

Oh too late. :laughing:


rohrric crafting tables

version 1.7.10
map version 2.9.4
"Rohirric Crafting tables" spawning all around the place in the game.

In the new generated chunks that is reloaded map and client just to make sure didn't help