Improve test coverage

The test coverage has gone a bit down in the past year: https://app.codecov.io/gh/wadpac/GGIR
Some functionalities are notoriously difficult to test in GitHub actions:

Parallel processing
Specific challenges with larger datasets
File reading functionality across all variations in file type, recording length and device configuration.

Some of the functionalities are possibly easy to address:

Function g.plot5 (visualreport): First page generation is entirely skipped, which may not be needed.
Function g.part5:

Does not consider light at the moment.
Explorative nap/nonwear classification is not well covered here.

Function check_params:

Several object checks are not tested

Function g.analyse.perday:

Does not test for qwindow with more than two values, or for activity diary?

Rather then trying to improve each of these tests, it may be easier to:

Put milestone data files from GGIR part 1 inside the package, e.g. from a couple of brands and study designs.
Add matching activity diary, advanced sleeplog and basic sleeplog.
Write separate unit tests to process those milestone data in a way we would do for a project.
Review how this improves the test coverage and where necessary revise these unit tests a bit to improve coverage.

In this way, it may be possible to improve the quality of the continuous integration for GGIR parts 2, 3, 4, 5.

Here some reflections on this:

Put milestone data files from GGIR part 1 inside the package, e.g. from a couple of brands and study designs.

Agree with that. But I would first test that reading data from different brands/formats result in the same milestone data in g.part1. So, we would test the reading and the generation of the milestone data in part 1 separately in a test including all brands/formats possible (maybe with 4-hour recordings is enough for that). Once we test that the all brands can be read and the output is the same, we can proceed with just 1 milestone data from part 1, where I would use meta data from a geneactive file to include also lux variables in the output and facilitate later testing. This has the challenge that we need to store such raw data somehow, it might be heavy for an R package.
In this separate test for part 1, we should make sure that we include:

Different data formats
Imputation of gaps
Appending of recordings
Part 1 output data is similar across brands/formats (e.g., metalong column names differ between axivity (EN for euclidean norm) and actigraph (en for euclidean norm)).
Short recordings and corrupted files are identified
Calibration process work well (for this we might need a longer test file)

Write separate unit tests to process those milestone data in a way we would do for a project.

After thinking about this, I'm not sure this would improve the test coverage. In a project we usually select a specific GGIR configuration, but the GGIR pipeline is so flexible that using the script from different projects would result in reprocessing the part 1 milestone data an infinite number of times. A potential solution is using the default GGIR configuration over the 5 strategies in part 2 to test the interaction between the 5 parts of the package. And then test the extra functionalities separately.

I left out part 1 from my proposal because testing part 1 as a whole for real life data does not seem feasible: We will have to add large files to the package or downloaded them, and even if we have those large files it will take time to process them.
So, I think testing the part 1 functionalities is best done based on specific unit test to cover each single functionality separately. This is what we have been doing and we will keeping doing.
Instead, I would like to focus now on creating a more high level unit test (integration test) that runs the other parts (2-5) with more realistic study data compared with our current synthetic data or tiny example files.
Even 10 MB of GGIR milestone data is too large for a package. So, a possible solution could be to include a numeric data.frame with 100 days worth of real ENMO, MAD, nonwear, anglez, temperature, and LUX with values rounded to 1 decimal place. This would then be based on real data from a variety of studies appended to each other. Next as part of the test we could write a function to convert this data.frame in semi-synthetic test milestone data files. For example, split up as multiple recordings.
Advantages I see:

Hopefully this offers a quick route to improving the test coverage for parts 2-5 without having to dive into every single function to search for possible opportunities to improve coverage. Ideally, running a realistic study scenario should already help to improve coverage.
We can use this to shorten test_chainof5parts.R afterward as some of that will no longer be needed, and by that test_chainof5parts.R will become more tidy.
These new unit-tests could even act as showcases of how GGIR can be used for new software contributors.

After thinking about this, I'm not sure this would improve the test coverage.

If we create a unit-test for let's say the Whitehall study then it will at the very least provide an integrated test of all functionalities they need, including LUX analysis, their specific sleeplog size, their specific, ID format, their specific  way of dealing with missing files.
Here, I do not want to loop over all possible functionalities GGIR offers as then we are just doing the same as test_chainof5parts.R. I only want to focus now on testing real life study scenarios which then could boost the test coverage but equally important helps to monitoring that the specific approaches to the data remain reproducible.


API secured with Keycloak, configs for Docker

Secured APIs with Keycloak, configs for Docker

@taliaga Pull request updated


DropWizard Metric Pool Usage, grows endlessly.

If you set an instance of MetricsRegistry to HikariCP it will start collecting statistical information about the performance of the Datasource. Nevertheless, the metric pool.Usage which uses an histogram to measure the distribution of values grows its reservoir endlessly. This is a problem if you plan to deploy Hikari in a web-server with instrumentation because that reservoir will continue growing and growing with no stop for every connection opened and closed (on every hit to the server).
The same DropWizard Metrics webpage recommends to use UniformReservoirs for long-term measurements and use only Histograms for short data sets.

@joebrassi As far as I know that is incorrect.  The default histogram uses an Exponentially Decaying Reservoir:

To compute a simple decayed aggregate exactly, such as decayed sum, can require
revisiting every input item to compute the contribution of that item (an exception is
exponentially decayed sum and counts, which can be tracked in constant space due
to properties of the decay function).

Uniform Reservoirs on the other hand are not very useful for pool monitoring:

Don’t use one where you’d want to know if the distribution of the underlying data stream
has changed recently.

Most users are interested in recent data.  If the server has been running for three months, recent but highly abnormal pool behavior will be completely "smoothed over" by the uniform median function.

Yes, you're definitely right. Somehow missed to see the class type of the reservoir during the tests. It actually uses an Exponentially Decaying Reservoir.


support private buffer ignore list

I think I could use boring buffers if I also changed my buffer display format to include the full path, but it would be nice to say "don't update buffers in ~/journal" so others don't see "X is editing personal-journal.org`.
Or maybe a whitelist would be better for this type of thing.

@codygman maybe you should just disable elcord-mode on those files yourself(?)

Problem I see is that by disabling elcord entirely your 'time spent' value will reset back to 0, whereas they'd just like to not report those buffers


Website seems not working?

The example website seems not working, http://rkritesh.in/ngEmbed

Thanks for notifying. The domain expired and I have renewed it so it must be working in some time.


double-hybrid density functionals

Dear developers,
Do you have any plans for implementing double-hybrid density functionals in the near future?

you can try this addon https://github.com/ajz34/Py_xDH
Although it's developed for XYG3, earlier functionals like B2PLYP are also supported.

you can try this addon https://github.com/ajz34/Py_xDH
Although it's developed for XYG3, earlier functionals like B2PLYP are also supported.

Addition to hebrewsnabla's https://github.com/pyscf/pyscf/issues/911#issuecomment-821795031, if single point energy is the only concern, many doubly-hybrid functionals may actually could be strightford without addons.
Py_xDH uses pyscf APIs to deal with doubly-hybrid gradient and hessian. Frankly to say, performance is terrible, and it stores all eri integrals in memory.
As pyscf user, some doubly-hybrid functionals are listed here.

If functionals are B2PLYP-like (bDH), with the following function defined:from pyscf import gto, dft, mp

def get_energy_bDH(mol: gto.Mole, xc_scf: str, c_pt2: float):
    mf = dft.KS(mol, xc=xc_scf)
    mf.conv_tol_grad = 1e-10
    mf.grids.atom_grid = (99, 590)
    mf.run()
    if not mf.converged:
        raise ValueError("SCF not converged.")
    mf_pt2 = mp.MP2(mf).run()
    return mf.e_tot + c_pt2 * mf_pt2.e_corr

All functionals available in Gaussian16 (no frozen core) and some in QChem (XC_GRID<PHONE_NUMBER>90) should be reproducible:mol = gto.Mole(atom="O; H 1 0.94; H 1 0.94 2 104.5", basis="6-31G").build()
# B2PLYP    Gaussian: -76.29076003
#           QChem   : -76.29075997
#           PySCF   : -76.29075994
get_energy_bDH(mol, "0.53*HF + 0.47*B88, 0.73*LYP", 0.27)
# mPW2PLYP  Gaussian: -76.29201022
#           PySCF   : -76.29201014
get_energy_bDH(mol, "0.55*HF + 0.45*mPW91, 0.75*LYP", 0.25)
# PBE0DH    Gaussian: -76.28179640
# PBE0-DH   QChem   : -76.28180689
#           PySCF   : -76.28180680
get_energy_bDH(mol, "0.5*HF + 0.5*PBE, 0.875*PBE", 0.125)
# PBEQIDH   Gaussian: -76.24493913
# PBE-QIDH  QChem   : -76.24495339
#           PySCF   : -76.24495333
# coefficients is different to Gaussian
get_energy_bDH(mol, "0.693361*HF + 0.306639*PBE, 0.666667*PBE", 0.333333)
# PBE0-2    QChem   : -76.21295796
#           PySCF   : -76.21295791
get_energy_bDH(mol, "0.793701*HF + 0.206299*PBE, 0.5*PBE", 0.5)

If functionals are XYG3-like (xDH, which framework is more flexible than bDH), with the following function defined:def get_energy_xDH(mol: gto.Mole, xc_scf: str, c_pt2: float, xc_dh: str or None=None):
    # SCF part calculation
    mf_scf = dft.KS(mol, xc=xc_scf)
    mf_scf.conv_tol_grad = 1e-10
    mf_scf.grids.atom_grid = (99, 590)
    mf_scf.run()
    if not mf_scf.converged:
        raise ValueError("SCF not converged.")
    e_nscf = mf_scf.e_tot
    # Non-consistent part calculation
    if xc_dh is not None:
        mf_nscf = dft.KS(mol, xc=xc_dh)
        mf_nscf.grids.atom_grid = (99, 590)
        e_nscf = mf_nscf.energy_tot(dm=mf_scf.make_rdm1())
    # PT2 contribution
    mf_pt2 = mp.MP2(mf_scf).run()
    e_pt2 = c_pt2 * mf_pt2.e_corr
    return e_nscf + e_pt2

Then XYG3 functional is reproducible:# XYG3      QChem   : -76.2910536660
#           PySCF   : -76.2910536679
get_energy_xDH(mol, "B3LYPg", 0.3211, "0.8033*HF - 0.0140*LDA + 0.2107*B88, 0.6789*LYP")

If SOS (Scaled Opposite-Spin) is used, then code is a little complicated (since PySCF currently doesn't do scaled MP2). The following script could reproduce XYGJ-OS and xDH-PBE0 functionals (by LT-SOS-RI-MP2), provided that system is restricted, enough memory (O(N^3)) is sufficient, and HOMO-LUMO gap is large (>0.2 Ha):from pyscf import df, lib
import numpy as np
import scipy

def get_LT_SOS_PT2_restricted(mf, auxbasis: str):
    # generate laplace transform grids
    # **TODO: MUST USE OTHER GRIDS** such as modified Gauss_Chebyshev grid
    #   instead of DFT grid to achieve better performance
    #   grids generation should take HOMO-LUMO gap into consideration
    grid_points, grid_weights = dft.radi.gauss_chebyshev(10)
    # auxiliary basis integrals
    # space complexity: n^2 naux, could be optimized by batch auxiliary basis
    # time complexity: n^2 nocc naux, or O(N^4)
    mol = mf.mol
    aux = df.make_auxmol(mol, auxbasis)
    nelec, naux, nao = mol.nelec[0], aux.nao, mol.nao
    int2c2e_half = np.linalg.cholesky(aux.intor("int2c2e"))
    int3c2e = df.incore.aux_e2(mol, aux)
    Y_ao = scipy.linalg.solve_triangular(int2c2e_half, int3c2e.reshape(-1, naux).T, lower=True).reshape((naux, nao, nao)).transpose((1, 2, 0))
    mo_coeff = mf.mo_coeff
    Y_mo = lib.einsum("uvP, ui, va -> iaP", Y_ao, mo_coeff[:, :nelec], mo_coeff[:, nelec:])
    # LT-SOS
    # time complexity: ngrid nocc nvir naux^2, or O(N^4)
    mo_energy = mf.mo_energy
    D_ia = mo_energy[:nelec, None] - mo_energy[None, nelec:]
    exp_D_ia = np.exp(D_ia * grid_points[:, None, None])
    X = lib.einsum("iaP, iaQ, gia -> gPQ", Y_mo, Y_mo, exp_D_ia)
    eng = - lib.einsum("g, gPQ, gPQ -> ", grid_weights, X, X)
    return eng

def get_energy_SOS_DH(mol: gto.Mole, xc_scf: str, c_pt2: float, xc_dh: str or None = None, auxbasis: str = "cc-pVDZ-ri"):
    # SCF part calculation
    mf_scf = dft.KS(mol, xc=xc_scf)
    mf_scf.conv_tol_grad = 1e-10
    mf_scf.grids.atom_grid = (99, 590)
    mf_scf.run()
    if not mf_scf.converged:
        raise ValueError("SCF not converged.")
    e_nscf = mf_scf.e_tot
    # Non-consistent part calculation
    if xc_dh is not None:
        mf_nscf = dft.KS(mol, xc=xc_dh)
        mf_nscf.grids.atom_grid = (99, 590)
        e_nscf = mf_nscf.energy_tot(dm=mf_scf.make_rdm1())
    # SOS PT2 contribution
    e_pt2 = get_LT_SOS_PT2_restricted(mf_scf, auxbasis)
    e_pt2 *= c_pt2
    return e_nscf + e_pt2

Some numerical discrepency occurs, which may originates from laplace-transform grid setting.# XYGJOS  QChem: -76.1460262831
#         PySCF: -76.1460283380
get_energy_SOS_DH(mol, "B3LYPg", 0.4364, "0.7731*HF + 0.2269*LDA, 0.2309*VWN3 + 0.2754*LYP", auxbasis="cc-pVDZ-ri")
# xDH-PBE0  PySCF: -76.226948505
get_energy_SOS_DH(mol, "PBE0", 0.5428, "0.8335*HF + 0.1665*PBE, 0.5292*PBE", auxbasis="cc-pVDZ-ri")

For functionals which includes PBE, results from PySCF script deviates to Gaussian at magnitude of 1e-5. I'm not sure why this discrepency happens. I guess it originates from different PBE correlation functional implementation of LibXC and Gaussian.

We are implementing double hybrids for the next major version of libxc, so I am also interested in this.

P.S.
Recently I found efficiency of laplace-transform OS-MP2 could be further improved. For evaluation of LT-OS-MP2 efficiently and accurately (within some threshold), the code above is not suitable.
For larger systems (> 50 electrons), LT-OS-MP2 should be faster than SCF (functional with exchange) in principle.

 Should be seen as scratch, not ready for opening an issue or PR
Algorithms should be simple. Just batching and using symmetry.
Unrestricted-shell code is not ready. Code efficiency has not been tested thoroughly. Scratch code for automatically generation of laplace-transform coefficients exists but not tested or rationalized mathematically, and that requires high precision library such as mpmath or gmpy2 (a temporary resolution refers to https://www.mis.mpg.de/scicomp/EXP_SUM/1_x/ and https://github.com/bhelmichparis/laplace-minimax).
Code below tested by molecule Azvudine (C9H11FN6O4) HF/def2-QZVPP on Intel Xeon Gold 6154, where RI-JK 330 sec, RI-MP2 cholesky decomposed ERI 56 sec, OS-MP2 energy evaluation 26 sec.
I guess computation of cderi (53 sec) is redundant for LT-OS-MP2; only 3c2e integrals (3 sec) are required. So evaluation of LT-OS-MP2, or even RI-MP2, could be much faster.
def eng_lt_os_rmp2(cderi_uov, mo_energy, lt_exponents, lt_weights, batch=None, verbose=lib.logger.NOTE):
    """ Restricted OS-MP2 by Laplace-Transform.

    Parameters
    ----------
    cderi_uov : np.ndarray
        Cholesky decomposed ERI in MO basis (occ-vir part).
    mo_energy : np.ndarray
        Molecular orbital energy.
    lt_exponents : np.ndarray
        Laplace-transform exponents.
    lt_weights : np.ndarray
        Laplace-transform weights.
    batch : int or None
        Number of batch for auxiliary basis. Default is 1/6 of number of auxbasis.
    verbose : int
    """
    log = lib.logger.new_logger(verbose=verbose)
    time0 = lib.logger.process_clock(), lib.logger.perf_counter()

    naux, nocc, nvir = cderi_uov.shape
    g_o = lt_weights[:, None] ** 0.25 * np.exp(+mo_energy[:nocc] * lt_exponents[:, None])
    g_v = lt_weights[:, None] ** 0.25 * np.exp(-mo_energy[nocc:] * lt_exponents[:, None])

    if batch is None:
        batch = naux // 6 + 1

    log.info(f"[LT-OS-MP2] Number of batch {batch} for {naux} auxiliary orbitals.")

    batched_pair = list(lib.prange(0, naux, batch))
    batched_pair_lt = list(itertools.product(batched_pair, range(len(lt_exponents))))

    def load_weighted_uov(pair_lt):
        p_pair, idx_lt = pair_lt
        sp = slice(*p_pair)
        weighted_Pia = cderi_uov[sp] * g_o[idx_lt][None, :, None] * g_v[idx_lt][None, None, :]
        return np.asarray(weighted_Pia.reshape(-1, nocc * nvir), order="C")

    eng_mp2_os = 0

    for (p_pair, idx_lt), weighted_Pia in zip(
            batched_pair_lt, lib.map_with_prefetch(load_weighted_uov, batched_pair_lt)):
        for q_pair in batched_pair:
            if p_pair[0] > q_pair[0]:
                continue
            sp, sq = slice(*p_pair), slice(*q_pair)
            factor = 2 if p_pair != q_pair else 1
            cderi_Qia = np.asarray(cderi_uov[sq].reshape(-1, nocc * nvir), order="C")
            eng_mp2_os_auxpair = weighted_Pia @ cderi_Qia.T
            eng_mp2_os += - factor * (eng_mp2_os_auxpair ** 2).sum()

    log.timer("eng_lt_os_rmp2", *time0)
    return eng_mp2_os

@ajz34 Hi Zhenyu, I directly copied your input. When running with pyscf-2.3.0, the output is
converged SCF energy = -71.8241761667855
E(MP2) = -71.9849226734216  E_corr = -0.160746506636033
E(SCS-MP2) = -71.9840156329011  E_corr = -0.159839466115577

However, when using pyscf 1.7.5, the output is
converged SCF energy = -76.248953112686
E(MP2) = -76.403793220144  E_corr = -0.154840107458023
-76.29075994169963

I think the difference comes from the SCF part, maybe the different definitions of some functionals?

@ajz34 Hi Zhenyu, I directly copied your input. When running with pyscf-2.3.0, the output is
converged SCF energy = -71.8241761667855
E(MP2) = -71.9849226734216  E_corr = -0.160746506636033
E(SCS-MP2) = -71.9840156329011  E_corr = -0.159839466115577

However, when using pyscf 1.7.5, the output is
converged SCF energy = -76.248953112686
E(MP2) = -76.403793220144  E_corr = -0.154840107458023
-76.29075994169963

I think the difference comes from the SCF part, maybe the different definitions of some functionals?

Which functional are you using?

Suppose you are talking about B2PLYP. My run with pyscf 2.3.0 gives -76.248 for SCF and -76.290 for the returned value.

@zhigang90
My observation is the same to @hebrewsnabla. I've checked pyscf's version to be 2.3.0. For B2PLYP
get_energy_bDH(mol, "0.53*HF + 0.47*B88, 0.73*LYP", 0.27)

my output is
converged SCF energy = -76.248953112686
E(MP2) = -76.4037932201444  E_corr = -0.154840107458369
E(SCS-MP2) = -76.4033930794894  E_corr = -0.15443996680342
-76.29075994169976

Currently, I haven't successfully reproduced erroneous results.

Not sure if the following experience is related.
Recently my colleagues encountered problem only when number of basis is small (H2O/6-31G), and number of threads is large (> 24). That seems to be caused by inconsistent MKL environment variable LD_PRELOAD and the MKL version built for numpy or pyscf.
Perhaps double check environment variables could resolve problem (not promised to be useful).

@ajz34 Thank you for your efforts. I have resolved the problem after discussing with @hebrewsnabla .The problem is that I complied the pyscf locally including libcint and libxc, and when compling libxc, I used a wrong option.

For functionals which includes PBE, results from PySCF script deviates to Gaussian at magnitude of 1e-5. I'm not sure why this discrepency happens. I guess it originates from different PBE correlation functional implementation of LibXC and Gaussian.

@ajz34: yes. As we discuss in arXiv:2307.07474, PBE in Gaussian uses slightly different parameters both for exchange and correlation. Also TPSS is different due to the difference in the PBE correlation.
The Gaussian variant of PBE is available as mf.xc = 'gga_x_pbe_gaussian,gga_c_pbe_gaussian'.


Routing Prototype

DO NOT MERGE

 Routing prototype in App (v1)

Did a rebase and force push to trigger a netlify deploy


Add MPRIS - DBus linux media control

Current
Nothing done :surfer:
Issue
Cannot play/pause , skip etc from an outside controller on linux
Suggestion(s)
Basic

[x] Use mpris_service with dbus

Elaborate

[x] Custom implementation ?

Implemented using audio_service_mpris package


Preserve current repo when removing repo shortcuts.

When you click on the minus sign to remove repository shortcuts, that all works, etc, but when you're done, the repo that you previously had active doesn't resume. Instead, Elegit loads up the last repo in the list. It should stick with whatever repo you already had active.
(And, it shouldn't even reload it, presumably,)

fixed via 580942d16f2ef9a1527a2f242faa4d7c94bba997


Paket pack not handling "SourceDir" correctly in *.csproj file

Description
If the "OutputPath" has $(SolutionDir) for then Paket pack has a DirectoryNotFoundException.  It appears to be concatenating the current working directory to the actual output directory, instead of using the true Visual Studio solution directory, and then concatenate the output directory.
Broken:
$(SolutionDir)buildoutput\Debug</OutputPath>
Works:
....\buildoutput\Debug</OutputPath>
Repro steps
Please provide the steps required to reproduce the problem

Change the definition of the "OutputPath" to use $(SolutionDir) instead of a relative path to the directory where the solution file actually exists.

Build the solution.

CD to the directory that has the *.csproj file and paket.template (type project) and run "paket pack . --version <IP_ADDRESS> --verbose --template .\paket.template"  Then the exception should be thrown.  The path to the DLL it is looking to pack should exist under the directory the *.csproj file exists but really exists in a directory relative to the actual Visual Studio solution directory.

If possible then please create a git repository with a repro sample or attach a zip to the issue.
Expected behavior
I would expect Paket to determine the directory location of the solution file and use that instead of the directory the "paket pack..." command is being run from as the $(SolutionDir).
Actual behavior
Seeing a DirectoryNotFoundException and the path indicated is expected under the directory the *.csproj file exists in and is not looking for the *.dll to be included relative to the solution file.
Known workarounds
Manually change the $(SolutionDir) in the OutputPath to a relative path.

I would upvote this one, and additionally I think it would be good to support other env variables as well. Our csproj files for example have set <OutputPath>$(BUILD_OUTPUTPATH)</OutputPath> in all csproj files to allow dynamic output handling depending on certain usecases. The CI will set the variable, and then would call paket pack to generate around 20 nupkg´s from that.
We would love to use paket here since the dynamic dependency detection, we simply cannot rely on manual updates for all nuspec files. But so far we can´t because of our dynamic output path.
So basically paket could scan over the csproj file, and try to substitute what it finds with env variables, if they are defined.

Just to give this issue some more publicity, I would also like to support this feature request. We have a code base from which we build multiple different products (complex beast with native code etc.) I would not recommend or glorify our build setup, but that is what we have. As a consequence, the normal dotnet workflow with building everything into the same obj/ and bin/ subdirectories simply does not work reliably, we need to change our various output paths to dedicated build directories. And this does not work reliably with SDK-style projects


Repo missing environments folder

When attempting to clone the repo in order to contribute, the dev build fails due to a missing /environments folder.

Ohh yeah, sorry about that. As I didn't find other people interested in helping out with developing, I removed the folder and let git build it.
Because I stored Firebase data and cardmarket API keys there.
Going to push them again in a few minutes, minus the cardmarket keys, so you can play around :D
If you got any questions you can add me on discord  TakaOtaku#8538.

Thanks! That resolved the build. I'm going to see if I can figure out why the modals aren't appearing on my phone. It's doing the same thing on my friend's phone so it's not isolated to just me.


Trouble exporting (fail to write, file in use, locked or lacking permissions)

I try to export my game on Windows, but I get this error:
Unable to write to file 'C:/.../MyGame/addons/ggs/plugin_data.tres', file in use, locked or lacking permissions
After which my editor closes. Any idea what causes this?

After updating Godot to 4.1, it went away! Sorry for the bother!

Turns out there was another problem causing the error: One of my shaders was still using SCREEN_TEXTURE, even though the update to Godot 4.0 changed that to uniform sampler2D screen_texture : hint_screen_texture, repeat_disable, filter_nearest;. I'm guessing the error interrupted the export process, even though none of my scenes are using said shader. Even so, it's something to be wary of in the future, if anyone is having the same issue!


fixed kustomize docs

related https://github.com/VictoriaMetrics/VictoriaMetrics/pull/6790
also fixes #1073

Thanks!


Read-only HDD

I can't copy or edit files on my HDD, even accessing retronas from another PC, I even took the hd out of the pc with retronas and put it in a case and test it on another machine and I can copy and paste, edit files quietly, if I put it in retronas I always get a read-only message

Even with the recording options selected in the file, the error persists:

My fstab configuration for games HDD:

Everything else works ok, ps3, ps2 and xbox 360 loading games normally, I just can't add more games or rename files over the network because of this error

ntfs can be problematic when used under Linux, and especially when writing too it. While the Linux support is there i wouldn't recommend using it particularly for writing too.
What is the output of the mount | grep games command in the terminal,  i assume the command will show it is mounted ro and I'd assume that is due to a dirty bit or some other general error ntfs seems to suffer from, Linux will not mount the drive rw if it has been uncleanly mounted to protect the data.
In Cockpit check the Logs sections for additional information if the mount was successful or not,
You will most like need to plug it into a windows machine and run chkdsk, then cleanly disconnect the drive and reconnect to RetroNAS. I don't think we'll support ntfs moving forward, its a closed source product and the Linux support should be considered "best effort"


can't set _retryHandler

What steps will reproduce the problem?
Yii::$app->getDb()->commandMap['mysql'] = [
    'class' => 'yii\db\Command',
    'retryHandler' => function (\yii\db\Exception $e, $attempt) {
        if (Str::contains($e->getMessage(), 'MySQL server has gone away')) {
            Yii::$app->getDb()->close();
            Yii::$app->getDb()->open();

            return true;
        }

        return false;
    }
];
Yii::$app->getDb()->commandMap['mysqli'] = Yii::$app->getDb()->commandMap['mysql'];

// `dd` is from larapack/dd
dd(Yii::$app->getDb()->createCommand(), Yii::$app->getDb()->createCommand("SELECT * FROM `user` WHERE `id`=:qp0", [':qp0' => 1]));

What is the expected result?
Command {#2666 ▼
  +db: Connection {#2661 ▶}
  +pdoStatement: null
  +fetchMode: 2
  +params: []
  +queryCacheDuration: null
  +queryCacheDependency: null
  -_pendingParams: []
  -_sql: null
  -_refreshTableName: null
  -_isolationLevel: false
  -_retryHandler: Closure {#2662 ▶}
  -_events: []
  -_eventWildcards: []
  -_behaviors: null
}

Command {#2667 ▼
  +db: Connection {#2661 ▶}
  +pdoStatement: null
  +fetchMode: 2
  +params: array:1 [▶]
  +queryCacheDuration: null
  +queryCacheDependency: null
  -_pendingParams: array:1 [▶]
  -_sql: "SELECT * FROM `user` WHERE `id`=:qp0"
  -_refreshTableName: null
  -_isolationLevel: false
  -_retryHandler: Closure {#2662 ▶}
  -_events: []
  -_eventWildcards: []
  -_behaviors: null
}

What do you get instead?
Focus on _retryHandler, Idon't why i can't set _retryHandler
Command {#2666 ▼
  +db: Connection {#2661 ▶}
  +pdoStatement: null
  +fetchMode: 2
  +params: []
  +queryCacheDuration: null
  +queryCacheDependency: null
  -_pendingParams: []
  -_sql: null
  -_refreshTableName: null
  -_isolationLevel: false
  -_retryHandler: Closure {#2662 ▶}
  -_events: []
  -_eventWildcards: []
  -_behaviors: null
}

Command {#2667 ▼
  +db: Connection {#2661 ▶}
  +pdoStatement: null
  +fetchMode: 2
  +params: array:1 [▶]
  +queryCacheDuration: null
  +queryCacheDependency: null
  -_pendingParams: array:1 [▶]
  -_sql: "SELECT * FROM `user` WHERE `id`=:qp0"
  -_refreshTableName: null
  -_isolationLevel: false
  -_retryHandler: null
  -_events: []
  -_eventWildcards: []
  -_behaviors: null
}

Additional info

Q
A

Yii version
2.0.31

PHP version
7.2

Operating system
Ubuntu

That seems to be expected. When new SQL is set, private properties are invalidated.

Hi. I'm facing the same problem. I'm looking for a way to make a handler for the SQLSTATE[HY000]: General error: 2006 MySQL server has gone away error. And just like the original author of this issue, I tried to use retryHandler. But it always resets because absolutely all the queries generated by the ActiveRecord use the setSql or setRawSql methods, which calls reset() and consequently removes the error handler.
So I think that something should be changed here so that we can handle non-critical database errors and continue the normal operation of the application.

What would you suggest?

The most obvious solution is simply to remove the $this->_retryHandler = null line from the reset() method. Otherwise, it's just a completely useless feature, because, as I wrote earlier, absolutely all requests generated by ActiveRecord call methods which call reset() inside their implementation.
I thought that maybe somewhere inside the framework there is low-level logic that uses this retryHandler, but I couldn't find anything. And it's impossible to use this functionality at the user level in its current form.

Sounds alright. Please create a separate issue / PR for it. Thanks.


Add support for structured logging

If https://github.com/containerd/rust-extensions/pull/212 goes in and we adopt it we can clean up our logging story so we have more metadata for logs.
For instance, write now our logs:
INFO] waiting for instance: df03285ac14ced6475f603924a4d2da31314bfb0f5056322d50b22a0f90e2a92
[INFO] calling start function
[INFO] redirect stdio
[INFO] Create a Store
[INFO] loading module from file "/wasi-demo-app.wasm"
[INFO] Creating `WasiEnv`...: args ["/wasi-demo-app.wasm"], envs: Vars { .. }
[INFO] Creating `WasiEnv`...: args ["/wasi-demo-app.wasm"], envs: Vars { .. }
[INFO] Running "_start"
[[INFO] Creating `WasiEnv`...: args ["/wasi-demo-app.wasm"], envs: Vars { .. }

This is challenging, What which instance is Creating WasiEnv...: args for? It gets harder with more replicates more things happening.
With strucutred logging we could have something like the following so we can distinguish between multiple instances running
time="2023-10-06T21:49:05.767259938Z" level=info instance="aslkdfasdfhasgh09234kl23j4l2k3j42lkj34" msg="Create a Store"
time="2023-10-06T21:49:05.767259938Z" level=info instance="df03285ac14ced6475f603924a4d2da31314bfb0f5056322d50b22a0f90e2a92" msg="Creating `WasiEnv`"
time="2023-10-06T21:49:05.767259938Z" level=info instance="aslkdfasdfhasgh09234kl23j4l2k3j42lkj34" msg="Creating `WasiEnv`"

Is this issue resolved? @jsturtevant

Not fully, the code is there to handle this but a wrapper/macro needs to be added to runwasi to add fields like  instance="aslkdfasdfhasgh09234kl23j4l2k3j42lkj34"


Trigger condition call flag to distinguish between writerows/deleterows events

An additional boolean argument (flag) is passed to the trigger condition call.
When it is false, it indicates that the row has been inserted (writerows event).
When it is true, it indicates that the row is being deleted (deleterows event).
This flag is used as input data for code that makes reactive table joins more efficient by caching the values of the fields used to join the tables.
Every time a new binary log event takes place, the caching code inspects this argument and can determine whether it should add the field to the cache or purge it.

This is a fine idea but this patch results in an inconsistent argument pattern for trigger condition functions and does not include any update to the documentation.
I have implemented this in the following commit: https://github.com/numtel/mysql-live-select/commit/6e20eaa09e5e0bc2f5cb8a3b52a58e79569e1956


Add path short-hand for "owner" Node

Godot version:
Godot 3.x
Issue description:
It would be convenient if, in addition to using "/path" to get a relative path from the root node, and "path" to get a relative path from the current node, you could do another shorthand for accessing a relative path from the owner node (e.g. a local scene root) using something like "//path".
Thoughts?

I'd rather prefere to have owner as a property in inspector tab. Sometimes it is more convenient ho set owner there instead in node script, expecially when packing node thru PackedScene.pack()

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Fix the tabbing

The tabbing needs some refinement both for usability and accessibility.

@astridu Just to confirm; this ticket is referring to the tabs that are created when more than one resource template is loaded and not the navigation tabs at the top?

This is wrt the interaction of tabbing (using the tab key to navigate) to the different sections, not the visual design of the tabs. Apologies for the confusion. I'll add more details soon.

I have noticed that catalogers are accustomed to using form based layout for input of data. Most screens they use for cataloging support tabbing over to fields because the form is not broken up into cards. This might be too big of a change, but I think doing something like "class="col-lg-6 col-xl-12" for the cards -- making it look more like what catalogers are accustomed to may help supporting navigation in the page, possibly.


Next release

[ ] Replace references to old repository name
[ ] Bump version
[ ] Fix changelog

Do we have everything we need for the next release or should we wait for something?

The asterisk integration won‘t work anymore.
It will not show the correct extension state.
Also the device[„tech“] is broken. We now only have pjsip endpoints which uses another event „EndpointList“ and has other fields in it, than the the old SIP peer stuff.

Opened this issue https://github.com/TECH7Fox/asterisk-hass-integration/issues/49

Yes, the card and integration work separate. The integration is far from done so I will some work more on it.
As long as the card and add-on work together out of the box, I think we can merge.

Just found a minor issue while reading the logs:
[Mar 15 09:03:55] ERROR[476]: loader.c:2508 load_modules: Error loading module 'format_ogg_vorbis.so': libvorbisenc.so.2: cannot open shared object file: No such file or directory

Besides the debian package libvorbis0a we also have to add libvorbisenc2 and libvorbisfile3 as well to the Dockerfile.

See PR https://github.com/TECH7Fox/asterisk-hass-addons/pull/120/

And a little polish https://github.com/TECH7Fox/asterisk-hass-addons/pull/121

I have tested an amd64 images with this PR merged with PR120 and PR121.
It works good now. Without any error messages.

I think we should bump version to 2.0.0.

I agree. This is a major update.

@TECH7Fox please merge if it's good for you. It's good for me.

@TECH7Fox @felipecrs
Let's freeze this branch now and I will manually test this again one more time.
And if it's good let's release it....

@TECH7Fox I have tested a local build for amd64 of this current state and it is working on my production system using latest HA-OS, HA core and HA supervisor.
The build was done by using the home assistant addon builder.
So 🟢 from my side!

@TECH7Fox you need to approve the PR before I can merge.

Sorry for the wait, currently at work.

Sorry for the wait, currently at work.

No problem :)

@nanosonde huuge thanks for your contributions!

Thanks @nanosonde and @felipecrs for your work! This is awesome. :)
@nanosonde, are you in the discord server?

Thanks @nanosonde and @felipecrs for your work! This is awesome. :)
@nanosonde, are you in the discord server?

Which dscord server? Never used such a thing. ;-)
BTW: I do not see any new release or new packages for 2.0.0. I did not look at the Github actions, but isn't this done automatically for this repo?

@nanosonde here is a link: https://discord.gg/qxnDtHbwuD


[5.2] Add within() method to InteractWithPages

I have needed this several times when writing tests for complex markups.
This method allows to narrow the test assertions or actions to specific areas of the web page.
For example, you have 2 links with the same text in the same page, but you want to make sure you click the one inside the content area:

            Link

            Link

    $this->within('.container', function () {
        $this->seeLink('Link')
                ->click('Link');
        });
    });

Same if you want to make sure you see several texts / elements within a certain area of the page (see tests in the commit).

Pretty cool addition. Maybe in the tests check for what happens when the container doesn't exist.

@daylerees thank you and good point, it should throw an exception.

Don't you technically need a subcrawler stack in case you have nested within calls?

@taylorotwell I thought about that, but couldn't think in any use case. Also tests could become messy if people start to stack within methods. You could just use another within method with a more specific selector. i.e. This:
$this->within('#id', function () {
    // tests
});
$this->within('#id .class', function () {
    // tests
});

instead of this:
$this->within('#id', function () {
    // tests
    $this->within('.class', function () {
        // tests
    });
});

But let me know your thoughts, it's easy to support nested within calls as well.

@taylorotwell still waiting for your confirmation on wether we should support nested within calls or not.

@taylorotwell added support for nested within calls

@lucasmichot thank you. Docblocks fixed.

Surely the change to that protected property is a massive breaking change for people?

@GrahamCampbell only if they are creating their own test methods. But let me use sub crawler again.

Ping @crynobone.

Ok done, if someone is using the protected property $crawler directly, it will work like before. If someone wants to use within with custom test methods, they'd need to use getCrawler instead :)

Thanks. Can you PR some docs for this?

I'll do! Thank you @taylorotwell

Hey guys. I just noticed API says that within() method should return instance of $this but actually it doesn't so there is no way to continue with fluent interface after using within() method.

@flashek I'll solve that issue, thank you.

done #13200


Correct the link in Move Production Environments section

The link to the Service Description is not correct that is in this section:
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/fin-ops/get-started/move-environments-data-center?context=%2Fdynamics365%2Fcontext%2Ffinance#move-production-environments
The link is directing to the now outdated service description PDF.
Please use this vanity URL for the link:
https://aka.ms/D365-Cloud-Service-Operations

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

ID: 8f44ca5d-f623-51e3-1ec3-0b02f5215dce
Version Independent ID: cc895276-ae7f-3508-877b-084c2cf3e159
Content: Move environments between data centers - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/fin-ops/get-started/move-environments-data-center.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @ClaudiaBetz-Haubold
Microsoft Alias: chaubold

Thank you for your feedback. @ClaudiaBetz-Haubold can you please take a look?

I've corrected the link.  The updated content will be published soon.


"azureFunctions.defaultFunctionAppToDeploy" Is Ignored Without Warning

Expected Behavior:

Edit setting in GUI or add to (workspace) settings.json:
"azureFunctions.defaultFunctionAppToDeploy": "/subscriptions/<uuid>/resourceGroups/<group>/providers/Microsoft.Web/sites/<app_name>"
Right click on Function App folder, choose "Deploy to Function App..."
VS Code will deploy to the indicated Function App without asking which Function App to deploy to.

Observed Behavior:

As above.
As above.
The setting is silently removed(!) from settings.json, and the user is asked which Function App to deploy to.

Additional Information:
Thinking that maybe it's some sort of bizarre field validation, I have attempted several variations of "full Azure Id" (although there doesn't seem to be an official consensus on what this means, I believe it's the same as the "Resource ID" as listed in the Azure portal under Properties). I also tried the URL, and just the short name of the app. The behavior is the same in all cases. I also tried placing the setting in the general settings.json--it doesn't get removed, but it's ignored anyway.
I'm not even sure if this is a bug with the extension, or with VS Code--but with no errors or warnings it's difficult to understand what is even happening. The setting simply disappears without any fanfare, which is... unsettling to say the least.
Versions:
VS Code: 1.55.2
Azure Functions Extension: 1.3.0

It appears we remove the setting if we can't find an app matching the id, which I agree is not great haha. We should at least show a warning or something
You are correct that it should be the "resource id". I believe our logic is case-sensitive, so that could be the problem here. As a workaround, I would right click on your function app, do "View Properties", and copy the id exactly as shown - hopefully that has the right casing.


Branching support

HI
Does SurveyJS support 'branching' - aka the ability to change what future questions are shown to the user based on the answer to a given question ?
Thanks
Dave

@dekhaus Sure, please take a look at this example.
visibleIf property is available in Question, Panels and Page elements. If a Page or Panel invisible then all their nested elements are invisible too.
Thank you,
Andrew


Deployment to TensorRT

Hi All,
Thank you for the amazing work! Can this model be exported to TensorRT for inference?
Thanks

The complete model maybe difficult to deploy to TensorRT due to Deformable Aggregation Function. But I think, parts of the model can be deployed on TensorRT. Like the Resnet50 Backbone or the FPN Neck.

I haven't tried to convert the model to ONNX yet. So far I have only been successful to convert the Backbone, Neck and the Enocoders to TensorRT using torch2trt. So, I'm not certain which is the best way to approach for ONNX. I think MMDeploy would be great since you can define custom plugins to convert Deformable Aggregation Fucntion (Haven't tried so not sure.). Maybe you can try that and let me know how it goes?

I am able to convert the model to onnx but the model outputs same results for different inputs, so something is wrong.
Here are the steps I performed

pip3 install onnx
Update forward definition for sparse4d.py, sparse4d_head.py, instance_bank.py, blocks.py & detection3d_blocks.py

# Change this:
def forward(self, img, **data):

# To this
def forward(self, img, timestamp=None, projection_mat=None, image_wh=None):

Use torch.onnx.export
with torch.no_grad():
model.eval()
torch.onnx.export(
model,
args,
output_path,
export_params=True,
input_names=input_names,
output_names=output_names,
opset_version=opset_version,
dynamic_axes=dynamic_axes,
keep_initializers_as_inputs=keep_initializers_as_inputs,
verbose=verbose)

The issue right now is that the model outputs same results for different inputs. Debugging in progress...

That's great! 👍
Can I ask what's the value set for keep_initializers_as_inputs ? If you haven't already tried, maybe setting it to False would change?

@shubhendu-ranadive Did some debugging...I think the issue may be with InstanceBank.
InstanceBank caches intermediate results from previous inputs & passes them along with the next set of inputs.
Therefore, that InstanceBank class contains logic that only gets triggered when it processes sequential batches of inputs, as opposed to a single batch.
When I try to export my model with dummy inputs with a single batch, the outputs are incorrect (likely because the InstanceBank logic is not traced/exported correctly via torch.onnx.export)
Do you know if its possible to pass sequential batches of inputs so that the model is traced correctly with all inputs?

@CMSC740Student Thanks for your reply. That indeed looks like a problem when creating a graph for ONNX.
I don't think torch.onnx.export allows sequential inputs.
The only thing I found after searching is maybe using torch.jit.script? This will try to trace dynamic control flow of the model which you can then try to convert to ONNX?

@CMSC740Student Did you get it working?

@CMSC740Student This repository may be useful for your deployment:https://github.com/ThomasVonWu/SparseEnd2End


Unclear semantics multiple for multiple HandTrackerEXT handles per hand

Is it allowed to create multiple HandTrackerEXT handles from xrCreateHandTrackerEXT or not? The spec isn't clear.

The WG reports that in general, you can create as many handles of a type as you want, unless otherwise specified though you may hit a "limit reached" at some point. The WG declined to add additional language explicitly stating this.


pipewire: 0.3.45 -> 0.3.47

Motivation for this change
https://gitlab.freedesktop.org/pipewire/pipewire/-/releases/0.3.46
https://gitlab.freedesktop.org/pipewire/pipewire/-/releases/0.3.47
Upstream is also shipping a minimal.conf for a couple of versions now, for use without a session manager. This updates the nixos module to use that minimal config as the config base when no session manger is enabled. (Untested)
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

All peachy on my machine, including running without a session manager (albeit that requires an extra config tweak to make it use the right sound card)


Firefox-AppImage stands out .

I changed the name of my repository so you can use the name Firefox-AppImage, your repository is better than mine so I recommend changing to Firefox-AppImage.
Thanks for your excellent job you had with the repository .

@cmatomic Wow Thanks! How did you come to know of this repository?

@srevinsaju
I was trying to find a solution to make an appimage of one of my repositories updateable related to "gh-releases-zsync"

Renamed, and updated the updateinformation to point to new repository. Thanks :tada:


[v3.0.0] Missing --install-extension in help menu

Hi, v3 broke my Dockerfile so thought I'd report here.  Fixed all the issues to get it to boot, everything works so far (for now). My Dockerfile failed because --force was not a valid flag anymore so I checked the help menu and noticed --install-extension is missing from the help menu.
# demyx @ test in ~ [23:44:37]
$ code-server -h
code-server 3.0.0

Usage: code-server [options] [path]

Options
      --auth              The type of authentication to use. [password, none]
      --cert              Path to certificate. Generated if no path is provided.
      --cert-key          Path to certificate key when using non-generated cert.
      --disable-updates   Disable automatic updates.
      --disable-telemetry Disable telemetry.
      --host              Host for the HTTP server.
   -h --help              Show this output.
      --open              Open in browser on startup. Does not work remotely.
      --port              Port for the HTTP server.
      --socket            Path to a socket (host and port will be ignored).
   -v --version           Display version information.
      --user-data-dir     Path to the user data directory.
      --extensions-dir    Path to the extensions directory.
 -vvv --verbose           Enable verbose logging.

Will fix, thanks!

cc: @code-asher

The flag shows in the help now. I also added the force flag.

Thanks guys.


fix: some highlighted markdown content not synced

The highlightedMarkdown was not yet uploaded when downloading so we should wait until the uploading to complete
Increase the download timeout to 10 mins

Hey @sywhb, I had an issue lately, do you think its relevant and this PR would solve it? (Want to gather some more info before I open a ticket)
I was reading and highlighting on an iPad without internet connection, after ~2 hours I connected the iPad to internet and I wanted to check on my phone whether Omnivore synced my recent updates. This action made all my highlights unreachable that I did over the 2 hours of no internet (very annoying for sure). I can still see the highlights in the iPad but can not open the note for them (clicking on "note" doesn't do anything). Also exporting doesn't recognize these highlights (nor the highlighted section, neither the note for it). The system also changed all of my highlight colors for the document to orange, expect for those that I can see (but not reach) in the iPad, these remain yellow. Do you have any suggestion on this issue if this PR is not the solution?

Hey @sywhb, I had an issue lately, do you think its relevant and this PR would solve it? (Want to gather some more info before I open a ticket)
I was reading and highlighting on an iPad without internet connection, after ~2 hours I connected the iPad to internet and I wanted to check on my phone whether Omnivore synced my recent updates. This action made all my highlights unreachable that I did over the 2 hours of no internet (very annoying for sure). I can still see the highlights in the iPad but can not open the note for them (clicking on "note" doesn't do anything). Also exporting doesn't recognize these highlights (nor the highlighted section, neither the note for it). The system also changed all of my highlight colors for the document to orange, expect for those that I can see (but not reach) in the iPad, these remain yellow. Do you have any suggestion on this issue if this PR is not the solution?

I think the underlying issue of your case and this PR could be both related to some race conditions when syncing highlights across devices but this PR will only fix the issue if you are using Obsidian plugin.
Feel free to open another ticket and if possible, can you please provide your username so I can further debug in our system?


Bad url in wiki/other/geyseroptionalpack.md

https://ci.opencollab.dev/job/GeyserMC/job/GeyserOptionalPack/job/master/lastSuccessfulBuild/artifact/GeyserOptionalPack.mcpack found 404
EMMMMMM

Maybe:https://download.geysermc.org/v2/projects/geyseroptionalpack/versions/latest/builds/latest/downloads/geyseroptionalpack
awa


Fix test coverage

Summary
Our coverage report is according to all files (other than those in node_modules) as of https://github.com/safe-global/safe-client-gateway/pull/1655. Despite ignoring certain folders (https://github.com/safe-global/safe-client-gateway/pull/1892), the report remains the same.
This adjusts configuration to only return coverage reports for the relative folders (src and migrations).
Changes

Only match and generate coverage reports from src and migrations folders
Remove unnecessarily ignored folders

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

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 45.874%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
4846

Relevant Lines:
8474

💛  - Coveralls


EfficientViT b0 pretrained in ImageNet

Have you tested EfficientViT-b0 on ImageNet? And could you share the pretrained weights of EfficientViT-b0 in ImageNet 1k?

Hi seefun,
You can download the pretrained weight of EfficientViT-B0 using the following link:
https://drive.google.com/file/d/1ganFBZmmvCTpgUwiLb8ePD6NBNxRyZDk/view?usp=drive_link

@han-cai  Efficientvit series has been added into TIMM as EfficientViT (MIT)
https://github.com/huggingface/pytorch-image-models


Deployments should be listable

Basically a GET on api/deployments should allow me to paginate thorugh the deployments starting at the most recent one.
This isn't needed now, but will be once we start throwing a web front-end on this sucker

+1

This is implemented by #48, pull request #80


How to send a packet to the server from a client?

Hello!
So i want to a key "E" to send the packet 22 to multiogar from my client.
Does anyone here know how to add the code??
the code:
.onkeydown = function(b) { switch (b.keyCode) { case 69: send(22) break;

This is MultiOgar, the server so we can't help with clients.
How are we supposed to know when we don't even know what the client is and how it looks?

Does it matter the client? every client is basically the same, ya know? Coded in html etx. I just need to know how to send a packet. Thats all :)

Not really.

Yes, i just need to know if i need to place it over the body for example or in a js etc.
Easy, but anyways i got it now.

You can close the issue by clicking "Close issue" button on the bottom (next to Comment button).


Add information about how to install frontend dependencies

Let me know if that's how you imagined, if not, feel free to directly edit the PR using the GitHub UI 👍

🚢 it!


Metrics to test chunking on OSN

I have started a list of possible metrics over on hackmd
@shfer suggested looking at previous work on LLC4320 to think of other metrics and I think that's a good idea, but we also agreed that we probably don't want to spend a very long time on this.
cc @dhruvbalwada @rsaim @TomNicholas @rabernat

Should we add what the "core dimensions" of each operation would be? i.e. the dimensions along which you ideally would not want the data to be chunked when performing this type of analysis.
I added examples for the ones I know about.

It seems as though we definitely want to test different chunksizes and compression levels.
@rabernat are you happy to stick with zlib for compression? The other kinds of compression do not seem to be implemented in xarray really.

:+1: to only zlib


kpod stats

Move kpod stats to the libpod backend.
Signed-off-by: baude<EMAIL_ADDRESS>

LGTM
@mheon PTAL

LGTM, we should get some examples into the man pages, but not a hold.

@baude Fix the issues @umohnani8 found and add examples.  Then we can merge.

A few nits, otherwise LGTM

One more little thing, otherwise LGTM

addressed the man page and JSON string usage.
@mheon I only did a lock on one of the two funcs .. lemme know if that is sufficient given one calls the other.

@mheon disregard comment about locks, thinking of the top PR.  I did add it here.

LGTM
@rh-atomic-bot r+

@baude looks like tests aren't happy.

bot, retest this please

@baude I think the CI cluster is having issues, nothing seems to be passing

@rh-atomic-bot r+

@rh-atomic-bot retry

bot, retest this please

@rh-atomic-bot r+


tomcat/jasper-compiler 5.5.12

Type: Missing
Summary:
tomcat/jasper-compiler 5.5.12
Details:
No license info found: https://repo1.maven.org/maven2/tomcat/jasper-compiler/5.5.12/
Resolution:
NONE
Affected definitions:

jasper-compiler 5.5.12

@capfei - For something else, I was looking at this version of this package: https://repo1.maven.org/maven2/tomcat/jasper-compiler/5.5.9/ and it does have a sources.jar and we can see the Apache-2.0 code comments in the file headers. I think we can use the curation guideline to update this package version to Apache-2.0 based on this other package version. Thoughts?

@ariel11 Yes, and same complaint as #20692 😆


Failed to resolve: com.yalantis:ucrop:1.3.+

When I add compile 'com.yalantis:ucrop:1.3.+' to project it show error
Error:(30, 13) Failed to resolve: com.yalantis:ucrop:1.3.+

i can not resolve it !
how to fix it ?

Could you please leave an issue here - https://github.com/Yalantis/uCrop


[BEAM-11503] bump rockdbjni to 6.15.2

bump rocksdbjni to 6.15.2
https://github.com/facebook/rocksdb
related

https://issues.apache.org/jira/browse/BEAM-11503
https://issues.apache.org/jira/browse/BEAM-11502

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @iemejia

@xinyuiscool since this affects only Samza better you check this one.


Firewall Rules

The rules option shown in the picture is still not available in the portal and not working when assigning manual OMA-URI values, any plans to make this available soon?

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

ID: b84a77a5-54c7-22c7-b699-c1414c7cd6a1
Version Independent ID: f78d7bc3-97c8-d8e9-d4f7-5ea82680c738
Content: Create Windows Firewall rules in Intune (Windows 10)
Content Source: windows/security/threat-protection/windows-firewall/create-windows-firewall-rules-in-intune.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@officedocsbot assign @mypil

@bvorland - Thank you for engaging with the community here for the docs.
I know the product teams are always looking for user feedback on how to make Windows 10 better. They keep a close tab on user feedback and it would be awesome if you could add your voice via Feedback Hub
You might find others have made a similar request that you can add to, or you can open a new one. Let us know here if you do so we can up-vote your request.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Add "Close form" button

The form detail page allows form creators to view form results and preview forms.  However, users can't close a form at the moment.
To fix this:

Add a button to the form detail page (which has a url like talkform.ai/forms/{form_id})
When clicked, the button should use the Supabase client to update the value for is_open for the correct row in the database
The button should appear next to the "View live form" and be styled just like it
When the form is closed, the button should be unclickable and appear grey-ed out

Here are Supabase docs: https://supabase.com/docs/reference/javascript/update

lgtm


enhancement Optimize the usage of bthread mem tracker and thread switch mem trakcer

Proposed changes
Issue Number: close #xxx
Problem summary

In #13528, I tried to separate the memory that was manually abandoned in the query from the orphan mem tracker. But in the actual test, the accuracy of this part of the memory cannot be guaranteed, so put it back to the orphan mem tracker again.
Use the thread call stack to save the old tracker when thread switch mem tracker.

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.49 seconds
load time: 559 seconds
storage size:<PHONE_NUMBER>7 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221024042109_clickbench_pr_32883.html


Validation of fractional step values for INPUT TYPE="number" result in false positives

Checklist for this issue
Created FORM for validation of factional values (think currency or decimal hours).  The jQuery.validate plugin incorrectly flags valid values in error due to the use of the modulus function.  The modulus function will only work reliably using integer values.
Subject of the issue
When validating INPUT TYPE="number" with a fractional step value (e.g. 0.1) the modulo function returns unpredictable results even when the value is a multiple of the step.
Your environment

jQuery.min.js is version 1.11.1
jQuery.validate.min.js is version 1.50.0
Using Microsoft Edge 25.10586.0.0

Steps to reproduce
See https://jsfiddle.net/JayAChTee/g9jL3tgj/ for a demonstration of the issue.
Expected behaviour
I would expect that all values would be valid even though all steps are fractional from 0.1 to 0.005.  To temporarily bypass the issue, I added the following code before the modulus function
		while (Math.floor(param) != param)
		{
			param *= 10;
			value *= 10;
		}

Actual behaviour
All but 0.40 and 10 are determined invalid.

This is how I fixed validate to accommodate when the step attribute is a decimal. Override the step method with this:
// added by Adam Lauzon 2016-09-08 (www.adamlauzon.com)
$.validator.methods.step = function( value, element, param ) {

	// I borrowed this next function (decimalPlaces) from the Internet
	// http://stackoverflow.com/questions/10454518/javascript-how-to-retrieve-the-number-of-decimals-of-a-string-number
	var decimalPlaces = function(num) {
	  var match = (''+num).match(/(?:\.(\d+))?(?:[eE]([+-]?\d+))?$/);
	  if (!match) { return 0; }
	  return Math.max(
	       0,
	       // Number of digits right of decimal point.
	       (match[1] ? match[1].length : 0)
	       // Adjust for scientific notation.
	       - (match[2] ? +match[2] : 0));
	}

	var type = $( element ).attr( "type" ),
		errorMessage = "Step attribute on input type " + type + " is not supported.",
		supportedTypes = [ "text", "number", "range" ],
		re = new RegExp( "\\b" + type + "\\b" ),
		notSupported = type && !re.test( supportedTypes.join() );

	// Works only for text, number and range input types
	// TODO find a way to support input types date, datetime, datetime-local, month, time and week
	if ( notSupported ) {
		throw new Error( errorMessage );
	}
	// old
	// return this.optional( element ) || ( value % param === 0 );

	// new (move the decimals out of being a fraction)
	return this.optional( element ) || ( (value * Math.pow(10,decimalPlaces(param)) ) % ( param * Math.pow(10,decimalPlaces(param)) ) === 0 );
}

@JayAchTee
If I recall correctly, this issue was fixed in 1.15.1 (but doesn't handle the scientific notation mentioned in @adamlauzon's comment). Could you confirm ?

Will close for now because of inactivity. Feel free to ask further questions/provide furhter information regarding this topic here. We can re-open if required.

This issue is still present in version 1.16.0. See fiddle below.
https://jsfiddle.net/e1deba4x/


Implement xconnect/l3xconnect

 Description
Since the wireguard encapsulates only IP packets we need to add L3 connection support.
This can be achieved with memif mechanism (MEMIF_MODE_API_IP) and l3xconnect implementation.

@edwarnicke cc

@glazychev-art So I guess we need to add into memif mechanism parameter Mode and have by default L2 mode

See also: https://jira.fd.io/browse/VPP-1975


How to fix missing post updated time?

Checklist

[x] I have read the newlest tutorials and know the correct effect of the functional design.
[x] There is no similar question on existing issues (including closed ones).
[x] I have tried to find the answer on Jekyll Forum and StackOverflow.
[x] My question is based on the latest code of the master branch.

Description

On your demo site -- it display both Posted time and Updated time of the posts

However,it only display Posted time on my site.May I know how to config it? or some items should added in the yaml part?thanks

Have you ever updated your post? This means that after you committed for a post, you edited and committed it one more time.
If you care about how this field is generated:
https://github.com/cotes2020/jekyll-theme-chirpy/blob/48f14e39ac81bbfb3b9913ea3ee789d775b2d1ae/_plugins/posts-lastmod-hook.rb#L10-L11
https://github.com/cotes2020/jekyll-theme-chirpy/blob/48f14e39ac81bbfb3b9913ea3ee789d775b2d1ae/_layouts/post.html#L22-L27

the issue is solved


Implement HEAD requests support for artifacts

We need to be able to respond to HTTP HEAD requests issued against the org.carlspring.strongbox.controllers.maven.MavenArtifactController and org.carlspring.strongbox.controllers.nuget.NugetPackageController classes.
Possible headers to return:

Content-Length
Checksum-MD5
Checksum-SHA1
Creation-Date
Last-Updated

This needs to be implement this for:

Hosted repositories
Proxy repositories
Group repositories

If the artifact can be resolved from one of the hosted repositories, get it from there
For artifacts that are in a group repository that resolves them via a proxy:

If the artifact is cached, get the date from the cache
If the artifact needs to be resolved from the remote, download it first and then send the response

Also, as suggested by @fuss86, we need to add the same headers to the GET requests.

Working on this .

@sanket407 ,
As per your request, I have updated the task's description.


Add support for rook 1.4.1

Copied addon directory for rook/1.0.4 to rook/1.4.1 and then adjusted it in a sequence of commits to match latest version of rook/ceph.

Looks good. Could you add 1.4.1 after 1.0.4 here.?

Done


Pass cParent into luaifyValueWithType

Fixes an error regarding cParent not being passed into a function.
  File "F:\Development\ALIM-Framework\ALIM-Core\Tools\ImGuiLuaGen\gen.py", line 241, in luaifyValueWithType
    print("unknown value type: ", k, s, ' ### parent = ', cParent.type.spelling, ' ', cParent.spelling)
NameError: name 'cParent' is not defined

Others will be able to use it without having to worry about it now.
And this is unrelated to this PR, but I've had to manually change some bits to make it work with my project, I've finally got it working though. I'll make another PR shortly when I'm not busy with other things. And thank you for making this public, I had already made one (well, was WIP but it worked), but this one is better.

Thanks for the fix!
Hope it's useful :)

Thanks for the fix!
Hope it's useful :)

No worries 👍
I'll take a look at fixing the other issues soon with newer versions of ImGui & non-related BeamNG code. Just got to fix some backend stuff, but it works great. Thanks for making it public


Include Twix version 0.9.0 on https://cdnjs.com

Please add current twix version on cdn.
Thank you!!!

@amtkmrsmn I don't control that CDN. I recommend you write a pull request to update it in their repository -- it even looks like they now support auto-updating from the NPM package, so you'd only ever have to do it once.

Thanks Issac
Thanks & Regards
Amit Kumar Suman
m +91<PHONE_NUMBER>
On Mon, Feb 29, 2016 at 7:18 PM, Isaac Cambron<EMAIL_ADDRESS>wrote:

@amtkmrsmn https://github.com/amtkmrsmn I don't control that CDN. I
recommend you write a pull request to update it in their repository
https://github.com/cdnjs/cdnjs -- it even looks like they now support
auto-updating from the NPM package, so you'd only ever have to do it once.
—
Reply to this email directly or view it on GitHub
https://github.com/icambron/twix.js/issues/84#issuecomment-190218487.

Thanks Issac

Actually, I bet that it doesn't work because this no longer has the right paths: https://github.com/cdnjs/cdnjs/pull/5039. There's no more locale and stuff is in files/ instead of bin/

This seems to have been resolved: https://cdnjs.com/libraries/twix.js


DB SQLite -> MySQL migration error

Actual behaviour
The migration script tries to SELECT * from a "settings" called table. That doesn't exist, and therefore the script errors out:
PHP Fatal error:  Uncaught exception 'PDOException' with message 'SQLSTATE[HY000]: General error: 1 no such table: settings' in ~/miniflux/scripts/migrate-db.php:44
Stack trace:
#0 ~/miniflux/scripts/migrate-db.php(44): PDO->prepare('SELECT * FROM s...')
#1 ~/miniflux/scripts/migrate-db.php(240): get_settings(Object(PDO))
...
Expected behaviour

Migration to complete without error.

Steps to reproduce

Setup miniflux for sqlite use, create a user next to admin, add some feeds, make sure it works.
Create a MySQL database, and add details (host, user, pass, db, etc) in config.php (don't forget to change engine to mysql too).
Logout, then go to the miniflux/scripts directory (shell/ssh), and try migrating with the following command: php migrate-db.php --sqlite-db=../data/db.sqlite --admin=1

Configuration

Miniflux version: latest git
Database type and version: sqlite3, mysql 5.6
PHP version: 5.6, 7
OS: Linux
Browser: n/a

Changing line 44 from SELECT * from settings to SELECT * from user_settings seems to do a bit of the trick:
$rq = $db->prepare('SELECT * FROM user_settings');
However that least to another problem:

Create user
PHP Notice:  Undefined index: username in ~/miniflux/scripts/migrate-db.php on line 72

Notice: Undefined index: username in ~/miniflux/scripts/migrate-db.php on line 72
PHP Notice:  Undefined index: password in ~/miniflux/scripts/migrate-db.php on line 73
Notice: Undefined index: password in ~/miniflux/scripts/migrate-db.php on line 73
PHP Notice:  Undefined index: api_token in ~/miniflux/scripts/migrate-db.php on line 76
Notice: Undefined index: api_token in ~/miniflux/scripts/migrate-db.php on line 76
PHP Notice:  Undefined index: bookmarklet_token in ~/miniflux/scripts/migrate-db.php on line 77
Notice: Undefined index: bookmarklet_token in ~/miniflux/scripts/migrate-db.php on line 77
PHP Notice:  Undefined index: feed_token in ~/miniflux/scripts/migrate-db.php on line 79
Notice: Undefined index: feed_token in ~/miniflux/scripts/migrate-db.php on line 79
PHP Notice:  Undefined index: fever_token in ~/miniflux/scripts/migrate-db.php on line 80
Notice: Undefined index: fever_token in ~/miniflux/scripts/migrate-db.php on line 80
PHP Notice:  Undefined index: username in ~/miniflux/scripts/migrate-db.php on line 81
Notice: Undefined index: username in ~/miniflux/scripts/migrate-db.php on line 81
PHP Notice:  Undefined index: fever_token in ~/miniflux/scripts/migrate-db.php on line 81
Notice: Undefined index: fever_token in ~/miniflux/scripts/migrate-db.php on line 81
SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'username' cannot be null


Initializing a client fails

the code that reproduces this issue or a replay of the bug
const DISCORD_SERVER_ID =<PHONE_NUMBER>422708234;
const DISCORD_CHANNEL_ID =<PHONE_NUMBER>282548788;
const DISCORD_TOKEN =  "<TOKEN>";

let { Midjourney } = require("midjourney");

let client = new Midjourney({
  ServerId: DISCORD_SERVER_ID,
  ChannelId: DISCORD_CHANNEL_ID,
  SalaiToken: DISCORD_TOKEN,
  Debug: true,
});

await client.init();

Describe the bug
Describe the bug
Following the README in this repo. Created a server and invited Midjourney bot to it. Assigned DISCORD_SERVER_ID, DISCORD_CHANNEL_ID, DISCORD_TOKEN variables and tried to call await client.init().
Expected behavior
Client initialization should be successful.
error log
Uncaught Error: Failed to get application_commands for command settings
    at MidjourneyApi.getCommand (/Users/alex/Projects/assistant/node_modules/midjourney/libs/command.js:82:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Midjourney.Connect (/Users/alex/Projects/assistant/node_modules/midjourney/libs/midjourney.js:32:13)
    at async Midjourney.init (/Users/alex/Projects/assistant/node_modules/midjourney/libs/midjourney.js:44:9)
    at async REPL12:1:33

got the same error here when using imagine api - api is very unusable now. Hopefully we can escalate this issue thanks!

got the same error

Experiencing the same error.

found the reason - likely salai token is invalid. Try using a new salai token

Becareful guys, ServerId and ChannelId should be passed as string or bigint or they will be rounded down by JS.

@bingyuyap Good call!

I encountered the same error. Despite logging out and back into Discord, and updating the Discord token, the issue persists without any success.

same error here

try this fix. it's working for me - https://github.com/erictik/midjourney-api/pull/256/files


Lineout cue drawn inches away from mouse pointer gesture path

Describe the bug
A user on LLNL RZ showed me this was happening running VisIt 3.4.1 in VNCViewer from DELL Laptop. He would drag a lineout and the visual cue would be miles away from the mouse pointer path on the screen. After releasing, it would compute the correct lineout curve and when he returned to the window showing the lineout cue, the cue would be in the correct place.
So, the issue is with the logic that draws and updates that lineout cue during the drag and release event.
He said the badness of the offset varies based on VNC window resolution.
I had Eric try on RZ from his laptop over VNC and everything worked fine. We tried varying VNC resolution with xrandr -s commands of various sorts and the lineout worked fine every time. We did notice that the lineout update drawing was very slow and took a while to catch up to Eric's mouse pointer gestures but it ultimately always settled on the correct cue position.
I also tried 3.4.1 on SCF with VNC viewer and it worked for me as it did for Eric.

This might be due to using some fancy X11 X-or-ing overlay to do these types of overlays.
We should see if there is a simpler way not that we live in "the future".

Might need to look at the vtk mapper classes in src/avt/QtVisWindow/vtk

The user reporting this is M. Adler

@brugger1  could this issue be similar to the rubber band zoom issues?

The code for rendering the rubber band line looks similar to the zoom lines. It isn't trying to do any fancy XORing. I have tried to reproduce this on my Dell laptop and haven't been able to. Mark wasn't able to reproduce on the closed in a VNC viewer. I believe Mark saw it on the users system so it is real, but I don't know what's special about his system.
Maybe we try on his system with the beta to see if it is still doing it. If it is, maybe we build a version with debugging information that outputs the coordinates of the lines being rendered to see if those are correct.


Microsoft Edge

Project description
A near-clone of Google Chrome
Metadata

homepage URL: https://aka.ms/microsoftedge
source URL: N/A
license: Unfree
platforms: Darwin, Linux, NT

https://github.com/NixOS/nixpkgs/issues/145318

Duplicate of #145318


Time indicator in plots view looks draggable but isn't

Quick solution is probably to just drop the triangle in the top of the time indicator for plots. Better solution would be to make it possible to drag the time slider in the plots view as well.

We should be able to use ui.interact for a rectangle covering the time cursor. It will steal all interaction from the underlying egui::Plot widget.

Requires some egui work: egui::Plot needs to return the plot_from_screen transform.


[T4A5][F09-B1]GuoShimin

Ready for review.

Some comments added. Please close the PR after reading comments. Think where you would print the tagging.


adapt elastic connector to support key-specified indexes

@dswbg my bastardization of the kafka elastic connector for your review

thanks a lot @dswbg


Add Snowplow tracking for behavior flags

resolves #10552
Problem
We need to track deprecated states of behavior flags in Snowplow so that we get telemetry for both Cloud and Core users.
Solution
Handle the event that is already firing for logging with a callback that tracks the event in Snowplow.
Checklist

[x] I have read the contributing guide and understand what's expected of me.
[ ] I have run this code in development, and it appears to resolve the stated issue.
[x] This PR includes tests, or tests are not required or relevant for this PR.
[x] This PR has no interface changes (e.g., macros, CLI, logs, JSON artifacts, config files, adapter interface, etc.) or this PR has already received feedback and approval from Product or DX.
[x] This PR includes type annotations for new and modified functions.

@cla-bot check

I think these tests are failing due to updates in dbt-common on which this is dependent. I'll rerun the tests once that PR is merged; it's currently queued.


[RFC] Bluetooth: host: add dedicated WQ for long-running tasks

Send long-running tasks to a dedicated low-priority workqueue.
This shouldn't increase memory usage since by doing this, we get rid of the
ECC processing thread.
This should fix issues like #43811, since the system workqueue runs at a
cooperative priority, and the new dedicated one runs at a pre-emptible
priority.
Open to any comments or other solutions. I'll change the PR title when we settled on something.
Signed-off-by: Jonathan Rico<EMAIL_ADDRESS>

What is the advantage of using this WQ instead of e.g. the system workqueue? Is it used for computationally heavy operations that may be blocking the other workqueue?
Is this useful for non-BT work? I.e. should we have a generic low-priority, heavy-workload workqueue instead?

I'll try to answer everything:

The 'only' advantage is that it's a pre-emptible priority, so can be pre-empted by any other coop thread (or sys wq). The linked issue shows a possible failure mode of using a coop wq.
Yes and yes. @alwa-nordic suggested exactly this this afternoon, but @carlescufi and I thought it's be good to have it first in use in bluetooth, and then we can argue to have it system-wide in a later PR.

Another point that was raised in the discussion was that @LingaoM added a way to use a 'bluetooth-dedicated' wq in a space-saving PR a while ago, and I could arrange to re-use that one (or use the syswq) for applications where space is a premium and where waiting 10ms-ish isn't that big a deal.

I understand that db hash does not occupy a separate task in the past implementation, but now it shares a task with ecc, is there an additional resource burden? After all, in the past mechanism, after ecc is closed, it will not there are other ecc tasks exist.

What do you mean by ecc is closed ? The previous implementation had a low-priority thread that ran the ecc tasks, blocking until ECC tasks are submitted, but isn't stopped, so the RAM used for its stack can't be reclaimed anyways. The new implementation has a low-priority workqueue, also staying idle if no more tasks are submitted.
The only difference (with the current patch) is that there will be an additional thread if you enable GATT caching but not ECC_TINYCRYPT. I will change that so a memory-conscious user can disable this new wq by setting CONFIG_BT_LONG_WQ=n, and then the gatt hashing will be sent to the system wq as was done before.

DNM until a potential regression is investigated.

@jhedberg @alwa-nordic can you please take another look?

rebased on today's main.
Changes add 56 bytes of ROM, 64 bytes of RAM, measured by compiling bt_shell on main vs on this rebased commit.

This blows up RAM usage for samples/boards/bbc_microbit/pong:
regionSRAM' overflowed by 1079 bytes`


Add missing status now returned by the API

The API changed to always return the status, so the integration test needs to be updated.

Looks good, only affecting the tests.

LGTM, merging. Thanks for the fix!


Migrator fails with SQLSTATE 42704

GORM Playground Link

https://github.com/go-gorm/playground/pull/433
Description
Model:
type Group struct {
	ID    string `gorm:"primaryKey"`
	GID   uint32 `gorm:"not null;autoIncrement"`
	...
}

Migrator error:
ERROR: type "bigserial" does not exist (SQLSTATE 42704)

Previously this table was created by GORM with type bigint.
I'm using latest versions of both GORM (v1.22.5) and potgres-driver (v1.2.3).
#4792 seems related, but there's nothing to upgrade to.
Switching to gorm:"type:bigserial" or gorm:"type:bigint;unique;autoIncrement" does not help.

@jinzhu please take a look at this.

This patch seems to fix the test, but I doubt that's the right way to do it.
func (m Migrator) AlterColumn(value interface{}, field string) error {
	return m.RunWithValue(value, func(stmt *gorm.Statement) error {
		if field := stmt.Schema.LookUpField(field); field != nil {
			fileType := clause.Expr{SQL: m.DataTypeOf(field)}
+
+			switch (fileType.SQL) {
+			case "smallserial":
+				fileType.SQL = "smallint"
+			case "serial":
+				fileType.SQL = "integer"
+			case "bigserial":
+				fileType.SQL = "bigint"
+			}
+
			return m.DB.Exec(
				"ALTER TABLE ? ALTER COLUMN ? TYPE ?",
				m.CurrentTable(stmt), clause.Column{Name: field.DBName}, fileType,
			).Error

		}
		return fmt.Errorf("failed to look up field with name: %s", field)
	})
}

Maybe it's better to not alter such columns at all:
func (m Migrator) AlterColumn(value interface{}, field string) error {
	return m.RunWithValue(value, func(stmt *gorm.Statement) error {
		if field := stmt.Schema.LookUpField(field); field != nil {
			fileType := clause.Expr{SQL: m.DataTypeOf(field)}

+			switch fileType.SQL {
+			case "smallserial":
+				fallthrough
+			case "serial":
+				fallthrough
+			case "bigserial":
+				return nil
+			}

			return m.DB.Exec(
				"ALTER TABLE ? ALTER COLUMN ? TYPE ?",
				m.CurrentTable(stmt), clause.Column{Name: field.DBName}, fileType,
			).Error

		}
		return fmt.Errorf("failed to look up field with name: %s", field)
	})
}

Any luck here? Can I do anything to help resolve this?

I'm experiencing similar issues: any update on this @jinzhu ?

https://www.postgresql.org/docs/current/datatype-numeric.html#DATATYPE-SERIAL
Seems we need to do special handling for serial in AlterColumn, it still works in CreateTable

i am having a similar issue with this:
	Position              int            `gorm:"type:serial" json:"position"`

It tries to automigrate this column time-and-time again, even when i dump the DB and start from scratch.

Should this be resolved in v1.23.5? I still get the type "bigserial" does not exist (SQLSTATE 42704) error. Is there a version I can check?

upgrade postgres driver.
On Tue, Apr 26, 2022 at 8:28 PM roboslone @.***> wrote:

Should this be resolved in v1.23.5? I still get the type "bigserial" does
not exist (SQLSTATE 42704) error. Is there a version I can check?
—
Reply to this email directly, view it on GitHub
https://github.com/go-gorm/gorm/issues/5047#issuecomment-1109735444, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABVO2TE6PD5AAM3EBPFI3VG7OQFANCNFSM5NO2INSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Best regards

Jinzhu
github.com/jinzhu

Thanks, it works!

Thanks, it works!

What version of Postgres driver worked for you?

@Toflex gorm.io/driver/postgres v1.3.5

I still get error type "bigserial" does not exist (SQLSTATE 42704) after update to gorm.io/driver/postgres v1.3.5


Starter dummie

Need one with normalize, and some base styles so I can get going more quickly.

Google font too

font awesome

My Initial thoughts on a starter template here:
https://gist.github.com/mhulse/3c4bccc5d82aeb074ae33edcdf570bdf


Change core option names to prevent conflicts

Prevents NP2K and vanilla NP2 libretro cores from overwriting each others' settings.

Thanks.
If there are other inconvenient features, please request.


Some modifications are not shown on the plot

Issue:
In some cases, modifications that are present in the data are not included in the legend of the plot and not shown. See attached example:

There was a bug for n-terminal modifications which I fixed in the preprocessing functions.


Relay

I don't get it. I can't relay signal. Maybe I do something wrong but is there tutorial for that?

There is a wiki  here. Only low-gain antennas (the whips) can communicate with other vessels. The high-gain antennas (the dishes) instead can only communicate with the home body.
 Then right-click on the low-gain and set 'relay' to true. That satellite will then relay data from/to the home body and other vessels. Make sure both antennas are deployed, else they will not work.
Feel free to ask a more specific question if the above doesn't help you.


[Bug] Graphics overlapping on scatter shot (Z-index) when there is a big amount of data (Exactly > 3000 dots)

Version
5.4.2
Link to Minimal Reproduction
https://jsfiddle.net/2ajk8L6w/
Steps to Reproduce
Code is on fiddle

Example with 3000 dots https://jsfiddle.net/2ajk8L6w/ (This is ok)
Example with 3001 dots https://jsfiddle.net/2ajk8L6w/1/ (This is with more than 3000 dots - this is the issue example)

Current Behavior
When I render a scatter graph with graphics with more than 3000 data points the graphs are rendered on  a bad Z position.
Expected Behavior
The render of graphics should have the same behavior and have the same position in both cases
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

use zlevel instead of "z" in rect and text


fix file same name editing

Review tmp folder struct to be a tree.
Files & folders will keep the same tree as in the remote server.
This allow us to edit multiples files from differents folder with the same name.

Hello,
How do i change the font size on SSH Client (vscode-ssh)?
TIA,

Hi @hisanoster,
You will find it in src\pages\xterm\js\index.js, at line 17.
Please, next time don't ask inside a PR.

Hi,
Thank you for your response, and I am sorry for my dumb question here but
how do I get that font size applied to my VS code on my laptop??
I would appreciate your quick walk throught if you don't mind.
Thanks,
hisanoster
On Mon, Apr 26, 2021 at 2:15 AM bazaim @.***> wrote:

Hi @hisanoster https://github.com/hisanoster,
You will find it in src\pages\xterm\js\index.js, at line 17.
Please, next time don't ask inside a PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cweijan-depreacate/vscode-ssh/pull/23#issuecomment-826572431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI52KJVAF33NLSAXDYDQF7DTKUHIFANCNFSM43BE2YTQ
.


Problems with markdown rendering on MacOS

Since using the Mac Theme with Joplin I discovered some Problems with markdown rendering that don't show up using the original theme. See the video attached. Rendering is only correct after pressing the button and switching between the modes. And even then the correct rendering is still slightly delayed.
https://user-images.githubusercontent.com/65334274/168908514-b4c8c75b-5ae8-4240-89cd-db83f6a59a41.mov

Good catch on this, @pschopen! I am seeing this issue as well running Joplin on Linux. I knew about this issue but assumed it was an issue with the core Joplin app since I exclusively use the MacOS theme.

My impression was, that it has something to do with the styling that the theme does to the markdown text. A solution could be to offer only the interface styling and have markdown styling only as an option within extension settings?

Hi @pschopen , can you check if this is solved after upgrading to the latest version? (1.3.5)

Thanks for checking this @pschopen & @ScriptInfra . With the newest version (v1.3.9) this should be fixed. Can you confirm if that is the case? If not, feel free to reopen this issue.

@andrejilderda - I am no longer able to reproduce this issue. Thanks so much for fixing this! What a huge quality improvement! I'll report back if I see any issues after further use.

Great! 🎉

@andrejilderda - I am no longer able to reproduce this issue. Thanks so much for fixing this! What a huge quality improvement! I'll report back if I see any issues after further use.

Same for me. Thank you very much; it makes a big difference!


Jenkinsfile env.BRANCHNAME returns null value

Hi,
I am trying to pass the branch name value from my Jenkins file to my groovy script. I tried to get the values as below but always getting empty value passed to the script. Is there any step i m missing? If I hardcode branchname then it works fine.
`node('master') {
checkout scm
functions = load 'pipeline-library/myfunctions.groovy'
    //def BRANCH_NAME=${env.BRANCH_NAME}
   //above one not working, passing null value

    //def BRANCH_NAME = sh(script: 'git rev-parse --abbrev-ref HEAD', returnStdout: true).trim()
   //above line not working, passing null value

   //def BRANCH_NAME ="develop"
   //working - passing value as "develop"

   myfunctions.MyGroovyScript("${BRANCH_NAME}")

}
`

bump

+1
env.BRANCH_NAME not working on pipeline but it does on multibranch pipeline

+1

+1


Rename Injectable to Inject

Why was this change made (Body)
In response to issue #25.
Footer

[ ] There are no breaking changes

If not why and what: breaks the API.

From Issue 25

Thank you very much for your contribution :)

I mean there are breaking changes, but they are intended! Would love to see a star @ishansharma :)


:sparkles: feat(config): Add jsconfig.json support to EVMts

Description
Added support for jsconfig.json to EVMts. If a jsconfig.json file exists evmts will autodetect it the same way it autodetects tsconfig.json
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #425   👈

This comment was auto-generated by Graphite.