sendEvent assert in Qt 5.15.2 about crossing different threads

Environment

OS Version: Ubuntu 20.04
Source or binary build?
main

Description

Expected behavior: Everything runs normally
Actual behavior: When starting Gazebo GUI compiled against Qt 5.15.2 it will complain with:

[GUI] [Err] [Application.cc:885] [QT] ASSERT failure in QCoreApplication::sendEvent: "Cannot send events to objects owned by a different thread. Current thread 0x0x55555622e390. Receiver '' (of type 'gz::gui::MainWindow') was created in thread 0x0x555555a1ebe0", file /mnt/ZLinuxDatos/qt-everywhere-src-5.15.2/qtbase/src/corelib/kernel/qcoreapplication.cpp, line 552

Steps to reproduce

Install Qt 5.15.2
Compile against it, using colcon build --cmake-args -DBUILD_TESTING=OFF -DBUILD_DOCS=OFF -DCMAKE_PREFIX_PATH=/opt/Qt/5.15.2/gcc_64 -DQT_QMAKE_EXECUTABLE=/opt/Qt/5.15.2/gcc_64/bin/qmake --merge-install
Go to the install folder
Run:

export LD_LIBRARY_PATH=/opt/Qt/5.15.2/gcc_64/lib
. setup.bash
gz sim shapes.sdf

Note: I'm running a custom-compiled (Debug version) of Qt 5.15.2; so it's possible additional debug checks are being ran that aren't made in the Release version

It is possible that this bug repros out of the box on Ubuntu 22.04 since it comes with Qt 5.15 by default.
Output
The error happens in GzRenderer::Render:
  if (gz::gui::App())
  {
    gz::gui::App()->sendEvent(
        gz::gui::App()->findChild<gz::gui::MainWindow *>(),
        new gui::events::PreRender());
  }

Qt does not like that sendEvent() call.
Full callstack:

1  __GI_raise                                          raise.c                   50   0x7ffff7a5100b
2  __GI_abort                                          abort.c                   79   0x7ffff7a30859
3  qt_message_fatal                                    qlogging.cpp              1914 0x7ffff14aaf32
4  QMessageLogger::fatal                               qlogging.cpp              893  0x7ffff14abc6f
5  qt_assert_x                                         qlogging.h                90   0x7ffff14a4d22
6  QCoreApplicationPrivate::checkReceiverThread        qarraydata.h              208  0x7ffff16cdc1d
7  QApplication::notify                                qapplication.cpp          2870 0x7ffff0977e10
8  QCoreApplication::notifyInternal2                   qcoreapplication.cpp      1063 0x7ffff16ce38d
9  QCoreApplication::sendEvent                         qcoreapplication.cpp      1458 0x7ffff16ce5c6
10 gz::gui::plugins::GzRenderer::Render                MinimalScene.cc           338  0x7fffca3afac8
11 gz::gui::plugins::RenderThreadRhiOpenGL::RenderNext MinimalSceneRhiOpenGL.cc  167  0x7fffca3cff89
12 gz::gui::plugins::RenderThread::RenderNext          MinimalScene.cc           732  0x7fffca3b2d17
13 gz::gui::plugins::RenderThread::qt_static_metacall  moc_MinimalScene.cpp      282  0x7fffca3d6cc3
14 QMetaCallEvent::placeMetaCall                       qobject.cpp               617  0x7ffff170150e
15 QObject::event                                      qobject.cpp               1314 0x7ffff17092a5
16 QThread::event                                      qthread.cpp               1012 0x7ffff14b535e
17 QApplicationPrivate::notify_helper                  qapplication.cpp          3632 0x7ffff096f1b8
18 QApplication::notify                                qapplication.cpp          2972 0x7ffff0978192
19 QCoreApplication::notifyInternal2                   qcoreapplication.cpp      1063 0x7ffff16ce38d
20 QCoreApplication::sendEvent                         qcoreapplication.cpp      1458 0x7ffff16ce5c6
21 QCoreApplicationPrivate::sendPostedEvents           qcoreapplication.cpp      1817 0x7ffff16d22d2
22 QCoreApplication::sendPostedEvents                  qcoreapplication.cpp      1676 0x7ffff16d2996
23 postEventSourceDispatch                             qeventdispatcher_glib.cpp 277  0x7ffff173d63b
24 g_main_context_dispatch                                                            0x7ffff072917d
25 ??                                                                                0x7ffff0729400
26 g_main_context_iteration                                                           0x7ffff07294a3
27 QEventDispatcherGlib::processEvents                 qeventdispatcher_glib.cpp 423  0x7ffff173cf28
28 QEventLoop::processEvents                           atomic_base.h             734  0x7ffff16cbdc5
29 QEventLoop::exec                                    qflags.h                  125  0x7ffff16cc22e
30 QThread::exec                                       qflags.h                  121  0x7ffff14b5294
31 QThread::run                                        qthread.cpp               616  0x7ffff14b5345
32 QThreadPrivate::start                               qthread_unix.cpp          329  0x7ffff14b6b44
33 start_thread                                        pthread_create.c          477  0x7ffff7f71609
34 clone                                               clone.S                   95   0x7ffff7b2d133

I'll keep researching

Oh what a hell this is.

Qt is right that the MainWindow does not belong to that thread.
However we guarantee both threads are not run simultaneously as we force serialization (since gazebosim/gz-sim#774)
Qt's intended way would be to call postEvent and wait for the results
However the documentation of PreRender explicitly states: /// It's safe to make rendering calls in this event's callback.

Due to how OpenGL works (context sticks per thread) any rendering call must happen in the rendering thread
Any code (Qt, OS, or Gazebo's) that relies on TLS (Thread Local Storage) could break if they are in the wrong thread

Hence we've run into a contradiction: Qt debug checks want the event to happen in the Main thread; Gazebo wants the event to happen in the rendering thread.
The simplest (and best) solution would be to move the whole rendering to the Main thread, which is something I've been wanting to do since gazebosim/gz-sim#774

OK I just confirmed this only repros with DEBUG versions of Qt.
The Release version will not trigger the assert.
I can also verify this by looking at Qt's source code in QApplication::notify:
#ifndef QT_NO_DEBUG
    QCoreApplicationPrivate::checkReceiverThread(receiver);
#endif

That check is left out in non-debug builds.
This makes me rest a little easier since there won't be a problem with Ubuntu 22.04


Move all dialogue export reader functions into utils

The problem
Currently there are multiple modules that read the dialogue export JSON files.
If we want to change the export structure we would then need to update all the different readers across DialogueKit.
Proposed solution
All these readers should be moved into one module that reads the JSON export and creates Dialogue objects. This way be only have one place to update the readers.
We will have to look at the structure of the JSON export and Dialogue to include the same information.

Done. See utils/dialogue_reader.py.


Centrally managed neurocommands

Installing neurocommands on our HPC works as expected, following the instructions from the documentation. This installs modules and containers in the user directory. What I would like to do is:

Install it for all HPC users, e.g. in /opt/neurocommand/. Can I ask our sys admin to do just that or is there something that needs to be adapted (e.g. the build script)?
Install it without changing the Python base environment. I know that can be done using venv (or conda or so), but would that require users to activate that environment before running neurocommands? Or is the Python venv not needed to run neurocommands and can I simply run the build script in the venv, fetch containers, deactivate it, and then load the module and run neurocommands in the base environment?

Thanks! Our /opt is mounted on all nodes, and yes, it would be nice if the
installation docs mention that the requirements are not needed for using
the containers, and that one best installs the python libs in a venv.
Thanks again, I will try to make things available centre wide, and will let
you know if we encounter issues. Cheers from Nijmegen :)

Sent from my phone
Op ma 29 jan 2024 23:40 schreef Steffen Bollmann @.***>:

Dear @marcelzwiers https://github.com/marcelzwiers
yes, you can install neurocommand in a global directory for all users on
an HPC. /opt/ might not be the best location and I would suggest a
network-mount that's available on all compute nodes.
The python environment is only needed for installing it, not for using it.
So you can do exactly what you said: Install the containers in the python
environment for neurocommand and then change back to the base environment
for using them. The users of the HPC would only need to add the modules
directory to their lmod modulepath (or run module use).
Let me know how you go with this and if the documentation needs more
details on this
Thank you
Steffen
—
Reply to this email directly, view it on GitHub
https://github.com/NeuroDesk/neurocommand/issues/261#issuecomment-1915699863,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADTUGL6R4S4CXSEY2EOXXC3YRAQMHAVCNFSM6AAAAABCPSNM6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGY4TSOBWGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great :) I added more details to the documentation.


PM - CTI Meeting Agenda

Overview
This issue tracks the agenda for the Civic Tech Index PM meetings.
Weekly Agenda
Agenda for the week of June 23, 2021
Agenda for Week  of June 30th, 2021
Agenda for the week of Jul 7th, 2021
Agenda for the week of Jul 14th, 2021

Resources/Instructions
CTI PM  wed meeting Agenda
CTI PM Meeting attending roster
CTI Launch Plan
Meeting minutes template

From a vote from the Jun, 17, 2021 meeting of all members, all members except 2 said NO to a meeting on Father's Day.

Agenda for the Jun 23rd PM meeting.

Discuss and prepare Launch plan #650

Agenda for the Jun 23rd PM meeting.

[ ] Review launch plan draft.
[ ] @Olivia-Chiong, can you please share the Django link - To update faqs.
[ ] We need to update the Slackbot reminder for Monday’s team meeting on the CTI channel.
[ ] We need to update the Zoom Meeting link for Monday’s team meeting.
[ ] Decide - Should we have a meeting for July 5th(since it's a federal holiday and the team was curious). Yes/No
[ ] I have created a PM Team attendance roster and Meeting Minutes Template (To follow up discussions with dev/UI UX teams).

Agenda for the Jul 7th PM meeting.
Project Updates:

The Zoom meeting link for Monday's CTI team meetings is the same as Thursday.
Calendar invite - updated.
Discussed upcoming Google Shared Drive migration process.
Guide for the migration process
Updated active/inactive team members in the roster

Issues to track:

Follow up with VRMS - CTI meeting time update
I have updated faqs on the Django database.#313

Agenda for the Jul 14th PM meeting.
Project Updates: Launch Planning

Define scope for Organizations page for a soft launch.
Discussed best practices for QA testing
Plan Bug bash - Test Plan Doc,  Created a Bug template.
Testing (manual, unit , functional) plan - Tools for test management, bug tracking, Automations

Important dates:

July 26 Bug bash with CTI team
July 28 Bug bash with Product managers COP.
Aug 15TH MVP.

Issues to track:

[ ] Installation of the Google analytics-Done
[ ]  Followed up with Cynthia and discussed On how to display links -Faqs on the Django database.#313 -Done

Agenda for the Jul 21st PM meeting.
Project Updates

PM meeting times have been updated at 1 pm every Wednesday. (Note: All PMs must make sure and reply to google meet calendar invite, yes/no. This helps to keep everyone informed of any changes to their schedule and saves time of others too)
Bonnie will be joining the team every Monday and guide us thru user testing and other project issues.
Discussed #19,
CTI Tag Generator document.

[ ] We need to create a communication style guide for the CTI project. ( might need help from UI/UX for this)
Communication (style) Guide for Document Creation template/examples

[ ] Migration of google drive.

Agenda for the Jul 28 PM meeting.
Project Updates:

[ ] We need to review the User testing document and sign off.
[ ] Finalize the testing tool - https://www.uxtweak.com/website-usability-testing-tool

Issues to track :

[ ] Contact us page
[ ] https://github.com/civictechindex/CTI-website-frontend/issues/746

Discussion:

Reviewed the user testing documents with Bonnie
Testng tool should be finalized by UI/UX team. (By using the tool  we should be able to know who the users are and document all those changes.)
Created Google form for contact us page - https://docs.google.com/forms/d/1pOgh_l-wcCwzsRSwG3DLewAmffA1sSVueDE9pSfm3V4/edit

Agenda for the Aug 04, 2021, PM meeting.
Project Updates:

[ ] Review and plan next steps for the launch - #650
Planning, process, Prioritize backlog
[ ] Quick retrospective with the team.- Template, Miro
[ ] Bug Tracking with the team on Thursday.- Discuss with PM’s
[ ] Post MVP planning.- Review and revise Onesheet.
[ ]  Need to work on Change management process- How do we track-  Change requests, approvals, Inform and update, etc.
Issues to track :
Last step -Tag generator #746- Review with UI.
User testing - #19

Agenda for the Aug 12, 2021, PM meeting.
Project Updates:

[ ] Review feedback from Bonnie:

Sizing of Issues
New team members
Tag Generator: Per Bonnie. "As the primary spokesperson/connector for Civic Tech Index inside the Code for America network I need the tag generator to work smoothly without explanation, so that people we introduce to it can share it with other people multiplying the effect.

Timing: we need a working version of the tag generator and to finish the brigade lead testing by the end of the month, because CfA / Democracy Lab and HfLA are going to run a workshop and publicity push with Brigade Leads to get them to tag their projects in time for the National Day of Civic Hacking on Sept 17th."

[ ] Assign New Issues from Prioritized backlog

Agenda for the Aug 16, 2021, PM meeting.
Project Updates:

[ ] Review feedback from Bonni: - Sizing of Issues Cont'd

Timing:
TAG GENERATOR
we need a working version of the tag generator and to finish the brigade lead testing by the end of the month, because CfA / Democracy Lab and HfLA are going to run a workshop and publicity push with Brigade Leads to get them to tag their projects in time for the National Day of Civic Hacking on Sept 17th."

[ ] Assign New Issues from Prioritized backlog

Project Updates:

[ ] Discussed Miro board insights and recommendations/issues.

Bonnie’s feedback  - Features that need to be fixed before launch. Some of which are the following:

[ ] Overview submenu to be converted into interactive slide deck instead of video.
[ ] Updating ‘Support The Index’ text to ‘Radical collaboration’.
[ ] Evaluate adding org functionality (backend solution vs workaround) then implementing path of least resistance.
[ ] Prioritize low effort level issues first and gear towards launch and iterate on the features later.

CTI PM meeting times updates:
Stakeholder(Bonnie) - weekly check-in - Sunday from 9 am to 9.55 am
PM’s - weekly check-in - Sunday at 10 am to 11 am
Issues to track :
#836

Adding the message here which  Kevin posted on slack to keep everyone updated.
Stakeholder meeting review on Sept 19th, 2021.
Discussed recruiting efforts for this project since we needed backend support. Here's the issue Bonnie created for that and it also serves as a template on how she wants future recruiting efforts to be done. https://github.com/civictechindex/CTI-website-frontend/issues/869
#869 Team Recruitment efforts
Overview
We need to have a viable recruitment strategy so that we always have a full team working on the project.
Action Items
Recurring Action items
☐ Document all of the recruiting platforms, including requirements
☐ location
☐ rules
☐ Instructions for how to post there
☐ Document all current recruitment posts on various platforms
☐ Annotate issue with posts filled
☐ Annotate issues when making posts to keep history up to date
☐ Write template language that can be used so that its easy for new PMS to know what to post
One time action items
☐ Reach out to Olivia and ask her to let us know the steps for posting on democracy lab, including what emails are tied to the account, where passwords are, etc. And document here
Resources/Instructions
Places to post
• Internal
• List of all the communities of practice
• Engineering
• HfLA engineering Community of Practice open roles board
• Instructions/rules
• HfLA Engineering Community of Practice SLACK channel
• Instructions: link your post to the open role on the CoP board. Do not post here for a position that is not tied to the board.
• External
• Democracy lab
• Instructions
Current posts
Backend engineer

Stakeholder meeting review for Oct. 3, 2021.
Site launch is aimed for Oct. 12/13. CTI team to create presentation for CfA lightning talk about the website.
Our priorities for site launch listed below:
Kevin/Levi

Levi - how many searches for topic tags are made per year

Kevin - index contributors check is not working. Create an issue to have live search of GitHub repos for orgs and identify civictechindex tagged projects

db to be reviewed and up to date
code for sloan to be swapped with civic software foundation

Levi - search results

affiliation topic tags should work
react sample
search should only search topic tags instead of description

Levi - Product to check how many unique organizations are in civictechindex on GitHub

after devs develop fix, check numbers that against it - should match CTI organizations count
identify which will be displayed on home page
check home page partners are using CTI tag
remove code for america off home page

Kevin - Civic Tech Partners (Home Menu)

update this to Index Contributors
footer, header, sitemap to be updated - discuss with Chali for details

Bonnie
- reach out to Geo regarding taxonomy link in brigade
- reach out to ED of code for KC to add CTI tags to their projects
Chali
- Levi - How to Add your Project section should have a summary before the instructions
- describe context and suggested tax
- text from Bonnie: “Your project is added to civictechindex when you add the topic tag: civictechindex. We recommend the following other tags: affiliation, subject area”
- obtain GitHub topic tag description and tweak for CTI
 - Kevin - Presentation on 10/12
    - update with new screenshots
    - clean up personas
    - send to Olivia/Bonnie before our next meeting to then work on script for presentation

 - New Favicon

 - Update Tag Generator
    - Create MVP version of single page process and share with Nandana (just boxes with context and step labels for MVP purposes)


: Adding scalar product, subtraction of CGPoints and linear interpolation.

Checklist

[x] New Extension
[x] New Test
[x] Changed more than one extension, but all changes are related
[ ] Trivial change (doesn't require changelog)

          2 Messages

  :book:
  Executed 148 tests, with 0 failures (0 unexpected) in 1.107 (1.332) seconds

  :book:
  Executed 136 tests, with 0 failures (0 unexpected) in 0.959 (1.107) seconds

  Generated by :no_entry_sign: danger

Current coverage is 36.79% (diff: 100%)

No coverage report found for master at 6110a95.

Powered by Codecov. Last update 6110a95...612fe6e

@Khalian Thanks!


Upgrade rpc module to v2.0.3

Summary
The actual version of rpc is depreciated and dates back several years.
The go.mod  use jcmturner/rpc.v0 v0.0.2 that is decrepeted . It would be interesting to upgrade to version jcmturner/rpc.v2 2.0.3
Proposal
Upgrade hdfs v1.1.4 to 2.3.0  it should upgrade the jcmturner/rpc version to the jcmturner/rpc.v2 2.0.3

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Would you like to submit a PR?

Hey, I would like to work on this issue. But don't know how to proceed with it.

Hi @terrytangyuan, I decided to work on this issue and already made a draft pull request! Still waiting for the tests to pass

Or run go mod tidy

I ran go mod tidy but it deletes the lines related to rpc.v2 v2.0.3 although I want to keep them.... Which is the redundant line you mentioned?

You can run make lint STATIC_FILES=false to reproduce

@terrytangyuan Hello can you assign this issue to me, I would like to work on it

As I check, this should upgrade gopkg.in/jcmturner/gokrb5.v5 on workflow/artifacts/hdfs/

This should be closed by https://github.com/argoproj/argo-workflows/pull/11543
@terrytangyuan Please help me close this.


Simplify input system

The recent input system refactor enabled batch event processing, but also made the interface a bit more complicated.
We should see if it could be made simpler again. Candidates for synchronization include 0-capacity bounded channels and condition variable.

Simplified! We are now using a polling-based system, without any need for threads and synchronization.


Framework crash in Version <IP_ADDRESS>

In the MappIntelligence Framework Version <IP_ADDRESS>, we have a crash at the startup of the app. The problem is, that after initialising we directly do a page tracking.
The exception is the following:
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[1]'
It crashes due to line 41 as the everID is nil, which we have configured during the init of Webtrekk, directly before the page track. It seems to be a multithreading issue, if I do the tracking later after initialisation it does not crash.

Code example: the App crashes if the configure directly executed after the init.

@Annika-EMP that is correct, the initialisation needs to be finished before the first page track request can be sent. Is there a reason you need the call inside the init/configuration method? You can either create the page request outside of the init/configuration method without and data loss, this is how the SDK is supposed to be configured. Or you can delay the execution of the trackPage method by adding the following line of code, this is not what we recommend though:
DispatchQueue.main.asyncAfter(deadline: .now() + 2.0) { // Change 2.0 to the desired number of seconds.    // Code you want to be delayed }

We encounter the same issue when deactivating anonymous tracking some time after init during regular runtime and directly afterwards e.g. trying to either track something or to get the current everId. Then everything crashes with the same exception as mentioned initally.

Thanks we were able to reproduce the issue and are working on a fix.

Hi all,
we have publish new version <IP_ADDRESS> ( https://cocoapods.org/pods/MappIntelligence ) Please try it and see if crash still appears. We have already tested on our end but we would also like your input.
Best Regards,
Stefan

Hi all,
I am closing this issue as we have released the new version now. If the issue still persist please re-open.


Add sdram_hw_test using BIST modules from LiteX

This commit adds an ability to perform memtest of RAM bypassing limitation of 32-bit address system bus.
It achieves this by usage of existing BIST modules from LiteX.
Right now, maximum supported size is dependent on selected memory module during build time.
After I add support to read memory size from the SPD, it should be mergeable.

I added documentation for sdram_hw_test and updates in LiteX and litedram commits include ability to read memory size from the SPD, so it should be ready to merge


Default color for changelog text from lab mr show  is not legible on white background

The default color for the merge change log text appears to be light grey, which is barely legible on a white terminal background.
Is there a config setting for .config/lab/lab.toml to change this?

There isn't.  It's because I'm lazy and assumed everyone would use a black/dark background :)  This is, FWIW, a dup of https://github.com/zaquestion/lab/issues/596


Could you compile an exe for Windows, please?

...

Made some packages for Win.
Please test it out respect to the versions.
If you find some bug, please report it in a new ticket.


Add Roadmap

What type of PR is this?
/kind documentation
What this PR does / why we need it:
add roadmap
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm


DEC-624: Bugfix item list on Safari

Why: Cannot see items for a collection in Safari
How: Problem was with material-ui Table. Symbol is undefined. We're pretty close to a deployment so I don't want to go through updating react and material-ui just to fix this, defined Symbol instead, see https://github.com/callemall/material-ui/issues/1531. Also fixed a problem with passing the key prefix to the pagination component. this.props.key was deprecated, need to pass as a different prop.

👍

👍


ENYO-5109: Fix to correctly reset marquee animation when children updates

Issue Resolved / Feature Added
MarqueeDecorator resets animation when children is updated which was checked by strict equality operator (===). This won't work for array type children. Proposing it to compare using shallowEqual to handle the case.
Enact-DCO-1.0-Signed-off-by: Stephen Choi<EMAIL_ADDRESS>

How does this reconcile with #1449 which removed our custom childrenEquals method which was trying to solve the same (class of) problem (iirc)?
@germboy - any thoughts on this one?

Doesn't it only resolve the string type children, though? ui/ToggleItem technically takes node type children, so it would fail to check if it contained <img> or <span> as part of the text.

That's a good point. What appears to allow this change to work is that React will return the same element if its props haven't changed. As a result, it passes shallowEqual even though they both objects because they're the same (strictly equal) objects. This might be change from prior versions of React.
For the common cases of Marquee, this is probably okay. Ultimately, MarqueeDecorator can't know from props if it needs to revalidate its bounds. A change in children doesn't necessarily equate to a change in dimensions. I think the only failsafe approach would be a combination of remeasuring on every update along with an optional method to trigger remeasures when wrapped component changes its state resulting in a change in dimensions.
We could investigate such a design but it's not necessary for this ticket. Would like to get @germboy's perspective on this before moving forward to validate we've avoid regressions.


GraalVM 21 flags?

I'm confused if the GraalVM 21 flags posted here are meant to be used or not due to " -XX:+DONTCOPYTHESEFLAGS ", could you please explain it to me?

-XX:+DONTCOPYTHESEFLAGS
Don't copy them, use the tldr to make your own flags


ORB Intelligence (Independent Publisher) (Student)

MLSA Student Hackathon Submission Connector Hackfest
Team Members:
Ankita Singh
Aaryan Arora

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)

[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Made the changes. Please review once again

@aaryan2134 We have reviewed your connector with our management and decided to move forward with the certification and production deployment of your connector.

[[certify-connector]]

@aaryan2134 Congratulations! The deployment of your connector and documentation page is now complete, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program. Thanks

@sriyen-msft Thank you so much for your guidance and for the whole certification process.
Thanks to @mamurshe for your help as well.

@sriyen-msft   so grateful to you for guiding us throughout the process
Thank you so much @mamurshe, to help us in understanding the faults in our project and suggesting the solutions
I learned a lot on this journey. Thank you, @aaryan2134, for being such an excellent partner in this journey.


Internet Explorer white screen

Bugs
MDL Elements does not show in Angular App using IE always. If developer console is opened the site appears, otherwise not.
The elements are invisible then.

What MDL Version are you using? (please be specific, e.g. major.minor.patch)
1.3.0

What browser(s) is this bug affecting (including version)?
Internet Explorer 11

What OS (and version) are you using?
Windows 10 Pro

What are the steps to reproduce the bug? Can you create a plunker/codepen/jsfiddle which reproduces it?
https://github.com/danieldaeschle/ferienwohnung-sonnenuhr

What is the expected behavior?
Should show the site.

What is the actual behavior?
Site is empty / White screen (MDL components does not render)

Any other information you believe would be useful?
It is an Angular 5 App

I'm having the same problem. I've done everything. Were you able to solve this bug?

No, it isn't compatible with angular :(
And it's so a nice css framework!


Both IOS and Android work perfectly in development environment, but not work in production environment

Both IOS and Android work perfectly in development environment (push notification can be received wonderfully), but not work in production environment.
IOS in production environment shows the following screen when loading the app:

But when I comment the usage codes and comment import OneSignal from 'react-native-onesignal';, the app can be loaded normally.
Android in production environment can open the app, but push notification doesn't work and OneSignal doesn't get the player ID.
My project is using ExpoKit (ejected from expo).
dependences from package.json:
"dependencies": {
    "expo": "^22.0.0",
    "moment": "^2.20.1",
    "react": "16.0.0-beta.5",
    "react-native": "https://github.com/expo/react-native/archive/sdk-22.0.0.tar.gz",
    "react-native-background-geolocation": "^2.10.1",
    "react-native-elements": "^0.18.5",
    "react-native-firebase": "^3.2.0",
    "react-native-onesignal": "^3.0.9",
    "react-native-vector-icons": "^4.4.2",
    "react-native-view-more-text": "^1.4.2",
    "react-navigation": "^1.0.0-beta.15"
  }

Anyone has any ideas? Thank you so much!

Hi @MelissaWeng this indicates that the OneSignal library isn’t being loaded correctly. Can you run using Xcode with a release profile and see if you get any errors printed to the console?

@Nightsd01 Oh yes, actually it's the profile problem. The app was in the store before I installed the oneSignal Library and there was no appropriate certificates for push notification. I generated new signed APK and uploaded to Play Store for Android, and made a new build for App Review for IOS, and did updates in the real phones (so certificates installed). Now it works!
I'm closing this issue.


[Bug]

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
When createSpark in SparkSqlEngine get sparksession or execute initialize.sql failed, we cannot get any error info in logs. At this time, currentEngine is None.
Kyuubi Server Log Output
No response
Kyuubi Engine Log Output
No response
Kyuubi Server Configurations
No response
Kyuubi Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

the issue title is missing


Missing CREATIVE_SDK_CLIENT_ID, CREATIVE_SDK_CLIENT_SECRET

When I run the installation command which is specified in the README, I get the following error:
Failed to install 'phonegap-plugin-csdk-image-editor':Error: Variable(s) missing: CREATIVE_SDK_CLIENT_ID, CREATIVE_SDK_CLIENT_SECRET

Full output can be found here: https://gist.github.com/vernondegoede/3a744ee6aa9222220074b5bb5e4acb4d
Where should these variables be set?

Nevermind... This was in the docs for the auth plugin.
https://github.com/CreativeSDK/phonegap-plugin-csdk-client-auth


Update Balances Pallet to use WeightInfo

polkadot companion: https://github.com/paritytech/polkadot/pull/1425
This PR updates the Balances Pallet to use the WeightInfo trait we have added as a part the new weights setup.
You will find the following changes in this PR:

Balances benchmarks remove all components since they are constant time benchmarks.
A new #[extra] tag is introduced into the benchmarks! macro, allowing users to mark benchmarks as "extra" and have them excluded from the default running benchmarks.
Added --extra feature flag to benchmarking-cli to enable users to run the extra benchmarks
Balances pallet updated to use WeightInfo trait for extrinsic weights
New weights file included in the runtime.
Some tests that were using hardcoded balance weight values instead use get_dispatch_info().weight since the weight may change in the future per CI automation.

The other runtimes (node-template, polkadot, kusama, westend) now also need an update in order to not fall back to the () dummy weights.

bot merge

bot merge

bot merge


Localhost refused to connect

Hello, eel community,
I created two projects say project A and project B.
I created an executable file for both projects. The executable file of project A is working fine but the executable file of project B is not doing fine. it says localhost refused to connect. Can you please help me asap? Tomorrow I need to submit to my client otherwise I am doomed.
project A - GitHub ===> https://github.com/AnirudhKumaran/wordfeeder
project B - GitHub ===> https://github.com/draptorhub/eniwas-desktop
I would be really grateful for your help.

Hey, thank you so much for the help it's doing fine now. I deployed the code on some other laptop and made an executable version it is doing fine. But the default 8000 behaviour is still happening on my laptop. But anyway thank you so much for your help. And I really love eel very much. A sincere thanks to this team and community for this wonderful contribution.


Remove fields not used from model

Description
We deprecated some fields but did not remove them from the model.
How to test?
Test the regular flow.
Expected behavior
Should work normally.

Opened by accident, sorry!


It is possible to include new Prisma Schema Folder preview feature compatibility?

I was curious about this, hadn't heard about the schema folder preview feature before. I tested and it seems like this package already works with that, which I guess is why you closed this.


[ADDED] ARM64v8 Build

Add linux/arm64 to the list of os/arch to build in the cross-compile script.
Also:

Drop Go 1.6.x from Travis build matrix
Add Go 1.9 to Travis build matrix (but exclude megacheck
for this version since there seem to be a problem with the
tool on Travis with that version)
Use Go version 1.8.3 for Docker files
Rename Dockerfile.win64 to Dockerfile.all to build all executables
for which we plan to have corresponding docker images. This will
be used in nats-docker repo.

Resolves #466

Coverage remained the same at 91.527% when pulling 3179575b15c463f5765d14ac64ba4c6aca52b716 on add_arm_build into f2854ee461ed40b2b4d47a431a72618707b77723 on master.


RUNNING PLUGIN: php_code_sniffer / PLUGIN STATUS: FAILED

I am working on a mac running Mamp Pro and...
I have seemingly got everything set up but have run into a few issues, firstly adding the PHPCodeSniffer plugin - everytime I ran Composer Update it errored, so I modified the composer.json to require dev and changed the phpcs version to '1.*' and that seemed to work but now when I run a build I just get:
Working copy created: /
RUNNING PLUGIN: composer
Using --prefer-source flag
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
  - Removing composer/installers (v1.0.8)
  - Installing composer/installers (v1.0.9)
    Loading from cache
  - Removing mjohnson/uploader (4.5.1)
  - Installing mjohnson/uploader (4.5.2)
    Loading from cache
Generating autoload files
PLUGIN STATUS: SUCCESS!
RUNNING PLUGIN: php_code_sniffer
PLUGIN STATUS: FAILED
BUILD FAILURE
Removing build.

Any suggestions?

How did you fix it?


A11y: TreeView expand/collapse is not accessible for keyboard user

Describe the bug
TreeView expand/collapse is not accessible for keyboard user
How to reproduce
Steps to reproduce the behavior:

Go to Stackblitz
Turn on the VoiceOver (CMD+F5)
Navigate to root node by pressing TAB
Press ENTER to expand the TreeView
Observe TreeView is not expanding and same goes for collapse

Expected behavior
User should be able to change expand/collapse state via keyboard. And screen reader should announce the expand/collapse state
Versions
Clarity project:

[ ] Clarity Core
[x] Clarity Angular/UI

Clarity version:

[ ] v3.x
[x] v4.x
[ ] v5.x

Framework:

[x] Angular
[ ] React
[ ] Vue
[ ] Other:

Framework version:
ie: Angular 11
Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional notes
Add any other notes about the problem here.

@Tushar2098 - thanks for reporting this we gonna investigate it a little bit more.

Clarity cares deeply about accessibility, and we work with our accessibility team to identify and triage issues. We have an internal tracking system to identify and verify accessibility issues, which has captured this already. Some issues with Clarity Angular are difficult to solve without causing side effects or breaking changes, and we have to balance concerns with fixing issues and breaking changes. As we build out Clarity Core, we are able to address these root cause accessibility issues and recommend adopting the Clarity Core implementations as they become available. We are closing this issue as we have the issues tracked internally.
The Clarity Core tree has better handling of Enter and Space keys, as can be seen from this example:
https://clarity.design/storybook/core/?path=/story/stories-tree-view--interactive
It is currently released in latest Clarity Core releases, but it still a BETA version.
We'll further investigate if we can deliver this fix for Clarity Angular too.


activity: refactor children tracking

Instead of a list, The backend now sends down a map of child activity summaries, but only a single level deep. This allows us to sum the child counts easier.
In addition, we now track combined "status" more actively by updating parents when childrens' statuses change. Now, the nav bar dots will immediately clear upon being seen.
Also takes into account threads for DM dots and sidebar filtering so that it is more accurately reflected.
Fixes TLON-1983
PR Checklist

[X] Includes changes to desk files
[ ] Describes how you tested the PR locally (test ship vs livenet)
[ ] If a new feature, includes automated tests
[X] Comments added anywhere logic may be confusing without context

@patosullivan if it's a ship that has already run %activity, you might need to run :activity &noun %migrate


Welcome！


Make solution file from workfile

Maybe you don't want to use the entire workfile. Maybe you want to generate a ton of work, and then make a solution file that is approximately 1 day worth of computation.
The flow should be something like this

Fixed in 260d41f092f9d2096f425c15ad694f0c8e0e7a09


[bug] Some nodes (2) could not be loaded. This can be ignored, but go to http://<IP_ADDRESS>:8188/mtb if you want more information.

Describe the bug

It doesn't show any error reason, it just tells that it failed to load, so what can I do to solve the problem?
Reproduction
No response
Expected behavior
No response
Operating System
Windows (Default)
Comfy Mode
Comfy Portable (embed) (Default)
Console output
No response
Additional context
No response

I'm having this issue as well, ComfyUI portable. I had this running earlier but needed to rebuild to accommodate dw pose. Even though there are no errors on Color Correct (mtb) the gamma/contrast, it is no longer working as well, its not effecting the image. This seems like it could be related to a Pytorch/Cuda/Onyx version mismatch with mtb nodes, but I couldn't find versions for these in the dependencies to try to figure out what might work.
Python 3.11.8
Pytorch 2.3.0+cu121
CUDA 12.1
onnxruntime-gpu==1.17.1

These nodes are just deprecated that's a warning not an error

It would be great if your example workflows (1, 2, 6, and 8) could be updated to avoid referencing deprecated nodes. Thanks.

See old notice here, I will make it clearer and provide much morer examples for all the nodes: https://github.com/melMass/comfy_mtb/blob/991af4f45ff8c660b2c45466bb219186699170ed/README.md#optional-nodes


agora_uikit: ^1.0.4 is incompatible with agora_rtc_engine: ^6.0.0

@Meherdeep @maxxfrazer @Ascenio @hxingpax
while updating the dependencies, I am getting this issue.
agora_uikit: ^1.0.4 is incompatible with agora_rtc_engine: ^6.0.0 because agora_uikit: ^1.0.4 is still using the older 5.0.2 version

Hi @waqadArshad, thanks for the message!
We're aware of this and an updated version is underway on this branch:
https://github.com/AgoraIO-Community/VideoUIKit-Flutter/tree/ngsdk-release
When updating to use that SDK, the changes for devs using the Flutter Video UI Kit will be very minor, despite RTC 6.x being very different from 5.x

@maxxfrazer so does this mean I should use 5.0.2 for now?

@waqadArshad Please use the updated v1.1.0


Card data arrived => Warning if greater than 3 minutes / default value from RPi-Reporter-MQTT2HA-Daemon is 5 minutes

Issue being Addressed
The card data arrived timestamp changes regulairly to warning (aka orange).

The default value from the RPi-Reporter-MQTT2HA-Daemon is 5 minutes.
Enhancement
Adjust the threshold in the lovelace-rpi-monitor-card also to 5 (or better 6) minutes.
Alternatives
Make the value (_colorReportPeriodsAgoDefault) adjustable for the user.
Additional information
No response

I'm using the coloring on my RPi farm to keep me aware of what's working as intended or not.
But I certainly could make this user configurable as you suggest.  Good idea!
Give me some time to get this done...


Expanding arrays

Does the plugin support expanding arrays?
I end up with the following structure in Elasticsearch;
content_types: {
  array: [
    "tag1",
    "tag2"
  ]
}

Objects however are expanded correctly.

Gotcha. Seems like a fair trade-off 👍


Broken 3DS (Power on Issues)

I don't know if this is the place to talk about this, but I did everything right on the guide after I was done I used MicroSD Transfer and after it shutdown my 3ds without warning!  Can you help me?

arm9loaderhax.bin is missing/corrupt on SD root. Replace it with the one from Luma3DS' .zip package.

Thank you so much!!!!!


build arm v7 docker image

LGTM.
I will build a new release then. Thank you very much for taking your time to create this PR 😄

thats brilliant... thanks :)


ID localization for access-application-cluster list-image

This PR translates /docs/tasks/access-application-cluster/list-all-running-container-images/ as mentioned #22296

Thanks for the translation! I've left some reviews to improve clarity @ariscahyadi

Thanks, @phanama for your review. I have addressed them all. Please check!

/lgtm
/approve


Plugin/mlflow custom flavor - fixed small bug

Thanks for adding the custom flavor option to mlflow. I tested it and only the flavor specific kwargs somehow got mixed up and tried to be called somewhere else which lead to a fatal error.
Changes

renamed class attribute to avoid conflicting calls

How I tested this

added unit test that doubles as minimal example for reproducing the error

Notes

only tested the DataSaver (due to internal legacy structure, DataLoader is not utilised/hard to implement)

Checklist

[x] PR has an informative and human-readable title (this will be pulled into the release notes)
[x] Changes are limited to a single goal (no scope creep)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.

Thanks @jernejfrank ! For the pre-commit failure you just need to do something like:
pip install pre-commit
pre-commit install
pre-commit run --all-files
And then check that change in  and you should be good to go. Thanks for the fix!

I'll plan to push this in the release on Monday. If you don't get to the pre-commit before I'll do it. Thanks!

Thanks @jernejfrank !


Enable Renovate

please configure app/permission to the repo/org at https://github.com/marketplace/renovate

Not familiar with this app? What is it exactly. Would be helpful to get a sense of the benefit to adding this in.

submit PRs keeping current various dependencies
please see https://github.com/gliptak/hcl2json/pulls?q=is%3Apr+renovate+

submit PRs keeping current various dependencies
please see https://github.com/gliptak/hcl2json/pulls?q=is%3Apr+renovate+

We are already using the Edgetest action which does exactly this.

I do see pytz somewhat behind at https://github.com/capitalone/datacompy/blob/develop/requirements.txt
https://pypi.org/project/pytz/#history

I do see pytz somewhat behind at https://github.com/capitalone/datacompy/blob/develop/requirements.txt
https://pypi.org/project/pytz/#history

Not following on your pytz comment?

The Edgetest configuration needs to be updated to push a new requirements.txt which is why you are seeing that with pytz.


Resize function stretch the canvas instead of resizing

Hi and thanks for this very good development!
I am trying to include three-dxf in a React component and making it responsive. I managed to achieve that by deleting the previous
canvas and rendering a new one but I noticed you wrote a function resize for the renderer:
https://github.com/gdsestimating/three-dxf/blob/20e14cbd13735e16a1b691f7b864936dd688fb74/src/index.js#L176
I gave it a try since it would save some processing not having to redraw the dxf but I only see the canvas being stretch, it doesn't keep the aspect ratio and when I zoom or move, it goes back to the previous size.
It seems like it doesn't update the camera parameters, but I am really not an expert with three.js

Hi
You need to call instance of ThreeDxf.Viewer something like
dxfViewer.renderer.setSize(width, height);
and then update that with
dxfViewer.render(scene, camera)


A uniform approach to the storage of passwords and usernames

We access credentialset throughout the code, up and down. Put it at the top or the bottom.

Not clear this is more than a clean up issue - if even that.


Ref to a field

Package version eg. v8, v9:
v9
Issue, Question or Enhancement:
How can I add a validator for fields password and confirmPassword?
Code sample, to showcase or reproduce:
at nodejs I am using Joi for the case and I can add ref to another field
{
 password: Joi.string().required(),
 confirmPassword: Joi.string().required().valid(Joi.ref('password'))
}

Hey @gangstaJS
your looking for the eqfield tag eg. validate:"eqfield=ConfirmPassword"
Please see the docs all tags should be thoroughly documented there.

Please let me know if this worked for you :)


updated to laminas-form v3

Q
A

Documentation
no

Bugfix
no

BC Break
yes

New Feature
yes

RFC
no

QA
no

Description
This PR updates laminas-form from v2 to v3. To keep the functionality of this package, doctrine/annotations has been added to the dependencies, while laminas-code has been removed. This allows annotation parsing with laminas-form v3, as laminas-code did with v2.
From my point of view this should be considered a BC break, as laminas-form v3 introduced typed parameters and return values. Furthermore, the switch from laminas-code to doctrine/annotations may - at least at some lower level, i.e. when users add custom annotations - bring some BC breaks as well.
Therefore, this should go in a 2.0.x, though that branch does not exist yet, hence the target of 1.3.x currently.

@froschdesign This the follow-up as discussed here: https://github.com/laminas/laminas-mvc-form/pull/6#discussion_r658754975

I am waiting for the workflow to start, but there is no code to check! 😄

@driehle
Thank you again for your time and your help! 👍


Can  we add the new smollm models

These small models would be a valuable  addition to ollama.
https://huggingface.co/collections/HuggingFaceTB/smollm-6695016cad7167254ce15966

+1

I'd also like to see the SmolLM models in ollama. They are well suited to local execution on consumer computers and for learning.

The llama.cpp convert scripts needs to be updated, https://github.com/ggerganov/llama.cpp/issues/8568.

Looks like a dup of #5731


Bug: ..#tracking falls in love.

Describe the bug

Steps to reproduce

ArchiveBox version

replace this line with the *full*, unshortened output of running `archivebox version`

SSH
22575

Describe the bug

Steps to reproduce

ArchiveBox version

replace this line with the *full*, unshortened output of running `archivebox version`

what in the wold is going on here


OpenJDK16.AdoptOpenJDK version <IP_ADDRESS>

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

[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This is a duplicate of AdoptOpenJDK.OpenJDK.16.


DatePicker.setShowWeekdayNames doesn't update

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=286

Fixed


Use the schedule page position to sort the schedule pages

Problem
We're currently sorting schedule pages by day instead of by position
Solution
Sort them by position. We have the position in Firestore but not in the DB and domain models, so we need to add that, and then do the fix (and test it).
Test(s) added
No but updated existing one that was affected
Paired with
Nobody

After discussing with @fourlastor we've decided against going with this, we'll keep sorting by dates and drop positions from our days model. This PR will just retain a couple minor refactors.


Check PGP signatures of binary hashes

See https://github.com/ManyMath/pgp-dart/issues/1

See https://www.getmonero.org/downloads/hashes.txt


finetuning_data and TEST_INDICES Release

Could you release the TEST_INDICES and finetuning_data used in graph_utils.py and generate_finetuning_dataset.py?
Thanks!

Thanks for pointing this out! We've released the TEST_INDICES in graph_utils.py. We will release the finetuning_data soon.

Thanks for pointing this out! We've released the TEST_INDICES in graph_utils.py. We will release the finetuning_data soon.

Thank you and very look forward to it!

Thanks for kindly sharing the training_set.zip!
By the way, could you also share the video data in the training_set.zip?
The current version includes belief_{num}.pik, camera_data_{num}.pik, graph_{num}.pik, init_graph.pik, instanace_colors.pik, env_info.pik, actions.pik.
if training_set.zip can include the vision data, such as video-related RGB and depth file, it would be helpful to reproduce and conduct multimodal training/finetuning experiments (I guess the path is in task_{episode}/script/0/).
Thanks!


GoDaddy Link Branding Workaround

Description of the change: Provide workaround for customers who use GoDaddy as a DNS host when following link branding docs
Reason for the change: GoDaddy appends the domain to the end of CNAME records
Link to original source:https://sendgrid.com/docs/ui/account-and-settings/how-to-set-up-link-branding/

Closes #

Hello @drespinoza,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @drespinoza,Thanks again for the PR!                                         You have earned 13 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


Added "interactable" attribute to portals which can disable the player portal interaction

Hi,
I added this feature especially for the "look-through" portals, which imo shouldn't feature the custom manipulation through the portal, so I added a boolean field which can prevent this and implemented it in the mirror class. No other portal is affected :)

Mirrors are meant to be see-through. Please implement this as an NBT option only and I will consider merging it

Found that kind of confusing that the mirror was meant to be reach-through tbh, is this just a personal preference or something else?

is this just a personal preference or something else?

Of both myself and @qouteall. He implemented subroutines into the code that specifically detect mirrors and alter the raycasting slightly so you can target blocks through them.

Oh, ok.

Maybe it could be a config option to not aim through mirrors? This can definitely be altered client-side.

Sure. Should I open another PR or add that to the current PR here?

You can add it here. It should be somewhat straightforward.

Is this fine?

Yep, looks good to me. Let me make a couple more review comments on your other code.


Hitting a key from the tokenSeparators array without typing anything errors

Error occurs on the examples page with the following code:
$(".js-example-tokenizer").select2({
  tags: true,
  tokenSeparators: [',', ' ']
})

Type a   (space) or , (comma) to trigger the error.
[Error] TypeError: null is not an object (evaluating 'data.selected = true')

Version 4.0

This is a duplicate of https://github.com/select2/select2/issues/3593, which will be fixed in 4.0.1.


spawn_local demands unnecessary static lifetime

Summary
spawn_local requires 'static lifetime. I have a type T owned by a shared pointer, Rc<RefCell<T>>. Doesn't this share pointer have static lifetime? After all, T does not get dropped until Rc<RefCell<T>> gets dropped.
Additional Details
I have a function here that takes a shared pointer to an ReactiveComponent and calls its render method whenever a message is received from the MPMCFutReceiver

fn spawn_listener<T>(
    component: Rc<RefCell<T>>,
    reciever: MPMCFutReceiver<Render>,
) where
    T: ReactiveComponent,
{
    let component_clone = Rc::clone(&component);

    spawn_local(reciever.for_each(move |_| {
        //-- Use `component_clone`  to render --

        Ok(())
    }));
}

The compiler complains:
error[E0310]: the parameter type `T` may not live long enough
   --> src/view.rs:110:5
    |
100 | fn spawn_listener<T>(
    |                   - help: consider adding an explicit lifetime bound `T: 'static`...
...
110 |     spawn_local(reciever.for_each(move |_| {
    |     ^^^^^^^^^^^
    |

This does not make sense to me because the closure takes ownership of the shared pointer and therefore T, so surely T itself doesn't need to meet the `static requirement since the shared pointer does?

The error is correct, you need to add a where T: 'static bound.
The reason is because T might be a reference, like &Foo. Consider this:
let foo = Foo::new();
let foo = &foo;
let rc = Rc::new(RefCell::new(foo));

In this case rc has the type Rc<RefCell<&Foo>>, which means that the Rc is not allowed to outlive the variable foo.
But when you spawn a Future, it is executed asynchronously, so when the Future runs, the variable foo no longer exists, so that would be a use-after-free (which is not allowed).
To prevent that, spawn_local requires a 'static lifetime, so that way it prevents you from putting a reference inside of the Rc<RefCell<T>>.

I believe @Pauan is right here in that T: 'static is required, so I'm going to close this.

Given that spawn_local is offered by a thread-local executor, there is an upper bound on when the Future may be executed; this is very similar to an iterator that captures a reference and then carries that reference’s lifetime with it. As long as the executor lives longer than the Future, all is good, right?
So would a patch be acceptable that added a lifetime parameter to LocalPool and used that as a lifetime bound in spawn_local?

@rkuhn The problem isn't the lifetime of the executor (the executor already lives basically forever). The problem is with the stack.
Futures are run asynchronously, therefore they will always outlive the stack, therefore they cannot contain stack references, because if they contained stack references they could trigger use-after-free by accessing the stack reference after the stack frame is gone.
That's why they must be 'static, which restricts them to contain only owned values or 'static references (no stack references). This isn't something which can be "fixed", it's just fundamentally how Futures (and Rust) works. It's the same reason why thread::spawn requires 'static.
Consider a program like this:
{
    let x = vec![];

    let r = &x;

    spawn_local(async move {
        sleep(5000).await;
        println!("{:?}", r);
    });
}

Here is what happens:

It allocates x on the stack.

It creates a reference to x (called r).

It creates a Future (using async) and moves the reference r into it.

It spawns the Future asynchronously.

It reaches the end of the scope, so it drops x.

The Future will wait for 5 seconds and then print r.

Do you see the problem? The problem is that immediately after spawning the Future, it then drops x, because it reached the end of the scope. This happens long before the Future prints r. Therefore, when 5 seconds have passed, the Future would be accessing r after x had been freed. That's use-after-free, which is undefined behavior.
This is exactly the sort of undefined behavior that 'static prevents.
The important thing to understand is that spawn_local does not wait for the Future to finish, it starts the spawning process and then immediately returns () (before the Future has finished), which means the Future is running in parallel with your code. That's what "asynchronous" means.
Why doesn't it wait for the Future to finish? Because that's impossible with JavaScript, there is no API for blocking.

What you’re saying is all correct for an executor that runs at some later point. The LocalPool, however, lives on the stack: I have a use-case (in a test) where the LocalPool is started in a function and dropped within that same function, with a Future being run in between. This Future is needlessly required to have 'static lifetime.

Just to be very clear: your point 4 above is not correct, LocalPool is not asynchronous — if you don’t run the pool on the current thread nothing happens, and nothing will happen after the LocalPool is dropped, either.

@rkuhn As I said, LocalPool and block_on and similar do not work in Wasm, because JavaScript does not provide a blocking API. But hypothetically, if they did work... you could just use them instead of spawn_local.
The spawn_local function is equivalent to tokio::spawn or runtime::spawn or LocalSpawn (all of which accept 'static).
They are designed that way because being able to run Futures concurrently in the background is very useful, and is the normal way of using Futures. Using block_on (et al) is only recommended inside of unit tests, because it blocks all Futures (and all computation in general) on the current thread.
Therefore, spawn_local cannot be changed, but even if it could change, it shouldn't, because the current design is correct and idiomatic.

My apologies, I searched the web for spawn_local and found this issue but didn’t check which repo it is in: I was talking about LocalPool while you were talking about something else entirely.


Fearure Request: Camera placeholder

Add a Camera placeholder where it shows the camera where i am pointing my camera is for better color picking.
Maybe in Bottom Left or Bottom Right and able to get hidden

Hmm, was thinking about that too. I'll implement it. Thanks!
Will close this issue after implementation 👍🏼

Hmm, was thinking about that too. I'll implement it. Thanks!

Love to see it!

https://github.com/Rahuletto/pickr/commit/33c5accb3e2aa5fd2fae3c787937f7c5088aad17 - feat: adding camera feed into it
Adds the feature of displaying the live video feed from the camera to focus without guessing it 👍🏼
Thanks for the feature request


Fix typos in FSharp.Core

Make this a separate PR since it will change generated XmlDoc shipped with FSharp.Core.dll.

Great - looks good to me, thanks

LGTM


How to get events which users participate in

I want to know how to get users who participate in a specific event.When I call :urlname/events/:id/attendance,it shows :
"errors": [
{
"code": "authorization_error",
"message": "you are not authorized to take attendance"
}
]
I need help!Waiting for answer!

You need to be a member of the group's lead team in order to take attendance for event. You can issue a request to /2/events with fields set to self and inspect the self.actions for "attendance" to detect whether or not you would be able to take attendance given a set of credentials


Workflow to check CLA/DCO by contributors

Description
This workflow supports the validation of code contributions by third parties.
Changes Made

add workflow

Related Issues
Fixes #69
Checklist

[x] I have used a PR title that is descriptive enough for a release note.
[ ] I have tested these changes locally.
[x] I have added appropriate tests or updated existing tests.
[ ] I have tested these changes on a dedicated VM or a customer VM [name of the VM]
[ ] I have added appropriate documentation or updated existing documentation.

Open questions to be discussed with @MarcoPignati:

Are we opting for CLA or DCO?
If we are opting for CLA, which terms? cf. https://contributoragreements.org/
Should we collect signatures in this repo or in a central repository (we can share signature across repositories in this way)?

Having seen what you did I'd say CLA. And of the CLAs, the recommended Fiduciary License Agreement (FLA). It requires a website where to collect the signatures, so we can point to this repo... Concerning this repo or a central, I'd start simple and use this.

https://github.com/alliander-opensource/ospo-code-scanner

several suggestions and open source tools to be used in this ppt from Alliander


can trtllm-build process on cpu?

System Info
NVIDIA A800 40G
Who can help?
@byshiue
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
can trtllm-build process on cpu? like parameter load_model_on_cpu in convert_checkpoint.py
Expected behavior
none
actual behavior
none
additional notes
none

trtllm-build requires NVIDIA GPUs to timing the kernel perf.

trtllm-build requires NVIDIA GPUs to timing the kernel perf.

i get it, thanks


buildCtlRuntime: better config management

Previously, it was not possible to override only part of a config item (for example, only setting the port for postgres). This PR fixes that, and as a bonus, also allows you to refer to the final value of config items.
A few examples to clear this up:
# Correctly changes the dbConnectionString
buildCtlRuntime ({ postgres.port = 9001; postgres.user = "asdf"; })
# Refer back to the final config:
buildCtlRuntime (final: { node.port = postgres.port + 1; })

Like this, or do you want more doc?

@Synthetica9 Would be cool if both ways to call buildCtlRuntime were mentioned (with attrset and with function as argument).

I'm going to close this and re-open it from a branch on the main repo


Palindrome Linked List

💥 Proposal
Given a linked list check if it is a palindrome or not in C++
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Route table support is incomplete

Hey guys,
Since 0.5.0 terraforming seems to support route_table resources. But there is an issue with the level of support implemented. It only works correctly for route tables w/o any routes. The reason is that current implementation does not generate routes in tfstate, causing terraform to try to change the routes in AWS route tables while they are perfectly correct.
I believe this is an issue and terraforming should actually generate route blocks in tfstate.

I'm currently trying to implement the routes tfstate and will update the ticket on the progress.

This issue is fixed at #146 and released as v0.6.2 :rocket: Thanks :exclamation:


1st README example does not pass TypeScript compiler

In the first example, the series of Foo, Bar and Baz fails to compile for TS.
Here is a screenshot of the error after copy / pasting.

It's complaining that type is required on each entry, but is not present in the example.
I am not sure if the following is correct, but it passes TS
import Vizzu, { AnimTarget } from 'vizzu'

let data: AnimTarget['data'] = {
    series: [
      { name: 'Foo', values: ['Alice', 'Bob', 'Ted'], type: 'dimension' },
      { name: 'Bar', values: [15, 32, 12], type: 'measure' },
      { name: 'Baz', values: [5, 3, 2], type: 'measure' }
    ]
  }

let chart = new Vizzu('myVizzu', { data })

If type is not actually required, perhaps make it so in the types. If it is, could you add it to the example?

thanks, fixed 690a55b

fixed in v0.3.3


how to run it locally with terminal?

how to run it locally with terminal?
thanks.

Hi @AliveDD -- which sample (which folder) are you trying to run? :)

@AliveDD The instructions here - https://codelabs.developers.google.com/codelabs/app-indexing/#12 - say to use adb shell command.  Are you familiar with adb shell?


Button frames should be made deconstructable

Button frames are currently unable to be deconstructed on-top of being intangible - meaning that their removal is downright impossible since you can't even destroy them the old fashioned way.

Will be fixed by #33195

Will be fixed by #33195

Great! I'll be closing this then.


Build basic controller to push configuration into Asterisk nodes

Build basic controller to push configuration into Asterisk nodes

Feel like this is a dupe, but, available in this tagged release of the asterisk controller: https://github.com/dougbtv/vnf-asterisk-controller/releases/tag/v0.0.2


DTEND should be non-inclusive

Hello,
I have discovered the bug in this library at the latest version 2.1. According to the RFC at 3.6.1:

The "DTEND" property for a "VEVENT" calendar component specifies the non-inclusive end of the event.

So for example, in this calendar:
BEGIN:VCALENDAR
VERSION:2.0
PRODID:+//IDN ki-wi.cz//NONSGML Player Lighter//EN
BEGIN:VEVENT
UID:197857
DTSTAMP:20190920T025726Z
DTSTART:20200101T000000
DTEND:20201231T235900
SUMMARY:simple event
END:VEVENT
END:VCALENDAR

When you make a selection, that includes only the DTEND, it shouldn't contain any event or occurrences, but it does. DTEND + 1 works as expected and no event is returned.
const date = new Date('2020-12-31T23:59');
const match = expander.between(date, date);
return Boolean(match.events.length || match.occurrences.length);

This code will return true, but it should return false.

I don't have time to look at this now, but if you provide a PR, I'm happy to consider merging

I understand. Luckily, this precision is not a must-have for the app I'm working on, but for someone it might be quite a problem, so I decided to at least report it.

IcalExpander.between() currently also has events in the result which end exactly at the beginning of the requested range.
This is probably a related problem.
My current workaround is to shrink the requested range by 1 millisecond from each end.


Fix runas requirement

runas 2 doesn't build with a recent node.js
Closes #8922

/cc: @zcbenz
Also, do you think we should add the latest node in our build matrix on Travis?

Also closes #8508 for real this time.

@zcbenz: do you think we should add the latest node in our build matrix on Travis?

Sounds good to me.

:ship:?

@mnquintana: I'll open a new branch with this commit as the base + Travis integration to make sure everything works as expected in the new node version.

Thanks, @g2p!


Update feature-flags-in-expo.md

🚀 Deployed on https://66fc74daf50664ea6c1b8a0c--cozy-fox-0defec.netlify.app


fix: update bouncycastle bcpkix to mitigate CVE-2023-33201

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


horizon-eda: 1.3.0 -> 1.4.0

Motivation for this change
Update horizon-eda to 1.4.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks!

Thanks!


self.neg_pos_ub 's meaning is confused

In RandomSampler.py, the arg neg_pos_up is supposed to be the upper bound number of negative and positive samples. But in the code, it's actually the upper bound of  (the number of the negative samples)/(the number of the pos samples). Why?
BaseSampler.py line 91
if self.neg_pos_ub >= 0:
            _pos = max(1, num_sampled_pos)
            neg_upper_bound = int(self.neg_pos_ub * _pos)
            if num_expected_neg > neg_upper_bound:
                num_expected_neg = neg_upper_bound

@Icecream-blue-sky There may be ambiguity in naming. If you are interested, you can create a PR to modify it.


Style updates

@jread-usgs Heres the caption class, was noticing the .list class started getting used in a different fashioned than I had imagined it and is causing a couple oddities in the app. So this caption class really should only be used for captions in the app.
The figure now switches out for mobile as well.

@mwernimont can you add a screen shot of how it looks?


Auto Downlod via clipboard

Auto Download via a clipbord.
Work with downloadpath rule seperat for every snglnsuppotet side or optional General download folder.

I'm migrating this issue to #17. Further updates will be provided in issue #17.
Closing this issue.


Spelling fixes for iOS strings

Why is the formatting UTF-16LE?

@freerdp-bot test
@comradekingu no idea, ios is currently unmaintained, would be great if someone could take a look at the client from time to time


Custom base URL for OpenAI API calls

This pull request introduces support for configuring a custom base URL for the OpenAI API. This allows users to specify an endpoint different from the default https://api.openai.com/v1.
The primary motivation for this change is to enhance flexibility and support scenarios where the API might be hosted on a custom server or a different URL.
Linked Issues

resolve #59
resolve #16

Summary by CodeRabbit
Release Notes

New Features

Introduced a setting for users to specify the base URL for OpenAI API calls, enhancing configurability.
Added a method to dynamically update the API endpoint based on user-defined settings.

Improvements

Enhanced the BrainModule and AIService to support a customizable base API URL.
Updated settings management to include flexible validation for API URLs.

Chores

Added a .gitignore file to keep the repository clean from unnecessary files.

Huge thanks to @jeffbeaulieu for this, I will do a check when I get home (currently in delivery room waiting for my baby to be born O_O), or I'll just check + merge while there's some downtime hopefully soon
This was a much requested feature, from a first glance it looks great, I'll test + ship ASAP! Cheers and welcome to being a SystemSculpt contributor :D


libmuser: Organise lm_ctx_create and friends

Current state of lm_ctx_create is quite messy. This cleans it up
considerably, moving the pci config space creation to a separate helper.
It gets rid of the 'extended' bool from dev_info, meaning that a client
that wants to have a 4KiB config header should specify it on the
corresponding region.
This may all need reviewing later if we decide that other regions may be
always handled by libmuser, in which case we can hide config, vga (and
maybe rom?) entirely from our clients.
Signed-off-by: Felipe Franciosi<EMAIL_ADDRESS>

I think this PR should have been broken into smaller commits to make the reviewing process easier.

It probably could... but I only had a few hours to kick off the work and I wanted to get this out there sooner rather than later. My bad. Now that you've reviewed this, let's try to get this in and I'll try to make future commits smaller.


Cannot type $emit on class style components

As mentioned in https://github.com/johnsoncodehk/volar/issues/1061#issuecomment-1073547589 it seems it is currently not possible to type $emit using the method suggested in https://github.com/johnsoncodehk/volar/pull/750#issuecomment-1023947885. The base interface uses a this return type and assigning void gives a TypeScript error that an arbitrary type can be passed.
Is there updated methodology for this? Or is this something that has fallen out of favor?

Hi @barfurth, I'm not really understand the problem, could you provide a complete code sample?
Just in case, make sure you have at least define a empty object type for $props to make to component is a valid JSX element. So it will not show red color in other components.

Certainly @johnsoncodehk, I added a component that is based entirely on https://github.com/johnsoncodehk/volar/pull/750#issuecomment-1023947885 linked above.
Here is the component: https://github.com/barfurth/volar-no-types-repro/blob/main/components/HelloWorld.vue
The auto-completion works with this, but it does not compile. When you open the component the $emit part is marked as an error.
Property '$emit' in type 'HelloWorld' is not assignable to the same property in base type 'object & Record<never, any> & Vue'.
  Type '{ (e: "my-event-1"): void; (e: "my-event-2", arg1: string, arg2: number): void; }' is not assignable to type '(event: string, ...args: any[]) => this'.ts(2416)

There seems to be some problem with this assignment. The $props and the $scopedSlots work perfectly. This is related to the Vue interface definition of $emit I believe.
interface Vue {
  // ...
  $emit(event: string, ...args: any[]): this;
  // ...
}

@sbarfurth just have a quick look, you could change this code to avoid error, but I'm not sure will it effect other types.
  $emit!: {
-    (e: 'my-event-1'): void
-    (e: 'my-event-2', arg1: string, arg2: number): void
+    (e: 'my-event-1'): any
+    (e: 'my-event-2', arg1: string, arg2: number): any
  }

@sbarfurth just have a quick look, you could change this code to avoid error, but I'm not sure will it effect other types.
  $emit!: {
-    (e: 'my-event-1'): void
-    (e: 'my-event-2', arg1: string, arg2: number): void
+    (e: 'my-event-1'): any
+    (e: 'my-event-2', arg1: string, arg2: number): any
  }

Hi there.
I tried this approach, but doesn't work as well.
Was necessary use of // @ts-ignore


Fix routes to work with normal npm start

It turns out the routes work in the normal npm start if you navigate using the app.  The issue occurs if you are typing the routes into the browser.

Yes, this needs to be fixed. Its a bug.

Fixed. Replaced custom routes implementation with React Router.


Why doesn't the Compphys notebook show any interactive plot?

It is working on my notebook. Have you checked using quantum lab or any other platform?

It is working on my notebook. Have you checked using quantum lab or any other platform?

No, I tried Google Colab and CodeSpace


Update meta plugin

V2 only. Replace deltas from the chron util with Discord timestamp format.

Updating scope of this issue to include the reformatting of other areas of the commands to conform more to the video one.


[Feat] Design Footnote Component

We have no design of the footnotes now; therefore, there is no css style applying to it.

Please visit: https://www.1chooo.com/blog/make-your-titles-more-readable-react-wrap-balancer

Since we have integrated Tailwind CSS, updating the design of the Footnote is not as urgent at the moment.


Selection and mouse click problems with multiple monitors

I have problems selecting text within PuTTY tabs which appears to be related to me having multiple monitors. With the SuperPuTTY window on either my 2nd or 3rd monitor, I can't select text at all. I am also unable to right-click. It appears mouse clicks are not registering at all.
On my primary monitor it works, though only within the top left area of my screen (about 7/8 of the way from the top and 3/4 from the left). Initiating selections or right-clicking in PuTTY tabs outside of that area do not work.

Additional info: SuperPuTTY <IP_ADDRESS> and PuTTY 0.66/0.67

I got windows 10 (but had windows 7 as well) with multiple monitors and didn't have any issues selecting. I put this down to windows drivers or some 3rd party application doing nasty stuff (seen Team Viewer do this). Sounds like a rendering or window manager issue really.
I just tried <IP_ADDRESS> and on all 3 monitors it works fine for me though windows 10 but I don't think being windows 7 would've made any difference.
Highly recommend fiddling with video drivers and/or check any window hooking applications if any.


fget_object raises error when local path has nested directory

The method MockMinioClient.fget_object doesn't work as expected when the file_path argument has a nested folder structure and doesn't exists.
https://github.com/oussjarrousse/pytest-minio-mock/blob/157693dad9caf505d0e460d59f2c42375875815c/pytest_minio_mock/plugin.py#L818-L864
Expected behavior:
fget_object should create the parent directory of file_path before attempting to get_object.
Also, this is the implementation from minio-py
https://github.com/minio/minio-py/blob/fd3571aa9b9402f050192ad4c6c087d2806ca9eb/minio/api.py#L1116
My proposed solution:
Add this line before calling self.get_object:
os.makedirs(os.path.dirname(file_path), exist_ok=True)

Let me know what do you think about this! Or is there any reason why makedirs is purposely not called?

Thank you for opening the issue, and for the fix suggestion.
I actually identified several other interface issues with fget_object().
Usually I encourage users to submit a pull request. but this time I am fixing it myself because it is a small issue and I am experimenting with a branching and CI/CD pipline.
Here is the pull request
https://github.com/oussjarrousse/pytest-minio-mock/pull/32
Let me know if you have any comments.
I will merge it and publish the new version over the weekend.

I just published a new release v0.4.14
https://pypi.org/project/pytest-minio-mock/0.4.14/
closing this issue!


Upgrade to react 15

Hi, Would be nice if the react-player will also require 15.0.1 (at the time of writing) and not depend only on 0.14.
Thanks!

I have upgraded to latest React locally and I've had to fix a couple of issues that came up. I also wasn't sure what to do about peerDependencies to prevent warning for people who want to remain on 0.14.0. For now I've used "react": "^15.0.1 || ^0.14.0" which hopefully does the job. I suppose "react": "*" would work too, like karma-mocha does with mocha.

ah ok! That is indeed the best way. So is that already in 0.5.7 version of react-player?

So is that already in 0.5.7 version of react-player?

Nope, I haven't pushed that yet. I want to make sure everything works ok, and I'm rejigging some of the test structure while I'm at it. Once I'm happy I'll push 0.5.8, or maybe 0.6.0 because of all the dependency changes.

Nice work man! Let me know if I can help you with something on my fork.

0.5.8 has been published 👍

WoW! 👍  That was quick and Thanks!

I couldn't see any more problems, and I didn't want to sit on it for too long ;)


Update to <IP_ADDRESS>

Updated Summernote to 0.8.15. I also fixed the update script given the fact that no asset was linked to the latest release so it can handle the process of an existing zip file in the tmp folder in this case.
I let you check this out @luciuschoi 😉

Hello! Any update about this pull request? Thanks in advance! 😉

Thank you for your PR.


KV storage implement

Now this project using the CloudFlare KV as the implement of the KV storeage for user's view count.
But this service is based on the CloudFlare and it's not available with native CRUD queries.
I prefer using the database which is more generic and not dependent on some specific platforms / companys.
Redis should be a good choice - open source and simple queries like get / set.

[ ] Find a Redis service or some RaaS (Redis as a service).
[ ] Code change with redis:// connection
[ ] Consider whether we should keep the CloudFlare KV data
[ ] Fallback?

deplicated issue. /close


[RETARGET] Fixed router app start bug without routes.

App doesn't crash when it doesn't have any routes to serve, and the
returned error is discarded, not checked. This checks if the error is
not nil and crashes if it should because no routes means nothing to do.

@sevki thanks for contributing to Revel!
Please retarget your PR to the develop branch per our contribution guidelines.
After the PR is targeting the correct branch, I would like to know the steps to reproduce this issue so we can confirm what problem is being solved and that it is working as expected.
Thank you!

Sorry about that. Will do.


Revert "Fix Tabs for latest browsers"

Reverts bitpay/bitpay-browser-extension#56

what's the reason for this? Otherwise, I can close this PR


Make the ExperimentDict compatible with QISKIT

The experiment dict is anyways heavily inspired by qiskit_experiments.framework.ExperimentData. So it might be quite nice to have mostly compatible keys...

This object is just waaaay to complex after further analysis. We will not get deeper into this.


Thundering herd problem in shared timers

Pr #376 (knowingly) introduced the "Thundering herd problem" for the shared timers. If multiple worker threads have the same timeout (based on the shared timers) and all attempt to schedule the same thread-safe actor because of it will pointlessly waste resources, as the actor can only be scheduled once.

This should be resolved when fixing #284.

Fixed in #423.

Reopened by commit e8c0e158c838f04e1b402e7e3f904520e724d819.


[Snyk] Upgrade @contentstack/utils from 1.3.10 to 1.3.11

Snyk has created this PR to upgrade @contentstack/utils from 1.3.10 to 1.3.11.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 22 days ago.

Release notes

Package name: @contentstack/utils

1.3.11 - 2024-09-05
 Fixed Embedded asset reference to html

1.3.10 - 2024-07-31
 Fixed the issue of JSON RTE Serializer not applying Bold and underline formatting to assets embedded as links in the HTML output

    from @contentstack/utils GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Closing this PR as its updated to latest version in this PR.


Heltec Wireless Tracker  >> POWER OFF   Button  ?

Hello,
is there any solution possible   powering OFF   a Heltec wireless Tracker....actual only  a message is seen in the Display  when
using Configuration Menue.  Ask because  charging the battery  in ON Mode is  not usefull.
tnx any info
herbert

Hi Herbert,
as far as I've tested the Heltect Wireless Tracker, if you charge the battery will also power on the board.
powering off (deepSleep) is done by pushing 3 times the user button.

Hi Ricardo,
good  to know :-))   mni tnx  agn  for the excellent work you are doing  !


[AutoPR hanaonazure/resource-manager] Add Create HanaInstance API

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5945

(message created by the CI based on PR content)

Codecov Report

Merging #5359 into restapi_auto_hanaonazure/resource-manager will decrease coverage by <.01%.
The diff coverage is 31.81%.

@@                              Coverage Diff                              @@
##           restapi_auto_hanaonazure/resource-manager    #5359      +/-   ##
=============================================================================
- Coverage                                      48.33%   48.32%   -0.01%
=============================================================================
  Files                                           6557     6557
  Lines                                         188657   188701      +44
=============================================================================
+ Hits                                           91183    91197      +14
- Misses                                         97474    97504      +30

Impacted Files
Coverage Δ

...aonazure/azure/mgmt/hanaonazure/models/__init__.py
56.25% <ø> (ø)
:arrow_up:

...re/azure/mgmt/hanaonazure/models/os_profile_py3.py
100% <100%> (ø)
:arrow_up:

...hanaonazure/models/hana_management_client_enums.py
100% <100%> (ø)
:arrow_up:

...azure/mgmt/hanaonazure/models/hana_instance_py3.py
100% <100%> (ø)
:arrow_up:

...anaonazure/operations/hana_instances_operations.py
47.19% <9.09%> (-8.68%)
:arrow_down:

Continue to review full report at Codecov.

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

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5459


Maximum number of items is not sufficient to prevent out of gas error

2022-06-06_Audit_Report_-_DAO_DAO_v0.1.pdf
Check issue 6 of the attached audit.

Resolved as we no longer instantiate items along with the core contract: https://github.com/DA0-DA0/dao-contracts/pull/315


Initial attempt at adding MSBuild props generation.

I've been tasked to fix some high priority bugs in VS by the overlords; I'll review this when I'm done that.

Closing this in favor of Garrett's #18


Weird dotted screen with X mousepointer instead of desktop

I followed all the steps in this manual, but I don't get the same results.
The first time I connected with my RDP client (KRDC) I got a error message "error - problem connecting", I pressed ok. The error message went away and I got the screen as shown in the picture after "Open your local remote desktop client and connect to the IP address or DNS name of your Linux VM. Enter the username and password for the user account on your VM as follows:"
I left the xrdp module on the default value: sesman-Xvnc. Since the manual doesn't mention anything about changing the module to something else.
After logging in, I did not see the xfce desktop environment appear as expected. But instead I got a weird "dotted" screen with a X mouse pointer.

I followed every step of this manual from a freshly installed Ubuntu 16.04 LTS VM. During that install I already provided a password and selected the RDP as allowed incoming traffic.
I also tried to connect with Remmina Remote Desktop Client, but there I get the same weird dotted screen with X mouse pointer.
Could you add to troubleshooting what I should do now?

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

ID: 077ee673-528a-d893-5832-7656b91dbec8
Version Independent ID: 2180e845-1ead-3b47-03be-c26aee00c6f7
Content: Use Remote Desktop to a Linux VM in Azure
Content Source: articles/virtual-machines/linux/use-remote-desktop.md
Service: virtual-machines-linux
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

Thanks for the feedback! We are currently investigating and will update you shortly.

I'm so sorry. I missed a very important step from the tutorial.
I only ran the command "sudo apt-get update", but forgot the command: sudo apt-get install xfce4
Now I followed every step, everything is working just like in the tutorial.

Sorry Micah

@microbart all good! Thanks for updating us and glad to hear it is working now :)


SubcommandRequired in yml

I can NOT run if I have the following line in the yml file, what's the correct way to config this?
setting: "SubcommandRequired"

Thanks! I'll put this in the queue, but it should be an easy fix.

I'm sorry, I was confused about what you were asking.
In your YAML file, putting the following will work just fine:
settings:
    - SubcommandRequired

The instruction you gave does not work, it just has no effect.
Similarly: global_setting: "ColoredHelp" will at least make help for main command colored.
but
global_setting:
    - ColoredHelp

would not make anything colored.

Still got the following result:

thread 'main' panicked at 'unknown AppSetting found in YAML file', ../src/libcore/option.rs:699

Ah found the typo bug!
Thanks! :+1:

#594 fixes this

Correction #595 fixes this

Is it pushed to crates as a new version?
Or I should install it directly from git repo?

I will upload v2.9.3 to crates.io when I get home tonight. I'll also post here once the new version is uploaded :wink:

great!

v2.9.3 is out on crates.io


Add project functionality accessible to regular users

You can add a project even is you are not logged in. This functionality should only be available to logged in users.

Fixed by #79


The type or namespace name 'WSA' does not exist in the namespace 'UnityEngine.VR'

Hi,
I got a HoloLens for a research project at school. I have downloaded a Toolkit here.
I am on Unity 2017.2.0b3. I believe I set everything right... But at my console, I am getting this error, only this one.
Assets/HoloToolkit-Examples/SpatialUnderstanding/SpatialUnderstanding-FeatureOverview/Scripts/SpaceVisualizer.cs(8,22): error CS0234: The type or namespace name WSA' does not exist in the namespace UnityEngine.VR'. Are you missing an assembly reference?
I googled for the issue. I couldn't solve the problem. Any idea?

Unity has changed that namespace from UnityEngine.VR to UnityEngine.XR. Everything under the namespace should be the same, from my experience!

Thanks! It works. But... I am facing another problem... Typical. haha

Thanks for the heads up.

The namespace change is one part of it.  Some of the handlers have changed in the SDK as well.  This resolves about 90% of the issues.

I'm going to go ahead and close this. Please be sure to use the Dev-2017.2 branch.


Static fields in changed code are lost upon reload

I am not sure whether this is expected behaviour or not, but I could not find any reference in the documentation to static fields not being supported.
I am trying to reload a static class that has a static Material field that is assigned on startup via reflection.
The static class content looks like this:
private static Material material;

public static void DrawReticleCircle(in ReticleDrawArgs args)
{
    var position = args.WorldPosition;
    // ...
    Graphics.DrawMesh(mesh, trs, material, 0);
}

static void OnScriptHotReloadNoInstance()
{
    Debug.Log($"Post-reload, asset is {material} (null={material==null})");
}

After changing the contents of DrawReticleCircle, the script is reloaded but the value of material is lost.
The debug log reads: Post-reload, asset is  (null=True)
I would expect for the value to be copied over, or for the new code to reference the field in the old assembly (if this is possible?).
Is this a known limitation or a bug? If it is a limitation, I think it is worth pointing out in the documentation.
Thanks.

Hi - yes it'll be a limitation. When reload happens your new code is technically in class/new assembly with changed name.
Static values are not reinitialized.
I'll leave this one open as it's possible to reinitialize those on reload


[precommit][CircleCI] Check for unused arguments

What does this PR do?

Add flake8-unused-arguments to check for unused arguments to pre-commit and CircleCI.
Fix currently existing issues (remove unused arguments or ignore the error).
Remove pre-compile go deps step on CircleCI (useless since #5670).

Motivation

Avoid errors: I found the python_runtimes argument was being ignored on tests while working on a different PR.
Keep codebase simple and clean.

Additional Notes

To avoid problems we should mark the python_linting job as not required temporarily.

Describe your test plan

This should be a no-op.

Merging now since changes from other teams are rather minimal/trivial and CI passes.


omero: Correct ZeroC forum URLs (all invalidated)

Cherry-picked relevant commit from https://github.com/openmicroscopy/ome-documentation/pull/1231

See https://ci.openmicroscopy.org/view/Failing/job/OMERO-5.1-merge-docs/601/

:+1:


missing params Typescript error when trying to update search parameter from sub-route

Describe the bug
I'm trying to update a search parameter through Link or useNavigate in a sub-route of a route containing params.
If I interpret the docs correctly, it should only be necessary to supply the from & search parameters to Link/useNavigate in this case.
This was also the behavior prior to version 0.0.1-beta.190, but after upgrading I get a Typescript error Property 'params' is missing in type '{ search: { entry: number; }; }' but required in type '{ params: true | ParamsReducer<{ postId: string; }, { postId: string; }>; }'.
If I supply params={true}, Typescript is happy again but I can't find any documentation of what the true part actually means/what the effect of it is..
See https://stackblitz.com/edit/tanstack-router-xfrkjw?file=src%2Fmain.tsx for a reproduction of the problem (in the postRouteSub route).
Your Example Website or App
https://stackblitz.com/edit/tanstack-router-xfrkjw?file=src%2Fmain.tsx
Steps to Reproduce the Bug or Issue

Go to the code for postRouteSub in the reproduction
Observe Typescript errors in both the navigate()-call and the <Link> node

Expected behavior
I would expect that params={...} would be unnecessary in this case, as we're merely attempting to update a search parameter in a route which already has the $postId parameter assigned.
Screenshots or Videos
No response
Platform

@tanstack/react-router version 0.0.1-beta.190+ (I've tested it on the current 0.0.1-beta.194 too)

Additional context
No response

I'm not sure why this is the case, but instead of using the route.id, just use a string and let auto-complete do it's job. I'll look into this more.


Support for Skorpion K2

Is it possible to get support for the device from CyberPC called Skorpion K2? Since this isn't a common keyboard I can understand why not.

I can't seem to find any SDK available for the device, so there isn't much I can do to support it unless someone can reverse engineer it.


JUnit assertThrows

Code could be cleaner if it could be Java 1.8.
I could set <maven.compiler.testSource>1.8</maven.compiler.testSource>... thoughts???

Ready to be merged, only looking at replacing @Test(expected = Exception.class) with assertThrows in this pr. I will raise another pr to cover replacing try/catch/fail blocks with assertThrows at a latest date.

Can this get reviewed?
All the current tests using @Test(expected = AException.class) are replaced with assertThrows.

Splitting into separate PR as if this takes ages to get merged it's easier to rebase.


Add Fast SpanExtensions overloads operating on bytes

e.g. SequenceEqual

Doesn't BlockEquals do this?

Maybe, the point is that we should measure perf of some of the generic operations and see if specialized byte overload would improve perf.

completed by @AtsushiKan


Unreal Engine 5.1?

Does the 5.0 version work together with 5.1 or do we need another version for 5.1? If so, do we have an ETA?

Sorry, 5.0 version doesn't work with 5.1, the code is done and under internal testing, hopefully make it public in March.

Unreal 5.1 supported in https://github.com/GameTechDev/XeSSUnrealPlugin/releases/tag/v1.2.6


Add support for logging to Jaeger

Description
This enables individual operators to export logs to Jaeger. However, operator packaging must also be modified to add a jaeger-agent sidecar and enable tracing where applicable.
Logging to Jaeger is opt-in, since otherwise it clogs up the local logs with errors about failing to submit traces.

Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)
[ ] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

bors r+