Jump to content

Template talk:Infobox mountain: Difference between revisions

Page contents not supported in other languages.
From Wikipedia, the free encyclopedia
Content deleted Content added
→‎Wikidata: RfC for wikidata into infoboxes was contentious: I think we need individual field icons to keep data reliable
Line 501: Line 501:
:Looking forward to your comments. [[User:Rehman|<span style="font-variant:small-caps; font-weight:bold; color:darkblue">Reh</span>]][[User talk:Rehman|<span style="color:green">man</span>]] 04:12, 31 March 2020 (UTC)
:Looking forward to your comments. [[User:Rehman|<span style="font-variant:small-caps; font-weight:bold; color:darkblue">Reh</span>]][[User talk:Rehman|<span style="color:green">man</span>]] 04:12, 31 March 2020 (UTC)
::{{ping|Rehman}} I think that the automatic conversions should not be removed. When the infobox is supplied with dimensions for a mountain or range, it automatically figures out the correct scaling for the geohack map (via a call to {{tl|infobox dim}}). Check out the code at {{para|data9}} in the infobox. If we delete {{para|length_*}}, {{para|width_*}}, or {{para|area_*}}, we lose this nice feature. — [[User:Hike395|hike395]] ([[User talk:Hike395|talk]]) 06:19, 8 April 2020 (UTC)
::{{ping|Rehman}} I think that the automatic conversions should not be removed. When the infobox is supplied with dimensions for a mountain or range, it automatically figures out the correct scaling for the geohack map (via a call to {{tl|infobox dim}}). Check out the code at {{para|data9}} in the infobox. If we delete {{para|length_*}}, {{para|width_*}}, or {{para|area_*}}, we lose this nice feature. — [[User:Hike395|hike395]] ([[User talk:Hike395|talk]]) 06:19, 8 April 2020 (UTC)
:::{{tl|Coordinates}} already has a default feature <code>type:mountain</code> used for scaling, which is for <code>peaks, mountain ranges, hills, submerged reefs, and seamounts</code>, and takes away the need of that chunk of code and complexity, and standardises all geohack maps even if length or width is not provided. For some reason, this infobox has implemented an override, which is now used by 1.3% of articles, or a maximum of 328 articles only. Is there a particular problem we are trying to solve by overriding the default? [[User:Rehman|<span style="font-variant:small-caps; font-weight:bold; color:darkblue">Reh</span>]][[User talk:Rehman|<span style="color:green">man</span>]] 04:50, 9 April 2020 (UTC)


==== Wikidata ====
==== Wikidata ====

Revision as of 04:50, 9 April 2020

WikiProject iconMountains Template‑class
WikiProject iconThis template is part of WikiProject Mountains, a project to systematically present information on mountains. If you would like to participate, you can choose to edit the article attached to this page (see Contributing FAQ for more information), or visit the project page where you can join the project and/or contribute to the discussion.
TemplateThis template does not require a rating on Wikipedia's content assessment scale.
WikiProject iconVolcanoes Template‑class
WikiProject iconThis template is within the scope of WikiProject Volcanoes, a collaborative effort to improve the coverage of volcanoes, volcanology, igneous petrology, and related subjects on Wikipedia. If you would like to participate, please visit the project page, where you can join the discussion and see a list of open tasks.
TemplateThis template does not require a rating on Wikipedia's content assessment scale.


Prominence from wikidata?

I noticed that Mount Everest currently has no figure for prominence. Apparently an explicit figure in the article was removed because {{Infobox mountain}} is supposed to pull it from Wikidata. It evidently does so for elevation, but not prominence, even though it does use {{convert|input=P2660}} as you'd expect. Hairy Dude (talk) 21:48, 3 June 2019 (UTC)[reply]

 Fixed {{convert}} gets confused when there are two values for prominence. Removed the imperial units, now it works. —hike395 (talk) 02:58, 4 June 2019 (UTC)[reply]

Map label blank if range coords are given

I added the range coordinates to the infobox for Mount Aylmer but now the map label is missing from the infobox. A bug or did I specify a wrong parameter for {{coord}}? RedWolf (talk) 17:11, 13 August 2019 (UTC)[reply]

