ArrayBuffer of an image data leaks somewhere in bowels of Pixelsmith#createImages

Spritesmith#processImages not freeing memory over time when creating multiple sprite images. vinyl files with Buffer in contents and plain file paths results in the same memory usage.
Related issue in spritesmith: Ensighten/spritesmith#72
I'm asking for help in leak detection in modules used in pixelsmith.
Simplified script to reproduce an issue
'use strict';

const fs = require('fs');

// const File = require('vinyl');
const Spritesmith = require('spritesmith');
const pixelsmith = require('pixelsmith');

const sources = [
    "./sprite/NRL-Canberra-Raiders__569847503__1493152203.png",
    "./sprite/apple_ex.png",
    "./sprite/insta-web.png",
    "./sprite/cart.png",
    "./sprite/jpg.jpg"
];

/*const sourcesContent = sources.map(imagePath =>
    new File({
        path: imagePath,
        history: [],
        content: fs.readFileSync(`${process.cwd()}/${imagePath}`)
    }));*/

const spritesmith = new Spritesmith({
    engine: pixelsmith
});

const ITERATIONS = 150;

process.nextTick(function printMemoryUsage() {
    console.log(JSON.stringify(process.memoryUsage()));
});

for (let i = 0; i < ITERATIONS; i++) {
    spritesmith.createImages(sources, (err, images) => {
        spritesmith.processImages(images);

        process.nextTick(function printMemoryUsage() {
            console.log(JSON.stringify(process.memoryUsage()));

            if (i % 50 === 0) {
                global.gc();
            }
        });

    });
}

Memory usage
{"rss":54697984,"heapTotal":36393760,"heapUsed":21949752}
{"rss":620531712,"heapTotal":65287968,"heapUsed":29065104}
{"rss":646225920,"heapTotal":64256032,"heapUsed":28251352}
{"rss":647573504,"heapTotal":64256032,"heapUsed":28290648}
{"rss":650907648,"heapTotal":64256032,"heapUsed":28326320}
{"rss":654422016,"heapTotal":64256032,"heapUsed":28348720}
{"rss":657559552,"heapTotal":64256032,"heapUsed":28380736}
{"rss":661073920,"heapTotal":64256032,"heapUsed":28401936}
{"rss":664236032,"heapTotal":64256032,"heapUsed":28430416}
{"rss":667480064,"heapTotal":64256032,"heapUsed":28447160}
{"rss":670859264,"heapTotal":64256032,"heapUsed":28473880}
{"rss":674373632,"heapTotal":64256032,"heapUsed":28489664}
{"rss":677515264,"heapTotal":64256032,"heapUsed":28517184}
{"rss":681029632,"heapTotal":64256032,"heapUsed":28532968}
{"rss":684163072,"heapTotal":64256032,"heapUsed":28559616}
{"rss":687677440,"heapTotal":64256032,"heapUsed":28575848}
{"rss":690810880,"heapTotal":64256032,"heapUsed":28602352}
{"rss":694325248,"heapTotal":64256032,"heapUsed":28618136}
{"rss":697442304,"heapTotal":64256032,"heapUsed":28653776}
{"rss":700956672,"heapTotal":64256032,"heapUsed":28672832}
{"rss":704200704,"heapTotal":64256032,"heapUsed":28688536}
{"rss":707444736,"heapTotal":64256032,"heapUsed":28706136}
{"rss":710959104,"heapTotal":64256032,"heapUsed":28721840}
{"rss":714203136,"heapTotal":64256032,"heapUsed":28737544}
{"rss":717717504,"heapTotal":64256032,"heapUsed":28753248}
{"rss":720850944,"heapTotal":64256032,"heapUsed":28778688}
{"rss":724365312,"heapTotal":64256032,"heapUsed":28799680}
{"rss":727609344,"heapTotal":64256032,"heapUsed":28825664}
{"rss":731123712,"heapTotal":64256032,"heapUsed":28841216}
{"rss":734367744,"heapTotal":64256032,"heapUsed":28856768}
{"rss":737882112,"heapTotal":64256032,"heapUsed":28872320}
{"rss":741126144,"heapTotal":64256032,"heapUsed":28887872}
{"rss":744640512,"heapTotal":64256032,"heapUsed":28906144}
{"rss":747884544,"heapTotal":64256032,"heapUsed":28921696}
{"rss":751398912,"heapTotal":64256032,"heapUsed":28937248}
{"rss":754642944,"heapTotal":64256032,"heapUsed":28952800}
{"rss":757886976,"heapTotal":64256032,"heapUsed":28968352}
{"rss":761401344,"heapTotal":64256032,"heapUsed":28983904}
{"rss":764645376,"heapTotal":64256032,"heapUsed":28999456}
{"rss":768159744,"heapTotal":64256032,"heapUsed":29015008}
{"rss":771403776,"heapTotal":64256032,"heapUsed":29030560}
{"rss":774918144,"heapTotal":64256032,"heapUsed":29046112}
{"rss":778162176,"heapTotal":64256032,"heapUsed":29061664}
{"rss":781406208,"heapTotal":64256032,"heapUsed":29077216}
{"rss":784920576,"heapTotal":64256032,"heapUsed":29093720}
{"rss":788434944,"heapTotal":64256032,"heapUsed":29109272}
{"rss":791678976,"heapTotal":64256032,"heapUsed":29124824}
{"rss":794923008,"heapTotal":64256032,"heapUsed":29140376}
{"rss":798437376,"heapTotal":64256032,"heapUsed":29155928}
{"rss":801681408,"heapTotal":64256032,"heapUsed":29171480}
{"rss":804925440,"heapTotal":64256032,"heapUsed":29187032}
{"rss":808439808,"heapTotal":64256032,"heapUsed":29202584}
{"rss":814039040,"heapTotal":65287968,"heapUsed":28634656}
{"rss":817278976,"heapTotal":65287968,"heapUsed":28656736}
{"rss":820748288,"heapTotal":65287968,"heapUsed":28672456}
{"rss":823992320,"heapTotal":65287968,"heapUsed":28688176}
{"rss":827236352,"heapTotal":65287968,"heapUsed":28703896}
{"rss":830750720,"heapTotal":65287968,"heapUsed":28719616}
{"rss":833994752,"heapTotal":65287968,"heapUsed":28735336}
{"rss":837509120,"heapTotal":65287968,"heapUsed":28751056}
{"rss":840753152,"heapTotal":65287968,"heapUsed":28766776}
{"rss":843997184,"heapTotal":65287968,"heapUsed":28785464}
{"rss":847511552,"heapTotal":65287968,"heapUsed":28805632}
{"rss":850755584,"heapTotal":65287968,"heapUsed":28821352}
{"rss":853999616,"heapTotal":65287968,"heapUsed":28837072}
{"rss":857513984,"heapTotal":65287968,"heapUsed":28852792}
{"rss":860758016,"heapTotal":65287968,"heapUsed":28872776}
{"rss":864272384,"heapTotal":65287968,"heapUsed":28888496}
{"rss":867516416,"heapTotal":65287968,"heapUsed":28904216}
{"rss":870760448,"heapTotal":65287968,"heapUsed":28922496}
{"rss":874156032,"heapTotal":65287968,"heapUsed":28945168}
{"rss":877670400,"heapTotal":65287968,"heapUsed":28960808}
{"rss":880914432,"heapTotal":65287968,"heapUsed":28976448}
{"rss":884158464,"heapTotal":65287968,"heapUsed":28992088}
{"rss":887672832,"heapTotal":65287968,"heapUsed":29007728}
{"rss":890916864,"heapTotal":65287968,"heapUsed":29023368}
{"rss":894160896,"heapTotal":65287968,"heapUsed":29039008}
{"rss":897675264,"heapTotal":65287968,"heapUsed":29062360}
{"rss":900919296,"heapTotal":65287968,"heapUsed":29077912}
{"rss":904433664,"heapTotal":65287968,"heapUsed":29093464}
{"rss":907677696,"heapTotal":65287968,"heapUsed":29109016}
{"rss":911192064,"heapTotal":65287968,"heapUsed":29124568}
{"rss":914436096,"heapTotal":65287968,"heapUsed":29140120}
{"rss":917950464,"heapTotal":65287968,"heapUsed":29155672}
{"rss":921194496,"heapTotal":65287968,"heapUsed":29171224}
{"rss":924708864,"heapTotal":65287968,"heapUsed":29186776}
{"rss":927952896,"heapTotal":65287968,"heapUsed":29202328}
{"rss":931467264,"heapTotal":65287968,"heapUsed":29217880}
{"rss":1087750144,"heapTotal":34329888,"heapUsed":28902344}
{"rss":1090994176,"heapTotal":34329888,"heapUsed":28924368}
{"rss":1094508544,"heapTotal":34329888,"heapUsed":28940048}
{"rss":1097752576,"heapTotal":34329888,"heapUsed":28955728}
{"rss":1100996608,"heapTotal":34329888,"heapUsed":28971408}
{"rss":1104621568,"heapTotal":34329888,"heapUsed":30436208}
{"rss":1107861504,"heapTotal":34329888,"heapUsed":30451888}
{"rss":1111105536,"heapTotal":34329888,"heapUsed":30467568}
{"rss":1114619904,"heapTotal":34329888,"heapUsed":30483248}
{"rss":1117863936,"heapTotal":34329888,"heapUsed":30498928}
{"rss":1121378304,"heapTotal":34329888,"heapUsed":30514608}
{"rss":1124622336,"heapTotal":34329888,"heapUsed":30530288}
{"rss":1128136704,"heapTotal":34329888,"heapUsed":30545968}
{"rss":1131380736,"heapTotal":34329888,"heapUsed":30561648}
{"rss":1134624768,"heapTotal":34329888,"heapUsed":30577328}
{"rss":1138139136,"heapTotal":34329888,"heapUsed":30139696}
{"rss":1143013376,"heapTotal":30214176,"heapUsed":23720136}
{"rss":1143013376,"heapTotal":30214176,"heapUsed":23742160}
{"rss":1143099392,"heapTotal":30214176,"heapUsed":23757840}
{"rss":1143099392,"heapTotal":30214176,"heapUsed":23773520}
{"rss":1144176640,"heapTotal":30214176,"heapUsed":23789200}
{"rss":1147420672,"heapTotal":30214176,"heapUsed":23804880}
{"rss":1150664704,"heapTotal":30214176,"heapUsed":23820560}
{"rss":1154179072,"heapTotal":30214176,"heapUsed":23836240}
{"rss":1157423104,"heapTotal":30214176,"heapUsed":23851920}
{"rss":1160667136,"heapTotal":30214176,"heapUsed":23870208}
{"rss":1164173312,"heapTotal":30214176,"heapUsed":23891360}
{"rss":1167417344,"heapTotal":30214176,"heapUsed":23912256}
{"rss":1170931712,"heapTotal":30214176,"heapUsed":23927872}
{"rss":1174175744,"heapTotal":30214176,"heapUsed":23943488}
{"rss":1177419776,"heapTotal":30214176,"heapUsed":23959104}
{"rss":1180934144,"heapTotal":30214176,"heapUsed":23974720}
{"rss":1184178176,"heapTotal":30214176,"heapUsed":23990336}
{"rss":1187422208,"heapTotal":30214176,"heapUsed":24005952}
{"rss":1190936576,"heapTotal":30214176,"heapUsed":24024448}
{"rss":1194180608,"heapTotal":30214176,"heapUsed":24040064}
{"rss":1197694976,"heapTotal":30214176,"heapUsed":24055680}
{"rss":1200939008,"heapTotal":30214176,"heapUsed":24071296}
{"rss":1204183040,"heapTotal":30214176,"heapUsed":24086912}
{"rss":1207697408,"heapTotal":30214176,"heapUsed":24102528}
{"rss":1210941440,"heapTotal":30214176,"heapUsed":24118144}
{"rss":1214320640,"heapTotal":30214176,"heapUsed":24140120}
{"rss":1217564672,"heapTotal":30214176,"heapUsed":24155736}
{"rss":1221079040,"heapTotal":30214176,"heapUsed":24171352}
{"rss":1224241152,"heapTotal":31246112,"heapUsed":24263000}
{"rss":1227722752,"heapTotal":31246112,"heapUsed":24301600}
{"rss":1231237120,"heapTotal":31246112,"heapUsed":24316952}
{"rss":1234481152,"heapTotal":31246112,"heapUsed":24332304}
{"rss":1237725184,"heapTotal":31246112,"heapUsed":24347656}
{"rss":1241239552,"heapTotal":31246112,"heapUsed":24363008}
{"rss":1244483584,"heapTotal":31246112,"heapUsed":24378360}
{"rss":1247727616,"heapTotal":31246112,"heapUsed":24393712}
{"rss":1251241984,"heapTotal":31246112,"heapUsed":24409064}
{"rss":1254486016,"heapTotal":31246112,"heapUsed":24424416}
{"rss":1258000384,"heapTotal":31246112,"heapUsed":24439768}
{"rss":1261244416,"heapTotal":31246112,"heapUsed":24455120}
{"rss":1264488448,"heapTotal":31246112,"heapUsed":24470472}
{"rss":1268002816,"heapTotal":31246112,"heapUsed":24485824}
{"rss":1271246848,"heapTotal":31246112,"heapUsed":24501176}
{"rss":1274761216,"heapTotal":31246112,"heapUsed":24516528}
{"rss":1278005248,"heapTotal":31246112,"heapUsed":24531880}
{"rss":1281249280,"heapTotal":31246112,"heapUsed":24547232}
{"rss":1284763648,"heapTotal":31246112,"heapUsed":24562584}

4 heap snapshots were captured over time shows huge amount of system/JSArrayBufferData allocations.

I'm assuming that images are leaking in streams of get-pixels, ndarray or pngjs-nozlib modules, being used in process.nextTick() in smith.js.
canvassmith doesn't result in memory leak, so it's an issue of pixelsmith

Heap snapshots with canvassmith:
canvassmith_heapsnapshots.tar.gz
Packages versions
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Wow, great issue content. Thanks!
Would it be possible to get the error-prone images as well?

Unfortunately, it doesn't depend on images. There are 81K+ sources for 800+ destination sprites in the project I'm working on, and the issue reproduces with any combination of it.
Anyway, images were used in the script above: sprite_images.tar.gz

Unfortunately, it doesn't depend on images. There are 81K+ sources for 800+ destination sprites in the project I'm working on, and the issue reproduces with any combination of images.
Anyway, images were used in the script above: sprite_images.tar.gz

I've run that sample script through valgrind and examined output
output.tar.gz with massif-visualizer
valgrind --tool=massif node ./test.js

Here is a screenshot of the heap usage of C++ calls. I haven't understand yet how to get JS stack from valgrind output. It shows that leak is near to zlibusage.

And run Valgrind's leak-detector shows that leaks are near to node::ZCtx::New(v8::FunctionCallbackInfo<v8::Value> const&) and node::ZCtx::Init(v8::FunctionCallbackInfo<v8::Value> const&)
valgrind --leak-check=full --show-leak-kinds=all node ./test.js

Full valgrind output: valgrind_leaks.log.tar.gz
Now I'm trying to get JS stack from the valgrind trace. Probably, it should be done with [lldb](http://www.brendangregg.com/blog/2016-07-13/llnode-nodejs-memory-leak-analysis.html) or something similar.

Cool, thanks for digging into this so much. I don't have time over the next few days to take a look at this. My recommendation is to isolate which of the libraries in get-pixels is causing the issue by using only JPGs or only PNGs and seeing when it occurs
Also, it looks like global.gc() is being run every 50 iterations. Shouldn't it be run every iteration and memory usage be dumped after that? We want to verify a leak, not worry about uncollected variables

Also, we should probably be using a callback function instead of a for loop. With this current variant, we are running all of the spritesmith actions in parallel instead of in series. I imagine we want to see the leak grow over time


Question: Is there an example of Configure-from-code in dotnet core?

Hello,
I currently have a fully setup NLog configuration using code for dotnet framework. We are looking to port our application into dotnet core and I have realized that there does not seem to be any documentation around doing code based configuration in dotnet core. I guess I have two related questions, did I miss the documentation somewhere? If not, does anyone have a sample I can look at on how to do this?
References

dotnet Framework Configure from Code
dotnet Core 2 Console

Examples like these:
https://github.com/NLog/NLog.Extensions.Logging/tree/master/examples/NetCore2

@snakefoot, I have seen those samples but since all the objects that the MEL-LoggingProvider uses are different I was not sure how to set the configuration on them. Here is a sample of what I am doing for the framework version of nlog.
// Set the root log configuration, this is used to log things about the log system.
InternalLogger.LogFile =<EMAIL_ADDRESS>InternalLogger.LogLevel = LogLevel.Info;

var config = new LoggingConfiguration();

var target = new FileTarget
{
     ArchiveAboveSize = ARCHIVE_LOG_FILE_ABOVE_SIZE,
     ArchiveEvery = FileArchivePeriod.Day,
     MaxArchiveFiles = DELETE_LOG_FILE_AFTER_THIS_AMOUNT,
     ConcurrentWrites = false,
     Name = "error",
     FileName =<EMAIL_ADDRESS>     Layout = "${longdate}|${level}|${logger}|${message}|${exception:format=tostring}",
};

config.AddTarget(name, target);
var rule = new LoggingRule("*", target);
rule.EnableLoggingForLevel(LogLevel.Error);
rule.EnableLoggingForLevel(LogLevel.Fatal);
config.LoggingRules.Add(rule);

var consoleTarget = new ColoredConsoleTarget
{
    Name = "console",
    Layout = BASIC_LAYOUT,
};
config.AddTarget(consoleTarget.Name, consoleTarget);
var consoleRule = new LoggingRule("*", LogLevel.Info, consoleTarget);
config.LoggingRules.Add(consoleRule);

LogManager.Configuration = config;

Would I just do the same thing and then register the logger with DI?

Would I just do the same thing and then register the logger with DI?

Yes it is a 2 stage setup;

First stage is where you setup NLog-targets (as your example above).
Second stage is adding NLog as MEL-LoggingProvider using extension-methods UseNLog() or AddNLog()

@kylelaverty Btw. since you have experience with configuring NLog-config from code, then you are very welcome to checkout #4128. It provides a more fluent API for setting up NLog-config from code. Any feedback is welcome.

@snakefoot I will take a look and comment in that linked PR if I have any questions/comments. Just for anyone looking at this later, this question has stemmed from trying to hook NLog into EF core's logging system so I can see what sql is actually being generated. Here is a sample from the Microsoft docs:
public static readonly LoggerFactory MyLoggerFactory
    = new LoggerFactory(new[] {new ConsoleLoggerProvider((_, __) => true, true)});

protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    => optionsBuilder
        .UseLoggerFactory(MyLoggerFactory)
        .UseSqlServer(@"Server=(localdb)\mssqllocaldb;Database=EFLogging;ConnectRetryCount=0");

The idea would be to use the NLogLoggerProvider instead of the ConsoleLoggerProvider from the sample above.

I guess you can decide if you want to do it like this:

EF UseLoggerFactory

Microsoft MEL-LoggerFactory

AddNLog LoggerProvier from NLog.Extensions.Logging

Or do this:

EF UseLoggerFactory
- NLogLoggerFactory  from NLog.Extensions.Logging

I assume your question has been answered, if not, please let us know!


Update machine-learning-r-csharp-normal-distribution.md

Hi @nk773, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (neerajkh). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@nk773
Please correct the date format. Month and day must be two digits each, so it's 02/28/2016. Thanks.


Add Travis CI

Adds Travis CI which will:

Build, test and bench the main library
Build the sample
Upload docs to a gh-pages branch on the master branch

@KodrAus thanks!


Make Search component icon clickable

A "search" icon on Search component should be an icon button, that would be able to be clicked to trigger a search event.
Maybe this component should even be wrapped in <form> (or Form) as suggested by @rabelloo?

This is blocked by https://github.com/onfido/castor/issues/466.


Support xhr request by Access-Control-Allow-Origin: *

Support xhr request by Access-Control-Allow-Origin: *
add the header so we can use xhr to get the image?

Added the header, but most of the content will be cached for one month. I also still need to commit the changes in Nginx-config to this repo.


Neogit not working after fresh clone of repo

After a pull today Neogit isn't working anymore.
I get "Not an editor command: Neogit"
I tried to manually load it in Packer but with no success.
Does anyone have any ideas what's going on?
Other git features work.
PackerStatus shows neogit as an installed plugin

Tanks for reporting!
I can confirm this with my Docker image. My local nvim does not have this issue.
I will dig into it later

@kristiankupsch which version of Neovim are you running?

NVIM v0.8.0
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>

Updated to:
NVIM v0.9.0-dev-1378+ga8d454816-dirty
Build type: Release
LuaJIT 2.1.0-beta3
Error still occurs

Thanks for the info
I found an issue in in Neogit which I guess is the root cause

FYI Here is the PR that will fix it. Just in case you want to track it:
https://github.com/TimUntersberger/neogit/pull/380

FYI: The PR in Neogit is merged

Yeah I saw it. Do you think https://github.com/Allaman/nvim/commit/092244719dfde6f670f69ff6d636b9858ec3c01e should be reverted. Actually, I like the workaround more than the original 😆

Well, I reverted it so see if (somehow magically) it solves the issue with color highlighting on OS X which is still not working correctly.
It doesn't. So I think leave it as is. Calling the Lua function directly seems less error prone.


FETCH

