GPU A100 Output Random Code

System Config:
LLM: 13b int4 version
A100 GPU, set "sm_80" for compute capability
Ubuntu 20.04, cuda version 12.2, driver version 535.104.12
Need help on this one, thanks!
Alex
TinyChatEngine by MIT HAN Lab: https://github.com/mit-han-lab/TinyChatEngine
Using model: 13b
Using AWQ for 4bit quantization: https://github.com/mit-han-lab/llm-awq
Loading model... Finished!
USER: mit
ASSISTANT:

$  #
#" ⁇ $
Xshel$!!$
Xshell ⁇ Xshell"
"!!" Xshell	       !
$
Xshell !Xshell XshellXshell!    #Xshel#
!
$	!$$
"##!Xshell⁇  ⁇      	$ ⁇
    $"!" ⁇ Xshell	#
    #                                                                                "Xshell

                                                                                $ ⁇

$
"# ⁇  ⁇ ##                                                                                #!"!"
$!"!""
Inference latency, Total time: 10.2 s, 18.6 ms/token, 53.7 token/s, 548 tokens

Hi @tuobulatuo, thanks for your interest in our work. Sorry that I can't replicate your issue firsthand as I don't have an A100 GPU on hand. That being said, we've just released the new version of our CUDA implementation. As mentioned in #58, the new version has been tested on various GPUs with compute capability from 6.1 to 8.9. Therefore, we suppose it should work on A100 now, but we can't guarantee it as we haven't been able to test it directly. Please feel free to try it out and give us feedback.
I hope this helps. Thanks!

I am experiencing the same issues when running the application on the A30. I attempted to resolve the issue by following the instructions in issue #58, but I am still receiving the above bug, do you have the solution for it yet @tuobulatuo. Thank you.


I can't play this mode with lunar client

Title is all.

Your screenshot doesn't seem to be uploaded correctly? Try again plz


Adding a number of fixes and new features

New features include:

User accounts/registration
LDAP support
Public/private profiles

Adding user account support, LDAP auth, public/private profiles plus a number of fixes.


DUTS-TR-Edge

DUTS-TR-Edge，How do I get this data set？

https://github.com/backseason/PoolNet

DUTS-TR-Edge，How do I get this data set？

https://github.com/backseason/PoolNet


Headless Events list needs work

The 11ty rendering of sorted content needs to render.
What's done so far:

[X] Fix the data structure & send it to the API
[X] Set up a development pipeline so we can test this without effecting our production release
[X] Refactor the WP plugin to generate a new Event Post List that can be structurally different from the original post list, which will make the 11ty sorting much easier.

Issues with 11ty - The Event layout has a slightly different template from press releases.
Think we need:

events.njk & events collection
new way of doing templates for posts & events.

Done - awaiting QA

It's not in design system yet.

Work was done and launched.
The DS is continuing to evolve and this will move into consideration as a "Pattern"
Closing & adding issue to list of "Patterns" for reference.


e2e: fix flaky test

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3755
[x] add cypress-network-idle. make sure to run next steps when the network completed the requests
[x] adjust viewportWidth
[x] adjust defaultCommandTimeout to 10s
[x] adjust retries to 3x

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged the recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 50 usd

pay 50

😍 Perfect, I’ve sent the payout
💵 $50 @ 54.64 USD/KSM ~ 0.915 $KSM
🧗 DY4SQF2iD456tH89aQtz5wv1EV3BbSW8wKKuMcwbmXaj1pM
🔗 0xee545bb36386b18db33159060acfa1e4e7b499104c398df0dcaeeb92ae66598d
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Think this would work for Foundry?

@DJWoodZ do you think this bot would work for Foundry too?  (Close to the same structure etc.)

Hi @TheG33kDev. This bot will only work with the Satisfactory dedicated server. A similar bot could possibly be created for the Foundry dedicated server however. I do not own Foundry, but I will look into it.


Revisit collapse shortcuts

VSCode Version: 1.49.insiders
OS Version: Windows 10

Steps to Reproduce:

Open an ipynb notebook
Collapse a cell using the context menu
Attempt to do the same with the keyboard shortcut ('C C').

Does not seem to work. Maybe I'm not understanding what C C means though. (I'm holding down SHIFT and hitting C twice).

Does this issue occur when all extensions are disabled?: N/A

No shift. Just c c. I'm not sure about this keybinding though, I think we should come up with a new one at some point. But I understand that Jupyter doesn't have a default keybinding for this.

These shortcuts do work for me - but there is screen cheese when collapsing output

There are a few issues with showing outputs right now that need some work

I'm adding the keybinding that executes the command 'notebook.cell.collapseCellContent' to the python ext. And it takes it as copy twice, I can make it work by using another keybinding, like 'S C' for example.

I'm adding the keybinding that executes the command 'notebook.cell.collapseCellContent' to the python ext.

We can fix the keybinding in vscode. I don't really care what it is, jupyter has no precedent from what I understand. We just have to pick one and change it. Would rather have a good common set of keybindings in vscode than have extensions override everything.

Changed to:

Collapse cell input: cmd+K cmd+C
Collapse cell output: cmd+K T

These are ctrl on windows


Error : 'System.AccessViolationException' occurred in InteropRuntimeImplementer.TessApiSignaturesInstance

While trying to run Samples with attached image. i am getting following error:
An unhandled exception of type 'System.AccessViolationException' occurred in InteropRuntimeImplementer.TessApiSignaturesInstance
Could please help me? Screen shot attached

Thanks,
Satish

Opps I forgot to update the Samples to the new Tesseract release (3.0). I'll have a look at doing this tomorrow.

If you've upgraded to 3.0, then it could be due to issue #232. I'll try and put together a fix tomorrow morning.

The AccessViolationException occurs not only x64, but also x86 version in 3.0.0 when I perform AnalyzeLayout(); method in 50% probability. @charlesw did you figure it out also?

Yes, I have noticed that some of the time I'm getting AccessViolationException  as well. I'll have a look but for the meantime I'd strongly suggest using version 2.4.1 instead.

Always appreciate your effort. thanks. arigatou.

Just rebuilt the native tesseract libraries and re-added them (ab4d5063c4). Seems to be working for me now however would appreciate it if someone else could also verify that it's working for them too.

Note to use the samples in dev you'll need to build local copy of the Tesseract nuget package (3.0.1) and place it in your nuget package path:

