plotly / plotly/Plotly.NET

Add suitable default open statement for FreeBSD

Open
#397 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Area: Meta Area: MissingAbstraction help wanted Type: Enhancement
Dominant language
F#
Stars
860
Forks
99
Avg merge
22m
Merged PRs (30d)
1

Description

Chart.show uses an OS-specific way of opening files:

https://github.com/plotly/Plotly.NET/blob/2f94e879d23b0bdd259ec76cff99ae8946b375b2/src/Plotly.NET/InternalUtils.fs#L93-L105

Since we do not have a match case for FreeBSD, you currently cannot display charts via Chart.show on that OS.

Since i am not familiar with that OS, I would need input from someone that actually uses it on how to proceed here.

My first google attempt comes up with this, but i have no way of testing it.

See also #384

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in src/Plotly.NET/InternalUtils.fs at lines 93-105 and inspect the OS-specific logic used by Chart.show. Research the appropriate FreeBSD open command, then add the FreeBSD case and verify that Chart.show can display charts on that OS; testing may require FreeBSD access.

Written by the indexing model from the issue text.

Assessment

Tech stack
fsharp
Domain
operating-systems
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.