[FEATURE] Replace SocialTraderToken.sol?

File(s): SocialTraderToken.sol, Unknown File Replacement
Test Files Required?: Unsure
What Testing Should Include:

Unsure

The Social Trader Token model seems unnecessarily complicated. It is expensive to interact with. This could easily be done without the need for a social trading token. Instead, the SocialTraderToken.sol is replaced with another file that social traders use to execute moves and handle pooled funds.

If it is to be replaced, a lot of functions would be transferred over, so technically the replacement is just a rename of the file and stripping of ERC677 and underlying contracts.

If it is to be replaced, a lot of functions would be transferred over, so technically the replacement is just a rename of the file and stripping of ERC677 and underlying contracts.


No NgModule metadata found for 't'

Using angular-ssr 0.1.103 + Angular 4.1.3 + CLI 1.0.3 + rxjs 5.4.0 + zone.js 0.8.10 + TypeScript 2.3.2, the following is consistently reproducible when running ng-render. I also tried the two workarounds from #28 (tsconfig flattening, --module/--symbol) and got the same result.
Edit: also reproducible with 0.9.10.
gibson@2f931fc84db6:~/cyph/cyph.com$ ./node_modules/.bin/ng-render
[info] Rendering application from source (working path: /tmp/aa881fca662ab)
[error] Failed to render application: No NgModule metadata found for 't'. (use --debug to see a full stack trace)
gibson@2f931fc84db6:~/cyph/cyph.com$ ./node_modules/.bin/ng-render --debug
[info] Rendering application from source (working path: /tmp/cda25c7b0c1a2)
[error] Failed to render application: Error: No NgModule metadata found for 'AppModule'.
    at NgModuleResolver.resolve (/home/gibson/cyph/cyph.com/node_modules/@angular/compiler/@angular/compiler.es5.js:13884:12)
    at CompileMetadataResolver.getNgModuleMetadata (/home/gibson/cyph/cyph.com/node_modules/@angular/compiler/@angular/compiler.es5.js:14468:58)
    at JitCompiler._loadModules (/home/gibson/cyph/cyph.com/node_modules/@angular/compiler/@angular/compiler.es5.js:25635:61)
    at JitCompiler._compileModuleAndComponents (/home/gibson/cyph/cyph.com/node_modules/@angular/compiler/@angular/compiler.es5.js:25594:21)
    at JitCompiler.compileModuleAsync (/home/gibson/cyph/cyph.com/node_modules/@angular/compiler/bundles/compiler.umd.js:25561:23)
    at ServerPlatform.compileModule (/home/gibson/cyph/cyph.com/node_modules/angular-ssr/source/platform/platform.ts:33:21)
    at /home/gibson/cyph/cyph.com/node_modules/angular-ssr/source/application/builder/from-source.ts:31:22
    at ZoneDelegate.invoke (/home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:381:26)
    at Zone.run (/home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:141:43)
    at /home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:805:57
    at ZoneDelegate.invokeTask (/home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:414:31)
    at Zone.runTask (/home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:181:47)
    at drainMicroTaskQueue (/home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:574:35)
    at ZoneTask.invoke (/home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:480:25)
    at data.args.(anonymous function) (/home/gibson/cyph/cyph.com/node_modules/zone.js/dist/zone-node.js:1370:25)
    at FSReqWrap.oncomplete (fs.js:114:15)

I think this is related to your project's configuration. This does not happen in either of the example applications that are in https://github.com/clbond/angular-ssr/tree/master/examples.
Based on the error message, probably what is happening is that multiple instances of reflect-metdadata are being loaded into the process and you are therefore losing the decorator metadata on all of your modules and components. You can probably fix this by altering your webpack configuration.
I would suggest trying to find what is different between your repository and the applications in the examples folder.

Hmm, well, this is just an Angular CLI project without a custom webpack configuration (our production build script ng ejects and adds some custom stuff, but this is just after a standard ng build). Could this be caused just by a library that our code depends on importing reflect-metadata?
Thanks, I'll take a look at those examples.

@clbond, found the problem, and it wasn't related to my project in particular (the same issue was reproducible with the cli example): ng-render doesn't like that my Docker image has node_modules located at /node_modules.
Is this something you think would be fixable in angular-ssr? I could work around it on my end, but it'd be a bit of a hassle, so I'd prefer to avoid it if the issue can be better mitigated here.

node_modules is located literally in / at the filesystem root?


Point Empty has no WKB encoding, which breaks WktToWkb and WkbToWkt

One recommended workaround is to use Point(NaN, NaN) to represent Point Empty.
Still, special care is needed.
Reference:
https://gis.stackexchange.com/questions/109889/wkt-what-is-the-reasoning-behind-the-concept-of-a-point-empty

Dispatch this to GDAL. We are to move to WKB, so no need to worry.


Add timer for time spent in seconds

Adds a _seconds_spent property to each HIT. By default 0. Auto-increments every second HIT is onscreen.

We plan to use this for calculating average time spent on each sample. It is of course a noisy estimate because annotators may leave the tab open while doing something else. This is how label-studio for example does it, too.


Cleanup c make

+1

@nocnokneo This should be ready to merge now.  It would be nice to clear a few of these items so that they don't stack up.

I'm not a big fan of these large changes but I guess it's good to get up-to-date with ITK standard since RTK is meant to be ITK-like. Thanks for the effort.


zsh-vi-mode overrides word separator

By default (if I disable this plugin) zsh will treat each directory in the path /home/hubro/Dropbox/Config as separate words, so when I press Ctrl+w repeatedly, it will first delete "Config", then the "/", then "Dropbox" and so on.
When using zsh-vi-mode, pressing Ctrl+w will remove the entire path, treating it as one long word.
To test, type "cd foo/bar/baz" into the terminal and press Ctrl+w. Then, run zsh without plugins (zsh -f) and repeat the test.
I think zsh-vi-mode should keep the default, or at least provide a way to configure word separators. If this is already possible, it should be documented in the README.

Hi @Hubro
Thanks for the reporting and this issue has been addressed and fixed. Please update your plugin to the latest version.
Thanks and regards

@jeffreytse What has been done? 😄 I don't see anything new in the README and I don't see any new behavior:
https://user-images.githubusercontent.com/597206/109783893-c154a800-7c0a-11eb-941f-08980fd97e1d.mov

Hi @Hubro, I have fixed the issue about the ctrl-w behavior in the insert modee, the behavior follows the emacs ctrl-w.

@jeffreytse Is there any way to get this behaviour in zsh-vi-mode?
https://user-images.githubusercontent.com/597206/109804237-51eab280-7c22-11eb-9a4e-d1a220d37c4e.mov
In other words, when writing a long path, I want to be able to erase backwards one word at a time using Ctrl+w, rather than deleting the whole path. I don't want the forward slash character to be considered a "word character".

It seems like the old behaviour (backward-kill-word) works as expected if I set WORDCHARS myself. I didn't know about this variable when I opened this issue, and it solves my issue.
I would probably revert the default binding of ^W to backward-kill-word, as that's what most Vi users will expect.

Hi @Hubro
I have reverted the default binding of ^W to backward-kill-word, as that's what most Vi users will expect.
Thanks and regards

Hi @Hubro
Could this issue be closed now? :)
Thanks and regards

Yes, for sure!

@jeffreytse I currently have this issue. It occurs with both my regular config and a completely blank config (aside from sourcing zvm).


Can't assign to conditional generic within generic

Hi I was trying to do something much more complicated than this with conditionals without success, I was reducing the issue down and down and I found this, which I find a bit weird, I was hoping someone might be able to enlighten me of a work around?

TypeScript Version:  v4.1.2

Search Terms:
assign to conditional generic
Code
type Param<T> = T extends undefined ? {
    first: string,
} : {
    first: string,
}
function run<T>(param: T) {
    const p: Param<T> = {
        first: 'hi'
    }
    return p;
}

Expected behavior:
No type error
Actual behavior:
type error
Playground Link:
link here
Related Issues:
I couldn't find any, sorry if this is a well known problem / duplicate!

Duplicate of #26933. Search terms used: assign conditional


Hand of the Apprentice should be available as a Universalist 1st Level Class Feat

Hand of the Apprentice should be available as a Universalist 1st Level Class Feat. It's Uncommon, but only because it's only available to Universalist wizards. As it is, you have to add it as a Homebrew Item.

I've looked at the choiceset, and the issue is that the Universalist choiceset will only return wizard feats without a pre-requisite, rather than feats without a pre-req OR with a pre-req of universalist. I don't know how to tweak the choiceset to make it cover both options, but that's the source of the error.
{
  "adjustName": false,
  "choices": {
    "query": "{\"system.featType.value\":\"class\",\"system.level.value\":1,\"system.prerequisites.value\":[],\"system.traits.value\":{\"$elemMatch\":\"wizard\"}}"
  },
  "key": "ChoiceSet"
}

Resolved by https://github.com/foundryvtt/pf2e/pull/4734


Add Wall of Browser Bugs entries for Selectors Level 4 :nth-child(An+B of S)

I propose adding these on the grounds that the of S feature would allow us to make our widgets more robust. We've had several issue reports (e.g. #15684, #13825, #16540) where folks tried to put stuff like <input type="hidden"> within .input-group or .btn-group, but this broke the :first-child/:last-child-based selectors we use to adjust rounded corners.
With this feature, we could restrict the children considered to only the relevant ones, thus ignoring other irrelevant elements. For example, we could use :nth-child(1 of .btn) in our .btn-group CSS to ignore non-.btn children and thus round the corners correctly in such cases.
Refs:

http://caniuse.com/#feat=css-nth-child-of
https://drafts.csswg.org/selectors-4/#the-nth-child-pseudo
https://crbug.com/304163
https://bugzil.la/854148

CC: @twbs/team for review

👍
With regards to the feature: Is it backwards compatible? I presume it isn't.

AIUI, it isn't. We'd probably only be able to use it in the far future when IE has been dropped completely. But I think we'd like this feature to be ready by the time that world arrives.

@patrickhlauke @XhmikosR LGTY?

LGTM although I do feel it's a little early but it doesn't hurt.


Improve support for Unsecured JWT

This allows for symmetric support of Unsecured JWT. It now works as any other algorithm for both encoding and decoding.
Fixes #323.
https://tools.ietf.org/html/rfc7519#section-6 allows for Unsecured JWT using "alg" value of "none". It does not forbid further processing of Unsecured JWT and thus claim verification still applies.
https://tools.ietf.org/html/rfc7518#section-3.6 requires the signature to be an empty string.
https://tools.ietf.org/html/rfc7518#section-3.6 and https://tools.ietf.org/html/rfc7518#section-8.5 forbid accepting Unsecured JWT unless explicitly allowed (this has already been addressed).
While not explicitly addressed by any RFC, based on common sense this commit explicitly forbids the use of any signing key for Unsecured JWT. As Unsecured JWT requires an empty signature and cannot possibly make use of a key, it should be considered an error to use a key together with algorithm value of "none". For an Unsecured JWT the signing key should be set to either false or nil.

Hi @hesalx , just want to remind you that. If you put the issue link in the commit message. Every time you force-pushed will add a message automatically in the original issue page this

Maybe you could remove the link from your commit message if this is not the behavior you expect.

@hesalx Are you still active in the Ruby world? I'm thinking we could include the suggested improvements to the none algorithm in a 3.0 version of the gem.


Unable to bundle react-router

In one of the file I'm bundling there is the following ES6 import:
import Router from 'react-router';

Here is the bundler configuration:
{
  paths: {
    'react-router': 'file:/' + require.resolve('react-router/lib/index.js')
  }
}

The index.js from react-router is importing other files through require which are not resolved.
'use strict';

exports.DefaultRoute = require('./components/DefaultRoute');
exports.Link = require('./components/Link');
...

My first issue is related to the upgrade to version 0.14.3
In version 0.14.3, it triggers the following stack:
TypeError: Cannot read property 'substr' of undefined
    at Trace.getLoadRecord (C:\Users\User\MyProject\node_modules\systemjs-builder\lib\trace.js:67:16)
    at Trace.getAllLoadRecords (C:\Users\User\MyProject\node_modules\systemjs-builder\lib\trace.js:377:15)
    at C:\Users\User\MyProject\node_modules\systemjs-builder\lib\trace.js:387:21
    at Array.map (native)
    at C:\Users\User\MyProject\node_modules\systemjs-builder\lib\trace.js:386:96
    at runMicrotasksCallback (node.js:337:7)
    at process._tickCallback (node.js:355:11)

Looking at the code, the depMap attribute of the load object is empty.
In version 0.13.6, I had a nicer error:
Unable to calculate canonical name to bundle file:///C:/Users/User/MyProject/node_modules/react-router/lib/components/DefaultRoute

My second issue is related to the unresolved files
I'm trying to use paths to resolve the file by adding a .js at the end (I assume this is the reason why it is not resolved). I tried the following without luck:
{
  paths: {
    'absolute_path_to_react-router/*': 'absolute_path_to_react-router/*.js'
  }
}

Looking at utils.js method getCanonicalName in v 0.13.6 this is what I have:
p = file:///C:/Users/User/MyProject/node_modules/react-router/lib/*
curPath = file:///C:/Users/User/MyProject/node_modules/react-router/lib/*.js
normalized = file:///C:/Users/User/MyProject/node_modules/react-router/lib/components/DefaultRoute
normalized.substr(0, wIndex) = file:///C:/Users/User/MyProject/node_modules/react-router/lib/
curPath.substr(0, wIndex) = file:///C:/Users/User/MyProject/node_modules/react-router/lib/
normalized.substr(normalized.length - curPath.length + wIndex + 1) = ute
curPath.substr(wIndex + 1) = .js
Did I misunderstood the paths API?

Thanks for posting. I'm not sure I follow exactly your API use here, but it sounds like you're trying to load a NodeJS module that won't be able to load in SystemJS without custom package config that you need to set up to add defaultExtension and other such properties, or to have this done automatically with a tool like jspm.
The error reporting issue above has now been fixed in https://github.com/systemjs/builder/issues/317. Posting out a patch shortly.

OK I managed to load correctly the react-router dependency using the packages configuration:
{
  baseURL: '../../../node_modules',
  transpiler: 'babel',
  babelOptions: {
    modules: 'system',
    moduleIds: true,
    keepModuleIdExtensions: true,
    sourceMaps: true
  },
  packages: {
    'react-router': {
      main: 'lib/index.js',
      //format: 'cjs',
      defaultExtension: 'js',
      map: {
        'object-assign': './node_modules/object-assign/index.js',
        'qs': './node_modules/qs/lib/index.js'
      }
    },
    react: {
      main: 'dist/react.js',
      //format: 'global',
      defaultExtension: 'js'
    }
  }
}

However, the output file still contains require, process or module variables.
I'm getting the following error Uncaught ReferenceError: process is not defined or Uncaught ReferenceError: module is not defined if I force the format as shown in the commented lines.
Seems like I'm missing a part of the API. Any idea?
PS: I'm using systemjs 0.19.2 and systemjs-builder 0.14.5

Yes you would need to ensure the format: 'cjs' option is set, and also add the process global via a shim library that can provide it as expected in Node, something like:
'react-router': {
      main: 'lib/index.js',
      format: 'cjs',
      defaultExtension: 'js',
      map: {
        'object-assign': './node_modules/object-assign/index.js',
        'qs': './node_modules/qs/lib/index.js'
      },
      globals: {
        process: 'some-process-shim-library'
      }
    },

This is all because SystemJS uses a completely different resolution system to NodeJS modules. jspm can handle some of this stuff for you if you do jspm install npm:react-router. But if you keep going along this path, it will work out absolutely fine, and it looks like you've already made good progress too.
Just let me know if you need any more help here at all.

Unfortunately I cannot access jspm/nodelibs shims for process or module because they are not available on npm registry and I could not find one for module.
This is where you could ask me why don't you use jspm? because I would like to use a bundler in a gulp task without having to maintain a config file or launch a CLI (as you could do with browserify for example). I don't think I can do that with jspm, but correct me if I'm wrong.

The shims to use are the Browserify shims. These are listed at https://github.com/substack/node-browserify#compatibility and can be installed through npm and configured to be used via the package configs. Note that the shim packages themselves will also in turn need to be configured with package configuration in this way.


Base registration component handler doesn't care about non-200 results

If my MFA component's backend RegisterHandlerInterface implementation throws an exception during the register() method, the result is a 400 response, but the frontend components don't check for it and continue on to generating backup codes. This leaves the user registered with only back up codes. It should return an error on the frontend instead and allow the process to continue.
Acceptance criteria

Throwing an AuthenticationFailedException (doesn't exist in master at time of writing) from a RegisterHandler::register() method will be caught by the MFA frontend app
As above, will not proceed to generation of backup codes
As above, will pass the exception's message down to React registration components as a prop (e.g. { hasError: true, errorMessage: $ex->getMessage() } (pseudo code - example only)

PRs are at https://github.com/silverstripe/silverstripe-mfa/pull/87 and https://github.com/silverstripe/silverstripe-totp-authenticator/pull/8.
Summary of changes:

We now catch non 201 responses in the Register component and pass the error message through to the method Register component (e.g. TOTP) as a prop
TOTP converts it to state, since it internally has its own second screen to manage, so cannot modify its own props
TOTP resets its error in state when going back from TOTP screen 2 (validate) to TOTP screen 1 (scan)
Base MFA Register component resets its error message when going back from the method's Register component to the SelectMethod component (so they don't bleed between registration methods)
Base MFA Login component resets its error message when switching between methods in more options menu (#80)


"localhost" is not considered a valid host for "openIdConnectUrl"

Problem
Linter doesn't like localhost in openIdConnectUrl
# File: openapi.yml
  securitySchemes:
    localAuth:
      type: openIdConnect
      openIdConnectUrl: http://localhost:9001/auth/realms/go-commerce/.well-known/openid-configuration

  Message :   openIdConnectUrl must be defined for openIdConnect property and must be a valid URL
  Rule    :   builtin
  Path    :   securitySchemes.localAuth
  Line    :   0

Workaround
A simple workaround is to use <IP_ADDRESS> instead.
Possible solutions
After some digging in the source code, I see you're using the helper isURL from package validate to check this url: https://github.com/IBM/openapi-validator/blob/main/src/plugins/validation/oas3/semantic-validators/security-definitions-ibm.js#L168
I see 2 solutions to this issue:

Use option { host_whitelist: ['localhost'] } as second argument to isURL. This seems the cleanest solution to me. Unfortunately, there is a related issue and PR. The fix is not yet merged on the latest release, but the option could already be provided. Therefore, the issue will be automatically solved when the version of validate can be bumped.
Use option { require_tld: false } as second argument to isURL. This will immediately solve the issue, but also allow non fully qualified domain names. Depending on how strict you want the check to be, this might be a show stopper.

Seems validator will never put it as default as it's not valid accordingly to the RFC. See https://github.com/validatorjs/validator.js/issues/1609 for more info.
Are you open to consider { host_whitelist: ['localhost'] }? If not, feel free to close this ticket.

Hey @JenswBE after some internal discussion, we decided not to support localhost as a valid URL. Thank you for the issue!


fix(array-repeat-strategy): prevent array being manipulated during an…

…imation
This will make a copy of the array to prevent it being manipulated by
another set of changes during animation.
Fixes #183

Thanks for working on this @martingust
Before we release this, I'd like to know that this isn't going to cause us any performance or memory pressure issues. @jdanyow Can you take a look at this? Can we run some benchmarks and take a look at how this affects dbmonster as well? I'd hate to do something that set us back in a big way. If this is only needed when animation is involved, maybe we can only clone the array in those circumstances?
What do you all think?

@EisenbergEffect Good points. We should probably only clone the array if animation is involved. I don't think this code will get hit in the dbmonster though. @jdanyow Thoughts?

Just ran some very basic heap profiles and timeline in chrome dev tool, adding 1 item every 1ms till 1000 items, could not see any noticeable difference between cloning the array or not. Not sure if I tested the right thing here though. It's going to be interesting to see @jdanyow take on this or some better benchmarks.

That's a good sign. Let's see if @jdanyow has anything to add.

This looks good to me.  If you really wanted to avoid the slice you could do something like this:
if (repeat.viewsRequireLifecycle) {
  ... slice it up ...
}

viewsRequireLifecycle will be true when the template uses animation:  https://github.com/aurelia/templating-resources/blob/master/src/analyze-view-factory.js#L45

@jdanyow Nice!
@martingust Can we make that change? If so, I can get it merged and released tonight.
Thanks guys!

As it turns out, the lifecycle check is in another method so when this method is called we already are sure that we need the lifecycle. I just left it as is :) Releasing shortly.
Good job @martingust This was a big one to squash!


Audiotica: New Warning - Input string was not in a correct format. [mscorlib]

Warning was reported on 1/20/2015 6:44:14 PM, exception type: System.FormatException, message: Input string was not in a correct format., details at https://insights.xamarin.com/app/07441d5120614908a18b36c146930559f0120995/issues/590974bbfcc1ffa9bb56313c24996d4a8618bb0e

When attaching to downloads it tried to parsed the file name as an int (id used in table) but current version is no longer using that file convention.
See c46a1a9


labad: Renamed 'small' to 'smallv' to avoid problems with define in rpcndr.h on MSVC

This PR solves a similar problem as https://github.com/xtensor-stack/xtensor-blas/pull/149 before, again on MSVC. This time the error comes from the header C:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared\rpcndr.h on MSVC that has a define
#define small char

which breaks arguments in labad.h and labad.tcc.

This should be pretty harmless for a quick merge... :-)


Title

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version:
Python version:
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Hello @abuoeissa9 ,
I request you please follow the instructions mentioned in installation document for installing the latest tensorflow
version 2.8.Thanks!


Cannot use dict as a type

from pydantic import BaseModel, Field
from funcchain import chain

class Output(BaseModel):

    plot: str
    kwargs: dict = Field(default_factory=dict)

def get_kwargs(user_query: str) -> Output:
    """
    Translate the user query to a dictionary of kwargs
    """
    return chain()

get_kwargs("scatter(color='red')")

Results in
AssertionError: Unrecognized schema: {'title': 'Kwargs', 'type': 'object'}

thanks will look into it! you can define other pydantic models and put them as type instead tho which is like a dict

Is it possible to define a model with unlimited/unexpected fields in pydantic, equivalent to **kwargs?

yeah true this would be a usecase for a pure dict as type

just pushed a fix that adds this to the schema converter for grammars!
the only issue is that the grammar is very open so the model might stumble across problems because its not tooo guided


Revert "Handle tap migrations in CaskLoader."

Reverts Homebrew/brew#16596 - PR seems to have broken some third-party taps

Closes https://github.com/Homebrew/brew/issues/16607

CC: @reitermarkus


VendorImporter doesn’t have find_spec() method

sys.meta_path entries should have a find_spec() method. As a fallback, importlib calls find_module(), but other users might not.

Thanks manueljacob. Would you consider putting together a patch?

Actually, let's put this effort on hold until #2052 is resolved, as that may remove the VendorImporter entirely.

Thanks. For now, I worked around the problem. For reference: https://www.mercurial-scm.org/pipermail/mercurial-devel/2020-May/141648.html


Clarify Python parameters behavior

Follow up to https://github.com/iterative/dvc.org/pull/2772.
I have a few additional questions:

Is there background on why Python parameter support was added (cc @skshetry)? If so, maybe we can explain a typical usage for it in the example? I guess it might be easier to import from a Python file to a Python script than to read from YAML, but I'm not sure I can think of another use.
Should complex expressions be ignored or raise a warning?
Should we add a note to https://dvc.org/doc/user-guide/project-structure/pipelines-files#parameter-dependencies?

Should complex expressions be ignored or raise a warning?

I think that we should start raising a warning, at the very least. In other parameter file formats, an invalid parameter will cause an exception to be raised.

background on why Python parameter support was added

