[R/export.R:122:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L122) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{lcPerimWeight} \tab \code{lcpPerWt} \tab Cost length of link from patch perimeters\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:123:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L123) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{startPerimX} \tab \code{strtPerX} \tab Coordinate of link endpoint on first patch (X)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:124:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L124) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{startPerimY} \tab \code{strtPerY} \tab Coordinate of link endpoint on first patch (Y)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:125:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L125) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{endPerimX} \tab \code{endPerX} \tab Coordinate of link endpoint on second patch (X)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:126:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L126) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{endPerimY} \tab \code{endPerY} \tab Coordinate of link endpoint on second patch (Y)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:142:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L142) *style:* **Lines should not be more than 100 characters.**
```r
#'   node \tab \code{polygonArea} \tab \code{polyA} \tab Area of polygon from grain voronoi raster \cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:144:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L144) *style:* **Lines should not be more than 100 characters.**
```r
#'   node \tab \code{totalPatchEdgeArea} \tab \code{patchEA} \tab Total area of all patch edges in polygon\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:145:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/export.R#L145) *style:* **Lines should not be more than 100 characters.**
```r
#'   node \tab \code{totalCoreArea} \tab \code{coreA} \tab Total area of patches in polygon excluding edges\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/RcppExports.R:26:1:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/R/RcppExports.R#L26) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-MPG.R:73:4:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/tests/testthat/test-MPG.R#L73) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:83:4:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/tests/testthat/test-MPG.R#L83) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:93:4:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/tests/testthat/test-MPG.R#L93) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:101:4:](https://github.com/achubaty/grainscape/blob/8bb0503be062446aef4bd3b79fc733ac3bc7d347/tests/testthat/test-MPG.R#L101) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


@kay0u \O/


@jfescobar18 what about having a CleanUpContribution() Method that deletes contributions to be called at the beggining of using() statement, (line 23).


@cass-cabrera We can actually shorten this by changing our query to only find questions that match the student username. Always good to think, "Is there anything I'm doing inefficiently here?". In this case, it's getting all the questions when we just need a subset.

This will require adding a new query to `QueryFactory`. You have two options here:

1. You add a query that accepts a student as an argument and returns questions from that student. Then, you run that query for all students.  The downside is that then you might have to send out 57 requests.
2. Better option, the query accepts an array of students, and the query searches for all questions that match any of the students. 👍🏼


@cass-cabrera this is a big change so let me know when you work on it and push the commit up!


It can't be called question because of our other model class "question" , the names would conflict


Copy of this build can be found at
https://builds.myetherwallet.com/build/3598e0230932b251805736763d9e09c36413b57c/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmXQTGuCLYszMz1A4jeUby2753sozz5aFDVv3QfPd3qjgw
                       Virus Total analysis
                       https://www.virustotal.com/file/1700822898b9d35759d0cf227d76f9c2cb592951577535d8a66cc7b6d982eea9/analysis/1564782575/


is it going to be able to solve ?? I hope so because you are excellent in all this


updateman.c it will display Hello Hoberton 32 times.


The following doc(s) have been deployed to the [localdevli](https://localdevli-static-docs.pantheonsite.io/docs) Multidev environment:
- [source/content/localdev.md](https://github.com/pantheon-systems/documentation/commit/84ba2d17383ec4f667e24bab6f9c5f4fac1bda5d/source/content/localdev.md)
- [source/content/redis.md](https://github.com/pantheon-systems/documentation/commit/68a38e4e211bfd837b85f89f8748d3069bb33074/source/content/redis.md)


returning false means that I don't want to override the default behavior that happens when you close a search view, which includes clearing the search text


TeamCity Glyssen / Glyssen pull requests [Build *.*.412.b47570a274dbbbf6105c44379121b837eec4336a](https://build.palaso.org/viewLog.html?buildId=158849&buildTypeId=Glyssen_GlyssenPullRequests2) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 2257, ignored: 19 Build time: 00:09:02
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateFromParatextData_ProjectHasCustomCharacterVerseDecisions_UserDecisionsReapplied("Boaz"): Test(s) failed. Test exceeded Timeout value of 8000ms

```