@RedWolf: It's not a bug, but it's a consequence of the merge between {{Infobox mountain range}} and {{Infobox mountain}}. Like Geobox, the old {{Infobox mountain range}} did not put labels next to the mark on the map, while {{Infobox mountain}} did. In order to do the merge and keep backwards compatibility, I had to make a guess at whether the article was about a single mountain or a whole mountain range. My assumption: if range coordinates were supplied, then it was about a range, otherwise a mountain. Thus, if range coordinates are supplied, the label is suppressed (to be backwards compatible with mountain ranges).
You're entering |range_coordinates= for single peaks, which I had never intended. Of course, the documentation is obscure and doesn't forbid this. But you're getting the mountain range formatting, which includes:
  • No label next to the mark on the map
  • Mark on map set to range_coordinates, not coordinates
  • Automatically computed dim: for geohack is applied to range_coordinates, not coordinates
You may find these surprising.
I think I should clarify the documentation to state that |range_coordinates= are intended only for mountain ranges, not for single peaks. I would suggest not adding them to articles like Mount Aylmer. —hike395 (talk) 07:04, 14 August 2019 (UTC)[reply]
  • @Hike395: Thanks for clearing up the mystery. I didn't want to dig into the infobox code given it uses IMHO the very distasteful template syntax rather than LUA. Yes, I agree that the documentation should be updated to match the behavior. I'm not sure what the rationale was for not including a label on maps for mountain ranges. I looked at a few other infoboxes and some have the labels and some don't. Perhaps it comes down to a personal preference among the leading editors that use the infobox. For the record, I prefer the labels. I've commented out the range coords from Mt. Aylmer to restore the map label. RedWolf (talk) 18:37, 14 August 2019 (UTC)[reply]

grid_ref_Ireland

Something seems to have gone wrong with the "grid_ref_Ireland" item in the infobox. Now giving an error of "Malformed OS grid ref: Malformed NGR" in all Irish mountain infoboxes (examples at Ben Lugmore and Mweelrea). Any idea what has happened? thanks. Britishfinance (talk) 09:35, 19 August 2019 (UTC)[reply]

The same error of "Malformed OS grid ref: Malformed NGR" is also appearing on the template page of "infobox mountain" under "OSI/OSNI grid" and "OS grid". Britishfinance (talk) 11:17, 19 August 2019 (UTC)[reply]
I notice that someone seems to have dynamically linked the grid ref to GeoHack maps (which is a good thing); however, where a reference was added for the grid ref, this is causing a problem and the error. I have deleted the refs for Ben Lugmore and Mweelrea above and it works now, however, there are lots of other Irish mountains with this problem that will need to be adjusted. Britishfinance (talk) 13:12, 19 August 2019 (UTC)[reply]
Britishfinance, it looks like Hike395 may be able to help. I put some experimental code to split the references from the text in Module:OS coordinates/sandbox, and it kind of works, but someone like Johnuniq could probably make it actually work. the other issue is that the text is passed twice by {{gbm4ibx}} and {{iem4ibx}}, so we may only want to put something like {{#invoke:Plain text|main| }} in {{gbm4ibx}} and {{iem4ibx}} and then have the ref splitting only on the link text? Frietjes (talk) 14:46, 19 August 2019 (UTC)[reply]
I could look at this in a couple of days if it's still a problem. A short test case showing the problem would be good. Johnuniq (talk) 00:13, 20 August 2019 (UTC)[reply]
It seems like this field was dormant (not dynamically linked to GeoHack), but Frietjes has been able to link it. This is a nice upgrade, however, there are many infoboxes where the grid_ref field contains the grid_ref plus a citation (from where the ref came), which is causing an error (e.g. Lugduff). However, it will be worth fixing this up if the update by Frietjes works? Perhaps if in the absence of a valid grid_ref, the system would avoid printing an error message and just leave the text? Britishfinance (talk) 00:17, 20 August 2019 (UTC)[reply]

Sorry for the inconvenience. For now, I will just have Module:OS coordinates produce unlinked text if it cannot parse the OS grid ref. I'll leave the error tracking category in place. —hike395 (talk) 02:10, 20 August 2019 (UTC)[reply]

Great, thanks, that works. Britishfinance (talk) 09:16, 20 August 2019 (UTC)[reply]
Also, I think it would be worth adding to the infobox template guide in the grid_ref section, to only insert gridrefs into those infobox fields and not any references? Britishfinance (talk) 09:19, 20 August 2019 (UTC)[reply]
I think Britishfinance is on the right track here. I don't see a way to fix this from within Lua (because by the time the reference has reach Lua, it's munged beyond recognition). What we need to do is add two new parameters to the infobox, |grid_ref_UK_note= and |grid_ref_Ireland_note=, and then move the references into the corresponding new parameter. Someone could run AWB to do a mass split. Unfortunately, my Windows computer is busted right now, so I can't run AWB. —hike395 (talk) 12:03, 20 August 2019 (UTC)[reply]

 Done --- added |grid_ref_UK_note= and |grid_ref_Ireland_note=. @Britishfinance: The problem doesn't show up on all Irish mountains, just the ones with references in the field. A brief spot check shows that this isn't terribly common --- you might be the only editor who is doing this? Could you possibly go back and split out the references into the new note fields on the mountains that you've edited? That would be a lot of help! —hike395 (talk) 12:36, 20 August 2019 (UTC)[reply]

Thanks Hike395, I will do that on the Irish ones for sure as it is a good improvement overall (and a critical data item). thanks again. Britishfinance (talk) 13:01, 20 August 2019 (UTC)[reply]
@Hike395: Was away and only getting back to my list and have seen that you had already sorted this for me – huge thanks for that! Britishfinance (talk) 09:02, 17 October 2019 (UTC)[reply]

Template-protected edit request on 10 October 2019

Please change template call {{ifempty}} to {{if empty}} to avoid the redirect. The call to that template is transcluded in nearly 24,000 articles through being called in this template, so that unnecessary redirect has to be followed every time one of those articles is viewed. Colonies Chris (talk) 15:43, 10 October 2019 (UTC)[reply]

 Not done for now: please establish a consensus for this alteration before using the {{edit template-protected}} template. For further discussion see User talk:Colonies Chris. --Trialpears (talk) 16:31, 10 October 2019 (UTC)[reply]

foot_montage font size

The font size in the caption produced by the foot_montage parameter is too small and contradicts MOS:FONTSIZE. See Sparrow Hills for an example. Please correct (unfortunately, I don't know how, since this is not even mentioned in the documentation, neither present in the source code). — Mikhail Ryazanov (talk) 10:38, 20 November 2019 (UTC)[reply]

 Not done: This parameter is in Template:Photomontage. I found and fixed some 80% text in this template, however, so your request was not for naught. – Jonesey95 (talk) 15:33, 20 November 2019 (UTC)[reply]
I have also adjusted the font size in Template:Photomontage, so Sparrow Hills looks more reasonable now. – Jonesey95 (talk) 15:41, 20 November 2019 (UTC)[reply]
@Jonesey95 and Mikhail Ryazanov: as a general rule, it's better to move the caption to the caption parameter provided by the infobox since you will get the same font-size as the font-size used by non-montage images. also, I find that multiple image does a better job of sizing the images to reduce non-uniform whitespace between the images. Frietjes (talk) 17:25, 20 November 2019 (UTC)[reply]

Infobox cleanup

Hello. I'm attempting to amend and clean-up the template code so as to achieve the below:

  1. Enable Wikidata values when no local value is present.
  2. Add non-static interactive map support (example)
  3. Clean up duplicate parameters (random example: range_coordinates and range_coords ) - to be done only together with another edit
  4. Remove redundant parameters (random example: state1 through state18) which can be instead included as a UBL in a single parameter - to be done only together with another edit
  5. Update documentation

As part of #1, I have added the Wikidata support codes at Template:Infobox mountain/sandbox for most critical parameters, with the documentation being drafted at Template:Infobox mountain/sandbox/doc. The code will be moved to live after another review, and after the documentation of Wikidata properties is complete. There should be no visual difference during this change, and no functional change to existing usages. Hence if there are any issues, please do raise it up here, and I'll get to it right away. At the same time, if anyone has any comments or suggestion, please do state here, and I will do my best to incorporate it. Best wishes! Rehman 11:53, 29 March 2020 (UTC)[reply]

@Rehman: I noticed that you deleted the bodystyle, labelstyle, datastyle, imagestyle and captionstyle in Template:Infobox mountain/sandbox, which does cause a visual difference. Can you kindly revert those deletions? I'm happy to discuss the formatting of the infobox, but that should be separate from the major edit that enables using Wikidata. — hike395 (talk) 06:10, 30 March 2020 (UTC)[reply]
Hi Hike395. That's just in the sandbox environment - I've trimmed it for my convenience of handling the code. The live edit will not have those changes. Rehman 06:39, 30 March 2020 (UTC)[reply]
@Rehman: I would find it easier to look through the Template:Infobox mountain/testcases if the formatting in the sandbox was the proposed formatting. May I restore them in the sandbox? I like having the sandbox be the exact proposal for the edit, rather than relying on a possible error-prone future edit. — hike395 (talk) 06:54, 30 March 2020 (UTC)[reply]
Sure, that makes sense. I will restore shortly. Rehman 07:48, 30 March 2020 (UTC)[reply]
Done. Rehman 03:32, 31 March 2020 (UTC)[reply]

Hi Plastikspork. Would you be able to assist in running your bot through articles using {{Infobox mountain}}, to make the following changes please?

Rename duplicate parameters
From To
coords coordinates
type mountain_type
topo topo_map
relief map_relief
part_type subdivision_type
parent range
language native_name_lang
Consolidate values through {{Convert}}
Parameter to clear Destination
area_km2 Move to area, within {{Convert}}
area_mi2
elevation_m Move to elevation, within {{Convert}}
elevation_ft
isolation_km Move to isolation, within {{Convert}}
isolation_mi
length_km Move to length, within {{Convert}}
length_mi
prominence_m Move to prominence, within {{Convert}}
prominence_ft
width_km Move to width, within {{Convert}}
width_mi
*If "destination" is already filled, then the other parameters can simply be deleted.
Convert to footnote and append output in the "misc" parameter
Parameter to clear Move to suffix of
area_note area
coordinates_note coordinates
coordinates_ref
coords_ref
range_coordinates_note range_coordinates
elevation_ref elevation
elevation_note
grid_ref_Ireland_note grid_ref_Ireland
grid_ref_UK_note grid_ref_UK
isolation_ref isolation
length_note length
width_note width
Consolidate parameter series using {{Unbulleted list}}
Parameter to clear Replacement action
border Move contents to border, in the same order, using {{Unbulleted list}}
border1
border2
border3
border4
border5
border6
border7
border8
city Move contents to city, in the same order, using {{Unbulleted list}}
city1
city2
city3
city4
city5
city6
city7
city8
city9
city10
city11
city12
city13
city14
city15
city16
country Move contents to country, in the same order, using {{Unbulleted list}}
country1
country2
country3
country4
country5
country6
country7
country8
district Move contents to district, in the same order, using {{Unbulleted list}}
district1
district2
district3
district4
district5
district6
district7
district8
district9
geology Move contents to geology, in the same order, using {{Unbulleted list}}
geology1
geology2
geology3
geology4
geology5
rock
part Move contents to subdivision, in the same order, using {{Unbulleted list}}
part1
part2
part3
part4
part5
part6
part7
part8
part9
part10
part11
part12
part13
part14
part15
part16
age Move contents to age, in the same order, using {{Unbulleted list}}
period
period1
period2
period3
period4
region Move contents to region, in the same order, using {{Unbulleted list}}
region1
region2
region3
region4
region5
region6
region7
region8
region9
region10
region11
region12
region13
region14
region15
region16
region17
region18
region19
region20
region21
region22
region23
settlement Move contents to settlement, in the same order, using {{Unbulleted list}}
settlement1
settlement2
state Move contents to state, in the same order, using {{Unbulleted list}}
state1
state2
state3
state4
state5
state6
state7
state8
volcanic_arc/belt Move contents to volcanic_region, in the same order, using {{Unbulleted list}}
volcanic_arc
volcanic_belt
volcanic_field

Uses of {{Infobox mountain range}} should be changed to {{Infobox mountain}} to avoid redirect.

I have manually fixed erroneous usages based on the previous parameter scan. Once the above changes are done, would you be able to run one final scan please?

Thank you so much! Rehman 05:34, 30 March 2020 (UTC)[reply]

@Plastikspork and Rehman: Please hold off on running the bot. We should come to a consensus of whether we want to drop these parameters. Some of them are convenient for editors. — hike395 (talk) 06:14, 30 March 2020 (UTC)[reply]
Hi Hike395. These are all duplicate parameters. None of the changes would remove/disable/change any output in live articles. Which parameter are you referring to, so that I may have a look? Rehman 06:39, 30 March 2020 (UTC)[reply]
@Rehman: See below. — hike395 (talk) 06:50, 30 March 2020 (UTC)[reply]
Plastikspork, sorry for the pings. Please hold the bot run pending below discussion. Rehman 07:48, 30 March 2020 (UTC)[reply]
I have added sub-heading for each discussion area, for easier navigation and reading. Rehman 03:38, 6 April 2020 (UTC)[reply]

Discussions

Volcano parameters

@Rehman: The merging of volcanic_arc/belt, volcanic_arc, volcanic_belt and volcanic_field to volcanic_region sounds like a good idea and it is something I have been thinking about for some time. There is something else I have thought about and that is stratigraphic units. It is not uncommon for mountains made of volcanic or sedimentary rocks to be part of groups or formations. However, there seems to be no parameter for such features. I am not sure what the best code name would be, perhaps stratigraphic_unit? Volcanoguy 22:46, 31 March 2020 (UTC)[reply]

I would also like to note that last_eruption should probably be renamed to last_known_eruption since it is commonly used for the youngest dated eruptions using, radiocarbon, potassium-argon and other methods. There is always the possibility of there being younger eruptions that have not been dated/witnessed, especially volcanoes in remote locations. Volcanoguy 23:37, 31 March 2020 (UTC)[reply]

Thanks for the feedback, User:Volcanoguy. Noted on the volcanic_xxx parameters. I went ahead and added stratigraphic_unit to the sandbox (pending live edit, together with the others). Do you have a couple of examples I could look at (i.e. the types of values that may be added to this parameter), so I can work on the Wikidata data model and also update the documentation? Cheers, Rehman 03:46, 1 April 2020 (UTC)[reply]
Brown Bluff currently has James Ross Island Volcanic Group in volcanic_field and Mount Erebus has McMurdo Volcanic Group in volcanic_belt but both should be moved to stratigraphic_unit once it becomes available. Volcanoguy 05:46, 1 April 2020 (UTC)[reply]
User:Volcanoguy, thanks! On a separate note, when volcanic_arc/belt, volcanic_arc, volcanic_belt, and volcanic_field, is renamed to volcanic_region, would contents of the new stratigraphic_unit also fit into volcanic_region? Or would you still prefer the separate parameter? Rehman 07:12, 1 April 2020 (UTC)[reply]
Stratigraphic units and volcanic regions are two different things, not to mention stratigraphic units are not limited only to volcanoes. I was the one who added the volcanic groups in the volcanic_field and volcanic_belt parameters, simply because there isn't an appropriate parameter to add them in. So yes stratigraphic_unit and volcanic_region would be better off as separate parameters. Volcanoguy 18:36, 1 April 2020 (UTC)[reply]
Noted, and thanks for the clarification. Rehman 02:17, 2 April 2020 (UTC)[reply]
Regarding last_eruption, changing the label to "Last known eruption" would widen the infobox or drop text to the next line. Would it suffice mentioning it in the documentation (as is the case now)? Rehman 04:11, 1 April 2020 (UTC)[reply]
Yes. Volcanoguy 05:46, 1 April 2020 (UTC)[reply]

@Rehman: I'm curious as to why you list coords and range_coordinates as duplicate parameters. coords is intended for individual mountains while range_coordinates is intended only for mountain ranges. Volcanoguy 01:15, 2 April 2020 (UTC)[reply]

User:Volcanoguy, range_coordinates was basically the generic coordinates parameter for the former {{Infobox mountain range}}. When that infobox was merged in to this sometime back, the parameter was added as an additional parameter instead of merging into this infobox's coordinates parameter, while only the documentation was updated as "range coordinates".
Ignoring the above for arguments sake, having two separate parameters still does not make sense because if a mountain-article states the coordinates of the mountain, that coordinates is valid for the mountain range as well. And if a mountain-range article states the coordinates for the range, that is still the only default coordinates for the article. What I'm trying to say is, the two parameters would not clash, and should have been maintained as a single parameter. Rehman 02:17, 2 April 2020 (UTC)[reply]
Good catch, Volcanoguy. coords and range_coordinates are not duplicates! coords are the coordinates of the highest peak or summit of a range, while range_coordinates are for the centroid of the range. range_coordinates center the range on the geohack map, are zoomed out, and appear in the title, while coords are zoomed in to the highest peak, and are inline only (if both are supplied). This allows readers to see both a contextual map of the entire mountain range, or a detailed map of the highest peak. — hike395 (talk) 02:23, 2 April 2020 (UTC)[reply]
Adding two coordinates should not be a problem. And I can also add Wikidata support for both, using qualifiers. But the newer version of geohack does not need two coordinates saved in order to pan or alter zoom. In fact if I'm not mistaken, the newer maps can geomask the whole mountain range, while also adding a placemark for the summit. I'll check on the latter and comment again, but I'm certain about not requiring two coordinates for map adjustment. Rehman 02:48, 2 April 2020 (UTC)[reply]
Sorry, perhaps I wasn't clear. For a concrete example, take a look at Rocky Mountains. coords makes a link under the "Highest Point" section: when you click on it, you get taken to a geohack map of Mount Elbert. range_coordinates makes a link under the Geography section, and when you click on it, you get taken to a zoomed-out geohack map of the entire range. The latter gets used as the title coordinates. For ranges, you want two coordinates in case the highest peak is not near the center of the range. — hike395 (talk) 03:02, 2 April 2020 (UTC)[reply]
Right, okay now it makes sense. Sorry I misunderstood your comment as something to do with adjusting the maps. I've updated the table above, and will model the same for Wikidata as well. Cheers, Rehman 03:24, 2 April 2020 (UTC)[reply]
Maybe coords should be renamed to peak_coordinates or something similar to avoid confusion? Volcanoguy 03:39, 2 April 2020 (UTC)[reply]
User:Volcanoguy, yes that would. I went ahead and updated the sandbox, but had to revert again after realising that coordinates has to be a standard, in compliance with WP:MOSINFOBOX. Rehman 11:45, 3 April 2020 (UTC)[reply]

Combining parameter series

@Rehman: The parameters that you are proposing to delete are not redundant. The proposed change would affect the appearance and functionality of the infobox. Namely:

  • country*, state*, region*, district*, settlement*, city*, border*, and part* do not currently behave as {{unbulleted list}}. Instead, if 5 or more entries are provided, the infobox uses {{collapsible list}}, if fewer than 5, it uses {{enum}} While a bot can certainly perform this logic once, future editors probably will not realize that they should be using {{collapsible list}} for long entries in the infobox and {{enum}} for short. By removing these parameters, we are encouraging future sprawl of the infobox.
  • period* and geology* are similar, in that they use {{enum}} for formatting. {{enum}} is more compact than {{unbulleted list}}. Deleting these parameters would have a similar effect as deleting the other parameters: it would encourage sprawl and non-uniformity of infoboxes.
  • elevation_*, prominence_*, isolation_*, length_*, width_*, area_*, volume_* are convenient shortcuts for editors. It seems like a shame to force future editors to use {{convert}} when the infobox used to do the conversion for them. The current infobox code also enforces unit conversion, while future editors can use whatever units they wish (e.g, furlongs).

It seems that you've proposed multiple major edits to the template at once. I'd recommend breaking the discussion and editing into three phases:

  1. Using Wikidata
  2. Deleting parameters
  3. Changing the formatting

Can we defer deciding on whether to delete the parameters until we've decided on whether to pull data from Wikidata? — hike395 (talk) 06:48, 30 March 2020 (UTC)[reply]

Thank you for the feedback.
  • #1 is complex, and requires the duplicate parameters to be cleared first, before we can fully integrate it. Most of the code is already prepared though. I'd like to reiterate that is only as a fallback - if a local value is present, wikidata value will not be shown. Hence we're only adding information on the infobox, not removing. If you have any concerns here, we could discuss, but otherwise I don't think this is a topic that requires discussion.
  • For #2, I'd like to emphasise that we are not deleting parameters, but instead removing duplicates and redundancy (most of which were results of a previous merger).
  • There is no #3 at this point.
  • Thanks for raising the concern regarding the listing style/conversion. I can most certainly change from UBL to prose style, but this is something I'd like to discuss further. I will comment again here shortly (I'm working from home, and wiki-ing at the same time, hence excuse me if I take a bit to reply).
Cheers, Rehman 07:48, 30 March 2020 (UTC)[reply]
@Rehman: I'm afraid I don't understand why these (non-redundant, IMO) parameters need to be deleted before Wikidata is used as a fallback. Could you explain? — hike395 (talk) 08:10, 30 March 2020 (UTC)[reply]
To put is simply, to make the code as neat and simple as possible for future editors. Those parameters would not only make my work now many times harder, but would also discourage future editors to attempt improving the template, because the code would look so much more complicated with those parameters. Rehman 08:36, 30 March 2020 (UTC)[reply]
That does not sounds like a necessity to me: that sounds like a preference. Let's separate the two discussions. — hike395 (talk) 14:33, 30 March 2020 (UTC)[reply]
Idea: It just occurred to me that I can eliminate a lot of the code complexity in the infobox by writing a small amount of Lua to parse multiple arguments (like city_*) and produce the {{collapsible list}}/{{enum}} hybrid. This should answer Rehman's objection to having these parameters, and make other editors (like RedWolf) happy also. I'll do that! — hike395 (talk) 14:52, 30 March 2020 (UTC)[reply]
(replying while at work) I think it is better we not use custom lua, for the sake of keeping the code neat and understandable for future editors, with the only lua used being WikidataIB - which is already a standard. For conversation sake, why do we need to keep those repetitive parameters? IMHO, it is unsustainable and quite manual, not to mention bloating articles that use it. For instance, if a new article need another slot for say - city, do we just keep creating?
Following standard infobox usages, I strongly suggest we take the sustainable/neater approach, and use one parameter per criteria. The articles which has so many to list in that (a very small subset), can easily add br tags or use the more standard UBL lists. The vast majority in the thousands or tens of thousands, don't need those repetitive parameters. Thoughts welcome. Rehman 15:23, 30 March 2020 (UTC)[reply]
Pinging the next 10 active talkpage participants to get more opinions: User:Droll, User:RedWolf, User:Volcanoguy, User:Pigsonthewing, User:MSGJ, User:Frietjes, User:Chris.urs-o, User:Bermicourt, User:Wwoods, User:Britishfinance. Participation is of course, voluntary. :) Rehman 15:37, 30 March 2020 (UTC)[reply]
Hike395, following my previous reply, I got some interesting stats. Less than 1% of all articles using this infobox, use the first parameter of the series:
  • border1 = 146 (0.60%)
  • city1 = 73 (0.30%)
  • country1 = 234 (0.96%)
  • district1 = 82 (0.34%)
  • geology1= 86 (0.35%)
  • part1 = 45 (0.18%)
  • period1 = 43 (0.18%)
  • region1 = 292 (1.20%)
  • settlement1 = 4 (0.02%)
  • state1 = 214 (0.88%)
