[HOLD] Update favicon location and static reference

As part of the Wordpress removal effort, we need to switch to using a favicon that's part of the Django app. Up till now the favicon has come from an internal Wordpress theme repo which gets added to the build artifact via no-drama's --aux option.
We also noticed that the existing <link/> reference to a favicon in the site's base template to a static file ('assets/favicon.ico') pointed to a favicon that, as far as we can tell, did not get placed on the filesystem as part of the existing Ansible deploy process. That is, the file did not seem to get placed there via collectstatic or some other build/deploy operation. We don't know how it got there.
To address the confusion we have moved an existing copy of the file (which is binary equal to the current favicon.ico being served on the site) so that it will be collected to the top of the STATIC_ROOT.
Also:

I did not remove this other favicon.ico
I'm assuming we do not care about this reference to a favicon.png
This PR has a corresponding PR in the GHE Ansible repo

:+1: This looks good to me. I don't really know anything about that other favicon.ico, or about the PNG. I can confirm when running locally that http://localhost:8000/static/favicon.ico works, as we'd hoped, and that pages point to it in their base template.
Can @mistergone @Scotchester give this a FEWD's eye?

I can't speak to the mystery file, but these changes look good to me!
👍

OK, after some helpful input from @chosak, here's my thinking:
The only other thing in the path proposed in this PR (cfgov/v1/static/) is the question_tips CSS file, which is for the Wagtail Admin, and I think that should get relocated to where the other Wagtail Admin CSS lives (cfgov/templates/wagtailadmin/css/). That would leave the favicon as the only thing in cfgov/v1/static/, which seems odd.
Additionally, the favicon belongs to the whole of consumerfinance.gov, and thus feels more at home outside of the v1 app. Andy suggested cfgov/core/static/, which I think makes good sense. It will still get picked up automatically by Django's collectstatic there, and it's a sort of kindred spirit with the robots.txt file that already lives there.

P.S.: Don't feel the need to deal with that admin CSS file in this PR. I can open a PR for that.

@Scotchester Please confirm this looks right now – thanks!


Replace protobuf library

Our current protobuf library requires a lot of boiler plate.
Use a better protobuf library that goes between struct <=> Vec
Potential option: serde-protobuf
Not quite [yet]. "Serialization is not yet implemented in this version."

This is no longer relevant, the only places where protobufs are used in Rust is as indirect dependencies for aesm and grpc.


Varnish looses backend connection if service ip changes

