[FEATURE] Add Show Password Option

Describe the bug
When a user enters an incorrect password in the password field, the field does not automatically clear itself. Instead, the incorrect password remains in the field until manually deleted by the user.
To Reproduce
Navigate to the login page.
Enter an incorrect password in the password field.
Observe that the incorrect password remains in the field.
Expected Behavior
After entering an incorrect password, the password field should automatically clear itself, ready for the user to re-enter their password without having to manually delete the incorrect input.
Screenshot/ Video

No response
Code of Conduct

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

@AhmedKarray005 If we would have "show password" it is not relevant.
@kabir0x23 what do you think?

@AhmedKarray005 You don't need to clear input if we have show password so the user could change typo and so on.
Let's wait @kabir0x23 to hear his opinion

@kabir0x23 ??

@kabir0x23

hey @AhmedKarray005 @ArkadiK94 sorry for the late response

You don't need to clear input if we have show password so the user could change typo and so on.

this one is also a good option

about clearing password

if user keep entering wrong password, it will lock their account for 30mins
example user is using caps lock without knowingly.

i thhink this is better option

You don't need to clear input if we have show password so the user could change typo and so on.

@AhmedKarray005 I changed the title, to add show password button, would you like to handle it?
Don't remove the incorrect text on wrong submission.

@ArkadiK94 yes i would like to

@ArkadiK94
https://github.com/thecyberworld/TheCyberHUB/assets/104280658/d5344314-895f-469b-9328-76503f0606d3

@ArkadiK94

@AhmedKarray005 I think it is better to add some margin bottom to emphasize that this related to the password input box. Beside it, looks great thanks. Could you open a pr?

https://react-icons.github.io/react-icons/ u can get icons from here
import { BiHide,BiShowAlt } from "react-icons/bi";

@kabir0x23 @ArkadiK94
https://github.com/thecyberworld/TheCyberHUB/assets/104280658/6bbc4dc5-8986-4491-82db-e47509cf470b

@ArkadiK94 i create a pr?

@AhmedKarray005 yes, even though, I am not sure of the icons. I think when you see the password the eye should be without cross and the opposite.

@kabir0x23 I need to look for similar examples. Do you know which way is correct? Of the icons?

i think it's fine,
example

I found example :)
I don't know haha very hard question, who is right
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/7ce31203-698e-48a2-be4b-8a86c281b095

yes in facebook, it is showing what is there
in my example and his pr, it is about what to do
on buttons we do that task
here

arrow is above, when we click, it will bring the tab and icon will be down after words
without getting confused, it's fine  for now, will it decide later

example of linkedin,
show means open eye
hide means crossed eye

@kabir0x23 ok great, @AhmedKarray005 yes please open the pr

@AhmedKarray005 is it ok the someone else will work on it? I saw that someone opened new pr


Clarify inline code choice

Hello, as I am implementing the translation from ast to latex, I need to be sure the inline code command you will use.
As a draft I pushed a POC with \texttt command but you use \verb command (and it seems you write it like that :
\verb`code`

In the code review @vhf informed me of that and we found that \texttt is better rated than \verb. Do you confirm the use of \verb or do you want to switch to \texttt.

As it said, the \verb macro is for verbatim text, i.e. to tell LaTeX to interpret the characters without their special meaning. \verb|\code| => \code , \texttt{\code} => error (because LaTeX try to use the \code macros).

ok. So I have to use your syntax, I think?

I'd rather have rebber do some escaping than using \verb if \verb breaks the layout and \texttt does not.

moreover, I have this question to add : is this correct :
\verb`a
multiline
inlinecode`

It isn’t. An other solution is to use \texttt and escape special characters => \texttt{\textbackslash code}.

you mean \texttt{\textbackslash{code}} or replace every \\ by \textbackslash?

replace every \ by \textbackslash

This. Which is what I suggested here https://github.com/zestedesavoir/latex-template/issues/45#issuecomment-307168394 👍

ok, I do that

So, you shoud replace every special character.

This. Which is what I suggested here #45 (comment) 👍

Yep, I didn’t see.

Thanks!
This issue can be closed once the latex fixture gets updated to use \texttt instead of \verb. 💯

Please check everything is escaped properly in inline codes: https://zestedesavoir.github.io/zmarkdown/public/latex.html
If it's not the case, please provide md fixtures reproducing the issue and the corresponding expected latex. 👍

I test with \ $ & % # ^_ { } ~, so it should be right.


Update dependency @changesets/types to v5.2.1

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@changesets/types
5.0.0 -> 5.2.1

Release Notes

changesets/changesets
v5.2.1
Compare Source
Patch Changes

#1033 521205d Thanks @Andarist! - Added changedFilePatterns property to Config and WrittenConfig types.

v5.2.0
Compare Source
Minor Changes

#662 8c08469 Thanks @JakeGinnivan! - Added support for the privatePackages property in the config.

v5.1.0
Compare Source
Minor Changes

#858 dd9b76f Thanks @dotansimha! - Added a new config option: snapshot.prereleaseTemplate for customizing the way snapshot release numbers are being composed.

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: pnpm-lock.yaml

added 1 package, and audited 2 packages in 698ms

found 0 vulnerabilities
Scope: all 5 workspace projects
Recreating /tmp/renovate/repos/github/OctopusDeploy/util-actions/node_modules
Progress: resolved 1, reused 0, downloaded 0, added 0
Progress: resolved 28, reused 0, downloaded 19, added 0
Progress: resolved 38, reused 0, downloaded 38, added 0
Progress: resolved 74, reused 0, downloaded 49, added 0
Progress: resolved 125, reused 0, downloaded 101, added 0
Progress: resolved 189, reused 0, downloaded 165, added 0
Progress: resolved 263, reused 0, downloaded 230, added 0
.                                        |  WARN  deprecated<EMAIL_ADDRESS>packages/add-changeset                   |  WARN  deprecated<EMAIL_ADDRESS>Progress: resolved 344, reused 0, downloaded 313, added 0
Progress: resolved 452, reused 0, downloaded 425, added 0
Progress: resolved 565, reused 0, downloaded 532, added 0
packages/add-changeset                   |  WARN  deprecated<EMAIL_ADDRESS>packages/add-changeset                   |  WARN  deprecated<EMAIL_ADDRESS>packages/add-changeset                   |  WARN  deprecated<EMAIL_ADDRESS>packages/add-changeset                   |  WARN  deprecated<EMAIL_ADDRESS>Progress: resolved 675, reused 0, downloaded 642, added 0
Progress: resolved 739, reused 0, downloaded 715, added 0
 ERR_PNPM_PEER_DEP_ISSUES  Unmet peer dependencies

packages/add-changeset
└─┬ jest-runner-eslint 1.1.0
  └── ✕ missing peer eslint@"^6 || ^7 || ^8"
Peer dependencies that should be installed:
  eslint@"^6 || ^7 || ^8"

packages/extract-package-details
└─┬ jest-runner-eslint 1.1.0
  └── ✕ missing peer eslint@"^6 || ^7 || ^8"
Peer dependencies that should be installed:
  eslint@"^6 || ^7 || ^8"

packages/find-and-replace-all
└─┬ jest-runner-eslint 1.1.0
  └── ✕ missing peer eslint@"^6 || ^7 || ^8"
Peer dependencies that should be installed:
  eslint@"^6 || ^7 || ^8"

hint: If you want peer dependencies to be automatically installed, add "auto-install-peers=true" to an .npmrc file at the root of your project.
hint: If you don't want pnpm to fail on peer dependency issues, add "strict-peer-dependencies=false" to an .npmrc file at the root of your project.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: packages/add-changeset/package.json
Command failed: npm i -g<EMAIL_ADDRESS>&& pnpm install && pnpm build
npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=16.14"} (current: {"node":"14.18.2","npm":"6.14.15"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>


Ensure invocation target in delegated zcaps does not change beyond permitted path attenuation

Or determine that only the ID of the invocation target matters wrt. authority and any other properties are advisory only.

invocationTarget MUST be a string now; this is addressed.


The file '/Admin/ControlPanel/ControlBar.ascx' does not exist.

Description of bug
I installed a clean DNN 9.9.0 installation on Azure with:

dutch language
empty portal template

After installtion I clicked on go to the site and got a 500 error.
In the log file it said something like:
The file '/Admin/ControlPanel/ControlBar.ascx' does not exist.
So I checked the hostsettings table for the controlpanel entry and indeed it showed
'/Admin/ControlPanel/ControlBar.ascx'
After I changed it to
'DesktopModules/admin/Dnn.PersonaBar/UserControls/PersonaBarContainer.ascx'
I was back in business...
Steps to reproduce
List the precise steps to reproduce the bug:

Install a fresh DNN 9.6.1 (I did this on Azure, but I think it does not matter)
Select dutch as language
Select the Empty website template
Install and check what happens

Current behavior
Installation went ok, but the site gave a 500 error
Expected behavior
Well....you know, a working site might be nice...
Error information
2021-03-02 14:27:35.520+00:00 [dw0sdwk00009R][D:17][T:21][FATAL] DotNetNuke.Framework.PageBase - An error has occurred while loading page.
System.Web.HttpException (0x80004005): The file '/Admin/ControlPanel/ControlBar.ascx' does not exist.
at System.Web.UI.Util.CheckVirtualFileExists(VirtualPath virtualPath)
at System.Web.Compilation.BuildManager.GetVPathBuildResultInternal(VirtualPath virtualPath, Boolean noBuild, Boolean allowCrossApp, Boolean allowBuildInPrecompile, Boolean throwIfNotFound, Boolean ensureIsUpToDate)
at System.Web.Compilation.BuildManager.GetVPathBuildResultWithNoAssert(HttpContext context, VirtualPath virtualPath, Boolean noBuild, Boolean allowCrossApp, Boolean allowBuildInPrecompile, Boolean throwIfNotFound, Boolean ensureIsUpToDate)
at System.Web.UI.TemplateControl.LoadControl(VirtualPath virtualPath)
at DotNetNuke.UI.ControlUtilities.LoadControl[T](TemplateControl containerControl, String ControlSrc)
at DotNetNuke.UI.Skins.Skin.InjectControlPanel()
at DotNetNuke.UI.Skins.Skin.OnInit(EventArgs e)
at System.Web.UI.Control.InitRecursive(Control namingContainer)
at System.Web.UI.Control.AddedControl(Control control, Int32 index)
at System.Web.UI.ControlCollection.Add(Control child)
at DotNetNuke.Framework.DefaultPage.OnInit(EventArgs e)
at System.Web.UI.Control.d__225.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Web.Util.WithinCancellableCallbackTaskAwaitable.WithinCancellableCallbackTaskAwaiter.GetResult()
at System.Web.UI.Page.d__523.MoveNext()
Affected version

[x] 09.09.00 latest supported release

might be an issue of the "Blank Website" template, or do you experience the same with "Default Website" template?

A few questions on this, as I've seen this before in Azure a few times.

What type of resources were you using?  And what was their Size?  (For example P1V3 AppService & S2 SQL)
Did you experience a timeout from Azure, or DNN during the installation?
Do you remember the total duration of installation time?

Azure PaaS offerings have a hard-set timeout of 230 seconds of inactivity, which can translate to longer given the DNN status updates, but often if installing on low-grade stuff (S1 AppService or S0 DB) it is possible to have issues.

Install was a bit slow indeed, though it went trough in the end without any errors...
I'll try again on a local installation...

Well, looks like that was the issue, on my local installation it went trough fine.
Though you might want to remove that entry in the database/ install scripts anyway to prevent this from happening just in case..
I believe the Admin/ControlPanel/ControlBar.ascx was for DNN 8, so on a clean install for DNN 9 it would be better to remove it all together

For DNN 10.x we are looking to "roll forward' the schema script, until we do that it isn't exactly possible.
I'm curious, what scaling configuration were you using in Azure?  (We have done 30+ installs in Azure in the last month without any issues, so just trying to find the scale point that is an issue so we can possibly document)

I think I did the install using:
Gen5, 2 vCores, 32 GB storage, zone redundant disabled

+1 for this issue and the solution above.
Installed on a shared webhost, with no errors.

A few discussions regarding this topic on the forums.
Like https://dnncommunity.org/forums/aft/982/afpg/2 mentioning "powered Web App resource (plan)"

I have the same issue. Controls are missing in my installation folder. Tried downloading some of them from GitHub but Admin/Skins/ControlBar.ascx does not exist anywhere. Not sure what could be the resolution.

This just happened to me with 9.10.0 on a clean install in my local testing environment.
Had to do this:
 UPDATE HostSettings
SET SettingValue = 'DesktopModules/admin/Dnn.PersonaBar/UserControls/PersonaBarContainer.ascx'
WHERE SettingName = 'ControlPanel';

This just happened to me with 9.10.0 on a clean install in my local testing environment.
Had to do this:
 UPDATE HostSettings
SET SettingValue = 'DesktopModules/admin/Dnn.PersonaBar/UserControls/PersonaBarContainer.ascx'
WHERE SettingName = 'ControlPanel';

I'm going to run through the process again to confirm. I may have done something wrong. Was in a hurry, as usual

You have done nothing wrong. I kept struggling with this framework and ditched it in favor of something that actually works. A big issue I noticed is that you can solve the issue and out of sudden the next day the issue pops up again. Utter failure.

@patrickryan79
did you check the previous setting value for the ControlPanel?
Did you use any custom template for site or install?

@sleupold
Yes, I think it was ... Admin/ControlBar.ascx or something like that.
No custom. all default

thank you, I'll try to verify.

I am moving this issue to a discussion until it becomes actionable.


abolfazl

.content {
position: relative;
width: calc(var(--sz) * 5);
height: calc(var(--sz) * 3);
display: flex;
align-items: center;
justify-content: center;
}

good

good

Hello @abolfazletehad2887 ,
I will close this issues as I have no idea what you are trying to do.
Best Regards,
Jon


Multiple UI buttons open duplicates of the windows

Description

You can click these buttons multiple times and they will open the same window over and over. Looks like it's mostly an issue with the XAML UIs.
ChangelogButton.cs and VoteCallMenuButton.cs both do this.
All the buttons in the admin menu that open separate windows do this. Kick, Ban, Teleport, Permissions Panel etc.
You can open multiple of the Set Outfit and VV menus when you right click an entity.
On the lobby GUI you can open multiple of the credits, changelog, call vote and options menu.
Screenshots

https://user-images.githubusercontent.com/49448379/119173387-ba457500-ba56-11eb-8e5a-58defdab3692.mp4
Additional context

The expected function for these buttons is that they toggle on when you click them and open a window.  The button also needs to toggle off when the window closes through other means, such as the X on the top right of the window.

xamlui just emits a regular ui, just how you'd define it in c#. so it most definitely is not related to xamlui. seems like the (uicommand?)buttons aren't keeping track about the window instance they opened

this still exists for the vote menu. huh.

uimanager.createwindow uicontroller stuff largely fixes this afaict should make issues for specific cases


Update tugboat client.

This will send the user that deployed to tugboat. See https://github.com/remind101/tugboat/pull/38

Awwww yeahh :+1:


Add background-blend-mode to CSS

Adds background-blend-mode property and associated values to CSS mode.

Merged (without failing tests) as d395fec651fd31


feat: property combine function

@raveclassic hi, fixes https://github.com/raveclassic/frp-ts/issues/10
I'm not sure what to do with CHANGELOG.md though, could you assist?

@raveclassic 🔥


Include LICENSE files in source distribution

Summary
Closes https://github.com/astral-sh/uv/issues/2724.
Test Plan
Ran maturin sdist:

So confused as to why this works locally but the schema is rejected in CI.


Update auth.yaml

@googlebot I signed it!
On Sun, Oct 11, 2020 at 1:38 PM google-cla[bot]<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot
I signed it! and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fgooglecodelabs%2Forchestrate-with-kubernetes%2Fpull%2F35
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlecodelabs/orchestrate-with-kubernetes/pull/35#issuecomment-706667739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALKU5G4KLOUJGFQ74C5HJHTSKFRZTANCNFSM4SLTPPQQ
.


Change session id format to $sid + "-" + $memcachedId [ + "." + $jvmRoute ]

From martin.grotzke on April 26, 2009 00:24:50
To prevent conflicts with the jvmRoute the sessionId format should be
changed. The jvmRoute is appended to the sessionId with a dot (".").
So the new format might be $sessionId + "-" + $memcachedId [+ "." + $jvmRoute]
which would be the following regexp: [^-.]+-[^.]+(.[\w]+)?
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=3

From martin.grotzke on April 25, 2009 15:26:30
Done
Status: Fixed


baconjs-router Example using React and Webpack

I've just finished putting together this project using baconjs-router along with React and Webpack:
https://github.com/fknussel/baconjs-router-example
Live version is deployed to https://baconjs-router-example-razvcrijaf.now.sh/
Figured I'd leave it here for future reference as the example.html file on the source is not using React nor transpiling the library using the Babel loader for Webpack :)

yay thanks!  Will take a peak.

Merged in https://github.com/FoxSportsAustralia/baconjs-router/pull/1
Best thanks!

I lied, I merged it in after your additional PR.  But it's there now :D
Yay again! Thanks :D


context.Forward sometimes not working

I have a Luis dialog, where each intent handler forwards to another Luis dialog. For instance, I have something like this:
[LuisIntent("intent.first")]
public async Task FirstDialog(IDialogContext context, LuisResult result)
{
await context.Forward(new NextDialog(), ResumeAfter, new Activity { Text = result.Query }, CancellationToken.None);
}
Sometimes, when I type something to the bot, it gets to the context.Forward line but then just goes to the next line and ends without actually going into NextDialog. Other times, when I type the exact same thing, it does go into NextDialog and works properly.
Any ideas on what the issue could be with it sometimes working and sometimes not? How could it get to the context.Forward line but then not actually go into the NextDialog?

What does NextDialog do?
For example, does NextDialog.StartAsync call context.Wait(MessageReceived) on all code paths?

I'm going to close this issue - @asram363 feel free to open a new issue with more information.


Spinning out docs CSS into separate files

Changes
Takes the CSS in the base.njk file and puts it into a css file so it's easier to organize, especially since we're building a lot more into different templates
Testing
Docs tests pass, do not merge until checking visual preview
Docs
Docs only

Made some changes:

Fixed the missing font (see screens)
Minified & combined CSS for better perf
Deduped Styles
Added Sass so now we can nest more-easily

Before

After

Woo! Looks great!

1 small fix needed for the company logos: https://github.com/pikapkg/snowpack/commit/8634e43ef8d1f9114f259b8d82f6d049d7263795

Thank you @drwpow !!!


Forbid assertions in release builds (assert!, panic!)

I cannot think of any legitimate use case for release assertions for Rerun in non-test code. We never want the viewer to crash, nor the SDK.
The existing clippy machinery doesn't seem capable of differentiating between app code and test code, unfortunately.
Update: build scripts, compile-time code and other codegen shenanigans are another example where it's fine.

Related to:

#6330


Update to TypeScript 5.0

.d.ts changes look chill: https://gist.github.com/rictic/1ddd4afd56b88f008609d05f05e4ce80


[BUG] to_string does not work inside omp parallel

Hi,
here is a minimal example
#include <iostream>
#include <string>

int main()
{
  int a = 2222;
#pragma omp parallel for
  for (int i = 0; i < 10000; i++) {
    std::cout << a << std::endl;
    std::cout << std::to_string(a) << std::endl;
  }
}

without archer it prints 2222 all the time with archer to_string prints 4986765
Version:
clang version 3.9.1 (tags/RELEASE_391/final)
Target: x86_64-unknown-linux-gnu

Hi,
thanks for reporting this problem. This is actually a known problem, our static analysis pass does not work well with C++ code, especially with third-party library (such as the std lib).
As a temporary fix you can run archer without static analysis just removing this string -Xclang -load -Xclang /your/path/LLVMArcher.so from clang-archer and clang-archer++ drivers.

I replaced to_string with snprintf, so this kind of solved the problem. But I suspect that archer may miss some race conditions, because I get non-deterministic result in my program when I shouldn't. Would you advise me not to use C++ with archer at all?

As I said, Archer static analysis phase has troubles with C++, if you remove the pass from the archer commands it would only apply the dynamic analysis at runtime and everything should work fine even with C++.

OK. Thank you.

@simoatze: should this limitation be documented somewhere? In particular, how do can disable the static analysis?

@dongahn I was thinking to add an option in the archer drivers to disable the static analysis

yeah, that be great.


examples: Langchain ChatOpenAI integration doesn't work

LocalAI version:

63e1f8f
Environment, CPU architecture, OS, and Version:

AWs g5.xlarge (single A10)
Describe the bug

It seems langchain's ChatOpenAI expects <base_url>/engines/<model_name>/chat/completions to respond, but doesn't seem LocalAI exposes that --  I tried your example code and it has the same issue: https://github.com/mudler/LocalAI/blob/master/examples/langchain/langchainpy-localai-example/full_demo.py
To Reproduce

Try the langchain example code
Expected behavior

Langchain's ChatOpenAI should work just by setting the base url to LocalAI (e.g. http://local-ai-host:8080/v1)
Logs

Error from langchain: openai.error.InvalidRequestError: Cannot POST /v1/engines/thebloke__llama2-chat-ayt-13b-gguf__llama2-chat-ayt-13b.q5_k_s.gguf/chat/completions
Error in LocalAI debug: [<IP_ADDRESS>]:34992 404 - POST /v1/engines/thebloke__llama2-chat-ayt-13b-gguf__llama2-chat-ayt-13b.q5_k_s.gguf/chat/completions
Additional context

This is the output of our /models endpoint in LocalAI: {"object":"list","data":[{"id":"thebloke__llama2-chat-ayt-13b-gguf__llama2-chat-ayt-13b.q5_k_s.gguf","object":"model"}]}
Calling it with this curl command works: curl http://localhost:8000/v1/chat/completions -H "Content-Type: application/json" -d '{                                                                                "model": "thebloke__llama2-chat-ayt-13b-gguf__llama2-chat-ayt-13b.q5_k_s.gguf",      "messages": [{"role": "user", "content": "hi"}],          "temperature": 0.9    }'

the example is outdated and needs some love. did you installed the requirements with pip?

the example is outdated and needs some love. did you installed the requirements with pip?

The langchain version is very outdated in requirements... so no, I'm using the latest Langchain (0.0.354).  I did a quick hack for now to fix it in api.go by adding this line app.Post("/v1/engines/:model/chat/completions", auth, openai.ChatEndpoint(cl, options))

Sorry, it was my mistake... you are right, as mentioned in another issue I was using an outdated openai package.


Chip sometimes does not expand in height for multiline Text

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Flutter 3.10.5
For text with 2 lines if the length is not long enough Chip may not increase height.
Expected results

Actual results

Code sample
Code sample
Chip(
          label: Text(
            getMenuName(menu),
            style: TextStyle(color: Colors.white),
            softWrap: true,
            maxLines: 3,
          ),
          backgroundColor: Colors.blueAccent,
          clipBehavior: Clip.none,
        ),

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.10.5, on Microsoft Windows [Version 10.0.19043.2364], locale en-US)
    • Flutter version 3.10.5 on channel stable at D:\Framework\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (6 weeks ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\khang\AppData\Local\Android\Sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.2)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31410.357
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2022.2)
    • Android Studio at D:\Software\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[√] Connected device (4 available)
    • SM S901N (mobile) • R5CT32Z693Z • android-arm64  • Android 13 (API 33)
    • Windows (desktop) • windows     • windows-x64    • Microsoft Windows [Version 10.0.19043.2364]
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 114.0.5735.199
    • Edge (web)        • edge        • web-javascript • Microsoft Edge 115.0.1901.183

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thank you for responding. I have updated flutter to the latest version 3.10.6 but it seems the problem still occurs. Here is the sample code. I am testing on Samsung S22. Text length may need to change with each device for errors to occur.

class TestPage extends StatefulWidget {
const TestPage({Key? key}) : super(key: key);
@override
State createState() => _TestPageState();
}
class _TestPageState extends State {
@override
Widget build(BuildContext context) {
return Center(
child: Chip(
label: Text(
"TEST1 TEST2 TEST3 TEST4 TEST5 TEST6 TEST7 TEST8 TES",
style: TextStyle(color: Colors.white),
softWrap: true,
maxLines: 3,
),
backgroundColor: Colors.blueAccent,
clipBehavior: Clip.none,
),
);
}
}

Hi @khanghyxda. Thanks for explaining further, I am able to reproduce this issue on the latest stable channel. This issue seems to occur with certain lengths of text relative to the device.

With a Certain length of text
With a letter addition to the previous length

I can produce a slightly different behaviour on the master channel, the texts after the first line is not visible at all

This issue is reproducible on both Android and iOS Device.
flutter doctor -v

[✓] Flutter (Channel stable, 3.10.6, on macOS 13.4 22F66 darwin-arm64, locale
    en-NG)
    • Flutter version 3.10.6 on channel stable at /Users/dammya/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (13 days ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version
    32.1.0-rc1)
    • Android SDK at /Users/dammya/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/dammya/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.80.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 11 (API 30) (emulator)
    • iPhone 14 Pro (mobile)    • DCF2031A-7EA9-42FF-8CD4-D4584F3E7A6D • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)           • macos                                •
      darwin-arm64   • macOS 13.4 22F66 darwin-arm64
    • Chrome (web)              • chrome                               •
      web-javascript • Google Chrome 114.0.5735.248

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[✓] Flutter (Channel master, 3.13.0-10.0.pre.27, on macOS 13.4 22F66 darwin-arm64, locale en-NG)
    • Flutter version 3.13.0-10.0.pre.27 on channel master at /Users/dammya/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 618ef7f236 (2 hours ago), 2023-07-26 02:31:08 -0400
    • Engine revision df12fff329
    • Dart version 3.1.0 (build 3.1.0-348.0.dev)
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/dammya/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/dammya/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.80.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone arm64 (mobile) • emulator-5554                        • android-arm64  • Android 11 (API 30) (emulator)
    • iPhone 14 Pro (mobile)    • DCF2031A-7EA9-42FF-8CD4-D4584F3E7A6D • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)           • macos                                • darwin-arm64   • macOS 13.4 22F66 darwin-arm64
    • Chrome (web)              • chrome                               • web-javascript • Google Chrome 114.0.5735.248

[✓] Network resources
    • All expected network resources are available.

• No issues found!

This may be due to the text layout not being limited by the shape of the chip. For example if the chip's child is SizedBox(height: 200, child: Text(...))

@LongCatIsLooong - I assume that we can't really do the kind of boundary fitting text layout that this example implies.  I don't believe that we can ask a shape to compute the smallest (text layout) box it contains either.
It's quite possible to do this kind of thing correctly, see for example http://hansmuller-webkit.blogspot.com/2014/03/a-simpler-algorithm-for-css-shapes.html, but we don't have plans to do so.
import 'package:flutter/material.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark().copyWith(
        scaffoldBackgroundColor: darkBlue,
      ),
      debugShowCheckedModeBanner: false,
      home: const TestPage(),
    );
  }
}

class TestPage extends StatefulWidget {
  const TestPage({Key? key}) : super(key: key);

  @override
  State createState() => _TestPageState();
}

class _TestPageState extends State {
  @override
  Widget build(BuildContext context) {
    return const Scaffold(
      body: Center(
        child: Chip(
          label: SizedBox(height: 200, child: Text(
            "TEST1 TEST2 TESTTEST4 TEST5 TEST6 TEST7 TEST8 TEST ES",
            style: TextStyle(color: Colors.white),
            softWrap: true,
            maxLines: 3,
          )),
          backgroundColor: Colors.blueAccent,
          clipBehavior: Clip.none,
        ),
      ),
    );
  }
}

This bug is still on with flutter 3.27.1
It is annoying because I do have dynamic text which is entered into these chips and I cannot prevent the label from having this "breaking but not breaking"-length.
Do you have any suggestions for a workaround maybe?


ArduCAM-Mini-2MP schematic

Hi @ArducamSupport ,
we have purchased three ArduCAM-Mini-2MP boards and we would like to probe VSYNC signal on the board. We are unable to find the corresponding pin.
It would be great if you can provide [ArduCAM-Mini-2MP] schematics or help us finding the VSYNC signal pin on the board
Thanks in advance.

hi, where you by a ArduCAM-Mini-2MP boards?


generate random struct and other data types

Is your feature request related to a problem? Please describe.
Currently, for example, we declare a source with struct as one of the fields and the connector = datagen.
create source will succeed.
And only when we create a materialized on top of the source, does the FieldGeneratorImpl complain that this is not implemented yet.
Describe the solution you'd like
We just implement a random data generator for struct type.
Describe alternatives you've considered
No response
Additional context
There are a couple of other data types that can not be generated now.

We still need some timestamp related gen


⚠ ClamAV has degraded performance

In dca4fde, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 52 ms

Resolved: ClamAV performance has improved in efc09a4.


Error code 400, GROQ context limit exceeded.

I was trying the LLM extraction  strategy to extract data from a website using GROQ api as a LLM backend with groq/llama3-70b-8192 and i am having an error
[ERROR] 🚫 Failed to crawl https://www.cen-change.com/achat-vente-devise/#Cours-et-Taux-de-Change, error: litellm.BadRequestError: BadRequestError: GroqException - Error code: 400 - {'error': {'message': 'Please reduce the length of the messages or completion.', 'type': 'invalid_request_error', 'param': 'messages', 'code': 'context_length_exceeded'}} None
I did change the instruction and lowered the lenght of it but still having this error.
How can i debug it ? thank you

That's true. Unfortunately, the version of Llama3 that Groq supports currently has a short context window of 8K. When they release the paid plan, you'll hopefully get a longer context window. However, you can extract the markdown and use chunking strategies to manage it. We might add the ability to summarize content for models with smaller context windows to our backlog. For now, you can use other models that support longer windows, like OpenAI, Claude, or open-source models like ollama/Llama3 or Phi from Microsoft, which offer much longer windows, like 128K. These should be sufficient for most tasks.


Sprite flickers then stops rendering completely when despawning and respawning it

Bevy version
0.9, but I also compiled against earlier releases (down to 0.5) and they all behaved the same.
What you did
This is a minimal reproduction of a bug I encounter in my game. There is a system that creates a sprite entity if none already exists so that there is always one. There is another system that despawns said sprite, on command, by pressing D (exactly what triggers the despawn doesnt matter though, this just gives manual control).
use bevy::prelude::*;

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_startup_system(setup)
        .add_system(spawn_sprite_if_not_there)
        .add_system(despawn_sprite_on_keypress)
        .run();
}

fn setup(mut commands: Commands) {
    commands.spawn(Camera2dBundle::default());
}

fn spawn_sprite_if_not_there(
    mut commands: Commands,
    query: Query<(Entity, &Sprite)>,
    asset_server: Res<AssetServer>,
) {
    let sprite_exists = query.iter().count() > 0;
    if sprite_exists {
        return;
    }

    commands.spawn(SpriteBundle {
        texture: asset_server.load("image.png"),
        ..Default::default()
    });
}

fn despawn_sprite_on_keypress(
    mut commands: Commands,
    query: Query<Entity, With<Sprite>>,
    keyboard_input: Res<Input<KeyCode>>,
) {
    if keyboard_input.just_pressed(KeyCode::D) {
        for entity in query.iter() {
            commands.entity(entity).despawn();
        }
    }
}

What went wrong
Initially the sprite appears as expected. When I press D to despawn it, another identical sprite entity is created, but it seemingly renders for a single frame then disappears. The entity exists, but it's not displaying.
Pressing D another time to delete the sprite and let another one be created results in a single frame flicker of the sprite's image and then back to nothing.
I would expect to just see the sprite.
Additional information
What I think is the most important bits of information I have is this:

Modifying the code to spawn multiple identical sprites instead of one will continue to replicate the bug only if they are all deleted at the same time. If deleting the sprites one by one (across frames), then the bug doesn't occur. In other words, it seems that the bug only occurs when at some point, the Image is no longer used by any entity and then used again in a later frame. So long as it remains in use by something, the bug doesn't occur.
I checked if it happens when using UI nodes instead of sprites, and it does.

Other pertinent information:

As mentionned above, I checked across Bevy releases to see if this was new to 0.9, but it's present in all the verisons I checked.
I checked on my Linux install on my desktop with a AMD GPU testing with Vulkan, on my Windows install on the same desktop testing with both Vulkan and Dx12, and on my laptop's integrated graphics on Linux. The bug happens in all of them.
The ordering of the two systems doesn't matter. Putting the systems in different stages doesn't matter.

By despawning your sprite, you trigger asset cleanup for your image.
When re-adding the sprite just after, the image is being reloaded, but cleanup is not unscheduled.
Asset cleanup happens next, and there isn't an image to display anymore


separate of storage.

#1417

Current coverage is 39.61% (diff: 0.00%)

Merging #2461 into master will increase coverage by <.01%

@@             master      #2461   diff @@
==========================================
  Files          1660       1660
  Lines         58406      58406
  Methods       11324      11324
  Messages          0          0
  Branches       8116       8116
==========================================
+ Hits          23136      23138     +2
+ Misses        35270      35268     -2
  Partials          0          0

Powered by Codecov. Last update ab79f72...1471df9


🌐 Add German translation for docs/de/docs/advanced/async-tests.md

Issues
← advanced/testing-dependencies.md (#10706)
→ advanced/settings.md (#10709)
German translation progress

📝 Docs preview for commit 58c26b327a17be2475ace2b95fbd88951c0ed47e at: https://f24dd7bb.fastapitiangolo.pages.dev

📝 Docs preview for commit 710a1a134864d38c4cecf6082e7285ac8c52fcf0 at: https://37ac3151.fastapitiangolo.pages.dev

📝 Docs preview for commit df68d7cafae72727c735c64bd5f5e1d4ddbccc8a at: https://28658617.fastapitiangolo.pages.dev

📝 Docs preview for commit e3476296e4ad489e252463d993b3d8016f10bb98 at: https://803dd44f.fastapitiangolo.pages.dev

📝 Docs preview for commit 58e0186621a3c04660cbaeadc2b67753eabced96 at: https://a1b8651e.fastapitiangolo.pages.dev

Awesome @nilslindemann! :rocket:
As you are the champion of German translations and you have done so many, and with so much attention to detail, I'm gonna go ahead and merge this. :tada:
If others see things that could be improved later, we can check that in new PRs. :nerd_face:


Fix the lambda modules for the elasticsearch example

Issue #, if available: https://github.com/aws-samples/graviton2-workshop/issues/7
Description of changes: Specify the package versions to prevent unexpected upgrades
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Successfully tested the updates to the requirements file with @mbednarzAWS


Services: first build: abs added

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_21u1drl2j7/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:70: documentation_overspecified: Using doc_url and doc_source_url is overspecified
clone/recipe/meta.yaml:70: license_file_overspecified: Using license_file and license_url is overspecified.

===== ERRORS =====

clone/recipe/meta.yaml:17: pip_install_args: pip install should be run with --no-deps and --no-build-isolation.
===== Final Report: =====
1 Error and 2 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_87fq_i9382/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:18: avoid_noarch: noarch: python packages should be avoided

===== ERRORS =====

clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
clone/recipe/meta.yaml:66: missing_license_family: The recipe is missing the about/license_family key.
===== Final Report: =====
2 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_f6frk064uv/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:21: avoid_noarch: noarch: python packages should be avoided

===== ERRORS =====

clone/recipe/meta.yaml:70: missing_license_family: The recipe is missing the about/license_family key.
===== Final Report: =====
1 Error and 1 Warning were found


Why not compress gzip with best-level compression?

Just glancing through your code and it seems like you may as well use best compression rather than default, since you're only going to compress each file once and then store in memory for a long time.

My understanding was that level 6 is somewhat of a sweet-spot for gzip compression - and best compression makes compile times even longer. But you make a fair point. We can trade of some compilation speed for runtime performance.

Published in v0.4.5


notcurses: 2.4.8 -> 2.4.9

Motivation for this change
Bump
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)
[X] Tested, as applicable: Ran the notcurses demo - works on my machine!(TM)

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

[X] 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/)
[X] Fits CONTRIBUTING.md.


Cannot install CLI package on win10 embedded Ubunto 18

When following the instructions on:
https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest
The step:
sudo apt-get install apt-transport-https
I get:
allan@AP-ZBOOK:~$ sudo apt-get install apt-transport-https
E: Malformed entry 1 in list file /etc/apt/sources.list.d/azure-cli.list (Component)
E: The list of sources could not be read.
E: Malformed entry 1 in list file /etc/apt/sources.list.d/azure-cli.list (Component)
E: The list of sources could not be read.
Any ideas?
Regards,
Allan

Delete the offending file (/etc/apt/sources.list.d/azure-cli.list) and try following the instructions again: https://docs.microsoft.com/en-us/cli/azure/install-azure-cli-apt?view=azure-cli-latest

Is CLI 2.0 not supported in Ubunto 18, running embedded on Win10 ?

@allanpoulsen There is not an official Bionic package available. See the top of the install page for the complete list of supported distributions. If you would like to try installing the Xenial package, you can manually set AZ_REPO to xenial rather than running lsb_release.


Add property to track deleted/to be deleted display objects

Calling display.remove( object ) or object:removeSelf() will not actually delete the display object until the next frame. In some rare instances, this can result in bugs and crashes if the developer attempts to perform certain operations on the objects after they've been removed (via references, for instance).
Some developers are currently checking to see if object.x or object.width are nil, for instance, to determine if a display object has been removed or not, but these properties only update the next frame after Solar2D has properly disposed of the display objects.
By including this new "private" convenience property, object._isRemoved = true, developers would always be able to know if an object has been removed or is about to be removed by next frame. Naturally, this would only affect display objects removed via display.remove, but it has long been Solar2D's recommend way of removing display objects.
Code example of the issue:
local rect = display.newRect( 0, 0, 100, 100 )

display.remove( rect )
print( rect.width ) -- 100

timer.performWithDelay( 1, function()
	print( rect.width ) -- nil
end )

This would also apply to widgets, for instance:
local widget = require( "widget" )

-- Function to handle button events
local function handleButtonEvent( event )
    if ( "ended" == event.phase ) then
        print( "Button was pressed and released" )
    end
end

-- Create the widget
local button1 = widget.newButton(
    {
        left = 100,
        top = 200,
        id = "button1",
        label = "Default",
        onEvent = handleButtonEvent
    }
)

display.remove( button1 )
print( button1._isRemoved ) -- true

This single property would remove the guesswork from knowing whether a display object in Solar2D has been removed or not.

why add even more weird api? just add display.isValid(object) & object:isValid()
like:
function display.isValid(object)
      return object ~= nil and object.x and object._isRemoved ~= true
end

object:isValid() would crash if object is nil, since you couldn't make the method call.
display.isValid( object ) would have a confusing name as it wouldn't actually validate if a given display object wasn't invalid or corrupted. It could be called display.isRemoved( object ), but if turned into a separate API, then the object._isRemoved property would need to be included in the method call as well and your example would need to be further modified.

object:isValid() would crash if object is nil, since you couldn't make the method call.

thats why display.isValid(object) needed

display.isValid( object ) would have a confusing name

what confuses you?
valid means - its object, its not removed, is it safe to work with this object.

as it wouldn't actually validate if a given display object wasn't invalid or corrupted. It could be called display.isRemoved( object ), but if turned into a separate API, then the object._isRemoved property would need to be included in the method call as well and your example would need to be further modified.

I mean, _isRemoved you want to add is a weird and little-functional api.
display.isValid would general purpose, this would help not only in your case, but also in other cases.
you suggest using if obj and obj.x and obj._isRemoved ~= true then ... end
instead, it would be better to have a good api, rather than using workarounds

thats why display.isValid(object) needed, method its alternative way for oop enjoyers

You literally cannot have a method to do a check like this. If the display object has been removed and has been properly set to nil already, then trying to call object:isValid() when object = nil, you will encounter an immediate runtime error and the application will crash. Using display.isValid( object ) would be the only option for something like this.

what confuses you?

Like I said, checking if 1) a given input is a valid display object, or 2) a given input is/was a display object that has been removed, are two different things.
I do agree that adding said convenience function would be a better approach, but its name shouldn't be display.isValid() because it does not check for the validity of a display object nor could it do so reliably. There's no way to identify if an image's texture is corrupted at that point if it was created using display.newImageRect(), for instance. If I was going to use an API to check if an input was a valid display object, then I'd expect it to check that it's not corrupted and not just that it hasn't been removed.

I agree that a convenience API would be a cleaner approach, but it does still require the display object property to be added. It would, however, need to be inserted inside the object.removeSelf() method call so that it covers all cases. This PR simply needs to be modified and extended.


[Feature]: Remove collapse feature from Navigation Drawer

Feature Summary
Left swipe hides the navigation drawer on mobile. We should remove the collapse functionality and always use the full size nav on desktop and the collapsed size on mobile.
From @itsalaidbacklife:

Actually let’s simplify the overall component by removing the toggle-able feature and just making it small on mobile and big on desktop. Then we don’t need to worry about the expand/collapse button conflicting with the logout button at the bottom of the drawer and generally the whole thing will be less complex.

This way we can remove the click handling functionality and the entire button for expand/collapse, and skip the associated data variable userHasCollapsed along with the isCollapsed computed prop. We can just use the breakpoint.

Detailed Description

I'd love to work on this if that's OK. @seriouslysean, Please assign me this issue.....

@singhkaran202 by all means, would love to have you! Try following the project setup, https://github.com/cuttle-cards/cuttle#project-setup-and-development, to see if you can get this running locally.
We use Vuetify 2 for the general layout so you might need to dig through their docs a bit to make sure you’re using the right syntax. https://vuetifyjs.com/en/components/grids/
Reach out in the #coding channel of our discord if you need help, https://discord.gg/9vrAZ8xGyh, and come try out Cuttle! Or, just ping @itsalaidbacklife, @peterancourt or myself on here and we’ll offer some assistance that way.
Best of luck!

Thanks alot!! @seriouslysean
Lessgo 🚀🚀

I think this can be solved by adjusting the isCollapsed() method in the NavigationDrawer.vue component, something like this:
isCollapsed() {
  // Always collapse on mobile regardless of user collapse setting
  if (this.isSmallDevice) {
    return true;
  }
  return this.userHasCollapsed;
},

Actually let’s simplify the overall component by removing the toggle-able feature and just making it small on mobile and big on desktop. Then we don’t need to worry about the expand/collapse button conflicting with the logout button at the bottom of the drawer and generally the whole thing will be less complex.
This way we can remove the click handling functionality and the entire button for expand/collapse, and skip the associated data variable userHasCollapsed along with the isCollapsed computed prop. We can just use the breakpoint.
@singhkaran202 does that make sense? Thank you for bearing with us. I know it’s a change to the original description but I think the end result will be simpler for devs and for users

@itsalaidbacklife makes more sense to me, nice 👍🏼

@itsalaidbacklife Yupp, I got that!!

Closing because instead we're going to completely remove the nav drawer and replace it with an upper nav bar


Outgoing commit warning message

I have the following message in VS2019 "The current branch does not track a remote branch. Push your changes to a new branch on the origin remote and set the upstream branch.  Learn more."  The learn more is a link to this page, but the page does not mention what this means, how to perform said actions and just outputs some error message in the output window.
"Pushing Tenant
Error encountered while pushing branch to the remote repository: Git failed with a fatal error.
repository 'repository url here' not found.

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

ID: 6d5ca6b5-523a-21d0-26f3-567846f7c298
Version Independent ID: c712c338-46d6-a70a-0ba1-0d1ec3cd6ac7
Content: Sync your changes to a remote Git repo - Azure Repos
Content Source: docs/repos/git/pushing.md
Product: devops
Technology: devops-code-git
GitHub Login: @steved0x
Microsoft Alias: sdanie

@Allann  Please see if the solution for issue https://github.com/MicrosoftDocs/vsts-docs/issues/5253 addresses this problem.

This was not an issue with my email address, this was the lack of information on correcting the problem in documentation we were lead to by the learn more link.
But thank you for letting me know, I don't use any other email addresses in development so find it interesting that that might be the problem.

@Allann Thank you for the follow-up.
Sorry, I had intended to link to issue https://github.com/MicrosoftDocs/vsts-docs/issues/5223 that more specifically addresses setting a remote.


Add Travis CI deploy workflow

[ ] Make Travis build docs automatically
[ ] Make Travis push docs to external repo (needs read/write access)
[ ] Make sure it happens only when code is committed to a specific branch (master)

Thanks for adding this. Will we still push the release docs to RTD manually?

@cekees as releases also come from master and docs would be rebuilt automatically with every commits to master, the idea would be that we would not have to push any new documentation files manually when there is a release, but we would only have to:

create a new branch (e.g. 1.7.3) on https://github.com/erdc/proteus-docs from the branch where the latest (release) docs were automatically built (e.g. gh-pages)
let RTD know about that new branch

It might actually be possible to have Travis automatically create new branches on proteus-docs when there is a new tagged release, but we would still have to tell RTD about it

@cekees @zhanga This is ready for review, it will allow for automatic generation of the docs every time there is a new commit in master, and push it to the proteus-docs master. I have added some safety checks to ensure that it adds, commits, and pushes it to the right repo


FreeBSD installation

Installation in a FreeBSD/FreeNAS Jail
Purpose
The purpose of this branch is to allow simplified installation of the Homebrewery project in a FreeBSD or FreeNAS jail. This is currently not possible as Docker is currently broken on FreeBSD and has been for some 15+ months, so the provided Dockerfiles do not allow the project to run.
Work Completed So Far

Get Homebrewery installed and working on a FreeBSD jail - COMPLETE
Determine minimum viable package (MVP) installation required for previous step - COMPLETE
Script installation of MVP - COMPLETE
Test with multiple available versions of MongoDB - COMPLETE
Create rc.d files to run Homebrewery as a service, supporting service commands "start", "stop", and "restart" - COMPLETE
Script installation of Homebrewery as a service - COMPLETE
Parameterize configuration options NODE_ENV and PORT - COMPLETE
Eliminate requirement for the current working directory to contain the server.js script - COMPLETE

Future Improvements/In Development

Add custom command "devmode" to service options to start in development auto-rebuild mode
Write FreeBSD install documentation and add to repository - IN PROGRESS (see instructions here)

Integration Considerations
FreeBSD/Install.sh
This script currently clones from my fork (https://github.com/G-Ambatte/homebrewery.git) rather than the master project. This will need to be corrected during the integration process.
The script also currently clones the project into a new "homebrewery" folder in the root directory; whereas another location may be deemed to be more appropriate (i.e. "/usr/local/etc/homebrewery"). This will have flow on effects to the freebsd/rc.d/homebrewery file.
Finally, this script runs the command npm audit fix -force, which updates and corrects any security issues with the NPM modules. However the inclusion of the -force parameter allows the installation of potentially breaking changes to the required packages, and while this has not caused any issues to date, the -force parameter should be removed from the default installation script before wider promulgation, in order to reduce the potential for future issues.
FreeBSD/RC.D/HomeBrewery
If the install directory for Homebrewery is changed, the "exec_cmd" will need to be changed to the new location of the "server.js" file.
Modified Files and Rationale
Server.js

As the service starts the command from an as yet undetermined working directory, Homebrewery would immediately crash due to files being unable to be found. The addition of process.chdir(__dirname); towards the start of the "server.js" file changes the current working directory of the calling thread to the directory containing the script, eliminating this as a source of issues.
Attempting to set the NODE_ENV environment variable prior to running the command "node server.js" was possible but not particularly clean, especially when writing the command into the script. Instead, I pushed this value into the config/default.json file as the key pair "environment", with a hard-coded default value of "local" if it is not set via environment variable or in the default configuration file.
Similar arguments can be made of the PORT environment variable, so I pushed this to config/default.json as the key pair "web_port". Again, if not set via environment variable or default configuration, it reverts to the hard-coded value of "8000".

Config/Default.json
New key pairs "environment" and "web_port" were added to support the changes as previously discussed in server.js.
Conclusion
I hope that I have managed to lay out a clear and concise description of the changes made and the reasoning behind them in order to simplify the process of installing Homebrewery in a FreeBSD jail.
I look forward to receiving any feedback that you might have.
-- G

This looks pretty good so far. This will certainly need some documentation for users who want to try this; we can link to it from the main repository readme.
Also, since I am not a FreeBSD user myself, you would likely be the main point of contact for any questions about this process.

you would likely be the main point of contact for any questions about this process

I don't have an issue with that, although I will add that I personally rate my FreeNAS/BSD experience as "enthusiast" rather than "professional", so all I can promise is my best effort.
I've pushed a few more changes to the PR, as per the following:
Task List
Improvements

[x] Write FreeBSD install documentation
[ ] Add devmode to service commands

Integration Considerations

[x] Remove -force from npm audit fix in freebsd/install.sh
[x] Point install.sh and install documentation at naturalcrit/homebrewery rather than the fork
[x] Change install location from /homebrewery to a more appropriate location, /usr/local/homebrewery

-- G

With the changes I've just pushed through, that should complete the last of the outstanding items from Improvements and Integration Considerations.

Task List
Improvements

[x]  Write FreeBSD install documentation
[x]  Add dev_mode to service commands

Integration Considerations

[x]  Remove -force from npm audit fix in freebsd/install.sh
[x]  Point install.sh and install documentation at naturalcrit/homebrewery rather than the fork
[x]  Change install location from /homebrewery to a more appropriate location, /usr/local/homebrewery

If you think there are any further changes that need to be made, or have any thoughts on improvements or changes, then please let me know.
-- G

Awesome. I'll take another look over this and let you know if I notice anything.

I just pushed a couple of commits through now; npm run lint and CircleCI returns cleanly now - the major one was eliminating a few lines of debugging code, which dropped the line count below the 200 limit.

Ok, it all looks good to me now. Do you want to put a link to your readme on the main readme file, similar to the "how to install with Docker" section?

Ok, it all looks good to me now. Do you want to put a link to your readme on the main readme file, similar to the "how to install with Docker" section?

I've done a basic copy and paste of the Docker section, and modified the link to point to the FreeBSD install instruction file.

I've done a basic copy and paste of the Docker section, and modified the link to point to the FreeBSD install instruction file.

...and I've just seen that the MongoDB package has been updated, which breaks the installation script. I'll push a commit that fixes it through later today.

...and I've just seen that the MongoDB package has been updated, which breaks the installation script. I'll push a commit that fixes it through later today.

I've pushed a change that fixes the install script - I've run this through my standard test process (creating a blank jail, running the script, and seeing if I have a working Homebrewery at the end of the process) and it worked.
I've also modified the service script so the user can easily change the NODE_ENV and port variables - they default to local and 8000 but the user can run sysrc homebrewery_NODE_END="local" or sysrc homebrewery_port=8000 to set the values for the service to use.

I've pushed a change that fixes the install script - I've run this through my standard test process (creating a blank jail, running the script, and seeing if I have a working Homebrewery at the end of the process) and it worked.
I've also modified the service script so the user can easily change the NODE_ENV and port variables - they default to local and 8000 but the user can run sysrc homebrewery_NODE_END="local" or sysrc homebrewery_port=8000 to set the values for the service to use.

Looks good to me! I'll push this live!

Looks good to me! I'll push this live!


Update ColumnarBatch.java

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
using Java 8 stream api which is an improved version in terms of performancer and will also help in parallelism in future.

Can one of the admins verify this patch?

Spark does not require Java 8, so this wouldn't compile. It's a bit less clear and less performant (even though it won't matter here). In general, PRs should have a descriptive title, and "Update X" is not sufficient. Please close this.


Spec 1 block finality

I created the following document to specify our changes to OP for 1-block finality: https://www.notion.so/clabsco/1-block-finality-0b7efa6b2c0a41909cd2a8f84947c011
TLDR:

This is about avoiding reorgs based on Ethereum reorgs. It's implemented by using only finalized Ethereum blocks as l1 origin blocks. Sideffects are slower deposits and withdrawals.
This proposal does not solve the problem of fault-proof induced reorgs. As we (initially) run the sequencer, this should not be a problem, but people will point this out. [maybe we should name this differently?]
 The contract for challenges does not exist yet and it's unclear how challenging windows and sequencing windows would interact.


Language version package proposal

A simple package with a LanguageVersion interface (and default implementation) to facilitate sharing of Dart language versions between different tools and packages. Currently Version from pub_semver is used by a few packages for this purpose, but it's not meant for language versions and is a bit heavy weight. The Version class in pub_semver could also implement this interface or have an extension to convert to this.
As Dart language versions are being used by more tools, some utility functions might make sense to share as well.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

3 of 5 (60.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-33.0%) to 60.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkgs/language_version/lib/src/concrete.dart
3
5
60.0%

Totals

Change from base Build<PHONE_NUMBER>4:
-33.0%

Covered Lines:
3

Relevant Lines:
5

💛  - Coveralls


cleanup: remove test/src-not-used/

Description
This PR removes unused project:  test/src-not-used/

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[x] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 9484

LGTM - smoketests aren't required if this was dead/unused code to begin with.

@blueorangutan test matrix

@DaanHoogland a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

LGTM - smoketests aren't required if this was dead/unused code to begin with.

right, didn't check that (or the PR number) I stopped the jobs and we should be fine to merge.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 12.24%. Comparing base (9d5d4e5) to head (78ee605).
Report is 1 commits behind head on 4.18.

Additional details and impacted files
@@            Coverage Diff             @@
##               4.18    #9007    +/-   ##
==========================================
  Coverage     12.24%   12.24%
- Complexity     9291     9292     +1
==========================================
  Files          4698     4698
  Lines        414259   414259
  Branches      51377    50710   -667
==========================================
+ Hits          50707    50716     +9
+ Misses       357251   357241    -10
- Partials       6301     6302     +1

Flag
Coverage Δ

unittests
12.24% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@blueorangutan test

@weizhouapache a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-10171)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41717 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9007-t10171-kvm-centos7.zip
Smoke tests completed. 108 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_create_pvlan_network
Error
0.07
test_pvlan.py

test_01_vpc_site2site_vpn
Failure
282.99
test_vpc_vpn.py

@blueorangutan test matrix

@weizhouapache a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-10185)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 45729 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9007-t10185-xenserver-71.zip
Smoke tests completed. 109 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_create_pvlan_network
Error
0.08
test_pvlan.py

[SF] Trillian test result (tid-10186)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8
Total time taken: 51582 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9007-t10186-vmware-67u3.zip
Smoke tests completed. 110 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

the test failures are not related to this pr
marvin.cloudstackException.CloudstackAPIException: Execute cmd: createnetwork failed, due to: errorCode: 431, errorText:The VLAN tag to use for new guest network, 2468 is already being used for dynamic vlan allocation for the guest network in zone pr9007-t10185-xenserver-71

and
Exception: Job failed: {accountid : '00dc36b4-0cb2-11ef-b0f4-1e00f60001d2', userid : '00dd8de2-0cb2-11ef-b0f4-1e00f60001d2', cmd : 'org.apache.cloudstack.api.command.admin.vm.MigrateVMCmd', jobstatus : 2, jobprocstatus : 0, jobresultcode : 431, jobresulttype : 'object', jobresult : {errorcode : 431, errortext : 'Cannot migrate VM, destination host is not in correct state, has status: Connecting, state: Enabled'}, jobinstancetype : 'VirtualMachine', jobinstanceid : 'aa2151a3-6879-4951-81ee-af6892ecbd5f', created : '2024-05-08T06:27:17+0000', completed : '2024-05-08T06:27:17+0000', jobid : '56824238-11a0-4adb-a089-76d8886476d9'}


Error compiling with Android

Hello,
I am having errors compiling the react-native-maps for Android - I have followed all of the steps outlined in the Installation instructions, but when I run react-native run-android, it start compiling then fails with :
:react-native-maps:compileReleaseJavaWithJavac
/Users/gsiry/Documents/kako/node_modules/react-native-maps/android/src/main/java/com/airbnb/android/react/maps/AirMapManager.java:72: error: getCurrentActivity() is not public in ReactContext; cannot be accessed from outside package
        return new AirMapView(context, this.appContext.getCurrentActivity(), this, this.googleMapOptions);
                                                      ^
/Users/gsiry/Documents/kako/node_modules/react-native-maps/android/src/main/java/com/airbnb/android/react/maps/RegionChangeEvent.java:16: error: no suitable constructor found for Event(int)
        super(id);
        ^
    constructor Event.Event(int,long) is not applicable
      (actual and formal argument lists differ in length)
    constructor Event.Event() is not applicable
      (actual and formal argument lists differ in length)
Note: /Users/gsiry/Documents/kako/node_modules/react-native-maps/android/src/main/java/com/airbnb/android/react/maps/AirMapView.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors
:react-native-maps:compileReleaseJavaWithJavac FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-maps:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 7 mins 44.56 secs
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/android-setup.html

Can anyone help ?

It looks like I was using an older version of react-native - after upgrading to the latest version all is good - thanks - closing


The link to show hidden files is broken

It seems that the link to the hidden files article in the miscelaneous section is no longer working

The link to show hidden files is broken. We are working on this issue, but we are unable to provide a specific date at this time. Please check back in a few days. Arcsight Training


Please increase Fancy Zone max cell count from 40 to 42

📝 42 Fancy Zone Max instead of 40
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
You would have a 42 zone max in the fancy grid instead of 40. It'll be used when someone wants to have an even number of cells in their grid. See screenshot and notice the cell on bottom right is wider than the rest. Now all my windows need to be 1 cell short of match that width if I want to use the zone snapping.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

@enricogior what drove the max number?  any reason to not increase it to like 64?

Not so simple, there are several many places where we have to make changes.
Closing as duplicate of https://github.com/microsoft/PowerToys/issues/7051


Character filter for MarkupBuilder

Could this work?

Also: Not sure what I should do about a timeout in the sonar checks. Does that have something to do with me?

Parts of this look good but we can possibly re-work to increase the likelihood of being able to extend to StreamingMarkupBuilder. I'll take a look.

Like so?

It's been a while since I delved into XML encoding rules. We take a slightly different approach for surrogates (potentially writing them as hex strings) in StreamingMarkupWriter:
https://github.com/apache/groovy/blob/master/subprojects/groovy-xml/src/main/java/groovy/xml/streamingmarkupsupport/StreamingMarkupWriter.java#L141-L170
Do you see anything wrong with that approach?

Merged with a few changes. Thanks!

Great! \o/
@Surrogates: No, as a default for writing HTML that's probably even desirable.
They just live in the intenional limbo of no MUST in XML, but not discouraged either.
Ha, and you caught me. I copy-pasted a bit too much on my explanatory comments.
Let met fix that real quick...

Over here: https://github.com/apache/groovy/pull/1372
Sorry for the extra round-trip.


🐛 [Bug] Torch-TensoRT does not write values to a Tensor when using non-consecutive indices

Bug Description
If a PyTorch model uses non-consecutive indices to write values to a Tensor like this:
t[first_idxs, :, second_idxs] = feats

then the Torch-TensorRT model will not write the values.
Reshaping the Tensor such that it uses consecutive indices, e.g.:
t[first_idxs, second_idxs] = feats

makes the Torch-TensorRT model work correctly.
The outputs of a PyTorch model that uses non-consecutive indices will wildly differ from its associated Torch-TensorRT model, but because no errors/warnings are raised when compiling, it's difficult to debug. As far as I could tell, this behavior isn't mentioned in the Torch-TensorRT documentation; however, when googling around about this issue I did find that it's covered in the PyTorch ONNX documentation:
# Multiple tensor indices that are not consecutive
data[torch.tensor([2, 3]), :, torch.tensor([1, 2])] = new_data
# Workarounds: transpose `data` such that tensor indices are consecutive.

To Reproduce
import torch
import torch_tensorrt

from torch import nn

class Indexer(nn.Module):
    def __init__(self, use_consec_idx, shape):
        super().__init__()
        self.use_consec_idx = use_consec_idx
        self.shape = shape

    def forward(self, feats, first_idxs, second_idxs):
        (N, C) = feats.shape
        t = torch.zeros(N, *self.shape).to(feats)
        if self.use_consec_idx:
            t[first_idxs, second_idxs] = feats
        else:
            t[first_idxs, :, second_idxs] = feats

        return t

torch_tensorrt.logging.set_reportable_log_level(torch_tensorrt.logging.Level.Error)

device = torch.device("cuda:0")
(N, C, L) = (10, 20, 5)
feats = torch.rand(N, C).to(device)
first_idxs = torch.randint(N, size=(N,)).to(device)
second_idxs = torch.randint(L, size=(N,)).to(device)
for use_consec_idx in [False, True]:
    if use_consec_idx:
        shape = (L, C)
    else:
        shape = (C, L)

    model = Indexer(use_consec_idx, shape).to(device)
    inputs = [
        torch_tensorrt.Input(feats.shape),
        torch_tensorrt.Input(first_idxs.shape, dtype=torch.int32),
        torch_tensorrt.Input(second_idxs.shape, dtype=torch.int32),
    ]
    trt_model = torch_tensorrt.compile(model, inputs=inputs)
    with torch.no_grad():
        pt_out = model(feats, first_idxs, second_idxs)
        trt_out = trt_model(feats, first_idxs.int(), second_idxs.int())

        print(f"use_consec_idx: {use_consec_idx}")
        print(f"(pt_out == trt_out).all(): {(pt_out == trt_out).all()}\n")

Expected behavior
Either work the same as non-consecutive indexing does in PyTorch or raise a warning or error if non-consecutive indexing is detected, and instruct the user to use consecutive indexing.
Environment

Build information about Torch-TensorRT can be found by turning on debug messages

Torch-TensorRT Version (e.g. 1.0.0): 1.4.0
PyTorch Version (e.g. 1.0): 2.0.1+cu117
CPU Architecture: i7-12800H
OS (e.g., Linux): Linux
How you installed PyTorch (conda, pip, libtorch, source): pip
Build command you used (if compiling from source):
Are you using local sources or building from archives:
Python version: 3.10.10
CUDA version: 11.7
GPU models and configuration: GeForce RTX 3080 Ti
Any other relevant information:

Additional context

I no longer experience this bug when using PyTorch 2.0.1, Torch-TensorRT 1.4.0, and CUDA 12.2, and compiling the model with:
trt_model = torch.compile(model, backend="torch_tensorrt")


How do I add a Notes tab, similar to Posts

I need three categories on tab, including notes, posts, and talks.
How do I achieve this, can anyone give me a list of code changes?

I've listed my notes here. You can let me know if it does not work for you.
https://github.com/satnaing/astro-paper/discussions/311

Updated!!!
You can check this discussion answered by @maheshrijal


Is realsense-camera-formats.patch compatible with SR300?

Required Info

Camera Model
SR300

Firmware Version
tested both <IP_ADDRESS> and <IP_ADDRESS>

Operating System & Version
Ubuntu 16.04

Kernel Version (Linux Only)
4.4.15-tegra

Build System
Makefile

Is realsense-camera-formats.patch compatible with SR300? This patch applies successfully for us on 4.4.15-tegra kernel but the depth image is corrupted: jumping, and cut in the middle (see the screenshot).
It seems that some formats from 0002-LR200-ZR300-and-SR300-Pixel-Formats.patch are different in realsense-camera-formats.patch. Please help to adapt realsense-camera-formats.patch to make it compatible with SR300 and 4.4.15-tegra kernel because this is the only patch that can be successfully applied on 4.4.15-tegra.

Hi, the patch is applied correctly, otherwise you wouldn't be able to see IR and Depth at all.
It seem that the frame is corrupted during transmit . You can see that the first several lines (in IR) image are not synced at all, the sync becomes stable, but occurs in the middle of the frame.
Is this a commercial SR300 or an engineering sample ? Did you try to use a different SR300 ?

Thank you for your answer. We have tried two different SR300 (an older version and a just-bought version) and both give the same kind of corrupted depth image. However, both cameras work well on a PC (Ubuntu 14 and 16) so the problem occurs only for Jetson TX2. What can be a problem?

This type of problems also occur frequently when working inside VM, or when CPU usage is high. uvcvideo kernel module is not keeping up and some of the URB packets are getting lost, resulting in a corrupt image.
There is a way to increase the number of URBs in the kernel module, but this is not a patch we can recommend (on some systems it had unexpected side-effects, and this change can never be upstreamed). This change, however, can help with the issue, at expense of slightly higher latency and higher memory consumption.

Thank you ! That makes sense.
Why does R200 and ZR300 work well on the same system ? with the same patch, same driver and the same USB port?

@nlyubova, unfortunately I don't have an answer for that. Jetson TX2 is not a platform we have ever tested and we don't have access to it for now. SR00 does not share the same firmware and USB micro-controller as R200 & ZR300. What we would usually do is look into the USB transactions using USB analyzer, such as Lecroy. If you could send Wireshark USB traces, this could possibly also reveal useful information.


Check that convolve doesn't go back in time

add a test

fixed by 76ae7c1723c8f2588d9a45f3bf8ecb2fb48ad075. thanks @ejolly!


FALSE negative: www.google.de

IMHO it should be an url ;)

www.google.com is also not a URL.
From the source:
/**
 * RegExps.
 * A URL must match #1 and then at least one of #2/#3.
 * Use two levels of REs to avoid REDOS.
 */

var protocolAndDomainRE = /^(?:\w+:)?\/\/(\S+)$/;


Support movableContentOf across protocol

Currently, due to #1899, we crash when the detached subtree is re-added because children IDs clash. Even once that's fixed, we need to detect the fact that it's a reused node (probably with something like a removed=true boolean on the node) and upon re-insertion, walk its children and add them all to the map.
Once guest-side is fixed, we basically have the same problem host-side. It has the map leak that will cause the crash (#1900), but once that's fixed we need the same tree walk on re-attach.
It is unlikely that we will retain the host-side nodes across recompositions. If you are just moving the content in a single composition, however, we can send a new boolean in the remove op to signal that this is a detach with a future op performing re-attach via insertion. This will keep all nodes in the map, and only perform a re-parenting from the old children to the new children.

Alright here's some observations:

Calls to sendFrame can trigger multiple recomposition passes, with movableContentOf being one of the causes of this.
We should not rely on onEndChanges on the applier, and instead check for changes after the initial composition and after each frame.
The testing setup is not prepared to handle this and needs changed, probably first.

Working on it!

Plan:

Change the protocol to eliminate range removal. All removes will be a single element.
Have protocol remove return an opaque token (read: change list index) which can be held by the widget. If the widget comes back through reuse in the same recomposition, use the token/index to update the original remove with detach=true.
Defer removing widgets from the map until the change list is realized, probably through a list that keeps track of removed widgets. We remove a widget from the removed widget list when it's reused. This is a rare event, so a quick list scan should be fine.


Don't print Validation Warning for "//" comments in package.json

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
package.json doesn't support comments. The official, npm-sanctioned way to add comments is to use "//" key:
{ "//": "this is the first line of a comment",
  "//": "this is the second line of the comment" }

Running https://repl.it/repls/ThornyHatefulVirus prints a Validation Warning:
● Validation Warning:

  Unknown option "//" with value "comment" was found.
  This is probably a typing mistake. Fixing it will remove this message.

  Configuration Documentation:
  https://facebook.github.io/jest/docs/configuration.html

What is the expected behavior?
No warning should be printed for "//" keys.

I don't know, is this a widely used practice? Honestly I see this for the first time, but maybe it's just me. We would probably need to add more if these special characters to the whitelist, which I'm not happy with.

Is this actually documented somewhere outside of node mailing list? I'd be happy to accept a PR if this really exists

This does not appear to be documented at docs.npmjs.com, but the mailing list looks pretty official to me. It was written by the creator (and current CEO) of npm.
Unfortunately github doesn't allow searching on special characters, so I can't see how common this is.
FYI the SO post is the first Google hit for comments package.json

I remember seeing this as the response on the npm issue tracker, I think I actually submitted a PR at one point to npm supporting comments. Lemme see if I can dig it up

Yeah, see https://github.com/npm/read-package-json/pull/46 and the linked https://github.com/npm/npm/issues/4482

I've seen this used before and since node people are directing folks to do it I'm also +1 for supporting 👍It's not like we're ever going to use "//" as a config for anything
Labeling good first issue! @melissachang any interesting in taking it?

Sorry, not at the moment :/

No sweat!

@rickhanlonii First-time contributor here. I’d like to take a stab at implementing this feature. Can you point me at the validation logic for Jest configuration in package.json?

That's awesome @andrew-pyle!
It's here: https://github.com/facebook/jest/blob/c22d9f584627a1ae6c476d44f10c3c186b480722/packages/jest-validate/src/validate.js#L59-L63
What I would do (I think, haven't tested it) is add // to the default blacklist. Feel free to ask questions if you're stuck 🙂

Pull request #7295 sent to implement @SimenB's suggestion of adding "//" to the default blacklist

Fixed in #7295


No gradient defined for Relu6Grad

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Yes. MWE below is not a stock example.

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 16.04

TensorFlow installed from (source or binary):
Binary (Miniconda)

TensorFlow version (use command below):
print(tf.GIT_VERSION, tf.VERSION)
('v1.2.0-5-g435cdfc', '1.2.1')

Python version:
2.7.13

Bazel version (if compiling from source):
N/A

CUDA/cuDNN version:
CUDA 8.0.61
CuDNN 5.1.10

GPU model and memory:
GeForce GTX TITAN X
Total Memory 11.91GiB

Exact command to reproduce:
See below.

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
One can not apply the gradient operator to a nn.Relu6 more than once.
Applying it once gives a Relu6Grad node,
applying it a second time (i.e applying it to the Relu6Grad)  throws an error.
Unless I have messed up my math (quiet possible),
Relu6 is infinitely differentiable, except at 2 points,
albeit very boring

f(x) = relu6(x)
df/dx = 1 if 0<x<6 else 0
d2f/dxx = 0
all further derivatives also 0.

How-ever, it is marginally more interesting if one is applying the chain rule to it

f(x1,x2) = relu6(x1*x2)
df/dx1 = x2 if 0<x1*x2<6 else 0
d2f/dx1x2 = 1 if 0<x1*x2<6 else 0

This is a feature request to make the gradient of Relu6Grad defined.
Source code / logs
MWE:

import tensorflow as tf
sess = tf.Session()

x1 = tf.placeholder(tf.float32)
x2 = tf.placeholder(tf.float32)

y = tf.nn.relu6(x1*x2)
d1 = tf.gradients(y,x1)
d2 = tf.gradients(d1, x2)

Errors on the final line  with
---------------------------------------------------------------------------
LookupError                               Traceback (most recent call last)
<ipython-input-7-4250ab355a8b> in <module>()
----> 1 d2 = tf.gradients(d1, x2)

...usr/lib/python2.7/site-packages/te
nsorflow/python/ops/gradients_impl.pyc in gradients(ys, xs, grad_ys, name, colocate_gradients_with_ops,
 gate_gradients, aggregation_method)
    512               raise LookupError(
    513                   "No gradient defined for operation '%s' (op type: %s)" %
--> 514                   (op.name, op.type))
    515         if loop_state:
    516           loop_state.EnterGradWhileContext(op, before=False)

LookupError: No gradient defined for operation 'gradients/Relu6_grad/Relu6Grad' (op type: Relu6Grad)

Yes, this feature is not implemented, but it is relatively easy to add. Would you be willing to contribute a pull request. In particular look at
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/nn_grad.py#L388
(for ReluGradGrad). You can follow suite for Relu6.

HI, @oxinabox. I believe your mathematical derivation is pretty correct, and implement it in PR #13268 . Please check whether it is right as you proposed if possible.

@facaiy I'm not sure if it is correct, as I don't know much of the internals of the python client.


Folder context not working

Hi,
I have the following in the list of context providers:
  "contextProviders": [
    {
      "name": "codebase",
      "params": {
        "nRetrieve": 25,
        "nFinal": 5,
        "useReranking": true
      }
    },
    { "name": "search" }
  ]

However, when I type @folder in the search box, nothing appears.
Please let me know what I may be missing.
Thanks!

@abhinavkulkarni you'll have to add the context provider to your list, like [..., {"name": "folder"}]. Going to close this issue, but you can rest assured we're going to make this more obvious (will be adding an option in the @ dropdown to "add more context providers"). and as always open to other ideas if you have them : )


rename TestError to ExampleError to avoid PytestCollectionWarning

When running tests, this warning is raised:
test/test_exc.py:4
  /Users/dani/bottle/test/test_exc.py:4: PytestCollectionWarning: cannot collect test class 'TestError' because it has a __init__ constructor (from: test/test_exc.py)
    class TestError(Exception):

this is because pytest is trying to collect all classes starting with the word Test. Since this class isn't an actual test, renaming it to something else is appropriate.
Another solution would've been to add   __test__ = False to the class, but renaming seemed straight-forward.
It's always nice to run tests and not see warnings, so I may fix some more in upcoming PRs :)

This should be solved by now. Thanks :)


feature request: add forRoot() for module to config datepicker format globally

I think it's good to have forRoot() in module to pass some configs like date format and ... .
is it possible?

Hi,
Could you please explain more what you expect forRoot() do for you?
Do you expect forRoot() changes default config for all Date Pickers or override them?

@fingerpich hi, for example i want to pass this config object in forRoot():
  datePickerConfig = {format: 'jYYYY/jMM/jDD'};
to take effect on all data pickers in modules.

Do you expect forRoot() changes default config for all Date Pickers or override them?

@fingerpich  Yes, exactly. Currently  we must define config in every component or pass it inline.

I said there is two way
1- forRoot() changes default config for every calendar
2- forRoot() override used config for every calendar
which one?

What is diff btw 1,2?

1 - A datepicker has a default config but if you changed the config for a datepicker forRoot() wouldn't change your config
2 - forRoot() wouldn't care about your change and all of them will change.

Any library have this feature , developer can change default configuration when import that by forRoot  , for example page size in pagination can change using forRoot and ,but developer can change page size for each component when using that ,

If you already know it please make a fork and send pr

Ok i'll try to do this.