temporary title: objects in slice not properly updated when iterating on them

The following program sample.go triggers an unexpected result
package main

import (
	"bufio"
	"flag"
	"fmt"
	"strconv"
	"strings"
)

var debug = flag.Bool("debug", false, `debug mode`)

type monkey struct {
	items        []int
	operation    func(int) int
	test         func(int) bool
	monkeyTrue   int
	monkeyFalse  int
	inspectCount int
}

func parseOperation(operation string) func(int) int {
	parts := strings.Split(operation, " ")
	self := false
	if parts[1] == "old" {
		self = true
	}
	operand, _ := strconv.Atoi(parts[1])
	sign := string(operation[0])
	if sign == "+" {
		return func(i int) int {
			if self {
				return i + i
			}
			return i + operand
		}
	}
	if sign == "*" {
		return func(i int) int {
			if self {
				return i * i
			}
			return i * operand
		}
	}
	panic(fmt.Sprintf("unsupported operation: %s", operation))
}

var input = `Monkey 0:
  Starting items: 79, 98
  Operation: new = old * 19
  Test: divisible by 23
    If true: throw to monkey 2
    If false: throw to monkey 3

Monkey 1:
  Starting items: 54, 65, 75, 74
  Operation: new = old + 6
  Test: divisible by 19
    If true: throw to monkey 2
    If false: throw to monkey 0

Monkey 2:
  Starting items: 79, 60, 97
  Operation: new = old * old
  Test: divisible by 13
    If true: throw to monkey 1
    If false: throw to monkey 3

Monkey 3:
  Starting items: 74
  Operation: new = old + 3
  Test: divisible by 17
    If true: throw to monkey 0
    If false: throw to monkey 1
`

func main() {
	flag.Parse()
	/*
		f, err := os.Open("./input.txt")
		if err != nil {
			log.Fatal(err)
		}
		defer f.Close()
	*/

	f := strings.NewReader(input)
	sc := bufio.NewScanner(f)

	var monkeys []*monkey

	debugCount := 0
	for sc.Scan() {
		if *debug {
			if debugCount > 34 {
				break
			}
		}
		debugCount++

		l := sc.Text()
		if !strings.HasPrefix(l, "Monkey") {
			panic("BIM")
		}
		kong := monkey{}

		sc.Scan()
		l = sc.Text()
		items := strings.Split(strings.TrimPrefix(l, "  Starting items: "), ", ")
		for _, v := range items {
			item, _ := strconv.Atoi(v)
			kong.items = append(kong.items, item)
		}

		sc.Scan()
		l = sc.Text()
		operation := strings.TrimPrefix(l, "  Operation: new = old ")
		kong.operation = parseOperation(operation)

		sc.Scan()
		l = sc.Text()
		if !strings.HasPrefix(l, "  Test: divisible by ") {
			panic(fmt.Sprintf("unsupported test: %s", l))
		}
		divisor, _ := strconv.Atoi(strings.TrimPrefix(l, "  Test: divisible by "))
		if *debug {
			println(len(monkeys), "DIVISOR: ", divisor)
		}
		// TODO: we should defer, and make it directly return the monkey destination
		kong.test = func(i int) bool {
			return i%divisor == 0
		}

		sc.Scan()
		l = sc.Text()
		kong.monkeyTrue, _ = strconv.Atoi(strings.TrimPrefix(l, "    If true: throw to monkey "))
		if *debug {
			println(len(monkeys), "MONKEYTRUE: ", kong.monkeyTrue)
		}

		sc.Scan()
		l = sc.Text()
		kong.monkeyFalse, _ = strconv.Atoi(strings.TrimPrefix(l, "    If false: throw to monkey "))
		if *debug {
			println(len(monkeys), "MONKEYFALSE: ", kong.monkeyFalse)
		}

		monkeys = append(monkeys, &kong)

		sc.Scan()
	}

	for i := 0; i < 20; i++ {
		for idx, mk := range monkeys {
			if *debug {
				println("MONKEY ", idx)
			}
			for k, item := range mk.items {
				if *debug {
					println(k, "OLD WORRY: ", item)
				}
				worry := mk.operation(item)
				worry = worry / 3
				// mk.items[k] = worry
				if *debug {
					println(k, "NEW WORRY: ", worry)
				}

				monkeyDest := 0
				if mk.test(worry) {
					monkeyDest = mk.monkeyTrue
				} else {
					monkeyDest = mk.monkeyFalse
				}
				if *debug {
					println(k, worry, "THROWING TO MK ", monkeyDest)
				}
				monkeys[monkeyDest].items = append(monkeys[monkeyDest].items, worry)
				mk.inspectCount++
			}
			mk.items = []int{}
		}
	}

	for i, mk := range monkeys {
		println("Monkey", i, "inspected items", mk.inspectCount, "times.")
	}

	// Answer: 54752
}

Expected result
% go run ./main.go
Monkey 0 inspected items 101 times.
Monkey 1 inspected items 95 times.
Monkey 2 inspected items 7 times.
Monkey 3 inspected items 105 times.

Got
% yaegi run ./main.go
Monkey 0 inspected items 99 times.
Monkey 1 inspected items 97 times.
Monkey 2 inspected items 13 times.
Monkey 3 inspected items 108 times.

Yaegi Version
eee72d1aae664bf6627ef955215d886dc7b105c0
Additional Notes
I was solving the 11th advent of code: https://adventofcode.com/2022/day/11
and I noticed the result given by yaegi is different from the one with the go compiler.
I haven't had time to narrow it down yet, and to write a small repro. However, I strongly suspect the bug is connected to:
				monkeys[monkeyDest].items = append(monkeys[monkeyDest].items, worry)
as the rest of the code is completely trivial.
I will update this issue with a better repro and title later.

Alright, here's the actual bug, which actually has nothing to do with what I was suspecting:
package main

import "strconv"

type monkey struct {
	test func() int
}

func main() {
	input := []string{"1", "2", "3"}

	var monkeys []*monkey

	//	for k, v := range input { // <-- no problem if we assign k to divisor, for example.
	for _, v := range input {
		kong := monkey{}

		// divisor := k <-- no problem
		divisor, err := strconv.Atoi(v)
		if err != nil {
			panic(err)
		}
		// divisor, _ := := strconv.Atoi(v) <-- also buggy
		println(divisor)
		kong.test = func() int {
			return divisor
		}
		monkeys = append(monkeys, &kong)
	}

	for _, mk := range monkeys {
		println(mk.test())
	}

}

% go run ./issues/1497/main.go
1
2
3
1
2
3
%
%
%
% yaegi run ./issues/1497/main.go
1
2
3
3
3
3

So I believe it's the combination of the double return/assignment (from strconv.Atoi in that case), and the use in a closure, that leads to the bug. If the assignment to divisor does not come from a func with a double return, then there's no  problem.


Fix example searching

Addresses issues found when running through search for each project.

[x] distribution search
[x] static search
[x] colorizer search
[x] alignment search
[x] classification search
[x] chargan search

This is already in 0.10, closing this pr


Question list page content enhancement. Add a two-line preview of article content below the title.

Is your feature request related to a problem? Please describe.
Question list page content enhancement. With only one title, it looks a bit thin.
Describe the solution you'd like
Add a two-line preview of the article content below the title.
The number of lines in the article content preview can be set in the settings.
This preview feature can be turned off or on in the settings.

Hi @Liamn0244 ,
I would like to pick up this taks. Can you assign this to me?

@LinkinStars @ivowolters
Tasks need to be assigned by LinkinStars.

Thank you @Liamn0244. This feature is quite subjective, and I need more opinions from other people.


Unable to connect with WalletConnect

I need to integrate my dapp with WalletConnect using the wallet selector.
When I scan the QR code using Trust Wallet (Android and iOS devices), I get a message saying that it will take a few second to process, but after some minutes it does not connect.
I also tried with Math Wallet (Android app) and Onto Wallet.
Is there any working example that uses WalletConnect with NEAR wallet selector?
Here is a short video with the issue
https://www.loom.com/share/8f80467c54b141519164a73f26bb4632
Thanks

Hey, @leomanza thank you for raising this issue.
At the moment there is no official Wallet that supports WalletConnect + NEAR.
If you need to test WalletConnect with NEAR we already have an implementation in near-wallet-selector/wallet-connect
You may find this PR useful: https://github.com/near/wallet-selector/pull/441 in the description there's a link to NEPs PR in the description there it's explained how to test out WalletConnect + NEAR with Wallet Selector.

Thanks @kujtimprenkuSQA!
Do you know if any wallet is planning to work on a NEAR provider for WalletConnect?
Thanks!

Thanks @kujtimprenkuSQA! Do you know if any wallet is planning to work on a NEAR provider for WalletConnect? Thanks!

Hey @leomanza , we can share more info on that soon. It was planned to be worked on. We will keep you posted.
I will close this issue for now. Thanks!


Projects detail window

This branch is dedicated to developing a window with detailed information about every project, the features are the following:
-A popup window is generated when the user clicks on "See project" button.
-The popup window has the corresponding information for every section.
-The popup windows close when you press the "X" icon.
-The popup windows have been adapted to work properly for mobile and desktop version.
-An object is used to store the information of all popup windows, when the user clicks on the "See project" button, the information is then selected depending of the work section from the object and displayed correctly, saving some lines of code.
-Made sure to make image sizes, buttons, etc, as close as possible to Figma template.

Summary of new changes:
-Deleted previous code and replaced the auto generated object with a group of objects that will correctly display the object information. There are no more empty objects.
-Added a feature to the "See live" and "See source" button. It will now properly direct you to the corresponding website on click.
Hope you like the new changes!


chore Split the compliation units to build them in parallel

Proposed changes
Issue Number: close #xxx
Problem summary
After investigate the intermediate object files generated by GCC, we can figure out that
vhash_join_node.cpp.o and function_date_or_datetime_computation.cpp.o are the first and second biggest objects. It took too much time to generate them.
This PR introduces a way to speed the building process up. We can split original source files (.cpp) into smaller ones. As a result, more translation units will be generated, but we can build them in parallel and save the times.
Result
time EXTRA_CXX_FLAGS='-O3' BUILD_TYPE=ASAN ./build.sh --be --fe --clean -j 32

Before: more than 1 hour.
After: around 35 minutes.
time ./build.sh --be --fe --clean -j 32

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 37.02 seconds
load time: 484 seconds
storage size:<PHONE_NUMBER>9 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221113160706_clickbench_pr_45229.html

LGTM


first attempt at transitioning to cookiecutter project structure

Hey all the data should be on data.world within the zip file. Everything on github should only be the code. I'm not sure if this would cause issues with the cookie cutter format. I imagine we might have to change the file tree of everything on data.world separately to be compliant with the cookie cutter data science format