Initially we wanted to avoid a fetch dependency and wrote the request implementation against XHR.
I think its cool to depend on fetch at this point.
It gives us a nicer API to allow subclasses to configure the behavior such as adding new headers.
Theres two ways to do it:
// shim an entire fetch API (replaces default window.fetch)
window.IncludeFragmentElement.prototype.fetch = myCoolFetch

or extend the initial request headers set:
const originalRequest = window.IncludeFragmentElement.prototype.request
window.IncludeFragmentElement.prototype.request = function() {
  const request = originalRequest.apply(self)
  request.headers.set('X-Cool': '1'})
  return request
}

CC: @dgraham @aroben

I really like that this lets apps customize the requests these elements send. I can't comment too much on the implementation, but I like the results!


python310Packages.awkward-cpp: 17 -> 21

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.awkward-cpp is: CPU kernels and compiled extensions for Awkward Array
meta.homepage for python310Packages.awkward-cpp is: https://github.com/scikit-hep/awkward
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 21 with grep in /nix/store/vw1y3272m4lm506jczj1ij2x3y8i61q0-python3.10-awkward-cpp-21
found 21 in filename of file in /nix/store/vw1y3272m4lm506jczj1ij2x3y8i61q0-python3.10-awkward-cpp-21

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.awkward
python310Packages.awkward-cpp
python310Packages.coffea
python310Packages.dask-awkward
python310Packages.mplhep
python310Packages.uproot
python310Packages.vector
python311Packages.awkward-cpp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vw1y3272m4lm506jczj1ij2x3y8i61q0-python3.10-awkward-cpp-21 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.awkward-cpp https://github.com/r-ryantm/nixpkgs/archive/27947f69dc15be3764ef8e9f669ad7033078cffa.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vw1y3272m4lm506jczj1ij2x3y8i61q0-python3.10-awkward-cpp-21
ls -la /nix/store/vw1y3272m4lm506jczj1ij2x3y8i61q0-python3.10-awkward-cpp-21/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

6 packages failed to build:

python310Packages.awkward
python310Packages.coffea
python310Packages.dask-awkward
python310Packages.mplhep
python310Packages.uproot
python310Packages.vector

2 packages built:

python310Packages.awkward-cpp
python311Packages.awkward-cpp

Maintainer pings
cc @veprbl for testing.

Result of nixpkgs-review pr 248284 run on x86_64-linux 1

12 packages failed to build:

python310Packages.awkward
python310Packages.awkward.dist
python310Packages.coffea
python310Packages.coffea.dist
python310Packages.dask-awkward
python310Packages.dask-awkward.dist
python310Packages.mplhep
python310Packages.mplhep.dist
python310Packages.uproot
python310Packages.uproot.dist
python310Packages.vector
python310Packages.vector.dist

4 packages built:

python310Packages.awkward-cpp
python310Packages.awkward-cpp.dist
python311Packages.awkward-cpp
python311Packages.awkward-cpp.dist


It's not possible to import two different things with the same name. Error 'Parsing error: Identifier "vuePhotoZoomPro" has already been declared'

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

It seems to be a documentation problem. I will fix it.

https://github.com/mater1996/vue-photo-zoom-pro/releases/tag/2.2.1


[Launcher] Context menu buttons are inconsistent

Some search results don't have the buttons when in focus, some do. This behaviour seems to vary depending on the search result.
Based on msi from commit 064d638

The looks to be the same root cause as #2304 I'll investigate.

closing in favor of #2420


Question: Interpretation of cuda results

Tristan, I'm going to officially do a PR, but wanted to run this preview by you and get your take on how to interpret the results. This is from a machine with Core i9 and RTX4090. Does it look right? What do these results mean (specifically in the cuda column)? Thanks!
Average benchmark:

Operation
cpu
cuda

Argmax
9.36
0.06

BCE
26.26
25.90

Concat
30.27
0.06

Conv1d
61.74
0.17

Conv2d
26.07
0.09

LeakyReLU
6.11
0.04

Linear
103.52
0.07

MatMul
81.08
0.05

PReLU
4.86
0.05

ReLU
10.29
0.06

SeLU
7.34
0.06

Sigmoid
7.88
0.04

Softmax
21.22
0.04

Softplus
7.25
0.04

Sort
61.68
0.11

Sum
13.22
0.06

SumAll
8.50
0.06

hi @alexziskind1, thanks for providing your results!
The results seem ok for me, the performance on GPU with cuda device is extremely fast compared to CPU, this explains the large difference between the two columns. I will upgrade tomorrow the benchmark to integrate cuda/cpu speedup! So that we can better understand the difference between cpu and gpu performance.
Your results are very interesting, thanks! I'll also provide Tesla V100 benchmarks very soon

I also have a M3, M3 Pro, and M3 Max, M1 Pro and M2 Max I can add :)

What i'd like to know, please, is what these numbers mean. I see that the cuda numbers are significantly different, but what are they? Thanks!

So the idea is that each value represents the average runtime in milliseconds for executing all examples in each operation.
You can check the detailed benchmark here with all the detailed examples.

So basically it takes 0,06ms in average to run the the Linear operations

It would be amazing if you can add your results to the benchmarks with these chips!!


Release 1.1.0

Version 1.1.0
New features

Provide version of module via 'OnNewStatusModuleVersion'
Serves 'updateListOfFiles' to trigger via UI
Check if features of module can be used on device and provide this via 'OnNewStatusModuleIsActive' event / 'getStatusModuleActive' function

Improvements

New UI design available (e.g. selectable via CSK_Module_PersistentData v4.1.0 or higher), see 'OnNewStatusCSKStyle'
'loadParameters' returns its success
'sendParameters' prepared to control if sent data should be saved directly by CSK_Module_PersistentData (but not used as there are no parameters available so far)
Added UI icon and browser tab information

Bugfix

'Disk usage' in UI showed free bytes and not used bytes

Following tasks were checked

[x] Module is coded and structured according to the "Developing guideline for modules" described within the CSK documentation
[x] All functions/events/parameters are documented within the manifest documentation
[x] The manifest description of the main CROWN includes main information about the purpose of the module and how to use it in general
[x] API docu based on the manifest was created and stored within the "docu"-folder of the repository
[x] Internal LUA code documentation exists for variables and non served functions
[x] All relevant infos are logged via the SharedLogger 'ModuleLogger'
[x] Module supports persistent data feature based on 'CSK_Module_PersistentData'
[x] Module supports user management based on 'CSK_Module_UserManagement'
[x] No open "ToDos" within the code or at least clearly explained comments why they exist...
[x] "Version" key in app manifest was updated following semantic versioning (and use '0.x.y' for test / experimental modules which are not yet ready to be officially released)
[x] Meaningful IDs used for UI elements
[x] Module was tested on an AppSpace device (at least on SICK AppEngine) with no error message
[x] README.md is up to date (incl. info of device + firmware the module was tested with)
[x] CHANGELOG.md is up to date

Thx for the review!


Allow changing password for db user

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
I have seen that there are several variables for the passwords of the root and db user inside the db container
MYSQL_PWD=db
MYSQL_ROOT_PASSWORD=root
MYSQL_PASSWORD=db

Would it be possible to read these variables from the .ddev/.env file, i.e. via
MYSQL_PASSWORD=db-changed
MYSQL_ROOT_PASSWORD=root-changed

and pass it to the /healtcheck.sh inside the db container?
While changing the root password currently works, changing the password of the db user seems to collide with the /healthcheck.sh of the db container, since the password is hardcoded
# If we can now access the server, we're healthy and ready
if  mysql --host=<IP_ADDRESS> -udb -pdb --database=db -e "SHOW DATABASES LIKE 'db';" >/dev/null;  then
    printf "healthy"
    touch /tmp/healthy
    exit 0
fi

and the .ddev/.env file is not passed to the db container.
Of course i could create another user for this, but it would be more elegant to change the password of the existing db user only.
Describe your solution

Include .ddev/.env in db container or source it in /healtcheck.sh from /mnt/ddev_config/.env
Adjust /healtcheck.sh inside the db container to use the environment variable MYSQL_PASSWORD

Describe alternatives
No response
Additional context
No response

Hi @h1nds1ght - thanks for opening this.
DDEV's strategy has always been to keep the db user as simple as possible (named 'db', with password 'db' and host and database 'db'.  But at the same time, it gives you complete control just like you'd have on your own server, see https://ddev.readthedocs.io/en/latest/users/usage/database-management/#database-management
Can you say more about what you'd want to accomplish? Since you have complete control, you can already add users access, or grant the db user access to any database.

It's easy but undesirable to change the db user password, but since you have full control you can do it anyway. You can also add additional users.
Closing since this can be done already and is fully under user control. Happy to continue the conversation here or elsehwere.

I am using ddev on a test development server for a drupal10 project and even there the user db:db would not fulfill the quite low security requirements, beside using hardened_images.
But I think overall it is the best idea to just create another user with a more secure password. For the moment I just create it on post start with credentials provided by .ddev/.env, included also in settings.local.php.
At post-start I also change the root password. The only thing I noticed is a message at ddev start which might be a result of a hard coded root password somewhere in ddev:

Unable to SET GLOBAL TRANSACTION ISOLATION LEVEL READ COMMITTED: composeCmd failed to run 'COMPOSE_PROJECT_NAME=ddev-some-project docker-compose -f /some/path/.ddev/.ddev-docker-compose-full.yaml exec -T db bash -c set -eu && ( mysql -uroot -proot -e "SET GLOBAL TRANSACTION ISOLATION LEVEL READ COMMITTED;" >/dev/null 2>&1)', action='[]', err='exit status 1', stdout='', stderr=''

Would be nice if you could have a look at this.
Beside this everything seems to work surprisingly good.
Thank you for your work, makes life easier every day 😄

You'll note that the command in question is using mysql -uroot -proot - I imagine you changed the root password?

Yes thats the point of this issue. I want to use ddev on a test development server and therefore change at least the passwords.
For the db user I can just create a new user, but for the root user I need to change the password, otherwise it would be pointless.

Again, DDEV is designed for local development. You're using it for other things. I should note that database passwords are pretty much useless anyway (as all passwords are pretty much useless.)


