CI is hanging

This is getting to be a pattern.
https://github.com/http4s/http4s/pull/5839#issuecomment-1008349663
https://github.com/http4s/http4s/runs/4767682086?check_suite_focus=true
https://github.com/http4s/http4s/runs/4767690258?check_suite_focus=true

Is this also related to the bind exceptions in #5843?

This is the earliest timeout I see, unless maybe someone reran an earlier one. https://github.com/http4s/http4s/actions/runs/1672936425

Has anybody observed this hang locally yet?  I am trying and have not.

But I can reproduce the bind exception erratically:
==> X org.http4s.ember.server.ConnectionSuite.close connection after response when request body stream is partially read 0.15s fs2.io.net.JavaScriptBindException: Address already in use
Caused by: scala.scalajs.js.JavaScriptException: Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:8080

Huh, I haven't seen Node.js jobs failing in CI lately, are they caught up in this too?
I think I know the story behind those bind exceptions though, multiple suites starting ember on the same port. I can probably fix that.

Has anybody observed this hang locally yet? I am trying and have not.

Our local machines all probably have more cores than a GHA runner, maybe this can only be observed with 1-2 cores.

@rossabaker couple quick questions—when you are running the test suite locally, I assume you are running test for everything at once right? i.e. not rootJVM/test and rootJS/test, like we do in CI. Also, have you ever observed the BindException on JVM?
I was trying to understand why it's happening, but I didn't take into account that on your machine the JVM and Node.js test suites may be running simultaneously and could be conflicting with each other. This is why it doesn't happen in CI, and this is why the Test / parallelExecution := false setting doesn't save us here, since ember.jvm and ember.js are technically independent projects in our build.

Our local machines all probably have more cores than a GHA runner, maybe this can only be observed with 1-2 cores.

Should be reproducible in Docker?

Yes, I run test.  I'm surprised it matters with regards to ports: we should be binding to any available port instead of hardcoding ports.  I think you're right that I should be targeting one or the other to reproduce it in general, though.
I have not previously had luck using Docker to reproduce CI-only failures, nor does running on an underpowered Chromebooke.  Adjusting the global ExecutionContext used to be helpful, but I'm not sure that will work in a modern Cats-Effect world.

I'm surprised it matters with regards to ports: we should be binding to any available port instead of hardcoding ports.

Gotcha, thanks. Then https://github.com/http4s/http4s/pull/5848 should fix that.

Our CI health just took another catastrophic plummet, see discussion in https://github.com/http4s/http4s/pull/5895#issuecomment-1016323697. It's exasperating.

I'm going to try to focus on this.  I don't think we can release past 0.22 in this state.

It's not perfect yet, but we're unhung.


Make Change password settings more user friendly

Pull Request Checklist

[:heavy_check_mark: ] Changes has been tested on an Android device or Android emulator with API 16
[ :heavy_check_mark:] UI change has been tested on both light and dark themes
[ :heavy_check_mark:] Pull request is based on the develop branch
[ :heavy_check_mark:] Pull request updates CHANGES.rst
[:heavy_check_mark: ] Pull request includes screenshots or videos if containing UI changes
[:heavy_check_mark: ] Pull request includes a sign off

Referencing #2898
@bmarty please check these.
Working demo (theme Status.im)

Dark theme

Light Theme

Hello @theLimitBreaker
Please revert all commit that added the whole SDK source in the Riot project (See https://github.com/vector-im/riot-android/pull/2986/files)
Also, please revert all extra useless new lines added to the code.
Also you can squash all the commits into a single one.
Thanks

I have to build against matrix-sdk-android.aar .(compile_against_project_lib). Right ?

There is some explaination here
What I see in this PR is that you have copied the source of the Matrix SDK, then committed them and push them. This is not correct.

@bmarty app doesn't work until or unless I build against matrix-sdk. If I use matrx-adk.aar , there are some unresolved references error

@bmarty I have reverted the requested things . Could you please check ? :)

Hi @theLimitBreaker ,
I'm sorry but I cannot merge this PR because of 22 commits that introduce dirty git history (I know it's now reverted).
Would you please rebase your branch, then squash all the commits into a single one?
Or open a new PR if it's easier for you.
Thanks
Benoît

NP :) . You are right . Git history is dirty . It should be cleaner.  I would rebase my branch and squash it .

@bmarty  please review .
Inlined "Inavlid Password Error and Prevent Dialog"

Hi @theLimitBreaker ,
Thanks for the update.
I will merge the PR in a temporary branch to do some test.
Benoît


documentation for integration with monolog and fos_user

would be great if there was some documentation to show people how to switch default SwiftMailer used in Monolog and Fos_User bundles to use your mailer

FOSUserBundle PR: https://github.com/Hipaway-Travel/HipMandrillBundle/pull/31

thx

see also: https://packagist.org/packages/astonishdesign/mandrill-bundle


How to distinguish authentication from user and root access key

Hi,
Reading from this document we can see for log version 2.0 there is a user principal field which logs if the storage account is accessed from a user, however, when we check our logs, this field is empty. Does it mean the request is using root access key to authenticate in this case?

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

ID: 30ebd39b-ae34-4988-5694-09acd19693e8
Version Independent ID: c36c476f-a7c3-1f68-7ef6-1a7e3664bbe1
Content: Azure Storage Analytics logging
Content Source: articles/storage/common/storage-analytics-logging.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@CharmanderJieniJieni Thanks for your question. We are checking on this and will respond to you soon.

@CharmanderJieniJieni User principal name is part of OAuth token when AAD is used. Please check the  in the logs. If it’s other type, it would be empty.
Reference: https://docs.microsoft.com/en-us/rest/api/storageservices/storage-analytics-log-format#log-entry-format-20

@CharmanderJieniJieni We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Use tsc for local development instead of webpack

I'm not sure this solution is really the best, but this PR makes it so that local debug task uses tsc to compile.
Only really drawback is that an out/server.d.ts is created (cannot be turned off because of the composite project).
But I think this is still better then requiring an additional plugin (I think this extension might solve the problem with the webpack watch task).
Also this makes it really fast to edit/start the debug instance.
For deployment webpack can still be used of course.
fixes #13

I don't feel strongly about the current dev environment, I just needed to get webpack hacked in so that we were distributing something of reasonable size.  I'm going to take this as-is, but feel free to keep sending PRs to tweak things if you come up with better approaches.  For example, moving /client and /server to be parallel directories so things are a little easier to find, or whatever.


[SPARK-45595] Expose SQLSTATE in error message

What changes were proposed in this pull request?

In this PR we include the SQLSTATE as part of the error message when
spark.sql.error.messageFormat = PRETTY (default)
[DIVIDE_BY_ZERO] Division by zero. Use `try_divide` to tolerate divisor being 0 and return NULL instead. If necessary set "spark.sql.ansi.enabled" to "false" to bypass this error. SQLSTATE: 22012
== SQL(line 1, position 8) ==
SELECT 1/0
       ^^^

Alternatives
Aside from minor changes (like colon vs equal etc) there more options on where to place the information.

Between error class and message

The state is added right before the message text, using a set of brackets (e.g. round):
[DIVIDE_BY_ZERO](22013) Division by zero. Use `try_divide` to tolerate divisor being 0 and return NULL instead. If necessary set "spark.sql.ansi.enabled" to "false" to bypass this error.
== SQL(line 1, position 8) ==
SELECT 1/0
       ^^^

At the very end after the context

[DIVIDE_BY_ZERO] Division by zero. Use `try_divide` to tolerate divisor being 0 and return NULL instead. If necessary set "spark.sql.ansi.enabled" to "false" to bypass this error.
== SQL(line 1, position 8) ==
SELECT 1/0
       ^^^
SQLSTATE: 22013

Both of these alternatives have issues:

The SQLSTATE itself is not interesting to a human. It is 5 alphanumerics that are hard to remember and have less information than the human-readable error class. As such it contributes nothing to anyone who wants to read the message.
The "context" which is currently optionally tailing the message can be length and complex. We may also decide to refine it in the futire. So adding SQLSTATE after it makes it hard to find. What, for example, if we want to nest error messages and the "stack-trace" should show multiple error messages. In that case SQLSTATE would be like a closing bracket, and hard to match.

Why are the changes needed?

To provide useful information for users to e.g. catch exceptions based on SQLSTATE or look up information.
Does this PR introduce any user-facing change?

Yes, error messages change
How was this patch tested?

Existing QA
Was this patch authored or co-authored using generative AI tooling?

No

@srielau About where to place the information. Could we reference other databases or system? e.g. Oracle.

The stream failure is unrelated, merging to master. thanks!


Native notifications

I'm really liking the QT version of whatsapp. However I think it would integrate better with desktop environments if it used native notifications (such as KNotification instead of QtWebEngineNotification)

You can switch the notification type in settings.


State topics

Is your feature request related to a problem? Please describe.
Now, state topics come somewhere under the homeassistant/<component>/* topic.
Describe the solution you'd like
I believe this should come under volvoAAOS2mqtt/* because these states have nothing to do with Home Assistant as such. The config topics for Home Assistant should then point towards the state topics under volvoAAOS2mqtt/*. This is also how other applications I have running are doing it.
In #100 someone is asking to run it without Home Assistant. While it is possible to run this solution without it, there would still be (string) references to Home Assistant when someone wants to read data from MQTT. If state topics are moved under volvoAAOS2mqtt/*, then there is no reference to Home Assistant at all.
Describe alternatives you've considered
None.
Additional context
None.

Yeah, this may be an improvement, but not yet on my to-do list, sorry!

Would you accept a PR? If so, I could have a look at it when I have some spare time.

For compatibility reasons, I would suggest to keep "homeassistant" currently as default, but make the topic changeable in the config.

Could be done by configuring a prefix for where the none-HA topics should come. Something like volvo_topic_prefix with a default value of homeassistant/{component}, where {component} would be replaced with switch, lock, and so on. The {component} part should be optional, so it would not return an error when you use volvoAAOS2mqtt as value.

@Dielee I've created a pull request to fix this issue. Please see details at PR #109.

I was able to run the Main.py successfully by just using the src folder ( and requirements.txt). Just edited config.json manually.

Hi bot! Please be patient, the PR is being tested.


Run a specific behaviour check

We need to provide a way to run specific tests instead of the whole suite.
But we also need to figure out how to refer these checks conveniently.
Will something like this work?
describe 'Some long description' spec(alias: 'myCheck1') {
  it('succeeds', alias: 'myCheck11') { }
  it('succeeds', alias: 'myCheck12') { }
}

And then we can use myCheck1, myCheck11, myCheck12 in the command line?

ping @nonkor

@roman-mazur I'd prefer to use line number like ./gradlew checkApiBehKcsbase:15 (not sure helium could handle such requirement), but using alias looks acceptable for me.

We can use approach to mark required scenario by some character like * or +.
So, basically, you just add the specific character to the describe to get *describe.
describe "Omitted scenario" spec {...

*describe "Scenario under debug that will be executed" spec {...

describe "Another omitted scenario" spec {...


Typo

@WhippsP : Thanks for your contribution! The author, @davidmu1, has been notified to review your proposed change.


fix(hosting.website-coach): update order funnel redirection

MANAGER-3497

Should target master ?

@cbourgois Since this ticket doesn't rely on the DTRUN project I think we can keep targeting develop branch instead.


AutoSuggestBox shouldn't be sealed, or NavigationView.AutoSuggestBox should allow templatization

Describe the bug
AutoSuggestBox is sealed and therefore disallows derived classes. The AutoSuggestBox property on the NavigationView requires an AutoSuggestBox and cannot be templatized. Title bar customization is only enabled on Windows 11. Therefore, an app that supports Windows 10 as well as 11 and wants to provide search functionality in the title bar will likely duplicate their ASB in the title bar and in the NavigationView, and enable one or the other depending on OS support. This means any customization and logic used by the ASB will be duplicated in both locations.
Ideally we'd create a user control containing an ASB, or a derived ASB class. But we can do neither in this case.
Steps to reproduce the bug
Read the documentation
Expected behavior
We should be able to user either a UserControl or a derived ASB class.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.4.3: 1.4.231115000
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

@mfeingol Thanks for filing the issue. Several types are sealed that we would like to unseal - this is a known problem with WinUI tracked here - https://github.com/microsoft/microsoft-ui-xaml/issues/780


Add a navbar to toggle between home and trip planner views

We are going to have a separate link for trip planner in navbar. So that people can go and forth between trip planner and our home page where we show weather forecast.

Hey @sumana2001 . Can you please assign this issue to @arpitBhalla and me?


Fix misspelling

@sivadeilra : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sivadeilra : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thank you! :-)


Create SimpleFactory to remove some duplication between existing fact…

Coverage decreased (-0.06%) to 90.987% when pulling 453d6ac9c21b9993d9b49978c3ebdd028926a099 on imed-bh:refactoring into 5374337bca804708e51954b6d6616f7d5fc30977 on JsonSchema-JavaUI:dev.


-, will confuse updates of crate with just .

libra-bitvec was confused for bitvec preventing notification of a security issue.
https://app.dependabot.com/accounts/libra/update-logs/28072396
updater | INFO <job_28072396> Checking if bitvec 0.17.3 needs updating
  proxy | 2020/03/31 07:56:53 GET https://crates.io:443/api/v1/crates/bitvec
  proxy | 2020/03/31 07:56:53 200 https://crates.io:443/api/v1/crates/bitvec
updater | INFO <job_28072396> Latest version is 0.17.4
  proxy | 2020/03/31 07:56:56 GET https://api.github.com:443/repos/rust-lang/crates.io-index/commits/master
  proxy | 2020/03/31 07:56:56 * authenticating github api request
  proxy | 2020/03/31 07:56:56 304 https://api.github.com:443/repos/rust-lang/crates.io-index/commits/master
updater | INFO <job_28072396> Requirements to unlock own
updater | INFO <job_28072396> Requirements update strategy bump_versions
updater | INFO <job_28072396> Updating bitvec from 0.17.3 to 0.17.4
  proxy | 2020/03/31 07:57:02 GET https://api.github.com:443/repos/rust-lang/crates.io-index/commits/master
  proxy | 2020/03/31 07:57:02 * authenticating github api request
  proxy | 2020/03/31 07:57:02 304 https://api.github.com:443/repos/rust-lang/crates.io-index/commits/master
updater | INFO <job_28072396> Handled error whilst updating bitvec: dependency_file_not_resolvable {:message=>"    Updating crates.io index\nerror: failed to select a version for the requirement `libra-bitvec = \"= 0.17.4\"`\n  candidate versions found which didn't match: 0.1.0\n  location searched: /home/dependabot/dependabot-updater/dependabot_tmp_dir/common/bitvec\nrequired by package `network v0.1.0 (/home/dependabot/dependabot-updater/dependabot_tmp_dir/network)`\nconsider running `cargo update` to update a path dependency's locked version\n"}

👋 sorry for the slow response.
We're starting to clean out our backlog... it's been nearly three years since this was filed, and we've shipped many improvements since then.
Is this still a valid issue or should it be closed?


x/net/http2/h2demo: stop running

The HTTP/2 demo program is currently served at https://http2.golang.org. The index page describes it as:

This server exists for others in the HTTP/2 community to test their HTTP/2 client implementations and point out flaws in our server.

It was created around the time when HTTP/2 support was novel and in active development, eventually being added to the standard library in Go 1.6 in 2016. By now it is still useful but probably significantly less so than it was earlier on.
Though it's sad to say this, I think it has served us well and maybe it's a good time to move on and stop running the HTTP/2 demo server at http2.golang.org, making it easier to focus our attention in other areas that would benefit Go more. (Its source code is public and can always be accessed in the future by anyone interested.)
CC @golang/release, @neild, @bradfitz.

CC @neild

SGTM. I don't think the effort of keeping a server running and up-to-date is worth the benefit, and it's easy enough for anyone to run their own local server if they need to test against Go's HTTP/2 implementation.

I've disabled the alerting and turned off the service. I'll delete the code in a few days if nothing screams.

end of an era :)

Would you consider moving the code of https://http2.golang.org/reqinfo to httpbin.org (which seems to be well maintained) ?

httpbin is a python project,
if they wanted to, they're free to take/fork and run the code and run it, no need to move things

Sure.
I'm just proposing a way to maintain this test request for clients who need it, without having to maintain your server.
I'll ask httpbin if they can implement a similar request.

QQ where is the source code of the old demo/server? I'm having issues with h2 handler and I would love to see the working code as a comparison to see what I may be doing wrong

https://go.googlesource.com/net/+/69e39bad7dc2bbb411fa35755c46020969029fa7/http2/h2demo/ contains the last version prior to deletion.


enhancement: replace stdlib json by jsoniter

github.com/json-iterator/go is currently one of the fastest libs available to process json.
Besides, it is a simple drop-in replacement for the standard lib, without much code change.
I am proposing PR #357 to validate the concept and expose benchmarks on signing and parsing methods.
NOTE: due to some changes in the order of marshalled claims, testable examples verifying the value of the signed string have been adapted to check for decoded claims instead.

expect！

Attention: notice that this breaks compatibility with older golang versions.
I don't think that keeping a compatibility promise back to versions older than 1.7 brings much value.
This PR imposes golang 1.7+.


Possible double counting of weights

https://github.com/jakevdp/wpca/blob/317b84153d904af1d0937bef2e862e76ca470421/wpca/wpca.py#L160
This appears to be calculating something more akin to cWWc. Should it instead be something like cWc = np.dot(cW,  self.components_.T)?

I performed a simple transform - inverse_transform test and it seems cWc = np.dot(cW, self.components_.T) is incorrect.


mark group size become huge in Sketch 53

the mark group created by plugin has a huge size, but the actual symbols don't. this didn't exist in Sketch 52.

@mepine  https://github.com/utom/sketch-measure/issues/536


Windows not appearing when adding node or opening settings in specific Godot Sessions

Godot version
4.1 Stable
System information
Windows 10
Issue description
When trying to add a node in my Godot Project. I get a flash on the screen for a split second, I see a tiny blip of a window opening and closing in my taskbar, and then Godot effectively freezes. There is no mouse input anymore.
However, when adding a node, I am able to hit enter and a Node2D is added in my scene and typing something like "anim" will add an AnimatedSprite2D to the scene as long as I don't click anything
https://github.com/godotengine/godot/assets/38440991/5839857f-ed2a-41e9-bf2b-70153072eb8e
Steps to reproduce

Open the project
Load into the test_level scene
Try to add a node to the player or test_level scenes
Window should not open

Minimal reproduction project
SidescrollerTutorial.zip

4.1 Stable

Can you test 4.1.2 or 4.2.beta1? We only support the latest patch release of each given version series.

I just tested in both of those versions and it's continuing to happen


Disable preview mode

Is there a way to disable preview mode on editor initialisation? I only want preview mode to be seen when I click the show preview icon button and not to be shown by default.

Sorry, there is no way to disable preview mode on editor init. But I think i can open up this parameter.

okay I will wait for the update. that'd be very nice. thank you

Please upgrade to 2.1.2. Then options setting has a showPreviewPanel property. Default is true, you can set it to false on init.

thank you!


Can not change header area border color

Unable to change the bottom border color hover doc

I tried by changing LspSagaBorderTitle group color but didnot work

Try:
highlight LspSagaDocTruncateLine guifg=#000000 " Replace the hex code with what you need

If you're intrested in changing the header border of other windows, you could try these:
highlight LspSagaShTruncateLine guifg=#000000
highlight LspSagaDocTruncateLine guifg=#000000
highlight LspSagaCodeActionTruncateLine guifg=#000000

You can explore the colour file of the plugin to find these. I found that the highlight groups determining these line colours has "...TruncateLine" in the name.

Thanks, it worked now.


Optimize Basis constructor for Axis Angle

I noticed that this constructor was doing many of the same things twice. I've optimized it, and now this constructor runs about 10-15% faster in C#. The behavior is the same as before.
These optimizations are roughly based on this implementation: https://www.euclideanspace.com/maths/geometry/rotations/conversions/angleToMatrix/

Could you benchmark the performance impact on C++ and/or GDScript?

Not sure how to set up a C++ benchmark, but the performance is a little under 5% better in GDScript. Not a significant improvement, but still an improvement.

Thanks!


Circle Chart Crash

Describe the bug
Adding a new variable to a circle chart sometimes crashes the extension.
To Reproduce
Steps to reproduce the behavior:

Create a histogram.
Change chart type to circle.
Delete y-variable.
Assign a datafield to the x-pill.
Add a new variable. This will crash the extension.

Expected behavior
User should be able to assign a datafield to the x-pill and add a new data pill.
Screen Recording
https://user-images.githubusercontent.com/83424843/129117934-423ae068-302a-421a-9ed1-fa6f36366142.mov

Tested with scatter, since we no longer support circle. Issue appears to be fixed


Search: Add is:followed search filter.

Description
With the is:followed filter, users can easily narrow down their message search results to include only those messages within topics they are actively following. Similarly -is:followed filter, excludes topics that are followed by the user.
Fixes: #27309

Screenshots and screen captures:
https://github.com/zulip/zulip/assets/91622060/e47a5eb3-a07e-4909-b75f-c0842d7b7593
Help Center

Improvements for the future: Creating more effective test cases. As this was my initial attempt at writing tests, there's room for improvement in both filter.test.js and test_message_fetch.py.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@nimishmedatwal can you rebase this onto the current version so it can be reviewed?

@nimishmedatwal can you rebase this onto the current version so it can be reviewed?

Sure!

@kennethnrk would you be up for helping review this one, and possibly putting up a revised PR for its issue?

As mentioned by Tim Abbott, your query is incorrect, It will be better to use an Exists or Join query to check for messages that have a topic that is followed.

Please combine the the tests and the feature in the same commit.
Add specific tests to check the is:followed operator is returning only messages that have a topic that is followed.
Update the API Change log

@kennethnrk would you be up for helping review this one, and possibly putting up a revised PR for its issue?

I have made the required changes to the PR and created a new PR #29667
Ready for your review @timabbott

Thanks for your work, @nimishmedatwal ! Closing in favor of the continuation in https://github.com/zulip/zulip/pull/29667.


pom.xml: Update to Java 17, parent pom 1.2.0

Update to latest LTS JDK. This required a parent POM update as well.

Why do you want to update this project to Java 17? Just because it is the latest LTS? If there is no other reason then I will reject this request.

Why do you want to update this project to Java 17? Just because it is the latest LTS? If there is no other reason then I will reject this request.

I think I also got a warning about using a language preview feature, that was the reason I bumped the version. But I did think you would want to be on latest LTS.

I can't seem to reproduce the warning that I thought I saw. Closing...


Add action with more commands as generic arguments

Create branch from main
issue-22-add-action-with-more-commands-as-generic-arguments


feat(GUI): add more sample

Add more sample about GPU GUI

looks like there are some api changed in this pr, better add those changes in pr content, or add highlight in titile


[DOCUMENTATION] Add info how to start dev-vm without showing the GUI of Parallels

Currently not possible because of upstream issue (missing feature in Parallels Desktop):
see vagrant-parallels#39

When I start the vm from commandline or tools like Vagrant Manager or other helper scripts, the Parallels Desktop GUI isn't shown.
I think we could close this issue therefore?!


Add MLJ interface tests

This PR:

adds generic MLJ interface tests
removes Manifest.toml to resolve #2
moves Test from [deps] to [extras] and adds [tests] section to Project

This is currently failing (locally, at least) because of:

[ ] https://github.com/ml-unito/PartitionedLS.jl/issues/4

Thanks for the suggestions and the pull request. I've fixed the problems and closed the issues.
Let me know if there is anything else to fix.
Best.


[FR] Support help text

I recently finished a sample Meteor app using AutoForm to illustrate basic features and to support comparison with a similar app I created previously using the Play Framework.
This process seems to indicate that AutoForm would benefit from built-in support for help text. Please see the last section (called "Issues: Form Help Text") of the following page for a more detailed explanation and sample code:
http://ics-software-engineering.github.io/meteor-example-autoform/

+1 for this one... maybe even in a schema like
  note: {
    type: String,
    label: 'Notes',
    labelHelp: 'Place for your note - use markup if you want to',
    }

@philipmjohnson: Did you manage to access your schema field in the form template?

@thebarty No, I'm crossing my fingers that support for help text will be added to the schema.


does it only take relative paths to images?

I was just trying this plugin, which looks awesome by the way, in my project with a few urls from google images but every time I get the following error in the console:
ERROR TypeError: Cannot read property 'replace' of undefined     at NgxGalleryHelperService.push../node_modules/ngx-gallery/bundles/ngx-gallery.umd.js.NgxGalleryHelperService.validateUrl (ngx-gallery.umd.js:121)     at NgxGalleryHelperService.push../node_modules/ngx-gallery/bundles/ngx-gallery.umd.js.NgxGalleryHelperService.getBackgroundUrl (ngx-gallery.umd.js:133)     at NgxGalleryImageComponent.push../node_modules/ngx-gallery/bundles/ngx-gallery.umd.js.NgxGalleryImageComponent.getSafeUrl (ngx-gallery.umd.js:367)     at Object.eval [as updateRenderer] (NgxGalleryImageComponent.html:3)     at Object.debugUpdateRenderer [as updateRenderer] (core.js:10872)     at checkAndUpdateView (core.js:10248)     at callViewAction (core.js:10484)     at execEmbeddedViewsAction (core.js:10447)     at checkAndUpdateView (core.js:10244)     at callViewAction (core.js:10484)
Here is my component.ts
`export class FefileComponent implements OnInit {
galleryOptions: NgxGalleryOptions[];
galleryImages: NgxGalleryImage[];
ngOnInit(): void {
  this.galleryOptions = [
    {
      'imageArrows': true,
      'imageSwipe': false,
      'thumbnailsArrows': true,
      'thumbnailsSwipe': false,
      'previewSwipe': true,
      'thumbnailsMargin': 0,
      'thumbnailMargin': 0,
      'thumbnailsColumns': 6,
      'width': '100%',
      'height': '450px',
    },
    {
      'imageArrows': true,
      'imageSwipe': false,
      'thumbnailsArrows': true,
      'thumbnailsSwipe': false,
      'previewSwipe': true,
      'thumbnailsMargin': 0,
      'thumbnailMargin': 0,
      'thumbnailsColumns': 6,
      'width': '100%',
      'height': '450px',
    },
  ];

  this.galleryImages = [
      {
          small: 'https://images.pexels.com/photos/34950/pexels-photo.jpg?auto=compress&cs=tinysrgb&h=350',
          big: 'https://cdn.pixabay.com/photo/2016/10/27/22/53/heart-1776746_960_720.jpg'
      },
      {
          small: 'https://images.hindi.news18.com/ibnkhabar/uploads/459x306/jpg/2018/02/p5-1.jpg',
          big: 'https://encrypted-tbn0.gstatic.com/images?q=tbn:ANd9GcQNWkbzIr0rgBhjLJ1OqE7gmDEU5RqOunqwW-na5kcqKfr4tZx5eg'
      }
  ];

}
}`
I have imported the Module in my module.ts properly. My html is:
`

        Angular2 File Upload

<ngx-gallery [options]="galleryOptions" [images]="galleryImages"></ngx-gallery>

`
I have mostly worked with Node till now so not really familiar with angular that much so I'd appreciate if you could just show me the way here. Thanks in advance.

The problem was that it takes URL for all 3 sizes, i.e., small medium and big. I was giving only small and big.


Setting GLOBAL_ROUTE_ARGS in SWEEP variables gets overriden

In the current SWEEP base configuration, we're setting GLOBAL_ROUTE_ARGS to increase the iterations from 5 to 30, but the GRT run is run with 5 iterations:
Running global_route.tcl, stage 5_1_grt
global_route -congestion_report_file bazel-out/k8-fastbuild/bin/reports/asap7/BoomTile/base/congestion.rpt -congestion_iterations 5 -congestion_report_iter_step 5 -verbose

So, we can't run experiments where we sweep through different iteration values, since it's getting overridden by:
    stage_arguments = {
        "grt": {
            # global route is futile for now, so do a few iterations to
            # get a global route artifact with .odb and .rpt file
            "GLOBAL_ROUTE_ARGS": "-congestion_iterations 5 -congestion_report_iter_step 5 -verbose",
        },
    },

Think https://github.com/The-OpenROAD-Project/megaboom/pull/201 will work


docs: fix ascii tree in ASTx ascii tree repr page

Attempt to fix the ascii tree visualization

@xmnlab this may have solved the issue. I looked at the .md file and it looked much better.


Standalone command support for local environment

We can support standalone commands which run without any external ps or worker hosts if possible.

Will be supported in v0.0.9 as default behavior.

Supported.


[BUG] - Data returns not dict

Data return in Version 1.0.1-alpha are sqlite row objects.

Need to be dict return


[Bug]: Edit option isn't shown for workflow in homepage when only edit permission is provided

Is there an existing issue for this?

[X] I have searched the existing issues

Description
https://github.com/appsmithorg/appsmith/assets/80334441/232dda91-5b0b-4283-b00f-c2bbd66ccf92

Steps To Reproduce

Create new role with only edit permission for workflow
Assign it to user, login as that user
Observe 3 dots are not shown on the homepage

Public Sample App
No response
Environment
Release
Issue video log
No response
Version
DP

Hey team! Please add your planning poker estimate with Zenhub @ayushpahwa @nsarupr @Parthvi12


See a space

Deviate from original story - need to see a space first before updating.

Closes #13


Allow global style client connection via Redis.current

Allow global access to the current client via Redis.current
Example
Redis.current = Redis::Client.new(URI.parse(ENV.fetch("REDIS_URL", "redis://<IP_ADDRESS>:6379/0")))

Setting Redis.current can only happen once to avoid accidental override

Thanks for the feedback @jgaskins, let me know your thoughts on this iteration.

I guess we always end up with multiple Redis DB quickly in projects, so closing this one.

@mjeffrey18 It's funny, I thought about this issue a couple weeks ago when I noticed that the Ruby redis gem had removed this exact feature.


broom

Related:

https://github.com/rstudio/sparklyr/issues/598

@samuelmacedo83 is taking a look at this one.

done:

ml_naive_bayes
ml_logistic_regression
ml_decision_tree
ml_random_forest
ml_gradient_boosted_trees
ml_bisecting_kmeans
ml_kmeans
ml_gaussian_mixture
ml_linear_regression
ml_generalized_linear_regression

in progress:

ml_aft_survival_regression
ml_isotonic_regression

not-done:

ml_als
ml_als_factorization
ml_association_rules
ml_fpgrowth
ml_linear_svc
ml_lda
ml_multilayer_perceptron
ml_pca

How do I call the implementation for visualization of a decision tree / random forest / etc.?

Hello, the code example at #598 no longer works. The error i get is
> invoke(model$model, "toDebugString") Error in UseMethod("invoke") :    no applicable method for 'invoke' applied to an object of class "c('ml_decision_tree_classification_model', 'ml_prediction_model', 'ml_transformer', 'ml_pipeline_stage')"
I was thinking of pulling the model into R (using a collect) and visualizing it with a rpart.plot(). But that doesnt seem to work either.
Any help or feedback will be greatly appreciated. :)

Alright, after some experimentation and help online , the code below helps you crudely visualize a tree
get the debug string
model_stage <- fitted_model$stages[[3]]  spark_jobj(model_stage) %>% invoke(., "toDebugString") %>% cat()


Implement FixedInterval for Clock.Timer

Used time of the previous event will be taken away from the next interval (if true), or intervals start from the end of last event.

You can see the difference through this test project
test project.zip

@ColinTree Please check over your changes for trailing whitespace...

@ColinTree We cannot accept this change as written.
The reality is that when we perform a release of MIT App Inventor, not everyone can immediately update their copy of the MIT AI2 Companion. So we are careful to ensure that a new release will work with an old Companion as long as new features are not used. So if your project worked prior to the release with the old (then current) Companion, it would still work after the release even though you didn't update your Companion.
The problem here is that Let's say I have a project with a Clock component. The first time I load it into App Inventor after the release, the project updater will add a FixedInterval property set to False. Because this is not the default value, the generated YAIL will have a (set-and-coerce-property "FixedInterval" False) line in it. This will be unrecognized by the older Companion which will then throw and exception.
I would recommend that you change the default value to False.

@jisqyv
I know what problem you are talking sir. But I did this similarly to the others work.
e.g. @ewpatton wrote similar here: https://github.com/mit-cml/appinventor-sources/commit/6e23c47c97882d517d4cd7b1bd6f4a414b7f094e

@ColinTree Actually that is not what is going on in the change you referenced. The default values of all of the new properties were such that a project loaded into the workspace would not assert any non-default value unless the user explicitly enabled them (and they would then need the new Companion).
Howeve Evan then set some of the properties in the build of the Companion itself. So in this case (themes) user projects would still have the older "legacy" theme while the new Companion would have the modern theme. However loaded projects would not automatically adopt the new modern theme. Instead the project owner would have to explicitly set the theme in their project, but then they would need to have the new Companion to use that project with the Companion.
-Jeff

Can one of the admins verify this patch?

Changed the default value to false on backward compatible purpose

@AppInventorWorkerBee ok to test


Add livebook version to the CLI

And it should print the Elixir and Livebook version.

We already support a CLI option:
➜  ~ livebook
Usage: livebook [command] [options]

Available commands:

  livebook server    Starts the Livebook web application

The --help and --version options can be given instead of a command for usage and versioning information.
➜  ~ livebook --version
Erlang/OTP 24 [erts-12.1.2] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [jit]

Elixir 1.13.0 (compiled with Erlang/OTP 24)

Livebook 0.4.1


安装依赖后，进webui报错

pip install -r requirements.txt 之后，python src/train_web.py打开webui，控制台提示的是：
D:\LLaMA-Factory-0.2.0\src>python train_web.py
===================================BUG REPORT===================================
Welcome to bitsandbytes. For bug reports, please run
python -m bitsandbytes
and submit this information together with your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
bin C:\Users\Adquan\AppData\Local\Programs\Python\Python310\lib\site-packages\bitsandbytes\libbitsandbytes_cpu.dll
C:\Users\Adquan\AppData\Local\Programs\Python\Python310\lib\site-packages\bitsandbytes\cextension.py:34: UserWarning: The installed version of bitsandbytes was compiled without GPU support. 8-bit optimizers, 8-bit multiplication, and GPU quantization are unavailable.
warn("The installed version of bitsandbytes was compiled without GPU support. "
function 'cadam32bit_grad_fp32' not found
CUDA SETUP: Loading binary C:\Users\Adquan\AppData\Local\Programs\Python\Python310\lib\site-packages\bitsandbytes\libbitsandbytes_cpu.dll...
FlashAttention-2 is not installed, ignore this if you are not using FlashAttention.
Running on local URL:  http://<IP_ADDRESS>:7860
To create a public link, set share=True in launch().
webui报错，提示的是：
Error
Expecting value: line 1 column 1 (char 0)
请问是哪里出了问题

没有 cuda 环境


Remove the primary key from the MultiIndex key specification

In MultiIndex, remove the primary key from the index key specification (i.e. when building / defining the `MultiIndex). That is, handle index key multiplicity internally, using the pk or another method.
This can be done by signalling the pk type with its own generic type (see #531), and then internally building an index key of type (IK, PK), that is, a tuple of Index key and Primary key. Of course, the index function must return (IK, PK) accordingly.
Alternatively, if the above is not possible make the index function type-aware on the pk element. That is, instead of using a byte slice, use a proper type for the pk element. So that it's easier to handle internally.

Breaking change, as it would change the internal representation of the index key. For 0.11, or pehaps, 0.12.


Flag unknown commands

See #81

$ vip blah
Error: `blah` is not a valid subcommand.

  Usage: vip [options] [command]

  Commands:

    app     List and modify your VIP Go apps
    help    Display help
    logout  Logout from your current session
    sync    Sync production to a development environment

  Options:

    -h, --help     Output usage information
    -v, --version  Output the version number

$ vip app list blah
Error: `blah` is not a valid subcommand.

  Usage: vip app-list [options] [command]

  Commands:

    help  Display help

  Options:

    -f, --format [value]  Format results (defaults to "table")
    -h, --help            Output usage information
    -v, --version         Output the version number


App misbehaves misbehave when force quit

I am new to Ionic and branch and I am facing the issue When I force to quit the sample app and then click on deep link, it opens the app but does not navigate to required page, and remains on rootpage. But When the app is in background and I open the app by clicking deeplink  it correctly goes to the requested page. Any help or insights will be helpful
this.platform.ready().then(() => {
this.statusBar.overlaysWebView(false);
this.splashScreen.hide();
this.setRootPage();
BranchInit();
});
this.platform.resume.subscribe(() => {
BranchInit();
});
const BranchInit= () => {
// only on devices
if (!this.platform.is('cordova')) { return }
const Branch = window['Branch'];
Branch.initSession(data => {
if (data['+clicked_branch_link']) {
//alert('Deep Link Data: ' + JSON.stringify(data));
this.nav.push(data['page'],{itemId:data['id']});
}
});
}

@tommyarn what versions are you using? Are you primarily using Android or iOS?

@csalmi-branch  I am using iOS  version 11.2 and Corvoda Branch SDK version 3.1.5

@tommyarn do you still have issues if you use the newest version we just released, 3.1.6? We updated which native SDKs the Cordova integration calls into, which should help with application interactions.

@csalmi-branch  yes the issue is still there on the new version.


Connect-MicrosoftTeams error in Azure function

Getting the error below when using the MicrosoftTeams PowerShell module v1.0.22 in an Azure function and trying to call the connect-microsoftteams commandlet and passing in the credential object.
[Error]   ERROR: Connect-MicrosoftTeams : password_required_for_managed_user: Password is required for managed user
Version 1.0.22 of the module is required to take advantage of the CsGroupPolicyAssignment commandlets

I notice that the connect-microsoftteams contains an identity parameter which although there is no documentation related to the parameter presumably it infers the ability to use a managed identity? Is anyone able to elaborate if its possible to use a managed identity when using the the connect-microsoftteams commandlet and if assigning the identity to a built-in role like teams service admin will solve my issue? Thanks in advance

@officedocsbot assign @yogkumgit

Hello @azureDev20
Please see the following https://github.com/MicrosoftDocs/office-docs-powershell/issues/5177#issuecomment-611114041
where someone from the team confirmed that they are currently working with -Identity parameter for future releases, the documentation does not reflect it because it is not ready to be used yet. (and also, module 1.0.22 is from the beta channel)
Hope this information helps you.

Thanks @get-itips I expected as much but thought I would ask so thanks for clarifying. Any information on how to use the connect-microsoftteams commandlet in an Azure function to successfully authenticate would be greatly appreciated.

@azureDev20
Are you using the cmdlet as described in the documentation? for example, passing a credential object to the credential parameter.

@get-itips see below script which is executed on the PowerShell runspace of an Azure function to access ms teams instance
$Username = $ENV:SvcAcctUsername
$P = $ENV:SvcAcctPwd
$Password = $P | ConvertTo-SecureString -AsPlainText -Force
$Credential = New-Object System.Management.Automation.PSCredential($Username, $Password)
Connect-MicrosoftTeams Connect-MicrosoftTeams -Credential $Credential
Error - [Error] ERROR: Connect-MicrosoftTeams : password_required_for_managed_user: Password is required for managed user
Script above works when run on-prem

@azureDev20 I am not an expert in Azure Functions, however, I received the same error trying to run the cmdlet under PowerShell Core, and Microsoft Teams PowerShell module is currently only compatible with Windows PowerShell, can you confirm you are using Windows PowerShell?

thanks @get-itips, at this time the only PowerShell runtime stack available in an Azure function is PowerShell core so that would explain the issue. Thanks for your continued assistance.

Thanks for returning and sharing that feedback!

No worries, I just checked the PowerShell Gallery website and it states that the MS Teams module is for both Windows PowerShell and PowerShell core, see URL below. Can you send me the URL which states that the module is only available to Windows PowerShell?
https://www.powershellgallery.com/packages/MicrosoftTeams/1.0.7

No worries, I just checked the PowerShell Gallery website and it states that the MS Teams module is for both Windows PowerShell and PowerShell core, see URL below. Can you send me the URL which states that the module is only available to Windows PowerShell?
https://www.powershellgallery.com/packages/MicrosoftTeams/1.0.7

I see that information was added recently, will try to cross check with the product team.

@azureDev20 Managed to report this behavior in Ps Core to the product team, they are aware and can work on it. Thanks!

Excellent, thanks @get-itips looking forward to a good outcome.

@get-itips  Thank you very much for the contribution and sharing this explanation. @azureDev20 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@get-itips Is there any timeline from the product team on when this will be fixed?

@get-itips Is there any timeline from the product team on when this will be fixed?

How are you? I am afraid not, I will reply to this thread (will remain closed as this is not a documentation issue) once I get an update.

Good thank you, it seems a pretty huge bug for it not to be worked on as a matter of priority as this exists on all the Microsoft PowerShell modules I have tested including the modules in GA. Is there another area in Github I can report this bug on?

Good thank you, it seems a pretty huge bug for it not to be worked on as a matter of priority as this exists on all the Microsoft PowerShell modules I have tested including the modules in GA. Is there another area in Github I can report this bug on?

There isn't, however, I have confirmation is already filed and triaged internally. Thanks

@get-itips  is there any idea when this will be fixed or any work around?


fix(tooltip): toggle tooltip when children of host element are focused/blurred

This enables tooltips for focused checkboxes, since the input element that receives the focus is a
child of mat-checkbox and not the host itself.
Fixes #15294.

True, but I would argue that a checkbox with a tooltip is a more common use case than a container with a tooltip and focusable children.

Either way both scenarios are likely. Couldn't you trigger the tooltip yourself programmatically using the focus monitor directive?


Upscaling problem please help

Upscaling gif from 34x34 to 680x680, frame 1 is crisp while the rest are blurry. Problem starts at step2 spritesheet.

@jalagar hows it going?

Step2 config: https://drive.google.com/file/d/1XNYcut4akYya3OeRdQ9ljvk0tPrUPj8A/view?usp=sharing
Global config: https://drive.google.com/file/d/1fmYo7pLfpoSK_4UqoBjo5hN3-m4PSKWP/view?usp=sharing

@Neonode22 I figured out what's going on. Your input layers are 34x34 GIFs. They need to be larger GIFs or else the resolution will get worse. If you change the global_config to height: 34 and width: 34 it looks fine.

@Neonode22 not sure that is a good question... it shouldn't be clear given the gif is so small

@jalagar I mean like why is the first frame clear? It's crisp in the 680x680 dimensions, but the rest are blurry, and appear to still look like they're 34x34

Yea I know what you mean, I'm not sure. But from a technical standpoint that doesn't make sense, there's no way a 34x34 can look clear like 340x340

Thank you sorry for the delay a lot has been going on


fix: Fixed a bug where results in stubs would not be named

Closes #100
Summary of Changes
Fixed a bug where results in stubs would not be named if the docstrings did not specify a result type. Now results named if the

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

v0.3.0
GitHub release

Your semantic-release bot :package::rocket:


[DOCS] Reference manual pages about visual elements should clearly point to their doc

Issue Description
We got a support ticket about documentation. Someone looked on Google for Taipy table doc and stumbled onto this page and complained that there was no information on the page: https://docs.taipy.io/en/release-3.0/manuals/reference/taipy.gui.builder.table/
This is because it is a reference manual page and not the page he was looking for. The page he was looking for is this one: https://docs.taipy.io/en/latest/manuals/gui/viselements/table/#__tabbed_1_3
There should be a big button on ref manual pages about visual elements that redirect to the visual element documentation.
Refers to Avaiga/taipy#1611
Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

Hi, can i work on this? @AlexandreSajus

Sure! Just so I'm clear: I want all the visual elements pages like these: https://docs.taipy.io/en/release-3.0/manuals/reference/taipy.gui.builder.table/
To point to their more useful counterparts like these: https://docs.taipy.io/en/latest/manuals/gui/viselements/table/#__tabbed_1_3
With a big visible button, something like what we did here:

The repo that contains our docs is here: https://github.com/Avaiga/taipy-doc
Assigned!

@AlexandreSajus
Would this please you?

@codermal7 @AlexandreSajus
Sorry I didn't spot this was assigned: this issue was not properly moved to the Contribution slot, and not even planned for the current sprint.
So I did the job because it was raining outside...
If @AlexandreSajus likes this proposal, we're done with that.

This pleases me.


add admin button to homepage and hamburger-menu

Description

added Admin button to homepage linking to adminDashboard

used FaUserShield as logo for Admin button

added Admin link to HamburgerNav linking to adminDashboard

Type of change
Please select everything applicable. Please, do not delete any lines.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] This change requires an update to testing

Issue

[x] Is this related to a specific issue? If so, please specify:
closes #405

Checklist:

[x] This PR is up to date with the main branch, and merge conflicts have been resolved
[x] I have executed npm run test and all tests have passed successfully or I have included details within my PR on the failure.
[x] I have executed npm run lint and resolved any outstanding errors. Most issues can be solved by executing npm run format
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Howdy! Button looks great!
Everything looks good. One request. Can you add button testing to cypress/e2e/landing.cy.js?
I will want to merge this after we finish #439 so that users do not have access to the menu button.

Hi Caleb,
Added testing for the new admin button.
Please let me know if any changes are needed.
Thanks.


Update to Django>=3.1.6

This project is stuck on Django 2. It needs some work to be updated to 3, for security and compatibility reasons.
I don't have time to do this currently, unfortunately.

What are some of the security and compatibility issues? Tests are passing with django 3.2.5

I'll be honest, the tests could do with some work; they only test the core functionality of the package but not the compatibility with Django. Therefore, the tests will pass with any Django version. (I do want to get around to fixing that though)
Before when trying to use the package with Django>=3 I was receiving errors about template tags. However I just made a new test project with Django 3.2.5 and it seems to work!
The security issues are to do with Django<3.1.12 I believe so not really to do with the package itself.
Send a PR with the bumped version and I'll update and release :)


[$500] Highlight applied on editing and main composer

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.39-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Send a message
Apply offline mode
Hover over the sent message > click on 'Edit' icon

Expected Result:
Highlight should only be applied on editing composer
Actual Result:
Highlight is applied on editing and main composer
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/43996225/887afbd0-dd9c-4c60-8c37-8ca4378fa7c8

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014ad3d9a06c61cce3
Upwork Job ID:<PHONE_NUMBER>592553984
Last Price Increase: 2024-02-09

Proposal
Please re-state the problem that we are trying to solve in this issue.
The composer fails to focus during the mounting process.
What is the root cause of that problem?
The root cause is to do with the recent changes in MarkdownTextInput (inside Composer).
Despite the state of isFocused in ComposerWithSuggestions is true on mount, the MarkdownText component does not get focused, resulting in an out-of-sync state for the isFocused property.
This gives the impression that both inputs are focused in edit mode, even though the main composer itself is not in focus.
What changes do you think we should make in order to solve the problem?
Add below effect to ComposerWithSuggestions component will fix the issue.
https://github.com/Expensify/App/blob/main/src/pages/home/report/ReportActionCompose/ComposerWithSuggestions/ComposerWithSuggestions.js#L542
    // run once on load
    useEffect(() => {
      if (!isFocused) {return}
      focus(true)
      // eslint-disable-next-line react-hooks/exhaustive-deps
    }, [])

What alternative solutions did you explore? (Optional)
None

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01eaa7c76af4ae1bf1

Related to https://github.com/Expensify/App/pull/35837, we're reverting the PR

We've reverted the offending PR https://github.com/Expensify/App/pull/35837, so this is no longer a blocker

Let's hold on https://github.com/Expensify/react-native-live-markdown/pull/175 in such case. We'll repeat the repro steps to double confirm our issue is fixed.

Tested locally on release 0.1.9. Issue is no longer reproductible - https://github.com/Expensify/react-native-live-markdown/pull/175 seems to solve this


The sprite could be smaller easily

Hi!
As you mentioned in the readme of jMinEmoji2, the sprite is quite big. 950K is too much especially for mobile sites.
Luckily the size of that emoji.png can be easily reduced just by recompressing it. I've been using tinypng.com for a while now, and they reduced the size from 950K to 212K, but you can use anything that works for you :)

You are right, they use quantization in order to reduce png colors, which reduce quality, but overall it doesn’t compromise the emoji, so I use pngquant in combination with other tools and updated the repo, commit: 06bbdb5


sendAck with custom options

How to send an ACK packet with custom options?

To emulate a WPAD (https://findproxyforurl.com/wpad-introduction/) i need this custom ack packet.


[Поле пошуку] The long input text is not completely visible and the cross icon covers this text

Environment: Windows 10 Pro., Google Chrome
Reproducible: always
Build found: last commit
Preconditions
Open website on a home page https://speak-ukrainian.org.ua/dev/
Steps to reproduce

Go to  'Поле пошуку'
Enter Програмування, робототехніка, STEM

Actual result
The input text is not visible completely and the cross icon covers this text

Expected result
The long input text is completely visible and the cross icon does not cover this text

"User story #77

The long input text is not completely visible

The bug is still reproducible


Allow menu customization from settings

Allowing customization (for example, change the order of items to have the most common at the top rather than in a submenu), of menus through settings would enable a simple and very useful customization without requiring a user to create an extension for the purpose.
Inspired by this recent change --  previously, the most common actions in my workflow - push and pull - were directly in the menu, but they have been moved to a cascading submenu, making them harder to get to.   Meanwhile, clone which you only ever run once on a repository is front and center.

Push and pull are now front and center again.
There's another feature request open for user customization of menus.


Add release-build option to tools/run-android.

So it's easier to test changes (such as those made for a React Native upgrade) that we reasonably expect might cause different behaviors in debug vs. release builds.

https://github.com/zulip/zulip-mobile/issues/4929#issuecomment-908603249 is somewhere where a user wanted to test something on Android in production mode instead of development.


Fix PMD violations in Thrift source

The affected files seem to be generated by the Thrift compiler.
Assuming there is a more recent version of the Thrift compiler that
emits code that passes the PMD checks, the proper thing to do would be
to upgrade.
I'm not familiar enough with the Java ecosystem and this project's CI to
perform that kind of expiriment at this time.  I'm just trying to fix
the CI build in order to make other changes, so please be lenient :-)

These are generated files and should not be manually edited. If you want to stop the warnings, better to prevent SpotBugs from running on these generated files. I'm not sure how to do it but suspect it should be controllable via the Maven build.

err, PMD

Yeah, I figured as much, but:

I have no idea what I'm doing;
I have very limited time to spend on this; and
I wasn't 100% sure this would even fix the build yet.

Now that we see the build passing, I'll mark this as WIP and see if I can exclude the files from the PMD checks.

After fiddling around with the PMD plug-in exclusion configuration for over 2 hours, I cannot get it to ignore violations by file and I cannot get it to ignore violations from inner classes (those with a $ in them).
I did manage to combine a mix of some exclusions and tweaking the threshold, which seems to fix the build.
See #325.

Closing in favor of #325.


feature: add cross-reference

I have looked in the documentation and in the issues but i did not find it.
Does this module support references to paragraphs, titles and pages?
How do i insert them?

+1 for this feature!

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

Is this feature currently available?

+1

+1

+1

+1

any updates on the create cross-references feature?

Hello :) Is there any news about this feature ?

+1 for this feature :)

We are still waiting for it =)

We really need this feature

I also need it

Just putting out there that there that this would still be really useful feature in 2021!

+1

Anyone with a soultion to this feature? Badly need this for a document

Anyone with a soultion to this feature? Badly need this for a document
See https://github.com/python-openxml/python-docx/issues/74#issuecomment-215678765

Anyone with a soultion to this feature? Badly need this for a document

See #74 (comment)
Does this work for cross page paragraph reference


runtime haulting error related to character portraits that are scenes

The problem
Describe the bug
When a character's portrait is a .tscn, and a character event set to "update" has an animation going with the "wait for animation" toggle unchecked, and a text event is initiated for that character and a portrait option other than "(don't change)" is selected in that text event, a runtime error occurs when the timeline is played. The specific error differs based on the exact animation from the character update event, but it often results from trying to get a function or property on an object of base 'Nil'.
To Reproduce
Steps to reproduce the behavior:
Start an empty project, import Dialogic 1.4.5, and create a character whose portrait is a .tscn file. In a timeline, add a 'join' character event for that character, then an 'update' character event with some animation selected and 'wait for animation' turned off, and a text event set to that character with the portrait on something other than "don't change". Under these specific circumstances, pressing 'Play Timeline' will cause the scene to hault when transitioning from the update event to the text event.
Expected behavior
The expected behavior would be for the program to continue running in some kind of way rather than hault with an error.
Screenshots

As you can see in this screen, the problem has to do with Anima- specifically, it's somehow getting passed a Nil value as an argument.
System (please complete the following information):

OS: Kubuntu 20.04
Godot v3.5.stable.official [991bb6ac7]
Dialogic 1.4.5

For now, this error appears specific enough that it's easy to get around, but I couldn't figure out for a while what was causing this to happen here and not in a fresh project.

An update to this issue: I've discovered that it occurs not just in these specific circumstances.
Anytime you have an animation from character update event with 'wait for animation' disabled, a text event that changes the character pose will cause the same hault. It's actually trickier to work around than I thought at first.


When disconnecting a display, the accrylic background on the taskbar is not updated

What it is supposed to look like

When the display is disconnected, it seems that the background acrylic of the other display is used on the other display where the backgound image is not in the same place

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


[xbar] xbar reset connection issue

in order to handle async transactions, the xbar takes as its inputs resets related to different clock domains.
Currently, the reset hooked up for connections to blocks like usb/spi_host/ are using the end points reset.
This is a problem because if the end point is reset by software, it may trigger unintentional request / responses due to one side of an asynchronous fifo being reset.  Instead of taking the end point's reset, it should just take the reset related to the appropriate clock domain.

If I understand correctly, a SW triggered reset for some peripheral was also resetting some part of the xbar... which I assume had undesirable side-effects.


bug: User not authenticated on setCredentials (Android 14)

Bug Report
Capacitor Version

 Capacitor Doctor

Latest Dependencies:

  @capacitor/cli: 5.6.0
  @capacitor/core: 5.6.0
  @capacitor/android: 5.6.0
  @capacitor/ios: 5.6.0

Installed Dependencies:

  @capacitor/cli: 5.3.0
  @capacitor/core: 5.6.0
  @capacitor/android: 5.6.0
  @capacitor/ios: 5.6.0

[info] Using Gemfile: RubyGems bundle installed
[success] iOS looking great! 👌
[success] Android looking great! 👌

Plugin Version

   Capgo Doctor

 OS: darwin Darwin Kernel Version 22.6.0: Wed Jul  5 22:22:52 PDT 2023; root:xnu-8796.141.3~6/RELEASE_ARM64_T8103

 Node: v20.8.1

 Installed Dependencies:

   @capgo/cli: 3.14.54
   @capgo/capacitor-native-biometric: 5.1.0

✓ Latest Dependencies:

   @capgo/cli: 3.14.54
   @capgo/capacitor-native-biometric: 5.1.0

✅ All dependencies are up to date

Platform(s)

Android

Current Behavior
Error on call setCredentials on Android API 34
Error: Failed to save credentials
Expected Behavior
Save credential
Code Reproduction
call setCredentials using Android version 14
Other Technical Details

npm --version output: 10.1.0
node --version output: v20.8.1
pod --version output (iOS issues only):
Additional Context
Failed to save credentials
  android.security.keystore.UserNotAuthenticatedException: User not authenticated

Device: Samsung S23
One UI 6.0
Android version 14

Another issue with Android reported for me as well by 1 user (Pixel 6, Android 14).
Credentials are not saved, so after closing/opening app - biometric authentication + retrieving credentials is failing.
Please advise

Same here, not working for android 14

We have encountered this issue in our application as well. I was able to reproduce the problem when, for some time, I didn't use biometrics, after which the application seems unable to save credentials anymore. I am investigating further, but it seems the issue resolves when biometrics are used in another app that supports it.
We definitely have several users complaining about the problem.

 Capacitor/Plugin  E  Failed to save credentials
	android.security.keystore.UserNotAuthenticatedException: User not authenticated
		at android.security.keystore2.KeyStoreCryptoOperationUtils.getInvalidKeyException(KeyStoreCryptoOperationUtils.java:126)
		at android.security.keystore2.KeyStoreCryptoOperationUtils.getExceptionForCipherInit(KeyStoreCryptoOperationUtils.java:152)
		at android.security.keystore2.AndroidKeyStoreCipherSpiBase.ensureKeystoreOperationInitialized(AndroidKeyStoreCipherSpiBase.java:354)
		at android.security.keystore2.AndroidKeyStoreCipherSpiBase.engineInit(AndroidKeyStoreCipherSpiBase.java:248)
		at javax.crypto.Cipher.tryTransformWithProvider(Cipher.java:2981)
		at javax.crypto.Cipher.tryCombinations(Cipher.java:2892)
		at javax.crypto.Cipher$SpiAndProviderUpdater.updateAndGetSpiAndProvider(Cipher.java:2797)
		at javax.crypto.Cipher.chooseProvider(Cipher.java:774)
		at javax.crypto.Cipher.init(Cipher.java:1289)
		at javax.crypto.Cipher.init(Cipher.java:1224)
		at ee.forgr.biometric.NativeBiometric.encryptString(NativeBiometric.java:317)
		at ee.forgr.biometric.NativeBiometric.setCredentials(NativeBiometric.java:202)
		at java.lang.reflect.Method.invoke(Native Method)
		at com.getcapacitor.PluginHandle.invoke(PluginHandle.java:138)
		at com.getcapacitor.Bridge.lambda$callPluginMethod$0(Bridge.java:797)
		at com.getcapacitor.Bridge.$r8$lambda$ehFTi5f4HhVNFKTbCKAYDkpQYRA(Unknown Source:0)
		at com.getcapacitor.Bridge$$ExternalSyntheticLambda3.run(Unknown Source:8)
		at android.os.Handler.handleCallback(Handler.java:958)
		at android.os.Handler.dispatchMessage(Handler.java:99)
		at android.os.Looper.loopOnce(Looper.java:257)
		at android.os.Looper.loop(Looper.java:368)
		at android.os.HandlerThread.run(HandlerThread.java:67)
 Capacitor   D  Sending plugin error: {"save":false,"callbackId":"85292839","pluginId":"NativeBiometric","methodName":"setCredentials","success":false,"error":{"message":"Failed to save credentials"}}

💊   Capacitor Doctor  💊

Latest Dependencies:

  @capacitor/cli: 5.7.4
  @capacitor/core: 5.7.4
  @capacitor/android: 5.7.4
  @capacitor/ios: 5.7.4

Installed Dependencies:

  @capacitor/core: 5.7.0
  @capacitor/cli: 5.7.0
  @capacitor/android: 5.7.0
  @capacitor/ios: 5.7.0

[success] Android looking great! 👌
[error] Xcode is not installed

💊   Capgo Doctor  💊
OS: linux #28~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Fri Mar 15 10:51:06 UTC 2

Node: v18.17.0

Installed Dependencies:
  * @capgo/cli: 4.3.0
  * @capgo/capacitor-native-biometric: 5.1.0

✓   Latest Dependencies:
  * @capgo/cli: 4.3.0
  * @capgo/capacitor-native-biometric: 5.1.0

✅ All dependencies are up to date

I also encountered this problem, and later found that it was a problem with the Google API. When I checked this line of code, it was executed correctly without any errors. After checking the Google documentation, I found that there was a bug in this method in Android 12 - Android 14, which has been fixed in Android 15.
https://developer.android.com/reference/android/security/keystore/KeyGenParameterSpec.Builder#setUnlockedDeviceRequired(boolean)

I also encountered this problem, and later found that it was a problem with the Google API. When I checked this line of code, it was executed correctly without any errors. After checking the Google documentation, I found that there was a bug in this method in Android 12 - Android 14, which has been fixed in Android 15.
https://developer.android.com/reference/android/security/keystore/KeyGenParameterSpec.Builder#setUnlockedDeviceRequired(boolean)

I wrote a plugin temporarily. Android 12 to 14 versions stopped using this API.
https://github.com/Above-Os/capacitor-native-biometric

Thanks @icebergtsn i back ported your changes: https://github.com/Cap-go/capacitor-native-biometric/commits/main/


NDcPPv3.0 Roadmap item - CCMB comments

Provide the location of the issue
Throughout SD and cPP
What is the enhancement request for the cPP? Please describe.
The CCMB performed a review of the Supporting Document related to the Network Devices cPP (more detailed: ND SD V2.2 related to ND cPP V2.2e).
Describe the solution you'd like
This issue is being worked through the NIT. The NITs decisions will become TDs for NDcPPv2.2e. This issue can be closed once all applicable TDs are added to the NDcPP for version 3.0.
Describe alternatives you've considered
NA
Additional context
NA

Resolved in PR #221


Add terms for LLPC neurons

Recent papers mention some lobula - lobula plate columnar neurons (LLPC neurons). Notably Scheffer et al. (2020) (FBrf0246888) found 6 types of LLPC neurons (LLPC1, LPPC2[a-d], and LLPC3). But information about those neurons is sparse.
Currently the ontology defines a “generic” lobula complex columnar neuron class which is narrowly aliased to, among other things, LLPC, but while it has more precise classes for, e.g., LPLC neurons, there is such class for LLPC neurons.
VFB has some LLPC1 neurons (e.g., FlyEM-HB:1780362273), which are classified as lobula complex columnar neuron, possibly for lack of a more precise class.

awaiting further publications describing these cell types


Improve error handling in parameters, when theres an extraneous comma

Description:
Refer the following cases:
Case I:
function foo(int x,) {
}

Output:
xxx.bal:2:20:missing type
xxx.bal:2:20:missing variable
--------------------------------------
function foo (  int x, MISSING(type) MISSING(variable) ) {
}

Case II:
function foo(,) {
}

Output
xxx.bal:2:14:missing type
xxx.bal:2:14:invalid token ','
xxx.bal:2:15:missing variable
--------------------------------------
function foo (  MISSING(type) MISSING(variable) ) {
}

Case III:
function foo(int x, , ,) {
}

output:
xxx.bal:2:14:missing type
xxx.bal:2:14:missing variable
xxx.bal:2:15:missing type
xxx.bal:2:15:missing variable
xxx.bal:2:16:missing type
xxx.bal:2:16:invalid token ','
xxx.bal:2:17:missing variable
--------------------------------------
function foo (  MISSING(type) MISSING(variable), MISSING(type) MISSING(variable), MISSING(type) MISSING(variable) ) {
}

In all 3 cases, the parser tries to recover by inserting another parameter after the comma. However, a better way to recover is to simply remove these additional commas.

As per the new requirements from parser and the LS, these recovered nodes are acceptable.
Hence closing the issue.


ClientException: Connection to localhost:48898 failed (socket error -4077) (read ECONNRESET)

Setup:

Node-red with node-red-contrib-ads-client v1.2.0
TwinCAT 3 4022.22 running on 64bit Windows 10 (remote PC & PLC)

I am getting a ClientExpection (see log below) when trying to set up notifications/subscriptions for a bunch of PLC variables:
Welcome to Node-RED
===================

5 Apr 12:39:56 - [info] Node-RED version: v2.2.2
5 Apr 12:39:56 - [info] Node.js  version: v16.14.0
5 Apr 12:39:56 - [info] Windows_NT 10.0.14393 x64 LE
5 Apr 12:39:57 - [info] Loading palette nodes
5 Apr 12:40:00 - [info] Dashboard version 3.1.6 started at /ui
5 Apr 12:40:00 - [info] Settings file  : C:\Users\User\.node-red\settings.js
5 Apr 12:40:00 - [info] Context store  : 'default' [module=memory]
5 Apr 12:40:00 - [info] User directory : \Users\User\.node-red
5 Apr 12:40:00 - [warn] Projects disabled : editorTheme.projects.enabled=false
5 Apr 12:40:00 - [info] Flows file     : \Users\User\.node-red\flows.json
5 Apr 12:40:01 - [info] Server now running at http://<IP_ADDRESS>:1880/
5 Apr 12:40:01 - [info] Starting flows
5 Apr 12:40:01 - [info] [ads-client-connection:MyPlcConn] Connecting to <IP_ADDRESS>.1.1:851...
  ads-client _connect(): Starting to connect localhost:48898 +0ms
5 Apr 12:40:01 - [info] Started flows
  ads-client _connect(): Socket connection established to localhost:48898 +12ms
  ads-client _connect(): Socket connect failed: Error: read ECONNRESET
  ads-client     at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  ads-client   errno: -4077,
  ads-client   code: 'ECONNRESET',
  ads-client   syscall: 'read'
  ads-client } +2ms
5 Apr 12:40:01 - [info] [ads-client-connection:MyPlcConn] Connecting to <IP_ADDRESS>.1.1:851 failed, keeping trying..
5 Apr 12:40:01 - [warn] [ads-client-connection:MyPlcConn] Failed to connect <IP_ADDRESS>.1.1:851 at startup: ClientException: Connection to localhost:48898 failed (socket error -4077) (read ECONNRESET)
  ads-client _disconnect(): Starting to close connection (force: false) +2s
  ads-client _unsubscribeAllInternals(): Unsubscribing from all notifications +1ms
  ads-client _unsubscribeAllInternals(): Unsubscribed from 0 notifications +1ms
  ads-client _registerAdsPort(): Failed to unregister ADS port - Timeout - no response in 2000 ms +2s
  ads-client _disconnect(): Connection closed successfully +2s
  ads-client setDebugging(): Debug level set to 2 +15ms
  ads-client _connect(): Starting to connect localhost:48898 +0ms
  ads-client _connect(): Socket connection established to localhost:48898 +6ms
  ads-client _connect(): Socket connect failed: Error: read ECONNRESET
  ads-client     at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  ads-client   errno: -4077,
  ads-client   code: 'ECONNRESET',
  ads-client   syscall: 'read'
  ads-client } +8ms
5 Apr 12:40:07 - [info] [ads-client-connection:MyPlcConn] Connecting to <IP_ADDRESS>.1.1:851...
  ads-client _disconnect(): Starting to close connection (force: false) +6ms
  ads-client _unsubscribeAllInternals(): Unsubscribing from all notifications +0ms
  ads-client _unsubscribeAllInternals(): Unsubscribed from 0 notifications +0ms
  ads-client _disconnect(): Connection closed successfully +2ms
  ads-client setDebugging(): Debug level set to 2 +1ms
  ads-client _connect(): Starting to connect localhost:48898 +0ms
  ads-client _connect(): Socket connection established to localhost:48898 +1ms
  ads-client _connect(): Socket connect failed: Error: read ECONNRESET
  ads-client     at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  ads-client   errno: -4077,
  ads-client   code: 'ECONNRESET',
  ads-client   syscall: 'read'
  ads-client } +1ms
5 Apr 12:40:07 - [info] [ads-client-connection:MyPlcConn] Connecting to <IP_ADDRESS>.1.1:851 failed, keeping trying..
5 Apr 12:40:07 - [error] [ads-client-subscribe:MyPlc] Error: Not connected to the target, retrying in the background.
  ads-client _disconnect(): Starting to close connection (force: false) +2s
  ads-client _unsubscribeAllInternals(): Unsubscribing from all notifications +1ms
  ads-client _unsubscribeAllInternals(): Unsubscribed from 0 notifications +0ms
  ads-client _registerAdsPort(): Failed to unregister ADS port - Timeout - no response in 2000 ms +2s
  ads-client _disconnect(): Connection closed successfully +0ms
  ads-client setDebugging(): Debug level set to 2 +0ms
  ads-client _connect(): Starting to connect localhost:48898 +0ms
  ads-client _connect(): Socket connection established to localhost:48898 +1ms
  ads-client _connect(): Socket connect failed: Error: read ECONNRESET
  ads-client     at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  ads-client   errno: -4077,
  ads-client   code: 'ECONNRESET',
  ads-client   syscall: 'read'
  ads-client } +1ms

The route to the PLC is enabled and Ads Communication is already running successfully using .NET Software.
I experience this error trying to connect to a remote PLC (setup 1) as well as running Node-RED locally on the desired PLC (setup 4).
I also tried adding EnableAmsTcpLoopback = 1 as registry key on the remote PC as well the PLC without any success.

Hi!
Please make sure you are using latest versions (node-red-contrib-ads-client 1.3.2 and ads-client 1.13.1). If the problem still occurs, please post all node connection settings.

Hi,
unfortunately the error remains.
I´m using the default configuration and just adding AmsNetId and Port:

Thanks a lot.

Damn. Haven't seen that kind of problem.
What happens if you run node-red on the PLC and try to connect using target address localhost. That should work basically always.
Could you also post logs with debugging level 3.

Well it really looks like its the same issue.
Here is the level 3 log using localhost and port 851 on the PLC:
7 Apr 09:15:13 - [info] Starting modified nodes
7 Apr 09:15:13 - [info] [ads-client-connection:MyPlcConn] Connecting to localhost:851...
  ads-client setDebugging(): Debug level set to 3 +6ms
  ads-client _connect(): Starting to connect localhost:48898 +1ms
7 Apr 09:15:13 - [info] Started modified nodes
  ads-client _connect(): Socket connection established to localhost:48898 +4ms
  ads-client:details _registerAdsPort(): Registering an ADS port from ADS router localhost:48898 +458ms
  ads-client:details IO out ------> 8 bytes +1ms
  ads-client _connect(): Socket connect failed: Error: read ECONNRESET
  ads-client     at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  ads-client   errno: -4077,
  ads-client   code: 'ECONNRESET',
  ads-client   syscall: 'read'
  ads-client } +9ms
  ads-client:details _registerAdsPort(): Socket connection errored. +2ms
7 Apr 09:15:13 - [info] [ads-client-connection:MyPlcConn] Connecting to localhost:851 failed, keeping trying..
7 Apr 09:15:13 - [warn] [ads-client-connection:MyPlcConn] Failed to connect localhost:851 at startup: ClientException: Connection to localhost:48898 failed (socket error -4077) (read ECONNRESET)
  ads-client _registerAdsPort(): Failed to unregister ADS port - Timeout - no response in 2000 ms +756ms
  ads-client _disconnect(): Connection closed successfully +0ms
  ads-client _connect(): Starting to connect localhost:48898 +3ms
  ads-client _connect(): Socket connection established to localhost:48898 +1ms
  ads-client:details _registerAdsPort(): Registering an ADS port from ADS router localhost:48898 +759ms
  ads-client:details IO out ------> 8 bytes +1ms
  ads-client _connect(): Socket connect failed: Error: read ECONNRESET
  ads-client     at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  ads-client   errno: -4077,
  ads-client   code: 'ECONNRESET',
  ads-client   syscall: 'read'
  ads-client } +2ms

Everthing works fine on my engineering PC running win10 and TwinCAT v3.1.4024.22.
So either its a problem with 4022.22 (same version @Hopperpop struggeled with) or my installation process on the remote PLCs. I have no internet access on the PLCs so I run the node.js installer and move the .node-red and npm folders manually. Is there something more to pay attention to when trying to set up node-red offline?

Hmm. That really looks 100% identical.
I really wonder what is the reason. I never have had that with few different versions but I got to admit I think I havent had 4022.10. @Hopperpop if you see this, did you ever find a solution?
Just to make sure:

Connection works on development PC with 4024 TwinCat when connecting to localhost (same machine)?
Connection does not work on development PC when connecting to PLC with 4022.10 using AmsNetID?
Connection does not work on PLC when connecting to localhost (same machine?)

I'm sorry I have no experience on installing node-red without internet connection.

Yes to all three questions. It is only 4022.22 causing the problem, 4024.22 is working fine. I will have a look on secure ADS.
Thanks a lot for your quick replies and effort.


Figure 1. in the README appears to be broken

I visited the repo for the first time today, and I don't see an image for Figure 1. Clicking on the link, https://raw.githubusercontent.com/tdewolff/canvas/master/resources/preview/preview.png, just brings up a blank page.

Thanks, this has now been fixed, sorry for the delay


Cannot get the functions correctly

The ofn get command does not get the function correctly.

It seems that we need to upgrade the version of OpenFunction in Cli.
From v1alpha1 to v1beta1.

/assign

Thanks for taking this @f10atin9.
Previously @wentevill add the client ability in this cli tool, the client code is implemented in this.
But a better way is to add client code in the main repo and that's why https://github.com/OpenFunction/OpenFunction/pull/176 and https://github.com/OpenFunction/OpenFunction/pull/243 is created for.
So going forward, we may need to replace the client library in this cli with the one in the main repo. Use the client library in the main repo to implement functions like ofn get fn

@wentevill has created a PR for this https://github.com/OpenFunction/cli/pull/41
Close this issue for now.


[1.13] Fix hello-world flaky teardown

High-level description
Tried wrapping the cosmos.uninstall_package in a 15min retry, but the retry complained that the package was already uninstalled. Switched to looking at completed_frameworks mesos /state response, but also ran into flake. I added some logs of the frameworks response, and found that in the flake, the framework remained under 'frameworks' object but the tasks list within eventually emptied out to 0 (moved to completed_tasks). I think this could be a good way of assessing the framework as been torn down sufficiently if there are no longer any running tasks.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-4568 test_metrics.deploy_and_cleanup_dcos_package Framework hello-world still running

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change: integration test change
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

@mesosphere-mergebot label ready for review


Use a greater-than sign in quotation marks with an error result

Description
The results of beautification are not what I expect.
Expected Results
The beautified code should have looked like:
import React, {Component} from 'react';
import {Breadcrumb} from 'antd';

class FrameContent extends Component {
    render() {
        return (
            <div className="frame-content">
                <Breadcrumb separator=">">
                    <Breadcrumb.Item>Home</Breadcrumb.Item>
                    <Breadcrumb.Item href="">Application Center</Breadcrumb.Item>
                    <Breadcrumb.Item href="">Application List</Breadcrumb.Item>
                    <Breadcrumb.Item>An Application</Breadcrumb.Item>
                </Breadcrumb>
            </div>
        );
    }
}

export default FrameContent;

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/e44716049dc0a603379fbeab13810630
Checklist

[x] I have reloaded (or restarted) Atom to ensure it is not a caching issue
[x] I have tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] Generated debugging information and added link for debug.md Gist to this issue
[ ] Searched for existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue

Yeah, this is broken in Pretty Diff v1.16.37 which is the last version published to NPM.  NPM fell out of favor with me.  I am working on an alternative right now at https://github.com/prettydiff/biddle
This is fixed in the current version of Pretty Diff.

Keeping everybody informed I have fixed biddle.  Check it out now - https://github.com/prettydiff/biddle
I will republish Pretty Diff using biddle in production tomorrow.  I am not ready to give it to Atom Beautify just yet though.  I actually need users to play around with this thing for a bit so that I can get a sense of where the pain points are and just how bad it works for fails for other people.  I wrote a getting started document at https://github.com/prettydiff/biddle/blob/master/documentation/gettingstarted.md
The documentation might be easier to read from the command line though as I included a markdown to CLI parser in biddle.  The number at the end defines the word wrap limit:
node biddle markdown documentation/gettingstarted.md 80


Design new demo app

The current demo app is very barebones. We would like to get something a bit nicer that a collection of widgets, to showcase how to use Jewel.
The mockup can contain components we don't have yet, and we can backfill the gaps as we go.

@c5inco any plan to do this? Or should we close for now?

The design is fine right now so let's close it and can reopen if it's a higher priority


Should strip query parameters when creating podcast's filename

Over at our mailing list, TonyK reports that Newsbeuter names the files downloaded from libsyn.com like this: podcast.mp3?dest-id=175971.
Currently we just apply basename to the download URL and use result as a filename. We should do a better job by using something that understands URL syntax and can strip query parameters.

Hello.
I think that a custom naming scheme (either directly based on the title of each item, or via a user-provided post-download script) could be beneficial to the users.
Thank you.

I'd love a way to specify the downloaded filename, e.g. the feed item date in datetime-format + " - " plus the feed item title.
Much better than 268510384-film-comment-hong-sangsoo.mp3 or bee_101716_010.mp3


Memory occupation is excessive

Issue Type: Performance Issue
My laptop only have 8G memory. Memory usage increased from 50% to about 70% when I opened my project. Hope you can solve this problem.
VS Code version: Code 1.22.1 (950b8b0d37a9b7061b6f0d291837ccc4015f5ecd, 2018-04-06T02:26:57.615Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz (8 x 1800)

Memory (System)
7.85GB (1.15GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
67%

Process Info
CPU %	Mem MB	   PID	Process
    0	    99	 23192	code main
    0	    86	  6408	   shared-process
    0	    84	  6564	   powershell.exe -NoProfile -ExecutionPolicy Bypass -Command "& {& 'c:\Program Files\Microsoft VS Code\resources\app\out\vs\base\node\ps-win.ps1' -ProcessName 'Code.exe' -MaxSamples 3}"
    0	     7	 23056	     console-window-host (Windows internal process)
    0	    85	  6840	   powershell.exe -NoProfile -ExecutionPolicy Bypass -Command "& {& 'c:\Program Files\Microsoft VS Code\resources\app\out\vs\base\node\ps-win.ps1' -ProcessName 'Code.exe' -MaxSamples 3}"
    0	     7	 23300	     console-window-host (Windows internal process)
    0	    62	  7424	   shared-process
    0	   131	 19800	   gpu-process
    0	   141	 20208	   window (GameConst.lua - Documents - Visual Studio Code)
    0	    10	 12436	     watcherService
    0	     5	 13788	       console-window-host (Windows internal process)
    0	    20	 15836	     searchService
    0	    10	 19860	     electron-crash-reporter
    0	    49	 22248	     extensionHost
    0	    21	 17012	       electron_node jsonServerMain.js
    0	   184	 21760	   window (VMCity.lua - SLG - Visual Studio Code)
    0	    11	  6232	     watcherService
    0	    10	 16468	       console-window-host (Windows internal process)
   14	   402	  9040	     extensionHost
    0	    38	 20596	     searchService

Workspace Info
|  Window (GameConst.lua - Documents - Visual Studio Code)
|    Folder (Documents): 1672 files
|      File types: lua(810) svn-base(388) png(222) docx(60) csv(55) xlsx(35)
|                  dll(29) json(10) atlas(8) plist(8)
|      Conf files:
|  Window (VMCity.lua - SLG - Visual Studio Code)
|    Folder (SLG): more than 20000 files
|      File types: svn-base(16845) h(1124) cpp(918) lua(527) class(129) c(50)
|                  java(39) mk(37) mm(36) cmake(35)
|      Conf files: cmake(35) sln(2) makefile(2);

Extensions (8)

Extension
Author (truncated)
Version

vscode-markdownlint
Dav
0.14.1

beautify
Hoo
1.3.0

svn-scm
joh
1.28.0

python
ms-
2018.3.1

sublime-keybindings
ms-
3.0.3

guides
spy
0.9.1

vscode-todo-highlight
way
0.5.12

luaide-lite
wel
0.1.5

(1 theme extensions excluded)

The extensionHost process on VMCity.lua stands out as using lots of memory. Is it better if you start with --disable-extensions? You might be able to figure out which extension is using lots of memory and see whether they have ideas. That workspace seems to have tons of files so you may not be able to improve it.

I have checked all my extensions, and found that luaide-lite 0.1.5 takes a lot of memory for parsing all my lua files.  Memory usage reduced from 16% to 7% when I disable this extension. Thank you guys.


Fix cron problem

The cron scheduler maybe don't implement all cron options and does not work fine with 0/6.
With 0/6 only execute at 0 and at 6, so, you need relogin every day (the token need refresh each 8 hours).
Changing 0/6 to */6 it works fine.