If having the data under revision control isn't the way to go, there is provision in cookiecutter to sync it from S3, is that a better option?
http://drivendata.github.io/cookiecutter-data-science/#data-is-immutable

Other than the data directory issue already raised, this seems like a good start.  Maybe you can just take out the data directory for now, and then we can have the where does data go conversation separately.  That way we can move forward on dealing with the code.
Regarding the data directory, I'm not sure about S3.  Even though they have a free option for at least 12 months, I'm not sure we can do it without giving them a credit card (in case we go over the free limits).
Another thing we could consider is git's large file support: https://help.github.com/articles/working-with-large-files/


Add a new job to redownload a register

Context
We want to be able to redownload a register from scratch with zero downtime (trello).
We need this in two situations:

For discovery/alpha registers, we don't enforce the append-only nature of the register. So when we go back and change history, we need to fetch everything again. This one we can do already as we don't care about downtime.

The java implementation stores system entries seperately, and these are not currently retrievable as part of a partial update. So as a workaround we need to rebuild the register in registers-frontend whenever new system entries are added. This we can't do now.

Changes proposed in this pull request
I've added a job which deletes a registers existing records, nullifies its root hash, and redownloads the register, inside a transaction.
The existing PopulateRegisterDataInDbJob job now calls the new job as a fallback if the register is invalid. This is equivalent to the existing behaviour - which would delete the data and then retry the job - but it does it inside a transaction.
I've factored out the part that calls the client library into a new class, and I've removed the refresh_data call, because I don't think we need it. The client library already calls this when you initialise a register client, so it was attempting to fetch data twice, which complicated the tests.
Guidance to review
I've tested this locally by manually changing the root hash of a large register (charity) in register-frontend's database, then running the job. The website still works while the job is running, and after the job has run the root hash is correct and there is the correct number of records.

@gidsg can you give this another review?


How to run executables on mac

Executables need ./ before their name in order to be executed

This PR is obsolete.


[shared_preferences] Fix deprecated API call

Right now, while building, users get this warning:
C:\Users\Cristian\AppData\Local\Pub\Cache\hosted\pub.dartlang.org\shared_preferences-0.5.6+1\android\src\main\java\io\flutter\plugins\sharedpreferences\SharedPreferencesPlugin.java:25: warning: [deprecation] getFlutterEngine() in FlutterPluginBinding has been deprecated
    setupChannel(binding.getFlutterEngine().getDartExecutor(), binding.getApplicationContext());
                        ^
1 warning

I think this has rolled to stable now so it can be fixed to not use a deprecated API.
Fixes https://github.com/flutter/flutter/issues/48211

LGTM thanks!

I don't know much about the Java version of plugins, but if @cyanglaz approves, it's probably OK!
(@cyanglaz ping me if you need a hand merging/publishing this)

@ditman This is calling a depreciated API in engine and since the new API is in stable, we can use the new API.
Thanks for offering to help! I will merge it as soon as the post-submit tree goes green :)

LGTM

Please accept this pr, the message is annoying and I need the plugin to my app

This is already tagged as "submit queue", it'll get merged.

@cyanglaz could you please review the patch? Would be very helpful.

Hi @Zazo032 are you available to work on this PR a bit further? The proposed change fixes the problem but is part of a bigger change we need to do for each plugin.
You can refer to this PR for the description / TODO list: https://github.com/flutter/flutter/issues/47153
You can refer to this PR for an complete example: https://github.com/flutter/plugins/pull/2590.
Thank you!

Our task list for cleanup on the plugins is a bit longer as @ened pointed out. But this is ok to merge incrementally for now.

I am still getting this, was it merged?


18486 - Pinia Upgrade

Issue #: /bcgov/entity18486
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Could you rebase this PR please?

created a new PR: https://github.com/bcgov/fas-ui/pull/195


[Dashboard] Add tokenURI section to the NFT page

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR enhances the TokenIdPage component by integrating a feature to display the Token URI. It adds a link to the Token URI, allowing users to easily access it.
Detailed summary

Added import for useThirdwebClient.
Imported ExternalLinkIcon and Link from next/link.
Created a new section displaying the Token URI.
Added AddressCopyButton for copying the Token URI.
Implemented a link that resolves the Token URI using resolveScheme.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5035  👈
main

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

Join @kien-ngo and the rest of your teammates on  Graphite


General and mongo-mocking specific doc. improvements

failing tests is no reason to stop at this point


Update "fr" language

I had issues (blank messages) when using DoIt in french:
The poweredBySome  translation key was replaced by both poweredBy and theme.

It is greate, thanks !
C'est super merci !


Error Message

Command that caused the error
&fakeping
Image that caused the error
No response
Steps to reproduce

Go to 'Discord channel and run the command &fakeping '
See error

Self-hosted instance?
No
Error file
Message: FetchError: request to http://balls.dm4uz3.pt:2333/loadtracks?identifier=https://raw.githubusercontent.com/esmBot/esmBot/master/assets/audio/ping.ogg failed, reason: connect EHOSTUNREACH <IP_ADDRESS>:2333
    at ClientRequest.<anonymous> (/home/esmBot/esmBot/node_modules/node-fetch/lib/index.js:1461:11)
    at ClientRequest.emit (node:events:394:28)
    at Socket.socketErrorListener (node:_http_client:447:9)
    at Socket.emit (node:events:394:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  type: 'system',
  errno: 'EHOSTUNREACH',
  code: 'EHOSTUNREACH'
}

Stack Trace: FetchError: request to http://balls.dm4uz3.pt:2333/loadtracks?identifier=https://raw.githubusercontent.com/esmBot/esmBot/master/assets/audio/ping.ogg failed, reason: connect EHOSTUNREACH <IP_ADDRESS>:2333
    at ClientRequest.<anonymous> (/home/esmBot/esmBot/node_modules/node-fetch/lib/index.js:1461:11)
    at ClientRequest.emit (node:events:394:28)
    at Socket.socketErrorListener (node:_http_client:447:9)
    at Socket.emit (node:events:394:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)

Additional context
No response

Seems that one of the image servers went down, I've removed it from the list for the time being. Will re-add it once it comes back up.


Protocols as Projects

As @NiteshKant and I have been playing with code this week it seems that we should separate protocols as projects (repos), not just modules. This would allow them to have different release and versioning cycles, which is important as the coming years will evolve and add things.
We suggest that instead of "reactive-ipc-jvm" being the home of everything, we add top level projects as protocols arrive such as:

ripc-tcp
With submodules as ...

ripc-tcp-spi
ripc-tcp-transport

ripc-tcp-transport-netty4
ripc-tcp-transport-netty5
ripc-tcp-transport-aeron
ripc-tcp-transport-etc...

ripc-tcp-apis

ripc-tcp-rxjava1
ripc-tcp-rxjava2

ripc-http
With submodules as ...

ripc-http-spi
ripc-http-transport

ripc-http-transport-netty4
ripc-http-transport-etc...

ripc-http-apis

ripc-http-rxjava1
ripc-http-etc...

ripc-udp

ripc-kafka

ripc-cassandra

ripc-dynomite

ripc-memcached

etc

This also makes more sense for transports since there is little to no commonality across protocols. For example, Servlet 3.1 as a transport can only be used for HTTP.

Considering the disparate protocols, it makes sense to separate. That said would there be no shared code across protocols (working with buffers, connection lifecycle handling, etc) and across transport implementations? Or do you see the TCP module as that lowest common denominator?
Would WebSocket be under ripc-http? A WebSocket connection starts with an HTTP handshake and is likely mapped (and secured) under a shared URL hierarchy within a (single) Web application process.  On the other hand beyond that the actual handling of WebSocket is much closer to TCP.
There is nothing to reflect the JVM nature any more with the "-jvm" suffix gone. Perhaps that suffix still makes sense to have on each new proposed repository?
For the "spi" vs "apis" naming, I think the distinction could be confusing depending on who is looking at this and it's also somewhat prescriptive in assuming how reactive-ipc would be used. Perhaps it would be more clear to call what is proposed as ripc-tcp-spi the API of reactive-ipc and anything "-rxjava1" an adapter layer. In other words:
ripc-tcp-spi => ripc-tcp-api
ripc-tcp-apis => ripc-tcp-adapter

would there be no shared code across protocols

There probably will be some, such as base implementations of Publisher. Possibly Buffer as well. As we implement 2 or 3 protocols we'll find what is common.
Perhaps an "ripc-core" project could be used for that?

Or do you see the TCP module as that lowest common denominator?

No I don't. HTTP for example should not be based on that as the transports will handle that implementation on top of TCP. UDP also wouldn't work on TCP.

Would WebSocket be under ripc-http?

I think so. My view is that HTTP should be multi-protocol to support HTTP/1.1, HTTP/2, WebSocket, and SSE.

Perhaps that suffix still makes sense to have on each new proposed repository?

Sure, that makes sense.

Perhaps it would be more clear to call what is proposed as ripc-tcp-spi the API of reactive-ipc and anything "-rxjava1" an adapter layer

That seems good to me. I wasn't thrilled with the "spi" naming anyways.

Modified names based on discussion above:

ripc-core-jvm
With common elements like Publisher and Buffer

ripc-tcp-jvm
With submodules as ...

ripc-tcp-api
ripc-tcp-transport

ripc-tcp-transport-netty4
ripc-tcp-transport-netty5
ripc-tcp-transport-aeron
ripc-tcp-transport-etc...

ripc-tcp-adaptors

ripc-tcp-rxjava1
ripc-tcp-rxjava2

ripc-http-jvm
Support for HTTP/1.1, HTTP/2, Websockets, SSE, etc and submodules as ...

ripc-http-api
ripc-http-transport

ripc-http-transport-netty4
ripc-http-transport-etc...

ripc-http-adaptors

ripc-http-rxjava1
ripc-http-etc...

ripc-udp-jvm

ripc-kafka-jvm

ripc-cassandra-jvm

ripc-dynomite-jvm

ripc-memcached-jvm

etc

Perhaps an "ripc-core" project could be used for that?

Yeah we should have ripc-core where we can have different modules like ripc-buffer, ripc-reactive-streams, etc and we can discuss these modules as and when the need arises.
Re: TCP being the LCD.
I think there are two aspects to it:

The relation appearing in the APIs (HTTP extends TCP)
The relation appearing in an implementation.

I see 2 above to be more meaningful as opposed to 1.
Re: WS, HTTP/2 & HTTP relationship.
I think HTTP/1.1 implementation should provide a Upgrade/Downgrade facility for server and client respectively, as per the generic HTTP upgrade semantics. We can decide then whether HTTP/2, WS exist in the same project or not. I can see HTTP/2 moving faster than HTTP/1.1 implementation from the nature of the protocol maturity itself and us understanding the usecases for it. That may have us separating HTTP/2 from HTTP/1.1 project purely from the perspective of release cycles.

ripc-tcp-adaptors

Those should be adapter (vs adaptor), right?

We can decide then whether HTTP/2, WS exist in the same project or not.

Perhaps separate modules but they should be the same project. Whatever the split it should be possible to run a single server with HTTP 1.1, HTTP 2 and WebSocket support.

Whatever the split it should be possible to run a single server with HTTP 1.1, HTTP 2 and WebSocket support.

Absolutely, that's a given!

Those should be adapter (vs adaptor), right?

No idea ... they seem interchangeable:

http://en.wikipedia.org/wiki/Adapter#cite_note-1
https://www.google.com/search?client=safari&rls=en&q=define+adaptor&ie=UTF-8&oe=UTF-8
http://dictionary.reference.com/browse/adaptor vs http://dictionary.reference.com/browse/adapter

Whatever the split it should be possible to run a single server with HTTP 1.1, HTTP 2 and WebSocket support.

Absolutely, that's a given!

Yup.

http://en.wikipedia.org/wiki/Adapter#cite_note-1

Heh, good point. FWIW adapter seems to be preferred in JDK classes at least. Lots of matches on "java..Adapter" and none on "java..Adaptor".

Then "adapter" it is!


feat: hash() - Generate a unique number based on input

Based on a unique combination of a seed, text and a variant number, it generates a fixed number which can be used for sorting in a random order. Typical seed could be the date of today, the text a file name, and the variant could be the line number of a list/task item.
As discussed in #2252 this is the alternate PR just having a hash() function, which is a simple logic, but still provides the needs necessary for sorting any list in a random order, but with fixed value for a given input sequence.
Given a date as the seed, this can be used to sort lists of files/tasks/lists, and then return a random item of that list, and the item for that given date will remain the same (as long as the other parameters stay the same). This deals with an issue of using dataview to receive random items, where it'll change for every refresh of the random query.
See also https://jsperf.app/bowuze for the reasoning why this particular hash function was used.

Attached here is a self-contained test file (which I added to my local test-vault in the untracked folder):
Testing hash().md

you are aware this is conflicting with #2271 due to the lint fix?

Why do you think it's in conflict with #2271?

Why do you think it's in conflict with #2271? Please explain your reasoning.
Even though the same fix has been made in both versions, they should cancel each other out when properly merged which is why this PR also has all of its checked passed.

@blacksmithgu usually does squash merges.
after one is merged, a conflict is likely to happen.
either he resolves it then, or you pull master and resolve it in your branch yourself.
we will see.
if it does come to a conflict, it's pretty easy to solve.

I can resolve both merge conflicts.

Good functionality! Thanks for the contribution.


Turn off rocksdb debuginfo in dev mode when tikv debuginfo is off

In https://github.com/tikv/tikv/pull/4689 I am turning off debuginfo for the dev, test, and bench profiles.
Unfortunately, this does not turn off debuginfo for RocksDB (and any other CMake project) because CMake's builtin "Debug" profile always includes debuginfo. So cmake-rs just ignores the debuginfo setting if opt-level = 0.
I don't know how to fix it offhand, but I think the best fix would be in cmake-rs, having it forcibly override the debuginfo settings for the Debug profile when appropriate.
As of now our dev tikv-server is still 187 MB.

I think the best place to fix this is in cmake-rs if its possible. Today there are two cases where it just ignores cargo's debuginfo choice.

Upstream issue https://github.com/alexcrichton/cmake-rs/issues/79

@brson
how about the compiling time now when disable rocksdb's debuginfo.

I haven't tried yet. I don't have the correct solution yet. When I get back to it I'll hack something together to get numbers.

Per a chain of comments across various issues and PRs, particularly https://github.com/alexcrichton/cmake-rs/issues/79#issuecomment-497563743. We could end up wanting debuginfo=1 for optimized rocksdb, and it looks very complicated to make that happen in the general case in cmake-rs. We may be forced to add hacks to our rocksdb build script to adjust the debuginfo.


registry - image exists - but doesn't show in catalog

I have a private docker-registry (registry:2) - using S3 storage & redis blobdescriptor cache.
I've recently pushed a new image (/prefix/image:2.2.0) -  but it doesn't show up in the catalog (curl -XGET <address>/v2/_catalog... other images show, including new images pushed afterwards)...
However, I am able to pull the image, see it in the redis cache (KEYS /prefix/image* & DUMP), see it on the S3 storage (s3://bucket/docker/registry/v2/repositories/prefix/image/) and list it's tags (curl -XGET <address>/v2/<image>/tags/list)..  etc...
Anyone know why an image wouldn't show up in the catalog, but work for all other operations?

@goatherder I'm having the same issue. Were you able to fix it?

Yes, this was an embarrassing one - pagination.  I just needed to append ?n=&last=


Improve password authentication and reset forms javascript issues

While logging Into the Zulip, when they ask for the password. I tried to give a password and deleted everything again. During this course of time whole of the text moved toward left and right which did not look good. This happens because of the different text that was shown as the error message. We can resolve the issue if we can show the error text on a new line.

Yeah this would be great to improve, the current thing is pretty annoying.  Agreed the solution is to move it to the next line.

Can I do it? I am new to this community and just ended doing the Getting Started part.

Yep!

Hi @singh1114
In order to merge the changes made by you in your forked repository, you must create a pull request. These links might be helpful:
https://help.github.com/articles/creating-a-pull-request-from-a-fork/
https://yangsu.github.io/pull-request-tutorial/
I would also recommend you to read:
http://zulip.readthedocs.io/en/latest/git-guide.html
On a side note: It's a good practice to create a new branch for a new feature or bug instead of using master branch.
Thanks

@jajodiaraghav
https://github.com/zulip/zulip/pull/3690
Is it what you are talking about or am I missing something?
The last suggestion is very helpful. I will keep that in mind.

@timabbott
Doesn't the new title signify something else?
If yes, please help me Understand.

The title means we should fix this jitter issue from zxvcbn errors being placed badly on all the forms, not just the one it was initially reported as affecting.

It would be great if you can specify all the places where it is needed to be done else few of them might be left out.
Also, is the way we are using to solve the problem well or should we use something like an 'alert' for specifying the errors?

Alert is I think a bad UI pattern for this.  @sampritipanda and @rishig can point to what we have in mind.
templates/zerver/login.html, templates/zerver/register.html, and templates/zerver/reset_confirm.html

Linking to #3119 (last few comments in that thread).

So, is someone assigned for the work, or can I do it?

Procedure:
@rishig

First of all, I will download the minified version from  and then paste it in a folder inside the static/third with a name bootstrapValidator.
Then I will change the HTML of the form so that the validations can be applied.
Finally, I will write some JavaScript such that the errors are only shown when the submit button is clicked.

Is this what needed to be done or something else?

Hi Ranvir, thanks for looking in to this and sorry for the cryptic message last time! Let's do just the following:

Change the error text positioning to be vertically aligned
Change the outline of the erroring boxes to red

This can just be done in our current codebase, without the bootstrap-validator dependency.

So, is someone assigned for the work, or can I do it?

Feel free to take it!

@singh1114 you can claim this by commenting @zulipbot claim

@timabbott @rishig can you take me off assignee and add @singh1114

@zulipbot claim

Fixed it for you @lonerz  :)

@rishig
For editing the box outline to red should I use JavaScript?

@singh1114 You'll probably have to use JS and some HTML/CSS

Also, when I am filling this form
http://localhost:9991/accounts/password/reset/
I don't get the email so I cannot check the changes that I have made.
@lonerz

There is no difference between error message and input field. Should I use the margin-bottom of the input field or add line break tag.

@singh1114 all emails are sent in the console where you ran ./tools/run-dev.py.

It says:
Password reset attempted for<EMAIL_ADDRESS>no active account.
But I double checked by logging into the zulip website and I was able to login.

@singh1114 you have to password reset one of the existing accounts like cordelia.

@lonerz Thank you!
Any idea on the use of line break or margin.

@singh1114 I honestly don't know; probably should look at other error fields on the site

This was resolved by @brockwhittaker some time ago.


Service page tabs become active correctly

Before

After

Reviewer: @Poltergeist

Thanks!

Would be great to have some integration tests for this, though! Out of scope?

@pierlo-upitup alright i added some integration tests.


Dead link in the readme

