Axiom for induction for church encoding

I'm trying to extend the system with an construct for induction for (non-recursive) church encodings.
I added the construct induction {T} x, with the typing rule:
x : T
---
induction {T} x : [generated induction principle for T and x]

For example for the unit type:
x : {t : *} -> t -> t
---
induction {{t : *} -> t -> t} x : {P : ({t : *} -> t -> t)-> *} -> P (\{t : *} (x : t). x) -> P x

The problem is with the reduction/semantic domain.
What we could do is simply
induction {T} x ~> \{P : T -> *}. x {P x}

But now type preservation (aka subject reduction) does not hold:
\(x : UnitType). induction {UnitType} x : (x : UnitType) -> {P : (UnitType -> *} -> P Unit -> P x
but
\(x : UnitType). \{P : UnitType -> *}. x {P x} : (x : UnitType) -> {P : (UnitType -> *} -> P x -> P x

Do you see anyway to preserve type preservation or is this approach doomed?

I found the solution, the church encoded value has to be fully applied.


Added test scenarios

While reviewing code in order to update our test grid spread sheet, I found minor bugs to fix and opportunities to easily cover a few extra test scenarios.
Also the recent addition of the Merge Train Dialogs feature was causing many tests to not complete correctly, and with one of the other changes it actually broke the test. So I fixed the train.Save function so that it looks for that Merge modal popup and clicks the Save As Is button when it is there.

:tada: This PR is included in version 0.356.2 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


registry_addressfromTestConf` is not used

https://github.com/bpfkorea/agora/pull/1293#r513308839_
It's set to a default in TestConf, but in the ValidatorConfig it's set to an empty string.

Would this solve it?
- validator : ValidatorConfig(true, key_pair, [self_address])
+ validator : ValidatorConfig(true, key_pair, [self_address], test_conf.registry_address)

Yes it would. However there is a bigger problem here. The registry address was null, and yet the tests still passed? That means there are no tests for the registry address (or they don't run properly).

Yes it would. However there is a bigger problem here. The registry address was null, and yet the tests still passed? That means there are no tests for the registry address (or they don't run properly).

checked the code and the network tests doesn't need the address
    public override RemoteAPI!NameRegistryAPI getNameRegistryClient (Address address, Duration timeout)
    {
        auto tid = this.registry.locate("name.registry");
        if (tid != typeof(tid).init)
            return new RemoteAPI!NameRegistryAPI(tid, timeout);
        assert(0, "Trying to access name registry at address '" ~ address ~
               "' without first creating it");
    }

we currently don't have docker based tests where that address might be useful
let me know if I misunderstood something

Maybe a better issue would have been to just use the network registry in the LocalRest tests.
This however requires #1263 to be fixed AFAIK.

Maybe a better issue would have been to just use the network registry in the LocalRest tests.
This however requires #1263 to be fixed AFAIK.

We do have a LocalRest test for NameRegistry: agora.test.NameRegistry
This uses the agora serialization, but that should be fine.
If there are no objections, I would suggest closing this issue.


Roll Flutter (stable) from b06b8b271095 to 7048ed95a5ad (5 revisions)

https://github.com/flutter/flutter/compare/b06b8b271095...7048ed95a5ad
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-stable-plugins
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter (stable): https://github.com/flutter/flutter/issues/new/choose
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


updated .net core build task versions

Updated the .net core build task versions in the vsts build definition from version 1 to 2

@DarqueWarrior it seems the private build agent is no longer working. Can you update the build to use the hosted VSTS agent instead?

I will be looking at all the PRs tomorrow and see what is going on. Thanks.

The build works as expected from master. I will see why it is not working from the PRs.

Awesome, thank you @DarqueWarrior!!! I'd love to start helping out more with this project, so any way at all that I can help, please let me know! One feature I would like to start working on is automatically adding pull request policies after the git repository is created.

Looking at the API documentation, looks like that feature isn't available do to yet. But if there's anything else you can think of for me to help out with, I'd be more than happy to help!

The issue is with how I clone the repo for the build. VSTS clones into a s folder. That does not work for npm modules. The folder name must match the module name. My original solution only works when pulled from master. I am fixing it now to work with PRs as well. Thanks for your patience and support.

When I run integration tests my builds are failing saying no test results are found.

Looks like it has to do with the box being checked to publish tests results with the same task.  When I revert to v1 of the task everything works as expected. I will continue to test and try to correct the build defs. Just changing the version to 2.* breaks the builds.  It sets defaults that don't work with the flow.

The hosted build agents are broken at the moment so all my integration tests are failing once they are patched I will return to this.

@DarqueWarrior any luck with getting the build to succeed?

I will be working on Yo Team this weekend. I have to update the pipeline that tests everything. Sorry for the delay.


Caching the dependencies

There are 2 sources of facts for the resolver:

The Registry tells us for a Dependency what versions are available to fulfil it.
The Summary tells us for a version (and features) what dependencies need to be fulfilled for it to be activated.

The Registry was cached with a RegistryQueryer back in #5112. This adds a DepsCache to cache the calculation of which dependencies are activated by features.
In the happy path flag_activated means that we don't get to reuse build_deps, but the more we backtrack the more time we save. In pathological cases like https://github.com/rust-lang/cargo/issues/6258#issuecomment-479204465, I have measured this as a 10% improvement with release.
This also means that build_deps can be run in a context free way, which may be useful in a follow up PR to solve https://github.com/rust-lang/cargo/issues/6258#issuecomment-482620770.

r? @nrc
(rust_highfive has picked a reviewer for you, use r? to override)

r? @alexcrichton

:umbrella: The latest upstream changes (presumably #6840) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #6860) made this pull request unmergeable. Please resolve the merge conflicts.

How does this look now?

@bors: r+
Looks great to me, thanks for the updates!

:pushpin: Commit 79714ba97418dbdd8cce54ecda1463e5411c7bc8 has been approved by alexcrichton

:hourglass: Testing commit 79714ba97418dbdd8cce54ecda1463e5411c7bc8 with merge 0c891a0091a6c658c8a34df6098f31499ba0a8d9...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: alexcrichton
Pushing 0c891a0091a6c658c8a34df6098f31499ba0a8d9 to master...


Upgrade to Rubocop 0.50.0 (fixes CVE-2017-8418)

Rubocop versions up to 0.49.1 are affected by CVE-2017-8418 (bbatsov/rubocop#4336).
Although it's a pretty minor issue, it makes Gemnasium unhappy:
This PR:

[x] upgrades Rubocop dependency to ~> 0.50.0
[x] fixes correctable offences with rubocop --auto-correct
[x] fixes Style/AsciiComments offence in lib/hyperclient/collection.rb
[x] updates .rubocop_todo.yml with rubocop --auto-gen-config
[x] fixes insecure git source warning from Bundler

Merged thanks


Bump sourceror to v0.8.4

Description
This PR bumps sourceror to v0.8.4.  It enables these optimizations through variables:

Variables that are written to with a known type or constant value will have their reads replaced with the constant or the read type assumed to be the known type.
Writes that are never read are eliminated.
Variables that are never read or written to are eliminated.

Performance improvement varies depending on the code being compiled.
This code is now 5-10% faster:
import {display, get_time} from "std/misc";

function g() {
    const x = 1;
    const y = 2;
    const z = x + y;
    const a = z + x * 2 + y * 3 + (y / x) * 4 + (x / y) * 5;
    const b = (a + a + a + a + a) / z;
    return b;
}
function f(x) {
    return x === 0 ? g() : f(x-1) + f(x-1);
}
const start = get_time();
display(f(26), "Answer:");
display(get_time() - start, "Time (millis):");

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to test
Try out the above code, and observe that it is slightly faster.  Since this is an optimization-only update, things that worked before should still work.
Checklist

[x] I have tested this code

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3178

Relevant Lines:
9793

💛  - Coveralls


Fixed #35140 -- Increased font size in debug views.

Ticket
https://code.djangoproject.com/ticket/35140
Description
Removed the font-size: small CSS property from Django's debug views (404, 500, etc) that was making some fonts be pretty small (13px).
Screenshots
404 Page before changes

404 Page after changes

500 Page before changes

500 Page after changes

@erosselli Thanks :+1: LGTM :astronaut:

Hey there,
Using absolute/fixed sizes in most cases can create an accessibility barrier. Therefore, I agree with removing the small tag; however, care must be taken not to simply remove small and then fall back to a font size value in pixels, as this would create a new barrier. I would suggest using (x, where x ≥ 1)rem in cases where a surgical PR is desired.
Nonetheless, often the most accessible option is the one that appropriately combines ems and rems. This is a much more complex issue that requires a deep understanding of the rest of the HTML/CSS. I see two options: scale and perform an analysis of all the font sizes in the HTML to create em and rem sizes that allow users to freely increase or decrease fonts ensuring consistency, or make a small improvement by applying rem.
In the case where only small is removed, I would verify that the fallback does not use a CSS value in pixels. The most important thing is allowing user to decrease or increase the font freely, for users with tunnel vision small font size is critical to be able to read the content but for low vision may be the opposite. Ensuring layout consistency when increasing or decreasing the font size is also an accessible requirement.
I hope these insights help!

@mugiwarafx Thanks :+1: I changed one font-size to use rem units.

Looking great! I was hoping to review that in time but got sick. Nice one @erosselli!


Refactor and improve NdkService tests

Goal
Expose through the NdkService interface the ability to get all native crashes.  Therefore, I replaced the mocked NDK repository with a fake after extracting its interface and wrote new tests (and modified old ones) to use that.
Testing
New unit tests

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1492
#1491  👈
#1490
main

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

Join @bidetofevil and the rest of your teammates on  Graphite

Merge activity

Oct 9, 1:46 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Calculate key based on multiple locations

It would be great to have the ability to calculate cache key based on multiple locations. Have a look at my example project: https://github.com/railsconfig/config
- uses: actions/cache@v1
  with:
    path: vendor/bundle
    key: ${{ runner.os }}-gem-${{ hashFiles('**/gemfiles/**') }}-${{ hashFiles('*.gemspec') }}

would be better expressed as:
- uses: actions/cache@v1
  with:
    path: vendor/bundle
    key: ${{ runner.os }}-gem-${{ hashFiles(['**/gemfiles/**', '*.gemspec']) }}

Unfortunately, there is no way to pass an array of files to the hashFiles function.

Just ran into this myself - for Gradle a recommended config is to have dependency versions declared in gradle.properties but in our build there are also version numbers in another build.gradle.kts file which in our case doesn't make sense to include in gradle.properties.
For cache purposes both files should be treated as a single unit.

Now that cache@v2 is out with support for multiple paths, it would be really nice to add this next. Example usage for dotnet:
- name: Cache packages
  uses: actions/cache@v2
  with:
    path: ${{ github.workspace }}/.nuget/packages
    key: ${{ runner.os }}-nuget-${{ hashFiles(['.config/dotnet-tools.json', '**/packages.lock.json']) }}
    restore-keys: ${{ runner.os }}-nuget-

i.e. I want to include the dotnet-tools.json file in the hash as well (without workarounds).

👋  Hi all, this is supported by hashFiles but not documented.
To specify multiple patterns, you can pass a comma separated list:
hashFiles('.config/dotnet-tools.json', '**/packages.lock.json')

I will make sure this is documented.

@joshmgross awesome, thanks! 😁
I'm curious, do you guys also own the Azure Pipelines cache task, and if so will this eventually be implemented there too?

do you guys also own the Azure Pipelines cache task, and if so will this eventually be implemented there too?

I, unfortunately, do not work on that task. I'd recommend filing an issue in their repository.

This is now documented: https://docs.github.com/en/actions/reference/context-and-expression-syntax-for-github-actions#example-with-multiple-patterns


vm/isolated: add initial support for fuzzing chromebooks

(WIP PR)
Add support for post_repair_script:

Modify Config{} to contain PostRepairScript.
Allow repair() to execute a post_repair script after reboot. This script executes on the host and expects ssh arguments(user,addr,port,key,options) to be set the environment variable $SSHARGS.

Add pstore support:

Modify Config{} to contain Pstore.
Modify Diagnose() to reboot the DUT and fetch pstore logs, conditional on inst.cfg.Pstore.
Add readPstoreContents().
Allow clearing previous pstore logs upon Create() and after use inside readPstoreContents().
Fetching pstore crashlogs relies on reliably getting lost connection on DUT reboot. Use "ServerAliveInterval=6 ServerAliveCountMax=5" ssh options when running syz-fuzzer with Pstore support enabled.

Allow passing pstore contents into OutputMerger:

Allow isolated.go:instance{} to keep track of instance of *vmimpl.OutputMerger so that it can be used from Diagnose().
Allow vmimpl.Multiplex() to take an argument to decide whether or not to close outc when ssh connection is lost.

Refactoring:

Move out some reusable parts of repair() to waitRebootAndSSH().
Have an early return inside repair() if inst.waitForSSH() fails.

Codecov Report

Merging #1589 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1589      +/-   ##
==========================================
+ Coverage   56.74%   56.79%   +0.04%
==========================================
  Files         144      144
  Lines       27029    27029
==========================================
+ Hits        15337    15350      +13
+ Misses      10913    10900      -13
  Partials      779      779

Impacted Files
Coverage Δ

prog/prog.go
79.89% <0%> (-1.01%)
:arrow_down:

prog/any.go
88.4% <0%> (-0.73%)
:arrow_down:

prog/rand.go
87.65% <0%> (-0.36%)
:arrow_down:

prog/rotation.go
95.41% <0%> (+1.52%)
:arrow_up:

prog/hints.go
91.12% <0%> (+1.61%)
:arrow_up:

prog/mutation.go
94.02% <0%> (+2.71%)
:arrow_up:

Continue to review full report at Codecov.

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


404 Error when downloading BEGIN benchmark

Hi, I encountered 404 Error when downloading BEGIN benchmark.
Maybe update the address, or proving the script to preprocess the open-sourced version BEGIN on github.
Thanks a lot.

you can find the files you are looking for here: https://github.com/google/BEGIN-dataset/tree/6f881995bca5405918f3138b04f29fdd2a1b73ae
Just download them and read them from file.

Thanks both! Submitted a fix, closing.


Challenge http://www.freecodecamp.com/challenges/waypoint-set-the-font-family-of-an-element Issue

When I was doing this challenge, I noticed that the "Go to my next challenge" button is enabled even before completing the line of code.  On line 8, I had to following code:
font-family: Monospace
This is an issue as the CSS line hadn't been closed out yet with a semicolon.  As soon as I typed the last letter of Monospace, the button was enabled.

Closing, see: #2465 for a similar issue


Automated Package Upgrade

:tada: This PR is included in version 25.4.101 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Loading image when next page is rendering

For when next button is clicked, when rendering next page, the image of previous page will stay until the new image is loaded, I think we should add a loading.gif to display before everything is rendered. It looks like a bug when next page is selected.

You can do this by adding a loading flag to your component. This flag should be changed to true when the pagination is changed and to false when the image is loaded, like this:
<template>
  <paginate
    name="items"
    :list="items"
    :per="1"
  >

      Loading...

  </paginate>
</template>

Of course, you can change the loader to anything else you want.


Widgets not showing

I have followed the installation steps and when activated the widgets are not shown.

Try to logout and login again.

I have tried disconnecting and logging in again but it still doesn't show the widgets

Please check your browser's console (Firefox: CTRL + SHIFT + K) for errors.
Can you share your Flarum's URL?

I have been investigating what the possible causes were and I have been able to verify that the widgets do not show if you log in with Google, FaceBook or Twitter. Logging in normally if you get to teach them. URL: https://foro.paranoiasgamers.com

Thanks for the report, I'll see what I can do.

https://github.com/flarum/core/issues/1994
It seems like the LoggedIn event for users isn't dispatched when logging in via an external identity provider. So we probably will wait for that to be fixed.


Ocp, docp, nlp models...

@PierreMartinon still some things to be discussed in the process ocp -> docp -> sol -> plot. Check this old issue

What do you mean ?

@PierreMartinon what I meant 🙂
Use case 1
using OptimalControl
using NLPModelsIpopt # solve (with Ipopt) is  (extension of CTDirect.jl)
using Plots # plot is available (extension of CTBase.jl)

@def ocp ...
sol = solve(ocp) # sol::OCPSolution
plot(sol)

Use case 2
using OptimalControl
using MyNlpSolver # has its own solve for an NLP (e.g., an ADNLPModel)
using Plots # plot is available (extension of CTBase.jl)

@def ocp ...
docp = DirectTranscription(ocp; model=:adnlp) # docp::ADNLPModel
nlp_sol = solve(docp)
sol = build(nlp_sol, ocp) # sol::OCPSolution, more or less OCPSolutionFromDOCP...
plot(sol)

Currently Use Case 2 looks like this
docp = directTranscription(ocp)
dsol = solve(docp, print_level=0, tol=1e-12)
sol1 = OCPSolutionFromDOCP(docp, dsol)

For a custom NLP solver we would typically get something a bit different from the dsol above (this one is an ipopt solution from NLPModelIpopt). We currently have the more low level subfunction actually used to build the solution
function OCPSolutionFromDOCP_raw(docp, T, X, U, v, P;
    objective=0, iterations=0, constraints_violation=0,
    message="No msg", stopping=nothing, success=nothing,
    sol_control_constraints=nothing, sol_state_constraints=nothing, sol_mixed_constraints=nothing, sol_variable_constraints=nothing, mult_control_constraints=nothing, mult_state_constraints=nothing, mult_mixed_constraints=nothing, mult_variable_constraints=nothing, mult_state_box_lower=nothing,
    mult_state_box_upper=nothing, mult_control_box_lower=nothing, mult_control_box_upper=nothing, mult_variable_box_lower=nothing, mult_variable_box_upper=nothing)

Also linked to the older
https://github.com/control-toolbox/CTDirect.jl/issues/74

@jbcaillau Yes this is what I went for.
Currently only the primal, ie sol=ocp_solution_from_nlp(docp, solution)
but the extension to sol=ocp_solution_from_nlp(docp, solution, multipliers) is almost ready.
The part that will be a bit more involved is the range constraints, since they are handled separately and may be less standardized among NLP solvers. An overhaul will be needed here anyway, since this is the scetion with the gazillion variables. I should have a look at other solver interfaces before I rewrite this, for better portability.

@PierreMartinon well, I'd say that it's definitely the user's job to retrieve the multipliers properly. But you are right: although we know exactly the structure of what we pass, i.e. an NLP structured as
F(X) \rightarrow \min\\
L_G \leq G(X) \leq U_G\\
L \leq X \leq U

there might indeed be some variability in the way constraints are treated / multipliers are ordered by the NLP solver.

See https://github.com/control-toolbox/OptimalControl.jl/pull/207 and https://github.com/control-toolbox/CTDirect.jl/pull/154
Have a nice weeekend !

Can we close?

@ocots @PierreMartinon about that, @PierreMartinon are you done with changes in main on build_solution, get_NLP... and everything (use case = user wanting to retrieve an NLP to use his favourite solver) so that we can

make a new release of CTDirect
release OptimalControl 1.0.0

The only change in the API is that the nlp is now outside of the DOCP, ie getNLP has disappeared and direct_transcription now returns both
docp, nlp = direct_transcription(ocp)

while (re)setting the initial guess in the DOCP is now
set_initial_guess(docp, nlp, sol0)

The change is only visible when handling the NLP directly and has no impact on the standard solve calls. The aim was to make DOCP a non mutable struct with dimensions recognized as constants, trying to improve performance.
The only updates needed in OptimalControl are

update the call to the DOCP solving inside solve
update the docs for the NLP parts

(as usual, releases will need to be done together for OptimalControl and CTDirect, but the changes in CTDirect are already merged in main so we can test OptimalControl with a dev'd CTDirect)

If the api has changed the new release of CTDirect should have a different Y.

Yes, that would be 0.10.0 I guess.
I can prepare a PR for OptimalControl too.

@ocots @PierreMartinon @0Yassine0 @frapac first try 🙂
julia> @def ocp begin # definition of the optimal control problem

           tf ∈ R, variable
           t ∈ [ t0, tf ], time
           x = (r, v, m) ∈ R³, state
           u ∈ R, control

           x(t0) == [ r0, v0, m0 ]
           m(tf) == mf,         (1)
           0 ≤ u(t) ≤ 1
           r(t) ≥ r0
           0 ≤ v(t) ≤ vmax

           ẋ(t) == F0(x(t)) + u(t) * F1(x(t))

           r(tf) → max

       end

julia> docp = direct_transcription(ocp; grid_size=5000)

julia> nlp = get_nlp(docp)
ADNLPModel - Model with automatic differentiation backend ADModelBackend{
  ReverseDiffADGradient,
  ReverseDiffADHvprod,
  ForwardDiffADJprod,
  ReverseDiffADJtprod,
  SparseADJacobian,
  SparseReverseADHessian,
  ForwardDiffADGHjvprod,
}
  Problem name: Generic
   All variables: ████████████████████ 20005  All constraints: ████████████████████ 15004
            free: ██████⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 5002              free: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
           lower: █████⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 5001             lower: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
           upper: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                upper: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
         low/upp: ██████████⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 10002          low/upp: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
           fixed: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                fixed: ████████████████████ 15004
          infeas: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0               infeas: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
            nnzh: ( 99.97% sparsity)   55011           linear: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
                                                    nonlinear: ████████████████████ 15004
                                                         nnzj: ( 99.97% sparsity)   105004

  Counters:
             obj: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                 grad: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                 cons: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
        cons_lin: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0             cons_nln: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                 jcon: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
           jgrad: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                  jac: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0              jac_lin: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
         jac_nln: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                jprod: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0            jprod_lin: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
       jprod_nln: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0               jtprod: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0           jtprod_lin: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
      jtprod_nln: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                 hess: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0                hprod: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0
           jhess: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0               jhprod: ⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅⋅ 0

julia> @time ipopt(nlp)
This is Ipopt version 3.14.16, running with linear solver MUMPS 5.7.3.

Number of nonzeros in equality constraint Jacobian...:   105004
Number of nonzeros in inequality constraint Jacobian.:        0
Number of nonzeros in Lagrangian Hessian.............:    55011

Total number of variables............................:    20005
                     variables with only lower bounds:     5001
                variables with lower and upper bounds:    10002
                     variables with only upper bounds:        0
Total number of equality constraints.................:    15004
Total number of inequality constraints...............:        0
        inequality constraints with only lower bounds:        0
   inequality constraints with lower and upper bounds:        0
        inequality constraints with only upper bounds:        0

iter    objective    inf_pr   inf_du lg(mu)  ||d||  lg(rg) alpha_du alpha_pr  ls
   0 -1.0100000e+00 9.00e-01 2.00e+00  -1.0 0.00e+00    -  0.00e+00 0.00e+00   0
   1 -1.0096633e+00 8.66e-01 8.39e+02  -1.0 2.60e+00    -  3.18e-03 3.81e-02f  1
   2 -1.0107757e+00 7.47e-01 2.04e+03  -1.7 1.03e+00    -  3.22e-02 1.38e-01f  1
   3 -1.0113617e+00 5.12e-01 1.01e+03  -1.7 7.47e-01    -  1.31e-01 3.14e-01f  1
   4 -1.0093886e+00 3.01e-01 2.26e+03  -1.7 5.12e-01    -  4.02e-01 4.12e-01f  1
   5 -1.0066535e+00 3.01e-03 4.85e+03  -1.7 3.01e-01    -  4.98e-03 9.90e-01h  1
   6 -1.0079763e+00 3.01e-05 4.52e+02  -1.7 2.89e-01    -  2.33e-01 9.90e-01f  1
   7 -1.0083752e+00 4.10e-06 1.43e+04  -1.7 2.44e-01    -  7.47e-01 9.93e-01f  1
   8 -1.0083340e+00 4.45e-08 3.08e+00  -2.5 3.46e-02    -  1.00e+00 1.00e+00h  1
   9 -1.0083454e+00 8.05e-09 2.72e-01  -3.8 7.88e-03    -  1.00e+00 1.00e+00h  1
iter    objective    inf_pr   inf_du lg(mu)  ||d||  lg(rg) alpha_du alpha_pr  ls
  10 -1.0083655e+00 8.40e-10 1.38e-02  -5.7 2.11e-03    -  1.00e+00 1.00e+00h  1
  11 -1.0096160e+00 3.12e-07 3.57e+01  -8.6 5.98e-02    -  8.06e-01 1.00e+00h  1
  12 -1.0119286e+00 2.37e-06 1.16e+01  -8.6 1.60e-01    -  6.74e-01 8.14e-01h  1
  13 -1.0123971e+00 9.85e-07 1.42e+00  -8.6 1.05e-01    -  8.78e-01 6.88e-01h  1
  14 -1.0125381e+00 4.57e-07 2.58e-01  -8.6 1.26e-01    -  8.18e-01 8.14e-01h  1
  15 -1.0125630e+00 1.51e-07 3.05e-02  -8.6 1.04e-01    -  8.82e-01 8.73e-01f  1
  16 -1.0125660e+00 5.50e-08 1.02e-03  -8.6 7.60e-02    -  9.67e-01 9.69e-01f  1
  17 -1.0125660e+00 2.82e-08 1.27e-04  -8.6 7.61e-02    -  1.00e+00 5.00e-01f  2
  18 -1.0125726e+00 2.57e-08 3.54e-07  -9.0 5.19e-02    -  1.00e+00 1.00e+00h  1

Number of Iterations....: 18

                                   (scaled)                 (unscaled)
Objective...............:  -1.0125726087110427e+00   -1.0125726087110427e+00
Dual infeasibility......:   3.5366498671414553e-07    3.5366498671414553e-07
Constraint violation....:   7.6224774270272633e-09    2.5654078028569671e-08
Variable bound violation:   1.1132245658748600e-31    1.1132245658748600e-31
Complementarity.........:   1.2766492761943900e-09    1.2766492761943900e-09
Overall NLP error.......:   7.6224774270272633e-09    3.5366498671414553e-07

Number of objective function evaluations             = 20
Number of objective gradient evaluations             = 19
Number of equality constraint evaluations            = 20
Number of inequality constraint evaluations          = 0
Number of equality constraint Jacobian evaluations   = 19
Number of inequality constraint Jacobian evaluations = 0
Number of Lagrangian Hessian evaluations             = 18
Total seconds in IPOPT                               = 11.305

EXIT: Optimal Solution Found.
 11.312735 seconds (35.63 M allocations: 1.828 GiB, 19.71% gc time)
"Execution stats: first-order stationary"

julia> @time madnlp(nlp)
This is MadNLP version v0.8.4, running with umfpack

Number of nonzeros in constraint Jacobian............:   105004
Number of nonzeros in Lagrangian Hessian.............:    55011

Total number of variables............................:    20005
                     variables with only lower bounds:     5001
                variables with lower and upper bounds:    10002
                     variables with only upper bounds:        0
Total number of equality constraints.................:    15004
Total number of inequality constraints...............:        0
        inequality constraints with only lower bounds:        0
   inequality constraints with lower and upper bounds:        0
        inequality constraints with only upper bounds:        0

iter    objective    inf_pr   inf_du lg(mu)  ||d||  lg(rg) alpha_du alpha_pr  ls
   0 -1.0100000e+00 9.00e-01 2.00e+00  -1.0 0.00e+00    -  0.00e+00 0.00e+00   0
   1 -1.0096633e+00 8.66e-01 4.25e+01  -1.0 2.60e+00    -  3.18e-03 3.81e-02h  1
   2 -1.0100423e+00 7.03e-01 6.82e+01  -1.0 8.66e-01    -  4.11e-02 1.88e-01h  1
   3 -1.0083987e+00 5.14e-01 8.02e+00  -1.0 7.03e-01    -  4.71e-01 2.69e-01h  1
   4 -1.0061304e+00 5.14e-03 1.01e+02  -1.0 5.14e-01    -  2.20e-02 9.90e-01h  1
   5 -1.0081414e+00 1.52e-03 2.59e+01  -1.0 4.28e-01    -  8.87e-01 7.04e-01h  1
   6 -1.0084646e+00 1.51e-05 1.38e+00  -1.0 1.43e-01    -  1.00e+00 9.90e-01h  1
   7 -1.0085001e+00 1.91e-07 1.35e+01  -1.0 3.29e-02    -  1.00e+00 9.96e-01h  1
   8 -1.0084940e+00 6.88e-10 1.82e-03  -1.0 1.34e-02    -  1.00e+00 1.00e+00h  1
   9 -1.0084940e+00 3.33e-16 1.25e+04  -3.8 2.50e-06    -  9.99e-01 1.00e+00h  1
iter    objective    inf_pr   inf_du lg(mu)  ||d||  lg(rg) alpha_du alpha_pr  ls
  10 -1.0085019e+00 1.44e-11 3.37e-06  -3.8 7.52e-04    -  1.00e+00 1.00e+00h  1
  11 -1.0085172e+00 1.08e-10 3.94e+01  -8.6 1.59e-03    -  9.97e-01 1.00e+00h  1
  12 -1.0114077e+00 3.68e-06 1.52e+01  -8.6 2.22e-01    -  6.13e-01 8.64e-01h  1
  13 -1.0121777e+00 1.94e-06 2.69e+00  -8.6 1.35e-01    -  8.24e-01 6.12e-01h  1
  14 -1.0124634e+00 8.75e-07 3.72e-01  -8.6 1.38e-01    -  8.62e-01 7.13e-01h  1
  15 -1.0125548e+00 3.65e-07 4.57e-02  -8.6 1.21e-01    -  8.87e-01 8.54e-01h  1
  16 -1.0125648e+00 1.16e-07 6.41e-03  -8.6 9.21e-02    -  8.65e-01 8.52e-01h  1
  17 -1.0125660e+00 2.89e-08 5.57e-04  -8.6 1.08e-01    -  9.14e-01 9.11e-01h  1
  18 -1.0125660e+00 4.16e-09 6.65e-09  -8.6 8.27e-02    -  1.00e+00 1.00e+00h  1

Number of Iterations....: 18

                                   (scaled)                 (unscaled)
Objective...............:  -1.0125660371054928e+00   -1.0125660371054928e+00
Dual infeasibility......:   6.6492546317747740e-09    6.6492546317747740e-09
Constraint violation....:   4.1625690672120186e-09    4.1625690672120186e-09
Complementarity.........:   3.1425345666755113e-09    3.1425345666755113e-09
Overall NLP error.......:   6.6492546317747740e-09    6.6492546317747740e-09

Number of objective function evaluations             = 19
Number of objective gradient evaluations             = 19
Number of constraint evaluations                     = 19
Number of constraint Jacobian evaluations            = 19
Number of Lagrangian Hessian evaluations             = 18
Total wall-clock secs in solver (w/o fun. eval./lin. alg.)  =  1.733
Total wall-clock secs in linear solver                      =  4.475
Total wall-clock secs in NLP function evaluations           =  6.742
Total wall-clock secs                                       = 12.950

EXIT: Optimal Solution Found (tol = 1.0e-08).
 12.951147 seconds (35.64 M allocations: 39.980 GiB, 23.45% gc time)
"Execution stats: Optimal Solution Found (tol = 1.0e-08)."

@PierreMartinon for the record: on the current release of CTDirect (v0.10) and associated syntax: to what extent do you need the docp and not the original ocp for post-processing (= plotting, if I am not missing sth)? is it just that you need the time grid, or sth else?

@PierreMartinon for the record: on the current release of CTDirect (v0.10) and associated syntax: to what extent do you need the docp and not the original ocp for post-processing (= plotting, if I am not missing sth)? is it just that you need the time_grid, or sth else?

@PierreMartinon please answer 🥹 and close!

Not sure I understand your question: you only need the solution for plot. You do need the docp to build the ocp functional solution from the discrete docp solution.
For instance from https://github.com/control-toolbox/CTDirect.jl/blob/bolza/test/suite/test_misc.jl
    docp, nlp = direct_transcription(ocp)
    tag = CTDirect.MadNLPTag()
    dsol = CTDirect.solve_docp(tag, docp, nlp, display=false)
    sol = OptimalControlSolution(docp, dsol)
    sol = OptimalControlSolution(docp, primal=dsol.solution)
    sol = OptimalControlSolution(docp, primal=dsol.solution, dual=dsol.multipliers)

You can then do plot(sol) normally.
Or is it something else ?

@PierreMartinon OK. Fine! Closing.


The border line of the table is not visible in dark mode

The problem is as described in the title, which makes it impossible for me to carefully distinguish each item corresponding to the table

Even if I turn off the transparent background, the color of the table borders is still not very distinguishable
Could you tell me how to modify it?

I found the style of the table border through devtools as follows.
.markdown-preview-view th, .markdown-preview-view td{
	border: 1px solid rgb(255 255 255 / 50%);
}

Then I successfully modified it.

I'm having the same issue but creating a snippet with the above style doesn't seem to do anything.

Obsidian version 1.5.3
.table-editor th,
.table-editor td,
.table-editor thead tr>th {
	border: 1px solid rgba(255, 255, 255, 0.3);
}

.markdown-reading-view th,
.markdown-reading-view td,
.markdown-reading-view thead tr>th {
	border: 1px solid rgba(255, 255, 255, 0.3);
}


Account selection isn't clear

Users have found it hard to figure out how to switch wallets. I think we need to make it clearer that this element is a drop down

What do you think about also consolidating the wallets and account selection modal into one? Less modal switching that way and the user can see it all in one place


Rename macro_rules! to macro!

 Issue by bluss
Friday Aug 02, 2013 at 12:50 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/8224
This issue was labelled with: A-syntaxext, B-RFC in the Rust repository

Use a shorter and easier to understand name for this oft-used syntax extension.

Now that 1.0 is out the door, I'm going to close this. We've got more plans for macros underway, however, and I hear they may even use macro...

Did anything ever come of this?

Yes, the experimental feature decl_macro.


Discovery controller has occasional database lock error

Few usages mutex RLock() should be Lock().

@jortel what issue are you seeing that is requiring this change?

Resolved by #639


uuid load error

Setup
Versions

Rust:rustc 1.55.0-nightly
Diesel:1.4.7
Database:postgresql
Ubuntu20.04

Feature Flags
diesel = { version = "1.4.7", features = [ "chrono","postgres", "r2d2","uuidv07","serde_json"] }
diesel_migrations = "1.4.0"
uuid = { version = "0.8", features = ["serde", "v4"] }
chrono = { version = "0.4.19", features = ["serde"] }
anyhow = "1.0.42"
thiserror = "1.0.26"
serde = "1.0.127"
serde_json = "1.0.66"
serde_derive = "1.0.127"
rocket = { version = "0.5.0-rc.1", features = ["json", "uuid"] }

Problem Description
#[derive(Debug, Clone, Serialize, Deserialize, Queryable, Insertable, Identifiable)]
#[serde(crate = "rocket::serde")]
#[table_name = "companys"]
pub struct Company {
    pub id: Uuid,
    pub phone: Option<String>,
    pub name: Option<String>,
    pub attach: Option<String>,
    pub create_at: NaiveDateTime,
    pub update_at: NaiveDateTime,
    pub verify: bool,
}

schema.rs:
table! {
    companys (id) {
        id -> Uuid,
        phone -> Nullable<Varchar>,
        name -> Nullable<Varchar>,
        attach -> Nullable<Text>,
        create_at -> Timestamp,
        update_at -> Timestamp,
        verify -> Bool,
    }
}

when insert company,it's ok; but when load the uuid, it's panic.
use self::companys::dsl::*;
let _id: Vec<uuid::Uuid> = companys.select(id).load(conn).expect("test error");

thread 'rocket-worker-thread' panicked at 'test error: DeserializationError(Error(Build(Error { expected: 16, found: 36 })))', src/company.rs:66:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

What are you trying to accomplish?
I want to get the uuid.
What is the expected output?
load the uuid ok
What is the actual output?
error occur : DeserializationError(Error(Build(Error { expected: 16, found: 36 })))
Are you seeing any additional errors?
Steps to reproduce

Checklist

[x] I have already looked over the issue tracker  and the disussion forum  for similar possible closed issues.

[ ] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)
[ ] This issue can be reproduced without requiring a third party crate

Please provide the complete code including the insert statement that is required to reproduce this, otherwise this is not actionable for anyone. At least the following information are required:

A minimal standalone project showing this issue. (Something where I can just run cargo run and observe the crash)
The corresponding database schema
The confirmation that you can reproduce this on a stable compiler without any third party dependency (depending on uuid is fine for that)

So I had a look at you repo and I think I found the issue: Your schema.rs does not match your actual database schema.
schema.rs:
table! {
    test_companys (id) {
        id -> Uuid,
    }
}

Database schema
CREATE TABLE test_companys (
  id VARCHAR PRIMARY KEY
)

Note that the database schema declares the field as VARCHAR, while your schema.rs declares the field as Uuid. This is an error on your side. You should use the diesel cli tool to generate the schema.rs file based on your database to prevent such mismatches from happening.
Closed as this is not an issue in diesel.

thanks


write_script doesn't get passed to workflow tasks

I'm not sure if the expected behavior should be that write_script on a workflow would get passed to all tasks, but currently it doesn't.

I am unsure that write_script does. The docs say "The Python task also accepts a write_script paramater that when set to 1 will write task scripts to disk before executing them. This aids in readability for interactive debuggers such as pdb."
My suggestions is to have it perform the write for each step of workflows as @danlamanna suggests.

Also, seems it should be set to True, not 1.


PHASE 1: Queue popup - Update text

Update title + text + Button label
https://xd.adobe.com/view/2f942ac5-cea3-4963-b328-7a5036cb11b4-7ac7/

@yaroslav-fedyshyn-nordwhale

A dot is displayed after smile
The "I'LL" is displayed on the button instead of the "I'll"

Checked on DEV V0.25.4-3
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.106
Desktop: Windows 10 // Google Chrome  83.0.4103.116

@AnastasiiaOdnoshevna @yaroslav-fedyshyn-nordwhale
"2" is my bad. It should stay "I'LL"

@yaroslav-fedyshyn-nordwhale then only dot should be removed after smile

@AnastasiiaOdnoshevna I pushed a fix for a dot afte smile into master branch. So after 30 minutes try to test it.

@yaroslav-fedyshyn-nordwhale the pop-up according to require
Checked on DEV V0.25.4-4
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.106
Desktop: Windows 10 // Google Chrome 83.0.4103.116


Prototype of {gsm.app} module plugin framework.

Overview
This branch adds a new parameter to run_gsm_app named lModules, which dynamically adds modules to the app. @jonthegeek, I didn't have time to fully work this out but it hopefully gives you something to work with in developing the approach.
Test Notes/Sample Code
if (basename(getwd()) != 'dev') {
    setwd('dev')
}

library(shiny)
library(DT)
load_all()

lModuleConfig <- yaml::read_yaml(
    'AETimelines.yaml'
)

# TODO: source module functions dynamically with config
source('mod_AETimelines_UI.R')
source('mod_AETimelines_Server.R')
source('AETimelines_Fetch.R')

# All modules should include the following:
# - config file with metadata, a data specification, and a function list
# - UI function
# - server function
# - data fetch function
lModules <- list(
    list(
        lConfig = lModuleConfig,
        fnUI = mod_AETimelines_UI,
        fnServer = mod_AETimelines_Server,
        fnFetch = AETimelines_Fetch
    )
)

run_gsm_app(
    dfAnalyticsInput = gsm.app::sample_dfAnalyticsInput,
    dfBounds = gsm.app::sample_dfBounds,
    dfGroups = gsm.app::sample_dfGroups,
    dfMetrics = gsm.app::sample_dfMetrics,
    dfResults = gsm.app::sample_dfResults,
    fnFetchParticipantData = sample_FetchParticipantData,
    lModules = lModules
)

Connected Issues

Closes #319
Closes #321

The failures are fixed in #317. They're from a change in the fontawesome package. That's the danger of snapshot tests! I'll merge that in once it's ready.

To do:

[x] Add function for reading plugin definitions from a folder.
[x] Test plugin_Read.
[ ] Abstract plugins in UI for easier maintenance.
[ ] Abstract plugins in Server for easier maintenance.
[ ] Test plugins in UI.
[ ] Test plugins in Server.
[ ] Add Dashboard tab.
[ ] Test Dashboard tab.
[ ] Add Timeline tab.
[ ] Test Timeline tab.
[ ] Check code/refactor for maintainability.
[ ] Write tickets for things that aren't done in this PR.

Next steps are to move the AE plugin to a dedicated repository (@jwildfire - maybe {gsm.app.plugins}) and add a vignette on fnFetchData.


[SPARK-46792][PYTHON][CONNECT] Refactor ChannelBuilder into DefaultChannelBuilder and ChannelBuilder

What changes were proposed in this pull request?
Refactor ChannelBuilder into ChannelBuilder and DefaultChannelBuilder. ChannelBuilder is a abstract class with certain base functionality other possible channel builders should inherit. DefaultChannelBuilder is the current implementation of ChannelBuilder.
Why are the changes needed?
The concept of ChannelBuilder was introduced specifically so that different implementations of Channel Builder would exist. However, the current version makes it hard to actually implement custom functionality, because the interface of channel builder and it's standard implementation are tied so closely.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing tests; new e2e test.
Was this patch authored or co-authored using generative AI tooling?
Nope :)

Let's fix up the linter failure

Thanks! Tests green now

Merged to master.


Which model do you load for downstream tasks? model_q or model_k?

Hi,
when I finetune your TARL pre-trained model on downsteam tasks, I notice that there are two models saved, model_q (online updated model) and model_k (momentum updated model). Which one do you load for down-stream tasks?

Hi! We load model_q you can check how we save it in this function: https://github.com/PRBonn/TARL/blob/7f883d658d52bfbedf531727bebcd78bdbdfb3a1/tarl/models/models.py#L52

Thank you for your reply.
The code you quote is only used once at the begining of training. After that, the ckpts are saved by
https://github.com/PRBonn/TARL/blob/7f883d658d52bfbedf531727bebcd78bdbdfb3a1/tarl/tarl_train.py#L47
I think that is the reason why there are both model_q and model_k in the ckpts.


When gun is raised and walking left the hand clips through gun

This is on top of https://github.com/HaywireInteractive/OnAllFronts/pull/267:

https://github.com/HaywireInteractive/OnAllFronts-Public/assets/44318936/1002bc75-d26b-4612-a03e-c968408afde9

https://github.com/HaywireInteractive/OnAllFronts-Public/assets/44318936/8d19f795-a7fe-43bf-8d03-c3d947628a46

https://github.com/HaywireInteractive/OnAllFronts-Public/assets/44318936/5f676d60-8c38-4469-a22b-9a92fd46f6c1


Updates for the switch from BEP-16 to BEP-24

This pull request will have to be reviewed and merged by someone affiliated with the IPDB Foundation.

Made various modifications to account for the switch from BEP-16 to BEP-24.
Also removed the "Shared Workspace Policy" (BEP-6) from the docs about contributing. It was for the BigchainDB GmbH team.
Also removed the line about BigchainDB GmbH hiring, from the docs about contributing.

Signed-off-by: Troy McConaghy<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2641   +/-   ##
=======================================
  Coverage   93.56%   93.56%
=======================================
  Files          45       45
  Lines        2687     2687
=======================================
  Hits         2514     2514
  Misses        173      173


I have trained a .pb model which is in tf1.2 and I want to use it in the environment of tf2.0, but I fail to use it successfully. Anyone knows how to solve the problem

from future import division
from future import print_function
try:
import tensorflow.compat.v1 as tf
tf.disable_v2_behavior()
except (ImportError, AttributeError):
import tensorflow as tf
import tensorflow as tf
import numpy as np
import os
    with tf.Graph().as_default():

        output_graph_def = tf.GraphDef()

        with open('./myModel.pb', "rb") as f:
            output_graph_def.ParseFromString(f.read())
            tf.import_graph_def(output_graph_def, name="")

        gpu_options = tf.GPUOptions(
            per_process_gpu_memory_fraction=self.gpu_memory_fraction,
            visible_device_list=self.visible_device_list,
            allow_growth=self.allow_growth)
        tfconfig = tf.ConfigProto(gpu_options=gpu_options, allow_soft_placement=False)
        tfconfig.gpu_options.allow_growth = True

        with tf.Session(config=tfconfig) as sess:
            sess.run(tf.global_variables_initializer())
            input_photo = sess.graph.get_tensor_by_name('placeholder/photo:0')
            output_photo = sess.graph.get_tensor_by_name('decoder/Sigmoid:0')

################################## erros are as follows:
Traceback (most recent call last):
File "/Users/govan/Project/test.py", line 31, in call
output_graph_def = tf.GraphDef()
AttributeError: module 'tensorflow' has no attribute 'GraphDef'

@Govan111,
Did you follow the instructions mentioned in the Tensorflow website to upgrade from 1.x to 2.x. Thanks

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


CBI consolidation?

First of all, huge props to this project :tada:
The project looks very similar to my CBI, which provides a vendor-neutral CRD interface for docker/buildkit/buildah/kaniko/img/gcb/acb/s2i, and I'm wondering whether we can consolidate if we share the goals:

Is there plan to support non-Google image builders such as BuildKit, S2I, or Azure Container Registry Build as well?
Is there plan to support non-GCS build context storage?
Is there plan to donate Knative to a vendor-neutral organization such as Kubernetes SIG[-apps?] or CNCF? Or planned to be permanently owned by Google?

Hi @AkihiroSuda :wave:
We'd love to work with CBI toward a single abstraction for Kubernetes-native container builds.

Is there plan to support non-Google image builders such as BuildKit, S2I, or Azure Container Registry Build as well?

Absolutely, definitely, 1000%. There are already some non-Google image builders in the build-templates repo. buildpacks were contributed by Pivotal, and there's a PR to add support for buildah. We'd gladly accept more. :hugs:

Is there plan to support non-GCS build context storage?

Absolutely, definitely, 1000%. When GCS support was proposed we discussed other object storage options, and again, we'd gladly accept proposals to support more. Even GCS's API surface is not set in stone, so we can consider other abstractions for common sources.
If there's something in particular you had in mind I would recommend proposing it as an issue on this repo and maybe bringing it up in the weekly Build working group.

Is there plan to donate Knative to a vendor-neutral organization such as Kubernetes SIG[-apps?] or CNCF? Or planned to be permanently owned by Google?

For that you'd have to ask @mchmarny and @dewitt, but I personally would gladly welcome that, for what it's worth.

@AkihiroSuda re your CNCF question, we just opened the repos today and are super excited to engage with the larger community. To be honest haven't spend much time thinking out that far yet

Perfect, thanks!
I'll try to port over CBI features to Knative Build.


Many tests fail on CREODIAS due to disordered times

On CREODIAS, a number of unit tests fail with errors like the following:
Mismatched elements: 3 / 3 (100%)
E       Max absolute difference:<PHONE_NUMBER>00000
E        x: array(['2020-12-02T10:00:00.000000000', '2020-12-03T10:00:00.000000000',
E              '2020-12-01T10:00:00.000000000'], dtype='datetime64[ns]')
E        y: array(['2020-12-01T10:00:00', '2020-12-02T10:00:00',
E              '2020-12-03T10:00:00'], dtype='datetime64[s]')

The differing dtypes and precisions are not the problem here (numpy's assert method handles them) -- the trouble is that the order of dates is shuffled compared to the expected ascending order.

The problem seems to be that the sort order for glob.glob is undefined, and happens to be differ on CREODIAS from what the test expects. Fortunately, many nc2zarr methods accept an input_sort_by parameter, so the tests just need to make use of this.

The problem seems to be that the sort order for glob.glob is undefined, and happens to be differ on CREODIAS from what the test expects. Fortunately, many nc2zarr methods accept an input_sort_by parameter, so the tests just need to make use of this.


promscale secret is missing

While deploying in kubernetes, check service cannot be initialized since it throws a CreateContainerConfigError since secret "tobs-promscale" is missing.
I just followed the instructions to kubectl apply the yaml files in the yaml folder.

Resolved, thanks for the report!


Mac developer install: a howto

Hello, I was able to install the project on a mac with a few tweaks to the python_dev_guide.md.
After making the virtualenv and running python3 -m pip install -e ., I got this error mesage:
 fatal error: 'librdkafka/rdkafka.h' file not found
    #include <librdkafka/rdkafka.h>
             ^~~~~~~~~~~~~~~~~~~~~~
    1 error generated.
    error: command '/usr/bin/clang' failed with exit code 1

Here's the fix:

install brew
install librdkafka by running the command brew install librdkafka
run brew info librdkafka, take note of the library install path (for example, "/opt/homebrew/Cellar/librdkafka/1.8.2/include"),
run export C_INCLUDE_PATH=$LIBRDKAFKA_INCLUDE_PATH, where $LIBRDKAFKA_INCLUDE_PATH is the include path found in step 2.
Run python3 -m pip install -e .

Happy to submit a PR with an update to the python_dev_guide.md if you think this info would be helpful to others; otherwise, I can close this ticket! Cheers

@lhayhurst Would be great if you can open a doc PR fixing this! Appreciate for the help!

@xiaoyongzhu , here's a small PR, cheers

PR is merged


Added a method to enable / disable bluetooth [android / ios]

Adds a method to enable / disable bluetooth. Closes: #31
Tasks:

[ ] Implement Android code
[ ] Implement JS code
[ ] Implement example code
[ ] Write tests
[ ] Ensure flow types are created
[ ] Write documentation

Test check list:

[x] Ran yarn
[ ] Ran yarn docs
[ ] Passed all tests with yarn test
[ ] Ran yarn prettier
[ ] Ran yarn lint
[ ] Tested in example app

Definitely need this feature, when do you plan to merge this?

Make enable/disable functions Promises as they are implemented asyncronously on Android

Sounds good.

Consider extending behaviour by enabling directly, via intent, by suggesting settings page.

I thought about this too. Since iOS won't be able to enable / disable bluetooth automatically like Android, do we want to:

Enable and disable bluetooth on Android, but on iOS open the settings screen. The problem with this is its inconsistent and might be confusing to the developer / user.
Or do we open the settings screen for Android and iOS. The problem with this approach is that you can open the settings screen on iOS, but I don't believe you can directly open the bluetooth settings page. And at this point does it even make sense to call the public API methods: disableBluetooth / enableBluetooth when it opens the settings screen on both Android and iOS.
Lastly, on iOS we don't support enableBluetooth / disableBluetooth and we instead throw an error like you said.

I think the best approach would be to enable and disable bluetooth on Android and throw an error on iOS, but I'm up for a discussion. What do you think @Cierpliwy @DariuszAniszewski ?

Make sure that Flow types are created and documentation is generated explaining implemented behaviour.

Added the flow types to the checklist above. Implementing documentation was already on the TODO list.

Also do you want to rename enableBluetooth() to enable() etc.? I went with enableBluetooth so it was clear what enable was enabling, but I'm for suggestions as I was torn on what the public API names should be.

There is already state() function so enable() and disable() is definitely not out of place.

I think the best approach would be to enable and disable bluetooth on Android and throw an error on iOS and leave it up to the developer to handle this case, but I'm up for a discussion.

Initially I was thinking about:

func
iOS
Android

disableBluetooth()
Error
Success(BleManager) / Error

enableBluetooth()
Error
Success(BleManager) / Error

enableBluetooth(ShowNativeDialog)
Error or custom dialog
Intent

enableBluetooth(GoToSettingsPage)
Go to main settings / BLE settings
Go to BLE settings

but this could be:

func
iOS
Android

disable()
Error
Success(BleManager) / Error

enable()
Error
Success(BleManager) / Error

enable(false)
Error or custom dialog
Intent

openSettings()
Go to main settings / BLE settings
Go to BLE settings

where 2nd function is defined as function enable(showNativeDialog: boolean = false).

There is already state() function so enable() and disable() is definitely not out of place.

Agreed. I will rename it to enable / disable.

but this could be:

I like this a lot more. However, I would simplify it like this:

func
iOS
Android

disable()
Error
Success(BleManager) / Error

enable()
Error
Success(BleManager) / Error

openSettings()
Go to main settings / BLE settings
Go to BLE settings

Thoughts:
The reason I'm not a fan of having showNativeDialog: boolean = false is for the following reasons:

Its trivial to implement a confirmation dialog that prompts the user to enable bluetooth for a developer. It seems like extra overhead in this library, when the developer can do this easily.
This library is not a UI library it should not dictate what the user experience is, the developer / designer should determine that.

To me its easy for a developer to write a native dialog like so and I would encourage them to do this, but its unnecessary to include this behaviour in this library. If they want this behaviour they can simply do:
// Works on both iOS and Android
Alert.alert(
  'Enable bluetooth?',
  'Are you sure you want to enable bluetooth.',
  [
    {text: 'Yes', onPress: () => ble.enable()}, // <----------------------- calls enable()
    {text: 'No', onPress: () => console.log('No Pressed')},
  ]
)

What do you think as after all this is your library? :)

I think this is reasonable. Still we should inform user that he/she is obligated to show alert if he/she enables or disables bluetooth automatically.
On iOS I think we could open BLE settings by calling:
UIApplication.shared.open(URL(string:"App-Prefs:root=Bluetooth")!, ...)
However path depends on iOS version so there will be big switch case ;)

@Cierpliwy Okay sounds good. I will work on this over the weekend.

@Cierpliwy Do you get this error when running yarn test?
 FAIL  __tests__/BleManager.js
  ● Test suite failed to run

    Cannot find module 'NativeModules' from 'react-native-implementation.js'

      at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:191:17)
      at Object.get NativeModules [as NativeModules] (node_modules/react-native/Libraries/react-native/react-native-implementation.js:108:32)
      at Object.<anonymous> (src/BleModule.js:548:45)

Versions:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Everything works fine for me. NativeModules is react-native dependency:
import { NativeModules } from 'react-native';
Are you sure that your RN version is installed properly?

@Cierpliwy
I think so. I did npm install -g react-native-cli. And when running react-native -v inside the root of this project I get:
$ react-native -v
react-native-cli: 2.0.1
react-native: n/a - not inside a React Native project directory

The react-native: n/a - not inside a React Native project directory might be the reason? I don't know why its not detecting the project as a react native project. Any ideas?

@Cierpliwy In this commit https://github.com/Polidea/react-native-ble-plx/pull/139/commits/fa108b57ba4afda45182c4b8dc0dd22d4b6108af I moved the jest config found in package.json into its own file. The reason I did this:

If you run cd examples/ReactBLEScanner and then yarn install you will get the following error when you run yarn test from the root of this project. The error says that there are two node_modules with react-native (one in the root project and one in examples/ReactBLEScanner) and cannot resolve which one to use. So in the jest config I ignore the one in examples/ReactBLEScanner/node_modules. @Cierpliwy you probably never ran into this issue because you never tried running the example and then running yarn test from the root of this project.

 FAIL  __tests__/BleManager.js
  ● Test suite failed to run

    The name `setupDevtools` was looked up in the Haste module map. It cannot be resolved, because there exists several different files, or packages, that provide a module for that particular name and platform. The platform is generic (no extension). You must delete or blacklist files until there remains only one of these:

      * `/Users/adriancarolli/workspace/react-native/react-native-ble-plx/examples/ReactBLEScanner/node_modules/react-native/Libraries/Devtools/setupDevtools.js` (module)
      * `/Users/adriancarolli/workspace/react-native/react-native-ble-plx/node_modules/react-native/Libraries/Core/Devtools/setupDevtools.js` (module)

I also only look for __tests__ inside the root directory. I did this because it will make jest faster and not try to run __tests__ outside of this project i.e. in the examples/ReactBLEScanner folder. They do this in the react repo at Facebook so its pretty reasonable:

https://github.com/facebook/react/blob/d7f6ece27cb7cde67ef0e9c9aea4669f873ddab3/scripts/jest/config.source.js#L16-L17

@Cierpliwy Now that I'm reading more about the Android enable() bluetooth documentation and what people are saying about it here
The docs say:

Bluetooth should never be enabled without direct user consent. If you want to turn on Bluetooth in order to create a wireless connection, you should use the ACTION_REQUEST_ENABLE Intent, which will raise a dialog that requests user permission to turn on Bluetooth. The enable() method is provided only for applications that include a user interface for changing system settings, such as a “power manager” app.
So it's not suitable for you to turn on Bluetooth by the method enable() unless you make the user know what you will do.

Maybe we should make it the default behaviour to request permissions given what it says above ^

@Cierpliwy I am trying to run examples/ReactBLEScanner and I want it to use my local copy of react-native-ble-plx for testing purposes. I tried adding this to package.json:
"react-native-ble-plx": "../../"

But then I run into all kinds of issues. How do you usually test locally when testing native code changes for react-native-ble-plx?

If you run cd examples/ReactBLEScanner and then yarn install you will get the following error when you run yarn test from the root of this project.

I would bother with ReactBLEScanner as there is a separate task to remove it and rewrite it (#189). It's currently in progress and @jerzyKleszcz is taking care of it. Please remove any changes to ReactBLEScanner in this PR.

Maybe you're right we should make it the default behaviour to request permissions given what it says above ^ I don't know I'm on the fence.

I think that requesting permissions is a job of another library.
We can also create special requestEnable function which uses Intent to enable Bluetooth. Unfortunately there isn't such thing as requestDisable. Still I want pure enable and disable functions to be available.

Unfortunately, I just don't have time to work on this. Additionally, the Expo team at Facebook is working on a bluetooth API for react native: https://expo.canny.io/feature-requests/p/bluetooth-api


Using the new SSTable format

@danielbwatson given that we're trying to deprecate the JSON output format, I wonder what's the best way for people to write downstream jobs that want to process data in a row manner?
It seems to be that there are two options:
(1) Run the same Mapper and Reducer used in aegisthus, but use a ChainReducer so that we can add a custom map stage after to do the application specific processing.
(2) The SSTables output by Aegisthus are actually special, since it's guaranteed that rows are non-overlapping and the columns are sorted in the right order. I'm wondering if we could expose this to a mapper in some smart way (and avoid the reduce step).
What do you think?

I'm going to close this issue and add a reference to it in the Enhancement section of the README.


"to two or more instances" appeared twice in the sentence

https://docs.microsoft.com/en-us/azure/app-service/monitor-instances-health-check#what-happens-if-my-app-is-running-on-a-single-instance
"Scale out to two or more instances to two or more instances"
[Enter feedback here]
"to two or more instances" appeared twice in the sentence
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 20c42a13-f6cb-1e60-0438-f7f32b86a336
Version Independent ID: 79e7dd15-24aa-2e6d-cee7-7714e1940c11
Content: Monitor the health of App Service instances - Azure App Service
Content Source: articles/app-service/monitor-instances-health-check.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@gwu70ca, That's a good catch! Thanks for taking time to share your feedback for Azure doc improvement.
We have raised a PR, once the PR is merged (shortly), the requested changes would be published.


⚠ Bofrost.de has degraded performance

In c1247ae, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 954 ms

Resolved: Bofrost.de performance has improved in e7d9774.


Docker 1.5+ required for building ISO

Docker 1.5+ required for building ISO. This is an issue because we want to edit your iso without installing docker. You have solution for this issue?

It's pretty difficult to remove the need for docker in the build.  Can you elaborate on why you can't or don't want to install Docker?  You're always free to modify the ISO using tools like genisoimage, but I'm not sure what you would be adding to the ISO.

@schugh01 I'm going to close this issue, but if you have further feedback, please feel free to reopen.


[$250] Animation to older messages are wrong and New message animation is also wrong

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

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

Open the Desktop app in mobile sized view
Scroll back to earlier in the chat
Get a direct link to a comment
Paste that link as a new message
Click on the link from within the chat

Expected Result:
Should take to the old message with correct animation and clicking on New Message bubble should have correct animation
Actual Result:
Animation to older message is wrong and clicking on New Message there's a right-to-left slide animation that happens
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/9d0323e0-0c50-474e-9a65-ec9879486bfe
https://github.com/Expensify/App/assets/38435837/1d405eeb-f4c0-410f-b55b-2c4e58432f62

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f4f1283f9f5001a1
Upwork Job ID:<PHONE_NUMBER>577249280
Last Price Increase: 2024-06-05

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

I am able to reproduce consistently, fwiw.

Adding a LOW for vip-vsb and opening up to the community.

Please feel free to assign me when it's time for an internal engineer to review the proposal. (I don't do the chore currently, so I will not be auto-assigned.)

Still waiting for proposals.

@JmillsExpensify To inform, I am OOO until 27th May. Either we can reassign to the other C+ or else I will check post my holidays.

Issue not reproducible during KI retests. (First week)

Going to wait and see if this is non-reproducible next week, and then we can close.

Still waiting on proposals

Still waiting for proposals

Issue not reproducible during KI retests. (Second week)

Going to close this since it's not reproducible in the last test run. This kind of thing is also less of a priority as we focus on expense management vs chat.


[Bug]: Canvas notes not coming up on search

Bug Description
Canvas notes not coming up on search in this workflow
Relevant Screenshot
No response
To Reproduce
Activate the workflow, type name of the Canvas note (or 'Canvas') and the notes won't come up.
Canvas issues with Shimmering Obsidian.txt
Debugging Log
Attached above

Workflow Configuration

Version & System Information
-------------------------------
Metadata JSONs: 4/3
Metadata String Length: 2883650
-------------------------------
INSTALLED VERSION
macOS: 12.6.2
Alfred: 5.0.6

Obsidian: 1.1.9
This Workflow: 2.26.8
Advanced URI Plugin: 1.32.0
Metadata Extractor: 1.1.0
-------------------------------
LATEST VERSION
Obsidian: 1.1.9 (Insider: 1.1.13)
This Workflow: 3.6.4
Advanced URI Plugin: 1.32.0
Metadata Extractor: 1.1.0
-------------------------------

Last working version
No response
Make sure you have done the following

[X] I have updated to the latest version of this workflow.
[X] I have updated to the latest version of the Metadata-Extractor and the Advanced URI plugin.
[X] If the o… main search does not work, I made sure that I haven't changed any of the default settings of the Metadata-Extractor. Furthermore, I have tried rerunning osetup and it still does not work.
[X] If notes in general can be found, but recently created or renamed notes cannot, I used the oupdate command to manually update the metadata.
[X] I have not renamed my Obsidian config folder, it's still .obsidian (About → Advanced → Override config folder).
[X] If I updated to version 3+ for the first time, I have disabled migrating workflow cutomizations. (If you haven't, you can simply re-download and install the latest release from GitHub and uncheck Migrate my workflow customizations.)

You have an outdated version of this workflow from before there was the canvas feature in Obsidian 😉
Update to the latest version of this workflow. (you will have to disable workflow migration, and run osetup once again)

Sorry about this, I wasn't aware the updating process had changed and I was confused about how to update it (you mentioning to disable migration did the trick for me).

You have an outdated version of this workflow from before there was the canvas feature in Obsidian 😉
Update to the latest version of this workflow. (you will have to disable workflow migration, and run osetup once again)


Missing Task(CancellationToken)

Hello, I did notice that methods returning tasks do not accept the CancellationToken. Do you mind if I add it?

Go ahead! 👍


1.2 - GEOMESA-1558 Not applying transform iterator for identity transforms

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

this needs some more work - causes issues in geoserver

this is working in geoserver now. for existing layers, it might require deleting and re-loading the layer in geoserver for GML to work


If security is disabled don't attempt to use JACC

Windows Build 1453 is now running using a merge of 24de827dd1a0dd81c20dc75fed78276c10c07d1c

Linux Build 6374 is now running using a merge of 24de827dd1a0dd81c20dc75fed78276c10c07d1c

Linux with security manager Build 1301 is now running using a merge of 24de827dd1a0dd81c20dc75fed78276c10c07d1c

Linux with security manager Build 1301 outcome was SUCCESS using a merge of 24de827dd1a0dd81c20dc75fed78276c10c07d1c
Summary: Tests passed: 826, ignored: 358 Build time: 0:05:28

Windows Build 1453 outcome was SUCCESS using a merge of 24de827dd1a0dd81c20dc75fed78276c10c07d1c
Summary: Tests passed: 2859, ignored: 414 Build time: 0:48:07

Linux Build 6374 outcome was SUCCESS using a merge of 24de827dd1a0dd81c20dc75fed78276c10c07d1c
Summary: Tests passed: 2858, ignored: 414, muted: 1 Build time: 0:50:00


Docker App for chatgpt history GUI

Hello there,
Hope you are doing well,
I am glad to share my changes with you. I developed your app in the docker app and made some changed hope you find that good.
Thank you,

Hey. I'm glad you made the docker version. Unfortunately I cannot accept your changes as-is, I would ask you to make certain changes first, before I can accept. Let me know if you're interested.

Hello Mr. Kuchin,
I hope you are doing well.
I am delighted to hear from you. Yeah, I am very interesting in modifying what you want to change.
Let's begin

oh sorry for these mistakes and the wrong request for review
I will make the changes then I will send another request

Hello Mr. Kuchin,
I hope you are doing well. I didn't hear from you for a while.
Is there any problems in the modifications? If there are any issue, Please comment what you need to modify in this  review request.
Yours Sincerely,
Abdullah


Reconfigure to Kindle PW3

Changed config to London
Doesn't wget to an image; just renders a local png
Added logging
Created a makefile helper for ssh, scp, and retrieving logs from device
Fixed suspend command for PW3

Oops, meant to PR this to my own fork! Closing.


Re-enable Fileinfo extension check on installation (was 5.8-beta6 Error Uploading JPG to File Manager)

Get this error below when attempting to upload jpg in 5.8.  The image will appear w/ a generic thumbnail image...

Error:
{"error":{"type":"Whoops\Exception\ErrorException","message":"Class 'finfo' not found","file":"/home/wildlabs/public_html/thefuture/concrete/vendor/league/flysystem/src/Adapter/Local.php","line":307,"trace":[{"file":"/home/wildlabs/public_html/thefuture/concrete/vendor/league/flysystem/src/Adapter/Local.php","line":307,"function":null,"class":"Whoops\Exception\ErrorException","args":["Class 'finfo' not found"]}]},"errors":["Class 'finfo' not found"]}

You need the fileinfo PHP exception... @aembler what about adding a new check to the install page?

You need the fileinfo PHP exception... @aembler what about adding a new check to the install page?

It's extension, not exception

Thanks @mlocati I will enable on the server

You're welcome!
PS: I'd leave this issue open since we resolve this issue (by requiring fileinfo or find an alternative)

Very good I reopened it, thanks again!

Hmm, this is strange – we used to have this as a requirement but we got rid of it when we forked Flysystem. I thought that getting back to Flysystem when we went PHP 5.5.9 and above would no longer require this but it looks like newer versions have brought back Fileinfo as a requirement ?

@aembler as far as I know that finfo fallback was never merged https://github.com/thephpleague/flysystem/pull/275

Heh. Well that makes it clear that they're probably not gonna be changing this any time soon ;)

Thanks for digging in. Thoughts? It's easy to re-enable the installation check for fileinfo but as you point out there are legitimate reasons for us to rather not make this a hard requirement. Do we know if hosts are still disabling this extension as much as they were 2 years ago? Not sure what we could do about it even if they were – I guess we could refork flysystem.

SiteGround and Arvixe have fileinfo enabled. TMD Hosting does not have it enabled by default on any of their PHP options right up to 7 (but you can turn it on in the cPanel... not ideal). I have one client left on GoDaddy and they do not have fileinfo turned on.  A quick Goggle shows some other frameworks having issues with this as well on some of the major hosting companies.

Thanks for the update @mhawke . Since we can't really get around the requirement at the moment I'm just going to re-enable the installation check. I'd really like to NOT have this be a rrequirement, of course, but perhaps it doesn't matter because fileinfo isn't seen as having problems on versions past our new minimum requirement (5.5.9)

I would like to use file manager without this error and without touching any extention on my server. Concrete5 is easy to use, it's main idea on it, I shouldn't think about ftp or php or extentions or ini file or whatever.
It would be big plus to find a solution on file manager system.


Obj files

The included .cc file in src needs to be compiled and stored in an object file, this will greatly decrease the compile time of this very simple program, I also just need to learn how to do it in the first place. Specifically for Simon Stores

I did not know it at the time but the solution to this problem was CMake. Implemented in the latest release.


Make the import of readline conditional for, eg, Windows

Ensure the module can install and run on Windows where readline is typically not available
AFAICT it's a nice-to-have for the module in general, so a simple conditional import seems reasonable

Thanks!


Capture global scope

TESTed could capture the global scope before and after running a test (main call, expression, statement) such that this global scope could also be evaluated by a custom oracle (the default behavior being that any global scope is accepted; no expected value can be set and no built-in oracle is needed). The global scope could be represented as a mapping of variable names (string) onto strongly typed objects. Custom oracles receive the state of the global scope before and after running the test, so the two can be compared, for example to see that no new global variables have been defined as a side-effect of running a test, that glopbal variables have not been modified, or that global variables have been created or modified).

Think about how this could enable simple exercise like:

The variable x is assigned an array [1, 2, 3, 4]. Add the integer 5 (int) to the end of this array.

So a possible test suite might look like:
- statement: 'x = [1, 2, 3, 4]'                       # setup; storage location of list must also be captured in global scope
- <execute submisson>                             # execute submission
- expression: 'x'                                          # test script
  return: [1, 2, 3, 4, 5]
  oracle: 'custom'                                        # not only checks equality but also identity of the object

This requires some form of setup (code executed before the submission itself is executed). It also requires a way to explicitly put the submission at some place in the test script (by default it is automatically inserted as the first statement/expressoin in the script). This may also serve some other purposes:

embed the submission in skeleton code (i.e. add a prefix and or suffix to the submission) provided in the test suite (e.g. student is only asked to complete a commando line: prefix is given, submission is used as a prefix; so-called fill-in-the-gaps exercise; we could also allow to modify the submission in a programmed way, as we do with test oracles)
???


.avi a little bit heavy

hi,
currently I use the command "scrcpy --record=file1.mp4". The files are a bit heavy. Is there a way to have smaller files?

Encoding is performed by the device. The bit rate is by default requested to 8Mbps (but the encoder decides depending on frame complexity).
Since it's live (hardware) encoding, it's not the most efficient.
For a better quality per size ratio, I suggest to record with a high quality (scrcpy --record=file.mp4 -b16M), then reencode with ffmpeg more slowly. For example:
In H264:
ffmpeg -i file.mp4 -c:v libx264 -qp 25 output.mp4

In AV1:
ffmpeg -i file.mp4 -c:v libsvtav1 -preset 7 -crf 35 -c:a copy output.mp4


feat: allow repositories with nested namespace

Hey, I just made a Pull Request!
I added the possibility to handle raw Gitlab URLs with nested namespaces, ex: https://gitlab.example.com/groupA/TeamB/repo/blob/master/c.yaml
The only problem with this feature is that it can create problems when you call some repository blob, or blob is contained in the namespace. To solve this problem we can add a warning when a URL without /-/blob is used.
I put the blob check starting from length 2 then we maintain at least the compatibility with the old code.
ref #11259
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@antoniomuso Looks good, but seems that there are conflicts remaining to resolve.

@freben Done!


Transfer NFT

Agents should be able to transfer their NFTs to another agent.

Transfer should be to a hidden collection initially to prevent a harassment vector.

Or maybe all NFTs should have a hidden flag that can be toggled by the owner. Hidden NFTs are private.


Long audios

Icefall has PR that aims to support long audios recognition.
How to handle long audios with sherpa?
Thanks in advance.

You can input arbitrary long audios at
http://huggingface.co/spaces/k2-fsa/generate-subtitles-for-videos
which uses VAD to segment long files.


Update scalatest to 3.1.1

Updates org.scalatest:scalatest from 3.1.0 to 3.1.1.
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.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

labels: test-library-update, semver-patch

Pull Request Test Coverage Report for Build 36

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 95.0%

Totals

Change from base Build 34:
0.4%

Covered Lines:
57

Relevant Lines:
60

💛  - Coveralls


Import method does not import disabled features when importing to ActiveRecord adapter

When following the steps to swap adapters (quoted below) to the ActiveRecord adapter, enabled features are added, but disabled features from the remote/source adapter do not get added to the new ActiveRecord adapter (omitted some of the example code for brevity):
# Say you are using redis...
redis_adapter = Flipper::Adapters::Redis.new(Redis.new)
redis_flipper = Flipper.new(redis_adapter)

# And you would like to switch to active record...
ar_adapter = Flipper::Adapters::ActiveRecord.new
ar_flipper = Flipper.new(ar_adapter)

ar_flipper.import(redis_flipper)

The issue occurs in the Flipper::Adapters::Sync::Synchronizer.sync method which is called during the import:
https://github.com/jnunemaker/flipper/blob/85fd90fc8958a83473173dc13f89d8d12126346c/lib/flipper/adapters/sync/synchronizer.rb#L33-L48
Specifically, when line 40 tries to access local_get_all[feature_key], the local_get_all hash is modified because of the way the ActiveRecord adapter defines the hash returned by its get_all method:
https://github.com/jnunemaker/flipper/blob/85fd90fc8958a83473173dc13f89d8d12126346c/lib/flipper/adapters/active_record.rb#L112
If feature_key was not already in the local_get_all hash, it gets added as a result of line 40 in the Flipper::Adapters::Sync::Synchronizer.sync method when @local is an ActiveRecord adapter (not sure if other adapters have similar behavior).
The result is that when execution reaches line 47 in the sync method, any feature_keys that were not already in the local_get_all hash have been added to the hash, and features_to_add will be an empty list and line 48 will not add any of the disabled features from the @remote adapter.
I believe this could be easily fixed by changing line 40 in the Flipper::Adapters::Sync::Synchronizer.sync method to be the following:
local_gates_hash = local_get_all.key?(feature_key) ? local_get_all[feature_key] : @local.default_config

This way the code only tries to access the local_get_all hash if the key is already present, preventing the local_get_all hash from being mutated in the loop. I can add output from a debugging session to help clarify what's happening if needed.
I'm happy to submit a PR for the proposed solution above if it seems reasonable! Otherwise, let me know if there's anything else I can do to help out with this issue.

Wow! Good find. Sorry about that pain it caused. That fix is fine with me. I'm actually thinking maybe having default config like that in the AR adapter is bad. Might be better to just not do that. But I can't currently remember why I did so I'll have to look at it a bit.

No pain, just some confusion! 😄
I went ahead and submitted a PR for what I suggested. If you'd rather take some time to look at the adapter code, no worries! I just suggested fixing the issue this way so the Synchronizer doesn't have to worry about how the individual adapters return the hash from their get_all method.


Feat: Add light and dark theme support

Add light and dark theme support + also add ss while pr

Hey, I would like to work on this. Thanks!

Yeah sure!!

@Rajkumar-justcoder  I would like to Contribute, Please assign this task to me

Yeah sure

I can add this feature to the website. Please assign me the same.

yup sure


disable feedback footer in

The feedback footer shows up in the search result. Not sure if that makes sense. User might feel tempted to evaluate the search result, which is something we cannot do much about. I would just remove it from there to avoid analytics noise.

I would vote to keep the feedback button here as i think it's useful to see if people think the results returned are in line with what they are searching for. I have an assumption that our search functionality is not great and it would be good to get some data to validate that assumption

Going to close as not a priority. Feel free to reopen if you disagree.


Merging issue/7/Support-duplicated-country-names into master

Implementing issue #7 in repository.
This includes:

Adding control to avoid crash when same country name is exported more than once.
Changing airport and country provider url.
Fixing some tests that were failing due to not handling duplicated country names properly.
Adding more tests to guarantee airports and countries are imported properly.

You need to change the version in the project file so that we can publish a new nuget component. I suggest setting it to 1.1.
Also, add the changes to the release notes (PackageReleaseNotes), e.g.
1.1.0:

Accepting duplicates in source data
Pinning source data to specific version
Upgraded dependency to CsvHelper to version 12.2.1


Sync GH release version to backend_version

The deployed grassgis-backend returns
{
  "api_version": "1.0.1",
  "backend_version": "0.2.0",
  "description": "GRASS GIS Driver",
  "endpoints": [
...

while the current github release is 1.0.2. This should be in sync.

Done #93


Adding a shortcut to a button adds extra text to the tooltip.

Godot version:
3.1 Alpha 2
Issue description:
When a shortcut is added to a button the button's tooltip gets extra text describing the shortcut, but there's no way to disable the shortcut from adding extra text to the tooltip. It's also kind of ugly and doesn't really inform the player of anything.

Steps to reproduce:
Create a button. Add a shortcut to it.

Fixed by #23731.

When completing the beginner game 'Dodge' on the Godot manual website, adding a shortcut to the button produces the tool tip as with op.
Godot Engine v3.0.6.stable.official.8314054

It was fixed in 3.1, not in 3.0

I'm still getting this unwanted tooltip in 3.1.1.stable.official.

See the referenced Pull Request, it was made optional, so you have to disable it with shortcut_in_tooltip if you don't want shortcuts in tooltips.

Is it possible to add this boolean in theme in Godot 3.5.1?


Translate jpa

I've done the translation from English to Japanese. What do you think?

@senk8 great thanks. I was hoping that you ll also integrate it in the docs so that users can also choose to look at the japanese version. Are/would you work(ing) on finishing that?

No, I'm not.May I work on this?

@senk8 Yes please ;)


Can no longer generate logo on QR code

Ever since I updated to Android 8.0 I am unable to display the logo on the QR code.
It was working before on Android 7.0 and below.

I have also faced this issue, got logs like this:
D/skia: --- SkAndroidCodec::NewFromStream returned null

@Marchuck were you able to resolve this issue? I have updated to 1.2.0 but I am still unable to generate a logo or background with an image in it

issue was resolved after upgrading my gradle version and build tools


fix: update user data from IDP response

:id:
fixes gravitee-io/issues#7647
:pencil2: A description of the changes proposed in the pull request
This fixes a bug where UserAuthenticationService was not parsing IDP responses for key fields such as email, username etc., and was only updating a user's additional properties. With this fix, a user profile will be updated whenever corresponding values are updated at the IDP.

gravitee-io/issues#7531

I think it's a slightly different fix as this issue deals with updates originating from the IDP.

After discussing with @farmborough same issue as https://github.com/gravitee-io/issues/issues/7531.
As 7531 will be merge in 3.15.x we close this one.

After chatting with @leleueri this does indeed appear to be a duplicate and is already fixed by https://github.com/gravitee-io/issues/issues/7531.


[Button] Not a pointer?

It seems that you guys made the cursor on a button not a pointer on purpose
I tried adding a Tailwind class (cursor-pointer) to make the cursor a pointer, but it didn't work
I was wondering how I can make it a pointer...

https://github.com/radix-ui/themes/issues/123#issuecomment-1778862252

Same problem.
Variables are not being modified
https://www.radix-ui.com/themes/docs/theme/token-reference#modifying-tokens
Radix theme: 2.0.0
NextJS: 14.0.1

I have the same issue, has this been fixed?


predeploy hook not triggering

When using a predeploy hook like:
services:
  web:
    project: ./src/web
    dist: build
    language: js
    host: staticwebapp
    hooks:
      predeploy:
        posix:
          shell: sh
          run: echo AAAAAAAAAAAAAAAAAAAAA && sleep 5
          continueOnError: false
          interactive: true
        windows:
          shell: pwsh
          run: node entrypoint.js -o ./build/env-config.js
          continueOnError: false
          interactive: true

The hook is not triggered when running azd deploy web
Note:  It is triggered by running azd hooks run predeploy

https://github.com/Azure/azure-dev/pull/3345


Add markup highlight style to docs?

I noticed that with a default configuration, syntax highlighting on the light/default theme can result in unreadable text (white text on a very light background).
I wasn't sure how best to fix this in CSS so I ended up looking at your website's repo and noticed you set a markup highlight style that also fixes this for me, so is this worth mentioning in the documentation if it can't be fixed with CSS directly?

You mean using algol theme for syntax highlighting?
Strange, I don't see any problems setting it to my local website:

Perhaps I am missing something.

Based on what I found, it looks like the highlighter's default style is monokai. I noticed you also override your style to bw so wondered if you had encountered the same issue originally.
To reproduce, either change markup.highlight.style = monokai or remove it from hugo.toml. This is what I saw until I set markup.highlight.style to something else.

I see this on code blocks using bash and python formatting and also your own code block if I annotate it as Rust with <three-backticks>rust

This works on plain backquoted code blocks without a language annotation.
Thanks!

Thank you, I added a point to the wiki, perhaps this needs a proper fix.


feat(pred): check and free VRAM for prediction with oversized patch

This PR addresses an issue where users had to restart PlantSeg to recover from out-of-memory (OOM) errors when the combined patch size and patch halo exceed the device's memory capacity. Now VRAM is freed properly.

[x] #268

Almost introduced a bug. Let's write tests.

Free all GPU memory used in between runs


How to change a prop on child element?

Hi,
I discovered this library yesterday and I already love it. Now, I've got a little problem, I'm sure it's just me not understanding the flow but...
I have an element, on which I use branch in order to do a check. If I have true, I need to display a modal, otherwise I display my component as intended. That part works.
Now when I have the modal opened, I have 2 choices, first to be redirected to another page (that works), second is to continue back to my element that was not shown in the first place.
Here's what I have so far on the parent component:
// Using compose to check if the user has enough credit limit remaining
const enhance = compose(
  connect(state => ({
    user: getUser(state), // First getting the user from state
    proceed: true,
  })),
  branch(
    props => props.user.remainingCreditLimit < props.invoice.amount || !props.proceed,
    // If above condition is true, then display the modal
    renderComponent(
      compose(
        withProps(() => ({
          pathToIncreaseCL: router.userIncreaseCreditLimitPath(),
        }))
      )(Modal)
    )
  )
);

// connectedForm is above, a simple connect() call
export default enhance(connectedForm);

Now in my Modal component, I do see pathToIncreaseCL but what I want to do is to modify proceed so that I can go on with my workflow. And I can't find how to get that value change to reflect on the parent component.
Thanks for any help

maybe use withStateHandlers above branch?
const enhance = compose(
  connect(state => ({
    user: getUser(state)
  })),
  withStateHandlers(
    { proceed = true },
    {
      doProceed: () => () => ({ proceed: true }),
      doNotProceed: () => () => ({ proceed: false })
    }
  ),
  branch(
    props => props.user.remainingCreditLimit < props.invoice.amount || !props.proceed,
    // If above condition is true, then display the modal
    renderComponent(
      compose(
        withProps(() => ({
          pathToIncreaseCL: router.userIncreaseCreditLimitPath(),
        }))
      )(Modal)
    )
  )
);

Now Modal should receive the doProceed handler that you can call.

Awesome @timkindberg ! That worked perfectly. Had a little scare, as I was on release 0.23.5 and somehow it didn't have withStateHandlers, but then I switched to 0.24.0 and got it working.
Thanks again.

My fault I've merged withStateHandlers but decided not to publish it, so it was in docs but not at lib itself.


fix: revert code, upgrade server

The main cause of the issue seems to be deep down related to Sentry not supporting way to parse headers with core nodejs causing server crash which caused the outage.
Current issue should be fixed after the upgrade: https://github.com/getsentry/sentry-javascript/pull/10938
logs looks fine

Enabled Always online for dvc.org

Used this opportunity to refactor error handling

@yathomasi so, why did we have the issue with GH itself though - what was the error Sentry was trying to report about?

@yathomasi so, why did we have the issue with GH itself though - what was the error Sentry was trying to report about?

It was trying to set headers to tracking outgoing requests, Github api in this case. I feel, it's not required in our project specially in the nodejs side.

It was trying to set headers to tracking outgoing requests, Github api in this case. I feel, it's not required in our project specially in the nodejs side.

kk, I see. thanks.


Be able to handle a smoking-crater broker

It's inevitable that we will need to be able to handle the case of a broker that suddenly goes away (ie, smoking crater situation).  If a broker suddenly disappears, we should be able to do the following:

Keep the active instances and bindings around (provided that they still work) but prevent new instances and bindings (which will not work anyway, if the broker is no longer around)
Be able to delete instances and bindings without having to contact the broker, but still leveraging finalization
Disable further broker re-lists (I think #705 should handle this)

I have some ideas about how to do this, but wanted to get what I think the requirements are out first.

do you foresee this requiring a manual step to say that I know this broker has gone away, or being able to detect it? Former is a bit easier :)

IMO its manual because it'll be hard to know when its just temporarily on vacation vs dead-and-buried.  :-)

I could see this being a manual step that would set a condition on the broker.


Cannot create Soul Shard

Not sure what is happening, but I'm unable to create a Soul Shard in singleplayer (have not tested multiplayer). Here's my modlist:

architectury-4.4.68.jar
soulshards-1.18.2-1.2.2.jar
polylib-forge-1801.0.2-build.13.jar
The pattern I tried (and using normal quartz blocks too, this is the picture with quartz pillars):

I tried downgrading Architectury a few versions, that didn't work too. I also tried to make a multiblock.json file in the root of a datapack (data/multiblock.json) as seen in the files of this repo, but it didn't work either. I tried standing on top and inside the glowstone block as well (using pistons).
I am right clicking the glowstone block in the picture with a diamond
Forge 40.1.25
Minecraft 1.18.2

Wrong quartz blocks...

Works for me in dev/test instance with just it + deps installed and in the Forgecraft instance.
https://ss.gigabit101.net/javaw_lgM5pU8Oj7.gif

Do I have to use Smooth Quartz? I'll retry again with it in a bit

You need to use Block of Quartz

Tried with block of quartz and it works now, apologies for the report and thanks for your help!

Block of quartz
On Mon, 30 May 2022, 22:26 R3GEN, @.***> wrote:

Do I have to use Smooth Quartz? I'll retry again with it in a bit
—
Reply to this email directly, view it on GitHub
https://github.com/CreeperHost/SoulShards/issues/5#issuecomment-1141485884,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABK26TRQK27QF4P5AUVLBI3VMUXBRANCNFSM5XJJNGHA
.
You are receiving this because you commented.Message ID:
@.***>


Gradio version

Gradio has been updated to 3.19 which is not compatible with the current setup. Should add "gradio" to requirements.txt and force the version tp 3.18.

Hey @AlonEfrati I just installed the latest version of gradio (3.19.1), and the app seems to work fine without changing anything, what exactly isn't working for you?

Note also that gradio is not a requirement of the project and is not in requirements.txt so you can install whatever version you want