Kobo: Separate series index (text) and series_index (float)?

[chaley]

Originally Posted by ownedbycats

I have "omnibus" in the admintags column.

Then this should work, again assuming that the fractional part is always 2 digits.

program:
	if '^omnibus$' inlist_field '#admintags' then
		series_start = floor($series_index);
		fraction = fractional_part($series_index);
		if fraction ==# 0 then
			result = series_start
		else
			series_end = floor(fraction*100);
			if series_end >=# 0 then
				result = series_start & '-' & series_end
			fi
		fi
	else
		result = $series_index
	fi;
	result

[ownedbycats]

I did some editing to remove 0s from both omnibuses and non-omnibuses. Worked as expected.

program:
	if '^omnibus$' inlist_field '#admintags' 	
		&& $series_index !=#0 
		then
		series_start = floor($series_index);
		fraction = fractional_part($series_index);
		if fraction ==# 0 then
			result = series_start
		else
			series_end = floor(fraction*100);
			if series_end >=# 0 then
				result = series_start & '-' & series_end
			fi
		fi
		elif 
			$series_index ==#0 
		then 
			result = ''
	else
		result = $series_index
	fi;
	result

Note the icons on the last column - signifies omnibus.





I have just one book that uses 6.1 for 6-10 (Calibre removes the trailing 0.) Suggestions? I was thinking of splitting it as I have a few books from the series in physical format anyways.

[chaley]

Originally Posted by ownedbycats

I have just one book that uses 6.1 for 6-10 (Calibre removes the trailing 0.) Suggestions? I was thinking of splitting it as I have a few books from the series in physical format anyways.

The template should produce "10" for "6.1". The fractional part (.1) is multipled by 100, producing "10".

Ahhh, I see. The fractional part of 6.1 can come out as ".099999999999" instead of ".1". To fix it, change line 10 to be

			series_end = round(fraction * 100);

[ownedbycats]

btw, when expected to come out of source? (preview or release.) Building from source is slow, and when I use stable I occasionally forget to change back to preview before connecting.

[chaley]

Originally Posted by ownedbycats

btw, when expected to come out of source? (preview or release.) Building from source is slow, and when I use stable I occasionally forget to change back to preview before connecting.

I submitted the changes to Kovid today. Whether he accepts them and if so, when they are released, is up to him.

Interesting what you say about running from source. I see almost no difference.

[ownedbycats]

It's mostly opening viewer and editor. Takes sometimes 30 seconds or a minute.

[Terisa de morgan]

Originally Posted by ownedbycats

btw, when expected to come out of source? (preview or release.) Building from source is slow, and when I use stable I occasionally forget to change back to preview before connecting.

You can do a different thing. If you know which are the files which have changed, compile them and replace the original ones in a standard installation. And yes, it works.

[kovidgoyal]

I have been travelling for my annual vacation so there will be some time before there is a release of any kind.

[chaley]

The changes are now in calibre source and will be in the next release/preview.

[JSWolf]

Originally Posted by kovidgoyal

I have been travelling for my annual vacation so there will be some time before there is a release of any kind.

Enjoy your vacation. You deserve it.

[ownedbycats]

Originally Posted by JSWolf

Enjoy your vacation. You deserve it.

[theducks]

Originally Posted by JSWolf

Enjoy your vacation. You deserve it.

[JSWolf]

I think this update broke the KoboTouch driver. I have no problem in 8.4.0 but do in 8.5.0. Now I'm getting told that my template for deciding if an ePub should be KePub or not is invalid. The template is working as I've send an ePub and KePub no problem. I'm thinking the error is due to the change for series numbering.

This is what I get when I make any changes to the options or just click OK.

calibre, version 8.5.0
ERROR: Invalid template for Use Kobo viewer for EPUB books: <p>The template "program:
	check_yes_no("#kepub", 0, 0, 1)" is invalid:<br>Interpreter: Internal error evaluating an expression: ''str' object has no attribute 'get'' - line number 2

[chaley]

Originally Posted by JSWolf

I think this update broke the KoboTouch driver. I have no problem in 8.4.0 but do in 8.5.0. Now I'm getting told that my template for deciding if an ePub should be KePub or not is invalid. The template is working as I've send an ePub and KePub no problem. I'm thinking the error is due to the change for series numbering.

This is what I get when I make any changes to the options or just click OK.

calibre, version 8.5.0
ERROR: Invalid template for Use Kobo viewer for EPUB books: <p>The template "program:
	check_yes_no("#kepub", 0, 0, 1)" is invalid:<br>Interpreter: Internal error evaluating an expression: ''str' object has no attribute 'get'' - line number 2

Attachment 216371

This problem has nothing to do with the series template. It was introduced on 10 May (by me) when the driver was changed to validate all templates instead of some of them.

I'm looking at how to fix it now.

[chaley]

Turns out that the changes in May are (mostly) correct. It is illegal to use check_yes_no() outside the GUI, for example in the kobo device driver. It worked by accident.

Replace the template with

program: $#kepub == 'yes'

EDIT: by "mostly" I mean that it detects a real error but gives the wrong message.