Uses/anborod/nuget documentation

This change contains the documentation for consuming MRTK as a NuGet package.

nit: the images used in this are cropped pretty tightly, some basic rule of thumbs that i've been sticking too.

When selecting something from the Unity editor toolbar, show the full application top bar (i.e. unity logo and project name) so that folks know where this menu is located. Consider showing the solution hierarchy as well so that folks know where the Unity toolbar is located relative to other content in the editor that they may be more comfortable with using. This is a suggestion compared to an ask. I know it makes the image bigger, but just having a small section of the toolbar shown doesn't necessarily provide much context around where the toolbar is found in the editor.

When using Unity window, include the entire window in the image. Cropping into a specific area may make the documentation shorter on github, but you lose context on where specific sections of the ui may be found. If someone is unfamiliar with the new nuget for unity window, there will already be a cognitive load associated with opening the window. having to open the window + search for content on the window may be easier when the full window is shown in a documentation image.

When referencing inspector (not done in this review), show the entire inspector. Although only one may script may matter for the step, showing the entire inspector may help users more easily locate the script they are learning about.

/azp run

/azp run

/azp run

/azp run


Chrome supports "" behind a Flag

see https://code.google.com/p/chromium/issues/detail?id=87553
This is not shown in caniuse homepage

Added recently. Thanks!


Translation efforts for Dutch and English

Here's the plan:

Here is the google sheet, I'll start copying strings over from the EJS files.
There are four columns: KEY, Norwegian, which I'll copy over, and empty Dutch and English
Filling in the Dutch and English translation is a free for all:

empty space: click on it so it's locked, find the translation, fill it in, move to the next
filled translation: click on it so it's locked, check if you agree, then make the cell green. If you don't agree, update and someone else will check

This way, translations are seen by at least two people without it being a hassle or blocking.

Green translations can be added to the repo. Not sure what the best course of action for that is (considering prevention of merge conflicts on the translation files) but maybe someone has an idea there (like, add the KEY to the sheet first? i dunno)

Update: Dutch is fully translated, please review lines and fill them with green if you agree with them, or rephrase them and leave them white.

Update: English is fully translated, please review lines and fill them with green if you agree with them, or rephrase them and leave them white. Same for the remaining white Dutch lines!

Update: English is green and ready to be implemented! :tada:

Claim your view here, then add translations from the translations sheet (some translations are used on multiple pages, so be sure to check that)

app/views/pages/404.ejs
app/views/pages/500.ejs
app/views/pages/confirm-profile.ejs
fapp/views/pages/rivillige.ejs
app/views/pages/helsenorge.ejs
app/views/pages/map.ejs
app/views/pages/privacy-policy.ejs (text not translated, titles are)
app/views/pages/report.ejs
app/views/pages/statistics.ejs
app/views/partials/disclaimer.ejs
app/views/partials/footer.ejs
app/views/partials/hamburger-menu.ejs
app/views/partials/head.ejs
app/views/partials/menu.ejs

If you claim one, let me know and I'll put your name in the list here, then after a PR it can be struck from the list.

Did "app/views/pages/helsenorge.ejs". Some pc problems now. Will make PR in a bit. Then continue with others.

I'll claim "app/views/partials/head.ejs"

PR in 205, closing this :tada:


[Go Live][April 05]

Push to live

Docs Build status updates of commit c2a1b9f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/meeting-app-extensibility.md
:white_check_mark:Succeeded
View

msteams-platform/assets/images/apps-in-meetings/desktop-in-meeting-dialog-view.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Still maintened ?

Hi,
Does this plugin is still maintained (ex: to use newer versions of grammalecte) ?

+1


NEXT-8493 Added missing blocks and JS plugins

1. Why is this change necessary?
By adding these blocks this page will be more extensible.
2. What does this change do, exactly?
This PR adds missing blocks and missing JS plugins on the order page under account.
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
#855
5. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

@tobiasberge something like this?

@BrentRobert Thank you, looks good already I just left a small suggestion.

please squash your commits :blush:

@mitelg squashed!

I thought I already commented here :thinking: QA approved :heavy_check_mark:

Thank you for your contribution :blue_heart: :tada:


Allow polarity to be Double or Int

We have quite a few internal ontologies that use a Double for polarity.  Rather than changing them all to Int and maybe changing them back later, and so that we can deal with both the old and new versions of the changed ontology, I'm allowing either format for now: 1 or 1.0.

@BeckySharp, in case you want to review this retroactively.

looks great, thanks!
On Tue, Oct 8, 2019 at 2:46 PM Keith Alcock<EMAIL_ADDRESS>wrote:

@BeckySharp https://github.com/BeckySharp, in case you want to review
this retroactively.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/687?email_source=notifications&email_token=ABJCPCPQMX4UKZCDFKUA7QTQNT5Q7A5CNFSM4I6XK2L2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAVW7VA#issuecomment-539717588,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJCPCM654LQBZ3VVH3PQQLQNT5Q7ANCNFSM4I6XK2LQ
.


Any method to mute the log outputs

Hello teams,
I'm trying to extract the data for the machine learning training. The dataset extractor opens the vrs files many times. I want to avoid the log output when a provider is created. e.g.
gt_provider = AriaDigitalTwinDataProvider(data_paths)
Its outputs always take multiple lines and make other outputs hard to find. Is there any method to mute the outputs?

Hello @Yongfan-Liu , thank you for your feedback,
This is a "known" issue, this log is coming from a third party library we are using (the vrs library).
Could you try to set an empty sync before calling your code and see if this is helpful to remove the logs:
import sys
class NullSink:
    def write(self, text):
        pass
null_sink = NullSink()
sys.stdout = null_sink

Hello @SeaOtocinclus, thank you for your help.
Unfortunately, it doesn't work on my side. Here is my sample code and it still outputs lines of logs. Please let me know if I missed something.
import os
from projectaria_tools.projects.adt import (
   AriaDigitalTwinDataProvider,
   AriaDigitalTwinDataPathsProvider
)

import sys
class NullSink:
    def write(self, text):
        pass

    def flush(self):
        pass

def main():
    sequence_path = "../ADT/Apartment_release_golden_skeleton_seq100_10s_sample"
    paths_provider = AriaDigitalTwinDataPathsProvider(sequence_path)
    all_device_serials = paths_provider.get_device_serial_numbers()
    sequence_name = os.path.basename(sequence_path)
    print("all devices for sequence ", sequence_name, ":")
    for idx, device_serial in enumerate(all_device_serials):
        print("device number - ", idx, ": ", device_serial)
    selected_device_number = 0
    data_paths = paths_provider.get_datapaths_by_device_num(selected_device_number)
    gt_provider = AriaDigitalTwinDataProvider(data_paths)

null_sink = NullSink()
sys.stdout = null_sink
main()

Currently, my solution is to print the content I focused on to a txt file. So I can avoid the logs by checking the txt file.  /w\
  with open('print_output.txt', 'w') as f:
      with contextlib.redirect_stdout(f):
          main()


add clarifying comment

https://github.com/mobilecoinofficial/forest/blob/68a86b818ed2dd52c5bb78a6ef0163058050b539/forest/core.py#L1244

bump?


mongo-proxy: handlers for OP_COMMAND and OP_COMMANDREPLY messages

Description: mongo-proxy filter was missing parsers for OP_COMMAND and OP_COMMANDREPLY messages as described in #2745. Classes handling those messages were added to mongo-proxy filter along with decoder's callbacks and encoder routines.
Risk Level: Low.
Testing: Added few unit tests to test decoding from wire format, to test decoder's callbacks and to test updating proxy stats.
Docs Changes: Yes - update Mongo statistics table.
Release Notes: Yes - add support for OP_COMMAND and OPCOMMANDREPLY messages.
Fixes: #2745

@cpakulski quick non-review note: This will need some small doc changes around stats, supported commands, etc. as well as release notes. Also, can you please add "fixes ..." to the description? Thank you!

@mattklein123 Done. Release notes and docs have been updated. https://github.com/envoyproxy/data-plane-api/pull/594

@htuch It is pending on small changes on my side after @alyssawilk code review. I will do those changes within 24h.

@cpakulski LMK when you are ready with this and I can take a quick skim through since I am very familiar w/ this code.

@mattklein123 I am done with this PR.


test(loaders): Add test for load_deletions #127

Add new unittest test_load_deletions() to increase loaders.py coverage.

@lsetiawan This PR is ready for review.

Thanks @madhavmk for this! This is great. However, you're missing the file_path case: https://app.codecov.io/gh/uw-ssec/offshore-geodesy/pull/179/blob/src/gnatss/loaders.py#L289. This is when the program is given the old deletens.dat file:
deletions:
    path: ./tests/data/2022/NCL1/deletns.dat

Thanks @madhavmk for this! This is great. However, you're missing the file_path case: https://app.codecov.io/gh/uw-ssec/offshore-geodesy/pull/179/blob/src/gnatss/loaders.py#L289. This is when the program is given the old deletns.dat file:
deletions:
    path: ./tests/data/2022/NCL1/deletns.dat

Right. I purposely skipped this testcase, as you mentioned we do not support it moving forward. Do you still recommend me to create a testcase for this?

Right. I purposely skipped this testcase, as you mentioned we do not support it moving forward. Do you still recommend me to create a testcase for this?

That makes sense, we won't support it, but for now, let's test for this. It shouldn't take too much lift and this ensures that the program can be compared with an old delete file.

Right. I purposely skipped this testcase, as you mentioned we do not support it moving forward. Do you still recommend me to create a testcase for this?

That makes sense, we won't support it, but for now, let's test for this. It shouldn't take too much lift and this ensures that the program can be compared with an old delete file.

Completed!

This is now ready to merge once the small change I made is passing. Thanks for this!


The canonical extension for patchmode files is .orig

.org has quite different associations

.org is good if a three letter extension is required.  But that's less common these days, let's mention .orig first.


db::path command-line override rules

if db::path is specified on the command line then if tablespaces are not also on the command line then put them under the base pg_tblspc path without linking.  If tablespaces are mapped in the config, then make sure a warning is thrown to show that they have been remapped.
This is to prevent an existing installation from being clobbered if the base path is remapped for the purpose of testing.

This is no longer needed with new mapping options and checked added recently.


looks like db_pre_request is not running on sturage upload request

create or replace function get_group()
    returns text language sql stable
as $function$
    select current_setting('request.groups', true)
$function$;

this function returns value if i write to database.
but return null if is upload image

Oh that's an interesting discovery. How are you calling get_group()? Are you triggering it on insert of one of the storage related tables? I will attempt to replicate in the local dev environment.

-- Step 2: Override the db_pre_request function
create or REPLACE FUNCTION public.db_pre_request () returns void language plpgsql stable security definer
set
  search_path = public as $$
declare
    groups jsonb;
    default_group_id uuid;
begin
    -- get current groups from auth.users
    select raw_app_meta_data->'groups' from auth.users into groups where id = auth.uid();
    -- store it in the request object
    perform set_config('request.groups'::text, groups::text, false /* applies to transaction if true, session if false */);

    -- custom logic
    SELECT id FROM public.groups into default_group_id WHERE metadata ->> 'groupName' = 'default_group';
    RAISE LOG 'group_id: %', default_group_id;
    perform set_config('request.default_group_id'::text, default_group_id::text, false /* applies to transaction if true, session if false */);
end;
$$;

create or replace function get_default_group_id()
  returns uuid language sql stable
as $function$
  select current_setting('request.default_group_id', true)::uuid
$function$;

CREATE OR REPLACE FUNCTION root_admin_check()
  RETURNS boolean
  AS $$
DECLARE
  user_roles jsonb;
BEGIN
  RAISE LOG 'root_admin_check: current_user %', to_jsonb(current_user);
  -- Extract the roles from the user's 'groups' using the default_group_id function
  user_roles := (auth.jwt() -> 'app_metadata' -> 'groups' -> get_default_group_id()::text -> 'roles');
  RAISE LOG 'root_admin_check: User roles %', to_jsonb(user_roles);

  -- Check if the user_roles is NULL or if 'root_admin' is not in the list of roles
  IF user_roles IS NULL OR NOT (user_roles ? 'root_admin') THEN
    RAISE LOG 'root_admin_check: Not a root admin %', to_jsonb(auth.jwt());
    RETURN FALSE;
  END IF;
  RETURN TRUE;
END;
$$
LANGUAGE plpgsql;

CREATE POLICY authorize_groups_all ON groups AS PERMISSIVE FOR ALL TO authenticated
using (
  root_admin_check()
);

currently I am doing something like this
It works when i do db actions.
But same function fail on Storage.
I use ts to upload images
supabase.storage
      .from("u_profiles")
      .upload(...)

I assume it is trying to write something to  "storage"."objects" and failed. because  db_pre_request is set to run for public only not storage

I added a few
RAISE LOG

in db_pre_request function and it was not called when i upload image

A simple way to reproduce is to install the extension and use Supabase's UI to create bucket / folder inside the bucket.  It will fail

I asked in the Supabase Discord and got the following reply:

db_pre_request is only for PostgREST calls to the database which storage does not use.

For anyone trying to get around it by using a trigger, the trigger uses the same session so the authorization functions (e.g. user_is_group_member) functions will still fail.
So the following will not work:
-- THIS WON'T WORK!

create table
  files (
    id uuid primary key default gen_random_uuid (),
    group_id uuid not null references groups (id)
    ...
  );

alter table files enable row level security;

create policy "Admins can CRUD files" on files as permissive for all to authenticated
using (user_is_group_member(group_id))
with
  check (user_is_group_member(group_id));

create function handle_storage_update() returns trigger language plpgsql as $$
declare
  file_id bigint;
  result int;
begin
  insert into files (group_id, ...)
    values (uuid_or_null(new.path_tokens[1]), ...)
    returning id into file_id;

  return null;
end;
$$;

create trigger on_file_upload
  after insert on storage.objects
  for each row
  execute procedure private.handle_storage_update();

@danielbank thank you.
If i understand right, we can't use any kind of caching methond in storage RLS. We should use select with join and it will be run every file in bucket? ( or all files in all bucket )
Should we create storage trigger feature request on supabase repo?

My take on it is that you can only check authentication (who are you?) in Storage calls via auth.uid().  You cannot immediately check authorization as part of any storage CRUD.  You'll have to add another layer of abstraction to guarantee authorization to files in storage.  In my case, I'm going to have a user action to assign their files to the groups they are a member of.  I'm assuming an edge function could also be utilized here

Thank you since it is not problem of this plugin, closing the issue

auth.jwt() should work in storage.

auth.jwt() should work in storage.

If that's the case then I think this issue should be reopened, @matart15.  There definitely seems to be a discrepancy with the following from the README:

The difference comes in when a query is performed. This library uses the postgrest db_pre_request hook to trigger on all incoming api calls to postgrest, fetch the current state of the claims cache (not from the JWT), and inject it into the request context. The included RLS policy convenience functions check this request context first, before falling back to the JWT method.

The experience we are seeing with storage is that the db_pre_request function is not called and the RLS policy convenience functions are not successfully falling back to the JWT claims

@danielbank I am not an employee of Supabase, but I believe this is the storage code that sets the claims... https://github.com/supabase/storage/blob/6b052d85b9f0c7eb2442a0377b49ca058a5809af/src/internal/database/connection.ts#L231
And unless something broke auth.jwt(), auth.uid() and auth.role() all work for storage requests.  BUT PostgREST is not involved with storage.  It goes directly to the database, so db_pre_request would never fire.  If that function is used to do some sort of security it would not apply to storage operations.

I was able to get around the issue by not using the helper functions and instead accessing the auth.jwt().  Less than ideal because you lose the freshness guarantee, but it's better than nothing:
Note in my example below, I am assuming the top-level folder (path_tokens[1]) indicates the group_id.
create policy "Admins can CRUD storage objects in the `files` bucket" on storage.objects as permissive for all to authenticated using (
  coalesce(
    (
      select
        auth.jwt ()->'app_metadata'->'groups'->(path_tokens[1])
    )?'admin',
    false
  )
)
with
  check (
    bucket_id='files'
    and private.uuid_or_null (path_tokens[1]) is not null
    and coalesce(
      (
        select
          auth.jwt ()->'app_metadata'->'groups'->(path_tokens[1])
      )?'admin',
      false
    )
  );

create policy "Members can read storage objects in the `files` bucket" on storage.objects for
select
  to authenticated using (
    coalesce(
      (
        select
          auth.jwt ()->'app_metadata'->'groups'
      )?(path_tokens[1]),
      false
    )
  );


JoinAcceptPayload DLsettings package added.

JoinAcceptPayload didn't contain DLsettings package so after the successful join the RX2DataRate in the end-node was getting set to a default value of 0 (so, DR_0) rather than, say, DR_8 in Australia, which needs to be pulled from the regional specific band file.

Thanks! I'll merge this into my develop branch as I'm currently working on completing the join-accept configuration (DLSettings, RxDelay and optional CFList) 😄

Good timing :)
Cheers


Better error message for not connecting to Android debug device - WIFI debugging

Steps to reproduce

Connect a physical Android device to debug
Leave it there unattended for some time until it shows on flutter doctor -v that it is offline (maybe only happens on wifi debugging?)
Try to debug a project with flutter run -v
See the output below (logs).

About point two here, not sure this is exactly when you can see this output. Look https://github.com/Dart-Code/Dart-Code/issues/5245 for more info. It happens to me whenever I have left my phone lock the screen by itself and have not used it for some time. If I lock it manually or recently this does not happen.
Expected results
Device is offline message. Or some better message handling when not in verbose mode.
Actual results
Connection closed before full header was received message only in verbose mode.
Code sample
Not important.
Screenshots or Video
Not important.
Logs
Logs
[   +1 ms] Latest build already installed.
[        ] executing: C:\Users\felip_0vh5fa6\AppData\Local\Android\sdk\platform-tools\adb.exe -s <IP_ADDRESS>:42307 shell -x logcat -v time -t 1
[ +355 ms] --------- beginning of system
                    09-02 17:13:23.818 I/wpa_supplicant( 1637): Heartbeat 62
[  +13 ms] executing: C:\Users\felip_0vh5fa6\AppData\Local\Android\sdk\platform-tools\adb.exe -s <IP_ADDRESS>:42307 shell am start -a android.intent.action.MAIN -c android.intent.category.LAUNCHER -f 0x20000000 --ez enable-dart-profiling true
--ez enable-checked-mode true --ez verify-entry-points true br.inf.sunsoft.produtores/br.inf.sunsoft.produtores.MainActivity
[ +157 ms] Starting: Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x20000000 cmp=br.inf.sunsoft.produtores/.MainActivity (has extras) }
[        ] Waiting for VM Service port to be available...
[+1327 ms] VM Service URL on device: http://<IP_ADDRESS>:45599/PSAwNkkAsUk=/
[   +1 ms] executing: C:\Users\felip_0vh5fa6\AppData\Local\Android\sdk\platform-tools\adb.exe -s <IP_ADDRESS>:42307 forward tcp:0 tcp:45599
[  +61 ms] 50529
[   +1 ms] Forwarded host port 50529 to device port 45599 for VM Service
[   +3 ms] Caching compiled dill
[  +78 ms] Connecting to service protocol: http://<IP_ADDRESS>:50529/PSAwNkkAsUk=/
[+31591 ms] Fail to connect to service protocol: http://<IP_ADDRESS>:50529/PSAwNkkAsUk=/: HttpException: Connection closed before full header was received, uri = http://<IP_ADDRESS>:50529/PSAwNkkAsUk=/ws
[   +2 ms] Error connecting to the service protocol: failed to connect to http://<IP_ADDRESS>:50529/PSAwNkkAsUk=/
[  +21 ms] "flutter run" took 66.603ms.
[ +176 ms]
                    #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
                    #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:874:9)
                    <asynchronous suspension>
                    #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1408:27)
                    <asynchronous suspension>
                    #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
                    <asynchronous suspension>
                    #4      CommandRunner.runCommand (package:args/command_runner.dart:212:13)
                    <asynchronous suspension>
                    #5      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:420:9)
                    <asynchronous suspension>
                    #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
                    <asynchronous suspension>
                    #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:364:5)
                    <asynchronous suspension>
                    #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:130:9)
                    <asynchronous suspension>
                    #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
                    <asynchronous suspension>
                    #10     main (package:flutter_tools/executable.dart:93:3)
                    <asynchronous suspension>

