NOTICK: Declare UtxoRepositoryImpl as a prototype persistence service.

Declare UtxoRepositoryImpl as a PROTOTYPE service for use in PERSISTENCE sandboxes. This allows Corda to construct its instance automatically. It also aligns the UTXO code with the Consensual code.
Also remove unsed EntityManagerConfiguration from the ConsensualLedgerRepositoryTest, because this is already handled by persistence-testkit.

Regardless of the future direction of Repository code, I strongly recommend merging this PR to make our existing code correct.

@chrisr3 has pointed out offline that whatever the future intentions, the current implementation is working by chance and should be addressed, which is a fair point.
My suggestion would be then that UtxoRequestHandlerSelectorImpl has the required services injected, and passes them directly (i.e. without the current sandbox lookup) to UtxoRepositoryImpl. That would resolve this specific problem without changing the architecture intended by the original developer and without clashing with our future plans for version upgrades.
Right now UtxoRepositoryImpl is an internal implementation detail of UtxoRequestHandlerSelectorImpl I don't see any compelling reason to change that.

My suggestion would be then that UtxoRequestHandlerSelectorImpl has the required services injected, and passes them directly (i.e. without the current sandbox lookup) to UtxoRepositoryImpl.

The services we wish to inject into UtxoRepositoryImpl are:

DigestService
SerializationService
WireTransactionFactory,

All of these services are customised for use with a given sandbox - otherwise there would be no need to fetch them from the SandboxGroupContext in the first place. So for your suggestion to work, UtxoRequestHandlerSelectorImpl would itself need to become a UsedByPersistence prototype service. Which would mean you'd then also need to rewrite DelegatedRequestHandlerSelectorImpl to obtain its UtxoRequestHandlerSelector from the sandbox context, in contrast to how it obtains its instance of ConsensualRequestHandlerSelector.

All of these services are customised for use with a given sandbox
Which would mean you'd then also need to rewrite DelegatedRequestHandlerSelectorImpl to obtain its UtxoRequestHandlerSelector from the sandbox context, in contrast to how it obtains its instance of ConsensualRequestHandlerSelector.

Understood. But I believe ConsensualRequestHandlerSelector is passed the sandbox too, from which it also extracts DigestService and SerializationService. Why does it not suffer from the same problem?

Because ConsensualRepositoryImpl is already forcing these services to exist (c.f. my change to UtxoRepositoryImpl):
@Component(
    service = [ ConsensualRepository::class, UsedByPersistence::class ],
    property = [ CORDA_MARKER_ONLY_SERVICE ],
    scope = PROTOTYPE
)
class ConsensualRepositoryImpl @Activate constructor(
    @Reference
    private val digestService: DigestService,
    @Reference
    private val serializationService: SerializationService,
    @Reference
    private val wireTransactionFactory: WireTransactionFactory
) : ConsensualRepository, UsedByPersistence {
    // etc
}

We know that ConsensualRepository exists because we've declared it as a corda.marker.only service.
We know that SerializationService exists because AMQPSerializationProvider (another corda.marker.only service) has explicitly added it to the context already.
We know that EntityManagerFactory exists because EntitySandoxService has explicitly added it to the context already.


php-cheatsheet

Is your feature request related to a problem or a new feature? Please describe.

A clear and concise description of what the problem or feature is.

Describe the solution you'd like

A clear and concise description of what you want to happen.

Additional context

Add any other context or screenshots about the feature request here.

@crescentpartha
https://github.com/crescentpartha/CheatSheets-for-Developers/pull/143
Please Check above PR for PHP-chetsheet.
Thanks!


Update google-cloud-pubsub to 1.129.5

About this PR
📦 Updates com.google.cloud:google-cloud-pubsub from 1.107.0 to 1.129.5
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #405.


Refactor CiliumExecContext() Retry Logic

Previously, the CiliumExecContext() function would retry the cilium command up to 5 times (limitTimes) without verifying the success of previous executions. This pull request introduces a logic enhancement that validates the return code of each executed command and exits upon success.
Additionally, this change optimizes test execution time by reducing unnecessary retries. With a 200ms delay between retries, the overall test execution time is expected to improve by at least 200ms multiplied by the number of cilium commands and the number of cilium pods involved.
Local Ginkgo tests, specifically those focused on 'K8sDatapathBGPTests' 'K8sDatapathCustomCalls' and 'K8sDatapathLRPTests' have shown significant improvements:
Before:
Ran 2 of 106 Specs in 168.563 seconds
After:
Ran 2 of 106 Specs in 233.348 seconds
Signed-off-by: Boris Petrovic<EMAIL_ADDRESS>

/test

/test


Vim debug mode could not work

Problem
Because of the following cmap in this plugin, the vim debug mode could not work in some case.
cmap <expr> <CR> <SID>handle_cr()

If remove this map, there won't be such problem.
How to reproduce
Execute :debug 1 and then press q, the debug mode could not be quitted. Following message will be shown:

I looked into that, but I don't see a way to fix it.
The reason why this mapping is necessary is that it is (to my knowledge) the only way to detect when the search is performed. There's no SearchDone autocmd, and there's not even CmdEnter autocmd that would allow setting this mapping condionally.
As a workaround, you can use Ctrl-J in debug mode - it's equivalent to <CR>, but not mapped. Let me know if you have an idea for a better fix.

I think that any mapping defined in command-line mode may interfere when you're in debug mode. But, IIRC, only if the rhs contains the evaluation of a custom function.
FWIW, I've defined a command to toggle on/off all mappings defined in command-line mode (:ToggleEditingCommands). I never use it manually, but I call it inside other custom functions.
Here's some code, using this command, which I think should get around the issue you describe:
cnorea <expr> debug getcmdtype() == ':' && getcmdline() =~# '\v^((tab<bar>vert[ical])\s+)?debug$' ? 'Debug' : 'debug'

com! -bar -nargs=1 Debug exe s:debug(<q-args>)
com! -bar -nargs=1 ToggleEditingCommands exe s:toggle_editing_commands(<args>)

fu! s:debug(cmd) abort
    try
        ToggleEditingCommands 0
        exe 'debug '.a:cmd
    catch
        return 'echoerr '.string(v:exception)
    finally
        ToggleEditingCommands 1
    endtry
    return ''
endfu

fu! s:toggle_editing_commands(enable) abort
    try
        if a:enable
            call s:tmp_mappings_restore(get(s:, 'my_editing_commands', []))
        else
            let lhs_list = map(split(execute('cno'), '\n'), 'matchstr(v:val, ''\vc\s+\zs\S+'')')
            call filter(lhs_list, '!empty(v:val)')
            let s:my_editing_commands = s:tmp_mappings_save(lhs_list, 'c', 1)
            for lhs in lhs_list
                exe 'cunmap '.lhs
            endfor
        endif
    catch
        return 'echoerr '.string(v:exception)
    endtry
endfu

fu! s:tmp_mappings_save(keys, mode, global) abort
    let mappings = {}
    if a:global
        for l:key in a:keys
            let buf_local_map = maparg(l:key, a:mode, 0, 1)

            sil! exe a:mode.'unmap <buffer> '.l:key

            let map_info        = maparg(l:key, a:mode, 0, 1)
            let mappings[l:key] =   !empty(map_info)
            \                     ?     map_info
            \                     :     {
            \                             'unmapped' : 1,
            \                             'buffer'   : 0,
            \                             'lhs'      : l:key,
            \                             'mode'     : a:mode,
            \                           }
            call s:tmp_mappings_restore({l:key : buf_local_map})
        endfor
    else
        for l:key in a:keys
            let map_info        = maparg(l:key, a:mode, 0, 1)
            let mappings[l:key] =   !empty(map_info)
            \                     ?     map_info
            \                     :     {
            \                             'unmapped' : 1,
            \                             'buffer'   : 1,
            \                             'lhs'      : l:key,
            \                             'mode'     : a:mode,
            \                           }
        endfor
    endif
    return mappings
endfu

fu! s:tmp_mappings_restore(mappings) abort
    if empty(a:mappings)
        return
    endif
    for mapping in values(a:mappings)
        if !has_key(mapping, 'unmapped') && !empty(mapping)
            exe     mapping.mode
            \ . (mapping.noremap ? 'noremap   ' : 'map ')
            \ . (mapping.buffer  ? ' <buffer> ' : '')
            \ . (mapping.expr    ? ' <expr>   ' : '')
            \ . (mapping.nowait  ? ' <nowait> ' : '')
            \ . (mapping.silent  ? ' <silent> ' : '')
            \ .  mapping.lhs
            \ . ' '
            \ . substitute(mapping.rhs, '<SID>', '<SNR>'.mapping.sid.'_', 'g')
        elseif has_key(mapping, 'unmapped')
            sil! exe mapping.mode.'unmap '
                                \ .(mapping.buffer ? ' <buffer> ' : '')
                                \ . mapping.lhs
        endif
    endfor
endfu

It installs an abbreviation, which automatically replace :debug with :Debug. The latter is a custom command used as a wrapper around :debug to temporarily disable all your mappings in command-line mode, including <CR>. They should be restored once you leave the debug mode.

Thanks for the pointers, @lacygoill. In the end, I figured out the problem is caused by the helper method handle_cr. Since it is written in Vimscript, the debugger tries to debug it in a recursive fashion. After inlining the method, everything works well.
CmdlineEnter could indeed be used to implement the mapping in a slightly less intrusive way. But given that many users may still run Vim 7, I'd have to support both implementations. Therefore, I decided to just keep the old one - but dropped the note to use CmdlineEnter should we see any more issues.


build problem

