Add positions to events

When a function returned value (which is looked up via events) doesn't match it error. This error doesn't contain the location of the return.
Many of the events could benefit from having a position (using Span) to mark where they occurred.
This should be quite straightforward, add Span to each of the variants then just follow the errors Rust produces

Hey @NitantP, thanks for the interest. Hopefully the following answers your questions

So currently the position field in ReturnedTypeDoesNotMatch just points to the return type annotation. What would be ideal is for that it to also contain the positions of the offending return statements. Getting these positions for these errors would be really simple if they were held on return events. This sort of thing is already implemented for AssignmentError::DoesNotMeetConstraint
https://github.com/kaleidawave/ezno/blob/7cf648d5f83d08e9782ac8eb721b181d8aa2427b/checker/src/context/mod.rs#L1390-L1394 so when it emits an error, it also adds a label (additional information) that outlines where the variable type annotation is
https://github.com/kaleidawave/ezno/blob/7cf648d5f83d08e9782ac8eb721b181d8aa2427b/checker/src/diagnostics.rs#L377-L392

and 3. Can't think of any more metadata that is needed on Event rn, so just a Span on each should be fine. I see that SetsVariable and Throw are currently the only two unnamed variants, feel free to make the named variants like the others!

I am hoping it shouldn't be too difficult to start on, I am happy to help with any problems that arise if you decide to give it go! Note: you do not have to fully connect the errors up, I am thinking it might need another change for it to work perfectly. But if all you could get events setup with a position for each then it would be really useful for now and future ideas!

Just a heads up @NitantP. #61 changes the structure of positions a lot so might be worth waiting for that to merge before starting

Understood - thanks, @kaleidawave!

Hi @NitantP #61 has been merged. So the repo code should be in a good state to work on this. Some updates:

The positions should be source_map::SpanWithSource (aka have both a position and a file). This is the same as diagnostics/errors so there should be some good examples of creating them
Since writing the issue I have reworked the return checking. (aka a function annotated as function x(): number, check that all returns are subtypes of number). Once events have positions, if you want to add the diagnostics you can check out https://github.com/kaleidawave/ezno/blob/88fc8e422fe216b936ca6b223d00492b2bd26f6d/checker/src/events/helpers.rs#L17
Specifically there is a place which would be ideal for using a position here
https://github.com/kaleidawave/ezno/blob/88fc8e422fe216b936ca6b223d00492b2bd26f6d/checker/src/events/helpers.rs#L58

LMK if there are any more questions :)

Hello, @kaleidawave. I'm trying to solve this issue on my end just to practice my Rust skills (no PR since I presume NitantP is still working on this issue.)
I understand how to modify the diagnostic messages and alter the ReturnedTypeDoesNotMatch to have the position for the return location.
But any thoughts on how to get the position of the location the value is returned?
I can see that get_return_from_events iterates through the events, but the pertinent event, Event::Return only stores a &TypeId and no position for when the event was triggered.
https://github.com/kaleidawave/ezno/blob/88fc8e422fe216b936ca6b223d00492b2bd26f6d/checker/src/events/helpers.rs#L23-L24

Hi @wzwywx - please feel free to submit a PR if you'd like. I haven't been able to make the progress I intended to given circumstances, so you may be in a better position to see this through.

@wzwywx awesome! Thanks for attempting it, if you do make any progress (working or not) would be great to put in a PR anyway as I am happy to help finish it if you make a start.
Might be worth reading the title 😅 but yes you have run into the exact problem!
What I would start with is adding a field to the return event variant, (events/mod.ts:68)
Return {
  returned: TypeId,
  // \/ field you should add \/
  position: SpanWithSource
}

then you will see rust error a bunch and you just have to follow those until it compiles again (so you will probably have to do things like modify return_value (context/environment.rs:713) to take a position via a parameter and modify the synthesis code to give that position when it calls that).
Then once you have that field you can go back and get the diagnostic working!
And if you find that okay you can add a position field to every variant as I need it for other diagnostics 😀
Also no worries @NitantP

Yeah, haha I’ve actually read the title, but I underestimated the effort needed to get the return position (which was foolish of me because I was unfamiliar with the API).
I tried to trace how the current position gets its value, and just recently found out it involves more functions than I expected. Still working on that; your tip about the synthesis code might be the thing I’m missing.
Anyhow, thank you for your guidance. I’ll give it another go and see how it goes this time.
21 Oct 2023 at 14:38 by @.***:

