larryprice / mongoose-simple-random Goto Github PK

(node:1824) DeprecationWarning: collection.count is deprecated, and will be removed in a future version. Use collection.countDocuments or collection.estimatedDocumentCount instead

for code:

const items_random = await Item.findRandom({}, {}, {limit: 6}, function(err, docs) { return docs });

Please update. Thanks

I have a collection with 10 documents. This will only select one of the first 9 randomly. Or, 100 documents and it will only select one of the first 99.

  var start = Math.max(0, Math.floor((num-limit)*Math.random()));

might be written as

  var start = Math.max(0, Math.floor(((num-limit)+1)*Math.random()));

I didn't test.

I use async\await, but I got this " Cannot read property 'skip' of undefined "

If I call findRandom with options that return no results, the query hangs indefinitely.

how to use async/await??? I use it's warning : callback is not function

Would I be right in thinking that the "find many" function selects the first document randomly but the rest are simply those documents that directly follow it in mongo's sequence?

If so, it might be good to document that clearly. For some applications it would be an undesirable limitation. (For example pulling 5 random playing cards from an unshuffled deck would nearly always return a straight run!)

Given that built in $sample methods work much more efficiently, would you consider unpublishing, or maybe changing mongoose-simple-random to take advantage of such improvements?

Any sort of pagination for this?