Bonjour,

désolé d'avoir envahi le site. J'ai posé ma question sur la boîte de
dialogue et je me suis, en suite, contenté de répondre aux mails qui
arrivaient sur ma messagerie. Je ne dispose donc pas d'adresse
individuelle. Dès que j'en aurais une, je n'arroserai plus personne.

Cordialement

D. Benveniste

Le 31/03/2015 17:46, dane-lyon a écrit :
>
> Évitez d'avoir un dialogue via GitHub car cela arrose beaucoup de monde.
>
> Utilisez nos mails : prenom.nom@ac-lyon.fr
>
> Simon voulait parler de la case shell dans l'EAD du Scribe.
>
> Cédric
>
>
> Le 31/03/2015 14:27, David Benveniste a écrit :
> > Bonjour,
> >
> > il s'agit effectivement des connexions des personnes qui appartiennent
> > au groupe profs.
> > En ce qui concerne le script, la seule case que j'ai trouvée correspond
> > au réglage de la fente de connexion (cf capture d'écran). S'il y en a
> > une autre je ne sais pas où la trouver.
> >
> > Cordialement
> >
> > D. Benveniste
> >
> > Le 31/03/2015 13:08, dane-lyon a écrit :
> > > Que voulez vous dire par "connexion adulte" ? si vous parlez d'un
> compte
> > > prof, cela fonctionne si la case pour les scripts shell est coché.
> > >
> > >
> > >
> > > Le 31/03/2015 12:13, David Benveniste a écrit :
> > > > J'ai dû mal rédiger mon message.
> > > > Nous avons des postes qui tournent sous Edubuntu 12.04 et pour
> > lesquels
> > > > nous n'avons aucun problème de connexion, ni pour les élèves, ni
> pour
> > > > les adultes.
> > > > Nous avons voulu tester des postes sous Mint pour différentes
> raisons.
> > > > Ces postes fonctionnent au problème près des connexions adultes.
> C'est
> > > > la raison pour laquelle je pensais que la difficulté pouvait
> > provenir du
> > > > script spécifique de cette distribution.
> > > >
> > > > Cordialement
> > > >
> > > > D. Benveniste
> > > >
> > > > Le 31/03/2015 11:05, dane-lyon a écrit :
> > > > > Bonjour,
> > > > >
> > > > > Les comptes utilisateurs "élèves" et "profs" fonctionnent sous
> Linux
> > > > > Mint si la case "client shell linux" est coché dans l'EAD,
> j'ai même
> > > > > testé en vitesse ce matin.
> > > > >
> > > > > cordialement;
> > > > >
> > > > >
> > > > > Le 31/03/2015 10:40, David Benveniste a écrit :
> > > > > > Bonjour,
> > > > > >
> > > > > > après enquête et divers essais, il apparait que le problème
> réside
> > > > dans
> > > > > > la connexion des utilisateurs adultes. Les élèves peuvent se
> > > connecter
> > > > > > sans aucun problème. En revanche, les adultes ne peuvent se
> > > logger, le
> > > > > > message leur indique que leur compte a été désactivé. Seul les
> > > comptes
> > > > > > administrateur réseau et administrateur local sont effectifs.
> > > > > > Le script client est bien activé par défaut pour les sessions.
> > > > > > Il y aurait peut-être un conflit entre le script et
> > > l'architecture des
> > > > > > comptes sur le serveur mais cela dépasse mes capacités.
> > > > > >
> > > > > > Cordialement
> > > > > > D. Benveniste
> > > > > >
> > > > > >
> > > > > > Le 31/03/2015 09:16, dane-lyon a écrit :
> > > > > > > Bonjour,
> > > > > > >
> > > > > > > Sous Linux Mint, il vaut mieux cocher la case "activer les
> > clients
> > > > > shell
> > > > > > > linux" dans l'EAD pour les utilisateurs sinon il peux y
> > avoir des
> > > > > > > problèmes de connexion, j'ai pu faire le test en créant un
> > > > utilisateur
> > > > > > > sans cette case coché et alors ça indiquait le message que
> > > > > > > l'administrateur a désactivé le compte et en la cochant
> puis en
> > > > > > > redémarrant le scribe/client, alors il pouvait se connecter
> > > > > normalement.
> > > > > > >
> > > > > > > En faite si vous voulez déployer du Linux, je vous
> conseille de
> > > > mettre
> > > > > > > plutôt "Xubuntu" (voir Lubuntu pour les très anciens pc) car
> > c'est
> > > > > plus
> > > > > > > léger et rapide et il n'y a pas besoin d'avoir la case des
> > clients
> > > > > shell
> > > > > > > linux coché, de plus Linux Mint est plutôt orienté pour une
> > > > > utilisation
> > > > > > > familiale/perso/multimedia et non professionnelle et consomme
> > > quasi
> > > > > > > autant de ressource que Ubuntu/Unity.
> > > > > > >
> > > > > > >
> > > > > > > cordialement;
> > > > > > >
> > > > > > >
> > > > > > > Le 30/03/2015 17:39, David Benveniste a écrit :
> > > > > > > > Bonjour,
> > > > > > > >
> > > > > > > > J'ai installé deux postes sous Mint avec le script
> téléchargé
> > > > sur le
> > > > > > > > site. La connexion fonctionne avec le compte
> > administrateur mais
> > > > > si on
> > > > > > > > essaye de se logger avec un compte utilisateur, un message
> > > > > indique que
> > > > > > > > l'administrateur a désactivé le compte.
> > > > > > > > Vérification faite, le compte existe bien dans scribe et
> reste
> > > > > > > > fonctionnel sur des postes sous windows ou Edubuntu 12.04.
> > > > > > > > De plus sur un des deux postes (le plus récent), une fois la
> > > > > > fenêtre du
> > > > > > > > message validée, l'écran devient noir et il n'est plus
> > > possible de
> > > > > > faire
> > > > > > > > quoi que ce soit. Redémarrage brutal du poste obligatoire.
> > > > > > > >
> > > > > > > >
> > > > > > > > Cordialement
> > > > > > > >
> > > > > > > > D. Benveniste
> > > > > > > >
> > > > > > > > Le 27/03/2015 15:47, dane-lyon a écrit :
> > > > > > > > > Bonjour,
> > > > > > > > >
> > > > > > > > > Il faut désactiver l'userlist avec le compte local (vous
> > > > pouvez le
> > > > > > > faire
> > > > > > > > > avant de lancer le script) et ensuite vous pourrez vous
> > > > > connecter en
> > > > > > > > > tapant le login et le mdp comme sous Windows, vous pourrez
> > > vous
> > > > > > > > > connecter a la fois avec un compte du domaine (compte
> > admin du
> > > > > > > domaine,
> > > > > > > > > compte prof, compte élève) mais aussi avec le compte
> locale.
> > > > > > > > >
> > > > > > > > > A noté que Mint est assez exigeant en ressource, si le pc
> > > n'est
> > > > > > > pas très
> > > > > > > > > performant, il vaut mieux utiliser Xubuntu voir Lubuntu.
> > > > > > > > >
> > > > > > > > > cordialement;
> > > > > > > > >
> > > > > > > > > Le 27/03/2015 15:32, David Benveniste a écrit :
> > > > > > > > > > Le document que j'ai téléchargé ne correspond pas à
> > > celui qui
> > > > > > > apparait
> > > > > > > > > > quand on active le lien de votre message. En faisant un
> > > copié
> > > > > > > collé de
> > > > > > > > > > la page web, j'ai reconstitué un fichier qui a
> installé le
> > > > > > > client sur
> > > > > > > > > > scribe.
> > > > > > > > > > En revanche, je ne parviens pas à me connecter. Je
> > vois une
> > > > > liste
> > > > > > > > > > d'utilisateurs (tous ceux du collège) et quand je
> > saisis le
> > > > > mot de
> > > > > > > > > > passe, la machine tourne dans le vide.
> > > > > > > > > > Je ne suis donc pas en mesure de paramétrer la
> machine en
> > > > > > > suivant les
> > > > > > > > > > commentaires du script et je ne parviens pas à me
> > connecter
> > > > > sur le
> > > > > > > > poste
> > > > > > > > > > en local.
> > > > > > > > > >
> > > > > > > > > > Cordialement
> > > > > > > > > >
> > > > > > > > > > D. Benveniste
> > > > > > > > > >
> > > > > > > > > > Le 27/03/2015 13:04, dane-lyon a écrit :
> > > > > > > > > > > Bonjour,
> > > > > > > > > > >
> > > > > > > > > > > Nous avons un script d'intégration pour le client
> > > Linux Mint
> > > > > > > > > 17/17.1 et
> > > > > > > > > > > il fonctionne (
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://raw.githubusercontent.com/dane-lyon/clients-linux-scribe/master/client_scribe_linuxmint_17.X.sh
>
> >
> > >
> > > >
> > > > >
> > > > > >
> > > > > > >
> > > > > > > >
> > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > ) mais il faut bien lire ce qui est en commentaire
> > dans le
> > > > > > > script au
> > > > > > > > > > > début.
> > > > > > > > > > >
> > > > > > > > > > > En faite, toutes les dérivés d'Ubuntu fonctionnent
> > avec le
> > > > > > script
> > > > > > > > > de la
> > > > > > > > > > > 14.04 sauf la dérivé avec KDE (Kubuntu) qui n'est pas
> > > > > > compatible.
> > > > > > > > > > >
> > > > > > > > > > > Voici ce qui fonctionne :
> > > > > > > > > > >
> > > > > > > > > > > - Ubuntu 14.04
> > > > > > > > > > > - Xubuntu 14.04
> > > > > > > > > > > - Lubuntu 14.04
> > > > > > > > > > > - Ubuntu Gnome 14.04
> > > > > > > > > > > - Ubuntu Mate 14.04
> > > > > > > > > > > - Linux Mint 17 Cinnamon/Mate
> > > > > > > > > > > - Linux Mint 17.1 Cinnamon/Mate
> > > > > > > > > > >
> > > > > > > > > > > A noté que Debian n'est pas compatible a notre script
> > > > mais il
> > > > > > > y a le
> > > > > > > > > > > script crée par quelqu'un d'autre pour ça.
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > Cordialement;
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > Le 27/03/2015 12:14, David Benveniste a écrit :
> > > > > > > > > > > >
> > > > > > > > > > > > Bonjour,
> > > > > > > > > > > >
> > > > > > > > > > > > j'ai testé avec succès le script sur une
> distribution
> > > > > Edubuntu
> > > > > > > > > 12.04.
> > > > > > > > > > > > En revanche, je ne parviens pas à cloner le
> poste avec
> > > > > Oscar.
> > > > > > > > > > > > Je viens d'essayer d'utiliser le script client
> > avec une
> > > > > > > > distribution
> > > > > > > > > > > > Mint 17.1 et la console me retourne le message
> > suivant :
> > > > > > Syntax
> > > > > > > > > error
> > > > > > > > > > > > : newline unexpected
> > > > > > > > > > > >
> > > > > > > > > > > > —
> > > > > > > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10429363>.
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > --
> > > > > > > > > > > Simon Bernard
> > > > > > > > > > > académie de Lyon - Délégation Académique au Numérique
> > > > Éducatif
> > > > > > > > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> > > *Simon
> > > > > > > Bernard
> > > > > > > > > > > Rectorat de Lyon
> > > > > > > > > > > DANE - Délégation Académique au Numérique Éducatif
> > > > > > > > > > > Assistance de niveau 2 aux utilisateurs du réseau
> > > > pédagogique
> > > > > > > > > > > * Téléphone : 04 72 80 64 88
> > > > > > > > > > >
> > > > > > > > > > > Adresse postale : 92 rue de Marseille - BP 7227 -
> > > 69354 Lyon
> > > > > > > > Cedex 07
> > > > > > > > > > > Locaux site Croix-Rousse (47 rue Philippe de Lassalle
> > > 69004
> > > > > > Lyon)
> > > > > > > > > > > *http://www.ac-lyon.fr*
> > > > > > > > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> > > > > > > > > > >
> > > > > > > > > > > —
> > > > > > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10430053>.
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > —
> > > > > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10432751>.
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > > Simon Bernard
> > > > > > > > > académie de Lyon - Délégation Académique au Numérique
> > Éducatif
> > > > > > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> *Simon
> > > > > Bernard
> > > > > > > > > Rectorat de Lyon
> > > > > > > > > DANE - Délégation Académique au Numérique Éducatif
> > > > > > > > > Assistance de niveau 2 aux utilisateurs du réseau
> > pédagogique
> > > > > > > > > * Téléphone : 04 72 80 64 88
> > > > > > > > >
> > > > > > > > > Adresse postale : 92 rue de Marseille - BP 7227 -
> 69354 Lyon
> > > > > > Cedex 07
> > > > > > > > > Locaux site Croix-Rousse (47 rue Philippe de Lassalle
> 69004
> > > > Lyon)
> > > > > > > > > *http://www.ac-lyon.fr*
> > > > > > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> > > > > > > > >
> > > > > > > > > —
> > > > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10432996>.
> > > > > > > > >
> > > > > > > >
> > > > > > > > —
> > > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10470632>.
> > > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Simon Bernard
> > > > > > > académie de Lyon - Délégation Académique au Numérique Éducatif
> > > > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/> *Simon
> > > Bernard
> > > > > > > Rectorat de Lyon
> > > > > > > DANE - Délégation Académique au Numérique Éducatif
> > > > > > > Assistance de niveau 2 aux utilisateurs du réseau pédagogique
> > > > > > > * Téléphone : 04 72 80 64 88
> > > > > > >
> > > > > > > Adresse postale : 92 rue de Marseille - BP 7227 - 69354 Lyon
> > > > Cedex 07
> > > > > > > Locaux site Croix-Rousse (47 rue Philippe de Lassalle 69004
> > Lyon)
> > > > > > > *http://www.ac-lyon.fr*
> > > > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> > > > > > >
> > > > > > > —
> > > > > > > Reply to this email directly or view it on GitHub
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10483837>.
> > > > > > >
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > >
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10486144>.
> > > > > >
> > > > >
> > > > > --
> > > > > Simon Bernard
> > > > > académie de Lyon - Délégation Académique au Numérique Éducatif
> > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/> *Simon
> Bernard
> > > > > Rectorat de Lyon
> > > > > DANE - Délégation Académique au Numérique Éducatif
> > > > > Assistance de niveau 2 aux utilisateurs du réseau pédagogique
> > > > > * Téléphone : 04 72 80 64 88
> > > > >
> > > > > Adresse postale : 92 rue de Marseille - BP 7227 - 69354 Lyon
> > Cedex 07
> > > > > Locaux site Croix-Rousse (47 rue Philippe de Lassalle 69004 Lyon)
> > > > > *http://www.ac-lyon.fr*
> > > > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > >
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10486570>.
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > >
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10487870>.
> > > >
> > >
> > > --
> > > Simon Bernard
> > > académie de Lyon - Délégation Académique au Numérique Éducatif
> > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/> *Simon Bernard
> > > Rectorat de Lyon
> > > DANE - Délégation Académique au Numérique Éducatif
> > > Assistance de niveau 2 aux utilisateurs du réseau pédagogique
> > > * Téléphone : 04 72 80 64 88
> > >
> > > Adresse postale : 92 rue de Marseille - BP 7227 - 69354 Lyon Cedex 07
> > > Locaux site Croix-Rousse (47 rue Philippe de Lassalle 69004 Lyon)
> > > *http://www.ac-lyon.fr*
> > > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10488735>.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10491313>.
> >
>
>
> --
> Cédric Frayssinet
> académie de Lyon - Délégation Académique au Numérique Éducatif
> <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/> *Cédric Frayssinet
> Rectorat de Lyon
> DANE - Délégation Académique au Numérique Éducatif
> Responsable des systèmes installés
> *
> Adresse postale : 92 rue de Marseille - BP 7227 - 69354 Lyon Cedex 07
> Locaux site Croix-Rousse (47 rue Philippe de Lassalle 69004 Lyon)
> *http://www.ac-lyon.fr*
> <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10498155>.
>


Yes but that has no impact on front-end


TeamCity xBIM :: XbimExchange :: Build [Build 173](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1260&buildTypeId=XBIM_XbimExchange_Build) is now running


No no, actualy thank you :)


添加图片


The velodyne has no fixed horizontal resolution


TeamCity xBIM :: XbimExchange :: Build [Build 173](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1260&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:02:54


A comment, were the 0.002 commes from would be nice...


It was there originally, I just didn't comment it out. I agree, it seems both stupid and unnecessary. Let's just remove it.


We might add a hint to the new CUTE version (4.11.0), which has been released on Monday. The features can be seen on the CUTE News page.


had->has


i know, and i am really sorry for that. I know i can do better than this but i kind of ran out of time this week and i didn't focus enough on school work.


Is 0 the correct way to mark a reading as invalid ?


quaality -> quality (`codespell` is your friend :-)


woith -> with


via magic bit sniffing _for_ ...


I would like it better if these were called `lng`/`lat`


1.0 + 1.0/3.0 would be better for higher precision


table creation and data adds script OK


It's last and worked version of requirements.txt


important commit


You should really try running this on some hardware other than the lab cluster.  It throws an exception immediately if the cli64 tool is absent.


bzip compressed sequence files ... (no zip, I dropped that)


Looking at this, this could probably start with td.bright_red_background


http://review.cyanogenmod.org/#/c/84259/ ;)


second test on commit


This line looks quite complicated. How about moving the Math.min to a separate variable, and adding a comment to it (what minimum is actually calculated and why)?


shouldnt be:

> if ((ch_md->version_number < MESH_VALUE_LOLLIPOP_LIMIT && version > ch_md->version_number) ||

as it will call the Update on the same version, thus continuously


...and same here


did you mean to remove the whole line?


Remember to init it's value, maybe in amodem_create().


I prefer '\0' instead of NULL for char type


This copy is pretty expensive (around 500 000 points).
Is there perhaps a way to fill in the data directly in the
right order ?


I was writing a comment to justify why I didn't split this out (much of the content is generated dynamically, and I was fighting browser caching) but I now think it might be possible to split this out as I made some simplifications to the rest of the logic...


Почему не setProperty(REQUEST_SERVICED, true)?


@mattharris5 Could you take one more look? Thanks a million


@brasmusson thanks for this fix, but I'm strongly against mutating these AST objects after construction. Value objects are much simpler to deal with if they're immutable. Let's try and find a way of passing this feature attribute into the `EmptyBackground` as it's constructed instead.


e.g. `com.android.webview.chromium.ContentSettingsAdapter` on Android 5.x


可以


Do we still need this function ?


The bannerId should be provided as a part of the config. BTW., why don't you allow to use any other selectors like: `.banner`, `[class*="banner-"]`, `div` or anything else? You could use `document.querySelector` for that.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13458/testReport/) for this commit**


Ah, I see that's already how it's done elsewhere. I thought we'd cleaned all that up!


Oo.
Maybe I nitpick, but what about that instead of goto:
```php
if (!empty($patch)) {
    $config = array('zf-rest' => array(
        $original->controllerServiceName => $patch,
    ));
    $this->configResource->patch($config, true);
}
```


`SOURCE_VERSION` is now available.


Зачем StringUtils? Тебе важен сам факт наличия этой проперти? Почему тогда просто не сделать RequestScope.getProperty(REQUEST_SERVICED) != null?


Почему нужно удалять эту пропертю?


Soo :cool: :sunglasses: !


Errorを上に逃がすんならより少ないコードで表現できるますよ。
} catch (RuntimeException e) {
	throw e;
} catch (Exception e) {
	throw new RuntimeException(e);
}


Oh yes of course, I didn't realize we had actual new features in Cute :-)


LGTM +1


You should follow the convention: `function () {}`, `function (param) {}`.


0.1.0-SNAPSHOT is the recommended initial version according to semver heuristics


You should use strict comparisons: `===`, `!==`


Or, more specifically - they have *exactly* the same issues that we have -
a 3rd party library (guava dependent), and a desire to use a later guava
version than is bundled with Jenkins.

Jenkins plugin system puts the core classloader *before* the plugin
classloader (well, you can reverse that - it's not recommended and the
guidance is 'bad things happen if you do this'). That's a pain, because you
get occasional (and therefore very hard to test, as you get sudden and
unexpected regressions) class cast and class reference errors - likely
because when a guava class is referenced that is present in 11.x, you get
the Jenkins version, and if it isn't present in 11.x, you get the plugin
one. Specifically, I discovered in beta-1 a brand new classcast exception
caused by exactly this in the buildpublisher.

jenkins-jclouds has a core jenkins developer working on it. I'm minded to
believe if that's the best solution they can come up with, then it's worth
copying as a starting point that *works now*. There may be better
solutions, but I rank them as lower value in allocating my time than adding
features.

On Wed, Apr 1, 2015 at 9:23 AM, Nigel Magnay <nigel.magnay@gmail.com> wrote:

> Because it works!
>
>
> On Tue, Mar 31, 2015 at 11:11 PM, Kanstantsin Shautsou <
> notifications@github.com> wrote:
>
>> Why do you copy everything from jclouds?!
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jenkinsci/docker-plugin/commit/bdde16c512c9f72eabc209b7d369f258c987a7af#commitcomment-10506696>
>> .
>>
>
>


Well, I have problems calling it that way, as it's simply not latitude and longitude, but some crappy web projection.


setSelection(1) cannot addto to this method???


Is the cookie info a navbar?


Раньше хоть вылетала OutOfScopeException, а теперь вообще NullPointerException.


```php
<?= $me->nomeCompleto(); ?>
```


So, you're saying that without `findMacroClassLoader` everything works?


Hej hej här testar jag att skriva lite :)


This code means you can't register more drivers later / from another module / from user code.
Consider doing these in class methods on BaseDriver that returns them.


fixed in e858e45


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13459/testReport/) for this commit**


How?


Nitpick: you should be very careful using so generic CSS classnames like: `message`, `action`. I'd suggest you to use BEM for this, so in the end it will look like this: `privacy-cookie-banner__message`, `privacy-cookie-banner__action`


In fact, @tooky correct me if I'm wrong, but we've tried to eliminate circular references in the AST, so child nodes should not know about their parents. I'll write more in the ticket.


Formatting fixed in e858e45


There are no setter for Handler.


Why public class? I don't think this is how you program in PHP


> `The Collection of all available {@link Criterion Criteria}`

But it returns `Collection<Visibility>`

Visibility != Criterion/Criteria


Missed 'textbox' completely ... but what about browsers that don't support types such as email, number, etc.? That's why there were just 2 types of inputs, text and password, then defender took care of the final/real type checking upon posting.
That's the reason we also don't have "required" on inputs, because what browsers think should be a correct input might not comply with defender's validation routines and will only confuse users.


You did not rename the file


This should be inside a class


I made changed name of disabled class to make more sense.


!IsIntraPoolMove() is the only check needed here, because we will display the transfer network cases for all cases except intra-pool move


Displaying credentials anywhere is dangerous


I moved the Math.min to a separate variable


Merge ?


If my thoughts are correct. You simply copy and pasted this code from somewhere. If you're gonna do that. Please observer proper indentation


Where does `$line4.$read.$iw.$iw.$iw.$iw` come from? Is it from the annotation produced by `bindMacroImpl`? It looks like the correct erased class name is `$line4.$read$$iw$$iw$$iw$$iw$`.


(false or true) == true


Also check that the projects GitExtensions and TranslationApp copy GitUI.dll (in reference settings)


In compilation yes, in the REPL the lightweight syntax doesn't work but I think this is a separate problem.


Why did you delete this? This was already the correct. All you need was to supply the functionality of the class and its methods


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13460/testReport/) for this commit**


This entire file is hard to review. Please observer proper coding standards


I think that reveals that we need to fix the equality checks on AST nodes. I've made #85.


Should be a<ins>n</ins> exclude filter.


As suggested https://lists.okfn.org/pipermail/ckan-dev/2013-November/006141.html


This release seems to break `bosh prepare deployment`:
```
Uploading 'logsearch-for-cloudfoundry/'
/home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh-workspace-0.8.5/lib/bosh/workspace/release.rb:22:in `join': no implicit conversion of nil into String (TypeError)
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh-workspace-0.8.5/lib/bosh/workspace/release.rb:22:in `manifest_file'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh-workspace-0.8.5/lib/bosh/cli/commands/prepare.rb:49:in `prepare_release'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh-workspace-0.8.5/lib/bosh/cli/commands/prepare.rb:39:in `block in prepare_releases'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh-workspace-0.8.5/lib/bosh/cli/commands/prepare.rb:38:in `each'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh-workspace-0.8.5/lib/bosh/cli/commands/prepare.rb:38:in `prepare_releases'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh-workspace-0.8.5/lib/bosh/cli/commands/prepare.rb:19:in `prepare'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh_cli-1.2905.0/lib/cli/command_handler.rb:57:in `run'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh_cli-1.2905.0/lib/cli/runner.rb:56:in `run'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/gems/bosh_cli-1.2905.0/bin/bosh:16:in `<top (required)>'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/bin/bosh:23:in `load'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/bin/bosh:23:in `<main>'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `eval'
	from /home/ubuntu/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `<main>'
u
```
@longnguyen11288, The bosh release in https://github.com/logsearch/logsearch-for-cloudfoundry is in a subfolder - https://github.com/logsearch/logsearch-for-cloudfoundry/tree/master/logsearch-for-cloudfoundry-boshrelease - how do I tell boshworkspace that the boshrelease is in a subfolder?


April 31st, eh? ;)


thanks, changed :smiley_cat:


TeamCity xBIM :: XbimExchange :: Build [Build 174](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1261&buildTypeId=XBIM_XbimExchange_Build) is now running


Remove /n


Cool. I was torn between whether to use `-n` or `-z`, I guess `-n` makes more sense if there's an `else` branch…


JavaSciprt!!


I directly download this rom,url:http://zulmat.net/games/sfc/roms/SuperMarioAll-Stars.sfc
you try again pls,thanks!


:bowtie:


This looks really great, and is exactly what i'm looking for.
Can you mail me when this is ready ?


UBERTC 5.0? ;)


Just guessing here, but first we would need to `npm link` in the root dir, then run `npm link antwar antwar-cli` in the demo folder?


mentioning people in commit is :japanese_ogre: I will have my revenge


Yeah, I prefer to keep the part that you're checking against close to where you're actually using it. (unless you're going to exit in the `if`)


TeamCity xBIM :: XbimExchange :: Build [Build 174](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1261&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:04:23


Nice start, interesting feature. We have an internal board on Trello.com, you could create an account there and send me account-name via mail, so I can add you. florian.kuepper@gmail.com


Could it be that you have multiple incompatible versions of scalameta/scalameta (in particular, of the `scala.meta.syntactic.Token` classes) embedded into your *tool* classpath? When sbt-assembly is involved, these things happen?


No, the pdf isn't outdated. But you seem to be following along the book (which uses Marionette v 2) while using v 1.7. Your first comment mentions changing `childView` to itemView to get things to work. `childView` is used in v >= 2, whereas `itemView` was used before (e.g. in v 1.7).

The version you reference is currently the latest version of the book. If you're refering to the latest version of the book but using Marionette 1.7, you should have had to change several other things too (e.g. the event "initialize:after" was renamed to "start" in v 2).

In summary:
* the pdf isn't outdated
* you're using Marionette v 1.7 to follow along with a book using v 2
* this major version difference means you'll need to adapt code to get it to work
* the easiest would be to update to the same lib version used in the book (https://raw.githubusercontent.com/davidsulc/marionette-gentle-introduction/master/assets/js/vendor/backbone.marionette.js)


yes, now it downloads, but in your html code you are using url
'/roms/SuperMarioAll-Stars.sfc'
try to change it to 'roms/SuperMarioAll-Stars.sfc'.
By the way, the emulator doesn't support .sfc extension, please also rename
it to *.smc and let me know

On Wed, Apr 1, 2015 at 12:07 PM, Dilshat <notifications@github.com> wrote:

> I directly download this rom,url:
> http://zulmat.net/games/sfc/roms/SuperMarioAll-Stars.sfc
> you try again pls,thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nesbox/emulator/commit/686b5255cd496c077e6aaa13e82e1436db6720f5#commitcomment-10515112>
> .
>

--
Vadim Grigoruk
grigoruk@gmail.com


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13461/testReport/) for this commit**


@javierjulio I think so


@Optimusprimeums no worries mate I know you're busy, I'll tag it up this week and just build when you get the spare time :)


missing `addNewItemToCart` call


could have called `iAddADollarItemWithWeight`


I wonder why the file mode was changed to 755


TeamCity xBIM :: XbimExchange :: Build [Build 175](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1262&buildTypeId=XBIM_XbimExchange_Build) is now running


inconsistent style (placement of braces) between methods


Well this one slipped, I'll remove it.
setNoticeTitle() from I can remember was necessary/used in conjunction with notify. Notify relies on directly outputting HTML/JS code and it only should be used for designing how notices are rendered in browser and not how they are handled in source code. I'll implement notify later as a showcase of how notices can be styled and customized.


@Dilshat:

Please read carefully: http://nesbox.github.io/emulator/, NESBOX don't
support .sfc so you cannot load the roms.

Thanks,

On Wed, Apr 1, 2015 at 4:07 PM, Dilshat <notifications@github.com> wrote:

> I directly download this rom,url:
> http://zulmat.net/games/sfc/roms/SuperMarioAll-Stars.sfc
> you try again pls,thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nesbox/emulator/commit/686b5255cd496c077e6aaa13e82e1436db6720f5#commitcomment-10515112>
> .
>

--

Ngo Anh Tuan - Webtretho Senior Developer
Phone: 0907.33.53.95


Duplicate of `addProduct`


missing type hint


```java
try {
  return gete();
}catch(RuntimeException e){
  throw e;
}catch (Exception e){
  throw new RuntimeException(e);
}
```
これだと`gete()`がThrowableを投げるのでコンパイルエラーになってしまい
```java
try {
  return gete();
}catch(RuntimeException e){
  throw e;
}catch (Exception e) {
  throw new RuntimeException(e);
}catch (Throwable e){
  throw new RuntimeException(e);
}
```
これだとErrorが最後のcatchで捕まってRuntimeExceptionになってしまいます(かつ既存コードの方が短い)。
そもそもSE.get()がThrowableなのが筋が悪いんですかね…。


@cbuescher I think this is the wrong import here? Should be hamcrest maybe?


Because it should only throw the error if `$first` was requested but not found. Otherwise we return an empty array. That's necessary to foreach over the result.


should be private


:smile:


the name could also be better, I feel like this is meant to alter the object's state


TeamCity xBIM :: XbimExchange :: Build [Build 175](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1262&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:02:15


That would mean the container cannot be built. My implementation simply set an empty array which would result in the same as if a specific packages is not installed.


Sidekz,

the code you want to is a small part of startLiveCast().


Well it's a method property, an IDE will tell you the default value ;-)


If None is being serialized to Js.Null then they will get passed into here when a case class contains an Option


Good point - thanks


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13462/testReport/) for this commit**


fint


should be `>=` instead of `>`


I got rid of `npm link`s and now we have something that works. It still fails but for a correct reason. See https://travis-ci.org/antwarjs/antwar/jobs/56693130 .


You can test camelcase in this project.


https://github.com/shaarli/Shaarli/commit/a5752e776c9ee5ee5a711a76e8c8066ee88f7192

What the f did I do. I'll fix this.


Track ressources i9100 for prevent problem device trees :/


Тут, в принципе, без разницы. Можно и RuntimeException.


@mrdavidlaing that issue has been fixed in https://github.com/cloudfoundry-incubator/bosh-workspace/pull/31 which is included in [v0.9.0.rc2](https://rubygems.org/gems/bosh-workspace/versions/0.9.0.rc2)


Yes, probably it would be have a default Reader for Option[T] so that Js.Null never gets here. Thanks


should be `$shipCost` instead of `shipCost`


Liten bokstav efter `:`.


Noice!


bad url


you can not do sub tasks in gulp ?


(apparently everything was already camelcased)


I copied from homepage, there is a bad URL too


This fixes #812, but hasn't landed in `socket.io.js` yet.


Ths file is missing en S3...


upload the favion there :)


