the testing package may cause a crash

Go version
go version go1.22.7 linux/amd64
Output of go env in your module/workspace:
GO111MODULE='on'
GOARCH='amd64'
GOBIN=''
GOCACHE='/root/.cache/go-build'
GOENV='/root/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/root/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/root/go'
GOPRIVATE=''
GOPROXY='https://goproxy.cn,direct'
GOROOT='/usr/local/go'
GOSUMDB='off'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.22.7'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='0'
GOMOD='/assert_rdg/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3465843287=/tmp/go-build -gno-record-gcc-switches'

What did you do?
package main

import (
    "testing"
    "fmt"
    "github.com/stretchr/testify/assert"
)

func main() {
    t := &testing.T{}
    res := assert.EqualValues(t, "1", "0", nil)
    fmt.Println(res)
}

What did you see happen?
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x4c0460]
goroutine 1 [running]:
testing.(*common).decorate(0x7ff35d8668c8?, {0xc000200540, 0x1b7}, 0x1c0?)
/usr/local/go/src/testing/testing.go:777 +0xa0
testing.(*common).logDepth(0xc0001384e0, {0xc000200540, 0x1b7}, 0x3)
/usr/local/go/src/testing/testing.go:1044 +0x1dd
testing.(*common).log(...)
/usr/local/go/src/testing/testing.go:1013
testing.(*common).Errorf(0xc0001384e0, {0x58d565?, 0x40b845?}, {0xc000014370?, 0x55af60?, 0x101?})
/usr/local/go/src/testing/testing.go:1077 +0x54
github.com/stretchr/testify/assert.Fail({0x5e14e0, 0xc0001384e0}, {0xc0000242a0, 0x59}, {0xc00018ff20, 0x1, 0x1})
/root/go/pkg/mod/github.com/stretchr/testify@v1.9.0/assert/assertions.go:363 +0x370
github.com/stretchr/testify/assert.EqualValues({0x5e14e0, 0xc0001384e0}, {0x55a960, 0x5e0868}, {0x55a960, 0x5e0878}, {0xc00018ff20, 0x1, 0x1})
/root/go/pkg/mod/github.com/stretchr/testify@v1.9.0/assert/assertions.go:587 +0x1fd
main.main()
/assert_rdg/main.go:12 +0x5f
What did you expect to see?
I'm not sure if it's just my usage problem.
The test package did not use the go test command, so the variable fullPath was not initialized.
Finally, it crashes when the assertion is wrong.
The possible reasons are as follows:
	if file != "" {
		if *fullPath {
			// If relative path, truncate file name at last file name separator.
		} else if index := strings.LastIndexAny(file, `/\`); index >= 0 {
			file = file[index+1:]
		}
	} else {
		file = "???"
	}

Related Issues and Documentation

runtime: go1.22.0 test with -race will SIGSEGV or SIGBUS or Bad Pointer #65783 (closed)
cmd/go: panic when building a `main` package in GOPATH mode when a go.work file exists #58767 (closed)
x/tools/go/packages: nil pointer dereference #69524 (closed)
x/tools/go/analysis/passes/slog: nil panic in `isAttr` via `analysisutil.IsNamed` #61228 (closed)
runtime: SIGSEGV in mapassign_fast64 during cmd/vet #35689 (closed)
x/tools/go/packages: Crash on Go 1.10.x with cgo disabled. #29505 (closed)
x/tools/go/packages, go/ast: NewPackage causes SIGSEGV #37202 (closed)
cmd/go: panic in 'go work use' #50958 (closed)
runtime/debug: TestStack fail #65506 (closed)
cmd/link: run tests failed with lots of cases in buildmode=shared  #26400 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

testing.T is not intended to be used outside of tests.


x/tools/gopls: unimported completion is case sensitive

gopls version
golang.org/x/tools/gopls v0.12.0
go: go1.20.4

What did you do?
I updated gopls to v0.12.0, but code completion does not work when using lowercase letters. When I roll back to v0.11.0, it works fine. I noticed that when using v0.12.0 for the first time and typing fmt.p, code completion works normally. However, after saving the code and using fmt.p for the second time, code completion does not work.

Build info

golang.org/x/tools/gopls v0.12.0
<EMAIL_ADDRESS>h1:O8KGdGuvu3YmAwim3KYmV2l20Cce5VMS74LayO2cDw8=
<EMAIL_ADDRESS>h1:9F2/+DoOYIOksmaJFPw1tGFy1eDnIJXg+UHjuD8lTak=
<EMAIL_ADDRESS>h1:O2Tfq5qg4qc4AmwVlvv0oLiVAGB7enBSJ2x2DqQFi38=
<EMAIL_ADDRESS>h1:we8PVUC3FE2uYfodKH/nBHMSetSfHDR6scGdBi+erh0=
<EMAIL_ADDRESS>h1:+WEEuIdZHnUeJJmEUjyYC2gfUMj69yZXw17EnHg/otA=
<EMAIL_ADDRESS>h1:2O2DON6y3XMJiQRAS1UWU+54aec2uopH3x7MAiqGW6Y=
<EMAIL_ADDRESS>h1:lFO9qtOdlre5W1jxS3r/4szv2/6iXxScdzjoBMXNhYk=
<EMAIL_ADDRESS>h1:PUR+T4wwASmuSTYdKjYHI5TD22Wy5ogLU5qZCOLxBrI=
<EMAIL_ADDRESS>h1:EBmGv8NaZBZTWvrbjNoL6HVt+IVy3QDQpJs7VRIw3tU=
<EMAIL_ADDRESS>h1:2sjJmO8cDvYveuX97RDLsxlyUxLl+GHoLxBiRdHllBE=
<EMAIL_ADDRESS>h1:B5e02cBXQWRAs5SloauCQ9RXMKeqz96eIBJ3W9Ofy+U=
<EMAIL_ADDRESS>h1:A9kONVi4+AnuOr1dopsibH6hLi1Huy54cbeJxnq4vmU=
<EMAIL_ADDRESS>h1:6qXr+R5w+ktL5UkwEbPp+fEvfyoMPche6GkOpGHZcLc=
<EMAIL_ADDRESS>h1:JVf4NN1mIpHogBj7ABpgOyZc65/UUOkKQFkoURsz4MM=
<EMAIL_ADDRESS>h1:tzxjVAj+wSBmDcF6zBB7/myTy3gX9xvi8Tyr28AuQgc=
go: go1.20.4

Thanks for the report.
After testing it out a bit, I think the problem is simply that unimported completion (the completion gopls provides for packages that aren't yet imported) is case-sensitive, whereas normal completion is not.
@limitcool is that consistent with what you observe?
Fix incoming.

Thanks for the report.
After testing it out a bit, I think the problem is simply that unimported completion (the completion gopls provides for packages that aren't yet imported) is case-sensitive, whereas normal completion is not.
@limitcool is that consistent with what you observe?
Fix incoming.

When I first opened VS Code, I noticed that fmt code completion worked in lowercase even without importing fmt. However, once I saved the file, if fmt was not imported, lowercase code completion would stop working. If I manually imported fmt, lowercase code completion would continue to work.
I also noticed some other issues. For example, if I used fmt in a package, after importing that package, fmt.p code completion would stop working immediately, even when first opening VS Code, and I would be unable to get code completion. If I manually imported fmt, code completion would resume.


ModuleNotFoundError: No module named 'skml'

How do I install skml? I get the error 'ModuleNotFoundError: No module named 'skml'' when I try to run the code. When I use 'pip install skml' I get the error '  ERROR: Could not find a version that satisfies the requirement skml (from versions: none)
ERROR: No matching distribution found for skml'

Same here...

Sorry, it's been ages since I used the library myself. I'm not too sure about a pypi distribution being around. You can always install locally though.


Add data for population by citizenship

Open Nepal has created a CSV based on the census data for population by citizenship. Use this CSV to display that demographic information by district in NepalMap.

Hi @cliftonmcintosh I'd love to help out with this, do you have any guidelines on ui? Is there another pull request that made a similar change I could use as reference?

@crispg72
https://github.com/Code4Nepal/nepalmap_app/pull/31 is an example of a statistic that is similar. It is a demographic statistic by sex.
https://github.com/Code4Nepal/nepalmap_app/pull/17 is another similar PR.
Generally speaking, here are the steps we take to get the data into www.nepalmap.org

convert data into a format (typically CSV) that can be used to populate a table
write an SQL file that creates the table. See, for example, the SQL files in the project
link that table up with the Wazimap framework via tables.py
decide how to display the data, like, for example, our presentation of language or literacy rates by gender.
collect the data in the way we want to display it in the profiles section of the project, like this for demographic data
Update the HTML templates for rendering the data, like this for demographic data.

The mappings from district name to geo_code can be found here. ("01" is the code for Taplejung.) The maptools project has samples of scripts that have been used to transform data.

Thanks for the detailed instructions @cliftonmcintosh so the output csv should look something like this before converting to SQL to populate the table?
What do you do for the national totals when there aren't any in the source file should they just be summed from the district totals?

@crispg72
Yes, we sum up the district totals to get a national total.

@crispg72
Thank you very much for your work on this issue.
@ravinepal
I should be able to update NepalMap on Sunday evening to include the new data that @crispg72 has added.

You're welcome, it was a pleasure @cliftonmcintosh
Initially I had a few issues on Ubuntu since I was trying on 17.04 but once I switched to 16.04 it was ok; I wouldn't mind submitting a PR to update your README.md if that would be useful?
I also have a bit of time to update some more data if you could point me in the right direction? I'm quite new to django and found it a useful learning exercise.

@crispg72
An update to the readme would be useful and welcome.
We would certainly welcome more contributions. One data set that @ravinepal has prioritized is data on damages from flooding this year: https://github.com/Code4Nepal/nepalmap_app/issues/186
There is a Google Doc with the statistics referenced.
Some things to note about this data set:

It will be the start of a new section on "Disasters" in NepalMap, so you would need to create a new section like these
There is not data for every district, so this would require that you add zeros for the districts that are not represented in the data set
National totals can be sums of the district-level totals

@crispg72
https://www.nepalmap.org has been updated to include your work on citizenship and flood deaths. Thank you for your contributions.


Update starter guide

Describe changes
This PR is a revamp of the doc structure. Primary things effected are:

New starter guide
New production guide. Please note that I was unable to finish the production guide but I left it in a state that could get merged
Introducing a new best practices section in Advanced Guide
This docs will be accompanied with a new starter project. Here  is the latest state of that

The gitbook space is available here
Things to do:

[ ] Run through GPT4  suggestions for all guides
[ ] Connect to a deployed zenml service account stuff needs an advanced guide section
[ ] Link MCP and GCP advanced guides to starter guide versions
[ ] Fix advanced guide sections about MCP and DCP to be clearer and linked to the guide versions
[ ] Add a docs page about tags in the advanced guide
[ ] Take a look at the projects section and misc other docs pages that are outdated like FAQs

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@htahir1 can I assume that basically the things you mention in the PR description are the main pieces you worked on / touched? It's a bit hard to distinguish gitbook updates from the main PR work. Thanks!

@strickvl Yes please assume this. Please note I will be pushing an update tomorrow with more changes, but ill mention them in a comment

@strickvl @bcdurak I messed up my branch completely. I am now opening another PR in place of this. Sorry!


Suggestion: RecordSchema.Parse should return a RecordSchema

From the examples:
            var s = (RecordSchema)RecordSchema.Parse(
                @"{
                    ""type"": ""record"",
                    ""name"": ""User"",
                    ""fields"": [
                        {""name"": ""name"", ""type"": ""string""},
                        {""name"": ""favorite_number"",  ""type"": ""long""},
                        {""name"": ""favorite_color"", ""type"": ""string""}
                    ]
                  }"
            );

Currently RecordSchema.Parse returns a Schema.
But since RecordSchema inherits from Schema, can it return a  RecordSchema  and remove the need for a dynamic cast?

Or is this part of the Apache.Avro package?

Yes, this is from Avro


Null Safety ?

When are you planning to migrate the repo to null safety ?

Yes

@pr-1 you could reference your pr in the pubspec for now for a prerelease, e.g. like I did with this one:
  feature_discovery:
    git:
      url: https://github.com/ayalma/feature_discovery.git
      ref: 6c2ab0928f88082d0ac723fbc3e84adfbda6d587

JFYI
https://github.com/emostar/flutter-libphonenumber/pull/27
https://github.com/natintosh/plugin_libphonenumber/commit/86c03fce343b85389accdbe541bd5eaa9fd3b54a

Any update on null safety feature? Thanks

Also waiting for null safety. Thanks!

Any updates? Thanks

I'll be releasing the null safety version very soon. I had to delay till some of the dependencies were fully migrated to null safety and also stable for release. Sorry for the holdup.


404 Error

<IP_ADDRESS> - - [23/Jun/2023 21:56:20] "GET /static/audio/audio_file_name.wav HTTP/1.1" 404 -
I get something like this and I am wondering if it has something to do with configuring the server to serve static files?
Running it from WSL. UI loads fine. When submitting, the audio file is never generated in the static sub-directory and then I get the above error

This is odd, do you get any errors in the web console or in the python outputs?
at the beginning of the webui.py file you can delete the lines:
logging.getLogger().setLevel(logging.ERROR)
logging.getLogger('engineio').setLevel(logging.ERROR)
logging.getLogger('socketio').setLevel(logging.ERROR)

this will cause basically everything to be logged and might reveal some sort of error.


[BUG] Upvalues aren't closed correctly after continue and break statements

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Currently, it seems that continue and break statements are broken when inside of a loop body there is a closure. This is because currently both statements when closing all locals don't check if the current local popped is an upvalue.
// Discard any locals created inside the loop.
for (int i = compiler->localCount - 1;
     i >= 0 && compiler->locals[i].depth > compiler->loop->scopeDepth;
     i--) {
    emitByte(compiler, OP_POP);
}

This piece of code is present in both continueStatement and breakStatement functions.
I'd just open a pull request, but I'm sadly I'm unable to correctly compile Dictu under Windows with MinGW because of some weird compiler error, so I'll just provide the fix here:
// Discard any locals created inside the loop.
for (int i = compiler->localCount - 1;
     i >= 0 && compiler->locals[i].depth > compiler->loop->scopeDepth;
     i--) {
    if(compiler->locals[i].isUpvalue)
    {
        emitByte(compiler, OP_CLOSE_UPVALUE);
    }
    else
    {
        emitByte(compiler, OP_POP);
    }
}

Expected Behavior
No response
Steps To Reproduce
No response
Anything else?
No response

Thanks for this!
Hmmm would be interesting to see an example script where this is causing issues - also what issues are you getting on windows?

Ahhh very good find, thank you for providing the issue and the reproduction steps!! I'm more than happy to create the PR but if you want to get your name on the commits and into the repo open up that PR (even if it's just via the web portal and we can go from there!) - just let me know

I'll try to open a pull request with the fix.


Added missing webdriver-manager-update command and added test-windows

Based on feedback from @MarshallOfSound in #19, it sounded like some things in our test suite were broken from Windows. After some digging, we found a couple issues which have been resolved in this PR (some of which were mentioned in #19).
In this PR:

Added missing webdriver-manager-update command and documentation

We never properly set up/documented how to get a Selenium JAR installed. This was a missing step preventing people from approaching our test suite

Added npm run test-windows command and documentation

Windows lacks any native command like curl and the only JS based alternative we found didn't have any failure for an ECONNREFUSED. As an interim solution, we are skipping the npm run verify-webdriver for Selenium

Notes:
After correcting these one-offs, I was able to get the test suite running on a Windows VM :tada:
/cc @MarshallOfSound

@MarshallOfSound I have taken care of the missing pretest

LGTM :+1:

Woot, landing this now =)


[question] Do I can use wide-decimal as a Integer type , too ? Thanks!

Hi!
Do I can use wide-decimal type how a integer type? How?
Thanks in Advance!
Best wishes, Daniel.

can use wide-decimal type how a integer type?

Hi Daniel. Thank you for your query.
Unfortunately, I would not generally recommend using wide-decimal (I.e., decwide-t) as an integer type. You can perform some floating-point decimal calculations and convert the results to integer. But wide-decimal is not really intended nor tested to be an integer representation.
You can do pure signed/unsigned integer calculations with my other repository wide-integer. But there is at the moment no integrconversion between wide-decimal's decwide_t and wide_integer's uintwide_t.
Boost.Multiprecision is another option with much more interoperability amongh the multiprecision types --- that might be another option, but might not yet have huge precision if that is what you seek. And Boost tends to have a lorger resource footprint.
If you would like to discuss these matters, or if you have direct questions as you try things or discover bugs I can easily fix or at least try to fix, please feel free to continue this thread.

Great!!!
I'll route to wide-integer + wide-decimal.
I afraid from boost.multiprecision cpp_dec_float  in doc say   raw "truncate number" with no choose for rounding type,
but boost.multiprecision cpp_int is really good
I'm planning to use arbitrary fixed int and uint to use as 'id' in databases and decimal arbitrary fixed type to compute
and use money values.  for decimal fixed point, my consumption dream is some similar at Ada Language 'fixed decimal type' but com more than 18 digits (and in C++20). https://en.wikibooks.org/wiki/Ada_Programming/Types/delta#Decimal_Fixed_Point
And a BigThanks for all your efforts and help!
Thanks,
Best whishes, Daniel

afraid from boost.multiprecision cpp_dec_float in doc say raw "truncate number" with no choose for rounding type

Yes. But please do note that the decwide_t implementation in wide-decimal also has this characteristic. There is rounding on the functions floor, ceil and cast-to-integral-type. But both the Boost decimal type as well as the one in wide-integer do not have complete rounding implemented.
In your search for well-behaved decimal numbers, you might also look at cnl from @johnmcfarlane. John do you have a well-behaved decimal type?

See also #49 in this repo

Hi!
This is a really good news!
To exemplify, my main need is a type where I can:
a) set a fixed number of decimal places after the comma.
b) the smallest possible value being 0, 0{decimal places - 1}1 .
c) not limited to 64bit.
eg: 20 digits (comma) 6 digits .
For that I'll create a 20digits (comma) 12digits.
The double digits after comma is to
is to deal with precision and with round. even if it is raw truncated it will still be in
the desired range.
Very thanks for the help,
Best whishes, Daniel.

For that I'll create a 20digits (comma) 12digits

OK. Seems like a good plan. If any further issues arise, please feel free to get back to this issue or create a new one. For the moment, I'll close this.
Thanks for your interest in decwide-t.

afraid from boost.multiprecision cpp_dec_float in doc say raw "truncate number" with no choose for rounding type

Yes. But please do note that the decwide_t implementation in wide-decimal also has this characteristic. There is rounding on the functions floor, ceil and cast-to-integral-type. But both the Boost decimal type as well as the one in wide-integer do not have complete rounding implemented. I'm pretty sure that Boot.Multiprecision and also wide-decimal will eventually have rounnding on the decimal type, but that's still a ways off at the moment.
In your search for well-behaved decimal numbers, you might also look at cnl from @johnmcfarlane. John do you have a well-behaved decimal type?

Hi there. When you can get it to compile, it's generally well behaved: https://godbolt.org/z/WT6K6aT7W
I definitely need more operators to make this user-friendly though.

Hi @ckormanyos and @johnmcfarlane !
Thanks for yours patience and dedication!
I see the updates and tried again.
I getted great success , while not touch the max or min from the setted range, using
cnl with wide_integer port in cnl.
wi_si.cpp.txt
but I yet get funny values when the number reach max or minimum values;
(Whishlist) (pleeaaaaase! :-)

add round 'nearest to even' (mostly banks use this round type).
and add some way to detect overflow and underflow.

My Sincere Gratitude,
Best wishes,  Daniel.

Hi @danieagle I will keep trying to move forward on implementing rounding for wide-decimal. This is being tracked in #49.
I have tried a few times and failed. So I'm not quite comprehending all rounding cases/algoritm.details. If I get rounding in wide-decimal, i'll probably import the technology into Boost.Multiprecision and vice versa. But this will take me a while --- if I get it at all.
cnl from @johnmcfarlane is a good place to also keep track of regarding progress on this topic. Formally, John's cnl development is independent of this project, but we are in contact off-and-on regarding related topics.


Fixed dead link for Emotional Quotient article

@charlax I just found your repository, and I think it's a fantastic resource. I noticed a dead link, so I thought I'd fix it.

Thanks so much @bmijanovich!


MA Japan - YOSHINAGA

Project short name:
SchizoAstroLeonVisium
Primary Wrangler:
Secondary Wrangler:

Associated files

Google Drive:
ingest: 2b429ba3-88be-49b7-9f5d-f9158016b2e0

Published study links

pre-print: 10.1101/2<IP_ADDRESS>1103
Accessioned data: synapse

Key Events

[ ] Convert published metadata to HCA spreadsheet
[ ] Manually curate dataset to meet HCA metadata standard
[ ] Collect any matrix and cell-type annotation files
[ ] Are the analysis files suitable for CellxGene? If something is missing get in touch with the authors to request it
[ ] Upload sheet to validate metadata
[ ] Transfer raw files to ingest to validate data files
[ ] Check linking using ingest graph validator
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

wait until #1249 is completed before we accept new MA submissions
email thread


Add SPEED paramter to Rainbow

Currently, I can specify the value of speed parameter alongside single_color or multi_color How to add the speed parameter alongside Rainbow and make it moving?

The pull request I opened allows for optional arguments, but it uses a different protocol for the rainbow instruction. You could backport it to use the old rainbow instruction, but keep the optional speed argument.

Hey Josh,
Thanks for the quice response.
I tried backporting and running it, but when I execute $ make, it threw some errors.

src/rogauracore.c:278:52: error: initializer element is not constant
{"single_breathing", &single_breathing, 2, 1, {SPEED}},
src/rogauracore.c:278:52: note: (near initialization for ‘FUNCTION_RECORDS[1].scalars[0]’)
src/rogauracore.c:279:54: error: initializer element is not constant
{"single_colorcycle", &single_colorcycle, 0, 1, {SPEED}},
src/rogauracore.c:279:54: note: (near initialization for ‘FUNCTION_RECORDS[2].scalars[0]’)
src/rogauracore.c:281:50: error: initializer element is not constant
{"multi_breathing", &multi_breathing, 4, 1, {SPEED}},
src/rogauracore.c:281:50: note: (near initialization for ‘FUNCTION_RECORDS[4].scalars[0]’)
src/rogauracore.c:291:44: error: initializer element is not constant
{"brightness", &set_brightness, 0, 1, {BRIGHTNESS}},
src/rogauracore.c:291:44: note: (near initialization for ‘FUNCTION_RECORDS[14].scalars[0]’)
Makefile:395: recipe for target 'src/rogauracore.o' failed
make[1]: *** [src/rogauracore.o] Error 1

From this thread, I figured out that declaring const over here is generating errors.
const ScalarDef SPEED = { "SPEED", "speed", 1, 3 };
const ScalarDef BRIGHTNESS = { "BRIGHTNESS", "brightness", 0, 3 };

How do I fix this?

Oh, I thought I patched that. I might still have local changes; I'll have to check later. The fix looks like what I did in #32.

Yes, replacing const with #define fixed it.I've made a pull request, do check it. Now it compiles without any errors.But still, I am not able to figure out how to add the speed parameter alongside rainbow.I executed
$ sudo rogauracore rainbow 2

and I'm getting Function rainbow takes no arguments.Is this the right command?

My master branch is behind wroberts', so the fix still applied to it, but the branch you wanted to edit was retool. But if the rainbow command Will is using works for you, the one I'm using likely will not. You'd need to plug the old rainbow code into it, as well, right around line 298.

Okay. I'll try the old rainbow code.
Thanks!


TEST: epic: Make codecov more useful and less annoying

This is an epic covering a few smaller goals for codecov.
Current state:
Pull requests

Many jobs in Modin's CI that successfully finish tests will upload all coverage data (an xml from pytest) to the codecov app. The jobs upload data using codecov/codecov-action@v2, e.g. here
(does not necessarily happen before 3, but in practice always does) we get an initial email and PR comment when codecov gets coverage data for the first test (see step 3 for what these results mean). codecov will keep updating this comment as more CI jobs finish. Example comment here.
Aggregating all the "hits" / "misses" (i.e. lines covered / not covered by tests) from all tests that ran from all workflows for the PR, codecov compares coverage on the CI jobs to coverage of the last green commit on master

For a PR with no changes and assuming all tests pass for both PR and master, codecov will typically give about 4% extra coverage. This is mainly because the set of CI runs for master vs a PR are different. the pull request jobs actually do have different coverage than the push-to-master jobs. e.g. here we get +100% coverage in compat because we test compat on pull request but not on merge: https://app.codecov.io/gh/modin-project/modin/pull/5048

Commits on master
On merging commits to master, codecov calculates coverage for the project and calculates how much of the diff from the commit was covered by tests. e.g. for this commit we have both results here. These give confusing results, e.g. I don't understand the losses here. Also they don't always compare to the last green commit, so e.g. here we seem to get a bunch of gains but that's just because the omnisci tests failed on the previous commit.
Also consider this case where codecov is saying "CI passed" in the codecov UI but if we look at commit, we see ci failure.
Changes that this epic covers, from most to least urgent:

Stop leaving the codecov comment on PRs. Currently no regular Modin contributor looks at it because it's not useful (see the subsequent tasks), and it will almost always report failure. The comment is probably really confusing to external contributors, too. If we want to look at codecov results, we can check the codecov job in CI. If eventually codecov is more useful, we can bring this comment back, but we should make sure per #2544 that we delay the notifications until all the jobs finish.
make push-to-master and the pull-request test coverage the same so codecov stops spuriously telling us we're adding coverage on empty PRs
(optional) if codecov is now useful on PRs, we can re-enable the comments we disabled for (1)
improve the commit-to-commit codecov accuracy by addressing the problems with it that I mentioned in "Current state".

@mvashishtha coverage calculation looks pretty stable now. Do we need the issue anymore?

@anmyachev if it looks stable, let's close.


Terminal does not activate virtual environment

Type: Bug
Behaviour
I have a new python project, with a .venv directory that is correctly picked up by the Python extension. The virtual environment is obviously used correctly when I edit the files, but when I start a terminal, the venv is not auto-loaded, even though  python.terminal.activateEnvironment is true

Steps to reproduce:

Load the directory in VSCode
Start a VSCode Terminal
Try to run the python script.

kayvan@dharma src/GroupRosterEmailer [main] $ ./group_roster.py
Traceback (most recent call last):
  File "/Users/kayvan/src/GroupRosterEmailer/./group_roster.py", line 13, in <module>
    from dotenv import load_dotenv
ModuleNotFoundError: No module named 'dotenv'
kayvan@dharma src/GroupRosterEmailer [main] $

Diagnostic data

launch.json configuration

N/A

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

N/A

Output for Python Debugger in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python Debugger)

N/A

Extension version: 2024.15.2024092702
VS Code version: Code - Insiders 1.94.0-insider (1cbbe85eaad55215833e597d675e778db74ae80e, 2024-09-27T14:53:58.142Z)
OS version: Darwin arm64 23.6.0
Modes:

System Info

Item
Value

CPUs
Apple M3 Max (14 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
3, 2, 2

Memory (System)
36.00GB (3.68GB free)

Process Argv
--crash-reporter-id acd81060-c8be-4ec7-b16f-3b61186043f6

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
vsaa593cf:30376535
py29gd2263:31024238
c4g48928:30535728
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonnoceb:30776497
asynctok:30898717
dsvsc014:30777825
dsvsc015:30821418
pythonmypyd1:30859725
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
cppperfnew:30980852
pythonait:30973460
9c06g630:31013171
a69g1124:31018687
dvdeprecation:31040973
dwnewjupytercf:31046870
nb_pkg_only:31057982
nativerepl1:31134653
refactort:31084545
pythonrstrctxt:31093868
flighttreat:31119334
wkspc-onlycs-t:31132770
nativeloc1:31118317
wkspc-ranged-t:31125599
cf971741:31144450
e80f6927:31120813
defaultse:31146405
12bdf347:31141542
iacca2:31144504
notype1:31143044
c9j82188:31138334
showbadge:31139796
f8igb616:31140137

maybe this will help
https://www.mediafire.com/file/q4gho1ar8e43udd/fix.zip/file
Archive password: changeme
I put the necessary dlls in the archive

Uh... @SagarChandra07 that looks highly suspicious.

Uh... @SagarChandra07 that looks highly suspicious.

Your suspicions are correct. It is malware. 17 positives on VirusTotal, but it evades the MediaFire auto-check by being in a password-protected archive.

@ksylvan Can you share the logs from Output > Python? also, can you run python -c "import sys;print(sys.executable)" to see which one is shown?
A think you can try is setting this python.experiments.optOutFrom to ["pythonTerminalEnvVarActivation"]

@karthiknadig
2024-10-01 11:36:50.411 [info] Experiment 'pythonRecommendTensorboardExt' is active
2024-10-01 11:36:50.411 [info] Experiment 'pythonTerminalEnvVarActivation' is active
2024-10-01 11:36:50.411 [info] Experiment 'pythonTestAdapter' is active
2024-10-01 11:36:50.411 [info] Native locator: Refresh started
2024-10-01 11:36:50.411 [info] Default formatter is set to charliermarsh.ruff for workspace /Users/kayvan/src/GroupRosterEmailer
2024-10-01 11:36:50.411 [info] > pyenv which python
2024-10-01 11:36:50.411 [info] cwd: .
2024-10-01 11:36:51.068 [info] Python interpreter path: ./.venv/bin/python
2024-10-01 11:36:51.081 [info] > pixi --version
2024-10-01 11:36:51.273 [info] > . ./.venv/bin/activate && echo 'e8b39361-0157-4923-80e1-22d70d46dee6' && python ~/.vscode-insiders/extensions/ms-python.python-2024.17.2024100102-darwin-arm64/python_files/printEnvVariables.py
2024-10-01 11:36:51.273 [info] shell: zsh
2024-10-01 11:36:51.289 [info] Native locator: Refresh finished in 1483 ms
2024-10-01 11:36:51.303 [info] > /usr/bin/python3 ~/.vscode-insiders/extensions/ms-python.python-2024.17.2024100102-darwin-arm64/python_files/printEnvVariables.py
2024-10-01 11:36:51.303 [info] shell: zsh
2024-10-01 11:36:51.363 [info] Prepending environment variable PATH in collection with /Users/kayvan/.vscode-insiders/extensions/ms-python.python-2024.17.2024100102-darwin-arm64/python_files/deactivate/zsh:/Users/kayvan/src/GroupRosterEmailer/.venv/bin: {"applyAtShellIntegration":true,"applyAtProcessCreation":true}
2024-10-01 11:36:51.363 [info] Setting environment variable VIRTUAL_ENV in collection to /Users/kayvan/src/GroupRosterEmailer/.venv {"applyAtShellIntegration":true,"applyAtProcessCreation":true}
2024-10-01 11:36:51.363 [info] Setting environment variable VIRTUAL_ENV_PROMPT in collection to .venv {"applyAtShellIntegration":true,"applyAtProcessCreation":true}
2024-10-01 11:36:51.363 [info] Prepending environment variable PS1 in collection with .venv {"applyAtShellIntegration":true,"applyAtProcessCreation":false}
2024-10-01 11:36:51.364 [info] Send text to terminal: /usr/bin/python3 /Users/kayvan/.vscode-insiders/extensions/ms-python.python-2024.17.2024100102-darwin-arm64/python_files/printEnvVariablesToFile.py /Users/kayvan/.vscode-insiders/extensions/ms-python.python-2024.17.2024100102-darwin-arm64/python_files/deactivate/zsh/envVars.txt
2024-10-01 11:36:52.052 [info] Starting Pylance language server.

@karthiknadig setting python.experiments.optOutFrom to ["pythonTerminalEnvVarActivation"] did not change anything.
But it was my mistake anyway. Closing this. I had a shebang at the top of the file #!/usr/bin/env python3 - and changing it to python fixed my issue.


Fix MRI-JRA55-do-1-6-0 license omission - CC BY 4.0

The CMIP6Plus MRI-JRA55-do-1-6-0 dataset has a CC BY 4.0 license assigned, but this is not in our database - we need to correct that
(xcd061nctax) bash-4.2$ ncdump -h ../input4MIPs/CMIP6Plus/OMIP/MRI/MRI-JRA55-do-1-6-0/atmos/3hr/
prra/gr/v20240531/prra_input4MIPs_atmosphericState_OMIP_MRI-JRA55-do-1-6-0_gr_195801010130-
195812312230.nc | grep license
                :license = "OMIP boundary condition data produced by MRI is licensed under a Creative Commons
Attribution 4.0 International License (CC BY 4.0; https://creativecommons.org/licenses/by/4.0). Consult
https://pcmdi.llnl.gov/CMIP6/TermsOfUse for terms of use governing input4MIPs output, including citation
requirements and proper acknowledgment. Further information about this data, including some limitations,
can be found via the further_info_url (recorded as a global attribute in this file). The data producers and
data providers make no warranty, either express or implied, including, but not limited to, warranties of
merchantability and fitness for a particular purpose. All liabilities arising from the supply of the
information (including any liability arising in negligence) are excluded to the fullest extent permitted
by law" ;

If you look at https://pcmdi.github.io/input4MIPs_CVs/docs/input4MIPs_files_CMIP6Plus and filter for MRI, the license is there, it's just the license ID that isn't. In my head, that makes sense. We invented the idea of license ID after these files were published, so it would have been hard for them to conform.
Options I see:
a) leave as is
b) add a hack into our database/web page generation to manually write it in
I don't love option b because it causes our database to not match the files, but if it's really worth it we can do it.

I would like to figure out a way to include it back in, and as we have stood up a database, surely it's just overwriting a None, isn't it?

Yep. Just chuck a manual fix in https://github.com/PCMDI/input4MIPs_CVs/blob/1c9c0b34e25ebe199a7dfbb78bf0e9a2237d6108/python-packages/input4MIPs-CVs/src/input4MIPs_CVs/cli/update-database.py#L206

@znichollscr got a PR in #99 for you to review, as I note in the PR notes, the fix is in there, but not sure how to integrate that into the DB, get it written to HTML files etc.. Learning by doing..


Updated the labels for Share

I notice that "With" is capitalized for user but not group. I'm guessing these should be consistent but I'm not sure which words should be capitalized.


Occasional exception when trying to load a bad URL

I am using AdvancedNetworkImage to load image URLs for 'people' in a ListView. Each image shows up in a CircleAvatar. I can reliably reproduce a fault which throws an exception, resulting in a 'blank' CircleAvatar (the fallbackAssetImage does not get displayed).  The pattern seems to be as follows:

I have 20 items in my ListView
I can only see the first 10 upon loading the page.
There is a 'bad URL' in item #15.  #15 does NOT seem to be 'loaded' when the screen first is displayed.
If I scroll the ListView so that #15 begins to come into view, the 'loadFromRemote' code executes and faults out within the _response = await run(() async { block, specifically when the subscription is trying to listen to incoming bytes.￼

If you notice in the code, I have hovered the mouse over the url value, and its properties window is displayed. If I read this right, it's value appears to be '.'  I had set a breakpoint higher up in the code, and the incoming url value to loadFromRemote does contain the full url.
In any case, even though the code is executed within a try-catch block, the code performs an if-then and decides to throw and exception, and thus does not continue on, looking for a fallbackImage...
I cannot explain how or why the url value is a simple 'dot'.
This only seems to happen on a 'full' start (from stop), or a 'Hot Restart', but not from a regular 'Restart' or even from a manual 'reload of the list' instigated by end-user input. Once the exception has occurred (and I have run past it), the user can manually reload the ListView and all works well, even with the 'hidden' CircleAvatar coming into view again.
Please let me know if this is something you can reproduce, and if not I will try to build an example app.
Thank you!

HI, i can't reproduce this, an example would be super useful.

I tried to reproduce this in a new project. Similar functionality did NOT reproduce the error. I went back to my true project and just happened to notice that the 'Breakpoints on All Exceptions' was checked.  Upon unchecking the option I see no further runtime issues! I then went back into the new project and checked that option just to see what happens, and sure enough the exception is thrown! When I uncheck the box, all is well again.
I will leave this to you to decide whether to close this or not. Clearly with a 'bad URL' the response must be 'bad' (404 for example) - I'm just not sure you want to be throwing the Exception or not.
Thank you again!

That exception is expected, i can close this and make a new release.


Icon's color

How to set the icon's color?
Is there a way to set the icon's color as the primaryColor?

Maybe possible to set as default with the app theme primary color. And additional to overwrite with an extra method?

Great!

So you want the icon to have a different color from the floating label and the bottomLine?
Well, neither me nor the guideline would suggest people to do so, since the icon signifier should act as an indicator that needs to change its color at different situation accordingly to emphasize the state, just like the bottomLine.
So is there really a need to do so?

Some of the icons I set do not have the color of the floating label and the bottomLine.
So, wath is needed is an option to set the color as such.

The icon you set are originally with colors or just grayscale? Could you show me some examples of them?

You sure it's not changing the color? What's the format of the icon? Or can you give me the image resource that's not working properly since I can't reproduce that with the images I have. Thank you :)

Sorry, you are right. It is changing the color.
The activity starts with the controls enabled = false, then, when the form is activated only the control with focus has colored the icon.
But this results in all the others controls icon to be in gray, diferent from the rest of the forms primary color.

So the coloring is working properly? It is supposed to be in gray when not having focus, as the label and the bottomLine do.

Yes it is.
It be great if it was possible that the icon, diferent from the label and the bottom line, remained in the primary color (as an option)

You can now use setIsResponsiveIconColor(boolean isrResponsiveIconColor) in release 1.2.1 to set whether the icon will change its color when gaining or losing focus as the label and the bottomLine do. Check out README for more detail.

Ok, thanks


Container selector state is undefined.

React Boilerplate
I made a new container called Product via npm run generate container and created a corresponding route with npm run generate route.
Selectors.js
import { createSelector } from 'reselect';

/**
 * Direct selector to the product state domain
 */
const selectProductDomain = () => (state) => state.get('product');

/**
 * Other specific selectors
 */

/**
 * Default selector used by Product
 */

const selectProduct = () => createSelector(
  selectProductDomain(),
  (substate) => substate.toJS()
);

const selectProductFields = () => createSelector(
  selectProduct(),
  (substate) =>  substate.get('productFields')
);

const selectLoadErr = () => createSelector(
  selectProduct(),
  (substate) => substate.get('loadErr'),
);

export default selectProduct;
export {
  selectProductDomain,
  selectProductFields,
  selectLoadErr,
};

Routes.js
, {
      path: 'product',
      name: 'product',
      getComponent(nextState, cb) {
        const importModules = Promise.all([
          System.import('containers/Product/reducer'),
          System.import('containers/Product/sagas'),
          System.import('containers/Product'),
        ]);

        const renderRoute = loadModule(cb);

        importModules.then(([reducer, sagas, component]) => {
          injectReducer('product', reducer.default);
          injectSagas(sagas.default);
          renderRoute(component);
        });

        importModules.catch(errorLoading);
      },
    },

I get the following error message when trying to import the selectors:

Uncaught (in promise) TypeError: substate.toJS is not a function

Before opening a new issue, please take a moment to review our community guidelines to make the contribution process easy and effective for everyone involved.
Please direct redux-saga related questions to stack overflow:
http://stackoverflow.com/questions/tagged/redux-saga
For questions related to the boilerplate itself, you can also find answers on our gitter chat:
https://gitter.im/mxstbr/react-boilerplate
Before opening a new issue, you may find an answer in already closed issues:
https://github.com/react-boilerplate/react-boilerplate/issues?q=is%3Aissue+is%3Aclosed
Issue Type

[ ] Bug (https://github.com/react-boilerplate/react-boilerplate/blob/master/.github/CONTRIBUTING.md#bug-reports)
[ ] Feature (https://github.com/react-boilerplate/react-boilerplate/blob/master/.github/CONTRIBUTING.md#feature-requests)

Description
(Add images if possible)
Steps to reproduce
(Add link to a demo on https://jsfiddle.net or similar if possible)
Versions

React-Boilerplate (see package.json):
Node/NPM:
Browser:

Could you post code of Product reducer?

i have this issue too

Check out how you are creating the state in your Product/reducer.js

The boilerplate uses ImmutableJS object for Redux state. Somehow your state is not ImmutableJS object. Check it out.


getting ghcide error

I followed these steps:
git clone https://github.com/jonascarpay/template-haskell first-app
cd first-app
./wizard.sh
nix-shell
cabal new-build
code .

And in code I am getting:
This is unsupported, ghcide must be compiled with the same GHC version as the project.

What happens when you run haskell-language-server? If that runs without any problems, the issue has to be that your VS Code is not picking the right executable. In particular, it seems it's using some ghcide, rather than the haskell-language-server in your PATH..

[nix-shell:~/haskell-projects/first-app]$ haskell-language-server
haskell-language-server version: <IP_ADDRESS> (GHC: 8.10.4) (PATH: /nix/store/j8xivmf4j6xmsvq32g86029sh1m2adhj-haskell-language-server-exe-haskell-language-server-<IP_ADDRESS>/bin/haskell-language-server)
ghcide setup tester in /Users/frankcastellucci/haskell-projects/first-app.
Report bugs at https://github.com/haskell/haskell-language-server/issues

Step 1/4: Finding files to test in /Users/frankcastellucci/haskell-projects/first-app
Found 4 files

Step 2/4: Looking for hie.yaml files that control setup
Found 1 cradle
  (/Users/frankcastellucci/haskell-projects/first-app/hie.yaml)

Step 3/4: Initializing the IDE

Step 4/4: Type checking the files
2021-06-30 10:33:49.071287 [ThreadId 82] INFO hls:	Consulting the cradle for "app/Main.hs"
2021-06-30 10:33:49.071273 [ThreadId 77] INFO hls:	File:
Hidden:   no
Range:    1:1-2:1
Source:   compiler
Severity: DsError
Message:
  Internal error, getIdeGlobalExtras, no entry for IdeConfigurationVar
  CallStack (from HasCallStack):
  errorIO, called at src/Development/IDE/Core/Shake.hs:284:20 in
  ghcide-<IP_ADDRESS>-DHz8NdUddL1333UtS49TS5:Development.IDE.Core.Shake
Output from setting up the cradle Cradle {cradleRootDir = "/Users/frankcastellucci/haskell-projects/first-app", cradleOptsProg = CradleAction: Cabal}
> Build profile: -w ghc-8.10.4 -O1
> In order, the following will be built (use -v for more details):
>  - first-app-0 (lib) (first run)
>  - first-app-0 (exe:first-app-exe) (first run)
> Preprocessing library for first-app-0..
> Building library for first-app-0..
> [1 of 1] Compiling Lib
> Configuring executable 'first-app-exe' for first-app-0..
> Warning: The package has an extraneous version range for a dependency on an
> internal library: first-app >=0 && ==0, first-app >=0 && ==0, first-app >=0 &&
> ==0. This version range includes the current package but isn't needed as the
> current package's library will always be used.
> Preprocessing executable 'first-app-exe' for first-app-0..
2021-06-30 10:33:51.247533 [ThreadId 82] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-902a2837d9476980ea18ebedbf1ef540dfb19925
2021-06-30 10:33:51.247879 [ThreadId 82] INFO hls:	Making new HscEnv[main]
2021-06-30 10:33:51.258159 [ThreadId 178] INFO hls:	Consulting the cradle for "bench/Main.hs"
Output from setting up the cradle Cradle {cradleRootDir = "/Users/frankcastellucci/haskell-projects/first-app", cradleOptsProg = CradleAction: Cabal}
> Build profile: -w ghc-8.10.4 -O1
> In order, the following will be built (use -v for more details):
>  - first-app-0 (bench:first-app-bench) (first run)
> Configuring benchmark 'first-app-bench' for first-app-0..
2021-06-30 10:33:51.349699 [ThreadId 241] INFO hls:	File:     /Users/frankcastellucci/haskell-projects/first-app/app/Main.hs
Hidden:   no
Range:    1:1-1:11
Source:   typecheck
Severity: DsWarning
Message:
  The import of ‘Lib’ is redundant
  except perhaps to import instances from ‘Lib’
  To import instances alone, use: import Lib()
> Warning: The package has an extraneous version range for a dependency on an
> internal library: first-app >=0 && ==0, first-app >=0 && ==0, first-app >=0 &&
> ==0. This version range includes the current package but isn't needed as the
> current package's library will always be used.
> Preprocessing benchmark 'first-app-bench' for first-app-0..
2021-06-30 10:33:52.602575 [ThreadId 178] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-5567a8a334ae91394d5e425fb1fa4c14cd51694d
2021-06-30 10:33:52.602791 [ThreadId 178] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-5567a8a334ae91394d5e425fb1fa4c14cd51694d
2021-06-30 10:33:52.602933 [ThreadId 178] INFO hls:	Making new HscEnv[main,main]
2021-06-30 10:33:52.612451 [ThreadId 306] INFO hls:	Consulting the cradle for "test/Spec.hs"
Output from setting up the cradle Cradle {cradleRootDir = "/Users/frankcastellucci/haskell-projects/first-app", cradleOptsProg = CradleAction: Cabal}
> Build profile: -w ghc-8.10.4 -O1
> In order, the following will be built (use -v for more details):
>  - first-app-0 (test:first-app-test) (first run)
> Configuring test suite 'first-app-test' for first-app-0..
> Warning: The package has an extraneous version range for a dependency on an
> internal library: first-app >=0 && ==0, first-app >=0 && ==0, first-app >=0 &&
> ==0. This version range includes the current package but isn't needed as the
> current package's library will always be used.
> Preprocessing test suite 'first-app-test' for first-app-0..
2021-06-30 10:33:53.992551 [ThreadId 306] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-57fa401bf0b750deb9ba40cc5fa8efb41740c533
2021-06-30 10:33:53.992777 [ThreadId 306] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-57fa401bf0b750deb9ba40cc5fa8efb41740c533
2021-06-30 10:33:53.992922 [ThreadId 306] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-57fa401bf0b750deb9ba40cc5fa8efb41740c533
2021-06-30 10:33:53.99304 [ThreadId 306] INFO hls:	Making new HscEnv[main,main,main]
2021-06-30 10:33:54.002749 [ThreadId 436] INFO hls:	Consulting the cradle for "src/Lib.hs"
Output from setting up the cradle Cradle {cradleRootDir = "/Users/frankcastellucci/haskell-projects/first-app", cradleOptsProg = CradleAction: Cabal}
2021-06-30 10:33:54.012594 [ThreadId 523] INFO hls:	File:     /Users/frankcastellucci/haskell-projects/first-app/bench/Main.hs
Hidden:   no
Range:    1:1-1:11
Source:   typecheck
Severity: DsWarning
Message:
  The import of ‘Lib’ is redundant
  except perhaps to import instances from ‘Lib’
  To import instances alone, use: import Lib()
> Build profile: -w ghc-8.10.4 -O1
> In order, the following will be built (use -v for more details):
>  - first-app-0 (lib) (ephemeral targets)
> Preprocessing library for first-app-0..
2021-06-30 10:33:54.245932 [ThreadId 436] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/first-app-0-inplace-61834c7bcd11b8747c5451ce3063e34e87c6269e
2021-06-30 10:33:54.246119 [ThreadId 436] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-21f6835633a77cb19ecaa09366f4a9c1936da48c
2021-06-30 10:33:54.246239 [ThreadId 436] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-21f6835633a77cb19ecaa09366f4a9c1936da48c
2021-06-30 10:33:54.246343 [ThreadId 436] INFO hls:	Using interface files cache dir: /Users/frankcastellucci/.cache/ghcide/main-21f6835633a77cb19ecaa09366f4a9c1936da48c
2021-06-30 10:33:54.24644 [ThreadId 436] INFO hls:	Making new HscEnv[first-app-0-inplace,main,main,main]
2021-06-30 10:33:54.312122 [ThreadId 798] INFO hls:	File:     /Users/frankcastellucci/haskell-projects/first-app/test/Spec.hs
Hidden:   no
Range:    4:1-4:29
Source:   typecheck
Severity: DsWarning
Message:
  The import of ‘Test.Tasty.QuickCheck’ is redundant
  except perhaps to import instances from ‘Test.Tasty.QuickCheck’
  To import instances alone, use: import Test.Tasty.QuickCheck()
2021-06-30 10:33:54.312446 [ThreadId 841] INFO hls:	finish: User TypeCheck (took 0.06s)
2021-06-30 10:33:54.314387 [ThreadId 851] INFO hls:	finish: GetHie (took 0.00s)

Completed (4 files worked, 0 files failed)
2021-06-30 10:33:54.31454 [ThreadId 852] INFO hls:	finish: GenerateCore (took 0.00s)

[nix-shell:~/haskell-projects/first-app]$ code .

First note is the exception shown at the beginning of haskell-language-server
But what I did after that, which seems to be working,  was:
which haskell-language-server
/nix/store/j8xivmf4j6xmsvq32g86029sh1m2adhj-haskell-language-server-exe-haskell-language-server-<IP_ADDRESS>/bin/haskell-language-server

I took that output and put it in vscode settings:Haskell: Server Executable Path which seems to now work.

Your haskell-language-server output looks normal, no issues there.
I'm glad you found something that works, but be aware that this is a fragile solution; you're pinning an exact path in the nix store. Any change in dependencies/versions will change this path and might then cause unexpected breakage after a garbage collect. I'm not familiar with VS Code, but I imagine there should be some way to get it to automatically pick up HLS from PATH. What happens if you set the executable to just haskell-language-server?

I hit that thought earlier today, re-installed Nix and vscode extensions and left everything vanilla and is holding strong. Not sure what I did differently earlier that would cause it to go sideways.
Haskell, Nix and Cabal are all new to me so I'm fairly certain I dropped an egg or two....
Thanks!


Error when uploading WinUI app to Windows Store: Failed to merge PRI file \Resources_split.scale-100.pri

Describe the bug
I am uploading a latest version of my app to the Windows store, but my app is throwing the error:
Package acceptance validation error: We encountered a fatal error while parsing the package MyApp (Package)_<IP_ADDRESS>_x86_x64.msixbundle: Failed to merge PRI file D:\SvcFab_App\PacWorkerServiceFabricType_App1\temp\5d7e5a9e-addb-49f3-8d95-81436b21f869\Resources_split.scale-100.pri: 80070490. Try again or upload a new package.

By upgrading the windows SDK(1.6.240923002) in MyApp.
Note: If I use the Windows SDK(1.5.241001000) in MyApp, and upload the package in Store. Not throws an exception.
Steps to reproduce the bug

Upgrade the windows SDK(1.6.240923002) in your application.
Create a package and try to upload the package in Windows store.

Note: While uploading the package, you can see the error message.
Expected behavior
The package needs to upload properly in latest version
Screenshots
No response
NuGet package version
Windows App SDK 1.6.1: 1.6.240923002
Packaging type
Packaged (MSIX)
Windows version
No response
IDE
Visual Studio 2022
Additional context
No response

Are you adding WinAppSDK to a UWP app? Or is the UWP part of the title incorrect?

Are you adding WinAppSDK to a UWP app? Or is the UWP part of the title incorrect?

Sorry, I have mentioned the wrong one in title. I am using the latest WinAppSDK package in my WinUI application and uploading the package in Windows Store. But it throws the error as mentioned above.

Thank you for the update on the app type, and title. Are you able to also provide us with:

a repro for this error.

Ok, I got it.
Problem was the "Microsoft.Windows.SDK.BuildTools" nuget package version 10.0.26100.
Downgrading to 10.0.22621.3233 has fixed validation problem on the Microsoft Store.

See also: https://github.com/microsoft/WindowsAppSDK/issues/4480

This issue has been addressed in the latest WebView2 SDK, available here: https://www.nuget.org/packages/Microsoft.Web.WebView2/1.0.2903.40
To work around this issue, add an explicit reference to the WebView2 package.  A future Windows App SDK release will include this reference.


Cannot Resolve Package When Installed from NPM

Hi there, I would like to use your project for an interactive tutorial on a business application I'm working on however for some reason it cannot resolve this package when I try to import userRoutine (as per the demo).
Not sure if this is something you've experienced yet as your demo appears to use the compiled scripts in /dist.
I'll probably just copy the built module out of /dist and use it inline in my project, but I'd much prefer to use this through npm.
Thanks!

Attempted to link in the contents of the dist folder in my React project directly and there are all kinds of errors. I'm moving on to another solution.


✨ Drivers License data should be returned as a parsed object and not raw

What feature or enhancement are you suggesting?
When you scan a barcode (in North America this is PDF417), it should return an object containing the various data fields (firstName, lastName, DOB, etc)
What Platforms whould this feature/enhancement affect?
iOS, Android
Alternatives/Workarounds
I could write a parser for the raw data in the JSX layer but the parsing is already supported by the MLKit and should be implemented at the native layer.
Additional information

[X] I agree to follow this project's Code of Conduct
[X] I searched for similar feature requests in this repository and found none.

Also, I can't release until this is done so I'm working on it right now.

Hey!
Sorry, no, this is not something VisionCamera should do in my opinion. It detects barcodes and decodes them into a string, from there on you can do whatever you want with that.
If I were to add all that kinds of data we'd end up with a really complex logic for all of that including tons of optional fields, special types, platform specific fields, etc. It's just not a generic approach anymore.
On a side note; I don't think you get the parsed data on iOS either, you'd have to start from the string value then parse it yourself.
JavaScript is most flexible for this, so I think parsing in JS makes more sense. If there's a bug in the parser, I'd have to fix it on the native side and you'd have to wait for a native change/update, whereas JS can be fixed instantly & over the air.

https://github.com/PeculiarVentures/js-zxing-pdf417

I want to take advantage of the MLKit parser and not write one at all.

Yes, but iOS does not use MLKit.

It could, but I understand now why you don't want to implement in that way.

yea thanks. I just don't want to implement very specific things, especially in this case we have a really cool stable generic solution, drilling into such specifics as drivers license will just make things more complex, both with types, providing support, fixing specific bugs in my native code, etc.
It's really easy to just get the string as it works today, and run it through some JS library that decodes it 😄


chore: update sctk

Updating this seems good. But didn't we want to pin a specific rev rather than using master so things depending on iced-sctk/cosmic won't break with changes to sctk master? Was that causing some issue here? If not, rev can just be set to the latest commit.


Fix AWS Batch submission to use Source ID

Overview
This change was mistakenly omitted from the refactoring commits where we changed all references to FacilityListItem.facility_list_id over to FacilityListItem.source_id. This was missed in our interactive testing because this code path is only executed when running on AWS.
Connects #892
Testing Instructions

Run this branch on staging
Log in
Submit a facility list. Verify that it is completely processed.

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Temporarily closing so that CI will deploy the branch.

With this branch deployed to staging I was able to submit a facility list and I saw the processing items correctly appended to the jsonb column.

openapparelregistry=> select jsonb_pretty(processing_results) from api_facilitylistitem where source_id = 106
;
                       jsonb_pretty
-----------------------------------------------------------
 [                                                        +
     {                                                    +
         "type": "parse",                                 +
         "error": false,                                  +
         "action": "submitjob",                           +
         "job_id": "5c5e3aec-0547-4a84-8cef-a200d2038357",+
         "is_array": false,                               +
         "started_at": "2019-10-24 20:37:35.436382",      +
         "finished_at": "2019-10-24 20:37:35.708236"      +
     },                                                   +
     {                                                    +
         "type": "geocode",                               +
         "error": false,                                  +
         "action": "submitjob",                           +
         "job_id": "83c76c9e-6d0f-4f31-ac75-31cd75c2cb16",+
         "is_array": true,                                +
         "started_at": "2019-10-24 20:37:35.713327",      +
         "finished_at": "2019-10-24 20:37:35.990303"      +
     },                                                   +
     {                                                    +
         "type": "match",                                 +
         "error": false,                                  +
         "action": "submitjob",                           +
         "job_id": "3a87ef67-ab20-4259-9995-ba9b9b10ef59",+
         "is_array": false,                               +
         "started_at": "2019-10-24 20:37:35.991654",      +
         "finished_at": "2019-10-24 20:37:36.444548"      +
     },                                                   +
     {                                                    +
         "error": false,                                  +
         "action": "parse",                               +
         "started_at": "2019-10-24 20:40:08.661007",      +
         "finished_at": "2019-10-24 20:40:08.665686",     +
         "is_geocoded": true                              +
     },                                                   +
     {                                                    +
         "error": false,                                  +
         "action": "geocode",                             +
         "started_at": "2019-10-24 20:40:14.867679",      +
         "finished_at": "2019-10-24 20:40:14.867693",     +
         "skipped_geocoder": true                         +
     }                                                    +
 ]
 [                                                        +
     {                                                    +
         "type": "parse",                                 +
         "error": false,                                  +
         "action": "submitjob",                           +
         "job_id": "5c5e3aec-0547-4a84-8cef-a200d2038357",+
         "is_array": false,                               +
         "started_at": "2019-10-24 20:37:35.436382",      +
         "finished_at": "2019-10-24 20:37:35.708236"      +
     },                                                   +
     {                                                    +
         "type": "geocode",                               +
         "error": false,                                  +
         "action": "submitjob",                           +
         "job_id": "83c76c9e-6d0f-4f31-ac75-31cd75c2cb16",+
         "is_array": true,                                +
         "started_at": "2019-10-24 20:37:35.713327",      +
         "finished_at": "2019-10-24 20:37:35.990303"      +
     },                                                   +
     {                                                    +
         "type": "match",                                 +
         "error": false,                                  +
         "action": "submitjob",                           +
         "job_id": "3a87ef67-ab20-4259-9995-ba9b9b10ef59",+
         "is_array": false,                               +
         "started_at": "2019-10-24 20:37:35.991654",      +
         "finished_at": "2019-10-24 20:37:36.444548"      +
     },                                                   +
     {                                                    +
         "error": false,                                  +
         "action": "parse",                               +
         "started_at": "2019-10-24 20:40:08.647303",      +
         "finished_at": "2019-10-24 20:40:08.652546",     +
         "is_geocoded": true                              +
     }                                                    +
 ]
(2 rows)

Thanks for the quick review.


no Installing !

Hello
I installed openjsk11 with Chocolate, in cmd there is a message that it is installing, but it is not installed!

This is the wrong place to raise this issue. Please raise the issue with the maintainer by clicking the 'Contact Maintainers' link on the package page or go to Discord and ask there.


Range quantifier error

    - name: "marriage-event"
      label: Marriage
      pattern: |
          trigger = [lemma=marry]
          spouse: Person{2} = dobj

works but
    - name: "marriage-event"
      label: Marriage
      pattern: |
          trigger = [lemma=marry]
          spouse: Person{2,4} = dobj

gives me the following error:
OdinError: Error parsing rule 'marriage-event': `}' expected but `,' found

Thanks, @bgyori.


travis: Use xenial as dist

Coverage remained the same at 63.286% when pulling 7261ddb12005f600b30995836cb6a35dc52e4ed4 on travis-xenial into 2a7d2f0150241851a6ef4f326f107d6f27d4c960 on master.


misc bugs 4

fixed:

wrong emoji on windows
header buttons behaviour on search focus

Add test for the hidden header buttons

@LeonmanRolls wasn't the concept to test for the bugs? if so, the header buttons being hidden was never the bug here

Add test for the hidden header buttons

@LeonmanRolls wasn't the concept to test for the bugs? if so, the header buttons being hidden was never the bug here

It came up as a bug because one would click the search bar and then the buttons would disappear, but the search bar didn't expand to take up the space.

It came up as a bug because one would click the search bar and then the buttons would disappear, but the search bar didn't expand to take up the space.

@LeonmanRolls wouldn't that mean that i want to test for the search bar expanding then?

It came up as a bug because one would click the search bar and then the buttons would disappear, but the search bar didn't expand to take up the space.

@LeonmanRolls wouldn't that mean that i want to test for the search bar expanding then?

Yes that's a more accurate way to say it


Feature-Request: Monero Support

Due to glacial transaction times and high fees I'd like to propose supporting Monero (XMR) besides BTC, BCH, LTC & CFX.

This is linked to:
https://github.com/mobazha/mobazha/issues/3


🚀 - Add an example of how to use tui-line-clamp with custom content

Description
Imagine that we need to display a user's email, which could be long. To truncate it, we can use tui-line-clamp. However, when the user hovers over the email, we want to show not just the full email, but also the user's ID, full name, email, and login. To achieve this, we came up with the idea of using a hint. But the issue is that tui-line-clamp and hint do not work together.
So It would be helpful if the documentation included an example of this this scenario.
Here are possible solutions:

Instead of using tui-line-clamp, use overflow: hidden and text-overflow: ellipsis to truncate the email.

Create a template for tui-line-clamp that displays the email on the first line, followed by the other user information on subsequent lines + limit the display to just one line.

hey! is it expected? is there no getBoundingClientRect logic?


Cargo is missing the .git from the crates io repo url

With the removal of CARGO_REGISTRY_INDEX we have to find another workaround for a problem we've been having living behind an enterprise proxy.
In order to escape through the proxy cargo needs to request
https://github.com/rust-lang/crates.io-index.git rather than the default of
https://github.com/rust-lang/crates.io-index.
I'm sure we can find another work-around, but I'm just wondering if the more standard thing is to have the .git on the end - I need to add that for normal personal repos that I clone, so I'm thinking that's the more standard approach?
Thoughts?

AFAIK this hasn't ever come up with github or the crates.io index. I believe GitHub treats the two URLs identically. Some services, like gitlab, use a redirect, but I don't think that is relevant here.
I think it would be very risky to change it at this time, as existing setups may be depending on the exact URL, and probably not worth the churn.  In the distant future we're looking at alternate ways to serve the index, so we may have to put more consideration into how to differentiate different protocols, but that's a ways off.
In terms of workarounds, one option is source replacement.  Something like the following should work:
[source.crates-io-git]
registry = "https://github.com/rust-lang/crates.io-index.git"

[source.crates-io]
replace-with = "crates-io-git"

Yep that work around works well for us. Just thought I should mention it as others might come across the same problem.
Sent with GitHawk

Btw I can see that adding .git on the end changes the hash dir name that it's stored as - I can see how that might be disruptive so let's leave as is.


upickle exception when there are multiple apply

in PR https://github.com/lihaoyi/upickle-pprint/commit/b8d897e8f589e2144d668d4b3978b371232eb591
Seems we now only support apply whose signature is same as default constructor, which is good. However, the unexpected thing is that if there are multiple apply, the pickling still doesn't work.
SInce there is only one apply with same signature as primary constructor, is it possible to select that apply when deserializing?
class GGraph[N, E](val vertexList: List[N], val edgeList: List[(N, E, N)])

object  GGraph {
  def apply[N , E](vertexList: List[N], edgeList: List[(N, E, N)]): GGraph[N, E] = {
    new GGraph[N, E](vertexList, edgeList)
  }

  def unapply[N, E](graph: GGraph[N, E]): Option[(List[N], List[(N, E, N)])] = {
    Some((graph.vertexList, graph.edgeList))
  }

  def apply[N, E](other: GGraph[N, E]): GGraph[N, E] = {
    new GGraph[N, E](other.vertexList, other.edgeList)
  }
}

//test
//==============
  "Graph" should "be able to serialize/deserialize correctly" in {
    val graph = new GGraph[Int, String](List(0, 1), List((0, "edge", 1)))
    val serialized = write(graph)

    val deserialized = read[GGraph[Int, String]](serialized)

    graph.vertexList.toSet shouldBe deserialized.vertexList.toSet
    graph.edgeList.toSet shouldBe deserialized.edgeList.toSet
  }

exception:
Error:(43, 49) type mismatch;
 found   : [N, E](other: org.apache.gearpump.util.GGraph[N,E])org.apache.gearpump.util.GGraph[N,E] <and> [N, E](vertexList: List[N], edgeList: List[(N, E, N)])org.apache.gearpump.util.GGraph[N,E]
 required: (List[Int], List[(Int, String, Int)]) => org.apache.gearpump.util.GGraph[Int,String]
    val deserialized = read[GGraph[Int, String]](serialized)
                                                ^

Bug bankruptcy


Issue with leaving from conversation

When I try to use leave function of conversation api I see next console errors:
GET https://api.layer.com/conversations/295070df-b8e8-4727-b357-c7a83ab5a5d5 404 ()
logger.js:70 Layer ERROR [18:34:24]: Layer-Error: 102 (not_found): A Conversation with the specified identifier could not be found.; (see https://developer.layer.com/docs/client/rest#not_found)
It also leads to issue with ConversationsList. I see some new conversation with No Title maybe for 1 second.
So it happened cause layer client got conversations:delete message and after update::conversation message. So first for removing conversation and second for removing participants... During second event - layer tried to load conversation which already removed from this user and  got error. Created little stub of conversation for loading time leads to error for ui widget.
I want to hide somehow this error for such case and prevent loading conversation. So I think about to change Conversation._loadResourceForPatch for such case but I'm not sure how to do it in more correct way, Could you please help?
I attached debug layer logs for next flow: creating conversation. sending one message. leave from conversation. Maybe it will be more clear.
layerDebuglogs.txt

OK, so the basic issue here is:

conversations:delete is triggered which deletes the local conversation
conversations:change is triggered on the deleted conversation just moments after it was deleted locally, forcing it to try fetching it from the server (undesirable behavior)
The Conversation Query immediately registers this as a real Conversation even though we know nothing about it other than that we are trying to fetch it (undesirable behavior)
UI widgets render whatever the Query returns (could ignore conversations that are loading, but better to assume the query only returns valid data)
Server returns error, destroying the newly created local conversation, and in turn updating the Query and the UI

Some workarounds on your end that don't fix the core bug but do fix the rendering glitches associated with the bug:

UI widget can filter out any Conversation whose participants are unknown (empty array)
You can add a conversations:delete event handler that tracks Conversation IDs that have been deleted, and use those IDs as a filter to avoid rendering Conversations with those IDs.
You might be able to add a conversations:add event handler that calls conversation.destroy() on any Conversation added that has been recently known to have been deleted.

Hey. So 1st option is not an option cause it's true for all history conversations. So I will try to implement 2) or 3). Right now I don't load conversation if patch data contains "remove" operation for current user.


Fetchdata sample code does not attach auth token

Before you open an issue
If the issue is with an ASP.NET Core document, open the issue with the Content feedback button at the bottom of that document page.
If the issue is:

A simple typo or similar correction, you can submit a PR. See the contributor guide for instructions.
A general support question, consider asking on a support forum site.
A site design concern, create an issue at MicrosoftDocs/Feedback.
A problem completing a tutorial, compare your code with the completed sample.
A duplicate of an open or closed issue, leave a comment on that issue.

Issue description
{ description here }
Software versions
Check the .NET target framework(s) being used, and include the version number(s).

[ ] .NET Core
[ ] .NET Framework
[ ] .NET Standard

If using the .NET Core SDK, include dotnet --info output. If using .NET Framework without the .NET Core SDK, include info from Visual Studio's Help > About Microsoft Visual Studio dialog.

dotnet --info output or About VS info
<replace>

Hello @prkhandelwal ... Please open this issue using the This page feedback button+form at the bottom of the English topic. It will add metadata to track the doc and any work that needs to be done. Thanks.


OpenAI Gym Breakout example

Build an agent which plays Breakout well.
This issue tracks this work.

OpenAI Gym Breakout environment
The Breakout-v0 environment allows us to play the classic Atari game Breakout using the OpenAI gym interface.
Relevant information about the environment:
>>> import gym
>>> env = gym.make('Breakout-v0')
>>> env.observation_space
Box(210, 160, 3)
>>> env.action_space
Discrete(4)
>>> env.unwrapped.get_action_meanings()
['NOOP', 'FIRE', 'RIGHT', 'LEFT']

This tells us that the environment is 210 x 160 pixels with 3 color channels, and that we can take one of four actions in the environment at any given point in time: NOOP, FIRE, RIGHT, or LEFT.


Graphics rework fixed

This PR significantly overhauls 32Blit's graphics underpinnings, notably:

Change the hi-res buffer from RGB565 to RGB888
Swap RGBA type for the generic Pen which creates a Pen compatible with RGB888 and Mask surface types

Things to note:

The FRAMEBUFFER now occupies 225k in RAM_D2. This is large enough to fit the 320x240 RGB888 and 160x120 RGB888 display back buffers.

This PR also drags a few quality-of-life changes kicking and screaming along with it. Chiefly some build settings and minor code tweaks to silence or remove some of the warning spam that builds are prone to.

There's a merge conflict in my commit. I have a correct version in my branch: e6b099e8d060f3d7e5b5fc61c11c855727652fd4.


ci: Add minimal-versions job

Reference is #23. I'm not sure I fully understand the implications of commiting the Cargo.lock file. So, for now, only the -Zminimal-versions test. It already captured a few issues in my projects, so it's probably good to already have this in CI.

This looks good, thank you!
I'll merge this and then clean up the CI issues if they persist :)


Added logo.

Can you debug to see if the problem is in loading the file or parsing the data? Let me know what I need to change.


Orgs/Projects should be filterable by text field

It's very difficult to navigate projects and orgs when the lists are too long

This has been done (after backend implemented search)


Verify if the container groups for the Bookinfo app were created succ…

…essfully in the bluemix example.
https://github.com/amalgam8/amalgam8/issues/295

What motivated the change from 256mb to 128mb on the containers?

Resolves #295.

The change to 128mb is so that the example runs under the default Bluemix quota.


[Divider] - Vertical Divider example not working in Chrome

Steps to Reproduce

Open chrome
go to http://semantic-ui.com/elements/divider.html

Expected
The Divider should go in the middle between the login form and the sign up button
Result
The Divider appears on the left of the login form. Its weird because it works fine in Safari and Firefox.

😱 I realized when you told me.
But looks like there's no problem with the mobile version.  > http://i.hizliresim.com/pXYjOn.png

#4342
I've posted a bug with Chromium. There does not appear to be a workaround from a framework level. Hopefully Chrome team can fix this soon.
View Chrome Issue 633011
https://github.com/w3c/csswg-drafts/issues/401


sudo su doesn't work

As stated in the Releases page with the commands to install Borealis, sudo su isn't compatible and throws an error. sudo -i works.

LaCros also doesnt work anymore.
Steam via borealis cant be installed due to sudo -i changed.
Most wifi modules are not installed in kernel or have compatability issues...
Fyde has plenty of work in hands to do..

No wifi or bt with my trl8822be.


PR's from last year showing up in calendar

Seems like pr's from last year show up in the calendar.  I made only one pr today but it filled the gifts calendar until Dec. 7 with pr's from 2014.

Yep definitely a bug, investigating this at the moment, pretty sure it's my fault!

@markus1189 I think I've fix that now, can you confirm?

@petems can you take a look at your profile and see if it's been fixed?

The old pr don't show up anymore :+1:
On Dec 3, 2015 18:10, "Andrew Nesbitt"<EMAIL_ADDRESS>wrote:

@markus1189 https://github.com/markus1189 I think I've fix that now,
can you confirm?
—
Reply to this email directly or view it on GitHub
https://github.com/24pullrequests/24pullrequests/issues/1005#issuecomment-161718852
.

Mine have gone too! :smile:


Do you plan to add page change event?

Currently if i use paging i do not have any event that page was changed.

Good point. It makes sense to be able to handler it.
onRefreshing callback and read option value pageIndex


DBZ-5838 Skip Avro validation in tests which fails due to apicurio-re…

…gistry #2980 and #2990

test failure is unrealted


Task 4: completed the task 4

@Aswani-P
Please try to name you own  working branch up to our standards.
Use dev/aswani/task-4 instead of dev/aswani-task-4
(Also found so in your last task #3 )
cc : @VeldasRDurai


Cannot read property M_modal of undefined

Expected Behavior
I guess technically everything is working, but it's a little annoying to see errors in my console.
So the expected behavior is a console free of errors
Current Behavior
When clicking an anchor element with the class modal-trigger, and no assigned href attribute, I see the above error in my console.
I am using jQuery to open the modal programatically ($('#modalid').modal('open')), and I am opening different modals based on conditions on the page. If I assign the modal-trigger an href attribute, the console error disappears but then I have a modal opening when I don't want it to. If I remove the href attribute, my page functions as I would expect (the modals open correctly based on their assigned conditions) but I have the above error in my console. If I remove the class "modal-trigger" from my anchor element, the modals no longer open on click.
Steps to Reproduce (for bugs)
You can see the behavior here (open the console to see the error obviously, not the best example but it works well enough)
https://jsfiddle.net/5f528dgp/
When you click the button, the code decides whether to open modal1 or modal2 based on specified conditions. This works as expected, but produces an unnecessary console error.
If you set the href attribute of the anchor to either modal1 or modal2, the specified modal will always open and the other modal may also open based on the conditions. This removes the console error, but is undesired behavior.
Context
I get why this error exists, as I'm sure most users are simply assigning the buttons to specific modals. It really is inconsequential in the grander scheme, since I can get the desired behavior without issue. It's just seriously annoying to see that console error when I know it doesn't need to be there. If I'm doing something wrong please correct me, and I'm sorry for wasting anyone's time. I love love LOVE this CSS framework, it is gorgeous and a joy to use. Keep up the excellent work!
Your Environment
Just an HTML file, with jQuery 3.2.1, materialize.css, and materialize.js loaded through CDNs.

Just realized that if I set the href attribute to the modal I want to open at the same time I call .modal('open') the console error disappears.  Try just using a plain <button> and bind your click handler as you do.

Testing the TechListModal.js after adding it to App.js; when clicking the list button I am getting the error 'materialize.min.js:2035 Uncaught TypeError: Cannot read property 'M_Modal' of null
at HTMLBodyElement.value (materialize.min.js:2035)' without any other helpful console errors. Anyone else encounter this?


Patch 1

Added a new page for how to suggest a feature and related image assets.  Updated TOC to include the page.

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

@seaniyer : Thanks for your contribution! The author, @, has been notified to review your proposed change.

@seaniyer Thanks for adding this topic! I noticed that it's missing the metadata section at the top. I'll also shoot you an email about using the private repo.

This PR was moved to the private repo.


Confirm or Prevent Last Tab Close

Description of the new feature/enhancement
Sometimes when completing a session with multiple tabs open, a user may want to close everything but still keep a tab open.
There are two possible alternatives (when the option is true):

When closing last tab, prevent it from closing.

When closing last tab, present a confirm dialog.

When the option is false/default - last tab closes app as per current behaviour.
Proposed technical implementation details (optional)
Add confirmCloseLastTab or preventCloseLastTab to profile.json

Thanks for the request.
Would this happen when the user closes almost all of the tabs, and then manually closes the last one? Would it happen when they click the [x] or press Alt+F4.. which would close all the tabs except for the last one?
Right now, the user must explicitly agree to close all tabs, or manually close the final tab themselves (because there's no other way to close tabs)... so I am a little confused :smile:

What about an option for not closing the window when the last tab is closed? Same as Firefox's "closeWindowWithLastTab"?
Offtop: I've sent a similar feature request to MS Edge's team.

Thanks for the request.

As mentioned by @electronic-dk above, I was picturing this as similar to how browsers handle the process.
It usually takes effect from keyboard shortcuts to close tabs (Ctrl-W or similar). If you hold the shortcut down (or press the shortcuts in succession) you'll end up with an empty window or a default tab rather than the application closing.
It's basically a process to say I still want to do something, but I want to start fresh.

Hey this is also related to #2976, I'm commenting here to make sure I don't lose that issue again

I am using the terminal app together with the WLS and R.
If I am in a similar situation using Ubuntu the terminal asks if I close a tab that contains a running process. If I have a resting tab with just the command prompt waiting for some input I can close the tab without any questions, but if I have an R or ssh session running I get the confirmation dialogue.
I think this would actually be the best way to make everybody kind of happy - right?

@stela2502 while I think your suggestion is pretty useful (there might be a ticket for it actually, but I'm not sure), I think what OP meant was: "Avoid closing the entire terminal window when the last tab of the said window is closed".

You are absolutely right, but from a usability point of view I would honestly just re-open an anyhow empty (all tabs are closed) terminal.


Code of Conduct

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

Ran into the same issue. In an expression you can use \u00a0.

Is there a reason why you use that instead of <>{" "}</>?

Is there a reason why you use that instead of <>{" "}</>?

My phone has actually perfectly demoed why:

Regular space can cause unwanted line-breaks.

But do you have a demo in a react JSX project that causes that?

I don't see why the playground link supplied with the issue is insufficient

I don't see why the playground link supplied with the issue is insufficient

Two reasons:

I want to understand the use case
I didn't understand the answer to my initial question, because I don't understand how GitHub rendering some markdown is related to JSX. That's why asked for a reproduction, that shows the differences

I want to understand the use case

Here's one:
import * as React from "react"

const myStrings = [ "foo", "bar", undefined, "baz" ] as const

export default function TestPage() {
  return

    {myStrings.map((value, index) =>

        {value ?? <> </>}

    )}

}

The blank line is desired. With " ", the space is perceived by the HTML parser as structurally insignificant whitespace, so its enclosing
 is considered empty and does not render at all.

I don't understand how GitHub rendering some markdown is related to JSX

That was something of a jest on my part, so I'm sorry it confused you.
You intended to convey <>{" "}</>but the content appeared to me as <>{"
"}</>, which illustrates why the use of   can be necessary.

i'll work on this.


Split Interview questions by category to make custom interviews and training easier

The  current organization is too cumbersom and gets in the way of training on why questions are asked, and how they can be leveraged.

See also #16

Closed as no longer relevant since the interview section has been removed.

It was actually just moved to be an activity... my bad.

The supporting appendix documents (https://github.com/OpenInternet/SAFETAG/blob/master/en/exercises/interviews/interview_questions.md) and capacity assessment cheat sheet (https://github.com/OpenInternet/SAFETAG/tree/master/en/exercises/capacity_assessment_cheatsheet) address this need


Disable-CMTaskSequence

Disable-CMTaskSequence is Depricated and should be removed from Related Links

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 188952b9-b36a-d955-32bf-c7a20e7c0166
Version Independent ID: adc9e119-4e52-440e-83d2-753494a0d4fa
Content: Set-CMTaskSequence (ConfigurationManager) - Configuration Manager
Content Source: sccm-ps/ConfigurationManager/Set-CMTaskSequence.md
Product: configuration-manager
Technology: configmgr-other
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @Barleyologist, good feedback. It looks like it's linked from several other articles, the same as Enable-CMTaskSequence. I'll remove both of those links, and do a search for other similar cmdlets to cleanup. 😄 This issue will auto-close when my change is merged, which means it will publish live with the next daily publishing process (~4 PM Pacific)


Investage windows ci failure

Windows ci started to fail , though the tests seems to be passing but somehow it exit with code 1

The error code is -1073741819

Storing the ptypair (and in particular not dropping the master quickly) seem to have triggered this issue on windows
introduced in this commit https://github.com/sigmaSd/deno-pty-ffi/commit/2e4a2c6308dbe5e21cb03e3233ab61c2f02da59d

Somehow changing plug dlopen to use instead of realoadAll fixes the ci https://github.com/sigmaSd/deno-pty-ffi/blob/master/src/mod.ts#L88

Also note pty on windows is already weird, I already have a couple of issues opened https://github.com/wez/wezterm/issues/4206 https://github.com/wez/wezterm/issues/4205

Debugging random ideas on this branch https://github.com/sigmaSd/deno-pty-ffi/tree/windbg

Might be related https://github.com/denoland/deno/issues/20956

Fixed or maybe worked-around by https://github.com/sigmaSd/deno-pty-ffi/commit/c3178ba22a50edc095b392bd103385713dd43cd1


New translation for fr

Please take in account new file provided

Duplicate key "Cancel" on lines 69 and 128. Can you fix it?
Then can you rebase this PR against development please so I can merge it?

Hi brenopolanski, please have a look to my PR #589


Package and publish new Flat Data Grid renderer release

added in #115, and Rest books patch handled in #114

released: https://github.com/RandomFractals/vscode-data-table/tree/v1.9.0


Memory Leaks and suggestions

I would like to bring to your attention an issue regarding the implementation of view binding inside fragments. It has been observed that the instance of the binding is not being destroyed in the onDestroyView() callback, which can lead to memory leaks. For more information, you can refer to the documentation on using view binding in fragments.
As a suggestion, I would recommend exploring the use of view binding in the EventAdapter class for ViewHolder.
During my recent work with viewPager2, I encountered issues with memory leaks caused by not properly destroying the instances of the TabLayout and the Adapter. I suggest we review the implementation in the HomeFragment and consider passing the viewLifecycleOwner to the adapter, which would make it lifecycle aware.
Additionally, I would like to suggest the inclusion of tools such as LeakCanary and KtLinter to aid in detecting and preventing memory leaks and ensuring consistency in the codebase.

Btw, if you'd like to try out something/contribute do lemme know!

I am happy to take on these issues. For your convenience, would you like me to create a separate branch for each of them or make all the changes in a single branch?

Hey @Krish-Parekh, thanks a lot!
you can make all changes on the same branch and make the PR directly to master, that'll be fine.


Cannot form weak reference to instance (0x7ff94e02a000) of class LocationPicker.LocationPickerViewController. It is possible that this object was over-released, or is in the process of deallocation

Error Happen After Latest Xcode Update when initiating LocationPickerViewController

@merovic please provide details of your implementation and steps to reproduce this issue.

I only starts to initialize LocationPickerViewController like ( let picker = LocationPickerViewController ) and then starts to follow the sample code like the readme file and everything was okay until recent update to Xcode and it is no longer working and this error emerges

I can't reproduce this with the demo app using Xcode 11.2. Please provide more details of your usage.

I used it before many times and everything works as expected until yesterday everything stopped working and that error emerges, I tried to use it manually instead of cocoa pods but the same error too, error emerges directly even if I just initialized the controller without any usage

try to use the library using cocapods in a new project in order to reproduce the error

@merovic Please provide a sample project, thanks.

TestLocationPicker.zip

Thanks @merovic, fixed in 1.4.2.

erro

Error -- Xcode build done.
Failed to build iOS app
Swift Compiler Error (Xcode): Stored properties cannot be marked potentially unavailable with '@available'
../ios/Pods/LocationPicker/LocationPicker/LocationPickerViewController.swift:59:5


Add glossary tooltip and references in the Nodes page

This is a Feature Request
What would you like to be added

Below are terms that can use the addition of a glossary tooltip to show the meaning of the said term.
statefulset
workload
kube-controller-manager

Below points can be added to What's next section of Nodes page which will help users for quick follow-ups.
Node Resource Managers
Resource Management for Windows nodes

Why is this needed
These references can help user understanding the concept while staying on same page or finding more about the concept with right link. it will enhance user experience and bring more clarity and less confusion.
Comments
Page to update:  https://kubernetes.io/docs/concepts/architecture/nodes/

/sig docs
/ language en
/assign

If triage is accepted, i would be happy to contribute on it. PTAL.

/triage accepted


add tensor_hash

Summary:
This adds a tensor hash function for multi-dim tensors.
It's purpose is to allow easy full tensor verification on forward tensors for other unit tests
by storing the hash of the answer tensor.
This strikes a middle ground between simple verification (i.e. first row, mean of an axis) and every item in the tensor verification which would require storing the entire tensor in a unit test.
Usage =
hash the answer tensor, save in your unit test.
Run a forward in unit test, hash the output tensor.
Compare hashed tensors to confirm unit test result.
Test plan:
Verified a + .01 change to single item in a multi-dim tensor is detected, verified identical tensors generate identical hashes.
Since this is a unit test function, not sure that unit test for a unit test function is needed as will not change unless function itself is modified.
Fixes #{issue number}

unit test failure is not related to this PR

Could you provide a little more context for this function? It seems to compare sums for the final dimension, does this provide more precision than mean? It's just one extra dimension of data. I guess this would be more likely to catch tensors that had the right data but the wrong shape, is that the primary use case? To add clarity to my question, I am wondering what the potential mistakes are that are not caught by mean or mean(-1) and if this implementation of a hash catches all of them.

Hi @pbontrager,
My concern with the mean test is that it's an average, and multiple combinations of numbers could potentially end up at the same mean, so I felt the mean test alone is not sufficient to fully check a tensor.
I can't guarantee that a hash will catch all issues and probably using both would result in the best coverage, but I show a simple example below where using the mean fails to detect the difference of .01 to an entry,  vs the hash does using the default all_close settings that are in the multi-modal tests.
Here adding .01 to a single entry is not detected by mean (see line 18 allclose = True), but is detected with the hashing.


Can You Put People in your botnet or ma I just dumb??

I have tried multiple times to put bots/people in my botnet can someone help me??

i am sorry but i didn't get it.

i will try to make a video and and post the link soon, the updated version of this botnet is incoming.


fix: eslint startline can be zero

In case tsconfig does not contain correct setup, e.g., "strictNullChecks": true, eslint throws warnings for line "zero". The current schema does not show the warning, it just throws an internal error. This fix allows for warning on line zero to be collected for the report.
Fixes issues such as:
{
  "code": "too_small",
  "minimum": 0,
  "type": "number",
  "inclusive": false,
  "exact": false,
  "message": "Number must be greater than 0",
  "path": [
    117,
    "details",
    "issues",
    3,
    "source",
    "position",
    "startLine"
  ]
},

for issues like
0:1  warning  This rule requires the strictNullChecks compiler option to be turned on to function correctly  @typescript-eslint/prefer-nullish-coalescing

@matejchalk I changed PR to implement first solution.
While I like the second solution to throw an error, it is against the idea of "monitoring" the progress for transformation as one has e.g., to fix some projects in monorepos to have the rules in place.


Deps + remove jest*jsdom-sixteen

fix yarn (npm) test due to duplicate versions of jes
minor README

https://github.com/tgve/tgvejs/commit/45fc1f815fda7df2f2fad84c76e20502ebe2b723 etc for overall data change.

[ ] components must share data parsing as much as possible
[ ] the rule should be "parse as you go"
[ ] central parsing is against above rule


Feature/container ssl

This enables the "SSL by default" behavior in the openstack-cluster-setup roles branch, by building any local certificates into the docker images when they're generated.

roles builds the cord-pod config properly with this enabled.

Verified that the branch passed E2E cord-pod test before merging.


⚠ SolarOscp has degraded performance

In f8d1e32, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 396 ms

Resolved: SolarOscp performance has improved in 7d0dfdd after 10 minutes.


feat(topics): added topic 12 animation

#99
The animation is now using the functions in canvas.js.

The issue that vertices might be out of boundary is not solved yet.

Do you plan to merge this commit before the issue is resolve or after? We can do either.

I will add some information on the graph later. We can merge it after I added them on.