@wzwywx https://github.com/wzwywx>  awesome! Thanks for attempting it, if you do make any progress (working or not) would be great to put in a PR anyway as I am happy to help finish it if you make a start.
Might be worth reading the title 😅 but yes you have run into the exact problem!
What I would start with is adding a field to the return event variant, (events/mod.ts:68)
Return>  > {>    > returned> :>  > TypeId> ,>   > // / field you should add />   > position> :>  > SpanWithSource> }
then you will see rust error a bunch and you just have to follow those until it compiles again (so you will probably have to do things like modify > return_value>  (context/environment.rs:713) to take a position via a parameter and modify the synthesis code to give that position when it calls that).
Then once you have that field you can go back and get the diagnostic working!
And if you find that okay you can add a position field to > every variant>  as I need it for other diagnostics 😀
Also no worries > @NitantP https://github.com/NitantP
—
Reply to this email directly, > view it on GitHub https://github.com/kaleidawave/ezno/issues/37#issuecomment-1773694088> , or > unsubscribe https://github.com/notifications/unsubscribe-auth/A2ZOS3W4NRA7PQF5L5TAZHDYANUVZAVCNFSM6AAAAAAZMML5VWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZTGY4TIMBYHA> .
You are receiving this because you were mentioned.> Message ID: > <kaleidawave/ezno/issues/37/1773694088> @> github> .> com>

Hi @wzwywx - please feel free to submit a PR if you'd like. I haven't been able to make the progress I intended to given circumstances, so you may be in a better position to see this through.

Ah, I see. No worries; hopefully your circumstances are improving on your end.


whereBelongsTo does not work with morphTo relationship unless ownerKey is specified explicitely

Laravel Version
10.13.5
PHP Version
8.1.11
Database Driver & Version
MySQL 8.0.31
Description
Hello.
I have a Comment model and a couple other models (Task, Document, etc) that Comments can belong to using a polymorphic relationship. When I am trying to do Comment::whereBelongsTo($model, 'commentable') query method I get InvalidArgumentException with the following message: Nested arrays may not be passed to whereIn method. .
After some debugging I've found out that I need to excplicitely specify ownerKey in the morphTo() relationship of the Comment model, otherwise it's never resolved and stays null.
Steps To Reproduce

I have a Comment model with the following relationship:

    public function commentable(): MorphTo
    {
        return $this->morphTo();
    }

Next I have a "commentable" model (Task, Document, etc),
Then I have the following code in one of my controllers

    public function show(Task $task)
    {
        // ...

        // This is a simplified example to demonstrate the problem, I know there are more elegant ways to do this ;)

        $comments = Comment::whereBelongsTo($task, 'commentable')->get();

        // ...
    }

An InvalidArgumentException exception is thrown with the following message: Nested arrays may not be passed to whereIn method.

I have spent some time debugging this and it led me to this piece of code in the \Illuminate\Database\Eloquent\Concerns\QueriesRelationships trait's whereBelongsTo method
        $this->whereIn(
            $relationship->getQualifiedForeignKeyName(),
            $relatedCollection->pluck($relationship->getOwnerKeyName())->toArray(),
            $boolean,
        );

the $relationship->getOwnerKeyName() returns null at this point, unless I explicitely provide ownerKey in the commentable relationship of the Comment model like this:
    public function commentable(): MorphTo
    {
        return $this->morphTo(ownerKey: 'id');
    }

While this solution works, I believe this should not be necessary. Additionally, different commentable models may have different key names. So alternative solution would be to fetch ownerKey name from the related model like this:
        $this->whereIn(
            $relationship->getQualifiedForeignKeyName(),
            $relatedCollection->pluck($relationship->getOwnerKeyName() ?? $related->getKeyName())->toArray(),
            $boolean,
        );

but I don't know the code good enough to know if this is a correct solution.
Thank you in advance and sorry if something is missing it is my first time submitting a bug report.

Ok, I have just realized that I cannot use whereBelongsTo with a MorphTo relationship because it never checks the commentable_type, only commentable_id. I am not sure if there's a bug with regular BelongsTo relationship in this case, so maybe this is irrelevant.

Thank you for investigating this but this indeeds seems unsupported.


enhance: Increase the testcoverage of sync() method in ByteChannelReader

Increase the code coverage of the tests that tests the sync() method in the class ByteChannelReader (in bt-core/data package).

I think with this method, I could find two tests that can be written to take two unexplored branches in the method.

This issue was closed through the merging of #9


Adding youtube video intro

Added Introduction video on the README.md file to help improve on the instructions.

Hi @MariaAngella! Thanks for trying to work on this. So the video you linked is to the ODK tools, not the ODK-X tools. I don't think we need a video embedded in the read me, but could point folks to the youtube as well as the docs if they want to learn more with just a link.

Thank you for the clarity 😊


added support for esbuild loader option

Adds support for a new esbuild.loader option.
Currently, loader cannot be configured and is guessed by the file extension. One possible use case is allowing users to use jsx loader for .js files.

I'm not sure which one's more desired:
a. Passing the result of fs.readFile to the loader functions code argument or
b. Passing the result of plugin.transform which we already have by the time loader is returned.
In this version I decided to go with the second option.

Hey @sxzz, any updates/instructions on this?
If you are undecided on my previous comment, we could just merge the first version of  loader?: Loader (constant value, no function) and wait for the feedback. That way a chunk of the use case can be addressed instantly. And if a function is ever implemented it won't be a breaking change


[Auto-generated] Update dependencies

Update dependencies
Automatically created PR from ci/dependabot-updates.
The PR is based on the 'CI - Update dependencies PR' workflow in SINTEF/ci-cd.
To do

[x] Check that the diff is sensible, and that tests and builds pass with the new dependency versions.
[x] Make sure that the PR is squash merged, with a sensible commit message.

Wait for pydantic/pydantic#5148 to be fixed and a new release of pydantic, which should fix the mypy v1.1.1 pydantic compatibility bug concerning model signature recognition.

Codecov Report
Patch coverage has no change and project coverage change: -0.09 :warning:

Comparison is base (5ef872d) 81.89% compared to head (41ebed9) 81.81%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #242      +/-   ##
==========================================
- Coverage   81.89%   81.81%   -0.09%
==========================================
  Files          41       41
  Lines        1127     1127
==========================================
- Hits          923      922       -1
- Misses        204      205       +1

Flag
Coverage Δ

linux
81.81% <ø> (ø)

linux-extra_libs
81.81% <ø> (ø)

windows
?

windows-extra_libs
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


How to set the orig when it in mergeView

after I set orig when codemirror init, I do not know, how to set it again

mergeView.rightOriginal().setValue("abc") should do it, I think.


TIPS FOR FAST IMPROVEMENT

 Issue by aleksandar-devedzic
Tue May  9 09:34:18 2023
Originally opened as https://github.com/codelucas/newspaper/issues/969

I have extracted some meta tags, you can try to identify title, text, description and date by replacing provided tags in :
meta[property='{}']
meta[name='{}']
meta[itemprop='{}']
Meta tags for publication and modification date:
published_date
published_time
cXenseParse:publishtime
pubdate
publish_date
PublishDate
dcterms.created
rnews:datePublished
article:published_time
prism.publicationDate
displaydate
OriginalPublicationDate
og:published_time
datePublished
article_date_original
article.published
published_time_telegram
sailthru.date
datePublished
date
Date
original-publish-date
DC.date.issued
dc.date
DC.Date
parsely-pub-date
publishtime
publication_date
uploadDate
coverageEndTime
publishdate
publish-date
publishedAtDate
dcterms.date
publishedDate
creationDateTime
pub_date
updated_time
og:updated_time
datemodified
last-modified
Last-Modified
DC.date.modified
article:modified_time
modified_time
modifiedDateTime
dc.dcterms.modified
lastmod
Meta tags for title:
dc.title
og:title
headline
articletitle
article-title
parsely-title
title
Meta tags for description:
description
og:description
Meta tags for body:
articleBody
articleText
FYI
It would be good if you can fix/improve/adapt the code so that it can extract full information from these websites since these websites are the most popular websites in the world.
By "full information" i mean title, publication date and article body
CNN - https://edition.cnn.com/
BBC News - https://www.bbc.com/news
Reuters - https://www.reuters.com/
The New York Times - https://www.nytimes.com/
The Guardian - https://www.theguardian.com/international
Al Jazeera - https://www.aljazeera.com/
Associated Press (AP) News - https://apnews.com/
NBC News - https://www.nbcnews.com/
Fox News - https://www.foxnews.com/
USA Today - https://www.usatoday.com/
ABC News - https://abcnews.go.com/
CBS News - https://www.cbsnews.com/
The Washington Post - https://www.washingtonpost.com/
Time - https://time.com/
Forbes - https://www.forbes.com/
Bloomberg - https://www.bloomberg.com/
The Wall Street Journal - https://www.wsj.com/
The Huffington Post - https://www.huffpost.com/
The Independent - https://www.independent.co.uk/
The Sydney Morning Herald - https://www.smh.com.au/
The Economist - https://www.economist.com/
The Times of India - https://timesofindia.indiatimes.com/
The Daily Mail - https://www.dailymail.co.uk/home/index.html
The Telegraph - https://www.telegraph.co.uk/
The Sun - https://www.thesun.co.uk/
The Mirror - https://www.mirror.co.uk/
The Daily Beast - https://www.thedailybeast.com/
The Atlantic - https://www.theatlantic.com/
National Geographic - https://www.nationalgeographic.com/
Science Daily - https://www.sciencedaily.com/
The Verge - https://www.theverge.com/
Wired - https://www.wired.com/
TechCrunch - https://techcrunch.com/
Engadget - https://www.engadget.com/
Mashable - https://mashable.com/
Forbes India - https://www.forbesindia.com/
Hindustan Times - https://www.hindustantimes.com/
CNN Business - https://www.cnn.com/business
Financial Times - https://www.ft.com/
CNBC - https://www.cnbc.com/
Business Insider - https://www.businessinsider.com/
Politico - https://www.politico.eu/
The Hill - https://thehill.com/
The Washington Times - https://www.washingtontimes.com/
The Boston Globe - https://www.bostonglobe.com/
The LA Times - https://www.latimes.com/
The Chicago Tribune - https://www.chicagotribune.com/
The Sydney Morning Herald - https://www.smh.com.au/
The Globe and Mail - https://www.theglobeandmail.com/
The Toronto Star - https://www.thestar.com/

added 0.9.2


Increase performance of Miss Forest Imputation

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

Miss forest imputation is a nice imputation algorithm with good results. But it comes at the cost of speed when imputing over multiple columns. This can be annoying especially on larger datasets.
Describe the solution you would like

Depending on the size of the dataset and the number of features involved, iterative fitting may take a long time. Try using all your processors for the job by setting n_jobs=-1 for both RandomForestRegressor and RandomForestClassifier and you may try to patch Scikit-Learn with Intel(R) Extension for Scikit-learn as described here: pypi.org/project/scikit-learn-intelex

See https://stackoverflow.com/questions/64900801/implementing-knn-imputation-on-categorical-variables-in-an-sklearn-pipeline if this could be a possible solution and helps with speed.

Out settings object has a number to control the number of CPUs and therefore jobs here. This implementation has to make use of it if it doesn't already.

Multi-output and sparse data is not supported by the intel extension. Is this an issue?


WEBDAV uploads

WEBDAV uploads do not work properly. I am using Mac OS X Finder as the client and the WebDavServer example running on iOS simulator with no changes.
1 - After a small file upload, the uploaded file appears listed but it results in an empty file.
2 - For larger files the WEBDAV server crashes at
NSAssert(requestContentStream == nil, @"requestContentStream should be nil");
Are Uploads properly supported ?
Is there anything I need to know to make it work?
Thanks.
Joan Lluch

Hi Joan,
I ran into exactly the same problem and after some debugging, I think I have a solution for this problem. If you look at lines 2215 onwards from HTTPConnection.m, you should find this snippet.
	if (tag == HTTP_REQUEST_CHUNK_SIZE)
	{
		// We have just read in a line with the size of the chunk data, in hex,
		// possibly followed by a semicolon and extra parameters that can be ignored,
		// and ending with CRLF.

		NSString *sizeLine = [[[NSString alloc] initWithData:data encoding:NSUTF8StringEncoding] autorelease];

		requestChunkSize = (UInt64)strtoull([sizeLine UTF8String], NULL, 16);
		requestChunkSizeReceived = 0;

		if (errno != 0)
		{
			HTTPLogWarn(@"%@[%p]: Method expects chunk size, but received something else", THIS_FILE, self);

			[self handleInvalidRequest:nil];
			return;
		}

As far as I understand, the idea is to use strtoull function to detect whether or not a correct data is received, we expect to receive data of the form {data-length in hex}{CR}{LF}. In the event of error, strtoull function sets the global variable errno to a non-zero value. I think the problem that we experience is caused by not resetting the global-variable errno. I put a breakpoint just before the line where strtoull function is called. The value of errno at that point was 2 and even if strtoull call was successful, errno was not reset. So what I did was to insert a statement
		errno = 0; /* Reset errno */

just before this line
		requestChunkSize = (UInt64)strtoull([sizeLine UTF8String], NULL, 16);

After that, I have no problems in using either my iPhone or simulator as the WebDAV server and Finder as the client.
Hope this helps.
CJ

"ctjhai"'s solution works on iPhone for uploading smaller files like upto 300kb...but for larger files it still does not upload then, though it does not crash now.

Ctjhai,
WOW!, Your solution worked for me!. I tested it on the iPhone Simulator running as server and the Finder as client, both on the same computer. I tried with a 5MB file and it still worked. I have not yet tested it across different computers or on a real iPhone. It seems a too easy fix to be true though. I searched for more occurrences of the errno variable and found a couple of cases in GCDAsyncSocket where it is not reset before IO operations, but it does not seem to affect operation in such cases. Could you bring some more light why it should be reset to zero before string-to-number conversion functions but not for IO functions such as read or write ?
HaseebYousaf,
Would you describe the particular scenario where it did not work for you?.
Thanks all !


CLI structure

PyFa plot .....
PyFa describe ....
PyFa timeseries ....
PyFa convert ......

pyfa -h currently imports alot of packages and therefore takes a lot of time.
This seems avoidable.

The dependencies are in #25  imported after arguments are parsed, so the -h runs faster.


bug: Bodyweight used in calculation for all exercises when enabled

Benchpress should not include body weight in calculation when enabled, but that seems to be the case below.

This is because we attempt to store a boolean type in the database, but that is not supported by sqlite apparently:
https://github.com/tekartik/sqflite/blob/master/sqflite/doc/supported_types.md#supported-types
So we should change this to uint or similar to get this to work.


Custom Scalar as argument

Is it possible to use a custom scalar as an argument in a resolver? It appears like convertToType expects a ClassType, but GraphQLScalarType returns an object?
Thanks!

Ah! I needed to map the scalar in buildSchema's scalarsMap property, and specify the underlying class as the @Arg type, rather than the GraphQLScalarType

Yes, you can provide the custom scalar to the @Arg decorator or map it with the class if custom scalar convert from/to class (like Date or ObjectId).
class SampleResolver {
  @Query()
  sampleQuery(@Arg(type => CustomScalar) data: any): boolean {
    return true;
  }
}

@19majkel94 are you sure the former is true? In the convertToType method it tries to call new CustomScalar(), where CustomScalar isn't a function/class but instead an object provided by GraphQLScalarType.

For getting query/mutation/resolver arg, it calls getGraphQLInputType, which calls convertTypeIfScalar, which checks:
if (type instanceof GraphQLScalarType) {
  return type;
}

That's strange. I've just experienced this same issue again, check this out:
Custom Scalar:
import { GraphQLScalarType, Kind } from 'graphql';
import { PaginationDirection } from '../input/PaginationArgs';

export interface CursorID {
  id: number | string;
  direction: PaginationDirection;
}

export interface CursorPaged {
  skip: number;
}

export type Cursor = CursorID | CursorPaged;

export const CursorScalar = new GraphQLScalarType({
  name: 'Cursor',
  description: 'Pagination cursor',

  parseValue(value: string): Cursor {
    return JSON.parse(Buffer.from(value, 'base64').toString('ascii'));
  },

  serialize(value: Cursor) {
    return Buffer.from(JSON.stringify(value)).toString('base64');
  },

  parseLiteral(ast): Cursor | null {
    if (ast.kind === Kind.STRING) {
      return JSON.parse(Buffer.from(ast.value, 'base64').toString('ascii'));
    }
    return null;
  },
});

Argument in my resolver:
@Arg('cursor', type => CursorScalar, { nullable: true })
cursor: Cursor

Error:
TypeError: Target is not a constructor
    at Object.convertToType (/app/node_modules/type-graphql/helpers/types.js:52:26)
    at Object.<anonymous> (/app/node_modules/type-graphql/resolvers/helpers.js:22:69)
    at Generator.next (<anonymous>)
    at /app/node_modules/type-graphql/resolvers/helpers.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/app/node_modules/type-graphql/resolvers/helpers.js:3:12)
    at Promise.all.params.sort.map (/app/node_modules/type-graphql/resolvers/helpers.js:17:88)
    at Array.map (<anonymous>)
    at Object.<anonymous> (/app/node_modules/type-graphql/resolvers/helpers.js:17:69)
    at Generator.next (<anonymous>)
    at /app/node_modules/type-graphql/resolvers/helpers.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/app/node_modules/type-graphql/resolvers/helpers.js:3:12)
    at Object.getParams (/app/node_modules/type-graphql/resolvers/helpers.js:16:12)
    at /app/node_modules/type-graphql/resolvers/create.js:23:44
    at Generator.next (<anonymous>)

