Two of these contributions are part of the tagged release, which lists me under new contributors.
Bu katkılardan ikisi etiketli sürümün parçası; sürüm notları beni yeni katkıcılar arasında listeliyor.
Flight software: adds a parameterLoaded hook that notifies parameterUpdated for each valid or default parameter as it loads
Uçuş yazılımı: her parametre yüklenirken geçerli ya da varsayılan değer için parameterUpdated çağıran bir parameterLoaded hook'u ekler
Code generator (FPP)Kod üreteci (FPP)Flight software: tandem change so the Comp test component accepts valid and default parameters under the new load hook
Uçuş yazılımı: yeni yükleme hook'u altında Comp test bileşeninin geçerli ve varsayılan parametreleri kabul etmesi için tandem değişiklik
C++ test fixC++ test düzeltmesiFlight software: command that packages a file into data products for downlink
Uçuş yazılımı: bir dosyayı yere indirmek için veri ürününe paketleyen komut
C++ featureC++ özellikFlight software: reserved packet descriptor for downlinking parameter values
Uçuş yazılımı: parametre değerlerini yere indirmek için ayrılmış paket tanımlayıcısı
Protocol definitionProtokol tanımıFlight software: reference examples use generated record size macros
Uçuş yazılımı: referans örnekler üretilen kayıt boyutu makrolarını kullanıyor
Example updateÖrnek güncellemeMars operations mapping: mission rename endpoint
Mars operasyon haritalama: görev yeniden adlandırma endpoint'i
Node featureNode özellikMars operations mapping: mission data paths now follow the configured folder
Mars operasyon haritalama: görev veri yolları artık yapılandırılan klasörü takip ediyor
Bug fixHata düzeltmeMars operations mapping: rename action in the mission configuration interface
Mars operasyon haritalama: görev yapılandırma arayüzünde yeniden adlandırma eylemi
React UIReact arayüzSatellite ground station network: support for below sea level altitudes
Uydu yer istasyonu ağı: deniz seviyesi altı rakımlar için destek
Django bugfixDjango hata düzeltmeFlight software: claimed, awaiting CCB design direction
Uçuş yazılımı: talip olundu, CCB tasarım yönü bekleniyor
Code generator (FPP)Kod üreteci (FPP)FileDownlink area (same family as #5561). Opened by Tim Canham. Static internal buffer (m_memoryStore) creates a design question before implementing. CCB approved, now in Backlog, unassigned.
FileDownlink alanı (#5561 ile aynı aile). Tim Canham açtı. Statik dahili buffer (m_memoryStore) uygulamadan önce bir tasarım sorusu yaratıyor. CCB bekleniyor.
Satellite ground station network: support for below sea level altitudes
Uydu yer istasyonu ağı: deniz seviyesi altı rakımlar için destek
Flight software: command that packages a file into data products for downlink
Uçuş yazılımı: bir dosyayı yere indirmek için veri ürününe paketleyen komut
Flight software: reserved packet descriptor for downlinking parameter values
Uçuş yazılımı: parametre değerlerini yere indirmek için ayrılmış paket tanımlayıcısı
Mars operations mapping: mission rename endpoint
Mars operasyon haritalama: görev yeniden adlandırma endpoint'i
Flight software: reference examples use generated record size macros
Uçuş yazılımı: referans örnekler üretilen kayıt boyutu makrolarını kullanıyor
Mars operations mapping: mission data paths now follow the configured folder
Mars operasyon haritalama: görev veri yolları artık yapılandırılan klasörü takip ediyor
Mars operations mapping: rename action in the mission configuration interface
Mars operasyon haritalama: görev yapılandırma arayüzünde yeniden adlandırma eylemi
The release notes list me under new contributors, and the changelog mentions the packet descriptor I reserved. From that point the work was not just on a development branch, it was in a tagged release that missions can adopt.
Sürüm notları beni yeni katkıcılar arasında listeliyor ve changelog ayırdığım paket tanımlayıcısından bahsediyor. O noktadan sonra iş sadece bir geliştirme dalında değil, görevlerin benimseyebileceği etiketli bir sürümdeydi.
Flight software: claimed, awaiting CCB design direction
Uçuş yazılımı: talip olundu, CCB tasarım yönü bekleniyor
Flight software: tandem change so the Comp test component accepts valid and default parameters under the new load hook
Uçuş yazılımı: yeni yükleme hook'u altında Comp test bileşeninin geçerli ve varsayılan parametreleri kabul etmesi için tandem değişiklik
Flight software: adds a parameterLoaded hook that notifies parameterUpdated for each valid or default parameter as it loads
Flight software: adds a parameterLoaded hook that notifies parameterUpdated for each valid or default parameter as it loads
Code generator (FPP)Kod üreteci (FPP)
nasa/fpp #1097, open (implements nasa/fprime#5697). A code generator change in F Prime's FPP toolchain (Scala). It adds a generated parameterLoaded(id, validity) hook that by default calls parameterUpdated when the parameter loaded successfully (valid or default, via FW_PARAM_OK); the hook is invoked from the load path as each parameter is loaded, and parametersLoaded stays a do-nothing notifier that components can still override. The new behavior required a tandem F Prime change (nasa/fprime#5877), now merged, so the Comp test component accepts valid and default. Verified with the Scala unit tests, the fpp-to-cpp reference tests, and a full F Prime integration test (FppTest, 19/19). The issue it implements (nasa/fpp#1098) is on the v3.4.0 milestone.
nasa/fpp #1097, açık (nasa/fprime#5697'yi uyguluyor). F Prime'ın FPP araç zincirinde (Scala) bir kod üreteci değişikliği. Üretilen bir parameterLoaded(id, geçerlilik) hook'u ekliyor; bu hook varsayılan olarak parametre başarıyla yüklendiğinde (geçerli veya varsayılan, FW_PARAM_OK ile) parameterUpdated çağırıyor. Hook, her parametre yüklenirken yükleme yolundan çağrılıyor; parametersLoaded ise component'lerin override edebileceği, varsayılan olarak bir şey yapmayan bir bildirici olarak kalıyor. Yeni davranış, Comp test component'inin geçerli ve varsayılanı kabul etmesi için tandem bir F Prime değişikliği (nasa/fprime#5877) gerektirdi; o değişiklik artık merge edildi. Scala birim testleri, fpp-to-cpp referans testleri ve tam bir F Prime entegrasyon testiyle (FppTest, 19/19) doğrulandı. Uyguladığı issue (nasa/fpp#1098) v3.4.0 milestone'una eklendi.
Components in F Prime do not hand-write their parameter handling. A compiler written in Scala reads the model files and generates the C++ base class, including the code that loads parameters at startup. The generated hook that runs after loading did nothing, so a component that centralises its handling in one place had to iterate over every parameter by hand. Finding that out meant learning a new repository, a new language and a new build system before writing a line.
F Prime'daki bileşenler parametre yönetimini elle yazmıyor. Scala ile yazılmış bir derleyici model dosyalarını okuyup C++ temel sınıfını üretiyor, parametreleri açılışta yükleyen kod dahil. Yüklemeden sonra çalışan üretilmiş hook hiçbir şey yapmıyordu, bu yüzden yönetimi tek yerde toplayan bir bileşen her parametre üzerinde elle dönmek zorundaydı. Bunu anlamak, tek satır yazmadan önce yeni bir depo, yeni bir dil ve yeni bir derleme sistemi öğrenmek demekti.
Rob Bocchino, who wrote the compiler, reviewed it himself. The shape changed across four rounds: a guard so only successfully loaded parameters notify, then the macro that accepts defaults too, then a new hook that receives the parameter id and its validity so no information is lost, then moving the call into the load path so it fires as each parameter arrives rather than in a block at the end. Every round regenerated twenty five reference outputs that the compiler's own tests compare against.
Derleyiciyi yazan Rob Bocchino review'u bizzat yaptı. Şekil dört tur boyunca değişti: önce sadece başarıyla yüklenen parametrelerin bildirim yapması için bir koruma, sonra varsayılanları da kabul eden makro, sonra parametre kimliğini ve geçerliliğini alan ve böylece bilgi kaybetmeyen yeni bir hook, sonra çağrının yükleme yoluna taşınması, yani sondaki blokta değil her parametre geldikçe tetiklenmesi. Her tur, derleyicinin kendi testlerinin karşılaştırdığı yirmi beş referans çıktıyı yeniden üretti.
The generator uppercases identifiers. On a Turkish system locale, uppercasing the letter i produces a dotted capital, so every generated reference output came out subtly corrupted. Nothing failed loudly; the diff just looked odd. Forcing an English locale in the launcher fixed it. Checking the diff line by line before committing is now part of the routine rather than an afterthought.
Üreteç tanımlayıcıları büyük harfe çeviriyor. Türkçe sistem yerel ayarında i harfini büyütmek noktalı büyük harf üretiyor, bu yüzden üretilen her referans çıktısı sessizce bozuldu. Hiçbir şey yüksek sesle hata vermedi, sadece diff tuhaf görünüyordu. Başlatıcıda İngilizce yerel ayarı zorlamak sorunu çözdü. Commit öncesi diff'i satır satır kontrol etmek artık rutinin parçası, sonradan akla gelen bir şey değil.
Open on GitHub GitHub'da açC++ test fixC++ test düzeltmesi
nasa/fprime #5877, merged (commit d7c7693, merged by LeStarch). The tandem F Prime change for nasa/fpp#1097. With that FPP change, the generated parameterLoaded hook calls parameterUpdated for parameters that load as DEFAULT as well as VALID. The FppTest Comp component's getParameter asserted valid == VALID, which fired on a DEFAULT parameter. This relaxes the assert to FW_PARAM_OK (accepting VALID and DEFAULT), matching the pattern used in Svc components. It is backward compatible: FppTest passes 19/19 both with released FPP v3.3.0 and with the fpp#1097 build, so F Prime CI stayed green regardless of the FPP merge order.
nasa/fprime #5877, merge edildi (commit d7c7693, LeStarch merge etti). nasa/fpp#1097 için tandem F Prime değişikliği. O FPP değişikliğiyle, üretilen parameterLoaded hook'u VALID kadar DEFAULT olarak da yüklenen parametreler için parameterUpdated çağırıyor. FppTest Comp bileşeninin getParameter fonksiyonu valid == VALID doğruluyordu ve bu DEFAULT bir parametrede tetikleniyordu. Bu değişiklik doğrulamayı FW_PARAM_OK'e gevşetiyor (VALID ve DEFAULT kabul edilir), Svc bileşenlerindeki desenle uyumlu. Geriye dönük uyumlu: FppTest hem released FPP v3.3.0 hem de fpp#1097 build'iyle 19/19 geçiyor, böylece F Prime CI FPP merge sırasından bağımsız yeşil kaldı.
Before the code generator change could be accepted, the maintainer asked me to build F Prime with it and run the unit tests. Eighteen of nineteen suites passed. The one that failed did so for a real reason: a test component asserted that a parameter was valid, and the new hook now also notifies for parameters that fall back to their default value.
Kod üreteci değişikliği kabul edilmeden önce bakımcı, F Prime'ı onunla derleyip birim testlerini koşmamı istedi. On dokuz paketin on sekizi geçti. Başarısız olan biri gerçek bir sebeple başarısız oldu: bir test bileşeni parametrenin geçerli olduğunu doğruluyordu, ve yeni hook artık varsayılan değerine düşen parametreler için de bildirim yapıyor.
I wrote up the root cause, confirmed it by running the same test against the released compiler where it passes, and offered two ways to fix it rather than picking one. The maintainers chose to relax the assert, so that is what this change does. I verified it passes both with the released compiler and with the new build, which meant the two repositories could merge in either order without breaking anything.
Kök nedeni yazdım, aynı testi geçtiği yayınlanmış derleyiciyle koşarak doğruladım ve birini seçmek yerine iki çözüm yolu sundum. Bakımcılar doğrulamayı gevşetmeyi seçti, bu değişiklik de onu yapıyor. Hem yayınlanmış derleyiciyle hem yeni derlemeyle geçtiğini doğruladım, yani iki depo hangi sırayla merge edilirse edilsin bir şey kırılmayacaktı.
Open on GitHub GitHub'da açC++ featureC++ özellik
nasa/fprime #5561, merged and shipped in v4.3.0, which lists me under New Contributors. F Prime flew on the Ingenuity Mars Helicopter, the first controlled interplanetary flight, on the ISS RapidScat instrument, and on several CubeSats. I added the GenerateDp command to Svc::FileManager: it packages an on-board file into F Prime's Data Product format, so the file can be downlinked through the existing pipeline. Offset-bounded reads mean an interrupted transfer can target just the missing range instead of restarting. On a missing file or read error it returns a warning rather than failing, so a single fault does not halt the whole command sequence. Reviewed by Tim Canham and LeStarch.
nasa/fprime #5561, merge edildi ve v4.3.0 sürümüyle yayınlandı, resmi sürüm notlarında Yeni Katkıcılar listesinde adım geçiyor. F Prime, ilk kontrollü gezegenler arası uçuş olan Ingenuity Mars Helikopteri'nde, ISS RapidScat aletinde ve çeşitli CubeSat'larda kullanıldı. Svc::FileManager bileşenine GenerateDp komutunu ekledim: gemideki bir dosyayı F Prime'ın Veri Ürünü formatına paketler, böylece dosya mevcut hat üzerinden yere indirilebilir. Offset sınırlı okuma sayesinde yarıda kesilen bir aktarım baştan başlamak yerine yalnızca eksik aralığı hedefleyebilir. Dosya bulunamazsa veya okuma hatası olursa komut başarısız olmak yerine uyarı döner, böylece tek bir hata tüm komut dizisini durdurmaz. Tim Canham ve LeStarch tarafından incelendi.
F Prime could turn telemetry into data products, but there was no way to take an arbitrary file on the spacecraft and send it down the same path. Tim Canham, one of the flight software leads at JPL, wrote the issue. It was the first piece of real work I picked up after reading the contributing rules, and it was approved by the change control board before I touched any code.
F Prime telemetriyi veri ürününe çevirebiliyordu, ama uzay aracındaki herhangi bir dosyayı aynı yoldan yere indirmenin yolu yoktu. Issue'yu JPL'deki uçuş yazılımı liderlerinden Tim Canham yazmıştı. Katkı kurallarını okuduktan sonra aldığım ilk gerçek işti ve koda dokunmadan önce değişiklik kontrol kurulundan onaylıydı.
GenerateDp reads a file in chunks and writes each chunk as two records: a header carrying the file name, the absolute offset and the size, then the raw bytes. Because every chunk carries its own offset, the ground can reassemble a file from chunks that arrived out of order or from separate commands. You can ask for a byte range instead of the whole file, choose between spreading the work across rate group ticks or doing it in one pass, and set the priority of the containers.
GenerateDp dosyayı parçalar halinde okuyor ve her parçayı iki kayıt olarak yazıyor: dosya adını, mutlak offset'i ve boyutu taşıyan bir başlık, sonra ham baytlar. Her parça kendi offset'ini taşıdığı için yer tarafı, sırasız gelen ya da ayrı komutlardan gelen parçalardan dosyayı birleştirebiliyor. Tüm dosya yerine bir bayt aralığı isteyebiliyor, işi rate group tick'lerine yaymakla tek seferde yapmak arasında seçim yapabiliyor ve konteynerlerin önceliğini belirleyebiliyorsunuz.
After the review rounds the tests passed on my machine and crashed in continuous integration, on Linux and macOS alike. The first instinct was to revert the code I had just added. That would have hidden the real problem. Rebasing onto the current development branch reproduced the crash locally, and the cause turned out to be unrelated to my change: the Fw::Buffer contract had changed underneath the branch, so setData could no longer initialise an empty buffer. Moving to the constructor fixed it. The lesson stuck: on a long-running pull request, rebase regularly, because the ground moves.
Review turlarından sonra testler benim makinemde geçti ve sürekli entegrasyonda çöktü, hem Linux'ta hem macOS'ta. İlk refleks yeni eklediğim kodu geri almaktı. Bu, gerçek sorunu gizlerdi. Güncel geliştirme dalına rebase edince çökme lokalde de üretildi ve sebep benim değişikliğimle ilgisiz çıktı: Fw::Buffer sözleşmesi dalın altından değişmişti, setData artık boş bir buffer'ı kuramıyordu. Constructor'a geçmek sorunu çözdü. Ders kalıcı oldu: uzun süren bir pull request'te düzenli rebase şart, çünkü zemin kayıyor.
Open on GitHub GitHub'da açProtocol definitionProtokol tanımı
nasa/fprime #5685, merged and shipped in v4.3.0. I set up the contract for parameter communication between flight software and the ground system (GDS): adding FW_PACKET_PARAM to the ComCfg APID descriptor enum reserves a packet type for downlinking parameter values from PrmDb or equivalent stores. The enum is the single definition point, aliased into C++ as Fw::ComPacketType, so there is no parallel list to keep in sync. I kept the scope to the descriptor itself; flight side and ground side handling are tracked as separate work. Reviewed by LeStarch and thomas-bc.
nasa/fprime #5685, merge edildi ve v4.3.0 sürümüyle yayınlandı. Uçuş yazılımı ile yer sistemi (GDS) arasındaki parametre iletişimi için sözleşmeyi kurdum: ComCfg APID tanımlayıcı enum'una FW_PACKET_PARAM eklemek, PrmDb veya eşdeğer depolardan parametre değerlerini yere indirmek için bir paket tipini rezerve eder. Enum tek tanım noktasıdır, C++ tarafında Fw::ComPacketType olarak takma adlanır, böylece senkron tutulması gereken paralel bir liste kalmaz. Kapsamı yalnızca tanımlayıcının kendisiyle sınırlı tuttum; uçuş ve yer tarafı işleme ayrı işler olarak takip ediliyor. LeStarch ve thomas-bc tarafından incelendi.
The request was to add a downlink packet descriptor for parameter values. The change itself is one line in a configuration model: the reserved range ran from command to idle, and the next value was free. But the issue did not say whether the work stopped at reserving the number or also had to define the packet layout, so I read the surrounding code, found how the data product headers document their own layout, and asked which of the two was wanted before writing anything.
İstek, parametre değerleri için bir yer indirme paket tanımlayıcısı eklenmesiydi. Değişikliğin kendisi bir yapılandırma modelinde tek satır: ayrılmış aralık komuttan idle'a kadar doluydu ve sonraki değer boştu. Ama issue, işin numarayı ayırmakla mı bittiğini yoksa paket yapısını da tanımlamak gerekip gerekmediğini söylemiyordu, ben de çevredeki kodu okudum, veri ürünü başlıklarının kendi yapılarını nasıl dokümante ettiğini buldum ve bir şey yazmadan önce hangisinin istendiğini sordum.
A reserved identifier is a contract. Once it is in the released framework, ground tools and other components can rely on it, and nobody else can take that number for something different. It shipped in v4.3.0 and appears in the release changelog.
Ayrılmış bir tanımlayıcı bir sözleşmedir. Yayınlanmış framework'e girdikten sonra yer araçları ve diğer bileşenler ona güvenebilir, ve o numarayı başka biri başka bir şey için alamaz. v4.3.0 ile yayınlandı ve sürüm changelog'unda görünüyor.
Open on GitHub GitHub'da açExample updateÖrnek güncelleme
nasa/fprime #5666, merged. The examples computed data product sizes by hand, which is easy to get wrong. I moved them to the generated size macros, and while doing so found several hidden arithmetic mistakes in the manual totals. Anyone learning F Prime from these examples now reads a correct, self documenting reference. Reviewed and merged by the F Prime maintainers.
nasa/fprime #5666, merge edildi. Örnekler veri ürünü boyutlarını elle hesaplıyordu, bu da hataya çok açık. Bunları üretilen boyut makrolarına taşıdım ve bunu yaparken elle yapılan toplamlarda birkaç gizli aritmetik hata buldum. Artık bu örneklerden F Prime öğrenen herkes doğru, kendini belgeleyen bir referans okuyor. F Prime bakımcıları tarafından incelenip merge edildi.
Two example components computed the size of their data products by hand, adding up record sizes with knowledge that lived only in the developer's head. The framework had since generated macros for exactly this. The issue asked for the examples to use them.
İki örnek bileşen veri ürünlerinin boyutunu elle hesaplıyordu, kayıt boyutlarını sadece geliştiricinin kafasında yaşayan bilgiyle topluyordu. Framework bu iş için makrolar üretir hale gelmişti. Issue, örneklerin onları kullanmasını istiyordu.
Replacing the sum meant checking it, and the hand-written total was wrong in four places: one record was serialised but never counted, three array records were missing their length prefix, one string array was measured with the wrong type, and the record count was one too high. Some errors cancelled others out, which is why the demo still ran. That is the argument for generated macros in a sentence.
Toplamı değiştirmek onu kontrol etmek demekti ve elle yazılmış toplam dört yerde yanlıştı: bir kayıt serileştiriliyor ama hiç sayılmıyordu, üç dizi kaydının uzunluk öneki eksikti, bir string dizisi yanlış tiple ölçülmüştü ve kayıt sayısı bir fazlaydı. Bazı hatalar diğerlerini telafi ettiği için demo yine de çalışıyordu. Üretilen makroların gerekçesi tam olarak bu.
The pull request was closed a few hours after I opened it, because the issue behind it had not been through the change control board yet. The code was not the problem, the order was. The board approved the issue two days later, a maintainer reopened the pull request, and it merged. Since then I check the board column before I start, every time.
Pull request açtıktan birkaç saat sonra kapatıldı, çünkü arkasındaki issue henüz değişiklik kontrol kurulundan geçmemişti. Sorun kod değil, sıraydı. Kurul iki gün sonra issue'yu onayladı, bir bakımcı pull request'i yeniden açtı ve merge edildi. O günden beri işe başlamadan önce her seferinde pano kolonunu kontrol ediyorum.
Open on GitHub GitHub'da açNode featureNode özellik
NASA-AMMOS/MMGIS #1030, merged. MMGIS is NASA-AMMOS's Mars operations mapping system, used to plan and visualize Mars surface operations. I implemented the previously empty /rename endpoint: it updates the mission across all config versions and the embedded msv fields, and moves the folder on disk. The hard part was concurrent renames: the existence check ran outside the transaction, so two requests could both succeed and merge two missions under one name. Since the check is for rows that do not exist yet, a row lock cannot help, so I take a transaction scoped PostgreSQL advisory lock on the target name and re-check inside the transaction. Reviewed and merged by the MMGIS maintainer.
NASA-AMMOS/MMGIS #1030, merge edildi. MMGIS, NASA-AMMOS'un Mars yüzey operasyonlarını planlamak ve görselleştirmek için kullanılan Mars operasyon haritalama sistemidir. Daha önce boş olan /rename endpoint'ini yazdım: görevi tüm yapılandırma sürümlerinde ve gömülü msv alanlarında günceller, diskteki klasörü de taşır. Zor kısım eşzamanlı yeniden adlandırmalardı: varlık kontrolü işlem (transaction) dışında çalışıyordu, yani iki istek birden başarılı olup iki görevi tek isim altında birleştirebilirdi. Kontrol henüz var olmayan satırlar için olduğundan satır kilidi işe yaramaz, bu yüzden hedef isim üzerinde işlem kapsamlı bir PostgreSQL advisory lock alıp işlem içinde yeniden kontrol ediyorum. MMGIS bakımcısı tarafından incelenip merge edildi.
MMGIS is the mapping system used for Mars surface operations. Its configuration interface promised a rename endpoint that had never been implemented. A mission name is not just a label there: it appears in the database, inside stored configuration objects, in folder names on disk and in relative paths that other missions use to reach shared data.
MMGIS, Mars yüzey operasyonlarında kullanılan haritalama sistemi. Yapılandırma arayüzü, hiç implement edilmemiş bir yeniden adlandırma endpoint'i vaat ediyordu. Orada görev adı sadece bir etiket değil: veritabanında, saklanan yapılandırma nesnelerinin içinde, diskteki klasör adlarında ve diğer görevlerin paylaşılan veriye ulaşmak için kullandığı göreli yollarda geçiyor.
The maintainer reviewed it closely and was right three times. People with mission manager permission were losing access, because those permissions are stored by mission name. The folder name was being overwritten even when it had been deliberately decoupled from the display name. And relative paths in other missions broke silently, which the endpoint now detects and returns as warnings instead of pretending the rename was clean.
Bakımcı yakından inceledi ve üç kez haklı çıktı. Görev yöneticisi yetkisi olan kişiler erişimini kaybediyordu, çünkü o yetkiler görev adıyla saklanıyor. Klasör adı, görünen addan bilinçli olarak ayrılmış olsa bile eziliyordu. Ve diğer görevlerdeki göreli yollar sessizce kırılıyordu; endpoint artık bunu tespit edip yeniden adlandırmanın temiz olduğunu iddia etmek yerine uyarı olarak döndürüyor.
Open on GitHub GitHub'da açBug fixHata düzeltme
NASA-AMMOS/MMGIS #1039, merged. Three code paths built the mission data path from the display name instead of the configured folder name, so any mission whose folder was named differently silently pointed at the wrong location. I moved all three to the configured folder path. Reviewed and merged by the MMGIS maintainer.
NASA-AMMOS/MMGIS #1039, merge edildi. Üç kod yolu görev veri yolunu, yapılandırılan klasör adı yerine görünen isimden üretiyordu, yani klasörü farklı adlandırılmış her görev sessizce yanlış konumu gösteriyordu. Üçünü de yapılandırılan klasör yoluna taşıdım. MMGIS bakımcısı tarafından incelenip merge edildi.
Three code paths built the location of a mission's data by pasting the mission name into a path. That works until someone configures a different folder, which the system explicitly allows. The fix routes all three through the helper that already existed for this, so the configured folder is respected everywhere.
Üç kod yolu, bir görevin verisinin konumunu görev adını yola yapıştırarak kuruyordu. Bu, birisi farklı bir klasör yapılandırana kadar çalışır, ki sistem buna açıkça izin veriyor. Düzeltme üçünü de bu iş için zaten var olan yardımcıdan geçiriyor, böylece yapılandırılmış klasöre her yerde uyuluyor.
Open on GitHub GitHub'da açReact UIReact arayüz
NASA-AMMOS/MMGIS #1041, merged. I added the rename action to the mission configuration UI in React: a modal with an option to move the folder too, a warning dialog, and a refresh of the active mission after rename. It connects the backend endpoint I wrote earlier directly to the operator, the same feature end to end from server to screen. Reviewed and merged by the MMGIS maintainer.
NASA-AMMOS/MMGIS #1041, merge edildi. Görev yapılandırma arayüzüne React ile yeniden adlandırma eylemini ekledim: klasörü de taşıma seçeneği olan bir modal, bir uyarı diyaloğu ve yeniden adlandırma sonrası aktif görevin yenilenmesi. Daha önce yazdığım arka uç endpoint'ini doğrudan operatöre bağlar, aynı özelliği sunucudan ekrana uçtan uca tamamlar. MMGIS bakımcısı tarafından incelenip merge edildi.
The endpoint existed but nobody could reach it without a terminal. This adds the rename action to the mission configuration interface, next to the actions it belongs with, including the confirmation step and the warnings the endpoint can return about other missions that referenced the old name.
Endpoint vardı ama terminal olmadan kimse ona ulaşamıyordu. Bu değişiklik, yeniden adlandırma işlemini görev yapılandırma arayüzüne, ait olduğu işlemlerin yanına ekliyor; onay adımı ve endpoint'in eski ada referans veren diğer görevler hakkında dönebildiği uyarılar dahil.
Open on GitHub GitHub'da açDjango bugfixDjango hata düzeltme
One more contribution is in review, a Django fix in the Libre Space Foundation's SatNOGS ground station network. SatNOGS is a global, open network of satellite ground stations. Observation stations below sea level could not be used because the altitude field rejected negative values. The fix allows negative altitudes so those stations work. In review.
İncelemede bir katkı daha var, Libre Space Foundation'ın SatNOGS yer istasyonu ağında bir Django düzeltmesi. SatNOGS, küresel ve açık bir uydu yer istasyonu ağıdır. Deniz seviyesinin altındaki gözlem istasyonları kullanılamıyordu çünkü rakım alanı negatif değerleri reddediyordu. Düzeltme negatif rakımlara izin veriyor, böylece o istasyonlar çalışıyor. İncelemede.
SatNOGS is a volunteer network of satellite ground stations run by the Libre Space Foundation in Athens. A station record may sit as low as five hundred metres below sea level, but the observation record copied that altitude into a field that only accepted positive numbers, so scheduling from such a station returned a server error. The project had the crash report to prove it.
SatNOGS, Atina merkezli Libre Space Foundation'ın yürüttüğü gönüllü uydu yer istasyonu ağı. Bir istasyon kaydı deniz seviyesinin beş yüz metre altına kadar inebiliyor, ama gözlem kaydı bu rakımı yalnızca pozitif sayı kabul eden bir alana kopyalıyordu, dolayısıyla böyle bir istasyondan planlama yapmak sunucu hatası döndürüyordu. Projenin bunu kanıtlayan çökme kaydı vardı.
The code was accepted on the first pass. It is still open for a scheduling reason rather than a technical one: pending releases will add their own database migrations, and this one has to be rebuilt on top of them so the numbering does not collide. The patient answer was to say so, promise the rebase, and not push.
Kod ilk turda kabul edildi. Hâlâ açık olmasının sebebi teknik değil, takvim: yayına hazırlanan sürümler kendi veritabanı göçlerini getirecek ve bunun numaralandırma çakışmaması için onların üstüne yeniden kurulması gerekiyor. Sabırlı cevap bunu söylemek, rebase sözü vermek ve zorlamamaktı.
Open on GitLab GitLab'da açCode generator (FPP)Kod üreteci (FPP)
nasa/fprime #5621, claimed. This is a code generator feature for F Prime's FPP toolchain (Scala): whenever parameters are defined for a component, generate a command to reload them at runtime without rebooting. I traced how parameter commands are autocoded, proposed adding a third command kind (Reload) alongside the existing Set and Save, reusing the existing loadParameters path so no new serialization logic is introduced, and asked the maintainers a design question on command granularity and opcode compatibility. Awaiting direction before implementing.
nasa/fprime #5621, talip olundu. Bu, F Prime'ın FPP araç zinciri (Scala) için bir kod üreteci özelliği: bir component için parametreler tanımlandığında, onları yeniden başlatmadan çalışma anında yeniden yükleyen bir komut üret. Parametre komutlarının nasıl autocode edildiğini inceledim, mevcut Set ve Save'in yanına üçüncü bir komut türü (Reload) eklemeyi önerdim, mevcut loadParameters yolunu yeniden kullanarak yeni serialize mantığı gerektirmeden, ve bakımcılara komut granülaritesi ile opcode uyumluluğu hakkında bir tasarım sorusu sordum. Uygulamadan önce yön bekleniyor.
Every component with parameters already gets a function that loads them, but it only runs once, during startup. There is no generated command to trigger it later, which is what this issue asks for. I traced how the existing parameter commands are generated and proposed adding a third kind alongside them, reusing the load path rather than writing new serialisation.
Parametresi olan her bileşen zaten onları yükleyen bir fonksiyon alıyor, ama bu yalnızca açılışta bir kez çalışıyor. Sonradan tetikleyen üretilmiş bir komut yok, bu issue da onu istiyor. Mevcut parametre komutlarının nasıl üretildiğini izledim ve yanlarına üçüncü bir tür eklemeyi önerdim, yeni serileştirme yazmak yerine mevcut yükleme yolunu yeniden kullanarak.
Should the command reload everything a component owns at once, matching the plural wording of the request, or should it work parameter by parameter, matching the granularity of the commands that already exist? The answer changes the opcode layout and backward compatibility, so the code waits for it.
Komut, isteğin çoğul ifadesine uyacak şekilde bileşenin sahip olduğu her şeyi tek seferde mi yüklemeli, yoksa zaten var olan komutların granülaritesine uyacak şekilde parametre parametre mi çalışmalı? Cevap opcode yerleşimini ve geriye dönük uyumluluğu değiştiriyor, bu yüzden kod bekliyor.
Open on GitHub GitHub'da aç