RSpec/ExampleWording autocorrect clobbers string

given
it 'should do something to it\'s subject' do
end

running rubocop --only RSpec/ExampleWording -a -- my_spec.rb
changes it to
it 'does something to it's subject' do
end

which now causes a syntax error (it lost the escaped single quote).
I would expect it to change it to
it 'does something to it\'s subject' do
end

preserving the single quote.
Version info:
1.21.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.11.0, running on ruby 2.7.2 x86_64-darwin20)
  - rubocop-rails 2.12.2
  - rubocop-rake 0.6.0
  - rubocop-rspec 2.4.0

Ruby:
ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-darwin20]

It goes beyond my understanding how things work.
A simple fix would probably be to replace node.value here with something like node.loc.expression.source and strip quotes from it. node.loc.expression.source returns "'should do \\' something'", while node.value/node.str_content is "should do ' something".
Ended up with node.value.include?("'") ? node.loc.expression.source[1..-2] : node.value.
This fixes your case and the spec is still green. However, if you'll go with this fix, I strongly suggest to check the modified cop against https://github.com/pirj/real-world-rspec
Just a side note that this specific style is not recommended https://rubystyle.guide/#consistent-string-literals-single-quote

Agreed about the style being bad. To work around the syntax error I changed it to double quotes, so no escaping is needed. This in a massive codebase with many many engineers working on it for many many years. I'm transitioning to Rubocop being enforced, and can't do all cops at once. Fighting the battle of consistent string literals will be one of the last things I apply.

codebase with many many engineers working on it for many many years. Rubocop being enforced

Familiar situation.
I would suggest to start with resolving the issue with strings first, and moving to rubocop-rspec after.
However, unfortunately enough, Style/StringLiteral doesn't handle this. Filed a ticket https://github.com/rubocop/rubocop/issues/10120
I'm actually on the fence if we should really handle the case you've reported. What do you think?

I think it's unlikely politically to get Style/StringLiteral enforced in my codebase (in either direction). I would still call it a bug that any cop, in isolation, especially one marked as Safe, can modify the code in such a way as to cause syntax errors. I could definitely see it argued that the bug lies in core rubocop (possibly a missing helper method), that outputs an invalid string. But either way, I would not call "run this other cop first" as a valid resolution to this bug.

Please keep in mind that most of us are not aware of the -A/--auto-correct-all option, as rubocop --help doesn't display it.

FYI:

It's there, but there is a lot going on in that help screen so I'm not surprised that it's hard to find. I have some plans to reorganize it.
Also, if there is something that can be autocorrected by -A and not-a, RuboCop will give you a notice about that after running.

Quite unlikely that the fix is possible.

Just ran into this as well. If it can't be fixed I guess I'd expect that this rubocop would only run with the -A option (safe and unsafe) and NOT with the -a (safe only) option.

The cop can be fixed, and there’s even a suggestion above how exactly. A PR is welcome, @mpvosseller .


CompositeValidator: some validators will be skipped

CompositeValidator.cs
1
       public CompositeValidator(IValidator[] configuredValidators)
       {
           Validators = configuredValidators
               .Concat(MandatoryValidators)
               .GroupBy(valiadator => valiadator.GetType())
               .Select(grouppedByType => grouppedByType.FirstOrDefault(validator => validator.TreatsWarningsAsErrors) ?? grouppedByType.First())
               .Distinct()
               .ToArray();
       }

typos: valiadator, grouppedByType
if user adds multiple instances of the validator with different options, only one will be run. It will be better to throw exception with "Fix the validators bla-bla-bla" message instead:)

2
        public bool TreatsWarningsAsErrors => Validators.All(validator => validator.TreatsWarningsAsErrors);

bad news: validator with TreatsWarningsAsErrors = false will win.
good news: the property is not used at all :)

Thanks for the report. Will be fixed in the next version.

if user adds multiple instances of the validator with different options, only one will be run. It will be better to throw exception with "Fix the validators bla-bla-bla" message instead:)

The stronger validator (TreatsWarningsAsErrors  = true) will always win and only one will be used. There is no need to throw (it can be an effect of merging two valid configs). To me this is a desired behaviour, I have even written test for that. "Is's not a bug, it's a feature" ;)

good news: the property is not used at all :)

It's used in unit tests, however it is not a good practice ;)
cc @ig-sinicyn @AndreyAkinshin

I have made it internall and fixed the typos.


Avoid numpy "parsing timezone aware datetimes is deprecated" warning in tile server

Describe the bug
The following warning appears when the server is serving tiles to the viewer:

xcube/xcube/core/_tile2.py:659: DeprecationWarning: parsing timezone aware datetimes is deprecated; this will raise an error in the future

To Reproduce
Start an xcube viewer and server, including the --warnings flag to the server to display warnings. Perform some typical user activities in the viewer (pan, zoom, etc.). The warning will appear in the server log output.
Expected behavior
It is expected that this warning will not appear.
Additional context
While this bug currently has no consequences beyond the warning, xcube does not set an upper limit on the numpy version in its dependencies. Therefore any future numpy release may upgrade this warning to an error and break all xcube server installs from that moment onward.
The error occurs because the viewer requests a tile with a timezone-aware datestamp (Z suffix) -- see #605 for more on this. While this might be considered incorrect in cases where the time dimension of the data is timezone-naive, the server should in any case be able to handle both timezone-naive and timezone-aware datestamps.
The warning can be reproduced in isolation like this:
import numpy as np
np.array('2018-12-31T00:00:00Z').astype('datetime64[ns]')

Probably the most sensible fix is to do a specific check in the _tile2 module for this particular case (datetime64 target type, timezone-aware string), convert to UTC, and force to naive -- in effect reproducing the old behaviour of numpy.

The warning type and message have now changed: it's now "UserWarning: no explicit representation of timezones available for np.datetime64". See https://github.com/numpy/numpy/issues/23904 and https://github.com/numpy/numpy/pull/24193 for details.
Also, xcube/xcube/core/_tile2.py, where the warning originates, has been renamed to tile.py.


GDS BFF API Server

Implements a Gin HTTP API server for the BFF using the config from #379.
Blocked by #379

Codecov Report

Merging #380 (bcb3484) into main (b1fb35f) will decrease coverage by 8.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #380      +/-   ##
==========================================
- Coverage   46.03%   37.68%   -8.36%
==========================================
  Files         225      151      -74
  Lines       11080     2367    -8713
  Branches      606      606
==========================================
- Hits         5101      892    -4209
+ Misses       5060     1302    -3758
+ Partials      919      173     -746

Impacted Files
Coverage Δ

directory/pkg/gds/store/index/container.go

directory/pkg/trtl/replica/stream.go

directory/pkg/sectigo/serializers.go

directory/pkg/gds/admin/v2/decode.go

directory/pkg/gds/gds.go

directory/pkg/trtl/mock.go

directory/pkg/trtl/jitter/jitter.go

directory/pkg/gds/models/v1/contacts.go

directory/pkg/gds/service.go

directory/pkg/sectigo/creds.go

... and 64 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b1fb35f...bcb3484. Read the comment docs.

@rebeccabilbro just to signal that it was in the review queue; thank you for bearing with me and only reviewing the files that needed reviewing!


[BUG] dexlib2 out of range

Information

Apktool Version (apktool -version) - 2.9.3
Operating System (Mac, Linux, Windows) - Linux
APK From? (Playstore, ROM, Other) - Other
Java Version (java --version) - openjdk 21.0.2 2024-01-16

Stacktrace/Logcat
Exception in thread "main" com.android.tools.smali.util.ExceptionWithContext: Exception occurred while writing code_item for method Landroidx/annotation/InspectableProperty$ValueType;->values()[Landroidx/annotation/InspectableProperty$ValueType;
	at com.android.tools.smali.dexlib2.writer.DexWriter.writeDebugAndCodeItems(DexWriter.java:1119)
	at com.android.tools.smali.dexlib2.writer.DexWriter.writeTo(DexWriter.java:415)
	at com.android.tools.smali.dexlib2.writer.DexWriter.writeTo(DexWriter.java:370)
	at brut.androlib.src.SmaliBuilder.build(SmaliBuilder.java:56)
	at brut.androlib.src.SmaliBuilder.build(SmaliBuilder.java:35)
	at brut.androlib.ApkBuilder.buildSourcesSmali(ApkBuilder.java:211)
	at brut.androlib.ApkBuilder.buildSources(ApkBuilder.java:145)
	at brut.androlib.ApkBuilder.build(ApkBuilder.java:99)
	at brut.apktool.Main.cmdBuild(Main.java:296)
	at brut.apktool.Main.main(Main.java:95)
Caused by: com.android.tools.smali.util.ExceptionWithContext: Error while writing instruction at code offset 0x2
	at com.android.tools.smali.dexlib2.writer.DexWriter.writeCodeItem(DexWriter.java:1392)
	at com.android.tools.smali.dexlib2.writer.DexWriter.writeDebugAndCodeItems(DexWriter.java:1115)
	... 9 more
Caused by: com.android.tools.smali.util.ExceptionWithContext: Unsigned short value out of range: 65536
	at com.android.tools.smali.dexlib2.writer.DexDataWriter.writeUshort(DexDataWriter.java:115)
	at com.android.tools.smali.dexlib2.writer.InstructionWriter.write(InstructionWriter.java:397)
	at com.android.tools.smali.dexlib2.writer.DexWriter.writeCodeItem(DexWriter.java:1352)
	... 10 more

Steps to Reproduce

apktool d target.apk
cat patch.txt > target/smali/by/saygames/SayKitActivity.smali (file attached below)
apktool b target/
See error

Frameworks
If this APK is from an OEM ROM (Samsung, HTC, LG). Please attach framework files
(.apks that live in /system/framework or /system/priv-app)
AOSP based rom
APK
If this APK can be freely shared, please upload/attach a link to it.
https://apkcombo.com/race-master-3d-car-racing/com.easygames.race/download/phone-4.2.0-apk
Patch for smali (payload generated by objection tool)
patch.txt
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? Yes, no error
If you are trying to install a modified apk, did you resign it? Cannot build it
Are you using the latest apktool version? Yes

As this error says

Caused by: com.android.tools.smali.util.ExceptionWithContext: Unsigned short value out of range: 65536

You've exceeded the amount of methods allowed per dex (unsigned short) - so since I cannot replicate with a disassemble/assemble without the patch - that patch is the cause.
So I'm closing.
➜  3539 apktool b 3539
I: Using Apktool v2.9.2-27-5422bd70-SNAPSHOT with 8 thread(s).
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes2 folder into classes2.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes5 folder into classes5.dex...
I: Checking whether sources has changed...
I: Smaling smali_assets folder into assets.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes4 folder into classes4.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes3 folder into classes3.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes7 folder into classes7.dex...
I: Checking whether sources has changed...
I: Checking whether resources has changed...
I: Smaling smali_classes6 folder into classes6.dex...
I: Copying raw resources...
I: Copying libs... (/lib)
I: Copying libs... (/kotlin)
I: Copying libs... (/META-INF/services)
I: Building apk file...
I: Copying unknown files/dir...
I: Built apk into: 3539/dist/3539.apk
➜  3539


Consider alternatives to GitHub Pages

GitHub Pages has several very problematic limitations for a project focused on HTTP standards:

It does not allow setting Content-Type except via file extension, which is not how our schemas work
It does not support HTTP redirects, only HTML-driven refresh-redirects

The first two are essential for us, as we want to have our JSON Schemas served as JSON, and we want to have a latest resource that uses redirects instead of serving a schema with a mismatched $id (this helps prevent people from thinking that the latest URI can be used in $refs, etc.).
Netlify seems to be the most popular suggestion on our Slack, with @jmertic having seen it used for similar purposes and at least one other person recommending it.  I've also seen Cloudflare Pages used.  Both can run from GitHub repos, and I'm pretty sure both can solve both of our main problems.  The distinction might be which has the most services available through any sort of open source foundation-oriented plan (I believe both have or at least once had such plans), which I have not investigated at all.

Also, jekyll and liquid frustrate everyone endlessly AFAICT.  We'd have much better sites in a better system.

I'm very much in favour of choosing a different/better platform. I'd like to note that learn.openapis.org does also use Jekyll.

@lornajane yeah spec.openapis.org, learn.openapis.org, and tools.openapis.org are all GitHub Pages/Jekyll/Liquid.  While the frustration is most acute with the spec site, it's a problem for the others as well, just in terms of the flexibility of layout and who is able to work on them.


chore: add basic ci tests

Fixes #36.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@e2a3107). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main       #39   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?         9
  Branches        ?         1
========================================
  Hits            ?         9
  Misses          ?         0
  Partials        ?         0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


NSInvalidArgumentException Crash on iOS (unrecognized selector)

UPDATE: I have confirmed the same error appears when using the flutter_web_view plugin as well. I think the error must be with how flutter is calling the platform code. I am on the flutter master channel.
I have opened an issue in the plugin that seems to be causing this crash, but I'm also opening an issue here because it seems to involve the way Flutter is calling the plugin and also because there isn't much activity on the plugin github.
https://github.com/fluttercommunity/flutter_webview_plugin/issues/225
Steps to Reproduce

Download the flutter_webview_plugin repo.
Enter the 'example' directory.
$ flutter build ios
$ flutter run
Tap the first button to launch the in-app webview.
app will crash with the following error:

-[FlutterEngine view]: unrecognized selector sent to instance 0x16635a80
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[FlutterEngine view]: unrecognized selector sent to instance 0x16635a80'
	*** First throw call stack:
	(0x2487391b 0x2400ee17 0x248792b5 0x24876ee1 0x247a2238 0x15e1941 0x15e095f 0x18d9a1 0x1aa2c7 0x1ad6b3 0x1eb0e7 0x1b626d 0x1b8743 0x2483658f 0x248361c1 0x2483400d 0x24783229 0x24783015 0x25d73ac9 0x28e57189 0x7160f 0x2442b873)

Flutter Doctor report
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v0.10.3-pre.89, on Mac OS X 10.12.6 16G29, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
[✓] Android Studio (version 3.1)
[✓] VS Code (version 1.28.2)
[✓] Connected device (2 available)

• No issues found!

Logs
running flutter run --verbose gives this:
[+1348 ms] ------ Debug phase ------
[        ] Starting debug of 9b16416a5d3362654fa62ab8cddc5a707f183167 (N94AP, iPhone 4S, iphoneos, armv7) a.k.a. 'Jeff Mikels’s iPhone' connected through USB...
[ +690 ms] [  0%] Looking up developer disk image
[ +424 ms] [ 95%] Developer disk image mounted successfully
[+1015 ms] [100%] Connecting to remote debug server
[        ] -------------------------
[  +26 ms] (lldb) command source -s 0 '/tmp/0DF0457A-21B5-470C-B820-4AFA75925F57/fruitstrap-lldb-prep-cmds-9b16416a5d3362654fa62ab8cddc5a707f183167'
[        ] Executing commands in '/tmp/0DF0457A-21B5-470C-B820-4AFA75925F57/fruitstrap-lldb-prep-cmds-9b16416a5d3362654fa62ab8cddc5a707f183167'.
[        ] (lldb)     platform select remote-ios --sysroot '/Volumes/Jeff/Library/Developer/Xcode/iOS DeviceSupport/9.3.5 (13G36)/Symbols'
[        ]   Platform: remote-ios
[        ]  Connected: no
[        ]   SDK Path: "/Volumes/Jeff/Library/Developer/Xcode/iOS DeviceSupport/9.3.5 (13G36)/Symbols"
[        ] (lldb)     target create<EMAIL_ADDRESS>[+2369 ms] Current executable set to<EMAIL_ADDRESS>(arm64).
[        ] (lldb)     script fruitstrap_device_app="/private/var/containers/Bundle/Application/C38DE8FC-E63B-4D88-8186-B065B28A00D0/Runner.app"
[        ] (lldb)     script fruitstrap_connect_url="connect://<IP_ADDRESS>:58599"
[        ] (lldb)     target modules search-paths add /usr "/Volumes/Jeff/Library/Developer/Xcode/iOS DeviceSupport/9.3.5 (13G36)/Symbols/usr" /System "/Volumes/Jeff/Library/Developer/Xcode/iOS
DeviceSupport/9.3.5 (13G36)/Symbols/System" "/private/var/containers/Bundle/Application/C38DE8FC-E63B-4D88-8186-B065B28A00D0"
"/Volumes/Jeff/Development/@Flutter/flutter_webview_plugin/example/build/ios/iphoneos" "/var/containers/Bundle/Application/C38DE8FC-E63B-4D88-8186-B065B28A00D0"
"/Volumes/Jeff/Development/@Flutter/flutter_webview_plugin/example/build/ios/iphoneos" /Developer "/Volumes/Jeff/Library/Developer/Xcode/iOS DeviceSupport/9.3.5 (13G36)/Symbols/Developer"
[  +32 ms] (lldb)     command script import "/tmp/0DF0457A-21B5-470C-B820-4AFA75925F57/fruitstrap_9b16416a5d3362654fa62ab8cddc5a707f183167.py"
[   +5 ms] (lldb)     command script add -f fruitstrap_9b16416a5d3362654fa62ab8cddc5a707f183167.connect_command connect
[        ] (lldb)     command script add -s asynchronous -f fruitstrap_9b16416a5d3362654fa62ab8cddc5a707f183167.run_command run
[        ] (lldb)     command script add -s asynchronous -f fruitstrap_9b16416a5d3362654fa62ab8cddc5a707f183167.autoexit_command autoexit
[        ] (lldb)     command script add -s asynchronous -f fruitstrap_9b16416a5d3362654fa62ab8cddc5a707f183167.safequit_command safequit
[        ] (lldb)     connect
[  +31 ms] (lldb)     run
[+2892 ms] success
[        ] (lldb)     safequit
[ +115 ms] Process 3115 detached
[  +32 ms] Application launched on the device. Waiting for observatory port.
[ +866 ms] You've implemented -[<UIApplicationDelegate> application:performFetchWithCompletionHandler:], but you still need to add "fetch" to the list of your supported UIBackgroundModes in your
Info.plist.
[   +2 ms] You've implemented -[<UIApplicationDelegate> application:didReceiveRemoteNotification:fetchCompletionHandler:], but you still need to add "remote-notification" to the list of your supported
UIBackgroundModes in your Info.plist.
[ +611 ms] Observatory URL on device: http://<IP_ADDRESS>:51999/
[   +1 ms] attempting to forward device port 51999 to host port 1024
[        ] executing: /usr/local/bin/iproxy 1024 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +8 ms] attempting to forward device port 51999 to host port 1025
[        ] executing: /usr/local/bin/iproxy 1025 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1026
[        ] executing: /usr/local/bin/iproxy 1026 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1027
[        ] executing: /usr/local/bin/iproxy 1027 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1028
[        ] executing: /usr/local/bin/iproxy 1028 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1029
[        ] executing: /usr/local/bin/iproxy 1029 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1030
[        ] executing: /usr/local/bin/iproxy 1030 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1031
[        ] executing: /usr/local/bin/iproxy 1031 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1032
[        ] executing: /usr/local/bin/iproxy 1032 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1033
[        ] executing: /usr/local/bin/iproxy 1033 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1034
[        ] executing: /usr/local/bin/iproxy 1034 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1035
[        ] executing: /usr/local/bin/iproxy 1035 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1036
[        ] executing: /usr/local/bin/iproxy 1036 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1037
[        ] executing: /usr/local/bin/iproxy 1037 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +4 ms] attempting to forward device port 51999 to host port 1038
[        ] executing: /usr/local/bin/iproxy 1038 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1039
[        ] executing: /usr/local/bin/iproxy 1039 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1040
[        ] executing: /usr/local/bin/iproxy 1040 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1041
[        ] executing: /usr/local/bin/iproxy 1041 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[   +5 ms] attempting to forward device port 51999 to host port 1042
[        ] executing: /usr/local/bin/iproxy 1042 51999 9b16416a5d3362654fa62ab8cddc5a707f183167
[+1009 ms] Forwarded port ForwardedPort HOST:1042 to DEVICE:51999
[        ] Forwarded host port 1042 to device port 51999 for Observatory
[        ] Installing and launching... (completed)
[   +2 ms] Connecting to service protocol: http://<IP_ADDRESS>:1042/
[+1128 ms] Successfully connected to service protocol: http://<IP_ADDRESS>:1042/
[   +2 ms] getVM: {}
[  +24 ms] getIsolate: {isolateId: isolates/980807789}
[   +1 ms] _flutter.listViews: {}
[+2454 ms] DevFS: Creating new filesystem on the device (null)
[        ] _createDevFS: {fsName: example}
[ +153 ms] DevFS: Created new filesystem on the device (file:///private/var/mobile/Containers/Data/Application/CA89B149-8FF1-471E-8947-6B6F6EF2DD83/tmp/examplepTV31W/example/)
[   +1 ms] Updating assets
[  +49 ms] Syncing files to device Jeff Mikels’s iPhone...
[   +1 ms] DevFS: Starting sync from LocalDirectory: '/Volumes/Jeff/Development/@Flutter/flutter_webview_plugin/example'
[        ] Scanning project files
[   +3 ms] Scanning package files
[  +75 ms] Scanning asset files
[        ] Scanning for deleted files
[   +8 ms] Compiling dart to kernel with 446 updated files
[   +5 ms] /Volumes/Jeff/Library/Flutter/bin/cache/dart-sdk/bin/dart /Volumes/Jeff/Library/Flutter/bin/cache/artifacts/engine/darwin-x64/frontend_server.dart.snapshot --sdk-root
/Volumes/Jeff/Library/Flutter/bin/cache/artifacts/engine/common/flutter_patched_sdk/ --incremental --strong --target=flutter --output-dill build/app.dill --packages
<EMAIL_ADDRESS>--filesystem-scheme org-dartlang-root
[+1087 ms] Updating files
[ +309 ms] flutter: ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
[  +25 ms] flutter: The following message was thrown during layout:
[   +3 ms] flutter: A RenderFlex overflowed by 131 pixels on the bottom.
[ +147 ms] flutter:
[        ] flutter: The overflowing RenderFlex has an orientation of Axis.vertical.
[        ] flutter: The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and
[        ] flutter: black striped pattern. This is usually caused by the contents being too big for the RenderFlex.
[        ] flutter: Consider applying a flex factor (e.g. using an Expanded widget) to force the children of the
[        ] flutter: RenderFlex to fit within the available space instead of being sized to their natural size.
[   +4 ms] flutter: This is considered an error condition because it indicates that there is content that cannot be
[   +1 ms] flutter: seen. If the content is legitimately bigger than the available space, consider clipping it with a
[   +3 ms] flutter: ClipRect widget before putting it in the flex, or using a scrollable container rather than a Flex,
[   +9 ms] flutter: like a ListView.
[   +1 ms] flutter: The specific RenderFlex in question is:
[   +4 ms] flutter:   RenderFlex#e6ab4 relayoutBoundary=up1 OVERFLOWING
[   +7 ms] flutter:   creator: Column ← MediaQuery ← LayoutId-[<_ScaffoldSlot.body>] ← CustomMultiChildLayout ←
[   +1 ms] flutter:   AnimatedBuilder ← DefaultTextStyle ← AnimatedDefaultTextStyle ← _InkFeatures-[GlobalKey#81b20 ink
[   +3 ms] flutter:   renderer] ← NotificationListener<LayoutChangedNotification> ← PhysicalModel ←
[   +6 ms] flutter:   AnimatedPhysicalModel ← Material ← ⋯
[   +1 ms] flutter:   parentData: offset=Offset(0.0, 76.0); id=_ScaffoldSlot.body (can use size)
[   +5 ms] flutter:   constraints: BoxConstraints(0.0<=w<=320.0, 0.0<=h<=404.0)
[   +1 ms] flutter:   size: Size(320.0, 404.0)
[   +2 ms] flutter:   direction: vertical
[  +12 ms] flutter:   mainAxisAlignment: center
[   +1 ms] flutter:   mainAxisSize: max
[   +4 ms] flutter:   crossAxisAlignment: center
[   +7 ms] flutter:   verticalDirection: down
[   +1 ms] flutter: ◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤
[   +3 ms] flutter: ════════════════════════════════════════════════════════════════════════════════════════════════════
[  +86 ms] DevFS: Sync finished
[        ] Syncing files to device Jeff Mikels’s iPhone... (completed)
[        ] Synced 0.7MB.
[        ] _flutter.listViews: {}
[  +80 ms] Connected to _flutterView/0x17e2931c.
[   +1 ms] 🔥  To hot reload changes while running, press "r". To hot restart (and rebuild state), press "R".
[        ] An Observatory debugger and profiler on Jeff Mikels’s iPhone is available at: http://<IP_ADDRESS>:1042/
[        ] For a more detailed help message, press "h". To detach, press "d"; to quit, press "q".
[+6699 ms] -[FlutterEngine view]: unrecognized selector sent to instance 0x17e25480
[        ] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[FlutterEngine view]: unrecognized selector sent to instance 0x17e25480'
[        ]   *** First throw call stack:
[        ]   (0x2487391b 0x2400ee17 0x248792b5 0x24876ee1 0x247a2238 0x157d94f 0x157c8f3 0x1299a1 0x1462c7 0x1496b3 0x1870e7 0x15226d 0x154743 0x2483658f 0x248361c1 0x2483400d 0x24783229 0x24783015
0x25d73ac9 0x28e57189 0xd60f 0x2442b873)
[   +2 ms] Service protocol connection closed.
[        ] Lost connection to device.
[   +1 ms] DevFS: Deleting filesystem on the device (file:///private/var/mobile/Containers/Data/Application/CA89B149-8FF1-471E-8947-6B6F6EF2DD83/tmp/examplepTV31W/example/)
[        ] _deleteDevFS: {fsName: example}
[ +252 ms] TimeoutException after 0:00:00.250000: Future not completed
[   +2 ms] "flutter run" took 52,659ms.

flutter analyze
Analyzing example...

   info • Prefer single quotes where they won't require escape sequences • lib/main.dart:108:24 • prefer_single_quotes
   info • Prefer single quotes where they won't require escape sequences • lib/main.dart:117:24 • prefer_single_quotes

flutter doctor -v

[✓] Flutter (Channel master, v0.10.3-pre.90, on Mac OS X 10.12.6 16G29, locale en-US)
    • Flutter version 0.10.3-pre.90 at /Volumes/Jeff/Library/Flutter
    • Framework revision db9fe3f827 (2 days ago), 2018-11-04 03:20:44 +0100
    • Engine revision 88cb78cdf6
    • Dart version 2.1.0 (build 2.1.0-dev.8.0 bf26f760b1)

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Volumes/Jeff/Library/Android/sdk/
    • Android NDK at /Volumes/Jeff/Library/Android/sdk/ndk-bundle
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = /Volumes/Jeff/Library/Android/sdk/
    • Java binary at: /Volumes/Jeff/Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Volumes/Jeff/Applications/Android Studio.app/Contents
    • Flutter plugin version 28.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] VS Code (version 1.28.2)
    • VS Code at /Volumes/Jeff/Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.20.0

[✓] Connected device (1 available)
    • Jeff Mikels’s iPhone • 9b16416a5d3362654fa62ab8cddc5a707f183167 • ios • iOS 9.3.5

• No issues found!

Any progress ? same issue on iOS
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.11.7, on Mac OS X 10.14 18A391, locale fr-FR)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.2)
[!] VS Code (version 1.28.2)
[✓] Connected device (2 available)

the flutter team need to fix that . We could not just always  use beta channel

Looks like this is fixed in the webview plugin in this patch.
Note that it's unsafe to assume that registrar.messenger is a FlutterViewController. It's slightly safer, but still unsafe to assume that the root view controller of an application is a FlutterViewController as well (e.g. add-to-app scenarios in which flutter is just one subview within the overall app.


Missing translation strings

From font face properties popup:
1.1 Full Name
1.2 Designer
1.3 Version Strings
1.4 Manufacturer
1.5 Trademark
1.6 Copyright Notice
1.7 License Description
All strings from glyph variants popup

Menu item "Restore" (full screen) in top left menu is missing too.


Provide an option to clear the consumer view

Provide an option to clear the consumer view
Fixes #40
Signed-off-by: azerr<EMAIL_ADDRESS>

This PR adds Clear Consumer command, see the following demo:

I have updated the documentation.

Thanks @angelozerr!

Thanks @fbricon for the review!


Improve footer of website

🚀 Feature
Improvements we can add to Footer are :
1)Find us with logo
2)Call us with logo
3)Mail us with logo
4)Website logo
5)What this website is about
6)Follow us
-Instagram link with logo
-LinkedIn link with logo
-Twitter link with logo
-etc
7)Useful links
-Home
-Services
-About Us
-Contact Us etc
8)Subscribe
-Email address
9)Copyrights
10) All the above with responsive UI
Have you read the Contributing Guidelines on Pull Requests?
yes
I would like to work on this issue!

Can I work on this issue? I am a GSSOC participant. I'm a beginner but I want to learn and work.

Hey @SrirajBehera I want to work on this, hence why created the issue, am waiting for their response :)

No need of such issue till now.


opts.live for createHistoryStream

