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

add :trig export tag for trigonometric functions #207

Closed
p6rt opened this issue Jul 31, 2008 · 12 comments
Closed

add :trig export tag for trigonometric functions #207

p6rt opened this issue Jul 31, 2008 · 12 comments

Comments

@p6rt
Copy link

p6rt commented Jul 31, 2008

Migrated from rt.perl.org#57444 (status was 'resolved')

Searchable as RT57444$

@p6rt
Copy link
Author

p6rt commented Jul 31, 2008

From jswitzer@gmail.com

Recent changes to the S29-trig tests have them properly fudged and passing
as of revision 29865. The attached file is the patch to the
spectest_regression to add them to the regression tests.

Thanks,

Jason "s1n" Switzer

@p6rt
Copy link
Author

p6rt commented Jul 31, 2008

From jswitzer@gmail.com

s29-trig.patch
Index: spectest_regression.data
===================================================================
--- spectest_regression.data	(revision 29865)
+++ spectest_regression.data	(working copy)
@@ -116,3 +116,5 @@
 S29-str/sprintf.t
 S29-str/ucfirst.t
 S29-str/uc.t
+S29-trig/e.t
+S29-trig/trig.t

@p6rt
Copy link
Author

p6rt commented Jul 31, 2008

From jswitzer@gmail.com

Check out pmichaud's comment on #​44211. This implies the S29-trig/e.t
test may no longer be a part of STD.

@p6rt
Copy link
Author

p6rt commented Jul 31, 2008

jswitzer@gmail.com - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Jul 31, 2008

From @moritz

jason switzer (via RT) wrote​:

Recent changes to the S29-trig tests have them properly fudged and passing
as of revision 29865. The attached file is the patch to the
spectest_regression to add them to the regression tests.

Thanks for your good work.

However I'm reluctant to add the test for Num​::e because e shouldn't be
available initially. To quote http://perlcabal.org/syn/S29.html#Num

Num provides a number of constants in addition to the basic mathematical
functions. To get these constants, you must request them​:

use Num :constants;

or use the full name, e.g. Num​::pi.

I notice that it's the same issue for pi, but there the probability of
name clashes isn't as high as for e.
So I'll wait for the time when Rakudo implements 'use' meaningfully, and
when the test is re-worked to adhere to the current spec.

I did add trig.t to spectest_regression though (r29906).

Cheers,
Moritz

--
Moritz Lenz
http://moritz.faui2k3.org/ | http://perl-6.de/

@p6rt
Copy link
Author

p6rt commented Nov 13, 2008

From @pmichaud

Can this ticket be closed?

Pm

@p6rt
Copy link
Author

p6rt commented Nov 13, 2008

From @moritz

Patrick R. Michaud via RT wrote​:

Can this ticket be closed?

I'd say we stall it until the :trig tag export works for the
trigonometric functions.

Moritz

--
Moritz Lenz
http://perlgeek.de/ | http://perl-6.de/ | http://sudokugarden.de/

@p6rt
Copy link
Author

p6rt commented Nov 13, 2008

@moritz - Status changed from 'open' to 'stalled'

@p6rt
Copy link
Author

p6rt commented May 1, 2009

From @jnthn

On Thu Nov 13 14​:03​:20 2008, moritz@​casella.verplant.org wrote​:

Patrick R. Michaud via RT wrote​:

Can this ticket be closed?

I'd say we stall it until the :trig tag export works for the
trigonometric functions.

I've seen that we have :Trig working now in p6eval; do we have tests
covering that, and can we now close this?

Thanks,

Jonathan

@p6rt
Copy link
Author

p6rt commented May 1, 2009

The RT System itself - Status changed from 'stalled' to 'open'

@p6rt
Copy link
Author

p6rt commented May 1, 2009

From @jnthn

On Fri May 01 10​:38​:19 2009, jnthn@​jnthn.net wrote​:

On Thu Nov 13 14​:03​:20 2008, moritz@​casella.verplant.org wrote​:

Patrick R. Michaud via RT wrote​:

Can this ticket be closed?

I'd say we stall it until the :trig tag export works for the
trigonometric functions.

I've seen that we have :Trig working now in p6eval; do we have tests
covering that, and can we now close this?

PhatEddy++ just confirmed on IRC that the tests use :Trig; resolving ticket.

Thanks!

Jonathan

@p6rt
Copy link
Author

p6rt commented May 1, 2009

@jnthn - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant