v4.0.0 UI lag

Since updating to v4.0.0, I'm experiencing a lot of lag with the UI catching up with the mouse cursor position and selectable elements of UI like play buttons when hovering over an album cover not showing or very delayed, volume slider delayed appearance, and menu items highlighted background color that shows when you are hovering over it very delayed.
The product unfortunately feels broken in this version.  Should I try a reinstall?  Let me know what additional info I can provide.
see the video for detail and pay attention to the mouse cursor:
https://youtu.be/HfmpdkQb82o
forgot to show proof of the version in the video, but took a screen shot afterwards:

My system info:
Windows 10.0.14393 x64
Processor	AMD FX-8370 Eight-Core Processor, 4000 Mhz, 4 Core(s), 8 Logical Processor(s)
8.00 GB RAM
NVIDIA GeForce GTX 950
where is this in the FAQs? I looked, but it's not listed on that page.

If appropriate please attach the DEBUG information zip file that you can generate by following the instructions in the FAQ
(https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ)

@meow81 Sorry that debug info FAQ appears to have vanished.  I added it back in
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ#how-do-i-generate-this-debug-zip-file

Regardless however can you try clearing your cache completely.
Delete the %APPDATA%\Google Play Music Desktop Player folder 👍

I'm seeing massive lag and full-screen glitches with 4.0 as well, makes it almost unusable. The glitches in particular, the screen flashes white every time it repaints (eg: navigation, background images change, etc).
Clearing application data didn't help.
On Macbook Pro 2015, MacOS Sierra latest

debug info:
debug_info.zip
delete this folder?

@meow81 Delete the entire Google Play Music Desktop Player folder.

tried that.  It didn't work.

I am also seeing this issue on Mac OSX.
Could be related to the new UI changes made?

just noticed the v4.0.1, updated, not sure if that has anything to do with this issue or not.  Regardless, I cleared cache as instructed before and experience the same lag issues.  Possibly a slight improvement, but the mouse cursor and UI are still not in sync.  UI elements are jumpy and flashing under the mouse cursor and selections are still a bit behind where my mouse actually is.

I'm experiencing the same issues with the recent update. It wasn't perfect before, but the two most recent updates unfortunately really destroyed usability for me :(.
Running Windows 10 Insider Preview on a Lenovo X1 Yoga (Core i7-6600U 2.81Ghz, 16GB RAM, 512GB SSD).

Guys to determine where the issue is.  Can you download raw Electron from here
https://github.com/electron/electron/releases/download/v1.4.3/electron-v1.4.3-win32-ia32.zip
And open the dev tools console and run the following command
window.location = "https://play.google.com/music/listen#/now"

Then play around and see if the UI lag is present.

Using the raw Electron release makes a WORLD of difference. Still not 100% perfect (just like most Electron-based apps I've used unfortunately), but a massive difference.

agree with bouland.  It was night and day, only experienced a small delay once or twice, but noticeable better experience.

Hm, without a device to reproduce the issue tracking down the lag will be tricky.  Can someone run a CPU profile from dev tools and upload it here?

I'm embarrassed to admit, but how do I do that?  I will, just need a little guidance.  This app is amazing and I'll do whatever to help out.

Type DEV_MODE into the go to URL modal.
Relaunch GPMDP
Open dev tools with the standard chrome shortcut.
Click profiles
Click "start".  Do some stuff for 30 seconds. Click "stop"
Upload it here

Can't upload that file type, so I zipped it.
CPU-20161018T221250.zip

@meow81 That profile appears to be from the webview not the main window.  Can you upload a profile for the main window?

I'm having trouble running the app to get dev mode.  I'm not sure what I did originally was correct.  Should I be using Electron?

No you should be using GPMDP.
I'll try be more clean with the steps.

Open GPMDP
Use the shortcut "Control + Shift + G"
Type in "DEV_MODE"
Click "Let's Go"
Relaunch GPMDP
Use the dev tools shortcut "Control + Shift + I"
Use the profiles tab to make a CPU profile

CPU-20161019T181635.cpuprofile.zip
Funnily enough the UI was reasonably smooth while recording that profile, maybe intermittent?

@seaneking I can't see any obvious issues with that profile.  Perhaps wait till it gets really bad and then start the profile 👍

Here's a profile from me. It was fairly slow during the recording, but probably not as slow as it has been at its worst before.
CPU-20161019T120723.zip

Here is another profile. The recording was the same as if I used the application normally with the lag.
CPU-20161019T062455.zip
I did find an error in the dev tools as well, not sure what it means:
"C:\Users\hydro\AppData\Local\GPMDP_3\app-4.0.1\resources\electron.asar\renderer\web-view\web-view.js:294 ::shadow pseudo-element is deprecated. See https://www.chromestatus.com/features/6750456638341120 for more details."

@hydrocane that's just a deprecation warning for the ::shadow selector, wouldn't effect UI lag.

CPU-20161019T191543.zip
Okay got it this time.  Have to say, running in DEV_MODE reduces much of the lag, so the capture won't be a good representation of what I experience running in a normal mode.  When I switched back the lag returned to a pretty severe level.
Also, noticed that randomly now when I open the app it appears to just be displaying the web UI of google play music and no desktop app settings are present, not even the menu.  Restarting the app fixes it.

Also, noticed that randomly now when I open the app it appears to just be displaying the web UI of google play music and no desktop app settings are present, not even the menu. Restarting the app fixes it.

Getting the errors from console would help a lot with figuring out this issue.  Someone weird is happening somewhere

Similar issues on MacOS: https://vid.me/hsPY

@meow81 That log appears to indicate some kind of event loop, i.e. an event causing another event to be fired and react's event batching is working overtime to handle it.
It's gonna be hard to track this down 🍆

In addition to the lag I'm seeing some other strange issues with version 4. I'm having frames drop. Animations appear to be around 10 to 20 frames per second. (I have an AMD RX 470 and had no issues with version 3.6.)
Also, when I'm in an album and hover over a song, the hover effect is showing on the wrong song. For example, when I hover over song #1 in an album, it shows the hover effect on #8.
In the following screenshot, I had my cursor over song #1.

Do we have any idea how widespread this?  Is it possible to revert back to the previous version 3 until the issue is identified?  I also realize you probably would like the logs/data from impacted users on v4.

Tbh, the easiest way for me to track this down is to get access to a device with the symptoms.  I have tried all my devices and numerous VM's and none of them have the issue.
The analytics I have access to don't show any massive errors or a decrease in user count so it appears to not be very wide spread (you guys might just be unlucky).  If anyone is OK with me using teamviewer and diving into the problem message me on Gitter and I'll try solve this for you guys 👍

@MarshallOfSound What GPU do you have? It seems that all of us with issues are on an AMD GPU.

@lucaseverett I'm on team green, but I know some people on AMD GPU's that aren't having this issue.  Could be related though.  not sure

@MarshallOfSound I'm open to a teamviewer session so you can get some additional data.

@lucaseverett I am using an Intel HD GPU, so it may not be AMD related

@meow81 Hit me up on Gitter and we can sort this out 👍

Is there anyway to prevent auto-update from occurring so I can stick with version 3.6.0?

@hydrocane Technically yes, but if you do it, you will not receive any updates, even when I fix the issue (which I still have no idea what it is).  Still need someone to let me have a look around on their machine so I can actually try to find out what's going on.

@MarshallOfSound If you have time now, you can take a look at my screen to sort it out.

@hydrocane Hit me up on Gitter and we can try get it sorted 👍

@MarshallOfSound I appologize, I am new to Gitter, how do I do this?

I'm having a similar issue, instead of lag though my mouse just acts like it's jumping around on the UI, I'll click the volume meter and it will immediately jump from 0-100% constantly back and forth. Same thing when clicking on songs or play/pause controls(selection jumps around).
Happens both on 4.0.0 and 4.0.1, I've already deleted the GPMDP folder in appdata. Let me know if you need more screen captures.
OS: Microsoft Windows 10 Pro (x64) • CPU: Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (2.81GHz) • Memory: 15.9 GiB  • Storage: 56.2 GiB / 109.2 GiB (53.0 GiB Free) • VGA: Intel(R) HD Graphics 520

@shadeslain I had the same issue. What type of computer are you using. Apparently because my laptop has a touch screen, it causes the UI to glitch out. Marshallofsound is currently working on this with the electron people.

@hydrocane Dell Latitude E7470, mine is the Non-touch screen FHD variant.

Mine is a MacBook (no touch screen) as well, UI still laggy and glitches out.

Big lag and mouse over events triggering on items far away from the mouse, also resizing the window is causing many problems.

Yes, I am having the same issue on my Macbook since upgrading to 4.0.x. The screen flashes way more than I could take, so I went back to 3.6 and its all good again.
MacBook Pro (Retina, 15-inch, Late 2013)
2.3 GHz Intel Core i7
16 GB 1600 MHz DDR3
NVIDIA GeForce GT 750M 2048 MB
Intel Iris Pro 1536 MB

I'm having major lag issues and sketchy, low-framerate animations. Dev mode seemed to significantly improve my experience, although it was still significantly laggy (albeit about as bad as the web UI).
Running Windows 10 w/ recent Anniversary Update on a Surface Pro 4 with a Core i5 and 8GB RAM. I've attached a CPU profile I took, it included some bad lag but I had to push to get it (normally the lag almost renders the app unusable from the moment after it starts).
CPU-20161024T160649.zip

@ericralph You probably have the touch screen issue @hydrocane has

You closed my issue i opened, so here is a CPU profile, dev mode reduces the FPS drop and UI jitter enough so i can type this out.
CPU-20161024T230557.zip
Its really bad in normal mode, to the point where my cursor just starts moving around by its self reaaaaaly slowly. I'm open to a hangouts call and a Teamviewer session. i WFH most of the time so im available.

@MarshallOfSound that last CPU Profile looks like the majority usage is with jQuery, or am I reading it wrong?

@jostrander Yeah, that one appears to be mainly jQuery but it is within normal usage bounds.  If you turn off "scroll lyrics automatically" in settings that usage will drop to 0.

after turning off that setting, do i need to restart the app?

Nope, the jQuery animation will stop immediately

lag still exists :(

Can you take another CPU profile without that enabled?

Here you go, i put it through its paces during this, lag was everywhere got my cursor doin the jitter thing again.
CPU-20161024T235051.zip

@VenomXII Are you sure the DEV_MODE improves the situation?

not really, its not much of improvement other than it lets me use my cursor while the program is minimized. if its in normal mode even when minimized or in the background my pc gets brought to a crawl.

@VenomXII In task manager can you screen shot which process is using the CPU.  Also screenshot the helper processes

If i force close the program and reopen it, it brings everything to a crawl/stop, then it comes back, its still in dev mode when i took these screenies.

@VenomXII Out of interest and a completely wild guess, can you install Bonjour Print Services.  Restart your computer, and see what happens 👍

It does look like GPMDP is indeed using a massive amount of CPU, albeit variably. Moments after opening (no music playing, haven't even opened the interface), it jumped to 100% CPU utilization and is hovering around 25-75% currently.

@MarshallOfSound Installing Bonjour Print Services and restarting resolved the issue here. I guess I never had it installed on my Surface but did  on my desktop.

@MarshallOfSound I'd rather avoid installing anything apple related on my windows machine. and per @Dogodile it seemed to not work.

@VenomXII I don't know if you read the same thing I did, but he said it resolved the issue. The bonjour print services are for chromecast and is a standard that Google decided to use.

To stop the "I have this issue as well" spam I'm going to lock this conversation.
This will hopefully be fixed by Electron 1.5.0 and a Chromium version bump.  As soon as I get my hands on a build for that version of Electron I'll link a build up here for you guys to test out.
If anyone has any magical insights on the cause or possible solutions for this issue please raise them in Gitter
I'll keep exploring possible causes and hopefully we'll get this fixed

I have reports that 4.0.2 fixes this issue.  Can you guys please test and confirm 👍

No change for me. I also deleted the "AppData\Roaming\Google Play Music Desktop Player" folder just to be safe.
I also did a clean install of Windows 10 and tried a different video card last week and there was no difference.

Is there a way to force 3.6 to not update? I've been reinstalling 3.6 often because even though I don't manually update it eventually it updates by itself.

Still bugged for me :(
I deleted the Roaming GPMDP folder too. I can start music, but I cannot touch anything after press play button.

Yes. It is still flashing for me on MacBook Pro. However, it only seems to happen when on the laptop (retina) monitor. If I use an external monitor (HDMI/1080P) there don't seem to be any glitches at all.
The glitch I am seeing (again, only on the retina screen) is best described as a flash (turns white for a split second) in the screen right after the each background artist image is loaded. Just distracting is all. My Parallels Windows 10 VM works great.
Maybe another clue is on my Dell XPS 13 with a quad hd touch display, I see glitchy issues as well (mouse roll-over flashing). Maybe has something to do with ultra HD rez monitors?
Thank you!

That's interesting, I'm using a 4k monitor with Windows recommendes scaling of 150% turned on.  Not sure if that helps.  @MarshallOfSound, offer still on the table to do TeamViewer if you think that will help with troubleshooting.

I also have a 4k monitor. As a test, I turned off display scaling and set it to 100% and all of the glitches go away. So this does appear to be an issue with scaling. Anything above a 1.0 scale factor glitches to the point of being unusable. With 3.6 it works at all scale factors.

Good find @lucaseverett .  I would expect it to work with scaling and hope the fix is prioritized (if it is related to that).  Without scaling on 4k, some apps look awful or are really tiny, almost unusable.

1080p display here, changing scaling from 150% to 100% and signing out/in solved the problem completely for me!

I also have display scaling set to 200% (Surface Pro 4 is nearly
nonfunctional without scaling). I can also verify that installing the
Bonjour Services did nothing noticeable to alleviate the problems I've been
having.
On Fri, Nov 4, 2016 at 2:20 PM, DericMcDerpy<EMAIL_ADDRESS>wrote:

1080p display here, changing scaling from 150% to 100% and signing out/in
solved the problem completely for me!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/1777#issuecomment-258550440,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANP6WS_CRhAE_QAlVlIUoROB0GKSEJ9Yks5q66GXgaJpZM4KYGh2
.

--
Eric Ralph
Science, Technology, & Society w/ Bioethics Emphasis | 2017
University of Puget Sound
3418 N Ferdinand St,
Tacoma, WA  98407
<EMAIL_ADDRESS>303.990.0977

Raised https://github.com/electron/electron/issues/7882
Can you guys with the DPI issue test this build.  https://mega.nz/#!eNdDBDCa!rPhFnzwTc5q3Soz8bkdBnmUNsLT6-Y3t4nuI4mkuJkY

Works the same here. Just tried it with no scaling, looks perfect. As soon as I scale to 150% the issue is back.

Here is another one 😆 This one theoretically should work 👍
https://mega.nz/#!KM0izAra!cAUickZUiQEdMJy7XiDMFB7mPNOUAtE6aOkqPC7G9q8

Seems to have done the trick! Works great in Windows at 150% scaling. I look forward to trying it out on Mac.
Thank you!

@donewert Although that one works it is not a fix.  It is simply determining the exact version number in Electron that the issue started occurring.  You're welcome to use it 😆 but it isn't fixed yet 😆

Ah, gotcha. I will enjoy this special version on my PC then. :)

I can verify that that modified build from the MEGA link works MUCH better.
Still not exactly snappy, but it is very functional.
On Fri, Nov 4, 2016 at 9:28 PM, donewert<EMAIL_ADDRESS>wrote:

Ah, gotcha. I will enjoy this special version on my PC then. :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/1777#issuecomment-258590164,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANP6WYBnmdrKnhD7wwJOqZMAldzRhkGzks5q7AXmgaJpZM4KYGh2
.

--
Eric Ralph
Science, Technology, & Society w/ Bioethics Emphasis | 2017
University of Puget Sound
3418 N Ferdinand St,
Tacoma, WA  98407
<EMAIL_ADDRESS>303.990.0977

The mouse hover issues are gone in that build but the frame rate is still very low.

I have a 4K monitor and had the same issue

I have a 4K screen and the same issue. The build you just posted fixes the hover spasing but the minimize and maximize buttons for the window stopped working. Framerate seems low (feeling <20).

Guys, please no one download and use that build.  It was a hack job with badly compiled dependencies and some missing ones.  It was PURELY FOR TESTING PURPOSES ❤
If you want to revert to 3.6 and not get bugged by update notifications just change your hosts file to point update.gpmdp.xyz somewhere else temporarily.  (This is not recommended as you won't get the potentially fixing future update)
I'm going to lock this up again to prevent spam.  I'm hoping that the next Chromium bump will fix this for you guys.  Stay tuned

OK guys, new theory to test out, can you guys try this build ASAP and let me know if it works.
https://mega.nz/#!jRNyzDIB!dAAEyDCwUrwed2q-7RubOwwqxzhg3xSb6lOqxkx4I2c

Sorry to say, it doesn't work well at all. Interface is laggy, mouse-over effects jump around all over the place.

@boulund Can you try launching the .exe file from CMD like so.
"Google Play Music Desktop Player.exe" --enable-use-zoom-for-dfs=false -- --enable-use-zoom-for-dfs=false

https://mega.nz/#!WUkDSSya!PSuPahIn3CF8DIqA9XuqTlgLdysbo_PwJ8cg6bNLVkg

No mouse jitters! Performance is also good when not maximized. Still rather sluggish when maximized on my second monitor (2560x1440), but that might be because of my machine being a laptop with a weaker processor...

@boulund Sweet, good to hear that I finally tracked it down 😆
Fix should go out next release 👍 Until then feel free to use that build I just uploaded 😄

So, what was the cause?

@boulund Chromium 53 changed the way they handle display scaling on renderer windows.   They automatically enabled the enable-use-zoom-for-dsf flag which apparently doesn't cope with webviews very well.  Forcing that flag off fixed the issue 👍
See the commit I just pushed for reference.

At least for me, the 11/10 build was much worse than the previous one.
Very, very laggy and could not even get the hover interaction aspects to
work at all.
On Thu, Nov 10, 2016 at 5:27 AM, Samuel Attard<EMAIL_ADDRESS>wrote:

@boulund https://github.com/boulund Chromium 53 changed the way they
handle display scaling on renderer windows. They automatically enabled the
enable-use-zoom-for-dsf flag which apparently doesn't cope with webviews
very well. Forcing that flag off fixed the issue 👍
See the commit I just pushed for reference.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/1777#issuecomment-259689499,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANP6WWPnpEMhjylgu6Z3Y7VFWbfwuBvLks5q8xvIgaJpZM4KYGh2
.

--
Eric Ralph
Science, Technology, & Society w/ Bioethics Emphasis | 2017
University of Puget Sound
3418 N Ferdinand St,
Tacoma, WA  98407
<EMAIL_ADDRESS>303.990.0977

this build worked for me:
https://mega.nz/#!WUkDSSya!PSuPahIn3CF8DIqA9XuqTlgLdysbo_PwJ8cg6bNLVkg

anyway to make a MacOS Siera version of this build or is this going into the mainline soon?

I just upgraded to the latest (4.0.3) on Sierra. The problems persist there. Would be happy to test any builds if needed.

the latest build actually seems worse than 4.0.2, i cant even see the queue or the left nav anymore.

4.0.3 still broken for me too

@seaneking @wcomartin Are you using High DPI Displays? (4k, retina, etc?)

When I have the issue it is on my retina screen, the 1080 doesn't have the issue

The windows version is a little laggy on my 4K screen, but not glitchy. The Mac version on Retina flashes so much I had to shut it off. 1080P seems fine, however, no background images (e.g. artists in a playlist) show up anymore. The regular web site is fine.

I'm on a retina MacBook, and yep Super glitchy

@MarshallOfSound With 4.0.3 and Windows 10, the mouse cursor glitches are gone, but it is still very laggy, low framerate animations, and background images are not showing up. However, in the last build you posted in this thread a few days ago, the background images do load. The lag/framerate is the same, though. Also, I noticed that the background images still don't show up even if I turn off Windows scaling.
All around, 3.6 is much more pleasant to use -- no issues at all. Thanks for sharing the trick with the hosts file.

@lucaseverett Without any way for me to replicate the low framerate issues I can't even begin to come up with a solution.  It is probably an underlying Chromium issue that will be handled by the bump to 54 which is slowly being worked towards over at electron/electron.
Regarding the background images that is due to my quick patch to get the dark theme and other functionality working on the latest google play music UI update.  I have more finely tuned fixes coming shortly but it was like 2am and I couldn't be bothered putting in the hours to get it working nicely so I went for "just working" 😆

To further explain my dilemma for anyone reading these.  I am faced with a  choice.  Currently we are on Electron 1.4.4, the last known version that didn't have these issues was 1.3.6.  Now with that in mind, you would obviously go "Hey, why not downgrade".  Which was my first thought as well, the problem is, 1.4.4 fixed two other crash bugs.
I.e. Downgrading to 1.3.6 would break more clients than it would "fix".

Does the issue persist with a naked Electron wrapper? I thought we ruled that out.
In any case, I think this issue should be reopened. It's obviously still effecting a lot of users.

Does the issue persist with a naked Electron wrapper?

The issue itself is running any website through a webview.  So anyone running the site just in Electron won't notice it unless you go through the trouble of setting up a webview element to test in.

In any case, I think this issue should be reopened. It's obviously still effecting a lot of users.

It's hard to tell how many and I don't think it is the same issue.  There is no way that the fix only worked for a portion of the users.  There must be at least two separate problems.  I think that everyone experiencing "UI glitches" saw this issue and jumped on board saying "me too" "me too" when I think there are multiple issues at play here.
For instance, there is the issue I just fixed (High DPI on Windows inside a WebView) but there is another issue in this thread for touchscreen devices in a WebView.  I think for my sanity's sake you should raise a new issue with the following information.

Host OS:
Host OS Version:
Display Resolution:
Display DPI:
Touchscreen: Yes / No
Clean and concise description:
Video clearly showing problem:

Then I can start categorizing these issues correctly. In addition, ANYONE jumping onto a thread with a comment like "me too" or "same here" or "can this be fixed" will be deleted and if you do it more than once you will be blocked from the repo.  I get that lots of you are having this issue but this thread is a prime example of what NOT to do, there are over 100 comments most of which are just repeats of ones before or contain 0 additional / helpful information.  If you agree with someone, give them the thumbs up reaction, don't add a new comment
End Rant

Host OS:  MacOS 10
Host OS Version: Sierra
Display Resolution: Retina - External monitor, no problmes
Display DPI: Scaled, more space
Touchscreen: No
Clean and concise description: Background white flashing, kind of like all pixels going white then back to whatever the background is. Especially noticeable when background images are loading.
Video clearly showing problem: I will try and upload later.
I also use windows 10 at home, on a 4k screen and do not see any issues worth complaining about that you have not already addressed.
I appreciate your effort and as a fellow dev, fully understand your frustration.

@donewert

raise a new issue

@MarshallOfSound while I understand your point, the me toos and same here comments at least give you an indication of the scale of the issue.  Otherwise how would you get a sense of the impact and be able to prioritize appropriately?  However, I realize that doesn't provide you any information to troubleshoot with and I can understand that being frustrating.
Also, what other alternative besides Github is there to report issues for this software?  Not all of the end users are developers and may not be comfortable or know what to provide here on Github.  I opened up an account just to report the and tried my best to provide some useful info.   I guess what I'm saying is to appreciate users reaching out to you to provide feedback even if it's not in the ideal format.  I work in software/web development, but on the project management side (not a developer) so I can understand the frustrations about receiving feedback when there's not much info provided or feedback that can be perceived as a complaint.
Is there anything else I can help test regarding the original issue?  Or do you think that's resolved now?


fix type hint for event constraint to allow None subtypes

This PR adjusts the type hints for event constraints such that the type hint explicitly allows None for the constraint.
The additional optional represents the check implemented here:
https://github.com/slackapi/bolt-python/blob/54114cad0ff0550d0e51c1492d8830ff87b48968/slack_bolt/listener_matcher/builtins.py#L146-L148
This change allows a type-safe usage of the explicit None subtype constraint:
@app.event(
    event={
        "type": "message",
        "subtype": None
    }
)
def handle_messages_without_subtype(client: WebClient, event: dict) -> None:
    ...

Category (place an x in each of the [ ])

[x] slack_bolt.App and/or its core components
[x] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[x] Document pages under /docs
[x] Others: builtin matchers

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

PS.: Thanks for the awesome framework, it really is fun to use! 💯
PPS.: The CLA tooling described in the contribution guideline doesn't really work for me (couldn't select anything), but I explicitly accepted the conditions by signing off the commit.

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

Thanks for the fast response! After some retries signing the CLA worked and I reverted the changes to the auto-generated docs.


Return empty list instead of null within QueryResultListCallback

DBFlow Version: 3.1.1
Issue Kind (Bug, Question, Feature): Feature
Please note if you are using Instant Run, there may be bugs where generated classes are not created. Ensure you are using
the apt or kapt plugins and that incremental compilation is off.
Description: Would it make sense to return an empty list in the event of an empty query, transaction error, etc, instead of a @Nullable one? Right now I have a custom list callback that handles this scenario to guarantee the list returned is never null, but it seems like a useful addition to the API so I figured I'd bring it up here for disucssion.

sounds good to me. follows pattern of ModelQueriable.queryList()

in develop.


[Feat] Turn the light-dark mode button into a toggle switch

Description
Currently the navbar has a button for light and dark mode which on clicking gives option to opt for a mode, I propose to make it more dynamic by adding a toggle switch which would dynamically detect the colour of the page and change the colour accordingly
Screenshots
No response
Additional information
No response

I would like to work on this issue under hacktberfest2023

All your's @karthiknadar1204  Assigned to you

Hi @Vishal-raj-1 and @ManishBisht777, I have resolved the issue please review my PR https://github.com/FrontendFreaks/Official-Website/pull/78


How does the VM handle multiple certificates with the same subject name ?

How does the VM handle multiple certificates with the same subject name ?

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

ID: e302570c-50f7-af0d-ab7c-62845ecd35cb
Version Independent ID: 679e51cc-2123-adb8-abe2-1c8bbacf46c3
Content: Create a cluster using certificate common name - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-create-cluster-using-cert-cn.md
Service: service-fabric
GitHub Login: @athinanthny
Microsoft Alias: atsenthi

@vnvaditya Thank you for sharing the query. We are looking into this and would get back to you soon on this thread.

@peterpogorski Can you please check and add your comments on this.

@dragav to comment

I'm sorry, I don't fully understand the question. The VM(SS) does not use the certificate itself, and is not even aware of the certificate's subject.
If you are asking how does ServiceFabric distinguish among different certificates with the same subject, the answer is "it depends":

the runtime (basic federation and in-cluster communication) will select the farthest valid (max NotAfter)
FabricUpgradeService (as well as other managed, add-on system services) will select the most recent (max NotBefore)

@vnvaditya Hope the above provided information is helpful. Do let us know if you have any further questions on this.

@vnvaditya We will now close this issue. If there are further questions regarding this, please tag me in comment. I will reopen it and we will continue the discussion.


App crashes when changing order of Dashboard's sections

I noticed that rearranging the Dashboard's sections always crashes the app.😟

Fixed by 2bf634887e66e60ff1b8ad7da8c09b27df072662


Allow Book to Shelf association during book creation/edit

Describe the feature you'd like
Currently it is possible to add books to as shelves in shelves.
It will be better to be able to associate a book to a shelf and (this way) when creating / editing the book.
Describe the benefits this feature would bring to BookStack users
The number of the shelves will be inferior to the number of books, so it will be easier to find the shelves.
in my case : the admin can only creates shelves, do the user can't modify the shelves.
currently, when the user create the book, he is not able to put it into as shelves. So with this feature, he will be able to do it

Very much related to #1483

Same :)


C++ Values LocalCoordinates is backwards

https://github.com/symforce-org/symforce/issues/395#issuecomment-2294152796
Values::LocalCoordinates(v) produces v - this, which is the negative of what's used everywhere else.  On the geo types and cam types, and Values.local_coordinates in Python, it returns this - v

As far as which is correct, I think the convention everything else uses is more intuitive, plus it's much less of a breakage to change Values::LocalCoordinates than everything else.
Above all else though, they should all be consistent (all of the instance methods, at least)


ci: add pylint as part of CI

add pylint to catch static issues of python
files in the repo.
User can now run make lint-py for pylint
check on python files.
Fixes: #1132
Signed-off-by: Yug Gupta<EMAIL_ADDRESS>

in Travis, pylint worked and returned the following issues:
pylint --score n --output-format=colorized ./troubleshooting/tools/tracevol.py
No config file found, using default configuration
************* Module tracevol
C:125, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:126, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:234, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:506, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:509, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
R:175, 0: Too many local variables (16/15) (too-many-locals)
R:238, 0: Too many local variables (16/15) (too-many-locals)

In the Fedora container-image, pretty-table is missing. That should normally get pulled in by RPM dependencies. Check if there is no bug against pylint in Fedora yet, and report it otherwise.
It should not be needed to install pretty-table separately in Travis CI either...

in Travis, pylint worked and returned the following issues:
pylint --score n --output-format=colorized ./troubleshooting/tools/tracevol.py
No config file found, using default configuration
************* Module tracevol
C:125, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:126, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:234, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:506, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
C:509, 0: Unnecessary parens after 'print' keyword (superfluous-parens)
R:175, 0: Too many local variables (16/15) (too-many-locals)
R:238, 0: Too many local variables (16/15) (too-many-locals)

In the Fedora container-image, pretty-table is missing. That should normally get pulled in by RPM dependencies. Check if there is no bug against pylint in Fedora yet, and report it otherwise.
It should not be needed to install pretty-table separately in Travis CI either...

I don't see any errors locally, make sure the pylint is compatible with python3

Ah, and the missing prettytable is a dependency of the python script that is tested, not of pylint itself. In that case, you need to install python3-prettytable too, and explain why it is needed.

Ah, and the missing prettytable is a dependency of the python script that is tested, not of pylint itself. In that case, you need to install python3-prettytable too, and explain why it is needed.

Yes, Locally also I had to install prettytable to resolve the issue. I need to install it too.

The Dockerfile.test looks good to me, just need to correct the .travis.yml


Bug: We need to enable the "validate" button option for users when they are saving their GitOps configuration for the first time

📜 Description
We need to enable the "validate" button option for users when they are saving their GitOps configuration for the first time.

👟 Reproduction steps
none
👍 Expected behavior
Should have validate option in 1st gitops saved
👎 Actual Behavior
No validation button is there
☸ Kubernetes version
1.28
Cloud provider

eks

🌍 Browser
Chrome
🧱 Your Environment
No response
✅ Proposed Solution
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#8586

Also, if there are issues like credentials or SSL error then the page breaks, we need to fix that as well


Day picker is missing option for saturday

Yliopiston ravintola is open on saturdays, this should be added as an option for the day picker

Yeah thats true! I can add that when I am bored and PRs also welcome regarding this.


Enable warnings of dmd command

Currently, the arguments of dmd command, the linter of D-lang, lack the option to enable warning messages (-wi). I just added it.

Cheers! :beers:


AOT and umd bundle

Is AOT  also supported if u use the library with the umd bundle (by setting the script tag) and by putting the library as an external dependency?
Can you still use import { x } from "library" with AOT?

To be compliant with AoT compilation the umd or es format is indifferent: whatever you need is the metadata file.
Import is always the same: import { ... } from 'myLibrary
Are you getting some error?

It complains it can't find the module which is defined as external on the consumer.

Also note  following issue where someone from angular cli says that cli can't work with externals because externals  dont work with aot:
https://github.com/angular/angular-cli

What do you mean with externals? Then Angular CLI works with webpack and es5 bundles, not umd.
Please, provide more info about what you are trying to do.

Well for my customer we need to make a webpack angular  consumer app. The js of angular and the library are not allowed to be bundled with the consumer app. So we have to use a cdn mechanism. The way to do this with webpack is to define externals through webpack.

externals: {
jquery: 'jQuery'
}

So my goal is to have this with aot.

CDN? I do not think AoT can work with it, and it does not depend on this starter.

OK thanks. We will not use aot for now


Change ARC secureshield example to non-multithreading mode

Modify ARC secureshield support:
secureshield will use non-multithreading mode, also modify the sample to fit this.

Great to have no-multithreading support. I see only a 1 line functional change to support this, wouldn't there be some more opportunities for simplifying the ARC architecture part when multi-threading is disabled (e.g. simpler stack management/guard checking or so). Though, these could be left as future optimizations for now.
Did you check if the minimal guaranteed features for no-multithreading (inits, interrupts, UART/GPIO/Timer, see documentation) are indeed working correctly without any changes?


HBASE-28027 Make TestClusterScopeQuotaThrottle run faster

This PR just changes a flaky UT, so running UTs does not actual verify anything. Let me merge.


rbd kernel module is missing

I'm trying to set up a simple Ceph cluster which requires the rbd kernel module. Turns out it's not currently available in the WSL 2 kernel. Hoping for support to run a simple storage server on WSL2.

bump

We also have this issue where we run a Rook K8S cluster and rbd is not found, causing the provisioner to deathloop


Some topics already are part of Cloud Adoption Framework

On the Enable successful cloud adoption page (which doesn't have a feedback block) it says
In addition to the Cloud Adoption Framework,…   and then the blocks link to other Microsoft docs.
However, Cloud Governance, and Technical Skills Building blocks are already part of the Cloud Adoption Framework.

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

ID: 909bcd2f-c3cd-6a51-7929-89181fe63188
Version Independent ID: af62a7e3-3f59-090d-5ac4-377748ac2ca2
Content: Innovate through cloud adoption - Cloud Adoption Framework
Content Source: docs/getting-started/innovate.md
Service: cloud-adoption-framework
Sub-service: overview
GitHub Login: @BrianBlanchard
Microsoft Alias: brblanch

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#190321

This topic has been updated.  Thanks!


Improving redirects

Implementing a function to check for redirects with the paper link pulled from our data. Pull request just to put the code up here, this isn't super refined at the moment.

You definitely have the right idea! My advice is now to fill out issue #40 to help us better define the feature/problem before doing any more coding.

As I'm looking into caching papers, a few questions came up on how we should implement it:

Do we need a separate cache for each topic, or just one cache storage for all the papers available?
Should the entire cache be deleted and recreated every night when getting data? Or, do we just add new cache for new papers for that day & delete outdated ones (assuming we display the latest papers in the last month)?

By storing the address in the json you’re effectively caching it for the day (assuming a daily update interval).

FYI: There are functions like uniqBy in lodash that make deduping much easier:

https://lodash.com/docs/4.17.15?q=uniq#uniqBy
https://lodash.com/docs/4.17.15?q=uniq#uniqWith


Error Using Node .

It says token invalid everytime i run the thingy just yoken invalid, but i have put my own token, my bots userid and then my userid

This error happens due to high usage on the API. Nevertheless we do not providing code support to this repository due to multiple reasons.


operators external-secrets-operator (0.4.4)

bump external-secrets-operator 0.4.4
fixes: https://github.com/external-secrets/external-secrets/issues/746

/merge possible

/merge possible


Notice in admin section does not make much sense for some users

If there are plugins with unmet dependencies, a notice is displayed in the admin section that says: "There are additional plugins that must be installed. Go to the Dependencies install page."

This notice is displayed even for users who do not have the capability to install plugins.  For example, the notice is displayed even for users with the Subscriber role.  The notice does not make much sense for such users because they can't actually do anything that the notice suggests.  If they click on the "Dependencies" link, they just get a page with an error message (which is to be expected, because they do not have the required capability).
I think it would be better to do either of the following:

I think it would be best simply to not display the notice at all if the user does not have the capability to install plugins.
If you still want to display a notice to such users, it would be best to reword it (for those users) to something like "There are additional plugins that must be installed.  Please notify the site administrator."  This is similar to the wording currently used in WordPress core when there is an update available.

Thanks for the report. I'll look into this.


File size 0 when quality set

I am loading and sizing images, however if I specify quality the image gets a file size of 0. This happens when I use $image->quality_png = 50; and $image->save($tmp_name, IMAGETYPE_PNG, 50);
But the images export fine with out setting a quality.
My code:
$image = new \Eventviva\ImageResize($_FILES['file']['tmp_name']);
$image->resizeToBestFit($image_size[0], $image_size[0]);
$image->save($tmp_name, IMAGETYPE_PNG, 50);

Hi Stephen, for IMAGETYPE_PNG you must set the "quality" with a number from 0 to 9.
In case of png images the quality value stands for compression level, 0 is for no compression (high quality).


Update byte-buddy, byte-buddy-agent to 1.14.7

About this PR
📦 Updates

net.bytebuddy:byte-buddy
net.bytebuddy:byte-buddy-agent

from 1.14.2 to 1.14.7
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.14.2).
You might want to review and update them manually.
website/yarn.lock

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "net.bytebuddy" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "net.bytebuddy" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #748.


build: Remove docs build requirement version pin

docutils is a sphinx-rtd-theme requirement: pinning was done
to workaround a bug that seems to now be fixed.
Signed-off-by: Jussi Kukkonen<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.343%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1118

Relevant Lines:
1134

💛  - Coveralls


Feat: creators hero

[x] Adds a hero
[x] Improve dev-workflow using yarn!

The .gitpod.yml file should be updated to use Yarn now. Let's make that after this PR is merged. 🙂


pip download error

Hi Andreas,
seems like the pip install does not work as expected:
python -m pip install autofff
Collecting autofff
  Using cached https://files.pythonhosted.org/packages/fa/93/699704c1a277bd51e6f9c0577fa6ed585ec674f72a6533aa6ff61675ec9f/autofff-0.3-py3-none-any.whl
Collecting overrides>=1.9 (from autofff)
  Downloading https://files.pythonhosted.org/packages/de/55/3100c6d14c1ed177492fcf8f07c4a7d2d6c996c0a7fc6a9a0a41308e7eec/overrides-1.9.tar.gz
Collecting validator>=2.0.6 (from autofff)
  Using cached https://files.pythonhosted.org/packages/09/42/00e6244cdc62bbef3dcdfdbe8232e6960fcca2e7a501f0d74e5c6258b06e/validator-2.0.6.tar.gz
    Complete output from command python setup.py egg_info:
    Download error on https://pypi.python.org/simple/yanc/: [Errno 11001] getaddrinfo failed -- Some packages may not be found!
    Couldn't find index page for 'yanc' (maybe misspelled?)
    Download error on https://pypi.python.org/simple/: [Errno 11001] getaddrinfo failed -- Some packages may not be found!
    No local packages or download links found for yanc==0.2.4
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "C:\Users\fsc6lr\AppData\Local\Temp\pip-build-r9e4eyy_\validator\setup.py", line 21, in <module>
        , 'yanc==0.2.4'
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\distutils\core.py", line 108, in setup
        _setup_distribution = dist = klass(attrs)
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\site-packages\setuptools\dist.py", line 268, in __init__
        self.fetch_build_eggs(attrs['setup_requires'])
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\site-packages\setuptools\dist.py", line 313, in fetch_build_eggs
        replace_conflicting=True,
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\site-packages\pkg_resources\__init__.py", line 836, in resolve
        dist = best[req.key] = env.best_match(req, ws, installer)
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\site-packages\pkg_resources\__init__.py", line 1081, in best_match
        return self.obtain(req, installer)
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\site-packages\pkg_resources\__init__.py", line 1093, in obtain
        return installer(requirement)
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\site-packages\setuptools\dist.py", line 380, in fetch_build_egg
        return cmd.easy_install(req)
      File "C:\CDDK_Example\Tools\Python\v3.4.4-1.1.0\lib\site-packages\setuptools\command\easy_install.py", line 623, in easy_install
        raise DistutilsError(msg)
    distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('yanc==0.2.4')

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in C:\Users\fsc6lr\AppData\Local\Temp\pip-build-r9e4eyy_\validator

It also does not work with a manual installation "python -m pip install ."
I tried with anaconda3 with python 3.6 and a standalone python with python 3.4, both are not working...
Hope you can fix it ;P

not reproducible


HTM-314: Add drawing buttons for square, rectangle, ellipse and star

Codecov Report

Merging #98 (4b21a4f) into main (5cf4f3f) will decrease coverage by 0.87%.
The diff coverage is 27.16%.

:exclamation: Current head 4b21a4f differs from pull request most recent head 4b95b56. Consider uploading reports for the commit 4b95b56 to get more accurate results

@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
- Coverage   60.74%   59.86%   -0.88%
==========================================
  Files         157      158       +1
  Lines        3444     3511      +67
  Branches      574      583       +9
==========================================
+ Hits         2092     2102      +10
- Misses       1242     1296      +54
- Partials      110      113       +3

Flag
Coverage Δ

unittests
59.86% <27.16%> (-0.88%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...rawing-button/create-drawing-button.component.html
100.00% <ø> (ø)

...-drawing-button/create-drawing-button.component.ts
67.69% <0.00%> (-9.51%)
:arrow_down:

...ects/shared/src/lib/services/icons/icon.service.ts
100.00% <ø> (ø)

...b/openlayers-map/tools/open-layers-drawing-tool.ts
12.08% <14.28%> (-0.42%)
:arrow_down:

projects/map/src/lib/helpers/drawing.helper.ts
17.02% <17.02%> (ø)

...drawing-style-form/drawing-style-form.component.ts
54.03% <100.00%> (-1.09%)
:arrow_down:

...onents/drawing/models/drawing-feature-type.enum.ts
100.00% <100.00%> (ø)

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 5cf4f3f...4b95b56. Read the comment docs.


laptop with 4 GB RAM and Core i5 can train R-CNN ?

It can train.Might take a few months.

😄😄😄😄
On Fri, 3 Jul 2020 at 22:14, akrsrivastava<EMAIL_ADDRESS>wrote:

It can train.Might take a few months.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EdjeElectronics/TensorFlow-Object-Detection-API-Tutorial-Train-Multiple-Objects-Windows-10/issues/498#issuecomment-653627313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKB2ML3WHSCG5FLI7NMWDALRZYGY3ANCNFSM4N5EGRMA
.

If you dont have a powerful computer, Dont woory you can use Google Colab to train your model.
https://medium.com/analytics-vidhya/training-an-object-detection-model-with-tensorflow-api-using-google-colab-4f9a688d5e8b
<EMAIL_ADDRESS>

You can download model zoo coco here:
https://github.com/tensorflow/models/blob/master/research/object_detection/g3doc/tf1_detection_zoo.md

Hello to every one, Am training a bird specie object detection. I have  6 classes and using:
software and libraire

tensorflow1.15
python3.6

model:

faster_rcnn_resnet152_pets.config

Laptop:

OMEN 15 which have RTX graphic card (6Go) on Ubuntu 18.04.

The level:
loss:1.4247278, step: 75185
I started the training since yesterday and I stopped the training and continued at this morning at the saved checkpoint as you can know.
But at this stage, I saved and export the model.*-7108 to test test prediction, but i did not get prediction and no boxes was drawn, only if I decrease the treshold from 0.6 to 0.1, i get some false and true prediction.
My question is: does that means my model was not learned very well? or should I continue the training until I reach 200k step.

An now still learning:


[Windows Install/FFMPEG Issue] Failed to Build ffmpeg-sys-next with Cargo on Windows 10

Let me preface this post by sharing how insanely captivating this project was to see, I ain't going to lie, I'm a newbie probably delving into something way out of my knowledge base, I'm not even sure if this application will run on my current system, but it's too enticing not to try. Sorry for the wall of text, I've spent the last 12 hours trying to overcome this, and I am willing to spend another week if necessary.
Issue: Failed to Run Custom Build Command for ffmpeg-sys-next v7.0.0
System Information:

Operating System: Windows 10
Development Tools:

Visual Studio 2022 Build Tools
vcpkg package manager
Cargo (Rust package manager)
FFmpeg (2 different sources attempted)
CUDA installed dependencies installed.

Hardware Specs:

CPU: AMD Ryzen 5 5600
RAM: 16GB @ 3200MHz
GPU: NVIDIA RTX 2060 Super

Problem Description:
Attempting to build a Rust project with the ffmpeg-sys-next crate using Cargo results in the following error:
error: failed to run custom build command for `ffmpeg-sys-next v7.0.0`

Caused by:
  process didn't exit successfully: `D:\Pothon Projects\screen-pipe\target\release\build\ffmpeg-sys-next-50830d51493a5d18\build-script-build` (exit code: 101)

Steps Taken to Troubleshoot and Resolve the Issue:

Installed Visual Studio Build Tools:

Installed Visual Studio Build Tools with the necessary components for C++ development.
Confirmed the installation path for cl.exe (Visual C++ compiler).

Set Up Environment Variables:

Set up environment variables for the Visual C++ tools.
Added the necessary paths to the system Path variable.

Installed vcpkg and FFmpeg:

Installed vcpkg and set it up.
Used vcpkg to install FFmpeg:.\vcpkg.exe install ffmpeg
.\vcpkg.exe integrate install

Confirmed that FFmpeg was installed via vcpkg and available in the x64-windows triplet.

Configured Environment Variables for FFmpeg:

Set the VCPKG_ROOT and PKG_CONFIG_PATH environment variables:$env:VCPKG_ROOT="D:\vcpkg"
$env:PKG_CONFIG_PATH="D:\vcpkg\installed\x64-windows\lib\pkgconfig"

Attempted to Build the Project Again:

Ran the build command in PowerShell:cargo build --release --features cuda

Encountered the Same Error:
Despite the steps above, the build failed with the same error message. The output indicated that the build script could not find the ffmpeg libraries, despite vcpkg confirming the installation of FFmpeg.

Additional Details:

The vcpkg installation path is D:\vcpkg.
The ffmpeg libraries are located in D:\vcpkg\installed\x64-windows\lib\pkgconfig.
Environment variables were set in the PowerShell session before running the build command, and permanently through Environment Vars.
Changed from the FFMPEG source mentioned in the readme for Windows to the latest version found here https://github.com/BtbN/FFmpeg-Builds/releases, but no cigar.

Error Message:
error: failed to run custom build command for `ffmpeg-sys-next v7.0.0`

Caused by:
  process didn't exit successfully: `D:\Pothon Projects\screen-pipe\target\release\build\ffmpeg-sys-next-50830d51493a5d18\build-script-build` (exit code: 101)

Request for Assistance:

Again, I'm a noob but this application lit a fire for me regarding ideas, and I'm eager to make it work. I've tried changing the FFmpeg release, and the specs I've shared as well. Looking for guidance on how to resolve the issue where the ffmpeg-sys-next build script cannot find the ffmpeg libraries installed via vcpkg. Any insights or suggestions for further troubleshooting steps would be greatly appreciated.
If I'm missing something absurd or any other info is necessary, let me know, and again, congrats on an amazing project, hope I'm not wasting anyone's time.

Hey @Fayehung thanks a lot for the enthusiasm, really appreciate the effort as we're trying to make this easier to install for all OSes!
I am not super familiar with windows but I'm not sure you need visual studio code c++ etc.
Did you try to install ffmpeg based on the official website?
https://www.ffmpeg.org/download.html#build-windows
I think then the ffmpeg command should be available in the terminal so that ffmpeg-sys-next would detect it. On Linux / Mac it's PATH which I think is similar on windows.
What about we have a call so I can help you install screenpipe?
This is my calendar scheduler:
https://cal.com/louis030195/screenpipe

That sounds amazing @louis030195, let me clear my environment and dig a bit more on this next week, just so I can have an outline of what the issue is in its essence to streamline the call, I'm pretty sure I'm doing something wrong, especially not being super familiar with rust. Thanks for the prompt response dude, I'll get back to you as soon as possible. Cheers.

@Fayehung any news? would appreciate feedback, to simplify install on windows or screenpipe itself 🙏

@louis030195 first of all, my apologies for leaving you hanging, had a few projects on the backburner and had to do a whole shift of environment, will reattempt the install on a clean windows machine this week, and report back to ya, if I mess it up somehow, I'll take you up on that offer for the 15 mins sesh, next week, you have my word!

Also, it was most likely an issue with my environment variables regarding the path for FFMPEG, since I was able to use it in a different scenario in A111 and batch editing.

@Fayehung hey FYI we released a windows desktop app -> https://screenpi.pe/onboarding


Handle disconnect of WALT better

Improvements to handling the WALT device being disconnected, including proper results from isConnected() and avoiding a crash
Fixes #20

Looks good. Merging.
Thanks.

+1


Can we get the contract address of a deployed contract?

Is there nay chance to get the contract address of a deployed contract ?

If you save a transaction hash, you can later ask the Ethereum node for a transaction details, and one of the fields will be the “contract address”. It’s possible to know the contract address before sending, although I don’t think I’ve made this method public

From: kobiwaldi<EMAIL_ADDRESS>Sent: Monday, August 13, 2018 4:56:19 PM
To: BANKEX/web3swift
Cc: Subscribed
Subject: [BANKEX/web3swift] Can we get the contract address of a deployed contract (#212)
Is there nay chance to get the contract address of a deployed contract ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/BANKEX/web3swift/issues/212, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGKv965vh5mBSX_4WeVyCy7rgBztoK9iks5uQYWDgaJpZM4V6jbF.

ty for your reply ! I got it


Create website

Create website:

Should use JamStack (because we are a markdown tool)
Should contain getting started guide
Should document all features
Should include blog
Should use mix of text and video
Should use our cherry icon as favicon
Should use our domain morello.tools.
Should link to

NuGet
GitHub Packages
GitHub Releases

Should outline differences between products
Could include release notes

CloudFlare Pages looks interesting.

Added a placeholder site at https://morello.tools.
May take  up to 24 hours to go live.

Currently unavailable.

A/C not fully met by #24 .  Reopening.

Site is live!  It looks pretty good on desktop 😄.  Not so hot on mobile (#25 ) 😢.


sources?

Hey, I this is extremely useful, thank you so much for creating this, but there is a lot of low level programming in here, and I'm just wondering what exactly was your reference when you created this, so that maybe I can sort of trace the logic a little better, because I've never really programmed a webserver at this low of a level.
Thanks for creating this!

Hey Zane,
In this specific example, part of the code is from an old legacy project from 10 years ago. I just took it, and extended it so that it fitted what I needed at the time using the official specifications of uPnP. It was only about following a PDF and implementing what was there.
Technically, there was no need to re-implement a HTTP server. If I had written the code myself from scratch, I would have used an existing server. This is exactly what I did when I wrote class UPNPHTTPServerHandler.
The only reason ssdp.py has its own HTTP server is that the code was working when I found it, so I reused it without rewriting it for the sake of rewriting it.

Ok, thanks!


BUG: fail in call to axon_segmentation from apply_model.py when no element is detected

When trying to apply a trained model using the axon_segmentation function from the apply_model.py file on a test image, this message error appears sometimes.
Could it be because the model applied hasn't learned enough? In that case, shouldn't the function just create a mask image with only background, and not try to generate myelin and axon masks?
The applied model:
https://drive.google.com/drive/folders/1BX-VgOFxGGzEYtfOQWDox3s0a7v9cpQL?usp=sharing
The test image

The used config file
https://drive.google.com/open?id=15-kcAF3m-rzYmuVEaNESVWOg3hy_ZIIh
Loading acquisitions ...
Rescaling acquisitions to the target resolution ...
Graph construction ...
Beginning inference ...
processing patch 1 of 6
processing patch 2 of 6
processing patch 3 of 6
processing patch 4 of 6
processing patch 5 of 6
processing patch 6 of 6
ValueError                                Traceback (most recent call last)
 in ()
5                                       config_dict=config_mikula,
6                                       verbosity_level=3,
----> 7                                       resampled_resolutions=0.02)
~/axondeepseg/AxonDeepSeg/apply_model.py in axon_segmentation(path_acquisitions_folders, acquisitions_filenames, path_model_folder, config_dict, ckpt_name, segmentations_filenames, inference_batch_size, overlap_value, resampled_resolutions, acquired_resolution, prediction_proba_activate, write_mode, gpu_per, verbosity_level)
275
276
--> 277                         axon_prediction, myelin_prediction = get_masks(os.path.join(path_acquisitions_folders[i], segmentations_filenames[i]))
278
279
~/axondeepseg/AxonDeepSeg/visualization/get_masks.py in get_masks(path_prediction)
27         tmp_path[0] = os.path.splitext(tmp_path[0])[0]
28
---> 29     imageio.imwrite(tmp_path[0] + '_seg-axon.png', axon_prediction.astype(int))
30     imageio.imwrite(tmp_path[0] + '_seg-myelin.png', myelin_prediction.astype(int))
31
~/miniconda3/lib/python3.6/site-packages/imageio/core/functions.py in imwrite(uri, im, format, **kwargs)
244     writer = get_writer(uri, format, 'i', **kwargs)
245     with writer:
--> 246         writer.append_data(im)
247
248     # Return a result if there is any
~/miniconda3/lib/python3.6/site-packages/imageio/core/format.py in append_data(self, im, meta)
490             im = asarray(im)
491             # Call
--> 492             return self._append_data(im, total_meta)
493
494         def set_meta_data(self, meta):
~/miniconda3/lib/python3.6/site-packages/imageio/plugins/pillow.py in _append_data(self, im, meta)
329                 im = image_as_uint(im, bitdepth=16)
330             else:
--> 331                 im = image_as_uint(im, bitdepth=8)
332             PillowFormat.Writer._append_data(self, im, meta)
333
~/miniconda3/lib/python3.6/site-packages/imageio/core/util.py in image_as_uint(im, bitdepth)
109             raise ValueError('Maximum image value is not finite')
110         if ma == mi:
--> 111             raise ValueError('Max value == min value, ambiguous given dtype')
112         _precision_warn(dtype_str1, dtype_str2,
113                         'Range [{}, {}].'.format(mi, ma))
ValueError: Max value == min value, ambiguous given dtype

@oumayb have you been experiencing this mostly in the Python 3.6 branch, or have you ever experienced it before while using Python 2.7?

@mathieuboudreau I've experienced it before while using Python 2.7.

I've run into the same issue on py3k (with commit d330de2b35979b4d0fcf13ecef5600f53edabf6d).

Closing, issue no longer exists currently.


fix(publish): Use yarn audit instead of npm audit

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[x] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[ ] No

The PR fulfills these requirements:

[x] It's submitted to the dev branch and not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
In projects that use Yarn, yarn audit should be used for dependency audits. Using npm audit will fail since Yarn projects do not have npm-shrinkwrap.json or package-lock.json files.

nice catch @rajivshah3


Need optional support on FTL Schema

We definitely need this so that we can define optional fields in our request/response data types. (and likely other places)

This also implies empty requests, but just calling it out explicitly!
Sadly, Kotlin doesn't support empty data class's so right now I'm having to do...
data class GetOfferingsRequest(val nothing: String)

Maybe we need to expose a custom type from the FTL Kotlin runtime package for empty request/response types.
Although, should it be empty or should it be null 🤔 🤔 🤔


expanded container color v2

its happening again on kodadot.xyz
Description in #4342

Hey @exezbcz, when issue closed and PR merged refer to beta.kodadot.xyz not kodadot.xyz (production)
@yangwao will make a new release in the upcoming days

Hey @exezbcz, when issue closed and PR merged refer to beta.kodadot.xyz not kodadot.xyz (production) @yangwao will make a new release in the upcoming days

Ic, my bad, thanks!


Bugfix 34 default browser configurations not being read

Fixed incorrect reference to an attribute. Default configurations are now read from file correctly.

@ty-desj When running the tests on the VM, I'm getting an issue in the tests/actions/test_selenium.py file about the lines with "browser.browser". It says that "AttributeError: type object 'ChromeDriver' has no attribute 'browser'".
Have you encountered anything like this?

Yup, I know exactly the issue! I had a no brain moment and forgot to include a change in the push. Will correct it and update the PR when it is ready!

@DominicParent the PR is actually ready for review this time. The good news, is that in going back to push the missing commit, I thought of a better, more detailed way of passing the error message to the subtests instead of just the browser name.

@ty-desj All seems good.


Script to generate widget theme data defaults from the Material token database.

With Material Design 3, there is a new token system that provides specific values for various visual properties of each widget. In order support these in the Flutter library, this script will generate theme data for each component based on the data extracted from the token database. See the README.md for more information.
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

Check me

LGTM although tests would be great!

Thanks for the review. I just added a couple of simple tests.

An idea to aid in developing these templates is a non-failing test that indicates which md.comp.fab tokens are used and which are unused.

Interesting.  I am not sure how easy it would be to generate this list given the current structure, but I will think about it.


Value of 0 is invalid

I'm using the 0.3.3 through the dropwizard - cloudwatch integration, but it looks like the code to filter out empty data is the same across versions.
I'm recieving the following in my logs:
ERROR [2015-10-22 04:06:24,815] com.blacklocus.metrics.CloudWatchReporter: Exception reporting metrics to CloudWatch. The data sent in this CloudWatch API request may have been discarded.
! com.amazonaws.services.cloudwatch.model.InvalidParameterValueException: The value 0 for parameter MetricData.member.9.Value is invalid.
! at com.amazonaws.http.AmazonHttpClient.handleErrorResponse(AmazonHttpClient.java:1182)
! at com.amazonaws.http.AmazonHttpClient.executeOneRequest(AmazonHttpClient.java:770)
! at com.amazonaws.http.AmazonHttpClient.executeHelper(AmazonHttpClient.java:489)
! at com.amazonaws.http.AmazonHttp...

I'm assuming that the filter should also be checking that input.getValue() > 0 as well as sample count?

0 values are allowed by CloudWatch since recording a 0 (versus recording no data point at all) can be significant.
I tried to tease out what this could be with the aws cli (aws cloudwatch put-metric-data ...), but since 0 values are allowed, I'm not sure what the actual problem is.
Having said all that this made me realize that if you wanted to record 0s (for counting behavior), you currently cannot, because the CloudWatchReporter omits 0 values to save money on API requests. So I'm opening an issue to make that configurable. #14

For reference, 0 values are allowed (via aws cli)
as a single data point
aws cloudwatch put-metric-data --namespace test --metric-data '[{"MetricName":"small_value","Value": 0}]'

as a statistic set
aws cloudwatch put-metric-data --namespace test --metric-data '[{"MetricName":"small_value","StatisticValues": {"SampleCount": 1, "Sum": 1, "Minimum": 1, "Maximum": 1}}]'

Data samples of 0 is not allowed, which is expected. That case is already handled by current CloudWatchReporter
aws cloudwatch put-metric-data --namespace test --metric-data '[{"MetricName":"small_value","StatisticValues": {"SampleCount": 0, "Sum": 1, "Minimum": 1, "Maximum": 1}}]'

Ok, using the Netflix link above, I was able to replicate the error once the positive small numbers got close to 0:
aws cloudwatch put-metric-data --namespace test --metric-data '[{"MetricName":"small_value","Value": 2.1289799200040754E-109}]'

You get that value by multiplying 1 by 0.5, keep multiplying current result by 0.5 another 360 times.
Evidently some of my metrics emit close enough to 0 numbers that AWS doesn't count them as valid.
I guess what we should do in this situation is just force the number to 0 if the number is sufficiently close to 0?

I'm seeing this as well.

InvalidParameterValueException: The value 0 for parameter MetricData.member.16.Value is invalid.
ERROR [2017-07-12 18:18:36,502] [com.blacklocus.metrics.CloudWatchReporter] Exception reporting metrics to CloudWatch. The data in this CloudWatch API request may have been discarded, did not make it to CloudWatch.
com.amazonaws.services.cloudwatch.model.InvalidParameterValueException: The value 0 for parameter MetricData.member.16.Value is invalid. (Service: AmazonCloudWatch; Status Code: 400; Error Code: InvalidParameterValue; Request ID: 858dd20f-672e-11e7-a6a7-33f4295acab5)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.handleErrorResponse(AmazonHttpClient.java:1588)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.executeOneRequest(AmazonHttpClient.java:1258)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.executeHelper(AmazonHttpClient.java:1030)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.doExecute(AmazonHttpClient.java:742)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.executeWithTimer(AmazonHttpClient.java:716)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.execute(AmazonHttpClient.java:699)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.access$500(AmazonHttpClient.java:667)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutionBuilderImpl.execute(AmazonHttpClient.java:649)
	at com.amazonaws.http.AmazonHttpClient.execute(AmazonHttpClient.java:513)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchClient.doInvoke(AmazonCloudWatchClient.java:1073)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchClient.invoke(AmazonCloudWatchClient.java:1049)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchClient.executePutMetricData(AmazonCloudWatchClient.java:957)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchAsyncClient$10.call(AmazonCloudWatchAsyncClient.java:642)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchAsyncClient$10.call(AmazonCloudWatchAsyncClient.java:636)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	... 3 common frames omitted


NameError: uninitialized class variable @@provider

I ran into an issue when attempting to run 'SupportArticle.embed!' via the Rails console.
I get the following error:
SupportArticle Load (9.3ms)  SELECT "support_articles".* FROM "support_articles" ORDER BY "support_articles"."id" ASC LIMIT $1  [["LIMIT", 1000]]
NameError: uninitialized class variable @@provider in SupportArticle

from /Users/williamahlering/.gem/ruby/3.2.2/gems/langchainrb_rails-0.1.10/lib/langchainrb_rails/active_record/hooks.rb:52:in `class_variable_get'

I followed the setup instructions to add pgvector and successfully migrated the db to add the 'embedding' vector column to my model 'SupportArticle'. I've also set my ENV["OPENAI_API_KEY"] which I can see correctly in the rails console, as well.
My gemfile includes the following:
gem 'pgvector'
gem 'neighbor'
gem "langchainrb_rails", "~> 0.1.10"
gem "ruby-openai", "~> 7.1"
gem 'ollama-ai', '~> 1.3.0'

just some notes of minor issues that need to address but can be fixed manually:
doing:
rails generate langchainrb_rails:pgvector --model=SupportArticle --llm=openai

didn't underscore the table name:
class AddVectorColumnToSupportarticles < ActiveRecord::Migration[7.1]
  def change
    add_column :supportarticles, :embedding, :vector,
      limit: LangchainrbRails
        .config
        .vectorsearch
        .llm
        .default_dimensions
  end
end

didn't add the openai gem either.

If anyone else runs into the @@provider exception message that @andreibondarev mentions here, double check that you've also explicitly added vectorsearch to the corresponding model file where your embedding column has been added.  This error occurs when that piece is missing and you attempt to run any of the embeddings commands against it.  I'm not sure if this helps with the original issue, but wanted to share where I saw it while getting things set up.

If anyone else runs into the @@Provider exception message that @andreibondarev mentions here, double check that you've also explicitly added vectorsearch to the corresponding model file where your embedding column has been added. This error occurs when that piece is missing and you attempt to run any of the embeddings commands against it. I'm not sure if this helps with the original issue, but wanted to share where I saw it while getting things set up.

I'm wondering if there's a way to gracefully handle this error, like rescue-ing and then raising a specific error that points out that your model file is missing the vectorsearch line.

If I run the agent from within a langchain tool, then even though the pgvenctor macro call is there, the @@provider is still nil.


feat(exchanges): only show crypto exchanges card if available

Description
Calling getExchanges, a cloud function, in SelectProvider to determine if any exchanges are available. In the event that there are no available exchanges, the exchange card on Select Payment Method screen is removed.
Other changes
N/A
Tested
Added mockExchanges to test/values to mock fetchExchanges.
Tested fetchExchanges
Tested for missing exchange card if no available exchanges
ExternalExchanges Screen will always have providers passed in via props
How others should test
N/A
Related issues

Fixes #[299]
https://linear.app/valora/issue/ACT-299/only-show-crypto-exchanges-if-they-are-actually-available

Splitting this issue into 2 PRs
Backwards compatibility
N/A

hey @MuckT Since NA is mentioned in how other should test section we are skipping this PR
Cc: @awtsui
Thanks!


Change "Py Test" to "pytest" on user settings UI

Steps to reproduce:
Go to File > Preferences > Settings and search for "Py test"

the occurences should be renamed to "pytest"

validated

Validated


Coverage Differs from Actual In Angular Tree Components

I can go through every line of this in Karma for PhantomJS & Chrome. Debug either & get ~98% of lines "hit" but the coverage does not reflect that & reflects different numbers for Phantom & Chrome.
Reusing the check function logic:
https://angular2-tree.readme.io/v4.0.0/docs/tri-state-checkboxes
I tried Mobx to observe the changing TreeNode & "fixture when stable." It seems not to register changes in the TreeNode map, which changes the path, using either the Observed or original TreeNode object. The code goes through all relate lines of code when debugged & the observable TreeNode shows correct values.
PhantomJS:

Chrome:

Dependencies:
Karma 3.0.0
karma-coverage-istanbul-reporter 3.0.0
karma-phantomjs-launcher 1.0.4
karma-coverage-istanbul-reporter 2.0.4
node v8.9.4
npm 6.4.0
Angular CLI: 1.6.7
Angular: 5.2.11

This lib is only a tiny wrapper over the istanbul api and only reports on what it was given by other tooling. I would recommend raising an issue over on the angular cli or istanbul repos with a minimal failing repro that demonstrates your problem.


Add tests for Stats

Add tests for Stats domain.

Unit tests? Any idea about edge cases that must be tested?

Yep unit tests! The public function! ;)

See https://github.com/PUGX/badge-poser/pull/391

it has been done! 🚀


LoRaWAN IPv6 support?

Hi,
I'm curious if there's been any discussion regarding the newly added support for IPv6 over LoRaWAN IPv6:
https://lora-alliance.org/lora-alliance-press-release/lora-alliance-launches-ipv6-over-lorawan-opens-wide-range-of-new-markets-for-lorawan/
This is relevant to my work, and am curious if this is feasible, and if help implementing this would be welcome?

Ah, just found this  thread and will follow up there: https://forum.chirpstack.io/t/using-chirpstack-on-ipv6-without-netids/16424/4

I'm not 100% sure if that topic is about the same. What you are asking is about this I believe:
https://resources.lora-alliance.org/technical-specifications/ts010-1-0-0-ipv6-adaptation-layer
I have not looked at the details, but I believe this is an application-layer protocol on top of LoRaWAN, e.g. it could be built on top of ChirpStack (much like the FUOTA server).


Performance issues with copying styles with getComputedStyle

copy-css-styles.ts
get-default-style.ts

It shouldn't work out for now ~

Hello, I have used your library, it's great but have performance issue when my DOM is complicate, getComputedStyle may block page render. Do you have any plan trying to solve this issue? Or any idea about it?


Suggestion for easier writing

('01a-bbbb-bbbb', {a: [0-1], b: [0-9]})
Don't you want to make it like this?

That's a good idea. But I don't understand your specific idea because it's a short issue. I'll write an example code for your words and show it to you in a moment.

new RegExpPatternBuilder('${a}').replace({a : 'test' }).expression; // it will be replaced with 'test'
new RegExpPatternBuilder('${a}${b}').replace({a : 'test', b : 'kakasoo' }).expression; // it will be replaced with 'testkakasoo'

@dddieon
A form in which parameters must be able to be transformed regardless of the number of them.

I will consider the emoji you left as a sign of agreement. But I don't know if I can solve this problem. I'll invest a week in this issue alone and ask for help if it's impossible.


Dates from API output are missing UTC time zone indicator

Following up on this Fluidd issue, I have now realized that the date & times returned by the Spoolman API are missing the time zone indicator as per ISO 8601.
Spoolman documentation states that all dates are UTC time zone based, but the serialized representation does not convey that - and I believe this to be incorrect!
For example, calling /api/v1/spool, we get a list of spools and I can see the data in all date fields have no time zone indication:

As these are all UTC dates & times, these strings should end with a single "Z" - following the ISO and thus solving this issue in Fluidd and any other frontend.
As an example, lets take the first date from that example, "2023-07-30T10:59:52", and assume that my system is using Europe/London time zone which is currently UTC+1 as per British Summer Time.
Here's what I get in JavaScript by using the Date constructor:

On the first example, as there is no indication of time zone, Date will assume that it is on local time and parse and convert to UTC.
On the second example, I just suffixed the date with "Z" so that the Date constructor understands that I am indeed passing an UTC value, so no conversion is needed.

Ah yes that is indeed a miss. I will make sure the right timezone marker is added.

@Donkie This issue still persists. It seems that the local time to UTC conversion is doubled up somewhere. When I edit the spool and set last used to the current time (using the GUI), upon save, it is converted to UTC. The GUI then shows the converted UTC time as local time. Then when you poll the API, the already converted UTC time is then converted again. I will provide screenshots of what I am seeing below.
Set to now (current local time)

Click Save
Spools home page shows converted time

And so does the edit page for the spool

When the API is polled (/api/v1/spool), the result is
[   {     "id": 1,     "registered": "2023-09-25T11:13:20Z",     "last_used": "2023-09-25T11:26:43.540000Z",     "filament": {       "id": 1,       "registered": "2023-09-25T11:12:56Z",       "name": "asdf",       "vendor": {         "id": 1,         "registered": "2023-09-25T11:12:28Z",         "name": "asdf"       },       "material": "PLA",       "price": 28,       "density": 1.24,       "diameter": 1.75     },     "used_weight": 800,     "used_length":<PHONE_NUMBER>3341153,     "archived": false   } ]


Cannot build v0.20.0 (amongst others) in go module mode with go v1.14.12.

Running the following minimal test:
mkdir tryk8s
cd tryk8s/
go mod init tryk8s
printf "package tryk8s\nimport (\n  _ \"k8s.io/client-go\"\n  _ \"k8s.io/api\" \n  _ \"k8s.io/apimachinery/v1\"\n)\n\n " > tryk8s.go
go get<EMAIL_ADDRESS>go get<EMAIL_ADDRESS>go get<EMAIL_ADDRESS>go build

Gives the following error on the last command:
go: finding module for package k8s.io/apimachinery
go: finding module for package k8s.io/client-go
tryk8s.go:5:3: module k8s.io/apimachinery@latest found (v0.20.2), but does not contain package k8s.io/apimachinery
tryk8s.go:3:3: module k8s.io/client-go@latest found (v11.0.0+incompatible), but does not contain package k8s.io/client-go

I am using go  go v1.14.12. I also tried other versions but they all gave this error or similar errors.
I don't how else I should be using the k8s modules. It seems the Go module support of the k8i go client isn't working very well "out of the box".  I see now, none of the modules has any usable content under the root, so that's why the imports aren't working correctly. That was the problem we were having. We were upgrading from a very old version of client-go and some packages had been moved around. Thanks a lot, I will close this issue now.

Thanks for the hint! I see now, none of the modules has any usable content under the root, so that's why the imports aren't working correctly. That was the problem we were having. We were upgrading from a very old version of client-go and some packages had been moved around. Thanks a lot, I will close this issue now.


fixes: slow listing of collections, pass an optional embedding_function

avoids unecessary fallback to the default embedding function

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

pass an optional embedding_function to list_collections to avoid the slow fallback to default embeddings.

Test plan

Before this fix, listing collections is slow even when using other embedding functions like OpenAI's one.
With this fix, if the embedding function is passed, listing collections is instant.

Is it bad practice to store API Key (for an embedding function) in the database?

Is it bad practice to store the API Key (for an embedding function) in the database?

@jeffchuber Do you think this can be reviewed/merged it would greatly improve the speed of listing collections and does not break the original signature.

@HammadB was this addressed in another way?

We need to rethink this from first principles. One stop-gap may be to change list_collections send back a list of strings.


Did someone take over the Idea Plugin account

What happened?
During today's update 2.11.7-241.1, i'm seeing, what looks like a different name Proxy OÜ, out of Estonia.
Relevant log output or stack trace
No response
Steps to reproduce
No response
CodeGPT version
2.11.7-241.7
Operating System
None

Good observation, and no, no one has taken over the account nor the extension in general.
Since I'm offering a paid plan for the users, I need to operate under some kind of entity, and Proxy OÜ is something that I had already registered and was easy to use.
I understand that the name can be really suspicious and misleading and I will most definitely change that in near future. Rest assured, it's still the same extension maintained by the same person - just with a different legal entity name displayed.

Awesome and thank you for adding the clarification.


Assertion error where assertion is correct

While debugging the trans TensorQTL
following error occurs
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
/tmp/2523984.1.high_mem.q/ipykernel_12464/2597448483.py in <module>
      4 trans_df = trans.filter_cis(trans_df, phenotype_pos_df.T.to_dict(), variant_df)
      5     ## Permutation
----> 6 perm_df = trans.map_permutations(genotype_df, phenotype_df, covariates_df)

~/miniconda3/lib/python3.9/site-packages/tensorqtl/trans.py in map_permutations(genotype_df, covariates_df, permutations, chr_s, nperms, maf_threshold, batch_size, logger, seed, verbose)
    289     if logger is None:
    290         logger = SimpleLogger()
--> 291     assert covariates_df.index.isin(genotype_df.columns).all()
    292     sample_ids = covariates_df.index.values
    293

AssertionError:

However, while testing, asserting the error statement didn't provide any error.
assert covariates_df.index.isin(genotype_df.columns).all()
covariates_df.index.isin(genotype_df.columns).all()
True

Line by line debugging needed.

As it turn out, phenotype_df is not needed  as input to this function, therefore the name and content not match.


Error : You need to allow media permissions to import songs

I allowed the media permissions and it still occurred

Same here, there's no pop-up asking for media access permission.

Same here, tried putting in a nong and this happened. And yes I do have permissions on
BTW this is happening on Android

I removed the permission check in the latest version, should be fine now


$daily upvote

Is your feature request related to a problem? Please describe.
Right we can just do $daily to collect free NT cash without having to actually do anything to give anything back to the bot or show any support. Members gain money while the bot is losing money. It's a win-lose situation. It's kinda not fair.
Describe the solution you'd like
A win-win situation for both parties.
Describe alternatives you've considered
Add a requirement to collect $daily. They need to upvote the bot first in order to claim their dailies. This is a thing for many other popular bots too. Like with some pokemon bot and waifu bot and RPG bot, you can't collect your daily unless you upvote them.
Additional context
All these are just different ideas
Image 1 : Maybe add the upvote link to $donate or $help

Image 2 : Or add it in the footnote

And then the following two choices:
Message appears when you try to do $daily
Image 3 : Get nothing at all unless they vote

Image 4 : Still get their daily streak even if they don't vote, just without the upvote bonuses

Never Gonna Give You Up

We can make this voting a requirement.

top.gg api being garbage so please no make requirement and make it boost thank


problem changing years

Hello, you did a great job with this component thank you. I post this message because I am struggling trying to change the years, there is no way I can modify by clicking on it.
Also, is there a way to modify the theme color (primary, secondary color...) ?

Thanks, Nicolas

Well everyone needed this so I added in version 2.6.0 that I just released an option to change year/month/day if you click on them in the header.
Also added a [calendar] option so you can pick whether you want to see the calendar at all or not.
Regarding the css overriding:
ionic2-datepicker, ionic2-datepicker .datepicker-wrapper{
background:linear-gradient(135deg, #172446 0%, #2a3c60 26%, #5393ae 61%, #5393ae 66%, #335e88 81%, #336084 100%);
    .datepicker-header{
                           background-color:rgba(0,0,0,0.25)!important;
                           .weekday-header{
                                  background-color:rgba(0,0,0,0.25)!important;
                           }
    }
    .button{
                                          background-color:rgba(0,0,0,0.25)!important;

    }
         .row {
         color:white!important;
         .datepicker-selected{
             color:black;
         }
         .datepicker-current{
             color:#bdbdbd;
         }
     }
}

Stuff like this can work


Make the text fields font size at least 16px

Description
On mobile, Safari will zoom into any text field that has text smaller than 16px.

https://css-tricks.com/16px-or-larger-text-prevents-ios-form-zoom/

We can apply some style just on mobile, probably GitHub already adds “touch” classes
This is super annoying:
https://github.com/user-attachments/assets/965c6895-20fe-4345-9e45-94afd6faf198
Example URLs
Any page with input/textarea

This might be possible with just:
textarea,
input[type=text] {
	-webkit-text-size-adjust: 120%;
}

That does indeed work but GitHub has a few duplicates in the DOM that it uses to extend these fields. That piece of code for example visually breaks the search bar. The solution is to also include these additional elements in the selector.


Update compare-versions.md

Clarification around types of pipeline triggers to include an event-based trigger with "blob post" example.

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

@mdmims Thank you so much for your contribution!!
#sign-off

@JennieHubbard: I'm sorry - only the author of this article, @kromerm, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Keyboard Layout Editor image for Miryoku-QWERTY (and other layouts?)

Hello good people -
I've decided to use my wonderful cheapino to try and learn Miryoku Qwerty. I have flashed the Qwert Miryoku firmware provided on the release page. However, I'm having difficulty figuring out a way to make a printable image of the keymap as a reference to help me learn (to be placed below my monitor).
www.keyboard-layout-editor.com seems like the right tool to generate such an image, as it allows for representations of multiple layers on the same keys, which is nice for making a compact reference image similar to this one, of plain colemak? miryoku.

However, I can see that the Cheapino Qwerty Miryoku firmware doesn't match this Miryoku in some ways. Can anyone recommend how to go about creating an image on www.keyboard-layout-editor.com based on the Miryoku Qwerty firmware provided for the Cheapino?
If it's simple enough I'll be happy to do it for this layout and any other ones released, as I think it will be a useful reference for anyone trying to learn a new keymap.
Tak!
p.s. Additional reference: https://www.reddit.com/r/ErgoMechKeyboards/comments/182nwwc/readable_miryoku_reference/

I just modified the layout above to make it QWERTY, easily enough. However, the Miryoku Qwerty available on the release page doesn't seem to match the layer behavior shown in the image. Perhaps there are differences because I'm on a Mac. But the media, num, and nav layers behave mostly as espected, just not copy paste. But the mouse and fun layers don't seem to behave quite right.

Hey, thats strange, I built the miryoku from the github repo...
Do you have an example of a key/sequence that is not as pictured?

Well, I think it's pilot error. I was confused about the symbols. It looks like everything is right save the (mouse or sim?) copy paste symbols on U I and O. Would I need to recompile with MIRYOKU_CLIPBOARD=MAC to fix that?

Ah, Okay. Im not sure how that works, but since ctrl c/v is cmd c/v on Mac, it makes sense there might be some diff there...
Try recompiling. Let me know if you struggle setting it up, i remember there was an issue about this a while ago...

Yep, I went through quite an odyssey, but did manage to get things working with qmk. However, I think it makes the most sense to just use your stock firmware even if the copy / paste keys don't work, and try to stay close to Miryoku in hopes that it's an emerging standard. So the graphic I shared a few posts back should do the trick for me (and feel free to post it somewhere as a resource for others, if you think it would be helpful to people learning.) Thanks!


napari plugin improvements

Fixes all bullets mentioned in issue #91 except:

multiple widgets introduced when opening local h5ad files - in no case should the center align / pairwise align widget be added to the dock if it already exists there.

The above seems to be an issue with all plugins that I've tried (there's no way to detect which plugins are already "open" at a time). Its probably fine and we can leave it up to the user to close plugin widgets that were opened before data selection was made.

I am trying to run Center Alignment using napari on this branch and the application crashes right after showing the message Stacking slices around center slice on the command line. The exact error message is
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'NSWindow should only be instantiated on the main thread!'

This issue is not encountered when running the test case for center_align_widget.

And the same crash does not happen on the main branch?

And the same crash does not happen on the main branch?

No, it does not.

@anushka255 - can you pull and try again?


Add GitHub's file finder in Code menu

On GitHub, you can press t on any repo and search for files in the repository to open.
It would be really awesome if you could add a search bar in Code similar to how there is a search bar in Issues and Pull Requests. This search bar will then allow user to search for a file to open. If the file is a markdown file, it would be nice if it would render it same as Overview.
Thank you.

I love this feature, def support this idea!
Sent with GitHawk

Ohh this sounds awesome!
Sent with GitHawk

Wanted to ask if there was any progress on this feature? I would love to help add this myself with some guidance.


Objects completely black

Testing this out all primitives seem a solid black color instead of being shaded white like they are in the sample image. Is there a missing step in the documentation perhaps to explain how to light shapes?

reinstalled, tried again and seems to work now

check if you have set a color to the material. In the lastest version it will be black if no color is set.


PodTopologySpread feature is unrelated to its feature gate name

This is a Bug Report
Problem:
When PodTopologySpread graduated to GA, the reference to its feature gate was lost.
See how it was referred to in older versions https://v1-18.docs.kubernetes.io/docs/concepts/workloads/pods/pod-topology-spread-constraints/#enable-feature-gate
This makes it hard for users to figure when the feature was enabled by default or graduated to GA, unless they explicitly go to an older version of the docs
Proposed Solution:
Either:

mention the old feature gate name EvenPodsSpread so users can refer to
https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates/
Add a header that says GA since 1.19? Although I think 1 is sill necessary.

Page to Update:
https://kubernetes.io/docs/concepts/workloads/pods/pod-topology-spread-constraints
/sig-scheduling

cc @Huang-Wei
@sftim for guidance

/sig scheduling

Maybe it's fine to add a note to remind users its correlated feature on non-GA versions is EvenPodsSpread?

remind users its correlated feature on non-GA versions is EvenPodsSpread

Sounds like a good idea.

/triage accepted
/language en
/kind cleanup

/assign @adtac

/close
fixed in https://github.com/kubernetes/website/pull/24764