Re: [Jmol-developers] Escape.e function misidentifying P4 types as T3...

2015-12-31 Thread Jonathan Gutow
Good idea.  It will speed things up slightly.  I will do it.

Jonathan

On Wed, Dec 30, 2015 at 7:52 PM, Robert Hanson  wrote:

> Go ahead the change that Escape.e to Escape.eP4 anyway, and check that in
> if you want.
>
> On Wed, Dec 30, 2015 at 3:02 PM, Jonathan Gutow  wrote:
>
>> I can verify that the reordering fixed the problem.
>>
>> Jonathan
>>
>> On Wed, Dec 30, 2015 at 10:50 AM, Robert Hanson 
>> wrote:
>>
>>> I checked in a new Escape.java, which I guess fixes the problem. Do an
>>> update on that, Jonathan. I guess we could leave the SurfaceTool the way it
>>> is then.
>>>
>>> On Wed, Dec 30, 2015 at 10:46 AM, Robert Hanson 
>>> wrote:
>>>
 In SurfaceTool.java, this:

   cmd.append(" plane ").append(Escape.e(plane))


 should read:

   cmd.append(" plane ").append(Escape.eP4(plane))

 similar with:

 .append(Escape.e(slice.leftPlane));
 .append(Escape.e(slice.rightPlane));

 and in Escape.java:

 if (x instanceof T3)
   return eP((T3) x);
 if (x instanceof P4)
   return eP4((P4) x);


 should read

 if (x instanceof P4)
   return eP4((P4) x);
 if (x instanceof T3)
   return eP((T3) x);


 On Tue, Dec 29, 2015 at 7:59 PM, Jonathan Gutow 
 wrote:

> It appears that at line 117 in Escape.java, type P4 objects are being
> misidentified as type T3.  This means that the value of the "w" subtype is
> not being returned as part of the returned text vector only "x", "y" and
> "z".  I noticed this because it breaks the surface tool which uses P4
> objects to define planes.
>
> It is not clear to me why this is happening.  Is P4 type a subtype of
> T3?
>
> Jonathan
>
> --
>Dr. Jonathan H. Gutow
> Chemistry Department gu...@uwosh.edu
> UW-OshkoshOffice:
> 920-424-1326
> 800 Algoma Boulevard FAX:920-424-2042
> Oshkosh, WI 54901
>http://www.uwosh.edu/facstaff/gutow/
>
>
> --
>
> ___
> Jmol-developers mailing list
> Jmol-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>
>


 --
 Robert M. Hanson
 Larson-Anderson Professor of Chemistry
 Chair, Department of Chemistry
 St. Olaf College
 Northfield, MN
 http://www.stolaf.edu/people/hansonr


 If nature does not answer first what we want,
 it is better to take what answer we get.

 -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900


>>>
>>>
>>> --
>>> Robert M. Hanson
>>> Larson-Anderson Professor of Chemistry
>>> Chair, Department of Chemistry
>>> St. Olaf College
>>> Northfield, MN
>>> http://www.stolaf.edu/people/hansonr
>>>
>>>
>>> If nature does not answer first what we want,
>>> it is better to take what answer we get.
>>>
>>> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>>>
>>>
>>>
>>> --
>>>
>>> ___
>>> Jmol-developers mailing list
>>> Jmol-developers@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>>>
>>>
>>
>>
>> --
>>Dr. Jonathan H. Gutow
>> Chemistry Department gu...@uwosh.edu
>> UW-OshkoshOffice:
>> 920-424-1326
>> 800 Algoma Boulevard FAX:920-424-2042
>> Oshkosh, WI 54901
>>http://www.uwosh.edu/facstaff/gutow/
>>
>>
>> --
>>
>> ___
>> Jmol-developers mailing list
>> Jmol-developers@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>>
>>
>
>
> --
> Robert M. Hanson
> Larson-Anderson Professor of Chemistry
> Chair, Department of Chemistry
> St. Olaf College
> Northfield, MN
> http://www.stolaf.edu/people/hansonr
>
>
> If nature does not answer first what we want,
> it is better to take what answer we get.
>
> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>
>
>
> --
>
> ___
> Jmol-developers mailing list
> Jmol-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>
>


-- 
   Dr. Jonathan H. Gutow

Re: [Jmol-developers] Escape.e function misidentifying P4 types as T3...

2015-12-30 Thread Robert Hanson
It's an ordering issue. Yes, P4 subclasses T4, which subclasses T3. I'll
fix it.

On Tue, Dec 29, 2015 at 7:59 PM, Jonathan Gutow  wrote:

> It appears that at line 117 in Escape.java, type P4 objects are being
> misidentified as type T3.  This means that the value of the "w" subtype is
> not being returned as part of the returned text vector only "x", "y" and
> "z".  I noticed this because it breaks the surface tool which uses P4
> objects to define planes.
>
> It is not clear to me why this is happening.  Is P4 type a subtype of T3?
>
> Jonathan
>
> --
>Dr. Jonathan H. Gutow
> Chemistry Department gu...@uwosh.edu
> UW-OshkoshOffice:
> 920-424-1326
> 800 Algoma Boulevard FAX:920-424-2042
> Oshkosh, WI 54901
>http://www.uwosh.edu/facstaff/gutow/
>
>
> --
>
> ___
> Jmol-developers mailing list
> Jmol-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>
>


-- 
Robert M. Hanson
Larson-Anderson Professor of Chemistry
Chair, Department of Chemistry
St. Olaf College
Northfield, MN
http://www.stolaf.edu/people/hansonr


If nature does not answer first what we want,
it is better to take what answer we get.

-- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
--
___
Jmol-developers mailing list
Jmol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jmol-developers


Re: [Jmol-developers] Escape.e function misidentifying P4 types as T3...

2015-12-30 Thread Robert Hanson
I checked in a new Escape.java, which I guess fixes the problem. Do an
update on that, Jonathan. I guess we could leave the SurfaceTool the way it
is then.

On Wed, Dec 30, 2015 at 10:46 AM, Robert Hanson  wrote:

> In SurfaceTool.java, this:
>
>   cmd.append(" plane ").append(Escape.e(plane))
>
>
> should read:
>
>   cmd.append(" plane ").append(Escape.eP4(plane))
>
> similar with:
>
> .append(Escape.e(slice.leftPlane));
> .append(Escape.e(slice.rightPlane));
>
> and in Escape.java:
>
> if (x instanceof T3)
>   return eP((T3) x);
> if (x instanceof P4)
>   return eP4((P4) x);
>
>
> should read
>
> if (x instanceof P4)
>   return eP4((P4) x);
> if (x instanceof T3)
>   return eP((T3) x);
>
>
> On Tue, Dec 29, 2015 at 7:59 PM, Jonathan Gutow  wrote:
>
>> It appears that at line 117 in Escape.java, type P4 objects are being
>> misidentified as type T3.  This means that the value of the "w" subtype is
>> not being returned as part of the returned text vector only "x", "y" and
>> "z".  I noticed this because it breaks the surface tool which uses P4
>> objects to define planes.
>>
>> It is not clear to me why this is happening.  Is P4 type a subtype of T3?
>>
>> Jonathan
>>
>> --
>>Dr. Jonathan H. Gutow
>> Chemistry Department gu...@uwosh.edu
>> UW-OshkoshOffice:
>> 920-424-1326
>> 800 Algoma Boulevard FAX:920-424-2042
>> Oshkosh, WI 54901
>>http://www.uwosh.edu/facstaff/gutow/
>>
>>
>> --
>>
>> ___
>> Jmol-developers mailing list
>> Jmol-developers@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>>
>>
>
>
> --
> Robert M. Hanson
> Larson-Anderson Professor of Chemistry
> Chair, Department of Chemistry
> St. Olaf College
> Northfield, MN
> http://www.stolaf.edu/people/hansonr
>
>
> If nature does not answer first what we want,
> it is better to take what answer we get.
>
> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>
>


-- 
Robert M. Hanson
Larson-Anderson Professor of Chemistry
Chair, Department of Chemistry
St. Olaf College
Northfield, MN
http://www.stolaf.edu/people/hansonr


If nature does not answer first what we want,
it is better to take what answer we get.

-- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
--
___
Jmol-developers mailing list
Jmol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jmol-developers


Re: [Jmol-developers] Escape.e function misidentifying P4 types as T3...

2015-12-30 Thread Jonathan Gutow
I wondered about that.  I figured you might know.  Thanks for dealing with
this.

Jonathan

On Wed, Dec 30, 2015 at 10:37 AM, Robert Hanson  wrote:

> It's an ordering issue. Yes, P4 subclasses T4, which subclasses T3. I'll
> fix it.
>
> On Tue, Dec 29, 2015 at 7:59 PM, Jonathan Gutow  wrote:
>
>> It appears that at line 117 in Escape.java, type P4 objects are being
>> misidentified as type T3.  This means that the value of the "w" subtype is
>> not being returned as part of the returned text vector only "x", "y" and
>> "z".  I noticed this because it breaks the surface tool which uses P4
>> objects to define planes.
>>
>> It is not clear to me why this is happening.  Is P4 type a subtype of T3?
>>
>> Jonathan
>>
>> --
>>Dr. Jonathan H. Gutow
>> Chemistry Department gu...@uwosh.edu
>> UW-OshkoshOffice:
>> 920-424-1326
>> 800 Algoma Boulevard FAX:920-424-2042
>> Oshkosh, WI 54901
>>http://www.uwosh.edu/facstaff/gutow/
>>
>>
>> --
>>
>> ___
>> Jmol-developers mailing list
>> Jmol-developers@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>>
>>
>
>
> --
> Robert M. Hanson
> Larson-Anderson Professor of Chemistry
> Chair, Department of Chemistry
> St. Olaf College
> Northfield, MN
> http://www.stolaf.edu/people/hansonr
>
>
> If nature does not answer first what we want,
> it is better to take what answer we get.
>
> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>
>
>
> --
>
> ___
> Jmol-developers mailing list
> Jmol-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>
>


-- 
   Dr. Jonathan H. Gutow
Chemistry Department gu...@uwosh.edu
UW-Oshkosh
Office:920-424-1326
800 Algoma Boulevard FAX:920-424-2042
Oshkosh, WI 54901
   http://www.uwosh.edu/facstaff/gutow/
--
___
Jmol-developers mailing list
Jmol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jmol-developers


Re: [Jmol-developers] Escape.e function misidentifying P4 types as T3...

2015-12-30 Thread Jonathan Gutow
I can verify that the reordering fixed the problem.

Jonathan

On Wed, Dec 30, 2015 at 10:50 AM, Robert Hanson  wrote:

> I checked in a new Escape.java, which I guess fixes the problem. Do an
> update on that, Jonathan. I guess we could leave the SurfaceTool the way it
> is then.
>
> On Wed, Dec 30, 2015 at 10:46 AM, Robert Hanson 
> wrote:
>
>> In SurfaceTool.java, this:
>>
>>   cmd.append(" plane ").append(Escape.e(plane))
>>
>>
>> should read:
>>
>>   cmd.append(" plane ").append(Escape.eP4(plane))
>>
>> similar with:
>>
>> .append(Escape.e(slice.leftPlane));
>> .append(Escape.e(slice.rightPlane));
>>
>> and in Escape.java:
>>
>> if (x instanceof T3)
>>   return eP((T3) x);
>> if (x instanceof P4)
>>   return eP4((P4) x);
>>
>>
>> should read
>>
>> if (x instanceof P4)
>>   return eP4((P4) x);
>> if (x instanceof T3)
>>   return eP((T3) x);
>>
>>
>> On Tue, Dec 29, 2015 at 7:59 PM, Jonathan Gutow  wrote:
>>
>>> It appears that at line 117 in Escape.java, type P4 objects are being
>>> misidentified as type T3.  This means that the value of the "w" subtype is
>>> not being returned as part of the returned text vector only "x", "y" and
>>> "z".  I noticed this because it breaks the surface tool which uses P4
>>> objects to define planes.
>>>
>>> It is not clear to me why this is happening.  Is P4 type a subtype of T3?
>>>
>>> Jonathan
>>>
>>> --
>>>Dr. Jonathan H. Gutow
>>> Chemistry Department gu...@uwosh.edu
>>> UW-OshkoshOffice:
>>> 920-424-1326
>>> 800 Algoma Boulevard FAX:920-424-2042
>>> Oshkosh, WI 54901
>>>http://www.uwosh.edu/facstaff/gutow/
>>>
>>>
>>> --
>>>
>>> ___
>>> Jmol-developers mailing list
>>> Jmol-developers@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>>>
>>>
>>
>>
>> --
>> Robert M. Hanson
>> Larson-Anderson Professor of Chemistry
>> Chair, Department of Chemistry
>> St. Olaf College
>> Northfield, MN
>> http://www.stolaf.edu/people/hansonr
>>
>>
>> If nature does not answer first what we want,
>> it is better to take what answer we get.
>>
>> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>>
>>
>
>
> --
> Robert M. Hanson
> Larson-Anderson Professor of Chemistry
> Chair, Department of Chemistry
> St. Olaf College
> Northfield, MN
> http://www.stolaf.edu/people/hansonr
>
>
> If nature does not answer first what we want,
> it is better to take what answer we get.
>
> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>
>
>
> --
>
> ___
> Jmol-developers mailing list
> Jmol-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>
>


-- 
   Dr. Jonathan H. Gutow
Chemistry Department gu...@uwosh.edu
UW-Oshkosh
Office:920-424-1326
800 Algoma Boulevard FAX:920-424-2042
Oshkosh, WI 54901
   http://www.uwosh.edu/facstaff/gutow/
--
___
Jmol-developers mailing list
Jmol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jmol-developers


Re: [Jmol-developers] Escape.e function misidentifying P4 types as T3...

2015-12-30 Thread Robert Hanson
Go ahead the change that Escape.e to Escape.eP4 anyway, and check that in
if you want.

On Wed, Dec 30, 2015 at 3:02 PM, Jonathan Gutow  wrote:

> I can verify that the reordering fixed the problem.
>
> Jonathan
>
> On Wed, Dec 30, 2015 at 10:50 AM, Robert Hanson 
> wrote:
>
>> I checked in a new Escape.java, which I guess fixes the problem. Do an
>> update on that, Jonathan. I guess we could leave the SurfaceTool the way it
>> is then.
>>
>> On Wed, Dec 30, 2015 at 10:46 AM, Robert Hanson 
>> wrote:
>>
>>> In SurfaceTool.java, this:
>>>
>>>   cmd.append(" plane ").append(Escape.e(plane))
>>>
>>>
>>> should read:
>>>
>>>   cmd.append(" plane ").append(Escape.eP4(plane))
>>>
>>> similar with:
>>>
>>> .append(Escape.e(slice.leftPlane));
>>> .append(Escape.e(slice.rightPlane));
>>>
>>> and in Escape.java:
>>>
>>> if (x instanceof T3)
>>>   return eP((T3) x);
>>> if (x instanceof P4)
>>>   return eP4((P4) x);
>>>
>>>
>>> should read
>>>
>>> if (x instanceof P4)
>>>   return eP4((P4) x);
>>> if (x instanceof T3)
>>>   return eP((T3) x);
>>>
>>>
>>> On Tue, Dec 29, 2015 at 7:59 PM, Jonathan Gutow  wrote:
>>>
 It appears that at line 117 in Escape.java, type P4 objects are being
 misidentified as type T3.  This means that the value of the "w" subtype is
 not being returned as part of the returned text vector only "x", "y" and
 "z".  I noticed this because it breaks the surface tool which uses P4
 objects to define planes.

 It is not clear to me why this is happening.  Is P4 type a subtype of
 T3?

 Jonathan

 --
Dr. Jonathan H. Gutow
 Chemistry Department gu...@uwosh.edu
 UW-OshkoshOffice:
 920-424-1326
 800 Algoma Boulevard FAX:920-424-2042
 Oshkosh, WI 54901
http://www.uwosh.edu/facstaff/gutow/


 --

 ___
 Jmol-developers mailing list
 Jmol-developers@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/jmol-developers


>>>
>>>
>>> --
>>> Robert M. Hanson
>>> Larson-Anderson Professor of Chemistry
>>> Chair, Department of Chemistry
>>> St. Olaf College
>>> Northfield, MN
>>> http://www.stolaf.edu/people/hansonr
>>>
>>>
>>> If nature does not answer first what we want,
>>> it is better to take what answer we get.
>>>
>>> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>>>
>>>
>>
>>
>> --
>> Robert M. Hanson
>> Larson-Anderson Professor of Chemistry
>> Chair, Department of Chemistry
>> St. Olaf College
>> Northfield, MN
>> http://www.stolaf.edu/people/hansonr
>>
>>
>> If nature does not answer first what we want,
>> it is better to take what answer we get.
>>
>> -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
>>
>>
>>
>> --
>>
>> ___
>> Jmol-developers mailing list
>> Jmol-developers@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>>
>>
>
>
> --
>Dr. Jonathan H. Gutow
> Chemistry Department gu...@uwosh.edu
> UW-OshkoshOffice:
> 920-424-1326
> 800 Algoma Boulevard FAX:920-424-2042
> Oshkosh, WI 54901
>http://www.uwosh.edu/facstaff/gutow/
>
>
> --
>
> ___
> Jmol-developers mailing list
> Jmol-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jmol-developers
>
>


-- 
Robert M. Hanson
Larson-Anderson Professor of Chemistry
Chair, Department of Chemistry
St. Olaf College
Northfield, MN
http://www.stolaf.edu/people/hansonr


If nature does not answer first what we want,
it is better to take what answer we get.

-- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900
--
___
Jmol-developers mailing list
Jmol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jmol-developers


[Jmol-developers] Escape.e function misidentifying P4 types as T3...

2015-12-29 Thread Jonathan Gutow
It appears that at line 117 in Escape.java, type P4 objects are being
misidentified as type T3.  This means that the value of the "w" subtype is
not being returned as part of the returned text vector only "x", "y" and
"z".  I noticed this because it breaks the surface tool which uses P4
objects to define planes.

It is not clear to me why this is happening.  Is P4 type a subtype of T3?

Jonathan

-- 
   Dr. Jonathan H. Gutow
Chemistry Department gu...@uwosh.edu
UW-Oshkosh
Office:920-424-1326
800 Algoma Boulevard FAX:920-424-2042
Oshkosh, WI 54901
   http://www.uwosh.edu/facstaff/gutow/
--
___
Jmol-developers mailing list
Jmol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jmol-developers