[FLINK-15131][connector/source] Add the APIs for Source (FLIP-27).

What is the purpose of the change
Add the top tier interfaces for FLIP-27. These classes are prerequisite for the rest of the PRs of FLIP-27. See the FLIP wiki for more details.
Brief change log
The classes introduced include:

Boundedness - A enum indicating whether the input stream is Bounded or Unbounded
Source - A factory style class that helps create SplitEnumerator and SourceReader at runtime.
SourceSplit - An interface for all the split types.
SplitEnumerator - Discover the splits and assign them to the SourceReaders
SplitEnumeratorContext - Provide necessary information to the SplitEnumerator to assign splits and send custom events to the the SourceReaders.
SplitAssignment - A container class holding the source split assignment for each subtask.
SourceReader - Read the records from the splits assigned by the SplitEnumerator.
ReaderInfo - A container class about reader information.
SourceReaderContext - Provide necessary function to the SourceReader to communicate with SplitEnumerator.
SourceOutput - A collector style interface to take the records and timestamps emit by the SourceReader.

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change only introduces the interfaces, so there is no tests needed.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (JavaDocs, FLIP)

@wuchong @StephanEwen would you have time to take a look? Thanks.

@StephanEwen Do you have further comments? I'll merge the PR later today if there is no more concerns.

Thanks @becketqin . Looks good to me.

+1 to merge this

Thanks for the review. @StephanEwen @wuchong
merged to master.
5a81fa0766abe3ef6c0e6b8f9217e9adcd18095f


Why does ObjectReference.StackTrace not contain the stack trace?

Every object has this string for the stack trace:
   at SharpDX.Diagnostics.ObjectTracker.GetStackTrace()

Configuration.EnableObjectTracking = true; is being set. Is there another stack trace setting I missed, or documentation for anything that can affect the stack trace? Or is it a bug?

This is a fkg bug of PCL :(
If you are on desktop, you should be able get get a reliable stack trace by overriding it on the ObjectTracker:
ObjectTracker.StackTraceProvider  =  () => Environment.StackTrace;

On other platforms, afaik, there are no workarounds.


Revert "Upgrading PIP to automatically resolve dependency issues."

Reverts Flux7Labs/central-repo-git-hooks#25

cfn-lint and checkov are conflicting with the common pkgs and one or the other failing. will work on finding other ways to fix the issue. Reverting back the changes for now.


Version REST API

Create a REST API endpoint for asking Scale for its version.
Version can be specified in the main settings.py.

We decided to implement our version API based on the Apache Mesos design:
http://mesos.apache.org/documentation/latest/endpoints/version/
Only the version field will exist initially, but we hope to expand it with more in the future.

Merged into master


[feature] Token holders

Hey, @adrienmo and @lucianoengel,
I think that we already talked about this some time ago, but I do not remember if it was already included in the future directions of neoscan.
Recently, I have been giving a seminar talking about "Transparency as a contribution to the end of inequalities".
The point that border me is that sometimes I am often showing etherscan.io instead of our blockexplorers.
The main reason is because I want to show to the audience the tokens percentage of holders and etc...
I think that we should move towards these kind of information as soon as possible.
What do you think?

@vncoelho I agree with you, could you detail the metrics you would like to have?

@adrienmo, I think that search for the list of most healthy address is enough (for NEO,GAS and NEPs5), because most of the other features of etherscan.io you already have.


Handle skipping toolchain verification internally

Specifying HUNTER_SKIP_TOOLCHAIN_VERIFICATION should be something Hunter does internally, when necessary.  Client projects shouldn't have to do this themselves.

Does make sense, thanks.


White border when launched in 1920x1080 Windows

Whenever I launch my WIndows version of EmulationStation in 1920x1080 there is a white border on the top and right side if the user interface. When I change my computers resolution to a lower setting it works fine, but I'd prefer to run it at 1920x1080. Any ideas what's up?

I was with the same problem. I just changed the shortcut and added the parameter: -resolution 1920 1080 / or -resolution 1280 720 and the emulationstation ran on the fullscreen without border.
In my case, I had to use the 1280 option, even with my native resolution in 1920x1080, I think that is because my dpi/font setting is changed to 175% and not 100%.

Thank you! So I made the shortcut and entered -resolution 1280x720 as the parameters in the target area, but nothing has changed. Am I doing something wrong?

notice the X in the resolution argument that you gave. "-resolution 1280x720" try with "-resolution 1280 720"
-resolution 1280x720 => Incorrect ( the x between numbers makes it invalid )
-resolution 1280 720 => Correct

Oh sorry, yeah I tried without the x at first and it didn't work. That's why I threw the x in.

Hm.. strange..
Can you please print screen the shortcut properties so I can take a look on the parameter?

Did you tried the -resolution 1280 720 without quotes?

Yes.

Can you post a printscreen of the emulation station starting with the borders?

I have followed the instructions located at https://github.com/Aloshi/EmulationStation (In the Configuring section)
I have just looked and its --resolution and not -resolution. Try again. I could not check this before, because I am currently away from my pc configured with this option

Thanks!! I'll try that. I couldn't printscreen because it was in fullscreen, so I took a video and took a screenshot of the video. Here's the result

I'm gonna try those new parameters now.

--resolution worked! Thanks so much man!

:D
Thats awesome!
Have fun!
I was with the same problem yesterday. Glad it works!

I had the same problem with 1080p res. Fixed it by going to properties and disabling DPI scaling.


Fix layers being reordered when working with layers and folders

Grouping (with CtrlG):

[ ] External consistency: Grouping the selected layers causes the group to be inserted at the top of the layer stack. It should go just above where the layer selected layers were located.
The current incorrect behavior:

-> CtrlG ->

[ ] Internal consistency: Grouping layers should also preserve the order that they were sorted once those layers end up in the new group
The current incorrect behavior:

-> CtrlG ->

Ungrouping (with CtrlShiftG):

[ ] External consistency: Ungrouping should also preserve order by placing the content in the hierarchy where the group used to be.
The current incorrect behavior:

-> CtrlShiftG ->

[ ] Internal consistency: While ungrouping, the content in the group itself should also be have its order preserved amongst its grouped siblings.
The current incorrect behavior:

-> CtrlShiftG ->

Creating a new folder  (with CtrlShiftN while in fullscreen mode of the web app, or the folder button in the top bar of the Layers panel):

[ ] Should be placed directly above the topmost selected layer in the hierarchy (at the level of the least nested layer)
The current incorrect behavior:

-> this button is clicked ->

@moOsama76 Please reply "Ok" so I can assign you this issue.

Okay!
On Wed, Feb 28, 2024, 9:19 AM Keavon Chambers @.***>
wrote:

@moOsama76 https://github.com/moOsama76 Please reply "Ok" so I can
assign you this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/GraphiteEditor/Graphite/issues/1633#issuecomment-1968373369,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQEFUOPYY6NXDSFHNYKUNNTYV3LAHAVCNFSM6AAAAABD5OCBROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYGM3TGMZWHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


go get  not working

go get github.com/google/go-github/tree/v32.1.0/github

or

go get github.com/google/go-github/v32/github

package github.com/google/go-github/tree/v32.1.0/github: cannot find package "github.com/google/go-github/tree/v32.1.0/github" in any of:
/usr/local/go/src/github.com/google/go-github/tree/v32.1.0/github (from $GOROOT)
/Users//go/src/github.com/google/go-github/tree/v32.1.0/github (from $GOPATH)

Please try:
go get -u github.com/google/go-github/...

@gmlewis Can we close old and inative issues like this and/or #1467 ?
Is there some minimum time for which an issue needs to remain open, post which only it can be marked as "inactive" and then closed ?

Yes, @chughpiyush - this can be closed.  I try to leave them open for a short time to give the people a chance to respond with more questions, comments, or clarifications, but I have not been diligent about cleaning up.  Periodically I run through all the issues and close them all out, but feel free to comment on old/obsolete issues when you find them, and I'll close them out... like you did here.
Thank you!
Closing issue as obsolete.

I know that this is closed, so my apologies if it's not appropriate to comment on this thread, but I thought I'd share a bit of context as to what is likely going on here since I found it interesting (I ran into a similar issue), for any other future wayfarers.
This looks as though it is occurring due to the use of gopher flakes at the root of the package because of the usage of the v32 version of this package inside the nested scrape module, which is using go modules. This only happens when you yourself are not using go modules in the directory from which you call go get from. This is because, when not using go modules in the calling directory, go get recurses into the scrape subpackage, determines that it needs to pull down the v32 version, but does not understand this, and so reports this issue at that point.
As such, if you're not using go modules or calling go get not from a directory using them, and so are pulling this down to your $GOPATH, you'll need to specify the /github subpackage, a la go get -u github.com/google/go-github/github/... which will then work.

I know that this is closed, so my apologies if it's not appropriate to comment on this thread, but I thought I'd share a bit of context as to what is likely going on here since I found it interesting (I ran into a similar issue), for any other future wayfarers.
This looks as though it is occurring due to the use of gopher flakes at the root of the package because of the usage of the v32 version of this package inside the nested scrape module, which is using go modules. This only happens when you yourself are not using go modules in the directory from which you call go get from. This is because, when not using go modules in the calling directory, go get recurses into the scrape subpackage, determines that it needs to pull down the v32 version, but does not understand this, and so reports this issue at that point.
As such, if you're not using go modules or calling go get not from a directory using them, and so are pulling this down to your $GOPATH, you'll need to specify the /github subpackage, a la go get -u github.com/google/go-github/github/... which will then work.

Very interesting! Thank you, @ttacon !
@willnorris and @dmitshur might be interested in this.

Very interesting! Thank you, @ttacon !
@willnorris and @dmitshur might be interested in this.

I'm discovering that other people are running across this same issue, and it looks like more investigation is needed to fix the issue involving the "scrape" subpackage, so I'm going to open this issue again.
Any volunteers are welcome to work on this.  Ideally, go get -u github.com/google/go-github/... would work again without having to jump through any hoops.

I'm discovering that other people are running across this same issue, and it looks like more investigation is needed to fix the issue involving the "scrape" subpackage, so I'm going to open this issue again.
Any volunteers are welcome to work on this.  Ideally, go get -u github.com/google/go-github/... would work again without having to jump through any hoops.

@gmlewis To make progress on this issue, it would help to provide more information about what issue people are running into. The original issue report was trying to go get an import path that does not exist in GOPATH mode, so an error is the correct behavior.
In module mode, go get -u github.com/google/go-github/... works without errors using Go 1.15.7:
$ cd $(mktemp -d)
$ go mod init m.test
$ go get -u github.com/google/go-github/...
go: found github.com/google/go-github/... in github.com/google/go-github v17.0.0+incompatible
go: finding module for package google.golang.org/appengine/log
go: finding module for package golang.org/x/crypto/ssh/terminal
[...]
$ echo $?
0
$ cat go.mod
module m.test

go 1.15

require (
	github.com/golang/protobuf v1.4.3 // indirect
	github.com/google/go-github v17.0.0+incompatible // indirect
	github.com/google/go-querystring v1.0.0 // indirect
	golang.org/x/crypto v0.0.0-20201221181555-eec23a3978ad // indirect
	golang.org/x/net v0.0.0-20210119194325-5f4716e94777 // indirect
	golang.org/x/oauth2 v0.0.0-20210126194326-f9ce19ea3013 // indirect
	golang.org/x/sys v0.0.0-20210124154548-22da62e12c0c // indirect
	golang.org/x/term v0.0.0-20201210144234-2321bbc49cbf // indirect
	google.golang.org/appengine v1.6.7 // indirect
)

However, go get -u github.com/google/go-github/... is not a command that's very useful to run, as module mode users should use v33 and not v17.

@gmlewis To make progress on this issue, it would help to provide more information about what issue people are running into. The original issue report was trying to go get an import path that does not exist in GOPATH mode, so an error is the correct behavior.
In module mode, go get -u github.com/google/go-github/... works without errors using Go 1.15.7:
$ cd $(mktemp -d)
$ go mod init m.test
$ go get -u github.com/google/go-github/...
go: found github.com/google/go-github/... in github.com/google/go-github v17.0.0+incompatible
go: finding module for package google.golang.org/appengine/log
go: finding module for package golang.org/x/crypto/ssh/terminal
[...]
$ echo $?
0
$ cat go.mod
module m.test

go 1.15

require (
	github.com/golang/protobuf v1.4.3 // indirect
	github.com/google/go-github v17.0.0+incompatible // indirect
	github.com/google/go-querystring v1.0.0 // indirect
	golang.org/x/crypto v0.0.0-20201221181555-eec23a3978ad // indirect
	golang.org/x/net v0.0.0-20210119194325-5f4716e94777 // indirect
	golang.org/x/oauth2 v0.0.0-20210126194326-f9ce19ea3013 // indirect
	golang.org/x/sys v0.0.0-20210124154548-22da62e12c0c // indirect
	golang.org/x/term v0.0.0-20201210144234-2321bbc49cbf // indirect
	google.golang.org/appengine v1.6.7 // indirect
)

However, go get -u github.com/google/go-github/... is not a command that's very useful to run, as module mode users should use v33 and not v17.

Thank you, @dmitshur - I was working with someone who attempted to perform
go get -u github.com/google/go-github/v32

and the command failed.  Having to add "/github@latest" to the end seems like a pain, does it not (or at the very least, it is non-intuitive).
Hmmm... I see that we don't have "Installation" instructions in the README.md... maybe this is all we need to address this issue?

Thank you, @dmitshur - I was working with someone who attempted to perform
go get -u github.com/google/go-github/v32

and the command failed.  Having to add "/github@latest" to the end seems like a pain, does it not (or at the very least, it is non-intuitive).
Hmmm... I see that we don't have "Installation" instructions in the README.md... maybe this is all we need to address this issue?

That command works for me too:
$ cd $(mktemp -d)
$ go mod init m.test
$ go get -u github.com/google/go-github/v32
go: downloading github.com/google/go-github/v32 v32.1.0
go: github.com/google/go-github/v32 upgrade => v32.1.0
$ echo $?
0
$ cat go.mod
module m.test

go 1.15

require github.com/google/go-github/v32 v32.1.0 // indirect

The "@latest" suffix can be omitted, I included it to make the version explicit.
Adding an "Installation" section to the README sounds good idea. I'm happy to review a PR for it. (An example of such a section can be found in the READMEs of blackfriday v1 and blackfriday v2.)

That command works for me too:
$ cd $(mktemp -d)
$ go mod init m.test
$ go get -u github.com/google/go-github/v32
go: downloading github.com/google/go-github/v32 v32.1.0
go: github.com/google/go-github/v32 upgrade => v32.1.0
$ echo $?
0
$ cat go.mod
module m.test

go 1.15

require github.com/google/go-github/v32 v32.1.0 // indirect

The "@latest" suffix can be omitted, I included it to make the version explicit.
Adding an "Installation" section to the README sounds good idea. I'm happy to review a PR for it. (An example of such a section can be found in the READMEs of blackfriday v1 and blackfriday v2.)

OK, great... thanks for testing this out, @dmitshur  - I'm glad it worked... I'm not sure what was going wrong before.
Perfect.  I'll open this issue up as a "good first issue" "enhancement" and open it up to volunteers.
For anyone who would like to take on this issue, here is what we need:

Adding an "Installation" section to the README sounds like a good idea. I'm happy to review a PR for it. (An example of such a section can be found in the READMEs of blackfriday v1 and blackfriday v2.)

OK, great... thanks for testing this out, @dmitshur  - I'm glad it worked... I'm not sure what was going wrong before.
Perfect.  I'll open this issue up as a "good first issue" "enhancement" and open it up to volunteers.
For anyone who would like to take on this issue, here is what we need:

Adding an "Installation" section to the README sounds like a good idea. I'm happy to review a PR for it. (An example of such a section can be found in the READMEs of blackfriday v1 and blackfriday v2.)

I came across the same issue today :sweat_smile:
I would like to work on this and send a PR so that it could help others who are starting out to use the library!
Can you please assign this issue to me?

Thank you, @DarthBenro008 - it's yours.


Please invite me to the Recode-Hive GitHub Community Organization

Name
Deepesh Gupta
Discord Username (if applicable)
deepeshmlgupta
Additional Context
Excited To join this open Source Community

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


Print

Hi
How i can use this plugin to print a ticket?

Hey,
You should be able to connect and write to and read from any generic bluetooth devices (such as printers). Check out the example for how to establish connections and manage them.
You might also want to take a look at #13 since it contains a reasonable example on how to send data to devices.

Excuse
The example did not me help
I could connect... but can not do anything else

Hey,
Check out #13 for an example of how to write data to the connected device.

Sorry.. but dont worked :(
I am using Intel XDK, That would be the problem ?

I can't really say. I've no experience with it.

Hello. Sorry it took me to answer.
The print worked with the code you provided me!!!

Click to Print

But I have to click three or four times to print...
I think my mistake is with Javascript :(

Hi,
To make sure the onclick handler is actually firing, you can do something like this instead:

Click to Print

Also take note that since the operation is asynchronous it will take some time to complete, so you shouldn't necessarily see the results instantly. Although usually they are pretty instant I guess...

Hi tanelih
The log not show anything unusual..
see the operation in video: https://www.youtube.com/watch?v=pFLX4xi-Z6s
And see the Log:
>bluetooth enabled
 state changed 3

>starting discovery...
 state changed 2

>found device
 Object

    address: "00:08:1B:95:3A:CA"
    name: "P25_027069_01"
    __proto__: —

>Connect

>I have been clicked!
 Wrote data succesfully.

>I have been clicked!
 Wrote data succesfully.

>I have been clicked!
 Wrote data succesfully.

>I have been clicked!
 Wrote data succesfully.

>I have been clicked!
 Wrote data succesfully.

>I have been clicked!
 Wrote data succesfully.

Thank's for your patience

Hi,
Cool video, it really helps to see the system in action. It looks like the message is printed as many times as you've clicked the write button. So in a sense the plugin seems to work just fine?
Have you tried writing a longer message, which would force the printer to write multiple lines with a single write command. There might be some sort of a minimum for the message length before it starts printing each message.
Is there a specific "code" to tell the printer to print (something like EOF)? You might need to write in something like that in order for the printer to recognize that it needs to stop waiting for input and print.

What happens if you click the write once and then press the button in printer to spit out the paper. Has it written the message still?

tanelih

If clicked only one time, and press the button in printer to spit out the paper comes out blank

Have you tried writing a longer message, which would force the printer to write multiple lines with a single write command. There might be some sort of a minimum for the message length before it starts printing each message.

tanelih

I do not think of it before. I realized a new test with a long message and it worked !!! :blush: :smile: :relaxed:
Not print few characters that are at the end of the long message.
In the first test were the last 37 characters, And In the second test with bigger message 46 the last characters.
Apparently he has a minimum length for the start and the end of the information sent.

Is there a specific "code" to tell the printer to print (something like EOF)? You might need to write in something like that in order for the printer to recognize that it needs to stop waiting for input and print.

tanelih

I will search these codes. because I want to print images too.
Thanks for your help , I try finish my project.. And if successful, i'm back to feedback

Good to hear you got it working.
In the original use case for which the plugin was developed for, we had to write ACK messages to the device to let it know we had the data and that it could send more. Maybe the printer expects similar messages?

Cool ... I'm enjoying working with it
There's a repository? where I can see those messages that you have written ?

Unfortunately I don't know where the repository is anymore. So I can't really help with that.

No problem!
Helped me a lot!
I'll keep trying
Thanks :wink:

Tanelih
Could you help me to send text messages in bold to printer?


⚠ Samsung has degraded performance

In 5941d49, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2323 ms

Resolved: Samsung performance has improved in 3ce2e30.


Make living entities die of old age.

Except for the player of course.

This could be accomplished by decreasing the entity's target energy every x ticks.


Add HotSpots To DenyList [Addition]:

Hotspot b58 Addresses
112AT5baYcYG6yHchYa9xnkqNJ4cXbgxCh8i8nvnfZeewAHJ8zKc
Clean Wooden Zebra
1125yrHWwKVDyn8bxQZn9GnHTwyUdu2exPcngeM3nLPKPPKwmkD1
Daring Cedar Nightingale
11ryHkbe2pYEg69C2i5xrQEM1pVyNn7ByF1SgwFXqke8aiJZwEq
Short Infrared Sloth
112bQKSN3TiaYMrsjNKGZotd14QPi7DB37FeV88rmVMgP4MgTK9
Amateur Tan Monkey

q
X_CHIC0_X
No response
Reason(s)

spoke to people in the area also they dont have hotspots or even know what the are
ghost hotspots no power no structures open desert

all these are from Helium OG

As an aside, this city has the coolest name ever:

I have not checked these HS, but worth noting that the asserted location in Hotspotty may not be the exact location, and it is generally considered owners are allowed to assert HS to within a few 100m of real placement, for privacy reasons. I.e. while Hotspotty shows the HS in desert, it MAY be in a building nearby, or it might be spoofed.
All 9 HS in the city are owned by the same account. It might be interesting to check other HS owned by this account?
If you are local could also mapper it? No-one seems to have gone through yet:

If they were pantherX, @KXI1  will spewing so many bad words to them, but , Oh, they are Helium Inc, they are real, read my lips, they are real~

i found another person in the area the has 2 hotspots but none of these 9 are his apparently he see it the same way the person that has these has some in three other city's we have tried to reach out to the on app.hotspotty have not received anything.


Add support for SPI Parameters

Parameters.in(…) should be accepted in Statement.bind(…) and the type should be propagated to the codecs to encode a value with a specific database type.
See also r2dbc/r2dbc-spi#160.

Currently does r2dbc supports calling Stored Procedure in mysql which accepts IN/OUT parameters ?If not is there any workaround?


Webdocumentation

Updating the documentation for building/deploying the website.

@gwsii is the https->http change okay?

LGTM!

@sainap I believe the change from https --> http should be fine.

And thanks for fixing this! Looks good.


Breaking change to Logging: Add grpc_ctxtags

To support open tracing and other bits of middleware, it is useful to have the same semantics for gathering tags about a request (request fields, peer info etc.) This is usable for other places, such as Tracing information (Google tracing, OpenTracing).
That's why this PR extracts the request extraction logic from individual interceptors for logging and moves it to grpc_ctxtags.
Breaking changes:

grpc_logging no longer has Metadata in it
all request extraction is in grpc_ctxtags, which is the "mutable" (for upwards/downwards propagation)
all loggers are now immutable in the context

@adam-26 since you've been interested in grpc logging middleware, and this may affect you
@yifanzz for PR
Changes pending:

update examples
fixup docs

Codecov Report

Merging #23 into master will decrease coverage by 5.45%.
The diff coverage is 32.58%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
- Coverage    71.4%   65.95%   -5.46%
==========================================
  Files          20       22       +2
  Lines         661      655       -6
==========================================
- Hits          472      432      -40
- Misses        164      202      +38
+ Partials       25       21       -4

Impacted Files
Coverage Δ

logging/logrus/options.go
38% <ø> (+2.15%)
:arrow_up:

logging/zap/options.go
38% <ø> (+2.15%)
:arrow_up:

tags/context.go
0% <0%> (ø)

tags/interceptors.go
0% <0%> (ø)

tags/options.go
0% <0%> (ø)

logging/logrus/context.go
71.42% <100%> (+4.76%)
:arrow_up:

logging/zap/context.go
81.81% <100%> (+15.15%)
:arrow_up:

logging/logrus/interceptors.go
87.09% <100%> (-1.95%)
:arrow_down:

logging/zap/interceptors.go
100% <100%> (ø)
:arrow_up:

tags/fieldextractor.go
82.14% <100%> (ø)

... and 3 more

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 6b9617e...238cc4f. Read the comment docs.

I wonder if we have a a method like grpc_tags.ContextWithNewTags(context, newTagName, newTagValue) and simply don't allow users to call t.Set(...). This should avoid people accidentally forgetting to set the tag back into the context and makes that operation more seemless.


Pagetree revamp

Pagetree styles and dragndrop
also no need to show error message about moving parent into child, it doesn't really trigger any errors, just being annoying
Screenshots:

LGTM


strange work "text_encoder"

-1- Yesterday I started training the model for 3000 steps with intermediate models saved.
As a result, I got an intermediate model for 2500 steps and a final model for 3000.
Today I started training the model again with exactly the same parameters, but the final model was 2500 steps long.
Now I am comparing "intermediate at 2500" and "final at 2500" and I understand that for some reason they give different results.
I think it's about "text_encoder", which only works for the final model, without affecting the intermediate ones.
-2- I also noticed such a bug: in the "Training" block with "resolution:" = 704, I get an error about the lack of memory in VRAM, but if I disable check the "fp16" checkbox, then there will be no error and the training will start.
-3- I would also recommend that you automatically generate a .txt file in the "Session" folder with all the parameters that were used to generate the model.
-4- although I disabled the "fp16" checkbox, but for some reason the output model is just fp16 in size

@TheLastBen

the text encoder (once frozen) should be included in the intermediary checkpoints.
for the VRAM issue, "fp32" takes a bit more VRAM, if you want to use higher resolution pictures, keep it fp16 and check the box "reduce vram usage"
I'll see what I can do about a settings txt file

the text encoder (once frozen) should be included in the intermediary checkpoints.
for the VRAM issue, "fp32" takes a bit more VRAM, if you want to use higher resolution pictures, keep it fp16 and check the box "reduce vram usage"
I'll see what I can do about a settings txt file
"Reduce_memory_usage: on" and fp16 did not change the situation in any way - the same error.

There's also a new error when saving intermediate steps is enabled - Keyword arguments {'requires_safety_checker': False} not recognized.

the V2 text_encoder is almost 3 times the size of the V1.5, so it takes more VRAM
for the warning, just ignore it, it doesn't break anything.

the V2 text_encoder is almost 3 times the size of the V1.5, so it takes more VRAM for the warning, just ignore it, it doesn't break anything.
Well, why then are intermediate models not generated?

it writes : "killed", that means not enough RAM

it writes : "killed", that means not enough RAM, using fp16 ?

I am using fp32.

check the box fp16 or if you're using pro, set the vm shape to high-ram

check the box fp16 or if you're using pro, set the vm shape to high-ram

error

What resolution are you using ?

What resolution are you using ?

Same - 704

the V2 has a bigger text_encoder, so it eats more VRAM

At Resolution = 576, intermediate models are killed, and at 512 there is an error in general

it's fixed now for training on 512 images, update the colab

it's fixed now for training on 512 images, update the colab

Now both V2 models work at Resolution=512, but intermediate models are still "Killed". Sadly.
I also noticed a small bug - if you forcibly complete the download of instance images, the process will not stop, but will restart and continue downloading.

That's how colab operates when it comes to a loop, only restarting the runtime will stop the process.

That's how colab operates when it comes to a loop, only restarting the runtime will stop the process.

Question about learning in multiple passes.
I have 250 images, if it takes 200 steps to train one image, then what is the best way to train:
-1- take 25 images, train for 5000 steps, and then use the trained model as a base for the next 25?
-2- take all 250 images at once, train for 5000 steps, and use the resulting model as a base 9 more times?
Due to a limitation in google colab compute blocks, I can only train for 5000 steps.

train on all 250 images, set the steps to 4000 for example, then later you can resume the session and continue training.

yes, share the session's folder in your gdrive, then paste the link in the session cell, and everything will load automatically, make sure you check the box "resume training" in the cell below.

yes, share the session's folder in your gdrive, then paste the link in the session cell, and everything will load automatically, make sure you check the box "resume training" in the cell below to continue training

why sometimes the message "Freezing the text_encoder" pops up?

if you set the text encoder to 2%, it will freeze at 2%

if you set the text encoder to 2%, it will freeze at 2%

That is, the minimum allowable value for a text encoder is greater than 2%?
I tried to use a session from another account, but during training I get an error:

But there are images and a zip archive in the session folder itself.

There should be a folder name "instance_images" that contains the input images, the zip file is there only in case you load the session from a different account.

why did you use that path? you use that only if you're loading a session from a different drive, and you use a shared link, not a path

I use this path because I created a folder shortcut

you can't use a path in the session link

you can't use a path in the session link

After training the model, the "tokenizer" folder appeared. Is that how it should be?

I also saved intermediate models every 500 steps, and at step 3999 a model is generated with the name "ByNispArt_step_4000.ckpt", and after a minute (as I understand it, the final one) ByNispArt.ckpt. If there are no differences between them, then I think it makes no sense to generate the same thing twice.

yes, share the session's folder in your gdrive, then paste the link in the session cell, and everything will load automatically, make sure you check the box "resume training" in the cell below to continue training

I tried to continue training, but for some reason the program does not find the model


Use a larger runner for macOS test jobs

From 3 to 6 (+8 GPU) cores, 7 to 14 GB ram.
Related:

https://github.com/astral-sh/uv/pull/5873

From 6m 8s to 4m 16s, as with #5873, entirely in the cargo test step. We should check the cost, but it seems worth it regardless.


Align dependabot config

v2.0.x branch dependabot allows for minor bump, which caused this PR https://github.com/spring-cloud/spring-cloud-app-broker/pull/779.
This PR aligns the configuration already present in main branch to v2.0.x. I have doubts whether having the config in only the main branch is enough, there are recent reports contradicting it https://github.com/spring-cloud/spring-cloud-app-broker/pull/779

Shouldn't we just remove this file from all branches other than main?

I am happy to do it, we'll see if everything works fine 🚀


server: allow calling Close() without Serve()

Useful for CI tests in other projects where a BeforeEach()
might create the server for multiple tests, but one or more
tests doesn't actually use it and thus doesn't call Serve().

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 73.084%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
4225

Relevant Lines:
5781

💛  - Coveralls

/lgtm

perhaps add a comment?

perhaps add a comment?

@jcaamano done

/lgtm


createChatInviteLink   Count the number of invitees

The count of the invites is doing in the telegram back end side and is not return in the Telegram Bot API.
Example case:

1 User enter with this link limit to 2 user (1/2)
Other user enter with this link (now 0/2 users can enter)
One of this users that joins with your limit invite link exit the group (Now the link allow to 1 user to enter, because the count change from 0/2 to 1/2)

The best is create a link also with expiration time or revoke it when the user is join

The link generated by createChatInviteLink ，not chat invite_link

The count of the invites is doing in the telegram back end side and is not return in the Telegram Bot API.
Example case:
* 1 User enter with this link limit to 2 user (1/2)

* Other user enter with this link (now 0/2 users can enter)

* One of this users that joins with your limit invite link exit the group (Now the link allow to 1 user to enter, because the count change from 0/2 to 1/2)

The best is create a link also with expiration time or revoke it when the user is join

How to count the number of invitees with robot

The count of the invites is doing in the telegram back end side and is not return in the Telegram Bot API.
Example case:
* 1 User enter with this link limit to 2 user (1/2)

* Other user enter with this link (now 0/2 users can enter)

* One of this users that joins with your limit invite link exit the group (Now the link allow to 1 user to enter, because the count change from 0/2 to 1/2)

The best is create a link also with expiration time or revoke it when the user is join

How to count the number of invitees with robot

At less you use a db for store it, can be complex to know


Failed to update to latest patch version

Describe the bug
It looks like shivammathur/setup-php@v2 is not respecting the patch version specified in the php-version variable nor the update parameter if it is set to true. Currently I couldn't found a way to enforce PHP version 8.1.9 on GitHub Actions. Also, some additional logs would be great to make it quickly noticeable that the version set up by the action is not the exact one specified by the parameter. (Also tried just setting the version to 8.1 with update: true, still 8.1.8 was selected)
Version

[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
runs-on: ubuntu-latest
PHP versions
8.1.9
To Reproduce
      - name: Setup PHP 8.1.9 with composer
        uses: shivammathur/setup-php@v2
        with:
          php-version: 8.1.9
          tools: composer
        env:
          update: true

Expected behavior
==> Setup PHP
✓ PHP Found PHP 8.1.9

==> Setup Tools
✓ composer Added composer 2.3.10

Screenshots/Logs
==> Setup PHP
✓ PHP Found PHP 8.1.8

==> Setup Tools
✓ composer Added composer 2.3.10

Additional context

Are you willing to submit a PR?
No

The GitHub Actions runner images and setup-php both use builds from ppa:ondrej/php. There is no build for 8.1.9 yet on the ubuntu PPA.
https://github.com/oerdnj/deb.sury.org/issues/1821

Thank you for the quick reply, issue is definitely not a bug than.
A few suggestions to consider:

Maybe add it to the readme where the PHP version is coming from. I found the package now at the end but this was not obvious for me when I looked for the PHP source.
Maybe add a comment to the GitHub Action's output in case the specified PHP version could not be exactly fulfilled, so it's more obvious the problem is not with the action itself or how it was used.

Thanks!

It should now update to 8.1.9 when update environment variable is set to true.
You can use shivammathur/setup-php@verbose to get the logs whenever you want to debug.


Document xxs breakpoint

What is the context of this PR?
Fixes: #1367
How to review
Docs make sense

 Issue #1367: Grid: Document new additional breakpoint (400px)


Unexpected UI errors after restarting shipyard controller container from Shipyard UI itself.

Hello,
Playing around with Shipyard I decided it would be interesting to restart shipyard's controller container using Shipyard's UI. Here is what happened:

Still need to take a look at the dev tools console errors, but this seems like an interesting scenario, albeit edge.

This happens when the UI cannot connect to the backend.  There's not much we can do until it restarts and you reload the page.  We give you enough rope ;)

@ehazlett I know! Thanks :wink:


help

Process returned exit code 1:
error during connect: in the default daemon configuration on Windows, the docker client must be run with elevated privileges to connect: Get "http://.pipedocker_engine/v1.46/version": open //./pipe/docker_engine: The system cannot find the file specified.

Hmm, is you docker installation configured to require Administrator privileges? If so, you need to launch XPipe as Administrator as well. If this is a remote system, you need to login as Administrator.

I will close this issue as inactive for now


Different file/folder structure in wasm and TS

In the go-wasm part, you are using the approach of having roles for files and on the ts side, you are using objects for folder names. I think this could be made more consistent, by also having role-based folders on the ts side.

Currently, we won't change the folder structure. We focused on keeping the file naming consistent (ignoring case sensitivity), which is role-based both in Go and TS. For Go, we only divide into wasm and non-wasm directories whereas in TS we need(ed) a different structure.


screenshots: update image golden data for E2E test for local runs

Bug: #547

Now the test passes both locally and in CI :D

I have merged by accident :( feel free to revert if not ready

Up to Maksim

I think we still have to find a way to test it regardless of platform. So event having the tests passing, I'd reopen the bug.


<source> tag in search/replace patches

ChatGPT generates invalid patches sometimes. With "<source>" tag before filename. And Aider treats it as "<source>filename" file name. Those patches can be fixed with a special but simple parsing workaround. Can you update a parser for that case?
Aider Version: v0.15.0
<source>config.ini
<<<<<<< HEAD
... content removed ...
=======
... content removed ...
>>>>>>> updated
</source>
...

> Allow creation of new file <source>config.ini? n

And sometimes it generates patches like that (Aider v0.16.3):
<source>markdown
data_structures.md
<<<<<<< SEARCH
... content removed ...
=======
... content removed ...
>>>>>>> REPLACE
</source>

ChatGPT loves "" tag for some strange reason.

Thanks for filing this issue.
Aider actually encourages GPT to use <source> when the file being edited already contains the normal triple-backtick fence.
But you have identified a corner case where aider isn't correctly noticing when GPT generates a malformed SEARCH/REPLACE block with <source>filename.ext. Aider normally notices such malformed S/R blocks and tells GPT to try again and fix the problem. I just made a fix so that aider correctly does this in the case you've identified.
The change is available in the main branch. You can get it by installing the latest version from github:
python -m pip install git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you.

I'm going to close this issue for now, but feel free to re-open or file a new issue any time.


[WIP] Feature/generic vectorizer refactor and tfidf

This PR:

Re-factors generic vectoriser to only include a switch between deep-learning and tf-idf
Adds tf-idf vectoriser

Todo:

[ ] Add tests

Another solution could be to name it WellcomeTfidfVectorizer or something that signifies that is is our custom extension and then either inherit and augment or purely maintain the same API with the additional parameters that you are adding i.e lemmatizing etc

Yes, I like that idea - I think I will move on with that one. Instead of inheriting from TransformerMixin it will inherit directly from TFidfVectoriser

Once the Travis CI passes, this is ready to review. I re-factored FrequencyVectorizer to inherit TfidfVectorizer and called it WellcomeTfidf. Also added tests.


Can not use web-assembly rust impl because of 'Cannot access '__wbindgen_throw' before initialization' error

I am trying to add some web-assembly to my project. On the beginning I created a simple method to check is my webpack works well and can I use my .wasm modules. So I created sth like this:
#[wasm_bindgen]
pub fn return_char() -> char {
    return 'a';
}

And then I used this inside constructor of my class (just to check is everything works fine):
constructor() {
    super(template);
    this.setUpElements();
    console.log(return_char());
}

This worked as expected I saw 'a' in my web-app console:

Then when I saw that everything works fine I created structure with implementation:
#[wasm_bindgen]
pub struct LexicalAnalyzer {
    expression: char
}

#[wasm_bindgen]
impl LexicalAnalyzer {
    pub fn new() -> LexicalAnalyzer {
        let expression: char = 'a';
        LexicalAnalyzer {
            expression
        }
    }

    pub fn expression(&self) -> char {
        self.expression
    }

    pub fn change_expression(&mut self) {
        self.expression = 'b';
    }
}

And then again I call these methods in my TypeScript class constructor and I should see 'a' and 'b' in my console
constructor() {
    super(template);
    this.setUpElements();
    const lexicalAnalyzer: LexicalAnalyzer = LexicalAnalyzer.new();
    console.log(lexicalAnalyzer.expression());
    lexicalAnalyzer.change_expression();
    console.log(lexicalAnalyzer.expression());
}

But this didn't happen and I saw only this error:

calculator_engine_bg.js:5 Uncaught (in promise) ReferenceError: Cannot access '__wbindgen_throw' before initialization at Module.__wbindgen_throw (calculator_engine_bg.js:5)

To build my rust code I used wasm-pack and build my code with wasm-pac build comand
I do not know why I am getting this error and how can I fix this. I also really do not know what I am doing wrong :/

I have this issue as well!

@jxmorris12 I solve this problem. If you are interested in solution i recommend you to check my answer on stack:

Can not use web-assembly rust impl because of 'Cannot access '__wbindgen_throw' before initialization' error.

And if you want to see an example I also created a repository with working code example:

kaczor6418/webpack-web-assembly-rust-impl-bug
On branch sync-web-assebmly-and-bootstrap-file.


Use jQuery plugin for minion polling

@santiph Unfortunately we can't add more moving pieces here regarding the javascript, as we're getting really tight in the schedule and we decided it doesn't make much sense to invest in a custom javascript architecture right now, as we we'll probably move away to some proper js library some time soon.
Thank's for the work on this, but we'll rather update the minion poller to do what we need for the nodes page


[Skip CI]by default, not to build OVA with customized property, which not supported by vSphere

background:
originally, there's a feature in RackHD OVA which during deployment , we can specific the IP of the OVA, in a non-DHCP environment.
that make convenience for batch deployment for lots of RackHD OVA without manually setup IP.
But it turns out this feature only support by VMWare vCenter, not supported by vSphere.
Taken many user still use vSphere, so I have to make this advanced feature not a default one.
What's Change:
I didn't change anything from original line 217 - 295.
but just put them inside a "if" block.
and this "if" is by default false.
Jenkins Change
now the below steps are managed in pipeline, so here's the change:
https://github.com/RackHD/on-build-config/pull/93
those two PR depends on each other.
for "ova-post-test" job, right after this PR merged,
we should add 3 lines in the job script as below, to convert the OVA(by default, no IP property ), use script to turn it into a OVA with IP property.
OVA="$PACKERDIR/rackhd-${OS_VER}-${RACKHD_VERSION}.ova"
+++++++
cd $PACKERDIR
./add_IP_property_to_ova.sh rackhd-${OS_VER}-${RACKHD_VERSION}.ova
cd -
+++++++
..
bash ./build-config/build-release-tools/post_test.sh \

Test Done

Build:
use below command locally, and build successful

OS_VER=ubuntu-16.04   BUILD_TYPE=vmware   ANSIBLE_PLAYBOOK=rackhd_ci_builds   ./HWIMO-BUILD

Deploy ova(without customized property) to vSphere ( SH Lab ESXi 245) - successful
ovftool<EMAIL_ADDRESS>
Opening OVA source: rackhd-ubuntu-16.04-bak.ova
The manifest does not validate
Opening VI target: vi://root@<IP_ADDRESS>:443/
Warning:
 - Line 98: Invalid value 'CONTROL' for element 'Connection'.
 - Line 109: Invalid value 'PDU' for element 'Connection'.
 - Line 120: Invalid value 'BMC' for element 'Connection'.
Deploying to VI: vi://root@<IP_ADDRESS>:443/
Progress: 9%

Deploy the  ova(with customized property) - also success. not shown the result.

Reviewer:
@yyscamper   @anhou   @PengTian0   @changev

BUILD RackHD #1350 : ABORTED

** BUILD Install-CentOs-6.5  #393: SUCCESS
** BUILD SmokeTest-CIT  #602: SUCCESS
** BUILD SmokeTest-FIT  #408: SUCCESS
** BUILD Install-ESXI-6.0  #24: ABORTED
** BUILD Install-Ubuntu-14-04  #371: SUCCESS

BUILD RackHD #1351 : ABORTED

** BUILD Install-Ubuntu-14-04  #372: SUCCESS
** BUILD Install-ESXI-6.0  #25: ABORTED
** BUILD Install-CentOs-6.5  #394: FAILURE
** BUILD SmokeTest-CIT  #603: SUCCESS
** BUILD SmokeTest-FIT  #409: SUCCESS

BUILD RackHD #1353 : ABORTED

** BUILD Install-ESXI-6.0  #28: ABORTED
** BUILD Install-CentOs-6.5  #397: SUCCESS
** BUILD SmokeTest-CIT  #606: SUCCESS
** BUILD Install-Ubuntu-14-04  #375: SUCCESS
** BUILD SmokeTest-FIT  #412: SUCCESS

Hi, @johren , thanks for review. I've fixed all your comments.
this PR depends on https://github.com/RackHD/on-build-config/pull/93

so embarrassed T_T...
@johren
please refer to  fix commit
https://github.com/RackHD/RackHD/pull/632/commits/e463f426cb4499a0ca01f370d1906ed57d1fc161
============
Test done:
build:  OS_VER=ubuntu-16.04   BUILD_TYPE=vmware   ANSIBLE_PLAYBOOK=rackhd_ci_builds   ./HWIMO-BUILD
deploy 1: ovftool   --overwrite --powerOffTarget --skipManifestCheck --X:waitForIp  --net:"ADMIN"="Peter DHCP" --datastore="xxxxx" --name="Peter_Test"  rackhd-ubuntu-16.04.ova<EMAIL_ADDRESS>
deploy 2:  ovftool --prop:adminIP=<IP_ADDRESS> --prop:adminGateway=<IP_ADDRESS> --prop:adminNetmask=<IP_ADDRESS>  --prop:adminDNS=<IP_ADDRESS>    --overwrite --powerOffTarget --skipManifestCheck --X:waitForIp  --net:"ADMIN"="Peter DHCP" --datastore="xxxxx" --name="Peter_Test"  rackhd-ubuntu-16.04.ova<EMAIL_ADDRESS>deploy 1 & 2 are against a vSphere IP(245). both success.

Hi, @johren , @gpaulos , @diontje , I've fixed according to your comment in commit https://github.com/RackHD/RackHD/pull/632/commits/45d0fa28d3117afd6474f196fe2ca7f225f7786a
please review again.
thank you !


Google drive clone

Google drive clone 30

Title : Google drive clone
Issue No. : 30
Code Stack : HTML , CSS
Close #<issue_no>

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md
[X] I have updated the Index.html file for my contribution

@apu52 merge it

@paulsuryanshu looks good! thanks for contributing!

but bro next time put ur projects in the Projects Folder!

ok


Remove unnecessary files

List of unnecessary files in the repo that removed:

README_BACKUP_6160.md
README_BASE_6160.md
README_LOCAL_6160.md
README_REMOTE_6160.md
.README.md.swp

Issue: Remove unnecessary files #740

Great thank you!
@all-contributors add @dkochetkov for code


docker container stops after creating (exit code 134)

Hi.
My docker script looks like this. I already tried different versions, this is the latest. Evertime I create the container it stops with an exit code 134 (from portainer). I already tried to give the container admin rights (UID 1026), but it makes no difference.
docker run -d
--name=mkvtoolnix
-e USER_ID=1026
-e GROUP_ID=100
-e PUID=1026
-e PGID=100
-e TZ=Europe/Berlin
-p 5805:5800
-v /volume1/docker/mkvtoolnix:/config:rw
-v /volume1/Downloads:/storage:rw
jlesage/mkvtoolnix
the log (exported to csv):
mkvtoolnix.csv

Same as #54.

@Sylvio82, lets continue the discussion under the initial issue.


Add --column argument

fixes #37  which I just saw also has another open pull request in #73

I have improved the design a couple of iterations and A/B compared it using bash time'd bash-scripts and lightly looked at pprof results..
My implementation only calculates column value if it's requested using --column..
I tested everything on about 1.5gb data residing on a  ram disk to avoid random fluctuations caused by  disk io. The difference for queries with lots of results were quite big, somewhere along the lines of 4 seconds vs 6 seconds..  This  adds a tiny bit of complexity to the code, and I chose to propagate the --column option through the Match type since that seemed like what was going on earlier as well.  I'm still not sure about if it's worth it generally but for searches in very large data sets on very fast io it probably makes sense.
Let me know if some changes should be made.

should probably save the match regions in full for use in both match highlight and column number

:+1: would be super helpful to have this -- some tools (eg emacs) depend on file-line-column to provide a reference point to jump to.

@vermiculus yeah  emacs ant pt.el is my use case for this patch, see https://github.com/Wilfred/ag.el/issues/46 ..  I do think the design is flawed though as I wrote in my last comment. I am hoping to get a comment/direction from @monochromegane because I do not want to spend more time implementing something that won't be merged.

Thanks for your PR, and I am sorry for the late response.
I tried your PR and https://github.com/monochromegane/the_platinum_searcher/pull/73.
Both PR is nice, but #73 has a problem as you say, so this time I will merge this PR.

thanks!


Reading New Mail Broken

For the past few months, K9 Mail no longer allows me to read unread mail in order. I order my mail with newest at the top. When I have new mail, I scroll down to the first unread messages and click. When I delete this message, K9 Mail previously showed me the next unread messages. Now it seems to sort the read email together which causes the 'next message' to be mail I have already read, requiring me to hit delete, next, next, next,... to get to the next  unread email. It is no longer practical to read new email with K9 Mail when sorting messages to show newest first.

Dear cketti,
I don't believe you are correct.  I've used K9 mailed since Android first began and I've never had this problem reading mail.  I have taken a video to show you.  Here's the scenario:

I have ~20 new emails all shown at the top.
I click 2 interesting emails which I want to read first, which are at random locations in the list.
These 2 read emails now change locations in the list display.  They are at the top of my read email, with all of my unread (bold) emails grouped together at the top. This is fine, but it causes the following problem when reading the rest of my emails... as follows...
Click the bottom (oldest) unread email.
It shows fine and I delete it
6.   Now the first of the 2 interesting emails (step 3) shows.
I have to click <, which shows the next of the 2 interesting emails I read (again from step 3).
I have to click < AGAIN, to show the next UNREAD email.
Click delete and repeat steps 6-9 to view and delete each unread email.  It's a horrible pain.

Video attached in next comment.  Notice at the start of the video the 2 READ emails at the top of the list at times 5:47am and 7:02am, which are shown out of order now to the rest of my UNREAD emails and are the two emails I have to skip over (steps 7 and 8) to read each next UNREAD email.

http://crosswire.org/~scribe/k9reorderedread.mp4


Add Lumen

Lumen is a PHP Microframework.
This PR will add a curated list of awesome Lumen related resources.

Put the contribution guidelines in a separate contributing.md file.

Thanks for accepting. I will put the contribution guidelines in a separate file


No change happens if delete all the resources through kpt ts functions

No change happens when we try to filter out all the resources through kpt function.
kpt version: 0.36.1
We have a kpt ts function filtering resource not fit the conditions. Simply like below:
export async function filterResources(configs: Configs): Promise<void> {

  configs.delete(
    ...configs
      .getAll()
      .filter(
        ko =>
          condition1 || condition2 || condition3 ...
      ),
  );

If all resources in the configs match and should be deleted, then the input source directory keeps as unchanged.
Expectation:
The input directory becomes empty.
Reproduce:
It is not hard to reproduce the bug.

Init a kpt work environment follow the official document.
Add an extra ts function like below:

import { Configs } from "kpt-functions";

export async function filterInfraResources(configs: Configs): Promise<void> {
  configs.deleteAll();
}

filterInfraResources.usage = "";

Build it and Run the function on the example configs.

$ mkdir result
$ kpt fn source ../../example-configs/shipping-dev | node dist/filter_infra_resources_run.js | kpt fn sink result
$ ls result

Run ls result, which shows all the files from the original folder.

But if we make the kpt function return at least one resource, this works. Modify the kpt function like below:
export async function filterInfraResources(configs: Configs): Promise<void> {
  configs.delete(
    ...configs.getAll().filter(ko => ko.metadata.name != "viewers"),
  );
}

Then, rebuild and re-run the above steps. I can get only rolebinding_viewers.yaml in the result folder.
So the deletion only works when there is at least one config left, so we have to add a temporary config into the configs to make sure kpt deletes the rest of the resources. And delete the temp resources after kpt fn run finishes in our Java code.
Any ideas?
Thanks in advance!

This behavior is from kio.Pipeline. https://github.com/kubernetes-sigs/kustomize/blob/14dc3dfb81ce52e1b9d7e90a1b673de7e78c0c52/kyaml/kio/kio.go#L124
Pipeline has an option ContinueOnEmptyResult to control it.

I am not sure what's the reasoning behind this behavior, but it seems to be inconsistent. Prioritizing it.


add date filter in facility apis

fix #1257
I added date filters (from and to) to the following apis:
input is timestamp

Coverage decreased (-0.2%) to 39.892% when pulling 0c74837dc5a5a1901215486e6b0075f24a21a8a5 on ann26:1257-date-filter-facility into 02ef55853bbdb36a68b4aeecf02bc15d15b77e3c on healthsites:develop.


Make Event a generic type accepting Kind

Allows using types like Event<3> and SignedEvent<1>, etc.
Using plain old Event will still work, and it preserves the current behavior. With no arguments it will match only the known Kinds, but you can also use Event<number> to match any kind.
As a result you can can do awesome stuff like this, where your functions guard against particular kinds of events: https://gitlab.com/soapbox-pub/mostr/-/blob/c67064aee5ade5e01597c6d23e22e53c628ef0e2/src/activitypub/transmute.ts#L184
I've been using this pattern in all my Nostr projects. It doesn't have the full discriminated-union behavior I wish it did in switch statements, but for what's currently possible in TypeScript, it's a huge improvement.
This is a backwards-compatible change.

I am continuously amazed at these type tricks you can do.


[WIP]   Nuke 7's correspondence

issues #9

resolve:    #9

Codecov Report

Merging #10 into develop will decrease coverage by 4%.
The diff coverage is 80%.

@@             Coverage Diff             @@
##           develop      #10      +/-   ##
===========================================
- Coverage    90.27%   86.27%   -4.01%
===========================================
  Files            5        6       +1
  Lines          144      153       +9
===========================================
+ Hits           130      132       +2
- Misses          14       21       +7

Impacted Files
Coverage Δ

Source/WebP/WebPDataDecoder.m
87.5% <100%> (ø)

Source/WebP/WebPDataDecoder.h
100% <100%> (ø)

Nuke-WebP-PluginTests/WebPDecodeTests.swift
93.61% <100%> (ø)
:arrow_up:

Source/Extensions/Data+WebPFormat.swift
76.92% <76.92%> (ø)

Source/WebPImage.swift
66.66% <77.77%> (-18.52%)
:arrow_down:

Nuke-WebP-PluginTests/LoadWebPImageTests.swift
84.37% <77.77%> (-10.87%)
: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 5b63dbe...c778e8f. Read the comment docs.

@kean Thank you so much for telling me!
When push finally passes, it will be released as 2.0.0 - beta 3!


Ignore sb- prefix when checking audience

Currently sb- isn't ignored when validating audiences from the token. By looking at the existing unit tests, additional CIDs that doesn't have sb- are added to the trusted client ID list, but this behaviour is more intuitive if it's done in JwtAudienceValidator.

 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.Edmund Ham seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @edmundham
thanks for your pull request. But the decision was to configure the JwtAudienceValidator in case of XSUAA

with the client id (which starts with 'sb-' prefix) and
with the Xsuaa application identifier, which is required in case of clone tokens.


Panel focus lost after selecting a command using the mouse

Issue Type: Bug
There's a inconsistant behavior regarding panel focus and the command palette. When an item in the command palette is selected using the mouse, the previous focus on a panel is lost, which is not the case when selecting an item using the keyboard.
(0. Open a JS document, for the steps 3 and 7)

Click on a panel (like the Timeline panel) so it gets the focus (blue outline)
Hit Ctrl+Shift+P to open up the Command Palette
Click on any command (i.e. Format Document) using the mouse
See how the Timeline panel is not focused anymore
Click on the Timeline panel again so it regains focus
Open up the Command Palette again
Select the Format Document command using the keyboard and validate with the Enter key
See how the Timeline panel automatically gets the focus back

Here's a screen record of the steps:

VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:35.169Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.93GB (4.31GB free)

Process Argv
--disable-extensions

Screen Reader
no

VM
0%

Extensions disabled

@chrmarti This looks straightforward to me, I can work on this. 🙂

@lambainsaan Great, looking forward to your PR!

/duplicate #114590
I know this is older but that one has more context and this one will be linked to that one.


Errors upon server load.

 These errors indicate that these files weren't run through the extension preprocessor, which can't happen.
Try reinstalling.

It might be caused by a third-party addon - but I don't know what reason a third-party addon would have to just blindly parser all these files. (Maybe it's some horrible broken 'security' addon that parses every Lua file to try and look for suspicious bytecode?)

It's CAC - https://www.gmodstore.com/scripts/view/460


Re-check #184

Description
https://github.com/CiscoDevNet/vManage-client/pull/184

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/issues/201


Add fold to List and a pointfree version for Arrays/Lists

Doing laundry since 1985

This PR adds the fold pointfree function as well as a fold function on the List crock in response to issue#95.
In addition, cleaned up the List error specs to make them just a bit better. Always room for some 💅

Coverage remained the same at 100.0% when pulling 1b3336feec4f8c0755069670b9638d59c20fdd80 on add-fold into 26b1e71bfd3f6c78197e85084f1338cdd49b6778 on master.


Mac download broken

Heads up that the Mac download from https://force-cli.heroku.com/ is broken. You get a file called "force".

Yes, that is the CLI app.  You may need to chmod +x the file.

Ah, apols. I was expecting a package. Thanks!


change administrate/rails version deps

change rails and administrate versions

Sorry about this dragging on, @dan-ding. I'll try and have a look at this sometime this week. I'll take a look at the CI issues too.

no worries. I wasn't sure where you wanted to go with things, so left them as is.

Thanks a lot for your help, @dan-ding! I'll release an updated version of the gem asap.


Build and publish exportarr binaries to Github releases

Sorry, I'm new to Prometheus and node_exporter.  I've downloaded your zip and also via the git clone, but nowhere in the resulting files is there any executable or make file.
I'm not using docker, just ubuntu 20.04.
The README.md says that to run it from CLI:
### Run from the CLI

```cmd
./exportarr --help
./exportarr sonarr --help
./exportarr radarr --help

Sonarr
./exportarr sonarr \
  --port 9707 \
  --url http://<IP_ADDRESS>:8989 \
  --api-key amlmndfb503rfqaa5ln5hj5qkmu3hy18 \
  --enable-episode-quality-metrics

But there's no "./exportarr" anywhere.
How do I use this with Prometheus?

Missed this.
I have not published binaries for exportarr, those steps are for if you compile it yourself with go.

I tried to download and compile, but it fails to build.

Your go version is very low, this is compiled against 1.16.
But you can try go build -o exportarr ./cmd/exportarr/

Binaries will be attached to the next release 🎉


Added workflow and downloads badge to README

Added workflow and downloads badge to README

It looks like this

LGTM

Looks great, thank you!


backgroundDismiss only works to left and right of the dialog with Firefox

jquery-confirm version:
v3.3.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request
[X] support request
Current behavior:
Hi :)
I use backgroundDismiss: true but that works only if I click to left and right of the dialog, not above or below of it... with Firefox 66.0b13, works with Falkon.
I use alert in a function.
Any idea why ?
Thank you !
Expected behavior:

Steps to reproduce:
Related code:
function FenetreInfo(Titre, Message, Couleur = 'red', Largeur = '500px', Icone = 'fas fa-exclamation-triangle')
{
    $.alert(
    {
        title: Titre,
        content: Message,
        icon: Icone,
        type: Couleur,
        typeAnimated: true,
        backgroundDismiss: true,
        escapeKey: true,
        boxWidth: Largeur,
        useBootstrap: false,
        buttons:
        {
            Exit:
            {
                text: "OK",
                btnClass: 'btn-' + Couleur,
                keys: ['enter']
            }
        }
    });
}

Other information:

I was having this same problem on firefox.   I added the following style after importing the default jquery-confirm css file and it seems to take care of the issue for me.
.jconfirm .jconfirm-bg { z-index: -999; }

Thank you :)


[unreleased] Dexie.Observable: TypeScript compile-time errors - missing properties

In the unreleased Dexie head commit 1f2cad1eee7166d1a0c4b3cb9ac13120a3b2131b , TypeScript compilation of a module that constructs a Dexie database after importing Dexie.Observable fails with the following tsc error:
        TS2322: Type '{ (db: Dexie): void; createUUID: () => string; on: ObservableEventSet; localStorageImpl: { setItem(key: string, value: string): void; getItem(key: string): string; removeItem(key: string): void; }; _onStorage: (event: StorageEvent) => void; }' is not assignable to type '(db: Dexie) => void'.
    Types of parameters 'db' and 'db' are incompatible.
      Type 'Dexie' is missing the following properties from type 'Dexie': observable, _localSyncNode, _changes, _syncNodes, _intercomm

A repro TypeScript module to reproduce this is:
import Dexie from 'dexie';
import observable from 'dexie-observable';

class Database extends Dexie {
    constructor() {
      super('dbname', {addons: [observable]});
    }
}

Naive assessment: it seems as if the extended properties from the Dexie.Observable addon are not present on the Dexie type when the compiler performs type-checking.
Note: the addons: [observable] option is provided to the super constructor because otherwise the on and subscribe functions do not appear to be added to the resulting database class.
Commit 3f364f3ea85e063026b20b6e0e7b95191e82331e seems to be the first version of the Dexie.Observable addon where this problem occurs.

Ok. Thanks for the feedback. I will investigate.

I tried to reproduce it using
npm i dexie@next dexie-observable@next

Get no typescript error there. But when downgrading dexie-observable to dexie-observable@latest I get many tsc errors.
Which version of dexie-observable did you test with? If you didn't try dexie-observable@next, it could be the reason it doesn't work.

Thanks for taking a look.  This was using a git checkout of Dexie + Dexie.Observable @ https://github.com/dfahlander/Dexie.js/commit/1f2cad1eee7166d1a0c4b3cb9ac13120a3b2131b.
I ran npm run-script build to compile dist files for each of them, and then refer to those via file paths in package.json -- I'll try to package an example up in a repro repository soon.

Repro for this is available in https://github.com/jayaddison/dexie-1144-repro (with instruction steps in the README).

I'm unable to reproduce this using<EMAIL_ADDRESS><EMAIL_ADDRESS>


Branding text section contains Asgardeo references

Describe the issue:

The branding text section contains Asgardeo references.

How to reproduce:

1.Login into the console
2.Go to branding section
3.Navigate to text section
Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

Closing this issue as this is no longer reproducable

Not reproducible in the latest build, hence closing


Upgrade guide for federation has wrong steps

Description
The process to upgrade a federation is wrong described in the documentation.
Expected behavior
Documentation should be updated with correct steps.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Also please add a reduced steps guide to upgrade from maintenance versions (5.12.1 -> 5.12.2). No need to upgrade DB for instance.
Basically, bring back this: http://docs.opennebula.io/5.10/intro_release_notes/upgrades/upgrade_510.html

PR to merge in 5.12 and master:

docs


Error: Unsupported Buffer file type

Enabled Google Static Maps API
OS: Windows 10 Pro 64bit
Commit: 295141830c8c7ebf3f81a5573b68ffe727453c4d

Terribly sorry. Was using the wrong API key.


[BottomNavigation] can bottomAction combine with Badge？

[BottomNavigation] can bottomAction combine with Badge？

Hi,
You can try to use custom element instead of icon name
<BottomNavigationAction
      ...
      icon={<Badge><YourIconElement /></Badge>}
/>

That is OK，thanks


fix: move readiness endpoint to technical debug

Checklist

[ ] I have read the coding guide
[ ] My change requires a documentation update and I have done it
[ ] I have added tests to cover my changes.

Description

Motivation and context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

This change is

Why does this have to be moved?

Why does this have to be moved?

Currently, our infra relies on /readiness endpoint being on the debug endpoint. As this was moved, staging nodes were not seen as ready.
As a stop-gap solution till we completely move the APIs, we decided to move this to debug. The change was verified by deploying this branch on staging.


fix: allow users to override #nitro sub-paths

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Previously it was impossible to override sub-paths (for example, #nitro/error) as #nitro was added first to the alias object.
Do not yet merge as there is an error that this reveals in Nuxt that needs to be resolved first.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

We might use #nitro-* alias for a temporary error alias workaround. I don't think reusing #nitro namespace is best idea in terms of stability.


Add AppName to the new header

minor update: update dependencies to remove vulnerabilities
update styling to match the new header

 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.


Fix symlinking on osx-arm64

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #20.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Fixes #20
Fixes #19

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

ping @conda-forge/python-app

Are there any lingering issues before this can be merged? Thanks!

Nope. waiting for @conda-forge/python.app. @conda-forge/core, I'm adding myself as a maintainer. Merge?


Support global configurations from domain projects

For all connectors/modules requiring config elements (eg. HTTP, DB etc), we extract information from global config object. This is done through the APIs provided by Mule. However this API isn't reaching out to any configurations that are in mule domain projects. This limits us to extract useful information about configurations. We need to find a way to reach to domain configurations and capture the information.
Related: #65

Updated documentation for domain config limitation - https://github.com/avioconsulting/mule-opentelemetry-module/pull/70/commits/c23bfbcd29c76ff8c6f1e621cdc2fc18e178afd0 in PR #70


Exception raised when using the expert mode.

Hello there,
Got an error "Call to a member function post() on null" at  …/user/plugins/form/form.php:184 when clicking the "save" button modifying a user properties in expert mode.
Steps to reproduce:

create a user via the users admin plug-in
back to the list of user
click expert for the user just created
change something (I changed the language)
click save

Environment:

Linux Mint Debian Edition "Betsy" (Debian 8 Jessie)
Apache 2.2.2
PHP 7.2.2
Grav v1.4.0-rc.2 - Admin v1.7.0-rc.3
Admin Addon User Manager v2.0.3

Post data :

raw
""" state: enabled\r\n email<EMAIL_ADDRESS>fullname: 'a test user'\r\n language: fr\r\n twofa_enabled: false\r\n twofa_secret: UK7MCEYWZ6LVRKYJJ3UMNTI7E77AHYGS\r\n access:\r\n   admin: {  }\r\n   admin-addon-user-manager: {  }\r\n hashed_password: $2y$10$WEV.rAA.ulF3h/w9Ag2l0eXFy.yUrbkQsOM8hAdnHW5ogc.z.VPzq\r\n """

form-nonce
"824ed52fa2613d207cc833986bf626b0"

I've tried various simple things like clearing the cache, changing the data, etc..., but the problem still occurs.
Thanks for the plug-in, very useful, makes admin life much easier...
Regards,
Michel.

Happens the same to me :wink:

Ok, thanks. Wait and see then...

Yes, solved with Form 2.11.5.


Connection/session timeout properties not set when not sending in batch mode

Released in 6.0.0-rc1.


[Feature]: Telegram Alerts

Description
I know there's Slack integration, but Telegram feels more appropriate for notifications.  Slack requires a workspace for you to connect the app. With Telegram, it can be sent through a bot.
Suggested Solution
The alerts could either come from wandb bot or a personal bot with the token provided by the user.
Alternatives
No response
Additional Context
No response

Hi @Y-T-G thank you for this great suggestion! I have created an internal ticket for our engineering teams and will keep you posted here on its progress!

Yeah, having telegram notification would be amazing!

Thank you for +1 those requests. May I please ask what exactly notifications you would like to receive, is this for alerts on runs that have crashed or completed? it might be now possible to receive notifications with webhooks for those events on artifacts or model registry actions, but that may not cover your use case.

I would like to have the same functionality for it as wandb has for Slack and emails. Like being able to send an alert using wand.alert().

Great, thank you for the added context, @Y-T-G will pass this to our Product team.


docs(component): document disabling hammertime

I had to check the source to see if it was possible to not apply the touchActionSelectors on a specific component. I figured this would be helpful to have in the documentation.

Thanks!


SPIRE indeed dispatches x509 key and pem files which can be used to establish TLS and mTLS — let Aegis SDK expose these, so that anyone who needs it can consume it. Also update Aegis Sidecar to save these to a secure place on the workload

Not sure it makes sense.
The certs in themselves are of little use because the workload will need to know which peers it can trust, and that is done through trust bundles.
So instead, it would be better for an app to use the SPIRE workload API to get both certs and the trust bundles themselves by using a library like go-spiffe.
While, Aegis can wrap around go-spiffe and provide those bundles, that wouldn’t have too much of a practical value.
Closing this as WONTFIX for now.


Document and test: what exactly does it mean to set value= for a Variable?

From https://github.com/convexopt/gpkit/issues/93, it's possible to declare the value of a constant at the same time as you declare the constant, with syntax like
rho = gpkit.Variable("\rho", value=1.2, units="kg/m^3")

Document exactly what behavior this syntax implements. Unit test that the return type is Variable. Unit test that the 1.2 value is stored in the expected attribute. Unit test any other expected behavior known by @bqpd.

http://gpkit.readthedocs.org/en/latest/basiccommands.html#declaring-variables could be a good place to do this.

And in the docstrings. In the above, should Monomial be changed to Variable according to our best practices?
Current docstrings:

Posynomial: does not mention value= kwarg
Monomial: empty
Variable: looks good; mentions value=

Yeah, that sounds right! Do you think Posy/Monomial should mention value=, considering that it doesn't always work with them?

re: Posy/Monomial mentioning value=, no -- if it doesn't always work with them, I think we should limit use of that keyword argument to Variable. I changed the original ticket text accordingly. We should add some unit tests to t_vars.py before closing this ticket.

There are now several tests for this in t_subs.py


[New game]: Flashlight Pointer Game

🎮 Game Request
Flashlight Pointer Game
Point down the features
Use your flashlight to find three rabbits hiding in the webpage somewhere out of sight.
Find all the three rabbits to win the game.

Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @Samriddhi15 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Add Path variant to NamedChannelOrUrl

Besides having Name or Url for NamedChannelOrUrl we should also have a Path variant to allow distinguishing relative paths.
Ideally, we used typed-path for the paths to ensure that Windows paths can also be represented on unix.

I think this is done, right?


Full Screen and Social Sharing Options for Albums

Adds options to control access to the full screen button and the social sharing buttons on an album.
Sharing Options:

Full Screen button on the photo view:

Social Sharing buttons on the album view:

Social Sharing buttons on the photo view:

Great! Can you do also sharing with bbcodes and stuff? And if have time also resizing photos?

@skyway22
BBCode is a markup language commonly used in online message boards. Lychee already provides access to the image file with a link

and that can easily be insterted into the BBCode corresponding to the image tag, something like this:
[img]http://lychee.website/path/to/image.jpg[/img]
Resizing photos is not really related to the enhancement in this pull request.

@jnsnkrllive Hey, thanks a lot for the PR. I'm sorry but it's currently not planned to add an option to enable/disable sharing. What speaks against the buttons?
I'm also against "Enable Full Screen View" as long as Lychee isn't able to zoom. Viewing big photos on small screens is a pain without a full-screen button. I planned to couple it with the download-option when we have a proper zooming feature.

@electerious Thanks for considering my PR. I really appreciate the hard work you have put into this project, and have found it very useful already. For my personal use of Lychee, I wanted to restrict access to the photos as much as reasonably possible, which was more than the current version allowed. The two options I identified first for restricting access were the social sharing buttons, and the full screen view.
I did look through the issues before to see if other were interested in the same restrictions. For example, at least 1 other person has wanted a way to disable the social sharing buttons (Issue #570).
As for the "Enable Full Screen View", this option was specific to my deployment and was part of my overall goal of restricting access to the source photo files as much as possible. Since there was already an option for downloading, I thought it would be appropriate to also have an option for the full screen view since they are related and shown in the same menu. It sounds like you have future plans for the download option and proper zooming, so maybe this option could be incorporated once that enhancement is added.
I think most people will find the social sharing buttons and the full screen view useful, and probably would not want to disable those options. But for the smaller group of admins who want to restrict access, I think at least having the option to easily do that would be nice.

This PR will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community.


Apply zig fmt to exercises and generate remaining patch files

Closes #21

@quexxon Fantastic! It was super satisfying to see Eowyn run a full testing pass on all 45 exercises for the first time!  Now it should be easy to catch regressions. 🎉
Reviewing the format changes, most of them were very minor. As I feared, we lost some of the compact clarity of some of the "one-liners" like exercises/28_defer2.zig where
    if (animal == 'g'){ std.debug.print("Goat", .{}); return; }
    if (animal == 'c'){ std.debug.print("Cat", .{}); return; }
    if (animal == 'd'){ std.debug.print("Dog", .{}); return; }

becomes
    if (animal == 'g') {
        std.debug.print("Goat", .{});
        return;
    }
    if (animal == 'c') {
        std.debug.print("Cat", .{});
        return;
    }
    if (animal == 'd') {
        std.debug.print("Dog", .{});
        return;
    }

But I'm not gonna make any exceptions because learners should be exposed to "standard" Zig formatting in ziglings, not my personal preferences. And there were only a couple things like that anyway.
🏆 Bonus points for formatting the example code in comments as well!  Did you put those bits into a stub file and pass them through zig fmt as well, or did you just do them manually?
Thanks so much for this work. It's significantly improved Ziglings!

Happy to help!

Did you put those bits into a stub file and pass them through zig fmt as well, or did you just do them manually?

Nothing as clever as that. I just fixed what I noticed manually. There are possibly other (less visually obvious) issues that could be caught by this approach though.


[Feature Request] Read Dirty State Outside

Related to #292.
I need to check if a form is dirty outside because I need to check if there are unsaved changes.
Would be awesome if you can include dirty/pristine as saved prop on form state, or maybe create a helper function isDirty(formName) just like getValues(formName).

@erikras any change implementing it soon? :D

+1 there are several issues related to this "outside form dirty checking". This is a valuable feature since some important use cases based on this (beforeunload and UI change dependent confirming).
Can an admin please check this feature again.

I apologize for not responding earlier. This feature will be present in the next version, and I will accept PRs for the current one, but I think my time will be best spent getting the new one published.

is this change implemented already?

+1

This would be really useful. Is it still planned for 6.0.0, @erikras?

@lynn v6 already has access to dirty/pristine state via the instance API, i.e. you will need to have a reference to the instance of the form component. However, a selector, as suggested in this issue, also seems like it could be useful. I will investigate how much work that would be to get into v6 before release.

I just made a very rudimentary version myself.
Well sortof, I wanted to looks for errors on a form so I just manually
looked up state.form[formName].asyncErrors
I'm guessing that we can be confident about the shape of the state not
changing?
On 30 July 2016 at 00:14, Erik Rasmussen<EMAIL_ADDRESS>wrote:

@lynn https://github.com/lynn v6 already has access to dirty/pristine
state via the instance API, i.e. you will need to have a reference to the
instance of the form component. However, a selector, as suggested in this
issue, also seems like it could be useful. I will investigate how much work
that would be to get into v6 before release.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/erikras/redux-form/issues/648#issuecomment-236191490,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACA_fyVx150xhuVtAPNSoMzk-FXXKaBSks5qagrHgaJpZM4Hauei
.

sooo ... I have dig in the source and I have found a way:
From any component of the app you can do:
import {isDirty} from 'redux-form'

const isFormTestDirty = isDirty('form-test-name')

const mapStateToProps = state => ({
  formDirty: isFormTestDirty(state)
})

@erikras please confirm this is a valid access and if so I think it would be awesome to have in docs

well I've just found that on the docs LOL
http://redux-form.com/6.3.1/docs/api/Selectors.md/

I need the dirty fields and its values outside the form. Is this feature currently available on this version of redux form ?

This can be done by using the isDirty selector.
Also closing due to months of inactivity.


Added connection failure as a reason

Can probably mirror the recently added participant close reasons to catch them all


Remove framework / library comparisons in the docs

Related to #857.
There's a new framework or library everyday. It would consume a significant amount of time to compare other libraries and frameworks, and I think detracts from Skate's focus. It might just be worth answering questions on a case-by-case basis, or blog about it instead.

I agree 👍

basic usage of how to use with framework/lib XYZ is definitely a must not just from PR perspective -> this will also broaden "knowledge" about WebComponents in general, because skate is just WC :)

so yes for no comparison docs, must have for integration docs

I agree with @Hotell; integration docs are essential. Helps make the case for WC and skate, helps authors of skate components who may not use all the same frameworks as their users.

yup @morewry  also we can reference just https://custom-elements-everywhere.com/ as a start for advanced devs ( they can look at the tests for usage :P )

Oh, nice! I haven't seen this yet, thanks for sharing.

Yeah I think this has transformed quite a bit over time. Definitely need integration docs, but may not need "comparison" docs. Maybe for Polymer / X-Tags? But not sure if that's the best thing to focus on and have to maintain. I'll close this for now and add a task or integration docs on the website issue.


feat: Add support for .NET 9

Description of Change
Builds binaries for emscripten 3.1.56 used by .NET 9
Bugs Fixed
None

API Changes
None.

Behavioral Changes
None.

Required skia PR
None.

PR Checklist

[ ] Has tests (if omitted, state reason in description)
[ ] Rebased on top of main at time of PR
[ ] Merged related skia PRs
[ ] Changes adhere to coding standard
[ ] Updated documentation

@mattleibow will it be possible to backport these changes to 2.x? Thanks!

Thanks! Backported!


Tag StatisticalRethinking 2.1.2

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/10620
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v2.1.2 -m "<description of version>" d1973240fbfce6468e12ba8f8d9e58a6c0098bfd
git push origin v2.1.2


flutter

Hi @nob356
what is the issue you are facing?
can please provide flutter run -v, flutter doctor -v, and minimal reproduction code?
thanks.

Hi @nob356,
Please don't create issues without any reason
https://github.com/flutter/flutter/issues/67719
https://github.com/flutter/flutter/issues/67717
https://github.com/flutter/flutter/issues/67377
When opening an issue, please try to provide all or some of the information indicated in the template.

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

Closing this issue, since no information has been provided.
Thank you.


Some parts of the content not synced

When there is  content in a page that occurs before the first header, all content up to and including that first header is not retrieved from BookStack. For example, given the following test page content:
*Content* before header, and span styling

### Header

Content after header.

Only the following is actually stored in the file when downloaded / updated to the client:
Content after header.

And when I add a header before that  content like so:
##### Some header of any level

*Content* before header, and span styling

### Header

Content.

The result is:
*Content* before header, and span styling

### Header

Content.

I'm not 100% sure whether it's the span tags causing this behavior, or perhaps anything before and including the first header being skipped by default (by the script and/or the api). However, on a page that doesn't include any span content before the first header, it syncs as expected, including all content of the page.
I'm also noticing that when a span before the first header (or maybe anywhere in the page?) was added - causing the behavior described above - this behavior persists for that page even when any formatting that would trigger span classes has been removed and synced again. This makes me wonder whether it is caused by BookStack switching the page from markdown formatting to html (ie its editor options), but the resulting file does format as markdown - just with anything before the first header on the page missing.
Also, a minor issue: I noticed that while BookStack allows for pages with the same title in the same location, these duplicates are not synced. For example, if I have shelf/book/page-title twice (so same 'page-title'), but with different page contents, only one of these gets synced - it seems the first created page.

To be sure: it seems to me that a page like this does sync the complete page, as long as there were never any span elements:
Some *markdown* formatted content that **doesn't** require html formatting like text color.

## First page header
Some content.


Missing Genome in matches.csv

Hi MEDI developers,
Thank you for making such a cool tool! I was exploring it several months back and was able to generate a database and results without issue.
I am now rebuilding the database for various reasons, and it seems that one of the genomes, GCA_036512215.2_SLM_r2.1, was perhaps removed or modified recently. When I visit: https://ftp.ncbi.nlm.nih.gov/genomes/all/GCA/036/512/215/GCA_036512215.2_SLM_r2.1/, there are no genomic.fna.gz files, which causes an error (I found this by having the download fail and then visiting .command.log).
I wanted to let you know in case there's something you need to do for the database. For now, I'll remove it from matches.csv and see if that is acceptable to the Nextflow pipeline.
--Katie

Indeed it is back! I'll re-run that part now too... I have been doing a lot of investigating since I wrote last week, as I am actively having trouble with the other kraken databases for similar reasons (prokaryote, even!). I get errors for different genomes that are missing every time I run build_kraken.nf (no --resume option), because the assembly_summary that kraken downloads would still include missing genomes (that would disappear or reappear).
I've written to NCBI about whether there are maintenance windows that we can avoid, since the instability can be quite problematic at times, really for anyone trying to build a database.

Yes I agree. In the initial version MEDI would just silently ignore genomes that couldn't be downloaded but the caveat was that it would build a different database every run which sometimes would miss an entire food item entirely.  So that did not seem too reproducible either.
The best solution would be to provide the database for download but we haven't found a repository that allows you to deposit 500GB. Open for suggestions here.
But it is a priority for us and we are working on it.


About special tokens

When I use vllm and fastchat by chatglm3-6b, model always generate "assistant" special tokens, how can i fix it?
like:
"春风轻拂绿意新，|assistant|助时光荏苒。\n|assistant|勤勉不懈怠，|assistant|智慧照万方。"

chang to another LLM model without chat fine tuning， baichuan-2 may be suit for you