ChartsOrg / ChartsOrg/Charts

Assertion Error after an Upgrade to XCode Version 14.2 and Charts Master

Open
#4,995 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
Swift
Stars
28k
Forks
6k
PR merge metrics
No merged PRs in 30d

Description

> ℹ Please fill out this template when filing an issue.
> All lines beginning with an ℹ symbol instruct you with what info we expect. You can delete those lines once you've filled in the info.
>
> Per our [*CONTRIBUTING guidelines](https://github.com/jjatie/Charts/blob/master/CONTRIBUTING.md), we use GitHub for
> bugs and feature requests, not general support. Other issues should be opened on Stack Overflow with the tag [`ios-charts`.](https://stackoverflow.com/questions/tagged/ios-charts)
>
> Please remove this line and everything above it before submitting.

* [x] I've read, understood, and done my best to follow the [*CONTRIBUTING guidelines](https://github.com/jjatie/Charts/blob/master/CONTRIBUTING.md).

## What did you do?

Upgraded XCode to Version 14.2 from 13 and Charts to branch->Master from older version 3.6

## What did you expect to happen?

I expected my app to run without any error and show chart as it was doing before these upgrades

## What happened instead?

XCode project was built successfully after I made some code changes as per #4860 (and made few other minor changes like changing addDataSet to append)
However, after the build, the simulator gets stuck and XCode throws following error after launching

Assertion failed: removeAll(keepingCapacity:) not supported for CombinedData

Error at :
Script: ChartData.swift
function: removeAll
LOC: assert(!(self is CombinedChartData), "\(#function) not supported for CombinedData")

## Charts Environment

**Charts version/Branch/Commit Number: Branch Master
**Xcode version: 14.2
**Swift version: Swift 5
**Platform(s) running Charts: XCode Simulator iPhone 14 pro (iOS 16.2)
**macOS version running Xcode: Ventura version 13.1 (22C65)

## Demo Project

ℹ Please link to or upload a project we can download that reproduces the issue.

Contributor guide

Open the contributing guide

Research direction

Start with ChartData.swift and the removeAll assertion for CombinedChartData, then review the code changes required by #4860 and the reported addDataSet-to-append changes. Reproduce the failure using Charts master with Xcode 14.2, Swift 5, and the iPhone 14 Pro simulator; done means the simulator launches without this assertion and the chart displays as before.

Written by the indexing model from the issue text.

Assessment

Tech stack
swift
Domain
data-visualization
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.