← Back to team overview

instant team mailing list archive

Re: What remains before we can release Instant 1.0 ?

 

There is some small stuff in the TODO file.
And there are some TODO and FIXMEs in the code.
This is probably overlapping.
In particular, there are some old arguments
that may be removed or must be explained.


TODO:
- Fix arguments not used in setup.py

- Remove unused csrcs stuff.

- Explain arrays argument.

- Use object files argument.

- Improve documentation

- Add tests for all variants of cache mechanisms available now

- Clean up imports, don't use import *


martinal@martinal-xps:~/dev/fenics/instant/src/instant$ egrep FIXME *
build.py:          FIXME: Describe this correctly. Tests pass arrays
of arrays of strings.
build.py:        instant_error("FIXME: setup.py doesn't use the C sources.")
codegeneration.py:      - additional_definitions (FIXME: comment)
codegeneration.py:      - additional_declarations (FIXME: comment)
codegeneration.py:      - arrays (FIXME: comment)
codegeneration.py:    #instant_warning("FIXME: Not using csrcs in
write_setupfile().")
codegeneration.py:    arrays = [] # FIXME: Example input here
martinal@martinal-xps:~/dev/fenics/instant/src/instant$ egrep TODO *
build.py:# TODO: Import only the official interface
build.py:        # TODO: Error instead? Indicates race condition
build.py:        - If you want to compile the files yourself. TODO:
Not yet supported.
cache.py:# TODO: We could make this an argument, but it's used
indirectly several places so take care.
__init__.py:# TODO: Import only the official interface
inlining.py:    # TODO: Something more robust? Regexp?
inlining.py:    # TODO: This isn't very general!
inlining.py:    # TODO: This isn't very general!
inlining.py:    # TODO: Is the second and third path here necessary?
inlining.py:    # TODO: Is the second and third path here necessary?


Martin



On Tue, Jan 13, 2009 at 9:59 AM, Kent Andre <kent-and@xxxxxxxxx> wrote:
>
> As always, improved documentation and better tests.
> Anything else ?
>
> Kent
>
> _______________________________________________
> Instant-dev mailing list
> Instant-dev@xxxxxxxxxx
> http://fenics.org/mailman/listinfo/instant-dev
>


Follow ups

References