I am getting inconsistent results executing the following in Julia.
using JuMP,CPLEX
I=[1,2];
m2 = JuMP.Model(solver=CPLEX.CplexSolver());
JuMP.@defVar(m2, w[I], Bin);
JuMP.@defVar(m2, y[I] >= 0);
JuMP.@setObjective(m2, Min, 10*w[1]+w[2] + y[1] + y[2]);
JuMP.@addConstraint(m2,y[1] + 2*y[2]≤ 5*w[1]);
JuMP.@addConstraint(m2,3*y[1] + y[2]≤ 10*w[2]);
solve(m2)

Either the model is solved (as it should) or I get the following error:
ERROR: error compiling model: error compiling __CplexMathProgModel#1__: error compiling call: could not load library "/Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib"
dlopen(/Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib, 1): Symbol not found: __ZNSt8ios_base4InitD1Ev
  Referenced from: /Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib
  Expected in: flat namespace
 in /Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib
 in buildInternalModel at /Users/frank/.julia/v0.4/JuMP/src/solvers.jl:238
 in solve at /Users/frank/.julia/v0.4/JuMP/src/solvers.jl:79

The error systematically occurs when I start the Julia shell session with the above instructions. I am on OSX using Julia 0.4.3. The CPLEX  deps.jl file points to libcplex1261.dylib:

ls -l ~/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib
lrwxr-xr-x  1 frank  staff  19 Apr  7 14:17 /Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib -> libcplex1261.jnilib

What is the failure rate like? Is it deterministic (every session, on the first try), or not?
(edited your comment a bit for formatting)

Starting the julia session with the following always breaks:
|using JuMP,CPLEX I=[1,2]; m2 = JuMP.Model(solver=CPLEX.CplexSolver());
JuMP.@defVar(m2, w[I], Bin); JuMP.@defVar(m2, y[I] >= 0);
JuMP.@setObjective(m2, Min, 10w[1]+w[2] + y[1] + y[2]);
JuMP.@addConstraint(m2,y[1] + 2y[2]≤ 5w[1]);
JuMP.@addConstraint(m2,3y[1] + y[2]≤ 10*w[2]); solve(m2)|
Le 4/11/16 11:45 PM, Joey Huchette a écrit :

What is the failure rate like? Is it deterministic (every session, on
the first try), or not?
(edited your comment a bit for formatting)
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/JuliaOpt/CPLEX.jl/issues/68#issuecomment-208703834

This looks the same as this issue from the mailing list. You said you have an old version of CPLEX lying around: can you try with that version to see if the same error appears?

Well, I installed libcplex1261 after getting the above with
libcplex1250. Will look for something older.
Indeed, the error seems related.
Le 4/11/16 11:51 PM, Joey Huchette a écrit :

This looks the same as this issue from the mailing list
https://groups.google.com/forum/#!topic/julia-opt/rpe4AroSe60. You
said you have an old version of CPLEX lying around: can you try with
that version to see if the same error appears?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/JuliaOpt/CPLEX.jl/issues/68#issuecomment-208704960

@francoisgilbert, could you paste the output of:
otool -L /Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib

$ otool -L
/Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib
/Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib:
/Users/cplexadmin/jenkins/agent-svcplexdev38/workspace/distrib-branch12.6.1/label/x64-osx.svcplexdev38/bin/x86-64_osx/static_pic/libcplex1261.jnilib
(compatibility version 0.0.0, current version 0.0.0)

Le 4/12/16 8:21 AM, Miles Lubin a écrit :

|otool -L
/Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/libcplex1261.dylib|

@tkelman, any guess on this missing symbol issue? stdlibc++ incompatibility between Julia and CPLEX?

Maybe. Dunno.

@joehuchette, do we still need the .jnilib hack following https://github.com/JuliaOpt/CPLEX.jl/issues/34?

Who knows. @francoisgilbert, can you try pointing libcplex to /Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/cplex and see what happens?

I works! But why...
We otherwise noted that http://pkg.julialang.org/ is pointing the CPLEX
package to tag 0.1.1 under Julia 0.4 but it's tag 0.0.19  that actually
gets install when you do Pkg.add("CPLEX") from the julia shell (julia
0.4.3).
Le 4/13/16 12:41 PM, Joey Huchette a écrit :

Who knows. @francoisgilbert https://github.com/francoisgilbert, can
you try pointing |libcplex| to
|/Users/frank/machinerie/IBM/ILOG/CPLEX_Studio1261/cplex/bin/x86-64_osx/cplex|
and see what happens?
—
You are receiving this because you were mentioned.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/JuliaOpt/CPLEX.jl/issues/68#issuecomment-209583038

And versioninfo()?

julia> versioninfo()
Julia Version 0.4.3
Commit a2f713d (2016-01-12 21:37 UTC)
Platform Info:
System: Darwin (x86_64-apple-darwin13.4.0)
CPU: Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz
WORD_SIZE: 64
BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
LAPACK: libopenblas64_
LIBM: libopenlibm
LLVM: libLLVM-3.3
julia>
Le 4/13/16 1:37 PM, Miles Lubin a écrit :

And |versioninfo()|?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/JuliaOpt/CPLEX.jl/issues/68#issuecomment-209587782

Have you run Pkg.update() recently?

Hum.  A few days back. Doing it now:
julia> Pkg.update()
INFO: Updating METADATA...
INFO: Updating MATLAB...
INFO: Updating BaseTestNext...
INFO: Computing changes...
INFO: Upgrading ColorTypes: v0.1.7 => v0.2.2
INFO: Upgrading Colors: v0.5.4 => v0.6.3
INFO: Installing Combinatorics v0.2.0
INFO: Upgrading Compose: v0.3.18 => v0.4.2
INFO: Upgrading Contour: v0.0.8 => v0.1.0
INFO: Upgrading DataFrames: v0.6.10 => v0.7.0
INFO: Upgrading DataStructures: v0.3.14 => v0.4.3
INFO: Upgrading Distances: v0.2.2 => v0.3.0
INFO: Upgrading DualNumbers: v0.1.5 => v0.2.2
INFO: Upgrading FixedPointNumbers: v0.0.12 => v0.1.2
INFO: Installing FixedSizeArrays v0.0.10
INFO: Upgrading Gadfly: v0.3.18 => v0.4.2
INFO: Upgrading HTTPClient: v0.1.6 => v0.2.0
INFO: Upgrading LibCURL: v0.1.6 => v0.2.0
INFO: Upgrading LightGraphs: v0.3.7 => v0.5.2
INFO: Upgrading MPI: v0.3.2 => v0.4.0
INFO: Installing MacroTools v0.3.0
INFO: Installing Measures v0.0.2
INFO: Upgrading NaNMath: v0.1.1 => v0.2.1
INFO: Upgrading PyCall: v1.2.0 => v1.4.0
INFO: Upgrading QuantEcon: v0.3.0 => v0.4.2
INFO: Installing Requires v0.2.2
INFO: Upgrading ReverseDiffSparse: v0.2.11 => v0.3.0
INFO: Upgrading StatsBase: v0.7.4 => v0.8.0
INFO: Upgrading URIParser: v0.1.1 => v0.1.3
INFO: Removing FactCheck v0.4.2
INFO: Removing ImmutableArrays v0.0.11
INFO: Removing IterationManagers v0.0.1
INFO: Removing MAT v0.2.14
INFO: Removing Zlib v0.1.12
INFO: Building LibCURL
INFO: Recompiling stale cache file
/Users/frank/.julia/lib/v0.4/BinDeps.ji for module BinDeps.
INFO: Recompiling stale cache file
/Users/frank/.julia/lib/v0.4/URIParser.ji for module URIParser.
INFO: Recompiling stale cache file /Users/frank/.julia/lib/v0.4/SHA.ji
for module SHA.
INFO: Building Homebrew
INFO: Recompiling stale cache file /Users/frank/.julia/lib/v0.4/JSON.ji
for module JSON.
HEAD is now at 53c5089 CoreTap#install: fix clone target setting
HEAD is now at d45fe22 Merge pull request #91 from staticfloat/staging
INFO: Building Blosc
INFO: Building HDF5
INFO: Building LightXML
INFO: Building MPI
INFO: Attempting to Create directory /Users/frank/.julia/v0.4/MPI/deps/build
INFO: Changing Directory to /Users/frank/.julia/v0.4/MPI/deps/build
-- The Fortran compiler identification is GNU 5.2.0
-- The C compiler identification is AppleClang <IP_ADDRESS>20053
-- Checking whether Fortran compiler has -isysroot
-- Checking whether Fortran compiler has -isysroot - yes
-- Checking whether Fortran compiler supports OSX deployment target flag
-- Checking whether Fortran compiler supports OSX deployment target flag

yes
-- Check for working Fortran compiler: /usr/local/bin/gfortran
-- Check for working Fortran compiler: /usr/local/bin/gfortran  -- works
-- Detecting Fortran compiler ABI info
-- Detecting Fortran compiler ABI info - done
-- Checking whether /usr/local/bin/gfortran supports Fortran 90
-- Checking whether /usr/local/bin/gfortran supports Fortran 90 -- yes
-- Check for working C compiler:
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler:
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Found Git:
/Applications/Julia-0.4.3.app/Contents/Resources/julia/libexec/git-core/git
(found version "<IP_ADDRESS>")
-- Found MPI_C: /usr/local/Cellar/open-mpi/1.8.6/lib/libmpi.dylib
-- Found MPI_Fortran:
/usr/local/Cellar/open-mpi/1.8.6/lib/libmpi_usempif08.dylib;/usr/local/Cellar/open-mpi/1.8.6/lib/libmpi_usempi_ignore_tkr.a;/usr/local/Cellar/open-mpi/1.8.6/lib/libmpi_mpifh.dylib;/usr/local/Cellar/open-mpi/1.8.6/lib/libmpi.dylib

