Skip to content

Add cbs branching for compact table problems.#1

Open
CarolineSabourin-Jean wants to merge 1 commit intoSaGagnon:upirfrom
CarolineSabourin-Jean:upir
Open

Add cbs branching for compact table problems.#1
CarolineSabourin-Jean wants to merge 1 commit intoSaGagnon:upirfrom
CarolineSabourin-Jean:upir

Conversation

@CarolineSabourin-Jean
Copy link

No description provided.

auto SendMarginal = [this](unsigned int prop_id, unsigned int var_id,
int val, double dens) {
if (!varIdToPos.isIn(var_id)) return;

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avez-vous trouvé pourquoi il y a des variables dans le propagateur qui ne se retrouvent pas dans le brancher? J'avais ajouté ce fix pour cette raison et je ne sais pas si c'est un comportement normal.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non on n'a pas trouvé, désolée :(

// We can change rho's value. Actually, there is no memory exception
// with rho = 1. But with other values of rho, there is a memory
// exception for some crossword problems.
float rho = 1;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gilles m'a dit qu'il y avait des fuites de mémoire avec rho < 1. Je vais regarder ça.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ouais exact :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants