onCreate callback is not being called, "tbl not found" exception

For whatever reason I cannot get the onCreate callback to run when opening the app, which is leading to the tables not existing when the first operation is attempted. I've tried clearing the cache, uninstalling and reinstalling, but nothing seems to work. The device is just stuck. I'm using a Pixel 4 XL running Android 13.
I'm calling await $FloorAppDatabase.databaseBuilder('app_database.db').build() in main(), but the full error is:
`no such table: tbl_tablename (code 1 SQLITE_ERROR): , while compiling: INSERT OR ABORT INTO tbl_tablename (...) VALUES (...)
#0      wrapDatabaseException (package:sqflite/src/exception_impl.dart:11:7)

#1      SqfliteDatabaseMixin.txnRawInsert. (package:sqflite_common/src/database_mixin.dart:576:14)

#2      BasicLock.synchronized (package:synchronized/src/basic_lock.dart:33:16)

#3      SqfliteDatabaseMixin.txnSynchronized (package:sqflite_common/src/database_mixin.dart:517:14)

#4      InsertionAdapter._insert (package:floor/src/adapter/insertion_adapter.dart:77:20)

#5      InsertionAdapter.insert (package:floor/src/adapter/insertion_adapter.dart:28:5)

#6      _$InternalDao.insertData (package:...)

#7      InternalRepo._callback (package:...)

#8      InternalBloc._handler (package:...)

#9      Bloc.on..handleEvent (package:bloc/src/bloc.dart:229:13)
`

I think that you should try uninstalling the old app from your device and running it again.  onCreate method in Sqflite will only execute the first time your app runs. Any code changes made after that will not create the database or modify the table.


[GitHub Usage] Filter checked out files by blob size

Define a limit for the config files that will be checkout and set it in the background Job

Work is complete. Deployment is tracked by https://github.com/nuget/nugetgallery/issues/7214


Action triggers for Check-ins #275

We need a pattern that can allow a user to take an action on a check-in like resend, edit, etc.
check-in should have action triggers (#90)
User can archive a check-in
User can delete a check-in
User can resend a check-in
User can author a new check-in with the same settings as the current check-in

I'd suggest we continue to lean mobile-centric and append a "more" button to the RollCall summary:

Selecting that button will take the user to that RollCall's detailed view, which includes actions in its toolbar

Note that, on larger screens, the toolbar actions include text labels:

How do we look, @caharding?

his looks great, can we put the action trigger button in the upper right hand corner?


Tracking the overhaul and re-organization of the book

Due to a number of factors the book is not currently in the best shape. Due to the nature of organic growth things are not necessarily structured how they should be, and information is spread out all over the place.
In general, in addition to the structural changes listed below, the intent of this effort is to improve the quality of text and remove anything which may cause confusion and/or lie beyond the scope of this book. We can relegate to other documents more than we currently are, for example.
Based on a meeting between @SergioGasquez and myself:

[ ] 1. Introduction

Why should I use Rust and/or ESP devices?
Make it clear that Espressif is involved in the project

[ ] 2. Ecosystem Overview

What are the different approaches? (std, no_std)
Why should you use one or the other?
Combine chip support tables into one
Additional sub-chapters or appendices detailing each approach

[ ] 3. Setting Up a Development Environment

Defer to the tooling's documentation when possible
NOT going to document installing Rust
NOT going to document install ldproxy
Move troubleshooting to an Appendix (or something)

[ ] 4. Writing an Application

At the very least both std and no_std should have the same steps in the tutorial
We will need to decide how to structure all this

[ ] 5. Tooling

For editors, only discuss using VS Code and Rust Analyzer
Get rid of the espmonitor chapter
This whole chapter just needs general improvement (especially debugging)

[ ] 8. Appendix A - Troubleshooting

If you have any questions, concerns, or feedback about these changes please leave a comment below.

I'm going to begin at... the beginning. Working on the Introduction chapter.

I'll probably close https://github.com/esp-rs/book/pull/50 but, it can be used as a reference for the espflash section of chapter 5

I am working on updating the Tooling and Writing Your Own Application chapters

All the content of the chapters are now updated, we well now start working on having a consistent format across the whole book, update the chapter titles and applying the guidelines.


Licence type?

Hi,
First of all thank you for the elegant solution to the frustrating lack of support from EF Core on temporal tables.
I'm keen to use this for a project at work but notice that a licence type has not been selected yet.
Obviously if your wishes are to not have this used in a corporate environment I respect your choice but if you are happy for it to be used would you consider tagging the repo with a MIT licence?
Thanks again

Hi,
Sorry for the missing license, I was so keen on providing a good solution for temporal tables in EF Core that I had forgotten to add such basic information to the project. I've added the project under MIT license and you and everyone else can use this library safely. Thank you for pointing that out.
I'd like also to thank you for your interest and your appreciation of the project and I would be happy if it's used by more people.

Thank you for your prompt response.
Absolutely no reason to apologise as it's an easy one to miss.


chore: nodejs-hello-world-2 to 0.0.1

chore: Promote nodejs-hello-world-2 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


This looks amazing!!!

I really would love to give this a try, but im having a hard time following. I apologize in advance as i am not a programmer. Is there a basic instruction sheet somewhere or could you give me a basic rundown of how to use these scripts? This truly is what ive been looking for but i just cant seem to wrap my head around it.
Thanks, and again great work!

Right now, it is only useful for programmers.  What do you want to do with
the data? If there is a particular dataset you are looking for, I may be
able to send it to you.
On Aug 1, 2017 12:05, "ogbobby"<EMAIL_ADDRESS>wrote:

I really would love to give this a try, but im having a hard time
following. I apologize in advance as i am not a programmer. Is there a
basic instruction sheet somewhere or could you give me a basic rundown of
how to use these scripts? This truly is what ive been looking for but i
just cant seem to wrap my head around it.
Thanks, and again great work!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACoMSTzjbNyaTe5ep9hbWeqyOeaYls_xks5sT1rMgaJpZM4OqBWO
.

I want to be a better dfs player.
On Aug 5, 2017 12:16 PM, "Eric Truett"<EMAIL_ADDRESS>wrote:

Right now, it is only useful for programmers. What do you want to do with
the data? If there is a particular dataset you are looking for, I may be
able to send it to you.
On Aug 1, 2017 12:05, "ogbobby"<EMAIL_ADDRESS>wrote:

I really would love to give this a try, but im having a hard time
following. I apologize in advance as i am not a programmer. Is there a
basic instruction sheet somewhere or could you give me a basic rundown of
how to use these scripts? This truly is what ive been looking for but i
just cant seem to wrap my head around it.
Thanks, and again great work!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ACoMSTzjbNyaTe5ep9hbWeqyOeaYls_xks5sT1rMgaJpZM4OqBWO>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320452467, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHki49s-awczrPBQ1S4Zj5yT7adI8J1_ks5sVJVkgaJpZM4OqBWO
.

 NBA dfs is a rough
game. Good luck!
On Aug 5, 2017 11:23, "ogbobby"<EMAIL_ADDRESS>wrote:

I want to be a better dfs player.
On Aug 5, 2017 12:16 PM, "Eric Truett"<EMAIL_ADDRESS>wrote:

Right now, it is only useful for programmers. What do you want to do with
the data? If there is a particular dataset you are looking for, I may be
able to send it to you.
On Aug 1, 2017 12:05, "ogbobby"<EMAIL_ADDRESS>wrote:

I really would love to give this a try, but im having a hard time
following. I apologize in advance as i am not a programmer. Is there a
basic instruction sheet somewhere or could you give me a basic rundown
of
how to use these scripts? This truly is what ive been looking for but i
just cant seem to wrap my head around it.
Thanks, and again great work!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ACoMSTzjbNyaTe5ep9hbWeqyOeaYls_xks5sT1rMgaJpZM4OqBWO>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320452467, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/AHki49s-
awczrPBQ1S4Zj5yT7adI8J1_ks5sVJVkgaJpZM4OqBWO>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320452784, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACoMSSJUepswTqKNuSRZAvQAWe1eq5vjks5sVJbsgaJpZM4OqBWO
.

Here is a link to a dataset of player boxscores from 1999-2016. It expires
in 24 hours or after one download.
https://send.firefox.com/download/4c6c15499c/#fdd1UPGrprkcC6GFg6OxJA
On Sat, Aug 5, 2017 at 11:23 AM, ogbobby<EMAIL_ADDRESS>wrote:

I want to be a better dfs player.
On Aug 5, 2017 12:16 PM, "Eric Truett"<EMAIL_ADDRESS>wrote:

Right now, it is only useful for programmers. What do you want to do with
the data? If there is a particular dataset you are looking for, I may be
able to send it to you.
On Aug 1, 2017 12:05, "ogbobby"<EMAIL_ADDRESS>wrote:

I really would love to give this a try, but im having a hard time
following. I apologize in advance as i am not a programmer. Is there a
basic instruction sheet somewhere or could you give me a basic rundown
of
how to use these scripts? This truly is what ive been looking for but i
just cant seem to wrap my head around it.
Thanks, and again great work!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ACoMSTzjbNyaTe5ep9hbWeqyOeaYls_xks5sT1rMgaJpZM4OqBWO>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320452467, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/AHki49s-
awczrPBQ1S4Zj5yT7adI8J1_ks5sVJVkgaJpZM4OqBWO>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320452784, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACoMSSJUepswTqKNuSRZAvQAWe1eq5vjks5sVJbsgaJpZM4OqBWO
.

--
Eric Truett
<EMAIL_ADDRESS>708-303-8625

Thanks man
On Aug 5, 2017 2:42 PM, "Eric Truett"<EMAIL_ADDRESS>wrote:

I'll send you a link to a dataset to get you started. NBA dfs is a rough
game. Good luck!
On Aug 5, 2017 11:23, "ogbobby"<EMAIL_ADDRESS>wrote:

I want to be a better dfs player.
On Aug 5, 2017 12:16 PM, "Eric Truett"<EMAIL_ADDRESS>wrote:

Right now, it is only useful for programmers. What do you want to do
with
the data? If there is a particular dataset you are looking for, I may
be
able to send it to you.
On Aug 1, 2017 12:05, "ogbobby"<EMAIL_ADDRESS>wrote:

I really would love to give this a try, but im having a hard time
following. I apologize in advance as i am not a programmer. Is there
a
basic instruction sheet somewhere or could you give me a basic
rundown
of
how to use these scripts? This truly is what ive been looking for
but i
just cant seem to wrap my head around it.
Thanks, and again great work!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ACoMSTzjbNyaTe5ep9hbWeqyOeaYls_xks5sT1rMgaJpZM4OqBWO>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320452467, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/AHki49s-
awczrPBQ1S4Zj5yT7adI8J1_ks5sVJVkgaJpZM4OqBWO>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320452784, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/
ACoMSSJUepswTqKNuSRZAvQAWe1eq5vjks5sVJbsgaJpZM4OqBWO>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sansbacon/nba/issues/3#issuecomment-320462936, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHki48LVgyAyb91PYKv45h6YGUaMRMWkks5sVLeVgaJpZM4OqBWO
.

This is great work @sansbacon. I'm actually working on some theories to predict player performance. I want to use your project as well. Do you happen to have any examples of how you've been using your work? Also, how can one contribute to your work?


stuck on AAPL: #[EB|LOG:EXITBS:START]. Opencore 0.9.3

hello (i'm new here - still getting my bearings).
I've followed the dortania guide to a T (thanks be to those folks), but keep sticking.  After brief a brief loading sequence, I get stuck at 'couldn't get array serialized at 0 index.' then, I'm able to select the USB, which reveals the apple logo (but no load bar). This stalls infinitely. the last line of the log reads 'AAPL: #[EB|LOG:EXITBS:START]'. (log attached).
I've also tried all prospective solutions at dortania, to no avail. Can I get any pointers to proceed?
System:
CPU: Intel 13600k
GPU: Asrock Radeon 6800xt
MOBO: Asrock Z790M
Thanks in advance. I know there's some real big brains here.
opencore-2023-07-30-213322.txt

i now realize this is likely not an appropriate place to post. like i said - still getting my bearings. I also tried to upload EFI, but it's too big. my apologies for my evident ignorance.

https://discord.gg/Wxam8aH


libfree gui tutorial

Tutorial for library free search using the alphaDIA GUI.

Tutorial as .pdf
Library-free DIA search using the GUI - alphadia documentation.pdf


Assert query plan

This is now working and the results are stable across runners (but not stable over time as staging content changes - but that's part of what we're testing).
I ran the test with an expected cost set low enough to trigger a failure both locally and in the CI to compare:
CI:
    ✖ test_packageMatchQuery, failed - Total cost of 266.48 above threshold of 70.0 (incl variation)
2477

M1 MacBook Pro:
/Users/sas/Projects/SPI/spi-server/Tests/AppTests/QueryPerformanceTests.swift:41: error: -[AppTests.QueryPerformanceTests test_packageMatchQuery] : failed - Total cost of 266.48 above threshold of 70.0 (incl variation)

I've now rebased this branch on main. I set the baseline specifically based on a7a82a700e5533d8722c3c589ce2333d0b686719 (merge after 2.58.1) to capture the baseline before the latest search changes merged in via #1570 and it's caught the first regressions:
    ✔ test_Search_authorMatchQuery (0.891 seconds)
    ✔ test_Search_keywordMatchQuery (0.057 seconds)
    ✔ test_Search_packageMatchQuery (0.051 seconds)
    ✔ test_Search_query_authorFilter (0.057 seconds)
    ✖ test_Search_query_keywordFilter, failed - Total cost of 3921.07 above threshold of 3900.0 (incl variation)
    ✖ test_Search_query_lastActicityFilter, failed - Total cost of 3921.02 above threshold of 3900.0 (incl variation)
    ✔ test_Search_query_licenseFilter (0.077 seconds)
    ✖ test_Search_query_noFilter, failed - Total cost of 3909.57 above threshold of 3900.0 (incl variation)
    ✖ test_Search_query_platformFilter, failed - Total cost of 3800.42 above threshold of 3800.0 (incl variation)
    ✖ test_Search_query_starsFilter, failed - Total cost of 3826.71 above threshold of 3800.0 (incl variation)


Fix example-runner-ash's validation layer errors

This PR fixes example-runner-ash's errors which are displayed with --debug-layer option.
> cargo run --bin example-runner-ash -- --debug-layer
   Compiling example-runner-ash v0.4.0-alpha.10 (C:\Users\hato2\Desktop\fork\rust-gpu\examples\runners\ash)
    Finished dev [unoptimized + debuginfo] target(s) in 2.48s
     Running `target\debug\example-runner-ash.exe --debug-layer`
    Finished release [optimized] target(s) in 0.17s
INFO:
GENERAL [Loader Message (0)] : Inserted device layer VK_LAYER_KHRONOS_validation (C:\VulkanSDK\<IP_ADDRESS>\Bin\.\VkLayer_khronos_validation.dll)

INFO:
GENERAL [Loader Message (0)] : Inserted device layer VK_LAYER_NV_optimus (C:\WINDOWS\System32\DriverStore\FileRepository\nv_dispi.inf_amd64_4a746d937e6a7240\.\nvoglv64.dll)

ERROR:
VALIDATION [VUID-VkShaderModuleCreateInfo-pCode-01091 (-1480880714)] : Validation Error: [ VUID-VkShaderModuleCreateInfo-pCode-01091 ] Object 0: handle = 0x20ea37c94d8, type = VK_OBJECT_TYPE_DEVICE; | MessageID = 0xa7bb8db6 | vkCreateShaderModule(): The SPIR-V Capability (VulkanMemoryModel) was declared, but none of the requirements were met to use it. The Vulkan spec states: If pCode declares any of the capabilities listed in the SPIR-V Environment appendix, one of the corresponding requirements must be satisfied (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/windows/1.2-extensions/vkspec.html#VUID-VkShaderModuleCreateInfo-pCode-01091)

ERROR:
VALIDATION [VUID-VkShaderModuleCreateInfo-pCode-04147<PHONE_NUMBER>)] : Validation Error: [ VUID-VkShaderModuleCreateInfo-pCode-04147 ] Object 0: handle = 0x20ea37c94d8, type = VK_OBJECT_TYPE_DEVICE; | MessageID = 0x3d492883 | vkCreateShaderModule(): The SPIR-V Extension (SPV_KHR_vulkan_memory_model) was declared, but none of the requirements were met to use it. The Vulkan spec states: If pCode declares any of the SPIR-V extensions listed in the SPIR-V Environment appendix, one of the corresponding requirements must be satisfied (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/windows/1.2-extensions/vkspec.html#VUID-VkShaderModuleCreateInfo-pCode-04147)

ERROR:
VALIDATION [VUID-VkShaderModuleCreateInfo-pCode-01091 (-1480880714)] : Validation Error: [ VUID-VkShaderModuleCreateInfo-pCode-01091 ] Object 0: handle = 0x20ea37c94d8, type = VK_OBJECT_TYPE_DEVICE; | MessageID = 0xa7bb8db6 | vkCreateShaderModule(): The SPIR-V Capability (VulkanMemoryModel) was declared, but none of the requirements were met to use it. The Vulkan spec states: If pCode declares any of the capabilities listed in the SPIR-V Environment appendix, one of the corresponding requirements must be satisfied (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/windows/1.2-extensions/vkspec.html#VUID-VkShaderModuleCreateInfo-pCode-01091)

ERROR:
VALIDATION [VUID-VkShaderModuleCreateInfo-pCode-04147<PHONE_NUMBER>)] : Validation Error: [ VUID-VkShaderModuleCreateInfo-pCode-04147 ] Object 0: handle = 0x20ea37c94d8, type = VK_OBJECT_TYPE_DEVICE; | MessageID = 0x3d492883 | vkCreateShaderModule(): The SPIR-V Extension (SPV_KHR_vulkan_memory_model) was declared, but none of the requirements were met to use it. The Vulkan spec states: If pCode declares any of the SPIR-V extensions listed in the SPIR-V Environment appendix, one of the corresponding requirements must be satisfied (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/windows/1.2-extensions/vkspec.html#VUID-VkShaderModuleCreateInfo-pCode-04147)

Although https://github.com/EmbarkStudios/rust-gpu/pull/693 is needed to run in my environment, it's not contained in this PR.

I believe #701 should have fixed the CI, could you rebase?

I've rebased and pushed.


Automatically check available ram before spawning subprocesses.

Automatically check available ram before spawning subprocesses.

Ideally we can move away from spawning so many subprocesses, I have a few thoughts on this but it's especially costly on windows.  I've played around with RPC implementations among other things.  The cost of initialization is super high on subprocesses so it's barely worth it in a lot of cases


Performance improvement in ToTensor GPU Kernel

Description
Earlier, we were using Kernel Launch/Map way of launching kernel to write common CPU and GPU code for ToTensor operator. However, I observed there are too many threads and blocks being launched with kernel causing significant performance implication.
To overcome, I wrote a separate CUDA kernel for GPU and moved out of Kernel launch/map.
Benchmarks below.
Benchmarks
Ran 1000 ToTensor operation on (512, 512, 3)
GPU
Before
('Average time per ToTensor 512,512,3 - ', 39.17948246002197)
After
('Average time per ToTensor 512,512,3 - ', 0.44632863998413086)
CPU
Before
('Average time per ToTensor 512,512,3 - ', 3.7258052825927734)
After
('Average time per Normalize 3,512,512 - ', 1.8473007678985596)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
[X] Code is well-documented:
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

Remove Kernel Launch/Map for ToTensor operator
Make independent kernel for CPU ToTensor
Add 2 separate CUDA kernel for ToTensor operator (3D and 4D) inputs.

@zhreshold @stu1130

@mxnet-label-bot add [pr-work-in-progress, Operator, Performance]

LGTM now, thanks for the efforts!


Remove unused variables / add linting

In my last few PRs I have noticed that there are many unused variables across the code. This can lead to issues as we saw in #745. This PR adds eslint to catch these sorts of errors. The eslint configuration is based on the standard "eslint:recommended" rule set. I have disabled:

no-prototype-builtins
no-empty
no-regex-spaces
no-useless-escape

I have disabled these so that this PR has minimal impact on the code. I recommend someone who knows both the code and the regular expressions to re-enable the no-regex-spaces and no-useless-escape rules and fixing the code accordingly.

ping: @isaacs @coreyfarrell


uguid: Update changelog

Codecov Report

Merging #83 (6723d65) into main (b7b4189) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   91.80%   91.80%
=======================================
  Files          17       17
  Lines        1440     1440
=======================================
  Hits         1322     1322
  Misses        118      118

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[pre-commit] Updated golangci-lint hook to v1.57.2

The version of golangci-lint hook configured in the pre-commit config file, v1.55.2, contains a dependency to a deprecated url of the library musttag, and installing pre-commit hooks from scratch caused this error:
    pkg/golinters/musttag.go:4:2: unrecognized import path "go.tmz.dev/musttag": parse https://go.tmz.dev/musttag?go-get=1: no go-import meta tags ()

This problem has been fixed with this [1] commit, so we have updated the hook to the last available version.
[1] https://github.com/golangci/golangci-lint/pull/4201

I'm closing the PR because i was trying pre-commit with golang 1.21 provided by Fedora packages. Installing golang 1.20.x solved the issue.


What new features / bugs are squashed when upgrading from 1.2 to 2.1 ?

I'm using Branch in an app where all code needs to be in C#.  The thin wrapper will not pass code review, since that staff  only knows C# not Obj-C.
We will continue with 1.2 -locked, however I want to know more about the statement below (what feature, what bugs?

This fixed a ton of bugs and added additional functionality, but will require you to change the interfaces.

Hi @makerofthings7 , first version of Xamarin SDK is written like a one more project.
From version 2.0.0 we have been started using of native iOS and Android libs in Xamarin SDK (through binding). That means all changes and new features from native iOS and Android appear in Xamarin SDK automatically without additional development.
You can investigate new features, fixes, etc. in Android and iOS threads.
Each SDK has changelog.md, and you can see all changes.
Note: If you plan to use new version, you have to wait our newest Xamarin SDK that we are creating right now.

Hi @makerofthings7 - we have updated our Xamarin SDK to version 3.


Amplify delete is taking too long, hours and it still not deleted!!!

App Id:
Region:
Step:
Status:

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
Error message
If there is an error message, please include it here.
Note: Be sure to check the message for sensitive information.
Additional information
Please add any other relevant information. Please feel free to include screenshots.
additional sample code or a sample repository to help us reproduce the issue. (Be sure to remove any sensitive data)

@rickirea
We need to gather some information about the project to further investigate the issue.

what is the Operating System? and it's version?
What is the Amplify CLI version?
What backend resource categories did you add to your project (AppSync API, function, storage, etc.)?

Please go to the Cloudformation, open the root stack template and see if you can get the errors in the CloudFormation delete operation.

Probably not an amplify issue, but a CloudFormation issue.

Closing the issue for now, feel free to open another one if you are still blocked.


Misc fixes

Error-tolerant handling of issues with file access permissions
Additional handling of "socket hangup" network issue

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Invalid lock file generated

Steps to Reproduce

unzip attached example project pkg-1.zip
run rye sync

Oddly enough, if you pin the python version to 3.11 or above, this error does not occur.
Expected Result
Valid requirements lock files are generated.
Actual Result
error: Couldn't parse requirement in `requirements-dev.lock` at position 24635
  Caused by: Expected a valid marker name, found ''
exceptiongroup==1.2.2 ; python_version < '3.11' and (python_version < '3.8' or python_version >= '3.11' or platform_machine != 'loongarch64' or platform_python_implementation == 'PyPy' or (python_version < '3.13' and python_version >= '3.12') or (platform_machine == 'loongarch64' and platform_python_implementation != 'PyPy') or (platform_machine == 'loongarch64' and platform_python_implementation != 'PyPy' and ()))

Version Info
rye 0.38.0
commit: 0.38.0 (3e3c8540f 2024-08-02)
platform: macos (aarch64)
self-python<EMAIL_ADDRESS>symlink support: true
uv enabled: true
Stacktrace
stack backtrace:
   0: std::backtrace::Backtrace::create
   1: anyhow::error::<impl anyhow::Error>::msg
   2: rye::uv::UvWithVenv::sync
   3: rye::sync::sync
   4: rye::cli::sync::execute
   5: rye::cli::execute
   6: std::panicking::try
   7: rye::utils::panic::trap_bad_pipe
   8: rye::main
   9: std::sys_common::backtrace::__rust_begin_short_backtrace
  10: std::rt::lang_start::{{closure}}
  11: std::rt::lang_start_internal
  12: _main

Fairly certain this is fixed in newer versions of uv

I can upgrade and cut a new release today, probably.

Yep, just built and ran rye from main branch and it is fixed.

Slightly off topic, but are there any notable rye features missing from uv at this point? I was reading through the documentation and it appears that there is (experimental) project support that looks almost identical to rye.


search results are returned when they shouldn't be, with non matching terms highlighted

If you go to the demo here https://mdn.pagefind.app/ and type in sdafadsfasdf (for example), 23 results are returned all seeming to match the sd part of the query.
I have seen in my own website that I'm developing even weirder behaviour where something like the word intermediate will be matched along with things like sd.
I can only assume this is a bug to do with stemming/backtracking etc. ?

Currently not a bug per-se, but just the effect of Pagefind's backtracking not having a cap, so it'll backtrack your query until it finds something.
Some sane limit should be put on this, but in the absence of picking some arbitrary sane limit I have left it with no limit 😅
Open to suggestions!

Any possibility to have it configurable with pagefind.options or similar?
I can see that in some circumstances this behaviour would be desirable but in other cases not, so letting the user configure it makes the most sense ( and the default can be as is, of course)


Error: fetching raw block for CID

I have documented my setup in this repo:
https://github.com/sloev/orbitdb-nodejs-local-demo
Usage
create db:
$ node create_db.js 13245 FOOBAR
running with port:13245 and ./FOOBAR
clientEventsDb address /orbitdb/zdpuAkn4eb7Xdy5UFCvFGEcYd5HDhufhJHWAgYgVMpWK7U5wF

run server
$ node server.js 13245 FOOBAR  /orbitdb/zdpuAkn4eb7Xdy5UFCvFGEcYd5HDhufhJHWAgYgVMpWK7U5wF
running with port:13245 and ./FOOBAR
clientEventsDb address /orbitdb/zdpuArg5XeMUYwsXT37iuhsSwxJBvuJB4M6FHEdWmmACTkgWq
[]
entry { op: 'ADD', key: null, value: 'hello from ./FOOBAR 1' }
(node:75788) MaxListenersExceededWarning: Possible EventTarget memory leak detected. 11 abort listeners added to [AbortSignal]. Use events.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)
entry { op: 'ADD', key: null, value: 'hello from ./FOOBAR 2' }

run clients (unique port and directory name!)
$ node server.js 13255 FOOBARBAZ  /orbitdb/zdpuAkn4eb7Xdy5UFCvFGEcYd5HDhufhJHWAgYgVMpWK7U5wF
running with port:13255 and ./FOOBARBAZ
clientEventsDb address /orbitdb/zdpuArg5XeMUYwsXT37iuhsSwxJBvuJB4M6FHEdWmmACTkgWq
[]
entry { op: 'ADD', key: null, value: 'hello from ./FOOBARBAZ 1' }
(node:76188) MaxListenersExceededWarning: Possible EventTarget memory leak detected. 11 abort listeners added to [AbortSignal]. Use events.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)
entry { op: 'ADD', key: null, value: 'hello from ./FOOBARBAZ 2' }
entry { op: 'ADD', key: null, value: 'hello from ./FOOBARBAZ 3' }

errors
r➜  saysheep node server.js 13245 FOOBAR  /orbitdb/zdpuAkn4eb7Xdy5UFCvFGEcYd5HDhufhJHWAgYgVMpWK7U5wF
running with port:13245 and ./FOOBAR

(node:78587) MaxListenersExceededWarning: Possible EventTarget memory leak detected. 11 abort listeners added to [AbortSignal]. Use events.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)

node:internal/process/promises:289
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[AggregateError: All promises were rejected] {
  [errors]: [
    Error: Want for bafyreiaeg347tyzk4o5ixpsgsstemg5u5fhyyui6isuqlhm2qvbbwi54eu aborted
        at EventTarget.<anonymous> (file:///home/nihil/Documents/saysheep/node_modules/ipfs-bitswap/dist/src/notifications.js:56:24)
        at [nodejs.internal.kHybridDispatch] (node:internal/event_target:814:20)
        at EventTarget.dispatchEvent (node:internal/event_target:749:26)
        at abortSignal (node:internal/abort_controller:371:10)
        at AbortController.abort (node:internal/abort_controller:393:5)
        at EventTarget.onAbort (file:///home/nihil/Documents/saysheep/node_modules/any-signal/dist/src/index.js:8:20)
        at [nodejs.internal.kHybridDispatch] (node:internal/event_target:814:20)
        at EventTarget.dispatchEvent (node:internal/event_target:749:26)
        at abortSignal (node:internal/abort_controller:371:10)
        at AbortController.abort (node:internal/abort_controller:393:5),
    AggregateError: unable to fetch raw block for CID bafyreiaeg347tyzk4o5ixpsgsstemg5u5fhyyui6isuqlhm2qvbbwi54eu from any gateway
        at TrustlessGatewayBlockBroker.retrieve (file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/trustless-gateway/broker.js:52:15)
        at processTicksAndRejections (node:internal/process/task_queues:95:5)
        at runNextTicks (node:internal/process/task_queues:64:3)
        at listOnTimeout (node:internal/timers:540:9)
        at process.processTimers (node:internal/timers:514:7)
        at async file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/utils/networked-storage.js:182:31
        at async Promise.any (index 1)
        at async raceBlockRetrievers (file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/utils/networked-storage.js:179:16)
        at async NetworkedStorage.get (file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/utils/networked-storage.js:91:27)
        at async BlockStorage.get (file:///home/nihil/Documents/saysheep/node_modules/helia/dist/src/storage.js:68:20) {
      [errors]: [
        Error: fetching raw block for CID bafyreiaeg347tyzk4o5ixpsgsstemg5u5fhyyui6isuqlhm2qvbbwi54eu from gateway https://dweb.link/ipfs/bafyreiaeg347tyzk4o5ixpsgsstemg5u5fhyyui6isuqlhm2qvbbwi54eu?format=raw was aborted
            at TrustlessGateway.getRawBlock (file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/trustless-gateway/trustless-gateway.js:71:23)
            at processTicksAndRejections (node:internal/process/task_queues:95:5)
            at runNextTicks (node:internal/process/task_queues:64:3)
            at listOnTimeout (node:internal/timers:540:9)
            at process.processTimers (node:internal/timers:514:7)
            at async TrustlessGatewayBlockBroker.retrieve (file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/trustless-gateway/broker.js:25:31)
            at async file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/utils/networked-storage.js:182:31
            at async Promise.any (index 1)
            at async raceBlockRetrievers (file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/utils/networked-storage.js:179:16)
            at async NetworkedStorage.get (file:///home/nihil/Documents/saysheep/node_modules/@helia/block-brokers/dist/src/utils/networked-storage.js:91:27)
      ]
    }
  ]
}

Node.js v20.10.0

This is possibly a result of Helia running an "in-memory" blockstore; it attempts to retrieve the block, which is in the orbitdb database but helia no longer stores the block because its blockstorage has been wiped.
To rule this out, try setting up a permanent blockstore.
You can also find out more about block storage on the Helia site.

@haydenyoung thanks for reply, i am indeed using a blockstore as you can see here: https://github.com/sloev/orbitdb-nodejs-local-demo/blob/master/server.js#L34 so that cant be the issue

It looks like the two helia peers can't find each other. I opened a pull request on your repo which uses mdns to quickly discovery local peers. This is working for me. Does it work for you?

yup, it worked fine! thanks!


Replaces pull request #252 as it includes the fix for JSON marshaller on top of that pr

test failure not related to the change in this PR.
Any objections?

@mrietveld could it be similar as on jbpm with json not dealing with long properly?
java.lang.AssertionError: Different collection values on ArrayList.classes
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.assertTrue(Assert.java:41)
	at org.kie.test.util.compare.ComparePair.compareObjectsViaFields(ComparePair.java:276)
	at org.kie.test.util.compare.ComparePair.compareObjectsViaFields(ComparePair.java:190)
	at org.kie.services.client.serialization.JsonRemoteSerializationTest.deploymentDescriptorTest(JsonRemoteSerializationTest.java:75)

jenkins retest this


Add slider gui for services (response is ignored)

Hi,
I was looking for a simple slider GUI but for services, so I created one script from your code.
I simply changed some parts to send a service Request instead of a message. The Response given by the server is ignored
I am creating the pull request in the case this is useful also for others

Yes, the nodes are really similar so probably fuse them together is better

Hi,
The slider_publisher can now call services:

ros1: https://github.com/oKermorgant/slider_publisher/releases/tag/1.1.1
ros2: https://github.com/oKermorgant/slider_publisher/releases/tag/2.0.1

Both versions were submitted for packaging into noetic and foxy.


Added Shortcut for Lookup of Types and Functions

Hey i added an Shortcut for Lookup of Types and Functions, it's now easier to open the docs.

Hi, isn't this effectively the same as CTRL+Left-Click?


image tracking and face tracking on the same app?

Can I have an app with both image tracking and face tracking functionalities, or can I only use one of them? I can see from the docs on the event handling part that both image tracking and face tracking have the arSystem variable, but on the facetracking I can switch the camera and on the image tracking I can't. Are they actually different, can I have both or must I choose to have an image tracking experience or a face tracking experience? Thanks!

You cannot run both at the same time

But can you stop one and start the other, on the same app?

@marcusx2 I haven't tried that, but I think it's possible.

I have both example snippets on the same page. They are not running at the same time, but the video changes from fullscreen when there is only one AR system on the page. When both are present, the video appears at the top of the page, leaving a blank black space at the bottom. Any idea?


Misleading Statement

This sentence "Alice cannot read the blobs inside a container, but she can read, write, and delete containers." is misleading. Being the Owner, Alice has permission to list the Access Keys, which will give her access to the content of the containers. RBAC Reader role might be a better example to illustrate the differences.

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

ID: e0091902-7cb0-8dc3-2a7a-dc644f39a961
Version Independent ID: d2757c88-1bad-8d27-b548-9e04a8ec7fe1
Content: Understand role definitions in Azure RBAC
Content Source: articles/role-based-access-control/role-definitions.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

Hi @zhusulai
Thanks for the feedback. I'll send to the team to review this example.

Hi @zhusulai
I've made some updates to this article that hopefully makes this more complete.
https://docs.microsoft.com/en-us/azure/role-based-access-control/role-definitions#data-operations-example
Let me know if you have any additional feedback.
thanks

@MicrosoftDocs/azure-cxp-triage
#please-close


Grid view

The new handler is at /grid.
Nothing changed except this new page available if you explicit go to /grid.

@Deleplace could you please rebase your PR?

@dgageot done: rebase'd and push --force-with-lease'd


Flaky UTxOIndexSpec coverage checks

Context
Test Case

e.g.
    monitor $ cover 80 (Set.size (UTxOIndex.assets u) > 1)
        "index has more than one asset"

Failure / Counter-example

ci/hydra-build:required

Failures:

 test/unit/Cardano/Wallet/Primitive/Types/UTxOIndexSpec.hs:146:9:
 1) Cardano.Wallet.Primitive.Types.UTxOIndex, Indexed UTxO set properties, Index Selection, prop_selectRandom_one_withAsset
 Insufficient coverage (after 3200 tests):
 92.84% index has more than one asset
 75.46% index has the specified asset
 75.46% selected an entry

 Only 75.46% index has the specified asset, but expected 80.00%
 Only 75.46% selected an entry, but expected 80.00%

 To rerun use: --match "/Cardano.Wallet.Primitive.Types.UTxOIndex/Indexed UTxO set properties/Index Selection/prop_selectRandom_one_withAsset/"

 test/unit/Cardano/Wallet/Primitive/Types/UTxOIndexSpec.hs:148:9:
 2) Cardano.Wallet.Primitive.Types.UTxOIndex, Indexed UTxO set properties, Index Selection, prop_selectRandom_one_withAssetOnly
 Insufficient coverage (after 3200 tests):
 92.84% index has more than one asset
 75.46% index has the specified asset
 24.01% selected an entry

 Only 75.46% index has the specified asset, but expected 80.00%

 To rerun use: --match "/Cardano.Wallet.Primitive.Types.UTxOIndex/Indexed UTxO set properties/Index Selection/prop_selectRandom_one_withAssetOnly/"

 test/unit/Cardano/Wallet/Primitive/Types/UTxOIndexSpec.hs:154:9:
 3) Cardano.Wallet.Primitive.Types.UTxOIndex, Indexed UTxO set properties, Index Selection, prop_selectRandom_all_withAsset
 Insufficient coverage (after 3200 tests):
 92.84% index has more than one asset
 75.46% index has the specified asset
 75.46% selected at least one entry

 Only 75.46% index has the specified asset, but expected 80.00%
 Only 75.46% selected at least one entry, but expected 80.00%

 To rerun use: --match "/Cardano.Wallet.Primitive.Types.UTxOIndex/Indexed UTxO set properties/Index Selection/prop_selectRandom_all_withAsset/"

 test/unit/Cardano/Wallet/Primitive/Types/UTxOIndexSpec.hs:156:9:
 4) Cardano.Wallet.Primitive.Types.UTxOIndex, Indexed UTxO set properties, Index Selection, prop_selectRandom_all_withAssetOnly
 Insufficient coverage (after 3200 tests):
 92.84% index has more than one asset
 75.46% index has the specified asset
 24.01% selected at least one entry

 Only 75.46% index has the specified asset, but expected 80.00%

 To rerun use: --match "/Cardano.Wallet.Primitive.Types.UTxOIndex/Indexed UTxO set properties/Index Selection/prop_selectRandom_all_withAssetOnly/"

Randomized with seed<PHONE_NUMBER>

Resolution

QA

Should be fixed.


Could the tiktoken dependency be optional?

We're deploying into cloudflare, and tiktoken is a large library. We don't particularly need the token error check, so having a way to only optionally include the large library would be great.

sorry but it's a bit tricky to remove and I don't have the time right now to implement & properly test. I do think this check is getting less relevant over time as large context models becomes prevalent, so will rework at some point, but going to close this for now.


Add instructions to configure local elevation service to install guide

Following from @cpitman-gis 15 June 2015 email:
Firstly, you will need to set up a cached Image Service (using ArcGIS for Server 10.3.1) in a web Mercator projection. You will also need to use LERC compression. Please read:
http://server.arcgis.com/en/server/latest/publish-services/windows/available-map-and-image-cache-properties.htm
Once this is set up and working, you need to reconfigure your Portal to point to your new Elevation service. In a web browser, open https://<your_server>/arcgis/sharing and login as an administrator.
Then (in the same web browser), navigate to https://<your_server>/arcgis/sharing/portals, and choose “Self”, then “Update”.
There will be an “Elevation3D” input box, which you should populate with:
[{ "url": "< new_elevation_service_url>", "id": "globalElevation", "layerType": "ArcGISTiledElevationServiceLayer" }]
Then, click “Update Organisation”.

Fixed in Esri\ops-server-config v.next


VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-09T00:26:45.394Z) - OS Version: Windows_NT ia32 10.0.14393

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-09T00:26:45.394Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.1.0

javadebugger
donjayamanne
0.0.2

Steps to Reproduce:

Please provide exact reproducable steps and what seems to be the issue and we can reopen this


Control simulated robot with real leader

What this does
Adds a script control_sim_robot.py in lerobot/scripts that has the same functionality and interface as control_robot.py but for simulated environments.
The script has three control modes:

teleoperate. Teleoperate a simulated robot with a real leader arm. Example:
record. Record episodes in simulation and creates a local dataset of your episodes. You also have the option to push the dataset to the hub by setting the --repo-id option.
replay. replays a collected trajectory in sim.

The dataset created contains more columns related to reinforcement learning like next.reward, next.success and seed.
Simulation environments
Along with the --robot-path argument, the scripts requires a path the configuration file of the simulation environment -- define in lerobot/configs/env.
Example of the configuration file for gym_lowcostrobot:
env:
  name: lowcostrobot
  fps: ${fps}
  handle: PushCubeLoop-v0
  state_dim: 12
  action_dim: 6

  gym:
    render_mode: human
    max_episode_steps: 100000

calibration:
  axis_directions: [-1, -1, 1, -1, -1, -1]
  offsets: [0, -0.5, -0.5, 0, -0.5, 0] # factor of pi

eval:
  use_async_envs: false

state_keys:
  observation.state: 'arm_qpos'
  observation.velocity: 'arm_qvel'