Am I doing something wrong @19majkel94 ?

There's a discrepancy between GraphQLScalarType used in my project and type-graphql.
  const schema = await TypeGraphQL.buildSchema({
    resolvers: [`${__dirname}/resolvers/**/*.ts`],
    scalarsMap: [{ type: Cursor, scalar: CursorScalar }],
  });

This results in a type warning: Property 'toJSON' is missing in type 'GraphQLScalarType'.
Which is strange, since my package.json defines ^0.13.2 (the same as type-graphql.)

Try npm dedupe to flatten the tree and use the same module.
Maybe I should hide graphql-js and just export GraphQLScalarType and other from type-graphql.

I had to upgrade my @types/graphql package, but I'm still experiencing the issue. The type instanceof GraphQLScalarType check you mentioned resolves as true.

Can you compile the app using tsc and see what TS reflection generates for this in your app:
@Arg('cursor', type => CursorScalar, { nullable: true })
cursor: Cursor

It should look something like this:
__decorate([
    src_1.Query(returns => [recipe_type_1.Recipe]),
    __metadata("design:type", Function),
    __metadata("design:paramtypes", []),
    __metadata("design:returntype", Promise)
], ExampleResolver.prototype, "recipes", null);
__decorate([
    src_1.Authorized() // only logged users can add new recipe
    ,
    src_1.Mutation(),
    __param(0, src_1.Arg("title")),
    __param(1, src_1.Arg("description", { nullable: true })),
    __metadata("design:type", Function),
    __metadata("design:paramtypes", [String, String]),
    __metadata("design:returntype", recipe_type_1.Recipe)
], ExampleResolver.prototype, "addRecipe", null);
__decorate([
    src_1.Authorized("ADMIN") // only admin can remove the published recipe
    ,
    src_1.Mutation(),
    __param(0, src_1.Arg("title")),
    __metadata("design:type", Function),
    __metadata("design:paramtypes", [String]),
    __metadata("design:returntype", Boolean)
], ExampleResolver.prototype, "deleteRecipe", null);
ExampleResolver = __decorate([
    src_1.Resolver()
], ExampleResolver);
exports.ExampleResolver = ExampleResolver;

I wounder to what the Cursor type is reflected and why it can't be newable. I could add this to exclusion list.

__decorate([
    type_graphql_1.FieldResolver(),
    __param(0, type_graphql_1.Root()),
    __param(1, type_graphql_1.Arg('cursor', type => Cursor_1.CursorScalar, { nullable: true })),
    __param(2, type_graphql_1.Arg('limit', { nullable: true })),
    __param(3, type_graphql_1.Arg('direction', type => PaginationArgs_1.PaginationDirection, { nullable: true })),
    __metadata("design:type", Function),
    __metadata("design:paramtypes", [University_1.University, Object, Number, String]),
    __metadata("design:returntype", Promise)
], UniversityResolver.prototype, "technologies", null);

I'm fairly new to Typescript so I'm not sure exactly what I'm looking at. Would it be the Object in design:paramtypes?

Object might be ignored but it works:
PS C:\WINDOWS\system32> node
> const o = new Object()
undefined
> o
{}
> o.test = "test"
'test'
> o
{ test: 'test' }
>

Unfortunately I don't know from the stacktrace on which argument it failed. Can you create a minimal reproducible example for that case? I would then be able to debug it and fix 😉

Here you go: https://github.com/ashleyw/type-graphql-issue-65-example :)

Ok, now I see that. You scalar returns custom object, which goes through the if that checks the type of data only, not the target type:
if (simpleTypes.includes(data.constructor)) {
  return data;
}

Fixed in 4176ed2 🎉
If you need transforming scalar object to class, use scalarsMap feature 😉


DNM: test ci on release-5.4

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/run-all-tests

/debug all

/test verify

/debug cdc-integration-kafka-test

/debug cdc-integration-kafka-test

/debug all

/debug cdc-integration-mysql-test

/debug dm-integration-test

/test all

/test ?


Implement gamemode switching between worlds (as part of the WorldBuilder)

(Hurrah to issue #1 on SpongeCommon!)
@bloodmc
When we pull over the world management impl to common, we need to implement switching gamemodes between worlds.
I think we considered making a permission in-which a player can override the gamemode switch. Something like
sponge.gamemode.override

?

If this is to have a creative world, and a survival world in the same server, wouldn't it make more sense to either remember the last used gametype per world, or going further and having the option to keep seperate userdata/inventory/falldamage/statistics per universe?
I've used multiverses gametype enforcing before and both allowing users to have an override permission allowing them to bring creative from creative to survival, or not having the override permission and suddenly dying as they teleport to their last known flying position in a survival world (due to being in creative in the survival world before going to the creative world) seem annoying.

We won't be keeping inventory/etc isolated per world. Remember we want to try and keep things mostly Vanilla gameplay.
With that said, having gamemode in the builder does complicate things. Namely cause it does break vanilla gameplay doesn't it.
@SpongePowered/developers thoughts?

If I understand correctly from reading the spongedocs, in sponge a world is a collection of dimensions, unlike Bukkit which relocated the dimensions into their own worlds?
So cross-dimension same world teleports is the "vanilla" behaviour and thus is why gamemode is preserved.
When is the players gamemode set in vanilla (without command intervention)? Is it a once off thing that gets remembered and maintained?
As far as I can see, the only difference a worlds gamemode makes, is what players who data is first generated gets assigned, changing the gamemode of the world (and not player) would just result in the player maintaining their original gamemode.
Also in singleplayer when a player changes worlds, it then loads the playerdata associated with that world. So bringing true multi-world support into servers means that inventories and everything would be   separate by default? players teleport across dimensions, but join and leave worlds.

@ryantheleach
In Sponge, each world has its own dimension and are treated as true worlds. The only reason why world and DIM1/-1 seem "linked" is cause of their portal targets.
As I mentioned above, we have to mostly keep vanilla functionality:

We won't isolate inventory per world. Plugin can do this.
Gamemode is tricky. In vanilla currently, you keep your gamemode.

So its probably better that we don't interfere here. If a plugin wants to isolate gamemode per world, we should let them do so.

Considering this is partially handled by data, during the event being thrown for someone transferring between worlds, this shouldn't be much of an issue from the implementation side.
That being said, I do believe it should be handled as vanilla deems fit, and plugins interacting with the player (setting game mode in the event) will take priority for obvious reasons.

Closing this as we follow what Vanilla does. Plugins can easily change the GameMode during events. I'll be removing it from the WorldBuilder.


Improved desktop UI performance

Superputty has a lot of performance problems related to scaling with many connections open and sessions. I have found that in some situations this impacts the rest of the system very badly.
I do not normally program in C# so this would benefit from a thorough review on standards.
This fix mitigates the problem by reducing the amount of interaction superputty has with the rest of the graphical system.
For every application container instantiated, SetWinEventHook was called for the range of events from EVENT_SYSTEM_FOREGROUND to EVENT_OBJECT_NAMECHANGE.
This is firstly inefficient because all of the hooks were global meaning that the hooks for each instance were identical duplicates. The cost of crossing the process boundary to is likely to be higher than the cost of using event listeners within the program to broadcast events.
The range of events was also 0x0003 to 0x800C meaning that most UI events would be captured including some that might be fired very frequently.
To solve this I made the hooks initialise once then provide an event handler accessible via the SuperPutty global class. I since improved this by allowing each Application Form instance to register the hooks needed for their child process filtering with the process id.
In profiling this patch reduced the number of event calls from 37500 (5.8%) in 60 seconds to 2800 (0.48%) in 15 minutes. I have further improved it since and expect an even lower call count with this patch.
Tests show that the system is still impacted by superputty (or possibly putty) but that this impact has been slightly reduced. This can be tested by moving a window and watching the framerate after loading a large number of connections. I am not sure if this problem manifests in all cases as it may depend on the other programs that are running.
I have also made some other improvements that are not yet submitted so it is hard to say how this patch impacts superputty performance in general. Some operations that were slow even with my latest changes now appear to be fast however I also cleaned up and minimised several configuration files which may have an impact as well.
I have additionally added a small fix for detecting putty/kitty command line errors by window title and a check to make sure that the putty window isn't initialised when it is not found. There are still a lot of potential issues with the Application Form and edge cases.

Please hold on merging this. There is a crash issue.

I think I have fixed the crash bug in the latest commit. It looks like it is a regression where .NET garbage collects the function wrapper.
I recommend thorough testing to be sure.
I have also added some simple handling for if the window is not found. I do not know if this is the best way to dispose of a process.

Please review and merge.

Performance improvements are always welcome!

Joey: Looks like you missed a file:
Error	CS0246	The type or namespace name 'GlobalWindowEventArgs' could not be found (are you missing a using directive or an assembly reference?)...PuTTY\superputty\SuperPutty\ctlApplicationPanel.cs	198

The file is there but you need to manually add it from existing arguments in visual studio.
I had that same problem :D. I forgot to mention.
I'm not familiar with visual studio and after its conversion process to update the solution I thought it best to leave the bit to you.

Gotcha, What version of Visual Studio are you using that required a conversion? I've unreverted the revert so your patches are now committed along with the addition of that file to the project file.
Thanks again
Jim

Community 2013, but now I've seen you do it I'll be able to do it next time manually or review the diff first.


HTTPS support for Nextcloud

There should be a way to have HTTPS support in Nextcloud.
It could be passed as a parameter to whiptail or a separate file (which would require more maintenance for 2 separate Nextcloud installations).
The NGINX .conf file (/etc/nginx/sites-enabled/nextcloud in apps/Nextcloud.sh) needs to change for HTTPS and Let's Encrypt needs to be able to create a certificate and auto-renew - email address for server is required (tutorial: https://www.digitalocean.com/community/tutorials/how-to-secure-nginx-with-let-s-encrypt-on-ubuntu-16-04). Sample Nginx HTTPS config for nextcloud: https://docs.nextcloud.com/server/12/admin_manual/installation/nginx.html
If the user doesn't want to use Let's Encrypt, later we could add a self-signed certificate or other trusted certificate option (maybe through copy/paste, decrypt the passphrase, etc.).

This is a legit suggestion, but I don't plan to implement it personnally for now.
HTTPS means to register a FQDN, and adding DNS entries in you provider. We also need to deal with an acme client, paired with a Cron. Or we can also generate a self-signed certificate.
By the way, a Caddy configuration entry is also present for those who have already installed it.
We can nevertheless implement the nginx configuration file described in the official docs for HTTP, with the HTTPS related lines commented.
This feature is not newbie friendly (what this project is aiming for) and will require time that I'haven't actually - I'm actually busy on a new version of DPlatform - dpk (DPlatform Package Manager), in which we can pass arguments like the FQDN and have basic templating engine.
If you feel good to make this, I will gladly welcome your work :)

