Create connection from command line "-con" parameter generate Meta Model error

DBeaver - Version 4.3.5 CE, 4.3.4 CE, <IP_ADDRESS> CE
DBeaver JDBC driver: Microsoft Driver or jTDS driver
Operating System: Windows 7 / Windows 8.1
Database Server: Microsoft SQL Express 2014
A change in how to a connection is created when connection is created using command line parameter "-con" seems to have been introduced in DBeaver version 4.3.3 CE (worked with version 4.3.2 CE).
If I execute the command line (using Windows):
C:\...> dbeaver.exe -con "name=TEST|driver=Microsoft Driver|host=localhost|port=1443"
The DBeaver start up and show the error message:

Can't instantiate meta model
org.jkiss.dbeaver.ext.mssql.model.SQLServerMetaModel.()

This worked as expected when using DBeaver version 4.3.1 CE.

Can't reproduce. I can open connection with -con parameter without any issues.
Please post error/debug logs (https://github.com/dbeaver/dbeaver/wiki/Log-files)

Here is my ${HOME}.dbeaver4.metadata\DBeaver-debug.log file (from Windows)
dbeaver-debug.log

Do you use custom (manually created) driver?
In DBeaver 4.3.4+ we have switched to SQL Server data source provider.
This bug happens only with Generic driver.
Thanks for report.

I do not quite understand what a custom (manually created) driver is. I use the **MS SQL Server / Microsoft Driver" that is supplied by DBeaver and downloaded within DBeaver.
Should I used another driver, downloaded a driver from other source, or should I have specified the command line parameter differently?
On that the bug only happens with Generic driver in DBeaver 4.3.4+ seems odd, since the fault already existed in DBeaver version <IP_ADDRESS>.

Well, I found yet another bug. Sometimes DBeaver detects driver incorrectly when you specify driver name in -con command.
Generally it is much better to specify driver ID. In your case it will be mssql_jdbc_jtds.
But this ID is not visible in GUI. So I'll add extra field in driver editor dialog:

Answering your questions:

No, you don't need to create/change any driver. Just use driver name mssql_jdbc_jtds. In DBeaver 5.0 you can use driver name (jTDS driver) as well.
Yes, this is quite old bug.


[Feature] More flexibility in customized curves' expressions

Issue

It is possible to modify axis through expressions, but if we have many curves on the same plot is not possible to add a different axis' customization for each of them; this is very annoying
it is possible to add multiple expressions for the y-axis but not for the x-axis

Proposed solution
To solve both the above problem, It'd be very helpful to be able to associate (in the expression's chart menu) expressions to single independent groups of run and put many 'customized' group's plots on the same chart
Describe alternatives you've considered
For now the alternative I'm using is to apply each expression on each group. This is very unpractical because I get a lot of useless curves that make the representation less clear. Plus in this way is not possible to change x-axis expression for different plot on the same chart
Additional context
To have a more precise description of the problem it was discussed in this post

Hey Emanuele, thanks for the suggestion. I'll file a ticket for your request.


Pascal's Triangle

Description
I have prepared a documentation on:
Pascal's Triangle
A fascinating pattern which can have as many equal number of rows and columns as the user wants. 👍
Do try it and enjoy! :)
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


docs:  Config for deptry incorrect

In https://github.com/fpgmaas/cookiecutter-uv/blob/main/docs/features/linting.md, the configuration listed under deptry is just a copy/paste of the configuration for mypy above.

Oops, thanks for raising this! Will update the page :)

Solved by #10


Ch license

Adding Apache 2.0 License for public use.

@federicoaaguirre, I think we can revert commit ec4737b so Github will be able to auto-detect the license and show it in the GUI, like in this repo.


build IJulia failed with Julia 1.0

I could not add IJulia, and got the below error, I know my internet connection is very bad, not sure if this has any impact:

(v1.0) pkg> build IJulia
Building Conda ──→ ~/.julia/packages/Conda/m7vem/deps/build.log
Building ZMQ ────→ ~/.julia/packages/ZMQ/ABGOx/deps/build.log
Building MbedTLS → ~/.julia/packages/MbedTLS/Qo8TN/deps/build.log
Building IJulia ─→ ~/.julia/packages/IJulia/k5o7j/deps/build.log
┌ Error: Error building IJulia:
│   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
│                                  Dload  Upload   Total   Spent    Left  Speed
10 36.2M   10 4078k    0     0  19833      0  0:31:56  0:03:30  0:28:26     0
│ curl: (56) LibreSSL SSL_read: SSL_ERROR_SYSCALL, errno 54
│ [ Info: Installing Jupyter via the Conda package.
│ [ Info: Downloading miniconda installer ...
│ ERROR: LoadError: failed process: Process(curl -g -L -f -o /Users/hasan/.julia/packages/Conda/m7vem/deps/usr/installer.sh https://repo.continuum.io/miniconda/Miniconda3-latest-MacOSX-x86_64.sh, ProcessExited(56)) [56]
│ Stacktrace:
│  [1] error(::String, ::Base.Process, ::String, ::Int64, ::String) at ./error.jl:42
│  [2] pipeline_error at ./process.jl:695 [inlined]
│  [3] #run#505(::Bool, ::Function, ::Cmd) at ./process.jl:653
│  [4] run at ./process.jl:651 [inlined]
│  [5] download(::String, ::String) at ./download.jl:48
│  [6] macro expansion at ./logging.jl:311 [inlined]
│  [7] _install_conda(::String, ::Bool) at /Users/hasan/.julia/packages/Conda/m7vem/src/Conda.jl:146
│  [8] _install_conda(::String) at /Users/hasan/.julia/packages/Conda/m7vem/src/Conda.jl:145
│  [9] runconda(::Cmd, ::String) at /Users/hasan/.julia/packages/Conda/m7vem/src/Conda.jl:111
│  [10] add at /Users/hasan/.julia/packages/Conda/m7vem/src/Conda.jl:174 [inlined] (repeats 2 times)
│  [11] top-level scope at logging.jl:311
│  [12] top-level scope at /Users/hasan/.julia/packages/IJulia/k5o7j/deps/build.jl:38
│  [13] include at ./boot.jl:317 [inlined]
│  [14] include_relative(::Module, ::String) at ./loading.jl:1038
│  [15] include(::Module, ::String) at ./sysimg.jl:29
│  [16] include(::String) at ./client.jl:388
│  [17] top-level scope at none:0
│ in expression starting at /Users/hasan/.julia/packages/IJulia/k5o7j/deps/build.jl:9
└ @ Pkg.Operations /Users/osx/buildbot/slave/package_osx64/build/usr/share/julia/stdlib/v1.0/Pkg/src/Operations.jl:1068

using Pkg
Pkg.build("IJulia")

Works for me? (I don't know about your (v1.0) pkg> build IJulia however..)

The error
Info: Downloading miniconda installer ...
 ERROR: LoadError: failed process: Process(curl -g -L -f -o  …

means that the download is failing, which indicates a problem with your network connection.  You could try again somewhere you have a better network connection.
build IJulia works fine on my Mac with 1.0.


Traning code of diffusion model release

Hi~ Thanks for your good work! I am quite interesting in it!
I noticed that you have uploaded lots of codes except the training part of diffusion model.
Do you plan to release the training code of diffusion model? if yes, when? Thank you!

Hi @zmj1203 thanks for your interest of our work! We intend to release the training code of diffusion model in mid Sep.


Types of property 'xxx' are incompatible

I`m switching from TS 0.9.5 to 1.1 CTP and have this simplified .d.ts:
interface FooArray<T> extends Foo<T[]> {
    subscribeTo(transform?: (val: any) => T): FooArray<T>;
}
interface Foo<T> {
    subscribeTo(transform?: (val: any) => T): Foo<T>;
}

Compilation has failed failed with:
error TS2429: Interface 'FooArray' incorrectly extends interface 'Foo<T[]>':
Types of property 'subscribeTo' are incompatible:
Type '(transform?: (val: any) => T) => Foo' is not assignable to type '(transform?: (val: any) => T[]) => Foo<T[]>':
Types of parameters 'transform' and 'transform' are incompatible:
Type '(val: any) => T' is not assignable to type '(val: any) => T[]':
Type 'T' is not assignable to type 'T[]':
Property 'length' is missing in type '{}'.
Note that the TS 0.9.5 compiles the below code while 1.0/1.1-CTP does not.
Questions:

Shouldn't FooArray.subscribeTo should be treated as an overload and added to the FooArray`s contract? I.e. FooArray will have two subscribeTo methods.
How to fix it?

@RyanCavanaugh Any specific reason for that? Makes it a copy/paste job to extend+overload interface members...

resolving overloads is order dependent, not merging results in a more predictable system.


[SPARK-31233][Core] Enhance RpcTimeoutException Log Message

What changes were proposed in this pull request?

askAbortable method throw TimeoutException while it does no complete in time. Currently, the error message contains null as remoteAddr when receiver is in client mode.
This change is to print out correct rpcAddress instead of null in the error message.
Why are the changes needed?

It provides the address of an endpoint which does not reply in time. It helps users to find slow executors when timeout happens.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Add a unit test.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Please revert the unnecessary code reformat changes.

OK to test

Hi @jiangxb1987 ,
it's reverted.

QQ: Under which case would remoteAddr be assigned a null value?

@jiangxb1987
Here is the Comment for class RpcEndpointAddress.

The rpcAddress may be null, in which case the endpoint is registered via a client-only

connection and can only be reached via the client that sent the endpoint reference.

rpcAddress is init by the following method
 override lazy val address: RpcAddress = {     if (server != null) RpcAddress(host, server.getPort()) else null   } and server is only init once it is not client mode
(if !config.clientMode)

ok to test

Test build #120695 has started for PR 28002 at commit 81b425a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/25394/
Test PASSed.

Test build #120695 has finished for PR 28002 at commit 81b425a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/120695/
Test PASSed.

Hi, @Huang-yi-3456 . Did you push your last commit?

Hi @dongjoon-hyun ,
Yes I did. It is 9c50b8cdc26cc8d3172a8218d8debd09c3288556 on my branch. But Github for some reason has not synced it here.

@dongjoon-hyun Please kindly review the changes, thanks.

Test build #120739 has started for PR 28002 at commit 524fe5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/25437/
Test PASSed.

Test build #120739 has finished for PR 28002 at commit 524fe5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/120739/
Test PASSed.

@jiangxb1987  @dongjoon-hyun
Could you please let me know if there is anything else required for this PR?
Thanks.

Test build #121596 has started for PR 28002 at commit f30ebc0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/26276/
Test PASSed.

Thank you for updating, @Huang-yi-3456 . The last commit is a string change in test suite and I verified that.

@Huang-yi-3456 . You are added to the Apache Spark contributor group and SPARK-31233 is assigned to you. Thank you again.

Test build #121596 has finished for PR 28002 at commit f30ebc0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121596/
Test PASSed.


[FLINK-17211] Enable JUnit 4 tests in flink-sql-parser module

What is the purpose of the change
Added junit 4 vintage engine to run junit 4 tests in junit 5 environment.
Verifying this change
Execute mvn clean install in flink-sql-parser module. Check these tests are executed:

org.apache.flink.sql.parser.TableApiIdentifierParsingTest
org.apache.flink.sql.parser.FlinkDDLDataTypeTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @danny0405


Outputting auxiliary information

Hi Pawel-
Since the Nifti format cannot hold information about time frames or the tracer and many modelling software needs that information, would it be possible to create a BIDS sidecar or a SIF file as part of the output to mmrchain so that it would be easier to use these files as inputs into tracer kinetics programs?
Many thanks
David Cash

Pawel and David:
To gather some information regarding time frames and tracer characteristics, I’ve used pydicom and interfile packages in my own methods that call NIPET.   Working codes are at:
https://github.com/jjleewustledu/NiftyPETy/blob/master/respet/recon/reconstruction.py <https://github.com/jjleewustledu/NiftyPETy/blob/master/respet/recon/reconstruction.py>
https://github.com/jjleewustledu/NiftyPETy/blob/master/tests/test_reconstruction.py <https://github.com/jjleewustledu/NiftyPETy/blob/master/tests/test_reconstruction.py>

Method createDynamic demonstrates the essential ideas.   Please note that these methods are embedded in a rather elaborate scheme for aligning NAC PET frames, registering mu-maps to each NAC frame and then creating AC frames.   Only the subset of methods called by createDynamic are likely to be of interest.
John

On May 2, 2019, at 9:56 AM, davecash75<EMAIL_ADDRESS>wrote:
Hi Pawel-
Since the Nifti format cannot hold information about time frames or the tracer and many modelling software needs that information, would it be possible to create a BIDS sidecar or a SIF file as part of the output to mmrchain so that it would be easier to use these files as inputs into tracer kinetics programs?
Many thanks
David Cash
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/pjmark/NIPET/issues/6, or mute the thread https://github.com/notifications/unsubscribe-auth/AD4SZZ2M6KUYSOGIAQSAIYLPTL6IHANCNFSM4HJ7EQPQ.

Dave, this is my plan to use BIDS for this, among other reasons.
John, is your pipeline compatible with BIDS?
Cheers,
Pawel

Pawel,
I’ve not seen many specifications for PET in BIDS and I may be missing significant BIDS requirements.   Have you any requirements lists that you advocate?
I’ve tried to follow BIDS except for circumstances for which there is a conflict with idioms from XNAT, in which case I use the XNAT idioms.   The json information I record is specific to my scheme for registering mu-maps to time-frames.   I’ll do a more thorough comparison with BIDS specifications and get back to you.
John

On May 16, 2019, at 1:04 PM, Pawel<EMAIL_ADDRESS>wrote:
Dave, this is my plan to use BIDS for this, among other reasons.
John, is your pipeline compatible with BIDS?
Cheers,
Pawel
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/pjmark/NIPET/issues/6?email_source=notifications&email_token=AD4SZZ2PFOREC2YSTQ3UDADPVWO2VA5CNFSM4HJ7EQP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVSTPWA#issuecomment-493172696, or mute the thread https://github.com/notifications/unsubscribe-auth/AD4SZZ4UWUAKXPEESDEK5ATPVWO2VANCNFSM4HJ7EQPQ.

Hi John-
The BIDS format for PET is indeed a work in progress, so I think some interim decisions about how best to fill in the auxiliary metadata will have to be taken. I think APPIAN has a reasonable looking first stab at what should be included and that is what I have been working to in terms of the information I have been filling in.
@pjmark I see some software that stores the prompts, randoms and trues in order to generate weights for non-linear fitting in kinetic modelling. Is that something that also be added to BIDS?

Some of my projects may benefit from APPIAN.  I’ll fork NIPET and see what I can do.
John

On May 17, 2019, at 05:58, davecash75<EMAIL_ADDRESS>wrote:
Hi John-
The BIDS format for PET is indeed a work in progress, so I think some interim decisions about how best to fill in the auxiliary metadata will have to be taken. I think APPIAN has a reasonable looking first stab at what should be included and that is what I have been working to in terms of the information I have been filling in.
@pjmark I see some software that stores the prompts, randoms and trues in order to generate weights for non-linear fitting in kinetic modelling. Is that something that also be added to BIDS?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


How can we change the default logging location

This document https://ballerinalang.org/docs/by-example/log-api specifies that the default logs are logged in the console. So how can we change this default behavior?

At the moment, the way to change the default behaviour is by making the necessary configurations through <BALLERINA_HOME>/bre/conf/logging.properties file. But this will be removed in near future since this is a config file for JDK logging and since we will be introducing a Ballerina config file and Ballerina specific log configurations through that config file.

Ballerina user level logs will be logged to the console. If there is a need to get these to a file, the user will have to redirect the logs to a file.

As per the above comment, I am closing this issue.


Wrong credentials or url crash node red instance

When I insert wrong credentials my node red docker container crashes. I have to manually restart it
node-red_1  |   Root causes:
node-red_1  |           security_exception: unable to authenticate with provided credentials and anonymous access is not allowed for this request
node-red_1  |     at SniffingTransport.request (/data/node_modules/@elastic/transport/lib/Transport.js:479:27)
node-red_1  |     at processTicksAndRejections (node:internal/process/task_queues:96:5)
node-red_1  |     at async Client.SearchApi [as search] (/data/node_modules/@elastic/elasticsearch/lib/api/api/search.js:66:12)
node-red_1  |     at async Helpers.scrollSearch (/data/node_modules/@elastic/elasticsearch/lib/helpers.js:115:24)
node-red_1  |     at async Search._inputCallback (/data/node_modules/nrc-elasticsearch-nodes/doc/search.js:63:38)

Hi.
This was tricky to recreate, however looks like the result of uncaught errors which were difficult to catch with the nest of Promises and async functions.
Now in 0.3.16 the connection node tried to get the cluster info, and only then creates a .client() method for other nodes to use. Until then a dummy .client() is provided that will emit a "Not connected" error and return undefined.
The point being: any "Tail" nodes will likely fail on their first attempt, because they will immediately try to search before the connection node has setup the connection. If the connection setup is valid, the Tail node will succeed on its second scheduled attempt, as by then then the connection node setup should be complete.
Oh the fun of the async world. Maybe Node-red connection nodes have a way to signal they are not ready, such was not apparent to me.
Ultimately incorrect connection info should no long result in an uncaught error.


:sparkles: feat(divider): add dashed variant

Summary

Use Case

Preferred Solution
No response
Additional Information
No response
Will this feature introduce breaking changes?
None
Code of Conduct

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

/cib


✨ Add support for TLS with custom CA certificate in BMO

What this PR does / why we need it:
This commit modifies the config folder to deploy BMO with a
custom CA cert to interact with Ironic. The CA cert must be
provided as a base64 encoded string without line wrap in
the IRONIC_CA_CERT_B64 environment variable for clusterctl

/test-integration

/test-integration

/test-integration

/test-integration

/test-integration

/test-integration

That was a mistake, sorry
/test-integration

/close
replaced by #134

re-opening for a more modular review

/retest

/test golint

/test-integration

/test-integration


Would it be possible for you to dump a small data file for use with the examples?

I'd like to run the example code to step through the code to better understand how to use it and how it works. Could you provide the data file used in samples or a link to it, or some other small dataset and code to run the algorithm on? Looks like an awesome tool, and pretty simple to use.

Ah, that makes total sense. The data file was mentioned in the talk for the release of the library here: https://www.youtube.com/watch?v=VINCQghQRuM, but it really should be added here. I'll probably add a PR for this later today.


REST endpoint to allow users to test their XDT

It is quite hard to verify that XDT that you wrote will work the way you expect.   In order to test, one need to get the XDT binary, download the applicationHost.config from one of their sites (not straight forward), write console app codes to do a transform and check resulting config.
The idea I have is a way for user to HTTP POST XDT to one of extension endpoint (may or may not be kudu) and the result is success/fail + the transformed applicationHost.config.    Think of it like a similar facility for server-side generate deployment script.

Hi
Kudu will continue to run in Azure App Service. However, this repo will no longer be maintained.
If the problem persists and is related to running on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Update test values for 1D discontinuous examples

This alters the compute_coefficients in the 1D problem setup using prevfloat or nextfloat to set truly discontinuous initial data. Doing so causes most tests that have discontinuous initial conditions (like an EC elixir) to fail and require updates to the test reference values. Additionally, so elixirs may need to adjust the cfl value to be stable.
I am not sure if additional modifications are required to have DGMulti in 1D also set truly discontinuous data.
I updated the Burgers' and ideal GLM-MHD tests on TreeMesh1D to get things started. The plan is that we can update the remaining tests as a team effort to take place after one of the Trixi meetings.
Below is a current screenshot of all the failing tests for quicker reference of the tests that need updated.

TODO

[ ] Update tests

[ ] elixir_euler_density_wave.jl
[ ] elixir_euler_ec.jl
[ ] elixir_euler_ec.jl with flux_kennedy_gruber
[ ] elixir_euler_ec.jl with flux_shima_etal
[ ] elixir_euler_ec.jl with flux_chandrashekar
[ ] elixir_euler_ec.jl with flux_hll
[ ] elixir_euler_shockcapturing.jl
[ ] elixir_euler_sedov_blast_wave_pure_fv.jl
[ ] elixir_euler_blast_wave.jl
[ ] elixir_euler_blast_wave_neuralnetwork_perssonperaire.jl
[ ] elixir_euler_blast_wave_neuralnetwork_rayhesthaven.jl
[ ] elixir_eulermulti_ec.jl
[ ] elixir_eulermulti_es.jl
[ ] elixir_mhdmulti_ec.jl
[ ] elixir_mhdmulti_ec.jl with flux_derigs_etal
[ ] elixir_mhdmulti_es.jl
[ ] elixir_mhdmulti_briowu_shock_tube.jl
[ ] Nonconservative terms in 1D (linear advection)

[ ] Add NEWS.md

DGMulti might need updating but since it specializes on compute_coefficients itself, this PR shouldn't affect its tests.

I added a list to the first post. It's just 2 tests for each of us if all join the micro-hackathon next week.

I thought of the following actually really simple approach to make this effort a bit easier:
I created this file by substituting the corresponding part starting here by

this:
    save_errors_fname = "errors.txt"
    open(save_errors_fname, "a") do io
      println(io, joinpath(splitpath($elixir)[end - 1], basename($elixir)))
    end

    # evaluate examples in the scope of the module they're called from
    @test_nowarn_mod trixi_include(@__MODULE__, $elixir; $kwargs...) additional_ignore_content

    # if present, compare l2 and linf errors against reference values
    if !$coverage && (!isnothing($l2) || !isnothing($linf))
      l2_measured, linf_measured = analysis_callback(sol)

      if Trixi.mpi_isroot() && !isnothing($l2)
        @test length($l2) == length(l2_measured)
        is_equal = true
        for (l2_expected, l2_actual) in zip($l2, l2_measured)
          @test isapprox(l2_expected, l2_actual, atol=$atol, rtol=$rtol)
          if !isapprox(l2_expected, l2_actual, atol=$atol, rtol=$rtol)
            is_equal = false
          end
        end
        if !is_equal
          open(save_errors_fname, "a") do io
            println(io, "      l2   = $l2_measured,")
          end
        end
      end

      if Trixi.mpi_isroot() && !isnothing($linf)
        @test length($linf) == length(linf_measured)
        is_equal = true
        for (linf_expected, linf_actual) in zip($linf, linf_measured)
          @test isapprox(linf_expected, linf_actual, atol=$atol, rtol=$rtol)
          if !isapprox(linf_expected, linf_actual, atol=$atol, rtol=$rtol)
            is_equal = false
          end
        end
        if !is_equal
          open(save_errors_fname, "a") do io
            println(io, "      linf = $linf_measured")
          end
        end
      end
    end

and then running:
using Test

TEST_DIR = "test"
run_files = ["test_tree_1d_euler.jl", "test_tree_1d_eulermulti.jl", "test_tree_1d_mhdmulti.jl"]
save_errors_fname = "errors.txt"
isfile(save_errors_fname) && rm(save_errors_fname)
@testset "Test files" begin
  for file in run_files
    open(save_errors_fname, "a") do io
        println(io, "Running file $file...")
    end
    include(joinpath(TEST_DIR, file))
    open(save_errors_fname, "a") do io
        println(io, "-"^100)
    end
  end
end

This way we get a plain text file that contains all the errors we need to update (and not much more we would have to scroll through when running the tests from the REPL!). From looking at the numbers no error seems to explode (like elixir_burgers_shock .jl did before the cfl was adapted by @andrewwinters5000) and no test fails because of a DomainError or similar (like elixir_mhd_briowu_shock_tube.jl did before cfl was adapted). Therefore, I assume for the all of the other tests we don't run into stability issues and cfl doesn't need to be adjusted? Once we double-checked that everything looks fine, the only thing that remains is to just copy/paste the values from the file to the corresponding tests. I think this way we can save some time, especially if we want to do this later also for 2D and/or 3D. Any thoughts on that?
Another question: From what I understood, with this PR we don't need the workarounds like this anymore and can just check for the value of x instead of element_id, right? If so, I think this would be quite a big gain from this PR as it would make these elixirs much easier to read, more intuitive and probably avoids some confusion to new users. Additionally, these kind of elixirs can be run with MPI, which isn't possible right now.

Great, this sounds good! Thanks a lot.

From what I understood, with this PR we don't need the workarounds like this anymore and can just check for the value of x instead of element_id, right? If so, I think this would be quite a big gain from this PR as it would make these elixirs much easier to read, more intuitive and probably avoids some confusion to new users. Additionally, these kind of elixirs can be run with MPI, which isn't possible right now.

Yes, exactly - that's the idea.

From what I understood, with this PR we don't need the workarounds like this anymore and can just check for the value of x instead of element_id, right?

Yeah, for these 1D elixirs we should be able to get rid of such workarounds that rely on the element number. One just needs to extract the x edge values for the given element. This will also be helpful because the user can then decrease / increase the resolution without adjustment of the ICs provided the new resolution still has an element boundary on the appropriate x coordinates.

All tests are passing 🥳


Way to prompt for password?

I've looked through all the documentation that I can find, but I can't figure out how to make the commandline prompt for a password, instead of storing it in a file, or passing it as an argument to the commandline.
I looked at the underlying ssh2 repo, and it looks like they have a 'keyboard-interactive' support, which would allow for the prompting of password, like regular ssh does.  How can I do that for grunt-ssh?

@iron-man  Am I right. You want to start your deployment with e.g. "grunt deploy" and then the commandline should show something like "Enter password:"?
Then you type your password for the server and grunt-ssh will continue?
May you have a look at grunt-prompt.


rfc: Refine S3 Source: List Objects in A Lazy Way

brief: https://github.com/risingwavelabs/risingwave/issues/11391#issuecomment-1674478643

Personally, I like 3 the best, but it may take longer work than 2. As long as we have sufficient time, I would vote for it.

close this for a new design combined with https://github.com/risingwavelabs/rfcs/pull/22


Tree View Fails to load when complex _id data types are present

When a dataset contains _id fields with more complex data types, the Tree View fails to render. This could be caused by:

Erroneous _id extraction logic.
Incorrect assumptions that _id is always of a specific type.

Task:

Investigate the cause of the issue.
Implement a fix to handle complex _id data types correctly.

Data in the Table View:

after switching to the Tree View:

I was curious about this one, so I tried replicating this demo with just what I see in the print screen above. The issue might be caused by the fact that the SlickGrid DataView requires unique "id" and I'm not sure if your Id in this case is _id? If it is then multiple entries as objects instead of strings might react in different ways. In order to replicate a working demo, I added a formatter in the _id column to deal with the string or object data and the biggest change to get a working demo was to modify the data output by using a map to provide truly unique Ids as string.
So here's the 2 modifications I've done
columnDefinitions = [
  {
        id: '_id', name: '_id', field: '_id', width: 220,
        filterable: true, sortable: true, exportWithFormatter: false,
        formatter: Formatters.multiple,
        params: {
          formatters: [
            // 1st formatter for ID
            ((_row, _cell, val) => {
              if (typeof val === 'object') {
                return JSON.stringify(val); // display stringified object
              }
              return val;
            }) as Formatter,
            // 2nd formatter for Tree Data icons
            Formatters.tree
          ]
        }
      },
      {
        id: 'type', name: 'type', field: '_id', minWidth: 90, filterable: true,
        formatter: (_row, _cell, val) => typeof val === 'string' ? 'ID' : '{}'
      },
];

and the use of map to add a unique Id as string
loadData() {
    const data = [
      { _id: '667591e', parentId: null },
      { _id: '67224b2', parentId: null },
      { _id: { mySpecialID: '223344' }, parentId: null },
      { _id: { mySpecialID: '424242' }, parentId: '667591e' },
    ];

    // make sure that the "id" is unique by using map
    const output = data.map((entry) => ({
      _id: entry._id,
      id: typeof entry._id === 'string' ? entry._id : entry._id['mySpecialID'],
      parentId: entry.parentId,
    }));

    console.log('data', output);
    return output;
  }

You can see a working Stackblitz of this change at this link (I modified the Slickgrid-React Example 27, so you'll need to route to that example to see it live)
https://stackblitz.com/edit/github-bnyowcpe?file=src%2Fexamples%2Fslickgrid%2FExample27.tsx


Add test for $.ready()

@wcastand
can you fix the indentation after your describe block?
It looks like the tests so far just test the triggering of the Promise returned by the $.ready event.
Under the covers it looks like ready is triggered by DOMContentLoaded event.
context.addEventListener("DOMContentLoaded", function(){
  resolve();
});

it would be great to have additional tests to ensure that the DOMContentLoaded event does indeed trigger the Bliss ready method to resolve the promise.
Also you could test the other usage of ready to handle a retuned Promise.
var promise = $.ready([context])

No need to close this pull request though, just keep pushing to your branch and this will update.
Thanks!

Under the covers it looks like ready is triggered by DOMContentLoaded event.

If DOMContentLoaded has already fired though, it will return a resolved promise. That's the difference of $.ready vs just using addEventListener("DOMContentLoaded")

Right... It would need context.readyState stubbed in the test to "loading" in order to capture the DOMContentLoaded event which would already have passed.
Ill post an example of how to stub that tomorrow. Also @wcastand if you want to just do the other two mentioned items, ill merge and I can tinker with that one.

Hi,
That's weird, in my editor (sublime text), the indentation looks good but not once i pushed the file.
I have to admit, i'm a bit lost right now. I just made this test and it feels to simple.
But i have no clue how to test this.
If that's ok i will let you handle this one, and look how you do it, so i can learn.
(maybe you can fix the indentation issue too because on my computer the indentation is okay)

sure thing, merged.

and BTW I wasn't saying anything was wrong with the test  you wrote. Its great, thank you. I was just suggesting some other ways to test this method as well, (and was having trouble explaining myself...)
 I completely agree with you, the test is not complete but i didn't know how to make it better.

@wcastand I added a few more to your spec file
https://github.com/LeaVerou/bliss/commit/6757db66f21bf78cc0ba6f417cdbca1db5b0bc99
I ended up mocking the document  as an element so i could control the readyState and when the event was dispatched.
If you have any ideas of how to make this better the suite better, obviously go for it.


Add an interface for file compress

hi @amoghrajesh , I just created an interface for the file compress. After this PR, you can help to support .xz files. Please check xz.go file.

thanks for helping me to review this.


Fix #2374 - fix wrong indentation

Fix #2374 - fix wrong indentation
Fix crash on exporting when camera and/or light is enable for export, but not animation pointer

Is replaced by #2378
Not sure why github doesn't handle correctly rebase on 4.3


Interactive forms: make choice widget options inheritable (issue 8094)

Even though the PDF specification does not state that Opt fields are inheritable, in practice there are PDF generators that let annotations inherit the options from a parent.
The unit test fails when this patch is not applied. If you also want to test this in the viewer, use the following steps:

Download the PDF file from issue #8094.
Open the preview viewer from below.
Open the developer tools when the viewer is opened and run PDFViewerApplication.preferences.set('renderInteractiveForms', true);.
Reload the viewer and open the downloaded PDF file. Verify that both dropdowns present three options instead of zero.

Fixes #8094.

/botio-linux preview

/botio-windows lint

/botio test


About rate limit

Google has so many search sites around the world that you can randomly select one at a time to reduce the ban on access 。

@DestroyLee I doubt it would be that simple. They will still record the IP.


Calendar width in dropdown menu

I cant change size in dropdown menu. It is always looks like in attached image

any idea how to chang it

try to change the font size.


Allow user to specify which tables to use at command line.

Further fixes for issue #2751. Updates the command-line and powershell command to allow the user to make use of the framework set up in PR #3214.

:shipit:

Thanks. Checked in with fcfe8ef.


seems no highlight for jade and scss

ST2, already installed SCSS and Jade Syntax Plugins

finally it works after I did some changes in vue.tmLanguage
<string>text.jade</string>  ==>  <string>source.jade</string>
<string>source.sass</string>  ==>  <string>source.scss</string>

How to adapt this fix to ST3?

Any help on this? I'd be happy to submit a PR with a fix if I could be pointed towards an initial direction on how to do the ST2 fix for ST3

this definitely works for ST3 so until it's merged just manually extract contents of Vue Syntax Highlight.sublime-package and make changes @riotkkwok suggested

Still having this issue but I resolved it by manually selecting view > syntax > jade on all Vue files. Syntax highlighting remains for

This is working properly now. We will be releasing a new version soon which should have this included.
I'm closing this issue now, please feel free to re-open the issue if you continue experiencing problems.


[Multi Admin] TH2  commission fail

Problem
dut:k32w061
<what's wrong or missing, please include any applicable:

expected behavior
TH2 established successfully

actual behavior
TH2 established failly

steps to reproduce
1.TH1 commissioned to dut follow by:

connect -ble 3840 20202021 12344321
zcl NetworkCommissioning AddThreadNetwork 12344321 0 0 operationalDataset=hex:0e080000000000010000000300000c35060004001fffe0020811111111222222220708fd89cb00346132ee051000112233445566778899aabbccddeeff030e4f70656e54687265616444656d6f010212340410730771d0684557909e54a3554a1e4f5b0c0402a0fff8 breadcrumb=0 timeoutMs=1000
zcl NetworkCommissioning EnableNetwork 12344321 0 0 networkID=hex:1111111122222222 breadcrumb=0 timeoutMs=1000
close-ble
resolve 12344321
2.TH1 commissioned successfully and send cmd(zcl AdministratorCommissioning OpenBasicCommissioningWindow 12344321 0 0 commissioningTimeout=360)
3.dut return success
4.TH2 send cmd(connect -ip fdc2:589:2fa0:9a63:8592:7a2b:52a0:6b30 20202021 12344321) to commission to dut but fail

screenshots, images, logs, etc.

DUT_k32w0.txt
TH1_log.txt
TH2_log.txt

Let me add more test details.
Test Environment: Matter light demo Thread Device(K32W06 DK), Thread Border Router, 2 admin Controller A&B (Raspbeerypi 4B+ with TE7)
Firmware: Branch test_event_7 and patch e38b265
Issue:

Pairing Device with Controller B(TH2) always fails in first time, but will succeed the second time if try again.
I can often see this error as following on the chip-device-ctrl side

[1637902882.092410][18636:18641] CHIP:EM: OnMessageReceived failed, err = ../../src/messaging/ExchangeMgr.cpp:298: CHIP Error 0x0000002A: Invalid message type
[1637902882.129530][18636:18641] CHIP:EM: Received message of type 0x10 with protocolId (0, 0) and MessageCounter:1070751282 on exchange 57952i
[1637902882.129641][18636:18641] CHIP:EM: OnMessageReceived failed, err = ../../src/messaging/ExchangeMgr.cpp:270: CHIP Error 0x00000070: Unsolicited msg with originator bit clear
[1637902882.132342][18636:18641] CHIP:EM: Received message of type 0x10 with protocolId (0, 0) and MessageCounter:1070751283 on exchange 57952i
[1637902882.132430][18636:18641] CHIP:EM: OnMessageReceived failed, err = ../../src/messaging/ExchangeMgr.cpp:270: CHIP Error 0x00000070: Unsolicited msg with originator bit clear

Test Steps with python chip-device-ctrl:

Controller A(TH1): connect -ble 3840 20202021 12344321
Controller A(TH1): zcl NetworkCommissioning AddThreadNetwork 12344321 0 0 operationalDataset=hex:0e080000000000010000000300000c35060004001fffe0020811111111222222220708fd89cb00346132ee051000112233445566778899aabbccddeeff030e4f70656e54687265616444656d6f010212340410730771d0684557909e54a3554a1e4f5b0c0402a0fff8 breadcrumb=0 timeoutMs=1000
Controller A(TH1): zcl NetworkCommissioning EnableNetwork 12344321 0 0 networkID=hex:1111111122222222 breadcrumb=0 timeoutMs=1000
Controller A(TH1): close-ble
Controller A(TH1): resolve 12344321
Controller A(TH1): zcl AdministratorCommissioning OpenBasicCommissioningWindow 12344321 0 0 commissioningTimeout=360
**Controller B(TH2):**connect -ip fdc2:589:2fa0:9a63:8592:7a2b:52a0:6b30 20202021 12344321
Second Commissioning Failure with Controller B(TH2)
Try the same command again.
Controller B(TH2): connect -ip fdc2:589:2fa0:9a63:8592:7a2b:52a0:6b30 20202021 12344321
It will succeed this time.

Log:
TH1_log_20211126.txt
TH2_log_20211126.txt
DUT_k32w061.txt

Failure of a test case on one platform. @doru91 please triage for K32W.

Update the latest info. With TE7.5, this issue has not been reproduced but other issues were found.
Test Environment: Matter light demo Thread Device(K32W06 DK), Thread Border Router, 2 admin Controller A&B (Raspbeerypi 4B+ with TE7.5)
Firmware SDK: TE7.5 Branch b0af6ba42b6ce494748f18ecf3e054a400b8f45b
K32W06 platform does not support hardware acceleration engine. Currently, the whole commissioning time is up to 30 seconds and the establishment time of CASE secure channel is up to 10 seconds (with mbedtls library).
To solve this issue, the temporary solution needs to increase the timeout of chip-tool.
https://github.com/project-chip/connectedhomeip/blob/master/examples/chip-tool/commands/pairing/OpenCommissioningWindowCommand.h#L40
https://github.com/project-chip/connectedhomeip/blob/master/examples/chip-tool/commands/clusters/ModelCommand.h#L45
NXP team is still optimizing this issue.

duplicate with #18084. Closing.


[BUG] Backup on Android leads to buffer error

Hi,
I have a repo which contains a few Pictures. Its about 40 MB in size in total.
The strange thing is that sometimes it works to do a backup so the push completes and everything is completely in snyc.
And then out of nowhere when trying to do a backup it says initializing push and directly after that buffer error.
It's like yesterday evening it is working then the next morning it says buffer error and then few hours later it's working again. Only adding few lines of text between the backup attempts.
Could I debug this somehow or is my repo already too big?

Bug is on Android and Plugin version is 2.1.0

I haven't done much research, but I guess it depends on the available ram on your device. E.g. how many other apps you have opened. 40mb isn't that big. I thought the limit would be higher.

Thats a good point. Will try to clean the cache before doing a backup and see how it goes

Since there isn't really anything, I can do to fix this. I'm closing this now.

Hi, I am getting same buffer error when attempting to push, and my repo is tiny , the commit said 10 files changed,  and device memory is big. I also closed all other apps. Any suggestions on what to do? I hope I can avoid cloning again

@Vinzent03 Hi, I am getting same buffer error when attempting to push, and my repo is tiny , the commit said 10 files changed,  and device memory is big. I also closed all other apps. Any suggestions on what to do? I hope I can avoid cloning again

Just wait and try again. Its working again after a while but its definately buggy. Sad that this is not investigated any further..

Also, clearing cache after getting the error didnt help.

I have the same problem with a small repo. Can't get mobile to sync without buffer error. Have cleared the cache with no luck.

@Vinzent03 maybe some advises on one could debug this?
It seems like there are a few people experiencing the issue

Or should I open the bug elsewhere?
Any help would be greatly appreciated

As said before I'm really sorry it's not working on your device, but I think there is nothing I can do. The underlying lib is not very memory saving. I know that's bad for a mobile. plugin, but that's the only way I found to make it work for at least some users.

Ok no worries than. It works most of the time though
Thanks for your work and sharing

Leaving this here in case someone got the same troubles and looks for a solution.
When you exclude the Obsidian app from memory management in Android settings, the plugin works just fine. I can do this in device management settings but this setting may dependt on the device.

Could you please explain further? Can't seem to find how

@Takaishisama
Succeeded to exclude the app from memory management, it is now working as expected. It depends of your android device, here how you can find the settings screen:

I'm experiencing the same problem with a small vault and small files.
I am using a Samsung Galaxy S10, Android 12.
The sync worked 1 or 2 times, now there's a buffer error every time and it's not pushing/pulling.
Clearing the app cache didn't help.
I checked the memory usage of the app, Android says max usage has been 333 MB, which seems normal. Average usage of 107 MB seems a bit high, but still fine.
I am very confused by the proposed solution by @0xShkk and @blqke.
@blqke you are changing the battery usage in your screenshot, which should not have anything to do with memory management?!
@0xShkk is that what you meant? I cannot find a setting to change memory usage behaviour of an app.
Also, might the sync of the app settings between PC and Android be a problem? Should these be excluded using .gitignore?
Thanks for your help!

@penfold00
I meant exactly this. It may be that this setting can't be found on all devices but just guessing here. For me it still works fine

As said before I'm really sorry it's not working on your device, but I think there is nothing I can do. The underlying lib is not very memory saving. I know that's bad for a mobile. plugin, but that's the only way I found to make it work for at least some users.

isomorphic git supports reducing the clone depth: https://isomorphic-git.org/docs/en/clone
You may find this makes things less memory intensive as there is no need to clone the entire git history - particularly if users have accidentally have larger files in their git history

@HarvsG Great idea. Available in v2.17.0

So it seems this error doesn't have a fix yet which kinda sucks since I have the same problem. Cloning a vault crashes the app. Tried this battery option even though it is strange like the other person said and it didn't work. Also tried turning off every other app that was running.

I am having this problem 100% of the time. My repo is only 1.5MB. I am on Android. I have tried the memory and the battery suggestions above with no success.
@Vinzent03, could you share what other configurations or conditions might trigger the "buffer error"?  Is there any obsidian-git logging I could capture?
I have searched isomorphic-git for "buffer error" and it does not appear on the web or in the obsidian-git repo.
Thank you!

FYI @Vinzent03, and for any others that may land here... For my case, it was a "user shot self in foot", "user solved self" PEBKAC case... I was curious about what other factors might contribute to this error.  I can tell you one...
Because of other errors I had experienced, I had removed and re-cloned my repo more than once. In one reconfiguration, I moved my repo to a subfolder of its original location.  So, originally my repo was in something like folder-a/notes and I moved it into folder-a/newhome/notes.  To do this, I removed my folder-a/notes directory, then re-cloned into folder-a/newhome/notes.  MY problem was that I left behind the folder-a/.obsidian directory.... and so, I ended up with two .obsidian folders in a hierarchy... so, as we know, the obsidian-git configs are under the .obsidian/plugins directory... and I assume there is an "up the chain" precedence in obsidian-git and with two conflicting configs, that was the cause of my "buffer error".   After removing my parent directory .obsidian folder, my issue was resolved.

First off thanks to the maintainers for this fantastic plugin.
I just wanted to comment for anyone who comes across this issue to say I've found a workaround that is currently working for me (Samsung Galaxy S21, Android 13).
In Settings -> Battery and Device Care -> Memory I increased the RAM Plus (which I assume is swap) to 8GB (seems overkill) and added Obsidian to the "Excluded apps" list. I haven't had any buffer error issues since. Still watching #572 intently though!

Wondering if people could report the versions of Android, phone, and RAM with their problem.
I'm able to do an initial sync from remote repo, but not a backup after making a small change.
buffer error
Pixel 5, 8GB RAM, Android 14 with Nov 5, 2023 security patch;
Obsidian maxed out at 3.9 GB of RAM usage.
How to Check RAM on Android 14?

Also getting this issue. Pixel 4a with 6 GB ram.
Obsidian shows as using only 280 MB of ram. I've also set it to unrestricted.
I've had this happen twice. The first one I don't remember what caused it, but the second seems to be caused by attempting to pull when the remote contains changes that modify a file that is already locally modified.
Deleting the vault and re-cloning the repo has fixed it in both cases for me. Restarting Obsidian doesn't seem to do anything (I haven't tried force killing the app though).


ADS: Mic and Camera permissions dialogs migration

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2587

PR #2649   👈
PR #2648
PR #2642
PR #2636
PR #2594
PR #2610
PR #2609
PR #2606
PR #2590
PR #2629
PR #2588

This comment was auto-generated by Graphite.


BGL PR bounty hunt

To get more people involved and provide motivation, we are announcing
Bitgesell Pull Request bounty hunt!
The rules are simple:

You can create any reasonable pull request that may contain any modifications, including,
but not limited to:

Refactoring and simplification;
Test fixes (1 test group/file fixed by single PR counts!);
Cleanup of features that are no longer used (e.g. non-segwit transactions);
Documentation and comments (but if no code changes then some reasonable amount of changes should be done to count);
Bug fixes;

Every PR would be paid $100 in USDT upon approval, larger PRs could receive bigger payouts.
Interested people are welcome to become maintainers of the project;

(multi-contributor bounty would be recreated when possible on Gitcoin soon)
(I miscreated issue on personal repo, please make PRs and fork them from the repo github.com/bitgesellofficial/bitgesell), sorry created issue in this one by mistake

hi @wu-emma
I notice the main project has no open issues.
I created a couple of pull requests yesterday to fix issues I came across while compiling but it was all minor stuff. Are there specific issues you would like to see fixed?

hi @wu-emma. @van-orton reached out to me about a week ago but since I haven’t heard back from him since, I’m not sure if the PRs I’m opening are helpful or wanted. I've added a couple more pull requests and opened issue for a problem with the functional tests I'd like to work on tackling over the next couple of days. Not sure if I should proceed or first wait for the open PRs to be reviewed. I now have a few open PRs.
I hope to hear from someone soon.

On 26 May,2021, at 18:16, Gitcoin.co Bot @.***> wrote:
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
@tkembo https://github.com/tkembo due to inactivity, we have escalated this issue https://gitcoin.co/issue/wu-emma/bitgesell/32/100025731 to Gitcoin's moderation team. Let us know if you believe this has been done in error!
reminder (3 days)
escalation to mods (6 days)
Funders only: Snooze warnings for 1 day https://gitcoin.co/issue/wu-emma/bitgesell/32/100025731?snooze=1 | 3 days https://gitcoin.co/issue/wu-emma/bitgesell/32/100025731?snooze=3 | 5 days https://gitcoin.co/issue/wu-emma/bitgesell/32/100025731?snooze=5 | 10 days https://gitcoin.co/issue/wu-emma/bitgesell/32/100025731?snooze=10 | 100 days https://gitcoin.co/issue/wu-emma/bitgesell/32/100025731?snooze=100
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/wu-emma/bitgesell/issues/32#issuecomment-848909136, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOOCLQ7YWSF6MZSZNZSKMTTPUNE3ANCNFSM4432KZHQ.

I have the following merge requests open at present:

https://github.com/BitgesellOfficial/bitgesell/pull/52 - paid ✔
https://github.com/BitgesellOfficial/bitgesell/pull/53 - awaiting review
https://github.com/BitgesellOfficial/bitgesell/pull/54 - awaiting review
https://github.com/BitgesellOfficial/bitgesell/pull/55 - awaiting review
https://github.com/BitgesellOfficial/bitgesell/pull/57 - awaiting review-
https://github.com/BitgesellOfficial/bitgesell/pull/58 - awaiting review
https://github.com/BitgesellOfficial/bitgesell/pull/59 - awaiting review

Additionally, I have a MR to join some of the above changes into Janus branch, where development appears to be ahead of master:

https://github.com/BitgesellOfficial/bitgesell/pull/56

I have further contributions already committed locally, and am eager to submit them, but have not yet done so pending resolution of the discussion at https://github.com/BitgesellOfficial/bitgesell/pull/52#issuecomment-911791689
I would be grateful if maintainers could find time to have a look at the MRs and discussion above, so I can proceed further.

@slowriot
Great job. I will review your PRs very soon.

Hello @janus  Currently I am going through PRs , can you suggest some good first issues which i can start looking in to.

@vijaydasmp  Happy to have you here.
Check any of these:

miner_tests.cpp
transaction_tests

They may look overwhelming but they would help you understand the inner workings of BTC/BGL.


Tar source plugin: needs better behavior at stage() time

In GitLab by @tristanvb on May 12, 2017, 09:33
Most tarballs we use are release tarballs which contain a directory at the root, named as the base name of the tarball without the tar extension.
The default behavior of stage() for a tar source should be to extract the content of that directory into the target directory, not to extract the tarball base directory itself into the build dir.
Of course not every (but almost every) release tarball will follow the standard, so the tar source should have an option for working with non standard tarballs as well.

In GitLab by @tristanvb on May 14, 2017, 11:29
This is fixed with the following commit:
commit c797267bf46282723ee71c76a62daa295c23579d
Author: Tristan Van Berkom<EMAIL_ADDRESS>Date:   Sun May 14 20:23:23 2017 +0900

    tar.py source plugin: Implement base-dir glob pattern parameter

    This is to address issue 25 by allowing some flexibility of what
    directory should be extracted from the source tarball, it will
    default to '*' which will match the first directory found at the
    root of the tarball, which will work for the vast majority of
    standard source code release tarballs.

In GitLab by @tristanvb on May 14, 2017, 11:29
closed


uninitialized constant Rake::TestTask

Tried clean setup, but setting up containers failed to work. Precisely, redmine container.
redmine_1     | Migrating plugins. Please be patient, this could take a while...
redmine_1     | rake aborted!
redmine_1     | NameError: uninitialized constant Rake::TestTask
redmine_1     | /home/redmine/redmine/plugins/redmine_issue_templates/lib/tasks/test.rake:7:in `block in <top (required)>'
redmine_1     | /home/redmine/redmine/plugins/redmine_issue_templates/lib/tasks/test.rake:1:in `<top (required)>'
redmine_1     | /home/redmine/redmine/lib/tasks/redmine.rake:202:in `block in <top (required)>'
redmine_1     | /home/redmine/redmine/lib/tasks/redmine.rake:202:in `each'
redmine_1     | /home/redmine/redmine/lib/tasks/redmine.rake:202:in `<top (required)>'
redmine_1     | /home/redmine/data/tmp/bundle/ruby/2.4.0/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:656:in `block in run_tasks_blocks'
redmine_1     | /home/redmine/data/tmp/bundle/ruby/2.4.0/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:656:in `each'
redmine_1     | /home/redmine/data/tmp/bundle/ruby/2.4.0/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:656:in `run_tasks_blocks'
redmine_1     | /home/redmine/data/tmp/bundle/ruby/2.4.0/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:515:in `run_tasks_blocks'
redmine_1     | /home/redmine/data/tmp/bundle/ruby/2.4.0/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:459:in `load_tasks'
redmine_1     | /home/redmine/data/tmp/bundle/ruby/2.4.0/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:190:in `public_send'
redmine_1     | /home/redmine/data/tmp/bundle/ruby/2.4.0/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:190:in `method_missing'
redmine_1     | /home/redmine/redmine/Rakefile:7:in `<top (required)>'
redmine_1     | /home/redmine/redmine/vendor/bundle/ruby/2.4.0/gems/rake-13.0.1/exe/rake:27:in `<top (required)>'
redmine_1     | (See full trace by running task with --trace)
dockerredmine_redmine_1 exited with code 1

I've never seen this error before.  What version of the image are you running?  How are you starting it?  What does clean setup mean to you?

It looks like a plugin is causing the failure.
plugins/redmine_issue_templates

This should be reported to the plugin maintainers.

@jcormier tried few images. It breaks on ubuntu 16.04, however on Debian Buster clean run works without errors.

Hmm could you check on the status of SELinux?  It might be tripping you up. https://github.com/sameersbn/docker-redmine/issues/311#issuecomment-334913607
I haven't had any issues with ubuntu 16.04 or 18.04 as host fyi.


No environments in the code

Hi,
There doesn't seem to be any environments provided in the github repo. Can you please add the envs, since it seems that the paper has custom envs (I am assuming this is possible but please let me know if it is not)?

Hi,
That was a gitignore error. Thank you for letting us know! You should be able to see the envs now.
Please let us know if there is any other problems.

Thanks a lot.


I'm using AppIntro

Hi, I'd like my app to be listed as using appintro, it's called Aware and can be found here:
https://play.google.com/store/apps/details?id=com.bunemekyakilika.aware

Readme updated! Thanks for using the lib!

No problem, thanks for a great library!


TextBox issues

When typing text into TextBox the caret blinks in an unusual way (too fast in my case). it looks like it reset by the timer additionally to blink by the terminal itself. It seems related to upgrade to newer Avalonia

When focusing on Mid-aligned and right-aligned textboxes, the initial caret position is set wrong

MultiLine Textbox seems  does not wrap text anymore. Navigating to the right side from visible area seems not possible

this seems to work on Windows. Is this linux/macos?

this seems to work on Windows. Is this linux/macos?

Yes, on fedora. Now I'm also thinking how to easily test on windows.  Virtual machine would be enough,  but windows has strict hardware requirements.  Do i need windows 11 for testing?

You can use windows 10, but you definitely need to turn on Windows Terminal. It's built into 11, but I think you need to enable it on 10.
https://learn.microsoft.com/en-us/windows/terminal/install
Windows Terminal is where they have full support for ANSI escape codes

I would be surprised if you couldn't run in VirtualBox.


run parseInt on skip and limit parameters

MongoDB 3.2 requires skip and limit to be numbers, so we must parse it internally.
express-restify-mongoose have a similar issue. https://github.com/florianholzapfel/express-restify-mongoose/pull/216
I was able to fix locally by updating controller.js lines to add parseInt
  var limit = parseInt(req.query.limit) || 25;
  var skip = parseInt(req.query.skip) || 0;

Sorry I cannot do a pull request.

There you go!


modified script

change column name from custmr-email to email

No conflict with the base branch. Okay to merge.


Deprecation of Result.flatMap function

Description
Our use of the Result type being removed like discussed in #104, we would like to deprecate with a warning level the Result.flatMap function.
The deprecation notice should advice users to use the type converters suffixed by OrNull instead, and should indicate that this function will be removed in the next major version (see issue #125).
Dependencies
This issue is blocked by the following ones:

[x] #84
[ ] #321

Checklist

[ ] Deprecate the function.
[ ] Check the generated documentation.
[ ] Refactor code that is using the function.
[ ] Update the Unreleased section in changelog.

This experimental declaration will be removed in the next minor release by the issue #125.


i forget my password

What's the problem?
i forget my passowrd always give me invalid password

How often you face this problem ?
first time

What is the expected output? What do you see instead?

i just saw a password wrong i can't access my file and its very important
picture
What version of the product are you using? On what version of Windows?

windows 7 , ver 3.10.3
Any previous version produced the same problem ?
no

What is the language of your operating system and keyboard ?

Please provide any additional information below and attach debug.txt file
generated in C:\Program Files\LocK-A-FoLdeR or C:\Program Files
(x86)\LocK-A-FoLdeR(64bit windows).

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 29 Sep 2014 at 10:06

is this issue solved?


chore: improved debug logs in both agent and serverless-collector

refs INSTA-12427

Could you please explain how these debug logs improved the debugging process? Including some example logs would be helpful.

Could you please explain how these debug logs improved the debugging process? Including some example logs would be helpful.

Initially, the spans were logged only on instance of PayloadTooLargeError
Now, the number of span data is logged on any unexpected error that can help determine whether there is any entry span present or not. Also, it can be checked against the expected number of spans in a certain condition while debugging.
Before:

After:

chore: improved debug logs in agent sending spans #1259

This can be shipped as a fix as its a user facing change.

This change could also be marked as refactor: improved debug logs in agent sending spans. There is the option to configure lerna to accept refactor as a user facing change. Therefor we need to configure lerna to use [changelogPreset](https://github.com/conventional-changelog/conventional-changelog-config-spec/blob/master/versions/2.2.0/README.md#types). Not in the scope of this PR, but we could consider making refactor not hidden.

Dropping the plan for optimization for now as the user can use any logger in their system.
Some loggers use different methods for identifying the log level,
e.g Bunyan uses {... 20: debug}
and, Pino and Winston use string notations like 'debug', 'info' etc.
So to cover all the types of loggers it might take some effort:

We would need to start with a logger utility module.
Which has generic functions such as getLevel which knows all the loggers and their API's.


⚠ Thumbnails API Endpoint has degraded performance

In 066f8a5, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 5690 ms

Resolved: Thumbnails API Endpoint performance has improved in 39451b6 after 7 hours, 5 minutes.


Add Development support

It seems that BetterDiscord only supports PTB, Canary and Stable. I decided to change this. This should now allow you to inject to Development by npm run inject development :D

This has already been discussed multiple times and even PR'd before including #687. I don't see the point of adding this if it's just canary again.

There are few slight differences. It doesnt affect the project at all by having it (considering it is just canary, it wont need more maintenence than it) and just gives more options to people.

It not even supported by Discord, so I don't see why we should support it. https://i.imgur.com/atJukTQ.png


kinode core v0.10.0

Problem
Kinode core v0.10.0 changes app_store to app-store, amongst other breaking changes.
Solution
Ensure kit stays up-to-date.
Docs Update
None
Notes
Don't merge until https://github.com/kinode-dao/kinode/pull/588

I added support for wit 1.0 for use with runtime/process-lib 0.10

ty


Resizing simple two columns full width column flickers and reflect odd behavior

Describe the bug (required)
Resizing simple two columns full-width column flickers and reflect odd behavior, please take a look at the example in codesandbox I made.
I took the example from docs to resizeable full width table: https://codesandbox.io/s/github/tannerlinsley/react-table/tree/master/examples/full-width-resizable-table?file=/src/App.js
and Modified the columns variable to fit my needs - just a simple two columns table and when I tried to resize it, I faced tons of issues.
Take a look on the example link below:
Provide an example via Codesandbox! (required)
https://codesandbox.io/s/shy-forest-9vq8t?file=/src/App.js:0-4735
Steps To Reproduce (required)
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/shy-forest-9vq8t?file=/src/App.js:0-4735
Try to resize any of the two colums

Expected behavior (Recommended)
The column start/end to follow the cursor, but instead it just behaves oddly.
Desktop (please complete the following information):

OS: mac OS
Browser:  chrome

I'm having the same issue, I wonder if it has to do with useFlexLayout

Duplicate of #2972


Flip Dead Master to Slave

This is more a question than issue .

Next step for me was to put back old master (mysql01) back as slave for mysql02 . I could not figure out how to do that using orchestrator .
The only way i found was clear every thing and start from clean db . I am asking this because my database is around 1 TB and starting over from clean db will take too much time.
Thanks

Discussed in https://github.com/github/orchestrator/issues/190#issuecomment-306023357

You can't expect it to be come a replica because it potentially (likely, in many environments) have applied more transactions that its replicas. That is, it is more up to date than the promoted replica. As such, it will likely break on replication.

If the master and replica happen to be in complete sync when you shut down the master, then use GTID or Pseudo-GTID to position the old master (now replica) under the new master.

If this answers your question, please consider closing the issue. Thanks!

Sorry i did not get back , yes tested it using GTID and worked perfectly .


Arabic Language Support

Hi,
First of all, I'd like to express my gratitude and appreciation for the work you've put into this project. It's truly impressive and provides a valuable resource.
I have been using your project and I'm quite excited about the features it offers. However, I couldn't find information about Arabic language support. I'm particularly interested in generating pdf in Arabic.
Could you please clarify whether this project supports the Arabic language or if there are plans to include Arabic language support in the future?
If Arabic language support is not currently available, I would like to understand if there's a roadmap or any guidance on how one might contribute to adding this functionality.
Thank you again for your hard work, and I'm looking forward to your response.

Hi @MichelleMa8, thanks for taking the time to open this issue. At the moment, Arabic is not supported because pdfje is still a one-person (me) project and I don't have the knowledge to implement it. I don't expect it to be in the roadmap anytime soon.
What I do know is that Arabic glyphs in fonts should be supported, but there's probably a lot more to rendering Arabic text properly.
My suggestion would be to have a look at fpdf2, which has a large (international) community, and they explicitly mention Arabic here.

Thank you for your suggestion. I'll explore the fpdf2.
Thanks again for your time and your great work on this project!


Allow hooking permission requests

Whenever a user tries to access any restricted resource which requires a permission, a function should be called including information about what permission was requested, the user and whether the access was granted. This function then calls the registered hooks based on the requested permission and success status.
This should be incorperated to the assertPermission (or similar) function.

Why? Motivation could greatly clarify how/why this should be implemented.

@thomwiggers I believe this is so we can identify when user's are trying to access content that they don't have access too. You could make the argument that they are breaking a golden rule of trying to find code exploits.
@lawnmowerwcd If we do indeed do this then logging should only occur when the user's access is denied to avoid redundant logging. You should reword your original post to reflect this.


Can't install Roots globally with npm

Every time I try to install roots I keep getting the following error on my command line:
><EMAIL_ADDRESS>install /usr/local/lib/node_modules/roots/node_modules/chokidar/node_modules/fsevents
> node-gyp rebuild

/usr/local/lib/node_modules/npm/bin/node-gyp-bin/node-gyp: line 3: node: command not found
npm WARN optional dep failed, continuing<EMAIL_ADDRESS>/usr/local/bin/roots -> /usr/local/lib/node_modules/roots/bin/roots

><EMAIL_ADDRESS>postinstall /usr/local/lib/node_modules/roots
> node ./post_install.js

sh: node: command not found
npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "-g" "roots"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn

npm ERR<EMAIL_ADDRESS>postinstall: `node ./post_install.js`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node ./post_install.js'.
npm ERR! This is most likely a problem with the roots package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./post_install.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls roots
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/maxwellantonucci/npm-debug.log

I already have bower installed globally and have installed other dependencies before, so am not sure what's going wrong.

I'm not sure if this is relevant, but I am facing the same issue on one of my macs.
I get this error though:
npm ERR! git clone --template=/Users/debastost/.npm/_git-remotes/_templates --mirror git://github.com/jenius/st.git /Users/debastost/.npm/_git-remotes/git-github-com-jenius-st-git-75440a74bcfef19581af498df6a5644e: Cloning into bare repository '/Users/debastost/.npm/_git-remotes/git-github-com-jenius-st-git-75440a74bcfef19581af498df6a5644e'...
npm ERR! git clone --template=/Users/debastost/.npm/_git-remotes/_templates --mirror git://github.com/jenius/st.git /Users/debastost/.npm/_git-remotes/git-github-com-jenius-st-git-75440a74bcfef19581af498df6a5644e: fatal: unable to connect to github.com:
npm ERR! git clone --template=/Users/debastost/.npm/_git-remotes/_templates --mirror git://github.com/jenius/st.git /Users/debastost/.npm/_git-remotes/git-github-com-jenius-st-git-75440a74bcfef19581af498df6a5644e: github.com[0: <IP_ADDRESS>]: errno=Operation timed out
loadDep:st                ▐ ╢██████████████████████████████████░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░npm WARN install Couldn't install optional dependency: Command failed: git clone --template=/Users/debastost/.npm/_git-remotes/_templates --mirror git://github.com/jenius/st.git /Users/debastost/.npm/_git-remotes/git-github-com-jenius-st-git-75440a74bcfef19581af498df6a5644e
npm WARN install Cloning into bare repository '/Users/debastost/.npm/_git-remotes/git-github-com-jenius-st-git-75440a74bcfef19581af498df6a5644e'...
npm WARN install fatal: unable to connect to github.com:
npm WARN install github.com[0: <IP_ADDRESS>]: errno=Operation timed out

It's quite strange considering I dont have an issue with any other repos on GitHub?

Never seen this one before. Definitely not an issue with roots through haha, it's an internet thing.

Sometimes I wish the internet behaved nicely just for one whole day, just
one! I'll look deeper into it as soon as I get a chance!
On 1 Dec 2015 23:41, "Jeff Escalante"<EMAIL_ADDRESS>wrote:

Never seen this one before. Definitely not an issue with roots through
haha, it's an internet thing.
—
Reply to this email directly or view it on GitHub
https://github.com/jenius/roots/issues/686#issuecomment-160956265.

Probably not but possibly a fix: sudo chmod a+rx /usr/local/bin /usr/local/bin/node
http://yeoman.io/learning/faq.html

For anyone coming across the postinstall.js error. Update npm to the latest stable version and you should be good to go.


Unresolved module bug in react-native project

Version of EOSJS
eosjs@beta
Describe the bug
It seems it only happens in react-native project, for now.
When import the module and initiate, project shows error as:
error: bundling failed: Error: Unable to resolve module `assert` from `/path/to/project/node_modules/eosjs-ecc/lib/aes.js`: Module `assert` does not exist in the Haste module map

I'm not sure this is the problem of eosjs or issue in react-native. If somebody knows about this, please let me know.
To Reproduce
Steps to reproduce the behavior:

Apply eosjs module in react-native default project
import module
Run project

Expected behavior
Works well
Smartphone (please complete the following information):

Device: Android emulator

I have the same issue, anyone can help?!

@djKooks Have you solved the problem?

This is a known issue, eosjs-ecc does not support react-native.

@yang24201 no...it seems known issue as in comment.
@jeffreyssmith2nd Is there a plan for fix?

maybe you can choose it in react native. https://github.com/Game-X-Coin/eosjs-rn

eosjs-rn also can not operate correctly

try this  https://github.com/mvayngrib/rn-nodeify.

The current develop branch dropped eosjs-ecc and uses the elliptic library instead.


Bug: Link selection in TextFormatFloatingToolbar [Playground]

Lexical version: 0.3.8
Steps To Reproduce

Select text from link from right to left

Link to code example: https://playground.lexical.dev/
https://user-images.githubusercontent.com/43907473/180993481-dd000b55-d2e0-45b3-9111-b74fc9943d34.mov
The current behavior
TextFormatFloatingToolbar is not shown
The expected behavior
TextFormatFloatingToolbar should be shown the same way as when user selects text from left to right

Validated that this now appears to have been fixed.


scene text recognition

Thank you for your release! Could you show me a sample of how data are stored in the data folder and train.txt?
Thank you!

Thank you for your information
发自我的iPhone
------------------ Original ------------------
From: Mohamed Ali Souibgui @.>
Date: Fri,Sep 15,2023 11:19 PM
To: dali92002/SSL-OCR @.>
Cc: Yan Shu @.>, Author @.>
Subject: Re: [dali92002/SSL-OCR] scene text recognition (Issue #1)
Check this:  https://github.com/dali92002/OCR-TR#prepare-data
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


https://go.paid4link.com

Link
https://go.paid4link.com/QvKbs8np NSFW
Version
Manifest-v3
What browser(s) are you using?
Chrome browsers (includes brave, edge, opera, etc)
What platform are you using?
Computer (Windows, Linux, MacOS)
(Optional) Anything else?
No response

in the meantime, use script from bloggerpemula while waiting for the update

ty


Add SDK support for cross-env private metadata

For apps, to be able to be cloned with entire env (staging promotion to production or other Saleor multi-tenant approach), apps must somehow persist their config.
In the future, we will have dedicated solution for that
At the moment, apps can be configured to use cross-env private metadata, storing a dictionary (with prod & stage values) instead a single value.
This will be subject of future migration

@krzysztofwolski & @przlada please coop on that


Create Preorder-Traversal.md

Hey, could you add Post-order and In-order as well?!

Hey, could you add Post-order and In-order as well?!

Sure!


In Opensearch miscellaneous behaviour in Alerts

Raising this issue after discussion with AWS support person. In Opensearch alert monitor trigger condition does not work as expected. After speaking to aws support we got to know that issue seems to be specific to mustache.js and the javascript ecosystem in general. In alert trigger condition decimal number query conversion is not happening. Kindly help us to get resolved this.

Can you please provide an example of the trigger condition?
If you are using variables/fields that are integers then it is expected that the result will get round off
for example if the condition is
params.sum / params.total *100 > 70 and params = { sum: 3, total: 4 } then the result be false since 3/4 will evaluate to 0
For this to work correctly we need to cast the field to double
(double)params.sum / params.total * 100 > 70 and that will evaluate to true


Provide a closeOnEscape prop

Specifies whether the dialog should close when it has focus and the user presses the escape (ESC) key.

It already does this
https://github.com/rackt/react-modal/blob/master/lib/components/ModalPortal.js#L145

Thanks!!


Help with Code Completion

Hi,
I'm trying to use OmniSharp to code complete whenever the User types Ctrl+Space. The operation is invoked in the LSP Server, but nothing appears in VsCode. It displays "loading..." forever without any json rpc message coming from LSP Server. Here is my implementation:
    public class MyCompletionHandler : ICompletionHandler {

        public CompletionRegistrationOptions GetRegistrationOptions(CompletionCapability capability, ClientCapabilities clientCapabilities) {
            return new CompletionRegistrationOptions {
                DocumentSelector = new DocumentSelector(new DocumentFilter() { Pattern = "**/*.xyz" }),
            };
        }

        public  Task<CompletionList> Handle(CompletionParams request, CancellationToken cancellationToken) {
            List<CompletionItem> itemsList = new List<CompletionItem>();
            for (int i = 0; i < 5; i++) {
                var item = new CompletionItem {
                    Label = "Item " + i.ToString(),
                    Kind = CompletionItemKind.Class,
                    InsertText= "Item " + i.ToString(),
                    Detail="Details"
                };
                itemsList.Add(item);
            }
            var result=new CompletionList(itemsList);
            return Task.FromResult(result);
        }
    }

Am I doing something wrong? The idea in this sample is to display 5 items to the user as code complete whenever Ctrl+Space is pressed.
Any ideas? :-)
Thank you!
Bruno.

Sorry, it was my mistake. I forgot the communication between VSCode and the LSP Server is through console. Since I had some Console.WriteLines in my code in the LSP Server, I guess the procotol got lost!


ActiveFixture does not delete actual data in unload() method

ActiveFixture unload does not call resetTable() in unload() and this causes trouble when you have dependant fixtures with foreign key constraints. As ActiveFixture tries to loadFixtures(), which calls resetTable() it fails as the dependant data is not deleted. The issue description is related with #5441

I have one ActiveFixture that depends on another. I'm using Codeception and following the Guide and taking hints from yii2-app-advanced.
If there's a fatal error during a test then the rows are left in the table. Re-running the test errors thus:
yii\db\IntegrityException: SQLSTATE[23000]: Integrity constraint violation: 1451 Cannot delete or update a parent row: a foreign key constraint fails (`spinitron2`.`persona`, CONSTRAINT `persona_fk_user` FOREIGN KEY (`user_id`) REFERENCES `user` (`id`))
The SQL being executed was: DELETE FROM `user`

I guess because unloading the fixtures in yii\codeception\TestCase::setUp() does not respect the fixture dependencies.
The fix (thanks CeBe) was to add to each of my fixture classes:
public function unload()
{
    $this->resetTable();
    parent::unload();
}

you can even trigger unload methom instead of resetTable in load method... If you chose this way...

I can see no other option but having resetTable() at both load() and unload().

but we already have yii fixture/unload-all or simple yii fixture/unload

As far as I see these commands are useless now, since ActiveFixture::unload() does not deletes data from table.
Yet another solution can be moving resetTable() to unload() and modify FixtureTrait::loadFixtures() to always call FixtureTrait::unloadFixtures():
public function loadFixtures($fixtures = null)
{
    $this->unloadFixtures($fixtures);
    ...
}

Hi, If you want unload fixture in load function you should care about depends.
I read documentation that says unload remove all fixture data.
Take a look at: https://github.com/yiisoft/yii2/blob/master/framework/test/FixtureTrait.php#L33
You should clean up this thing, because you implement a lot of ways to do the same thing.

Order in fixtures() on FixtureTrait
depends property in  yii/test/Fixture

And still it does not WORK! Do you want provide reliable solution OR NOT?

The resetTable() workaround cited by @tom-- does not work for me.
This is my mysql workaround and should be added to each fixture model that has some dependant table:

<?php
namespace tests\codeception\common\fixtures;

use yii\test\ActiveFixture;

class VariationFixture extends ActiveFixture
{
    public $modelClass = 'common\models\Variation';

    public function beforeLoad() {
        parent::beforeLoad();
        $this->db->createCommand()->setSql('SET FOREIGN_KEY_CHECKS = 0')->execute();
    }

    public function afterLoad() {
        parent::afterLoad();
        $this->db->createCommand()->setSql('SET FOREIGN_KEY_CHECKS = 1')->execute();
    }
}

@slinstj thank you for your crutch. :)

@githubjeka :+1:

Is there same solution for Postgres?

What do you think about adding one more field to Fixture (public $unloadFirst = [] or something like that) which will contain links to all the dependent fixtures. Using this field it will be not too complicated to build unloading mechanism actually clearing all the tables in proper order and without any foreign key violations.
We don't need any hard changes in loading fixtures - just add check if we need to unload data in any dependent tables.
For example, fixture OrdersFixture is dependent of UserFixture. In database schema we also have foreign key not allowing us to have Orders not belonging to any User. We could build these fixtures like that:
class OrderFixture extends ActiveFixture
{
    public $modelClass = 'common\models\Order';
    public $depends = ['tests\codeception\unit\fixtures\UserFixture'];
}

class UserFixture extends ActiveFixture
{
    public $modelClass = 'common\models\User';
    public $unloadFirst = ['tests\codeception\unit\fixtures\OrderFixture'];
}

Case 1: We call UserFixture->load, it will check it's own $unloadFirst array and perform clearing data for all fixtures there before calling resetTable() in UserFixture. This will unload OrderFixture prior to any actual user data loading. So after this we'll have data in table_user and empty table_order table.
Case 2: We call OrderFixture->load, it will check it's own $depends, will call UserFixture->load, which will do all the same as described in Case 1 (unload orders, load users). And after that will load it's own orders data as expected - into already emptied table. Both fixtures loaded and no any problems with foreign key violations.
Can we enhance Fixture this way or I missing something? This also can be used to make more complex depedencies (inherited dependencies, multiple dependencies). I see the only one significant consequence - it can make difficult the tracking of fixture loading/unloading order. But I don't think it's too complicated for developers who knows how to use foreign keys in their database. Those who don't use foreign keys with restrictions, may continue to use $depends field only.

ActiveFixture is basically unusable in current state because of this issue. More than 2 years passed since the original post. Can we do something about it?

There are many opportunities to improve the architecture around fixtures. Fact is that focus lies on other topics. With respect to the Fixture topic and testing. A good alternative is to create records right before you are using them. This is less error prone and easier to understand. In terms of performance, it doesnt really matter but in terms of debugging during tests, a lot of time is saved.

@Kolyunya  what do you mean? We're using fixtures with depedences and everything seems fine.

Does not make send that resetTable happens on load method, it's necessary to move to unload.
I did understand why is not possible todo this change?

@leandrogehlen how about failed tests? Unload won't happen and everything would stuck, right?

Exactly, i did this change https://github.com/Codeception/Codeception/pull/4016, but this was not enought,
Because resetTable does not happens on unload method

how about failed tests?

class CategoryFixtrue extends ActiveFixture
{
     public $modelClass = 'app\models\Category';
}

class ProductFixtrue extends ActiveFixture
{
     public $modelClass = 'app\models\Produto';
     public $depends = ['app\tests\fixtures\CategoryFixtrue'];
}

How the unload method don't execute resetTable after tests the tables will have records,
in the next execution, the resetTable will happens on load method and the first command will be:
`delete from `category``

if table `product` have fk we will have fk constraint exception

Hey guys
I spent hours confusing why unload doesn't work.
If you gonna consider that unload is useless then you should take of that from documentation.
It saying that unload clear data, but in fact it doesn't. It really confuses and lost energy thinking that I did wrong test.

I would edit documentation myself if u let me to prevent other developers from that confusing.

@nurbeknurjanov , i think that the problem is the feature and not the documentation.
I think that edit the documentation does not solve it

@leandrogehlen
This issue is from 2014. Still is not solved. And is not going to be solved soon. And untill this is not solved people are reading the documentation and trying to implement this function. Off course it is not working, and we think that we did something wrong in fact that documentation feeds us not correct information.
Or it would be solved, but I doubt it would be soon, so untill that moment documentation I guess should have some note information letting us to know, that this function is not prepared.

I have started the work to solve this issue, then i found https://github.com/yiisoft/yii2/blob/master/framework/test/InitDbFixture.php that must be used when we have DB-related tests.
I will open the PR with necessary changes to work without yii\test\InitDbFixture
I don't know what decision will be chosen:

Accept the PR and  remove yii\test\InitDbFixture
Change docs to clarify how to use Fixtures with DB-related tests

@leandrogehlen we're using InitDbFixture it's quite helpful in part of turning off foreign checks and removing it is definitely BC.

As of https://github.com/yiisoft/yii2/pull/14001 fixtures are broken for us. If a test errors out, unload() does not get called and the data remains in the DB for the next test. This is causing Integrity Constraint violations. Leaving resetTable() in the load() method solves the problem.

Can you show an example how this error happens?

I can give you the scenario. By example, are you wanting to see code?
We setup a test with the appropriate fixtures to add data to the DB. Not all of the PKs are auto generated and some columns have a unique index. When the first test runs, the fixture data is loaded into the DB, but the test produces an error (any error that is not fatal to the app but it interrupts the test). The subsequent test gets loaded with the fixture data still loaded into the DB which causes the Integrity Constraint issues. Reviewing the DB after the test run, fixture data is still present in the DB.

I can't see the problem this way, and if you pay attention, the unload method is invoked before load
this way the first thing to happens before a test is to clean data.
Maybe you can build a small project with this scenario to show the problem

@leandrogehlen Found the heart of the problem. We have a sizeable project with quite a few tests still using yii2-codeception. Since it is deprecated, changes are not propagating back to the old extension thus breaking our tests.
This wouldn't be a big deal if the documentation didn't rely on the extension: http://www.yiiframework.com/doc-2.0/guide-test-fixtures.html

please open a new issue if there is a problem that needs to be fixed.


TOML dates no longer support equality tests against now

I recently upgraded from an old version (in the 0.50's I think) and have noticed a problem with a template that does a filter against some data that's in TOML using dates.
I have items such as the following:
{ date = 2022-06-20, name = "End to end GraphQL (workshop)", link = "https://ndcmelbourne.com/agenda/end-to-end-graphql/daba56ef9e98", event = "NDC Melbourne 2022" },

And use it in a template like so:
{{ if .Site.Params.speaking.enable }} {{ with .Site.Params.speaking }}

{{ .title }}

        {{ $talks := where .talks "date" ">" now }} {{ if eq (len $talks) 0 }}

{{ .noTalks }}

        {{ else }}

            {{ range sort $talks "date" "asc" }}

                {{ partial "talk" . }}

            {{ end }}

        {{ end }}

{{ .content }}

            My Available Talks

{{ end }} {{ end }}

Upon upgrading to the latest version, the where condition no longer works, I'm assuming because the two types (the TOML date and now) can no longer be compared.
Repo containing the full code is here: https://github.com/aaronpowell/aaronpowell.github.io
Relevant files are:

TOML: https://github.com/aaronpowell/aaronpowell.github.io/blob/main/src/config.speaking.toml
Template: https://github.com/aaronpowell/aaronpowell.github.io/blob/main/src/themes/aaronpowell/layouts/partials/speaking.html

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.100.1-0afb4866e345d31cbbcbab4349e43f1d36122806+extended linux/amd64 BuildDate=2022-06-01T10:11:48Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
Yes.

We have a similar issue in #9949 -- which is harder to fix.
This one, though, is fixable. Thanks for the report and sorry about the site breakage.

We have a similar issue in #9949 -- which is harder to fix (but that one has a good workaround).
This one, though, is fixable. Thanks for the report and sorry about the site breakage.

No worries about the breakage, just my blog so it's not the end of the world 😝
I had seen #9949 and thought they might be related, but it was different enough to raise seperately.