Yeah that looks good!


this is right


.ico - > unsupported type


though it was pretier before


the constants should be accessed like this `self::RATE_FREE` or `static::RATE_FREE` or `Cart::RATE_FREE`


Раньше исключение просто перебрасывалось, это было изменено на прошлом ревью. Баг, поправлю.


This fixes #812, but hasn't landed in socket.io.js yet.


Остатки изменений, убранных после предыдущего ревью. Можно убрать для ясности.


https://scalingo-cdn.s3.eu-central-1.amazonaws.com/favicon.ico

Here it is, but not yet available there http://cdn.scalingo.com/favicon.ico


"ras" -> "etnicitet".

"eller brist därav" klingar lite negativt också, man kanske kan ändra om det till "avsaknad"?


I checked that. The have Copy Local = True


This works, but could also have used assertions


should probably have used strict comparison


"ras" -> "etnicitet".


@radeklos would be good to get some doc strings on that ``Queue`` and ``Random`` Classes in `/logic/player`.


I guess you could implement your own sub-task thingy by calling a function with custom params, but there's nothing in the base library to do that.


This is not using the fact that the product has a `$quantity` property


I changed '/roms/SuperMarioAll-Stars.sfc' to 'roms/SuperMarioAll-Stars.sfc',and rename it to SuperMarioAll-Stars.smc , but not working...I download other smc games try again,thank you!


You're right, sorry. Thankfully this seems to be an unused import, just needs to be deleted. Should I remove that with subsequent commit or should I do that with any other of my next PRs? #10148 is hopefully also ready to go in soon, what do you think?


It was! Wish there was a rule for smart indent. Maybe there is, haven't found one yet.


Thanks!


Hi @julienbourdeau ! I think there is an URL error, here ;-)


@synopsys-arc-automation-bot
It causes a StackOverflow bug


Ok favicon is fixe on homepage and changelog.


Потому что проперти - строки.


Ah, j'ai compris.
Et donc goTo est une fonction déjà existe alors


You are correct, that's a bug. Thank you, I'll see if we can push a fix for it.


looks good to me.


don't forget to rename to smc here
var flashvars =
                {
                    system : 'snes',
                    url : 'roms/SuperMarioAll-Stars.smc'
                };


Making the Pusher.BODY_SERIALISER public would work. If you're unhappy about the dependencies, stick it in a Util class.


Потому же, почему и во многих других местах проверка на null часто меняется на StringUtils - не надо различать null и пустые строки.


taken from the velodyne manual: Angular resolution (horizontal) ~0.16° at 600rpm


Could you tell me what this commit fixes?


don't worry just push a new commit to remove it. doesn't make sense to push it as part of #10148 cause it doesn't touch this class anymore.


Super cool commit bro


Reviewed and approved.


