Compiler crash: Iterables do not have same length

Flutter 3.13.2
environment:
sdk: '>=3.1.0 <4.0.0'

After a specific dio 'post' call I'm getting this error:
Unhandled exception:
Invalid argument(s): Iterables do not have same length.
#0      MapBase._fillMapWithIterables (dart:collection/maps.dart:178:7)
#1      new LinkedHashMap.fromIterables (dart:collection/linked_hash_map.dart:246:13)
#2      Substitution.fromPairs (package:kernel/type_algebra.dart:263:13)
#3      InferenceVisitorBase.getGetterTypeForMemberTarget (package:front_end/src/fasta/type_inference/inference_visitor_base.dart:1465:35)
#4      InstanceAccessTarget.getGetterType (package:front_end/src/fasta/type_inference/object_access_target.dart:489:30)
#5      InferenceVisitorImpl._computePropertyGet (package:front_end/src/fasta/type_inference/inference_visitor.dart:6663:36)
#6      InferenceVisitorImpl.visitPropertyGet (package:front_end/src/fasta/type_inference/inference_visitor.dart:7672:61)
#7      PropertyGet.acceptInference (package:front_end/src/fasta/kernel/internal_ast.dart:2984:20)

#8      InferenceVisitorImpl._inferExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:257:27)
#9      InferenceVisitorImpl.inferNullAwareExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:310:40)
#10     InferenceVisitorImpl.visitIndexGet (package:front_end/src/fasta/type_inference/inference_visitor.dart:5227:48)
#11     IndexGet.acceptInference (package:front_end/src/fasta/kernel/internal_ast.dart:1824:20)
#12     InferenceVisitorImpl._inferExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:257:27)
#13     InferenceVisitorImpl.inferNullAwareExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:310:40)
#14     InferenceVisitorImpl.visitIndexGet (package:front_end/src/fasta/type_inference/inference_visitor.dart:5227:48)
#15     IndexGet.acceptInference (package:front_end/src/fasta/kernel/internal_ast.dart:1824:20)
#16     InferenceVisitorImpl._inferExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:257:27)
#17     InferenceVisitorImpl.inferNullAwareExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:310:40)
#18     InferenceVisitorImpl.visitIndexGet (package:front_end/src/fasta/type_inference/inference_visitor.dart:5227:48)
#19     IndexGet.acceptInference (package:front_end/src/fasta/kernel/internal_ast.dart:1824:20)
#20     InferenceVisitorImpl._inferExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:257:27)
#21     InferenceVisitorImpl.inferNullAwareExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:310:40)
#22     InferenceVisitorImpl.visitIndexGet (package:front_end/src/fasta/type_inference/inference_visitor.dart:5227:48)
#23     IndexGet.acceptInference (package:front_end/src/fasta/kernel/internal_ast.dart:1824:20)
#24     InferenceVisitorImpl._inferExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:257:27)
#25     InferenceVisitorImpl.inferNullAwareExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:310:40)
#26     InferenceVisitorImpl.visitIndexGet (package:front_end/src/fasta/type_inference/inference_visitor.dart:5227:48)
#27     IndexGet.acceptInference (package:front_end/src/fasta/kernel/internal_ast.dart:1824:20)
#28     InferenceVisitorImpl._inferExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:257:27)
#29     InferenceVisitorImpl.inferExpression (package:front_end/src/fasta/type_inference/inference_visitor.dart:291:40)
#30     InferenceVisitorImpl.visitReturnStatement (package:front_end/src/fasta/type_inference/inference_visitor.dart:7894:11)
#31     ReturnStatement.accept (package:kernel/ast.dart:10127:43)
#32     InferenceVisitorImpl.inferStatement (package:front_end/src/fasta/type_inference/inference_visitor.dart:242:26)
#33     TypeInferrerImpl.inferFunctionBody (package:front_end/src/fasta/type_inference/type_inferrer.dart:198:17)
#34     BodyBuilder.parseSingleExpression (package:front_end/src/fasta/kernel/body_builder.dart:1780:62)
#35     SourceLoader.buildExpression (package:front_end/src/fasta/source/source_loader.dart:1320:21)
<asynchronous suspension>
#36     IncrementalCompiler.compileExpression.<anonymous closure> (package:front_end/src/fasta/incremental_compiler.dart:2033:39)
<asynchronous suspension>
#37     IncrementalCompiler.compileExpression (package:front_end/src/fasta/incremental_compiler.dart:1853:12)
<asynchronous suspension>
#38     FrontendCompiler.compileExpression (package:frontend_server/frontend_server.dart:981:28)

<asynchronous suspension>
#39     listenAndCompile.<anonymous closure> (package:frontend_server/frontend_server.dart:1452:11)
<asynchronous suspension>

3
the Dart compiler exited unexpectedly.
Exited (1)

@burekas7 Did this happen when evaluation an explicit expression during debugging or when hovering expressions in the editor? Also was this part of a hot-reload cycle (i.e. was the code "recently" modified) ?

+1 , got same error when hovering on the expressions in VS CODE editor , using Dio package to debug the responses .
flutter doctor
`Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.2, on macOS 13.5 22G74 darwin-x64, locale en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.81.1)
[✓] Connected device (3 available)
[✓] Network resources
• No issues found!`

@heshesh2010 Do you have any idea what expression was?

same

+1 here having a similar issue :
     List<List<int>> l = List.generate(list.length, (index) => []);
     List<dynamic> guide = [
       {
         "field": null,
         "nextList": null,
         "selection": l,
         "max_selected": 1000,
       }
     ];

this generate a debugger crash :
Unhandled exception:
Invalid argument(s): Iterables do not have same length.
#0      MapBase._fillMapWithIterables (dart:collection/maps.dart:178:7)
#1      new LinkedHashMap.fromIterables (dart:collection/linked_hash_map.dart:246:13)
#2      Substitution.fromPairs (package:kernel/type_algebra.dart:263:13)
#3      InferenceVisitorBase.getGetterTypeForMemberTarget (package:front_end/src/fasta/type_inference/inference_visitor_base.dart:1465:35)

This seems to originate from a bad encoding of the arguments passed to the expression compiler - at least I can recreate the crash by creating such a bad message. I'm still not still sure the where in the tool chain this encoding is created, though.
Likely related to https://github.com/dart-lang/sdk/issues/53382

cc @jensjoha

@jensjoha @mraleph Can we be sure that a raw type of a generic class in the VM has the right number of types arguments? I.e. could the VM be creating the encoding ["dart:core", "List", "1", "0"] for List in CollectStringifiedType ?

@johnniwinther I think they should have correct length. Type object in the VM should carry TAV which matches the number of type parameters the class has (cc @alexmarkov)
However the code in CollectStringifiedType is a bit suspicious - it does not seem to handle raw types (e.g. List<dynamic>) correctly. VM used to denote those using null TAV and CollectStringifiedType does not seem to handle that at all. Maybe we can try to reproduce by trying expression compilation in a context with a variable which has a raw type, e.g. final v = <dynamic>[] and see if it blows up?

In my case i was having this problem using for each to desserialize a json response that was inside a list, when a change it to a "for in" it stopped to crash

Does anyone know if this bug was fixed?
Because I see this issue is closed.

The issue has been fixed in the Dart repository and is available in the lastest dev release. It will be in the the next beta and stable releases.

@johnniwinther I see you said this 3 weeks ago, so I made sure to update to new flutter & dart versions released after your comment.
I still experience this issue with these updates that were released sept 27th (about 2 weeks after your comment).
flutter: 3.13.6
dart: 3.1.3
environment:   sdk: '>=3.1.3 <4.0.0'

It seems you're on flutter stable meaning you don't get the dev releases of dart.

It seems you're on flutter stable meaning you don't get the dev releases of dart.

As was mentioned above, It will be in the the next beta and stable releases.. I do see that that doesn't specify which next stable release though...

It seems you're on flutter stable meaning you don't get the dev releases of dart.

As was mentioned above, It will be in the next beta and stable releases.. I do see that that doesn't specify which next stable release though...

@shawnAshton @mraleph  Please push the latest release on the stable branch. I am stuck nothing is working, my timelines are disturbed because of this error.

@kashiflab
Agree.
It is completely difficult to debug api calls like this,
I put a breakpoint and a few seconds later it crashes the application and debug.
When the stable release with the fix will be ready for use?

Stable release is in the final rounds of testing and preparations for release. You could switch to Flutter beta channel if you want the fix already today.

I bring good news guys, today I was looking for the problem and came across this topic.
The new version of flutter (3.16.0) released yesterday has fixed the problem.


Allow absolute URLs for examples

When writing examples with (for example) images, it would be best to use external URLs, since there's no easy way to use TikZ code or include an image file. Currently a URL to an image does not work.

Turns out this was a mistake on our part. They are supported, period.


Indenting list block of one type inside the other

If I create a OL of 2 items and change depth of 2nd item to 2, like this:
1. ITEM1
      2. ITEM2

If I change type of 2nd item to UL it looses it depth:
1. ITEM1

2. ITEM2

But now if I try to indent second item, it works
1. ITEM1

    2. ITEM2

While indenting its checked that block above is list, but its not checked if its list of same type. Also, please find gif attached. I can create a PR for the fix.

Thanks for the GIF -- very helpful. :)
I don't recall whether this was intentional, but block depth is always reset when setting type: https://github.com/facebook/draft-js/blob/master/src/model/transaction/setBlockTypeForContentState.js#L35
I'd be open to a PR to change this, since it seems reasonable to preserve depth on list items.

My results are a bit different than what the GIF shows. In the GIF, re-indenting the now ordered
 at the end causes it to become nested with the
 of the unordered list. However, when I try to recreate this, the lists do not nest. In fact, I can't created truly nested lists--from the perspective of the DOM--with Draft.js at all. Instead, all nesting seems to be achieved solely through the use of class names, e.g. .public-DraftStyleDefault-depth1, which in combination with adding margins via CSS can be made to appear nested.
While this is a clever approach that surely simplifies things a lot, it has the disadvantage of not being able to support ordered list numbering when the
s of the ordered list have children
s:

And the corresponding DOM:

Adding on to my previous comment, I see that the Facebook notes editor can supprt the following:

And the corresponding DOM:

The numbering changes are handled by applying list-style-type: none to the li elements and using :before psuedo-classes to add in the correct number via content. I was actually starting to implement this approach myself using a Custom Block Renderer when I came upon this.
Any chance this will get rolled into DraftJS core? It's pretty standard functionality. And while this doesn't achieve what the OP set out to do (nesting unordered & ordered lists together), that may be easy to build on top of this ordered list nesting functionality.

Is there anybody teach me how to do this issue, thx very much

finally I did it... mark


Install error

Hi,
looks like one of your dependencies has been removed from npm:
$ npm install azer/prova\#2b305cd
npm WARN deprecated<EMAIL_ADDRESS>use ua-parser-js instead
npm ERR! Linux 4.3.4-300.fc23.x86_64
npm ERR! argv "/home/tomekwi/.local/bin/node" "/home/tomekwi/.local/bin/npm" "install" "azer/prova#2b305cd"
npm ERR! node v5.4.1
npm ERR! npm  v3.3.12

npm ERR! No compatible version found<EMAIL_ADDRESS>npm ERR! Valid install targets:
npm ERR! 2.0.0, 0.0.1-security
(truncated)

By the way, 👏 for your decision at https://medium.com/@azerbike/i-ve-just-liberated-my-modules-9045c06be67c – but I admit it does seem a bit scary!

it should be fixed now


⚠ GitHub has degraded performance

In 665bc34, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 117 ms

Resolved: GitHub performance has improved in 4744965 after 1 hour.


Quick full screen

This PR is for investigating/adding the quick full screen functionality that @jordwalke requested in #666 (I guess I'm doing the devil's work here 😱 ).
Currently I have it working quickly by just expanding the window to take up all available space on the screen. It feels a little hacky to me, but it is fast :smile:
TODO:

[x] Enter quick full screen mode
[x] Leave quick full screen mode

[x] Restore the window to the previous size/location it was in before entering full screen

[x] Toggle the menu item appropriately
[x] Add accelerators for Linux
[x] Figure out if there is a more elegant way to do this

I tried the simple stuff first (i.e., using BrowserWindow.setFullScreen, but that still uses the OS X animations.
EDIT: After making some tweaks and sleeping on it, I think that my current implementation is probably the best way to go about it, given the constraints of Electron.

I hope that other electron apps can learn from your approach.

@jordwalke If it works well I can see about abstracting it into an Electron plugin so other apps can add the same functionality easily 👍

Here it is in action: https://gfycat.com/BitterScentedBarnowl

How about ^⌘F ?

(not sure it's better, but seems more standard?)

@rauchg ^⌘F is already reserved for normal full screen on OSX (which triggers the animation).

Greta work, can't wait for it to be merged. Worth noting that the hotkey for entering full screen in iTerm is also ⌘↵.

Detected a couple of issues after using it for longer time:

it remembers the window size after exiting, so when it's restarted it will show in full screen - maybe not a problem, but it's unable to exit full screen as a result
the event is not firing every time the hotkey is pressed, it fires after 2-3 attempts occasionally

@shime Yea, I was aware of the first issue, but I'm not totally sure what the best way to handle it is.
Because Hyper saves your terminal position and size when it exits (as part of another feature).
I was debating between:

Simply not saving the position/size when in quick full screen

This would restore the position/size you had before entering full screen on the next load

Saving the "is full screen" state to the Electron config (where the position and size are recorded)

This would bring you back to full screen on the next load, but you would be able to toggle it off using the hotkey

The former is easier since we just have to track the state in-memory, as opposed to the latter where it must be written out when the app exits.
cc @rauchg @jordwalke Any input on how this should work?

@shime Also, that second issue is interesting. I've only done cursory testing, so the next time I have a look at it I will see if I can reproduce and fix the issue.

Me? I'm not picky about which of those two solutions you go for as long as it's not buggy. I'm just happy to have full screen!

Did I miss why this was reverted?

@jordwalke There was an issue with it where sometimes you would get locked in quick full screen and not be able to get out. Usually it happened if you switched back and forth too fast.
I've been meaning to have a look at it, but haven't had any time to devote to it lately :slightly_frowning_face:

damn i was excited about this

Bump for this, it is a must have feature for productivity in my opinion.

Any plans about the Cmd+enter action to fullscreen on mac?

you can do that with the custom keymaps @asantos00

So it will not be added as default behaviour? I think it is default behaviour for all OSX apps :(


Throw correct SQLState and vendorCode with SQLException when out of index from ResultSetMetaData.getColumnName()

The current ResultSetMetaData can throw SQLException without SQL State. Please fill the correct SQLState and vendorCode in SQLException.
Please fix on classes under TODO fill correct SQL state comment:

[ ] ShardingResultSetMetaData
[ ] EncryptResultSetMetaData

hi, i want to do it

@zjcnb Thank you, I just assigned to you.

hi, I tested it,
state is S1002 (Invalid column number)
vendorCode is 0


Update type names to be prefixed with CT also

:tada: This PR is included in version 1.28.0-dev.26 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.28.1-dev.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Change slug for democratic-republic-of-the-congo in multiple smartanswers

Trello card: https://trello.com/c/PT4PdGxw
Related PRs:
#2785
#2786
#2787
#2788
#2791
Motivation
A next-node not found error was raised in errbit for check-uk-visa: https://errbit.publishing.service.gov.uk/apps/533c35ae0da1159384044f5f/problems/5807188e65786313bece1400
In check-uk-visa, if the user is from Democratic Republic of the Congo, and is studying in the uk for less than six months, a next-node not found exception is thrown. (https://www.gov.uk/check-uk-visa/y/democratic-republic-of-the-congo/study/six_months_or_less)
However, the other option, studying for over 6 months, renders correctly (https://www.gov.uk/check-uk-visa/y/democratic-republic-of-the-congo/study/longer_than_six_months).
The reason is that the path for six_months_or_less checks whether Democratic Republic of the Congo is in the list of DATV countries, but the longer_than_six_months does not.
The list of countries and their slugs (first question) is populated from Whitehall admin. Recently the slug for Democratic Republic of the Congo changed from democratic-republic-of-congo to democratic-republic-of-the-congo, but we had not updated our own country lists.
We need to change every occurrence of democratic-republic-of-congo in smartanswers to democratic-republic-of-the-congo.
The same issue is also occurring for users who select they are coming to work in the UK for less than six months.
UPDATE: A similar issues is also occurring for overseas-passports and marriage-abroad
Expected Changes:
check-uk-visa
URLs on GOV.UK
Before: https://cloud.githubusercontent.com/assets/5793815/19520624/c703f46e-9608-11e6-8f42-fca4c2d291a9.png
After: https://cloud.githubusercontent.com/assets/5793815/19520930/cc12c754-9609-11e6-9c30-7156f3e80e0d.png
overseas-passports
URLs on GOV.UK
Before: https://cloud.githubusercontent.com/assets/5793815/19554311/e6b912e2-96af-11e6-90a8-96fce78b866b.png
After: https://cloud.githubusercontent.com/assets/5793815/19554336/094a07da-96b0-11e6-8f02-22d2c6ca7c23.png
marriage-abroad
URLs on GOV.UK
Before: https://cloud.githubusercontent.com/assets/227328/19564538/c70c9d7a-96db-11e6-8908-1d70b1b7416d.png
After: https://cloud.githubusercontent.com/assets/227328/19564550/d392290c-96db-11e6-8a78-1ed487ae1549.png
help-if-you-are-arrested-abroad
URLs on GOV.UK
Before: https://cloud.githubusercontent.com/assets/5793815/19524693/7d50fbb0-9616-11e6-9002-7774a3c38cae.png
After: https://cloud.githubusercontent.com/assets/5793815/19524909/30a9e60e-9617-11e6-8710-1932f960aa9f.png
register-a-birth
URLs on GOV.UK
Before: https://cloud.githubusercontent.com/assets/227328/19555079/3a1af11e-96b3-11e6-83bd-b5be775155ca.png
After: https://cloud.githubusercontent.com/assets/227328/19555103/558a5c32-96b3-11e6-93e3-4eb8cfbb80c5.png
register-a-death
register-a-death
Before: https://cloud.githubusercontent.com/assets/227328/19555205/c1a50e26-96b3-11e6-85a2-106f3c6d9019.png
After: https://cloud.githubusercontent.com/assets/227328/19555211/ca9741fc-96b3-11e6-8119-a87a73a2bb49.png

LGTM 👍


feat(cli): implement dependency manager and show check list for init command

Description
This pull request changes the following:

isolate dependency checks in DependencyManager to evolve it later
show check list for init command

Related Issues

Closes #590

:tada: This PR is included in version 0.14.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[test/plugin] immigrate jdk http scenario

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[x] New feature provided

[ ] Improve performance

Related issues
#3583

Let's wait for the test result. Seems an easy plugin test :)

Codecov Report

Merging #4035 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4035   +/-   ##
=======================================
  Coverage   26.96%   26.96%
=======================================
  Files        1128     1128
  Lines       24872    24872
  Branches     3587     3587
=======================================
  Hits         6707     6707
  Misses      17568    17568
  Partials      597      597

Continue to review full report at Codecov.

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


How present data in dashboard

Hi,
Many thanks for this extension !!
I'm wondering what is the best presentation to integrate data in HA dashboard ?
Could you share some examples of dashboard integration please ?
Thank you !

I think it would better to read https://forum.hacf.fr/t/api-ecowatt/15451 (in :fr: ) and ask there. I'm using the default auto-generated dashboard of HA.


Export JSON Colored Printer

Hello, thanks a ton for this project--I was really happy to hear someone made JQ in pure-Go, and to fix so many other issues on top of that is just incredible (and even more so to expose it all as a library)! That said, I have a use case in which I'd like to be able to output JSON like the gojq CLI does--colors when writing to a TTY by default. It seems like this is all private to the cli package--would it be possible to make this a public package that one can use when importing the gojq module?

Unlikely to export these functions from this project. The encoder implementation is coupled with the command and hard to expose as library (for example, making the colors configurable is not straight forward). Instead, I recommend using the gojq.Marshal with the go-prettyprint library. The library automatically detect the TTY output. Here's sample code.
package main

import (
	"fmt"
	"log"
	"math"

	"github.com/fatih/color"
	"github.com/hokaccha/go-prettyjson"
	"github.com/itchyny/gojq"
)

// Same colors with gojq.
var format = &prettyjson.Formatter{
	KeyColor:    color.New(color.FgBlue, color.Bold),
	StringColor: color.New(color.FgGreen),
	BoolColor:   color.New(color.FgYellow),
	NumberColor: color.New(color.FgCyan),
	NullColor:   color.New(color.FgHiBlack),
	Indent:      2,
	Newline:     "\n",
}

func main() {
	// Use gojq.Marshal instead of json.Marshal to handle NaN and Infinities.
	bs, err := gojq.Marshal(map[string]interface{}{
		"x": 1,
		"y": math.NaN(),
		"z": []interface{}{nil, false, true, "abc", math.Inf(0)},
	})
	if err != nil {
		log.Fatal(err)
	}
	bs, err = format.Format(bs)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Printf("%s\n", bs)
}


panic: base32 decode failed: illegal base32 data at input byte 0

What version are you using?
tag horizon-v2.17.0-branch

What did you do?
run horizon with captive-core

What did you expect to see?
warn: base32 decode failed: illegal base32 data at input byte 0
INFO[2022-05-22T12:54:24.092+03:00] ...
What did you see instead?
INFO[2022-05-22T12:54:24.042+03:00] Herder: Quorum information for 40984476 : {"agree":23,"cost":34148728,"delayed":0,"disagree":0,"fail_at":6,"hash":"b68a69","lag_ms":2752,"ledger":40984476,"missing":0,"phase":"EXTERNALIZE"}  pid=25484 service=ingest subservice=stellar-core
INFO[2022-05-22T12:54:24.053+03:00] Ledger: Got consensus: [seq=40984478, prev=39484d, txs=313, ops=1000, sv: [ SIGNED@SatoshiPay_DE_Frankfurt txH: 11e595, ct:<PHONE_NUMBER>, upgrades: [ ] ]]  pid=25484 service=ingest subservice=stellar-core
INFO[2022-05-22T12:54:24.092+03:00] Tx: applying ledger 40984478 (txs:313, ops:1000, base_fee:5000)  pid=25484 service=ingest subservice=stellar-core
panic: base32 decode failed: illegal base32 data at input byte 0
goroutine 136 [running]:
github.com/stellar/go/xdr.MustAddress(...)
/opt/monorepo/xdr/account_id.go:84
github.com/stellar/go/services/horizon/internal/ingest.offerToXDR({{0xc04eea2b40, 0x38}, 0x352474ed, {0x1, 0xc05b5c9aa0, 0x0}, {0x0, 0x0, 0x0}, 0x15d195b77d2, ...})
/opt/monorepo/services/horizon/internal/ingest/verify.go:535 +0x153
github.com/stellar/go/services/horizon/internal/ingest.(*OrderBookStream).verifyAllOffers(0xc0000c4360, {0x12c3510, 0xc00007fb80}, {0xc0d9a04000, 0x2715f2d02715f2d, 0x1a5255})
/opt/monorepo/services/horizon/internal/ingest/orderbook.go:234 +0x6f8
github.com/stellar/go/services/horizon/internal/ingest.(*OrderBookStream).Update(0xc0000c4360, {0x12c3510, 0xc00007fb80})
/opt/monorepo/services/horizon/internal/ingest/orderbook.go:355 +0x2ff
github.com/stellar/go/services/horizon/internal/ingest.(*OrderBookStream).Run(0x0, {0x12c3510, 0xc00007fb80})
/opt/monorepo/services/horizon/internal/ingest/orderbook.go:387 +0x133
created by github.com/stellar/go/services/horizon/internal.(*App).Serve
/opt/monorepo/services/horizon/internal/app.go:93 +0x205

Can you share the exact sequence of command you ran?

Can you reproduce (e.g. through reingestion)? We didn't observe that error at SDF

I can not reproduce it.
It happened only once.

@dpblnt did you build the binary yourself? Did you make any changes to the code? I'm looking at paths (/opt/monorepo) and this binary was not built by SDF.

Yes it was built by me.
No I have not modified anything in the code.

Can you share the logs before the panic event?

I had no log-file set, this was captured from sdtout


Created the grid

Created the grid but it's not complete. I'm only merging to begin hosting the game using Pages.

Testing...

Created the grid but it's not complete. I'm only merging to begin hosting the game using Pages.

[!NOTE] The grid is not showing on the hosted page...


No peer_backup_ip in pandevice.ha.HighAvailability

There does not appear to be a way to set the HA1 backup links peer IP.  peer_ip is available for setting HA1 peer IP but there doesn't appear to be a peer_backup_ip in order to get HA1 redundancy configuration.  Seems like this would be required in order to utilize the pandevice.ha.HA1Backup.  Without setting peer, the HA1Backup doesn't do much good.  Or am I missing something?
https://pandevice.readthedocs.io/en/latest/module-ha.html#pandevice.ha.HighAvailability

Submitted a pull request for this.
https://github.com/PaloAltoNetworks/pandevice/pull/177

Fixed by #180, thanks @patrickdaj


4 convert jpg to png

feat: Add new tool : Convert JPG to PNG
test: Add e2e tests for all image tools
Add jimp libary
Refactor: Extract areColorsSimilar() util function
Refactor: allow test-id props for ColorSelector

Thank you. You can try to implement any other tool you want


extend SwaggerUi::new to add multiple path for the ui

actual
SwaggerUi::new("/swagger-ui").url("/api-docs/openapi.json", api_doc)

wanted
SwaggerUi::new(["/swagger-ui", "/api", "/doc"]).url("/api-docs/openapi.json", api_doc)

How should it work? Should it serve the SwaggerUI the same way as the first does? Or should they  be redirects to the first path? I guess the redirects would be sufficient.

the redirect is the best for this case

Okay, could be added in future. Some thought need is needed to to make it consistent across the frameworks. Also PR:s are welcome if interested.


Confirming "don't ask me again" with moving a file doesn't move the file

Issue Type
Bug
Description
I accidentally created a file in a folder I didn't want it in. When I did a drag/drop to move it, it confirmed with me. I checked the box to not ask again and hit "confirm" and the file didn't move. I repeated the steps multiple times with the same result (so it didn't remember not to ask me either) until I finally didn't check the box to not ask me again, clicked "confirm" and it moved fine.

Missed the above comment, sorry. Yes, every single time. Can do a vid if it helps, or run some stack trace if you point me where to look.


Not working after updating pycall to 1.5.2

I use snowflake connector through Pycall in Rails application.
Throwing queries asynchronously to Snowflake and then retrieving the status of the queries.
After updating pycall to 1.5.2, the screen froze when retrieving the status of the query.
However, looking at the logs, it appears that the retrieving of the query status completes successfully, and even the subsequent rendering of the view completes successfully.
Fixed Pycall version to 1.5.1 and did a bundle update and confirmed that it works correctly.
On the other hand, when I did a bundle update pycall, it didn't work properly.
No particular errors in the logs, even if they do not work properly.
What could be the cause?
Versions:

ruby: 3.1.2
rails: <IP_ADDRESS>

we also saw hanging after updating to 1.5.2 using pycall in the context of ruby-spacy gem. We had to revert as well. Haven't had time to dig into what was happening.
These were all single threaded workers that just froze up and stopped processing anything, but without any errors or hints as to why.


Add link to code of conduct, test Prow

/assign @lionelvillard

@evankanderson can you add the missing files to make Prow happy? AFAICT, only ./test/presubmit-tests.sh is needed.

Added.

/assign @lionelvillard

/lgtm
/approve


Create additional info blocks

You can make your resume less official by adding more entertaining info, like your hobbies and interest, or what do you like, some interactive content or joke, easter egg, etc.

🔥 🔥 🔥


Examples in main do not produce graphical output anymore

trying out the Examples (after having applied workaround reported before on the README file) it turns out that the graphical output is not available anymore. The previous master did produce a window with a graphical representation of the network. The current master only produces text output.

You are right, it's still there and it's just been put in comment here and in a def  because:

the graphical network representation requires the location coordinates in the metadata
if this information is not present the code crashes
with all the new network topology examples, there is no coordinates and I didn't want that to stop us
=> so we just need #TODO modify the def so that it accepts metadata with 0.0 coordinates

the graphical representation is certainly not something that should block the development. I would see it as a useful extension that helps to understand the examples we've put out. Or in other words: examples with coordinates should (optionally) produce graphical output. We may probably want to use a switch to indicate if graphical output is desired and describe our examples accordingly.


Fullscreen remains on different service

Expected Behavior
The app should not remove the top bar when a video is in full screen in one of the service.
Current Behavior
The top bar disappears when full screen content is playing in a service
Steps to Reproduce

Make a video in full screen mode, inside a service
Navigate in the other service

Context
I think the top bar should always remain visible or maybe only disappear in the service where content is in full screen, since it can be misleading and nor very intuitive to locate which tab has full screen element opened in order to be able to  close / reduce the Franz window
Environment

Franz Version used: 5.0.0-beta.16
Operating System and version: Ubuntu 17.10

Similar behavior with franz-setup-5.0.0-beta.17 for Windows:
Went full-screen in a video stream in FB Messenger, then Franz is stuck full-screen. I cannot resize it or move it elsewhere (multi-screen).
Minimize works fine.


lowercase dom instead of Dom

Going with dom instead of Dom which is current as of handsontable-0.32.0
preventing F2 from inserting 'q' into the input field

Thanks!


Remove support for Beta networkpolicies

This PR removes support for the old model of Beta NetworkPolicies that was in use before Kubernetes 1.8
It also removes support for the egress-enabled flag

Depends on https://github.com/aporeto-inc/trireme-lib/pull/485


GH-38022: [Java][FlightRPC] Expose app_metadata on FlightInfo and FlightEndpoint

Making necessary changes in Java to expose the newly added app_metadata.

Closes: #38022

You'll want to rebase again, also unskip Java here:
https://github.com/apache/arrow/blob/6f8f34bd344c5cdf158aacb0215c3409d2996c5e/dev/archery/archery/integration/runner.py#L615-L619

Ah, sorry - then you need to implement the integration test scenario for Java to do the same thing as Go/C++: add an AppMetadataFlightInfoEndpointScenario.java at https://github.com/apache/arrow/tree/main/java/flight/flight-integration-tests/src/main/java/org/apache/arrow/flight/integration/tests following appMetadataFlightInfoEndpointScenarioTester in Go (https://github.com/apache/arrow/blob/5b9f4b976118052824c5aeaa2220886839b6f4be/go/arrow/internal/flight_integration/scenario.go#L1241)

No worries! Started on this but the java setup is quite different.
For the Producer, should I extend IntegrationProducer or NoOpFlightProducer? I've seen instances of both (and then there's FlightSqlScenarioProducer which implements the whole thing).

Start with NoOpFlightProducer here. IntegrationProducer should probably be renamed since it's specific to the roundtrip integration tests

Something interesting I noticed from the Go test... the Ticket on the FlightEndpoint can be null?

the Ticket on the FlightEndpoint can be null

It's Protobuf, so any field can be omitted, but this is the same as having a ticket whose value is an empty string. I don't think we need to expose this separately in other languages.


I made a CLI with notifications and filtering(!)

Hi! Apologies in advance if I shouldn't be creating an issue for this (feel free to close it) but I made a command-line interface for checking the website every x seconds, filtering by zip code and vaccine manufacturer, and sending notifications when matches are found. [I'm pretty unfamiliar with Windows, so I have no idea if it'll work there, and I think notifications definitely won't.]
Here's the repo, and it's installable using Homebrew with brew install jltml/tap/vaccine-spotter or as a Ruby gem with gem install vaccine-spotter.

If anyone actually ends up using it, pull requests are more than welcome!

This is awesome! I also made an email bot here: https://github.com/chitalian/covid-vax-emailer
If you want to merge the two - feel free 👍 :)

Thanks so much! And that's super awesome too — I'll totally take a look at merging them if it seems feasible! I'm pretty new to all this but fingers crossed :)
And great idea; I'll absolutely add an alarm sound option too — a couple weeks ago I was using a Python script someone made for finding CVS appointments that just beeped a bunch when one was found, and it worked wonderfully, especially at 2am… so that's definitely a great idea!


No icon in activity bar

Saw the icon when I first installed it. Tried to select a folder, but it seemed to get stuck. Since then no icon / functionality. Tried to disable / uninstall a few times, but still no icon. Restarted VSC too.
VSC 1.40.1
Ubuntu 18.04

Thank you! I'll check this soon.

I have the same issue, with extension info:
Name: Compare Folders
Id: moshfeu.compare-folders
Description: Compare folders by contents, present the files that have differences and display the diffs side by side
Version: 0.2.0
Publisher: MoshFeu
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=moshfeu.compare-folders

@JonathanTroyer this is a great feedback. I'll check this.

First. The activity bar is shows up only when there is only one workspace.
https://github.com/moshfeu/vscode-compare-folders/blob/80bb718fb6202960ecd225fc01567aa0345c2a9b/package.json#L105
I wasn't aware the need of multiple workspaces 😕
First thing, I'll update the README so people will know what to expect.
Then, I'll figure it out how to support multiple workspaces.

I just published a new version (v0.3.0) that supports multiple workspaces (so the icon will shows up).
Can you verify please?
Thank you for your help!

So, I get the icon now, but I cannot compare folders.
To reproduce,

Create a folder (I named it folder) and add content (I added test.txt with contents "1").
Duplicate that folder, rename it (I named it folder2) and change the content (changed test.txt to contain "2")
Either
3.1. Open the first folder directly
3.2. Open a blank workspace, add the folder to it, and save
Navigate to the compare pane (which is now possible if you do 3.2., thanks!!)
Attempt to compare a folder. Whenever I open a folder (folder2), nothing happens

Current extension information:
Name: Compare Folders
Id: moshfeu.compare-folders
Description: Compare folders by contents, present the files that have differences and display the diffs side by side
Version: 0.3.0
Publisher: MoshFeu
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=moshfeu.compare-folders

I'm on Windows 8.1 Pro, VSCode 1.41.0 if that helps.

Thanks @JonathanTroyer. I tried to reproduce, here is the result. What am I missing?

Hi again @JonathanTroyer and @wuffo.
Just published v0.4.0 can you check if it's still happening?
Thanks!

I'm closing it for now.
If anyone get this problem too, I'll be very happy if you could open a new issue


Improve usability and documentation

After getting some feedback from our first "customers" there are serveral things that need to be improved:

[ ] Give the user a feedback of how good/bad the calibration is.
Suggestions/Ideas:

[ ] Run the python evaluation step after the optimization and optionally plot the results
[ ] Write a summary pdf at the end of the calibration with the most important plots:

[ ] initial time alignment
[ ] initial 3D trajectory, after dual quaternion calibration, after optimization
[ ] RMSE plot after dual quaternion, after optimization

[ ] Provide a separate script to plot the 3D trajectory before and after some given calibration
[ ] Add a short wiki description that describes the 3 steps (time alignment, coarse initial calibration, optimization) and what the plots mean.

What are the good solutions for plotting the trajectories? I'd love to have something nice here.

We already have functions that plot the 3D trajectories, but TBH the 3D plotting tools of matplotlib are not very nice to look at. Actually I just recently started using mayavi/mlab in python and it is very easy to use and a lot nicer and faster (you can still move the 3D plot even with 1000s of elements in it). Here is sth i did there:

I've already implemented the plotting of a pose graph with coordinate frames at the nodes, so I could transfer this.

Ah, yes, I totally forgot about these 3d plots - I think I could hardly use them on Trusty. Sorry. mayavi/mlab looks very interesting.

@mfehr if you can easily transfer them, it of course would be awesome :)


Create Aws container EC2 launch

AWS containerized launch vessel

Let's ship it to forking orbit!

Ship assets

Gogogo

Yup


missing copyright statement in select2-bootstrap.css

Please add copyright statement to select2-bootstrap.css.
Thanks.

@kevin-brown I guess that explicitly mentioning the license as Select2 itself does in its readme should suffice. If so, could you please check the current select2-bootstrap-theme license and let me know if this is okay like it is or if I should change something.

After a second, more thorough read: @onlyjob could you clarify why you need the copyright in select2-bootstrap.css? Not too familiar with the legal things. Would a link to the license (like Select2 itself does) be okay?

I hope this is resolved in v0.1.0-beta.5 with adding the Copyright and License information to the README.

why you need the copyright in select2-bootstrap.css

Best practice. This CSS file is often (re-)distributed by other projects. Copyright statement is important to document ownership (i.e. who grants the license).
Do not mistake _copyright statement (e.g. Copyright 2015-2016 Florian Kissling<EMAIL_ADDRESS>and grant of license.

Would a link to the license (like Select2 itself does) be okay?

No, they are doing it wrong. First of all they call license "MIT" which is ambiguous. FSF recommends calling it "Expat".
Secondly they do not use canonical URL of the license: https://opensource.org/licenses/MIT
Finally text of license is short and it is arguably better to include full text of the license accompanied by copyright statement.
Committed LICENSE file documents copyright and license perfectly. However because CSS file is often bundled with other software I recommend to include copyright statement to CSS file as well.
Thank you.
See more:

http://www.producingoss.com/en/license-quickstart.html

Thanks for the info @onlyjob! Maybe @kevin-brown can chime in and we can fix this for both this and the main Select2 repository?

This CSS file is often (re-)distributed by other projects. Copyright statement is important to document ownership (i.e. who grants the license).

I'm going to agree that not including a banner in the distributed CSS files was an oversight in the main project which still needs to be fixed. I'm not sure about the need for the copyright statement (though I don't see anything wrong with not including it), as that should ideally be covered in the linked license.

First of all they call license "MIT" which is ambiguous. FSF recommends calling it "Expat".

This ambiguity should be resolved by following the link to the license and verifying the text is accurate and complete. As much as I appreciate the FSF attempting to resolve ambiguity by calling it "Expat", this can be resolved by looking at the license text (which people should be doing anyway).

Secondly they do not use canonical URL of the license: https://opensource.org/licenses/MIT

That's the license template, not the actual license file.

That's the license template, not the actual license file.

This is canonical text of the license, contrary to maybe-customised license file which may not be included. Link to authoritative OSI approved text of the license + copyright statement is the best for situations when including full text of the license is not desirable. Also It may be useful to mention URL of the project (backlink).


ld: warning: object file (...) was built for newer macOS version (12.0) than being linked (11.3)

I'm getting the following errors when building my go program.
# command-line-arguments
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000012.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000013.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000014.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000015.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000016.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000017.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000018.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000019.o) was built for newer macOS version (12.0) than being linked (11.3)
ld: warning: object file (/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-link-1414785648/000020.o) was built for newer macOS version (12.0) than being linked (11.3)

macOS version
$ sw_vers
ProductName:	macOS
ProductVersion:	12.5
BuildVersion:	21G5027d

Go env
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users/plorenzo/dev/gorkspace/bin"
GOCACHE="/Users/plorenzo/Library/Caches/go-build"
GOENV="/Users/plorenzo/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/plorenzo/dev/gorkspace/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/plorenzo/dev/gorkspace"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.18.2/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.18.2/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/plorenzo/dev/gorkspace/scr/gitlab.com/plorenzo/test/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/yy/16g_7s717l358cwmrh87jc4w0000gn/T/go-build3374803040=/tmp/go-build -gno-record-gcc-switches -fno-common"

Minimun reproducible example
package main

import (
	"database/sql"

	_ "github.com/mattn/go-sqlite3"
)

func main() {
	sql.Open("sqlite3", "db")
}

It looks like you installed Go via Homebrew. As per this, you may need to update CommandLineTools (i.e., XCode).

Thank you for the pointer. Indeed reinstalling command line tools did the trick.
sudo rm -rf /Library/Developer/CommandLineTools
sudo xcode-select --install


fix(meta & client): fix create op tx failed bug

What this PR does / why we need it:
fix create file tx op bug.
fix #2322
Special notes for your reviewer:
Release note:

LGTM


Closing/Opening seems to be swapped

Home Assistant release with the issue:
0.97.0
Component/platform:
https://www.home-assistant.io/components/homekit_controller/
Description of problem:
The state of cover is going from Closed → Closing → Open → Opening → Closed

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
  - entity: cover.velux_window_entre

Wow, good spot @tleegaard. I've double checked in the HAP spec and it does look like this is inverted. This line here:
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/homekit_controller/cover.py#L36
Should be something like (off the top of my head):
CURRENT_WINDOW_STATE_MAP = {0: STATE_CLOSING, 1: STATE_OPENING, 2: STATE_STOPPED}

Could you possibly try making that change?
Even better, if it works would you mind raising a pull request again. If you do you'll need to update one of the tests in two places:

https://github.com/home-assistant/home-assistant/blob/dev/tests/components/homekit_controller/test_cover.py#L96
https://github.com/home-assistant/home-assistant/blob/dev/tests/components/homekit_controller/test_cover.py#L100

(Just need to swap opening and closing round)
(I don't have any HomeKit window covers so I have no way to test changes like this myself)

I will look into it :-)


v0.59.0 Failed to install provider

11:16 $ terraform init

Initializing the backend...

Initializing provider plugins...
- Finding build-trust/ockam versions matching "0.59.0"...
- Installing build-trust/ockam v0.59.0...
╷
│ Error: Failed to install provider
│
│ Error while installing build-trust/ockam v0.59.0: checksum list has unexpected SHA-256 hash c0dfb1fba5d6b956105f17f4c05bc1b1fdb1670633d58f27cd89158bd72ae952 (expected c4c471bfd25eff21448fb2f702e33d80e69caab1a35e886fa24654e3f4bf9db2)

Seeing this as well, but not sure where that other hash is coming from because it's in the file, correctly. 0.1.6 works just fine.

Looks like the Action ran twice. I guess it mutated the shasum file but that isn't allowed for the registry. My recommendation is to release a v0.59.0-1 or something.

Thank you both.
Terraform registry didn't pull the assets created by the second run.
There is now https://registry.terraform.io/providers/build-trust/ockam/0.59.1 to get terraform registry to pull a new release. Even the the ockam binary it provisions is actually 0.59.0.
Let me know in case this doesn't fix the problem.

That worked @mrinalwadhwa. This can be closed. Thanks!

Works for me as well. Nice!


Discrepancy in Image ID Alignment Between M3IT and VideoChat2IT

Could you please provide a script or JSON file of the ID map from M3IT to VideoChat2IT? Matching different files can be quite challenging. For example, coco  llava  minigpt4  paragraph_captioning  textcaps (VideoChat2IT/caption) v.s. coco  coco-cn  flickr8k-cn  image_paragraph_captioning  msrvtt  textcap (M3IT/captioning). In addition, the image IDs do not completely match; for instance, COCO images in VideoChat2IT have an additional directory compared to those in M3IT. I believe it would be beneficial to fully opensource this.

You mean manually check the file for each split? That's fine, but solely changing file names is confusing and adds unnecessary workload without any benefits.

Hi, I didn't find image/caption/minigpt4 from M3IT, how can I obtain these images?

https://drive.google.com/file/d/1nJXhoEcy3KTExr17I7BXqY5Y9Lx_-n-9/view


Map

[ ] make provider size smaller at large zoom
[ ] max RP size smaller
[ ] max one point on hover, avoid selection of one RP + one provider point at the same time.

Could be cool to bring up the closest provider popup when selecting a particular RP!

Slight changes to RPs:
Before:

After:


re: missing whitespace in root node

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions
https://github.com/syntax-tree/hast-util-to-html/issues/33
Link to runnable example
No response
Steps to reproduce
https://github.com/syntax-tree/hast-util-to-html/issues/33

This is required by the HTML standard, which we follow.

so my input html is invalid?
<!doctype html>

    <title>test</title>

test

is valid html per https://validator.w3.org/nu/#textarea
the unified toolchain mangles this to
<!doctype html>
    <title>test</title>

test

the unified toolchain

i dont know where exactly the bug is.
maybe in hast-util-to-html, maybe somewhere else

That behavior is not always useful.

its certainly useful for writing lossless transformers, which produce minimal diffs
maybe this could cause trouble in some tree consumers like hast-util-select
which expect "no whitespace in the root node"?

If you want pretty HTML, use rehype-format!

no, i want to preserve the original whitespace
Expected behavior
https://github.com/syntax-tree/hast-util-to-html/issues/33
Actual behavior
https://github.com/syntax-tree/hast-util-to-html/issues/33
Affected runtime and version
https://github.com/syntax-tree/hast-util-to-html/issues/33
Affected package manager and version
No response
Affected OS and version
No response
Build and bundle tools
No response

You can still comment on closed issues.
This isn’t about valid or invalid.
This is about the HTML standard prescribing how to parse HTML.
We follow how the HTML standards prescribes how to parse HTML.
We use an AST (abstract). Not a CST (concrete). We don’t generate the exact input document. We generate an equivalent document.

sorry .__.
please feel free to merge the issues

We follow how the HTML standards prescribes how to parse HTML.

rendering html != transforming html

We use an AST (abstract). Not a CST (concrete). We don’t generate the exact input document. We generate an equivalent document.

the parse tree is 99% concrete already.
it contains all the whitespce, except around the root node.
making it 100% concrete is a small fix with a large gain (at least for my use case).
at least this could be made optional
the resulting parser may be less "strict" and more "loose" than a perfectly spec-compliant parser ...
but so what? whats the punishment of breaking the spec?
except that html rendering is 0.1% slower, to skip some whitespace nodes

please feel free to merge the issues

That’s not possible through the GH UI, unfortunately!

making it 100% concrete is a small fix

It’s extremely complex. A concrete tree would house all original sources of character references. It would house whitespace inside tags. It would house the casing of attributes. It would include information on whether double or single quotes are used. It would information in doctypes, processing instructions. It would contain casing of tag names. It would be months of work to create.

at least this could be made optional

It’s a ton of work, probably a full year to implement a new parser and design the data structure. If you are interested in paying me to make it reality I would likely quote you $200k.

the resulting parser may be less "strict" and more "loose" than a perfectly spec-compliant parser ...

This project adheres to the HTML spec.

whats the punishment of breaking the spec?

In this particular case of only the whitespace? It would indeed probably not be terribly complex to implement, but parse5 doesn’t support it either. Feel free to investigate.
But we have had similar conversations about this: hast is for the HTML spec. If you want XML, use xast. If you want to use a different project that does not adhere to the spec, feel free to make it yourself.

thanks.

feel free to make it yourself.

yes. i would use a concrete parser like

tree-sitter-html with treesitter-to-unist
lezer-parser-html to avoid WASM

with a different transformer (different node names, different data structure)
which is cheap, because my transformer is simple


[5.8] Support macros of grammars

In Blueprint method_exists is used. It cant find magic methods declared via macros. So if we will use "try catch", we can get around this. In most cases exception will not be thrown.

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!


ieeeFlags unittest: the optimizer might remove operations that the te…

…st is depending upon
The optimizer removes all traces of a because it is const-folded to nan. That means no side effects appear.
This test is failing here: https://dev.azure.com/dlanguage/dmd/_build/results?buildId=3982&view=logs&jobId=995050be-48e9-58a0-3553-d9a547fe6c20&taskId=3bcde2a0-396a-5c5e-c7d5-d0568bb0ee4b&lineStart=4275&lineEnd=4276&colStart=1&colEnd=1
Not sure why it doesn't appear on other platforms, but it triggers when using dmd built against the MS C runtime.

Just noticed that there are two more unittests that almost do the same. Amended similar fixes, but they could also just be deleted (they don't look like useful examples with these workarounds).

Ping! Any reason for holdup?


Bugs when render multibyte chars

With creators update, a new issue surfaced with rendering PowerLine multibyte chars.
Powerline works with customized fonts specifically created for prompts, with symbols for source control, etc. For example, they have the reverse separator, 0xE0B2, that is most often rendered as a backwards triangle.
The following is a sample of the text that does not work well any more.
here is '0xE0B2'
\esc[s\esc[79G\esc[45m\esc[90m\esc[100m\esc[30m0:00:00.0190\esc[100m\esc[37m\esc[47m\esc[30m08:59:47\esc[37m\esc[45m
https://github.com/powerline/fonts/tree/master/DroidSansMonoSlashed is a powerline font that can be used for testing.
Images displaying the issue can be found here: https://github.com/Jaykul/PowerLine/issues/13

Cheers!

It's not just with multibyte chars, I think this is related to an issue I found out about how the console renders chunks of texts with different colors. It regularly ends up being off by one characters when computing the starting column of a new chunk depending on characters and DPI scaling : https://github.com/Microsoft/BashOnWindows/issues/1907

Yep, I can confirm that this is the same "ambiguous width characters" issue that @miniksa described in #1907. I'm gonna close this issue as a dupe of that one, to hopefully condense the discussion into one thread.


feat: add fmt check.

Codecov Report

Merging #46 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #46   +/-   ##
======================================
  Coverage    83.2%   83.2%
======================================
  Files           1       1
  Lines         125     125
======================================
  Hits          104     104
  Misses         20      20
  Partials        1       1

Continue to review full report at Codecov.

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


Roll Flutter Engine from 2d55e29dfa79 to 1c17ee27a6e7 (6 revisions)

https://github.com/flutter/engine/compare/2d55e29dfa79...1c17ee27a6e7
2023-10-16<EMAIL_ADDRESS>Roll Dart SDK from 12f6559bd6ed to f3e1cd38e8b0 (1 revision) (flutter/engine#46977)
2023-10-16<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 6E-cSq679DjzBMcqY... to Y9mDBoH4BSC6pWFXV... (flutter/engine#46974)
2023-10-16<EMAIL_ADDRESS>Roll Skia from 8919fecf15c1 to 751358929d1f (4 revisions) (flutter/engine#46972)
2023-10-16<EMAIL_ADDRESS>[Impeller] GPUTracer for Metal. (flutter/engine#46846)
2023-10-16<EMAIL_ADDRESS>Fix Platform.script for flutter_tester (flutter/engine#46911)
2023-10-16<EMAIL_ADDRESS>Roll Skia from 85c8dca08cbe to 8919fecf15c1 (1 revision) (flutter/engine#46966)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from 6E-cSq679Djz to Y9mDBoH4BSC6
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jsimmons@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add loading and failure states for balances on home

Either one state for both Bitcoin and Stacks-related balances or two separate ones
Two separate states seem perhaps best so that one query that's taking a long time doesn't hold up the display of the other
For failure state, do we tell the user to refresh or click a link to try fetching data again?

I would also prefer two seperate states for Bitcoin and Stacks Balance. Issues loading bitcoins are more prevalent compared to Stacks.


Fix broken trees in PXE

Broken by https://github.com/ManageIQ/manageiq/pull/12320 .
Closes https://github.com/ManageIQ/manageiq/issues/12414
Closes https://github.com/ManageIQ/manageiq/issues/12401
@skateman @martinpovolny please review.
@miq-bot add_label ui, bug

@ZitaNemeckova is it possible to add specs for this bug?

Specs added.


Add --no-upgrade flag to pip-compile

This should prevent executing version updates, but shoud add/remove dependencies as they have (dis)appeared in *.in files. This would be useful to add a new dependency to in *.in file, and have pip-compile only resolve that new package and add it to the *.txt file.
There are a lot of edge cases though, so getting the semantics right here is non-trivial.

I'm glad this was tackled, it greatly improves interoperability with pip-review because it separates updating from pinning.


Question about figure 5

Hi, I'm suprised about your paper, and i want to try the visualization. could you please give me the code about the figure 5? thanks a lot.

Thanks for your attention. Here's the raw data and code. Since it's been so long, I can't guarantee they'll work well. But I hope it helps.


Add class "photon"

Description of the issue
From the oeo-dev-meeting on energy subclasses 2021-02-02:
We need a class photon as subclass of portion of matter. photon should be used as energy carrying material entity for radiative energy.
Ideas of solution
If you already have ideas for the solution describe them here
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

photon: A photon is a portion of matter that is a light particle.

Sounds good.
And add the relation photon has quality some radiative energy (corresponding to air and wind energy)?!
Do you want to implement? @0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q


Minify HTML and CSS

HTML compression is a combination of this template for compression and me copying the layout file out of the gem, which is best practice.
JS seems hard, so I didn't do it.

seems fine. how was it tested?


Hello, everyone there?

I can't use Discord due to our government's limit.
I'm glad to hear that. I want to ask some questions.

Can I use Forge and Fabric mods together with this loader quite now?
Is this loader based on Forge, Fabric or others? I think it's Forge.

For your first question, as of right now, the loader has some issues with mixins causing most Fabric mods to not work and not crashing Cloud. After the issue is resolved, it might work, but I'll have to see.
And for the second, you are correct in that it is internally based on Forge, and there's still quite a few files (like README.md) that need to be updated for Cloud.


WIP: Add goproxy to accelerate ci

As title.

We should download https://github.com/go-swagger/go-swagger/releases/download/v0.19.0/swagger_linux_amd64 directly but not go get, it spent too many time on build.

Codecov Report

Merging #7744 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7744      +/-   ##
==========================================
+ Coverage   41.27%   41.28%   +<.01%
==========================================
  Files         472      472
  Lines       63845    63845
==========================================
+ Hits        26353    26359       +6
+ Misses      34054    34047       -7
- Partials     3438     3439       +1

Impacted Files
Coverage Δ

modules/process/manager.go
76.81% <0%> (-4.35%)
:arrow_down:

routers/repo/view.go
43.25% <0%> (+1.01%)
:arrow_up:

modules/log/log.go
72.07% <0%> (+2.7%)
:arrow_up:

models/unit.go
67.56% <0%> (+5.4%)
:arrow_up:

Continue to review full report at Codecov.

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

I think this PR is superseded by https://github.com/go-gitea/gitea/pull/7745 and following PR
@lunny sorry, I didn't see your PR before.

@sapk  so this could be closed.


Stim detector of opposite parity

Is it possible to declare detectors of opposite parity to a set of measurements.  E.g. a detection event when some measurement is 0 instead of 1.  I would imagine a command like
DETECTOR rec[-1] 1 or
OPPOSITE_DETECTOR rec[-1]?
Of course, this could be circumvented by adding a noiseless 1-measurement and comparing against that, but its less elegant.  Seems peculiar that one can add a trivially off detector, but not a trivially on detector.  Wondering if this functionality already exists somehow or could be easily added.

Detectors already handle the "parity is expected to be odd" case automatically. The expected parity is computed automatically in order to make comparisons. For example this produces a 10% detection fraction instead of a 90% detection fraction:
X 0
X_ERROR(0.1) 0
M 0
DETECTOR rec[-1]


Baked GI Issues

https://www.youtube.com/watch?v=LTunq-4elgc
(texGenFlags[texGenIndex] & TexGenFlags.Discarded) == TexGenFlags.Discarded

Lightmaps completely break when you modify some geometry. Also modifying geometry is really slow. Everything in the scene is static using baked lighting for everything.
Also I undo the deletion of the floor brush and it takes a good 5 seconds on a very sparse scene.

I'll look into it

+1

not sure yet why the baking stops working, but slowdown is simply because unity is rebuilding everything every time the csg meshes change ... turning off auto lighting + modifying things isn't slow
I'm not entirely sure what, if anything, I could do about that

I do not have auto-build lighting enabled in the example video. (visible on the right at the start)

I think is partially related to #116 and I think the other part is just .. Unity :-/
The thing is Unity will rebuild the lightmap for the generated mesh once it changes .. so if you modify a brush, the mesh that's generated for that brush will be changed & Unity will rebuild the lightmaps (hopefully only a subset) and I have no control over that.
The garbage you see when modifying the brushes is the existing lightmap being used for a modified mesh that has different uv coordinates for the next lightmap being created by Unity.
Maybe I can just stop that lightmap from being shown somehow, but other than that there's nothing I can do about that.
At first I thought the baking stopped working but in that video when you look in the bottom right corner you see that the lighting is being recalculated, and it's just .. slow
(Unity 5.5+ should be a lot faster by the way)
The Undo being slow is a different issue entirely (and there's another ticket for that #29 )
The slowness of modifying geometry while baking is partially my fault (which I will fix today), and partially the lighting being recalculated in the background.

Some of these issues should be fixed in 1.328!
You can download this version at the update site https://prenominal.nl/updates/

I made some more changes in 1.329 that should remove the garbage that you see when modifying brushes.
It does need to rebuild the lighting when you change something, but there's no way around that unfortunately. Let me know what you think.
As usual:
You can download this version at the update site https://prenominal.nl/updates/
I will close this ticket once it's available from the asset store (tm)


Troubles with genome-attributes-viewer in the new Docker image

@vinisalazar wrote:

I rebuilt the Docker image today after the updates. I still can successfully run the benchmarks from the Docker container, but I was having trouble with the genome-attributes-viewer app, it started, I ran a couple of queries, and then it crashed with this traceback:

 root@d8798c592652:/fastsubtrees/docker# ./start-example-app
Dash is running on http://<IP_ADDRESS>:8050/

 * Serving Flask app 'start' (lazy loading)
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: on
./start-example-app: line 15:   823 Killed                  ./start.py
root@d8798c592652:/fastsubtrees/docker# Traceback (most recent call last):
  File "/fastsubtrees/genomes-attributes-viewer/start.py", line 171, in <module>
    app.run_server(debug=True, host='<IP_ADDRESS>')
  File "/usr/local/lib/python3.10/dist-packages/dash/dash.py", line 2033, in run_server
    self.server.run(host=host, port=port, debug=debug, **flask_run_options)
  File "/usr/local/lib/python3.10/dist-packages/flask/app.py", line 920, in run
    run_simple(t.cast(str, host), port, self, **options)
  File "/usr/local/lib/python3.10/dist-packages/werkzeug/serving.py", line 1000, in run_simple
    _rwr(
  File "/usr/local/lib/python3.10/dist-packages/werkzeug/_reloader.py", line 418, in run_with_reloader
    ensure_echo_on()
  File "/usr/local/lib/python3.10/dist-packages/werkzeug/_reloader.py", line 398, in ensure_echo_on
    termios.tcsetattr(sys.stdin, termios.TCSANOW, attributes)
termios.error: (5, 'Input/output error')

I could not yet reproduce this.
I fixed the version of the Werkzeug package, because I had other issues with newer versions.
I will try again to change its version, to see if using a newer version helps.

Oh no... it was a problem of the automatisms... I never noticed the issue in my system, because the files existed already and were not re-created.
I fixed it in the latest commit eef44fd2b9559c83eb67eb36d634d310a8fc9079 and uploaded to Pypi a version 1.3 of the genome_attribute_viewer package with the fix.

Can confirm, it works now for me!


Cloud Functions for Firebase - pricing page link broken

https://codelabs.developers.google.com/codelabs/firebase-cloud-functions/index.html?index=..%2F..%2Fio2017#9
"Pricing page" link at the bottom of the page is broken

Hi @w-a-n this link seems to be working, so closing.


[Skills workflow] Removing skill from an empty bot shows error for missing LUIS keys

Describe the bug
Removing skill from an empty bot shows error for missing LUIS keys
Version
Version:  2.0.0-nightly.260493.e017820
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
To Reproduce
Steps to reproduce the behavior:

Add PVA Skill to an empty bot in Composer
Do not opt in to use orchestrator
Remove Skill
Observe the error

Expected behavior
Composer should not require LUIS Key/Region for an empty bot
Screenshots

Additional context

I found that is not a skill specific issue and it can't be consistently reproduced, I tried to add a trigger and remove it, the trigger name also remains in the lu file.
In an empty bot, I added a trigger "aaaaaaa" and a skill "Skill Test Marina", then I remove "Skill Test Marina" and "aaaaaaa", Here is what I see from the lu editor:

@boydc2014 @lei9444 @zhixzhan


docker - tomcat accessible but vcf-miner cannot be found

Hi,
I am using the docker version on ubuntu 16.04 64 bit.
I can access the tomcat server via a webbrowser but cannot access vcf-miner, no matter how long I wait.
How can I diagnose if the vcf-miner app is actually starting within the docker container correctly ?
I get a tomcat webpage
http://myip:7777/
I get a tomcat 404 vcf-miner
http://myip:7777/vcf-miner/
HTTP Status 404 - /vcf-miner/
Any ideas ?
Thanks Colin

Same here. Can reproduce on Arch, Ubuntu, and MacOS.

Can you verify you have a mongod instance running in the background?

I am running the docker image from your docker hub repo. Mongo is running in the container and the mongoshell is accessible. Tomcat is running and the root address yields the tomcat webpage. Can you confirm this image should work? I rebuilt the image and encounter the same problem. Could be a problem on my side...

Ditto. I'd say this is a ports problem, as I now have alternative software running on port 8080 (this wasn't there before however).
Can you recommend an alternative port for this command and explain where to make the changes in the docker image (I assume in interactive mode?) ?
Command:
docker run -e NO_LDAP=1 -d -p 8888:8000 stevenhart/vcf-miner:latest /home/start.sh
Thanks

Apologies for taking so long, but I am looking into it.

OK. I've updated the README.  You should now be able to start it without any problems.

I just rebuilt the docker image and the problem still persists. Using the docker image i can access the root page but not vcf-miner (404). Same issue with the provided docker image from docker hub.

can you let me know what commit you are running off of?

OK. I have updated the readme and build specifications.  This should resolve your problem.  Can you comment when you try it out?


MRG: Add SourceSimulator class

Squashed version of #6212. Should be rebase+merged (or standard merged) instead of squash+merged so that commit authors retain credit/blame.

Great team effort, thanks @makkostya @ngayraud @sdeslauriers @ikojcic @agramfort !

Awesome! Thx @larsoner


Implement cupy.lexsort.

This PR implements cupy.lexsort following the implementations of cupy.sort in #1817 and cupy.argsort in #2057 .

Should we review this PR after #1817?

Yes.

OK, I will wait it.

This PR is reposted to cupy/cupy#132 after CuPy separation from Chainer, so I close this.


TravisCI test runs unstable

Some tests failed on the first run with go 1.5 but then passed using travis re-run button.
Restarted #35 and #37

The last builds were fine. Closing this now.


Use old_relation for renaming in default materializations

resolves #2161
Description
change renamed relation to make sure dbt generates the right alter ... rename statement is the materilization type has changed from previous model run
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

I checked locally and the only failed test is test/integration/004_simple_snapshot_test/test_simple_snapshot.py, which I doubt is influenced by my changes on table and view materializations.
So I tried a git checkout of the last commit previous to my changes, and this test is giving me the same error.

@danielefrigo When you get a chance, could you resolve conflicts / update the changelog? Then this is good to go!

sorry for the delay, I've been on vacation with no chance to work on it.
is it ok now?


Add support for Angular

I'm loving the work that you and the community have done on knip! Although I do have one request. Would it be possible to add support for Angular projects? E.g. When running the tool I'm getting all of my class variables flagged under the Unused exported class members section of the report and when running with the performance flag I'm getting back functions that don't exist in my project.

Not sure what exactly support for Angular projects means? If you have an example I can have a look.
The --performance flag shows the time spent internally in Knip functions, so indeed isn't about the performance of your code :)

Closing this due to inactivity.

Knip would need to understand Angular component templates. Angular components are class-based. Component class methods are used almost exclusively by Angular's custom string-based component templates so most Angular component event handlers are marked as unused by Knip.

Same here. I come from ts-prune https://github.com/nadeesha/ts-prune and maybe I'm missing something in my config file

Maybe someone can point me to an example rep(r)o? I would need to see examples, etc. Please help me help you :)

Heya guys!
Sorry for hijacking the thread, but i'm very interested on this topic as well.
What I've found out in the past few days:

in @ice-blaze's repo the entry should be changed to src/main.ts (this is one of the default values in knip config)

with this approach all the files will be checked (main.ts is the main entrypoint for an angular app)

some angular dependencies will be listed as unused dependencies/dev dependencies

i've cleaned up my project, so not all are listed here, but please take a look

"ignoreDependencies": [
    "@angular-devkit/build-angular",
    "@angular/compiler-cli",
    "jest-preset-angular",
    "ts-node"
]

ts-node is used internally, so it should be there. I bet there are some other fake warnings in the given example, but please note

angular/forms can be used directly in module files (or in standalone components)
angular/compiler-cli cannot be used directly, it must be ignored manually
angular-devkit/build-angular cannot be used directly, it must be ignored manually
angular/platform-browser-dynamic is used in the main.ts file

Regarding the static code analysis in angular, AFAIK ng lint is the best way to do that. Unfortunately angular uses declarative templating, so knip should understand that. This solution has two steps (based on my understanding):

make knip able to load html files (templateUrl: './app.component.html') or parse html strings (template: '
Hello!
')
then knip should be able to parse them (under the hood angular parses the declarative templates into imperative ones - ref)

Please create reproducible cases so I can find the false positives. Then I have something to work with. Perhaps best to modify/extend that repo?

Take a look at this repo: https://github.com/kfarkasHU/angular-knip-repro
I've added all the errors to the knip config file, so you can check everything in once place.
The issue i was facing with:

there are dependencies such as forms and router that are there, but not used always

when using ng new we have the option to not install router, but once it is installed, it should be used or knip reports it
forms is always there, but not always used (to be heuristic without using this lib, it makes no sense to use angular)

Thanks for your work!

Thanks, I've released a first version you can install like npm install -D knip@angular. Doing 0.0.0 based snapshot releases so please bear with me.

The angular.json is traversed to find some dependencies.
This includes the main script so you don't need to add it as an entry (and in case of src/main.ts, this was already a default entry of Knip).
This includes @angular-devkit/build-angular dependency, resolved from e.g. the @angular-devkit/build-angular:karma builder.

 So there's still false positives here.
Decide what to do with public members eg:

export class TestComponentComponent {
  public publicVariable = 'public variable';
  public form = new FormControl(true);
}

I'll look into what can be inferred from e.g. @Component by Knip. But for now you can/should mark the as public using the @public tag, like so:
export class TestComponentComponent {
  /** @public /*
  public publicVariable = 'public variable';
  /** @public /*
  public form = new FormControl(true);
}

Questions

Is the TypeScript definition for angular.json available somewhere? The thing provided with @angular/cli (workspace-schema.d.ts) is incomplete. For now I shipped a manual conversion from @angular/cli/lib/config/schema.json.

To be continued :)

Feel free to share large projects (yours or not) so we can find false positives and fix 'em! Feel like helping out? Pull requests are most welcome :)

@webpro you are faster than Lucky Luke!

The main.ts change is cool and works as expected 👍
I can't share my main project sadly but will keep you in touch if there are other issues that I'll reproduce in the example project.
Karma/Jasmine are the default unit test dependencies but angular has started to support Jest, and I have the feeling that the community is shifting to Jest. So maybe it's not so needed to put a lot of effort if ignoring them is enough.
About the TypeScript definition for angular.json I'm sorry but I don't it's a bit out of my league. I'll try to check more in the future.

Thanks for this tool.
I'm using the knip@angular in my project, and found that knip is reporting my public vars in my exported component (needed in the html template for my component) as Unused exported class members
@Component({
  selector: 'mycomponent,
  templateUrl: './mycomponent.html',
  styleUrls: ['./mycomponent.scss']
})
export class MyComponent {
   publicVarNeededInHTML: string = 'test'
}

@fjrial I'm thinking in Angular projects you should use --exclude classMembers, as Knip doesn't know what public members are actually used in those HTML templates. Maybe Knip can find some heuristics to improve on this, but not sure yet.

:rocket: This issue has been resolved in v2.26.0. See Release 2.26.0 for release notes.

I just pulled the trigger and published the initial version. Let's see how this pans out :)

Thank you very much 👍 I'm just not such a fan of the @public but I get it, it's the expected behavior.  Thanks again for the job you have done :)

Nah you don't want to add @public everywhere, maybe --exclude classMembers is alright.
Better ideas most welcome :)


[local_auth] Application Crash on Start Up on Android

Hello,
I simply created a new flutter project and added local_auth: ^1.1.8 to my dependences then changed my MainActivity.java
to
package com.meme.flutter_app_local_auth_crash;
import android.os.Bundle;
import io.flutter.app.FlutterFragmentActivity;
import io.flutter.plugins.flutter_plugin_android_lifecycle.FlutterAndroidLifecyclePlugin;
import io.flutter.plugins.localauth.LocalAuthPlugin;

public class MainActivity extends FlutterFragmentActivity {
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        FlutterAndroidLifecyclePlugin.registerWith(
                registrarFor(
                        "io.flutter.plugins.flutter_plugin_android_lifecycle.FlutterAndroidLifecyclePlugin"));
        LocalAuthPlugin.registerWith(registrarFor("io.flutter.plugins.localauth.LocalAuthPlugin"));
    }
}

I also added <uses-permission android:name="android.permission.USE_FINGERPRINT"/> to my manifest tag.
I don't understand what the problem is as I have followed the instructions to the letter.
Here is my flutter doctor -v
[√] Flutter (Channel stable, 2.5.3, on Microsoft Windows [Version 10.0.19043.1348], locale en-US)
    • Flutter version 2.5.3 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (5 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\Moneer\AppData\Local\Android\Sdk
    • Platform android-31, build-tools 30.0.2
    • ANDROID_HOME = C:\Users\Moneer\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2021.1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart

[√] VS Code (version 1.62.2)
    • VS Code at C:\Users\Moneer\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.19.0

[√] Connected device (3 available)
    • sdk gphone x86 64 arm64 (mobile) • emulator-5554 • android-x64    • Android 11 (API 30) (emulator)
    • Chrome (web)                     • chrome        • web-javascript • Google Chrome 95.0.4638.69
    • Edge (web)                       • edge          • web-javascript • Microsoft Edge 92.0.902.62

• No issues found!

Hi @moneer-muntazah, please provide the output of flutter run -v.
The output is likely to be huge, so kindly put it into a .txt file and attach it.
The error logs will help us know how to proceed with addressing this issue.
Thank you

Hi @danagbemava-nc here is the file. The last line reads
[   +1 ms] Waiting for observatory port to be available...
log.txt

Hi @moneer-muntazah, the documentation for local_auth is out of date.
There's an issue to update the readme https://github.com/flutter/flutter/issues/83169.
Kindly follow up there for more information.
As for solving your issue, replace your current MainActivity.java with the following and let me know if it solves your issue
import android.os.Bundle;
import io.flutter.embedding.android.FlutterFragmentActivity;

public class MainActivity extends FlutterFragmentActivity {

}

Yes it did solve it thank you sir