I'm closing this issue – this feature will be not added on this project, because it risk to became quite unmaintaintable.
If someone is interested to help on this, I ask him/her to wait the release of dpk, the DPlatform Package Manager.

This feature is not newbie friendly (what this project is aiming for) and will require time that I haven't actually - I'm actually busy on a new version of DPlatform - dpk (DPlatform Package Manager), in which we can pass arguments like the FQDN and have basic templating engine.
If you feel good to make this, I will gladly welcome your work :)

I should have said this a long time ago. Sorry about that. At the time, I didn't have as much time to contribute to that particular issue. I agree, maintaining dpk is a better idea since it will be more usable in the future.


no kcl errors are being thrown anymore in the app

again a situation where playwight could have likely caught

this is broken in the current release as well

This program has two bugs (there's a # instead of a // for the initial comment, and the extrude command only has 1 arg instead of 2). Neither bug shows up in the KCL Errors panel, and neither shows up as a CodeMirror diagnostic either.


Remove unnecessary echo $output line from cleaning tests

I left what was essentially a print statement in the tests for the "cleaning" part of the lab. This breaks things for students who try to run the tests on a machine that someone else has run the tests on. The first person to run the tests creates a /tmp/output file and leaves it behind, and when the second person tries to overwrite that file, it turns out that they don't have permission to do so, and the test script fails as a result.

Fixed via commit 22fbcc2 courtesy of @dstelljes.


Graph not plotted/partially plotted when the date is mocked and dimension is specified

Describe the bug
Graph is not plotted/partially plotted when the date is mocked and dimension is specified. This issue exists only when there are multiple canvases.
To Reproduce

Mock the date.
Specify the dimension property for only one of the graph out of the multiple graphs.
(Works fine when the dimension property is specified for all the multilple graphs).

See working example of issue here:
https://codesandbox.io/s/wonderful-mestorf-nkovn
Expected behavior
Graph should get plotted when the date is mocked and dimension is specified.
Screenshots

Desktop (please complete the following information):

OS: Mac
Browser Chrome

Additional context
Add any other context about the problem here.

We have to mock the date to show the last load indicator in the stella timeline for the date specified, which is using new Date.
And i tried using the mockdate so that it doesn't impact globally. But the graph is not plotted completely.

I would check the DOM element id for svg->defs->clipPath, if they are same then your mocked date is impacting carbon and not just the input JSON.

I have created this fork which contains 4 canvases.
https://codesandbox.io/s/xenodochial-hodgkin-xor8j
I have specified dimension for first 3 canvases and the 4th canvas has no dimension. I have observed certain scenarios in which the graph is getting plotted completely and in certain cases, the graph is not plotted completely.
These are the few workflows i have tested - below shows the clip-path id and the dimension of the canvas. Its in the same order the canvases are present in the carbon.
Without mocking the date - Graph plotted completely:
carbon-1580988926107-clip — Dimension not specified
carbon-1580988926235-clip -Dimension not specified
carbon-1580988926263-clip — 290
carbon-1580988926295-clip — 190
With mocking the date - Graph plotted completely:
carbon-1589025600000-clip — 290
carbon-1589025600000-clip — Dimension not specified
carbon-1589025600000-clip — 190
carbon-1589025600000-clip — 190
With mocking the date - Graph not plotted completely:
carbon-1589025600000-clip — Dimension not specified
carbon-1589025600000-clip — Dimension not specified
carbon-1589025600000-clip — 290
carbon-1589025600000-clip — 190
With mocking the date - Graph plotted completely: ( when dimension is not specified for any canvas)
carbon-1589025600000-clip — Dimension not specified
carbon-1589025600000-clip — Dimension not specified
carbon-1589025600000-clip — Dimension not specified
carbon-1589025600000-clip — Dimension not specified
I have 3 dimension sizes - 90, 190, 290 and one canvas has to be without dimension. When i change the dimension of the canvases to different values, in certain cases -  the graph is plotted completely and in certain cases - the graph is not plotted completely.
For instance in the second scenario i have listed above, even after mocking the date and specifying the dimension, the graph is being plotted completely. But if i change the dimension of the 4 canvases to some other values, graph is not completely.


File share snapshot

Description
Outline the issue here:

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here:
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here:
OS Version: What OS and version are you using?
Answer here:
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here:

https://github.com/Azure/azure-cli/pull/4619


[BEAM-9919] Isolating xlang transforms by namespace

All components of the xlang transform that aren't uniquely identified by topological ordering are now isolated into their respective namespaces.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @lostluck

CC: @chamikaramj


Add option to run TensorFlow job on the preferred device

System information

TensorFlow version (you are using): 1.15 or 2.x
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
Now, we could run our code on GPU only via adding GPU dependencies to the classpath.
But the basic Python API provides an ability to set up the preferred device (GPU or CPU via device name)
The basic option also is available for low-level builder here
Will this change the current api? How?
Let's add the function tf.withDevice(“/GPU:0”) to the Scope class.
Who will benefit with this feature?
Anyone who trains neural network in distributed mode on different GPU/CPU devices.
Any Other info.

Sounds good @zaleslaw !
I suggest that we go with a better typed API, maybe using some sort of enum, instead of hardcoded string like this (e.g. something like tf.withDevice(Device.GPU) or tf.withDevice(Device.GPU, 1), etc. Anyhow, something we can define together as we go, wdyt?

I'm not very familiar with all possible device id strings, but it sounds
good for me.
пн, 22 июн. 2020 г. в 17:32, Karl Lessard<EMAIL_ADDRESS>
Sounds good @zaleslaw https://github.com/zaleslaw !
I suggest that we go with a better typed API, maybe using some sort of
enum, instead of hardcoded string like this (e.g. something like
tf.withDevice(Device.GPU) or tf.withDevice(Device.GPU, 1), etc. Anyhow,
something we can define together as we go, wdyt?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/java/issues/77#issuecomment-647556638, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJEUHPT3LKD6ERYZRFK4MLRX5TO5ANCNFSM4OET2PHA
.

I'm in favour of an enum, and then an optional device ID number. We'll need it to specify what GPU something runs on, and I use shared GPU resources so I need to not contend with other users on different GPUs.

Agree with Adam here: GPU/CPU mode with optional String IDs
пн, 22 июн. 2020 г. в 18:43, Adam Pocock<EMAIL_ADDRESS>
I'm in favour of an enum, and then an optional device ID number. We'll
need it to specify what GPU something runs on, and I use shared GPU
resources so I need to not contend with other users on different GPUs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/java/issues/77#issuecomment-647605588, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJEUHLZQEHI7JHWBCR4WSDRX533JANCNFSM4OET2PHA
.

Yes, that was my proposal as well. I just want to make sure that we are not missing anything, I don't know if TensorFlow supports more options than simply "/:" in their string format, can you please double check @zaleslaw ?

The string also accepts host names and a lot of other things we'll have problems fitting in just enums:
https://www.tensorflow.org/guide/gpu
https://www.tensorflow.org/guide/tpu

So probably some kind of builder could be more appropriate then? e.g. Device.job("localhost").replica(0).task(0).gpu(1)

Drive by comment.
Maybe look into existing python class?
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/framework/device_spec.py

I'm sending https://github.com/tensorflow/java/pull/83 for getting the DeviceSpec ported as a start.

Great idea!  Happy to help test this with a multi-GPU system (#131).
Additional question: is there a command-line option to java to explicitly select which GPU should be used?  GPU 0 is selected by default, but my system has 4 GPUs, so I'd like to run 4 java processes at once, with each java process explicitly selecting a different GPU.

Additional question: is there a command-line option to java to explicitly select which GPU should be used?

As expected, the CUDA_VISIBLE_DEVICES environment variable may be used to select GPU 1, for instance:
# export CUDA_VISIBLE_DEVICES=1; nice java -jar my.jar ...

@aday00 and @karllessard looks like we have working device spec now, but have no ability to apply it for our graph or model (we have only setDevice option in GraphBuilder, but we could not pass it through the current TF operands and as a result we have no control on device yet.)
Please, correct me if I'm wrong.

Maybe in eager mode it's better move to runner, but for the static graph look like we need to pre-define it earlier

Yet one problem with imported graphs and models. When and how we could bind the device spec? Maybe after import we should have a method on graph, like addGradients? But in reality we need to rebuild our graph with new device specs.

Hey @zaleslaw , we should probably add the ability to add the DeviceSpec to the Scope when building an operation, similar to what we do with the operation name.
So we can end up doing something like:
tf.withName("myOp").withDevice(DeviceSpec.newBuilder()...).nn.avgPool(...)
If you feel that the syntax for building a device spec can still be improved, feel free to update it, thanks!

The reason for doing it at the scope level is to limit the number of operation factories, otherwise you will double-up the number of endpoints we have in the Ops classes (where one factory method = one endpoint). So instead of having 1000+ endpoints, we'd have now 2000+ endpoints :)

I agree, that 2000 endpoints instead of 1000 is not so good, but the main idea of op device API has a very fine and detailed control over where operations are performed.
Scope is too high and does not give that degree of control

Could you please explain what is the problem with endpoints increasing? Maybe I miss something in performance or this framework development?

Well at some point we'll run out of space in the class file.

Well at some point we'll run out of space in the class file.

I'd like to see that. Check what it looks like for MKL and it's still fine:
https://github.com/bytedeco/javacpp-presets/blob/master/mkl/src/gen/java/org/bytedeco/mkl/global/mkl_rt.java

Like we discussed during the call today, my main concern of adding a new factory, and therefore duplicate all ops endpoints, is the noise that will create in the IDE's auto-suggest feature, plus the fact that Scope was especially designed to handle this case.

Excited to see this feature develop!
Will tf.withName("myOp").withDevice(DeviceSpec.newBuilder()...).nn.avgPool(...) allow different parts of a neural network to be trained on different GPUs simultaneously?
May some parts of the neural networks be trained on CPU and other parts on GPU?
Happy to test this as well, when appropriate.

I'll experiment with different modes. But I agree that it could be useful
to make backprop and multiple matrices on different GPUs for different
branches in models like InceptionV3 and so on.
пн, 2 нояб. 2020 г. в 05:09, aday00<EMAIL_ADDRESS>
Excited to see this feature develop!
Will
tf.withName("myOp").withDevice(DeviceSpec.newBuilder()...).nn.avgPool(...)
allow different parts of a neural network to be trained on different GPUs
simultaneously?
May some parts of the neural networks be trained on CPU and other parts on
GPU?
Happy to test this as well, when appropriate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/java/issues/77#issuecomment-720198504, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJEUHJNYHCUVF3DQ6UBON3SNYIGTANCNFSM4OET2PHA
.

Sounds great @zaleslaw!  For me, the models become so large that 1 GPU and 12GB GPU RAM is not enough to hold the entire model.  Partitioning the model across 2 or more GPUs in a computer would make larger models work.


feat: ckEth orchestrator

Motivation
Add the ckEth orchestrator (source) to @dfinity/ledger-cketh and expose function get_orchestrator_info.
This additional canister is responsible for spinning up Ledger and Index canisters for ckERC20 tokens when (and if) the respective proposals are approved and executed (for example, proposal  129750).
One might want to use this canister to fetch at runtime the list of ledgers and indexes or the list of supported ckERC20 tokens, but most probably, as in Oisy, one might use this feature to set up a job that periodically checks automatically if new tokens were approved and deployed.
Changes

set-up did and idl generation for orchestrator.did
init and expose CkETHOrchestratorCanister (no particular options)
implement get_orchestrator_info (nothing particular, readonly function)
add canister to docs.js to generate its documentation

ckErc20 will require some additional information, such as their logo for Ethereum, which, unlike on the IC, is not provided by their respective smart contracts, so, it probably will still requires a bit of manual review/work.
We also periodically fetch and embed information statically, such as for Snses, rather than dynamically at runtime. This approach ensures that the UX/UI does not feel glitchy, as it does in the NNS dapp. While this may result in a delay of a few days before new tokens are available, it has no functional downsides since users can add custom tokens.

Closed and reopened the PR to restart the checks that were stuck.


Replace dynamic_cast for static_cast for speed

I changed the TokenBase and Token a little bit so we can keep track of the type of the Token. This way we can use static_cast instead dynamic_cast, which should be more efficient.

Looks good, thanks!


Anyway to add a socket to HABridge

I've got a new Alexa enabled bulb that works fine..but as Alexa sees it as a socket style device rather than a bulb, I cant add it to a group so I can say "Alexa lights off".  instead, I currently need to specify the light bulb name.
Is there a way via HAbridge to make Alexa think this is a bulb so this cant be accomplished.
TIA   Merry Christmas

What brand device is it?

It's a generic Chinese brand...works on the efamily plus apps...It has a
mac address and IP address but I cant tell you any more than that
really...very vague I know but thought I would enquire all the same
Happy Christmas
On 24 December 2017 at 15:34, BWS Systems<EMAIL_ADDRESS>wrote:

What brand device is it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/856#issuecomment-353790435,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOp_ReyURUkZGkZ4RlcbMu99uZ_ZaqVeks5tDm8UgaJpZM4RLqYc
.

You probably can as there is tcp device in the ha-bridge. You just need to find out what to send it.


forward slash error with path array inside IronRouter

path: ['/legal'] = bad
path: ‘/legal’ = good
This is an issue in IR when I have pages like path: ['/legal', '/legal/:_id']
Error: Error: path (/legal) must begin with a leading slash "/"
at Object.route (packages/meteorhacks_fast-render/packages/meteorhacks_fast-render.js:178:1)
at handleRoute (packages/meteorhacks_fast-render/packages/meteorhacks_fast-render.js:646:1)
at packages/meteorhacks_fast-render/packages/meteorhacks_fast-render.js:597:1
at Array.forEach (native)
at processRoutes (packages/meteorhacks_fast-render/packages/meteorhacks_fast-render.js:594:1)
at /Users/markvandop/Desktop/employer.resume.com/.meteor/local/build/programs/server/boot.js:249:5

We only support paths starting with /. We don't support paths with arrays. I didn't know that was a feature anyway.

If you like, you can work on a fix. Here's the place to start: https://github.com/kadirahq/fast-render/blob/master/lib/server/iron_router_support.js


MismatchedInputException when getting form data using the envelopes API

Whenever I make a call to get the form data of an envelope envelopesApi.getFormData(accountId, envelopeId), I get the following exception:
! com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot deserialize value of type `java.util.ArrayList<com.docusign.esign.model.FormDataItem>` from Object value (token `JsonToken.START_OBJECT`)
!  at [Source: (org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream); line: 1, column: 1815] (through reference chain: com.docusign.esign.model.EnvelopeFormData["prefillFormData"])
! at com.fasterxml.jackson.databind.exc.MismatchedInputException.from(MismatchedInputException.java:59)

I am able to log into Docusign and download the form data manually, but I need to be able to do this via the API.

@tylerbarker-toast the fix was just release please update to 3.13.1-RC1. Feel free to reopen if the problem persists.
Please refer to this test case for a working example: https://github.com/docusign/docusign-esign-java-client/blob/master/src/test/java/SdkUnitTests.java#L1670

Those who use SDK version 2 can  upgrade to 2.21.1-RC1 if they're not ready to make the move to version 3.


Added missing condition to fetch entities by ID

Description
This PR fixes #
This PR add missing condition for entities filters to fetch them by ID
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

// @amitamrutiya


Update summary table on individual listing page

Update the unit summary table to reflect mock up columns (and available data)

Recording what I remember from conversation with Tom: we want this table to have three columns ("Unit Type", "Availability", and "Total Count"). For now, the "Availability" column will only have "waitlist" (or whatever placeholder text we want), but we'll be able to replace that with the available count once we have that data.

We no longer have this table at all in our ListingView. The reason: amiValues (populated here: https://github.com/CityOfDetroit/bloom/blob/13adcbb7a7c033e93b43cddc24c66c46eaf42f54/sites/public/src/ListingView.tsx#L52) is empty and that makes it so we don't create a GroupedTableGroup of unit types.

I'm confirming that we do still want this table to appear. Assuming that's true, I believe this should read from the new Listing.unitsSummary instead of the derived Listing.unitsSummarized.

Blocked by #345


Directory Not Empty: linalg Error

While trying to install, Initially faced a "pypandoc" error which I was able to resolve installing "pypandoc" seperately but after that I face this error
" ERROR : Could not install packages due to an enviroment error : [ERROR 39] Directory not empty : linalg "
Laptop (please complete the following information):

Windows 10 Pro
chrome

Hi @avinish1,
Could you please try to install Optimus in a conda environment?

@avinish1 Closing this. Let me know if I could help with the issue.


can-submit should always prevent default

You never want the form to submit if CanJS is handling it.

I mostly agree ... but this would be a breaking API change.  It's something we could do for ($submit) however.

btw, for the chat app, use can-enter on the input

It's too late for this.
We could ... allow multiple expressions in call-expressions.
<form ($submit)="%event.preventDefault(); yourMethod()">

But I think some special syntax might be ok too.
<form ($!submit)="yourMethod()">

Closing for: https://github.com/canjs/can-stache-bindings/issues/10


chore(main): release 1.28.3

:robot: I have created a release beep boop
1.28.3 (2024-12-28)
Performance Improvements

♻ Optimize device details performance (38d1433)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.28.3
:sunflower:


No way to navigate to other page

Hey, I would like to open this issue under JWOC.
In the below images for example, there is no way to move from one page to another. This is the case in majority of the pages. I wish to provide an option to navigate in the navbar of all the pages

I want to work on this issue under jwoc.

@nisha331 can I work on this issue ?

@ritika728 i will assign this to u but after few things are complete

@ritika728 i will assign this to u but after few things are complete

Okay sure!

@nisha331 can i start working on it now ?

@nisha331 can I work on this issue now ?


Feature: Alcatraz should appear in dock tile menu

I think it would make sense to appear in the dock menu in the same way as any other open window. I wrote it up on my fork but when I was about to make a pull request I noticed that I forgot to look at the guidelines for contributing, and that I should raise an issue before adding a feature. Also I am yet to write tests, I'm working on that now. Here is my fork: https://github.com/dwicke01/Alcatraz/tree/DockTileMenuOption

I have same problem on Xcode Version 7.1 (7B91b).

@dwicke01 thanks for the suggestion!
We're considering making Alcatraz an .app which would solve this.
I don't think swizzling is the way to go :)


Fix autorest client generation

Which issue this PR addresses:
relates to ADO-15132425
What this PR does / why we need it:
Minor changes to autorest image and client generation:

remove superfluous flags when installing autorest
remove no longer needed sudo commands
run sed operation inside a container, to eliminate flag differences between BSD/Darwin and GNU/Linux versions of sed

Test plan for issue:
Tested locally
Is there any documentation that needs to be updated for this PR?
No, these are just small fixes to scripts

/azp run ci


Update bower, put tooltip content in content, remove controller from fieldinfo

Please merge this to https://github.com/uwdata/voyager too.


Give NickCrews permissions

Triage access
👋 I am a defund12.org collaborator requesting triage permissions so I can assist in moderating issues, and doing simple/obvious tweaks. IDK exactly what the permission levels are, sorry I can't be more precise, just give me the lowest one and I can start there.

Starting you off with triage access to apply labels. Please thoroughly review the teams page on the workflow and how to get started.

Done!


[TF] Cleanup echo matchers

Moves matchers to the match package, which aids in readability from the call sites.

Moved match methods out of echo.Instances to avoid circular dependency.

Added a few more options for Filters and Matchers that will be used in subsequent PRs.

Please provide a description of this PR:

@nmittler: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
283252e33c00cb3a349eb13049dd3c4c95b761d5
link
true
/test unit-tests_istio

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.

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
283252e33c00cb3a349eb13049dd3c4c95b761d5
link
true
/test unit-tests_istio

lint_istio
283252e33c00cb3a349eb13049dd3c4c95b761d5
link
true
/test lint_istio

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.


Uncouple screens from old Accounts structure

Making this as a catch-all issue to track various screens that need to be refactored in ordered to function with the new accounts structure introduced in #1663. (There may well be some that I'm missing, and I'll try to update this as I find them.)
Ideally, these changes can be merged into that PR's branch before merging that PR itself.

Accounts -> Send screens (#2039)
Contacts -> Send screens
Receive screens
Voucher Scanner
Funding Sources Screens
ManageBackup Screens
Accounts -> TwoFASetup
Accounts -> Transactions Screens

Removing tag 1.4.0 from this as this has been done for 1.4.0

Closing this as this was an generic task for all account management.


Add provider for rke2

Initial commit to add the rke2 provider

Good feedback, my goal is to delete the install script ASAP and rely on upstream once https://github.com/rancher/rke2/pull/3248 is merged/released


The Hide annoying ads option is blocking articles on some websites

Describe the bug
On some websites, articles are blocked after I enabled Hide annoying ads in the menu of Kiwi Browser.
Expected behavior
The articles should not be blocked when the option is enabled.
Screenshots
https://www.kocpc.com.tw/archives/394990
With the option disabled

With the option enabled

Smartphone (please complete the following information):

Device: Xiaomi POCO M3
Android Version : Android 10
Kiwi Browser Version 93.0.4577.18

as its really hard to do ad blocking correnct for all websites (espefcially non latin languages) I suggest to use a adblocker addon. they usually has blocklists for different countries and languages plus simple opt out for individual sites.
e.g. ublock origin

I too would strongly suggest you use ublock origin for this purpose

I agree with @gnadelwartz and @Oranko.
In fact, it could be discussed if Kiwi's ad blocking is actually a feature worth to maintain, considering the many options Kiwi offers by supporting extensions. @kiwibrowser

Tested on Kiwi v116, as built-in adblock has been greatly improved, article is no longer blocked.


Disabled button doesn't pass AA contrast requirement for accessibility

Issue report
Disabled button text doesn't pass AA contrast requirement for accessibility
Expected Behavior

The button color and text color needs a minimum contrast of 4.5

The button color and text color needs a minimum contrast of 4.5 currently it is at 4.33
Current Behavior

Possible Solution

Darkening the text seems like the best solution in this case since the background of the button is already really light.

Context

We need to release the routemap and can't have accessibility issues once its in prod.

Several people from the UX/UI team researched this within the larger digital color palette conversation. While creating the specs for various form elements, the specific issue of disabled states was reviewed.
"1.4.3 Contrast (Minimum): The visual presentation of text and images of text has a contrast ratio of at least 4.5:1, except for the following: (Level AA)

Large Text: Large-scale text and images of large-scale text have a contrast ratio of at least 3:1;

Incidental: Text or images of text that are part of an inactive user interface component, that are pure decoration, that are not visible to anyone, or that are part of a picture that contains significant other visual content, have no contrast requirement.

Logotypes: Text that is part of a logo or brand name has no minimum contrast requirement."

Based on input from @clockeby I am closing this issue.


Dark mode documentation

Please could docs be updated to include a dark-mode toggle? :)
This should be pretty easy with mkdocs, see these PRs for inspiration:

https://github.com/cashapp/sqldelight/pull/3637
https://github.com/square/okio/pull/1174

I'm willing to make a PR, if you'd like

P.S. it would be nice to add the documentation URL on the GitHub About section

Oh "these" dark mode people. :-) Thanks for the suggestion. The links contrast when using dark mode now, lack a bit of contrast in my taste, but I guess it's fine for now.
I've adjusted the link as well.

Wonderful, thank you on behalf of the dark mode people! :)
Yes, the link contrast is very low... It looks like it might be this issue https://github.com/squidfunk/mkdocs-material/issues/4600? Although it does look like kalasim is using the latest versions of Mkdocs and Mkdocs Material... but I'm not very familiar with Mkdocs or Python.

Not sure how to indicate a more recent version in https://github.com/holgerbrandl/kalasim/blob/master/.github/workflows/manual.yml
The mkdocs version comes along somehow via the github action system.


(ISO 6709) Reference to a "locality range" drops actual locality

When I change to use a "locality range", I get this output:
Using the character string delimiters specified in
<<item_6-4-a;to!item_6-4-i>> the following structures are defined:

However, this text is in 6.6, and therefore should actually be "6.4 List 1 a) to List 2 e)"
Originally posted by @ronaldtse in https://github.com/metanorma/iso-6709/issues/10#issuecomment-1117050052

The common shared container was not being added to combined xrefs.

Confirm fixed. Thanks.

Confirm fixed. Thanks!


Project: Videocall your AI

Project name
callyour.ai
Description
Textual interfaces are the norm for chatbot interaction. This is not optimal for many users. This is why I created callyour.ai, where you can have a videocall with your AI, which is a more natural way of conversation for many users.
Language
Almost all common languages
Project Repository URL
callyour.ai
Deployed Endpoint URL
No response
Project video
https://youtu.be/SQHI_G8UUmo?si=MjJLJBKlsVx_SFCi
Team members
Jonasdawi
Showcase Consent
Yes

I think i cannot send the code. Thank you!


Custom props addded through 'getHeaderProps' not available in 'Header' callback

What version of React-Table are you using?
6.7.4
What bug are you experiencing, or what feature are you proposing?
I have some flags stored at the Redux level that I need to expose to React-Table headers and cells. For cells I'm using the getProps callback and attaching my flags which in turn are being applied to ReactTable as top level props. This works ok but it fails in the headers. getPropsHeader works but the props I add are not then available in the Header renderer.
Our columns settings are big and complicated so they are in a separate library file where there is no Redux to pass the props directly to the callbacks, like I've seen in most examples. Is there another way to pass down these top level props on  to Header callbacks as well?
https://codesandbox.io/s/r788rpnjqm
What are the steps to reproduce the issue?

Return new props using 'getHeaderProps'
Access the props parameter in the Header callback and confirm the new props are not present. This same method works for getProps in the Cell callback.

I understand what you are saying, but, unless your columns are changing dynamically (and it just makes it more complex) you can decorate your columns inside you wrapping component before the first render (either in the constructor or one of the React lifecycle methods).  That way, you can add in-scope functionality when the component is initialised.  The case where the columns may change outside requires you to either use something like the new React context mechanism or the old lifecycle methods for changing props).
I hope this makes sense.  There is unlikely to be any change to the existing version of ReactTable as that would require a code-wide change.  It may be considered for the next major re-write.
@tannerlinsley might want to review this request.

The columns don’t change dynamically so that’s not a problem. At first render is enough.
So what you’re suggesting is to intercept the columns settings in the container component and mutate them BEFORE passing them to React Table? How, exactly? Are there any callbacks I can use or would it be overriding the ‘Header’ callback that was set in the original object? I’m not sure I follow so hopefully you can add more details or an example.
Ideally I’d be able to pass these props to the renderer callbacks and just use them when their called. It may not be possible then. I just don’t want to mess up the container component with logic that stricyly belong there the columns. Perhaps I need to wrap my columns into an high order function that receives the Redux state and then outputs the columns object.
Thank you very much for your time and this library.

What you have described is definitely what I meant - I don't have a generic example (everything is wrapped up in my code).  However, the selectTableHOC example augments the columns on the way in to ReactTable - so that might provide some hints.
Another possibility is to pass a "getColumns" function as a prop and then call that back and pass in the "this" of the wrapping component so you can get the context you need that way.

Closing due to issue age. If this issue was a question, please ask it again on https://spectrum.chat/react-table. If you think this issue needs to be reopened or if it should be turned into a pull-request, please check the latest version of React-Table for the issue or feature once more and reopen or create a PR if appropriate. Thanks!


SW-5356 Added modules dispatch into ParticipantProvider, used projects with modules only for Participant Experience

#2603  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite

We have this ticket tracking it:
https://terraformation.atlassian.net/browse/SW-5331


Allow creating GooseConfiguration from code

Hi! Thanks for this awesome tool!
We use it to compare different implementations of our service against each other.
Unfortunately there is currently no way to run GooseAttacks with different configs from the same process because there is a private field on GooseConfiguration which prevents its creation.
And when several -H flags are provided on the command line, gumdrop silently drops all but the last one.
We resort to spawning sub-processes with different command lines which causes some inconveniences.

You can configure defaults in code -- what are you trying to do that's different than this allows?
https://book.goose.rs/config/defaults.html

What are you using to parse the CLI? In theory you should be able to make this work by dynamically setting the "defaults".

As Goose already "owns" the command line, I'd typically handle this with environment variables. For example, maybe something like:
#[tokio::main]
async fn main() -> Result<(), GooseError> {
    let custom_host = match std::env::var("HOST") {
        Ok(host) => host,
        Err(_) => "".to_string(),
    };

    GooseAttack::initialize()?
        .register_scenario(scenario!("LoadtestTransactions")
            .register_transaction(transaction!(loadtest_index))
        )
        .set_default(GooseDefault::Host, custom_host.as_str())?
        .execute()
        .await?;

    Ok(())
}

In this way, you can set a custom default, for example:
HOST="http://local.dev/" cargo run --release

And you can still override it from the command line, for example:
HOST="http://local.dev/" cargo run --release -- --host http://apache.fosciana/

This is what I ended up doing, thanks for the hint!


Documentation inconsistant with implemnation for Can/Cannot Components

The documentation describes the usage of Can/Cannot as follows:
<template>
  <Can
    :ability="editPost"
    :args="[post]"
  >
    <button>Edit</button>
  </Can>
</template>

Looking at the implementation, the components expect the prop to be named :bouncer-ability="...".
I'll be happy to create a PR, just need to know If we want to fix implementation or documentation.

Hey,
This is an issue within the documentation. 👍
Could you make a PR?

Ah, already fixed. Thanks!


MSKINS-143: Allow setting GA options for forceSSL and anonymizeIP

This change allows to set the GA tracker options for forceSSL
and anonymizeIP. This may be required due to data protection laws.

I think something like this should not be necessary and both settings shall always be present with true: no options.

I think this decision should be left to the user of this skin.
However I would agree that the default should be true for both options.

Can you change the logic to be true by default?

Sure.

I just pushed the update. I contains three integration tests now (explicit true, explicit false, default). The default is true now.


Add Notification model and handle notification event

I plan to submit a PR similar to https://github.com/home-assistant/core/pull/45729 where we fire an event when a notification is received for a node.

Please rebase to solve the merge conflict.

@raman325 maybe use the same logic to fire the event ? I can adjust my PR so it will also forward these events ?

The event data will be different but otherwise yeah that makes sense. I was thinking it would be nice eventually to tie these events to the primary "entity" but that might be easier said than done.

Entities don't fire events, besides the built in state changed events, in Home Assistant.

sure I meant add entity_id as part of the event data to use as criteria for automations

We can discuss that.


Missed target sound isn't loud enough

Should try stopping game music first, then playing missed target sound.

Done as part of #41.


Create Module Get-LocalUsers.ps1

Created new conf module Get-LocalUsers.ps1
Will list all local users on the host detailing the following:
- Disabled (bool),
- Name,
- Full Name,
- Caption,
- Domain
- SID.

Requested changes have been completed


Don't hide comments inside of code block samples

Some sites frequently show blocks of code that include comments. No reason to hide those comments.

https://plainjs.com/javascript/manipulation/wrap-an-html-structure-around-an-element-28/
https://developer.mozilla.org/en-US/docs/Web/API/Node/insertBefore

Possibly use :not somehow?

[ ] asdf
[ x ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)


Webdav server in a prefixed url

When the root of a webdav server is hosted like https://example.com/sub/root/, and we query the root folder, the name of the resource in a PROPFIND response is / and not /sub/root/ leading to a key error.
I would propose to add an optional parameter in the constructor, would it be ok ? Or should we prefer another way ? I can make a PR.

FYI: #171 might be somewhat related


Keeping lessons and data together

@ttimbers, I noticed that you created a data directory. I feel like it might be better to keep the lessons and data in one place, so that it's easier to find one from the other. For instance, I would create a directory for the lesson here (e.g. 2015-07-14) and store both the lesson markdown notes and data therein. What do you think?

Sure, go ahead and do that, please just update the link in the issue so
people can easily grab the data tomorrow.
On Jul 13, 2015 5:15 PM, "Bruno Grande"<EMAIL_ADDRESS>wrote:

@ttimbers https://github.com/ttimbers, I noticed that you created a data
directory https://github.com/ttimbers/studyGroup/tree/gh-pages/data. I
feel like it might be better to keep the lessons and data in one place, so
that it's easier to find one from the other. For instance, I would create a
directory for the lesson here
https://github.com/ttimbers/studyGroup/tree/gh-pages/lessons (e.g.
2015-07-14) and store both the lesson markdown notes and data therein.
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/ttimbers/studyGroup/issues/25.

When you move things, please make sure you link to the .zip file as I did
in the issues. This makes it easier for non Git/Github people to download
the data. Thanks.
On Jul 13, 2015 5:15 PM, "Bruno Grande"<EMAIL_ADDRESS>wrote:

@ttimbers https://github.com/ttimbers, I noticed that you created a data
directory https://github.com/ttimbers/studyGroup/tree/gh-pages/data. I
feel like it might be better to keep the lessons and data in one place, so
that it's easier to find one from the other. For instance, I would create a
directory for the lesson here
https://github.com/ttimbers/studyGroup/tree/gh-pages/lessons (e.g.
2015-07-14) and store both the lesson markdown notes and data therein.
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/ttimbers/studyGroup/issues/25.

Done! Thanks.


Add Disqus provider

I've just created a Disqus provider.
Thanks for this library.

Thank you for contributing! 🎉


Fix #1875: Azure OpenAI: propagate all exceptions in chat models

Issue
Paritally fixes #1875
Change
This is a breaking change!

Changed AzureOpenAiChatModel  to propagate all exceptions thrown from Azure OpenAI SDK to the caller.
Changed AzureOpenAiStreamingChatModel  to propagate all exceptions thrown from Azure OpenAI SDK to the caller via StreamingResponseHandler.onError(e).
Changed AzureOpenAiChatModel and AzureOpenAiStreamingChatModel to propagate all exceptions thrown from Azure OpenAI SDK via ChatModelListener.onError(e).

Next Steps
A follow-up PR with similar changes should be submitted for AzureOpenAiImageModel, AzureOpenAiLanguageModel and AzureOpenAiStreamingLanguageModel. InternalAzureOpenAiHelper.contentFilterManagement() should be removed. cc @jdubois could you please handle this?
General checklist

[ ] There are no breaking changes
[X] I have added unit and/or integration tests for my change
[ ] The tests cover both positive and negative cases
[X] I have manually run all the unit and integration tests in the module I have added/changed, and they are all green
[ ] I have manually run all the unit and integration tests in the core and main modules, and they are all green
[ ] I have added/updated the documentation
[ ] I have added an example in the examples repo (only for "big" features)
[ ] I have added/updated Spring Boot starter(s) (if applicable)

cc @myrosia

@mariofusco if I apply spotless, it will introduce so many changes, it will be hard to review

@mariofusco if I apply spotless, it will introduce so many changes, it will be hard to review

Agreed. The only alternative that I see for now is disabling it or at least avoid that it makes to fail the build.

OK this is in fact quite simple, I'm waiting for this to be merged and then I'll do the other models.

@mariofusco @jdubois thank you for the review!


Why would a successfully installed chart release be ignored by a subsequent helmfile diff command?

I have a situation where one chart, prometheus-operator, is successfully installed with helmfile apply, yet when a helmfile diff command is executed after the install it presents a diff showing what appears at first glance to be all the resources that were previously installed.  I imagine that helmfile defers to the installed helm-diff plugin and that it is, for some reason, returning new versions of the previously installed manifests.  For all of my other chart installs, they properly report no diff needing to be applied.
This started happening during migration to helm 3 use.  I never saw this happen when using helm 2.  My environment is:
$ helm version
version.BuildInfo{Version:"v3.0.0", GitCommit:"e29ce2a54e96cd02ccfce88bee4f58bb6e2a28b6", GitTreeState:"clean", GoVersion:"go1.13.4"}

$ helm plugin list
NAME    	VERSION   	DESCRIPTION
2to3    	0.2.0     	migrate and cleanup Helm v2 configuration and releases in-place to Helm v3
diff    	3.0.0-rc.7	Preview helm upgrade changes as a diff
helm-git	0.5.0     	Get non-packaged Charts directly from Git.
registry	0.7.0     	This plugin provides app-registry client to Helm.
        	          	usage:
        	          	  $ helm registr...

$ helmfile --version
helmfile version v0.93.1

$ uname -a
Darwin aus-allengooch-osx 18.7.0 Darwin Kernel Version 18.7.0: Sat Oct 12 00:02:19 PDT 2019; root:xnu-4903.278.12~1/RELEASE_X86_64 x86_64

If the problem is strictly within the helm-diff plugin, I'll go ahead and add an issue in its repo.  Is there any possibility of it also being caused, at least in part, by helm or helmfile commands?
Many thanks!

Hey! Thanks for reporting.

Is there any possibility of it also being caused, at least in part, by helm or helmfile commands?

I beieve, no.
Are you able to reproduce it by running helm diff alone?

I ran a helm -n monitoring diff upgrade prometheus-operator stable/prometheus-operator --reuse-values command and got the same diff output, so the problem seems to be there.  I assume helmfile is passing in the namespace option necessary now that releases are namespace scoped with helm 3.
I'll go ahead and add an issue in the helm-diff repo.


Incorrect algorithm result on specified values

I'm use demo from this repo.
If you set the following coordinates for objects, the value is not calculated correctly.
Start: x: 27.5 z: -33.5
Sphere: x: -2 z: 1.5
I partially studied the algorithm and came to the conclusion that the problem is here:
if (positionX % half! = 0 || positionX % 1 == 0)
How best to solve this problem? Thanks)

