Grand. I didn't intend to throw you guys a whole lot of work, but if a
newcomer's first experience of analysis is such a user-unfriendly error
message, they might be discouraged.
On Thu, 19 Mar 2009, Wayne Boucher wrote:
> OK, I've added a more sensible error message in this one context. (I hope
> "alphanumeric" means something to everyone.)
>
> Wayne
>
> On Thu, 19 Mar 2009, Wayne Boucher wrote:
>
>> I agree with Rasmus. As it happens, though, we already check that the
>> project name has no white space before tossing it to the API so we could
>> also add in the other constraints in this specific context. This code
>> duplicates the API checks of course but perhaps worth it in this context.
>>
>> Wayne
>>
>> On Thu, 19 Mar 2009, Rasmus Fogh wrote:
>>
>>> Dear Dan,
>>>
>>> That could be done - in that particular instance. The problem is whether
>>> we want to fix only this error message, or all the other similar ones that
>>> appear in other contexts.
>>>
>>> Yours,
>>>
>>> Rasmus
>>>
>>>
>>> ---------------------------------------------------------------------------
>>> Dr. Rasmus H. Fogh Email: [log in to unmask]
>>> Dept. of Biochemistry, University of Cambridge,
>>> 80 Tennis Court Road, Cambridge CB2 1GA, UK. FAX (01223)766002
>>>
>>> On Thu, 19 Mar 2009, Daniel O'Donovan wrote:
>>>
>>>> I don't know if this is what Brian meant, but maybe Analysis could parse
>>>> the suggested project name for badness and then pop up an error box if
>>>> bad rather than sending a bad name to the data model (and thence the ugly
>>>> output)..
>>>>
>>>> My two p,
>>>>
>>>> Dan
>>>>
>>>> On 19 Mar 2009, at 09:51, Rasmus Fogh wrote:
>>>>
>>>>> Dear All,
>>>>>
>>>>> It would not be that easy to do. We have constraints on objects,
>>>>> attributes, and data types. Constraints on DataTypes, especially, could
>>>>> appear in all kinds of contexts, so when you write the actual constraint
>>>>> you may know very little about the context you are dealing with. And we
>>>>> woudl not want to handcode error messages for every single attribute of
>>>>> type Word, for example.
>>>>>
>>>>> We could try to make ti a bit more user friendly, but it is too big a
>>>>> job to be done any time soon.
>>>>>
>>>>> Yours,
>>>>>
>>>>> Rasmus
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------------
>>>>> Dr. Rasmus H. Fogh Email: [log in to unmask]
>>>>> Dept. of Biochemistry, University of Cambridge,
>>>>> 80 Tennis Court Road, Cambridge CB2 1GA, UK. FAX (01223)766002
>>>>>
>>>>> On Wed, 18 Mar 2009, Wayne Boucher wrote:
>>>>>
>>>>>> We figured we'd want to scare you enough so that you never, ever did it
>>>>>> again.
>>>>>>
>>>>>> Well, the serious point is that that error message is the generic kind
>>>>>> of message we get from the CCPN API. In Analysis we don't try and turn
>>>>>> it into a more sensible error message because that would be a big
>>>>>> translation job (i.e. lots of things could go wrong so we'd need to do
>>>>>> a big translation job). Now whether the API could produce more user
>>>>>> friendly error messages is a question for Rasmus (that would still be a
>>>>>> big job).
>>>>>>
>>>>>> Wayne
>>>>>>
>>>>>> On Wed, 18 Mar 2009, Brian Smith wrote:
>>>>>>
>>>>>>> Got the following traceback when I mistakenly tried to create a new
>>>>>>> project with a "-" in the name. Maybe a less scary warning would be
>>>>>>> nice?
>>>>>>> Traceback (most recent call last):
>>>>>>> File "/usr/lib/python2.4/lib-tk/Tkinter.py", line 1345, in __call__
>>>>>>> return self.func(*args)
>>>>>>> File
>>>>>>> "/usr/local/ccpnmr/temporaryReleaseDir/ccpnmr/ccpnmr2.0/python/ccpnmr/analysis/AnalysisPopup.py",
>>>>>>> line 1117, in newProject
>>>>>>> project = Impl.MemopsRoot(name=name)
>>>>>>> File
>>>>>>> "/usr/local/ccpnmr/temporaryReleaseDir/ccpnmr/ccpnmr2.0/python/memops/api/Implementation.py",
>>>>>>> line 5794, in __init__
>>>>>>> File
>>>>>>> "/usr/local/ccpnmr/temporaryReleaseDir/ccpnmr/ccpnmr2.0/python/memops/api/Implementation.py",
>>>>>>> line 7939, in checkValid
>>>>>>> ApiError: memops.Implementation.MemopsRoot.checkValid: name:
>>>>>>> name constraint value_is_alphanumeric_or_underscore violated by
>>>>>>> value:
>>>>>>> Dr. Brian O. Smith ---------------------- B Smith at bio gla ac uk
>>>>>>> Division of Molecular & Cellular Biology,
>>>>>>> Faculty of Biomedical & Life Sciences,
>>>>>>> Joseph Black Building, University of Glasgow, Glasgow G12 8QQ, UK.
>>>>>>> Tel: 0141 330 5167/6459/3089 Fax: 0141 330 4600
>>>>>>> -------------------------------------------------------------------
>>>>>>> The University of Glasgow, charity number SC004401
>>>>>>
>>>>
>>>> Daniel O'Donovan
>>>> [log in to unmask]
>>>
>>
>
Dr. Brian O. Smith ---------------------- B Smith at bio gla ac uk
Division of Molecular & Cellular Biology,
Faculty of Biomedical & Life Sciences,
Joseph Black Building, University of Glasgow, Glasgow G12 8QQ, UK.
Tel: 0141 330 5167/6459/3089 Fax: 0141 330 4600
-------------------------------------------------------------------
The University of Glasgow, charity number SC004401
|