Essential elements:

Name of the environment that defines the gym package to be able to import it correctly.
The environment handle to create the specific task within the package.
Calibration arguments: Necessary for transforming the real leader position to the simulated ones.
State keys (optional): for mapping the different state names to lerobot standard state keys in the LeRobotDataset.

How to test
First install the gym_lowcostrobot environment and add the environment's config file in yaml format.
Test teleoperation:
python lerobot/scripts/control_sim_robot.py teleoperate \
  --robot-path lerobot/configs/robot/koch.yaml --sim-config lerobot/configs/env/mujoco.yaml

Test data collection and upload to hub:
python lerobot/scripts/control_sim_robot.py record  --robot-path lerobot/configs/robot/koch.yaml --sim-config lerobot/configs/env/mujoco.yaml   --fps 40  --root data --repo-id $USER/test_mujoco  --episode-time-s 30  --num-episodes 50   --push-to-hub 1

Replay the episodes:
python lerobot/scripts/control_sim_robot.py replay   --fps 30   --root data    --repo-id $USER/test_mujoco    --episode 0 1 2 3 --sim-config lerobot/configs/env/mujoco.yaml

In the script we save the seed in the dataset which enables us to reset the environment in the same state when the data collection was happening which makes the replay successful.
Finally visualize the dataset:
python lerobot/scripts/visualize_dataset_html.py --repo-id $USER/test_mujoco --episodes 0 1 2 3 4

TODO:

[ ] Test with more simulation environments, brax, maniskill, IsaacLab ...

[ ] Add keyboard control of the end-effector.

Note: You might need to run mjpython if you're using MAC.

Note: The current script requires a real leader in order to teleoperate sim environments. We can add support for keyboard control of the end effector for people who don't have the real robot.

Thanks @marinabar who tested the script on her setup.

I'm noticing quite a bit of the new scripts could be DRY'ed up, since it's rehashing a fair bit of the original control_robot.
I'm curious- what issues are you finding with using control_robot with simulated environments? Maybe there are ways of improving control_robot so it better handles abstractions like simulation. That way there's less code to maintain :grin:

Hey @apockill! You're right, we might be able to find a general solution in control_robot.py, but I feel there are few elements that could make the script ugly.

env vs robot. In control_robot.py reading and writing to the robot is done using only the Robot class. In simulation, we would require an additional environment instance along with the robot that has to also be passed to all the functions. We could maybe modify lerobot/common/robot_devices/control_utils.py for instance and put ifs and elses everywhere to account for that but I think it would add an unnecessary complexity.
There are some smaller details like the dataset you acquire in simulation also has additional labels that we don't have in the real environments like rewards, successes and env seed. Also handling the  frame rate fps on the real system vs. in simulation is different.

So even though the two scripts resemble each other I still think it is cleaner to have them separate. What do you think? If you have some vision of how we can improve on that or merge the two scripts I would be happy to chat or have a look :D


Pass objects from browser to renderer-side at window creation, as argument of "new BrowserWindow()" or ".loadUrl()", to avoid pollution of the global scope

The remote module is really handy, however I don't like to pollute the global scope of the browser-side just to provide some objects to the renderer's remote module, using remote.getGlobal(name).
It would be nice if we can pass some objects from browser-side to renderer-side, either at the BrowserWindow creation or when using .loadUrl().
Implementation example, browser-side:
var somethingIWantToShare = {
  myFunction: function() {
    // some code
  } ,
  ...
  // some properties and methods here
} ;

mainWindow.loadUrl( 'file://' + __dirname + '/html/main.html' , {
  rendererSideName: somethingIWantToShare ,
  anotherRendererSideName: ... // another object to share
} ) ;

Renderer-side:
remote.rendererSideName.myFunction() ;
remote.anotherRendererSideName.myOtherFunction() ;

You can do it in this way:
browser side:
mainWindow.rendererSideName = somethingIWantToShare;

renderer side:
remote.getCurrentWindow().rendererSideName.myFunction();

@cronvel I wrote https://github.com/jprichardson/electron-window for this purpose. It uses the same method as atom. It also keeps track of global window refs so that you don't have to worry about it.

@jprichardson Thanks a lot, I will look into it!

@zcbenz I tried that in v0.36.4 and it doesent work. I basically need to pass an object between two windows.

This approach didn't work for me. Any possible reason?

@meghadev I think some things changed have since 2015 but I can tell you a solution that works for me in Electron 1.6
In the main process:
e.g. index.ts
// typed to 'any', because we add a custom property later
 let window: any = new BrowserWindow({
        height: 500,
        width: 700,
        frame: false,
        resizable: false
    });
.....

 window.custom = {
        'whatever': 'youwant'
    };

Then on the renderer side it's accessible like this:
e.g. index.html
<script>
var electron = require('electron');
var currentWindow = electron.remote.getCurrentWindow();

console.log(currentWindow.custom);

I'm not happy that it breaks the typing of the BrowserWindow and having said that, I'm pretty sure that this isn't best practice.
To use it in JS just remove :any

can we get a 'opened' event to send data please.

+1

@zcbenz , I saw your method it did work, but is there a way where i can pass from render process to main process, I tried the reverse as its, but it did not work

@meghadev I think some things changed have since 2015 but I can tell you a solution that works for me in Electron 1.6
In the main process:
e.g. index.ts
// typed to 'any', because we add a custom property later
 let myWindow: any = new BrowserWindow({
        height: 500,
        width: 700,
        frame: false,
        resizable: false
    });
.....

 myWindow.custom = {
        'whatever': 'youwant'
    };

Then on the renderer side it's accessible like this:
e.g. index.html
<script>
var electron = require('electron');
var currentWindow = electron.remote.getCurrentWindow();

console.log(currentWindow.custom);

I'm not happy that it breaks the typing of the BrowserWindow and having said that, I'm pretty sure that this isn't best practice.
To use it in JS just remove :any

Sir, I used your code. but I face an issue.

Here's how to pass references from main to any process (window or webview, doesn't matter):
In Main:
global.foo = { n: 123 }

Then in any Electron browsing context:
import { remote } from 'electron'

console.log( remote.getGlobal( 'global' ).foo ) // { n: 123 }

You can implement an API this way to pass reference from window to window, through main.
NOTE! This ties up the renderer threads, which means executing a function from main inside a browsing context will lock/sync the main thread to the execution of the browser thread (it's been a while since I've researched, so I'm not sure if things have changed in that regard).

After much tinkering, I've discovered that doing stuff like remote.getGlobal('global').foo = 123 will not work as expected, because the call to getGlobal('global') returns some sort of proxy-like object, and multiple calls to getGlobal('global') can return differing proxy objects, so on subsequent calls the result of remote.getGlobal('global').foo will be undefined. This is always true, for example, when you set a value in one window, then try to read the value in a new <webview>.
To get around this problem, store values in lexical scope of the main process. For example, see https://discuss.atom.io/t/how-to-set-global-variable-of-main-process/24833/12?u=trusktr

@trusktr That's not required, the issue here is that foo was undefined to begin with.
If you instead had
// Main Process
global.foo = 1;

The setters / getters in each renderer would work.  Defining an undefined properties is not proxied through the remote module.  Redefining a defined property is proxied and works as expected

@MarshallOfSound Not sure what you mean, as in my example above foo is defined.
But what I found is the following happens (when done in that sequence for example)
// main process:
// open a BrowserWindow

// in the BrowserWindow,
remote.getGlobal('global').someObj = {...}

// later, in the same BrowserWindow, in a require'd file:
console.log(remote.getGlobal('global').someObj) // undefined (???)

// at any time, in a <webview>:
console.log(remote.getGlobal('global').someObj) // always undefined (???)

The value does not stick around.
Now, if I apply a getter/setter trick in the main process,
let _someObj

Object.defineProperty(global, 'someObj', {
  get() { return _someObj },
  set(value) { _someObj = value }
})

then the value sticks around for all of the above cases that were previously returning undefined:
// later, in the same BrowserWindow, in a require'd file:
console.log(remote.getGlobal('global').someObj) // {...}

// at any time, in a <webview>:
console.log(remote.getGlobal('global').someObj) // {...}

@MarshallOfSound Perhaps a reciprocal remote.setGlobal('someObj', {...}) method would help, which would abstract away all the gotchas.

@trusktr The real thing here is IMO people shouldn't be using remote.  If you don't fundamentally understand how the backing IPC works it's full of raceconditions and footguns.  Plus its performance impact is noticeable as it is backed by synchronous IPC.

From a DX perspective, its so easy to pass objects (instances of classes) around, with their prototypes and functions, instead of having to make IPC wiring. It'd be great if these parts had more polish.

@trusktr That's the thing, remote doesn't magically send objects and classes and prototypes.  It serializes the entire thing (including a custom representation of the prototype) and sends it over syncronous IPC.  It's all faked, it's easy / helpful sure, but it's a huge race condition / performance hog if you use it for anything important in your application.  remote isn't a magical proxy of objects, it's just got really complicated serialization / deserialization that supports prototype chains.  It causes issue every major release and there are constant timing / racing issues.
As I said before, you're better off not using it, yes you sacrifice simplicity, but you gain back control, performance and predictability

It serializes the entire thing (including a custom representation of the prototype) and sends it over syncronous IPC.

Ooof, that sounds like a pain.
What about for example what native window.open does, where we can read/write actual references on a child window? Could it be possible to add a getNativeWindow method to WebContents that can let us do this? This could make things easy. Maybe the processes (parent and child windows f.e.) would need to be sync'd together when doing this (with relevant options, f.e. to turn off OOPIF for <webview>s), but in many cases this may not be a performance concern.

// main process or renderer process 1
data = {"age": 12, "healthy": true}

let win = new BrowserWindow({
        webPreferences: {
          nodeIntegration: true
        }
      });

win.loadFile("public/print.html", {query: {"data": JSON.stringify(data)}});

// renderer process 2
const querystring = require('querystring');
let query = querystring.parse(global.location.search);
let data = JSON.parse(query['?data'])

ref. https://stackoverflow.com/a/59347064/3969942


CORS Error

Need to fix CORS for POST requests on backend.

I think this was raised due to IP error(not whitelisted) for logging into database server and I have fixed it.
Need to check calender's working one more time.


FR: core nodes to permit input and output to properties other than the hard coded payload

Discussed and triaged with @dceejay but again, i am failing to find the supporting information.  Dave listed out all the nodes where this made sense.  I will update this once I locate the preparations/supporting discussion.
The PR proposes that, where it makes sense, we permit the input and output property to be user configurable.
It was determined this would be strictly a msg.xyz only option as this simplifies implementation and promotes the use of msg properties (over context or other potentially mutating values)
The aim is to reduce friction when flows are made in a serial fashion whereby users are not aware they can move or copy the payload to an alternate msg property (using change nodes). The first instinct many users seem to apply is to store the payload in context and then pick this up at the end of the flow. This leads to subtle timing bugs when messages flow in quick succession and async operations are involved.

A Draft PR implementing the majority of Dave's suggestions was raised in #4656
Below are screenshots of before and after for evaluation and agreement before I proceed with tests and doc updates.
Notes:

Split, Join, trigger, Exec, TCP/UDP nodes were not addressed in this initial offering as they have complicated relationships (streaming etc) that can be addressed at a later time (if at all)
Naming the "Property": I went with "Property In" and "Property Out".

Other considerations made were:

"Input Data" and "Output Data"
"Payload In" and "Payload Out"
Please feel free to offer up more suitable suggestions

CSV

Move name to top
Add property in
Add property out

JSON

Move name to top
Add property in
Add property out

YAML

Move name to top
Add property in
Add property out

XML

Move name to top
Add property in
Add property out

RANGE

Move name to top
Add property in
Add property out

FILE

Move name to top
Add property out

HTTP REQUEST

Move name to top
Add property in
Add property out

TEMPLATE

Move Property Out to bottom of form
2. Update icon to match other nodes
3. Update label to clarify what it is for

Looking good so far ! Great work

I'm not yet won over on this.
For some nodes, I can see the benefit of providing more flexibility of what message property they operate on - such as the parsers and range nodes. The template node - I see the benefit, at the cost of reducing the screen space of the main part of the editor dialog (always been wary of adding too many options for that node).
For others... it feels like we're adding more clutter to the node UI to cater for some edge cases. I can understand the desire to have more flexibility and consistency... just want to be careful we don't make it more complicated.
Some stick out to me more than others. For example, the HTTP Request node. In that instance, we already have a field labeled 'payload' (only if the method is GET) which configures what to do with the incoming message payload. Given it now may not be payload, that field makes less sense to be presented in that way. I personally think that's a good example of a node that should not have the property in field at the top - as its a choice you make after you've configured the method and URL. Likewise the 'return' field at the bottom - feels more logic to set what property you want to use and then the format to be used.
This is why I don't think we should have an absolutely rigid approach. It needs to be what makes sense for the individual node.

I'll have more of a play and try to come up with some more constructive suggestions for moving forward on it. We may want to reduce the initial scope rather than try to solve them all in one go.

I agree with Nick on this one.
As much as I like Steve's nodes, how a common node works should be like this:

Input:

msg.payload - essential data that the node will process on
msg.property - optional configuration for the process

Output:

msg.payload - the data resulting from the processing
msg.property - optional data (request status, extra data...)

I see the benefit for some configuration parameters but not for the payload - the payload remains the payload. i.e. Specificationand Byte swap in your buffer-parser node.
As for the FR reason, I come back to the idea of adding a light beginner's course. (mentioned in 4659)
PS: I prefer Name to be at the top.


[plugin|tiling] Window rules

Wondering what people would use this for, except for setting float and different accepted AXRoles.
What kind of filters would you like to see and why.
This feature was originally (and currently just a way to get some applications to work alongside kwm), but some potential ideas can't hurt.

I mostly use it for floating windows depending on their window title.
Some examples from my Kwm rules:
kwmc rule owner="QuickTime Player" name="Screen Recording" properties={float="true"}
kwmc rule owner="Tweetbot" properties={float="true"} except="Main Window"
kwmc rule owner="Trash" properties={float="true"}
kwmc rule owner="Finder" name="Copy" properties={float="true"}

What kind of filters would you like to see and why.

The same as above to make some applications work with chunkwm, most importantly the name and except filters in combination with float.
I would like to see smooth solution for floating every window of an application except for the first one (per space). It's just hard to do if the main window doesn't have a fixed title.
That's it! But I would like to see the system be less prone to errors. With rules in Kwm, having a syntax error often silently breaks everything. That shouldn't be happening.

I have a few games set to float because they have to maintain a certain aspect ratio. Then what happens is KWM would resize them, then the game would resize, ∞

I use it mostly to specify which monitor / space the window is forced to and to set float on some applications which needs to maintain the aspect ration.

# section - window rules

# regex filters (pattern is case sensitive):
#  --owner | -o
#  --name  | -n

# properties:
#  --state | -s
#   values:
#    float
#    tile

chunkc rule --owner "System Preferences" --state tile
chunkc rule --owner Finder --name Copy --state float

Currently, the only thing you can do with rules is force a window to tile regardless of the accessibility role or whatever, and make a window float by default. Windows that have been affected by these rules can still be unfloated / floated manually afterwards.

The same as above to make some applications work with chunkwm, most importantly the name and except filters in combination with float.

The except filter has been removed, but you should be able to get the old behaviour by using a regex pattern for the name filter.

I would like to see smooth solution for floating every window of an application except for the first one (per space). It's just hard to do if the main window doesn't have a fixed title.

I have not yet looked into this, but could be a potential addition in the future.

That's it! But I would like to see the system be less prone to errors. With rules in Kwm, having a syntax error often silently breaks everything. That shouldn't be happening.

There are two errors that we currently check for, missing filters and missing state.
These are done at parsetime, and will print to stdout

I use it mostly to specify which monitor / space the window is forced to

There is no support for this for now, but will probably take a look at adding this in the future.

I someone figures out how to match a title that doesn't contain a string let me know, no idea how to do it with posix regex

https://www.gnu.org/software/findutils/manual/html_node/find_html/posix_002dextended-regular-expression-syntax.html

I someone figures out how to match a title that doesn't contain a string let me know, no idea how to do it with posix regex

This seems to be particularly hard in POSIX regex, so I'll probably have to add a separate except filter.

@choco

I someone figures out how to match a title that doesn't contain a string let me know, no idea how to do it with posix regex

This seems to be particularly hard in POSIX regex, so I'll probably have to add a separate except filter.

Added except for this. It performs a regex match on the pattern, and simply asserts that no matches was found (pretty sure this is how it worked in kwm).

Added except filter because of this. It performs a regex match on the pattern, and simply asserts that no matches was found (pretty sure this is how it worked in kwm).

Works well for my use case 👍

Looks like the window rules patterns you implemented works fine for my usecase too. Thanks! 👍


[3.1] Revert "Disallow conversions from IUOs to extistentials."

Revert 7c665c105b30a5f37493731293bf6e36d53d0b25 because it results in a
much worse source compatibility regression than the one it was intended
to fix.
The problem is that it results in force-unwrapping IUOs, which can lead
to cases where we prefer type-checking solutions that choose methods
over properties where we used to choose properties. So for example a
collection literal like [x] will result in a collection with a method
named 'x' in it rather than a property named 'x'.
I'll be looking at other solutions which fix both the original
compatibility regression as well as retain the behavior in the new test
added here.
Fixes SR-3715 and rdar://problem/30176166.
(cherry picked from commit c57dc58dab7018861bac9c5d4c467e78e99c241f)

@swift-ci Please test

@swift-ci Please test


[Android] Remove tar.py and the pack_* targets from gyp

pack_xwalk_core_library was created years ago when we had our own
Cordova 3 fork and suggested people extracted the xwalk_core_library
tarball into its checkout.
This has not been the case for a long time, and our release build system
does not use any of those pack_* targets to create its zip files.
Anyone wishing to create a tarball out of the directories in out/ can
do so manually.

@lincsoon @heke123 @wuhengzhi @darktears

Should pack_xwalk_core_library_src also be removed?

As far as I can see, pack_xwalk_core_library_src is an action that's being removed as part of pack_xwalk_core_library.

Oh, LGTM, thanks.

LGTM


Creating links to a specific point on the Wiki

This is less of an issue and more of what I found on creating a markup link to a specific spot on the wiki page (rather than to the top of the wiki page) for the information of anyone interested.
Add this to the spot you are linking to (updating the name as needed).  The tags don't need to surround any text and in fact will remove the formatting if you do.

The link looks like this (changing the name of the tag or url of the wiki page as needed):
[Link Text](https://github.com/OSSHealth/ghdata/wiki/Queries#Location)

Actually, the above doesn't show the markup (should have expected that) I'll try to get it to display.

FYI: GitHub automatically does this for any headings. For example: https://github.com/OSSHealth/ghdata/wiki/Queries#contributors-for-a-project.
If you hover over the heading you'll see a little paperclip icon. Clicking on that will update the URL in your browser with the link to that heading.

Ok thanks


Custom view show animation

Sometimes custom view's animation will be quickly that is different from default.

I'm not sure if this is a bug or a feature request.

I'am trying to modify like this  =========
window.setLayout(width <= 0 ? ViewGroup.LayoutParams.MATCH_PARENT : width, mBuilder.view != null ? ViewGroup.LayoutParams.MATCH_PARENT : ViewGroup.LayoutParams.WRAP_CONTENT);
==========================this is the custom view layout===============================

<com.kennyc.bottomsheet.CollapsingView
android:id="@+id/collapsingView"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:layout_alignParentBottom="true"
android:background="@android:color/transparent"/>

You don't need to add a CollapsingView to your custom layout. It will automatically be wrapped when created https://github.com/Kennyc1012/BottomSheet/blob/master/library/src/main/java/com/kennyc/bottomsheet/BottomSheet.java#L212

I know, i just find the animation of bottom show is not correct sometimes,It's a bit unstable.

Any chance of seeing an example, a video perhaps?

how can i solve it?


Parsing of CDDL fails unexpectedly

Parsing of webdriver-bidi CDDL file fails.

Download https://github.com/w3c/webdriver-bidi/actions/runs/12085975317/artifacts/2254085949
Extract cddl.zip.
Run npm install cddl
Run node and type import('cddl').then(({parse}) => parse('cddl/all.cddl'))

Yields this error:
> Uncaught:
Error: cddl/all.cddl:135:34 - error: group identifier expected, received "{"Type":"}","Literal":"}"}"

session.AutodetectProxyConfiguration = (
                                  ^
                                  |

    at Parser.parserError (file:///path/to/node_modules/cddl/build/parser.js:731:16)
    at Parser.parseAssignments (file:///path/to/node_modules/cddl/build/parser.js:52:24)
    at Parser.parse (file:///path/to/node_modules/cddl/build/parser.js:721:32)
    at parse (file:///path/to/node_modules/cddl/build/index.js:5:19)

Note: The cddl crate parses the file without issues.

@caugner thanks for flagging this issue. I am not sure when I get the time to look at this so if this is a pressing issue for you, any support would be appreciated.

I don't know about the provenance of cddl.js, but the old cddl tool needed a little help with two rules:
script.RegExpRemoteValue = ({
        ? handle: script.Handle,
        ? internalId: script.InternalId,
    }) .and script.RegExpLocalValue
script.DateRemoteValue = ({
        ? handle: script.Handle,
        ? internalId: script.InternalId,
    }) .and script.DateLocalValue

You can get this help from the new ool:
cddlc -tcddl all.cddl

Grüße, Carsten

Oh, there is a stray semicolon in line 127; maybe cddl.js is allergic to empty comments?


feat: Add appsync arguments to support appsync security features

Description
Add support for the new AppSync GraphQL controls for query depth, run complexity and introspection.
Motivation and Context
This enables users to utilise the new AppSync features that improve the security of their AppSync APIs.
Breaking Changes
This increases the minimum AWS provider version to 5.37, as this was the terraform provider that implemented these controls.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects
[x] I have executed pre-commit run -a on my pull request

@antonbabenko could I request your review on this PR please?

@antonbabenko - is there anything else you would need me to do on this PR before it can be merged? Thanks.

@antonbabenko - done. Would you mind re-reviewing please?

This PR is included in version 2.5.0 :tada:


Add the script for Money.

Add the service to give money. There is no need to create a new branch. Please send me pull request by today if possible (Feb 12).

Dan's code already added money? no need to add branch, you mean to add it directly to your ADAPTSbranchv2?

I just checked, award-dollar.service.ts is already under survey folder, what else you need?

Does this code give extra money on day 1?

Nope, I just added service, Dan hooks up awardDallar in userprofile/localstorage, he can answer this question more
accurately.
Liying

From: Mashfiqui<EMAIL_ADDRESS>Sent: Wednesday, February 12, 2020 1:43 PM
To: StatisticalReinforcementLearningLab/SARAv2<EMAIL_ADDRESS>Cc: Huang, Liying<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [StatisticalReinforcementLearningLab/SARAv2] Add the script for Money. (#10)
Does this code give extra money on day 1?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FStatisticalReinforcementLearningLab%2FSARAv2%2Fissues%2F10%3Femail_source%3Dnotifications%26email_token%3DADXW5ZKOJDG6ZC45WX5IYJDRCQ7NZA5CNFSM4KT4FUPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELR5PPA%23issuecomment-585357244&data=02|01|lxh37%40psu.edu|7f83861bb7114aa9695708d7afeb7b11|7cf48d453ddb4389a9c1c115526eb52e|0|1|637171298231180182&sdata=g2Afp%2FOEVdgzhYhWeWz0ToUFQynkwTzpp3PFb%2Btt8nY%3D&reserved=0, or unsubscribehttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADXW5ZKTG3TCG6JDXEDVSPTRCQ7NZANCNFSM4KT4FUPA&data=02|01|lxh37%40psu.edu|7f83861bb7114aa9695708d7afeb7b11|7cf48d453ddb4389a9c1c115526eb52e|0|1|637171298231190177&sdata=mdu1xzjcULaXPt2YUp9UM3dgljzK034cwi%2FsK3ibgEY%3D&reserved=0.

Liying, I assigned it to myself. I found the "award-dollar.service.ts."

This issue has been closed. Now "award-dollar.service.ts" pays 1 for each three-day streak.


Looping in a state when there's no explicit jump

No more of this nonsense
10:
  [...]
  => 10

Agreed

https://github.com/acbabis/factorio-state-machine/pull/11


Jv

What article on docs.github.com is affected?

What part(s) of the article would you like to see updated?

Additional information

On Wed, 9 Jun 2021 at 5:28 PM docubot @.***> wrote:

Closed #7219 https://github.com/github/docs/issues/7219.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/issues/7219#event-4864693902, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AUEDMN7GENOSE7IVSC42DG3TR5M65ANCNFSM46L6AEKA
.

--
Ihsan Kapahi

Thanks


chore(package): update @types/jquery to version 3.2.8

Closes #189

Codecov Report

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

@@          Coverage Diff          @@
##           master   #270   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         133    133
  Lines        3019   3019
  Branches      652    652
=====================================
  Hits         3019   3019

Continue to review full report at Codecov.

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

Seems like there are a couple type errors when building and testing (https://travis-ci.org/coveo/react-vapor/builds/253703674?utm_source=github_status&utm_medium=notification)


Fix clippy on main by boxing large error variant

I don't really understand why this only happens on windows clippy and not on linux too, but as usual, boxing the error variant fixes it.
Fixup for #3585

PathBuf is a byte larger on Windows, which is why this wasn't triggered anywhere else.

Wow

@ibraheemdev is a fount of knowledge


Loader doesn't recognise 64 bit XCOFF binaries for POWER9.

Issue Description
When attempting to open an XCOFF-64 binary (compiled for the Power9 architecture, on AIX, using IBM's compiler suite), GHIDRA does not recognise the binary at all and I cannot seem to find a "sane" way to load the binary.
To Reproduce
Try open the linked file in GHIDRA, it doesn't seem to recognise the format at all. This is just one random binary that was causing issues.
Link to file: https://ufile.io/n4u42uom
(Apologies for the crap filehost link, I couldn't find a way to attach the binary - I'll upload elsewhere later).
Expected Behaviour
GHIDRA would handle this as it has good PPC support and an XCOFF loader.

I don't think we actually have an XCOFF loader, despite having some of the file format implemented.  Duplicate of #385.

I am going to close this as a duplicate.  If you want to carry on the conversation, let's use issue #385.  Thanks!


GSoc all frontend project

@mihir2226 would you please give some context of opening this issue?

@kaiwalyakoparkar I seen kunals post on linkdin and I also following frontend developer roadmap on community classroom site. I am searching for projects on which I work or learn to work.

@mihir2226 sorry but I am still not able to understand the purpose of this issue 😓

@kaiwalyakoparkar he said to come to repo and make PR here. I think I got wrong 😅

@mihir2226 oh, no problem. You can open a new issue and pr if you want to suggest some open source projects, jobs boards or communities and add them to the current list

Ohk


Error handling enhancement

We must throw a expection if a error occurs. In this case, who called it can make a decision based on the errror!

Added raise statements on functions:

add_utterance
delete_intent
delete_utterance


README.md google example is working with errors

Hi mate,
I have tried this with the very simple example which you have described.
Thys, under a new site.http file I coppied that lines and I compiled.
set URL http://google.com
do GET /
show $HTML
The output is showing the destination HTML page but also some errors as follow:
Traceback (most recent call last):
File "httplang.py", line 7, in
httplang.run(sys.argv[1])
File "/whatever/Desktop/HTTPLang/httplang/httplang.py", line 8, in run
parse.parse(line)
File "/whatever/Desktop/HTTPLang/httplang/parse.py", line 7, in parse
if line[0] in utils.keywords:
IndexError: list index out of range
Also, had you any thoughts to port it to python3?
Thanks.

Ahhh! I see the error here and I am going to fix it.  forgot to take out
blank lines haha. As for the porting to Python3 I had not really thought
about it, but I may at some point.
On Mon, May 25, 2015 at 5:25 PM, Johnny STANESCU<EMAIL_ADDRESS>wrote:

Hi mate,
I have tried this with the very simple example which you have described.
Thys, under a new site.http file I coppied that lines and I compiled.
set URL http://google.com
do GET /
show $HTML
The output is showing the destination HTML page but also some errors as
follow:
Traceback (most recent call last):
File "httplang.py", line 7, in
httplang.run(sys.argv[1])
File "/root/Desktop/HTTPLang/httplang/httplang.py", line 8, in run
parse.parse(line)
File "/root/Desktop/HTTPLang/httplang/parse.py", line 7, in parse
if line[0] in utils.keywords:
IndexError: list index out of range
Also, had you any thoughts to port it to python3?
Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/Max00355/HTTPLang/issues/1.


AndroidManifest.xml not found in the repo

Hey Boris, Could you please remove the *.xml from the .gitignore and commit the android version. All the xml files are ignored including the layouts and menus

Thanks fixed and updated, here is the commit link https://github.com/googlesamples/android-classyshark/commit/9a276c92a9ff51538b3f92880b2db42d1a7ab34e


Fix uconvert

Codecov Report

Merging #60 (7890682) into master (7794c99) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   85.77%   85.77%
=======================================
  Files          21       21
  Lines         766      766
=======================================
  Hits          657      657
  Misses        109      109

Impacted Files
Coverage Δ

src/grids/units.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Adding the usage of in use

This is a commonly used phrase which some write as in-use. This cannot be separated with a hyphen as we do not use it as a compound adjective. I guess what I am proposing is correct according to the standards. If not, please correct.

Hi, Is this PR reflected on the UI of the MS Style Guide [1]? Currently, I cannot find it.
[1] https://docs.microsoft.com/en-us/style-guide/a-z-word-list-term-collections/i/input


Add Intermittent Failure feature

This will allow a special return code (85) to trigger a new status
in Civet (Intermittent Failure).

Figuring out unittests for Civet was fun =D

@permcody @aeslaughter Changes can be seen on https://rod.inl.gov


Sign In to iCloud

When this plugin is added to a Cordova project and run on iOS8, the iPhone/iPad will throw up a prompt on app startup asking for the Apple ID password.
"
Sign In to iCloud
Enter the Apple ID password for
(your account)
[password]
Cancel | OK
"
Removing the plugin stops this occurring, adding the plugin back in makes it happen.
"cordova plugin add cordova-plugin-x-socialsharing"

And what is the problem?


Retire ECIP-1049 network

PR description
With the withdraw of ECIP-1049 we can remove the network meant for testing.
This PR also removes Keccak pow/hasher but I'm good with keeping those.

@macfarla I continued syncing an ETC mainnet full bonsai node and I started to full sync Mordor and ETH mainnet bonsai nodes. All without issues after ~2 hours


Di Runtime Compiler Definition

Seems that the current CompilerDefinition class has not been maintained with changes that have been introduced in the RuntimeDefinition. This results in different class definitions generated by the CompilerDefinition than those generated by the RuntimeDefinition. To resolve this I have created RuntimeCompilerDefinition which extends RuntimeDefinition and adds the functionality of CompilerDefinition. It has been designed to be used as a drop in replacement for the CompilerDefinition, and as such should be usable any where that was previously been used.

@nickpeirson GO FOR IT!
If you can do this by 9 March 2015, we can include it in 2.4. :)

Done. I did have to make one tweak to maintain BC, which I'm not sure about.
In summary, there's a test to see if CompilerDefinition::hasMethodParameters() returns false when the class doesn't exist. However in RuntimeDefinition::hasMethodParameters(), which is where the implementation now comes from, throws a ReflectionException in this case. I've overridden the method in CompilerDefinition to catch the exception and return false, which maintains BC, but my preferred solution would be to change the test and be consistent with RuntimeDefinition.
Maintaining BC seemed to be the path of least resistance, but I'm happy to switch it over if there's a consensus on the small BC break.

Travis is showing failed, but it looks like the same failures as on develop, but shout if I missed something that I need to fix.

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html
New issue can be found at: https://github.com/zendframework/zend-di/issues/18


which sdk needed ?NETSDK1045

Hello , vs playing weird , it doesn't use the latest sdk ,
this error showing in my error list;
NETSDK1045
Error	NETSDK1045	The current .NET SDK does not support targeting .NET 7.0.  Either target .NET 6.0 or lower, or use a version of the .NET SDK that supports .NET 7.0.	HybridWebView	C:\Program Files\dotnet\sdk\6.0.400\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.TargetFrameworkInference.targets	144

here's my info of  dotnet --list-sdks
6.0.400 [C:\Program Files\dotnet\sdk]
7.0.302 [C:\Program Files\dotnet\sdk]
8.0.100-preview.4.23260.5 [C:\Program Files\dotnet\sdk]

VS doesn't show (install missing sdk -workloads message )

and here's the dotnet list package
output :
Project 'MauiCSharpInteropWebView' has the following package references
   [net7.0-android31.0]: No packages were found for this framework.
   [net7.0-ios15.4]: No packages were found for this framework.
   [net7.0-maccatalyst15.4]: No packages were found for this framework.
   [net7.0-windows10.0.19041]: No packages were found for this framework.
Project 'HybridWebView' has the following package references
   [net7.0]: No packages were found for this framework.
   [net7.0-android31.0]: No packages were found for this framework.
   [net7.0-ios15.4]: No packages were found for this framework.
   [net7.0-maccatalyst15.4]: No packages were found for this framework.
   [net7.0-windows10.0.19041]: No packages were found for this framework.
Project 'MauiReactJSHybridApp' has the following package references
   [net7.0-android31.0]: No packages were found for this framework.
   [net7.0-ios15.4]: No packages were found for this framework.
   [net7.0-maccatalyst15.4]: No packages were found for this framework.
   [net7.0-windows10.0.19041]: No packages were found for this framework.

BTW I enabled preview sdks in vs settings :

solutions I tried ;
dotnet workload update maui
updated .
tried to remove sdk& runtimes and reinstall them
-I tried fixing vs with vs tools (repair)
removing golbal.json file
I'm not comfortable with  the <TargetFrameworks Condition> in the solution.
I did :
dotnet restore and
dotnet workload restore   too no luck

ugh , I went with the trouble of removing the entire vs then reinstalling it , I'm intermediate level but vs sometimes playing weird for me , wish we had dev home before and a button on github project that take configuration easily, step by step interactive tutorials easily  , lol

Hmm, good question. I've been using a 7.0.x SDK on my machine (7.0.300-preview.23179.2 to be precise).
I'm seeing if adding a global.json file will help with this: https://github.com/Eilon/HybridWebView/pull/23/files
You could try adding the file locally to the root of the repo and see if that helps.

Hi, this is a bit out of date by now, but the repo has been updated to .NET 8.0 and I added a global.json to set the .NET SDK version as well. Hopefully that resolves any issues.


Can not open large files

VSCode Version:1.7.1
OS Version:Debian Testing

Steps to Reproduce:

Generate a 50mb file, about 2000000 lines

$:bash
$:i= 0
$:while [ $i -le 3000000 ]; do echo "asdfghjjklooiyerqwuhgsdjg" >>hello; i=$(($i + 1)); done

Open a 50mb file

Closing as duplicate of #15393


Display a warning if parallel=True is set but not possible.

This patch makes it such that if a user:

decorates a function with parallel=True but the function
does not contain parallel semantics
has the environment variable NUMBA_WARNINGS set to non-zero

then a warning is displayed referring to the decorated function
informing the user that the parallel tranforms have failed.
Closes #2461.

Codecov Report

Merging #2493 into master will increase coverage by <.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master    #2493      +/-   ##
==========================================
+ Coverage   86.54%   86.54%   +<.01%
==========================================
  Files         316      316
  Lines       61458    61484      +26
  Branches     6772     6774       +2
==========================================
+ Hits        53189    53212      +23
- Misses       7198     7199       +1
- Partials     1071     1073       +2


*: remove namespace from EtcdClusterRef

ref: #1770
Part 1/2 of #1785
EtcdClusterRef has been simplified by removing the namespace field. The namespace for the reference cluster is always assumed to be the same namespace as the restore-operator.
Support for restoring from an EtcdCluster in a different namespace can be added later.
/cc @hongchaodeng

LGTM


actions: bump node 20.8.1

Applied mkpr on .github/workflows/*.yml
sed s/20.8.0/20.8.1/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
167

Relevant Lines:
174

💛  - Coveralls


refactor: enable AddClosureVoidReturnTypeWhereNoReturnRector to add void return on closure

Description
enable AddClosureVoidReturnTypeWhereNoReturnRector to add void return on closure
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Ready to merge 👍

@paulbalandan thank you for the review, let's merge ;)


Update dynamodb to 2.15.33

Updates software.amazon.awssdk:dynamodb from 2.15.29 to 2.15.33.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk", artifactId = "dynamodb" } ]

labels: library-update, semver-patch

@saeltz looks like we need to disable the settings that mandates up-to-date branches for merging

@joroKr21, can't Scala Steward be configured to update the branch automatically as dependabot would do?

Not if we run it once a week 😄

What about using something like this https://github.com/marketplace/actions/auto-update?

That could work - but I suspect it would mean for N update PRs running the build N + (N - 1) + ... + 1 = N * (N + 1) / 2 times.
And the alternative is not so bad - I think it's perfectly fine to disable the up-to-date requirement for a library.
If there is a semantic merge conflict we will find out during the publish step that runs on merge.

@JannikArndt, you're an Admin. Could you please allow merging non-up-to-date branches?

Done ✅ (although you @saeltz have admin rights as well. You should be able to change it here: https://github.com/moia-dev/scynamo/settings/branch_protection_rules/15022101)


Added warning message when registering an existing email

#152

Nice PR. Checked the code and everything looks good 💯


Shaka repeats download of segment on adaptation, video and audio go out of sync

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
v4.1.1
Can you reproduce the issue with our latest release version?
Yes
Are you using the demo app or your own custom app?
Demo App
What browser and OS are you using?
GoogleChrome, Firefox, Ubuntu
I have a media content with video and audio encoded separately and different rates. During its streaming,  sometimes when an adaptation event occurs and bitrate is changed , shaka is repeating the download and playback of the last video segment, with a different rate, that is leading video and audio out of sync.
I've experienced this behaviour on shaka player demo and on a custom application using v4.1.1 and v3.4.0 on chrome and firefox, and changing the bandwidth using the browser network tools.  I also made changes in the main playlist to be similar to a hls main playlist example provided by apple at https://devstreaming-cdn.apple.com/videos/streaming/examples/img_bipbop_adv_example_ts/master.m3u8, but it didn't solve the problem.
Link to the main playlist of the encoded video:
https://storage.googleapis.com/akiry-public/video-and-audio-4s/upload_41af1db0-eb71-11ec-85da-710193d27b0f.m3u8
Screenshot of shaka player demo

@joeyparrish can you review it? Thanks!

I can reproduce the issue.  I'll take a closer look at the content.  I suspect that this is a problem with your encoding, but I'll try to verify.

Okay, I understand the problem now.  This would occur with any content whose video segments or audio segments are not all aligned across playlists.
For example, the first four segment lengths across 3 different streams:

x
360p
720p
1080p

seg 0
4.000
4.066
4.000

seg 1
4.000
4.033
4.000

seg 2
4.000
3.966
4.000

seg 3
4.000
4.000
4.000

Switching between 360p and 1080p would be fine, but between 360p and 720p or 720p and 1080p would trigger the issue.
This appears to be a bug in StreamingEngine, which doesn't account for adaptation.  On adaptation, just as on seek, we need to ask MediaSourceEngine to reset the timestamp offset, to ensure the new content is correctly placed on the timeline.

We've had to turn off ad stitching and we think this issue is the root cause. Just wanted to add my voice to the issue, and it's pretty serious for us.

Thanks.  I bumped the priority and tentatively assigned the issue to @JulianDomingo.  I'll take it back if Julian doesn't have the bandwidth for it.

We could probably help with providing test streams if needed (or using yospace hls cmaf reference streams should work).


Leaflet 1.9.3 error - can't access property "properties", layer.feature is undefined

This issue was introduced with Leaflet 1.9.3 and affected bindPopup and bindTooltip of L.esri.featureLayer, as described here: https://github.com/Esri/esri-leaflet/issues/1348
It was fixed by https://github.com/Esri/esri-leaflet/pull/1350
but it also affects L.esri.Cluster.featureLayer, as you can see in this sample: https://codepen.io/PedroNGV/pen/QWBOpJd
TypeError: can't access property "properties", layer.feature is undefined
https://github.com/Esri/esri-leaflet/issues/1348#issuecomment-1396998637

@gavinr could you apply the same fix I did for L.esri.FeatureLayer and see if that fixes this for cluster as well?

The fix for this was released in v3.0.1.


Refactor tweaks

This is technically a breaking change (removing a parameter from State::hops()), however prior to version 0.11.0 Trippy has no official public API so there is no need to call this out in the CHANGELOG.


Select only parent node

I want in jstree with checkboxies to select only parent node how can i do that. There is any way?

Read the docs: http://www.jstree.com/api/#/?q=three_state&f=$.jstree.defaults.checkbox.three_state

thanks


Allow configuring the fragment creator

Would fix #29 but I would love if @developit could take a look and let me know if this works as intended.

yup, this works for me!


Add npm audit hook to CI

We should keep our libraries secure from known bad packages. We can use npm audit in our CI as a step towards this goal.

We can't actually use this yet  unless we switch off of yarn :(

There is a command call yarn audit to use for this now.

fwiw, my current project hasn't found yarn audit to be very usable because:

It fails an any flagged dependency (even it's an info warning, and you've set --level critical)
It fails on deeply nested dependencies that would require multiple PRs in multiple repos to solve

So the effort involved in this would be:

Adding a wrapper that ignores exit codes that aren't >= --level (or a PR to change Yarn's behaviour!)
Constantly rabbit-holing through PRs in multiple 3rd party repos
Also adding a 💩-list to the wrapper that unblocks CI while the above PRs churn

So I'm not saying it's useless, but it's not a 🎁 killer feature 😞


Fixed issue ! #422

Hello, PA
issue fixed - #422

I just change only feedback button position..
How should this be tested?

[x] Test A
[x] Test B

Mandatory Tasks

[x] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@YadavAkhileshh

Thanks