labofsahil/github-event-dataset-2012 huggingface dataset:
Can this be a problem if input driver is init and deinit several times?
One should use tinyint(1) for booleans.
Are you changing the public API here? Please don't remove existing tests, we need to know that everything that existed before your patch will still work after it is applied.
why is this in here ? "waiting" for the other thread to finish !?
proper way would be to have an internal queue (of wanted episodes (for new shows)) and let the addShow thread call back at this
```
pos : array, shape = (n_pos, 3)
    Locations in meters.
```
wouldn't 

if not idle_only or not p['active']:

be easier?
That would break backwards compatibility for submitting lists to this function. I guess we can live with that?
Not that it really matters, but why don't we just use np.sqrt and np.ceil? I think these wrap anyway, and would save an import line.
Sure. You want me to do this and resubmit?

Carolyn

Sent using a mouse-sized keyboard with feigned autocorrect intelligence.

On Dec 25, 2012, at 10:26 AM, Mike Fiedler <notifications@github.com> wrote:

> In soft_skills.rst:
> 
> > +to that information.  Organizational leaders often do not understand
> > +how these solutions are implemented so it is up to the Operations
> > +professional to communicate the threat, solution, cost, impact to
> > +the organization of implementing the solution.
> > +
> > +Advising within Current Limitations
> > +-----------------------------------
> > +
> > +The Operations professional who advises an organization must also
> > +consider limitations that impact the potential solution. Cost,
> > +timing, expertise within the organization, available time of the
> > +people who would implement the solution,or IT security issues may
> > +be considerations.  For example, decision makers within the
> > +organization will need to know what is possible and for what cost
> > +so they can make the decision how to spend the organization's money.
> > +Good, fast, or cheap (pick two): it may be the Operations professional's
> Is it worth making a citation to the http://en.wikipedia.org/wiki/Project_management_triangle ? I think this is a core concept that we might need to explain further somewhere.
> 
> —
> Reply to this email directly or view it on GitHub.
>
> So if a user has some actions that need CSRF protection, and some that don't, they need to put a big list of paths in Global
Or he can use regex, String.startWith, etc...

>  the browser will always submit a content type
true. CSRF should be disable on anything but the 3 mentioned content types.


> No it's not, looking at the body is the default behaviour.

```scala 
if(POST_LOOKUP)
    body.flatMap(_.get(TOKEN_NAME)).orElse(request.queryString.get(TOKEN_NAME))
```

As you can see it's always looking in the query String
because i'm an idiot
It calls `valueForBinding("css")` so it is used in wod bindings. It is public API, so users may be using this binding in their own apps and so we cannot just delete it without first ensuring that it is not going to break apps.
Those lines just moved down to 545 and 546.
@caike https://github.com/rails/rails-observers/commit/ad733ef5c73acd6fc6eea70a389c84248590c714 should fix it. You don't need to add jouney there.
Capitalize `Contents`. We typically capitalize nouns in titles.
@fazae, I don't see any changes with the shorthand `::` for php code notation, which I mentioned twice?
There should be a comma (`,`) after this line
or --dev as well, zipballs don't contain the tests only if they were filtered with .gitattributes maybe provide a link to composer docu on this
ok i have made the typo changes and commited my changes.

is it ok ?
It's possible that there are other params set from the form, if user overrode view or something. 

So the approach is to explicitly set the stuff we know about, and do the best we can to preserve prior behavior for anything else in the params. 



Sent from my iPhone

On Apr 4, 2012, at 22:36, Dan Croak<reply@reply.github.com> wrote:

