Trying to install an npm module returns an error

Description
Thanks for this tool! I'm trying to implement a spellchecker inside nativefier. I added some code and the concept of it all works pretty good.
The problem starts when I try to require a module inside mainWindow.js. I tried including several libraries in two formats:
Require
import fs from 'fs';
import path from 'path';
import electron from 'electron';
import windowStateKeeper from 'electron-window-state';
import helpers from './../../helpers/helpers';
import createMenu from './../menu/menu';
import initContextMenu from './../contextMenu/contextMenu';

const {BrowserWindow, shell, ipcMain, dialog} = electron;
const {isOSX, linkIsInternal, getCssToInject} = helpers;

/* MY CODE */

var SpellChecker = require('simple-spellchecker');
var myDictionary = null;

import
import fs from 'fs';
import path from 'path';
import electron from 'electron';
import windowStateKeeper from 'electron-window-state';
import helpers from './../../helpers/helpers';
import createMenu from './../menu/menu';
import initContextMenu from './../contextMenu/contextMenu';
import SpellChecker from 'simple-spellchecker'; // MY CODE

const {BrowserWindow, shell, ipcMain, dialog} = electron;
const {isOSX, linkIsInternal, getCssToInject} = helpers;

/* MY CODE */

var myDictionary = null;

Both return an error when I try to build the app using npm run dev-up
Steps to Reproduce Issue

cd app/
npm install --save simple-spellchecker
import the file inside mainWindow.js
(inside nativefier main folder) run npm run dev-up

Specifications

Version of Nativefier (run $ nativefier --version): v6.13.0
Version of Node.js (run $ node --version): v4.2.3
OS: OS X El Capitan
Architecture x64
Stack trace from the error message (if any)

><EMAIL_ADDRESS>dev-up /Users/neilkalman/Development/redkix-nativefier/nativefier
> npm install && (cd app && npm install) && npm run build

npm WARN package.json Dependency 'async' exists in both dependencies and devDependencies, using<EMAIL_ADDRESS>from dependencies
npm WARN package.json Dependency 'lodash' exists in both dependencies and devDependencies, using<EMAIL_ADDRESS>from dependencies
npm WARN package.json Dependency 'shelljs' exists in both dependencies and devDependencies, using<EMAIL_ADDRESS>from dependencies
npm WARN package.json Dependency 'tmp' exists in both dependencies and devDependencies, using<EMAIL_ADDRESS>from dependencies
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No README data

><EMAIL_ADDRESS>build /Users/neilkalman/Development/redkix-nativefier/nativefier
> gulp build

[16:33:23] Requiring external module babel-register
[16:33:28] Using gulpfile ~/Development/redkix-nativefier/nativefier/gulpfile.babel.js
[16:33:28] Starting 'build'...
[16:33:28] Starting 'clean'...
[16:33:28] Finished 'clean' after 45 ms
[16:33:28] Starting 'build-cli'...
[16:33:28] Starting 'build-static-js'...
[16:33:28] Starting 'build-static-not-js'...
[16:33:28] Starting 'build-tests'...
[16:33:29] Finished 'build-static-not-js' after 683 ms
[16:33:29] Finished 'build-static-js' after 793 ms
[16:33:29] Starting 'build-static'...
[16:33:29] Finished 'build-static' after 17 μs
[16:33:29] Starting 'build-app'...
[16:33:29] Finished 'build-tests' after 1.29 s
[16:33:33] Version: webpack 1.13.1
      Asset     Size  Chunks             Chunk Names
    main.js  1.11 MB       0  [emitted]  main
main.js.map  1.33 MB       0  [emitted]  main

WARNING in ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs/fs.js
Critical dependencies:
11:12-19 require function is used in a way in which dependencies cannot be statically extracted
 @ ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs/fs.js 11:12-19

WARNING in ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs/package.json
Module parse failed: /Users/neilkalman/Development/redkix-nativefier/nativefier/app/node_modules/simple-spellchecker/node_modules/unzip/node_modules/fstream/node_modules/graceful-fs/package.json Unexpected token (2:10)
You may need an appropriate loader to handle this file type.
SyntaxError: Unexpected token (2:10)
    at Parser.pp.raise (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:923:13)
    at Parser.pp.unexpected (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1490:8)
    at Parser.pp.semicolon (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1469:73)
    at Parser.pp.parseExpressionStatement (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1994:8)
    at Parser.pp.parseStatement (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1772:188)
    at Parser.pp.parseBlock (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:2009:21)
    at Parser.pp.parseStatement (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1753:19)
    at Parser.pp.parseTopLevel (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1666:21)
    at Parser.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1632:17)
    at Object.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:885:44)
 @ ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs ^\.\/.*$

WARNING in ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs/LICENSE
Module parse failed: /Users/neilkalman/Development/redkix-nativefier/nativefier/app/node_modules/simple-spellchecker/node_modules/unzip/node_modules/fstream/node_modules/graceful-fs/LICENSE Unexpected token (1:4)
You may need an appropriate loader to handle this file type.
SyntaxError: Unexpected token (1:4)
    at Parser.pp.raise (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:923:13)
    at Parser.pp.unexpected (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1490:8)
    at Parser.pp.semicolon (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1469:73)
    at Parser.pp.parseExpressionStatement (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1994:8)
    at Parser.pp.parseStatement (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1772:188)
    at Parser.pp.parseTopLevel (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1666:21)
    at Parser.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1632:17)
    at Object.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:885:44)
    at Parser.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/lib/Parser.js:902:15)
    at DependenciesBlock.<anonymous> (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/lib/NormalModule.js:104:16)
 @ ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs ^\.\/.*$

WARNING in ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs/README.md
Module parse failed: /Users/neilkalman/Development/redkix-nativefier/nativefier/app/node_modules/simple-spellchecker/node_modules/unzip/node_modules/fstream/node_modules/graceful-fs/README.md Unexpected character '#' (1:0)
You may need an appropriate loader to handle this file type.
SyntaxError: Unexpected character '#' (1:0)
    at Parser.pp.raise (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:923:13)
    at Parser.pp.getTokenFromCode (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:2831:8)
    at Parser.pp.readToken (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:2526:15)
    at Parser.pp.nextToken (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:2518:71)
    at Parser.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:1631:10)
    at Object.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/acorn/dist/acorn.js:885:44)
    at Parser.parse (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/lib/Parser.js:902:15)
    at DependenciesBlock.<anonymous> (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/lib/NormalModule.js:104:16)
    at DependenciesBlock.onModuleBuild (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/webpack-core/lib/NormalModuleMixin.js:310:10)
    at nextLoader (/Users/neilkalman/Development/redkix-nativefier/nativefier/node_modules/webpack-stream/node_modules/webpack/node_modules/webpack-core/lib/NormalModuleMixin.js:275:25)
 @ ./app/~/simple-spellchecker/~/unzip/~/fstream/~/graceful-fs ^\.\/.*$

WARNING in ./app/~/tap/~/only-shallow/index.js
Module not found: Error: Cannot resolve module 'buffertools' in /Users/neilkalman/Development/redkix-nativefier/nativefier/app/node_modules/tap/node_modules/only-shallow
 @ ./app/~/tap/~/only-shallow/index.js 14:24-46

WARNING in ./app/~/tap/~/deeper/index.js
Module not found: Error: Cannot resolve module 'buffertools' in /Users/neilkalman/Development/redkix-nativefier/nativefier/app/node_modules/tap/node_modules/deeper
 @ ./app/~/tap/~/deeper/index.js 14:21-43
[16:33:33] Finished 'build-cli' after 4.79 s
[16:33:33] Finished 'build-app' after 3.98 s
[16:33:33] Finished 'build' after 4.85 s

I'm probably importing this wrong... I see require function is used in a way in which dependencies cannot be statically extracted.
But I didn't see anything in the development wiki to say how to import things correctly.
thanks in advance!

Hmm, I've looked into this and you're importing the module correctly!
This problem seems to because of the unzip module dependency used by simple-spellchecker, with something to do with how the require function is used and preventing webpack from being able to recursively bundle all the app dependencies. This is somewhat mentioned in dcodeIO/protobuf.js#339, and I believe one of the dependencies of simple-spellchecker is the source of this problem.
Without going into too much detail, webpack is not exactly necessary when building the app, but the reason why I used it was as a workaround because there are issues when I try to publish nativefier to npm with nested node_modules directories when I want to maintain the same folder structure, as follows.
node_modules <- contains dependencies required by the cli
app/node_modules # <- contains dependencies required for the placeholder app

From what I remember, npm consolidates all of these modules in the root node_modules directory, and hence I bundled the dependencies required by the app when publishing to npm as a pretty ugly workaround.
Having said that, it would probably be possible to use the same module you require, with some tweaking of the gulpfile to not use webpack to bundle the app. I could go into more detail regarding this if you wish though.

I would appreciate it if you can :-)
Thanks
If succeed, do you want me to open a pull request with this?

Oh, and I will also appreciate if you know some way to debug the electron part of the wrapper (when I open the deveolper tools of the nativefier app, I can only access the actual site + the preload.js script)

I was able to get something working. Now, I highlight words based on a dictionary (no suggestions yet). So, if a word isn't in the dictionary, it's marked as incorrect.
But I was unsuccessful in loading an actual dictionary. I removed unzip and tried to load an actual .dic file instead of a zip. inside a node.js script I was able to load the dictionary that way and everything worked.
But inside of nativefier, something went wrong and the dictionary wasn't loaded. I think it's because the code is being compiled and the relative path to the dictionary file should change. But that is just a guess.
Do you think it's a reasonable assumption? Any hints on why the file isn't found? I'm not sure I understand the gulp build script yet.

Actually is electron/electron#942 relevant to implementing spell check? I believe the support is documented here. It'll be great if we could use the native electron api for the spell check! If this works, you won't have to mess around with the gulp files and all the building :D

I've also encountered the problem where it is difficult to debug the main electron process when the app is packaged. There's a helper function you can use here, which directs the output to the chromium inspector

Cool! Thx for the hints :-)
I am using the function you send. It's the provider in that function that needs a dictionary file to check. But maybe the library they recommended works in a way that will allow that. I'll try it out
And by saying "I've also encountered the problem where it is difficult to debug the main electron process when the app is packaged", are you applying that there's a way to debug it uncompiled? Couldn't find a gulp command for that :-)