The link to the blogpost (https://codemuch.tech/2021/06/07/fuzzabble/) is dead.
Is it available somewhere else ?

@Segwaz https://codemuch.tech/2021/06/07/fuzzable/


Fix a minor typo

Since this isn't ever used, this doesn't really affect anything, but just for clarity, fix the typo of 'prometheus' in the full-api example.

thank you @paultag! Much appreciated!


Building from the command line shows supported configuration warnings

When I run the msbuild command line
cmd> msbuild Markpad.sln

I get a couple of warnings about supported configurations
Build started 1/31/2012 4:14:00 PM.
Project "D:\Users\anthony.mastrean\Dropbox\Projects\DownmarkerWPF\MarkPad.sln" on node 1 (default targets).
ValidateSolutionConfiguration:
  Building solution configuration "Debug|x86".
ValidateProjects:
  The project "MarkPad.XAML" is not selected for building in solution configuration "Debug|x86".
  The project "MarkPad.Services" is not selected for building in solution configuration "Debug|x86".

Is that a problem? I'm able to produce Debug and Release builds that appear to run fine.

 Anyway, fixed now.

  And I run from an Visual Studio tools x64 prompt, so I have the extra pain of a constant platform=x64 property set in my environment.


chore(examples): add empty results design to e-commerce demo

This adds the empty state when no hits are returned by the engine. It doesn't change the SFFV no results design.
Implementation
If no filter are applied, the empty message asks the user to change the query. If there are filters applied, the empty message shows a different message and offers the user to clear the filters.
Since it's not possible to add a widget within another widget template, I used the ClearRefinements widget within a panel that is hidden when there are results.
I added removeWordsIfNoResults to allOptional so that results are returned most of the time.
Preview

Preview with only query leading to empty results
Preview with filters leading to empty results
Default preview

Related

Desktop design
Mobile design

@tkrugg The error you see was fixed in #3842, it's not from this PR. The base branch wasn't rebased, which is why the fix wasn't propagated here.

I can't try the demo out with that error 🤔

@Haroenv Try the links in the description. Interacting with the menu can cause the error.


Fix windows #5

It seems that the issue was with this NOSHORTS option

On Windows 32 bits:
Problem LIN2A
Iter:  0 Ap: 0.00e+000 Pobj:  1.7129989e+003 Ad: 0.00e+000 Dobj:  0.0000000e+000
Iter:  1 Ap: 1.00e+000 Pobj:  8.2000000e+001 Ad: 1.00e+000 Dobj:  1.1112447e+003
Iter:  2 Ap: 1.00e+000 Pobj:  8.2000000e+001 Ad: 1.00e-002 Dobj:  1.1011470e+003
Stuck at edge of primal feasibility, giving up.

Noooooo :'(

@EQt  Could you kill the appveyor's build for 45a9173d977f6d1f122375d4b2b678c04f11599e, it is looping for 10 minutes and prevents the build for this PR to start :-P

Stuck at edge of primal feasibility, giving up.

This may also be related to a bug in libblas: I compiled the original csdp executable and tested on soc.prob.1 (from our unit tests). Depending on which implementation you use (atlas, liblapack, or openblas), it gets stuck or completes without errors. Don't know, what to do with this strange behavior.

Which one works best of atlas, liblapack and openblas ?

On Windows 32 bits:
Problem SOC1
INFO: Writing problem to C:\Users\appveyor\.julia\v0.5\CSDP\test/soc.prob.1
Iter:  0 Ap: 0.00e+000 Pobj:  0.0000000e+000 Ad: 0.00e+000 Dobj:  0.0000000e+000
Iter:  1 Ap: 1.00e+000 Pobj:  1.4348779e-001 Ad: 1.00e+000 Dobj:  4.0421557e+001
Iter:  2 Ap: 1.00e+000 Pobj:  2.8851926e-001 Ad: 9.00e-001 Dobj:  4.3018215e+000
Iter:  3 Ap: 1.00e+000 Pobj:  1.1383216e+000 Ad: 1.00e+000 Dobj:  3.1449719e+000
Iter:  4 Ap: 1.00e+000 Pobj:  1.1841119e+000 Ad: 1.00e+000 Dobj:  2.1874361e+000
Iter:  5 Ap: 1.00e+000 Pobj:  1.2811728e+000 Ad: 1.00e+000 Dobj:  1.7828340e+000
Iter:  6 Ap: 1.00e+000 Pobj:  1.3400083e+000 Ad: 1.00e+000 Dobj:  1.5908381e+000
Iter:  7 Ap: 1.00e+000 Pobj:  1.3748455e+000 Ad: 1.00e+000 Dobj:  1.5002595e+000
Iter:  8 Ap: 1.00e+000 Pobj:  1.4142124e+000 Ad: 9.00e-001 Dobj:  1.4228155e+000
Iter:  9 Ap: 1.00e+000 Pobj:  1.4142136e+000 Ad: 9.00e-001 Dobj:  1.4150722e+000
Iter: 10 Ap: 1.00e+000 Pobj:  1.4142136e+000 Ad: 1.00e+000 Dobj:  1.4142118e+000
Stuck at edge of primal feasibility, giving up.

Same issue :'(

Atlas and liblapack (as shipped with Ubuntu) seem fine. OpenBLAS (for both, Ubuntu and the version included in Julia, 64bit) gets stuck.

Win32 solves the two SDP test from MathProgBase juste fine :) but fails on LIN, SOC and SOCRotated which I have disabled for Win32 until we find a solution so that the tests pass on master.
We can reenable them in the PR that tries to find a solution.

I have finished this PR, you can review it :)


Reference Skeleton to get Page and Tuple

@hongsudt commented on Tue Jul 12 2016
This task focuses on getting the Record2 data without any annotation (e.g. contextualization). Here is a list of methods that should be implemented:
module.resolve  /* use the existing parser */

.Reference
  .columns
  .read
.Page
  new Page(reference, data)
  ._data
  .tuples
.Tuple
  new Tuple(reference,data)
  ._data
  .values

This issue is resolved with the changes in PR #100.


fix chomp miss-use in helm chart

fixes Error: INSTALLATION FAILED: YAML parse error on pachyderm/templates/pachd/deployment.yaml: error converting YAML to JSON: yaml: line 159: mapping values are not allowed in this context

closed. fixed in #9103


Sort by Rating

Currently the extension only sorts by the total number of reviews (an indication of popularity).
An indication of quality would be helpful, namely the average user rating.
Choices for UX:

Implement additional configuration page -- https://developer.chrome.com/docs/extensions/mv3/options/
Create an additional button -- [sort by count] & [sort by average]
Dual use of the existing button -- click once for count sorting; click again for sorting by the average rating
Switch fully -- use only sorting by the average; not great for existing users; could have secondary extension

Work:

Besides the UX work, extracting the average rating value is just a matter of switching the class name searched for, and using parseFloat instead of parseInt, the rest would remain the same

Further idea:

Laplace smoothing to pull restaurants/stores/etc with very few reviews towards either, (1) the average for the results set, or (2) towards a perfect score (which up weights new locations as Yelp does)

Hi @justinormont, it can be a great addition. Would you like to implement it?


ISPN-9093 Samples from query docs should not be affected by branding

replace {brandname}IndexManager back to InfinispanIndexManager in query docs
avoid using {brandname} in query samples

https://issues.jboss.org/browse/ISPN-9093

updated!


Problem: Bigchain class is unused and redundant

Solution
Remove Bigchain class and BigchainDB's dependency on it.

Codecov Report

Merging #2366 into master will decrease coverage by 0.3%.
The diff coverage is 68.88%.

@@            Coverage Diff             @@
##           master    #2366      +/-   ##
==========================================
- Coverage   83.07%   82.77%   -0.31%
==========================================
  Files          38       38
  Lines        2245     2281      +36
==========================================
+ Hits         1865     1888      +23
- Misses        380      393      +13

For codecov I wouldn't mind merging it with a decrease of .3
We need to check the coverage later anyways

There are some conflicts in this change now, probably because of #2342

I think the changes haven't been merged from the master carefully, i.e. please have a look at this vs this


add networkpolicy cases for kube-burner

Description
Cases are explained here https://docs.google.com/document/d/1OQtOXtqqHzV79vJ04r3z1icv5b-9y5AtWGrEgUMnvEg/edit

LGTM so far, I'm currently working on a mechanism to verify policy enforcement. Will add a commit to this PR as soon as I have something.


Razer Book 13

Successfully used your script on my Razer Book 13.
Thanks for the awesome work!!

Thank you !


[Frontend] Integrate the Delete Team Member API with frontend Fixed

Issue that this pull request solves
Issue Link resolve #894
Closes: #894
Brief description of what is fixed or changed
I found the bug of multer and fixed it and made that image display is fine. and added delete feature. and a small thing related to manage Q&A that is also fixed. Now Images display fine and delete works fine and manage q&a also.
Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] My changes does not break the current system and it passes all the current test cases.

Screenshots

@Kajol-Kumari  can you please check and merge it.


make_scorer on sklearn metrics.py wrong attribute

The make_scorer from aif360/sklearn/metrics/metrics.py file has the attribute ratio which I think it should be is_ratio instead since if the attribute is ratio it will always be True.

Could you make a PR to the master branch with this change? Thanks!

Sure


broken links in README.md

