proposal: net/http: improve graceful termination for http1 servers

Proposal Details
Currently, when gracefully terminating an http1 server using http.Server#Shutdown(), it will:

Close all listeners
Call user-supplied shutdown hooks in goroutines (w/o waiting for them to finish - https://github.com/golang/go/issues/32116)
Treat active connections that didn't receive request headers for 5 seconds - as idle (https://github.com/golang/go/issues/22682, https://github.com/golang/go/issues/59037)
Close idle connections
Loop from 2 until all connections are idle and closed

I'd like to hear the community's thoughts about minor changes to the points above,
which I believe could help implement a safer graceful termination flow:

User-supplied hooks - would it make more sense to wait for the hooks before exiting Shutdown() ?
otherwise we don't guarantee the hooks will execute before Shutdown() is finished, and potentially the process is terminated.

Hard-coded 5 seconds - the code mentions:

// Issue 22682: treat StateNew connections as if
// they're idle if we haven't read the first request's
// header in over 5 seconds.

would it make more sense to make it controllable by utilizing ReadHeaderTimeout w/ a fall-back to 5 seconds ?

Closing idle connections - for http1, when connections are reused, they are marked as idle as soon as a response is served;
this means that reused connections can toggle rapidly between active / idle when serving requests, and get shutdown if caught idle in the loop above.

I think we should take a small nuance into account for graceful termination - a client might just be before re-using an idle connection, while the server closes it, leading to an EOF on the client side;
for non idempotent requests (say, POST), this wouldn't be retried automatically by clients - the request is usually dropped, even if there's a new server already accepting requests (say, a version roll of a server, gracefully terminating the former).
This is the nature of http1, of course.
I would like to propose a minor change that would help servers better control this scenario, as I've found, could appear in critical applications (e.g. security-enforcing k8s admission controllers)
I believe we can:
3.1. Allow controlling the window of - when an idle connection is treated as idle - by introducing a time.Duration, where only connections that had been idle for at least that long will be closed (similar to IdleTimeout, but for closing), or
3.2. Introduce a pre-close hook that gives the caller a bit more control over the termination flow (similar to the current shutdown hooks, but for closing connections, and synchronous)
With either of the above, the server can configure that, say, it would only close connections that were idle for 30 seconds - increasing the odds that the connection is truly idle, and is not about to be active in a brief moment.
This has similar, but different semantics to IdleTimeout, which would close connections at runtime - as it would only scope it to the termination period;
this could make more sense for http1, as we can never fully guarantee a connection is not just about to be re-used.
Appreciate your Time !

golang-nuts discussion: https://groups.google.com/g/golang-nuts/c/G6Ct8kzZiRU

Related Issues and Documentation

net/http: make Server.Shutdown treat new connections as idle after 5 seconds
net/http: Server.Shutdown waits connections which are in new state, but Client puts such connections into idle connections pool #21204 (closed)
x/net/http2: Long running http2 requests are closed immediately on server.Shutdown #29764 (closed)
proposal: net/http: context based graceful server shutdown #52805 (closed)
net/http: add Server.Close & Server.Shutdown for forced & graceful shutdown
net/http, x/net/http2: http server shutdown doesn't gracefully shut down HTTP2 connections #39776 (closed)
net/http/server: Shutdown() may return without an error (timeout) but leave dangling connections #34163 (closed)
net/http: mark http/2 connections active
net/http: graceful shutdown race condition #36819 (closed)
net/http: idle timeout added in CL 121419 is undocumented and surprising #59037

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

RegisterOnShutdown is documented as: "This can be used to gracefully shutdown connections that have undergone ALPN protocol upgrade or that have been hijacked.". If there's no more connections and the shutdown proceeds, then there's nothing for the hook to do anyway.

Also, please include proposed API changes in your proposal.

Thanks Sean !
Valid point about the hooks - I thought it might make sense for Shutdown() to wait on all hooks to finish before returning, just in-case the caller set-up hooks which outlive the server itself;
the doc is formulated well though, so this is probably a moot point.
I've edited the original message to be more concrete on the proposals.


bug/issue-type

PR Overview
This PR will address the following Issue/Feature: #107
This PR will result in the following new package version:

v0.15.0
full refresh needed due to affected incremental model

Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:

Updated the jira__daily_issue_field_history model to make sure issue_type values are correctly joined into the downstream issue models. This applied only if issue type is leveraged within the issue_field_history_columns variable.

Reorganized some of the logic in jira__daily_issue_field_history so that the for loops run less often and used nested conditionals.

PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:

[x] dbt compile
[x] dbt run –full-refresh
[x] dbt run
[x] dbt test
[ ] ~dbt run –vars (if applicable)~

Before marking this PR as "ready for review" the following have been applied:

[x] The appropriate issue has been linked and tagged
[x] You are assigned to the corresponding issue and this PR
[x] BuildKite integration tests are passing

Detailed Validation
Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review":

[x] You have validated these changes and assure this PR will address the respective Issue/Feature.
[x] You are reasonably confident these changes will not impact any other components of this package or any dependent packages.
[x] You have provided details below around the validation steps performed to gain confidence in these changes.

See internal ticket for link to validation worksheet.

Standard Updates
Please acknowledge that your PR contains the following standard updates:

Package versioning has been appropriately indexed in the following locations:

[x] indexed within dbt_project.yml
[x] indexed within integration_tests/dbt_project.yml

[x] CHANGELOG has individual entries for each respective change in this PR

[x] README updates have been applied (if applicable)

[ ] ~DECISIONLOG updates have been updated (if applicable)~
[ ] ~Appropriate yml documentation has been added (if applicable)~

dbt Docs
Please acknowledge that after the above were all completed the below were applied to your branch:

[x] docs were regenerated (unless this PR does not include any code or yml updates)

If you had to summarize this PR in an emoji, which would it be?

:dancer:

@fivetran-joemarkiewicz Thanks for reviewing! I have pushed the updates to remove the elses. As for other custom fields, the customer did run with a ton of the columns listed in the issue (see the dbt project configs), and said there weren't any other affected fields.
However now that I think of this more, I'm still a little uncertain about what they are supposed to display. I tested out some additional fields like resolution, priority and project, and they display their id rather than a name. However, I updated the issue_type to show the name rather than id. Is this actually what it's supposed to be?

However now that I think of this more, I'm still a little uncertain about what they are supposed to display. I tested out some additional fields like resolution, priority and project, and they display their id rather than a name. However, I updated the issue_type to show the name rather than id. Is this actually what it's supposed to be?

@fivetran-catfritz yup this is what I was leaning towards with my probing question around if other custom fields need a similar update. The ultimate goal of this jira__daily_issue_field_history model is to display the latest field history (of those selected) value for an issue on a given day. This model is primarily generated by the stg_jira__issue_field_history model as well as a few others. The end model consolidates, pivots, and creates a slowly changing dimension from that staging model. In that staging model the values for some of the fields are foreign keys to tables which the connector syncs. This is what we saw originally with components and then status, and now what you are seeing with issue_type. I was curious if this is also the case for other field types that have their own tables (ie. priority, project, resolution) and that does seem to be the case. With this, I would recommend opening a FR on this repo to expand the logic we have for components, status, and issue type to include the other fields with a similar FK relationship.
You can see that this is not the case for fields that do not have their own tables synced via the connector. Here is an example of my including a field that has a FK (shows as and ID) and one that does not. Notice how summary is not an ID but the actual value since there is no raw summary table. Ideally, we could create an end model that includes the value as opposed to the ID for the fields with their own tables. However, that is out of scope for this ticket and something we should handle in a future update.

Thanks @fivetran-joemarkiewicz that makes sense now. Opening up the FR now!
I would say this is ready for re-review now too!


How do I dynamically change the item type in add_option type=“select”?

I think block.add_option can set the type of element to select as follows.
How can I receive this item candidate from node's input?
I would like to create a node that receives a data frame and selects a specific column within that data frame, as in the code example below. However, after the items are first rendered, they seem to remain the same regardless of the node's connection state.
def xy_selecter() -> Block:
    block = Block(name="XY Selecter")

    block.add_input(name="In(df)")
    block.add_output(name="Out(X, df)")
    block.add_output(name="Out(y, df)")

    df = block.get_interface(name="In(df)")
    if df is None:
        items = ["select y"]
    else:
        items = df.columns
    block.add_option(
        name="select y",
        type="select",
        items=items,
    )

    def compute_func(self: Any) -> None:
        df = self.get_interface(name="In(df)")
        y_col = self.get_interface(name="select y")
        y = df[y_col]
        X = df[df.columns != y_col]
        self.set_interface(name="Out(X, df)", value=X)
        self.set_interface(name="Out(y, df)", value=y)

    block.add_compute(compute_func)

    return block

hey @hodakamori , you can use the get_option method that is here: https://github.com/krish-adi/barfi/blob/04ddebb2ae6a518d77b58b68011724e4a3f6964d/barfi/block_builder.py#L169
i am working on the documentation to make this better. there is also a newer version for this project that is being built here: https://github.com/krish-adi/barfi/tree/migrate-to-react-flow

I am sorry it has taken so long.
And thank you for your comments.
Sorry, what I want to do is to take the information of the node's input and set it to the items of the option.
In other words, in this code, for example, when the columns of In(df) are [“a”, “b”, “c”], items will be [“a”, “b”, “c”], and when the columns are [“dog”, “cat”, “sheep”], items will be [“dog”, “cat”, “sheep”]. This is what we want.
However, the option items is always [“select y”], which is df is None behavior. From what I can see, it seems that the items in block.add_option are determined when the block is added and then do not change dynamically based on the node's connection state.
We would like to create a node where the user uploads a csv file, reads the csv as a dataframe, and then creates a node that selects a specific column and does some processing.


Relationship route error when relationship is marked as hidden

Steps to reproduce:

define route relationship, example:

		$server
			->resource('declarations', \App\Http\Controllers\Api\V1\DeclarationController::class)
			->only('store', 'show', 'update', 'delete')
			->relationships(function (Relationships $relationships) {
				$relationships->hasOne('patient')->only('related', 'show');
				$relationships->hasOne('signature');
			});

mark relationship in schema as hidden:

			HashId::make()->alreadyHashed(),
			DateTime::make('createdAt')->sortable()->readOnly(),
			DateTime::make('updatedAt')->sortable()->readOnly(),

			BelongsTo::make('patient')->readOnly()->hidden(),
			BelongsTo::make('signature')->type('attachments'),

try to access /api/resource/id/relation:

in my case: {{url}}/api/declarations/{{declaration_id}}/patient
Stack trace:
[2021-07-14 05:42:05] local.ERROR: Unexpected relationship patient on resource declarations. {"userId":10,"exception":"[object] (LogicException(code: 0): Unexpected relationship patient on resource declarations. at vendor\\laravel-json-api\\core\\src\\Core\\Resources\\JsonApiResource.php:267)
[stacktrace]
#0 vendor\\laravel-json-api\\core\\src\\Core\\Responses\\Internal\\RelatedResourceResponse.php(108): LaravelJsonApi\\Core\\Resources\\JsonApiResource->relationship('patient')
#1 vendor\\laravel-json-api\\core\\src\\Core\\Responses\\Internal\\RelatedResourceResponse.php(97): LaravelJsonApi\\Core\\Responses\\Internal\\RelatedResourceResponse->metaForRelationship()
#2 vendor\\laravel-json-api\\core\\src\\Core\\Responses\\Internal\\RelatedResourceResponse.php(81): LaravelJsonApi\\Core\\Responses\\Internal\\RelatedResourceResponse->allMeta()
#3 vendor\\laravel-json-api\\core\\src\\Core\\Responses\\RelatedResponse.php(110): LaravelJsonApi\\Core\\Responses\\Internal\\RelatedResourceResponse->toResponse(Object(Illuminate\\Http\\Request))
#4 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Router.php(776): LaravelJsonApi\\Core\\Responses\\RelatedResponse->toResponse(Object(Illuminate\\Http\\Request))
#5 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Router.php(763): Illuminate\\Routing\\Router::toResponse(Object(Illuminate\\Http\\Request), Object(LaravelJsonApi\\Core\\Responses\\RelatedResponse))
#6 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Router.php(695): Illuminate\\Routing\\Router->prepareResponse(Object(Illuminate\\Http\\Request), Object(LaravelJsonApi\\Core\\Responses\\RelatedResponse))
#7 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(128): Illuminate\\Routing\\Router->Illuminate\\Routing\\{closure}(Object(Illuminate\\Http\\Request))
#8 vendor\\laravel-json-api\\laravel\\src\\Http\\Middleware\\BootJsonApi.php(103): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#9 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): LaravelJsonApi\\Laravel\\Http\\Middleware\\BootJsonApi->handle(Object(Illuminate\\Http\\Request), Object(Closure), 'v1')
#10 app\\Http\\Middleware\\ProfileJsonResponse.php(22): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#11 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): App\\Http\\Middleware\\ProfileJsonResponse->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#12 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Middleware\\ThrottleRequests.php(127): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#13 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Middleware\\ThrottleRequests.php(103): Illuminate\\Routing\\Middleware\\ThrottleRequests->handleRequest(Object(Illuminate\\Http\\Request), Object(Closure), Array)
#14 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Middleware\\ThrottleRequests.php(55): Illuminate\\Routing\\Middleware\\ThrottleRequests->handleRequestUsingNamedLimiter(Object(Illuminate\\Http\\Request), Object(Closure), 'api', Object(Closure))
#15 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Illuminate\\Routing\\Middleware\\ThrottleRequests->handle(Object(Illuminate\\Http\\Request), Object(Closure), 'api')
#16 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(103): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#17 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Router.php(697): Illuminate\\Pipeline\\Pipeline->then(Object(Closure))
#18 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Router.php(672): Illuminate\\Routing\\Router->runRouteWithinStack(Object(Illuminate\\Routing\\Route), Object(Illuminate\\Http\\Request))
#19 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Router.php(636): Illuminate\\Routing\\Router->runRoute(Object(Illuminate\\Http\\Request), Object(Illuminate\\Routing\\Route))
#20 vendor\\laravel\\framework\\src\\Illuminate\\Routing\\Router.php(625): Illuminate\\Routing\\Router->dispatchToRoute(Object(Illuminate\\Http\\Request))
#21 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Kernel.php(166): Illuminate\\Routing\\Router->dispatch(Object(Illuminate\\Http\\Request))
#22 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(128): Illuminate\\Foundation\\Http\\Kernel->Illuminate\\Foundation\\Http\\{closure}(Object(Illuminate\\Http\\Request))
#23 vendor\\barryvdh\\laravel-debugbar\\src\\Middleware\\InjectDebugbar.php(67): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#24 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Barryvdh\\Debugbar\\Middleware\\InjectDebugbar->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#25 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Middleware\\TransformsRequest.php(21): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#26 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Middleware\\ConvertEmptyStringsToNull.php(31): Illuminate\\Foundation\\Http\\Middleware\\TransformsRequest->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#27 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Illuminate\\Foundation\\Http\\Middleware\\ConvertEmptyStringsToNull->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#28 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Middleware\\TransformsRequest.php(21): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#29 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Middleware\\TrimStrings.php(40): Illuminate\\Foundation\\Http\\Middleware\\TransformsRequest->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#30 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Illuminate\\Foundation\\Http\\Middleware\\TrimStrings->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#31 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Middleware\\ValidatePostSize.php(27): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#32 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Illuminate\\Foundation\\Http\\Middleware\\ValidatePostSize->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#33 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Middleware\\PreventRequestsDuringMaintenance.php(86): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#34 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Illuminate\\Foundation\\Http\\Middleware\\PreventRequestsDuringMaintenance->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#35 vendor\\fruitcake\\laravel-cors\\src\\HandleCors.php(52): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#36 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Fruitcake\\Cors\\HandleCors->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#37 vendor\\fideloper\\proxy\\src\\TrustProxies.php(57): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#38 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(167): Fideloper\\Proxy\\TrustProxies->handle(Object(Illuminate\\Http\\Request), Object(Closure))
#39 vendor\\laravel\\framework\\src\\Illuminate\\Pipeline\\Pipeline.php(103): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(Illuminate\\Http\\Request))
#40 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Kernel.php(141): Illuminate\\Pipeline\\Pipeline->then(Object(Closure))
#41 vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Kernel.php(110): Illuminate\\Foundation\\Http\\Kernel->sendRequestThroughRouter(Object(Illuminate\\Http\\Request))
#42 public\\index.php(52): Illuminate\\Foundation\\Http\\Kernel->handle(Object(Illuminate\\Http\\Request))
#43 {main}
"}

Looking at this, even if we fix it at the point this error is occurring, there's a chance it might also then fail further down the encoding stack.
Primarily here:
https://github.com/laravel-json-api/encoder-neomerx/blob/f41bb36f2fa32e35f41116425a402eaf88abec7d/src/Schema/Schema.php#L182
And then again here:
https://github.com/laravel-json-api/encoder-neomerx/blob/f41bb36f2fa32e35f41116425a402eaf88abec7d/src/Schema/Schema.php#L202
Though the code-path for that might be for relationship identifiers, rather than related resource. However the problem is the same - as the relationship field is hidden, we cannot acquire the information from the resource.
The problem with the encoder package is the return type isn't nullable - you have to return a link. This presents a significant challenge for this scenario.
Workaround:
The work-around is to not hide the relationship and have it as a links-only relationship. I.e. the relationship is not hidden, but uses the cannotEagerLoad() method to ensure it can never have data. This means the relationship still appears in the resource but only has the links member. This is semantically-correct - i.e. it exposes the links for the relationship so the client can programmatically obtain the relationship value if needed.
This actually explains why I've never encountered this problem... because I always use links-only relationships rather than hiding relationships.

Potential Solution
Potentially the $resource->relationship($name) method could return a relationship object even if the relationship is hidden. I.e. we use $resource->relationships() when encoding, so perhaps the $resource->relationship() method could be used to obtain relationship information even if the relationship is hidden. The resource can do this because it has access to the underlying schema.
This would be preferrable to exposing the schema, which is not desirable because the use of the schema by the resource is an internal implementation detail. I.e. calling code shouldn't expect the resource to have the schema.

Actually, #111 would be a good solution for this - though we'd have to make the relationship() method on the resource class return a nullable type to fix. Worth investigating.

Closing as will be in the 1.1 release.


QG4

@adkumar1 next time add a proper PR description. QG4 is not desprictpive for anyone outside of catena. Also be awre of my comment


You can add IVONA to 'Supported Services'

Hi,
Awesome project, thanks a bunch - you saved me a lot of hassle : )
I just wanted to let you know that you can add IVONA Speech Cloud API to list of supported services in README - I use it here and it works like a charm.

Thanks @pawelad, I'll include it in the next release


Event.StateDidSwitch Firing at the Wrong Time (Twice, and with the wrong from/to)

Below is the code for a prototype made in Framer Studio, click on the green layer while the console is open to see the problem. It looks like Event.StateWillSwitch and Event.StateDidSwitch are not firing at the right times/in the right order.
The code below uses the technique referenced in https://github.com/koenbok/Framer/issues/111, which would be just fine if the events fired at the right time/in the right order. This bug is currently making it very difficult to get chained animations to work.
setVisibilityWhenAnimating = (layer, showStateName = 'showing', hideStateName = 'hidden') ->
	layer.on Events.StateWillSwitch, (from, to) ->
		console.log new Date(), 'Will switch ' + from + ' => ' + to
		if to is showStateName or to is 'default'
			layer.visible = true
			console.log 'will show, should set visible true (' + from + ' => ' + to + ')'
		return

	layer.on Events.StateDidSwitch, (from, to) ->
		console.log new Date(), 'Did switch ' + from + ' => ' + to
		if to is hideStateName
			layer.visible = false
			console.log 'just hid, should set visible false (' + from + ' => ' + to + ')'
		return

	return

animatingLayer = new Layer
	x: 0
	y: 100
	width: 100
	height: 100
	backgroundColor: 'green'

animatingLayer.states.add
	showing: {x: 100, opacity: 1}
	hidden: {x: 200, opacity: 0}
animatingLayer.states.switchInstant 'hidden'

controlLayer = new Layer
	x: 100
	y: 500
	width: 100
	height: 100

controlLayer.on Events.Click, ->
	console.log new Date(), 'Click!'
	animatingLayer.states.next("hidden", "showing")

setVisibilityWhenAnimating animatingLayer

console.log animatingLayer.states.current

setVisibilityWhenAnimating animatingLayer

Looks like one main problem is that it looks like the Events.StateDidSwitch event is happening twice. It's wrong the first time, saying there was a switch from the from state to the from state, and then correctly with the right from and to.

:point_up:

🎶 it's been soon long 🎶

Bumpity bump bump bump.


error: invalid command 'bdist_wheel'

Environment:
Ubuntu18.04
python3.6.5
When i install flask after i created virtual environment using python3 -m venv venv, this error occurs:
Collecting flask
Using cached https://files.pythonhosted.org/packages/7f/e7/08578774ed4536d3242b14dacb4696386634607af824ea997202cd0edb4b/Flask-1.0.2-py2.py3-none-any.whl
Collecting Jinja2>=2.10 (from flask)
Using cached https://files.pythonhosted.org/packages/7f/ff/ae64bacdfc95f27a016a7bed8e8686763ba4d277a78ca76f32659220a731/Jinja2-2.10-py2.py3-none-any.whl
Collecting click>=5.1 (from flask)
Using cached https://files.pythonhosted.org/packages/34/c1/8806f99713ddb993c5366c362b2f908f18269f8d792aff1abfd700775a77/click-6.7-py2.py3-none-any.whl
Collecting itsdangerous>=0.24 (from flask)
Using cached https://files.pythonhosted.org/packages/dc/b4/a60bcdba945c00f6d608d8975131ab3f25b22f2bcfe1dab221165194b2d4/itsdangerous-0.24.tar.gz
Collecting Werkzeug>=0.14 (from flask)
Using cached https://files.pythonhosted.org/packages/20/c4/12e3e56473e52375aa29c4764e70d1b8f3efa6682bef8d0aae04fe335243/Werkzeug-0.14.1-py2.py3-none-any.whl
Collecting MarkupSafe>=0.23 (from Jinja2>=2.10->flask)
Using cached https://files.pythonhosted.org/packages/4d/de/32d741db316d8fdb7680822dd37001ef7a448255de9699ab4bfcbdf4172b/MarkupSafe-1.0.tar.gz
Building wheels for collected packages: itsdangerous, MarkupSafe
Running setup.py bdist_wheel for itsdangerous ... error
Complete output from command /home/bl/flasky/venv/bin/python3.6 -u -c "import setuptools, tokenize;file='/tmp/pip-build-zgmblvtp/itsdangerous/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" bdist_wheel -d /tmp/tmp14_h7_l2pip-wheel- --python-tag cp36:
usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: -c --help [cmd1 cmd2 ...]
or: -c --help-commands
or: -c cmd --help
error: invalid command 'bdist_wheel'

Failed building wheel for itsdangerous
Running setup.py clean for itsdangerous
Running setup.py bdist_wheel for MarkupSafe ... error
Complete output from command /home/bl/flasky/venv/bin/python3.6 -u -c "import setuptools, tokenize;file='/tmp/pip-build-zgmblvtp/MarkupSafe/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" bdist_wheel -d /tmp/tmpj3po5ahmpip-wheel- --python-tag cp36:
usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: -c --help [cmd1 cmd2 ...]
or: -c --help-commands
or: -c cmd --help
error: invalid command 'bdist_wheel'

Failed building wheel for MarkupSafe
Running setup.py clean for MarkupSafe
Failed to build itsdangerous MarkupSafe
Installing collected packages: MarkupSafe, Jinja2, click, itsdangerous, Werkzeug, flask
Running setup.py install for MarkupSafe ... done
Running setup.py install for itsdangerous ... done
Successfully installed Jinja2-2.10 MarkupSafe-1.0 Werkzeug-0.14.1 click-6.7 flask-1.0.2 itsdangerous-0.24
pip list:
click (6.7)
Flask (1.0.2)
Flask-Script (2.0.6)
itsdangerous (0.24)
Jinja2 (2.10)
MarkupSafe (1.0)
pip (9.0.1)
pkg-resources (0.0.0)
setuptools (39.0.1)
Werkzeug (0.14.1)
Remark:
It's fine when i use virtualenv -p /usr/bin/python3.6 venv then install flask
I saw you use python3 -m venv venv here:
https://blog.miguelgrinberg.com/post/the-flask-mega-tutorial-part-xvii-deployment-on-linux-even-on-the-raspberry-pi

This could be related to changes in newer versions of pip since the book was released. Run pip install --upgrade pip wheel to make sure you have the latest version of the installer, then try the pip install command again.

This wasn't so much of a problem in the past, but now it seems it is always a good idea to upgrade pip and wheel when you create a new virtual environment. If you prefer to use the virtualenv command, that's fine too, nothing wrong with that.

@miguelgrinberg ok, thanks a lot for your help


Remove maturity label from main charts csv

Remove maturity label from main charts csv inside chaos-charts and retain maturity label only at
experiment level.

fixed by : https://github.com/litmuschaos/chaos-charts/pull/73


Feature/PizzaPriceRestock

add a boolean special price for categories
if this special price is true then during the restock the all price will be Ceten price except for externe which will be externe

how to test this pr:

go to admin/categories
check a special price checkbox for a category
go to panel/products/reappro
add a product which is in the category you check (has special price) and one not in a category with special price
go to admin/produits
check that the price of the item with special price are correct (all price will be Ceten price except for externe which will be externe) and the other

Special case handle:

if a item don't have a category it give a one by default with "basic" price

lgtm, juste ce serait bien qu'on puisse choisir de mettre les prix spéciaux quand on crée une catégorie, je merge cette PR et j'ouvre une issue pour ça.


[Feature] Make light overlay show only above solid blocks

It's a bit annoying to see light overlay is above vines, grass, and many other blocks, that do not have collision boxes, so mobs can still spawn here. It would look much better if overlay would show only above solid blocks, as like air above.
How it looks like now:

How I want it to look like:

Sounds reasonable

However there are currently no ways to differentiate between snow layers and let's say torches. as snow layers actually cover the overlay

Make an exclusion for snow or similar blocks, that also cover the solid block and do not have collision box.

I feel like I will do the opposite the of that, make an exclusion for flowers and torches to not offset the display.

Is there any update on this enhancement? 😃


feat: import jsx function from hono/jsx/dom/jsx-runtime

Fixes #33

Hi @usualoma !
Thanks for your first PR for HonoX! Exactly, we should use hono/jsx/dom/jsx-runtime. I'll merge this now.


Add Bank Storage + EMI support

This PR adds support for Bank Storage's "banks" as item suppliers for construction wands, as well as EMI as an alternative to REI for viewing item info.

Hey there!
Thank you for your work! More mod compatibility is always welcome, especially since Fabric doesn't really have one universal API for this kind of stuff.

After further testing, it looks like this PR does not function properly when the client is connected to a dedicated server, wrt the Bank Storage integration :s
I think it's because CachedBankStorage is used client-side, while something else is used server-side? The mod doesn't have an API, unfortunately, so I'll have to take a closer look at it.

I think I managed to fix it in 890fe54. It is a bit awkward because using Util.getBankItemStorage(inventoryStack, player.getWorld()); seems to crash the game when used in single player, so it will use CachedBankStorage instead.
I don't really know this mod though, so please let me know if I missed something.

I managed to fix it on both singleplayer and multiplayer worlds with https://github.com/unilock/ConstructionWand-Fabric/commit/764596d5234a68f7a4c847b92e52532b859077e8. This also fixes the item count in banks not updating until observed in multiplayer.
If you want to revert that last commit, I can make another pull request, or you can just merge the changes manually :)