>> @@ -31,4 +31,13 @@ def flash_failure_after_create
>>   def url_after_create
>>     '/'
>>   end
>> +
>> +  def user_from_params
>> +    user_params = params[:user] || Hash.new
>> +    email, password = user_params.delete(:email), user_params.delete(:password)
>> +    Clearance.configuration.user_model.new(user_params).tap do |user|
> 
> The goal is to explicitly set the attributes and not use mass-assignment? It seems like passing `user_params` to `.new` is unnecessary.
> 
> How about something like this?
> 
>    def user_from_params
>      Clearance.configuration.user_model.new.tap do |user|
>        user.email = params[:user].try(:delete, :email)
>        user.password = params[:user].try(:delete, :password)
>      end
>    end
> 
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/thoughtbot/clearance/pull/188/files#r648700
Some other ideas:
* remove this object from the model and let the user of the REST API send a Wiki object (we need to add missing information in the Wiki object which should be done anyway) and a URL parameter for the template which is pretty much what we have in the wikimanager API
* pretty much the same thing but really introduce the template information in the wiki descriptor and make wiki manager indicate it when creating a wiki from a template (which is something I remember seeing someone asking for)
Also, how is this class used? Is it threadsafe?
Shouldn't this be CamelMessage(%s, %s) otherwise I don't think anyone will be able to distinguish it from any other Message.
This will use either the context class loader or the loader of OldConfigurationLoader; the magic in findClassLoader doesn’t know about OldConfigurationLoader or GlobalActorSystem). So, what I’m saying is that the magic determination of who called this thing is not available here. This might be good enough.
The config being passed in here is the specific dispatcher section, while your Settings only look at the defaults. You should extract the Settings from

    config.withFallback(defaults)