Describe the bug
As soon as the service ip of nginx (or any service running behind varnish) is changed varnish looses the connection as the dns lookup is done just once during vcl load (https://varnish-cache.org/lists/pipermail/varnish-misc/2016-May/024952.html)
🎉 props to M. Hutter from VSHN for finding this issue.
To Reproduce
Steps to reproduce the behavior:

Visit site behind varnish
recreate nginx service (this will trigger an ip change)
visit a non-cached site
you should get a backend-error from varnish

Expected behavior
Varnish does not loose connection

looks like https://github.com/nigoroll/libvmod-dynamic would solve that

learn more https://info.varnish-software.com/blog/truly-dynamic-backend-definition-varnish


Bookmark favicons not displaying after M115.0.5790.172

System Details

OS: Manjaro (testing branch, Kernel 6.4.11) & 2 different installations of Windows 11. All with latest updates.
Thorium Version M115.0.5790.172

Problem
Bookmarks show the default greyed-out globe placeholder favicon instead of site-specific favicons post-update to M115.0.5790.172.
This first happened on my Manjaro install after updating Thorium-Bin from AUR. Upon starting Thorium I was greeted with a popup warning that something in my profile couldn't be read (different wording, but that was the premise) and I noticed this issue.
I also updated Thorium AVX2 on my Windows 11 install on the same machine, and on a different W11 machine, and observed the same problem, this time with no popup related to a profile issue.
Screenshots

Tracked down the exact profile warning I got on Manjaro:

@turtlshell Were you using the portable flags?

@turtlshell Were you using the portable flags?

No, I wasn't

Please use the new user data, and only import bookmarks, and then check whether the favicons of bookmarks can be displayed normally
@turtlshell


TypeError: requestsById[request.id].push is not a function

I understand this project is no longer actively maintained. I'm just posting this bug report here in case any other users of this project might be able to suggest a fix. :)
I installed and ran local-npm yesterday and it worked fine. Then this morning when I tried to launch it I got this error message:
~/programs/local-npm > local-npm

Welcome!
To start using local-npm, just run:

  $ npm set registry http://<IP_ADDRESS>:5080

To switch back, you can run:

  $ npm set registry https://registry.npmjs.org

A simple npm-like UI is available here: http://<IP_ADDRESS>:5080/_browse

PouchDB Server listening on port 16984.
Navigate to http://localhost:16984/_utils for the Fauxton UI.

/usr/lib/node_modules/local-npm/node_modules/pouchdb/lib/index.js:247
      requestsById[request.id].push(request);
                               ^

TypeError:` requestsById[request.id].push is not a function
    at /usr/lib/node_modules/local-npm/node_modules/pouchdb/lib/index.js:247:32
    at Array.forEach (native)
    at bulkGet (/usr/lib/node_modules/local-npm/node_modules/pouchdb/lib/index.js:245:12)
    at doBulkGetShim (/usr/lib/node_modules/local-npm/node_modules/pouchdb/lib/index.js:5438:9)
    at /usr/lib/node_modules/local-npm/node_modules/pouchdb/lib/index.js:5460:13
    at Request._callback (/usr/lib/node_modules/local-npm/node_modules/pouchdb/lib/index.js:5128:7)
    at Request.self.callback (/usr/lib/node_modules/local-npm/node_modules/pouchdb/node_modules/request/request.js:200:22)
    at emitTwo (events.js:87:13)
    at Request.emit (events.js:172:7)
    at Request.<anonymous> (/usr/lib/node_modules/local-npm/node_modules/pouchdb/node_modules/request/request.js:1067:10)
    at emitOne (events.js:82:20)
    at Request.emit (events.js:169:7)
    at IncomingMessage.<anonymous> (/usr/lib/node_modules/local-npm/node_modules/pouchdb/node_modules/request/request.js:988:12)
    at emitNone (events.js:72:20)
    at IncomingMessage.emit (events.js:166:7)
    at endReadableNT (_stream_readable.js:921:12)
    at nextTickCallbackWith2Args (node.js:442:9)
    at process._tickCallback (node.js:356:17)

npm: 2.15.9
node: 4.6.1
local-npm: 1.6.0
From the stack it looks like it's an issue relating to pouchdb. The version of pouchdb I've got installed under local-npm, is 5.4.5
> npm -g ls pouchdb
/usr/lib
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>

Update: The quick, dumb fix, was to just delete all the files local-npm created in the directory where I was running it. But I assume that means that all the data it cached has now been cleared out, and it'll have to re-download a bunch of stuff.

I had the same issue and I just reinstalled local-npm and it worked again.

I had the exact same problem, and found a possible solution.
I've changed index.js@248 from:
if (request.id in requestsById)
to
if ((request.id in requestsById) && (Array.isArray(requestsById[request.id])))
Look like the object requestsById sometimes didn't hold the expected Array.

thank you Shaviest it's work for me after the change in index.js

@Shaviest would you be interested in opening a PR for that fix?

@gabrielcsapo actually I don't know how, since the fix has to be made to one of the node modules (pouchdb)  and not to local-npm itself.

I cross posted the issue in pouchdb hopefully we can get some momentum from that project to help us fix this.

looks like this was fixed by simply updating pouchdb.


App pathways set up

Buttons made to direct people to their specific component when working on their section on the client side.

Looks good


React Starter Code

This merge will add the /front-end to our project with the React starter code for new projects.
I've made the default page say 'Hello KrispyKopters' with a spinning donut to verify it works.

Hmm I have a few questions about this and I think we'll need to get into the infrastructure a bit to get it working:

How will this be hosted? It's a little different from what I originally thought since it seems like we'll have two servers.
Can you set up unit and integration tests?
Can you prototype communicating between FE/BE?


Updated Debian 9.1 to Debian 9.2.

Upgraded to latest version of Debian 9.
PS - only tested with builder=virtualbox, as I do not have vmware or qemu installed.

I'm sorry for making you wait. It works for me with both vmware and qemu as well. Thank you!


minusDiv -> inverseLerp

Someone didn't look at this function very carefully :P

Is inverseLerp the best name for this? Maybe something like getProgress(value, start, end) would be a better name, since calculating the "progress" from the start value to the end value given the current value.

            this.lastNetherPortalPosition = new BlockPos(pos);
            NetherPortalBlock class_2423 = (NetherPortalBlock)Blocks.NETHER_PORTAL;
            BlockPattern.Result result3 = NetherPortalBlock.findPortal(this.world, this.lastNetherPortalPosition);
            double double4 = (result3.getForwards().getAxis() == Direction.Axis.X) ? result3.getFrontTopLeft().getZ() : ((double)result3.getFrontTopLeft().getX());
            double double6 = Math.abs(MathHelper.minusDiv(((result3.getForwards().getAxis() == Direction.Axis.X) ? this.getZ() : this.getX()) - (double)((result3.getForwards().rotateYClockwise().getDirection() == Direction.AxisDirection.NEGATIVE) ? 1 : 0), double4, double4 - result3.getWidth()));
            double double8 = MathHelper.minusDiv(this.getY() - 1.0, result3.getFrontTopLeft().getY(), result3.getFrontTopLeft().getY() - result3.getHeight());
            this.lastNetherPortalDirectionVector = new Vec3d(double6, double8, 0.0);
            this.lastNetherPortalDirection = result3.getForwards();

Fyi this is like the only place where this method is used (in entity's portal code).

inverseLerp is definitely a better name than getProgress, especially since this is the math class. However I approve of the start and end parameter names.

inverseLerp is definitely a better name than getProgress

The term "progress" is used in many other places in yarn to refer to this kind of value. I don't see why you think the method should be described as the "inverse of lerp" when it can be directly described as "progress". It would be a bit like calling division "inverse multiplication".
The javadocs also focus on this being "the delta value needed to interpolate between", even though its only usage isn't to find the delta for an interpolation.

especially since this is the math class

I don't see what you mean by that.

After some more thought and discussion on discord, I have decided to name the function getLerpProgress. getProgress on its own is too vague, whereas inverseLerp is a little difficult to read, especially given that it only inverts for one variable.
I will also update the javadocs to describe the function first without using lerp, then later explain its link to lerp.

I also made a couple of other improvements to MathHelper.

Can you rename randomUUID to randomUuid as well @Earthcomputer?

Done

Can be merged once branch is 20w09a

Done


python3Packages.ldappool: fix build

Motivation for this change
The build was failing due to the update of PrettyTable.
Also, missing licenses were added.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108847 run on x86_64-darwin 1

1 package marked as broken and skipped:

python39Packages.ldappool

2 packages built:

python37Packages.ldappool
python38Packages.ldappool

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108847 run on x86_64-darwin 1

1 package marked as broken and skipped:

python39Packages.ldappool

2 packages built:

python37Packages.ldappool
python38Packages.ldappool

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108847 run on x86_64-linux 1

1 package marked as broken and skipped:

python39Packages.ldappool

2 packages built:

python37Packages.ldappool
python38Packages.ldappool

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108847 run on x86_64-linux 1

1 package marked as broken and skipped:

python39Packages.ldappool

2 packages built:

python37Packages.ldappool
python38Packages.ldappool


Prepare for Snapshots release 0.5.0

Current dependencies on/for this PR:

main

PR #20

PR #21

PR #22   👈

This comment was auto-generated by Graphite.


More .md fixes

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.
The regexp is smarter now and takes care of # too:
/s/\((\{.*)\.md\s*(\#|\))/($1$2

sigh, bad merge, fixing

Issues resolved!


The cards are not in the right fit in the divine activities for all generation page.

I would like to fix its position or I can add a horizontal scroll bar . gssoc'23 participant

@P10swati There already exists an issue for the same and also has been assigned,please look out for some other issues to contribute to.
Thank you

Okay . Thanks for letting me know.
On Sun, May 21, 2023, 3:24 PM karthik nadar @.***>
wrote:

@P10swati https://github.com/P10swati There already exists an issue for
the same and also has been assigned,please look out for some other issues
to contribute to.
Thank you
—
Reply to this email directly, view it on GitHub
https://github.com/akshitagupta15june/Moksh/issues/239#issuecomment-1556135267,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7B3XVNJRXEGQ5XMHF3O57LXHHQ5HANCNFSM6AAAAAAYJIRTFM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Type mismatch: inferred type is MainActivity but FlutterEngine was expected

I tried to implement your wallpaper apply method, but it fails, even tried cloning the reWall but the same error.
console message:
e: D:\projects\amazewall_wallpapers\android\app\src\main\kotlin\com\example\amazewall_wallpapers\MainActivity.kt: (24, 44): Type mismatch: inferred type is MainActivity but FlutterEngine was expected

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileDebugKotlin'.
> Compilation error. See log for more details

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 30s
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                        31.5s
Exception: Gradle task assembleDebug failed with exit code 1

@varadgauthankar Can you add in the information from flutter doctor?


Polish spring-cloud-consul.adoc.

Fix missing back bracket.

Cherry-picked to 3.1.x and merged to main via 9e594cbd9d9e70f913f65888d9209d5d76f3b9bd


AssemblyAPI service code, HTTP services, thrift generated code

Added AssemblyAPI server code and client test driver for manual verification
Minor fixes to TaxonAPI
Converted service drivers and clients to use HTTP
Converted custom install command in setup.py to a custom build command for thrift server code
Bugfix to build thrift clients command to properly capture output for each language target
Checked in all thrift generated client and server code

Current coverage is 59.76%

Merging #97 into develop will decrease coverage by -2.26% as of 3aee3ab

@@            develop     #97   diff @@
=======================================
  Files            14      15     +1
  Stmts          2212    2324   +112
  Branches          0       0
  Methods           0       0
=======================================
+ Hit            1372    1389    +17
  Partial           0       0
- Missed          840     935    +95

Review entire Coverage Diff as of 3aee3ab

Powered by Codecov. Updated on successful CI builds.


Differentiating a continuous time Markov chain with respect to the soujourn times

First issue I've done in my whole life
Edit: first line of code was not visible because I don't knon Markdown
This code simulates a continuous time Markov chain (CTMC) given some transition probabilities $P$ and a vector of rates of jumps $\theta_i$ with $i=1,\dots,n$, $n$ being the number of nodes or states in the CTMC, meaning that if you are at state $i$ the time it takes to jump out of that state follows an $\text{Exp}(\theta_i)$. As you can see the code starts at state $i=2$ and simulates the MC up to time $T=1.5$. The expected value of the vector $\pi$ is $p_{ij}(T)$ the probability of being at state $j$ at time $T$ having started at $i$. I want to differentiate that w.r.t. the vector of rates $\theta$, I just don't have a clue on how to write this in the form of the paper
n = 7; P = rand(n,n); P[1:n+1:end] .= 0;
D = zeros(n,n); D[1:n+1:end] .= 1 ./sum(P,dims=2); P = cumsum(D*P,dims=2); # matrix of cumulative probabilities
π = zeros(n); # final distribution

θ = rand(n) # vector of rate of jumps
i = 2; # init state
T = 1.5; # final time
t = -log(1-rand())/θ[i] # first jump

while t < T
    i = argmax(rand().<=P[i,:]) # pick new state
    t += -log(1-rand())/θ[i] # generate random time
end
# the result is i
π[i] += 1 # I want to differentiate this w.r.t θ hihi, so it is an nxn matrix

This is what I've managed to do for now using the clue at 4. Limitations and outlook, rewrite if branches with array indexing, still can't wrap my head around writing the while
using StochasticAD, QuadGK, Distributions

v = rand(2) # final vector
T = 0.7 # final time

function X(θ,T,v)
    p = quadgk(t->pdf(Exponential(θ),t),0,T)[1] # probability of jump
    idx = rand(Bernoulli(p)) + 1
    return v[idx]
end

X(θ) = X(θ,T,v)
st = stochastic_triple(X,0.6)

samples = [derivative_estimate(X,0.6) for i in 1:1000]

println("d/dp of E[X(p)]: $(mean(samples)) ± $(std(samples)) / sqrt(1000) ")

include("MC_exp_diffing.jl")

function create_random(n)
    Q = rand(n,n)
    θ = rand(n)
    Q[1:n+1:end] .= 0
    P = copy(Q)
    for i=1:n
        Q[i,:] /= sum(Q[i,:])
        Q[i,i] = -sum(Q[i,:])
        P[i,:] = cumsum(P[i,:]./sum(P[i,:]))
    end
    return P, Q, θ
end

n = 11
P, Q, RATE_VEC = create_random(n)
END_VEC = randn(n)

DET_DER = DET_exp_diffing(RATE_VEC,Q,1,END_VEC,2)
MC_DER = MC_exp_diffing_OUTER(RATE_VEC,P,1,END_VEC,2,1000000,100)

@show DET_DER
@show MC_DER
@show maximum(abs.(DET_DER-MC_DER))

Hey! First, would you mind reporting the standard error of your Monte Carlo measurements along with the means? This can help us figure out if the issue is due to high variance or bias in the estimate.

function rand_walk(STATE_CHAIN,RATE_VEC,END_VEC,T)
    p = 1-exp(-T*RATE_VEC[STATE_CHAIN[1]]) # probability of jump
    B = rand(Bernoulli(p))+1 # 1 is no jump, 2 is jump
    # rng for exponential conditioned to jump before T
    JUMP = -log(1-rand()*(1-exp(-RATE_VEC[STATE_CHAIN[1]]*T)))/RATE_VEC[STATE_CHAIN[1]]
    # following vector should be fully computed if there has been a jump
    aux_bool = copy(B.value) # does not support this, need to somehow copy
    if aux_bool == 2
        aux = [END_VEC[STATE_CHAIN[1]]; rand_walk(STATE_CHAIN[2:end],RATE_VEC,END_VEC,T-JUMP)]
    else
        aux = [END_VEC[STATE_CHAIN[1]]; false]
    end
    return aux[B]
end

Directly accessing the value property of the stochastic triple would indeed introduce bias. Would you mind providing the most natural to write version of this function without worrying about being StochasticAD compatible, so I can understand what you are trying to do here? I can then help you write it in a StochasticAD compatible way.

This is a MWE of what I want to do, without trying to write it StochasticAD compatible, in this random seed you can see that the estimator is unbiased, sorry for answering so late, I was trying to do SPA and Malliavin weights unsuccesfully (ㆆ _ ㆆ), if you were to solve this my mental health would go up 9000 points probably
using Random, LinearAlgebra, Distributions
Random.seed!(2024)
# number of states
n = 10
# infinitesimal generator
Q = rand(n,n); Q[1:n+1:end] .= 0; for i=1:n; Q[i,:] ./= sum(Q[i,:]); Q; end; Q[1:n+1:end] .= -1
# embedded Markov chain
P = copy(Q); P[1:n+1:end] .= 0; P = cumsum(P,dims=2)
# rewards
D = -rand(n)
# rates
θ = rand(n)
# final reward
u0 = randn(n)
# initial state
init_state = rand(1:n)
# time
T = 0.15

# deterministic solution
solution = (exp(diagm(θ)*(diagm(D)+Q)*T)*u0)[init_state] # 1.0582327608691373

# stochastic solution
nsims = Int(1e8)
samples = zeros(nsims)

for sim=1:nsims
    i = copy(init_state); t = 0 # reset simulation
    τ = rand(Exponential(1/θ[i])); t += τ
    η = exp(θ[i]*D[i]*τ)
    while t < T
        i = argmax(P[i,:].>rand())
        τ = rand(Exponential(1/θ[i])); t += τ
        η *= exp(θ[i]*D[i]*τ)
    end
    η /= exp(θ[i]*D[i]*(t-T)) # subtract reward that was cutted short by end of simulation
    samples[sim] = η*u0[i]
end

sample_mean = mean(samples) # 1.0582814906302531
sample_sem = std(samples)/sqrt(nsims) # 4.6125383852028576e-5
println(abs(sample_mean - solution) < 1.96*sample_sem ? "we are so back" : "I should quit the phd") # we are so back in Random.seed!(2024)

# I would like to compute this
# I've tried SPA and the score function but... can't get it :(, maybe I was getting it but there
# was so much bias on the FD solution that I discarded the functions, specially when the derivatives
# are of order 1e-5 and FD is no longer that precise
dsoldθ = zeros(n)
for i=1:n
    θpert = copy(θ); θpert[i] += sqrt(eps(Float64))
    solpert = (exp(diagm(θpert)*(diagm(D)+Q)*T)*u0)[init_state]
    dsoldθ[i] = (solpert-solution)/sqrt(eps(Float64))
end

The reason StochasticAD fails is because the t contains a dual component which is dropped in the t < T comparison. (See the fifth bullet point of https://gaurav-arya.github.io/StochasticAD.jl/stable/limitations.html)
One simple way of working around this is to supply a rate bound for the exponential distribution that is independent of the current state i, and then possibly "do nothing" to account for this rate being larger than the true rate. I've implemented the solution for you in the below gist!
https://gist.github.com/gaurav-arya/e98dde330be1902c3cbb4bbecd8b16b5


Changed the way of getting the refresh token

https://app.clickup.com/t/24308805/VDM-1665

Could we add in the README of MailChimp Insert how the Sequence works with Slack?
It would be beneficial to the users.
Thanks


Fix capitalization

https://github.com/cmungall/capital-offence/issues/1
To be capitalized
lieberkuhn

CL:0002250 ! intestinal crypt stem cell [SYNONYM: "stem cell of intestinal crypt of  lieberkuhn " (exact)]
CL:1000338 ! enterocyte of epithelium of crypt of  lieberkuhn  of small intestine
CL:1000345 ! paneth cell of epithelium of crypt of  lieberkuhn  of small intestine
UBERON:0001241 ! crypt of Lieberkuhn of small intestine [SYNONYM: "crypt of  lieberkuhn  of small bowel" (exact)] [SYNONYM: "crypt of  lieberkuhn  of small intestine" (exact)] [SYNONYM: "intestinal gland of small bowel" (exact)] [SYNONYM: "intestinal gland of small intestine" (exact)] [SYNONYM: " lieberkuhn  crypt of small bowel" (exact)] [SYNONYM: " lieberkuhn  crypt of small intestine" (exact)] [SYNONYM: "small bowel crypt of  lieberkuhn " (exact)] [SYNONYM: "small bowel intestinal gland" (exact)] [SYNONYM: "small bowel  lieberkuhn  crypt" (exact)] [SYNONYM: "small intestinal crypt of Lieberkuhn" (exact)] [SYNONYM: "small intestine crypt of Lieberkuhn" (exact)] [SYNONYM: "small intestine crypt of  lieberkuhn " (exact)] [SYNONYM: "small intestine intestinal gland" (exact)] [SYNONYM: "small intestine  lieberkuhn  crypt" (exact)]
UBERON:0001983 ! crypt of Lieberkuhn [SYNONYM: "crypt of  lieberkuhn " (exact)] [SYNONYM: "crypt of LieberkC<hn" (exact)] [SYNONYM: "crypts of LieberkC<hn" (exact)] [SYNONYM: "follicles of  lieberkuhn " (exact)] [SYNONYM: "intestinal crypt" (exact)] [SYNONYM: "intestinal crypts" (exact)] [SYNONYM: "intestinal gland" (broad)] [SYNONYM: "intestinal gland of Lieberkuhn" (related)] [SYNONYM: "Lieberkuhn crypt" (exact)] [SYNONYM: " lieberkuhn  crypt" (exact)] [SYNONYM: "Lieberkuhn gland" (exact)] [SYNONYM: "Lieberkuhn's gland" (exact)] [SYNONYM: "Lieberkuhn's glands" (exact)]
UBERON:0001984 ! crypt of Lieberkuhn of large intestine [SYNONYM: "intestinal gland of large intestine" (exact)] [SYNONYM: "large intestinal crypt of  lieberkuhn " (exact)] [SYNONYM: "large intestine crypt of  lieberkuhn " (exact)] [SYNONYM: "large intestine intestinal gland" (exact)] [SYNONYM: "large intestine  lieberkuhn  crypt" (exact)] [SYNONYM: " lieberkuhn  crypt of large intestine" (exact)]
UBERON:0013485 ! crypt of Lieberkuhn of colon [SYNONYM: "colonic crypt" (exact)] [SYNONYM: "colonic crypt of  lieberkuhn " (exact)]
UBERON:0013486 ! crypt of Lieberkuhn of appendix [SYNONYM: "appendiceal crypt" (exact)] [SYNONYM: "appendiceal crypt of  lieberkuhn " (exact)]

henle

CL:1000909 ! kidney loop of  henle  epithelial cell
CL:1001016 ! kidney loop of  henle  ascending limb epithelial cell
CL:1001021 ! kidney loop of  henle  descending limb epithelial cell
CL:1001106 ! kidney loop of  henle  thick ascending limb epithelial cell
CL:1001107 ! kidney loop of  henle  thin ascending limb epithelial cell
CL:1001108 ! kidney loop of  henle  medullary thick ascending limb epithelial cell
CL:1001109 ! kidney loop of  henle  cortical thick ascending limb epithelial cell
CL:1001111 ! kidney loop of  henle  thin descending limb epithelial cell
UBERON:0001288 ! loop of Henle [SYNONYM: "ansa nephroni" (exact)] [SYNONYM: "Henle loop" (exact)] [SYNONYM: " henle 's loop" (exact)]
UBERON:0003455 ! inner renal medulla loop of  henle  [SYNONYM: "Henle loop of inner medulla of kidney" (exact)] [SYNONYM: "Henle loop of inner renal medulla" (exact)] [SYNONYM: "Henle loop of inner zone of medulla of kidney" (exact)] [SYNONYM: "Henle loop of inner zone of renal medulla" (exact)] [SYNONYM: " henle 's loop of inner medulla of kidney" (exact)] [SYNONYM: " henle 's loop of inner renal medulla" (exact)] [SYNONYM: " henle 's loop of inner zone of medulla of kidney" (exact)] [SYNONYM: " henle 's loop of inner zone of renal medulla" (exact)] [SYNONYM: "inner medulla of kidney Henle loop" (exact)] [SYNONYM: "inner medulla of kidney  henle 's loop" (exact)] [SYNONYM: "inner medulla of kidney loop of Henle" (exact)] [SYNONYM: "inner renal medulla Henle loop" (exact)] [SYNONYM: "inner renal medulla  henle 's loop" (exact)] [SYNONYM: "inner renal medulla loop of Henle" (exact)] [SYNONYM: "inner zone of medulla of kidney Henle loop" (exact)] [SYNONYM: "inner zone of medulla of kidney  henle 's loop" (exact)] [SYNONYM: "inner zone of medulla of kidney loop of Henle" (exact)] [SYNONYM: "inner zone of renal medulla Henle loop" (exact)] [SYNONYM: "inner zone of renal medulla  henle 's loop" (exact)] [SYNONYM: "inner zone of renal medulla loop of Henle" (exact)] [SYNONYM: "kidney inner medulla loop of  henle " (exact)] [SYNONYM: "loop of Henle of inner medulla of kidney" (exact)] [SYNONYM: "loop of Henle of inner renal medulla" (exact)] [SYNONYM: "loop of Henle of inner zone of medulla of kidney" (exact)] [SYNONYM: "loop of Henle of inner zone of renal medulla" (exact)]
UBERON:0005096 ! descending thin limb [SYNONYM: "descending limb of  henle %27s loop" (related)] [SYNONYM: "descending limb of loop of  henle " (related)] [SYNONYM: "descending thin limb of loop of Henle" (exact)] [SYNONYM: "loop of Henle descending thin limb" (exact)] [SYNONYM: "loop of  henle  thin descending limb" (exact)] [SYNONYM: "pars descendens (tubulus attenuatus)" (exact)] [SYNONYM: "thin descending limb" (exact)] [SYNONYM: "thin descending limb of loop of  henle " (related)] [SYNONYM: "thin descending limb of the loop of  henle " (related)] [SYNONYM: "thin descending loop of  henle " (related)]
UBERON:0006541 ! outer medulla inner stripe loop of  henle
UBERON:0006542 ! outer medulla outer stripe loop of  henle
UBERON:0034996 ! outer renal medulla loop of  henle  [SYNONYM: "loop of Henle, outer medullary portion" (exact)]
UBERON:0034997 ! renal medulla loop of  henle  [SYNONYM: "loop of Henle of renal medulla" (exact)] [SYNONYM: "loop of Henle, medullary portion" (exact)]

Lugaro

CL:0011006 !  lugaro  cell [SYNONYM: "Lugaro cell" (exact)]

Purkinje

CL:1000376 !  purkinje  myocyte of interventricular septum

CL:1000479 !  purkinje  myocyte of atrioventricular node
CL:1000483 !  purkinje  myocyte of internodal tract
CL:1000484 !  purkinje  myocyte of atrioventricular bundle

Peyer

CL:1000411 ! endothelial cell of  peyer 's patch
UBERON:0004693 ! Peyer's patch epithelium [SYNONYM: " peyer 's patch epithelium" (exact)]
UBERON:0010386 ! Peyer's patch follicle [SYNONYM: " peyer 's patch follicle" (exact)]

I can fix these manually, does that work?

Go for it - I assume you will just do in a text editor?
obv. the uberon ones will be fixed at src

Do you just want the labels to have proper capitalization? So the synonym in the first example would be 'stem cell of intestinal crypt of Lieberkuhn'
I would just do this in Protege, right?

On 6 Sep 2016, at 14:10, nicolevasilevsky wrote:

Do you just want the labels to have proper capitalization? So the
synonym in the first example would be 'stem cell of intestinal crypt
of Lieberkuhn'

labels and synonyms, and yes that's the correct capitalization for the
synonym

I would just do this in Protege, right?

if that's the way you really want to do it. I mean this is designed so
that we can just run the script to do it but I won't have time to do it

Ah, I see. I can just fix it via Protege, it shouldn't take me too long.

I fixed all the CL terms except
CL:0011006 ! lugaro cell [SYNONYM: "Lugaro cell" (exact)]
If I change the label to Lugaro cell, it will then match the synonym. Should I do that and remove the synonym?

Yes, remove it
On 6 Sep 2016, at 16:43, nicolevasilevsky wrote:

I fixed all the CL terms except
CL:0011006 ! lugaro cell [SYNONYM: "Lugaro cell" (exact)]
If I change the label to Lugaro cell, it will then match the synonym.
Should I do that and remove the synonym?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/obophenotype/cell-ontology/issues/438#issuecomment-245130590

A handful of cases of capitalization still need to be fixed, I'll address. Findable by searching the following in Protege with "Case sensitive" checkbox:
lieberkuhn
peyer


Loading notes locally takes 900ms in grunge stack compared to 30ms in indie

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce

Create a new grunge stack and run it
Create account
Create 2 notes
Open network developer tool in browser
Click between the two notes repeatedly and watch the network call durations

Expected Behavior
Loading notes takes 30ms (like it does in Indie stack)
Actual Behavior
Loading notes takes 800-900ms

Yes, I have the same issue. It takes about 100-300 miliseconds with the indie stack and 1.6 seconds on AWS Lambda (1.1 seconds with ARC sandbox) with the grunge stack.


How disable create class in JS code?

I reread a lot in the Wiki on the topic of banning the external constructor, but nowhere did I find C ++ code. Can I prevent the creation of an object from JS code? To give an error like in Chrome
new HTMLDocument ();
VM459: 1 Uncaught TypeError: Illegal constructor
     at <anonymous>: 1: 1

But so that there are no problems with creating an object in C ++ code

All Duktape/C functions are "constructable" from script point of view, but you can reject such calls explicitly using https://duktape.org/api.html#duk_is_constructor_call:
if (!duk_is_constructor_call(ctx)) {
    duk_type_error(ctx, "not constructable");
}

Internally each function actually does have a separate CONSTRUCTABLE flag, and there are some internally created functions which are not constructable. But the C check quoted above produces roughly the same result; there's a small difference in the traceback because the function is entered before the error is thrown. The ability to control the CONSTRUCTABLE flag or create a non-constructable function needs to be added to the API IMO.

As for being able to create the instance from C code, you could use a structure such as this:
static void push_new_instance(duk_context *ctx) {
    /* Push and initialize instance; leave it on stack. */
    duk_push_object(ctx);

    /* ... */

    /* Finalize object's internal prototype, to inherit from what you want. */
    duk_eval_string(ctx, "HTMLDocument.prototype");  // push whatever object you want to inherit from
    duk_set_prototype(ctx, -2);

    /* If it matters, set .constructor property. It usually has little impact. */
    duk_eval_string(ctx, "HTMLDocument");
    duk_put_prop_string(ctx, -2, "prototype");
}

duk_ret_t my_function(duk_context *ctx) {
    if (duk_is_constructor_call(ctx)) {
        duk_type_error(ctx, "not constructable");
    }
    /* XXX: Do what you want with a non-constructor call. */
}

Any calls to the constructor via scripts (or even C code) would be rejected. But you'd be able to create instance with push_new_instance().

Well, then there is still a small bug, or I don’t know, it can be so. When converting an object to a string, when no toString is specified, [object Object] is returned. Although the prototype object is different.

Could you try evaluating:
instance.toString === Object.prototype.toString

If this is true, then the inheritance chain (internal prototype) is still wrong.
You can also check what the prototype object is using:
Object.getPrototypeOf(instance)

and see which object that returns.

@kirill-782 Any update on this?


Create rounded theme

This has a few subtle adjustments to make the discourse theme a bit less blocky and a bit more rounded. It also improves the landing page with a smooth gradient over the banner image to fade out to the bottom.
Detailed list of changes:
Homepage: Banner with search bar & rounded category boxes

Rounded topic map (the section below the first post)

Code blocks are rounded and have no max-height

Quotes are rounded too

Categories use circles instead of squares

Just noticed discourse uses a slightly different layout on mobile (so some stuff still has the old style there). I can adjust this theme to work with mobile too, but before I do so some feedback would be nice so that I don't do the work twice

I really like the changes here, I think we can use it as a base and continue building on top of it. BTW I don't have permissions to merge, not sure if you are the one that has to give them to me 😅

@coppolaemilio Added you as Admin, and the Website team with Write access.


gss_bayes.csv

Should gss_bayes.csv  (Chapter 1) be in /data ?
(found it at https://raw.githubusercontent.com/AllenDowney/BiteSizeBayes/master/gss_bayes.csv after opening the notebook BTW)

please refer In[4] ==> notebook chap01
Load the data file
from os.path import basename, exists
def download(url):
filename = basename(url)
if not exists(filename):
from urllib.request import urlretrieve
local, _ = urlretrieve(url, filename)
print('Downloaded ' + local)
download('https://github.com/AllenDowney/BiteSizeBayes/raw/master/gss_bayes.csv')


Enable dynamic batching.

Enabling dynamic batching, which means merging requests that come sufficiently close in time, to increase utilization of DALI pipeline.
It required refactoring some of the request handling functions. The executor was already prepared to work in such case.
First measurements show that the change can improve perf significantly in some cases.
I tested the L0_DALI_GPU_ensemble setup with quite small requests (4 samples, each ~10kB), changing the request concurrency from 4 to 10.
With the current version there was no significant change in throughput when increasing the concurrency (as expected).
With the dynamic batching support the throughput increased by 40% (5900 -> 8370 inf/sec), when increasing request concurrency from 4 to 10.
For low concurrency levels, the change does not affect the performance.

CI MESSAGE: [2428492]: BUILD STARTED

CI MESSAGE: [2428593]: BUILD STARTED

CI MESSAGE: [2428593]: BUILD FAILED

CI MESSAGE: [2428593]: BUILD PASSED

CI MESSAGE: [2428721]: BUILD STARTED

CI MESSAGE: [2428721]: BUILD PASSED

CI MESSAGE: [2428492]: BUILD PASSED

CI MESSAGE: [2447407]: BUILD STARTED

CI MESSAGE: [2447407]: BUILD PASSED

CI MESSAGE: [2452591]: BUILD STARTED

CI MESSAGE: [2452591]: BUILD PASSED


Separate codesign logic from electron builder for macOS

These changes should be included in the build CLI tool as following:

Release Command should have an argument code-sign that should provide the path for the signing configuration file.
Configuration file should include all possible paths and command + Anything that can be changed often to avoid direct changes on Build CLI code.
Configuration file format can be anything serialized with serde. It would be nice to have it in JSON format.
Changes can be push directly to this PR
Program should fail if this argument is provided on Linux or Windows

Hi @itsmesamster @DmitryAstafyev,
I've implemented the code signing and notarizing for MacOS as following:

I've added the CLI argument --code-sign or -c which accept the path for the configuration file of code signing.
The implementation of code signing is a part of release on MacOS now when the configuration file & Environment variables are provided.
The configuration file is in toml format because we can add comments to it unlike json format.
I've included all paths, commands and environment variables in the configuration file and created a configuration file with the current configurations talken from rake implementation under the path cli/codesign_config/macos.toml. The file is well documented to explain the meaning of those configurations
An example for running the code signing:

cargo chipmunk release -c cli/codesign_config/macos.toml

@AmmarAbouZor thanks for PR. A couple of moments:

as soon as application/holder/hooks/afterSign.js isn't used - makes sense to remove it.
please also make with @itsmesamster tests with failed signing process to make sure we have in output relevant error messages. For example, put into credentials some error, like APPLEIDPASS = FAKEPASS.


fix PostgreSQL name

Requirements is also misspelled

I would recommend moving the Requirements section to the top of the README.
I like using shields.io badges under the name as well.

Done!

@owen2345 This can be closed, correct?

Merged Manually


Feature/optimize h2 use stream

Issues associated with this PR
https://github.com/mosn/mosn/issues/1751
Solutions
serverStreamConnection在handleFrame的函数中，判断第一个data帧是否是endStream，如果是，即使配置了流式也不用，如果不是，再走流式。这样可以避免大部分unary请求的损失，因为大部分unary请求data不超过一帧（16kb）.
UT result

✘ xzy@xzydeMacBook-Pro  go test -v -gcflags "all=-N -l"
2021-08-25 18:25:05,264 [INFO] register a new handler maker, name is default, is default: true
2021-08-25 18:25:05,265 [INFO] [config] processor added to configParsedCBMaps
2021-08-25 18:25:05,265 [INFO] [network] [ register pool factory] register protocol: Http2 factory
=== RUN   TestDirectResponse
--- PASS: TestDirectResponse (0.00s)
=== RUN   TestStreamConfigHandler
=== RUN   TestStreamConfigHandler/test_stream_config
=== RUN   TestStreamConfigHandler/test_invalid_default
--- PASS: TestStreamConfigHandler (0.00s)
--- PASS: TestStreamConfigHandler/test_stream_config (0.00s)
--- PASS: TestStreamConfigHandler/test_invalid_default (0.00s)
=== RUN   TestServerH2UseStream
--- PASS: TestServerH2UseStream (0.00s)
=== RUN   TestClientH2UseStream
--- PASS: TestClientH2UseStream (0.00s)
=== RUN   Test_get_prefixProtocolVar
--- PASS: Test_get_prefixProtocolVar (0.00s)
=== RUN   Test_get_scheme
--- PASS: Test_get_scheme (0.00s)
PASS
ok      mosn.io/mosn/pkg/stream/http2   0.444s

Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result
xzy@xzydeMacBook-Pro  ~/Desktop/projects/gopath/src/github.com/XIEZHENGYAO/mosn   feature/optimize_h2_use_stream ●✚  golint pkg/stream/http2/stream.go
pkg/stream/http2/stream.go:84:10: if block ends with a return statement, so drop this else and outdent its block
pkg/stream/http2/stream.go:151:6: exported type StreamConfig should have comment or be unexported
pkg/stream/http2/stream.go:152:2: struct field Http2UseStream should be HTTP2UseStream

另外可以考虑补一个集成测试，就按照类似issue里描述的场景就可以

Done.


Commit are not attached to branch in repository

Describe the bug
When I use this tool on some repository, I can see the commit for the versioning. And when I check the commit of the tag I have this message display : "This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository."
To Reproduce
Steps to reproduce the behavior:
Don't know.
Can be see at :

https://github.com/neuroglia-io/framework with the commit https://github.com/neuroglia-io/framework/commit/e26fb43827b8adbd43ac8d82431a3705e8305775
https://github.com/serverlessworkflow/synapse with the commit https://github.com/serverlessworkflow/synapse/commit/d4615489200090afbbf3c60452204b45916872cf

To give more details, the executed command was:
dotnet-version increment-version-with-git -g "." --branch-name main --author-email<EMAIL_ADDRESS>See https://github.com/neuroglia-io/framework/actions/runs/3135533373/jobs/5091322057 for instance
From what I understood from the IncrementVersionWithGitIntegrationCommand handling
https://github.com/cbcrouse/Versioning.NET/blob/4f37fb0fc8b68ce610e9e5ac5ba4e054c8d260eb/src/Application/GitVersioning/Handlers/IncrementVersionWithGitIntegrationHandler.cs#L90
The commit should have been pushed to main.

Both main branches of the said projects are "protected", not allowing unsigned commit. The GitHub action bot doesn't sign the commits so they are not attached to main I guess.

Trying to sign commits/tags/pushes with crazy-max/ghaction-import-gpg@v4 does seem to work so far.

I had to work around, attach my personnal repo to an organisation, set the bot as admin and allow the admins to bypass the protected branch restrictions.
Here is the resulting pipeline:
# IMPORTANT NOTES:
# For a repo using "protected branches" strategies (approved PR & signed commits or other checks),
# it is mandatory to use a "power" account PAT because the default git action bot/token
# doesn't have the required permissions to bypass the aforementioned conditions.
# To bypass those conditions, "Do not allow bypassing the above settings" must be unchecked
# in the repo > Settings > Branches > main .
#
# In the case of a personal repo, the repo owner will have to give a PAT of its own account
# as there is no other "admin" possible for the repo, only "collaborators."
#
# In the case of an organization repo, an other account can be used. If the org have a free plan, that
# account will have to be given the "admin" role. If the org have a paid plan, a more restrictive
# "custom role" could be forged and assigned to that user.
#
# In any case, 3 secrets will have to be added to the repo > Settings > Secrets > Actions
# - BOT_USERNAME : the GitHub username of the "power account"
# - BOT_EMAIL : the GitHub email addreess of the "power account"
# - BOT_PAT : the personal access toek of the "power account" with enough permissions associated
#
# For now, `dotnet-version` doesn't seem to support signing commits/tags but, in the future
# `BOT_USERNAME` and `BOT_EMAIL` could be replaced by `BOT_GPG_PRIVATE_KEY` instead.
# In that case, when generating the GPG key, use the bot username and email when asked
# for a name and email (https://docs.github.com/en/authentication/managing-commit-signature-verification/generating-a-new-gpg-key)
# After exporting the public key and associating it to the GH account, export the private key using (and set the repo's secret):
# `gpg --armor --export-secret-key<EMAIL_ADDRESS># where<EMAIL_ADDRESS>is the email configured in the previous step. (you might need to add `-w0` to the command).

name: Auto versioning .NET

on:
  push:
    branches: [ main ]

jobs:
  build:
    runs-on: ubuntu-18.04
    steps:
      - name: Checkout
        uses: actions/checkout@v3
        with:
          fetch-depth: 0
          token: ${{ secrets.BOT_PAT }}

      - name: Reset Origin
        run: |
          git remote set-url origin "https://${{ secrets.BOT_USERNAME }}:${{ secrets.BOT_PAT<EMAIL_ADDRESS>github.repository }}.git"
          git checkout ${{ github.ref_name }}

      - name: Install Versioning.NET
        run: |
          dotnet tool install --global Versioning.NET

      - name: Increment Version
        run: |
          dotnet-version increment-version-with-git -g "." --branch-name ${{ github.ref_name }} --author-email ${{ secrets.BOT_EMAIL }}

      # The following could be used to sign the commit/tag but `dotnet-version` doesn't seem to sign the said commit.
      # It could also be used to infer the bot user name (${{ steps.import-gpg.outputs.name }}) and
      # email address (${{ steps.import-gpg.outputs.email }}) from the private key ${{ secrets.BOT_GPG_PRIVATE_KEY }}.
      # For now, instead of the private key, two secrets will be used, ${{ secrets.BOT_USERNAME }} and ${{ secrets.BOT_EMAIL }}
      # In the future, if `dotnet-version` support signing commits/tags, it would be better to use the following.

      # - name: Import GPG key
      #   id: import-gpg
      #   uses: crazy-max/ghaction-import-gpg@v4
      #   with:
      #     gpg_private_key: ${{ secrets.BOT_GPG_PRIVATE_KEY }}
      #     git_config_global: true
      #     git_user_signingkey: true
      #     git_commit_gpgsign: true
      #     git_tag_gpgsign: true

      # - name: Reset Origin
      #   run: |
      #     git remote set-url origin "https://${{ steps.import-gpg.outputs.name }}:${{ secrets.BOT_PAT<EMAIL_ADDRESS>github.repository }}.git"
      #     git checkout ${{ github.ref_name }}
      ##  + git config user/email/signkey ???

      # - name: Install Versioning.NET
      #   run: |
      #     dotnet tool install --global Versioning.NET

      # - name: Increment Version
      #   run: |
      #     dotnet-version increment-version-with-git -g "." --branch-name ${{ github.ref_name }} --author-email ${{ steps.import-gpg.outputs.email }}

@cbcrouse I still didn't figure out why I wasn't able to sign the commit/tag after importing the GPG key and enabling it "globally". If you could investigate, that would be great.

@JBBianchi Sorry to get back to you so late on this, I have been quite busy. It looks like there may be a couple reasons for this not working.

It's possible that the author mail needs to be associated with the GPG key based on this documentation.
It also looks like signing commits must be enabled in the git config and then -S must be supplied with the git commit command to actually sign the commit.

I will work on getting commit signing supported which will start showing up as Verified in the commit history for validation.

I will also write up some documentation in the Wiki regarding signing after I do the investigation and get it working so you'll know exactly what you need to do.

This will be nice once GitHub supports something like what is suggested here:
https://github.com/actions/runner/issues/667

I had to work around, attach my personal repo to an organisation, set the bot as admin and allow the admins to bypass the protected branch restrictions (signed commits & approved PRs only).

Signing commits appears to not be supported by LibGit2Sharp in any convenient way. Someone did add a contribution that shows how it can read a signed commit, but no explicit instruction on how to utilize those APIs within the library to create a GPG signed commit. Unfortunately, this may take some time to support, so you will have to continue using your workaround.


Suggestion for dark theme

I think it would be good to put more contrast in the presentation of the result grid. The presentation is not cool on the dark theme.

Actually it was a bug in results background color definition for dark theme.


support deletion of a single AP

add "Delete AP" button.
Maybe clicking red circle can be used as collision detection?

introduced in 62516a53f705f1ba502b03d30f90804a9f6445d7


Generate critical css only when ELEVENTY_ENV is for production

Hi There,
I am new to Eleventy, and luckily I found this plugin. May I know how to run this plugin only when the environment is for production? I have this under _data/env.js
module.exports = {
  environment: process.env.ELEVENTY_ENV,
};

And this is my npm script
"build": "ELEVENTY_ENV=prod run-s build:* postcss",

I only want to run this plugin when ELEVENTY_ENV is set to prod.
Thank you.

Wherever you add the eleventy-critical-css plugin, you can check the environment variable ELEVENTY_ENV that you have set in your build script, like so:
module.exports = function (eleventyConfig) {
  if (process.env.ELEVENTY_ENV === "prod") {
    eleventyConfig.addPlugin(require("eleventy-critical-css"));
  }
};

Wherever you add the eleventy-critical-css plugin, you can check the environment variable ELEVENTY_ENV that you have set in your build script, like so:
module.exports = function (eleventyConfig) {
  if (process.env.ELEVENTY_ENV === "prod") {
    eleventyConfig.addPlugin(require("eleventy-critical-css"));
  }
};

Hi, yes I was figured it out yesterday. But thanks for your answer

Hi, yes I was figured it out yesterday. But thanks for your answer


make SeekArc usable in ScrollView

Added code to stop ScrollView from stealing the MotionEvent.

This change has been merged into v1.1, thanks for your contribution.


Improve the text at the top of the page

Received the latest home page text from the product team

 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.:x: WolffJosephN sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Unable to install on Pharo 9

Using the instructions, I get a warning that a package depends on OCClassScope, not sure if this breaks something or not, just wanted to check.

We are working on it! I hope to commit a working version later today (just one test is failing...)

OK, wonderful. All of this is just awesome, by the way.

Sadly it is hard to find the time that is needed to push it forward...

We have a PR that should make it loadable (with green tests): https://github.com/pharo-ide/Seamless/pull/47
But: the tools need to now be ported to use it. Which means this, in itself, is not that useful.

OK, that sounds good. I'll see if I can help out a bit over the summer, when I have more time myself. Which parts do you think would be the best to start on if I wanted to contribute?

On our TODO next is to work on the playground.
Maybe the easiest next is the inspector: this is I think mostly the special inspector, that is look at what we need to do to have
SeamlessProxy>>#gtInspectorRawIn: for the newtools inspector.


Expected release date

Is there any information on when this library will be released? We would like to integrate it into our iOS app.

hey @robingenz, this should be good to use right now. what blocker are you running into?

I need to set a version at from when I include the package. I don't have much experience with SPM (only with cocoapods), but doesn't a git tag need to be created for this?
.package(url: "https://github.com/VapiAI/ios", from: "<VERSION>"),

There is also no GitHub release in this repository.

does this help: https://github.com/VapiAI/ios?tab=readme-ov-file#installation?
if not, throw claude or o1 on it?

I have already looked at the installation guide. As I said, I have to set a version when I include the package. This is the error message from XCode:

@nikhilro My client would really like to use your SDK. Are there any updates here? The installation guide seems to be outdated (see Xcode error message).

It seems as if the documentation has been updated.

perfect, sorry i forgot to update here


help

    _ __           __

___ () /________  / /
/ _ `/ / / / _ / _
_, //_/_/  _/_./
/___/ by @michenriksen
gitrob v2.0.0-beta started at 2018-07-16T16:03:54+08:00
Loaded 91 signatures
Web interface available at http://<IP_ADDRESS>:9393
Gathering targets...
panic: sync: negative WaitGroup counter
goroutine 1 [running]:
sync.(*WaitGroup).Add(0xc42042b700, 0xffffffffffffffff)
/usr/local/go/src/sync/waitgroup.go:73 +0x133
main.AnalyzeRepositories(0xc420138800)
/Users/michael/go/src/github.com/michenriksen/gitrob/main.go:101 +0xd4
main.main()
/Users/michael/go/src/github.com/michenriksen/gitrob/main.go:228 +0x42c

[root@localhost opt]# go version
go version go1.9.4 linux/amd64

happens on go 1.10.3 as well

@derricksong @ykosaraju @hanjian007 Can you please pull the latest master from this fork and confirm if the issue still exists when the correct token is passed in either by env variable or the flag. This project is no longer maintained so any further issues or bugs should be opened up in that new fork. Thanks!


Open Plugin in  OpenSim 4.4 (Windows Version)

Hi!
I want to open the following model in OpenSim 4.4:
https://simtk.org/projects/kuo-head-neck
I face the following error:
Object::newInstanceOfType(): object type '{DynamicLigamentForce}' is not a registered Object! It will be ignored.
I learned that I must create the plugin and add it to my OpenSim. There is a file named "osimDynamicLigamentPlugin.dll," so I think my plugin is already ready (am I right?)
So my problem is how to add this plugin to OpenSim 4.4.
I tried to do the following link:
https://simtk-confluence.stanford.edu:8443/display/OpenSim/Creating+Plugins
But I think it is not validated for OpenSim 4.4.
I appreciate your help!
Thanks
Nima

Using OpenSim 3.3 (the appropriate one with the available .dll) solved the problem.
Thanks


Error in datageneration.py

I get this error and can't start the generation the version I'm using is 0.8.2
Traceback (most recent call last):
File "datageneration.py", line 521, in
main()
File "datageneration.py", line 512, in main
game.execute()
File "datageneration.py", line 152, in execute
reset = self._on_loop()
File "datageneration.py", line 190, in _on_loop
measurements, sensor_data = self.client.read_data()
File "/media/aakram98/New Volume/CARLA_0.8.2/PythonClient/carla/client.py", line 128, in read_data
return pb_message, dict(x for x in self._read_sensor_data())
File "/media/aakram98/New Volume/CARLA_0.8.2/PythonClient/carla/client.py", line 128, in
return pb_message, dict(x for x in self._read_sensor_data())
RuntimeError: generator raised StopIteration

Hey! Apologies for the late response. Maybe you have solved this issue by now, but this looks to be an issue with your configuration of the simulator, not the datageneration itself.
See here for more details :)

@Ozzyz The problem was in python version indeed, I've downgraded it to 3.6 and it worked

Great! Happy to help : )


MSBuild integration uses targets that don’t exist

Describe the bug
vcpkg.targets has the AppLocalFromInstalled target run before the “CopyFilesToOutputDirectory” target. This target no longer exists, and it took me a bit of time to track down the reason why my vcpkg dependencies weren’t being copied between output directories properly.
To make it work, I changed the target to depend on the BuiltProjectOutputGroup target, and to add the vcpkg binaries into the BuiltProjectOutputGroupOutput item.
Environment

OS: Windows 11 build 22621.1413
Compiler: MSVC 19.35.32216.1
Visual Studio 17.5.3

@wjk Could you please provide me with the modified target file you mentioned?

Please find attached. Thanks!

Keep active

Keep active

I have essentially the same issue. A mixed C++/C# project where the C++ library has vcpkg dependencies, and a C# project that depends on the C++ library. Unfortunately, the vcpkg DLLs are not copied across correctly to the C# output directory.
I previously worked around this by using a common output directory, but unfortunately this does not work for multi-targeted for iOS/Catalyst builds since Xamarin iOS does not support absolute output directories.:
https://github.com/xamarin/xamarin-macios/issues/18997
I've been able to get part way. First by getting the C++ DLL copied to the C# library output path like so:
  <ItemGroup>
    <ProjectReference Include="..\Dll1\Dll1.vcxproj">
      <ReferenceOutputAssembly>true</ReferenceOutputAssembly>
      <OutputItemType>Content</OutputItemType>
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
    </ProjectReference>
  </ItemGroup>

I tried a similar method to the one presented by @wjk but by attempting to fix up the built project output group in the C++ DLL vcxproj rather than modifying vcpkg:
	<Target Name="VcpkgDependencies" AfterTargets="CopyFilesToOutputDirectory" BeforeTargets="BuiltProjectOutputGroup;RegisterOutput">
		<ItemGroup>
			<BuiltProjectOutputGroupOutput Include="@(VcpkgAppLocalDLLs)" />
		</ItemGroup>
		<Message Text="Adding vcpkg Dependencies @(VcpkgAppLocalDLLs)" Importance="Normal" />
	</Target>

The message shows the vcpkg DLLs were picked up OK, but aren't being transitively copied to the C# output path.
I can do other approaches to obtain the desired behaviour, but it feels like this should just be working out of the box already.
@Cheney-W Do you have any suggestions/recommendations on how to handle this situation?


feat(turbopack-ecmascript): add sourceURL to chunk items for better stack traces

Description
This also adds some indentation and a leading comment to chunk items.

doesn't work without putting everything into an eval unfortunately


[chrome-web-store] Add tests, update labels

Update 'downloads'->'users' as label,
added /users/ as option,
kept /d/ as an option for backwards compatibility,
moved example from 'Downloads'->'Misc' section,
Removed 'total' from badge value (keep it consistent with amo badges).
example old badge:
example new badge:
Closes #1016

All looks good. Thanks!


"Săucești" commune is listed in the drop-down menu

Description:
The commune is listed in the drop-down menu according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Type "Săucești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Săucești, Bacău"


Provide clarification on the field component_name

Provide clarification on the field component_name.

LGTM.  I would add another eg for clarity; eg: component_name: chassis0/linecard0

LGTM. I would add another eg for clarity; eg: component_name: chassis0/linecard0

Added another example as suggested

Can you elaborate a bit on how it is supposed to be used / what is the purpose of the field? In what cases, for example, linecard0 component is used?
I would imagine that in most cases, accounting records are generated by control modules (in chassis-based systems) or a control module equivalent in a fixed system. Does it mean that the chassis returns the name of the active control module (at the time when the record was created)?

Yeah, it is more or less clear with the control modules.
What would constitute an accounting event on a line card? Is it for systems that allow users to directly connect to each line card (e.g. the linecard has a management IP, etc)?

Some systems have redundant routing engines, multiple chassis with their own routing engines/control cards, and some have a linecards/cards in general that have a cli accessible in-line or from a serial port.  It is reasonable to expect that these cards might log/account these activites.  This field allows the origin or the records to be identified, which I would expect to be platform/vendor specific.  eg: (WAGs)
fpc0
chassis1/fpc0
re0
re1
chassis1/re1
0/RP0/CPU0


[DOC] Update shapelet notebook

Describe the issue linked to the documentation
we have more shapelet based alogorithms and no references on the classification/shapelet_based notebook
Suggest a potential alternative/fix
No response

I wanted to do a notebook about RDST shapelet interpretation (#973), would it be a good fit in this notebook or should this go in a separate one ?

I'd put it in this one, so we can profile them together. I will end each category with a comparison of performance from the redux


[T4A2][T15-A1] Anshul Aggarwal

@aanshul20
Some comments added. Please ack & close the PR after reading comments.


Update finagle-http to 20.5.0

Updates com.twitter:finagle-http from 20.4.1 to 20.5.0.
Changelog
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!

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

labels: library-update, semver-minor

Codecov Report

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

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   98.07%   98.07%
=======================================
  Files           1        1
  Lines          52       52
  Branches        2        2
=======================================
  Hits           51       51
  Misses          1        1

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 b2596d6...06d7b41. Read the comment docs.


Allow user to set their name on /settings page

Currently we pull the user's name from their GitHub profile, but not all
users will use their real name.
Allow them to set their real name, which is useful since we include it
in the exported ICS files and so would display in a calendaring
application.

Fixes #17.


fdb Segmentation fault

when i try to deloy cluster in local k8s, it's shows that fdb start error

i got this log below
Defaulted container "foundationdb" out of: foundationdb, foundationdb-kubernetes-sidecar, foundationdb-kubernetes-init (init)
sh: line 1:    11 Segmentation fault      fdbmonitor --conffile /var/dynamic-conf/fdbmonitor.conf --lockfile /var/dynamic-conf/fdbmonitor.lockfile --loggroup byconity-fdb >> /var/log/fdb-trace-logs/fdbmonitor-$(date '+%Y-%m-%d').log 2>&1

Has anyone experienced this problem ?
My env:
mac m1 max
docker mem 12G
steps:
kind create cluster --config examples/kind/kind-byconity.yaml
helm upgrade --install --create-namespace --namespace byconity -f ./examples/kind/values-kind.yaml byconity ./chart/byconity --set fdb.enabled=false
helm upgrade --install --create-namespace --namespace byconity -f ./examples/kind/values-kind.yaml byconity ./chart/byconity

Hi, it is a known FDB issue when running in docker on Apple Silicon architecture (e.g., M1). https://github.com/apple/foundationdb/issues/4111. Currently, we are also looking for FDB alternative to have more metadata backend.
@xingcici Do you have linux to run it?

apple/foundationdb#4111

sure，i will try it on linux/amd64 device. thanks !


nixos-rebuild: Print commands as they are run on verbose flag

nixos-rebuild would forward all verbose flags to nix, but not actually
increase its own verbosity.
Which would make it hard to see what’s going on without resorting to
tools like strace or adding set -x to the script.
We add a simple exec wrapper around the most “interesting” commands,
that is nix tool invocations. If any verbosity flag is given, it will
set the verbosity flag of the nixos-rebuild script all commands
wrapped into runCmd will start logging their invocation.
This is done via a logVerbose function, which can also be used to
print logging messages when in verbose mode, to aid debugging why
nixos-rebuild does what it does. A few messages are added, but
potentially we could log a lot more details.
Now, testing all of the changed code paths turns out to be rather
hard, so I am unsure how to verify that for all changed commands
behaviour stays indeed the same.

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

I can only test it on my local machine, trying to hit a few codepaths, but I don’t know if there is any testsuite that tests more of nixos-rebuild?

Based on some ideas by @aszlig I added all installer tests to the test closure, which should give us a good integration test that at least nothing broke that had an existing test.

@GrahamcOfBorg test nixos-rebuild

Hm, maybe all of these tests are a bit too much for ofborg …

I put only a single installer test, and now ofborg seems to be happy. So I’m reasonably sure nixos-rebuild still works for the most common case.

Added Github suggestions to reviewer list.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/770

Sorry for the messy force push, I removed runCmd from two places where it would only print the name of an internal function instead of a real command.

I’d merge soon if there are no counterpoints.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/verbose-flag-for-nixos-rebuild-merged/18259/1


Include information about the recovery time on the new primary replica when CT is enabled

As per the internal TFS #13093998, databases with Change Tracking enabled will be restarted upon the occurrence of a failover. The reason for the restart is to avoid  internal table syscommittab inconsistent between replicas.

Learn Build status updates of commit e8d80d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/database-engine/availability-groups/windows/replicate-track-change-data-capture-always-on-availability.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@mashamsft
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @MashaMSFT, I hope you are doing good.
Please let me know if you need any additional information regarding this PR.
Regards,
Dineu

Learn Build status updates of commit 3f51783:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/database-engine/availability-groups/windows/replicate-track-change-data-capture-always-on-availability.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


feat: add free rotate

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

new feature
add free rotate feature to enable rotate in all directions
onHover

onMove

这块和 @JujieX  沟通了，应该有更自然的旋转方式，待后续优化。


Add build-eslint-rules to npm prepare script

My expectation is that my editor should be able to function properly in the TypeScript codebase after npm install && npm run build. This ensures custom eslint rules are built as part of the ~latter~ former command to avoid getting this:

@weswigham that works, done


Added Mouse On Wheel Loader

Fixes Issue🛠
Closes #233
Description👨💻
Created a " Mouse On Wheel " Loader in the Loaders Section
Type of change📄

[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
Tested on local system and it worked as expected
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷
https://github.com/Rakesh9100/Beautiify/assets/140608790/db89ac4c-4779-4a28-a029-beb1309836c0

@Rakesh9100 I'll do it please don't close

Okay, opening but complete it asap @JiyaGupta-cs

@Rakesh9100 Done

Pls rename your folder to Mouse-On-Wheel-Loader and make the necessary changes @JiyaGupta-cs

@Rakesh9100 Done sir, I hope it resolves all the conflicts now.

Good, Merged 🥳 @JiyaGupta-cs


Selecting multiple files and using cut (ctrl+x) crashes the program.

To Reproduce
Steps to reproduce the behavior:

select all files
Click on cut or ctrl+x
See error, or not (:

Expected behavior
A clear and concise description of what you expected to happen.
Desktop (please complete the following information):

OS:  Windows 10 2004 19041.450
App version: v <IP_ADDRESS>

Additional context
I think title is self explanatory.
Log file
debug.txt

Thanks for this feedback item. I can reproduce this in both ListView and GridView modes as long as there are enough items in the folder to make scrollbars appear. 👍

This issue was fixed with #1820.


Fixed invalid links that led to a 404 page

Description

I tested all the links in the readme files and if they didn't work, I replaced them with the valid links that were meant to be used
Motivation and Context

Contribution to maintain the quality of the samples
How Has This Been Tested?

I went through the READMEs and clicked on the new links to see if they actually worked
Types of changes

[x] Improvement (non-breaking change that improves a sample)
[ ] Bug fix (fixes an issue with a current sample)
[ ] New Sample (adds a new sample)
[ ] Config and build (change in the configuration and build system, has no impact on code or features)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@dotnet-policy-service agree

The repository also contains the following links, which are not accessible:
http://docs.nanoframework.net/api/nanoFramework.GiantGecko.Adc.html
http://docs.nanoframework.net/api/nanoFramework.Hardware.GiantGecko.Utilities.html
http://docs.nanoframework.net/api/nanoFramework.System.Collection.Hashtable.html
http://docs.nanoframework.net/api/Windows.Devices.Gpio.html
http://docs.nanoframework.net/api/System.Device.UsbClient.html
I couldn't find the documentation myself, and I'm not sure why it's missing, so I couldn't replace these links.
I hope they are still valid and that you just need to update the documentation to make everything work.

@Hartmannlight Here is a sample fix for the Usb Client: https://github.com/nanoframework/nanoframework.github.io/pull/186
Note that the Windows.Decices namespace is deprecated, so it might be worth finding those links and noting as such.
The nanoFramework.System.Collection.Hashtable.html should actually be collections, so if you can track that down and fix, it would be helpful.

[ ] nanoFramework.GiantGecko.Adc
I think, you can add this to your pull request @networkfusion

[ ] nanoFramework.Hardware.GiantGecko.Utilities
It looks like it does not exist

[x] nanoFramework.System.Collection.Hashtable

[x] Windows.Devices.Gpio

[x] System.Device.UsbClient

@Hartmannlight URLs for all missing Gecko libraries have been added. Feel free to use and update as needed. 😉


List of programming language does not only contain programming languages

The list I am talking about is here
Ruby on Rails, for instance, is a framework. But ruby is a programming language.
I understand that the list is only there for the purpose of the demo, and that this is not a real issue, but I just think it would be good to fix it.

I came here to submit a PR about that. Glad I'm not the only one that noticed that.

https://github.com/LeaVerou/awesomplete/commit/718adf1d1ffb5b47589eb86bba4574a7f872424a#commitcomment-9820079

Oh well, if the author doesn't even care.. :rocket:

Please, people!
It's like discussing if 'Fiat' belongs in a list of cars when it's a brand or that a bike shed can't be colored black because black is not actually a color...

You could argue that ad naseum, but we were just trying to give a little help with consistency. It's a community for a reason. There are pull requests for a reason. There are issues for a reason.

rolls eyes This kind of OCD-like behavior also reminds me of Well, Actually.
I completely agree with @Flygenring. It’s a demo. It’s purpose is not to teach what programming languages are, but to offer varied examples of strings.
However if it bothers you that much, I wouldn’t object to a PR that changes the “Pick a programming language” label to something that also encompasses frameworks as well. Perhaps “What do you code in?” or something.

I can understand that you don't care about what is written in your example, fine, but why reject a pull request for it for those it bothers ?
You don't care that's fine but things like having ruby on rails categorized as a language is a common and annoying misconception and the fact you made it might just be a hint.
Your library looks nice anyway ;)

If the case is that pedantics ruins an example of how to use something (awesome), then I'd suggest to just change the label from "Pick a programming language" to "Pick a computer related term" or even "Pick a sample string" –

If the case is that pedantics ruins an example of how to use something (awesome), then I'd suggest to just change the label from "Pick a programming language" to "Pick a computer related term" or even "Pick a sample string from the list I chose" – just to make it a catch-all of sorts :)


Get all tests green

@1aurabrown can you paste an overview of the currently failing tests and, where you have the info, add why you think they’re failing?

For now I’m adding allotting a fair bit of time to this task, namely pretty much you and I till the end of this week. i.e. 3 days both.
[ESTIMATE] 6 days

This includes #370.

Can’t assign two people ¯_(ツ)_/¯
But I’mma let you tell me what to do on this, so there you go :smile:

@alloy: they are all some variation on this error: https://github.com/artsy/eigen/issues/377
constraints with a multiple that is zero, which is illegal. This is legit, as I've seen the sign in screen crash with this error. I am not sure why this just started happening -- probably newest iOS does something different in layout.
There is also one snapshot diff that is failing -- I can look into this too:

Re: that ^ snapshot, I'm willing to bet the fair object needs to have its maps stubbed out so that it doesn't do async

can i close this? i believe everything that's left is just flakey, not consistently failing.

:+1:


Inconsistency between express and supertest when using escaped parenthesis and single quotes in routes

express version: 4.17.1
supertest version: 4.0.2

TLDR; There seems to be some sort of inconsistency between express and supertest when handling routes with single quotes. Or I'm missing something trivial.
I have two express routes. I'm escaping parenthesis in the routes with \\ as suggested in the docs.
app.get('/api/hello-one\\(:msg\\)', (req, res) => res.send({ num: 1, msg: req.params.msg }));
app.get('/api/hello-two\\(\':msg\'\\)', (req, res) => res.send({ num: 2, msg: req.params.msg }));
I have two supertest tests (using mocha to run them):

it('should return correct response for hello-one', (done) => {
const app = createExpressApp(null);
console.log(app._router.stack);
request(app).get(`/api/hello-one('foo')`).expect(200).expect({ num: 1, msg: `'foo'`}, done);
});
it('should return correct response for hello-two', (done) => {
const app = createExpressApp(null);
console.log(app._router.stack);
request(app).get(`/api/hello-two('foo')`).expect(200).expect({ num: 2, msg: `foo`}, done);
});

The first test passes as expected.
The second test fails unexpectedly, with an actual status of 404 instead of the expected 200.
But in the browser (using Firefox),
GET http://localhost:<port>/api/hello-one(foo) and
GET http://localhost:<port>/api/hello-two('foo')
both return the correct objects as expected.
What's going on here? Any help is appreciated!

 It is an issue with supertest, ultimately (as you can see by your web browser not having issues). Sorry for any confusion around this. This mainly comes down to URL normalization, where the web browsers follow it (not encoding a single quote), while superagent (what is behind supertest) encodes too many characters, different from the URL normalization standards. You would need to use %27 in your route path to handle those types of agents, currently.

Thanks for the reply @dougwilson :)
I can confirm that changing the second route from '/api/hello-two\\(\':msg\'\\)' to '/api/hello-two\\(%27:msg%27\\)' fixes the failing supertest test, although that then breaks the browser route that used single quotes instead of the %27 encoding.
So now in the browser
GET api/hello-two(%27foo%27) sends the expected response but
GET api/hello-two('foo') 404s.
Is there a recommended best practice to handle those two requests in express without maintaining two separate routes?

Currently in Express they are treated as separate routes when it is part of the literal route. You can app.use some middleware at the top of you app to "fix" the value of req.url if you like--the used value is coming direct from the underlying Node.js http module.


migrate from winapi to windows-rs

Winapi is no longer maintained and windows-rs is the official crate from Microsoft for Rust.

Thanks for the PR! I'll find a Windows machine (someplace) and once I give it a whirl I'll pull this in.

Finally got around to testing this today. It works in a Wine prefix so I think we're good to go. Thanks for catching this!


display-managers: rely on socket to activate pulseaudio

Motivation for this change
So I can use audio like I normally do. See #44854.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Thanks, just tested, no more permission errors. :)


ORIGINAL_DST for upstream IP only, with custom port

Title: ORIGINAL_DST for upstream IP only, with custom port
Description:
It would be great to allow an ORIGINAL_DST cluster that only takes the original IP address, not the port. Example use cases:

In a mesh, I have a cluster for google.com. I want ORIGINAL_DST, not STRICT_DNS, because doing DNS lookup 2x (client and envoy) can add some overhead. However, I want the client to send requests to port 80 in plaintext, and have Envoy add TLS and direct to port 443.

Strawman API proposal:
In OriginalDstLbConfig, add upstream_port_override int. Usage like
original_dst_lb_config:
  upstream_port_override: 443

I could imagine more complex use cases (ex map 80->443, but port 8080->8080), but I don't think that level of complexity is needed, as it can be solved with two clusters, unlike the above use case.
cc @stono @lambdai

I can understand this is useful regarding the http port 80 and 443. But it doesn' make sense for tcp modulo http.
Ideally, the destination port is determined by the application layer, e.g. a http filter rewrite the original dest including ip and port.
I am think about a powerful http original dst to support X-forwarded-Port/X-Forwarded-Host.
What do you think? @howardjohn

I think this is still useful. Also I do think TCP makes sense, this is protocol agnostic. Its basically just a rewrite of port... we can do it for all cluster types except ORIGINAL_DST

@howardjohn shall we reopen the issue, to continue further discussion? @phlax would it be possible to get some help in triaging or assigning someone for the issue?

i can reopen and mark as help wanted
wrt assigning the issue - that would need to be someone who wants to take it on
it also has to be agreed that the change is desirable, and this is not really within my area/remit

i can reopen and mark as help wanted
wrt assigning the issue - that would need to be someone who wants to take it on
it also has to be agreed that the change is desirable, and this is not really within my area/remit

thanks @phlax

Would below extended field be good? There might be security concern but it's no worse than the existing use_http_header
CC @htuch
 message OriginalDstLbConfig {
    option (udpa.annotations.versioning).previous_message_type =
        "envoy.api.v2.Cluster.OriginalDstLbConfig";
    bool use_http_header = 1;

+   message ValueOverride {
+      oneof {
+         string value = 1;
+         // The value is recovered from this downstream http header name in the lbcontext.
+         string header_name = 2;
+         // potential: from filter_state path.
+       }
+   }
+
+    ValueOverride ip = 2;
+    ValueOverride port= 3;
  }

That would meet our needs for sure. We would set:
OriginalDstLbConfig {
  port: { value: 443 }
}

If it was not clear, we do not personally need header_name, although I certainly don't mind it being added
Will leave to others to answer if its good for other contexts :-)

I would suggest only including the support you need for your use case right now, but leave it in a oneof if you do anticipate future additions.

@htuch

fixed port value
This is an optimization which saves a few clusters in istio as described as the initial post.

ip from header and port from header
This is used to terminate http CONNECT request. The current HCM route can only select cluster as the upstream.
I am expecting the cluster is OriginalDst cluster.

In my proposed Config,
the below CONNECT request
:method : CONNECT
preselected-ip-header-name: "<IP_ADDRESS>"
preselected-port-header-name: "80"
:authority: a-reserved-host-to-match-envoy-route

Can be routed to cluster
type: original_dst
original_dst_lb_config: { ip: "preselected-ip-header-name", port: "preselected-port-header-name"}

This actually achieves what CONNECT is invented for.
I believe there is questions

Why not consume from :authority in CONNECT?
In our design, CONNECT can and usually wrap a HTTP request and route matching a hopreselected-ip-header-name

Why not relay the original address via 'x-envoy-original-dst-host"?
We need to transmit the ip and port on the wire which mess up the existing settings to strip "x-envoy-" headers.

Shouldn't CONNECT be able to replace wholesale the original dst information? I'm not sure I follow what you have in (1). In any case, I think it would be good for Istio folks to achieve consensus on the preferred option.

@htuch
re: Shouldn't CONNECT be able to replace wholesale the original dst information?
connect method can support host as www.example.com:443, where dns need to be involved. If we concede the original dst should cover the format of domain name, a dns resolver need to introduced in original_dst. Part of me think that's beyond what my case needs because my case guarantees ip formart. On other hands, I'd like to to see a new setting in original_dst cluster that consuming :authority.
re: (1)
part of the design is using a connect request to carry a single HTTP GET request rather than a tcp stream. In the latter case, it would be great to override the "host" header of CONNECT by the "host" of GET request. That's is more friendly to route lookup. I will revisit the design to evaluate the benefits at the cost of anti general CONNECT pattern.

The extended form of CONNECT that aims to support UDP now uses URL templates in the 8th draft, e.g. https://masque.example.org/.well-known/masque/udp/{target_host}/{target_port}/.
The authority is the proxy authority, and the original destination is templatized in the path. Given that Envoy lacks an ability to parse arbitrary path templates, there is no simple way to standardize this in Envoy yet, so it should be reserved to a custom filter.

SGTM. One thing we should be aware of though is that at some point this degenerates into just regular dynamic forward proxy. How far are we from that?

There are a few unknowns chaining with dynamic forward proxy.
DFP cluster might work if we make similar changes there and unify DFP and original_dst.
Are you aware of a discussion to replace original_dst by DFP? Should I create one?

I'm not familiar with this, but it seems like a place there might be reasonable convergence when we're using CONNECT as the basis for overriding. Vanilla original_dst is different enough I can imagine never unifying, it really depends how far you will go with the CONNECT stuff here.

There is a variety of dynamic cases where the destination IP:PORT is derived from a request header or a filter state, and when they combine with overrides, it's hard to predict the effects. I wonder if we have enough weight to add a first class API here as opposed to a custom extension that can decide how to "resolve" a remote address based on some custom logic. Maybe the right thing to do is to define what these "resolvers" are as an extension class so that they can be shared with DFP.

Fixed by https://github.com/envoyproxy/envoy/pull/23040

It seems that overriding the ORIGINAL_DST upstream port still creates 2 upstream connections, even if they both resolve to the same address post-override.
e.g. if I have two connections, A with original_dst = <IP_ADDRESS>:555 and B with original_dst = <IP_ADDRESS>:999, and they both have a upstream_port_override = 111, Envoy will still create 2 separate upstream connections despite the destination resolving to the same host/port combo.
This can be observed on HTTP/2 CONNECT tunnels where multiplexing should have merged these into a single connection.


[FTG] Fix segfault due to undefined pointers in ftg_allocate_and_read_pointer_*

I have changed the logic of allocation in the FTG interface.
In ftg_allocate_pointer_* variables are now always nullified and then allocated. ftg_allocate_and_read_pointer_*now always just calls ftg_allocate_pointer_*. Before, it first checked if the pointer variable was already associated and if so if the size matches etc. This still led to seg faults from time to time when undefined pointers were passed.
For consistency, I have also changed ftg_allocate_allocatable_* and ftg_allocate_and_read_allocatable_* in a similiar way.
I think, the new behavior that the subroutines called ftg_allocate_* always (re)allocate is perfectly fine.

launch jenkins

Thanks a lot and Happy Holidays!


Fix Connection Modal Hidden by Loading Modal

Fixes edge case where the loading modal is in front of the connect modal when the login fails, which hides the connect modal.
@rmisio this is a css fix, I'm going to merge it since it can leave the user trapped if they mistype their password. If there's an easy way to fix the js so the loading modal doesn't stack on top of the config modal and hide it, that would be ideal.

hmm... when the login fails, the server connect modal should remain open. What is causing the loading modal to open? That's the underlying issue that should be fixed. This css fix is just masking that.
The reason the loading modal comes on top of the connect modal is because it must be being opened after the connect modal.
Can you reproduce the issue?

I couldn't find what was spawning the new loading modal, but I didn't dig deeply.
To reproduce:
Go to Settings/Advanced, change your connection settings, and type the wrong password in one of your connections.
The connection modal will be in the DOM, but the loading modal will be on top of it.
It seems to get ordered correctly if you refresh the app when it's in that state.
The CSS is just a patch until we can fix the order issue, so people aren't trapped with no way to fix their login.

I should also mention, I did try calling app.loadingModal.close() when the login failed, that did fire the close method on the loading modal, but it did not remove the loading modal that was on top of the connect modal.
It looked like the loading modal was opened multiple times.

ok, i'll look into.

I'm not able to reproduce this. When I try to connect to a valid server, but with invalid creds, the Connect modal lets me know that authentication failed (no loading modal):

What am I missing?

When I did it, I had to do it in a certain sequence:

connect to a good node
edit the settings for a 2nd node that already exists, and change the password to be invalid
the login will fail and you'll be stuck on the loading screen.

Still not able to reproduce this. Unless this is something that we can reliably reproduce, I think we should get rid of the CSS bubble gum patch, since I'm not sure about what side effects it can produce and it might cause some other weird bugs. I think there's too many of these scenarios where we're just blindly throwing darts hoping we hit a target :(


Fix Day/Night cycle

Fix day night cycle and Render distance being too high

Does this fix #58?

Does this fix #58?

Yes it does


const number type throws error

this schema doesnt work
(in my other example there was also type:"number" but it had the same effect)
{
    "type": "object",
    "properties": {
        "delay": {
            "const": 10
        }
    },
    "$schema": "http://json-schema.org/draft-07/schema#"
}

i tried adding this at the start of the Javascript.ts#nameStyle method which at least fixes the runtime error
unfortunatly it is still treated as a string

protected nameStyle(original: string, upper: boolean): string {         if (typeof original === "number") {             return original         }

found it

🤔 now i just need to get out the number |

Looks like a duplicate of #2333 #2371

true , but mine has a PR linked :3


[feat] any plans to update wix v5 from wix v3?

Describe the problem
I installed WiXToolset v5 because it was written on the official site that support for this (wix v3) version would end soon, so I downloaded the latest one.
Describe the solution you'd like
I tried to integrate Tauri with WiX v5 using a PowerShell command, but it couldn't find my application (because I can't compile it). For compilation, Tauri uses the candle.exe and light.exe commands, which are no longer supported. Instead, the wix build command shoul be used.
Alternatives considered
No response
Additional context
Sorry for my bad english, I'm translated it from russian.

Yes there are plans, but it's not a priority at the moment and it's not a trivial change either, especially with wix' new distribution mode.
So yeah, don't wait for it unless you want to contribute to Tauri yourself, but we won't work on it until after the v2 stable release.

Thanks for fast answer.

Actually, let's keep this open so the request is properly tracked :)


create: 448-Find-All-Numbers-Disappeared-In-An-Array

File(s) Modified: 448-Find-All-Numbers-Disappeared-In-An-Array.cpp
Language(s) Used: c++
Submission URL: _https://leetcode.com/submissions/detail/839269141/_

File(s) Modified: 118-Pascals-Trianle.cpp
Language(s) Used: c++
Submission URL: https://leetcode.com/submissions/detail/840011258/


add multi meas initiator & non linear w tests

Altered the linear initiator to incorporate non-linear co-ordinates and changed name to simple. Added multi measurement initiator. Altered test file to test new parts.

NOTE: this request will not pass until inverse function request is approved as there are dependencies :)

Does this just support a set of M consecutive detections or does it allow for M of N?

It can indirectly driven by the deleter, which can differ from the one in the tracker itself (recently experimented with this using 3 of 5 for initialisation, but more forgiving deleter in the main tracker).


Do you have a plan to support abstract class?

I'd like to use masking feature even in abstract class (or sealed abstract class) and I know if this library only supports case class.
Do you have a plan to support it in this library?

@kenchan0130 Do you mean something like:
@customize
abstract class RootClass

case class Child(@mask field: String)

?

@kciesielski
For example,
@customize
sealed abstract class RootClass(val value: String)

case object Child1 extends RootClass("child1")
case object Child2 extends RootClass("child2")

like this.

I see, thanks. An interesting case, I'll try to look into implementing it. What kind of data do you store in the "value" field? I'm asking because I'm curious what needs to be masked from toString which is, at the same time, hardcoded in the sources.

I'm so sorry.
The preceding example was not concrete.
trait SomeLibraryTrait[T] {
	val value: T
	override def toString() = value.toString()
}

@customize
abstract class EmployeePrivateInfo(@mask val value: Int) extends SomeLibraryTrait[Int]

case object Bachelor extends EmployeePrivateInfo(1)
case object AlreadyMarried extends EmployeePrivateInfo(2)

Specifically, it is here.
Suppose that SomeLibraryTrait is an uncontrolled trait provided by libraries and so on.
Certainly, I can also create custom trait with string masking like:
trait MaskTrait[T] extends SomeLibraryTrait[T] {
	override val value: T
	override def toString() = "*****"
}

I see, but still the masked values 1 and 2 are hardcoded. What business case can be modeled with hardcoded constant values which need to be masked in toString? Are these some keys or other kinds of identifiers?

I have the case I want to mask in a complex situation.
In the case where it is important personal information when two items are combined,
I want to mask some strings
For example,
case class Employee(name: EmployeeName, privateInfo: EmployeePrivateInfo)

Of course, because of the different responsibilities, there may be some opinion that it should be marked on the upper layer like:
@customize
case class Employee(name: EmployeeName, @mask privateInfo: EmployeePrivateInfo)

However, in this case, some developers may forget to mask value because EmployeePrivateInfo is not primitive value.

Thanks for explaining. I can see how supporting abstract classes fits to this case. I will look into supporting abstract classes.