Ouch.. why use an <a> tag? Do this instead:
<%= link_to image_tag(pick.image_url1, :width => 90, :height => 90, :border => 10), :controller => "admin", :action => "show", :id => pick %>
Oh, by the way, I'd say @picks = Product.find_person(session[:user_id], :order => "id").reverse is better than doing @pics.sort_by later on.
Whats the output of @picks.inspect?