The Ugly next Lobby 2.6.0

Post here if you want to help developing something for FAF.

The Ugly next Lobby 2.6.0

Postby Xinnony » 03 Jan 2015, 05:48

Hi everyone,
I want to explain what is going a little behind the scenes.

For several years I develop and fixes bugs in the FA Lobby.
Often people on FAF wonder of the new features I note and I publish.

A new developer is arriving there shortly, and start editing, deleting and renaming all my scripts causing some problems.
Which made me a little furax because now I also do not recognize my own scripts.

I said that without me the Lobby would remain in this state : (without the 16 slots)
http://media.moddb.com/images/downloads ... enshot.PNG

I start the first state.

PNG conversion in the DDS Lobby and new fonts and colors.

Before IceDreamer or other pic its crisis, I said that I would attack anyone, I just said what bothers me, so no need to insult me again.

I find that the changes he has done makes ugly Lobby is my opinion, but I turn to the community as it is for them that we develop in this game, not for us personally.
I find that the quality is degraded, and new colors, fonts are not very pretty.

Before :
Image
(Original : http://i.imgur.com/t2Fx1Ji.png)

After :
Image
(Original : http://i.imgur.com/vjph38C.png)
User avatar
Xinnony
Contributor
 
Posts: 551
Joined: 19 Feb 2012, 02:49
Has liked: 70 times
Been liked: 34 times
FAF User Name: Xinnony

Re: The Ugly next Lobby 2.6.0

Postby arkitect » 03 Jan 2015, 07:38

I did notice that the lobby did not look quite as nice recently, but I don't mind since it's only for getting a game started and the functionality is the most important thing for me.

It is important to keep the code maintainable though, and if it's making it hard for you perhaps you need to alter the change/collaboration process? who is the "product owner" for the lobby?
arkitect
Avatar-of-War
 
Posts: 134
Joined: 08 May 2014, 13:40
Has liked: 23 times
Been liked: 81 times
FAF User Name: arkitect

Re: The Ugly next Lobby 2.6.0

Postby Rogueleader89 » 03 Jan 2015, 07:56

Of course the obvious answer here is that the png image is noticeably better, I don't know what exactly is going on behind the scenes on the lobby lately, but it sounds like a communication issue; I'm guessing this is in part from a lack of having a designated lead developer to approve various things in development or requiring a consensus of some sort before changes are done. Regardless, this seems like something that needs to be sorted out largely behind the scenes, probably with the aid of people more in the know than myself or the general faf community.

All this said, I think we all appreciate your work Xinnony and I hope that you manage to get some of these issues sorted out with the new developer.
Rogueleader89
Avatar-of-War
 
Posts: 130
Joined: 15 Jan 2014, 08:22
Has liked: 5 times
Been liked: 26 times
FAF User Name: JadeXyan

Re: The Ugly next Lobby 2.6.0

Postby Xinnony » 03 Jan 2015, 08:19

Unfortunately not, he spends his time has brag of her graduate and delete what does not like or what he does not understand.
My script is destroyed day after day with more and more limited functions or remove, I shouted several recovery and am angered me, but every time I got spit on.

This is Sheoo which approves new, and unfortunately it lets modifications that should not.

There are others damages ...

Eg, the "Map Background" option on the Lobby, the new developer, Sheoo and IceDreamer really wanted to remove, if Visionnik would not have expressed disagreement, he'd be available any more.
Just because they do not like her.
If I put a lot of options, it is to ensure that everyone can be customized according to their preference Lobby.
I do not think that has me personally, not like some ...

Thank Visionnik for avoiding that one delete options without reasons.

@Rogueleader89, The Source code is Open Source, but is Sheoo approving the changes.
User avatar
Xinnony
Contributor
 
Posts: 551
Joined: 19 Feb 2012, 02:49
Has liked: 70 times
Been liked: 34 times
FAF User Name: Xinnony

Re: The Ugly next Lobby 2.6.0

Postby Sheeo » 03 Jan 2015, 13:59

Xinnony wrote:Hi everyone,
I want to explain what is going a little behind the scenes.

For several years I develop and fixes bugs in the FA Lobby.
Often people on FAF wonder of the new features I note and I publish.


Hey Xinnony,

This isn't behind the scenes. Anyone interested in what we're doing can do so publicly on github and contribute themselves. I don't know what you mean by "Often people on FAF wonder of the new features I note and I publish." -- we've got public changelogs updated very frequently, these are replicated here on the forums as well.

Xinnony wrote:A new developer is arriving there shortly, and start editing, deleting and renaming all my scripts causing some problems.
Which made me a little furax because now I also do not recognize my own scripts.

I said that without me the Lobby would remain in this state : (without the 16 slots)
http://media.moddb.com/images/downloads ... enshot.PNG


FAF is truly open source now. This means that we need the code in a state that is as readable as possible, and technical improvements are welcomed as they improve the long-term maintainability of the project immensely, allowing people who haven't worked on it for years to jump in and contribute.

Xinnony wrote:I start the first state.

PNG conversion in the DDS Lobby and new fonts and colors.

Before IceDreamer or other pic its crisis, I said that I would attack anyone, I just said what bothers me, so no need to insult me again.

I find that the changes he has done makes ugly Lobby is my opinion, but I turn to the community as it is for them that we develop in this game, not for us personally.
I find that the quality is degraded, and new colors, fonts are not very pretty.


I haven't noticed the visual changes myself, and I don't believe these are an intended consequence of the work that has done, so this is a regression. We'll get this reported this on github so we can track it.

solarflare_nz wrote:I did notice that the lobby did not look quite as nice recently, but I don't mind since it's only for getting a game started and the functionality is the most important thing for me.

It is important to keep the code maintainable though, and if it's making it hard for you perhaps you need to alter the change/collaboration process? who is the "product owner" for the lobby?


These visual regressions aren't released yet -- they would've gone in with patch 3639 if we Xinnony hadn't reported them.

Xinnony wrote:Unfortunately not, he spends his time has brag of her graduate and delete what does not like or what he does not understand.
My script is destroyed day after day with more and more limited functions or remove, I shouted several recovery and am angered me, but every time I got spit on.


This is important: lobby.lua does not belong to you, me or anyone (Maybe someone who bought GPG can claim it). I, and everyone else, very much apreciate your contributions but they are just that. Contributions to something we can all collaborate on and improve together.

This requires that no single person takes ownership of any parts of the code.

Xinnony wrote:This is Sheoo which approves new, and unfortunately it lets modifications that should not.


The changes you mention here were all discussed publicly on github, this is not a behind the scenes approval process. I did not approve visual changes and I assumed the previous look was preserved.

If this was mentioned in the discussions before merging, I would've waited until it had been resolved before merging.

Xinnony wrote:There are others damages ...

Eg, the "Map Background" option on the Lobby, the new developer, Sheoo and IceDreamer really wanted to remove, if Visionnik would not have expressed disagreement, he'd be available any more.


I guess you're talking about this public discussion? https://github.com/FAForever/fa/issues/227.

I'll quote myself:

Sheeo wrote:If it's in use, then let's keep it though :)



Xinnony wrote:Just because they do not like her.
If I put a lot of options, it is to ensure that everyone can be customized according to their preference Lobby.
I do not think that has me personally, not like some ...

Thank Visionnik for avoiding that one delete options without reasons.

@Rogueleader89, The Source code is Open Source, but is Sheoo approving the changes.


Being merged with develop branch does not mean it's released or finally approved. We go through a review process before develop makes it to master (Which so far is always the latest available version) -- details of how this is done are still to be settled by the council. Perhaps this part of the process is where you want more openness?
Support FAF on patreon: https://www.patreon.com/faf?ty=h

Peek at our continued development on github: https://github.com/FAForever
Sheeo
Councillor - Administrative
 
Posts: 1038
Joined: 17 Dec 2013, 18:57
Has liked: 109 times
Been liked: 233 times
FAF User Name: Sheeo

Re: The Ugly next Lobby 2.6.0

Postby Blackheart » 03 Jan 2015, 14:11

If you do a "public" discussion on the top of Mt. Everest and say, hey, if you wanna join & listen sure just go ahead that is not very helpful or public because its still on Mt Everest.

Except the contributing people, maybe 5 people on FAF checked the github stuff and know seriously about it. People need links to click, stuff to see, not work into github interface and search for stuff 24/7. At least forums would be known by alot more ppl, so making a thread and at least giving the more important changes out would be good. And that is what is meant by openness, not the hypothetic availability somewhere nobody goes to.
User avatar
Blackheart
Priest
 
Posts: 376
Joined: 04 Jul 2012, 13:26
Has liked: 294 times
Been liked: 299 times
FAF User Name: various

Re: The Ugly next Lobby 2.6.0

Postby Ze_PilOt » 03 Jan 2015, 14:19

Sheeo wrote:FAF is truly open source now.


I fail to see how it wasn't "before".
Nossa wrote:I've never played GPG or even heard of FA until FAF started blowing up.
User avatar
Ze_PilOt
Supreme Commander
 
Posts: 8985
Joined: 24 Aug 2011, 18:41
Location: fafland
Has liked: 18 times
Been liked: 376 times
FAF User Name: Ze_PilOt

Re: The Ugly next Lobby 2.6.0

Postby Sheeo » 03 Jan 2015, 14:24

Blackheart wrote:If you do a "public" discussion on the top of Mt. Everest and say, hey, if you wanna join & listen sure just go ahead that is not very helpful or public because its still on Mt Everest.

Except the contributing people, maybe 5 people on FAF checked the github stuff and know seriously about it. People need links to click, stuff to see, not work into github interface and search for stuff 24/7. At least forums would be known by alot more ppl, so making a thread and at least giving the more important changes out would be good. And that is what is meant by openness, not the hypothetic availability somewhere nobody goes to.


We have indeed talked about how we get more people to become mountain climbers -- or potentially make mt. everest less tall.

Actually there have so far been 163 unique visitors to the FA repository, and 37 people have forked the repository.


I'll post links to relevant discussion that Xinnony is talking about:

Cleanup pull requests and associated discussions:
https://github.com/FAForever/fa/pull/202
https://github.com/FAForever/fa/pull/197
https://github.com/FAForever/fa/pull/209
https://github.com/FAForever/fa/pull/211
https://github.com/FAForever/fa/pull/213
https://github.com/FAForever/fa/pull/223
https://github.com/FAForever/fa/pull/238

A search for discussions involving Xinnony:

https://github.com/FAForever/fa/pulls?q=is%3Apr+xinnony
Support FAF on patreon: https://www.patreon.com/faf?ty=h

Peek at our continued development on github: https://github.com/FAForever
Sheeo
Councillor - Administrative
 
Posts: 1038
Joined: 17 Dec 2013, 18:57
Has liked: 109 times
Been liked: 233 times
FAF User Name: Sheeo

Re: The Ugly next Lobby 2.6.0

Postby Xinnony » 03 Jan 2015, 14:38

lol why you remove your comment in : https://github.com/FAForever/fa/issues/227 ?
User avatar
Xinnony
Contributor
 
Posts: 551
Joined: 19 Feb 2012, 02:49
Has liked: 70 times
Been liked: 34 times
FAF User Name: Xinnony

Re: The Ugly next Lobby 2.6.0

Postby Sheeo » 03 Jan 2015, 14:46

Xinnony wrote:lol why you remove your comment in : https://github.com/FAForever/fa/issues/227 ?


Who is this adressed to? That discussion hasn't been moderated in any way, to my knowledge.
Support FAF on patreon: https://www.patreon.com/faf?ty=h

Peek at our continued development on github: https://github.com/FAForever
Sheeo
Councillor - Administrative
 
Posts: 1038
Joined: 17 Dec 2013, 18:57
Has liked: 109 times
Been liked: 233 times
FAF User Name: Sheeo

Next

Return to Contributors

Who is online

Users browsing this forum: No registered users and 1 guest