Good work Charlotte!

Some feedback on your work.

Instead of having long unbroken paragraphs, try and use `<ul>` and `<li>` tags to bullet point your job description.

Also, to differentiate the sections of your resume, try adding smaller headers using another `<h2>` tag, and and `<hr>` tag.

I rewrote your code with some more specific HTML5 tags that will help you write clean, maintainable code moving forward. This kind of formatting is especially important when we start to write CSS, as it'll help you hone in on certain parts of your code quickly.

Please take a look at this and come tap me on the shoulder before class if you have any questions about the changes. You can also schedule office hours for tomorrow (tuesday) night and we can over them then!

Again, good work!
Zack


I randomly thought about this and realized this will break unit testing. If we replace certain plugs with `mock.Mock()`, it won't be a subclass of `BasePlug` anymore.


Whenever dynamic cast is used, the code could expect the casting to fail and therefore it is usually followed by an if statement. Whenever static cast is used, the code always expects the casting to succeed, usually it means the dev knows exactly what the object type is in the first place or like you said, it is unsafe to do the static cast to any arbitrarty type. So, although I am fine with the idea to only have dynamic cast variant and called them GetChildAs(), I am not sure how we can substitute the existing code using static cast variant with GetChildAs() cleanly, i.e. it seems weird to me to have dynamic cast and dereference without first checking the casting result but it makes perfect sense for static cast cause the dev is sure it cannot fail.


set up as a dedi on my PC, direct IP connect


Does not work for hierarchical modules.


plant-autosizing-gaps (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1111 of 1112 tests passed, 0 test warnings)

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


Hello,

I was wondering of there have been any advancements on this ?

Gerhard,


any error in the server log?


With this one I'm pretty sure you don't need a target either, so wouldn't that make the dummy effect fail if the player has no target when he uses it?


plant-autosizing-gaps (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


50s-granularity is not really usable here :) That one is used for kilometres.
But just using the original string (exceeded max speed) could be enough for not(tts).

Not all languages have a proper free TTS module so e.g. for mine pre-recorded is the only choice for common users. I'll upload a maintained and complete version soon.


First thought is always the best ;)

But I thought maybe because of the databinding stuff, cause somehow it should be notified if another option changes and therefore the pointedByList changes... But then again, how should that be possible without selecting the other element and make changes there?
I'm too preoccupied by my own editor, cause it worked with like 50 simultanously opened, Windows, so that I can just leave an ConversationOption open, than in the other window add a new event, so that it can be linked to the ConversationOption ... Really heavy with databinding stuff, which was/is fairly new to me :)

Also something that should be kept in mind is crossconversation pointing... But that's also fine to handle in a second (or third) step, after the main functions are working :+1:


Many thanks for this info, @tasmo!

Two requests:

1. When you have the chance, could you please expand on the installation/configuration steps you used? For example, if/how you installed the [required version](https://github.com/certbot/certbot#system-requirements) of python, the commands used to download, unpack, move, and set permissions for certbot? Since nginx support is described as "highly experimental", were there any other steps you had to go through?

2. In https://github.com/HistoryAtState/hsg-project/blob/master/deploy/automate_letsencrypt_ca.md should the `/letsencrypt/...` paths be prepended with a `~` or `/path/to`?


Any chance to see CHANGELOG for `1.3.4.4`?


Yeah. That was biting me in the rear. I thought it was by design! ;)


:+1:  Looks great so far!


@julianpistorius Doesn't look like guacamole has an authentication signature for the checksum


nope which is odd

	Line 775: 18:50:41 "Loading parking misc functions"
	Line 776: 18:50:41 "Loading parking misc functions complete"
	Line 780: 18:50:41 "Loading parking interact functions ..."
	Line 781: 18:50:41 "Loading parking interact functions complete"
	Line 785: 18:50:41 "Loading parking saving functions ..."
	Line 786: 18:50:41 "Loading parking saving functions complete"
	Line 788: 18:50:41 "Loading parking functions ..."
	Line 789: 18:50:41 "Setting up parking terminals ... "
	Line 811: 18:50:42 "Creating parking terminal at Parking1 (Grid 064053)"
	Line 811: 18:50:42 "Creating parking terminal at Parking1 (Grid 064053)"
	Line 816: 18:50:42 "Creating parking terminal at Parking2 (Grid 022055)"
	Line 816: 18:50:42 "Creating parking terminal at Parking2 (Grid 022055)"
	Line 823: 18:50:43 "Creating parking terminal at Parking3 (Grid 033029)"
	Line 823: 18:50:43 "Creating parking terminal at Parking3 (Grid 033029)"
	Line 832: 18:50:43 "Parking terminals setup complete"
	Line 833: 18:50:43 "Loading parking functions complete"


I think you want `v is not false` to correspond to standard Scheme behaviour.


Rebranding to accessNav


Done


changed to success: false


Im truely shock wagic even still runs on psp to be honest, watch you debugger(which unfortunately our release are debug releases), and youll see we run 130 to 160mb of ram with psp imagea its closer to 90, but that means psp is literally dumping ram as fast as it can load it. Anyways not the case if the user is on a cfw vita, those are equal to a weaker android phone


Well, the thing is, I think in blizz code, the script_effect summons the mob near caster, and the spell_effect_dummy does any attacking, I have noticed this trend all over ^^ we can do both in one


<script> теги ніколи не пиши у <head>, це блокує початкове відображення сторінки й погано впливає на швидкість першого завантаження сторінки. Завжди пиши в кінці <body>, перед його закриваючим тегом


This commit made my code break. I rely on the following line in position.js:

`$.ui = $.ui || {};`

I did not have any other dependency than position.js till now. Now, I need add version.js as another dependency. However, I liked being able to use only a single js file...


Yes, `foo+#` is much more reasonable, even if both orderings are technically equivalent in what they permit.  The semantics attached to to `foo#+` are nonsensical, as they imply the grouping of `a b, c d` is `(a) (b,c) (d)`, when it's clearly meant to be `(a b) (c d)`.

And there's no need for extra brackets, you can just stack the multipliers directly.


Why do we need this _ = e ? does not compile?


Hahahahahahaha


`fusion_pairs.txt` is fixed and it's sourced from https://github.com/ndaniel/fusioncatcher/blob/master/bin/generate_known.py (big thumbs up and thanks to Daniel)

If VEP is used for SVs then the script will kind of break apart I would imagine. Only tested with SnpEff. Sorry, my oversight totally. Never used VEP.


Good habit ^^


var number = String(inputNumber), translation = '', triplets = [], i, k, n, j   -    плохо

var number = String(inputNumber),
      translation = '',
      triplets = [],
      i,
      k,
      n,
      j;
хорошо


Можна одразу повернути 1 звідси, тобто замість цього рядка - `return 1;`


We'll be putting out a blog post (and later hopefully a manuscript) describing all the prioritisation levels etc.


Yeah, I meant to revert this.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.133](http://build.clear-measure.com/viewLog.html?buildId=9791&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


This should be a `IEnumerable<T>`


Can you put this in log.debug if this is required.


We need a better name. Maybe `IDataManager`? Not much better, though :(


👍   next time though, lets make it a PR.


`T` here is the partition type, right?


You may like this then: https://www.youtube.com/watch?v=ZZ5LpwO-An4


1. Never use generic types without type parameters!
2. According to the 'D' in SOLID (Dependency Inversion Principle), always rely on abstraction, never on a concreet class. This means that for static types, always choose the most generic (an interface or a superclass).


Oh im excited about this change, at work now, make sure it still works in other themes tho.


We might want to give blocks an order. So instead of injecting the block ids, maybe inject an `IPartition` which is `IEnumerable<IBlock>`? We might add random block access in the future, but I don't think we need it now.


Yes, cross-conversation pointers will be more advanced concept. The same goes with cross-package events, conditions etc. I'll do that later :stuck_out_tongue:


Can you log the git checkout failure with log.Errorf and return the error as below.


This probably should be immutable.


I was thinking the dummy was used for the random roll and the summons; a lot of dummy effects are basically summon effects that blizzard couldn't handle natively in their summon effects; just look how many engineering trinkets had dummy effects, just to do the summon properly. Our summon spells are a bit more advanced though and take care of all that natively so we have quite a few dummy effects which are basically nothing more than a generic spellcast (we handle some things more generically)


`elif` not `eif`


Same, probably immutable.


Btw You can pass a `style` param with the same style options as shields.io


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.133](http://build.clear-measure.com/viewLog.html?buildId=9791&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 72, ignored: 2 Build time: 00:04:17


Same. probably immutable


The reason I did that that was because "Volt Switch" was indexed [532], but it didn't seem to have any problem with the person that presented this issue.

If you do have an error with a pokemon using "Scald", post it on issues section.


What is the meaning behind this?
You start with an empty Map (called map) and a List of your test data (called servers).
You put each element from the list into the map and then you immediately put every entry value of the map into a DIFFERENT List.

Doesn't quite makes sense.


More advanced, maybe, but I think blizz spellsystem is much less heavyweight.


I think golint will error out on 'Url' and want URL - you probably will have to rename this field to catalogURLBranch


I have no idea... the double "Creating" entries makes it even weirder. Is that the vanilla version from my repo?


cc @jsha this should fix LetsEncrypt OCSP response.


Yes; we 'script' and handle a lot of stuff in special cases which I very much doubt blizzard didn't have generic behaviour for. We'll get there some day..


@mvglasow my vision of android software keyboard feature is at android-software-keyboard branch, please feel free to adjust, fix, merge or drop on the floor :)


As JavaFX/Swing relies on the OS for window decorations /some style elements there are some variations but I tested it now on Win/Ubuntu and both look fine.


Is it worth listing all the `property-*-blacklist` and `property-*-whitelist` rules individually? That would make it possible to use your editor or browser's text search to find the relevant entry.

Upgrading from 6.9.0 to 7.0.0 it's easy enough to read the changelog, but if it's a larger range it could be tricky to find this entry.


@pie-flavor that is*


I'm managing data binding by refreshing the whole view :no_mouth:  Not too sophisticated, but if it works it's not stupid :wink:


Загубив крапку з комою в кінці рядкуа


@pfalcon this line means that one can't build the esp8266 port with a build directory different to "build", eg make BUILD=blah does not work.  Is it safe to change this line to simply `*.o(.rodata*)`, or even `*(.rodata*)`?


Small cloak buff for worst AT gun.


even across packages? oh no please :stuck_out_tongue_winking_eye:

Yeah I saw the refresh-methods... never forget to KISS, Keep It Simple and Stupid :+1:


TaskCluster: @cgsheeh does not have permission to trigger tasks.


Sorry I have given up got all the errors fixed but now the designer hangs when you try to open a product

Sent from my iPhone

> On Jul 11, 2016, at 10:57 AM, Adam Moehrig <notifications@github.com> wrote:
>
> Has it ever worked?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


TeamCity Glyssen :: Glyssen pull requests [Build *.*.143.8b0719c8dceebcf76bd12a363fc4541e3fe9df4b](http://build.palaso.org/viewLog.html?buildId=74731&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1277, ignored: 17 Build time: 00:04:09


Define all struct at top for readability.


This brakes the whisper feature, lele is sad :unamused: :crying_cat_face:


Not sure if this is your issue Matt but this might help.
https://github.com/A3Armory/A3A_A3Wasteland.Stratis/issues/1


I like to uniform the menus to the latest hd theme (like the action stack) so I think we need to update the deckmenu layout. If We can change the deck editor layout. I just disabled the lowres horizontal and vertical pole here and replaced with a much simpler menu


I don't advise naming functions with a `$` prefix. The `$` prefix is generally reserved for jQuery elements.

See https://github.com/airbnb/javascript/tree/master/es5#jquery


Good thinking. Done in the PR.


Inaccurate comment.


You guys should really review with:
http://www.ibiblio.org/apollo/ScansForConversion/Luminary099/
http://www.ibiblio.org/apollo/ScansForConversion/Comanche055/

And report issues at the actual original source of these listings:
https://github.com/rburkey2005/virtualagc

That way we get the most accurate and historical place at the location that is associated with:
http://www.ibiblio.org/apollo/


					Enter Private Key
					<input type="text" id="detailprivkey" value="" onfocus="this.select();" onkeypress="if (event.keyCode == 13) ninja.wallets.detailwallet.viewDetails();" />
					<input type="button" id="detailview" value="View Details" onclick="ninja.wallets.detailwallet.viewDetails();" />
					<input type="button" name="print" id="detailprint" value="Print" onclick="window.print();" />

						Key Formats: WIF, WIFC, HEX, B64, B6, MINI, BIP38

						Enter BIP38 Passphrase <input type="text" id="detailprivkeypassphrase" value="" onfocus="this.select();" onkeypress="if (event.keyCode == 13) ninja.wallets.detailwallet.viewDetails();" />
						<input type="button" id="detaildecrypt" value="Decrypt BIP38" onclick="ninja.wallets.detailwallet.viewDetails();" />

						Your Litecoin Private Key is a unique secret number that only you know. It can be encoded in a number of different formats. Below we show the Litecoin Address and Public Key that corresponds to your Private Key as well as your Private Key in the most popular encoding formats (WIF, WIFC, HEX, B64).

						Litecoin v0.6+ stores public keys in compressed format. The client now also supports import and export of private keys with importprivkey/dumpprivkey. The format of the exported private key is determined by whether the address was generated in an old or new wallet.

							Litecoin Address

							Litecoin Address Compressed

						Public Key (130 characters [0-9A-F]):

						Public Key (compressed, 66 characters [0-9A-F]):

							Private Key WIF
51 characters base58, starts with a '5'

							Private Key WIF Compressed
52 characters base58, starts with a 'K' or 'L'

						Private Key Hexadecimal Format (64 characters [0-9A-F]):

						Private Key Base64 (44 characters):

						Private Key Mini Format (22, 26 or 30 characters, starts with an 'S'):

						Private Key Base6 Format (99 characters [0-5]):

						Private Key BIP38 Format (58 characters base58, starts with '6P'):

							How do I make a wallet using dice? What is B6?

An important part of creating a Litecoin wallet is ensuring the random numbers used to create the wallet are truly random. Physical randomness is better than computer generated pseudo-randomness. The easiest way to generate physical randomness is with dice. To create a Litecoin private key you only need one six sided die which you roll 99 times. Stopping each time to record the value of the die. When recording the values follow these rules: 1=1, 2=2, 3=3, 4=4, 5=5, 6=0. By doing this you are recording the big random number, your private key, in B6 or base 6 format. You can then enter the 99 character base 6 private key into the text field above and click View Details. You will then see the Litecoin address associated with your private key. You should also make note of your private key in WIF format since it is more widely used.

					Donations: 1NiNja1bUmhSoTXozBRBEtR8LeF9TGbZBN

					GitHub Repository
						(zip)

					Version History (//version)

				527B 5C82 B1F6 B2DB 72A0
ECBF 8749 7B91 6397 4F5A

					(PGP)
					(sig)

				Copyright bitaddress.org.
				JavaScript copyrights are included in the source.
				No warranty.


This is mostly work related to
1) using cv2 module to rescale image are reduce input size
2) mean centering and scaling images
3) PCA analysis on the full image and of segments of the images (this is related to an idea I had for initializing neural network weights, but note that by calculating Jenson-Shannon divergence of positive and negative images onto PC vectors we can find feature (PC vectors or linear combos of these vectors)  which should do a better job of distinguish between classes and could be used as input for leaning algorithm besides neural networks


Inaccurate comment


spin off game derived from early stage of a Real Time Strategy Project.  Goal is to keep player controlled blobs away from enemy blob.  Player controlled blobs are selected with right-mouse button and moved with left-mouse button.


jajaj como robas con este código todavía jajaj``


I don't think so. As $versions is a zero based array, getting to the previous entry is count() - 2. count() -1 is the last entry.

But if you think otherwise, can you generate a unit test that shows the issue please.


Цей цикл можна замінити одним єдиним методом namesArr.includes - https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/includes


This looks good.  Still wondering about the newlines.


need to rethink how i can say that you need to read first - to understand the steps.

- create replica set
- 'add' database
- create user

maybe I just write that!


Скрипти закжди підключай лише в саомому кінці тегу <body>, якщо будеш писати їх тут, це блокуватиме перший показ сторінки. Це погано для швидкодії


Line above is `*/frozen.o`, so that should work. Feel free to test ;-).


@gijsbertwerner I think we can you use your N2 fixing paper as reference also because it is a published example of the samp approach (removing random proportion of data in phylogenetic comparative methods).


Missing an opening `<ul>` tag


This changeset conflicts with the proposed fix for https://code.djangoproject.com/ticket/17657


Забери пробіл між назвою масиву і його індексом (`namesArray[i]`) і постав пробіли між `(i + 1)`


I don't recommend insert extra spaces into translatable string. This may produce some troubles if translator forgot to add space into begin of line. I highly recommend you to make full string like tr ("meow %1, bla bla bla, because %2 happen"), and every extra add-on pass via .arg("cat").arg ("damn thing");, but conditionally:
tr ("Meow: %1").arg(some?"ahh?":"hey!");


#5768-Example-files-which-show-discontinutity-in-air-or-water-flow-rate-through-fans-and-pumps (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2262 of 2262 tests passed, 0 test warnings)

![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)


Увесь цей цикл можна замінити Array.includes

`namesArray.includes(name) ? alert(name + 'success') : alert('Error!');`


you can use go for : range  for looping


@dhanraju : Now start working on pylint issues on existing code. Please fix it.

$ pylint programs/factorial.py
No config file found, using default configuration
************* Module factorial
W:  7, 0: Found indentation with tabs instead of spaces (mixed-indentation)
W: 14, 0: Bad indentation. Found 6 spaces, expected 8 (bad-indentation)
C:  1, 0: Missing module docstring (missing-docstring)
W:  4,21: Redefining name 'num' from outer scope (line 20) (redefined-outer-name)
C:  4, 0: Missing function docstring (missing-docstring)
W: 12,31: Redefining name 'num' from outer scope (line 20) (redefined-outer-name)
C: 12, 0: Missing function docstring (missing-docstring)
C: 20, 4: Invalid constant name "num" (invalid-name)
W: 20,14: Used builtin function 'input' (bad-builtin)


It's best practice to close your `<p>` tags right after the text inside is finished.


ActivityTracker has a function like that already, please check it out
You can call ActivityTracker.stageBlablabla. and then somehow get the latest CodeStamp


Якийсь во істину здоровезний відступ :)


Are we committing this file to the repo? we should log the "file not found" error

We should have the file empty I guess, else the sample repo will get cloned in all installs


#5768-Example-files-which-show-discontinutity-in-air-or-water-flow-rate-through-fans-and-pumps (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


#tbt


The `|| false` here does not clarify things. I'd suggest coercing to boolean with `return !!Template.instance().filter.get('region');`. This makes clear the helper always returns a boolean.


краще видали це. Не зберігай "хлам" у коді "на майбутнє"


відступ між селекторами елементів не більше 1 рядка


Good work Shrangie! Your resume looks slick, and I love that you used CSS in your code! Way to jump ahead!

Overall, your code looks very good. There are a few style and organizational guidelines that I think'll help you beef up your HTML going forward

I rewrote your code below with some more specific HTML5 tags that should help you write clean, maintainable code. This kind of formatting is especially important when we start to write more CSS, as it'll allow you hone in on certain parts of your code quickly.

Please take a look at this and come tap me on the shoulder before class if you have any questions about the changes. You can also schedule office hours for tomorrow (tuesday) night and we can over them then!

Again, great work!
Zack


тут можна прибрати останній нуль. Таким чином використаєш скорочення https://developer.mozilla.org/en-US/docs/Web/CSS/margin#Syntax


This change breaks `make -C ulib/ml'

```
ulib/ml
$ make
ocamlfind opt -thread -package batteries,zarith -linkpkg -g   -c -I . -package stdint -linkpkg MkPrims.ml
ocamlfind opt -thread -package batteries,zarith -linkpkg -g   -c -I . -package stdint -linkpkg prims.ml
ocamlfind opt -thread -package batteries,zarith -linkpkg -g   -c -I . -package stdint -linkpkg FStar_ST.ml
make: *** No rule to make target 'FStar_Mul.o', needed by 'all'.  Stop.```


між відкриваючою/закриваючою фігурною дужкою і властивостями CSS не має бути жодних пустих лінійок


<script> не після `</body>`, а одразу ж перед


Весь этот цикл получения триплетов можно заменить одной строкой.
Забейте в Гугле "regex javascript match three numbers" и прочитайте первые пару ссылок.


True, the places where the code has practically constructed the hierarchy by itself in code and can be sure of the type, static cast without checking certainly makes sense. In pathological case a user might have injected another element into the hierarchy, in which case dynamic cast + nullcheck would be only way to be safe, but it's rather far-fetched.


I think I've got it pretty much figured out. I'll put up a pull request.


Перша версія нікуди не годиться. Це називається "Hardcode'"ом. Й вкрай не вітається в жодному колі програмістів


Здесь скобки не нужны вокруг translation.


I knew that, but I ignored it because I'm certain you'll fix the code to allow your whisper feature, too.


awe are iterating over CatalogObjectJson and modifying the same collection. Not sure about go, but in Java this is to be avoided.

Instead, can we just loop over CatalogObjectJson and then append to CatalogObject if not empty.


Що, якщо користувач введе пустий рядок? Тобто по суті нічого не введе?


ззавеликий відступ


Hey! I found this after someone asked me about a FOSS license on the upstream fork under the cla-rce org. Do you have a specific need (like travisci needing public github.com) for keeping this on github.com?  At this point, the upstream on cla-rce is way out of date from the true upstream at UMN. I could add this feature to the up-upstream, and we're considering putting it on github.com as a supported origin.  I was actually about to add this functionality anyway.


Увесь цей цикл, як і наступний "if" можна написати одним рядком, використовуючи метод масивів "includes" https://developer.mozilla.org/uk/docs/Web/JavaScript/Reference/Global_Objects/Array/includes


між одператорами і змінними завжди має бути по одному пробілу


Здесь тоже каждую переменную записывайте через запятую с новой строчки с отступом


Should be "$bundle"?


... looks like we're missing FStar_Buffer.ml also, which is harder for me to reconstruct than FStar.Mul.ml


Перепишите массив с подмассивами


I really don't think this is the way to solve exceptions like this.

I'd like to keep the codebase clean of hacks like this, even if it means a minor inconvenience for a user for a (short) period of time


Записать каждую переменную с новой строчки и еще после объявления всех переменных нужно добавлять пустую строку


Well what do you know, it looks like I already did add this to the up-upstream last winter as `setCustomIdPEndityId($customEntityId)`


CatalogsCollection = make(map[string]*Catalog) should be done out of the loop
If we are initializing on every iteration, wont CatalogsCollection just contain the data for the last catalogURL?


I love the polishing you have done, yes deck select and other menus look like garbage now, no need to sugar coat. Whole process from load to actually in a match needs a good polishing or a complete recode.


adding additional info....  Test 3


Adding additional info.


Yes, good idea!

---------------------------------------------------
Dr. Gijsbert Werner
Newton Fellow
Department of Zoology
University of Oxford
South Parks Road<x-apple-data-detectors://1/1>
Oxford<x-apple-data-detectors://1/1>
OX1 3PS<x-apple-data-detectors://1/1>
United Kingdom<x-apple-data-detectors://1/1>

On 11 Jul 2016, at 19:48, Gustavo Paterno <notifications@github.com<mailto:notifications@github.com>> wrote:

@gijsbertwerner<https://github.com/gijsbertwerner> I think we can you use your N2 fixing paper as reference also because it is a published example of the samp approach (removing random proportion of data in phylogenetic comparative methods).

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://github.com/paternogbc/sensiPhy/commit/452baa003edf5cbe03d88b940fdf8a847ce7f8d9#commitcomment-18200227>, or mute the thread<https://github.com/notifications/unsubscribe/AGauyGzHkKHVjFQsD5VHwPWVMTCsMtNhks5qUpAAgaJpZM4JJqdv>.


Does this release support webrtc M51<= releases? I seem to be stuck inside ice connection state 'checking' since M51. I know Promise based RTPeerConnection stuff was released in M51 but I'm not sure if its backwards compatible-- guessing not.


You are OK!


ye haw


Закрывающая фигурная скобка съехала на один лишний отступ вправл


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


地方


持ってないしビルド予定も無いし更新も頻繁に行われる端末のツリーにUBERTC対応入れて更新追うのめんどいんだよ許して


程序


Glad you dind't advise me...


Seems like newCatalog.catalogBranchDir is having same value as newCatalog.catalogRoot?


I didn't realize the github.com repo was out of date.  I've been trying to keep all the elevator-related stuff on public GH, since the app itself is public.  But, I'd rather stay on the up-to-date repo, I'll look at switching over.


This is interesting. I started a similar script a couple of days ago! Guess it won't be needed if you're working on this also :)


How does this even compile? Isn't `Entry` supposed to be `ceylon.language::Entry`? Weird.


Can you add a failing test showing that `String` parameters are not identified as an upload?


Not for me


It's inherited from `AbstractSet`.


Здесь запишите объявление и инициализацию переменных через запятую


Apart field should not be part of serialized definition, so this had to work :-)


Since we are having "name" property, why do we need the url in format "library=https://github.com/rancher/rancher-catalog"? What is the use of "name":"rancher-rancher-catalog"?

If we want to load the catalog input as "library=https://github.com/rancher/rancher-catalog" then url property should be just "https://github.com/rancher/rancher-catalog" and the name property should be "library"


That's for the future :laughing:


👍 Fantastic, that'd do it


Is "$this->" required here? (#PHPnoob)


I said stack.copy with that line...


What's inherited? I mean, I didn't realize the import:
```ceylon
    JMap=Map {
        JEntry=Entry
    }
```
exposed *both* `JEntry` and `Entry`. But apparently it does.


Is  this correct?


@julienperret :+1:


It's not, I was just confused by the fact the file was empty.  Will submit a revert PR.


no comment


Плохой свитч.
Куча ретурнов почти в каждой строчке. Тяжело разобраться в коде.
Лучше создайте переменную выше свитча и в нее сохраняйте результат работы функции. И уже внизу функции напиши ретурн этой переменной.


There are some newer features, custom attr prefix, custom entityId.  There's also support for parsing course enrollment and instructor assignment from shib attrs, but I think that belongs in a different package and I'll probably split it gracefully (it probably doesn't belong on public GH). I didn't know the cla-rce github.com repo existed until yesterday, or I'd have pushed to get it in sync long ago.


elbasztad


This url gives 404, please fix the url


Get them from c->model->basicabilities...model is data we store of unchanged cards that we reference. We never modify model, so you have a base to compare from.


separate Requires: from BuildRequires: move to its own block


I know. I forgot. Sorry.


ditto


TaskCluster: @cgsheeh does not have permission to trigger tasks.


This is a good idea. We should add `Ord` to all the entity reference types in this module. Feel free to commit that directly to the main repo.

At some point, the specific numbers used for entity references are going to affect the details of code generation. For example, reordering the predecessor list on an EBB may cause some algorithm to make a different arbitrary choice. This could then lead to problems reproducing bugs by loading `.cton` files if the loaded IR has different numbers.

When that becomes a problem, I think we should fix it in the parser. We could add an option to the parser that makes it preserve entity numbers (or die trying). No need to worry about that just yet.


```
Finished at Mo 11. Jul 06:59:41 CEST 2016
3:46:39 elapsed time, 15:47:59 cpu time, factor 4.18
```


pozdrav,

imas opet jako puno syntax gresaka.. toplo ti preporucujem da koristis hackintosh, jer ces uvidjeti gdje grijesis kad radis ovako

LP

PS. Ja sam ti opet napravio zadaca 3 playground koji mozes povici sa gita pod zadaca 3


I think we are iterating over catalogURL, and pull the branch from  CatalogObject[counter] - and hoping that the value from catalogURL will match CatalogObject[counter]. Url
This may not be the case always, so we need to use a map to pull the exact CatalogObject for the catalogURL


why delete tests? I suggest let them be part of the same package. Alternately you can create a -tests sub package.


please don't push to master without running through ci


Also anything not on model is added from an ability or effect, which handles its own removel.


@RedBlueDragon please create pull request first on separate branch and give people few hours to review changes, this will helps with conflicts and errors in master branch


TeamCity Bender :: Deploy [Build 3.0.50.0](http://build.mikeobrien.net/viewLog.html?buildId=2634&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3387, ignored: 5 Build time: 00:00:32


Well, then… why is this just hostnames? If I have a domain record that includes a scheme and port, this will be lost for the redirect. I checked, and the redirection code seems to treat that as an opaque string for storage purposes, but it would need work to make the lookup aware of that, I think. Should we create an issue for that, @dfeyer?


@RedBlueDragon just remove it if you don't need it


Also, AFAIK our convention is to use [kebab-case](http://stackoverflow.com/a/17820138/3018033) for DOM classnames and IDs.


Just include Group/License at the top, dont need it for sub packages.


Its c->model or c->data->model, i forget exactly


@EmilyK here is my commit


Or maybe we should just build a static dictionary cache.


In the future, try to avoid `<br>` tags whenever possible. In general, creating spacing with CSS is a much easier approach.


My only recommendation is to move everything to separate files and use main.scss as the index. Basically use the main.scss to import all other files, but other than that great work.


Forgot to resize them to 256x256 will do later.


#5768-Example-files-which-show-discontinutity-in-air-or-water-flow-rate-through-fans-and-pumps (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1112 of 1112 tests passed, 0 test warnings)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-34.63%25-green.svg)


nope no duplicates, Thanks Gig but this is vanilla Stratis i'm running at the mo so it should work ?


looks looks like a little bit too long line - please reformat or split it


The `%` really shouldn't be reverted I think. There are too many outputs giving percentages and they will trigger false positives with operator fontification.


No need to indent `<h5>` tags here, since they're not a child of the `<h4>` tag.


ok


molim te napravi integraciju na slack


summary should not end with a dot


clearly, no one reviews changes. thanks.


Good work Elizabeth!

I rewrote your code below with some more specific HTML5 tags that should help you write clean, maintainable code. This kind of formatting is especially important when we start to write more CSS, as it'll allow you hone in on certain parts of your code quickly.

Please take a look at this and come tap me on the shoulder before class if you have any questions about the changes. You can also schedule office hours for tomorrow (tuesday) night and we can over them then!

Again, great work!
Zack


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


conservation-tests (rraustad) - Win64-Windows-7-VisualStudio-12: OK (2225 of 2228 tests passed, 10 test warnings)

 * 10 tests had: ERR diffs.
 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.

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


Looks great, my only recommendation is to cut all the content of main.scss from from `.browsehappy` to the end and paste it in `_content.scss` so main.scss is your index for all your styles, but again other than that great work.


Agreed.
I've been taking a closer look at this, with regards to how we are using thending data to initialte a show fetch for adding to the favourites.
It appears we used to use the slug_id, but replaced this with the imdb_id to get around issues with being trakt.tv messing up fetches of shoes that have numbers as titles.
However, it seems the imdb_id is also problematic in some cases, with 4 out of the 500 trending shows have a null imdb_id, and several more where the imdb_id is incorrect.
I suspect we need to add a sanity check.
I propose using the ibdb_id first, but then checking that the title we fetched is the same as the title in the trending that spawned the fetch.
If it is not the same, then try using the slug_id.


@shiffman Sorry I was testing there, Georgia is back.


OIC, you meant `AbstractMap`.


Would it be cleaner to use a `Vec<(Predecessor, Ebb)>` vector here?

This pattern of collecting data into a separate vector, then using it to update the data structure after bugs me a bit. It seems to be a fundamental limitation of Rust's borrow checker. It's OK for this algorithm, but we eventually need to figure out a way that lets us iterate over a function or an EBB *while modifying it*. I'm not sure how to do that yet.


We really need spellscripts. @xfurry


da, imas previse gresaka... preporucijem ti da ovo uradis ponovo na hackingtoshu, jer su mnoge grese syntax errors koje ce ti swift sam priajviti


Great use of mixins, and overall great work. My only recommendation is to cut all the content `main.scss` from `.browsehappy` to the end and paste it inside your `_content.scss` and your `main.scss` will work as your index file for all styles.


I almost forgot. Last night when I was testing stuff on Altis I had someone join the server and they said they couldn't see the parking markers but I could still see them. I thought it might be a apex preview bug.


Yes, sorry, that's exactly what I mean.


use %license tag for LICENSE


plant-autosizing-gaps (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1669 of 1671 tests passed, 0 test warnings)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.88%25-green.svg)


The only hypothesis I have is the invisible sign dummy thing I used like for sticky charges, so that the table doesn't move around due to explosions and other shenanigans. I'll probably try switching `_dummy hideObjectGlobal true` to `_dummy setObjectTextureGlobal [0,""]`


What was the idea behind this change?  The https://github.com/just-containers/base-alpine-without-s6/blob/master/Dockerfile still has `FROM alpine:[...]` in it.


@ifoche something went wrong while pushing this (i got it working but today I realized something was wrong).

I think this should be cherry picked https://github.com/EyeSeeTea/pictureapp/commit/89486e6a82bb9db5a171067ae343e9e6bb31b332


Great use of mixins and great work overall. My only recommendation is the following: cut from `browsehappy` in your `main.scss` and paste it in your `_content.scss` file.


hajd napisi ovo, ali da na kraju zadnjeg elementa ne bude zarez :)


wait, the invisible signs aren't used on Stratis, the sheds are network objects so the terminals attach directly to them... this is getting awkward


Using the `print` statement here breaks Python 3 compatibility. Could this be changed to use `print(...)`?


Do we need to clear all of the predecessor sets before this loop in case they contain outdated info?