-- Detecting Fortran/C Interface
-- Detecting Fortran/C Interface - Found GLOBAL and MODULE mangling
-- Looking for MPI_Comm_c2f
-- Looking for MPI_Comm_c2f - found
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/frank/.julia/v0.4/MPI/deps/build
Scanning dependencies of target gen_constants
[ 12%] Building Fortran object
CMakeFiles/gen_constants.dir/gen_constants.f90.o
[ 25%] Linking Fortran executable gen_constants
[ 25%] Built target gen_constants
Scanning dependencies of target version
[ 25%] Built target version
Scanning dependencies of target gen_functions
[ 37%] Building C object CMakeFiles/gen_functions.dir/gen_functions.c.o
[ 50%] Linking C executable gen_functions
[ 50%] Built target gen_functions
Scanning dependencies of target mpijl-build
[ 62%] Generating mpi-build.jl
[ 62%] Built target mpijl-build
Scanning dependencies of target mpijl
[ 75%] Generating compile-time.jl
[ 75%] Built target mpijl
Scanning dependencies of target juliampi
[ 87%] Building Fortran object CMakeFiles/juliampi.dir/test_mpi.f90.o
[100%] Linking Fortran shared library libjuliampi.dylib
[100%] Built target juliampi
[ 25%] Built target gen_constants
[ 25%] Built target version
[ 50%] Built target gen_functions
[ 62%] Built target mpijl-build
[ 75%] Built target mpijl
[100%] Built target juliampi
Install the project...
-- Install configuration: ""
-- Installing: /Users/frank/.julia/v0.4/MPI/deps/src/./compile-time.jl
-- Installing: /Users/frank/.julia/v0.4/MPI/deps/usr/lib/libjuliampi.dylib
INFO: Building PyCall
INFO: Recompiling stale cache file /Users/frank/.julia/lib/v0.4/Conda.ji
for module Conda.
INFO: PyCall is using /usr/local/opt/python/bin/python2.7 (Python
2.7.10) at /usr/local/opt/python/bin/python2.7, libpython =
/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/Python
julia>
Le 4/13/16 1:45 PM, Miles Lubin a écrit :

Have you run |Pkg.update()| recently?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/JuliaOpt/CPLEX.jl/issues/68#issuecomment-209590724

Remove the CPLEXOLD directory. It is blocking the updates.

So this was all my fault. After update:
julia> Pkg.status()
20 required packages:

BaseTestNext                  0.2.1+             master
CPLEX                         0.1.1
Cbc                           0.2.0
Clp                           0.2.0
Conda                         0.1.9
ConfParser                    0.0.7
Dates                         0.4.4

Le 4/13/16 1:51 PM, Miles Lubin a écrit :

Remove the |CPLEXOLD| directory. It is blocking the updates.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/JuliaOpt/CPLEX.jl/issues/68#issuecomment-209592708

That's really odd that having an unregistered, dirty copy of a package prevents Pkg from updating the real thing. Sounds like it might be a bug.

My guess is that there was an upper bound on MPB in requires.
On Apr 14, 2016 4:25 PM, "Tony Kelman"<EMAIL_ADDRESS>wrote:
That's really odd that having an unregistered, dirty copy of a package
prevents Pkg from updating the real thing. Sounds like it might be a bug.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/JuliaOpt/CPLEX.jl/issues/68#issuecomment-210134044

Oh good call, that does sound likely. Having one package with a restrictive upper bound holds back everything. This is probably the main use case I can think of where something virtualenv-like (or just setting JULIA_PKGDIR on a per-project basis) would fix matters.

Hello everyone,
I am having the similar problems with Pkg.build("CPLEX"). So far I have tried:

Creating a symlink to "cplex"
Creating a symlink to "libcplex1263.jnilib"
Adding push!(libnames, "cplex") in build.jl of CPLEX package
Adding push!(libnames, "libcplex1263.jnilib") in build.jl

None of these work. Is there anything I can do to solve this issue?
Thank you in advance!

Closing as stale. The new Pkg environments fix this.


Black screen when showing interstitial ads

The screen is just black when showInterstitial ad is called using the cordova-plugin-admobpro plugin>>
Calling this to preload the ad:
if (AdMob) AdMob.prepareInterstitial({
    publisherId:'pub-xxxx',
    adId: ca-app-xxxx,
    autoShow: false, isTesting: false
});

Then I'm calling this later to show the ad
AdMob.isInterstitialReady(function (ready) {
    if (ready) {
         AdMob.showInterstitial();
      }
});

When I log onAdLoaded, onAdShow and onAdFailLoad, everything seems to be loading, showing and not failing, but the screen is just black. I have tried different adIDs and have tried adding other ad networks to no avail
here is the screenshot : https://drive.google.com/file/d/0ByAYq0kXNuoVUy02ZWM3NGtLRms/view?usp=sharing

if (AdMob) AdMob.prepareInterstitial({
    publisherId:'pub-xxxx', // please remove this line, publiserId is not a valid key for admob pro
    adId: ca-app-xxxx, // please use the valid ad unit id, like 'ca-app-xxx/yyy'
    autoShow: false, isTesting: false
});

Please set isTesting:true to see the result of test Ad, if it works, the plugin and your code is correct.
Then, change isTesting:false, and run your app in a device.
BTW, if your admob account is new, it will take some time before your ad unit id becomes effective. (for AdMob sync their data among systems)