Right... Haha I'm sorry, it's not exactly documented. If you have https://github.com/electron-userland/electron-prebuilt installed, you can run $ npm start and anything from the main process will appear in the terminal while the app is running (though it is not packaged). I'll have to add this into the dev documentation

I'm have this same issue, I'm new to js, and all these build tools. any way to get a complete solution to build the spell checker with electron?

I'm still working on it. I already have misspelled words marked with a red underline, but I still fight with suggestions in the context menu + I don't have a lot of time atm
I'll notify this issue as soon as I have something I can share

Hi @Thatkookooguy, thanks for your investment into this! I'm closing this thread as part of an issue triage, but feel free to come back with questions anytime. Also, other users showed interest into a spellchecker in #283, maybe chime in there as people might be willing to help and hack on a solution :)

@ronjouch sure! I'll be happy to help anyone who wants to work on this. I kind of forgot about the whole spellchecker code. I'll try and go through it and see if it's easy to create a PR from it.


Release: SQL blog updates for Errata on April 13, 2020

Docs Build status updates of commit ef50ce9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

documentation/blog/MS-SQLINTBLOGLP/042a5f35-8f99-4653-912e-520835adad7b.md
:white_check_mark:Succeeded

documentation/blog/MS-SQLINTBLOGLP/4f7d7e62-9c63-424b-a70b-e6401ca09a8f.md
:white_check_mark:Succeeded

documentation/blog/MS-SQLINTBLOGLP/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


The module at ClientApp/dist/server/main.js does not export a default function, and you have not specified which export to invoke

I have the next problem when I run my asp .net core app:
NodeInvocationException: Prerendering failed because of error: Error: The module at ClientApp/dist/server/main.js does not export a default function, and you have not specified which export to invoke.
This is my startup.cs file:
` app.UseSpa(spa =>
{
// To learn more about options for serving an Angular SPA from ASP.NET Core,
// see https://go.microsoft.com/fwlink/?linkid=864501
    spa.Options.SourcePath = "ClientApp";

    spa.UseSpaPrerendering(options =>
    {
        options.BootModulePath = $"{spa.Options.SourcePath}/dist/server/main.js";
        options.BootModuleBuilder = env.IsDevelopment()
            ? new AngularCliBuilder(npmScript: "build:ssr")
            : null;
        options.ExcludeUrls = new[] { "/sockjs-node" };
    });

    if (env.IsDevelopment())
    {
        spa.Options.StartupTimeout = new System.TimeSpan(0,5,0);
        //spa.UseAngularCliServer(npmScript: "start");
        spa.UseProxyToSpaDevelopmentServer("http://localhost:4200");
    }
});`

and if I did: node -e "console.log(require('./ClientApp/dist/server/main.js'))"
this is the result:
{ LAZY_MODULE_MAP: { './admin/admin.module#AdminModule': NgModuleFactory_ { moduleType: [Function: AdminModule], _bootstrapComponents: [], ngModuleDefFactory: [Function] }, './investor/investor.module#InvestorModule': NgModuleFactory { moduleType: [Function: InvestorModule], _bootstrapComponents: [], ngModuleDefFactory: [Function] }, './restaurant/restaurant.module#RestaurantModule': NgModuleFactory { moduleType: [Function: RestaurantModule], _bootstrapComponents: [], ngModuleDefFactory: [Function] } }, AppServerModuleNgFactory: NgModuleFactory { moduleType: [Function: AppServerModule], _bootstrapComponents: [ [Function: AppComponent] ], _ngModuleDefFactory: [Function] }, AppServerModule: [Function: AppServerModule] }
Someone can help me?
I need to finish the implementation of angular universal on my app.

Hi @maurisa11, can you share more details on how to reproduce this issue? See https://github.com/aspnet/AspNetCore/blob/master/.github/ISSUE_TEMPLATE/bug_report.md#to-reproduce for an outline of the steps we are normally looking for.
cc @mkArtakMSFT - could be an issue with Node Services or templates...added to both areas.

@ryanbrandenburg can you please look into this? Thanks!

Hi. It looks like this is a question about how to use ASP.NET Core. While we do our best to look through all the issues filed here, to get a faster response we suggest posting your questions to StackOverflow using the asp.net-core-mvc tag.


fix(docs): remove schema mention

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the main branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

We no longer use schemas to validate the translation json objects,
instead using the English files as the single source of truth.

Thanks for catching this @ShaunSHamilton :)


Small typo

Hi, thanks for the great work! There is a small typo with the installation instructions markdown file. cd tlr/ should be cd trl/.

Thanks, I'll fix that! 🤗


Fixed typographical error, changed availabe to available in README.

google, I've corrected a typographical error in the documentation of the roboto project. You should be able to merge this pull request automatically. However, if this was intentional or you enjoy living in linguistic squalor please let me know and create an issue on my home repository.

We certainly don't want to live in linguistic squalor! However I'm not sure what's our protocol on handling contributions from a bot so I'll close this and fix the typo myself.

Out of curiosity (since there is a real person behind this two-day-old bot), why couldn't you accept a change from a bot? Is it a licensing problem?

Yep

So if a licence* was added (especially in the message) would this be
sufficient? I ask because I've submitted a pull request to an MS repo and
got a similar response.
*Something to the effect of all edits are freely given to the public domain?
On Tue, Sep 29, 2015 at 2:40 PM, James Godfrey-Kittle <
<EMAIL_ADDRESS>wrote:

Yep
—
Reply to this email directly or view it on GitHub
https://github.com/google/roboto/pull/112#issuecomment-144151055.

I doubt it. With cases like this I'd probably refer to a legal department for guidance. It's a little insane but such is the nature of working with large companies I suppose. Which is why it's easier to just reject the PR.
(I do know off the top of my head that public domain != licensed)

Understood! Thanks for the information.
I'm going to be using the number of successful pull requests as a marker
for the efficacy of the bot, so I have slightly selfish reasons for
increasing the number of pulls. :)
Also, thanks for you (and google) working on open-source projects!
On Tue, Sep 29, 2015 at 2:47 PM, James Godfrey-Kittle <
<EMAIL_ADDRESS>wrote:

I doubt it. With cases like this I'd probably refer to a legal department
for guidance. It's a little insane but such is the nature of working with
large companies I suppose. Which is why it's easier to just reject the PR.
(I do know off the top of my head that public domain != licensed)
—
Reply to this email directly or view it on GitHub
https://github.com/google/roboto/pull/112#issuecomment-144153253.

See the CONTRIBUTING.md file.  Basically you need to sign an agreement
before we can accept patches.  Small spelling corrections like this,
though, we should be able to take regardless.
On Tue, Sep 29, 2015 at 11:50 AM, Travis Hoppe<EMAIL_ADDRESS>wrote:

Understood! Thanks for the information.
I'm going to be using the number of successful pull requests as a marker
for the efficacy of the bot, so I have slightly selfish reasons for
increasing the number of pulls. :)
Also, thanks for you (and google) working on open-source projects!
On Tue, Sep 29, 2015 at 2:47 PM, James Godfrey-Kittle <
<EMAIL_ADDRESS>wrote:

I doubt it. With cases like this I'd probably refer to a legal department
for guidance. It's a little insane but such is the nature of working with
large companies I suppose. Which is why it's easier to just reject the
PR.
(I do know off the top of my head that public domain != licensed)
—
Reply to this email directly or view it on GitHub
https://github.com/google/roboto/pull/112#issuecomment-144153253.

—
Reply to this email directly or view it on GitHub
https://github.com/google/roboto/pull/112#issuecomment-144154286.


Question: will setAppElement(document.body) work as expected?

As I understand it, setAppElement() is needed so everything in the background, behind the modal, will be set to "hidden" for screen readers. The documentation specifies that setAppElement() should be applied to a React app (top-level) element.
However, what if we are developing a smaller React app that gets injected to a larger existing (third-party) page that does not use React. How do we make sure the rest of the whole page gets "hidden" when our modal is open? I would think I would just do ReactModal.setAppElement(document.body), but I'm not sure if the plugin was designed to handle this.
No errors appear to be thrown, but I don't know if it will actually work because I do not own a Screen Reader. Also could not find anything about this in the docs.

Hi @vincerubinetti, ReactModal.setAppElement(document.body) is not a good idea because the modal will be a children of the element that has the aria-hidden.

So what is the right way to do it then?

I am guessing that we should have a div inside the body like this:

     Put all your html here

and give it root?

The idea is that the div where all modals will be "mounted" on is different from the rest of the application.

So react-modal can do


RxSwift support

What do you think, if we add support RxSwift?
I think need one update:
public final class VK: **NSObject** { //... }

After we can make call as:
VK.API.Wall.get(Parameters).rx.send() //type: Observable<Data>
//etc.

Thank You for using SwiftyVK and your feedback)
Do You wanna add RxSwift as a dependency inside SwiftyVK?
Why You can't use SwiftyVK sending syntax? In my opinion, it's similar to Promises / Rx and very conveniently.
I'm not Rx fan and don't wanna add new dependencies without without much need

@SeRG1k17, I have not received a response from you in a month. So, I close this issue. If you have something to answer, you can reopen it again.
Best regards)


blocks, tortoise: merge for/against exceptions into a single list, vote neutral on layers instead of blocks

As described in https://github.com/spacemeshos/SMIPS/issues/2, the initial version of verifying tortoise includes three explicit lists of exception votes:
Block {
   Base_Block: <block_id>
   For_Exception_List: [<block_id>, <block_id>, <block_id>...] // List of blocks I vote For and Base_Block doesn't
   Against_Exception_List: [<block_id>, <block_id>, <block_id>...] // List of blocks I vote Against and Base_Block doesn't
   Neutral_Exception_List: [<block_id>, <block_id>, <block_id>...] // List of blocks I vote neutral and Base_Block doesn't or blocks that just come after Base_Block.
}

As discussed on the R&D call with Tal this week, we can optimize this in a few ways:

