VolumeProfile and TimeProfile hang when the value area volume percentage is 1
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 82/100
Línea de trabajo
Empieza en Indicators/MarketProfile.cs en MarketProfile.CalculateValueArea, donde el bucle expande los índices low/high mientras el volumen del área de valor es menor o igual que el objetivo; con un porcentaje de 1, el objetivo es igual al volumen total, así que ambos índices se fijan en los extremos y el bucle nunca termina. Añade una salida cuando ni el índice low ni el high puedan moverse, lo que también corrige TimeProfile ya que comparte la clase base. Hecho significa que una prueba de regresión que construya new VolumeProfile(2, 1m) y llame a Update una vez retorne, con el área de valor abarcando todos los niveles de precio.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Expected Behavior
new VolumeProfile(2, 1m) (a value area that holds 100% of the volume) updates normally and reports a value area that spans the whole profile.
Actual Behavior
The first Update call never returns. MarketProfile.CalculateValueArea keeps expanding the value area while its volume is less than or equal to the target. With a percentage of 1, the target equals the total volume, so the condition stays true after the area covers every price level. Both neighbor indexes are clamped to the edges and the loop runs forever. TimeProfile has the same problem because it shares the base class.
Potential Solution
In Indicators/MarketProfile.cs, CalculateValueArea, exit the loop when neither the low nor the high index can move.
Reproducing the Problem
var vp = new VolumeProfile(2, 1m);
vp.Update(new TradeBar { Symbol = Symbols.AAPL, Close = 10, Volume = 100, Time = new DateTime(2000, 1, 2) });
// never returns
System Information
Ubuntu x86_64, .NET SDK 10.0.403, master at 22404ea28b8d3f3c9eb74d6ed9f0444426f8e417.
Checklist
- I have completely filled out this template
- I have confirmed that this issue exists on the current
masterbranch - I have confirmed that this is not a duplicate issue by searching issues
- I have provided detailed steps to reproduce the issue
- Lenguaje dominante
- C#
- Estrellas
- 22.1k
- Forks
- 5.3k
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 32
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de QuantConnect/Lean
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
QuantConnect/Lean#9955 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
QuantConnect/Lean#9953 ·
Los mantenedores suelen responder en 1 día
-
WebullFeeModel charges SPXW options the SPX exchange feePosiblemente ocupada @Amadeus-22 la tomó hace 9 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
QuantConnect/Lean#9922 ·
Los mantenedores suelen responder en 1 día
-
TradeStationBrokerageModel rejects extended-hours limit orders on equity optionsPosiblemente ocupada @AlexCatarino la tomó hace 8 días. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
QuantConnect/Lean#9920 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
AlpacaBrokerageModel refuses extended-hours limit orders with the GTC time in force, which Alpaca acceptsPosiblemente ocupada @AlexCatarino la tomó hace 16 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
QuantConnect/Lean#9892 ·
Los mantenedores suelen responder en 1 día
Todos los issues de QuantConnect/Lean
Issues similares
-
Importer: an empty or non-numeric Error cell is imported as 0 instead of missingPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Open-Systems-Pharmacology/OSPSuite.Core#3049 ·
Los mantenedores suelen responder en 1 día
-
needs attention public-client untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
AzureAD/microsoft-authentication-library-for-dotnet#6242 ·
Los mantenedores suelen responder en 2 días
-
docs: HTML config transform example sets advancedCustomization at a path the UI does not readAbierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 90/100
criticalmanufacturing/cli#882 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Dome-Works/DOME#129 ·
Los mantenedores suelen responder en 1 día
-
bug sev:L
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Systemorph/MeshWeaver#7480 ·
Los mantenedores suelen responder en 1 día