Opened 7 years ago

Closed 6 years ago

#1292 closed defect (fixed)

Registration code not handling some date formats

Reported by: rk97@… Owned by: Conrad Huang
Priority: normal Milestone:
Component: Infrastructure Version:
Keywords: Cc:
Blocked By: Blocking:
Notify when closed: Platform: all
Project: ChimeraX

Description

The following bug report has been submitted:
Platform:        Linux-4.17.0-9-generic-x86_64-with-debian-buster-sid
ChimeraX Version: 0.8 (2018-09-01)
Description
The current version of chimerax deas not recognize "common" extensions of *.pdb1 and *.ent. Files with these extension were recognized by previous versions of chimerax software. It would be great if this functionality (i.e. the ability to read files with pdb1 and ent extensions) was restored. 

Log:
Startup Messages  
---  
warning | Traceback (most recent call last):  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/site-
packages/chimerax/core/toolshed/info.py", line 411, in initialize  
api._api_caller.initialize(api, session, self)  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/site-
packages/chimerax/core/toolshed/__init__.py", line 1198, in initialize  
return cls._get_func(api, "initialize")(session, bi)  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/site-
packages/chimerax/registration/__init__.py", line 41, in initialize  
nag(session)  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/site-
packages/chimerax/registration/nag.py", line 29, in nag  
_check_usage(session)  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/site-
packages/chimerax/registration/nag.py", line 167, in _check_usage  
usage = _get_usage()  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/site-
packages/chimerax/registration/nag.py", line 204, in _get_usage  
usage["dates"].append(datetime.strptime(value, TimeFormat))  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/_strptime.py", line 565, in
_strptime_datetime  
tt, fraction = _strptime(data_string, format)  
File "/usr/lib/ucsf-chimerax-daily/lib/python3.6/_strptime.py", line 362, in
_strptime  
(data_string, format))  
ValueError: time data 'Tue Jan 30 11:47:14 2018' does not match format '%a %b
%d %H:%M:%S %Y'  
note | available bundle cache has not been initialized yet  
  
UCSF ChimeraX version: 0.8 (2018-09-01)  
How to cite UCSF ChimeraX  

open /home/rajmund/Downloads/pdb5v8m.ent

Unrecognized file suffix '.ent'  

open /home/rajmund/Desktop/3w1k.pdb1

Unrecognized file suffix '.pdb1'  

Unrecognized file suffix '.pdb1'  

open /home/rajmund/Desktop/3w1k.pdb1

Unrecognized file suffix '.pdb1'  

open /home/rajmund/Downloads/pdb5v8m.ent

Unrecognized file suffix '.ent'  

open /home/rajmund/Desktop/3w1k.pdb1

Unrecognized file suffix '.pdb1'  

Unrecognized file suffix '.pdb1'  




OpenGL version: 3.3.0 NVIDIA 390.87
OpenGL renderer: GeForce GTX 970/PCIe/SSE2
OpenGL vendor: NVIDIA Corporation
File attachment: 3k3a.pdb1

3k3a.pdb1

Attachments (3)

3k3a.pdb1 (1.1 MB ) - added by rk97@… 7 years ago.
Added by email2trac
preregistration (164 bytes ) - added by rk97@… 7 years ago.
Added by email2trac
printenv.out (3.3 KB ) - added by rk97@… 7 years ago.
Added by email2trac

Download all attachments as: .zip

Change History (12)

by rk97@…, 7 years ago

Attachment: 3k3a.pdb1 added

Added by email2trac

comment:1 by Eric Pettersen, 7 years ago

Component: UnassignedInput/Output
Owner: set to Eric Pettersen
Platform: all
Project: ChimeraX
Status: newaccepted
Summary: ChimeraX bug report submissionopen .pdb1 et al

comment:2 by Eric Pettersen, 7 years ago

Thanks for reporting this. We hadn't noticed that the ability to open those files had been lost during some code reorganization. The fix will be in the next daily build.

The log included with your bug report indicates another error, so I will be changing this ticket to one describing that error.

comment:3 by Eric Pettersen, 7 years ago

Component: Input/OutputInfrastructure
Owner: changed from Eric Pettersen to Conrad Huang
Status: acceptedassigned
Summary: open .pdb1 et alRegistration code not handling some date formats

comment:4 by Conrad Huang, 7 years ago

Status: assignedfeedback

This is very odd. When I try parsing the same date string using the same code I do not get an error:

>>> x = datetime.strptime("Tue Jan 30 11:47:14 2018", "%a %b %d %H:%M:%S %Y")
>>> x
datetime.datetime(2018, 1, 30, 11, 47, 14)

The only thing I can think of is that the locale has changed and the day or month abbreviation no longer matches what is in the usage file. Can you please send me two items:

  • the file named "preregistration" in the directory reported by the ChimeraX command:
    info path user unversioned data
    
  • the output of the Unix command:
    printenv
    

Thanks.

in reply to:  6 comment:5 by rk97@…, 7 years ago

Dear conrad,

I am sending the "preregistration" file (attached to this message), as well as the ptintenv output as you requested in your message.

Best regards
Rajmund Kazmierkiewicz


W dniu 2018-09-11 23:14:55 użytkownik ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu> napisał:


preregistration

printenv.out

by rk97@…, 7 years ago

Attachment: preregistration added

Added by email2trac

by rk97@…, 7 years ago

Attachment: printenv.out added

Added by email2trac

comment:6 by Conrad Huang, 7 years ago

Hmm. I'm stuck. The preregistration file works fine on my machine, and changing the locale (language and language conventions) made no difference on our Linux machine. Do you get that traceback every time ChimeraX starts? If so, can you try removing the preregistration file and running ChimeraX again to see if the problem continues to happen? Thanks.

in reply to:  10 comment:7 by rk97@…, 7 years ago

Dear conrad

I did remove preregistration and chimerax starts without any complains (at least I do not see any comment in the terminal when I start chimerax in the command line). It still does not open *.pdb1 and *.ent files.

Best regards
Rajmund Kazmierkiewicz 

W dniu 2018-09-12 23:37:25 użytkownik ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu> napisał:



in reply to:  11 comment:8 by Conrad Huang, 7 years ago

Can you please try the current daily build?  Yesterday's build failed 
and therefore did not pick up the fix.

Conrad

On 9/13/2018 12:45 AM, rk97 wrote:

comment:9 by Conrad Huang, 6 years ago

Resolution: fixed
Status: feedbackclosed

Fixed in 0062071d.

Note: See TracTickets for help on using tickets.