the travis-ci link (links to https://travis-ci.com/sras/elminator which is a 404 for me, on line 9 and again on line 170),
the sample generated code link (links to https://bitbucket.org/sras/elminator-test/src/master/elm-app/src/Autogen.elm which I think has been deleted, on line 101).

I think Travis-ci stopped their free tier. I ll try to move the CI to somewhere else. Thanks for reporting.


[Issue]: Proto file causes error in ui.perfetto.dev

Problem Description
I ran the omnitrace-sample executable from the omnitrace-1.11.2-ubuntu-22.04-PAPI-OMPT-Python3 installer with no arguments, just my own C++ executable+args, then it exits normally:
...
[omnitrace][8461][0][omnitrace_finalize] Finalizing perfetto...
[omnitrace][8461][perfetto]> Outputting '/home/.../omnitrace-annotate-output/2024-12-04_08.37/perfetto-trace-8461.proto' (42908.89 KB / 42.91 MB / 0.04 GB)... Done
[omnitrace][8461][metadata]> Outputting 'omnitrace-annotate-output/2024-12-04_08.37/metadata-8461.json' and 'omnitrace-annotate-output/2024-12-04_08.37/functions-8461.json'
[omnitrace][8461][0][omnitrace_finalize] Finalized: 2.874768 sec wall_clock,  374.208 MB peak_rss, -299.405 MB page_rss, 2.930000 sec cpu_clock,  101.9 % cpu_util
[387.781]       perfetto.cc:60128 Tracing session 1 ended, total sessions:0

But when I attempt to use ui.perfetto.dev to load the proto file, it fails and returns this error:
Oops, something went wrong. Please file a bug.
UI: https://ui.perfetto.dev/v47.0-1b4776303

Unknown field type

Trace: not available (FILE). Provide repro steps.
UA: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:133.0) Gecko/20100101 Firefox/133.0
Referrer:

Operating System
NAME="Ubuntu" VERSION="22.04.5 LTS (Jammy Jellyfish)"
CPU
AMD Ryzen Threadripper 1950X 16-Core Processor
GPU
None
ROCm Version
ROCm 6.2.3
ROCm Component
No response
Steps to Reproduce
I am happy to send the output files if there is an appropriate way to do it
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
bash: /opt/rocm/bin/rocminfo: No such file or directory
Additional Information
No response

Hi @rlorigro. Internal ticket has been created to investigate your issue. Thanks!

@rlorigro Omnitrace was rebranded to ROCm Systems Profiler and a fix for this issue is being addressed here: https://github.com/ROCm/rocprofiler-systems/pull/30
in the meantime, somewhere in the docs, there should be browser link to an older release of ui.perfetto.dev which can still load them.
@dgaliffiAMD

@rlorigro Ah, actually, if you upgrade to v1.13.0, this is already fixed in Omnitrace: https://github.com/ROCm/omnitrace/releases/tag/v1.13.0 via #411

Awesome thank you for the quick response, and sorry for missing that. I will try it tomorrow

Using v1.13.0 (and also the latest rebranded release), It loads now and for the most part looks very nice in the browser, but it seems that it is only reporting threads 0 and 1, despite there being 32 child threads launched (implemented in C++ with pthread).
Invoking --trace or --profile options result in the program not executing at all, which is also confusing.
Honestly I mostly care about summary stats, and less about the timeline. Probably I am doing something wrong?


chrisproject.org/code needs to be implemented

Page: Code
Since there is no content on this page, should there be a link for it?
see https://github.com/FNNDSC/website/issues/20#issuecomment-740165487

Hey @mairin. Is this issue open to work? I wish to contribute in the same.


Inconsistent return type from click.getchar()

It seems that click.getchar() will return bytes under python 3 / Windows, and str under any other combination of python version and platform. Is this the intended behavior or a bug?
I wanted to update the stub for click but if it's a bug and can be fixed soon, I can leave the typeshed stub unchanged.

The documentation  states:

This will always return a unicode character

I've witnessed the same behavior (bytes w/Python3 and Windows, str elsewhere) and to me, this inconsistency is a bug.

The python 3 doc's is clear with this:

Read a keypress and return the resulting character as a byte string

In python 2 the doc say this:

Read a keypress and return the resulting character

Those are docs for the standard library, not for click.

@JelleZijlstra I know, but if you see the code of getchar you understand why I mentioned the module msvcrt

Closed by #1108


Python: Extend hasAttribute to unknown-but-defined module variables. (ODASA-8028)

Eliminates the false positive seen here: https://discuss.lgtm.com/t/python-undefined-export-false-positive-when-reexporting/2259
(and possibly others reported on the forum).
@AlexTereshenkov

Related to https://github.com/Semmle/ql/issues/1792

I'm not keen on a fix that modifies an API that we plan to deprecate. What happens when we move to the new API. We should fix the Value API, IMO.
Moving your fix to a new ModuleValue.hasAttribute(string name) predicate and wrapping that in ModuleObject.hasAtttibute(string name) would be better, IMO.

That's a good point. I'll rewrite it the way you suggest.

Closing in favour of #1840


Ui improvements + higher angular version

Coverage remained the same at 96.48% when pulling 7df9c2e72e9e05922049b4c94ecc25c6e51b3efa on master into 85a562f341a5283e02e8d9a228e52364fbe9935b on release.


Update TUF dependency

What does this PR do?
Update dependency for datadog-checks-downloader
Motivation
Fix a bug
Additional Notes
Should have no functional difference for customers
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Are we updating for a specific reason/changelog? Or just to keep up to date?

/azp run

/azp run

/azp run


Using a bfloat16 causes Double Free Exception and Crash

Please describe the bug
Hi,
Using a bfloat16, whether by initializing an embedding layer or casting a float32 to bfloat16, causes a double free exception and crash.
Sometimes it just prints out that there was a segmentation fault or that a worker died, without a verbose explanation - it can depend on the method selected for alpa's parallelize function.
This happens to me using either Shard Parallel or Pipeshard Parallel, but it used to work in an earlier version of Alpa.
Please describe the expected behavior
The data is used as a bfloat16 or cast to bfloat16 from float32 without issue.
System information and environment

OS Platform and Distribution (e.g., Linux Ubuntu 16.04, docker):
RHEL 8.5, Docker image: cuda_11.8.0-cudnn8-devel-ubuntu22.04 (also in Docker image 11.5.2-cudnn8-devel-ubuntu20.04)
Python version:
3.9.6
CUDA version:
11.8 (Also happens with Cuda 11.5.1-75)
NCCL version:
(2, 10, 3)?
cupy version:
cupy-cuda11x==11.5.0
GPU model and memory:
20x Nvidia A100 80GB
Alpa version:
This git commit (It's one older than the latest posted on Feb 13th, 2023)
https://github.com/alpa-projects/alpa/commit/c164f3e6d94b500bb2fc23cb85e9a3236e15b59a
TensorFlow version:
tensorflow==2.8.0
JAX version:
jax==3.22.0

To Reproduce
Steps to reproduce the behavior:
(I am starting this using a SLURM script)

Run this py file with ray (It's a modified included test file)

If you comment out line 52, it will work again.

x = x.astype(jnp.bfloat16) ## Comment this line out to fix the problem

This line also causes an issue

x = nn.Embed(256, 768, dtype=jnp.bfloat16)(inputs) # <-- This fails, too

Screenshots
-------------------------------------------------------------
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::trace: 3.10 s
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::jaxpr operations: 0.02 s
alpa.pipeline_parallel.stage_construction.cluster_layers_and_slice_mesh():  num_devices: 20,  num_stages: 20
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::stage construction: 0.05 s
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::apply grad: 0.05 s
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::shard stages: 28.51 s
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::launch meshes: 1.06 s
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::runtime emitter: 49.50 s
alpa.compile_executable.debug_compilation_time(): compile_pipeshard_executable::driver executable: 8.64 s
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) free(): double free detected in tcache 2
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) *** SIGABRT received at time=1676287900 on cpu 22 ***
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) PC: @     0x7febf204337f  (unknown)  raise
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)     @     0x7febf2b6dc20  309695680  (unknown)
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)     @     0x7febf208d5ec  (unknown)  malloc_printerr
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) [2023-02-13 06:31:40,764 E 1468195 1468195] logging.cc:361: *** SIGABRT received at time=1676287900 on cpu 22 ***
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) [2023-02-13 06:31:40,764 E 1468195 1468195] logging.cc:361: PC: @     0x7febf204337f  (unknown)  raise
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) [2023-02-13 06:31:40,764 E 1468195 1468195] logging.cc:361:     @     0x7febf2b6dc20  309695680  (unknown)
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) [2023-02-13 06:31:40,764 E 1468195 1468195] logging.cc:361:     @     0x7febf208d5ec  (unknown)  malloc_printerr
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) Fatal Python error: Aborted
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>) Stack (most recent call first):
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/collective/worker_nccl_util_cupy.py", line 225 in xla_buffer_to_cupy
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/collective/worker_nccl_util_cupy.py", line 91 in recv_tile
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/collective/worker_nccl_util.py", line 11 in _switch_impl
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/collective/worker_nccl_util.py", line 27 in recv_tile
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/device_mesh.py", line 493 in recv_tile
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/.local/lib/python3.9/site-packages/ray/util/tracing/tracing_helper.py", line 466 in _resume_span
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/device_mesh.py", line 457 in run_resharding_recv_task
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/.local/lib/python3.9/site-packages/ray/util/tracing/tracing_helper.py", line 466 in _resume_span
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/pipeline_parallel/pipeshard_executable.py", line 543 in execute_on_worker
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/alpa/alpa/device_mesh.py", line 279 in run_executable
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/.local/lib/python3.9/site-packages/ray/util/tracing/tracing_helper.py", line 466 in _resume_span
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/.local/lib/python3.9/site-packages/ray/_private/function_manager.py", line 674 in actor_method_executor
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/.local/lib/python3.9/site-packages/ray/_private/worker.py", line 763 in main_loop
(MeshHostWorker pid=1468195, ip=<IP_ADDRESS>)   File "/home/sblouir/.local/lib/python3.9/site-packages/ray/_private/workers/default_worker.py", line 231 in <module>
2023-02-13 06:31:40,994	WARNING worker.py:1839 -- A worker died or was killed while executing a task by an unexpected system error. To troubleshoot the problem, check the logs for the dead worker. RayTask ID: ffffffffffffffffeeb376d8eb140bbb2046b3f001000000 Worker ID: a69bcad3d3518114e11fec2e1e8259b4cf28a5c5fa8f66dcbacf55a7 Node ID: b42a40ba7bcb1b78f72ba5de6b2cae8d5bf5652a51d18ae67b496891 Worker IP address: <IP_ADDRESS> Worker port: 10008 Worker PID: 1468195 Worker exit type: SYSTEM_ERROR Worker exit detail: Worker unexpectedly exits with a connection error code 2. End of file. There are some potential root causes. (1) The process is killed by SIGKILL by OOM killer due to high memory usage. (2) ray stop --force is called. (3) The worker is crashed unexpectedly due to SIGSEGV or other unexpected errors.

Code snippet to reproduce the problem
"""Utilities for testing."""
from functools import partial
import unittest
from collections.abc import Iterable
from typing import Callable, Optional

import jax
import jax.numpy as jnp
from jax.tree_util import tree_leaves
from jax.experimental.maps import FrozenDict as FrozenDictJax
import numpy as np
import optax
from flax import linen as nn
from flax.core.frozen_dict import FrozenDict as FrozenDictFlax

import alpa
from alpa.api import init, shutdown, parallelize, value_and_grad
from alpa.model.bert_model import BertConfig, FlaxBertLayer
from alpa.model.model_util import FlaxBaseModelOutput, DynamicScale, TrainState
from alpa.parallel_method import PipeshardParallel
from alpa.pipeline_parallel.layer_construction import (AutoLayerOption,
													   ManualLayerOption)
from alpa.pipeline_parallel.primitive_def import mark_pipeline_boundary
from alpa.pipeline_parallel.stage_construction import (UniformStageOption,
													   StageOption)
from alpa.shard_parallel.auto_sharding import AutoShardingOption

from typing import Any, Callable, Optional, Union
from typing import NamedTuple, Optional, Tuple, Callable
from optax._src import base
from optax._src import clipping
from optax._src import combine
from optax._src import factorized
from optax._src import transform
import functools
import numpy as np
from optax._src import base
import optax
import chex
from optax._src import linear_algebra
from optax._src import base

ScalarOrSchedule = Union[float, base.Schedule]
MaskOrFn = Optional[Union[Any, Callable[[base.Params], Any]]]

class BasicModel(nn.Module):
	num_layers:int
        \#  This doesn't fix it, either
	\#def setup(self) -> None:
	\#	self.embeddings = nn.Embed(256, 768, dtype=jnp.bfloat16)

	@nn.compact
	def __call__(self, inputs=None, labels=None, attention_mask=None, loss_mask=None, target_input_ids=None, *args, **kwargs):
                \#  x = nn.Embed(256, 768, dtype=jnp.bfloat16)(inputs) \# <-- This fails, too
		x = nn.Embed(256, 768)(inputs)
		x = x.astype(jnp.bfloat16) ## Comment this line out to fix the problem
		x = nn.LayerNorm()(x)
		for _ in range(self.num_layers):
			x = nn.Dense(16)(x)
		x = nn.Dense(768)(x)

		y = nn.Embed(256, 768)(labels)
		y = nn.LayerNorm()(y)

		x = x.astype(jnp.float32)
		loss = (y-x)**2
		return jnp.sum(loss)

def get_bert_layer_train_state_and_step(batch_size, seq_len, num_layers,
										hidden_size, num_heads,
										clip_by_global_norm, use_dynamic_scale,
										add_manual_pipeline_marker):
	rngkey = jax.random.PRNGKey(0)
	inputs = jax.random.randint(rngkey, (batch_size, seq_len,), 0, 384,)
	labels = jax.random.randint(rngkey, (batch_size, seq_len,), 0, 384,)
	attention_mask = jnp.ones((batch_size, seq_len), dtype=jnp.int8)
	loss_mask = jnp.ones((batch_size, seq_len), dtype=jnp.int8)
	target_input_ids = jax.random.randint(rngkey, (batch_size, seq_len,), 0, 384,)

	batch = {
		"inputs": inputs,
		"attention_mask": attention_mask,
		"labels":labels,
		"loss_mask":loss_mask,
		"target_input_ids":target_input_ids,
		}

	model = BasicModel(num_layers=num_layers,)
	params = model.init(rngkey, **batch,)

	tx = optax.adam(learning_rate=1e-2)

	if use_dynamic_scale:
		use_master_copy = False
		dynamic_scale = DynamicScale()
	else:
		dynamic_scale = None
		use_master_copy = False

	state = TrainState.create(apply_fn=model.apply,
							  params=params,
							  tx=tx,
							  dynamic_scale=dynamic_scale,
							  use_master_copy=use_master_copy)

	def train_step(state, batch):

		def loss_func(params):
			loss = state.apply_fn(params, **batch,)
			return loss

		dynamic_scale = state.dynamic_scale
		if dynamic_scale:
			grad_fn = dynamic_scale.value_and_grad(loss_func)
			dynamic_scale, is_fin, val, grads = grad_fn(state.params)
		else:
			grad_fn = value_and_grad(loss_func)
			val, grads = grad_fn(state.params)

		new_state = state.apply_gradients(grads=grads)

		if dynamic_scale:
			new_state = new_state.replace(
				opt_state=jax.tree_map(partial(jnp.where, is_fin),
									   new_state.opt_state, state.opt_state),
				params=jax.tree_map(partial(jnp.where, is_fin),
									new_state.params, state.params),
				master_copy=jax.tree_map(partial(jnp.where,
												 is_fin), new_state.master_copy,
										 state.master_copy),
				dynamic_scale=dynamic_scale)
		return new_state, val

	return state, batch, train_step