for a suitable definition of "defaults".
That's pretty good start. I noticed that something like (n + m)*(n - m) doesn't get factored back.
I think 0 would be more convenient. Then the user can always know that he can just multiply the result of `coeff` by `x**e` and get the term back.  There's no information lost by returning 0 instead of None because there's no way that 0 could really be there in the first place (except maybe evaluate=False, but I don't consider that case to be too important).
```python
>>> a = (exp(x + y) + cos(x - y)).series(x, 0, 6)
>>> for i in range(6)[::-1]:
...  print i, a.coeff(x, i)
...
5 exp(y)/120 + sin(y)/120
4 exp(y)/24 + cos(y)/24
3 exp(y)/6 - sin(y)/6
2 exp(y)/2 - cos(y)/2
1 exp(y) + sin(y)
0 exp(y) + cos(y)

>>> a = expand((1 + x)**6 - 1)
>>> for i in range(6)[::-1]:
...  print i, a.coeff(x, i)
...
5 6
4 15
3 20
2 15
1 6
0 None
```

What do you think about that None -- should coeff just return 0 in all
cases where None is currently being returned?
А если в `accountID` будет уникод? `QString::fromUtf8 (accountID)` полущ.
the translation string are not tied to a bundle. The same translation domain can be used in several bundles. And there is no default here as there is no built-in translations in KnpMenuBundle. The default domain depends of what you do in your own theme
Note - I used the emacs command `M-x align-regexp` to get the aligned spacing between two columns of text like that.
Readability nit: Please add spaces around operators (=, +, *, etc) throughout...

     cu = Sketchup.active_model.options["UnitsOptions"]["LengthUnit"]
It might be more safe to use
´´´
return mapSources.containsKey(sourceId)
´´´
or something like that. That way is stays consistent also for future changes or refactorings.
We really should just lombok these up, thoughts?
this is a _get_ method, while the javadoc comments on the class explained we'd need to synchronize on write operations. I think this synchronization is correct for visibility, care to clarify the javadoc? BTW, not sure why the other read methods won't need a synchronization then..
I've been living in AS7 land where that's how they are versioning parsers. I do prefer the cleaner (no underscores) approach
`self` may be `nil` if `[super init]` fails. We usually use a pattern such as the following:

```objc
self = [super init];
if (self == nil) return nil;

// do other initialization

return self;
```
I'm working on a PR to remove deprecated stuff from this module; There's also something that was deprecated in 2010. I'm guessing that could also be removed (it was in 0.99).
@WeatherGod Well, iterators don't necessarily have length, so some assumptions about `collection._paths` must be made, and it seems to be a list in all cases that occur so far. I think that without going beyond the scope of this pull request, there's no further improvement to the check. What I find a bit ugly is that both `_paths` and `_offsets` are internal to collection, and should preferably not be used from outside of collections.py .

Is there anything else to be done with this pull request?
Hmmm, I wonder if this method is used...
getIntPref will throw an exception if the preference cannot be found. Given that all the code below will not be executed and we bail out. As you said, we do NOT start the server in such a case.
why not just ``__init__(start, uid, timeout, last_call)``?
just trailing whitespace
It can't be moved but because of the `for` in label but I added a new `data-id="${m.id}_chk"`
OK, that sucks... and we can't even make it configurable through behat.yml since that won't be available to dev/tests/startsession
(strictly speaking it is available as a file, but I think we shouldn't build any behat assumptions into startsession).

Why do we need to call either dev/tests/startsession or dev/tests/selectsession via CLI? Can't we trigger them both through Mink?
UploadField shouldn't make any assumptions about the UI controls used in the edit form (like chosen.js) - I've refactored this to a more generic onchange listener
But the problem isn't that exact invocation of `experiment.save`, it's that any invocation of save beyond the first one results in incorrect behavior, right? IMO the way to deal with that is not to say "never call #save more than once", but rather to change the code so that you can do multiple invocations of save.
__wt_evict_server_wake() can fail.
> Agreed, but the only alternative I thought of was to look for information in the table cursor when we open the indexes -- is there a better alternative?

Yes, don't try to configure files through cursors ;-)

Michael.
I disagree.  The file pointer should be initialized to zero.  I don't know what else would be a sane initial value.
Cool, I like traits :)
Is it possible to add a @param annotation for parameter ``$sql`` which defines a more specific range of types; something like "string|array", or "null|MyObject"? This would help me, and IDEs to better understand your code.
Are you sure ``version_compare($install...Version, '5.3.8', '>=')`` has the correct type as ``*`` is not a sub-type of ``boolean``?
Done. Added also comment to review this after Jira task: TIMOB-9051 will be done.
Oh sorry about that.  I have no strong opinion on removing it or not.
Ah, the hover is ok - its only, that the heading itself is hidden under the page header (the menu bar). 

Here is a screenshot of a current www.dartlang.org article and the modified page
https://plus.google.com/photos/108931164613660994654/albums/5790749355271980401?authkey=CKWE_Oyk_83flAE
I hate to keep nagging, but these seem like 2 tests.  I prefer more descriptive test names.

* test_issues_with_blank_url
* test_issues_with_custom_url
Same here with the `count` as in `my-some`. On the other hand, from the result of `(filter (complement pred) a-seq)` you can check the result easily.
What's `popular_videos` now that we need a queryset attribute to make it a list? Won't this break templates which expect `popular_videos` to be something iterable?
Since the view are already separate classes, can we move the defaults into those classes as well?
find jars plugin containing resources(css,wigetset etc...)
This is ugly. You should use callbacks when everything finish. Not just some arbitrary timeout.
You will have to read Tastypie documentation to understand, especially the [request-response cycle](http://django-tastypie.readthedocs.org/en/latest/resources.html#flow-through-the-request-response-cycle). So there is a lot going on. The method `obj_create` is used somewhere in the middle to access the database and create object there. But after that a lot more happens, and it can fail. Some of that is for example serialization (or dehydration, how Tastypie calls this). And this is something which is quite error prone. And not to talk about bugs. Ours, or in Tastypie or elsewhere.

So, we should put pushing as far to the end of this cycle as possible. There are cases, when there is necessary to do that when working with database, so this signal is useful. (For example, when you want to store also something else in the database, like notifications about new post).

But in our case we want to in fact multicast response. So by doing a signal in `post_list` method (instead of `obj_create`) we could maybe do that. And simply push the already made response. So just before it is send to the author, we just multicast it around.

But now that I am thinking, author does not really get the response at the end of POST request, just URI of the new object? OK, but still, we could intercept it at the end, I believe.
It is not challenging after I know how it is done, I hate finishing and polishing :P.
Tuples, not lists. And please indent better.
Hm, empty line, why? You can put docstring here, though. :-)
This also not.
True, the tests are completely outdated.
I thought it would be possible to directly pass the audience to the default "verify" function, eg. `browserid.verify(BID_ASSERTION, ("*.mozilla.com",))`.
This could be simpler than having to instantiate the verifier with the wildcard.
This is a very good suggestion. I have just added a [commit](https://github.com/moodmosaic/AutoFixture/commit/524589c03a811dac33a862e40d75da868899d1db) for that.
i think `custom_texts` can dropped, too
You might consider using a LinkedList here, pushing items on the head of the list in registerHandler, and using a less complex for loop construction (for Thread.UncaughtExceptionHandler handler :: handlers) {}) in uncaughtException.
Wie bei #41: Da fehlt noch:

    android:fillViewport="true"
    android:overScrollMode="always"
    android:isScrollContainer="true"
    android:scrollbarAlwaysDrawVerticalTrack="false"
    android:scrollbarStyle="outsideInset"
    android:scrollbars="vertical"
Why is this a module and not a class. Since it has a ```@loggers``` it does carry state of it's own it might as well be a class.
Does this really need inheritance? I'm certainly inheriting across projects in Celluloid elsewhere (e.g. Celluloid::IO::Mailbox, Celluloid::ZMQ::Mailbox), and I think that's a bad thing I'd like to get rid of it if possible ;)
we should use a parallel verb tense here: support....and enable the project to build the ....
I'm not sure it really matters - my thought that if someone were skimming this, they'd need to see what they shouldn't do first. If you feel strongly about it though I will def. change.
good riddance!!!
The main thing that strikes me as odd here is mixing the resolve/reject observers in the array. That might be confusing to new people coming to the code. I'd probably add an object `{resolve: onResolve, reject: onReject}` to the array instead and destructure from the shift. It might be slightly faster as you say, but how often are we actually going to have enough observers for that to matter?

The shift thing seems fine, though a single for loop is probably faster, but again unless we're going to see a lot of observers it is probably irrelevant.
pushされたソース自体へのコメントテスト。
yかnで答えなかったら鬼畜ループへ
Fixed up in 5afd772 and 883e776
Quite a few whitespace issues in this file.
Thanks! Done: 02edebf81a5a
...I thought you guys had a dependency on pandas these days?
Actually the hook doesn't work. However what I was fixing was the fact that you could not submit the config page, even if you had welcome message turned off, it still required a sender -- which you could not even see.  Fixing the hook is next on my list.  I didn't expect all these other fixes to be attached to the same pull request -- everything else I do seems to get added.
This calculation is wasted FLOPS if we are setting this via fwhm or scale_radius.  Should move inside the switch block.
Each of these conditional blocks have the potential to call the same function twice. This could be written much better.
You can write

```
"--with-lablgldir=%{lib}%/lablgl" {"%{lablgl:installed}%" }
```
Kelvinator used to download by full URL, upload via boto.  Now both via boto (default_storage) so no need to pass along ugly hardcoded paths like this anymore.  Much cleaner.
There's an annotation in guava \@VisibleForTesting
No problem. I think it is good to check for npe's as quick as possible. 

So will fix.
changed here to be consistent with usage elsewhere, but changing across the board to refer to deployment instead of WAR seems a good idea
It won't be the default exception, it will be InvalidArgumentException
I'm having trouble getting the .gitignore to ignore it.  So I'll let you give it a shot.
There is some similar code. I will fixes them all.
Nice: thanks for adding the comment on where the $40 is coming from.
Ok, so let's delete it :)
Don't use `t '.information'`. Put within a namespace.
On Tue, 20 Nov 2012, Hannes Magnusson wrote:

> >  	} else {
> > -		php_error_docref(NULL TSRMLS_CC, E_WARNING, "The value %ld is not valid as read preference type", read_preference);
> > -		RETVAL_FALSE;
> > +		php_error_docref(NULL TSRMLS_CC, E_WARNING, "The value %s is not valid as read preference type", read_preference);
> 
> Maybe the %s should be in quotes like in the query parameter parsing?
> Just in case they pass in false or empty string?

Fixed

cheers,
Derick
I agree, that's a POST we want here, not a PUT, regarding the http spec.
Did you mean `.shen`?
I've just been writing `Copyright (C) 2012` for new files. Should I be including the full range of years?
This `if/else` can be removed :-)
That will also allow other sources (PATH and COOKIE). Sure you want to add those too?

Also `$request->get()` is slow, as per the notice here: https://github.com/symfony/symfony/blob/master/src/Symfony/Component/HttpFoundation/Request.php#L460. On the other hand it is only run on login success.
a couple things here

1. Are you checking to === or == to true? Is it a string? A boolean? You might want to tighten that up. 
2. `.continu` a typo? `.continue`?
sip, lo cambio a utf8. Es que no tenía forma de meter utf8..., ya sabes los input de las terminales como se las gasta a veces :)
split onto multiple lines.
I'm curious what your intended use is with this...
Is this needed anywhere?
Чтобы не плодить copy-paste, можно использовать String.CASE_INSENSITIVE_ORDER (без учёта регистра) и написать Comparer<String> для учёта регистра. И использовать его.
Все остальные объекты, а не только Object.
Maybe instead of commenting these video sources out, we could move up the youtube one to the top of the array.
Let's follow the strategy above and comment the fluid.log statements out. We're likely to lose this code altogether fairly soon when the features are moved into the Infusion framework.
Agreed that we're probably going to want to cache more, but lets add them in as we need so we get it right?
Does that work? Afaik, `gViewSourceUtils` is part of the Firefox window. We are not allowed to go there.
Very nice, and very pythonic. :-)
PEP8 issue: base.py:57:9: E303 too many blank lines (2)
Please check with PEP8, I believe there should be two blank lines.
This probably only affects people with older Cython versions...
Hm not sure I actually get the example :-/ maybe need to think a bit more about it.
Maybe add a 1 liner description that explains the gist of what the SMACOF algorithm computes.
I feel this should also use label binarizer. I don't understand the code well enough to know where that should live, though.
If you do `y_new = np.zeros(len(y), dtype=int)` instead of `zeros_like`, then you can skip both the `asarray` and the `astype` below.
Note that 
``
a = []
a.pop(-1)
``
give 

Traceback (most recent call last):
  File "&lt;string&gt;", line 2, in &lt;module&gt;
IndexError: pop from empty list
what is the benefit of not doing this in ``fit`` but having a separate private function?

Btw, as this function is "private", the docstring will never be visible to the user and will not show on the website.
>
>
> Well, it makes sense in clustering (where I came across it) and it would
> also make sense to get the in-sample error...


Indeed, fit_predict could be useful to assign the nearest centroid in
KMeans.
even this should go into fit.
well this is what we have been doing in other bundles so far.
there is a popular "standard role" called ``ROLE_ADMIN``
What about this comment: https://github.com/D-Programming-Language/dmd/pull/1421#issuecomment-11759461
I agree with Dave's comment, only want to specify that there shouldn't be any undocumented magic values like 229.0 256.0 and 25.4.

If you use these, put them in const vars with a descriptive name.
Not at the moment. If we do multi-threading, we should use message queues to synchronize. V8 requres everything to be on the same thread unless we use isolates and lockers.
I used an autogenerated file here because it seems convenient. I had to remove the Extensions/ line because of the changes I made to the template project.
I assume this doesn't add a new resource if the user scope resource already exists?
Do you turn off if compressed?
couldn't it be as low as 1?
Please use the following rather than `strtoll()`:

```c
parse_value (token, &beancounter.values[FAILCNT], DS_TYPE_DERIVE);
```
It's probably worth noting that I didn't actually change the formatting here.
``self.click_url, created = ClickUrl.objects.get_or_create(url=url)[0]`` is slightly more explicit, but this is okay
I'm suspicious of all these `!important` bits. Why do they need to be like that? You normally never use `!important` except in user styles or if you have some kind of old and annoying external dependency you need to monkey-patch...
FreeBSD packages often include version numbers - in this case I don't think there is any kind of meta package that will get "the right one" for you. That said, you're only defining defaults here. You already have a workable solution in place for when the defaults aren't what the user wants: [a package name parameter](https://github.com/puppetlabs/puppetlabs-mysql/blob/master/manifests/server.pp#L18).
Missing a ">".  Also, please convert "@" to [at]
I think returning the segmented image should be the default behaviour, since it is usually what people wish to get. The full probabilities are useful for a more advanced use of the algorithm.
Can you make the CellList an `<ul>` with the list items as `<li>` ? To be semantically correct, and have the CSS work without modification.
Yes, 'ShadyCartesian' was 6 to 10% faster.  Once I removed the isNaN it was only 1.0 to 2.5% faster.  I'm going to replace all isNaN calls with typeof 'number' and we should be good to go.
From this, it looks like a primitive's bounding volume needs to be defined in world coordinates.  I believe we should define it in model coordinates instead, and the engine should transform it to world coordinate - or whatever - so it is in the same coordinate system for frustum culling.  It will be slower for some cases, but it is consistent with the primitive itself, e.g., it positions, etc.
Depending on how node handles situations where it cannot run the specified command, we may need some error checking here.
I am ok with that, but as that value seems quite arbitrary it needs to be documented i think.
Strange...aside from the fact that it isn't deprecated (it's the way forward for when Android and iOS finally implement DOM3), that just doesn't make sense given MW's lack of real existence prior to 2.0.
.. active state _and fires_ the `resumed` event; or moves to the background state _and fires the `paused` event.
This is the Android documentation of SOFT_INPUT_ADJUST_PAN: "set to have a window pan when an input method is shown, so it doesn't need to deal with resizing but just panned by the framework to ensure the current input focus is visible." When a textfield is focused, it will try to scroll to make that visible.
I just left a comment on it as I was expecting there to be a node API for checking if the file was open or not.  I figured since I made that assumption someone else might do the same and I wanted to provide a quick explanation of why I did what I did.
Indentation is still screwed up
I suggest you use bogus names instead of something that might be on your machine :)

Note, I ran the tests and they do work on machines that don't have them - so just a suggestion for clarity.
"exists should be singular"

Specify what the non-existent key gets set to: "set to zero and then incremented by the passed value".
я по точке выравнивал
I thought this info might be useful for others to use in a plugin. That's why I put it into the $info array
This is related to http://d.puremagic.com/issues/show_bug.cgi?id=3449
The static is redundant right now as dmd always treats immutable member variables as manifest constants, but this is a bug and may change at any time. And then  ```static``` will be necessary.
Misunderstanding. I can't continue discussion. Sorry.
quite a test for inliner!
We are using the notation Q^* here, which generally refers to the conjugate transpose of a matrix and applies only in the complex case. In the real case this should just be Q^T
We should have a comment for this dependency
Remove PSU_CDN define.
Please follow the coding style—include a space following the colon in this line.
So what is a proper way to mark that I've written it as an employee of IMAGIN IT?
Not sure if adding my name is the convention or not. Feel free to remove if it isn't.
Strange way to initialize it
Yeah, I'll remove the extra -Werror.
`s/exposed /exposed the /`
this doesn't work as intended, since
 * the velocity-data isn't accurate enough to deduce a position
 * movements as indicated by navData are relative to the drone-coordinate-space, not the main coordinate-space of the scene. The vector of the position-deltas needs to be multiplied with the current transformation-matrix of the drone in order to get the correct directions
Я в таких случаях пишу слева auto, чтобы не дублировать один и тот же тип дважды в одной строке.
yes it makes
Fixed throughout the widget.
Also, my same comments from the htmlblock widget apply here for the most part, so I'm going to leave those out of here.
Wat. Seriously?
QUERY: We have a mix of definition guards and pragma once to protect headers from being included multiple times, should we pick one type of guard and stick with it?
опечатка в "информацю"
このコメントは削除してください。
メソッドコメントを、83行目からのを参考に書いてください。
this is a performance optimization: how many hierarchy levels of child nodes to prefetch and cache
Диаграмма обновлена не была, версию править не нужно.
«Ничего необычного» лучше заменить на что-то более неитральное. Для кого-то данный код может быть очень необычным…
docblock and type hint are missing
Rather than making two calls, we can simply combine both events.
Imo that's not needed, because we use FOSRestBundle the structure of the urls is fixed. That's why we can assume `base_url/_threadid_/comments`. If you don't import our defined routing but do your own you're creating non-rest routes and the reference JS will be broken. I don't think this is an issue.
Ate the donuts.
finally set up the user's corpora/corpuses div so that we can see our corpora and switch between them.
What two files are you talking about?
Also, general note - *always* use === and !==, not == and !=, == and != have nasty, unpredictable gotchas.
@jeffheifetz you didn't add back the "file:///" in your fix....
> <para>You can decide to do a few operations:</para>

<para>For each occurring event you can respond with one of the following actions: </para>
This is crap because MultimapHandling is weak. This will be hardened up in the following pull request.
I should have added here:
} else if (major > o.major) {
  return 1;
}
I don't know if it's worth alternatively commenting these out, in case you want to quickly throw them back in for debug sessions.
TVL, ja hledam to nastaveni pro NETBEANS a nemuzu jej najit. NEmam cas jeste vsechno kontrolovat.
I think the code is bizarre unless you remember that arguments is not a real array but an array-like object
execAsync if we can
jak potrzebuje get i set to uzywam auto a jak tylko odczyt to backing field

W dniu 11 marca 2012 14:25 użytkownik Przemysław Lewandowski <
reply@reply.github.com
> napisał:

> > @@ -0,0 +1,26 @@
> > +﻿using System;
> > +using System.Collections.Generic;
> > +using System.Linq;
> > +using System.Text;
> > +
> > +namespace MerkantyliuszBarterowy.Model
> > +{
> > +    class Balance
>
> Przydało by się zdecydować czy używamy autoproperty czy property with
> backing field.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/PrzemyslawLewandowski/Merkantyliusz-barterowy/pull/19/files#r543379
>
Brakuje odstępu między memberami.
Rename environmentCommunicationMessages::Map map please :)