Go Back   EQEmulator Home > EQEmulator Forums > Development > Development::Server Code Submissions

Reply
 
Thread Tools Display Modes
  #1  
Old 08-06-2008, 09:30 PM
Angelox
AX Classic Developer
 
Join Date: May 2006
Location: filler
Posts: 2,049
Default

I saw this one coming right after I diff'd the update -
Put kraegers fix in, then, when you make a better fix, take his out, put yours in! Meantime, people get a bald players fix, kraeger gets his earned recognition, and everyone is happy.
Regardless of what you think, people need recognition for their work - you all have your titles and colored names, and that's recognition for your work!
Reply With Quote
  #2  
Old 08-06-2008, 10:10 PM
trevius's Avatar
trevius
Developer
 
Join Date: Aug 2006
Location: USA
Posts: 5,946
Default

The only problem I see with that idea is that if a work-around gets put into the source, there is a high chance that it will become the permanent solution as many don't take the time to go back and do it the right way if it already mostly works correctly. Then you wind up with source code full of work-arounds instead of permanent real fixes. It isn't the worst thing in the world, but it probably makes for somewhat sloppy code that is hard to make sense from when others decide to write more code for it.

It took quite a while before this issue was even looked into. So, if it had a work around, I doubt it will ever be looked into again. It is only a cosmetic issue after-all, so it isn't like it is top priority to get perfect code in for it. Not that we appreciate the cosmetic fixes less than any other type, but they probably aren't the top of most coder's mind when considering what to fix next.

I fully agree that people need credit for their work. But, in this case, I think it is a misunderstanding at best. KLS wrote the code that went into the source, so really Kraeger's name shouldn't be on her code. But, like I said, he definitely should at least get credit for the work he did on getting the issue brought to the table and a good work around to fix it. Even if the note in the change log was something like "(Thanks for Kraeger's work on getting this fix started)" or maybe even (KLS and Kraeger) for some generic credit.

IMO, I am sure that most people who would care about who added to code already are well aware that Kraeger started the work to get this issue resolved and that he has some nice code to resolve it.

I have had one of my code adjustments (even though it was minor) added to the source without even a note about it being added lol. I didn't really care about getting my name in there, but I did make a comment to ask that a note get added for it just so people were aware that it went in.

I am proud to have a pink name for the help I do around here. But, even if I didn't I would still help out just as much. I realize that things sometimes move a little slow around here. I also realize that everything the team does, they do with the best intentions. So, if something gets missed, I know better than to get upset about it. It wasn't intentional. I just stay patient and discuss it or send reminders if needed.
__________________
Trevazar/Trevius Owner of: Storm Haven
Everquest Emulator FAQ (Frequently Asked Questions) - Read It!
Reply With Quote
  #3  
Old 08-06-2008, 10:33 PM
steve
Discordant
 
Join Date: Jan 2002
Posts: 305
Default

Quote:
Originally Posted by trevius View Post
I am proud to have a pink name for the help I do around here.
Looks purple to me... :P
Reply With Quote
  #4  
Old 08-06-2008, 11:14 PM
unknown405
Fire Beetle
 
Join Date: Oct 2007
Posts: 2
Default

Code:
--Bald players bug fixed.
Original code fix/inspiration by Kraeger
Update by KLS
That seems the best way to me. I completely side with Kraeger here as an unbiased comment. I don't know KLS and I don't know Kraeger, I'm new to the project myself so I have no bias at all but to be honest I think Kraeger is in the right here for the most part.

KLS I am very new to this project and I know exactly who you are, as everyone else does. You don't need any more recognition then the load of it that you already have. Everyone knows who you are and the fact is this never would have been done without Kraeger (at least not anytime soon.)

So I think the patch notes I put at the top of this post will make everyone happy. I would have reacted the same way as Kraeger, but the fact is not too many people read the patch notes. The only people that actually read the patch notes (for the most part) read the forums as well and know what the situation is and know who did what.