Execute build.bat to build the nuget package and copy it to ``C:\Nuget packages``` (or wherever you want to store your local nuget packages
From VS 2015 (note its a similar process in older versions):

Click Tools -> Options
Select Nuget Package Manager -> Package Sources
Click the Add button to add a new source and C:\Nuget packages for the source and a meaningful name and click Update

VS should now be able to find your copy of the Tesseract nuget package

Note: You can skip step 2 if you've already got this setup of course.

Did this ever get solved?  I am using 3.04 from NuGet package install.  I am receiving the same error:  System.AccessViolationException' occurred in InteropRuntimeImplementer.TessApiSignaturesInstance
when GetText() is called.
What version should I use if I build my own nuget package?

I'm guessing that the answer is no as you're still getting the error. I think that something is wrong with the native build of the tesseract library. Not sure what though.

I'm having the same problem as well.

I've split of the samples into the own repo (https://github.com/charlesw/tessseract-samples) to make it easier to maintain and so they can be improved independently of this wrapper.
I've also had a quick further look at this bug and unfortunately I cannot reliably reproduce it (just ran the console sample 20+ times and it didn't occur once). Can anyone else check this out and ideally provide an example more info on how to reproduce this?

@charlesw, I get the error every time without fail in the same way as @satishreddy81.  I started down the path of building all of the dependencies from scratch, but it didn't help.  This happens when I run any number of inputs, not just the samples not just on occasion.
I tried version 3.0.0 specifically as I had tried 3.0.1, all both fail in the same way.

Note, if I force the x86 platform it's intermittent.

Thanks for the extra info. I did do some more testing to see if I could
find a way to reproduce the issue reliable bit still no luck in my
environment. Does it also occur for you outside of vs?
On 11/01/2016 3:46 am, "Rodbourn"<EMAIL_ADDRESS>wrote:

Note, if I force the x86 platform it's intermittent.
—
Reply to this email directly or view it on GitHub
https://github.com/charlesw/tesseract/issues/231#issuecomment-170366355.

It does, unfortunately.  My work around for the moment is to force the platform to x86 and swallow the error if it happens.  With the x86 binaries it's much less common.

Quick update on this issue: Unfortunately I'm not making much headway here and still cannot reliably reproduce the issue. I'll look at doing a new build using VS 2015, thanks to the work by @tofutim, and hope that resolves the crash though I'm not confident it will given I have no idea what's causing this.
I'm off to Europe very shortly so may not be able to do until mid\late next week.

Hi, i'm currently having the same issue IF I specify a different language, but if I use "eng", no error occurs. BTW I happen to try "chi_sim" and "chi_tra" and both happens to have an AccessViolationError.

I've released a new prerelease (Tesseract.<IP_ADDRESS>-alpha1) on nuget if people can test it out and respond with if it works or not on there setup that would be great. Note that this release used vs2015 to build the dependencies so you'll need to ensure that the vs2015 runtime is installed. VS2015 or .Net 4.6 is not required.

Thanks charles, i'll definitely test this one out. best!

Hi Charles, good news for the chi_sim language, i'm now able to convert the document into text. This is a great progress for your <IP_ADDRESS> alpha1! I'm just not sure how to test the other languages. Hopefully someone from the group may help. Thanks a lot again, wonderful work!
For those who wish to test, please execute to your package manager -

Install-Package Tesseract -Version <IP_ADDRESS>-alpha1

Hi charlesw,
I had the same error but your prerelease solved the problem.
So the version "Tesseract.<IP_ADDRESS>-alpha1" works well on VS2015 with French data.
Thank you very much for the correction.

Great, I'll do another set of test runs and assuming all is good do a
proper release tomorrow evening (German time).
On Thu, Feb 11, 2016 at 4:42 PM, dotadam<EMAIL_ADDRESS>wrote:

Hi charlesw,
I had the same error but your prerelease solved the problem.
So the version "Tesseract.<IP_ADDRESS>-alpha1" works well on VS2015 with French
data.
Thank you very much for the correction.
—
Reply to this email directly or view it on GitHub
https://github.com/charlesw/tesseract/issues/231#issuecomment-182921883.

Version 3.0.2 has now been released which should hopefully have resolved this issue :)

Very nice, we also hope to see the home page of this project updated as well if you don't mind though this has nothing to do with a bug or any. Thanks again!

Hi Charles, i'm very sorry but right now when i'm trying to install the project, i'm having these.

Looks like something is wrong with your nuget environment. Have you tried
uninstalling & reinstalling the tesseract package.
On 17 Feb 2016 08:28, "kenneth lacoste"<EMAIL_ADDRESS>wrote:

Hi Charles, i'm very sorry but right now when i'm trying to install the
project, i'm having these.
[image: image]
https://cloud.githubusercontent.com/assets/10647176/13102616/133ddf5a-d58b-11e5-85a8-b5c94926e2f0.png
—
Reply to this email directly or view it on GitHub
https://github.com/charlesw/tesseract/issues/231#issuecomment-185073506.

that was actually a clean project..have you tried re-installing / installing the tesseract package? are you having any issues? because if not, then probably its due to my network security..

And, its ok now..hehe my apologies. :+1:

Hi i have the same issue with 3.2.0-alpha2 when using the GetText() method on x64. By the way i am doing this OCR stuff on parallel programming with .Net task(similar to multi threading). My guess is that it fails when GetText() is called within a multithreaded application like a windows service or any other parallel programming. Could you help me out with this?

I'm using 3.2.0 alpha x64 GetText() multi-thread without a problem, but you must only have one thread per engine.  So, when doing multi-thread create a different engine instance for each.

Hours after I said it was all working we reproduced the problem.

Seems to be Page.Recognize().  Another place I see it is here.  I don't see ResultRenderer class on the GitHub version of the code.  Is this stack wonky or is there a different code set?
System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
at InteropRuntimeImplementer.TessApiSignaturesInstance.TessApiSignaturesImplementation.BaseApiRecognize(HandleRef handle, HandleRef monitor)
at Tesseract.Page.Recognize()
at Tesseract.ResultRenderer.AddPage(Page page)

Check out the bevelop branch, master is only for released stable code.
On Sat, 18 Mar 2017, 09:06 Hintz<EMAIL_ADDRESS>wrote:

Seems to be Page.Recognize(). Another place I see it is here. I don't see
ResultRenderer class on the GitHub version of the code. Is this stack wonky
or is there a different code set?
System.AccessViolationException: Attempted to read or write protected
memory. This is often an indication that other memory is corrupt.
at
InteropRuntimeImplementer.TessApiSignaturesInstance.TessApiSignaturesImplementation.BaseApiRecognize(HandleRef
handle, HandleRef monitor)
at Tesseract.Page.Recognize()
at Tesseract.ResultRenderer.AddPage(Page page)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/231#issuecomment-287483110,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyJXeTP8blc7kLB_IXPKxRfIylheSks5rmwPTgaJpZM4G5H3F
.

Destructor should release unmanaged resources when used without dispose.  Would you like me to make a pull so you can review?

You can, but I'd be surprised if the resources aren't being released since
all classes that manage resources should implement the disposable pattern.
That being said you really shouldn't be relying on the destructor to free
unmanaged resources as you have absolutely no control over the order in
which they're freed and I wouldn't be surprised if you get
accessviolationexception or other unpredictable behaviour.
On Sat, 18 Mar 2017, 12:23 Hintz<EMAIL_ADDRESS>wrote:

Destructor should release unmanaged resources when used without dispose.
Would you like me to make a pull so you can review?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/231#issuecomment-287506610,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyG0fpw_sP9zTMDvfrdx7j_mq6MHVks5rmzH-gaJpZM4G5H3F
.

It isn't the access violation cause.  I just noticed. It only matters for those not as careful.

I can't push the change.  Was I supposed to fork and then make the change?

Yes that's how it normally works. If you just cloned the repo just fork and
change the origin. The exact command/process can be found on stack
overflow.
BTW what class are you looking at changing? As i said all should implement
the disposable pattern and should therefore be cleaning up after
themselves, with the previously mentioned caveats.
On Sat, 18 Mar 2017, 14:02 Hintz<EMAIL_ADDRESS>wrote:

I can't push the change. Was I supposed to fork and then make the change?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/231#issuecomment-287511927,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyEKYB9NfChvuO_YWy_9UgId1YWpLks5rm0k2gaJpZM4G5H3F
.

Page.cs from
protected override void Dispose(bool disposing)
{
if (disposing) {
Interop.TessApi.Native.BaseAPIClear(Engine.Handle);
}
}
to
    protected override void Dispose(bool disposing)
    {
        if (! IsDiposed) {
            Interop.TessApi.Native.BaseAPIClear(Engine.Handle);
        }
    }

I ran a stress test on  a memory limited machine and experienced a similar problem when the machine exhausted memory.  I repeated the test with 10,000 pages on 2 threads on a dual core with enough memory and did not experience the symptom.

@charlesw can you please look into below issue:
https://github.com/tesseract-ocr/tesseract/issues/2680
It's same issue but, Tesseract version: <IP_ADDRESS>-beta3
Thanks!


Single Dashboard for multiple stacks

It would be great to have a single dashboard for a whole app. Right now it seems like if I pass a watchful instance from one stack to another I get a circular dependency
      throw new Error(`'${target.node.path}' depends on '${this.node.path}' (${cycle.join(', ')}). Adding this dependency (${reason}) would create a cyclic reference.`);
            ^
Error: 'Stack1' depends on 'Stack2' (Stack1 -> Stack2/myLambda/Resource.Ref, Stack1 -> Stack2/myLambdalLambdaTimeoutAlarm/Resource.Arn). Adding this dependency (Stack2 -> Stack1/alarmTopic/Resource.Ref) would create a cyclic reference.

What is the particular action that you're taking that causes a cyclical dependency? Seems like you may be subscribing a resource in Stack2 to an alarm created by watchful?


[1pt] Add unit tests for all methods  (Team member #1)

Add unit tests for all implemented methods.

Hello @Zuhaib042,
Please review my pull request


Use Python launcher instead of python3

Currently enjarify.bat calls python3, but Python installer doesn't install python3.exe. According to PEP 397, the preferred way is to use Python launcher.

It sounds like the launcher scripts were a mistake in the first place. What do you recommend? Should I just get rid of the scripts and add a shebang line?

Did you mean that you would like to add another Python script to replace enjarify.bat? Or to add the shebang to enjarify/main.py?
I'm not sure what's the best way to do so. If we keep the batch file, then simply replace python3 with py -3 (without changing any existing option) will do the trick. If we instead create a Python script, then I think there are more than one way to do it.
Actually I'm trying packaging enjarify with Chocolatey, a package manager in Windows. That is easier than download from GitHub, since this package is not on PyPI (right?).

Closed by Storyyeller/enjarify#10


Add support for BrowserBack and BrowserForward

VSCode Version: 1.13.0
OS Version: amd64 Debian 8

Steps to Reproduce:

Have ThinkPad laptop with the next/previous close to the cursor keys
Have custom xmodmapings to map them to F14 and F15 keys

keycode 166=F14
keycode 167=F15

Try to assign these keys to some functions (I get "unknown" instead of naming the shortcut and I can't assign it to the action)

I think the "unknown" message is the KeyCode.Unknown defined in vs/base/common/keyCodes. But I'm not sure what logic is setting it up, could it be some mismatch with scan codes?

@truhlikfredy Could you please try our Keybinding troubleshooting guide -- https://github.com/Microsoft/vscode/wiki/Keybindings#troubleshoot-linux-keybindings

Solved it with workardound in the settings.

"keyboard.dispatch": "keyCode"

But if it's reading original scanCodes, then it should work as well, originally the key are multimedia back / forward (code 166 and 167). Often browsers react to it going to previous website, and actually that was the reason why i remapped it, because I didn't like the original behavior applications had with these keys.

Let's continue tracking in #23991 in this case.

#23991 case is for remaps, but in my case it's 2 things, if it would accept the original keycode even ignoring the remaping would be grand as well. If I would remove the remappings, still the visual studio doesn't recognize back/forward keys.

I think we recognize F keys up to F19.

Yes, but that is the mapped value, original is this two keys
https://c1.staticflickr.com/8/7277/6863731296_af6a4baf78_b.jpg

I can look into adding support in VSCode for them. Can you please press them at https://jsfiddle.net/v1gz5c5m/1/ in Chromium and let me know what values each one has.

This are the original keys
keydown --- code: BrowserForward, keyCode: 167, key: BrowserForward
keydown --- code: BrowserBack, keyCode: 166, key: BrowserBack

Can you remove the duplicate tag? Even if this problem is bit overlapping with other issues, but still this one is unique.

How exactly did you manage to work around this issue? I don't see any way in keybindings.json to specify a keyCode or code and the keybindings editor shows unknown.
The current keybinding design in vscode seems limiting. Why doesn't it just support the names from "code" when keyboard.dispatch=code mode?
Changing keyboard.dispatch to keyCode didn't seem to make a difference for me.
Somewhat related issue: #8641, the same issue for the mouse button equivalent (which I don't believe send keycodes on Linux).

Yes that what I said it works with me because I used xmodmap to map them to F14 and F15 and then added this to vscode settings:

"keyboard.dispatch": "keyCode"

Just because I have workaround it doesn't mean it's not an issue so that's why I asked to remove the "duplicate tag" etc...
I looked at the source code and the keyboard management seems too complicated without much gain. Plus now it doesn't respect the remaps which is probably the first application I have seen and has issues with this.
To debug what you get from your linux I like to use

xev

It shows exactly what events you are emitting (keyboard, mouse etc...)

Keyboard management is complicated because it needs to work correctly in:

VSCode on Windows (based on Chromium + native-keymap to present correct labels to the UI) -- uses keyCode to act the same as all other Windows applications
VSCode on Linux (based on Chromium + native-keymap to bind our default US based keybindings to something different depending on the keyboard layout and to present correct labels in the UI) -- uses code by default
VSCode on OSX (based on Chromium + native-keymap to bind our default US based keybindings to something different depending on the keyboard layout and to present correct labels in the UI) -- uses code by default
Monaco Editor on Chrome on Windows/OSX/Linux -- uses keyCode
Monaco Editor on Firefox on Windows/OSX/Linux -- uses keyCode
Monaco Editor on IE11 and Edge on Windows -- uses keyCode
Monaco Editor on Safari on OSX -- uses keyCode

Wow, that is impresive, didn't know the Monaco Editor is based on this as well, then not really surprised of the over-complications. It needs to be universal and consider many different combinations.

Just to say I also have this exact issue.
My keyboard has standalone keys for browser Back and browser forward, that I wanted to use for code folding, but they come up as "unknown" in vscode.
I used the workaround described by @truhlikfredy to assign my keys for F13 & F14.
For anybody else facing the same (on Ubuntu), here is the workaround:

In vscode open key bindings (ctrl P & "leyboard shortcuts"),
click the "record keys" button the right of the input box
press your key. it will sau "unknown" in the box, but if you hover over that word ift will say something like "code: BrowserBack", keyCode:166, key: "
in terminal run: xmodmap -pke > ~/xmodmap_original : this backup up your current keymap.
in terminal run: echo "keycode 166 = F13 F13 F13" => ~/.Xmodmap : this maps your new key binding
(for good measure) run xmodmap ~/.Xmodmap
In vscode open settings (ctrl P & "settings", choose "open settings JSON") & add "keyboard.dispatch": "keyCode"
youshould now be able to go back to the vscode key bindings editor & use your new key

It stopped working on me recently as it looks like vscode update reset some settings. I forgot I was making this issue and by accident googled my own solution, in this case the "keyboard.dispatch" got reset to default value so I had to set it back to the "keyCode" value.
I have modified other editors, even rebuild chromium from scratch to support the F13/F14 as previous and next tabs, physically these buttons are located close to the cursor arrows and it's very pleasant to have editors/browsers setup so the whole UI is consistent and productive.


feat: added copy functionality from #152

hello :) this is my first PR to an opensource tech! all crit feedback is appreciated

implemented optional clipboard output flag
added related test cases
updated pkg dependecnies to follow a clipboard-based package
modified config types to support new style

note: linted + tested - please test on different systems (I'm running MacOS)

Hi, @vznh!
Thank you for this great PR! This is a very nice first contribution.
The feature looks almost perfect and the code looks good. I've left some comments for minor improvements.
Could you please take a look at them?
Looking forward to merging this feature!


MOD:remove custom patched k8s.io/client-go/tools/portforward version and use k8s native

this pr delete the custom github.com/txn2/kubefwd/pkg/portforward package and use k8s.io/client-go/tools/portforwardinstead.
This simplifies the code and avoids meaningless errors.The following is a detailed explanation and reasons

"github.com/txn2/kubefwd/pkg/portforward" package is a custom patched version of k8s.io/client-go/tools/portforward.
https://github.com/txn2/kubefwd/blob/68259d219c4a0d086085f0959f47e805123e417f/pkg/portforward/portforward.go#L18-L21
The difference from the original is the addition of localip. in custom pkg we get the localip translate to ipv4 & ipv6 and listenOnPortAndAddress.
https://github.com/txn2/kubefwd/blob/68259d219c4a0d086085f0959f47e805123e417f/pkg/portforward/portforward.go#L198-L214
but the k8s native "k8s.io/client-go/tools/portforward" pkg has function NewOnAddresses, This method can accept the address([]string) as a parameter.
func NewOnAddresses(dialer httpstream.Dialer, addresses []string, ports []string, stopChan <-chan struct{}, readyChan chan struct{}, out, errOut io.Writer) (*PortForwarder, error) {
	if len(addresses) == 0 {
		return nil, errors.New("you must specify at least 1 address")
	}
	parsedAddresses, err := parseAddresses(addresses)
	if err != nil {
		return nil, err
	}
	if len(ports) == 0 {
		return nil, errors.New("you must specify at least 1 port")
	}
	parsedPorts, err := parsePorts(ports)
	if err != nil {
		return nil, err
	}
	return &PortForwarder{
		dialer:    dialer,
		addresses: parsedAddresses,
		ports:     parsedPorts,
		stopChan:  stopChan,
		Ready:     readyChan,
		out:       out,
		errOut:    errOut,
	}, nil
}

and in parseAddresses, the method can parse string address to []listenAddress.
func parseAddresses(addressesToParse []string) ([]listenAddress, error) {
	var addresses []listenAddress
	parsed := make(map[string]listenAddress)
	for _, address := range addressesToParse {
		if address == "localhost" {
			if _, exists := parsed["<IP_ADDRESS>"]; !exists {
				ip := listenAddress{address: "<IP_ADDRESS>", protocol: "tcp4", failureMode: "all"}
				parsed[ip.address] = ip
			}
			if _, exists := parsed["::1"]; !exists {
				ip := listenAddress{address: "::1", protocol: "tcp6", failureMode: "all"}
				parsed[ip.address] = ip
			}
		} else if net.ParseIP(address).To4() != nil {
			parsed[address] = listenAddress{address: address, protocol: "tcp4", failureMode: "any"}
		} else if net.ParseIP(address) != nil {
			parsed[address] = listenAddress{address: address, protocol: "tcp6", failureMode: "any"}
		} else {
			return nil, fmt.Errorf("%s is not a valid IP", address)
		}
	}
	addresses = make([]listenAddress, len(parsed))
	id := 0
	for _, v := range parsed {
		addresses[id] = v
		id++
	}
	// Sort addresses before returning to get a stable order
	sort.Slice(addresses, func(i, j int) bool { return addresses[i].address < addresses[j].address })

	return addresses, nil
}

So we don't need to maintain a custom portforward package, just use the k8s portforward package is well.

test in k8s v1.16.0

any question pls let me know. thanks.

@calmkart I never liked having to hack around the client-go. Looks like NewOnAddresses was added not long after my hack and I never noticed it. This is a great contribution thank you!


Add spatie/laravel-translation-loader

Some of the packages do not conform to these guidelines: https://github.com/chiraggude/awesome-laravel/blob/master/CONTRIBUTING.md#must-haves-for-packages


[5.4][Runtime] Fix StableAddressConcurrentReadableHashMap threading crash.

Cherry-pick https://github.com/apple/swift/pull/35185 to 5.4.
StableAddressConcurrentReadableHashMap::getOrInsert had a race condition in the first lookup, where the snapshot was destroyed before the pointer was extracted from the returned wrapper. Fix this by creating the snapshot outside the if so that it stays alive.
rdar://problem/71932487

@swift-ci please test

@swift-ci please test windows platform

Sneaky. I wonder if the snapshot's API could all be turned into & methods, so that they're impossible to accidentally use on a temporary snapshot, e.g.:
struct Butt {
  void foo() &;
};

void butt() {
  Butt().foo(); // an error, because the method requires an lvalue but is given a temporary
}


Add jobs to the Person class

Adds a hashmap of Jobs linked to each Person; calculates salary based on the status of the Job at the time of being added.

Codecov Report

Merging #78 (dbd7856) into master (ed6bcd5) will increase coverage by 0.71%.
The diff coverage is 91.48%.

@@             Coverage Diff              @@
##             master      #78      +/-   ##
============================================
+ Coverage     69.57%   70.28%   +0.71%
- Complexity      450      464      +14
============================================
  Files            76       77       +1
  Lines          1374     1417      +43
  Branches        151      159       +8
============================================
+ Hits            956      996      +40
  Misses          384      384
- Partials         34       37       +3

Impacted Files
Coverage Δ

src/main/java/peoplesoft/model/person/Person.java
95.06% <90.24%> (-4.94%)
:arrow_down:

src/main/java/peoplesoft/model/job/Job.java
84.21% <100.00%> (+0.87%)
:arrow_up:

.../model/job/exceptions/IllegalPaymentException.java
100.00% <100.00%> (ø)

src/main/java/peoplesoft/model/job/Money.java
95.23% <0.00%> (+4.76%)
:arrow_up:

Continue to review full report at Codecov.

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


Not able to move around the hoverMesh in Editor

Not sure why , but the hoverMesh can only move along one axis in Editor. (Master branch) .
It works however in the public example.
Tested in both Chrome and FF browsers.

changed the  z coordinate to y for to the editorWorldPos in Input.update and the hoverMesh started moving around properly.
if (hit) {// flip things around a little to fit to our rotated grid
				this.editorWorldPos.x = hit.point.x;
				this.editorWorldPos.y = hit.point.y; // grids are rotated on axis so that objects keep their y+ orientation
				this.editorWorldPos.z = hit.point.z;
			}

Thanks, I updated it in the master branch. I didn't update the build though, so you'll have to run gulp. You might want to check out the dev branch since it's a bit cleaner code. Although I'm not sure how stable it is, but it's the API that will be used eventually, if you're worried about forward compatibility.


Inconsistent questions about entry date

Continuous residence affidavit - using users[i].last_entry_date_month and users[i].last_entry_date_year (from screener)
should be users[i]..date_of_last_entry (from id: U.S. entry info and travel documents) - this is what shows on I-821.
I think we should:

Use just one question for this
Make it ask for the entry date using 3 parts, with the day being optional

Also see this feedback:
Statement regarding Continuous Residence Addendum -- it's filling in the MONTH YEAR as entry date instead of the Actual Date of Entry

I think this was somewhat inconsistent feedback about whether we always need to ask for the day or just use month/year (because applicant won't know the exact day). but we should add a new 3-part question using a grid for each component as described above, and logic to show the day when it's defined only


Where is the download for Linux?

I reckon it's not very hard to just compile it for Linux

Unfortunately, one of the plugins we use does not support linux.

Duplicate of #1. Please refrain from opening issues that already exist, instead, make a comment on the already-existing issue. Also, there were only 2 issues before you made yours lmao, hardly a chore to check. Maybe you weren't aware that it's standard not to open duplicate issues, that's fine, just please refrain from doing so in the future, in any repository, thanks.


Reporting

Please don't worry about this until AFTER your vacation. Please. We don't want to rush this, and we are so close. I just did the math, to get these all annotated, it's going to cost a ton anyway, and I need to wait until the Fall to get the early fall to get the extra money.
So, right now this survey has 8 users that we're set to generate a report for:
http://surveyor.codeslick.com/survey/9/stats
So, each one saw 48 tweets so the minimum score for each user would be 48 (if they saw 48 tweets, and didn't annotate any of them). Yet, only A1XAOEDK0Y2KBT scored above 48. So something in the reporting must be off.

"minimum score" = sum of a user column in the reporting.

I don't see any scores that come to 48. The scores are normalized to 0.0 - 1.0. Is it still the case that you want minimum score to be the sum o fa user column? I'm not sure I quite understand what that is.

We're talking about two different things.
Download the CSV report for survey 9:
http://surveyor.codeslick.com/survey-report/9
If you open it in a spreadsheet editor, Column D (user ADVIE0ZHLWDDH) has 1s and 2s for each question. If we summed those up, the minimum score for each user would be 48 (if they saw 48 tweets, and didn't annotate any of them). Yet, the score is 45 for ADVIE0ZHLWDDH, 32 for Column E, and so on...

Not everything gets marked with 1s and 2s. There is also a 0 condition, which I think means that the user did not see that tweet. I don't remember - but I also don't know if that has any bearing on this.

Right, but a user sees a minimum of 48 tweets, so the min score should be 48.

what if a quota is filled? I'm not really sure that is the case here, but wouldn't that result in fewer tweets getting to the user?

It would. But that isn't the case here (Quota = 3) Tweets = 4,500. Total number of surveys to serve would be 13,000/45.

By chance, after three surveys we could have started to fill quotas. But no score was above 45. At least the first three should have been?

That logic seems sound to me. I'm sure there is still some issue with the way I am putting the report together. The data model design as I have it is not ideal for the way we are doing this. I don't want to change the data model, but I will dig into the reporting code and see what I can find.

remind me again what 0,1, and 2 indicate

0 = User did not see tweet
1 = User saw tweet, but did not "check"/annotated it
2 = User saw it, "checked"/annotated it
Chris J. Vargo
University of Alabama
Assistant Professor, Ph.D.
Advertising & Public Relations
[image: Twitter] http://www.twitter.com/chrisjvargo [image: LinkedIn]
http://www.linkedin.com/in/chrisjvargo [image: Google Plus]
https://plus.google.com/105471295838192826025 [image: WordPress]
http://www.chrisjvargo.com/
On Wed, Aug 26, 2015 at 5:09 PM, Scott Bradley<EMAIL_ADDRESS>wrote:

remind me again what 0,1, and 2 indicate
—
Reply to this email directly or view it on GitHub
https://github.com/scott2b/surveyor/issues/6#issuecomment-135191131.

Ok, it seems like the problem is not really the code, but that the server is timing out before the full csv is delivered. Because it is delivered as a stream, there is no error condition that results. The code may or may not be right. As is, it looks like the CSV is about 11M rather than about 200k, so it's a really big download. I might be able to extend the timeout on the server, but I'm not sure yet if that is the right solution. In the meantime, let's do a sanity check. I'm getting the following stats on the full CSV file:
133057 rows of data (7392 tweets * 9 annotations = 66,528, so there is a factor of 2x I haven't figured out yet)
sums for individual users are in the 500 range. Not sure if that is expected.

Ah wait, if I open the full csv in Excel, I get a correct number of rows of 66,528. There is a newline in the tweets or something that threw off my wc -l.
Ok, so # of rows looks good (I think).
~500 per user also seems justified: 48 tweets x 9 annotations (1 point for each view) + 48 (a second point for each tweet assuming a user gives a single annotation to each tweet) == 480
So, let me think about how to handle this long download.

Shot in the dark, maybe zipping it?
On Aug 26, 2015 8:06 PM, "Scott Bradley"<EMAIL_ADDRESS>wrote:

Ah wait, if I open the full csv in Excel, I get a correct number of rows
of 66,528. There is a newline in the tweets or something that threw off my
wc -l.
Ok, so # of rows looks good (I think).
~500 per user also seems justified: 48 tweets x 9 annotations (1 point for
each view) + 48 (a second point for each tweet assuming a user gives a
single annotation to each tweet) == 480
So, let me think about how to handle this long download.
—
Reply to this email directly or view it on GitHub
https://github.com/scott2b/surveyor/issues/6#issuecomment-135226405.

yeah, zipping will help in terms of the actual download itself, but it is not an immediately implementable solution. Problem is that the csv comes down as a stream. So you are actually downloading the rows of the csv as they are generated. There is a fair amount of heavy querying going on. This is why I did it this way. I actually thought this would prevent a timeout issue, because the stream should stay open until explicitly closed.
Even if I can fix this in the server configs this to keep the stream open, I'm not sure downloading an uncompressed 11M csv is the right thing. I think I will need to look into an async/offline approach. You click a button to generate the report, and somehow get a message when it is ready, with a link to the report. It's a fair amount of more work than just opening a stream and sending it back, but I think it is getting to where this is really the right way to do it.

I think we're good! Counts for the first user (ADVIE0ZHLWDDH):
0 66096
1 348
2 84
The counts add up to 432 (total annotations possible) / 9 (annotations per
question) = 48 (questions).
Works for A2S3MXU8T2I5H0 as well!
0 66096
1 354
2 78
Chris J. Vargo
University of Alabama
Assistant Professor, Ph.D.
Advertising & Public Relations
[image: Twitter] http://www.twitter.com/chrisjvargo [image: LinkedIn]
http://www.linkedin.com/in/chrisjvargo [image: Google Plus]
https://plus.google.com/105471295838192826025 [image: WordPress]
http://www.chrisjvargo.com/
On Wed, Aug 26, 2015 at 8:12 PM, Scott Bradley<EMAIL_ADDRESS>wrote:

yeah, zipping will help in terms of the actual download itself, but it is
not an immediately implementable solution. Problem is that the csv comes
down as a stream. So you are actually downloading the rows of the csv as
they are generated. There is a fair amount of heavy querying going on. This
is why I did it this way. I actually thought this would prevent a timeout
issue, because the stream should stay open until explicitly closed.
Even if I can fix this in the server configs this to keep the stream open,
I'm not sure downloading an uncompressed 11M csv is the right thing. I
think I will need to look into an async/offline approach. You click a
button to generate the report, and somehow get a message when it is ready,
with a link to the report. It's a fair amount of more work than just
opening a stream and sending it back, but I think it is getting to where
this is really the right way to do it.
—
Reply to this email directly or view it on GitHub
https://github.com/scott2b/surveyor/issues/6#issuecomment-135229766.


Histograms statistic sets sending the wrong max, min, and sum values

On logging several histograms to cloudwatch i was looking out for max values to match what i was seeing in my server logs. They were not the same. I investigated and looked at the stageMetricDatum method which is being used to convert both timer and histogram statistic sets. This method does a conversion by a factor depending on the time duration which makes sense for a timer but doesnt seem to make sense for max, min, sum over time as scaling by the duration shouldnt matter in this case. I added a simple test to show this, and a simple fix in the code base. I have a pull request to hopefully fix this if i understand the code correctly

Closing since it was addressed in: https://github.com/azagniotov/codahale-aggregated-metrics-cloudwatch-reporter/pull/6


Put misc. 3D tool visible instances on their own layer

This makes that visible stuff invisible to ReflectionProbes, whose preview in the editor shouldn't involve them.
Otherwise you get this in the editor (not a big deal, but neither desirable):

This code is generously donated by IMVU.

Can this be used to fix #8485 / #8140?

Good question, but I don't think so.

They are separate issues. Post-process effects are applied after everything is rendered to the viewport, by then any information on the render layer is already lost.
In order to keep gizmos unaffected by post-processing we would have to render the scene first, apply the effects, and then render the gizmos on top of that.

Thanks!


Well-formedness check

For user commands, add check_well_formedness, which looks for:

insufficient fee
too-big zkApps
zero vesting period in updates to account timing

Call that check when:

transactions are added to the pool
gossiped blocks are received
transactions are submitted via GraphQL

Create unit tests for the zero-vesting-period detection.
Closes #13264. Supersedes #13254.
Probably fixes #13249.

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet


Question about the operation parameters

Hello. Thanks for your excellent job.
I have read your paper and I have some confusion about the details.
You mentioned that you have three operation parameters [γba, γbl and (R, t)] which represent the joint angles, bone length, and camera location respectively. They are generated by the augmentor.
I wonder that whether the [γba, γbl] are both floating number or a vector? It seems unlikely to be vectors in mathematic expression since they're writen in lower case, but if they are floating numbers, are all the bone length and bone angles will be changed with the same value?

Hi, thanks for your interest in our work.
\gamma_{ba} and \gamma_{bl} are vectors. Different bones have different values.

Hi, thanks for your interest in our work.
\gamma_{ba} and \gamma_{bl} are vectors. Different bones have different values.

Thank you. I wonder when the code will be released?


missing mouse example gif

Readme

Raw

@maaslalani mouse isn't something we can script with VHS yet is it?

I believe the Pipe example does have a GIF https://github.com/charmbracelet/bubbletea/tree/master/examples/pipe
I've removed the README so that it is not missing an image which is consistent with the other examples that do not have GIFs currently.

Sounds good. Thank you!

Closing this for now as I think it's fine as-is. If people feel differently we can discuss and potentially re-open. Happy New Year!


Computercraft turtles can't fell trees

Simple issue: When a turtle performs a turtle.dig() on a dynamic trees trunk only the block the turtle targets gets destroyed, but the tree still stands. I am using CC-tweaked (from SquidDev), a maintained fork of CC.

This might be because the code is casting something to a player-related class, or using player-related events.

Or it may be an error in the way Turtle.dig() is made.

I always assumed that the computercraft turtle used a fake player.

They might just directly use World.

The use fake players so permissions work. I was wondering too, why that doesn't work.

public class TurtlePlayer extends FakePlayer {

Yep. But something is breaking somewhere.

So here's the problem:
https://github.com/dan200/ComputerCraft/blob/914df8b0c7cc28451ea56a53808024511ff52642/src/main/java/dan200/computercraft/shared/turtle/upgrades/TurtleTool.java#L290

So how should this be handled? Since it can also cause other issues with any code that uses World.

Might be worth trying now in the latest alpha on curseforge.


Refactor axes

Refactor the Axis object in Scan to make it a public object, integrate it better into the class. Axes now have track data reference stored internally, and will be reset when the selected tracks is reset.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 91.42857% with 6 lines in your changes missing coverage. Please review.

Project coverage is 89.39%. Comparing base (fabd5ff) to head (bd9922e).

Files with missing lines
Patch %
Lines

src/cr39py/scan/base_scan.py
91.42%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   89.72%   89.39%   -0.34%
==========================================
  Files          13       13
  Lines         808      839      +31
==========================================
+ Hits          725      750      +25
- Misses         83       89       +6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Inconsistency between Inference Results and Paper Examples and some failed cases in VITON-HD.

Thank you for your contribution once again. We have been working on a VTON (Virtual Try-On) project and have run your inference on VITON-HD unpaired test set. However, we have noticed some inconsistencies between the inference results and the examples shown in the paper.
In one of the examples, the generated neckline looks a little strange compared to the image appended in the paper, where it looks good. Below are the details of the example:

Image: 00654_00.jpg
Clothing item: 05838_00.jpg

Additionally, I followed the guidelines mentioned in the README.md file. The warped clothes were downloaded from the provided link:
https://github.com/bcmi/DCI-VTON-Virtual-Try-On/blob/20f75c60498ce1aeb5433b716f3b9cd5b4c71542/README.md?plain=1#L47
The checkpoint used was from the following link:
https://github.com/bcmi/DCI-VTON-Virtual-Try-On/blob/20f75c60498ce1aeb5433b716f3b9cd5b4c71542/README.md?plain=1#L72
The only changes I made were to the test.sh file:

I modified the checkpoint/data/output directories.
I decreased the n_samples from 8 to 1, as I only needed one inference due to time constraints. (It took us approximately 4 hours and 40 minutes to complete the inference on the unpaired test set using an A100 server.)

I also found some failed cases - When changing from short sleeves to long sleeves, the wrong shape is generated unexpectedly.

Image: 00071_00.jpg
Clothing item: 02151_00.jpg

I added some lines in test.py to save inputs of pipeline and it looks okay.

Could you please help me verify if the committed code is correct or if the uploaded checkpoints are correct? Any assistance would be greatly appreciated.

For the inconsistant results, I have checked it on my device, there seems to be no problem. May be caused by other environmental influences.

These failed cases are due to the inpainting mask selection. In subsequent experiments, we updated the inpainting mask to a form similar to HR-VTON. This approach can well alleviate this type of error.

(person, clothes, new version, old version)

Thanks for your efforts and reply.

Regarding the inconsistency, I will investigate further and try using another device.
As for the failed cases:

When changing from short sleeves to long sleeves, the wrong shape is generated unexpectedly.

Image: 00071_00.jpg
Clothing item: 02151_00.jpg

The inpainting mask could be a possible reason for this issue. We have also experimented with a more aggressive masking strategy, which has improved many problematic cases. However, there are still some failed cases, like the aforementioned ones (image=00071_00.jpg and clothes=02151_00.jpg). One possibility we suspect is an overfitting problem.
The unpaired images are from the test set, and theoretically, such issues should not occur. However, the VITON-HD dataset shares many models between the train and test datasets. In fact, based on our observations, each model generally has more than three images with different garments. It is possible that the training dataset includes many images where the model, such as the one in 00071_00.jpg, is wearing short sleeves. Consequently, during the 40-epoch training on UNet, which has over 800 million trainable parameters, the model overfits to this particular distribution and leads to the mentioned issue.

Overfitting may indeed be the cause of this error. However, in our previous experiments, as the number of training epochs increased, the performance of the model on the test set in the paired setting kept improving, so we ended up training for 40 epochs. Although we have not verified it carefully, there should be no overlapping data in the training set and test set, in which the clothes and person images are the same.

@wenhao728 can you tell me how were you able to run the model , because I'm running it on google colab but some errors are coming in the pytorch-lightning ,Please Help

Hi @24thTinyGiant. I managed to execute the code successfully on the laboratory server, and all appears to be functioning well. Could you please provide the error messages you're experiencing?
Furthermore, if you would like to open a new issue and outlining the problems you've faced, myself and the other contributors would be more than happy to assist you and anyone else who might be dealing with similar difficulties. :)
Here are my environment info using a build-in method in PyTorch
from torch.utils.collect_env import get_pretty_env_info

print(get_pretty_env_info())

Output:
PyTorch version: 1.11.0
Is debug build: False
CUDA used to build PyTorch: 11.3
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.5 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.31

Python version: 3.8.5 (default, Sep  4 2020, 07:30:14)  [GCC 7.3.0] (64-bit runtime)
Python platform: Linux-5.10.112-005.ali5000.alios7.x86_64-x86_64-with-glibc2.10
Is CUDA available: True
CUDA runtime version: 11.3.109
GPU models and configuration: GPU 0: NVIDIA A100-SXM4-80GB
Nvidia driver version: 470.199.02
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.2.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.2.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.2.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.2.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.2.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.2.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.2.0
HIP runtime version: N/A
MIOpen runtime version: N/A

Versions of relevant libraries:
[pip3] numpy==1.24.4
[pip3] pytorch-lightning==1.4.2
[pip3] torch==1.11.0
[pip3] torch-fidelity==0.3.0
[pip3] torchmetrics==0.6.0
[pip3] torchvision==0.12.0
[conda] blas                      1.0                         mkl
[conda] cudatoolkit               11.3.1               h2bc3f7f_2
[conda] ffmpeg                    4.3                  hf484d3e_0    pytorch
[conda] mkl                       2021.4.0           h06a4308_640
[conda] mkl-service               2.4.0            py38h7f8727e_0
[conda] mkl_fft                   1.3.1            py38hd3c417c_0
[conda] mkl_random                1.2.2            py38h51133e4_0
[conda] numpy                     1.24.4                   pypi_0    pypi
[conda] pytorch                   1.11.0          py3.8_cuda11.3_cudnn8.2.0_0    pytorch
[conda] pytorch-lightning         1.4.2                    pypi_0    pypi
[conda] pytorch-mutex             1.0                        cuda    pytorch
[conda] torch-fidelity            0.3.0                    pypi_0    pypi
[conda] torchmetrics              0.6.0                    pypi_0    pypi
[conda] torchvision               0.12.0               py38_cu113    pytorch

this is the error I'm getting in google colab everytime I'm testing the code:
/content/DCI-VTON-Virtual-Try-On
2023-12-04 06:57:16.348203: E tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:9342] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2023-12-04 06:57:16.348257: E tensorflow/compiler/xla/stream_executor/cuda/cuda_fft.cc:609] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2023-12-04 06:57:16.348305: E tensorflow/compiler/xla/stream_executor/cuda/cuda_blas.cc:1518] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-12-04 06:57:18.551409: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Global seed set to 23
Loading model from /content/DCI-VTON-Virtual-Try-On/viton512.ckpt
Global Step: 58240
LatentTryOnDiffusion: Running in eps-prediction mode
DiffusionWrapper has 859.54 M params.
making attention of type 'vanilla' with 512 in_channels
Working with z of shape (1, 4, 64, 64) = 16384 dimensions.
making attention of type 'vanilla' with 512 in_channels
^C
and after this the code stops, can you provide a solution to it ?

Thanks for your information. It looks like a compatibility issue between TensorFlow and cuda. If you want to start run the DCI-VTON method, you can create a new virtual environment following the instructions in README rather than use the default environment of colab. It works well for me.
You don't need to worry about the TensorFlow issue, as this repo depends on PyTorch.

For the TensorFlow issue, you may need to downgrade your package versions as in https://github.com/tensorflow/tensorflow/issues/62075#issuecomment-1808652131

okay , Thanks for your reply

@24thTinyGiant @wenhao728  i also got same error in colab did you fix it

The issue is due to the limited GPU Ram provided by the Colab  ,You need to use higher ran GPU for inference


Vulnerability check

PR Description

Scheduled Security Vulnerability Check.

Checklist

[x] I agree with the Code Contribution License Agreement in CONTRIBUTING.md

Closed, will be part of the https://github.com/spryker/docker-php/pull/77


Update class transform to fallback to react-create-class package

If React.createClass cannot be converted to a plain JavaScript class,
it falls back to using the react-create-class package instead.

Great work, thanks! LGTM, feel free to ping me if I can help when you run it internally


react-show v3.0.4 breaking change

react-show v3.0.0 beta used to not call .transition("clean"), but in v 3.0.4 calls .transition("clean") without a second argument. This sets the styles for tooltip-wrap to not have a left, top, height, width defined, and breaks the Tooltip object.
I would suggest updating package.json to point to the beta version (https://github.com/react-tools/react-show/commit/0081679bd8b963743f18c4498056fad99eb40f74#diff-1fdf421c05c1140f6d71444ea2b27638) or finding a way to update Animate so tooltips still work.

No longer applicable in v2


Cannot read properties of undefined (reading 'exportMode')

What is the location of your example repository?
No response
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
@shopify/hydrogen@latest
What version of Remix are you using?
No response
Steps to Reproduce
I create new application by running npm create @shopify/hydrogen@latest
Everything is good if i run pnpm run dev. but if i run pnpm run build. i got this error
Cannot read properties of undefined (reading 'exportMode')                                                                              │
│                                                                                │
│  To investigate the issue, examine this stack trace:                                                                                │
│    at getRenderedDependencies (.../node_modules/rollup/dist/es/shared/node-entry.js:16815)         │
│    at render (.../node_modules/rollup/dist/es/shared/node-entry.js:16366)                          │
│    at (.../node_modules/rollup/dist/es/shared/node-entry.js:18049)                                 │
│    at map                                                                                │
│    at renderChunks (.../node_modules/rollup/dist/es/shared/node-entry.js:18049)                    │
│    at generate (.../node_modules/rollup/dist/es/shared/node-entry.js:18294)                        │
│    at processTicksAndRejections (node:internal/process/task_queues:95)                                                                   │
│    at (.../node_modules/rollup/dist/es/shared/node-entry.js:20845)                                 │
│    at async catchUnfinishedHookActions                                                                                │
│    (.../node_modules/rollup/dist/es/shared/node-entry.js:20274)                                    │
│    at build (.../node_modules/vite/dist/node/chunks/dep-DyBnyoVI.js:65391)
Expected Behavior
Need to build success
Actual Behavior
getting error like description

Thanks for getting back to me.
I realize the problem because of tailwindcss. Event i just create new app
and do npm or pnpm run build. It will getting that error.
On Fri, 20 Sep 2024 at 22:56 Helen Lin @.***> wrote:

I cannot reproduce this on my end. Can you give me more information about
the environment you are executing this?

Operating system
node version
npm version
pnpm version

Does the build command works if you run it with npm command instead?
—
Reply to this email directly, view it on GitHub
https://github.com/Shopify/hydrogen/issues/2552#issuecomment-2364043411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJC4XRKCZX3LSQ5BYLHTOWLZXRAS5AVCNFSM6AAAAABORJPCQKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRUGA2DGNBRGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm generating project with tailwind option. I can't reproduce the error you are getting

Maybe this would help #2557

Not sure if this is the same issue as I was having, but try this.
pnpm add<EMAIL_ADDRESS>

Maybe this would help #2557

Not sure if this is the same issue as I was having, but try this.
pnpm add<EMAIL_ADDRESS>
This worked for me :) Thank you :)
Also as a side note: I tried this with (at the time of writing) latest tailwindcss alpha of<EMAIL_ADDRESS>and the build still fails even after performing the steps above. So for now, will keep it locked to 17 for some reason.

I think it worked up to alpha.21

Yes, it worked!
On Mon, 7 Oct 2024 at 03:09 Ingólfur Valsson @.***>
wrote:

Maybe this would help #2557
https://github.com/Shopify/hydrogen/issues/2557
Not sure if this is the same issue as I was having, but try this.
pnpm add @.***
This worked for me :) Thank you :)
Also as a side note: I tried this with (at the time of writing) latest
tailwindcss alpha of @.***, and the build still
fails even after performing the steps above. So for now, will keep it
locked to 17 for some reason.
I think it worked up to alpha.21
—
Reply to this email directly, view it on GitHub
https://github.com/Shopify/hydrogen/issues/2552#issuecomment-2395569145,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJC4XRKLTSQBVEOQCEGZFC3Z2GKIZAVCNFSM6AAAAABORJPCQKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGU3DSMJUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Accessibility Config: update eslint config to add eslint-plugin-jsx-a11y

Closing this as Console-lite is on hold indefinitely


Find a better way to figure out if the replica directory in the data path exists

Refer to https://github.com/longhorn/longhorn/issues/989
Right now Longhorn just relies on the command ls <data path>/replicas/ to check if the directory exists. If the command errors out, Longhorn simply considers the directory as not existing and tries to create it. See here  for details.
Actually, the error can be triggered by some other issues, e.g., the data path is not accessible.

The issue doesn't apply anymore after https://github.com/longhorn/longhorn-manager/commit/147bc6cfff391e63647ecc7959086c08da024e67


chore(cache) make the scope of the Docker cache more generalized

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


Make ADS timeout increase from 500 to 2000

sponge/3d4de79e-c5bd-4cb8-a8aa-23db2f18d1a2
Under MSAN, the timeout 500ms might be too short that XdsClient unable to complete several round-trip of ADS to create corresponding CDS representation. This PR increases it to 2000. This is longer than the wait in REQUESTED client status test which appears to never be flaky.

This change is

1000 MSAN run: https://source.cloud.google.com/results/invocations/3b87552b-3471-4e0c-aaa5-9233051d093a
This test only flake under MSAN, according to BigQuery.

I don't understand this explanation of the problem:

Under MSAN, the timeout 500ms might be too short that XdsClient unable to complete several round-trip of ADS to create corresponding CDS representation.

The timeout we're setting here is the time between when the XdsClient sends a request for a given resource and when it gives up waiting for a response to that request.  That's a single round-trip -- actually, just half of one.  Any earlier round-trips on the same ADS stream are irrelevant to that computation.  So what was actually causing the flakiness here, and why does increasing this timeout make it less flaky?

sponge/3d4de79e-c5bd-4cb8-a8aa-23db2f18d1a2
Based on the log, within 500 ms, the test case itself only able to complete half of the first ADS round-trip to fetch the Listener. Then the RPC times out. XdsClient doesn't got the time to ACK the response. That's why I think adding more timeout should solve this problem.
What you mentioned about the separation between ADS calls is true. This test case needs the XdsClient to subscribe to the resources first, if the ADS communication stops at LDS, the XdsClient won't ever subscribe to Clusters or Endpoints. According to the BigQuery result, only ClusterDoesNotExist and EndpointDoesNotExist has flaky problem.   Num CQs: 1, Min pollers: 1, Max Pollers: 2, CQ timeout (msec): 10000
I0330 15:25:22.832375122      19 xds_end2end_test.cc:2269]   Backend server startup complete
I0330 15:25:22.858123958      19 xds_end2end_test.cc:2257]   starting Admin server on port 4379
I0330 15:25:22.863084907     893 server_builder.cc:329]      Synchronous server. Num CQs: 1, Min pollers: 1, Max Pollers: 2, CQ timeout (msec): 10000
I0330 15:25:22.874261666      19 xds_end2end_test.cc:2269]   Admin server startup complete
I0330 15:25:22.884184494      19 xds_end2end_test.cc:566]    ADS[0x7110000260c0]: Unsetting type.googleapis.com/envoy.config.cluster.v3.Cluster resource cluster_name; resource_type_version now 2
E0330 15:25:23.468275302      36 xds_resolver.cc:813]        [xds_resolver 0x717000040500] received error from XdsClient: {"created":"@1617117923.467406832","description":"timeout obtaining resource {type=type.googleapis.com/envoy.config.listener.v3.Listener name=server.example.com} from xds server","file":"src/core/ext/xds/xds_client.cc","file_line":198,"grpc_status":14}
I0330 15:25:23.511529326      38 subchannel.cc:1064]         New connected subchannel at 0x70300002d870 for subchannel 0x71400000c800
I0330 15:25:23.529627683     890 xds_end2end_test.cc:703]    ADS[0x711000026108]: StreamAggregatedResources starts
I0330 15:25:23.565371538      25 subchannel.cc:1064]         New connected subchannel at 0x7030000286b0 for subchannel 0x714000034c00
I0330 15:25:23.579439447     890 xds_end2end_test.cc:747]    ADS[0x711000026108]: Received request for type type.googleapis.com/envoy.config.listener.v3.Listener with content node {
  id: "xds_end2end_test"
  cluster: "test"
  metadata {
    fields {
      key: "foo"
      value {
        string_value: "bar"
      }
    }
  }
  locality {
    region: "corp"
    zone: "svl"
    sub_zone: "mp3"
  }
  user_agent_name: "gRPC C-core linux"
  user_agent_version: "15.0.0"
  client_features: "envoy.lb.does_not_support_overprovisioning"
}
resource_names: "server.example.com"
type_url: "type.googleapis.com/envoy.config.listener.v3.Listener"

I0330 15:25:23.581291356     890 xds_end2end_test.cc:1076]   ADS[0x7110000260c0]: subscribe to resource type type.googleapis.com/envoy.config.listener.v3.Listener name server.example.com state 0x7f66792e5c00
I0330 15:25:23.583427086     890 xds_end2end_test.cc:896]    ADS[0x711000026108]: Sending update for type=type.googleapis.com/envoy.config.listener.v3.Listener name=server.example.com
I0330 15:25:23.640629318     890 xds_end2end_test.cc:760]    ADS[0x711000026108]: Sending response: version_info: "1"
resources {
  [type.googleapis.com/envoy.config.listener.v3.Listener] {
    name: "server.example.com"
    api_listener {
      api_listener {
        [type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager] {
          route_config {
            name: "route_config_name"
            virtual_hosts {
              domains: "*"
              routes {
                match {
                  prefix: ""
                }
                route {
                  cluster: "cluster_name"
                }
              }
            }
          }
          http_filters {
            name: "router"
            typed_config {
              [type.googleapis.com/envoy.extensions.filters.http.router.v3.Router] {
              }
            }
          }
        }
      }
    }
  }
}
type_url: "type.googleapis.com/envoy.config.listener.v3.Listener"
nonce: "1"

test/cpp/end2end/xds_end2end_test.cc:11137: Failure
Value of: csds_response.config(0).xds_config()
Expected: contains at least one element that is an object whose given property equals ClustersConfigDump
  Actual: { 40-byte object <60-D3 1B-92 66-7F 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 50-BE 00-00 70-70 00-00 00-00 00-00 03-00 00-00>, 40-byte object <60-D3 1B-92 66-7F 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 B0-15 03-00 50-70 00-00 00-00 00-00 06-00 00-00>, 40-byte object <60-D3 1B-92 66-7F 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-20 03-00 50-70 00-00 00-00 00-00 02-00 00-00>, 40-byte object <60-D3 1B-92 66-7F 00-00 00-00 00-00 00-00 00-00 00-00 00-00 30-70 00-00 50-20 03-00 50-70 00-00 00-00 00-00 04-00 00-00> }

Before the test start, the test case will unset two resource and then start the RPC. It seems by the time the RPC failed by XdsResolver, the ADS communication hasn't started. I wonder what happened to the 0.6 seconds here.
I0330 15:25:22.884184494      19 xds_end2end_test.cc:566]    ADS[0x7110000260c0]: Unsetting type.googleapis.com/envoy.config.cluster.v3.Cluster resource cluster_name; resource_type_version now 2
E0330 15:25:23.468275302      36 xds_resolver.cc:813]        [xds_resolver 0x717000040500] received error from XdsClient: {"created":"@1617117923.467406832","description":"timeout obtaining resource

MacOS build artifact: ERROR: Aborting VM command due to timeout of 9000 seconds,
Bazel C++ MacOS: this job run has no log at all, suspect Kokoro failure.


Update XmlDocumentation to raise 2 different diagnostic ids

Use CC0090 for incorrect params (params that exist in the docs but not on code) and CC0097 for missing parameters (params that exist on code, but not on docs).
Make 90 an Info, and 97 a Warning.

CC0090 duplicates compiler warning CS1572. Since the severity of this compiler warning can be configured in a rule set file and you can create a code fix which uses this ID, the CC-specific warning for it can be removed.

It will only show up when you ask for it using /doc, or change the ruleset file. We should keep it.


[Feature Request] Increase readability of the Article page

Screenshot

The former: hexo-theme-cupertino.  The later: Apple Newsroom.
Describe the proposal
It seems that the line spacing and the font size is too small and not suitable for reading. And I suggests increasing readability of the Article page. All the adjustments can be done simply by modifying the CSS files.
Hope you'll adopt my proposal. Thanks! :)

In addition, I love the Apple Newsroom's page margin. I hope you will imitate it.

49581a8 updated the line height.

There's another feature request that I'd love to add here.

Could you please modify the color of these links. I think gray might be a best choice.

There's another feature request that I'd love to add here.

Could you please modify the color of these links. I think gray might be a best choice.

019312e changed this.

Thanks for these modifications but I wonder if you can make the animation of these links the same as the links in the top bar.

Thanks for these modifications but I wonder if you can make the animation of these links the same as the links in the top bar.

Maybe not. Apple's site doesn't have any transition for the footer links, and it is not neccesary, so I won't add animation to these links. Sorry.


⚠ Bofrost.de has degraded performance

In 00d7dbc, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1212 ms

Resolved: Bofrost.de performance has improved in b81cdea.


Updating dependencies

Updating project to use rustls-tls with reqwest.

I don't feel very strongly about it, but for some context, the reason I end up doing this is for security purposes. Bugs and vulnerabilities happen and by having non-explicit versioning, you open yourself up to the following scenario:

Project uses dependency foo-1.0 which resolves to foo-1.0.3
foo-1.0.4 is released that has a vulnerability
Project build occurs which starts to automatically use foo-1.0.4

By having explicit versions you are ensuring that changes don't occur without you knowing it and you also have a record of when your project was using what versions. That papertrail can be really helpful if and when you need to perform an audit.

I figure, we better stick with majors for now, this is a small project, if we were developing axum itself, certainly minors are a better option.
Thanks anyway! :)


Backports don’t work (consistently)

#1017 is part of a milestone but didn’t trigger the meeseks bot. We need to

[ ] identify un-backported PRs:

#1015
#1017
TODO

[x] fix config error or whatever else blocks Mr Meeseeks

[ ] direct the bot to backport all those PRs

Looks like the previous mentions aren't working either:

https://github.com/scverse/anndata/pull/1017#issuecomment-1607318671

Missing dash in the 'on-merge' in the PR description for 0.9.2

@Carreau thank you!

Ahh... thanks for catching that!


Question about symmetric noise rates.

I recently read your paper and very interested in it. When I reading, I'm confused about how to implement "symmetric noise rates". Code of the paper just provides the path of symmetric noise rates JSON file but not provides the detail of it.
So could you share the code detail of add noise? Your help will be greatly appreciated.

Thanks a lot. The code could be found in Lines 190-220 of src/noisydataset.py.

Thanks for your reply. It's helpful.


multiple input video training possible?

Hello.
Is it possible to input more than one input video for training? like 3 videos of a person riding a bicycle to generalize the motion I want to get more variety and flexibility like the original paper.
If so what would the comfyui look like?

hi @SWAPv1
I think the current code supports this but the developer did not add an json example for this I do not know how to do this.
It would be great if the developer @kijai  added a json file for this process in the examples

You can load multiple videos with this node:

I've never actually trained anything with multiple videos, so I don't know more about the settings etc.

hi @kijai  Can you share the workflow please.
It would be nice if it was like the examples you uploaded because they are very good.
So it would be great if you could attach this file to increase the motion quality.
and Is it possible to enter Prompt for each video? For example, there are three videos of a person riding a bike, but I want to enter a different description for each video because it contains different information about each one.
This can produce higher quality motion output.

hi @kijai Can you share the workflow please. It would be nice if it was like the examples you uploaded because they are very good. So it would be great if you could attach this file to increase the motion quality.
and Is it possible to enter Prompt for each video? For example, there are three videos of a person riding a bike, but I want to enter a different description for each video because it contains different information about each one. This can produce higher quality motion output.

Not possibly to do different prompts right now. There's no workflow to share, it's the same workflow, but you just add that node I showed and different videos to it's inputs.

You can load multiple videos with this node:

I've never actually trained anything with multiple videos, so I don't know more about the settings etc.

Thank you I think this is what I am looking for.

hi @kijai Can you share the workflow please. It would be nice if it was like the examples you uploaded because they are very good. So it would be great if you could attach this file to increase the motion quality.
and Is it possible to enter Prompt for each video? For example, there are three videos of a person riding a bike, but I want to enter a different description for each video because it contains different information about each one. This can produce higher quality motion output.

I am guessing you would use the existing example and just add the multi image batch as the input. For the prompt I guess you would resort to a general action and input the same type of motion but from different camera angles or subjects. I believe they have some examples in the original developers page.


weex动画库支持

weex现在有没有一些支持的比较好的纯js动画库？或者简单移植过来就能用的？

This PR/issue doesn't received response since Weex migrated to Apache two years ago and this repository is no longer active yet.
Feel free to ask question in new repository and thanks for your contribution.

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


[minor] docstring fix (features.time)

Just adding a couple of missing arguments in the docstring of waveform_length

Codecov Report

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

@@           Coverage Diff           @@
##           master      #51   +/-   ##
=======================================
  Coverage   71.24%   71.24%
=======================================
  Files          18       18
  Lines        1137     1137
=======================================
  Hits          810      810
  Misses        327      327

Impacted Files
Coverage Δ

axopy/features/time.py
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Sorry for the delay, looks good to me.
Some time ago I played around with programatically generating parts of these docstrings because they share keyword arguments (something like seaborn). I can't remember if it was causing issues with the sphinx setup or something but it might be worth looking into that again.


Mempool Status and Lock updates, and small refactors

Current dependencies on/for this PR:

feature/proof-of-stake

PR #578

PR #647   👈

This comment was auto-generated by Graphite.


Labels should be added to mpl drawer

What is the expected enhancement?
Labels were added to text drawer in #4361, #4262, and #4420. Labels are user-supplied text that overwrites the names of gates and can be added to controls. This needs to be implemented for the mpl drawer. This issue is intended as a discussion to come to a consensus on design before we start coding.

The control labels can be placed on the highest control or below the lowest control if there are no controls above. And placed inside or outside the control line. In addition to what's shown here, some other possible questions would be,

If there are no params, should the original gate name be placed in the gate box underneath the gate label in smaller text? Might help prevent confusion if someone did YGate(label='X')
Or should the original gate name be placed in parens at the end of the label? This would work whether there are params or not and since most gate names are short would not take up too much space in most cases.
What do we do with gate labels for swap, cz, cu1, rzz, cx's and possibly others that don't have a box to display in? Should we use sidetext for these?
Currently all gate names are capitalized. Should all labels be capitalized or left as the user enters them?

One other thing. qc.cy(0, 1, label='Y Label') will always produce a control label. In the current implementation, the data structure is the same as
qc.append(YGate().control(1, label='Y Label'), [0, 1]).
If you want a gate label, you have to use a YGate form.

New mpl drawer version
@1uciano @ajavadia Over the last several weeks (stuck in lockdown) I started going through the mpl drawer code and discovered there were a lot of redundancies, inefficiencies, and unused code. I decided to see how I might be able to restructure the code and also wanted to see if I could implement the text width calculations and gate and control labels.
After restructuring and adding the features, I now have a version that incorporates all the open issues, including text widths to determine gate and layer widths for a proportional font, and labels. Even with adding these features, the code is 140 lines shorter than the current master version.
I would like to do a PR, and I know you prefer incremental changes, but I feel like incremental changes will not produce an efficient, easily maintained mpl drawer in a reasonable time. I have tested the code extensively with many matplotlib backends and many circuits. Someone should probably still take some time to go through what I've done in some detail in order to approve it.
So should I submit a PR, which would list in detail all the changes I made, or would you prefer to stick with the current master and try to bring it up to a better standard over a longer period of time?
Thanks.
Below are images from the current master and from my revised version. isometry and initialize are not in the master image since these crash the drawer.
Current master

New version


Safari iOS, images don't display if over the resource limit (and useCanvas =  true)

Preamble:
Images in Safari iOS are subject to resources limits
In essence, newer iOS devices have an image area limit of 5,242,880 (5 * 1024 * 1024)
Step to recreate issue:
• Make an image (PNG, JPG, etc) that is over the iOS resource limit, e.g. 2984px x 1757px = 5,242,888px
•  open this image in OpenSeadragon via a Legacy Image Pyramid - the image will not load:
{
  "tileSources": {
    "type": "legacy-image-pyramid",
    "levels": [{
      "url": "images/5242888.jpg",
      "width":  2984,
      "height": 1757
    }]
  }
}

• change useCanvas to false - the image will now load.

I'm not sure if it's the canvas limit or the image limit that is causing this issue. But the threshold image area required for it not to display is exactly 5,242,880.
If the same over-limit image is opened directly in the browser, it will be displayed, because the browser automatically reduces the size to bring it under the limit.
Perhaps this is not happening in OpenSeadragon because the image is being opened programatically by JS so it can be drawn to the canvas?
It is possible to work around this issue by setting useCanvas to false, perhaps by including an additional User Agent sniff in the $.supportsCanvas function. But this is quite unsatisfactory of course.
I'll continue to look into this issue, but thought I would log it here for the community also :)

Thanks @iangilman @monospaced for my solution was I detect the the iOs browser and then have a look to keep to reducing the dimensions of the image until it's under the threshold. I then make the request to the server which dynamically resizes the image. Not great but it least it works.

Thanks for the update!

I'm using OpenSeadragon for a museum collection and ran into this iOS isssue. I'm pulling the images from an external server and can't and won't copy and resize them on our server. I hoped that reducing the canvas size to below 5 megapixels in the tileSources would solve the issue, but as been noted here before it is reading the actual image file that causes the problem.
However, I did the maths for reducing the scale whilst keeping proportions and all that and thought I should share it if anybody wants to save themselves some headache.
    // if image bigger than the iOS limit of 5,242,880 px
    if(( $originalWidth * $originalHeight) > 5242880 ) {
        // divide the original size with the limit and get the square root
        $divisor = sqrt( ($originalWidth*$originalHeight) / 5242880 );
        // divide height and width with the divisor to get the new lengths
        // these two multiplied will make exactly 5,242,880
        $newHeight = (int)( $originalHeight / $divisor );
        $newWidth = (int)( $originalWidth / $divisor);
    }

Was there any progress on this bug? If not I am unsure how to make use of the library noted above or the snippet. Any assistance would be appreciated!

It doesn't look like there has been any progress (what you see here is what you get).
Is it possible for you to tile your images? If so, that's your easiest fix.
Another possibility might be to use the useCanvas: false option when creating your viewer. This may bypass the issue entirely, though the noncanvas viewer may have other issues.
Otherwise, if you're up for trying to fix the issue in OSD I'm happy to support your effort. The first step would be to figure out which strategy to try of the above mentioned ideas. I'm not sure what's best; maybe try a couple strategies and see which works. I'm thinking:

See about not storing the tiles in offscreen canvases but instead drawing the image directly to the onscreen canvas as needed. This would be a little bit like what we already do when there's no canvas support at all.
See about breaking the image into offscreen canvas tiles when it's first loaded and use that for the tile drawing. This would be similar to the pyramid building ImageTileSource does.

You'd want to do something relatively quick and dirty just to verify that one of those strategies solves the problem, and then refine after that.

@nmacd85 If you are only loading one image I suggest resizing it. As it's only one image hopefully you can do that statically. My solution was using IIP Server so I was able to resize on the fly. However we moved to using the deep zoom functionality (DZI) and this problem went away. Probably because each individual tile in DZI is well under the problematic threshold.

@terenceingram thanks for the reply, I was able to get DZI working also! I think I renamed something I shouldn't have originally which caused the issue and me looking for ways to use a single image. Luckily, all set! I haven't officially started development yet but I just need to get the overlays working as expected.

@nmacd85 we are very happy using DZI. It's worked a treat on a our historic pictures, digitised books, manuscripts and map collections. I originally was using the legacy image pyramid but once I hooked up DZI we were floored at the result. Here's an example of a panorama: http://nla.gov.au/nla.obj-151651939 and map: http://nla.gov.au/nla.obj-148371790. The bigger images are around 1 GB TiFF files. So for many images we have lots of detail. And some are huge, like a a panorama that is usualy a few metres wide. Using OSD together with IIP has been a massive win for us. We serve up over 1.3 million images using IIP (it's growing each week). Of those we use OSD to serve up around 280 000 images. The rest are served up using IIP into the internet archive's book reader as they are digitised books or ephemera that have been OCR'ed.
However the DZI technology has absolutely wowed our public users and stakeholders. We used the same digitised images - just converted into JPG2000, we digitised some of them 15 years ago, but the detail we can now show online is amazing.
Post up a URL to your application once it's complete so we can see what you have developed :)
@iangilman

@terenceingram That's wonderful to hear! I love what you guys are doing!

I'm having this issue with a project I'm currently working on: https://urbanomonte-4db4a.web.app/
Currently it will not work on Safari or Chrome on my Iphone.
Interesting findings:
My openseadragon div is set to be 100% width and 100vh height, covering the whole window. If I make it smaller it seems to work, and if I then go into fullscreen it continues working! I would prefer not to use the fullscreen mode, but I'm curious as to why it works...

@tommypreger
Looks good on my iPhone, tested on Safari, Chrome, and Edge

@msalsbery
Thanks for testing! I came here to edit the post. I did some changes to the code and now it seems to work for me too.
I inspected the size of the canvas in Fullscreen mode vs my width/height 100% div and found that the Fullscreen canvas actually was a bit smaller, which probably is why it worked.
In my research I found out that 100vh includes the part of the screen hidden behind the bottom navigation bar on webkit browsers. I removed the CSS and used a resize javascript listener and resized the canvas to window.innerHeight instead.

@tommypreger That's a really cool project! When you're ready to share it more widely, it would be great if you could add it to http://openseadragon.github.io/examples/in-the-wild/… Just make pull request against https://github.com/openseadragon/site-build/blob/master/www/in-the-wild.html to do so.


How to get the operationName inside subscriptionExchange?

I'm running a GraphQL API with graphql-ruby and needed to get subscriptions work with Action Cable, so I've created this SubscriptionForwarder, based on the official ActionCableLink implementation:
import * as ActionCable from '@rails/actioncable'
import { SubscriptionOperation } from '@urql/core/dist/types/exchanges/subscription'
import Observable from 'zen-observable-ts'

class ActionCableExchange {
  cable: ActionCable.Cable
  channelName: string
  actionName: string
  connectionParams: object

  constructor(options: { cable: ActionCable.Cable; channelName?: string; actionName?: string; connectionParams?: object }) {
    this.cable = options.cable
    this.channelName = options.channelName || 'GraphqlChannel'
    this.actionName = options.actionName || 'execute'
    this.connectionParams = options.connectionParams || {}
  }

  request(operation: SubscriptionOperation) {
    return new Observable(observer => {
      var channelId = Math.round(Date.now() + Math.random() * 100000).toString(16)
      var actionName = this.actionName
      var subscription = this.cable.subscriptions.create(
        Object.assign(
          {},
          {
            channel: this.channelName,
            channelId: channelId,
          },
          this.connectionParams,
        ),
        {
          connected: function() {
            this.perform(actionName, {
              query: operation.query ? operation.query : null,
              variables: operation.variables,
              // TODO: add this to support persisted queries. But we need to get the operationName at first
              // operationId: (operation as { operationId?: string }).operationId,
              // operationName: operation.operationName,
            })
          },
          received: function(payload) {
            if (payload.result.data || payload.result.errors) {
              observer.next(payload.result)
            }

            if (!payload.more) {
              observer.complete()
            }
          },
        },
      )

      // Make the ActionCable subscription behave like an Apollo subscription
      return Object.assign(subscription, { closed: false })
    })
  }
}

const cable = ActionCable.createConsumer('ws://localhost:3000/cable')
const actionCableClient = new ActionCableExchange({ cable })

const defaultExchanges = [
  dedupExchange,
  cacheExchange,
  errorExchange,
  fetchExchange,
  subscriptionExchange({
    forwardSubscription: operation => {
      return actionCableClient.request(operation)
    },
  }),
]

I'd like to add this to the graphql-ruby repository and then add some documentation about the usage with ActionCable to urql. But I need to figure out how to get the operationName of the query in my actionCableClient at first, since graphql-ruby needs this to support persisted operations. I took a look at the source of the subscriptionExchange, but I think that it's not possible to get the operationName at the moment, isn't it?
Also I was thinking about the possibility to use the operationName to invalidate the cache (little bit off topic, but I don't want to open a second issue). Sometimes it would be helpful to invalidate certain queries, since I don't want to invalidate all of my viewer queries if I just could invalidate GetViewerIssues for example. Any thoughts about this? 😊
Last but not least: thank you so much for urql. I've switched from Apollo and everything feels way more "under control" now 😄

Hey @gopeter
Thank you so much for the kind words, means a lot!
Getting the operationName should be possible by looking at the definitions of the Query, like this. That sounds a bit like the document-cache which will automatically invalidate by __typename and you can pass in additionalTypenames, more information about that here.
I think you are referring to the refetchQueries option of apollo, we don't currently support this.

Thanks for the quick reply 😊
Great, I'm able to get the operationName now. Still have to look for the operationId, which I should get from the persisted-fetch-exchange.
Yes, I think it's basically the same as refetchQueries. I'm using the graphcache-exchange, but maybe I find another way to invalidate specific viewer-queries.
Thanks for your help, closing this for now 😊

Oh yeah, on spectrum there is something about this. You can read more about invalidate here

Nice, thank you very much!

Hi @gopeter, were you able to figure this out? It's possible to share it please? I have no idea how to make subscriptions work with rubys action cable. Thanks

More or less, I've omitted the operationId for now and don't use persisted queries, but the subscription works:
import * as ActionCable from '@rails/actioncable'
import { Exchange, createClient, dedupExchange, fetchExchange, subscriptionExchange } from 'urql'
import { Cache, cacheExchange as createCacheExchange } from '@urql/exchange-graphcache'
import { SubscriptionOperation } from '@urql/core/dist/types/exchanges/subscription'
import { devtoolsExchange } from '@urql/devtools'
import Observable from 'zen-observable-ts'
import { DocumentNode, Kind, parse } from 'graphql'

// based on https://github.com/rmosolgo/graphql-ruby/blob/master/javascript_client/src/subscriptions/ActionCableLink.ts
class ActionCableExchange {
  cable: ActionCable.Cable
  channelName: string
  actionName: string
  connectionParams: object

  constructor(options: { cable: ActionCable.Cable; channelName?: string; actionName?: string; connectionParams?: object }) {
    this.cable = options.cable
    this.channelName = options.channelName || 'GraphqlChannel'
    this.actionName = options.actionName || 'execute'
    this.connectionParams = options.connectionParams || {}
  }

  getOperationName(query: DocumentNode): string | undefined {
    for (let i = 0, l = query.definitions.length; i < l; i++) {
      const node = query.definitions[i]
      if (node.kind === Kind.OPERATION_DEFINITION && node.name) {
        return node.name.value
      }
    }
  }

  request(operation: SubscriptionOperation) {
    return new Observable((observer) => {
      const channelId = Math.round(Date.now() + Math.random() * 100000).toString(16)
      const actionName = this.actionName
      const operationName = this.getOperationName(parse(operation.query))
      const subscription = this.cable.subscriptions.create(
        Object.assign(
          {},
          {
            channel: this.channelName,
            channelId: channelId,
          },
          this.connectionParams,
        ),
        {
          connected: function () {
            this.perform(actionName, {
              query: operation.query ? operation.query : null,
              variables: operation.variables,
              // TODO: add this to support persisted queries. But we need to get the operationName at first
              // operationId: (operation as { operationId?: string }).operationId,
              operationName: operationName,
            })
          },
          received: function (payload) {
            if (payload.result.data || payload.result.errors) {
              observer.next(payload.result)
            }

            if (!payload.more) {
              observer.complete()
            }
          },
        },
      )

      // Make the ActionCable subscription behave like an Apollo subscription
      return Object.assign(subscription, { closed: false })
    })
  }
}

const actionCableClient = new ActionCableExchange({ cable })

const defaultExchanges: Exchange[] = [
  devtoolsExchange,
  dedupExchange,
  cacheExchange,
  errorExchange,
  fetchExchange,
  subscriptionExchange({
    forwardSubscription: (operation) => {
      return actionCableClient.request(operation)
    },
  }),
]


Any Keypress?

I posted on the website about a symbol for any keypress, as I believe it may be useful to have access to. Possibly something like app.keyboard.any

Pushed new version - app.keyboard.any should be there
There is a problem if you press keys, then switch tab or window - I will fix that later.

I did a little digging myself as I found that sometimes playground.js reported that -1 buttons were being pressed, so I added a small fix that catches the bottom limit. At line 2300 I changed this.any--; to this.any = this.any - 1 < 0 ? 0 : this.any--` and the -1 problem is no more.

