What is our default CI service?

With the introduction of #1172, we assumed that GitHub Actions is our default CI service. I love the new CI template. It's long overdue and something I've missed from Suspenders over the years. I would challenge the assumption that GitHub Actions is our default CI service. I lean towards CircleCI as a default. This is a personal preference, but we expect Suspenders to represent thoughtbot's opinion. What is thoughtbot's default CI service?

Thank you for opening this. For context, I defaulted to GitHub Actions to keep parity with Rails in an effort to keep Suspenders lean and easier to maintain.
I think it would help to understand why we currently use CircleCI on other internal projects. Is it because, at one point, GitHub Actions either did not exist, or was not as effective as CircleCI? What does CircleCI offer that GitHub Actions does not?
One benefit to using CircleCI is that the repository does not need to be hosted on GitHub.

The main reason for why we started standardizing around Github Actions for pipelines in general on Mission Control was that we found it much more likely for developers to investigate build failures when it was all in the same UI.
For example, when a non-test step would fail on another pipeline, such as AWS code build, or CircleCI, people tended to immediately ping the folks who created the pipelines, and asked them to take a look, even if the issue wasn't related to the pipeline, and was something that was in their change.
On the other hand, having it all in Github Actions, and in the same UI, made it significantly more likely that a developer would look at it, and probably resolve the issue on their own, without being gated by waiting on someone else, speeding up the feedback loop.
I don't have statistics or numbers to back this up, but that was the general feeling we noticed.
And granted that is more for continuous integration (build steps) and deployments, not CI in the sense of testing, which is what I think this issue is more about, but I think having a few things already in one place made it easier for other people to keep the tests in the same location as well.
And especially for things like dependecies for deployments, i.e. I want to push, run tests, then deploy if tests pass, is much simpler to do in one pipeline, rather than orchestrating between multiple.

I agree. Suspenders should diverge from Rails as little as possible. This saves us time and effort with future work. Suspenders should also automate what we're doing instead of what we'd like. If we are using GitHub Actions as our current default, then let's reflect that here.
Currently, Suspenders has a CI generator that generates a CircleCI configuration.
The handbook also has explicit references to CircleCI. As does the Playbook.
The only reference I could find to the decision to move to CircleCI is from 2015.
I have no strong opinions about what service we use. My only opinion is that Suspenders should reflect current thoughtbot practices.

The handbook also has explicit references to CircleCI. As does the Playbook.
The only reference I could find to the decision to move to CircleCI is from 2015.

Good find! Thank you for sharing those. I think this issue could continue to serve as a conversation around this topic.

Now that I've had a chance to sleep on these comments, I've had more thoughts.
How do we change a standard once we've set one?
Do we see where the prevailing wind of the team takes us and update things from there?
Does the existence of a standard hold us back from being more experimental?
These questions are outside this issue's scope, but I wanted to share them.

How do we change a standard once we've set one?

Pull requests, either against the Playbook or, more practically, against Suspenders. Typically the Playbook PR comes after the Suspenders one.

I had another "shower thought" about this subject today.
I looked to see what the Rails team's default CI service was.
Even before the recent changes, they used GitHub Actions.
If we wanted to keep parity, this would be a clear indicator.
The main sticking point for me is why we use a particular service; apart from that, I have no strong opinions.

@purinkle now that https://github.com/thoughtbot/handbook/pull/3804 has merged and Suspenders has been released, do you feel comfortable closing this issue?


DBZ-2637 Enhance Postgres validation checks

DBZ-2637 Enhance Postgres validation checks

add wal_level config check
add LOGIN and REPLICATION roles check
add replication slot is already in use check

closes https://issues.redhat.com/browse/DBZ-2637

@rk3rn3r Applied, thanks!

Hi, I've noticed that this change seems to break Debezium on AWS RDS where some roles have non-standard names e.g. rds_replication instead of replication

@rafalwytrykus Could you please raise a Jira issue?

thank you @rafalwytrykus! I will create the Jira issue for it

@rafalwytrykus I checked the query on Amazon RDS Postgres and it looks like it's working. Can you maybe review my comment in the task and validate it's the same for you? https://issues.redhat.com/browse/DBZ-2800

@rk3rn3r Thank you for checking it. I am not sure about classic RDS but for Aurora the master user does not have replication role:
postgres=> SELECT rolcanlogin, rolreplication FROM pg_roles WHERE rolname = current_user;
 rolcanlogin | rolreplication
-------------+----------------
 t           | f
(1 row)

master is however a member of rds_replication:
postgres=> \du
                                                                  List of roles
       Role name       |                         Attributes                         |                          Member of
-----------------------+------------------------------------------------------------+-------------------------------------------------------------
 full_sch_udm_data     | Cannot login                                               | {}
 full_sch_udm_data_pii | Cannot login                                               | {}
 master                | Create role, Create DB                                    +| {rds_superuser,rds_replication}
                       | Password valid until infinity                              |
 rds_ad                | Cannot login                                               | {}
 rds_iam               | Cannot login                                               | {}
 rds_password          | Cannot login                                               | {}
 rds_replication       | Cannot login                                               | {}

The Postgres version I am using is 11.6

@rafalwytrykus @rk3rn3r I have ran into the same issue on classic RDS, running PostgreSQL 12.4-R1:
SELECT rolcanlogin, rolreplication FROM pg_roles WHERE rolname = current_user;
 rolcanlogin | rolreplication
-------------+----------------
 t           | f
(1 row)

and the master is a member of rds_replication:
\du postgres
                                List of roles
 Role name |          Attributes           |            Member of
-----------+-------------------------------+---------------------------------
 postgres  | Create role, Create DB       +| {rds_superuser,rds_replication}
           | Password valid until infinity |

Oddly enough, it looks like rds_replication itself doesn't have rolreplication=true
SELECT rolcanlogin, rolreplication FROM pg_roles WHERE rolname='rds_replication';
 rolcanlogin | rolreplication
-------------+----------------
 f           | f
(1 row)

I've downgraded to 1.3.1 and can confirm that the connector works given my current setup described above ^

Thank you @rafalwytrykus @RemingtonReackhof for checking.
Can you share the output of SELECT * FROM pg_roles WHERE rolname = current_user; for the master user.
When I checked on a fresh AWS RDS Postgres I got (for master user):
postgres=> SELECT rolcanlogin, rolreplication FROM pg_roles WHERE rolname = current_user;

 rolcanlogin | rolreplication
-------------+----------------

 t           | t

Happy new year @rafalwytrykus @RemingtonReackhof !
And sorry to bother you again again.
Can you share the output of SELECT * FROM pg_roles WHERE rolname = current_user; for the master user so I can see the complete list of available roles for the current user.
Thx again!

Happy new year @rafalwytrykus @RemingtonReackhof !
And sorry to bother you again again.
Can you share the output of SELECT * FROM pg_roles WHERE rolname = current_user; for the master user so I can see the complete list of available roles for the current user.
Thx again!

See DBZ-2800 for what we'll do for 1.4.0.Final: add the RDS specific role name to the check, and also make it more lenient (i.e. log the error about missing roles, but don't make it an actual validation error).

See DBZ-2800 for what we'll do for 1.4.0.Final: add the RDS specific role name to the check, and also make it more lenient (i.e. log the error about missing roles, but don't make it an actual validation error).

Thank you @rk3rn3r and Happy new year!
This is the output from Aurora 11.6:
=> SELECT * FROM pg_roles WHERE rolname = current_user;
 rolname | rolsuper | rolinherit | rolcreaterole | rolcreatedb | rolcanlogin | rolreplication | rolconnlimit | rolpassword | rolvaliduntil | rolbypassrls | rolconfig |  oid
---------+----------+------------+---------------+-------------+-------------+----------------+--------------+-------------+---------------+--------------+-----------+-------
 master  | f        | t          | t             | t           | t           | f              |           -1 | ********    | infinity      | f            |           | 16399
(1 row)

Thank you @rk3rn3r and Happy new year!
This is the output from Aurora 11.6:
=> SELECT * FROM pg_roles WHERE rolname = current_user;
 rolname | rolsuper | rolinherit | rolcreaterole | rolcreatedb | rolcanlogin | rolreplication | rolconnlimit | rolpassword | rolvaliduntil | rolbypassrls | rolconfig |  oid
---------+----------+------------+---------------+-------------+-------------+----------------+--------------+-------------+---------------+--------------+-----------+-------
 master  | f        | t          | t             | t           | t           | f              |           -1 | ********    | infinity      | f            |           | 16399
(1 row)

Thank you @rafalwytrykus !
I took a closer look at a AWS RDS Postgres installations today. When I understand it correctly, the debezium connector user needs to have one of these roles assigned: rds_superuser, rdsadmin or rdsrepladmin in order to have the rolreplication rights that allow starting a logical replication. Correct?
When I get it right, that's also why we currently recommend to use the RDS master user as debezium user. Can you confirm my findings?

Thank you @rafalwytrykus !
I took a closer look at a AWS RDS Postgres installations today. When I understand it correctly, the debezium connector user needs to have one of these roles assigned: rds_superuser, rdsadmin or rdsrepladmin in order to have the rolreplication rights that allow starting a logical replication. Correct?
When I get it right, that's also why we currently recommend to use the RDS master user as debezium user. Can you confirm my findings?

@rafalwytrykus @RemingtonReackhof Can you have a look at #2037 and also at the tasks https://issues.redhat.com/browse/DBZ-2893 and https://issues.redhat.com/browse/DBZ-2894. This should fix the issue with AWS, and also with other environments later in a more generic way.b WDYT?

@rafalwytrykus @RemingtonReackhof Can you have a look at #2037 and also at the tasks https://issues.redhat.com/browse/DBZ-2893 and https://issues.redhat.com/browse/DBZ-2894. This should fix the issue with AWS, and also with other environments later in a more generic way.b WDYT?

@rk3rn3r apologies for the late response. #2037 looks like the right approach, I appreciate the fix 👍 . I'm having issues creating an account on redhat so I can't take a look at the issues right now. I appreciate the work on this :)

@rk3rn3r apologies for the late response. #2037 looks like the right approach, I appreciate the fix 👍 . I'm having issues creating an account on redhat so I can't take a look at the issues right now. I appreciate the work on this :)


Increase build number

Checklist

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

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

@conda-grayskull show requirements

Working on your request...

Working on your request...

@conda-grayskull show requirements

Working on your request...


1.21 Port

Hi, your mod is amazing and super useful, but I saw it's available for 1.20.1 latest. I've ported it to 1.21 (minimal changes to the code), so please use this PR if you wish.

Hi! Thank you so much!! I am uploading the jar file now with credit. 😁👍


[Search] Add Search GA 11.4.0 features

For Search GA 11.4.0 release, adding below features on the top of last 11.3.0 GA release:

AAD support including multi cloud support
Any bug fixes since last GA release
Troubleshooting guide

Fixes: https://github.com/Azure/azure-sdk-for-net/issues/30325

@pallavit - Here is the API view comparing with GA 11.3.0 API.

/check-enforcer override


npm deprecation warnings

You might want to upgrade these deps:

npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>All versions below 4.0.1 of Nodemailer are deprecated. See https://nodemailer.com/status/

There may be more but I got bored watching the install so maybe I'll update the post once install finished.

Thanks @mk-pmb!
It’s not as straight forward to update them. Some context
the minimatch seems to come from @hoodie/admin which we is a bit stale in its development, it’s also a sub sub sub sub sub dependency so I’m not sure how easy it will be to update for us
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └─┬<EMAIL_ADDRESS>│     ├─┬<EMAIL_ADDRESS>│     │ └─┬<EMAIL_ADDRESS>│     │   └──<EMAIL_ADDRESS>
we want to remove the nodemailer dependency altogether from @hoodie/account-server and have people implement things like email reset password flows themselves via plugins
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
we could probably update nodemailer to v4 in @hoodie/acount-server though as its a direct dependency

i am also very very love it,I am Amos.z from The forefront of absolute fashion trends, exclusive news presentation! If you want to know more, please continue to pay attention to @http://www.bootsshoppingonline.com and turn on those alerts for more updates on this release. Otherwise, take a look at listed retailers/ raffles below and good luck on the morning of release.

npm install -g nodemailer
    // use this command

@digitalhemanth If installing the deprecated module globally turns off the deprecation warning on your node, I'd consider that a hint that your node might have problems with dependency resolution.

Is this something I can work on?

Sure, feel free to :)

created the following PR
#891

I can start on the devDependencies next

created PR #892
for deprecation warnings under dev dependencies!
Thanks


Introduce raw literal option

This introduces a new option to handle default values in their raw form.
When the default.raw option is enabled, the default value is treated exactly as provided, without any modification or expansion.
Fixes #118.

Сlosing this
Because I messed up the git history while working with keys and signatures


HDDS-3958. Intermittent failure in Recon acceptance test due to mixed stdout and stderr

What changes were proposed in this pull request?
Make curl less verbose to avoid connection-related messages being mixed into the response from Recon.
https://issues.apache.org/jira/browse/HDDS-3958
How was this patch tested?
Verified output in acceptance test:
<arg>${output}</arg>
</arguments>
<msg timestamp="20200714 14:02:46.735" level="INFO">[{"volume":"vol-3-10121","bucket":"bucket-4-58535","fileSize":16384,"count":5},{"volume":"vol-4-32109","bucket":"bucket-2-76236","fileSize":16384,"count":5},{"volume":"vol-3-10121","bucket":"bucket-3-34240","fileSize":16384,"count":5},{"volume":"vol-2-74673","bucket":"bucket-2-24567","fileSize":16384,"count":5},{"volume":"vol-0-21163","bucket":"bucket-1-46076","fileSize":16384,"count":5},{"volume":"vol-0-21163","bucket":"bucket-3-20203","fileSize":16384,"count":5},{"volume":"vol-1-98932","bucket":"bucket-4-69150","fileSize":16384,"count":5},{"volume":"vol-2-74673","bucket":"bucket-1-35826","fileSize":16384,"count":5},{"volume":"vol-0-21163","bucket":"bucket-0-06750","fileSize":16384,"count":5},{"volume":"vol-1-98932","bucket":"bucket-1-85094","fileSize":16384,"count":5},{"volume":"vol-3-10121","bucket":"bucket-2-49899","fileSize":16384,"count":5},{"volume":"vol-0-21163","bucket":"bucket-2-24533","fileSize":16384,"count":5},{"volume":"vol-3-10121","bucket":"bucket-0-55405","fileSize":16384,"count":5},{"volume":"vol-1-98932","bucket":"bucket-0-69261","fileSize":16384,"count":5},{"volume":"vol-1-98932","bucket":"bucket-3-37904","fileSize":16384,"count":5},{"volume":"vol-1-98932","bucket":"bucket-2-74383","fileSize":16384,"count":5},{"volume":"vol-4-32109","bucket":"bucket-3-53360","fileSize":16384,"count":5},{"volume":"vol-2-74673","bucket":"bucket-4-90899","fileSize":16384,"count":5},{"volume":"vol-0-21163","bucket":"bucket-4-09448","fileSize":16384,"count":5},{"volume":"vol-4-32109","bucket":"bucket-0-21307","fileSize":16384,"count":5},{"volume":"vol-3-10121","bucket":"bucket-1-37420","fileSize":16384,"count":5},{"volume":"vol-4-32109","bucket":"bucket-1-96158","fileSize":16384,"count":5},{"volume":"vol-4-32109","bucket":"bucket-4-81674","fileSize":16384,"count":5},{"volume":"vol-2-74673","bucket":"bucket-0-72641","fileSize":16384,"count":5},{"volume":"vol-2-74673","bucket":"bucket-3-10948","fileSize":16384,"count":5},{"volume":"87083-rpcwoport","bucket":"bb1","fileSize":32768,"count":0},{"volume":"87083-rpcwoport2","bucket":"bb1","fileSize":32768,"count":1},{"volume":"87083-rpcwport","bucket":"bb1","fileSize":32768,"count":0},{"volume":"87083-rpcwoscheme","bucket":"bb1","fileSize":32768,"count":0},{"volume":"fstest370","bucket":"bk1","fileSize":32768,"count":1},{"volume":"fstest1","bucket":"bucket1-ofs","fileSize":1024,"count":4},{"volume":"fstest1","bucket":"bucket1-ofs","fileSize":32768,"count":3},{"volume":"fstest2","bucket":"bucket3-ofs","fileSize":1024,"count":0},{"volume":"fstest1","bucket":"bucket1-o3fs","fileSize":32768,"count":3},{"volume":"fstest1","bucket":"bucket1-o3fs","fileSize":1024,"count":4},{"volume":"fstest2","bucket":"bucket3-o3fs","fileSize":1024,"count":0},{"volume":"s3v","bucket":"bucket-83530","fileSize":1024,"count":2},{"volume":"s3v","bucket":"bucket-89656","fileSize":1024,"count":1},{"volume":"s3v","bucket":"bucket-53511","fileSize":8388608,"count":4},{"volume":"s3v","bucket":"bucket-53511","fileSize":16777216,"count":2},{"volume":"s3v","bucket":"destbucket-83303","fileSize":1024,"count":1},{"volume":"s3v","bucket":"bucket-54425","fileSize":1024,"count":2},{"volume":"s3v","bucket":"bucket-59094","fileSize":1024,"count":1},{"volume":"vol-0-15917","bucket":"bucket-0-74640","fileSize":2048,"count":10}]</msg>

@adoroszlai Thanks for working on this!

Thanks @vivekratnavel for the review.


ML 7.2: Adaboost (D)

Description 📜
Added details about AdaBoost and its implementation
Fixes #7003

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Machine Learning

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@Isha307 Is this ok now?


integration specs for views and fix n+1 problems

Changes introduced. 😎

Write integration tests with Capybara gem. ✔
Secure app from n+1 problems. ✔

Note.

The Capybara test requires appropriate data to test the app. this can be done with rails db:seed RAILS_ENV=test, of course after migration.
N+1 problems are tested on the browser and disabled in a rspec test, this is to prevent mixing up tests with N+1 issues. but can be enabled with uncommenting the https://github.com/Tadesse-Alemayehu/Rail-blog-app/blob/8fec5993649f3565a11bf30ae939344ed897b57d/config/environments/test.rb#L12 line on /config/environments/test.rb

https://user-images.githubusercontent.com/69077061/186495208-a97812a4-93cc-4d50-bb6c-103b56e6aab6.mp4
.

Thanks @RolandM99, Nice to have your aproval


Add "Connecting to SQLite database" to documentation

It would be nice to add documentation on how to connect to SQLite database. Related: http://www.ch-werner.de/sqliteodbc/html/index.html

https://github.com/mkleehammer/pyodbc/wiki/Connecting-to-SQLite

Thank you!


db-proxy(VeritasDB) on rust-sgx-sdk

#116
This is a code implementation of VeritasDB on rust-sgx-sdk.
It will guarantee on database integrity verification and database privacy.
I will continue focusing on it. :smile:

Although there are some things need to be done, I think this pr could be merged if you agree with it. :smile:
Performance Optimizations should be considerd in the paper so I will take some time on it.

Sure! Thank you so much for your contribution!

Wow nice one ! Thanks
On Mon 27 May 2019 at 19:50 Yu Ding<EMAIL_ADDRESS>wrote:

Merged #119 https://github.com/baidu/rust-sgx-sdk/pull/119 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/baidu/rust-sgx-sdk/pull/119?email_source=notifications&email_token=AGFVWNK7QJFFMWG5A4X737LPXQNNVA5CNFSM4HOH4W52YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGORVBIJTQ#event-2369946830,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGFVWNLN2RZDYEIYIQ5NWNTPXQNNVANCNFSM4HOH4W5Q
.

@davidp94 This is a simple implement for go and I will translate it to rust.
more work is on the way! :beers:

@bradyjoestar Do you have a plan to finish it? I really like it!

@dingelish Sorry I am recently busy working on sqlite in the sgx at the production enviroment :-)
I couldn't assure when this pr is completed. :smile:


Add dlib::rdrand kernel to support digital random numbers generation with RDRND instructions

This adds the dlib::rdrand kernel extension to support the generation of digital random numbers with RDRND instructions. A digital random number generator uses a processor resident entropy source to repeatedly seed a hardware-implemented cascade construction RNG. It includes a high-quality entropy source implementation, being very useful for scientific applications.

I believe that std::random_device doesn't need to be non-deterministic and some implementations might be an improved version of a pseudo RNG seeding over time(NULL). For this reason the proposal for an implementation that queries information directly from the CPU.

@davisking I made some changes, thanks for your review. If you think it's not interesting to insert these features in Dlib, no problem, just let me know.

Some googling makes it appear that common implementations of the standard library do use rdrand, which is the idea behind std::random_device.  I.e. that when there is actually hardware randomness to be had it gives it to you.  So I think dlib users would be better off using the C++ standard tooling for this.  I try to avoid replicating stuff in the standard library, since in general dlib can't do as well as what's in the standard library.  For example, code that uses std::random_device will be generally more portable.  Since say, you move to an ARM machine that has some other way of getting hardware randomness and you automatically benefit from the standard implementation on that platform which should Do The Right Thing.  I do realize that sometimes people bork up code though.  But usually the standard library implementations are pretty solid.
Anyway, I'm not saying there is anything wrong with your code here.  It looks good.  I just don't want to duplicate std::random_device in dlib.  Especially considering this dlib::rdrand is inherently non-portable, and one of the goals of dlib is to be super portable.  So it would have to have a fall back to a non-rdrand back end.  But then that's again exactly what std::random_device is supposed to be.

I see, it makes sense to me, thanks again. I will reserve this for my projects.  I appreciate your feedback.

Sweet :)


Imported tasks not linked to the correct users.

After importing it doesn't seem like it links entries to the correct users in the workspace. It only links all the entries to the account holder.

Thanks looking forward to the update. :)

@cobuscoetsee I wanted to give an update on this. I've been swamped at work and haven't had a chance to dedicate too much time to this but I've been working on it when I can. It might be another week or so due to the holiday this weekend. I'll keep you posted.

@mikerourke thanks so much. No rush, we decided only to start to use the tool 1 January 2019. :)

The transfer of all entries is indeed linked only to the account holder of the workspace. We are also looking forward to having this feature to work with our multiple-user database :+1:

It looks like you're gonna need API token for each user if you want to create entries in their name.
Maybe, for a start, transfer only one person's entries (the one doing the transfer). Then, each person can repeat the process and transfer their own entries. This way, a team can transfer all their data by running the importer multiple times and there's less development headaches

Any update on this? I just installed this importer this morning, put in my API tokens for my account as per the instructions and now have a giant jumble of entries in Clockify since it has all of the entries for 3 different staff people in 2018.
I guess I'll nuke the clockify account and start over but there doesn't seem to even run the user by user approach mentioned above.

@ethanclarke Managing multiple users might be tricky. I'm still hashing out how to accomplish that. I could probably add it to the web version I'm working on, but the order of operations would need to be very specific (i.e. before using the tool, setup workspaces with user email addresses that exactly match Toggl, then perform a transfer). Testing it is going to be a little clunky and problematic, but I do want to add this feature in.
I've been trying to finish the damn thing but I've just been slammed at work. I have some time opening up the week after next. I have a list of people that I'll alert when I have something ready. I apologize for the delay and I'll let you know when it's good to go. Thanks!

@mikerourke Have you had any time to work on this? I'd love to move my team to Clockify but we're stuck on Toggl for now! Even just a command line version of this would be fine.
Thanks in advance!

@ethanclarke I apologize for the delayed response. Without getting into too much personal detail, I been in the process of selling my house for the past 2 weeks so my life has been complete bedlam. We’re closing tomorrow and my availability is about to skyrocket. I’ve been incredibly bothered by the fact that I haven’t been able to get this tool in a usable state despite my best efforts. I blew the dust off the web version last weekend and figured out what needs to be done. I’m planning on spending this weekend working on it. I appreciate your patience and I completely understand that it’s probably wearing thin. I would like to work with you on transferring team entries to ensure I have the workflow correct. I am committed to getting this thing up and running!

Hi Mike,
No need to apologize!
The impatience I have is for Toggl who increased their prices such that a small business like mine struggle to afford them. But we can carry on for a month or two more.
Good luck with the move!

Hello Mike,
I just wanted to say thanks for everything until now. I am in the same boat as Ethan as well. Toggl has been getting more and more expensive and I am looking for a change.
If you post a bitcoin / ETH address or paypal email I would be happy to send you an advance right away. I know that you are a volunteer and I know that your work is going to save a lot of people a lot of money. I'd be happy to pay for the privilege.
As a minor suggestion, you may consider importing a Toggl CSV instead of using the API access. The Toggl CSV has all the data for all users (active and inactive) nicely organized and categorized. You can then use the Clockify API to export to Clockify. You may need to have API keys for each clockify user but that may not be a big deal.
Thanks again. Keep up the good work. Let me buy pizza and beers for the next couple weekends!

@908E That's very kind of you to offer, I propose you cover congratulatory pizza when the tool is up and running! With regard to the CSV thing, I have a few contentions:

I don't think you can export to a CSV if you're on the free account, which would exclude a huge percentage of users.
There's always some weird edge case or idiosyncrasy that needs to be accommodated for when importing/exporting files.
At this point I'm so close to being done with the APIs that it makes more sense to continue on my current path.

I'll be working on the time entries/workflow stuff this week. I'll ping you and @ethanclarke when I have something to step through. I may just mock something up and have you go through the workflow (without actually transferring anything) just to make sure everything makes sense and functions correctly.  Thank you for being so accommodating, I appreciate it!

Thanks! Congratulatory pizza sounds like a good idea!
I thought that the CSVs would be a good way to work around the separate user issue but I agree that it may add some unnecessary complexity at this stage.


Search implementation

Closes #16

Merging as this is because the tests are running against an environment that is down and have run correctly (intermittenly) locally. This will be fixed by #47.


Document that update works differently to normally dictionary

Looking at the source of addict, I noticed that the update method helpfully recurses into nested dictionary to do the update. That isn't documented though, and could be a surprise since it's inconsistent with the base dict class. For example:
>>> d = {'a': {'b': 3}}
>>> d.update({'a': {'c': 4}})
>>> print(d)
{'a': {'c': 4}}
>>> D = Dict({'a': {'b': 3}})
>>> D.update({'a': {'c': 4}})
>>> print(D)
{'a': {'b': 3, 'c': 4}}

Thanks @bmerry. Will add it to the documentation. 👍


Fix ThriftDocServicePlugin maybe failed when thrift proto compile with fullcamel option

Motivation:
Fix the problem that ThriftDocServicePlugin maybe failed when thrift proto compile with fullcamel option.
For example, thrift proto like this
service GreetService {
    string greet_hi(1: string name);
}

when use java:fullcamel in thrift compile, the doc service will failed to start, with exception like this
Caused by: java.lang.IllegalStateException: failed to find a class: hello.GreetService$greetHi_args
	at com.linecorp.armeria.internal.server.thrift.ThriftDocServicePlugin.newMethodInfo(ThriftDocServicePlugin.java:210) ~[armeria-thrift0.9-1.23.1.jar:?]
	at com.linecorp.armeria.internal.server.thrift.ThriftDocServicePlugin.lambda$newServiceInfo$4(ThriftDocServicePlugin.java:181) ~[armeria-thrift0.9-1.23.1.jar:?]
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193) ~[?:1.8.0_291]
	at java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948) ~[?:1.8.0_291]
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:482) ~[?:1.8.0_291]
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:472) ~[?:1.8.0_291]
	at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708) ~[?:1.8.0_291]
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) ~[?:1.8.0_291]
	at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499) ~[?:1.8.0_291]
	at com.linecorp.armeria.internal.server.thrift.ThriftDocServicePlugin.newServiceInfo(ThriftDocServicePlugin.java:183) ~[armeria-thrift0.9-1.23.1.jar:?]

Modifications:

Add a method functionName in ThriftServiceEntry to get the proto method by ThriftFunction
Add ThriftServiceEntry to com.linecorp.armeria.internal.server.thrift.ThriftDocServicePlugin.Entry as a member
Add extra process to get the ThriftFunction's name for argsClassName in ThriftDocServicePlugin#newMethodInfo
Assure the MethodInfo.name always use the right thriftFunctionName, otherwise the example requests will not work.

Result:

thrift proto service method name like "say_hi" with fullcamel option will generate right thrift docs service for debug and view.
thrift proto service method name like "say_hi" with fullcamel option could process the example request right.

Thanks, @richieyan ❤


Error getting token ends up on JS error

I am frequently getting an error

always preceded by an identity API error

I am not managing the refresh token myself, so what could do that the resfresh token is invalid ?

I receiving the same error any solution?


Disable Tabnine for some project

Make it possible to disable Tabnine for certain project
Some customers of mine, (freelancer here) aren't directly open to use their code with AI.
So is it possible to disable the plugin for some project (instead of disabling and restarting phpstorm al together)
gz#22584(related to Zendesk ticket #22584)

@PaulMenheere
At the moment you can only disable Tabnine for the IDE or for certain files (for instance .py files)

Is disabling on project base something that we'll see in the future ?


Added remove_property method to GenericNode

This is useful to have, especially for some serializers which require certain properties to not exist, not just be undefined.

Thanks!


Implement Sound Effects

Find sounds/ create them
Edit them
Make them dynamic in game

I have announcer sounds in, and have recorded my boyfriend making some sounds for Jibbers (or another male character) being hit, but haven't added those yet.

Done


Adding a rights param

Hey,
I am looking at this list - https://developers.google.com/custom-search/v1/cse/list
and I realize that one of the crucial ones seems to be rights.
Is it that unless we add that to the _search_params_keys, it’ll not be accepted as a parameter?
Or does your code accept arbitrary params?
If the former, I can try to add rights as a key and send over a PR.
Thanks!

Hey @nitinthewiz,
Thanks for your contribution.
Feel free to add rights (or any other from the list) into self. _search_params_keys by creating pull request.

Hey @nitinthewiz,
still interested in this?

Thanks for closing this. I'll get to it at some point.


Updating mysql configuration to follow mysql container documentation

To resolve issue brought up in https://github.com/laradock/laradock/issues/583

Please also remove startup script.

@Mahmoudz I missed the comment on this PR while on vacation. Was this closed only because I hadn't followed up on the comment from @appleboy or for another reason?

@gabrielpreston I just noticed around 150 issues and over 20 PR today so I thought closing some issues especially old ones quickly just to clean up. If an issue still exist someone is gonna open it again or ask "like you" for reopening. And that's exactly what I'm gonna do. Thanks,

@gabrielpreston "sorry I left the wrong comment before".
I just closed it while closing a big number of Issues and PR's todays. Mainly old ones and those who seems pending for over a week (PR's in that case).
I'll reopen again that now. Thanks,

Removed the unused file and resolved the merge conflicts

For me it only worked using the YAML array notation, as below:
    mysql:
        build:
            context: ./mysql
        environment:
            - MYSQL_DATABASE=homestead
            - MYSQL_USER=homestead
            - MYSQL_PASSWORD=secret
            - MYSQL_ROOT_PASSWORD=root

can be closed. see #758


Non-latin query error

[ERROR: alfred.workflow.input.scriptfilter] Code 1: load_list.rb:53:in `=~': incompatible encoding regexp match (ASCII-8BIT regexp with UTF-8 string) (Encoding::CompatibilityError)
	from load_list.rb:53:in `tab_matches_title?'
	from load_list.rb:59:in `tab_matches_query?'
	from load_list.rb:195:in `block in <main>'
	from load_list.rb:194:in `each'
	from load_list.rb:194:in `<main>'

Thanks for the submission, I will have a look into this but I may be in need of searching for someone who is an absolute GOD with regex to fix this as the regex I had to use is immensely complex (and even I only understand it to a point).
Therefore if there is anyone that is a regex guru and might be able to help out please get in touch. I will have a look in the mean time however.
Cheers,
Stuart


Wrong UTC time in containers?

Using Windows 10, Docker Desktop
In container:
Thu Feb 13 17:49:33 Vilnius 2020
Google says:
Thu Feb 13 18:49:33 Vilnius 2020
Tried:
apt-get update && apt-get install tzdata
ENV TZ=Europe/Vilnius
ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone
All other containers that use ubuntu images are OK. This is the first I encountered;
Please help ? I'm not too familiar with docker, and this is quite a problem if you ask me

I think you are getting an incorrect timezone, not an incorrect UTC time.
Hmm, whenever you deploy your container it will probably be in a datacenter that might be in a different timezone different than yours, so I'm not sure how useful it would be to assume a timezone in your container.
It would probably be better to use UTC in your code. Or if you are going to depend on timezones, it should be the timezones of your clients or timezones defined in your settings.
For example with things like:
from datetime import datetime

now = datetime.utcnow()
print(now)

Otherwise, please provide more info and a minimal self-contained example that shows the error.
Currently, I have no way of knowing how you are getting those dates, or how to help you debug it.
Also, it seems you created the exact same issue in 2 repos: https://github.com/tiangolo/uvicorn-gunicorn-fastapi-docker/issues/29
But it's still not clear where (in which repo) is your problem...

Same as https://github.com/tiangolo/uvicorn-gunicorn-fastapi-docker/issues/29


Fix: don't open POI when zooming with double click

Description
Proposes a solution to fix the problem of double click/tap zooms also triggering a click on the map, which always opens the POI panel even if the user just wanted to zoom in.
Cause
The problem is due to the way click and dblclick events are managed by the browser.
On desktop browsers (no touch), when the user double click with the mouse, 3 events are thrown, always in this order:

a first click event
a second click event
a dblclick event
The second click event and the dblclick event are the same underlying object, with a property detail which indicates the number of actual clicks at the same coordinates (here detail will be 2).

Each click triggers our click listener on map, which calls the clickOnMap function, opening the POI panel.
Because the doubleClickZoom option is true, the dblclick event is listened internally by the Mapbox-GL instance which performs the zoom after.
On touch-based devices, there is no dblclick event, but MapBox-GL has heuristics to consider two close (in time and distance) touchend events as a single equivalent action (https://github.com/mapbox/mapbox-gl-js/blob/master/src/ui/handler/dblclick_zoom.js).
Principle of the fix
To prevent the POI action from happening when zooming with double click, we need to delay the handling of a click until we are sure no dblclick event or the touch equivalent will happen just after the first click.

in the click listener, we delay the call to clickOnMap with setTimeout.
here there are two cases
a. if it was a single click action by the user, the clickOnMap will be called normally after the delay has passed
b. if the user actually double clicked/tapped, the click listener will be called a second time immediately, before the end of the setTimeout. We can then cancel the timeOut, which will prevent the POI panel from opening.

The most complicated part is detecting the cancellation condition on mobile, i.e. detecting two touchend events formed a double tap that happened within the authorized delay. For now I used a really simple rule based on timestamp only, ignoring the multitouch dimension and the distance between the positions of the touch. It seems to work fine, but I hope it won't cause bugs…
For the delay, I used the same value as MapBox to consider what is a double click/tap, i.e. 300ms.
Of course, this has the consequence of delaying all click actions on the map by 300ms, making it seem less responsive…
Update: when clicking on a POI, we can safely skip the delay and trigger clickOnMap. Thanks @amatissart for the suggestion :)
Maybe I overlooked some better solution but I don't think there is one, as we can't predict the user is in the middle of a double click/tap or not.

@xem

I'd go even further, and request the api for the PoI panel's info immediately when a click is detected, and only allow the panel to appear after 300ms, if no second click has been made. (GG Maps doesn't seem to do it)

Nice perf tweak! It would be for the "POI anywhere" only, as the clicks on interactive elements are resolved immediately.
As it would require some non-trivial refactoring, I won't do it here but it would be cool to look into it later.


Update README.md

Added the following to provide the open source community with more information:

This project uses third-party libraries, which are licensed under their own respective Open-Source licenses.
Also added a reference to Code of conduct and govermance.md in contributing section

Signed-off-by: Jonas van den Bogaard<EMAIL_ADDRESS>

Can one of the admins verify this patch?


Update for loading pretrained checkpoints

Now users can run scripts for testing pretrained checkpoints without the need to specify the cfg.yaml files. The scripts load the correct cfgs for pretrained checkpoints automatically.

LGTM for now. We need to later borrow the design of timm and huggingface/transformers


bcrypt and bcrypt-pbkdf implementation

Hi,
This is a request for implementing bcrypt hashing and bcrypt-pbkdf.
The reference C implementation for bcrypt-pbkdf can be seen here.
https://github.com/openssh/openssh-portable/blob/master/openbsd-compat/bcrypt_pbkdf.c
Thanks!

I added bcrypt and bcrypt-pbkdf (commits 5834affd7aea86e26fc1b5f2d119007a098d02a8 and b002b740a320ed8d1e415d906bee12f98fd5ed54).
Note that the derive-key method returns the raw hash, not the $2b$[cost]$[22 radix64 character salt][31 radix64 character hash] bcrypt string.

Thanks, @glv2 !

I've packaged things up in a system for working with bcrypt hashes here, thanks to the newly added code in ironclad.
You can find the system here, if interested.
https://github.com/dnaeon/cl-bcrypt

Nice!


Help link on Camera error page is broken

Describe the bug
In Uppload 3.2.1, the error message on the camera service contains a broken link ("Need Help?" goes to https://uppload.js.org/help/camera which doesn't exist anymore).
To Reproduce
Steps to reproduce the behavior:

Deny camera access
Go to Camera service

Expected behavior
The "Need Help?" link should go to the proper help page.
Additional context
I think it may be a good idea to rewrite the error message as well, currently it says:
"We're not able to read your camera's video. This may be because you didn't grant the required permission, or because your device doesn't support camera access."
I recommend "We're not able to read your camera's video. This may be because you didn't grant the required permission, or because your camera is in use by another application."

Thanks for the report! The link should go to https://uppload.js.org/help/services/camera and not https://uppload.js.org/help/camera. Can you make a PR to fix this, and the error message? 😄


Refactor case BuiltinOperator_GATHER_ND in flatbuffer_conversions

PR1 for issue #46268.

@rsun-bdti  Can you please resolve conflicts? Thanks!

@rsun-bdti  Can you please resolve conflicts? Thanks!


How to show delete confirmation window in custom treeview (VSCode Extension)

I would like to have a popup window like this in my custom treeviewprovider. Is there any api I could use?

vscode.window.showInformationMessage('Hello', { modal: true }, { title: 'OK' });

vscode.window.showInformationMessage('Hello', { modal: true }, { title: 'OK' });

Is it possible for this dialog to show in the center of editor?

It's showing in the center. Try it yourself.

vscode.window.showInformationMessage('Hello', { modal: true }, { title: 'OK' });

Thanks for the help. How can I achieve the style like the one you posted? Mine seems really ugly..

I have the "window.dialogStyle": "custom" setting. You can't change this only for your dialogs.

"window.dialogStyle": "custom"

Can I write this in my extension settings.json?

Yes, although not recommended.
For example
    const configuration = vscode.workspace.getConfiguration('window');
    let oldValue = configuration.get('dialogStyle');
    if (oldValue !== 'custom') {
      await configuration.update('dialogStyle', 'custom', vscode.ConfigurationTarget.Global);
    }
    await vscode.window.showInformationMessage('Hello', { modal: true }, { title: 'OK' });
    if (oldValue !== 'custom') {
      await configuration.update('dialogStyle', oldValue, vscode.ConfigurationTarget.Global);
    }

All the suggestions above are good (including that it's not recommended to overwrite user settings unexpectedly), thanks @yume-chan!


22 evaluation data

Hello.
I am planning to evaluate my surface reconstruction method with your benchmark dataset.
You says "we randomly sample 22 synthetic surfaces of object instances" in Section 5.1, but I could not find any information on which files these 22 object instances are.
Please tell me the names of these 22 files so that I can make a fair comparison with the scores presented in your paper.
Thank you.

Thanks for your interest!
We will continue publicizing these 22 synthetic surfaces later on our benchmark page.


ReadTheDocs fixes

This PR:

removes the generated HARKmanual related markdown, which was ill-formed and breaking the recommonmark parser, and brings back the parser. See #420
points to the Gentle notebook in DemARK (#409), rather than the one in Examples/ (which is now deleted #417 #419 )
adds some introductory text to the RTD index, explaining purpose of docs and linking students to DemARK.
refactoring the index.rst logic so that it can include the autosummarized API docs without an awkard 'placeholder' element with a dead link.

I'm still seeing the placeholder element with the dead link:


Kiosk mode in an android webview

Kiosk mode 5.4.0
Hi.
I'm developing an overlay app that will show HA information in a floating webview on top of a TV running android TV.  So, my app is effectively just an android app with the ability to show a webview.
I have kiosk mode working in chome on my PC (so know it's working):

However, when I open the same URL in the webview on my android simulator (or android device), the webview is shown with the blue top banner still shown:

I don't know what techniques you use to get rid of the banners but any ideas why it might not work in the webview?
Note, to use the webview without needing to sign into HA, I use external authentication which adds an additonal query string to the URL:
http://<IP_ADDRESS>:8123/dashboard-gardencamonly/0?external_auth=1?kiosk
however, I have also tried:
http://<IP_ADDRESS>:8123/dashboard-gardencamonly/0?external_auth=1
with:
kiosk_mode:
hide_header: true
views:
added to the lovelace dashboard.
Any help would be great, thank you!

Chaining parameters is not standardized but the de facto standard is to use ampersand (&)
Anyway we have a similar situation were the blue top stays with ios devices but it seams in your case it's not only the blue top but the whole menu.
I guess we would need a check if kiosk mode is loaded in this situation maybe it is blocked by something.

http://<IP_ADDRESS>:8123/dashboard-gardencamonly/0?external_auth=1?kiosk

This is a malformed URL:

This is a correct URL:

Try to debug the logs and search for WebView errors. Make sure to enable JavaScript (most probably you have it enabled) and also kiosk-mode uses `localStorage, make sure you enable the DOM storage API.

Thanks for your quick support.  Thanks for pointing out my misunderstanding with the chaining of parameters.  I'll get that fixed.
I enabled DOM storage in my webview and can confirm that when I have kiosk mode enabled in the dashboad yaml, the blue banner is removed.  It is briefly shown as the page is drawn and then dissapears.  It would be good to fix that too but that's a problem for another day...
Thanks again - great feedback!

Good to know @mph070770!
Unfortunately, a delay will occur on page load because kiosk-mode needs to wait for the elements being rendered and accessible through the DOM to be able to manipulate their styles.
Regards 👍🏼


Update "Add Superseded Template step"

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/3775

@mapalko The PR that addresses issue #3775 is available for review. Thanks.

@mapalko Following up for updates.

@AndreaBarr The PR that has been approved by the document author is ready to be merged. Thanks.


 Thanks again for your work!
It's very subjective of course, but I'm not very fond of this overly long slider! What do you think about clamping it to a reasonable size?

@xou816 from a practicality standpoint I really like the length in the screenshot, since especially for podcasts and such fine tracking can get difficult. I'm ambivalent from an aesthetics standpoint but clamping length would probably reduce usability a bit at least.

Would it potentially make sense to integrate the seekbar with the dividing line between the bottom bar and the rest of the UI? Sort of like how the spotify app does when the song is collapsed at the bottom (not in player view). It would allow the longest possible seekbar length without taking up any extra room.

@xou816 personally I prefer the longer seek bar for the better accuracy when seeking. Maybe we could make this toggle-able in a settings menu?

Nah, you've all convinced me :) but how about some extra spacing in the main bottom box? a little bit more, so long as it still allows the window to shrink to 360px
@Douile there seems to be one useless box in the hierarchy (maybe this was there before, not sure!)

@szbergeron that's an interesting idea! I don't know where we'd put the track duration display though?

I'll spare you my bad gimp skills, but elapsed/total time could either go under the play button or could display on hover over the seekbar (maybe only display the seekbar handle on hover too?) Think how the youtube seekbar UI works, just with the hovered time instead of the video preview.

I really like this idea! Might be worth trying out! Maybe not in this PR, but we can totally experiment with this!

About the spacing: Here is why I think some spacing is necessary :

Ye I see what you mean, I added some margins to avoid that, lmk what you think. I believe should expand the seek bar to the right where the space isn't being used.
Video of resizing with margins

This is better, thanks! Merging :)


Failed to add hook

I have tried backdooring 3 apks so far, 2 have given me the [!] Failed to locate smali file to hook bug, the last one (Pokerstars) gave me this error:
[--everything OK so far---]

[] Locating smali file to hook in original project...done.
[] Adding hook in original smali file...done.
[!] Failed to add hook

End of run.log:

File: /root/Desktop/backdoor-apk-master/backdoor-apk/original/smali/com/core/adkit/a.smali
Target: "/"
Reg: v5
Shift count: 03
Shift tr value: [X-ZA-Wx-za-w]
Replacement: "03/"
File: /root/Desktop/backdoor-apk-master/backdoor-apk/original/smali/com/core/adkit/a.smali
Target: ""
Reg: v0
Shift count: 23
Shift tr value: [D-ZA-Cd-za-c]
Replacement: "23"
File: /root/Desktop/backdoor-apk-master/backdoor-apk/original/smali/com/core/adkit/c.smali
Target: "SHA-1"
Reg: v0
Shift count: 08
Shift tr value: [S-ZA-Rs-za-r]
Replacement: "08KZS-1"
File: /root/Desktop/backdoor-apk-master/backdoor-apk/original/smali/com/core/adkit/c.smali
Target: "WWWW"
Reg: v0
Shift count: 09
Shift tr value: [R-ZA-Qr-za-q]
Replacement: "09NNNN"
File: /root/Desktop/backdoor-apk-master/backdoor-apk/original/smali/com/core/adkit/c.smali
Target: "Invalid certificate"
Reg: v1
Shift count: 11
Shift tr value: [P-ZA-Op-za-o]
Replacement: "11Xckpaxs rtgixuxrpit"
Found launcher line in manifest file: 63
Activity lines found above launcher line: 1
Value of android_target_activity:
Value of android_name: com/pyrsoftware/pokerstars/home/HomeActivity
The launcher line appears to be within an activity element
Value of tmp: com/pyrsoftware/pokerstars/home/HomeActivity
The smali file to hook: /root/Desktop/backdoor-apk-master/backdoor-apk/original/smali/com/pyrsoftware/pokerstars/home/HomeActivity.smali
Forcing cleanup due to a failure or error state!

I am having issues with my connection as my country internet is really unstable, so I just downloaded the master.zip and extracted it, I am thinking maybe the zip file was corrupted from having to resume download after pause but everything seems OK.

@hahqmlta88 I bet this is a duplicate of #75 but I'll try to confirm before closing this issue.

@hahqmlta88 Can you attach the APK file you have?

Closing as duplicate of #75

I have got the APK on an other hard drive. Just wanted to say, after looking at #75 and decomposed the apk with apktool on that day it seems very likely it is in fact a duplicate.

i was trying to compile cclean.apk when i got this message :
[!] Failed to add hook


Add a "Trusted Executables" feature

The existing Trusted Workspace feature is insufficient when the user creates a new workspace, which they trust, but then an extension executes a process on their computer outside of the workspace which may not be trusted. If a user is the only one with access to their machine, maybe they can say all executables on their computer are trusted (not sure though), but on a shared computer it wouldn't be. In that case, if an extension wants to execute some executable not in the user's workspace, then a trusted executables feature could be used to manage which executables to trust.
Maybe a folder could be specified, so that all executables under a particular folder could be regarded as trusted for scenarios where there are lots of executables that need to be run at particular location, e.g. build tools -- the assumption being that only the particular user or an admin would have access to that location. Perhaps the feature could only apply if a different non-admin user has access to the location and/or users could be warned that the executables permissions are such that another user may have supplied the executable maliciously. Maybe a Trusted Computer option could be available to allow users to trust all executables on their machine (i.e. the current behavior).
Otherwise, it would help if some security guidelines could be provided to extension authors on how to handle this situation. It seems unusual that every extension in this situation would be expected to implement their own Trusted Executables feature, when the Trusted Workspaces feature was implemented by VS Code.

Well, it sounds like the plan is go to with an alternative design for our extension, so I'll close this unless someone else is interested in this feature and wants to re-open it.


HOL2897: OCW24-MySQLHeatwave-FinalPR

Add here your WMS ID
HOL2897: OCW24-MySQLHeatwave-FinalPR
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[Yes] All filenames are lower case (including folders, images, files, etc.)
[Yes] Filenames are descriptive
[Yes] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[Yes] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[Yes] Image references in markdown contain an alternate text

@carmenberdant1  Resolved the task numbering. Please review and approve. Thanks.


Static type bindings for Querying constructs

Issue
Today we use collection/index field names to define filter queries as:
const logicalFilter: LogicalFilter<Student> = {
			op: LogicalOperator.AND,
			selectorFilters: [
				{
					op: SelectorFilterOperator.GTE,
					fields: {
						"address.zipcode": 1200,
					},
				},
				{
					op: SelectorFilterOperator.LTE,
					fields: {
						balance: 1000,
					},
				},
			],
		};

where field names address.zipcode and balance are plain strings/object keys and not tied to schema. If a user renames/removes any of these fields, the filters would break and user would have to do a static search across the codebase to find potential usages.
Other than filters, there are places where collection/index field names are being referenced:

In SearchQuery
In FindQuery
and UpdateQuery

What needs to be done?

Potentially we can make all these generics and tie them to the collection class or interface type. We had a solution that was reverted in the past and covered most of the use cases except a couple ones:

It would be great if it can support dot notation for writing nested fields like - address.zipcode
The optional fields in collection/index schema were not getting typed. E.g. - if your collection schema defined id: string and an optional employeeName?: string property, the type checking was only available for id and not employeeName

Otherwise, we were getting type safety and auto completion in IDE with above.

/bounty 100

/bounty $75

/bounty $75

/bounty $75

/bounty $75

I think the recent filter improvement PR  https://github.com/tigrisdata/tigris-client-ts/pull/322 that we have merged, should fix this issue as well.

@akhill10 Yes, we just need to mark all these constructs as generics of that typed DocumentPath

@adilansari if no one is working on this, can you please assign this to me?
I am interested to work on this!

@logicncode let me know if you are still planning to work on this.


Firestore permissions missing in centralized deployment model

When using Firestore to hold state in the centralized deployment model, the service account for the scaler component does not have cross-project permissions to write the state to the Firestore DB.

Fixed by #77, closing.


InitialTargets cannot reference a target from a PackageReference .targets file

Steps to reproduce

Use or migrate a project with/to PackageReference (as opposed to package.config)
Reference a NuGet package that provides a .targets file which defines a Target
Reference that target in your Project tag's InitialTargets e.g. <Project ToolsVersion="15.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003" InitialTargets="MyTarget">, where MyTarget is the target defined in the package.

Expected behavior
MSBuild should run the MyTarget target as part of initial targets
Actual behavior
MSBuild cannot find the MyTarget target . This, however, works fine with package.config type projects.
error MSB4057: The target "MyTarget" does not exist in the project.
Environment data
msbuild /version output: <IP_ADDRESS>107
OS info: Windows 10 18362.657
If applicable, version of the tool that invokes MSBuild: Visual Studio 2019

The difference between package.config and ProjectReference here is that restore happens inside MSBuild in ProjectReference-driven builds. That means that they must be valid projects both with and without restore--because running Restore means running targets in the project.
In your case, before restore there's no MyTarget, because it's defined in the package. But MSBuild expects InitialTargets to exist, so it errors.
Can you hook the new target to a point very early in the build instead of as an InitialTargets? For example BeforeTargets="BeforeBuild"? That also helps in some other circumstances where InitialTargets causes problems, for example Visual Studio design-time builds (where you may or may not be surprised to run the new target).

Thanks for the detailed explanation, that is insightful. The issue here is that in my case, MyTarget defines variables in a PropertyGroup which are then used in the Condition attribute of some of my <Project>'s Target.
An example would be a Target which reads the .suo to extract the currently selected Solution configuration to make it available as a MSBuild variable (I think they are called macro ? Apologies if I got the terminology wrong).
As far as I could infer, the Condition clause is verified before running BeforeTargets or DependsOnTargets, which leaves no opportunity for the macro to be defined.

to make it available as a MSBuild variable (I think they are called macro ? Apologies if I got the terminology wrong).

There are two types of MSBuild variables, properties (single-valued, strings) and items (multi-valued, can be annotated with metadata). "Macro" is sometimes used in Visual Studio contexts but honestly I don't understand why.

As far as I could infer, the Condition clause is verified before running BeforeTargets or DependsOnTargets, which leaves no opportunity for the macro to be defined.

That's true, but there are many targets defined in common.targets that unconditionally run pretty early in the build. Is BeforeTargets="BeforeBuild" too late for you? That should be before any of the load-bearing targets, which generally hook onto CoreBuild.
Extremely detailed docs on target ordering: https://docs.microsoft.com/visualstudio/msbuild/target-build-order?view=vs-2019#determine-the-target-build-order

I had not completely understood the implications of your proposed solution. I have implemented it today, and my issue was indeed solved. Even better, my targets will now be run automatically after adding my NuGet package, without manually editing the InitialTargets.
Thank you for your assistance and for the educative process.

@rainersigwald I have a similar issue. In my target in the NuGet package, I add an item group with an Android archive file, something like this:

Jars\my.aar

However, it is not sufficient for my project to recognize this .aar file if I use BeforeTargets="BeforeBuild", I need to locate it earlier in the MSBuild process. Pre-VS 16.8 I used BeforeTargets="ResolveLibraryProjects", but with the latest VS update this does not seem to work anymore.
Any other target I could reference in my .targets file?


Audio not working on Vivo V2105 (Android 14)

Environment

OS: Windows 11
scrcpy version: 2.3.1
installation method: Windows release
device model: Vivo X70 (V2105)
Android version: 14 (newly updated)

Describe the bug
Audio doesn't work when mirroring
Please paste terminal output in a code block.

scrcpy 2.3.1 https://github.com/Genymobile/scrcpy
INFO: ADB device found:
INFO:     -->   (usb)<PHONE_NUMBER>000YS            device  V2105
C:\scrcpy-win64-v2.3.1\scrcpy-server: 1 file pushed, 0 skipped. 4.7 MB/s (66007 bytes in 0.013s)
[server] INFO: Device: [vivo] vivo V2105 (Android 14)
INFO: Renderer: direct3d
INFO: Texture: 1080x2376
[server] ERROR: Failed to invoke AudioRecord..
java.lang.NoSuchMethodException: android.media.AudioRecord.native_setup [class java.lang.Object, class java.lang.Object, class [I, int, int, int, int, class [I, class android.os.Parcel, long, int]
at java.lang.Class.getMethod(Class.java:2937)
at java.lang.Class.getDeclaredMethod(Class.java:2914)
at com.genymobile.scrcpy.Workarounds.createAudioRecord(Workarounds.java:293)
at com.genymobile.scrcpy.AudioCapture.startRecording(AudioCapture.java:116)
at com.genymobile.scrcpy.AudioCapture.start(AudioCapture.java:130)
at com.genymobile.scrcpy.AudioEncoder.encode(AudioEncoder.java:190)
W       at com.genymobile.scrcpy.AudioEncoder.lambda$start$0$com-genymobile-scrcpy-AudioEncoder(AudioEncoder.java:121)
at com.genymobile.scrcpy.AudioEncoder$$ExternalSyntheticLambda1.run(Unknown Source:4)
at java.lang.Thread.run(Thread.java:1012)
ARN: Demuxer 'audio': stream explicitly disabled by the device
[server] ERROR: Exception on thread Thread[audio-encoder,5,main]
java.lang.RuntimeException: Cannot create AudioRecord
at com.genymobile.scrcpy.Workarounds.createAudioRecord(Workarounds.java:324)
at com.genymobile.scrcpy.AudioCapture.startRecording(AudioCapture.java:116)
at com.genymobile.scrcpy.AudioCapture.start(AudioCapture.java:130)
at com.genymobile.scrcpy.AudioEncoder.encode(AudioEncoder.java:190)
at com.genymobile.scrcpy.AudioEncoder.lambda$start$0$com-genymobile-scrcpy-AudioEncoder(AudioEncoder.java:121)
at com.genymobile.scrcpy.AudioEncoder$$ExternalSyntheticLambda1.run(Unknown Source:4)
at java.lang.Thread.run(Thread.java:1012)
Please do not post screenshots of your terminal, just post the content as text instead.

In addition, the audio is completely fine in android 13, but it has failed since upgrading to 14.

Thank you for your report.
Was already fixed on dev branch by #4492.


Shakapcker v8, Yarn Berry and PNP as nodelinker

Hello,
According to the latest removal for readme.md, it seems that shakapacker v8 should be compatible with yarn v2 and pnp as nodeLinker, but it seems that the default configuration is not working.
Is there a way to get shakapacker working easily with yarn berry and pnp as nodeLinker ?
Thanks !
Expected behavior:
Out of the box support for Yarn Berry + pnp as nodelinker
Actual behavior:
The current install script will add
  "babel": {     "presets": [       "./node_modules/shakapacker/package/babel/preset.js"     ]   },
which is not compatible with pnp as a nodelinker
Small, reproducible repo:
Create a new rails app, add a package.json with yarn 4.2.2, enable yarn berry, install yarn package then add shakapacker.
The web server will not compile the JS.
Setup environment:

Ruby version:  3.3.1
Rails version: 7.1.3
Shakapacker version: 8.0.0

We're investigating, @tomdracz @G-Rath.

I'll have a look later this week, but you should be able to use our rails-template a reference on how you can do this

Ok so this is just a thing of Yarn PnP which makes sense in that context: configuring babel configs/presets via package.json involves it automagically requiring those files across the package boundary which is directly against Yarn PnP.
You need to use a babel.config.js in this case, which extends the Shakapacker babel config:
'use strict';

const defaultConfigFunc = require('shakapacker/package/babel/preset.js');

/** @type {import('@babel/core').ConfigFunction} */
const config = api => {
  const resultConfig = defaultConfigFunc(api);

  resultConfig.plugins = [...resultConfig.plugins];

  return resultConfig;
};

module.exports = config;

I'll do a PR to update the readme outlining that as a requirement if you're wanting to use PnP - the main other thing Shakapacker could do would be to switch to just using a babel.config.js instead of package.json, which I personally have no problem with doing 🤷

Thanks @G-Rath Merged! That might be enough in some setups to get this going.
We do have few places where we reference node_modules directly like
https://github.com/shakacode/shakapacker/blob/57786174df5085f027f8db63ac30b1c64df54c20/package/environments/base.js#L70
https://github.com/shakacode/shakapacker/blob/57786174df5085f027f8db63ac30b1c64df54c20/package/rules/less.js#L16
https://github.com/shakacode/shakapacker/blob/57786174df5085f027f8db63ac30b1c64df54c20/package/rules/stylus.js#L17
And not sure if any further changes would be required here 🤔 Think the module resolution should automagically account for PNP modules and exclusions might be redundant with PNP though


ability to change queue (for server side swift)

Why do you need this?

Not working with main queue (on perfect framework)

OK, well irrelevant of my question, I guess we should expose the same interface as Alamofire does.

Closed by b1d6b23a35b5fc08bf69ca8681315523ce37661e, thanks!


(maint) Unskip resolved device spec test

This commit removes the rspec pending designation as a fix for this test appears to have been released with puppet 6.5

I would like to understand why this is now fixed. Anybody know allready?

https://github.com/puppetlabs/puppet-resource_api/pull/180/files seems likely

Ohhh, I see. I didnt see a gem release, but it looks like 1.8.4 is in new puppet agent
[root@rdxhwxc72t09kbu ~]# find / -name resource_api
/opt/puppetlabs/puppet/lib/ruby/vendor_gems/gems/puppet-resource_api-1.8.4/lib/puppet/resource_api
[root@rdxhwxc72t09kbu ~]# puppet --version
6.5.0


Create settings component

A user must have access to change settings such as:

DONOR:

Username
Email
Password

LIFEBANK:

Email
Password

SPONSOR:

Email
Password

@murillojorge can you help us with a design proposal for these?
A user clicks on their profile icon. A dropdown menu is displayed with a "settings" option to edit their account info.

@murillojorge Telegram has this like setting component. We could hold something like:

Name
Photo
Edit profile (link to edit profile)
Notifications (settings)
Language
FAQ
Ask a Question

What you think @murillojorge

Name, password, confirm password, email (unless signed by gmail), profile pic (only for donors).


Allow distinguishing between the 4 math sizes/styles

Description
Can we check if the current display mode is display/inline etc. style? (similar to TeX's \mathchoice? Note that the exponential-time inefficiency of \mathchoice will not be a problem here because of delayed evaluation, not all 4 branches need to be evaluated they can just be kept unevaluated)
Perhaps we can just add to https://typst.app/docs/reference/meta/locate/ location object a field like math-style() which returns the object/function math.inline, math.display etc. which can also be compared. (and maybe return none if it's not in math mode)
Use Case

currently only op(x, limits: true) can "behave differently" in display and inline math mode. It may be desirable for someone to want to draw a custom operator and want it to have similar behavior.
sometimes if the user want to draw some kind of figure by composing multiple characters, fine-tuning would be useful, and the amount of space needed may be different in different modes.

In the future, the math sizes should be handled like normal set rule styles and then this would be covered by #763.


Enhance the player controller

The current ThorVG player has a somewhat amateurish design, the slider is not enough long to control the frames detail. (Just quick made for PC, even buggy).

I's possible that the ThorVG viewer can display a similar visual look and functionality at the bottom of the screen when any animation is enabled."

Expection result:


Pub sub refactoring

Implemented pub-sub pattern in TableFilter and related modules (issue #80)
Updated corresponding unit tests

All changes already reviewed and published in the npm next channel.
:+1:


[Question] Not able to login into TB Edge (localhost:8080)

Not able to login into TB Edge (localhost:8080)

Generic

Description
So I followed the docker installation in Linux for TB-Edge, and set all the necessary parameters in the yml file. I am receiving the attribute updates in demo.thingsboard.io as shown below.

But not able to login into the localhost:8080 with the credentials I've signed in with demo.thingsboard.io.

Environment

OS:  Ubuntu
Browser: Firefox 115.0.2

@AIOTEL-sanjanaa
There are two potential scenarios that might be causing this:

Sync Issue: The edge may not be synchronizing correctly with the server. To help us diagnose this possibility, could you please attach the logs from your edge container to this ticket?

Authentication Method: Your user account on cloud might not have a password, indicating you're using OAuth2 for authentication. Please verify that your user account on demo.thingsboard.io has a password set and that you're not relying solely on OAuth2 for authorization.

The password was not set. Thank you. Issue got solved.


Namespace Juicebox V1 contract constants

What does this PR do and why?

Move constants/contracts.ts to constants/v1/contracts.ts
Move ContractName typedef to constants/v1/contracts.ts. Delete contract-name.ts file.
Rename ContractName type to JuiceboxV1ContractName
Rename Contracts type to JuiceboxV1Contracts

Acceptance checklist

[x] I have evaluated the Approval Guidelines for this PR.
[-] I have tested this PR in all supported browsers.

First one for you @peripheralist 🏀

let's gooo! @aeolianeth think we could get away with using a more terse V1ContractName etc, instead of JuiceboxV1ContractName? "Juicebox" feels kinda implicit

let's gooo! @aeolianeth think we could get away with using a more terse V1ContractName etc, instead of JuiceboxV1ContractName? "Juicebox" feels kinda implicit (also applies to #436 )

@peripheralist I don't feel strongly! A simple V1 prefix is super repeatable as well for things like UserContext (we just change it to V1UserContext).
Changed and updated #436 and #438 🚀

love this @aeolianeth !


feat: Added code_rag.ipynb notebook for Code-RAG Generative AI use case

Added notebook to demonstrate Code-RAG use-case.

[x] Follow the CONTRIBUTING Guide.
[x] You are listed as the author in your notebook or README file.

[x] Your account is listed in CODEOWNERS for the file(s).

[x] Make your Pull Request title in the https://www.conventionalcommits.org/ specification.
[x] Ensure the tests and linter pass (Run nox -s format from the repository root to format).
[x] Appropriate docs were updated (if necessary)

@holtskinner Responded to comment on the notebook cells


Option to disable WIFI Prov BLE force encryption for Bluedroid (IDFGH-7649)

Is your feature request related to a problem? Please describe.
In the latest implementations made to the framework, it was implemented an option to disable WIFI Provisioning BLE Force encryption for NimBLE in this commit: https://github.com/espressif/esp-idf/commit/916533ef92ce8cf50ef53fa94262cf94a5124bb5 to avoid displaying the request of pairing while connecting a device to your smart phone. This works great when you are using NimBLE in your project but they forgot to add it to Bluedroid.  So since IDF v4.4, in applications where Blueroid is used, this pairing request is displayed all the time and could cause me some issues with my users as it also ask you to provide access to your contact list.
Describe the solution you'd like
Not limit it to NimBLE.
Additional context
You can have more context based on this issue that I once opened: https://github.com/espressif/esp-idf/issues/8880

@rahult-github why is this marked as done?

Hi @sch0bert  , it is part of automated process. Change is done and pushed on tree internally. It will be synced on github branch soon. The bot has updated this issue.

Thanks! Could you share the commit? I would like to do a cherry-pick

@rahult-github Will it be included in v4.4.1 or is it going to be released as part of another version?

Hi @sch0bert ,
Please find patch for the changes done attached which you can use currently, till the github tree gets synced. Please let me know if any issue is still observed. 0001-WifiProv-Extended-support-for-configuring-if-link-en.txt
The change needs to be backported to respective release branches , which i have initiated. There is a internal procedure to be followed to make this change part of next release.
Thanks,
Rahul

I applied the changes in the framework, it works. Thanks!

@rahult-github is v4.4.1 pairing with Bluedroid not working in iOS?

Hi @sch0bert , you mean to say that with attached patch included, on v4.4.1 , you still observe issue ?

Hello @rahult-github
Seems like it hasn't been integrated this fix into v4.4.1 yet
https://github.com/espressif/esp-idf/issues/7647
Could you confirm?

Hi @sch0bert ,
Sorry, but I am not clear, if you query for merging in 4.4.1 is for current issue , or #7647
The current issue #9201 :  So, as mentioned above,  we are in process of backporting change to relevant release branches. Once they are done, they would be available in the respective branch releases.
Issue #7647 :  The change you provided was picked and merged after release 4.4.1 was made. Hence you won't see it in that release version.
Also, i think, we should not cross connect the 2 issues on same page, Update on respective issue page is also easy for tracking purpose.
Thanks,
Rahul

Sure, sorry for the confusion. We can then close this issue.


no issue

@hudingchang i have the same issue "GPU memory usage increasing as images processing " seems like you resolved it, could you help me out? how did you do that


Generate readers and writers for set/list/map

There are stubs for that in the compiler which have to be implemented.
Map is serialized the same way as a struct, as a sequence of fields in the key, value order.
The list and the set are serialized as a sequence of fields, one for each contained value.

Added serialization for containers with a limitation: keys and set members have to be default-hashable. Relevant test cases are added (and commented out) to ThriftTest.thrift.
The root cause is that HashMap, HashSet and f64 are not hashable by default. Using hashable structs as keys could be possible if we check them for absence of maps/sets/floats as members.


move api,types,bin files to respective controllers

@mrunalp PR to the multitenant branch.
Just moves the bin/ scripts to respective controllers. Also moves the api/types.go and registry/registry.go into ovssubnet/ because the api and registry is specific to that only.

LGTM


Multiple 'New message' markers appearing

Running 56a9ad1 when it happened, not sure what caused it, or how to reproduce. It's a pretty recent commit, so @xPaw didn't think it would be fixed yet, so making this issue.
They appear in the correct position (ie. before new messages), but way too many of them. When clicking on the channel again they are all at the bottom, and not hidden. Will update when I can reproduce it.

I got same result when i was connected with PC and tablet. And only when i registered push notification on tablet. Getting private message ended up both PC and tablet lounge having new message spam.


Add for_each_concurrent

r+ modulo minor nit

When would one prefer this over stream.for_each(|item| tokio::spawn(f(item)))?


Set URL parameters on RestPool

An example: I have two routes: GET /user/ and GET /user/{0}. I want make a request each asynchronous on my project.
But... I found a bug! Looking the doc (below the Async section) and I can't pass the param on second route, because it only accepted on RestPool@addResource on fourth param.
<?php
use Snorlax\RestClient;
use Snorlax\RestPool;
use Snorlax\RestResource;

class PostResource extends RestResource
{
    public function getBaseUri()
    {
        return 'https://jsonplaceholder.typicode.com/posts';
    }

    public function getActions()
    {
        return [
            'all' => [
                'method' => 'GET',
                'path' => '/',
            ],
        ];
    }
}

$client = new RestClient([
    'resources' => [
        'posts' => PostResource::class,
    ],
]);

$pool = new RestPool();

$pool->addResource('postsByUserOne', $client, 'posts.all', [
    'query' => [
        'userId' => '1',
        'cache' => rand(11111, 99999), // bypass cache
    ],
]);

$pool->addResource('postsByUserTwo', $client, 'posts.all', [
    'query' => [
        'userId' => '2',
        'cache' => rand(11111, 99999), // bypass cache
    ],
]);

$resquests = $pool->send();

I talked with @pedrommone about this. We decided to pass an anchor parameters on the array of fourth parameter to create the request normally.


Extreme delay between picture on TV and on LEDs

Hi, I am quite new to hyperion. I got it working on my pi zero. It works smooth with my ws2812b leds connected to a nodemcu with wled.
Running videos from the pi itself (eg. retropi) is working flawlessly.
But running picutures from external sources like my receicer/playstayed resulted in extrem delay between picture and led lights (1-2 seconds).
I am using the following grabber which might already cause the issue (but I dont know): https://de.aliexpress.com/item/4001061243548.html?spm=a2g0x.search03<IP_ADDRESS>4c77527YNJQg&ws_ab_test=searchweb0_0,searchweb201602_0,searchweb201603_0,ppcSwitch_0&algo_pvid=181c3825-501c-4bd5-b134-3075fcab9429&algo_expid=181c3825-501c-4bd5-b134-3075fcab9429-28
I already tried a lot of changes in the config without success.
Is there anybody who can help?
Edit: User manual of the grabbers says:
Computer HW config reqs:
CPU: PC i5-3400 or above; NB i7-3537U 2.0GHZ or above
Graphics card: PC NVIDIA GT630 or above; NC NVIDIA GT735M or above
Run memory: 4G RAM

@Paulchen-Panther may I ask why you close my issue without any comment?

@Paulchen-Panther may I ask why you close my issue without any comment?

Follow the issue template please.


nosleep.enable() not working on bookmark saved as an application in iphone

Device Name: iPhone
Device Version: 11
Browser Name: Safari
Browser Version: latest
NoSleep.js Version: ^0.12.0

Expected Behavior
The noSleep.enable() function should prevent the screen from sleeping when the web application is launched from a bookmark saved as an application on an iPhone.
Actual Behavior
The noSleep.enable() function does not prevent the screen from sleeping when the web application is launched from a bookmark.
Code
useEffect(() => {
    const noSleep = new NoSleep();
    const documentOnClick = async () => {
      try {
        const a = await noSleep.enable();
        console.debug("no sleep enabled", a);
      } catch (e) {
        console.debug("no sleep prevented");
      }
    };
    document.addEventListener("click", documentOnClick);
    return () => {
      document.removeEventListener("click", documentOnClick);
      noSleep.disable();
    };
  }, []);

This is a known bug on iOS: https://bugs.webkit.org/show_bug.cgi?id=254545

The video method works both in PWA and Safari on iOS 17. But the library doesn't use it because navigator.wakeLock is available. So the library maintainer should either use both navigator.wakeLock and video, or provide an option for us to choose.
As user, you can copy the library code to your website and modify it so both methods are used.

@Finesse thank you for response, I tried that by break if else if else and it will enter into both condition but still has same problem on PWA any more ideas to fix or did you manage you to fix that

@tazo-taz No, I failed to make it work reliably. I chose to use my app in Safari, because wake lock works more reliably there. Though it requires user action to activate, so I made an infinite loop waiting for any user action, then waiting for a lock release, then waiting for user action and so on.


guide

Is there a guide some where ?
I opened the dotnet reactor but that is not same what you discriped in the discription.
Can you help me ?

Hello @GekkoFreak
No there is no guide.


fix(server): default to log fcc namespaces, unless env set

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

We have/had this set in far too many places. Anyway, now i can set DEBUG=-fcc:server in .env and not see those logs

Quick question - what's the behaviour with DEBUG=true in the .env under this change?

Quick question - what's the behaviour with DEBUG=true in the .env under this change?

Same as we currently have - all levels are logged.


Fix Sumita glass catalog download link

Turns out there's nothing wrong with SCHOTT - the website was probably down temporarily.
As for Sumita, they update the AGF file every so often (see https://www.sumita-opt.co.jp/en/download/) and this changes the SHA256 checksum, so we just need to update this whenever they do.

Codecov Report

Merging #248 (29001a4) into main (ba8c1f6) will increase coverage by 0.05%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##             main     #248      +/-   ##
==========================================
+ Coverage   54.68%   54.73%   +0.05%
==========================================
  Files          69       69
  Lines        6494     6486       -8
==========================================
- Hits         3551     3550       -1
+ Misses       2943     2936       -7

Impacted Files
Coverage Δ

src/GlassCat/sources.jl
87.50% <71.42%> (-3.41%)
:arrow_down:

src/Vis/Visualization.jl
7.37% <0.00%> (+0.11%)
:arrow_up:

src/Geometry/CSG/CSG.jl
83.52% <0.00%> (+0.31%)
:arrow_up:

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 ba8c1f6...29001a4. Read the comment docs.


use proxy feature

In connection with the blocking of Telegram in Russia, the possibility of using a proxy is very important.
It solve how to substitute the https://api.telegram.org end-point with proxy? #63 issue

Great, but monkey-patching Faraday::Adapter::NetHttp is a really bad idea. WDYT about subclass or some another way?

Anyway i'm not really happy about such workarounds in the gem. Why not to implement your own Faraday adapter? You always can pass it as an argument.

We can only transfer the adapter type but not the instance, unfortunately.

@lenferer why do you need an instance? Instances of adapters are being instantiated internally in Faraday.
You just need to register your adapter as a middleware see.


Do not attempt to start on failed Dial

Hello,
I'm cleaning up some forks on a repo. I'm curious would you have any interest in the following?
We have a case where it is better to fail if the Dial connection fails as we have a different agent controlling the start of the server.
Let me know what you think
@zach-klippenstein Thank you for the great project as well
Cheers

Hey, I'm doing some clean up of my stale PRs. @zach-klippenstein let me know if you're not interested. I'll be closing in a few days.

Closing
@zach-klippenstein Thank you for the awesome project!


add 'content' parameter to zend_common::license

The 'content' parameter would be helpful for things like 'vault_lookup'. We commonly use Vault for secrets, certificates and licenses.

Any particular reason this was closed? This would be a good feature to have, and I don't want to just steal your PR if you plan on still tidying it up and contributing.


detect "check" button instead of get from index

to support a question with audio (bcs the another "yellow" btn was added)

d2c87f6


[UI] Runtime Error in UI

Current Behavior

When the following command is run
make run-ui-dev

A server runs at localhost:3000.
When the following is done:

Go to http://localhost:3000/performance/profiles and click the icon shown in the image below

The error shown in the image below pos up:

2.Visit http://localhost:3000/smi_results. The error in the image below will be shown:

Desired Behavior

Proper error handling should have been done and corresponding flash messages should be displayed.
Environment

Browser: Brave
Host OS: Zorin 15(Ubuntu-18)
Meshery Server Version: Not sure
Meshery Client Version: Not sure
Platform: Didn't configure

I just ran make run-ui-dev as I was working with UI. Didn't run make run-local

Contributor Guides and Resources

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@K-Kumar-01 Thanks for testing but I checked twice the issue does not exist. Could you please do some checks like

login to 9081 before going to 3000.
Check the console/log and networks. If there some you can ask in slack with screenshots

Im working on the dev server right now, seems to work fine🤔.

I just ran make run-ui-dev as I was working with UI. Didn't run make run-local
You guys missed this 😅
This happens when meshery server is not running.
@K-Kumar-01 , most of the time, contributors won't run ui dev server without running meshery server, but still we have to handle the errors gracefully, will you create a PR for this ?

@Chadha93 @warunicorn19 I didn't run the meshery server as @sudo-NithishKarthik mentioned in the comment.
@sudo-NithishKarthik
Yeah, I will create a PR for this.
Also, should i do optional chaining additionally as an extra safety measure?

Ohh 😅, I had no idea. Also, sometimes(for me) if I run the ui-server and then the Meshery server, the UI server just terminates itself, without any warning or even a proper error message, maybe we could track that in this issue itself? @sudo-NithishKarthik @K-Kumar-01

Also, should i do optional chaining additionally as an extra safety measure?

Yes. Actually, we are working on re-implementing the UI according to this proposal, So, we will have a more robust implementation of Redux where the redirection will happen in a higher layer. So, this issue will be tackled along with that.

Also, sometimes(for me) if I run the ui-server and then the Meshery server, the UI server just terminates itself, without any warning or even a proper error message, maybe we could track that in this issue itself? @sudo-NithishKarthik @K-Kumar-01

Yes, @warunicorn19 , can you open an issue for that ?

Sure, I'll do.

Also, should i do optional chaining additionally as an extra safety measure?

Yes. Actually, we are working on re-implementing the UI according to this proposal, So, we will have a more robust implementation of Redux where the redirection will happen in a higher layer. So, this issue will be tackled along with that.

@sudo-NithishKarthik Just a question, why are we not using context API? Ig what we need from redux can also be done via context API. Also, there won't be need to install extra dependencies like react-redux or redux-middlewares.

Also, should i do optional chaining additionally as an extra safety measure?

Yes. Actually, we are working on re-implementing the UI according to this proposal, So, we will have a more robust implementation of Redux where the redirection will happen in a higher layer. So, this issue will be tackled along with that.

So currently, should I provide a basic logic check as someone may break the UI if they do no run the server. Later, that logic will get replaced by redux

We have solid use cases for Redux in our app, it might not be very obvious based on the current state of our system, but considering what we want to be able to do with Meshery UI in the future which would involve very complex state management , we will not be able to handle such level of complexity with React Context API effectively.

We have solid use cases for Redux in our app, it might not be very obvious based on the current state of our system, but considering what we want to be able to do with Meshery UI in the future which would involve very complex state management , we will not be able to handle such level of complexity with React Context API effectively.

So currently, should I provide a basic logic check as someone may break the UI if they do no run the server. Later, that logic will get replaced by redux

Yes


PR: Build arm64 versions of Spyder's standalone macOS application

Description of Changes
Add arm64 to build matrix

Update Python to 3.10.11 (arm64 builds not available for Python 3.9 from setup-python)
Update DMG name to include Spyder version and architecture

Note that py2app 0.28.5, I believe, should have updated the black recipe and resolved #19743. However, there is still an issue; see ronaldoussoren/py2app#518. The kluges from #21629 are still required.

Fixes #20232
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

@mrclary, to fix the error on CIs you need to change the Python version in this line
https://github.com/spyder-ide/spyder/blob/a45cdf49dff3406cb49a39802ce740f14d70d4db/.github/workflows/installer-macos.yml#L92
to 3.10 and then you need to make the necessary adjustments to this line:
https://github.com/spyder-ide/spyder/blob/a45cdf49dff3406cb49a39802ce740f14d70d4db/.github/workflows/installer-macos.yml#L108
by changing the Python version there too and the name of the file 629853fdff261ed89b74__mypyc, which is Python version dependent. For the latter part, you need to install Black in a Python 3.10 env and get it from it.

and the name of the file 629853fdff261ed89b74__mypyc, which is Python version dependent.

FYI, the glob pattern is identical for the Python 3.10.11 environment (629853fdff261ed89b74__mypyc.cpython-310-darwin.so), so no change was needed.


missing destroy

just as this is an example, but atomic haxe does not yet have it
https://github.com/AtomicGameEngine/AtomicExamples/blob/master/NewSpaceGame/Resources/Components/Bullet.js#L117

That link should now be https://github.com/AtomicGameEngine/AtomicExamples/blob/master/SpaceGame/Resources/Components/Bullet.js#L117


Twitter communities

Some community such as JakartaEE , Quarkus  has the twitter community . I suggest Microprofilers  create Microprofile community in Twitter

MicroProfilers or MicroProfile ?

I think it is MicroProfilers, Majid.
If others like you, think that creating a twitter community for MP is healthy, then Admins/volunteers will have to create such account and manage it.  That is where you need to sell it.  Alone you won't be able to push this forward. At least 3 people might help with neutral set or similar.
The MP Admin Access - Trusted Stewards comes into play for all MPWG INFRA.
I recommend you discuss it again next Monday in the marketing/ and community calls to see if you find supporters for the initiative.

MicroProfilers community has been created so we can post on it .

MicroProfilers commynity has been created in Twitter.

URL :
https://twitter.com/i/communities/1570499885488607232

We need some volunteer for manage MicroProfilers community in twitter

@majidmostafavi Hola,
As discussed on this week's marketing call, can you please draft a Twitter message that MP twitter can share about the new Twitter community group you created?
Doing so will bring awareness to anyone in twitter who might wish to learn about MP and to those who already know about MP.
I am happy to review your social text on that push before MP pushes it via its media channels.