Thanks!


ci: Change ADO feeds used by perf benchmarks + small refactor

Description
Updates the feeds that the performance benchmarks pipeline uses to get packages, to match the ones used by the e2e tests pipeline (build feed for all fluid scopes except @fluid-internal, which comes from the dev feed).
I suspect the issue we see every now and then where the performance benchmarks pipeline complains that it can't find some version of a package that it wants to install, has to do with a delay between the time when the packages are published to the build feed, and when they can be downloaded from the dev feed (which has build as an upstream). Telling the pipeline to download packages straight from the source should (hopefully) get rid of the intermittent error.
Relevant teams thread (msft internal).
Also, small refactor:

Simplify the use of conditionals around some variable assignments
Add a few missing variables/parameters to the output for troubleshooting
Removed the artifactPipeline parameter from a template that didn't use it
Moved printing of variables/parameters for the performance benchmarks pipeline to be closer to the start

Testing
(msft internal links)

E2E tests pipeline run that shows the change in logic to generate the feed, devFeed and artifactPipeline variables still works, and the appropriate feed/devFeed values for a test/ branch.
Performance benchmarks pipeline run showing the same thing.

We can't really validate that the issue is gone since it's so non-deterministic.

/azp run Real Service End to End Tests


bug: wezterm has dayfox theme from this repo missing

Neovim version (nvim -v)
v0.10.0-dev-2465+ga7788c2e2
Operating system/version
MacOS 14.4
Describe the bug
WezTerm doesn't have the dayfox theme provided by the extras
Steps To Reproduce

Proceed to https://wezfurlong.org/wezterm/colorschemes/d/index.html#dayfox
Search for proper dayfox
cry 😢

Expected Behavior
I added the `dayfox theme locally, but I'd be much more convenient to have the proper one shipped with wezterm
Repro
No response

It looks like wezterm is getting the colors from iterm2-color-schemes repo. These colors are coming from a PR that I made to the repo in 427. This is correct and is still the same as this project.

Thank you for your reply. If you search for say ‘dawnfox’ this repo will be mentioned, while for ‘dayfox’ it looks like themes collided with iterm2. That being said, ‘dayfox’ iterm2 theme lack tab bar colors for Wezterm, that’s why I raised this issue in the first place.

@EdenEast could you please double-check this?


Remove --accept-defaults flag from the cluster join command

The configuration of the cluster should be set on the bootstrap node. So it seems to be a bug that the --accept-defaults flag exists on the cluster join command, as there should be nothing for the joining node to accept defaults from.

What about bootstrapping the first node only as database and then joining a second node as haproxy? The prompt for haproxy will appear only when joining the second node. So it is valid to say that instead of getting the prompt, you want to accept the defaults during join. Is this a fair point for the continuity of the --acept-defaults flag in the cluster join command?

Ah yeah, that makes sense. Thanks for answering! :)


FDG-5325 Add WOtD Insight to FDG Site Navigation

CC: 89.54
https://federal-spending-transparency.atlassian.net/browse/FDG-5325
Builds locally
Passes all tests locally

Closing since we aren't merging these until sprint 83


Sonoff S20 not working

I have 2 Sonoff Basic, 2 Sonoff  Dual R2 working using homebridge-mqtt-switch-tasmota in EVE and HOME.
The also defined S20 (working in FHEM via MQTT Mosquitto) are not working via homebridge.
What's wrong or is the S20 not supported by homebridge-mqtt-switch-tasmota?

Sorry. My fault. Topic misspelling.


RFE: option to avoid exception when query timeout is reached

Based on behavior difference between 0.9.0 and latest version.
Query like this
SELECT DISTINCT account_key FROM point_entry WHERE MAP_VALUE(account_fields_keys, 'type', account_fields_values) = 'manual_billing_uncollected' and created_at >=<PHONE_NUMBER> LIMIT 11 option(timeoutMs=20000)
in this case the entire query would have taken much longer than the timeout to finish.
On 0.9.0, when timeout reached, it was able to simply return data it gathered so far and respond to client
But on the latest version, it throw TimeoutException on client side, while retrying the query on controller UI, it returns
{
    "errorCode": 427,
    "message": "207 servers [...] not responded"
  }

request an  option to avoid exception when query timeout is reached

I see the following changes were implemented in the following PR. I also see in java-client, we can ignore the exception.
However, not able to find any such option on the UI.


pww: init at 0.0.6

Description of changes
https://github.com/abenz1267/pww

Simple program that emits a json with metadata info you can use in f.e. Waybar. Additionally you can let it auto-pause media players you provide, f.e. starting a video in your browser will pause spotify.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Thanks for your review, I implemented your requests :)