Unlimited api request functions (REPLACEMENT for #188)

This is an improved replacement pr for #188. If this gets merged, #188 needs to be closed.
Right now, if you try to request more than 40 data entries from various functions with api requests, it won't work.
This is fixed here by introducing api_iterative_data and api_iterative_simple which iteratively fetch data from a url until the end.
These functions are moved to a separate file: commons.py
This module is not imported from __init__ (not exposed). Now it also contains common headers for requests.
TASKS TO DO:

[x] api_iterative_data
[x] A simple wrapper for api_iterative_data
[x] Rework all project functions and retain compatibility
[x] Rework all studio functions and retain compatibility
[x] Adapt for new commits (since Apr 2)
[ ] Better headers and cookies management
[ ] Rework all user iterative functions and retain compatibility
[ ] Rework get_cloud_logs
[ ] Move some stuff to commons.py
[ ] Adapt (and possibly expand) wiki

The PR FORMATS project.py, studio.py (idk why this whole project isn't formatted) with black. This can create incompatibilities for existing PRs but will MASSIVELY help in the future with diffs.
Other changes:

Created commons.py
Moved headers to commons.py
fixed a bug where if the comments num % 40, comments and get_comment_replies fetch excess batches up to limit
fixed typo in studio.host code
improved class attr check

@TimMcCool please review this approach and implementation

The new commons file seems also like a great idea.

should be merged alredy

This is a WIP, and i can't make sure to develop it until @TimMcCool gives some feedback

@TimMcCool respond

This looks like a great addition to scratchattach, I'll merge most of this with some small additions tomorrow. Sorry for the wait

Do we need to split (to separate PRs)? WDYM "most of this"?
Basically only user functions and get_cloud_logs are left to do.
If you want a split, i would suggest this:
PR 1: Commons.py and project, studio and user functions
PR 2: Better headers and cookies management, Move some stuff to commons.py, Adapt wiki

what about the weird "cachebust" url param?

Btw this project has a lot of code smells and mistakes, can work on a pr to clear that up

After looking over the code i decided to merge all of thus. I added the cachebust parameter because I suspected Server-side Caching on some API endpoints. i know that there are dome flaws in the library, thanks for improving it

THIS IS NOT DONE


Baking Issues

Hey @svermeulen sorry to bother you with this. But I am still struggling with those baking issues since 8.0.1: https://github.com/modesttree/Zenject/issues/603
I am now using namespace pattern ^ and I tried checking All generated Assemblies and still see some types not being found in the baking data like:
LayoutGroup
GraphicRaycaster
TMP_InputField
...

Actually all these types I don't need Zenject to Inject. Is there maybe a way to annotate a field to be ignored in the Injection phase to? Something like [NoInject]?
I am on Unity 2<IP_ADDRESS>f1

There is the [NoReflectionBaking] attribute.
One important thing to be aware of is that zenject is only able to add reflection baking to libraries that reference Zenject-usage.dll (since baking needs to add code that makes use of that to the library).   So if you have a library that has MonoBehaviours in it, which doesn't reference zenject, and you include that a zenject scene, that it will have to use runtime injection.
If you're sure that everywhere that you need injection is baked, then you can use the ReflectionBakingCoverageModes.NoCheckAssumeFullCoverage mode.  This will not fallback on runtime reflection so you can avoid the performance penalty for monobehaviours you don't care about, but you have to be careful that everything is fully baked for that to work.   FallbackToDirectReflectionWithWarning is useful to list all the classes would be skipped


refactor(configure): register resources synchronously

@EisenbergEffect follow the changes in https://github.com/aurelia/framework/pull/858

Since this change depends on https://github.com/aurelia/framework/pull/858, I will update the PR by adding aurelia-framework as a peer dependency with the correct version of aurelia-framework once that PR gets pulled in and published.

The base at aurelia/framework#885 got merge, this is good to go @davismj @EisenbergEffect

@davismj Please merge when you get a chance.


Help with Migration

Hi I am getting the following error when running migration:
Unable to create an object of type 'EquinoxContext'. For the different patterns supported at design time, see https://go.microsoft.com/fwlink/?linkid=851728 ---> System.MissingMethodException: No parameterless constructor defined for this object.
Duplication steps:

cd to Equinox.infra.Data folder
Run dotnet ef migrations add InitialCreate --context EquinoxContext --verbose
Encounter error

Please any advise on how to get past this error. Currently using VS Community 8.2.5 on MacOS

Hello,
In NativeInjectorBootStrapper.cs change services.AddScoped<'EquinoxContext'>(); to services.AddDbContext <'EquinoxContext'>();
Same to EventStoreSQLContext.
Hope this helps.

I resolved by changing the context class.
Now without ctor.

same to :
EventStoreSQLContext and
ApplicationDbContext

Hi,
The error happens because the context class does not have an empty constructor, which makes it impossible for migration to know how to instantiate the class:

To solve it, you can create a class that fabricates the context:

I resolved by changing the context class.
Now without ctor.

same to :
EventStoreSQLContext and
ApplicationDbContext

Hi,
Your solution solves the problem because the context class has no constructor.
But the options should be passed via constructor, so it does not create concrete dependency within the class and facilitates tests.


Update sea-orm and async-graphql

PR Info

Closes https://github.com/SeaQL/seaography/issues/123

Breaking Changes

[ ] Updates sea-orm to v0.11 and async-graphql to v5

Yes, I will look at the breakages in a few days, and update the PR.
The other features discussed in #123 should probably be a separate PR, right?


++debug reporting single PE for provisioning

Everything seems to be working correctly, but when running with ++debug, the executions in the GDB windows report:
Charm++> No provisioning arguments specified. Running with a single PE.
Use +auto-provision to fully subscribe resources or +p1 to silence this message.

even when running with multiple PEs. The terminal where the user actually invokes the program reports the PE count and execution correctly (as does the GDB window does when it starts execution, it just also reports the provisioning warning first, which is misleading).

I'm guessing the print statement for this message should only run if CkMyPe() == 0.

I'm guessing the print statement for this message should only run if CkMyPe() == 0.

I don't think that's sufficient, it should also be cognizant of the fact that a +p argument was provided and not print out anything at all.


Question about .shuffle function reproducibility

Hi,
May I ask if there is a way to set the seed for the shuffle function for the reproducibility consideration? Thanks!

There are two separate shuffles happening, shard shuffles and sample shuffles. Both are entirely under your control.
WebDataset just uses random.shuffle for shards and random.randint for samples, so you can set random.seed for getting a reproducible sequence.
If you want more control, you can override the rng= argument to iterators.shuffle for sample shuffling, and the random argument for ShardList/WebDataset for the shard shuffle.
TODO: add a test case

Got it, thanks for the reply!


Add Power Support ppc64le and update versions

Added power support for the travis.yml file with ppc64le. This is part of the Ubuntu distribution for ppc64le. This helps us simplify testing later when distributions are re-building and re-releasing.

loose-envify is using org instead of com as per below, please convert to travis-ci.com as org is getting deprecated in few weeks.
https://travis-ci.org/github/zertosh/loose-envify/builds/750401906
I have used com and the jobs ran good as per below.
https://travis-ci.com/github/nagesh4193/loose-envify/builds/209637381


The output value is null after doing a GET request on the url of the " statusQueryGetUri"

OS: Win 10
Build Version: 20230128.2
Repro Steps:

Create and open a JavaScript project with "Durable Function orchestrator" function and select "Netherite" durable storage type.
Create functions with "Durable Functions activity" and "Durable Functions HTTP Starter" templates.
F1 -> Azurite: Start.
F5 -> Select "Use Local Emulator".
Create a new file request.http in the project -> Type GET Url -> Click "Send Request" -> Copy statusQueryGetUri in the Response file -> TypeGET statusQueryGetUri-> Click "Send Request".
Check the output value.

Expect:
The output value should display as below.

Actual:
The output value is null.

Oh I had a thought, this sometimes happens when your filenames do not match the calls being made inside the index.js files.

@MicroFish91  This issue does not reproduce after changing the filename to match inside the index.js files.


feat(spanner): add samples for query with GFE, gRPC and query stats

Go counterpart of https://github.com/GoogleCloudPlatform/java-docs-samples/tree/master/spanner/opencensus.

@mayurkale22


[3.0.0-alpha.0] build-storybook is looking for polyfills.js from @kadira/storybook

Building storybook ...
Failed to build the storybook
Module not found: Error: Cannot resolve 'file' or 'directory'<EMAIL_ADDRESS>in ...

May be a mistake on my part. Closing for now.


LinkType "EventSubscribers" is not documented

I have a situation where I need to perform a management task on all of the EventSubscribers of a UnifiedGroup. When I use Get-Help -Name Get-UnifiedGroupLinks, there is a value called "EventSubscribers" however when I try to use this value with Get-UnifiedGroupLinks -LinkType, I receive an error message: "Invalid value for LinkType parameter."

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

ID: 3a66e3b7-94f5-f1f1-3b95-4fd8337b15b3
Version Independent ID: c849c0a5-96f0-fd15-2991-b75b02169424
Content: Get-UnifiedGroupLinks (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Get-UnifiedGroupLinks.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@larspassic, see related Issue 7207. The content on the web has been updated to remove EventSubscribers as a possible value for the LinkType parameter. Although this same content is used for help at the command line (Get-Help <Cmdlet>), there's always a lag. The content on the web is always more up-to-date than the help at the command line.
As per the other issue, if you need to get a list of only subscribers to the group, please open a support case and ask how you're supposed to do that without the EventSubscribers value for the LinkType parameter.

@larspassic Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


openapi-validation should only validate json type request for request body validation

If we expect some other data type in the request body. for example text/csv, XML, plain text.
The openapi specification defined as below as sample:
requestBody:
description: The contents of the csv file to be uploaded. Expected UTF-8 format.
required: true
content:
text/csv:
schema:
type: string
And user indicated the header content-type is text/csv.
When user submit request, will get validation error:
{"statusCode":400,"code":"ERR11014","message":"VALIDATOR_REQUEST_BODY_MISSING","description":"Method post on path /user/async/bulkimport requires a request body. None found.","severity":"ERROR"}
So for the request body validation, we can skip it if the content type indicated and it is not  "application/json"

I think we still need to validate the request in the case that text/csv is the content type. The only thing that we need to skip is the schema validation for the body. Header, query parameters, and path parameters all need to be validated. What do you think?

@stevehu
As you suggested, only skip request body validation if the content type is indicated as non-application/json.

 openapi-validation should only validate json type request for request body validation

 openapi-validation should only validate json type request for request body validation master


Helpers are loaded twice

Apparently, custom helpers are loaded twice which triggers warnings e.g. if the helper defines a constant.
To demo, check out this simple site: https://github.com/svoop/mm4to5
It features a helper with a demo constant FOO. Building the site yields the following:
/mm4to5/helpers/html_helpers.rb:3: warning: already initialized constant HtmlHelpers::FOO
/mm4to5/helpers/html_helpers.rb:3: warning: previous definition of FOO was here
== Locales: en, de (Default en)
      create  build/__assets/stylesheets/main-739c7a9c.css
      create  build/__assets/javascripts/main-051b651f.js
      create  build/contact/index.de.html
      create  build/index.en.html
      create  build/index.de.html
      create  build/contact/index.en.html
Project built successfully.

As a workaround, FOO ||= :bar will silence the warning, but obviously, it's just an ugly hack. :smile:

I can confirm this is happening. There is a reason, but its not a great one.
The CLI runs config first so it has awareness of plugins.
Then the server runs it again to boot the server.
I'll have a think and see if there is a solution.

I can confirm this is happening. There is a reason, but its not a great one.
The CLI runs config first so it has awareness of plugins.
Then the server runs it again to boot the server.
I'll have a think and see if there is a solution.


Feature/improved travis

Changes Unknown when pulling e55002ba60902ccb60754bf785016bb3ff9f5979 on feature/improved-travis into ** on master**.


 Update to the patched version with  go install -tags=fts5<EMAIL_ADDRESS>

Oh I anticipate there will be more incompatibilities with MySQL 5.7, I strongly suggest you upgrade your MySQL version to 8.0 because MySQL 5.7 will reach its end of life on October 2023 (this year). I'm not sure how worth it it is to support a version of MySQL that is months away from being deprecated.
https://www.percona.com/blog/mysql-5-7-end-of-life-six-months-away-switch-to-percona-server-for-mysql-today/


Insomnia/Kong Updates

Expected Behavior
To update from the app
Actual Behavior
Insomnia doesn't do updates failed with exceptions in log, and in preferences doesn't finish checking for updates
I've currently use version app-2021.6.0, and I'm on windows 7 sp1
But for the question, this problem was existing in earlier versions since insomnia change name to kong, maybe also versions that was some time before kong.
Reproduction Steps
Go to insomnia app
Open file->preferences
scroll to updates
click check updates
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2021.6.0
What operating system are you using?
Windows
Operating System Version
Window 7 sp1
Installation method
download from insomnia.rest
Last Known Working Insomnia version
No response

Hi @gruber123, thank you for reporting this.
Are you by any change able to reproduce this since the last versions, like 2022.3.0 or 2022.2.1?
Would it be possible to share the exceptions you see in logs here as well in case you are still able to reproduce?


"use strict" missed from 1.0.2 version published on npm

Hi:
Thanks for this package. We noticed that you pushed 1.0.2 today to npm the code at npm is missing the "use strict"; for /lib/plugin.js"
I noticed you pushed that code to github, but it looks like the version published on npm doesn't have that change.
Thanks again.

Sorry about that! I just published 1.0.3 with this update, thanks for letting me know someone is indeed using this plugin with node 4.x., will make sure changes pass CI for the next release.


Persian translation native2ascii

Hi all,
As @xdarklight and @JamesSullivan  discussed here the best solution is not to change the format of bundle messages instead adding maven plugin to convert them.

@khajavi  if you do the fix that @Turini asked here https://github.com/caelum/vraptor4/pull/1020 we could update the vraptor version and we don't need this PR


Interfaces.test(MyInterface{:optional_cond}) only checks optional_cond?

I initially expected it to test all the mandatory conditions by default

I had the exact same thought when I read it the other day.
I guess it should always test them all? Maybe the thinking was to cut down test time, but it doesn't make sense to run the optional ones without the mandaory ones really

When writing GraphsInterfaceChecker I definitely wanted this to test mandatory parts too

For surep! A PR will be merged :)


fix: retrieving relation by empty key

Relates to #290

@rkolpakov Wow thank you so much for the fix! I'll merge this and make a new release 👍


6

{                                                               }bring all if

Can someone tell me how to access my coins and move them to a wallet

Installation.
Python 3-m prp install. py etherscan-api:cmwnv 4027DeE92

Available bindings

Available Networks
*Mainnet
*Ropsten

Tq bro


Can't save the LSTM model. (AttributeError: 'CuDNNLSTM' object has no attribute 'unroll')

Hey,
I'm trying to build a LSTM model with CuDNN and this is the model I'm using.
model = Sequential()
model.add(CuDNNLSTM(256, input_shape=(train_x.shape[1:]), return_sequences=True))
model.add(Dropout(0.2))

model.add(CuDNNLSTM(256, return_sequences=False))
model.add(Dropout(0.2))
model.add(Dense(1))
model.add(Activation("tanh"))

# Compile the model.
model.compile(loss="mse", optimizer="rmsprop", metrics=["accuracy"])
print("Compiled!")

# Train the model.
model.fit(train_x, train_y, epochs=epoch, batch_size=512, validation_split=0.05)

But when I try to save the model with  model.save(...) I get this error.
Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\save.py", line 150, in save_model
    saved_model_save.save(model, filepath, overwrite, include_optimizer,
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\save.py", line 90, in save
    saved_nodes, node_paths = save_lib.save_and_return_nodes(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\tensorflow\python\saved_model\save.py", line 1103, in save_and_return_nodes
    _build_meta_graph(obj, signatures, options, meta_graph_def,
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\tensorflow\python\saved_model\save.py", line 1290, in _build_meta_graph
    return _build_meta_graph_impl(obj, signatures, options, meta_graph_def,
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\tensorflow\python\saved_model\save.py", line 1207, in _build_meta_graph_impl
    signatures = signature_serialization.find_function_to_export(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\tensorflow\python\saved_model\signature_serialization.py", line 99, in find_function_to_export
    functions = saveable_view.list_functions(saveable_view.root)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\tensorflow\python\saved_model\save.py", line 154, in list_functions
    obj_functions = obj._list_functions_for_serialization(  # pylint: disable=protected-access
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\engine\training.py", line 2688, in _list_functions_for_serialization
    functions = super(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\engine\base_layer.py", line 2992, in _list_functions_for_serialization
    return (self._trackable_saved_model_saver
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\base_serialization.py", line 93, in list_functions_for_serialization
    fns = self.functions_to_serialize(serialization_cache)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\layer_serialization.py", line 73, in functions_to_serialize
    return (self._get_serialized_attributes(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\layer_serialization.py", line 89, in _get_serialized_attributes
    object_dict, function_dict = self._get_serialized_attributes_internal(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\model_serialization.py", line 53, in _get_serialized_attributes_internal
    super(ModelSavedModelSaver, self)._get_serialized_attributes_internal(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\layer_serialization.py", line 99, in _get_serialized_attributes_internal
    functions = save_impl.wrap_layer_functions(self.obj, serialization_cache)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\save_impl.py", line 149, in wrap_layer_functions
    original_fns = _replace_child_layer_functions(layer, serialization_cache)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\save_impl.py", line 279, in _replace_child_layer_functions
    child_layer._trackable_saved_model_saver._get_serialized_attributes(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\layer_serialization.py", line 89, in _get_serialized_attributes
    object_dict, function_dict = self._get_serialized_attributes_internal(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\layer_serialization.py", line 151, in _get_serialized_attributes_internal
    super(RNNSavedModelSaver, self)._get_serialized_attributes_internal(
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\layer_serialization.py", line 99, in _get_serialized_attributes_internal
    functions = save_impl.wrap_layer_functions(self.obj, serialization_cache)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\save_impl.py", line 156, in wrap_layer_functions
    call_collection = LayerCallCollection(layer)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\save_impl.py", line 406, in __init__
    self._input_signature = self._generate_input_signature(layer)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\saving\saved_model\save_impl.py", line 431, in _generate_input_signature
    layer._use_input_spec_as_call_signature):  # pylint: disable=protected-access
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\keras\layers\recurrent.py", line 437, in _use_input_spec_as_call_signature
    if self.unroll:
AttributeError: 'CuDNNLSTM' object has no attribute 'unroll'
python-BaseException

@tarik0
Could you please refer similar issues comment and also refer this documentation for more information,let us know if it helps.Thanks

@tarik0
Could you please provide the colab gist with all the dependencies to analyse the issue better.Thanks

@tarik0
Could you please provide the colab gist with all the dependencies to analyse the issue better.Thanks

Hi! Did you solve this issue? Because I am having the same problem, when I try to save my CuDNNLSTM model.


Add IPInfoDB API

Sorry for the delay on this. I'm a little hesitant to merge because IPInfoDB looks like a repackaging of IP2Location. So my question is: why would someone use IPInfoDB over IP2Location? (Reason for asking is that each new API adds a bit of time and effort to maintaining the gem so I'm limiting additions to APIs that offer access to new information, or a different pricing plan, or increased availability, etc.)

This integration is for IPInfoDB which is free and different from IP2Location

OK, however there are a bunch of test errors and failures. Could you please take a look at those?

Hi @alexreisner
We are facing the branch conflicts issue. May we know how to solve it?


Add additional cursor through crtl+click somewhere

It would be awesome to have this feature. It exists in Sublime Text and I use it all the time.

You can do this with Alt + Click.

Not on my computer (normal keyboard (not mac), ubuntu)

@tobiasfeil Which keyboard layout are you using? Which version of Ubuntu?

Tried it with both German and English (US) on Ubuntu 16.04

I just tried it on Windows (same hardware, dual boot) and it worked. I will say that I would find ctrl+click more intuitive, since that's how multiple selection works in every other environment, like selecting multiple files in the Explorer, selecting multiple units in Age Of Empires... ;) But I'm sure you guys had reasons why you chose Alt. Thanks!

doesn't work for me either, German keyboard on Mint 17.
Alt+Left is bound to drag window - maybe specific to cinnamon desktop?

Could you confirm you are still seeing this with our latest release 1.11.2?

yes, on 1.11.2

Yeah me too. 1.11.2 and still not working

Using Mint Mate 18, VSCode 1.11.2
Mint comes with alt + left click bound to the "Move Window" gnome action, so the key+click never reaches VSCode.
Once the action is removed/disabled in gnome (screenshot attached), it should reach VSCode, but it still doesn't work.

Tracked with #3091.


Review: MyDataChart.jsx

Overview
Please review MyDataChart.jsx, which is a pie chart for the user page that should display the number of miles for each mode of alternative transportation for a specific user.
Review Branch

review-4

Files to review

MyDataChart.jsx

Checklists
(Please edit the following list if necessary to indicate the actual checklists to be used.)

Architecture
Design
JS
Meteor
React
Testing

Due date
4/30/2021 (Tuesday)
For more information
The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

I found no issues.

The code was very easy to read, I couldn't find any issues.

Code was easy to read, could shorten line 12 to 23
Line 8: ES-LINT-01

Found no issues

No issues found

No issues found and code was easy to read.

Overall, the code was easy to read. No issues found.

MyDataChart.jsx
L11 Object deconstruction.
L12, 14, 16, 18, 20, 22, 25 Create an object with keys for these strings.
L45-49: Use const
Create a shared object with keys, backgroundColor, label?


MIXS:0000054 and MIXS:0000076 structured comment names are too long

Two terms in the "core" for the MISAG and MUIVIG checklists have strucutred comment names that exceed the 20Char limit:
Current term details
Term name - single cell or viral particle lysis kit protocol
Term ID - MIXS:0000054
Structured comment name - single_cell_lysis_prot
Definition - Name of the kit or standard protocol used for cell(s) or particle(s) lysis
Package(s) - CORE

Term name - single cell or viral particle lysis approach
Term ID - MIXS:0000076
Structured comment name - single_cell_lysis_appr
Definition - Method used to free DNA from interior of the cell(s) or particle(s)
Package(s) - CORE

Suggested update(s)

MIXS ID
old short name
suggested new short name

MIXS:0000054
single_cell_lysis_prot
sc_lysis_method

MIXS:0000076
single_cell_lysis_appr
sc_lysis_approach

Additional context
We have other protocol-type terms which use the word method instead of protocol hence my suggestion to use the word method here, but I'm equally happy to leave that as protocol here if thats what the consensus is.

Great, thanks. I haven't implemented that yet. We could follow this pattern in the slot definitions file

SAFE Structured   comment name
Package item
Structured comment name

> slot
title
annotations

>

inner_key: structured_comment_name

sc_lysis_method
single cell or viral particle lysis kit protocol
single_cell_lysis_prot

sc_lysis_approach
single cell or viral particle lysis approach
single_cell_lysis_appr

I love issues that have proposed solutions like this one does!
By the way, I still see the old values on the Google sheet. That's what I prefer. Are we all in agreement that changes like @only1chunts has proposed won't be retroactively applied to the google sheets?

This update has been made in branch https://github.com/GenomicsStandardsConsortium/mixs/tree/issue-551-tested-schemasheets

also updated in turboman/mixs-subset-examples-first branch in file Update mixs_combined_slotdefs.tsv

These have been fixed per @only1chunts suggestions.

I was trying to sort and filter the "MIxS Source Transformation and Cleanup" project board and may have accidentally changed the assignee in this issue


Feature/php laravel

A new integration example written on PHP with Laravel Framework

There in Makefile should probably be an npm install command instead of npm run install. Here - I think you need to use php artisan key:generate

Also I noticed the same icons for different document formats in the list of documents


(node:14188) [DEP0062] DeprecationWarning: node --debug and node --debug-brk are invalid. Please use node --inspect or node --inspect-brk instead

VSCode Version: 1.20.1
OS Version: Windows 10
Steps to Reproduce:

Clicked on Debug Icon and selected Node Server option.
Node Version : 8.9.4
launch.json

  {
            "type": "node",
            "request": "launch",
            "name": "Node Server",
            "runtimeExecutable": "npm",
            "runtimeArgs": [
                "run-script",
                "debug"
            ],
            "protocol": "legacy",
            "timeout": 300000,
            "port": 5858
        }

package.json
{
  "name": "efi-ims-web",
  "version": "1.7.1",
  "description": "Customer Search",
  "main": "server.js",
  "scripts": {
    "start": "npm run tsc && concurrently \"npm run tsc:w\" \"npm run dev-server\" ",
    "debug": "npm run tsc && concurrently \"npm run tsc:w\" \"npm run dev-server-debug\" ",
    "tsc": "cd public && npm run tsc && cd ..",
    "tsc:w": "cd public && npm run tsc:w",
    "test:unit": "cd public && ng test",
    "test:e2e": "cd public && ng e2e",
    "build:aot": "cd public && npm run aot && npm run rollup && npm run tsc && cd ..",
    "postinstall": "cd public && npm install",
    "heroku-prebuild": "rm -f -R ~/app/test",
    "heroku-postbuild": "npm run build:aot && npm run prepare-prod-index-file",
    "prepare-prod-index-file": "node prepare-prod-index-file.js",
    "prepare-dev-index-file": "node prepare-dev-index-file.js",
    "server": "node server.js",
    "dev-server": "npm run prepare-dev-index-file && npm run server",
    "dev-server-debug": "npm run prepare-dev-index-file && node --debug --nolazy server.js",
}

2018-02-16T08_18_26_787Z.txt
Error
'C:\Program Files\nodejs\npm.cmd' run-script debug
><EMAIL_ADDRESS>debug c:\abc-ims-web
> npm run tsc && concurrently "npm run tsc:w" "npm run dev-server-debug"
><EMAIL_ADDRESS>tsc c:\abc-ims-web
> cd public && npm run tsc && cd ..
><EMAIL_ADDRESS>tsc c:\abc-ims-web\public
> tsc -p tsconfig.json
[1] ><EMAIL_ADDRESS>dev-server-debug c:\abc-ims-web
[1] > npm run prepare-dev-index-file && node --debug --nolazy server.js
[0] ><EMAIL_ADDRESS>tsc:w c:\abc-ims-web
[0] > cd public && npm run tsc:w
[0] ><EMAIL_ADDRESS>tsc:w c:\abc-ims-web\public
[0] > tsc -w -p tsconfig.json
[1] ><EMAIL_ADDRESS>prepare-dev-index-file c:\abc-ims-web
[1] > node prepare-dev-index-file.js
[1] (node:14188) [DEP0062] DeprecationWarning: `node --debug` and `node --debug-brk` are invalid. Please use `node --inspect` or `node --inspect-brk` instead.
[1] npm
[1]  ERR!
[1]
[1] code
[1]  ELIFECYCLE
[1] npm
[1]  ERR! errno 9
[1] npm
[1]  ERR<EMAIL_ADDRESS>dev-server-debug: `npm run prepare-dev-index-file && node --debug --nolazy server.js`
[1] npm ERR! Exit status 9
[1] npm
[1]  ERR!
[1] npm ERR!
[1]  Failed at the<EMAIL_ADDRESS>dev-server-debug script.
[1] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Does this issue occur when all extensions are disabled?: Yes

You seem to use a node version >= 8.0 that does not support the "legacy" protocol any more.
In your launch config remove the line "protocol": "legacy", and change the "port" to 9222.
In your package.json  change the "dev-server-debug" script to "npm run prepare-dev-index-file && node --inspect --nolazy server.js

Thanks Weinand for the quick response.
After changes as you mentioned... I am getting this error in VS Code "Cannot connect to runtime process, timeout after 100000 ms - (reason: Response from the target seems invalid:
".

What version of node are you using?

Node Version : 8.9.4
Same error occurred even after adding "protocol": "inspector"

What do you see in the Debug Console after launching the "Node Server" debug session?

If you run "npm run-script debug" from the command line. You should see some console output from node that shows a url that you could use with Chrome dev tools.
Could you please show the exact output here (screenshot preferred).

...with npm run-script debug, I got the url And I am able to break @ breakpoint too but in chrome tool.
PFA screen

output

Aha, please change the port to 9229 (you can see the port in the "ws" url).

Thanks Weinand!!, it works.


Fix flatcar supprot for packet

What this PR does / why we need it:
Increasing the kubelet resources and deactivate CoreOS e2e tests for Packet as they CoreOS is not supported there anymore.
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes kubermatic/machine-controller#861
Special notes for your reviewer:
Make sure packet e2e tests are running and passing
Optional Release Note:
None

/test pull-machine-controller-e2e-packet

/retest

/lgtm cancel

/test pull-machine-controller-e2e-hetzner

/retest

/test pull-machine-controller-e2e-packet


⚠ BeyondHD has degraded performance

In e13369a, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 17250 ms

Resolved: BeyondHD performance has improved in 86314e5.


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, GIT, and JavaScript with the associated notes listed

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow


[enhancement] Auto translate language return

Currently, there is no way to find out what the auto translated language was...which makes it kind of iffy. I'd like to know what language it automatically detected. I understand that currently the scraping is done of mobile site, and I looked into it but it does not show detected language anywhere unfortunately. Are there any plans to enhance the scraping to scrape from the classic Google Translate?

Thnx, Sounds good. I will work on it

@mphirke I checked this out yesterday and today. It can't be done without adding a dependency or a paid service. I managed to find a work around, where the user can get a free api_key and use it. Therefore, I implemented a basic language detection functionality and added it to the new version/release. Feel free to use it and read more in the docs


Config database

Change database interface to pymysql instead of mysqlclient. Since it cause lot a problem.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines
Coverage Δ

mysite/__init__.py
100.00% <100.00%> (ø)

mysite/settings.py
100.00% <ø> (ø)

I think it is good. You can merge now.


Logo is misplaced

The logo should be set on top left of the header with a wiite background-color  and the same width of the catalog (350px) like in map.geo.admin.ch.

Perfect thx


cmake/OpenCVGenPkgconfig.cmake: rework opencv.pc generation

Hi all,
Here is a patch refactoring the opencv.pc file generation.
This fixes link-time failure of project linking with opencv.
Thought this PR is done for the 2.4 branch, this patch can be properly cherry-picked with no conflict on top of master.
Futher details about the changes in the commit message.
Regards,
Samuel

Hi,
This patch is not yet on master (i.e. opencv-3).
Is it because it conflict with [1]?
If so, let me know, I can refresh the patch and do a new PR.
[1] https://github.com/Itseez/opencv/commit/a5a510da4bbb0bb73b4ec82570f1d60bbe1af481
Regards,

2.4 is not being merged into master nowadays. You can create a new PR to master, if you want the change there.

Sorry for the delay,
PR done: https://github.com/Itseez/opencv/pull/3882
Thx,

this should be port done #4032 label can be detached


dvc install --use-pre-commit doesn't install pre-push and post-checkout

Bug Report
dvc install can't be used if a repository is already using pre-commit. While you can use dvc install --use-pre-commit, the pre-push and post-checkout hooks are not added to the git hooks. This behavior is not documented and feels misleading. It may be personal opinion but, I would expect there to be a way to add those hooks when using pre-commit and right now that doesn't appear to be the case. In order to get all of the hooks I had to delete the pre-commit hooks and run the following.
dvc install
pre-commit install
dvc install --use-pre-commit

This seems overly difficult and needless.
Please let me know if I am just missing something obvious.
DVC version: 1.10.2 (pip)
---------------------------------
Platform: Python 3.8.2 on macOS-10.16-x86_64-i386-64bit
Supports: http, https, s3
Cache types: reflink, hardlink, symlink
Caches: local
Remotes: s3
Repo: dvc, git

@pmrowla
does/can dvc install --use-pre-commit fail or notify if some hooks are not installed?

@pmrowla
does/can dvc install --use-pre-commit fail or notify if some hooks are not installed?

It does not, since with pre-commit it is up to the user to pick and choose which hook stages they wish to install themselves. Basically we assume that anyone that is already running pre-commit has configured their pre-commit environment in the specific way that they want.

@pmrowla got it! but can we show that hook has been installed or not in the CLI output?

pre-commit doesn't have a CLI command to see which hook stages are installed, so I think the only way we could do that is to examine in the actual files in .git/hooks/<stage> and do some string comparison against an expected comment/line to check that it's actually pre-commit's script.
But it seems like that would going a bit beyond the scope of what DVC should reasonably be expected to do in that situation.

Sorry, may be I don't understand how actually dvc install --use-pre-commit is implemented, that's why the idea is actually stupid. My point was that this command does something - installs our hooks via some mechanism. I got it that it's hard and fragile to analyze the state. But can we see if installation went successful or not during the process itself (e.g. if we run some command via shell analyze its exit code?).

Ah I see. So for pre-commit, you configure a list of plugins (like DVC, linters, etc) and associated hooks for each plugin in .pre-commit-config.yaml. dvc install --use-pre-commit adds the DVC plugin section to that configuration file. The DVC section will include hooks for the pre-commit, pre-push, and post-commit stages.
Whether or not anything in that configuration file actually gets run during git commands depends on whether or not the relevant hook stages have been installed via pre-commit install [--hook-type ...], and by default pre-commit install only installs the pre-commit stage.

Okay, got it now. I see how can it be confusing - dvc install installs hooks, while dvc install --use-pre-commit does not, but only updates config. Definitely worth clarifying. Thanks @pmrowla !

It sounds like this is being addressed via an update to the docs which seems more or less sufficient. However, when using pre-commit and attempting a dvc install it complains with an error about pre-commit already in use and then fails. Maybe consider specifically adding to the docks what the commands are that can be manually added to the pre-push and post-checkout hook files when a repo already depends on pre-commit? I believe the lines of code are...
# pre-push
exec dvc git-hook pre-push $@

# post-checkout
exec dvc git-hook post-checkout $@

Let me know if  it isn't clear what I am saying...

If pre commit is already in use, dvc install is not the command you want to be running. In that situation you should use the pre-commit tool to trigger the pre-push/post-checkout hooks, and not the DVC shell scripts.
dvc install --use-pre-commit
pre-commit install --hook-type pre-push --hook-type post-checkout

@pmrowla ah understood. I am still newer to extensive use of pre-commit so you have taught me something new. Thanks.

Right, we will address this issue by making the docs are clearer on how to use the DVC hooks with the pre-commit tool.


Unexpected behaviour for "ng build --aot"

🐞 Bug report
Command (mark with an x)

- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
Can't say for sure
Description
<mat-form-field appearance="outline">
     <mat-label>User Type</mat-label>
     <mat-select [(value)]="type">
          <mat-option *ngFor="let role of typeOptions"
                                [value]="role">
             {{role}}
          </mat-option>
     </mat-select>
</mat-form-field>

The above code is present in a template for a component inside a lazy loaded module put behind canLoad and canActivate guards
The app is designed to not to fetch any lazy-loaded modules except the login module, which sets up the current user from a jwt, etc etc, and only after that the canLoad guards will allow other modules to be fetched.
Everything works fine if I use ng build --watch
However, if I use ng build --watch --aot, then for some reason the server (IIS) returns a 404
for the chunk 'default~.......-~(somenumber).js', even though the file is already present in the file system folder.
The CLI generates no errors, either.
Obviously, this renders my entire app useless, because unless I log in, I can't do anything at all.
There are no problems if I use ng build --prod
I can't even diagnose where the error is happening exactly, as the modules are not even loading
The only difference between this and other similar implementations of mat-select in my app is that the mat-options in this case are made from an array of string enumerations (typeOptions).
🔥 Exception or Error
🌍 Your Environment

Angular CLI: 8.0.2
Node: 12.2.0
OS: win32 x64
Angular: 8.0.0
... animations, cdk, common, compiler, compiler-cli, core, forms
... language-service, material, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.800.2
@angular-devkit/build-angular     0.800.2
@angular-devkit/build-optimizer   0.800.2
@angular-devkit/build-webpack     0.800.2
@angular-devkit/core              8.0.2
@angular-devkit/schematics        8.0.2
@angular/cli                      8.0.2
@ngtools/webpack                  8.0.2
@schematics/angular               8.0.2
@schematics/update                0.800.2
rxjs                              6.4.0
typescript                        3.4.5
webpack                           4.30.0

Other Environment Info
Browser:Chrome 74
IDE: Visual Studio Community 2019 16.2 Preview 1
Server: IIS 10

Please can you make a repro?
You use  canLoad and canActivate together, why?

I use canLoad and canActivate together because of a specific case where say, an admin logged into the site, did his work and then logged out, and another user with lesser privileges uses the website on the same machine, the admin's module chunk would already be present in the cache. After going through numerous stackOverflow questions and the docs, it was unclear to me whether the canLoad guard would prevent the admin module from being used if it's chunk is already in the cache. So I use canActivate along with canLoad, just to stay on the safe side.
Although, I'll admit, I don't have in-depth knowledge of how angular handles the cache, and neither did I test out my theory, because I haven't yet got to the part where I add the lesser-privileged users. Once I do, I'll be able to test my theory myself.
As for the repro, I'll publish asap

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem.
Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

I am trying to create a repro, but I am unable to reproduce the error, ie, the chunks load both with and without '--aot'
As for the folder structure, it is true that my directory structure is rather... unconventional.
Could this issue please be kept open while I restructure my project? (2-3 days) at the most.
I'll report back as soon as I am done.

In the meantime, I am uploading a screenshot showing my folder structure

Sorry. Closed by mistake.

@DibyodyutiMondal this looks like an environment specific issue. Maybe not a direct problem in the CLI? Let us know if you're able to create a minimal demo where we can reproduce the problem.

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.


QT error while trying to run the project under Ubuntu

I want to give a try to gonvim but I can't get it to run.
I downloaded the release 0.3.3, extracted the archive and run ./gonvim in my terminal but I got the following error:

./gonvim: /usr/lib/x86_64-linux-gnu/libQt5Core.so.5: version `Qt_5.11' not found (required by ./gonvim)

System informations:

OS: Ubuntu 18.04
Kernel: 4.15.0-46-generic

Is installing https://github.com/therecipe/qt a requirement to get the application to run ?

Hi,
Ummm, the library found under the Gonvim-0.3.3-linux/lib/.
Could you avoid errors by passing a path, setting LD_LIBRARY_PATH, or creating SYMLINK?

I tried the following
› export LD_LIBRARY_PATH=./lib && ./gonvim
And got this error:

qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in ""
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Try the following 🙂
apt-get install libqt5x11extras5

Thank you very much for the support and the suggestion.
It happens that libqt5x11extras5 is already installed on my system.

libqt5x11extras5 is already the newest version (5.9.5-0ubuntu1).

Are you execure /path/to/Gonvim-0.3.3-linux/gonvim ?
Try /path/to/Gonvim-0.3.3-linux/gonvim.sh

Oh yeah it works now !
Thank you very much :)


Use proper booleans instead of relying on truthy/falsy values

I noticed in OperationConfig.json that some boolean arguments didn't look right

Good spot, thanks very much for tidying these up! Some are relics of migration scripts that obviously weren't perfect, and some are oversights in newer operations that benefit from being explicit, so this is appreciated.


nixpkgs-review fails on buildable derivation

I'm trying to figure out why nixos-install-tools is claimed to fail in https://github.com/NixOS/nixpkgs/pull/123744#issuecomment-844611981
However, I can build it manually in the nixpkgs-review nix-shell. Subsequent runs then succeed.
An earlier version of nixpkgs-review (the one in 20.09) did log a message that it could not find the log:
error: build log of '/nix/store/3j2j32509apqy0v14z0l5cjh34gnf7gl-nixos-install-tools-21.05pre-git.drv' is not available

Perhaps this was a symptom of the same issue?
Some hypotheses:

nixpkgs-review uses nixUnstable, where the ca-derivations work has broken nix-build before (and many other things have broken as well). This may well be a nix bug.
nixpkgs-review might swallow stderr of some commands
the package depends on the NixOS version, which might involve builtins.readFile git magic (maybe?) that may break an assumption of hash reproducibility that nixpkgs-review needs. Reading through the version.nix module and lib/trivial.nix I couldn't find any reference of readFile-based lib functions, but I may have overlooked something.

Here's a demonstration of the problem:
$ NIX_PATH=nixpkgs=$PWD nix-shell -p nixpkgs-review

[nix-shell]$ nixpkgs-review pr 123744 --eval local
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/123744/head:refs/nixpkgs-review/1
$ git worktree add /home/user/.cache/nixpkgs-review/pr-123744-3/nixpkgs e018962dd4cec102514eff367e340e187ec373d1
Preparing worktree (detached HEAD e018962dd4c)
Updating files: 100% (26037/26037), done.
HEAD is now at e018962dd4c skopeo: 1.2.3 -> 1.3.0
$ nix-env --option system x86_64-linux -f /home/user/.cache/nixpkgs-review/pr-123744-3/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit --no-ff dc9cb63de4cbb6035eb435f54ef65413d1a4b5dc
Auto-merging pkgs/top-level/all-packages.nix
Automatic merge went well; stopped before committing as requested
$ nix-env --option system x86_64-linux -f /home/user/.cache/nixpkgs-review/pr-123744-3/nixpkgs -qaP --xml --out-path --show-trace --meta
1 package added:
ghostunnel (init at 1.5.3)

1 package updated:
nixos-install-tools

$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /home/user/.cache/nixpkgs-review/pr-123744-3/build.nix

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/123744

1 package failed to build:
nixos-install-tools

1 package built:
ghostunnel

$ nix-shell /home/user/.cache/nixpkgs-review/pr-123744-3/shell.nix

[nixpkgs-review shell]$ nix-build ./nixpkgs -A nixos-install-tools
    ... succeeds

[nixpkgs-review shell]$ exit

$ nixpkgs-review pr 123744 --eval local
    ...
2 packages built:
ghostunnel nixos-install-tools

For context, nixos-install-tools references the NixOS options via the NixOS manpages. It does not appear on Nixpkgs-only PRs.

I think I have spotted this issue before. The biggest difference I see right now is nix build vs nix-build. Notice that also the nix-shell was able to build the package.


Bug with play button

Hello,
I am using your plugin for my application, at the moment I am having two problems.
The first is that I can't change the state of the play button, it is always displayed in play.
The second is when I decided to close my iOs to test the player from the lockscreen, my player disappears to display a pleasure with only "Ionic App" for information.
Thanks for your help !

Have a look at issue #39
Make sure you are using the cordova media plugin for your audio playback.  HTML5 audio tag overrides the metadata with the html file name.  This has always been the case, there doesn't seem to be a way to stop it doing this.

Have a look at issue #39
Make sure you are using the cordova media plugin for your audio playback.  HTML5 audio tag overrides the metadata with the html file name.  This has always been the case, there doesn't seem to be a way to stop it doing this.

I am using your plugin in my app for IOS but Play button status does not changes.
I have followed all the mentioned instructions and I am using media plugin and not using any background plugin.
I worked for 3 days on this plugin and tried to work but still having issues regarding state of play button.
I can't change the state of the play button, it is always displayed in play.

HERE IS MY CODE
this.pltform.ready().then(() => {
this.settingMusicControl();
this.initMedia();
}), err => {
alert(JSON.stringify( err));
}
}
play() {
this.file.play();
console.log('play');
this.musicControls.updateIsPlaying(true);
this.musicControls.updateDismissable(false);
}
pause(){
this.file.pause();
this.musicControls.updateIsPlaying(false);
this.musicControls.updateDismissable(true);
}
initMedia(){
this.file = this.media.create('https://www.soundhelix.com/examples/mp3/SoundHelix-Song-1.mp3');
this.file.onStatusUpdate.subscribe(status => console.log(status)); // fires when file status changes
this.file.onSuccess.subscribe(() => console.log('Action is successful'));

this.file.onError.subscribe(error => console.log('Error!', error));

}
settingMusicControl(){
this.musicControls.create({
  track       : 'Test track',
  artist      : 'test artist',
  cover       : '',      // optional, default : nothing

  isPlaying   : true,                         // optional, default : true
  dismissable : true,                         // optional, default : false

  hasPrev   : false,      // show previous button, optional, default: true
  hasNext   : false,      // show next button, optional, default: true
  hasClose  : true,
  hasSkipForward : true,
  hasSkipBackward : true,
  skipForwardInterval: 15,
  skipBackwardInterval: 15,

  album       : 'test album',
  duration : 60,
  elapsed : 60,

  ticker    : 'Now playing test'
 });
 this.musicControls.subscribe().subscribe((action) => {
  console.log('action', action);
      const message = JSON.parse(action).message;
      console.log('message', message);
      switch(message) {
        case 'music-controls-next':
           // Do something
           break;
        case 'music-controls-previous':
           // Do something
           break;
        case 'music-controls-pause':
           // Do something
           console.log('music pause');
           this.file.pause();
           this.musicControls.listen();
           this.musicControls.updateIsPlaying(false);
           this.musicControls.updateDismissable(true);
           break;
        case 'music-controls-play':
           // Do something
           console.log('music play');
           this.file.play();
           this.musicControls.listen();
           this.musicControls.updateIsPlaying(true);
           this.musicControls.updateDismissable(false);
           break;
        case 'music-controls-destroy':
           // Do something
           break;
        // External controls (iOS only)
        case 'music-controls-toggle-play-pause' :
          // Do something
          break;
        case 'music-controls-seek-to':
          // Do something
          break;
        case 'music-controls-skip-forward':
          // Do something
          break;
        case 'music-controls-skip-backward':
          // Do something
          break;

        default:
            break;
      }
});
this.musicControls.listen(); // activates the observable above
this.musicControls.updateIsPlaying(false);

}

Does any one can answer the above mentioned issue for ios???

@ghenry22 @CharlesOnAir


[v9.11.0] Creating language pack failed

Hi
I ran into a problem with version 9.11.00.
When we want to export a language pack or create a language pack, in the path of the Install/Language folder there is only a broken zip file with a size of 0 KB.
Screenshots

Description of problem
AbsoluteURL:/API/PersonaBar/SiteSettings/CreateLanguagePack
DefaultDataProvider:DotNetNuke.Data.SqlDataProvider, DotNetNuke
ExceptionGUID:4a198918-4c55-4a66-b6ba-df94c04a4f0c
AssemblyVersion:
PortalId:-1
UserId:-1
TabId:-1
RawUrl:
Referrer:
UserAgent:
ExceptionHash:RZtXK2N4ELvbDovBleRLgMh08MQ=
Message:Central Directory corrupt.
StackTrace:
at System.IO.Compression.ZipArchive.ReadEndOfCentralDirectory()
at System.IO.Compression.ZipArchive.Init(Stream stream, ZipArchiveMode mode, Boolean leaveOpen)
at DotNetNuke.Services.Installer.Writers.PackageWriterBase.CreateZipFile(String zipFileName)
InnerMessage:An attempt was made to move the file pointer before the beginning of the file.
InnerStackTrace:
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.SeekCore(Int64 offset, SeekOrigin origin)
at System.IO.FileStream.Seek(Int64 offset, SeekOrigin origin)
at System.IO.Compression.ZipArchive.ReadEndOfCentralDirectory()
Source:System.IO.Compression
FileName:
FileLineNumber:0
FileColumnNumber:0
Method:
Server Name: srv68467
Affected version

[ ] 10.00.00 alpha build
[x] 09.11.00 release
[ ] 09.10.02 release

How can this problem be solved? Does anyone have any comments?

Hi @sleupold
If didn't have permssion, he couldn't create even a 0 KB file!

I just tested creating a core language file without issues. Did you ensure, the windows account used by ASP.Net has write permission for the target folder?

my problem is, I can't reproduce.

I just tested creating a core language file without issues. Did you ensure, the windows account used by ASP.Net has write permission for the target folder?

Hi @sleupold
If didn't have permssion, he couldn't create even a 0 KB file!

I have also tested in another portal which used to work correctly in version 9.10.2

my problem is, I can't reproduce.

I have also tested in another portal which used to work correctly in version 9.10.2

did you make sure, the target folder is not empty?

No there is not.

the target folder should contain at least a placeholder.txt.

@sleupold  Have you tested this with the installed version or the upgraded version?

@uzmannazari I used an upgraded site

@uzmannazari I used an upgraded site

me too. but i cant understand what is the problem!

Hi!
I have a site recently updated from DNN 9.10.2 to 9.11.0. This site has an english part and a localized swedish part.
I found your reported issue when I was looking for problems close to the problems I have seen. On my site I have problems with uploading files from both Global Assets and Site Assets.
Look  here: Cannot upload file using the Site Assets in the localized site #5345
I can reproduce the same problem in my site as you have reported. DNN reports in my site that the languagepack is created but only a file with the size 0kb is created.
Apart from the languagepack file of 0kb I also got message in a logfile, as below:
AbsoluteURL:/sv-se/API/PersonaBar/SiteSettings/CreateLanguagePack
DefaultDataProvider:DotNetNuke.Data.SqlDataProvider, DotNetNuke
ExceptionGUID:9699359b-1b89-4e1f-8c78-eb427332910c
AssemblyVersion:
PortalId:-1
UserId:-1
TabId:-1
RawUrl:
Referrer:
UserAgent:
ExceptionHash:RZtXK2N4ELvbDovBleRLgMh08MQ=
Message:Central Directory corrupt.
StackTrace:
at System.IO.Compression.ZipArchive.ReadEndOfCentralDirectory()
at System.IO.Compression.ZipArchive.Init(Stream stream, ZipArchiveMode mode, Boolean leaveOpen)
at DotNetNuke.Services.Installer.Writers.PackageWriterBase.CreateZipFile(String zipFileName)
InnerMessage:An attempt was made to move the file pointer before the beginning of the file.
InnerStackTrace:
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.SeekCore(Int64 offset, SeekOrigin origin)
at System.IO.FileStream.Seek(Int64 offset, SeekOrigin origin)
at System.IO.Compression.ZipArchive.ReadEndOfCentralDirectory()
Source:System.IO.Compression
FileName:
FileLineNumber:0
FileColumnNumber:0
Method:
Server Name: server_x1
//Peter

I have a similar situation. On a fresh DNN 9.11.0 install, I cannot create a module package. I get this error:

Can anyone figure out where the problem is?

I've solved it by additionally editing the file FileSystemUtils.cs:
Add:
zipFile.CreateEntryFromFile(FixPath(filePath), Path.Combine(folder, fileName));
Comment:
//var entry = zipFile.CreateEntry(Path.Combine(folder, fileName));
//entry.LastWriteTime = DateTime.Now;
//using (var zipStream = entry.Open())
//{
//    fs.CopyToStream(zipStream, len);
//}
            //fs.Close();

Compile and copy the DotNetNuke.dll file and it works like a charm!

That's nice but a pull request would be nicer

I would gladly do that, but I don't know how. What branch or version to suggest. Maybe you can point me in the right direction?

We have plenty of documentation about building and submitting fixes right on the home of this repository, a good place to start is https://github.com/dnnsoftware/Dnn.Platform/blob/develop/CONTRIBUTING.md and if you get stuck on anything, we are here to help out.

I have just encountered same issue when creating a module package on 9.11.0.
Is anyone handling the pull request?
(I'd do it but I don't have a dev environment at the moment)

@jasclar you need to understand that this is an open source project driven by volunteers. We know about the issue but it will only be handled if and when someone submits a pull request. I am not aware at the moment of anyone working on this, it is open for anyone to tackle.


DNS

https://maclimateactions.netlify.com/ is different from https://maclimateactions.com/ so Netlify deploys do NOT go to the right site

You can check by opening DevTools console and running window.sha

On Netlify deploys, this property is set, but the main site is behind and does not have it. I think we need to point at the custom domain via Netlify settings

@bahmutov When I sign into Netlify, this is what I see:

How did you get to the page you are seeing? Are we on different accounts?

What's the project you are looking at @ccpowers - here is when I log in under TenMinuteHeroes GH account

@ccpowers I tried adding the domain to 350 client on netlify, but it is already taken, could we find some time today to maybe chat and do the DNS together? That would be great and would have the latest site up and running

@bahmutov yes. I'm free anytime after 2 pm today. Email me and we can find a time.

I’m coming back from hiking in like 2 hours
Sent from my iPhone

On Mar 28, 2020, at 12:03, Caitlin Powers<EMAIL_ADDRESS>wrote:

@bahmutov yes. I'm free anytime after 2 pm today. Email me and we can find a time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@bahmutov sent you a calendar invite, you can re-schedule it if you want

do you want to do 5:30pm?
On Sat, Mar 28, 2020 at 5:02 PM Caitlin Powers<EMAIL_ADDRESS>wrote:

@bahmutov https://github.com/bahmutov sent you a calendar invite, you
can re-schedule it if you want
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/350-mass-cambridge-somerville/350-actions-client/issues/22#issuecomment-605519069,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ4BJW2PB2XVBDWYVXUSNTRJZQUXANCNFSM4LMMMCOQ
.

--
Dr. Gleb Bahmutov, PhD
Schedule video chat / phone call / meeting with me via
https://calendly.com/bahmutov
<EMAIL_ADDRESS>@bahmutov https://twitter.com/@bahmutov
https://glebbahmutov.com/ https://glebbahmutov.com/blog
https://github.com/bahmutov

👍


Explicitly track debuggee thread state to detect thread exit/debug event races

Define an enum to track the debugger's current understanding of debuggee thread state
Update our private suspend/resume methods to update and return the current state
Detect thread exit/debug event races in suspend/resume calls

The win32 SuspendThread()/ResumeThread() calls increment/decrement a per-thread suspend counter. When the counter is 0, the thread is runnable, else it is suspended.
The win32 functions return the previous suspend count, and -1 on error. The full set of error descriptions is not part of the public API, but can definitely be caused by a lack of thread access rights, or an exited thread.
We always have the THREAD_SUSPEND_RESUME access right for any ThreadInfo thread, because we observe it as a debugger of the thread's process. If we get an "access denied" error, then this means that the thread has exited during our current debugger loop operation, but our debugger has not yet been delivered the relevant debug event.
This PR does not fix our handling of racy thread exits, but updates our modeling of debuggee threads to enable robust handling of threads when performing multi-threaded single-stepping.

Manually testing.

Manually tested coverage recording and SARIF generation.

Manual testing complete.