Great! Please create another PR and I will gladly merge it!


Update to discord.js v13

Woah ! I did not have the time to release version 1.2.0 including discord.js v12 and it is already time to move to v13 🚨 !

v14 here 😰.


refactor: Update product-config

Part of https://github.com/stackabletech/product-config/issues/95

A changelog entry would be nice, as this is not a random third-party crate, but a core Stackable library


Tool description in help

Pdtm is a golang-based tool designed to simplify the management of open source projects.
It offers a straightforward way to install, update, and remove tools.
By ProjectDiscovery

@xm1k3 not sure about By ProjectDiscovery part . since it is already mentioned in pdtm banner .
Above description sounds like pdtm manages all open source projects .
I think this line from README.md should be enough

pdtm is a simple and easy-to-use golang based tool for managing open source projects from ProjectDiscovery


New Search Field

Add new search field at the top, and it allow user to search Name/Title/Description.

This looks cool, nice work!
It is a tiny bit slow though. Maybe we can reduce the searched fields to just the name, as (at least in this project) that is the one with the data. I also might consider replacing _ with SPACE in the data to help it parse those long names. What do you think?

CC @cmolivi

:book: Reference #23

I am thinking does it need a button to click instead of auto-filter? Like top right menu, click to filter.

Awesome work @taojun545 it seems faster now.


Fancy Zones doesn't respect Remote Deskop Remote App Windows

Microsoft PowerToys version
0.75.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce
Set up and initiate a Fancy Zones layout, drag Remote Apps (applications published from AVD for example) across screens, they don't snap to a zones.
✔ Expected Behavior
Remote Apps (applications published from AVD for example) snap to Fancy Zones when dragged across the monitor.
❌ Actual Behavior
Remote Apps (applications published from AVD for example) snap to Fancy Zones when dragged across the monitor.
Other Software
AVD (Azure Virtual Desktop) Remote App

There is a compatibility issue, the app steals all the keyboard/mouse events, so FancyZones doesn't receive them to process.
https://github.com/microsoft/PowerToys/wiki/Application-Compatibility#applications-that-cannot-be-zoned
/dup https://github.com/microsoft/PowerToys/issues/441


feat: add snapshot to addressResolvers

Fix #262
Lookup snapshot user's name from addresses
🛠 Tests
Run
curl --location 'http://localhost:3008' \
--header 'Content-Type: application/json' \
--data '{
    "method": "lookup_addresses",
    "params": [
        "0x91fd2c8d24767db4Ece7069AA27832ffaf8590f3", "0xeF8305E140ac520225DAf050e2f71d5fBcC543e7", "0x24F15402C6Bb870554489b2fd2049A85d75B982f", "0x76A6D08b82034b397E7e09dAe4377C18F132BbB8"
    ]
}'

It should return a list of addresses, with the username from snapshot for the first 3 addresses, and ENS name for the last one

mhhh. wait , test are failing

Test passing now, just a temp thing


Ingress controller: robust validation

The ingress controller accepts many inputs from the user to be put into envoy config: a list of hosts is arbitrary, TLS certs are just files. Some errors can be caught during validation, but some are only discovered at the ingress proxy. We should make sure that ingress proxy does not crash on a bad ingress to provide ingress resource isolation. There are two places where this is lacking:

envoy requires hosts to be distinct
envoy crashes on invalid cert files

not as big of an issue with LDS/RDS


Configuring the font size on x- and y-axis

Is it possible to change the font size on one or both axes? I have not been able to discover how, if it is possible. Thanks.

I've been looking through the ApexCharts docs to see if there's any option for it: https://apexcharts.com/docs
I haven't found anything yet.
I've left a request on AxisCharts to see if this is possible: https://github.com/apexcharts/apexcharts.js/issues/684

Ok, it's possible to do this with ApexCharts, but I'll need to wire it up for you. Get back to you soon.

Thanks for looking into that

You can now configure font size and family on x axis, y axis and the legend.
Be sure to use Data-Forge Plot v0.6.0.
Please see example 12 to see how to use it.

If features are missing please keep asking for them! That's what drives this forward.

Hi, Thanks very much for the help. I looked at example 12 and I think there's a small understanding. Using the example to guide me, I can change the x-axis label font, but I am actually interested in the font of the labels at the tick marks. For example, in this example I would want to control the font size of the month names (Jan, Feb, etc.). Is that also possible already in v0.6.0?

Ok I see! I should have covered that case as well. Give me a day or two to look into it!

Check out Data-Forge Plot version 0.6.2.
I've updated example 12 to show how to configure the font for tick marks.
Please let me know if this does what you need.

Yes, it's working exactly the way I wanted. Thanks :-).

That's great.
Please be sure to star my repos if you haven't already:
https://github.com/data-forge/data-forge-ts
https://github.com/data-forge/data-forge-plot
Thanks!


Expose manageUI.lua locals (see description)

Please expose the following locals to the module level:

modeInfos
mainContentView
uiObjectsByModeType

Hi Lich! No problem, exposed in <IP_ADDRESS>


Question: please what do lg, xs and md means in Responsive Grid

Hi awesome developer,
Please can you help me out, I need to understand what lg, xs, and md means in Responsive Grid so that when I'm using such properties, I will know where it fits in.
Thanks for your awesome feedback.

the library is meant to be used the same way as Bootstrap web framework
for each size category you provide the ratio it should fill from the width assuming its total width is 12
and it applies for larger size category unless overridden
xs: extra small
sm: small
md: medium
lg: large
xl: extra large
the threshold widths that determines the current size category is the same as Bootstrap's
xs: width < 576
sm: width < 768
md: width < 992
lg: width < 1200
xl:  width >= 1200
ex: say you want to make two containers to stack on small width, but starts to place side by side starting from large width, set:
xs: 12  and lg: 6
or just
lg: 6
only, because it's always 12 by default
hope this explanation helps :)
more info on bootstrap grid system:
https://getbootstrap.com/docs/3.3/css/#grid

thanks so much for the feedback.


ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved

Recurrring instances of this error in the logs following integration installation.
2021-12-10 08:28:45 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 134, in _handle_refresh_interval
    await self._async_refresh(log_failures=True, scheduled=True)
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 265, in _async_refresh
    update_callback()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 325, in _handle_coordinator_update
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 486, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 519, in _async_write_ha_state
    state = self._stringify_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 492, in _stringify_state
    if (state := self.state) is None:
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 273, in state
    value = self.native_value
  File "/config/custom_components/sunpower/sensor.py", line 213, in native_value
    return self.coordinator.data[METER_DEVICE_TYPE][self.base_unique_id][self._field]
KeyError: 'Power Meter'

Power meter not yet providing data:

HA 2021.11.5, HAOS 6.6, Supervisor 2021.10.8

the parsing robustness should have fixed this sort of error, feel free to reopen if this is seen again.


Error: SerdeJsonError(Error("missing field price", line: 1, column: 949))

Describe the bug
When price info is missing json parsing fails.
To reproduce
Steps to reproduce the behavior:

Configure env / fusion key to work in example business_search
Modify payload like so:

    let business_search_payload: BusinessSearchPayload = BusinessSearchPayload::builder()
        .coordinates(Coordinates::new(50.18841046, -119.3882348))
        .radius(300)
        .categories(vec![String::from("bathroom")])
        .limit(10)
        .build()
        .unwrap();

Run example + see error

Expected behavior
The optional data should resolve to None.
    /// Price level of the business.
    ///
    /// Value is one of $, $$, $$$ and $$$$.
    pub price: Option<PriceType>,

Screenshots

Note:
This repository seems locked down, I cannot make a branch.

Sorry for the long wait on this Issue. I'll be working on it this weekend.


octavia-ingress-controller: Add provider configuration

What this PR does / why we need it:
Add provider support in octavia-ingress-controller configuration.
For more details about provider in Octavia: https://docs.openstack.org/octavia/latest/admin/providers.html
An example config file:
cluster-name: devstack-k8s
kubernetes:
    api-host:
    kubeconfig: /home/ubuntu/.kube/config
openstack:
    user-id: ${user_id}
    password: password
    project-id: ${project_id}
    auth-url: ${auth_url}/v3
    region: RegionOne
octavia:
    subnet-id: ${subnet_id}
    floating-network-id: ${public_net_id}
    manage-security-groups: false
    provider: amphora

Which issue this PR fixes: fixes #610
Special notes for your reviewer:
Release note:

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/lgtm
#610 indicate the bug already fixed 👍

/retest

/test cloud-provider-openstack-unittest

/approve


query param false positive for expressions with no variable use

Example code
<cfquery name="queryName" datasource="#datasourceName#">
update users
set updated = "#createODBCDateTime(now())#"
</cfquery>

Note there no (reasonable) way SQL injection could happen there.

Should we report on this then? I can't even think of a "weird" way of abusing the lack of queryparams here.

The previous comment is for createODBCDateTime(now())specially. I suppose there might be scenarios in which one might have an expression without a variable that could cause a problem.

I can't think of any. The SQL injection is only  likely to only occur with function arguments, form and url scopes anyway and not all variables anyway. I picked the above because it seem fairly common thing to do.
I think another area where it comes up is when setting boolean / int  values like so:
<cfif someValue>1<cfelse>0</cfif>

I guess reporting those scenarios would then still be useful, right? Even though no injection risk. And people can no ignore them using the annotation if they don't want to deal with it.

The issue with ignoring them it that it ignore them all right including valid ones right?

no, we've in the meantime have implemented an annotation-based ignore. Check "Annotation-based configuration" in README.

So, this:
<cfquery name="queryName" datasource="#datasourceName#">
update users
<!--- @CFLintIgnore CFQUERYPARAM_REQ --->
set updated = "#createODBCDateTime(now())#"
</cfquery>

if you don't want to deal with it.

Thanks think we can close the one

Even with no risk of SQL injection, it's still usually a good idea to parameterize to create a reusable query plan. There are some other minor benefits as well. Is the rule only concerned with SQL injection?

ok, @justinmclean, ta! :)

@KamasamaK Yeah, I agree. So, we'll leave as is, it will report and who doesn't want it to, needs to ignore it.


broken doc link

this link appears to be broken today, July 23
http://kubernetes.io/v1.0/docs/api-reference/operations.html
I tried to click on 'Kubernetes API Reference' in the left nav bar on the main documentation
site and it shows the above as the URL.  Nothing shows in the right side panel.
http://kubernetes.io/v1.0/docs/api-reference/operations.html

cc @lavalamp

@RichieEscarez

fix by #11717


Set interfaces routes after the interface is up

Fixes #485
The code previously relied on an uninteded side effect. When the
interface name was set, this causes the interface to come up
prematurely. Once that side effect was removed, routes could
no longer be set.
This change ensures that routes are only set after the interface
is brought up.
ping @mrjana @mavenugo ptal

LGTM

Thanks @tomdee. LGTM


tf.py_function could return a dictionary of tensors

Usually, a transformers tokenizer encodes an input as a dictionary.
{"input_ids": tf.int32, "attention_mask": tf.int32, "token_type_ids": tf.int32}

And to archive better performance handling with a large dataset, is a good practice implement a pipeline which includes using Dataset.map to apply a tokenizer function to each element of an input dataset. Exactly the same as done in the Tensorflow tutorial: Load text.
However, the tf.py_function (used to wrap the map python function) doesn't support returning a dictionary of tensors as shown above.
For instance, if the tokenizer (encoder) in the Load text returns the following dictionary:
{
    "input_ids": [ 101, 13366,  2131,  1035,  6819,  2094,  1035,  102 ],
    "attention_mask": [ 1, 1, 1, 1, 1, 1, 1, 1 ]
}

how can someone set the Tout parameter of the tf.py_function to get the desired dictionary of tensors:
{
    'input_ids': <tf.Tensor: shape=(16,), dtype=int32, numpy = array(
    [ 101, 13366,  2131,  1035,  6819,  2094,  1035,  102 ], dtype=int32)>

    'attention_mask': <tf.Tensor: shape=(16,), dtype=int32, numpy=array(
     [ 1, 1, 1, 1, 1, 1, 1, 1 ], dtype=int32)>
}

?

I think this would be a nice feature but I think you can work around this limitation by constructing your py_function appropriately as tf.data supports dictionaries
python```
import tensorflow as tf
def tokenizer(x):
return { "input_ids": [ 101, 13366,  2131,  1035,  6819,  2094,  1035,  102 ], "attention_mask": [ 1, 1, 1, 1, 1, 1, 1, 1 ]}
def py_func(x):
d = tokenizer(x)
return list(d.values())
def ds_map_fn(x):
flattened_output = tf.py_function(py_func, [x], [tf.int32, tf.int32])
return {"input_ids": flattened_output[0], "attention_mask": flattened_output[1]}
ds = tf.data.Dataset.range(2)
ds = ds.map(ds_map_fn)
for value in ds:
print(value)

See also the related issue #27679, concerned with py_function accepting dictionaries of tensors. I would really like to see both features implemented in TensorFlow. In real-world problems, data with heterogeneous features is quite common, and managing such features is made unnecessarily complicated by poor support for feature dictionaries.

@rohan100jain,
Although the limitation of   tf.py_function I could use it to encode my dataset. Thank you.

@novog, it would be great.

Absolutely. It is very useful.


Support aggregations on a date field

This PR should fix #42
I intentionally ran this against an ES6 host, and learned more about elasticsearch :smile: .

Hi @giganteous, good catch!
I'd like to use an alternate fix - checking the type of the values before adding them to metrics. It's a more general solution - I probably shouldn't have been blindly adding values in the first place...
Thanks for the PR, but for simplicities sake, I'll close it and implement the other fix separately.


cmd/go: emits warning when GOPATH == GOROOT

Go version
go version go1.22.0 linux/amd64
Output of go env in your module/workspace:
warning: GOPATH set to GOROOT (/home/kumaneko/go) has no effect
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/home/kumaneko/.cache/go-build'
GOENV='/home/kumaneko/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/kumaneko/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/kumaneko/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/home/kumaneko/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/home/kumaneko/go/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.22.0'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build871568154=/tmp/go-build -gno-record-gcc-switches'

What did you do?
I noticed that go 1.22 started emitting
  warning: GOPATH set to GOROOT ($HOME/go) has no effect

and VSCode plugin noisily emits "Failed to run ..." messages
due to this warning message.
Installing under $HOME worked without problems until go 1.21.7.
Also, full bisection between go1.21.0 and go1.22.0 pointed at
commit fa72f3e034fd ("[release-branch.go1.22] go1.22rc1").
If this is an intentional change, please update https://go.dev/doc/install
to explicitly comment that installing under $HOME is not possible.
Or maybe remove this warning from src/cmd/go/main.go , for this warning
is not fatal enough to justify "Failed to run ..." messages?
What did you see happen?
$ tar -C ~ -xzf go1.22.0.linux-amd64.tar.gz
$ export PATH=$PATH:~/go/bin/
$ go version
warning: GOPATH set to GOROOT (/home/kumaneko/go) has no effect
go version go1.22.0 linux/amd64
What did you expect to see?
$ tar -C ~ -xzf go1.22.0.linux-amd64.tar.gz
$ export PATH=$PATH:~/go/bin/
$ go version
go version go1.22.0 linux/amd64

If Go is installed in /home/kumaneko/go (GOROOT), then GOPATH must be set to a different directory.

What does "must" here mean? Did the meaning of "must" changed in 1.22?
No warning was emitted and plugins silently worked until 1.21.7, without making GOPATH and GOROOT different.
Interestingly, no warning is emitted on current head of go.git tree.
go version devel go1.23-4a7f3ac8eb Sat Feb 10 02:14:22 2024 +0000 linux/amd64
This warning is emitted if I do
$ echo 'go1.23devel' > VERSION
$ echo 'time 2024-02-11T15:14:00Z' >> VERSION
before building current head of go.git tree.
Why does creating the VERSION file cause this warning message to appear?

The VERSION file has no effect, your testing of it is wrong.
I believe this is working as intended,
the warning has been in place for a long time, but cmd/go wasn't emitting the warning previously when it should have. See CL 543955.

The VERSION file has no effect, your testing of it is wrong.

The VERSION file indeed has some effect. Full bisection between go1.21rc1~1 and
go1.22rc1~1 with the VERSION file added pointed at commit 841e63e480ca ("cmd/go:
don't warn about GOROOT equal to GOPATH when both are the empty string").

I believe this is working as intended, the warning has been in place for
a long time, but cmd/go wasn't emitting the warning previously when it
should have. See CL 543955.

Since I was able to use go without problems using default GOPATH value, I don't
know what will break by setting GOPATH to a different location. What value is
recommended for GOPATH when GOROOT is set to $HOME/go ?
My expectation is that we don't need to take care of GOPATH and GOROOT because
GO111MODULE is enabled by default since 1.16. Then, I expect cmd/go not to emit
this warning, for emitting warnings on unused setting is confusing/misleading.

Although GOPATH is not used as much for modules, it is still used.  For example, go install will install commands into $GOPATH/bin.
If you use $HOME/go for GOROOT, there is no particular recommended value for GOPATH.  Personally I use $HOME/gopath.

The warning is confusing because (a) it triggers with a default configuration (b) it doesn't describe how to resolve the warning.
A fresh install of go, with no env vars set in .profile etc, triggers a warning.
A better warning would be "GOPATH must be set to a dir that is distinct from GOROOT".
I hunted everywhere in my shell and vs code config to see where I was setting GOPATH, only to realize that the default settings are triggering the warning.
# make sure env is cleared
$ GOROOT= GOPATH= env | egrep -i 'goroot|gopath'
GOROOT=
GOPATH=

# clear GOROOT, GOPATH and confirm go uses default settings and warns
$  GOROOT= GOPATH= go env | egrep -i 'goroot|gopath'
warning: GOPATH set to GOROOT (/home/tonymet/go) has no effect
GOPATH='/home/tonymet/go'
GOROOT='/home/tonymet/go'

# confirm go warns with default settings for all commands
$ GOROOT= GOPATH= go version
warning: GOPATH set to GOROOT (/home/tonymet/go) has no effect
go version go1.23.2 linux/amd64

https://github.com/golang/go/issues/70092


Plugin "ChromeExtensionLiveReloadPlugin": append js-code to the chunk.

Hi. I write plugin "ChromeExtensionLiveReloadPlugin" for autoreloading the extension after files change (webpack watching):
https://github.com/WebDevArchive/Webpack4ChromeExtension/blob/master/utils/webpack/ChromeExtensionLiveReloadPlugin/ChromeExtensionLiveReloadPlugin.js
Is this a correct way to append a code to the chunk (background.js)?

You can write a plugin that adds code to the webpack runtime.
I. e. here is how to HMR Plugin adds it's code to the runtime: https://github.com/webpack/webpack/blob/master/lib/HotModuleReplacementPlugin.js#L129-L139

Thanks, Sokra.
Sorry for my English.
I don't quite understand - can I use mechanism of HotModuleReplacement or not? Will it really reload the chrome extension on the fly?
My thoughts:
To reload the extension when the file changes, call Chrome API chrome.runtime.reload() from the background page (background.js).
Two ways how to tell background.js when to do it:

Make the check (of the file with build-timestamp) via setInterval and XMLHttpRequest.
Sockets (webpack-dev-server?) // overhead in my opinion.
In my plugin, I just embed js-code with "chrome.runtime.reload()" to compilation.assets.
And it works well. Do you think this is the wrong way?


change on broker's autogen files

bin/bazel_to_go.py makes this change, and it seems similar change
happened as #1529, so I guess this change should also be accepted?
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

none

/assign @xiaolanz @rkpagadala

@douglas-reid is this the same change that you checked in?

/lgtm

@mandarjog yes. this is the exact same change. i'm fine with going with this version of the change.

/retest

can someone in toplevel owner approve this?  (seems no OWNERS file in broker -- but why?)

can someone in toplevel owner approve this?  (seems no OWNERS file in broker -- but why?)

I will add an OWNER file to broker.

This is checking in an autogen file.


Internal Server Error during import first spreadsheet oh data.

Hi all,p
sorry for the loose error description, but it's all the system says.
I'm trying to import a spreadsheet of data. In the config.properties I've specified
data.import.mode = IMPORT
server logs are in the following (along with the test spreadsheet)
ROOT.2020-11-19.log
localhost_access_log.2020-11-19.txt
germplasm-mcpd.xlsx
Any suggestion?
Thanks in advance.
/gp

Can you please verify which version of the Germinate Docker you are using?

(As a comment though, you're using an MCPD file but uploading it via the "genotype" tab, which isn't going to work. This is unrelated to the error cause though).

Thanks, I'm using 4.1.1 (installed a couple of days ago with docker-compose using most settings from the suggested .yml file)
/gp

Would you be able to run some commands for me please?
docker exec -it <YOUR_DOCKER_IMAGE_NAME> bash

cd /usr/local/tomcat/webapps/ROOT/WEB-INF/lib/

ls -l

And post the output here, please?

sure!
it is in the following attachment
web-inf.20201120.txt
thank you
/gp

Now that is weird. The error message complains about a method (FileUtils.copyInputStreamToFile) not being available during runtime. It's part of the Apache Commons IO library, which is included in your listing (commons-io-2.6.jar)...
I'll try and run a fresh setup of the 4.1.1 docker image locally and see if I can narrow it down.

Ok, so I ran a completely fresh setup with Germinate 4.1.1 and Gatekeeper 4.1.1 inside a Docker compose each with their own MySQL docker image and the data upload/import works fine. I'm not sure why it isn't for you.
Can you try restarting the docker setup?

ok,
I've given,
docker-compose stop
docker-compose down
docker volume rm
docker system prune
and restarted with docker-compose up.
Same result (Internal Server Error). If you have specific commands to cleanup, my containers are named "germinate" and "mysql_germi" for germinate, "gatekeeper" and "mysql_gatek" for gatekeeper: feel free to suggest them to me.
Thanks a lot.

What OS are you running this on?
Can you try and remove the germinate container and image so that it re-downloads when you restart the docker-compose?
Are there any other error messages in the log files?
I've attached my test run configuration that works fine.
germinate.zip

Hi,
I'm running ubuntu 20.04 LTS.
With the docker commands I've already given (see previous post), it re-downloaded images, for that I understand: if you have other cleanup commands feel free to suggest them to me.
I've seen the same error message: here are the new logs
localhost_access_log.2020-11-20.txt
ROOT.2020-11-20.log
But now I try with your very same configuration. Stay tuned.

UPDATE: cleaned up volules/containers/images and tested with your very test run configuration - the one you attached here in this issue.
Docker downloaded all images from network and started.
THEN:

Under gatekeeper I've added another database, the "germinate" one (of the test run configuration), and a new user with administrative privileges on it.
From germinate, I've tried to upload mcpd data: same result.

The java runtime is downloaded along with the container, so it should not be a java issue... but why this error?
Please let me know what have I to try.
Best Regards
/gp

That's the thing I don't understand. The docker image contains the java runtime and Germinate contains all its dependencies (the ones I asked you to list here). The dependency that contains the class/method causing the error is present, so I don't see how it would not work, especially because I can run the same docker image with the same configuration here without any issues.
Is there another machine that you could try to run this on? Just to see if it makes any difference?

sure, good idea.
the ubuntu machine I'm using is a virtual one running under virtual box (running ubuntu as host) - so I could fire up another linux flavour, or ubuntu release. please let me know what distro/release you prefear.
thanks a lot.
/gp

It shouldn't matter which distro or release. I'm just trying to figure out if there is some remnant of a corrupted image/container/whatever that's causing this.

IT WORKS!
a new ubuntu 20.04 LTS. a fresh docker-compose from your test run configuration. I was able to load the first sample spreadsheet of mcpd data (from github repo).
Thanks a lot very much. I close this issue.
/gp

Glad it's working. Docker sometimes has its hiccups...


Warnings when building the examples

When trying to build the examples, seeing a lot of warnings:
~/workspace/temp/uwebsockets/uWebSockets $ CXX=g++-11 ./build examples
--> g++-11 -O3 -Wpedantic -Wall -Wextra -Wsign-conversion -Wconversion -std=c++20 -Isrc -IuSockets/src -flto examples/Client.cpp  uSockets/*.o -lz -o Client

In file included from examples/Client.cpp:1:
src/ClientApp.h: In constructor 'uWS::ClientApp::ClientApp(uWS::WebSocketClientBehavior&&)':
src/ClientApp.h:21:45: warning: unused parameter 'behavior' [-Wunused-parameter]
   21 |         ClientApp(WebSocketClientBehavior &&behavior) {
      |                   ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
src/ClientApp.h: In member function 'uWS::ClientApp&& uWS::ClientApp::connect(std::string, std::string)':
src/ClientApp.h:25:41: warning: unused parameter 'url' [-Wunused-parameter]
   25 |         ClientApp &&connect(std::string url, std::string protocol = "") {
      |                             ~~~~~~~~~~~~^~~
src/ClientApp.h:25:58: warning: unused parameter 'protocol' [-Wunused-parameter]
   25 |         ClientApp &&connect(std::string url, std::string protocol = "") {
      |                                              ~~~~~~~~~~~~^~~~~~~~~~~~~

I know this is a header-only library, but was wondering are there any specific instructions for building the C++ examples in this library? Looked at the manual can't seem to find anything specific to building.

3 warnings is not a lot. They all come from the client, which is unfinished. Exclude that example and you'll have no warnings


Freezer, music player, package

Free music streaming client for Deezer based on the Deezloader/Deemix "bug".

homepage URL: https://freezer.life
source URL: https://git.freezer.life/exttex/freezerpc
license: gpl3
platforms: android, linux, windows (as far as i know)

Already have a default.nix for it, for the appimage and the .deb. I am willing to make a PR for it tomorrow.

i was sadly unabunable to build it from source.

Package is not maintained anymore as Deezer has patched up. Closing.


[FYI] CDK Version 2 synthesizes a 'BootstrapVersion' CFN Parameter

Issue:
Whilst I'm mindful that this sample repository was created for CDKv1, CodePipeline Deploy_Uat and Deploy_Prod stages fail after upgrading to CDKv2.
Cause:
CDK (v2) synth creates a new BootstrapVersion parameter for generated CFN templates
    "BootstrapVersion": {
      "Type": "AWS::SSM::Parameter::Value<String>",
      "Default": "/cdk-bootstrap/XXXXXXXXX/version",
      "Description": "Version of the CDK Bootstrap resources in this environment, automatically retrieved from SSM Parameter Store. [cdk:skip]"
    }

Solution:
Prod and UAT environments need to be CDK bootstrapped. Otherwise CDK's SSM parameter doesn't exist in deployment accounts, and the pipeline fails. The CloudFormationDeploymentRole also needs permissions to read from SSM parameter store.
A potential alternative solution is to override the CDK synthesizer discussed here.
From CloudTrail
{
    "eventVersion": "1.08",
    "userIdentity": {
        "type": "AssumedRole",
        "principalId": "XXXXXXXXXXXXX:AWSCloudFormation",
        "arn": "arn:aws:sts::XXXXXXXXXXXXX:assumed-role/CloudFormationDeploymentRole/AWSCloudFormation",
        "accountId": "XXXXXXXXXXXXX",
        "accessKeyId": "XXXXXXXXXXXXX",
        "sessionContext": {
            "sessionIssuer": {
                "type": "Role",
                "principalId": "XXXXXXXXXXXXX",
                "arn": "arn:aws:iam::XXXXXXXXXXXXX:role/CloudFormationDeploymentRole",
                "accountId": "XXXXXXXXXXXXX",
                "userName": "CloudFormationDeploymentRole"
            },
            "webIdFederationData": {},
            "attributes": {
                "creationDate": "2022-03-20T04:54:58Z",
                "mfaAuthenticated": "false"
            }
        },
        "invokedBy": "cloudformation.amazonaws.com"
    },
    "eventTime": "2022-03-20T04:54:58Z",
    "eventSource": "ssm.amazonaws.com",
    "eventName": "GetParameters",
    "awsRegion": "ap-southeast-2",
    "sourceIPAddress": "cloudformation.amazonaws.com",
    "userAgent": "cloudformation.amazonaws.com",
    "requestParameters": {
        "names": [
            "/cdk-bootstrap/XXXXXXXXX/version"
        ]
    },
    "responseElements": null,
    "requestID": "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
    "eventID": "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
    "readOnly": true,
    "resources": [
        {
            "accountId": "XXXXXXXXXXXXX",
            "ARN": "arn:aws:ssm:ap-southeast-2:XXXXXXXXXXXXX:parameter/cdk-bootstrap/XXXXXXXXX/version"
        }
    ],
    "eventType": "AwsApiCall",
    "managementEvent": true,
    "recipientAccountId": "XXXXXXXXXXXXX",
    "eventCategory": "Management"
}

This is interesting... I haven't done much with CDK v2 but one of the nice things about this solution as originally written is that one does not have to bootstrap the CDK in the target accounts. I'll have to take a look at this and see if there's a way to keep that advantage even if it's upgraded. I don't have an ETA but wanted you to know I saw it and will have to research. Thanks for calling this out.

From what I've been reading, it sounds like V2 might have native support for the cross-account deployments, which would eliminate the need for those circular scripts at the beginning (at least from my understanding).  Yes, you need to bootstrap in each target account, but I'm thinking with a stack-set + Organizations it wouldn't be too bad.
Just curious, are there any plans to update (possibly on a separate branch) this repo to use the CDK V2?  No worries if not, learned a ton from this presentation and appreciate the repo as a reference!

Short answer: yes, I would like do, because I don't want customers to have to stick with CDK v1 just to use the solution. However, CDK Pipelines isn't always a good fit, in this case when the roles need to be managed outside of / by a separate team than the CDK code, or if you want to use a specific KMS key or S3 bucket. That's why I want to explore one of the mechanisms you suggested to prevent myself from having to bootstrap in every account, and continue to demonstrate the way CFN and CDK can work together to allow collaboration across dev / non-dev skill sets (covered more in the YouTube video than the README here).
If you'd like to give it a shot feel free to create a PR! Otherwise this will be on my backlog TBD :)

Totally understandable!  I actually went ahead and tested out the stack-set idea that I outlined above, and it works really well, especially in the scope of AWS Organizations + ControlTower (which we use).  Have an admin go in and run the stack-set once, and then the dev team can do their work without any further steps!
Given that CDK V1.x goes into maintenance mode on June 1st, I'm wondering if it might be helpful to folks to have a short note at the top of the README linking to the "modern" bootstrap template?
I think the talk definitely still applies to CDK 2.x and wouldn't want this small detail getting in the way of that!  Thanks again for putting all this together!

Finally upgraded to CDK v2 due to critical security vulnerabilities. Thanks for engaging!


Fixes incorrect beta calculation

⚠ Note: Regression tests have not been updated yet. Waiting on peer review.

Description

Fixes incorrect beta calculation

Reworks Sampling portion of AlgorithmManager loop
Fixes issue with miscalculation of various portfolio statistics
Fixes data alignment issue in statistics calculation

We now truncate the first few values of the "Daily Performance"
and benchmark performance series in order to align the series
correctly and attain accurate statistics at the cost of reducing
our sample size by 1 or 2 in the StatisticsBuilder class.
The shape of the data has also changed due to these changes.
Changes to the shape are:

The zero at the end of the "Daily Performance" series is gone
"Daily Performance" series now begins with zero (may be a bug, would like to discuss below).

We also now sample at the end of the AlgorithmManager loop to ensure
that the sampling events are fired even if we have no remaining points
left. This means that sampling after the Stream(...) loop is no longer
needed (if the algorithm successfully ran to the end), though we do
still force sample at the end just in case.
And finally, we've removed the Sample...(...) methods from the
AlgorithmManager and are now sampling inside the
results.ProcessSynchronousEvents() call.
Related Issue

Closes #3927
Motivation and Context

Ensure statistics are calculated correctly
Requires Documentation Change

Users of the BacktestResultPacket, LiveResultPacket, BacktestResult, LiveResult  classes should be aware of this change since existing behavior has changed.
How Has This Been Tested?

New unit test, manual verification of statistics. Tested on QC cloud, potential plotting issue.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Closing pending new solution.


Fix deprecated imports from werkzeug and collections.

Looks like Flask-WTF has a newer version that fixes deprecated imports:
https://github.com/lepture/flask-wtf/commit/dc786301c5b6c10a8b1b256d9820c8a7a932d99c#diff-b0508adba031c447ece5a495a136a3d1

:+1:
I'm a little leery that the newer version of Flask-WTF had its build fail on 3.6.7, but I tried running this upgrade on the same version and it seems fine. Thanks!


Mentiony Not working in mobile MS-Teams Application

When try to use mentiony from the responsive browser it works fine but  if access the same from MS-Teams mobile application it's not working. After digging out I observed key press event not getting fire.
Is there any solution for the same?

Finally got the solution. In detail explained below.
We need to modify the Mentiony Jquery for the Mobile as the  "Touch Event is NOT supported" by the jquery-mentiony JS.
Find and replace below functions in "jquery-mentiony" js:
function onInputBoxInput(e) {
// log('onInputBoxInput');
events.input = true;
var ua = navigator.userAgent.toLowerCase();
var isAndroid = ua.indexOf("android") > -1; //&& ua.indexOf("mobile");
if (isAndroid) {
if (!needMention)
needMention = true;
//certain versions of android mobile browser don't trigger the keypress event.
}
settings.onInput.call(this, elmInputBox, elmInputBoxContent);
    }


Move integration tests to miden crate

This PR addresses #144 and moves all integration tests from the processor to the integration tests directory in the miden crate.
It also refactors the auxiliary table trace into a module and refactors its tests from integration tests to unit tests.

@bobbinth addressed comments & rebased to migrate the latest tests as well


OAK-9751 : handled cases where path changes in lucene exceeds max bui…

…lder size

BTW, thanks a lot for the PR! It is great that you have found and fixed the issue!


Escape

4.Clone your CodeIgniter repo: git@github.com:/CodeIgniter.git
to
4.Clone your CodeIgniter repo: git@github.com:<your-name>/CodeIgniter.git
Signed-off-by: Lloric Mayuga Garcia<EMAIL_ADDRESS>

Oops - this fell under the radar, in favor of the "contributing" section in the user guide. I will merge this, and then rework the page, to conform to the user guide :-/

not to be too picky, but we expect commits to be GPG-signed :)


Hi, please i need help with this. "_firebase2.default is not a constructor"

i got this error when i run my react App:
create.js?84fe:13 Uncaught TypeError: _firebase2.default is not a constructor
just in the line when a create a new Firebase:
const ref = new Firebase('https://my-project.firebaseio.com/');
Can any one help me please, i use the latest version of firebase, reactfire, react-mixin

What version of Firebase are you using? This library has not yet been upgraded to the just-released Firebase 3.x.x. See #91 for details.

You either need to:

Downgrade to Firebase 2.X (force version in package.json) and continue as normal,
Or upgrade to the new Firebase, and use firebase.initializeApp() instead of new Firebase() - but you'll probably have to wait for ReactFire 1.0.

When you upgrade, also make sure you do npm un --save firebase && npm i --save firebase to clean up old versions.

hoooo i see... i use Firebase 3, ok i will wait for the next release of the reactfire.... very thanksss to all!!!

thanks!

اپلیکیشن خدمات منزل ,  برنامه نویسی اندروید و آیفون , برنامه نویسی موبایل


Add deploy to kubernetes action

Add https://github.com/steebchen/kubectl to the list.

@steebchen your changes have been merged, thanks for your contribution 👍


Improve email subsystem

https://github.com/gratipay/gratipay.com/pull/4335 turned up some dryrot. This is a start on cleaning it up.
Todo

[x] change base back to master

This one is ready next, @kaguillera @nobodxbodon.

Rebased, was 88cc37151b4cf5387298f01370f772fa643208d6.

I kicked Travis and we're green here again, @nobodxbodon et al.

Looks good and straightforward refactoring to me. Merging now.

Awesome, thanks @nobodxbodon!


Provide social sharing tags

Facebook
Warnings That Should Be Fixed
Inferred Property The 'og:image' property should be explicitly provided, even if a value can be inferred from other tags.
Missing Properties The following required properties are missing: og:url, og:image, fb:app_id
Twitter
Add missing card tags
Image, Description, URL... See https://cards-dev.twitter.com/validator / documentation
<meta name="twitter:card" content="summary" />
<meta name="twitter:site" content="@nytimesbits" />
<meta name="twitter:creator" content="@nickbilton" />
<meta property="og:url" content="http://bits.blogs.nytimes.com/2011/12/08/a-twitter-for-my-sister/" />
<meta property="og:title" content="A Twitter for My Sister" />
<meta property="og:description" content="In the early days, Twitter grew so quickly that it was almost impossible to add new features because engineers spent their time trying to keep the rocket ship from stalling." />
<meta property="og:image" content="http://graphics8.nytimes.com/images/2011/12/08/technology/bits-newtwitter/bits-newtwitter-tmagArticle.jpg" />

https://github.com/kentcdodds/kentcdodds.com/blob/master/src/components/seo/index.js
https://www.gatsbyjs.org/docs/seo/
https://www.gatsbyjs.org/docs/add-seo-component/
https://www.gatsbyjs.org/packages/gatsby-plugin-social-cards/


Update README.md

i don't know If i'm allowed or not to post twice please let me know :)

I don't see why not! broke them onto two lines, though.

Oh, Okay Thanks :)


Email Address Length

Do you mind if I ask about why a valid email address is limited to 61 characters? (ie. && email.length < 61). In some locations length limitation is discussed to be 254:
http://stackoverflow.com/questions/386294/what-is-the-maximum-length-of-a-valid-email-address

Because realistically nobody has an email address longer than 60 characters. If you find any real world example that break this rule I'd be glad to reconsider.
Look at it as an optimization: if you ask the library to check an email address that is longer than 60 characters, 99.9999% it means the email doesn't exist. So it can give you the response without wasting time in several round-trip web checks.
Hope this helps to clarify, cheers!


The Letter - Horror Visual Novel

Tried "The Letter-Horror Visual Novel", does not catch the text.

Tried the demo available here and that seems to work just fine. Can you confirm if this demo works for you?

Demo Works, but in the full version of Steam does not catch the text.

Sounds like the full game may be compiled with IL2CPP.
Please provide a screenshot of the files in the The Letter_Data\Managed folder.

Sounds like the full game may be compiled with IL2CPP.
Please provide a screenshot of the files in the The Letter_Data\Managed folder.

No IL2CPP to see there. Once I obtain the game I can take a closer look.

No IL2CPP to see there. Once I obtain the game I can take a closer look.

I will try to download on Yandex.Disk, in game like there is no protection. Starts with Steam off.

No need. Already tried it out now.
The issue is the same as https://github.com/bbepis/XUnity.AutoTranslator/issues/49
Apparently there is a certificate validation issue in this Mono Runtime.

No need. Already tried it out now.
The issue is the same as #49
Apparently there is a certificate validation issue in this Mono Runtime.

It helped, thank you very much.


Reachability tests

Trying to diagnose why reachability tests are often failing on Travis. Do not merge yet.

Current coverage is 2.72%

Merging #3435 into master will decrease coverage by -75.52% as of 8f63e45

@@            master   #3435   diff @@
======================================
  Files            5       5
  Stmts         2491    2491
  Branches         0       0
  Methods
======================================
- Hit           1949      68   -1881
  Partial          0       0
- Missed         542    2423   +1881

Review entire Coverage Diff as of 8f63e45

Powered by Codecov. Updated on successful CI builds.


Analytics

If you want keep it small and fast, maybe no analytics functions embedded

I was thinking it might be interesting to track what how many people use the app. Not sure if this will be implemented or not though. There are no plans to track the sites people visit obviously.

Phoning home with information is always tempting, but seldom necessary.
For such a single purpose app I don’t think it is worth tracking usage. Since tracking what websites are being displayed is a pretty big red flag there is little else you could track, right? Position on screens and window size maybe?
Instead you could advertise the number one statistic everyone is interested in: number of downloads. Maybe a download analytics page like Minimal Status Bar has (source)?

That's exactly what I meant. I wanted usage analytics (aka how many users we have). That's really all I was considering. Might not be worth the effort though.
Sent from my iPhone

On Apr 16, 2015, at 4:16 PM, Martijn van der Ven<EMAIL_ADDRESS>wrote:
Phoning home with information is always tempting, but seldom necessary.
For such a single purpose app I don’t think it is worth tracking usage. Since tracking what websites are being displayed is a pretty big red flag there is little else you could track, right? Position on screens and window size maybe?
Instead you could advertise the number one statistic everyone is interested in: number of downloads. Maybe a download analytics page like Minimal Status Bar has (source)?
—
Reply to this email directly or view it on GitHub.

That’s exactly what I meant.

Sorry for jumping to other conclusions. I think it was triggered by the first comment talking about embedding functions.

Might not be worth the effort though.

It is something that might look good on the page. Maybe just a total number of downloads? If you are not going for charts like MSB did the code becomes really easy. Here is a JSBin I whipped up: total number of Helium downloads.

That's fantastic! Exactly what I wanted! :D I'll add it to the site :)


Topic form improvements

Re-order fields to match the guide form
Consistently use 'Slug' and generate 'Final URL' rather than requiring an entire path
Automatically generate the slug from the title
Use auto expanding text area for the description rather than text field

Before

After

https://trello.com/c/ED6wNNn5/444-re-order-top-fields-of-the-topics-edit-pages-in-the-pub-app

Thanks for the review @cbaines!


Fix Vulnerability in everit-json

Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:
Fix Vulnerability in everit-json
Special notes for reviewers:
Additional information (if needed):

thanks @manick02 !


Uncaught TypeError: Cannot read property 'height' of undefined

[Enter steps to reproduce below:]

Just updated for 0.210.0

Atom Version: 0.210.0
System: Mac OS X 10.10.4
Thrown From: color-picker package, v2.0.5
Stack Trace
Uncaught TypeError: Cannot read property 'height' of undefined
At /Users/makingoff/.atom/packages/color-picker/lib/extensions/Alpha.coffee:147

TypeError: Cannot read property 'height' of undefined
  at Object._this.control.setSelection (/Users/makingoff/.atom/packages/color-picker/lib/extensions/Alpha.coffee:147:39)
  at /Users/makingoff/.atom/packages/color-picker/lib/extensions/Alpha.coffee:181:30
  at Object.emit (/Users/makingoff/.atom/packages/color-picker/lib/modules/Emitter.coffee:11:23)
  at Object.emitInputColor (/Users/makingoff/.atom/packages/color-picker/lib/ColorPicker-view.coffee:232:22)
  at Object.open (/Users/makingoff/.atom/packages/color-picker/lib/ColorPicker-view.coffee:309:23)
  at atom-text-editor.<anonymous> (/Users/makingoff/.atom/packages/color-picker/lib/ColorPicker.coffee:42:68)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:238:29)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:520:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:355:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:180:20)

Commands
     -1:01.5.0 core:move-up (atom-text-editor.editor.is-focused)
     -1:01.1.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.is-focused)
  3x -1:01.0 core:move-left (atom-text-editor.editor.is-focused)
  5x -0:59.5.0 core:move-down (atom-text-editor.editor.is-focused)
     -0:58.5.0 core:move-left (atom-text-editor.editor.is-focused)
     -0:58.2.0 core:select-left (atom-text-editor.editor.is-focused)
     -0:57.4.0 core:save (atom-text-editor.editor.is-focused)
     -0:38.5.0 core:move-down (atom-text-editor.editor.is-focused)
     -0:38.2.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.is-focused)
     -0:37.9.0 emmet:insert-formatted-line-break-only (atom-text-editor.editor.is-focused)
     -0:37.9.0 editor:newline (atom-text-editor.editor.is-focused)
     -0:36.2.0 emmet:expand-abbreviation-with-tab (atom-text-editor.editor.is-focused)
     -0:35.3.0 core:undo (atom-text-editor.editor.is-focused)
     -0:34.9.0 emmet:expand-abbreviation-with-tab (atom-text-editor.editor.is-focused)
     -0:32.9.0 core:save (atom-text-editor.editor.is-focused)
     -0:02.0 color-picker:open (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "autosave",
      "welcome",
      "minimap",
      "atom-color-highlight",
      "select-line"
    ],
    "themes": [
      "atom-light-ui",
      "atom-light-syntax"
    ],
    "projectHome": "/Users/makingoff/Dropbox/projects",
    "destroyEmptyPanes": false
  }
}

Installed Packages
# User
SFTP-deployment, v1.0.0
atom-pair, v1.1.6
color-picker, v2.0.5
emmet, v2.3.10
gulp-control, v0.4.2
hotkeys, v0.0.0
language-slim, v0.4.0
language-twig, v1.6.1
pane-split-moves-tab, v0.0.1
space-tab, v0.4.2
tabs-to-spaces, v0.11.0

# Dev
No dev packages

Hi, @makingoff. Is this bug still present?

Sorry, @thomaslindstrom. My answer so late. Everything fine with height.


spec: update signature spec for OCI artifact

This issue is to update signature-specification for Notary v2 to use OCI artifact manifest to store the signature in the registry.
For registries don't support OCI artifact manifest, need to decide whether notation needs to fallback to OCI image manifest.

Per community discussion, the scenario fallback to OCI image manifest will be put in rc.2. Only using OCI artifact manifest to store the signatures in rc.1. I will create a new issue for rc.2. This issue focus on rc.1 scenario.
@priteshbandi @toddysm @shizhMSFT

Issue created for rc.2 https://github.com/notaryproject/notation/issues/444


browser: Simplify title address extraction

This refactors the browser.address implementation to be clearer, making the following changes along the way:

The same word is no longer checked to be a valid URL twice.
Check the first word is a URL before checking the rest of the words in reverse. The suggested Firefox extension prepends the URL to the window title (and does not wrap it in brackets).
Avoid checking the empty string, in case a webpage has consecutive spaces in its title.

Thanks!


⚠ Fortnite has degraded performance

In 06f6452, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1599 ms

Resolved: Fortnite performance has improved in aa42e5c.


Install problem on 3.4

Trying to install, and have this issue:
Couldn't load plugin 'seismic_importer' due to an error when calling its classFactory() method
ImportError: cannot import name 'processing' from 'qgis' (C:/PROGRA~1/QGIS3~1.4/apps/qgis-ltr/./python\qgis_init_.py)
Maybe something is missing in my install ?
Thanks

Hi,
You would need to install the processing module of QGIS to be part of your installation. Please make sure that is available.
Thanks,
Gurpreet


Adds cache instructions and fixes GetPerpTrades

Status
Type
⚠ Core Change
Issue

Ready
Feature/Refactor
No
#77

Problem
What problem are you trying to solve?
The need to cache prices / root banks and perp markets when trying to test stuff on devnet

Adds new instructions

The perp trades endpoint wasn't flattening the response and was changed so needed to adapt that

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

73 of 73 (100.0%)  changed or added relevant lines in 3 files are covered.
89 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-2.09%) to 78.501%

Files with Coverage Reduction
New Missed Lines
%

Solnet.Mango.Historical/Models/PerpTrade.cs
3
78.57%

Solnet.Mango.Historical/Converters/StringToDateTimeJsonConverter.cs
4
0%

Solnet.Mango.Historical/Converters/PerpTradeJsonConverter.cs
82
0%

Totals

Change from base Build<PHONE_NUMBER>:
-2.09%

Covered Lines:
2425

Relevant Lines:
3056

💛  - Coveralls


Refactor K8s Utils

Need to refactor K8s Utils

Done - https://github.com/kaasops/vector-operator/commit/370a2d2c680a36d34fb303d7071c3a0f33a3f6ab


COPY TO with an invalid destination does not error as expected

Description
Currently working through azure support, and have this for an slt:
statement ok
COPY ( SELECT 1 AS a, 2 AS b, 3 AS c ) TO azure
	OPTIONS (
		account_name = '${AZURE_ACCOUNT}',
		access_key = '${AZURE_ACCESS_KEY}',
		location = 'azure://glaredb-test/copy_to/with_opts.csv'
	);

Surprisingly this passes even though I haven't implemented the relevant stuff for azure.

Can you elaborate here? Could you let me know what you implemented? Seems like something intermediate would be irrational to evaluate. But I may have understood this issue wrong


/articles: line spacing too large

I think it should be reduced by about 10%.

@melissasamworth

I like the line spacing! Let's take this as a data point but I want a few more data points until I'm ready to make the change. So close it for now but mentally noted :)


Bug when using k8s namespaces other than default

I recognized a bug when using namespaces for deployments with keyvault-flexvol
I've a dedicated namespace development
After deploying theAzureIdentity, AzureIdentityBinding and a Pod to the development namespace. KeyVault driver isn't able to mount the key vault flex volume.
As a workaround the isse can be fixed by deploying both AzureIdentity and AzureIdentityBinding to the default namespace.
Once that's done, the Pod from namespace development can grab data from Keyvault.
I think this is a critical bug which prevents me - and I can imagine others may agree on that one - from using keyvault-flexvol for real deployments

I believe that this behavior is caused by https://github.com/Azure/aad-pod-identity/issues/115

Great, @serbrech thanks for linking that one


Update title of "new bot" dialog

Is your feature request related to a problem? Please describe.
The 2nd step of the "new bot" flow currently has a modal where the project name and location are selected. However, the title of this modal is "define conversation objective" which is a legacy title from when this modal was shared with other parts of the app.
Describe the solution you'd like
The title and description of this modal should be updated.  Consider removing "description" field which is not currently displayed anywhere.
Title: New Bot Project
Description: Specify a name and location for your new bot project.

e2e test in a draft PR:

Thanks!!!


fix regression: only skip jar-scanning completely if scanSet is empty

The commit 21d08e1952d6612da9a712a173c81174958f4b8b introduced a regression to the behavior of jar-scanning. Since version 9.0.31 it skips all jars, if a wildcard is found in the skipSet - no matter the content of the scanSet!
The documentation for jarsToScan clearly states:

overrides the default jarsToSkip list above

This PR restores this behavior by also respecting the "jarsToScan".

Fixed with https://github.com/apache/tomcat/commit/9c6563bf6ac723cda4e78d0e8c1a996fa1b8ce56

Fixed with 9c6563b

Hm, that would ignore the case, that jarsToScan is specified but empty. (Because then defaultScan would be an empty String instead of null)
Is that intended?

I hadn't seen this PR at all as there was a BZ there already and it didn't mention it.


TypeScript type definitions

Currently, this library is not usable with TypeScript. Does it make sense to add a definition file to this project? I'd be happy to work on a .d.ts file and send in a pull request.

I'd really recommend any help here =) I am not using TypeScript myself.

Alrighty, I'm currently working on the definition file. I'mma send in a pull request as soon as everything works.

Still open to send a PR? Will close this issue for now as I personally don't need TypeScript definitions anyways, but feel free to reopen and add em :)


Not Working

Not working with Next.js v13.5.6.
Code:
 <AnimatedNumbers                   includeComma                   animateToNumber={parseInt(achievement.value)}                   locale="en-US"                   className="text-white text-4xl font-bold"                   configs={(_, index) => {                     return {                       mass: 1,                       friction: 100,                       tensions: 140 * (index + 1),                     };                   }}                 ></AnimatedNumbers>

Nextjs 13.5.4 ,example code ,whatever using dynamic import or not ,the output just show 0,000

https://github.com/heyman333/react-animated-numbers/issues/51#issuecomment-1742257182
this might work

I have same issue.

The config callback is working, but the outcome of the animated-numbers doesn't work.
"dependencies": {
...
"next": "14.0.1",
"react-animated-numbers": "^0.16.0",
}

Also not working on my nextjs 13.6 project. previously worked on earlier vertion

Updated library to use framer-motion. Please check again with v0.17.0


Breakout rooms

Especially for games with a max number of players #85. It could also be set up as a tournament mode

inspo https://www.youtube.com/watch?v=jbPpdyn16sY


feat: check if pypi package is conda package

High overview logic:
from parselmouth side:

collect all conda packages from conda-forge/subdir/repodata.json
collect all missing packages where ( [1] hash of package is missing in our extensive mapping or [2] pypi_normalized_names of that package in mapping is None )
request pypi by package name from list of missing packages
if package name is present and it's not it compressed mapping version,  we save it in non_pypi_names.json list - this means that conda package is non the same as pypi even if names are the same. Some older conda-packages can't be parsed with get_artifact_info, so their hash is missing but name is present in compressed mapping.
if package name is missing from pypi we add it to non_existing_pypi_packages.json list - this means that for conda-package the python package is get directly from source so there are no pypi package for it. This should be addressed in future to deal with the fact that other pypi package can be published by this name even if it's different.

from pixi side:

during the amendment step if conda name is not present in compressed or extensive mapping we check if it's name is in non_pypi_names.json map. If yes, this means that the conda package is not pypi one - ( pandoc issue ).
We write the purl for it with the type purl:/non-pypi that will tell us later that the package is not coming from pypi artifactory so we should install also the pypi one.
Downside for this is that we possible break the contract of purl spec:
A purl is a URL string used to identify and locate a software package in a mostly universal and uniform way across programing languages, package managers, packaging conventions, tools, APIs and databases. because we can't locate it by purl.

if the name is also missed from non-conda map we rollback to our old logic that assume that they are the same

Hey @nichmor, I'm not sure this extra map is going to be that comfortable. Assuming automated maps can be wrong and we already need this overwrite in the configuration, adding this to the configuration aswell seems like a big overhead for the user.
Would it be possible to extend the map with some more info so the info could be fetched from the existing maps?

as discussed with team I should rethink the current approach and find a better solution for this.
closing it for now


Can a function that uses "assert(...)" be marked noexcept? Should it be?

Channel
Is this a "C++Weekly" or "The [Fill in the Blank] Programmer" episode request?
Your choice, could event just be a tid-bit.
Topics
What topics are important to cover for this episode?
assert() under the hood?
Length
Should this be bite-sized (5-10 minutes) or more long form (10-20 minutes)?

I like this one


Passive skills and named effects

Ola.
I'm trying to use this to remove investigation and perception and combine them into a new skill.
Awareness(INT)
I got it all setup, but now, using monk token bar, see passive awareness instead of passive investigation and perception.
How do I go about finding out what the active effect naming convention is for the new skill.
Instead of skills.prc.passive?

Is the skill active effect "cus_0"?

probably yes, you should be able to see all the custom skills (cus_X) and custom abilities (cua_X) in the setting page between parentesis.


Revert "build(deps): bump github.com/coopnorge/mage-kubernetes-lib from 0.6.0 to 0.7.0 in /images/devtools-kubernetes-v1beta1/context/magefiles"

Reverts coopnorge/engineering-docker-images#2338

:+1:


fluid.bufampslice~ musical example plays loads of clicks

The p "pick a slice" subpatch plays from the start of a slice until the next, meaning that the end of each slice contains a tiny moment of the start of the next slice, leaving a click.
Shaving 100 samples off the end makes it sound much better (and makes sense since this patch isn't trying to demonstrate _off_sets).

This is no longer the same file.


Show globally-installed packages in ignite doctor

Please verify the following:

[x] yarn ci:test jest tests pass with new tests, if relevant

Describe your PR
@nirre7 had a problem that was caused by an older version of react-native-cli being installed globally (which is no longer recommended); he suggested having Ignite warn when this happens. https://infiniteredcommunity.slack.com/archives/CD8UA5RJ4/p1609336353033800?thread_ts=1609248567.028100&cid=CD8UA5RJ4
I think that's a great idea, and I thought it'd be useful as a preliminary step to see all the globally-installed packages in ignite doctor's output, so that's what this PR adds. Because npm and yarn store global packages in different places, it lists them separately -- the Javascript section of its output now looks like this (I just have one package in each group):
JavaScript (and globally-installed packages)
  node                14.15.1      /Users/stearns/.asdf/installs/nodejs/14.15.1/bin/node
  npm                 6.14.8       /Users/stearns/.asdf/installs/nodejs/14.15.1/bin/npm
    expo-cli          4.0.17
  yarn                1.22.5       /Users/stearns/.yarn/bin/yarn
    create-next-app   10.0.4

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[ON HOLD] [dunfell] Fix SRCREV for all components

SRCREV is fixed for all components
'tag' is replaced with SRCREV for dbus-cxx to have consistent approach

Tested by the build of the DomD of the prod-cockpit-rcar with fixed commit on development (origin of this PR) branch.

Ack is added. Tested.
Ready to be merged.