Further in the series (2, 3, etc) are almost 0% - used by only a very tiny amount of articles. Hence, I strongly feel we should not bloat the template just for those handful of uses. Rehman 06:02, 31 March 2020 (UTC)[reply]
@Rehman: Even if small in number, those are high-traffic pages, like Himalayas, Appalachian Mountains, Alps, Andes, and Catskill Mountains. When I finish my Lua, it won't be bloated. One field will look like {{#invoke:Compact list|main|border}}, and that's it. The Lua should be simple, too. Give me a couple of days to put it together. Thanks! — hike395 (talk) 02:32, 1 April 2020 (UTC)[reply]
Hike395, high-traffic pages should have no issue using a single parameter, and only indicates that more experienced editors would be around - so using UBLs or br tags would be more convenient on those pages. There would also be no visual difference anyway. Can we please gain consensus before we add custom Lua to this template? Considering that this infobox is quite straightforward (i.e. no complicated functions or features), I don't see a need to add separate Lua code, considering that it only complicates things for future editors. Let me crosspost this discussion on WP:Mountains and see if we can get more opinions on this. Cheers, Rehman 03:35, 1 April 2020 (UTC)[reply]
@Rehman: Don't worry, I won't make any changes to the live template without discussion. Conversely, if you want to change the automated use of collapsible lists and enum to manually-specified UBL, I would also bring that up at WT:MOUNTAINS. But if you could please wait until I have a prototype, because it may change the outcome of the discussion. — hike395 (talk) 04:18, 1 April 2020 (UTC)[reply]

Automatic conversion

I'm not an expert on template coding but as a user, I want templates to be simple to understand and use. In particular, I don't want to have to manually add 'convert' templates. Also I suspect the shims used to very neatly convert infoboxes from other Wikis rely on being able to point e.g. at the elevation in metres (which is what everyone bar a couple of English-speaking countries use, otherwise we wouldn't need conversion at all). What I have spotted though is that the copy and past template example in the documentation (under Standard Usage) omits several of the parameters anyway and that certainly needs fixing. Finally I would say please don't roll out major changes without giving editors a chance to view, trial and comment on the changes first. This recently happened with Template:Sfn and that caused a huge number of spurious red links and lots of angry responses. Bermicourt (talk) 18:32, 30 March 2020 (UTC)[reply]

Hi Bermicourt. Thanks for the feedback!
  • For simplicity, the vast number of infoboxes simply let the user decide what unit to use, or if to use convert at all (random example: Template:Infobox comet). But that being said, would adding the convert code within the copy-paste section help (like Template:Infobox docks)? This saves editors the time in looking for the code.
  • The documentation has already been outdated for quite some time. The page is currently undergoing revamping, so you should be able to see the updated code soon :)
Looking forward to your comments. Rehman 04:12, 31 March 2020 (UTC)[reply]
@Rehman: I think that the automatic conversions should not be removed. When the infobox is supplied with dimensions for a mountain or range, it automatically figures out the correct scaling for the geohack map (via a call to {{infobox dim}}). Check out the code at |data9= in the infobox. If we delete |length_*=, |width_*=, or |area_*=, we lose this nice feature. — hike395 (talk) 06:19, 8 April 2020 (UTC)[reply]
{{Coordinates}} already has a default feature type:mountain used for scaling, which is for peaks, mountain ranges, hills, submerged reefs, and seamounts, and takes away the need of that chunk of code and complexity, and standardises all geohack maps even if length or width is not provided. For some reason, this infobox has implemented an override, which is now used by 1.3% of articles, or a maximum of 328 articles only. Is there a particular problem we are trying to solve by overriding the default? Rehman 04:50, 9 April 2020 (UTC)[reply]

Wikidata

I'm sorry, Rehman, but we also need to discuss the implementation of Wikidata fallback.

In general, I'm a big fan of Wikidata fallbacks --- it allows centralized data to be shared consistently across wikis from all languages. That is a good thing. But, one issue that has come up in other Wikidata discussions: most editors do not know how or where to edit the data that comes from Wikidata. When you use default Wikidata, you should supply an editing icon that allows editors to change the field. Such an editing icon is created using the {{EditOnWikidata}} template. Could you add this template to the infobox when Wikidata default data is used?

One more comment and a question:

  • The downside of using Wikidata is that the watchlist of an en user does not get updated when data is changed. I've seen ignorant users change the elevation of mountains to be the "well-known" (but inaccurate) values. For example, many people believe that Mount Whitney is 14494', but it is really 14505' according to the latest measurements. I guess we'll need to keep common mountain data on en in order to keep an eye on it.
  • The new code that you've added to the infobox defines |qid=, |fetchwikidata=, |suppressfields=, and |onlysourced=. Would you like to explain what they do? — hike395 (talk) 08:08, 30 March 2020 (UTC)[reply]
No need to apologise; discussions make things clearer for everyone :) I'm glad you started this thread. Yes, "edit on wikidata" link will be shown. This is an example of a fully Wikidata supported infobox. And this is a random example of an article that uses Wikidata - notice the edit link.
Wikidata edits does of course show up on the local watchlist. That is, as long as the local article is on your watchlist. Have a look at Special:Preferences#mw-prefsection-watchlist, and tick "Show Wikidata edits in your watchlist". That should be default if I'm not mistaken.
The additional wikidata parameters are not directly relating to mountains, but rather are for more advanced testing and use cases. Those parameters are present in almost all Wikidata-supported infoboxes, and should not cause any issue.
Hope I answered your questions? Rehman 08:36, 30 March 2020 (UTC)[reply]
@Rehman: That's a great tip for the "Show Wikidata edits in your watchlist" preference! I didn't know about it, and it was off for me (probably because my preferences are very old). That makes Wikidata fallbacks even better!
The new parameters are not well-explained --- do we need them? I know we need |qid= for testing, do we need the other ones, or can they be removed? Each call to Module:WikidataIB has a very large number of parameters. Are these parameters changing the default? Can we eliminate many of them?
Adding Wikidata edit links will change the functionality and layout of the infobox. Can you finish implementing the proposal in the sandbox so that other editors can see what it entails? Can you add the Wikidata edit links (which adds icons)? Otherwise, we don't know what we're supporting or objecting to. — hike395 (talk) 14:47, 30 March 2020 (UTC)[reply]
Hike395: Glad it worked :) I went ahead and added the Wikidata edit link to the sandbox. What do you think? Regarding the new parameters, those are standardised params supported from WikidataIB mainly for user's preference (it is not something only created in this infobox). I personally prefer to keep it simple and to remove them, but there has been cases where issues were raised due to no way for suppressing certain calls or due to sourcing issues. Hence these standard calls are maintained for all Wikidata-based infoboxes (you'd see them in other wd infoboxes too, sometimes not necessarily documented). Rehman 03:54, 31 March 2020 (UTC)[reply]

@Rehman: The last RfC to discuss Wikidata use in infoboxes was contentious: there was a lot of concern about the accuracy of the information in Wikidata. I think we need to follow the lead of {{Infobox telescope}} and allow editors to edit each field, individually. I think most Wikipedia editors and readers don't know how to navigate Wikidata: the pen icon will help anyone to edit Wikidata (which supports the "the encyclopedia anyone can edit" philosophy).

Now, I think the icons may make the infobox look crowded. I will create a sandbox version that /only/ implements the Wikidata change, and we can look at it and decide. I will not implement any of the other parameter changes, yet. I believe we should have one discussion at a time, because all of these issues are separable. — hike395 (talk) 06:44, 8 April 2020 (UTC)[reply]