Response required ReadableStreamInterface

Icicle\Http\Message\Response requires ReadableStreamInterface in its constructor:
public function __construct(
    $code = 200,
    array $headers = null,
    ReadableStreamInterface $stream = null,
    $reason = null,
    $protocol = '1.1'
) {
    parent::__construct($headers, $stream, $protocol);

    $this->status = $this->validateStatusCode($code);
    $this->reason = $this->filterReason($reason);
}

Is there a reason for that? I would say, it should be WritableStreamInterface, but it wouldn't match with abstract Message class.
Solution would be to move constructor away from Message. What do you think?

Both Request and Response require a ReadableStreamInterface object because the stream needs to be readable at a minimum in order for the object to be useful. MessageInterface::getBody() returns a ReadableStreamInterface object for this same reason. Server reads the stream from the Response object and pipes it to the client socket stream.
Yes, often a DuplexStreamInterface object is provided so the stream may be written to by the application, but writing to the stream isn't strictly necessary. Perhaps you'd provide a read-only file stream (which is a component that still needs to be written) as the stream in a Response object.
Also consider a Response object in the client context (which is also a part of this library). In this case the response object would be read by the application code.
Is there a problem you've run into when trying to use these objects? Maybe I could help. Sorry there isn't documentation yet. My time has been occupied by other work.

No, I don't have any problems, it's just that I don't like how it currently works for handling server-side requests. Say we have a middleware that takes a requests, fills response with data and then sends it back to client. What would be the best, elegant way to ensure that response is writable, without adding IF instruction within middleware itself?
I fully agree with what you wrote above, but how should we handle my use case?

My understanding is that such middleware would return a new response object with a different stream of it's choosing. Since the messages are immutable, middleware really should be returning a new instance rather than writing to the stream of another object. Does that make sense, am I understanding middleware correctly?

Yeah, it makes sense. Actually, after getting into coding, I had to change my approach, so this is not a problem anymore.

I would be interested in seeing what you're doing. I'm debating if I should switch up some things about Server, so I'd like to have some more input. I wanted to get this package relatively stable so I could build the WebSocket package on top of it soon.

I'm mostly playing around now ;-)
It is related to Psr7Bridge, trying to build a stack of PSR-7 middlewares that can be easily plugged into Icicle.
At first I wanted to covert Icicle's request into PSR-7 message pass it, and wait for response to come back, convert it and send to user. During that time I thought that Response's body should always be writable, so I filled this ticket :)
But, in the end, it doesn't work as expected, complicated things and looked ugly. So now I'm going to skip converting response, in favor of writing emitter that takes PSR-7 message and sends it via Icicle.
After getting to something interesting I'll publish my results to Gist or separate repository.
What do you want to change in Server? ?

I'm mostly playing around now ;-)
It is related to Psr7Bridge, I'm trying to build a stack of PSR-7 middlewares that can be easily connected to Icicle.
At first I wanted to convert Icicle's request into PSR7 message, pass it along, take the response, convert it, and send back to user. When working on it, I thought that Response body should always be writable, so I filled this ticket :)
But, it was a dead end, it didn't work as expected, complicated things and looked ugly. So now I'm going to skip converting response in favour of writting emitter that takes PSR-7 message and sends it via Icicle.
After getting to something interesting I'll publish my results to Gist or a repo.
What changes are you considering? If you want my opinion, open an issue and @-ping me :-)


Fixed expansion positioning on threebase_voidray example.

Nexus was misplaced on some maps. I noticed this on Catalyst when the bot was placed lower right. Adding placement_step=1 also fixed the issue I think.

Thanks!
Finally merged in https://github.com/Dentosal/python-sc2/commit/edc49f389a4a58f6ef34e62129883e194317b5fd


[Create Ride Request] 'From:' and 'To:' edit boxes are enabled

Environment: Android 10, Samsung A7.
Reproducible: always.
Build found: Build found: v1.0.383
Steps to reproduce

Log in to 'Car'
Tap 'Search For A Ride'
Select all required fields ( e.g. From: Lviv, To: Vinnytsia, departure time:26.10; Tue; 13:00, passenger: 1, Fee: All )
Tap 'Search'
Tap 'Create ride request'

Actual result
'From:' and 'To:' edit boxes are enabled.
Expected result
'From:' and 'To:' edit boxes are disabled.
User story https://github.com/ita-social-projects/Car-Back-End/issues/16
Labels to be added
"Bug" ("bug"), Priority ("pri:low ")

fixed in v1.0.395


Revert "[5.1] Add keys to transform"

Reverts laravel/framework#8767

@JosephSilber reverted this as passing a second array to array_map causes it to not preserve keys, which would be a breaking change.


Implement ResourceAny -> Resource<T> conversion

Feature
Refs #7688
Currently, developers can define resources via Linker::resource, but they can not get a handle to the Resource<T> from a ResourceAny, which is passed to e.g. Linker::func_new functions.
Benefit
Most importantly, it allows usage of resources for dynamically-typed functions, e.g. such as when WIT definition is not known at compilation time
Implementation
This should be quite similar to #7688, we should make sure to account for the own/borrow state and possibly call the destructor on Drop
Alternatives
The only alternative seems to be not supporting dynamically-typed functions utilizing resources, developers may use macros/templating to generate all possible type bounds of imported functions at compile time and choose the right one at runtime, but that is very complex and hard to maintain
A reasonable alternative could be using the "partially-typed" bound with https://github.com/bytecodealliance/wasmtime/issues/7701, but it still seems preferable to support fully-dynamic use cases

cc @alexcrichton, we briefly discussed this earlier


Aggregate functions don't properly work on MONEY fields.

Aggregate functions don't properly work on MONEY fields.
Doing sum('amount') results in NaN.  I believe it has something to do with string parsing.
Originally posted by @mordechaim in https://github.com/romeerez/orchid-orm/issues/356#issuecomment-2310677320

I figured out that it is not a bug:
SELECT avg(x)
FROM (
  SELECT 1::money x
  UNION
  SELECT 2::money x
) t

[42883] ERROR: function avg(money) does not exist Hint: No function matches the given name and argument types. You might need to add explicit type casts. Position: 8

You should use decimal or numeric (they are the same) for storing exact numbers with floating point, or store money as cents integer, Stripe does it so.

But sum is a valid aggregate function though

That's not correct, I'll change it so TS will complain when you do db.table.sum('amount') for a money column.

It is, I just tried

Oh, I tried