**/*.test.js was good too, more flexible as well


Den sista meningen fungerar nog bättre såhär: "Vi förväntar oss ett samarbete där alla deltagare bidrar till att säkerställa en trygg miljö för alla.".


aaaa
![h42](https://cloud.githubusercontent.com/assets/11749292/6938323/84c42c92-d89d-11e4-8fc8-db23d612fd8f.gif)
ggg


Looks better with emerald!


taken from Sascha's issue: Use 0 (to near),... to express measurement errors


@oranagra :dancer: yeah :-) Thanks.


Она выставляется с целью чтения в данном классе, как и прежняя ADAPTER_ABSTAINED_NOTE. Можно ее не удалять, но после завершения данного метода исчезает контекст выполнения, в котором она имела смысл.


You'll do better next week then ^^


import


could have used assertions instead of exceptions


those could have been removed


That's what we thought, but Rabl has some issues with it.


:)  hehe,it is working, thank you , cheers!!!
and how to use the server? I haven't find  instructions.


30414/Dolineanu A. Vlad Ionut/Assignment 3 - extra/main.c

you could just use ```strcmp("+",tempData)``` instead of this


Hm, using the force localization aapt flag (-z) is causing:
```
Error: This attribute must be localized. (at 'contentDescription' with value 'user_avatar').
```

Building with the `tests` module tag disables this flag and the build goes okay.


could have called `iHaveAnEmptyCart`


It fixes the problem, thanks!


All invalid type are handled as text. The most of the new types in HTML5 just help the user to fill the input correctly. Programmers have to validate the data on server side in every case.

If you know about an issue in connection with this, please, change back the input type in HTML (using text type of course). I did not know it was intentionally only password and text.

"required" can be an annoying attribute even in a new browser. In my experience it is not handled always perfectly. For example when the input is replaced dinamically by a JS plugin with something else. The browser is not able to show you which input was required.


great
multiplayer with server has been temporary disabled

On Wed, Apr 1, 2015 at 12:37 PM, Dilshat <notifications@github.com> wrote:

> :) hehe,it is working, thank you , cheers!!!
> and how to use the server? I haven't find instructions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nesbox/emulator/commit/686b5255cd496c077e6aaa13e82e1436db6720f5#commitcomment-10515557>
> .
>

--
Vadim Grigoruk
grigoruk@gmail.com


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13463/testReport/) for this commit**


It would be much less hacky to have this as a boolean in the group in the config. Or as a boolean for each subject provided by the server. For instance, this requires code duplication in the Android app.


the use of class constants is preferred over global constants
it is also a good practice to have files that do not have any side effect


I don't see any reason to make that property public


For some reason it won't decorate the users when using Rabl.


Хотя нет, не получится. Используемый интерфейс Grizzly HttpHandler.service(request, response) throws Exception.


`set` would have been a better prefix than `add`


Are you able to document this function? (and the other new ones)


added comment


可以自动刷新了。。。


So far this is the only one and it's not likely to change. We'll keep that in mind though.


Warum hat es keinen Konstruktor?


is it possible to merge and publish a new npm please ?


thank you


Commenting
```r
if (library("ncdf4", logical.return = TRUE)) {
- library(ncdf4)
-} else install.packages("ncdf4")
```


Fun times :P


I think gpio should not be a "CPU peripheral".


*cough* :D


Not tested or seen this error myself but I'll merge it!


all fixed


Thanks PrestaEdit, I just fixed it https://github.com/PrestaShop/build.prestashop.com/commit/e299d377c0067ebb857cb14ca97ea9de24101903


I like how the logic was split


as it is called as a static method, it should have probably been defined as a static method


Warum sind die Felder ```public```?
Benutze getter und setter oder nur getter, wenn du alle Felder über den Konstruktor schon setztest.


I am not sure assigning the constant to a variable brings much value


:+1: ;-)


Most won't use the make file to install, however given that we do at least provide one, I guess that's probably the only point of reference we have.  Let's add a comment that at least indicates where these are expected (and that its using the makefile as the reference) rather than making folks count the "../" groupings.


@longnguyen11288 - can we / how do we upgrade the shared the environment to use v0.9.0.rc2 of bosh-workspace?

@rkoster - do I need to change how I am referencing the `logsearch-for-cloudfoundry-boshrelease`, or should it just automagically find the bosh-release in the subfolder?


- http://www.vogella.com/tutorials/JavaSerialization/article.html
- Wenn du eine eigene Klasse machst, solltest du ```toString()```, ```hashCode()``` und ```equals(Object object)``` Methoden überschreiben.


does it really cause a copy? taken from eigen map description: "While one option is to make a copy of the data, most commonly you probably want to re-use this memory as an Eigen type. Fortunately, this is very easy with the Map class."


I have no idea reading this what's going on


`position_escaped`: You might be right, I do not really know how regexes are matched and just used this because it seemed to work. It would be trivial to make 'pythonic' check out of `position_escaped` that just counts the amount of backslashes from behind. I might try the performance of this for several cases.


It doesn't compile for me on Ubuntu 64 bit.

To be sure not having any submodules issues, I deleted the previous directory and downloaded all freshly:
```bash
git clone https://github.com/CoolProp/CoolProp --recursive
cd CoolProp
mkdir build && cd build
cmake .. -DCOOLPROP_64BIT_SHARED_LIBRARY=ON
cmake --build .
```
And it gives this:
```cmake
[ 20%] Building CXX object CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o
In file included from /home/johndoe/CoolProp/src/Backends/IF97/IF97Backend.h:6:0,
                 from /home/johndoe/CoolProp/src/AbstractState.cpp:17:
/home/johndoe/CoolProp/externals/IF97/IF97.h:316:5: error: braces around scalar initializer for type ‘double’
     };
     ^
/home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘double IF97::Region3Backwards::Region3_v_TP(char, double, double)’:
/home/johndoe/CoolProp/externals/IF97/IF97.h:1517:83: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                     throw std::out_of_range::out_of_range("Unable to match region");
                                                                                ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:1517:83: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘double IF97::Region3Backwards::DividingLine(IF97::Region3Backwards::DividingLineEnum, double)’:
/home/johndoe/CoolProp/externals/IF97/IF97.h:1713:90: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                     throw std::out_of_range::out_of_range("Unable to match dividing line");
                                                                                ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:1713:90: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘char IF97::Region3Backwards::BackwardsRegion3SubRegionDetermination(double, double)’:
/home/johndoe/CoolProp/externals/IF97/IF97.h:1720:68: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                throw std::out_of_range::out_of_range("Out of range");
                                                                    ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:1720:68: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘char IF97::Region3Backwards::BackwardsRegion3RegionDetermination(double, double)’:
/home/johndoe/CoolProp/externals/IF97/IF97.h:1764:78: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                 throw std::out_of_range::out_of_range("pressure out of range");
                                                                              ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:1764:78: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h: In member function ‘double IF97::Region3::output(I97parameters, double, double)’:
/home/johndoe/CoolProp/externals/IF97/IF97.h:1977:74: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                 throw std::out_of_range::out_of_range("bad key to output");
                                                                          ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:1977:74: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘IF97::IF97REGIONS IF97::RegionDetermination_TP(double, double)’:
/home/johndoe/CoolProp/externals/IF97/IF97.h:2075:65: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
             throw std::out_of_range::out_of_range("Out of range");
                                                                 ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:2075:65: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h:2082:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                 throw std::out_of_range::out_of_range("Out of range");
                                                                     ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:2082:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h:2087:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                 throw std::out_of_range::out_of_range("Out of range");
                                                                     ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:2087:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h:2101:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
                 throw std::out_of_range::out_of_range("Out of range");
                                                                     ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:2101:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
/home/johndoe/CoolProp/externals/IF97/IF97.h:2111:65: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
             throw std::out_of_range::out_of_range("Out of range");
                                                                 ^
/home/johndoe/CoolProp/externals/IF97/IF97.h:2111:65: note: for a function-style cast, remove the redundant ‘::out_of_range’
CMakeFiles/CoolProp.dir/build.make:195: recipe for target 'CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o' failed
make[2]: *** [CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o] Error 1
CMakeFiles/Makefile2:63: recipe for target 'CMakeFiles/CoolProp.dir/all' failed
make[1]: *** [CMakeFiles/CoolProp.dir/all] Error 2
Makefile:116: recipe for target 'all' failed
make: *** [all] Error 2
```

I also tested to update submodules `git submodule foreach git pull origin master`, it update some externals modules but the error remain the same.

Did I miss anything?


Spaces missing around `%.`


Spaces missing around `%`.


Thanks! I was just in process of doing a pull to make this separate. Read my mind.


Hey Max,
Can I still reach you at this emai?

Sent via the Samsung Galaxy S® 5 ACTIVE™, an AT&T 4G LTE smartphone

-------- Original message --------
From: mmmaxwwwell <notifications@github.com>
Date:08/29/2013  12:02 AM  (GMT-05:00)
To: mmmaxwwwell/KolaHoops-Galaxy <KolaHoops-Galaxy@noreply.github.com>
Cc: bigmike3516 <bigmike3516@yahoo.com>
Subject: Re: [KolaHoops-Galaxy] remove mag & acc, housekeeping (801c4c9)

Hi Mike,
My name is Max, I'm 25, my day job is delivering fitness equipment, and I
have a businesses on the side, check out our website at
www.kolahoops.comand look us up on youtube and instagram (#kolahoops)
for more.

Everything I do is open source. It helps because we have lots and lots of
resources and we don't have to pay any licensing fees. This also means any
software I come up with is free to download, modify and use.

As for rate this depends on what your trying to do. Are you trying to come
out with a product or are you just cooking something up for yourself? I did
some research and found something about a dj booth, light wand and a touch
screen, but didn't really get a feel for what your trying to do. What do
you want it to do? Where and how do you see it being used primarily?

If you only need software my rate would be by the job.

Because you're still considering processors I'm assuming that you don't
have a working prototype, and I can help with that as well.

If you need the finished product to show up assembled and ready to go the
rate would be by item. Or if you have another idea i'm open to suggestions
and would love to hear it.

If your working on a hobbyist project and you want to collaborate I'm
interested in what you have to bring to the table and will definitely try
to get your project off the ground.

Thanks Mike
Max
On Aug 28, 2013 8:12 PM, "bigmike3516" <notifications@github.com> wrote:

> Hello,
> I appreciate the response. I also apologise for using my yahoo account, I
> didnt want to post my professional email in an open forum. I would like to
> start by getting some background on you like; credentials, projects
> previous and current (general info), and most importantly rate. I am
> currently trying to work with the lpd8806 and am exploring microprocessor
> options. If you feel prominent in either code writing for the lpd8806
> and/or chips let me know.
> Regards,
> Mike S.
>
>
> Sent via the Samsung GALAXY S®4, an AT&T 4G LTE smartphone
>
> -------- Original message --------
> From: mmmaxwwwell <notifications@github.com>
> Date: 08/28/2013 7:24 PM (GMT-04:00)
> To: mmmaxwwwell/KolaHoops-Galaxy <KolaHoops-Galaxy@noreply.github.com>
> Cc: bigmike3516 <bigmike3516@yahoo.com>
> Subject: Re: [KolaHoops-Galaxy] remove mag & acc, housekeeping (801c4c9)
>
> Hello, sorry for the delayed response I lost your e-mail in a sea of spam.
> I would be interested in paid work. What did you have in mind?
> On Aug 21, 2013 6:24 PM, "bigmike3516" <notifications@github.com> wrote:
>
> > Would you be interested in paid work for me? If so contact me at
> > bigmike3516@yahoo.com
> >
> > —
> > Reply to this email directly or view it on GitHub<
> https://github.com/mmmaxwwwell/KolaHoops-Galaxy/commit/801c4c984c422b67d2e8697b7ead5a9bed3e0ce8#commitcomment-3915974>
>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mmmaxwwwell/KolaHoops-Galaxy/commit/801c4c984c422b67d2e8697b7ead5a9bed3e0ce8#commitcomment-3968506>
> .
>
—
Reply to this email directly or view it on GitHub.


Should have used true/false instead of 1/0


I've republished all artifacts in org.scalameta to make sure that they don't collide (i.e. that scalahost assembles exactly the classfiles that scalameta has). You could try rebuild and see what happens. Maybe that was my problem not yours.


Maybe you should run it through [pep8](https://pypi.python.org/pypi/pep8) as well.
We've been conforming to PEP8 for the last years and we are making sure that all contributions follow it.
You may also want to use `.format(...)` instead of `%`, which is the new syntax that is preferred in Python 3. We are also using it on new code.


Ok I can understand. It was a bad assumption from me. I don't really understand why a getRequestDispatcher method was introduced in ServletContext since a RequestDispatcher can only be used inside a real request scope. Actually you could close an related request I opened few days ago: https://bugs.eclipse.org/bugs/show_bug.cgi?id=463390

Thanks


Maybe a simpliest way : instanceof


I think the previous version was actually better, as it perfectly fit in one line and there is no need to enumerate al those parameters in such an explicit way.


Where do I get `lens >=4.8.0.1` ?


I'm not really fan of the fact that the implementation details are somewhat leaked with the index


@sidekz
There is no static member called ProtocolGame::casts. What you meant is m_liveCasts, however that is private from this context so you need to call the getLiveCasts() member to retrieve a const reference to it.
'''
if(!g_config.getBoolean(ConfigManager::ENABLE_LIVE_CASTING) && !ProtocolGame::getLiveCasts().size())
'''


done


@magnayn person that did shading doesn't remember what he did and why. I already cleaned some crap of your copy-paste in my WIP branch, but your coding style cleanup will broke all my work. It looks like you even didn't check what is happening and how shading should work. Second shading should relocate guava classes, issue happens because shade doesn't work with hpi packaging.


Throwableを送出する宣言は概ね誤りであると言えます。


Might be better to phrase this as an `if` within an `each`, without the `select`:

```
if self.send("#{nation}_policy_url").blank?
  errors.add(nation, "must have a URL for its relevant policy")
end
```

Is more readable and has the minor benefits of checking for empty string as well as nil, and only looping the array once.


In general, I see that you are splitting many lines in half. Are you maybe using a maximum length of 80 characters? We normally use 120 in new Indico code.


RIP


prod ?


Still spaces missing around `%`. PEP8 would help here.


@mrdavidlaing you can just update the Gemfile and do a bundle install (as I have done in [this](https://github.com/cloudfoundry-community/logsearch-boshworkspace/pull/13) unmerged PR). The bosh-workspace should [automagically find the bosh-releases in the subfolder](https://github.com/cloudfoundry-incubator/bosh-workspace/blob/master/lib/bosh/workspace/release.rb#L47-L50).


Hade problem med port 3000 så ändrade det men inget som borde pushats. Fixar det asap


はい…。


got it，Thank you for your patience to answer！and how to I translate it to my language?


It doesn't even return the name :-)


The idea is to move the function `RegistrationPlugin.registerPlugin` to `slicer.util` and use it within the Editor, DICOMPlugin, ...

I am keeping it here until we finalize the design ...


This commit appears to have been mis-labeled. You never touched the `seeds` file...


Opa vc conseguiu finalizar? ou fez somente isso?


it should start at 12 now with the object layout not having the static indicator


It means that the file `PlmRegister.py` could be updated by using the following two lines at the end:

```
from RegistrationLib import registerRegistrationPlugin
registerRegistrationPlugin(os.path.splitext(os.path.basename(__file__))[0], globals())
```


might have edited the file from within the vm


Thanks for an explanation! I am trying to figure out how to use `el-get`, `use-package` and `req-package` together. It looks like el-get already provides features similar to `use-package` already. Does `req-package` use only fetching capabilities of el-get to get packages or does it still go through the full recipe? `el-get` already has `:before` and `:after` hooks and `use-package` has same.


Looks like its fixed on `are-we-there-yet` master


I was curious when this was going to make an appearance. I'm reading the commits one by one, watching the story of your app unfold. Having an actual value of the `Rating` was foreshadowed by the name.

:+1:


makeArrayBanana$ also uses ebx, either we change that or we push ebx here


Yeah, that one ended up messy. It saved a line though. The sentence still makes sense, right?


@rtopitt @wpolicarpo @leandrost Achei que seria interessante colocar o inch-ci no projeto, achei a ferramenta bem interessante http://inch-ci.org/github/myfreecomm/emites-client-ruby


with ebx having to be pushed, another pop needs to be done


1) It would be preferable if you wrote the user story and it's number at the top of your UML diagrams
2) underline the pages used in your UML diagram ->> easier to be recognized


How much speed up is due to wkt -> wkb vs non-thread-local -> thread-local?


Logging an error at info level? In this case it seems to warrant a warning, perhaps?  If not, a brief comment explaining why.  Also not that you're logging it twice (L185 and L187)


Another commit message that seems to incorrectly describe the diff... :smirk_cat:


are you trying to move eax into ebx? if so it should be mov ebx, eax


Same comment as above.


check


@andyli travis fails on `macro` now complaining about `hxcs` not being installed :-(


@melissa or @branan - can you take a look?


I'm happy with any OSS license or public domain, so you have my permission. I agree that it would be good to preserve git history. Moving the repo would also preserve stars and automatically provide a redirect from this address, I believe. You just need to delete the rust-gnome/gtk project, then move this one to the rust-gnome organization, then rename it to gtk. I think that is possible.


Yep! I was just impressed by your deftness.


I usually try to avoid committing empty files. The message is accurate, but it seems out of place to do this now.


90119792FourPipeInd_ErrMsg (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-69.6%25-red.svg)


Is it possible to request for a documentation? How do I proceed with such request?


Manque les scripts CURL pour tester :)


@khadijaibrahim  @samaasallam  I modified my Sequence Diagram as requested


c'est pas bon


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1253
success: http://build.servo.org/builders/linux2/builds/1251
success: http://build.servo.org/builders/android/builds/1287
success: http://build.servo.org/builders/mac1/builds/1466
success: http://build.servo.org/builders/mac2/builds/1469
success: http://build.servo.org/builders/gonk/builds/432


MarkdownDocs (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1668 of 1670 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.88%25-yellow.svg)


hey @alexcrichton is this ... okay? I would have assumed that `-1` is/was some sort of error signal?  is this just old code that is now buggy?


:tada:


It shouldn't be extra a module


As defprotocol defines a module


ce test ne passe pas.


- C'est cool de vouloir faire des test unitaires, mais il faut les lancer pour être sûre que ça marche :)
- Pourquoi avoir créer un controller Addresse oO ?


NOTE: The project, as imported in Eclipse from the Git plugin, should now build. These files were missing and critical to the build, apparently.


91069532-FixUnitTestFailures-Part2 (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1680 of 1680 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


fast-forwarding master to auto = 3294778b


Yeah this test is pretty ancient and it's not really necessary any more, `-1` in theory means that it successfully read `uint::MAX` bytes (bad)


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/59afada75e8eab6c6a173824488f1cd3fafa5d35


merging basho/riak_cs/cse-rewrite-releasenotes-2.0 = 59afada7 into borshop-integration-1112-cse-rewrite-releasenotes-2.0


basho/riak_cs/cse-rewrite-releasenotes-2.0 = 59afada7 merged ok, testing candidate = ec1692b3


why do you delete a part of the argument?


What was the change in FillData? Was there something wrong with the K-Nearest neighbors NAN filling method?

Need to know if I need to merge your changes in and rerun my results for the linear regressors.


If merging a mod that had itemblocks into another mod, what is the correct way to map the item and the block?
remap the block only or both the item and the block with getItemFromBlock for the itemblock part?


This commit is full of good stuff. Simple enough, but it makes a big difference to the user.


This will cause valid credentials to report a failure as is...


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/10642/testReport/) for this commit**


Perfect (Y)
Changes Confirmed


We could do that... but why would it be needed? It's like putting parents around a strlen().


Should be wrapped in try/catch as JHttp throws exceptions on failed connections


saw approval from jdm
at https://github.com/pcwalton/servo/commit/12336defa575be4071f7666683fddbdb6953a6eb


merging pcwalton/servo/reparent-child = 12336def into auto


pcwalton/servo/reparent-child = 12336def merged ok, testing candidate = c70e2645


This line here: getCdmSchema() should be replaced with getOhdsiSchema() and the token you replace is somethign like OHDSI_schema (it can be whatever you put into your sql script).  This is assuming you are going to move this into the OHDSI schema (which i think is what you should do here).


Some pictures still not working. Not quite sure why


You need to add a new one or remove the extra one. The program do not compile.


Here's another case: just change @CDM_schema with @OHDSI_schema throughout.

I know this is just naming that you could just send in your OHDSI_schema in as your CDM_schema, but I'm thinking of the people who are going to be looking at your SQL scripts and confused that you say CDM_schema when you actually mean OHDSI_schema.


What "wurstbot" is going to be?


Is the implementation of this function OK? Has added it to allow to coding of further control types if needed for something.


In my tests, the non-thread-local vs thread-local gives about 20-30%
improvement (serialization to JValue only), while the WKT -> WKB gave
anywhere from 3x - 10x.

On Wed, Apr 1, 2015 at 5:04 PM, Chi Tang <notifications@github.com> wrote:

> How much speed up is due to wkt -> wkb vs non-thread-local -> thread-local?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/socrata/soql-reference/commit/d0422af1cbd82870ddde43766cd20c5d8ed39b7c#commitcomment-10536222>
> .
>

--
Evan Chan   |   Principal Engineer, Socrata     |    @evanfchan
*Socrata <http://www.socrata.com/> *| The Data Experience Company™
*Socrata isn't about worldly success, but about changing the world
successfully. This is why I'm excited to be here.*
*What does Socrata do? <https://www.youtube.com/watch?v=yyz86MlkriQ> *(watch
our 2 min video)


Nit: Move to previous line to be inline with overall code style.


You shouldn't need the last line of this method for two reason:

1. Assignment should return the value that was assigned.
2. You are assigning to the instance variable inside the method, never using the return value.

The other way to write this would be to hide the "lookup logic" in the helper method, and assign the return value in the controller action.

```ruby
def get_game
  VideoGame.find(params[:id])
end

get '/video_games/:id' do
  @vgame = get_game
  erb :video_games:show
end
```
Neither is necessarily better, cases could be made for both, but it's good to know your options.


Basically you don't want to do `double.toString` if at all possible.  :)

On Wed, Apr 1, 2015 at 5:18 PM, Evan Chan <evan.chan@socrata.com> wrote:

> In my tests, the non-thread-local vs thread-local gives about 20-30%
> improvement (serialization to JValue only), while the WKT -> WKB gave
> anywhere from 3x - 10x.
>
> On Wed, Apr 1, 2015 at 5:04 PM, Chi Tang <notifications@github.com> wrote:
>
>> How much speed up is due to wkt -> wkb vs non-thread-local ->
>> thread-local?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/socrata/soql-reference/commit/d0422af1cbd82870ddde43766cd20c5d8ed39b7c#commitcomment-10536222>
>> .
>>
>
>
>
> --
> Evan Chan   |   Principal Engineer, Socrata     |    @evanfchan
> *Socrata <http://www.socrata.com/> *| The Data Experience Company™
> *Socrata isn't about worldly success, but about changing the world
> successfully. This is why I'm excited to be here.*
> *What does Socrata do? <https://www.youtube.com/watch?v=yyz86MlkriQ> *(watch
> our 2 min video)
>

--
Evan Chan   |   Principal Engineer, Socrata     |    @evanfchan
*Socrata <http://www.socrata.com/> *| The Data Experience Company™
*Socrata isn't about worldly success, but about changing the world
successfully. This is why I'm excited to be here.*
*What does Socrata do? <https://www.youtube.com/watch?v=yyz86MlkriQ> *(watch
our 2 min video)


Because `date` invocation changes according to:
``` if [ ${HISTORY} -gt 28 ]```


Oh, I see now... you did not add a pair of parens, just a left one. I'll fix.


Has leaved the loop here on parent, this also OK?
Currently the return value of "ParseGroupControl" becomes ignored, was not sure if it is OK to break the loop about one bad element.


ty fixed


oops. screwed the other check as well both fixed


It's unsafe to return SHRINK_STOP for older kernels when `sc->nr_to_scan == 0`.   It causing the following warning because the -1 is used to calculate `shrinker->nr`, you need to do something like this:

```
       if (sc->nr_to_scan && current->flags & PF_FSTRANS) {
               ...
       }
```

```
shrink_slab: ...+0x0/0x30 [spl] negative objects to delete nr=-9150747060186628098
```


Fixed in 1d72e60, thanks!


MarkdownDocs (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1003


I like seeing your thinking. You might consider adding these questions to the bodies of your commit messages rather than inline, but I'd rather have them than not.

I think the traditional way I've seen this done is to use `new` instead of `create`, and then call `save`.

```ruby
post '/video_games' do
  @vgame = VideoGame.new(title: params[:title], published_on: params[:published_on])
    if @vgame.save
      erb :video_games_new
    else
      redirect "/video_games/#{@vgame.id}"
    end
end
```
I think this is a little more concise, and I like having the "success case" first. For more context, I believe this is the way that the default Rails generators output controller code, so I've seen it more often. I think both ways are pretty intention-revealing though.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13510/testReport/) for this commit**


Umm, if this is a prank, well done.


@zurbchris

do you know why this was done?


I don't know about any new instructions yet. But there are some instructions that are supposed to be available in AVX-512, but missing in MIC, such as _mm512_shuffle_epi8. So some functions, like SWAP_ENDIAN, need to be implemented separately for MIC. I'm still working on this.


You'll be happy to know that this pattern of DRYing up your basic controller actions is so common that there are several gems that make this really easy. Good call. :+1:


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2103 of 2196 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-95.77%25-yellow.svg)


Due to accidentally makeing a .gitignore when creating the repo


You have 2 calls to `get_game`. This will perform 2 queries, which I don't think you ever intended for it to do.

I believe this is the only place you actually ever use the return value of that method. Since `get_game` is currently setting the instance variable, I'd probably just use it explicitly: `@vgame.destroy`.


Works for me.


It's just an example of using spring based configuration in OSGi. It's not the recommended approach, as Blueprint is preferred. I just keep this for legacy purposes.


@paul90 Hi Paul. What happened to these backend files? I assume they were just moved somewhere else?


donate


Yes, `npmlog` will be updated in the next version of npm.


You used the "side-effect-y erb" snippet here. The `title` and the `published_on` properties aren't being inserted into the template. A simple `<%= %>` should fix it right up.


my cluster still shows kubernetes/pause, is this something that changed recently?


out of curiosity, why ps -a, don't we want to verify the pause pod is running now?


@PeterOGB Thanks. I will try your suggestion. But could you please let me know on which platform this code will not work? I have only tested it on RPi B and RPi 2B.

BTW, I found another issue in the code yesterday: it cannot download Modules.symver file correctly. The download path is incorrect. But this code does works several weeks ago, I have not get enough time to figure out the reason.


You already know my opinion on writing code for a feature you aren't working on yet. If I were pairing with you, I'd have had you delete this particular form.


Sorry, @jimis, I am still not sure I follow.

Are you saying that with this change the invocation of the `date` command is now clearly dependent on the `HISTORY` variable value?  If so, then I am still not sure I see the dependency, unless you are saying it was possibly broken before your change.  In other words, can you show that the resulting value in `DATE` is now different and correct by moving that line of code (excluding a new configuration change to `YESTERDAY`)?


bug fixes mainly include the view when tabs are being clicked...guess some mess up in class calls..need to sort that and we are done with the userpage


I set a limit on the maximum distance of neighbors - 1000 bp. if there is
not a neighbor within this distance the mean is taken over the samples at
the position. In the paper i was reading it said that the correlation
between sites drops off dramatically after 1000 bp. If you have already run
it I wouldn't worry about it too much.

On Wed, Apr 1, 2015 at 8:13 PM, JonathanShor <notifications@github.com>
wrote:

> What was the change in FillData? Was there something wrong with the
> K-Nearest neighbors NAN filling method?
>
> Need to know if I need to merge your changes in and rerun my results for
> the linear regressors.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536329>
> .
>


I'm not sure why you replaced the link to the edit form here. I think this was a mistake?


:+1: Looks right.


I'm going to partially accept this. I don't think Santa is asking a question.


I believe @jlowdermilk changed this recently. If you do a kube-up at head, you don't see `google_containers/pause`?


Oh, I guess he is. Merging


is df sdf


I don't know. I have also wondered this. I'm happy to change this if you'd like, I was just afraid to.


Haven't tried, willing to believe.


BTW I just pushed the other feature set in RandomForest.py (using each
sample at each site for the training set).

On Wed, Apr 1, 2015 at 8:36 PM, Elizabeth Paul <epaul@princeton.edu> wrote:

> I set a limit on the maximum distance of neighbors - 1000 bp. if there is
> not a neighbor within this distance the mean is taken over the samples at
> the position. In the paper i was reading it said that the correlation
> between sites drops off dramatically after 1000 bp. If you have already run
> it I wouldn't worry about it too much.
>
> On Wed, Apr 1, 2015 at 8:13 PM, JonathanShor <notifications@github.com>
> wrote:
>
>> What was the change in FillData? Was there something wrong with the
>> K-Nearest neighbors NAN filling method?
>>
>> Need to know if I need to merge your changes in and rerun my results for
>> the linear regressors.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536329>
>> .
>>
>
>


Really?!?


Pre-filling a User's password on the page probably won't work, or is at the very least not a great idea. I'd leave `value` blank for password fields.


So what about:

```
SAFILES_REGEX='/sar?[0-9]{2,8}$'
COMPRESSED_SAFILES_REGEX="$SAFILES_REGEX(.(Z|gz|bz2|xz|lz|lzo))?$"
```
This avoids the repeated regex.  Just a suggestion.


:+1: This is the proper place to do this.


It's still jarring to read your commits where you keep editing the same migration. :scream_cat:


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Thanks for the quick fix!
Adding a comment why the buffer is copied would be great. It'd be even better if a test is added :-)


단지 하나의 엘리먼트에만 적용하기 위해 id를 css에 활용하는 건 안 좋다. 웬만하면 클래스로 하자.


I think you meant "E column" here.


Empty file commit.


Once again, you shouldn't need to try and return this value on another line.


Oh fuck no


Or maybe just `!evt.isCancelled()`


Was changed in #6322, so fairly recently.


Shouldn't we assert if `regs.size() > array_size(gp.regs)`?


Yeah, I'm not sure why it wasn't working before.

On Wednesday, April 1, 2015, PEMapModder <notifications@github.com> wrote:

> Or maybe just !evt.isCancelled()
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/94f2f1c66c1e4b799894c4c9afb34b068e24426e#commitcomment-10536659>
> .
>

--


91069532-FixUnitTestFailures-Part2 (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1680 of 1680 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Pinging Alter Ego @HP-Test/owners and @HP-Test @d4n13 @DanHP


I don't understand. What's so special about x0..x12 for gdb?


@samaasallam @khadijaibrahim


And is there any reason you don't just start iteration from 13 instead of 0?


You didn't even ban Shay-S in the ladder even though it was in the thread, what are you ebin


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/1252


fast-forwarding develop to borshop-integration-1112-cse-rewrite-releasenotes-2.0 = ec1692b3


Just wanted to thank you @gorhill for all the work you have done.


You might have forgotten a set of parentheses here.


hpwh_residential (nmerket) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-69.43%25-red.svg)


Since this is immediately after cluster creation, plain docker ps should return the pause pod.


Seems unused?


Fixed #308


Hey Jocelyn, yes people can send post requests through other means beyond a
browser, like cURL

On Wed, Apr 1, 2015, 10:43 PM Jocelyn Ong <notifications@github.com> wrote:

> Hey @KernCheh <https://github.com/KernCheh>! Thanks for the advice! :)
> Hmm, but would it still be dangerous even if there are no input fields but
> only a button to execute the POST request?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jawslyn/paypal-demo/commit/299250e2e58660e63d747f495d163a08f2ae1b3b#commitcomment-10524116>
> .
>


能力越强，责任越大


90119792FourPipeInd_ErrMsg (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1160 of 1161 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.91%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.59%25-green.svg)


Done :)


All this junk logging was cleaned up by the next commit ("Clean up all the extra logging statements in the new ServiceController."). I can rebase them together before merging if you'd like.

This would seem to be the downside of reviewing changes by looking at individual commits rather than the full PR.


I can't believe how well timed this is. I've been debugging this for the last 2 hours and was just about to post an issue when I noticed this commit!


I like you proactively adding seed data. Keeps you honest, as your screens will have some data on them.


I love that you're adding validation errors to the page. Probably a good candidate for DRYing up into a partial, since this looks very similar to the block in the video games form.


Still fighting with redirect? :P


Branch switching now works.


91069532-FixUnitTestFailures-Part2 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


fast-forwarding 2.0 to borshop-integration-1113-cse-rewrite-releasenotes-2.0 = ec1692b3


Great to see you using `locals` instead of just accessing the instance variables inside the partials. It becomes really hard to trace back errors once you start getting nested partials that access instance variables.


I am pretty sure this have broken one unit test in http://ci.seleniumhq.org:8080/job/Grid%20e2e%20tests/

Specifically, http://ci.seleniumhq.org:8080/job/Grid%20e2e%20tests/1035/testReport/junit/org.openqa.grid.e2e.node/DefaultProxyFindsFirefoxLocationsTest/testBrowserLocations/


如果瞎子失败了 -》如果下载失败了


This did not get its own language config.


Cool. Be sure to mention that & ref it in the paper.

Once I merge the change, it's not hard to rerun. Scripts all setup.

Jonathan Shor
-Sent from my iPhone

On Apr 1, 2015, at 20:36, Elizabeth Paul <notifications@github.com<mailto:notifications@github.com>> wrote:

I set a limit on the maximum distance of neighbors - 1000 bp. if there is
not a neighbor within this distance the mean is taken over the samples at
the position. In the paper i was reading it said that the correlation
between sites drops off dramatically after 1000 bp. If you have already run
it I wouldn't worry about it too much.

On Wed, Apr 1, 2015 at 8:13 PM, JonathanShor <notifications@github.com<mailto:notifications@github.com>>
wrote:

> What was the change in FillData? Was there something wrong with the
> K-Nearest neighbors NAN filling method?
>
> Need to know if I need to merge your changes in and rerun my results for
> the linear regressors.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536329>
> .
>

—
Reply to this email directly or view it on GitHub<https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536563>.


91069532-FixUnitTestFailures-Part2 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1680 of 1680 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


DesignDaysGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Would be a good idea so people don't have to reload the plugin manually/reboot the server.


Oh is it changed/all working? I'll take a look soon.

Jonathan Shor
-Sent from my iPhone

On Apr 1, 2015, at 20:39, Elizabeth Paul <notifications@github.com<mailto:notifications@github.com>> wrote:

BTW I just pushed the other feature set in RandomForest.py (using each
sample at each site for the training set).

On Wed, Apr 1, 2015 at 8:36 PM, Elizabeth Paul <epaul@princeton.edu<mailto:epaul@princeton.edu>> wrote:

> I set a limit on the maximum distance of neighbors - 1000 bp. if there is
> not a neighbor within this distance the mean is taken over the samples at
> the position. In the paper i was reading it said that the correlation
> between sites drops off dramatically after 1000 bp. If you have already run
> it I wouldn't worry about it too much.
>
> On Wed, Apr 1, 2015 at 8:13 PM, JonathanShor <notifications@github.com<mailto:notifications@github.com>>
> wrote:
>
>> What was the change in FillData? Was there something wrong with the
>> K-Nearest neighbors NAN filling method?
>>
>> Need to know if I need to merge your changes in and rerun my results for
>> the linear regressors.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536329>
>> .
>>
>
>

—
Reply to this email directly or view it on GitHub<https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536596>.


There is an issue with duplicates for LA legislators.  I am getting 164 legislators in the response.  There are only 144 total legislators: 39 upper / 105 lower


These loggers should be using AB.log(), and making sure that [PluginName] is removed (if you use AB.log(), since the tag is automatically injected).


DraweeViews只有有限的功能,但他提供的这些是具有决定性的 -》 DraweeViews 的功能不多,但都是至关重要的。


辛苦了么么哒～


Se agrego a Dario Pfeng a desarrolladores
Se agrego descripcion de instalacion en bloque


Also just noticed, disabling the plugin isn't a good idea for not being able to load the language file because there are already default values in the Enum.  So if it fails to load, it'll use the default language (English).


Changed the naming to provide consistency. Here is the link to the podcast mentioned above.

 [Adventures in Angular 35: The Current State of Angular with Brad Green, Igor Minar, and Miško Hevery](http://devchat.tv/adventures-in-angular/035-aia-the-current-state-of-angular-with-brad-green-igor-minar-and-mi-ko-hevery)


Alright, my bad :P. I'll fix it tonight. Otherwise, any complaints with the enum?

Sent from my iPad

> On Apr 1, 2015, at 8:06 PM, Joe <notifications@github.com> wrote:
>
> Also just noticed, disabling the plugin isn't a good idea for not being able to load the language file because there are already default values in the Enum. So if it fails to load, it'll use the default language (English).
>
> —
> Reply to this email directly or view it on GitHub.
>


again offset should start at 12


it needs to be string_literal.length() * 4


I know I missed several of these spots during my implementation of language, so feel free to ignore me through the rest here as it's not meant to be criticism or anything, just as a note for you and I to know which spots need to be added to the language configuration.


Language configure simplifies it because you can get a prepare one and swap it out for different languages. (I'm working on a Spanish one.)

Sent from my iPad

> On Apr 1, 2015, at 8:02 PM, Joe <notifications@github.com> wrote:
>
> This did not get its own language config.
>
> —
> Reply to this email directly or view it on GitHub.
>


makeArrayBanana$


No worries.  I like the individual commits.  I have to head off tonight,
but will finish off in the morning.

On Wed, Apr 1, 2015 at 5:55 PM, Alex Robinson <notifications@github.com>
wrote:

> All this junk logging was cleaned up by the next commit ("Clean up all the
> extra logging statements in the new ServiceController."). I can rebase them
> together before merging if you'd like.
>
> This would seem to be the downside of reviewing changes by looking at
> individual commits rather than the full PR.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/a-robinson/kubernetes/commit/cfee72c9001db705df1d83649aa5989b45c8f8dc#commitcomment-10536775>
> .
>


This did not get it's own language configuration (refer to my first note about this).


그리고 .form-control을 그대로 쓰는 것보다, 의도가 드러나게 클래스를 만드는 게 좋을 것 같다. 같이 고쳐보자.


No worries, I will be sure to implement it as best I can.

Sent from my iPad

> On Apr 1, 2015, at 8:08 PM, Joe <notifications@github.com> wrote:
>
> I know I missed several of these spots during my implementation of language, so feel free to ignore me through the rest here as it's not meant to be criticism or anything, just as a note for you and I to know which spots need to be added to the language configuration.
>
> —
> Reply to this email directly or view it on GitHub.
>


This also fixes #1.


yes, seems impossibru


Do you mean it doesn't generate or that it shouldn't?

Sent from my iPad

> On Apr 1, 2015, at 8:09 PM, Joe <notifications@github.com> wrote:
>
> This did not get it's own language configuration (refer to my first note about this).
>
> —
> Reply to this email directly or view it on GitHub.
>


I thought it had the wildcard too, so I was like, wait... where'd it go? So I added it back in.


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


It's probably more Scala-y to make these `apply(...)` methods.


From a quick analysis, this needs to be injected with the login delay interval that comes from the main configuration.


Taking botting to a whole new level


This file should be dynamically created from the Enum, not have it already as a resource (I personally find dropping files from the JAR a pet peeve of mine).


I'm on mobile at the moment; what, specifically?

Sent from my iPad

> On Apr 1, 2015, at 8:14 PM, Joe <notifications@github.com> wrote:
>
> From a quick analysis, this needs to be injected with the login delay interval that comes from the main configuration.
>
> —
> Reply to this email directly or view it on GitHub.
>


Alright. I can make it. Might be a bit of pain, but I can :P

Sent from my iPad

> On Apr 1, 2015, at 8:15 PM, Joe <notifications@github.com> wrote:
>
> This file should be dynamically created from the Enum, not have it already as a resource (I personally find dropping files from the JAR a pet peeve of mine).
>
> —
> Reply to this email directly or view it on GitHub.
>


jjghghgjhg


Can use `JsonUtil.renderJson(result)`


Hi,
Follow me on APACGAMONDE


Do you want to merge this?


There seems to be code in there already, so it isn't too much work besides deleting this file.


lgtm


Check for selected state for options with optgroups.


Woo, that looks like just the feature set I was picturing. I just tried it, and a nan got generated in X or y somehow tho. Gonna see if I can figure where.

Do you use any instant message service we could talk chat in pseudo real time, like gchat or something? Convenient when coding I find :)

Jonathan Shor
-Sent from my iPhone

On Apr 1, 2015, at 20:39, Elizabeth Paul <notifications@github.com<mailto:notifications@github.com>> wrote:

BTW I just pushed the other feature set in RandomForest.py (using each
sample at each site for the training set).

On Wed, Apr 1, 2015 at 8:36 PM, Elizabeth Paul <epaul@princeton.edu<mailto:epaul@princeton.edu>> wrote:

> I set a limit on the maximum distance of neighbors - 1000 bp. if there is
> not a neighbor within this distance the mean is taken over the samples at
> the position. In the paper i was reading it said that the correlation
> between sites drops off dramatically after 1000 bp. If you have already run
> it I wouldn't worry about it too much.
>
> On Wed, Apr 1, 2015 at 8:13 PM, JonathanShor <notifications@github.com<mailto:notifications@github.com>>
> wrote:
>
>> What was the change in FillData? Was there something wrong with the
>> K-Nearest neighbors NAN filling method?
>>
>> Need to know if I need to merge your changes in and rerun my results for
>> the linear regressors.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536329>
>> .
>>
>
>

—
Reply to this email directly or view it on GitHub<https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536596>.


試一下tryGet還是怎樣，如果沒有的話給預設值
現在唯有這樣了 :sweat:


this throws an error:
java.lang.IllegalArgumentException: logaudit is not a valid syslog facility string

talk to ray and agree on one ... local0-local7 I'm gonna guess?


Aight. Also, I'm thinking we might make loadLang() a static method so we can use it on reload. Any way you know of we could circumvent the static reference to a non static method?

Sent from my iPad

> On Apr 1, 2015, at 8:17 PM, Joe <notifications@github.com> wrote:
>
> There seems to be code in there already, so it isn't too much work besides deleting this file.
>
> —
> Reply to this email directly or view it on GitHub.
>


A whole lot of nitpicking here:

I tend to prefer `.nil?` instead of `== nil`. It is just as intention revealing, but I can't easily typo it and *assign nil* instead.

When assigning to the session, I like being more clear with the value I'm logging a user in with. So instead of `session[:id]`, I'd usually use `session[:user_id]`. (At the end of the day, in a production environment, I'd usually prefer to use an authentication token, rather than the `User#id`, but that's too far right now.)

The `if` statement is the last expression in this method, so both branches are the return value for the method. You don't need the explicit `return` in `return nil`.

Not having an else case is the same as having an else case with a value of nil, so you don't truly even need the "failure case" here at all.

If you want to be more explicit about it though, I tend to prefer a quick short circuit return, or having my "success case" first.

If I were going to write this myself, I'd probably end up with either:

```ruby
def current_user
  @current_user ||= User.find(session[:user_id]) if session[:user_id]
end
```
or in the explicit case:

```ruby
def current_user
  return nil if session[:user_id].nil?

  @current_user ||= User.find(session[:user_id])
end
```


And overall, what do you think of the Language enum rather than a regular class?

Sent from my iPad

> On Apr 1, 2015, at 8:17 PM, Joe <notifications@github.com> wrote:
>
> There seems to be code in there already, so it isn't too much work besides deleting this file.
>
> —
> Reply to this email directly or view it on GitHub.
>


Yeah I use gchat. I'll try adding you.

On Wed, Apr 1, 2015 at 9:18 PM, JonathanShor <notifications@github.com>
wrote:

> Woo, that looks like just the feature set I was picturing. I just tried
> it, and a nan got generated in X or y somehow tho. Gonna see if I can
> figure where.
>
> Do you use any instant message service we could talk chat in pseudo real
> time, like gchat or something? Convenient when coding I find :)
>
>
> Jonathan Shor
> -Sent from my iPhone
>
> On Apr 1, 2015, at 20:39, Elizabeth Paul <notifications@github.com<mailto:
> notifications@github.com>> wrote:
>
> BTW I just pushed the other feature set in RandomForest.py (using each
> sample at each site for the training set).
>
> On Wed, Apr 1, 2015 at 8:36 PM, Elizabeth Paul <epaul@princeton.edu
> <mailto:epaul@princeton.edu>> wrote:
>
> > I set a limit on the maximum distance of neighbors - 1000 bp. if there is
> > not a neighbor within this distance the mean is taken over the samples at
> > the position. In the paper i was reading it said that the correlation
> > between sites drops off dramatically after 1000 bp. If you have already
> run
> > it I wouldn't worry about it too much.
> >
> > On Wed, Apr 1, 2015 at 8:13 PM, JonathanShor <notifications@github.com
> <mailto:notifications@github.com>>
> > wrote:
> >
> >> What was the change in FillData? Was there something wrong with the
> >> K-Nearest neighbors NAN filling method?
> >>
> >> Need to know if I need to merge your changes in and rerun my results for
> >> the linear regressors.
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536329
> >
> >> .
> >>
> >
> >
>
> —
> Reply to this email directly or view it on GitHub<
> https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536596
> >.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10537001>
> .
>


yeah we can use LOCAL0-7 or have to use the correct log audit value "LOG_AUDIT"


I'll check and see what's up..

On Wed, Apr 1, 2015 at 9:20 PM, Elizabeth Paul <epaul@princeton.edu> wrote:

> Yeah I use gchat. I'll try adding you.
>
> On Wed, Apr 1, 2015 at 9:18 PM, JonathanShor <notifications@github.com>
> wrote:
>
>> Woo, that looks like just the feature set I was picturing. I just tried
>> it, and a nan got generated in X or y somehow tho. Gonna see if I can
>> figure where.
>>
>> Do you use any instant message service we could talk chat in pseudo real
>> time, like gchat or something? Convenient when coding I find :)
>>
>>
>> Jonathan Shor
>> -Sent from my iPhone
>>
>> On Apr 1, 2015, at 20:39, Elizabeth Paul <notifications@github.com
>> <mailto:notifications@github.com>> wrote:
>>
>> BTW I just pushed the other feature set in RandomForest.py (using each
>> sample at each site for the training set).
>>
>> On Wed, Apr 1, 2015 at 8:36 PM, Elizabeth Paul <epaul@princeton.edu
>> <mailto:epaul@princeton.edu>> wrote:
>>
>> > I set a limit on the maximum distance of neighbors - 1000 bp. if there
>> is
>> > not a neighbor within this distance the mean is taken over the samples
>> at
>> > the position. In the paper i was reading it said that the correlation
>> > between sites drops off dramatically after 1000 bp. If you have already
>> run
>> > it I wouldn't worry about it too much.
>> >
>> > On Wed, Apr 1, 2015 at 8:13 PM, JonathanShor <notifications@github.com
>> <mailto:notifications@github.com>>
>> > wrote:
>> >
>> >> What was the change in FillData? Was there something wrong with the
>> >> K-Nearest neighbors NAN filling method?
>> >>
>> >> Need to know if I need to merge your changes in and rerun my results
>> for
>> >> the linear regressors.
>> >>
>> >> —
>> >> Reply to this email directly or view it on GitHub
>> >> <
>> https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536329
>> >
>> >> .
>> >>
>> >
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub<
>> https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10536596
>> >.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/epaul626/COS-424-HW-2/commit/473eae0ea0801d4429da91b63b003216e199b063#commitcomment-10537001>
>> .
>>
>
>