TeamCity Glyssen / Glyssen pull requests [Build *.*.413.9c8fd7add3e94757ec3f91c0dc695a84eeb7bd8a](https://build.palaso.org/viewLog.html?buildId=158850&buildTypeId=Glyssen_GlyssenPullRequests2) outcome was **SUCCESS**
Summary: Tests passed: 2169, ignored: 18 Build time: 00:09:03


Successfully aliased the URL <https://site-f8nlzp681.now.sh> to the following aliases:

* https://hivib.es
* https://site.azurikai.now.sh
* https://site-git-master.azurikai.now.sh


Let's go with `Questions` then.

Same goes for the other classes inside `QueryFactory`. Let's drop the `Query` suffix, and to match convention with `Questions` and to escape conflicts, let's make them plural


@jkallini what is the difference between `mMessages` and `mAllMessages`?


Successfully aliased the URL <https://jordans-game-of-life-4ghqi4vsu.now.sh> to the following aliases:

* https://jordans-game-of-life.jordan-stoddard.now.sh
* https://jordans-game-of-life-git-master.jordan-stoddard.now.sh


Successfully aliased the URL <https://wigglyprickles-react-4h4qj48to.now.sh> to the following aliases:

* https://wigglyprickles.com
* https://wigglyprickles-react.pricklywiggles.now.sh
* https://wigglyprickles-react-git-master.pricklywiggles.now.sh


fixes #49 - cocked up releasing - been a while...  actual fix back in .17


Este es mi villano favorito


Copy of this build can be found at
https://builds.myetherwallet.com/build/3d277388dab21a6657b6791b152e7a9b4ccb7e4f/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmNTzNpbqK8BQatjhf9JuPQUDVwhxbfUxf4HchqSqd8T62
                       Virus Total analysis
                       https://www.virustotal.com/file/7630ce460e6daba6efcb910d69cd99d653f050a9c188a1ccc7909fdc0c2b627d/analysis/1564783027/


The following doc(s) have been deployed to the [gatsby](https://gatsby-static-docs.pantheonsite.io/docs) Multidev environment:


Absolutely! I wrote this and pushed it quickly in order to fix a bug a found when pulling new changes. I'll make this prettier right now.


# Notas
Entiendo porque agregaste a deadshto


[R/export.R:122:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L122) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{lcPerimWeight} \tab \code{lcpPerWt} \tab Cost length of link from patch perimeters\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:123:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L123) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{startPerimX} \tab \code{strtPerX} \tab Coordinate of link endpoint on first patch (X)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:124:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L124) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{startPerimY} \tab \code{strtPerY} \tab Coordinate of link endpoint on first patch (Y)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:125:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L125) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{endPerimX} \tab \code{endPerX} \tab Coordinate of link endpoint on second patch (X)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:126:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L126) *style:* **Lines should not be more than 100 characters.**
```r
#'   link \tab \code{endPerimY} \tab \code{endPerY} \tab Coordinate of link endpoint on second patch (Y)\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:142:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L142) *style:* **Lines should not be more than 100 characters.**
```r
#'   node \tab \code{polygonArea} \tab \code{polyA} \tab Area of polygon from grain voronoi raster \cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:144:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L144) *style:* **Lines should not be more than 100 characters.**
```r
#'   node \tab \code{totalPatchEdgeArea} \tab \code{patchEA} \tab Total area of all patch edges in polygon\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/export.R:145:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/export.R#L145) *style:* **Lines should not be more than 100 characters.**
```r
#'   node \tab \code{totalCoreArea} \tab \code{coreA} \tab Total area of patches in polygon excluding edges\cr
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/RcppExports.R:26:1:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/R/RcppExports.R#L26) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-MPG.R:73:4:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/tests/testthat/test-MPG.R#L73) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:83:4:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/tests/testthat/test-MPG.R#L83) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:93:4:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/tests/testthat/test-MPG.R#L93) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:101:4:](https://github.com/achubaty/grainscape/blob/51c7aec25a6808130783c14410ab9a211dcf3bd3/tests/testthat/test-MPG.R#L101) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