I think it was done just since it's native to DVC and a common format in the space. Idk if it's actually used. I've mainly noticed people using yaml and json for this. And if you do have params in a Python file it's probably the same file script used for the stage cmd so you probably can't even define it as a params file.
I imagine "typical usage" would be when a project has a separate python config file (common in web frameworks for example although that's not a good example here of course). Their values may even be loaded from env vars.

Should complex expressions be ignored or raise a warning?

Raising a warning seems logical except if you're loading a huge script that only has a few globals defined at the beginning and then hundreds or more lines of code. Maybe just raise a single warning irrespective of how many complex expressions are being ignored?

Should we add a note to https://dvc.org/doc/user-guide/project-structure/pipelines-files#parameter-dependencies?

No need I think. It's already in https://dvc.org/doc/command-reference/params

Raising warnings sounds ideal

p.s. this would also be a core issue though. So for this ticket we may have left just improving the motivation and example(s) given?

@dberenbaum feel free to put tickets that are specifically C: ref like this one in your back log, or manage them (mark stale, close, etc.)

Sorry, @jorgeorpinel, too many tickets to manage 😓 . Happy to close this as stale. I'll try to take a look at other issues of mine when I have a chance.

Sorry didn't mean to pressure @dberenbaum . This is not THAT old (< 1 Y). Could keep it open for a few more months too. Up to you

No problem at all, thanks for the reminder. Like I said, happy to close this one.


Galactic ➡ Humble

➡ Forward port
Port galactic to humble
Branch comparison: https://github.com/gazebosim/ros_gz/compare/humble...galactic
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

Might want to check off DCO.
Some tests are failing in fortress+humble:

test_launch_test_gz_subscriber.launch.py, which I guess is fine
In cppcheck_rosidl_generated_c, couple of Converting unsupported gz::msgs::Any [QUATERNIOND] failed

Might want to check off DCO.

Done.

Some tests are failing in fortress+humble:

Must have been flakes.


Add keepalive support (periodic ping messages)

I still think that this feature makes perfect sense :+1: However, there are currently no immediate plans to build this from my end (no demand at the moment), but I would be really happy to accept PRs :+1:
(If you need this for a commercial project and you want a quote, please check out my profile and send me an email)


libfreerdp2-2 (from 2.0.0~rc2) Makes vinagre segfault on authentication failure

This bug was reported to Debian's BTS after upload of 2.0.0~rc2 to Debian (see [1]).
After upgrading libfreerdp2-2, authentication failures (mistyped password) started causing segfaults:
May 11 11:41:29 jtriplet-mobl2 vinagre.desktop[9277]: [11:41:29:080] [9277:9277] [ERROR][com.freerdp.core] - freerdp_set_last_error ERRCONNECT_LOGON_FAILURE [0x00020014]
May 11 11:41:29 jtriplet-mobl2 vinagre.desktop[9277]: [11:41:29:080] [9277:9277] [ERROR][com.freerdp.core.rdp] - rdp_recv_callback: CONNECTION_STATE_NLA - nla_recv_pdu() fail
May 11 11:41:29 jtriplet-mobl2 vinagre.desktop[9277]: [11:41:29:080] [9277:9277] [ERROR][com.freerdp.core.transport] - transport_check_fds: transport->ReceiveCallback() - -1
May 11 11:41:29 jtriplet-mobl2 kernel: vinagre[9277]: segfault at 9 ip 00007fbc84bf6ab0 sp 00007ffc05377a40 error 4 in libfreerdp2.so.2.0.0[7fbc84b1c000+137000]

The issue can easily reproduced, just by entering an incorrect username and password in vinagre:
Thread 1 "vinagre" received signal SIGSEGV, Segmentation fault.
clear_context_free (clear=0x1) at ./libfreerdp/codec/clear.c:1216
1216	./libfreerdp/codec/clear.c: No such file or directory.
(gdb) bt full
#0  0x00007ffff528bab0 in clear_context_free (clear=0x1) at ./libfreerdp/codec/clear.c:1216
        clear = 0x1
#1  0x00007ffff522a9cd in codecs_free (codecs=0x555555dd62b0) at ./libfreerdp/core/codecs.c:213
#2  0x00007ffff5224c77 in freerdp_disconnect (instance=0x555555d14d00) at ./libfreerdp/core/freerdp.c:508
        rc = 1
        rdp = <optimized out>
#3  0x0000555555584769 in vinagre_rdp_tab_dispose (object=0x555555cfe920) at plugins/rdp/vinagre-rdp-tab.c:182
        rdp_tab = 0x555555cfe920
        priv = 0x555555cfe730
#4  0x00007ffff5be1e03 in g_object_unref () at /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
#5  0x00007ffff6b5da39 in gtk_container_remove () at /usr/lib/x86_64-linux-gnu/libgtk-3.so.0
#6  0x0000555555570748 in vinagre_notebook_close_tab (nb=0x555555b06230, tab=0x555555cfe920) at vinagre/vinagre-notebook.c:697
        position = 0
        notebook = 0x555555b06230
        previous_active_tab = 0x555555cfe920
        __func__ = "vinagre_notebook_close_tab"
#7  0x0000555555583074 in idle_close (tab=0x555555cfe920) at plugins/rdp/vinagre-rdp-tab.c:272
#8  0x00007ffff59030f5 in g_main_context_dispatch () at /usr/lib/x86_64-linux-gnu/libglib-2.0.so.0
#9  0x00007ffff59034c0 in  () at /usr/lib/x86_64-linux-gnu/libglib-2.0.so.0
#10 0x00007ffff590354c in g_main_context_iteration () at /usr/lib/x86_64-linux-gnu/libglib-2.0.so.0
#11 0x00007ffff5ec3cdd in g_application_run () at /usr/lib/x86_64-linux-gnu/libgio-2.0.so.0
#12 0x00005555555655cf in main (argc=1, argv=0x7fffffffdf68) at vinagre/vinagre-main.c:196
        app = 0x5555557e91a0
        res = <optimized out>

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=898448

Please reopen this issue, it is not fixed, yet.

@sunweaver found the issue, looks like vinagre does session disconnect regardless of connection state. The behaviour slightly changed since freerdp 1 and was buggy for quite some time in 2 (which is why this might have gone unnoticed)


panic client.PubObject with body length 0

Output of restic version
restic 0.6.1
compiled with go1.8.3 on linux/amd64
How did you start restic exactly? (Include the complete command line)
restic backup /var/lib/vz/dump
What backend/server/service did you use?
Amazon S3
Expected behavior
To backup my data
Actual behavior
I'm getting a panic while trying to backup 83GB of vm dumps to Amazon S3.
scan [/var/lib/vz/dump]
scanned 1 directories, 22 files in 0:00
panic: client.PutObject: Put https://hugops-proxmox.s3.amazonaws.com/restic/data/03a04ac59d081bae84e4d78724936a253981364413d1268e179724549e726dd9: http: ContentLength=6960726 with Body length 0

goroutine 50637 [running]:
restic/archiver.(*Archiver).saveChunk(0xc420178690, 0x19761070f, 0x564733, 0x96dfcfea00000, 0xc437798000, 0x564733, 0x5da000, 0xc42009a790, 0xb76400, 0xc42051c030, ...)
        src/restic/archiver/archiver.go:163 +0x270
created by restic/archiver.(*Archiver).SaveFile
        src/restic/archiver/archiver.go:237 +0x381
using parent snapshot d2dcc97c

Steps to reproduce the behavior
restic backup /var/lib/vz/dump with a bunch of proxmox vms
Do you have any idea what may have caused this?
Not sure. Maybe the size of the Objects?
$ cd /var/lib/vz/dumps
$ du -hs *

4.0K    vzdump-qemu-100-2017_06_03-08_39_42.log
5.4G    vzdump-qemu-100-2017_06_03-08_39_42.vma
4.0K    vzdump-qemu-100-2017_06_22-04_25_55.log
5.0G    vzdump-qemu-100-2017_06_22-04_25_55.vma.lzo
4.0K    vzdump-qemu-101-2017_06_03-08_42_49.log
8.4G    vzdump-qemu-101-2017_06_03-08_42_49.vma
4.0K    vzdump-qemu-101-2017_06_22-04_27_44.log
7.3G    vzdump-qemu-101-2017_06_22-04_27_44.vma.lzo
4.0K    vzdump-qemu-102-2017_06_03-08_48_34.log
3.3G    vzdump-qemu-102-2017_06_03-08_48_34.vma
4.0K    vzdump-qemu-102-2017_06_22-04_29_28.log
2.6G    vzdump-qemu-102-2017_06_22-04_29_28.vma.lzo
4.0K    vzdump-qemu-103-2017_06_03-08_50_33.log
26G     vzdump-qemu-103-2017_06_03-08_50_33.vma
8.0K    vzdump-qemu-103-2017_06_22-04_38_09.log
23G     vzdump-qemu-103-2017_06_22-04_38_09.vma.lzo
4.0K    vzdump-qemu-104-2017_06_03-09_00_03.log
4.0K    vzdump-qemu-104-2017_06_03-09_11_01.log
4.0K    vzdump-qemu-105-2017_06_22-04_48_19.log
2.4G    vzdump-qemu-105-2017_06_22-04_48_19.vma.lzo
4.0K    vzdump-qemu-107-2017_06_22-04_49_09.log
2.0G    vzdump-qemu-107-2017_06_22-04_49_09.vma.lzo

Thanks for leaving an issue. I've seen this error before, it is caused by S3 closing the connection before data could be sent. What's you upstream bandwidth to s3? Could you retry with with the master branch and -o s3.connections=2? Sometimes this error is caused by having too many open connections to s3 so that one connection starves and doesn't get any data through...

root@proxmox:/usr/local/bin# restic -o s3.connections=2 backup /var/lib/vz/dump
scan [/var/lib/vz/dump]
scanned 1 directories, 56 files in 0:00
panic: client.PutObject: invalid argument73.098 GiB  10 / 57 items  0 errors  ETA 38:55
goroutine 1511 [running]:
restic/archiver.(*Archiver).saveChunk(0xc420286eb0, 0xc1e4e0, 0xc4200123d8, 0x6c4dba9, 0x477d00, 0x7aabe25f00000, 0xc42e8b2000, 0x477d00, 0x4ba000, 0xc420352b00, ...)
        src/restic/archiver/archiver.go:165 +0x395
created by restic/archiver.(*Archiver).SaveFile
        src/restic/archiver/archiver.go:239 +0x3a4

I have 1Gb up.

Another note this backup seems to always fail around the 70gb mark every time.
Today I decided I would try B2 Cloud since that is what I wanted to use anyways and now I got master installed I could use it.
I uploaded 213gb without any errors using B2 Cloud. I can still help debug with s3 though that data hasn't gone away yet.

Hm, very strange. I cannot reproduce it and I'm out of ideas what's the problem here. minio-go (the library we're using for accessing s3) has merged a fix for a similar issue. I've created PR #1066, could you please test branch update-minio-go and retry?

I encountered this issue today. It happens when I'm trying to do a first backup of a large directory (20+ GB).

@fd0 do I just pull & build restic again to get the new minio-go?

pull and checkout the branch with git checkout update-minio-go, then build restic with go run build.go and use ./restic to run.

This seems to have fixed it for me at least. I ran it with the old version before I tried the new one. Four times in a row it died at the 70Gb mark. Tried the new binary uploaded 310Gb without fail. Thanks for the help!

I'm glad it helped, thanks for the feedback!

I've merged #1066, please leave a comment if this happens again to anybody. For now, I'm going to close this issue.


removed packages that weren't being used

So the build is failing according to github.
I opened an issue related to this and investigated
According to this:
https://github.com/microsoft/node-pty/issues/319
It's something to do with node v 12 and windows builds.
They recommended installing node-pty@beta as a dev dependency which I did.
I'll review the travis logs to see if this resolves that.
The bundlesize command under package.json wasn't working.
updated bundlesize.config.json to accept module blobs
path": "dist/**/*.js",

tested and confirmed it can now review the bundle sizes of the dist folder.
removed lodash package from package.json
and added the two individual modules that are being used in this project
.isEmpty & .map
removed giphy package as it wasn't being used
giphy-api is being used, left as is.
removed electron-debug package as it wasn't being used.
switched .travis.yml back to use npm run lint since yarn doesn't seem to be configured with this project.
tested this by cloning a fresh project down to computer when you run yarn start
yarn: command not found

Let me know if there is anything else that needs to be changed!

Looks good to me! Thanks a lot for your work! @dhuang612! 👍

Hey @joshghent, this looks  fine by me. I won't merge it until you approve it too though. Feel free to merge yourself if you approve 👍
@dhuang612 Thanks for the help here 👍 Good stuff.

@jamescallumyoung thanks for reviewing that mate 👍 Great work @dhuang612 🎉


TPU Socket Closed

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
Platform: Colab TPU
TensorFlow version (use command below): 2.1.0rc1

Describe the current behavior
TPU_Socket_Traceback.pdf
Using model.fit raises an error which does not occur with GPU backend.
UnavailableError: Socket closed Additional GRPC error information: {"created":"@1579711911.023658059","description":"Error received from peer","file":"external/grpc/src/core/lib/surface/call.cc","file_line":1039,"grpc_message":"Socket closed","grpc_status":14}
Describe the expected behavior
Code should run the same in both instances.
Code to reproduce the issue
https://colab.research.google.com/drive/156Q2BsM9gVS3cvDytovQzEGKmjJpP4s8
Other info / logs
Attached.

same error here when I use tpu on colab, and  any update?

 So this error message strongly suggest that there's something wrong in your implementation, especially the parts that invokes host machine functions.

Any chance we can get an update on this? It seems like a pretty fundamental problem for using TPUs, and support was introduced a while ago suggesting it should work in simple cases at the least. Just to know that some progress is being made, or someone is working on this, would be great.

Any chance we can get an update on this? It seems like a pretty fundamental problem for using TPUs, and support was introduced a while ago suggesting it should work in simple cases at the least. Just to know that some progress is being made, or someone is working on this, would be great.

Hi @Ryandry1st ,
Took a quick look at your code and this seems to be the same problem as Issue #36996 .
I changed your code to
x = np.random.rand(10000, 5, 2).astype(np.float32) y = np.random.rand(10000, 1).astype(np.float32)
and everything works as expected:

Train on 10000 samples
Epoch 1/10
10000/10000 [==============================] - 5s 547us/sample - loss: 0.1424 - mse: 0.1419
Epoch 2/10
10000/10000 [==============================] - 2s 164us/sample - loss: 0.0872 - mse: 0.0872
...

Also there's a small typo, it should be metrics=['mse'].
Hope that helps!

Yes, it appears that issue is in reference to the same problem. It is interesting that the issue seems to be solvable by simply casting the data to float32.
Also trying to cast the data into other types causes it to break in different ways as well:

float64 leads to the same socket closing error
float16 produces a warning about missing an Identity for the OpKernel for TPU for DT_HALF, although in the list of included OpKernels, DT_HALF is available.
int16 does the same as float16
int32 does the same as int16 and float16
unsigned integers have the same output as signed integers
complex64 has the same missing OpKernel
float128 does not have an associated TF data type
booleans also have the missing OpKernel

So it appears that we are limited to float32 representations for everything.
Also the typo was intentional, I was desiring to see the mean absolute error as well.
Here is the full error for other types besides float32:
No registered 'Identity' OpKernel for 'TPU' devices compatible with node {{node Identity}}
(OpKernel was found, but attributes didn't match) Requested Attributes: T=DT_HALF
.  Registered:  device='XLA_GPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_UINT8, DT_INT16, ..., DT_HALF, DT_UINT32, DT_UINT64, DT_RESOURCE, DT_VARIANT]
device='XLA_CPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_UINT8, DT_INT16, ..., DT_HALF, DT_UINT32, DT_UINT64, DT_RESOURCE, DT_VARIANT]
device='XLA_TPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_COMPLEX64, DT_INT64, DT_BOOL, DT_BFLOAT16, DT_UINT32, DT_UINT64, DT_RESOURCE, DT_VARIANT]
device='XLA_CPU'; T in [DT_UINT8, DT_QUINT8, DT_UINT16, DT_INT8, DT_QINT8, ..., DT_DOUBLE, DT_COMPLEX64, DT_COMPLEX128, DT_BOOL, DT_BFLOAT16]
device='TPU'; T in [DT_INT32, DT_UINT32, DT_BFLOAT16, DT_FLOAT, DT_DOUBLE, DT_BOOL, DT_COMPLEX64, DT_INT64, DT_UINT64]
device='TPU_SYSTEM'
device='GPU'; T in [DT_HALF]
device='GPU'; T in [DT_BFLOAT16]
device='GPU'; T in [DT_FLOAT]
device='GPU'; T in [DT_DOUBLE]
device='GPU'; T in [DT_INT64]
device='GPU'; T in [DT_UINT16]
device='GPU'; T in [DT_INT16]
device='GPU'; T in [DT_UINT8]
device='GPU'; T in [DT_INT8]
device='GPU'; T in [DT_COMPLEX64]
device='GPU'; T in [DT_COMPLEX128]
device='GPU'; T in [DT_VARIANT]
device='DEFAULT'; T in [DT_STRING]
device='DEFAULT'; T in [DT_VARIANT]
device='DEFAULT'; T in [DT_RESOURCE]
device='CPU'
 [[Identity]] [Op:PrefetchDataset]

Is there any new progress here? I attempted to run the gist after updating tensorflow to 2.2rc0, however, this fails in tpu initialization, so I cannot test newer versions for a fix.
It has been 2 months though and this encompasses all tpu use on colab that does not use purely 32-bit floating point data, so it is a pretty widespread issue.

I am facing a similar issue, for a different script, I am beginning to think that when I use TPU's through google colab, the underlying engine that manages TPU instances just cuts out one of the TPU workers, in an attempt to manage loads, I am not a premium colab user, and colab preimum isnt available in my location yet... and as wild guess this issue would not appear for colab preium users, since they are promised dedicated instances for longer.
The same script works on Kaggle kernels though

I suspect this issue is a memory leak.
In my experience I'm seeing a constant slow rise in memory usage while training, until the TPU crashes:

This appears to be an exception raised in a variety of different situations, so it can be quite difficult to debug. Things to try that've solved issues for me thus far:

Cast everything to tf.float32 (i.e. if working with images, using tf.uint8 is a non-starter)
Be careful using tf.dataset.Dataset.cache with large datasets ( @mxbi thanks for the inspiration - this solved my most recent issue).
Use from_logits=True when using a categorical crossentropy loss.
Upgrade to Colab Pro for less disconnects. They do disconnect runtimes from time to time.

I'm sure I'll be back to add more. Would be incredible to have insightful error messages on these.

I reran the gist using tf2.2 and found that it runs. I also tried the various data types and it seems to be running now under most cases. The cases that I found it did not work were for:

uint8, uint16, uint64
int8, int16

All other cases worked including booleans and half/double precision floats. When an error occurs it is for a missing OpKernel  type as mentioned earlier.
So it seems like the issue is resolved. An additional issue could (should) be raised to handle the missing datatypes especially since uint8 is pretty fundamental to image processing.

I have the same error when using TPU on TF2.2. It is not an error caused by model implementation because, trying on CPU, it works fine. I'm currently trying to run the code with the following data preparation steps:
bs = 128

train_data = tf.data.Dataset.from_tensor_slices((X_data_training.astype('float32'), y_data_training.astype('float32')))
train_data = train_data.shuffle(len(X_data_training)).repeat()
train_data = train_data.batch(bs)

val_data = tf.data.Dataset.from_tensor_slices((X_data_validation.astype('float32'), y_data_validation.astype('float32')))
val_data = val_data.shuffle(len(X_data_validation))
val_data = val_data.batch(bs)

I've also tried to change data types but without any success. I get the same error even feeding to the fit function the data without using tf.data.Dataset.from_tensor_slices. Any idea about how to quick solve this issue?

I am not sure, I have not run into that problem since updating to TF 2.2, although I do use the rc4 which may have a difference. Looking at the snippet you provided, I cannot see any reason to get the error. It also looks like it is something solved, or believed to be solved, in TF 2.3 coming to release candidate soon?

Hi, unfortunately, I've just tried the same piece of code after upgrading both my GCP VM and the TPU to TF2.3-rc2 and I am getting the same kind of error, even if with some more details (which are certainly not insightful from my perspective):

UnavailableError: Socket closed
Additional GRPC error information from remote target /job:worker/replica:0/task:0:
:{"created":"@X.X","description":"Error received from peer ipv4:X.X.X.X:X","file":"external/com_github_grpc_grpc/src/core/lib/surface/call.cc","file_line":1056,"grpc_message":"Socket closed","grpc_status":14}

I hit the same issue with 2.2, 2.3 and 2.4 nightly, however it was due to me trying to cache a large dataset into memory, so please do check if you are trying to cache a large dataset!

I hit the same issue with 2.2, 2.3 and 2.4 night ly, however it was due to me trying to cache a large dataset into memory, so please do check if you are trying to cache a large dataset!

How can I check and mitigate this? Apologies if I am new to this.


Version bump

Could you please bump the version so that it is possible to use the latest patch for the download.js script?

Updated - thanks for the reminder.


Fix robots.txt setAllow() and setDisallow()

1. Why is this change necessary?
Calling setAllow() or setDisallow() for a route tthat is currently disallowed respectively allowed has not the expected effect.
2. What does this change do, exactly?
It ensures that an allowed route is output in /robots.txt even if it was previously disallowed and vice versa.
3. Describe each step to reproduce the issue or behaviour.
Open /robots.txt. There should be an entry Allow /widgets/ because of https://github.com/shopware/shopware/blob/fba270614c5007167774a9ddb64b1d71eea0dee7/themes/Frontend/Bare/frontend/robots_txt/index.tpl#L26
But it is missing because it has been previously disallowed by https://github.com/shopware/shopware/blob/fba270614c5007167774a9ddb64b1d71eea0dee7/themes/Frontend/Bare/frontend/robots_txt/index.tpl#L13
The entry {$robotsTxt->setAllow('/widgets')} is also a bug I think and should be fixed in a separate PR.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-26024
Please use this issue to track the state of your pull request.

Thanks for your Contribution! 💙


10207: add CDC elims metabase queries to source control

This PR adds the queries written for the CDC ELIMS team to source control
Test Steps:

Go to https://staging.prime.cdc.gov/metabase/collection/49-cdc-elims-collection and check out all the queries

As a note @arnejduranovic  has already tested these as part of figuring out the functionality
Changes

Adds the queries for the CDC ELIMS team

Checklist
Testing

[x] Tested locally?
[ ] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

#10207

To Be Done
Specific Security-related subjects a reviewer should pay specific attention to

this is out of scope of this PR, just a fyi:
got syntax error when I ran after putting in some random parameter value:

@JFU-NAVA-PBC Updated that query to work around a metabase bug, so this is good to go


Add text search functionality

Add a text search box.
Ideally the search and filters will be applied to the document list when a search is run.
It should be an AND operation.

Search text is ANDed with the filter options


Unable to log in

I think you are posting questions in the wrong repo. You might want to ask in https://github.com/Hacker-GPT/HackerGPT-2.0 instead.
Regardless, your local verification emails are accessible from inbucket at http://localhost:54324 by default.


Live demo link is broken

Proxy Error

The proxy server received an invalid response from an upstream server.
The proxy server could not handle the request GET /.

:(

still down.. 😩

Still broken 👎

Demo site is back online. It stopped working due to exceeding 100% of disk quota from file uploads.


Add go:build line

This was added by Go 1.17. Go official recommends to use both new and old forms.
Ref: https://golang.org/doc/go1.17#build-lines

Why don't we wait 6 months and then we can delete the old form when 1.16 is out of support?

Hm, that's a fair point. I'm not sure that we can use the old form until when. But if that will come next version, we can delete the old form simply and add the new form. So I agree to we should not do for now.


Parallelize tx decoding/EVM preprocessing

Describe your changes and provide context
The decoding/preprocessing process is stateless so can be parallelized without non-determinism concerns.
Testing performed to validate your change
unit test

Codecov Report
Attention: Patch coverage is 79.16667% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 62.23%. Comparing base (3162956) to head (3a48763).

Additional details and impacted files

@@            Coverage Diff             @@
##            seiv2    #1461      +/-   ##
==========================================
+ Coverage   62.01%   62.23%   +0.21%
==========================================
  Files         352      352
  Lines       24365    24374       +9
==========================================
+ Hits        15109    15168      +59
+ Misses       8385     8335      -50
  Partials      871      871

Files
Coverage Δ

app/app.go
65.80% <79.16%> (+1.66%)
:arrow_up:

... and 3 files with indirect coverage changes


undefined window.activeTextEditor for binary files

VSCode Version: Code - Insiders 1.16.0-insider (aa42e6ef8184e8ab20ddaa5682b861bfb6f0b2ad, 2017-09-01T15:40:17.463Z)
OS Version: Windows_NT x64 10.0.15063
Extensions: none

window.activeTextEditor and e from onDidChangeActiveTextEditor callbacks are null when selecting/ opening a binary file (ie .png or exe)
I cannot find any other accessor for non-text file/editors.  How do I get the uri for my own extension to operate on?  (ie, showing scm status)

Yeah, that's why we call it activeTextEditor. There is no representation for other editors and contents in the system. We might cover this in #15178

Also, duplicate of #14483


Expo SDK 48

The plugin is not working with the latest expo SDK. Are you able to fix, or anyone's up for a PR maybe?

Hey there, maintainer of expo-community-flipper (an upstream dependency) here!
With SDK48, flipper is now officially supported. This means for SDK 48 you should remove the expo-community-flipper dependency and add expo-build-properties.
Most options carry over 1:1 under the new config object, including the ability to specify a semver version if you're using a version of react-native-flipper that differs from the one bundled with your react-native version. If after you've switched to the official build-properties plugin, things still aren't working, can you share the generated Podfile from expo prebuild? It's very helpful to ensure your Flipper is being defined properly when using an Expo setup.

I can confirm, after changing from expo-community-flipper to expo-build-properties it's working again.
Before I've seen an error I couldn't find in Google [ios.dangerous]: withIosDangerousBaseMod: Cannot add flipper arguments to the project's ios/Podfile. Please report this with a copy of your project Podfile. You can generate this with the expo prebuild command.
This issue can be closed.

PS. Code related to Flipper needs to be removed in AppDelegate.mm and in package.json. However, I couldn't find in the settings Start Measuring.

PS. Code related to Flipper needs to be removed in AppDelegate.mm and in package.json. However, I couldn't find in the settings Start Measuring.

Same here. Any solutions regarding the missing Start Measuring?


feat(scrollable_time_table): add tappable message to navigate to the Cabins page

To help users add their first cabin, tapping the Add a cabin to begin message will navigate to the Cabins page.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 49.165%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
530

Relevant Lines:
1078

💛  - Coveralls


Change the internal structure of the Dashboard

Actually the route Dashboard is rendering the Calendar page component. Personally I found this bad for the semantics and being honest, ugly.
I suggest to modify to internal structure of the component.
class Dashboard extends from React.Component {
  render() {
    return (

        <NavBar />
        <LeftSide />
        <RightSide />

    );
  }
};

Maybe a readable form of use the Context Providers (if is needed) can be:

class Dashboard extends from React.Component {
  render() {
    return (

        <NavBar />
        <BookProvider>
          <RoomProvider>
            <LeftSide />
            <ProfileProvider>
              <RightSide />
            </ProfileProvider>
          <RoomProvider>
        <BookProvider>

    );
  }
};

I know, i'm the first that hate the context API and the drilling, but maybe in cases where the component are going to use directly the consumer is a good option manage the providers in this way.
Obvious, if you are going to pass the props of the context through other components, is a better idea to use the provider inside the component.

Completed in #84 .


Open source

Request for an open-source documentation and code formatting.

@ledong0110 Can you help us resolve conflicts?

#1 resolved


Fixing grammar error in portuguese documentation

Fixing grammar error in portuguese documentation.

/lgtm

/approved

/cc @edsoncelio
/LGTM

/APPROVED

/lgtm

/approve


Create a static website for Inkscape Open Symbols

It would be nice to have a small static website displaying the list of symbols and how to use them. Something as the README but with better User Experience.
Any ideas on what would be nice to have?
Basics:

A list of symbols sets
A basic how to install them on different platforms
A how to contribute and current contributors.
Latest releases.

More?

A screencast of how to use them
Choose a representative symbol for each symbol set.
Does this project have/need an icon?
https://inkscape-open-symbols.github.io ?
What most people in the non-inkscape world (and largely in the inkscape world) don't know yet, is that this is an extremely powerful addition to a vector drawing program. Many designers of apps and web sites potentially could start using this. So at the same time, to continue this project, add possibility for donations via bitcoin, flattr, paypal and bountysource. The number of symbol sets is already large, so better throttle the addition of new ones and use donations to keep maintaining this project and making it as easy to install and use and see what license belongs to what symbol. Perhaps give a talk on it at for example FOSDEM 2018 in the open design devroom when this project has completely matured. Anyway, just my thoughts on this at this moment.

+1 for the icon (how would an icon for icons look like?...), and for the basics @Xaviju mentioned. The other suggestions from @PanderMusubi go beyond the basics, for me, although they're good ideas for later on (of course if someone happens to be at FOSDEM anyway, a short 5-min flash talk could be nice :)).
(Playing with them, I just found another (fun + silly) use: In children's drawing apps, there are often stamps that make it easy to create a drawing. I made a field with carrots ;-) )

For sure I will mention it quickly in my talk on FOSDEM next month, but no flash presentations there are possible.

They call it 'lightning talks', but the program is full by now, of course :) (didn't know the dates)

You'll be talking about fonts, right?

Yeah. You going too?

Nope. We've been thinking about going, to present our software deployment thing, but standing in a booth twice a year is sufficient at the moment. So much else to do! (but I heard only good things about it)

Ok so summary of contents of the website for a first MVP we agreed:

A list of symbols sets
A basic how to install them on different platforms
A screencast of how to use them
A how to contribute and current contributors.
Latest releases.
Choose a representative symbol for each symbol set.

About the last one, I'll go with a screenshot of the symbolset instead of a representative icon because not all symbolsets are meant to be icons. A symbolset, as in HumbleUI, could be a set of UI components. But I get the idea of having a representation of each one. Just my thought.
Does this project have/need an icon?
@PanderMusubi
I was thinking of asking for help in the [Open Source Design](Does this project have/need an icon?) repo. Give them this information of what contents the page should have and work with them in creating an image for the project and maybe some wireframes/UI of the page. I'll be happy to work on the front-end code.
(Playing with them, I just found another (fun + silly) use: In children's drawing apps, there are often stamps that make it easy to create a drawing. I made a field with carrots ;-) )
@Moini
That looks like an amazing amazing project. it would be nice to find an illustrator to create some templates with background illustrations and symbols attached to that item. Childs could start playing with Inkscape.
@PanderMusubi I'll be at FOSDEM next month!

This is the link for job proposals for the Open Source Design project:
http://opensourcedesign.net/opensrcdesignjobs/
Probably a three- or even just two-page static site would be sufficient, no?
One for presentation, one or two for documentation (with optional space for video). Not sure if more than the current release needs to be listed, we could just link to the list here on gh (but I don't assume they'd be of much interest).
The other iconset websites are kept very minimalistic, too, and that appears suitable to me. People just want to find out what's there, then download their set, and start working with it :)
I can help by providing list images of icons and texts. Design isn't my specialty :)

Done! The form they provide didn't work so I had to do a pull request so it took a bit longer.  https://github.com/opensourcedesign/jobs/pull/142

@Moini I won't focus only on iconsets, the project can also provide UI stencils, avatar sets, etc. although icons are the very first thing that comes to our minds because are very useful.

Please check the referenced issue for the work being done at Open Source Design if you want to add some feedback

I'll watch and learn, @Xaviju. I'm a total noob to the process (but I agree about the thunderbolt :) - in my very simple understanding, I didn't get the 'open' reference, and it just looked a bit off).

Just had an idea, not sure if it could be built upon:

(looking at it now, I think this somehow looks familiar... but can't say what it's similar to...)

Hi there! this issue is linked in an Open source design.net job: https://opensourcedesign.net/jobs/jobs/2017-01-14-inkscape-open-symbols-ux-ui
Can I close the job request?
(If I don't get a reply in 2 weeks from this date I'll close it :) )

Hi @Erioldoesdesign
I opened the job a while ago, and I think it can be closed safely. Sorry I didn't do it before.
Thanks!


Error Appears

all works fine in my ASP.NET MVC Core Project
But every time i lunch the app in VS appears this mensagge
No transport could be initialized successfully. Try specifying a different transport or none at all for auto initialization.

Try to remove this line at the end of the toastr.js :
//# sourceMappingURL=toastr.js.map
This solve incompatibilities in MVC...


ci(windows): add SHA256 hashes for artifacts

Fulfills #2997
cd.yml changes are always blind so it may break the draft release when it goes into main. Just let me know.
I should probably just switch it to Bash so it's easier to test.

The changes in _rust.yml work at least:

LGTM. The msi is the installer package, while the exe can be run standalone?

Correct. The exe might be nice for power users who want multiple versions installed side-by-side, but it's harder to support than the MSI, which will almost always install to the same C:/Program Files/Firezone Windows Client path, unless they pick a custom path or install to some other drive.

@ReactorScram In the docs I'll be sure to link to the MSI then.

Good to go

@ReactorScram Ah, sorry I totally missed this. I think there's a format for these sha256sum files we need to adhere to:
98fa59093124f62b434312fc133b70b846930d89cc72d38ba998ef33db6bb099  linux-client-arm

Which is:
<sha hash><space><space><filename corresponding to hash>

https://www.ibm.com/docs/en/zos/2.4.0?topic=descriptions-sha256-calculate-check-sha-256-cryptographic-hashes
Or maybe Windows has its own format? The above link is for the sha256 tool on *nixes

Yeah it's something strange the Powershell hash cmdlet does.
I guess it's meant for end users who download it to verify on their own systems? So I can check how to validate with Powershell.
If it has to match the Linux style, I can try using Bash instead of Powershell, the Github runners might have sha256sum in their Bash environments

If it has to match the Linux style, I can try using Bash instead of Powershell, the Github runners might have sha256sum in their Bash environments

Naw, if that's what Powershell generates / expects then let's go with that.

Actually I have no clue how to validate it, I'd just been eyeballing it.
Opened a ticket here https://github.com/firezone/firezone/issues/3221


chore: tweaks error block appearance

Changes
Changes the badge list to no longer be absolutely position in order to avoid it overlapping other content.
Puts icon and error title in one row.
Removes details element when showing invalid parameters.
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>Screenshots
Before:

After:

@johncowen Even though we’re not gonna use this component for the Zone create view anymore, it’ll be used for the general error case of most of our views. I’d still like to merge this because I think the layout is a little cleaner. What do you think?

Yeah sounds good to me! Agree the layout like this looks cleaner 👍


Installation Fails

Installation fails with the below message.
Unable to run package manager: exit status 1

There is no further message or logs to debug the cause.

Thanks for bringing this to our attention.
I believe this is actually an error thrown from the Akamai CLI.  I would recommend posting a comment there (https://github.com/akamai/cli)
but my guess is that its not able to find NPM/Node.  Did you install NVM per the readme?  If you are using a mac, you could do "brew install nvm" and then do "nvm install v8.9.1"

@jnchin-james I have referenced the installation issue under CLI. Just to let you know, the above node/npm installation also didn't resolve.

I see.
I would recommend installing NVM and using NVM to install npm.  Try to use the instructions on this site here:
http://nvm.sh/
Let me know if you are able to install nvm and then npm

2018-10-16T07_55_50_047Z-debug.log
Here are the logs when i try installation manually.

Thank you @jnchin-james . The issue was the registry was pointing to the wrong repository. Correcting that resolved the issue.

I am facing this issue as well @jnchin-james @vreddhi.
Here is the bash output,
➜  npm --version
7.15.1
➜  nvm use node
Now using node v16.3.0 (npm v7.15.1)
➜  akamai install property-manager
Attempting to fetch command from https://github.com/akamai/cli-property-manager.git[OK]
Installing... ... [FAIL]
unable to execute package manager: npm
 ERROR[0012] unable to execute package manager: npm command=install
 ERROR[0012] INSTALL ERROR: Unable to install selected package command=install
Unable to install selected package```

@tankchintan any luck with that issue?

if someone ever runs into this issue inspect your .npmrc file and make sure you have correct npmjs registry setup


Returning snapshot of dict causes double free

fn main() -> @Felt252Dict<u32> {
    let mut dict: Felt252Dict<u32> = Default::default();
                dict.insert(2, 1_u32);

    @dict
}

free(): double free detected in tcache 2
Aborted

I think we should also handle snapshot_take for dicts

fn main() -> @Felt252Dict<Nullable<Array<u32>>> {
    let mut dict: Felt252Dict<Nullable<Array<u32>>> = Default::default();
    dict.insert(2, NullableTrait::new(array![0]));

    @dict
}

Apparently they indeed can hold types with complex drops

I think we need to take a step back and think carefully about how snapshots and containers are supposed to work, rather than taking a reactive approach on failures.
@igaray WDYT?


Spelling Fixes

Additional Spelling fixes @Court72

@ankitaduttaMSFT,
Can you review the typo fix in articles/site-recovery/azure-to-azure-autoupdate.md?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@MicrosoftDocs/public-repo-pr-review-team - The rest of the typo fixes have been reviewed and look good to merge.


[Feature] Disabling encryption/decryption for Match

Issue Description
Add parameter to disable encryption/decryption operations for match.
Environment
✅ fastlane environment ✅
Stack

Key
Value

OS
10.11.5

Ruby
2.3.1

Bundler?
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.0

Git
git version 2.8.4 (Apple Git-73)

Installation Source
/usr/local/rvm/rubies/ruby-2.3.1/bin/fastlane

Host
Mac OS X 10.11.5 (15F34)

Ruby Lib Dir
/usr/local/rvm/rubies/ruby-2.3.1/lib

OpenSSL Version
OpenSSL 1.0.2h  3 May 2016

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

credentials_manager
0.16.2
✅ Up-To-Date

fastlane_core
0.53.0
✅ Up-To-Date

spaceship
0.36.2
✅ Up-To-Date

deliver
1.14.5
✅ Up-To-Date

snapshot
1.16.2
✅ Up-To-Date

frameit
3.0.0
✅ Up-To-Date

pem
1.3.2
✅ Up-To-Date

cert
1.4.3
✅ Up-To-Date

sigh
1.11.2
✅ Up-To-Date

produce
1.2.1
✅ Up-To-Date

gym
1.11.3
✅ Up-To-Date

pilot
1.11.1
✅ Up-To-Date

scan
0.13.1
✅ Up-To-Date

supply
0.7.1
✅ Up-To-Date

screengrab
0.5.5
✅ Up-To-Date

fastlane
1.106.2
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded*
Loaded gems

Gem
Version

did_you_mean
1.0.0

executable-hooks
1.3.2

bundler-unload
1.0.2

rubygems-bundler
1.4.4

bundler
1.13.5

io-console
0.4.5

highline
1.7.8

commander
4.4.0

terminal-table
1.4.5

json
1.8.3

multipart-post
2.0.0

faraday
0.9.2

faraday_middleware
0.10.0

dotenv
2.1.1

net-ssh
3.2.0

net-sftp
2.1.2

plist
3.1.0

rubyzip
1.1.7

security
0.1.3

krausefx-shenzhen
0.14.10

slack-notifier
1.5.1

i18n
0.7.0

thread_safe
0.3.5

tzinfo
1.2.2

minitest
5.8.3

activesupport
<IP_ADDRESS>

colored
1.2

claide
1.0.1

xcodeproj
1.3.3

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.7.1

addressable
2.3.8

xcode-install
2.0.7

word_wrap
1.0.0

multi_json
1.12.1

babosa
1.0.2

excon
0.54.0

gh_inspector
1.0.2

credentials_manager
0.16.2

fastlane_core
0.53.0

multi_xml
0.5.5

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20160826

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
1.6.8

mini_magick
4.5.1

xcpretty-travis-formatter
0.0.4

uber
0.0.15

representable
2.3.0

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

hurley
0.2

little-plugger
1.1.4

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
<IP_ADDRESS>

google-api-client
0.9.19

generated on: 2016-10-26

@AlexTatarinchik Thanks for your idea. Right now we aren't planning on implementing this. Can you let me know your use case, I'd be interested.

Why do you want to enable/disable encryption for match?

The profiles and certificates are  stored on the private secure git server. Nobody have access to this git server without permission. Why I should use additional security level to protect profiles an certs?

@AlexTatarinchik Thanks for following up. We'll leave this open for now in case other users are interested in this use case. We'd be interested to know. Hope that you have a great week!

A different use case might be to be able to directly inspect the certificates/profiles inside the repo if you clone them down manually, without having to manually decrypt them.
Sometimes I get an error where a profile in my repo is no longer available on the portal, and I would like to be able to see the UUID, devices, etc. The CLI tool suggests nuke, but all the profiles on the portal are valid, and the rest of the profiles in the repo are also valid. Just deleting the one profile manually will solve the issue.

Interesting, I can see the use-case. If you feel like it, feel free to work on this feature, however it's not something we can invest time in. Personally I'm not sure if it's worth the effort.


Create AnalysisContext API for Python (#4371)

This creates an API for AnalysisContext that can be used from Python, as described in https://github.com/Vector35/binaryninja-api/issues/4371

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

I marked this as a draft because I'm not entirely sure how to try this out properly. I tested the changes with copy and paste, but I might have copied something wrong and I'm not sure how to check out the API in binja itself?

Should be ready, will close #4371

This looks good to me, and @bpotchik said it's probably fine too. Thank you for the PR! I appreciate the wrapper.


Make cruft update more robust

This commit makes cruft update more robust when applying updates to a
project.

We terminate early if the project we are updating is a git repo
and the repo is unclean. This is to avoid git 3 way merge from failing
and the user applying changes via cruft update that they cannot recover.
This doesn't affect the user if the repo they are working with is not
a git repo.

When applying updates, if the user has deleted any files generated
from the cookiecutter template, we now ignore any updates to those
files. This makes the update more robust as git apply won't fail
and if the user has explicitly deleted any generated files, we shouldn't
try and update those.

This should also ensure that cruft, doesn't update the cruft file when updates fail
as described in #44 and instead, cruft terminates early and warns the user instead.
Fixes #44

Codecov Report

Merging #45 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #45   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          268       292   +24
=========================================
+ Hits           268       292   +24

Impacted Files
Coverage Δ

cruft/_cli.py
100.00% <100.00%> (ø)

cruft/_commands/update.py
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 53d6f9f...0479044. Read the comment docs.

@timothycrosley can you please release 2.2.0 once you merge this?


Formatting incorrectly indents previous line in nested lambda scenario

Note: This call path originates from OnAutoInsert where C# is instructing us to indent the curly braces differently. We then apply on-type-formatting mechanics ontop of the typed content in order to format it. However, in this case we format it incorrectly.
Starting text:
    public RenderFragment RenderFoo()
    {
        return (__builder) =>
        {
            @if (true){}
        };
    }

This is https://github.com/dotnet/razor-tooling/issues/5676. You've effectively just created the same conditions that InputSelect does, only manually. Removing the @ works around it. Will be fixed by https://github.com/dotnet/roslyn/issues/57465.

Going to reopen this, because whilst document formatting works for it, on type doesn't


Fixed the Dark Mode not working Issue (Issue #21)

Fixes Issue

Closes #21
Closes #3

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

@Ivan2001otp, please refrain from creating a pull request if the task has not been assigned to you. Kindly wait until someone assigns it to you. Thank you for your understanding.


[Event Request] Codeunit 99000774, change Parameter of publisher to var

As already requested in https://github.com/microsoft/ALAppExtensions/issues/4227 we need Parameter "NextCapNeedLineNo" as VAR in Codeunit99000774 - Calculate Routing Line - Publisher "OnAfterCreateCapNeeded".
In Codeunit "Calculate Planning Route Line", 'OnAfterCreatePlanningCapNeed' the Parameter is implemented as VAR!
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


[release-2.4] Update moment version for sec issue

Signed-off-by: Zack Layne<EMAIL_ADDRESS>Related Issue:  stolostron/backlog#24423
Description of changes

Updated moment version to acceptable 2.29.4

/hold cant merge until 2.4 stream is reopened.

/lgtm
This might conflict with #239

Closing for -> https://github.com/stolostron/search-ui/pull/241


Support for attach files

Is there a block for uploading files?

Currently there is no extension support for attaching files. I will reseach later

@JohnnyDevJs I added an extension to the attachment

@hunghg255 I saw the new fit, it looks really good. Thanks!


Stack feed filter

Add ability to filter the sources that are displayed in the Stack feed

@bsmith11 by filtering are you wanting the ability to remove just 1(or more) source(s)? eg see everyone except sludgebrown?

Yes. This would only apply in the "Stack" feed. We need to find a good place to display the filters, whether it be in the Sources screen or in Settings


[AUTOPATCHER] Patches CVE-2021-3667 for libvirt

        Patches CVE-2021-3667 for libvirt.
        This patch and PR was automatically generated and needs human verification before merging.

Duplicate of #2526


UnexpectedError: The command failed with an unexpected error. Here is the traceback: list index out of range

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az acr run
Errors:
list index out of range
Traceback (most recent call last):
Temp\pip-unpacked-wheel-tuw8_17y\knack\cli.py, ln 215, in invoke
cli\core\commands\__init__.py, ln 579, in execute
Temp\pip-unpacked-wheel-tuw8_17y\knack\parser.py, ln 262, in parse_args
argparse.py, ln 1734, in parse_args
azure\cli\core\parser.py, ln 304, in parse_known_args
argparse.py, ln 1766, in parse_known_args
argparse.py, ln 1954, in _parse_known_args
argparse.py, ln 1931, in consume_positionals
argparse.py, ln 1840, in take_action
argparse.py, ln 1137, in __call__
azure\cli\core\parser.py, ln 304, in parse_known_args
...
argparse.py, ln 1137, in __call__
azure\cli\core\parser.py, ln 304, in parse_known_args
argparse.py, ln 1766, in parse_known_args
argparse.py, ln 2001, in _parse_known_args
azure\cli\core\parser.py, ln 167, in error
azure\cli\core\command_recommender.py, ln 61, in __init__
azure\cli\core\command_recommender.py, ln 244, in _normalize_parameters
IndexError: list index out of range

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az acr run --anandregistry

Expected Behavior
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.12.1

Extensions:
logic 0.1.0

Additional Context

acr

dupe of #15465


CombinatorSystem is unsound

Bevy version
0.14.1
What you did
use bevy_ecs::system::{CombinatorSystem, Combine, IntoSystem, Resource, System};
use bevy_ecs::world::World;
use scoped_tls_hkt::scoped_thread_local; // scoped_tls_hkt = "0.1.4" in Cargo.toml

#[derive(Clone, Copy)]
struct BadFn<'a>(&'a dyn Fn());

scoped_thread_local!(static BAD_FN: for<'a> BadFn<'a>);

struct BadCombine;

impl<A: System<In = (), Out = ()>, B: System<In = (), Out = ()>> Combine<A, B> for BadCombine {
    type In = ();
    type Out = ();

    fn combine(
        _input: Self::In,
        a: impl FnOnce(A::In) -> A::Out,
        b: impl FnOnce(B::In) -> B::Out,
    ) -> Self::Out {
        let a = std::cell::Cell::new(Some(a));
        let bad_fn = BadFn(Box::leak(Box::new(move || (a.take().unwrap())(()))));
        BAD_FN.set(bad_fn, || b(()));
    }
}

#[derive(Resource)]
struct Foo(String);

fn a(world: &mut World) {
    world.remove_resource::<Foo>();
}

fn b(world: &mut World) {
    world.insert_resource(Foo("foo".to_string()));
    let foo = world.resource::<Foo>();
    BAD_FN.with(|bad_fn| (bad_fn.0)());
    println!("{}", foo.0);
}

fn main() {
    let mut world = World::new();

    let a_system = IntoSystem::into_system(a);
    let b_system = IntoSystem::into_system(b);
    let mut combinator_system =
        CombinatorSystem::<BadCombine, _, _>::new(a_system, b_system, "".into());

    combinator_system.initialize(&mut world);
    combinator_system.run((), &mut world);
}

What went wrong
This should either fail to compile or throw an error at runtime. Instead, this produced UB due to b holding a reference to a resource that has been removed. MIRI also flags this as UB:

```
error: Undefined Behavior: not granting access to tag <24319> because that would remove [Unique for <19222>] which is strongly protected because it is an argument of call 4818
   --> E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ptr-0.14.1\src/lib.rs:574:18
    |
574 |         unsafe { &mut *self.get() }
    |                  ^^^^^^^^^^^^^^^^ not granting access to tag <24319> because that would remove [Unique for <19222>] which is strongly protected because it is an argument of call 4818
    |
    = help: this indicates a potential bug in the program: it performed an invalid operation, but the Stacked Borrows rules it violated are still experimental
    = help: see https://github.com/rust-lang/unsafe-code-guidelines/blob/master/wip/stacked-borrows.md for further information
help: <24319> was created by a SharedReadWrite retag at offsets [0x0..0x458]
   --> src/main.rs:22:55
    |
22  |         let bad_fn = BadFn(Box::leak(Box::new(move || (a.take().unwrap())(()))));
    |                                                       ^^^^^^^^^^^^^^^^^^^^^^^
help: <19222> is this argument
   --> src/main.rs:34:6
    |
34  | fn b(world: &mut World) {
    |      ^^^^^
    = note: BACKTRACE (of the first span):
    = note: inside `<&std::cell::UnsafeCell as bevy_ecs::bevy_ptr::UnsafeCellDeref<'_, bevy_ecs::world::World>>::deref_mut` at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ptr-0.14.1\src/lib.rs:574:18: 574:34
    = note: inside closure at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\system\combinator.rs:194:48: 194:65
note: inside closure
   --> src/main.rs:22:55
    |
22  |         let bad_fn = BadFn(Box::leak(Box::new(move || (a.take().unwrap())(()))));
    |                                                       ^^^^^^^^^^^^^^^^^^^^^^^
note: inside closure
   --> src/main.rs:37:26
    |
37  |     BAD_FN.with(|bad_fn| (bad_fn.0)());
    |                          ^^^^^^^^^^^^
note: inside `BAD_FN::>::with::<{closure@src/main.rs:37:17: 37:25}, ()>`
   --> src/main.rs:8:1
    |
8   | scoped_thread_local!(static BAD_FN: for<'a> BadFn<'a>);
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
note: inside `b`
   --> src/main.rs:37:5
    |
37  |     BAD_FN.with(|bad_fn| (bad_fn.0)());
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = note: inside ` fn(&'a mut bevy_ecs::world::World) {b} as std::ops::FnMut<(&mut bevy_ecs::world::World,)>>::call_mut - shim(for<'a> fn(&'a mut bevy_ecs::world::World) {b})` at E:\Programmi\Rust\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\ops\function.rs:166:5: 166:75
    = note: inside `std::ops::function::impls:: for &mut for<'a> fn(&'a mut bevy_ecs::world::World) {b}>::call_mut` at E:\Programmi\Rust\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\ops\function.rs:294:13: 294:35
    = note: inside ` Out>>::run::call_inner::<(), &mut for<'a> fn(&'a mut bevy_ecs::world::World) {b}>` at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\system\exclusive_function_system.rs:218:21: 218:42
    = note: inside ` fn(&'a mut bevy_ecs::world::World) {b} as bevy_ecs::system::ExclusiveSystemParamFunction>::run` at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\system\exclusive_function_system.rs:221:17: 221:53
    = note: inside closure at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\system\exclusive_function_system.rs:111:23: 111:58
    = note: inside `bevy_ecs::world::World::last_change_tick_scope::<(), {closure@ fn(&'a mut bevy_ecs::world::World) {b}> as bevy_ecs::system::System>::run::{closure#0}}>` at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\world\mod.rs:2215:9: 2215:23
    = note: inside ` fn(&'a mut bevy_ecs::world::World) {b}> as bevy_ecs::system::System>::run` at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\system\exclusive_function_system.rs:103:9: 119:11
    = note: inside closure at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\system\combinator.rs:196:21: 196:68
note: inside closure
   --> src/main.rs:23:31
    |
23  |         BAD_FN.set(bad_fn, || b(()));
    |                               ^^^^^
note: inside `BAD_FN::>::set::<{closure@src/main.rs:23:28: 23:30}, ()>`
   --> src/main.rs:8:1
    |
8   | scoped_thread_local!(static BAD_FN: for<'a> BadFn<'a>);
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
note: inside ` fn(&'a mut bevy_ecs::world::World) {a}>, bevy_ecs::system::ExclusiveFunctionSystem fn(&'a mut bevy_ecs::world::World) {b}>>>::combine::<{closure@ fn(&'a mut bevy_ecs::world::World) {a}>, bevy_ecs::system::ExclusiveFunctionSystem fn(&'a mut bevy_ecs::world::World) {b}>> as bevy_ecs::system::System>::run::{closure#0}}, {closure@ fn(&'a mut bevy_ecs::world::World) {a}>, bevy_ecs::system::ExclusiveFunctionSystem fn(&'a mut bevy_ecs::world::World) {b}>> as bevy_ecs::system::System>::run::{closure#1}}>`
   --> src/main.rs:23:9
    |
23  |         BAD_FN.set(bad_fn, || b(()));
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = note: inside ` fn(&'a mut bevy_ecs::world::World) {a}>, bevy_ecs::system::ExclusiveFunctionSystem fn(&'a mut bevy_ecs::world::World) {b}>> as bevy_ecs::system::System>::run` at E:\Programmi\Rust\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_ecs-0.14.1\src\system\combinator.rs:189:9: 197:10
note: inside `main`
   --> src/main.rs:50:5
    |
50  |     combinator_system.run((), &mut world);
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = note: this error originates in the macro `scoped_thread_local` (in Nightly builds, run with -Z macro-backtrace for more info)
```

Additional information
The issue is that CombinatorSystem gives to the Func::combine implementation two closures, each of which holds an UnsafeWorldCell that assumes is valid for calling that system:
https://github.com/bevyengine/bevy/blob/91fa4bb64905121a18b40df0062dbd85714aa3ce/crates/bevy_ecs/src/system/combinator.rs#L185-L198
The safety comment assumes that since the functions are !Send + !Sync + !'static then they cannot possibly be called in a parallel way. However this fails to see that they can still be called in a reentrant way, that is one function calling the other while it's running. This effectively also creates overlapping references to the world or its contents and thus leads to UB.
Actually exploiting this issue is not that easy though, as can be seen in the code above, since one closure cannot be smuggled directly to the other, thus having to go through some global state. This might seem that it requires 'static (and in fact using only the stdlib is does), but there are libraries that allow doing this like scoped-tls-hkt, which allows to store non-'static data in a thread local, ensuring it will be overwritten before it goes out of scope. With this one closure can actually be smuggled into the other and lead to the overlapping access to the world and thus UB.

The easy fix here might be to make Combine an unsafe trait.

A more complex alternative which would however keep Combine safe would be to change Combine's parameters to a single one that allows running either A or B, and returns a handle to call the other one. This way CombinedSystem could even be implemented without using unsafe.
It would look something like this (name totally up for discussions):
struct RunBoth<'a, A, B> {
    world: UnsafeWorldCell<'a>,
    a: &'a mut A,
    b: &'a mut A,
}

impl<'a, A; System, B: System> RunBoth<'a, A, B> {
    fn run_a(self, input: A::In) -> (A::Out, RunOne<'a, B>) {
        let output = unsafe { self.a.run_unsafe(input, self.world) };
        (output, RunBoth { world: self.world, s: self.b })
    }
    fn run_b(self, input: B::In) -> (B::Out, RunOne<'a, A>) {
        let output = unsafe { self.b.run_unsafe(input, self.world) };
        (output, RunBoth { world: self.world, s: self.a })
    }
}

struct RunOne<'a, S> {
    world: UnsafeWorldCell<'a>,
    s: &'a mut S
}

impl<'a, S: System> RunOne<'a, S> {
    fn run(self, input: S::In) -> S::Out {
        unsafe { self.s.run_unsafe(input, self.world) }
    }
}

pub trait Combine<A: System, B: System> {
    type In;
    type Out;

    fn combine(
        input: Self::In,
        systems: CallBoth<'_, A, B>,
    ) -> Self::Out;
}

a single one with typeCallBoth<A, B>, with a method fn call_a(self, A::In) -> (A::Out, Call<B>) and a


@extend rules

Can I use @extend rule with precss plugin? For sass it's look like:
%test {
color: red;
}
.test
@extend %test

Like this? https://jonathantneal.github.io/precss/

Thanks! That's what I need


Jk metadata search dump

This PR creates a new version of dump_species_search_data.py that pulls data from the metadata database. Doing so resolves missing data and missing fields.
This PR works towards resolving ENSWBSITES-2179
Example output
"fields": [
{
    "name": "id",
    "value": "18f3832a-a765-4e6e-aa8c-115b7e4532af"
},
{
    "name": "common_name",
    "value": "mouse"
},
{
    "name": "scientific_name",
    "value": "Mus musculus"
},
{
    "name": "assembly",
    "value": "129S1_SvImJ_v1"
},
{
    "name": "assembly_accession",
    "value": "GCA_001624185.1"
},
{
    "name": "unversioned_assembly_accession",
    "value": "GCA_001624185"
},
{
    "name": "type_value",
    "value": "129S1/SvImJ"
},
{
    "name": "parlance_name",
    "value": ""
},
{
    "name": "type_type",
    "value": "strain"
},
{
    "name": "coding_genes",
    "value": "20720"
},
{
    "name": "n50",
    "value": "14243"
},
{
    "name": "has_variation",
    "value": false
},
{
    "name": "has_regulation",
    "value": false
},
{
    "name": "annotation_method",
    "value": ""
},
{
    "name": "annotation_provider",
    "value": "Ensembl"
},
{
    "name": "genome_uuid",
    "value": "18f3832a-a765-4e6e-aa8c-115b7e4532af"
},
{
    "name": "url_name",
    "value": ""
},
{
    "name": "tol_id",
    "value": ""
},
{
    "name": "is_reference",
    "value": false
},
{
    "name": "species_taxonomy_id",
    "value": 10090
},
{
    "name": "organism_id",
    "value": 139
},
{
    "name": "rank",
    "value": 0
}
]

Is it worth moving this script ( + all other data dumping script ) to dedicated repository. This is not needed for running API per say.

Lets leave them here for now. I will catch up with Marc to see what he is planning. I know the query will live in metadata api. He might already have a repo he is planning to put this kind of thing in.


One-click spam button

Problem
We get quite a lot of unsollicited/spam emails.
Solution
Would love to have a 1click in slack to mark as spam, delete the message from Slack (or at least collapse it as those spam emails tend to take up a lot of channel real estate) and  never let emails from that sender through again.
Describe alternatives you've considered
I manually 'mark as resolved', then delete the slack message itself so it doesn't distract anyone else. Usually however these people will follow up so I need to repeat the song and dance.
Additional context
Add any other context, screenshots, or related issues about the feature request here.
Testing
Describe how this should be tested.
Questions, or need help getting started?
Feel free to ask below, or ping us on Slack :)
(You can also check out our CONTRIBUTING.md)

good idea @timgl — would you also want these marked as spam within Gmail (e.g. we could label it as spam via the API), or just internally within Papercups?

I think marking them as a spam in gmail is a nice to have, doesn't really affect us that much (except for punishing spammers which is always good :))

Any other thoughts on this? A decent chunk of the messages we have to wade through now are spam messages and makes us lose track of real messages in Slack so would be awesome to have this

@timgl as a first pass, we could surface a "delete" button in slack that handles deleting the thread both in slack and in the dashboard -- would that be ok for now?

Yep that'd be great!


What is the minimum number of elements to train?

In addition to the doubt of the title, are you creating the training images with any specific program?

@dpramirez The tusimple dataset was used in this program and you may refer to their website to check out the detailed description about it. Also the model can be trained on culane dataset and you're welcomed to do such a experiment:)

@MaybeShewill-CV  I understand and it is quite clear to me, my problem is how I generate the binary image and the instance, I generated it manually occupying gimp but when adding it, the training falls. Also I just wanted to add a couple more images that interested me and Culane is more information than I need.

@dpramirez You may look at this https://github.com/MaybeShewill-CV/lanenet-lane-detection/blob/master/tools/generate_tusimple_dataset.py which will guide you to generate the training samples:)

I saw it, I appreciate the answer. But what I do not understand is where I get the json and how I produce it, I saw in other doubts that there is another repository that generates them but there must be a simpler way.
If you can give me a guide or help I would be very grateful. my image is as follows:

Quiero detectar las siguientes lineas:

Sorry I'm still a novice, I don't want to cause you trouble.

@dpramirez Google the usage of labelme which should help you to annotate such a image segmentation task:)

some example of the json that you use for the generate_tusimple

@dpramirez Seems that the dataset are not available now on Tusimple website. You'd better to google it yourself:)

@MaybeShewill-CV @dpramirez , Do we need to make binary images and instance images from labelme tool for like 1000 images?


Update emulator-command-line-parameters.md

added semi-colon to PowerShell snippet to prevent the environment variable 'contaminating' itself by directly appending the new module path to the tail of the previous path

@identity-jason : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off


please help my challenge guys

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

thank you!!!

@romercoder Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!


Upon installation, Opportunity Record Type Exclusion for Rollup fields pull from Contact RT instead of Opportunity

Hello,
I didn't see an existing case, and this might actually be a Mr. Belvedere issue instead. Might also be lower priority, because it can be worked around by simply editing the settings and choosing valid values.
Just after installation (in an existing org), Health Check shows a Failure for Opp Record Types that are "Invalid". This makes sense, because the Record Type that is invalid is a Contact Record Type, and not an Opp RT at all.
Somewhere, the initial values being set on these Donor Statistics settings is describing the schema for Contact when it intends to describe Opportunity.

Thanks,
Steve

sounds like our code for initializing empty settings is doing the wrong thing!  thanks for reporting this.

Hi Steve, those values pull from old NPSP settings objects, in order to keep settings intact on upgrade. If you had a previous installation of NPSP before upgrading to NPSP 3, then those values were already present in those settings.

I see. I seem to remember seeing this issue in older versions of NPSP, now that I think about it.
I created a new Sandbox with the old NPSP installed to create a checklist for myself before performing an upgrade for Production. No data was brought over; but if this was a Custom Settings object, I think those values DO come over with the deployment. Does that still sound consistent with your thinking? If you would like me to try anything, I'm happy to. Otherwise, thanks for the response.

Custom settings values seem to be copied over to sandboxes as of Summer '13, so you can check in your sandbox under Setup > Develop > Custom Settings > click 'Manage' next to Households Settings.
I'm fairly certain that this settings value was an issue before the upgrade, it just never manifested itself since the opportunity record type didn't exist, so there was nothing to filter. Issues like this are raison d'etre for Health Check, so it's great that it caught it!