I will add it as a temporary fix, but we should really find when it happens :)
I am only aware of -1 problem when you press a key being in application - but release it in another window.


Feature Request: Right click to copy action

When looking at the log monitor (or anything else with a list of actions), it'd be nice to quickly be able to copy the action to your clipboard, or maybe add that action to the dipsatcher. Possibly something like right click the action and a menu pops up, of which one of the options would be copy to clipboard or copy to dispatcher (or both).

In the latest versions of the extension, there's a raw view for action and state (for Inspector Monitor):

Also I'm planning to add there a button to Redispatch and another one to Edit (so you'd be able not only to dispatch a new action with the same payload, but also to change the payload of already dispatched action and have the state recomputed according to it).
However, having a context menu for react-json-tree as described in https://github.com/gaearon/redux-devtools-log-monitor/issues/60 would be much useful. That's something which would be much welcome if someone could implement there.


Upgrade to Slevomat Coding Standard 7.1.0

Remove support of PHP 7.1 (as it was removed in Slevomat Coding Standard v7.1.0).
Upgrade PSR-12 Ext to the version supporting Slevomat Coding Standard v7.1.0.

Implemented in #5.


Include LICENSE in pip package

Thanks!

@rgieseke By the way I created a conda-forge package are you interested to join me as maintainer? https://github.com/conda-forge/cite-feedstock

Yeah, i saw that! Thanks for that, i also did a new release which should includes the License.
I'm grateful that you maintain that and am happy to make any changes you might need, but i'm trying to keep the things i have to feel responsible for low.
In my experience and where i said yes to being a co-maintainer, conda updates are working fairly automatic but if not i just don't want to be the person who feels they need to fix it :-)

sounds good to me

Added a conda badge! https://github.com/rgieseke/cite
:rocket: Thanks again for maintaining this!