Builder giving undexpected error/warning .drfit file

Describe the bug
Below is the log, now sure why is it happening and not sure what to make sense out of it.

[WARNING] drift_dev on lib/config/app_db/query/file_remote_details_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/file_remote_details_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class FileRemoteDetailsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/file_local_details_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/file_local_details_query.dart: Could not read tables from @DriftAccessor annotation!

Please make sure that all table classes exist.
  ╷
8 │ class FileLocalDetailsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/one_tap_login_query.dart:
line 11, column 7 of package:mytrailpals/config/app_db/query/one_tap_login_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
11 │ class OneTapLoginQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/file_upload_details_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/file_upload_details_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class FileUploadDetailsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/chat_sync_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/chat_sync_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class ChatSyncQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/file_metadata_query.dart:
line 9, column 7 of package:mytrailpals/config/app_db/query/file_metadata_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
9 │ class FileMetadataQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/image_metadata_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/image_metadata_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class ImageMetadataQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/file_transformation_links_query.dart:
line 18, column 7 of package:mytrailpals/config/app_db/query/file_transformation_links_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
18 │ class FileTransformationLinksQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/notification_stats_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/notification_stats_query.dart: Could not read tables from @DriftAccessor annotation!

Please make sure that all table classes exist.
  ╷
8 │ class NotificationStatsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/route_time_tracking_query.dart:
line 12, column 7 of package:mytrailpals/config/app_db/query/route_time_tracking_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
12 │ class RouteTimeTrackingQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/post_query.dart:
line 10, column 7 of package:mytrailpals/config/app_db/query/post_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
10 │ class PostQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/video_metadata_query.dart:
line 14, column 7 of package:mytrailpals/config/app_db/query/video_metadata_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
14 │ class VideoMetadataQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/route_timer_preferences_query.dart:
line 12, column 7 of package:mytrailpals/config/app_db/query/route_timer_preferences_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
12 │ class RouteTimerPreferencesQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/pending_upload_file_entity_link_query.dart:
line 15, column 7 of package:mytrailpals/config/app_db/query/pending_upload_file_entity_link_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
15 │ class PendingUploadFileEntityLinkQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/schema/pending_upload_file_entity_link.drift:
line 23, column 1: Expected another table constraint
   ╷
23 │ );
   │ ^
   ╵
[WARNING] drift_dev on lib/config/app_db/schema/pending_upload_file_entity_link.drift:
line 3, column 1: The imported file, `package:mytrailpals/common/enum/app_level_common_enum.dart`, does not exist or can't be imported.
  ╷
3 │ import 'package:mytrailpals/common/enum/app_level_common_enum.dart';
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
Could not deserialize DriftElementId(package:mytrailpals/config/app_db/schema/one_tap_login.drift, one_tap_login_table)
Internal error while deserializing DriftElementId(package:mytrailpals/config/app_db/schema/one_tap_login.drift, one_tap_login_table): type 'Null' is not a subtype of type 'Map<dynamic, dynamic>' in type cast at
#0      ElementDeserializer.readDriftElement (package:drift_dev/src/analysis/serializer.dart:466:60)
<asynchronous suspension>
#1      DriftResolver._restoreOrResolve (package:drift_dev/src/analysis/resolver/resolver.dart:48:16)
<asynchronous suspension>
#2      DriftResolver.resolveEntrypoint (package:drift_dev/src/analysis/resolver/resolver.dart:42:12)
<asynchronous suspension>
#3      DriftAnalysisDriver.resolveElement (package:drift_dev/src/analysis/driver/driver.dart:270:16)
<asynchronous suspension>
#4      DriftAnalysisDriver._analyzeLocalElements (package:drift_dev/src/analysis/driver/driver.dart:255:7)
<asynchronous suspension>
#5      DriftAnalysisDriver.resolveElements (package:drift_dev/src/analysis/driver/driver.dart:322:5)
<asynchronous suspension>
#6      FileAnalyzer.runAnalysisOn (package:drift_dev/src/analysis/resolver/file_analysis.dart:35:30)
<asynchronous suspension>
#7      DriftAnalysisDriver.fullyAnalyze (package:drift_dev/src/analysis/driver/driver.dart:335:22)
<asynchronous suspension>
#8      _DriftBuildRun._analyze (package:drift_dev/src/backends/build/drift_builder.dart:196:20)
<asynchronous suspension>
#9      _DriftBuildRun.run (package:drift_dev/src/backends/build/drift_builder.dart:174:9)
<asynchronous suspension>
#10     DriftBuilder.build (package:drift_dev/src/backends/build/drift_builder.dart:109:5)
<asynchronous suspension>
#11     runBuilder.buildForInput (package:build/src/generate/run_builder.dart:83:7)
<asynchronous suspension>
#12     Future.wait.<anonymous closure> (dart:async/future.dart:520:21)
<asynchronous suspension>
#13     scopeLogAsync.<anonymous closure> (package:build/src/builder/logging.dart:32:40)
<asynchronous suspension>

[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 56, column 20: Semantically equivalent date time values may be formatted differently and can't be compared directly. Consider comparing the `unixepoch()` values of the time value instead.
   ╷
56 │     AND child.date = parent.date
   │                    ^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 59, column 5: This compares two date time values lexicographically. To compare date time values, compare their `unixepoch()` value instead.
   ╷
59 │     parent.date BETWEEN :date_from AND :date_to
   │     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 95, column 20: Semantically equivalent date time values may be formatted differently and can't be compared directly. Consider comparing the `unixepoch()` values of the time value instead.
   ╷
95 │     AND child.date = parent.date
   │                    ^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 98, column 17: Semantically equivalent date time values may be formatted differently and can't be compared directly. Consider comparing the `unixepoch()` values of the time value instead.
   ╷
98 │     parent.date = :date
   │                 ^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 41, column 11: Could not find image_metadata_table. Available are: links (alias to file_transformation_links_table), processed_details (alias to file_local_details_table)
   ╷
41 │ LEFT JOIN image_metadata_table image_metadata
   │           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 8: Unknown table or view: image_metadata
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │        ^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 34: Unknown table or view: image_metadata
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │                                  ^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 42, column 42: Unknown table or view: image_metadata
   ╷
42 │     ON processed_details.file_md5_hash = image_metadata.file_md5_hash
   │                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 8: Expression has an unknown type, the generated code can be inaccurate.
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │        ^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 34: Expression has an unknown type, the generated code can be inaccurate.
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │                                  ^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/file_remote_details_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/file_remote_details_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class FileRemoteDetailsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/file_local_details_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/file_local_details_query.dart: Could not read tables from @DriftAccessor annotation!

Please make sure that all table classes exist.
  ╷
8 │ class FileLocalDetailsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/one_tap_login_query.dart:
line 11, column 7 of package:mytrailpals/config/app_db/query/one_tap_login_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
11 │ class OneTapLoginQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/file_upload_details_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/file_upload_details_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class FileUploadDetailsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/chat_sync_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/chat_sync_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class ChatSyncQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/file_metadata_query.dart:
line 9, column 7 of package:mytrailpals/config/app_db/query/file_metadata_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
9 │ class FileMetadataQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/image_metadata_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/image_metadata_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
  ╷
8 │ class ImageMetadataQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/file_transformation_links_query.dart:
line 18, column 7 of package:mytrailpals/config/app_db/query/file_transformation_links_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
18 │ class FileTransformationLinksQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/notification_stats_query.dart:
line 8, column 7 of package:mytrailpals/config/app_db/query/notification_stats_query.dart: Could not read tables from @DriftAccessor annotation!

Please make sure that all table classes exist.
  ╷
8 │ class NotificationStatsQuery extends DatabaseAccessor<AppDatabase>
  │       ^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/query/route_time_tracking_query.dart:
line 12, column 7 of package:mytrailpals/config/app_db/query/route_time_tracking_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
12 │ class RouteTimeTrackingQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/post_query.dart:
line 10, column 7 of package:mytrailpals/config/app_db/query/post_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
10 │ class PostQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/video_metadata_query.dart:
line 14, column 7 of package:mytrailpals/config/app_db/query/video_metadata_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
14 │ class VideoMetadataQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/route_timer_preferences_query.dart:
line 12, column 7 of package:mytrailpals/config/app_db/query/route_timer_preferences_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
12 │ class RouteTimerPreferencesQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/query/pending_upload_file_entity_link_query.dart:
line 15, column 7 of package:mytrailpals/config/app_db/query/pending_upload_file_entity_link_query.dart: Could not read tables from @DriftAccessor annotation!
Please make sure that all table classes exist.
   ╷
15 │ class PendingUploadFileEntityLinkQuery extends DatabaseAccessor<AppDatabase>
   │       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/schema/pending_upload_file_entity_link.drift:
line 23, column 1: Expected another table constraint
   ╷
23 │ );
   │ ^
   ╵
[WARNING] drift_dev on lib/config/app_db/schema/pending_upload_file_entity_link.drift:
line 3, column 1: The imported file, `package:mytrailpals/common/enum/app_level_common_enum.dart`, does not exist or can't be imported.
  ╷
3 │ import 'package:mytrailpals/common/enum/app_level_common_enum.dart';
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
Could not deserialize DriftElementId(package:mytrailpals/config/app_db/schema/one_tap_login.drift, one_tap_login_table)
Internal error while deserializing DriftElementId(package:mytrailpals/config/app_db/schema/one_tap_login.drift, one_tap_login_table): type 'Null' is not a subtype of type 'Map<dynamic, dynamic>' in type cast at
#0      ElementDeserializer.readDriftElement (package:drift_dev/src/analysis/serializer.dart:466:60)
<asynchronous suspension>
#1      DriftResolver._restoreOrResolve (package:drift_dev/src/analysis/resolver/resolver.dart:48:16)
<asynchronous suspension>
#2      DriftResolver.resolveEntrypoint (package:drift_dev/src/analysis/resolver/resolver.dart:42:12)
<asynchronous suspension>
#3      DriftAnalysisDriver.resolveElement (package:drift_dev/src/analysis/driver/driver.dart:270:16)
<asynchronous suspension>
#4      DriftAnalysisDriver._analyzeLocalElements (package:drift_dev/src/analysis/driver/driver.dart:255:7)
<asynchronous suspension>
#5      DriftAnalysisDriver.resolveElements (package:drift_dev/src/analysis/driver/driver.dart:322:5)
<asynchronous suspension>
#6      FileAnalyzer.runAnalysisOn (package:drift_dev/src/analysis/resolver/file_analysis.dart:35:30)
<asynchronous suspension>
#7      DriftAnalysisDriver.fullyAnalyze (package:drift_dev/src/analysis/driver/driver.dart:335:22)
<asynchronous suspension>
#8      _DriftBuildRun._analyze (package:drift_dev/src/backends/build/drift_builder.dart:196:20)
<asynchronous suspension>
#9      _DriftBuildRun.run (package:drift_dev/src/backends/build/drift_builder.dart:174:9)
<asynchronous suspension>
#10     DriftBuilder.build (package:drift_dev/src/backends/build/drift_builder.dart:109:5)
<asynchronous suspension>
#11     runBuilder.buildForInput (package:build/src/generate/run_builder.dart:83:7)
<asynchronous suspension>
#12     Future.wait.<anonymous closure> (dart:async/future.dart:520:21)
<asynchronous suspension>
#13     scopeLogAsync.<anonymous closure> (package:build/src/builder/logging.dart:32:40)
<asynchronous suspension>

[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 56, column 20: Semantically equivalent date time values may be formatted differently and can't be compared directly. Consider comparing the `unixepoch()` values of the time value instead.
   ╷
56 │     AND child.date = parent.date
   │                    ^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 59, column 5: This compares two date time values lexicographically. To compare date time values, compare their `unixepoch()` value instead.
   ╷
59 │     parent.date BETWEEN :date_from AND :date_to
   │     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 95, column 20: Semantically equivalent date time values may be formatted differently and can't be compared directly. Consider comparing the `unixepoch()` values of the time value instead.
   ╷
95 │     AND child.date = parent.date
   │                    ^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 98, column 17: Semantically equivalent date time values may be formatted differently and can't be compared directly. Consider comparing the `unixepoch()` values of the time value instead.
   ╷
98 │     parent.date = :date
   │                 ^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 41, column 11: Could not find image_metadata_table. Available are: links (alias to file_transformation_links_table), processed_details (alias to file_local_details_table)
   ╷
41 │ LEFT JOIN image_metadata_table image_metadata
   │           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 8: Unknown table or view: image_metadata
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │        ^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 34: Unknown table or view: image_metadata
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │                                  ^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 42, column 42: Unknown table or view: image_metadata
   ╷
42 │     ON processed_details.file_md5_hash = image_metadata.file_md5_hash
   │                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 8: Expression has an unknown type, the generated code can be inaccurate.
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │        ^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 34: Expression has an unknown type, the generated code can be inaccurate.
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │                                  ^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[INFO] 1m 37s elapsed, 705/705 actions completed.
[INFO] 1m 39s elapsed, 721/721 actions completed.
[INFO] 1m 41s elapsed, 737/739 actions completed.
[INFO] 1m 42s elapsed, 737/739 actions completed.
[INFO] 1m 43s elapsed, 749/765 actions completed.
[INFO] 1m 47s elapsed, 749/765 actions completed.
[INFO] 1m 48s elapsed, 770/786 actions completed.
[INFO] 1m 49s elapsed, 794/809 actions completed.
[INFO] 1m 50s elapsed, 808/823 actions completed.
[INFO] 1m 51s elapsed, 825/840 actions completed.
[INFO] 1m 52s elapsed, 850/865 actions completed.
[INFO] 1m 53s elapsed, 864/873 actions completed.
[INFO] 1m 54s elapsed, 870/873 actions completed.
[INFO] Running build completed, took 1m 55s

[INFO] Caching finalized dependency graph...
[INFO] Caching finalized dependency graph completed, took 614ms

[SEVERE] const_to_enum_generator:constToEnumBuilder on lib/src/rust/frb_generated.web.dart (cached):

This builder requires Dart inputs without syntax errors.
However, package:mytrailpals/src/rust/frb_generated.web.dart (or an existing part) contains the following errors.
frb_generated.web.dart:598:11: This requires the 'inline-class' language feature to be enabled.

Try fixing the errors and re-running the build.

[SEVERE] json_serializable on lib/src/rust/frb_generated.web.dart (cached):

This builder requires Dart inputs without syntax errors.
However, package:mytrailpals/src/rust/frb_generated.web.dart (or an existing part) contains the following errors.
frb_generated.web.dart:598:11: This requires the 'inline-class' language feature to be enabled.

Try fixing the errors and re-running the build.

[SEVERE] Failed after 1m 55s
)
[2024-11-30T20:17:54.533Z WARN C:\Users\harsh\.cargo\registry\src\index.crates.io-6f17d22bba15001f\flutter_rust_bridge_codegen-2.6.0\src\library\commands\command_runner.rs:153] command="powershell" "-noprofile" "-command" "& \"fvm\" \"--version\"" stdout= stderr=& : The term 'fvm' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.
At line:1 char:3
+ & "fvm" "--version"
+   ~~~~~
    + CategoryInfo          : ObjectNotFound: (fvm:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

[2024-11-30T20:17:54.534Z INFO C:\Users\harsh\.cargo\registry\src\index.crates.io-6f17d22bba15001f\flutter_rust_bridge_codegen-2.6.0\src\library\commands\fvm.rs:15] Has .fvmrc but no fvm binary installation, thus skip using fvm.
[2024-11-30T20:17:55.917Z WARN C:\Users\harsh\.cargo\registry\src\index.crates.io-6f17d22bba15001f\flutter_rust_bridge_codegen-2.6.0\src\library\commands\command_runner.rs:153] command="powershell" "-noprofile" "-command" "& \"fvm\" \"--version\"" stdout= stderr=& : The term 'fvm' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.
At line:1 char:3
+ & "fvm" "--version"
+   ~~~~~
    + CategoryInfo          : ObjectNotFound: (fvm:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

[2024-11-30T20:17:55.918Z INFO C:\Users\harsh\.cargo\registry\src\index.crates.io-6f17d22bba15001f\flutter_rust_bridge_codegen-2.6.0\src\library\commands\fvm.rs:15] Has .fvmrc but no fvm binary installation, thus skip using fvm.
[0.8s] Parse
  └── [0.7s] Cargo expand & syn parse
  └── [0.0s] Parse HIR
  └── [0.0s] Parse MIR
[0.1s] Generate
[130.3s] Polish
  └── [126.9s] Run Dart build_runner
  └── [1.4s] Run Dart fix
  └── [1.7s] Run Dart formatter
  └── [0.2s] Run Rust formatter
  └── [0.0s] Auto upgrade

Version
drift_dev: ^2.16.0
 drift: ^2.16.0

Regarding the "could not read tables" error, are you listing a table that was defined from a .drift file in the tables list? That is unsupported because the type doesn't exist at the time drift's analyzer runs (given that the table class still needs to be generated at that step). If you want to include tables from a .drift file in a Dart DAO, you need to use the include parameter on the annotation.

line 23, column 1: Expected another table constraint

This is probably due to a trailing comma in the table definition, which is not allowed.

Could not deserialize DriftElementId

This one is kind of interesting, can you post the one_tap_login_table definition? From the line numbers, it also sounds like you're on an older drift_version though - can you try upgrading and see if the error still happens on the latest version?
The error about a table being unknown is probably due to a missing import statement in the drift file itself.
I also recommend checking the datetime errors - if you have the option to store datetimes as text enabled, it's better to do unixepoch(child.date) = unixepoch(parent.date) for comparisons instead of comparing the unparsed datetime values.

Thank you, while most of issues seems to have resolved, still few persist.
[WARNING] drift_dev on lib/config/app_db/schema/one_tap_login.drift:
line 1, column 1: The imported file, `package:mytrailpals/common/enum/app_level_common_enum.dart`, does not exist or can't be imported.
  ╷
1 │ import 'package:mytrailpals/common/enum/app_level_common_enum.dart';
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/schema/pending_upload_file_entity_link.drift:
line 3, column 1: The imported file, `package:mytrailpals/common/enum/app_level_common_enum.dart`, does not exist or can't be imported.
  ╷
3 │ import 'package:mytrailpals/common/enum/app_level_common_enum.dart';
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 41, column 11: Could not find image_metadata_table. Available are: links (alias to file_transformation_links_table), processed_details (alias to file_local_details_table)
   ╷
41 │ LEFT JOIN image_metadata_table image_metadata
   │           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 8: Unknown table or view: image_metadata
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │        ^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 34: Unknown table or view: image_metadata
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │                                  ^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 42, column 42: Unknown table or view: image_metadata
   ╷
42 │     ON processed_details.file_md5_hash = image_metadata.file_md5_hash
   │                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 8: Expression has an unknown type, the generated code can be inaccurate.
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │        ^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
[WARNING] drift_dev on lib/config/app_db/app_database.dart:
line 37, column 34: Expression has an unknown type, the generated code can be inaccurate.
   ╷
37 │        image_metadata.blur_hash, image_metadata.image_id
   │                                  ^^^^^^^^^^^^^^^^^^^^^^^

Below are the tables
import 'package:xxx/common/enum/app_level_common_enum.dart';

CREATE TABLE IF NOT EXISTS one_tap_login_table (
   id INTEGER PRIMARY KEY AUTOINCREMENT,
   email TEXT,
   password TEXT,
   fullName TEXT,
   avatar_image_local_path TEXT,
   last_logged_in_auth_type ENUM(AuthType)
);

line 1, column 1: The imported file, package:mytrailpals/common/enum/app_level_common_enum.dart, does not exist or can't be imported.

Could you check whether there's a typo or something in the drift file that might prevent us from resolving that import? If there isn't, do you have a special build configuration or something that might make that file inaccessible to drift?
Are there any syntax errors related to image_metadata_table? Or is that just not generated at all? Are you importing the drift file defining that table into your database?

Could you check whether there's a typo or something in the drift file that might prevent us from resolving that import? If there isn't, do you have a special build configuration or something that might make that file inaccessible to drift?

It seems there is no syntax error.
Also, the path of the app_level_common_enum is lib\common\enum\app_level_common_enum.dart
And path of the db file is lib\config\app_db\schema\one_tap_login.drift.
And we are importing in the one_tap_login.drift as below
import 'package:mytrailpals/common/enum/app_level_common_enum.dart';

CREATE TABLE IF NOT EXISTS one_tap_login_table (
    id INTEGER PRIMARY KEY AUTOINCREMENT,
    email TEXT,
    password TEXT,
    fullName TEXT,
    avatar_image_local_path TEXT,
    last_logged_in_auth_type ENUM(AuthType)
);

Are there any syntax errors related to image_metadata_table?
Yaa, it seems everything is valid again sharing the .drift file below

------------- import table-------------
import 'package:mytrailpals/config/app_db/schema/file_metadata.drift';
------------- import ends -------------

CREATE TABLE IF NOT EXISTS image_metadata_table (
    id INTEGER PRIMARY KEY AUTOINCREMENT,
    image_id TEXT NOT NULL UNIQUE,
    file_md5_hash TEXT NOT NULL UNIQUE,
    blur_hash VARCHAR(255) NOT NULL,
    parent_image_id TEXT,
    created_at DATETIME NOT NULL DEFAULT  (strftime('%Y-%m-%dT%H:%M:%fZ', 'now')),
    updated_at DATETIME NOT NULL  DEFAULT (strftime('%Y-%m-%dT%H:%M:%fZ', 'now')),
    FOREIGN KEY(parent_image_id) REFERENCES image_metadata_table(image_id)
        ON UPDATE CASCADE
        ON DELETE SET NULL,
    FOREIGN KEY (file_md5_hash) REFERENCES file_metadata_table(md5_hash)
       ON DELETE CASCADE
       ON UPDATE CASCADE
);

-- Index
CREATE INDEX IF NOT EXISTS idx_image_metadata_file_md5_hash ON image_metadata_table(file_md5_hash);
CREATE INDEX IF NOT EXISTS idx_image_metadata_image_id ON image_metadata_table(image_id);

-- Trigger
CREATE TRIGGER IF NOT EXISTS image_metadata_updated
AFTER UPDATE ON image_metadata_table
FOR EACH ROW
BEGIN
    UPDATE image_metadata_table SET updated_at = (strftime('%Y-%m-%dT%H:%M:%fZ', 'now')) WHERE id = OLD.id;
END;

-- Query
insertReplaceImageMetadata(
    REQUIRED :image_id AS TEXT,
    REQUIRED :file_md5_hash AS TEXT,
    REQUIRED :blur_hash AS TEXT,
    REQUIRED :parent_image_id AS TEXT OR NULL
):
INSERT OR REPLACE INTO image_metadata_table(image_id, file_md5_hash, blur_hash, parent_image_id)
VALUES (:image_id, :file_md5_hash, :blur_hash, :parent_image_id);

Or is that just not generated at all?

It seems to be getting generated.

Are you importing the drift file defining that table into your database?

Umm no that does not seems to be the case


documented SAS token process not working

The document mentions this is the REST API operation that should be used to get the SSO token:

This does not work. Using this method, the user will not get an error when logging in, but the logged in menu will not be displayed (profile, reports etc. missing).
https://docs.microsoft.com/en-us/rest/api/apimanagement/current-ga/user/get-shared-access-token
As a workaround, users can use this:
https://docs.microsoft.com/en-us/rest/api/apimanagement/current-ga/user/generate-sso-url
However, they should change the resulting URL hostname to point to new developer portal (not legacy) - from servicename.portal.azure-api.net to servicename.developer.azure-api.net.

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

ID: 1ca841cb-5a8f-ca9a-ec92-d088f43e0401
Version Independent ID: a56f0f32-4fb5-55b5-54d4-d26b39384316
Content: How to delegate user registration and product subscription
Content Source: articles/api-management/api-management-howto-setup-delegation.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

Thank you for your feedback! We will review and update as appropriate.

@vlad610 Thanks for sharing the workaround and my apologies for the late reply. The documented method, which is the official guidance, should work as described. If there's an issue using it as described, please report with additional details on the following repo: https://github.com/Azure/api-management-developer-portal/issues
Since we didn't determine any changes to this documentation upon reviewing this feedback, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[cloud_firestore] Fix document pagination

Implements https://github.com/flutter/plugins/pull/1591#issuecomment-500405394 and tries to address https://github.com/flutter/flutter/issues/34135.
@collinjackson I only implemented it on Android for now as I wanted someone to have a look first and I am not sure how to write idiomatic iOS code. If you could implement it, that would be great.

I thought it was working before, however, I encountered a problem with the collection group part. I will close this for now and reopen or open another PR once I have figured things out.


DRY up help site

Details
This refactoring DRYs up the help site and consolidates a lot of information into _data/routes.yml, while keeping the layout reused. The motivation behind this is to make the site easier to parse and update, but also to reduce room for error if non-engineers want to add articles to the site.
The only visible change introduced by this PR is that it removes this unnecessary/ugly In this article preamble:

Before
After

Fixed Issues
$ n/a
Tests / QA Steps
Run the Expensify Help site, visit all pages and verify that there are no regressions. Make sure to test on mobile and check that the meta tags from the SEO plugin are still working as expected as well.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The reviewer will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @marcochavezf in version: 1.2.23-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@roryabraham I dont believe we are able to QA this in Staging. If we can, can you share URL?
Is there a plan to add PRODQA label or checklist for App as well?

The URL is https://help.expensify.com

The other help site PR had to be tested once the PR hit prod. I assume it's the same for this one?

Help site PRs are deployed straight to production when they're merged to main. We should get a separate set of deploy comments for them, but we haven't invested in that yet

🚀 Deployed to production by @Julesssss in version: 1.2.23-9 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Box corner?

I'm using Startify in MacOS, and the quote box shows some ugly ansii characters instead of the continuous line and corners I see in the sample image. How can I fix this?
Thanks!

Hi!
See :h g:startify_fortune_use_unicode. :)


Auto wrap with col does not work in Safari

This code will not auto wrap in Safari 10.0.3  (OSX 10.12).
Basically using class="col" in order to auto wrap columns will not work. The columns are displayed outside page margins.
Chrome and FF behave as expected.

Some text here

Some text here

Some text here

        .... add more columns until they should wrap

Some text here

Some text here

couldnt get the tags to show. sorry.

See http://stackoverflow.com/questions/42174385/safari-and-bootstrap-4-automatic-grid-breaks for example.

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin and report back with specific browser and OS details.

I made an example available.  Sorry.

Created JSBin
Reproduced this issue on the following browsers:
iOS 7, Safari 7
iOS 8, Safari 8
iOS 7 & 8, Chrome 47

Turns out this is flexbug 11, where Safari doesn't wrap flex items properly. As I understand it, Safari uses different calculations for determining how to wrap flex items. The only workaround is using flex-basis to size the columns :. In this case, you'd need to add:
.col {
  flex: 1 0 50%; // where 50% is the intended size of the items
}

This is the same problem #21620 faces as well. Digging into even more, a border: 1px solid transparent on the .col will also fix this. Will work on a docs update to suggest these in context.

I got this as well. Site: danherzig.com. Happened on iOS browsers. Don't have an iPhone but got these screenshots. Fixed by reverting to prior columns (col-sm-12 and col-md-6). So the official fix was adding an invisible border?

In my case I overrode the class .row:before, row:after, and set display: inline-block; That did the trick and did not affect Chrome and Firefox

@pimisi suggestion wasn't working for me, but it worked changing it to display: block;

Here's the solution:
https://github.com/twbs/bootstrap/issues/9282#issuecomment-413421690
Worked like a charm!


Fixed zoomy bug with scrolling

If your browser is zoomed in, often what you tell the browser to scrollTo will be off by a pixel. I imagine there's a rounding issue in the rendering engine since the pixels are larger on the screen(??).
This seems to fix it.

Also, piggybacking an off-by-1 error on this as it was minor. There was a little odd behavior on the first and last image on the page.


feat: Enable iframe rendering in markdown code block

This PR allows users to set Markdown code blocks as 'iframe' to embed videos or external music links from services like Apple Music or Spotify.

Please solve the error from linter first.

Please solve the error from linter first.

@boojack issue has been fixed

我这里为什么会这样呢. 有一个code的背景和标签


python311Packages.google-cloud-firestore: 2.13.0 -> 2.13.1

Changelog: https://github.com/googleapis/python-firestore/blob/v2.13.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 266830 run on x86_64-linux 1

4 packages built:

python310Packages.google-cloud-firestore
python310Packages.google-cloud-firestore.dist
python311Packages.google-cloud-firestore
python311Packages.google-cloud-firestore.dist


Create an Individual Contribution Report for the milestone 2

[x] Create an individual report for the milestone 2
[x] Put that report in the wiki

Added the individual report


Hangs when invalid hostname is supplied

Expected Behavior

Invalid hostname or IP address should fail promptly and mark only Roomba accessory as unavailable.
Current Behavior

When I supply bogus hostname e.g. foobar homebridge hangs in “updating” status.
No error are logged to the console even when debug logging is enabled.
Versions

Homebridge: Version 0.4.53
Node:   v12.13.1
Plugin:  homebridge-roomba-stv v1.5.0

Configuration
`"keepAliveEnabled": true
Homebridge Log / Command Output

Running status requested
Battery status requested

Notes: Why is it important? My Roomba got stuck for a few days. At some point DHCP lease expired and roomba’s hostname no longer could be resolved. This unexpectedly caused the whole Homebridge to become unresponsive.

You are not alone.
https://github.com/esteban-mallen/homebridge-roomba-stv/issues/36


fix: Support overwrite of @map and @@map ERD prisma schema usage

I haven't used parcel before so I wasn't spending much time running this. I just copied over the function to search the data model string for @map and @@map to replace in your render SVG from DML functions.

I commented it out while I was troubleshooting the 500 error. It's turned back on now. Here are separate examples. You should be able to just add @map("db_name") to any of the rows and @@map("table_name") to any of the models at the bottom of the rows.
test @map
model User {
      id              Int                 @id @default(autoincrement())
      hashedPassword  String?             @map("hashed_password")
}

test @@map
model User {
      id              Int                 @id @default(autoincrement())
      hashedPassword  String?
      @@map("users")
}

Still doesn't appear to be working. hmm.

No, datamodelString is the already-parsed string. req.body is what you're looking for, I think.


Windows ARM64 prebuilt binary

Summary
Would it be possible to include a prebuilt binary for win32/arm64?  Devices in this category are the more recent Surface Pro lines.
Proposed implementation

napi-v6-win32-unknown-arm64
napi-v3-win32-unknown-arm64

Would happily welcome a CI change to support this - I'm not sure what it'd need right now. Presumably we can cross compile?


Native iOS files weren't correctly ignored

We'd adapted from a website, but we hadn't dealt with the fact that Git implies a leading ** on rules that don't begin with a /.  Thus, a lot of our native .gitignore rules weren't matching.

@andrewferrier Can you please check this & merge.

Looks great, thx.


The value is incorrect. It should be 5000 instead of 500

@kirillg

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

The author is not available, we will merge this using other means while keeping the author's identity tied to the commit.
#hold-off
#please-close


Incomplete examples for sqlstore, %%sql --save and %sqlrender

We need to add the dummy data to those examples so it's easier to follow.
Adding some sample data would be useful. We can add some "create table" and "insert into" statements at the beginning to show to it works. This is the source: https://github.com/ploomber/jupysql/blob/master/doc/api/magic-render.md (edited)

we've deprecated %sqlrender so this is no longer applicable. we also updated the documentation to better reflect how to use --save


2.3.1 and 2.2.7 versions isn't working after update

Previously I was on 2.2.4.
What I see in node_modules now:

No src folder, no scss files.
My js code:

$(document).ready(function() {
    //YouTube lazy load
    let elements = $('.lazyframe');
    lazyframe(elements, {
        apikey: 'my-api-key,
    });

});

No changes in code except loading css with @import '~lazyframe/dist/lazyframe.css';, but now all YT videos isn't working:

Videos don't play, clicks don't work.
What is it?

Hi @Sogl, thanks for opening an issue.
That's weird. When I'm installing 2.2.7 I get the src directory.

Could you supply a bit more info on how you are working with the package?
Is 2.2.4 still working for you?

Could you supply a bit more info on how you are working with the package?

I wrote above how I work with the library, this code I use in html:

Some text

Is 2.2.4 still working for you?

Yes.
I tried 2.2.7 one more time with command yarn add<EMAIL_ADDRESS>and now it works. So my issue is about v 2.3.1 only.

2.3.1 is still in beta and I will look at it when I get the chance. Thanks for reporting


turn on bias correction for amsua ch14 and atms ch15

AMSU-A channel 14 and ATMS channel 15 have been used without bias correction (BC) in GEOS. Experiments have been conducted to compare the forecast skills among three configurations for these two channels: active but without BC, passive, and active with BC. The experiments showed favorable results for the configuration of active with BC.  This PR is made for the changes needed to turn on bias correction for AMSUA ch14 and ATMS ch15, so any further assessment experiments could be conducted.

@gmao-yzhu can I ask you to please change the base of where this change is going to? You have it going to "main" - which is the default (I will ask Matt to change this). The easiest way to correct this is to perhaps close this PR , checkout develop, make your change there and submit the the change to develop - but using the knob above. If you are confused ... let me know - I can do it.

In my first attempt, I checked out develop and made changes, however, when I tried to commit the changes I got so many error messages. So I had to started over again, and found that there was no “develop” when I typed “git branch”, but there was “main”. That’s why I checked out main in my second attempt, and I committed changes successfully.
From: rtodling @.>
Date: Tuesday, October 8, 2024 at 11:04 AM
To: GEOS-ESM/GEOS_mksi @.>
Cc: Zhu, Yanqiu (GSFC-6101) @.>, Mention @.>
Subject: [EXTERNAL] [BULK] Re: [GEOS-ESM/GEOS_mksi] turn on bias correction for amsua ch14 and atms ch15 (PR #8)
CAUTION: This email originated from outside of NASA.  Please take care when clicking links or opening attachments.  Use the "Report Message" button to report suspicious messages to the NASA SOC.
@gmao-yzhuhttps://github.com/gmao-yzhu can I ask you to please change the base of where this change is going to? You have it going to "main" - which is the default (I will ask Matt to change this). The easiest way to correct this is to perhaps close this PR , checkout develop, make your change there and submit the the change to develop - but using the knob above. If you are confused ... let me know - I can do it.
—
Reply to this email directly, view it on GitHubhttps://github.com/GEOS-ESM/GEOS_mksi/pull/8#issuecomment-2400107550, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARUMEBXBDRIND4ZZJEMCCJ3Z2PYADAVCNFSM6AAAAABPKVBIPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGEYDONJVGA.
You are receiving this because you were mentioned.Message ID: @.***>

please let me know after Matt gets “develop” there, I will check out develop and submit another PR. Thanks.
From: Zhu, Yanqiu (GSFC-6101) @.>
Date: Tuesday, October 8, 2024 at 11:22 AM
To: GEOS-ESM/GEOS_mksi @.>, GEOS-ESM/GEOS_mksi @.>
Cc: Mention @.>
Subject: Re: [EXTERNAL] [BULK] Re: [GEOS-ESM/GEOS_mksi] turn on bias correction for amsua ch14 and atms ch15 (PR #8)
In my first attempt, I checked out develop and made changes, however, when I tried to commit the changes I got so many error messages. So I had to started over again, and found that there was no “develop” when I typed “git branch”, but there was “main”. That’s why I checked out main in my second attempt, and I committed changes successfully.
From: rtodling @.>
Date: Tuesday, October 8, 2024 at 11:04 AM
To: GEOS-ESM/GEOS_mksi @.>
Cc: Zhu, Yanqiu (GSFC-6101) @.>, Mention @.>
Subject: [EXTERNAL] [BULK] Re: [GEOS-ESM/GEOS_mksi] turn on bias correction for amsua ch14 and atms ch15 (PR #8)
CAUTION: This email originated from outside of NASA.  Please take care when clicking links or opening attachments.  Use the "Report Message" button to report suspicious messages to the NASA SOC.
@gmao-yzhuhttps://github.com/gmao-yzhu can I ask you to please change the base of where this change is going to? You have it going to "main" - which is the default (I will ask Matt to change this). The easiest way to correct this is to perhaps close this PR , checkout develop, make your change there and submit the the change to develop - but using the knob above. If you are confused ... let me know - I can do it.
—
Reply to this email directly, view it on GitHubhttps://github.com/GEOS-ESM/GEOS_mksi/pull/8#issuecomment-2400107550, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARUMEBXBDRIND4ZZJEMCCJ3Z2PYADAVCNFSM6AAAAABPKVBIPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGEYDONJVGA.
You are receiving this because you were mentioned.Message ID: @.***>

You are right, we don’t need to have a line w/ channel count zero. I put it there just for easy reading. I don’t mind removing it.
Regarding transition date – the date I have there right now is the day when I submitted the PR, it’s tentative. It needs to be changed to a date prior to your experiment period.  If later we decide to turn on bias correction, the date will need to be re-set.
From: rtodling @.>
Date: Tuesday, October 8, 2024 at 12:50 PM
To: GEOS-ESM/GEOS_mksi @.>
Cc: Zhu, Yanqiu (GSFC-6101) @.>, Mention @.>
Subject: [EXTERNAL] [BULK] Re: [GEOS-ESM/GEOS_mksi] turn on bias correction for amsua ch14 and atms ch15 (PR #8)
CAUTION: This email originated from outside of NASA.  Please take care when clicking links or opening attachments.  Use the "Report Message" button to report suspicious messages to the NASA SOC.
@rtodling commented on this pull request.
@gmao-yzhuhttps://github.com/gmao-yzhu I am confused about your changes ... why do you only want to do this from a certain point on? I can understand you wanting to do bias correction for ch14 but just not sure why these are the dates you picked for the transition - also I don't think you need a line w/ channel count zero ... you can simply just have an end date for the bias correction and not lines w/ zeros.
The issue I have w/ your changes is that when we run a test in 2023 (our default settings) we'd not be testing the bias correction being on ... since it's only on in Jan 2024 !
—
Reply to this email directly, view it on GitHubhttps://github.com/GEOS-ESM/GEOS_mksi/pull/8#pullrequestreview-2355043662, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARUMEBVUO5KEROQ3IQSP7KLZ2QEO3AVCNFSM6AAAAABPKVBIPCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNJVGA2DGNRWGI.
You are receiving this because you were mentioned.Message ID: @.***>


Bump to v0.2.3

Just bump version.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.731%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1439

Relevant Lines:
1586

💛  - Coveralls


[Icephys Support] Icephys Formats

Analog of #175 for icephys
In order for this app to actually see widespread adoption it needs to make full use of NeuroConv support for various formats
This will be the central issue tracking progress adding support for Icephys interfaces. Sub-issues might be added to track progress for more complex formats
The strategy for adding these step by step will require a more polished testing suite in order to scale well, but each should also be tested at least once manually (by a human) using the GIN testing data
Progress

[ ] ABF

Summarized and moved to https://github.com/NeurodataWithoutBorders/nwb-guide/issues/847
Re-open again whenever this starts active development


Add warning about port 80/443 require sudo right?

People with experience know port 80/443 requires sudo/admin right, but to new users, perhaps a warning is more user-friendly?
Currently serve reports "port in use" even though the problem is lack of permission. Not sure it's detect-able in Node.js though.
$ serve -p 80 test/

   ┌────────────────────────────────────────────────────────────┐
   │                                                            │
   │   Serving! (on port 33113, because 80 is already in use)   │
   │                                                            │
   │   - Local:            http://localhost:33113               │
   │   - On Your Network:  http://xxx.xx.xxx.xx:33113           │
   │                                                            │
   └────────────────────────────────────────────────────────────┘

$ sudo serve -p 80 test/

   ┌─────────────────────────────────────────────────┐
   │                                                 │
   │   Serving!                                      │
   │                                                 │
   │   - Local:            http://localhost:80       │
   │   - On Your Network:  http://xxx.xx.xxx.xx:80   │
   │                                                 │
   └─────────────────────────────────────────────────┘

This should now be working with the latest version.
Thank you for the effort you've put into this!
Please also read the details about the new release.


[BUG] Crash for kmir run --output if output is either program, json, kast

Problem
kmir run crashes on working tests if --output is changed to either program, json, kast
Reproducer
cd mir-semantics/kmir
poetry update
make build
source set_env.sh

either:
poetry run kmir run --output kast src/tests/integration/test-data/handwritten-rust/const-arithm-simple.mir
poetry run kmir run --output program src/tests/integration/test-data/handwritten-rust/const-arithm-simple.mir
poetry run kmir run --output json src/tests/integration/test-data/handwritten-rust/const-arithm-simple.mir

Investigation
Output exists for specified a specified depth before crashing:
poetry run kmir run --debug --verbose --output program --depth 0 src/tests/integration/test-data/handwritten-rust/const-arithm-simple.mir
poetry run kmir run --debug --verbose --output kast --depth 3 src/tests/integration/test-data/handwritten-rust/const-arithm-simple.mir
poetry run kmir run --debug --verbose --output json --depth 3 src/tests/integration/test-data/handwritten-rust/const-arithm-simple.mir

Solution
Unknown at this stage, all three options are permissible according to pyk. It might not be worth fixing this directly, and instead changing the KMIR object and call to run as outlined in Solution 3. of #208

I am closing this issue, as the output options of json and kast are working now.
The program option crashed due to an K bug: https://github.com/runtimeverification/k/issues/3861


Add toml configs for RecoveryEndpointConfig.properties

Description
The following three properties in the RecoveryEndpointConfig.properties were templated with a fix[1].
identity.server.service.contextURL=https://localhost:9443/services
identity.server.service.access.username=admin
identity.server.service.access.password=admin

Therefore the above configs can be configured via deployment.toml as follows. This config needs to be documented.
[identity.auth_framework.endpoint]
identity_server_service_url="https://$ref{server.hostname}:9443/services"
username="admin"
password="admin"

Related issues https://github.com/wso2/product-is/issues/9752, https://github.com/wso2/product-is/issues/9778
[1] https://github.com/wso2/carbon-identity-framework/pull/3207
[2] https://is.docs.wso2.com/en/latest/develop/hosting-authentication-endpoint-on-a-different-server/#moving-the-accountrecoveryendpoint-from-wso2is-and-hosting-it-on-a-separate-web-server

I believe the example identity_server_service_url is wrong as the /services is extraneous.
When set to "https://$ref{server.hostname}:9443/services", error is shown: <faultstring>The service cannot be found for the endpoint reference (EPR) /services/api/identity/recovery/v0.9/captcha?tenant-domain=carbon.super&captcha-type=ReCaptcha&recovery-type=password-recovery</faultstring>. The error is fixed by removing the /services segment from identity_server_service_url.
In older version of API doc, the example request is https://localhost:9443/api/identity/recovery/v0.9/captcha?tenant-domain=carbon.super&captcha-type=ReCaptcha&recovery-type=username-recovery, in which there is no segment of /services after the host:port.
In newer version of API doc, the captcha APIs are gone. But the rest of the APIs are still with the base path /api/identity/recovery/v0.9.
Relevant change can also be found at https://github.com/wso2/identity-apps/commit/714c12e3711a36b6d19bd9a6d1ba0266bc1caa8f#diff-30def54dcec306001906788bfa928f6043f2a5ee29d7d477367db08ba4dc9fe8L18
Therefore this example was probably based on outdated source code.


[PyTorch] fused CUDNN attention kernel not properly handling strides

Hello team,
we noticed discrepencies when using the transformer_engine.pytorch.TransformerLayer in combination with fused attention kernels and multi/group-query attention, fuse_qkv_params and qkv_weight_interleaved. All in all, this problem boils down to the following code snippet:
import torch

from transformer_engine.pytorch.attention import FlashAttention, FusedAttention
from transformer_engine.common.recipe import DelayedScaling
import transformer_engine_torch as tex

seqlen, q_heads, kv_heads, kv_channels = 2048, 16, 1, 64

qkv = torch.randn(seqlen, 1, q_heads + 2 * kv_heads, kv_channels, dtype=torch.float16, device="cuda")

q, k, v = qkv.split([q_heads, kv_heads, kv_heads], dim=2)
#q, k, v = [t.contiguous() for t in (q, k, v)]

flash_attn = FlashAttention(1.0)
fused_attn = FusedAttention(1.0)

output_flash = flash_attn(q, k, v, "sbhd_sbhd_sbhd", window_size=(-1, 0))
output_fused = fused_attn(q, k, v, "sbhd_sbhd_sbhd",
                          fused_attention_backend=tex.NVTE_Fused_Attn_Backend.NVTE_F16_arbitrary_seqlen,
                          window_size=(-1, 0),
                          fp8_meta=dict(recipe=DelayedScaling()))

print("diff:", torch.max(torch.abs(output_fused - output_flash)).item())

On H100 with CUDA 12.5 and CUDNN 9.2.1 we get a max error of 6.1953125. If I uncomment the line that applies contiguous() the error is down to 0.001953125, accordingly I suspect that the issue is related to the handling of strides in the fused attention backend.
Could you please have a look at this and try to reproduce my results? Thanks!

I think this issue is caused by that we're doing stride checks in get_qkv_layout() in DotProductAttention. https://github.com/NVIDIA/TransformerEngine/blob/a68acd71d3500d41b2d75caf249a1a4f4445ba69/transformer_engine/pytorch/attention.py#L4675
If you directly calls FusedAttention, it will not do such checks, while FA does these checks in https://github.com/Dao-AILab/flash-attention/blob/53a4f341634fcbc96bb999a3c804c192ea14f2ea/flash_attn/flash_attn_interface.py#L90.
cc @cyanguwa

I just had a second look at the q, k, v tensor strides in my example. The strides are (1152, 1152, 64, 1) for all tensors. So at least the requirement that the last stride is 1 is fulfilled 😉
I added the get_qkv_layout() to my sample but this did not fail and returned the sbhd_sbhd_sbhd layout

The requirement for the last dimension's stride to be 1 seems to be fine here. But I think the problem is that the provided q, k, v tensors do not have a sbhd_sbhd_sbhd layout. It's more like a sb(h+2h_g)d layout :). .contiguous() will force it to sbhd_sbhd_sbhd, which is why the test will pass in that case.
DotProductAttention has some sort of a check (through get_qkv_layout()) to make sure users' inputs are in the correct format, but FlashAttention and FusedAttention modules don't.
The get_qkv_layout() also returns sbhd_sbhd_sbhd because it missed something in the last elseif logic :). I've tweaked the function and included some extra check in #1214, so it should run a .contiguous() on the tensors now, before passing them to FlashAttention or FusedAttention.
import os
import torch

from transformer_engine.pytorch.attention import FlashAttention, FusedAttention, DotProductAttention, _attention_backends
from transformer_engine.common.recipe import DelayedScaling
import transformer_engine_torch as tex

seqlen, q_heads, kv_heads, kv_channels = 2048, 16, 1, 64
seqlen_kv = 1024

qkv = torch.randn(seqlen, 1, q_heads + 2 * kv_heads, kv_channels, dtype=torch.float16, device="cuda")

q, k, v = qkv.split([q_heads, kv_heads, kv_heads], dim=2)
#q, k, v = [t.contiguous() for t in (q, k, v)]

flash_attn = DotProductAttention(q_heads, kv_channels=kv_channels, num_gqa_groups=kv_heads, softmax_scale=0.1)
fused_attn = DotProductAttention(q_heads, kv_channels=kv_channels, num_gqa_groups=kv_heads, softmax_scale=0.1)

os.environ["NVTE_FUSED_ATTN"] = "0"
os.environ["NVTE_FLASH_ATTN"] = "1"
output_flash = flash_attn(q, k, v)

os.environ["NVTE_FUSED_ATTN"] = "1"
os.environ["NVTE_FLASH_ATTN"] = "0"
_attention_backends["backend_selection_requires_update"] = True
output_fused = fused_attn(q, k, v)

print("diff:", torch.max(torch.abs(output_fused - output_flash)).item())

Results:
NVTE_DEBUG=1 NVTE_DEBUG_LEVEL=1 python test.py
[INFO     | DotProductAttention]: Running with FlashAttention backend (version 2.4.2)
[INFO     | DotProductAttention]: Running with FusedAttention backend (sub-backend 1)
diff: 0.0009765625

Great, thanks for the solution and the detailed explanations. I just tested it and it works in our training code now as well 😃


Saving without a file extension makes interface disappear.

I entered a filename without an extension and the page went blank. Just a File menu remained. I exited and returned and all was fine after that.

Fixed in next release


feat: create SymbolIterator for block parsing

This PR adds a SymbolIterator to iterate over the Symbol slice returned from the Scanner.
It enables convenient tokenization and parsing for block elements,
by allowing to nest iterators, and define functions to match for block end and to strip line prefixes.
This is part of preparation for parsing more complex elements like lists.
Decisions

Nesting iterators
Because block elements may allow nesting, iterators may also be nested
to make parsing of elements nesting agnostic.
Parent iterators only forward symbols to children, if the parent is not at the end,
and an optional prefix is stripped.
Iterators keep track of their nesting depth, to allow nesting of sequences like (outer (inner)).

Functions/Closures for end/prefix matching
It was decided to allow functions to be passed for end/prefix matching instead of multiple Symbol
slices. This allows fine grained matching, and allows to distinguish between peeked and consumed
matching.
Prefix matching is always consuming, because nested iterators must not see prefix symbols of parent
elements.

Future work

Use SymbolIterator for inlines
Currently, inline parsing is done on a symbol slice.
Inline parsing internally uses get_symbol(index) quite often, which makes it hard to adopt the
iterator approach. For now, blocks retrieve a Symbol vector for inline content,
and then pass this to the inline parser.
With the new iterator approach supporting end detection, it might be possible to use
the same tokenization and parsing approach as done for blocks.
The handling of ambiguous tokens like *** won't be trivial, but I am confident that
it can be done by also matching for Bold and Italic directly in the ambiguous parser.
Changing inline parsing will take time, and possibly break things.
I would suggest that we first implement more blocks to see if the iterator approach is feasible.
If the new approach seems to work well for blocks, we can try to adopt it also for inlines.

Iterators all the way
The scanner creates the vector of Symbols using the grapheme segmenter of icu.
This segmenter itself is an iterator, so we might be able to directly embed this iterator
in the SymbolIteratorRoot. This would require some form of a symbol cache to allow peeking,
but using an internal VecDeque should do the trick.
Note: This effectively removes the option to get a slice of remaining symbols,
and therefore the option to get size hints of remaining symbols.
The size hint could still be approximated by using the utf8 length of the input string.
This won't be accurate on Windows, because new lines have utf8 length of two due to \r\n.
I am also unsure about the effect on performance and memory usage, when switching to full iterator.
Memory should in theory become better, because we do not allocate memory for all symbols.
Compared to allocation needed for the transformed content, this improvement might be neglectable
though.

I had to fix icu dependencies and check in Cargo.lock, because icu has internal version conflicts between 1.2.0 and 1.3.0.
Clippy currently fails due to old code.
This is fixed in PR #105 so I will merge it once this PR is merged into main.

I also implemented rendering for newline and whitespace, because I wanted to make sure the resulting HTML is still correct using the iterator approach.

After seeing the failed actions due to icu dependencies yesterday,
I updated to icu 1.3.0, which allows us to use the grapheme iterator without any generated data.
This also allows to remove the pin to a specific version, because we only need icu_segmenter and icu_locid both of which probably won't create version conflicts.

I will wait for PR #105 to be merged, and then merge this one to resolve the clippy warnings that keep failing the action.

I now merged changes from PR #105.
There were some merge conflicts, but all tests are passing now, and it should be ok to merge this PR.
For lexer tests in inline I had to remove the EOI symbol before running the tests.
With the EOI symbol, all lexer tests failed with an unwrap on None.
Assuming that we will update all of inline parsing to be based on iterators, I would leave this workaround as is for now.
Because inlines for now won't see the EOI symbol, because block elements strip it for them.


Approach

I set some tools as eslint, prettier, husky, vscode settings.
Also, when we try to commit new code, the hook pre-commit will check the unit test and code style, if there is a problem with your code won't be able to be committed

Important: I don't use vscode however I write some settings, if you don't like those setting, feel free to modify it
Where should the reviewer start?
hooks, prettier and eslint rules
This PR addresses a:

[ ] Bug
[ ] Feature
[ ] Technical Debt
[X] Chore
[ ] Vulnerability

I think the config is correct but I have a question @fernandoox , Does run SH files in Windows O.S. ?

I don't know, that sh file was created by husky automatically 😆, if anyone has Windows O.S could test it.


Rollback cherrypick #5140

#5140 has other dependencies that are not available in airbnb-master. have to rollback for now.
@michellethomas @john-bodley

Codecov Report

Merging #89 into airbnb-master will increase coverage by <.01%.
The diff coverage is 83.33%.

@@                Coverage Diff                @@
##           airbnb-master      #89      +/-   ##
=================================================
+ Coverage           63.2%   63.21%   +<.01%
=================================================
  Files                351      351
  Lines              22043    22039       -4
  Branches            2465     2465
=================================================
- Hits               13933    13931       -2
+ Misses              8098     8096       -2
  Partials              12       12

Impacted Files
Coverage Δ

superset/assets/src/utils/common.js
60% <ø> (-0.53%)
:arrow_down:

superset/viz.py
80.79% <0%> (+0.08%)
:arrow_up:

superset/assets/src/visualizations/nvd3_vis.js
9.41% <100%> (+0.38%)
:arrow_up:

superset/assets/src/modules/colors.js
77.08% <100%> (-0.47%)
:arrow_down:

Continue to review full report at Codecov.

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


Compiler issue on Mac OS X

I don't have a Mac capable of running OpenVR. Also I believe that only Vive is supported at the moment, I don't think you can run an Oculus Rift even if you have the external GPU or one of the new iMacs.
Anyway, I can't test anything on Mac but I did try to compile it just for fun. Got errors, haven't had the time to investigate. Just opening this issue as a reminder I have to find the time :)

Closing this, we're compiling the build fine, just don't know if it actually works.


Document storage

I'd like to use this for uploading files to google cloud storage and notice there's some code hanging around for it but it's not documented in the README or the docs. Perhaps storage is not yet feature complete, so also wondering on ETA for it. Thanks! :+1:

Storage is in active development, and is able to create/delete buckets and upload/download files now. The documentation is a bit rough ATM but will be improved shortly as well. If you have any questions feel free to ask.

Thanks, I'll take a look.


Introducing Structured Logs: fix quote marks

Fixed quote marks in latest blog post so that the code can highlight correctly.

Fixed in #23698


Update to sdk-go V2.0.0-RC1

Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>Release Note
Update sdk-go to v2.0.0-RC1

/hold

/unhold

/lgtm
/approve

Looks like you need to run update-codegen :(

Looks like you need to run update-codegen

I did it, but there is nothing new to commit!

/retest pull-knative-eventing-contrib-integration-tests

/retest pull-knative-eventing-contrib-integration-tests

/retest

/lgtm
/approve


make storage operations atomic

see https://github.com/TBD54566975/dwn-sdk-js/pull/263#discussion_r1140898784

This is now handled by resumable tasks: https://github.com/TBD54566975/dwn-sdk-js/pull/778.


ForceSync can be used if there is more than one tenant in the database

ForceSync: This mode is also intended for use during development, but unlike the Development and Add modes, it supports destructive schema changes (like removing fields, renaming them, changing their datatypes, and more). This mode is useful in development because it enables you to make these kinds of changes without losing data. Because destructive changes are not backwards compatible, this mode should not be used in production. To use this mode, the App must first be uninstalled from the tenant, and there can be only one tenant in the database.
Should be changed to
ForceSync: This mode is also intended for use during development, but unlike the Development and Add modes, it supports destructive schema changes (like removing fields, renaming them, changing their datatypes, and more). This mode is useful in development because it enables you to make these kinds of changes without losing data. Because destructive changes are not backwards compatible, this mode should not be used in production. To use this mode, the App must first be uninstalled from the tenant.

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

ID: 7a0487be-7680-3da5-d47b-e39fd04f449e
Version Independent ID: 5fdebecd-6373-6472-77bb-c9eb68dd10a2
Content: Sync-NAVApp (Microsoft.Dynamics.Nav.Apps.Management) - Dynamics NAV
Content Source: businesscentral-ps-18/Microsoft.Dynamics.Nav.Apps.Management/Sync-NAVApp.md
GitHub Login: @jswymer

Thanks! The article has been fixed and the update will be published soon.


The attribute principal_type for role assigments

Hello everyone,
in our case, we need to grant permissions for the Azure Key Vault to a group.
In order to do this, we need to set the type of the pricipal in the Terraform code.
However, the corresponding attribute is missing.
In my pull request, I implemented the attribute principal_type and tested it in our environment.
It would be nice if these modifications could be integrated into the module.
Many thanks and best regards
andy

thanks - closing as incorporated in #113


Need to be able to inject Auth Headers into request URL

Great module, almost works beautifully out of the box.
Can we somehow find a way to hack our Authorization header into the post request on the service?
Many image upload usage are via logged in users through an API, which requires a token to be passed in the Authorisation header.

First of all, thank you for such testimonials.
Yeah, I can create an interface for this. I need to find out how auth header should look in a multipart/form-data request.

Woot! Thanks for the awesome response!
I cloned the new branch, and noticed the folder structure and code structure is different?
Anyways, I renamed lib folder to src so the paths would get recognised, and also ran " tsc image-upload.component.ts " to compile it to js, but am getting an error on init.

Uncaught Error: Can't resolve all parameters for ImageUploadComponent: (?).
at CompileMetadataResolver.getDependenciesMetadata (http://localhost:4200/main.bundle.js:30279:19)
at CompileMetadataResolver.getTypeMetadata (http://localhost:4200/main.bundle.js:30180:26)
at CompileMetadataResolver.getDirectiveMetadata (http://localhost:4200/main.bundle.js:29955:28)
at http://localhost:4200/main.bundle.js:30024:49
at Array.forEach (native)
at CompileMetadataResolver.getNgModuleMetadata (http://localhost:4200/main.bundle.js:30017:44)
at http://localhost:4200/main.bundle.js:30005:50
at Array.forEach (native)
at CompileMetadataResolver.getNgModuleMetadata (http://localhost:4200/main.bundle.js:29992:44)
at RuntimeCompiler._compileComponents (http://localhost:4200/main.bundle.js:43787:47)

Also can you give a little instruction on how to set the headers? Thanks

Woot, thanks for the awesome response! It works.
I'm not sure if it could be optimised to be neater, because the standard way to set custom headers is like this:

let headers = new Headers();
headers.append('Authorization','Bearer ' + localStorage.getItem('mytoken') );

for the module, I'd have to set it like this in the constructor:

this.headers = [
{
header: 'Authorization',
value: 'Bearer ' + localStorage.getItem('mytoken')
}
]

It's just a nitpick though.
Thanks so much, I think you can merge it into master and npm.

Look, you can add your headers in the HTML attribute like this
<image-upload [url]="'my-url.com'"
  [headers]="[
    {header: 'Authorization, vaule: 'Bearer' + localStorage.getItem('mytoken)}
  ]"></image-upload>

So you don't need to use anything directly from the library, just use the interface :)

Feature added to the master branch and published to NPM. Thanks for great proposal :)

Hi, you wrote image tag like this to add header.But this code piece does not work please help me about this problem.Because I want to upload image file with myToken
<image-upload [url]="'my-url.com'"
  [headers]="[
    {header: 'Authorization, value: 'Bearer' + localStorage.getItem('mytoken')}
  ]"></image-upload>

@caglarturkurka are there any error logs or it's just don't behave like you expect it to?

my code like this
<image-upload
                  [max]="1"
                  [url]="'localhost:8080/user/photo/upload'"
                  [headers]="[
    {header: 'Authorization, value:' 'Bearer' +uploadPhotoService.getAccount().jwtToken}
  ]"
                ></image-upload>

ERROR :
EXCEPTION: Uncaught (in promise): Error: Directive parse errors:
Parser Error: Missing expected } at column 40 in [[
    {header: 'Authorization, value:' 'Bearer' +uploadPhotoService.getAccount().jwtToken}
  ]] in UserAddUpdateComponent@19:18: in Directive DefaultValueAccessor in ./DefaultValueAccessor
Parser Error: Missing expected } at column 40 in [[
    {header: 'Authorization, value:' 'Bearer' +uploadPhotoService.getAccount().jwtToken}
  ]] in UserAddUpdateComponent@19:18: in Directive DefaultValueAccessor in ./DefaultValueAccessor

can you help me about adding header with token to upload image

Now I encounter a problem about uploading image
my code like this
<image-upload
                  [url]="'http://localhost:8080/user/photo/upload'"
                  [headers]="[ {header: 'Authorization, value:'+'Bearer '+uploadPhotoService.getAccount().jwtToken + ''} ]"
                ></image-upload>

ERROR:
EXCEPTION: Failed to execute 'setRequestHeader' on 'XMLHttpRequest': 'Authorization, value:Bearer eyJhbGciOiJIUzUxMiJ9.eyJzdWIiOiJhZG1pbiIsImF1dGgiOiJST0xFX0FETUlOIiwiZXhwIjoxNDg2ODE0NTgwfQ.HWapNAN18DOhccCPVlFLRaUpFLqRsaayqnrXhjz0iJNqeSCgHdmfraH57f8RhZD2maCRBP8J28UkzmeCZvfZIA' is not a valid HTTP header field name.

Oh, I see. This code is just not valid because of the typo in the README.
Write the following:
<image-upload
[url]="'http://localhost:8080/user/photo/upload'"
[headers]="[ {header: 'Authorization', value: 'Bearer '+uploadPhotoService.getAccount().jwtToken + ''} ]"
>
The error occurs because of the closing quote. Value is the field of a header object so you need to close quotes after "Authorization". I have a typo in the README so it's my fault, sorry.

Thanks you for your attention.Also I have solved my problem.But also I sent a new problem about angular-cli and ImageUploadModule please check that and please help me about this problem.


Add access.TokenController for monitoring-aware token-based access control

This change adds access.TokenController which is capable of admitting or rejecting clients based on an "access_token" HTTP parameter. This change explicitly allows monitoring to issue access_tokens, and assign a context value to the HTTP request context, so that subsequent access controllers can a) identify monitoring requests, b) allow the test even if it would ordinarily be rejected.
To provide the option of exempting monitoring clients, the TokenController should be applied first in a sequence of access controllers.
This controller is not yet enabled.

This change is

Pull Request Test Coverage Report for Build 1229

55 of 55 (100.0%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.4%) to 80.856%

Files with Coverage Reduction
New Missed Lines
%

ndt7/upload/sender/sender.go
3
73.53%

Totals

Change from base Build 1225:
0.4%

Covered Lines:
1757

Relevant Lines:
2173

💛  - Coveralls


[libpq, openssl, librtmp] libpq 12 and others with openssl 1.1.1d

Lets see how far CI gets with this PR and push the openssl upgrade a bit

So most errors found so far:
windows-static: Missing linkage to crypt32.lib or some other
linux: Missing linkage to pthread.
Most common point of failure. Detection logic for openssl 1.1 fails due to the one above......
remaining linux issues: Probably missing pthread either in the detection or the later link stage

Concerning the UWP build error.
I only took a look @ quickfix and the problem is: Openssl defines PEM_read_X509 as:
PEM_read_X509                           1287	1_1_0	EXIST::FUNCTION:STDIO
and the UWP patch does:
        disable         => sub { [ 'ui-console', 'stdio', 'async', 'uplink',
Since OpenSSL master has the same target definition and stdio is disabled, I have to assume that the missing Symbols should have never been there in the first place and so that the UWP regressions are false negatives.

Yeah linux is green. now just 16h more for osx to decide what it wants

/azp run

/azp run

Hi @Neumann-A, seems there are some conflicts, please merge master into this branch,
Thanks.

arm-uwp
test
notes

offscale-libetcd-cpp
Skip
Missing port dependency

grpc
Fail
Regression

googleapis
Skip
Missing port dependency

xmlsec
Fail
Regression

gsoap
Pass

x64-uwp
test
notes

libpq
Skip
Missing port dependency

quickfix
Fail
Regression

@JackBoosY
grpc error:
       C:\vsts\_work\2\s\buildtrees\grpc\src\v1.23.1-ebfd5c51df\src\core\tsi\alts\crypt\aes_gcm.cc(85,5): error C3861:  'snprintf': identifier not found [C:\vsts\_work\2\s\buildtrees\grpc\arm-uwp-dbg\grpc.vcxproj]
seems unrelated to this PR. (Could be that OpenSSL was wrongly pulling in snprintf in UWP before)
xmlsec error:
         x509.obj : error LNK2019: unresolved external symbol BN_print_fp referenced in function xmlSecOpenSSLX509CertDebugDump [C:\vsts\_work\2\s\buildtrees\xmlsec\arm-uwp-dbg\libxmlsec-openssl.vcxproj]
BN_print_fp requires stdio which is marked as not available in UWP builds of OpenSSL. As such it is a false positiv and should have not worked before
quickfix errors:
       C:\vsts\_work\2\s\buildtrees\quickfix\src\v1.15.1-5086b0da20\src\C++\UtilitySSL.cpp(713,5): error C3861:  'ERR_print_errors_fp': identifier not found [C:\vsts\_work\2\s\buildtrees\quickfix\x64-uwp-dbg\src\C++\quickfix.vcxproj]
        C:\vsts\_work\2\s\buildtrees\quickfix\src\v1.15.1-5086b0da20\src\C++\UtilitySSL.cpp(866,8): error C3861:  'PEM_read_X509': identifier not found [C:\vsts\_work\2\s\buildtrees\quickfix\x64-uwp-dbg\src\C++\quickfix.vcxproj]
        C:\vsts\_work\2\s\buildtrees\quickfix\src\v1.15.1-5086b0da20\src\C++\UtilitySSL.cpp(909,8): error C3861:  'PEM_read_PrivateKey': identifier not found [C:\vsts\_work\2\s\buildtrees\quickfix\x64-uwp-dbg\src\C++\quickfix.vcxproj]

basically the same as before. OpenSSL marks those functions as requiring stdio which OpenSSL say is not available. As such this is also a false positiv and it should have not worked before.
see also https://github.com/microsoft/vcpkg/pull/8566#issuecomment-541460751

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

Passing all CI checks.
Thanks for your excellent work @Neumann-A!

@vicroms you should thank the others too.


DG changes

I'm requesting @donljh to review as well
Changes

Update ParserClasses diagram
Update ModelClass diagram
Update DeleteItem sequence diagram
Update EditItem sequence diagram

Codecov Report

Merging #217 (6d55cca) into master (ce0705d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #217   +/-   ##
=========================================
  Coverage     63.15%   63.15%
  Complexity      634      634
=========================================
  Files           105      105
  Lines          2399     2399
  Branches        310      310
=========================================
  Hits           1515     1515
  Misses          758      758
  Partials        126      126

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


CI add specific build job

ref https://github.com/actions/checkout/tree/v3/#checkout-pull-request-head-commit-instead-of-merge-commit try to use github specific to check out code and meanwhile for container image tag.

as a try for #640

@rootfs , so far it require a full git commit SHA for example https://github.com/SamYuan1990/kepler/actions/runs/5004169317/jobs/8966392722
https://github.com/SamYuan1990/kepler/actions/runs/5004190998/jobs/8966440203 (feature branch commit hash as adf1106abc5d3dfb4bf9a227da7196c170c90fe9)
any other suggestions?


Although I successfully installed and logged in, the message always shows you seem alone in the world!

Thanks for the great example. But, I do not know exactly how to use it. After I logged in and tried starting conversation, the message always you seem alone in the world. Is there anyone who can instruct how to use the app?

I am getting the same thing

I am also facing same issue..

You need to install it on at least two different devices (I did it on one iPhone and one iPad) and then use different accounts. After that, they will see each other and you can test communicating between them.

Thanks for your reply. I tried opening two simulators but, still did not work. Why is it necessary to open the same app from different devices? Any difference?

That's the setup that worked for me.

Work around - In firebase go to database and click "Try Firestore BETA", click "enable" the application should start working.

Can I ask one question? If it's needed to work around, then does the example still have the value for reference to make other similar projects?

Based on two different chat apps that i've looked at, the problem seems to be that the firebase database is not enabled.
Enabling "Firestore BETA" seems to enable the firebase realtime database that the chat app needs to work. So doing the same thing on your project should work.

I did the “ Firestone Beat “, but still doesn’t work!!

Enable Database and Storage (to store profile images).
Install the app on multiple devices.
Should work after that.

I got the same issue and i am using firebase 4. after enable the firestone still got same issue can't see anyone in the app and profile is not updating.. didn't get the data back from firebase.
seeing following error in the log:
2017-12-29 10:28:28.114781-0500 QuickChat[3746:218730] TIC Read Status [1:0x0]: 1:57
2017-12-29 10:28:28.114954-0500 QuickChat[3746:218730] TIC Read Status [1:0x0]: 1:57
2017-12-29 10:28:28.469512-0500 QuickChat[3746:218607] Could not inset legal attribution from corner 4
2017-12-29 10:28:28.534033-0500 QuickChat[3746:218607] Could not inset compass from edges 9
2017-12-29 10:28:28.534254-0500 QuickChat[3746:218607] Could not inset scale from edge 9
2017-12-29 10:28:28.534571-0500 QuickChat[3746:218607] Could not inset legal attribution from corner 4
2017-12-29 10:28:28.535154-0500 QuickChat[3746:218607] Could not inset compass from edges 9
2017-12-29 10:28:28.535283-0500 QuickChat[3746:218607] Could not inset scale from edge 9
2017-12-29 10:28:28.535460-0500 QuickChat[3746:218607] Could not inset legal attribution from corner 4
2017-12-29 10:28:47.438864-0500 QuickChat[3746:218758] [Firebase/Database][I-RDB038012] Listener at /users failed: permission_denied
2017-12-29 10:28:47.550301-0500 QuickChat[3746:219443] [Firebase/Database][I-RDB038012] Listener at /users/GJtEiLDECUOBf4HMxQboXeSWhfg2/conversations failed: permission_denied

Just want to update after enable the firestone and deleted the account and registered i can see the user and able to chat.

Everytime when i tried to login it is showing Wrong Input parameters please try again. please help..


Add trait similar to CheckedBitPattern

We should add a trait similar to bytemuck's CheckedBitPattern. It should be possible to write a custom derive for this trait just like bytemuck does.

Closing as a duplicate of #5.


fix(style): Fix bug that caused large table rows when displaying only…

… a few results

Fix to bug #399

:+1:

:+1: