hrntsm/Tunny

View on GitHub

Showing 55 of 128 total issues

File Algorithm.cs has 609 lines of code (exceeds 250 allowed). Consider refactoring.
Open

using System;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text;
Severity: Major
Found in Tunny/Solver/Algorithm.cs - About 1 day to fix

    File Storage.cs has 600 lines of code (exceeds 250 allowed). Consider refactoring.
    Open

    using System;
    using System.Collections.Generic;
    using System.Data.SQLite;
    using System.IO;
    using System.Linq;
    Severity: Major
    Found in Optuna/Storage/RDB/Storage.cs - About 1 day to fix

      File GrasshopperInOut.cs has 511 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      using System;
      using System.Collections.Generic;
      using System.Globalization;
      using System.Linq;
      using System.Windows.Forms;
      Severity: Major
      Found in Tunny/Util/GrasshopperInOut.cs - About 1 day to fix

        Class OptimizationWindow has 36 methods (exceeds 20 allowed). Consider refactoring.
        Open

            public partial class OptimizationWindow
            {
                private void TpeDefaultButton_Click(object sender, EventArgs e)
                {
                    TLog.MethodStart();
        Severity: Minor
        Found in Tunny/UI/Tab/SettingsTab.cs - About 4 hrs to fix

          File Storage.cs has 352 lines of code (exceeds 250 allowed). Consider refactoring.
          Open

          using System;
          using System.Collections.Generic;
          using System.IO;
          using System.Linq;
          
          
          Severity: Minor
          Found in Optuna/Storage/Journal/Storage.cs - About 4 hrs to fix

            Class SqliteStorage has 32 methods (exceeds 20 allowed). Consider refactoring.
            Open

                public class SqliteStorage : IOptunaStorage
                {
                    private readonly Dictionary<int, Study.Study> _studies = new Dictionary<int, Study.Study>();
                    private readonly SQLiteConnectionStringBuilder _sqliteConnection;
                    private int _nextStudyId;
            Severity: Minor
            Found in Optuna/Storage/RDB/Storage.cs - About 4 hrs to fix

              File SettingsTab.cs has 321 lines of code (exceeds 250 allowed). Consider refactoring.
              Open

              using System;
              
              using Optuna.Sampler;
              
              using Serilog.Events;
              Severity: Minor
              Found in Tunny/UI/Tab/SettingsTab.cs - About 3 hrs to fix

                Class JournalStorage has 30 methods (exceeds 20 allowed). Consider refactoring.
                Open

                    public class JournalStorage : IOptunaStorage
                    {
                        private readonly Dictionary<int, Study.Study> _studies = new Dictionary<int, Study.Study>();
                        private int _nextStudyId;
                        private int _trialId;
                Severity: Minor
                Found in Optuna/Storage/Journal/Storage.cs - About 3 hrs to fix

                  Method CreateBaseTables has 90 lines of code (exceeds 25 allowed). Consider refactoring.
                  Open

                          private void CreateBaseTables()
                          {
                              var commands = new StringBuilder();
                              commands.Append("CREATE TABLE IF NOT EXISTS alembic_version(");
                              commands.Append("   version_num VARCHAR(32) PRIMARY KEY NOT NULL");
                  Severity: Major
                  Found in Optuna/Storage/RDB/Storage.cs - About 3 hrs to fix

                    Class IOptunaStorage has 28 methods (exceeds 20 allowed). Consider refactoring.
                    Open

                        public interface IOptunaStorage
                        {
                            // Basic study manipulation
                            int CreateNewStudy(StudyDirection[] studyDirections, string studyName);
                            void DeleteStudy(int studyId);
                    Severity: Minor
                    Found in Optuna/Storage/IOptunaStorage.cs - About 3 hrs to fix

                      Method RunSingleOptimizeStep has 79 lines of code (exceeds 25 allowed). Consider refactoring.
                      Open

                              private TrialGrasshopperItems RunSingleOptimizeStep(OptimizationHandlingInfo optInfo, Parameter[] parameter, int trialNum, DateTime startTime)
                              {
                                  TLog.MethodStart();
                                  dynamic trial;
                                  int progress = trialNum * 100 / optInfo.NTrials;
                      Severity: Major
                      Found in Tunny/Solver/Algorithm.cs - About 3 hrs to fix

                        Class Algorithm has 26 methods (exceeds 20 allowed). Consider refactoring.
                        Open

                            public class Algorithm : PythonInit
                            {
                                public Parameter[] OptimalParameters { get; private set; }
                                public EndState EndState { get; private set; }
                        
                        
                        Severity: Minor
                        Found in Tunny/Solver/Algorithm.cs - About 3 hrs to fix

                          Class GrasshopperInOut has 25 methods (exceeds 20 allowed). Consider refactoring.
                          Open

                              public class GrasshopperInOut
                              {
                                  private readonly GH_Document _document;
                                  private readonly List<Guid> _inputGuids;
                                  private readonly GH_Component _component;
                          Severity: Minor
                          Found in Tunny/Util/GrasshopperInOut.cs - About 2 hrs to fix

                            Method RunSingleOptimizeStep has a Cognitive Complexity of 32 (exceeds 20 allowed). Consider refactoring.
                            Open

                                    private TrialGrasshopperItems RunSingleOptimizeStep(OptimizationHandlingInfo optInfo, Parameter[] parameter, int trialNum, DateTime startTime)
                                    {
                                        TLog.MethodStart();
                                        dynamic trial;
                                        int progress = trialNum * 100 / optInfo.NTrials;
                            Severity: Minor
                            Found in Tunny/Solver/Algorithm.cs - About 2 hrs to fix

                            Cognitive Complexity

                            Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                            A method's cognitive complexity is based on a few simple rules:

                            • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                            • Code is considered more complex for each "break in the linear flow of the code"
                            • Code is considered more complex when "flow breaking structures are nested"

                            Further reading

                            Method SolveInstance has 54 lines of code (exceeds 25 allowed). Consider refactoring.
                            Open

                                    protected override void SolveInstance(IGH_DataAccess DA)
                                    {
                                        CheckVariablesInput(Params.Input[0].Sources.Select(ghParam => ghParam.InstanceGuid));
                                        CheckObjectivesInput(Params.Input[1].Sources.Select(ghParam => ghParam.InstanceGuid));
                                        CheckArtifactsInput(Params.Input[3].Sources.Select(ghParam => ghParam.InstanceGuid));
                            Severity: Major
                            Found in Tunny/Component/Optimizer/BoneFishComponent.cs - About 2 hrs to fix

                              Method GetTrial has 46 lines of code (exceeds 25 allowed). Consider refactoring.
                              Open

                                      public Trial.Trial GetTrial(int trialId)
                                      {
                                          var trial = new Trial.Trial();
                                          using (var connection = new SQLiteConnection(_sqliteConnection.ToString()))
                                          {
                              Severity: Minor
                              Found in Optuna/Storage/RDB/Storage.cs - About 1 hr to fix

                                Method Solve has 44 lines of code (exceeds 25 allowed). Consider refactoring.
                                Open

                                        public void Solve()
                                        {
                                            TLog.MethodStart();
                                            EndState = EndState.Error;
                                            OptimizeLoop.IsForcedStopOptimize = false;
                                Severity: Minor
                                Found in Tunny/Solver/Algorithm.cs - About 1 hr to fix

                                  Method SetInputSliderValues has 44 lines of code (exceeds 25 allowed). Consider refactoring.
                                  Open

                                          private void SetInputSliderValues(List<VariableBase> variables)
                                          {
                                              TLog.MethodStart();
                                              int i = 0;
                                  
                                  
                                  Severity: Minor
                                  Found in Tunny/Util/GrasshopperInOut.cs - About 1 hr to fix

                                    Method CreateNewStudy has 43 lines of code (exceeds 25 allowed). Consider refactoring.
                                    Open

                                            public int CreateNewStudy(StudyDirection[] studyDirections, string studyName)
                                            {
                                                long maxLength;
                                                using (var connection = new SQLiteConnection(_sqliteConnection.ToString()))
                                                {
                                    Severity: Minor
                                    Found in Optuna/Storage/RDB/Storage.cs - About 1 hr to fix

                                      Method StorageLoadTest has 43 lines of code (exceeds 25 allowed). Consider refactoring.
                                      Open

                                              [Theory]
                                              [InlineData(@"TestFile/sqlite.db", "sqlite")]
                                              [InlineData(@"TestFile/journal.log", "log")]
                                              public void StorageLoadTest(string path, string type)
                                              {
                                      Severity: Minor
                                      Found in OptunaTests/Storage/StorageSameBevaviorTests.cs - About 1 hr to fix
                                        Severity
                                        Category
                                        Status
                                        Source
                                        Language