blocks only ever vote neutral on an entire layer, never on individual blocks in a layer (and only in the case where they're still waiting for a Hare result for that layer). So there's no need to list individual block ids in the neutral list, just layer IDs.
the For and Against exception lists can be collapsed down to a single list. For any block on this list, the block's vote is implicitly the opposite of the base block's vote.

Regarding neutral votes in the base block, by default a block citing it should also be considered to vote neutral for the same layers, unless the block includes any explicit for or against votes for any blocks in that layer (this may already be how things work, I'm not sure)
CC @y0sher, @sudachen
Addendum: a block does not need to encode an explicit vote for its base block, this can be made automatic and implicit. (discussion)

Related: in block fast validation, make sure the exception lists are valid (i.e., don't contain contradictions), and invalidate a block if they do:
https://github.com/spacemeshos/go-spacemesh/blob/9d3a379994e3652eb0378d2cc555588bba91005e/blocks/blocks.go#L153-L159

should also incorporate https://github.com/spacemeshos/go-spacemesh/issues/2674 when this will be implemented


python extension refactoring tools not working

The Python extension refactoring tools "Extract Variable" "Extract Method" and "Sort Imports" do not do anything. These are the errors:

Web Browser: New Edge
Local OS: Windows
Remote OS: Linux
Remote Architecture: Linux
code-server --version: 3.7.4 11f53784c58f68e7f4c5b3b8dae9407caa41725b

Oh, I think this is a feature that requires PyLance, which is not an open-source extension and can't be added to code-server, right?)
https://devblogs.microsoft.com/python/python-in-visual-studio-code-december-2020-release/

What version of the Python extension do you have installed? Refactor used to not be a PyLance only feature; I think it could be a bug.

It is v2020.12.422005962. I also tried with the previous version v2020.11.37

Hello, I had the same problem and managed to solve it by following these steps:
Remove all python related extensions
Go to /config/extensions from the terminal delete the ms-python folder
Restart the code-server
Download the ms-python.python-2020.10.332292344 version from the github extension page and install manually
Restart and configure the python interpreter

Hey there; tested this with the latest versions (code-server v3.10.2, python v2021.5.926500501, jupyter v2021.6.832593372) and it all works. Closing.


Always force push to gh-pages branch

...Otherwise, it fails 😥 https://github.com/astrojl-test/lumache/runs/4251705343?check_suite_focus=true#step:3:188
  Setting up git configure
  No local changes to save
  Setting up branch gh-pages
  Branch exists, chekcout to it
  error: pathspec 'gh-pages' did not match any file(s) known to git

@astrojuanlu Sorry, I do not understand what you mean, can you expian it in another way?


Use hash version if current branch tag differs from latest tag

Attempt to resolve https://github.com/go-gitea/gitea/issues/3194.
My idea is that if a releaser is on a release branch, their latest tag on the current branch (identified by git describe) will be the same as the latest tag of all branches (identified by git tag). If this is the case, use the v0.0.0+123-hash version scheme. In all other situations, only use hash for the version.

I might have to reconsider this. It's be nice if we could just show the latest tag across all branches and the number commits since that tag. Basically git describe, but across all branches.

Closing as I think this isn't the right solution. The number of commits since a tag is valuable information, which shouldn't be lost. Maybe something better comes out of https://github.com/go-gitea/gitea/issues/3194.


[tests] refactor test contracts to remove warnings

Example:
Warning Generated during Contract Compilation
                                      Warning=ef50c3be35f6f68d9c4d26fbfeea0edd493bc8971de6e1ef2bd9ba87411bd1e3.sol:6:3: Warning: No visibility specified. Defaulting to "public".
  function set(uint x) {
  ^ (Relevant source part starts here and spans across multiple lines).
ef50c3be35f6f68d9c4d26fbfeea0edd493bc8971de6e1ef2bd9ba87411bd1e3.sol:10:3: Warning: No visibility specified. Defaulting to "public".
  function get() constant returns (uint retVal) {
  ^ (Relevant source part starts here and spans across multiple lines).
ef50c3be35f6f68d9c4d26fbfeea0edd493bc8971de6e1ef2bd9ba87411bd1e3.sol:17:2: Warning: No visibility specified. Defaulting to "public".
	function create() returns (address GSAddr) {
 ^ (Relevant source part starts here and spans across multiple lines).
ef50c3be35f6f68d9c4d26fbfeea0edd493bc8971de6e1ef2bd9ba87411bd1e3.sol:22:2: Warning: No visibility specified. Defaulting to "public".
	function getLast() returns (address GSAddr) {
 ^ (Relevant source part starts here and spans across multiple lines).
ef50c3be35f6f68d9c4d26fbfeea0edd493bc8971de6e1ef2bd9ba87411bd1e3.sol:22:2: Warning: Function state mutability can be restricted to view
	function getLast() returns (address GSAddr) {
 ^ (Relevant source part starts here and spans across multiple lines).

assertion still succeeds so NBD (for now)

migrated from https://github.com/monax/bosmarmot/issues/43

Feel like we are some % done on this - @seanyoung ?

Nearly all warnings are gone. solidity complains that some functions which contain "assembly mstore" should be declared pure, which is just wrong. Otherwise all done


Some AST pointers

Hey, some additional pointers for the AST traversal.
Sometimes there is duplicate information inside the syntax tree:

When a return type is present inside a SynBinding, this body expression will be wrapped inside a SynExpr.Typed. The type inside SynExpr.Typed will be the same that is present in SynBindingReturnInfo.
You can ignore one of the types. Although, be aware that a SynExpr.Typed could also be used as the body of a SynBinding when there is no return type.
SynValData can be skipped in your algorithm,  the same information will be present in SynType.
SynExpr.Lambda contains an optimized version of the arguments and body inside parsedData, you can use that instead of looking into the nested SynExpr.Lambda.
For infix operators (SynExpr.App(_, true, ...)(example), you can safely assume the operator will be SynExpr.LongIdent. No need to traverse it, of course in practice that won't make much difference.

On some levels, you could easily collect the information in parallel (using PLinq or something else):

SynModuleDecl
SynModuleDeclSig
SynMemberDefn
SynMemberDefnSig
The bindings inside a recursive let SynModuleDecl.Let
Recursive types inside SynModuleDecl.Types
SynMatchClause
SynUnionCase

visitMulti could become something along the lines of
open System.Linq

let visitMulti (f: 't -> 'i seq) (items: 't seq) : 'i seq =
    items.AsParallel().SelectMany(f)

I hope that helps.
Cheers!

This is no longer actionable.


Add a key down press event into SearchBox

Purpose

add a key down press event into SearchBox for quick access to the Treeview
add a key enter press event into TreeView to quick open a command

Declarations
Check these if you believe they are true

[ ] This PR fix bug
[x] This PR for new feature
[ ] The codebase is in a better state after this PR
[ ] The level of testing this PR includes is appropriate
[ ] User facing strings, if any, are extracted into *.resx files
[ ] Snapshot of UI changes, if any.
[ ] This PR modifies some build requirements and the readme is updated

Reviewers
@chuongmep
FYIs
@chuongmep

@sapiens-code  Whether we can add more Event when user use keyboard Arrow Up from TreeviewItem to Search again ?


enstablish github communication

criteria:

commit message format
branching rules
github projects

github workflow: trunk
commit message format: default trunk approach (header and meaningful description (optional))
projects are good


Pass &self in map_regs in Function?

This would be a nice thing if implementation of Function need to do some other work when this happens. In my VM for example I don't have register classes and all registers is just u32 values, and when allocating register regalloc.rs may panic that some register from some instruction is not "real" and I had to use statc mut there:
    fn map_regs(
        insn: &mut Self::Inst,
        pre_map: &Map<VirtualReg, RealReg>,
        post_map: &Map<VirtualReg, RealReg>,
    ) {
        unsafe {
            MAPED = true;
        }
        insn.map_regs_d_u(
            /* define-map = */ post_map, /* use-map = */ pre_map,
        );
    }

    fn get_regs(&self, insn: &Self::Inst) -> regalloc::InstRegUses {
        let (d, m, u) = insn.get_reg_usage(unsafe { MAPED }); // if `MAPED` is true instead of creating virtual registers we create real register (`Reg::new_real`)

        regalloc::InstRegUses {
            used: u,
            defined: d,
            modified: m,
        }
    }

Hi @playXE, the reason the interface is currently this way is that usually the Function implementer is some sort of container for instructions, and map_regs needs a mutable borrow for an instruction. This instruction borrow would conflict with an immutable borrow of the whole container. (We perhaps could have designed it to instead take an Inst owned, and return a new Inst, but that might be slightly more expensive than mutating in place.)
For your issue, it's not entirely clear to me what MAPED means, or why you're returning either virtual or real registers conditionally. The way the interface is intended to be used, we have register slots in instructions that can hold either virtual or real registers, they mostly start as virtual, and map_regs is guaranteed to be called to change all virtual regs to real regs. So you shouldn't need to do this mutation (or change what you return) yourself. In other words, just store a Reg for each register.
Hopefully that made sense! Perhaps I'm still not understanding your use-case though?

Thanks for your reply @cfallin ! I don't really need this thing in my bytecode now, every "real" register is <= 32 and registers bigger than 32 is now virtual register :)


Mistake in ports asm

The clobber is "edi" but used "esi".
Here and two more lines.

Thanks for reporting!
This assembly line comes from gz/rust-x86, which took it from Tobba/libcpu. I don't know the exact requirements for inline assembly (e.g. do I need to add input registers to the clobber list), but I try to find it out.
Maybe @Tobba, @gz, or @Ericson2314 know more?

It's usually additional registers which get changed apart from the ones declared in output (i.e., some may change as a result of side-effects which the compiler can't figure out)

@phil-opp  You are right. Looks like the code is ok.

Ok, thanks for reporting anyway! It's always good to double check inline assembly.


[bug] multiple values for proxy_header templates

I'm receiving quadruple values for templated headers of proxy_header:
Incoming Headers
X-Forwarded-Proto: https, https, https, https
X-Forwarded-Host: hellabit.com, hellabit.com, hellabit.com, hellabit.com

Caddyfile
https://hellabit.com {
  gzip
  tls /srv/walnut/certs/live/hellabit.com/fullchain.pem /srv/walnut/certs/live/hellabit.com/privkey.pem
  root /srv/walnut/sites-enabled/hellabit.com/
  proxy /api http://localhost:4080 {
    proxy_header Host {host}
    proxy_header X-Forwarded-Host {host}
    proxy_header X-Forwarded-Proto {scheme}
  }
}

Uhhh that's cool... I'll see if I can reproduce it.

But it doesn't happen for the Host header?

My guess is that it does but node's http parser ignores the illegal ',' and all trailing characters.

Also, I had 4 entries in my Caddyfile. It would be interesting to test with 3 or 5 entries and see if the multiple changes accordingly.

Yes, please do and let me know.
By "entries" you mean server blocks?

Yes, server blocks.
Confirmed that the number of times that a value exists is the number of server blocks.

Just upload a fix, it makes http header block scoped instead global share.
BTW, we may also set a default global one, it may be used when proxy block
doesn't set its own one.

@tw4452852 That's awesome - thank you! And that's true, we'll keep that in mind if it turns out we need a default global header map for the proxy.

@coolaj86 I've merged in Tw's fix - if you pull the latest from master it should be working now. Thanks for the report, and the fix, both of you!

Haven't tested yet, but wanted to say thank you all the same.
:-)


Not receiving any callback from onInstall listeners

Had to ensure the listeners were set up prior to initSdk call; issue on my end!


now dev not serving serverless gatsby functions on Windows machine

I haven't been able to get serverless functions to work using now dev on my Windows machine. It seems that there is an issue with the port being set, (log print NaN).
Windows 10
node v10.16
npm 6.10
now 16.3.3
Setup
now init gatsby-functions
cd gatsby-functions
npm i
now dev -d
log
$ now dev -d
> [debug] [2019-12-11T12:43:29.628Z] Using Now CLI 16.6.3
> [debug] [2019-12-11T12:43:29.631Z] user supplied known subcommand
> Now CLI 16.6.3 dev (beta) — https://zeit.co/feedback/dev
> [debug] [2019-12-11T12:43:29.802Z] The yarn executable is already cached, not re-downloading
> [debug] [2019-12-11T12:43:29.806Z] Reading `package.json` file
> [debug] [2019-12-11T12:43:29.807Z] Reading `now.json` file
> [debug] [2019-12-11T12:43:29.808Z] No `now.json` file present
> [debug] [2019-12-11T12:43:29.809Z] Searching files inside of C:\Users\sylva\Documents\Projects\Lotto\gatsby-functions\
> [debug] [2019-12-11T12:43:32.336Z] Found 27095 and filtered out 27072 files
> [debug] [2019-12-11T12:43:32.358Z] Locating files C:\Users\sylva\Documents\Projects\Lotto\gatsby-functions
> [debug] [2019-12-11T12:43:32.359Z] Ignoring C:\Users\sylva\Documents\Projects\Lotto\gatsby-functions\.cache
> [debug] [2019-12-11T12:43:32.360Z] Ignoring C:\Users\sylva\Documents\Projects\Lotto\gatsby-functions\.gitignore
> [debug] [2019-12-11T12:43:32.360Z] Ignoring C:\Users\sylva\Documents\Projects\Lotto\gatsby-functions\README.md
> [debug] [2019-12-11T12:43:32.361Z] Ignoring C:\Users\sylva\Documents\Projects\Lotto\gatsby-functions\node_modules
> [debug] [2019-12-11T12:43:32.366Z] Locating files C:\Users\sylva\Documents\Projects\Lotto\gatsby-functions: 7.029ms
> [debug] [2019-12-11T12:43:32.399Z] No builders need to be installed
> [debug] [2019-12-11T12:43:32.655Z] Adding build match for "api/date.ts"
> [debug] [2019-12-11T12:43:32.678Z] Adding build match for "package.json"
> Creating initial build
> Building @now/static-build@latest:package.json
> [debug] [2019-12-11T12:43:32.692Z] Using `<EMAIL_ADDRESS>> [debug] [2019-12-11T12:43:32.693Z] Creating build process for package.json
Installing dependencies...
yarn run v1.17.3
warning package.json: No license field
$ gatsby develop -p $PORT
> [debug] [2019-12-11T12:43:34.054Z] No builders were updated
Something is already running at port NaN
Would you like to run the app at another port instead? [Y/n] y
success open and validate gatsby-configs - 0.057 s
success load plugins - 0.351 s
success onPreInit - 0.039 s
success initialize cache - 0.048 s
success copy gatsby files - 0.181 s
success onPreBootstrap - 0.045 s
success source and transform nodes - 0.085 s
success Add explicit types - 0.037 s
success Add inferred types - 0.082 s
success Processing types - 0.075 s
success building schema - 0.307 s
success createPages - 0.037 s
success createPagesStatefully - 0.091 s
success onPreExtractQueries - 0.038 s
success update schema - 0.058 s
success extract queries from components - 0.133 s
success write out requires - 0.056 s
success write out redirect data - 0.037 s
success run static queries - 0.040 s
success run page queries - 0.076 s — 2/2 69.68 queries/second
success start webpack server - 1.149 s — 1/1 3.55 pages/second
 DONE  Compiled successfully in 2536ms
                                                                              8:43:48 PM
You can now view gatsby-starter-default in the browser.

  http://localhost:52080/                                                                                8:43:48 PM

View GraphiQL, an in-browser IDE, to explore your site's data and schema

  http://localhost:52080/___graphql

Note that the development build is not optimized.
To create a production build, use npm run build

i ｢wdm｣:
i ｢wdm｣: Compiled successfully.

result:

nothing on localhost:3000
on localhost:52080.

other test done with same result
now dev -d --listen 3000
Workaround found
It is working on WSL,

Hi @sylc
Thanks for reporting this issue!
I believe the workaround is to modify package.json and change the dev script to use the Windows style environment variable syntax.
Before
"dev": "gatsby develop -p $PORT",

After
"dev": "gatsby develop -p %PORT%",

thanks for the fix! :)


unable to get bgpv4 metric using snappi convergence

use the request in this way to get bgp metrics
    req.bgpv4.column_names = []
    req.bgpv4.device_names = []
    res = cvg_api.get_results(req)


Double engineering doors all access on Almayer.

Description
All of the double doors in engineering on the Almayer are all access and accessible without an ID.
Reproduction
Attempt to open any double engineering door on the Almayer without an ID.
Screenshots

Additional context
N/A

#2393

Dupe of #3737 (all doors are effected)


run_DAG with problem "RuntimeError: CUDA error: device-side assert triggered"

Hi, thanks for the contributions.
I run into the problem "RuntimeError: CUDA error: device-side assert triggered" with run_DAG. It is related with the function "_get_targets" in detector. I used the faster_rcnn config with COCO_2017_val. The visualization of detectron works correctly.
Is there anything wrong with my settings?


Use TextInputLayout in all fields of the edit form

https://developer.android.com/reference/android/support/design/widget/TextInputLayout

I can work on it as soon as the pull request 2252 can be accepted / merged ( to avoid too many merge issue).
Could we split this issue in 2: one for Nutritions Fact and one for Overview ?


Does it work with Safari 10.1.1

I try to install it but with no effect.

It should work. I am on Safari 10.1.2 and am able to install it without issue.


Added Praxis Web and Api documentation framework

Another micro framework to be aware of

Great. Thanks. Cheers.


Weird behaviour when multiple tabs are open

Needs further investigation!

Haven’t managed to recreate this… I think it might just be a problem in development. Closing for the timebeing.


Can't reproduce locally, can you provide a detailed git log before and after the amending the commit ?
Something like git log --graph --abbrev-commit --decorate --all.

Hmm, that appears sometime in your gitlab pipelines ... I look forward to find a better solution for reproduce ....
I got on my disk also not reproduce it ATM.

Reminds me of this https://github.com/libgit2/libgit2/issues/3058
Is your gitlab pipeline public ?

Now i build also an public pipeline, where it failed:
https://gitlab.com/kukoon/mediathek/media-ui/-/jobs/3538884990

Can you disable shallow clone  and try again ?
see :
https://docs.gitlab.com/ee/ci/pipelines/settings.html#limit-the-number-of-changes-fetched-during-clone
https://docs.gitlab.com/ee/ci/large_repositories/#shallow-cloning

The Problem is, i maybe do not know - how many commits happen since last release / tag ...

Closing since this is related to git2 shallow clone support and won't fix.


Column (box) text selection not implemented

The shortcut in VSC for this is shit+alt+(mouse click)
https://code.visualstudio.com/docs/editor/editingevolved#_column-box-text-selection
In IDEA it's just alt+(mouse click)
https://www.jetbrains.com/help/webstorm/2016.2/selecting-text-in-the-editor.html

@skysteve
There is two issues.

VS Code can't bind keys to mouse click: https://code.visualstudio.com/docs/customization/keybindings#_accepted-keys
The command id column box text selection is not available: https://code.visualstudio.com/docs/customization/keybindings#_basic-editing

Fair enough

😥


[DO NOT MERGE] New snippets and API for image lib

Changes:

New webcam class
Ability to use relative model links
New snippet
Bump in version because of breaking changes

Once we review the API, I'll update the readme then merge

Test snippet now looks like this


[terraform-ibm-ocp-all-inclusive] Expose new security group variables

In this PR, new variables were added to support attaching additional security groups to the worker pools, VPE and load balancers.
They ended up in this release -> https://github.com/terraform-ibm-modules/terraform-ibm-base-ocp-vpc/releases/tag/v3.15.0
This module should be updated to consume this version (or later) and the new variables should be exposed

Change done in https://github.com/terraform-ibm-modules/terraform-ibm-ocp-all-inclusive/pull/243


web vs. gcs datasets

Hi, so far I've been using the dataset from web links which are structured according to dataset.proto, but newest Tensorflow3D downloads from gcs and uses this spec
where pointcloud is directly in xyz points. Thus, they don't use waymo repo at all. Am I missing something?
Regards,

That data is hosted by Waymo. They simply processed the data. But we do not provide any support for that.


write and wall ubuntu commands not work on windows ubuntu shell

those commands not recognize that the other user is even logged in

No template.

/dupe #573

I didnt understood from your answer if the problem solved, and if it's
solved - how to fix it?
בתאריך יום ג׳, 25 במאי 2021 ב-21:27 מאת msftbot[bot] <
@.***>:

Hi! We've identified this issue as a duplicate of another one that already
exists in this repository. This specific instance is being closed in favor
of tracking the concern over on the referenced thread.
Thanks for your report!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/WSL/issues/6987#issuecomment-848110265, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQGNZPBK2MU6N64A7643US3TPPTX7ANCNFSM45PPXATQ
.

--
בברכה,
פריאל חזן
לאתר העסק: Stringel Website https://stringel.netlify.app/


Clean up test rulesets

Justification
Removed temporary, repo-specific overrides from thet rulesets. These rulesets should reflect our ideal coding conventions and not contain repo-specific tech debt. These are significantly trimmed down and shouldn't change much over time.

You only put the localization rules in the utility ruleset. Don't they need to be in the test ruleset also?

You only put the localization rules in the utility ruleset. Don't they need to be in the test ruleset also?

Yes they needed to be. The changes were unsaved in VS Code. I saved and pushed the updates. Good catch, thanks!


Upload export items

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: pete-woods sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Whoops, wrong fork


Support loading plugins from multiple packages.

Description
This PR allows Data Prepper to support plugins in packages other than com.amazon.dataprepper.plugins.
Data Prepper reads all resources (files embedded in jars) of META-INF/data-prepper.plugins.properties. It then looks for a property org.opensearch.dataprepper.plugin.packages from each one. The value of this property is a comma-delimited list of packages.
Data Prepper will scan all of these packages for plugins.
Note that because of the uber-jar, Data Prepper can only have one file for all packages. However, plugin authors can create their own jar files with this file. As long as it is on the classpath, their custom plugin will be available.
It may be worthwhile to move some plugins in future PRs.
Issues Resolved
#379
Check List

[x] New functionality includes testing.
[ ] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I created a sample Data Prepper plugin which demonstrates this behavior: https://github.com/dlvenable/data-prepper-sample-plugin

Codecov Report

Merging #948 (c215372) into main (f7cdbd7) will decrease coverage by 0.05%.
The diff coverage is 88.57%.

@@             Coverage Diff              @@
##               main     #948      +/-   ##
============================================
- Coverage     91.40%   91.35%   -0.06%
- Complexity      673      681       +8
============================================
  Files            83       84       +1
  Lines          1955     1989      +34
  Branches        165      168       +3
============================================
+ Hits           1787     1817      +30
- Misses          129      132       +3
- Partials         39       40       +1

Impacted Files
Coverage Δ

...zon/dataprepper/plugin/PluginPackagesSupplier.java
87.87% <87.87%> (ø)

...on/dataprepper/plugin/ClasspathPluginProvider.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


WIP Auto update support on Linux

Can't merge until GH fixes whatever AWS thing they broke for user content. The zip files on the releases page are borked...

Releases are back online!


Revert "Revert "dTest""

Reverts christopherrolfe198/testing-webhooks#3

/merge

/merge

/merge

/merge

/merge

/merge

/merge

/merge

/merge

/merge

/merge

/merge

PR is not currently mergable

/merge


Lovasz loss

Hi,
Thank you for the code. I am attempting to run your code with the lovasz hinge loss, and run into the following error -- please let me know if there is a quick way to fix this.
File "..../lovash_losses.py", line 105, in lovasz_hinge_flat
errors = (1. - logits * Variable(signs))
RuntimeError: Variable data has to be a tensor, but got Variable
Apologies if this is a simple question -- recently converted from Keras, so not that familiar with Torch.
-Qing

Hi, setting the per_image flag to False seems to have fixed this issue.  I will close the issue in a day or so after completing the training. Thx!


chore: Clean up Rector skip config on SimplifyRegexPatternRector and RemoveUnusedPromotedPropertyRector

Description

[x] The SimplifyRegexPatternRector not used
[x] The RemoveUnusedPromotedPropertyRector bug already resolved on latest rector.

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Ready to merge 👍

@kenjis I am ok for performance reason then, thank you for the review, let's merge ;)


Predicates / Gates on productions should be in addition to standard lookahead

Not completely replace the basic lookahead calculation.
.

This means that Predicates / Gates should only be used to to add constraints to the grammar's flow.
The basic lookahead function is always computed using the grammar information.
Previously the predicate completely replaced the lookahead function. Therefore if someone wanted to implement a constraint on the grammar they also had to implement the lookahead function.


error not found when running barrel replication tests

Description
In travis the following trace appear from time to time:
%%% barrel_http_replicate_SUITE ==> {failed,
    {{badmatch,{error,not_found}},
     [{barrel_http_replicate_SUITE,one_doc,1,
          [{file,
               "/home/travis/build/barrel-db/barrel-platform/_build/test/lib/barrel_http/test/barrel_http_replicate_SUITE.erl"},
           {line,95}]},
      {test_server,ts_tc,3,[{file,"test_server.erl"},{line,1529}]},
      {test_server,run_test_case_eval1,6,
          [{file,"test_server.erl"},{line,1045}]},
      {test_server,run_test_case_eval,9,
          [{file,"test_server.erl"},{line,977}]}]}}

Reproduction guide

run rebar3 ct

probably related to #60

fixed in latest master


error: org.freedesktop.Sdk/x86_64/21.08 not installed

instructions don't work.
$ flatpak-builder --force-clean build-dir com.example.FlutterApp.json
error: org.freedesktop.Sdk/x86_64/21.08 not installed
Failed to init: Unable to find sdk org.freedesktop.Sdk version 21.08

$ flatpak install flathub org.freedesktop.Platform
fixed it.
I'm a noob, but this might help the next person...

Now, how do I install this to test?  no instructions.

$ flatpak run com.example.FlutterApp
error: app/com.example.FlutterApp/x86_64/master not installed

ok, finally got it.
$ flatpak-builder --user --install --force-clean build-dir com.example.FlutterApp.json
$ flatpak run com.example.FlutterApp
It runs, but I get this:
Gtk-Message: 17:16:17.304: Failed to load module "canberra-gtk-module"
Gtk-Message: 17:16:17.304: Failed to load module "canberra-gtk-module"
** (flutter_flatpak_example:2): CRITICAL **: 17:16:17.522: Failed to read XDG desktop portal settings: GDBus.Error:org.freedesktop.portal.Error.NotFound: Requested setting not found
** (flutter_flatpak_example:2): CRITICAL **: 17:16:17.524: Failed to read XDG desktop portal settings: GDBus.Error:org.freedesktop.portal.Error.NotFound: Requested setting not found

As mentioned in the README, Flatpak builds should be done in a reproducible environment like GitHub actions or Docker. Where are you trying to build from? Did you reference the linked example workflow?

Thank you @Merrit for providing this repo and thank you @sgehrman for adding those missing instructions.
In my case I am using Ubuntu and that's how I've built Flatpak executable:
sudo apt install flatpak flatpak-builder
flatpak remote-add --if-not-exists flathub https://flathub.org/repo/flathub.flatpakrepo
flatpak install flathub org.freedesktop.Sdk//21.08
flatpak install flathub org.freedesktop.Platform
flatpak-builder --user --install --force-clean build-dir com.example.FlutterApp.json
flatpak run com.example.FlutterApp

I agree with you @Merrit that builds shall be done in a reproducible environment, but I believe that everyone will set up their CI/CD pipelines at their own discretion anyway, whereas it's good to have minimal sets of instructions, so you can actually play around and validate whether Flatpak is what you really wanted in first place.

I've just pushed a large update that should hopefully make things clearer and easier :)


Proposal: Rename OptionToken

We should rename this.
At some point we are going to add other types of tokens (like futures) it doesnt make sense to call the ERC1155 contract OptionToken

How about DerivativeToken


trigger click to play sound

Hi,
I tried using jquery trigger click event to play the sound. The function is triggered but the sound doesn't play. Wonder whether you can help me with this?
Thanks.
ian

Why you need to trigger "click"? Just call ion.sound.play(...);

Below is my code.
for some reason, it doesn't auto play when it is not included in the click event but the click event manage to play make it played once I actually click it. yet, using jquery trigger doesnt trigger it to autoplay.
$(document).ready(function()
{
ion.sound
({
sounds:
[
{
alias: "s5",
name: "billie_jean"
}
],
 path: "sounds/",
 preload: true,
 multiplay: true,
 volume: 1
});

ion.sound.play("s5");

    var toPlay = function()
{
	ion.sound.play("s5");
	console.log('lalalala');
	for(var i = 1; i <= 10; i++)
	{
		var setVolume = volumeUp + (i*1/10);
		slowlyFadeIn(i, setVolume);
	}
};
$("#b01_play").on("touchstart click", toPlay);

});

Hey, you are trying to autoplay music on some iPhone/iPad etc? If so, it won't. Mobile devices required direct user action to enable media. This is an official restriction. Read this for example: https://developer.apple.com/library/safari/documentation/AudioVideo/Conceptual/Using_HTML5_Audio_Video/Device-SpecificConsiderations/Device-SpecificConsiderations.html

nope. Its not. I run it on desktop with chrome as my browser.

Hi, did you manage to find out what happens?

It should be fixed now. It was problem with autoplay.


v3.3.2 throw error in IE9.

<EMAIL_ADDRESS>throw error in IE9.
The error is caused by performance.now()
lib<EMAIL_ADDRESS>images:

I can't reproduce this error when testing on IE9 via browserstack.com. @laoboxie can you clarify exactly what code you're running that's causing this error?
I'm trying the following, which doesn't error for me in IE9.
<script>
(function () {
  var script = document.createElement('script');
  script.src =<EMAIL_ADDRESS>  script.onload = function () {
    webVitals.getTTFB(console.log);
  };
  document.head.appendChild(script);
})();
</script>

Also, when I look at the code, I do not believe it's possible for the part of the code that calls performance.now() to be reached in IE9 because that line is within a conditional that only runs if the browser finds a navigation performance entry (which would not be true in IE9). See: https://github.com/GoogleChrome/web-vitals/blob/1d2c191e6763ea3ebeef05e9141847dbce554228/src/onTTFB.ts#L71-L82
So unless I'm missing something then I think the documentation is correct in that the onTTFB() function doesn't produce a result in IE9 but it also doesn't error:

The web-vitals code has been tested and will run without error in all major browsers as well as Internet Explorer back to version 9.

And I think it's fine to continue to make this claim because I do think we want people to be able to run this code anywhere without worry that it will error and potentially break other critical functionality on their site.

Closing this as no further updates heard back. Feel free to reopen if necessary.


Pass SHA256 benchmark

SHA256 now compiles to ZK ASM and runs to completion after a hacky set of changes, but the result of execution is still incorrect.
The missing pieces right now are:

Global variables - these are declared in WAT file and live at specific memory offsets. We can replace them with ZK ASM Global variables and would need to detect the access to these offsets and replace them with loads/store from globals
Data segment (e.g. "hello world" string in SHA256 benchmark) - these are very similar to global variables - reads from a very large offset. But this time it's from a contiguous slice of memory, rather than discrete slots of globals - one way to handle these is to dedicate a memory region and fill it with data at the start of the program
Loads from frame pointer + offset - I'm not sure what these are, likely part of the function ABI and would have to use stack or registers instead of the frame. Right now these are commented out

Let's fix them!
- [x] Global variables
- [x] Data segments
- [ ] Loads from frame pointer + offset

We now have fully working global variables and initial work for data segments (preamble generation).
For fully working data segments we need to implement a form of https://github.com/near/wasmtime/issues/43 and https://github.com/near/wasmtime/issues/110.
We also still need to select an implementation for frame pointer reads.

After the implementation for the frame pointers and fixes to stack handing, the benchmark now runs till completion, but still fails on the asserts: https://github.com/near/wasmtime/pull/185
I will try to simplify it next to make sure it consumes fewer cycles and is easier to debug.

https://github.com/near/wasmtime/pull/185 is now submitted and we finally pass SHA256 benchmark on main!


Old PyPI release

Hey,
would it be possible to release a new blocksec2go version to PyPI? The version out of PyPI is 1.1 even though the repo is on 1.2. The observer example included in the repo doesn't work with the current PyPI version.
Thanks!

True that:
Traceback (most recent call last):
File "praesidium.py", line 18, in
from blocksec2go.comm import observer
ImportError: cannot import name 'observer' from 'blocksec2go.comm' (C:\Users\Pachler\AppData\Local\Continuum\anaconda3\lib\site-packages\blocksec2go\comm_init_.py)
we will update! :)

Thank you for updating blocksec2go on PyPI!


[BUG] correct parameter name in TestAllObjects all_objects call

This corrects the parameter name in TestAllObjects all_objects call, from exclude_estimators to exclude_objects.

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

Comparison is base (1b5d4a8) 83.55% compared to head (225c1b0) 83.55%.
Report is 7 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #236   +/-   ##
=======================================
  Coverage   83.55%   83.55%
=======================================
  Files          41       41
  Lines        2833     2833
=======================================
  Hits         2367     2367
  Misses        466      466

Files
Coverage Δ

skbase/testing/test_all_objects.py
79.43% <ø> (ø)

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


Update README.md

Hello, I don't think we want to take this change. We use the term ".NET". Thanks though.


Is there any docker file available?

Thanks for your great work for distributional RL.
I will be appreciated if you can address some questions I have met.

Is there any docker file available?
Also, I noticed that IDAC supports Tensorflow versions 1.15. Do you have planed and tested in higher versions, such as TensorFlow 2.5?
Which python version do you test, python 3.6 or python 3.7?
Thanks for your early reply.

Hello, thanks for your interest!
We do not have a docker file yet, and the currently implementation uses TF1. However, the environment setup is exactly the same with the requirements from the stable-baselines.
The python version is 3.7.

Thanks for your reply. I am appreciated for your great help.
In addition, do you have planed PyTorch version of this code?
By the way, it is a great and deeply inspirational paper. Thanks for your contributions to Distributional Reinforcement Learning.

The PyTorch version is provided.


[BUG] test_engine_image_not_fully_deployed_perform_volume_operations failed on Longhorn v1.3.3 with SLE Micro

Describe the bug (🐛 if you encounter this issue)
I have observed that the test case test_engine_image_not_fully_deployed_perform_volume_operations &  test_engine_image_not_fully_deployed_perform_dr_restoring_expanding_volume
are failing on Longhorn v1.3.3 and SLE Micro 5.3 & 5.4.
To Reproduce

Rerun the test case  on Longhorn v1.3.3 and SLE Micro 5.3 & 5.4.

Expected behavior
We should have consistent test results on all distro.
Support bundle for troubleshooting
longhorn-upgrade-tests-sle-micro-arm64-3-bundle.zip
Environment

Longhorn version:v1.3.3
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: v1.24.15+k3s1

Number of management node in the cluster: 1
Number of worker node in the cluster: 3

Node config

OS type and version: SLE Micro  5.4
Kernel version:
CPU per node: 4 cores
Memory per node: 16G
Disk type(e.g. SSD/NVMe/HDD): SSD
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): AWS
Number of Longhorn volumes in the cluster: 1

Additional context
N/A

Did the manual reproduction also fail at the same step as the test script?

Did the manual reproduction also fail at the same step as the test script?

Hi @chriscchien :
I discovered that the issue is caused by https://github.com/longhorn/longhorn/issues/5815.
I will close the ticket.


Set Position / Scrubbing / Seeking

How can I set the start position within the stream? For example, a user might want to click/seek to a given point in the streamed file and I want to set the player to that position to resume playback from that point (without the waiting for the whole file to load).

Unfortunately, these playback demos do not provide examples for seeking.  You may also wish to see the conversation at https://github.com/AnthumChris/opus-stream-decoder/issues/1


[release-0.22] Upgrade to latest dependencies

webhook fixes -dprotaso
/cc knative/operations-wg-leads
/assign knative/operations-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 7c0b477...6aa0b41:
  > 6aa0b41 Use new label for defining a cluster-local Knative Service (# 5403)
  > 6d63984 Add podAntiAffinity labels to remaining HA control plane pods (# 5414)
  > 0d8411e Drop --strict flag on ko (# 5513)
bumping knative.dev/serving 813aa65...3e9a4e3:
  > 3e9a4e3 [release-0.22] run kind e2e in release branches (# 11139)
  > 95fe811 Change magic DNS to sslip.io (# 11299)
bumping knative.dev/pkg 952fdd9...983897f:
  > 983897f [release-0.22] allow unknown metadata fields (# 2257)
  > 6ea2c43 Drop redundant pointers and decoders (# 2262)

closing - will reach out to Vincent next week


Question: Best practises for forwarding logs from flynn to external analyzer?

Hey!
I'm willing to update the docs in return if you can share few pointers how log forwarding should be done from flynn cluster to eg. graylog or elk-stack.
I'm wondering this because for my typical application can output different kind of logs like:

nginx error logs
nginx access logs
application logs
application error logs
container startup/debug logs
...

If I just combine them all to container stdout/stderr it'll be harder to use them later on.

We have plans to support forwarding logs from the built-in logging system to external servers (#1422).
For now you can use something like rsyslog to collect the on disk logs on each host in /var/log/flynn (there is a file per app, with each log line being an RFC5424 syslog Message).

Sorry I didn't find #1422. My bad!


Merge Python Integrations to a single page

In this PR:
Merged the Logz.io Python logging handler, OTEL custom metrics and OTEL auto traces instrumentation to a single document.

@Doron-Bargo @nico-shishkin @mirii1994 I'm leaving it open as in order for the Python docs table of content to align we need to modify the headers of the included files which can impact other docs.
{@include: ../../_include/tracing-shipping/python-steps.md}
{@include: ../../_include/tracing-shipping/docker.md}

Reverted docker includes & unpacked its content into the Python doc so the headers will align.


Run this demo in Electron, but failed

Hello, I want to run this demo in Electron, but it failed. Could you tell me how to run it in Electron? Thanks !!!
This is my code.

This is the result.

@dannyrb @swederik

Hi @zhangluwei1997
You might just need a route in electron to either /viewer or /local

@eolasd  What do you mean by  'route in electron' ?   Do I need to create a new folder called viewer or local then move HTML to that folder ?

I just did a quick test with a clean electron project and got it working.
If you create a new viewer.html file, and paste in all the OHIF code that you have in Index.js it should work.
You can also just change the   mainWindow.loadFile to point straight to your viewer.html file if you don't need and index page.
The routes used in OHIF react are defined here...
https://github.com/OHIF/Viewers/blob/f9fe4232e755d2478dae4c70dac90a7b42e34846/platform/viewer/src/routes/routesUtil.js#L34
I can't guarantee this is the best way to do this, but at least it should get you to a working viewer.
I think within electron you might need to use hash routing - but I am not an electron expert.

Thanks @eolasd
Could you show me your code in  CodeSandbox ?

Sorry,  cant get electron in codesandbox right now.
But this is what I did:
Install Electron quick start demo
git clone https://github.com/electron/electron-quick-start
cd electron-quick-start
npm install

Add OHIF Viewer file / route
Create new file called viewer.html with code:
<!DOCTYPE html>

	<meta charset="utf-8" />

	<meta name="description" content="Open Health Imaging Foundation DICOM Viewer" />
	<meta name="viewport" content="width=device-width,initial-scale=1.0,minimum-scale=1,maximum-scale=1,user-scalable=no" />
	<meta name="theme-color" content="#000000" />
	<meta http-equiv="cleartype" content="on" />
	<meta name="MobileOptimized" content="320" />
	<meta name="HandheldFriendly" content="True" />
	<meta name="apple-mobile-web-app-capable" content="yes" />

	<link href="https://fonts.googleapis.com/css?family=Sanchez" rel="stylesheet" />

	<title>OHIF Standalone Viewer</title>

	 You need to enable JavaScript to run this app.

Update app entry point to new viewer.html file
Change to this line in the main.js file, line 16:
  // and load the index.html of the app.
  mainWindow.loadFile('viewer.html')

npm start to test application.
You still get the "sorry page does not exist" on first load, but if you click the link it will go to the study list screen and should display the sample study list.
Not perfect, but should be a starting point.

Thanks for you help !
I'm in the same situation as you. I thought you didn't get the "sorry page does not exist" on first load. so I want to see your code.
Thank you very much !

No problem.
I just realised that it doesn't matter what the viewer.html file is called, it actually loads regardless.
I thought it needed a hosted /viewer route - which is what happens when you try run it on a web server.
So I think what is happening is that React has to load everything first before the route exists, so then the link works.
It shouldn't be too hard to get working - but it probably needs hash routing or to embed react more granular in electron using proper react install as opposed to the UMD package.
Best of luck!


When value is rgba

I have a field with a default value of rgba(80,80,80,0.25), which could be random rgba. Used for styling box shadows. But spectum does not even register it. It defaults to rgb(0,0,0).
$("#caption-box-color").spectrum({preferredFormat: "rgb", showInitial: true, showInput: true, showAlpha: true});
After selecting and changing it to like rgba(0,0,0,0.50) it is saved as #000000?

It seems to work fine for me in a demo like this: http://jsfiddle.net/2ageh2ox/1/.  Can you send over a failing test case?

I was using type="color" not type="text" now it works. Thanks!

One last ting....
input name="font-color" value="#936" id="font-color" class="form-control" type="color"
Shows as black. I see demo's using the 3 string color values.

All my fields with 3 string color values show as black.

Yeah, that is defined by the spec: http://www.w3.org/TR/html-markup/input.color.html#input.color.attrs.value.  I'd recommend just using text inputs if you want to do anything more than 6 character hex values.

Ok, I see. change them all to text and just gavce a class, then ran:
$(".colors").each(function(){
	$(this).spectrum({
		color: $(this).val()
	});
});

Getting what I need, thanks.


Error Contents: invalid literal for int() with base 10: '0+matrix' during background service startup

Netflix add-on version
1.18.7+matrix.1
Operative systems used
LibreELEC
Kodi version used
Kodi 19 (Matrix)
Description of the bug
After startup of Kodi the UI shows an error message that the background services for the plugin could not be started. I downgraded to version 1.18.6+matrix.1 of the plugin where everything still works.
Debug Log hopefully gives all relevant insights in what is going on. Let me know if further information are needed. Imho: Timestamp of relevant error: 2022-06-16 22:26:54.274 T:1003
Steps to reproduce the behavior
Have Libreelec 10.0.2 (Kodi 19.4.0)
Install 1.18.7+matrix.1 of the plugin
Startup Libreelec
:)
Debug log - mandatory
https://paste.kodi.tv/bezusacecu.kodi
Possible fix
No response
Additional context
No response
Screenshots
No response

I have/had the same problem with my Raspberry PI 3B+/4B with XBian/Debian Bullseye, Kodi 19 and 20.
I had done some troubleshooting in the routine in question using a
try:
...
expect:
return True
and the service started without any problems. Even after deleting the change, it still works. So I can't reproduce this again.
Everything is very strange

already received a similar report from other user the problem is weird,
the repository appears with a version string that is different from the original repo version string
from your log,
repository.castagnait v2.0.0+matrix installed
has +matrix suffix appended in version string
then return version v2.0.0+matrix
instead the repo file have the version without suffix, so:
repository.castagnait v2.0.0 installed
then return version v2.0.0
in my tests i am not able to reproduce this problem,
to me not append the suffix, i will try again
looks like a kodi bug in to manage version strings
sometimes append +matrix suffix other times no
or there are a particular situation when this is appied that i dont know
remove the suffix via code will fix the problem
but i would like to understand the reason why happens only in some cases

Same problem here on Raspi 4 with libreelec 9.2.8 (Leia).
Kodi-Log:
ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--

NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <type 'exceptions.ValueError'>
Error Contents: invalid literal for int() with base 10: '0+leia'
Traceback (most recent call last):
File "/storage/.kodi/addons/plugin.video.netflix/service.py", line 15, in  run(sys.argv)
File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/run_service.py", line 170, in run check_service_upgrade()
File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/upgrade_controller.py", line 54, in
check_service_upgrade_perform_service_changes(service_previous_ver, service_current_ver)
File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/upgrade_controller.py", line 98, in
_perform_service_changes migrate_repository()
File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/upgrade_actions.py", line 106, in migrate_repository
if not is_less_version(Addon('repository.castagnait').getAddonInfo('version'), '2.0.0'):
File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/common/misc_utils.py", line 181, in is_less_version
return list(map(int, version.split('.'))) < list(map(int, max_version.split('.')))
ValueError: invalid literal for int() with base 10: '0+leia'

I deinstalled the repo which solves the problem.

someone that has this problem on Kodi19 can update with this:
plugin.video.netflix_1.18.7+matrix.1_20220618_t1.zip
and let me know if works?

On Leia its Kodi 18 and netflix addon 1.12.11. Do you have a bugfix for this version.

@MarcusRoeckrath
leia test: plugin.video.netflix_1.12.11_20220618_t1.zip

Thanks; no crash anymore on leia.

someone that has this problem on Kodi19 can update with this: plugin.video.netflix_1.18.7+matrix.1_20220618_t1.zip and let me know if works?

Tested on Kodi19 and works again. Thanks!

someone that has this problem on Kodi19 can update with this: plugin.video.netflix_1.18.7+matrix.1_20220618_t1.zip and let me know if works?

I can verify it works with this one but did not with the automatically updated one (Kodi19).

released the version fixed

Can confirm everything working again after installing the update.

Upgraded to 1.12.12 - no problem at start of app.


summary of quadrilateral table

Is your feature request related to a problem? Please describe.

Want to add summary of quadrilaterals in form of table in properties of quadrilateral.

Describe the solution you'd like
It will we like this.
Please assign this issue to me.

Go ahead
Deadline 29 April,2021
Use Bootstrap Table classes


I want to make "GO TO CENTER OF IMAGE" button when zoom-in.

When zoom-in, the image of photoview will larger than mobile screen.
So, I want to make button that move mobile screen to center of the image, and the image still zoomed.
I have no idea which variable I have to set in OnClickListener.
Could someone guide me please.
just move mobile screen to specific position such as (100, 200) etc.

This is out-of-scope of this issue tracker


Work without explorer

Closes #92

@zargarzadehm can you tell us if your original request was because you also did not know that null could be given, or if you actually wanted to have no ExplorerApiClient initialization?

I think the previous code was ok, just sometimes it is necessary to use a special explorer (in my case I have a separate internal network for the development environment), I think by default there should be no need to enter the exploreUrl (The same Mainnet and Testnet are sufficient), so I think the following change is enough.
public static ErgoClient create(String nodeUrl, NetworkType networkType, String apiKey, String explorerUrl = "" `or null` )

But the current implemented logic is also a good logic.

@MrStahlfelge @zargarzadehm the conditional initialization of Explorer client may be useful in situation (hypothetical) when local node is accessed, but remote explorer is not available. It will still be possible to build a transaction and send it via Node API alone.

Yep, I understand that the change is useful. I think just a clarification on Javadoc RestErgoApiClient.create needed (or removing the need to call the static create method as an alternative).


"This page does not include AngularJS" error dialog popped out when click the "A" button on the toolbar of chrome.

chrome 49.0.2623.112 m
ng-inspector ng-inspector-0.5.8
$ npm -v
4.0.5
$ node -v
v4.4.4

With nothing happening on this project for over a year, I'm wondering if it's dead and only meant for Angular 1. React, Ember, and Vue all have great dev tool extensions, very strange that a Google product wouldn't have a terrific Chrome extension.


[SSPROD-25325] test: add functional testing framework

adds functional testing apparatus
adds one simple test example

Leverages a design principle that whatever CI does should also be reproducible locally. This means GHA shouldn't directly contain requisite setup configurations.

I assume motoserver  mocks aws resources for TF. How would we do it for GCP/Azure? On the same lines have you looked at terratest which doesn't mock but does actual aws based testing and i assume it has support for gcp/azure. I am not sure if e2e testing on actual cloud vs mock is something which has been discussed.

I assume motoserver mocks aws resources for TF. How would we do it for GCP/Azure? On the same lines have you looked at terratest which doesn't mock but does actual aws based testing and i assume it has support for gcp/azure. I am not sure if e2e testing on actual cloud vs mock is something which has been discussed.

This is an excellent observation, one that's been plaguing me since the inception of this service. No analogues of moto exist  for other clouds that I'm aware of.  We'll have to devise a different strategy, and I have do have some ideas, but none of them are that great.

I should note that the failing lint check is set to be fixed by another card


EdgeGaussians - 3D Edge Mapping via Gaussian Splatting

Short Summary - 3D Edge mapping can be cast as directly learning oriented points along the edges using supervision from multi-view 2D edge maps. Geometric regularization of 3D Gaussians for this task leads to a method that is efficiently trainable and produces state-of-the art results.
arXiv: https://arxiv.org/abs/2409.12886
Github: https://github.com/kunalchelani/EdgeGaussians
Might be a useful addition to the list?

Sorry for the late response. We will have a small "hackaton" next Thursday. The goal is to collaborate to fix some issues and to get it updated. We will add your paper as well.


[mini.test] Ability to change screenshot directory

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.test
Description
Currently expect.reference_screenshot places all screenshots in tests/screenshots/ directory. Also it accepts a complete path to the screenshot. I think it can be useful to define the default directory to store screenshots. And allow to override it per expect.reference_screenshot. Why? Because the logic to generate the unique filenames is still useful, but I would like to store screenshots closer to the actual tests, not in a big pile in one directory. Potentially the option can also be a function that returns a screenshot directory for the current test file.

Thanks for the suggestion!
I think this use case makes sense and code change is fairly minimal. I'll look into it.

This now should be possible on latest main with opts.directory option. Thanks again for the idea!


Labels info ommitted while inspecting self-defined network

Labels info included while executing the docker network inspect, and there are Labels info in other md docs such as networking.md:
$ docker network inspect my-network
[
{
"Name": "my-network",
"Id": "7m2rjx0a97n88wzr4nu8772r3",
"Scope": "swarm",
"Driver": "overlay",
"EnableIPv6": false,
"IPAM": {
"Driver": "default",
"Options": null,
"Config": [
{
"Subnet": "<IP_ADDRESS>/24",
"Gateway": "<IP_ADDRESS>"
}
]
},
"Internal": false,
"Containers": {
"404d1dec939a021678132a35259c3604b9657649437e59060621a17edae7a819": {
"Name": "my-web.1.63s86gf6a0ms34mvboniev7bs",
"EndpointID": "3c9588d04db9bc2bf8749cb079689a3072c44c68e544944cbea8e4bc20eb7de7",
"MacAddress": "02:42:0a:00:09:03",
"IPv4Address": "<IP_ADDRESS>/24",
"IPv6Address": ""
}
},
"Options": {
"com.docker.network.driver.overlay.vxlanid_list": "257"
},
"Labels": {}
}
]

LGTM

LGTM


Pretrained HyperNetwork model

Do you happen to have a Pretrained HyperNetwork?
I looked it up, but I can't see it. Can I get it??

still try to get usable result
it is quite difficult to reproduce the result

is hypernetwork dreambooth google collab available ,or any way i can try the that
Thank you

Would you be able to share the latest checkpoint even if its not useable yet?


feat: feature flag implementation

Issue #, if available:
feat: Add feature flag functionality to Amplify CLI.
Description of changes:
This PR adds a feature flag framework to Amplify CLI which enable the team to ship fixes and experimental features in the future.
Usage:
FeatureFlag class must be initialized with project path and a CLI environment name provider as a singleton. Because it needs to be accessible from GraphQL Transformers it cannot live in CLI context as of Today.
It supports number, boolean and string type feature flags.
Sample usage:
const transformerVersion = FeatureFlags.getNumber('graphQLTransformer.transformerVersion');
const isDefaultQueryEnabled = FeatureFlags.getBoolean('keyTransformer.defaultQuery');

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Scanned through the PR but it's not clear to me how this will be used in the future. Can you include a comment or edit the description with example usage?

@attilah @yuth did we have a review about the amplify.json file being added to Amplify projects? I'm afraid we already have too many .json files in the project root and one goal we have in native projects is to reduce that. The chosen name - amplify.json - is too generic, doesn't communicate its intent and ultimately it seems like it should live somewhere else, or at the very least be hidden by default (.amplifyfeatures.json maybe?).
Do we expect customers to modify that file? If so, can you point me to the docs?

@drochetti that was part of the design docs. Amplfiy CLI itself already has more files, this one is a start to try to unify more of the state into a single file, feature flags were just a starter, we plan to gradually move more config/state. Yes customers will be able to edit this file in this case to enable, disable features.
In this case feature flags must be available as soon as possible during CLI execution, even has to be supported for uninitialized projects (like to run a new init flow as a theoretic example), so putting into any dirs that does not exist is not the best way to go about it. In the node ecosystem it is pretty common to have project level config in the root of the project.
Here is the docs: https://docs.amplify.aws/cli/reference/feature-flags


[wavpack] Update to 5.7.0

Update Wavpack to 5.7.0

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] When updating the upstream version, the "port-version" is reset (removed from vcpkg.json).
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

The following usage passed on x64-windows:
wavpack provides CMake targets:

  # this is heuristically generated, and may not be correct
  find_package(wavpack CONFIG REQUIRED)
  target_link_libraries(main PRIVATE WavPack::WavPack)

wavpack provides pkg-config modules:

  # wavpack library
  wavpack


Add an alias and some related topics to babel

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

I was just looking at a repository that had the babel-preset topic. When I clicked the topic, I noticed the babel-preset topic page has no information about what Babel is. Listing babel-preset as a related topic of babel will change this.
I did a search for "babel" and added the most used babel-related topics to the babel topic index.md to encourage further use and awareness. https://github.com/search?q=babel&type=Topics

Thanks again @cheshire137!


Fix typing imports in extraction usecase

The person class here: https://python.langchain.com/docs/use_cases/extraction#pydantic-1 has attributes dog_breed and dog_name that use Optional from typing, but it hasn't been imported. Fixed the import here

Thanks @bholagabbar ! Great catch


Easy way to show notie on the bottom of the window.

I tried doing it with css overrides but it seems to break the hide functionality.  Is there an easier way?

Duplicate of #78


dat.writeFile() doesn't allow "filenames with spaces" which work fine on modern file systems

Operation System: ubuntu 17.04
Beaker Version: 0.7.3
to reproduce

manually create a file named: "a b.txt" content: test in the Site director
open it with beaker: dat://<key>/a b.txt
URL shows: dat://<key>/a%20b.txt

isn't nice but the file is reachable. FireFox does much better here.

now try this:
var archive = new DatArchive(datUrl)
await archive.writeFile('/b c.txt', 'world', {encoding: utf8})

I get:
err = InvalidPathError: Path contains invalid characters at createError (/home/mario/git/beaker/app/webview-preload.build.js:1555:16) at onCbReply (/home/mario/git/beaker/app/webview-preload.build.js:1538:17) at EventEmitter.onIPCMessage (/home/mario/git/beaker/app/webview-preload.build.js:1477:14) at emitMany (events.js:127:13) at EventEmitter.emit (events.js:204:7) {name: "InvalidPathError", invalidPath: true, stack: "InvalidPathError: Path contains invalid characters…27:13)↵ at EventEmitter.emit (events.js:204:7)", message: "Path contains invalid characters"}
401
        debugger
402
        return callback(err)})

expected

it should be possible to create any filename, that the OS allows

Yeah this is something that I did because I wanted Dat filepaths to always be hygienic inside of URLs. If we support spaces, we end up with %20 interspersed. That example would be /b%20c.txt
I recognize that might not be a good enough reason to disallow spaces. Happy to hear other opinions.

Oh yeah. As you pointed out, I cant really control this outside of the DatArchive API
Ok, I'll relax this constraint

That example would be /b%20c.txt
I recognize that might not be a good enough reason to disallow spaces. Happy to hear other opinions.

If you have a look at FireFox it displays the link nicely in the URL input area. .. If you copy paste the URL it inserts the url-encoding again. ... So the display is nice, but the URL is ugly :)
with tiddlywiki we have a function like this: $tw.utils.transliterate(baseFilename.replace(/ |<|>|\:|\"|\||\?|\*|\^/g,"_")); which replaces unwanted chars in filenames with underline.
I just added "space" again, which we removed some time ago, to fix my problem :)
This solution isn't perfect since eg: $:/aa and $/:aa will produce $__aa, which may overwrite content ... but we didn't get issues so far.

We also use transliterations for Cyrillic chars like so (list is shortened):
/*
Transliterate string from eg. Cyrillic Russian to Latin
*/
var transliterationPairs = {
	"Ё":"YO",
	"Й":"I",
	"Ц":"TS",
	"У":"U",
	"К":"K",
	"Е":"E",
	"Н":"N",
	"Г":"G",
	"Ш":"SH",
	"Щ":"SCH",

All the quirks is needed, since users want to use every possible character for their content titles, which are partially reflected in the filename ...
just some thoughts.

👍

thx ... will test


Duplicate movies in folder view

Hi, congratulations for the plugin, however I would like to report a bug that occurs when the folder view is active ... when entering a group folder, the films appear doubly duplicated.
The folder view is useful for grouping movies into custom collections that correspond to path folders where movies belonging to the same group are contained. Is there any way to fix this BUG without giving up folder view?

Hi, is there any solution for this problem? Tanks.

Hi, is there any solution for this problem? Tanks.

Hey i have also the Same Bug


Possible error suppression if the argument is not a number

If the argument passed to at is not a valid number, maybe because in an erroneous situation it has become null or undefined or NaN, the current spec has as a consequence the fact that the first element of the array (if it is not empty) is always returned:
[1,2,3].at(null) === 1;
[1,2,3].at(undefined) === 1;
[1,2,3].at(NaN) === 1;

This is weird, it's different than the semantic of the usual way to access arrays' values, and I think it could lead to subtle bugs because in some situations it could be considered as error supression IMO.

The problem is here https://tc39.es/proposal-relative-indexing-method/#sec-string.prototype.at, lines four, where this is used to convert the argument.

Yup, this is intentional - it matches the existing behavior of the other array methods, like .slice(). It is indeed different from normal member access, but consistency across methods was considered more important here.
(Those methods probably should have thrown when given an object that coerces to NaN, but alas we're two decades too late for a change like that.)

But we can choose to do better for new ones! @tabatkins

No, we can’t, because often inconsistency is worse than repeating decisions we wish hadn’t been made.

Yup. Future is longer than past, so if we think there's a larger body of future proposals that would benefit from this change, we could consider a precedent break. But (a) that's probably not the case here, we don't expect to continue to add a ton more array methods, and (b) it still needs to justify itself against the pain of inconsistency, and imo this doesn't meet the bar.


Building sqlproj fails in Windows

Azure Data Studio Version:
OS Version:

Steps to Reproduce:

Open a sql proj in vs code
build sql peoj

Expected: builds successfully
Actual: seeing this error:
.vscode\extensions\microsoft.sql-database-projects-vscode-0.13.0\BuildDirectory\Microsoft.Data.Tools.Schema.SqlTasks.targets" was not found.

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

@llali which platform is this on? Can you also share a screenshot with the error what's in the output?
on mac it looks for the targets file here:

and on windows here:

@kisantia the bug heppend in Windows. I didn't put the full path but it's looking for the build target in extension folder

if you check that folder, is the targets file there? Maybe sqltoolsservice wasn't finished downloading yet when build tried to find the file.

no the targets are not there. the buildDirectory folder only has systemDacpacs in it

@kisantia sqltoolsservice is installed. should I see the targets inside sqltoolservice folder?

huh that's weird. Yeah can you check if it's in the sqltoolsservice folder? It should be in
C:\Users\username\.vscode\extensions\ms-mssql.mssql-1.10.1\sqltoolsservice\3.0.0-release.124\Windows (or whatever the most recent sqltoolsservice version is)

my path is different that yours: ms-mssql.mssql-1.10.1\sqltoolsservice\1.10.0\Windows. maybe I'm using the wrong version of mssql?

Looks like there's a different sqltoolsservice version for dev and prod, so that'll need to be updated and tested before the extension gets released.
dev:
https://github.com/microsoft/vscode-mssql/blob/2e6ac4b633cc33383969b1ccfe001fa97fb6087f/src/configurations/dev.config.json#L3-L4
prod:
https://github.com/microsoft/vscode-mssql/blob/cbac9c44fc8cb8dca3c9ede450041cfa84fa180d/src/configurations/production.config.json#L3-L4
The last update to the prod version was November last year though, so it should have the necessary targets and other sql project build files. There might be a bug with copying over the files with that different path, but that would be a little weird if the dacpacs were copied over but the targets file wasn't :/

I just installed the same extensions in mac and everything works fine


[V2 data] Implement Core Data PUT /event/pushed V2 API

🚀 Feature Request
Relevant Package

 This feature request is for coredata
Description
 A clear and concise description of the problem or missing capability...
Per https://app.swaggerhub.com/apis-docs/EdgeXFoundry1/core-data/2.x#/default/put_event_pushed, implement the V2 API for
PUT /event/pushed

PR merged.  Close this feature request.