So with that said, can you just do what I suggested to make everyone happy and then forget about this issue. KLS you may not understand, but people who are unknown (ironic I know) and have no reputation make these code fixes for the appreciation and recognition more than to fix the problem for the most part.

I don't know Kraeger, maybe he's helped before. But my point is that when someone spends their time fixing something and then sees it completely re-written with different credit given it's a horrible feeling in the gut. I know because I've written several plugins for MQ2 that were stolen by other people and sold as their own, and that just leaves a wrenched feeling in my stomach.
Reply With Quote
  #5  
Old 08-07-2008, 12:34 AM
trevius's Avatar
trevius
Developer
 
Join Date: Aug 2006
Location: USA
Posts: 5,946
Default

Quote:
Originally Posted by unknown405 View Post
Code:
--Bald players bug fixed.
Original code fix/inspiration by Kraeger
Update by KLS
Well, not to beat this into the ground, but I disagree on many points you make here. For one, the patch note you made isn't exactly accurate. Yes, Kraeger did make some code to fix it, but KLS didn't update his code to make the code that went into the source. She wrote her own from scratch. Again, he should get some kind of credit, but wording it correctly isn't easy since none of his code actually went into the source. So it would have to be more like:

Code:
KLS: Fix for bald characters hopefully (inspired by code from Kraeger).
Though, she has already said that her code still needs some work, so she will still need to add more to it to get it working perfectly. Maybe she can use some of what Kraeger has written to get hers working right. If so, then I am sure she will have his name in there, because it will rightfully deserve to be there.

Quote:
Originally Posted by unknown405 View Post
That seems the best way to me. I completely side with Kraeger here as an unbiased comment. I don't know KLS and I don't know Kraeger, I'm new to the project myself so I have no bias at all but to be honest I think Kraeger is in the right here for the most part.
I agree with this.

Quote:
Originally Posted by unknown405 View Post
KLS I am very new to this project and I know exactly who you are, as everyone else does. You don't need any more recognition then the load of it that you already have. Everyone knows who you are and the fact is this never would have been done without Kraeger (at least not anytime soon.)
Didn't you just say you don't know who she is in the previous paragraph? :P Trust me when I say that she didn't write the patch notes the way she did to get herself credit. I highly doubt she even cares much about it. Since the code she put in doesn't even work perfectly yet, it is probably better for Kraeger that his name wasn't on it, lol. Again, I agree that this would definitely not be in if Kraeger hadn't taken the initiative.

I have written some code (based on other code submissions) to allow admins to disable Call of the Hero on a Per Zone Basis. If someone completely rewrote the code so that it worked on a perimeter to disable spells, does that mean I should get any credit for it? Not in my opinion. Sure, maybe I posted code to get it to work one way, but if someone rewrote it, and that new code was put into the source, I wouldn't expect my name to be on it, nor care if it was.

Quote:
Originally Posted by unknown405 View Post
So I think the patch notes I put at the top of this post will make everyone happy. I would have reacted the same way as Kraeger, but the fact is not too many people read the patch notes. The only people that actually read the patch notes (for the most part) read the forums as well and know what the situation is and know who did what.
This almost sounds like you are suggesting some corruption in the Project. But, I assure you, there is not.

Quote:
Originally Posted by unknown405 View Post
So with that said, can you just do what I suggested to make everyone happy and then forget about this issue. KLS you may not understand, but people who are unknown (ironic I know) and have no reputation make these code fixes for the appreciation and recognition more than to fix the problem for the most part.
I disagree with this. I do everything I can here just to help out the project. Whether that is posting code, quests, wiki pages, or just answering questions and helping people to get things working. I could just as easily keep everything I do to myself, but that doesn't do anyone any good. It slows the progress of the project and it is just plain selfish IMO. Overall, this community is the most mature and helpful group that I have seen anywhere on the net. There is very limited drama and many many people are willing to help out as much as they can. I think most of us do it just to make the Emulator better. So, to say that people do it just for appreciation and recognition more than just to help out isn't true in many cases. I think most of us just do it to better the project. Any other reason is small and almost pointless.

Quote:
Originally Posted by unknown405 View Post
I don't know Kraeger, maybe he's helped before. But my point is that when someone spends their time fixing something and then sees it completely re-written with different credit given it's a horrible feeling in the gut. I know because I've written several plugins for MQ2 that were stolen by other people and sold as their own, and that just leaves a wrenched feeling in my stomach.
We don't make any money off of this project. It isn't like MQ2 where there is money coming in from sales. EQEmu is open source and 100% free. Some of the admins do take money for their server, but most of them only do so to save for part upgrades to make their servers better.

We keep a close eye for any types of wrong doing in these forums. If someone was being wronged, it would be openly discussed (as this is) and whatever it takes to make it right again would be done.

For the record, Kraeger has helped before with his Random Name Generator fix. The entire community is very appreciative of any fixes that come in. I know one of my favorite things about coming here is when I see new server code submissions and get to see what has been fixed!

I think this situation is border-line. No harm or offense was meant in what was done. And, I see nothing blatantly wrong with what KLS did. I do think Kraeger over-reacted a little. And I also think that KLS probably could have added his name in there somehow to avoid this, but I don't blame her for anything. She is a coder after-all, not a customer service representative! I am sure that she wasn't intentionally trying to short-change Kraeger even if he saw it that way. I wouldn't be surprised if she has already changed the note right after this issue was first brought up. It takes a couple of days sometimes for Sourceforge to update that information.

For someone new to this project, I would suggest to please have some patience and keep following the project. This obviously isn't going to give the best impression to someone who just started recently. But I promise that if you keep following these forums you will learn to respect and trust the team. Given that we all have busy lives and things of our own to deal with, I think the team and community here is impeccable. You would have to dig seriously deep in these forums to find issues with the team or the project. And even then, I have never found one that looked anything like there was corruption or greed of any sort. I have probably read more of these forums than 99% of the people here (including very old posts and archives), and I still say that it is a very honest and mature community, team and project minus a very few exceptions here and there.
__________________
Trevazar/Trevius Owner of: Storm Haven
Everquest Emulator FAQ (Frequently Asked Questions) - Read It!

Last edited by trevius; 08-07-2008 at 08:38 AM..
Reply With Quote
  #6  
Old 08-07-2008, 09:21 PM
unknown405
Fire Beetle
 
Join Date: Oct 2007
Posts: 2
Default

I didn't read the last two paragraphs but yah when I said for the most part people who are unknown make fixes for recognition I am not lying. Take credit away from ANYONE who is not very well known and posting code and see how they react. They like the credit, they want the credit, they wouldn't care as much without the credit.

PS: About the whole I don't know KLS, I do know KLS. I meant I don't actually know her, I've never read one post made by her but I know what she does and I always see her name around the emulator forums on patch notes or just mention of the name. Sorry I know that sounded really bad, I don't know her at all then next paragraph I know exactly who you are
Reply With Quote
  #7  
Old 08-07-2008, 11:14 PM
KLS
Administrator
 
Join Date: Sep 2006
Posts: 1,348
Default

Basic rule of thumb: if the code goes in to source in any form there will be credit given, otherwise there wont.

To be honest I don't think "inspiration" is a good precedent to be setting; I've never credited it before and I doubt I'll ever credit it again except in special circumstances like this one. I will be doing it for this change, just because I agree that it probably wouldn't of happened without him.

Change will be going in tonight we'll see if it works better, I haven't run into any issues with it but I can only check so many race/gender/hair combos.
Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

   

All times are GMT -4. The time now is 12:47 PM.


 

Everquest is a registered trademark of Daybreak Game Company LLC.
EQEmulator is not associated or affiliated in any way with Daybreak Game Company LLC.
Except where otherwise noted, this site is licensed under a Creative Commons License.
       
Powered by vBulletin®, Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
Template by Bluepearl Design and vBulletin Templates - Ver3.3