I can sort of get a live stream of log records by listening for the 'append' event on the db, but it would be nice to have an opts.live option to createHistoryStream()`. This option would pair well with passing in a sequence number to do ranges which is sort of what #51 suggests.

I implemented this in 2.0.0 in order to write hyperdb-index, but hasn't been re-added yet in the 3.0.0 RC. The APIs should end up identical (or very close to it), so if you dev against<EMAIL_ADDRESS>you should be OK to upgrade to 3.0.0 once it's complete.


Set up Github action to run pytest on pr

Using example code at https://github.com/snok/install-poetry#testing which just needs minor changes to work

working on push.


[CloudShell] Capture HTTPError 400 in Cloud Shell

Description
This is temporary mitigation for #12757: Cloud Shell fails with 400 Client Error for non-supported resources/audiences/scopes, like

https://ossrdbms-aad.database.windows.net
https://digitaltwins.azure.net

Currently Azure CLI will show an ugly error message:
$ az account get-access-token --resource https://digitaltwins.azure.net

The command failed with an unexpected error. Here is the traceback:

400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token
Traceback (most recent call last):
  File "/home/user1/env383/lib/python3.8/site-packages/knack/cli.py", line 215, in invoke
    cmd_result = self.invocation.execute(args)
  ...
  File "/home/user1/azure-cli/src/azure-cli-core/azure/cli/core/_profile.py", line 392, in _get_token_from_cloud_shell
    auth = MSIAuthenticationWrapper(resource=resource)
  File "/home/user1/env383/lib/python3.8/site-packages/msrestazure/azure_active_directory.py", line 592, in __init__
    self.set_token()
  File "/home/user1/env383/lib/python3.8/site-packages/msrestazure/azure_active_directory.py", line 598, in set_token
    self.scheme, _, self.token = get_msi_token(self.resource, self.port, self.msi_conf)
  File "/home/user1/env383/lib/python3.8/site-packages/msrestazure/azure_active_directory.py", line 486, in get_msi_token
    result.raise_for_status()
  File "/home/user1/env383/lib/python3.8/site-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token

With this PR, an instruction is shown as a workaround:

Failed to retrieve a token in Cloud Shell for resource https://digitaltwins.azure.net. Please run az login and try again.

$ az account get-access-token --resource https://digitaltwins.azure.net
Failed to retrieve a token in Cloud Shell for resource https://digitaltwins.azure.net. Please run `az login` and try again.

Traceback (most recent call last):
  File "/home/user1/azure-cli/src/azure-cli-core/azure/cli/core/_profile.py", line 394, in _get_token_from_cloud_shell
    auth = MSIAuthenticationWrapper(resource=resource)
  File "/home/user1/env383/lib/python3.8/site-packages/msrestazure/azure_active_directory.py", line 592, in __init__
    self.set_token()
  File "/home/user1/env383/lib/python3.8/site-packages/msrestazure/azure_active_directory.py", line 598, in set_token
    self.scheme, _, self.token = get_msi_token(self.resource, self.port, self.msi_conf)
  File "/home/user1/env383/lib/python3.8/site-packages/msrestazure/azure_active_directory.py", line 486, in get_msi_token
    result.raise_for_status()
  File "/home/user1/env383/lib/python3.8/site-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token

However, this is only a temporarily fix. For long-term, Cloud Shell needs to remove the allowlist and make all resources/audiences/scopes available.
⚠ Side effect: This will hide intermittent 400 failures for supported resources (#11749) , such as https://management.core.windows.net/. If the user follows the instruction to run az login as a workaround, the actual problem will be unreported. So I am not sure if this PR is a good choice.
Testing Guide
In Cloud Shell, follow #13567 to install Python 3.8 and checkout Azure CLI from source code. Then run
az account get-access-token --resource https://digitaltwins.azure.net

CloudShell

This issue should be resolved by Cloud Shell service, instead of worked around by the user.

This PR is superseded by #15574.


Error in documentation

The sentence Since there a are two fields names user_id, it's a blocker. seems to be wrong ;)

Thanks! :)


Metering ui change

This P/R includes:

Dashboard UI changes to add metering dashboard navigation in applications' topology view if das stats publishing is enabled and minor change in gadgets.
README file update.
Removing scaling decision events from das extension metering service capp.

Closing this p/r as it is merged.


Dockerize natural4 server

@kharus @mengwong to keep in the loop
Note that this should not be merged yet, we should merge all the dockerization branches across the different repos at once for better adoptability. Any changes to the current docker set up is welcomed
This will also not work out of the box as it still relies on v8k/rsyncing. It'll also be a lot easier to do deployment with docker hub images, once I set the CI/CD up for that.

Can you please check that prod works?
gunicorn.8080.py changes python path and does all sorts of thing which might not be in agreement with poetry setup.
And it has hard-coded paths to mengs directories.

Ah yes I will check again, but this is not intended to be merged in yet. This docker setup depends on having json be the intermediate data representation for the web. Once we can make the transition to using json, all the hard-coded paths (which are for v8k), can be removed.


miscolored text on hover

the text for the first locked upgrade is briefly colored the yellow of the upgrade seed storage button when hovering over it. this is the text that says "maybe how many times we prestiged?".

Can't reproduce, care to show a picture or video?

i can't reproduce it either anymore. it must have been a really specific window size. if i end up recreating it i'll record

Thankyou! Will close for now.


Something wrong when I run postwrf.sh to convert netCDF to Geotiff on Ubuntu OS

I have installed NCL by conda using this code:
conda create -n ncl_env -c conda-forge ncl
And I can get the version of NCL I installed:

Here is the full report:
1)

2)

Is the command convert available on your system? If not, you need to install ImageMagick by the command bleow:
sudo apt install imagemagick

Hi Amir,
I have installed the convert command using sudo apt install imagemagick, and the version is ImageMagick 6.9.12-98 Q16 x86_64 18038. However, I am still encountering the same error. My NCL version is 6.6.2
It seems the file geo_eq_ok.txt is missing in the postwrf-master/modules directory.
Thank you for your assistance!
Best regards

Why does your WRF file have a .nc suffix? Is it not the original file? Please send me wrfout_d01_2024-11-19.nc, so I can better investigate the issue.

Yes, the file I am using has a .nc suffix. Since I am unable to upload .nc files directly, I have compressed it into a .zip file for your reference.
Thank you for your assistance!
wrfout_d01_2024-11-19.zip

Okay. Thanks for sending the file.
I think wrfout_d01_2024-11-19.nc is not a standard WRF output. It has been manipulated in some way and does not have any attributes. A standard WRF output must have attributes similar to the list below:
$ ncl_filedump wrfout_d01_2023-12-17_12\:00\:00.nc | head -n 55
 Copyright (C) 1995-2019 - All Rights Reserved
 University Corporation for Atmospheric Research
 NCAR Command Language Version 6.6.2
 The use of this software is governed by a License Agreement.
 See http://www.ncl.ucar.edu/ for more details.

Variable: f
Type: file
filename:	wrfout_d01_2023-12-17_12:00:00
path:	wrfout_d01_2023-12-17_12:00:00
Number of Attributes:131
    TITLE	: 	 OUTPUT FROM WRF V4.3 MODEL
    START_DATE	: 	2023-12-17_12:00:00
    SIMULATION_START_DATE	: 	2023-12-17_12:00:00
    WEST-EAST_GRID_DIMENSION	: 	90
    SOUTH-NORTH_GRID_DIMENSION	: 	90
    BOTTOM-TOP_GRID_DIMENSION	: 	45
    DX	: 	30000
    DY	: 	30000
    AERCU_OPT	: 	0
    AERCU_FCT	: 	 1
    IDEAL_CASE	: 	0
    DIFF_6TH_SLOPEOPT	: 	0
    AUTO_LEVELS_OPT	: 	2
    DIFF_6TH_THRESH	: 	0.1
    DZBOT	: 	50
    DZSTRETCH_S	: 	1.1
...

But the attributes information in your file are missing:
$ ncl_filedump wrfout_d01_2024-11-19.nc | head -n 55
 Copyright (C) 1995-2019 - All Rights Reserved
 University Corporation for Atmospheric Research
 NCAR Command Language Version 6.6.2
 The use of this software is governed by a License Agreement.
 See http://www.ncl.ucar.edu/ for more details.

Variable: f
Type: file
filename:	wrfout_d01_2024-11-19
path:	wrfout_d01_2024-11-19
   file global attributes:
   dimensions:
      iy = 540
      jx = 690
      Time = 1
      south_north = 540
      west_east = 690
      ncl5 = 9
   variables:
      float pre ( iy, jx )
         grid_mapping :	crs
         units :	mm
         long_name :	Precipitation
         _FillValue :	-9999

      integer iy ( iy )
         long_name :	y-coordinate in Cartesian system
         standard_name :	projection_y_coordinate
         units :	m
         axis :	Y
         _CoordinateAxisType :	GeoY

      integer jx ( jx )
         long_name :	x-coordinate in Cartesian system
         standard_name :	projection_x_coordinate
         units :	m
         axis :	X
         _CoordinateAxisType :	GeoX

So for the moment, I recommend using the original WRF output.

Thank you for your feedback and for reviewing the file. I understand now that the wrfout_d01_2024-11-19.nc file is not a standard WRF output, leading to the absence of necessary attributes.
I will check the source of this file and try to obtain the original, unaltered WRF output that includes the required attributes. Once I have the correct file, I’ll test again and reach out if further assistance is needed.
Thank you for your guidance!


Implement __getitem__ on minos.common.MinosConfig instance.

 Issue by garciparedes
Friday May 21, 2021 at 09:02 GMT
Originally opened as https://github.com/Clariteia/minos_microservice_common/issues/251

This issue is intended to explain and resolve the __getitem__ (a.k.a config[...]) functionality.
Because internally, the MinosConfig class implements a way to access the config fields following a dotted string style (like "events.queue.host"), it is interesting to expose this functionality to be used publicly as config["events.queue.host"].
The main complication to do that is the cases like config["events.queue"] in which the expected result is a minos.common.QUEUE instance instead of a raw dictionary.
┆Issue is synchronized with this ClickUp task by Unito

Closing this issue as it will be resolved into #150


Add openapi specification of the API

What

Added plugin configuration to generate openapi specification docs from current API spec.
Upgraded version of docaurus
Generated and included in the sidebar the openapi speciification of the docs

Why
Upgrade of docsaurus version was necessary as the documentation generated by openapi plugin was for the new version.
Adding openapi specification to the docs was defined in https://github.com/semaphoreci/semaphore/issues/54
Checklist:
Use a checklist to confirm:

[x] The branch is up-to-date with the main branch.
[x] Update follows the docs style guide.
[x] Changes have been tested locally.

In this PR the api-method is not shown on the API specification sidebar.
One more drawback I see with this currently is that we need to regenerate the docs each time openapi specification of our API is changed.


Restructure et_relay subpackage

Summary

Make et_replay structure easier to refactor: added an entry point to export public symbols, used it in train code.
Enabled tests run as part of GHA CI.

Test Plan
CI
Additional Notes
—

cc @TaekyungHeo

The change is a good move to make et_replay more a package. But let's keep the imports absolute, they are not long enough yet for us to make them relative. Relative imports spoil readability, pep8 friendliness and refactoring capabilities. Within this single PR we are importing the ExecutionTrace class in the following manners:
from .. import ExecutionTrace, from ..execution_trace import ExecutionTrace, from .execution_trace import ExecutionTrace.

@sanrise thank you for the feedback! I mainly agree with you on relative imports topic, but there are a reason I made it like this:

Editors are going crazy: et_replay is a directory and at the same time it is a package. But the structures are different, lib doesn't exist in the package, but exist in file structure. So I need to either keep lib for both, or use relative imports.
While relative imports have bad reputation, they are all over python's code itself. And it kinda makes sense inside packages.

Anyway, I can use absolute imports and also keep lib and yet export some symbols from init. Seems fine to me. What do you think?

Editors are going crazy: et_replay is a directory and at the same time it is a package. But the structures are different, lib doesn't exist in the package, but exists in the file structure. So I need to either keep lib for both, or use relative imports.

Sounds like presence of lib is vestigial at this point, and we can get rid of it. Without it, the content of lib moves up a level.

Do you see a benefit of keeping it around just to have have it remapped in the toml? I don't, and I believe this will make everything more cleaner/easier imo.

Do you see a benefit of keeping it around just to have have it remapped in the toml? I don't, and I believe this will make everything more cleaner/easier. Thoughts?

Generally I prefer having a top-level folder for a package or even have src-layout. It makes packaging more standard and manageable. But in our case IMO flat structure is fine too. I've updated my PR accordingly.

Do you see a benefit of keeping it around just to have have it remapped in the toml? I don't, and I believe this will make everything more cleaner/easier. Thoughts?

Generally I prefer having a top-level folder for a package or even have src-layout. It makes packaging more standard and manageable. But in our case IMO flat structure is fine too. I've updated my PR accordingly.

thanks a lot! will import and run the linters / static checks shortly.

@sanrise, please don't merge this PR until we inform you.

@sanrise , you can continue code review at https://github.com/facebookresearch/param/pull/129

@TaekyungHeo  after import we are seeing some Black formatting error. Did you guys run black formatter already?
If yes then we might need to disable the linter either outside or in meta, if you haven't i can share the format suggestions

@TaekyungHeo also can you rebase to latest param main commit?

@sanrise shall we close this PR now?

Let's reopen if needed.

@amaslenn, I believe we can close this PR as it has been merged by @sanrise. https://github.com/facebookresearch/param/pull/133


Statically linked code is not removed

A number of statically-linked functions are detected in the LEGO Star Wars demo (https://www.gamepressure.com/download.asp?ID=7338) but are not removed from the final decompiled code, e.g. __pow_pentium4(). This goes both for apparently standard functions and functions from user-specified sigfiles.

@PeterMatula Can you please take a look at this?


get_gpu_info() using py3nvml is unreliable.  Sometimes returns false for GPU systems.

have_gpus=False
TEST FAILED
Probably should avoid py3nvml for this and just use a cuda call.

Also sometimes hangs.  Fixed by https://github.com/h2oai/h2o4gpu/commit/b98d3c8dab16b2e5c2c045e81bca14d8cce5c7d6


Run benchmarks in more configurations daily

Is your feature request related to a problem? Please describe.
Currently benchmarks are run in CI in limited configurations mainly as a test that the benchmarks work and some specific scenarios don't massively degrade. This doesn't give us a great overview of performance it ensures that things mostly work.
Describe the solution you'd like
A new workflow that runs the benchmarks in all configurations once a day for greater analysis.
Describe alternatives you've considered
Running them manually: takes extra effort and can forget and introduce problems by doing other things on the machine at the same time.
Additional context
Whilst CI isn't the ideal environment for performance tests it might be somewhat comparable within a run.

Outputs of this workflow could be an artifact with results (may be too much data), and a notebook file/html converted notebook for a nicer view at the data.


[charts] New, REST API

CATEGORY
Choose one

[ ] Bug Fix
[X] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
New charts REST API
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Codecov Report

Merging #8917 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8917   +/-   ##
=======================================
  Coverage   58.97%   58.97%
=======================================
  Files         359      359
  Lines       11333    11333
  Branches     2787     2787
=======================================
  Hits         6684     6684
  Misses       4471     4471
  Partials      178      178

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1dbf17b...a955d45. Read the comment docs.


Add a Market Manipulations Widget

Would you please see if i can participate in the challenge with my code.py? It works for json-files

It can work with a logarithmic scale for volume distribution, to show this I changed the volume data a bit.

@Annyvichi, the challenge is about integrating a dynamic website widget that uses provided REST endpoint to show real-time data, not generating static visualizations out of JSONs. You also have other pending pull requests with review comments that you haven't addressed.


Example fails on 2010 MBP

It's one of the dead dGPU series so running on just the iGP, which is probably the cause but I figure I might as well report it.
Error (w/ backtrace):
thread 'main' panicked at 'Couldn't create a window!', /Users/rustbuild/src/rust-buildbot/slave/stable-dist-rustc-mac/build/src/libcore/option.rs:715
stack backtrace:
   1:        0x1057f32da - std::sys::imp::backtrace::tracing::imp::write::hd3b65cdfe843284c
   2:        0x1057f49ff - std::panicking::default_hook::{{closure}}::hf2b7428652613d83
   3:        0x1057f46a7 - std::panicking::default_hook::h5da8f27db5582938
   4:        0x1057f4e66 - std::panicking::rust_panic_with_hook::hcef1e67c646c6802
   5:        0x1057f4d04 - std::panicking::begin_panic::hc2e8ca89533cd10d
   6:        0x1057f4c22 - std::panicking::begin_panic_fmt::h60990696c3c3a88d
   7:        0x1057f4b87 - rust_begin_unwind
   8:        0x10581c880 - core::panicking::panic_fmt::h10231c789bd0e97d
   9:        0x10581c8ed - core::option::expect_failed::h77d0b34eebcbdfc8
  10:        0x10574e2cb - lorem_ipsum::main::h449476b86ae0c048
  11:        0x1057f5c5a - __rust_maybe_catch_panic
  12:        0x1057f5206 - std::rt::lang_start::h87cb84a8b6cb187e

iGP spec (from System Report):

Intel HD Graphics:
Chipset Model:	Intel HD Graphics
Type:	GPU
Bus:	Built-In
VRAM (Dynamic, Max):	288 MB
Vendor:	Intel (0x8086)
Device ID:	0x0046
Revision ID:	0x0012
gMux Version:	1.9.21

OpenGL Extension Viewer report:

Renderer: Apple Software Renderer

Vendor: Apple Inc.

Memory: 0 MB

Version: 4.1 APPLE-12.1.0

Device: MacBookPro6,2

Shading language version: 4.10

Core features
v3.0 (100 % - 23/23)

v3.1 (100 % - 8/8)

v3.2 (100 % - 10/10)

v3.3 (100 % - 10/10)

v4.0 (100 % - 14/14)

v4.1 (100 % - 7/7)

v4.2 (15 % - 2/13)

v4.3 (0 % - 0/23)

v4.4 (0 % - 0/10)

v4.5 (0 % - 0/11)

vARB 2015 (0 % - 0/13)

vARB 2016 (0 % - 0/1)

Interesting. This may be a GLFW problem. What version of macOS are you running?

10.11.6 (latest El Capitan, not running Sierra yet)

I've tagged this issue as "classic", as it likely only applies to Pathfinder 1 ("classic Pathfinder"). Once Pathfinder 2 matures, this issue will be closed. Feel free to reopen if it is still an issue with the new version. Thanks!

This issue affected Pathfinder 1, which is now obsolete. Please feel free to reopen if this is still an issue with Pathfinder 2.


Mock-up - New Task - Loan hub

An entry point for loan management, reporting, and filtering.
Similar to Source Hub. Connects to numerous different loan-centric tasks.
See mock-up here:

Connects to #2124

Connects to #2125

Connects to #2127

@tmcelrath Does the loan filter meet these needs now?


KOGITO-3195: Fork KIE Editors

We finally have a snapshot version of the new forked editors from kogito-editors-java. This PR is a huge milestone for Kogito Tooling, so it deserves a good description. I'll try to summarise everything included on this PR, as it's the result of a couple of month's work and we did many improvements that were not planned beforehand!

Update Editors to 8.0.0-SNAPSHOT-20210430.130828
Remove editorContext from the init method of the Envelopes. This context is now part of the Init args we pass when opening Editors.
With the Ace editor removed from the forked Editors, we needed a new way to handle invalid files, so we added Monaco to the Online and Desktop channels, so there are some new UIs there.
Made Chrome Extension's token text input be closed by default.
Manually tree-shook @patternfly/* dependencies. When we're using ESM, this will be done automatically, but for now, we have to be extra-careful with our imports to not bring the entire PatternFly parts into our bundles.
Made the Editor interface accept custom APIs. This will be very useful when we need to provide different APIs for some of our Editors. Note that the Channels are still using the Editors with the default Editor API.
Added a custom action upon undo/redoing stuff on the StateControl class.
Removed CompositeEditorFactory as we don't want to Editors to be bundled in the same Envelope.
Improved CSS loading on the Online Editor. Please note that further improvements will be done on KOGITO-3560.

Great work Tiago, please take a look at the tests before merging it!

@tomasdavidorg @jstastny-cz @domhanak Hi guys! I'm getting Standalone Editors, Desktop, Online and Chrome Extension test failures on this PR, however, they're not failing consistently, as they pass on 2 out of 3 operating systems they're running on. Do you think we can assume it's random failures?

@tiagobento I looked at standalone editors, however I have no mac to try and reproduce :( and video is not present, it seems we can't load a page with DMN and BPMN editors on mac-os. Did you encounter a green run on mac-os at least once? If yes I would say it is random, but it is worth investigating.

@tiagobento The desktop, Chrome and online editor tests are random failures. I ran all the tests locally in Windows virtual machine and they passed but I tried to lower the number of processors in the virtual machine to 2 and in some of the runs I was able to see this screen with two loading popups which is suspicion:

I saw this only when the online editor server was started and cypress tests were executed (processor usage was on 100%). Manually (without Cypress) I was not able to reproduce it.

@tomasdavidorg Thanks for taking a look! Chrome has a feature called CPU throttling that might be able to help reproduce that issue, have you tried it?

@domhanak Yes, Standalone Editors and Desktop are ok locally on my macOS.

@tomasdavidorg Thanks for taking a look! Chrome has a feature called CPU throttling that might be able to help reproduce that issue, have you tried it?

Thanks for the tip but I was still not able to reproduce it. With the The CPU throttling the loading popup is just there for a longer time and at the end the editor appears.
However I was able to reproduce it by opening 10 windows of online editor. On every I clicked "Try dmn sample" and 4 of them end up with the loading issue. In the console I found:
Uncaught (in promise) TypeError: Cannot read property 'languages' of undefined
    at uQk (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24045)
    at vQk.wQk [as VC] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24343)
    at xQk.EQk [as VC] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24343)
    at dXk.aXk [as vD] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24344)
    at EXp.GXp [as KK] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24377)
    at e8o (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:7944)
    at c8o (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:19103)
    at Function.t8o (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24372)
    at d (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:17236)

I am able to reproduce one more issue. If I open 10 tabs with the online editor and in each I click "Create new decision model" I got this error in some of them:

:9001/#/editor/dmn:1 Uncaught (in promise) TypeError: Cannot read property 'iR' of undefined
    at dXk.UWk [as dD] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24344)
    at nXk.vXk [as dD] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24344)
    at FVk.GVk [as dD] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24344)
    at RVk.VVk [as dD] (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24344)
    at $wnd.GWTEditor.getContent (org.kie.workbench.common.dmn.showcase.DMNKogitoRuntimeWebapp-0.js:24012)
    at DmnEditorImpl.module.exports../src/common/GwtEditorWrapper.tsx.GwtEditorWrapper.getContent (index.js?7f42:351)
    at DmnEditorEnvelopeApiImpl.module.exports../src/envelope/KogitoEditorEnvelopeApiImpl.ts.KogitoEditorEnvelopeApiImpl.receive_contentRequest (index.js?a878:1140)
    at EnvelopeBusMessageManager.module.exports../src/common/EnvelopeBusMessageManager.ts.EnvelopeBusMessageManager.receive (index.js?69c4:345)
    at Object.receive (index.js?69c4:234)
    at EnvelopeBusController.module.exports../src/envelope/EnvelopeBusController.ts.EnvelopeBusController.receive (index.js?69c4:516)
    at eventListener (index.js?69c4:497)

@tomasdavidorg I can't reproduce those. Can you please give more details? Are you running it locally with yarn start on packages/online-editor? Are you using CPU throttling?

@tiagobento I am able to reproduce it only on Windows 10 virtual machine (Virtual box). The machine has 2 CPUs, 7 GB RAM. Inside machine I start yarn run start from the online editor package. Then I open 10 tabs in Chrome browser with the online editor (http://localhost:9001/).

To reproduce the issues with 2 loading popups, I click Try sample (DMN) in all the tabs.
To reproduce the issue with opening error, I click Create new decision model" in all the tabs.

https://user-images.githubusercontent.com/4498639/117101565-c81caa00-ad76-11eb-9277-0dc2efe95aa3.mp4

Hi @kelvah I've added you for review too.

01
Some monaco editor issue (https://github.com/kiegroup/kogito-tooling/pull/494#pullrequestreview-652133910) is present also for DMN in VS code.
Once autocompletion suggestions are displayed for DMN literal expression, the icons seems to be not loaded.
VS Code

dmn.new (online editor)

Hi @jomarko thanks for reporting that. I really just don't wanna block this PR by this issue, if possible. Do you think we can create a separate Jira for this one?

@tomasdavidorg Can you report this one under https://issues.redhat.com/browse/KOGITO-5039?

@tomasdavidorg Can you report this one under https://issues.redhat.com/browse/KOGITO-5039?

Reported: https://issues.redhat.com/browse/KOGITO-5068

@tiagobento I have approved; but please be sure to communicate "...why the need to change imports? Unless you communicate the reasoning to all contributors the old style will creep back in and whatever issue the change fixes will return".

@manstis The change on imports is related to item 5 on the description. Basically by importing the classes directly from their most specific directory, we are only importing what we really need. That's called tree-shaking. This is done automatically when using ES Modules, but since we're still on CommonJS, we had to do this change. There's an "automatic" way of doing it, but it's quite "magical" so I preferred to do that manually.
On the upcoming weeks we'll be migrating from CommonJS to ESM, meaning that tree-shaking will be automatic and we will be able to import @patternfly/react-core as a whole without inflating our bundles unnecessarily. For the mean time, please use the most specific import path you can find.

@kelvah @domhanak @tomasdavidorg Already got approvals for QE and the PMML Editor team, can we proceed and merge this PR?

@tiagobento Please wait for review by @LuboTerifaj. We agreed that @LuboTerifaj will do some more check with the BPMN editor. Thanks.

Thanks @LuboTerifaj.


Add Excel-related deps to analysis env

There are several possible dependencies for this. If we restrict ourselves to new-style Excel format (xlsx, as opposed to xls) then I think I understand correctly that openpyxl is all we need. This is currently included in our analysis metapackage, so I'm going to close this old issue.
Reference: http://www.python-excel.org/


Update purchasing number

Coverage increased (+0.004%) to 78.517% when pulling 014c2c5d2559265dec8fddb22e1537d335d7bc77 on achensetyawati:dev into 1b511e32371179db23e656c43cceed88781c202e on danliris:dev.


Cleaned up the readme file

Switched instances of inline three graves to one graves (test >> test)
Added a relative hyperlink to view the example folder (relative so that it works on different forks/branches)
Added a hyperlink to the Issue Tracker & Source Code sections under Contribute

@JCharante I will drain all your knowlage :D Thanks.


Global ref handling cleanups

Targeting post-v1 to be conservative.

Also I believe the target branch for this PR should be post-v1.0.0, no?

yes.

Consider also using transformGlobalVarIdent instead of referenceGlobalName+js.Ident() in def doAssign, while you're getting rid of explicit calls to referenceGlobalName.

Good point. Will update.


[FLINK-6766] Update documentation about async backends and incrementa…

…l checkpoints
This PR introduces some documentation about async heap backends and incremental snapshots with RocksDB.

CC @rmetzger

Thanks for the input @alpinegizmo ! I updated the text accordingly.

I'll merge the change.


onArrowPress How to use

onArrowPress How to use because is not a callback function.

Hi,
thanks for your question. I have added a bit more info in the docs, indeed it was missing some explanation as this callback is a bit different from the others.
Basically, this callback is called when the component is focused and any arrow key is pressed, but it HAS to return either "true" or "false".
true = default behavior - navigation will proceed in the given direction.
false = navigation will be blocked.
This callback was created as a "blocker" check, in case you want to manually restrict navigation in certain directions.
As per TS declaration:
onArrowPress?: (
    direction: string,
    props: object,
    details: KeyPressDetails
  ) => boolean;


Update IDN_2019_Sakernas_v02_M_v01_A_GLD_ALL.do

@gronert-m, 2019 has been updated and is ready for merging.

@Junying-T Thanks!


HRV Status sensor

Hi @cyberjunky!
Love this integration - expanding what we can use HA for :-)
One thing (measurement) I can't seem to find, is HRV Status - is it just me or isn't this sensor included?
This value/measurement is a really good indicator for current "body status" (for lack of a better explaination) and would be really cool to have that up and front on HA.
Is this not exposed by Garmin, or is it just not something you have included in your integration?
Would absolutely love to have this value!
Best regards
Jesper

I agree - very important measure and would love to have it on here!

Will try to squeeze it in, make more use of the new python-garminconnect library.

Don't mean to nag, but is there any hope for getting the HRV status in this awesome integration?
Thanks.


test #2

Hello again.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 432527f3-ddd9-0a90-ccf8-55b264b797ad
Version Independent ID: 05976f6f-c69f-9d2f-7a07-f3ea0bd44f7c
Content: Receive a notification when a metric value meets a condition
Content Source: articles/monitoring-and-diagnostics/monitor-quick-resource-metric-alert-portal.md
Service: azure-monitor
GitHub Login: @anirudhcavale
Microsoft Alias: ancav

@meganbradley We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and tag me in your reply. We will gladly continue the discussion.


Adding Content for Go Lang Basics

I want to add some more content for Go Lang Basics
Kindly let me know if i can go ahead with this issue for hacktober fest
Thanks
Regards

go ahead, make sure to add simplest explanations, with examples and code snippets , under code tag, check GFG website for the reference,
strict no USAGE OF AI, OTHERWISE THE PR WILL BE MARKED AS SPAM

any update @RP2025 ?


Crash on plugin removal or exiting DAW

I can successfully insert and use Fabfilter ProQ 3 on a channel strip of Harrison Mixbus. But when I want to remove the plugin or exit Mixbus, the Airwave wine process crashes and freezes the DAW.
I used AirwaveManager 1.3.3 installed from the dpkg provided on this site. I am on Debian 10 amd64. Wine version is 4.0 64-bit from the official Debian repo. My plugin host is Harrison Mixbus 32C 6.0.
Here's the backtrace:
backtrace.txt
After disabling 2D graphics rendering in Mixbus, the backtrace shows some more human readable information:
backtrace2.txt

If it happens when you open multiple airwave plugins, I would suggest to try my branch to see if you have still an issue: https://github.com/asb2m10/airwave
The problem is related to the signal handler and I've removed it since it is not required anymore with Renoise.

@asb2m10 Hi, what wine version do you use?
I'm unable to run any plugin with recent Wine 5.2
Unhandled exception: page fault on read access to 0x00000060 in 64-bit code (0x0000000180001efd).

If it happens when you open multiple airwave plugins, I would suggest to try my branch to see if you have still an issue: https://github.com/asb2m10/airwave
The problem is related to the signal handler and I've removed it since it is not required anymore with Renoise.

Thanks for bringing your fork to my attention. I compiled it in order to see if it solves my problem. Unfortunately using the manual build failed to successfully scan my plugin. I am now using LinVST which seems to work for me, so I didn't try to investigate this any further.

@asb2m10 Hi, what wine version do you use?
I'm unable to run any plugin with recent Wine 5.2
Unhandled exception: page fault on read access to 0x00000060 in 64-bit code (0x0000000180001efd).

I am using Wine 4.0 (Debian 4.0-2).

@asb2m10 Hi, what wine version do you use?
I'm unable to run any plugin with recent Wine 5.2
Unhandled exception: page fault on read access to 0x00000060 in 64-bit code (0x0000000180001efd).

I'm using Wine 5.2. I've tested Pro-Q 3 on my side and it worked fine.
If you build it in debug with my branch, there should be more details on the stacktrace when the plugin crash.

Run with debug build. Can it help?
out.txt

The plugin crashed on first call... usually this means a wine build issue or not.

Put the plugin log level in flood mode (with the airwave-manager). Then open the plugin while leaving the airwave-manager open so you can see the logs from airwave-manager.

Wine could log the issue but there is almost no way to see it. To see this change the file /opt/airwave/bin/airwave-host-64.exe at the end from

# and try to start the app
exec "$WINELOADER" "$apppath" "$@"

to
export WINEDEBUG=fixme-all,warn,trace+thread

# and try to start the app
exec "$WINELOADER" "$apppath" "$@" >> /tmp/airwave.log 2>&1

I hope that /tmp/airwave.log will have a better description on what is going on...

Sorry for long no answering, but I've stuck with building recent Carla. When I resolve that issue - I'll post the results.

No problem. If you pull from my latest branch, I've added a tester that logs wine logs on
the command line.
$ export WINEDEBUG=fixme-all,warn+all
$ /opt/airwave/bin/airwave-tester '~/.vst/DUNE 3.so'

Thanks.
I see same backtrace only. Did I missed smth?
out.txt


Add new devices

Pull Request Test Coverage Report for Build 1246

3 of 3 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1237:
0.0%

Covered Lines:
975

Relevant Lines:
975

💛  - Coveralls


Change animation package

Describe the bug 🐛
On scrolling bottom -> top. The animation gets reversed. (Occurs unexpectedly few a times)

Hey! I would like to solve this issue by using - animate.css and wow.js. Please assign this to me!

Sorry for the late reply,
Sure!


Fix profile picture upload not working in account settings

Fixes #12547. Regression in fc05b3eb596589f4262db24db3c5b6835d594573.
This view does not use the generic EditView yet, and it uses a separate form for each panel, without a root-level edit handler panel that gives you a form that contains all the fields. Thus, there's no singular form instance to be included in the context data.
In that commit, the <form> markup is removed from the template so that we reuse the markup from generic/form.html instead (and get the goodies like unsaved changes warning config). Unfortunately, we lost the enctype="multipart/form-data" attribute because if form.is_multipart doesn't evaluate to True.
I suppose the most correct solution is to do something like any(form.is_multipart() for form in panel_forms). However, the current view code doesn't make this straightforward to do. The easiest way to do it might be to do the logic in get_media() and storing the value to self, but that would add a side-effect to the method. Unless we don't care about calling get_panels() and panel.get_form() multiple times, another way to do it would be to refactor the methods and introduce cached properties like panels and panel_forms, but that's a few lines more than I'd like for a bugfix.
Looking at the original code before fc05b3eb596589f4262db24db3c5b6835d594573, it seems the template always used enctype="multipart/form-data" regardless. So I opted for a simpler fix of just passing True directly in the context data.

Merged in 246f3c7eb5 (main) and 47a910683677992fc1a5e5f5b5ad97100b7e54cf (stable/6.3.x).


[Enhancement] Show chatters name in chat log

Currently the chat messages are shown in the console, however the messages have the chatters numerical UID added to the end of the message.
It would be better to see the chatters username prefixed to the messages
i.e.
petertaylortx: omg did you see that amazing thing
djmater: Yes, best thing ever
petertaylortx: Do you think Crumble will ever find that pizza again?
djmater: No, b/c I ate it
dj_crumble21: I hate you both

Fixed issue with latest commit 45d665e


Project dependencies

Identify how to handle dependencies between projects.
Standard use case

Solution with 2 apps, frontend + backend.
Need to deploy backend changes before frontend, etc.

@wbreza Gentle ping 😸. Is this still needed?

@wbreza @savannahostrowski Closing this as a duplicate of this

https://github.com/Azure/azure-dev/issues/121


Improve reporting of Ignition failures

After Ignition fails, we should:

Explain to the user that it has failed, and why
Offer to drop to an initramfs emergency shell
If the user doesn't respond within 5 minutes, reboot the system (to try again)

It appears that, instead, we currently rerun Ignition indefinitely.
In CL this is handled by the 99emergency-timeout dracut module.

Thinking of taking a look into this - I think for now we can bring 99emergency-timeout forward straight into ignition-dracut (making sure it works in FCOS, and RHCOS too if we backport it). Future adjustments could be made to the module here if needed.


Easy debugging tooling or guidance

When something error happens in kata-agent, cloud-api-adaptor, and kata-shim, we should have some guidance to help developer identify what's happening journalctl can be used of cause but we might need a more convenient manner, for example, copy the logs from peerpod to pod logs folder on worker, even stream it to kubectl logs by enhance the containerd/crio if possible.

@huoqifeng is this issue still relevant or can be closed?
If it's still relevant to what release do you think we should map it to (mid-November, end-December, mid-February etc...)?


[Issue]: rocm/6.2.0 installation from source  on NVIDIA (Perlmutter machine)

Problem Description
I was following the commands to install hip using the instructions provided here
I get the following issue
cmake -DHIP_COMMON_DIR=/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip -DHIP_PLATFORM=nvidia -DCMAKE_INSTALL_PREFIX=/global/cfs/cdirs/nstaff/rgayatri/software/hip/clr/build/build/install -DHIP_CATCH_TEST=0 -DCLR_BUILD_HIP=ON -DCLR_BUILD_OCL=OFF -DHIPNV_DIR=/global/cfs/cdirs/nstaff/rgayatri/software/hip/hipother/hipnv ..
-- The C compiler identification is GNU 12.3.0
-- The CXX compiler identification is GNU 12.3.0
-- Cray Programming Environment 2.7.30 C
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /opt/cray/pe/craype/2.7.30/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Cray Programming Environment 2.7.30 CXX
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /opt/cray/pe/craype/2.7.30/bin/CC - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- HIPCC Binary Directory: /opt/rocm/bin
CMake Error at CMakeLists.txt:51 (message):
  Please pass hipcc/build or hipcc/bin using -DHIPCC_BIN_DIR.

Am I missing a step as I am unsure of why the build is looking for /opt/rocm
Operating System
SLES
CPU
AMD EPYC 7713 64-Core
GPU
AMD Instinct MI300X
ROCm Version
ROCm 6.2.0
ROCm Component
No response
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Edit - The GPU version listed here is not correct. Perlmutter machine has NVIDIA A100 GPU. I had to put a value there in order to submit the issue and the NVIDIA GPUs were not available in the list of options.

You need hipcc as well.
can you search any package named hipcc and install it. Point to it via -DHIPCC_BIN_DIR=<dir>

I am trying to install hipcc via this package. Is that a different package?

Its a different package.
you can clone it: https://github.com/ROCm/llvm-project/tree/amd-staging/amd/hipcc and point it to hipcc/bin dir

Thanks @cjatin . I am a bit confused at this point. In order to build hipcc, I need to point it to HIPCC_BIND_DIR ?
Additionally, while I was able to build the hipcc compiler using your solution, when I tried a simple test program, I got the following error
hipcc.bin not present; install HIPCC binaries before proceeding

so HIPCC can be a perl-script or a cpp application.
I think by default it tries to use the cpp application, you can bypass it by setting env variable : HIP_USE_PERL_SCRIPTS=1
In case you want to build hipcc.bin:
go to llvm-project/amd/hipcc
mkdir build && cd build
cmake .. -DCMAKE_INSTALL_PREFIX=<where to install hipcc>
make -j install
This will install hipcc.bin to the desired location

I am trying to build a simple test from hip-tests and it looks like hipcc is unable to find cuda. Is there a way to pass the location of nvcc ? It is installed in a non standard location. I tried passing CUDA_TOOLKIT_ROOT_DIR to install and app-compilation but they got ignored.
rgayatri@perlmutter:login40:/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build> cmake -DCMAKE_CXX_COMPILER=hipcc -DCUDA_TOOLKIT_ROOT_DIR=$CUDA_HOME ../
-- The C compiler identification is GNU 12.3.0
-- The CXX compiler identification is unknown
-- Cray Programming Environment 2.7.30 C
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /opt/cray/pe/craype/2.7.30/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - failed
-- Check for working CXX compiler: /global/common/software/nstaff/rgayatri/gpu/rocm/6.2.0/bin/hipcc
-- Check for working CXX compiler: /global/common/software/nstaff/rgayatri/gpu/rocm/6.2.0/bin/hipcc - broken
CMake Error at /global/u1/r/rgayatri/.local/cmake/share/cmake-3.23/Modules/CMakeTestCXXCompiler.cmake:62 (message):
  The C++ compiler

    "/global/common/software/nstaff/rgayatri/gpu/rocm/6.2.0/bin/hipcc"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build/CMakeFiles/CMakeTmp

    Run Build Command(s):/usr/bin/gmake -f Makefile cmTC_46803/fast && /usr/bin/gmake  -f CMakeFiles/cmTC_46803.dir/build.make CMakeFiles/cmTC_46803.dir/build
    gmake[1]: Entering directory '/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build/CMakeFiles/CMakeTmp'
    Building CXX object CMakeFiles/cmTC_46803.dir/testCXXCompiler.cxx.o
    /global/common/software/nstaff/rgayatri/gpu/rocm/6.2.0/bin/hipcc    -o CMakeFiles/cmTC_46803.dir/testCXXCompiler.cxx.o -c /global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build/CMakeFiles/CMakeTmp/testCXXCompiler.cxx
    sh: /usr/local/cuda/bin/nvcc: No such file or directory
    failed to execute:/usr/local/cuda/bin/nvcc  -Wno-deprecated-gpu-targets  -isystem /usr/local/cuda/include -isystem "/global/common/software/nstaff/rgayatri/gpu/rocm/6.2.0/include" -x cu  -o "CMakeFiles/cmTC_46803.dir/testCXXCompiler.cxx.o" -c /global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build/CMakeFiles/CMakeTmp/testCXXCompiler.cxx
    gmake[1]: *** [CMakeFiles/cmTC_46803.dir/build.make:78: CMakeFiles/cmTC_46803.dir/testCXXCompiler.cxx.o] Error 127
    gmake[1]: Leaving directory '/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build/CMakeFiles/CMakeTmp'
    gmake: *** [Makefile:127: cmTC_46803/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:23 (project)

-- Configuring incomplete, errors occurred!
See also "/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build/CMakeFiles/CMakeOutput.log".
See also "/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build/CMakeFiles/CMakeError.log".
rgayatri@perlmutter:login40:/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square/build> which nvcc
/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/bin/nvcc

Could you try setting the env var CUDA_PATH=/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2 and see if that works around the non-standard location?

Could you try setting the env var CUDA_PATH=/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2 and see if that works around the non-standard location?

No that did not help.

Update:
Looks like in both CMake and regular compilation the issue is with the following flag -Wno-deprecated-gpu-targets
rgayatri@perlmutter:login40:/global/cfs/cdirs/nstaff/rgayatri/software/hip/hip-tests/samples/0_Intro/square> hipcc square.cu -o square.ex
sh: -Wno-deprecated-gpu-targets: command not found
failed to execute:CUDA_PATH=/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/bin/nvcc  -Wno-deprecated-gpu-targets  -isystem CUDA_PATH=/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include -isystem "/global/common/software/nstaff/rgayatri/gpu/rocm/6.2.0/include"  -Wno-deprecated-gpu-targets -lcuda -lcudart -LCUDA_PATH=/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/lib64  square.cu -o "square.ex

Hi @rgayatri23,
I was able to reproduce your issue when using cmake like cmake -DCMAKE_CXX_COMPILER=hipcc ../ and found that cmake is trying to add the flag -rdynamic when trying to compile with nvcc, which is a GNU flag that is not supported with nvcc. I was able to isolate it to this file: Linux-GNU.cmake from the cmake source code.
However, I was able to get cmake working without passing in hipcc as follows:
cmake -DHIP_COMPILER=nvcc -DHIP_PLATFORM=nvidia -DHIP_RUNTIME=cuda ..
Please give that a try and let me know if you run into any issues. Thanks!


block creation of StorageClass until prerequisites are met

CephBlockPool waits for CephCluster to be ready.
Similarly, StorageClass needs to wait for CephBlockPool to be ready.
Same for CephFilesystem.

Unless I'm missing something, this doesn't seem to take into account that either of CephBlockPool or CephFilesystem could be disabled. Would it make more sense to extend StorageClusterConfiguration to include a "required kind" or something so that we can just do the check on a per-SC basis here? https://github.com/openshift/ocs-operator/blob/master/controllers/storagecluster/storageclasses.go#L88-L93

Don't we always create both the StorageClasses?
I don't think a "required kind" is necessary since we know what is required.

Also, CI failures look valid. Check make ocs-operator-ci output.

Yes, they are valid and I give up on fixing it. It requires re-factoring entire unit test framework (which is actually running integration test internally).
Unless someone has a better way to fix it.

Unless I'm missing something, this doesn't seem to take into account that either of CephBlockPool or CephFilesystem could be disabled. Would it make more sense to extend StorageClusterConfiguration to include a "required kind" or something so that we can just do the check on a per-SC basis here? https://github.com/openshift/ocs-operator/blob/master/controllers/storagecluster/storageclasses.go#L88-L93

Don't we always create both the StorageClasses?

No. If there's no CephBlockPool created it makes no sense to create a StorageClass pointing to something that doesn't exist.

I don't think a "required kind" is necessary since we know what is required.

It would be a cleaner implementation. Rather than having an if/else reading the StorageClass name and checking for the appropriate Kind based on that, We could just read the same field for all configs. But... eh, you're right, it's not strictly necessary, so I won't block on it.

Also, CI failures look valid. Check make ocs-operator-ci output.

Yes, they are valid and I give up on fixing it. It requires re-factoring entire unit test framework (which is actually running integration test internally).
Unless someone has a better way to fix it.

The ocs-operator-ci fixes are straightforward linting problems, so please at least deal with that. :P
The e2e failures look like CI infra issues, so we just need to retest. That can wait until LGTM.

Unless I'm missing something, this doesn't seem to take into account that either of CephBlockPool or CephFilesystem could be disabled. Would it make more sense to extend StorageClusterConfiguration to include a "required kind" or something so that we can just do the check on a per-SC basis here? https://github.com/openshift/ocs-operator/blob/master/controllers/storagecluster/storageclasses.go#L88-L93

Updated to check if particular StorageClass is enabled before trying to wait. But, it seems like StorageClass creation doesn't care about the setting anyway and that's a different PR.

Also, CI failures look valid. Check make ocs-operator-ci output.

Fixed the annoying uppercase/lowercase issue.

/retest

/override ci/prow/ci-index-dev-master-dependencies


Closure logic

The current version of this program:

tries to open as many connections as requested
keeps the connections open unless the other end closes the connections (normally by timeout) or refuses the connections
and, only when all connections are closed or errored, the program finishes

We should think about a better termination logic. Maybe:

a flag stating the number of seconds we want to keep each single connection open before forcing a closure in our end
or, maybe we can just start by: keep the program running until all connections are in the state "closed, errored or established", so there's no connections waiting to be dialed or dialing. When the program finishes, print a final report with some stats, including how long it took to reach this desired state.

In this stage, I'd bet for the second option

Following your suggested approach, I would change net.Dial to net.DialTimeout to put a Connection as error after a defined Timeout, if not I'm not sure a connection is closed eventually

Nice suggestion, but this timeout only applies to "how long it will take to try to establish a connection", not "how long the connection will remain open". But I covered this in the referenced PR.

(implementation idea: extra goroutine, which gets notified from the routine updating the map with connection statuses and captures signals, so it prints a nice output when finishes)

(finding that may change the implementation idea: read https://blog.golang.org/pipelines . Closing the channel may be enough and prevents us sending as many closure channels as goroutines)

(its a good moment to return a consistent exist status )

(we should also capture signals)

(dial timeout should be a flag)

(move printing outside of the main go file)

everything covered in https://github.com/dachad/check-max-tcp-connections/pull/20 , but metrics, that is a separate issue


[BUG] Load lambda's local font file

Hi,
is there some possibility how to load "local" font file uploaded in zip package (with serverless)?
Something like: await chromium.font('/var/task/fonts/Montserrat-Regular.ttf');
Fonts folder is uploaded to lambda in ZIP file and path is valid. (I can read and printcontent of ttf file to console, so there is no problem with TTF file).
Chromium returns  Invalid URL: /var/task/fonts/Montserrat-Regular.ttf so is there any other function for local files or this usecase is not possible?
PS: Is there any other way how to load fonts? For example if I want screenshot webpage, where I dont know which fonts are used on page (google fonts, @font-face or something)
Thanks

Yes, being able to support local font would be cool, so that we can package it into layer, and save bandwidth from the host/CDN every time a Lambda container is created, while local loading should be faster as well.

@bendadaniel @Zeeeeta Good idea!


Create Individual doc pages (WIP)

Description
This PR fixes #2608
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@leecalcote I recently noticed that the command tracker spreadsheet is now updated. So do I need to update them side by side then? Like previous PR I made??

@alphaX86 Yes, the Command Tracker page should be the true value everytime. So any changes made there should be reflected back here. But I do not think any major changes happened recently. Could you specify what change you are referring to?

@alphaX86 Yes, the Command Tracker page should be the true value everytime. So any changes made there should be reflected back here. But I do not think any major changes happened recently. Could you specify what change you are referring to?

Well I meant some experimental commands are added and some new commands are added in pattern. That's why I asked should I update them

The pattern commands are still WIP I guess. I guess once we have it ready we can add to it. It should not be an issue.

@leecalcote @navendu-pottekkat doc pages are ready and I linked them in the command reference table (some commands aren't updated as they're new) and Lee, I didn't feel that _config.yml must be changed, as it printed exactly the changes and output is fine. But review them and suggest any feedback. (Especially on the layout)

The URL now works... I need feedback on layout though

Hmm, need to refer those docs then for idea

I hope it ends within Mon, coz I need to look on internal exams too. 😫 I'll try to finish this within this week...

@alphaX86 Thanks for the update! Ping me and I will try to review things ASAP and not be a blocker.

Thanks! I'll let you know once it's done

Almost done... But IDK how to remove the sidebar on this one....

Plus, some formatting is a bit upside down, IDK what causes this

And @navendu-pottekkat how to add comments like this for examples as in GitHub CLI Docs? Any ideas? Or should I include
description part (in YAML) as such (may cause repetition)?

@navendu-pottekkat could you check and tell me what it lacks and needs to be done? IG I can work upto Tuesday for this...

Thank you @alphaX86 . I will review it and update ASAP.

Hmm, separate pages for subcommands huh? Might take time but will do it then

@navendu-pottekkat PR is ready :)
There may be slight variations for example validate coz command is just one and I don't think it needs a separate subcommand page.
Other than that, I've created pages. This should hold for long

If anything needs to be done, I think I or someone can do it in next PR (I mean after 2 weeks coz internals starting from Fri-Sat) so yep...

Did a quick check. The created individual pages looks good. There are some missing pages though, I think once we add that we are good to go.

Did a quick check. The created individual pages looks good. There are some missing pages though, I think once we add that we are good to go.

I also guessed it. But won't it be a bit vague if we just included only command and example, and no flags for those commands? 🤔 We need to decide later on... @layer5io/meshery-docs-contributors what do you think of this guys?

@alphaX86 I'll do a quick review here...

Good to go

Awesome, @alphaX86. 💯 It was nice getting to chat with you on this, too.


Internationalization + Japanese Language

Supercedes #412
Uses Lingui for translations - still some places where translations need to be added, but looking pretty good!
@reindex-ot I've tried to copy paste your translations over - please let me know if I've made any mistakes.
Install dependencies using pnpm i, go to the apps/expo directory, and run:
pnpm extract

And it will find all the translatable tasks in the project and tell you the status of the translations. Current status:
┌──────────┬─────────────┬─────────┐
│ Language │ Total count │ Missing │
├──────────┼─────────────┼─────────┤
│ en       │     465     │   465   │
│ ja       │     465     │   175   │
└──────────┴─────────────┴─────────┘

hi, i18n supported?

hi, i18n supported?

Yes, translation files are kept in apps/expo/src/i18n/locales (I think you saw already)

I have created a crowdin project to make this easier:
https://crowdin.com/project/graysky

I have created a crowdin project to make this easier:
https://crowdin.com/project/graysky

That's what the Ukrainian translation team for Bluesky uses, and pfrazee said it's on his TODOs to look into using it (sorry if I'm telling you stuff you already know :)

I've added some outstanding things that need translation

Almost done! Just found a couple more untranslated strings


Make a distinction between stopped and crashed

There's no way to distinguish between a projection stopped on purpose vs a projection that has crashed.
Stretch goal: show the reason for the crash.

blocking for BRV-48


Fix goreleaser-cross image that was missing to reference image on GCR

Also fix the use of deprecated flag --skip-validate with suggested updated flag usage --skip=validate.

Looks like this is a public/private GH token docker login issue (I'm not well versed on the GitHub packages/container registry and the not found errors when not logged into the docker registry through GitHub personal access token were misleading).
Nevermind but we also may want to consider what things live in which container registry, if it's truly private it should be behind our canonical locked down registry (which right now is GCR) so we have one place to tighten security and lock down rather than spread containers across two private registries.
For public facing OSS dev images then that makes sense obviously not to live in our private registries and a different solution makes sense there.


feat: 🎸 Add procedures to clear and remove a MetadataEntry

Description

Add procedure to clear an Asset Metadata value
Add procedure to remove any Local Asset Metadata
Also, adds a method isModifiable to MetadataEntry to return whether any metadata entry can be modified

Breaking Changes
NA
JIRA Link
DA-885, DA-950, DA-951
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 23.0.0-alpha.37 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 24.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 24.0.0-confidential-assets.1 :tada:
The release is available on:

npm package (@confidential-assets dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 24.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 24.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Users/kmkumaran/azurefilecopymasterpatchversion

Updating patch version for AzureFileCopy that was missed earlier.

Dropping this PR as the changes are not required.


Make StringName and GString Hashable

Closes #344, closes #379.
It's probably possible to make it match Swift string hashing:
extension String: Hashable {
  /// Hashes the essential components of this value by feeding them into the
  /// given hasher.
  ///
  /// - Parameter hasher: The hasher to use when combining the components
  ///   of this instance.
  public func hash(into hasher: inout Hasher) {
    if _fastPath(self._guts.isNFCFastUTF8) {
      self._guts.withFastUTF8 {
        hasher.combine(bytes: UnsafeRawBufferPointer($0))
      }
      hasher.combine(0xFF as UInt8) // terminator
    } else {
      _gutsSlice._normalizedHash(into: &hasher)
    }
  }
}

by using toUtf8Buffer, but I don't think there's an effective way to get actual byte array from PackedByteArray.

Oh this is delightful!   Thank you @tishin !


Resources fail to load

Scott, I think this is an awesome project. I'm using it to teach students how to create a front-end app from an API with Vue. I would love for them to be able to use this module.
I hate to bother you, but I really would love to solve this. I tried to build an app with the older webpack based version, and then switched to this much cleaner build, as the students would have less to contend with. However, I'm getting the same problem as I had when I tried to adapt your webpack version.
Setting baseUrl is working, but each css or js resource returns the home page on the front-end of October. I suspect the issue is with my implementation. I run yarn build with the baseUrl set to my theme folder, but they all return 404 errors. If you get time, I'd love your feedback on this.
Here is my vue config:
const path = require('path');
const webpack = require('webpack');
const config = require('./config/index.js');
var isProduction = process.env.NODE_ENV === 'production';
const HtmlWebpackPlugin = require('html-webpack-plugin');

module.exports = {
  baseUrl: isProduction
      ? config.build.assetsPublicPath
      : config.dev.assetsPublicPath,
  outputDir: '/assets/',
  configureWebpack: {
    resolve: {
      extensions: ['.js'],
      alias: {
        'jquery': 'jquery/dist/jquery.slim.js',
      }
    },
    plugins: [
      new webpack.ProvidePlugin({
        '$': 'jquery',
        jQuery: 'jquery',
        Popper: ['popper.js', 'default'],
        'Util': "exports-loader?Util!bootstrap/js/dist/util"
      }),
      // minify our root .htm file and inject production assets
      new HtmlWebpackPlugin({
          chunksSortMode: 'dependency',
          filename: path.resolve(__dirname, 'pages/index.htm'),
          inject: true,
          minify: {
              collapseWhitespace: true,
              removeAttributeQuotes: true,
              removeComments: true,
          },
          template: 'src/template/index.htm',
      }),
    ]
  },
}

I have copied your config from your "opinionated" version, and this is my config/index.js:
var path = require('path');

module.exports = {
    build: {
        assetsPublicPath: '/themes/themename/assets/',
        assetsRoot: path.resolve(__dirname, '../assets'),
        assetsSubDirectory: 'static',
        bundleAnalyzerReport: false,
        env: require('./prod.env'),
        productionGzip: false,
        productionGzipExtensions: ['js', 'css'],
        productionSourceMap: true,
    },
    dev: {
        assetsPublicPath: '/',
        assetsSubDirectory: 'static',
        autoOpenBrowser: true,
        cssSourceMap: false,
        env: require('./dev.env'),
        proxyTable: {},
    },
};

Here is a snippet of the built theme index.htm:
<html class="no-js" lang="en">
<head><meta charset="utf-8">
<meta http-equiv="x-ua-compatible" content="ie=edge">
...
<link href="/themes/themename/assets/css/chunk-vendors.f5b6289e.css" rel="stylesheet">
<link href="/themes/themename/assets/css/app.0811f92a.css" rel="stylesheet">
</head>

So I am mystified why I get the home page served for every resource. Here's what the css file downloads as:
<!DOCTYPE html><html class=no-js lang=en><head><meta charset=utf-8><meta http-equiv=x-ua-compatible content="ie=edge">etc

Thanks for your time...

Hmm, this is tricky to debug looking at the compiled output. Would you be able to upload the theme to a GitHub repo so myself or others can clone it down and take a look?

Thanks for that, I have just figured it out, actually, it was my .htaccess. Thanks for your time ;-)