search

oasis-open / cti-stix-slider

OASIS TC Open Repository: The repository cti-stix-slider supports development of a Python application to convert STIX 2.0 content to STIX 1.x content
https://cti-stix-slider.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
21 stars 15 forks source link

Update convert_stix.py to include cache_clear() #27

Closed emmanvg closed 6 years ago

emmanvg commented 6 years ago

Add cybox.utils.caches.cache_clear() to prevent RAM consumption issue.

rpiazza commented 6 years ago

Was there a reason why in the elevator you have 3 calls to this method, but in the slide, just one?

emmanvg commented 6 years ago

We should probably do the same in the elevator.

rpiazza commented 6 years ago

If you want to - I was just wondering if something was different.

codecov-io commented 6 years ago

Codecov Report

Merging #27 into master will increase coverage by 0.03%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   64.62%   64.65%   +0.03%     
==========================================
  Files          16       16              
  Lines        5029     5031       +2     
==========================================
+ Hits         3250     3253       +3     
+ Misses       1779     1778       -1
Impacted Files Coverage Δ
stix2slider/convert_stix.py 65.86% <100%> (+0.09%) :arrow_up:
stix2slider/convert_cyber_observables.py 73.65% <0%> (+0.14%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f3d064f...ff5f0fd. Read the comment docs.