@sl1pkn07 It won't break it since there's a #ifndef in `Conditionals_post.h`, but I'm wondering why it was removed:

https://github.com/MarlinFirmware/Marlin/blob/52a7a8cb265d89c20e352bfaf8053e0e72fc1d0f/Marlin/src/inc/Conditionals_post.h#L1156-L1158


TeamCity Glyssen / Glyssen pull requests [Build *.*.414.a8330be722720eb66b67271bdf91704344ae91d0](https://build.palaso.org/viewLog.html?buildId=158851&buildTypeId=Glyssen_GlyssenPullRequests2) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 2166, ignored: 18 Build time: 00:08:17
### Failed tests
```
ControlDataIntegrityTests.dll: ControlDataIntegrityTests.CharacterVerseDataTests.DataIntegrity_RequiredFieldsHaveValidFormatAndThereAreNoDuplicateLines: Test(s) failed.   Line with extra space(s): ROM	10	6	God		 righteousness which is of faith (God)	Quotation
  Expected: False
  But was:  True

   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)
   at NUnit.Framework.Assert.IsFalse(Boolean condition, String message)
   at ControlDataIntegrityTests.CharacterVerseDataTests.DataIntegrity_RequiredFieldsHaveValidFormatAndThereAreNoDuplicateLines() in C:\BuildAgent\work\9269096ed6fdb26d\ControlDataIntegrityTests\CharacterVerseDataTests.cs:line 86

GlyssenTests.dll: GlyssenTests.ProjectTests.SetReferenceText_ChangeFromEnglishToFrenchWithOneBlockMismatched_ReferenceTextClearedForAllRelatedBlocks: Test(s) failed.   Expected: True
  But was:  False

   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)
   at NUnit.Framework.Assert.IsTrue(Boolean condition)
   at GlyssenTests.ProjectTests.SetReferenceText_ChangeFromEnglishToFrenchWithOneBlockMismatched_ReferenceTextClearedForAllRelatedBlocks() in C:\BuildAgent\work\9269096ed6fdb26d\GlyssenTests\ProjectTests.cs:line 1156

GlyssenTests.dll: GlyssenTests.BookScriptTests.ApplyUserDecisions_AllSourceBlocksAlignedToReferenceTextWithManualSplitInMiddleOfVerse_TextUnchanged_SplitAppliedAndAllBlocksAligned: Test(s) failed.   String lengths are both 2. Strings differ at index 1.
  Expected: "26"
  But was:  "25"
  ------------^

   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)
   at NUnit.Framework.Assert.AreEqual(Object expected, Object actual)
   at GlyssenTests.BookScriptTests.ApplyUserDecisions_AllSourceBlocksAlignedToReferenceTextWithManualSplitInMiddleOfVerse_TextUnchanged_SplitAppliedAndAllBlocksAligned() in C:\BuildAgent\work\9269096ed6fdb26d\GlyssenTests\BookScriptTests.cs:line 1397

```


ci is coming ;-)


Will require changes to be pushed through to hard directories, but the tool at least throw an error if the parameter is missing.


c2() decipher didn't work with a padding

ps - yes the change on line 53 is redundant...


I'll look into it! There might be a way to do on through the xml


there was an extra 8 bytes (coming from .final perhaps?) using des.js (caveat, browserify injected it)


Hi there, @Chevreuil41 I have updated the main project with [a similar fix](https://github.com/fuzzzerd/fmdata/pull/98/commits/a7dcc73f96b35333844d041093c262440f8ceb94) after verifying it still works with FMS17.

This has been published to NuGet as v4.0.1. Please see if it resolves this problem. If you're still having trouble, feel free to open an issue (and/or pull request) on the main repository.

Thanks again for pointing this out.