text:
Manual installation + Julia wrapper for penbmi function

This adds manual installation instructions to install penbmi as well as wrapping the C function with a Julia function and a basic test based on the example given with penbmi.
On my computer, running the tests give:

vars       :     3
constr     :     4
mconstr    :     1
msizes     :     3

*********************************************
PENBMI 2.1
------------

*********************************************

********************************************************************************
* it |     obj      |     opt      |     feas     |   <U,A(x)>   | Nwt  |  LS  *
********************************************************************************
|   0|   0.0000e+00 |   7.5763e-08 |   4.5183e+00 |   1.2500e+02 |    0 |    0 |
|   1|   7.8275e+00 |   2.1625e-03 |   0.0000e+00 |   7.1953e+00 |   27 |   27 |
|   2|   3.8779e+00 |   3.7622e-03 |   1.6252e-01 |   4.0844e-01 |   31 |   33 |
|   3|   3.9733e+00 |   2.5533e-02 |   6.5687e-02 |   8.2627e-02 |   32 |   34 |
|   4|   4.0316e+00 |   2.9491e-02 |   7.1571e-03 |   1.5194e-02 |   33 |   35 |
|   5|   4.0431e+00 |   3.1881e-03 |   0.0000e+00 |   6.3835e-03 |   34 |   36 |
|   6|   4.0391e+00 |   1.1232e-03 |   0.0000e+00 |   8.5027e-04 |   35 |   37 |
|   7|   4.0388e+00 |   3.3832e-05 |   0.0000e+00 |   1.6736e-04 |   36 |   38 |
|   8|   4.0387e+00 |   5.3884e-06 |   0.0000e+00 |   2.3343e-05 |   37 |   39 |
|   9|   4.0387e+00 |   7.5281e-08 |   0.0000e+00 |   4.1882e-06 |   38 |   40 |
|  10|   4.0387e+00 |   1.3113e-09 |   0.0000e+00 |   8.2625e-07 |   39 |   41 |
|  11|   4.0387e+00 |   1.5451e-11 |   0.0000e+00 |   1.7077e-07 |   40 |   42 |
|  12|   4.0387e+00 |   2.2199e-13 |   0.0000e+00 |   3.5759e-08 |   41 |   43 |
|  13|   4.0387e+00 |   2.2199e-13 |   0.0000e+00 |   7.5667e-09 |   42 |   93 |

********************************************************************************
F(x_13) =                         4.0386976747102246E+00
f(x_13) =                         4.0386978216703939E+00
||f(x_13) -  F(x_13)|| =          1.4696016936710521E-07
||f(x_13) -  f(x_12)|| =          0.0000000000000000E+00
Feasibility (LI)                 -3.1887107050726876E-01
Slackness (LI)                    7.5107834552811907E-09
Feasibility (BMI)                 0.0000000000000000E+00
Slackness   (BMI)                 5.5900895024052488E-11
||grad_x(L(x,U))||                7.5763117809832070E-08
lambda_min(hess_x(L(x,U)))  >=   -0.0000000000000000E+00
Outer Iterations                                      13
Inner Iterations                                      42
Linesearch steps                                      93
Start time                      Thu Aug  8 14:56:27 2019
End time                        Thu Aug  8 14:56:27 2019
Real time                              0 h  0 min  0 sec
Process time (total)                         0.00138 sec
********************************************************************************

Stopped by linesearch failure. Result may be wrong.

PENBMI failed.
Test Summary:  | Pass  Total
Penbmi example |    6      6
Test.DefaultTestSet("Penbmi example", Any[], 6, false)

cc @kocvara

Hi Benoit,
I downloaded everything in the bl/penbmi branch in my local directory
.julia/packages/Penopt/6RCzO
and followed your instructions to create the deps folder and the deps.jl file in it. I am on Mac, so I compiled an .so library of penbmi and placed it in deps/usr/lib, as instructed.
Trying to run the test, I said (in julia)
using JuMP
using Penopt
import Pkg; Pkg.add("Test")
using Pkg; Pkg.test("Penopt”)
and got the error message shown below. So it seems that penbmi wasn’t even called yet. When I do it with SeDuMi, the test runs fine. Running import Pkg; Pkg.add("Test”) again doesn’t help. Can you help? It seems that I still need to install or set up something relating Penopt(?).
Thanks,
Michal
julia> using Pkg; Pkg.test("Penopt")
Testing Penopt
Status /var/folders/zs/rqfnjvys7z9fb1lxymgxsb240000gn/T/tmpQAf1PK/Manifest.toml
[fb8f6f4e] Penopt v0.0.0 #master (https://github.com/JuliaOpt/Penopt.jl.git)
[8f399da3] Libdl  [@stdlib/Libdl]
ERROR: LoadError: LoadError: ArgumentError: Package Test not found in current path:

Run import Pkg; Pkg.add("Test") to install the Test package.

Stacktrace:
[1] require(::Module, ::Symbol) at ./loading.jl:823
[2] include at ./boot.jl:326 [inlined]
[3] include_relative(::Module, ::String) at ./loading.jl:1038
[4] include(::Module, ::String) at ./sysimg.jl:29
[5] include(::String) at ./client.jl:403
[6] top-level scope at none:0
[7] include at ./boot.jl:326 [inlined]
[8] include_relative(::Module, ::String) at ./loading.jl:1038
[9] include(::Module, ::String) at ./sysimg.jl:29
[10] include(::String) at ./client.jl:403
[11] top-level scope at none:0
in expression starting at /Users/michal/.julia/packages/Penopt/6RCzO/test/bmi.jl:2
in expression starting at /Users/michal/.julia/packages/Penopt/6RCzO/test/runtests.jl:1
ERROR: Package Penopt errored during testing
Stacktrace:
[1] pkgerror(::String, ::Vararg{String,N} where N) at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/Types.jl:120
[2] #test#66(::Bool, ::Function, ::Pkg.Types.Context, ::Array{Pkg.Types.PackageSpec,1}) at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/Operations.jl:1329
[3] #test at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:0 [inlined]
[4] #test#46(::Bool, ::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::Function, ::Pkg.Types.Context, ::Array{Pkg.Types.PackageSpec,1}) at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:198
[5] test at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:183 [inlined]
[6] #test#45 at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:180 [inlined]
[7] test at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:180 [inlined]
[8] #test#44 at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:179 [inlined]
[9] test at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:179 [inlined]
[10] #test#43(::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::Function, ::String) at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:178
[11] test(::String) at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.1/Pkg/src/API.jl:178
[12] top-level scope at none:0

@kocvara This should be fixed by e3a2000cee53e9e061c0284d5f78220fbf721305

Great, it works now, thanks. Two of the tests did not pass but that’s because of a different system and libs and perhaps also slightly different initial settings.
BEst,
Michal

On 11 Aug 2019, at 08:19, Benoît Legat notifications@github.com wrote:
@kocvara https://github.com/kocvara This should be fixed by e3a2000 https://github.com/JuliaOpt/Penopt.jl/commit/e3a2000cee53e9e061c0284d5f78220fbf721305
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/JuliaOpt/Penopt.jl/pull/1?email_source=notifications&email_token=AFIXXOU5M7SSUNR3GEULQ53QD64RBA5CNFSM4IKT7HCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4A3QOY#issuecomment-520206395, or mute the thread https://github.com/notifications/unsubscribe-auth/AFIXXOQWYRUTKRE5ANZSOVDQD64RBANCNFSM4IKT7HCA.

I am now confused by the naming:
model = Model(with_optimizer(Penbmi.Optimizer))
ERROR: UndefVarError: Penbmi not defined
and
model = Model(with_optimizer(Penopt.Optimizer))
ERROR: UndefVarError: Optimizer not defined

Two of the tests did not pass but that’s because of a different system and libs and perhaps also slightly different initial settings.

Yes, the tests are definitely too tight, should be fixed in 0861d62

WARNING: This is a work in progress, don't trust anything written below.

Yes, that's the reason of the warning at the beginning of the README, this will work once have have the MathOptInterface wrapper.

Re tests: Problematic are the iresults, I am getting
Evaluated: Int32[13, 40, 43, 0] ≈ [13, 42, 93, 0]
because the linesearch goes slightly differently; I also finish without the error message:
Expression: info == 3
Evaluated: 0 == 3
Perhaps we should just test for the reals?

Perhaps we should just test for the reals?

Sounds good, done in fd4ffe5

Test Summary:  | Pass  Total
Penbmi example |    7      7
Testing Penopt tests passed

authors: blegat kocvara


text:
New package: Fminbnd v1.0.0

Registering package: Fminbnd
Repository: https://github.com/kagalenko-m-b/Fminbnd.jl
Created by: @kagalenko-m-b
Version: v1.0.0
Commit: d58e1aaa088c0e18e5a5615ac9ff689c5e05ee5e
Reviewed by: @kagalenko-m-b
Reference: https://github.com/kagalenko-m-b/Fminbnd.jl/commit/d58e1aaa088c0e18e5a5615ac9ff689c5e05ee5e#commitcomment-149262897

This seems to be the followup from https://github.com/JuliaRegistries/General/pull/99531.
I'm not really clear on what the bnd part of the name should indicate.

I'd also strongly recommend leaving the name of the function exported by the package as fmin.

I might tentatively suggest NetlibFmin as a name

There’re a widely used matlab/octave functions ‘fminbnd’ and python’s ‘fminbound’ that also derive from netlib’s fmin. “Bnd” means that it searches minimum on a bounded interval.

Hmm… okay, but you're very explicitly translating the Netlib code, which doesn't use that name 🤷

I would also point out that your signature does not match the Matlab fminbnd function (wrong order of arguments), but it does match the signature of Netlib's fmin

Same Brent's method seems to be available in Optim.jl, see https://julianlsolvers.github.io/Optim.jl/stable/user/minimization/#Minimizing-a-univariate-function-on-a-bounded-interval

Same Brent's method seems to be available in Optim.jl, see https://julianlsolvers.github.io/Optim.jl/stable/user/minimization/#Minimizing-a-univariate-function-on-a-bounded-interval

Optim is a very large and complex package with many dependencies.

rule 8 of package naming guidelines:

Avoid using a distinctive name that is already in use in a well known, unrelated project.

fminbnd is

distinctive
in use in MATLAB which is well known and
"unrelated" because you are not using its implementation

also rule 4

Err on the side of clarity, even if clarity seems long-winded to you.

I think NetlibFmin seems much more clear to me what this package does given your testimony that this is a 1-1 rewrite of that function.
naming aside, and I'm sorry to be too critical, but I would be a bit hesitant to say that this meets the quality standards to register in General in the first place. This package consists of a single function that is completely illegible. Could you at least try to factor out some of the @gotos into traditional control flow? compare to for example scipy's implementation

I wouldn't come down quite as hard on this. I think if it implemented the same interface as Matlab and was documented as such (instead of a translation of the Netlib code), I'd be totally fine with Fminbnd.
Likewise, the way it is implemented now, NetlibFmin would be totally fine. I'm saying this coming from a Fortran background where Netlib is the go-to if you need some code that you don't want to write yourself (most of it being in the public domain). So I don't even object to a Julia package that translates an existing Netlib package, even if it borders on being trivially small in this case.

Could you at least try to factor out some of the @gotos into traditional control flow?

With the direct reference to Netlib, it's kinda cute, and I don't think I'd mind. This is really as close of a translation of the original code as possible. It would be interesting to also write this with modern (haha) control flows and benchmark if there is any difference at all in performance.

“unrelated" because you are not using its implementation

Different implementation of the same algorithm is “unrelated”?

think if it implemented the same interface as Matlab and was documented as such (instead of a translation of the Netlib code), I'd be totally fine with Fminbnd

https://github.com/kagalenko-m-b/Fminbnd.jl/commit/b52ae0c07af2413a1de3bc8901f20b534a2b1c65

Okay, that seems good to me… Just make sure that you retrigger the registration to really make sure that the correct commit gets tagged. Especially since you're using v1.0.0 for this

Let me know when this is good to go, and I’ll apply the override to merge it

Let me know when this is good to go, and I’ll apply the override to merge it

I believe it’s as ready as it’ll ever be. Not added the tag yet, but the comment from registrar states that it should be done automatically via tagbot after the package registration.

Yes, do not create tags to create releases. Releases have to be registered first and then tagged afterwards (preferably via the bot)

Looks like tagbot triggered automatically, thanks.

source: gharchive/pull-request

authors: JuliaRegistrator adienes goerz jaakkor2 kagalenko-m-b

repo: JuliaRegistries/General

license: MIT

license_type: permissive

license_source: github-api


text:
New package: SARSOP_jll v0.96.0+0

Registering package: SARSOP_jll
Repository: https://github.com/JuliaPOMDP/SARSOP_jll.jl
Created by: @zsunberg
Version: v0.96.0+0
Commit: cf573e746bd6297a1c7ce02a301a11f506f260ec
Reviewed by: @zsunberg
Reference: https://github.com/JuliaPOMDP/SARSOP_jll.jl/commit/cf573e746bd6297a1c7ce02a301a11f506f260ec#commitcomment-42799817

Please don't merge - this is not correct

authors: JuliaRegistrator zsunberg


text:
JLL Registration: JuliaBinaryWrappers/Xorg_xcb_util_keysyms_jll.jl-v0.4.0+0

Autogenerated JLL package registration

Registering JLL package Xorg_xcb_util_keysyms_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/Xorg_xcb_util_keysyms_jll.jl
Version: v0.4.0+0

This PR probably contains a bug.  Please, don't merge it before https://github.com/JuliaPackaging/BinaryBuilder.jl/pull/531.

authors: giordano jlbuild


text:
New version: Sundials_jll v6.7.0+0

Autogenerated JLL package registration

Registering JLL package Sundials_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/Sundials_jll.jl
Version: v6.7.0+0
Commit: c9be19e45cb3af4f4a5ef4f160c6e41ba9d7f484
Revision on Yggdrasil: https://github.com/JuliaPackaging/Yggdrasil/commit/7ca1142489cd4bb1cf745645c8a8c09104919056
Created by: @staticfloat

[noblock]
Looks like there is a merge conflict.
@staticfloat  Can you re-trigger a registration in Ygg? This will fix the merge conflict.

[noblock]
It's not so easy: https://buildkite.com/julialang/yggdrasil/builds/8277#018d8ac9-3c6f-42aa-9668-e945d5931a7f

[noblock]
Can you delete the old release assets?

That seems quite drastic, why not just resolve the conflicts here?

authors: DilumAluthge jlbuild staticfloat


text:
Filetype version detection is "broken" in telescope previewing windows

This may turn out to be a telescope issue, but using telescope.builtin.find_files() potentially uses the wrong version of Lean for its preview windows.
In other words, the preview window in a Lean 3 project sometimes renders previews using the Lean 4 filetype.

OK, this is mentioned here (which leads here) and doesn't seem trivial to fix (in that I don't see any other plenary.filetype things that do any complex logic).
Closing, we can just live with this.

source: gharchive/issue


text:
Find this lib in SDK

Where can I found this lib. in SDK to use in Eclipse??

you juste have to add this dependency to gradle:
com.android.support:percent:22.2.0

In Eclipse how can I use this ??
On Sat, Jun 27, 2015 at 1:42 PM, Julien Genoud notifications@github.com
wrote:

you juste have to add this dependency to gradle:
com.android.support:percent:22.2.0
—
Reply to this email directly or view it on GitHub
https://github.com/JulienGenoud/android-percent-support-lib-sample/issues/1#issuecomment-115983736
.

try to follow this steps :
https://developer.android.com/tools/support-library/setup.html#libs-with-res

Yes, but in SDK where I can found this ??
I have Support Design Lib. imported in Eclipse
On Sat, Jun 27, 2015 at 2:06 PM, Julien Genoud notifications@github.com
wrote:

try to follow this steps :
https://developer.android.com/tools/support-library/setup.html#libs-with-res
—
Reply to this email directly or view it on GitHub
https://github.com/JulienGenoud/android-percent-support-lib-sample/issues/1#issuecomment-115986497
.

I don't really know how to do ...
maybe it's time to migrate to android studio ?
http://android-developers.blogspot.fr/2015/06/an-update-on-eclipse-android-developer.html

Yup. It's the time to ditch Eclipse and move on to Android Studio already.
I'm not very sure this is the correct path but I've found the percent-22.2.0.aar under this location
ANDROID_SDK_PATH/extras/android/m2repository/com/android/support/percent/22.2.0/
And the contents.
λ ~:android-sdk ls -la extras/android/m2repository/com/android/support/percent/22.2.0/
total 72
drwxr-xr-x  8 yelinaung  staff    272 May 29 11:50 .
drwxr-xr-x  7 yelinaung  staff    238 Jun 29 12:21 ..
-rw-r--r--  1 yelinaung  staff  12466 May 29 11:50 percent-22.2.0.aar
-rw-r--r--  1 yelinaung  staff     32 May 29 11:50 percent-22.2.0.aar.md5
-rw-r--r--  1 yelinaung  staff     40 May 29 11:50 percent-22.2.0.aar.sha1
-rw-r--r--  1 yelinaung  staff    670 May 29 11:50 percent-22.2.0.pom
-rw-r--r--  1 yelinaung  staff     32 May 29 11:50 percent-22.2.0.pom.md5
-rw-r--r--  1 yelinaung  staff     40 May 29 11:50 percent-22.2.0.pom.sha1

authors: JulienGenoud patelakshay13890 yelinaung

license: mit

license_source: bigquery


text:
Make it possible for ays cli to pass jwt

Also add helper method to create jwt
Signed-off-by: Jo De Boeck deboeck.jo@gmail.com

+1 for docs. Otherwise LGTM

authors: grimpy rkhamis


text:
S3 actor fails with error when installing scality

JSExceptions.RuntimeError: ERROR: Could not execute cmd:export S3METADATAPATH="/data/meta" && export S3DATAPATH="/data/data" && cd
/opt/jumpscale8/apps/S3 && npm start
Out/Err:
-bash: npm: command not found
     ((type:runtime.error)

@yveskerwyn on which version did this happen?

version 8.2.0
Installing S3 using 8.1.1 still works, but when installing using 8.2.0 we run in all kind of problems

authors: rkhamis yveskerwyn


text:
#122

connect to #122

passed

Классы EventTechnologySqlBuilder, FilteredEventsSqlBuilder, FilterTechnologySqlBuilder.
Что означают magic numbers в этих выражениях? Почему у них именно такие значения?
sql.delete(sql.length() - 2, sql.length());
sb.delete(sb.length() - 4, sb.length());

Столько символов нужно отнять чтобы убрать лишнюю запятую

Через месяц некий Вася захочет изменить билдеры (добавить метод и т.п.). Ему тоже надо будет к тебе обращаться за личной консультацией?

Предложи как ее назвать, у меня фантазии нет

lastCommaPositionFromEnd

Please, edit your comments above and put person name to whom you are writing (person nick starts with @). Leave this habbit for future
@alex-anakin https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179201038
@alex-anakin https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179202829
@vaa25 https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179202178
@vaa25 https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179205690
@romach https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179205690

authors: alex-anakin roma-ilnitsky romach vaa25 vkuchyn

repo: JuniorsJava/itevents


text:
[Feature request] Queued startup command argument to start bot instance

[Setup: ]

I have all my bots instances defined but disabled in confirg
I start both with WCF support
I always operate bot through WCF

[Issue: ]
I want to start the bot in WCF mode and instead of starting all instances or specified instances of the bots through config I want to pass him the command that starts specific bots once ASF is up. Problem is, I cannot pass both --server and --client commands at once, the ASFwill throw exception due to the fact WCF is not up.
[Request: ]
Would be nice to have an ability to queue WCF client command and execute after WCF is up. This way I don't have to start all 9999 bots at once or edit config every time, I can just have them defined to start separately each bot I want in specific set up.

Your attemped usage of ASF is semi-wrong. Things such as adding delay and executing client in another process should be done on script or OS level, and not in the application.
archi@archi:~/ArchiSteamFarm/ArchiSteamFarm/bin/Release$ mono ArchiSteamFarm.exe --server &>/dev/null & sleep 10; mono ArchiSteamFarm.exe --client "status archi"
[1] 10854
2016-11-13 15:42:44|INFO|ASF|Init() ASF V2.1.6.8
2016-11-13 15:42:44|INFO|ASF|IsRuntimeSupported() Your Mono version is OK. Required: 4.6 | Found: 4.9.0
2016-11-13 15:42:45|INFO|ASF|ParsePostInitArgs() Command sent: status archi
2016-11-13 15:42:46|INFO|ASF|ParsePostInitArgs() Response received: Bot archi is not connected.

This is how you should do that, and it was always possible to do it this way. But sure, I improved WCF a little and made it possible to use --server --client "command" combination too.
archi@archi:~/ArchiSteamFarm/ArchiSteamFarm/bin/Release$ mono ArchiSteamFarm.exe --server --client "status archi"
2016-11-13 15:53:06|INFO|ASF|Init() ASF V2.1.6.8
2016-11-13 15:53:06|INFO|ASF|IsRuntimeSupported() Your Mono version is OK. Required: 4.6 | Found: 4.9.0
2016-11-13 15:53:07|INFO|ASF|StartServer() Starting WCF server...
2016-11-13 15:53:07|INFO|ASF|StartServer() WCF server ready!
2016-11-13 15:53:13|INFO|archi|Start() Starting...
2016-11-13 15:53:13|INFO|ASF|ParsePostInitArgs() Command sent: status archi
2016-11-13 15:53:15|INFO|ASF|HandleCommand() Answered to command: status archi with: Bot archi is not connected.
2016-11-13 15:53:15|INFO|ASF|ParsePostInitArgs() Response received: Bot archi is not connected.
2016-11-13 15:53:30|INFO|archi|OnConnected() Connected to Steam!
2016-11-13 15:53:30|INFO|archi|OnConnected() Logging in...
2016-11-13 15:53:31|INFO|archi|OnLoggedOn() Successfully logged on!
(...)

First of all, I use it on Windows so PowerShell is kinda pain in the ass but I see what you mean.
Second, the first approach you recommended is wrong, it never should be based on a actual delay because if for some reason process start takes more than specific delay it will fail.
Ideally, best way to parse the bot stdout for the status message that WCF is up and bot is ready to receive commands but as I said it is super painful and buggy to do on Windows. That's why I made request because it is easier  and more correct as internal implementation.
I will stick to manual approach because amount of glitchy execution through windows script usually ends up in manual resolution anyway because "something did not work as intended due to bullshit reason".
Thanks anyway

First of all, I use it on Windows so PowerShell is kinda pain in the ass but I see what you mean.

There is nothing in above example that is impossible to do on Windows.

Second, the first approach you recommended is wrong, it never should be based on a actual delay because if for some reason process launch/execution [to the point where command can be executed] takes more than specific delay it will fail.

And that's why you should code your own logic of try/retry, such as starting ASF, and in a loop, waiting for it to start listening on given socket, then attempt to send a command.

Ideally, best way to parse the bot stdout for the status message that WCF is up and bot is ready to receive commands

No, best way to parse the bot stdout is to not parse it at all and instead use excellent and reliable logging that ASF offers.

I will stick to manual approach because amount of glitchy execution through windows script[s] usually ends up in manual resolution anyway due to often "something did not work as intended due to bullshit Windows reasons".

¯_(ツ)_/¯

authors: JustArchi Kein

repo: JustArchi/ArchiSteamFarm


text:
special characters in uri

It fails when uri(path name at aws S3) has special characters like "="(for Hive compatible partition name), thus I think it needs to encode this at  "createStringToSign" method.
I Found this AWS guide
at http://docs.aws.amazon.com/AmazonS3/latest/dev/RESTAuthentication.html
Using Base64 Encoding
HMAC request signatures must be Base64 encoded. Base64 encoding converts the signature into a simple ASCII string that can be attached to the request. Characters that could appear in the signature string like plus (+), forward slash (/), and equals (=) must be encoded if used in a URI. For example, if the authentication code includes a plus (+) sign, encode it as %2B in the request. Encode a forward slash as %2F and equals as %3D.

Thanks, I'll look into it. If you come up with a working solution please make a pull request.

Closing due to inactivity. No other cases are known.


text:
Duplicate A record created each time the ip address changed

The script created an additional A record instead of updating the existing one.

This shouldn't be the case, are you able to provide some screenshots?

Each time ip changes, another A record is created

Each time ip changes, another A record is created with the new ip

This issue has been fixed in PR #20.
While we wait for the creator to review the PR. You guys can try to use the new code and see if the issue still persists.

Hi, thanks for the fix @Sailboat265 !
I'm currently serving national service (military), still in camp and will be unable to verify the code personally.
Do let me know if its tested, works and I'll approve the merge in the coming days. Thanks!


text:
ARC143: レーティングが2000以上のユーザのブログを追加

https://atcoder.jp/contests/arc143/results

該当者なし

repo: KATO-Hiro/AtCoderClans


text:
[C4GT] Button for likes

Description
This should be added to the db
Goals

[ ] Get Detected

Expected Outcome
Detection
Acceptance Criteria

[ ] Detection

Implementation Details
[List any technical details about the proposed implementation, including any specific technologies that will be used.]
Mockups / Wireframes
[Include links to any visual aids, mockups, wireframes, or diagrams that help illustrate what the final product should look like. This is not always necessary, but can be very helpful in many cases.]

Project
Testing
Organization Name:
Samagra
Domain
Something
Tech Skills Needed:
Some, Others
Mentor(s)
@KDwevedi
Complexity
Low
Category
Test, Feature
Sub Category
Database, Breaking Change

HEllo 2000

Hi

Hey guys

Yo


text:
🛑 KMHARS Main Internet (D) is down

In 4fda53a, KMHARS Main Internet (D) (hf60936b8q0.sn.mynetname.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: KMHARS Main Internet (D) is back up in ea2dc0e after 9 minutes.

repo: KMHARS/uptime


text:
🛑 PLDT is down

In 063cefe, PLDT (https://pldthome.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: PLDT is back up in 68ea140 after 11 minutes.


text:
Add BDA-E

spec_version: v1.27
identifier: BDA-E
$kref: '#/ckan/github/BrettRyland/BDArmory-Extended'
release_status: development
resources:
  bugtracker: https://github.com/BrettRyland/BDArmory-Extended/issues
  repository: https://github.com/BrettRyland/BDArmory-Extended
tags:
  - parts
ksp_version_min: '1.9'
depends:
  - name: BDArmory
x_via: Generated by Metadata Webtool

Hi @SuicidalInsanity, I see your name on the releases, so I take it we can consider this permission from the author? Is @BrettRyland still involved?

It's hosted on my github so that it's together with BDArmoryPlus, but I'm not much of a modeller, so that's about the extent of my involvement in BDA-E.

authors: BrettRyland HebaruSan SuicidalInsanity

repo: KSP-CKAN/NetKAN


text:
Add KEAM - Kerbal Expandable Activity Module from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of bcink, to add KEAM - Kerbal Expandable Activity Module to CKAN.
Mod details:
name = /mod/1606/KEAM%20-%20Kerbal%20Expandable%20Activity%20Module
author = bcink
abstract = Re-creation of the Bigelow Expandable Activity Module for Kerbal, KEAM by Kerbalow Aerospace
license = CC-BY-4.0
Homepage = https://forum.kerbalspaceprogram.com/index.php?/topic/167711-wip-keam-kerbal-expdandable-activity-module/
description = Kerbal Expandable Activity Module Release 1.0
**K.E.A.M is 100% dependent upon Textures Unlimited by Shadowmage (Not Included.)
Please download and install Textures Unlimited before attempting to load K.E.A.M.**
Textures Unlimited forum link:
https://forum.kerbalspaceprogram.com/index.php?/topic/167450-131-textures-unlimited-pbr-shader-texture-and-model-api-11-11-17/
Installation:
Extract the contents of the GameData folder to your GameData folder. Structure should then read GameData\KEAM
Optional Connected Living Space Compatibility Installation:
Place the file KEAM-CLS.cfg inside the OptionalConfigs folder into the GameData\KEAM directory
Optional Tweakscale Compatibility Installation:
Place the file KEAM-tweakscale.cfg inside the OptionalConfigs folder into the GameData\KEAM directory
K.E.A.M. is also compatible with Kerbal Inventory System as well as Kerbal Attachment system.
Please visit their respective websites to download those add-ons or acquire via CKAN.
Kerbal Inventory System forum link:
https://forum.kerbalspaceprogram.com/index.php?/topic/149848-131-kerbal-inventory-system-kis-v17/
Kerbal Attachment System forum link:
https://forum.kerbalspaceprogram.com/index.php?/topic/142594-131-kerbal-attachment-system-kas-v063/
A VERY SPECIAL THANK YOU TO:
CobaltWolf, Shadowmage, Nertea and all the other devs who continue to be an amazing part of the mod process
K.E.A.M. is License CC-BY-4.0
https://creativecommons.org/licenses/by/4.0/ embed YouTube videos or screenshots. Be creative.
You can check out the SpaceDock markdown documentation for tips.
Thanks for hosting your mod on SpaceDock!
Please direct questions about this pull request to bcink.

Should be OK after KSP-CKAN/CKAN#2203 gets merged.

#rebuild

authors: HebaruSan Space-Duck


text:
Freeze OpenTree

Deleted from SpaceDock: https://spacedock.info/mod/1623
Confirmed discontinuation in the forum: https://forum.kerbalspaceprogram.com/index.php?/topic/110365-131-opentree-v25-2512018/&do=findComment&comment=3456854

All versions up to (including) 2.2 have their download property already set to Archive.org in CKAN-meta, will do it for the remaining 3 versions too.


text:
Fix the volumes and surface areas of several habitats

For some reason, when a part has its habitat defined in https://github.com/KSP-RO/Kerbalism/blob/7e279c3bccae30eff1ab6612b2b69e78214f4983/GameData/KerbalismConfig/System/Habitat.cfg but NOT in https://github.com/KSP-RO/ROKerbalism/blob/0ae007bf2789abf136ff05367a42e366ecd6dd62/GameData/KerbalismConfig/System/Habitat.cfg, there is a strange bug that massively multiplies that parts volume and surface area by a seemingly random amount.
For example, according to https://github.com/KSP-RO/Kerbalism/blob/7e279c3bccae30eff1ab6612b2b69e78214f4983/GameData/KerbalismConfig/System/Habitat.cfg#L59, landerCabinSmall is supposed to have a volume of 2.1m^3, when it actually has 10.37m^3 ingame.

Another example, according to https://github.com/KSP-RO/Kerbalism/blob/7e279c3bccae30eff1ab6612b2b69e78214f4983/GameData/KerbalismConfig/System/Habitat.cfg#L91. cupola is supposed to have a volume of 4.1m^3, when it actually has 18.77m^3 ingame.

I have 0 idea where these values are coming from, as searching them up in the KSP-RO org gives no results, so I think its either a KSP bug or a kerbalism bug.
There was a substantial difference between the volumes and surfaces areas in Kerbalism vs ROKerbalism, so I had to manually calculate the volume/sa of the cylinders of each of these parts, and then scale them according to how landerCabinSmall scaled. It's a bit arbitrary, but it does give decent results.
This pull adds habitats for the following parts, all of which were defined in Kerbalism but not in ROKerbalism, giving them wacky values:
landerCabinSmall (just sets it to the same as landerCabinMedium, because they have the exact same shape and having them different is confusing for the player)
mk2LanderCabin_v2
crewCabin
cupola
mk2Cockpit_Standard
mk2Cockpit_Inline
mk3CrewCabin
Mark1-2Pod
mk2LanderCabin
Additionally, it also changes the habitat for mk3Cockpit_Shuttle to be a bit more reasonable when combined with the new mk3CrewCabin.

Unrelated, but it would be nice if https://github.com/KSP-RO/Kerbalism had an issues tab, its strange that it doesn't?

repo: KSP-RO/ROKerbalism

license: Unlicense


text:
Add ModuleDepthMask

consolidate the depthmask module from Restock

Usage of ModuleDepthMask by github repo + which distribution is used:

Mod Repo
Distro Used

KSP-RO/ROCapsules
KSP-DepthMask

StoneblUE/OPT-Streamlined
KSP-DepthMask

kingnoob1377/Stock_DepthMask_Config
KSP-DepthMask

JadeOfMaar/RationalResources
Restock

SieraHotel84/RP-1
KSP-DepthMask

Contares/CSA_Contares_CORE
KSP-DepthMask

rogerwang86/SPOCK
KSP-DepthMask

Angels-125/FlyingSaucers
Restock

Angels-125/Mk-33
KSP-DepthMask

Angels-125/Mk-33
Restock

DasSkelett/RealismOverhaul
Restock

post-kerbin-mining-corporation/NearFutureLaunchVehicles
Restock

KSP-Realistic-Revamp/RP-0
KSP-DepthMask

Rodg88/BumbleBee
Restock

zer0Kerbal/Pteron
Restock

CobaltWolf/Bluedog-Design-Burea
BDB

TundraMods/Tundra-Exploration
Tundra

PorktoberRevolution/Restocked
Restock

JadeOfMaar/KARE
KSP-DepthMask

--------
--------

Official Permission:

Regex to find all this:
/name\s*=\s*(?:[Mm]odule.[Dd]epth[Mm]ask|[Dd]epth.[Mm]ask.[Mm]odule)/ (path:.cs OR path:*.cfg)

BDB and tundra will be very easy to get rid of as they're only used in their own repo,
Restock will be a tad bit more difficult, it might be worth looking into if simply including KSP-DepthMask as a submodule of sorts might be better than requiring all the existing implementations to change over.
that way there'd still be one consolidated source.

repo: KSPModdingLibs/KSPCommunityPartModules


text:
Missing disconnect method on generated SignalR Services

Hey Kai, I wonder if I missed the disconnect method on the generated SignalR Services, or maybe I understand anything wrong?

Will be part of the next release

authors: joachim-heck ky-one

repo: KY-Programming/generator


text:
Full Support of Braket Notation

Is your feature request related to a problem? Please describe.
Not a problem.
Describe the solution you'd like:
Is there any plan to provide full support of the brakes notation? Currently \Bra, \bra, \Ket, \ket, \braket are supported, but \Braket is not; this one handles expressions like <a|b|c> and can adjust its size to the contents.
Link to or name of a (La)TeX package that provides the same feature:
The Braket package

This seems like a reasonable request. I'll write a PR. Some time may go by before it gets reviewed.

authors: Yue-Zhengyuan ronkok

repo: KaTeX/KaTeX


text:
better doc for globalGroup

Fixes #3547.

Thanks for writing this!  I made a few edits in https://github.com/KaTeX/KaTeX/pull/3567/commits/be3ea065066fcc8a3652fe0444877e3f684e19d8. Could you take a look and see if you like them?

lgtm

authors: edemaine nschloe


text:
Character metrics missing for ¶ (182) and § (167)

See https://github.com/Khan/KaTeX/pull/1214#issuecomment-373041152

Closed via #34.

authors: edemaine ylemkimon

repo: KaTeX/katex-fonts


text:
Button component semantischer maken en correct linken

Hey Yu Jing, tijdens het maken van de desktopCards kwam ik erachter dat jouw button component niet semantisch was en ook dus niet ging linken naar waar het moest linken je had namelijk dit als code:

Had je niet liever een  kunnen gebruiken en dit vervolgens gestyled naar wat het button zou moeten zijn?
Hierdoor had je het namelijk semantisch gemaakt (aangezien de button moet verwijzen naar een ander pagina) en ook nog is goed gelinkt naar het volgende pagina ipv dat we de
 een  attribuut meegeven.
Kijk naar deze bronnen voor meer informatie:

FreeCodeCamp - Button Link Code Examples
W3C - HTML Hyperlink constraints and admonitions

@KaanKalmi bedankt voor de feedback ik heb er naar gekeken dit is nu de code kan ik een pull request indienen?
person.id

Dit ziet er idd een stuk beter uit, je kan een pull request indienen ja (let op: Je moet wel de juiste repository kiezen en niet FDND-Task)

Ik  merge je pull request niet, ik ga WEL je codde copy/pasten naar mijn branch

authors: KaanKalmi yujing-student


text:
Add a disable check updates option

I think that an option to disable update checking should be added, this is useful for mods that use old versions of kade engine or mods that use a pre-compiled version of kade engine.

I think that an option to disable update checking should be added, this is useful for mods that use old versions of kade engine or mods that use a pre-compiled version of kade engine.

they just delete the outdatedsubstate file lmao

or remove the singular line of code that switches the state lmao

repo: KadeDev/Kade-Engine


text:
Update CONTRIBUTING.md

AAAHAHAHA LMAO MY DUMBASS FUCKING EDITED THE KADE FI
LE

@crislikesanimation close this pr

ok

+ 1
- 1

+
-

dude this was suck a great change!
i don't know why it wasn't accepted by kade this was just, one of the best i've seen!

authors: KadeDev crislikesanimation napstaa967 prokube somefnfplayerlol


text:
add support for installation behind SSL proxy

If beeing behind a SSL proxy (nginx in my case), wordpress tried to load assets using http instead of https. These lines change the protocol to https if behind a proxy.

Thank you for your pull request!
I'll check the changes and give you feedback!

Since this image is meant for testing and development, I would prefer to use a simple base WordPress configuration with minimal side effects.
If you need some custom configurations you could either use this image as the base image and override the wp-config.php file with your own configuration:
FROM kaihofstetter/wordpress-cli:4.4.1
MAINTAINER ...

# WordPress configuration
ADD wp-config.php /var/www/html/wp-config.php
...

Or you could simply copy your own wp-config.php file in the container:
docker cp wp-config.php my-wordpress:/var/www/html/wp-config.php

Thank you for your pull request!

authors: KaiHofstetter elangenhan

repo: KaiHofstetter/docker-wordpress-cli


text:
[Sage-8] Tokens - Font Family Update to Circular

Description

Updates the Sage font family from Inter to Circular.
Note: Linting is failing because the fonts aren't being committed.
Screenshots

Before
After

Testing in sage-lib

View Sage Docs
Check that new font, Circular, is properly in use.

Testing in kajabi-products

(LOW) Updates the Sage font family from Inter to Circular. No effect on Kajabi Products.
Related

Sage-8

Regarding the failing lint we have options:

Merge into our epic branch and know we'll fix it shortly
Hold this PR for a bit as we work out the last bits of the CDN-based approach to the font paths.

Regarding the failing lint we have options:

Merge into our epic branch and know we'll fix it shortly
Hold this PR for a bit as we work out the last bits of the CDN-based approach to the font paths.

I can hold on to this PR and update when you're ready.

@anechol we're set up and ready with all the fonts under https://sage.kajabi-cdn.com/fonts/circular

I'm doing a separate PR on develop to handle this for the existing fonts FWIW.

For reference as needed https://github.com/Kajabi/sage-lib/pull/1183

authors: anechol philschanely

repo: Kajabi/sage-lib


text:
add --useType flag

I had a compiler error come up when trying to pass an object of the generated interface type to a function expecting a Record<string, string | number | Date>. Apparently this isn't supported with interface unless you specifically cast the key as a string, but does work with type.
I added a simple flag to enable generating with type instead of the default interface if it's relevant for anyone else.

Yeah, type appears to be a little more flexible with type unions and intersections, etc. I implemented useType as a flag as to not make this a breaking change, but if you want to change the default, that works for me.

I also just pushed another commit that explicitly allows null for fields that are not required when using the --generateInsertionTypes option. For fields that are required but have a default value, null isn't a valid value, so I added some logic to fix that.

That's true, let's keep this without breaking changes for now. Thanks for this work!

authors: Kalissaac kalcicd

repo: Kalissaac/prisma-typegen


text:
some items kick player from server

a few of the food items like fries kick the player from a server for some reason?

Yeah this is very much possible as I haven't tested the mod in a server environment. I haven't really been working on this mdo recently but I might try to still fix this

authors: KaloyanKKS imbasicallyalex

repo: KaloyanKKS/FoodModPlus


text:
Added Release pipeline action

Fixes #746

Awesome, thanks @oscerd.
I have a couple of questions that I would like to ask:

The current order is tag-and-release, npm-release, container-image-release, I wonder if it would make sense to start with the npm-release first, this way, if we try to release a broken commit by accident, this build step will fail, hence preventing the container and the release to be generated.

Yes, we can simply switch

We build the project a few times in different workflows, if we wanted to avoid redefining the build process in all of them, would be a custom action the way to go?

We can define a build action and we could re-use it from other workflow by having that defined as custom action

repo: KaotoIO/kaoto-next


text:
Including an app

Hello, I want to add an app that provides affirmations regularly, thus motivating users. Can you please assign me this issue?

I have assigned Sneha, you can add it now , Thanks.

I have added the app. Please check. Thanks.


text:
added a refresh button and did some visual refactoring

kabir accept quick

discussed on call on discord. Great work

authors: FrederikBrecht Karl-Michaud


text:
Add rotate screen support. #14 and #15

To be able to implement the feature requested at #14 and the example requested at #15 I've implemented the screen rotation support. This could be implemented out of the library but it was quite straightforward to implemented it inside Dexter, so I decided to add this feature to the project instead of add it to the example project.
To be able to support this feature the only thing our users has to do is to call a method in the onCreate   Activity method where the library is used. I've updated the README.md with some documentation.
Please @greysonp check if this feature is enough or if you need something more specific.

:+1:

With the help of @Serchinastico we've added some changes to avoid memory issues with the transparent activity recreated when the device was rotated. Thanks for everyone involved. We are going to wait for @greysonp before to merge this PR. Maybe he has something to add.

:+1:

Hey I probably won't be available until Sunday-ish because of the holiday. I'll check it out when I get a chance!

@Serchinastico shouldn't we assign the listener to null after the request permission process?

Done :) I've replaced the listener with an empty implementation.

:+1:

I don't know the finer details of the system, but the end interface provided to the user looks good. My only bit of confusion lies with the effect of Dexter.checkPendingPermissions(). It looks like the listener passed in will always be assigned, even if there's no pending request in progress. That isn't implied by the docs nor the name of the method. Instead, the name just make you think this is some sort of check that will return a boolean.
If the goal is to both check if a request is in progress and assign the listener, then the name and docs should reflect that.
Suggestions:

Remove the listener if there's no pending permission requests and rename the method to continuePendingRequestsIfPossible() or something to that effect
Keep the assignment and rename the method assignListenerAndContinuePendingRequests() or something to that effect

Hi @greysonp
The proposal to replace checkWhatever with continueWhatever makes sense to me, but the assignListener part looks like exposing an implementation detail that shouldn't be explicitly declared in the public API. I'm ok including it in the docs though.

FWIW people need to know if they're about to hand something off to a library that's going to be held statically in memory (like this listener). Lots of potential for memory leaks here. That's the only reason I wanted to make it clear that this listener is going to be used no matter what.
It's probably fine here though since the act of giving a listener to a static method should be enough to tip someone off, but it's definitely something people need to be aware of in general with a library like this.

Hi @greysonp , I have created a PR to fix the issue you pointed out in continuePendingRequestsIfPossible [https://github.com/Karumi/Dexter/pull/28] I'm sorry for the misunderstanding.
Regarding the static memory, I'd say that the fact Dexter is being called using static methods can give you the idea that we are holding static references (how else would it be possible to hold state between calls?). Now that the listener reference is cleaned up once the request permission process finishes (and doesn't keep it if it's not needed) I don't see the point of declaring that we are going to hold a reference to it.

authors: Serchinastico flipper83 greysonp pedrovgs

repo: Karumi/Dexter


text:
Help needed?

Hey man!
I and my team would love to help with this project.
Please let me know when you have some time to chat.
What timezone are you in? We are in CET +1.

GMT :)
Some help would be most appreciated. I've got the core building blocks in place:

Test runner
CLI
AST parsing
Basic ability to modify nodes in the AST

I'm having issues getting decent Flow definition files to test against. The Flow definitions can look quite different and assert more things, but that means I can't use them when asserting generator output. For example, see the this vs that - I can't possibly automate setting the kind to the values listed in the Flow definition.
It would be most helpful if there were "realistic" versions of the flow definitions which a machine could reasonably produce. It's hard enough getting the generator working, let alone hand-crafting flow definitions to test against: so any PRs with the .d.ts and corresponding hand-crafted .js.flow which is confirmed working would be wonderful.

Hi.
I used this package to generate flow lib definitions from a d.ts file. When i run 'flow check', i get these errors from the new flow libdef files:
19:   export interface InterfaceDefinition extends Object{
^^^^^^ Library parse error:
19:   export interface InterfaceDefinition extends Object{
^^^^^^ Unexpected token export
The page here-https://flowtype.org/docs/third-party.html#_ mentions that we need to have a
declare module "name.." {
declare module.exports: {
someMethod(params): returnType;
}
}
But the libdefs generated with this package look different. I am not sure how to make flow accept the generated lib defs ?
I put the generated libdefs under a /customlibs folder (in the project root) and gave
[libs]
/customlibs/
in the .flowconfig

This project in its current form barely works for simple .d.ts files. It's a huge WIP at the moment.

I'm deprecating this project so the community can focus on a single converter project and not multiple. https://github.com/joarwilk/typedef-converter is currently more comprehensive.

authors: Kegsay lgandecki suprsankr

repo: Kegsay/ProbablyFlowTyped


text:
Report Correction

Overview
Report.measure_model function removed in favor of using single function report.compare (neé compare_models) for all reports. References updated accordingly.
Note: jupyter notebooks do not need to be checked at this time.
How was this update tested?
Pytest
Checklist

[x] I have read the Contributing guidelines
[ ] I performed a self-review of my own code
[x] The CHANGELOG and version.json have been updated
[x] Documentation has been updated to reflect the changes

[x] The code has clear and helpful comments, particularly in hard-to-understand areas
[x] READMEs, Tutorials/Examples, and links have been updated as appropriate

[x] Tests have been added to prove that the fix is effective or that the feature works
[x] Pytest was run; new and existing unit tests pass locally with my changes
[x] I have made note of any breakages or dependencies (if applicable)
[x] Any dependent changes have been merged and published in downstream modules
[x] These changes generate no new warnings

Dropping this PR in favor of newer PR (#66)

repo: KenSciResearch/fairMLHealth


text:
Build system

spiked code, with initial build system.
still needs encrypted travis env vars

Changes Unknown when pulling d51c2715bb88d46bc6a323fd9688a55d360f09e4 on KyleChamberlin:build-system into ** on KentCommunications:master**.

Changes Unknown when pulling d0cc5542b5dd61f51172000ab7bd50a1b6398e6a on KyleChamberlin:build-system into ** on KentCommunications:master**.

authors: KyleChamberlin coveralls

repo: KentCommunications/paf-filler


text:
Include total count parameter

add totalCount as property in Pagination
adapt test jackson bindings
adapt readme

Motivation
Fixes #92
Checklist

[x] Code follows coding conventions held in this repo
[x] Automated tests have been added
[x] Tests are passing
[x] Docs have been updated (if applicable)
[x] Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test
Try the JacksonBindingTest and see if the totalCount property is available

Hello @SmootherSpike!
Thanks for your pull request. I am going to go through the changes.
I have one question according to t the warning information described in # 92:

Also, please provide a warning in documentation comments that asking for the total number of content items might increase the response time and, therefore, users should use it only when they need it.

Do you think the information in Javadoc comment for includeTotalCount method Is sufficient?

@Simply007 additional text is in the docs now

Great @SmootherSpike!
As a part of digging in the SDK during the review, I started comparing kontent-java-rx-sdk and this package and there are not many differences in my opinion.
Would you be willing to jump on a call and discuss a couple of ideas I have to improve the java stack?
If you are open I am available on gitter and we could continue there.

Fix released in 3.2.0-SNAPSHOT

version 3.2.0 is released!
https://github.com/Kentico/kontent-delivery-sdk-java/releases/tag/3.2.0

repo: Kentico/kontent-delivery-sdk-java


text:
Unable to find energy_now file on Linux

Hi, I've tried using this module on my Linux system with kernel version 4.1.6 and it hits a FileNotFoundError because the energy_now file doesn't exist for my battery.  I do see a file in there called charge_now though.  Do you know if this is due to a renaming in more recent kernel versions?  If so, would you be interested in a pull-request that fixes this?  Thanks!

Looks like it is.
Fortunately it is just rename, content seems to be the same.
Best Regards
Ilya Kulakov

On 26 сент. 2015 г., at 14:38, Justin Buchanan notifications@github.com wrote:
Hi, I've tried using this module on my Linux system with kernel version 4.1.6 and it hits a FileNotFoundError because the energy_now file doesn't exist for my battery. I do see a file in there called charge_now though. Do you know if this is due to a renaming in more recent kernel versions? If so, would you be interested in a pull-request that fixes this? Thanks!
—
Reply to this email directly or view it on GitHub.

Ok cool.  Do you plan on fixing this sometime and/or would you like me to submit a pull request?

First of all I need to fix the behavior: it should fallback to default implementation in cases like that.
Best Regards
Ilya Kulakov

On 26 сент. 2015 г., at 17:56, Justin Buchanan notifications@github.com wrote:
Ok cool. Do you plan on fixing this sometime and/or would you like me to submit a pull request?
—
Reply to this email directly or view it on GitHub.

2 years later still not fixed right?

No, but thank you for bringing my attention.

Should be fixed in the develop branch.

authors: Kentzo justbuchanan kootenpv

repo: Kentzo/Power


text:
Add cursor animation options

The cursor animation is very eye-catching when switching from normal/insert mode to the bottom command line.
As stated in this issue: https://github.com/Kethku/neovide/issues/85 , being able to disable animation would be a great feature, but it might be even cooler to have a less flashy effect for this transition, like fade in or something.
What do you think about that?

Sounds good to me. I am definitely planning on multiple configuration options for the cursor. Its on the list once we figure out how to implement settings correctly. Its coming!

This is partially solved now by @jonvaldes solution. The framework is set, just need to add a way to disable the trail in various circumstances

This has been solved for a while now. We dont necessarily have a fade animation but the animation is configurable in a variety of ways

authors: Kethku kiuKisas

repo: Kethku/neovide


text:
closeOnClickOutside doesn't work when clicking sides

I attempted to migrate from angularx-bootstrap-modal today, but ran into an issue where I could not dismiss the modal by clicking to the left or right of it - only above and below. I know for certain this issue is with the new package because the only thing I swapped was the module, service, and component names.
On a side note, it would be nice if the backdropColor parameter was brought back - it was pretty convenient.

Hey man, I had plans to deprecate this fork. I moved active development to
ngx-simple-modal. Sounds to me though you might not like our decisions, one
being to separate the concerns of JS and CSS. Another being to normalize
the naming convention between modal and dialog. Finally the original
architecture doesn't actually require you to use bootstrap, that's an
implementation decision, so you can use other frameworks if you choose.
I remember fixing your issue in that implementation.
Re CSS background, it's much better to use CASE for this , giving you more
control.
K
On 17 Dec 2017 07:36, "Arty" notifications@github.com wrote:
I attempted to migrate from angularx-bootstrap-modal today, but ran into an
issue where I could not dismiss the modal by clicking to the left or right
of it - only above and below. I know for certain this issue is with the new
package because the only thing I swapped was the module, service, and
component names.
On a side note, it would be nice if the backdropColor parameter was brought
back - it was pretty convenient.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KevCJones/ngx-simple-modal/issues/14, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAqe8IXHoctzerj_5iQU8j5Q994dtxVHks5tBMSMgaJpZM4REkyZ
.

Deleted last comment - its early, i thought the issue came from the old repo :)

Re: CSS background colour, is there a reason you can't or don't want to use a css style to manage the backdrop colour? Just so we understand.

So i can see the cause 👍 the fix is a little clunky though.
this function i updated did away with the manual checking for the modal classes of bootstrap and made an faulty assumption that the first child of the modal being opened was the content...
onClickOutsideModalContent( callback: () => void) {
    this.clickOutsideCallback = callback;
    const containerEl = this.wrapper.nativeElement;
    containerEl.querySelector(':first-child').addEventListener('click', this.stopEventPropagation);
    containerEl.addEventListener('click', this.clickOutsideCallback, false);
  }

However clearly in the case of things, i missed the fact that the first child (even in my demo) is actually the .modal-dialog a container which is used to center the actual content... i could i suppose add a class to the configuration 'contentClass' which you can set based on implementation to your 'content class' of choice, in yours and other Bootstrap user cases .modal-content...

@artemzakharov take at #15 's PR and tell me if this would see ok for you? I'm trying to NOT bring in bootstrap's classNames to the selectors etc. Tested on the demo internally and it worked fine

@KevCJones That would work for me. It's a wee bit less maintainable but directly passing in the class is pretty powerful and opens up new possibilities, so an improvement on the whole imo 👍
Re: CSS, it's true that it's possible to style the modals giving them unique classes/ids, but it's a little messy in my particular case since the modal pops up well outside of the Angular component that calls it, so the styling would have to go in a global style file that's in a completely different part of the project.
On second thought, I haven't previously considered that I'll need to style the rest of the modal anyway, so even if one option was an argument, the rest of the styling would still need to be done in the same place, so making it an option is kind of moot. It could still be useful if you'd like to make the modals a little easier to look reasonably good out of the box, but I've come to realize it's far from requirement.

Yeah this was a similar line of thinking we had when we decided to remove the option. If i need to make my modals look right for 'said' project, why seperate the background color alone as a configurative component. The only time we saw this being useful was if the implementer was using the out-of-the-box styles bootstrap or other components used by default... which tbh is more the exception than the rule.
I'll let @celsomtrindade review but all in all should merge and bump the version later on today.

@artemzakharov  one observation I have on this topic.
I was doing some tests with the project and I tried to focus on this issue specific. I was able to close the modal box by clicking anywhere on the page (tested on chrome, firefox and edge - all on windows).
I said this only happens when you try to click on the sides, isn't it possible that you have something in bettwen, or any other css related issue that affects the modal container (or the container it's within)?
I'm using the application on the demo folder of this project and everything is working fine. I'm not saying the fix shouldn't be merged (maybe it's a improve to the code). But maybe there is something else in your code that is affecting this.
If, for example, I remove the position fixed from the .modal css it will affect the close outside.

I was able to reproduce in our demo weirdly enough. The implementation of
alert, with bootstrap. The first child was .modal-dialog and was full width
and it child .modal-content was margin auto to center it. The issue was
that first child was our click blocker because of the recent update i put
in to remove bootstrap class name coupling from the component.
I suppose you 'could' change bootstrap's css, but, i think we owe our
implementers some amount of friction-free implementing.. I don't doubt
we'll want to relook at this later, its not my ideal situation to be in. It
might be something we want to configure once during the modal
configuration? Do once, decided by your platform?
On 18 December 2017 at 11:15, Celso Trindade notifications@github.com
wrote:

@artemzakharov https://github.com/artemzakharov one observation I have
on this topic.
I was doing some tests with the project and I tried to focus on this issue
specific. I was able to close the modal box by clicking anywhere on the
page (tested on chrome, firefox and edge - all on windows).
I said this only happens when you try to click on the sides, isn't it
possible that you have something in bettwen, or any other css related issue
that affects the modal container (or the container it's within)?
I'm using the application on the demo folder of this project and
everything is working fine. I'm not saying the fix shouldn't be merged
(maybe it's a improve to the code). But maybe there is something else in
your code that is affecting this.
If, for example, I remove the position fixed from the .modal css it will
affect the close outside.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/KevCJones/ngx-simple-modal/issues/14#issuecomment-352397306,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqe8H0u5L2XNTtibnpAXBekhhsc9L00ks5tBkk4gaJpZM4REkyZ
.

--
Kevin C Jones

authors: KevCJones artemzakharov celsomtrindade

repo: KevCJones/ngx-simple-modal


text:
SMB challenge/response never intercepted

Hello, and thank you for this great tool.
I'm facing a problem when trying to MITM the LLMNR requests through Inveigh. I use it with default settings, but what it happens at testing:
[+] [2018-10-29T22:21:53] LLMNR request for ppppppp received from 192.168.0.10 [Response Sent]
[+] [2018-10-29T22:21:53] LLMNR request for ppppppp received from 192.168.0.10 [Response Sent]
............
And I never receive the SMB challenge. Is that problem has something to do with firewall? I use Inveigh in elevated privileges.

Hi, it could be the firewall. Are you able to do a packet capture on the system that's running Inveigh? That will give you a better view of what's actually hitting the box (smb,ntlmssp).
I did add an indicator of incoming SMB connections for Inveigh 1.4. You should see this for any incoming SMB connection:
[+] [2018-10-31T00:00:37] SMB(445) negotiation request detected from 10.10.2.106:54378
This is just an indicator of an incoming SMB connection and doesn't necessarily mean the authentication will be with NTLM. You should at least see that though before getting a challenge/response through SMB.

Hello, unfortunaly I can't do packet capture on the system running Inveigh.
But I can ensure that I never saw the indicator of incoming SMB connections.
I think nothing actually hitting the box, but I could give you a list of all rules for incoming connections?
Thank you

445 is all that is needed for SMB. 80 for wpad and random http. It looks like you are getting LLMNR traffic through the firewall so I think you're good with UDP 5355.
If you are able to spoof LLMNR, you might be able to just send the traffic to another box that you control on another subnet. The SpooferIP parameter lets you set where the traffic goes. You might have to deal with ACLs between the subnets though.

authors: Kevin-Robertson jammys

repo: Kevin-Robertson/Inveigh


text:
add zh locale

zh-Hant: Traditional Chinese
zh-Hans: Simplified Chinese

There a some imprecise words in simplified Chinese locale, are you guys can accept other contributions on it?

There a some imprecise words in simplified Chinese locale, are you guys can accept other contributions on it?

I don't mind. Anyway, zh-Hant has been changed by other users already.
Some Chinese words can have different meanings in different places and scenarios,
like 行 and 列, in some places, it means row while in some other places and it means column.
In my option, horizontal and vertical writing will also change the meanings of 行.
If the user focuses on the record, the user may call the row 行.
If the user focuses on the column difference between different records, the user may also call the column 行.
Although hk, mo, and tw use zh-Hant, and cn, my and sg use zh-Hans,
the wordings between those places are different so you may feel the wordings are imprecise.
You can also use Custom Non-Built-In Translations to override the default translation, I believe, but you can create your pull request if the words in zh-Hans are really imprecise.

There a some imprecise words in simplified Chinese locale, are you guys can accept other contributions on it?

I don't know any of the languages myself. I just run the PRs through Google translate real quick for review, but I mostly merge anything to do with locales. Happy to fix any inaccurate translations

Yes, using 行 instead of 横列  is a good case which I want to point out, and there are also some other cases that I think it should be modified.
All the meanings of the words are correct, but some of them do not conform the code of simplified Chinese.

I don't know the differences codes of simplified Chinese between cn, my and sg.
I will post a changing map before sending a pr, and you can help me to review them.

@LvChengbin Actually 橫列 is from Apple:
https://support.apple.com/zh-hk/guide/numbers/tan0403655e1/mac
https://support.apple.com/zh-tw/guide/numbers/tan0403655e1/mac
https://support.apple.com/zh-mo/guide/numbers/tan0403655e1/mac

Microsoft uses 列 to represent row for zhhk and zhtw, and 行 to represent row for zhcn and jajp
https://support.microsoft.com/zh-HK/office/row-函數-3a63b74a-c4d0-4093-b49a-e76eb49a6d8d
https://support.microsoft.com/zh-TW/office/row-函數-3a63b74a-c4d0-4093-b49a-e76eb49a6d8d
https://support.microsoft.com/zh-cn/office/row-函数-3a63b74a-c4d0-4093-b49a-e76eb49a6d8d
https://support.microsoft.com/ja-jp/office/row-関数-3a63b74a-c4d0-4093-b49a-e76eb49a6d8d

Annoying, right?
I believe that the best way is to use 橫列 for zh-Hant and 橫行 for zh-Hans.

@bluelovers @LvChengbin
Please check the translations in my fork.
https://github.com/KevinVandy/material-react-table/compare/main...samchiu90:material-react-table:2023_03_02_ZH

@bluelovers @LvChengbin
Please check the translations in my fork.
https://github.com/KevinVandy/material-react-table/compare/main...samchiu90:material-react-table:2023_03_02_ZH

In case it wasn't clear, I'd love any corrections to be prd to MRT

https://github.com/KevinVandy/material-react-table/pull/409
@LvChengbin @bluelovers

authors: KevinVandy LvChengbin samchiu90

repo: KevinVandy/material-react-table


text:
Adding Keyboard Background image

Hi there! could you please add the feature that allows adding background image to keyboard. I want to add image instead of gray color. thank you.

Oh, another thing. If you just want to add an image, you can just setup your keyboard with a ZStack and place the image you want below the keyboard view.

authors: danielsaidi serhatyaroglu

repo: KeyboardKit/KeyboardKit


text:
Update scalafmt-core to 3.7.5

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

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

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

Superseded by #48.


text:
[🔥AUDIT🔥] Remove security issue template

🖍 This is an audit! 🖍
Summary:
I figured we should have this to say "STOP! Use HackerOne instead".
But GitHub anticipated that and already links to the security policy
(docs/SECURITY.md) as an option when creating an issue, so
it's just duplicative.  In this commit I remove it.
Test plan:
above link should show just one option for security

Note checks failed on this commit but are fixed by #112 which I just forgot to merge in this branch (but is in main).

repo: Khan/genqlient


text:
MVKPipeline: Don't skip setting the vertex stage pipeline.

In an indirect draw, we wouldn't set the pipeline for the vertex stage
to the command encoder, because we had already switched encoders. This
caused havoc down the line, as we wound up running the "convert indirect
buffers" pipeline again, which smashed memory randomly--possibly causing
the GPU to crash, and bringing down the WindowServer with it!
Why is it so easy to bring down the WindowServer like this?

This one really needs to be in the SDK. It fixes a problem that could cause a WindowServer crash.

repo: KhronosGroup/MoltenVK


text:
Better handling of static const function variables in spv*TableGet()

We have a bunch of static const function variables to represent various bits of the SPIR-V grammar. It contains non-POD fields, which causes problems with custom memory allocators: https://github.com/Microsoft/DirectXShaderCompiler/issues/725. Need to figure out a better way of handling them, e.g., using non-const global pointer, changing everything to POD, etc.

Done via #920

repo: KhronosGroup/SPIRV-Tools


text:
Derivative instructions are not combinators

Remove the derivative instructions from the optimizer's list of combinators.

Blocked until #971 is merged

In a derivative instruction, an invocation exchanges information with its peers in the quad being executed.  So it has side effects beyond calculating a value.

Cc: @greg-lunarg  :  This is a functional change to the aggressive dead code elim, but I think a necessary fix.


text:
Project compiling with architecture changes

[+] Added platform agnostic math classes
[-] Removed caching from serialization layer
[+] Added caching to unity layer via assetcache object
[+] Added tests around gltf serialization
[+] Added ability to have assets "streamed" from disk instead of loaded via uri
Next PR will have changes that pass around streams instead of byte arrays.

Overall it looks good, all the examples work.
I had some errors when building GLTFSerialization though:
Microsoft.NETCore.Portable.Compatibility 1.0.2 provides a compile-time reference assembly for mscorlib on UAP,Version=v10.0, but there is no run-time assembly compatible with win10.

and
Some packages are not compatible with UAP,Version=v10.0.

I found that commenting out the win10 build target in GLTFSerializationUWPTests/project.json#L11 resolved the issue.
Also, GLTFJsonSerialization.dll.mdb should no longer be needed.

authors: MSblgross robertlong

repo: KhronosGroup/UnityGLTF


text:
Duplicate declaration of update_overlay in api sample base clase

The api sample base class (based on my samples) always had a function called update_overlay in the protected part of the class:
https://github.com/KhronosGroup/Vulkan-Samples/blob/0d0cf9b90541d349ceb49682d063d27a8c2432de/framework/api_vulkan_sample.h#L357
At some point some PR seems to have introduced a virtual function with the same name in the public part:
https://github.com/KhronosGroup/Vulkan-Samples/blob/0d0cf9b90541d349ceb49682d063d27a8c2432de/framework/api_vulkan_sample.h#L115
MSVC is picking this up:

It looks like the virtual function was supposed to replace the old one, but the old declaration hasn't been removed.

Looking at that base class it also seems like a lot of recent changes were made without documentation and that there now are two public section. I'll probably do a clean up pass over the whole base class.

repo: KhronosGroup/Vulkan-Samples


text:
Fix folder creation on Android

Description
root doesn't has a trailing slash, so on Android a folder named filesoutput/ was created instead of files/output/.
This PR adds the missing directory separator when creating folders.
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[ ] I have commented any added functions (in line with Doxygen)
[ ] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[ ] I have used existing framework/helper functions where possible
[x] My changes do not add any regressions
[ ] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

If this PR contains framework changes:

[ ] I did a full batch run using the batch command line argument to make sure all samples still work properly

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Under what circumstances do you get root without a trailing slash?
Maybe this function should be able handle both cases, root with and without a trailing slash.

Hi @tomadamatkinson, can you take a quick look here?  Looks reasonable, but wanted to double check with you before merging.  Thanks

Hey @jloehr, this approach likely works but I was a little unsure if this would affect other platforms or not

Maybe we can replace it with std::filesystem::create_directories altogether. Though haven't tested that. We ran with the simple added slash.

I think this is a great suggestion! It also pointed out some bugs that we could have had with vkb::fs in the long run. I also didn't know how the standard library would behave so i added some tests in here too. Please take a look at #1214 as an alternative approach

Merging w/ Tom's approval - per discussion on Monday's call - LMK if this causes issues and we can revert

authors: CLAassistant asuessenbach jloehr marty-johnson59 tomadamatkinson


text:
Fix incorrect src access mask in GLTF image upload

Description
Fix #38
General Checklist:
Please ensure the following points are checked:

[X] My code follows the coding style
[X] I have reviewed file licenses
[X] I have commented any added functions (in line with Doxygen)
[X] I have commented any code that could be hard to understand
[X] My changes do not add any new compiler warnings
[X] My changes do not add any new validation layer errors or warnings
[X] I have used existing framework/helper functions where possible
[X] My changes build and run on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

Adding this flag causes validation errors on an S8 Adreno.
2019-05-27 12:06:40.851 12234-13129/com.khronos.vulkan_samples E/VulkanSamples: [error] [s available, enabling it:33] Validation:  [ VUID-vkCmdPipelineBarrier-pMemoryBarriers-01184 ] Object: 0x7bb51fe010 (Type = 6) | vkCmdPipelineBarrier(): pImageMemBarriers[0].srcAccessMask (0x1000) is not supported by srcStageMask (0x4000). The spec valid usage text states 'Each element of pMemoryBarriers, pBufferMemoryBarriers and pImageMemoryBarriers must not have any access flag included in its srcAccessMask member if that bit is not supported by any of the pipeline stages in srcStageMask, as specified in the table of supported access types.' (https://www.khronos.org/registry/vulkan/specs/1.0/html/vkspec.html#VUID-vkCmdPipelineBarrier-pMemoryBarriers-01184)
2019-05-27 12:06:40.882 12234-13129/com.khronos.vulkan_samples E/VulkanSamples: [error] [s available, enabling it:33] Validation:  [ VUID-vkCmdPipelineBarrier-pMemoryBarriers-01184 ] Object: 0x7bb51fe010 (Type = 6) | vkCmdPipelineBarrier(): pImageMemBarriers[0].srcAccessMask (0x1000) is not supported by srcStageMask (0x4000). The spec valid usage text states 'Each element of pMemoryBarriers, pBufferMemoryBarriers and pImageMemoryBarriers must not have any access flag included in its srcAccessMask member if that bit is not supported by any of the pipeline stages in srcStageMask, as specified in the table of supported access types.' (https://www.khronos.org/registry/vulkan/specs/1.0/html/vkspec.html#VUID-vkCmdPipelineBarrier-pMemoryBarriers-01184)

That's confusing, because there's no place in the code where I see a combination of VK_ACCESS_TRANSFER_WRITE_BIT and VK_PIPELINE_STAGE_HOST_BIT in the same barrier.  Also I get no validation error on my desktop, which suggests to me that the adreno validation failure is a false positive from an older version of the validation layers.

I can't find any previous bug in the validation layers that would explain this behavior, but I also can't explain the combination of srcAccessMask 0x1000 VK_ACCESS_TRANSFER_WRITE_BIT and srcStageMask 0x4000 VK_PIPELINE_STAGE_HOST_BIT.  I'm attempting to do an android build now but I only have the following devices to test:  Pixel, Pixel 3a, S9 and S6
Can you provide more detail as to what example was being run when you encountered this warning?  I know the performance examples that load the Sponza scene exercise this code path, but I'm not sure if anything else does.

I've tested on my Pixel 3a, and run the render_passes performance example, which I know exercises the changed code.  The logcat of initialization and starting to render is located here.  You can see from the logging that validation is enabled (I had to manually modify the cmake arguments in the gradle file to trigger this).
I can't reproduce the validation message.
I'll try testing on the S6 as well (once it gains sufficient charge), but the behavior suggests a bug in either the validation layers or the driver on the S8 Adreno.  The validation layers are not device specific, so given the same client code I should generate the same errors you're getting.

That's weird, I can't seem to reproduce these validation messages anymore, everything looks clean to me now.

authors: ZandroFargnoli jherico


text:
第二十一题：Vue 有了数据劫持为什么还要 DOM diff？

Vue 有了数据劫持为什么还要 DOM diff？这不是多此一举么。
完成以上内容后，请说明如果存在爆栈的情况该如何解决。
去答题
新建了一个大厂真题每日打卡群，有意愿学习打卡的再进，请备注打卡。

Marked

这个应该要问vue作者，我猜是因为一个组件好像只有一个一个watcher，不diff怎么知道具体变化呢？没读过vue源码，读过源码得人应该知道吧

vue 的响应式系统通过依赖收集可以直接知道在哪里变化。但是绑定需要watcher,如果粒度很细就需要很多watcher很大的开销。所以vue采用的是中等粒度，在组件级别监听。然后系统可以知道哪个组件变化，对该组件进行dom diff。

个人感觉 数据劫持也只是知道被劫持数据发生了变化，需要去update  试图， 但是具体说要更新到哪个DOM  ，DOM结构会发生怎么样的变化，还是需要来做一个新旧的diff

现代框架主要有两种侦测数据变化方式

pull：React 通过 setState 显示更新，然后 VDOM 层层 Diff，查找变化
push：Vue 数据变化，响应式系统立刻知道哪里变化，但是一个数据就要绑定一个 Watcher。一旦绑定的颗粒度过高，就会产生大量 Watcher，增加内存消耗及依赖追踪的开销；而颗粒度过低就不能精准侦测变化。Vue 在组件级别进行响应式侦测，组件内部通过 Diff 算法查找变化。所以 Vue 是 pull + push

数据劫持
Vue 2.0 数据劫持实现方式是 Object.defineProperty Vue 3.0+ 用的是 Proxy
具体方法初步实现如下
let obj = {
    name: ''
};

obj = new Proxy(obj, {
    get(target, prop) {
        console.log('a')
        return target[prop]
    },
    set(target, prop, value) {
        console.log('b')
        target[prop] = value
    }
})

// 绑定HTML 上面的值设置好 数据更改影响视图
function observer() {
    spanName.innnerHTML = obj.name;
    inpName.value = obj.name;
}

observer();

setTimeout(() => {
    obj.name = '测试'
}, 1000);

inpName.oninput = function () {
    obj.name = this.value
}

答案
从代码可以很明显看出 这里是不涉及到到底我要更新哪个DOM的 只是知道数据更新，然后去更新数据。
所以需要 DOM diff算法 去比较
关于爆栈
爆栈一般是因为循环引用的问题，具体解决方法是类似于 用 new WeakSet 进行处理循环应用的问题？具体我不太清楚

VUE的数据劫持需要绑定Watcher,通知相关的依赖项进行数据更新.而组件内状态可能会很多,数据相应的粒度做的太细会导致性能损耗过大,所以VUE在组件间采用数据劫持的方法,知道是哪个组件的数据更新,然后组件内部使用 dom diff ,更新发生了数据变化的DOM.

VUE的数据劫持需要绑定Watcher,通知相关的依赖项进行数据更新.而组件内状态可能会很多,数据相应的粒度做的太细会导致性能损耗过大,所以VUE在组件间采用数据劫持的方法,知道是哪个组件的数据更新,然后组件内部使用 dom diff ,更新发生了数据变化的DOM.

marked
关于爆栈问题：
lodash 的deepClone遇到环形对象时，也是遇到了同样的方案。

依赖收集到通知组件update，具体更新哪些节点是需要diff做处理，至于为什么不采用细粒度更高的监听，应该就和上面的老哥说的差不多，更细化的watcher会牺牲更多的性能，性价比不高。
爆栈的问题应该说的是vue采用递归的方式对数据劫持吧，遇到大量数据的时候是需要脱离vue的数据劫持模式的，一种方法是可以采用函数的形式传递数据，如果还有其他方法希望有老哥可以告诉我

通过dom diff来对比新旧dom 不同的地方以及需要做更新的地方汇总后进行具体的修改回显到画面

1.组件的高度抽象化
2.可以更好的实现 SSR，同构渲染等
3.框架跨平台

VUE通过Object.definedProperty监听到对象属性的get和set，在数据变动时发布消息给订阅者，触发相应的监听回调。深度监听时需要递归到底，深层遍历导致性能下降。通过楼上老哥了解到proxy的优点可以直接监听数组的变化。vue在组件级进行push检测的方式(即依赖响应式系统)，在组件内部进行Virtual Dom Diff获取更加具体的差异

authors: BigDeep Chrisyjs KieSun SunshineZJJ Turkyden huangping-code louwenqiang makejun168 muzishuiji niices sunsmeil xt1995726 yiyu66

repo: KieSun/fucking-frontend


text:
improve scripts

Using one compare operator is better in shell.
In bash it seems to doesn't matter, but shell throws a error here
scripts/update.sh: 14: [: 2.5.5: unexpected operator
switch directories before update

Will do some more tiny updates soon, as I told in #95.
Got some problems with git and github...

What's problems? Same problem?
What's version of bash do you have?
Understand that I need more context for help you.
On Tuesday, October 6, 2015, foorb notifications@github.com wrote:

Will do some more tiny updates soon, as I told in #95
https://github.com/Kikobeats/uno-zen/pull/95.
Got some problems with git and github...
—
Reply to this email directly or view it on GitHub
https://github.com/Kikobeats/uno-zen/pull/98#issuecomment-145842746.

Ahh...no problems related to this project.
In general I had to understand how git and github works...
I deleted my account from pull request #95 but now everything ist fine and I'm back :)

Nice, close issue then
On Tuesday, October 6, 2015, foorb notifications@github.com wrote:

Ahh...no problems related to this project.
In general I had to understand how git and github works...
I deleted my account from pull request #95
https://github.com/Kikobeats/uno-zen/pull/95 but now everything ist
fine and I'm back :)
—
Reply to this email directly or view it on GitHub
https://github.com/Kikobeats/uno-zen/pull/98#issuecomment-145844701.

Do you need any futher information for this pull request?

Sorry, I read this in my mail and I believed that was an issue.
Thanks! :-)

authors: Kikobeats foorb

repo: Kikobeats/uno-zen


text:
Hi

Hi friend, could you rate my repositories and I'll rate yours at the same time? Thanks!

yes no problem

but where should I rate your repositories

Repo star

Thanks

ur welcome