Welcome to Doom9's Forum, THE in-place to be for everyone interested in DVD conversion. Before you start posting please read the forum rules. By posting to this forum you agree to abide by the rules. |
2nd September 2009, 08:33 | #22 | Link | |
Avisynth Developer
Join Date: Jan 2003
Location: Melbourne, Australia
Posts: 3,167
|
@Forteen88,
Quote:
All, Time is precious so I am skipping the frills for expedience sake. I am also restricting my participation in this forum, don't feel offended if I appear to be ignoring you, I am applying self discipline and not responding to anybody ( I am skimming the posts every few days for any active items). This way I actually have time to work on the code. As I saidAnd I guess the caveat is for scripts using 2.6 features what sort of collateral are we looking at. |
|
4th September 2009, 12:21 | #23 | Link | ||
Moderator
Join Date: Nov 2001
Location: Netherlands
Posts: 6,364
|
Quote:
Quote:
Last edited by Wilbert; 4th September 2009 at 12:27. |
||
4th September 2009, 15:40 | #25 | Link |
AviSynth plugger
Join Date: Nov 2003
Location: Russia
Posts: 2,183
|
There is an opinion to move all filters from core to separate plugin(s).
__________________
My Avisynth plugins are now at http://avisynth.org.ru and mirror at http://avisynth.nl/users/fizick I usually do not provide a technical support in private messages. |
7th September 2009, 18:02 | #26 | Link |
the dumbest
Join Date: Oct 2002
Location: Malvinas
Posts: 494
|
Yes please, that would make keeping the project up to date much easier.
I Guess that moving internal( now basic set of) filters to external plug-ins would help if we at last move to higher quality/precision color depth. |
7th September 2009, 19:37 | #27 | Link |
Resize Abuser
Join Date: Apr 2005
Location: Seattle, WA
Posts: 623
|
morsa has a great point. This means that avisynth would no longer be able to say "batteries included", but with the big changes with color depth, ect... it might be the smart thing to do. Either way moving the never used functions to various plugins is 100% smart IMHO.
__________________
Mine: KenBurnsEffect/ZoomBox CutFrames Helped: DissolveAGG ColorBalance LQ Animation Fixer |
8th September 2009, 03:16 | #28 | Link |
x264aholic
Join Date: Jul 2007
Location: New York
Posts: 1,752
|
I agree. It seems like the core could be simplified quite a bit if you just create a "legacy" plugin of sorts that contains older, underused filters, along with additional plugins that contain the more frequently used functions.
__________________
You can't call your encoding speed slow until you start measuring in seconds per frame. |
18th September 2009, 21:31 | #30 | Link |
Compiling Encoder
Join Date: Jan 2007
Posts: 1,348
|
Hmmm there were originally a large portion of
function definitions that are useful for plugin writers were defined in avisynth.h however with the latest 2.6 merge, avisynth.h has turned into purely prototype-based code and the definitions wandered into interface.cpp (For those nonprogrammers that don't completely understand what I'm saying, basically means that avisynth.h was an all-in-one-package and is no longer so) so are you changing it to that we should now have interface.cpp in the plugin code to be able to successfully write plugins with the new system? |
18th September 2009, 23:18 | #31 | Link | ||
Moderator
Join Date: Nov 2001
Location: Netherlands
Posts: 6,364
|
Quote:
Quote:
I'm sure i will be corrected if i got it wrong. Last edited by Wilbert; 18th September 2009 at 23:22. |
||
18th September 2009, 23:52 | #32 | Link |
Registered User
Join Date: Mar 2002
Posts: 1,075
|
I don't think it's ready yet (still hard coded, no function pointers) but yes, in the future you are meant to add an extra source file to your plugin projects (or link a LIB, but really just adding the file is easier and less prone to getting out of date ... a LIB would just be an useless extra build step). I told them they could just make it an .ICBIAH (I can't believe it's a header) file and #include it in avisynth.h ... but they didn't agree with me
The intention is to have the stubs in interface.cpp use function pointers passed from avisynth proper (the DLL, so the functions can be changed in future versions and have the plugins use them). Last edited by MfA; 19th September 2009 at 00:03. |
19th September 2009, 00:09 | #33 | Link |
Avisynth Developer
Join Date: Jan 2003
Location: Melbourne, Australia
Posts: 3,167
|
As Wilbert said, except linking to avisynth.lib is not a serious contender. The 2 leading contenders for plugin linkage are funktors or stub routines, with a struct of function pointers, either packaged as macros in avisynth.h or a new module for plugins to include. I have an awk script to generate the initial text but I have another dose of additions/changes to apply to avisynth.h before I commit the function linkage, so it will be a few weeks before this last part gets some CVS cycles.
Refer to the "Baked Code" thread in the "Avisynth Development" forum for previously discussed details Oh! Snap MfA |
19th September 2009, 09:35 | #34 | Link |
Registered User
Join Date: Aug 2007
Posts: 374
|
I think there should be no imports in plugins from avisynth core. More importantly i think that plugin source recompilation from 2.5 to 2.6 must not be trivial as there should be requirement for 2.6 filters to be thread safe and specify themselves correct MTMode, setting video cache hints also should be mandatory (but probably in different notation than now).
|
27th September 2009, 11:20 | #35 | Link | |
Registered User
Join Date: Feb 2008
Posts: 733
|
Just installed it
Quote:
|
|
27th September 2009, 13:19 | #36 | Link | |
Moderator
Join Date: Nov 2001
Location: Netherlands
Posts: 6,364
|
Quote:
Perhaps i will add this in Immaavs or Imagewriter. |
|
27th September 2009, 15:17 | #38 | Link | ||||||||
Avisynth Developer
Join Date: Jan 2003
Location: Melbourne, Australia
Posts: 3,167
|
Okay, the Alpha 2 release. See post 1 above for full details.
Highlights * New Softwire. * Massive speed up in Vertical resizer on all platforms. * ConditionalReader/WriteFile full refactor. * Nearly all experimental branch code reviewed and imported. (One more push should do it) Get AviSynth_090927.exe (1.1MiB) from SourceForge. ------------ Quote:
------------ Quote:
Quote:
Quote:
Quote:
Quote:
Quote:
Last edited by IanB; 28th September 2009 at 01:16. |
||||||||
27th September 2009, 15:35 | #39 | Link |
Compiling Encoder
Join Date: Jan 2007
Posts: 1,348
|
i noticed the update last night when i did a CVS pull, some minor compilation-based issues that should be fixed with the next push are:
Code:
convert_planar.cpp: PVideoFrame __stdcall ConvertToY8::GetFrame() { ... const srcMod = srcPitch + (vi.width * pixel_step); should instead be const int srcMod = srcPitch + (vi.width * pixel_step); conditional_reader.h: class ConditionalReader : public GenericVideoFilter { ... CleanUp(void); should instead be void CleanUp(void); conditional_reader.cpp: ConditionalReader::CleanUp(void) should be void ConditionalReader::CleanUp(void) |
|
|