Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error frequency chart displaying dates into the future #51

Closed
arowla opened this issue Nov 29, 2010 · 3 comments
Closed

Error frequency chart displaying dates into the future #51

arowla opened this issue Nov 29, 2010 · 3 comments

Comments

@arowla
Copy link

arowla commented Nov 29, 2010

I have a Sentry installation which is working just fine, except that the error frequency chart is showing dates until Dec. 18th (today is 11/29). The max() value of datetime in the Sentry message table in my DB is 11/29, so it seems to be a Highcharts/Javascript display issue. Not sure what other information to include other than maybe the code that's generating the chart:

                <script type="text/javascript"> 
                $(document).ready(function() {
                    new Highcharts.Chart({
                        chart: {
                            renderTo: 'chart',
                            zoomType: 'x',
                            marginLeft: 50,
                            marginRight: 20,
                            marginTop: 20,
                            marginBottom: 40,
                            backgroundColor: '#eeeeee',
                            showAxes: false,
                            height: 200
                        },
                        title: {
                            text: false
                        },
                        xAxis: {
                            type: 'datetime',
                            maxZoom: 24 * 3600000, // 1 day
                            title: {
                                text: false
                            },
                            showFirstLabel: false,
                            dateTimeLabelFormats: {
                                second: '%H:%M:%S',
                                minute: '%H:%M',
                                hour: '%d %b %H:%M',
                                day: '%e. %b',
                                week: '%e. %b',
                                month: '%b \'%y',
                                year: '%Y'
                            }
                        },
                        yAxis: {
                            title: {
                                text: false
                            }
                        },
                        tooltip: {
                            formatter: function() {
                                return '<strong>'+
                                    Highcharts.dateFormat('%A %B %e %Y<br/>%I:00 %p', this.x) + '</strong><br/>'+
                                    this.y+' records';
                            }
                        },
                        series: [{
                            pointStart: new Date() - 30 * 3600000,
                            pointInterval: 3600 * 1000,
                            type: 'area',
                            data: [31, 33, 1, 7, 28, 34, 0, 0, 0, 25, 0, 4, 42, 45, 39, 9, 30, 37, 47, 0, 0, 22, 20, 27, 53, 24, 0, 19, 1, 1, 1, 0, 32, 12, 11, 9, 5, 7, 15, 33, 22, 26, 19, 13, 23, 27, 25, 44, 13, 2, 24, 17, 17, 47, 29, 13, 2, 33, 0, 0, 0, 0, 10, 21, 34, 42, 10, 10, 39, 32, 25, 37, 33, 18, 8, 29, 27, 4, 1, 3, 13, 26, 36, 37, 20, 16, 36, 37, 43, 43, 45, 24, 23, 39, 35, 34, 44, 32, 31, 37, 31, 28, 47, 27, 25, 45, 23, 21, 33, 27, 23, 40, 25, 39, 5, 16, 30, 51, 38, 18, 31, 42, 37, 10, 8, 39, 36, 38, 4, 30, 19, 16, 22, 31, 32, 52, 11, 36, 25, 34, 54, 30, 3, 42, 50, 34, 30, 24, 22, 38, 38, 0, 27, 50, 10, 40, 0, 0, 13, 19, 24, 36, 40, 39, 17, 31, 14, 23, 42, 0, 35, 42, 38, 39, 39, 18, 43, 25, 20, 24, 33, 0, 0, 0, 1, 40, 0, 10, 0, 20, 3, 43, 10, 10, 10, 20, 0, 44, 20, 10, 0, 10, 0, 21, 22, 21, 0, 0, 1, 41, 0, 0, 0, 0, 0, 38, 42, 2, 0, 10, 0, 1, 41, 20, 16, 39, 31, 37, 31, 40, 42, 25, 24, 4, 15, 19, 8, 6, 9, 2, 3, 6, 5, 7, 1, 3, 2, 21, 3, 6, 3, 1, 5, 11, 18, 15, 21, 23, 26, 41, 32, 45, 15, 30, 29, 30, 31, 0, 41, 24, 28, 41, 20, 37, 38, 10, 2, 43, 10, 10, 0, 0, 43, 22, 31, 5, 15, 5, 6, 26, 10, 10, 57, 20, 20, 10, 25, 20, 22, 10, 34, 23, 43, 32, 48, 38, 16, 24, 35, 46, 48, 42, 47, 37, 23, 36, 26, 0, 5, 45, 20, 26, 20, 53, 33, 15, 35, 23, 16, 14, 39, 30, 29, 10, 13, 14, 26, 0, 0, 23, 0, 0, 20, 23, 47, 0, 0, 0, 10, 20, 21, 21, 0, 0, 0, 41, 22, 0, 0, 0, 0, 22, 20, 21, 11, 0, 0, 0, 0, 24, 29, 21, 7, 38, 26, 17, 0, 0, 0, 22, 3, 21, 0, 0, 0, 29, 21, 20, 10, 3, 27, 24, 32, 38, 0, 11, 10, 0, 24, 39, 18, 29, 33, 40, 16, 16, 18, 32, 34, 17, 17, 28, 27, 38, 41, 23, 19, 3, 34, 9, 26, 12, 6, 43, 5, 1, 13, 24, 6, 4, 6, 5, 31, 7, 0, 9, 27, 15, 9, 11, 15, 32, 14, 19, 9, 13, 3, 28, 33, 31, 13, 8, 4, 4, 5, 23, 37, 29, 19, 16, 3, 16, 6, 17, 30, 32, 40, 14, 10, 8, 5, 12, 8, 4, 11, 11, 2, 0, 1, 1]
                        }],
                        legend: {
                            enabled: false
                        },
                        plotOptions: {
                            area: {
                                lineWidth: 1,
                                marker: {
                                    enabled: false,
                                    states: {
                                        hover: {
                                            enabled: true,
                                            radius: 5
                                        }
                                    }
                                },
                                shadow: false,
                                states: {
                                    hover: {
                                        lineWidth: 1
                                    }
                                }
                            }
                        },
                    });
                });
                </script>
@dcramer
Copy link
Member

dcramer commented Nov 30, 2010

I have to dig into it, but I have a feeling we're not doing any GMT conversion on the dates

@arowla
Copy link
Author

arowla commented Nov 30, 2010

We thought about timezone issues, too, but couldn't figure out how that would make it show dates ~3 weeks into the future.

@andrewwatts
Copy link
Contributor

This issue could be closed, it was fixed with dcramer/django-sentry@e742722

@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2020
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants