Camera2D/Camera3D Properties Per PhantomCamera Node

Project Type
2D, 3D
Feature Description
To change the properties, such as FOV on Camera3D node, you have to manually adjust the property via code, even if the individual PhantomCameranode demands different settings.
The Zoom property for Camera2D is currently supported per PhantomCamera, but it would be nice to be able to adjust all Camera2D/Camera3D properties.
Use Cases
Can be used in various cases where the camera needs to dynamically switch between FOV, H/V Offset depending on which PhantomCamera node is active.
Importance
Medium - the feature would unlock new possibilities, but it's not a showstopper
Usage
Often - a significant amount of projects can find this useful
(Optional) Proposed Solution
No response

Added FOV, Cull Maskand H/V Offset property override to PhantomCamera3D.
There might be other properties that are worthwhile to include, but am not able to identify others.
Closing the issue until there's a need for additional ones.


Can't compile LESS from bower package

I've installed both bootstrap and bootstrap-rtl from bower.
In my main.less file, I'm importing both bootstrap.less and bootstrap-rtl.less.
bootstrap-rtl won't compile, becuase it's looking for bootstrap's less files in a sibling /bootstrap directory.

I see there's a /bootstrap directory in the main /bootstrap-rtl directory, but it's empty.
I see 2 possible solutions:

Don't import bootstrap's less files from bootstrap-rtl. Instruct the user the import them in their project.
Create a variable for bootstrap's directory, so the user could override it in their project.

Just ran into this problem again in a new project.


Add test case about format "a % b"

There is a bug about format a % b caused by go interprets % as a special things. The PR fixes this and add a test case.

The bug is already fixed, so just keep test cases. PTAL @tiancaiamao .

LGTM
b.t.w, this should already been covered here https://github.com/pingcap/parser/blob/master/opcode/opcode_test.go#L33

PTAL  @XuHuaiyu


Manual empty dust bin

Hi,
First I'd like to thank you for the great plugin!
I was wondering if it would be possible to add support for the empty dust bin action - like in the image below - as another stateless switch?

Hi, Don't have such a deebot will check if possible as soon as I have some time.

The command for empty dust bin is not implemented in the library yet.
I also do not own a model with an empty station.
@nicoduj
I will inform you when the command is implemented.

Thanks guys for looking into it, really appreciate it! :)

Hey guys, any update on this one? :) any way I could help?

@rimonhanna

Hey guys, any update on this one? :)

Not yet, because I do not own such a device. Since there is no publicly available API documentation, someone who owns such a device has to find out which command is provided by the Ecovacs API for that 😉

any way I could help?

Yes, you can figure out, what command the app sends to empty the dust bin 😃
One possibility how to do this can be found here.

@rimonhanna
Meanwhile, someone has figured out the command.

@mrbungle64 That's brilliant! Looking forward to it!

stopped using homebridge and switched to using home-assistant, way more stable core and addons


CI: Update molecule scenarios

Change Summary

Create new molecule scenario eos_designs_negative_unit_tests

@gmuloc - I'm good with the change you have made, I think we can merge this if you agree!

@gmuloc - I'm good with the change you have made, I think we can merge this if you agree!

Good for me to go!


Apply dotnet format

Apply dotnet format
Description
File encoding was not uniform. Now it honors configuration from .editorconfig
Checklist

[x] The existing code style is followed
[x] The commit message follows our guidelines
[x] Performed a self-review of my code
[x] Hard-to-understand areas of my code are commented
[x] The documentation is updated (as applicable)
[x] Unit tests are added/updated
[x] Integration tests are added/updated (as applicable, especially if feature/bug depends on roslyn or framework version in use)

Related conversation: https://github.com/riok/mapperly/pull/864#discussion_r1382375517

It seems that csharpier does not enforce file encoding. Which is probably a root cause of this


What happened to minified/concatenated version?

Just curious, why the change to src only and not dist?

See #3524

Ah, thank you.


tower always sigsegv and panic

Problem:
Tower is panic and agent connect lost.

Toplogy:
Host KS-- Tower(Public Internet) <--------Agent(Internal network)---K8S/KS
Tower Log
Seemed to Recv error payload and Sig Segment Error

always output proxy error

Agent Log

Can you executing the following command
kubectl -n kubesphere-system logs [tower-pod-name]

to get the full logs the container, and paste it here ?

/ # kubectl logs tower-7668df8696-gjqql -n kubesphere-system
I1112 12:27:54.864227       1 options.go:46] CA set to "/ca.crt".
I1112 12:27:54.864290       1 options.go:47] CA key file set to "/ca.key".
I1112 12:27:54.864296       1 options.go:48] Host set to <IP_ADDRESS>
I1112 12:27:54.864300       1 options.go:49] Agent port set to 8080.
I1112 12:27:54.864304       1 options.go:50] Kubeconfig set to "".
I1112 12:27:54.864308       1 options.go:51] Leader election set to false
W1112 12:27:54.864652       1 client_config.go:541] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This mightnot work.
I1112 12:27:54.977494       1 proxy.go:234] Listening on <IP_ADDRESS>:8080...
I1112 12:27:57.068200       1 proxy.go:205] Connection established with cluster-a
I1112 12:27:57.373382       1 proxy.go:205] Connection established with cluster-b
I1112 12:27:57.973635       1 proxy.go:205] Connection established with cluster-c
I1112 12:27:58.364305       1 proxy.go:205] Connection established with cluster-d
I1112 12:28:01.065250       1 proxy.go:205] Connection established with cluster-e
I1112 12:28:03.164617       1 proxy.go:205] Connection established with cluster-f
2020/11/12 12:55:40 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:53 http: proxy error: context canceled

/ # kubectl logs tower-7668df8696-gjqql -n kubesphere-system
I1112 12:27:54.864227       1 options.go:46] CA set to "/ca.crt".
I1112 12:27:54.864290       1 options.go:47] CA key file set to "/ca.key".
I1112 12:27:54.864296       1 options.go:48] Host set to <IP_ADDRESS>
I1112 12:27:54.864300       1 options.go:49] Agent port set to 8080.
I1112 12:27:54.864304       1 options.go:50] Kubeconfig set to "".
I1112 12:27:54.864308       1 options.go:51] Leader election set to false
W1112 12:27:54.864652       1 client_config.go:541] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This mightnot work.
I1112 12:27:54.977494       1 proxy.go:234] Listening on <IP_ADDRESS>:8080...
I1112 12:27:57.068200       1 proxy.go:205] Connection established with cluster-a
I1112 12:27:57.373382       1 proxy.go:205] Connection established with cluster-b
I1112 12:27:57.973635       1 proxy.go:205] Connection established with cluster-c
I1112 12:27:58.364305       1 proxy.go:205] Connection established with cluster-d
I1112 12:28:01.065250       1 proxy.go:205] Connection established with cluster-e
I1112 12:28:03.164617       1 proxy.go:205] Connection established with cluster-f
2020/11/12 12:55:40 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:47 http: proxy error: context canceled
2020/11/12 12:55:53 http: proxy error: context canceled

Thanks for the feedback, working on it

how's the progress ?  thanx

/assign @yuswift

how's the progress ? thanx

Still working on it.

@XinxingZh Unfortunately I couldn't reproduce this error. Please leave a comment here if you can find a way to reproduce the bug.

/close


SnippetString.appendChoice() with placeholder=0 inserts snippet marked into the editor

The following code:
const a = new vs.SnippetString();
a.appendChoice(["test", "test2"], 0);
await editor.insertSnippet(a, new vs.Range(new vs.Position(0, 0), new vs.Position(0, 0)));

Inserts raw snippet markup into the document:

It's not very explicit in the docs/tooltips but I guess that the "final tabstop" does not support choices. If this is the case, it would be better if appendChoice throws if given an invalid value, or VS Code just does some default behaviour (like setting the tabstop, inserting the first choice, etc.) rather than inserting snippet markup into the document.

@jrieken I was working on this, but found a conflict with an old issue (#31599) in the tests (snippetParser.test.ts#L644):
test('Snippets: make parser ignore `${0|choice|}`, #31599', function () {
    assertTextAndMarker('${0|foo,bar|}', '${0|foo,bar|}', Text);
    assertTextAndMarker('${1|foo,bar|}', 'foo', Placeholder);
});

Anyway, I'll submit the PR to fix the current issue. Feel free to close it.

Yeah, not supporting choice on the final tabstop is on purpose. Correctly me if I am wrong but I believe that's how it works in Textmate itself and we are using that as reference

@jrieken However, this was not much unfortunate. While checking the tests, I encountered a somehow paralyzed assertion that didn't surface until now. I'm now submitting a PR for them.

closing because of https://github.com/microsoft/vscode/issues/150745#issuecomment-1156402023

I understand why this might not be supported in snippet text (#31599) but I'm not sure I understand why appendChoice can't be made to throw in this case. If it's invalid to do this, it seems odd to silently do something strange rather than just throwing so it's clearer to the (extension) developer what they have done wrong?

(and FWIW, you didn't actually close the issue if you intended to :-) )


Reexport all core modules under piston::*

This is used for global import.

I had a better idea: Since the top level is not used, we can reexport everything there.


[Executive Perms] /ignoreclaims in admin claims.

Executives are unable to ignore admin claims on titan but are able to do so in both orion/atlas

Be [Executive] rank
Teleport to admin claims on titan
use command /ignoreclaims
Attempt Edit
Repeat on Atlas/Orion
See error

For Executives to be able to ignore admin claims to make important fixes.
Titan

Elryi fixed it poggers.


can we add asset image as marker on map?

i have checked examples. but there was one example which supports weblink of image for marker.
can we add asset image as well? if so, how can we add it?
Thanks

Hello. I apologize for the delay in my response.
The library I've created currently loads HTML directly on top of a Flutter webview. However, this approach doesn't support adding local asset images easily.
To make it possible, you might consider using the flutter_inappwebview library in your development project to set up a local server and add assets. This approach would allow you to work with local asset images more effectively.
Thank you for your understanding.


Publishing doesn't work

The request ended up with error:
{
  "status": "Form Content-Type not supported"
}

https://github.com/w3c/gamepad/runs/253545997#step:3:115

Closed via https://github.com/sidvishnoi/respec-w3c-auto-publish/commit/9531a09062bb07095fc91880c6e8678f31e63eb9


Add SGX webhook operator as dependency of sgx-nfd

I am deploying sgx-device-plugin on Azure SGX VMs.
According to the SGX device plugin doc:

There is a kustomization for deploying everything:
$ kubectl apply -k ${INTEL_DEVICE_PLUGINS_SRC}/deployments/sgx_plugin/overlays/epc-nfd/

However this does not start the device operator by default. Therefore /dev/sgx_enclave will not be mapped into the containers when resource sgx.intel.com/epc is requested.
Adding operator/default initiates the webhook and fixes the issue.
Although I still have to start the cert-manager by running
$ kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.3.1/cert-manager.yaml
I am not sure if adding this as dependency makes sense.

I am deploying sgx-device-plugin on Azure SGX VMs.

I have understood that if you install the azure confidential computing add-on, you get all the necessary kubernetes pieces automatically. Is that not the case?

I assumed you meant the addon for Azure Kubernetes Service?
We are deploying directly on a VM with a self-managed kubernetes cluster, the add-on does not seem to be available.
This issue does not seem to be specific to Azure VMs though.

@0x161e-swei OK, thanks for the clarification.
Would it be possible for you to move to the operator based deployment model:

install cert-manager
install nfd
install deviceplugin operator
deploy SgxDevicePlugin

That's the preferred mode we were thinking. We recently added a separate SGX admissionwebhook image too to help with the non-operator flows. For this PR, it might be better to set the dependency to that instead of the operator. (NB: the image is not yet published but it will be shortly).

@0x161e-swei what do you think about the operator based deployment option? I'm fine with this approach too but instead of deploying the operator, let's deploy sgx_admissionwebhook

Hi @mythi , sorry for getting back late here, was distracted by some admin work.
I do not have a strong opinion against the operator based method.
From a user point of view (especially for those just getting started), a single click or a oneliner greatly smoothens the learning curve.
How would you like to proceed with this PR, shall we wait for the image to be published?

can you also add a comment to SGX README that the "first approach" depends on cert-manager deployment.
By the "first approach", do you mean the "DaemonSet" approach? The README doc reads to me as the "pre-built operator image" is the first approach?

By the "first approach", do you mean the "DaemonSet" approach?

Yes. That approach deploys what this PR changes and it won't work without having cert-manager deployed too.

@mythi Ok I have modified the DaemonSet dependency and edited the SGX README for the cert-manager dependency.
Commits can be squashed too if needed.

Codecov Report

Merging #659 (ef57064) into main (777f955) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #659   +/-   ##
=======================================
  Coverage   59.07%   59.07%
=======================================
  Files          32       32
  Lines        2375     2375
=======================================
  Hits         1403     1403
  Misses        890      890
  Partials       82       82

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 777f955...ef57064. Read the comment docs.

Hi @mythi, could you review again to see if this PR needs further edits?

@0x161e-swei I was out on vacation, sorry. Can you squash the three commits to one. Otherwise, LGTM

@mythi Done. Thanks.


Implementing the Make a Plan Page

It's not done, but this is a good start. Also includes a fix for the bottom navigation covering the content.

@codingbandit
I was just watching a recording of the hackathon stream with @shawnwildermuth. I couldn't find the code he was working on in the repo. Was there a reason why this wasn't merged? Did I miss something?


Two of the issues reported on free5gcstage3 are not fully merged

For the two issues I had reported

https://github.com/free5gc/free5gc-stage-3/issues/4 : Readme points to incorrect location for the git clone #4
https://github.com/free5gc/free5gc-stage-3/issues/3 : Build failure seen in upf while building libgtp5gnl  - The build failure is fixed but the change done in PR#https://github.com/free5gc/free5gc-stage-3/pull/5 for the testgtpv1 is not merged.

Hi,
For free5gc/free5gc-stage-3#4, because we turn out to use free5gc/free5gc as our only repo, we have fix the git clone path issue in free5gc README

For PR#free5gc/free5gc-stage3#5, we are thinking about waiting for @tf-enya to help us re-open the PR here. Due to the different git history, it seems that we cannot just format-patch the PR here.
Or if there is any good idea or comment is welcomed!


Uncaught ValueError using imputer on grub_damage

AutoBazaar version: 0774d4c
Python version: 3.6.9
OS: Darwin-18.7.0-x86_64-i386-64bit

Uncaught ValueError when running search on sample dataset.
$ abz search -b 3 LL0_1026_grub_damage
Using TensorFlow backend.
Processing Datasets: ['LL0_1026_grub_damage']
###################
#### Searching ####
###################
WARNING: Logging before flag parsing goes to stderr.
E0826 17:46:01.013370<PHONE_NUMBER> mlpipeline.py:223] Exception caught fitting MLBlock sklearn.preprocessing.Imputer#1
Traceback (most recent call last):
  File "/usr/local/miniconda3/envs/autobazaar/lib/python3.6/site-packages/mlblocks/mlpipeline.py", line 221, in fit
    block.fit(**fit_args)
  File "/usr/local/miniconda3/envs/autobazaar/lib/python3.6/site-packages/mlblocks/mlblock.py", line 246, in fit
    getattr(self.instance, self.fit_method)(**fit_args)
  File "/usr/local/miniconda3/envs/autobazaar/lib/python3.6/site-packages/sklearn/preprocessing/imputation.py", line 158, in fit
    force_all_finite=False)
  File "/usr/local/miniconda3/envs/autobazaar/lib/python3.6/site-packages/sklearn/utils/validation.py", line 527, in check_array
    array = np.asarray(array, dtype=dtype, order=order)
  File "/usr/local/miniconda3/envs/autobazaar/lib/python3.6/site-packages/numpy/core/numeric.py", line 501, in asarray
    return array(a, dtype, copy=False, order=order)
ValueError: could not convert string to float: '6f'

Thanks for reporting this @micahjsmith
However, this is an expected error message which has no further consequences than just that, the reporting.
This happens because the encode argument from Featuretools is a tunable hyperparameter, and when it is False and the original dataset has strings, the next primitive fails.
And the traceback is the default MLBlocks logging, which reports the primitive exceptions.


CLI epub Table of Content suggestions

Remove

Table Of Contents -

Chapter 1

Update with (show 25-30 letter from tweets)

This is also applicable for
Of course, you can turn a

Out of my research on epub-gen's doc, I could only add chapters to Table of contents by adding chapters.
Then I rendered tweet by tweet and created a seperate chaptered epub. But because of the page breaks I think it is a bit annoying.

Page breaks for each tweets doesn't seem good.
May be remove the contents
And add little text to chapter 1

If works, I'll add a custom contents page and add anchor links to navigate.

I see that epub-gen is also using anchor links, but from a reason I yet don't understand, the ids that I set to elements won't work. I created an issue to epub-gen's repository hope someone will help

@johnjacobkenny for you to help when you get a chance

with the PR #787 this issue is no longer needed I suppose.


Updated jace so that jace-core-runtime and AWT are no longer needed.

Coverage decreased (-0.01%) to 62.61% when pulling d45a0a867650345f8c7cb12cd69ea945d028e3e2 on jace-update into b7a6de214a6653c6277ffc93c3d87359f7a00c2e on master.


broken server side

https://pastebin.com/mB55LERt

Patched With Multiloader Rewrite, Should Hopefully Be Available Soon :)


[WIP] VI & ADVI improvements

Overview
This PR aims to improve VI, in particular ADVI, by adapting the new Bijectors.jl interface together with Stacked<:Bijector from this PR, introducing a new optimiser DecayedADAGrad, and general improvements. See the full list below.
Worth noting that there are things to do to improve this further, but as we will soon separate out the VI parts of Turing into AdvancedVI.jl, I think this is a good-enough impl for Turing and then we can improve further as we make the separation.
Changes

meanfield(model::Model) which returns a mean-field approx of the model which in turn can be passed to vi(model, alg, q). This accommodates future plans of allowing "arbitrary" variational posterior q.
This branch doesn't perform assignments in the same way we did before, so now we can do TrackedArray instead of Array{TrackedReal} (with the help of a bit of behavior-piracy from Tracker.jl to make it work with SubArray)
Added DecayedADAGrad which is currently what's in use in Stan and suggested in [2]. TruncatedADAGrad is an impl of what they proposed in their paper on ADVI, but looking at the source code it seems they've moved away from this.[1] DecayedADAGrad seems to do better for more complex models, e.g. MNIST example it seems like choosing the truncation length for TruncatedADAGrad affects the convergence quite a bit (+ it's much more memory-intensive than DecayedADAGrad). TruncatedADAGrad works very nicely for smaller models though. Now we can use both :)
Using softplus instead of exp when enforcing positiveness of variances of TuringDiagNormal which prevents a numerical errors I've been running into with exp in certain cases. There was a brief discussion about exp vs. softplus on our Slack, and the consensus seemed to be that there's no clear winner in general. Nonetheless I think it's better to go for the most numerically stable one given that we've already run into numerical issues with exp. Also worth noting that in [Section 3.3, 2] they actually observe better softplus end results than with  exp.
setadbackend now also sets the ADBackend of Bijectors.jl, which was introduced in the interface PR.
MeanField has been fully replaced by TransformedDistribution{<:TuringDiagNormal, <:Stacked}. This results in several improvements:

Flexibility. With this change, the only thing part of VI / ADVI that is hard-coded is the handling of the parameters of variational posterior q. This we do because we need to enforce the positive-constraint for variance of TuringDiagNormal. As soon as we have a way of dealing with constraints on the parameters of the distributions themselves (not the support) in a more generic manner, we'll be able to plug in any q with the correct dimensionality and run ADVI using this instead of a TuringDiagNormal as a base distribution. Even now, if you have a custom base distribution, e.g. normalizing flow, we can implement vi(model::Model, alg::ADVI, q::TransformedDistribution{<:NormalizingFlow}) and similarily for (elbo::ELBO)(...) and we're good. With current advi we can't do VI on LDA because of singular Jacobians when using AD. When this PR with analytical expressions for jacobian of SimplexBijector is merged in Bijectors.jl, we'll able to do ADVI on LDA without any changes to the Turing-side of things.
Type-stability. Though my testing hasn't been extensive, from what I can tell, everything but Variational.meanfield(model::Model) is now type-stable (same as for samplers, etc. in current Turing.jl). The optimize call inside vi(model, advi) works as a function-barrier, thus everything other than the first line of vi(model, advi) is type-stable.
Performance. All of the above gives us a perf-increase. Moreover, since everything involving the actual transformations, etc. is now outsourced to Bijectors.jl it's much easier to optimize performance in the future + gives us analytical expressions for logabsdetjac for all standard distributions. Should also benefit from eventual support for batch computation in Bijectors.jl (see this issue.
Less code. The entire src/variational/distributions.jl is now redundant and so has been removed.

Some extra cleanups of redundant code & comments left over from the previous ADVI PR.
~Added weight argument to (elbo::ELBO)(...). This can be used to weight the  𝔼_q[log p(x, z)] term of the ELBO. This is necessary for stuff like stochastic variational inference (SVI) (see SVI PR), more complex initialization, e.g. Eq. (20) in [3].~ This weight argument is now baked into make_logjoint which makes more sense.
Now vi have the following definitions:

vi(model, alg): uses default variational posterior q which is alg dependent (in case of ADVI this is a mean-field approx with unconstrained-to-constrained Bijector)
vi(model, alg, q, θ_init): allows the user to pass in an arbitrary variational posterior q as long as Variational.update(d::typeof(q), θ::AbstractArray)::typeof(q) is defined.
vi(model, alg, getq, θ_init): don't need to pass in an initial variational posterior  q but instead pass a function getq(θ::AbstractArray)::VariationalPosterior, i.e. maps from params to distribution.
In all of the above, we can also replace model with logπ(z::AbstractArray)::Real which  represents the map z → log p(x, z), where x are the observations, making this applicable even for non-Turing models (will prob. be more useful when we separate out VI into AdvancedVI.jl).

Examples
ELBO implementation
Old
# Everything is hard-coded for mean-field Gaussian
elbo_acc = 0.0

for i = 1:num_samples
    # iterate through priors, sample and update
    idx = 0
    z = zeros(T, num_params)

    for sym ∈ keys(varinfo.metadata)
        md = varinfo.metadata[sym]

        for i = 1:size(md.dists, 1)
            prior = md.dists[i]
            r = md.ranges[i] .+ idx

            # mean-field params for this set of model params
            μ_i = μ[r]
            ω_i = ω[r]

            # obtain samples from mean-field posterior approximation
            η = randn(length(μ_i))
            ζ = center_diag_gaussian_inv(η, μ_i, exp.(ω_i))

            # inverse-transform back to domain of original priro
            z[r] .= invlink(prior, ζ)

            # add the log-det-jacobian of inverse transform;
            # `logabsdet` returns `(log(abs(det(M))), sign(det(M)))` so return first entry
            # add `eps` to ensure SingularException does not occurr in `logabsdet`
            elbo_acc += logabsdet(jac_inv_transform(prior, ζ) .+ eps(T))[1] / num_samples
        end

        idx += md.ranges[end][end]
    end

    # compute log density
    varinfo = VarInfo(varinfo, SampleFromUniform(), z)
    model(varinfo)
    elbo_acc += varinfo.logp / num_samples
end

# add the term for the entropy of the variational posterior
variational_posterior_entropy = sum(ω)
elbo_acc += variational_posterior_entropy

return elbo_acc

New
function (elbo::ELBO)(
    rng::AbstractRNG,
    alg::ADVI,
    q::VariationalPosterior,
    logπ,  # ← arbitrary callable z → log p(x, z)
    num_samples
)
    _, z, logjac, _ = forward(rng, q)
    res = (logπ(z) + logjac) / num_samples

    res += entropy(q.dist)

    for i = 2:num_samples
        _, z, logjac, _ = forward(rng, q)
        res += (logπ(z) + logjac) / num_samples
    end

    return res
end

with
function make_logjoint(model; weight = 1.0)
    # setup
    ctx = DynamicPPL.MiniBatchContext(
        DynamicPPL.DefaultContext(),
        weight
    )
    varinfo = Turing.VarInfo(model, ctx)

    function logπ(z)
        varinfo = VarInfo(varinfo, SampleFromUniform(), z)
        model(varinfo)

        return varinfo.logp
    end

    return logπ
end

to make this work nicely with Turing.Model.
TODOs

[x] Wait for Stacked<:Bijector PR has been merged so we don't have to depend on github branch.
[x] Improve docstrings for the following:

TruncatedADAGrad
DecayedADAGrad
ELBO and (elbo::ELBO)(...)

[x] Tests for

DecayedADAGrad
TruncatedADAGrad

References
[1] Kucukelbir, A., Ranganath, R., Gelman, A., & Blei, D. M., Automatic variational inference in stan, CoRR, (),  (2015).
[2] Kucukelbir, A., Tran, D., Ranganath, R., Gelman, A., & Blei, D. M., Automatic Differentiation Variational Inference, CoRR, (),  (2016).
[3] Rezende, D. J., & Mohamed, S., Variational Inference With Normalizing Flows, CoRR, (),  (2015).

Now that Stacked is merged into Bijectors.jl, I'll finish up this and then complete the tutorial on ADVI (which depends on this branch).

What's left to do for this PR?

What's left to do for this PR?

There's the two remaining points on the TODO above, and I'd like to get most of the Bijectors.jl PRs merged first and then look into making this work with arbitrary variational posteriors q. To make that work, we need some way of doing parameter-updates for any q.
I'll have a proper look at this soon, then I'll be able to better judge what's left to do.

@torfjelde, maybe focus on this PR for the next 2 weeks, when you got time.

It seems there are some conflicts. Can you fix them?

On it! I'll also go through it one last time before we merge :+1:

Cool. We plan to do another review tonight.

Is it ready to merge?

I will merge this in 24 hours if no objection.

Merged! Thanks for the great work from @torfjelde and everyone who helped review this PR!


Why modulating attention by w&h works?

I have some doubts on line https://github.com/IDEA-opensource/DAB-DETR/blob/main/models/DAB_DETR/transformer.py#L242 .
refHW_cond = self.ref_anchor_head(output).sigmoid() # nq, bs, 2

This line asks the model to learn absolute value of w, h from output. But NO supervision is applied. Besides, the 'output' tensor is used to learn the OFFSET of bbox (x, y, w, h).
So, I am wondering whether the model can learn width and height as expected?

The results show that our models get performance gains with the modulated operation.


Feature read indicate

This time, the refactoring will work I'm sure

Perfect!

Ship it ! All code climate is fixed


Billing System in Python

Maked a New Billing System Software Using Python.
Pls, accept this pull request with label of Hactoberfest.
Thanks for Considering it.

Hey, @Techiral , Pls accept This one .
Maked a New Billing System Software Using Python.
Pls, accept this pull request with label of Hactoberfest.


With the following code:
/test/route.tsx
export const Route = createFileRoute('/test')({
  validateSearch: z.object({
    status: z.enum(['one', 'two', 'three']).catch('one'),
  }),
});

/test/route.lazy.tsx
import { createLazyFileRoute } from '@tanstack/react-router';

export const Route = createLazyFileRoute('/test')({
  component: TestComponent,
});

function TestComponent() {
  const { status } = Route.useSearch();

  return
Hello from test with {status}
;
}

If I try to manually update the status query param for a value that is not within the enum set. E.g: ?status=four, validateSearch will catch the error and set it to 'one' but the route.lazy.tsx won't be rendered.
I created the following snippet. https://stackblitz.com/edit/tanstack-router-4bpt3z?file=src%2Froutes%2Ftest%2Froute.lazy.tsx
Your Example Website or App
https://tanstackrouter4bpt3z-j5cp--3001--dc4d7514.local-credentialless.webcontainer.io/test?status=one
Steps to Reproduce the Bug or Issue

Go to https://tanstackrouter4bpt3z-j5cp--3001--dc4d7514.local-credentialless.webcontainer.io/test?status=one
Change status query param from one to four
the content of the /test route won't show up.

Expected behavior
Expected behaviour would be to render the route.lazy.tsx component.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
@tanstack/react-router: 1.38.1

Additional context
No response

Can confirm I am encountering this issue in my application.
I was looking into #1656 why my loader was throwing a "context not defined" error when directly loading the URL without any search params, but it worked fine when using Link to navigate to the same URL without any search params. Disabling validateSearch stopped the error being thrown but obviously lost search param validation... This hint that "rewriting" invalid search params causes the route to not load made me check my validateSearch schema. I removed all fallback or manipulated values (e.g. forcing an undefined/non-number/negative page number value to "1") and the loader error is no longer being thrown.

this will be fixed by #1907
reproducer with the pre-release package: https://stackblitz.com/edit/tanstack-router-rcbzst?file=src%2Froutes%2Ftest%2Froute.lazy.tsx

Wonderful! Looking forward to it being released


Update printmanager.md

Removed references to Charms menu from Windows 8

@ptorr-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Have a trajectory learner example and reform the documentation build flow

This is updated version of #142 , with rebase at #156 . Ideally we need #156 to go in after which we can merge this. Opened a new PR cause I dont know why I ended up with 2 branches and started working on this one :P

I changed the base back to master to see the CD kick in and build docs. But I still dont see it.. @paquiteau what else do I need to do?

Docs are generated, see here

@alineyyy can you handle reviewing the trajectory learner example.
@paquiteau will handle the general CI / CD and have a look at the docs etc.

I will add @paquiteau to go through some "new" changes :P  @alineyyy if you already started your review, please watchout for a lot of new changes... Can you please tell if the code is understandable and if required we can add more comments for clarity


[DNM] shrink region-snapshot size to reduce memcpy

Signed-off-by: zhangjinpeng1987<EMAIL_ADDRESS>What have you changed? (mandatory)
Every time we create a RegionSnapshot, we must new or clone a Region struct, but RegionSnapshot only use Region's region_id, start_key and end_key fields. This pr replace Region with RegionInfo which only contains region_id, start_key and end_key.
What are the type of the changes? (mandatory)

Improvement (change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)

[ ] benchmark

Does this PR affect documentation (docs) or release note? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No

/release


Is g++ 4.8.X supported for building peloton ?

Build succeeds with g++ 4.9.4, but fails with  g++ 4.8.4
Building with 4.8.4 throws the following error :
/home/ubuntu/src/peloton/src/threadpool/worker.cpp: In constructor ‘peloton::threadpool::WorkerPool::WorkerPool(size_t, peloton::threadpool::TaskQueue*)’:
/home/ubuntu/src/peloton/src/threadpool/worker.cpp:56:38: error: ‘make_unique’ is not a member of ‘std’
     std::unique_ptr<Worker> worker = std::make_unique<Worker>();
                                      ^
/home/ubuntu/src/peloton/src/threadpool/worker.cpp:56:61: error: expected primary-expression before ‘>’ token
     std::unique_ptr<Worker> worker = std::make_unique<Worker>();
                                                             ^
/home/ubuntu/src/peloton/src/threadpool/worker.cpp:56:63: error: expected primary-expression before ‘)’ token
     std::unique_ptr<Worker> worker = std::make_unique<Worker>();
                                                               ^
make[2]: *** [src/CMakeFiles/peloton.dir/threadpool/worker.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [src/CMakeFiles/peloton.dir/all] Error 2
make: *** [all] Error 2

Wiki at Complier Environment states that g++ > 4.7 is supported but travis builds use g++-4.9 and g++-5

On 2017-10-30 14:17, Hemanth Malla wrote:

Build succeeds with g++ 4.9.4, but fails with g++ 4.8.4
Building with 4.8.4 throws the following error :
|/home/ubuntu/src/peloton/src/threadpool/worker.cpp: In constructor
‘peloton::threadpool::WorkerPool::WorkerPool(size_t,
peloton::threadpool::TaskQueue*)’:
/home/ubuntu/src/peloton/src/threadpool/worker.cpp:56:38: error:
‘make_unique’ is not a member of ‘std’ std::unique_ptr worker =
std::make_unique(); ^
/home/ubuntu/src/peloton/src/threadpool/worker.cpp:56:61: error:
expected primary-expression before ‘>’ token std::unique_ptr
worker = std::make_unique(); ^
/home/ubuntu/src/peloton/src/threadpool/worker.cpp:56:63: error:
expected primary-expression before ‘)’ token std::unique_ptr
worker = std::make_unique(); ^ make[2]: ***
[src/CMakeFiles/peloton.dir/threadpool/worker.cpp.o] Error 1 make[2]:
*** Waiting for unfinished jobs.... make[1]: ***
[src/CMakeFiles/peloton.dir/all] Error 2 make: *** [all] Error 2 |
Wiki at Complier Environment
https://github.com/cmu-db/peloton/wiki/Environment#compiler states
that g++ > 4.7 is supported but travis builds
https://travis-ci.org/cmu-db/peloton use g++-4.9 and g++-5

My nearly-finished Jenkins test setup agrees with this.
The situation with Travis is that although they call their build
environment "Trusty", they include versions of software that casual
users cannot install via the Ubuntu package repositories.
--
-Chad

Fix is merged in #865. Thanks to @seojungmin!


Need method to render ajax content.

What steps will reproduce the problem?
What is the expected result?
What do you get instead?
Additional info

Q
A

Version
1.0.?

PHP version

Operating system

What's the difference from regular render?

What's the difference from regular render?

See https://github.com/yiisoft/view/blob/1773b7a993ae0f7acc4c7edbcf73e46fefd6e800/src/WebView.php#L189


It doesn't work

I tried both my private IP address and the one that is offered via ZeroTier. Both of them didn't work properly. I got the following logs in the console of both the server and client.

Hello, the program isn't designed to work on a virtual machine. I suspect if you wanted to do that, you'd have to go through extra channels (since whatever is coming into the virtual machine from the network has to go through the host).
Also the screenshots you provided shows your IP address. I'm unsure whether it's your router's IP or your local IP. If it's your router's IP, I strongly suggest you hide it.

I am not using it on a Virtual Machine. I am using it on a Macbook as a Server and my PC as a Client.
Its an IP address used by ZeroTier (which is useless to other people unless they join the network I made that uses that IP address range).

The error code in the client refers to
"Software caused connection abort.
An established connection was aborted by the software in your host computer, possibly due to a data transmission time-out or protocol error."
(https://docs.microsoft.com/en-us/windows/win32/winsock/windows-sockets-error-codes-2)
I'm unsure what might have caused it. Did it immediately connect disconnect ?

No, I just clicked the Terminate/Disconnect button.

then that's probably why the program terminated.
If you clicked the terminate button, the server shutdown and therefore the client wasn't connected to the server anymore.
If you clicked the disconnect button the client disconnected from the server and therefore the server wasn't sending any inputs.
There is one issue though, and that is if you disconnect while the mouse is on the other screen, you still need to bring the mouse back to the server screen before being able to control the server computer.

I think it should use words instead of showing an error code when terminating the server/disconnecting the client.


Setting up the base frontend

Setting up the base frontend

[x] #9
[x] #8
[ ] #10
[ ] Setting up the main frame
[ ] Connection to endpoints

Done in 812d7a9e9ec1d658d89c6a63ec36b94c43c12482


More interpreter examples with memory, tables, etc.

Writing a test for the interpreter (for example using WASM generated from C++), would it be fine for you if the WASM would be embedded (self containted)  as u8-array into the rs-test-file (plus  C++ and WAST aside in commented blocks)?

Sure, if the example will provide a little bit of utility when ready
otherwise, I would call it integration test


WIP: ContinuousIntegration Debug build on Windows (do not merge!)

Aims to reproduce issue https://github.com/InsightSoftwareConsortium/ITK/issues/2661
Not intended to be merged onto the master branch!

This WIP pull request is no longer relevant, now that issue #2661 is closed, and fixed by the release of CMake 3.21.1.


Feature: Drilled Config

This PR resolves #104.
Introduces :

Config drilled from parent to child functions, making it more pure and removing dirty access of global config.
New TestConfigBuilder with sync config method and only build async method.

coverage: 62.689% (-1.9%) from 64.606%
when pulling 5a57002d4944db7d796aae8471f0f956c11de0fd on feature/drilled-config
into f84ba4101f6ca3cc58f5fd375bfa0a9b1392028c on main.


ubuntu-cluster setup does not include DNS

@mazswojejzony DNS is a pretty important part of kubernetes - it really ought to be started when the cluster comes up.

@thockin I am the maintainer of k8s deployment of ubuntu.I have already include dns deployment with v0.15.0.Once this pr is merged https://github.com/GoogleCloudPlatform/kubernetes/pull/8047, the dns should work well with v0.15.0 k8s. Since the higher k8s version need dns token to successfully deploy dns service,and as far as I know only gce/gke can use saltstack to do this now, I am working on it now.


supercharged-dhbw:1.0.0

I am submitting

[x] a new package
[ ] an update for a package

Unofficial thesis template for DHBW using the existing guidelines to achieve consistent looks.
The template contains options to configure different settings like authors or what pages to show.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] exclude PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Hey there, thank you for your submission. According to our guidelines, the package can only have the descriptive name dhbw-thesis if it is officially endorsed by DHBW. Otherwise, you will need to add a non-descriptive part (like an adjective) to the package name to allow for other implementations of the style.
If you are a full-time staff member (e.g. Professor, Academic Administrator) of DHBW or one such staff member endorses this package, please send an email to<EMAIL_ADDRESS>using your or their institutional mail to verify this. Otherwise, we will ask you to update the package name to include some non-descriptive part so we can proceed to merge this.

Hey @reknih, I updated the naming.
Just out of curiosity, is it possible to still change the naming at a later time?

Hope I now change the naming everywhere 🙏🏻

This should be it now 😅

I refactored the usage of the bib file and added font installation instructions. The changes should now be ready for merging.

Thank you!


Demo page isn't loading

Please provide a more detailed explanation. How are you trying to run the demo? I'm closing this but you can re-open if needed.


feat: you no longer need to modify your /etc/hosts to run the nexodus…

… service on your local machine.

A couple of questions:

I'm pretty sure this will break development on a machine not connected to the internet. I suppose it's still possible by adding these entries to /etc/hosts, though?

Does the development environment still work properly when accessed remotely? At a quick glance, it seems to assume the only clients are local. Does the UI work from another host? Can nexd connect from a remote host?

I'm pretty sure this will break development on a machine not connected to the internet. I suppose it's still possible by adding these entries to /etc/hosts, though?

Yep.  Stil possible.

Does the development environment still work properly when accessed remotely? At a quick glance, it seems to assume the only clients are local. Does the UI work from another host? Can nexd connect from a remote host?

The remote hosts would need to setup /etc/hosts entries.    But yes, they would.

this is part of #684


Is fp16 supported under multi-gpu ？I see that mixed_precision related code is commented in custom_optimization.py.

when i uncomment mixed_precision related code in custom_optimization.py, i get ValueError: Outputs of true_fn and false_fn must have the same type: int64, bool ..

This project does not support using FP16 under Multi-GPU mode, because LossScaleOptimizer haven't been updated for DistributionStrategy. And I can't solve this compatibility problem at the moment, even if I hacked the code.
If you solve this problem, pull requests are welcome. Thanks!
REF: https://github.com/tensorflow/tensorflow/issues/25080


1.16 support

just support it lol

I will as soon as spigot posts a major release.

Added in 2.8-SNAPSHOT. Happy coding!


Consider running Clang format over Java sources in the engine.

Currently, the Java source files in the engine are not passed over a formatter. Apparently, Clang format can handle these files just fine. We should adopt a style and run a formatting pass over all files.

cc @jason-simmons

Presumably Google has a java styleguide and clang format rule? :)
https://google.github.io/styleguide/javaguide.html
https://github.com/google/google-java-format

Hi @chinmaygarde is this issue still meant to be open?

Yes

We format java now with google3-java-format


Updates to Kubernetes scheduler to fix edge cases

Fix for #3542
Some notable changes:

Kubernetes scheduler now creates a Headless Service to pair with the StatefulSet.
Upgraded Kubernetes client from 7.0.0 to 8.0.0
Removed old compatibility code that deleted all pods. I don't see the code that used to launch individual pods, so I assume it is no longer needed. For now we only launch StatefulSets, although I do have thoughts on how to better manage the topology pods.
Heron Executor now checks for POD_NAME environment variable. If found, it assumed Kubernetes environment in which we use the fully qualified domain name which should be composed of pod name, headless service, namespace and cluster domain.

Testing environment:
namespace: heron
No additional service added mannually.
Tested with the example topology ackingTopology. I am still seeing the acking-1 complaining about:
File: metricsmgr-1.log.0
[2020-07-01 05:05:02 +0000] [INFO] org.apache.heron.common.network.HeronClient: Connecting to endpoint: acking-0:6001
[2020-07-01 05:05:02 +0000] [SEVERE] org.apache.heron.metricsmgr.sink.tmaster.TMasterSink: TMasterClient dies in thread: Thread[Thread-1117,5,main]
java.nio.channels.UnresolvedAddressException
	at java.base/sun.nio.ch.Net.checkAddress(Net.java:130)
	at java.base/sun.nio.ch.SocketChannelImpl.connect(SocketChannelImpl.java:675)
	at org.apache.heron.common.network.HeronClient.start(HeronClient.java:129)
	at org.apache.heron.metricsmgr.sink.tmaster.TMasterClient.run(TMasterClient.java:128)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:834)

and
File: container_1_exclaim1_1.log.0
[INFO] org.apache.heron.network.StreamManagerClient: Stop writing due to not yet connected to Stream Manager.

It is still trying to resolve acking-1 instead of acking-1.acking.heron.svc.cluster.local OR acking1.acking

@windhamwong What environment are you running in? I'd like to recreate.
A few questions.

Do you see the Headless service created at submit time?
Does the acking-0 Executor log not show the FQDN?
Does the acking-1 Executor log also not show the FQDN?

DrKNa in the Slack was able to test this and verified that it worked in his environment which previously did not work. But I'd like a review if the logic I added is the best option. Any suggestions before merging in?


In game gui for perms

Description

Title
Proposed behaviour

Make command like /lp promote open a chest with different ranks to allow you to easier promote and demote people in game cause rn its a bit of a pain, especially if they have a high rank cause then you have to do the command for every rank

Love the web editor btw

There is a set track command for you to set it directly a user to any rank in a track. See https://luckperms.net/wiki/Parent-Commands#lp-usergroup-usergroup-parent-settrack-track-indexgroup-context

But still a gui would be nice

But still a gui would be nice

See https://github.com/lucko/LuckPerms/issues/619#issuecomment-353216667

~3 year old comment still holds true as ever 😆


my localhost 3000 is not loading and says that GET .html

i am having a great issue in this . kindly help me anyone. i wanted to create a website and wanted to attach a new file to it. but it is not opening . kindly help me in this issue.

@TrendTrickster Hi , can you explain the issue a little bit more.


Tailwind styles not applied on gmail

Hi, thanks for your work on Tailwind, the performance has drastically improved (we went from 44s to 3-4s of rendering!)
However, if I understood correctly,  twind generate the styles in a

        Hello

If I manually edit the css, it is correctly rendered:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />

        Hello

Update:
Currently, I noticed that the two main issues are css variables (e.g. var(--tw-text-opacity)) and escaped chars in css classnames (e.g. !font-base or w-[400px]).
The second one can be mitigated using twind hash feature (using isProduction on Tailwind component) but as a temporary quick workaround I chose to manually replace the variables in the generated html (using replaceAll) and hashing classes prevent to able to do so.
So my current setup is to manually replace tailwind variables in the generated html and use juice to inline the css:
const Renderer = (props: React.PropsWithChildren<TailwindProps>) => {
    const initialHtml = useData(props, () => jsxToString(<>{props.children}</>))
    const { shimmedHtml, styleTag } = renderTwind(initialHtml, props)
    const finalHtml = juice(`${shimmedHtml}${styleTag}`, {
        xmlMode: true,
    })

    return

}