Hey Vyachaslav,
Thanks for spotting this! I currently don't have a computer to run this
program on. Do you mind posting a screenshot or list of the enumerated
results?
Best Regards,
John
On Tuesday, May 1, 2018, Vyacheslav Spirin<EMAIL_ADDRESS>wrote:

I'm use demo from this repo.
If you set the following coordinates for objects, the value is not
calculated correctly.
Start: x: 27.5 z: -33.5
Sphere: x: -2 z: 1.5
I partially studied the algorithm and came to the conclusion that the
problem is here:
if (positionX % half! = 0 || positionX % 1 == 0)
How best to solve this problem? Thanks)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SnpM/Pan-Line-Algorithm/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHXqp_RhsucxSdftG3d1DRhDWcjyFY0iks5tuTvFgaJpZM4Tu2o2
.


Vulnerability form populates activities from all orgs

Need to fix the Vuln form not updating the activity when updating the Vulnerability


SpringBoot Failed to Start Due to EntityManagerFactory

Cloned this project and import the complete project into my eclipse and run exactly as is except for the DB credentials but the application failed to start  because it was looking for entityManagerFactory.

Description:
Field userRepository in hello.MainController required a bean named 'entityManagerFactory' that could not be found.
Action:
Consider defining a bean named 'entityManagerFactory' in your configuration.

I missed to see the other thread with the same issue as I have- it was closed already and just followed the solution provided which is to clear out the m2 cache and it did work for me. The solution is a bit odd but it worked.


Escaping logic for # in text nodes interacts with parsing for Cicero templates

Describe the bug
Roundtripping CiceroMark templates may include # which is used for blocks (e.g., {{#if}} ... {{/if}}.
This creates conflicts with the escaping logic which turns # in text blocks into \#. The escaping logic should me made more robust to avoid that issue.
Example of parsing issue in Cicero resulting from this:
  65) Template
       #markdown
         should load copyright-license markdown template and parse:
     Error: invalid syntax at line 17 col 3:

  {{\#clause paymentClause}}Payment. As consideration in full for the rights granted herein, Licensee shall pay Licensor a one-time fee in the amount of {{amountText}} ({{amount}}) upon execution of this Agreement, payable as follows: {{paymentProcedure}}.{{/clause}}
    ^

Fixed in #170


feat: relay to EVM via ABCI

Description
A few significant features:

Users can use the x/gmp/createPayment msg to create a payment to periodically push updates to Axelar Contracts
The x/gmp module's endblocker will push updates for each payment until they run out of funds
A new x/gas_estimation module, which has parameters and a KV store that the oracle module & gmp module can access without running into circular dependencies
Updates to the x/oracle/abci package to also use ABCI++ to calculate gas estimation and set it in the KV store

Other updates:

go version update to v0.23

Todo:

Fix e2e tests

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

This is pretty much ready for review, I just need to update e2e, liveness, and the abi test suite to be compatible with the new logic / module

E2E tests are periodically failing because sometimes it tries to submit a gov proposal while the validator account is doing other things. Doesn't seem related to this PR


Add support for Macro Bar drag & drop.

Originally created by @ghost (https://github.com/fh-fvtt/zweihander/issues/44):
https://trello.com/c/EJEAAGdP/78-add-support-for-macro-bar-drag-drop
Implementing this feature would allow players to e.g. drag their favored weapon attack to the Macro Bar for quick access, allowing them to make certain rolls without opening the character sheet, thus speeding up play.

https://foundryvtt.wiki/en/development/guides/SD-tutorial/SD16-Adding-macrobar-support-to-your-Items

Implemented for Weapons and Spells.


[Feature] Display the amount of processed data when running tests

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
When executing the run or build command, dbt displays the amount of data processed to create tables, I think it would be nice it also displayed the amount of data processed when running tests
Describe alternatives you've considered
No response
Who will this benefit?
We had a situation that after writing some new models, a chart in a report tracking the amount of processed data in BigQuery showed a very sharp increase... we looked at the output from dbt and found that it was processing a lot of data to create some tables, we fixed that but the amount of data processed still increased more than expected, we learned that we had missed some tests that were scanning a lot of the same data and refactored the tests downstream to other models.
Are you interested in contributing this feature?
Yes but I don't know the codebase, no idea where I would start.
Anything else?
No response

In the meantime, did you take a look at your target/run_results.json?
According to https://github.com/dbt-labs/dbt-core/issues/2964 and https://github.com/dbt-labs/dbt-bigquery/issues/559, the bytes processed should be available there.

Yes, indeed they are available there, thanks, didn't know about that.

But it seems like your ask is specific to the CLI output like this and you want PASS replaced with something like PASS (1.0 rows, 0 processed)

Yes, displaying it in the CLI output would be great.

Agreed, I asked for this feature back in https://github.com/dbt-labs/dbt-bigquery/issues/14#issuecomment-1295698710 and it just occurred to me that it never showed up even though I thought that https://github.com/dbt-labs/dbt-bigquery/issues/559 had added it! Didn't realise it was only in the JSON either.


Application Insights custom dependency tracking disconnected from request

I am tracking custom dependency calls with Application Insights in a web API built with ASP.NET Core (v1.1.1, .NET Framework 4.6 and using the WebListener host).
The problem is that the dependency trace does not appear related to the request when I browse the traces in the portal. Also, the request duration is smaller than the dependency duration.
I don't see this problem with non-Core ASP.NET web APIs, so I suspect there is something wrong with the ASP.NET Core implementation...
The operation implementation does not contain any asynchronous calls.
Request trace:
{
  "name": "Microsoft.ApplicationInsights.Dev.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.Request",
  "time": "2017-03-08T14:21:50.1802621Z",
  "iKey": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx",
  "tags": {
    "ai.internal.sdkVersion": "aspnet5f:2.0.0",
    "ai.application.ver": "<IP_ADDRESS>",
    "ai.internal.nodeName": "jcsurf3.cid.dom",
    "ai.operation.id": "80000023-0003-ff00-b63f-84710c7967bb",
    "ai.location.ip": "::1",
    "ai.cloud.roleInstance": "jcsurf3.cid.dom",
    "ai.operation.name": "GET Assure/Get [id]"
  },
  "data": {
    "baseType": "RequestData",
    "baseData": {
      "ver": 2,
      "id": "oNSnkVbILw0=",
      "name": "GET Assure/Get [id]",
      "duration": "00:00:00.0166757",
      "success": true,
      "responseCode": "200",
      "url": "http://localhost:5508/Assure/42",
      "properties": {
        "AspNetCoreEnvironment": "Development",
        "httpMethod": "GET",
        "DeveloperMode": "true"
      }
    }
  }
}

Dependency trace:
{
  "name": "Microsoft.ApplicationInsights.Dev.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.RemoteDependency",
  "time": "2017-03-08T14:21:50.1323509Z",
  "iKey": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx",
  "tags": {
    "ai.internal.sdkVersion": "dotnet:2.2.0-54037",
    "ai.application.ver": "<IP_ADDRESS>",
    "ai.internal.nodeName": "jcsurf3.cid.dom",
    "ai.operation.id": "oNSnkVbILw0=",
    "ai.location.ip": "::1",
    "ai.cloud.roleInstance": "jcsurf3.cid.dom",
    "ai.operation.name": "GET Assure/Get [id]"
  },
  "data": {
    "baseType": "RemoteDependencyData",
    "baseData": {
      "ver": 2,
      "name": "FakeAssureRepo",
      "id": "B16EXLDPLWU=",
      "data": "Read",
      "duration": "00:00:00.0542780",
      "success": true,
      "properties": {
        "AspNetCoreEnvironment": "Development",
        "DeveloperMode": "true"
      }
    }
  }
}

What version of the SDK are you using?

My project references the package Microsoft.ApplicationInsights.AspNetCore version 2.0.0, which has dependencies on other application insights packages version 2.2.0.

/cc @pakrym

@jcsharp where do you track a dependency? Can you share a repro code?

Here's a repro:
    [HttpGet]
    [Authorize(policy: "Viewer")]
    public string Get()
    {
        bool result = true;
        var startTime = DateTimeOffset.Now;
        var stopwatch = System.Diagnostics.Stopwatch.StartNew();
        try
        {
            Task.Delay(100).Wait();
            return "Hello";
        }
        catch (Exception)
        {
            result = false;
            throw;
        }
        finally
        {
            stopwatch.Stop();
            var telemetry = new Microsoft.ApplicationInsights.TelemetryClient();
            telemetry.TrackDependency("Dependency", "Test", startTime, stopwatch.Elapsed, result);
        }
    }

And the traces:

Note how the request duration is much shorter than the dependency call.

Try injecting TelemetryClient into constructor instead of creating new: https://github.com/Microsoft/ApplicationInsights-aspnetcore/blob/develop/test/MVCFramework45.FunctionalTests/Controllers/HomeController.cs#L13

Better. When I inject the TelemetryClient in the controller, the "all telemetry for this operation" link shows both the request and the dependency traces together.
However, I still see a much shorter duration on the request than for the dependency.

If I look at the traces on the Azure portal, I still don't see the dependency call on the request:

Looks like a bug, filed https://github.com/Microsoft/ApplicationInsights-aspnetcore/issues/370

@jcsharp Let's move the discussion there.


Add translation

https://addons.mozilla.org/zh-TW/firefox/addon/image-picka/reviews/969619/

Added in 0.6.0.


mat-input-underline and mat-select-underline - Inconsistent element and property use

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Be consistent with the HTML elements and css properties
What is the current behavior?
mat-input-underline is using a div element and css border property
mat-select-underline is using a span element and css background-color property
What is the use-case or motivation for changing an existing behavior?
Easier styling. Especially for custom modifications.
Which versions of Angular, Material, OS, browsers are affected?

OS: macOs Sierra
Chrome: Version 56.0.2924.87 (64-bit)
Angular: 4.0.0-rc.2
Material: 2.0.0-beta.2

It looks like mat-select-underline is missing .

Tracking via #4673

+1


HBCU Faculty Invitation

[ ] Email to returning faculty
[ ] Email to new faculty
[ ] Set Calendly options for new faculty
[ ] Email for top of year follow up to invitation
[ ] Confirm total # capacity for HBCU faculty awardees

Emails from last year

[ ] Send emails to new
[ ] Send emails to returning