Shotgun Ammunition type field updates #40

Closed
opened 2023-04-10 16:31:05 -04:00 by aaron_nad · 4 comments

Hi, whilst looking over the new schema - which is amazing that its supported!
I might have missed asking for the muzzle velocity to be added initally
81f21a02c4/lib/cannery/ammo/type.ex (L81)

Can the Bullet Type field be ommitted - this is not required on shotgun cartridges (we have the shot type to cover the use of the cartirdge for geese, pidgeons, clay and foxes etc)

As always everything else is amazing as always and I really appreciate the time spent adding new features!

Hi, whilst looking over the new schema - which is amazing that its supported! I might have missed asking for the muzzle velocity to be added initally https://gitea.bubbletea.dev/shibao/cannery/src/commit/81f21a02c4dbf632b3cca576ea3a0dcbad835f26/lib/cannery/ammo/type.ex#L81 Can the Bullet Type field be ommitted - this is not required on shotgun cartridges (we have the shot type to cover the use of the cartirdge for geese, pidgeons, clay and foxes etc) As always everything else is amazing as always and I really appreciate the time spent adding new features!
Owner

based on recommendations, i've instead added DRAM equivalent as the replacement for a shotgun slug's muzzle velocity, since that seems to be the most widely used equivalent for measuring muzzle velocity with shotgun ammunition, and is printed on shotgun ammunition boxes at least here in the united states. You're free to simply enter muzzle velocity in the dram equivalent field in your instance, and I will add a note about choosing between DRAM equivalent or muzzle velocity units into #13.

For the bullet type showing for shotgun types, I think I may need to fix that in this ticket😄

based on recommendations, i've instead added DRAM equivalent as the replacement for a shotgun slug's muzzle velocity, since that seems to be the most widely used equivalent for measuring muzzle velocity with shotgun ammunition, and is printed on shotgun ammunition boxes at least here in the united states. You're free to simply enter muzzle velocity in the dram equivalent field in your instance, and I will add a note about choosing between DRAM equivalent or muzzle velocity units into #13. For the bullet type showing for shotgun types, I think I may need to fix that in this ticket😄
Author

Thanks for the speedy reply! Yep, no problemo, i'll do that. I was just reading up on DRAM conversions - its not widely used here in the UK and we still stick with the FPS for everything involved.

Im happy with this thank you

Thanks for the speedy reply! Yep, no problemo, i'll do that. I was just reading up on DRAM conversions - its not widely used here in the UK and we still stick with the FPS for everything involved. Im happy with this thank you
Owner

I'm curious, do they not print DRAM on shotgun ammo in the UK?

I'm curious, do they not print DRAM on shotgun ammo in the UK?
Owner

Removed bullet type for shotgun ammo in v0.9.2

Removed bullet type for shotgun ammo in v0.9.2
shibao added this to the cannery project 2023-04-10 22:16:47 -04:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: shibao/cannery#40
No description provided.