Not Working

Plugin Fails for me. The following results in empty file.
Gulp Task
var gulp = require("gulp");
var gSass = require("gulp-sass");
var gNotify = require("gulp-notify");
var moduleImporter = require("sass-module-importer");

gulp.task("sass", function(){
  return gulp.src(["main.scss"])
    .pipe(gSass({
      importer: [
        moduleImporter()
      ]
    }))
    .on("error", gNotify.onError())
      title: "SASS Compile Error",
      message: "<%= error.message %>"
    }))
    .pipe(gulp.dest(CONFIG.dist.styles));
});

main.scss
@import "normalize-scss";

NPM Package
Testing with: Normalize

Same with colors.css for me

Same with https://www.npmjs.com/package/sass-module-importer

Not working here also:
var gulp = require('gulp'),
	sass = require('gulp-sass'),
    moduleImporter = require('sass-module-importer'),

gulp.task('styles', function () {
	return gulp.src('./searchBox.scss')
		.pipe(sass({
            importer: moduleImporter()
        }))
		.on('error', sass.logError)
		.pipe(gulp.dest('./'));

Cannot import bootstrap and bourbon, too. Using Laravel Elixir
elixir(function(mix) {
    mix.sass('app.scss', null, { importer: moduleImporter() });
    mix.livereload(null, {
    	port: 35729
    });
});

Same, not working

Hey guys,
I've mentioned on #2 that the original implementation of this plugin was very naive.
I just released a new version (1.2.0) and tested with the following modules:
@import "bulma";
@import "colors.css";
@import "normalize-scss";
@import "sass-spacing";
@import "font-awesome";
// or even require files inside that module, like bellow:
@import "font-awesome/scss/font-awesome.scss";

// mixin from normalize-scss
@include normalize();

.tile {
  // mixins from sass-spacing
  @include p(1);
  @include mt(2);
}

And this time, no problems. The dependencies are being properly resolved on a more robust way.
It will even inline css files, since Sass cannot do that (e.g. font-awesome is css).
I'm very confident that this will resolve most of the issues you guys were having. I will close it for now, but let me know if it comes back.

I'm still running into somewhat of a conflict with the "colors" dependency in NPM colliding with a _colors.scss import ( as @import "colors" ).
I was able to resolve my issue by namespacing the file as _mysite_colors.scss and now everything compiles perfectly.
Only running into this since upgrading to NPM 3 with the flat file structure.

Hey @Radau - I'm working on it at the moment.
But there are two different issues from what I can see.

One of them is to distinguish between a local file vs a module. But that's not really a issue, you should use ./ on the local file, otherwise it would be impossible to know which one to import:
@import 'colors'; /* returns the "colors" module */
@import './colors'; /* returns the local "colors" file */

The other one is the one I'm working on. At the moment, every single import goes through this library and is resolved.
So in the next release, this library will only handle external modules and it will ignore the local ones. So it means it shouldn't conflict or cache local files anymore.


allow user to select between google accounts on login

allow user to select which google account to log in with, rather than defaulting to whatever the users is logged in currently as

When I try either Login or Sign-up, in both those cases, Google already shows me a screen that says:
Choose an account
With multiple choices...
Do you not see that option?

@sdarwin you get this behaviour if you haven't already logged in with an account in that browser. If you have already logged in, it doesn't ask you what account you would like to use, and just sends you an email. It's rather confusing if you don't know what's happening. Most websites will ask as a default, and it's a minor settings fix.

Hmm, ok.  The behavior I described above happens in these cases:

You are already logged into multiple gmail accounts
You are not logged into any accounts

The issue you are solving occurs when:

You are logged into only one account.

If it's just a minor settings adjustment, that makes sense.


Feature Request: Footnote Support

Loving SunEditor so far, but I do am missing a footnote feature. I tried looking through the docs, but couldn't find such functionality yet. If I missed anything, please point me to it. Otherwise, it would be awesome to have Footnote Support within SunEditor.

I'll add it to version 3.0.0.


Doc page lists unsupported parameters: dotnet list packages does not actually support --format parameter.

Using dotnet cli 7.0.102 with the list command, the --format parameter listed on the page is not supported or there is some additional command not obvious from the docs that needs to be applied.
run this command: dotnet list package --format json
Result leads with these errors:
Unrecognized command or argument '--format'
Unrecognized command or argument 'json'
running dotnet list --help does not list the --format parameter, so I believe the doc page is out of date for this version. Doc says it's applicable to Core 3.1+
Logging a bug b/c I'm honestly not sure if this shouldn't be a bug report to the CLI instead.
[Enter feedback here]

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

ID: 8400fd6a-d987-1056-283f-9364fbce3c86
Version Independent ID: 3d4b052c-33fa-cc83-4df6-4a3341f12763
Content: dotnet list package command - .NET CLI
Content Source: docs/core/tools/dotnet-list-package.md
Product: dotnet-fundamentals
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Associated WorkItem - 65771

@jimleonardo Thanks for reporting that. I'm adding notes explaining that these options (--format and --output-version) are available starting in SDK 7.0.200.


View rendered dispited SharedManager attach redirect

I have a simple ACL
if (!$e->getViewModel()->acl->hasResource($route) || !$e->getViewModel()->acl->isAllowed($role, $route)) {
        //Naughty trying to get somewhere they shouldn't (Clear there identity force them to login again)
        $response = $e->getResponse();
        //location to page or what ever
        if($request->isPost())
        {
            $response->setStatusCode(401);
            return $response;
        }
        else
        {
            return $e->getTarget()->getEventManager()->getSharedManager()->attach('Zend\Mvc\Controller\AbstractActionController', 'dispatch', function($e)  {
                $controller = $e->getTarget();
                $controller->redirect()->toRoute('auth');
            }, 1000);
        }
    }

after i added a sql-query that need the userID the redirect gets interrupted on my dev-server and shows the notice that no userID is defined.
On the prod-server the redirect works but i still get a error_log entry that the userID is not defined. That means that the redirect is executed AFTER the Action is called.
In my understanding it should redirect before everything else in DISPATCH.

Didn't read, some issue in posted code: return $controller->redirect()->toRoute('auth');

ah i understand, to interrupt the code i have to return the redirect...
sry can be closed


dsub for aws

Hi,
I have previously used dsub with google cloud and was very satisfied with it. Is there a way that I can use it for AWS batch operations?
If not, do you know of anything like dsub that would work with amazon?
thanks!

We would love for someone to contribute an AWS provider!
We've got an architecture to support different providers and documentation of the details of the environment that a new provider would be expected to provide.


[cmake] ceres specify SuiteSparse as required

Need to add it on travis.

Now rebased on popart_develop.

also, while at it, even if is a bit out of scope, we can remove the FindCeres.cmake in src/cmakemodules, as it was previously used by cctag and it is not the case anymore (and also is quite wrong)

@simogasp: I don't understand you remark regarding FindCeres.cmake. It has been removed in cctag repo, but here it's needed.

nope, if you look at the history it has been added with FindCCTag by Lilian. When we build ceres we don't use the FindCeres, we give the path of the CeresConfig.cmake (which is better as it contains the definitions used for compiling the lib etc)


Build failure: qtbase:x64-windows

Host Environment

OS: Windows 10
Compiler: MSVC

To Reproduce
Steps to reproduce the behavior:
.\vcpkg install qtbase
Failure logs
Computing installation plan...
The following packages will be built and installed:
  * qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x64-windows -> 6.2.2#2
    qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x86-windows -> 6.2.2#2
  * sqlite3[core]:x86-windows -> 3.37.0#1
  * zstd[core]:x86-windows -> 1.5.2
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
Detecting compiler hash for triplet x86-windows...
Restored 0 packages from C:\Users\Redouane\AppData\Local\vcpkg\archives in 539.9 us. Use --debug to see more details.
Starting package 1/4: qtbase:x64-windows
Building package qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x64-windows...
-- Using cached F:/Software/DEV/vcpkg/downloads/qtbase-61fbf7596beda0178e213a7ba945bc0314153366.tar.gz
-- Cleaning sources at F:/Software/DEV/vcpkg/buildtrees/qtbase/src/0314153366-104b8f293d.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source F:/Software/DEV/vcpkg/downloads/qtbase-61fbf7596beda0178e213a7ba945bc0314153366.tar.gz
-- Applying patch allow_outside_prefix.patch
-- Applying patch clang-cl_source_location.patch
-- Applying patch config_install.patch
-- Applying patch dont_force_cmakecache.patch
-- Applying patch fix_cmake_build.patch
-- Applying patch harfbuzz.patch
-- Applying patch fix_egl.patch
-- Using source at F:/Software/DEV/vcpkg/buildtrees/qtbase/src/0314153366-104b8f293d.clean
-- Configuring x64-windows
CMake Warning at scripts/cmake/vcpkg_configure_cmake.cmake:410 (message):
  The following variables are not used in CMakeLists.txt:

      CMAKE_DISABLE_FIND_PACKAGE_CUPS
      CMAKE_DISABLE_FIND_PACKAGE_PPS
      CMAKE_DISABLE_FIND_PACKAGE_X11
      CMAKE_DISABLE_FIND_PACKAGE_XCB
      CMAKE_DISABLE_FIND_PACKAGE_XKB
      CMAKE_DISABLE_FIND_PACKAGE_XKB_COMMON_X11
      CMAKE_DISABLE_FIND_PACKAGE_XRender
      FEATURE_appstore-compliant
      INPUT_sqlite
      QT_NO_MAKE_EXAMPLES
      QT_NO_MAKE_TESTS

  Please recheck them and remove the unnecessary options from the
  `vcpkg_configure_cmake` call.

  If these options should still be passed for whatever reason, please use the
  `MAYBE_UNUSED_VARIABLES` argument.
Call Stack (most recent call first):
  ports/qtbase/cmake/qt_install_submodule.cmake:88 (vcpkg_configure_cmake)
  ports/qtbase/cmake/qt_install_submodule.cmake:257 (qt_cmake_configure)
  ports/qtbase/portfile.cmake:216 (qt_install_submodule)
  scripts/ports.cmake:142 (include)

-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:156 (message):
    Command failed: F:/Software/DEV/vcpkg/downloads/tools/cmake-3.21.1-windows/cmake-3.21.1-windows-i386/bin/cmake.exe --build . --config Release --target install -- -v -j9
    Working Directory: F:/Software/DEV/vcpkg/buildtrees/qtbase/x64-windows-rel
    See logs for more information:
      F:\Software\DEV\vcpkg\buildtrees\qtbase\install-x64-windows-rel-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_cmake.cmake:106 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_cmake.cmake:44 (vcpkg_build_cmake)
  ports/qtbase/cmake/qt_install_submodule.cmake:262 (vcpkg_install_cmake)
  ports/qtbase/portfile.cmake:216 (qt_install_submodule)
  scripts/ports.cmake:142 (include)

Error: Building package qtbase:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `.\vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+qtbase
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[qtbase]+Build+error
including:
  package: qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x64-windows -> 6.2.2#2
    vcpkg-tool version: 2021-12-09-724cea8b33cbf06c645f5095fa29773697da9761
    vcpkg-scripts version: 291b84e65 2022-01-18 (10 hours ago)

Additionally, attach any relevant sections from the log files above.

Additional context
I tried the following commands:
As suggested in the error report:
-> "git pull" and ".\vcpkg update"
As suggested in #22584:
.\vcpkg install qtbase[core,gui,doubleconversion]:x64-windows-static
.\vcpkg install qtbase[core,gui,doubleconversion,pcre2]:x64-windows-static

Please provide:    F:\Software\DEV\vcpkg\buildtrees\qtbase\install-x64-windows-rel-out.log
It is a bit suspicious that only the release build fails.

install-x64-windows-rel-out.log

C:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\winrt\wrl\module.h(1427): error C2039: 'CheckForDuplicateEntries': is not a member of 'Microsoft::WRL::Details'

https://help.appveyor.com/discussions/problems/26088-checkforduplicateentries-is-not-a-member-of-microsoftwrldetails
Probably an issue with the Windows SDK. Not related with the port itself, so not my issues ;P

And what is this Windows SDK?

And what is this Windows SDK?

Search engine (e.g. google)?
https://en.wikipedia.org/wiki/Microsoft_Windows_SDK
C:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\ <- That is the Windows SDK installation .
It gets installed alongside Visual Studio unless you manually untick it. Remove that version and replace it with something newer (at least 10.0.18362.0 as indicated by the linked discussion in https://github.com/microsoft/vcpkg/issues/22632#issuecomment-1016292420)

Thank you, I'll try it and report back with the results

The installation now has completed without any errors, now all is left is to verify the code that I wrote if it works properly.
Thank you so much for your help!
`Computing installation plan...
The following packages will be built and installed:

qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x64-windows -> 6.2.2#2
qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x86-windows -> 6.2.2#2

sqlite3[core]:x86-windows -> 3.37.0#1

zstd[core]:x86-windows -> 1.5.2
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
Detecting compiler hash for triplet x86-windows...
Restored 0 packages from C:\Users\Redouane\AppData\Local\vcpkg\archives in 542.5 us. Use --debug to see more details.
Starting package 1/4: qtbase:x64-windows
Building package qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x64-windows...
-- Using cached F:/Software/DEV/vcpkg/downloads/qtbase-61fbf7596beda0178e213a7ba945bc0314153366.tar.gz
-- Cleaning sources at F:/Software/DEV/vcpkg/buildtrees/qtbase/src/0314153366-104b8f293d.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source F:/Software/DEV/vcpkg/downloads/qtbase-61fbf7596beda0178e213a7ba945bc0314153366.tar.gz
-- Applying patch allow_outside_prefix.patch
-- Applying patch clang-cl_source_location.patch
-- Applying patch config_install.patch
-- Applying patch dont_force_cmakecache.patch
-- Applying patch fix_cmake_build.patch
-- Applying patch harfbuzz.patch
-- Applying patch fix_egl.patch
-- Using source at F:/Software/DEV/vcpkg/buildtrees/qtbase/src/0314153366-104b8f293d.clean
-- Configuring x64-windows
CMake Warning at scripts/cmake/vcpkg_configure_cmake.cmake:410 (message):
The following variables are not used in CMakeLists.txt:
CMAKE_DISABLE_FIND_PACKAGE_CUPS
CMAKE_DISABLE_FIND_PACKAGE_PPS
CMAKE_DISABLE_FIND_PACKAGE_X11
CMAKE_DISABLE_FIND_PACKAGE_XCB
CMAKE_DISABLE_FIND_PACKAGE_XKB
CMAKE_DISABLE_FIND_PACKAGE_XKB_COMMON_X11
CMAKE_DISABLE_FIND_PACKAGE_XRender
FEATURE_appstore-compliant
INPUT_sqlite
QT_NO_MAKE_EXAMPLES
QT_NO_MAKE_TESTS

Please recheck them and remove the unnecessary options from the
vcpkg_configure_cmake call.
If these options should still be passed for whatever reason, please use the
MAYBE_UNUSED_VARIABLES argument.
Call Stack (most recent call first):
ports/qtbase/cmake/qt_install_submodule.cmake:88 (vcpkg_configure_cmake)
ports/qtbase/cmake/qt_install_submodule.cmake:257 (qt_cmake_configure)
ports/qtbase/portfile.cmake:216 (qt_install_submodule)
scripts/ports.cmake:142 (include)
-- Building x64-windows-dbg
-- Building x64-windows-rel
-- Installing: F:/Software/DEV/vcpkg/packages/qtbase_x64-windows/share/qtbase/copyright
-- Installing: F:/Software/DEV/vcpkg/packages/qtbase_x64-windows/share/qtbase/qt_port_details.cmake
-- Installing: F:/Software/DEV/vcpkg/packages/qtbase_x64-windows/tools/Qt6/bin/qmake.debug.bat
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\Redouane\AppData\Local\vcpkg\archives\29\29bc6fe2914537f5377d71208b8d33f7754cd5375b731d825226606e145e24e1.zip
Installing package qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x64-windows...
Elapsed time for package qtbase:x64-windows: 23.79 min
Starting package 2/4: sqlite3:x86-windows
Building package sqlite3[core]:x86-windows...
-- Using cached sqlite-amalgamation-3370000.zip.
-- Cleaning sources at F:/Software/DEV/vcpkg/buildtrees/sqlite3/src/3370000-2bb42c52da.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source F:/Software/DEV/vcpkg/downloads/sqlite-amalgamation-3370000.zip
-- Applying patch fix-arm-uwp.patch
-- Using source at F:/Software/DEV/vcpkg/buildtrees/sqlite3/src/3370000-2bb42c52da.clean
-- Configuring x86-windows
-- Building x86-windows-dbg
-- Building x86-windows-rel
-- Fixing pkgconfig file: F:/Software/DEV/vcpkg/packages/sqlite3_x86-windows/lib/pkgconfig/sqlite3.pc
-- Using cached msys-mingw-w64-i686-pkg-config-0.29.2-2-any.pkg.tar.zst.
-- Using cached msys-mingw-w64-i686-libwinpthread-git-<IP_ADDRESS>06.c9a21571-1-any.pkg.tar.zst.
-- Using msys root at F:/Software/DEV/vcpkg/downloads/tools/msys2/aa5af7b2aa7e90e8
-- Fixing pkgconfig file: F:/Software/DEV/vcpkg/packages/sqlite3_x86-windows/debug/lib/pkgconfig/sqlite3.pc
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\Redouane\AppData\Local\vcpkg\archives\07\077809c9b7649f8e9bb274369c961eb00891b1515b0d1451f20dc8cf2246ea35.zip
Installing package sqlite3[core]:x86-windows...
Elapsed time for package sqlite3:x86-windows: 21.29 s
Starting package 3/4: zstd:x86-windows
Building package zstd[core]:x86-windows...
-- Using cached facebook-zstd-f4a552a3fa24d9078f84157bd40e4f1bad49c488.tar.gz.
-- Extracting source F:/Software/DEV/vcpkg/downloads/facebook-zstd-f4a552a3fa24d9078f84157bd40e4f1bad49c488.tar.gz
-- Applying patch install_pkgpc.patch
-- Using source at F:/Software/DEV/vcpkg/buildtrees/zstd/src/1bad49c488-399422de62.clean
-- Configuring x86-windows
-- Building x86-windows-dbg
-- Building x86-windows-rel
-- Fixing pkgconfig file: F:/Software/DEV/vcpkg/packages/zstd_x86-windows/lib/pkgconfig/libzstd.pc
-- Using cached msys-mingw-w64-i686-pkg-config-0.29.2-2-any.pkg.tar.zst.
-- Using cached msys-mingw-w64-i686-libwinpthread-git-<IP_ADDRESS>06.c9a21571-1-any.pkg.tar.zst.
-- Using msys root at F:/Software/DEV/vcpkg/downloads/tools/msys2/aa5af7b2aa7e90e8
-- Fixing pkgconfig file: F:/Software/DEV/vcpkg/packages/zstd_x86-windows/debug/lib/pkgconfig/libzstd.pc
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\Redouane\AppData\Local\vcpkg\archives\08\0894f4480ff4d49a35478ac9535881b8a066c0d007028f484cd3d75ac7e7b459.zip
Installing package zstd[core]:x86-windows...
Elapsed time for package zstd:x86-windows: 16.62 s
Starting package 4/4: qtbase:x86-windows
Building package qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x86-windows...
-- Using cached F:/Software/DEV/vcpkg/downloads/qtbase-61fbf7596beda0178e213a7ba945bc0314153366.tar.gz
-- Cleaning sources at F:/Software/DEV/vcpkg/buildtrees/qtbase/src/0314153366-104b8f293d.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source F:/Software/DEV/vcpkg/downloads/qtbase-61fbf7596beda0178e213a7ba945bc0314153366.tar.gz
-- Applying patch allow_outside_prefix.patch
-- Applying patch clang-cl_source_location.patch
-- Applying patch config_install.patch
-- Applying patch dont_force_cmakecache.patch
-- Applying patch fix_cmake_build.patch
-- Applying patch harfbuzz.patch
-- Applying patch fix_egl.patch
-- Using source at F:/Software/DEV/vcpkg/buildtrees/qtbase/src/0314153366-104b8f293d.clean
-- Configuring x86-windows
CMake Warning at scripts/cmake/vcpkg_configure_cmake.cmake:410 (message):
The following variables are not used in CMakeLists.txt:
  CMAKE_DISABLE_FIND_PACKAGE_CUPS
  CMAKE_DISABLE_FIND_PACKAGE_PPS
  CMAKE_DISABLE_FIND_PACKAGE_X11
  CMAKE_DISABLE_FIND_PACKAGE_XCB
  CMAKE_DISABLE_FIND_PACKAGE_XKB
  CMAKE_DISABLE_FIND_PACKAGE_XKB_COMMON_X11
  CMAKE_DISABLE_FIND_PACKAGE_XRender
  FEATURE_appstore-compliant
  INPUT_sqlite
  QT_NO_MAKE_EXAMPLES
  QT_NO_MAKE_TESTS

Please recheck them and remove the unnecessary options from the
vcpkg_configure_cmake call.
If these options should still be passed for whatever reason, please use the
MAYBE_UNUSED_VARIABLES argument.
Call Stack (most recent call first):
ports/qtbase/cmake/qt_install_submodule.cmake:88 (vcpkg_configure_cmake)
ports/qtbase/cmake/qt_install_submodule.cmake:257 (qt_cmake_configure)
ports/qtbase/portfile.cmake:216 (qt_install_submodule)
scripts/ports.cmake:142 (include)
-- Building x86-windows-dbg
-- Building x86-windows-rel
-- Installing: F:/Software/DEV/vcpkg/packages/qtbase_x86-windows/share/qtbase/copyright
-- Installing: F:/Software/DEV/vcpkg/packages/qtbase_x86-windows/share/qtbase/qt_port_details.cmake
-- Installing: F:/Software/DEV/vcpkg/packages/qtbase_x86-windows/tools/Qt6/bin/qmake.debug.bat
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\Redouane\AppData\Local\vcpkg\archives\d3\d35dec4f10605158943b2b7967c62dbd08940cc443a1a4947a14708c71e8a474.zip
Installing package qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,widgets,zstd]:x86-windows...
Elapsed time for package qtbase:x86-windows: 23.16 min
Total elapsed time: 47.7 min
The package qtbase provides CMake targets:
find_package(Qt6 CONFIG REQUIRED)
# Note: 46 target(s) were omitted.
target_link_libraries(main PRIVATE __PPS DB2::DB2 PPS::PPS Platform)

find_package(Qt6Concurrent CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::Concurrent Qt6::Concurrent Qt::ConcurrentPrivate Qt6::ConcurrentPrivate)

find_package(Qt6Core CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::Core Qt6::Core Qt::CorePrivate Qt6::CorePrivate)

find_package(Qt6DBus CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::DBus Qt6::DBus Qt::DBusPrivate Qt6::DBusPrivate)

find_package(Qt6DeviceDiscoverySupportPrivate CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::DeviceDiscoverySupportPrivate Qt6::DeviceDiscoverySupportPrivate)

find_package(Qt6EntryPointPrivate CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::EntryPointPrivate Qt6::EntryPointPrivate Qt6::EntryPointImplementation)

find_package(Qt6FbSupportPrivate CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::FbSupportPrivate Qt6::FbSupportPrivate)

find_package(Qt6Gui CONFIG REQUIRED)
# Note: 8 target(s) were omitted.
target_link_libraries(main PRIVATE Qt::Gui Qt6::Gui Qt::GuiPrivate Qt6::GuiPrivate)

find_package(Qt6Network CONFIG REQUIRED)
# Note: 4 target(s) were omitted.
target_link_libraries(main PRIVATE Qt::Network Qt6::Network Qt6::QNLMNIPlugin Qt::NetworkPrivate)

find_package(Qt6OpenGL CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::OpenGL Qt6::OpenGL Qt::OpenGLPrivate Qt6::OpenGLPrivate)

find_package(Qt6OpenGLWidgets CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::OpenGLWidgets Qt6::OpenGLWidgets Qt::OpenGLWidgetsPrivate Qt6::OpenGLWidgetsPrivate)

find_package(Qt6PrintSupport CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::PrintSupport Qt6::PrintSupport Qt::PrintSupportPrivate Qt6::PrintSupportPrivate)

find_package(Qt6Sql CONFIG REQUIRED)
# Note: 2 target(s) were omitted.
target_link_libraries(main PRIVATE Qt::Sql Qt6::Sql Qt::SqlPrivate Qt6::SqlPrivate)

find_package(Qt6Test CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::Test Qt6::Test Qt::TestPrivate Qt6::TestPrivate)

find_package(Qt6Widgets CONFIG REQUIRED)
# Note: 1 target(s) were omitted.
target_link_libraries(main PRIVATE Qt::Widgets Qt6::Widgets Qt::WidgetsPrivate Qt6::WidgetsPrivate)

find_package(Qt6Xml CONFIG REQUIRED)
target_link_libraries(main PRIVATE Qt::Xml Qt6::Xml Qt::XmlPrivate Qt6::XmlPrivate)

find_package(find-modules CONFIG REQUIRED)
target_link_libraries(main PRIVATE EGL::EGL X11::XCB GLIB2::GLIB2)

find_package(kwin CONFIG REQUIRED)
target_link_libraries(main PRIVATE XKB::XKB gbm::gbm Libdrm::Libdrm Libinput::Libinput)`

We will discover whether this issue is about Windows SDK.

I guess it is because Windows SDK was the only variable, still, further investigation is always good. Thank you again.

@JackBoosY  It is an SDK issues. It has been fixed in a newer version. It is not a vcpkg issue. Case closed, nothing to see here from a vcpkg perspective. General advice: Don't use old WinSDKs since you can select the WinAPI by a macro independent of the SDK version. (https://docs.microsoft.com/en-us/cpp/porting/modifying-winver-and-win32-winnt?view=msvc-170)


instrucntion not clear

Go to the patch editor and choose whether to start with recording/tap screen.
All the codes are in script.js
I fail .i dont know where to connect

please check the new instruction

thanks a lot ,,,,,,,,,,,,

by the way ,, i hope you make another game :D ,,,


Add Topics API Preview

This would be a very small change, adding application/vnd.github.mercy-preview+json (mediaTypeTopicsPreview) to activity_star.go and would allow the client to get topics for a starred repository.

Thank you, @gkze, but could you please be more specific?
I'm looking through the GitHub Developer API docs here:
https://developer.github.com/v3/activity/starring/
and don't see any mention of this custom header.

Sure, this feature was introduced a year ago: https://developer.github.com/changes/2017-07-17-update-topics-on-repositories/ and also works on user/starred, but this seems to be unclear / undocumented.
It seems to only be documented in the "Repositories" section: https://developer.github.com/v3/repos/#list-all-topics-for-a-repository

OK, so if I understand you correctly, adding the mediaTypeTopicsPreview custom header to the following endpoints will return the topics for the starred repo:

ActivityService.ListStarred
ActivityService.IsStarred
ActivityService.Star
ActivityService.Unstar

correct?

No - this header should only be passed in ActivityService.ListStarred as the rest of the calls do not return that data.

I would like to volunteer for this issue.


Merge Issue is resolved

Related Issue

Information about the related issue

Closes: #
Describe the changes you've made

Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

@parshviSri issue no?

@parshviSri Fix merge conflicts. Also, mention issue no


[Question] Stop output

Hi, How can i tell the package to stop outputting ?

Assuming you mean the log messages?
When running, you can redirect the output to whatever file, including /dev/null
Assuming you have a bash shell.
For example, if you want to discard everything:
./guerrillad serve >> /dev/null 2>&1 &
Or you want to save it:
./guerrillad serve >> /home/mike/smtpd_log 2>&1 &
The >> mean redirect-and-append.
The 2>&1 means redirect errors and output.
The & at the end means run it in the background.
You can than open another console and tail it
tail -f /home/mike/smtpd_log

That is an obvious thing. but what if the package is a part of your software? you want to store the software log to examine it, but dont want to store the guerrilla log? isn't there an option to turn logging off?

Yes, you can also control it programmatically too. The package uses logrus which gives you a lot of control https://github.com/sirupsen/logrus
Try this

Import: log "github.com/Sirupsen/logrus"
At the start of your program: log.SetOutput(os.DevNull)

Since the argument to SetOutput is a writer, you can also feed it your own writer. See guerrilla_test.go init() func to see how it's done to a buffer.
You can also control the debug level, eg:
log.SetLevel(log.DebugLevel)
or
log.SetLevel(log.FatalLevel)
Logurus also supports hooks, but haven't tried them yet.
Anyway, to make it easier, the ability control output via config will be added in the future release. See #21

It worked. thanks
ioutil.Discard


Engine Initialisation and Lifecycle improvement

This issue will deal with a restructure of the central engine initialization and lifecycle. The goal of this restructure is:

Assist in the removal of CoreRegistry
Fix a variety of interdependency and ordering issues, such as the dependencies between serializers and systems that may not exist until later.
Centralize core lifecycle concerns such as the ModuleEnvironment, AssetManager and EntitySystem lifecycles.
Move concerns to Subsystems where appropriate.
Expand on the subsystem lifecycle events, and ensure each event has a well defined meaning and usage.

Some additional possibilities:

Handle the main menu and ingame states more generically. May move main menu into a module, and have the game start with a default module set. Starting a game is just a transition to a different module set. Loading a game is a transition to a different module set, but also providing a save state. So essentially all major state changes involve a module set change and optionally a save state.
Removal of the State classes.

Firstly the GameEngine needs some cleanup. It has a number of odd methods that it really shouldn't in it. Additionally, there is an obvious need for monitoring the status of engine from outside (currently the Terasology engine has a hardcoded linkage to the splashscreen which is questionable.
Something like this would satisfy external needs.
public interface GameEngine extends AutoCloseable {
    /**
     * Registers a Subsystem
     * @throws IllegalStateException if the engine is running or disposed
     */
    void register(EngineSubsystem subsystem);

    /**
     * The current status of the engine (e.g. loading config, updating asset system, etc)
     */
    EngineStatus getStatus();

    /**
     * Subscribe for notification of engine status changes.
     */
    void register(EngineStatusSubscriber subscriber);

    /**
     * Runs the engine, which will block the thread.
     * Invalid for a disposed engine
     */
    void run(Set<Name> initialModules);

    /**
     * Request the engine to stop running
     */
    void shutdown();

    /**
     * Cleans up the engine. Can only be called after shutdown.
     * This method should not throw exceptions.
     */
    void close();

    /**
     * @return Whether the engine is running
     */
    boolean isRunning();

    /**
     * @return Whether the engine is shutting down
     */
    boolean isShuttingDown();

    /**
     * @return Whether the engine has been disposed
     */
    boolean isDisposed();
}

Internally I think the only methods needed are to trigger environment switch, and to access the current context:
    Context getCurrentContext();

    void switchEnvironment(Set<Name> modules);

Loading games, hosting/joining games, starting solo games and loading the menu would all sit above this.

Some number issues, for easy replying:

So, current GameStates would become "environment switches" and it would then be up to the modules to organize their own startup/shutdown sequence and deal with their own dependencies, right?

"Statuses" are instead more fine-grained, engine-specific operations, i.e. what is currently done during the initialization phase and perhaps other things happening on disposal? In this context I'm wondering why bother exposing status-related functionality at all? Is it because some modules would now be loaded much earlier, i.e. modules providing serializers?

Also, I kind of hate that the AutoCloseable interface imposes a close() method when what we'd really want to have is a dispose(). Should we perhaps add that too then? It would follow the pattern used elsewhere in Terasology's codebase, it would have the javadoc proposed for close() but would perhaps be allowed to throw exceptions. We could then change the close() method javadoc to "Required by the AutoCloseable interface, will probably just call dispose() but must not throw exceptions."

I like the idea of registering subsystems instead of statically providing them on construction by the way. This theoretically allows for registration anytime at runtime though. Is that intended?

Finally: why are there subsystems at all? Could they be made into "simple" modules?

No. It would be up to engine subsystems to deal with what environment switches mean to them. Modules would be unaffected - they should all be working through component systems / asset types / other objects with their own defined lifecycles. My thought is that some things which are currently hardcoded into the engine (main menu mainly) could be handled through these mechanisms instead.

I envision Status as basically being an enum-like thing (with some space for extension). They exist to provide external code a sense of what the engine is doing. They don't actually do anything though.

close() is allowed to throw exceptions. I think that quibbling over the exact name of the method is not worth it. close() in Java is used for disposal, so it shouldn't be confusing or anything.

As the javadoc says, registering a subsystem when the engine is running will provoke an IllegalStateException. I'm not sure whether I'll actually have this method though - setting them in the constructor or through a builder is probably cleaner.

No. Subsystems are still a core part of the engine itself - they slot into its main lifecycle and provide a wide range of functionality with high security requirements. They exist for the lifetime of the engine, and have state across any given module environment. Modules are more transient - it is not appropriate for modules to provide this sort of functionality. Not to mention module support itself either comes from the engine or a subsystem in the first place.

Ok, I don't know enough yet about module lifecycle and the lifecycle of the systems/assets/object provided by module so I'll just leave it at that.

From the perspective of the code interacting with the proposed GameEngine interface and disregarding game states as they do not seem to be part of this, the engine has effectively two sets of statuses/states: one that can be obtained via getStatus()/EngineStatusSubscriber, the other via the is*() methods. Never mind the latter are not explicitly labelled as returning a status: if the engine isRunning, from the perspective of a piece of external code that's -also- the state/status of the engine, alongside the result of getStatus(). I find this potentially confusing and I wonder if it can be clarified or handled differently. I.e. can we think to the status that can be ascertained through the isRunning/isShuttingDown/isDisposed methods as a more coarse-grained status of the engine compared to the fine-grained status returned by getStatus()?

You mention close() is allowed to throw exceptions while the javadoc says it should not. I can see how the term "should" does not imply it cannot, but it's a bit of a fine point that could be made more explicit. Nevertheless, I'll drop the issue.

I had missed the exception thrown if registering a subsystem while the engine is running/disposed. Which seems to leave the door open to registering a subsystem while the engine is shutting down? And also this seems to hint to a fourth "uninitialized" state, as in the current implementation. Disregarding the discussion in point 2, wouldn't this warrant an isUninitialized() method? Or do you want the external code registering subsystems realize that's not the right time simply by handling the exception? Given what you say though, it does look like there is no need for a registerSubsystem() method and working through the constructor as is currently done is probably good enough.

Thank you for clarifying this.

I.e. can we think to the status that can be ascertained through the isRunning/isShuttingDown/isDisposed methods as a more coarse-grained status of the engine compared to the fine-grained status returned by getStatus()?

Yes. I could rename the method to getRunningStatus() perhaps.

I don't know where you looking, but the definition of close() in the Autoclosable interface explicitly throws Exception.

Shutting down is still running. Running only is false after shutting down is complete.

I think I will go with using a builder for setting subsystems and remove the register method.

Linking this to #1797 although I assume that is just one step and doesn't complete this yet.

Very not ready, but you can get a feel for the direction I'm heading from 90c7579d71359ff596293b20cffe1ec0cf745601
A lot of the initialization logic currently in states and their load processes would be moved into subsystems, and occur in response to well defined engine lifecycle events. This would result in logic for managing a particular aspect being self-contained, and ensure they are treated the same way in both main menu and in game.
This is already solving some initialisation ordering issues which @flo highlighted (e.g. populating component libraries between environment creation and asset environment switching).
The down side is that it can make the order of initialisation between subsystems unclear, especially where two subsystems need to react to the same lifecycle event and one needs to be after the other. For the moment I am seeing whether this will be an issue, and if it is considering whether additional lifecycle events would make sense and solve things. Otherwise I may add some method annotations for providing execution order, though that is a last resort.
Currently there are lifecycle events for main engine initialisation, environment switching, main loop update, and shutdown. Entity system initialisation/loading and network related lifecycle events are also probable.
For environment switching there is a changeModuleEnvironment() method in engine, but I'm considering having something more along the lines of createGame()/loadGame()/joinGame(), with networking and saved game options built in. The main menu would also be a game, just with a specific module.
The branch is currently somewhat non-functional. I'm pretty sure you cannot actually start a game, although the menu works at least. Much work to go.

Very cool :-) Thanks for the update.
Ping: @MarcinSc @Josharias @emanuele3d @msteiger @skaldarnar

Curiously, @immortius, is this paired with #1234 (entity system overhaul) or its own thing? Wondering if this should be a v2.0.0 item.


@^(UPDATED)^^⇢Nintendo eShop Gift Card Codes 2024 🥀🥀 Free Nintendo eShop Code Generator

Welcome to the Free Nintendo eShop Gift Card Codes event! Looking for a way to get your hands on some free Nintendo eShop Gift Card Codes? Look no further! Our Generator 2024 is here to help you out. Join us online to learn how to get your hands on these free codes. Don't miss out on this opportunity to enhance your gaming experience without breaking the bank.  Well, it's as easy as pie! Simply use the Nintendo Gift Card Generator to generate your very own unique gift card code. It's quick, it's easy, and best of all, it's completely free. No more emptying your wallet to fuel your gaming passion - with the Nintendo Gift Card Generator, the power is in your hands.
And the fun doesn't stop there. Once you've got your hands on your free Nintendo gift card, it's time to redeem it and dive into a world of gaming bliss. Whether you're itching to explore the latest Nintendo releases or you're on the hunt for some epic in-game rewards, your free gift card is your golden ticket to unlocking it all.
So, what are you waiting for? Get ready to level up your gaming experience with the Nintendo Gift Card Generator. It's time to power up, gear up, and embark on an epic gaming journey like never before. Don't miss out on this golden opportunity to score some awesome freebies and elevate your gaming adventures to new heights.
With the Nintendo Gift Card Generator, the world of Nintendo gaming is yours to explore.


[R-package] [LightGBM] [Warning] No further splits with positive gain, best gain: -inf

R latest, tidymodels latest, lightGbm latest
This message...
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
... is occupying the whole screen, while training, even with 'verbose=-1'.
it drives me nuts
model_spec <- boost_tree( 	trees      = as.numeric(resObj$bestParams$trees) 	,tree_depth = as.numeric(resObj$bestParams$tree_depth) 	,min_n      = as.numeric(resObj$bestParams$min_n) ) %>% set_mode("regression") %>%   set_engine("lightgbm", objective = "rmse",verbose=-1)
Any ideas?

I actually want to know how to suppress this message in R.

Here's a minimal, reproducible example showing one way to train a LightGBM model in R and suppress all warning-level log messages.
library(lightgbm)

dtrain <- lightgbm::lgb.Dataset(
    data = matrix(rnorm(1000L), ncol = 4L)
    , label = rnorm(250L)
)

# produces logs, including "No further splits with positive gain"
bst <- lightgbm::lgb.train(
    data = dtrain
    , nrounds = 5L
    , obj = "regression_l2"
)

# no logs
bst <- lightgbm::lgb.train(
    data = dtrain
    , nrounds = 5L
    , obj = "regression_l2"
    , verbose = -1L
)

I installed {lightgbm} v3.3.2 from CRAN with install.packages().
output of sessionInfo() (click me)
R version 4.1.0 (2021-05-18)
Platform: x86_64-apple-darwin17.0 (64-bit)
Running under: macOS Mojave 10.14.6

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/4.1/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods
[7] base

other attached packages:
[1] lightgbm_3.3.2 R6_2.5.1

loaded via a namespace (and not attached):
[1] compiler_4.1.0    Matrix_1.3-4      tools_4.1.0
[4] grid_4.1.0        data.table_1.14.0 jsonlite_1.7.2
[7] lattice_0.20-44

We'd be happy to help you, but cannot unless you provide this type of information.

I see- that's enough info. Then it's highly likely to be a third party issue (tidymodels or treesnip).
thx.


Error in <TabsWithStdin> using the example code

Hi 👋
When I try to use the example code in the Readme or the code in the demo directory, I get an error shown in the screenshot below:

I tried to wrap the whole code in StdinContext.Consumer imported from ink but didn't help.

OS
Mac OS High Sierra

Command Line interface
Iterm2 Build 3.2.8

Sample code I'm using and the error is thrown in:
App.js
const React = require("react");
const {
	Static,
	Text,
	render,
	Color,
	Box,
	StdinContext
} = require("ink");
const { Tab, Tabs } = require("ink-tab");
const { Fragment, useState } = React;

const MainContent = ({ activeTab }) => (
	<Box>
		{activeTab === "foo" && 'Selected tab is "foo"'}
		{activeTab === "bar" && 'Selected tab is "bar"'}
		{activeTab === "baz" && 'Selected tab is "baz"'}
	</Box>
);

class TabExample extends React.Component {
	constructor(props) {
		super(props);

		this.state = {
			activeTab: null
		};

		this.handleTabChange = this.handleTabChange.bind(this);
	}

	handleTabChange(name, child) {
		this.setState({
			activeTab: name
		});
	}

	render() {
		const { direction } = this.props;

		return (
			<Box
				flexDirection={
					direction === "column" ? "row-reverse" : "column"
				}
				justifyContent={
					direction === "column" ? "flex-end" : "flex-start"
				}>
				<MainContent activeTab={this.state.activeTab} />

				{direction === "column" && <Box> </Box>}

				<Tabs
					onChange={this.handleTabChange}
					flexDirection={direction}
					width={direction === "column" ? 20 : "100%"}
					keyMap={{ useTab: false }}>
					<Tab name="foo">Foo</Tab>
					<Tab name="bar">Bar</Tab>
					<Tab name="baz">Baz</Tab>
				</Tabs>
			</Box>
		);
	}
}

render(
	<TabExample
		direction={
			process.argv.includes("--column") ? "column" : "row"
		}
	/>,
	{ debug: false }
);

index.js
const importJsx = require("import-jsx");
importJsx("./App");

Hi,
just to be sure : do you use ink >= 2.1.0 ?
If I remember correctly, setRawMode is not in 2.0.0.
If you are unsure, can you copy the content of your package.json and package-lock.json or yarn.lock files ?
Thank you

Hey, @jdeniau Thanks for the fast reply.
I'm using ink: ^2.1.1 in this setup.
This is the package.json I have right now:
{
	"name": "CLI",
	"version": "1.0.0",
	"main": "index.js",
	"license": "MIT",
	"dependencies": {
		"@babel/cli": "^7.4.3",
		"@babel/core": "^7.4.3",
		"import-jsx": "^2.0.0",
		"ink": "^2.1.1",
		"ink-select-input": "^3.1.0",
		"ink-spinner": "^3.0.0",
		"ink-tab": "^2.1.1",
		"react": "^16.8.6"
	},
	"devDependencies": {
		"@babel/preset-env": "^7.4.3",
		"@babel/preset-es2015": "^7.0.0-beta.53",
		"@babel/preset-react": "^7.0.0",
		"nodemon": "^1.18.11"
	},
	"scripts": {
		"compile": "babel index.js --out-file index-compiled.js",
		"serve": "npm run compile && nodemon index-compiled.js"
	}
}

I do not know why, but the problem is with nodemon.
According to https://github.com/remy/nodemon/issues/1367 you need to start with --no-stdin option:
nodemon --no-stdin index-compiled.js

@jdeniau Awesome. Thanks for the help. Tried it and it worked.

You are welcome 😉

I added a propTypes for setRawMode. This way, you should see a warning about it (even it is just a nice display against an error 😁 )


Plugin API

Before the first release, there should be a clearly defined plugin API and easy way to create new behaviors as node packages. The plugin API should have convenience methods for performing common actions on the item that triggered the event (e.g. comment on a PR, update statuses, apply labels, etc.).
The naive example in the README right now is something like this:
robot.on('pull', function(event) {
  robot.comment("Thanks for the pull request! We'll review it within 72 hours!");
});

The first behavior is looking something like this:
module.exports = {
  webhook: 'issues', // name of webhook event
  action: function(event, github) {
    // event - the webhook event
    // github - API client: https://mikedeboer.github.io/node-github/
  }
};

I'm planning to implement a few other behaviors before figuring out what the API should look like.

Here's the new plugin API:
module.exports = robot => {
  robot.on('push', event => console.log(event));
};

Which you run with
$ probot ./index.js

🎩 @groundwater


Isopleths-Operator Defined-Freeform symbol issue

This is an issue in:

[ ]  2525C
[ ]  2525B.
[ ]  APP-6-D.

It is not an issue in APP-6-B.
It is fixed in 2525D (https://github.com/Esri/military-symbology/issues/270)

Fixed
SIDC
Name
Feature Type

- [ ]
45180700
Isopleths-Operator Defined-Freeform
Atmospheric Line

Draws dashed, solid option not available
Should be:

This issue was moved to ArcGIS/military-symbology-styles#80


[Bug]: Regression in V28<EMAIL_ADDRESS>types toHaveBeenCalledWith forcing at least one argument.

Version
28.0.3
Steps to reproduce

Write a sample typescript test:

import { beforeAll, describe, expect, it, jest } from '@jest/globals';
import { Mocked } from 'jest-mock';

describe('sample', () => {
  interface SampleClass {
    foo(...args: unknown[]): void;
  }

  let sampleMocked: Mocked<SampleClass>;

  beforeAll(() => {
    sampleMocked = {
      foo: jest.fn(),
    };
  });

  it('sample', () => {
    expect(sampleMocked).toHaveBeenCalledWith();
  });
});

Compile the code

Expected behavior
Typescript should compile with no errors
Actual behavior
The following error is thrown:
Expected at least 1 arguments, but got 0.

Additional context
No response
Environment
System:
    OS: Linux 5.13 Ubuntu 20.04.4 LTS (Focal Fossa)
    CPU: (16) x64 AMD Ryzen 9 4900HS with Radeon Graphics
  Binaries:
    Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
    npm: 8.8.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
  npmPackages:
    jest: 28 => 28.0.3

Trying to understand. Do you mean: types should allow a case there a mock was called with no arguments? For example:
import { describe, expect, test, jest } from "@jest/globals";

const mockFn = jest.fn();

test("toHaveBeenCalled", () => {
  mockFn();
  mockFn(123);

  expect(mockFn).toHaveBeenCalledTimes(2);

  expect(mockFn).toHaveBeenCalledWith();
  expect(mockFn).toHaveBeenCalledWith(123);
});

Exactly, that was allowed in jest@27.
Sorry if the tittle is not the best one, feel free to update it :)

Makes sense to me for args to be optional 👍


CASSANDRA-16663 - Request-Based Native Transport Rate-Limiting

some rough initial work on the V4 side of the request limiting apparatus

https://app.circleci.com/pipelines/github/maedhroz/cassandra/252/workflows/66e43c3a-29dc-4935-b769-3c1597b70889


setup.* cleanup, compatibility fixes

Significant changes

Drop support for Python 3.4 -- it's no longer maintained and we weren't testing it anyway
Add official support for Python 3.7 and 3.8
Block installation on unsupported Python versions using python_requires directive

Testing

Make tox environments consistent with what we claim to support
Use tox in Travis
Add .coverage artifact to gitignore

Compatibility

Don't use old-style classes on Python 2
Python 2 doesn't support encoding kwarg in open()
Python 2 doesn't have json.JSONDecodeError; use its superclass instead

General cleanup

Remove most I/O in setup.py by moving description to setup.cfg
Consolidate installation requirements
Run tests through black like we do with the rest of the codebase

This is great. Thank you and sorry for the delay in merging it

Thank you for your help @dargueta
It's been released on https://github.com/gtalarico/airtable-python-wrapper/releases/tag/0.13.0


Upg: heartbeat during github repo tarball extraction. Filter early

Description
We have a workflow stuck because github repo tarball extraction takes too long than the heartbeat timeout.
This PR should fix it by heartbeating every 100 items during extraction and extraction time should be shorter because filtering is done earlier.
Risk
None
Deploy Plan
Deploy connectors

Did you test locally?

I copied my working code directly on a connector-worker pod and executed there.
The only part I didn't test is the "heartbeat" as I was out of activity context.


Code Logic Completion

There are a few codes that need to be completed to get the correct output.
Please complete the code to fix the issue.

Completed Successfully!


lapack: include lapacke_utils.h in alternatives/lapack

Motivation for this change
Although not present in our (old) version of libsurvive, there's an
upcoming issue
(https://gitlab.freedesktop.org/monado/monado/-/issues/130) which can be
worked around by using mkl as a lapack implementation.
This isn't a fix, but it makes things a little easier for those
overriding to a newer version, and it's the correct thing to do anyway.
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

I updated the PR title to better reflect the large-impact change here, rather than the motivating change.

I am afraid this might have to go to staging first.

@prusnak I opened https://github.com/NixOS/nixpkgs/pull/146205 targeting staging.

Closing this PR in favor of https://github.com/NixOS/nixpkgs/pull/146205


StackNavigator with List of items extemly slow on Android

Problem

On my screen I have 2 companies, for example Lidl and Tesco
I click on Lidl and detail shows up, where I load some products of company, but Lidl hasnt got any products. (Everything is working fine)
I go back
I click on Tesco and load 25 different products of company  (Everything is working fine)
I go back and click again on Lidl, but when i tap on button which include this.props.navigate() function it takes more than 3s to go on detail

My code

Simple get data from redux and show record in  from nativebase compoment
I'm using map function on this

What i try

use another component like  instead of using external library
disable all logs
disable nested navigation (I have main drawer and inside it I have StackNavigator)

Posible solution

Use modals (bad solution)
Use another navigation JS library
Use native navigation
Best solution is fix this problem :)

My package.json
https://pastebin.com/fpD1zjX4

@jamie78 from what I can see, it may not be react-navigation's problem. If you can provide me with a simple example, I can help you to investigate.

Hey @khanghoang, I already figured out my issue. You are totally right. I tried to use different navigation library, but my issue occurs again. So it was a render problem. I used nativebase library to make easier with design, but when you render for example 150 items in cards it is a little bit problem for render() method. So I updated my code with simple check statement, if everythink is already loaded then call render() function. So my issue is clear. Hope this will help others. :)

Hi @jamie78,
I am facing the same problem. can you please post the Gist, what exactly you did.
Thanks in advanced :)


Alex/updatetoreleaseextensions

This PR adds the vbumped extensions from https://github.com/microsoft/azuredatastudio/pull/16991 to the insider gallery.
Product Build containing the extensions:
https://mssqltools.visualstudio.com/CrossPlatBuildScripts/_build/results?buildId=118768
These must be added to the sqlops extensions gallery

@smartguest It looks like most of these extensions had already been vBumped - so this update is skipping the version numbers farther ahead than would be expected.
e.g. profiler 0.11.0 -> 0.12.1, the version probably should have been either 0.11.1 or 0.12.0.
It's not a huge deal so it's fine to leave it like this for now, but please keep that in mind in the future when doing extension vBumps to always check the existing version of the extensions and only bump them if necessary.


Allow specifying values + priorities in one step

(Based on issue 10 - 3rd comment and issue 10 - 4th comment by phillip-haydon)
Idea: Something like this:
var options = Out.Of().WeightedValues(lootable.Select(x => new WeightedValue(x, x.Weight));

var drop = options.PickOne();

Advantage:

No mistakes with order (order of values and percentages/weights) need to match when using Out.Of().Values(...).AndWeights(...) syntax
Easy way if the values and percentages/weights are already combined in an object.

Possibility 1:
var elements = new List<MyValuePriorityPair>();
Out.Of().PrioritizedElements(elements)
    .WithValueSelector(x => x.Value)
    .AndPercentageSelector(x => x.Percentage)
    .PickOne();
Out.Of().PrioritizedElements(elements)
    .WithValueSelector(x => x.Value)
    .AndWeightSelector(x => x.Weight)
    .PickOne();

(+) Good readability
(+) More fluency (it's a fluency project)
(+) Performance: optimal
(+-) Difficulty: intermediate

Possibility 2:
var elements2 = new List<(string Value, int Priority)>();
Out.Of().WeightedValues(elements2)
    .PickOne();
Out.Of().PercentageValues(elements2)
    .PickOne();

(+) Good readability
(-) Makes the API harder to use (Out.Of().<Value,Values,WeightedValues,PercentageValues>)
(+) Performance: optimal
(+) Difficulty: very easy

Possibility 3:
var elements3 = new List<X>();
Out.Of().WeightedValues(elements3.Select(x => new WeightedValue(x.Value, x.Weight)))
    .PickOne();
Out.Of().PercentageValues(elements3.Select(x => new PercentageValue(x.Value, x.Percentage)))
    .PickOne();

(-) Not good readability
(-) Makes the API harder to use (Out.Of().<Value,Values,WeightedValues,PercentageValues>)
(+-) Performance: Maybe redundant instantiations
(+) WeightedValue/PercentageValue could possibly be used internally, too (see ValuePriorityPair
(-) Difficulty: more difficult (one had to know that the types WeightedValue/PercentageValue exist)

I think I prefer approach 1.

What's possible at the moment:
var vals = new List<IWeightedValues>();
var x = Out.Of().Values(vals).WithWeights(vals.Select(x => x.Weight)).PickOne();

x is a whole IWeightedValue.
With the new feature of this issue, this could be possible:
var elements = new List<IWeightedValue>();
var x = Out.Of().PrioritizedElements(elements)
    .WithValueSelector(x => x)
    .AndWeightSelector(x => x.Weight)
    .PickOne();

Maybe, the WithValueSelector should be optional, so that it's not required to write x => x if you want to return the whole element:
var elements = new List<IWeightedValue>();
var x = Out.Of().PrioritizedElements(elements)
    .WeightSelector(x => x.Weight)
    .PickOne();

x is a whole IWeightedValue again.

New feature is available. It works with version 3.3.0.
Example:
class Item {
    public int Rarity { get; set; }
    public string Name { get; set; }
}

var items = new Item[]
{
    new Item { Name = "Stone", Rarity = 5 }, // common
    new Item { Name = "Silver helmet", Rarity = 2 }, // uncommon
    new Item { Name = "Gold sword", Rarity = 1 }, // rare
};

var itemName = Out.Of()
                  .PrioritizedElements(items)
                  .WithValueSelector(x => x.Name)
                  .AndWeightSelector(x => x.Rarity)
                  .PickOne();

// itemName is "Stone" in 5/8 of the cases, "Silver helmet" in 2/8 of the cases and "Gold sword" in 1/8 of the cases.
// If no value selector is specified, the whole item object will be returned instead of only its name.


deno: reenable Deno CI tests

Thanks to the work by @mash-graz the latest Deno release now supports far more of the test suite.
This reenables the Deno CI test suite and also enables more of the tests - we are now passing 64/106 tests in Deno.

I was just waiting for this new release before I send you some jco resp. preview2-shim patches to eliminate a few more compatibility issues,
Many of the reported errors are not very important in practice. Most of them are just reporting, that deno still reports port number 0 instead of the automatically chosen one and also for wildcard interface <IP_ADDRESS> a similar error will be displayed. But both cases are nearly negligible.
But other compatibility issues have to be fixed. For example deno doesn't support setting the TTL value for UDP packets. It will throw an NotImplemented Error, but jco doesn't catch this case, and every UDP connect will fail...
Some more complex workarounds, which I had to figure out, are not necessary anymore, because some deno maintainers spend a lot of time improving the node:workers_threads implementation. Most of the grave defects of this node compatibility module are now solved in deno.
There is still one show stopper remaining: The file access in different threads resp. worker contexts. I already reported it here as well. Although we could work around this issue by utilizing the synckit worker indirection also for all file system related operations, but this would have negative consequences for the performance. @bartlomieju thinks, that this incompatibility can be also solved on denos side after another complex change. (see: https://github.com/denoland/deno/issues/23012)

Great to hear there may still be some low-hanging fruit. I've also disabled the Windows Deno tests for now as they had a few more errors as well.
It was fantastic to see the worker improvements - if anything this Jco compat does ensure Deno has very strong Node.js compat, so glad we can put pressure on that through these tests.
I'm open to your suggestions for any workarounds needed in Jco. Moving all filesystem descriptors to the worker could well be an option - alternatively we could even make that an implementation detail that is Deno-specific if it helps and this can't be fixed upstream.

Hey @guybedford @mash-graz.
Thanks for the ping and thanks for surfacing this project to us. It's great seeing that we have 60% compatibility, but we'd like to get this number closer to 90% and 100% preferably. I'll schedule some time to figure out which tests are failing and we will keep our eye on https://github.com/denoland/deno/issues/23012 as it seems like a must to get to that number.

@bartlomieju it was really great to see this progress! So glad you are interested in seeing these worked through. I can also aim to do some more issue isolation in due course as these cases progress.


dubbo 2.7.1-SNAPSHOT NoClassDefFoundError when use springboot

[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.1-SNAPSHOT
Operating System version: MacOS
Java version: 1.8

Steps to reproduce this issue

add latest dubbo version and spring-boot-starter-dubbo
try to start springboot
java.lang.NoClassDefFoundError: org/apache/curator/retry/ExponentialBackoffRetry

Actual Result
java.lang.NoClassDefFoundError: org/apache/curator/retry/ExponentialBackoffRetry
	at org.apache.dubbo.configcenter.support.zookeeper.ZookeeperDynamicConfiguration.<init>(ZookeeperDynamicConfiguration.java:64) ~[dubbo-2.7.1-SNAPSHOT.jar:2.7.1-SNAPSHOT]
	at org.apache.dubbo.configcenter.support.zookeeper.ZookeeperDynamicConfigurationFactory.createDynamicConfiguration(ZookeeperDynamicConfigurationFactory.java:29) ~[dubbo-2.7.1-SNAPSHOT.jar:2.7.1-SNAPSHOT]
	at org.apache.dubbo.configcenter.AbstractDynamicConfigurationFactory.getDynamicConfiguration(AbstractDynamicConfigurationFactory.java:33) ~[dubbo-2.7.1-SNAPSHOT.jar:2.7.1-SNAPSHOT]...

Have to manually add curator-framework and curator-recipes and then it starts successfully. But it seems not convenient. Any problem with the pom.xml setting?
必须手动增加curator-framework和curator-recipes两个依赖，否则无法通过springboot启动，请问下是配置有问题吗
The dependency:
    <dependency>
      <groupId>org.springframework.boot</groupId>
      <artifactId>spring-boot-starter-web</artifactId>
    </dependency>

    <dependency>
      <groupId>org.apache.dubbo</groupId>
      <artifactId>dubbo</artifactId>
      <version>2.7.1-SNAPSHOT</version>
    </dependency>

    <dependency>
      <groupId>io.dubbo.springboot</groupId>
      <artifactId>spring-boot-starter-dubbo</artifactId>
      <version>1.0.0</version>
      <exclusions>
        <exclusion>
          <groupId>com.alibaba</groupId>
          <artifactId>dubbo</artifactId>
        </exclusion>
      </exclusions>
    </dependency>

    <dependency>
      <groupId>org.apache.curator</groupId>
      <artifactId>curator-framework</artifactId>
      <version>4.0.1</version>
      <exclusions>
        <exclusion>
          <groupId>org.apache.zookeeper</groupId>
          <artifactId>zookeeper</artifactId>
        </exclusion>
      </exclusions>
    </dependency>
    <dependency>
      <groupId>org.apache.curator</groupId>
      <artifactId>curator-recipes</artifactId>
      <version>4.0.1</version>
      <exclusions>
        <exclusion>
          <groupId>org.apache.zookeeper</groupId>
          <artifactId>zookeeper</artifactId>
        </exclusion>
      </exclusions>
    </dependency>

可以看一下示例：https://github.com/apache/incubator-dubbo-spring-boot-project/tree/master/dubbo-spring-boot-samples
应该仅仅添加：

  <groupId>org.apache.curator</groupId>
  <artifactId>curator-framework</artifactId>

就可以

I think it's a side effect of defaulting the Registry (only if it's zookeeper) as the Config Center.
I guess you are using zookeeper registry and zkclient, right?

I think it's a side effect of defaulting the Registry (only if it's zookeeper) as the Config Center.
I guess you are using zookeeper registry and zkclient, right?

yes, I use zk as my registry

用curator-client 就可以解决

we need to refresh README when we release 2.7.1


Knowledge - More connection error handling (retries) around the OpenAI embeddings API

knowledge - File ingestion reports failure with - couldn't create embedding of document: couldn't read response body: stream error: stream ID 307; INTERNAL_ERROR
Steps to reproduce the problem

Create an agent and add knowledge files from https://docs.gptscript.ai
Wait for ingestion to be complete

2 of the files failed to ingest with following errors:
{"time":"2024-10-08T10:40:47.695176-07:00","level":"ERROR","msg":"Failed to add documents to collection (generate embeddings)","flow":"ingestion","rootPath":"/Users/sangeethahariharan/Library/Application Support/otto/workspaces/7943aa72-748c-47ad-bea4-94e235770cfc","filepath":"/Users/sangeethahariharan/Library/Application Support/otto/workspaces/7943aa72-748c-47ad-bea4-94e235770cfc/rks1zf4tv/docs.gptscript.ai/examples/local-files.md","phase":"store","num_documents":28,"stage":"vectorstore","vectorstore":"chromem-go","status":"failed","error":"couldn't add document 'e56c9b0d-20a4-4e53-8a25-b527c7593db2': couldn't create embedding of document: couldn't read response body: stream error: stream ID 307; INTERNAL_ERROR; received from peer"}\n{"time":"2024-10-08T10:40:47.695266-07:00","level":"ERROR","msg":"Failed to add documents","error":"couldn't add document 'e56c9b0d-20a4-4e53-8a25-b527c7593db2': couldn't create embedding of document: couldn't read response body: stream error: stream ID 307; INTERNAL_ERROR; received from peer"}\n{"time":"2024-10-08T10:40:47.695275-07:00","level":"ERROR","msg":"Failed to add documents to collection (generate embeddings)","flow":"ingestion","rootPath":"/Users/sangeethahariharan/Library/Application Support/otto/workspaces/7943aa72-748c-47ad-bea4-94e235770cfc","filepath":"/Users/sangeethahariharan/Library/Application Support/otto/workspaces/7943aa72-748c-47ad-bea4-94e235770cfc/rks1zf4tv/docs.gptscript.ai/command-line-reference/gptscript_credential_show.md","phase":"store","num_documents":6,"stage":"vectorstore","vectorstore":"chromem-go","status":"failed","error":"couldn't add document 'ff8acb2a-1503-4e29-8aed-494972d53c59': couldn't create embedding of document: couldn't read response body: stream error: stream ID 201; INTERNAL_ERROR; received from peer"}\n{"time":"2024-10-08T10:40:47.69531-07:00","level":"ERROR","msg":"Failed to add documents","error":"couldn't add document 'ff8acb2a-1503-4e29-8aed-494972d53c59': couldn't create embedding of document: couldn't read response body: stream error: stream ID 201; INTERNAL_ERROR; received from peer"}\n{"time":"2024-10-08T10:40:47.695365-07:00","level":"INFO","msg":"failed to add documents: couldn't add document 'e56c9b0d-20a4-4e53-8a25-b527c7593db2': couldn't create embedding of document: couldn't read response body: stream error: stream ID 307; INTERNAL_ERROR; received from peer"}\n"

Full stack trace logs -
calls (19).json

This issue is being tracked in https://github.com/otto8-ai/otto8/issues/454#issuecomment-2469096851.
Closing this issue.


[33] Implements solution 1. for issue 33

fixes #33

LGTM. Recommend merge.
Maybe windows users (as in #33) who run sls from inside powershell might get a (for them) superfluous shell process wrapped around the java process, causing some tiny additional memory overhead. But handling that would add complexity. This PR strikes a good balance IMHO.


its not working

dsio --es-uri http://localhost:9200/ --kibana-uri http://localhost:5601/app/kibana data/cardata_sample.csv
Loading the data...
Done.
data found from 2013-05-24 23:51:37 to 2013-05-25 00:14:13
Converting to milliseconds ...
Done
Adding time offset of<PHONE_NUMBER>.54 seconds
Setting speed to 1x
Done
Traceback (most recent call last):
File "C:\Program Files\Python36\Scripts\dsio-script.py", line 11, in
load_entry_point('dsio', 'console_scripts', 'dsio')()
File "c:\windows\system32\src\dsio\dsio\main.py", line 155, in main
cols=int(args.cols)
File "c:\windows\system32\src\dsio\dsio\main.py", line 70, in restream_dataframe
port=bokeh_port, update_queue=update_queue
File "c:\windows\system32\src\dsio\dsio\dashboard\bokeh.py", line 74, in generate_dashboard
if io_loop._running: # Assume we're in a Jupyter notebook
AttributeError: 'AsyncIOMainLoop' object has no attribute '_running'

downgrade tornado to 4.5.3 should work
pip3 install tornado==4.5.3

downgraded to tornado 4.5.3 and still getting this same error with   datastream=0.5.19=pypi_0

downgrading did not fix the problem and has caused other conflicts.

Making small change to source codes, it works! Changing function io_loop._running to io_loop._running() in bokeh.py, as shown in the screenshot below.


0.11.0 is minimum optparse version

For some reason, optparse 0.10.0 has an eitherReader which takes two arguments:
https://hackage.haskell.org/package/optparse-applicative-0.10.0/docs/Options-Applicative-Builder.html#v:eitherReader
Which causes this error compiling elm-make:
$ cabal install
Resolving dependencies...
Notice: installing into a sandbox located at
/Users/laszlopandy/dev/elm-compiler/.cabal-sandbox
Configuring elm-make-0.1.2...
Building elm-make-0.1.2...
Failed to install elm-make-0.1.2
Last 10 lines of the build log ( /Users/laszlopandy/dev/elm-compiler/.cabal-sandbox/logs/elm-make-0.1.2.log ):
        (bound at src/Arguments.hs:128:1)
    Probable cause: ‘Opt.eitherReader’ is applied to too few arguments
    In the expression: Opt.eitherReader reader
    In the expression:
      let
        reader arg
          = case fromString arg of {
              Just a -> ...
              Nothing -> ... }
      in Opt.eitherReader reader
cabal: Error: some packages failed to install:
elm-make-0.1.2 failed during the building phase. The exception was:
ExitFailure 1

Thanks, sorry for the trouble!


Add repo visuals

Adds a repo codebase visualization based on https://next.github.com/projects/repo-visualization#integrate-into-your-own-projects

Awesome, I like anything that's shiny and colorful :)

Codecov Report

Merging #497 (a5c720b) into master (0914a71) will decrease coverage by 3.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #497      +/-   ##
==========================================
- Coverage   98.06%   94.67%   -3.40%
==========================================
  Files          22       22
  Lines        2533     2515      -18
==========================================
- Hits         2484     2381     -103
- Misses         49      134      +85

Impacted Files
Coverage Δ

robocop/checkers/errors.py
80.76% <0.00%> (-16.16%)
:arrow_down:

robocop/checkers/spacing.py
88.10% <0.00%> (-10.39%)
:arrow_down:

robocop/checkers/misc.py
90.97% <0.00%> (-8.34%)
:arrow_down:

robocop/checkers/comments.py
94.77% <0.00%> (-5.23%)
:arrow_down:

robocop/checkers/naming.py
96.23% <0.00%> (-3.35%)
:arrow_down:

robocop/utils/misc.py
97.52% <0.00%> (-1.02%)
:arrow_down:

robocop/checkers/lengths.py
99.41% <0.00%> (-0.59%)
:arrow_down:

robocop/config.py
94.31% <0.00%> (-0.07%)
:arrow_down:

robocop/checkers/duplications.py
99.27% <0.00%> (-0.02%)
:arrow_down:

robocop/rules.py
100.00% <0.00%> (ø)

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 0914a71...a5c720b. Read the comment docs.


Box uploads are not capped

I should be limited to 5GB on file upload from box.  It seems like the new code no longer does that limit.
This could be a configuration difference, but we should verify that is the case as this could cause some strange behavior if users start uploading very large files.
On QA I can upload a 7GB file:

In Production I can not upload a 7GB file:

This was a configuration issue.


Font color

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

Project coverage is 100.00%. Comparing base (22dd541) to head (d57fa73).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #467   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           30        30
  Lines          298       298
  Branches        72        70    -2
=========================================
  Hits           298       298

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


Pin this repo in the GH Org

Is your feature request related to a problem? Please describe.
The MicrosoftDocs organization has 800 public repos now so finding this repo can be difficult if trying to find it directly on GitHub.
Describe the solution you'd like
Adding feedback is an important aspect so this repo should be one of the pinned repos when viewing https://github.com/MicrosoftDocs

@nschonni Thanks for you great suggestion. I will create a internal ticket to organize pinned repos

#log-suggestion

🚀 ATTENTION: Internal request logged.

@nschonni This issue is resolved. Please double check.


intel_edison_fab_c.c: fixed the pins not being freed

The pins for the Edison Arduino expansion where not being freed in the case where a tristate was not successful
Signed-off-by: Houman Brinjcargorabi<EMAIL_ADDRESS>

Merged, thanks


Yii2. Saving $form (widget/ActiveForm) in Session

I am trying to save $form (After doing $form = ActiveForm::begin(())
$fofArgs = [
           'form' => $form,
        ];
 Yii::$app->session->set('fofArgs', $fofArgs);

Then I go to a different controller/action:
var_dump(Yii::$app->session->get('fofArgs'));
But it prints null. I tried serializing before adding it to the session but it returns an error, "Closures can not be serialized", so maybe the null result has to do with closures too?
Additional info

Q
A

Yii version
2

PHP version
5.6

Operating system
OSX

Can you serialize the config array you create the form with instead?

Why you trying save ActiveForm into session? "Closures can not be serialized" - expected result.

Hi Guys, thanks for answering. Yes I have reviewed and Closures can not be serialized, and that is how PHP works, althought you can use SuperClosure but that works only if you try to serialize directly a closure and not an object which has closures in it.
I have a widget inside a form, which through Pjax adds fields dinamically, so I am trying to pass $form to be able to create those dynamic fields from $form.

Yes, it's not possible to save closures so I'd store config representation of the form as @Alex-Code suggested.

@samdark and how you get the config representation of the form?


Fix HllSketchHolderObjectStrategy#isSafeToConvertToNullSketch.

The prior code from #15162 was reading only the low-order byte of an int representing the size of a coupon set. As a result, it would erroneously believe that a coupon set with a multiple of 256 elements was empty. At the default lgK (12) this is possible when the sketch has exactly 256 items in it. (At lgK = 12, before it reaches 512, it would be promoted from coupon set to full HLL.)

The changes generally look good to me.  We should probably go and ask the apache-datasketches folks if we can get an "isEffectivelyEmpty" method added to the Sketch so that we can remove all of this introspective logic to begin with.  That should enable a much more future-proof way of dealing with this.


Scope out action items for UVIS HSP and HDAC migration projects

An excerpt from minutes from our team meeting of 8/6/20:

Cassini migration
UVIS: MKG’s understanding is that we will generate PDS4 bundles for EUV/FUV, send them to Richard Chen for him to look at, then generate bundles for HSP (MRS: we can also do HDAC, it was working correctly and we didn’t know it, but now we understand).  We have working pipelines for HSP and HDAC?  MRS: We have a working pipeline for HSP, but the times are wrong.  MWE: We have an overall understanding of how HSP timing worked, but we disagree with how they did it.  MRS: The pipeline did not use any of the work MWE did, just the PDS3 labels which were insufficiently accurate.  MWE: Timing is inaccurate on msec level.  MRS: But way worse than that in the labels.  We could migrate all four data sets and get valid results, but HSP timing would be wrong and we don’t want to do it without fixing the timing.
MKG: For this month, what are our plans?  Other than HSP timing, we must fix HDAC documentation, fix timing documentation for HSP, labels for timing files?  MRS: We are providing timing files, have not yet put labels on them.  Just assume we’ve done nothing on HSP.  We can convert valid PDS3 products to valid PDS4 products, but we don’t want to deliver those until we’ve fixed the timing.  We can proceed with EUV, FUV, and HDAC.
MKG: Can we turn the crank and make those happen this month?  MRS: What’s the urgency?  MKG: We were supposed to have them done by the end of the FY, though probably nobody will hold our feet to the fire.  MWE’s email to Tim, we’re supposed to talk about impacts of the pandemic on our work.  MRS: Definitely slowed down migration, because moving large amounts of data around is impacted.  MKG: Providers at academic institutions are shut down from doing anything other than preparing to teach online.  That also impacts peer reviews as well as data providers.  Anyway, would be good to show progress on UVIS this month.  We can get EUV and FUV going, can say we’re ready to do HDAC and are working on the timing for HSP.

Slides that I presented to the PDS MC in July 2020.
HighlightsRMS_July20vf2f_v3_Slide10.pdf
HighlightsRMS_July20vf2f_v3_Slide11.pdf

Slides that I presented to the RMS Node Users Group in December 2020.
RMSNUG_AnnualMeeting20_v3_Slide25.pdf

Mark, Matt and I also discussed how I can proceed:

Start with HDAC (rather than HSP).
Familiarise myself with the relevant files in the rms-data-projects repo that provide useful logic for timing:

uvis_data_raw_labeler.py
uvis_data_raw_labeler-v1.py
hdac_data_raw_template.xml.

Additionally, there's a textfile and spreadsheets, HDAC_timing.txt, HDAC.numbers, HDAC-overlaps.numbers, which Mark generated by scraping labels that he used in analysis to show that HDAC had timing issues too (entirely separate from HSP timing issues).
In the process, decide whether it makes sense to continue using xmltemplate.py (that I've been using for UVIS migration so far, adapted for EUV/FUV) or switch over to the newer pdstemplate.py

Just to add briefly to Mia's notes: I believe Mark was saying that the start times for HDAC files are probably all correct, but that the stop times are all twice as far from the start time as they should be. This is because whoever created the PDS3 data files somehow thought that each data product consist of 32 readings taken in series when they were actually two sets of 16 being taken in parallel.


Feat/interactions

There were the following issues with your Pull Request

Commit: 74a5416f653ea4075f6625ec260a25a7e3dc1822
Your commit message does not contain a body
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/ExtendRealityLtd/VRTK.Unity.Core

This message was auto-generated by https://gitcop.com


Smart Github workflows for downloading engine repo

Description
Instead of hard-coding where to download the engine repository from, create a script that checks for alternate places to download from. In order, it checks for:

A user fork with the same branch name
A matching branch name in the project engine repo

If neither of those exists and $DEFAULT_BRANCH is set (via clone_engine_repo.conf) it uses that branch from the project engine repo. Otherwise, it uses the default branch for the project engine repo (ie, git clone without the --branch option).
Signed-off-by: Jim C. Nasby<EMAIL_ADDRESS>Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

What would be the use case with this automation, Jim?


pipx run resiliparse faild: ModuleNotFoundError: No module named 'joblib'

user@box:~$ pipx run resiliparse
Traceback (most recent call last):
  File "/home/user/.local/pipx/.cache/42f25da10f76b98/bin/resiliparse", line 5, in <module>
    from resiliparse.cli import main
  File "/home/user/.local/pipx/.cache/42f25da10f76b98/lib/python3.9/site-packages/resiliparse/cli.py", line 18, in <module>
    from joblib import Parallel, delayed
ModuleNotFoundError: No module named 'joblib'
user@box:~$ pipx install resiliparse
  installed package resiliparse 0.11.1, installed using Python 3.9.2
  These apps are now globally available
    - resiliparse
done! ✨ 🌟 ✨
user@box:~$ resiliparse
Traceback (most recent call last):
  File "/home/user/.local/bin/resiliparse", line 5, in <module>
    from resiliparse.cli import main
  File "/home/user/.local/pipx/venvs/resiliparse/lib/python3.9/site-packages/resiliparse/cli.py", line 18, in <module>
    from joblib import Parallel, delayed
ModuleNotFoundError: No module named 'joblib'
user@box:~$

user@box:~$ pipx run --verbose resiliparse
pipx >(setup:729): pipx version is 1.0.0
pipx >(setup:730): Default python interpreter is '/home/user/.local/pipx/venvs/pipx/bin/python'
pipx >(needs_upgrade:69): Time since last upgrade of shared libs, in seconds: 1561712. Upgrade will be run by pipx if greater than 2592000.
pipx >(run_subprocess:172): running /home/user/.local/pipx/.cache/42f25da10f76b98/bin/python -c import sysconfig; print(sysconfig.get_path('purelib'))
pipx >(run:103): Reusing cached venv /home/user/.local/pipx/.cache/42f25da10f76b98
pipx >(run_subprocess:172): running /home/user/.local/pipx/.cache/42f25da10f76b98/bin/python -c import sysconfig; print(sysconfig.get_path('purelib'))
pipx >(exec_app:387): exec_app: /home/user/.local/pipx/.cache/42f25da10f76b98/bin/resiliparse
Traceback (most recent call last):
  File "/home/user/.local/pipx/.cache/42f25da10f76b98/bin/resiliparse", line 5, in <module>
    from resiliparse.cli import main
  File "/home/user/.local/pipx/.cache/42f25da10f76b98/lib/python3.9/site-packages/resiliparse/cli.py", line 18, in <module>
    from joblib import Parallel, delayed
ModuleNotFoundError: No module named 'joblib'
user@box:~$

joblib is an optional dependency, that is installed by running pip install resiliparse[cli]. There should be a check for this, though.

found also some more missing dependencys
(venv) user@box:/tmp/chatnoir-resiliparse$ python3 -m venv venv && source venv/bin/activate
(venv) user@box:/tmp/chatnoir-resiliparse$ pip3 install resiliparse[cli]
Requirement already satisfied: resiliparse[cli] in /tmp/venv/lib/python3.9/site-packages (0.11.1)
Requirement already satisfied: fastwarc==0.11.1 in /tmp/venv/lib/python3.9/site-packages (from resiliparse[cli]) (0.11.1)
Requirement already satisfied: tqdm in /tmp/venv/lib/python3.9/site-packages (from resiliparse[cli]) (4.63.0)
Requirement already satisfied: click in /tmp/venv/lib/python3.9/site-packages (from resiliparse[cli]) (8.0.4)
Collecting joblib
  Using cached joblib-1.1.0-py2.py3-none-any.whl (306 kB)
Installing collected packages: joblib
Successfully installed joblib-1.1.0
(venv) user@box:/tmp/chatnoir-resiliparse$ pip3 install fastwarc
Requirement already satisfied: fastwarc in /tmp/venv/lib/python3.9/site-packages (0.11.1)
Requirement already satisfied: click in /tmp/venv/lib/python3.9/site-packages (from fastwarc) (8.0.4)
Requirement already satisfied: tqdm in /tmp/venv/lib/python3.9/site-packages (from fastwarc) (4.63.0)
(venv) user@box:/tmp/chatnoir-resiliparse$
(venv) user@box:/tmp/chatnoir-resiliparse$ resiliparse html benchmark /tmp/warcs/WARCPROX-20220315191329244-00000-icvgw961.warc
HTML parser benchmark <title> extraction:
=========================================
Resiliparse (Lexbor):  1 documents in 0.98s (1.02 documents/s)
[Not benchmarking Selectolax, because it is not installed.
[Not benchmarking BeautifulSoup4, because it is not installed.
(venv) user@box:/tmp/chatnoir-resiliparse$ pip3 install Selectolax BeautifulSoup4
Collecting Selectolax
  Downloading selectolax-0.3.6-cp39-cp39-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_12_x86_64.manylinux2010_x86_64.whl (5.0 MB)
     |████████████████████████████████| 5.0 MB 2.5 MB/s
Collecting BeautifulSoup4
  Using cached beautifulsoup4-4.10.0-py3-none-any.whl (97 kB)
Collecting soupsieve>1.2
  Downloading soupsieve-2.3.1-py3-none-any.whl (37 kB)
Collecting Cython>=0.29.23
  Using cached Cython-0.29.28-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl (1.9 MB)
Installing collected packages: soupsieve, Cython, Selectolax, BeautifulSoup4
Successfully installed BeautifulSoup4-4.10.0 Cython-0.29.28 Selectolax-0.3.6 soupsieve-2.3.1
(venv) user@box:/tmp/chatnoir-resiliparse$ resiliparse html benchmark /tmp/warcs/WARCPROX-20220315191329244-00000-icvgw961.warc
HTML parser benchmark <title> extraction:
=========================================
Resiliparse (Lexbor):  1 documents in 1.09s (0.92 documents/s)
Selectolax (Lexbor):   1 documents in 0.88s (1.14 documents/s)
Selectolax (MyHTML):   1 documents in 0.64s (1.56 documents/s)
Traceback (most recent call last):
  File "/tmp/venv/bin/resiliparse", line 8, in <module>
    sys.exit(main())
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/tmp/venv/lib/python3.9/site-packages/resiliparse/cli.py", line 147, in benchmark
    BeautifulSoup(bytes_to_str(content, detect_encoding(content)), 'lxml').title
  File "/tmp/venv/lib/python3.9/site-packages/bs4/__init__.py", line 245, in __init__
    raise FeatureNotFound(
bs4.FeatureNotFound: Couldn't find a tree builder with the features you requested: lxml. Do you need to install a parser library?
(venv) user@box:/tmp/chatnoir-resiliparse$ pip3 install Selectolax BeautifulSoup4 lxml
Requirement already satisfied: Selectolax in /tmp/venv/lib/python3.9/site-packages (0.3.6)
Requirement already satisfied: BeautifulSoup4 in /tmp/venv/lib/python3.9/site-packages (4.10.0)
Collecting lxml
  Using cached lxml-4.8.0-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl (6.9 MB)
Requirement already satisfied: soupsieve>1.2 in /tmp/venv/lib/python3.9/site-packages (from BeautifulSoup4) (2.3.1)
Requirement already satisfied: Cython>=0.29.23 in /tmp/venv/lib/python3.9/site-packages (from Selectolax) (0.29.28)
Installing collected packages: lxml
Successfully installed lxml-4.8.0
(venv) user@box:/tmp/chatnoir-resiliparse$ resiliparse html benchmark /tmp/warcs/WARCPROX-20220315191329244-00000-icvgw961.warc
HTML parser benchmark <title> extraction:
=========================================
Resiliparse (Lexbor):  1 documents in 1.00s (1.00 documents/s)
Selectolax (Lexbor):   1 documents in 0.87s (1.15 documents/s)
Selectolax (MyHTML):   1 documents in 0.63s (1.58 documents/s)
BeautifulSoup4 (lxml): 1 documents in 3.16s (0.32 documents/s)
(venv) user@box:/tmp/chatnoir-resiliparse$

(venv) user@box:/tmp/chatnoir-resiliparse$ resiliparse lang benchmark /tmp/warcs/WARCPROX-20220315191329244-00000-icvgw961.warc
Skipping FastText benchmark, since no model has been specified.
Skipping langid benchmark, since it is not installed.
Run "pip install langid" to install it.
Traceback (most recent call last):
  File "/tmp/venv/bin/resiliparse", line 8, in <module>
    sys.exit(main())
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmp/venv/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/tmp/venv/lib/python3.9/site-packages/resiliparse/cli.py", line 538, in benchmark
    in_data = open(infile, 'r').read().replace('\n', ' ')
  File "/usr/lib/python3.9/codecs.py", line 322, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 41278: invalid start byte
(venv) user@box:/tmp/chatnoir-resiliparse$

found also some more missing dependencys

Those are conditionally imported only for the benchmark and should not be part of the package dependency spec, but I could create an extra target, I guess.


DEX-246 Search API for individuals

Pull Request Overview

Adds elasticsearch and logstash to the composition
Adds gumby library to the project
Provides configuration for loading the individuals index from the EDM database using logstash
Provides an endpoint for proxying elasticsearch requests through houston
Adjusts the integrations checks to include elasticsearch

Review Notes
It's difficult to see the results of this work thus far. If everything is running properly and edm has some data in it, then maybe you'll see results in a search. For example, you could run echo '{"query": {"match_all": {}}}' | http -v POST localhost:9200/individuals/_search. (Though, in most cases this isn't going to give back anything because the edm has no data in it.)
If you'd artificially like to load the EDM with data (get the giraffe-ng.sql from Jon). The do the following:

instructions for loading edm with data
```
    alias dc=docker-compose
    # drop the existing schema and data
    dc exec db psql -U wildbook -d wildbook -c "drop schema public cascade; create schema public;"
    # load the database
    cat ng-giraffe.sql | dc exec -T db psql -U wildbook -d wildbook
    # verify the import...
    dc exec db psql -U wildbook -d wildbook -c "select count(*) from \"MARKEDINDIVIDUAL\";"
    # count should be 6437
```

REST API Updates
This primarily adds the /api/v1/search/<string:index> endpoint. The goal at this point is to simply proxy the request from to elasticsearch. Later adjustments will likely lock this down.
For example, the following is a search against the individuals index that searches for all records:
POST /api/v1/search/individuals
{"query": {"match_all": {}}}

Invoke CLI Updates
The app.integrations.check has been updated to include checking the connection to elasticsearch.

/rebase

Last check all things are passing/green. So I've opened this up for review.

/rebase


Twisted error on Windows with Python 3.X

A customer was using Python 3.4 in a Windows environment and getting a traceback ending with
from twisted.python import lockfile, failure
File "C:\Python34\lib\site-packages\twisted\python\lockfile.py", line 52, in <module>
_open = file
NameError: name 'file' is not defined

when importing the datasift module.
Apparently this is due to a known problem on Windows with Python 3.X (glyph is a core maintainer of twisted): http://www.scriptscoop.net/t/7d436f5544a8/twisted-work-with-python-3-3.html
Can we update the README advising Windows users to use Python 2.7?

https://github.com/datasift/datasift-python/commit/ca14f983ee3c53e4a4ef1b8341ef5321416d5768

Hopefully Twisted adds support for python 3 on windows soon, until then we can leave the README with that caveat


Fix alignments and dashes in disaggregated data columns

Issue
Having trouble scanning disaggregated data in the IPTT because of (what appear to be) the mismatched cell alignments. Also, noticed a discrepancy in dashes used for fields where there is no entered value.
Acceptance criteria

Alignments

[x] 1. # / % column is consistently right-aligned.

[x] 2. Baseline column is consistently right-aligned.

[x] 3. Target column is consistently right-aligned.

[x] 4. Actual column is consistently right-aligned.

[x] 5. % Met column is consistently right-aligned.

Dashes

[x] 1. Dashes used to indicate "nothing entered in this field" are consistently the longer style of dash.

[ ] 2. When data is unavailable for a time or target period, we always display a long dash in the Target and % Met columns, in addition to the Actual column. (Right now we show it in the Actual columns but not the other two.)

I believe this is fixed along with #2141:

Right alignments look good and I am consistently seeing gray em-dashes where there is no number available.