Re: [E-devel] Terminology support added on mdcat

[Vinícius dos Santos Oliveira]

Thanks for the chat. This helped a lot.

2018-02-01 6:14 GMT-03:00 Carsten Haitzler :

> On Wed, 31 Jan 2018 22:20:58 -0300 Vinícius dos Santos Oliveira
>  said:
>
> > 2018-01-31 19:37 GMT-03:00 Carsten Haitzler :
> >
> > > tycat doesn't work because it needs to know size before emitting
> escapes
> > > thus
> > > has to load the file and tycat wont bother to download :) but yes
> - the
> > > basic media object can just download to a tmp file itself
> > >
> >
> > In the code I've written, I just set it to occupy half of the terminal
> > height and I use ic escape sequence to not lose the aspect ratio.
>
> h. that's a bit of cheating... but sure. :)
>
> > I think the best solution would be to turn the tycat into a pager, but I
> > don't know what is the author opinion on this.
>
> well i wrote it... :) really the tyls, tycat etc. tools are samples of how
> to
> make the escapes work for something real. they are not serious cmdline tool
> efforts. i always thought that'd ultimately be done by existing cmdline
> tools
> like your mdcat :)
>
> > The second best solution I see is to keep doing what I do.
> >
> >
> > --
> > Vinícius dos Santos Oliveira
> > https://vinipsmaker.github.io/
>
>
> --
> - Codito, ergo sum - "I code, therefore I am" --
> Carsten Haitzler - ras...@rasterman.com
>
>


-- 
Vinícius dos Santos Oliveira
https://vinipsmaker.github.io/
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Carsten Haitzler]

On Wed, 31 Jan 2018 22:20:58 -0300 Vinícius dos Santos Oliveira
 said:

> 2018-01-31 19:37 GMT-03:00 Carsten Haitzler :
> 
> > tycat doesn't work because it needs to know size before emitting escapes
> > thus
> > has to load the file and tycat wont bother to download :) but yes - the
> > basic media object can just download to a tmp file itself
> >
> 
> In the code I've written, I just set it to occupy half of the terminal
> height and I use ic escape sequence to not lose the aspect ratio.

h. that's a bit of cheating... but sure. :)

> I think the best solution would be to turn the tycat into a pager, but I
> don't know what is the author opinion on this.

well i wrote it... :) really the tyls, tycat etc. tools are samples of how to
make the escapes work for something real. they are not serious cmdline tool
efforts. i always thought that'd ultimately be done by existing cmdline tools
like your mdcat :)

> The second best solution I see is to keep doing what I do.
> 
> 
> -- 
> Vinícius dos Santos Oliveira
> https://vinipsmaker.github.io/


-- 
- Codito, ergo sum - "I code, therefore I am" --
Carsten Haitzler - ras...@rasterman.com


--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Cedric Bail]

>  Original Message 
> Subject: [E-devel] Terminology support added on mdcat
> Local Time: January 31, 2018 9:12 AM
> UTC Time: January 31, 2018 5:12 PM
> From: vini.ipsma...@gmail.com
> To: Enlightenment - dev 
>
> Just to let you guys know that support to Terminology extended escape
> sequences has been added to mdcat: https://github.com/lunaryorn/mdcat

That's great, especially with our new documentation being markdown. There is 
just one glitch, it can't find the absolute path being given as they are 
starting with '/'. So I was wondering if there is a way to set some environment 
variable or something to point to a base hierarchy for this image starting by 
'/'.

Cedric
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Vinícius dos Santos Oliveira]

2018-01-31 19:37 GMT-03:00 Carsten Haitzler :

> tycat doesn't work because it needs to know size before emitting escapes
> thus
> has to load the file and tycat wont bother to download :) but yes - the
> basic media object can just download to a tmp file itself
>

In the code I've written, I just set it to occupy half of the terminal
height and I use ic escape sequence to not lose the aspect ratio.

I think the best solution would be to turn the tycat into a pager, but I
don't know what is the author opinion on this.

The second best solution I see is to keep doing what I do.


-- 
Vinícius dos Santos Oliveira
https://vinipsmaker.github.io/
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Carsten Haitzler]

On Wed, 31 Jan 2018 15:47:00 -0300 Vinícius dos Santos Oliveira
 said:

> I've tried to pipe the output to `less`, but it didn't work. I'll test it
> against tmux later.
> 
> 2018-01-31 15:21 GMT-03:00 Carsten Haitzler :
> 
> > On Wed, 31 Jan 2018 14:12:27 -0300 Vinícius dos Santos Oliveira
> >  said:
> >
> > > Just to let you guys know that support to Terminology extended escape
> > > sequences has been added to mdcat: https://github.com/lunaryorn/mdcat
> >
> > that's awesome man!
> >
> > is there anything we can help you with? i just quickly scrolled down
> > https://github.com/lunaryorn/mdcat/pull/16 ...
> >
> > i see 2 issues. 1 is remote references. they woth for popups but not inline
> > (tycat won't work, but typop will).
> 
> 
> They worked on my tests. Maybe new code landed and you missed, but the
> media is discarded as soon as you scroll down and re-downloaded again when
> you scroll up.

tycat doesn't work because it needs to know size before emitting escapes thus
has to load the file and tycat wont bother to download :) but yes - the
basic media object can just download to a tmp file itself

that's why it re-downloads. terminology will delete the media obj and re-create
as you scroll around (to save memory).

you suffer the same issues. needing to know when to keep files or not or
re-download, and to know their size without downloading them first so you know
how to flow text around them... :)

> Caching the media so I don't have to download it every time I scroll down
> or up again would be great.
> 
> even if we did download them... we'd have
> > to STORE them somewhere... because we're sure not going to keep them in
> > RAM. at
> > least stored they don't take precious ram all the time. so terminology will
> > also use tmp files too with references to remote url's anyway. if you do
> > it or
> > terminology does... it won't make much of a difference. :)
> 
> 
> There's one important difference.
> 
> If Terminology creates the files and cache them, then Terminology will keep
> an open fd to these files. If I clear my `/tmp` folder, Terminology will
> still work properly. mdcat can't do that as the application closes as soon
> as the file is printed.

actually like the above scrolling... it won't do that. it just remembers the
uri/file path and re-fetches to tmp file... :) sure - while it's visible there
is an open file handle...

but you could do the same. you would have to not EXIT at the end of the cat,
but wait for a key press to exit. like less/more would do. an interactive pager
built-in...

> the other is
> > fr/fs/fx ... that literally is for file sending. it's zmodem for
> > terminology.
> > tysend is a quick handy tool to run to do this. sample code is really just
> > that
> > - sample code, but it will begin a file send transaction and send file
> > blocks.
> > the terminology side will see the request and pop up a file selector with
> > the
> > ability to abort, and then put up a progress bar. it's not what you want i
> > think.
> >
> 
> Thanks for the explanation.
> 
> -- 
> Vinícius dos Santos Oliveira
> https://vinipsmaker.github.io/


-- 
- Codito, ergo sum - "I code, therefore I am" --
Carsten Haitzler - ras...@rasterman.com


--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Vinícius dos Santos Oliveira]

I've tried to pipe the output to `less`, but it didn't work. I'll test it
against tmux later.

2018-01-31 15:21 GMT-03:00 Carsten Haitzler :

> On Wed, 31 Jan 2018 14:12:27 -0300 Vinícius dos Santos Oliveira
>  said:
>
> > Just to let you guys know that support to Terminology extended escape
> > sequences has been added to mdcat: https://github.com/lunaryorn/mdcat
>
> that's awesome man!
>
> is there anything we can help you with? i just quickly scrolled down
> https://github.com/lunaryorn/mdcat/pull/16 ...
>
> i see 2 issues. 1 is remote references. they woth for popups but not inline
> (tycat won't work, but typop will).


They worked on my tests. Maybe new code landed and you missed, but the
media is discarded as soon as you scroll down and re-downloaded again when
you scroll up.

Caching the media so I don't have to download it every time I scroll down
or up again would be great.

even if we did download them... we'd have
> to STORE them somewhere... because we're sure not going to keep them in
> RAM. at
> least stored they don't take precious ram all the time. so terminology will
> also use tmp files too with references to remote url's anyway. if you do
> it or
> terminology does... it won't make much of a difference. :)


There's one important difference.

If Terminology creates the files and cache them, then Terminology will keep
an open fd to these files. If I clear my `/tmp` folder, Terminology will
still work properly. mdcat can't do that as the application closes as soon
as the file is printed.

the other is
> fr/fs/fx ... that literally is for file sending. it's zmodem for
> terminology.
> tysend is a quick handy tool to run to do this. sample code is really just
> that
> - sample code, but it will begin a file send transaction and send file
> blocks.
> the terminology side will see the request and pop up a file selector with
> the
> ability to abort, and then put up a progress bar. it's not what you want i
> think.
>

Thanks for the explanation.

-- 
Vinícius dos Santos Oliveira
https://vinipsmaker.github.io/
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Vinícius dos Santos Oliveira]