class PipelineBasicTest(unittest.TestCase):

	def setUp(self):
		init(cluster="ray")

	def tearDown(self):
		shutdown()

	def run_n_layer_bert(self,
						 num_layers=20,
						 batch_size=16,
						 seq_len=256,
						 hidden_size=512,
						 num_heads=512 // 64,
						 use_remat=False,
						 clip_by_global_norm=False,
						 use_dynamic_scale=False,
						 inject_train_step=None,
						 manual_pipeline_layer=True,
						 stage_option: Optional[StageOption] = None,
						 as_option: Optional[AutoShardingOption] = None,
						 do_numerical_test: bool = True):
		method = PipeshardParallel(
			num_micro_batches=4,
			default_auto_sharding_option=as_option or AutoShardingOption(),
			layer_option=ManualLayerOption(remat_layer=use_remat)
			if manual_pipeline_layer else AutoLayerOption(
				layer_num=num_layers,
				remat_mode="coarse_grained_remat" if use_remat else "none"),
			stage_option=stage_option or UniformStageOption())
		# method = alpa.ShardParallel()

		# Init model
		state, batch, train_step = get_bert_layer_train_state_and_step(
			batch_size=batch_size,
			seq_len=seq_len,
			num_layers=num_layers,
			hidden_size=hidden_size,
			num_heads=num_heads,
			clip_by_global_norm=clip_by_global_norm,
			use_dynamic_scale=use_dynamic_scale,
			add_manual_pipeline_marker=manual_pipeline_layer)
		if inject_train_step is not None:
			assert isinstance(inject_train_step, Callable)
			train_step = inject_train_step

		# Compile
		serial_train_step = train_step
		parallel_train_step = parallelize(train_step, method=method)
		executable = parallel_train_step.get_executable(state, batch)

		for _ in range(100):
			state, loss = parallel_train_step(state, batch)
			print(f"  loss: {loss}")

		hlo_text = executable.get_hlo_text()
		return hlo_text

if __name__ == "__main__":
	t = PipelineBasicTest()
	t.setUp()
	x = t.run_n_layer_bert(
		num_layers=alpa.get_global_num_devices(),
		manual_pipeline_layer=False,
		do_numerical_test=False,
	)
	print(f"*" * 60,)
	print(f"  x: {x}")
	print(f"*" * 60,)
	t.tearDown()

Additional information
Add any other context about the problem here or include any logs that would be helpful to diagnose the problem.
Please let me know if any more information can help. This doesn't seem to happen when casting a float32 to float16.

Bumping this hoping someone managed to resolve this.
I'm hitting a similar problems on the tests with CUDA 11.8 (CUDNN 8).


⚠ Samsung has degraded performance

In 5ed7e5e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2488 ms

Resolved: Samsung performance has improved in 2ed6444.


Modified Pooling layer

Accuracy increase on my end when using the modified pooling circuit.

Added alternative convolution and pooling circuit


fixed help message error when using slash command

Wow idk why the codacy static code analysis check wasn't successful

don't worry about it


Code completion: Stop suggesting old-style protocol metatype syntax in Swift 6 mode

In Swift 6 mode, [.]Protocol[^1] is no longer a valid suggestion. Per SE-0335, we should suggest two distinct [.]Type[^1] completions for a plain protocol/composition — one aiming at any P.Type and another aiming at (any P).Type — that also insert a missing any prefix, and parenthesize the acted-upon existential in the second case.
Test case (should not succeed):
// RUN: %empty-directory(%t)
// RUN: %target-swift-ide-test -batch-code-completion -source-filename %s -filecheck %raw-FileCheck -completion-output-dir %t -swift-version 6

protocol P {}
P#^COMPLETE?check=META^#
// META: Keyword/CurrNominal: .Protocol[#(any P).Type#]; name=Protocol

Environment

Swift 5.9-dev (c1d5118c21d)

[^1]: [x] means x is optional

@Rajveer100 After updating the description, I am no longer positive this can be qualified as a beginner-friendly issue, but here is some tips:

Code completion can be tested using the swift-ide-test executable found in the same place as swift-frontend (see test/IDE for example invocations).
The place the Protocol completion is suggested is CodeCompletionCallbacksImpl::addKeywords.

Yes, I agree that this could be a good first issue.

Hi Sir,
Can you assign this to me as I think this could solve it.
protocol P {}
P#^COMPLETE?check=META^#
// META: Keyword/CurrNominal: .Protocol[#(any P).Type#]; name=Protocol
changes to
protocol P {}
P.#^COMPLETE?check=META^#
// META: Keyword/CurrNominal: .Type[#any P.Type#]; name=Type
// META: Keyword/CurrNominal: .Type[#(any P).Type#]; name=Type

Note that a solution should also handle the any P and (any P) cases. Only the existential metatype completion should be suggested for any P, and only the singleton metatype completion — always spelled using any syntax, not only in Swift 6 mode — should be suggested for (any P).

// META: Keyword/CurrNominal: .Type[#any P.Type#]; name=Type
// META: Keyword/CurrNominal: .Type[#(any P).Type#]; name=Type
The completion suggestions include .Type[#any P.Type#], which represents the existential metatype for any P, and .Type[#(any P).Type#], which represents the singleton metatype for (any P).
isn't this correct Sir?

Your completion suggestions are fine for P#^COMPLETE^#, but not for any P#^COMPLETE^# or (any P)#^COMPLETE^#. Only one .Type completion is reasonable for those two.

any P.#^ANY_COMPLETE?check=META^#
// META: Keyword/CurrNominal: .Type[#any P.Type#]; name=Type
(any P).#^PAREN_COMPLETE?check=META^#
// No completion suggestion
When we trigger code completion at any P.#^ANY_COMPLETE?check=META^#, the completion suggestion is .Type[#any P.Type#], which represents the existential metatype for any P.
There is no completion suggestion provided for (any P)#^COMPLETE?check=META^# because any P.Protocol is not a sensible type and (any P).Protocol is a mixture of the old and new syntaxe
I hope it works.


Not able to use a TeamsPersonalApp webpart with mobile client; Cannot configure webpart in windows native app

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
After deploying a SPFX webpart as an Teams app it should be accessible via the mobile client.
Additionally I should be able to configure the webpart.
Observed Behavior
The deployed SPFX webpart is not visible in the native Teams app.
It works in the browser (although there were some token timeout issues) and the native Windows app.
It is not possible to configure the webpart in either the browser or the native Windows app
Steps to Reproduce
Deploy an SPFX webpart with supportedHosts TeamsPersonalApp and sync it with Teams via the SharePoint AppCatalog. Add the app via the Teams client or the Teams admin UI.
Greetings,
Frank

@Gitkohn , thanks for reporting that back. SPFx based tabs are available on Teams Mobile Client under "developer preview". Please follow the documentation reported here to check if you can access Teams Personal Apps that are SPFx based in Teams Mobile: https://docs.microsoft.com/en-us/microsoftteams/platform/resources/dev-preview/developer-preview-intro

@lucabandMSFT thanks for the info.
I activated the development preview. I still can't see the personal app inside the mobile client, but maybe it will take 24 hours for the settings to take effect.
Additionally remarks:

I still don't know how to configure the webpart settings (or if it's possible) when using a Personal Teams App
The performance between Teams and Graph seems to be better when running the developer preview
I was able to get rid of the token timeout error by using the Chrome dev console and disabling cache and reloading the page - I tried using CTRL+F5 before but the error persisted. Maybe CTRL+F5 and "Disable Cache" works differently although I thought they were practically the same

hey @FrankMuraschow , couple of answers:

Personal apps in Teams do not have any settings experience. That is indipendent to SPFx and it is a peculiarity of Teams personal Apps
for the disabled cache and fore the fact that you may need 24 hours for the personal tab to show up .. I will defer to @lokimeyburg to provide additional feedbacks


minor wording fix in (slim) readme

"we want to restore a model from a checkpoint
whose variables have different names to those in the current graph."  (add the to in the line)

Can one of the admins verify this patch?

Jenkins, test this please.

Flaky test, retrying.


A benchmark module for Parquet would be nice

As we discussed in the Parquet sync up here is the parquet-bencmark module that I put together when I started implementing vectorization. This would primarily be useful for keeping an eye on the performance across releases.
Reporter: Nezih Yigitbasi / @nezihyigitbasi
Assignee: Daniel Weeks / @danielcweeks
Related issues:

Release parquet-mr 1.6.0 (blocks)

PRs and other links:

PR#104
PR#155

Note: This issue was originally created as PARQUET-165. Please see the migration documentation for further details.

Daniel Weeks / @danielcweeks:
New pom file had wrong rc version, which will cause master to break.  Updating.

Nezih Yigitbasi / @nezihyigitbasi:
https://github.com/apache/incubator-parquet-mr/pull/104

Daniel Weeks / @danielcweeks:
Issue resolved by pull request 104
https://github.com/apache/incubator-parquet-mr/pull/104

Daniel Weeks / @danielcweeks:
Issue resolved by pull request 155
https://github.com/apache/incubator-parquet-mr/pull/155

Nezih Yigitbasi / @nezihyigitbasi:
Created a new PR: https://github.com/apache/incubator-parquet-mr/pull/155


[Question] Does extractor capture the PR when I change the current revision from one to another.

Release version
v6.1.1
Question Details
Hi
I have updated the current revision on portal from one revision to another. The extractor pipeline did not capture the current revision in the pull request. Is this not handled at the code level?
Expected behavior
I expect extractor to capture the change in the current revision.
Actual behavior
No pull request created
Reproduction Steps
Update the revision on portal and change the current revision to a different one and run the extractor

I just realized that there is a bug already raised on this issue - https://github.com/Azure/apiops/issues/646


Update the IQPE class docstring

Summary
Clears a grammatical error in the docstring for IQPE class.

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

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


