Train a ControlNet plugin instead of full-scale fine-tuning?

This work is very inspiring and exciting. Marigold makes huge progress in discriminative diffusion models by showing that general-purpose pre-training can benefit later fine-tuning for discrimination, so that we no longer train discriminative diffusion models from scratches.
Now the problem is the FULL-SCALE fine-tuning. In fact there are alternative ways in generative diffusion models. For example, ControlNet keeps the backbone U-Net frozen and trains a plugin instead, where the plugin can toggle the behavior of the backbone to certain purposes. This approach is more efficient and more flexible.
So I wonder if you can train a plugin-Marigold with all the other settings unchanged? If this approach can be demonstrated feasible (or even infeasible), the community can get very useful insights.

Thanks for your suggestions. ControlNet-based method is indeed an interesting topic to be investigated. However, it's beyond our current plan.
The training code has been released. You are welcome to adapt to ContolNet and contribute.
Discussions along this line are also welcome in this issue.


[MINOR] Refactor component tests (move layout tests into componentx)

Just for testing if moving all layout tests into an excluded componentx would fix the github workflows

ok, the following reconfiguration seems to have fixed it - next should be individual ablation studies.
<reportFormat>brief</reportFormat>
<shutdown>kill</shutdown>


Fine-tune the graphs

This covers different things:

ensuring correct data on the graphs
maybe manually filtering out some 'strange' flukso or sensorids
graphical pimping of the graphs

Manual filtering is implemented.  Sensor categorization is explicitly addressed in #54 so I close this issue.


fix rtl repo, typos, set off if install fails

fixes: https://github.com/raspiblitz/raspiblitz/issues/4388

Tested on RPi5 with CLN, RTL install works well now.


postgres connection support

I've built from source with --features "database" but running :
./roapi --table "postgres://
Returns :
{ source: Some(TableError { source: Extension { msg: "unsupported extension in uri: postgres://postgres:dune@localhost:5432/postgres" } })
I do see postgres in https://github.com/roapi/roapi/blob/ea01ffe0d3978af0bb24065e711431bc52e6c96d/columnq/src/table/mod.rs#L294 though
Does ROAPI support connecting to a postgres database ?

@jonathandune  ROAPI supports Postgres. The error you encountered is because your schema is not correct. Change the URL to postgresql://postgres:dune@localhost:5432/postgres.

Thanks, indeed just realised this by looking at https://github.com/roapi/roapi/blob/ea01ffe0d3978af0bb24065e711431bc52e6c96d/columnq/src/table/mod.rs#L461
I am know getting a panic :
./roapi --table "postgresql://postgres:dune@localhost:5432/postgres"
[2023-10-17T08:21:53Z INFO  roapi::context] loading uri(postgresql://postgres:dune@localhost:5432/postgres) as table postgres
thread 'main' panicked at 'Cannot start a runtime from within a runtime. This happens because a function (like block_on) attempted to block the current thread while the thread is being used to drive asynchronous tasks.', /Users/jonathan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/postgres-0.19.7/src/connection.rs:66:22
stack backtrace:
0:        0x1060daef0 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h782e083efa4700b5
1:        0x104dfb7f8 - core::fmt::write::hdcd44492977669f9
2:        0x1060c8ba4 - std::io::Write::write_fmt::h7f32f27b72970ad5
3:        0x1060dad10 - std::sys_common::backtrace::print::h6868989d298a5d93
4:        0x1060c9a4c - std::panicking::default_hook::{{closure}}::he58f0ab435f26824
5:        0x1060c9718 - std::panicking::default_hook::h0f0ccb5a1a0c4466
6:        0x1060ca34c - std::panicking::rust_panic_with_hook::hf392a622566eb9b6
7:        0x1060db46c - std::panicking::begin_panic_handler::{{closure}}::h07b7cf65243520e5
8:        0x1060db400 - std::sys_common::backtrace::__rust_end_short_backtrace::h4972088065ba8a0d
9:        0x1060c9f48 - _rust_begin_unwind
10:        0x1064736f0 - core::panicking::panic_fmt::hc67fbbe16e2d8abd
11:        0x105bc96c8 - postgres::client::Client::simple_query::h60c2e9dc0ad79aad
12:        0x104d6c134 - r2d2::Pool::get::h187342ef99e68157
13:        0x104d1c0b0 - <connectorx::sources::postgres::PostgresSource<P,C> as connectorx::sources::Source>::fetch_metadata::h11cfb7f0932e2cee
14:        0x104da7b9c - connectorx::dispatcher::Dispatcher<S,D,TP>::run::hb7e68ff2f5a3d964
15:        0x104d99908 - connectorx::get_arrow::get_arrow::he4d30bdaeb7f87e2
16:        0x104cf8824 - columnq::table::database::imp::::to_mem_table::he6f7249e7cbcfa3d
17:        0x104686090 - columnq::table::load::{{closure}}::h551117c6361446a1
18:        0x104697c70 - columnq::columnq::ColumnQ::load_table::{{closure}}::h084c8f95586f07c2
19:        0x1045ea32c - roapi::startup::Application::build::{{closure}}::hd413c2f4aaff7301
20:        0x104709e2c - roapi::main::{{closure}}::h8001bfde547176a0
21:        0x10470608c - roapi::main::h88b9d870bf2f24d6
22:        0x10453372c - std::sys_common::backtrace::__rust_begin_short_backtrace::h87db7e2494fcaf82
23:        0x104535ea0 - std::rt::lang_start::{{closure}}::he3e2d9a6e41ef503
24:        0x10470ccc8 - _main
The DB is up and running and have other processes connected to it so not sure what the issue is.
Is this due to Tokio ?

This error arises from crate (postgres):
/// Opens a connection to a PostgreSQL database.
    pub fn connect<T>(&self, tls: T) -> Result<Client, Error>
    where
        T: MakeTlsConnect<Socket> + 'static + Send,
        T::TlsConnect: Send,
        T::Stream: Send,
        <T::TlsConnect as TlsConnect<Socket>>::Future: Send,
    {
        let runtime = runtime::Builder::new_current_thread()
            .enable_all()
            .build()
            .unwrap(); // FIXME don't unwrap

        let (client, connection) = runtime.block_on(self.config.connect(tls))?;

        let connection = Connection::new(runtime, connection, self.notice_callback.clone());
        Ok(Client::new(connection, client))
    }

We can see that the code would create a new Runtime, but our own program is already in a Runtime, so that causes this error.
None of those are part of ROAPI, so I have no idea how to fix this now. :(

It looks like we are using the wrong connectorx api, we are effectively calling a blocking api  from an async function, would be better to call the async self.config.connect(tls) method directly from roapi.

It looks like we are using the wrong connectorx api, we are effectively calling a blocking api from an async function, would be better to call the async self.config.connect(tls) method directly from roapi.

Maybe we should call the connectorx api in an independent thread?
pub async fn to_mem_table(
            &self,
            t: &TableSource,
        ) -> Result<datafusion::datasource::MemTable, ColumnQError> {
            debug!("loading database table data...");
            let queries = CXQuery::naked(format!("SELECT * FROM {}", t.name));
            let source = SourceConn::try_from(t.get_uri_str())
                .map_err(|e| ColumnQError::Database(e.to_string()))?;
            let hd = tokio::runtime::Handle::current();
            hd.spawn_blocking(move || {
                let destination = connectorx::get_arrow::get_arrow(&source, None, &[queries])
                    .map_err(|e| ColumnQError::Database(e.to_string()))?;
                Ok(datafusion::datasource::MemTable::try_new(
                    destination.arrow_schema(),
                    vec![destination
                        .arrow()
                        .map_err(|e| ColumnQError::Database(e.to_string()))?],
                )?)
            })
            .await
            .map_err(|e| ColumnQError::Database(e.to_string()))?
        }

@holicc that feels wrong, do they not have an async api we can call directly from an async context?

Hi there,
Any news concerning the Postgresql connector ?

@anvarknian i am not actively working on this right now, I am ok with merging a workaround to unblock this in the short run, but the proper way to fix this is to invoke an async api from their end. Have you filed an upstream connectorx ticket to get some ideas on how to best achieve this?

@houqp , Hi, I just wanted to confirm that using postgres from roapi is still not advisable?  On a side note, in order to enable connecting to a database in the first place, do I need to compile from source with --features "database" as indicated in the first post on this thread?
Currently, when I try to connect to a postgres database with roapi version 0.11.1, I get the error below:
thread 'main' panicked at /private/tmp/roapi-20240325-5739-76oqyr/roapi-roapi-v0.11.1/roapi/src/startup.rs:63:14:
Failed to create Roapi context: Whatever { source: Some(TableError { source: InvalidUri { msg: "failed to register `uri(postgresql://roapi:sqlapi@<IP_ADDRESS>:5432/roapi)` as table `user`, unsupported table format `postgres`" } }), message: "Failed to load table", backtrace: Backtrace [{ fn: "std::backtrace::Backtrace::create" }, { fn: "roapi::startup::Application::build::{{closure}}" }, { fn: "roapi::main::{{closure}}" }, { fn: "roapi::main" }, { fn: "std::sys_common::backtrace::__rust_begin_short_backtrace" }, { fn: "_main" }] }

Thanks for any clarity you can provide.
Regards,
Chris

@chrisfw yes, you need to compile from source for now, the official build doesn't have the mysql database feature enabled by default.

Thanks for your response @houqp.  The same holds true for postgres, correct?  Can you please provide the exact flag and value that need to be passed?  (--features "postgres" ?)   Also, is the postgres support stable as it seemed there were some connector-x threading issues?
Regards,
Chris


Class broken when containing file is updated (Cannot convert the "ReproType" value of type "ReproType" to type "ReproType")

Just like re-executing Add-Type for a snippet of C# code that has not changed, reloading a module that uses classes should also continue to work--even if a script file that contains the [unmodified] class has changed.
(I felt this problem was covered by #14250, but the owner closed it, and this problem is still irritating us, to the point we might just rip out all usage of PowerShell classes.)
This does not only come up during development scenarios. For example, I have a module that auto-updates itself, and then instructs the end user to run "Import-Module [...] -Force" to pick up the changes. (And then I get complaints from users when the module breaks in a really nonsensical-looking way.)
The key to the repro below is touching the file where the class is defined (before re-importing the module). You don't have to change the class definition; just add some whitespace somewhere else in the file. Which makes me think that the logic out there for deciding whether to generate a new runtime type for the class is making its decision based on whether the containing file has changed or not, but could do better by looking at just the extent where the class is defined.
Steps to reproduce
mkdir Repro_gh_14250 | Out-Null
pushd Repro_gh_14250
New-ModuleManifest -Path .\Repro.psd1 -RootModule Repro.psm1

@'
$script:data = [ordered] @{
    First = @("Primo", 123)
    Second = @("Secundo", 456)
}

class ReproType
{
    [string] $Name
    [int] $Number
}

$TheList = @()
foreach( $key in $script:data.Keys )
{
    $newThing = [ReproType]::new()
    $newThing.Name = $script:data[ $key ][ 0 ]
    $newThing.Number = $script:data[ $key ][ 1 ]

    $TheList += $newThing
}

function Get-TheList
{
    [CmdletBinding()]
    param()

    process { return $TheList }
}
'@ > Stuff1.ps1

@'
function Show-TheList
{
    [CmdletBinding()]
    param( [ReproType[]] $TheItems )

    process
    {
        foreach( $item in $TheItems )
        {
            Write-Host $item.Name -Fore Cyan -NoNewline
            Write-Host ": " -NoNewline
            Write-Host $item.Number -Fore Yellow
        }
    }
}

function Repro
{
    [CmdletBinding()]
    param()

    $stuff = @( Get-TheList )
    Show-TheList $stuff
}
'@ > Stuff2.ps1

@'

. "$PSScriptRoot\Stuff1.ps1"
. "$PSScriptRoot\Stuff2.ps1"

'@ > Repro.psm1

ipmo .\Repro.psd1 -Force
Repro
'# nothing' >> .\Stuff1.ps1
ipmo .\Repro.psd1 -Force
Repro    # EXPECTED: it works. ACTUAL: Cannot process argument transformation on parameter 'TheItems'. Cannot convert the "ReproType" value of type "ReproType" to type "ReproType".

Expected behavior
No problems after reloading the module.
Actual behavior
After reloading the module, I get errors when using the [unmodified] class:
Cannot process argument transformation on parameter 'TheItems'. Cannot convert the "ReproType" value of type "ReproType" to type "ReproType"

Environment data
Name                           Value
----                           -----
PSVersion                      7.2.0-preview.3
PSEdition                      Core
GitCommitId                    7.2.0-preview.3
OS                             Microsoft Windows 10.0.20293
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Just like re-executing Add-Type for a snippet of C# code that has not changed reloading a module that uses classes should also continue to work--even if a script file that contains the [unmodified] class has changed.

A couple of questions on the expectations of the change:

Which SessionState (module) would you expect it to be bound to?
Would you expect all static properties to remain unchanged or would you expect them reinitialized to default and for static constructors to be reinvoked?
If you have a static constructor that changes module state, would you expect the new module instance to reflect those changes?

Which SessionState (module) would you expect it to be bound to?

The new one. (The old version of the module is effectively gone, replaced by force-importing the updated version.)

Would you expect all static properties to remain unchanged or would you expect them reinitialized to default and for static constructors to be reinvoked?

Reinitialized to default; static constructors re-invoked.

If you have a static constructor that changes module state, would you expect the new module instance to reflect those changes?

Yes.
In other words, I expect force-importing the module to be [nearly*] equivalent to just importing that version of the module in a session where no previous version had been loaded. (*of course that does not apply to global session state)

Just like re-executing Add-Type for a snippet of C# code

I should add that yes, I understand my expectations for C# code are not quite the same as for PowerShell classes. I think this is simply fallout from the fact that C# classes added via Add-Type are not tied to session state, but are necessarily global. This is due to limitations of .NET (can't unload the type when a module is unloaded, for example).
Because C# classes are effectively global state, that means they continue to work just fine when the module is reloaded. It is true that static constructors are not re-run when the module is re-imported, so if I were storing state in C# static members, I would have to understand that these would be tied to global session state, not module (and therefore state from before the reload could be around afterward). But for PowerShell classes (a first-class, built-in thing), if they are going to be tied to the module, then they should be tied to the module, and continue to work, even when the module is reloaded.

But for PowerShell classes (a first-class, built-in thing), if they are going to be tied to the module, then they should be tied to the module, and continue to work, even when the module is reloaded.

I should clarify that they do work when reloaded, the difference is largely in design.  using module is a parse time concept where Import-Module is a runtime concept.  It's possible that this could be made to work in very specific situations by reusing the type definition and manually reinitializing everything only when Import-Module -Force is used.
That's a pretty big work item with a lot of risk but I can't quickly see why it'd be impossible.  Though the situations where that is feasible would be pretty limited, so for a user outside of this thread it would be challenging to figure out why it's working sometimes and failing others.

I should clarify that they do work when reloaded

Sorry, I don't mean to be obtuse, but: the point of this Issue is that they demonstrably do NOT work when reloaded (see the repro steps). I am not sure where our miscommunication is... can you please clarify?

Sorry, I don't mean to be obtuse, but: the point of this Issue is that they demonstrably do NOT work when reloaded (see the repro steps). I am not sure where our miscommunication is... can you please clarify?

Ahh I see, this is different form the issue you linked as none of it is exposed outside the module.  Type is probably cached on the CommandInfo and isn't correctly invalidated. Outside of function signatures (parameter types, attributes, etc) it'll work with forced reloads though.
I went ahead and hid my other comments as outdated, thanks for the correction.


Missing information about OTP versions

Missed info about minimum OTP versions, and which OTP versions are supported. It can be add to README and/or rebar.config and also proof in GitHub actions in matrix level.

Fixed, added build dependencies to the README.
As soon as there exist tests we'll now if this is true
also when running the code.

Cool! There was missed some point - it need to use % for comments in Erlang code block - fixed at https://github.com/etnt/etacacs_plus/pull/9

Oh.. wrong ticket - but for this issue was added minimum OTP version in rebar config level


SDK-1690: Test Coverage

Added

Additional test coverage
SonarCloud badges to README

Changed

AgeVerification{}.New() changed to NewAgeVerification()
Reusing parseEncryptedProto() for application profile

Removed

Go master from Travis (we already allowed failures for this and it was taking 4 times longer than stable releases)

Oh also, was going to change all the fmt.Fprintf(os.Stdout, "error: %s", err.Error()) to fmt.Printf("error: %s", err.Error()), but I'll do that in a separate PR


class results scraper

Describe the problem:
The format of the course template output that is scraped is not standardized. Some integer outputs are stored as strings.
Expected behavior
For any field that is a number, convert it to an integer.
name: 'calculus II'
id: 1020
credits: 4

Fixed in #131 , closing issue


[size limit]: enforce size limit on external events

Reject large signal, activity input/output, workflow input/output.
Need to figure out what is a proper up limit.

This is supported through dynamic config now.


Add node examples to the documentation

The documentation should have examples based on this library.

Issue already opened here https://github.com/blockcypher/docs/issues/36.


Fix mis-typed onpublished hook, update version, fanpool defaults

Converts second OnPublish hook to OnPublished, and adds an additional OnPublished when returning from qos=0 publishes, as per #118
Adjusts Fanpool default values to be slightly more performant, as per #117
Updates version number

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.325%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4991

Relevant Lines:
5076

💛  - Coveralls


zstd: Make sure huf_decompress is only applied to 64-bit x86.

Fixes #96295

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 4.3.1.


Deferred credential provider version 2

@fabianfett This is an alternative to your version that takes a closure. I have tried to keep all the code in the same place so they are easy to compare.
Setup interface to be the same as RotatingCredentialProvider. ie init(eventLoop: EventLoop, client: CredentialProvider)
Moved the StaticCredential+Shared extension into a ConfigFileCredentialProvider struct.
Fixed up tests
Added test to verify the client credential provider code is only called once.

In theory we could just use RotatingCredentialProvider. The DeferredCredentialProvider doesn't need to check if credentials are expiring. A minor optimisation


If you pass a non-integer to BrowserWindow.setPosition it gives confusing error

browserWindow.setPosition(0.5, 0.5);

results in
[TypeError: Error processing argument -1.]

it would be nice if it was more informative e.g.
[TypeError: Argument must be an integer.]

or it just coped with it..

Related https://github.com/atom/electron/issues/4307 and has been fixed on master. Also the reason for not showing Argument must be an integer is because we build with no-rtti. More info here

Ok, will retry after next release. Closing to reduce noise. Thanks for info!


Remove classes/zip.class.php

The classes/zip.class.php was written back before Gazelle was using PHP 5 and it was necessary. However, it now basically just mimics the builtin Zip class which we should switch to using instead.

Have a look at https://github.com/maennchen/ZipStream-PHP
We used this to allow the creation of arbitrarily large zip files without exhausting memory on the server.


[SPARK-41483] Last metrics system report should have a timeout, avoid to lead shutdown hook timeout

What changes were proposed in this pull request?

In the stop method of SparkContext, a timeout limit is added to metricsSystem#report. When it is not completed for more than 1s, continue with the subsequent steps.
Why are the changes needed?

When the user does not explicitly call the stop method of SparkContext, the stop will be executed by ShutdownHook, and the Hadoop shutdown hookmanager used by Spark needs to complete the hook entry within 10s, so when the metricsSystem takes a long time to complete the report (for example, the sink occurs time out), other hooks may be blocked and cannot be executed successfully.
For example, the unregister operation in ApplicationMaster may fail to execute because the metrics report costs too much time, so the Spark job has been successfully completed, but Yarn still considers it a failed job.
Does this PR introduce any user-facing change?

No
How was this patch tested?

test manually, due to the difficulty of constructing the timeout execution of metrics#report. Unit test want help!

@packyan mind creating a JIRA and linking it to PR title? See also https://spark.apache.org/contributing.html

@packyan mind creating a JIRA and linking it to PR title? See also https://spark.apache.org/contributing.html

Sorry, I will do it later.

Can one of the admins verify this patch?

Any one can give me some suggestions?


vsock: process OP_RST on stream connections

OP_RST may be received on reverse TCP connections if the actual endpoint (the userspace application) has stopped listening, so process them and release the proxy managing the connection.
Fixes: #116

@blenessy Could you please give this one a try?

Superseded by #120


New NICs are comming soon, one for hypervisor (as in #9): create LAGs.

In the past half year, I had 2 network issues, both related to e1000 kernel module, which means I'd lost "just" remote control, messages network (DNS, MQ), storage network (Ceph) and tunnel interface network (Neutron + GRE). I've bought more 4 NICs, and I'll move from 2 isolated NICs (em1 and p5p1) to a single LAG.

Commit 630f1a2658093377da04a86831f835181f601322: NIC bonding for all physical interfaces, and only tagged traffic allowed.


I dont understand what is breaking here?

Please tick all applicable boxes.

[x] I am using Python 3.5.3 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x] The latest master version (release-260419)
[ ] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
Cannot run code:
iMac:MusicBot smackintosh$ ./run.sh
./run.sh: line 7: declare: -A: invalid option
declare: usage: declare [-afFirtx] [-p] [name[=value] ...]
./run.sh: line 8: declare: -A: invalid option
declare: usage: declare [-afFirtx] [-p] [name[=value] ...]
./run.sh: line 9: python3.5: command not found
./run.sh: line 10: python3.6: command not found
./run.sh: line 11: python3.7: command not found
[INFO] launcher: Starting sanity checks
[INFO] launcher: Checking for Python 3.5+
[INFO] launcher: Checking console encoding
[INFO] launcher: Ensuring we're in the right environment
[INFO] launcher: Required checks passed.
[INFO] launcher: Optional checks passed.
[INFO] launcher: Moving old musicbot log
Traceback (most recent call last):
File "run.py", line 438, in
main()
File "run.py", line 347, in main
finalize_logging()
File "run.py", line 179, in finalize_logging
dlh.setFormatter(logging.Formatter('.'))
File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/logging/init.py", line 576, in init
self._style.validate()
File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/logging/init.py", line 429, in validate
raise ValueError("Invalid format '%s' for '%s' style" % (self._fmt, self.default_format[0]))
ValueError: Invalid format '.' for '%' style
Steps to reproduce

Running script normally
Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
musicbot.log

Help! I'm getting an error! Invalid format '.' for '%' style
This is a result of using python 3.8, please either downgrade to a version below that such as 3.7.x, switch to review, or switch to team-jsb.
How do I switch to review?
If you have already cloned MusicBot master branch, simply navigate to your MusicBot folder through a terminal, or even easier, go to your MusicBot folder, and click open git bash here. Once there you can type git checkout review.
Don't have MusicBot yet? No worries.
use git clone https://github.com/Just-Some-Bots/MusicBot.git ~/MusicBot -b review
How do I use team-jsb?
The process of doing this is very easy, surprisingly! The commands listed below will set the git repo of the bot to the Team-JSB branch, and pull the latest version.
git remote set-url origin https://github.com/Team-JSB/MusicBot.git
Then do a git pull.


Nail down relationship to hellorust.com

The http://www.hellorust.com/ site is at attempt to gather news and resources related to Rust and wasm. Right now, this repo points to a handful of pages there.
It'd be good to pin down the roles for each, and ensure that between the two we can keep exhaustive early tracking of e.g. Rust-to-wasm projects.

While this didn't quite happen I think we're in an alright place today, so closing.


Typo in smdx_00005.xml "andatory"

See https://github.com/sunspec/models/blob/master/smdx/smdx_00005.xml#L98 - there's "andatory" instead of "mandatory".

same in 00006

Issue resolved with pull request.


Unity: iOS Support

As iOS libraries are statically linked in Unity, the DLLImport attributes might need to be configurable.
See unity docs example for native libraries. Of course, this would not be done with a weird UNITY_IPHONE flag, but with some other generic mechanism.
Any ideas? I am willing to contribute, but need some hints. How could we approach this?

Note: As wasm3 is the only wasm library that does not require the use of System.Reflection.Emit, it is the only option for running wasm on iOS. Therefore, Wasm3DotNet should probably have it as a goal to support iOS.
Disclaimer: I did not actually test whether it is currently already possible to run on iOS, just assuming the unity docs are still valid.

Inside of Unity, we cannot pass any flags to the managed dll. But, we can define constraints that can control which dll is selected in which platform. So maybe we can have two different managed dlls, one for static linking and one for dynamic linking, and then in Unity choose the static linking variant on iOS?

Since I have no knowledge about iOS development, I cannot help with actual programming on this issue.
However, supporting iOS might be possible with the three steps:

Creating a static library (.a) equivalent of wasm3_dll.
Adding #if UNITY_IOS conditional compilation in NativeFunctions.cs.
Distributing Unity version of Wasm3DotNet as C# sources, instead of managed DLL.

Sorry for forgetting this issue for long time.

Don't worry, it's fine :) due to iOS being very restrictive about code execution, this will probably not get into to AppStore either way. They only allow code downloaded and executed by their WebKit view.... which might include web assembly, but definitely excludes wasm3


MyPet & Tamed Pet Pay Permission Not Working

Players with permission jobs.petpay.100 for ex. do not get money when their MyPet kills a MythicMob/Vanilla Mob
This also goes for Tamed Pets given the permission jobs.petpay players do not gain money when their tamed pet kills a MythicMob or Vanilla mob.
Jobs version: <IP_ADDRESS>
Server Type: Purpur
Server Version (using /ver): Current: git-Purpur-1623 (MC: 1.18.2)*
Relevant plugins: Latest MyPet release

3a2b60db896d1e3c866fa4f70dcd10fb9216e161


Add groupby to itertools

I really like your package! I have implemented the groupby method on my side, I might actually do a PR at some point, but until then you can just reuse this if you want to (i.e. and type, test and document).
const groupby = function* groupby(iterable, key = x => x) {
  const it = iter(iterable);

  let currentValue;
  let currentKey = {};
  let targetKey = currentKey;

  const grouper = function* grouper(tgtKey) {
    while (currentKey === tgtKey) {
      yield currentValue;

      const { done, value } = it.next();
      currentValue = value;
      if (done) return;
      currentKey = key(currentValue);
    }
  };

  for (;;) {
    while (currentKey === targetKey) {
      const { done, value } = it.next();
      currentValue = value;
      if (done) return;
      currentKey = key(currentValue);
    }

    targetKey = currentKey;
    yield [currentKey, grouper(targetKey)];
  }
};

console.log([...map(groupby('aaabbbbcddddAA'), ([k, v]) => [k, [...v].length])]);
/* [
  [ "a", 3 ],
  [ "b", 4 ],
  [ "c", 1 ],
  [ "d", 4 ],
  [ "A", 2 ]
] */

Hi @sgenoud – that's really cool! I'd love to include this in itertools. If you want to make a PR, make sure to annotate the Flow type correctly for this function, and also add unit tests. Thanks!

(Closing since this isn't an issue.)


Change default Affiliation

Change the default affiliation when users are signing up for an account from Code for Charlotte to blank.

Fixed in e2540f4cebf16e62cf0a6312e05b74f869ccc01b


Can we use geobuf?

I discovered this code to reduce GeoJSON to about 3% of original size (in a use-case I learned about locally from 180mb to less than 2mb) . Could you tell me if this is usable with Tangram? My load times aren't too painful but, this kind of speed boost is significant and would be appreciated by users I'm sure.
https://github.com/mapbox/geobuf
Thanks!

Geobuf is designed for lossless encoding of geojson. Given that tangram will be reading tiles, existing binary formats like mvt seem to make more sense, or topojson.

Thanks @pnorman. My interest, which I should have made more clear, is in large file sizes of points of interest. My file size is typically 20+MB, which if I could cut down to 1MB or less that would be great. I understand topojson and mvt to be more for polygons no? Thanks again.

mvt can store points. The big difference is going to be tiling vs not. geojson to geobuf has about a 2.5:1 compression ratio (when transferred with http and gzip encoding), so if you have files that big, you'll want to tile with either, and when using tiled data, mvt is an option already.
For only points, topojson doesn't offer any real advantages afaik.

Ok cool. Thanks @pnorman


Add "traceSampled" to MDC in the Slf4JEventListener

This resolves https://github.com/micrometer-metrics/tracing/issues/755
As mentioned this is useful for filtering by looks which will also appear in the tracing tool.
API is backwards compatible and tests are added.
Put "since 1.4.0" on the new constructor, assuming this will be part of next minor release. Happy to change that.

Closing this as the maintainers saying they are working on a broader related feature set:

The behavior of not adding the sampling flag to the MDC dates back to Sleuth. It used to add it out of the box but it was removed I think because of performance reasons. With Brave, it should be possible to register a “customizer” where you can add any fields you want to the MDC from the current span. We think having the sampling flag in the logs is a useful feature and we are going to look into doing something similar to the Brave “customizer” in the OTel bridge so that we can cover a more general use-case and users can decide what they want to add to the MDC.

If anyone runs into this problem you can extend Slf4JEventListener with this functionality and add it as a bean.


Replaced question.md issue-template with contact_links (Fix config.yml)

Followup to #4912

Hrm strange. Still not seeing the contact-link in the new-issue-template:
https://github.com/NLog/NLog/issues/new/choose
But it did remove the create-blank-issue option.

Created #4921 with one more attempt to make GitHub happy.


Enable alternate stylesheet feature.

Now that the feature is in master, maybe we should enable it?
Are there any rules or checks that have to be done?

Generally Fyrd tries to fill in most of the Unknown datapoints before enabling features.

Yup, @cvrebert is correct. Eventually I may get around to filling in the blanks myself, but since this is not a much-requested feature it is not a priority.


Drop Debian 8 support and update default template

The primary motivation was to update the /etc/default-isc-dhcp-server file to the new template introduced in Debian 9. Debian 8 is EOL so there's no point in trying to support the old format.

EL6 failures should be fixed in https://github.com/theforeman/puppet-dhcp/pull/198.


initiating [short] protocol field fix.

In some earlier bugfix it turned out that the [short] noted protocol field has serialized with different semantic in the driver (according to native_protocol_v3.spec). The protocol sometimes interpretes it as signed integer (mainly to denote null value), however it can be identified in the driver as well.

Negative values for null and unset are handled in the driver. See FrameWriter.WriteBytes(), FrameReader and OutputRows for example.
In the case you mention in this pull request, it is not possible to send a batch with a negative value of statements to represent no items, and _requests.Count is always positive.
Remember that you can use JIRA to post issues or the mailing list for discussions.

yes, you are right, what I wanted to solve regarding to my earlier #143/144 request is in case of batch count and list count those fields are interpreted as insigned and not as signed so I just want to keep preciseity, I mean form the servers point of view 32768 is a valid count for collection types and for batch statement count, however according to your request I changed it back to shourt.MaxValue. I've tested in the phase when it was on my machine an ushort (however because of the bit compatibility it doesn't matter), and the server was able to handle up to 65535 batch statement, and up to 65535 element of collection type. What I wanted to reach, that driver should provade the same limits as the server, and let users/developers choose to use it or not, depending on what load is put on the server cluster.


Issue with English Docs

First of all, Thank you so much for this amazing theme.
I was trying to use this theme as a git submodule. But a minor typo is creating issues.
The command as written in the docs is: git submodule add https://github.com/isunjn/serene.git theme/serene
But the official docs of Zola instructs to install a theme in the themes directory.
So the correct command to use the theme as a git submodule should be: git submodule add https://github.com/isunjn/serene.git themes/serene

Fixed, Thank you!


[AIRFLOW-189] Highlighting of Parent/Child nodes in Graphs

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-189

Reminders for contributors:

Your PR's title must reference an issue on Airflow's JIRA. For example, a PR called "[AIRFLOW-1] My Amazing PR" would close JIRA issue #1. Please open a new issue if required!
You must add an Apache License header to all new files.
Please squash your commits when possible and follow the 7 rules of good Git commits.

I am +1, but I do like @r39132 to take a look at it due to the d3 upgrade.

Updated to avoid upgrading dagre-d3


Remove blueprint

As title. Blueprint feature is no longer needed.
A followup PR will be create to drop the table in schema repo.
similar work in API https://github.com/ManageIQ/manageiq-api/pull/139

@miq-bot add_label technical debt
@miq-bot assign @gmcculloug


Remove MountPoint.Named and replace it with IsNamed()

What I did
I noticed that Named is not set by any of the Parse functions, so it's possible that the value is incorrect if it gets accessed before something sets it. I removed Named and replaced it with IsNamed() which checks for a name.

This defeats the purpose of the field.
All volumes get named, this field is currently specifically used for specifying that a mountpoint was named like so -v name:/foo rather than -v /foo.
This is checked when cleaning up volumes for containers removed with docker rm -v.

Oh, it's the MountPoints in sections 1 and 2 that wont have named set?


Chore/add prettier as linter

Hi, while working on this, I found there are a lot of mismatch in coding style, probably coming from other contributors.
so I added a .prettierrc to make it easier to get everyone on the same style.
I fully understand if you don't want to use this though, because it adds a lot of code to review

@dargmuesli should be fixed now. I also changed to singlequote:false because it seems to be the prefered style and added all other files as well. I will enable issues on my repo though should there be anything else. I plan to add at least dotenv support and splitting this up in microservices.

@dargmuesli should be fixed now. I also changed to singlequote:false because it seems to be the prefered style and added all other files as well. I will enable issues on my repo though should there be anything else. I plan to add at least dotenv support and splitting this up in microservices.


gtk3: 3.22.24 -> 3.22.26

Motivation for this change
Changelog

The HighContrast theme has received numerous improvements

Middle-click on scrollbars jumps to the position again

Wayland:

Terminal sizing problems are fixed
The Wayland backend now implemented the KDE server-side decorations protocol. Eleminates need of 3rd party solutions like:
https://aur.archlinux.org/packages/?O=0&SeB=nd&K=gtk+csd&outdated=&SB=n&SO=a&PP=50&do_Search=Go

X11:

Problems with menus on several X servers are fixed

Lot of bug fixes
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

/cc @peterhoeg

The Wayland backend now implemented the KDE server-side decorations protocol

This removes client side window decorations under Wayland compositors which draw own decorations, as I understand compositor should support this protocol - Sway WM is one of examples where it works.

This looks good, but I think it should be merged into staging.

@disassembler Done, thanks!


Add temporary escape hatch for customization wheel generation?

I am trying to port my project pyhepmc https://github.com/scikit-hep/pyhepmc/pull/57 to meson/meson-python.
I am building a Python extension module and a library (a dependency). I compile the library myself. What I would like to do is to compile the library as a shared library and ship it with the Python module. The library has to be a shared library, not a static library or compiled directly into my app, to respect the LGPL license (my project is released under a less restrictive license than the library). meson-python informs me that this is only supported for Linux at the moment.
        File<EMAIL_ADDRESS>line 561, in build
          assert platform.system() == 'Linux', 'Bundling libraries in wheel is currently only supported in POSIX!'

I looked into the source code and the reason for this seems to be that there are no tools implemented yet to change RPATH on MacOS and Windows.
I am willing to live with a workaround until this is supported officially. Is it possible to implement an escape hatch as a temporary solution which allows me to put the shared library into the Python extension directory and allow me to run a custom command to change the RPATH in the Python extension module?
I would like to control the location where the library is installed, because I think that the RPATH in the Python extension module may actually support loading the library from the same directory where the extension module is located. What I have in mind is a undocumented configuration setting, that allows me to specify the location of the library and another setting to run a custom python script on the python extension module. Setting this should bypass the assert, too.
Or is there maybe another solution to my problem? I tried a few workarounds. I got pretty far with integrating the library via <dependency>.as_link_whole(), which works on Linux and MacOS, but fails on Windows. This still violates the LGPL, though, so it is not a proper solution.
Link to my meson.build https://github.com/scikit-hep/pyhepmc/blob/3d5372824e08e1cd4317eda6dd924ea649d68984/meson.build

For Windows and Python>3.8 can you use os.add_dll_directory?

For Windows and Python>3.8 can you use os.add_dll_directory?

We're doing something similar to this for SciPy, to ship libopenblas.dll inside the SciPy wheel. It's a little fiddly, but this is pretty robust by now: https://github.com/scipy/scipy/blob/main/tools/openblas_support.py#L199-L236. That _distributor_init.py is generated for wheel builds to be uploaded to PyPI only, and imported in scipy/__init__.py. Wheels are generated with cibuildwheel: https://github.com/scipy/scipy/blob/main/tools/wheels/cibw_before_build_win.sh

I looked into the source code and the reason for this seems to be that there are no tools implemented yet to change RPATH on MacOS

For macOS you can use delocate for this job, it works just like auditwheel.
tl;dr shipping shared libraries like this works on all platforms, but meson-python provides builtin support only on Linux, and you need a few more hacky tools on macOS/Windows.

gh-260 adds support for macOS.

@rgommers Thank you for the comprehensive reply! I used scipy as a reference already for pyhepmc, the build scripts are sophisticated and well documented. I found anwers to most of my obstacles there. I missed the openblas support that you pointed out, thank you, I will look into it.

https://github.com/mesonbuild/meson-python/pull/260 adds support for macOS.

Excellent!

@HDembinski with #260 being merged, is this still relevant for your use-case?
If so (or if anyone else in the future is reading this issue), achieving wheel customization can be done by creating a custom backend. Your custom backend would call meson-python, and then after getting the wheel, you are free to modify it, before passing it to the final user.

pyproject.toml

[build-system]
build-backend = 'my_custom_backend'
backend-path = ['.']

my_custom_backend.py

from typing import Any, Dict, Optional

import mesonpy

def build_wheel(
    wheel_directory: str,
    config_settings: Optional[Dict[Any, Any]] = None,
    metadata_directory: Optional[str] = None,
) -> str:
    # do something before
    wheel = mesonpy.build_wheel(wheel_directory, config_settings, metadata_directory)
    # do something with the wheel
    return wheel

def __getattr__(name: str) -> Any:
    return getattr(mesonpy, name)

For more info about module-level __getattr__ see PEP 562
As such, I am gonna close this, but feel free to re-open if there's anything else you think I failed to consider, and please let me know if you have any further questions. Thanks!

Just noting this down for the future: this could be made a guide or tutorial for the docs.

This looks great and is exactly what I had in mind, thanks!
This whole customization become obsolete once support for embedding shared libs on Windows is also added. I haven't followed the development for a while, so I am not sure whether that has landed already.


Internal Server Error accessing my profile on try.gitea.io

Description
If I go to https://try.gitea.io/strk, logging in via github OAuth, I get a 500
Gitea Version
Gitea Version: 1.17.0+dev-736-g719eb4a87
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Only on try.gitea.io
Database
No response

Looking at the logs it appears that this is related to GetActiveStopwatch #20008
router.log:2022/06/17 17:59:08 [62acc0ec] router: completed GET /strk for <IP_ADDRESS>:38034, 500 Internal Server Error in 3.2ms @ repo/issue_stopwatch.go:81(repo.GetActiveStopwatch)

Still happens as of 1.17.0+dev-738-gea9997a9d

The fixing commit is:
v1.17.0-dev-741-gae446b13f

I also have 500 error when authorizing but could be different, probably related to 2FA

I'd need to know when you tried to do this so we could check the logs.

Closed as I cannot reproduce it in the try site now. Fell free to reopen it if it occurs again.


RescueTime doesn't work due to SSL CA certificates

Issue description
When rescuetime is launched, it fails to check the status. The error message is:
Problem with network, please contact RescueTime Support. (77): Problem with reading the SSL CA cert (path? access rights?).

Steps to reproduce
$ nix-env -iA rescuetime
$ rescuetime

Technical details

Nixpkgs version: Git revision fa3062d1ef7802fc38ecb6a5b1e2748620c8b860

strace -f -eopen and a bunch of filtering shows:
open("/home/timbeaudet/rescuetime-dependencies/openssl/openssl.cnf", O_RDONLY) = -1 ENOENT (No such file or directory)

Which looks suspcious (I am not timbeaudet!)

Ok, that was a red herring. It seems that RescueTime works if you have /usr/share/rescuetime/curl-ca-bundle.crt, but I'm not sure how to best fix this problem.

Patch it to find certificates the same as most of other nixpkgs? IIRC it's /etc/ssl/certs/ca-certificates.crt and overridable by $SSL_CERT_FILE.

Oh, I didn't notice it's a binary package.

@ocharles did you fix this? Of just symlink the crt file?

So far I have created the sym link from /usr that Rescue Time is expecting.
On 14 Oct 2016 4:42 a.m., "Ben Ford"<EMAIL_ADDRESS>wrote:

@ocharles https://github.com/ocharles did you fix this? Of just symlink
the crt file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/issues/18314#issuecomment-253742649,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABRjliVbZEDnMl65BNp6c7ZdRplYp7wks5qz0BvgaJpZM4J0_PO
.

I ran into this same issue, and the above didn't fix it. The below did:
Found this in the strace:
[dnelson@sulaco ~]$ strace /usr/bin/rescuetime |& grep crt
open("/etc/ssl/certs/ca-certificates.crt", O_RDONLY) = -1 ENOENT (No such file or directory)

And then ran:
[dnelson@sulaco ~]$ sudo ln -s /usr/share/rescuetime/curl-ca-bundle.crt /etc/ssl/certs/ca-certificates.crt

Which solved the problem in Fedora 25.

Thanks @DakotaNelson. I can confirm your suggestion works in openSUSE Tumbleweed 20161014.

I've just stumbled upon this from a support ticket, and indeed have had a few issues supporting Linux securely, I have this stackoverflow question which I'm trying to get answered and maybe someone here can help out or at least up vote for more visibility.

Are there any updates to this issue, please?

This seems to be a problem that only affects Nix on non-NixOS.
We probably just need to wrap the executable to point SSL_CERT_FILE at cacert.


EHPR-94 | Resolve Database connection

This PR to fix immediate crash of API lambda.

Could you explain what the issue was? Looks like a file was just moved up a level?

Please check https://github.com/bcgov/ehpr2/commit/75afd99d6a3f2f33b30864a5c51993829126bd42 it  may resolve this issue already, we possibly have duplicate DTOs in common

Please check 75afd99 it may resolve this issue already, we possibly have duplicate DTOs in common

In current main branch, building of web app is failing due to this issue. So, my focus is to deploy a crash free app in lambda.


26-as-a-user-i-want-to-validate-if-metadata-is-hip-412-compliant

Description:

Related issue(s):
Fixes #26
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Merged in hedera-nft-utilities


Bing Spell Check API does not provide any privacy or security around user data

Is your feature request related to a problem? Please describe.
PROBLEM:  Bing Spell Check provides no privacy/security around the data my customers enter.
Describe the solution you'd like
Please offer a way for Bing Spell Check NOT to store or use my customer data in any way; but instead to provide security & privacy around the data my customers enter.
Additional context
When creating a Bing Spell Check v7 resource in the Azure Portal, there is a disclaimer, which is an absolute SHOW-STOPPER for my company: "Microsoft will use data you send to Bing Search Services to improve Microsoft products and services. Where you send personal data to this service, you are responsible for obtaining sufficient consent from the data subjects. The General Privacy and Security Terms in the Online Services Terms to not apply to this Service."

@LauraActionJackson Please give product feedback at https://account.microsoft.com/account/privacy


[WIP] Stream based token parser

This replaces the token parser inside tedious with a stream based version based on the dissolve module.
This is pretty much a big ugly hack at the moment, but it passes all tedious unit and integration tests. (Also, the application I'm working on passes all it's tests with the new parser). Beware, here be dragons.
The idea behind these changes is to make the token parsing code more performant. Right now, if a token is split over multiple tds messages, parsing is retried each time a new message is received from scratch. Obviously, this results in incredibly bad performance.
The dissolve module gives us a way to do chunk based parsing of tokens, without having to throw away the parsing results even in the case that tokens span multiple packages.
Here's some performance numbers (see benchmarks/benchmarks.coffee):
Old Parser
$ node_modules/.bin/coffee benchmarks/query.coffee
Benchmarking 'nvarchar (small)'
nvarchar (small) x 68.06 ops/sec ±0.32% (22 runs sampled)
Memory: 5.4609375
Benchmarking 'nvarchar (large)'
nvarchar (large) x 1.84 ops/sec ±1.45% (14 runs sampled)
Memory: 19.265625
Benchmarking 'varbinary (small)'
varbinary (small) x 66.09 ops/sec ±0.14% (14 runs sampled)
Memory: 3.6171875
Benchmarking 'varbinary (large)'
varbinary (large) x 1.89 ops/sec ±1.64% (14 runs sampled)
Memory:<PHONE_NUMBER>5
Benchmarking 'varbinary (huge)'
varbinary (huge) x 0.02 ops/sec ±9.94% (5 runs sampled)
Memory:<PHONE_NUMBER>5
Done!

New Parser
$ node_modules/.bin/coffee benchmarks/query.coffee
Benchmarking 'nvarchar (small)'
nvarchar (small) x 69.67 ops/sec ±0.50% (29 runs sampled)
Memory: 7.828125
Benchmarking 'nvarchar (large)'
nvarchar (large) x 7.01 ops/sec ±1.82% (37 runs sampled)
Memory: 31.6328125
Benchmarking 'varbinary (small)'
varbinary (small) x 65.99 ops/sec ±0.19% (13 runs sampled)
Memory: 6.296875
Benchmarking 'varbinary (large)'
varbinary (large) x 8.09 ops/sec ±2.41% (41 runs sampled)
Memory: 51.69140625
Benchmarking 'varbinary (huge)'
varbinary (huge) x 0.80 ops/sec ±1.13% (8 runs sampled)
Memory:<PHONE_NUMBER>5
Done!

As you can see, the switch to dissolve hugely improves the performance when receiving data that spans multiple tds messages, while not impacting the performance of data that is contained inside a single message.
There is a slight increase in the memory usage reported in the benchmarks, and I'm not really sure what is causing this, but I'll take a closer look.
Overall, this is definitely not ready yet, but is meant to show off some of the work I've been doing and to gather some feedback on this.

The unit tests are failing right now, I'll get that fixed up ASAP. But the integration tests are all passing fine.

The latest performance numbers:
Benchmarking 'nvarchar (small)'
nvarchar (small) x 75.82 ops/sec ±0.47% (51 runs sampled)
Memory: 6.64453125
Benchmarking 'nvarchar (large)'
nvarchar (large) x 9.07 ops/sec ±2.04% (46 runs sampled)
Memory: 20.6328125
Benchmarking 'varbinary (small)'
varbinary (small) x 66.26 ops/sec ±0.17% (14 runs sampled)
Memory: 4.609375
Benchmarking 'varbinary (large)'
varbinary (large) x 10.58 ops/sec ±2.98% (51 runs sampled)
Memory: 19.91796875
Benchmarking 'varbinary (huge)'
varbinary (huge) x 1.03 ops/sec ±1.67% (10 runs sampled)
Memory:<PHONE_NUMBER>5
Done!

This requires some patches to the dissolve module. I'll open a PR for that shortly on their github project page.

@arthurschreiber are you interested in becoming a maintainer on Tedious? Quite frankly, we could use a couple of new maintainers who are smart and actively using Tedious. @rossipedia and I were making significant contributions for a while, but we ended up porting the project we were using it for away from node.js, so our time and motivation to maintain Tedious has dropped off quite a bit. @patriksimek is still around here and there, but no one has the time to give it the attention it deserves.
You've started tackling streams, performance, and testing concerns, which have all been on our wish list for a very long time. If your intentions are to continue contributing improvements like this, then I don't think anyone would have a problem adding you as a collaborator.

@bretcope Sure, why not. :smile: I also got added as a maintainer for the dissolve module that I've been using in this POC.
With some more changes that I made to the dissolve module, I get the following performance numbers now:
Benchmarking 'nvarchar (small)'
nvarchar (small) x 69.76 ops/sec ±0.37% (29 runs sampled)
Memory: 6.1796875
Benchmarking 'nvarchar (large)'
nvarchar (large) x 10.65 ops/sec ±1.27% (52 runs sampled)
Memory: 11.9140625
Benchmarking 'varbinary (small)'
varbinary (small) x 66.47 ops/sec ±0.22% (15 runs sampled)
Memory: 4.35546875
Benchmarking 'varbinary (large)'
varbinary (large) x 13.04 ops/sec ±2.24% (60 runs sampled)
Memory: 20.3046875
Benchmarking 'varbinary (huge)'
varbinary (huge) x 1.36 ops/sec ±1.47% (11 runs sampled)
Memory: 183.859375
Done!

So, for fetching a 5MB varbinary column out of the database, this is around 6 times faster, while a 50MB varbinary, the streamig token parser is more than 67 times faster.
Next steps will be to land all the performance improvements I made in the dissolve module, release a new version of that, and then clean up this PR.

Any further work being done on this? This seems like great work

Yeah, I've not forgotten about this. I'm also thinking about rewriting the part of tedious that serializes data back to the stream to make use of concentrate to reduce memory usage and hopefully also improve performance.
What currently bugs me a bit is that both the old and new parsers are (subjectively) super fugly. Does anyone have any idea on how we could better organize the code?

Does this mean that the horrible, nasty, ugly try/catch in token-stream-parser will go? That would be awesome. It's always bugged me.
(It would also mean that the Tokens that span Packets page could go too.)

@pekim Yup, that'll be completely gone. That (and the ensuing reparsing of tokens) is one of the reasons memory usage and performance really go down the drain.

@arthurschreiber imho you're trading one fugly solution for another fugly solution that improves performance/memory consumption, so I'd just merge the code in and then worry about making it look pretty :smile:. If you can get it passing all tests, I'd say its prime for merging (and the concentrate change can come in another PR).

I'll be looking into bang this up into a more acceptable state over the course of next week and to make this available for others to take on a test ride. Sorry for the delay, I know many people are really looking forward to this.

Hey all. Just wanted to let you know that I'm still working on this. I made some performance and memory improvements to the dissolve module I'm using here, which I need to upstream first. Afterwards, I'll finish up this PR for final review.
I'll try to get at it this weekend.

Okay, I just rebased all the changes here against the latest master changes and fixed up the unit tests to pass.
Here's the latest benchmark result:
$ node_modules/.bin/coffee benchmarks/query.coffee
Benchmarking 'nvarchar (small)'
nvarchar (small) x 69.54 ops/sec ±0.46% (27 runs sampled)
Memory: 4.91796875
Benchmarking 'nvarchar (large)'
nvarchar (large) x 8.70 ops/sec ±2.61% (45 runs sampled)
Memory: 12.40625
Benchmarking 'varbinary (small)'
varbinary (small) x 65.67 ops/sec ±0.20% (11 runs sampled)
Memory: 5.01171875
Benchmarking 'varbinary (4)'
varbinary (4) x 65.82 ops/sec ±0.23% (12 runs sampled)
Memory: 0.55859375
Benchmarking 'varbinary (large)'
varbinary (large) x 19.77 ops/sec ±1.94% (49 runs sampled)
Memory: 41.0078125
Benchmarking 'varbinary (huge)'
varbinary (huge) x 2.27 ops/sec ±1.69% (15 runs sampled)
Memory: 92.015625
Done!

These results are from running on Node.js 0.10.38. Overall, these are incredible performance and memory improvements over what tedious offered before, when reading large results back from the database. And there's no noticable impact when reading small results.
The next step would be to also make the sending side be stream based, but I want this to happen in a separate PR.
@bretcope @pekim @patriksimek What do you guys think?

Just out of curiosity... are these changes likely to make it easier to
implement streaming large column values such as varbinary(max) columns? (In
ODBC it's possible by calling SQLGetData repeatedly with a small buffer.)
On 23 July 2015 at 16:55, Arthur Schreiber<EMAIL_ADDRESS>wrote:

Okay, I just rebased all the changes here against the latest master
changes and fixed up the unit tests to pass.
Here's the latest benchmark result:
$ node_modules/.bin/coffee benchmarks/query.coffee
Benchmarking 'nvarchar (small)'
nvarchar (small) x 69.54 ops/sec ±0.46% (27 runs sampled)
Memory: 4.91796875
Benchmarking 'nvarchar (large)'
nvarchar (large) x 8.70 ops/sec ±2.61% (45 runs sampled)
Memory: 12.40625
Benchmarking 'varbinary (small)'
varbinary (small) x 65.67 ops/sec ±0.20% (11 runs sampled)
Memory: 5.01171875
Benchmarking 'varbinary (4)'
varbinary (4) x 65.82 ops/sec ±0.23% (12 runs sampled)
Memory: 0.55859375
Benchmarking 'varbinary (large)'
varbinary (large) x 19.77 ops/sec ±1.94% (49 runs sampled)
Memory: 41.0078125
Benchmarking 'varbinary (huge)'
varbinary (huge) x 2.27 ops/sec ±1.69% (15 runs sampled)
Memory: 92.015625
Done!
These results are from running on Node.js 0.10.38. Overall, these are
incredible performance and memory improvements over what tedious offered
before, when reading large results back from the database. And there's no
noticable impact when reading small results.
The next step would be to also make the sending side be stream based, but
I want this to happen in a separate PR.
@bretcope https://github.com/bretcope @pekim https://github.com/pekim
@patriksimek https://github.com/patriksimek What do you guys think?
—
Reply to this email directly or view it on GitHub
https://github.com/pekim/tedious/pull/225#issuecomment-124150183.

@lee-houghton In theory, this might be possible. I just really don't know how the API would have to look like to support this. I'm also not familiar with ODBC.

I can't see a reason not to merge this if you're passing all the tests and you are seeing that kind of increase, but I might be missing something. So, LGTM :+1: from me!

Sounds good to me.

@arthurschreiber this is great, thanks!
Please take a look at the mssql test suite, there are few test that doesn't pass. There's a quick guide how to set up test config.
When I was trying to debug the protocol to make sure it's not a mssql issue I have found that config.options.debug.token switch doesn't work. It is propably caused by this line that is missing in the PR.
Please let me know if I can be helpful.

@patriksimek Thanks for the heads up. Looks like the tedious test suite has some holes. I'll try to check what running the mssql tests will show.

@patriksimek I pushed some additional changes that should fix the node-mssql issues. There's still some issues left, but I believe they are caused by node-mssql assuming XACT_ABORT is on by default (which it is not).

@arthurschreiber thanks, fixed that. I had that enabled in my config by mistake. There are still some issues with local datetime. Not sure if you can see them because it propably depends on your current time zone. I'll try to investigate it.

So, I have no idea how to patch this PR so I made a gist with changed value-parser.coffee. With that all test are passing.
https://gist.github.com/patriksimek/a0dabc66e41657af5612

@patriksimek Thanks. I'll try to incorporate that change ASAP.

Closed in favor of #285.


Side missions spawn in the water on rare occassions

0.955 Altis
Area of Pefkas Outpost
Side mission: rescue pilots from crash site
Hey,
yesterday we encountered this bug, where crash site spawned in water. It had soldiers patrolling around it, so they had to swim sometimes. There was an enemy tank and truck submerged or on those rocks in the picture. When we the cleared area one pilot inside a water, once I clicked capture on him, he dawned, died immediately and mission was a failure.
Pictures: https://steamuserimages-a.akamaihd.net/ugc/830202454034331017/917BB25CD6F8040B3F8E3D378EB255384A429CD4/
https://steamuserimages-a.akamaihd.net/ugc/851594469172824330/53C162A8608E45D5676C9D7AD99434511F35535A/

Hey there and sorry for the late reply.
The helicopter position is random around the defined spawnpoints for sidemissions. We can check if there are some maybe too close to the water but basicly it's something which could rarely happen, yes.


Search Functionality is not working in all grid of Admin panel.

Framework Version :0.1.1
Unable to search anything from search button.This issue exist for all grid.

The search works but it does not works when you hit enter you have to press the search icon to search inside the grid.

I am issuing a fix for keyboard code on the search field of the grid.


Add SensitiveParameter attribute

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#89

@ev-gor Why you close PR?
It looks good.


Roles cannot be assigned with this plugin

@atomarch
I am using ur plugin and need to work with roles for different users. After authenticating with google I was landed in admin interface but not able to assign user roles. This will limit our capacity to use ur plugin.
The problem is current yourls user is not set with google user. So while enabling the auth manager plugin, we are getting authentication failed exception.
Is it possible to set the google user as current yourls user? Or somehow do we need to write the config file with username and dummy password? So that we can use Auth manger plus with your plugin.
If you could make the compatibility possible, it’s a great feature to use. Please let me know your thoughts.

Similar issue with using Yourls API, seems to break with this plugin:
Your secret signature token: Cannot generate auth signature: no username (It's a secret. Keep it secret)


feat: [FilterBridge] Add URI/URL filter option to FilterBridge

This is a good change. But I notice that the user input is not escaped for regex and I think it's intentional. This means that trying to filter on e.g. http://google.com would fail because / is used as delimiter. It also means the dot . in google.com can be any character.


Review using StringBuilder in pinvokes

From https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/interop-pinvokes.md#strings

StringBuilder marshalling always re-allocate/re-copy buffers. Make sense review our code to reduce such allocations.

From @daxian-dbw https://github.com/PowerShell/PowerShell/pull/9066#issuecomment-472975185

it seems StringBuilder should never be used for P/Invoke as it incurs 4 allocations. I think it make sense to review our uses of it and replace with ArrayPool when appropriate, because we will copy the existing pattern when writing new code.

From above docs:

[USE] Char arrays from ArrayPool or StringBuffer.

We are using StringBuilder for the [out] string parameter in P/Invoke almost everywhere.

Is everything so bad? :-)

I could only find 3:
[Microsoft.PowerShell.Commands.Native]::LCIDToLocaleName()
[Microsoft.PowerShell.Commands.ProcessNativeMethods]::CreateProcess()
[Microsoft.Windows.Appx.PackageManager.Commands.NativeMethods]::PackageFamilyNameFromFullName()

In CoreFX repo I found 25 files with StringBuilder and name *interop*.cs.
Interesting most of them has [out] attribute!
It seems we need to invertigate it more deeply. Where (repo) is the StringBuilder marshalling implemented?

Well, some may not be a out parameter in the underlying Win32 API. I found the following by a quick search in the files @iSazonov touched in #8847
[Process.cs] CreateProcessWithLogonW, CreateProcess
[ConsolControl.cs] GetConsoleTitle, ReadConsole
[Native.cs] LookupAccountSid, LookupAccountNam, FormatMessage
[FileSystemProvider.cs] QueryDosDevice ...

Sample from CoreFX for GetWindowTextW - they use pointer instead of StringBuilder.
https://github.com/dotnet/corefx/blob/81bd671efda5db20fae1eb381aeff2cf8ea727ac/src/System.Diagnostics.Process/src/System/Diagnostics/Process.Win32.cs#L227-L234
https://github.com/dotnet/corefx/blob/8c5260061b11323dfd97fbab614d54402405513f/src/Common/src/Interop/Windows/User32/Interop.GetWindowTextW.cs#L14

One reference to DllImport marhalling https://github.com/dotnet/coreclr/blob/78cbd968cabc2bd6c84bfe6a0a240658e564a6d5/src/vm/ilmarshalers.cpp#L810
Not clear how it works.

Important: [Out] (OutAttribute) is very different from out (keyword). The attribute [Out] is trying to express that information needs to flow back to the caller (so if a copy had to be made during marshaling, when the call completes, the CLR needs to copy data back into the managed object). For blittable things that just get pinned and modified directly in place, it does not matter if you forget [Out], because the memory was modified in-place.
The out keyword, though, also indicates the level of indirection. I.e. if you are passing a Foo to native code via PInvoke, and Foo is a class, and the PInvoke signature says "Foo f", that basically turns into "Foo* f". But if the signature says "out Foo f", then the equivalent native signature is "Foo** f". Big difference!
(The ref keyword is similar in terms of indicating another level of indirection; but also implies [In] and [Out], which sometimes the marshaler may need to know if it needs to do something to marshal a pointer in as well as out, such as a COM interface.)
(Maybe you folks already know this stuff, but if so, there may have been some typos in this thread that made me think it might need clearing up (it's very easy to mix [Out] and out; I did it while typing this), so I wanted to spell it out to make sure we're on the same page about stuff. Sorry to re-explain if you were already aware.)
It may be the case for StringBuilder that it is special-cased and magically works with or without out, but I would not have guessed that (which is why I got involved in the first place when I saw the out added to that PR).
I have always believed that the marshaler had special knowledge of StringBuilder and would just pin the buffer. And those comments I found on the internet (from that long-gone MSDN article) seems to indicate that perhaps it did indeed work that way once. But the doc linked by iSazonov at the top seems clear, and I would trust current documentation from CoreFX people way more than my old memory.
And in fact, I bet these are the issues that inspired that (current) documentation:

https://github.com/dotnet/coreclr/issues/1842
https://github.com/dotnet/coreclr/pull/21785
https://github.com/dotnet/coreclr/pull/21800

Sounds like there was a fly in the ointment regarding null terminators; perhaps in ancient history it did work as I thought, but then had to be pessimized to be correct for certain required cases.
On the bright side, at least there is a good recommendation for alternatives that are better.

@jazzdelightsme Thanks for clarify and confirmation!
I added "[USE] Char arrays from ArrayPool or StringBuffer." in the issue description.
We need to look our P/Invokes on hot paths.

I hope it is full list:
src\Microsoft.PowerShell.Commands.Management\commands\management\GetComputerInfoCommand.cs
src\Microsoft.PowerShell.Commands.Management\commands\management\Process.cs
src\Microsoft.PowerShell.ConsoleHost\host\msh\ConsoleControl.cs
src\Microsoft.PowerShell.CoreCLR.Eventing\DotNetCode\Eventing\UnsafeNativeMethods.cs
src\Microsoft.PowerShell.LocalAccounts\LocalAccounts\Native.cs
src\System.Management.Automation\cimSupport\cmdletization\ScriptWriter.cs
src\System.Management.Automation\engine\hostifaces\HostUtilities.cs
src\System.Management.Automation\engine\hostifaces\NativeCultureResolver.cs
src\System.Management.Automation\engine\NativeCommandProcessor.cs
src\System.Management.Automation\namespaces\FileSystemProvider.cs
src\System.Management.Automation\namespaces\Win32Native.cs

@daxian-dbw @SteveL-MSFT I finished (I hope) reviewing the case. Some PR is waiting a code review. Please look the list above and confirm that I do not skipped something.


difference between translate filter and directive interpolation with messageformat

Hi !
using angular-translate v2.7.2, I can only use directive declaration to translate this chain :
"TOTAL_WEBPAGES" : "{TOTAL, plural, one {one page saved.} other {# pages saved.}}",

<span
    translate="TOTAL_WEBPAGES"
    translate-values="{ TOTAL: website.count_webpage }"
    translate-interpolation="messageformat"></span>

When trying to use filter declaration, I always get the singular translation. I feel like scope variable website.count_webpage can't be accessed. In docs, I can only find a hard-coded example for TOTAL value. Is there another way to write it to make it works ?
{{ 'TOTAL_WEBPAGES' | translate:"{ TOTAL: website.count_webpage }":"messageformat" }}

Thanks for your suggestions !

Since AJS 1.3, must have to apply any arguments to a filter function (even the scope). A filter does not have access to the scope any more.


type & image to openGraph.basic usage example

Updated the doc to include type & image in openGraph.basic since they're required.

Looks good! Will merge and fix the failing formatting issues (I introduced) seperately


Not able to fetch all text data & Not able to extract text, table data in proper format

Hi Team,
I am using layout parser & detectron2 to detect everything i.e. text, tables, title, list but not figures  from the pdf(which I converted into image using pdf2image). I wanted to then extract the detected text, title, table, list in .txt format
Issues:
1)It seems like model is not recognizing all of text data properly
2) While extracting data in .txt format , it appears that:
a)I am not bale to print text data in sequence as it appears on pdf
b) I am not able to extract table data in tabular format
Can you please suggest how I can resolve above issues? Thank you!
Code:
Install necessary libraries
#install detectron2:
!pip install<EMAIL_ADDRESS>#install layoutparser
!pip install layoutparser
!pip install layoutparser[ocr]
##install opencv, numpy, matplotlib
!pip install opencv-python numpy matplotlib
!pip3 install pdf2image
!sudo apt install build-essential libpoppler-cpp-dev pkg-config python3-dev
!apt-get install poppler-utils
!pip install --upgrade google-cloud-vision
!pip uninstall google-cloud-vision
!pip install google-cloud-vision
!apt install tesseract-ocr
!apt install libtesseract-dev
!pip install pytesseract
import os
from pdf2image import convert_from_path
import shutil
import cv2
import numpy as np
import layoutparser as lp
from pdf2image import convert_from_path
Define Pdf_path
pdf_file='7050X_Q_A.pdf'
Define your output file name here
output_file = 'output.txt'
with open(output_file, 'w', encoding='utf-8') as f:
for i, page_img in enumerate(convert_from_path(pdf_file)):
img = np.asarray(page_img)
    model3 = lp.models.Detectron2LayoutModel(
        'lp://PubLayNet/faster_rcnn_R_50_FPN_3x/config',
        extra_config=["MODEL.ROI_HEADS.SCORE_THRESH_TEST", 0.5],
        label_map={0: "Text", 1: "Title", 2: "List", 3: "Table", 4: "Figure"}
    )

    layout_result3 = model3.detect(img)

    text_blocks = lp.Layout([b for b in layout_result3 if b.type != "Figure"])

    h, w = img.shape[:2]

    left_interval = lp.Interval(0, w / 2 * 1.05, axis='x').put_on_canvas(img)

    left_blocks = text_blocks.filter_by(left_interval, center=True)
    left_blocks.sort(key=lambda b: b.coordinates[1])

    right_blocks = [b for b in text_blocks if b not in left_blocks]
    right_blocks.sort(key=lambda b: b.coordinates[1])

    text_blocks = lp.Layout([b.set(id=idx) for idx, b in enumerate(left_blocks + right_blocks)])
    viz=lp.draw_box(img, text_blocks,box_width=10,show_element_id=True)
    display(viz)
    ocr_agent = lp.TesseractAgent(languages='eng')
    for block in text_blocks:
           segment_image = (block
                            .pad(left=5, right=5, top=5, bottom=5)
                            .crop_image(img))

           text = ocr_agent.detect(segment_image)
           block.set(text=text, inplace=True)

        # Write text to the output file
    for txt in text_blocks.get_texts():
        #print(txt, end='\n---\n')
        f.write(txt + '\n---\n')

print("Text extraction completed. Check the output file:", output_file)
Environment

Windows
Layout Parser & layoutparser[ocr] version 0.3.4
PyTorch version: 2.1.0+cu121
!pip install torch==2.1.0 torchvision==0.16.0 torchaudio==2.1.0 --index-url https://download.pytorch.org/whl/cu121
google-cloud-vision-3.5.0
google-api-core Version: 2.11.1
6.Python 3.10.6

Thanks
Reema Jain

Hi Team,
Can someone please help on resolving above issue?
Thank you for the help!
Reema Jain


Fix skipping builds in netlify

Our netlify build is often not rebuilding on content commits because they are outside the site directory (we're symlinking content files into the site directory).
Fix seems to be listed here and involves adding some netlify config:
https://answers.netlify.com/t/builds-cancelled-for-a-new-branch-due-to-no-content-change/17169/2

FIXED. (I hope) in 0bd5d2f3f8947e3dddcd294e9cc8574a37c66e48


Settings page: anchor links to Language / Volume

Including associated changes and other corrections in language files.

This was my first pull request ever. Please let me know, if I did it right (or not) :)

HI @Yordan1976
looking good! the only thing I changed: besides the master branch, I have a develop branch. So I am not merging this straight with the master, but the develop branch first.


Add Profile: Akshu-on-github

Created a file Akshu-on-github.json to add my profile to the link tree

Oops! Not sure how that happened, thanks for pointing that out @voidz7 - I'll create another PR, one sec


Component: Datepicker

Do some research to find a fitting component for a datepicker.
Expand this list:

https://vaadin.com/docs/latest/components/date-picker

Closing as not relevant for now


Document Initiative Setup Process

Currently we're lacking documentation around creating an initiative.  Since this is coming up a bit with Outreach, I figured it would also be a good time to discuss documenting some of the content that is needed so this process is repeatable.
Templates that are needed in this repo:

[ ] Template for inclusion of the Node.js Code of Conduct (example)
[ ] Template for CONTRIBUTING.md (example)
[ ] Meeting minutes configuration (if needed – see this for the four kinds of templates)
[ ] Add to the STRATEGIC-INITIATIVES.md list

reference: previous discussion/issue

I can help mentor any person who wants to explore this!

We could also add:

Initiative should have at least 1 CommComm member and 3 member collaborators.
Initiative should have a working meeting schedule.
Initiative should have an almost-proper documentation.

Correct me if all these are really not necessary.
Some criteria’s an initiative should meet even before creating a repo. So, we don’t have repos that are not being used, and have no definitions.

@codeekage I believe we've generally seen success with 1 CommComm member and 2 initiative collaborators. Also not entirely sure what the last one would entail 😅

@bnb @chowdhurian I'd happy to help :) sure, if it relevant.

@lidoravitan definitely still is! Happy to help if you're still interested. My suggestion:

Fork this repo
Create a new file, something like governance/INITIATIVE_PROCESS.md
Start documenting what I listed in the original post – these can be basic documentation and can be expanded upon later by others if needed
Submit a PR!

@bnb I want to push it forward soon, please can you give me a reference to the original post, ("Start documenting what I listed in the original post..."). it would be very helpful, thank you so much!

@chowdhurian @bnb  thanks! I've opened a new pull request, hope it fits the requirements. 💚

This has been PR'ed and merged. Thank you @lidoravitan.


Update existing performance tests to use more "modern" coding styles

Existing test scenarios are written in somewhat outdated ballerina syntax (example). This may or may not having performance effect currently. Eitherway it is better to update these to use the newer (and recomanded) ballerina coding styles
Required for https://github.com/ballerina-platform/ballerina-lang/issues/43414

PR: https://github.com/ballerina-platform/ballerina-performance/pull/63

Fixed with https://github.com/ballerina-platform/ballerina-performance/pull/63


[Bug] ORDER BY and LIMIT using subqueries

Hello!
I'm trying to upgrade 2.4.8 to 2.5.14 and i've been having some issues using ORDER BY and LIMIT using 1 to Many JOINs.
The follow query will run normally in my mysql client:
SELECT distinct(p.id)
FROM product p
INNER JOIN product_translations pt ON pt.id = p.id AND pt.lang = 'en' # this is just one record
ORDER BY pt.name
GROUP BY p.id DESC
LIMIT 20;

product has a 1 to many relation with product_translations and this works fine on 2.4 and below from 2.5 everytime i try to run that same query i get a

Cannot select distinct identifiers from query with LIMIT and ORDER BY on a column from a fetch joined to-many association. Use output walkers.

This was a change done as part of this PR: https://github.com/doctrine/doctrine2/pull/1353 more precisely this file: https://github.com/doctrine/doctrine2/commit/608dfa2f571f2b93ccdb2fcd9ab66c4160f02a9e#diff-e7afd03e36cf8d3fe30efd893375da43
Looking around i'm not sure, but i thin KNP Paginator is even using the old one: https://github.com/KnpLabs/knp-components/blob/master/src/Knp/Component/Pager/Event/Subscriber/Paginate/Doctrine/ORM/Query/LimitSubqueryWalker.php as a work around for the issue.

Why do you need the distinct? The GROUP BY already results in one row per product.
Changing the SELECT distinct(p.id) to SELECT p.id should give you the same result.

@naitsirch yes you are right - although the problem has nothing to do with distinct i think. It's with LIMIT and ORDER.


[Bug]: SQLSTATE[HY000]: General error: 1 near ")": syntax error (Connection: , SQL: create table "flatfile_documentations" ())

What happened?
I have installed the package with success on my Mac in development and have an issue similar to M40. The error is: SQLSTATE[HY000]: General error: 1 near ")": syntax error (Connection: , SQL: create table "flatfile_documentations" ())
How to reproduce the bug
I have tried to create a new install and it has the same issue.
It is fixed after adding the schema:
``
protected $schema = [
  'id' => 'string',
  'slug' => 'string',
  'path' => 'string',
  'content' => 'text',
  'title' => 'string',
  'group' => 'string',
  'icon' => 'string',
  'parent' => 'string',
  'order' => 'integer',
]

``
Package Version
1.10
PHP Version
8.3.12
Laravel Version
11.28.1
Which operating systems does with happen with?
Linux
Notes
It seems that the database (MariaDB for Linux and MySQL for MacOs) is the key difference.

The file edited is: vendors\guava/filament-knowledge-bases/src/Models/FlatfileDocumentation.src

Thanks for the headsup! Seems like Sushi throws this error when the dataset is empty (you haven't created any documentations yet) and adding the $schema manually fixes it.
Fixed in 1.10.3

Thanks!Regards, AxelOn 26 Oct 2024, at 10:01 PM, Lukas Frey @.***> wrote:
Thanks for the headsup! Seems like Sushi throws this error when the dataset is empty (you haven't created any documentations yet) and adding the $schema manually fixes it.
Fixed in 1.10.3
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Update Composer dependencies (2021-07-01-00-09)

Visual regression test passed!
View the visual regression test report


[SwiftASTContext] Print fatal error if there is one

Cherry-picking changes from #606 to swift-4.2-branch.

@swift-ci please test and merge


Support external ca certificate import function when creating a cluster

What's it about?

What's the reason why we need it?
Support external ca certificate import function when creating a cluster

I believe this is an important feature for KubeClipper. There're a few use cases:

case Source k8s supports the use of external ca certificates

Please leave your comments below if there's anyone agrees with me. Or just give me a thumb up.
Area Suggestion

/area api

/kind feature-request

/close


Regarding accuracy of the Scratch B models in the paper

Hello,
Thanks for an interesting paper. I was looking at the accuracy of Scratch B models compared to big-unpruned networks and it seems Scratch B is performing better than unpruned networks most of the time. This seems to be counter intuitive as the bigger network if could be trained effectively should outperform smaller networks. Do you think the difference is statistically significant?
Thanks

Hi,
Thanks for your interests in our paper and code. Indeed sometimes Scratch B is performing better than unpruned networks. This could be due to the following reasons:

Prune ratio is relatively small.
Standard training schedule may not be enough for convergence.

For CIFAR-10 experiments, we report the average of five runs. For ImageNet, each result is from one run. It is safe to say that the difference in CIFAR-10 is statistically significant.


Make ColorScheme a WinRT object

Summary of the Pull Request
ColorScheme is now a WinRT object in the TerminalApp project.
References
#7141 - ColorScheme is a settings object
#885 - this new settings object will be moved to a new TerminalSettingsModel project
PR Checklist

[x] Tests passed

Detailed Description of the Pull Request / Additional comments
Like in TerminalSettings, colors in the ColorScheme are exposed as UInt32.
All of the JSON stuff can't be exposed via the idl. So the plan here is that we'll have the TerminalSettingsModel project handle all of the serialization when it's moved over. These functions will be exposed off of the implementation namespace, not projected namespace.
Validation Steps Performed

[x] Tests passed
[x] Deployment succeeded

I don't get it. All that 7a3ca00a3df630a50802cd177a89c739803e3d8b did was rename some variables.
The responsibility to apply a color scheme to an IControlSettings lies inside TerminalApp where TerminalSettings lives. You haven't done anything to break the dependency from colorscheme to terminal control (where IControlSettings now lives)

I don't get it. All that 7a3ca00 did was rename some variables.
The responsibility to apply a color scheme to an IControlSettings lies inside TerminalApp where TerminalSettings lives. You haven't done anything to break the dependency from colorscheme to terminal control (where IControlSettings now lives)

At some point I had ColorScheme.idl referencing a TerminalSettings. That shouldn't be allowed. Making it reference an IControlSettings is better, but you're right, all of that logic should be moved to TermApp.
I was thinking of this little change being a stepping stone to #885's last step (Move the new WinRT settings objects into TerminalSettings project). We don't need the separation until then, at which point I'll also move the similar logic over for Profile.
TLDR: referencing IControlSettings > TerminalSettings

I actually disagree that it’s better!
Eventually, the interfaces for settings will become read-only and you will need to transition this code back to using a class type that has setters on it. Moving to the interface right now is going to be a roadblock for when you inevitably have to concentrate the logic in App.

(cf #7219)


i18n(zh-cn): Update images.mdx

Description (required)
Update images.mdx
Related issues & labels (optional)

#6797
#6799

Thank you for the update. The check link CI failed.

Sorry. Withdrawn LGTM. The missing-image-dimension.mdx also has outdated content that needs updating.
See: https://github.com/withastro/docs/pull/6895


swag-docs-plugin-quick-start / remove unused code from services.xml

please adjust your services.xml file like in your docs https://docs.shopware.com/en/shopware-platform-dev-en/internals/plugins/plugin-quick-start#the-services-xml
because is a confuse

This issue was closed, because it is outdated. If this is still an issue for you, feel free to create a new ticket with further information.


Compatibility with Gatsby v5

Expected Behavior
Gatsby v5 has been out since Nov 2022. Currently, gatsby-source-storyblok doesn't officially support Gatsby v5 but would be great if it does.
Current Behavior
gatsby-source-storyblok requires installation with the --legacy-peer-deps npm flag or override to work with Gatsby 5
Steps to Reproduce

Set up a repo with the Gatsby v5 starter
Try to install gatsby-source-storyblok without the --legacy-peer-deps flag
See errors

Hi @yvonnetangsu
It's a work in progress to upgrade gatsby-source-storyblok. Please expect there will be an update about that.
In short, Gatsby's Partial Hydration includes React Server Components. Our visual editor needs client-side configs as we use hooks for loading the JS Bridge.

Awesome. Thank you for the update @schabibi1 ! Glad to know it's in the works.

FYI I am running gatsby-source-storyblok with Gatsby v5 using --legacy-peer-deps flag and it works.
As I understand it from this thread, there is work required to update the hooks & methods for working with Storyblok editor — we don't actually use these, we just use this plugin as a source plugin.
It would be great if the gatsby deps could be upgraded so we don't have to use the legacy peer deps flag to install.
I think longer term, it would be worthwhile to consider seperating the concerns of sourcing the data from the storyblok (what this plugin says it does) and providing the handles to work with the data in the visual editor.

Agree with what @alex-mcgovern said. Is it possible for the gatsby deps be updated first in the meantime while we wait for the larger update to the visual editor related things, @schabibi1?

I am having an issue getting draft content from the visual editor, I have it set to use draft version in gatsby-config.js. I can see the draft json file looks correct.  Would this issue have anything to do with that?

Upgrading Gatsby dependencies to Gatsby 5 will happen relatively soon as I have solid info confirmed internally.
Gatsby 5 uses React Server Components for users who want to use Partial Hydration. It breaks visual editor behaviors from what we tested to upgrade our React SDK.
Gatsby SDK imports our React SDK and it was the blocker. But I'm happy to tell you the communication will happen quite soon and Gatsby SDK will be able to adopt RSC support which Gatsby 5 uses for Partial Hydration.

That's wonderful news @schabibi1 ! 🎉  Thanks for the update.

Any news regarding this @schabibi1 ? ❤

@rasmusbe As we announced just recently our React SDK now supports RSC, gatsby-source-storyblok will follow RSC support which is necessary for upgrading to Gatsby 5.
https://dev.to/storyblok/storyblok-react-sdk-now-supports-react-server-components-1hk6
We're testing Partial Hydration as it includes RSC and once we confirm, we can release Gatsby 5 supported version of gatsby-source-storyblok.

We released Gatsby 5 supported gatsby-source-storyblok v6.0.0. #327
Communication through updated README, announcement on Discord, announcement article, and update Ultimate Tutorial series will happen upcoming days.
Thank you for everyone being patient and we're glad to tell this news first with you.

Thanks for the update @schabibi1! Great news.


Autobuyer constantly fail purchese

the script doesn t buy that fast even on a 2-3 second search

same for me


Set simplify_before_intersect=true for earth/water.

This is necessary for the code-changes that fix tile seams in mapzen/TileStache#34 and mapzen/tilequeue#16 to kick in.

:+1:


merge config

If you do not publish the config, it will lead to an error, so it is better to merge the config with the default one.

If you do not publish the config, it will lead to an error, so it is better to merge the config with the default one.

Thanks for your edits!


Implement joeattardi/emoji-button

ModernDeck 7.5 included a new emoji picker that was removed from 8.0 due to serious performance concerns.
Todo: Implement the emoji picker from https://github.com/joeattardi/emoji-button

This was implemented, and the UI skinned, but I am not happy with the performance of it. I guess we'll see what we can do.


Some Icon Didn't Work

Hi folks
I'm working locally and using Semantic UI for my blog design, and recenlty try the icon set.
It work fine for the icon with 1 attribute like list icon book icon, the problem appears when i use the icon that have more than 1 attribute like id card, it just show a blank square
Any ideas?

When I set @icon     : 'default'; in my theme.config and build with gulp, the icon never shows. But When I set @icon     : 'basic';, it works like a charm.
Another thing: Icon Set classes aren't matching the icons.

@maateen
thanks alot 👍


add drmmtks preprocessor

Codecov Report

Merging #508 into 2.0-dev will decrease coverage by 0.22%.
The diff coverage is 81.35%.

@@             Coverage Diff             @@
##           2.0-dev     #508      +/-   ##
===========================================
- Coverage    90.58%   90.36%   -0.23%
===========================================
  Files           62       63       +1
  Lines         1572     1629      +57
===========================================
+ Hits          1424     1472      +48
- Misses         148      157       +9

Impacted Files
Coverage Δ

matchzoo/data_pack/build_vocab_unit.py
100% <ø> (ø)
:arrow_up:

matchzoo/data_pack/build_unit_from_data_pack.py
100% <100%> (+7.69%)
:arrow_up:

matchzoo/preprocessors/__init__.py
100% <100%> (ø)
:arrow_up:

matchzoo/preprocessors/drmmtks_preprocessor.py
80.35% <80.35%> (ø)

matchzoo/data_generator/data_generator.py
97.29% <0%> (+2.7%)
:arrow_up:

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 8708f63...5b9f142. Read the comment docs.


#58 add a test for indexes, to check the relation to properties

PR about #58
I am not sure you want this test in this file, hope this is okay though.

updated title of the test


More Recent Activity

The Recent Activity on the homepage is really nice but if its being stored it would be great to see more than just whats on the home page for full audit and logging purposes.

FYI the entire action history is available via the admin UI.


A quick fix for site_url() vs home_url()

WordPress sometimes resides in its own directory. For example, see this and this skeleton repos. In this case site_url() returns wrong URL - http://example.com/custom_wp_dir. When you pass wp-login.php to the site_url() function - it's absolutely correct, but site's frontend should be reached via home_url() which points to the root of your custom install.
I've replace some occurences of site_url() with home_url() and added home_url() output to debug info. Tested on my local install, haven't found any bugs, but it would be nice if you check the code and see if I haven't edited smth that shouldn't be.
Some other thoughts on custom setups (which are pretty common and totally allowed by WP):

for WordPress Multisite there's another trick with network_site_url(), network_home_url(). I'll have a chance to test WSL on WP MS next week, will test everything and see if we need another PR
some plugins (and custom developed code) are changing the login url via hooks, for example, preventing access to wp-login.php and forwarding requests to /login/ or /signon/. That pages can the original wp-login.php or custom pages with login form via [wp_login_form()](https://codex.wordpress.org/Function_Reference/wp_login_form) function. See login_url, register_url for details.

Will be happy to help making it work with that custom setups as I use them on most of the sites.
 White-labeling and having no API request limits is great! And the code is total pleasure to read :+1:

Ooops, my Sublime Text removed a lot of hidden whitespace characters, the commit is dirty. Should I revert and make it cleaner or you still can make some use of it?

I did use ?w=1 to ignore the whitespaces in your PR and it looks good to me. I'll make few tests this sunday and merge it if everything is okey.
Thank you very much.


Run Configuration: Allow selecting version

I want to use PS core 6, not the builtin PS 2.

IDE uses the PS from your PATH. What is your OS? Do you have PS core 6 installed? Thanks.

Windows 10, PS core 6.1.0 is installed!

Then you must be actually using this. Why do you think PS version 2 is used?

Because

the startup message didn’t say anything about 6.1.0 as it does when I open that one.
Executing a script containing #Requires -Version 6 fails complaining that I only have version 2.

What startup message do you mean? Could you show a screenshot?
Can you post the full command line that is executed and full output you get? Thanks.

I have a fresh installation of Windows and IntelliJ, and I have the same problem.
Run configuration opens C:/WINDOWS/System32/WindowsPowerShell/v1.0/powershell.exe
not PowerShell 6. This path is visible at the first line of the Run window.
But when I open Terminal, it uses PowerShell 6.
In my previous Windows it was working fine.

Fixed in 2.0.7 version.


⚠ Avatar API Endpoint has degraded performance

In 36adedd, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1008 ms

Resolved: Avatar API Endpoint performance has improved in 2d33ee9.


Why is the non-blocking go implementation included?

I can't understand the technical reason for including the non-blocking version.

The non-blocking version is idiomatic Go, the blocking one is not. Comparing the idiomatic server implementations from different languages is interesting, and in fact, probably more so than comparing the optimized versions. The optimized versions will, in the end, have the same threads doing the same syscalls, and will probably differ very little in terms of performance.


cockpituous: remove releases except for fedora-31

As part of the move to using osbuild-composer instead of lorax, we should only release on fedora-31 since the other distros will fail to install osbuild-composer. This PR is required for #868

Codecov Report

Merging #871 into master will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #871      +/-   ##
==========================================
- Coverage   77.79%   77.67%   -0.12%
==========================================
  Files          69       69
  Lines        3386     3386
  Branches      710      710
==========================================
- Hits         2634     2630       -4
- Misses        752      756       +4

Impacted Files
Coverage Δ

components/Notifications/Notification.js
68.25% <0%> (-1.59%)
:arrow_down:

components/Modal/CreateBlueprint.js
93.61% <0%> (-1.07%)
:arrow_down:

core/sagas/inputs.js
71.42% <0%> (-1.03%)
:arrow_down:

core/sagas/blueprints.js
73.97% <0%> (-0.52%)
: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 0f58f6d...16df4e7. Read the comment docs.

If we do this, we also need to ensure that we don't push a newer lorax-composer into F30 that breaks cockpit-composer, as then we wouldn't have a chance any more to fix it. This is fine, but I wanted to point it out.

If we do this, we also need to ensure that we don't push a newer lorax-composer into F30 that breaks cockpit-composer, as then we wouldn't have a chance any more to fix it.

How could pushing a new lorax-composer break cockpit-composer?

How could pushing a new lorax-composer break cockpit-composer?

Not sure if that was a rhetorical question, but it could change API/behaviour and such. If that doesn't happen, then it's fine of course. I'm just interested in how we ensure that the current c-composer keeps working in F30.

I see what you mean now. I was thinking that cockpit-composer for F30 was switching to osbuild-composer, and in that case there's no way lorax-composer could effect it.
FWIW there are no planned changes for F30 at this point, only bugfixes if needed.

Thanks @bcl, that seems fine then.


Add factorial docs

Thank you for contributing to @AllAlgorithms, your pull request was approved to be merged ✨
Make sure that you ⭐ the project 🤓, and continue helping us!