I pushed improvement for aggregate argument types.
Not just for money, but I listed actual postgres arg types for every aggregate function in ORM and listed arg types literally, so min and max support numbers, text (wasn't expecting that), dates, timestamps, intervals; sum supports numeric types + money, avg the same as sum but without money, and some more of this.

Thank you.
Btw I noticed the money parse regex assumes the currency to be $, how does it handle other currency symbols? (Just reading the code, haven't tested it)

I tried the new version, the sum function now works, but the return value is not parsed to js numbers, instead it returns the string value as $1.00.
By now I think I should just use numeric...

how does it handle other currency symbols? (Just reading the code, haven't tested it)

I don't know, that's coming from and handled by postgres.

instead it returns the string value as $1.00.
By now I think I should just use numeric

Numeric/decimal types are returned as strings as well, because postgres handles them in a precise way, but JS numbers are more limited and imprecise.
I should research on the topic and write a doc section that suggests how to deal with money and other precise amounts.

I'm referring to
https://github.com/romeerez/orchid-orm/blob/3a92c5346c6a97f012e782f6f89958984a2bf391/packages/qb/pqb/src/columns/string.ts#L335
Which will fail if the locale is set to any country that uses a different currency symbol.

Thanks for pointing, I'll update that piece.


Breaks anonym.to redirects

Description
Since anonym.to is blocked by this hosts file.
Example URL

https://anonym.to/?https://github.com/4skinSkywalker/anti-porn-hosts-file/issues/
https://anonym.to/?https://www.lifewire.com/legalities-of-linking-3468972

Do you suggest to remove anonym.to from the file?


Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/thestonefox/SteamVR_Unity_Toolkit
https://github.com/thestonefox/VRTK

HTTPS Corrected URLs

Was
Now

http://altvr.com/
https://altvr.com/

http://unity3d.com/learn/tutorials/topics/virtual-reality/
https://unity3d.com/learn/tutorials/topics/virtual-reality/

http://www.amazon.it/gp/product/1466511842
https://www.amazon.it/gp/product/1466511842

http://www.getfove.com/
https://www.getfove.com/

Other Corrected URLs

Was
Now

http://julesthuillier.com/vrtracker/
http://vrtracker.xyz/

http://ovrvision.com/order-ovr1/
http://ovrvision.com/

http://surre.al/
http://surrealvr.com

http://www.htcvr.com/
http://www.htcvive.com/us/

https://developer.oculus.com/documentation/intro-vr/latest/concepts/bp_intro/
https://developer3.oculus.com/documentation/intro-vr/latest/concepts/bp_intro/

https://projectsansar.com/
https://www.sansar.com/

https://www.oculus.com/en-us/rift/
https://www.oculus.com/rift

https://www.thalmic.com/en/myo/
https://www.myo.com/

hm some of the substitution is no good, sorry


transpile extension string to code

Since I keep json in a database, I need everything to be string.
However there are cases that we need function / variable in some fields.
My solution would be to save to string and create an extension that transpile code.
but I'm having scope problem now, because as soon as he transpiles from string to code, he doesn't find that function inside tranpile, there is a better solution or a way to bind what is inside the component and send it together
json code, see autocomplete:
{
    key: 'Autocomplete',
    type: 'autocomplete',
    templateOptions: {
        required: true,
        label: 'Autocomplete',
        placeholder: 'Placeholder',
        filter: '(term) => of(term ? this.filterStates(term) : states.slice())',
    }
}

transpile:
import { FormlyFieldConfig } from '@ngx-formly/core';

export class TranspileStringToCode {
    constructor() {}
    prePopulate(field: FormlyFieldConfig): void {
        const to = field.templateOptions || {};

        console.log('field', field);
        //typeof t2 === 'string' ? eval(t2) : t2;

        field.expressionProperties = {
            ...(field.expressionProperties || {}),
            'templateOptions.options': (typeof to.options === 'string') ? eval(to.options) : to.options,
            'templateOptions.filter': (typeof to.filter === 'string') ? eval(to.filter) : to.filter,
        };
    }
}

export function registerTranspileStringToCode(): any {
    return {
        extensions: [
            {
                name: 'TranspileStrCodVimbo',
                extension: new TranspileStringToCode()
            }
        ]
    };
}

error example:

Ali is giving error in of but could give error in any other function or variable

of is an rxjs function, which is not known as a 'window' function.. A dirty workaround would be something like this:
import { FormlyFieldConfig } from '@ngx-formly/core';
import { of } from 'rxjs/operators';

declare var window: any;

export class TranspileStringToCode {
    constructor() {
        window.of = of;
    }
    prePopulate(field: FormlyFieldConfig): void {
       // ...
    }
}

This does not solve my problem, since the transpile file only converts string to / var function, to be used only where json will be loaded and not in the transpiler.
I have no way to call all possible functions inside the transpiler.
What I would look for would be a way to bind (this) where json is and send it together, so when I use eval to turn a string into a function call for example, it would not return a referenceError

using eval is not the right way at all, it will make your form hard to maintain and vulnerable to XSS attack, if I'm going to fully rely on json I'd prefer to use an http calls for the filter so in that the filter option will contains the api link.
    prePopulate(field: FormlyFieldConfig): void {
        ...,
        field.options = this.httpClient.get(to.optionsUrl);
    }

another way is to define some utils which you may link them to field, for example filterStates:
    prePopulate(field: FormlyFieldConfig): void {
        ...,
        if (field.templateOptions.filter === 'filterStates') {
          field.templateOptions.filter = (term) => of(term ? this.filterStates(term) : states.slice());
        }
    }

@aitboudad
would have one usage example for each of the two examples in stackblitz ?

sorry but I think you can made it by yourself 🤔, I've provided enough details on that specific part and I don't think a stackblitz example would add any relevant info.

@aitboudad  you're right.
but I'm here studying some ways to do this in a way that I can enjoy the service, which already has the functions.
I almost got it, but I still haven't found a way to pass the term, just see how I'm doing.
in my component where I define the field, I left:
                 {
                        key: 'city',
                        type: 'autocomplete',
                        templateOptions: {
                            required: true,
                            label: 'Autocomplete',
                            placeholder: 'Placeholder',
                            appearance: 'outline',
                            filter: 'this.filterStates("city", term)',
                            filerKeys: ['sigla', 'nome']
                        }
                    }

and I created a function to transform into a function, since it's string.
prePopulate(
        field: FormlyFieldConfig
    ): void {

        const to = field.templateOptions || {};

        field.expressionProperties = {
            ...(field.expressionProperties || {}),
            'templateOptions.filter':
                typeof to.filter === 'string' ? (term) => of(new Function('return ' + to.filter)(term)) : to.filter
        };
    }

But is giving error in the term, you know how I could solve this, following this idea?


Bug: Internal Server Error on Lambda

Have you experienced this bug with the latest version of the template?
yes
Steps to Reproduce
Deploy the template to AWS using the arc deployment
Attempt to sign up
Expect a 500 internal server error
Expected Behavior
Sign up should succeed
Actual Behavior
There is an internal server error with no logging at the time bycryptjs attempts to hash the password.
I added a try catch, still no error. So at that point I assumed this is a very low level issue.
Indeed, I added more memory to the aws lambda deployment to 1024 MB from the default 128 MB and hashing did not fail, I was able to log in.
I would suggest that the declaration:
memory 1024

Is added to the lambda, potentially 512 could fix the issue. I was not able to determine if it was due to CPU or RAM and both scale on memory within lambda

strange, I'm not getting a 500 on https://du9zh091t2.execute-api.us-west-2.amazonaws.com using the default settings

Closing this issue as no response was provided.
Feel free to re-open it if it's still a problem.

I have seen this after adding a VPC to the lambda, when you do that the lambda can no longer talk to the dynamodb server and the page will crash very low level.  Tried adding an endpoint gateway to dynamodb which should allow the lambda in the VPC to talk to the public dynamo but it doesn't work.  More info here https://repost.aws/questions/QUsOBNlERPQZejZsyo3rtxGg/lambda-in-private-subnet-cannot-reach-dynamo-db


Colorbox Resizing Parent's Width

Hi,
We have used Colorbox many times and never seen this issue before. When the Colorbox opens, the width of the parent's content is being resized.
Without Colorbox Open:

With Colorbox Open:

Please take note of the right side of the screen. Either the width of the content is shrinking when the Colorbox is opened, or an excessive amount of right margin/padding is being added. Any suggestions on fixing this?

What options are you passing Colorbox? Have you tried using the maxWidth setting? This isn't enough info to diagnose your issue with any confidence.

The Colorbox initializes with the following:
if(windowwidth > 1000 && windowheight > 1000)
$.colorbox({ href: href1, height:'70%', width:'90%' });
else
$.colorbox({ href: href1, maxHeight:'400px', height:'90%', width:'90%' });
The problem is fixed if I supply a fixed width on my content, but that is not feasible for what we are doing. The Colorbox also stays inside of the window, it is only shrinking the content of the parent.

The Colorbox also stays inside of the window, it is only shrinking the content of the parent.

This shouldn't happen. I feel like something else is going on that would not be possible to diagnose without seeing a demo that could be inspected.

There was a weird style on the parent that was causing this issue.


PendingDeprecationWarning on use of import imp

Install flask:
pip install flask
If test.py contains:
import flask

A PendingDeprecationWarning is emitted:
python  -Werror test.py
Traceback (most recent call last):
  File "test.py", line 1, in <module>
    import flask
  File "C:\Users\John Hagen\AppData\Local\Programs\Python\Python35-32\lib\site-packages\flask\__init__.py", line 21, in <module>
    from .app import Flask, Request, Response
  File "C:\Users\John Hagen\AppData\Local\Programs\Python\Python35-32\lib\site-packages\flask\app.py", line 28, in <module>
    from .config import ConfigAttribute, Config
  File "C:\Users\John Hagen\AppData\Local\Programs\Python\Python35-32\lib\site-packages\flask\config.py", line 12, in <module>
    import imp
  File "C:\Users\John Hagen\AppData\Local\Programs\Python\Python35-32\lib\imp.py", line 33, in <module>
    PendingDeprecationWarning, stacklevel=2)
PendingDeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses

Possible duplicated as #1449, PR #1450 should fix this problem. Please try the fresh version on github.


Feat/search button

Codecov Report

Merging #139 (5745bf7) into master (f75f1c8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #139   +/-   ##
=======================================
  Coverage   95.71%   95.71%
=======================================
  Files           7        7
  Lines         490      490
=======================================
  Hits          469      469
  Misses         21       21

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 f75f1c8...5745bf7. Read the comment docs.


Update common files.

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
48d6c162fe56105d6ee3cb929edf82d238461633
link
/test operator-lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Argh @geeknoid. Looking at my common-files PR that included the changes wasn't targeted for master but the forOperator branch. My bad I guess. I'll need to recreate the PR I think.

@sdake Sure.

Argh @geeknoid. Looking at my common-files PR that included the changes wasn't targeted for master but the forOperator branch. My bad I guess. I'll need to recreate the PR I think.

thanks for taking on the convergence of my work with @geeknoid 's work in https://github.com/istio/common-files/pull/21.
I expect we will roll out new updates of common-files as time passes.  The immediate goal is to get this structure rolled out across the various repositories in a way that doesn't break the existing repos.  This PR should do that - as it disables the docker build trial by default and makes opt-in a feature.
Cheers
-steve

/retest

#167 has been merged, please rebase to resolve the linter issue.

rebases are not required with repos on the tide plan.


Messages: Fix for production using Globalize runtime

Fixes #32

@alunny, the above fix works for me. It would be great if you could test it in your projects too before we land.
PS: We really need to get #1 addressed. :see_no_evil:

@alunny and @kborchers if there's no objection, I plan to land this and create a new patch release.

I don't have a project to test in but reading it lgtm 👍

Also agree on needing tests. If I can find any free time (lol) I would love to look into Intern or QUnit + https://github.com/airbnb/enzyme


Do less file system crawling when we have the Dart dependency set

[x] Skip scanning the file system if we already have the Dart dependency set.

@eseidelGoogle @jason-simmons
Fixes #9376
Performing hot reload...
Reloaded 1 of 418 libraries in 888ms.

Performing hot reload...
Reloaded 1 of 418 libraries in 871ms.

** UNTAR dragontail under project root **
Performing hot reload...
Reloaded 0 of 418 libraries in 443ms.

** UNTAR dragontail under lib/ **
Performing hot reload...
Reloaded 0 of 418 libraries in 385ms.

@jason-simmons @tvolkert @cbracken PTAL

LGTM

Very kind of you, thank you. :)


spl-token-cli: Can't create additional account for token

I tried to make new account based on https://spl.solana.com/token#example-transferring-tokens-to-an-explicit-recipient-token-account

$ spl-token create-account 2JVy7J2ac482nbbnYwUW9mKGbJucW7vgviuDWZRCEJVK
Creating account DXLXQQ42f3poH5MkFLrCqEmUskqFFKwU8skr1rDUSNJ5
Error: Account already exists: DXLXQQ42f3poH5MkFLrCqEmUskqFFKwU8skr1rDUSNJ5

I have also different look of list of accounts
$ spl-token accounts
Token                                         Balance
---------------------------------------------------------------
2JVy7J2ac482nbbnYwUW9mKGbJucW7vgviuDWZRCEJVK  199999667

spl-token-cli 2.0.9
solana-cli 1.6.1
RPC URL: https://devnet.solana.com

spl-token create-account tries to create the associated token account for the owner by default.  If you would like to create an auxiliary account, you need to pass the optional KEYPAIR positional parameter (solana-keygen new --no-passphrase -so throwaway-address.json).  This keypair is only used to reserve the new account's address and can be deleted after the account is created, --owner will have full control.
$ spl-token create-account --help
spl-token-create-account
Create a new token account

USAGE:
    spl-token create-account [FLAGS] [OPTIONS] <TOKEN_ADDRESS> [--] [KEYPAIR]

[SNIP]

ARGS:
    <TOKEN_ADDRESS>    The token that the account will hold
    <KEYPAIR>          Specify the account keypair. This may be a keypair file or the ASK keyword. [default:
                       associated token account for --owner]

I have also different look of list of accounts

Thanks for the heads up. We made some changes to this call recently and need to update docs

Looks like this was fixed awhile ago in the docs


Reloadable WebServer TLS during runtime

Fixes #2863

Yes, those already connected are still working fine, but the new ones are validated against the new TLS.


Gracefully stop the PHP process before max execution time is reached

If our lambda reach the maximum execution time, it gets shut down. Since we use an external log system, we don't get any log in that case and we still have to check on cloudwatch what happened.
We could use set_time_limit() or ini_set() with the max_execution_time directive. But the issue with those functions is that you can't take control back when the limit is reached, the PHP process is immediatly stopped. The other issue is that systems calls are not taken in account (eg sleep(), file_get_contents(), etc.).
So I'm trying to find a solution to handle that case.
I had the following idea: we could configure a specific timeout  (obviously shorter than the lambda max execution time) which, when it's reached, trigger a signal sending to the PHP process (eg SIGTERM).
This could let some time to the PHP process to gracefully shut down and do whatever is required during the time left (eg log what happened).
And this should obviously be handled in the parent process, the javascript one (and I'm not sure to be skilled enough in javascript to achive that).
This is just an idea, any other idea is very welcome.

I made a simple POC to validate my idea.
Here a simple PHP script which handle signals and runs forever:
<?php
declare(ticks = 1);

class SigtermException extends \Exception {}

pcntl_signal(SIGTERM, function () {
	throw new SigtermException('SIGTERM received');
});

set_exception_handler(function ($exception) {
	// Log exception as usual
	echo "Exception thrown: " , $exception->getMessage(), "\n";
	// Is it's a Sigterm one, stop execution
	if ($exception instanceof SigtermException) {
		exit(1);
	}
});

for(;;){
	echo "Running\n";
	usleep(100000);
}

And here is the javascript handler which runs the PHP script:
const spawn = require('child_process').spawn;

var script = spawn('php', ['test.php']);
var timeoutHandler = setTimeout(() => script.kill('SIGTERM'), 2000);

script.on('close', (code) => {
	console.log('Process closed with code: ' + code);
	clearTimeout(timeoutHandler);
});

script.stdout.on('data', function(data) {
    console.log(data.toString());
});

script.stderr.on('data', function(data) {
    console.log('[STDERR] ' + data.toString());
});

And finally the execution output:
$ nodejs handler.js
Running

Running

[...]

Running

Exception thrown: SIGTERM received

Process closed with code: 1

So it seems to be working as expected and would allows us to to gracefully shut down the PHP process before the lambda shuts down.

Yes that is really interesting!
Just to be the devil's advocate: when not using AWS lambda (i.e. Apache/Nginx setup) there is a time limit too and there is the same problem right? So in a way my opinion is that since it's not a new constraint specific to lambdas, it's not critical to offer a solution for this.
However if it's easy and doesn't impact a lot the architecture of Bref then why not! And since we control the PHP binary we can make sure pcntl is available.

Just to be the devil's advocate: when not using AWS lambda (i.e. Apache/Nginx setup) there is a time limit too and there is the same problem right? So in a way my opinion is that since it's not a new constraint specific to lambdas, it's not critical to offer a solution for this.

We shouldn't think as if we were working on a classic setup. On natively supported languages, AWS provides a context  object which provides a method to get the remaining execution time. That's exactly why I had this idea, in other languages we can check the time left and that's not possible at the moment with PHP/Bref.

A simple solution is the following:
pcntl_async_signals(true);
pcntl_signal(SIGALRM, function () {
     throw new \RuntimeException('Maximum execution time reached');
});
pcntl_alarm(2); // a SIGALRM will be triggered after 2 seconds

This could work and wouldn't require any modification in Bref.


PHPDoc Tag - update valid tags

List of "valid" tags are not up to date.
I'd like to create a PR, but first I need to know if deprecated tags are considered valid and where to target it - 2.2 or master?

deprecated tags are valid IMO, but cc @GrahamCampbell
2.2 as LTS shall be the target

ping @GrahamCampbell

ping @GrahamCampbell

See https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3140.

Actually only PhpdocSeparationFixer and PhpdocSeparationFixer are relying on the list of "valid" tags, so it does not have much impact on the codebase.


remove .d from types extension

.d is meant for ambient types, for example module augmentation; in these cases however, types are explicitly exported and used directly. It's not a huge deal, but it doesn't work in some environments, for example, with tsconfig-paths-jest, which expects plain index.ts files.

Can you also fix the scripts renaming index.d.ts here and here and here. Otherwise tests will fail.
Also can you pull again from dev. I just merged eslint feature. So there are easy conflicts.

fixed and rebased on eslint, should work

Travis reports problem in app/types/index.ts 41:3, but there's only 38 lines in that file? idk
I find react-boilerplate to be extremely convoluted and overengineered in general tbh

Yeah I know that bug. I will fix soon. Thats why I merged. Now after merging dev branch throws more eslint errors on my local and didn't even show up in travis ;(


Automated cherry pick of #6375: fix(esxi): Set 'IsInit' as true when rebuildroot

Cherry pick of #6375 on release/3.1.
#6375: fix(esxi): Set 'IsInit' as true when rebuildroot

Codecov Report

Merging #6377 into release/3.1 will increase coverage by 0.04%.
The diff coverage is 17.03%.

@@              Coverage Diff               @@
##           release/3.1   #6377      +/-   ##
==============================================
+ Coverage         7.45%   7.49%   +0.04%
==============================================
  Files              584     585       +1
  Lines            98663   98839     +176
==============================================
+ Hits              7354    7409      +55
- Misses           90738   90857     +119
- Partials           571     573       +2

Flag
Coverage Δ

#aFlag
7.49% <17.03%> (+0.04%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/appsrv/workers.go
65.46% <ø> (-0.25%)
:arrow_down:

pkg/cloudcommon/db/db_dispatcher.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/tenantcache.go
1.31% <0.00%> (ø)

pkg/cloudcommon/policy/policy.go
1.58% <0.00%> (+0.04%)
:arrow_up:

pkg/cloudprovider/cloudprovider.go
1.70% <0.00%> (-0.13%)
:arrow_down:

pkg/cloudprovider/instance.go
0.00% <ø> (ø)

pkg/compute/guestdrivers/aws.go
3.96% <0.00%> (-0.14%)
:arrow_down:

pkg/compute/guestdrivers/base.go
0.00% <0.00%> (ø)

pkg/compute/guestdrivers/managedvirtual.go
0.00% <0.00%> (ø)

pkg/compute/models/cloudaccounts.go
0.79% <0.00%> (+<0.01%)
:arrow_up:

... and 41 more

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 99e30e2...e385bb2. Read the comment docs.


Cannot specify type = drop for cloudflare_email_routing_catch_all

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
$ ./terraform -v
Terraform v1.3.1
on linux_amd64
+ provider registry.terraform.io/cloudflare/cloudflare v3.24.0

Affected resource(s)
cloudflare_email_routing_catch_all
Terraform configuration files
terraform {
    required_providers {
        cloudflare = {
            source = "cloudflare/cloudflare"
        }
    }
}

provider "cloudflare" {
    api_token = "0123456789012345678901234567890123456789"
}

resource "cloudflare_email_routing_catch_all" "foobar" {
    zone_id = "YYY"
    name = "Catch-all rule"
    enabled = true
    matcher {
        type = "all"
    }
    action {
        type = "drop"
        value = []
    }
}

Link to debug output
https://gist.github.com/yan12125/f683cbbd61c09b8e2544c30cd6acad37
Panic output
No response
Expected output
Catch-all rule for email routing is enabled and configured to drop all mails.
Actual output
The provider does not like type = drop for cloudflare_email_routing_catch_all.
Steps to reproduce

Run terraform plan

Additional factoids
Per https://api.cloudflare.com/#email-routing-routing-rules-update-catch-all-rule, the catch-all rule accepts type = drop, forward or worker.
References
No response

moving to enhancement as this is a new option.
feel free to send over a PR if you'd it addressed immediately.

Thanks! It's not urgent. I may have a look recently.


No output to Android console

I'm trying to add the logger and flutter_logger packages to my app.
When I run them using the Android Studio emulator I get the output on the Studio run console, but calling LogConsole.open gives me an empty console display on the phone.
Is this expected?  If so, is there some way to get output to the phone, or even both, so I can test my phone logging is working correctly?

Seems like this is expected.  Slightly more assembly required than the sample app suggested.  Got it sorted though.  I'll push it later.


Check for update to the app on launch

If there is a new release on github, the app should inform the user about it so they can download the update.

https://developer.github.com/v3/repos/releases/#get-the-latest-release

Yep, was just looking at that page :P


Remove version: Peppy.Osu! version 2023.908.1

https://github.com/microsoft/winget-pkgs/issues/152768#issuecomment-2102735187

Resolves #152768

Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/ppy/osu/releases/download/2023.908.1/install.exe
OK
(Automated message - build 882)


How mysql with 'like' or 'where'?

I want to make my sql:
select uuid,name,age from user where name='www' limit 1,10;

and i write code:
connection.query('select ?? from user where ? limit ?,?', [
	['uuid', 'name', 'age'], {
		name: 'www'
	}, (page.currPage - 1) * page.pageSize, page.pageSize
], function(err, result) {
       //some code
})

But, if i want this sql:
select uuid,name,age from user where name like 'www' limit 1,10;

I donot know how to write ；
——————————————
and more，
if i want this sql:
it Support
select uuid,name,age from user where name like 'www' limit 1,10;

At the same time，it Support：
select uuid,name,age from user  limit 1,10;

I donot know how to write ；
————————————————
Thank you looking forward to answer

I Used sequelize to solve the problem

Hi @wangwenjie1314 since this is just a simple substitution, you would have to write select ?? from user where name like ? limit ?,? as the query. If you're looking for a full-features sql builder, sequalize can do it :)


[ai anomaly detector] Update to core-tracing 1.0

Packages impacted by this PR

[@azure/ai-anomaly-detector]

Issues associated with this PR

#21199

Describe the problem that is addressed by this PR
Updating to core-client and core-rest-pipeline as well as core-tracing 1.0 GA.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-anomaly-detector

@mpodwysocki @xirzec I have generate RLC for anomaly detector https://github.com/Azure/azure-sdk-for-js/tree/main/sdk/anomalydetector/ai-anomaly-detector-rest. I wonder if we still need to upgrade the core-tracing dependency for it ?

@mpodwysocki @xirzec I have generate RLC for anomaly detector https://github.com/Azure/azure-sdk-for-js/tree/main/sdk/anomalydetector/ai-anomaly-detector-rest. I wonder if we still need to upgrade the core-tracing dependency for it ?

Ah good point. I think we can merge this, but should consider removing this SDK to put all of our energy into the RLC


ndimage.convolve/correlate (u)Int64 Underflow

When using int64/uint64 images, weights/kernel, and output in ndimage.convolve/correlate the results drop many of the least significant digits. This seems to be because even though the input and output are kept as their types the weights/kernel are always converted to float64 and each block is computed using those types.
The only solution I can think of is having a completely separate branch for when all three of input, weights, and output are integral and one of input or weights and the output has a larger bit-depth then the mantissa of float64 (52).
Personally I am dealing with uint8 inputs but uint64 kernels and needing the result in a uint64 output and data is being lost.
Reproducing code example:
import numpy as np
import scipy.ndimage as ndi
im = np.array([[2**62 + 1]], np.int64)
kernel = np.array([[1]], np.int64)
output = np.empty((1,1), np.int64)
ndi.convolve(im, kernel, output=output)
output[0,0] == im[0,0] # this is False but should be True

Changing the 62 in the above to 52 results in True since it matches the size of the mantissa.
Scipy/Numpy/Python version information:
1.3.0 1.16.4 sys.version_info(major=3, minor=7, micro=3, releaselevel='final', serial=0)

Semi related: https://github.com/scipy/scipy/issues/6986 where int64 precision is lost via round trip pto float64


js error when adding catalog item under non-(super)admin user

@dajohnso please validate

Downstream BZ - https://bugzilla.redhat.com/show_bug.cgi?id=1336888

@chessbyte , reproduced and has the above BZ (1336888)

Yes, this problem happens when adding a service catalog item for a provider type, for which
there are no vm templates available. Bug https://bugzilla.redhat.com/show_bug.cgi?id=1336888

works for me after
https://github.com/ManageIQ/manageiq/pull/8810
thanks @mzazrivec


Add the Read DAL for Logins using MongoDB

As a solution architect I want to choose which Data Storage technology I use for a deployment. Currently Keyin Coders are supporting both PostgreSQL and MongoDB for our customers.

We only got to implementing the READ aspects of the CRUD.


"undefined method `wait_for_train' for Class:Class (NoMethodError)" when running pilot upload

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Complete output when running fastlane, including the stack trace and command used
Command I ran:
fastlane pilot upload -i path/to/build.ipa
Output:
[21:55:46]: ---------------------------------------------------------------------------------------------------------------
[21:55:46]: --- Successfully uploaded package to iTunes Connect. It might take a few minutes until it's visible online. ---
[21:55:46]: ---------------------------------------------------------------------------------------------------------------
[21:55:46]: Successfully uploaded the new binary to iTunes Connect
[21:55:46]: If you want to skip waiting for the processing to be finished, use the `skip_waiting_for_build_processing` option

Looking for related GitHub issues on fastlane/fastlane...

Found no similar issues. To create a new issue, please visit:
https://github.com/fastlane/fastlane/issues/new
Run `fastlane env` to append the fastlane environment to your issue
/Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/fastlane_core/lib/fastlane_core/build_watcher.rb:32:in `wait_for_build': [!] undefined method `wait_for_train' for Class:Class (NoMethodError)
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/pilot/lib/pilot/build_manager.rb:34:in `upload'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/pilot/lib/pilot/commands_generator.rb:59:in `block (2 levels) in run'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/commander-4.4.3/lib/commander/command.rb:178:in `call'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/commander-4.4.3/lib/commander/command.rb:178:in `call'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/commander-4.4.3/lib/commander/command.rb:153:in `run'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/commander-4.4.3/lib/commander/runner.rb:446:in `run_active_command'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:38:in `run!'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/commander-4.4.3/lib/commander/delegates.rb:15:in `run!'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/pilot/lib/pilot/commands_generator.rb:158:in `run'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/pilot/lib/pilot/commands_generator.rb:12:in `start'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/fastlane/lib/fastlane/cli_tools_distributor.rb:49:in `take_off'
	from /Users/.../.rbenv/versions/2.0.0-p647/lib/ruby/gems/2.0.0/gems/fastlane-2.18.1/bin/fastlane:15:in `<top (required)>'
	from /Users/.../.rbenv/versions/2.0.0-p647/bin/fastlane:23:in `load'
	from /Users/.../.rbenv/versions/2.0.0-p647/bin/fastlane:23:in `<main>'

Environment
✅ fastlane environment ✅
Stack

Key
Value

OS
10.11.6

Ruby
2.0.0

Bundler?
false

Git
git version 2.10.0

Installation Source
~/.rbenv/versions/2.0.0-p647/bin/fastlane

Host
Mac OS X 10.11.6 (15G1217)

Ruby Lib Dir
~/.rbenv/versions/2.0.0-p647/lib

OpenSSL Version
OpenSSL 1.0.2f  28 Jan 2016

Is contained
false

Is homebrew
false

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

Xcode Version
8.2.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
No Fastfile found
`./fastlane/Appfile`
itc_team_name "..."

fastlane gems

Gem
Version
Update-Status

fastlane
2.18.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
1.5.1

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.2.0

public_suffix
2.0.5

addressable
2.5.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.5.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander
4.4.3

excon
0.55.0

faraday
0.11.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20161129

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.0.1

gh_inspector
1.0.3

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

multi_json
1.12.1

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
2.8.3

google-api-client
0.9.28

json
1.7.7

mini_magick
4.5.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.2.0

bundler
1.14.5

faraday_middleware
<IP_ADDRESS>

i18n
0.7.0

thread_safe
0.3.5

tzinfo
1.2.2

minitest
5.8.4

activesupport
<IP_ADDRESS>

xcodeproj
1.1.0

claide
1.0.1

generated on: 2017-02-21

Thank you for the report. We are going to ship a hot-fix release to address this. 👍

fastlane 2.18.2 has been released. Please run fastlane update_fastlane and let us know if the latest versions fixes this for you! Thanks!


Where is the wavenet input "tacotron_output/gta/map.txt"?

I have trained out a Tacotron model and  the directory structure is as follows:
tacotron_output
├──eval
│              ├──map.txt
│              └──*.npy
└──logs-eval
├──plots
└──wavs
And then I wanted to train the WaveNet model but it confused me that I need the input tacotron_output/gta/map.txt. Where can I find it? Is it tacotron_output/eval/map.txt? But the matadata fetched in the feeder seemed not the proper one.

You need to make a GTA synthesis using the Tacotron model. If you trained
using "Tacotron-2" or "Both" as model name, did it not launch automatically?
On Fri, 25 May 2018, 03:47 Leo Ma<EMAIL_ADDRESS>wrote:

I have trained out a Tacotron model and the directory structure is as
follows:
tacotron_output
├──eval
│ ├──map.txt
│ └──*.npy
└──logs-eval
├──plots
└──wavs
And then I wanted to train the WaveNet model but it confused me that I
need the input tacotron_output/gta/map.txt. Where can I find it? Is it
tacotron_output/eval/map.txt? But the matadata fetched in the feeder
seemed not the proper one.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Rayhane-mamah/Tacotron-2/issues/52, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AhFSwMnXv0azJi7PxJ5toKP6el1qCTO1ks5t13DGgaJpZM4UNSM0
.

@Rayhane-mamah It still throws exception in feeder.py after I merged your latest commit. It seems the args.input_path kept the old one.

Traceback (most recent call last):
File "train.py", line 83, in
main()
File "train.py", line 75, in main
wavenet_train(args, log_dir, hparams, args.wavenet_input)
File "/home/leoma/Tacotron-2/wavenet_vocoder/train.py", line 243, in wavenet_train
train(log_dir, args, hparams, input_path)
File "/home/leoma/Tacotron-2/wavenet_vocoder/train.py", line 163, in train
feeder = Feeder(coord, input_path, args.base_dir, hparams)
File "/home/leoma/Tacotron-2/wavenet_vocoder/feeder.py", line 36, in init
with open(metadata_filename, 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'tacotron_output/gta/map.txt'

I believe the correct way to run the synthesis is:

python synthesize.py --mode=synthesis --GTA=True --model='Tacotron'

Since (ec0cd50d24d7f5561f696a78d0cd47d5a2a14a43) I have maid the "eval" mode default behavior for synthesis assuming "Tacotron-2" will be default behavior for synthesis in the future, so to generate using GTA, one must explicitly set mode=synthesis
It will not work if you use "Tacotron-2" For synthesis on all dataset because it supposes you want to generate using Both Tacotron and Wavenet, using model=Tacotron will do the task you wanna achieve. I will retest the entire pipeline later to check if this is still a problem..

Solved similar problems by placing absolute paths, especially for base_dir.
I'm using Ubuntu 16.04, Python 3.5, Tensorflow 1.6 and Keras 2.0.2. All the rest of the libraries are the exact versions required (the ones I mentioned earlier are ones whose versions aren't explicitly mentioned in this repo).
Encountered the following problems before I did this:

Can't find metadata.csv during preprocessing; and
griffin_lim issue (missing hparam arg) during training, specifically right after the training script created a checkpoint and was doing eval.
P.S.  I'm using 'Tacotron-2' (both) for training.

@MXGray please verify that this line is not commented in your train.py:
https://github.com/Rayhane-mamah/Tacotron-2/blob/fb5564b7584ae0dc62ffecaa89d463ff24a3c251/train.py#L34
Run the training again with Tacotron-2, it will skip Feature Prediction Network training whem it finds the checkpoint. It should run GTA synthesis then wavenet training. Tell me how it goes

@Rayhane-mamah
Thanks! Seems to be working now - Used --model='Tacotron-2', and training skipped immediately to wavenet training after detecting the latest feature prediction checkpoint (50KS).
It's now at 3% synthesis; and
Here's what I de-commented and commented in train.py:
input_path = tacotron_synthesize(args, hparams, checkpoint)
#input_path = 'tacotron_output/gta/map.txt'

Cheers! :)

Yeah these lines were wrong in previous version of the repo but are fixed now. Feel free to share your results with Wavenet once it's fixed

@Rayhane-mamah
Been encountering errors while trying to synthesize custom text using the saved checkpoints for both the encoder network and the wavenet vocoder; and
My CLI command is:
python synthesize.py --model='Tacotron-2' --mode='eval'
And, I get this:
synthesize(args, hparams, taco_checkpoint, wave_checkpoint, sentences)
synthesize.py, line 33, in synthesize
wavenet_synthesize(args, hparams, wave_checkpoint)
/wavenet_vocoder/synthesize.py", line 78, in wavenet_synthesize
run_synthesis(args, checkpoint_path, output_dir, hparams)
/wavenet_vocoder/synthesize.py", line 19, in run_synthesis synth.load(checkpoint_path, hparams)
/wavenet_vocoder/synthesizer.py", line 27, in load  sh_saver = create_shadow_saver(self.model)
/wavenet_vocoder/train.py", line 53, in create_shadow_saver return tf.train.Saver(shadow_dict, max_to_keep=5)
/tensorflow/python/training/saver.py", line 1293, in init self.build()
/tensorflow/python/training/saver.py", line 1302, in build self._build(self._filename, build_save=True, build_restore=True)
/tensorflow/python/training/saver.py", line 1339, in _build build_save=build_save, build_restore=build_restore)
/tensorflow/python/training/saver.py", line 796, in _build_internal restore_sequentially, reshape)
/tensorflow/python/training/saver.py", line 449, in _AddRestoreOps restore_sequentially)
/tensorflow/python/training/saver.py", line 847, in bulk_restore return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
/tensorflow/python/ops/gen_io_ops.py", line 1030, in restore_v2
shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper op_def=op_def)
/tensorflow/python/framework/ops.py", line 3271, in create_op op_def=op_def)
/tensorflow/python/framework/ops.py", line 1650, in init self._traceback = self._graph._extract_stack()  # pylint: disable=protected-accessNotFoundError (see above for traceback): Key model_1/model/ResidualConv1dGLU_0/residual_block_cin_conv/bias_residual_block_cin_conv/ExponentialMovingAverage not found in checkpoint
[[Node: model_1/save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_model_1/save/Const_0_0, model_1/save/RestoreV2/tensor_names,model_1/save/RestoreV2/shape_and_slices)]]
What could be the problem?

Hello again @MXGray, yes I have encountered same problems too, didn't find time to upload corrections today, I will upload it in a couple of minutes

Hi @Rayhane-mamah,
Just wanted to kindly ask if you've already uploaded the corrected files?
Please advise. Thanks! :)

Yes I did, the bug is mainly due to Exponential Moving Average requesting checkpoints from a wrong scope at synthesis time. I don't quite remember which scopes were wrong, but this issue is supposed to be fixed 2 commits ago, naturally if you're using a model trained before the scopes bug was fixed, checkpoint files will still have wrong scopes in them..
I am hoping you didn't train the Wavenet for too long..

i trained the model with the latest code(e2f9780b18648b8d0b671e1c85dd95d20d6f8553), but when i executed 'python synthesize.py', the WaveNet model still load failed， the details #82

@MXGray
when i train with tacotron2:
python train.py --model='Tacotron-2'
it stuck in wavenet train.
[2018-06-25 12:49:19.476]  No model to load at logs-Tacotron-2/wave_pretrained/
[2018-06-25 12:49:19.550]
Generated 62 test batches of size 1 in 0.073 sec
[2018-06-25 12:49:19.553]
Generated 32 train batches of size 2 in 0.076 sec
[2018-06-25 12:49:23.835]  Exiting due to Exception: Conv2DCustomBackpropInputOp only supports NHWC.
[[Node: model_1/inference/conv2d_transpose/conv2d_transpose = Conv2DBackpropInput[T=DT_FLOAT, data_format="NCHW", dilations=[1, 1, 1, 1], padding="SAME", strides=[1, 1, 1, 16], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:CPU:0"](model_1/inference/conv2d_transpose/stack, model/inference/conv2d_transpose/kernel/read, model_1/inference/ExpandDims_1)]]
Caused by op 'model_1/inference/conv2d_transpose/conv2d_transpose', defined at:
File "train.py", line 127, in
main()
File "train.py", line 121, in main
train(args, log_dir, hparams)
File "train.py", line 75, in train
checkpoint = wavenet_train(args, log_dir, hparams, input_path)
File "/data02/zhangyb/Tacotron-2/wavenet_vocoder/train.py", line 244, in wavenet_train
return train(log_dir, args, hparams, input_path)
File "/data02/zhangyb/Tacotron-2/wavenet_vocoder/train.py", line 167, in train
model, stats = model_train_mode(args, feeder, hparams, global_step)
File "/data02/zhangyb/Tacotron-2/wavenet_vocoder/train.py", line 117, in model_train_mode
feeder.input_lengths, x=feeder.inputs)
File "/data02/zhangyb/Tacotron-2/wavenet_vocoder/models/wavenet.py", line 169, in initialize
y_hat = self.step(x, c, g, softmax=False) #softmax is automatically computed inside softmax_cross_entropy if needed
File "/data02/zhangyb/Tacotron-2/wavenet_vocoder/models/wavenet.py", line 435, in step
c = transposed_conv(c)
File "/data02/zhangyb/Tacotron-2/wavenet_vocoder/models/modules.py", line 333, in call
return self.convt(inputs)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/layers/base.py", line 696, in call
outputs = self.call(inputs, *args, **kwargs)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/layers/convolutional.py", line 1653, in call
data_format=utils.convert_data_format(self.data_format, ndim=4))
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/nn_ops.py", line 1257, in conv2d_transpose
name=name)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/gen_nn_ops.py", line 805, in conv2d_backprop_input
dilations=dilations, name=name)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3271, in create_op
op_def=op_def)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1650, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
InvalidArgumentError (see above for traceback): Conv2DCustomBackpropInputOp only supports NHWC.
[[Node: model_1/inference/conv2d_transpose/conv2d_transpose = Conv2DBackpropInput[T=DT_FLOAT, data_format="NCHW", dilations=[1, 1, 1, 1], padding="SAME", strides=[1, 1, 1, 16], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:CPU:0"](model_1/inference/conv2d_transpose/stack, model/inference/conv2d_transpose/kernel/read, model_1/inference/ExpandDims_1)]]

when train with option "tacotron2": python train.py --model='Tacotron-2'
it stuck in wavenet train:
[zhangyb@localhost Tacotron]$ tail nohup.out
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3290, in create_op
op_def=op_def)
File "/data02/zhangyb/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1654, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
InvalidArgumentError (see above for traceback): Conv2DCustomBackpropInputOp only supports NHWC.
[[Node: model/inference/conv2d_transpose/conv2d_transpose = Conv2DBackpropInput[T=DT_FLOAT, data_format="NCHW", dilations=[1, 1, 1, 1], padding="SAME", strides=[1, 1, 1, 16], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:CPU:0"](model/inference/conv2d_transpose/stack, model/inference/conv2d_transpose/kernel/read, model/inference/ExpandDims_1)]]

@MXGray do you have successfully run "python synthesize.py --model='Tacotron-2' --mode='eval'"?

I believe this is fixed with latest commit, if the problem persists please reopen the issue!

Can you tell me how to fix the bug?I'm using someone else's code basing on older version of this,i can't simply replace it with current code.Thank you!


Remove some dependencies of Aries FSM on upper layers

Signed-off-by: Patrik Stas<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (refactor/send-message@0d28903). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             refactor/send-message     #251   +/-   ##
========================================================
  Coverage                         ?   52.09%
========================================================
  Files                            ?      148
  Lines                            ?    18776
  Branches                         ?     4283
========================================================
  Hits                             ?     9781
  Misses                           ?     5952
  Partials                         ?     3043

Flag
Coverage Δ

unittests
52.09% <0.00%> (?)

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

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 0d28903...e76126e. Read the comment docs.


Cleanup cli inspect code

I dunno, there is more code lines and it became harder to read for me because of all this continue.

Updated.
It's a few extra lines in the actual code (I think +4), but I find this more readable.
WDYT?

@cpuguy83 Now looks better for me. I can read through code and understand all of it.

Yeah, but there's some issues :(
This is a real mess.

@cpuguy83 indeed. At least we have tests :)


Add the MailChimp as the maillist.

Adding the mail list to the specific project, my personal blog is too broad, and not valuable for others.


Cannot display whale emoji inside docker tmux

There is a bug while displaying emoji inside of docker tmux. It replaces emoji with __

Just founded a solution here. Please refer to the link.
A fix is on the way:)

Just founded a solution here. Please refer to the link.
A fix is on the way:)


fix: ensure irods env settings get priority

Ensure that settings in ~/.irods/irods_environment.json have higher priority than default setting values, so that users don't have to copy settings from the environment file to the Snakefile.
(This is a proposed fix for issue #4)

@stsnel do you want to become a maintainer of this repo? This way, you would also be able to merge and release yourself.

@johanneskoester Yes, please, thank you :+1:

Sorry for the delay, I made you a maintainer. Note the conflict in this PR.


'sLSTMLayer' object has no attribute 'step'

Hi, thank so much for your work.
I am testing the model on multiple configs. While using step() method to get both the output and the states, I observed that models with sLSTM layer does not have method step. Instead, to get the state, we must use the argument return_last_state=True. This causes the xLSTM Language model cannot get state. This is my code used:
from omegaconf import OmegaConf
import torch
from dacite import from_dict
from dacite import Config as DaciteConfig
from xlstm import xLSTMLMModel, xLSTMLMModelConfig

xlstm_cfg = """
vocab_size: 50304
mlstm_block:
  mlstm:
    conv1d_kernel_size: 4
    qkv_proj_blocksize: 4
    num_heads: 4
slstm_block:
  slstm:
    backend: vanilla
    num_heads: 4
    conv1d_kernel_size: 4
    bias_init: powerlaw_blockdependent
  feedforward:
    proj_factor: 1.3
    act_fn: gelu
context_length: 256
num_blocks: 7
embedding_dim: 128
slstm_at: [1]
"""
cfg = OmegaConf.create(xlstm_cfg)
cfg = from_dict(data_class=xLSTMLMModelConfig, data=OmegaConf.to_container(cfg), config=DaciteConfig(strict=True))
model = xLSTMLMModel(cfg)

x = torch.randint(0, 50304, size=(4, 256)).to("cpu")
model = model.to("cpu")

model.step(torch.Tensor([1]).unsqueeze(dim=0).long())

This is the error:
AttributeError: 'sLSTMLayer' object has no attribute 'step'
I wonder can you change the method in the sLSTM for analogousness. Thank you so much.

which is your version？


No tags or branches in repo for releases

Hello,
I like gatling project, but I would like to see some improvements regarding its release management. Project is evolving without much compatibility between versions and thus to extend it (in any way) is a bit harder. For example I have coded AMQP (with proper request/reply support) plugin/protocol and it is not usable after dependency version change. This behavior is OK, as I can depend on some specific version and I have sources for this version.
I would like to have tags (or branches) in github repo for all publicly released/accessible gatling versions. I.e. I can migrate prometheus datawriter to current version easyli. Now I am struggling to have right "Engine.scala" version, correct GatlingConfiguration class (compatible with runner), e.t.c.
Please add tags for all released gatling libraries to github repo.

User visible changes are documented in our migration guides, which is enough for standard usage.
The components you're referring to are internals and subjects to changes at any time without any further notice.
Then, regarding extending, it can mean different things:

choosing to develop a independent third party instead of contributing upstream. This sure comes at the price: maintenance. If such plugin was to be contributed upstream, we could maintain it ourselves and it would help our broader community.
forking to workaround our rule that improvements fall in the scope of FrontLine, typically stats and reporting. That's entirely the author of such unfriendly fork's responsibility. This is what the project you're referring to is. If you want Prometheus integration, you can use the official Prometheus Graphite exporter instead.

To some extent, we can answer questions from authors of independent yet non-unfriendly third parties regarding internals changes. But we won't do any kind of change and extra work that doesn't serve a purpose in official Gatling components.
I hope you'll understand.

Thanks for long and clarifying answer. I see it now and I agree. I hit given prometheus plugin because of "cheap way to add some ad-hoc test to monitoring". I have not found prometheus graphite exporter and it seems it would solve my task.
PS: Would you be interested in PR with some "javadoc" mostly for http DSL? I miss it and frequently to have a look at gatling cheat sheet. Perhaps I would mostly copy/paste given things (which is not ideal) and add http link directly to cheat sheet. If something is nontrivial to use (for example checkIf), I would add also some sample code.
PS2: Are you interested also in contribution for "weird" protocols? I have on my mind for example DLMS. I will (in a few months) consider implementing one basic perf test using gatling. Have on mind, that DLMS is huge protocol and I will implement just some basic things for simulating many gas/power/water meters.

Thanks for your understanding.

Would you be interested in PR with some "javadoc" mostly for http DSL?

Absolutely! Please beware scaladoc's syntax is slightly different from javadoc's. Also, we're yet to switch to latest Scala major version so some syntax might no work yet.

Are you interested also in contribution for "weird" protocols?

Sadly, when there's no market, no. When we accept contribution for new protocols (WebSocket and JMS originated this way), we have to learn the protocol so we can maintain ourselves should the original contributor stop contributing, be able to run tests, etc. This is an investment, and we can only afford doing so if the user base is large enough.
Typically, we'd be very grateful for contributions on AMQP, Kafka, gRPC...


Could not find an installed version of Gradle either in Android Studio, or on your system to install the gradle wrapper. Please include gradle in your path, or install Android Studio

Ionic version:
[x] 3.x
I'm submitting a ...
[x ] bug report
Current behavior:

ionic run android --prod results in below error
Could not find an installed version of Gradle either in Android Studio, or on your system to install the gradle wrapper. Please include gradle in your path, or install Android Studio
Expected behavior:

ionic run android --prod must build and run on device connected
Ionic info:
Cordova CLI: 7.0.0
Ionic Framework Version: 3.1.1
Ionic CLI Version: 2.2.3
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.7
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: macOS Sierra
Node Version: v6.10.1
Xcode version: Xcode 8.2.1 Build version 8C1002

Did you do as suggested by the error message?
You need to install Android Studio make sure it's in your PATH.
Here's my PATH for example:
PATH="/Users/kim/.node/bin:/usr/local/bin:/usr/local/sbin:~/bin:/Users/kim/Library/Android/sdk/platform-tools:/Users/kim/Library/Android/sdk/tools:/Users/kim/Library/Android/sdk/build-tools/22.0.1:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin"

@mburger81 - how to get this on Mac OS?

google how to install gradle on Mac! :smile:

@mburger81 - I meant i am using android sdk via local directory, all development was moving smoothly till now, it was only after v3.1.1 upgrade the gradle stopped working; still i need to install Studio?

Here is the solution I found for Mac
Download the Gradle Binary Only Distribution from https://gradle.org/gradle-download/
Open a terminal
Run the following to unzip the android sdk to the Development directory that is under your user home directory
unzip ~/Downloads/gradle-3.1-bin.zip -d ~/Development
Open up the vi editor and edit your bash profile. We need to add in the GRADLE_HOME environment variable
vi ~/.bash_profile
To edit in vi hit i to enter edit mode and add the text below to the .bash_profile
export GRADLE_HOME=~/Development/gradle-3.1/bin
Press the esc key to exit edit mode
Press : (colon key) to enter command mode.
type wq and press enter to save and exit vi
Run the following to make the change active in your existing session
source ~/.bash_profile
Test it by running
echo $GRADLE_HOME

What to do on windows ?!

Here are the steps for windows:
http://bryanlor.com/blog/gradle-tutorial-how-install-gradle-windows

For me this happened because I upgrade Cordova to version 7.0 and gradle version had to be upgraded to 4.0. I upgraded with brew update && brew install grade on MacOS.
https://gradle.org/install/

I am using Linux Mint 18 Cinnamon and was experiencing the same problem. The problem was happening even though I have an ionic 1 setup and Android that is building with no issues.
I resolved it by installing gradle sudo apt-get install gradle
Gradle was automatically added to the PATH and the various ionic build commands worked without a hitch.
ionic info:
@ionic/cli-utils  : 1.7.0
ionic (Ionic CLI) : 3.7.0

System:

Android SDK Tools : 25.2.2
Node              : v7.10.0
OS                : Linux 4.4
npm               : 4.2.0

I have the same issue. I have android studio. How do i get it in my path?

Run this command to install gradle on a mac:
brew update && brew install gradle

Upgrading gradle worked for me

This issue is closed currently, but it still does not seem to work without explicit installation of Gradle.
In my case I have:

latest Android Studio installed (v3.0.1) (under Applications/ folder which is default location)
latest android-sdk installed

This setup works smoothly for Android native apps development (via Android Studio) but I am still getting error when trying to build ionic based project when trying to compile it with cordova compile android --debug:
CordovaError: Could not find an installed version of Gradle either in Android Studio,
or on your system to install the gradle wrapper. Please include gradle
in your path, or install Android Studio

My ionic info output:
cli packages: (...)

    @ionic/cli-utils  : 1.19.1
    ionic (Ionic CLI) : 3.19.1

global packages:

    cordova (Cordova CLI) : 8.0.0

local packages:

    @ionic/app-scripts : 3.1.5
    Cordova Platforms  : android 7.0.0 ios 4.5.4
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 26.1.1
    ios-sim           : 6.1.2
    Node              : v9.4.0
    npm               : 5.6.0
    OS                : macOS High Sierra
    Xcode             : Xcode 9.2 Build version 9C40b

Environment Variables:

    ANDROID_HOME : /Applications/android-sdk

Misc:

    backend : pro

i solved it by installing it with homebrew :
brew install gradle

This issue shouldn't be closed, the issue is still present using the latest Android Studio, cordova and ionic

This issue should not be closed, I have the latest android studio, ionic and cordova and it will not build even the sample project.

I had this same issue after updating the android ionic platform, my solution was to open Android Studio and SDK Manager to update android 8.0 (Oreo) API Level 26.

Same issue here.
Run:  brew install gradle

export GRADLE_HOME="/usr/local/lib/gradle-4.6/bin"
export PATH="$PATH:$ANDROID_HOME:$ANDROID_HOME/tools:$ANDROID_HOME/platform-tools:$GRADLE_HOME"

On macOS, if you'd prefer to avoid installing an extra copy of Gradle, you can simply use the tooling installed by Android Studio by adding the following to your ~/.bash_profile (or equivalent):
export JAVA_HOME=/Applications/Android\ Studio.app/Contents/jre/jdk/Contents/Home
export ANDROID_HOME=/Users/$USER/Library/Android/sdk
export PATH=${PATH}:$ANDROID_HOME/tools:$ANDROID_HOME/platform-tools:/Applications/Android\ Studio.app/Contents/gradle/gradle-4.4/bin

Note: in the future, you'll likely need to change the version above from gradle-4.4. To check the version you have installed, just:
ls /Applications/Android\ Studio.app/Contents/gradle/

Thank you mburger81.
I can fix the bug in my android project by your help with your blog.
Highly recommend you.
:)

Rather than redundantly installing another copy of Gradle, add your Android Studio gradle to PATH as @bitjson mentioned.
However, it didn't work for me at first, you have to be aware of the space in the path.
(Backslash escaping won't work inside quotes)

<EMAIL_ADDRESS>(install sdkman, sdk install gradle 3.0)working for cordova<EMAIL_ADDRESS>

Thanks to bitjson I could configure the .bash_profile to be:
export JAVA_HOME=$(/usr/libexec/java_home)
export ANDROID_HOME=/Users/$USER/Library/Android/sdk
export PATH=${PATH}:$ANDROID_HOME/tools:$ANDROID_HOME/platform-tools:/Applications/Android\ Studio.app/Contents/gradle/gradle-4.4/bin

And this command (I was gettind EACCESS error):
sudo chmod a+x "/Applications/Android Studio.app/Contents/gradle/gradle-4.4/bin/gradle"
I could ionic cordova run android my app!
(Mac OS X + Ionic + NodeJS = last versions)


Support for Alternate Plotters

Do you have plans to create support for other plotters (such as madMA43v3r)? I am testing this now, but it would be great if I could use this newly refactored plotter in place of the built-in Chia one. This would obviously come with the usual use at your own risk warnings.
https://github.com/madMAx43v3r/chia-plotter

I saw it today and thought about it. If its feasible, I plan to do so. Im checking it.

Notes for me:

For poolkey and farmerkey see output of chia keys show.
tmpdir needs about 220 GiB space, it will handle about 25% of all writes. (Examples: './', '/mnt/tmp/')
tmpdir2 needs about 110 GiB space and ideally is a RAM drive, it will handle about 75% of all writes.
Usage:
chia_plot [OPTION...]
-n, --count arg      Number of plots to create (default = 1, -1 = infinite)
-r, --threads arg    Number of threads (default = 4)
-u, --buckets arg    Number of buckets (default = 128)
-t, --tmpdir arg     Temporary directory, needs ~220 GiB (default = $PWD)
-2, --tmpdir2 arg    Temporary directory 2, needs ~110 GiB [RAM] (default = )
-d, --finaldir arg   Final directory (default = )
-p, --poolkey arg    Pool Public Key (48 bytes)
-f, --farmerkey arg  Farmer Public Key (48 bytes)
--help           Print help

@chefsas: I will definitely add it.

Yeah it looks like more and more the MadMax plotter is going to become the gold standard. The testing I am showing is approximately a 4x improvement in overall plot times vs the default plot process.

@chefsas: Support for Stotik is implemented in testing branch.
https://github.com/tydeno/Ploto/tree/testing-stotikmadmax
Its absolutely untested.
Also longterm; I think we will see madmax's plotter as the official chia plotter. Gene Hoffmann said something along that.

I can't seem to get it to work. I'm didn't go through the install so I'm going to try that next (I didn't want to type everything in again) but I compared and make sure it matched the version in GIT.  I get the attached error.

Here's a copy of my scrubbed config file:
{
"PlotterName": "SASCHIAPLOT",
"EnableAlerts": "true",
"EnablePlotoFyOnStart": "true",
"ChiaWindowStyle": "normal",
"PlotterUsed": "Stotik",
"PathToUnofficialPlotter": "C:/Users/Scott/plotter/chia_plot.exe",
"PathToPloto": "C:/Users/Scott/.chia/mainnet/Ploto.psm1"
"DiskConfig": [
{
"TempDriveDenom": "temp",
"Temp2Denom": "temp",
"OutDriveDenom": "plot",
"EnableT2": "false",
"DenomForOutDrivesToReplotForPools": "replot"
}
],
"JobConfig": [
{
"KSizeToPlot": "32",
"ReplotForPool": "false",
"IntervallToCheckInMinutes": "5",
"InputAmountToSpawn": "50",
"WaitTimeBetweenPlotOnSeparateDisks": "15",
"WaitTimeBetweenPlotOnSameDisk": "5",
"MaxParallelJobsOnAllDisks": "2",
"MaxParallelJobsOnSameDisk": "2",
"MaxParallelJobsInPhase1OnAllDisks" : "2",
"StartEarly": "true",
"StartEarlyPhase": "4",
"BufferSize": "3390",
"Thread": "12",
"Buckets": "128",
"Bitfield": "true",
"FarmerKey": "KEY",
"PoolKey": "KEY",
"P2SingletonAdress": ""
}
],
"SpawnerAlerts": [
{
"DiscordWebhookURL": "https://discord.com/api/webhooks/URL",
"WhenJobSpawned": "true",
"WhenNoOutDrivesAvailable": "true",
"WhenJobCouldNotBeSpawned": "true"
}
],
"PlotoFyAlerts": [
{
"DiscordWebhookURL": "https://discord.com/api/webhooks/URL",
"PeriodOfReportInHours": "1"
}
]
}

Yeah the install script isn't working either. I ran it in Powershell as administrator and I get this kind of error after every single property the script tries to save.
The property 'InputAmounttoSpawn' cannot be found on this object. Verify that the property exists and can be set.
At C:\users\scott.chia\mainnet\Install-Ploto.ps1:344 char:24

$config.JobConfig | % {$_.InputAmounttoSpawn  = $InputAmountToSpawn}

                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : PropertyNotFound

In addition, and this may be unrelated, the script says it saved it, but when I checked that folder there's no file there.
Copy-Item : Cannot find path 'C:\users\scott.chia\mainnet\PlotoSpawnerConfig.json' because it does not exist.
At C:\users\scott.chia\mainnet\Install-Ploto.ps1:424 char:17

...             Copy-Item -Path $PathToConfig -Destination $destination - ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : ObjectNotFound: (C:\users\scott...wnerConfig.json:String) [Copy-Item], ItemNotFoundExce
ption
FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.CopyItemCommand

Either use // or \ in Paths. JSON does not like solos /

OK so I fixed the JSON errors and now when I try to run it, I get errors. These kind of errors go on for a long long time and eventually it returns you back to the command prompt.
PS C:\users\scott.chia\mainnet> Import-Module "C:\Users\Scott.chia\mainnet\Ploto.psm1"
PS C:\users\scott.chia\mainnet> Start-PlotoSpawns
PlotoManager @ 6/13/2021 12:04:37 AM : Ploto Manager started.
PlotoManager @ 6/13/2021 12:04:37 AM : Checking paths...
PlotoManager @ 6/13/2021 12:04:37 AM : plotter folder available.
PlotoManager @ 6/13/2021 12:04:37 AM : Loading config from  C:\Users\Scott.chia\mainnet\config\PlotoSpawnerConfig.json...
PlotoManager @ 6/13/2021 12:04:37 AM : InputAmountToSpawn: 50
PlotoManager @ 6/13/2021 12:04:37 AM : Intervall to wait: 5
PlotoManager @ 6/13/2021 12:04:37 AM : PlotoFy is set to startup. Checking for active PlotoFy jobs...
PlotoManager @ 6/13/2021 12:04:37 AM : We have found active PlotoFy jobs. Stopping it and starting fresh...
Id     Name            PSJobTypeName   State         HasMoreData     Location             Command

7      PlotoFy         BackgroundJob   Running       True            localhost            ...
PlotoManager @ 6/13/2021 12:04:37 AM : Started PlotoFy successfully. Check your Discord
You cannot call a method on a null-valued expression.
At C:\users\scott.chia\mainnet\Ploto.psm1:1525 char:9

    $PlotterUsed = ($loggerPlotter.ToString()).TrimStart("Plotter ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : InvokeMethodOnNull

You cannot call a method on a null-valued expression.
At C:\users\scott.chia\mainnet\Ploto.psm1:1646 char:29

...             $IsPoolablePlot = ($loggerRead -match "IsPoolablePlot").l ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : InvokeMethodOnNull

You cannot call a method on a null-valued expression.
At C:\users\scott.chia\mainnet\Ploto.psm1:1647 char:29

...             $P2Adress = ($loggerRead -match "P2SingletonAdress").line ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : InvokeMethodOnNull

Can you try flushing your existing log in folder „plotter“?

IT WORKS....WHOOOOOHOOOOOO. I will let you know how my testing goes. It's running the first set of jobs now.

It looks like when the script is stopped by using control-C and you attempt to restart it you get the following error. Clearing out the log files directory and starting the script again causes it to work just fine from there.
lotoManager @ 13/06/2021 23:33:24 : Amount of spawned Plots in this iteration: 1
PlotoManager @ 13/06/2021 23:33:24 : Overall spawned Plots since start of script: 1
__
PlotoManager @ 14/06/2021 00:13:30 : Detected aborted jobs. Removing them...
PlotoRemoveAbortedJobs @ 14/06/2021 00:13:30 : Found aborted Jobs to be deleted: 821d19cf-1e86-49a7-9b47-6a4ea90fb038
PlotoRemoveAbortedJobs @ 14/06/2021 00:13:30 : Cleaning up...
PlotoStopJob @ 14/06/2021 00:13:30 : Found the job to be aborted with JobId:  821d19cf-1e86-49a7-9b47-6a4ea90fb038
Get-ChildItem : Cannot find path 'C:\Users\scott\2' because it does not exist.
At C:\users\scott.chia\mainnet\Ploto.psm1:1894 char:29

    $T2FileArrayToDel = Get-ChildItem $job.T2Drive | Where-Object ...

                        ~~~~~~

CategoryInfo          : ObjectNotFound: (C:\Users\scott\2:String) [Get-ChildItem], ItemNotFoundException
FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.GetChildItemCommand

One thing I wanted to mention is this section of the config I'm using in case it's relevant.

Yea it has troubles getting the temp files on -2 (Stotik uses the defined temp drive as T2 if non is configured.) Need to find a way to get past this when clearing aborted jobs.

New build (updated config too) doesn't show errors on open; however, it doesn't loop to wait for when the current jobs finish. Just takes me back to prompt. I started it when 2 jobs were running.
PS C:\Users\Scott.chia\mainnet> Start-PlotoSpawns -verbose
PlotoManager @ 6/14/2021 10:00:31 AM : Ploto Manager started.
PlotoManager @ 6/14/2021 10:00:31 AM : Checking paths...
PlotoManager @ 6/14/2021 10:00:31 AM : plotter folder available.
PlotoManager @ 6/14/2021 10:00:31 AM : Loading config from  C:\Users\Scott.chia\mainnet\config\PlotoSpawnerConfig.json...
VERBOSE: Loaded config successfully.
PlotoManager @ 6/14/2021 10:00:31 AM : InputAmountToSpawn: 0
PlotoManager @ 6/14/2021 10:00:31 AM : Intervall to wait: 0
PlotoManager @ 6/14/2021 10:00:31 AM : PlotoFy is set to startup. Checking for active PlotoFy jobs...
PlotoManager @ 6/14/2021 10:00:31 AM : We have found active PlotoFy jobs. Stopping it and starting fresh...
Id     Name            PSJobTypeName   State         HasMoreData     Location             Command

645    PlotoFy         BackgroundJob   Running       True            localhost            ...
PlotoManager @ 6/14/2021 10:00:31 AM : Started PlotoFy successfully. Check your Discord
VERBOSE: PlotoSpawner @ 06/14/2021 10:00:32: Invoking PlotoJobs started.
VERBOSE: PlotoSpawner @ 06/14/2021 10:00:34: No Jobs spawned as either no TempDrives available or max parallel jobs
reached. Max Parallel Jobs: 0Current amount of Jobs: 2
We are done!
PS C:\Users\Scott.chia\mainnet>

Upgraded to <IP_ADDRESS>.<IP_ADDRESS>.9 and re-ran the install script. Everything went fine (but still had to clear out the log files to get it to start without errors) except it only runs through one job on each of my 3 temp drives then it dies. Here's the error I see at the end.

I've fired off v<IP_ADDRESS>.<IP_ADDRESS>.9 on two Plotters yesterday evening. It is still going as of right now. Can you try flushing logs once again?

Upgraded to <IP_ADDRESS>.<IP_ADDRESS>.9 and re-ran the install script. Everything went fine (but still had to clear out the log files to get it to start without errors) except it only runs through one job on each of my 3 temp drives then it dies. Here's the error I see at the end.

I've fired off v<IP_ADDRESS>.<IP_ADDRESS>.9 on two Plotters yesterday evening. It is still going as of right now. Completed Jobs are not considered as aborted anymore. Can you try flushing logs once again?


Avoid inference guessing in trait solver (e.g., favoring where-clauses over impls)

The current trait solver performs "inference guessing" in various places, leading to incompleteness (failure to find solutions that exist). We avoid unsoundness by eliminating incompletness in coherence. But it may be possible to refactor the trait solver to instead return "guidance"; it would be up to HIR type check or other phases to decide if/when to apply this guidance. This would lead to more programs being compiled but also a cleaner overall setup. Experimentation is required to see if this truly works.
cc https://github.com/rust-lang/rust/issues/41756

update: not actively worked on, requires the solver to be more settled, currently not a main focus.


feature: bring custom type and id column name to polymorphism

[x] Do only one thing
[x] Non breaking API changes
[x] Tested

What did this pull request do?
This pull request enables users to customize the type and id column names for polymorphic associations in GORM.
User Case Description
Before this Pull Request
Previously, polymorphic associations were limited to default naming conventions (<type>_Type and <type>_ID).
After this Pull Request
Users can now specify custom type and id column names for polymorphic associations, offering greater adaptability to diverse database schemas.

close #6717

Could you please check these test cases for errors?

✅  I forgot updating RunMigrations function, now it's ok :)

Thank you for your pull request. Additionally, please ensure to update the documentation on gorm.io, which can be found at github.com/go-gorm/gorm.io ;)


Windows 11 Version 22H2 UUP Updates Fail with ConfigMgr Update Source

When using ConfigMgr as the update catalog source, the new UUP CUs from Microsoft do not get parsed properly and fail to download.

Based on the current code, WIM Witch uses the SourceURL attribute returned from the WMI lookup to the update catalog. The problem is, the new UUP updates have multiple source URLs and WIM Witch is not built to handle multiple source URLs being returned from the update catalog.

Additionally, even if we were to rewrite the script to look for the update CAB file, Microsoft has stated that this CAB will not work to service a WIM and we should instead retrieve the full MSU directly from the Microsoft Update Catalog.

David Segura has also removed these updates from OSDSUS due to the same issue.
It seems like a rewrite is needed to look up the update and download it directly from Microsoft without relying on data from ConfigMgr itself.

AFAIK . the CM is used for WHAT to download but it is downloaded from MS directly and not from CM.. SO - IT might still work.. I will definitly look into this.. SOunds like and worthy challenge =)

Right, it gets downloaded from Microsoft but it's grabbing the download URLs from the ConfigMgr update catalog metadata.
Per the updated documentation on the UUP updates, the ~122 MB CAB file that is part of the UUP update package (http://dl.delivery.mp.microsoft.com/filestreamingservice/files/f47a805a-18d1-48b2-9b7d-24892d58301e/public/windows11.0-kb5025239-x64_160a4310610c3eadec55ff9a7dd6ec641e411523.cab) will fail to install and we should instead be using the ~298 MB MSU directly from the Microsoft Update Catalog: https://catalog.sf.dl.delivery.mp.microsoft.com/filestreamingservice/files/9a4464d3-f890-4b6c-a3a9-e15ddbe7c9ec/public/windows11.0-kb5025239-x64_e9f2f715cfce56adc6b875512f0088aa4b3b10eb.msu.

AH IC ! - SO the DL URL from CM is not the valid one. Dang... Hmm, where can we get the proper one - Needs investigating... (Sigh.. Microsoft...=) )

Haha yea, that's our lovely challenge! I wonder if the basis of this module still works? https://gist.github.com/potatoqualitee/b5ed9d584c79f4b662ec38bd63e70a2d
Though this does make me wonder if this broke in-console servicing for Windows 11 Version 22H2 since I'm pretty sure that uses DISM under the hood.

Haha yea, that's our lovely challenge! I wonder if the basis of this module still works? https://gist.github.com/potatoqualitee/b5ed9d584c79f4b662ec38bd63e70a2d
Though this does make me wonder if this broke in-console servicing for Windows 11 Version 22H2 since I'm pretty sure that uses DISM under the hood.

In-console servicing of Windows 11 still appears to be broken in SCCM 2203 with the latest hotfix.


Transform toggle animation

Aim

to add transform toggle animation

Details

It will have 4 cards.
When you click on one card the other card becomes light and small in size and many more effects
The card which is clicked also has some effects.

Please assign me this under hssoc


YAML linter configuration

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
NONE
What this PR does / why we need it:
This PR simply adds a basic yamllint configuration in order to also start the process of establish (and document) the coding style of the repo's YAML files (rules, falco config etc.).
The YAML lint config created extends the default one disabling some linting rules in order to minimize the mismatch from the current YAML style the repo contains.
Please notice this PR does NOT reformat any YAML file.
Which issue(s) this PR fixes:
Refs #619
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: leodido
If they are not already assigned, you can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: caeba50f04d453bafdbda9cd7ae021b0d77ed568

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash:<PHONE_NUMBER>94ca57183d5d05ac521d746e61c8ee


Heartbeat retry main

there's another PR for this
https://github.com/ebi-uniprot/uniprot-rest-api/pull/452


SVGTOPDF

Converting my svg to pdf dynamically results in a d3 image without colors in my pdf. Please help me on the same...I would be happy if i could get a source code working on the same.
Cheers,
Sreejith Girish,
<EMAIL_ADDRESS>Ph:+91-9632615759

SVG is now in master :) Documentation to follow!


feat: allow multiple keys to be passed to pluck

This would be really helpful for me.
Curious what you think! Can write tests + update docs if you think it's a good idea.

It makes sense. A nice and backwards compatible addition. I will appreciate a full PR.

Thinking a bit more your implementation and func interface itself is not backwards compatible. Passing a tuple as a key already has meaning:
pluck(('a', 'b'), [{('a', 'b'): 42}]) == 42

So maybe pluck(*keys, mappings) instead.

Only hashable objects can be used as keys, so passing pluck a list of keys wouldn't cause the same ambiguity that passing a tuple of keys does.
And it makes sense to specify multiple keys as a collection, especially a list or set.
Tuples aren't viable due to being valid keys, but they would be fairly conceptually iffy here anyway. A list is a sequential collection containing separate objects, but a tuple is a single object that just happens to have a sequential internal structure. Passing a tuple of keys doesn't really make sense.

The implementation here is converting list to tuple anyway, so not adding anything new.
@SirPavlova you probably need some other behavior


Not displaying keyboard short cuts in menus

Menu items are not showing keyboard shortcuts to the right even if it is set through code, is this intentional

OK...

@sonyraj @ButchersBoy i've tested the main demo and it seems that there is no issue... ?

@punker76, @sonyraj talks about InputGestureText property of menuitem, which is not working in materialdesign menu.

The only workaround i've found is to format the menu header a fixed length string and then appending the input gesture text to the end of it. But it still have issues with alignment

I have fixed this issue, pr will come after fee hours

:+1:

Great? Do i have to change any thing to make it display shortcuts

Just sit tight and wait for the fix :wink:

:-)

Its working Yeah.......... Thanks man @l1pton17


Travis unit tests are frequent source of problems

We can discuss here how to solve the problem of providing unit test results for all commits that are creazed.
Current problems are mostly related to

communication delay between us-based travis server and europe-based Redpitaya of the order of 100ms
issues due to broken connection during testing, related to the above.

The best solution would be to run the unit tests on a europe-based machine with direct access to a Redpitaya.
Another option would be to locally run the tests before committing code changes and prove that they were successful, e.g. with a hash of the commit or sth alike. This would require a proper way to locally create an environment similar to the one of travis.
Another good option would be to find a europe-based ci Service Provider, or use our own AWS server.
Ano

But Leo, at the moment is there a redpitaya physically accessible for Travis? Some time ago it was at my place, but since I moved, I didn't reinstall the necessary hardware... Moreover, this was rather cumbersome since my internet provider doesn't provide a fixed IP address.


Have a service variable for each container's mDNS

Each service should have a configurable mDNS hostname. For example, if the fleet variable MDNS = 1 and GRAFANA_MDNS_NAME = grafana then the grafana container should broadcast on grafana.local.

Relates to #11

ok

I'm working on this here too https://github.com/MatthewCroughan/octobalena/issues/3
So I'll implement it there first and then suggest changes here.


Remove heading variable and assignments

please put this PR on hold. still investigating on this. 😬

ok. this causes build error.
basically because there is no getFloat for ReadableArray
and the heading is not being used for view.animateToRegion
so i removed the lines.

Please take a look at https://github.com/parkling/react-native-maps/pull/3


New Component HtmlTextEditor

Feature request type
New component
Component name
MudTextEditor
Is your feature request related to a problem?
No response
Describe the solution you'd like
I would like you to add a new component For Html Text Editor
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Don't count on it. See #4361

Closing as it's out of scope.
See comments: https://github.com/MudBlazor/MudBlazor/issues/5048#issuecomment-1214291076, https://github.com/MudBlazor/MudBlazor/issues/4361#issuecomment-1090607523


FSx for NetApp ONTAP for Linux

Currently SaaS Boost only supports Amazon FSx for NetApp ONTAP (FSxN) file systems for Windows based services. One of ONTAP's many strengths is its support for both the SMB protocol as well as the NTFS protocol allowing a heterogeneous set of clients to communicate with the file system using their native protocol. This feature would enable selecting FSx as a shared file system type for Linux services as an alternative choice to Amazon EFS. Note that FSxN is still only compatible with the EC2 launch type, so this choice will not be available for services using the Fargate launch type.

[x] #400

Resolved with merge of #400


AvaTaR agent: 'Action' object has no attribute 'strip'

(dspy-ai 2.5.0)
Hi dspy-team,
I am getting the following error after upgrading dspy-ai to 2.5.0 when using the AvatarOptimizer.
Starting the task...
Processing examples:  14%|█▍        | 1/7 [00:01<00:06,  1.09s/it]
'Action' object has no attribute 'strip'
from dspy.teleprompt import AvatarOptimizer

avatar = Avatar(
    tools=tools,
    signature=LogRootifyReasoner,
    verbose=True,
    max_iters=6,
)

optimizer = AvatarOptimizer(
    metric=metric,
    max_iters=2,
    lower_bound=0.6,
    upper_bound=0.7,
    max_negative_inputs=10,
    max_positive_inputs=10,
)

optimized_avatar = optimizer.compile(
    student=avatar,
    trainset=trainset
)

Iteration 1/2
Starting the task...
Starting the task...
Starting the task...
Starting the task...
Starting the task...
Starting the task...
Starting the task...

The error happens when the DSPy framework processes the action_1 output field in signatures.py.
The framework expects the output to be a string (possibly a JSON string), but it receives an Action object instead.
Action doesn't have a strip() method.
dspy/predict$ less avatar/signatures.py:
import dspy
from dspy.predict.avatar.models import Action

class Actor(dspy.Signature):
    """You will be given `Tools` which will be a list of tools to use to accomplish the `Goal`. Given the user query, your task is to decide which tool to use and what input values to provide.

You will output action needed to accomplish the `Goal`. `Action` should have a tool to use and the input query to pass to the tool.

Note: You can opt to use no tools and provide the final answer directly. You can also one tool multiple times with different input queries if applicable."""

    goal: str = dspy.InputField(
        prefix="Goal:",
        desc="Task to be accomplished.",
    )
    tools: list[str] = dspy.InputField(
        prefix="Tools:",
        desc="list of tools to use",
    )
    action_1: Action = dspy.OutputField(
        prefix="Action 1:",
        desc="1st action to take.",
    )

dspy/predict/avatar/models.py:
class Action(BaseModel):
    tool_name: Any = Field(..., description="Name of the tool to use.")
    tool_input_query: Any = Field(..., description="Query to pass as input to the tool.")

@krypticmouse is investigating.

Looking into this!

from pydantic import BaseModel
class Complaints(BaseModel):
patient_complaints: list[ComplaintItem] | None
class ComplaintExtractorSignature(dspy.Signature):
"""
--------
"""
input_text: str = dspy.InputField(desc="============")
output_lang: str = dspy.InputField(
    desc="Output language for the refined complaints and complaints writeup"
)
complaints: Complaints = dspy.OutputField()

  File "/home/danaro/repo/-----/app/env/lib/python3.12/site-packages/dspy/functional/functional.py", line 254, in <lambda>
parser=lambda x, from_json=from_json: from_json(_unwrap_json(x, from_json)),
                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/home/danaro/repo/---/app/env/lib/python3.12/site-packages/dspy/functional/functional.py", line 409, in _unwrap_json
output = output.strip()
^^^^^^^^^^^^
File "/home/danaro/repo/-------/app/env/lib/python3.12/site-packages/pydantic/main.py", line 828, in getattr
raise AttributeError(f'{type(self).name!r} object has no attribute {item!r}')
AttributeError: 'Complaints' object has no attribute 'strip'


Issue 8357 still actual, please, reopen

Before you start please confirm the following.

[X] Yes, I've searched similar issues on GitHub.
[X] Yes, I've checked whether this issue is covered in the Portainer documentation or knowledge base.

Problem Description
Issue #8357 is actual, but closed. Please reopen it.
Expected Behavior
Issue #8357 is actual, but closed. Please reopen it.
Actual Behavior
Issue #8357 is actual, but closed. Please reopen it.
Steps to Reproduce
Issue #8357 is actual, but closed. Please reopen it.
Portainer logs or screenshots
No response
Portainer version
2.21.2
Portainer Edition
Business Edition (BE/EE) with 5NF / 3NF license
Platform and Version
Issue #8357 is actual, but closed. Please reopen it.
OS and Architecture
Issue #8357 is actual, but closed. Please reopen it.
Browser
No response
What command did you use to deploy Portainer?
No response
Additional Information
No response

Re-opened.