[  +46 ms] I/flutter (16413): Drift: Sent SELECT * FROM sqlite_master; with args []
[   +2 ms] I/flutter (16413): Drift: Sent select 1 with args []
[ +211 ms] ensureAnalyticsSent: 253ms
[   +1 ms] Running 2 shutdown hooks
[   +5 ms] Shutdown hooks complete
[  +26 ms] exiting with code 2

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.24.1, on Microsoft Windows [Version 10.0.22631.4037], locale pt-BR)
    • Flutter version 3.24.1 on channel stable at C:\Users\felip_0vh5fa6\.puro\envs\stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5874a72aa4 (2 weeks ago), 2024-08-20 16:46:00 -0500
    • Engine revision c9b9d5780d
    • Dart version 3.5.1
    • DevTools version 2.37.2

[√] 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\felip_0vh5fa6\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Users\felip_0vh5fa6\AppData\Local\Programs\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0--11852314)
    • 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.10.6)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.10.35201.131
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2024.1)
    • Android Studio at C:\Users\felip_0vh5fa6\AppData\Local\Programs\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.11+0--11852314)

[√] VS Code (version 1.92.2)
    • VS Code at C:\Users\felip_0vh5fa6\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.92.0

[√] VS Code (version 1.88.0-insider)
    • VS Code at C:\Users\felip_0vh5fa6\AppData\Local\Programs\Microsoft VS Code Insiders
    • Flutter extension version 3.64.0

[√] Connected device (4 available)
    • SM G780F (mobile) • <IP_ADDRESS>:41835 • android-arm64  • Android 13 (API 33)
    • Windows (desktop) • windows              • windows-x64    • Microsoft Windows [Version 10.0.22631.4037]
    • Chrome (web)      • chrome               • web-javascript • Google Chrome 128.0.6613.114
    • Edge (web)        • edge                 • web-javascript • Microsoft Edge 128.0.2739.42
    ! Device <IP_ADDRESS>:39975 is offline.

[√] Network resources
    • All expected network resources are available.

• No issues found!

Hi @FMorschel, curious, what device are you using? And is that the only device you're seeing this on? I have been trying to get my pixel 7 to read as offline, but after the phone locks for several times, it is still able to connect

I've used a Samsung Galaxy S23U and had this experience but currently downgraded to see the behaviour on older devices and am using a Samsung Galaxy S20 FE. Not sure if this is only a Samsung thing. As I said on the issue for Dart-Code, it seems weird that the phone receives the app and installs it just fine bit is not able to connect to the service protocol.
As I said, it is pretty inconsistent as well. I'm not really sure how to make this happen. The one thing I noticed is that I connect it through wifi debugging and leave the phone locked for some time while doing other things (not deving or deving without debugging) and whenever I come back, I always need to manually unlock my phone to make debugging work.

I'm guessing this might be because adb might be dying on us. Possibly related https://github.com/flutter/flutter/issues/9707
Labeling this as a proposal for an improved message.

The Windows version also crashes. Not sure about what you said but maybe you're right.


cl.exe can't access file in mixed case sub-directory of parent created through wsl

This is new in the latest Insider build:
Microsoft Windows [Version 10.0.17093.1000]
(c) 2017 Microsoft Corporation. All rights reserved.
-- reproduction steps:
In wsl shell:
$ mkdir -p /mnt/c/Users/roger/first/mixedCase
$ touch /mnt/c/Users/roger/first/mixedCase/test.cpp
in Developer Command Prompt for VS2015:

cd \Users\roger\first\mixedCase
cl.exe test.cpp

Microsoft (R) C/C++ Optimizing Compiler Version 19.00.24215.1 for x86
Copyright (C) Microsoft Corporation.  All rights reserved.
test.cpp
c1xx: fatal error C1003: Cannot open source file: 'test.cpp': No such file or directory
--
'dir' shows the file exists and has zero length (which is not significant here.)
--
It seems to be necessary to have a two-level directory - /mnt/c/Users/roger/mixedCase/test.cpp works fine, as does /mnt/c/Users/roger/first/second/test.cpp

@SvenGroot - Could you take a look?

The same issue occurs if the mixed case subdirectory is created in Windows (but it's okay if the parent 'first' is created in Windows).

Can work around by re-mounting /mnt/c with -o case=off

I ran into this too, and I believe I've worked around it semi-permanently with this /etc/wsl.conf:
[automount]
options=case=off
All of the following was with options=case=dir:
I created a new c++ console project in visual studio 2015 names "SimpleTest".  Changed nothing in it, just built it.  It builds fine.  Closed Visual Studio.
Then I create a directory (not under lxss, under my windows user directory) from the WSL prompt, and I moved SimpleTest into that directory using explorer.  I reopened SimpleTest, using explorer, and built.  Intellisense tells me it can't find headers to import, the compiler says it can't find files.  I can open files up and edit them in visual studio, but I can't compile.
If I move this back it fixes it.  If I use fsutil file setcasesensitiveinfo to turn sensitivity off for the parent folder, then it works.
From the outside it feels like the bug is in cl.exe.  Everything else, explorer.exe, notepad.exe, linux utilities, the editing and project organization part of visual studio are behaving as I expect.

I noticed in vs 2013 and vs 2017 that certain features of the IDE also failed.  e.g. I was unable to search through code for strings, leading me to believe it's not just cl.exe that's having issues with this.

Something is really wrong with casesensible folders in build 17134 and 17666 too that could explain this issue and #3163 .
I think I have a repo that always shows the bug. These would be the steps:

Use WSL to create a casesensible folder in C:
mkdir /mnt/c/sensible
Copy any executable in that folder
cp /mnt/c/Windows/System32/calc.exe /mnt/c/sensible
Open the sensible folder in Windows Explorer and create a new folder with CTRL+SHIFT+N and press ENTER
Create another new folder and rename it to anything
Copy the calc.exe file inside every new created folder

You will see Windows Explorer can't show the Icon of the executable inside the not renamed folder. Process Monitor says PATH NOT FOUND when trying to access the executable.

I also recommend using options=case=off until this is fixed.

General pro-tip since this issue is chirping crickets since February. Make your working directory C:\....\Devel\source\repos or something along those lines if you can. The -o case=off workaround appears to work for some folks, but at least for the time being you'll save heartache by just avoiding mixed case paths in the first place.

Just chiming in with a few more tips as I struggled to overcome this issue in a mixed wsl-vcvars build environment:
For me adding the case=off option to etc/wsl.conf as recommended above eventually did the trick, but only after the following extra steps:

It needed a full reboot, simply closing all my ubuntu shells wasn't enough for me. One simple way to make sure the option has been taken into account is to type mount in a bash shell, you should see the option listed at the end:  C: on /mnt/c type drvfs (rw,noatime,uid=1000,gid=1000,case=off). I didn't see it until I did a full machine restart.
I needed to re-checkout the parent folder from WSL once mounted with the case=off option. Just deleting the mixed case folder itself and checking it out again with git didn't solve the problem. It's as if the case information was stored in the parent folder, but it is consistent with what's been reported in one of the other comments above.

What's really confusing to me is why cl.exe (or its c1xx frontend) would care about file casing in the first place, and how the parent folder comes into play. Given that the bug manifests itself completely outside of WSL, I'm guessing it's more of a cl.exe bug than a WSL bug. Maybe the compiler is using a obsolete/undocumented Win32 API call to access the input file that's not correctly handling the new metadata added for the per-directory file-casing support added by WSL?

@kromain Here Sven explains how they treat this and similar case sensitive issues. https://blogs.msdn.microsoft.com/commandline/2018/06/14/improved-per-directory-case-sensitivity-support-in-wsl/
One explanation of why this issue happens is applications like explorer.exe and cl.exe still transform filenames to use all upper or lower case.

@onomatopellan thanks for pointing me to the Sven's blog, somehow I missed it (though I've read the previous ones). Glad to hear there's a fix coming in soon!
Keep it up guys, I'm loving WSL, haven't been that excited about an OS in a long time :)

The default mount option has been changed to case=off as of Insider Build 17704

The default mount option has been changed to case=off

Sad face. Is this a temporary work-around for the Fall release, or is this "the plan"? The blog states:

While in the long term the preferred solution is to fix these applications to correctly handle case sensitive directories

That does not work because you can't update all of them. Say I were highly motivated and hacked up libuv, electron, and VS Code tonight to support the per-directory case sensitivity flag. Great. But I can't stop someone from doing a mkdir from the CMD prompt or creating the folder in Explorer. I can't stop them from opening the project in Sublime or Webstorm. The inheritance helps, but at base doesn't work because I don't know if they are using the latest-greatest per-directory case sensitivity version Windows Git to check out the project (and remembered to check Windows Git's new "use case sensitivity" flag). Neither do they probably. If the flag was not set, okay I leave it alone in libuv. But then they breathe on the directory in WSL with case=dir and the behaviour changes without warning. Not tenable.

Yes, unfortunately, using "case=off" is the plan for the foreseeable future. The fact that DrvFs always created case sensitive directories, everywhere on your system, regardless of the fact that many applications aren't prepared to deal with that. The scenario you describe is exactly one of the reasons why inheritance alone isn't enough with case=dir, and we believe using case=off with more fine-grained control is a better option.
We hope that by switching the default to case=off, providing an easy way to change it in WSL itself (the extended attribute), and turning on inheritance, we've made it easier for you to control where case sensitive directories are used, leading to less random "app X can suddenly no longer access these files" type of problems. We are open to any and all feedback about how we could handle this better.
Note that changing the default to "case=off" only affects newly created directories. Previously created directories that were marked case sensitive are still case sensitive, even after this change.
In conjunction with this change, we've also made it so that the information printed in /proc/self/mounts now always includes the case option, even if you didn't explicitly specify it when mounting. This way, you don't have to guess what the default is based on what build you're running, but can now always see what mode you're running in.

We are open to any and all feedback about how we could handle this better.

Equivalent of mount --bind with a case-sensitivity flag on the Windows side. Which has been Cygwin's solution (/cygdrive/c) for about a quarter century.
Someone accessing their 20 year old Visual Studio codebase that has their header file #include directives all case sloppy never operate on that mount. They get the tried-and-loved case insensitivity behaviour. Users of MSVS that open a project with Foo.h and foo.h in the same directory on a case-insensitive mount (classic C:) eat cake, which is no different than it has always been. And they don't care, because they don't name files like that. They just want their "Windows.h" and "windows.h" header includes to work.
Users of VS Code built with a new-fangled libuv/nodejs who operate on the case sensitive mount get all the case sensitive and metadata goodness.

Note that changing the default to "case=off" only affects newly created directories. Previously created directories that were marked case sensitive are still case sensitive, even after this change.

Exactly. And they don't know, because Linux ls or Gnome Nautilus and CMD dir and Windows Explorer don't tell them. They have to remember if they touched some directory three months ago with case=dir.
To sum up: Case sensitivity behaviour isn't sticky. The directory case-sensitivity bit isn't going to be added to the tar file format or git database (though for $7.5b you could, I guess). It is a mount property.

Outside the context of this bug, case sensitive / case insensitive are not the only options.
Filenames are by their nature human-readable labels for files; the underlying implementation should treat a name as a raw byte array; only the UI needs to convert to and from glyphs (or audio...) and there it should try and avoid ambiguity. It it's open to debate how you police non-UI filename generation and potential spoofing. (See e.g. RFC 3490).
Files	should have multiple identities; their 'raw bytes' canonical name (always) and (e.g.) their 'english label' and their 'chinese label'. If, in (e.g.) 'english label view' two label look the same (i.e. the names differ only by case), a visual alert would be appropriate.
The problem here is that wsl assumes raw bytes <=> english label, while Windows doesn't allow two labels that look the same.

Mercy "discussion ran course". Arguably by-design per:

...using "case=off" is the plan for the foreseeable future.

Struggled a lot to pinpoint the issue, from now on, I'm just going to avoid using git with WSL, and use git-sdk instead. (I'll keep using WSL for everything else though).


Suddenly Dash. struggles through domain, works fine through local IP

Environment
Docker
Version
0.12.2
Describe the problem
I have an Open Media Vault 6+Docker+Portainer running on a Orange Pi 3 LTS with Armbian lite.
I also have a Pihole installed bare metal outside Docker running on the same machine, IP is <IP_ADDRESS>.
I noticed Homarr takes some time to load. This didn't happen until a few days ago.
It is reverse proxied with NPM to a Cloudflare domain, usually no problem and all my other services work fine.
What seems to slow down Homarr is Dash., but only if I open Homarr from my .com domain.
If I open it from local IP, it loads fine and quick.
The difference is that the .com version shows an empty slot in place of Dash. If I hover with mouse, the words CPU and RAM appear, but the tiles are invisible.
While this happens, Homarr log says:
ℹ Requested frontend content of configuration '2023-04-01'
Error: 'sharp' is required to be installed in standalone mode for the image optimization to function correctly. Read more at: https://nextjs.org/docs/messages/sharp-missing-in-production
 ERROR  Unexpected error: connect ECONNREFUSED <IP_ADDRESS>:443
 ERROR  Unexpected error: connect ECONNREFUSED <IP_ADDRESS>:493
 WARN  Unexpected response: Request failed with status code 403
 WARN  Unexpected response: Request failed with status code 401
 WARN  Unexpected response: Request failed with status code 404

Those 2 IPs are
<IP_ADDRESS> = second Pihole bare metal on Orange Pi Zero 2, but port is uses is 8883, not 443
<IP_ADDRESS> = an x86 Open Media Vault machine where most of my services run, no idea what port 493 is, I did not set that for any service.
Dash. runs on the same machine with Homarr (<IP_ADDRESS>) and log says this:
    type: 'Wired',
    publicIp: ''
  },
  gpu: { layout: [] },
  config: {
    port: 3001,
    running_in_docker: true,
    accept_ookla_eula: false,
    use_network_interface: undefined,
    speed_test_from_path: undefined,
    fs_device_filter: [],
    fs_type_filter: [
      'cifs',
      '9p',
      'fuse.rclone',
      'fuse.mergerfs',
      'nfs4',
      'iso9660',
      'fuse.shfs'
    ],
    fs_virtual_mounts: [],
    disable_integrations: false,
    show_dash_version: undefined,
    show_host: false,
    custom_host: undefined,
    page_title: 'dash.',
    use_imperial: false,
    always_show_percentages: false,
    widget_list: [ 'os', 'cpu', 'storage', 'ram', 'network' ],
    os_label_list: [ 'os', 'arch', 'up_since' ],
    cpu_label_list: [ 'brand', 'model', 'cores', 'threads', 'frequency' ],
    storage_label_list: [ 'brand', 'type', 'size', 'raid' ],
    ram_label_list: [ 'brand', 'type', 'size', 'frequency' ],
    network_label_list: [ 'type', 'speed_up', 'speed_down', 'interface_speed' ],
    gpu_label_list: [ 'brand', 'model', 'memory' ],
    os_widget_grow: 2.5,
    os_widget_min_width: 300,
    enable_cpu_temps: false,
    cpu_widget_grow: 4,
    cpu_widget_min_width: 500,
    cpu_shown_datapoints: 20,
    cpu_poll_interval: 1000,
    storage_widget_items_per_page: 3,
    storage_widget_grow: 3.5,
    storage_widget_min_width: 500,
    storage_poll_interval: 60000,
    ram_widget_grow: 4,
    ram_widget_min_width: 500,
    ram_shown_datapoints: 20,
    ram_poll_interval: 1000,
    speed_test_interval: 240,
    network_widget_grow: 6,
    network_widget_min_width: 500,
    network_shown_datapoints: 20,
    network_poll_interval: 1000,
    gpu_widget_grow: 6,
    gpu_widget_min_width: 700,
    gpu_shown_datapoints: 20,
    gpu_poll_interval: 1000,
    override: {
      os: undefined,
      arch: undefined,
      cpu_brand: undefined,
      cpu_model: undefined,
      cpu_cores: undefined,
      cpu_threads: undefined,
      cpu_frequency: undefined,
      ram_brand: undefined,
      ram_size: undefined,
      ram_type: undefined,
      ram_frequency: undefined,
      network_type: undefined,
      network_speed_up: undefined,
      network_speed_down: undefined,
      network_interface_speed: undefined,
      network_public_ip: undefined,
      storage_brands: {},
      storage_sizes: {},
      storage_types: {},
      gpu_brands: [],
      gpu_models: [],
      gpu_memories: []
    }
  }
}
Running speed-test (this may take a few minutes)...
First measurement [CPU]: [
  { load: 77.01748486886348, temp: 0, core: 0 },
  { load: 68.71937275400197, temp: 0, core: 1 },
  { load: 74.30436276193748, temp: 0, core: 2 },
  { load: 70.9532523706538, temp: 0, core: 3 }
]
First measurement [Network]: { up: 0, down: 0 }
First measurement [Storage]: [<PHONE_NUMBER>4,<PHONE_NUMBER>, 0, 0,<PHONE_NUMBER> ]
First measurement [RAM]:<PHONE_NUMBER>
Speed-test completed successfully [speedtest-cli] {
  speedDown: 178520952.22912505,
  speedUp: 99204239.35138471,
  publicIp: undefined
}

Logs
same as above
Homarr:
ℹ Requested frontend content of configuration '2023-04-01'
Error: 'sharp' is required to be installed in standalone mode for the image optimization to function correctly. Read more at: https://nextjs.org/docs/messages/sharp-missing-in-production
 ERROR  Unexpected error: connect ECONNREFUSED <IP_ADDRESS>:443
 ERROR  Unexpected error: connect ECONNREFUSED <IP_ADDRESS>:493
 WARN  Unexpected response: Request failed with status code 403
 WARN  Unexpected response: Request failed with status code 401
 WARN  Unexpected response: Request failed with status code 404

Dash.
    type: 'Wired',
    publicIp: ''
  },
  gpu: { layout: [] },
  config: {
    port: 3001,
    running_in_docker: true,
    accept_ookla_eula: false,
    use_network_interface: undefined,
    speed_test_from_path: undefined,
    fs_device_filter: [],
    fs_type_filter: [
      'cifs',
      '9p',
      'fuse.rclone',
      'fuse.mergerfs',
      'nfs4',
      'iso9660',
      'fuse.shfs'
    ],
    fs_virtual_mounts: [],
    disable_integrations: false,
    show_dash_version: undefined,
    show_host: false,
    custom_host: undefined,
    page_title: 'dash.',
    use_imperial: false,
    always_show_percentages: false,
    widget_list: [ 'os', 'cpu', 'storage', 'ram', 'network' ],
    os_label_list: [ 'os', 'arch', 'up_since' ],
    cpu_label_list: [ 'brand', 'model', 'cores', 'threads', 'frequency' ],
    storage_label_list: [ 'brand', 'type', 'size', 'raid' ],
    ram_label_list: [ 'brand', 'type', 'size', 'frequency' ],
    network_label_list: [ 'type', 'speed_up', 'speed_down', 'interface_speed' ],
    gpu_label_list: [ 'brand', 'model', 'memory' ],
    os_widget_grow: 2.5,
    os_widget_min_width: 300,
    enable_cpu_temps: false,
    cpu_widget_grow: 4,
    cpu_widget_min_width: 500,
    cpu_shown_datapoints: 20,
    cpu_poll_interval: 1000,
    storage_widget_items_per_page: 3,
    storage_widget_grow: 3.5,
    storage_widget_min_width: 500,
    storage_poll_interval: 60000,
    ram_widget_grow: 4,
    ram_widget_min_width: 500,
    ram_shown_datapoints: 20,
    ram_poll_interval: 1000,
    speed_test_interval: 240,
    network_widget_grow: 6,
    network_widget_min_width: 500,
    network_shown_datapoints: 20,
    network_poll_interval: 1000,
    gpu_widget_grow: 6,
    gpu_widget_min_width: 700,
    gpu_shown_datapoints: 20,
    gpu_poll_interval: 1000,
    override: {
      os: undefined,
      arch: undefined,
      cpu_brand: undefined,
      cpu_model: undefined,
      cpu_cores: undefined,
      cpu_threads: undefined,
      cpu_frequency: undefined,
      ram_brand: undefined,
      ram_size: undefined,
      ram_type: undefined,
      ram_frequency: undefined,
      network_type: undefined,
      network_speed_up: undefined,
      network_speed_down: undefined,
      network_interface_speed: undefined,
      network_public_ip: undefined,
      storage_brands: {},
      storage_sizes: {},
      storage_types: {},
      gpu_brands: [],
      gpu_models: [],
      gpu_memories: []
    }
  }
}
Running speed-test (this may take a few minutes)...
First measurement [CPU]: [
  { load: 77.01748486886348, temp: 0, core: 0 },
  { load: 68.71937275400197, temp: 0, core: 1 },
  { load: 74.30436276193748, temp: 0, core: 2 },
  { load: 70.9532523706538, temp: 0, core: 3 }
]
First measurement [Network]: { up: 0, down: 0 }
First measurement [Storage]: [<PHONE_NUMBER>4,<PHONE_NUMBER>, 0, 0,<PHONE_NUMBER> ]
First measurement [RAM]:<PHONE_NUMBER>
Speed-test completed successfully [speedtest-cli] {
  speedDown: 178520952.22912505,
  speedUp: 99204239.35138471,
  publicIp: undefined
}

Context
From Cloudflare domain:

From local IP

Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

We will most likely rework or remove the dashdot integration in future updates, please read the following : https://github.com/ajnart/homarr/issues/1503


BAM file checking

Hi,
Great tool! Thank you!
Couple of minor quirks. The code at present tests for the presence of a bam index file but does so by assuming the bam index is in the bai format and not the more recent csi - however, the csi file type does work with methylartis and so should be probably be allowed in the code as standard?
In addition, whilst the code checks for the presence of the index file, it doesn't actually check that the bam file itself is present in the user specified location. Thus if the user passes an incorrect path you get the warning "bam index not found" when in fact the path is incorrect.
Apologies for not submitting a pull request to fix this.
Thanks

Thanks for this and apologies for the delay - should be fixed in 8e1cd69ef7b762fb2a368c2584bf625bb99a8126 and 9757a12345f1250518a0b1578c9fd79cf6cc819c
--Adam


i8ln for the admin module

#345

So my issue with this implementation which is my fault, since i didn't add any documentation during the initial implementation.
initially i designed the localization's file to have two main categories.
-labels
-messages
-titles
since then people have added
-buttons
-alerts
which is fine , they would have been merged into labels and messages.
any translation in these categories, if used in multiple places around the application would be in the first level of the hash. if they are only used in a specific place, they would be nested under there area for example a button in appointments would go under
appointments -> buttons -> myButton.
in this pr you are nesting them under admin, you have added route, i can understand why.
you have also added loaddb, and lookup. which i am not so sure about. i feel most of these could go under messages -> errors or something like that.
i am just opening up conversation, so that we can standardize the file.

@billybonks we are having an ongoing conversation on how to best organize the localizations.  There are a couple of things we are thinking about:

Reuse can potentially be an issue because depending upon language/context you may not want to reuse the value.  I think there are some use cases where reuse is ok, but it probably should be used in moderation.
Dividing the localizations into separate files per module and then importing them into the main translations file.

If you have additional thoughts, please feel free to bring them up in slack.  I appreciate your feedback and input!

@dswanson I believe we should remove the .route and .controller namespaces


VRHelper usage issues

Lewis pointed out a couple issues:

[x] if you disconnect & reconnect the controllers while in VR, the teleportation no longer works.
[x] The helper will always creates a new DeviceOrientationCamera, which is used when not in VR. It would be great if you could provide an option such that the developer could choose to use provide their own camera rather than the Experience Helper creating one. By default the DeviceOrientationCamera is created. The option can be used to avoid this behavior, Alternatively, move the onExitingVR observable to after you set the active camera, so that the developer re-apply a custom camera on VR exit?
[ ] Add support for TrackPad and second pad from WMR controllers
[ ] Check if rotation and teleportation are correct. @davrous mentioned it could be wrong still
[x] Fix Vignette flickering
[x] http://playground.babylonjs.com/#JA1ND3#24 has an issue to align teleportation circle on stairs

Could you please also double check:

[ ] multi mesh configurations for teleport
[ ] starting the helper while in VR

THX :-)

Created new issue with remaining problems
https://github.com/BabylonJS/Babylon.js/issues/3298


Add field “canUpdateStatus” in ReplyConnection

Adds a boolean field for replyconnection so that UI can know whether we should display delete / restore button

Coverage increased (+0.2%) to 86.207% when pulling a393b1708a27746bae38287b0ed6bfbe8d8dad7d on features/can-edit-field into 956515d8e783b10bb7ac18d7a70dc54b41fe1466 on master.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_3bht3gs5cu/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_3bht3gs5cu/clone/recipe/meta.yaml:19: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_3bht3gs5cu/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_3bht3gs5cu/clone/recipe/meta.yaml:39: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_3bht3gs5cu/clone/recipe/meta.yaml:39: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_3bht3gs5cu/clone/recipe/meta.yaml:39: missing_description: The recipe is missing a description
ERROR: /tmp/abs_3bht3gs5cu/clone/recipe/meta.yaml:39: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_63ibo63x4b/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_63ibo63x4b/clone/recipe/meta.yaml:19: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_63ibo63x4b/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_63ibo63x4b/clone/recipe/meta.yaml:39: missing_description: The recipe is missing a description
ERROR: /tmp/abs_63ibo63x4b/clone/recipe/meta.yaml:39: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_63ibo63x4b/clone/recipe/meta.yaml:39: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_63ibo63x4b/clone/recipe/meta.yaml:39: missing_dev_url: The recipe is missing a dev_url
Errors were found


"Save Button Missing in Safari and Chrome Extensions: Need Assistance and Conducting Tests"

The user is unable to see a save button in the Safari and Chrome extensions, even after successfully loading and using the extensions. They have tried deleting and reloading the extensions but it did not solve the issue. They are willing to subscribe and conduct further tests to resolve the problem.

Summarized feedback from<EMAIL_ADDRESS>
The user is unable to see a save button in the Safari and Chrome extensions, even after successfully loading and using the extensions. They have tried deleting and reloading the extensions but it did not solve the issue. They are willing to subscribe and conduct further tests to resolve the problem.


Prepare release 1.7.0

What does this PR do?
This PR prepares release 1.7.0.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #144 (e4806e3) into release/1.7.0 (e630ea8) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/1.7.0     #144   +/-   ##
================================================
  Coverage            86.57%   86.57%
  Complexity             178      178
================================================
  Files                   21       21
  Lines                  581      581
  Branches                75       75
================================================
  Hits                   503      503
  Misses                  49       49
  Partials                29       29

Impacted Files
Coverage Δ

...m/datadog/gradle/plugin/internal/OkHttpUploader.kt
84.43% <ø> (ø)


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
5 seconds
sanity-label-check

✅ Succeeded
6 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
5 seconds
sanity-label-check

✅ Succeeded
6 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary


[tfswitch-parity] Support a default version unless a more specific version is requested

Describe the solution you'd like
Please replicate this feature from tfswitch. https://tfswitch.warrensbox.com/Quick-Start/#set-a-default-tf-version-for-cicd-pipeline
Additional context
It's backport of the original issue from tofuenv: https://github.com/tofuutils/tofuenv/issues/34
cc @skyzyx

The tenv use already allows that, or do i miss something ?


RSI Value is different using this lib in comparison with manual calculation of RSI using the formula

i used this class to get list of close price. From the API i get the LIST of one year close price as well as open, high, low, volume, and date.
apidataclass.txt
I used this second class to calculate all the indicators.. the other indicator value seems all correct but the value of RSI is incorrect.
calculationclass.txt
This is the Json output from the API that is in the apidataclass.txt file
API-Json-Output.zip

See #90.


Float2 and Float3 replaced with Vector2 and Vector3

Description of the changes:
Custom vector structures (Float2 and Float3) replaced with a .net implementation from System.Numerics namespace.

Before submitting a Pull Request:

[x] I reviewed CONTRIBUTING.md
[x] I built my changes locally
[x] I ran the unit tests
[ ] I ran the functional tests with a device
[ ] I ran the performance tests with a device

I tested changes on:

[x] Windows
[ ] Linux

Tested with existing unit tests and via manually running provided sample applications.

/azp run


Does eosio.cdt support safe math library

Sometimes we have to do some big int arithmetic in a contract . Does eosio.cdt support a safe math operating ?

Has there been any work done on this?


EHN: output_resolution required + tests, FIX Path of unset FRESURFER_HOME

Changes proposed in this pull request

--output_resolution marked as required (issue #37)
in case of never set or unset FREESURFER_HOME, specify empty string default to suppress Path() error
bids_validation and freesufer arg parsing pulled out into their own functions
add opt parsing tests (sorry, a bit little kludgy and overkill for such a small part of code)

Documentation that should be reviewed
Maybe a note about having no FREESURFER_HOME variable. Currently looks for license file in ./ if unset.

Ahh I see always requiring output-resolution breaks other tests! Should the flag only be required if not --recon-only?

Correct. By the time the reconstruction workflows start the preprocessed data should already be in whatever output resolution the user requested during preprocessing. So --output-resolution is only mandatory if they haven't specified --recon-only.

This looks good, only a few nitpicks from the flake8 check.

I went ahead and made these changes but found out I'm not allowed to push to your PR branch. Could you please enable this? https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/allowing-changes-to-a-pull-request-branch-created-from-a-fork

sorry again I've been so slow to respond.  It looks like "allow edits from maintainers" is already checked, at least here (https://github.com/PennBBL/qsiprep/pull/50). I added you as a collaborator on the fork too in case that frees things up

Strange, I'm still getting permission errors:
(base) mcieslak$ git push origin test-args
ERROR: Permission to LabNeuroCogDevel/qsiprep.git denied to mattcieslak.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I see "Awaiting mattcieslak’s response" in the collab settings page of the lncd forked repo.
But it's so strange authentication doesn't work here.
I toggled the check box out of superstition

That should do it, right?
.

Got it!


Opening links with targets

I'm not an expert on org mode, but from some searches I've made it looks like you can create a link to a specific part of a page with a syntax similar to this one:
[[file:./projects.org::10][Get to line 10 of file "projects.org"]]
[[file:./todo.org::Keyword][Move to file "todo.org" and do a search for "Keyword"]]

I've tried both this syntax using this plugin, but it doesn't seem to work. The only thing I could do was pointing to a specific file, which works correctly.
Is this supported?

Those cases are not supported, but you can create link to headline by title, or by CUSTOM_ID.
# projects.org

* My first headline
  :PROPERTIES:
  :CUSTOM_ID: my_id
  :END:
* My second headline

By headline title
[[file:./projects.org::*My second headline]]

By custom id:
[[file:./projects.org::#my_id]]

Hi kristijan,
Thank you for the explanation! It would be perfect for my use-case, but even that doesn't seem to work on my pc...
This is what I wrote in my todo.org file:
[[file:~/Dropbox/org/progetti.org::#amazon_fb][Project link]]

[[file:~/Dropbox/org/progetti.org::**Migrations][Work website migrations]]

And this is what I have in my "progetti.org" file:
** Amazon-fallback
   :PROPERTIES:
   :CUSTOM_ID: amazon_fb
   :END:

** Migrations
   :PROPERTIES:
   :CUSTOM_ID: migrations
   :END:

Still, when using the shortcut it doesn't work. If I remove the "targets" it works fine.

Did you give it a try with minimal configuration?
Please provide information about your system and configuration. You can follow bug template for this.

Just tried with a minimal config, keeping just treesitter and org. I've only set the leader key and the "conceallevel" and "concealcursor" options as intended by this plugin.
Still not working.
However, I checked the same situation on my ubuntu laptop (running debian) and there it's working, so it's probably a windows-related issue.
OS: Windows 10
Terminal emulator: Windows Terminal (Windows App)
nvim --version:
NVIM v0.7.0
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compiled by runneradmin@fv-az320-113

Features: -acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM\sysinit.vim"
         fallback $VIM : "C:/Program Files/nvim/share/nvim"

I've troubleshooted the problem and found out the "solution".
The problem is the way paths are handled by windows vs linux. Windows uses "\" while linux uses "/".
If I set the file path the way I did it, it will work in linux but not in windows, while if I exchange all slashes with backslashes, the file will work on windows, but not in linux.
The problem becomes clear here: https://github.com/nvim-orgmode/orgmode/blob/b9de38e8ddb61832d24fcb8db460d7f811a3c936/lua/orgmode/org/hyperlinks.lua#L25
After this line, the returned file path has mixed backslashes and forward slashes on windows.
One way that a user can solve it, is to set the following option in init.lua on the windows machine:
vim.opt.shellslash = true

This way, expanding using the fnamemodify function will always return forward slashes.
Fixing this in the plugin seems trivial, but I'm not exactly sure what would be the best way to do it.
Hope that I was of some help :)

Thanks for figuring it out!

Fixing this in the plugin seems trivial, but I'm not exactly sure what would be the best way to do it.

I agree, but maybe not everyone wants this behavior.
I would just update readme troubleshoot section with this information so people know how to fix it.

Yes, indeed it seems like a good idea.
I was thinking about a function that would return properly formatted paths, but I'm not sure that's even possible, to be honest. Adding it to the troubleshooting section will be very helpful for sure!

Troubleshoot section added: https://github.com/nvim-orgmode/orgmode#jumping-to-file-path-is-not-working-for-paths-with-forward-slash
Thanks for our help!

I'm sorry to open this again. I just wanted to give a warning: after setting the "shellslash" option, Packer.nvim stopped working correctly. I currently opened an Issue on their github repo to address this, but up until then, if someone is using packer, it's probably better not to set that variable.


feat: Replace the existing Resources search with Azure AI Search

:tada: This PR is included in version 2.0.0-next.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Impossible to send Authorization headers

Home Assistant release with the issue:
0.82.1
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io
Component/platform:
http
Description of problem:
I tried to use api in react application using axios :
import axios from "axios";

axios.get("http://my-hassio-instance/api/states", {
  headers: {
    "Authorization": "Bearer xxxx"
  }
});

But I have this error on preflight request (OPTIONS) :
CORS preflight request failed: headers are not allowed: AUTHORIZATION

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
api:

frontend:

http:
  # Secrets are defined in the file secrets.yaml
  api_password: !secret http_password
  cors_allowed_origins:
    - http://localhost:3001

The react app is running on 3001 port on localhost
Traceback (if applicable):

Additional information:
I think Authorization headers must be allowed in cors file
ALLOWED_CORS_HEADERS = [
    ORIGIN, ACCEPT, HTTP_HEADER_X_REQUESTED_WITH, CONTENT_TYPE,
    HTTP_HEADER_HA_AUTH]

Same issue on 0.84.1

The correct way is to not require authorization on pre-flight because the browsers doesn't support adding auth headers on OPTIONS pre-flight.
See this: https://stackoverflow.com/questions/15734031/why-does-the-preflight-options-request-of-an-authenticated-cors-request-work-in
Would it be ok if I create a PR for this?


Creo clase Dice

✅ geniio!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
174

Relevant Lines:
174

💛  - Coveralls


Error: Can't resolve 'gsap/DrawSVGPlugin'

Summary
Getting this error when registering drawSVG into nuxt

Steps done:

nuxt.config.js:

gsap: {
    extraPlugins: {},
    clubPlugins: {
      drawSVG: true,
    },
    registerEffect: []
  },

using in template with directive:

<circle v-gsap.from="{
      duration: 1,
      drawSVG: '0 100%'
  }" cx="121.89451" cy="325.76588" r="45" fill="#6c63ff"/>

I have checked how it is actually registered in the module/templates/plugin:
<% if (options.clubPlugins.drawSVG) { %>
    const { DrawSVGPlugin } = require('gsap/DrawSVGPlugin')
    inject('DrawSVGPlugin', DrawSVGPlugin)
    gsap.registerPlugin(DrawSVGPlugin)
<% } %>

I can't take vague assumptions at this point, but I am thinking that it might have some buggy behaviors 🤔

DrawSVG is a premium plugin, so you must have a Club GreenSock membership to be able to use the premium plugins.
Club GreenSock
nuxt-gsap-module supports Club GreenSock premium plugins. They can be easily activated via module settings, just like the free ones.
Installation

Follow the official instructions and install the premium plugins as usual.
After installation, simply activate the desired plugins and that's it, you're ready to go!

Appreciate the response :), Aha! I see now, so for future, if anyone stucks at this point, we should subscribe first in a premium Club GreenSock

This only applies to specific premium plugins. No need to emphasize this, I think everything is explained in the module documentation and on the official GSAP website.
You can use the module without a subscription. See Licenses.

I have the same problem after using the DrawSvg plugin to animate a small image of my portfolio I find myself blocked after deploying it. I think it's a shame ☹😞


Docs for DBSCAN converted

Description
Converted documentation for DBSCAN (see output)

@vepifanov by the way, what'd you say for a Details page to have definitions in a form of glossary? it's possible then to link to the definitions there from anywhere

Details page looks nice!

I've updated the output  with all the changes.

make test

@napetrov I don't think errors in CI are caused by my commits, could you check?

I also don't think that they are related.  Let's restart to double check . I will merge this later today. ok?

restart


fix(vite.config.ts): fix font loading bug

The bug still seems to be present in my local host, but pushing for extras eyes on my code.
fix #243

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description
This PR fixes the font loading bug.
Related Tickets & Documents
Fixes #243
Mobile & Desktop Screenshots/Recordings

I don't see a change in my local host 😅

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[x] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
No I don't believe so.
[optional] What gif best describes this PR or how it makes you feel?
I know I'm completely supported and standing in like 5 inches of water, and yet... Life as a junior dev I suppose lol

Per mine and @0-vortex's comment. It is indeed not loading, the syntax is off and skipping the fonts. You commit vortex's solution and see it work. Once done we can merge this.

🥳🥳🥳

Yeah this is it: 🍕 🚀

🥳🥳🥳

Was just posting a screenshot myself, congrats! Merging this in now 🍕 🚀


Apache Cordova

How can I port this component to use it in Apache Cordova Project?
Bests

Your guess is as good as mine. I've never used Cordova.


Update Learn section highlights

According to @PeterTurcan assessment in https://github.com/boostorg/website-v2-docs/issues/244:
User Guide Learn page:

Remove these two links completely

Library Names & Organization
Header & Compiled Binaries

Add the following text and link:
Text:   Introduction to the User Community
Link:   https://www.boost.io/doc/user-guide/user-community-introduction.html

Contributor Guide Learn page:

Design Guide link is broken - change it to:
https://www.boost.io/doc/contributor-guide/design-guide/design-best-practices.html

After "Getting Involved" - add a link to the Contributors FAQ:
Text:   Contributors FAQ
Link:   https://www.boost.io/doc/contributor-guide/contributors-faq.html

Add a link to the Super-Project
Text:   The Super-Project
Link:  https://www.boost.io/doc/contributor-guide/superproject/overview.html

Change the text from "Libraries" to "Library Requirements" - no other changes to this one

Remove the text and links for these (as they are accessed from Library Requirements)
Licenses
Portability
Code Organization

Add the text and link:
Text:   Introduction to the Contributors Community
Link:   https://www.boost.io/doc/contributor-guide/contributor-community-introduction.html

These changes should result in both sections containing 8 links into the docs, and be fairly robust going forward.

@joaquintides Can this be closed?


model_dump_json discards significant figures in timestamps that end in 0

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
(Pdb) pp obj.modified
datetime.datetime(2023, 8, 11, 1, 12, 16, 55400, tzinfo=TzInfo(UTC))
(Pdb) pp str(obj.modified)
'2023-08-11 01:12:16.055400+00:00'
(Pdb) pp obj.model_dump_json(include="modified")
'{"modified":"2023-08-11T01:12:16.0554Z"}'

055400 indicates 6-digit precision while 0554 only indicates 4-digit precision.
Truncating trailing zeroes implies that either

those 2 digits are not significant, or
the timestamp has infinite precision, such that an arbitrary number of trailing zeroes may be appended

Neither of which are true.
Moreover, the dumped JSON can't be reloaded using datetime.fromisoformat:
(Pdb) pp datetime.fromisoformat(str(obj.modified))
datetime.datetime(2023, 8, 11, 1, 12, 16, 55400, tzinfo=datetime.timezone.utc)
(Pdb) pp datetime.fromisoformat(json.loads(obj.model_dump_json(include="modified"))["modified"])
*** ValueError: Invalid isoformat string: '2023-08-11T01:12:16.0554Z'

Example Code
No response
Python, Pydantic & OS Version
             pydantic version: 2.1.1
        pydantic-core version: 2.4.0
          pydantic-core build: profile=release pgo=true mimalloc=true
                 install path: /ifs/home/tucked/projects/some-project/.tox/py38/lib/python3.8/site-packages/pydantic
               python version: 3.8.14 (default, May  4 2023, 21:08:14)  [GCC 7.5.0]
                     platform: Linux-5.14.21-150400.24.74-default-x86_64-with-glibc2.29
     optional deps. installed: ['typing-extensions']

Selected Assignee: @davidhewitt

Thanks - duplicate of #6761, and we're working to fix this 👍


Refactor Routing into separate project

The routing package in Go has been refactored since the .NET implementation was done and is now standalone. The same should be done here.
Also a bit related, maybe the Proto namespace should be moved to Proto.Actor, so we align the Go packages/.NET assemblies.

👍 on both parts.

Will do that as well before moving on to persistence, so we have the fundamentals in order.

Closed by #17


Clarification on the amount of requests

Greetings,
out of curiosity I tried out gptcommit yesterday for the first time with three commits. Two of them were a bit larger with 265 additions and 214 deletions according to git. Much to my surprise there were two dozen requests and got billed for $0.55. I'd like to ask whether it's expected that the amount of requests (and thus the price) is that high or whether this might indicate a bug?

Hey @ChristianIvicevic , you bring up a great point here.
In the current design, gptcommit issues N+2 prompts, where N is the number of modified files with diffs under the max_token_limit. The other prompts are the title and summary.
OpenAI Completions are billed by "tokens" that are both sent and generated. Pricing per token depends on the model used. The number of tokens generated are generally predictable (as a commit message is usually only so big) but gptcommit could be sending over a lot of tokens in the form of diff data.
Today, I see two low-hanging solutions for reducing cost:

Switch to a different model using the openai.model configuration option
Reduce the side of prompts and diff data sent to OpenAI

OpenAI's pricing page can be found at
https://openai.com/api/pricing/#faq-completions-pricing

Thanks for the summary. I might have a proposal that may be an option to take into consideration. As of right now I can technically create a patch for a specific commit and without its subject line ask ChatGPT about a descriptive commit message. Unlike gptcommit it would not summarize the changes of each touched file, but it's basically a single sentence or paragraph based on a patch file.
Given that GPT can't explain the reasoning behind certain changes summarizing the details of each file feels less valuable so I'd like to know whether the described approach with less output might be a feature of gptcommit to condense output and maybe reduce costs.

Contributions are very much welcome! Feel free to a pull request!
--
Roger Zurawicki
zura.wiki ( https://zura.wiki ) | linkedin.com/in/zurawiki ( http://linkedin.com/in/zurawiki )
On Thu, Feb 02, 2023 at 2:27 PM, Christian Ivicevic < @.*** > wrote:

Thanks for the summary. I might have a proposal that may be an option to
take into consideration. As of right now I can technically create a patch
for a specific commit and without its subject line ask ChatGPT about a
descriptive commit message. Unlike gptcommit it would not summarize the
changes of each touched file, but it's basically a single sentence or
paragraph based on a patch file.
Given that GPT can't explain the reasoning behind certain changes
summarizing the details of each file feels less valuable so I'd like to
know whether the described approach with less output might be a feature of
gptcommit to condense output and maybe reduce costs.
—
Reply to this email directly, view it on GitHub (
https://github.com/zurawiki/gptcommit/issues/41#issuecomment-1414253180 ) ,
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AALFNYZV3K4JPRO42IJLKLTWVQDDVANCNFSM6AAAAAAUPGC2XA
).
You are receiving this because you commented. Message ID: <zurawiki/gptcommit/issues/41/1414253180
@ github. com>

https://github.com/markuswt/gpt-commit/blob/f2a8a239bcc7cac36fb73dcd817d5a56782f7d98/gpt-commit.py#L8-L9
This repo uses a shorter prompt.

Given the new Chat GPT models and cheaper pricing, is this issue still relevant?


Mailbags should state a specification version.

Is your feature request related to a problem? Please describe.
It is currently not clear which version of the Mailbag Specification that each mailbag is compliant with.
Describe the solution you'd like
Mailbags should state what version of the spec they are in compliance with in bag-info.txt.

addressed by #201


Update Dockerfile to LTS

Always follow the latest LTS version
(Now Node 20)
What kind of change does this PR introduce?
To switch to use the real Node LTS version without changing code.
E.g. latest LTS version is now 20, in the codebase it is still 18.
Did you add tests for your changes?
If relevant, did you update the documentation?
Summary
Use latest LTS version from Node as soon as possible.
Does this PR introduce a breaking change?
Should not break things, maybe in the future?
Other information
Let me know if you like this, or just use node:20
closes: https://github.com/puppeteer/puppeteer/issues/11290

We usually fix the version so semantics are clear. Nevertheless, fixed in #11296.


Allow deserialisation of types without a parameterless constructor

WCF's DataContract deserialisation uses FormatterServices.GetUninitializedObject to instantiate an instance of a type without calling a constructor. This allows types like this to be serialised and deserialised:
public class Foo
{
    public string Name { get; set; }

    public Foo(string name) { Name = name; }
}

msgpack-cli version 5.0 cannot deserialise this type as it requires a parameterless constructor. Version 6 adds some support via #52, but I don't think that would help here unless the property name was also lower-case.

Althhough finally I ignored the case of member/parameter name, Foo is not serializable because it has setter, so the default constructor is still required.
Why are you unable to add default constructor while the type is mutable?

I can add a default constructor, but I'd rather not as I want to force all users of the type to construct instances via a parameterised constructor, so I can perform validation on instances populated by code.
Right now I just mark these constructors as [Obsolete("Only used for serialisation")]. But I have a lot of these message classes and it's uglier having these.

I think you can do same thing via object initialization syntax and validation code in setter of properties. If you force to set multiple properties at once, you should not explore public setters. Do I miss some points?

I will not add GetUninitializedObject usage because it is not available on non desktop platform and I cannot believe it is usual situation.
It is welcome reopen this issue if you believe it is common issue. Thanks!


Refactor getDataFromBuffer

This patch rearranges getDataFromBuffer to not rely on nested buffers to
unvoid unnecessary allocation & deallocation.
This patch provides notable speedup for some cases.
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #626. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

MNIST result matches. No problem :)


test: use T.TempDir to create temporary test directory

Type of change

Test update

Description
A testing cleanup.
This pull request replaces ioutil.TempDir with t.TempDir. We can use the T.TempDir function from the testing package to create temporary directory. The directory created by T.TempDir is automatically removed when the test and all its subtests complete.
This saves us at least 2 lines (error check, and cleanup) on every instance, or in some cases adds cleanup that we forgot.
Reference: https://pkg.go.dev/testing#T.TempDir
func TestFoo(t *testing.T) {
	// before
	tmpDir, err := ioutil.TempDir("", "")
	require.NoError(t, err)
	defer os.RemoveAll(tmpDir)

	// now
	tmpDir := t.TempDir()
}

Additional details
Related issues

Rebased onto main.

I particularly like this change since the ioutil package has been deprecated since Go 1.16. 👍

@yacov are you thinking this change should only go into main and not the release-2.5 branch? If that is the case, it seems like a wide-ranging change to unit test like this could cause issues, as all changes made for v2.5 will also need to be applied to main.

@yacov are you thinking this change should only go into main and not the release-2.5 branch? If that is the case, it seems like a wide-ranging change to unit test like this could cause issues, as all changes made for v2.5 will also need to be applied to main.

I guess we could put it in release-2.5 so backporting bug fixes would be easier. I don't see any reason other than that.

I'm not sure what's causing TestEtcdraft to fail. I could not reproduce the error on my local machine.

I'm not sure what's causing TestEtcdraft to fail. I could not reproduce the error on my local machine.

Welcome to Fabric :-)

This is really weird. The same CI passed without errors in my fork https://github.com/Juneezee/fabric/actions/runs/3042353726/jobs/4900486546

Rebased onto main and resolved conflicts in orderer/common/server/main_test.go.


How to run the test suite?

I wanted to implement some fuzzing tests using Hypothesis. Before, I wanted to run the test suite unchanged. Unfortunately I couldn't manage to do so. The project neither has a setup.py nor a meaningful Poetry setup nor a tox setup.
I think one of these should be there to allow externals to reproduce the test suite. If they all are undesired, a brief section in the README.md would be very helpful.

Thanks for the push, yes, I should make it more clear.
For now, what you need is:
# I suggest creating your own virtualenv here
# Install dependencies
pip install -r dev-requirements.txt

# Run package installer
flit install --sym

# Run tests
pytest

Let me know if it works for you!
And that should be it. I need to see if adding poetry makes sense since we already have flit (poetry does more than flit, but flit is simpler to publish IMHO).

I created a new contributing doc and created a new issue to track the new CI/CD flow (with Poetry) that would benefit the whole project: https://github.com/guilatrova/tryceratops/issues/19


Openh264v1.5

Merge pull request #2123 from sijchen/fixsimul
[Encoder] fix for simulcast case when the frame rate of lower resolution is higher
Source of Pull request MERGE is HaiboZhu HaiboZhu

Meaningless about this pull request, close it.


Failures on Test field promotion during top level inference.

There are new test failures on Test field promotion during top level inference..
The tests
language/inference_update_2/top_level_type_inference_test CompileTimeError (expected Pass)

are failing on configurations
cfe-strong-linux
cfe-weak-linux
dart2js-hostasserts-strong-linux-x64-d8
dart2js-hostasserts-weak-linux-x64-d8
dart2wasm-hostasserts-linux-x64-d8
dartdevk-strong-linux-release-chrome
dartdevk-weak-linux-release-chrome
dartk-strong-linux-debug-x64
dartk-strong-linux-release-x64
dartk-weak-asserts-linux-debug-x64
dartk-weak-asserts-linux-release-x64

@johnniwinther ping.  Have you had a chance to look into this?  I'm hoping to get back to field promotion in early January and this is one of the few remaining blocking items.

I landed the fix yesterday, so you should be unblocked.

I didn't see any tests that started passing, though. Do we still need to update how this is queried in the CFE?

Unassigning myself because I'm not actively working on field promotion right now; I will likely pick it up again later in the year.


Add the ability to sort the cavity output table

Current status
The cavity table outputs cavities ordered by volume.
Issue
The user may be interested in features other than the cavity volume, such as surface or what type of cavity it is (after #88 has been implemented).
Proposed change
Add the ability to sort the table by any column, by clicking on the corresponding header cell. Since we have unicode support we can add a symbols ▼ and ▲ to the header cells to give an indication of how the list will be sorted when the header cell is clicked.

It's a good idea. We should first check if such options are already present in wxWidgets. In such case, it could be very simple to implement this feature.

There appears to be not built-in way to sort but there is a sorting event thrown by wxGrid. So it would be possible to handle the event and draw up a sorted grid in the event function.
https://stackoverflow.com/questions/14048043/how-do-i-handle-the-wxevt-grid-col-sort-event-to-sort-a-grid


Download Doesn't Start

Download does not start at all. It's always 0 KB/s. I can't click on pause and play buttons. This issue is only for one particular course. Name of the course is (42 Saat) Python : Sıfırdan İleri Seviye Programlama (2019)

Same is happening to me. The issue is for only one particular course. Name of the course -:
learn-complete-python-tutorial-in-simple-way url : https://www.udemy.com/course/learn-complete-python-tutorial-in-simple-way/


Can't find/use tmLanguage scope "source.json"

I've installed the Asciidoc extension (https://marketplace.visualstudio.com/items/justusadam.language-asciidoc) but can't get syntax highlighting on [source,json] blocks.
Analysing the ".tmLanguage" file I've tested switching "source.json" with "source.js" and it worked as expected (for javascript), however the syntax is not the one expected for json (like in .md files):
Before change:

After change:

Expected:

I've also looked for "source.json" scopeName in all files of this repository (Branch: master) but couldn't find it.
Thanks in advance for your attention. =)

We don't use text mate to colorize JSON yet, that's why you didn't find it.
I now added the JSON text mate grammar so your reference should work.


Update the installation via CocoaPods

At the moment, the installation instructions via CocoaPods are as follows:
pod 'JsonRPC.swift', '~> 0.1.0'
However, the current version can only be installed if you use the following command"
pod 'JsonRPC.swift', :git => 'https://github.com/tesseract-one/JsonRPC.swift', :branch => 'main'

we didn't release a new version yet. It will be released as 0.2.0


Possible regression in microbenchmarks_ios_MatrixUtils_affine_transformRect_iteration &

Also seen in
microbenchmarks_ios_MatrixUtils_persp_transformPoint_iteration, microbenchmarks_ios_combineSemanticsInfoList_iteration, microbenchmarks_ios_MatrixUtils_affine_transformPoint_iteration, microbenchmarks_ios_combineSemanticsInfoSyncStar_iteration, and microbenchmarks_ios_gesture_detector_bench, microbenchmarks_ios_stock_build_iteration, microbenchmarks_ios_stock_layout_iteration, microbenchmarks_ios_traverseIterableSyncStar_iteration, and microbenchmarks_ios_traverseIterableSyncStar_iteration. Many of these are quite noisy, but the trend appears to start markedly at or about #41329. (Commit https://github.com/flutter/flutter/commit/0b0942a60caafff56c19fea07d8f7b14e6286e1c). It's not clear why to me why this refactor would do it, though.

Removing the TODAY issue since it's not clear that this should be a release blocker.

@kf6gpe : the regression seems to be strangely fixed by https://github.com/flutter/flutter/commit/e778686e72e799e3555adf0777bee548eecb3103 and it's now even lower than the number before the regression (before regression: ~85ms, after regreesion: ~95ms, now: ~80ms).
@flar : any idea how this could happen?

I see no reason for the recent improvements. I searched 4 or 5 changes on either side of the first result that was faster and didn't see anything that would have affected it. It is pure dart code so this would have to be something that changed in the dart runtime and/or the Float64List support...?
Also, the description talks about a regressive trend that started around #41329, but I don't see that either when I look at the history of these benchmarks...?

Closing this as the regression is gone. (Although we don't know how it's introduced, and how it's fixed; maybe some infra flakiness again?)


How do a load a file to emulate?

I want to emulate the UID of a RFID tag I read with one of the available tools, how do I load it in your application so it can be emulated?
thank you

Is it possible to emulate UID with this tool without root?


<EMAIL_ADDRESS>decompress error

Updating 'gimp' (2.10.38 -> 2.10.38-1)
Downloading new version
gimp-2.10.38-setup-1.exe (323,0 MB) [=========================================================================] 100%
Checking hash of gimp-2.10.38-setup-1.exe ... ok.
Uninstalling 'gimp' (2.10.38)
Removing shim 'gimp-console-2.10.shim'.
Removing shim 'gimp-console-2.10.exe'.
Removing shim 'gimp-console.shim'.
Removing shim 'gimp-console.exe'.
Removing shim 'gimp.shim'.
Removing shim 'gimp.exe'.
Removing shim 'gimptool-2.0.shim'.
Removing shim 'gimptool-2.0.exe'.
Removing shim 'gimptool.shim'.
Removing shim 'gimptool.exe'.
Unlinking ~\scoop\apps\gimp\current
Installing 'gimp' (2.10.38-1) [64bit] from 'extras' bucket
Loading gimp-2.10.38-setup-1.exe from cache
Extracting gimp-2.10.38-setup-1.exe ... ERROR Exit code was 1!
Failed to extract files from C:\Users\pablo.hernandez\scoop\apps\gimp\2.10.38-1\gimp-2.10.38-setup-1.exe.
Log file:
~\scoop\apps\gimp\2.10.38-1\innounp.log
CONTENTS OF innounp.log:
Signature detected: Inno Setup Setup Data (6.3.0)
This is not directly supported, but i'll try to unpack it as version 5602
; Version detected: 6300
Critical error: The setup files are corrupted. Please obtain a new copy of the program.

Duplicates #14126.


Can't Load Palettes in 49.1

Since updating to 49.1, when you click Load Palette nothing pops up to select a palette so I can't get any new palettes.

I just pushed a new version which should ix this issue. You can update it in Sketch or grab it from the link below. Let me know if you still have any issues!
https://github.com/andrewfiorillo/sketch-palettes/releases/latest


[CP] Mac certificate updates for 2023-2024

Issue Link
https://github.com/flutter/flutter/issues/137166
Commit Hash
20e518d185b150dc9eb804e1a821b4a5f6b2f514
Target
stable
PR Link
https://github.com/flutter/flutter/pull/137779
Changelog Description
Mac code signing certificates have expired and needed renewal. This change points to the cipd package version with the new certificates.
Impacted Users
All mac builds that require code signing
Impact Description
If the update is not applied, the old (expired) certificates will be used, causing relevant mac builds to fail.
Workaround
There is none.
Risk
low
Test Coverage
yes
Validation Steps
Mac builds that use code signing continue to pass, specifically: Mac_build_test flutter_gallery__transition_perf_e2e_ios

Marking obsolete. We've promoted 3.16 to stable, and have no plans to ship a new 3.13 stable.


Allow arbitary tag to server

This patch (plus its rediculous merge history) allows assigning an arbitary object to the server. Can be used to establish context across requests.

This looks interesting. Can you provide me with a use case and some sample code for this? That way I can update the documentation appropriately.  Thanks!

Scott,
sorry for the late response - seems that mail asking for it went lost.
Use case - attaching state objects, business logic, etc. That way, you get around keeping this globally, which arguably is not testable overly well.
Right now, my sample code cannot be shared. If you need it, I'll see if I can make up something.


different responses for consecutive calls with identical requests

Is there a way to have different responses for consecutive api calls to the same endpoint and with the same request data? For example, simulating the refreshing of an api access token. Each call will be identical, but the response will come back with a new api access token and a new expiration date. Right now I am just confirming that I call the endpoint more than once, which really does suffice, but I was wondering if it would be possible to change the response with each consecutive call.

You should be able to implement this using the up_to_n_times method. If you create two mocks, each with up_to_n_times(1), wiremock uses the first mock for your first request and the second mock for the second request. Example:
use wiremock::matchers::method;
use wiremock::{Mock, MockServer, ResponseTemplate};

#[async_std::main]
async fn main() {
    let mock_server = MockServer::start().await;

    Mock::given(method("GET"))
        .respond_with(ResponseTemplate::new(200).set_body_string("TOKEN_1".to_string()))
        .up_to_n_times(1)
        .mount(&mock_server)
        .await;

    Mock::given(method("GET"))
        .respond_with(ResponseTemplate::new(200).set_body_string("TOKEN_2".to_string()))
        .up_to_n_times(1)
        .mount(&mock_server)
        .await;

    let body = surf::get(&mock_server.uri())
        .await
        .unwrap()
        .body_string()
        .await
        .unwrap();
    assert_eq!(&body, "TOKEN_1");

    let body = surf::get(&mock_server.uri())
        .await
        .unwrap()
        .body_string()
        .await
        .unwrap();
    assert_eq!(&body, "TOKEN_2");
}

Perfect! Thanks @SebRollen and @LukeMathWalker and thanks @LukeMathWalker for your book! It has been extremely helpful and I look forward to you completing it.


Reflect the errors of report generating

Description
At the moment, all of our Reporter here does not return anything even when errors happen while generating the reports.
Therefore, it's better for us to return the error accordingly so that the Analyzer could decide on the exit status code (https://github.com/oracle-samples/macaron/issues/175).

Description At the moment, all of our Reporter here does not return anything even when errors happen while generating the reports. Therefore, it's better for us to return the error accordingly so that the Analyzer could decide on the exit status code (#175).

Do you have any specific scenario in mind? In most cases the report generator should not make the process exit.

In most cases the report generator should not make the process exit.

Yes that is true. I was thinking about whether we should explicitly show the user that the reports could not be generated (of an expected internal error). For example, at the moment, if Macaron cannot generate the report, the return status code would still be 0 (no error) if the analysis succeed. I am thinking that the user might not aware of it and look into the old version of the reports or could not find the report anywhere.

I am thinking that the user might not aware of it and look into the old version of the reports or could not find the report anywhere.

We should remove old results before producing new results to avoid this. I'm not sure if we should return an error code, which would be considered as Macaron's analysis failure.


MAINT: disable failing conda bundling.

Until fixed, otherwise all PRs everywhere are failing.
Please open an issue to track re-enabling when merging this PR.

thanks!

https://github.com/napari/napari/issues/4879

It's a known issue: https://github.com/conda/conda/issues/11657
Setting the CONDA_NUMBER_CHANNEL_NOTICES =0 env var can be used as a workaround.