[prev in list] [next in list] [prev in thread] [next in thread]
List: nepomuk
Subject: Re: [Nepomuk] Nepomuk File Metadata widget
From: Vishesh Handa <me () vhanda ! in>
Date: 2012-12-20 15:45:55
Message-ID: CAOPTMKDBMt3n4RRU8AreO00XdgBW=Hb9o5JyMR32HWgq3k-DZQ () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
On Thu, Dec 20, 2012 at 8:47 PM, Frank Reininghaus \
<frank78ac@googlemail.com
> wrote:
> Hi Vishesh,
>
> 2012/12/20 Vishesh Handa:
> > Hey Frank
> >
> > I know I'm very very late, but I finally fixed the few annoying bugs in
> > Nepomuk2::FileMetadataWidget, and even made it show data when Nepomuk \
> > in
> not
> > running.
>
> Sounds good, thanks!
>
> > I think this satisfies all the conditions that you had.
> >
> > Would it be possible to use it for 4.10? I would really like it cause
> then I
> > don't have to fix the bugs in KFileMetadatWidget for 4.10. I've \
> > attached
> a
> > patch which uses it instead of the KFileMetadataWidget.
> >
> > Current problems -
> >
> > 1. When right clicking on the file -> Properties -> Information. This
> > Information is supposed to be the same as what is displayed in the
> > Information Panel. However, its code resides in kio, and cannot be made
> to
> > use Nepomuk::FileMetadataWidget.
> >
> > 2. Configure Shown Data -> Internally, Nepomuk's FileMetadataWidget \
> > also uses the same config file, but this might have problems. Maybe I \
> > should
> have
> > my own different config file.
> >
> > I really want to get this into 4.10 cause it makes my job a lot easier. \
> > I don't need to maintain the kdelibs/nepomuk code then. It would also \
> > make
> bug
> > fixing a lot easier.
> >
> > What do you think?
>
> We are in hard feature freeze since November 8 [1], so it seems to me
> that this can only go into the master branch.
>
Well. I was hoping you wouldn't consider this a feature. It's just a port
of KFileMetadataWidget to Nepomuk2.
At the end, it's your call. There are significant advantages of moving to
the new MetadataWidget. Since the previous one used a separate process, it
could not utilize any of the cache, and had to query the db each time a
file was selected. Considering that Dolphin internally uses Nepomuk to show
a lot of metadata, the data would already be loaded in memory and we
wouldn't have to query for it again and again.
I hate the idea of users having to wait 8 months to get these improvements.
Anyway, it's your choice. I do hope you reconsider.
> Best regards,
> Frank
>
> [1]
> http://techbase.kde.org/Schedules/KDE4/4.10_Release_Schedule#Thursday.2C_November_8.2C_2012:_KDE_SC_4.10_Hard_Feature_Freeze
>
--
Vishesh Handa
[Attachment #5 (text/html)]
<div dir="ltr"><br><div class="gmail_extra"><br><br><div \
class="gmail_quote">On Thu, Dec 20, 2012 at 8:47 PM, Frank Reininghaus \
<span dir="ltr"><<a href="mailto:frank78ac@googlemail.com" \
target="_blank">frank78ac@googlemail.com</a>></span> wrote:<br> \
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex">Hi Vishesh,<br> <br>
2012/12/20 Vishesh Handa:<br>
<div class="im">> Hey Frank<br>
><br>
> I know I'm very very late, but I finally fixed the few annoying \
bugs in<br> > Nepomuk2::FileMetadataWidget, and even made it show data \
when Nepomuk in not<br> > running.<br>
<br>
</div>Sounds good, thanks!<br>
<div class="im"><br>
> I think this satisfies all the conditions that you had.<br>
><br>
> Would it be possible to use it for 4.10? I would really like it cause \
then I<br> > don't have to fix the bugs in KFileMetadatWidget for \
4.10. I've attached a<br> > patch which uses it instead of the \
KFileMetadataWidget.<br> ><br>
> Current problems -<br>
><br>
> 1. When right clicking on the file -> Properties -> Information. \
This<br> > Information is supposed to be the same as what is displayed \
in the<br> > Information Panel. However, its code resides in kio, and \
cannot be made to<br> > use Nepomuk::FileMetadataWidget.<br>
><br>
> 2. Configure Shown Data -> Internally, Nepomuk's \
FileMetadataWidget also<br> > uses the same config file, but this might \
have problems. Maybe I should have<br> > my own different config \
file.<br> ><br>
> I really want to get this into 4.10 cause it makes my job a lot \
easier. I<br> > don't need to maintain the kdelibs/nepomuk code \
then. It would also make bug<br> > fixing a lot easier.<br>
><br>
> What do you think?<br>
<br>
</div>We are in hard feature freeze since November 8 [1], so it seems to \
me<br> that this can only go into the master \
branch.<br></blockquote><div><br></div><div>Well. I was hoping you \
wouldn't consider this a feature. It's just a port of \
KFileMetadataWidget to Nepomuk2.<br><br></div><div>At the end, it's \
your call. There are significant advantages of moving to the new \
MetadataWidget. Since the previous one used a separate process, it could \
not utilize any of the cache, and had to query the db each time a file was \
selected. Considering that Dolphin internally uses Nepomuk to show a lot of \
metadata, the data would already be loaded in memory and we wouldn't \
have to query for it again and again.<br> <br></div><div>I hate the idea of \
users having to wait 8 months to get these improvements. Anyway, it's \
your choice. I do hope you reconsider.<br><br></div><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">
<br>
Best regards,<br>
Frank<br>
<br>
[1] <a href="http://techbase.kde.org/Schedules/KDE4/4.10_Release_Schedule#Thursday.2C_November_8.2C_2012:_KDE_SC_4.10_Hard_Feature_Freeze" \
target="_blank">http://techbase.kde.org/Schedules/KDE4/4.10_Release_Schedule#Thursday.2C_November_8.2C_2012:_KDE_SC_4.10_Hard_Feature_Freeze</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><span \
style="color:rgb(192,192,192)">Vishesh Handa</span><br> </div></div>
_______________________________________________
Nepomuk mailing list
Nepomuk@kde.org
https://mail.kde.org/mailman/listinfo/nepomuk
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic