'ld: bitcode bundle could not be generated' iOS app build error

Problem description
I am creating new app from scratch with Flutter 2.x.y and adding Adjust plugin to it. After attempt to build the app in Xcode in Debug configuration, following compile time error happens:
ld: bitcode bundle could not be generated because '/Users/uerceg/Development/flutter/bin/cache/artifacts/engine/ios/Flutter.xcframework/ios-armv7_arm64/Flutter.framework/Flutter' was built without full bitcode. All frameworks and dylibs for bitcode must be generated from Xcode Archive or Install build file '/Users/uerceg/Development/flutter/bin/cache/artifacts/engine/ios/Flutter.xcframework/ios-armv7_arm64/Flutter.framework/Flutter'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Steps to reproduce

Create new Flutter app (I created it with Android Studio new project wizard, but I assume this makes no difference in comparison to CLI).
Add the Adjust SDK plugin to this app like advised in this chapter.
Make use of Adjust SDK in your app. Example Flutter app doing that can be found in here.
Try to build app from Xcode with having Debug configuration selected and above mentioned error occurs (same error happens if one tries to run the app on iOS device from Android Studio).

Expected outcome
App should be built successfully in Debug configuration and run on the iOS device.
Actual outcome
Building the app fails with error from Problem description chapter above.
Additional info

flutter doctor -v output
[✓] Flutter (Channel stable, 2.0.2, on macOS 11.2.3 20D91 darwin-x64, locale en-GB)
    • Flutter version 2.0.2 at /Users/uerceg/Development/flutter
    • Framework revision 8962f6dc68 (7 days ago), 2021-03-11 13:22:20 -0800
    • Engine revision 5d8bf811b3
    • Dart version 2.12.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/uerceg/Development/android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/uerceg/Development/android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 53.1.5
    • Dart plugin version 203.6912

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (3 available)
    • Pixel 4a 5G (mobile)      • 0B061JECB12057            • android-arm64  • Android 11 (API 30)
    • Uglješa’s iPhone (mobile) • 00008030-0011104A228B802E • ios            • iOS 14.5
    • Chrome (web)              • chrome                    • web-javascript • Google Chrome 89.0.4389.90
    ! Error: Living Room is not connected. Xcode will continue when Living Room is connected. (code -13)

• No issues found!

Asked this in unrelated issue and @jmagman was unable to reproduce the error from example Flutter app I made and mentioned in step 3 of Steps to reproduce chapter.
(copy / paste from there) In case I switch configuration to Release, app builds and runs successfully.

In case you need any additional info, feel free to ping.
Cheers

@uerceg Can you attach the output of flutter run with the -v verbose reproducing this, since I couldn't reproduce it from the example app?   I want to know which target is failing to link on the Flutter framework.

@jmagman Here it is.
log.txt

I didn't realize your example app had the run config already set to Release. When I swap it back to Debug I can reproduce.

The problem is that all the Pods have the Flutter build settings applied:
Podfile:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)
  end
end

podhelper:
  target.build_configurations.each do |build_configuration|
...
    build_configuration.build_settings['OTHER_LDFLAGS'] = '$(inherited) -framework Flutter'

So the transitive dependency Adjust is unnecessarily linking on Flutter.  And Adjust has a full bitcode blob, so can't link against the debug version of Flutter, which only has a bitcode marker.

Caused by https://github.com/flutter/flutter/pull/72151.

Validated @uerceg's example app in debug builds on fix at https://github.com/flutter/flutter/pull/78592.
Workaround is for plugin consumers to add this to their ios/Podfile:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)

    # Add these lines:
    target.build_configurations.each do |build_configuration|
      build_configuration.build_settings['ENABLE_BITCODE'] = 'NO'
    end
  end
end

Also validated the example app builds with this workaround.
Requesting stable hotfix.  This causes build failures for iOS apps using certain Flutter plugins with transitive dependencies.

@jmagman Thank you for detailed explanation! So at the end of the day, if I understood correctly, we need to wait for https://github.com/flutter/flutter/pull/72151 to become part of the official Flutter release?

we need to wait for #72151 to become part of the official Flutter release?

Yes.  Your customers can work around the issue for now by adding that ['ENABLE_BITCODE'] = 'NO' workaround to ios/Podfile I mentioned in https://github.com/flutter/flutter/issues/78589#issuecomment-802559522. Unfortunately there's nothing you can do on the plugin side to work around this.
Sorry for the inconvenience, we'll get the fix merged ASAP.  Thanks for the detailed issue and example app, you made it easy to track down!

No problem and thank you for lightning fast replies in here. I'll advise the workaround for now and check if things got better after PR is merged. Also in case you need any additional input from my end on this topic, feel free to ping. Cheers!

Hi @jmagman
Sorry for the ping, but since I am unfamiliar with Flutter release patterns, can you maybe explain me when is this merged PR scheduled to become part of stable channel release?

@uerceg Great question. At this point in the release cycle, the hotfix train leaves every other week, approximately. Last week we shipped 2.0.3 on Friday; assuming we get enough cherrypicks to warrant a hotfix release next week, it should come out then. Stay tuned! You'll see the cp: 2.0 complete label on this issue when it's picked over and a hotfix release is imminent.

Amazing. Thank you for clarification @kf6gpe! 🍺

Thanks a lot.

Hi @kf6gpe
Sorry for dummy question, but this fix should be part of 2.0.4 release?
Cheers

@uerceg The fix was included in the 2.0.4 release: https://github.com/flutter/flutter/wiki/Hotfixes-to-the-Stable-Channel#204-april-2-2021
Are you still seeing issues on that version?

Thanks for the update! Actually not sure, still haven't tested, but will do and update in here about the results.

\o/ It works. Thank you one more time. Cheers! 🍺

I'm on 2.0.4 and I still seeing this issue. BTW the above workaround worked.

@crizant Did you flutter clean && flutter build ios?

@crizant Did you flutter clean && flutter build ios?

Yes I did.

@crizant Can you attach your flutter build or run logs with the --verbose flag?  It's large, you'll need to attach it.

@crizant Can you attach your flutter build or run logs with the --verbose flag? It's large, you'll need to attach it.

Sure.
build_log.txt

@crizant that log shows it successfully building...
[ +711 ms] Built /Users/crizantl/Documents/my_project/build/ios/iphoneos/Runner.app.
[        ] "flutter ios" took 182,821ms.
...
[        ] exiting with code 0

@crizant that log shows it successfully building...
[ +711 ms] Built /Users/crizantl/Documents/my_project/build/ios/iphoneos/Runner.app.
[        ] "flutter ios" took 182,821ms.
...
[        ] exiting with code 0

Yes but i failed to archive project in Xcode, it shows the error message as the topic of this issue.

@crizant that log shows it successfully building...
[ +711 ms] Built /Users/crizantl/Documents/my_project/build/ios/iphoneos/Runner.app.
[        ] "flutter ios" took 182,821ms.
...
[        ] exiting with code 0

Yes but i failed to archive project in Xcode, it shows the error message as the topic of this issue.

Gotcha. I need the --verbose version of the log that reproduces the issue.

Or the build log from Xcode if you aren't reproducing from flutter build ipa

@crizant any update?  I just need the Xcode logs that show the linker error on 2.0.4.

@crizant any update? I just need the Xcode logs that show the linker error on 2.0.4.

Sorry, I'm using a library provided by my client. It could be an issue from their side.
Anyway, here is the log:
Ld /Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/authlib_plugin.build/Objects-normal/armv7/Binary/authlib_plugin normal armv7 (in target 'authlib_plugin' from project 'Pods')
    cd /Users/crizantl/Documents/my_project/ios/Pods
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -target armv7-apple-ios10.0 -dynamiclib -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS14.4.sdk -L/Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/BuildProductsPath/Release-iphoneos/authlib_plugin -F/Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/BuildProductsPath/Release-iphoneos/authlib_plugin -F/Users/crizantl/flutter/bin/cache/artifacts/engine/ios-release/Flutter.xcframework/ios-armv7_arm64 -F/Users/crizantl/Documents/my_project/ios/Pods/../.symlinks/plugins/authlib_plugin/ios/ExternalLibrary -filelist /Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/authlib_plugin.build/Objects-normal/armv7/authlib_plugin.LinkFileList -install_name @rpath/authlib_plugin.framework/authlib_plugin -Xlinker -rpath -Xlinker /usr/lib/swift -Xlinker -rpath -Xlinker @executable_path/Frameworks -Xlinker -rpath -Xlinker @loader_path/Frameworks -dead_strip -Xlinker -object_path_lto -Xlinker /Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/authlib_plugin.build/Objects-normal/armv7/authlib_plugin_lto.o -fembed-bitcode -Xlinker -bitcode_verify -Xlinker -bitcode_hide_symbols -Xlinker -bitcode_symbol_map -Xlinker /Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/BuildProductsPath/Release-iphoneos/authlib_plugin -fobjc-arc -fobjc-link-runtime -L/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphoneos -L/usr/lib/swift -Xlinker -add_ast_path -Xlinker /Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/authlib_plugin.build/Objects-normal/armv7/authlib_plugin.swiftmodule -lc++ -lsqlite3 -lxml2 -lz -framework authlib -framework omslib -framework Flutter -framework Foundation -compatibility_version 1 -current_version 1 -Xlinker -dependency_info -Xlinker /Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/authlib_plugin.build/Objects-normal/armv7/authlib_plugin_dependency_info.dat -o /Users/crizantl/Library/Developer/Xcode/DerivedData/Runner-airnpugbtngbjyaicapjhltvolxp/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/authlib_plugin.build/Objects-normal/armv7/Binary/authlib_plugin

ld: bitcode bundle could not be generated because '/Users/crizantl/Documents/authlib_plugin/ios/ExternalLibrary/authlib.framework/authlib(AuthLibAPI.o)' was built without full bitcode. All object files and libraries for bitcode must be generated from Xcode Archive or Install build for architecture armv7
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Bitcode bundle could not be generated because '/Users/crizantl/Documents/authlib_plugin/ios/ExternalLibrary/authlib.framework/authlib(AuthLibAPI.o)' was built without full bitcode. All object files and libraries for bitcode must be generated from Xcode Archive or Install build for architecture armv7

@crizant This issue is tracking that non-Flutter pods were being incorrectly linked against the Flutter framework.  Your log is not complaining about Flutter.framework, it's complaining about authlib.framework in authlib_plugin, so that's caused by https://github.com/flutter/flutter/pull/72151 but not fixed by https://github.com/flutter/flutter/pull/78592.
If you own the plugin, the right thing to do may be for authlib_plugin.podspec to set s.pod_target_xcconfig = { 'ENABLE_BITCODE' => 'NO' }?  If you're just a client of that plugin, you can have the same result by editing your Podfile to something like:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)
    target.build_configurations.each do |build_configuration|
      build_configuration.build_settings['ENABLE_BITCODE] = 'NO'
    end
  end
end

In any case, as of https://github.com/flutter/flutter/pull/72151 it's up to plugins to correctly keep track of whether they enable bitcode, we should have never just blanket turned it off.


ReferenceError: btoa is not defined

I'm still having issues with btoa is not defined.

I am having this issue as well. Everything seems to work correctly if I run my koa app locally and serve it through ngrok, but once I build and deploy the app, Koa app gives the following error when requesting the install route:
  ReferenceError: btoa is not defined
      at /var/task/node_modules/simple-koa-shopify-auth/lib/top-level-oauth-redirect.js:82:24
      at Generator.next (<anonymous>)
      at fulfilled (/var/task/node_modules/simple-koa-shopify-auth/lib/top-level-oauth-redirect.js:5:58)

I am running:

@shopify/shopify-api 5.0.1
simple-koa-shopify-auth 2.1.8


Stop using CloudFlare for frontend, set up a Load Balancer

*sigh* So after much deliberation (and wasting a bit of money on Comodo SSL certificates), I've accepted that the only way we're going to fix the SSL+connection+cookie problems is by removing CloudFlare and Lets Encrypt from the frontend, moving the API to the root domain, and running our own Load Balancer with a Comodo certificate. Having to tell users to whitelist our API domain just isn't user-friendly.
So yay, that means we need 2 new servers (Load Balancer and a File Storage).

Actually I got it working.


Missing data for several days ETH-USD

First I thought yahoo was limiting queries to my ip but I replicated the issue on gcp. Is yfinance depreciated? What's up with this?

When I pull data for other tickers there isn't any missing data.

When I pull data for other tickers there isn't any missing data.

I have the same issue today (2021-01-27) with BTC-USD.
Could that be because these are categorized by finance.yahoo.com as "Cryptocurrency - CCC" ?
I have no problem with GBTC, categorized as "Equity - PNK".
Would be nice to see a fix for this though.

I have the same issue today (2021-01-27) with BTC-USD.
Could that be because these are categorized by finance.yahoo.com as "Cryptocurrency - CCC" ?
I have no problem with GBTC, categorized as "Equity - PNK".
Would be nice to see a fix for this though.

Best yfinance can do is price repair. Otherwise, this is a Yahoo issue.


is it a best practice to call establish_connection for every request?

Happy to take this to an IRC channel/slack group, but couldn't find one. Not sure if rust-beginners is appropriate, either.
I'm writing a web app using diesel and rocket, and in every route handler I'm calling establish_connection (straight from the getting started guide). Is this unnecessary overhead? Should I store a single connection to a database somewhere, and reuse that one?

In production, you probably want to use connection pooling. Have a look at r2d2-diesel (and lazy_static). I think there is an iron-diesel crate, maybe you can have a look at how they did it.
We have a gitter.im channel (linked in the Readme; also works via IRC) if you want to say hello :)

Am 28.12.2016 um 06:56 schrieb Robert Balicki<EMAIL_ADDRESS>Happy to take this to an IRC channel/slack group, but couldn't find one. Not sure if rust-beginners is appropriate, either.
I'm writing a web app using diesel and rocket, and in every route handler I'm calling establish_connection (straight from the getting started guide). Is this unnecessary overhead? Should I store a single connection to a database somewhere, and reuse that one?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

thank you!


put retry around db query.

Issue #3 in case of db file being overwritten, put a retry and delay around the db query.
haven't had chance to test this, but it should in principle work, as it makes the DB connection inside the retry, so should pick up when the file has finished updating.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

15 of 20 (75.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.2%) to 65.537%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

mbtiles.go
15
20
75.0%

Files with Coverage Reduction
New Missed Lines
%

mbtiles.go
1
70.36%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
232

Relevant Lines:
354

💛  - Coveralls

Sorry, forgot to put that bit back. I was experimenting with using ?
Parameters and positional params
On Tue, 16 Jul 2024, 21:18 Brendan Ward, @.***> wrote:

@.**** commented on this pull request.
In mbtiles.go
https://github.com/brendan-ward/mbtiles-go/pull/4#discussion_r1680018056
:

query, err := con.Prepare("select tile_data from tiles where zoom_level = $z and tile_column = $x and tile_row = $y")

query, err := con.Prepare("select tile_data from tiles where zoom_level = $1 and tile_column = $2 and tile_row = $3")

Why the change from z, x, y to 1, 2, 3?
—
Reply to this email directly, view it on GitHub
https://github.com/brendan-ward/mbtiles-go/pull/4#pullrequestreview-2181231887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUQJAU2SOKKMFOND4ERPDX3ZMV5ZRAVCNFSM6AAAAABK65TIHKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCOBRGIZTCOBYG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

@rspython have you had a chance to test this?

I haven't no.  I just need to work out how to build a module locally and
use that in the calling code, then i can make a version with this code in
and test it.  I've done some reading online and i think i understand how to
do that now.  I also want to try and put another improvement in once i've
tested this.  if there are errors such as diskio errors on the pool
connections, the pool connections stay 'dead' and never get refreshed, so i
want to put something in whereby if a pooled connection is 'bad' it gets
removed from the pool and a replacement added, so that it tries to
'recover' if it can
On Tue, 23 Jul 2024 at 17:02, Brendan Ward @.***> wrote:

@rspython https://github.com/rspython have you had a chance to test
this?
—
Reply to this email directly, view it on GitHub
https://github.com/brendan-ward/mbtiles-go/pull/4#issuecomment-2245643585,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUQJAU3Q52CFEWMSXCL3D4DZNZ5CBAVCNFSM6AAAAABK65TIHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBVGY2DGNJYGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Okay, so i managed to test it.  with a test consisting of overwriting an mbtiles file once a second, and a browser open, zoomed in, scrolling around like crazy, i can get the retry to trigger sometimes.  however it doesn't fix the problem unfortunately.  I think what's happening is the underlying sqlite library kind of 'gives up' on that connection, as whenever it hits this retry, it just retries 3 times over 6 seconds and the returns err containing "connection could not be opened" which then results in the tile server getting that error, instead of the database corrupt error.  I think if it hits this problem, we need to look at how we could trigger into the file system watcher type logic so that we can reconnect to the file properly.  so yeah, while this code 'works' in terms of it does retry, it doesn't work in terms of what it was trying to correct

Does it need to be hooked to the file system watcher? It sounds like the case we want to cover here is:

initial connection is good and everything sets up properly (i.e., we aren't starting from scratch from a bad connection)
at some point database connection becomes unstable / broken, as indicated by a tile request failing
when connection is broken, try a retry loop to reopen that connection, and if successful finish the in-flight request, and return an error otherwise

This sounds like what we want, and would probably cover the possibility of items in the connection pool going bad too, as we would just kill the item in the pool and recreate it.  as a long term java dev, but someone who grew up on machine code etc, I'll have to try and brush off my C hat and see if i can remember pointer and pointer reference logic.  Unless of course you have any time :)


Add TobMistakeTracker v0.1

Tracks mistakes made by players throughout the Theatre of Blood.
This new plugin will track mistakes for you and your teammates in the Theatre of Blood. It will also track mistakes for other raiders while you're spectating.
By default, when detecting a mistake, all players with this plugin will receive a public message of the mistake, a chat overhead above the player who made the mistake, and the mistake will be added to the Tob Mistake Tracker side-panel.
Currently, only the first couple of rooms (and deaths throughout the raid) are finished and being submitted in this version. I want to get this out the door in its current state so players can start to use it, and release more features very shortly. Refer to the README for more information and screenshots.

Regarding why I think this plugin is allowed: I don't feel as if it's doing anything against the spirit of the game. All mistakes are shown to the player after they've already made them, and are actively being punished by the game already. I made sure to never give the player information too early so that they could use it for decision making (like showing where/when certain mistakes could/would happen).
For example, when you die, well, you're already dead anyway, and you can't do anything about that. When you get stunned by a bloat hand, the mistake doesn't notify you until after it's already too late to do anything about it. For maiden blood, you're already actively losing hp, prayer, and healing the boss, and I make sure not to alert you until the tick where that's relayed to you already, not any sooner.
If for some reason any of this is deemed against the spirit of the game, I'm more than happy to have a discussion about what we can do to fix it, including adding additional delays on when mistakes are revealed to the player, if we deem necessary.
NB: I know this is a big PR, and I really appreciate your time taking a look and considering it. I've spent countless hours the past week implementing all this, and I'm really passionate about it. I look forward to the discussion!

Code structure change recommendations:

Method is never used, but should be
https://github.com/QuestingPet/TobMistakeTracker/blob/703ae3253a395bbb4cb94d8376b3a2c25d50603a/src/main/java/com/tobmistaketracker/TobRaider.java#L43-L45

Should be enum
https://github.com/QuestingPet/TobMistakeTracker/blob/8dbd1440626176f58f402800b5b9d5f697be94f3/src/main/java/com/tobmistaketracker/TobMistakeTrackerPlugin.java#L68-L70

Can be moved out of startUp
https://github.com/QuestingPet/TobMistakeTracker/blob/8dbd1440626176f58f402800b5b9d5f697be94f3/src/main/java/com/tobmistaketracker/TobMistakeTrackerPlugin.java#L137

Too deeply nested
https://github.com/QuestingPet/TobMistakeTracker/blob/8dbd1440626176f58f402800b5b9d5f697be94f3/src/main/java/com/tobmistaketracker/TobMistakeTrackerPlugin.java#L327-L341

Never used?
https://github.com/QuestingPet/TobMistakeTracker/blob/6e87a19eba104f7c6cc7b0f15da01df2bcf9ee9b/src/main/java/com/tobmistaketracker/detector/BloatMistakeDetector.java#L42

Doesn't need to extend BaseTobMistakeDetector, and likely shouldn't if it exclusively delegates, and doesn't detect mistakes on its own
https://github.com/QuestingPet/TobMistakeTracker/blob/d155f609420259398eebcc75c72a828c7945dabc/src/main/java/com/tobmistaketracker/detector/MistakeDetectorManager.java#L26


Change default from L2 rescaling to Pock-Chambolle

This is configuration appears to do better in our ablation experiments.

I'm not convinced that pock-chambolle scaling is superior to l2 scaling, see results shared in chat. The PR lgtm purely code-wise though.


596: Use read-only openpyxl representation to reduce memory usage

Workaround for Windows test failures - tests pass. Not pretty but hopefully allows #596 to proceed.
Not sure workbook.close() is/was fully to blame. Or maybe it was both problems, per commit message:

presumably, Windows antivirus scanning is being activated by 1)
a new file created and 2) that file being read. Seems to take a while
to release the file lock so tests fail on permission error.
approach here is truncate files and clear them up on a subsequent run.
possibly a nicer approach is to not use files at all, but refactor
so that we can pass around an io.BytesIO object or similar.

presumably, Windows antivirus scanning is being activated

🤯
Thanks so much for this. 🤩 No wonder I was spinning my wheels! I never would have thought an external process could be to blame.

I explored for a bit whether the pyxform process could lock the file instead but I couldn't come up with anything.

Just a note for posterity about my guess for antivirus. Main clue was the Windows "Resource Monitor" tab for Disk activity. This activity list shows active file paths and the PID and image name doing read/write on them. When running the tests, the file was being read by a process with image name "System" and description "NT Kernel & System". Not sure what else the system would be doing with files except to scan them - but whatever it's doing we can't exactly terminate the system process 💥
The same thing happened when I changed the temp root to pyxform/tests/test_output so it doesn't seem to be related to using the system/user temporary directory.


Missinge calculateHeight in swiper3

I am looking for the calculateHeight option in swiper3 but I can not find , and I can not find a replacement in the doc .
help please , thank in andvance

As it doesn't more required here, just set height:'auto' on .swiper-container


Fix issue 209

This adds the functionality referenced in issue #209.
It should also have

[ ] some checks that the run start and end times are compatible
[ ] some checks that there are no nan after all the merges

Codecov Report
Attention: Patch coverage is 0% with 48 lines in your changes missing coverage. Please review.

Project coverage is 60.42%. Comparing base (6c2a5b9) to head (96d995a).
Report is 44 commits behind head on patches.

Files
Patch %
Lines

mth5/utils/fc_tools.py
0.00%
48 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           patches     #210      +/-   ##
===========================================
- Coverage    60.89%   60.42%   -0.48%
===========================================
  Files          141      143       +2
  Lines        15933    15786     -147
===========================================
- Hits          9703     9538     -165
- Misses        6230     6248      +18

Flag
Coverage Δ

tests
60.42% <0.00%> (-0.48%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Merging into patches to see if codecov passes.


Packaging requests: jitsi-meet plugin's jibri.

Project description
@illustris
Is it possible to add a plugin of jibri on the jitsi-meet module?
Thanks.
Metadata

homepage URL:  https://github.com/jitsi/jibri
source URL: https://github.com/jitsi/jibri
platforms: linux

Hi. Last time I tried, I ran into too many issues, gave up and just use a managed jibti service for my jitsi :sweat_smile:
I'll give it another go next weekend

Wow, thank you for helping

Package will be added by the above PR. Module still needs to be written. I'll take a look at that next weekend.

Can not thank you enough.


add: deep_transformer module

I forgot to write tests for this transformer. Please wait until I am done with it!


Table inheritance

Suppose I have some fields that I want to have in all my tables. I can declare those once and then I can inherit them in other tables.
/// base table
table "base" {
  schema = schema.public
  column "created_at" {
    null    = false
    type    = timestamptz
    default = sql("now()")
  }
  column "updated_at" {
    null    = false
    type    = timestamptz
    default = sql("now()")
  }
}

/// Main table
table "projects" {
  schema = schema.public
 inherits = table.base
  column "id" {
    null = false
    type = bigserial
  }
}

Is this supported? This is a good feature.

Hey @adpande 👋
We do not support this option right now, but we want to add support for resource composition. Something like mixin with resource embedding. Here's a rough example, and I will appreciate your feedback on it.
mixin "base" {
  column "id" {
    type = int
  }
  column "name" {
    type = text
  }
  primary_key {
    columns = [column.id]
  }
}

table "customers" {
  schema = schema.test
  embed = mixin.base // Or, a list.
  column "email" {
    type = text
  }
}

I wish I could do this instead, but this is not supported by HCL (yet)
table "customers" {
+ mixin.base
  schema = schema.test
  column "email" {
    type = string
  }
}

Hi @a8m , This will definitely work. Will mixin also have option to define common index, triggers and  RLS?

:wave: Opened a PR which attempts to fix this - did not discuss this with maintainers so would be grateful for an initial review :)


xinclude fallback support and recursive parsing

Add support for XInclude/Fallback element and recursive XInclude parsing

Thanks for your contribution. As always, we really appreciate your commitment!


flutter has exited unexpectedly

I had installed the flutter SDK, when I execute the following command
git clone https://github.com/2d-inc/HistoryOfEverything.git
cd HistoryOfEverything/app
git submodule init
git submodule update
flutter run
There is something wrong as follow:
bogon:app mq$ flutter run
Running "flutter packages get" in app...                     7.0s
Launching lib/main.dart on ALP AL00 in debug mode...
Initializing gradle...                                          ⢿
Oops; flutter has exited unexpectedly.
Sending crash report to Google.
Failed to send crash report due to a network error: SocketException: OS Error: Operation timed out, errno = 60, address =
clients2.google.com, port = 51139
Crash report written to /Users/mq/flutter/demo/HistoryOfEverything/app/flutter_01.log;
please let us know at https://github.com/flutter/flutter/issues.
bogon:app mq$
And  my network is well  ，What is wrong ? help me ,thanks

and this is the  flutter_01.log :
Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter run
exception
ProcessException: ProcessException: Process "/Users/mq/flutter/demo/HistoryOfEverything/app/android/gradlew" exited abnormally:
Unzipping /Users/mq/.gradle/wrapper/dists/gradle-4.4-all/9br9xq1tocpiv8o6njlyu5op1/gradle-4.4-all.zip to /Users/mq/.gradle/wrapper/dists/gradle-4.4-all/9br9xq1tocpiv8o6njlyu5op1
Exception in thread "main" java.util.zip.ZipException: error in opening zip file
at java.util.zip.ZipFile.open(Native Method)
at java.util.zip.ZipFile.(ZipFile.java:219)
at java.util.zip.ZipFile.(ZipFile.java:149)
at java.util.zip.ZipFile.(ZipFile.java:163)
at org.gradle.wrapper.Install.unzip(Install.java:214)
at org.gradle.wrapper.Install.access$600(Install.java:27)
at org.gradle.wrapper.Install$1.call(Install.java:74)
at org.gradle.wrapper.Install$1.call(Install.java:48)
at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:65)
at org.gradle.wrapper.Install.createDist(Install.java:48)
at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:128)
at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)
Command: /Users/mq/flutter/demo/HistoryOfEverything/app/android/gradlew -v
#0      runCheckedAsync (package:flutter_tools/src/base/process.dart:251:5)
<asynchronous suspension>
#1      _initializeGradle (package:flutter_tools/src/android/gradle.dart:185:9)
<asynchronous suspension>
#2      _ensureGradle (package:flutter_tools/src/android/gradle.dart:166:37)
<asynchronous suspension>
#3      _readGradleProject (package:flutter_tools/src/android/gradle.dart:95:31)
<asynchronous suspension>
#4      _gradleProject (package:flutter_tools/src/android/gradle.dart:87:34)
<asynchronous suspension>
#5      getGradleAppOut (package:flutter_tools/src/android/gradle.dart:81:29)
<asynchronous suspension>
#6      AndroidApk.fromAndroidProject (package:flutter_tools/src/application_package.dart:96:23)
<asynchronous suspension>
#7      getApplicationPackageForPlatform (package:flutter_tools/src/application_package.dart:283:30)
<asynchronous suspension>
#8      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:269:21)
<asynchronous suspension>
#9      HotRunner.run (package:flutter_tools/src/run_hot.dart:272:39)
<asynchronous suspension>
#10     RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:398:37)
<asynchronous suspension>
#11     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:401:18)
#12     _asyncThenWrapperHelper.<anonymous closure> (dart:async/runtime/libasync_patch.dart:77:64)
#13     _rootRunUnary (dart:async/zone.dart:1132:38)
#14     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#15     _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#16     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:642:45)
#17     Future._propagateToListeners (dart:async/future_impl.dart:671:32)
#18     Future._complete (dart:async/future_impl.dart:476:7)
#19     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#20     _AsyncAwaitCompleter.complete (dart:async/runtime/libasync_patch.dart:28:18)
#21     _completeOnAsyncReturn (dart:async/runtime/libasync_patch.dart:295:13)
#22     RunCommand.usageValues (package:flutter_tools/src/commands/run.dart)
#23     _asyncThenWrapperHelper.<anonymous closure> (dart:async/runtime/libasync_patch.dart:77:64)
#24     _rootRunUnary (dart:async/zone.dart:1132:38)
#25     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#26     _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#27     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:642:45)
#28     Future._propagateToListeners (dart:async/future_impl.dart:671:32)
#29     Future._complete (dart:async/future_impl.dart:476:7)
#30     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#31     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async/runtime/libasync_patch.dart:33:20)
#32     _rootRun (dart:async/zone.dart:1124:13)
#33     _CustomZone.run (dart:async/zone.dart:1021:19)
#34     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#35     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#36     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#37     _runPendingImmediateCallback (dart:isolate/runtime/libisolate_patch.dart:115:13)
#38     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:172:5)

flutter doctor
[✓] Flutter (Channel stable, v1.0.0, on Mac OS X 10.13.6 17G65, locale zh-Hans-CN)
    • Flutter version 1.0.0 at /Users/mq/flutter
    • Framework revision 5391447fae (4 weeks ago), 2018-11-29 19:41:26 -0800
    • Engine revision 7375a0f414
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.1)
    • Android SDK at /Users/mq/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)
    • All Android licenses accepted.

[!] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    ✗ libimobiledevice and ideviceinstaller are not installed. To install with Brew, run:
        brew update
        brew install --HEAD usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    ✗ ios-deploy not installed. To install with Brew:
        brew install ios-deploy
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
        For more info, see https://flutter.io/platform-plugins
      To install:
        brew install cocoapods
        pod setup

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 23.2.1
    • Dart plugin version 171.4424
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[!] IntelliJ IDEA Ultimate Edition (version 2018.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] Connected device (1 available)
    • ALP AL00 • UYT0218126002025 • android-arm64 • Android 8.1.0 (API 27)

! Doctor found issues in 2 categories.

I'm not sure, I've tried on a couple devices here and builds are working for both Android and iOS. Could you follow the suggestion in the log and post an issue at https://github.com/flutter/flutter/issues with the crash report that was written to your filesystem?


Prevent "require('parse/node') instead" warning on server side rendered applications.

Hi,
I'm currently building a React application using Parse as the backend. The application is server side rendered, so on the initial render (happening on the server side) the process logs the following warning (as per this line):
It looks like you're using the browser version of the SDK in a node.js environment. You should require('parse/node') instead.

One solution to prevent that would be to check in the same line if a process.env variable exists, for example process.env.SSR !== true.
Happy to send a PR if this makes sense.

Sure, I can understand there are cases where the warning isn't useful, and should be capable of being hidden.
SSR sounds a bit too obscure, let's be verbose with process.env.SERVER_RENDERING and just check its truthiness (so users can run SERVER_RENDERING=1 node server.js). Happy to accept a PR.

PR done !

btw, when I try to do import Parse from 'parse/node' I get dependency error

@johhansantana those are all modules that ship as part of Node.js, and none of them are dependencies of Parse. You should look at your other code to see what's attempting to pull them in.


Opening paragraph edits

A few more comments and some edits to show how I think the clarity of the writing can be improved. Comments on the remainder of the manuscript will follow

@aaronpeikert  is this the correct way to give feedback and contribute to the writing? I didn't hear from you

Dear Casper, I am sorry that I have not responded to your edits, I was out of office for a few days, but already looked at the suggestions and decided to accept most of them, but have not pushed any changes yet.
I value any contribution from you and would accept it in any form. This form is perfectly fine!

Alright! I'll keep it up then :) Just double checking that I'm not screaming into the void


Hi 🙋🏼♂

Let us know if you have any/questions request for us. We weren't sure how to find you on social media 😊

@wcandillon will do! it's been a couple of weeks since I don't play with Skia but I plan to try out the new UI thread rendering for https://github.com/mateoguzmana/react-native-fiesta as I was experiencing some laggy renders there. Thanks for reaching out :D


Wiki

Would be great to see a basis wikipage explaining how to implement your script.  Having parsing issues here http://stackoverflow.com/questions/31086403/simplexmlelement-parsing-errors and would love to be able to debug these well. But still more of a frontend than backend guy so could use some tips on how to use your Simple XML debugger.

You pass an object to the function, it outputs some info. That's it.

@IMSoP just got the email on this issue. Had to look it up as I opened it 6 years ago. Did work out the issue in Stack Overflow thread at the time.  That's it. _
would have helped much. Code example would have been better to be honest.
But perhaps others will understand and be able to use it in the future. So thanks for that.

Yes, I was tidying up old issues, and couldn't actually think of anything to add that's not already in the README https://github.com/IMSoP/simplexml_debug#what A code example would just be simplexml_dump($simpleXml); or simplexml_tree($simpleXml); where $simpleXml is whatever object you're trying to debug.
My plan is to completely rewrite this to integrate better with var_dump etc, and then hopefully make it the default behaviour in PHP itself. At that point, there will be a bit more to explain, and I'll update the README, but there's still not going to be enough for an entire documentation wiki.

Apologies if the brief message appeared blunt / rude, however. That was not my intention.

My plan is to completely rewrite this to integrate better with var_dump etc, and then hopefully make it the default behaviour in PHP itself. At that point, there will be a bit more to explain, and I'll update the README, but there's still not going to be enough for an entire documentation wiki.

Thanks. Looking forward to seeing that. And thanks for getting back to me with some details. Much appreciated!


Added scale subcommand to scale cluster

Add scale subcommand to scale the cluster
major refactoring

https://asciinema.org/a/LTa4fecvlgoABrdQUoiTxfpou


Correct MM Derived Report Instances-Publication for naming consistency

Ensure that the id and hrid have instance in front of these and the names use the property names. If the property name has camel, the name in the column will be that name all lower case and using underscore to separate words.

Updated the naming conventions - table names in the singular, spelled out instance in the id / hrid column names, and removed table name prefixes from column names (inst_pub_).


Update Nvidia.GeForceExperience to <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This is (kind of) a duplicate of #10113. GeForce Experience requires a Nvidia GPU to be detected to install successfully, and the auto validation process doesn't use a machine with one of those. As such, it can't be updated right now.


[Hotfix] Distinct preprints [OSF-9058]

Purpose
Patch regression
Changes

Use distinct

QA Notes
Check /v2/preprints/ and ensure there are no dupes
Side Effects
None expected.
Ticket
[OSF-9058]

Can the dupes issue be reproduced in a test?

Paired to discuss. Short answer: not without knowing the exact conditions that are causing dupes, and we don't at present. Tests should be created when we solve the underlying problem


Setting default volume.

When ever a video is played user have to adjust volume settings, with these changes every new video played will inherit volume settings from last played video.

Thank you for your contribution!


🐛 Fixed broken "retry" link when an Unsplash API request failed

closes https://github.com/TryGhost/Ghost/issues/9041

add missing public retryLastRequest() method to Unsplash service

Coverage decreased (-0.01%) to 72.34% when pulling faea56156ea2e7ffbec754ad0bc9382614b215bc on kevinansfield:fix-unsplash-retry-link into baa8565ad9421c881c135922a466c364f400d6e0 on TryGhost:master.


Fix CamelCaseToSnakeCase and testdata

Fixes #2
The algorithm is as follows:

For any character, if there is an uppercase followed by a lowercase character, output as lowercase followed by an underscore.
For any lowercase number or digit followed by an uppercase, output as lowercase followed by an underscore.
Otherwise, output a lowercase character.

I believe this is correct, please LMK if I'm wrong. Subsequently, I fixed a couple of the tests that didn't follow this pattern, but everything else passes.

@googlebot rescan


OCHEM integration: Apply on new datataset

I try to infere new points (I recreate my model based on save weights) using saved model, but I see some variations on few models cases can you check this behaviour.
model use the following preprocessings:
        self.node_preprocess = node_preprocessing
        self.edge_preprocess = edge_preprocessing
        self.LPE = LaplacianPositionalEncoding()

maybe LPE is not saved properly in the weights ?

Laplacian positional encoding introduce som randomness (random flipping of signs). Is that what you are observing? Can you try to clarify what the problem is? :)
You can check that the weights have been loaded correctly by looking at the actual weights of the model before and after it has been loaded. Let me know if you need any further assistance with that.

I'm currently in the process of writing more tests (this time with unittest), to make make sure things are working as expected. Although I'm already running tests, it is not really extensive enough. Regarding model saving specifically: I think it is likely that it works well as I've been testing it out a lot (especially offline). However, I can't be 100% sure everything works as expected; I think it is great, and I'm also grateful, that you are trying things out :)

Yes, unittest is a must to have. Happy to help there. Is the "Laplacian positional encoding" randomness part is like in dropout only on training ? If it is not the case I see it like a simple augmentation technic that can be leverage for variability of prediction. But I did not notice this aspect in any papers this is somehow not explored yet or I miss one paper comment ?

They bring it up in the paper. As the sign out the eigen vectors can be either negative or positive (though only one of them are computed arbitrarily) it is strongly recommended to randomly flip them. You are right that it seems to be done during training, and I guess you could also see it as some kind of data augmentation. I should add a if training: ... perform random sign flip ...
From e.g. this paper:
"Although these eigenvectors have multiplicity occuring due
to the arbitrary sign of eigenvectors, we randomly flip the
sign of the eigenvectors during training, following Dwivedi
et al. (2020)."

it was my filling. So basically you have two only two possible results + or - ? What is the correct answer the mean ?

The mean would be 0, I would say. The correct answer is both (either). Hopefully the model will learn both. Consequently, during prediction time, either the outcomes (of the sign flipping) should will work well.


[Data collection] Improve confirmation screen shown after data is submitted

This would replace the modal dialog currently displayed. Note that the labels in the header are placeholders for the job and LOI names.
Graphic:

@anandwana001 FYI, in case you're looking for a good starter FR!

@rawbzz Can we find a graphic that's more relatable to our target audience for data collection (small-holder farmers, indigenous community members, and local staff)?

@rawbzz Could you kindly paste the latest designs for this confirmation screen here?

@rawbzz Thank you!


Dao card view

GitHub Issue
https://github.com/HausDAO/daohaus-monorepo/issues/243
Changes

Toggles between table and cards
Displays cards 3 max in desktop, wraps to 2 tablet, 1 mobile

Desktop

Mobile

Packages Added
NA
Checks
Before making your PR, please check the following:

[x] Critical lint errors are resolved
[x] App runs locally
[x] App builds locally (run the build command for any impacted package and check for any errors before the PR)

Huge!! 🚀


remove extra space

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to the menu bar
Click on dark mode
Scroll down to Dark mode
See error

Expected behavior
after clicking on the dark mode button, extra space occurs because of the Home page videos. As the number of videos increases, the height of space also increases
Screenshots

Additional context
Add any other context about the problem here.

Describe the bug A clear and concise description of what the bug is.
To Reproduce Steps to reproduce the behavior:

Go to the menu bar
Click on dark mode
Scroll down to Dark mode
See error

Expected behavior after clicking on the dark mode button, extra space occurs because of the Home page videos. As the number of videos increases, the height of space also increases
Screenshots

Additional context Add any other context about the problem here.

can I work on this issue ?


Notebook UI brain dump from implementing in PowerShell extension

@jrieken asked me to open an issue with just a brain dump of the things I've been thinking about and experiencing as I've been implementing support for the Notebook UI in the PowerShell extension.
This is a living issue! I'll update it as I learn things.
cc @rebornix
My "vision"
A "Notebook Mode" exposed in the PowerShell extension that lets you open any PowerShell file and render it like a Notebook (comments are markdown cells, script is code cells).
Pressing play will be the equivalent of selecting the text and pressing F8 which runs the selected text (or cell in this case) in the PowerShell Integrated Console (PSIC) which is in the terminal pane.
Language features (provided by PowerShellEditorServices (PSES)) will still be supported (intellisense, references, diagnostics) since these all operate partially based on the file but also the state of the PSIC.
You will be able to easily swap between Notebook mode and text editor mode with the click of a button OR you can mark a file with #!PSNotebook at the top (or something like that) and that document will always be treated like a Notebook.
Parsing the file
Well I'll be honest, @rebornix did this work for me.
So far it's based on regular comments:
# this will be markdown
"this will be code"

Metadata

NEW: 4/28

I recently added support for including block comments as Markdown cells but I wanted to give the user the ability to control how new markdown cells are saved (aka as block comments or line comments).
That's when I realized that I would have to handle my own metadata for now... because there's no way to attach metadata to cells for use later (like when saving). I opened the following two issues to make this experience better:
https://github.com/microsoft/vscode/issues/96433
https://github.com/microsoft/vscode/issues/96435
In the meantime, I am using a Map<number, CommentType (enum)> to achieve out-of-object metadata.
Notebook Mode UX
I've not started this yet since VS Code just added an editor command for me to use to open a file in a different editor.
I'll be working on this today most likely.
Hooking up the language server
I'll break this down since there are a few important pieces.
Representation of files in PSES
After talking to @rebornix @jrieken there were 2 ways to go:

Each cell is represented as its own file (default behavior)
Concat the cells into one file on the client and do the mapping between the concat doc and the cell docs

At first I thought... I might be able to get away with one because PSES does a great job of handing untitled files (we even support full debugging experiences of untitled files!) which are similar to these Notebook cell files.
But then I thought the second approach was what I was going to have to do because diagnostics and References weren't working! However, looking at PSES's code closer, I noticed a few places we special case file and untitled schemes:
https://github.com/PowerShell/PowerShellEditorServices/blob/5f1253e375557fe53f0cbd918fe74fe45306c9a9/src/PowerShellEditorServices/Utility/PathUtils.cs#L41
Once I included vscode-notebook in these checks, things started to "just work". I should think about how to refactor PSES to have one place where supported schemes live (and schemes that are in memory/not in memory).
At this point, most things just work. The next sections will be focused on some small things that I've noticed.
Diagnostics
All-in-all, diagnostics are "just working". They're mostly showing up when they need to and CodeActions are functioning as expected:

There was a small thing I noticed... Keep in mind, each cell is a single document. Why is this important? Because of this lint rule:
https://github.com/PowerShell/PSScriptAnalyzer/blob/master/RuleDocumentation/UseDeclaredVarsMoreThanAssignments.md

Foo is used... but from PSES's perspective, they're in a different file so it's not actually a reference of that variable.
Does this warrant switching to Concat documents? I don't think so. We can probably just disable that rule since it probably doesn't make sense in this interactive experience.
References
This is the trickiest one so far because References like everything else are a mix of state of the PSIC and the contents of the file. From within a cell references are found:

Note it looks like 2 are coming from notebook.ps1... I've yet to debug this but I wanna check to see if VS Code is sending an additional text document notification or if this is bad counting in PSES. It's possible that PSES is looking at the file system here and seeing notebook.ps1 has 2 references. I may have to figure out how to skip that one.

Anyway, if the FetchGitHubRepos function is in a different cell, Go To Definition and CodeLens do not update (in other words, it's not being treated as a reference).
Unfortunately, running the cell does not save us in this current situation because that's just a glorified copy/paste into the integrated console below so it's not known that that function comes from Cell 1.
However, in PowerShell, there are a few ways to run in-memory files so that this kind of metadata persists (this is how untitled file debugging works) so I'm going to try to mess with that to see if I can at least get these references showing up after running the cells.
The only way to do this without running would be to use the concat document, but that seems like a lot more work than getting the above working.
Completions

Keep in mind, each cell is a single document.

Completions also for the most part, "just work" since completions in PowerShell are mostly based on the state of the PSIC but some are also file based. For example:
$foo = 5

$f|

Given that script, a completion of foo will be given because PSES (well really PowerShell is handling completions using the completion API).
Now, think about 2 cells:
Cell 1:
$foo = 5

Cell 2:
$f|

Cell 2 will not have completions for foo because each cell is a different file. However, there's a simple workaround:
Run cell 1
Once cell 1 is run, the variable will show up in the state of the PSIC and will then show in completions for Cell 2.
I still didn't think the concat doc was worth it since Notebooks are about running cells
Misc client side things
Running cells
As mentioned above, today the cell is just copy pasted into the integrated terminal below. This allows the state of the PowerShell language features provided by PSES to be updated.
I will investigate how I can run the cells with extra metadata so that references work better.
Why not show the output above?
It's hard. We could show the output stream in the cell output, but PowerShell isn't just about the output stream (there's a Verbose, Debug, Progress, Error, Warning, Output, Information stream) so it'd be quite challenging to wire those all up...
Why not have a separate "hidden" PowerShell instance for the Notebook UI?
We did this in .NET Interactive... it works well... but then the state of the PSIC and PSES will be different than this "hidden PowerShell instance". It's not ideal.
Where's the code?
The code will live in these branches. Diff it with the PowerShell org's master branch to see full changes:
vscode-powershell: https://github.com/TylerLeonhardt/vscode-powershell/tree/notebook-ui-support
PowerShellEditorServices: https://github.com/TylerLeonhardt/PowerShellEditorServices/tree/notebook-ui-support
What else am I looking for:
Features:

So way that I can say "this ps1 file should be opened as a Notebook but not this ps1 file"

Bugs:

https://github.com/microsoft/vscode/issues/95028
https://github.com/microsoft/vscode/issues/96078
https://github.com/microsoft/vscode/issues/96105
https://github.com/microsoft/vscode/issues/96101

Other notes
I think PowerShell will be an outlier (it usually is in most things...) when it comes to not needed the concat document. If other languages don't have the same kind of state management, they'll have no other option.
Amazing work to those involved with this support (especially @rebornix) it's been a great dev experience so far!

I think we are beyond this at this point

I think we are beyond this at this point


Add checker for duplicate properties

Discussion in #33.

Switched to a Haddock comment, thanks!

Looks great, thanks again, and have a great weekend!


Custom shulker end ship datapack replacement doesn't work. Better end conflicts with terraforged. Both on forge.

When I use datapack to replace old end ship,  it corrupts and has missing blocks.
I did issue report on terraforged mod page however they said that both two errors are not thier problem.
They said that terraforged doesn't even change end generation.
Ship issue with screenshots and evidence.
https://github.com/TerraForged/tracker/issues/4
Terraforged creator also said same about world gen issue.
He explained that biome didn't generate because creator didn't make support for terraforged.
I tried energetic expansion on forge with terraforged with same version for forge.
And it worked.
Evidence with his screenshots and explanation.
https://github.com/TerraForged/tracker/issues/3
Please fix it

P.S. Version 1.16.4

P.S. Version 1.16.4

This is Fabric version, all Forge issues need to be reported to Forge version tracker

This is Fabric version, all Forge issues need to be reported to Forge version tracker

Why do i lose time to write changelogs and informations on the mod page if people don't read them :/ ? It is clearly stated that support for Terraforged was added in BetterEnd 1.4.0 changelog, and there is a compatiblity section on the mod page you should read. Please, the next time report issue on the BetterEnd issue page, not on other mods one, unless you absolutely sure that the issue is coming from their side. Here there was no reason to think that the datapack structure not creating properly was an issue with terraforged, and the explanation is very simple. The datapack was made for Fabric, in Forge we used a different mod id and so modded blocks registered with Fabric mod id were not recognized

Why do i lose time to write changelogs and informations on the mod page if people don't read them :/ ? It is clearly stated that support for Terraforged was added in BetterEnd 1.4.0 changelog, and there is a compatiblity section on the mod page you should read. Please, the next time report issue on the BetterEnd issue page, not on other mods one, unless you absolutely sure that the issue is coming from their side. Here there was no reason to think that the datapack structure not creating properly was an issue with terraforged, and the explanation is very simple. The datapack was made for Fabric, in Forge we used a different mod id and so modded blocks registered with Fabric mod id were not recognized


[epoch_processing] Implement process_effective_balance_updates

https://github.com/lambdaclass/lambda_ethereum_consensus/blob/66059de3653c17ae620ce2aca2db61c558057f6a/docs/specs/beacon-chain.md?plain=1#L2517-L2530
Acceptance criteria:

Run corresponding spec tests by removing https://github.com/lambdaclass/lambda_ethereum_consensus/blob/66059de3653c17ae620ce2aca2db61c558057f6a/test/spec/runners/epoch_processing.ex#L17

I would like to take this up. Not sure though which file this should go into?

Hi @ayushm2003 . Maybe lib/lambda_ethereum_consensus/state_transition/epoch_processing.ex?

hey,

when running the test make spec-test-epoch_processing, it gives the following output -

mix test --no-start --only runner:epoch_processing
Compiling 1 file (.ex)
Excluding tags: [:test]
Including tags: [runner: "epoch_processing"]

***********************************************************************************************************************************.******************************.
Finished in 0.1 seconds (0.1s async, 0.00s sync)
2 properties, 9234 tests, 0 failures, 9073 excluded, 161 skipped

Randomized with seed 702167

Not sure if the function is being tested since all of them are either excluded or skipped? (I have removed the relevant function from disabled_handlers)

Should we define presets like EFFECTIVE_BALANCE_INCREMENT in the config file?

A good way to test if those are run is just using a good ol' assert false in the relevant test function. Another way is to add the --trace flag to the mix test command, like this: mix test --trace --no-start --only runner:epoch_processing. Also, keep in mind there are not that many tests for effective_balance_updates (I think there's one in minimal and one in mainnet).
You can access the preset/config constants like ChainSpec.get("SLOTS_PER_EPOCH")


[Feature Request] Iterate/list loaded chunks

I added new feature with ZS - planets where Blood Magic meteors spawning naturally and bombarding planet surface.
The problem is that ZS can't properly find random chunk - I used "hack" with onBlockNeighborNotify to randomly chose one of the chunks when this event happens, since planet is full of liquids that regularly raising this event.
So, my request is to list all loaded chunks in a world or pick random loaded chunk in a world.
P.S. List all loaded chunks would be also handy for creating custom commands to detect mechanics that keeps chunk loaded due bugs.

I have to expose many things about chunk for it...
Hmm, I would not like to make such repetitive labour, you can trace #45. I'm working for it now. Close this issue for now.


Bugsnag causing events to be duplicated in iOS UIWebView

We have a Cordova-based mobile app using UIWebView on iOS. We use the bugsnag-js library to monitor for errors in this app, and I've recently upgraded to the v7 module. Since then, I've been noticing some odd app behaviors regarding events -- both events not being fired correctly, and also being duplicated in certain circumstances. The net result is that our app isn't working properly.
I've found that if I comment out loading the Bugsnag JS, then the issues go away. Beyond that, I'm not certain what's triggering this behavior, or how to dig more deeply to figure out why this is happening. For two specific examples of troubles:

We have a <input type="file"> to allow image uploads. After selecting an image, I'm finding that the change event is not consistently fired when Bugsnag is loaded. When Bugsnag is not loaded, it fires 100% of the time.
We have a jQuery Mobile multi select element on screen. Tapping on an item to select it is triggering duplicate click events to be fired, causing it to be selected and unselected immediately. When Bugsnag is not loaded, only a single click event is fired per tap and everything works perfectly.

In the debugger, I can see that the events are all coming via inline-script-content.js, so I tried to adjust the configuration to disable this module with the following:
Bugsnag.start({ trackInlineScripts: false })

With the above config, the events are working properly. So, clearly, there seems to be some type of issue with this module and UIWebView on iOS. Any suggestions on how to further debug and fix this issue? We are using jQuery and jQuery Mobile in the app to wire up the event handlers.

Hi @dpolivy
We don't officially support the use of Cordova yet, it is something that is on our roadmap but have no timeline for this yet. Thanks for reporting this so we will be sure to look into when we come to adding support for Cordova. For now I would suggest sticking with the workaround as you have found.


New user icon seems to be auto grabbed from facebook

Description
Okay so i just started using jellyseerr and i added a new user filling in their email/password and jellyseerr automatically added a photo only used on their facebook account?
Not sure why this happened but I absolutely do not want jellyseerr interacting with facebook or pinging facebook telling them a user has been made on my server.
The last thing i want to do is link jellyfin to facebook in any way….. (personally i dont use facebook and do not appreciate their abuse of privacy / peoples data )
How do i stop this from happening? I dont want any lookup or reporting to remote servers when an email/user is added
Version
1.7.0
Steps to Reproduce
Users>create local user>create
Screenshots
No response
Logs
No response
Platform
desktop
Device
Laptop
Operating System
Windows 10
Browser
Librewolf
Additional Context
No response
Code of Conduct

[X] I agree to follow Jellyseerr's Code of Conduct

um what? jellyseerr in no way interacts with facebook. The profile photo comes from gravatar for local users.....

um what? jellyseerr in no way interacts with facebook. The profile photo comes from gravatar for local users.....

Thank you you that information and your reply, i was a bit alarmed by the fact it added the facebook profile photo to the user account automatically, as far as we know facebook is the only place that photo is used so it was concerning, ill have to figure out how exactly it is linked to gravatar.

Thank you you that information and your reply, i was a bit alarmed by the fact it added the facebook profile photo to the user account automatically, as far as we know facebook is the only place that photo is used so it was concerning, ill have to figure out how exactly it is linked to gravatar. (Might have made an account and forgot or something im not sure as its not my email)

you can just go to gravatar.com and type in your email

Thank you you that information and your reply, i was a bit alarmed by the fact it added the facebook profile photo to the user account automatically, as far as we know facebook is the only place that photo is used so it was concerning, ill have to figure out how exactly it is linked to gravatar. (Might have made an account and forgot or something im not sure as its not my email)

you can just go to gravatar.com and type in your email

Is there a way to disable gravatar image pulling?
I really do not want any information leaving the server about users and making the request to gravatar is basically reporting email’s associated with this instance

you can just go to gravatar.com and type in your email

Is there a way to disable gravatar image pulling?
I really do not want any information leaving the server about users and making the request to gravatar is basically reporting email’s associated with this instance

Just change your avatar on gravatar.

you can just go to gravatar.com and type in your email

Is there a way to disable gravatar image pulling?
I really do not want any information leaving the server about users and making the request to gravatar is basically reporting email’s associated with this instance

Just change your avatar on gravatar or delete your gravatar account. It is not reporting any email to out if you dont have a gravatar account
It seems you have a slight misunderstanding on how gravatar works. Jellyseerr DOES NOT request gravatar WITH ANY INFORMATION to outside of your server. Your gravatar picture is a public avatar. That anyone can visit if they just do https://gravatar.com/ {youremail without the @} so for example if your email is<EMAIL_ADDRESS>then your gravatar link is https://gravatar.com/johnDoe. This picture is directly displayed as your profile picture. No information about your server leaves your server. Its just a picture.
Its public information. Because you have a gravatar account anyone can go to that link. AND THATS what is displayed as your avatar. IT IS NOT REQUESTING gravatar. There is no information about the server leaving your server. Its the same as you writing the direct link to your github avatar in your local jellyfin server as a profile picture.

That would be the same would it not? Just for privacy concerns, the server searching x user its pretty easy to tell x instance searched for x accounts so the people who own those accounts are probably linked to x instance
Is it possible to edit a file to remove the gravatar search for an icon?

I still dont understand what you mean here:
That would be the same would it not? Just for privacy concerns, the server searching x user its pretty easy to tell x instance searched for x accounts so the people who own those accounts are probably linked to x instance
Do you want to prevent your server users from seeing the avatars? Because otherwise NO your server information is NOT REACHING GRAVATAR. All its doing is fetching the pic. From a public link. There is no requests done to gravatar. There is no api used (this would yes send information about your server to whaterver service). IT is not the same as just grabbing a picture from a link.
There is no server details leaked.
No privacy concerns because YOUR SERVER DETAILS ARE NOT SHARED.
Your other users dont see your pictures either only admin sees other users.
Any picture on gravatar is because you added it.
I REPEAT ONCE AGAIN, There is no server details leaked or sent to gravatar. That is not how it works. Its the same as you right clicking on your github picture and saving it. Does github know you saved that? no.

If a random server/ip saves the picture from github it has to be requested or it wouldn’t save the picture, it has to connect from the server to get the picture.
That does leak data in a small way because it is possible to see that an ip saved that picture.
it just seems a bit irrelevant to me, so I don’t really want jellyseerr to save the photo from a remote server at all, ideally id like to just be able to specify a random username and password without adding email at all but it says emails required to make a new user (which is also pointless for local accounts where the email isnt used for anything. )
is there any way to just disable the email requirement or disable it from saving the image at all?

If a random server/ip saves the picture from github it has to be requested or it wouldn’t save the picture, it has to connect from the server to get the picture.

Once again. That is not how its working. There is no saving. Its literally a link. So no no ip is sent to them. IMAGES ARENT SAVED. ITS JUST A LINK. A TEXT.

If its a link to the image live that pings the server every time its loaded thats even worse really

If its a link to the image live that pings the server every time its loaded thats even worse really

There is no information leaked. If its worse, then that makes no sense? youre already pinging several stuff, like tmdb, github and stuff. When you use jellyfin or plex or emby and you have an avatar youre literally pinging them. There is no information or ip leaked. I cannot emphasise that enough. If you dont believe me, the code is open source. Check it.

Anyone could be loading images from random content but why would someone else be loading your user icon? Thats my point. I dont want it to load the user icon at all.
It is also impossible for it to just magic onto the jellyseerr instance it has to have been requested at some point and if its just a link requesting it every time its loaded that would be worse because its then loading the image over and over from gravatar and deleting gravatar (again its not my email so not my account) wouldn’t stop it from making the request? Gravatar doesn’t just magically know who wants that image.
But either way, i just would like to disable it because i am uncomfortable with anything related to users leaving the server in any way even if its just jellyseerr looking up the user icon, regardless of if we agree on whether its a risk is irrelevant, could you tell me what file i would need to edit to remove it or remove the email requirement so i can just use a username? it should be fairly simple to just remove that line of code i would think

If you still insist your server details are being leaked (even though it is not), feel free to fork jellyseerr and remove any gravatar related code. It's licensed as MIT.

It is also impossible for it to just magic onto the jellyseerr instance it has to have been requested at some point and if its just a link requesting it every time its loaded that would be worse because its then loading the image over and over from gravatar and deleting gravatar (again its not my email so not my account) wouldn’t stop it from making the request? Gravatar doesn’t just magically know who wants that image.

I know it's not magic. That is not what I meant. But it's not making the request as your email. It's not leaking any server info. From that argument, you should also remove tmdb api, and github api, because they are also getting information about your server and ip. But I digress.

You can remove the gravatarUrl generation for local users in server > routes > user. Just replace the avatar in there with any images.

I am just worried about user information being exposed as its making external requests for user icons which are specific to the users and that is identifiable information, unlike the icons and content related to random media anyone in the world could be loading and while thats not leaking server information it is exposing users of the server in a very small and obscure way.
How does the tmbd api leak information?

Tmdb api requests, github api requests, all these would log everytime you request them.
When you make requests to the TMDB API, your IP address is typically included in the request headers. This is a standard practice for web APIs to identify and respond to client requests. However, TMDB, like most reputable services do not log them.
In the case of gravatar you're not even requesting them to return the avatar link for your email. Because the has is figured out locally and just adding it. So if you don't have a gravatar account, what gravatar receives is a nonsense hash that returns a placeholder image.

@AncientMystic also you can use jellyfin/emby users as users. It doesn't have any emails to begin with. And it uses your jellyfin/emby set avatar. Instead of local users I mean. Easier that way than to edit the codebase


fix the hamburger menu for mobile screens

Hi @fykaa I would to work on this issue. Can you please assign it to me.

Hey @fykaa , I would like to work on it.
Can you assign this to me?

@21omkarsase will assign if the first person to comment fails to send pr within the time.

@fykaa Can you please elaborate what needs to be done like do I need to fix the css or make the hamburger menu working?


oracle date type conversion issue

Hello, testing oracle->crdb migration using oracle sample HR schema - csv import seems to work fairly well, except for date conversion -
oracle column
HIRE_DATE                                 NOT NULL DATE
Crdb column
hire_date      | DATE
ex
oracle
select hire_date, to_char(hire_date,'DD-MON-YYYY') sdate from employees
2  where rownum  = 1;
HIRE_DATE SDATE

17-JUN-03 17-JUN-2003
crdb
HR@:26257/hr> select hire_date from employees limit 1;
hire_date
2017-06-03 00:00:00+00:00
im assuming i have a formatting issue on my end but havent found it yet - any idea? thanks

here is the import statement - different table but same issue
HR@:26257/hr> import into job_history
csv data (
'nodelocal://self/job_history.csv') with skip = '1', nullif = '';
ERROR: nodelocal://self/job_history.csv: error parsing row 3: parse "start_date" as DATE: parsing as type date: field day value 97 is out of range (row: "101,21-SEP-97,27-OCT-01,AC_ACCOUNT,110")

Hi @seadba, thanks for reaching out. This is the CockroachDB docs repo (for filing issues re: docs bugs, additions, etc.). For help troubleshooting, you can reach out on Slack, in the Forum, or contact Support.


set up comment spec

Codecov Report

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

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

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


k3s: 1.26.0+k3s2 -> 1.26.1+k3s1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for k3s is: A lightweight Kubernetes distribution
meta.homepage for k3s is: https://k3s.io
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.26.1+k3s1 with grep in /nix/store/33r73m5dvc0fbcqr8cmbcicgip4qq8f7-k3s-1.26.1+k3s1
found 1.26.1+k3s1 in filename of file in /nix/store/33r73m5dvc0fbcqr8cmbcicgip4qq8f7-k3s-1.26.1+k3s1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
k3s

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/33r73m5dvc0fbcqr8cmbcicgip4qq8f7-k3s-1.26.1+k3s1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A k3s https://github.com/r-ryantm/nixpkgs/archive/e28154dccd325b424ca7a164fb9a1360636216ee.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/33r73m5dvc0fbcqr8cmbcicgip4qq8f7-k3s-1.26.1+k3s1
ls -la /nix/store/33r73m5dvc0fbcqr8cmbcicgip4qq8f7-k3s-1.26.1+k3s1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

k3s

Maintainer pings
cc @euank @Mic92 for testing.

Result of nixpkgs-review pr 213071 run on aarch64-linux 1

1 package failed to build:

k3s

Result of nixpkgs-review pr 213071 run on aarch64-linux 1

1 package built:

k3s


Seeders and leechers?

Is it possible to get seeders and leechers data for an info hash?

You can surely do this using call method, check the aria2 endpoitns


Add generic method to test _include parameter

Suggest adding PractitionerRole search test for _include parameter: PractitionerRole:practitioner on the identifier parameter.
Since the conformance level is SHOULD in AU Core, the additional and optional search test can be repeated if any PractitionerRole resources have a reference to Practitioner.
AU Core version: 1.0.0-ballot

Check why generator doesn't pick up information about this search parameter: https://build.fhir.org/ig/hl7au/au-fhir-core/StructureDefinition-au-core-practitionerrole.html#mandatory-search-parameters.
SHOULD support chained searching of practitioner canonical identifier practitioner.identifier (e.g. practitioner.identifier=[system|][code])

@StephThaoOng Hello Steph
This test already exists in the suite (1.22.08)
Could you confirm it?

@StephThaoOng Hello Steph This test already exists in the suite (1.22.08) Could you confirm it?

Hi @projkov isn't test 1.22.08 testing this conformance requirement:

SHALL support searching using the practitioner search parameters:
SHOULD support chained searching of practitioner canonical identifier practitioner.identifier (e.g. practitioner.identifier=[system|][code])

Also I'm still confused why the test is failing since there are resources on the Sparked FHIR server that have PractitionerRole.practitioner populated.
So the result set from:
GET [base]/PractitionerRole?identifier=http://ns.electronichealth.net.au/id/medicare-provider-number|2449211J&_include=PractitionerRole:practitioner
should include the Practitioner resource(s). This was verified using Postman.
Am I on the right track? Perhaps we can discuss during sprint meeting.

@StephThaoOng Hello Steph
I found that we didn't have _include tests, except including medication in MedicationRequest tests. At the current state, the generator gets information from the CapabilityStatement on what params should be used in _include or _revinclude (PractitionerRole:practitioner for PractitionerRole test group is too). But unfortunately in tests only MedicationRequest:medication is processed.
We have 2 options:

Add another specific function for testing PractitionerRole:practitioner (similar to the MedicationRequest:medication)
Add a generic function for testing any included resources.

It will be nice to work on the second option because it can help us in the future, when (perhapse?) we need to add new include tests.

H

@StephThaoOng Hello Steph
I found that we didn't have _include tests, except including medication in MedicationRequest tests. At the current state, the generator gets information from the CapabilityStatement on what params should be used in _include or _revinclude (PractitionerRole:practitioner for PractitionerRole test group is too). But unfortunately in tests only MedicationRequest:medication is processed.
We have 2 options:

Add another specific function for testing PractitionerRole:practitioner (similar to the MedicationRequest:medication)
Add a generic function for testing any included resources.

It will be nice to work on the second option because it can help us in the future, when (perhapse?) we need to add new include tests.

Hi @projkov yes I agree with the second approach. Can you please edit the issue title, or close this one and create a new issue that better describes the enhancement request? Thanks :)

23.09.2024 decided on the team meeting to implement a generic method to test "_include" parameter

@StephThaoOng Hello Steph
Requests with the _include parameter are available for search tests for MedicationRequest:medication and PractitionerRole:practitioner. You can check them in production. Please note that this is not a separate test but is included in the existing search tests, such as searches by _id, etc.
Just a note:
Requests for MedicationRequest:medication are created by static code through special cases. I have mentioned this issue in related tickets. We will remove this special case once the SearchParameter becomes available in the AU Core Implementation Guide.

@StephThaoOng Hello Steph
Requests with the _include parameter are available for search tests for MedicationRequest:medication and PractitionerRole:practitioner. You can check them in production. Please note that this is not a separate test but is included in the existing search tests, such as searches by _id, etc.
Just a note: Requests for MedicationRequest:medication are created by static code through special cases. I have mentioned this issue in related tickets. We will remove this special case once the SearchParameter becomes available in the AU Core Implementation Guide.

@projkov thanks Pavel.
Before we close, do you think it's worthwhile adding in the "About" tab for the PractitionerRole tests "If any PractitionerRole resources use external references to Practitioner resources, the search will be repeated with _include=PractitionerRole:practitioner"?

@projkov I suggest to introduce a separated test for _include.
It is important to make test granular to easily identify what is working and what is not.

@StephThaoOng Hello Steph.
Could you please check it on production?

Update about section:
...and the _include= PractitionerRole:practitioner parameter

According to the narrative text, _include parameter should used with these search params:

MedicationRequest

patient
patient+intent
patient+intent+status
_id
patient+intent+authoredon

PractitionerRole

identifier
practitioner
_id
specialty

@StephThaoOng Hello Steph
Today's changes are available in the production environment. Available changes:

Updated about section;
If the test does not pass then it will be skipped, and not failed;
I have checked the narrative for PractitionerRole and MedicationRequest and have updated the generator. Now we have the next new tests:

MedicationRequest

patient
patient+intent
patient+intent+status
_id
patient+intent+authoredon

PractitionerRole

identifier
practitioner
_id
specialty


Remove references to DocumentDB (old name)

Hi team,
Azure DocumentDB is now Azure CosmosDB SQL. Therefore, to maintain consistency, this project should make that clear in all references to Azure webpages, documentation, and eventually in the API and project name as well.
https://azure.microsoft.com/en-us/blog/dear-documentdb-customers-welcome-to-azure-cosmos-db/

@brunoborges thanks for reminding. Actually we're aware of the renaming thing at very beginning of this project. Main reason we keep using documentdb term is because java sdk's name keeps as azure-documentdb https://github.com/Azure/azure-documentdb-java. To avoid confusing user, this module name aligned with sdk name. Any plan about the sdk renaming?

pinging @selvasingh on the status of renaming DocumentDB SDK.

Still some words related to documentdb excluding sdk side from our repo, need to clean up all of them.
Search Result


Swift UITableView with Custom Subclasses of UI*ViewCell

hello
Where is the prototypereuseidentifier ? I can see identifier in my cell (added to a tableview), but I can t seem to find online any information about a prototypereuseindetifier, is it the same thing ?

If you're using prototype cells (in storyboards), you're going to use the constructor with prototypeReuseIdentifier, otherwise, if you would have otherwise used a call to UITable/CollectionView registerClass: forCellWithReuseIdentifier you should use cellReuseIdentifier. #16 has a long discussion on this as well.
For more information, check out this helpful post on working with UITableViews which can explain when to use either.


add TreeItem API/provider

TreeItems got added to 1.13
https://code.visualstudio.com/docs/extensionAPI/vscode-api#TreeItem
https://code.visualstudio.com/docs/extensionAPI/extension-points#_contributesviews

Just to note, I think the existing "documentSymbols" is almost exactly the right API to provide an "Outline View" as used in Atom, IntelliJ, Eclipse, VisualStudio. The only thing it needs is one small addition: it currently emits containerName which is enough to reconstruct the outline view in most cases, but it should also emit containerLocation so that outline view can be reconstructed in all cases.
Anyway, I don't actually understand what "TreeItem" is for or what it looks like, and didn't get enough understanding from those two links. But I wanted to flag that if it's a tree for outline view, then I think we should tread carefully.

It's for any kind of tree that might be useful in the Explorer view: alternative file explorer, project navigation, convenient command pane, etc.  General purpose.

However, I should clarify that I don't necessarily think this should be a core part of the LSP.  Since this is a VS Code UI feature and not necessarily something that's generally applicable to editors, not sure if it makes sense as a first-class member of the protocol.
That's just my opinion, though, I'd be fine if there was first-class support for tree views since I'm going to implement in my language server either way :)

fyi @jrieken

Yeah it is basically a vscode only feature but I think it would actually be quite useful in any generic editor. And as servers & clients both don't need to support all features I don't think it would hurt to add it. It isn't a big deal if it doesn't get added as we can add it with custom functions. Usage example for this feature is for example a nodejs dependency viewer like in the visual studio plugin or a dependency viewer (also like in visual studio), just this could be used and customized by language server authors.

I fully support previous comment from @daviwil .
The TreeItem from VSCode, as I understand it, doesn't seem at all related to programming languages but more an advanced widget in VSCode for rendering trees (like other UI toolkits may already have). It's more specific to VSCode's UI layer than a feature of an IDE related to a programming language. I don't think it makes sense in the LSP.
There are already multiple solutions to build a nice and complete tree from the documentSymbols: using containers, using range inclusion... There is no need for a new concept to provide tree, the LSP already covers that properly.

I don't think it would hurt to add it.

Adding stuff that's not consistent is making the API a bit messier and then harder to evolve and maintain. We should avoid being to greedy and keep focused on finding operations or notifications that makes sense for an IDE to request to a language.

Just to note, I think the existing "documentSymbols" is almost exactly the right API to provide an "Outline View"

Having something like an outline view is tracked here: https://github.com/Microsoft/vscode/issues/5605. Indeed, there are already extensions using the tree API and api command to get to an outline view. I think that is a valid stepping-stone until we are tackling that in the provider-spirit. By that I mean, will spec a new DocumentSymbolProvider that supports a tree of symbols. The UI implementation is then up to the editors (as we have done it already with things like reference search or completion) and will most likely a tree but also something like a break-crump view atop of a file will be possible when having the right data-api spec'd.

The TreeItem from VSCode, as I understand it, doesn't seem at all related to programming languages but more an advanced widget in VSCode for rendering trees

That is correct. The TreeItem represents what is shown in the UI, where the tree data provider is the source of item that are to be rendered (in a tree structure)

So it seems like we all agree it's a vscode request ;) Should we close this issue?
@jrieken Eclipse IDE already support a hierarchical/tree-based outline populated with document symbols from LSP. See https://git.eclipse.org/r/plugins/gitiles/lsp4e/lsp4e/+/0.2.0/org.eclipse.lsp4e/src/org/eclipse/lsp4e/outline/SymbolsModel.java#34 for an example of how to use the ranges returned by the documentSymbols to create a tree-like structure. And breadcrumbs are a great idea, I've tracked it here for addition in Eclipse IDE https://bugs.eclipse.org/bugs/show_bug.cgi?id=518182

Should we close this issue?

Yeah, I'd say it will be better to track this in the vscode-repo. Ideas is to branch out from the catch-all-issue, with define provider and data model, define bread-crumbs ui, define tree ui

@mickaelistria I'm looking at the eclipse implementation of reconstructing outline view from documentSymbols.
This implementation trusts that the "location.range" returned from documentSymbols will be the range of the entire symbol, from the start of line 10 right up to the end of line 40 in the code below.
10:  class Foo
20: {
30:    ...
40: }

Is that really correct? My understanding was that documentSymbols returns the range purely of the symbol identifier itself, purely the range of the Foo on line 10. I guess @bruno-medeiros had the same doubt: https://github.com/Microsoft/language-server-protocol/issues/132

Not sure it's correct conforming to the LSP spec, but at least, it works for VSCode CSS, JSon and HTML language servers. But I agree there are some good reasons to doubt about it ;)

I agree that we should close the issue and will do so.
The design goal of the LSP was to provide language 'smartness' that can be shared between tools not to implement UI that can be shared between tools. I think there is some space for shared descriptive UI as well but I would argue that this should be a different protocol than LSP.


what is the online form url in about page?

Description:
when clicking the online form on the about page, it will refresh the page.

now the form linked to https://docs.google.com/forms/d/e/1FAIpQLScsab93B7uPg389gxCNfCSgG4sMNIFk_mxDFTFF_-UC2TcSJQ/viewform


Swift dynamic cast failure

When I try to us the non NSPredicate notation I get an Swift dynamic cast failure.
#1	0x0000000103153d7e in QueryKit.< infix <A>(QueryKit.Attribute<A>, A) -> ObjectiveC.NSPredicate at /Users/dirk/Development/Test_App/dummyDB/Pods/QueryKit/QueryKit/Attribute.swift:59

Working code
var predicate = NSPredicate(format: "age == 42")
var qs2 = Person.queryset(moc).filter(predicate!)

Error code
var qs = Person.queryset(moc).filter(Person.attributes.age == 42)

Model is
Entity:Person
Attribute:age::NSNumber
Attribute:name::String

Build with mogenerator template.

@dirkfabisch Just to check, is your attribute for age an optional?
var age:Attribute<NSNumber?> {
  return Attribute< NSNumber?>("age")
}

vs
var age:Attribute<NSNumber> {
  return Attribute< NSNumber>("age")
}

It is an optional attribute
   var age:Attribute<NSNumber?> {
        return Attribute<NSNumber?>("age")
    }

The complete repository you will find here: (https://github.com/dirkfabisch/dummyDB)

@dirkfabisch Thanks for reporting this, me and @neonichu got to the bottom of this problem and I've released QueryKit 0.9.1 with a fix of using optional types with Attribute.

I think you did a very good thing, thank you very much ~ 🍻🍻🍻


Anyone get this to work within eshell?

I know only certain shells are officially supported, but anyone have any tips for making this work within Eshell?

As before, please feel free to ask usage questions on our Gitter usage-issues channel.


chore(go/adbc): bump minimum version to go1.21, update deps

Bumping the minimum go version (via go.mod) to go1.21 and updated appropriate indirect dependencies accordingly.

This is ready to review now!

I'm gonna try to replicate this for the main Arrow build and release scripts to update the Arrow Go libs directly. Thanks again @kou!


Installing Matplotlib causes version `GLIBCXX_3.4.30' not found

Using Python 3.9, Cuda 11.8, on Ubuntu 22.04 with 4090, everything works fine unless I install matplotlib in order to use the benchmarking tool, then I am presented with the following error message on any import of triton. If I then simply uninstall matplotlib, everything works again.
  File "/media/adam/shared_drive/PycharmProjects/triton_test/experimental_block_pointer.py", line 95, in <module>
    import triton
  File "/home/adam/Downloads/triton/python/triton/__init__.py", line 12, in <module>
    from .runtime import (
  File "/home/adam/Downloads/triton/python/triton/runtime/__init__.py", line 1, in <module>
    from .autotuner import (Autotuner, Config, Heuristics, OutOfResources, autotune,
  File "/home/adam/Downloads/triton/python/triton/runtime/autotuner.py", line 7, in <module>
    from ..testing import do_bench
  File "/home/adam/Downloads/triton/python/triton/testing.py", line 7, in <module>
    import triton._C.libtriton.triton as _triton
ImportError: /home/adam/anaconda3/envs/triton_test/bin/../lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /home/adam/Downloads/triton/python/triton/_C/libtriton.so)

For those who experience the 'GLIBCXX_3.4.30' not found error, see this. It appears to be an issue with conda environment not seeing the correct version of libstdc++.so.6.
https://github.com/pybind/pybind11/discussions/3453#discussioncomment-1926590


Replace Insert by Insert().Into()

as per consistency (Select().From("tableName")) the same should happen for Insert
Insert().Into("tableName").values("...")

And with explicit columns the sintax does look like that Insert().Into("tableName").Columns("column1","column2").Values("value1",value2)? What do you think?