juice is interesting, I'll take a look at that as an option. out of curiosity, what does build --check return for your template? I'd like to know if it has warnings for those characters for gmail. We should probably also look at replacing css vars.

This is likely what we'll use https://github.com/tani/rehype-inline-css since juice uses cheerio and we've moved onto rehype/unified. I'm thinking that we provide an option to inline the css, since it's a more expensive operation.

@xsyann I think that jsx-email@latest fixes both of the concerns here. production: true compiles the multiple classes into one with compliant characters, and we're automatically replacing css variables now.

I'm using the new unified jsx-email package, but still getting the same problem

@merunga could please you open a new issue with a reproduction for what you're seeing?
This one is about a slightly different topic

Going to close this one for now. If anyone finds new problems, please open a new issue with a reproduction and we'll take a look quickly.


Add Option for disable observer clocks

Allow to disable observer clocks: https://www.spigotmc.org/attachments/observerclock-gif.365936/

Build is broken because repo of Aikar has error 503

Close because this patch was for 1.17


maggioli bio and publications

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
05c31273048d257a900b7654e2c5e2df672f4428

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/63f8c7098de8d00008bec16b


Inclusion of scalar data type to arthimatic nodes

Description

[x] I've included a concise description of what each node does

Styleguide

[x] My node adheres to the styleguide for Flojoy nodes

Docs

[x] I've submitted a PR for a documentation page for the new node(s) that contains usage examples (see docs.flojoy.io)

Testing
TODO

[ ] This PR includes a unit test (example here and/or ideally a screenshot of the node's output on an example app.

(base) dal@dal-desktop:~/Documents/flojoy/20230703/studio/PYTHON/nodes/TRANSFORMERS/ARITHMETIC$ pytest .
============================================ test session starts =============================================
platform linux -- Python 3.10.9, pytest-7.1.3, pluggy-1.0.0
rootdir: /home/dal/Documents/flojoy/20230703/studio/PYTHON/nodes, configfile: pytest.ini
plugins: cov-4.1.0
collected 5 items

ABS/ABS_test_.py .                                                                                [ 20%]
ADD/ADD_test_.py .                                                                                [ 40%]
DIVIDE/DIVIDE_test_.py .                                                                               [ 60%]
MULTIPLY/MULTIPLY_test_.py .                                                                           [ 80%]
SUBTRACT/SUBTRACT_test_.py .                                                                           [100%]

============================================= 5 passed in 0.48s ==============================================


First release

I would like to release version 0.1.0 of Disq. Most, if not all, of the bugs we've found so far have been fixed, and it would be good to go through the mechanics of actually doing a release. We are still in pre-1.0 so it's possible to change the public API in minor releases (and I expect we will).

I would consider #44 / #27 to be a blocking issue for release.
#13 #15 #18 should also be completed before a release.  In BigDataGenomics projects, we use Github Milestones to triage issues to releases.  Would it work to do the same here?

#27 / #52 is the only remaining blocking issue now.

All the blockers are closed now. I'm going to try to do the release tomorrow.

Disq 0.1.0 has been released. Thanks to everyone who contributed to the release.
I've opened #60 to document the release process that I followed.

Yay!


Allow to pass integer as trackTotalHits parameter

Based on documentation track_total_hits can be type of boolean or integer.

Thanks!


Translate FamiTracker text exports

Apparently more chiptune musicians in the West are familiar with FamiTracker than with MML (PPMCK) or LilyPond, the notations that inspired Pently syntax. Competing NES music engines, such as FamiTone, offer one-click conversion of a subset of FamiTracker text exports to their own format. At first I thought the differences between FamiTracker and Pently were so insurmountable. But eventually I figured out a subset that might be workable.
The following command produces a text export:
famitracker.exe etude.ftm -export etude.txt

Interpretation of the text export would not be without compromises:

Expansion sound: Only MMC5 is supported, and the first MMC5 channel would translate to the attack channel.
Release phases of envelopes are initially unsupported, as are pitch and high-pitch envelopes and the volume channel in patterns.
The following effects are supported

0xy -> ENxy
3xx -> Set slur to true if x is nonzero else false
Bxx -> dal segno, setting frame x as the segno
Cxx -> fine
Dxx -> pattern break. x is ignored and treated as if it were 00.
Fxx -> set Pently tempo based on FT speed, FT tempo, current scale, current time signature, and frame rate.
Gxx -> grace wait for x frames then note
Qxy -> Note for one row then slur into new pitch y semitones up, ignoring x
Rxy -> Note for one row then slur into new pitch y semitones down, ignoring x
Sxx -> grace note for x frames then rest

Each pitch at which a noise note is played would need to become a separate sound effect.

But a conversion still needs to specify the time signature (what's a measure?) and the scale (how many rows is an eighth note?).

This would be huge. Cant wait to get home and look at these

NovaSquirrel is working on this in ft2pently

Wonderful. Going to be so useful!

I've used ft2pently, and it appears to be stable. Further issues can be filed at its issue tracker.


Wrong WMO liquids parsing.

Stormwind.

Undercity.

Note to self: Azeroth (-8833, 628) for Stormwind.  I can't find an obvious explanation for this yet.

Smaller WMOs from 0.5.3 and 1.12.1 that have MLIQ data, for comparison:

world\wmo\dungeon\az_deadmines\az_deadmines_a.wmo
world\wmo\azeroth\buildings\stranglethorn_bootybay\bootybay.wmo
world\wmo\azeroth\collidable doodads\elwynn\abbeygate\abbeygate01.wmo

Azeroth ADT (31, 48) has world\wmo\azeroth\collidable doodads\elwynn\abbeygate\abbeygate01.wmo

1.12.1:

Alpha:

For 1.12.1, the MLIQ chunk has a base vertex of (-70.8333, 120.8333, -27.3881588).  For 0.5.3, the base vertex is (-54.166, 120.833, -27.38815).  The difference here is probably what is setting this off.  Note that the y values are the same, and x is shifted to the left by 16.666, which matches visually the screenshots from the previous post.

For reference: https://wowdev.wiki/WMO#MLIQ_chunk
For 0.5.3:

Num verts = (5, 5)
Width = 4
Height = 4
Base = (-54.1666641, 120.833328, -27.3881588)

For 1.12.1:

Num verts = (5, 5)
Width = 4
Height = 4
Base = (-70.8333282, 120.833328, -27.3881588)

When calculating the first vertex in

Thanks to schlumpf pointing out that this is likely the result of mirroring.

Should be fixed with https://github.com/namreeb/namigator/commit/bb4a2221cc1671cb059011710dd88c491925a81c


jest-jasmine2: pretty-print non-Error errors

Summary
See #3839 - hopefully this helps. If not, properly showing non-Error objects is still a win.
Test plan
This is mostly lifted from https://github.com/jasmine/jasmine/commit/8807bbb85042a845c81149bf6c2865b477989399#diff-f41e57b2889bae3046aa9f352699785c . The original tests for Env were removed so I just tested manually and that seemed fine.

LOL, I added the formatting to the wrong place, but the tests showed me where to go instead. :)

Oh, fail in Env.js is still used for done(). Adding back.

I wonder if we should create an error in this case ourselves (but keep the message you added), so that we can point back to the user's code?

With the error creation, do you mean creating a stack if one is missing? That is already happening by stackFormatter…

The problem is that by the time we get to report the error, the stack is only Jest because it's async. Maybe that functionality is for another PR.
Looks like we're almost there, but I still have a problem with some tests getting color codes for some reason. Note the first commit I needed to add to force runTests to set color to 0.
Am I running the tests wrong? I cd into integration_tests and run node ../jest

Ah ok. I agree we can do it in a follow-up then.

You should runt the tests from root, btw. Just ./jest

Codecov Report

Merging #5980 into master will decrease coverage by <.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #5980      +/-   ##
==========================================
- Coverage   64.36%   64.36%   -0.01%
==========================================
  Files         217      217
  Lines        8292     8289       -3
  Branches        3        3
==========================================
- Hits         5337     5335       -2
+ Misses       2954     2953       -1
  Partials        1        1

Impacted Files
Coverage Δ

packages/jest-jasmine2/src/jasmine/Env.js
0% <0%> (ø)
:arrow_up:

...es/jest-jasmine2/src/expectation_result_factory.js
100% <100%> (ø)
:arrow_up:

packages/jest-config/src/normalize.js
94.04% <0%> (-0.1%)
:arrow_down:

packages/jest-config/src/utils.js
88.67% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@SimenB Changelog entry added, no further changes. I took a look at the code coverage but couldn't find anything actionable.

Nice work :) Thanks for the PR!


Turn on LLVM loop unrolling by default

This looks fine, but now experimental and default are the same. Do we want to keep running the two or do you want to just drop experimental all together. When you add a new suite for mmt4d, we can call it something something mmt4d.

I'd say better than fine. Major improvements on single-threaded :-D I have a PR out to use the experimental config for MMT4D (https://github.com/google/iree/pull/8407). The way things are set up right now, the names are based on specific attributes of the benchmark and creating a new one creates a new series and pollutes a global namespace. The idea with the "experimental-flags" config was that it was a generic name for whatever experimental flags someone wanted to test at the time. I don't think https://github.com/google/iree/pull/8407this is the best setup, but within the current framework it seems alright. So I'd like to submit this as-is and then submit the PR to switch it to MMT4D so that the benchmark report on that PR is just showing the changes from MMT4D.


CUDA-Team: migrate to autoAddOpenGLRunpathHook

Where possible, migrate tree-wide from manual applications of addOpenGLRunpath to autoAddOpenGLRunpathHook. This will reduce package breakage and ease of maintenance.
A quick grep of Nixpkgs reveals the inclusion in following packages (barring definitions). They should be changed, or a comment added describing why such a change is incorrect or impractical.

[ ] blender
[ ] cemu
[ ] cntk
[ ] cuda-library-samples
[ ] cuda-samples
[ ] cudaPackages.cuda_nvprof
[ ] cudaPackages.cudatoolkit
[ ] cudaPackages.cudnn (update outdated comment referencing addOpenGLRunpath)
[ ] cudaPackages.cutensor
[ ] cudaPackages.nccl
[ ] cudaPackages.nsight_systems
[ ] cupy
[ ] darling
[ ] davinci-resolve
[ ] faiss
[ ] feishu
[ ] ffmpeg
[ ] firestarter
[ ] geekbench_4
[ ] geekbench_5
[ ] geekbench
[ ] gpu-burn
[ ] gst_all_1.gstreamer
[ ] hover
[ ] hqplayerd
[ ] jaxlib-bin
[ ] jaxlib
[ ] level-zero
[ ] libglvnd
[ ] libgpuarray
[ ] libnvidia-container
[ ] libtorch-bin
[ ] mangohud
[ ] mathematica
[ ] mpv
[ ] numba
[ ] nvidia-vaapi-driver
[ ] nvtop
[ ] obs-studio
[ ] ocl-icd
[ ] openai-triton-bin
[ ] openai-triton
[ ] openmm
[ ] openvino
[ ] pycuda
[ ] pynvml
[ ] renderdoc
[ ] seamly2d
[ ] tensorflow-bin
[ ] tensorflow
[ ] torch-bin
[ ] torchaudio-bin
[ ] torchvision-bin
[ ] vulkan-loader

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cuda-team-roadmap-and-call-for-sponsors/29495/1

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cuda-team-roadmap-update-2023-08-29/32379/1


Add files via upload

@alEXTREME6 mesota.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Have Docker images built with bash preinstalled

Requirement - what kind of business use case are you trying to solve?
Support bash by default in the docker images.
Problem - what in Jaeger blocks you from solving the requirement?
The docker images publisher in the Docker hub do not have bash, which means that when they're used in a Dockerfile with the RUN command (for example to add a wait-for script in a docker compose script) these fail.
#5 0.497 container_linux.go:367: starting container process caused: exec: "/bin/sh": stat /bin/sh: no such file or directory

Proposal - what do you suggest to solve the problem or improve the existing situation?
Expose builds of these images that contain bash, or use another base distribution of linux.
Else, document how to add Dockerfile RUN commands for the existing images, when there is no bash preinstalled.

for example to add a wait-for script in a docker compose script

Could you expand on the use case? Are you using that to make Jaeger wait for a dependency to start before running the entrypoint?

I am closing this as we didn't get the exact use case.
Btw. Jaeger images do have shell
docker run --rm -it --entrypoint /bin/sh jaegertracing/jaeger-collector:latest


Transfer of domain name to PE

Looks like this has been missing in the list.

And it's transferred now.


Very Slow Frame Rate

Splitting slow frame rate issue from #14 here
The Problem
Observed very slow frame rate on a ThinkPad with python 3.9
The Solution
Updated ThinkPad's BIOS
Context

Googled complaint about Lenovo USB serial issue, and Lenovo has a trouble shooting page indicated updating BIOS: https://support.lenovo.com/ca/en/solutions/ht080247-troubleshooting-usb-devices-thinkpad, guessed if it was related to the issue we had, and my new ThinkPad was just prompting for updating BIOS, I accepted the update and the problem gone.

Added to Troubleshooting guide - see https://github.com/OnionIoT/tau-lidar-server/commit/698bfec0c27fcdf22d7ba30986649a7d02a5d814


Invalid resource group location when example's setting different from default on Azure docker-based cluster

When create a docker-based cluster on Azure.
example.yaml
# Cloud-provider specific configuration.
provider:
    type: azure
    region: westus

default.yaml
# Cloud-provider specific configuration.
provider:
    type: azure
    location: westus2

Encounter an exception as below
Cluster: docker

Checking Azure environment settings
2022-04-01 09:36:00,456 INFO config.py:60 -- Using subscription id: ......
2022-04-01 09:36:00,456 INFO config.py:73 -- Creating/Updating Resource Group: intelML
Traceback (most recent call last):
  File "/root/miniconda2/envs/azure/bin/cloudtik", line 8, in <module>
    sys.exit(main())
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/scripts/scripts.py", line 1444, in main
    return cli()
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/core/_private/cli_logger.py", line 807, in wrapper
    return f(*args, **kwargs)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/scripts/scripts.py", line 480, in up
    use_login_shells=use_login_shells)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/core/_private/cluster/cluster_operator.py", line 236, in create_or_update_cluster
    config = _bootstrap_config(config, no_config_cache=no_config_cache)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/core/_private/cluster/cluster_operator.py", line 322, in _bootstrap_config
    resolved_config = provider_cls.bootstrap_config(config)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/providers/_private/_azure/node_provider.py", line 341, in bootstrap_config
    return bootstrap_azure(cluster_config)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/providers/_private/_azure/config.py", line 41, in bootstrap_azure
    config = _configure_resource_group(config)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/cloudtik/providers/_private/_azure/config.py", line 75, in _configure_resource_group
    resource_group_name=resource_group, parameters=params)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/azure/mgmt/resource/resources/v2021_04_01/operations/_resource_groups_operations.py", line 158, in create_or_update
    map_error(status_code=response.status_code, response=response, error_map=error_map)
  File "/root/miniconda2/envs/azure/lib/python3.7/site-packages/azure/core/exceptions.py", line 105, in map_error
    raise error
azure.core.exceptions.ResourceExistsError: (InvalidResourceGroupLocation) Invalid resource group location 'westus2'. The Resource group already exists in location 'westus'.
Code: InvalidResourceGroupLocation
Message: Invalid resource group location 'westus2'. The Resource group already exists in location 'westus'.

This is not a bug. In your example, you use:
region: westus
But for Azure, it should be:
location: westus
which means you didn't specify the location to westus and then use the default "westus2". But the resource group do belongs to westus. It reported the right error.

This is not a bug. In your example, you use: region: westus
But for Azure, it should be: location: westus
which means you didn't specify the location to westus and then use the default "westus2". But the resource group do belongs to westus. It reported the right error.

Yes, this is not a bug, cloudtik reported the right error for my incorrect configuration of yaml. I will close it.


Gym not honoring "BuildSystemType" in project/workspace settings.

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Gym does not honor the BuildSystemType that is in the project file.
I want Gym to use Xcode's "New Build System" for faster building.
Looking at the documentation for fastlane / gym I couldn't see anywhere where I could provide additional flags to the xcodebuild command either.
Is it possible to use Xcode's new build system with Gym?

That did work.
It would be a nice feature to add to fastlane to honor the project file setting for the build system though.

@SeanRobinson159 This probably won't get added as an option to fastlane since Xcode will most likely end up using the new system transparently in a new version which would make the change not necessary 🙃 But thanks for bringing this issue up so that others can find a solution 🙌


apps: tolerate control-plane taint

What this PR does / why we need it: to support the new node-role.kubernetes.io/control-plane:NoSchedule taint
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #970
Checklist:

[x] Added relevant notes to WIP-CHANGELOG.md
[x] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[ ] is completely transparent, will not impact the workload in any way.
[x] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Chart checklist (pick exactly one):

[x] I upgraded no Chart.
[ ] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

I think I addressed all comments. Please check again as I also updated the migration script.


[Feature request] make the hwid actually useful

like in the gamesense theme, make the hwid under the profile tab and under the activate subscription and only allow the user to see this if the subscription is active. u can also add api stuff to make it so it checks if the hwid correlates with the thingy then it says in the tab "hwid true (hwid)" and if not it returns "hwid false (hwid)"

Hi and thanks for your suggestion.
So you want the hwid of a user to be displayed in the profile tab when he has an active sub.
The API should give an error if this is not correct.
Did I understand that correctly?

yes

also make it so u can change the hwid, forgot to mention that

In the latest release the HWID is now displayed if it is set and the user has an active subscription.
I have not implemented the check in the API.
Because it is intended that this happens in the loader.
The hardware id can already be reset by administors and supporters on the admin page.

where would u reset the hardware id on the admin page? i dont see it

In the admin dashboard
under user there is a button that looks like a micro chip. with this you can reset the hwid of your users

If you have any other questions feel free to open another issue :)


Setup GraalVM Kokoro Scripts

This PR introduces GraalVM integration testing.
@Neenu1995 @lesv

@dzou Are you asking for a review?  build.sh is templated and once you have something that works, we can help you modify the templates to distribute it everywhere.

The pom.xml is probably also templated.

@dzou Are you asking for a review? build.sh is templated and once you have something that works, we can help you modify the templates to distribute it everywhere.

Hey @lesv, @Neenu1995 -- Sorry for any confusion. I just wanted to create this PR to see if the GraalVM Kokoro Job would build and run successfully in the presubmit.
I think we have a success for the GraalVM kokoro run:

Here is the link: https://source.cloud.google.com/results/invocations/3639ad3e-8411-49e4-9c01-382bda6b00b2/targets
I think this PR is a good summary of the changes we would need to make (no need to for a full merge to run the tests). Thoughts on the next steps? Maybe you may be amenable to me updating the templates -- It seems like it might be possible to add the changes to the templates and extend things cleanly: changing both the build.sh and pom.xml should be a no-op, and I think it's only the addition of the graalvm-native.cfg which opts-in a repository for GraalVM testing (which can be done one-by-one or slowly).


Update old link in python_library.md

Update an outdated link to OpenStack CLI installation instructions
Add a link to "the last chapter" Configure your terminal environment for Openstack

There is an error that should be fixed (see Travis checks).
Found web urls pointing to docs.csc.fi
Please change these to internal markdown links

Url https://docs.csc.fi/cloud/pouta/install-client/ on line 4 in file docs/data/Allas/using_allas/python_library.md points to docs.csc.fi

Url https://docs.csc.fi/cloud/pouta/install-client/#configure-your-terminal-environment-for-openstack on line 6 in file docs/data/Allas/using_allas/python_library.md points to docs.csc.fi

Goes to teach me that I should always read the style guide carefully before doing anything else... Well, anyhow, I guess it works now.

Thanks for your contribution!


[AutoPR cognitiveservices/data-plane/TextAnalytics] Change default generation to TextAnalytics 2.1

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4533

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/557


test14

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d7cce4b). Click here to learn what that means.
The diff coverage is n/a.

Flag
Coverage Δ

packit-e2e
51.77% <0.00%> (?)

testsuite
24.07% <0.00%> (?)

unittests
11.11% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


"Cizer" , "Plesca" , "Pria" are listed in the drop-down menu

Description:
When user writes "Cizer" , "Plesca" , "Pria" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Plesca"

Expected results
The location appears in the drop-down menu as "Sat Plesca, SĂLAJ (Cizer)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.


add glossary defs client-side, server-side, client-server

add glossary defs client-side, server-side, client-server.

Maybe in a new ticket - should we rename
https://unifiedid.com/docs/guides/integration-javascript-server-side
https://unifiedid.com/docs/guides/integration-prebid-server-side
as client-server guide? technically it is asking pub to generate tokens on server side and then pass the identity to the js sds/prebid.js  in the client side to auto refresh so it is client-server by your definition?


Bug 1823714: Update PkgManifest upon catsrc update

The catsrc update sync triggred by the informer does not update pkgmanifest.

Description of the change:
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

/retest

/retest

/test e2e-aws-olm

/retest

/hold cancel

/retest

/retest e2e-aws-olm

/test e2e-aws-olm

/cherry-pick release-4.4
/cherry-pick release-4.3
/cherry-pick release-4.2

/cherry-pick release-4.3

/cherry-pick release-4.2

Can we skip the err nit, that is really a nit and breaks the way how this function is structured.

/lgtm

/approve

/approve

/retest


Rename 'matplotlib:d3' option to 'matplotlib:mpld3'

This is a suggested name change in the backend option of the output magic. I think this would be more consistent and clearer - it makes sense to mention the actual name of the dependency/library used.

Implemented in dc246cd and subsequent fixes.


fix: bug where only first order clause is applied

Closes https://github.com/supabase-community/postgrest-dart/issues/59

