[Bug] Update Sankey graph node, animation discontinuous

Version
5.4.1
Link to Minimal Reproduction
https://echarts.apache.org/examples/zh/editor.html?c=sankey-vertical&code=MYewdgzgLgBAJgQyggKgU2jAvDA2gKBhgG9Cj4kEAuPM8kmMBAWzRoHIF2AaGASzgcuvPlDTMAylACeAGzYNQskACcOAYgBmmgCxwdm9jAC-vOGgAOUABY0ADCe51yxRiwXsARj36CYXn1FxKTkFVyVVDW1gAFYAZjgjU3hLGxoARkdnIlcmVg5gQL92QpExSRl5GnCQZTV_LU1YhKSzVNsYTNNshjyPRJFigf5ykKrFWsiGuIA2YCaATlaUqw6AJiz6HLd8_zQijn2y4MqwmAj69nVZ-eAlxxW0mA3urd73DkNBz8DR0-rzpNLtc5otluZVjQXk43rkPv4AOYHRG_E6hAEXDQ3MEPCFPF7OAC6MPIsj4YAA1hAaARYTAICAAK4qYAeYQwZAqBFoKAcby8ABuCFkjIUM02W1cDOZrKEPk53N5_lKMCFIoUOgl9ClTJZHn5HIQXJ5HGGatFNDiWpc9N1sv8BoVJr2PnNCitr0ltplHhVTqV7C-quFFpgHpJ2u9eoK8qNio4SMFIYUXSJZE9pHoiGQNJ6cN2nGRheOFXREzqUV0-kMuPa9mt2z6fKLBqCpfGNQrDWi8UStchnQb7wLKoEMZLYzOmO7TV74Lrg89kab_mGY9XqPbU6BWNBd3nA-heZ2HiOvkOm8nGJ30z392SePWQ_zHiD68Dl_-5amV2x-_7-LPieCZFomIxoh2gJdr-d4HoBxhEhGMBkpS1K0HS0rRv47L-s2SbqjQmpLjamH2sWhrGgGKpupaQGkfqsaUaarrJrRxHbPReEUfGLr4aGXRIRxdq-oxPEfnxChHhhwnjtxzrsGBNGLmmRAZs42bUOhXoruR77sm2V7fsC2h6AYcEdA47HDgx3wOp-Zadj-jTNH2D4LgJx46aOxSjn8DlQU5PYtABHQeXSOlrkM9mQdOMG3PebSHkBOlnu-qV-TFN5xTiblJVZL4_LZ4ngVu17QSC8XmVCyXwgpoHRdu5V_gljxPgh9DEs4KFUrm0k-nKvC4XZEk0AA7HRMnYaJ8nUaxMBEYJDCccNckBmac1SV6y0BINcbyWeSkxBN_XKtNAZBkpYVbZNJRnSBI1hiYKlPYSADc-D4CAVh8OA2AkGQEBoCofAYL1Lg9O-EAIJSaDSDwPQ6VDMNw4tMgWB4SMUrD8NvBpNKdW83VobgBNbNDfDMEgP1gAAIsyVPgBkMR2HYi3k5TUDU3TKgM2AACqFjZimzOswjIDmAA6gITzpKLbyMmAfACkDUOyCgPOQKI1M0FAKiiotsgINITJQAAkmIPOc-AaFy1sqgg2AAbKyonPAMKONbChaCGZmbwBZcCI83ADtQOwPTteQEeEvgxjvfg8iwOS5gAB5_XY72A2bjtA2qAAUucAJTYAAfP9RBfVbYAAHSA8DGC4HYhJVxpf0aeg0C4EnaDJ03GnveX33gDXQMgxADdN0TreUO3UCd2AKcT-SVL9zAzDSAAwtYRpQMPUAAPKD2AucV9TBcrwA9OfgKQLUaBV8oCK52vm_b1XioAFoqIXNdQKoCDclXAA-ioEAIAoBj3SE3QBwBrB8FkHAFQaAwDjyrgIM-ZAu7JwANRYJXnwTQMBc6YJgMXHAbcMA73kGABENgi6-yIMQnA6d0wx14DoFmdgz5AA
Steps to Reproduce
Visit the link above and wait a few seconds
Current Behavior
Nodes are redrawn each time, discontinuous animation
Expected Behavior
With the same id, simply update the style property of the node, not redraw it.When the Sankey graph data changes, the movement of nodes is tracked through animation
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
Or could please explain why have to regenerate the node every time

fix it: Sankey diagram itself has no updated animation, The animation is caused by universalTransition. But the cause of the discontinuity is still not found

yes, universalTransition is not listed as option in sankey, so it should not be used.
Without universalTransition the transition and animation look correct. Where is the discontinuity ?

I delete universalTransition and try again:custom sankey, lost morph animation. How can I get morph animation without universalTransition

I delete universalTransition and try again:custom sankey, lost morph animation. How can I get morph animation without universalTransition

Supplement: I mean when there is morph animation, the animation only looks continuous. The Sankey chart is now updated with almost no animation


feature: Opening an Issue for Enhancements to Questions Page (Loader, Confirmation Dialog, UI Changes, Mark as Resolved)

Hi Maintainers,
I’m opening an issue to track the improvements I’m working on for the Questions Page. Here’s a quick summary of what I’m working on:
Planned Enhancements:

Loader on Submit Button
A loading spinner will be added to the submit button when a question is being submitted to enhance user feedback.

Confirmation Dialog for Deleting a Question
I  will add a confirmation dialog to prevent accidental deletions.

Minor UI Changes
I’ll be removing the toast notification that appears when a vote is removed from a question.
Mark as Resolved Feature

I plan to add a "Mark as resolved" option when viewing a question.
I’m currently working on these changes and will update the issue once everything is complete. Please let me know if you have any concerns or suggestions.

Thanks,
Rahul

Have u already started working on it?

yes , will be done by tommorow

I have checked linting issues if they are failing again , please let me know


prostats.gg

We can create a Website called prostats.gg using all the artifacts from different EDAs.
It will be an interactive read-only website where users can see stats about different players and related graphs and images.

Please feel free to create issues and discuss the requirements for this project.

can I work on this issue??


feat: update golang-github-anacrolix-missinggo to 2.1.0

Stuff that's missing in Go stdlib, or hasn't made it into its own repo.
Issue: https://github.com/deepin-community/sig-deepin-sysdev-team/issues/552
Log: update repo

/lgtm
/approve


Hide trailing separator in PR and commit files dropdown

Test URLs

PR file
Commit file

Screenshot
Before

After

Isn't this a GitHub bug? 😃


Cole-Cole

I received an email today, stating:

The question I have is related to induced polarization. A popular representation to 'describe' this phenomenon is through Cole-Cole parameters. Am I right in that empymod cannot handle this particular implementation? I have been looking at the documentation and examples, but apparently missed something (I hope).

This issue is hence to create a routine to calculate IP with Cole-Cole-parameters.

Resolved by #29


Hello

Description of the change

Description here

Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related Ticket

#1w3e1n3

Code review

[ ] This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

Task linked: CU-1w3e1n3 Hi


Option -a to forward argument

Hello,
In the Golang version, I used a lot the argument -a that allows to forward option to ssh (https://github.com/quantumsheep/sshs/pull/51).
Would it be possible to add the same argument on the rust version ?
Thank you

Hello ! You can use the --template option:
sshs --template 'ssh "{{{name}}}" arg1 arg2'

Does that fit your need?

Yes thank you I did not think about that.


Don't necessarily clone nodes, but move them, when using data-turbolinks-permanent

Currently, if I use data-turbolinks-permanent, nodes are being cloned into a new document.
This is not really the behaviour I need, because:

it removes event bindings
f.e. if this element has some iframe elements in it, they will be loaded again

If we would just move the node from one document to the new one, we could avoid it.
What do you think about it and how we could implement it?
P.S. My usage example: I want to create a "music player widget" that is fixed to the whole webpage and shouldn't be reloaded between requests. It has some event bindings (f.e. $(".play-button").on("click", startPlayer)). Also it can contain f.e. a youtube iframe in itself - and obviously I don't want the iframe contents to be reloaded, so the video can keep playing.

This issue came up in https://github.com/rails/turbolinks/pull/472/files#r25563876
I'll look into it today or tomorrow.


Add a wiki: Fear & Hunger Wiki

Link to origin wiki: https://fearandhunger.fandom.com/wiki/"Fear_and_Hunger:_the_Tormentpedia"_Wiki
Link to destination (independent) wiki: https://fearandhunger.wiki.gg/wiki/"Fear_and_Hunger:_the_Tormentpedia"_Wiki
Indepedent wiki exists and is of better quality than fandom wiki. Content warning for NSFW and general gruesome imagery and themes.

Fear & Hunger was added to Indie Wiki Buddy in v3.4.0 (released for Chrome on January 29, and Firefox on February 1). Please let me know if you're experiencing issues with the redirect.

oh, huh, mine's just not updated. my apologies.


script silently swallowing errors

@xinredhat @Roming22
There are many occurrences in  the ckcp script of things like this where the error output is redirected to the standard ouptut, which is sent to /dev/null
We should never do things like that. It makes troubleshooting impossible.

I totally agree. For now  It's very difficult to debug them when failure happened. Every time I took lot of time to reproduce the failure by removing the output redirection.

This issue was fixed, we can close it.


Initializing timer with some start value.

Hi,
I  came across this easy to use task-timer a few days back, it's really wonderful. I want to show elapsed time in hh:mm:ss format, which I could show. But now when I have to give some start-time to timer, it is not showing perfect time. I wanted to initialize the start time to 4 seconds so, I wrote the code below:
<timer interval='1000', start-time='4000'>  {{hhours}}:{{mminutes}}:{{sseconds}}</timer>

The code above doesn't seem to work as timer takes some wrong value as shown below:
12:09:28
Can some one tell me why it happens or there is certainly something wrong in my syntax? Plus can we change the start time of the task timer to some value dynamically?
Thanks,
Supriya

The issue is that it is passing the value of start-time to moment which is seeing it as the number of milliseconds in a Unix timestamp. This means that you are some time in 1970

"Angular-timer" probably uses UNIX Timestamp.
A UNIX timestamp, also known as Epoch Time or POSIX timestamp, is a representation of a moment defined as the time that has elapsed since a reference known as the UNIX epoch: 1970-01-01 00:00:00 UTC
start-time translate the given parameter to milliseconds since 1970-01-01 00:00:00


.board()

It just doesn't exist. Even looking through your code. There's no function .board() that returns an array.

https://github.com/jhlywa/chess.js/blob/master/chess.js#L1356
Not pushed to NPM yet.


Full coverage tests

Looking at our fuzz failures like https://github.com/jgm/djot/issues/106, I think it would be fun to arrange "full-coverage" tests. Basically, theer's only a handful of characters djot recognizes as special [ -*|[]{}()_`:a]. So we can do the following: select every subset of 4 characters, and generate every string of length five, for 1_397_760 tests.
cc https://matklad.github.io/2021/11/07/generate-all-the-things.html

Feel free to play around with fuzz.lua.  The code is pretty simple.
Note that in generating the random strings, I already privilege the special characters (assigning them higher probability).  But getting full coverage on small strings is a nice idea.

There is also room to improve the failure minimization code. Currently I just subtract from beginning and end, but another transformation to try would be removing all non-special characters.


push images feature should use existing docker credentials

This seems like something that should be possible. As is you have to specify username/password via args ... which means you probably have your password plaintext in a file.

This is implemented.


Output reads with many sequence errors in the control as BAM/control/sequence_errors.bam, and exclude them from the analysis.

          Output reads with many sequence errors in the control as `BAM/control/sequence_errors.bam`, and exclude them from the analysis.

Additionally, exclude reads from samples that are similar to those in BAM/control/sequence_errors.bam from the analysis as well, outputting them as BAM/sample/sequence_errors.bam.
Originally posted by @akikuno in https://github.com/akikuno/DAJIN2/issues/60#issuecomment-2423584016

Solved: https://github.com/akikuno/DAJIN2/commit/b2e717faa5ed17cdc18217d537cf49de6ca7c0b4
It will merge at v0.5.6.


[ADD] Added *.pyc and *.DS_Store

Reasons for making this change:
I would like to simply add *.pyc and *.DS_Store into the python .gitignore

.DS_Store files are not Python-related.

Ok i will update it and remove

Why add *.pyc? It looks like this pattern already has you covered.
https://github.com/github/gitignore/blob/a60d59238afb22523038b549b9b27f15734f7563/Python.gitignore#L3

yes but it doesn't seem to work, perhaps this happen on my local so i guess i will close it


Icons dont render if dont wait on startup

Describe the bug
I have some icons in a note that I show in a canvas and normally they render fine. However when I open obsidian and then open said canvas quickly some of the icons aren't properly displayed. Reopening the canvas solves this issue.
To Reproduce
Steps to reproduce the behavior:

Add some icons to a note
Add that note to a canvas
Close the canvas (or go to another note)
Close obsidian
Open obsidian
Quickly open the canvas before obsidian finishes starting up
See that some icons dont render properly

Expected behavior
The icons to show properly, or maybe just hidden until they are fully loaded
Screenshots

Additional context

I have the same issue, I hacked up a way around this for now by using the Commander plugin. I made a macro that waits 4s and then closes the active tab and then reopens the page and then the icons render correctly.


feat(kudoctl): get information about nodes on the cluster

Adds "get nodes" and "get node " commands

Please rebase that PR and I'll merge it.

rebased !


net/http: Docs update for connection reuse

What version of Go are you using (go version)?
go version go1.10.2 windows/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\firefart\AppData\Local\go-build
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\firefart\go
set GORACE=
set GOROOT=C:\Go
set GOTMPDIR=
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\firefart\AppData\Local\Temp\go-build959778686=/tmp/go-build -gno-record-gcc-switches
Suggested Change
Currently the docs on https://golang.org/pkg/net/http/ say that you need to close the response Body after you are done with it so golang can reuse the connection. In my tests it turned out, that you have to actually consume the body before closing it otherwise the connection will not be reused.
I think it should be stated everywhere in the referenced docs where Close is mentioned that you have to consume the body too. If you do not consume the body golang will create new connections on every request.
If you aren't interested in the body, you need to do smth like
io.Copy(ioutil.Discard, resp.Body)
defer resp.Body.Close()

This blog post also describes the problem:
https://awmanoj.github.io/tech/2016/12/16/keep-alive-http-requests-in-golang/
Here is a sample program to demonstrate the problem:
package main

import (
	"flag"
	"io"
	"io/ioutil"
	"net/http"
	"sync"
	"time"
)

var (
	client = &http.Client{
		Timeout: 30 * time.Second,
	}
	discard bool
	threads int
	url     string
	wg      sync.WaitGroup
	c       chan struct{}
)

func main() {
	flag.BoolVar(&discard, "discard", false, "Consume body")
	flag.IntVar(&threads, "threads", 8, "threads")
	flag.StringVar(&url, "url", "", "url")
	flag.Parse()
	wg.Add(threads)
	c = make(chan struct{}, threads)
	for i := 0; i < threads; i++ {
		go func() {
			for range c {
				req, _ := http.NewRequest(http.MethodGet, url, nil)
				resp, err := client.Do(req)
				if err == nil {
					if discard {
						io.Copy(ioutil.Discard, resp.Body)
					}
					resp.Body.Close()
				}
			}
			wg.Done()
		}()
	}

	for i := 0; i < 200; i++ {
		c <- struct{}{}
	}
	close(c)
	wg.Wait()
}

Here is the timing output from only closing the body, and the other time with consuming it before. Each test was executed twice to make sure it's no network based problem. Also a https URL was used to demonstrate the problem because the TLS handshake takes a lot of time.
PS C:\Users\firefart\go\src\> Measure-Command { .\test.exe -url https://firefart.at }

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 3
Milliseconds      : 487
Ticks             : 34877955
TotalDays         : 4,03680034722222E-05
TotalHours        : 0,000968832083333333
TotalMinutes      : 0,058129925
TotalSeconds      : 3,4877955
TotalMilliseconds : 3487,7955

PS C:\Users\firefart\go\src\> Measure-Command { .\test.exe -url https://firefart.at }

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 3
Milliseconds      : 511
Ticks             : 35116646
TotalDays         : 4,06442662037037E-05
TotalHours        : 0,000975462388888889
TotalMinutes      : 0,0585277433333333
TotalSeconds      : 3,5116646
TotalMilliseconds : 3511,6646

PS C:\Users\firefart\go\src\> Measure-Command { .\test.exe -url https://firefart.at -discard }

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 0
Milliseconds      : 948
Ticks             : 9482683
TotalDays         : 1,09753275462963E-05
TotalHours        : 0,000263407861111111
TotalMinutes      : 0,0158044716666667
TotalSeconds      : 0,9482683
TotalMilliseconds : 948,2683

PS C:\Users\firefart\go\src\> Measure-Command { .\test.exe -url https://firefart.at -discard }

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 0
Milliseconds      : 904
Ticks             : 9042128
TotalDays         : 1,04654259259259E-05
TotalHours        : 0,000251170222222222
TotalMinutes      : 0,0150702133333333
TotalSeconds      : 0,9042128
TotalMilliseconds : 904,2128

You can also verify this behaviour using wireshark and have a look at the source port. If it changes on every request there is no connection reuse.
So it would be great if you can add this case to the net/http docs

https://golang.org/pkg/net/http/#Response

The default HTTP client's Transport may not
// reuse HTTP/1.x "keep-alive" TCP connections if the Body is
// not read to completion and closed.

I believe this is sufficient, and pretty clearly mentioned.
Please let us know if you had anything else in mind or maybe you missed this part ?

Here https://golang.org/pkg/net/http/#Client.Do:
If the returned error is nil, the Response will contain a non-nil Body which the user is expected to close. If the Body is not closed, the Client's underlying RoundTripper (typically Transport) may not be able to re-use a persistent TCP connection to the server for a subsequent "keep-alive" request.

I think it would also be good on top in the examples section to put a example with io.Copy(ioutil.Discard, resp.Body) if you just do a GET request to check the return code.

@agnivade the text you've highlighted is clear on what needs to be done (read to completion and closed) to avoid something bad (may not reuse connection), but I think the problem is the other places where closing the body is mentioned. This is a very common complaint so I think it's worth digging deeper on this.

Overview: "The client must close the response body when finished with it" The code that follows has ioutil.ReadAll, but "finished with it" is vague. "finished with it" could be interpreted as used it to completion OR not interested in using it all and ignoring it. So someone might think that if they aren't using the response body, then this rule doesn't apply to them.
Client.Get: "Caller should close resp.Body when done reading from it." "done reading from it" is less vague than "finished with it", but for some reason the word "should" is used instead of "must", which weakens the point. But even if it had "must" someone might think that if they don't want to read the body, this rule doesn't apply to them. Also this rule doesn't explain the consequences of ignoring it.
Client.Post: "Caller should close resp.Body when done reading from it."
Client.PostForm: "Caller should close resp.Body when done reading from it."

Perhaps we've been trying to avoid duplicating:

The default HTTP client's Transport may not reuse HTTP/1.x "keep-alive" TCP connections if the Body is not read to completion and closed.

all over the docs because it's verbose and because some people might not care about connection reuse. One alternative is to to be more concise and say "the caller must read the body to completion and close" without saying why, but that's dishonest since they don't really need to in all cases.
@bradfitz am I missing any nuance here?

I see.
Then this line  If the Body is not closed, the Client's underlying RoundTripper (typically Transport) may not be able to re-use a persistent TCP connection to the server for a subsequent "keep-alive" request. in https://golang.org/pkg/net/http/#Client.Do is technically incorrect I believe. Because you need to close and read body.

Exactly. I would love to see it documented in every function but it's a lot copy paste. Are there references in godocs which we can use so the phrase is only documented once?
Another option would be to change the Body.Close() method to automatically read all data before closing it but I think this would be a bigger change

Didn't this "read body to completion" come up as a security issue at some point? I don't remember if it was just server or client related (or both), but basically if Go automatically read the body to completion than it would be trivial for the program to be DoS'd (cpu/memory consumption). The counterpart could just send an infinite body and the Go program would sit there consuming it all, and if you're not consuming it explicitly in your code (ioutil.Discard or otherwise) then the developer of the program might not know it was happening.
I thought I saw a commit a while back that had Go read part of the body to a certain point but if it kept going then it would close the connection abruptly, to avoid this potiental DoS. If thats the case, and you make another request while some goroutine is doing this, it would make sense that a new connection would be made (what you're seeing here and why reading the body to completion is the "solution"). In fact, if you don't read the body to completion from both a security and "HTTP" standpoint doesn't it make more sense to close the connection on (or near) body close and reopen a new connection when needed? That way it's only one connection at a time.

The default HTTP client's Transport may not reuse HTTP/1.x "keep-alive" TCP connections if the Body is not read to completion and closed.

To be precise, what does it mean for a Body to be read to completion?

If the Body is not closed, the Client's underlying RoundTripper (typically Transport) may not be able to re-use a persistent TCP connection

The RFC verbiage doesn't help me understand under what conditions it may or may not reuse the connection. I suspect this was chosen because the request might return an error, but this information is already fuzzy when reading the package documentation:
# note: godoc is gone in 1.12
# it now has to be installed separately
# go get github.com/golang/tools/cmd/godoc
; godoc net/http

package http
    import "net/http"

    Package http provides HTTP client and server implementations.

    Get, Head, Post, and PostForm make HTTP (or HTTPS) requests:

        resp, err := http.Get("http://example.com/")
        ...
        resp, err := http.Post("http://example.com/upload", "image/jpeg", &buf)
        ...
        resp, err := http.PostForm("http://example.com/form",
                url.Values{"key": {"Value"}, "id": {"123"}})

    The client must close the response body when finished with it:

        resp, err := http.Get("http://example.com/")
        if err != nil {
                // handle error
        }
        defer resp.Body.Close()
        body, err := ioutil.ReadAll(resp.Body)
        // ...

Above, the example fails to convey the careful dance required to make connections reusable. The error
handling and return is replaced by a // handle error comment that falls through to the defer.
Second, the Body is consumed and assigned to a variable called body. This doesn't show what the user needs to do if they don't care about the Body.
The number of users that do this correctly in private repositories is none. I have never seen it, nor can I remember the last time I've seen it in open source projects. I suspect it's because of the term finished, which does not imply that the body must be read in full. I don't think saying must be read is clear either, because an io.Reader isn't guaranteed to consume the Body with one call to read. The term is likely to be lost in translation.
I think the term consume is more appropriate than finished or read.

paging @bradfitz. Thoughts on this ? We need some clarity on how do we make the docs more clear. In particular @meirf's comment here outlines the various places where things are not uniform.

Took me some hours to find this hack
Adding io.Copy(ioutil.Discard, resp.Body) and I finally get connection reuse

From what i see in the source code, response.Body.Close() method should take care of draining body:
https://github.com/golang/go/blob/9d23975d89e6cc3df4f2156b2ae0df5d2cef16fb/src/net/http/transfer.go#L979

This is the server code, the client body implementation is in transport.go

@dmarkhas @drscre this issue has gone stale. I am going to close it based on @bradfitz 's comment https://github.com/golang/go/issues/26095#issuecomment-455369447.
If there is work unaddressed I would appreciate if it you can open a new issue that captures the delta. Thank you.


transports don't need read/write locks

We use locks for read/write on the transports, but it turns out that the various transports are always run single threaded.  Therefore, there is no chance of interleaving.  This actually means that we can save both the lock/unlock for every message sent, but also a defer.

I should just look at the code, but does this work because the messages get serialized at the upper write queue? Basically multiple "user" threads are writing messages into a channel which is then processed by a single mangos thread?

the reason is that the protocol implementations always fire off a single goroutine to service each transport endpoint.  So there is no chance that there will ever be concurrency to worry about in this case.

Cool, I imagine this might actually improve performance a fair amount even though there's no lock contention, since it's on every message. And for free! :)


can the drawing pad starts with an image?

If I need to "edit" a signature can I get the old signature in order to see the old one?
thanks!

It could as a background with the drawingpad on top of it, but I don't think the image will get exported with that approach when you get the drawing through the API. There are some ways to do that on android I'm aware of, probably same for iOS just not familiar.

You can combine two bitmaps with nativescript-bitmap-factory plugin to get one, this answer shows the Example


Mass Bans?

What's the most efficient way to handle mass bans?
Right now looping through all bannable models and then applying the ban is making 3 queries each (So if I was to ban say 1k users, that's 3k queries and isn't good TBH). So if I was to do mass ban, then it's not really efficient.
So I wanna know if there's a better way to achieve this?

Please take a look at these tests I did on a project where I have this use-case.
My current implementation which is very simple and works based on status column with an int value and makes one query (The other query is auth, so we can exclude that from calculation for this).
Current implementation: 41ms / 1 Query:

VS
Laravel-Ban: 407ms / 297 Queries.

Interesting use case @irazasyed! I haven't thought about it before.
It will require refactoring to decrease queries count because right now logic as simple as possible.
Here is how it works:

We are creating Ban model (1st query).
In created method of BanObserver we are getting bannable model (2nd query).
In created method of BanObserver we are updating banned_at timestamp flag (3rd query).

There is one more possible solution. It will work pretty same as above, but will require queues.
Remove created event from BanObserver and add firing of ModelWasBanned event to BanService ban method. Then add listener which will perform updating of the Bannable model on background (via queue).
Then banMany method will accept iterable collection of Bannable models and call ban method for each of them.
By this way for 100 bans we'll have 100 queries with bans creations, 100 queued jobs and 100 more queries in those queues with updating Bannables timed flags.

By this way for 100 bans we'll have 100 queries with bans creations, 100 queued jobs and 100 more queries in those queues with updating Bannables timed flags.

That sounds like a lot of processing and work using Queues and all.
Can't we just use createMany directly somehow using the Ban model and bulk update the Bannables timed flag in one query and then fire the event with all this info.
Anyone listening to that event can simply check if it's a collection or a single Bannable model, etc. (whatever is being passed -- I haven't checked in-depth).
I'll explore and play around with this and see if I can figure something out.

I'm not sure it's good to fire event with all banned models as collection, because it will be harder to make users to listen only their own models events.

So I just setup a job to ban many users and while it does the job fine but I'm now facing another issue due to the observer as I'm banning the users over an API that's stateless and because of the way the observer is implemented, it now won't fill the created_by_id and created_by_type and there is no way to manually fill that either (Tried passing these two attributes to the ban method and also forced too). It's always null.
I think you should make these two values fillable and let me pass the $bannedBy modal manually (In my case, I can get the user who took this action over API using their token and API guard -- FYI, I'm using Laravel Passport).
I think we need to look into this to also optimize the number of queries somehow. Perhaps, Your first solution is good enough unless you have alternate ideas you thought of over the past month or so!

@irazasyed I agree that Observers usage should be revised. This feature is sweet, but I've started to face many issues with applications which using them a lot.
Their behaviour is not obvious. It's not that easy to omit event in Observer (especially if you need to skip only part of it).

@antonkomarev So until that is revised and or we come up with some solution, Can you at least add those two columns to the fillables array? That would be great for the time being and would be very helpful.

@irazasyed made changes in PR #35. It will be available in v3.5.0 today.


Allow users to use GPT-4 if they have access to it.

Hey! I created a PR that will allow setting the model to GPT-4 by setting the env var ORPHIC_USE_GPT4 to true!
This was a fairly straightforward process, and seems to work well.

This is a good idea- I'm still on the waitlist for GPT4 so it hasn't crossed my mind yet :-)
I don't think that doing this via an environment variable is the best idea though. I would rather contain the use of envs to just the API key, which is something that a variety of different pieces of software would want to access. I think it would be better to do this via a cli flag. In the future maybe a config file might also be a good idea.

Yeah, didn’t want to change too much of your project by creating a config file but I agree that’s the best case. Maybe even allow the OpenAI key to be overridden in it for people who don’t love global envs

Added this as a cli flag for now.


Fix Habitica auth problem for Facebook and Google connected accounts

Coverage increased (+0.06%) to 68.31% when pulling 3c95eba754902699639c5cc7012cb44e296e759a on feature/fix-habitica-login into 3c6d99d38a4fad517f24c347920aca0ce17210d5 on master.


Relative path set in href directs to fixture.page url value

What is your Test Scenario?
setting fixture.page(url = example.com)
and href property on the link = '../testRun/testRunId
and current url = https://example.com/projectId/testFolder
click the link and verify navigation to https://example.com/projectId/testRun/testRunId

What is the Current behavior?
Browser page navigates to to the url passed to fixture.page(url) + relative link https://example.com/testRun/testRunId
What is the Expected behavior?
Browser page navigates to the proper page relative to current url
What is your web application and your TestCafe test code?
web application is secure
occurs on t.click(Selector('a').withAttribute('href', '../test/relative/url/path')
Your Environment details:

testcafe version:                  1.9.1
node.js version:                    12.16.3
command-line arguments:  testcafe chrome -s path=output,takeOnFails=true
browser name and version: Chrome 84.0.4147.135
platform and version:          macos 10.15.4

Hi @jburgs
When the browser location is https://example.com/projectId/testfolder', the current fileistestFolderand the currentdirectoryishttps://example.com/projectId`. Since the link href value starts with .., the browser's location will be switched to the parent of the current directory. So, the testrunId/testId URL will be appended to https://example.com.
This behavior can be reproduced without TestCafe. Run the following code in the browser's DevTools console:

You can change the link's href value to ./testRunId/testid, or navigate the browser to https://example.com/projectId/testFolder/index.html instead. If you still feel that there is some problem caused by TestCafe, please share an example page that can be used to reproduce the problem.


Two-way binding model using a custom component

Hi, thanx for this component.
I'm using it with Vue 2.0 (version 2.0.0-beta.13) and I'm having troubles with v-model
If I directly use <multiselect> inside an HTML file, everything works (even with Ajax, I'm using axios); if I try to write a component (let's say <my-select>), using a .vue file, I can't make it work: the component renders correctly, but my model doesn't get updated when I select a value from the select.
I read the documentation, searched through issues, but I can't figure out how to use two-way binding with v-model, :value / :selected (which of these two?) @insert / @update (which of these two?) .
Sorry for bothering you. Tried various combinations, but I can't make it work.
When I use , which directives / options have I to pass in order to have my model updated?
Could you please provide a short example?
Thanks a lot

Here a jsfiddle: https://jsfiddle.net/eleftrik/o3x2zzow/
Thank you

For Vue 2.0 this will always be v-model and :value + @input. 1.x uses :selected but doesn’t support v-model for obvious reasons.
You forgot to add the this.$emit('input', newVal) in the updateSelected method.
Here’s a working fiddle: https://jsfiddle.net/o3x2zzow/7/

Thank you so much @shentao for your time, I really appreciate your help.

Sure! Always happy to help! 😃


[webview_flutter] fix not working choose file

Description
fix webview_flutter not working choose file
Only applicable to android5.0 or above
Related Issues
https://github.com/flutter/flutter/issues/27924
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@googlebot  I signed it!

I hope to merge soon.

Hello,
I have been select this fix. It is will crash when second time go to webview and select file.
E/AndroidRuntime(31232): FATAL EXCEPTION: main
E/AndroidRuntime(31232): Process: pl.cormo.ocenapolis, PID: 31232
E/AndroidRuntime(31232): java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=18, result=-1, data=Intent { dat=content://com.android.providers.media.documents/document/image:13549 flg=0x1 }} to activity {pl.cormo.ocenapolis/pl.cormo.ocenapolis.MainActivity}: java.lang.IllegalStateException: showFileChooser result was already called
E/AndroidRuntime(31232): 	at android.app.ActivityThread.deliverResults(ActivityThread.java:4423)
E/AndroidRuntime(31232): 	at android.app.ActivityThread.handleSendResult(ActivityThread.java:4465)
E/AndroidRuntime(31232): 	at android.app.servertransaction.ActivityResultItem.execute(ActivityResultItem.java:49)
E/AndroidRuntime(31232): 	at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)
E/AndroidRuntime(31232): 	at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)
E/AndroidRuntime(31232): 	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1831)
E/AndroidRuntime(31232): 	at android.os.Handler.dispatchMessage(Handler.java:106)
E/AndroidRuntime(31232): 	at android.os.Looper.loop(Looper.java:201)
E/AndroidRuntime(31232): 	at android.app.ActivityThread.main(ActivityThread.java:6810)
E/AndroidRuntime(31232): 	at java.lang.reflect.Method.invoke(Native Method)
E/AndroidRuntime(31232): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:547)
E/AndroidRuntime(31232): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:873)
E/AndroidRuntime(31232): Caused by: java.lang.IllegalStateException: showFileChooser result was already called
E/AndroidRuntime(31232): 	at in.onReceiveValue(PG:8)
E/AndroidRuntime(31232): 	at io.flutter.plugins.webviewflutter.FlutterWebViewChromeClient.onActivityResult(FlutterWebViewChromeClient.java:204)
E/AndroidRuntime(31232): 	at io.flutter.app.FlutterPluginRegistry.onActivityResult(FlutterPluginRegistry.java:204)
E/AndroidRuntime(31232): 	at io.flutter.app.FlutterActivityDelegate.onActivityResult(FlutterActivityDelegate.java:132)
E/AndroidRuntime(31232): 	at io.flutter.app.FlutterActivity.onActivityResult(FlutterActivity.java:142)
E/AndroidRuntime(31232): 	at android.app.Activity.dispatchActivityResult(Activity.java:7590)
E/AndroidRuntime(31232): 	at android.app.ActivityThread.deliverResults(ActivityThread.java:4416)

@NorbertSzydlowski This is a known problem and you go to the first filePathCallback every time, I think it has something to do with the mechanism of flutter, you can set up the public static filePathCallback in the WebViewFlutterPlugin to solve it, but I don't think it's an elegant solution

@crazecoder Second problem is selected multiple files.
intent.putExtra(Intent.EXTRA_ALLOW_MULTIPLE, allowMultiple);
It isn't correct get URI from Intent.
WebChromeClient.FileChooserParams.parseResult(resultCode, data)
I have been used this method (from flutter_webview_plugin):
private Uri[] getSelectedFiles(Intent data) {

        if (data == null) {
            return null;
        }

        if (data.getData() != null) {
            String dataString = data.getDataString();
            if (dataString != null) {
                return new Uri[]{Uri.parse(dataString)};
            }
        }

        if (data.getClipData() != null) {
            final int numSelectedFiles = data.getClipData().getItemCount();
            Uri[] result = new Uri[numSelectedFiles];
            for (int i = 0; i < numSelectedFiles; i++) {
                result[i] = data.getClipData().getItemAt(i).getUri();
            }
            return result;
        }

        return null;
    }

@NorbertSzydlowski @crazecoder  how to use this PR before it's merged?

@fyeeme select PR as dependency
https://dart.dev/tools/pub/dependencies

希望修复

We apologize for the long delay in triaging this PR. We’re in the process of overhauling our PR triage system to respond much more quickly, as well as working through the backlog.

[ ]  My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).

