Merge remote-tracking branch 'origin/text_auto_width' into auto_width

Author: duffyjp

.gitignore

@@ -1,4 +1,5 @@
 .ruby-version
+.idea/
 try.c
 try.rb
 gen.rb
@@ -11,6 +12,6 @@ libxlsxwriter/dev
 libxlsxwriter/docs
 libxlsxwriter/examples
 libxlsxwriter/test
-
+ext/fast_excel/text_width_ext.o
 ext/fast_excel/Makefile
 ext/fast_excel/text_width_ext.bundle

Makefile

@@ -4,6 +4,8 @@ Q=
 endif
 
 all :
+	@echo "Compiling ext/text_width ..."
+	rake compile
 	@echo "Compiling libxlsxwriter ..."
 	$(Q)$(MAKE) -C libxlsxwriter
 

Rakefile

@@ -22,3 +22,11 @@ task :examples do
     require './' + file.sub(/\.rb$/, '')
   end
 end
+
+task :compile do
+  %x{
+    cd ext/fast_excel
+    ruby ./extconf.rb
+    make
+  }
+end

benchmarks/auto_width.rb

@@ -0,0 +1,37 @@
+require_relative 'init'
+
+HEADERS = ["id", "name", "age", "date"]
+
+DATA = []
+1000.times do |n|
+  DATA << [n, "String string #{n}", (n * rand * 10).round, Time.at(n * 1000 + 1492922688)]
+end
+
+Benchmark.ips do |x|
+  x.config(time: 10, warmup: 2)
+
+  x.report("Normal") do
+    workbook = FastExcel.open(constant_memory: true)
+    worksheet = workbook.add_worksheet("benchmark")
+
+    worksheet.write_row(0, HEADERS)
+    DATA.each_with_index do |row, i|
+      worksheet.write_row(i + 1, row)
+    end
+    workbook.read_string
+  end
+
+  x.report("With auto_width") do
+    workbook = FastExcel.open(constant_memory: true)
+    worksheet = workbook.add_worksheet("benchmark")
+    worksheet.auto_width = true
+
+    worksheet.write_row(0, HEADERS)
+    DATA.each_with_index do |row, i|
+      worksheet.write_row(i + 1, row)
+    end
+    workbook.read_string
+  end
+
+  x.compare!
+end

benchmarks/profiler.rb

@@ -0,0 +1,27 @@
+require_relative '../lib/fast_excel'
+require 'ruby-prof'
+
+DATA = []
+1000.times do |n|
+  DATA << [n, "String string #{n}", (n * rand * 10).round, Time.at(n * 1000 + 1492922688)]
+end
+
+RubyProf.start
+
+100.times do
+  workbook = FastExcel.open(constant_memory: true)
+  worksheet = workbook.add_worksheet("benchmark")
+  worksheet.auto_width = true
+
+  DATA.each_with_index do |row, i|
+    worksheet.write_row(i + 1, row)
+  end
+  workbook.read_string
+
+  print '.'
+end
+result = RubyProf.stop
+
+# print a flat profile to text
+printer = RubyProf::FlatPrinter.new(result)
+printer.print(STDOUT)

benchmarks/write_value.rb

@@ -0,0 +1,62 @@
+require_relative '../lib/fast_excel'
+require "benchmark/memory"
+require "benchmark/ips"
+
+DATA = []
+1000.times do |n|
+  DATA << [n, "String string #{n}", (n * rand * 10).round, Time.at(n * 1000 + 1492922688)]
+end
+
+5.times do
+  workbook = FastExcel.open(constant_memory: true)
+  worksheet = workbook.add_worksheet("benchmark")
+
+  DATA.each_with_index do |row, row_num|
+    row.each_with_index do |val, cell_num|
+      worksheet.write_value(row_num + 1, cell_num + 1, val)
+    end
+  end
+  workbook.read_string
+end
+
+
+workbook = FastExcel.open(constant_memory: true)
+worksheet = workbook.add_worksheet("benchmark")
+
+DATA.each_with_index do |row, row_num|
+  row.each_with_index do |val, cell_num|
+    worksheet.write_value(row_num + 1, cell_num + 1, val)
+  end
+end
+workbook.read_string
+
+Benchmark.ips do |x|
+  #x.config(time: 10, warmup: 2)
+
+  x.report("Normal") do
+    workbook = FastExcel.open(constant_memory: true)
+    worksheet = workbook.add_worksheet("benchmark")
+
+    DATA.each_with_index do |row, row_num|
+      row.each_with_index do |val, cell_num|
+        worksheet.write_value(row_num + 1, cell_num + 1, val)
+      end
+    end
+    workbook.read_string
+  end
+
+  x.report("Auto-width") do
+    workbook = FastExcel.open(constant_memory: true)
+    worksheet = workbook.add_worksheet("benchmark")
+    worksheet.auto_width = true
+
+    DATA.each_with_index do |row, row_num|
+      row.each_with_index do |val, cell_num|
+        worksheet.write_value(row_num + 1, cell_num + 1, val)
+      end
+    end
+    workbook.read_string
+  end
+
+  x.compare!
+end

examples/example_auto_width.rb

@@ -0,0 +1,26 @@
+require_relative '../lib/fast_excel'
+
+workbook = FastExcel.open("example_auto_width.xlsx", constant_memory: false)
+
+# this is required to make auto-width works correctly
+workbook.default_format.set(
+  font_size: 13,
+  font_family: "Arial"
+)
+
+worksheet = workbook.add_worksheet
+worksheet.auto_width = true
+
+['Arial', 'Calibri', 'Times New Roman'].each_with_index do |font, index|
+  col_format = workbook.add_format(font_family: font, font_size: 17)
+  worksheet.set_column(index * 4, index * 4 + 3, 10, col_format)
+
+  worksheet.write_value(0, index * 4 + 2, font)
+  worksheet.write_value(1, index * 4, "tini")
+  worksheet.write_value(1, index * 4 + 1, "Longer")
+  worksheet.write_value(1, index * 4 + 2, "Some longer text!")
+  worksheet.write_value(1, index * 4 + 3, "This gem is FFI binding for libxlsxwriter C library")
+end
+
+workbook.close
+puts "Saved to file example_auto_width.xlsx"

ext/fast_excel/extconf.rb

@@ -0,0 +1,3 @@
+require 'mkmf'
+
+create_makefile('ext/text_width_ext')