Jump to content

latios

Members
  • Posts

    25
  • Joined

  • Last visited

  • Days Won

    1

Everything posted by latios

  1. Hi @Hakahay, querying other language/region DP databases using dp2rathena is not possible yet. Though Divine-Pride does have the feature of querying different language/region databases through their website and API.
  2. Hi again @exyoupjkbrxd, in this case I think you may want to reach out to Dia, the admin of Divine-Pride for support: https://www.divine-pride.net/forum/index.php?/staff/ It sounds like there may be an issue with your API key if you aren't able to make any standard API calls even without using dp2rathena ?
  3. Hi @exyoupjkbrxd, that's quite strange ?. I have a few questions and ideas: If you copy and paste the URL (with your actual API key) into a browser, are you able to see the relevant JSON data? https://divine-pride.net/api/database/monster/20592?apiKey=yoursecretapikey Does it work if you call the command by passing in your API key? dp2rathena -k yoursecretapikey mob 20592 There might be some whitespace at the beginning/end of your API key. I was able to successfully pull the data for mob 20592 with my own API key:
  4. Hi there @kennykiro, it's unclear to me what kind of struggle you are having. Do you have an example of where you are stuck? Additionally, are you already familiar with using command-line (CLI) tools, if so, do you have an examples of CLI tools you like to use? I may be able to give better examples if I understand what you are familiar with. @kennykiro I've updated the GitHub repo with slightly clearer information and also a small terminal animation, which may or may not help you but hopefully it does ?
  5. Hi @Snaehild, apologies I haven't quite finished getting mob_db.txt conversion feature yet, but have just released the mob_db.yml conversion feature in case that's still relevant to you. I've got mob_db.txt feature at the top of my list now.
  6. Hi there Snaehild! I'm so glad you like it! I hope to release support for mob_db.txt by the end of this month ?
  7. I've just released version 0.3.0 which allows for converting mob skills and creating a mob_skill_db.txt file. To update your dp2rathena to the latest version, you'll need to run: pip install -U dp2rathena Instructions for how to use the new feature are listed at the top of this thread and within the tool help text. For example running the help command: dp2rathena mobskill --help Will produce this help text: Let me know if you come across any issues or have any requests/suggestions. I'll do my best to reply promptly
  8. Hi all, I put together a command-line tool to help save time when porting lots of existing items to an rAthena server: https://github.com/Latiosu/dp2rathena I noticed that the rAthena community often refers to divine-pride.net (DP) as a source of truth, so I figured it would be useful to connect with their API. Summary The tool uses data from divine-pride.net and attempts to map the API response to a desired rAthena database format. Installation Requires Python 3.6+ and pip installed: pip install dp2rathena Usage Generate a divine-pride.net API key if you don't have one yet (registration required), then you can start converting data: Example 1: Store API key and convert items with id 501 and 1101 dp2rathena config dp2rathena item 501 1101 Produces this output: Example 2: Convert mobskills for mobs with id 1002, 1049 and 1299 and save to a file called "attacks.txt" (re-uses stored API key if present) dp2rathena mobskill 1002 1049 1299 > C:\Users\Latios\Desktop\attacks.txt Produces a file in the Desktop folder called "attacks.txt": Example 3: Convert mob with id 20357 to mob_db.yml format dp2rathena mob 20357 Produces following command-line output: Further instructions can be found at the repository: https://github.com/Latiosu/dp2rathena Limitations I've moved this section to the GitHub repository above to keep it up-to-date. --- Let me know if you'd like to be able to convert other data from DP, some other suggestion or any questions. Edit: Add terminal animation for a visual explanation
  9. @ahloi007 The first issue is saying that "j" hasn't been defined on line 2705 in file pc.cpp. You'll want to look around the code there and try to figure out why the definition (e.g. "int j;") is missing. For example, in this part of the diff there is an "int j;" at the top, but isn't there for some reason in your code. @@ -2627,7 +2638,7 @@ void pc_exeautobonus(struct map_session_data *sd, std::vector<s_autobonus> *bonu int j; unsigned int equip_pos_idx = 0; //Create a list of all equipped positions to see if all items needed for the autobonus are still present [Playtester] - for(j = 0; j < EQI_MAX; j++) { + for(j = 0; j < EQI_MAX_BONUS; j++) { if(sd->equip_index[j] >= 0) equip_pos_idx |= sd->inventory.u.items_inventory[sd->equip_index[j]].equip; } The second issue is saying that the constant "EQI_MAX_BONUS" isn't found anywhere in the pc.cpp or pc.hpp files. In the diff, there is a section which indicates it should be added to pc.hpp. You'll want to check if it's there and if it is, figure out why it might not be found in pc.cpp. Here is the part of the diff where it gets added: diff --git a/src/map/pc.hpp b/src/map/pc.hpp index b52784b..23d1a85 100644 --- a/src/map/pc.hpp +++ b/src/map/pc.hpp @@ -84,7 +84,8 @@ enum equip_index { EQI_SHADOW_SHOES, EQI_SHADOW_ACC_R, EQI_SHADOW_ACC_L, - EQI_MAX + EQI_MAX, + EQI_MAX_BONUS = 10 };
  10. What is the best way to help approved pull requests (not on hold) get merged? For example this filtered list of pull requests: https://github.com/rathena/rathena/pulls?q=is%3Apr+is%3Aopen+review%3Aapproved+-label%3Astatus%3Aonhold Would adding things such as screenshots or a video of testing help? I suppose I'm wondering how to help aspiring contributors understand what work is remaining on a PR before it gets merged. It'd be helpful to know if we can do anything else to assist devs for a given PR.
  11. I have also been wondering why Dancer/Bard songs aren't showing skill effects. I'm looking through the commit to see if I can spot anything that might have caused it. @CyberDevil Are you able to confirm that the Dancer/Bard skills work again when you deploy the previous commit 850904a? @CyberDevil Ah, it seems like there is meant to be new skill animations for Bard/Dancer songs/dances; as a part of this update: https://www.divine-pride.net/forum/index.php?/topic/3453-kro-mass-skills-balance-1st-2nd-and-transcendent-classes-skills/ I'd imagine you can revert the changes by removing the #ifndef and #endif statements in skill.cpp. Plus add the units back into each song/dance in db/re/skill_db.yml
  12. Apologies, I may not be able to fix it such that it works with "git apply" for some time. It will have to be a manual process for now.
  13. @ahloi007 Ah apologies, I think I've made a mistake in the uploaded diff. I'll post a fixed version soon, one that works when you run: git apply "Costume System v2.diff"
  14. Thanks @Emistry, that makes sense. Although if I wanted to create a Pull Request to amend values in mob_db.txt based on values in the kRO database on Divine-Pride (e.g. Combat Basilisk Base & Job EXP https://www.divine-pride.net/database/monster/3503/combat-basilisk) would that be rejected since it's not 100% proof direct from kRO (like in-game screenshots)? I suppose I could also just create the PR and see if it's rejected or not. Basically, I'm wondering what alternative sources (besides AEGIS conversion or direct from kRO) I could use to assist with updating rAthena DBs.
  15. Hi there fellow rAthenians (is that right?), I've loved playing RO since I was rather young (back in 2004) and have played it on and off since. I have many fond memories of the original servers and various private servers when I discovered them many years later. I've recently setup my own private server based in Sydney using rAthena and it's been a blast. I'm really loving all the wonderful contributions and content the community here has made. I'm planning to keep an eye out on how I can give back to rAthena for the many joys it has enabled me to have. Hello again and thanks to everyone here in advance! - Latios
  16. Would anyone here happen to know what rAthena uses as a reference or source of truth for correcting mistakes in mob_db.txt? In other words, how would someone know whether a line in mob_db.txt has a mistake in it or not? I have done some digging around the forums and previous GitHub commits but haven't found that states where the values come from or what process to use to confirm correctness. I personally have been using a combination of Divine-Pride, RMS Mob DB and iRO Wiki DB and seeing which has most realistic values and correlating with similar mobs.
  17. @Amir Azman Nyliar's script for Charleston Crisis still works for the rAthena: rAthena Script: https://github.com/Nyliar/RagnarokContent/tree/master/Instance/Officials/Rathena/CharlestonCrisis Client & Server files (in case you are missing them): https://github.com/Nyliar/RagnarokContent/tree/master/Instance/Officials/Common Files/CharlestonCrisis Also remember to update the following line in your "db/re/instance_db.txt" (example diff below): -// 30,Charleston in Distress,3600,300,1@mcd,127,282 +30,Charleston Crisis,3600,300,1@mcd,127,282 I also replaced Nyliar's merchant with CrazyArashi's merchants since I liked the layout much more. Here is how it looks on my server:
  18. @Captain Oh I see what you mean. There is a nice example of how to remove this limitation in this thread: Hopefully that meets your needs.
  19. Hi all, I've modified secret's script to service third jobs only with the jRO costume. This was to get around the confusing issues that appear with skills when switching base classes. For those who were still wondering how to fix the issue where costume doesn't stay after player reconnects, you'll need to turn on "save_body_style" in your battle config. Here's what rathena's doc/script_commands.txt says: Only the look data for hairstyle, hair color and clothes color are saved to the char server's database and will persist. Body style will also persist if 'save_body_style' configuration is enabled in '/conf/battle/client.conf'. Screenshot of modified script (and NPC sprite): DressRoom_ThirdOnly.txt
  20. @Captain Yes, that's correct. Although a way around this is to "costume" the item on another character who can wear it and then put it in storage to transfer back to your other character to wear. I don't believe the item permissions (storage/trade/drop/etc ...) are modified, so if you could trade it before you can still trade it after "costuming" the item.
  21. Hi all, I've taken the time to update this script and tested it on my server running latest version of rAthena (version at the time of writing Git Hash: f8d5d45). Additional Changes NPC name can now be easily customised by replacing 2nd argument in the function call at the bottom of npc/custom/etc/costume.txt NPC checks whether the user already has the same item being restored equipped or in inventory and will prevent restoring. Fixed/tweaked some typos and grammar for the NPC dialogue and atcommand messages. costume.txt has been included inside the diff Screenshot Costume_System_v2.diff Edit: I actually just discovered that the wonderful secretdataz had already updated the diff to work with newer versions of rAthena here (https://github.com/secretdataz/rathena/pull/8/files). The main differences to my edit is secret's also handles costuming garments and there is atcommand documentation. The diff I have uploaded comes with an updated NPC script.
  22. @Secrets I've submitted a few Pull Requests to fix an issue with the Amount field and also update the README.md to be similar to your other projects.
×
×
  • Create New...