test: loosen the test expectation

The test expects the output location to be temp directory layout which
may not be the case.  Loosen the test to enable moving the temporary
directory.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 5b6ef8649c12e8327b111f51d0f750637018ee55

/usr/local/bin/sccache /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/buildbot_incremental/llvm-macosx-x86_64/./bin/clang++  -DCMARK_STATIC_DEFINE -DGTEST_HAS_RTTI=0 -DLLVM_DISABLE_ABI_BREAKING_CHECKS_ENFORCING=1 -D_DEBUG -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Istdlib/toolchain/Compatibility50 -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/stdlib/toolchain/Compatibility50 -Iinclude -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/include -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/llvm-project/llvm/include -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/buildbot_incremental/llvm-macosx-x86_64/include -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/llvm-project/clang/include -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/buildbot_incremental/llvm-macosx-x86_64/tools/clang/include -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/cmark/src -I/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/buildbot_incremental/cmark-macosx-x86_64/src -Wno-unknown-warning-option -Werror=unguarded-availability-new -fno-stack-protector -fPIC -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -Wimplicit-fallthrough -Wcovered-switch-default -Wno-class-memaccess -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wstring-conversion -fdiagnostics-color -Werror=switch -Wdocumentation -Wimplicit-fallthrough -Wunreachable-code -Woverloaded-virtual -DOBJC_OLD_DISPATCH_PROTOTYPES=0 -O3  -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk   -UNDEBUG  -fno-exceptions -fno-rtti -Werror=gnu -fno-sanitize=all -Wall -Wglobal-constructors -Wexit-time-destructors -fvisibility=hidden -DSWIFT_COMPATIBILITY_LIBRARY=1 -DSWIFT_TARGET_LIBRARY_NAME=swiftCompatibility50 -target armv7k-apple-watchos2.0 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS6.2.sdk -arch armv7k -F/Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS6.2.sdk/../../../Developer/Library/Frameworks -mwatchos-version-min=2.0 -O2 -g0 -UNDEBUG -DSWIFT_ENABLE_RUNTIME_FUNCTION_COUNTERS -std=c++14 -MD -MT stdlib/toolchain/Compatibility50/CMakeFiles/swiftCompatibility50-watchos-armv7k.dir/ProtocolConformance.cpp.o -MF stdlib/toolchain/Compatibility50/CMakeFiles/swiftCompatibility50-watchos-armv7k.dir/ProtocolConformance.cpp.o.d -o stdlib/toolchain/Compatibility50/CMakeFiles/swiftCompatibility50-watchos-armv7k.dir/ProtocolConformance.cpp.o -c /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/stdlib/toolchain/Compatibility50/ProtocolConformance.cpp
In file included from /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/stdlib/toolchain/Compatibility50/ProtocolConformance.cpp:20:
In file included from /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/stdlib/toolchain/Compatibility50/Overrides.h:18:
In file included from /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/include/swift/Runtime/Metadata.h:20:
In file included from /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/include/swift/ABI/Metadata.h:44:
In file included from /Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS6.2.sdk/usr/include/objc/runtime.h:27:
In file included from /Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS6.2.sdk/usr/include/objc/objc.h:33:
/Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS6.2.sdk/usr/include/objc/objc-api.h:29:10: fatal error: cannot open file '/Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS6.2.sdk/usr/include/AvailabilityMacros.h': Operation not permitted
#include <AvailabilityMacros.h>
         ^
1 error generated.

CC: @shahmishal

@swift-ci please test macOS platform


Remove double slashes in Strip control codes challenge

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #38436

@RandellDawson Yes, thank you! 😊

@agatapst Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Symfony CS preset

@joelwurtz IIRC you wanted to add some exclusions.

the only one i remember is short array syntax. we should try adding it to a code repo and see what it would change to be sure.

Well, I already added this preset to the message factory, where I use short syntax, but nothing changed, but I think we should explicitly add it, because there is a long_array fixer as well, which suggests that by default no fixing is done.
@joelwurtz there was something about whitespaces, no?

+1 for this will propose other rules in another pr


perf: use @ng-icons instead of Font Awesome icons

Font Awesome requires a quirk and is using around 60KBs of bundle size (see issue). That's half of what the app takes just for 4 icons. The reason is that there are many features in there: animations, masks, ... but don't need those and aren't tree shaken
Using ng-icons instead. Allows more than Font Awesome and doesn't include that bloat.

#432  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


Adds simple caching functionality

Resolving DID Documents can be expensive. It is in most cases best to cache DID documents. By default caching is now enabled in the resolver.
It depends on #9 to support no-cache param.
The built in cache uses a WeakMap, but does not have an automatic TTL, so entries don't expire. This is fine in most web, mobile and serverless contexts. If you run a long running process you may want to use an existing configurable caching system.
Here is an example using js-cache which has not been tested.
var cache = require('js-cache')
const customCache : DIDCache = (parsed, resolve) => {
  // DID spec requires to not cache if no-cache param is set
  if (parsed.params && parsed.params['no-cache'] === 'true') return await resolve()
  const cached = cache.get(parsed.didUrl)
  if (cached !== undefined) return cached
  const doc = await resolve()
  cache.set(parsed, doc, 60000)
  return doc
}

const resolver = new DIDResolver({
  ethr,
  web
}, customCache)

The Cache can also be disabled by passing in a false value to the constructor:
const resolver = new DIDResolver({
  ethr,
  web
}, false)

I agree with @mirceanis, this feature should be "opt-in".

OK I will make it optional

Agreed. Opt-in

@mirceanis please re-review


"Expensify chat" is shown in contact list for "start chat"

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

Version Number: 9.0.3-6
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @shawnborton
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1719906408139169
Action Performed:
Scenario1:

Go to staging.new.expensify.com and login
Click FAB > Start chat
Observe the contact list
Scenario 2:
Create a group with "concierge" and "Expensify"

Expected Result:

"Expensify" should not display in the contact list as can't initiate a chat
Shouldn't be able to create a group for the above reason

Actual Result:

"Expensify" appears in the contact list
In gmail domain able to initiate the chat
Able to create a group

Workaround:
unknown
Platforms:

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

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

Screenshots/Videos

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
"Expensify chat" is shown in contact list for "start chat"
What is the root cause of that problem?
We use the isGroupChat here as a prop, but we don't ever set it.
https://github.com/Expensify/App/blob/fc045a8cf01a74e2879ed911c64e5ece2dca9ecf/src/pages/NewChatSelectorPage.tsx#L34
This needs to be variable on the page, and may have been missed over the refactors.
What changes do you think we should make in order to solve the problem?
We should have isGroupChat as a variable in NewChatPage calculated as:
const isGroupChat = selectedOptions.length > 0;

This will be used here and here to carry out the group chat logic.
This will mean that we will remove this from prop definition here and here

Proposal
Please re-state the problem that we are trying to solve in this issue.
Expensify notification chat is able to be added to a group.
What is the root cause of that problem?
In https://github.com/Expensify/App/pull/41290, we allow the Expensify notification chat to be listed in the list (specifically this change).
But we forgot to cover the case where the user is able to add the Expensify notification to a group.
The isGroupChat prop is a very old prop added first in https://github.com/Expensify/App/pull/5138. At that time, we had a new chat and a new group page separated. New group page uses new chat page and pass down the isGroupChat.
What changes do you think we should make in order to solve the problem?

To keep both behavior of https://github.com/Expensify/App/pull/41290 and https://github.com/Expensify/App/pull/5138, we can hide the "Add to group" button when it's the excluded account.
https://github.com/Expensify/App/blob/5ab6be6ecb01d6398eb2503dbd3f6dad60d6638f/src/pages/NewChatPage.tsx#L210-L214
if (item.isSelfDM || excludedGroupEmails.includes(item.login ?? '')) {
    return null;
}

Then, consider removing the unused isGroupChat prop.

I actually think I'm just going to close this, as we did decide that Expensify should still remain in your contacts list.
And I can't reproduce being able to chat with the Expensify contact using a Gmail account login.

@shawnborton I think we can take this chance to not allow Expensify to be added to a group, so we can prevent this issue too.

Well I'm not actually sure if we want to prevent that or not - it seems like we want to just treat this contact like a normal contact? cc @danielrvidal in case you have thoughts there.


Fix compilation error on UT and resolve error for scalajs-angular_sjs0.6_2.11

Story
Parent Issue
Travis build is failing, fix errors and add to PR

merge into parent issue #952


DllNotFoundException: Unable to load DLL 'e_sqlite3' in "Prism modular" application

Hi I am trying to write a modular application that uses a sqlite db but I am getting an error when trying to access the db as it seems the project is not directly linked to the main exe project.
If I add a reference to sqlite in my main project it works, or if I add a reference to the project it works but this is breaking the modularity of the application as I want it to be free from knowing the database layer.
I am using .net 5.0. Here is my sample app that duplicates the problem: ModularAppSQLiteProblem.zip
Error:
`System.TypeInitializationException
HResult=0x80131534
Message=The type initializer for 'Microsoft.Data.Sqlite.SqliteConnection' threw an exception.
Source=Microsoft.Data.Sqlite
StackTrace:
at Microsoft.Data.Sqlite.SqliteConnection..ctor(String connectionString)
at Microsoft.EntityFrameworkCore.Sqlite.Storage.Internal.SqliteRelationalConnection.CreateDbConnection()
at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.get_DbConnection()
at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.Open(Boolean errorsExpected)
at Microsoft.EntityFrameworkCore.Sqlite.Storage.Internal.SqliteDatabaseCreator.Exists()
at Microsoft.EntityFrameworkCore.Storage.RelationalDatabaseCreator.EnsureDeleted()
at Microsoft.EntityFrameworkCore.Infrastructure.DatabaseFacade.EnsureDeleted()
at ModularAppSQLiteProblem.Modules.ModuleName.Repository.Initialise() in C:\Users\james.baker\source\repos\ModularAppSQLiteProblem\ModularAppSQLiteProblem\Modules\ModularAppSQLiteProblem.Modules.ModuleName\Repository.cs:line 21
at ModularAppSQLiteProblem.Modules.ViewModels.ModuleName.DataBaseViewModel..ctor(IRepository repository) in C:\Users\james.baker\source\repos\ModularAppSQLiteProblem\ModularAppSQLiteProblem\Modules\ModularAppSQLiteProblem.Modules.ModuleName\ViewModels\DataBaseViewModel.cs:line 18
Inner Exception 1:
TargetInvocationException: Exception has been thrown by the target of an invocation.
Inner Exception 2:
DllNotFoundException: Unable to load DLL 'e_sqlite3' or one of its dependencies: The specified module could not be found. (0x8007007E)
`

Possibly related to #252

Probably @bricelam, shame nothing has happened in 2 and a half years though 😬

I tried running the repro sample and it just brings up an empty window, no exception thrown.  How do I get the error you posted?  I'm not familiar with Prism.

@ericsink sorry, you might have to make sure you rebuild the solution, because the module project is so loosely coupled I don't think it will have built and so it will not be trying to access sqlite.
It should then error.

@ericsink updated original sample with correct project dependencies so it should build correctly now.

Still nothing.
I unzip the repro file.  I tried building and running from Visual Studio and from the command line.  In both cases, the main window comes up empty, no errors.
You're saying that if you unzip your repro zip file to a clean directory and then build and run it, you get the exception?  If so, please explain the exact steps.

Hmmmm @ericsink yeah I've just tried it. Have you re-downloaded from my original post?
If you put a breakpoint in ModuleNameModule.OnInitialized()? Does it hit. As long as that gets hit it should error. If it isn't being hit, then possibly try a rebuild on the solution to make sure everything builds. I am doing this from VS.

Yes, I re-downloaded.  And verified by running diff on the two directories.
Yes, I tried a rebuild.
I just added the breakpoint and it does hit.  So I looked again for the exception and eventually found it.
I'll attempt a fix and see what happens.

Update:
All I'm trying to do here is setup a simple dev loop where I (1) repro the problem, (2) make a change to use buildTransitive (aka #252), and (3) try again to see if the problem can still repro.
But I have not yet been to repro this with a tree I can currently develop in, for various reasons.
The repro app from OP uses net5.0 with M.D.Sqlite (and friends) at 5.0.9.  If I just add  PackageReference for my 2.0.5-pre development versions, I get MissingMethodExceptions, as 5.0.9 is referencing things I have deleted.  This raises questions about whether 2.0.5 is going to be compatible with EFCore 5.x stuff, which is a separate issue.
I restored those missing methods in my dev tree, but I can't quite get the build to stop trying to include 2.0.4 stuff.  Not sure why.  It's complaining that the provider (with a 2.0.4.x version number) has no implemented of the soft_heap_limit function which was added after 2.0.4.  So it's clearly got some 2.0.5 and 2.0.4 stuff mixed up.  PackageReference with 2.0.4 and ExcludeAssets="all" didn't help.
So instead, I tried updating the repro app to 6.0, because I know 2.0.5 development tree works with EFCore 6.0-pre stuff.  Now the repro app won't work with Visual Studio 2019, which means I would need to install VS 2022 preview.
So instead I tried just using the command line, but the repro app is swallowing the exception, so I can't tell if the problem is happening or not.
So instead I tried using the repro app unmodified from the zip file, do a build, and then hackishly use buildTransitive by modifying the layout of the files in the nuget package cache for 2.0.4.  At this point I realized that the SQLitePCLRaw.lib.e_sqlite3 nuget package is only using the build folder for net461 for the props and targets files, and the actual e_sqlite3 binary.  So the buildTransitive folder should make no difference to a net5.0 app anyway.  Which would mean that maybe #252 isn't relevant here.
But oh, wait, 2.0.4 didn't actually target net5.0, and the warning message says the packages are getting restored using net461, which means the buildTransitive folder would be in play after all.  But 2.0.5 shouldn't have that problem, since it multitargets net5.0.
So I did the package cache hack anyway and renamed the build folder to buildTransitive, which didn't make a difference in the repro app.  Not sure why, but the failure is happening in the dynamic provider, which is another thing that wouldn't be used with 2.0.5 in this case.
So I cleared the package cache (again), and did a git checkout of my dev trees back in time to when 2.0.4 was released, made a change to use buildTransitive, and rebuilt.  The repro app here still has the same problem.  The only difference now is that my test suite fails.
Summary (for now):
I don't know for sure if buildTransitive #252 is relevant to this situation or not.
Because 2.0.5 multitargets net5.0, the problem here may be getting fixed another way, but I don't know for sure.
Issue #252 should have been fixed a long time ago, but given current conditions, it may be less important than ever.
I don't know if 2.0.5 is going to be compatible with efcore 5.x.
This test case might be unusual.  I wish the repro didn't involve GUI code, because a console app tree would be easier to test.  But the specific details of what "modular" means here w.r.t. Prism might in fact be the very thing that makes this case tricky.  The app project in the repro sample appears to have no reference to the module project.

This raises questions about whether 2.0.5 is going to be compatible with EFCore 5.x stuff

Keep in mind that EF Core 5.0 is a current (non-LTS) release and will go out of support three (maybe six) months after EF Core 6.0 is released in November. IMHO, it's probably fine if it doesn't work because anyone on 5.0 who wants to upgrade should also be upgrading to 6.0 at the same time.

I'm trying to take a fresh look at this old issue.
First of all, using the most recent repro sample project, I can reproduce the problem.
I'm pretty sure that #252 and the buildTransitive notion was a red herring.  NuGet targets files are not involved here, as this is simply .NET 6.
Issues of EF Core 5 and my version 2.0.5 etc are also no longer in play.
One of my struggles in the past was that this problem is Prism-centric, and is using a definition of "modular" which is specific to Prism.
A light bulb went on for me just now when I was looking through the repro sample.
I do see the 3 main projects:

The app
The Services/IRepository assembly
The module

The module project is not actually referenced (in the sense of an msbuild project reference) from either of the other two, which I understand to be the main point of this architecture, and tied to the notion of "modular" in this context.
But what kept confusing me is how the build products of the module project were ending up in the output directory of the app project without such a reference.
Then I found that the csproj for the module project is modifying its output path manually.  It literally does stuff like this:
    <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">
        <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
        <OutputPath>$(SolutionDir)\$(SolutionName)\$(SolutionName)\bin\x64\Debug</OutputPath>
    </PropertyGroup>

This seems very fragile.  For example, I suspect it won't work with dotnet publish.
And I also suspect this is the reason why the DllImport stuff can't find the native library.  In theory, it should be possible to configure a special resolver to allow the native library to be found.
Bottom line:  Without special effort, I don't think this approach is likely to "just work" with SQLitePCLRaw, because MSBuild project references are the way that .NET configures the ability for DllImport to find things.
OTOH, if this Prism/modules approach is working well with other cases which involve the use of a bundled native code library, I would be interested in how they're doing it.

Thank you for looking into this @ericsink. We have a similar issue with System.DirectoryServices.AccountManagement and yes are aware of the donet Publish issues.
We are planning to revisit this and see if we can come up with a better solution. Any recommendations to gain a similar outcome that works would be appreciated. Plan to start looking at this https://learn.microsoft.com/en-us/dotnet/core/tutorials/creating-app-with-plugin-support

I don't think I have any useful suggestions on how to do plugins.  There are lots of approaches.  Interop with a native code library will be tricky in most cases, but it can be done.
I'll go ahead and close this issue.  Best wishes.


Improve UX for adding ingredients to a recipe

Right now there is an add button at the end of each add ingredient row that needs to be hit to add a new ingredient. It's confusing and needs to be changed.

the "environment variables" editor in the beanstalk config might provide some inspiration.

This is being replaced with "reconciliation" for ingredient matching and has been removed from the add process, so closing.


Verify that git lfs is installed when building

Adds a check to the parent SConscript that ensures that at least one tar.gz file in the repo is valid.  This implies that git lfs is installed and is set up correctly.
Fixes #68

Worked like a charm once I figured out how to uninstall git-lfs :)


TranslationRecognizer can't download crl.

1、install apk such as the android smaples
2、invoke recognizer.startContinuousRecognitionAsync();
3、speak something but have no result
4、use wireshark grab bags,find that  tcp packet had lost. Sometimes the http for crl request even have not be called.
5、uninstall apk
6、repeat 1-5
We test this in China network. It offten happens. When test it in USA,it become normally.
We make sure that the network works well in China because we invoke http request "http://mscrl.microsoft.com/pki/mscorp/crl/Microsoft IT TLS CA 4.crl" to download this crl nomorlly.

Please try with 1.11.0 version of Speech sdk and let us know if Is it still reproable ? If not could you please close the issue ?

Closed since no update, please reopen if necessary.


Customize your Fruition site with over 65 CSS snippets and multiple design themes - with Fruition Plus.

Fruition Plus is a collection of crowdsourced design templates, custom CSS style snippets, & more for  Fruition websites.
It took quite some while to figure out all the different css styles and I wanted to give back to the Fruition community that allowed me to launch 6 websites for free!
Check it out here, any feedback appreciated.
https://fruitionplus.com/

Hey @Simonheu
Very interesting, good change I will try it for any inconvenience to share solutions, thanks and greetings.


Support for non-UTF encoding in xml parser

Very excited about the enhanced XML support in 1.4.0 :) I've been experimenting with it in https://github.com/http4s/http4s-scala-xml/pull/25 and running into trouble with non UTF encodings. FTR I'm no expert in these things :)
For example this request:
Content-Type: application/xml

<?xml version="1.0" encoding="iso-8859-1"?><hello name="Günther"/>

as used in this test:
https://github.com/http4s/http4s-scala-xml/blob/1ca64f2ab7ef500d384d2ec5f8caf88df600e6a6/scala-xml/src/test/scala/org/http4s/scalaxml/ScalaXmlSuite.scala#L198-L209
Furthermore the RFC specifies:
Since the charset parameter is not provided in the Content-Type
header and there is no overriding BOM, conformant XML processors must
treat the "iso-8859-1" encoding as authoritative.  Conformant XML-
unaware MIME processors should make no assumptions about the
character encoding of the XML MIME entity.

https://datatracker.ietf.org/doc/html/rfc7303#section-8.3
I'm not sure if there is a way to support this without an XML parser that operates directly on bytes instead of chars/strings 😕 any thoughts? Thanks!

Thanks for reporting. My parser indeed works on decoded characters, so you need to decode it in the right way before going through the XML event pipe. This can be done with fs2-data by using the appropriate import which brings the decoder in scope from Byte to something that can emit Chars.
The parser operates then on these bytes without changing the decoding if it does not match the one provided.
I remember thinking about implementing this behavior but I decided to stay with an implementation that is not aware of the encoding. It trusts blindly the strings that were decoded for it.
In the case of your issue, I am not entirely sure how the string is decoded from bytes, I should have a closer look at it.

Thanks for the response! I'm digging into this again since it's all rather confusing.
Currently looking at https://www.w3.org/TR/2008/REC-xml-20081126/#charencoding
Which says:

Each external parsed entity in an XML document may use a different encoding for its characters.

Which again suggests to me, that XML-parsing should operate directly on byte-streams, rather than decoded Strings.

Parsed entities are DTD related (you have internal and external ones). What this says is that every externally defined entity (i.e. in a DTD that is physically in another file) might use a different encoding.

Basically the approach taken in fs2-data is:

for textual format, you need to decode the bytes into characters first.
the textual parsers get a stream of characters that is assumed to be correct.

In your case, I would expect bodyText to decode text according to the declared charset (which was set to the source).
I am not familiar with http4s types anymore, I need to dig a bit more to see where the problem would lie. If the text is actually latin9 encoded, and the message declares this charset, I would expect the string coming out of bodyText to be properly decoded, but I might be wrong, I need to check.

In your case, I would expect bodyText to decode text according to the declared charset

Right. I think this situation is testing when there is no declared charset, except for the encoding specified in the XML itself.
Still, I think are you right that this might be a problem to solve in http4s. Further reading:

In the absence of information provided by an external transport protocol (e.g. HTTP or MIME), it is a fatal error for an entity including an encoding declaration to be presented to the XML processor in an encoding other than that named in the declaration, or for an entity which begins with neither a Byte Order Mark nor an encoding declaration to use an encoding other than UTF-8.

https://www.w3.org/TR/2008/REC-xml-20081126/#charencoding

In the test, can you try making an implicit ISO-8859-1 charset available in scope of the test and see if it solves it? bodyText takes an implicit charset, which is UTF-8 by default.

Right, so the test is currently declared like this:
  test("parse omitted charset and 8-Bit MIME Entity") {
    // https://datatracker.ietf.org/doc/html/rfc7303#section-8.3
    encodingTest(
      Chunk.array(
        """<?xml version="1.0" encoding="iso-8859-1"?><hello name="Günther"/>""".getBytes(
          StandardCharsets.ISO_8859_1
        )
      ),
      "application/xml",
      "Günther",
    )
  }

That fails.
However, if I specify the charset like this, then it passes:
diff --git a/scala-xml/src/test/scala/org/http4s/scalaxml/ScalaXmlSuite.scala b/scala-xml/src/test/scala/org/http4s/scalaxml/ScalaXmlSuite.scala
index 8739d94..cfd9622 100644
--- a/scala-xml/src/test/scala/org/http4s/scalaxml/ScalaXmlSuite.scala
+++ b/scala-xml/src/test/scala/org/http4s/scalaxml/ScalaXmlSuite.scala
@@ -203,7 +203,7 @@ class ScalaXmlSuite extends CatsEffectSuite with ScalaCheckSuite {
           StandardCharsets.ISO_8859_1
         )
       ),
-      "application/xml",
+      "application/xml; charset=iso-8859-1",
       "Günther",
     )
   }

But the whole point of that test is to pass without specifying the charset.
test("parse omitted charset and 8-Bit MIME Entity")

No I meant something adding implicit val charset = Charset.forName("iso.8859.15") or similar in the test itself. Since no encoding is provided in the answer, you need to specify how to decode the body.

Sorry, I guess I'm confused 😕 I understand the code changes you are suggesting (and I expect it will work), but I don't understand what it will demonstrate? Since in practice there would be no way to know what the implicit charset should be.

@rossabaker if you have a moment to weigh in here it would be appreciated 🙏

