Add Fortran build dependeny

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

wohooo it works again. @guyer I guess this should be ready for merge. The last diff only updates the build-number...

That's great. I don't know what it was broken. I don't know why it's fixed. Those are the perfect circumstances to merge, right? Thanks for poking at this. Now I just need to figure out why conda-forge/pysparse-feedstock is broken since I submitted it..


Enable precompiled headers on Linux

Probably this was should be added to #2841 but i forgot about that PR. Anyway, here is little patch to enable compilation of PCH on CMake builds

Thanks!


[bug]: Predictive Back Gestures not working with NavigationService

Describe the bug
Support for predictive back gestures was added to Stable in v3.17 and it works while using the default Flutter navigator. However when using the NavigationService in stacked_services the animation isn't triggered and the view is popped.
What operating system do you use?
Windows
Information about the installed tooling
[!] Flutter (Channel stable, 3.19.2, on Microsoft Windows [Version 10.0.22631.3235], locale en-US)
• Flutter version 3.19.2 on channel stable at C:\Users\thebb\fvm\default
! Warning: flutter on your path resolves to C:\Users\thebb\fvm\versions\stable\bin\flutter, which is not inside
your current Flutter SDK checkout at C:\Users\thebb\fvm\default. Consider adding C:\Users\thebb\fvm\default\bin to
the front of your path.
! Warning: dart on your path resolves to C:\Users\thebb\fvm\versions\stable\bin\dart, which is not inside your
current Flutter SDK checkout at C:\Users\thebb\fvm\default. Consider adding C:\Users\thebb\fvm\default\bin to the
front of your path.
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision<PHONE_NUMBER> (2 weeks ago), 2024-02-27 16:51:22 -0500
• Engine revision 04817c99c9
• Dart version 3.3.0
• DevTools version 2.31.1
• If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly
to perform update checks and upgrades.
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at C:\Users\thebb\AppData\Local\Android\sdk
• Platform android-34, build-tools 34.0.0
• Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.2)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.7.34018.315
• Windows 10 SDK version 10.0.22621.0
[✓] Android Studio (version 2023.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
[✓] VS Code (version 1.88.0-insider)
• VS Code at C:\Users\thebb\AppData\Local\Programs\Microsoft VS Code Insiders
• Flutter extension version 3.84.0
[✓] Connected device (4 available)
• Pixel 7 Pro (mobile) • 31031FDH3000Y5 • android-arm64  • Android 14 (API 34)
• Windows (desktop)    • windows        • windows-x64    • Microsoft Windows [Version 10.0.22631.3235]
• Chrome (web)         • chrome         • web-javascript • Google Chrome 122.0.6261.112
• Edge (web)           • edge           • web-javascript • Microsoft Edge 122.0.2365.80
[✓] Network resources
• All expected network resources are available.
Steps to reproduce the issue

Create a new Flutter app
Add stacked_services as a dependency and call NavigationService.back() from any viewmodel.

Expected behavior
No response
Screenshots
No response
Additional Context
No response

Hey, just to be clear, the problem here is that when you call back on the navigation service it stays on the same view and doesn't go back?

Oh nope, the navigation itself works fine with the service but there's no predictive back animation like there is for a flutter app using the contextual navigation. As far as I know it has already been implemented right? Please correct me if I'm wrong

Sorry man, I'm not up to date with those animations, I don't even know what it's supposed to do haha.
I don't know what's expected. It might be implemented, if you're seeing it on a default stable Flutter project.

Got it 😂 And well the animations work when using contextual navigation but not when using the service from stacked so I'm not sure. I'll add a video for reference for what the animation looks like
296241818-96e4be68-6d3f-4e89-842e-53bd5b5bfaf5.webm

Ooooh interesting, I've never seen that and I only use Android devices 😅 that looks really cool.
We don't do anything special, we just call the built-in navigator functions under the hood.
Let's keep it open, maybe one of us can look at it when we have some time.
Thanks for filing the issue.

Got it, thanks!

I'm not sure what exactly changed but the gesture animation seems to work now? 😅 So I'll close the Issue, if someone else can't access it in the future they can reopen it

Okay, so apparently the gesture animation is triggered when on the initial page but not when you navigate to a second page. I also tried using the contextual navigation but that doesn't show the animation either so I guess it's not implemented in Flutter natively yet (Atleast not completely)


merge from konsum-frontend-svelte

package.json

chore(deps): add ^3.50.0 remove ^3.49.0 (devDependencies.svelte)

.github/workflows/browser_ci.yml

chore: add ${{ matrix.os }} (jobs.test.runs-on)

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

npm package (@latest dist-tag)
v1.33.1
GitHub release

Your semantic-release bot :package::rocket:


Remove repeating UCAST check 1.1.9

Checking that the corosync uses UCAST was already implemented in 1.1.6,
so there is no need in 1.1.9 that does the same. The runtime version however remains.

gj 🍻


BACKLOG-10107: Add test script for all packages

Hi,
I checked this PR to build js-components, some console errors related to dependencies.
I suppose it is better to isolate commits when updating dependencies from configuration or code implementation commits (if we can, of course).
This PR is on hold for the moment.
Thank you

Thank you for the change


404: Page not Found

Describe the bug
There is a link with a Page Not Found message.
To Reproduce
Steps to reproduce the behavior:

Go to Accounts And Permissions
Click on EOSIO accounts
See error

Expected behavior
Show content.
Screenshots

[ ] the broken link is fixed


Early encrypted download heuristic (multiple share)

IF the following conditions are met :

mobile X is uploading (or has finished uploading) multiple encrypted resources R*, and hasn't scanned QR yet,
a client Y is currently open and listening to a channel C,
we think there is high probability that X will scan Y  (e.g. because it has already scanned Y 2mn ago)

THEN Y may start downloading the encrypted resources R*.
This will speed up the subsequent "download" step when X does scan Y.
This is a backend & client concern, the mobile apps (Android and iOS) probably don't need to care at all about it.

This optimization may be implemented ONLY for encrypted resources. The client must not have the decryption key until it has been properly provided, after X has scanned Y.

It is technically possible to have several clients Y download the same resources early, if they all have a high enough probability to become the target. Only zero or one of them will be scanned in the end, in current action.
There is a cost (for the user, and for the cloud project) associated with downloading data that we're not sure to use. Consider limiting the number of MB eligible to early download, e.g. 20MB.

For a single resource share action, see #514

See also #60 and #94

See also #60 and #94

Note that the client is listening to pre-cast signals only when displaying the QR-code, for now (not when displaying the result of the previous share action). See #520, #521.


mbedtls and littlefs on a STM32L4

Discussed in https://github.com/zephyrproject-rtos/zephyr/discussions/37329

Originally posted by jhfigueiredo July 30, 2021
I'm quite new to Zephyr so this might just be some missing configuration or lack of knowledge.
The objective is to load a x509 certificate from the filesystem (littlefs) using the mbedtls mbedtls_x509_crt_parse_file function. For that I need to include MBEDTLS_FS_IO. But when I do include this configuration it includes <dirent.h> which then returns the following error:
In file included from /home/jhf/zephyr-sdk-0.12.4/arm-zephyr-eabi/arm-zephyr-eabi/sys-include/dirent.h:39,
from /home/jhf/zephyrproject/modules/crypto/mbedtls/mbedtls/library/x509_crt.c:78:
/home/jhf/zephyr-sdk-0.12.4/arm-zephyr-eabi/arm-zephyr-eabi/sys-include/sys/dirent.h:10:2: error: #error "<dirent.h> not supported"
10 | #error "<dirent.h> not supported"
|  ^~~~~
I have a work around but would imply changes to the mbedtls original code, which I really would try to avoid.
Any help would be appreciated.

@jhfigueiredo - please continue in the original discussion.


User control over cargo warnings

Problem
Today, cargo maintainers are very cautious as to what warnings are created because they cannot be turned off by the user when they are intended.  The problem is we've not had a mechanism to control it.  With #12115, we'll now have a mechanism
Items blocked on diagnostic control

#11782
Publish size warnings (https://github.com/rust-lang/rfcs/pull/3397#discussion_r1213330487)
#9058
Warn on old build-directive syntax (#11461)
Control the warning from #10910
Warn in more cases than #10910
#5340
#2568
Maybe #2489

Proposed Solution
As package and workspace level lint control, turning existing warnings into diagnostics where possible
Notes
There will still be lints that are not related to the package but related to the user or / invocation.  At a later stage, we can consider user-level lint levels.

From https://github.com/rust-lang/cargo/issues/10160#issuecomment-1573883951

Just had a talk with @Muscraft. This is mostly my idea but it would be great if we have this kind of error overhaul. It will help things like JSON format message (#8283) or developing Cargo's own lint rules with -Zlints (#12115). It would also open a door to diagonostics translation (rust-lang/rust#100717). Probably the modularization of Cargo could benefit from it as well.

imo we need structured diagnostics which we pass to the diagnostic system which will then be turned into errors or something else

An obvious way to implement this is to shove the lint config in and our of Config as we enter and leave the relevant contexts.  This is a bit more brittle and lowers our chances for doing some operations with threading as Config needs to be mutable everywhere
One thought for how we could implement this

Rename Config to Context (we might want to do this anyways)
Refactor Context to allow decorating it (wrapping it)
Whenever we start down a workspace or package specific code path, we decorate the Context with that workspace or package's lint configuration

One thought I had from #13621 was more about the linting system as a whole.
Should we use cargo::rust_2024_compatibility group or should we key off of rust::rust_2024_compatibility group?

Take unused_lints.  iirc rust::unused_lints affects both rustc and clippy.
So the question would be

Is rust::unused_lints global or specialized for rustc linters
If not global, does that mean cargo::unused_lints should affect all Cargo.toml linters (e.g. if we moved Cargo lints from clippy-driver to cargo clippy)

Or put another way, does a user say "I want to see 2024 Edition compatibility warnings" or do they say "I want to see Rust 2024 Edition compatibility warnings separately from Cargo 2024 Edition compatibility warnings".  The Edition is the same concept, so it seems weird to split it by tool.  However, rustfmt does have a distinct Edition so maybe we should keep it more loose generally?


custom_derive! + HasMany! not working for me

I have a setup similar to your getting started guide and want to add the HasMany! macro with custom_derive.
Crate list in main.rs:
#![feature(custom_derive, custom_attribute, plugin)]
#![plugin(diesel_codegen, dotenv_macros)]

#[macro_use] extern crate diesel;
extern crate dotenv;
#[macro_use] extern crate custom_derive;

My user model:
custom_derive! {
    #[derive(Identifiable, HasMany(items, foreign_key = user_id))]
    #[table_name(users)]
    pub struct User {
        pub id: i32,
        // Other fields
    }
}

Cargo.toml excerpt
diesel = "0.6.2"
diesel_codegen = { version = "0.6.2", default-features = false, features = ["nightly", "postgres"] }
dotenv = "0.8.0"
dotenv_macros = "0.8.0"
custom_derive = "0.1.5"

When I run the code, I get the following output:
   Compiling my-app v0.1.0 (file:///Users/arne/Developer/my-app)
<custom_derive macros>:114:32: 114:33 error: macro undefined: 'Identifiable!'
<custom_derive macros>:114 $ ( $ user_drvs ) * $ new_user (  ) , ) } } ; (
                                                          ^
<custom_derive macros>:120:32: 120:33 error: macro undefined: 'HasMany!'
<custom_derive macros>:120 $ ( $ user_drvs ) * $ new_user ( $ ( $ new_user_args ) * ) , ) } } ; (
                                                          ^
error: aborting due to 2 previous errors
error: Could not compile `my-app`.

To learn more, run the command again with --verbose.

I'm thankful for any help 😊

HasMany! is not available in version 0.6.2, as it was introduced just 2 days ago in https://github.com/diesel-rs/diesel/pull/386. The docs are rendered for the master branch. While there should be a new release 0.7 soon, you can track master in the meantime.

@killercup: Thanks for your help, didn't think of that. 🙈


Parquet outputs

Background
This PR is a branch off of #3232 . Most of the changes come from that branch/pr with a few additions which attempt to answer unresolved questions from that ongoing PR. Specific changes here include: refactoring where the check_foreign_keys logic lives, reverting the use of unittest for consistency (I don't feel too strongly either way, but consistency is nice), and removing the check for extra columns from enforce_schema which was causing CI to fail.

Looks like there's an issue with accessing _sqlite_io_manager
AttributeError: 'IOManagerDefinition' object has no attribute '_sqlite_io_manager'

Just switching this to draft for the moment so every small changes doesn't incur costs.

I'm running the full ETL locally and getting some errors:
Stale parquet_path() argument in PudlParquetIOManager

Removed the db_name="pudl" arguments

Stale check_foreign_keys() argument

replaced engine with constructed engine using SQLite URI.

Reading TZ aware datetime type
Affected assets
Error occurred while loading input "core_ferc714__hourly_demand_pa" of step "load_hourly_demand_matrix_ferc714"
Error occurred while loading input "core_ferc714__hourly_demand_pa" of step "out_ferc714__summarized_demand"

Error:
TypeError: Cannot use .astype to convert from timezone-aware dtype to timezone-naive dtype. Use obj.tz_localize(None) or obj.tz_convert('UTC').tz_localize(None) instead.
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/dagster/_core/execution/plan/utils.py", line 54, in op_execution_error_boundary
    yield
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/dagster/_core/execution/plan/inputs.py", line 831, in _load_input_with_input_manager
    value = input_manager.load_input(context)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/code/catalyst/pudl/src/pudl/io_managers.py", line 112, in load_input
    return self._parquet_io_manager.load_input(context)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/code/catalyst/pudl/src/pudl/io_managers.py", line 350, in load_input
    return res.enforce_schema(df)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/code/catalyst/pudl/src/pudl/metadata/classes.py", line 1482, in enforce_schema
    df = self.format_df(df)
         ^^^^^^^^^^^^^^^^^^
  File "/Users/zane/code/catalyst/pudl/src/pudl/metadata/classes.py", line 1463, in format_df
    .astype(dtypes, copy=False)
     ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/generic.py", line 6513, in astype
    res_col = col.astype(dtype=cdt, copy=copy, errors=errors)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/generic.py", line 6534, in astype
    new_data = self._mgr.astype(dtype=dtype, copy=copy, errors=errors)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/internals/managers.py", line 414, in astype
    return self.apply(
           ^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/internals/managers.py", line 354, in apply
    applied = getattr(b, f)(**kwargs)
              ^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/internals/blocks.py", line 616, in astype
    new_values = astype_array_safe(values, dtype, copy=copy, errors=errors)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/dtypes/astype.py", line 238, in astype_array_safe
    new_values = astype_array(values, dtype, copy=copy)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/dtypes/astype.py", line 180, in astype_array
    values = values.astype(dtype, copy=copy)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zane/miniforge3/envs/pudl-dev/lib/python3.11/site-packages/pandas/core/arrays/datetimes.py", line 708, in astype
    raise TypeError(

EPA CEMS output location

Previously we had an epacems directory at the top level of $PUDL_OUTPUT which contained the quarterly EPA CEMS parquet outputs prior to consolidation.
Those outputs are consolidated into a single top-level epacems parquet file.
For uniformity, it seems like moving all of that process into the parquet directory and then only plucking the CEMS output from the parquet directory for distribution in the nightly build outputs might be better?
This would require a small tweak to the nightly build script. Would it affect anything else?
We could continue to have the (5GB) epacems parquet output distributed both as a top level file alongside the SQLite DBs, and in the parquet-only output bucket.

@zaneselvans I'll take a look at the errors you've outlined.

Defunct PandasParquetIOManager?

As for the PandasParquetIOManager, we are using it for reads, but to avoid running out of memory it returns Dask DataFrame, so we definitely should change the name. We could make it the DaskParquetIOManager, but I think it just EpaCemsIOManager might be better since it's pretty specific to CEMS. I also think it would be nice to expand it to handle writes as well instead of doing writes inside the asset itself, which is kinda confusing.

EPA CEMS output location

Would it make sense to output the partitioned CEMS files in the parquet/ directory and put the consolidated file at the top level?

I agree renaming PandasParquetIOManager makes sense. Should we do that now or get all this merged in and tweak?
Will we be able to get the output method inside an EpaCemsIOManager even though it doesn't work on single assets. It takes a whole year's worth of quarterly data, and slices it up into state-level annual data?
I guess writing the quarterly CEMS parquet files out to the same parquet/ directory would be fine. It's like 100 files, but we already have 100+ files in there.
For distribution, we would want to make sure:

core_epacems__hourly_emissions.parquet is available at the same path as now for public outputs.
core_epacems__hourly_emissions.parquet is also made available inside the parquet/ directory for the parquet-only distribution.
The quarterly outputs which get consolidated are not distributed to gs://parquet.catalyst.coop/nightly
The quarterly outputs do get saved in the gs://builds.catalyst.coop/$BUILD_ID bucket in case something goes wrong and we need to look at them.

I'll go ahead and rename the PandasParquetIOManager, and get the outputs in the right place. I think we could apply the EpaCemsIOManager to the ops that produce the per-year CEMS slices, but I'm not totally sure how easy this will be/if it will actually be cleaner. I'll think about making this change while rearranging where the outputs live, but I won't spend much time on it because I don't want this PR to languish.

@bendnorman I think we're just waiting to see if your comments were all addressed.

Awesome work!


No module for Azure Maps calls, just management

Why isn't there a package to do calls to Azure Maps from Python?

Hi @eavanvalkenburg
Somehow I missed your message, it's available now:
https://pypi.org/project/azure-mgmt-maps/
Thanks,

Hi @lmazuel that is not what I was looking for, I meant being able to actually talk to the API's of Azure Maps, basically a call to this endpoint, but then wrapped in a nice way: https://docs.microsoft.com/en-us/rest/api/maps/route/getroutedirections, so a MapsClient, to complement a MapsManagementClient!

Gotcha, sorry for the misunderstanding, re-opening the issue.

Any updates on this?

@kaerm is this team in your backlog of data-plane? There is no swagger but it seems a RestAPI exists.

@jongio is maps in the inventory?

Confirmed, this is accounted for in our data-plane SDK backlog, but no ETA as for now

The Maps team is currently working on these SDKs.

Hi @eavanvalkenburg,
We have published the following packages for Azure Maps. Sorry for waiting so long. Happy coding! :)

PyPi Links
README
Samples

azure-maps-search
README
Samples

azure-maps-route
README
Samples

azure-maps-render
README
Samples

azure-maps-geolocation
README
Samples

You can also refer to all Azure Maps REST SDK on Microsoft Learn

Thanks for circling back, I will have a look!


🕳 Prevent trailing whitespaces in logs

This PR eliminates the trailing whitespace in logger outputs.

tip: we need to test the error log


Update base image to debian bookworm

As requested in https://github.com/flo-at/minmon/issues/209.
Running fine on an arm64 VPS.

Thank you for the PR! I'll manually verify asap as it turns out there's no CI check for the docker build. Maybe I'll fix that while I'm on it.

Checked this on ARM64. Works as expected :+1:


Remove indirection for installation instructions

In GitLab by @kwanzknoel on Sep 9, 2021, 02:41

Closed as instructions have been clarified in https://github.com/Glow-Lang/glow/commit/b54c344c23ca327faba1e9f6f504e489d2550c9b


Prevent side-effect binding elimination

In Optimized AST, prevents binding elimination of pattern match tests with side-effects.
(i.e. prevents multiple executions of the same pattern match test with side-effects).

Almost everything is passing, except fsharpqa:
3199 tests passed (99.94%)
2 tests failed (0.06%)
0 tests were cascaded failures (0.00%)
0 tests returned "no result" (0.00%)
0 tests timed out (0.00%)
0 tests had test errors (0.00%)
========
3201 Total executed tests

Perhaps the IL for those 2 tests can be updated to match the new behavior.

Rebased to latest.

Including us, by the looks:
CodeGen\EmittedIL\TestFunctions (TestFunction9b4.fs) -- failed
CodeGen\EmittedIL\QueryExpressionStepping (Linq101Aggregates01.fs - CodeGen) -- failed

@KevinRansom Yes, that's what is mentioned in my second comment, see above.
IMO the tests just need to be fixed to reflect the new (arguably safer, i.e. more conservative) behavior.

@Tihan The test included in the PR provides an example.
The let binding in the match test get eliminated and converted into multiple test executions in the if-then chain:
match side_effect() with

will be converted to (in pseudo-code):
if side_effect() then ...
else if side_effect() then ...
else ...

@ncave Thanks, that gives me clarity, specifically your pseudo-code. It's great that you have identified this.
In principle, I think the optimized AST should not have a representation of multiple executions from the match optimization of what you described:
if side_effect() then ...
else if side_effect() then ...
else ...

This doesn't feel right for both side effect and side effect free executions, not just side effects.
However, the actual IL generated doesn't have the multiple executions; somewhere down the line it gets re-written or IL generated in a special way. With your change, the IL gets generated differently; the binding elimination doesn't happen and it creates a local. Therefore, this hurts an existing optimization for us.
 Is that correct?

@TIHan Yes, this issue (as stated in the PR) is manifesting in Optimized AST only.
The argument the PR is making is that this optimization is (arguably) perhaps insufficiently constrained.
I'm unaware who else (besides Fable) is a consumer of optimized F# AST.

I agree with @TIHan so far, the optimization as implemented to this seems to be sound.
So I'm not understanding which Fable problem this is fixing -  @ncave could you add an issue for this?  I assume it is different to the issue you fixed here https://github.com/Microsoft/visualfsharp/pull/6333

@dsyme Yes, it's unrelated to #6333.
I have included a test in the PR that showcases the issue.
In plain words, the match test binding elimination optimization is incorrect when there are multiple cases (more than 2) to match, and the match test has side effects.
But it's possible it may be a bit strong in its current form, perhaps we need to only check for side effects if more than 1 replacement is made. I'll see if I can fix that.

@ncave  Yes, I understand that this must fix an issue in the optimized expression trees being seen via the FCS API.  However it seems to affect generated code too, by disabling the application of the optimization when it is sound.
Two questions

What are those examples giving at the moment in the FCS API?
Is there an example that shows the optimization unsound in regular F# code - i.e. apart from its reveal through the FCS optimized expressions API?   (It could be the mistake is in the preparation of TASTs for the FCS API, or there is another interaction going on)

Either way I really want to dig into what's up here, understand it, make sure we have full info on the issue and .  THanks :)

Keeping it downstream as not many clients use optimized AST.


Create Video slides for XC3 V2

User stories
#1 As a XC3 team member, I have to create video slides for XC3 V2 using canva. Template is given in link.
Acceptance Criteria
What must be achieved to consider this story complete?

[x] Slide of new features available in XC3 V2.
[x] Slide for each feature and its benefits.
[x] Slides for installation process of XC3 V2.
[x] Other slides.

Created slides and shared with team. Got feedback and will add more details for each features and future work. Also will address the feedback.

Created slides and shared with team. Got feedback and will add more details for each features and future work. Also will address the feedback.

Addressed the feedback on slides.

Slides are done.
https://www.canva.com/design/DAFy5hKwyYw/9-fpw-5nQbTg1lg6jY_ikw/edit?utm_content=DAFy5hKwyYw&utm_campaign=designshare&utm_medium=link2&utm_source=sharebutton


Changes to avoid compilation errors in Arduino 2.2

avoids WIFI errors in current Arduino version
e.g.: error: 'union arduino_event_info_t' has no member named 'disconnected'; did you mean 'eth_connected'?
Serial.println(info.disconnected.reason);
Note:
it is not necessary to add the library "ESP32 BLE for Arduino" BLE library is included in Arduino 2.2

In the next few days there will be a new release with built-in wifimanager and using the arduino 2 bledevice library for esp32


Make the Calendar UI appear and customize it

Try to have the Calendar appear in some form within the app, replacing the previously existing browser.

#16 Added Basic functionality but more to be done


iotrace: add iotrace dump and load zstd compress support

Adding the difftest trace feature enables ZSTD for streaming compression, which compresses the trace volume by about 98%, enabling printing more cycle traces to disk

Also add CI check for ZSTD feature.


Branch v1.x is out of sync

composer installs it when minimum-stability = dev, but it shouldn't.

Thanks for letting us know.
@trowski Do you remember why that branch exists at all? I guess because we planned to return a promise while releasing a lock, but rolled that back?

I've deleted the branch, so it should be fine now. Unfortunately, GitHub's mobile site is broken and doesn't offer the delete button, even in desktop mode.

composer still install v1.x-dev eafe96a
maybe something to clean on packagist?

Yup, I've deleted it there. :+1:


🐛 [firebase_messaging] Redirect not working on iOS

I would like to redirect to a specific screen on tapping the push notification..
      _firebaseMessaging.configure(
          onMessage: (Map<String, dynamic> message) async {
            print("onMessage: $message");
        if (message['data'].containsKey('thread_id')) {
          _navigateToInbox(message);
        }
      }, onLaunch: (Map<String, dynamic> message) async {
        print("onLaunch: $message");
        if (message['data'].containsKey('thread_id')) {
          _navigateToInbox(message);
        }
      }, onResume: (Map<String, dynamic> message) async {
        print("onResume: $message");
        if (message['data'].containsKey('thread_id')) {
          _navigateToInbox(message);
        }
      });

The _navigateToInbox looks like this:
  Future<Object> _navigateToInbox(Map<String, dynamic> message) {
    final threadId = int.parse(message['data']['thread_id']);
    return MyApp.navigatorKey.currentState
        .pushNamed(ThreadScreen.routeName, arguments: threadId);
  }

So i am using the global navigatorKey. This works super fine for Android, iOS just opens the app and does nothing :( Not sure if i am missing anything?
Here is the payload from onResume:
{notification: {}, data: {collapse_key: key, thread_id: 35626, google.original_priority: normal, google.sent_time:<PHONE_NUMBER>832, google.delivered_priority: normal, google.ttl: 2419200, from:<PHONE_NUMBER>56, click_action: FLUTTER_NOTIFICATION_CLICK, google.message_id: 0:1602751694837303%3d33f9053d33f905}}

Here is the output of flutter doctor -v
[✓] Flutter (Channel stable, 1.22.1, on Mac OS X 10.15.7 19H2, locale en-DK)
    • Flutter version 1.22.1 at /Users/turbz/flutter
    • Framework revision f30b7f4db9 (7 days ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/turbz/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/turbz/Library/Android/sdk
    • Java binary at: /Users/turbz/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6821437/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Users/turbz/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6821437/Android Studio.app/Contents
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (2 available)
    • ONEPLUS A6003 (mobile)      • 1b1258f                   • android-arm64 • Android 10 (API 29)
    • iPhone von Jacob (mobile) • 00008030-001125360AE1802E • ios           • iOS 13.7

• No issues found!

Is this a bug? Or intended? Do I miss something?

Hi @Thorbenl
Does the navigation issue work outside of messaging.
For example if you created a button to see if navigation still works as expected?

Same problema here only on ios

@RenanDelfanti Could you provide a minimal example?
We need to confirm that navigation is working as expected as i'm not sure why this would be related to messaging

@dackers86 Yes, navigation works normally in the entire app.
The @Thorbenl  example is very similar to mine.
And on android it works normally.

The only difference between my code and his is that I test a parameter I receive from FCM.
EX:
Future<void> _navigateToBlog(Map<String, dynamic> message) {
    final extra = int.parse(message['data']['extra']);
    if(extra  != null) {
       Get.to(BlogPage(extra: extra));
    }
  }

On Android we need to add on manifest to work:
<intent-filter>     <action android:name="FLUTTER_NOTIFICATION_CLICK" />     <category android:name="android.intent.category.DEFAULT" /> </intent-filter>
On iOS do we need to do something similar to work?

I found that the message object on android is different from iOS
I solved my problem with this code:
extra = Platform.isAndroid ? message['data']['extra'] : message['extra'];


Not able to Resubmit/Retry workflow with volumeClaimTemplates

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issues exists when I tested with :latest
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?
Hi Team,
We are using volumeClaimTemplates to create a Persistent Volume Claim (PVC) at the beginning of the workflow, making it accessible to various tasks or steps within the workflow. However, when attempting to resubmit the workflow, it continues to reference the outdated PVC, which was deleted due to the volumeClaimGC process triggered upon workflow completion. Please suggest how can we resubmit workflow.

Version
v3.4.5
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: workflow-issue-
spec:
  volumeClaimGC:
    strategy: OnWorkflowCompletion
  entrypoint: main
  volumeClaimTemplates:
    - metadata:
        name: workspace
      spec:
        accessModes: [ ReadWriteOnce ]
        resources:
          requests:
            storage: 1Gi
  volumes:
    - name: workspace
      persistentVolumeClaim:
        claimName: "{{workflow.name}}-workspace" ## PVC gets created with {{workflowname}}-{{volumeClaimTemplates.name}}
  templates:
    - name: main
      script:
        image: python:3.9
        command: [ bash ]
        source: |
          ls -lrt /workspace
          exit 1
        workingDir: /workspace
        volumeMounts:
          - name: workspace
            mountPath: /workspace

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

A resubmit will create a new Workflow, which will create a new PVC.
A retry will re-use the same existing Workflow.
See also #11625

Did resubmit also not work @om3171991 ?

@juliev0  @agilgur5 - Yes resubmit started working after making some changes to PVC configs. I guess some more information is required put in documentation specially with PVC usage.
Retry seems still a problem. In my use-case, we are firstly cloning a git repo in PVC as input artifacts and then same PVC is used across all other steps in Dag. Hence once Workflows fails, that PVC gets cleaned up and with retry its fails saying mount point is empty.
Also a point to highlight here, once workflow is failed/completed - we are sending that workflow status to gitNotifer workflow to update workflow status to PR which will be always a success and might be this causing a PVC clean up.
Please suggest If I am doing wrong here or any way to mitigate this issue or disable retry button for Devs.

Yes resubmit started working after making some changes to PVC configs. I guess some more information is required put in documentation specially with PVC usage.

@om3171991 what changes needed to be made for resubmit? It should work without changes as it just creates a new Workflow.

Please suggest If I am doing wrong here or any way to mitigate this issue or disable retry button for Devs.

Yea so retry is much more complicated. It only retries failed steps. All successful steps will not be retried. So your clone step will not be re-run and creating the PVC would not be rerun.
For stateful workflows, retry is often not a good option.
There is not currently a way to disable retries on specific workflows. See #6490 as a feature request for granular RBAC.
In the case of volumeClaimTemplates, it might make sense to disable the retry button entirely. While there are cases where some steps in a Workflow that uses volumeClaimTemplates can be retried, most steps usually cannot be retried as they rely on the state of the volume


[Feature] Network module: Network connected but no internet format

Attempt to resolve #1487.
It seem to work with wifi but I couldn't test it with ethernet because I lack an ethernet port on my laptop.
The network module will try pinging the list of servers in the array "ping-servers" (the module will keep its default behaviour if it is empty or not initialised) every "interval" seconds. I also added the format for WiFi without internet "format-wifi-no-internet" and the format for ethernet without internet "format-ethernet-no-internet" if the pings fail

Hey, I saw that this pull request never had any comments or anything. Would it be possible to review my code and tell me if I need to modify it, or if it is an eldritch horror of an abomination, or anything in between ?
I guess my pull request just went unnoticed x) (tbh i forgot it too)


Adding NoSync to clarify this options

There is this error message in Repl and it is not clear which option is NoSync
"The subscription(s) have been marked inactive and must be reinitialized. NoSync subscriptions will need to be dropped and recreated."

@PiJoCoder : Thanks for your contribution! The author, @stevestein, has been notified to review your proposed change.

@PiJoCoder - thanks for updating this.
#sign-off


The app was excluded from Flathub

The app was excluded from flathub, will that be corrected, please?

I noticed too, saw it just couple of hours before and couldn't install it from flathub anymore

I noticed too, saw it just couple of hours before and couldn't install it from flathub anymore

So sad about it. A simple app that does what we need it to do: opens WhatsApp and notify about messages without need for tweak on settings.

That's terrible.

Does anyone know why the app was removed from the flathub store? :(
will it be added back?

Does anyone know why the app was removed from the flathub store? :( will it be added back?

I would like to know also.

It was delisted by the Flathub theme due to trademark violation: https://github.com/flathub/io.github.mimbrero.WhatsAppDesktop/issues/6.
Theoretically, it could be revived when the maintainer of this app changes its name and icon to better indicate that it's not officially affiliated with WhatsApp.

It was (temporarily) delisted by the Flathub team due to trademark violation: flathub/io.github.mimbrero.WhatsAppDesktop#6.
Theoretically, it could be revived if the maintainer of this app changes its name and icon to better indicate that it's not officially affiliated with WhatsApp.

Now I see why. It's sad. Let's wait for @mimbrero to fix that for us, please.
God bless you all!

I switched to ZapZap, which is even better.

@mimbrero  can you upload the flatpack file here? We are still waiting for you app to come back. Please!


Moving test to JAVA REST API

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


the pix loss

Thank you very much for you job  firstly
When I try to exchange    l_g_pix = self.l_pix_w * self.cri_pix(self.fake_H, self.var_H)   with    l_g_pix = self.l_pix_w * self.cri_pix(self.var_H,self.fake_H),  I found the value of l_g_pix  is  differend ,How to explain this?
Hope for reply

It is strange because cri_pix is position-independent.
Can you attach your log or provide more details?


JWT VCs relative kid

handle relative kid in the JWT-VC. Whenkid is a relative URI, it is relative to the URI contained in iss.
support for both relative & absolute kid

Also see: https://github.com/decentralized-identity/jwt-vc-presentation-profile/issues/44


[DOCFIX] update docs/cn/Configuring-Alluxio-with-S3.md

update docs/cn/Configuring-Alluxio-with-S3.md, sync with docs/en/Configuring-Alluxio-with-S3.md

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14384/
Test PASSed.


Update question in optimize media assets lecture

Here is the second question for that lecture video
Which file type should you prefer?

That is really vague. Here is a better question to use in its place.
## --text--

Which of the following is NOT a valid image file type?

## --answers--

TS

---

PNG

### --feedback--

Review the video starting at the 53 second mark to see which of these options was not mentioned.

---

JPG

### --feedback--

Review the video starting at the 53 second mark to see which of these options was not mentioned.

---

WEBP

### --feedback--

Review the video starting at the 53 second mark to see which of these options was not mentioned.

## --video-solution--

1

Here is the file you need to update
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/25-front-end-development/lecture-working-with-media/67167835def3588873f339c6.md

This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum
Happy coding


preserve colors in diagnostics

rustc uses a lot of colors and highlighting in its errors. sometimes the output is hard to read when those are removed. consider for example this diagnostic, where it's hard to parse that the - is meant to be a highlight:
it would be nice if playground preserved the colors.
repro here

It might be enough to use anstyle-svg for this, but that would require additional work for the existing links. A likely better option would be to have the json output emit the colored ANSI output and use some other crate to turn the codes into HTML that we can style.

rustc uses a lot of colors and highlighting in its errors. sometimes the output is hard to read when those are removed

Should that be a general concern for people who don’t have colors for whatever given reason?

@shepmaster I make an effort to make sure that all output is readable as text only, but undoubtedly color makes the output easier to scan and reduces the likelihood of misreading things. In the screenshot, someone very knowledgeable mistook the - as a connector between the ^^ and help.... With colors, the help appears blue, like the -, making it easier to differentiate from the ^^. That being said, this suggestion in particular is also being changed to be verbose style so that we no longer have this confusion. I want us to move away from inline suggestions entirely (but that requires me building a PR while --blessing stderr files for every platform, which is a bit of a pain), but that will still leave some corner cases where non-suggestion short labels can be misread slightly.

Should that be a general concern for people who don’t have colors for whatever given reason?

note that this particular case is being fixed upstream in rustc: https://github.com/rust-lang/rust/pull/132780
but it would be nice to fix it in the general case as well; it's never going to be possible to communicate as much information as clearly with text alone.


Here I tried to add the installer package for the C/C++ compiler MinG…

…W, so everyone can download this package through Winget easily.

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

Microsoft Reviewers: Open in CodeFlow

I hope this will turn out good. I've tried out my best.

Wonderful  contribution!

Approved!

I've tried to add Silent and SilentWithProgress switches but none of them worked for the setup.

@ritchielrez I did some searching and I found a very old reference to a request to support a silent install for minGW.
https://sourceforge.net/p/mingw/bugs/2176/
I'm not sure if there is a way to provide a scripted install with default options applied or not. That issue is quite old.
We also don't have support for filtering or preferring silent install which is why we require silent in the community repository.
https://github.com/microsoft/winget-cli/issues/910
Once that issue has been implemented, we may consider lifting that restriction. Users would be able to avoid interactive installers for their automated install processes like import.
https://github.com/microsoft/winget-cli/issues/910

The manifest schema has also changed. This needs to be updated to the v1.0 schema.

I don't use Windows anymore. If anyone can resolve this issue it would be a contribution.


Newb Nightly build

Resources:
Before submitting an issue, please consult our troubleshooting guide (http://ionicframework.com/docs/troubleshooting/) and developer resources (http://ionicframework.com/docs/developer-resources/)
Please make sure you are posting an issue pertaining to the Ionic Framework. If you are having an issue with the Ionic Pro services (Ionic View, Ionic Deploy, etc.) please consult the Ionic Pro support portal (http://support.ionicjs.com)
Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x ] 3.x
[ ] 4.x
$ ionic --version
3.12.0
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

$ sudo npm install ionic-angular@nightly --save
Expected behavior:

install without warnings ;-) ??
Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

npm WARN saveError ENOENT: no such file or directory, open ‘/Users/benjohansen/package.json’
/Users/benjohansen
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>
npm WARN enoent ENOENT: no such file or directory, open '/Users/benjohansen/package.json’
npm WARN enoent ENOENT: no such file or directory, open '/Users/benjohansen/node_modules/node.js/package.json’
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN benjohansen No description
npm WARN benjohansen No repository field.
npm WARN benjohansen No README data
npm WARN benjohansen No license field.

Hello! Thank you for opening an issue with us!
I am going to close this as this is not an issue. The current nightly build has specific version requirements for various libraries and your project apparently has the wrong ones. Update the libraries in your project as listed and you won't get the warnings.
 My suggestion is to use sudo only when it is needed, and it should never be needed for local stuff only global stuff.
Thank you for using Ionic

Thanks so much for replying… ;-)
after reading this reply several time, it dawned on me that i needed to run
npm install<EMAIL_ADDRESS>—save
in the project folder.
ok, works now.
If, I might add one question. if you can spare for newbie.
I have iPHone 6 plus, which is on iOS 11 (I can explain events why phone is on 11 if you want :-)
I have set the project in Xcode to be 11.
All is good, I can install it on phone and it runs.
But it covers, the phone status bar (the one with batter and time and server connect.
How can i get the app to stop covering this?

On Sep 28, 2017, at 3:44 AM, Ken Sodemann<EMAIL_ADDRESS>wrote:
Hello! Thank you for opening an issue with us!
I am going to close this as this is not an issue. The current nightly build has specific version requirements for various libraries and your project apparently has the wrong ones. Update the libraries in your project as listed and you won't get the warnings.
 My suggestion is to use sudo only when it is needed, and it should never be needed for local stuff only global stuff.
Thank you for using Ionic
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ionic-team/ionic/issues/13003#issuecomment-332798915, or mute the thread https://github.com/notifications/unsubscribe-auth/Ae2rjBftKmA_RXv9WMXTZAjDuxwqsHKEks5sm3ilgaJpZM4Pmyxa.

Ben Johansen
http://www.webspinr.com
<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>

I assume you are following these directions for iOS 11 - https://blog.ionic.io/ios-11-checklist/
Make sure you have the correct version of the status-bar plugin. I believe the instructions in the blog are still correct on that one.
Otherwise, I am not sure. You may have to duplicate it in a sample project, share the sample project via GitHub, and open an issue.

thanks so much, for your help.
I added these lines to config.xml

and all is good, thanks

On Sep 28, 2017, at 4:10 AM, Ken Sodemann<EMAIL_ADDRESS>wrote:
I assume you are following these directions for iOS 11 - https://blog.ionic.io/ios-11-checklist/ https://blog.ionic.io/ios-11-checklist/
Make sure you have the correct version of the status-bar plugin. I believe the instructions in the blog are still correct on that one.
Otherwise, I am not sure. You may have to duplicate it in a sample project, share the sample project via GitHub, and open an issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ionic-team/ionic/issues/13003#issuecomment-332804590, or mute the thread https://github.com/notifications/unsubscribe-auth/Ae2rjL5okyc-K1H7JPtDOVB0p0IRoozMks5sm364gaJpZM4Pmyxa.

Ben Johansen
http://www.webspinr.com
<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>


Does application supports service and configmap componentKinds ?

Hi ,
I was trying to create application which is having  service, configmap and statefulset componentkinds.
 componentKinds:
    - group: core
      kind: Service
    - group: apps
      kind: StatefulSet
    - group: core
      kind: ConfigMap

But in application components list I am seeing only statefulset status. no service and configmap. Is this expected behaviour?
status:
  components:
  - link: /apis/apps/v1/namespaces/default/statefulsets/sample
    name: sample
    status: Ready
  - link: /apis/apps/v1beta1/namespaces/default/statefulsets/sample
    name: sample
    status: Ready
  - link: /apis/apps/v1beta2/namespaces/default/statefulsets/sample
    name: sample
    status: Ready

Here is my sample application
sample_application.txt
apiVersion: app.k8s.io/v1beta1
kind: Application
metadata:
  name: "sample"
  labels:
    app.kubernetes.io/name: "sample"
spec:
  type: "sample"
  selector:
    matchLabels:
     app.kubernetes.io/name: "sample"
  componentKinds:
    - group: core
      kind: Service
    - group: apps
      kind: StatefulSet
    - group: core
      kind: ConfigMap
  version: "1.0"
  description: "Sample application to showcase Application usage with diff components"
  maintainers:
    - name: Hemantha
      email<EMAIL_ADDRESS>  owners:
    - name: Hemantha
      email<EMAIL_ADDRESS>  keywords:
   - "components"
   - "sample"
  links:
    - description: About
      url: "https://sample.org/"
---
apiVersion: v1
kind: Service
metadata:
  name: sample-hsvc
  labels:
    app.kubernetes.io/name: "sample"
spec:
  ports:
    - port: 8080
  selector:
    app.kubernetes.io/name: "sample"
  clusterIP: None
---
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: sample
  labels:
    app.kubernetes.io/name: "sample"
spec:
  selector:
    matchLabels:
      app.kubernetes.io/name: "sample"
  replicas: 1
  serviceName: sample-hsvc
  template:
    metadata:
      labels:
        app.kubernetes.io/name: "sample"
    spec:
      containers:
      - image: registry1-docker-io.repo.lab.pl.alcatel-lucent.com/busybox
        name: sample-app
        command: ['sh', '-c', 'echo Hello Kubernetes! && sleep 3600']
        ports:
        - containerPort: 8080
          name: test
---
apiVersion: v1
kind: Service
metadata:
  name: sample-svc
  labels:
    app.kubernetes.io/name: "sample"
spec:
  ports:
    - port: 8080
  selector:
    app.kubernetes.io/name: "sample"
---
kind: ConfigMap
apiVersion: v1
metadata:
  name: example-config
  labels:
    app.kubernetes.io/name: "sample"
data:
  example.property.1: hello

I dont see service and configmap in obseved resources, here is the logs from application controller pod
2019/09/03 16:05:28 *v1beta1.Application/default/sample Validating spec
2019/09/03 16:05:28 *v1beta1.Application/default/sample Applying defaults
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  { reconciling component
2019/09/03 16:05:28 Matching labels:  map[app.kubernetes.io/name:sample]
2019/09/03 16:05:28 Matching labels:  map[app.kubernetes.io/name:sample]
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  Expected Resources:
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   exp: default/*v1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   exp: default/*v1beta1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   exp: default/*v1beta2.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  Observed Resources:
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   obs: default/*v1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   obs: default/*v1beta1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   obs: default/*v1beta2.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  Reconciling Resources:
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   nochange: default/*v1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   nochange: default/*v1beta1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   nochange: default/*v1beta2.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  } reconciling component
2019/09/03 16:05:28 *v1beta1.Application/default/sample Validating spec
2019/09/03 16:05:28 *v1beta1.Application/default/sample Applying defaults
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  { reconciling component
2019/09/03 16:05:28 Matching labels:  map[app.kubernetes.io/name:sample]
2019/09/03 16:05:28 Matching labels:  map[app.kubernetes.io/name:sample]
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  Expected Resources:
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   exp: default/*v1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   exp: default/*v1beta1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   exp: default/*v1beta2.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  Observed Resources:
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   obs: default/*v1beta2.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   obs: default/*v1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   obs: default/*v1beta1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  Reconciling Resources:
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   nochange: default/*v1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   nochange: default/*v1beta1.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)   nochange: default/*v1beta2.StatefulSet/sample
2019/09/03 16:05:28 *v1beta1.Application/default/sample(cmpnt:app)  } reconciling component

There is a bug that we need to fix.
Can you check the logs and search for "delete"

group: v1
kind: ConfigMap

@hemantha-kumara


add "no-unused-vars" to ignore list in base configuration

ESLint's recommended rule "no-unused-vars" triggers if a YAML comment starts with the word "global", because ESLint takes this as file-level definition of global symbols, which is useless in YAML.
# global symbols
data: [my, global, symbols]

gives
  1:11  error  'symbols' is defined but never used  no-unused-vars

Thus, this rule should go into the base config next to "no-irregular-whitespace" and "spaced-comment".

I think it's fine to turn off no-unused-vars in the base config.


"ForEachVoronoiCell" and "ForEachVoronoiEdge" yield inconsistent drawing results.

Hello, I used the ForEachVoronoiCell method to draw all the edges of the cells, and the result is inconsistent with using ForEachVoronoiEdge. There are overlapping parts in ForEachVoronoiCell. I found that there is also this situation in the example of the documentation.

Hey, thanks for pointing this out!
To help check out the issue, could you share a minimal example, mainly the list of points you're using? That would make it easier to reproduce the problem with ForEachVoronoiCell and ForEachVoronoiEdge and see what’s going on.
Thanks!

Test.json

Thank you for your reply, I am using this library in Unity.
`
using DelaunatorSharp;
using DelaunatorSharp.Unity.Extensions;
using System;
using System.Collections.Generic;
using System.Linq;
using UnityEngine;
namespace Assets.Test
{
public class Test : MonoBehaviour
{
    private Delaunator _delaunator;

    void Awake()
    {
        var jsonPath = "Test";
        var json = Resources.Load<TextAsset>(jsonPath).text;
        var points = JsonUtility.FromJson<Vector2Wrapper>(json).vectors;
        _delaunator = new Delaunator(points.Select(point => new Vector2(point.x, point.y)).ToPoints());
    }

    [System.Serializable]
    private class Vector2Wrapper
    {
        public List<Vector2> vectors;
    }

    private void OnDrawGizmos()
    {
        if(_delaunator == null) return;
        _delaunator.ForEachVoronoiCell(cell =>
        {

            IPoint[] newArray = new IPoint[cell.Points.Length + 1];
            Array.Copy(cell.Points, newArray, cell.Points.Length);
            newArray[newArray.Length - 1] = cell.Points[0];
            for (int i = 0; i < newArray.Length; i++)
            {
                if (i == newArray.Length - 1) return;
                var point = newArray[i];
                var nextPoint = newArray[i + 1];
                Gizmos.color = Color.white;
                Gizmos.DrawLine(new Vector3((float)point.X, 0, (float)point.Y), new Vector3((float)nextPoint.X, 0, (float)nextPoint.Y));
                Gizmos.color = Color.yellow;
                Gizmos.DrawCube(new Vector3((float)point.X, 0, (float)point.Y), Vector3.one * 2);
            }

        });

        //_delaunator.ForEachVoronoiEdge(edge =>
        //{
        //    Gizmos.DrawLine(new Vector3((float)edge.P.X, 0, (float)edge.P.Y), new Vector3((float)edge.Q.X, 0, (float)edge.Q.Y));
        //});
    }
}

}
`

Unfortunately, I couldn't find a solution to this problem.
I suspect it might be related to LINK
"...constructing the Voronoi cell along the convex hull requires projecting the edges outwards and clipping them. The Delaunator library doesn’t provide this functionality."
I've committed an update to the WPF example, where I recreated a minimal case. The issue occurs for DrawVoronoi > ForEachVoronoiCell > cell.Index == 3. If anyone can find a solution, contributions are welcome.

Isn't this happening because the default behaviour is to draw the voronoi diagram using the triangle centroids (not the circum-centres) as the voronoi diagram nodes. The true voronoi diagram uses circumcentres.


Fix parser to correctly return None with an unavailable class.

Recreated for automated coverage tests.
Currently, due to a change server side, pyvt doesn't return None when an unavailable class section is queried, instead creating an invalid Section object which only consists of a 'CRN' that is actually an error message. It seems the timetable used to return a blank table, but now returns a table with an error message.
I added some checks to ensure unavailable/invalid sections return as None.

Yikes. Well now I know how the 'coverage' command works. I'll see if I can fix this in my spare time today.

@LordSamanon Try running these commands for when you're changing something:
First, install the the required packages and whatnot by running
pip install -r requirements.txt
pip install coveralls

Then, to run the tests, run
coverage run --source=pyvt -m unittest discover -s tests -p "test_*.py"

I have now. Should have run those before I made the pull request.
Found the issue though. We decided it was worth the overhead of calling _parse_row() twice for readability. The tests expected otherwise. I'll update the tests to reflect that.

also make sure that when you're working with this, you run it inside of a python 3.6 virtualenv (or docker container) as that is what Travis CI is using. Some of the mocking found in the tests I believe is exclusive to 3.6

just reviewed, looking good @LordSamanon

also for a future note, instead of running coveralls with coverage run --source=pyvt -m unittest discover -s tests -p "test_*.py" you can run python unittest discover -s tests -p "test_*.py". Coveralls is a utility for checking the code coverage from our tests.


Fixed #25068 -- Adding migrations support for models with metaclasses.

Original ticket: https://code.djangoproject.com/ticket/25068
First proposed fix: https://github.com/django/django/pull/4968
I also encountered the same problem as kosz85.
I have included the relevant test from the pull request 4968.
It seems to make no difference to the test-run times compared to master.
Happy new year!

I've found that the migrations take a long time to run for our 170 or so models, almost all of which have metaclasses, to inherit some common security/logging/concurrent update checking functionality. The tests take around 35 seconds to run the migrations before starting. After some simple profiling and reading the code, I believe the problem is that the algorithmic complexity of StateApps.render_multiple is O(n2), i.e. that this is a more general problem. I guess this is a known issue.

Referring to my comment above about complexity, I tried removing almost all of our metaclasses and it made no different to the time to run migrations.

Closing as per "not the right approach" comment on the ticket.


[POC] add hide functionality to menu components

WHY
BEFORE - What was wrong? What was happening before this PR?
We've implemented menu components in https://github.com/Laravel-Backpack/CRUD/pull/5135/ . One nice feature Pedro mentioned was the ability to hide a component, upon a condition. For example, when a user is not an admin, hide a menu item.
AFTER - What is happening after this PR?
You can do exactly that, by using the :hidden attribute on all components we have:
<!-- Users, Roles Permissions -->
@php
$is_admin = true;
@endphp

<x-backpack::menu-dropdown title="Authentication" icon="la la-group" :hidden="$is_admin">
    <x-backpack::menu-dropdown-item title="Users" icon="la la-user" :link="backpack_url('user')" />
    <x-backpack::menu-dropdown-item title="Roles" icon="la la-group" :link="backpack_url('role')" />
    <x-backpack::menu-dropdown-item title="Permissions" icon="la la-key" :link="backpack_url('permission')" />
</x-backpack::menu-dropdown>

HOW
How did you achieve that, in technical terms?
The hidden attribute can be:

a boolean
a Closure

That gets evaluated, and if it's false the PHP Component itself won't even load the blade file. No HTML gets written on page.
Is it a breaking change?
No - it's non-breaking. No PRs to themes is necessary.

The PR is 100% working but... I don't really like it:

I'm not 100% sure we should use :hidden and not :visible as the component attribute name;
I'm not 100% sure we are accepting all variable types that devs might want;
I'm not 100% sure we should do this in the first place, since a dev can easily wrap the <x-something /> into a conditional; so from this point of view we aren't adding any features, we're just polluting the components for something that can already be done, very easily;

For the reasons above, I'm not going to spend any more time on this right now. We have a launch to finish, so let's do that. This is a COULD/SHOULD and non-breaking, so let's postpone thinking or talking about this until after the launch.


Entropy grid regeneration(File import): Not supporting Number/Hexadecimal grid.

In case of Number or Hexadecimal grid it is always showing words.

Verified this issue on Tribe app v 2.2.2(592)


Backwards compatibility policy

It would be nice to have an explicit backwards compatibility policy.
Some suggestions:

Everything must be backwards compatible
Semantic versioning
Some window of deprecation warnings

I'm really okay with anything. They all seem unideal, but my feeling is something just needs to be written down and adhered to; the alternative is leaving a lot of PRs (#2092, #2450, ...) in purgatory + avoiding anything but minor changes to the repo. This feels unsustainable.
An alternative model is to move toward an approach like cleanrl, where core algorithmic details are designed to be copy-and-pasted, duplicated, etc. I like this but probably the code would need to be architected very differently to make the experience nice. 🙂
cc @vye16

Thanks for addressing this, I agree that a policy for this is needed as new approaches to NeRF may require substantial changes to the code base in the future.
Personally, I prefer the semantic versioning + deprecation window approach.
And while I appreciate the cleanrl approach, I think this code base has been going in the exact opposite direction with all its abstractions that converting this code base to a non-modular approach would take longer than creating a cleannerf repo from scratch.

Fully agree with last paragraph of @brentyi. I think this framework under active development right now and did not support lots of functionality (gaussian, as easy example). After supporting this, can go to backwards compatibility. You can define a list of features that should be completed before this and move on to a new stage of versioning.

The new-ish Nerfstudio template ( https://github.com/nerfstudio-project/nerfstudio-method-template ) is a pretty decent "API" for isolating core from research space.  Could require any new algos to start there and then "graduate" to batteries-included Nerfstudio, and then try to support backwards-compat for only the nerfstudio core versus particular algos, which may include nerfacto.
The template could also be expanded a bit to include:

API for data processing pipelines / ns-process-data.
API for offline rendering; _render_trajectory_video() is doing a lot of work...

There are of course things like the cited camera optimization PR that are pretty sweeping and don't really fit the template.  BUT the automated testing for nerfstudio today is a bit lacking; there are several changes that got merged and then broke lots of stuff and required follow-up PRs.  So today, it's easy to end up with a sweeping PR that only really works on your own computer.
So for larger changes, one strategy could be to force such changes to use the template, and to even copy-pasta core code as needed, then "graduate" certain things to core nerfstudio.  E.g. a 'contrib module' approach, where backwards compatibility and other expectations are relaxed prior to graduation.  Nerfstudio-the-project would probably want at least one full-time maintainer to help shepherd contrib->core, and it's totally possible for that to be too much to ask for at this point of the project.
For the "version-managed core," a key problem for backwards compatibility is schema evolution of the config files, transforms, camera path, and maybe even some checkpoints.  But could easily embrace a lightweight protobuf-like convention:

Add a field? Throw an error if it's required and missing
Never change the type of a field; only create new fields.
Delete a field?  Comment it out (don't delete it, leave breadcrumbs) and write some sort of deprecation warning if some code ever reads it.

For a pure-python, no-protobuf impl, stuff like pydantic offers a lot of the boilerplate validation built-in.  Maybe tyro already has some of that?
For weights, my vote is to assume NO backwards compatibility, and that the git commit + nerfstudio version info should be baked into checkpoints.  Seems par for the course in ML that you have to either re-train or do your own weight surgery when the code changes substantially.  But that can certainly be a foreign concept to e.g. 3D artists.
A good case study could be taking a bit of time to estimate what it would take to merge SDFStudio back into Nerfstudio, and think about where backwards compatibility plays in there.  While I doubt such a merge would ever happen, it's a pretty good example of the copy-pasta lifecycle for this repo.

Well if there’s limited support for backwards compatibility, would be cool
if at least the configs and torch checkpoints all get stamped with the
nerfstudio version + git commit if available.   That makes archeology
easier and can help anybody who opts in to implementing backwards support
to figure out if the user is trying to do something deprecated etc.

Makes sense to me! Perhaps we can just write a file with information like a commit hash or version number in the outputs directory where the checkpoints/config files are serialized.


Predicting a bugfix release?

It looks like 1.2.2 is in the metadata but only 1.2.1 is released on the forge. Is this on purpose because the release is pending? What if the next release has a feature instead of a bugfix?
If you could keep a changelog documenting that it is a pending release, that would be great! http://keepachangelog.com/ documents a great changelog layout. (And it would be helpful to see the semantic versioned changes between releases.)

Hey Hunter! Thanks for the feedback! I'll put together a backdated changelog tonight, with the intent on keeping it updated in the future.
As for the version being 1.2.2 in the metadata, I tend to use the release rake task in blacksmith that automatically increments the patch level in anticipation for the next release. I don't have anything planned for 1.2.2 yet (although I've been contemplating removing the dependency on camptocamp/archive from metadata and noting it in the README, as it's not essential for the core functionality of the module). If we do introduce a new feature, I would release it as 1.3.0.

@hunner if you agree that this is resolved with #9, I'll go ahead and close this issue.


shuffle files around to make local development of multi_inventory easier

move gce.py and aws.py to a location where multi_inventoy.py can just work straight out of the git repository
update RPM spec to account for new location
clean up some unused/unpackaged files

The build bot errors are from the vendored ec2.py and gce.py.
:+1:


Why do we need a separately stored car_nerf_state_dict?

Dear MARS team, thank you for opensourcing this awesome work! I'm trying to understand the codes but encountered some problems:
It seems mandatory to properly fill the car_nerf_state_dict_path from cicai_config.py in order to start a regular training. But is this car_nerf_state_dict_path really necessary? Here's my confusion:
It looks like the car_nerf_state_dict_path was only used in line 82-83 from car_nerf.py to initialize the self.fields of a CarNeRF class. However, since CarNeRF is a subclass of nn.Module, self.fields will be automatically discovered by PyTorch and saved into a training checkpoint under the nerfstudio training framework. This would mean the weights loaded from car_nerf_state_dict_path will be later overwritten by the weights from the training checkpoint file outputted by nerfstudio, during its trainer.setup() method.
Since the weights from car_nerf_state_dict_path would be overwritten anyway, it seems the separately stored car_nerf_state_dict doesn't serve any purpose. Can you help me with this confusion?
I am working on the master branch (commit ID: 5d7567504220f9a8a09aa20a199954f481a26484).
Thank you!

Hi, I would recommend you referring to the CarNeRF paper. We use the pretrained carnerf to initialize, that is the reason why we need to load a checkpoint separately.

Thanks for the reference!


Outline nested items gui performance issue

Rendering more than 50 nested levels causes lag in Outline and the dropdown Breadcrumbs view
In my test cases:
The provideDocumentSymbols() can handle about 600,000 DocumentedSymbols before I start getting OutOfMemory errors
Takes around 30sec to load 600,000 sibling symbols
But once loaded, gui scrolling is smooth
provideDocumentSymbols() can only handle a nesting depth of around 950 levels
going further results in getting call stack errors
however the gui loads instantly when at the top of the outline view
But rendering more than 50 nested levels at once starts to lag when scrolling
at 500 nested levels it takes in excess of 3minutes to scroll
notice the scrolling starting out smoothly
then progressively getting laggier
https://github.com/microsoft/vscode/assets/33529441/dc5b554c-ea13-4e2b-87d6-8e99da1ddf61
Rendering horizontally should be optimized
as only a small porition of the outline space is actually in view
{
	"scopeName": "source.nesting",
	"patterns": [{ "patterns": [{ "patterns": [{  }] }] }]
}

Please share renderer profile and the spec of your machine
Hit F1 > Toggle Developer Tools. In the overflow menu select 'More Tools > JavaScript Profiler'. In there select start, let it profile for 30 to 60 seconds, stop it, save it to a file and attach the file here. Thanks.
https://user-images.githubusercontent.com/1794099/120606621-a97e0180-c44f-11eb-8e32-9fdafdc949ec.mov

found it
I had to enable it under Settings > Experiments

heres the cpu profile
CPU-20240222T101438.cpuprofile
and a much longer profile
CPU-20240222T101927.cpuprofile

The performance drop is situated in the gui tree graph view rendering
having hundreds of thousands of sibling nodes is fine, but going above 50 nested levels tanks performance during render updates
The tree view is used in multiple places
Outline, References, Breadcrumbs, Explorer, SCM, Run & Debug etc
Call Hierarchy References:

how come my previous comment was marked as off-topic?
were you able to confirm the performance issue?


update user guide

Motivation
Update user guide and make more modular.

Kind of an odd appearance for the schema sections - they show as separate tabs on the left but are contained on the same actual page (index just routes to the other section heading). Guessing this is intentional but it looked/felt a bit strange for them to not be on separate pages since all the other tabs are.
Not a huge deal, just noting.

There are some merge conflicts, though. CLI-level ones that can't be resolved on the web interface.


v8-315: formula description added

brew audit --strict v8-315 returns:
homebrew/versions/v8-315:
 * Formula should have a desc (Description).

--

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

This PR eliminates this warning.

Thanks again @miguelarauj1o!


The system will handle cookie based session authentication

This will be tested against Django authentication.

See https://stackoverflow.com/questions/10628275/how-to-use-curl-with-django-csrf-tokens-and-post-requests


[Custom Logo Build]: test if svg works

Board Name
delbin
Logo

@CoolPenguin27 would this be the expected PNG form of the SVG image?


Functionality to have NodeJS inbuilt modules to the top

Hi there, thanks for this package, really cool! Will consider recommending it to our bootcamp students at @upleveled.
One suggestion that came to mind would be to make it the default sort order to sort imports using the new node: scheme / prefix to the top. What do you think?

Any update on this? If not, I could take a look.

Howdy! I've picked this up and a PR will be incoming momentarily.

In case anyone here is still looking for this, until this project adds support, you could use my fork, which is published on npm and supports this feature: https://github.com/IanVS/prettier-plugin-sort-imports#importorderbuiltinmodulestotop

Maybe it could be a importOrder classifier like <THIRD_PARTY_MODULES>?


No camera device found on the system on some Android device

Package Name: azure/communication-calling
Package Version: 1.6.3
Operating system: Android 9
[ ] nodejs

version: v14.19.0

[ ] browser

name/version: chrome on Android version 105

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
I face with this issue "No camera device found on the system" even though camera and microphone allowed in chrome permissions already. Also when I tried testing webRTC on this device (https://webrtc.github.io/samples/src/content/getusermedia/gum/) it also returns no camera. But when testing on Samsung note 9, both ACS JS sdk and webRTC works fine.
To Reproduce
Steps to reproduce the behavior:

run this sample https://docs.microsoft.com/en-us/azure/communication-services/quickstarts/voice-video-calling/get-started-with-voice-video-calling-custom-teams-client from my target device, it is without gms installed, on chrome Android latest browser.

Screenshots
If applicable, add screenshots to help explain your problem.

This is very likely to be a Chromium/Chrome bug. At least on Google Chrome 105 (I tested on several Linux devices), mediaDevices.getUserMedia() doesn't even ask the user to authorize access to the camera. Even adding Camera access permissions manually on the browser for a particular website results in the failure of the browser to capture the camera device.
Developer tools only shows:
DOMException: Requested device not found

Thanks! @TheCodeTherapy
I tested webRTC on Firefox and it also works fine but I see that ACS not support for Firefox yet.
Is it always the case that if webRTC works then ACS deviceManager.getCameras() works, if webRTC failed then ACS also failed?

@Cartier0745 ACS added Firefox support in 1.9.1 beta, currently it's still in preview
I'll close this issue but let me know in comments please if you have additional questions


[POC] Convert custom breadcrumb capture to use DataSource

Goal
Converts the capture of custom breadcrumbs to use the new DataSource interface. The CustomBreadcrumbDataSource is the most important class to look at - it shows how DataSource can be used to write a SpanEvent.
This should not be merged yet as the payload schema is not agreed upon/supported by the backend, but it's useful as an illustration of how this should work.

@bidetofevil can I get a sanity check that the SpanSanitizer approach is the way we want to go with spans? I tried using a SpanProcessor but figured it would be simpler to sanitize the payload at this point as it avoids having to decorate OTel types.


Support hudi connector

Support read data from hudi datasource.

Fixed in https://github.com/TuGraph-family/tugraph-analytics/pull/80


Error while quering the gameDaoCrowdfunding campaigns

Describe the bug
nice error screen appears when you visiting GameDao beta portal
To Reproduce
Steps to reproduce the behavior:

Go to https://beta.gamedao.co/app
See error

Expected behavior
normal site loading
Screenshots

When you quickly navigate through site you can observe stack of 3-4 of these error messages.

Console shows:


bug:  cant type anything in plugins and themes

when i open up the "plugins" button and try importing a plugin it tells me to type in a link but theres nowhere to put the link into

I think the fix you did (the one mentioned on mareks PR) causes a crash when clicking add plugin/themes button


Thank you

This is not a bug or anything, it's just to say thank you. Your library is really GREAT. We're now using it in our open source PYBOSSA software for improving our CSV exporters with Pandas.
Again, thanks for sharing this work!

@teleyinex you're very welcome. Makes me very happy that you guys are finding it useful.

So far, it works really well. It looks like we've found an issue with utf8 in the keys :-) We'll debug it, and if it's a bug I'll send you a PR.

👍

Yep, it does not work when there's utf8 key/values :-) You have a new PR with tests included. It's a very simple fix.


Update dependencies

This project uses semver, please check the scope of this pr:

[ ] #patch# - backwards-compatible bug fix
[x] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Added additional builds to CI to make sure addon works with latest versions of Ember.
Removed files from npm package that aren't necessary (all of the various config files).
Updated dependencies to latest versions.

:+1:


scrwfile package incompatible

The scrwfile package redefines \@starttoc in a way that breaks tagging. The example
\DocumentMetadata
  {
    lang=en-US,
    pdfversion=2.0,
    pdfstandard=ua-2,
    testphase={phase-III,math,table,title,firstaid}
  }
\documentclass{article}
\usepackage{scrwfile}
\TOCclone[Summary Contents]{toc}{stoc}
%\usepackage{hyperref}
\addtocontents{stoc}{\protect\value{tocdepth}=1}

\begin{document}

\tableofcontents
\section{A section}
\subsection{A subsection}
\listofstoc

\end{document}

warns
Package tagpdf Warning: something is wrong with the mcid--rerun

and the tags for the TOC are empty. Note the \TOCclone and \listofstoc are not necessary to show the issue.

I think that is a bug in the ltproperties. Unlike the standard \label  properties are not written if \if@filesw is false as there is an additional test and so the tagging code can not write the needed labels.  It works fine if I remove the additional test:
\ExplSyntaxOn\makeatletter
\cs_set_protected:Npn \__property_record:nn #1#2
  {
    %\legacy_if:nT { @filesw }
     % {
        \protected@write \@auxout {}
          {
            \token_to_str:N \new@label@record
              {#1}
              { \clist_map_function:nN {#2} \__property_record_value:n }
          }
      %}
  }
\ExplSyntaxOff
\DocumentMetadata
  {
    lang=en-US,
    pdfversion=2.0,
    pdfstandard=ua-2,
    testphase={phase-III,math,table,title,firstaid}
  }
\documentclass{article}
\usepackage{scrwfile}
\TOCclone[Summary Contents]{toc}{stoc}
%\usepackage{hyperref}
\addtocontents{stoc}{\protect\value{tocdepth}=1}

\begin{document}

\tableofcontents
\section{A section}
\subsection{A subsection}
\listofstoc

\end{document}

I would suggest to remove that test. Does that sounds ok @josephwright @FrankMittelbach ?

@josephwright if you use  \@fileswfalse in the preamble then both \label and \RecordProperties do not write.  But if you set the boolean only locally, then \label still works:
\documentclass{article}
%\makeatletter \@fileswfalse \makeatother
\begin{document}
\section{abc}
{\makeatletter \@fileswfalse \makeatother
\label{blub}\RecordProperties{bla}{page}}
\end{document}

So the behaviour is inconsequent.

I would suggest to remove that test. Does that sounds ok @josephwright @FrankMittelbach ?

yes that sounds ok to me (despite Joseph's missgivings :-))
The normal LaTeX interface consists of three parts: the switch \if@filesw, \nofilesand a redefinition of \protected@write if \nofiles is used.
So for normal LaTeX behavior the test around \protected@write is pointless as that command is silenced by \nofiles, thus the outer test can go.
In the package Markus uses the fact that the switch turns of  opening  and closing aux files for writing, but without \nowfiles the \protected@writestill "writes" which is necessary for his approach, ie writing of auxiliary files is not stopped but changed by the package and @filesw is kind of misused.
My suggestion to people is using this package here is to switch to morewrites as as that can handle arbitrary many additional file streams and just 3 more and does not depend on altered latex internals. It might be a bit slower though (not tested) but it works well.


Uno-choice Scriptler configuration displays static cached parameter values

Sorry that Scriptler script testing is coming so late in the process!
Our famous example of Brazilian State-Cities cascade has now been implemented with a Scriptler script. Re-opening the configuration I see a new Scriptler parameter (States ) that I did not add .

Q: I know that the referenced parameters are passed into the script binding. Is this what you are trying to display here? If that's the case, the static value displayed for the States parameter is confusing
I vote not to show this parameter as it is more likely to confuse.
Also, I'm re-opening #16 as it does not appear to work in v0.22. It seems that the Scriptler script requires a hard coded reference into the job parameter. The two should be decoupled using the Scriptler parameters.

Hi @imoutsatsos, I'm going have to ask you to check if this issue has been fixed. I couldn't test it before fixing two other issues today. But think it is not happening anymore.
The best explanation I have, is that before we were "merging" the UI parameters, with the Scriptler job configuration parameters. Causing the "cached" value (actually the value selected in the UI, that got merged with the scriptler params) to appear in the job configuration screen.
I believe we can close it as won't fix, but please give it a few tries and let me know if that still occurs. Going to bed now, will have another development cycle this weekend or during the next weekend. Cheers!

Thanks @kinow ! I will retest all the issues you fixed this weekend this coming week. Looks like things are really ready for releasing the Jenkins hosted uno-choice v1.0! Exciting!!

Thank you too for your patience and thorough testing @imoutsatsos !
And indeed, 2015 seems to be a promising year for Uno-Choice plug-in. Once tis released I'll bother you other ideas :)

Looks good on our Jenkins Test server @kinow ! I will further test with some projects at work today (where scriptlets are heavily used) and I will close. All of my project have now been transferred to the new code base and I'm quite pleased! Uno-choice is quite solid! Thanks!!

Yay, thanks for (again) thoroughly testing the changes. Some users reported the plug-in being slow in some cases. I'm waiting for their feedback, but I think it should not be a blocker for a 1.0 (after all, 1.1 is just a matter of few shell commands to release).

I think the old code base was slow performing. I had Jenkins build forms with 6-10 Uno-choice controls and you did have to wait for a few seconds to load and update. However, I'm finding the new code base much improved. I will give you some additional feedback once my UC v0.23 projects have been deployed for general use. Doing a 'production' roll out today!

Good luck Ioannis! Drop me a message if you need anything :D

Sees to be working now in 1.0. Please feel free to re-open an issue in https://issues.jenkins-ci.org/ with component name active-choices-plugin in case the issue persists. Thanks!


Run tests on pull requests

Updates the CI test actions to additionally run on pull requests, to enable running tests on PRs opened from forks

I'm going ignoring the Cabal failures here - this doesn't change anything about how the cabal source works and is likely just another badly versioned breaking change 😢


one method for updating object attributes

I miss an underscore method that would take an object and update all it's attributes, within a iteratee and preserver all keys. Say I've got {a:1, b:2, c:3} and I want to use a square function to receive: {a:1, b:4, c:9}. Current solution is to do something like this:
_.object(_.map(original, function (value, key) {
    return [key, value * value];
}));

but I don;t like that. It's not that readable - you need to use another 2 functions and know that _.object constructs another object from pairs. There could be one simple function for this, since updating all object attributes is not a rare task. How about adding it to underscore? An example solution can be found here:
http://stackoverflow.com/questions/26013802/underscore-js-iterate-over-object-attributes-and-modify-them

fully agree with @megawac. This is a common case afterall. (@tkoomzaaskz)

Any chance to get this into 1.8 e.g. with #1953?

@jashkenas can you make a decision on this and #1953?

This is a dupe of the implemented #1953, no? Let's move over there.


Import everything except for some

Description
Feature request #3065 is useful. Based on that, it seems to be an opportunity to also offer this "import-not" syntax:
#import "package.typ": not (
  i-dont-want-this,
  this-either
)

This way, every symbol from that file is imported (maybe except for those beginning with underscores? — debatable) except the ones designated above to avoid collisions.
The Dart language offers a similar syntax import ... hide ... which imports everything importable except for the designated names.
This Dart feature was based on research on UI dev experience, especially on large & complex projects. Since typesetting a document is similar to developing a UI, same insight should be applicable here, too.
Use Case
Users may use a lot of features offered by a package but only except for some. It often happens when typesetting a large document for a particular field that can mostly be done with the help of one (or only few) dedicated package. Also, like during UI development, users usually add names to the import list while they are writing the paper (instead of writing down the full set of symbols they will use later from the start) and this import-not syntax could make this user experience smoother since they no longer need to scroll up.

I have a different syntax idea, use #import "package.typ": * followed by exclude keyword.
#import "package.typ": * except (
  i-dont-want-this,
  this-either
)

I don't like this at all. This seems to give the illusion of fine control over your imports while keeping all of the downsides of *. To use this you must know of every feature you want to use and all features you don't want to use. And any reader of the code will have twice rhat problem—why are you telling them about what you aren't using? They don't care, they want to understand the things you did do.
If you change your package version it moves your errors to all the wrong places, and the argument about scrolling up seems completely unfounded to me—either you start with importing everything and when you're done you keep only what you used (which can be done using both import styles), or you start by importing nothing and over time you add what you're using (which can be done using both styles). Simply start with a * and when you're mostly done switch to listing each import.

For package authors I recommend the lib.typ trick: add a lib.typ file and only import symbols that should be visible to users.
https://github.com/mitex-rs/mitex/blob/main/packages/mitex/lib.typ#L1

I agree with @emilyyyylime here. Since a lot of other people do as well, I will go ahead and close this as not planned.


Missing information resetting skill output cache

On the page there is some text about invalidating skill outputs in the cache.
"The reset skills API accepts a POST request with the list of skill outputs in the cache that should be invalidated".
The full details are supposedly in the linked documentation for the reset API, but there is no information about it there.  What does a reset skills post request look like with the list of skill outputs included?
In addition, does the indexer show it is running when reprocessing is triggered?  I tried altering the url of a custom web function in the skillset definition but could never get reprocessing to run automatically.

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

ID: d4f0444e-1fea-b9be-8cca-08a26d995e60
Version Independent ID: e401363d-e606-6133-bef0-8c90fc0d17ad
Content: Incremental indexing (preview) - Azure Cognitive Search
Content Source: articles/search/cognitive-search-incremental-indexing-conceptual.md
Service: cognitive-search
GitHub Login: @vkurpad
Microsoft Alias: vikurpad

@NeilMeredith Thanks for the feedback! We are investigating the issue.

Thanks, any update on this?  One other comment, is the cached data from the indexer visible in our storage accounts?  After setting the storage connection string and enabling caching in the indexer I get a cache id back with enableReprocessing set to true in my indexer definition, but I do not see it created anywhere in my storage account.
From what I can tell the incremental indexing is not working at all. I have updated a custom skill url to test but reprocessing is not triggered and the indexer does not indicate that it is running.

For example, editing a skillset and updating the URL for a custom skill will result in the indexer invalidating the cached results for that skill

I'm sure there is some configuration issue on my end, but any additional clarity around how this should work would be helpful.

Hi @NeilMeredith, thanks for your patience. We reached out to the engineering team internally  to see if your scenario is expect behavior. Will update you when we hear back from them.

@vkurpad can you please provide some insights on this issue? Thanks!

@NeilMeredith Can you direct message me your search service name and subscription id? One possible cause of this could be that we did have a feature flag that needed to be set. This requirement has been removed, but the update itself may not have made it to your search service. Can you also try adding the enableAnnotationCache property to the indexer definition?
"parameters": {
"configuration": {
"enableAnnotationCache": true
}
}

@vkurpad I tried setting the enableAnnotationCache property but got the following error message:
{
    "error": {
        "code": "",
        "message": "Configuration property 'enableAnnotationCache' is not supported."
    }
}

I emailed you my search service name and subscription id.  Thanks.

Thank you for your help with this @vkurpad.  After you reviewed my search index state this time when I reset the indexer the cache was created, and updating a skillset definition and then re-running the indexer now recognizes the change and reprocesses the document.
Can you let me know what the reset skills post request would look like for the incremental indexing (to just reset specific skills and not the whole cache), since that is not currently shown in the documentation?  Thanks.

@NeilMeredith - Apologies. We missed creating documentation for the Reset Skillset REST API.  That doc is in the pipeline and should be published in the next day or so.

The docs are published:  https://docs.microsoft.com/en-us/azure/search/preview-api-resetskills
Thanks again for bringing this content gap to our attention!
#please-close

@HeidiSteen Great, thank you for adding this documentation!