Yea, I did test it on isTesting Tue, and everything was working fine, is just when I set it to false for the final release, it was just black. I wrote my own cordova-admob plugin from scratch (only supports android ATM, don't know swift or objectiveC yet, so no ios support for now) and it works, but I'll still help you guys debug

As everything works fine when isTesting:true, the code and plugin has nothing wrong. Please double check your ad unit id.

I also see such black interstitial, it seems the problem of ad content, not the plugin or SDK issue. please contact admob service team for this issue.


[Documentation] Improvements

Small typos
args is not between quotes

pylint-changed-files is in fact python-changed-files and changed-files is python-changed-files too.

Plus the hierarchy tree is not clear.
Misunderstandings
https://mookme.org/features/#reusable-steps
There is no example of "reusable-step" so we don't know if we should put 1 step per file, multiple steps in a list, or in a "steps" key… ?
Example issues
This script does not work on Mac.
#!/usr/bin/env bash
git diff --cached --name-only --relative | while read line; do [ ! -f \"$line\" ] || printf \"%s\\n\" \"$line\"; done | grep -P '\\.py$' | xargs -d '\\n' $@

This one is way more elegant & works on both Max & Ubuntu:
#!/usr/bin/env bash
git --no-pager diff --cached --name-only --relative | perl -nle'print if m{.py$}' | xargs -n1 $@

I'm not the owner of the project, but great feedback! Just wanted to add one more improvement to the example script:
#!/usr/bin/env bash
git --no-pager diff --cached --name-only --diff-filter=AM --relative -- "***.py" | tr '\n' '\0' | xargs -0 "$@"

Or if we want it to be completely generic (ie. changed-files instead of python-changed-files):
#!/usr/bin/env bash

sex -ue

file_filter=$1
shift

git --no-pager diff --cached --name-only --diff-filter=AM --relative -- "$file_filter"" | tr '\n' '\0' | xargs -0 "$@"

Anyways, I know that a PR with all these improvements would definitely be welcome. You should be able to run the docs locally with cd packages/docs && npm install && npm run dev.
Also, I just tested it out and the format shared hooks is a single top level json formatted the same as a hook. For example:
# /.hooks/shared/flake8.json
{
  "name": "Ensure Style (flake8)",
  "command": "flake8 $(python-module) --ignore E203,W503 --max-line-length 90",
  "onlyOn": "**/*.py"
}

(Notice how you can use partials like in this example python-module within the shared hooks)

(Oh whoops lol didn't realize you were part of the project xD. My bad for speaking to you like a new user lol)

In fact, it is nice to have the sharing features explicitely documented somewhere, but we could also use have used the Recipes section of the docs to provide more examples of this. I'm especially thinking about NodeJS examples or other languages examples.
I would love to have a way of getting some feedbacks on how people use Mookme in their projects, so that we could improve this section !


Invalid Keyword Argument for ActivationResult

With using they toy examples on my windows 10 machine, I get an invalid keyword argument for the ActivationResult seen here: https://github.com/fohrloop/wakepy/blob/5792c00cde782931e21f937a5f7846c434a8cdf8/wakepy/core/mode.py#L220
Checking the signature for this class, it is clear that init does not accept this: https://github.com/fohrloop/wakepy/blob/5792c00cde782931e21f937a5f7846c434a8cdf8/wakepy/core/activationresult.py#L193

Hi @EEsDoNotItNow and thank you for the report! Could you confirm that which version or branch you're referring to?

I guess the tests were using the latest dev (master) branch at a time, which was unfortunately broken for some time. This should be fixed.


Future Roadmap

I feel like we have reached the point where the crate is mostly done, and we are only left with a few unanswered questions/design decisions. Here they are:

Are we correctly handling html attributes? An alternative to passing common attributes such as class, style, width, etc. (what we are currently doing) would be to use the attr feature of leptos.
Since this crate has become pretty stable in the recent months, should we try to match our version to leptos' version? (e.g., publishing version 0.7 once leptos v0.7 comes out)

TODOs

[ ] Update the crate to support leptos v0.7

What needs to be done for 0.7 support?

I made an initial attempt at this: https://github.com/bicarlsen/leptos-icons/tree/0.7
KNOWN ISSUE
Generic attribute spreading does not work. This seems to have something to do with icondata_core::Icon being passed as an argument into leptos_icons::Icon causing it to not impl AsRef<Element> which is required to impl leptos::prelude::ElementExt.


Remove pytest-runner

Line https://github.com/cdgriffith/Box/blob/6d8790e35b6d2f20eb3bb0f4b450b0413daa94c1/setup.py#L36 should be removed.
Issue 1 (my immediate one): I'm using automated builds for an embedded device and the build environment is minimal, and doesn't include pytest. Since I'm not testing.
Issue 2: https://pypi.org/project/pytest-runner/#deprecation-notice

Fixed in 4.0


When graph is tabbed out, F button doesn't take you to fullscreen

I'm not sure if this is intentional or not, but I keep trying to do it so it feels like a bug to me.

it's intentional, because keypresses and mouse goes to the graph ... but it does feel weird. @matt thoughts?


Trying to Open the same Modal inside a Modal won't work / Event swallowed?!

Hello, currently I'm trying to open a modal with the id myModal inside a already existing myModal via data-toggle=modal.
Basically it works, however (what I expected to happen) is that when clicking the link that the modal will close (This is not the Bug, I would expect that to happen).
However what is happening now, is that the Modal will close and then nothing will happend, no navigation to the new page, no content loading.
So Basically what I expected to happen:

Open myModal
Event show.bs.modal called (and after shown.bs.modal)
Click Link in myModal with data-target myModal
Event hide.bs.modal called (and after hidden.bs.modal
Event show.bs.modal called (and after shown.bs.modal)

however step 5 will never happen, not as step 4, it will only call the hide.
i.e. I guess that either one of the following should happen:

call the events and reopen the modal
never close the modal, refresh the content
just navigate the link (if it is a link)

however that it closes the modal and nothing happens is kinda unexpected.

data-toggle=modal the keyword here is toggle=modal. It tells bootstrap to open or hide open modal. If you want a transition to new modal, you will need to implement it yourself I'm afraid.

well maybe something like data-open=modal would be a great idea. especially since there is a data-dismiss attribute.

Bootstrap does not support multiple modals and discourages it as a poor behavior.
So i believe no step will be taken in the direction of supporting it.
You'll have to do as @rafalp says above and solve this on your own, maybe instead of intercepting a generic event for any modal you can handle the close of the old and then call the new modal, doing whatever you need/want between those two tasks.
Anymore on how-to is a thing for stackoverflow, do the question there and paste the link here so people can help you.


Add missed dev dependencies for examples/using-preact

examples/using-preact is broken at the moment because react and react-dom dependencies are required in development mode but missed. So npm run dev and npm run build don't work.

Going to merge this for now.


Allow URL to be set via config, too

We'd prefer that :sunglasses:

Thanks DerGuteMoritz! :)
Included your change in latest version 0.2.1


Image comparison view not showing in cypress 11

Image Diff View is not working in Cypress 11
After upgrading to cypress 11.1.0 I cannot get to the image comparison view.
To Reproduce
Steps to reproduce the behavior:

Upgrade to cypress 11.1.0
Make changes to an image so it fails.
Click see comparisons link.
Nothing happens

Expected behavior
The image compassion screen should be showing after clicking on the comparisons link
Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following information:

OS and version: macOS Ventura 13.0.1
Browser and version: Google Chrome Version 107.0.5304.110 (Official Build) (x86_64 translated)
Cypress version: 11.1.0

Hey @obiyoda!
Thanks for filing this issue. Indeed seems that Cypress did some undocumented breaking changes reading runner queue API. I've just created a PR fixing it: #192

Check 3.2.1

The image comparison screen is now showing. However, it doesn't seem to be updating the image when clicking the button.

@obiyoda can you try updating Cypress to 11.2.0?

As I see no activity in this thread and I cannot reproduce this behaviour I'm closing this ticket. Please open up a new one with reproduction if you're still experiencing this issue


New in 2012-02-12: service bus topic options that are missing

According to http://msdn.microsoft.com/en-us/library/windowsazure/1b759576-0668-4018-8483-846e7bb193be#RESTAPI_2
Name: Xml Element Name
====================
MaximumNumberOfSubscriptions: MaxSubscriptionsPerTopic
MaximumNumberOfSqlFilters: MaxSqlFiltersPerTopic
MaximumNumberOfCorrelationFilters: MaxCorrelationFiltersPerTopic
EnableDeadLetteringOnFilterEvaluationExceptions: DeadLetteringOnFilterEvaluationExceptions

The first one will be covered in ARM (package azure-arm-servicebus to come), the three others don't exist anymore even in the "classic" RestAPI:
https://msdn.microsoft.com/en-us/library/azure/hh780749.aspx


TypeError: 'NoneType' object is not iterable

Traceback (most recent call last):
File "./manage.py", line 15, in
execute_from_command_line(sys.argv)
File "/home/ubuntu/.virtualenvs/demo_for_djcelery/lib/python3.5/site-packages/django/core/management/init.py", line 381, in execute_from_command_line
utility.execute()
File "/home/ubuntu/.virtualenvs/demo_for_djcelery/lib/python3.5/site-packages/django/core/management/init.py", line 375, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/home/ubuntu/.virtualenvs/demo_for_djcelery/lib/python3.5/site-packages/django/core/management/init.py", line 224, in fetch_command
klass = load_command_class(app_name, subcommand)
File "/home/ubuntu/.virtualenvs/demo_for_djcelery/lib/python3.5/site-packages/django/core/management/init.py", line 36, in load_command_class
module = import_module('%s.management.commands.%s' % (app_name, name))
File "/home/ubuntu/.virtualenvs/demo_for_djcelery/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 985, in _gcd_import
File "", line 968, in _find_and_load
File "", line 957, in _find_and_load_unlocked
File "", line 673, in _load_unlocked
File "", line 697, in exec_module
File "", line 222, in _call_with_frames_removed
File "/home/ubuntu/.virtualenvs/demo_for_djcelery/lib/python3.5/site-packages/djcelery/management/commands/celery.py", line 11, in
class Command(CeleryCommand):
File "/home/ubuntu/.virtualenvs/demo_for_djcelery/lib/python3.5/site-packages/djcelery/management/commands/celery.py", line 16, in Command
tuple(base.get_options()) +
TypeError: 'NoneType' object is not iterable
===================================================================
Hello there, I got an error when I run command ./manage.py celery worker -l info -B in django.
Here's the environment:
python: v3.5.5
celery: v4.3.0
djcelery: v3.3.1
redis: v3.3.11

how did you solve it?
i have got same problem.

Yes, it has been fixed before.
You can see the commit here. https://github.com/celery/django-celery/commit/95c3953321cb25f5e75464d26e03832d3bb56d79


Support loading system root certificates for Win32/OSX

Is your feature request related to a problem? Please describe.
When connecting to a TLS enabled GRPC endpoint the system root certificates are loaded on Linux/Android. When running on Win32/OSX the cert needs to be provided.
Related issue: https://github.com/grpc/grpc/issues/25167
Describe the solution you'd like
Similar to the grpc impl for java/c# the client should load the trusted certs for operating systems beyond Linux/Android.
Describe alternatives you've considered
The alternative is for each application to load the system certs and manually specify when creating the channel.

Please don't close out! It would be really nice to have feature parity for Mac/Win/Linux.

I think this should be reopened because:

The example provided on the website doesn't work as provided and only has a rather easy-to-miss "Note" at another place further up that it indeed won't work on non-POSIX systems.
There is no example code found for Windows, which would be very helpful. The only hint I got is from a comment made on issue https://github.com/grpc/grpc/issues/25167. IMHO that issue should not be closed either, as the problem mentioned in the title is not solved ("Default SSL credentials don't work on Win32/OsX").
Other implementations like grpc-dotnet don't have this issue with their default SslCredentials instances, which at least got me quite confused. Once I figured out the WinAPI to manually set up the credentials I'll probably post it as an answer there.

(I originally mentioned running into this trouble at https://github.com/grpc/grpc/issues/25829 but noticed that issue was reported on Linux specifically, while I'm attempting to do this on Windows 10, so I'm not sure if that issue is related.)

It looks like I don't have the ability to re-open this issue.
While that example code solves windows, we still have a similar issue on OSX as well.

Isn't macOS a POSIX OS? The warning in the authentication guide is slightly inaccurate then if it is not working with default SslCredentials.

The MacOS support is added in https://github.com/grpc/grpc/pull/29957. Once gRPC has its new release, it should include these changes. Credits to @tdbhacks for bringing that to gRPC!

@ZhenLian Great news. Shouldn't this issue be reopened until Windows support is added aswell?

@RayKoopa just created #34276 to eventually get your code merged!

Just adding a comment to say that the provided code sample allowed me to get gRPC / TLS working with opentelemetry-cpp.

In python grpc I was able to work around this issue of grpc not reading from the windows certificate store using this:
credentials = grpc.ssl_channel_credentials()
# On Windows, the default root certificates are not
# loaded from the system
# This makes self signed certificates fail handshake
# even if the certificate is in the windows trusted store
if sys.platform == "win32":
    certs = ssl.create_default_context().get_ca_certs(True)
    cert_string = ""
    for cert in certs:
        cert_string += ssl.DER_cert_to_PEM_cert(cert)
    credentials = grpc.ssl_channel_credentials(root_certificates=cert_string.encode('utf-8'))

channel = grpc.secure_channel(conn_address, credentials, options=channel_options)

This was fixed in #34874.

#34874 seems to fix the problem for Windows only, this bug is for Win32 and macOS too.  Did I get that wrong?  If not, maybe you should reopen this bug.


'Dockerfile parse error' pushing Pi-Hole to RPi4

I’m trying to run pi-hole on a Raspberry Pi 4B, but when I attempt to push the project code to my device, I always get the following error message. I am deploying the code from a Windows machine and made sure to push with --convert-eol.
[Error]           Some services failed to build: [Error]             Service: pihole [Error]               Error: Dockerfile parse error line 1: unknown instruction: DOCKERFILE.ARMV7HF
Has anyone else run into this? Any suggestions? Thanks in advance!

I was also getting the same error on Windows. What worked for me was copying the contents of DOCKERFILE.ARMV7HF into Dockerfile.aarch64 in the pihole directory.  This resulted in a different error which was fixed by replacing FROM balenalib/raspberrypi3-debian:jessie-build AS build with FROM balenalib/rpi-debian:jessie-build AS build. Hope this helps!

I had forgotten that symlinks don't work with WIndows workspaces. I've pushed the fix now.

This seems to have solved the problem! Thanks for the prompt response.


Outsource SVG's

As a developer
i want to outsource my svg's
so that my code is cleaner
AC

[ ] SVG's get implemented by path
[ ] No SVG tags in the source code

Tasks

[ ] Store SVG's as files
[ ] Replace SVG tag with source path

Good idea! 👍🏼


laravel 5.4 install error

Laravel Version: 5.4.*
Voyager Version: 0.11.5
PHP Version: 7.0.1
Database Driver & Version: mysql 5.5

Description:
AppServiceProvider is add  Schema::defaultStringLength(191);
Steps To Reproduce:
install error
λ php artisan voyager:install
Publishing the Voyager assets, database, and config files
Copied Directory [\vendor\tcg\voyager\publishable\assets] To [\public\vendor\tcg\voyager\assets]
Copied Directory [\vendor\tcg\voyager\publishable\database\migrations] To [\database\migrations]
Copied Directory [\vendor\tcg\voyager\publishable\database\seeds] To [\database\seeds]
Copied Directory [\vendor\tcg\voyager\publishable\demo_content] To [\storage\app\public]
Publishing complete.
Publishing complete.
Migrating the database tables into your application
Migration table created successfully.

  [Illuminate\Database\QueryException]
  SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length
  is 1000 bytes (SQL: alter table `translations` add unique `translations_table_name_column_name_fore
  ign_key_locale_unique`(`table_name`, `column_name`, `foreign_key`, `locale`))

  [Doctrine\DBAL\Driver\PDOException]
  SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length
  is 1000 bytes

  [PDOException]
  SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length
  is 1000 bytes

@zanjs did you try this? https://laravel-news.com/laravel-5-4-key-too-long-error

This is an issue with Laravel 5.4, not with Voyager.  Also, it has been asked and answered a few times already.  The solution exists in the link @wzulfikar posted.

AppServiceProvider.php   It has been modified
namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use Illuminate\Support\Facades\Schema;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        //
        Schema::defaultStringLength(191);
    }

    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        //
    }
}

But still  is error！
λ php artisan voyager:install --with-dummy
Publishing the Voyager assets, database, and config files
Copied Directory [\vendor\tcg\voyager\publishable\assets] To [\public\vendor\tcg\voyager\assets]
Copied Directory [\vendor\tcg\voyager\publishable\database\migrations] To [\database\migrations]
Copied Directory [\vendor\tcg\voyager\publishable\database\seeds] To [\database\seeds]
Copied Directory [\vendor\tcg\voyager\publishable\demo_content] To [\storage\app\public]
Publishing complete.
Publishing complete.
Migrating the database tables into your application
Migration table created successfully.

  [Illuminate\Database\QueryException]
  SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes (SQL: alter table `tran
  slations` add unique `translations_table_name_column_name_foreign_key_locale_unique`(`table_name`, `column_name`, `foreign_key`, `locale
  `))

  [Doctrine\DBAL\Driver\PDOException]
  SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes

  [PDOException]
  SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes

@zanjs make sure /config/app.php has this:
https://github.com/laravel/laravel/blob/master/config/app.php#L174

@abdgad  Thinks！ Let me try.

@zanjs I have same issues like you.
I resolved my problem, from @marktopper in Slack:

Can you please try modifying the migration and replace.

$table->increments('id');

$table->string('table_name', 64);
$table->string('column_name', 64);
$table->integer('foreign_key')->unsigned();
$table->string('locale', 2);

$table->text('value');

$table->unique(['table_name', 'column_name', 'foreign_key', 'locale']);

$table->timestamps();

If any body facing same problem, Please follow the below steps to fix the issue:
1. Update the "config/database.php" for 'mysql'

'engine' => null,

with

'engine' => 'InnoDB ROW_FORMAT=DYNAMIC',

++++++++++++++++++++++++++++++++++++++++++++
2. Update the "app/Providers/AppServiceProvider.php" with
<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use Illuminate\Support\Facades\Schema;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        // Specified key was too long error, Laravel News post:
        Schema::defaultStringLength(191);
    }

    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        //
    }
}

++++++++++++++++++++++++++++++++++++++++++++
3. Execute Command:

php artisan cache:clear
php artisan config:clear
php artisan voyager:install --with-dummy

Now, everything works! :)

@neerajsinghsonu  Can you explain why using 'engine' => 'InnoDB ROW_FORMAT=DYNAMIC',?
Have only some people meet this issues?
Thanks.

@vanloc0301
DYNAMIC and COMPRESSED Row Formats:

When a table is created with ROW_FORMAT=DYNAMIC or ROW_FORMAT=COMPRESSED, InnoDB can store long variable-length column values (for VARCHAR, VARBINARY, and BLOB and TEXT types) fully off-page, with the clustered index record containing only a 20-byte pointer to the overflow page. InnoDB also encodes fixed-length fields greater than or equal to 768 bytes in length as variable-length fields. For example, a CHAR(255) column can exceed 768 bytes if the maximum byte length of the character set is greater than 3, as it is with utf8mb4.

UTF8MB4:

Laravel 5.4 uses the utf8mb4 character set by default, which includes support for storing "emojis" in the database. If you are upgrading your application from Laravel 5.3, you are not required to switch to this character set.

More : MySQL reference

@neerajsinghsonu your solution worked for me.

@neerajsinghsonu your solution worked for me, too.

After some searching I found below solution and want to share it with you
1- set default string size in "app/Providers/AppServiceProvider.php" by adding \Schema::defaultStringLength(191); boot method
Schema::defaultStringLength(191);
2- change charset and engine in "config/database.php" by replacing
'mysql' => [             'driver' => 'mysql',             'host' => env('DB_HOST', '<IP_ADDRESS>'),             'port' => env('DB_PORT', '3306'),             'database' => env('DB_DATABASE', 'forge'),             'username' => env('DB_USERNAME', 'forge'),             'password' => env('DB_PASSWORD', ''),             'unix_socket' => env('DB_SOCKET', ''),             'charset' => 'utf8mb4',             'collation' => 'utf8mb4_unicode_ci',             'prefix' => '',             'strict' => true,             'engine' => null,         ],
To
'mysql' => [  'driver' => 'mysql',             'host' => env('DB_HOST', '<IP_ADDRESS>'),             'port' => env('DB_PORT', '3306'),             'database' => env('DB_DATABASE', 'forge'),             'username' => env('DB_USERNAME', 'forge'),             'password' => env('DB_PASSWORD', ''),             'unix_socket' => env('DB_SOCKET', ''),             'charset' => 'utf8', 'collation' => 'utf8_unicode_ci',             'prefix' => '',             'strict' => true,             'engine' => 'InnoDB ROW_FORMAT=DYNAMIC',         ],

I've done following to get rid of the issue-
Drop the tables in the database [every one, including the migration table]
composer dump-autoload -o php artisan migrate

This. Issue. Is. Closed.


Updating to the newer checkout action. As this one causes a warning.

https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Thanks @gs-emp for filing this.
I've tested out the action on serverless and it's working. Since it's the same action I'm assuming it works for the other workflows too.
Tagging in @gibsondan and @shalabhc (since you've recently touched this repo) -- any concerns about landing this? Any additional testing we should do?


fix 204 response

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#17094

Merged. Thank you!


Add Ability to Enter More Than 1 Donation Site Before Submission

Desired/Expected behavior
For those entering new donations sites manually, it would be nice to be able to add multiple locations at once before hitting the submit button, instead of having to open a whole new page each time.

Actual behavior
Must enter/submit one at a time.
Steps to reproduce the problem
Add a new donation site.
Computer Information (Only fill this out if you are a diaper bank)
MacOS High Sierra
10.13.4
Browser: Chrome

Add a "Create and Add Another" button, displayed alongside the existing Create button

[ ] Redirect the user back to the form if they want to add multiple and close if they want to add one donation site.

Our UX Expert (from the meta team) suggested having the form embedded directly onto the page and then posting back via XHR and having it refresh the table.
Given that it's only 2 fields, I think this would be an awesome solution!
Would you feel comfortable setting that up or maybe pairing with someone on it?

This is how the styling is currently working -- as you add elements, it prepends the rows to the collection:

The old behavior is still available at the /new endpoint, but the "Create" button was removed. I'm not sure if we should delete this to remove confusion and dead code, or leave it for compatibility (or something).


Arrow Forward Icon issue

Description
After updating native-base to 3.4.13, arrow forward icon is getting cropped.
CodeSandbox/Snack link
https://snack.expo.dev/Rz0f7tZ9g
Steps to reproduce
In below image, icon is getting cropped from right.

Code Snippet:
            <Button    size={'lg'}    variant={'outline'}     rightIcon={<ArrowForwardIcon size="5" />}     onPress={() => onNextQuestion()}>                 Next </Button>
NativeBase Version
3.4.13
Platform

[X] Android
[ ] CRA
[ ] Expo
[X] iOS
[ ] Next

Other Platform
No response
Additional Information
No response

Hi @PragyaSingla, Thanks for reporting this issue. We will fix it soon.

Thanks @Viraj-10  for consideration. Let me know once PR would be merged.

Hey @PragyaSingla, This issue is fixed in 3.4.14.


Load ktlint dependency always when Kotlinter applied

Before we only add the dependency configuration when a compatible kotlin plug is detected which doesn't work for projects with only custom kotlinter tasks. Fixes #423

Really ought to have a project test for this, but will catch up after getting the fix out.


Shuffle both sides at the same time for md.merge

What do these changes do?

For now, we shuffle left and right separately and make reducer data locate in different workers, this leads to additional transfer when execute merge operand. This PR only shuffles once for both left and right, so that reducer data locate in same worker and no transfer for following execution.
Related issue number

Fixes #xxxx
Check code requirements

[ ] tests added / passed (if needed)
[ ] Ensure all linting tests pass, see here for how to run them

As this PR changes the tile logic of merge, it will affect assign time which make asv benchmark fail in CI.


Improve UI for 3 main sections requiring user input on Line plots

Discussed in 4/4/2022 EDA UX meeting
There are 3 main sections requiring user input on line plots.

X-axis and Y-axis variable selectors → standard on all visualization tools
Y-axis aggregation function selector → specific to line plots
Stratification variable selectors → standard on all visualization tools

Improvements needed:

[x] The UX for these 3 sections needs better organization, perhaps there can be 3 columns for these 3 sections, instead of having the Y-axis aggregation selector below the other two. Place the y-axis aggregation function selector section between the other 2 sections, as it goes along with the y-axis variable selector and is required for making the plots.

[x] Gray out the aggregation function selector until the y-axis variable is chosen (currently, the function can be selected 1st, and no constraints are imposed on the variables)

[x] Remove the title from the numerator / denominator selector that appears when proportions are chosen to aggregate the data

[x] For proportion mode, place a division line or symbol between the numerator and denominator selectors to make what the selections are doing more clear

It looks very nice now!


⚠ Bofrost.at has degraded performance

In 340b103, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2482 ms

Resolved: Bofrost.at performance has improved in d80a9ce after 9 minutes.


ByteBuffer from component iteration must keep unsafe memory alive

Motivation:
It is possible to use forEachReadable or forEachWritable to create a ByteBuffer that is backed by native memory from a Buffer.
If we are using Unsafe, then we cannot rely on the underlying ByteBuffer implementation to keep the native memory alive for as long as it is accessible.
We must guard against the situation where forEachReadable/Writable is used on an UnsafeBuffer, then a readable/writeableBuffer is created, and this buffer is siphoned out of the iteration, and then kept alive for longer than the UnsafeBuffer.
We can only keep the UnsafeMemory alive, though.
We cannot guard against use-after-free, because we don't have a way to nerf a ByteBuffer once it has been released.
Modification:
Change the PlatformDependent.directBuffer method to take an attachment object, and pass it to the DirectByteBuffer constructor.
Then use this from the UnsafeBuffer *Component implementation to ensure that the underlying UnsafeMemory is kept alive and strongly reachable by any ByteBuffer instances we create.
Result:
Reduce the possibility of causing JVM crash through misuse of the ability to get ByteBuffers out of an UnsafeBuffer.

Just curious, why not https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/lang/ref/Reference.html#reachabilityFence(java.lang.Object) in such methods?

@franz1981 Because forEachReadable/Writable is public API, and we cannot control the code around such calls.


Error no matter the start prompt

Hash: bd89e826cf21de3dd0272ebaeb7dbaf69038dd26 (2013-04-19)
Debug: Prompt:
Debug: Response:
Sort-Object : A parameter cannot be found that matches parameter name 'Depth'.
At Q:\Auto-GPT-Powershell\module\General.ps1:66 char:59
+ ... luginType | Sort-Object { (GetProperty -properties (& $_.FullName -Fu ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (Output_CodeSaver.ps1:PSObject) [Sort-Object], ParameterBindingException
    + FullyQualifiedErrorId : ExpressionEvaluation,Microsoft.PowerShell.Commands.SortObjectCommand

Sort-Object : A parameter cannot be found that matches parameter name 'Depth'.
At Q:\Auto-GPT-Powershell\module\General.ps1:66 char:59
+ ... luginType | Sort-Object { (GetProperty -properties (& $_.FullName -Fu ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (Output_SessionLog.ps1:PSObject) [Sort-Object], ParameterBindingException
    + FullyQualifiedErrorId : ExpressionEvaluation,Microsoft.PowerShell.Commands.SortObjectCommand

ConvertFrom-Json : A parameter cannot be found that matches parameter name 'Depth'.
At Q:\Auto-GPT-Powershell\module\General.ps1:103 char:72
+ ... uginsJson = Get-Content $pluginsJsonPath | ConvertFrom-Json -Depth 10
+                                                                 ~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [ConvertFrom-Json], ParameterBindingException
    + FullyQualifiedErrorId : NamedParameterNotFound,Microsoft.PowerShell.Commands.ConvertFromJsonCommand

ConvertFrom-Json : A parameter cannot be found that matches parameter name 'Depth'.
At Q:\Auto-GPT-Powershell\module\General.ps1:103 char:72
+ ... uginsJson = Get-Content $pluginsJsonPath | ConvertFrom-Json -Depth 10
+                                                                 ~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [ConvertFrom-Json], ParameterBindingException
    + FullyQualifiedErrorId : NamedParameterNotFound,Microsoft.PowerShell.Commands.ConvertFromJsonCommand

It looks like, maybe, you need to delete your plugins file, but if you do then you need to go threw and enable the SessionLog plugin or else it will not write the session file.

Hmmm... Now it did this... 👍

Ok I got it fixed, try to delete the plugins.json and try again. But as of now, I need some sleep.

I have deleted the json-file, but it still doesn't do much. It does something though 🤔

Sleep tight! I'll monitor this repo for sure 👍🏻

I have changed this so the properties update automatically so you want have to delete the plugins.json file. (If you need to clear a plugins properties you can open it in a JSON editor and delete the section with the "FullName" property of the plugin you want to delete.)  Regardless a update should be coming soon. I will mark this as complete once I update the code, if you have another problem please start a new issue.

Code Updated. Might need to think of a versioning system.

I don't get errors any more, but I don't know how to use it to generate... Well... Anything...
No matter the starting prompt I write I get nothing back. It just loops through 5 times of nothing

When you first start it, lick "y" to goto the options, and "Turn On Debug" ad the Debug Messages show up, I am adding text to show you where in the loop it is and what it is doing. But when it is done running there should be a:Sassion" folder and in there will be the data that was generated.

I have debug on and do get a sessions-folder, but the output file only shows this

System:
Prompt:
Response:
System:
Prompt:
Response:
System:
Prompt:
Response:
System:
Prompt:
Response:
System:
Prompt:
Response:

OK, I will be working on finishing up a couple things tonight, and I will add this to the list. Did you use the "Setup .." menus?
Also if you could send me what the "Advance Options" look like, I can tell you what is going on.

I did use the Setup menu

Not sure what you mean by "advanced options"

Do you have "gpt4all-lora-quantized.bin" in the same folder as "gpt4all-lora-quantized-win64.exe"?  I am also making it to use "StableLM-Tuned-Alpha-7b Chat" (if your PC can handle it, I can run it with very basic settings, so it will not work on my PC, but maybe someone wout there will have one strong enough to test it. So the GPT4ALL model is going to be replaced, because it does not work, it response with responses that seem off topic, and do not follow the prompt sent to it.  So to really use this, OpenAI is the only thing so far, when I wrap up the code I am doing, it will have a Text To Image (using DALLE or local Stable Diffusion) and Bark (for creating speech from text) Hopefully I can get this wrapped up tonight, I am just caught doing work for my job.

I have the model in the same folder yes. I can run it separatedly through the exe.
The StableLM-Tuned-Alpha-7b model is really bad. Lol!
Sounds like your plans are on the right track :) SD and Bark is great. DallE is to expensive.


Update messages in tests/tools/test_pylint_works.py

Pylint 3.1.0 updated the messages of the wrong-import-order check:
https://github.com/pylint-dev/pylint/releases/tag/v3.1.0
This commit updates the messages in the test accordingly so the test passes again.
Closes #838

thanks @funkeleinhorn !


Duplicate content on pull request templates

The Pull Request Template in Code Reviews and the Pull Request section under Source Control should be consolidated, or linked

I reviewed the content and suggest to drop the content inside code reviews section. Content in source control section is decent and very well written. I believe though pull request template topic belongs to  Code review section and then suggest to move it there.  Please give your view: @TessFerrandez @magencio @tompaana

I agree with @katyamust
Although I'd perhaps associate PRs more with code reviews, the existing content under source control topic is better so it's easier just to remove the PR template under code reviews and update links.


Children in cards might receive input. and fixed backwards compatibility.

7701dee00dbdfc8ba4006d1bd43d5f8919f11290 fixed styling issues with the ripple. It also introduced a bug where if the card had children that were buttons or other elements that could receive input they would never receive the input.
This checks whether the Card has an onPress, and if it does set the pointerEvents to 'box-only', else set it to 'box-none'. 'box-only' implies that only the view itself can receive input, and no children. 'box-none' implies that the view itself won't receive input, and only the children.

Codecov Report

Merging #239 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #239   +/-   ##
=======================================
  Coverage   76.49%   76.49%
=======================================
  Files          19       19
  Lines         770      770
  Branches      113      113
=======================================
  Hits          589      589
  Misses        150      150
  Partials       31       31

@SanderVerkuil Thank you ;)
Let's talk about this a bit. I think both of those use cases should work together .. I mean if the user taps on card it should trigger onPress .. if user taps on button (or anything else inside of card) it should trigger component's onPress and not Card's on press? That's how works pointeEvents="auto", right? ..
What do you think?

It might, but I'm not certain. We'd need to test it. I'll give it a try in
the coming hour.
Op wo 6 dec. 2017 18:28 schreef Jiří Otáhal<EMAIL_ADDRESS>
@SanderVerkuil https://github.com/sanderverkuil Thank you ;)
Let's talk about this a bit. I think both of those use cases should work
together .. I mean if the user taps on card it should trigger onPress .. if
user taps on button (or anything else inside of card) it should trigger
component's onPress and not Card's on press? That's how works
pointeEvents="auto", right? ..
What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xotahal/react-native-material-ui/pull/239#issuecomment-349713956,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGwKT9gkhOamE4JWqEe_bqKeq6eodhebks5s9s62gaJpZM4Q2-na
.

It might even trigger both, but this is just guesswork. The docs on React native aren't that concise about it.

Well, I gonna try that .. we'll see what's happening ...

I tried it, and it works! They aren't triggered both, just the view that receives the onPress. If an element in the view has an onPress that will trigger, otherwise the cards' onPress will trigger.

Yep, it makes sense ;) Btw, I believe it's a default value of pointerEvents

Can we get a release for this, please? I think that would resolve #248.

I've manually updated the local copy of Card.react.js on my system and confirms that both onPress of the Card and the onPress of other Touchables within the Card are triggered correctly if the  pointerEvents="auto" on the main content variable.
This should be the desired behaviour for most applications. :+1:
A release would make us smile.


rviz_marker_tools CMake Error at CMakeLists.txt:6 (find_package):

System Environment:
Ubuntu 22.04
ROS2 Humble
Problem Description:
When following the tutorial to install and build MoveIt2 from source, I encountered some issues that caused the build to fail. Upon checking the terminal output, the following errors were displayed:
garyhuang@garyhuang-ubuntu:~/Documents/cobot_ws$ colcon build --mixin release
Starting >>> moveit_resources_panda_description
Starting >>> moveit_resources_pr2_description
Starting >>> srdfdom
Starting >>> moveit_common
Starting >>> launch_param_builder
Starting >>> moveit_resources_fanuc_description
Starting >>> moveit_resources_prbt_support
Starting >>> moveit_task_constructor_msgs
Starting >>> rviz_marker_tools
Starting >>> yolo_msgs
--- stderr: moveit_task_constructor_msgs
CMake Error at CMakeLists.txt:6 (find_package):
By not providing "Findcatkin.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "catkin", but
CMake did not find one.
Could not find a package configuration file provided by "catkin" with any
of the following names:
catkinConfig.cmake
catkin-config.cmake

Add the installation prefix of "catkin" to CMAKE_PREFIX_PATH or set
"catkin_DIR" to a directory containing one of the above files.  If "catkin"
provides a separate development package or SDK, be sure it has been
installed.

Failed   <<< moveit_task_constructor_msgs [0.28s, exited with code 1]
--- stderr: rviz_marker_tools
CMake Error at CMakeLists.txt:4 (find_package):
By not providing "Findcatkin.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "catkin", but
CMake did not find one.
Could not find a package configuration file provided by "catkin" with any
of the following names:
catkinConfig.cmake
catkin-config.cmake

Add the installation prefix of "catkin" to CMAKE_PREFIX_PATH or set
"catkin_DIR" to a directory containing one of the above files.  If "catkin"
provides a separate development package or SDK, be sure it has been
installed.

Failed   <<< rviz_marker_tools [0.29s, exited with code 1]
Aborted  <<< srdfdom [0.31s]
Aborted  <<< moveit_resources_prbt_support [0.30s]
Aborted  <<< moveit_resources_panda_description [0.31s]
Aborted  <<< moveit_resources_fanuc_description [0.30s]
Aborted  <<< moveit_common [0.31s]
Aborted  <<< moveit_resources_pr2_description [0.31s]
Aborted  <<< yolo_msgs [0.44s]
Aborted  <<< launch_param_builder [0.47s]
Summary: 0 packages finished [0.78s]
2 packages failed: moveit_task_constructor_msgs rviz_marker_tools
8 packages aborted: launch_param_builder moveit_common moveit_resources_fanuc_description moveit_resources_panda_description moveit_resources_pr2_description moveit_resources_prbt_support srdfdom yolo_msgs
2 packages had stderr output: moveit_task_constructor_msgs rviz_marker_tools
68 packages not processed
The problem seems to be related to rviz_marker_tools, which requires Catkin to build as it is a ROS1 package. However, I am using Ament to build in ROS2. How can I resolve this issue? Looking forward to your response.

Looks like you checked the wrong branch, make sure you are using humble for this repo

Looks like you checked the wrong branch, make sure you are using humble for this repo

Thank you for your reply. However, I tried rebuilding using the Humble, Main, and ROS2 branches, but the following error occurred every time:
CMake Error at CMakeLists.txt:6 (find_package):
By not providing "Findcatkin.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "catkin", but
CMake did not find one.
Could not find a package configuration file provided by "catkin" with any
of the following names:
catkinConfig.cmake
catkin-config.cmake

Add the installation prefix of "catkin" to CMAKE_PREFIX_PATH or set
"catkin_DIR" to a directory containing one of the above files.  If "catkin"
provides a separate development package or SDK, be sure it has been
installed.

Can you share the output of vcs status .?


Long humans.txt pushes input container out of reach.

If you enter a website with long humans.txt, it would push input container out of the view and you couldn't scroll it up. This forces users to reload website.
Examples:

adultswims.com
medium.com
trello.com

indeed. this is an UI issue.
@VACostin can you help me with fixing this from CSS ?

@sebiboga I can fix it. Do you agree that I should work on it?

@arashjafari please go ahead. I assigned this bug to you. Happy coding...


Help: included toolchains not compatible on Termux

in your screen shot you useing android device
But i am getting error how can I solve?

Hello.
Included prebuilt toolchains are for x86_64 hosts, see https://github.com/grm34/ZenMaxBuilder#toolchains
Kernel compilation on Termux is pain, you can try to build your own toolchains or use stuff like proot.

Hello.
Included prebuilt toolchains are for x86_64 hosts, see https://github.com/grm34/ZenMaxBuilder#toolchains
You can disable HOST_LINKER in etc/user.cfg to bypass the warning but the build will fail.
Kernel compilation on Termux is pain, you can try to build your own toolchains or use stuff like proot.

I am not useing termux.. i am useing userland like proot

Hello.
Included prebuilt toolchains are for x86_64 hosts, see https://github.com/grm34/ZenMaxBuilder#toolchains
You can disable HOST_LINKER in etc/user.cfg to bypass the warning but the build will fail.
Kernel compilation on Termux is pain, you can try to build your own toolchains or use stuff like proot.

And i really don't know how to build toolchain..
Can it not possible build in userland or termux proot-distro?

Userland on Termux, but I would like to say on Android to be more precise.
See https://github.com/grm34/ZenMaxBuilder#common-warnings-and-errors
x86_64 prebuilts needs x86_64 host, simple of that.

Userland on Termux, but I would like to say on Android to be more precise.
See https://github.com/grm34/ZenMaxBuilder#common-warnings-and-errors
x86_64 prebuilts needs x86_64 host, simple of that.

Now see useing x86_64 host..

I'm not going to do kernel building support here...
See https://github.com/grm34/ZenMaxBuilder#more-information
I help this time: see the error "CROSS_COMPILE_ARM32 not defined" 👉 check/edit your makefile

I solved my problem with this way.. (host-clang)

@grm34  zmb segmentation fault and freeze,

@grm34 zmb segmentation fault and freeze,

A package seems broken or an instance of zmb is not well closed.
Try to purge your cache, upgrade your system and identify broken package then reboot.
Some informations are asked while opening an issue:
- OS name and version
- bash version
- running shell
- terminal used
- ...


RFC: use output of dhcp-lease-list instead of using regex

This PR simply shows how leases could be gather in a simpler way

Much better to migrate to use ISC Kea instead of isc-dhcp-server


release(chart): helm chart changes for v3.3.0

Signed-off-by: Niladri Halder<EMAIL_ADDRESS>This PR contains all of the changes to the openebs-localpv/localpv-provisioner helm chart for v3.3.0 release.

Codecov Report

Merging #139 (7384001) into develop (4fbb888) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #139   +/-   ##
========================================
  Coverage    38.37%   38.37%
========================================
  Files           36       36
  Lines         3289     3289
========================================
  Hits          1262     1262
  Misses        1947     1947
  Partials        80       80

Continue to review full report at Codecov.

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


feat: add policyReportFields to the policy object directly

Details

This PR adds the policy report fields to the policy object itself instead of a separate policyFields_id object.
Fixed Issues

$ https://github.com/Expensify/App/issues/36170
PROPOSAL: N/A
Tests

On a Paid workspace, in Old dot, set report fields of different types.
Open ND and create an expense report on the above workspace.
Open the expense report and make sure  that canUseReportFields beta is enabled.
Verify that all the custom fields show up.
Click on each of the field.
Enter a value and save it.
Make sure that the field gets saved successfully.
For dropdown fields, verify that selecting an already selected dropdown option again deselects it.
For text fields, verify that removing the text from the input removes the field.

[x] Verify that no errors appear in the JS console

Offline tests

Same as above. Verify that everything works in offline mode.
QA Steps

On a Paid workspace, in Old dot, set report fields of different types.
Open ND and create an expense report on the above workspace.
Open the expense report and make sure  that canUseReportFields beta is enabled.
Verify that all the custom fields show up.
Click on each of the field.
Enter a value and save it.
Make sure that the field gets saved successfully.
For dropdown fields, verify that selecting an already selected dropdown option again deselects it.
For text fields, verify that removing the text from the input removes the field.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/30054992/a330df04-8d10-4f79-9c03-3e92937bd91a

Android: mWeb Chrome

https://github.com/Expensify/App/assets/30054992/9d1c0008-7aef-40ba-8c93-0015ed0b2fef

iOS: Native

https://github.com/Expensify/App/assets/30054992/7fc9be7a-e878-4a3c-91a1-d2d05e7be577

iOS: mWeb Safari

https://github.com/Expensify/App/assets/30054992/47f46ff1-4fcc-49ce-a368-e808a524627d

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/30054992/228ab781-b0f3-4639-bee3-fe49377fe20f

MacOS: Desktop

https://github.com/Expensify/App/assets/30054992/54fed34b-571b-43e9-b3ec-3ad89934df6d

I can!

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@allroundexperts I guess we don't support disabling dropdown options here yet?

I don't remember adding it although @thienlnam can confirm.

BUG: Fields that get added via pusher updates open to a not found page. Works after refresh.
https://github.com/Expensify/App/assets/27287420/cd4651c0-1c10-4549-8ec4-ffffca9514d8

@jjcoffee Can you please share the url you see at top when the not found page is seen?

Looks to me like the pusher is sending back the fields without the expensify prefix.

Looks to me like the pusher is sending back the fields without the expensify prefix.

@allroundexperts Bingo! /w/6D05DA21B79C2F3A/r/143260955149233/edit/policyField/6D05DA21B79C2F3A/field_id_TEST

Looks like it only updates the fields on the policy, not the report.
"updates": [
    {
      "data": [
        {
          "key": "policy_6D05DA21B79C2F3A",
          "onyxMethod": "merge",
          "value": {
            "fieldList": {
              "field_id_TESTING_PUSHER": {
                "defaultExternalID": null,
                "defaultValue": null,
                "deletable": false,
                "disabledOptions": [],
                "externalID": null,
                "externalIDs": [],
                "fieldID": "field_id_TESTING_PUSHER",
                "isTax": false,
                "keys": [],
                "name": "Testing Pusher",
                "orderWeight": 4,
                "origin": null,
                "target": "expense",
                "type": "text",
                "value": null,
                "values": []
              }
            }
          }
        }
      ],
      "eventType": "onyxApiUpdate"
    }

That's fine. The issue is that pusher is not appending expensify_ with the added field id. cc @thienlnam

Ah yeah I see the policy has duplicate fields with/without the expensify prefix after refreshing.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/thienlnam in version: 1.4.56-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.4.56-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


fix fix commit may fail due to duplicated reports

Proposed changes
Ignore duplicated tablet reports from the same dest backend.
Add BE UT to test the commit info function.


outer 1x2 eyes?

my skin has 1x2 eyes that are 1 pixel off to the sides so the ones available dont really work

I'll take those in count.


SUGGESTION: introduce a hidden dataType

A hidden field would work like string or text so it can be used for vectorizing data (or any other preprocessing module function), it will not be stored after vectorizing. This is ideal for searching through classified or sensitive documents. When querying, one can not collect the original data because it was not stored at all.
Explained based on an example:
{
    "things": {
        "classes": [
            {
                "class": "Email",
                "description": "An email",
                "properties": [
                    {
                        "dataType": [
                            "Address"
                        ],
                        "description": "From email address",
                        "name": "from"
                    },
                    {
                        "dataType": [
                            "Address"
                        ],
                        "description": "To email address",
                        "name": "to"
                    },
                    {
                        "dataType": [
                            "hidden" // <== PROPOSED FIELD
                        ],
                        "description": "Body of the email",
                        "name": "body",
                        "vectorizeClassName": false,
                        "index": true  // <== IT CAN BE USED FOR INDEXING
                    }
                ]
            }
        ]
    }
}

A new use case for this: if one only wants to store the vectors of a document and some related meta-data (the actual original document is stored somewhere else)

I think this would also come in handy for an img2vec module. Images can be huge and in production it's usually smart to serve them from blob storage or a CDN, so there's not need to necessarily store them in weaviate. However, if the img2vec module should vectorize it, the user still has to provide it.
In this case the motivation is not "hiding" it, but rather only saving resources, so I think we could/should come up with a more generic name, but essentially it would be exactly the same feature.
I also think it wouldn't have to be its own type, but it could rather be a flag similar to indexInverted, because I don't think this is specific to only text/string, but might also work for a potential blob type, etc.


Documentation updates content + style

I had a few thoughts while looking at #132 that may be useful for improving our documentation content and styling
Content:

[ ] "Getting started" page - usually the first page I click on, which should be mostly code and have steps for install, instatiation and simple read/write examples.
[ ] Examples/cookbook/common use cases (download files, listing dir, globbing, persistent cache)
[ ] Maybe the authentication/caching pages go under an "Advanced" header in the TOC?
[ ] URI schemes should feature prominently on the homepage/getting started page/methods table, etc. (you have to dig in to find these at the moment)
[ ] Would be helpful to have some diagrams, especially on the caching page
[ ] Example usage on the AnyPath page link AnyPath usage page from the AnyPath API reference page.

Style:

The layout of method signatures can be hard to read. A couple of potential ideas to improve are: (1) remove type hints when laying these out since duplicated below, (2) automatically style these to put each param on a new line, (3) decrease font size, (4)
Can we strip the In/Out blocks from the notebooks that we use in the docs like we do on the DrivenData blog.

automatically style these to put each param on a new line

There's an open issue on mkdocstrings requesting this feature.

we strip the In/Out blocks from the notebooks

We could try mknotebooks instead of mkdocs-jupyter. Based on this demo it seems like it doesn't have cell numbers.


Use OSP17 RabbitMQ image by default instead of the upstream one

/ok-to-test

Got linting working in my editor and per its complaining I removed unnecessary type annotations

/lgtm


OpenThread building problem (IDFGH-13689)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hi everyone,
I'm trying to build an OpenThread Border Router with an Esp32 C6 and an Esp32 H2. I did the installation of ot_br on Esp32 C6 without problems, but when I install ot_rcp on Esp32 H2, I get an error: "RCP is only supported for the SoCs which have IEEE 802.15.4 module".
Can anyone help me? Thank you

Hi @marcokldm02  Which examples did you build? And did you set the target esp32h2 for building OT_RCP?

Thanks for reporting, would you please help share if any updates for the issue? Thanks.

Thanks for reporting, will close due to inactivity, feel free to reopen with more updates.


Verify all appropriate rspec related gems are installed

We uise these rspec gems elsewhere, these are likely sufficient:
  gem 'rspec'
  gem 'rspec_junit_formatter' # used by CircleCI
  gem 'rspec-rails'

Also add rubocop for rspec.


⚠ Bofrost.at has degraded performance

In 15630dc, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2100 ms

Resolved: Bofrost.at performance has improved in 07af225 after 20 minutes.


Support for older versions?

Would've loved to load this immediately into my 1.19.2 modpack. :D
Could you mention in your mod description whether or not you have plans for older versions, like 1.19.2?

Hi! I'm happy you like the mod! :D
I didn't have plans for older versions, but I'm pretty sure it wouldn't be hard to port it for those.
I have a few exams early next week I have to prepare for, but I'll start porting it to all 1.19 versions after, and I'll start with 1.19.2, so you can play as early as possible. Sounds good?

hehe. sounds awesome :D.
I followed the mod on Modrinth already (so I'll get notified of your update) and already added it to my clientside-only modpack.
I'll add the 1.19.2 version to my bigger modpack when you publish it :D.
Well done on doing this mod! I recently was so annoyed by needing to always scroll or reach far to the higher hotbar buttons.
So the timing of this mod is really good.
Gonna try it in a combination with Switcheroo, AutoHUD and Slot Cycler :).

Thanks for the kind words! The idea is not mine though, I just wanted to finally get to learning modding, and make this as a simple client mod. Check out https://github.com/Lizard-Of-Oz/Inventorio, it does the same idea and much more, although it's not client-only.

oh wow. Haven't seen that before! VEEEERY good ideas he has there :D.

yep :D

heyyo, sorry for the long wait, exams hit me harder than expected and I couldn't work on the project in spite of my hopes, but I just finished v0.3.0 with some much needed compat and refactoring, and I'm now going to do the whole 1.19-1.20 lineup. Hoping to release all these in like a day :D

ported it to 1.19.2 and the twenties, but I think I'll hold off on others because it's useless work if nobody wants it

Thank you very much!!! :D
Using it now :D


Move createInternalRequest to InternalRequestFactory

I am thinking about if we should move the internal request creation method into a separate interface.

This is going to be solved by the separation.


fuzzing

Problem
Currently this library is not tested by fuzzing.

A bug #20 is found by go-fuzz. Thank you.