Integrating methanol with Direct Ocean Capture CO2 by jmartin4u · Pull Request #239 · NatLabRockies/H2Integrate

jmartin4u · 2025-09-04T21:21:39Z

Integrating methanol with Direct Ocean Capture CO2

This sets up a new example in which the existing direct ocean capture tech is used to send CO2 to a CO2 hydrogenation methanol production plant located offshore on the Gulf. This plant also uses hydrogen from electrolysis powered by offshore wind with H2 storage and dispatch, tying a bunch of tech nicely together. The current example uses 312 MW of electrolysis (~ the size found feasible in the ECO project) combined with 2000 tons of H2 storage to keep a 100,000 tpy methanol plant fullyk supplied with the H2 it needs for most of the year, with some shortfall in the summer months when wind is lower. Currently there is not a CO2 storage and dispatch (at least not a working one - @kbrunik is there something we could implement from the DOC model?), so the methanol plant does not operate at full capacity.

Type of Contribution

General PR Checklist

CHANGELOG.md has been updated to describe the changes made in this PR
Documentation
- Docstrings are up-to-date
- Related docs/ files are up-to-date, or added when necessary
- Documentation has been rebuilt successfully
- Examples have been updated (if applicable)
Tests pass (If not, and this is expected, please elaborate in the tests section)
Added tests for new functionality or bug fixes
PR description thoroughly describes the new feature, bug fix, etc.

Impacted areas of the software

examples/03_methanol/co2_hydrogenation_doc: Added this example of DOC integrating with methanol, particular files of interest:
- examples/03_methanol/co2_hydrogenation_doc/run_co2h_methanol.py: Script to run this example
- examples/03_methanol/co2_hydrogenation_doc/plot_co2h_methanol.py: A plotting function imported into the above script which draws out all the material flows going into the methanol plant
converters/co2/marine/direct_ocean_capture.py: Fleshed out docstrings, added a co2_out output for interconnection
converters/co2/marine/marine_carbon_capture_baseclass.py: Added a co2_out output for interconnection
converters/methanol/methanol_baseclass.py: Added a total_methanol_produced output for ProFast finance calculations
converters/methanol/smr_methanol_plant.py: Added a total_methanol_produced output and corrected Opex calc for ProFast finance calculations
converters/methanol/co2h_methanol_plant.py: Added a total_methanol_produced output and corrected Opex calc for ProFast finance calculations
core/h2integrate_model.py: Removed methanol from lists of techs not to include in ProFast financial calculations

Additional supporting information

Test results, if applicable

review-notebook-app · 2025-09-05T18:50:46Z

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

jmartin4u · 2025-09-05T20:35:12Z

@johnjasa and @kbrunik - I built in a test to run my new integrated DOC/methanol example and it is passing locally. But on GitHub we're getting this error with the new test. Any reason why this error would pop up in my integrated test when it doesn't in the standalone DOC test? Idk how to debug since the test passes locally.

ImportError: The `mcm` package is required to use the Direct Ocean Capture model. Install it via:
 pip install git+https://github.com/NREL/MarineCarbonManagement.git

jmartin4u · 2025-09-12T15:28:05Z

@johnjasa and @kbrunik - I built in a test to run my new integrated DOC/methanol example and it is passing locally. But on GitHub we're getting this error with the new test. Any reason why this error would pop up in my integrated test when it doesn't in the standalone DOC test? Idk how to debug since the test passes locally.
ImportError: The `mcm` package is required to use the Direct Ocean Capture model. Install it via:
 pip install git+https://github.com/NREL/MarineCarbonManagement.git

I realized there was a line added before all the other tests with mcm:

@unittest.skipUnless(importlib.util.find_spec("mcm") is not None, "mcm is not installed")

Now tests are passing both locally (with mcm) and on GitHub (without mcm)

kbrunik · 2025-09-17T21:04:57Z

@jmartin4nrel we can use the feature add from #248 for CO2 storage, it just got added to develop so it should be good to go

kbrunik

Thanks for making this Jonathan! It's a really cool extension of H2I's features with multiple flows combining into a single fuel. I'd love to see a few of the more recent feature adds in H2I get utilized in this PR and we can talk today about it. The other thing would be making the example a little more realistic by moving it near the coast or fully offshore. I can certainly help with some of these requested changes

kbrunik · 2025-09-23T12:15:14Z

-      commodity: "methanol"
-      finance_groups: ["methanol"]
-      technologies: ["methanol"]
+    # methanol:


@elenya-grant this is the finance handling issue I'm talking about

Still relevant?

Yes haven't figured out why this is happening yet

johnjasa

This looks nice to me! I really appreciate your example additions. The plotting method you added is real slick, too, that'll be quite useful for folks looking for more examples of how to postprocess H2I runs.

I pushed some small changes directly to the branch based on a comment from @kbrunik. Only last remaining question is about those two potentially extraneous wind resource files that Kaitlin noted. I won't request changes here so as to not hold up the PR!

johnjasa · 2025-10-07T02:52:48Z

-      commodity: "methanol"
-      finance_groups: ["methanol"]
-      technologies: ["methanol"]
+    # methanol:


Still relevant?

johnjasa · 2025-10-07T03:35:08Z

+from h2integrate.core.model_baseclasses import CostModelBaseClass
+
+
+class CO2TankPerformanceModel(om.ExplicitComponent):


Noting this for the future, no need to change in this PR. But for the next tank class we add to H2I we should genericize. I made an issue here: #292

removed this since it wasn't used and generic storage was in the example

kbrunik

I think it looks good to go! Thanks @jmartin4nrel for all your work on this effort

jmartin4u added 10 commits August 18, 2025 10:59

Example set up to use doc co2, need to updated mcm locally

e160884

Running with co2 connection (not correctly)

b564188

Running example with plots

4ea37bf

Making tighter plots

e94bb3d

Plot colors

f791fa7

Making new folder for methanol doc example

64680e8

Repopulating co2h example

e4c1a9b

Methanol doc with financials

2d953eb

Merge remote-tracking branch 'upstream/develop' into meoh-dev2

b0b6681

Config updates from merge

b98d80e

jmartin4u mentioned this pull request Sep 4, 2025

Making pipe and cable substance-agnostic #241

Merged

15 tasks

jmartin4u added 3 commits September 5, 2025 09:28

Merge remote-tracking branch 'upstream/develop' into meoh-dev2

379a328

Getting tests passing

7467b99

Example chart set up

087c6a4

Changelog update

ad69753

jmartin4u requested a review from kbrunik September 5, 2025 19:12

jmartin4u added 2 commits September 5, 2025 13:32

Tests fixes

b131e0d

Fix test

cd09412

Adding line to skip mcm test if not installed

a020f4d

jmartin4u marked this pull request as ready for review September 12, 2025 15:29

Merge branch 'develop' into meoh-dev2

bce8387

kbrunik requested changes Sep 19, 2025

View reviewed changes

kbrunik added 2 commits September 23, 2025 07:02

merge develop

ba0b2e0

update comments

917d3de

kbrunik reviewed Sep 23, 2025

View reviewed changes

johnjasa added the medium priority label Sep 23, 2025

jmartin4u and others added 13 commits September 24, 2025 11:11

Merge remote-tracking branch 'upstream/develop' into meoh-dev3

20408c5

Splitting electricity

a0a67ab

Merge branch 'meoh-dev3' into meoh-dev2

7046939

CO2 storage working, need to check timeseries

e156ace

Remove wind file

6c4b489

Reasonable asset sizing

fccfc8a

Merge remote-tracking branch 'upstream/develop' into develop

1b7baec

Merge branch 'develop' into meoh-dev2

ee0b2b0

Optimized

80dda6a

Test fix

fe8c3b7

Fix precommit

d67adfa

Merge remote-tracking branch 'upstream/develop' into meoh-dev2

16c92eb

Removed co2 capture rate mt

1eac56d

johnjasa approved these changes Oct 7, 2025

View reviewed changes

kbrunik and others added 3 commits October 7, 2025 09:04

minor clean up

bbe1528

minor clean up

c7f279d

Merge branch 'develop' into meoh-dev2

cbdf07d

kbrunik approved these changes Oct 7, 2025

View reviewed changes

Removing commented out config block

539b5d8

jmartin4u enabled auto-merge October 7, 2025 17:17

jmartin4u merged commit e950dd8 into NatLabRockies:develop Oct 7, 2025
5 checks passed

jmartin4u deleted the meoh-dev2 branch October 16, 2025 21:31

		from h2integrate.core.model_baseclasses import CostModelBaseClass


		class CO2TankPerformanceModel(om.ExplicitComponent):

Conversation

jmartin4u commented Sep 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Integrating methanol with Direct Ocean Capture CO2

Type of Contribution

General PR Checklist

Impacted areas of the software

Additional supporting information

Test results, if applicable

Uh oh!

review-notebook-app Bot commented Sep 5, 2025

Uh oh!

jmartin4u commented Sep 5, 2025

Uh oh!

jmartin4u commented Sep 12, 2025

Uh oh!

kbrunik commented Sep 17, 2025

Uh oh!

kbrunik left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kbrunik Sep 23, 2025

Choose a reason for hiding this comment

Uh oh!

johnjasa Oct 7, 2025

Choose a reason for hiding this comment

Uh oh!

kbrunik Oct 7, 2025

Choose a reason for hiding this comment

Uh oh!

johnjasa left a comment

Choose a reason for hiding this comment

Uh oh!

johnjasa Oct 7, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

johnjasa Oct 7, 2025

Choose a reason for hiding this comment

Uh oh!

kbrunik Oct 7, 2025

Choose a reason for hiding this comment

Uh oh!

kbrunik left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jmartin4u commented Sep 4, 2025 •

edited

Loading