@satabin I solved the problem with a small hack in http4s-fs2-data-xml, see https://github.com/http4s/http4s-fs2-data/blob/main/scala-xml/src/main/scala/org/http4s/scalaxml/ElemInstances.scala#L53-L134.
Would you see this as something we could pull into fs2-data (partially, of course we don't have HTTP headers, only the XML prolog)? Maybe as some kind of configurable behaviour? I'm a bit on the fence…it brings us closer to the XML spec, but would be one of the least elegant parts of this project as it goes against our abstractions like CharLikeChunks. Or maybe we find a reasonable way of back-channeling charset info which we can implement in 2.0.
If we don't pull it in or explore alternatives, I would close this issue as http4s has a way forward.

I am not a fan of what I did with the CharLikeChunks, it is not flexible enough, and I was thinking about a better approach for 2.0, that would allow for this kind of behavior.
The current approach in fs2-data 1.x is to have characters decoded outside of fs2-data, which does not work well with this kind of of format.
I would rather not integrate it currently and think about a better approach for this problem and abstraction and integrate it in 2.0. WDYT?


Ability to remove autofill

Is your feature request related to a problem? Please describe.
Hi! I am trying to remove the autofill for web desktop versions.
Having AutofillHits.oneTimeCode prompts desktop web browsers to autofill user passwords in the pin code field.
When using Pinput() constructor, best I can do is provide empty array [] to autofillHints, since if I provide null the value is resolved to:
this.autofillHints = const [
AutofillHints.oneTimeCode,
],
When I provide empty array, there is still autofill configuration created for the input. For desktop web browsers it still hints user passwords for the pin field.
https://github.com/user-attachments/assets/3684ed64-d4c3-4e29-be84-318fa3618105
Only way to disable the autofill configuration is to provide null to this property, and only way to do that is to use Pinput.builder()
However, I want to use the widgets provided by you package, and Pinput.builder() forces me to provide a builder.
Describe the solution you'd like
I think the easiest solution would be to make builder in Pinput.builder() nullable. This way I can pass null to autofillHints to remove the autofill (since there isn't a default value in the Pinput.builder() constructor) and I could also pass null to builder so I can use the package provided widgets.
@Tkko

Hi @Yagorundy, if you provide null the autofillHints should be resolved as null instead of the default value.

null
array

@Tkko
All is good, providing null to autofillHints works. Thanks!


Add error message for user when ledger fails when attempting to sign

Description
When a user attempts to perform a deposit with a Ledger device, if Blind Signing and Debug Data is not enabled, the transaction will fail with seemingly no reason. This has been a problem for years and still people ask about it on the EthStaker discord. Though this issue may be resolved by updating the web3-react library from v6 to v8, a brief attempt at doing so resulted in dependency hell and would be quite a drastic change for this one small issue. I had a quick chat with @wackerow about this and we agreed an error message is the best approach.
The error message will appear if either Blind Signing or Debug Data are disabled or, sadly, if the user simply rejects from the Ledger device. We have no way to differentiate but I feel the benefit of the guidance outweighs the false positive.
I am a little uncertain on exact wording or if there is a stylistic standard around presenting error messages but happy to update as necessary.
Related Issues

#549

Friendly ping on this.
I am still assisting users with this issue and though this isn't the optimal solution, it will still help.

Updated version


IE11 - XMLHttpRequest hangs indefinitely

Hi there, I'd like to submit a bug regarding IE11 XHR support for FilePond. I've included a fix for the bug as well. (At least, the bug that appears on my system.)
I am using FilePond 3.0.2.
Summary
When FilePond prepares a POST request in IE11, the request hangs indefinitely.  It never fires a network request at all.
How to Reproduce
NB: I know the steps for repro are for React, but the wrapper uses the core FilePond module to work so I figured it'd be more relevant here.
Fresh install of create-react-app, filepond-react, @babel/polyfill and filepond-polyfill.
Tested with the following in src/App.js in the render method:
<FilePond
  allowMultiple
  maxFiles={20}
  server={My Super Secret API URI}
/>

Attempting to make a network request to the server URI will never fire. (Drag and drop or browse actions.)
I've tested this in other browsers (Chrome, Firefox, Edge) and they don't have this issue.
Expected Behaviour
Network requests should fire immediately and not hang indefinitely.
Additional Information
This bug can be fixed by swapping some lines under sendRequest around so that it reads:
    // add headers
    xhr.open(options.method, url, true);

    // set timeout if defined
    if (isInt(options.timeout)) {
      xhr.timeout = options.timeout;
    }

... instead of:
    // set timeout if defined
    if (isInt(options.timeout)) {
      xhr.timeout = options.timeout;
    }

    // add headers
    xhr.open(options.method, url, true);

You can find a reference for this fix here.
No new bugs introduced from my tests in Chrome 69.0.3497.100, Firefox 62.0.3 or Edge 41.16299.611.0.

Environment
Version

OS
Windows

Device
Dell Optiplex 9020

Browser
IE 11 Version 11.611.16299.0

Thanks for the high-detail bug report @glennpenridge , super helpfull for fixing the issue.

Thanks mate!


Use 'new' expression in places where the type is evident for content

Requires https://github.com/space-wizards/space-station-14/pull/2587
Separated into one commit per project to make any of it easier to revert individually.
Engine PR: https://github.com/space-wizards/RobustToolbox/pull/1415

After some other stuff gets merged and conflicts fixed I'll merge it coz this will conflict with E V E R Y T H I N G

Ripping off that bandaid.


I was missing the postPersistLogEntry()  hook

since createLogEntry() method is private, it was not possible to extend it and the only way I found was to create a postpersistLogEntry hook.
I tried using the doctrine.postPersist event, but it was too generic and I was finding myself in a PostPersist loop, so the best way was to create a postPersistLogEntry method.

Hi, make createLogEntry protected instead of private in PR.
and make sure your line endings are not CRLF etc not \r\n

thats much better, thanks!


Hotfix: minor typo

I will be contributing more soon, but reading the docs I found a real small typo so figured I'd push it.

Thank you.


fix example 404.rs

it was giving me
error[E0432]: unresolved import `iron::StatusCode`
 --> src/main.rs:4:5
  |
4 | use iron::StatusCode;
  |     ^^^^^^^^^^^^^^^^ no `StatusCode` in the root

hi @phlmn , I kinda learning rust, when compile with the old code, it said:
error[E0432]: unresolved import `iron::StatusCode`
 --> src/main.rs:4:5
  |
4 | use iron::StatusCode;
  |     ^^^^^^^^^^^^^^^^ no `StatusCode` in the root

This pull request solve it (on my computer - at least), but on travis, now it said:
error[E0432]: unresolved import `iron::status`
 --> examples/404.rs:3:5
  |
3 | use iron::status;
  |     ^^^^^^^^^^^^ no `status` in the root

You're probably using examples from the master branch (where work happens) with version 0.6. Take a look at the 0.6 tag https://github.com/iron/iron/tree/0.6.0, there you can find examples for that version.


7th version

Hello,
I'm using Rhino 5 and the last Grasshopper on it. The last version I could use is the 5th is it possible to port your last versions also to rhino 5?
I really love by the way your plugin. I use it with a abb 1600-145 and it works pretty good of course with the right settings. I'm sure you will add in the future more features keep going :)
Best

I'm sure you will add in the future more features keep going :)

Either I spend time on new features or in maintaining backwards compatibility, you can't ask for both :stuck_out_tongue: (although lately it's neither)
What's holding you to Rhino 5? Rhino 6 has been out for a long time now.

Well, I understand you :) The problem is I'm living in turkey (€ is not that stable...) and I'm an educator. So no really money to update Rhino 6 :) ...

You can purchase a much cheaper educational version if you're teaching, did you check it out?

Yes I know...And It's not just the money. I'm also a reseller of Rhino. And one of the biggest promoter here in Turkey for Rhino. And in my opinion there is not a major difference between rhino 5 and 6. I mean that I was waiting for more functions, which were implemented in the WIP versions... So in my opinion Rhino 6 is just a boosted Rhino 5 thats it; for that reason I'm waiting also for major updates or it looks like that I will wait until 7 is out which can be years :D

Being able to use the latest plugins is worth the upgrade.

Functions like a good working Cycles, Mesh2, SubD, which will make a huge impact. The next thing will be also maybe an implementation of evee... And yes new plugins of course but the most are working better on rhino 5 for now...


Metadata: expose chromaSubsampling for AVIF

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[x] Running npm install sharp completes without error.
[x] Running node -e "require('sharp')" completes without error.

If you cannot confirm both of these, please open an installation issue instead.
Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

If you cannot confirm this, please upgrade to the latest version and try again before opening an issue.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package instead.
What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?

  System:
    OS: Windows 10 10.0.22621
    CPU: (8) x64 Intel(R) Core(TM) i7-9700 CPU @ 3.00GHz
    Memory: 11.85 GB / 31.68 GB
  Binaries:
    Node: 18.14.0 - ~\scoop\apps\volta\current\appdata\tools\image\node\18.14.0\node.EXE
    Yarn: 1.22.19 - ~\scoop\apps\volta\current\appdata\tools\image\yarn\1.22.19\bin\yarn.CMD
    npm: 9.3.1 - ~\scoop\apps\volta\current\appdata\tools\image\node\18.14.0\npm.CMD
  npmPackages:
    sharp: ^0.31.3 => 0.31.3

What are the steps to reproduce?

put JS below and put images at the same directory
Run the JS
Check the output

What is the expected behaviour?

await sharp("./image.avif").metadata()
await sharp("./lossy.webp").metadata()

should include chromaSubsampling field like JPEG. That of Lossy WebPs is always "4:2:0". Most AVIF use YUV (4:2:0 or 4:4:4).
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem

import sharp from "sharp";
import YAML from "yaml";

/**
 * @param type {string}
 * @param img {import("sharp").Sharp}
 */

async function printMeta(type, img) {
  const { icc: _icc, iptc: _iptc, ...meta } = await img.metadata();

  console.log(`${type} meta:\n\n${YAML.stringify(meta)}`);
}

await printMeta("AVIF", sharp("./avif.avif"));
await printMeta("WebP", sharp("./webp.webp"));

Please provide sample image(s) that help explain this problem

https://e-port.co.jp/wp/wp-content/uploads/b83a40e6b68e9e09c7318f59899df563.jpg.webp
https://cdn.glitch.com/f4525b23-4268-473b-a57c-bdb57f3f7494%2FIMG_5666.avif?v=1599083333763 (from https://avif-support-test.glitch.me/)

For AVIF, libheif exposes subsampling metadata via get_image_chroma_from_configuration but we'd need to add support for this to libvips first. There might be some complexity around different chroma subsampling for alpha vs non-alpha channels.
A libvips PR to expose this would be welcome, if you're able.
For WebP, libwebp does not expose metadata about whether subsampling (a.k.a. "sharp_yuv") was used at compression time, so there's not a lot we can do here (I suspect it will be buried somewhere in the compressed bitstream).

we'd need to add support for this to libvips first.

I see.

For WebP, libwebp does not expose metadata about whether subsampling (a.k.a. "sharp_yuv") was used at compression time

4:2:0 for ALL lossy WebP whether sharp YUV is used or not unlike JPEG or AVIF.
Lossless ones use a different algorithm.


Turning inline fold on automatically

Is there a way to turn the folding on automatically instead of doing it the manual way?
I tried using Neovim's autocmd feature but, I can't seem to find the right event. I tried:

BufNew
BufRead
BufReadPost
FileRead
FileReadPost

When I've set it to BufWritePost it works. But I'd really like for it to turn on after opening a new buffer. 😞
Here is my config:
-- init.lua
local autocmd = vim.api.nvim_create_autocmd

autocmd({ "BufWritePost" }, {
  pattern = {
    "*.svelte",
    "*.jsx",
    "*.tsx",
    "*.html",
  },
  command = "InlineFoldToggle",
})

Thank you @3719e04! The autocmd from your pull request works. I'm really new to all of this neovim stuff.


Compatibility with watchOS or tvOS?

Why podspec not compatible with apple watchOS or tvOS. Is it any issue to use DFU library on those platforms?

No, we just didn't do any tests there. Please, prepare a PR with required changes and we'll merge it and release in the next version.

I am testing now on aw current lib and when all test would be finished can provide changed podspec and readme files in this case

Please, create a PR to develop branch.

Merged.


Very unsafe Clone implementation over *mut u8

The amount of things that can happen here are endless. Basically I have the possibility of having two or more mutable pointers to the same place, imagine it in concurrency. Fortunately, Send has not been implemented, so it cannot be passed between threads, although that does not ensure disaster.
https://github.com/Kogia-sima/sailfish/blob/master/sailfish/src/runtime/buffer.rs#L13-L18
This is the prelude to the apocalypse:
#[derive(Clone)]
struct Any<T>(* mut T)