2018-01-31 15:13 GMT-03:00 Gustavo Sverzut Barbieri :

> hey, now you must add the jump marks so he can add those as well ;-)
>

Honestly, I don't think they are very useful, so I wouldn't bother with
them. But if support for these escape sequences is added, then I can code
the feature.

-- 
Vinícius dos Santos Oliveira
https://vinipsmaker.github.io/
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Carsten Haitzler]

On Wed, 31 Jan 2018 14:12:27 -0300 Vinícius dos Santos Oliveira
 said:

> Just to let you guys know that support to Terminology extended escape
> sequences has been added to mdcat: https://github.com/lunaryorn/mdcat

that's awesome man!

is there anything we can help you with? i just quickly scrolled down
https://github.com/lunaryorn/mdcat/pull/16 ...

i see 2 issues. 1 is remote references. they woth for popups but not inline
(tycat won't work, but typop will). even if we did download them... we'd have
to STORE them somewhere... because we're sure not going to keep them in RAM. at
least stored they don't take precious ram all the time. so terminology will
also use tmp files too with references to remote url's anyway. if you do it or
terminology does... it won't make much of a difference. :) the other is
fr/fs/fx ... that literally is for file sending. it's zmodem for terminology.
tysend is a quick handy tool to run to do this. sample code is really just that
- sample code, but it will begin a file send transaction and send file blocks.
the terminology side will see the request and pop up a file selector with the
ability to abort, and then put up a progress bar. it's not what you want i
think.

> -- 
> Vinícius dos Santos Oliveira
> https://vinipsmaker.github.io/
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


-- 
- Codito, ergo sum - "I code, therefore I am" --
Carsten Haitzler - ras...@rasterman.com


--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Gustavo Sverzut Barbieri]

hey, now you must add the jump marks so he can add those as well ;-)

On Wed, Jan 31, 2018 at 4:08 PM, Vinícius dos Santos Oliveira
 wrote:
> PS.: details about the support are commented in this PR:
> https://github.com/lunaryorn/mdcat/pull/16
>
> 2018-01-31 14:12 GMT-03:00 Vinícius dos Santos Oliveira <
> vini.ipsma...@gmail.com>:
>
>> Just to let you guys know that support to Terminology extended escape
>> sequences has been added to mdcat: https://github.com/lunaryorn/mdcat
>>
>> --
>> Vinícius dos Santos Oliveira
>> https://vinipsmaker.github.io/
>>
>
>
>
> --
> Vinícius dos Santos Oliveira
> https://vinipsmaker.github.io/
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel



-- 
Gustavo Sverzut Barbieri
--
Mobile: +55 (16) 99354-9890

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Vinícius dos Santos Oliveira]

PS.: details about the support are commented in this PR:
https://github.com/lunaryorn/mdcat/pull/16

2018-01-31 14:12 GMT-03:00 Vinícius dos Santos Oliveira <
vini.ipsma...@gmail.com>:

> Just to let you guys know that support to Terminology extended escape
> sequences has been added to mdcat: https://github.com/lunaryorn/mdcat
>
> --
> Vinícius dos Santos Oliveira
> https://vinipsmaker.github.io/
>



-- 
Vinícius dos Santos Oliveira
https://vinipsmaker.github.io/
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Re: [E-devel] Terminology support added on mdcat

[Boris Faure]

On 18-01-31 14:12, Vinícius dos Santos Oliveira wrote:
> Just to let you guys know that support to Terminology extended escape
> sequences has been added to mdcat: https://github.com/lunaryorn/mdcat

That's neat! Thanks for letting us know about it!
-- 
Boris Faure
Pointer Arithmetician


signature.asc
Description: Digital signature
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel