profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/kiranshila/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Kiran Shila kiranshila KAS Microwave LLC Pasadena, CA kiranshila.com Computational electromagnetics, RF systems, device characterization, and Julia! I work on cryogenic LNAs for radio telescopes and quantum computing.

kiranshila/cybermonday 77

Markdown as Clojure Data

coyt/LOST60 18

Bluetooth Mechanical Keyboard Firmware

kiranshila/Doplarr 18

A better *arr request bot for Discord

coyt/LOST60-Zephyr-Test 4

LOST60 Bluetooth Mechanical Keyboard Firmware

kiranshila/FDFD 4

FDFD Solver for Julia

kiranshila/esprit-skeleton 3

A skeleton project for Esprit, using the Esprit board

kiranshila/EuroSID 3

A better SID Eurorack Module

kiranshila/AdventOfCode 2

Clojure Solutions to AoC

kiranshila/.doom.d 1

Doom configuration

kiranshila/AoC2019 1

Advent of Code 2019

push eventkiranshila/Doplarr

Kiran Shila

commit sha 7c74f8c06e8e04b5395659dba33cf11c97d8f665

Added FAQ to readme and switched adoptopenjdk to adoptium

view details

push time in 2 hours

issue closedkiranshila/Doplarr

Support Lidarr

For music!

closed time in 3 hours

kiranshila

issue commentkiranshila/Doplarr

Support Lidarr