Edit: Sorry, yes, you implement Send:
   let buf = Buffer::with_capacity(32);
   let buf2 = buf.clone();
   let mut buf = buf.into_string();
   let j = spawn(move || write!(buf, "this"));
   let mut buf2 = buf2.into_string();
   write!(buf2, "or this");
   j.join().unwrap();
   println!("{}", buf2)

https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=42d37db0830717872d53b14b48c28575
What do you see at your screen?

Thank you for pointing out my mistake! It is definitely a bug. I'll fix this soon.


Seas we() updates

This has a conflict with time-series-w-NA but I'll hopefully get the merge sorted out since there are a lot of updates here I want to keep (and the US Nowcasting example).

Codecov Report

Merging #30 into master will decrease coverage by 0.24%.
The diff coverage is 40.35%.

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   41.31%   41.07%   -0.25%
==========================================
  Files          10       10
  Lines        1941     1984      +43
==========================================
+ Hits          802      815      +13
- Misses       1139     1169      +30

Impacted Files
Coverage Δ

R/seas_factors.R
67.85% <100%> (ø)
:arrow_up:

R/seas_we.R
54.78% <39.28%> (-14.67%)
:arrow_down:

Continue to review full report at Codecov.

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

OK... should be good to merge now. The main thing here is putting forecasts of SA factors back in sa_we() with ahead = 0, (though maybe we should change it to forecast = 0 as forecast is probably more obvious?) as well as automatic log transformation or the option to specify 'log' or 'none'. This also deals with non-stationary series using loess() as discussed earlier. I'll keep working on things today... but probably good to merge soon to avoid more conflicts.


Add http4s 0.23.6 support

The change fixes regression tests filtering by framework: scalaFrameworks and javaFrameworks contained projectName before but ExampleCase required framework name instead. When scalaFrameworks.toSet was used, frameworks which have different name and projectName were filtered out. Considering that all scalaFrameworks.toSet usages were replaced by list of frameworks to prevent code generation where it's not required.
The http4s update change follows the suggestion from the comment. Also it copies http4s sample and tests for 0.22 to control regressions.

Reviewing this now. Presumably if this builds and tests pass locally but not in CI here, it's because the build is attempting to use the released versions of inner modules. I'm still iterating on how to ensure we maintain bincompat moving forward after the modularization, so the failure to build here likely isn't a reflection of an actual problem.

that all scalaFrameworks.toSet usages were replaced by list of frameworks to prevent code generation where it's not required.

The ability to compile at all is part of the regression tests, even if there are no runtime tests utilizing those -- it's to ensure that source compatibility doesn't drift with the underlying libraries

that all scalaFrameworks.toSet usages were replaced by list of frameworks to prevent code generation where it's not required.

The ability to compile at all is part of the regression tests, even if there are no runtime tests utilizing those -- it's to ensure that source compatibility doesn't drift with the underlying libraries

good point, returned usage of the scalaFrameworks

Reviewing this now. Presumably if this builds and tests pass locally but not in CI here, it's because the build is attempting to use the released versions of inner modules. I'm still iterating on how to ensure we maintain bincompat moving forward after the modularization, so the failure to build here likely isn't a reflection of an actual problem.

how can I address the problem with the CI, maybe, split the PR?

how can I address the problem with the CI, maybe, split the PR?

I think during CI, if the major or minor labels are applied to the PR, it should disable the ABI checking

In the meantime, I've run runtimeScalaSuite locally and it seems to fail with the same errors here, so those will need to be resolved before this can be merged. The errors from the most recent build look correct

... and it looks like I don't have permissions to set labels 😅

Hi, @blast-hardcheese! I'm surprised a bit by the coverage report, could you assist finding a reason of such coverage falling?

Hi, @blast-hardcheese! I'm surprised a bit by the coverage report, could you assist finding a reason of such coverage falling?

codecov highlights the fact that for in ScalaModule we went from having "http4s" to "http4s", "http4s-v0.22", "http4s-v0.23" -- I'm not worried about such a small reduction in code coverage, don't worry 😁


Added a way to provide the database files to the DatabasePeerManager module.

Created an interface to provide database files at the time of a peer connection. This allows applications that create databases in directories other than the default one to have their data viewable in the Database section.

I'm curious for your thoughts at a high level on the API for configuring modules.  I believe at present you are writing something like:
new InspectorModulesProvider() {
  public Iterable<> get() {
    HashSet<> modules = new HashSet<>();
    Collections.add(modules, Stetho.defaultInspectorModulesProvider(context).get());
    replaceByType(Database.class, new Database(context, new MyDatabaseProvider(...)));
    return modules;
  }
}

We're doing this internally in FB4A/Messenger, but I'm really unhappy with it.  Seems like these use cases should be expressed more conveniently, perhaps with some kind of pattern like:
Stetho.newInspectorModulesBuilder(context)
   .addDefaults()
   .add(new Database(...));
   .build();

Internally addDefaults would be a boolean which operated like:
for (ChromeDevtoolsDomain domain : mCustomModules) {
  modules.add(domain);
  customModuleNames.add(InspectorUtil.getDomainName(domain));
}
if (mAddDefaults) {
  if (!customModuleNames.contains("Database")) {
    modules.add(...);
  }
  ...
}

I'm curious for your thoughts at a high level on the API for configuring modules.

I'm still playing around with things so I haven't given it a lot of thought. Right now what I have done is copied the list of modules from the defaults and add each of them individually.
        ArrayList<ChromeDevtoolsDomain> modules = new ArrayList<ChromeDevtoolsDomain>();
        modules.add(new Console());
        modules.add(new CSS());
        modules.add(new Debugger());
        modules.add(new DOM());
        modules.add(new DOMStorage(context));
        modules.add(new HeapProfiler());
        modules.add(new Inspector());
        modules.add(new Network(context));
        modules.add(new Page(context));
        modules.add(new Profiler());
        modules.add(new Runtime());
        modules.add(new Worker());
        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB) {
          modules.add(new Database(context, myDatabaseFilesProvider));
        }
        return modules;

I do like your proposed method though.

Looks good, clean up the minor nits then rebase/squash and we can merge.

Do you happen to have an Intellij code style file published anywhere?

@jferlisi no, working on it tho :)

@jasta Pushed changes.

Looks good, please squash into 1 commit (http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html), then git push -f into your github fork branch.

@jasta Done, thanks for the help on this.


⚠ Bofrost.be has degraded performance

In b170068, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1881 ms

Resolved: Bofrost.be performance has improved in f3b61d5 after 9 minutes.


Support Java 11

This commit upgrade Gradle used in the project to the latest available
version 5.5 to support Java 11.
It also switches from (now deprecated) FindBugs plugin in SpotBugs for
the same reason.
gradle/wrapper directory and gradlew.{bat|sh} are re-generated
automatically by Gradle.

Sorry, missed this issue with Javadoc, fixing it.

 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.

Would you consider merging this?

Yes! I just want to verify that this will still continue to work with older versions of java?

@mtagle it will. It's checked on Travis.
Also, I found that sourceCompatibility hasn't been set correctly. I fixed this in a new commit and now it should always target compatibility with Java 8 despite the JDK version being used for building.


Add a retry mechanism for VM migration

Description

This PR adds a retry mechanism controlled by two settings (number of retries & wait between retries) when migrating a vm.
The VM initial deployment process already has such retries which help circumvent hypervisors issues and race conditions when allocating resources.

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)
[X] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
This change is trivial.
Feature/Enhancement Scale

[ ] Major
[X] Minor

Bug Severity

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

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

@shwstppr Default value has been changed to respect original behaviour.
VM migration is already tested in the vm_lifecycle test suite.
Anything else could prevent this from moving forward ?

@shwstppr Default value has been changed to respect original behaviour. VM migration is already tested in the vm_lifecycle test suite. Anything else could prevent this from moving forward ?

Thanks for addressing the suggestion.
I don't think VM migration in vm_lifecycle test suite will be able to test retries. Have you done any testing where migration failed first but succeeds in retries?

@blueorangutan package

@shwstppr a 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: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5961

@blueorangutan package

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7582

@blueorangutan test

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

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

Test
Result
Time (s)
Test File

test_02_upgrade_kubernetes_cluster
Failure
592.74
test_kubernetes_clusters.py

Ping @benj-n Can you check/address the comments. Thanks.

@benj-n , I moved this to the next major release as there is little activity on this. cc @JoaoJandre
I think it is almost done though.


test: add missing aztec-address tests

Doable now that https://github.com/noir-lang/noir/issues/4635 is closed.

Benchmark results
Metrics with a significant change:

note_successful_decrypting_time_in_ms (8): 208 (-27%)
note_successful_decrypting_time_in_ms (32): 631 (+17%)
note_trial_decrypting_time_in_ms (8): 107 (+80%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,412
6,400
6,424

l1_rollup_execution_gas
585,745
585,733
585,757

l2_block_processing_time_in_ms
1,319
4,804 (+1%)
9,330

note_successful_decrypting_time_in_ms
:warning: 208 (-27%)
:warning: 631 (+17%)
1,030 (+8%)

note_trial_decrypting_time_in_ms
:warning: 107 (+80%)
39.9 (-14%)
47.1 (-38%)

l2_block_building_time_in_ms
10,408
37,010
72,761 (-1%)

l2_block_rollup_simulation_time_in_ms
10,294
36,619
72,005 (-1%)

l2_block_public_tx_process_time_in_ms
4,902
19,131
38,234

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,932 (-5%)
27,473 (+2%)

note_history_successful_decrypting_time_in_ms
1,158 (+1%)
2,231

note_history_trial_decrypting_time_in_ms
42.0 (+22%)
85.1 (+8%)

node_database_size_in_bytes
18,616,400
35,049,552

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
144 (+1%)
44,377
26,164

private-kernel-inner
157 (+1%)
71,744
26,164

private-kernel-ordering
108 (+1%)
50,830
39,026

base-parity
3,933 (+2%)
128
311

root-parity
1,059 (-5%)
1,244
311

base-rollup
13,276
110,464
861

root-rollup
49.5
4,359
725

public-kernel-app-logic
79.6 (-1%)
47,695
40,661

public-kernel-tail
112 (-1%)
53,372
7,125

merge-rollup
11.2 (+4%)
2,568
861

public-kernel-teardown
77.3 (+1%)
47,695
40,661

public-kernel-setup
76.4
47,695
40,661

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1
16.0
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.586
0.493
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
45.8
72.5 (+1%)
231
446
874
1,730
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.470
0.447 (+1%)
0.421
0.416
0.415
0.414
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.4 (+1%)
107
338
662
1,312
2,612 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.480
0.480
0.458
0.455
0.454
0.455
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.2

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.534

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
40,372
496,534

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,195 (+2%)
830 (+1%)
4,041 (+1%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
421
283 (+1%)
733 (+1%)
370 (+2%)
1,203
434 (+1%)


Cannot display date time stamp in american format

Describe the bug
Right now, the date and time stamp is displayed in international format .. DD/MM/YY - but unfortunately Americans are in reverse. And I don;'t think there is a way to change it.
Steps to reproduce
go to any chat room and see the date/time stamp
Example:

Expected behavior
The ability to display Feb 6, 23 2:20 PM  so its globally compatible
Screenshots

Device (please complete the following information)
Mobile App - across all platforms
Additional context

You can set timestamp in rooms > lastMessage and messages props however you want, it doesn't need to be a date object, it can be a string.
So you can format the date on your side and pass timestamp: 'Feb 6, 2:20 PM'

Closing this, please reopen if you still have an issue


only serves one image after every start-up, and "TcpVideoStream Timeout" error will occur.

1.ffmpeg version

2.TcpVideoStream error

3.run "node test/run.js"

Hi, @BiGrabbit, thanks for the report.  The test failure issue has been reported: See #78 and #49.
The TcpVideoStream timeout happens sometimes, but the code is designed to reconnect when that happens, and it looks like that's working as designed in your case.
I'm going to close this for now, but let me know if you don't think I've addressed your issues.
(In the future, please paste text instead of screenshot images--images are not searchable, but text is.)

hello, wiseman! thanks for your reply in time. I want to know how to solve the problem "pngSteram returns only one png after every start-up".

Ok, for that issue see #99.


Error connection timeout

After getting result on the question, in many case I started to get that error when I'm clicking on "Clear" button.

in the console (verbose==True) I don't see anything suspicious :
Post-Generate Langchain: 2024-02-29 09:54:47.236477 decoded_output: 13402
INFO:     <IP_ADDRESS>:61784 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:61784 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:61784 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:61784 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:61784 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:61784 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62041 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62041 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62041 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62041 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62041 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62041 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62044 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62044 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62044 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62044 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62044 - "POST /queue/join HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:62044 - "GET /queue/data?session_hash=31z1275j41a HTTP/1.1" 200 OK

I've seen that sometimes too, although rarely.  I think it is due to gradio 4.
If you want to go back to gradio 3, h2oGPT is compatible with that.  You just need to do:
pip uninstall gradio gradio_client gradio_pdf -y
pip install gradio==3.50.2

let me know if this helps.  In meantime, I'll see what gradio team think.

@pseudotensor  it's failing with that version of gradio (3.50.2)
/mnt/AI/h2ogpt/gradio_utils/prompt_form.py:209: GradioUnusedKwargWarning: You have unused kwarg parameters in Chatbot, please remove them: {'likeable': True}
  text_output = gr.Chatbot(label=output_label0,
/mnt/AI/h2ogpt/gradio_utils/prompt_form.py:214: GradioUnusedKwargWarning: You have unused kwarg parameters in Chatbot, please remove them: {'likeable': True}
  text_output2 = gr.Chatbot(label=output_label0_model2,
Traceback (most recent call last):
  File "/mnt/AI/h2ogpt/generate.py", line 20, in <module>
    entrypoint_main()
  File "/mnt/AI/h2ogpt/generate.py", line 16, in entrypoint_main
    H2O_Fire(main)
  File "/mnt/AI/h2ogpt/src/utils.py", line 69, in H2O_Fire
    fire.Fire(component=component, command=args)
  File "/mnt/AI/miniconda3/envs/python311/lib/python3.11/site-packages/fire/core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/AI/miniconda3/envs/python311/lib/python3.11/site-packages/fire/core.py", line 475, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
                                ^^^^^^^^^^^^^^^^^^^^
  File "/mnt/AI/miniconda3/envs/python311/lib/python3.11/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
                ^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/AI/h2ogpt/src/gen.py", line 2194, in main
    go_gradio(**locals())
  File "/mnt/AI/h2ogpt/src/gradio_runner.py", line 1268, in go_gradio
    text_output, text_output2, text_outputs = make_chatbots(output_label0, output_label0_model2,
                                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/AI/h2ogpt/gradio_utils/prompt_form.py", line 349, in make_chatbots
    rating1.click(ratingfn1, outputs=rating_text_output, js=review_js1)
TypeError: EventListenerMethod.__call__() got an unexpected keyword argument 'js'

Yes thanks, recent commit from external contrib.  We don't test for this backwards compat.  I fixed the issue, please try again.

@pseudotensor Working much better, stable, Thanks


We have yarn.lock in repo but also .gitignore

We pushed the yarn.lock file to the repo but also added to the .gitignore file.
As far as I know, there are some different ideas about this issue. I think adding a lock file is better because some changes in libraries could create some problems. Also, big projects are adding a lock file.
Examples:
https://github.com/facebook/react
https://github.com/vuejs/vue
https://github.com/facebook/create-react-app
https://github.com/mui/material-ui

Hello @OrhanOzkercin, can you open a pr for this?

Of course! I just wanted to discuss what we want. If it is okay, I can remove the .lock file from the . gitignore

Of course! I just wanted to discuss what we want. If it is okay, I can remove the .lock file from the . gitignore

Ofc!

@OrhanOzkercin could you please update this issue's status?


Restrict Comments: District Admin Only

Is your feature request related to a problem? Please describe.
Currently, multiple user roles (Staff Developers, Doctor Developers, etc.) can comments in the system, which poses a risk because irrelevant comments.
Describe the solution you'd like
Implement restricted comment management permissions where:

Only District Admins should have the ability to delete or edit comments
Other roles (Staff Developers, Doctor Developers) should only have read and create permissions for comments

As you can see in the picture, many irrelevant comments are present.

Can i work on this issue?

Can i work on this issue?

in production there's no irrelevant comments like these


Cells not showing on gall page

The gall description page does not show the cells despite them being an enterable field on the gall edit page.

Oops. I messed it up when adding Form. I will fix it and get it deployed tomorrow.


Docs: Update v0.3.0 changelog

Provides the v0.3.0 Changelog. Also attempts to fix RTD builds by pinning to Py3.6 which passes on my laptop. @Lnaden seems to be an issue with Pydantic if you have ideas.
RTD Errors

Running Sphinx v1.8.1

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/validators.py", line 262, in find_validators
    if issubclass(type_, val_type):
TypeError: issubclass() arg 1 must be a class

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/config.py", line 368, in eval_config_file
    execfile_(filename, namespace)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/util/pycompat.py", line 150, in execfile_
    exec_(code, _globals)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/docs/source/conf.py", line 13, in <module>
    import qcfractal
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/__init__.py", line 5, in <module>
    from . import interface
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/__init__.py", line 5, in <module>
    from . import collections
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/__init__.py", line 2, in <module>
    from .biofragment import BioFragment
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/biofragment.py", line 15, in <module>
    class BioFragment(Collection):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/biofragment.py", line 43, in BioFragment
    class DataModel(Collection.DataModel):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/main.py", line 124, in __new__
    config=config,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 107, in infer
    schema=schema,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 87, in __init__
    self.prepare()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 135, in prepare
    self._populate_sub_fields()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 241, in _populate_sub_fields
    self.key_field = self._create_sub_type(self.type_.__args__[0], 'key_' + self.name)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 257, in _create_sub_type
    model_config=self.model_config,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 87, in __init__
    self.prepare()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 136, in prepare
    self._populate_validators()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 266, in _populate_validators
    self.model_config.arbitrary_types_allowed)),
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/validators.py", line 265, in find_validators
    raise RuntimeError(f'error checking inheritance of {type_!r} (type: {display_as_type(type_)})') from e
RuntimeError: error checking inheritance of ~KT (type: KT)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/cmd/build.py", line 303, in build_main
    args.tags, args.verbosity, args.jobs, args.keep_going)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/application.py", line 201, in __init__
    self.config = Config.read(self.confdir, confoverrides or {}, self.tags)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/config.py", line 203, in read
    namespace = eval_config_file(filename, tags)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/config.py", line 380, in eval_config_file
    raise ConfigError(msg % traceback.format_exc())
sphinx.errors.ConfigError: There is a programmable error in your configuration file:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/validators.py", line 262, in find_validators
    if issubclass(type_, val_type):
TypeError: issubclass() arg 1 must be a class

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/config.py", line 368, in eval_config_file
    execfile_(filename, namespace)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/util/pycompat.py", line 150, in execfile_
    exec_(code, _globals)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/docs/source/conf.py", line 13, in <module>
    import qcfractal
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/__init__.py", line 5, in <module>
    from . import interface
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/__init__.py", line 5, in <module>
    from . import collections
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/__init__.py", line 2, in <module>
    from .biofragment import BioFragment
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/biofragment.py", line 15, in <module>
    class BioFragment(Collection):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/biofragment.py", line 43, in BioFragment
    class DataModel(Collection.DataModel):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/main.py", line 124, in __new__
    config=config,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 107, in infer
    schema=schema,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 87, in __init__
    self.prepare()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 135, in prepare
    self._populate_sub_fields()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 241, in _populate_sub_fields
    self.key_field = self._create_sub_type(self.type_.__args__[0], 'key_' + self.name)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 257, in _create_sub_type
    model_config=self.model_config,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 87, in __init__
    self.prepare()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 136, in prepare
    self._populate_validators()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 266, in _populate_validators
    self.model_config.arbitrary_types_allowed)),
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/validators.py", line 265, in find_validators
    raise RuntimeError(f'error checking inheritance of {type_!r} (type: {display_as_type(type_)})') from e
RuntimeError: error checking inheritance of ~KT (type: KT)

Configuration error:
There is a programmable error in your configuration file:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/validators.py", line 262, in find_validators
    if issubclass(type_, val_type):
TypeError: issubclass() arg 1 must be a class

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/config.py", line 368, in eval_config_file
    execfile_(filename, namespace)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/sphinx/util/pycompat.py", line 150, in execfile_
    exec_(code, _globals)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/docs/source/conf.py", line 13, in <module>
    import qcfractal
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/__init__.py", line 5, in <module>
    from . import interface
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/__init__.py", line 5, in <module>
    from . import collections
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/__init__.py", line 2, in <module>
    from .biofragment import BioFragment
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/biofragment.py", line 15, in <module>
    class BioFragment(Collection):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/checkouts/latest/qcfractal/interface/collections/biofragment.py", line 43, in BioFragment
    class DataModel(Collection.DataModel):
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/main.py", line 124, in __new__
    config=config,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 107, in infer
    schema=schema,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 87, in __init__
    self.prepare()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 135, in prepare
    self._populate_sub_fields()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 241, in _populate_sub_fields
    self.key_field = self._create_sub_type(self.type_.__args__[0], 'key_' + self.name)
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 257, in _create_sub_type
    model_config=self.model_config,
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 87, in __init__
    self.prepare()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 136, in prepare
    self._populate_validators()
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/fields.py", line 266, in _populate_validators
    self.model_config.arbitrary_types_allowed)),
  File "/home/docs/checkouts/readthedocs.org/user_builds/qcfractal/conda/latest/lib/python3.7/site-packages/pydantic-0.14-py3.7.egg/pydantic/validators.py", line 265, in find_validators
    raise RuntimeError(f'error checking inheritance of {type_!r} (type: {display_as_type(type_)})') from e
RuntimeError: error checking inheritance of ~KT (type: KT)

Status

[x] Ready to go

Fun. I'll take a look at it

Seems to work fine in Python3.6. Yay...


Update pom.xml

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/4228759367

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/4228759367
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/4228759367

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/4228759367
Status: success


[ZEPPELIN-1878] added npm-debug.log on gitignore

What is this PR for?
zeppelin-web/npm-debug.log file is only used debugging.
So, we do not tracking.
What type of PR is it?
Improvement
Todos

[x] - fix .gitignore

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1878
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

LGTM

CI failure is irrelevant to this change. Merge into master if there are no more comments.

CI failure is irrelevant to this change. Merge into master if there are no more comments.


[RESOLVED] High ICMP ping response time

Problem started at 10:52:33 on 2023.06.16
Problem name: High ICMP ping response time
Host: S100-Santiago
Severity: Warning
Operational data: Value: 741ms
Original problem ID: 28855
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23518&eventid=28855

Problem has been resolved in 6m 0s at 10:58:33 on 2023.06.16
Problem name: High ICMP ping response time
Host: S100-Santiago
Severity: Warning
Original problem ID: 28855


Bb 386 input of projected coordinates tab 2 pnt

Added logic to detect when the coordinate system is not GCS_WGS_84 and create points from two fields with projected x and Y values (such as State Plane or UTM).
Description
Enhancement to resolve issue 386 - combine the features of Convert Coordinate Notation (current capability) with XY Table To Point.
Related Issue
#386
Motivation and Context
Allows input of X and Y values that are in projected coordinate systems
How Has This Been Tested?
Tested tool on Naperville test data included with issue.
To test:

Unzip the Naperville Crime data to a local folder.
In ArcGIS Pro, open Table To Point.
Set naperville_crime table as Input Table
Set DD_2 as Input Coordinate Format
Set POINT_X and POINT_Y for X Field and Y Field respectively.
Set Spatial Reference to NAD_1983_HARN_StatePlane_Illinois_East_FIPS_1201_Feet
Run the tool.
Output shows new points on the map.

OR, for UTM:

Use POI_UTM_X_Y.csv from the military-tools-geoprocessing-toolbox\utils\testdata\CSV test data folder.
In ArcGIS Pro, open Table To Point.
Set POI_UTM_X_Y.csv table as Input Table
Set DD_2 as Input Coordinate Format
Set POINT_X and POINT_Y for X Field and Y Field respectively.
Set Spatial Reference to WGS 1984 UTM Zone 10N
Run the tool.
Output shows new points on the map.

Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.

@kgonzago @csmoore - actually, please hold off on review and merge.  I thought of a couple of cases that I want to test, to be sure it is behaving correctly.

Added a test case and made some changes so would work in ArcMap in https://github.com/Esri/military-tools-geoprocessing-toolbox/pull/412/commits/2994cfee133e79a1b2279af19b1bb5e5f46317b0


Simplify line count in formatDoc

This avoids unnecessarily splitting the document lines. Although it does simplify the code slightly and improves its performance, my primary motivation for this change is that the previous line count was actually one higher than it should have been. This would cause some editors, such as helix, to reject the change as it was out of bounds.

Looks good to me. I also tested and verified it continues to work in vscode. Thank you for the contribution!


What is Bad introspection data? when I tried to  reload docs.

Describe the bug
It cause an error Bad introspection data, when I tried to  reload docs.

@imolorhe  Can you print the error?  so that I can find the cause of the error.

src/app/services/gql.service.ts

  getIntrospectionSchema(data) {
    try {
      if (data && data.__schema) {
        const schema = buildClientSchema(data);

        // One type => many fields
        // One field => One type
        return schema;
      }
      return null;
    } catch (err) {
      console.error('Bad introspection data.');
      return null;
    }
  }

@mokeyish I'd look into that. You could also create a PR to log err.


issue wit complex filters

Hi, thank you very much for the plugin. It works very well for me, runnung again an Open-Ldap-Server. I have only two issues:
One issue is that I cannot use complex filters like (&(objectclass=posixAccount)((!(uid=%s)(mail=%s)))
I got these messages:
Warning: sprintf(): Too few arguments in /kunden/563849_51429/webseiten/wordpress/wp-content/plugins/authldap/ldap.php on line 247
Warning: ldap_search(): Search: Protocol error in /kunden/563849_51429/webseiten/wordpress/wp-content/plugins/authldap/ldap.php on line 202

Have you tried using %1$s instead of $s? That should do the trick...

thanks. No, this is not working for me.
Now i cannot log in anymore :(

I more or less described your exact use-case in the README-file. Have a look at Filter

Warning: ldap_search(): Search: Bad search filter in /kunden/563849_51429/webseiten/wordpress/wp-content/plugins/authldap/ldap.php on line 202

So, i deactivated all plugins via database, then check all entries - there was a typo :)
Everything ist working well now.
Thank you very much!


Add Celery Worker to Automatically Import from UISP

We should now be able to synchronize automatically with UISP on an hourly basis
When there is a diff with MeshDB, we take the UISP values, and we notify admins of the fact this has occurred, via the new slack notification system introduced in #450
Stole some import logic from the existing import script. However, a lot of that is tightly coupled to spreadsheet nonsense and should be shipped out to sea, which means most of this is new code anyway.
Also adds types for the UISP API, made in collaboration with ChatGPT :)
Remaining work:

[ ] Create the correct device subtype (Sector vs Device vs AccessPoint)
[ ] Fix logging message TODOs
[ ] Test coverage? Pretty good RN but the bot might complain
[ ] Carefully think through LOS sync edge cases (i.e. null abandon date)
[ ] Admin notification for LOS modification?
[ ] Fix linter

Closes #383

Hmm perhaps the types should be excluded from coverage information

Ran the celery task locally. Seems to work, and takes ~5 minutes on my laptop


Profanity Filter v2

Currently under development.
Supercedes #44.
Closes #6.

Looking at this, would it be possible to make this use the database to store the filter words instead?

Possibly, however it would require a bit more work...what would be the benefit of doing so over just a json file like it was previously?

It would make it easier to add and remove words. Unfortunately because this uses Docker, it's not as simple as putting a JSON file on the repo and expecting it to work, as any adaptations then couldn't be made through the bot.
But then of course there's the option of preloading the words and just working with those.
I guess process as you were for now, and we'll figure this out later. Theoretically a switch to database usage wouldn't be too difficult to achieve.

Ok...but didn't like support v1 use a JSON file with Docker and it worked right?

It's less whether it'd work right, and more that files in Docker volumes are difficult and annoying to deal with.
Also Support V1 used the JSON format as a memory dump basically lmao. It was far from ideal 😆
I think it should be fine as a file tbh. Provided it doesn't read the file every time it does a check, which I don't think it does.

No if my memory is correct it does not. So getting the words from the DB could be easy enough. The only thing is that the easiest way to pass the words into content filter is via a JSON file in my opinion. However, if you'd like I should be able to make the words just get stored in the DB relatively easy, whichever you prefer.

By the looks of things it only opens a file every time if we use the default words, which we won't be. So yeah just use a JSON for now, should be fine. If it turns out to be an issue then we'll make the switch to the database.

Ok sounds good.

Closing as same behaviour can be achieved using the automod.


Windows Defender found Virus Trojan:AndroidOS/MuItiverze

Summary
Provide brief overview and context for the discovered bug.
Steps to Reproduce

Download the Release
Open the Zip Package
Find Windows Defender issuing problems

Maybe add a warning to the readme or change the code?

Thanks, not sure why it happened.
I will look at that next month because I am OOO.
We will check if we can solve it or like your suggestion, add a note about that.

Also identified as "Wacatac.H!ml"

I added a warning:
https://github.com/cyberark/PipeViewer#warning
We can't change it because it depends on the NtObjectManager package which is flagged as a virus.


[OpenCV] unable to build for x64-windows using v141 toolset

Again, due to #1619, OpenCV is now broken also on x64-windows with my toolchain (v141), because a depencency (ceres) can only be built using VS 2015 (which is not available on my machine), due to this line in the portfile (line 19 in ports/ceres/portfile.cmake)
set(VCPKG_PLATFORM_TOOLSET "v140") # Force VS2015 because VS2017 compiler return internal error

To be honest, OpenCV 3.3 PR has been stagnant for so long I was hoping because of tests, but now that it is merged OpenCV is impossible to build, in any configuration...
Did you tested it? Or breaking previous available triplets without any notice is an intended effect?
Isn't this port mechanism (instead of an automatic system) in place just to avoid these problems?

I only tested in VS 2015 instead of VS 2017. The ceres dependency is from opencv_contrib. You can temporarily go into the portfile to disable opencv_contrib and it should build.

Thank you very much for opening this issue. I've disabled the ceres dependency for all architectures for now; we'll look at conditionally re-enabling it for x64 only once we're able to fix the VS2017 incompatibility.

There are still some lingering issues which we are actively working to resolve.

Ok, all the above builds should be fixed in the latest master (3.3.0-4).
We will test OpenCV more in the future to avoid issues like this, especially considering how widely used it is.

@ras0219-msft hi, before install opencv, I have installed eigen3 (the newest version 3.3.4-1) from the port. when I install opencv, the opencv_sfm module failed with the flowing errors:
   “D:\Development\Env\vcpkg\buildtrees\opencv\x64-windows-rel\install.vcxproj”(default target) (1) ->
   “D:\Development\Env\vcpkg\buildtrees\opencv\x64-windows-rel\ALL_BUILD.vcxproj”(default target) (3) ->
   “D:\Development\Env\vcpkg\buildtrees\opencv\x64-windows-rel\modules\sfm\src\libmv_light\libmv\multiview\multiview.vcxproj”(default target) (12) ->
   (ClCompile target) ->
     d:\development\env\vcpkg\installed\x64-windows\include\eigen\src\core\redux.h(237): fatal error C1001: An internal error has occurred in the compiler. [D:\Development\Env\vcpkg\buildtrees\opencv\x64-windows-rel\modules\sfm\src\libmv_light\libmv\multiview\multiview.vcxproj]

347 Warning(s)
1 Error(s)

as a result, I have to disable the opencv_sfm module by adding "-DBUILD_opencv_sfm=OFF" option  in my portfile.cmake or removing eigen3.3.4 from my vcpkg environment to make opencv build successfully.
but, I'll be very interested in how to fix this issue? thanks:)


[2.x backport] Avoid timing side-channels in signature validation

See https://github.com/99designs/http-signatures-php/pull/28

Great, thanks. We'll get this released today. Very much appreciated.

I've just created a new release for this as 2.0.5. Thanks again @afk11.


publish, 9/27

Docs Build status updates of commit 29065a8:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/64/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Improved Tag Management

Improved Tag management:

Tags can now be added 1 by 1 one using the t/ prefix instead of overwriting the entire tag list
Tags can be deleted using the dt/ prefix followed by the name of the tag to be deleted
Tags can also be mass deleted using dt/all
dt/ and t/ prefixed are allowed to be used simultaneously on the same line

Developer sided changes:

Changed getTags from unmodifiableSet to a normal Set so that i can edit the tags
Added dt/ prefix to relevant files to accept delete tag commands
Added tagsToDel in editPersonDescriptor to indicate a change required, include getter and setter methods

Changes Unknown when pulling 9e7deaa656c553ae0dfeba28df9582b1ee462258 on Juxarius:master into ** on CS2103AUG2017-T13-B1:master**.

About the t/, i posted an BUG issue. With this feature t/ will not do anything but it does not trigger a missing tag statement even tho the regex suggests +, which means a tag is required

Changes Unknown when pulling 30fe3ec76757c46c2ccbd6c361a46953ef7e0ba4 on Juxarius:master into ** on CS2103AUG2017-T13-B1:master**.


Activate pin on Attiny84

Hello,
I would like use pin PB2 on Attiny84 but I don't know how make.
You say :

You may activate them in the setting file (advanced).

But where is this file and how make ?
Thank you

https://github.com/NicoHood/PinChangeInterrupt/blob/master/src/PinChangeInterruptSettings.h#L221

ok, but pin INT0 is not disable by default?
because I try
attachPinChangeInterrupt(digitalPinToPCINT(2),check_btn,FALLING);
but it doens't work
                      _________
      	              |  |_|  |
                      |       |
                VCC 1 |       | 14  GND
      (D 0) PB0     2 |  	  | 13  AREF ( D10)
      (D 1) PB1     3 |	      | 12  PA 1 (D 9)
                    4 |       | 11  PA2 (D 8)
    INT0 (D 2) PB2  5 |	      | 10 PA 3 (D 7)
      (D 3) PA7     6 |    	  | 9   PA 4 (D 6)
      (D 4) PA6     7 |    	  | 8  PA 5 ( D5)
     		          |_______|

It should not. please try all pins and report which work an which not. I've had bugs in the code before for attiny, so its possibly that i've missed this. I've never tested the attnys myself.

I have check all pin with this code
#include <PinChangeInterrupt.h> // http://code.google.com/p/arduino-tiny/downloads/list

void blinks()
{

  digitalWrite(3, HIGH);   // turn the LED on (HIGH is the voltage level)
  delay(1000);              // wait for a second
  digitalWrite(3, LOW);    // turn the LED off by making the voltage LOW
  delay(1000);              // wait for a second
}

// the setup function runs once when you press reset or power the board
void setup() {
  // initialize digital pin 13 as an output.
  pinMode(3, OUTPUT);
 attachPinChangeInterrupt(digitalPinToPCINT(2),blinks,FALLING);   // attache l'interruption externe n°0 à la fonction check_btn

}

// the loop function runs over and over again forever
void loop() {
}

Pins D0, D1, D2 didn't work, other pins are ok.

Have you enabled PortB a suggested?

Yes I have change this code
#if defined(__AVR_ATtiny24__) || defined(__AVR_ATtiny44__) || defined(__AVR_ATtiny84__)
// Port1 is connected to reset, crystal and Pin Interrupt 0
// Deactivate it by default
#if defined(PCINT_ENABLE_PORT1)
#undef PCINT_ENABLE_PORT1
#endif
#endif

IN
if defined(__AVR_ATtiny24__) || defined(__AVR_ATtiny44__) || defined(__AVR_ATtiny84__)
// Port1 is connected to reset, crystal and Pin Interrupt 0
// Deactivate it by default
if defined(PCINT_ENABLE_PORT1)
undef PCINT_ENABLE_PORT1
endif
endif

This is expected. You need to  comment the undef:
#if defined(__AVR_ATtiny24__) || defined(__AVR_ATtiny44__) || defined(__AVR_ATtiny84__)
// Port1 is connected to reset, crystal and Pin Interrupt 0
// Deactivate it by default
#if defined(PCINT_ENABLE_PORT1)
//#undef PCINT_ENABLE_PORT1
#endif
#endif