@bdlukaa @dshukertjr Will there be a new version with this change published soon?


Compile fails on Solaris/x64 with syntax error in assembler

I'm using Sun C v5.13 compiler and the configure line:
$ Configure --prefix=/export/home/nse/workspace/openssl-solaris10_x64_sunc513/install shared enable-trace -m64 solaris64-x86_64-cc

Which produced this configure output:
Command line (with current working directory = .):

    /export/home/nse/workspace/openssl-solaris10_x64_sunc513/local_perl/bin/perl ../Configure --prefix=/export/home/nse/workspace/openssl-solaris10_x64_sunc513/install shared enable-trace -m64 solaris64-x86_64-cc

Perl information:

    /export/home/nse/workspace/openssl-solaris10_x64_sunc513/local_perl/bin/perl
    5.24.0 for i86pc-solaris

Enabled features:

    aria
    asm
    async
    autoalginit
    autoerrinit
    autoload-config
    bf
    blake2
    camellia
    capieng
    cast
    chacha
    cmac
    cmp
    cms
    comp
    ct
    deprecated
    des
    dgram
    dh
    dsa
    dso
    dtls
    dynamic-engine
    ec
    ec2m
    ecdh
    ecdsa
    engine
    err
    filenames
    fips
    gost
    idea
    legacy
    md4
    mdc2
    module
    multiblock
    nextprotoneg
    pinshared
    ocb
    ocsp
    padlockeng
    pic
    poly1305
    posix-io
    psk
    rc2
    rc4
    rdrand
    rfc3779
    rmd160
    scrypt
    secure-memory
    seed
    shared
    siphash
    siv
    sm2
    sm3
    sm4
    sock
    srp
    srtp
    sse2
    ssl
    static-engine
    stdio
    tests
    threads
    tls
    trace
    ts
    ui-console
    whirlpool
    tls1
    tls1-method
    tls1_1
    tls1_1-method
    tls1_2
    tls1_2-method
    tls1_3
    dtls1
    dtls1-method
    dtls1_2
    dtls1_2-method

Disabled features:

    afalgeng            [not-linux]      OPENSSL_NO_AFALGENG
    asan                [default]        OPENSSL_NO_ASAN
    buildtest-c++       [default]
    crypto-mdebug       [default]        OPENSSL_NO_CRYPTO_MDEBUG
    devcryptoeng        [default]        OPENSSL_NO_DEVCRYPTOENG
    ec_nistp_64_gcc_128 [default]        OPENSSL_NO_EC_NISTP_64_GCC_128
    egd                 [default]        OPENSSL_NO_EGD
    external-tests      [default]        OPENSSL_NO_EXTERNAL_TESTS
    fuzz-libfuzzer      [default]        OPENSSL_NO_FUZZ_LIBFUZZER
    fuzz-afl            [default]        OPENSSL_NO_FUZZ_AFL
    ktls                [default]        OPENSSL_NO_KTLS
    makedepend          [unavailable]
    md2                 [default]        OPENSSL_NO_MD2 (skip crypto/md2)
    msan                [default]        OPENSSL_NO_MSAN
    rc5                 [default]        OPENSSL_NO_RC5 (skip crypto/rc5)
    sctp                [default]        OPENSSL_NO_SCTP
    ssl-trace           [default]        OPENSSL_NO_SSL_TRACE
    ubsan               [default]        OPENSSL_NO_UBSAN
    unit-test           [default]        OPENSSL_NO_UNIT_TEST
    uplink              [no uplink_arch] OPENSSL_NO_UPLINK
    weak-ssl-ciphers    [default]        OPENSSL_NO_WEAK_SSL_CIPHERS
    zlib                [default]
    zlib-dynamic        [default]
    ssl3                [default]        OPENSSL_NO_SSL3
    ssl3-method         [default]        OPENSSL_NO_SSL3_METHOD

Config target attributes:

    AR => "ar",
    ARFLAGS => "r",
    CC => "cc",
    CFLAGS => "-xO5 -xdepend -xbuiltin",
    HASHBANGPERL => "/usr/bin/env perl",
    RANLIB => "ranlib",
    RC => "windres",
    asm_arch => "x86_64",
    bn_ops => "SIXTY_FOUR_BIT_LONG",
    build_file => "Makefile",
    build_scheme => [ "unified", "unix" ],
    cflags => "-xarch=generic64 -xstrconst -Xa",
    cppflags => "-D_REENTRANT",
    defines => [  ],
    disable => [  ],
    dso_scheme => "dlfcn",
    enable => [  ],
    ex_libs => "-lsocket -lnsl -ldl -lpthread",
    includes => [  ],
    lflags => "-mt",
    lib_cflags => "",
    lib_cppflags => "-DFILIO_H -DL_ENDIAN",
    lib_defines => [ "OPENSSL_BUILDING_OPENSSL" ],
    module_cflags => "-KPIC",
    module_cxxflags => undef,
    module_ldflags => "-G -dy -z text -Wl,-Bsymbolic",
    multilib => "/64",
    perl_platform => "Unix",
    perlasm_scheme => "elf",
    shared_cflag => "-KPIC",
    shared_defflag => "-Wl,-M,",
    shared_defines => [  ],
    shared_ldflag => "-G -dy -z text -Wl,-Bsymbolic",
    shared_rcflag => "",
    shared_sonameflag => "-Wl,-h,",
    shared_target => "solaris",
    thread_defines => [  ],
    thread_scheme => "pthreads",
    unistd => "<unistd.h>",

Recorded environment:

    AR =
    BUILDFILE =
    CC =
    CFLAGS =
    CPPFLAGS =
    CROSS_COMPILE =
    CXX =
    CXXFLAGS =
    HASHBANGPERL =
    LDFLAGS =
    LDLIBS =
    OPENSSL_LOCAL_CONFIG_DIR =
    PERL =
    RANLIB =
    RC =
    RCFLAGS =
    WINDRES =
    __CNF_CFLAGS =
    __CNF_CPPDEFINES =
    __CNF_CPPFLAGS =
    __CNF_CPPINCLUDES =
    __CNF_CXXFLAGS =
    __CNF_LDFLAGS =
    __CNF_LDLIBS =

Makevars:

    AR              = ar
    ARFLAGS         = r
    CC              = cc
    CFLAGS          = -xO5 -xdepend -xbuiltin -m64
    CPPDEFINES      =
    CPPFLAGS        =
    CPPINCLUDES     =
    CXXFLAGS        = -m64
    HASHBANGPERL    = /usr/bin/env perl
    LDFLAGS         =
    LDLIBS          =
    PERL            = /export/home/nse/workspace/openssl-solaris10_x64_sunc513/local_perl/bin/perl
    RANLIB          = ranlib
    RC              = windres
    RCFLAGS         =

NOTE: These variables only represent the configuration view.  The build file
template may have processed these variables further, please have a look at the
build file for more exact data:
    Makefile

build file:

    Makefile

build file templates:

    ../Configurations/common0.tmpl
    ../Configurations/unix-Makefile.tmpl
    ../Configurations/common.tmpl

During the build the error happens here:
...
cc -KPIC -xarch=generic64 -xstrconst -Xa -xO5 -xdepend -xbuiltin -m64 -DFILIO_H -DL_ENDIAN -DOPENSSL_BUILDING_OPENSSL -DOPENSSL_PIC -DOPENSSLDIR="\"/export/home/nse/workspace/openssl-solaris10_x64_sunc513/install/ssl\"" -DENGINESDIR="\"/export/home/nse/workspace/openssl-solaris10_x64_sunc513/install/lib/engines-3\"" -DMODULESDIR="\"/export/home/nse/workspace/openssl-solaris10_x64_sunc513/install/lib/ossl-modules\"" -D_REENTRANT -DNDEBUG  -c -o crypto/aes/libcrypto-lib-aes-x86_64.o crypto/aes/aes-x86_64.s
cc: Warning: -xarch=generic64 is deprecated, use -m64 to create 64-bit programs
Assembler:
	"crypto/aes/aes-x86_64.s", line 2659 : Syntax error
	Near line: "	.section ".note.gnu.property", "a""
	"crypto/aes/aes-x86_64.s", line 2665 : Illegal mnemonic
	Near line: "	.asciz "GNU""
	"crypto/aes/aes-x86_64.s", line 2665 : Syntax error
	Near line: "	.asciz "GNU""
	"crypto/aes/aes-x86_64.s", line 2665 : Syntax error
	Near line: "	.asciz "GNU""
cc: fbe failed for crypto/aes/aes-x86_64.s
*** Error code 2
make: Fatal error: Command failed for target `crypto/aes/libcrypto-lib-aes-x86_64.o'
Current working directory /export/home/nse/workspace/openssl-solaris10_x64_sunc513/build-solaris10_x64_sunc513

It looks like this was broken by #11044 or #10985
I am not sure the assumption was correct that these files are only used by intel.

It looks like this was broken by #11044 or #10985
I am not sure the assumption was correct that these files are only used by intel.

Ping @hjl-tools. Your thoughts on this?

#11137 is a quick and appropriate fix.  A longer term fix would be to push through values like $cet_property through the translator (the translator translates .asciz directives to the corresponding .byte directive) as well instead of appending it to the output verbatim.  That can be done, with some careful coding.

@jon-oracle Hopefully this addresses the issue

The build failed again, in a different place:
cc -KPIC -xarch=generic64 -xstrconst -Xa -xO5 -xdepend -xbuiltin -m64 -DFILIO_H -DL_ENDIAN -DOPENSSL_BUILDING_OPENSSL -DOPENSSL_PIC -DOPENSSLDIR="\"/export/home/nse/workspace/openssl-solaris11_x64_sunc515/install/ssl\"" -DENGINESDIR="\"/export/home/nse/workspace/openssl-solaris11_x64_sunc515/install/lib/engines-3\"" -DMODULESDIR="\"/export/home/nse/workspace/openssl-solaris11_x64_sunc515/install/lib/ossl-modules\"" -D_REENTRANT -DNDEBUG  -c -o crypto/aes/libcrypto-lib-aes-x86_64.o crypto/aes/aes-x86_64.s
Assembler:
	"crypto/aes/aes-x86_64.s", line 2659 : Syntax error
	Near line: "	.section ".note.gnu.property", "a""
cc: fbe failed for crypto/aes/aes-x86_64.s
*** Error code 2
make: Fatal error: Command failed for target `crypto/aes/libcrypto-lib-aes-x86_64.o'
Current working directory /export/home/nse/workspace/openssl-solaris11_x64_sunc515/build-solaris11_x64_sunc515
*** Error code 1
make: Fatal error: Command failed for target `build_sw'
make

Maybe let me know if/when there's a fix and I can test it in my env?

So it looks like it addressed one of the issues

Correct me if I'm wrong, but x86_64 is Intel, isn't it? So this is rather a tool chain issue...

Either way, we should probably look more closely at the perlasm flavor

The build failed again, in a different place:
cc -KPIC -xarch=generic64 -xstrconst -Xa -xO5 -xdepend -xbuiltin -m64 -DFILIO_H -DL_ENDIAN -DOPENSSL_BUILDING_OPENSSL -DOPENSSL_PIC -DOPENSSLDIR="\"/export/home/nse/workspace/openssl-solaris11_x64_sunc515/install/ssl\"" -DENGINESDIR="\"/export/home/nse/workspace/openssl-solaris11_x64_sunc515/install/lib/engines-3\"" -DMODULESDIR="\"/export/home/nse/workspace/openssl-solaris11_x64_sunc515/install/lib/ossl-modules\"" -D_REENTRANT -DNDEBUG  -c -o crypto/aes/libcrypto-lib-aes-x86_64.o crypto/aes/aes-x86_64.s
Assembler:
	"crypto/aes/aes-x86_64.s", line 2659 : Syntax error
	Near line: "	.section ".note.gnu.property", "a""

What is the correct syntax to generate a NOTE section with SHF_ALLOC on
Solaris?

Either way, we should probably look more closely at the perlasm flavor

Is there a way not to generate it on Solaris?

Ah!  From a manual I found:
For ELF targets, the assembler supports another type of .section directive for compatibility with the Solaris assembler:
.section "name"[, flags...]

Note that the section name is quoted. There may be a sequence of comma separated flags:
#alloc
section is allocatable
#write
section is writable
#execinstr
section is executable

So maybe works more universally?
	.section ".note.gnu.property", #alloc

Jon - do you want to try that in crypto/perlasm.x86_64-xlate.pl to see if that works?

Ah! From a manual I found:

For ELF targets, the assembler supports another type of .section directive for compatibility with the Solaris assembler:
.section "name"[, flags...]
Note that the section name is quoted. There may be a sequence of comma separated flags:
#alloc
section is allocatable
#write
section is writable
#execinstr
section is executable

So maybe works more universally?
	.section ".note.gnu.property", #alloc

This doesn't work on Linux:
crypto/aes/aes-x86_64.s: Assembler messages:
crypto/aes/aes-x86_64.s:2659: Error: character following name is not '#'
make[1]: *** [Makefile:18608: crypto/aes/libfips-lib-aes-x86_64.o] Error 1

Yeah I've found that:
.section .note.gnu.property #alloc
Works for gcc on linux, but for Solaris/cc this is required:
.section .note.gnu.property, #alloc
I haven't found a combination which works for both as yet.

Considering that the gas manual does specify a comma there, that's a bug in gas and should be reported back to the gas folks.
That doesn't help us, though...
Suggested solution in #11191

Yeah I've found that:
.section .note.gnu.property #alloc
Works for gcc on linux, but for Solaris/cc this is required:

Not really since # starts a comment.

.section .note.gnu.property, #alloc
I haven't found a combination which works for both as yet.

Not really since # starts a comment.

Does it?  You have to consider that this is post xlate text


TechEmpower ban 2024; the lore

Sour critique leads to ban
His ego couldn't take me pointing out that his quality control is zero (which it is), so he permanently banned me on GitHub, Twitter, removed and banned uWebSockets.js from the top and added a code of conduct. Yes the project itself is lifetime banned now.
The tweet:

The response:
"[...] Happy to remove your framework. Seeing your tweets now as well. You're not interested in reviewing code and making things better; you're interested in being right and you're too toxic about it. [...]"

The ban:

The block:

1st place in TechEmpower performs half that of uWS.js
TechEmpower is a flawed benchmark for two main reasons:

Fake Participants: Many top participants use obvious hacks like simply splitting the TCP data on "\r\n\r\n", doing no real parsing or standards compliant interpretation whatsoever. There is no quality control to catch these fakes. For example, "gnet," which won Round 22, uses such a hack, making the entire test meaningless. "faf" also uses this hack.
Misleading HTTP Pipelining: By enabling the most bizarre level of HTTP pipelining ever seen in a benchmark, the test only measures how fast the parser can process these tightly packed requests, not its total I/O performance. Since all top participants simply fake the HTTP parsing step as explained above, the entire TechEmpower benchmark is meaningless.

We can very easily see this by running a regular non-pipelined benchmark showing uWS.js massively outperforms "mrhttp", which is the very 1st in TechEmpower.

Most recent run before ban
TechEmpower is not a credible benchmark.
Either way, uWS.js scores insignificantly different from 1st. If excluding obvious fakes & incorrect participants, it scores actually 1st (however, even this should be taken with a grain of salt given how statistically insignificant the win would be).

Some important nitpicks (elaborating the critique)

We send more data per request: "Server: uWebSockets.js" and "uWebSockets: 20" vs. only sending "Server: F" like FaF does. It means we occupy more bandwidth per request. This could be fixed in a second run, but we are banned and can't participate any longer. TE does not keep track of CPU-usage, so we can only guess whether this single point makes all the difference, or not. Most likely, this has the most impact on the score.
We are entirely standards compliant (RFC 9110) with all the request smuggling checks, invalid field name checks, conflicting body length checks, missing "host" header checks, PROXYv2 header check, performed according to spec. None of the others in top 5 do even close to this. Most of them simply scan until "\r\n\r\n" and assume that's a valid request. Most of them can't even receive a body because they don't even check "content-length".
We do dynamic URL routing with wildcard, static, parameter routes. They (gnet, mrhttp, faf, pico.v) do just a hardcoded if (memcmp("GET") == 0) [[likely]] kind of "routing" and just copy a preformatted hardcoded buffer back.
Pipelining benchmarks only stress the HTTP parser and URL router which conveniently are the very parts that are seriously non-compliant, outright broken or entirely missing in the TE top performers (gnet, mrhttp, faf). Anyone can make a fast HTTP parser that just skips to "\r\n\r\n" using 2 lines of SIMD intrinsics.  Native uWS does not use libuv, but links statically to its own epoll implementation with LTO linking and O3 optimizations, producing even better results. But, only uWS.js was added to TE, not native uWS. There is also no need for uWS.DeclarativeResponse in native uWS, cutting even more overhead. Likewise, the PROXYv2 parser pass can be disabled in native uWS, eliminating even more overhead.
Results are insignificantly different among the top 5. There is a cutoff plateau where ordering is very tight and depends on essentially random noise. A 7th placement can be a 2nd placement next run, as the variance is enough to scramble the list between runs.

Knowing GitHub, I can still insert a link https://github.com/TechEmpower/FrameworkBenchmarks/pull/9160 on his PR 😉

@remittor I got banned from the project, so I will respond here: yes I absolutely agree mrhttp is a broken project that nobody can use for anything real. 100% agree. But at least it does most of the basics of HTTP somewhat correctly. Comparing that with gnet, which does nothing correctly whatsoever - mrhttp is a saint. But yes, of course, none of those are usable in any kind of production app. They are joke projects. But there are worse offenders than mrhttp in my opinion, esp. since it actually has a somewhat novel approach with AVX2 (not really, but still)

Very strange things.
It turns out that if I review the code for gnet, I can also be banned?
Maybe there was another reason for the ban? (I didn’t keep track of what exactly caused this reaction)

How this slipped past quality control is beyond me, but also doesn't shock me.

This got him red faced

Ok I will clean this thread (remove comments) and post the last complete run with uWS.js here, and lock the thread when it finishes (3 days remains). Then at least we have 1 full run to refer to.

@remittor ok I will remove this comment soon, thanks for responding but I want this thread clean and locked without comments :D

Ok, looks like the project can come back if someone other than me reverts the removal:
git clone https://github.com/TechEmpower/FrameworkBenchmarks
git checkout -b add_uwebsocketsjs
git revert 4b0a91f07147386c8f11b36b1410f00b34c7611c
git commit
git push origin add_uwebsocketsjs
and make a PR of that 👌

If you're fine with it, I'll add it?

Absolutely, it's good PR regardless of credibility

If you want you can also remove the MySQL subject since it's very obvious all the top ones use PostgreSQL, so why even have MySQL there at all

The criticism has been given for many many years in far more civilised ways before. The problem with that appraoch is that nobody cares and nothing happens. Also, sensationalism sells. I have never seen any significant traffic to a civilised opinion. If you want to highlight something truly, fundamentally wrong, you need to kick the front door in and deliver the message. In my case here, it resolved itself as someone else can add it back now.
I don't even want to be in the test (because it is a nonsense test by incompetent people who don't understand how things work, for many reasons) - but that test is a necessary evil since most people use it for reference. So it's a de facto test you have to be in, regardless how poor its quality is.
This quote is pretty much spot on:

The critiques I've seen - I think - all seemed valid. Could it be that Alex might already have a negative stance because his time is also wasted by people clamouring about his project not being represented here, and how sub-par projects are faster than his and only float to the top by employing bad tactics and incomplete implementations?

In an ideal world, people would verify benchmarks themselves. But people rather see to authority to tell them what is true and what is false. I don't want any centralized source be the authority, but that people critically compare things themselves. But this is not how most peolpe work.

They could easily design a test that is impossible to cheat in, like this test:
https://github.com/SaltyAom/bun-http-framework-benchmark?tab=readme-ov-file#bun-http-framework-benchmark
By requiring the extraction of certain headers, queries and so on, you automatically weed out all the hardcoded cheats. But this far, they couldn't think. Because again, they don't understand what they are doing. They are typical "hardware geeks" that only think in terms of "oh wow, this server hardware is so beefy, we need it now".
Had they used their head, thought about the basics, they could have automatically weeded out all the fakes by just designing the test slighty more complex like linked. And all those steps are entirely web server related.

So, nobody added it back?

Done ;) https://github.com/TechEmpower/FrameworkBenchmarks/pull/9189
Got delayed because I Wanted to make it run myself too first but that was unfortunately not as straight forward as it could have been.

Ok. Thanks, guess that puts an end to this thing

Aha, vacation. Pfft, no time for vacation

Sure thing: https://github.com/TechEmpower/FrameworkBenchmarks/pull/9194

@porsager aha, it is uWebSockets._cfg

Already changed it ;)

It needs to be uWebSockets._cfg can't be webserver._cfg

https://github.com/TechEmpower/FrameworkBenchmarks/pull/9194/commits/389ed041de728ae9448d7092249137fb548291ba ;)


SupportedOSPlatform attributes so that API usage can be version-safe

The .NET 5 platform compatibility analyzer uses SupportedOSPlatform attributes to warn you if you use an API for an OS version older than your project's minimum without safely checking first:

https://docs.microsoft.com/en-us/dotnet/standard/analyzers/platform-compat-analyzer
https://github.com/dotnet/designs/blob/main/accepted/2020/platform-checks/platform-checks.md

I accidentally used https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-getdpiforwindow, not intending to limit my library to Windows 10 version 1607 and prevent it from running on any version of Windows Server. If the generated PInvoke.GetDpiForWindow method had been annotated with SupportedOSPlatform, maybe I would have caught this right away instead of overlooking and first noticing by seeing EntryPointNotFoundExceptions in production.

That is consistent with what we have talked about internally as well, so thanks for filing an issue to track. I'm transferring it to the metadata repo as the attribution will have to start there.


Leave empty blocks on the same line

For example:
function test() {}

@nayeemrmn Thanks again! Opened #77