Searching for term with number doesn't work

If you pull down this branch and do
rspec ./spec/collectionspace/helpers_spec.rb:66

You should get a passing test, as it is able to find the concept "Goats"
Now, if you go into helpers_spec.rb and uncomment lines 78 and 93 (adding the search for concept "QA TEST Concept 1") and re-run the test, it will fail.
That term is in core dev, as I just grabbed the expected URL from there.

The explanation for this was... I had pasted the "QA TEST Concept 1" term value from the rendered XML payload in my browser, which had collapsed the double spaces that actually appear in the term in the UI, underlying XML, and database: "QA TEST  Concept 1"
It's exact matching, so no double-spaces = no match. Fun times.


status handles powered of VM gracefully

@thieman
Resloves #521

LGTM "reslove"

seplling how does it work


Python Jedi couldn’t connect to Python Language Server

Type: Bug
I'm running VS code version 1.85.0 on Windows 11. You can see my report at this website: https://discuss.python.org/t/python-jedi-wont-connect-to-python-language-server/40512?u=coderdjjd
I also found a much needed error log:

Extension version: 2023.22.0
VS Code version: Code 1.85.0 (af28b32d7e553898b2a91af498b1fb666fdebe0c, 2023-12-06T20:48:09.019Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
Intel(R) Pentium(R) Gold 7505 @ 2.00GHz (4 x 1997)

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_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
7.80GB (2.00GB free)

Process Argv
--crash-reporter-id 90aa4b7b-cc80-4d95-8289-dd6215474c6f

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30899288
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
282f8724:30602487
f6dab269:30613381
showlangstatbar:30737416
962ge761:30917236
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
welcomedialogc:30910334
pythonnosmt12:30797651
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
dsvsc013:30795093
dsvsc014:30804076
dsvsc015:30845448
pythontestfixt:30902429
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
aa_t_chat:30882232

### Tasks
- [ ] https://github.com/microsoft/vscode-python/issues/11812
- [ ] https://github.com/microsoft/vscode-python/issues/21265

### Tasks
- [ ] https://github.com/microsoft/vscode-python/issues/20117
- [ ] https://github.com/microsoft/vscode-python/pull/22132
- [ ] https://github.com/microsoft/vscode-python/pull/22466

,mmmmkmkmkmkmkmkmkmkmkmkmkmkm @mdboom

Found more errors to my problem:

@CoderDJJD The error seems to be EACCESS which usually means you don't have permissions to run from that executable or that file. Can you please provide details on which path you have used to select python?
Also, you can set language server to Pylance using python.languagServer setting.

I'm using the interpreter ~\AppData\Local\Programs\Python\Python312\python.exe
and still have the problem occasionally.

From: Karthik Nadig @.>
Sent: Wednesday, December 13, 2023 11:51 AM
To: microsoft/vscode-python @.>
Cc: CoderDJJD @.>; Mention @.>
Subject: Re: [microsoft/vscode-python] Python Jedi couldn’t connect to Python Language Server (Issue #22610)
@CoderDJJD The error seems to be EACCESS which usually means you don't have permissions to run from that executable or that file. Can you please provide details on which path you have used to select p
DuckDuckGo removed one tracker. Morehttps://duckduckgo.com/-xF8BQ7ilDkvf4ZwpP6G2XtK4AoXBPU7d1Ad8jajzVMZYlHX-g2pjEWYbpIvjOMCgdI4E2TSyIAi299XN8Iths_kF3Fi7Zsv1bXWfaDqJWRGwmT2HgCutmZ_3KzhE0IKFwUvzyxVfJhDb2kIQ3fx-2b0pxFAcjJ2qz2CrMvlKLc4DP7pEPmsJvBPSNcRxdUHFfez4NE_U8175_B81p7PHSimAMSVWhE9iA2r7MFo8o1GsHm5anUA9Zkwyd3Cm1CohZYVzFSlZNwzx4EUDS-MlaGu6S4QYoMs6U4rkYdkLxL__gV8sQqO4nsZ78uIhQvNyZ95MJ2Pf-ZrMVxSGnSx_MrfHn9lMDkBxu7IBI_TvcSgj6xAVTif69pj367bou1eeMB1njxPeTyWeSi8PNB93OQyrdKTlyGj77-gAxmmJ1BMiSKP7-gqpR2_dUYpFKKJxu1YTh6H90iN0TaHbjxiuzC_19PI-yYDvCLU_
Report Spam https://duckduckgo.com/-xF8BQ7ilDkvf4ZwpP6G2XtK4AoXBPU7d1Ad8jajzVMZYlHX-g2pjEWYbpIvjOMCgdI4E2TSyIAi299XN8Iths_kF3Fi7Zsv1bXWfaDqJWRGwmT2HgCutmZ_3KzhE0IKFwUvzyxVfJhDb2kIQ3fx-2b0pxFAcjJ2qz2CrMvlKLc4DP7pEPmsJvBPSNcRxdUHFfez4NE_U8175_B81p7PHSimAMSVWhE9iA2r7MFo8o1GsHm5anUA9Zkwyd3Cm1CohZYVzFSlZNwzx4EUDS-MlaGu6S4QYoMs6U4rkYdkLxL__gV8sQqO4nsZ78uIhQvNyZ95MJ2Pf-ZrMVxSGnSx_MrfHn9lMDkBxu7IBI_TvcSgj6xAVTif69pj367bou1eeMB1njxPeTyWeSi8PNB93OQyrdKTlyGj77-gAxmmJ1BMiSKP7-gqpR2_dUYpFKKJxu1YTh6H90iN0TaHbjxiuzC_19PI-yYDvCLU_
@CoderDJJDhttps://github.com/CoderDJJD The error seems to be EACCESS which usually means you don't have permissions to run from that executable or that file. Can you please provide details on which path you have used to select python?
Also, you can set language server to Pylance using python.languagServer setting.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-python/issues/22610#issuecomment-1853774840, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCB5FMXA6AFL2GSVLRQMAUDYJGJEBAVCNFSM6AAAAABAL6TCYSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJTG43TIOBUGA.
You are receiving this because you were mentioned.Message ID: @.***>

If you switch to using pylance you should not see the error with Jedi. Make sure you use pylance as the language server and it should work as expected. Please share exact error from Output > Python Language server if you see something failing again.


RaggedTensor processing float type  causes memory leak

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

TF 1.15.3
Linux
python 3.7

Describe the current behavior
text_input_flatten is a 3D tensor, [batch_size, seq_length, embedding_size], dtype=tf.float32
text_mask is a 2D tensor, [batch_size, seq_length], dtype=tf.bool
text_input_ragged = tf.RaggedTensor.from_tensor(text_input_flatten)
text_input_ragged = tf.ragged.boolean_mask(text_input_ragged, text_mask, name="ragged_text_input")
new_text_input = text_input_ragged.to_tensor(default_value=0)
new_length = tf.shape(new_text_input)[1]
new_text_input = tf.pad(new_text_input, [[0, 0], [0, seq_length * emb_size - new_length], [0, 0]], constant_values=0)
the memory usage is increasing rapidly, finally, the program will crash.
When I changed the text_input_flatten dtype to tf.int32, it works normally.

Describe the expected behavior

@skywalker1990
TensorFlow 1.x is not actively supported. Could you please update TensorFlow to the latest stable version v2.4 and check if you are facing the same issue.
If the issue persists in 2.4 version,can you please share colab link or simple standalone code with supporting files to reproduce the issue in our environment.It helps us in localizing the issue faster.Thanks!

@skywalker1990
TensorFlow 1.x is not actively supported. Could you please update TensorFlow to the latest stable version v2.4 and check if you are facing the same issue.
If the issue persists in 2.4 version,can you please share colab link or simple standalone code with supporting files to reproduce the issue in our environment.It helps us in localizing the issue faster.Thanks!

Sorry, the version of our production environment is 1.15.x. We have no plan to upgrade the version to 2.x yet.

Sorry, the version of our production environment is 1.15.x. We have no plan to upgrade the version to 2.x yet.

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


Execute selection and then move cursor to next line of selection

It would be nice to have an alternative to CTRL + C that goes to the next line of the selection.

So you would like to run a selection line by line ? Ctrl-C currently runs the whole selection (doesn't it ?), so you can't really go to the next line of selection after that.
Or did I misunderstand something ?

I would like to run current selection and then go to the next line (down) instead of going back to the beginning of the selection as it currently dows. The same way nvim-R does \d
https://github.com/jalvesaq/Nvim-R/blob/master/doc/Nvim-R.txt#L382
Current behavior https://cl.ly/b23166eabab6
And what behavior I'd like (example using R): https://cl.ly/367b972d2b35
The behavior is different as after executing the selection it goes to the next line. Let me know if this is not clear enough.

Ah ok, make sense ! Then yes, I agree it would be a nice addition. I can't promise anything because I'm quite busy currently, but if you feel like doing a PR, please go ahead :)

Thanks @julienr, I wonder if we could just do a shortcut somehow that invoke CTRL-C as a quick hack at least.

Yeah I guess it would be a matter of writing another function that calls RunTmuxPythonChunk  and then goes to the next line. I'm actually pretty bad at vimscript, so I don't even know how to go to the next line, but something like simulating a k should do.


"Error in" without message Extract text webpack plugin error

I'm receiving an error that seems to be find in development (dev server works) however it is causing rspec test failures on Circle CI. Without passing --verbose the only message I see is "ERROR in". With verbose there is a bit more output:
ERROR in
Child extract-text-webpack-plugin node_modules/extract-text-webpack-plugin/dist node_modules/css-loader/index.js??ref--3-2!node_modules/postcss-loader/lib/index.js??ref--3-3!node_modules/sass-loader/lib/loader.js??ref--3-4!app/javascript/bundles/shared/components/styles/link.scss:
    PublicPath: /packs/
    Entrypoint undefined = extract-text-webpack-plugin-output-filename

I've searched for "Entrypoint undefined" and everything I can find is related to a Webpack 4 upgrade while Webpacker is still on v3.
I have a sample repo to recreate the issue: https://github.com/nburt/react_rails_template
Clone it, run bundle install && yarn install and then run bin/webpack --verbose to see the error.
Any ideas?
Thanks!

What version of rails/webpacker?

seeing it with webpacker 3.5.5 and rails 5.2.1 and with webpacker 3.5.5 and rails 5.1

@justin808 just wanted to follow up - have you had a chance to take a look/been able to reproduce this as well?

@nburt My team and I have lots of experience setting up Circle CI and troubleshooting such issues.
However, I didn't get a chance to look into this as I'm booked helping my paying clients.
Would your company be interested in our ShakaCode Pro Support plan?

I've identified the issue as coming from the flow status webpack plugin. I've decided to just skip prepending that loader if we're building on CI.
This is how we're prepending the plugins currently (development.js). I've just added the if checking process.env.CI which circle adds.
if (process.env.CI !== 'true') {
  environment.plugins.prepend(
    'NoEmitOnErrorsPlugin',
    new webpack.NoEmitOnErrorsPlugin()
  );

  environment.plugins.prepend(
    'FlowStatusWebpackPlugin',
    new FlowStatusWebpackPlugin({
      failOnError: true
    })
  );
}

Thanks @nburt!


Support for Descriptors

For most of the BLE devices, we need to write/read/notify from their descriptors. As far as I checked, this library does not support this functionality. Is there any work around or plans to support descriptors?

Hi!
There is no workaround, but there are definitely plans to support descriptor operations. I'm unable to give you any specific date, but it will happen rather sooner than later.

I used another ble lib for testing, it had the options to set notify to true, for example like this characteristic.setNotifyValue(true);
and then
characteristic.listen() ..... for values
For this char uuid 00002a5e-0000-1000-8000-00805f9b34fb and it worked
I am trying to do the same with this lib, but it seems that this is not possible for now or i dont know the right way doing it.
I/flutter (27954):  Characteristics: 00002a5e-0000-1000-8000-00805f9b34fb
I/flutter (27954):  is Indicatable: true
I/flutter (27954):  is Notifiable: false
I/flutter (27954):  is Readable: false
I/flutter (27954):  is Writable With Response: false
I/flutter (27954):  is Writable Without Response: false

Hi, @JureLokovsek.
If you want to get notifications/indications from a characteristic, call characteristic.monitor().listen().
Here's an example: https://github.com/Polidea/FlutterBleLib/blob/master/example/lib/test_scenarios/peripheral_test_operations.dart#L207
I'd appreciate it, if you created a separate issue next time.

BTW @aytunch I'm currently working on support for reading from/writing to descriptors. Should be out somewhere in January/early February for both FlutterBleLib and BLEmulator.

Hi, @mikolak
Thank you for your answer, the suggestion you gave worked fine for me.
I am doing some researching and testing on flutter with some bluetooth devices, next time I will open seperate issue. Tnx again.

@mikolak I am very glad to hear this. Hopefully after the descriptor support, I will be able to finish up my miband3 project:D

@aytunch Descriptors have just been released in 2.2.0.

@aytunch Descriptors have just been released in 2.2.0.

@mikolak Just a small reminder. the Todo does not reflect the descriptor support. thanks btw.

@aytunch What do you mean? What Todo?

@mikolak I am sorry, I was talking about readme:
https://github.com/Polidea/FlutterBleLib/blob/develop/README.md

Ah, sorry! Forgot about it! Thank you for letting me know, fixing it right away!


Adding grammar for precondition a attributes

Hi, I've just added the required grammar for parsing precondition a attributes based on RFC 3312.
Sample:
v=0
.....
a=curr:qos local none
a=curr:qos remote none
a=des:qos mandatory local sendrecv
a=des:qos none remote sendrecv
a=sendrecv
.....

I added parse and write tests for precondition added grammar.

Hi Eirik,
I've just added the MRCP attributes support as well (grammar + test).

Hey, i've updated testing setup now, if you're able to update - and it's all green - i can try to push this out later :-)

Codecov Report
Patch coverage is 100.00% of modified lines.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files Changed
Coverage

lib/grammar.js
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


Export response type when fetching from storage

Hello people,
I wanted to use the response type of supabaseClient.storage.from('picture').upload('fileName', file) but there was no type exported so I wrote a type definition and exported it.
I assumed, that no issue would be needed for my PR (https://github.com/supabase/storage-js/pull/177) but there it is.

I hope "documentation" is appropriate.


Clarify how to check for modifier keys in Keyboard events

The documentation surrounding keyboard events did not match the examples given, since there are actually two proposed methods of getting modifier key state.
This change documents both methods in more detail.

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

@Karl-Bridge-Microsoft: I've proposed a small docs change here, but I think we need to update the code samples as well.

@citelao @Karl-Bridge-Microsoft - If the update is approved, would you correct the URL in line 108 to add the missing / (.com/uwp)? Thanks.

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

@citelao @Karl-Bridge-Microsoft - If the update is approved, would you correct the URL in line 108 to add the missing / (.com/uwp)? Thanks.

Fixed :D

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

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

I resolved the merge conflict. There's an update in master, added since this PR was opened, that changes line 106. So, it's incorporated into this PR.

#sign-off

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


Order specifications semantically rather than by upload date

The solution here may prove helpful (though ApiCenter's SQL dialect doesn't support PostgreSQL string_to_array)

Would it make sense to force a certain versioning scheme (i.e. SemVer)?

I think so, yes, I don't see how we can reliably order versions without. Especially considering ApiCenter will possibly in future track dependencies between versions.


SDC-16182: BYTE_ARRAY input support to JSON Parser

This avoids the exception generated by a BYTE_ARRAY simply input by adding support for it.  It assumes UTF8 as the input charset since that's the canonical format for JSON.  If another format is wanted, the field has to be manually converted via a Field Type Converter.
This probably fixes the original issue unless there is another field type which can't be converted to String.  I am pretty sure the InputStreamReader also throws some kind of exception if the byte array can't be parsed as UTF8 but I found nothing in the JDK documentation.
There is a test case attached to the Jira issue.

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


Don't Have Access - 2.1

I get the "You don't have access to go here" message on my main world. I don't have any other worlds besides a nether. I get this message when I first join the server. I also spawn inside a block. When I run the "mvlist" command no worlds show up but if I run the command in the console the worlds show up.
After some more testing I have found that if I set myself as OP then I have none of the problems above. I have opfallback set to true in the config.yml. Comparing this to 2.0 installed, I have none of these issues.

opfallback has no effect in 2.1 I have taken it out.
Give yourself (and your users multiverse.access.*) and this problem will go away.

Setting that permission did get rid of everything but I am unable to use portals. I have "multiverse.access.*" and nothing happens when I go through a portal. The portals work if I set myself as OP.

K will investigate, think these two are related/the same
https://github.com/Multiverse/Multiverse-Core/issues/174


Tag new version

Codecov Report

Merging #105 (ce5f76d) into main (8d56082) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
- Coverage   64.18%   64.02%   -0.16%
==========================================
  Files          21       21
  Lines        1921     1921
==========================================
- Hits         1233     1230       -3
- Misses        530      534       +4
+ Partials      158      157       -1

Impacted Files
Coverage Δ

test/util.go
80.87% <0.00%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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


fix(hooks): prevent duplicated page view events

Addresses OT-332.
Purpose
The usePageViewTrigger fired multiple times when used inside a useEffect hook, leading to duplicated events. This happened because the function returned by usePageViewTrigger was recreated on every render.
Approach & Changes

wrap the page view trigger in useCallback

hi hi @connor-baer, super thanks for the quick fix.
Just a quick check. When you try this, did you also see page-reactivated being trigger 2 times? I keep seeing this in my test code but not sure if this happens to you too?


corrected a one_hot example, fixed mark down formats

This PR corrects a one_hot() example, additionally fixes mark down formats.

Can one of the admins verify this patch?

Our docs are automatically generated from source: can you modify the version here instead: https://github.com/tensorflow/tensorflow/blob/d8e8b872eae63188c75046d5bb068e03a81b3f85/tensorflow/python/ops/array_ops.py#L1760

Now most issues are gone :-p

@tensorflow-jenkins test this please
(our markdown generator I guess isn't perfect in terms of spaces and tabs I guess?)


Color

Is there a way to change the color that lights up when you type a character or remove one? I see your script but I'm not sure where it is so I could edit the colors. Thanks c:

look at line 38 :)

Ye but where is the file located so I can edit it? I might have worded that incorrectly.

ah, if you installed it via AUR then it's in /usr/bin/betterlockscreen

Thanks!

next time just type whereis <program> and it'll tell you where it's located :)


Add Kubernetes example

This example starts a fairly complete Kubernetes cluster and showcases
perfectly reproducible remote execution via the local remote execution
toolchain containers.
This example uses a three-layer setup process:

The infra layer is a kind cluster with Cilium and MetalLB. This layer
is built to be easily swappable with more "production grade"
clusters.
The operations layer deploys a few standard applications that are not
inherently required for NativeLink, but are solid deployments that
one would likely want running in a cluster. This includes monitoring
and handling image availability.
The application layer is a straightforward kubectl apply -k . which
deploys a NativeLink CAS, Worker and Scheduler. This deployment
differs from the Docker Compose setup in that it does not make use of
any system paths and doesn't allow visibility "outside" of the node
itself. That is, it's a hard requirement that the worker image is
self-contained. Storage is fully ephemeral in this example and a
kubectl delete -k . will destroy the cache for quick iterations and
cache testing.

This change is

Thanks @aaronmondal !


Format Document chokes on comments

It happens when there are comments in the .yml file. The issue happens on this line:
emitter.Emit(parser.Current);

...in this file https://github.com/aaubry/YamlDotNet.Editor/blob/master/YamlDotNetEditor/CommandHandlers/FormatDocumentCommandHandler.cs#L55
So the error happens in the YamlDotNet NuGet package somewhere that can't be fixed from the extension itself.
Repro steps
In a YAML file containing this:
# version format
version: 1.0.{build}

Now invoke the Format Document command (Ctrl+K+D) and you get the error.

The problem must be that the parser is configured to return comments, but comments are not officially part of the YAML document and the emitter does not supports them.

So perhaps the fix for now is to remove the support for Format Document then?

Not necessarily. Since the parser has support to return comments, it would make sense to support them in the emitter. I added support for comments because it was necessary for this extension, but at the time I did not need to emit comments, so I forgot to add them to the emitter.
I will take a look to the code to see what would be required.


Removed specific VC toolset requirement.

Prevents needing to change this setting to default in order to build with updated toolsets.

I don't know if the specific toolset is set for a good reason, but here's a fix if not.
It's a small fix, if you decide to keep the change feel free to do it yourself and reject this request.

Closing. I probably should have used the issues page, in retrospect. Derp.


update go-manifold

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

What's the status on this one, Dom?


Add documentation of the .visual metadata

WIP: needs to be expanded.
Addresses issue #64.
@cdeil, @joleroi - as we expand and complete this, please commit directly to this branch

@keflavich - Please rebase.

@keflavich - This PR is from May. Do you plan to still work on this? (It says WIP at the top) Or would it be OK to merge this in now and then you or anyone else can continue with other PRs any time?

best merge this & raise a new issue

Merging now. Thanks!


fix: update cache if owns entities are changed

Cache was not updated for owned entities:
modelBuilder.Entity<User>().OwnsOne(u => u.Notifications);
This is because GetChangedEntityTypes returns only the owned entities and does not return the owner entities.
Unfortunately, I have no way to fix this for netstandard1.3 and net451.

Thank you.