Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

error in process sentinel: ement--resize-image wrong type argument #95

Closed
gcentauri opened this issue Sep 9, 2022 · 5 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@gcentauri
Copy link

gcentauri commented Sep 9, 2022

Login with my user at my homeserver (running dendrite 0.9.6) open a room buffer, I start seeing this error happening periodically:

error in process sentinel: ement--resize-image: Wrong type argument: consp, nil
error in process sentinel: Wrong type argument: consp, nil

(fboundp 'imagemagick-types) returns nil for me

Using Emacs 28.1 and just installed the stable version of ement.el

@gcentauri gcentauri changed the title error in process sentilel: ement--resize-image wrong type argument error in process sentinel: ement--resize-image wrong type argument Sep 9, 2022
@alphapapa
Copy link
Owner

Hi Grant,

Thanks. Could you enable debug-on-error and show me the backtrace when it happens again? Also, what build of Emacs do you have installed, i.e. from where?

@gcentauri
Copy link
Author

Yeah, idk what was going on yesterday but toggle-debug-on-error wasn't triggering. I restarted things and got a backtrace this time though.

Debugger entered--Lisp error: (wrong-type-argument consp nil)
  image--set-property(nil :max-height 29)
  ement--resize-image(nil nil 29)
  ement-room-list--entry(#s(ement-session :user ... :server ... :token "B-enBYb93doLPP..." :transaction-id 3267306750 :rooms ... :next-batch "s9058_7124_172..." :device-id "lLOMC1-6" :initial-device-display-name nil :has-synced-p t :account-data ... :events #<hash-table equal 492/730 0x159271abe115>) #s(ement-room :id "!yWuuGCTDPPX6v..." :display-name "emacs" :prev-batch "t604_9995" :summary ... :state ... :timeline ... :ephemeral ... :account-data ... :unread-notifications ... :latest-ts 1662763202809 :topic "endless muckin..." :canonical-alias "#emacs:cicadas..." :avatar ... :status join ...))
  #f(compiled-function (room) #<bytecode 0x1b0092cf010c7d49>)(#s(ement-room :id "!yWuuGCTDPPX6vuI3:ci..." :display-name "emacs" :prev-batch "t604_9995" :summary (... ...) :state (... ... ... ... ... ... ... ... ... ... ... ...) :timeline (... ... ... ... ... ... ... ... ... ... ... ... ... ... ... ... ... ... ... ...) :ephemeral (... ...) :account-data (...) :unread-notifications (... ...) :latest-ts 1662763202809 :topic "endless mucking abou..." :canonical-alias "#emacs:cicadas.surf" :avatar #(" " 0 1 ...) :status join :type nil :invite-state nil :members #<hash-table equal 2/65 0x159271a1622f> :local (... ... ...) :receipts #<hash-table equal 2/65 0x159271a1624f> :displaynames #<hash-table eql 2/65 0x159271a18e65>))
  ement-room-list--set-entries()
  tabulated-list-revert(nil nil)
  revert-buffer()
  ement-room-list-auto-update(#s(ement-session :user ... :server ... :token "B-enBYb93doLPP..." :transaction-id 3267306750 :rooms ... :next-batch "s9058_7124_172..." :device-id "lLOMC1-6" :initial-device-display-name nil :has-synced-p t :account-data ... :events #<hash-table equal 492/730 0x159271abe115>))
  run-hook-with-args(ement-room-list-auto-update #s(ement-session :user ... :server ... :token "B-enBYb93doLPP..." :transaction-id 3267306750 :rooms ... :next-batch "s9058_7124_172..." :device-id "lLOMC1-6" :initial-device-display-name nil :has-synced-p t :account-data ... :events #<hash-table equal 492/730 0x159271abe115>))
  ement--sync-callback(#s(ement-session :user ... :server ... :token "B-enBYb93doLPP..." :transaction-id 3267306750 :rooms ... :next-batch "s9058_7124_172..." :device-id "lLOMC1-6" :initial-device-display-name nil :has-synced-p t :account-data ... :events #<hash-table equal 492/730 0x159271abe115>) (... ... ... ... ... ...))
  apply(ement--sync-callback (... ...))
  #f(compiled-function (&rest args2) #<bytecode -0xae4d49652b789dd>)(((next_batch . "s9058_7124_17253_3629_54_20079_6199_3865_0") (account_data) (presence) (rooms) (to_device) (device_lists)))
  #f(compiled-function () #<bytecode 0xedd51cdaa20f7ec>)()
  plz--sentinel(#<process plz-request-curl<5>> "finished\n")

There's a lot of folded data in there, but I drilled down into the room bit:

#s(ement-room :id "!yWuuGCTDPPX6vuI3:cicadas.surf" :display-name "emacs" :prev-batch "t604_9995" :summary ((m\.invited_member_count . 0) (m\.joined_member_count . 2)) :state ... :timeline ... :ephemeral ... :account-data ... :unread-notifications ... :latest-ts 1662763202809 :topic "endless muckin..." :canonical-alias "#emacs:cicadas.surf" :avatar #(" " 0 1 (display nil))

This is a room on my homeserver, and it looks like the avatar bit is coming back nil.

My build of Emacs is from source, the 28.1 tag on the main repo.

@gcentauri
Copy link
Author

I've been able to suppress the error for the moment by turning off the ement-room-list-avatars

@alphapapa
Copy link
Owner

Thanks.

Does that room have an avatar? If so, does it appear properly in other clients?

@alphapapa alphapapa added the bug Something isn't working label Sep 29, 2022
@alphapapa alphapapa self-assigned this Sep 29, 2022
@alphapapa
Copy link
Owner

This was likely fixed by #147. If this happens again, please open a new issue. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants