Sequelize association count producing incorrect SQL and results

With the following query:

models.School.findOne({
  where: { id },
  include: [{ attributes: [], model: models.Teacher }, { attributes: [], model: models.Pupil }],
  attributes: [
    'School.*',
    [Sequelize.fn('COUNT', Sequelize.col('Teachers.id')), 'teachers'],
    [Sequelize.fn('COUNT', Sequelize.col('Pupils.id')), 'pupils']
  ]
})

In the model definitions I have a belongsTo/haveMany association set up e.g.:

School.hasMany(models.Teacher, {
  foreignKey: 'schoolId',
  allowNull: false
});

School.hasMany(models.Pupil, {
  foreignKey: 'schoolId',
  allowNull: false
});

Teacher.belongsTo(models.School, {
  foreignKey: 'schoolId',
  allowNull: false
});

Pupil.belongsTo(models.School, {
  foreignKey: 'schoolId',
  allowNull: false
});

The SQL that gets produced is:

SELECT `School`.`id`,
`School`.*,
COUNT(`Teachers`.`id`) AS `teachers`,
COUNT(`Pupils`.`id`) AS `pupils` 
FROM `Schools` AS `School`
LEFT OUTER JOIN `Teachers` AS `Teachers` ON `School`.`id` = `Teachers`.`school_id` AND `Teachers`.`deletedAt` IS NULL 
LEFT OUTER JOIN `Pupils` AS `Pupils` ON `School`.`id` = `Pupils`.`school_id` AND `Pupils`.`deletedAt` IS NULL 
WHERE (`School`.`deletedAt` IS NULL AND `School`.`id` = '34fa61f7-d078-4560-a81d-4351d5ce5179');

If I run this query in my app, all I get back is an object like this:

{
    teachers: 200,
    pupils: 200
}

However if I run this in my SQL app I get back the row with all fields for a school, but I get two ID columns. The other thing is there is only 100 pupils and 2 teachers associated with that school (and I can confirm this by running a single count against the table directly with the above school_id).

Any ideas?

[tanepiper]

Fixed: Had to do this:

models.School.findOne({
        where: { id },
        include: [{ attributes: [], model: models.Teacher }, { attributes: [], model: models.Pupil }],
        attributes: {
          include: [
            [Sequelize.fn('COUNT', Sequelize.fn('DISTINCT', Sequelize.col('Teachers.id'))), 'teachers'],
            [Sequelize.fn('COUNT', Sequelize.fn('DISTINCT', Sequelize.col('Pupils.id'))), 'pupils']
          ]
        }
      })

[janmeier]

There is no way to do that I think - because it would break the limit - you would be limiting the number of rows from the cartesian join, not the number of schools.

You can maybe try adding the counts to the includes for teacher and pupil - Not sure if it will work, but that will at least remove them from the inner query