I think this will make things more complicated than just video media - considering we brought in overseerr (which doesn't support lidarr)

kiranshila

comment created time in 3 hours

issue closedkiranshila/Doplarr

Support Readarr

For books!

closed time in 3 hours

kiranshila

issue commentkiranshila/Doplarr

Overseerr Support

Ok whenever y'all get a chance, please give the latest build a try

kiranshila

comment created time in 3 hours

push eventkiranshila/Doplarr

Kiran Shila

commit sha 4232a82ca84c7a134b26f8a4eab62cf47156d2f0

Moved all the interaction flow to a multimethod-based state machine

view details

Kiran Shila

commit sha 479457bba68de778e8ae57ff749b0ff178666ab9

Renamed the backend

view details

Kiran Shila

commit sha a2c161b59a91439132ca8d9e620eb19b7b1bb0d5

Finished making sure direct connection works

view details

push time in 3 hours

push eventkiranshila/Doplarr

Kiran Shila

commit sha a2c161b59a91439132ca8d9e620eb19b7b1bb0d5

Finished making sure direct connection works

view details

push time in 3 hours

push eventkiranshila/Doplarr

Kiran Shila

commit sha 479457bba68de778e8ae57ff749b0ff178666ab9

Renamed the backend

view details

push time in 4 hours

create barnchkiranshila/Doplarr

branch : state_machine

created branch time in 4 hours

issue commentkiranshila/Doplarr

Overseerr Support

@VP-EN I found the bug that was causing the those issues, I forgot to pass in the bool that checks if the request was 4K. I also overhauled the response messages for the different media states. I'm finishing up these patches, I basically rewrote all the interaction flow control so now there is a single state machine that controls it, making the whole thing easier to modify instead of a ton of nested ifs.

kiranshila

comment created time in 4 hours

startedbsless/stress-server

started time in 8 hours

issue commentkiranshila/Doplarr

Overseerr Support

Very helpful! Thank you!

Yeah the requested deal has a few more states that I missed. I also noticed that things currently downloading also show up as un-requested.

kiranshila

comment created time in a day

issue commentkiranshila/Doplarr

Overseerr Support

Alright, give the most recent docker image on this branch a try - should have a message on the thing being requested, and fix the number of users error.

kiranshila

comment created time in a day

push eventkiranshila/Doplarr

Kiran Shila

commit sha 4eb9b48e272bce81c0d205d8cf655847b527d593

Fixed getting all users, used smaller poster image, returns message on duplicate request

view details

push time in a day

issue commentkiranshila/Doplarr

Overseerr Support

Alright here is my solution, ping the endpoint once to find number of ids, then get all of them. That's the only way without having a magic maximum users number

(defn num-users []
  (a/go
    (->> (a/<! (GET "/user" {:query-params {:take 1}}))
         (then #(s/select-one [:body :pageInfo :results] %))
         (else utils/fatal-error))))

(defn all-users []
  (a/go
    (->> (a/<! (GET "/user" {:query-params {:take (a/<! (num-users))}}))
         (then #(->> (s/select-one [:body :results] %)
                     (map :id)
                     (into [])))
         (else utils/fatal-error))))
kiranshila

comment created time in a day

issue commentkiranshila/Doplarr

Overseerr Support

Lol requestrr does exactly what you proposed @VP-EN

https://github.com/darkalfx/requestrr/blob/7c42356bcd1855e8d4fab2f18c3d0bd854f5a720/Requestrr.WebApi/RequestrrBot/DownloadClients/Overseerr/OverseerrClient.cs#L351

kiranshila

comment created time in a day

issue commentkiranshila/Doplarr

Overseerr Support

Yeah all of that code is now

(defn discord-id [ovsr-id]
 (a/go
   (->> (a/<! (GET (str "/user/" ovsr-id)))
        (then #(s/select-one [:body :settings :discordId] %))
        (else utils/fatal-error))))

So much better.

kiranshila

comment created time in a day

issue commentkiranshila/Doplarr

Overseerr Support

Yeah I'd say make a separate issue.

@VP-EN https://overseerr.urlFromRemoteHost.com/api/v1/user/66 shows the correct discordId of the dpzTEST user

This is genius. Absolutely nowhere in the API docs does it say that the user endpoint returns the discord id. That makes everything so much simpler, I don't have to do this whole get all users thing. https://api-docs.overseerr.dev/#/users/get_user__userId_

It doesn't even mention the settings block.

kiranshila

comment created time in a day

issue commentkiranshila/Doplarr

Overseerr Support

Awesome detective work @VP-EN! Nowhere in the docs could I find that the users endpoint is paged as well. The docs clearly say get "all" users.

Great find. I'll get it fixed.

kiranshila

comment created time in a day

issue commentkiranshila/Doplarr

Overseerr Support

@VP-EN I just tried a Local User and Plex User in overseerr and have not been able to reproduce the issue.

kiranshila

comment created time in 2 days

startedadoptium/adoptium

started time in 2 days

issue commentkiranshila/Doplarr

Overseerr Support

This seemed a little hacky anyway, but there wasn't an obvious way to connect the overseerr user account to the discord ID

kiranshila

comment created time in 2 days

issue commentkiranshila/Doplarr

Overseerr Support

...uhhh haha. I'll take a look. Something is up with https://github.com/kiranshila/Doplarr/blob/5c68b56c35be43e597b6600211ea28c4202920a2/src/doplarr/overseerr.clj#L34-L46 then

kiranshila

comment created time in 2 days

issue commentkiranshila/Doplarr

Overseerr Support

Ooo interesting - I suppose I should try to add more than one user to my test setup here haha

kiranshila

comment created time in 2 days

issue commentkiranshila/Doplarr

Overseerr Support

I'm wondering how I can limit the bot to only work in a specific channel though. Do you have any thoughts on this/ideas?

It's not possible with the current slash command API, that's why we need the role.

kiranshila

comment created time in 2 days

issue commentkiranshila/Doplarr

Overseerr Support

Slightly annoying, but that's not your fault smile I guess I'll create some trash role for the sole purpose of the bot +1

An alternative idea would be to make the role bit optional, if you want everyone on the server to have access to it, then we could just not use a role at all. Those are the two options the discord API has, all or nothing.

kiranshila

comment created time in 2 days

issue commentkiranshila/Doplarr

Overseerr Support

@VP-EN Yeah, I should probably bold that or something - that seems to be the most common thing people are missing. This is an artifact of the discord slash command permissions nonsense, there isn't really a way around it.

kiranshila

comment created time in 2 days

startedcljs-audio/cljs-audio

started time in 2 days

startedKristofferC/OhMyREPL.jl

started time in 5 days

startedclojure/data.json

started time in 5 days