Per Flutter policy, this will need tests before it can move forward; I believe that means it is blocked on the webview_flutter portion of https://github.com/flutter/flutter/issues/83358 since this would show native UI. We can revisit once that scaffolding is in place (and if anyone is interested in moving this forward, a PR to do that would gladly be accepted to accelerate the process).
Also, could you could elaborate on the discussion above? If this crashes when used more than once, then it doesn't sound like it's ready for review. Is there a blocking issue preventing fixing that crash?

Since this hasn't received any updated based on the comment above, and since the Android implementation of the plugin has been very substantially changed since this PR was created, I'm going to close it as obsolete. If this is still something you are interested in implementing, please feel free to open a new PR! (Or, preferably, collaborate on https://github.com/flutter/plugins/pull/3225)


New screen shot for rebooting server

... as Hetzner has updated the page.

Still the same for me:


BOCM-59 + BOCM-62

Describe your changes
Implementation of remaining case officer journeys
Refactor existing case officer code to share code with new pages
Refactor @pins packages for all /packages
Remove excessive jsdoc comments
Accessibility fixes for existing pages
Issue ticket number and link
https://pins-ds.atlassian.net/browse/BOCM-59
https://pins-ds.atlassian.net/browse/BOCM-62
Type of change 🧩

[ ] Bug fix (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
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[ ] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Parallelize Circle build

The Problem/Issue/Bug:

We're going to need to be able to do more parallel builds for Apache webserver types, and it wouldn't do us any harm to do it for multiple CMSs.
We need parallel of staticrequired anyway.
It's been too long coming :)

How this PR Solves The Problem:
Reorganize the circleci build
Manual Testing Instructions:
Review the builds (and look at the fancy workflows tab on circleci also, https://circleci.com/gh/drud/workflows/ddev
Automated Testing Overview:
This will actually let us lose the buildkite staticrequired build, so will save some resources on our buildkite stuff.
Related Issue Link(s):
Release/Deployment notes:

@andrewfrench I think this is ready for review. This is building at https://circleci.com/workflow-run/de08e69e-093c-4e73-9624-7da7b3de3d27 (unfortunately on rfay account instead of drud, I'll try to change that after this build completes)
I'm not going to try to do the nightly build or tag_build yet because they're lightly used at this point and I don't want to break something that might trip us up like that. We'll see regular build stuff right away so we can fix it, but those might trip us in the future.
If we can get this in I'll try to do nginx and apache versions of the build by keying on the build name and making minor changes to our code to test with different config.


logger_no_op.sql is Invalid

In the logger_no_op.sql file (at least in version 3.1.1), the logger.pkb component is not present. The file reads as such:
prompt *** logger.pkb ***

create or replace
begin
*
ERROR at line 1:
ORA-24234: unable to get source of PACKAGE BODY "GIFFY"."LOGGER", insufficient privileges or does not exist
ORA-06512: at "SYS.DBMS_SYS_ERROR", line 105
ORA-06512: at "SYS.DBMS_PREPROCESSOR", line 189
ORA-06512: at "SYS.DBMS_PREPROCESSOR", line 109
ORA-06512: at line 2

/

Can confirm. The script itself has the ORA Erros in it.


EditorFontSize equivalent for Explorer text

Will be good to have a settings flags for edit the explorer look, like  text size and font .

This issue has already been reported. Please use the search function next time before reporting an issue and/or this time if you are curious about the progress of the issue.

Really sorry I did not know baser


Commit to contributing to and supporting the development of related Web standards

In his review @triblondon made the following comment:

Assuming that AMP's vision is as an expression of the future capabilities of the web platform, then it is a de-facto polyfill, and the governance policy should commit AMP to contributing to and supporting the development of the standards that it promotes, where there remains a gap between what is supported by AMP and what can be done without it.

This feels like something that needs to be expressed in the Vision/Mission statement, not in the governance policy itself.

It would be nice to see some resolution to this, perhaps in the form of a vision statement that AMP's goal is to cease to exist, and that the project will work towards that by realising AMP's benefits into the fabric of the web platform.

While I don't think that there is broad agreement that AMP's goal should be to cease to exist (though I understand what's behind that statement), I do think that a manifesto that captures what AMP about in more depth than the current vision/mission statement (and in relation to the Web) would be a good thing. It's one of the items that I want to put on the AC's agenda.


feature request: add "approved for data access" role fulfulled as requirement for linking accounts

not sure if this is a "role" or some other sort of drupal user profile item, but we'd like to be able to confirm that a user has been approved by the PI for data access before they can link their AnVIL account.
in the future, we'll likely want further integration, such as for offboarding or potentially assigning users to groups (such as uploaders), but this can be done manually for the time being (and for a long time, if this would be difficult).

Closed by #235
The second paragraph is its own issue, so we can a new one when we need it.


Expose Webpack aliases for files processed by PostCSS

Proposal
A way to use aliases defined in webpack.config.js so we can use it in commonJS files processed by PostCSS
Use Case
For example using tailwindcss, we will be able to define an alias and referenced it in tailwind.config.js (which is processed by PostCSS via postcss-loader) to manage paths in a more simplier way.
//webpack.config.js
module.exports = {
  resolve: {
    alias: {
      TailwindPlugins: path.resolve(__dirname, 'src/assets/styles/plugins')
    }
  }
};

Implementations expected to work
//tailwind.config.js
module.exports = {
  //...
  plugins: [
    require('./src/assets/styles/plugins/reset'),  //✅
    require('@/assets/styles/plugins/reset'),      //doesn't work actually
    require('TailwindPlugins/reset')               //doesn't work actually
  ]
}

Implementation

[ ] I'm open to work on this
[ ] I need help/mentorship
[x] I'm unable to work on this

If expose aliases defined in webpack.config.js is not possible maybe an option available in postcss.config.js can be way to solve it.

require is part of node.js we can't change how they works, sorry


feat(#801): Correctly Map String Values With Unicode Characters

In this PR I successfully fixed the bug related to transformation of String values with unicode characters.
Related to #801.

PR-Codex overview
This PR focuses on enhancing the handling of Unicode characters in the Eolang project, particularly in the App, DataType, and XmlValue classes, while also adding tests to ensure proper encoding and decoding of these characters.
Detailed summary

Added a static final String Φ in the App class.
Modified the DataType class to use StandardCharsets.UTF_8 for byte conversion.
Updated XmlValue class to decode hex values into a UTF-8 string.
Added parameterized tests for Unicode character handling in XmlValueTest and XmlFieldTest.
Implemented a test for converting directives field names with Unicode in DirectivesFieldTest.

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

@yegor256 Could you review these changes, please?

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here.

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 10min).


getFile returns SFN

Hello,
I'm going to replace the old SdFat library on my project. (STM32 Rogers Core).
So far so good, after changing write/readBlocks to write/readSectors it works, BUT:
in SdFatConfig.h:
#define USE_LONG_FILE_NAMES 1
in my code:
file.getName(xstring, 20);
returns the SFN only.  (like "BIBIUN~1")
Did I missed something?
regards
Matthias

The default is LFN with:
#define USE_LONG_FILE_NAMES 1
Modify the OpenNext example like this at about line 87
 //    file.printName(&Serial);   << remove this line
    char str[50];           // ADD
    file.getName(str, 50);  // ADD
    Serial.print(str);      // ADD

Here is the result with some test files.
Type any character to start
         0 2019-05-07 10:43 System Volume Information/
        65 2014-11-11 10:14 With.Two dots.txt
       151 2014-11-14 11:29 A long name can be 255 characters.txt
        45 2014-11-11 10:03 LFN,NAME.TXT
       186 2014-11-11 10:14 lower.txt
       188 2014-11-11 10:14 MIXCASE.txt
       184 2014-11-11 10:14 mixed.TXT
        67 2014-11-11 08:57 Not_8_3.txt
        30 2014-11-11 08:54 OK%83.TXT
        33 2014-11-11 08:59 STD_8_3.TXT
        59 2014-11-11 09:59 With Blank.txt

Unicode is only supported for exFAT as I stated in a previous post.  You can format small SD cards with the ExFatFormatter example, it forces exFAT format.  You can also format small cards exFAT on a PC with Linux, Windows, or a Mac.
Try the ExFatUnicodeTest example.  It requires this setting:
USE_UNICODE_NAMES must be nonzero in SdFat/src/ExFatLib/ExFatCongfig.h
Since Arduino does not support printing Unicode, you will need to check the name on a computer that supports Unicode.  Here is the name.
Euros € test.txt

Ok, thank you for your reply. Now it works, I must have forgotten something in my master code (hundreds of lines to check...).
Unicode for EXfat works either, but the restrictions are to many to switch it on my project for now.
So, the thread can be closed, because it's sure a mistake of mine.
I have just one last simple question:
I want to change the directory by index and not by char*
So I know that the file index of my "MP3/" folder is 19.
I have:
SdExFat sd; ExFile dir; ExFile file;
I'm now at root:
  if (!dir.open("/MP3/")){     error("dir.open failed");   }
Works as expected.
But changing to index: ("19" is the index got with file.dirIndex() )
  if (!dir.open(&dir,19,O_READ)){     error("dir.open failed");   }
gives the error condition.
So here is my question:
Isn't it possible to open/change a directory by index number?
(necessary avoiding unicode file-/directory names)
1000 Thanks
Matthias

So here is my question:
Isn't it possible to open/change a directory by index number?
(necessary avoiding unicode file-/directory names)

How can you know the index of a file without opening it by name?  The index is not a file index it's a directory entry index.
A file uses several directory entries in a contiguous group.  The size of the group depends on file properties.
There may be unused entries between files.

I can get the directory entries with openNext and dirIndex, so I got back, as example:
"Mai Cocopelli - Einmal Weltraum und zur?ck.mp3", 8 ((= file index)
real Name : "Mai Cocopelli - Einmal Weltraum und zurück.mp3"
There is no chance opening the file by name caused by "... zurück" - So I need the index.
Same for folders: Example:
"Österreichische Musik/"
I cannot open it by name or short name, because it begins with "?":
"?sterreichische Musik/" so I need the index number of this folder to open it - again with openNext and dirIndex
The whole code is here:
https://github.com/madias123/STM32duino-RFID-Mp3-Player/blob/master/MP3-player-beta_SH1106_17/SD_Card.ino
With this method the user can see the file or folder name on the display and open it.

Maybe a good future improvement would be to allow the Unicode "C1 Controls and Latin-1 Supplement".
See this chart.
These are codes 0X0080 - 0X00FF. Since they are 8-bit they could be supported in 8-bit char strings.  I need to understand how Microsoft maps these to short names.

Good day. I have a similar problem with SdFat. Think you can implement unicode for FAT16 or FAT32 in the near future. Or the solution you mention in your last post. well thank you

I have no plans to expand the character set for FAT16/FAT32 in the near future.

Can you help me to solve this problem. I use your SdFat 1.1.3. When opening a file that uses special characters {Czech-Slovak}. these are replaced with {?}.
I cannot open those files by the pad, because the file names do not match. Example: (h? ll?! = hello)
Loading file name:   1.(file.openNext(sd.vwd(),O_READ))
2. file.getName(filename, sizeof(filename));
//Open the file in read mode.
if(!track.open(filename, O_READ)) this does not match, because the loaded filename is not as sure as it is stored on the sd card.
Of course, for names without special characters (English texts) it works.
Could you please help me? for example, if you didn't insert {?} instead of special characters, would it work? Is it possible to modify SdFat 1.xx? Well thank you

Could you please help me? for example, if you didn't insert {?} instead of special characters, would it work?

No, the special characters must be translated to Unicode in order to open the file.
The problem is that codes 0X80 - 0XFF in 8-bit character strings represent different characters depending on the locale for your computer.  I can't translate these code to Unicode.
See Locales and Code Pages.

OK so differently. I'm not interested in displaying special characters. Just that when the file with special characters load from the sd card, it was possible under the loading "name" to open for reading.
Example: name file is hello.txt , we use the read function, we get some numbers in dec. 72,101,108,108,111. The same numbers are used when opening the file.
If you use extra characters, they actually have only other numbers. example: 72,233,108,108.111
Sdfat does this by replacing the number 233 with another number (the "?" Character).
The fact that the number 233 will be represented absolutely do not mean the character, respectively, without display, does not matter.
The important thing is that the loaded "name" will be identical to the team in the file. If there is a number 244 representing some character, let it stay there and not change.
Practically, I need Sdfat to load all characters (more precisely, the numbers that represent the characters) in the range of 0x00 to 0xFF.
yes I will not assigned characters values 0x7F to 0xFF but it does not matter. These values are assigned by the characters of the superior system or the system that displays them.

You can try the following:
Edit SdFat-beta/src/FatLib/FatFileLFN.cpp
At line 92 make these changes to lfnLegalChar()
inline bool lfnLegalChar(uint8_t c) {  // <<-- replace char with uint8_t
  if (c == '/' || c == '\\' || c == '"' || c == '*' ||
      c == ':' || c == '<' || c == '>' || c == '?' || c == '|') {
    return false;
  }
  return 0X1F < c;  // <<-- allow 0X20 - 0XFF
}

This may allow opening files for read.  It must not be used to create files.
I was able to open a file with this name Héllo.txt
  char name[] = {'H', 233, 'l', 'l', 'o', '.', 't', 'x', 't', 0};
character code must have these Unicode values.

Hi. Please help me. Sdfat 1.4. It does not open a quote file if the ascii
contains a character greater than 7F.
Sdfat.Open (filename, O_READ).
It is possible to edit somewhere to open a file that contains an ascii
character larger than 7F. I have already edited the quote with your help.
So loading characters with ascii larger than 7F is not a problem.
Well thank you.
Dňa po, 23. mar 2020, 15:59 Bill Greiman<EMAIL_ADDRESS>napísal(a):

You can try the following:
Edit SdFat-beta/src/FatLib/FatFileLFN.cpp
At line 92 make these changes to lfnLegalChar()
inline bool lfnLegalChar(uint8_t c) {  // <<-- replace char with uint8_t
if (c == '/' || c == '\' || c == '"' || c == '*' ||
  c == ':' || c == '<' || c == '>' || c == '?' || c == '|') {

return false;

}
return 0X1F < c;  // <<-- allow 0X20 - 0XFF
}
This may allow opening files for read. It must not be used to create files.
I was able to open a file with this name Héllo.txt
char name[] = {'H', 233, 'l', 'l', 'o', '.', 't', 'x', 't', 0};
character code must have these Unicode
https://www.w3schools.com/charsets/ref_utf_latin1_supplement.asp values.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/greiman/SdFat-beta/issues/26#issuecomment-602653229,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4SVJRJC7BLZ3IQMMBV5JTRI52OLANCNFSM4JAILGIA
.

Ascii characters are not larger than 7F.  You must use Unicode characters.
You can use Latin-1 Supplement characters see this table.


Arial font in mac os

On mac os (12.2.1) the Arial font is not loaded due to a name problem with the font file. To fix it I changed the line 102 in my local file flowkey_dl.py with :
fnt = [ImageFont.truetype("Arial.ttf", sz) for sz in font_size]

This font issues with different platforms is really annoying. I hope I found a solution that works cross platforms now, based on matplotlib font_manager. If it does not work, please open new issues.

Thx It works


Managable results

I know, that request returns Objects, with table rows and coll's.
But we could add clause to request that it'll return smart objects, which could be editable.
prisma.users.findOne(...).then((x)=>{
<EMAIL_ADDRESS>    x.save()
})

This can work like wonders, if you have to e.g. fetch url from db, and then insert fetched data once you fetch url, or some other edgy use case.
prisma.users.findOne(...).then((x)=>{
    x.articles.create({...data...})
    x.save()
})

The parameter for resulting fetches like this could be called return smart objects or so.

Is this similar to https://github.com/prisma/prisma/issues/17512, implementing an Active Record pattern?

Yes, it is simillar.

I'll close this issue then to simplify things here. If you want to add something, please comment over at #17512. Thanks! And thanks for opening this issue here originally.

Yes, it's completely active report. But find a way, that it automatically
saves debounced using proxy javascript contstruct, so it's autosaved.
On Sun, Sep 17, 2023, 12:57 Jan Piotrowski @.***> wrote:

I'll close this issue then to simplify things here. If you want to add
something, please comment over at #17512
https://github.com/prisma/prisma/issues/17512. Thanks! And thanks for
opening this issue here originally.
—
Reply to this email directly, view it on GitHub
https://github.com/prisma/prisma/issues/20541#issuecomment-1722450632,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBROII3JXHUF4ZRDZDQY63DX23JRNANCNFSM6AAAAAA3DJIKGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


[dunfell][gatesgarth][hardknott][honister] {galactic} Upgrade to 2021-08-25 release

https://discourse.ros.org/t/new-packages-for-galactic-geochelone-2021-08-25/22032/1

Merged to other branches as well:
6678c3816..462371dae  dunfell -> dunfell-next
f300e419d..5d0806cd9  gatesgarth -> gatesgarth
f300e419d..5d0806cd9  gatesgarth -> gatesgarth-next
8376b977d..3bff8ae30  hardknott -> hardknott
8376b977d..3bff8ae30  hardknott -> hardknott-next
8692879fd..d65cf4d6e  master -> master
8692879fd..d65cf4d6e  master -> master-next


Update button on Edit Screen

The "enviar" (send) button should be replaced with an "atualizar" (update) button if the data is already on server and is possible to edit it.
To do so, for now, it is need to do a mutation that will do like:
mutation updateTimeentries {
  delTimeEntry(date: '2018-01-25')
  addTimeEntry(TimeEntry)
}

but, to avoid inserting before deleting on async calls, maybe should create a new mutation editTimeEntry(TimeEntry) that would do that in one call. This should be implemented on server/schema/resolver.js

This was done recently.


Added workspace option for Wfs editing

Fixes origo-map/origo-documentation#300 by adding workspace and prefix options to wfs layer source section. Also rephrased the top level source section and added a heads up in editor section.

The info looks good however I can't seem to make the new table of the WFS source options render properly

My bad. I only checked the rendering in another md processor, which obviously was more forgiving. This version I actually built and checked!


Add notwaving to Contributors list

@notwaving, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


xandikos: add tests and module

Motivation for this change
Add tests and module for xandikos, which is included in #75532.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @matthiasbeyer

@GrahamcOfBorg test xandikos

@GrahamcOfBorg test xandikos

I've reduced the vm-count on nixos test.
But at least I'd like to cover the main two use-cases (standalone and behind proxy).
If there is no more improvements, I'd like to rebase commits.
Thanks for the reviews.


bug: there are 2 suggested supertab configurations in website

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
% nvim -v NVIM v0.10.0 Build type: Release LuaJIT 2.1.1713484068 Run "nvim -V1 -v" for more info
Operating system/version
Linux Ubuntu LTS
Describe the bug
Hi,
There are 2 code snippets for configuring supertab. One is in the ./lua/plugins/example.lua and the other is under the recipes section of the website https://www.lazyvim.org/configuration/recipes. Which one to use as they are slightly different

Steps To Reproduce
copy both code-snippets in separate files and diff them
Expected Behavior
Only one suggested code-snippet is availabel for configuring supertab
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  { "LazyVim/LazyVim", import = "lazyvim.plugins" },
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Removed the example

Thank you very much @folke . All of your work is a blessing . Gratitude :pray:


Create ID2 real identities

Document type: Passport
Nationality: Latvian
Name: Pavels
Surname: Silaraups
DOB: 03.11.1991
Account Adress: 4t4L6VyZSJ4w1HEkJFTc8DDGpc9WfhCH5f1YafYAmyNTfFWCo5
Device Model:
IPHONE: HUAWEI P20
MODEL: EML-L29
ANDROID 10
PROCESSOR: HISILICON KIRIN 970
RAM 4.0

rejected:ID2
reason:We cannot fully verify this submission. It looks like you are trying to submit ID2, however, it seems like there is just one address submitted.

rejected:ID2
reason:We cannot fully verify this submission. It looks like you are trying to submit ID2, however, it seems like there is just one address submitted.


Next Citadel update won't support the Umbrel migration script anymore

I'm currently focused on buiilding Citadel and can't implement a fix for this, but the next Citadel update will change the data layout by renaming the umbrel dir to citadel. This means the exisiting migration logic needs changes to work with Citadel.

@AaronDewes just to ask are you also using the default password moneyprintergobrrr for the lnd wallet like umbrel since 0.3.9 or did you implemented changes to that part?

Added code changes for citadel for v1.7.2 .. to be tested with fresh sd card build.

just to ask are you also using the default password moneyprintergobrrr for the lnd wallet like umbrel since 0.3.9 or did you implemented changes to that part?

I did implement some changes to the unlocking logic so lnd.conf is a bit different from Umbrel's default, but you seem to be overwriting it anyway, so it shouldn't matter.
The password stays the same though (currently). With the security changes we're discussing right now, I don't think a separate password for LND would still make sense in Citadel.

OK cool ... if you change something on that LND wallet password side give me a heads up so that I can adjust the migration procedure.

Sure! Anything else would be vendor lock-in in my opinion by breaking migration logic of other systems.

Migrationtest from Citadel to RaspiBlitz looked good in testing ... closing issue for v1.7.2 release.

what is the default user and password wen u connect to the ssh ?

what is the default user and password wen u connect to the ssh ?

Please ask this somewhere else, not here. GitHub issues are mostly for tracking bugs, not questions. Also, it's not possible to know from your question if you're talking about RaspiBlitz, Umbrel or Citadel. Please ask at the appropriate place for the project you want to know this.

Citadel

sry for posting here but i try to find all over the social media, its about Citadel because i use migrate function from umbrel and its downloding all the node data again and i need to delete the umbrel node data. sry again for posting here but i really need help to delete the data and i cant connect via ssh to do so

Please contact me at t.me/runcitadel or DM @runcitadel on Twitter (German or English).
Citadel does not have a migration system from Umbrel, you can not keep Umbrel data when moving to Citadel right now, so your data will be erased when setting up Citadel.


Pr babel next handler

Close #45

Coverage increased (+0.06%) to 95.385% when pulling d5d225fc20bca2a2c46d384cc83430167552bde7 on pr-babel-next-handler into 4532cd9254720087ef748cbccfe9d691200e7347 on master.


boards: disco_l475_iot1: fix corruption of storage partition

Fixes #26413
The devicetree for disco_l475_iot1 declares the flash partitions
"storage" and "image-scratch" with overlapping address spaces.
As a result, when one of these partitions is used it can corrupt the
contents of the other one. This is the case when mcuboot performs an
image swap using the scratch partition.
To fix the bug, the size of the scratch partition is reduced.
Signed-off-by: Hans Wilmers<EMAIL_ADDRESS>

Change commit line:

Fixes bug: #26413
to:
Fixes #26413.

Thank you @nandojve, I was not aware of this. Changed in commit message and PR description.

@hwilmers can you rebase your branch to latest master ?
This is required to get CI pass

rebase


Switchboard etc.

Hello again,
somehow everything is working fine. But as soon as I want to make use of my ndisplay config, the tracking stops working ...
The actual problem is that the tracking just doesn't work within the Switchboard mode. Has no one figured out any solution for this? It would be so great to use it in UE 5.3. Right now I'm working without the inner Frustum an UE 5.2.1 and Aximmetry ...
I'm curious how long it will take to get a version of the Unreal Editor for Aximmetry for UE 5.3 ...
If anyone has any information on this issue, please let me know.
Best wishes

Does anybody have some screenshots of their UDP Messaging, Ndisplay settings for me? I am kind of in a dark tunnel and I can't see any light ...


Update LQRamsey

Needs Test and Plots in separate PR.

Looks good to me. Added random seeds for when we get around to testing.


Update SceneSystemGettingStarted.md

Several typo fixes.
By the way, there is no " ! [ ] (../Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.png)"

/azp run mrtk_docs

Several typo fixes.
By the way, there is no " ! [ ] (../Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.png)"

file exists for me in mrtk_development

Several typo fixes.
By the way, there is no " ! [ ] (../Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.png)"

file exists in mrtk_development https://github.com/microsoft/MixedRealityToolkit-Unity/blob/mrtk_development/Documentation/Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.PNG

No picture is displayed though when I click on the page. Seems the relative reference is incorrect then. It seems that the system is picky about the lower-/uppercasing of the "PNG" part. Either change the file name to a lowercase png or change it in the text. Otherwise nothing is displayed.

Several typo fixes.
By the way, there is no " ! [ ] (../Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.png)"

file exists in mrtk_development https://github.com/microsoft/MixedRealityToolkit-Unity/blob/mrtk_development/Documentation/Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.PNG

No picture is displayed though when I click on the page. It seems that the system is picky about the lower-/uppercasing of the "PNG" part. Either change the file name to a lowercase png or change it in the text.  I didn't realize the file ending bc the docfx build didn't report it as an error either (that process should actually check any relative broken link)

Several typo fixes.
By the way, there is no " ! [ ] (../Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.png)"

file exists in mrtk_development https://github.com/microsoft/MixedRealityToolkit-Unity/blob/mrtk_development/Documentation/Images/SceneSystem/MRTK_SceneSystemProfileEditorSettings.PNG

No picture is displayed though when I click on the page. It seems that the system is picky about the lower-/uppercasing of the "PNG" part. Either change the file name to a lowercase png or change it in the text.  I didn't realize the file ending bc the docfx build didn't report it as an error either (that process should actually check any relative broken link)

Yeah, weird! I would have also thought it would throw some broken link error. I guess, we could advise to use some alternative text in the square brackets to make it more obvious for a visual check at least.


JasperSoft -> iText -> PDF encryption fails without DEREncodable

My application uses the following dependencies:
Jaspersoft Reports: 6.5.1
iText: 2.1.7
BouncyCastle: 1.63
Everything was working fine until we tried to export a PDF file with password protection. The error is
 java.lang.NoClassDefFoundError: org/bouncycastle/asn1/DEREncodable
There are a few posts online explaining the issue but no necessary fix. I would presume the actual fix is putting the interface and any other required classes back into the BC library.
My current workaround is including
shared-bouncycastle-reduced: 0.9.12
However, my maven duplicate finder does not like this since there are many duplicate dependencies.
So here I am asking what is the solution for this issue, if any? If none, can we please revert and include these classes?
Thank you!

Adding the 'shared-bouncycastle' library is now causing a cyclic issue during startup in Tomcat, so having it included is not a good workaround.
Caused by: java.lang.IllegalStateException: Unable to complete the scan for annotations for web application [] due to a StackOverflowError. Possible root causes include a too low setting for -Xss and illegal cyclic inheritance dependencies. The class hierarchy being processed was [org.bouncycastle.asn1.ASN1EncodableVector->org.bouncycastle.asn1.DEREncodableVector->org.bouncycastle.asn1.ASN1EncodableVector]

See here for the JasperReports issue relating to this.
As you can see iText 2.17 (built in 2009) is dependent on BouncyCastle 1.38 (built in 2007). The fix is for JasperReports to upgrade the version of iText that it uses. For a workaround I would suggest using BouncyCastle 1.38.
According to the BC release notes it would appear that there was a major re-write of the ASN1 code for 1.47, so you MAY be able to get away with version 1.46. Unfortunately the BC gitHub codeBase only goes back to v1.49, so I cannot verify this.

Thanks for your response @tonywasher ! I am aware of that post and the dependency hell (hehe) that I'm currently in. The reason I upgraded BC is to prepare to upgrade to Java 9.
In the meantime to fix the original issue, and the issue in my comment (illegal cyclic inheritance) I decided to create my own internal jar with the missing bouncycastle classes and included that in my project and remove the shared-bouncycastle library. Now I am able to encrypt my PDF and start up the application in Tomcat with no issues.

As of bcprov-jdk15on-1.63 bouncycastle has signed their jars. You can no longer provide a custom DEREncodable class unless you remove the bouncycastle signatures.

Can you please try with OpenPDF 1.3.27 instead of iText 2.1.7?
https://github.com/LibrePDF/OpenPDF/releases/tag/1.3.27


pip 安装报错：找不到

$ pip3 install evernote-export
ERROR: Could not find a version that satisfies the requirement evernote-export (from versions: none)
ERROR: No matching distribution found for evernote-export

$ pip install evernote-export
ERROR: Could not find a version that satisfies the requirement evernote-export (from versions: none)
ERROR: No matching distribution found for evernote-export

tested in Python2.7
not support Python3


does it use too much memory ?

cat config/word2vec_sample_config.yml
skip_window: 5
neg_samples: 1
vocabulary_size: 100000
min_count: 100

Can you provide more detail? E.g. parameters in the config file, how much memory was consumed, etc.

cat config/word2vec_sample_config.yml
skip_window: 5
neg_samples works slightly differently than uniform (3/4 power), so the approx. using
SPMI will be off, especially when it is large. Usually neg_samples <= 4 gives accurate results.
neg_samples between 1-5 is suggested (Mikolov et al)
neg_samples: 1
vocabulary_size: 100000
min_count: 100

https://github.com/ziyin-dl/word-embedding-dimensionality-selection/blob/master/utils/tokenizer.py#L62
can we pipeline the data ?  The list cost extremity memony

Vocab 100000 is a bit too much for now, it will require distributed algorithms. The bottleneck will be memory and CPU since there will be a matrix of size n by n (n is the vocabulary size).
However, from the observation we found that vocab size is not very important to the dimensionality, as long as the signal-to-noise ratio characteristics is not changed. Using 10k-20k vocab should reflect the right dimensionality.

Hi @ziyin-dl  I tried the example on text8, when I use default config, vocab==10000, the dimension I got is 123, when I changed the vocab==15000, the program use all the vocab, the vocab_size==11815, the dimension I got changes to 142. I have a question, if the dimension can change with the change of vocab_size in the same corpus, how to get a most suitable dimension for the corpus, or maybe vocab_size = corpus_size is the best choose? thanks!

Yes, the dimensionality depends on on the vocab_size, and other parameters in the config file. And this is expected; consider an extreme case where you set vocab_size=10; This means only the top 10 most frequent tokens are embedded, and all the information is essentially contained in a 10 by 10 co-occurrence matrix. The dimensionality in this case should definitely be different from using a vocab_size=10000.
In general, vocab_size, min_count, window_size, etc.. are other hyper-parameters in the embedding algorithm. There is no standard way of choosing them; for choosing the vocabulary, a usual way is to set a cut-off threshold (a.k.a min_count). If the frequency of a token is less than min_count, it will be mapped to a special token '<unk>', indicating its appearance is not enough for a stable estimation (say a token appear only once in the corpus, how can we learn a meaningful embedding for it?). I would suggest using min_count=50 or 100. On text8 corpus, min_count=100 roughly lead to a vocabulary size of 10000.

ok, in my understanding, the best way should be min_count was set, then you can get the number of words which appearing more than min_count, then the number should be set as vocab_size? If so, why don't set the vocab_size automatically according to min_count? When training embedding use gensim, the min_count and window_size should be set as the same value in config file.

I believe in gensim only min_count is needed. In the program, setting vocab_size=corpus_size will make sure the vocabulary is constructed using min_count only.
The purpose of adding a vocab_size parameter is to give people another way to control the vocabulary; it happens quite often (in both research and industrial applications) that people want a vocabulary with a nice-looking number (like 10000, 20000, etc.)

Right, what the code does is

Construct a vocabulary V1 using min_count
Construct a vocabulary V2 using vocab_size
return whichever is smaller: V1 < V2 ? V1 : V2.

Ok, thanks! 😃

I use vocabulary_size: 100000 and my computer has 16G . Actually, it has 500000 vocabularys in the file.
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/workspace/guoxiang/lls/src/github.com/word-embedding-dimensionality-selection/main.py", line 31, in
min_count=cfg.get('min_count'))
File "utils/tokenizer.py", line 62, in do_index_data
self.tokenized = self.tokenize(data)
File "utils/tokenizer.py", line 23, in tokenize
tokenized = pool.map(_lower, splitted)
File "/usr/lib/python2.7/multiprocessing/pool.py", line 251, in map
return self.map_async(func, iterable, chunksize).get()
File "/usr/lib/python2.7/multiprocessing/pool.py", line 567, in get
raise self._value
MemoryError
dear ahhygx:
I meet the same problem as you.I used the text8 data to run the program, but there were still errors
vocabulary_size=10000
Traceback (most recent call last):
File "/home/li/anaconda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/li/anaconda3/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/li/program/word-embedding-dimensionality-selection-master/main.py", line 37, in
signal_matrix.estimate_signal()
File "/home/li/program/word-embedding-dimensionality-selection-master/matrix/signal_matrix.py", line 39, in estimate_signal
matrix = self.construct_matrix(self.corpus)
File "/home/li/program/word-embedding-dimensionality-selection-master/matrix/word2vec_matrix.py", line 60, in construct_matrix
PMI = np.log(Pij) - np.log(np.outer(Pi, Pi)) - np.log(k)
Memory error
I want to know why ？In fact, the data of text8 is not very large. Why are there memory errors?
Have you solved the problem yet?Looking forward to hearing from you


Grafana helm chart pod failing

Is this a BUG REPORT or FEATURE REQUEST?: Bug Report
Version of Helm and Kubernetes:
Helm version: 2.8.2
Kuernetes Version: 1.9.4
Which chart: stable/grafana
What happened: grafana pod fails to start with the following error.This was previously working, and discussion in the rook.io slack with @galexrt turned up that this may be related to a recent security patch in kubernetes.
chown: changing ownership of '/var/lib/grafana/dashboards/..data': Read-only file system
chown: changing ownership of '/var/lib/grafana/dashboards': Read-only file system

How to reproduce: Install the helm chart with default configurations on a node running 1.9.4

I've submitted a PR with the same strategy I used with the rabbitmq-ha chart.
It still feels a little clunky to do this, but unless containers stop writing to their configdirs or Kubernetes allows writing to ConfigMaps again (or adds some primitives to do what I did in this PR automatically), I don't see another way.

For a temporary workaround, you can add the following command for the docker container in the deployment.yaml (taken from the init script)
command: ["/bin/bash", "-c", "chmod 777 /var/lib/grafana/ && gosu grafana /usr/sbin/grafana-server --homepath=/usr/share/grafana --config=/etc/grafana/grafana.ini cfg:default.paths.provisioning=/etc/grafana/provisioning cfg:default.log.mode=console cfg:default.paths.data=/var/lib/grafana/ cfg:default.paths.logs=/var/lib/grafana cfg:default.paths.plugins=/var/lib/grafana/plugins"]

Related: https://github.com/grafana/grafana-docker/issues/141

alternative workaround:
uncomment and modify the dashboardLocalPath value in the server object:
  dashboardLocalPath: /var/lib/grafana-dashboards
modify in grafana.ini the path value:
[dashboards.json]     enabled = true     path = /var/lib/grafana-dashboards

but looks like the fix is on its way:
https://github.com/grafana/grafana-docker/blob/master/run.sh#L9

alternative workaround:
in server object:
`extraEnv:

name: GF_PATHS_CONFIG
value: /etc/grafana-cm/grafana.ini

Grafana local config path
Default '/etc/grafana'

configLocalPath: /etc/grafana-cm
Grafana local dashboards path
Default: '/var/lib/grafana/dashboards'

dashboardLocalPath: /var/lib/grafana-dashboards
and modify grafana.ini cm:  [dashboards.json]
enabled = true
path = /var/lib/grafana-dashboards`

Thanks @csabakollar and all for the tips. I've been banging my head with this for a few hours.  I was able to just switch my image to grafana/grafana:master for now to solve the issue.


Reporting a vulnerability

Hello!
I hope you are doing well!
We are a security research team. Our tool automatically detected a vulnerability in this repository. We want to disclose it responsibly.  GitHub has a feature called Private vulnerability reporting, which enables security research to privately disclose a vulnerability. Unfortunately, it is not enabled for this repository.
Can you enable it, so that we can report it?
Thanks in advance!
PS: you can read about how to enable private vulnerability reporting here: https://docs.github.com/en/code-security/security-advisories/repository-security-advisories/configuring-private-vulnerability-reporting-for-a-repository

Hi @igibek the private vulnerability reporting is now enabled, feel free to contact us there. :slightly_smiling_face:


JBA update

adding brain atlas version for v2.9
updating version identifiers for all PEVs

@UlrikeS91 can you check the files I've committed already now? my next commits will increase the changed files to many hundreds (where "just" the version id is changed...)

Looks pretty good. Couldn't find anything wrong with the two new brainAtlasVersions, but the brainAtlas that has turned into a brainAtlasVersion has a few bugs:

change name from "Julich-BrainAtlas.jsonld" to "JBA_v2.9_MNI-ICBM-152-2009c-nonlin-asym.jsonld" (or whatever it should be called)
change location of the jsonld from the brainAtlas-folder to the brainAtlasVersio-folder

General question: If the brainAtlas "Julich-BrainAtlas" disappears now, what brainAtlase(s) will the 3 versions be linked to? New ones that will be added later on?


Bug: Nested Page Names in Title

I think I found a bug.
[[[[Page1]] [[Page2]] [[Page3]] Page4]]
It seems that when there are many pages linked in a single page title, not all of them get loaded in context.
More specifically, the main page linked (in this case, Page4), doesn't ever seem to get loaded.
In my tests, it seemed that Page1 sometimes  got loaded, but Page4 never got loaded.
https://user-images.githubusercontent.com/3792666/219494716-851565b0-7034-4fc8-9732-6af1fa77bc7f.mp4
Awesome plugin btw 🌟

ahh good catch! I'll find time to fix this