TryFrom for builtin types

#173

I added a way to try_into() options.
I'm also thinking about AsRef<Ipld> IDK if it make sense yet.

There's a Clippy warning unrelated to your change. I'll take care of it in a separate PR.

I added some tests.
Your example in #173 would not work as is because get() return a reference.

What to do with f32 there's not way to go from f64 -> f32 exect with as?

What to do with f32 there's no way to go from f64 -> f32 except with as?

I suggest not implementing it for f32. If someone needs it, they would need to convert first to f64 and then be on their on doing it whichever way they like.

I just noticed no-std doesn't build.
Should I gate the try_from with std? Seams required for the error conversion.

Should I gate the try_from with std? Seams required for the error conversion.

Yes please do. I haven't checked if it would theoretical be possible doing it in a no-std environment. But that can then be a separate PR in case it's possible/someone finds the time to tackle it.


F12 and go to definition, go to implementation does not work

Issue Type: Bug
open a react project in vscode
go to a component name or a function and click F12
expected result: the file with the clicked component or function will be opened
VS Code version: Code 1.40.1 (8795a9889db74563ddd43eb0a897a2384129a619, 2019-11-13T16:49:35.976Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz (4 x 2712)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.89GB (3.63GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.3.0

vscode-eslint
dba
1.9.1

githistory
don
0.4.6

gitlens
eam
10.1.2

easy-icons
jam
0.3.1

git-graph
mhu
1.18.0

debugger-for-chrome
msj
4.12.1

abracadabra
nic
0.11.0

vscode-react-refactor
pla
1.0.5

html-css-class-completion
Zig
1.19.0

(6 theme extensions excluded)

/needsMoreInfo

/needsMoreInfo

what info is missing?

Please provide repo steps and example code

I'm opening my code in React 16.9, going to a method or class used,
clicking on F12 and nothing happens...
expecting F12 to pass me to the method or the class clicked.
 :)
On Tue, 19 Nov 2019 at 21:30, Matt Bierner<EMAIL_ADDRESS>wrote:

Please provide repo steps and example code
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUCWHLUW4HRBV46X7R3QUQ5GJA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEPNZBY#issuecomment-555670663,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUDUCDIQ5O4W5EHJZY3QUQ5GJANCNFSM4JN2A2VQ
.

Please share some example code. React intellisense works just fine for me

I don't think this is an issue of code. F12 just doesn't work for me...
On Thu, Nov 21, 2019, 20:34 Matt Bierner<EMAIL_ADDRESS>wrote:

Please share some example code. React intellisense works just fine for me
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUHBFLRE6G3I22M3GNLQU3IDLA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE3G6HI#issuecomment-557215517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUCWX4LEOPP3BJBCRZ3QU3IDLANCNFSM4JN2A2VQ
.

Please share some example code

Here we go:
const mapState = (state) => {
const queryResult = selectors.getQueryResult(state.search);
const results     = queryResult.search_result;
return {
queryResult,
cMap: cMapFromState(state, results),
cuMap: cuMapFromState(state, results),
postMap: postMapFromState(state, results),
query: selectors.getQuery(state.search),
pageNo: selectors.getPageNo(state.search),
sortBy: selectors.getSortBy(state.search),
deb: selectors.getDeb(state.search),
pageSize: settingsSelectors.getPageSize(state.settings),
language: settingsSelectors.getLanguage(state.settings),
wip: selectors.getWip(state.search),
err: selectors.getError(state.search),
};
};
click on any yellow procedure calls does nothing... even though in this
case all these functions are in the same file
previously, F12 on a function used even to lead me into other files of my
react project
Gabi
"What R we living for?" :)
On Fri, 22 Nov 2019 at 21:33, Matt Bierner<EMAIL_ADDRESS>wrote:

Please share some example code
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUEGXMTRNQX56PWDEBDQVAXZXA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE6URDQ#issuecomment-557664398,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUA74P57U2HKZMYCFMTQVAXZXANCNFSM4JN2A2VQ
.

this one of the functions which i spoke above
const cMapFromState = (state, results) => (
results && results.hits && Array.isArray(results.hits.hits)
? results.hits.hits.reduce((acc, val) => {
if (val._source.result_type === 'collections') {
const cID = val._source.mdb_uid;
const c   = mdbSelectors.getDenormCollection(state.mdb, cID);
if (c) {
acc[cID] = c;
}
}
return acc;
}, {})
: {}
);
Gabi
"What R we living for?" :)
On Tue, 26 Nov 2019 at 03:25, Gabi Wissotzky<EMAIL_ADDRESS>wrote:

Here we go:
const mapState = (state) => {
const queryResult = selectors.getQueryResult(state.search);
const results     = queryResult.search_result;
return {
queryResult,
cMap: cMapFromState(state, results),
cuMap: cuMapFromState(state, results),
postMap: postMapFromState(state, results),
query: selectors.getQuery(state.search),
pageNo: selectors.getPageNo(state.search),
sortBy: selectors.getSortBy(state.search),
deb: selectors.getDeb(state.search),
pageSize: settingsSelectors.getPageSize(state.settings),
language: settingsSelectors.getLanguage(state.settings),
wip: selectors.getWip(state.search),
err: selectors.getError(state.search),
};
};
click on any yellow procedure calls does nothing... even though in this
case all these functions are in the same file
previously, F12 on a function used even to lead me into other files of my
react project
Gabi
"What R we living for?" :)
On Fri, 22 Nov 2019 at 21:33, Matt Bierner<EMAIL_ADDRESS>wrote:

Please share some example code
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUEGXMTRNQX56PWDEBDQVAXZXA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE6URDQ#issuecomment-557664398,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUA74P57U2HKZMYCFMTQVAXZXANCNFSM4JN2A2VQ
.

another one: F12 on Link class, which is imported on the top of the file,
does nothing as well...
i expect that F12 will open the file where MultiLanguageLink is implemented
(it is implemented in my project)
import React from 'react';
import PropTypes from 'prop-types';
import { withNamespaces } from 'react-i18next';
import { Container, Divider } from 'semantic-ui-react';
import Link from '../Language/MultiLanguageLink';
const DidYouMean = ({suggestion, onClick, t}) => {
const didYouMeanStr = "Did you mean "; //t(${section}.header.text);
return (
suggestion
?
{didYouMeanStr}
{suggestion}

: null
);
};
Gabi
"What R we living for?" :)
On Tue, 26 Nov 2019 at 03:25, Gabi Wissotzky<EMAIL_ADDRESS>wrote:

this one of the functions which i spoke above
const cMapFromState = (state, results) => (
results && results.hits && Array.isArray(results.hits.hits)
? results.hits.hits.reduce((acc, val) => {
if (val._source.result_type === 'collections') {
const cID = val._source.mdb_uid;
const c   = mdbSelectors.getDenormCollection(state.mdb, cID);
if (c) {
acc[cID] = c;
}
}
return acc;
}, {})
: {}
);
Gabi
"What R we living for?" :)
On Tue, 26 Nov 2019 at 03:25, Gabi Wissotzky<EMAIL_ADDRESS>wrote:

Here we go:
const mapState = (state) => {
const queryResult = selectors.getQueryResult(state.search);
const results     = queryResult.search_result;
return {
queryResult,
cMap: cMapFromState(state, results),
cuMap: cuMapFromState(state, results),
postMap: postMapFromState(state, results),
query: selectors.getQuery(state.search),
pageNo: selectors.getPageNo(state.search),
sortBy: selectors.getSortBy(state.search),
deb: selectors.getDeb(state.search),
pageSize: settingsSelectors.getPageSize(state.settings),
language: settingsSelectors.getLanguage(state.settings),
wip: selectors.getWip(state.search),
err: selectors.getError(state.search),
};
};
click on any yellow procedure calls does nothing... even though in this
case all these functions are in the same file
previously, F12 on a function used even to lead me into other files of my
react project
Gabi
"What R we living for?" :)
On Fri, 22 Nov 2019 at 21:33, Matt Bierner<EMAIL_ADDRESS>wrote:

Please share some example code
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUEGXMTRNQX56PWDEBDQVAXZXA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE6URDQ#issuecomment-557664398,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUA74P57U2HKZMYCFMTQVAXZXANCNFSM4JN2A2VQ
.

Sorry but it is not possible to test those snippets without having definitions for all the  symbols they use.  Can you please share a self contained example or the project itself?
Also, if you enabled semantic checking in your JS by adding // @ts-check at the top, do you see any errors?

you can try getting that project from here
https://github.com/Bnei-Baruch/kmedia-mdb
open it with vscode latest release and try use F12 which doesn't work
Gabi
"What R we living for?" :)
On Tue, 26 Nov 2019 at 07:06, Matt Bierner<EMAIL_ADDRESS>wrote:

Sorry but it is not possible to test those snippets without having
definitions for all the symbols they use. Can you please share a self
contained example or the project itself?
Also, if you enabled semantic checking
https://code.visualstudio.com/Docs/languages/javascript#_type-checking
in your JS by adding // @ts-check at the top, do you see any errors?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUDVVQHMXSOXHGGBLK3QVSVGDA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFEXJEQ#issuecomment-558462098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUAYNOPOMYZHIPGPSZLQVSVGDANCNFSM4JN2A2VQ
.

F12 works for me.
Again, please share specific repo steps. For example: open file x.js and run f12 on line 20. Also describe what you expect to happen in that case

for example i put above:
Find cMapFromState function and open the file
put cursor on mdbSelectors.getDenormCollection
click F12
nothing happens for me, but i expect that a js file in which we have the
definition of getDenormCollection will be opened and cursor will be set on
that function definition.
Gabi
"What R we living for?" :)
On Tue, 3 Dec 2019 at 06:07, Matt Bierner<EMAIL_ADDRESS>wrote:

F12 works for me.
Again, please share specific repo steps. For example: open file x.js and
run f12 on line 20. Also describe what you expect to happen in that case
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUDIICTAYYWJBTAA6Y3QWXLOVA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFYAHPY#issuecomment-560989119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUEVSP2GZIZIOLRWADLQWXLOVANCNFSM4JN2A2VQ
.

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

In insider it works properly!
Gabi
"What R we living for?" :)
On Wed, 4 Dec 2019 at 01:53, Matt Bierner<EMAIL_ADDRESS>wrote:

[image: Screen Shot 2019-12-03 at 3 53 14 PM]
https://user-images.githubusercontent.com/12821956/70099906-08f21b80-15e5-11ea-9915-017a391ce01d.png
Does this reproduce in the latest VS Code insiders build
https://code.visualstudio.com/insiders/ with all extensions disabled
https://github.com/microsoft/vscode/wiki/Performance-Issues#run-with-extensions-disabled
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUCMITCBSBZQMR5XQ6DQW3WRDA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF3HFCA#issuecomment-561410696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUEYLOGH64VXAQK6RCDQW3WRDANCNFSM4JN2A2VQ
.

but in the main release with all extensions disabled it does not work
Gabi
"What R we living for?" :)
On Fri, 6 Dec 2019 at 04:43, Matt Bierner<EMAIL_ADDRESS>wrote:

Closed #84908 https://github.com/microsoft/vscode/issues/84908.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/84908?email_source=notifications&email_token=AHVVEUCPZRN23Z24H2HG5K3QXG33ZA5CNFSM4JN2A2V2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVJ36BOI#event-2859983033,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVVEUFFZAZYRXGWJXHE4HLQXG33ZANCNFSM4JN2A2VQ
.


[event hubs] Update OTEL to latest spec

Packages impacted by this PR

@azure/event-hubs

Issues associated with this PR
Describe the problem that is addressed by this PR
Updating to latest OTEL messaging standards.  This is also explained in a gist from @lmolkova here
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)

#25492

Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Ensure that RiakCluster uses a new instance of DefaultNodeManager

Fixes issue #49 (CLIENTS-384)

Tests pass and code looks clean.

:+1: 9c8a976

Thanks @christophermancini


Fix retention policy configuration

This pull request fixes several issues with the current code used to configure retention:

createRetentionPolicy only being called in case of errors at database creation time,
retention policy name conflict with the default policy,
retention policy configuration not implemented for events which use the same database.

This change is

I signed it!

@mpvaniersel @piosz PTAL

Sorry, I don't have my dev environment for heapster set up anymore. I can't be much help I'm afraid.

Is it expected behaviour that when you change the retention policy in heapster, the retention policy won't be updated ?
Currently, it only sets the retention policy to the correct duration value when no k8s database exists.

yes, that's expected, I believe

Any update on this?


Providing computation resource in simulator.run mandatory

It is now mandatory to specify a computational resource when submitting a task, as a result of no longer providing a shared queue. The on argument doesn't have a default, but remains a keyword argument.
The provider_id argument is now removed, since it is redundant with the provider_id of the machine group.
Tests and examples that rely on default machine group need to be updated.

@luispcunha is this a still a draft ?

Didnt we miss the QA step?
-------- Mensagem original --------De : Luís Cunha @.> Data: 11/09/24  17:54  (GMT+00:00) Para: inductiva/inductiva @.> Cc: Paulo Barbosa @.>, Review requested @.> Assunto: Re: [inductiva/inductiva] Providing computation resource in simulator.run mandatory (PR #1577)
Merged #1577 into development.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because your review was requested.Message ID: @.***>


Can not repeat WHAMandWHAMR/enhancement/sepformer-whamr-DM.yaml

Hi
After 200 epoches training, I can get only 2.20 PESQ, not 2.84 PESQ as README mentioned. Has any one can repeat this result?

Hi @poor1017 , sorry for my very late reply. Did you train with 16k, or 8k? The given results in the README are with 8k.

Ok, I am closing this issue since you are saying that you are able to get close to the one we obtained, when trying with 8k.


restic ls shows MISSING files

Output of restic version
restic 0.6.1
compiled with go1.8.3 on linux/amd64
How did you start restic exactly? (Include the complete command line)
restic -r /mnt/media/restictest ls latest | grep MISSING
What backend/server/service did you use?
local
Expected behavior
the output from ls should match what is on the filesystem, excluding excludes
Actual behavior
weirdly named files e.g. ./.config/gconf/desktop/gnome/interface/%!g(MISSING)conf.xml
Steps to reproduce the behavior
$ restic -r /mnt/media/restictest ls latest | grep MISSING
./.config/gconf/apps/%!g(MISSING)conf.xml
./.config/gconf/apps/GnomeBaker/%!g(MISSING)conf.xml
./.config/gconf/apps/GnomeBaker/Devices/%!g(MISSING)conf.xml

So ./.config/gconf/apps/GnomeBaker/%gconf.xml becomes ./.config/gconf/apps/GnomeBaker/%!g(MISSING)conf.xml
Do you have any idea what may have caused this?
Maybe restic is mishandling the % character.

Huh, that's indeed a bug. Thanks for taking the time to report this!


Patches

Hi Ryan, sorry for not getting back to you earlier.
A bit of further cleaning is provided in the additional commits.
Thanks.

Hey, finally got around to merging this - thanks for the patch, I think people will be quite happy about this one! ;)


feat: support graphql body mode

Closes https://github.com/loadimpact/postman-to-k6/issues/36
Following this blog post, I've switched all my API testing collections to graphql body mode.
https://blog.getpostman.com/2019/06/18/postman-v7-2-supports-graphql/
There's probably more nuance in the solution but wanted to get something going as a starting point. Ran all tests locally and they still continued to pass and ran bin/postman-to-k6.js locally gave me appropriate output.
Let me know if there's anything else I can assist with! Thanks for all you do! 🙇

Thanks a lot for this, it looks great. Does just the right thing.
Could you add a test case? There's a test I exported, or whatever test you had. You could put it in eg test/material/2.1/graphql.json then add a test to test/int/basic.js.
{
	"info": {
		"_postman_id": "505fff05-4b0c-4356-8964-ca8a1ae68999",
		"name": "GraphQL",
		"schema": "https://schema.getpostman.com/json/collection/v2.1.0/collection.json"
	},
	"item": [
		{
			"name": "Test Request",
			"protocolProfileBehavior": {
				"disableBodyPruning": true
			},
			"request": {
				"method": "GET",
				"header": [],
				"body": {
					"mode": "graphql",
					"graphql": {
						"query": "query getCollectionDetails {\n    collections {\n        name\n    }\n}",
						"variables": "{\n\t\"name\": \"{{artist}}\"\n}"
					}
				},
				"url": {
					"raw": "http://example.com/graphql",
					"protocol": "http",
					"host": [
						"example",
						"com"
					],
					"path": [
						"graphql"
					]
				}
			},
			"response": []
		}
	]
}

What do you think about the GraphQL variables? Are you using them? They need some additional logic but if you're not using them maybe they could be left out of the first version.

@bookmoons GraphQL variables make life so much easier. I use them heavily so for the first version it'd probably be ideal.
Just to note, Postman exports variables in a simple JSON object and the test added to the repository reflects that the data.graphql entry lifts out the query and variables entry appropriately. On GET requests they are just leveraged as a query parameter.
I'm not sure the implications of GraphQL variables linked with Postman environment variables.
Any additional guidance would be appreciated! 👍

Alright, I think this is what it looks like:

GraphQL vars can be parsed to an object and passed down in a new postman[Request] arg. The GraphQL query can be passed as the body.
Postman vars need to be replaced at runtime. This happens in lib/shim/core.js. postman[Request] can detect graphql vars, run evaluate on each one to replace vars, then run a GraphQL var replacement on the body.

There's a standard Postman var replacement that happens on the string body. Probably it needs to be determined whether that still happens for graphql and in what order with graphql vars, so it can be either disabled or correctly ordered when doing a graphql request.

@bookmoons Just getting back to this. I found out my implementation will work for our purposes without the variables. Went ahead and got this PR to cleanly merge with the last few weeks of changes. Let me know if it's good to go or should await the full implementation.

Thanks @dmhalejr! Will look this over soon.

Tested this against the GitHub API and it worked just fine. It also happens to run the query through Postman var replacement, so that's a usable alternative to GraphQL vars. Thanks a lot @dmhalejr.
@robingustafsson After the little update above this change has my approval.

Thanks for updating that @dmhalejr.

Great, thanks @dmhalejr for contribution and @bookmoons for review. Sorry for delay, I'm merging this.


Add caller location where Faraday was called. (Draft: for Discussion)

I am interested to see the location where a call is made within the metadata.
I thought of doing something like this. which would work for apps calling Faraday directly. But I think most people wraps Faraday into their own client.
would there be a better way to report the location within the application?
would removing all lines that contains gems from the caller array be an option?

Why would such functionality be specific to Faraday? It would be interesting to be able to add this for all traces...

Why would such functionality be specific to Faraday? It would be interesting to be able to add this for all traces...

Why would such functionality be specific to Faraday? It would be interesting to be able to add this for all traces...

There is a non-trivial overhead to calling caller on every span. We certainly wouldn't want to do this by default for every span. It may be worthwhile as an opt-in capability either for specific auto-instrumentation or globally for kind=client spans.
The spec contains semantic conventions for source code. If we add this capability, we should follow those conventions.

Why would such functionality be specific to Faraday? It would be interesting to be able to add this for all traces...

There is a non-trivial overhead to calling caller on every span. We certainly wouldn't want to do this by default for every span. It may be worthwhile as an opt-in capability either for specific auto-instrumentation or globally for kind=client spans.
The spec contains semantic conventions for source code. If we add this capability, we should follow those conventions.

@fbogsany @johnnyshields updated my draft

@fbogsany @johnnyshields updated my draft

waiting on another round of comments before I fix the tests.
I would need to add an ENV var to Github Actions if I proceed with what I am currently thinking and mentioned in my above comment.

waiting on another round of comments before I fix the tests.
I would need to add an ENV var to Github Actions if I proceed with what I am currently thinking and mentioned in my above comment.

I'd be curious to know what the use case is. Everything in observability comes at a cost to the application, and calling caller for each span comes at a fairly high cost. Adding configuration can mitigate some of the cost, but adds complexity and a long term maintenance burden.

I'd be curious to know what the use case is. Everything in observability comes at a cost to the application, and calling caller for each span comes at a fairly high cost. Adding configuration can mitigate some of the cost, but adds complexity and a long term maintenance burden.


Tikz drawing using spot colour not shown

Compilation seems to go right, and the PDF file contains the Tikz path data (same size when using a standard colour) but nothing shows up and my previewer tells me there is an error in the PDF file. Am I missing something, or is the package incompatible with LaTeX?
Here is my MWE (compiled with LuaLaTeX):
\documentclass[a4paper]{article}
\usepackage{luatextra}
\usepackage{colorspace}
\usepackage{tikz}

\definespotcolor{gold}{SPOT GOLD}{1,0,0,0}

\begin{document}
\begin{tikzpicture}
\begin{scope}[shift={(-87.5,-55.49435)}]
  \begin{scope}[cm={{0.96036,0.0,0.0,0.96036,(11.89152,0.52957)}},fill=white]
        \path[draw=gold,fill=white,line join=round,line cap=round,miter
      limit=4.00,nonzero rule,line width=2.400pt] (351.4121,230.1831) .. controls
      (361.2415,243.5007) and (349.6947,254.4950) .. (339.6205,246.9064);

  \end{scope}
\end{scope}

\end{tikzpicture}
\end{document}

It used to work, so I wonder if there are some changes in the pgf code (colorspace uses a couple of hacks for it to work with pgf). I must investigate where the problem is.

If it helps, the TeX distribution is TeXlive 2020.
Le dim. 7 juin 2020 à 07:45, Javier Bezos<EMAIL_ADDRESS>a
écrit :

It used to work, so I wonder if there are some changes in the pgf code (
colorspace uses a couple of hacks for it to work with pgf). I must
investigate where the problem is.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jbezos/colorspace/issues/8#issuecomment-640161368,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMF2Z7J4RCWWGKAOVPMPD7DRVMSQ7ANCNFSM4NV7N24Q
.

Seems related to #6, which I'd just forgotten altogether 🤦♂.

In fact it's weirder than it seems. My "main" tikz drawing works when I put
another simple one before, but doesn't work alone...
Le lun. 8 juin 2020 à 18:58, Javier Bezos<EMAIL_ADDRESS>a
écrit :

Seems related to #6 https://github.com/jbezos/colorspace/issues/6,
which I'd just forgotten altogether 🤦♂.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jbezos/colorspace/issues/8#issuecomment-640750905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMF2Z7I4TCZGGPWCSY7FLMTRVUKDXANCNFSM4NV7N24Q
.


Check post_asset_folder before get banner url

If post_asset_folder is true, the url is not built as expected

Great! Thank you for your contributing!


Add GitHub workflow files template and README.md

Sorry for this unsolicited PR, but I figured it would be great to have a CI/CD workflow in the template for folks who want to save time on learning CI/CD and implementing their own automation!
It contains two workflows:

main.yml - runs Unit/Integration tests using GUT
release.yml - which:
detects version changes via commit messages
updates/creates CHANGELOG.md
Creates a GitHub release and artifact
Pushes the artifact to Godot AssetLib
Pushes the artifact to Itch.io

Removed usage of PAT and using proper secrets definition in workflow
Fixed spelling and usage of environment variables
Added environment variable to avoid workflow-wide string-replacement for the addon name
Updated README.md to reflect changes

Need to remove usage of deprecated set-output usages

Found an issue with not using a PAT. Because if I currently use only the GITHUB_TOKEN, the release prep commit will not trigger a re-run. Thus, the actual publish will never happen.
Discovered this as I updated my workflow files with these in godot-playfab.


Error running pod install Error

I got this error after add this plugin to dependencies of pubspec.yaml and call flutter packages get

I'm not implement anything in MainApp.
I try to clean the Pod and re-install as well. What should I do?

You need change your project ios/Podfile file according to the example:
# Uncomment this line to define a global platform for your project
-# platform :ios, '8.0'
+platform :ios, '9.0'

https://github.com/blankapp/flutter_flipperkit#installation


push new item to field array from parent field array

Description
Hi, I have form with following format of fields:
{ groups: [ { words: [{}] } ] }

At now when I'm working with FieldArray component
<FieldArray name="groups" component={myGroupRenderer}/>

in myGroupRenderer function, library passes me the fields of groups and I can manipulate them (push, remove). It would be nice to have possibility to manipulate also the child words like this:
function myGroupRenderer ({fields}) {
    return <button onClick={ () => fields.at(0).child('words').push({}) } />
}

As I understand to add groups outside of myGroupRenderer function, I can do this.props.array.push('groups', {}) and it's work fine, but if I want to push new item that requires specifying index of its parent - I don't have ability to do this.
Are you submitting a bug report or a feature request?
feature request
What's your environment?
redux-form: 7.2.1

Probably not a feature we would add.
A few ways you could work around this are:

Use change action creator
Get the value at that index, remove it, change the value however you want, and insert it at the same index

There are certainly more ways, but I can't think of them now.


Split retrieving fresh model to separate method

Since #1295 was closed, here is a more simple implementation to split some logic to a separate method. This allows overwriting the method.

@niekbr could you expand your description to include use cases for this, as well as add unit tests for the feature? Thank you! :+1:


withStyles: allow implicitly used component styling from parent component

To customize the components using Higher Order Components, using withstyles(), should have convenient approach for styling implicitly used components. The implicitly used components should have capabilities to modify/override the styles from parent component itself.

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡

Examples 🌈
Select component.
It implicitly uses the Input component.  To override the ::before and ::after psudo elements its not feasible to achieve directly using its corresponding classes.

https://next.material-ui.com/components/selects/
https://next.material-ui.com/api/select/

Motivation 🔦

@herin0506 I'm confused by the issue's description. Could you please clarify the following:

The screenshot shows v5. The docs API urls shows v4. withStyles is a v4 API. Which version are you using?
Why did you not use the global class names to customize the element you are interested into?

@herin0506 I'm confused by the issue's description. Could you please clarify the following:

The screenshot shows v5. The docs API urls shows v4. withStyles is a v4 API. Which version are you using?
Why did you not use the global class names to customize the element you are interested into?

I have updated the link and the added the snapshot.
I am working on creating the a library using storybook with all the material-ui components with few customization in look and feel. And exporting those components to consume it into my main application. So to achieve this we are using HOC approach, being the best option, and also making sure that styling of one component do not affect any other component.
Please feel free to ask more details so that I get the best solution for it.


Is the diode direction for SW17 labeled incorrectly?

Thank you for such a great 40% design--the extra thumb key and organic shape are delightful.
The diode direction on the PCB for switch 17 (left half) is reversed from the surrounding key switches, and from switch 38 (right half).
Is this by design, or a mislabeled diode direction?

I do not know if it's related, but I've mounted the diode on switch 38 in both polarities, but that half does not send any key codes.

@Wesztman
Would you mind answering this question when it's convenient? Thank you.

Hi @twalker!
Sorry for not responding, I had managed to turn of all notifications except @ 😉 so good call there!
Thanks for noticing, I checked my own keyboard and in KiCad and I seem to have rotated this at some point, probably just to get a shorter path for the traces. Not thinking about that it might be confusing in the future 😅
But you should mount all diodes according to the labels 😊 so the one on switch 17 should be switched.
Then there should be no issues, that is how it looks on my own keyboard 😊
It's worth noting that diodes should only be mounted on the "BOTTOM" side of the PCB:s, I might add that to the readme actually.
I should go ahead and add to my todo list to rotate the 17 diode in the next release 😊

Thank you for clearing up my confusion. Both reversing switch 17s diode direction, and noting that all diodes should be on the bottom of the PCB will help prevent confusion. All the other labeling on the board is quite clear.
I've done a fair bit of troubleshooting on the right half of my build and cannot get it to send keycodes. I'll rebuild the right half with more confidence on the diode orientation. Thanks again.

@Wesztman ,
FYI: I built a new right half following the diode labeling, and have successfully paired it with the left half. I'm pretty sure the issue with my build was not hardware-based, but not being able to pair the halves w/ZMK. However, the nice!view on the right half is still not displaying any information.
Should the nice!view display something on both displays when both halves have a display?

The nice!view should work on both halves :)

Thank you @iamDrakkir, that helps me rule out the firmware and narrow down my issue to the microcontroller.
What a classy looking build. I can't wait to get a case printed and be using a brain as a daily driver. 😸

FWIW, the root cause was the firmware. After flashing the erase reset firmware, the nice!view displayed properly and the halves paired.


auction: define domain and proto types

Part of #4196, this ticket tracks completion of adding the following domain types and protobuf definitions to the auction component:

[x] AuctionId
[x] AuctionNft
[ ] DutchAuctionDescription
[ ] DutchAuctionState
[ ] DutchAuction
[ ] ActionDutchAuctionSchedule
[ ] ActionDutchAuctionEnd
[ ] ActionDutchAuctionWithdraw
[ ] ActionDutchAuctionWithdrawView
[ ] ActionDutchAuctionScheduleView

Current shape of things:

the action handlers should be defined under penumbra_auction::component::action_handler::dutch.
the main Dutch* data structures live in the crate root and are namespaced via a auction::dutch module.
the general data structures e.g. AuctionId and AuctionNft live in the crate root in an auction module

This is done, we're circling back to the elementary data structures:

implementing auction description hashing
augmenting the auction nft with denom metadata

to make way for an action handler validation + execution push.


Faction Quest Test

Client code necessary to set up simple faction quest test.

Agent knows item is in room
Other agents can pick up item in room
Agent knows about other agents that could pick up item
Agent has to go retrieve item and give to quest giver

can we close this?

Oh yea this was done a long time ago.


Fix: Assistant builder instructions have max height

Description

Risk

Deploy Plan

Uncontroversial, merging @spolu @Duncid fyi


add share function in order to share rich content. see https://developer...

this new feature enable you to share rich content by API.

Hi @degzcs ! thanks for your contribution! Its a really great enhancement in wrapper. Would it be possible for you to update README.md as well so it could give public tutorial about how to use it? I am accepting pull request by the way. Thanks again! :+1:


Use a dict instead of a list for storing task definition revisions

Before when a task definition revision was de-registered all revisions after that changed their revision id. This doesn't match the way it is handled in AWS. Using a dict and manually increment the revision id solves that.

Coverage decreased (-0.06%) to 93.049% when pulling feef7b2b5ac662b97d331f58aa5e5027e170ed65 on labd:fix-ecs-task-definition-revisions into 9346a999d8d1ce726b71ad5dc8d95b1dd91b23c6 on spulec:master.

Looks great; thank you!


Documentiation fix

Both password & password confirm should be run through MD5 to match, Fix #3843

Nice catch.
However, usage of MD5 should be discouraged, so ... 0bb5cd4b1bc794f67f0089ae309ffa9b8000e831


Moderation of Comments

The Issue
Having comments that can not be moderated is a dream come true for trolls. Creators should not have to tolerate slanderous/abusive/degrading/hurtful comments directed at themselves on their own content.
Expected Behavior
Being able to either disable the comments section, or the ability to moderate the comments section on your own content.
Actual Behavior
No ability to disable/moderate comments.
Suggested Solutions
I know that comments can't be fully removed if they go on the blockchain, but it should at least be possible to disable/hide comments on the app side. This is possible on Steemit (scroll to the bottom of the post to see how comments are hidden).
Anything Else
Personally, I think it's less important for comments to be decentralized/censorship resistant. Everyone already have the ability to publish whatever they want on their own channels, should everyone have the right to spout whatever they want on other people's channels too? It's an open question.
I think the important distinction here is that one is someone doing what they want in their own little space, on their own channel, those who don't like it can simply ignore. The other is someone being able to do whatever they want in other people's space, on other people's channel/content.
My concern is that trolls being able to roam freely on creators content can quickly end up in creators leaving the platform.

Thanks for the write up @m160 ! We'll review and prioritize. Most of the work would need to be done from the UI side, but there would be no way to "block" comments on chain / on the database.

Moderating comments on the UI side sounds like a little censorship.
I think if comments are a subset of claims, then users could tip useful comments as well as abusive ones in case the published content really deserves it (as a manner of voting) and the UI could sort them out based on tips.
maybe setting a minimum of fee based on the popularity of the content (total amount of tips and support) to leave comments makes it less viable for trolls to do their annoying job.

For an organization that wishes to avoid setting standards for speech, I think it's very important that we provide tools to the channel page owner to do what they wish with it. I don't see how any market or democracy is necessary for you to kick an obnoxious person out of your house, so I don't think it's very central to censorship.

This might be either a bad suggestion or an obvious one, but it seems to me that it would be quite easy to have a special tag such as "comments disabled" which, if added to a claim, causes the (official) app not to display the comment section. This could be added to any new publish similar to how "mature" is a suggested tag with special behaviour.

The problem is that a creator wouldn't necessarily want to disable the comments as a whole, they'd just not want haters/spammers/etc to have a platform on their content. Some creators would feel forced to disable comments because of a few trolls, when what they really wanted was a comments section without trolls.
I think Aether has a very interesting solution to not only solve how to keep it fully decentralized, but also the problem of moderation. They even have a up/down voting mechanism figured out. Certainly worth exploring.
As far as I can tell, LBRY and Aether is a match made in heaven. LBRY got the decentralized content sharing sorted, Aether got the decentralized commenting/voting/moderation sorted. Burak, the developer, also seems like a super nice helpful guy.
I'd love to hear the thoughts of someone more knowledgable than me on this :)

I don't think we're going with a different solution if the long term goal is commenting on the blockchain, but maybe @kauffj / @grin / @seanyesmunt have some ideas here.
In the meantime, I think we should add the ability for channel owners to mark other comments as moderated, store this in the comments database, and use that information to alert other users that comments have been hidden. If they still want to see the comments under a hidden "these may be offensive/etc" section - should users be able to see the moderated comments?

@tzarebczan we're going to continue to build out and refine the centralized version of the system significantly more before moving to decentralized one.
My inclination is to something close to your second paragraph. Comments would not be permanently deletable by creator but people would have to take extra steps to see them. One reason this is important to support is that comments are one way of fighting abusive / misleading / malicious content.

For what it's worth I think the proposal in Tom's second paragraph is good (with users able to see the comments after a warning).

Blocked by 2 SDK issues (visible if you have ZenHub)

Just a small idea: in France there recently was an online experiment for collaborative legislation proposal platform (le vrai débat), and they faced the same issue. As a balance between freedom of speech and censorship of malevolent comments, what they did is that they first used a flagging and voting system to tag these comments, but the interesting part is that the comments were not deleted, but rather all separated from the main comment box and were still accessible in a "comments bin", which was still accessible to any reader but you needed to actively seek and click this link to read the malevolent comments (the link was quite small, at the very bottom and even separated from the main comment box). Unfortunately the platform is now closed so I can't provide a direct link.

Using #4947

Using #4947


PostgreSQL binding - Numeric data type column and uuid column response bug fix

Expected Behavior

When using PostgreSQL binding to run a query that gets a column of numeric data type, example:
select numeric_data_type_column from table
Then the response should be:
[[31680], [305300]]

For uuid column type, example query:
select uuid_type_column from table
Then the response should be:
[['c4d239d2-a9c1-41e8-b987-59b6fd228b6f']]

Actual Behavior

Response for numeric column actually is:
[[{'Int': 316800, 'Exp': -1, 'Status': 2}], [{'Int': 305300, 'Exp': -1, 'Status': 2}]]

Response for uuid column actually is:
[[[196, 210, 57, 210, 169, 193, 65, 232, 185, 135, 89, 182, 253, 34, 139, 111]]]

Steps to Reproduce the Problem
Create a table with at least one column of numeric data type and another column of uuid. Populate with some data.. Create a PostgreSQL binding. Run a select query for that column.
Release Note

For now the workaround is:

Cast numeric column as float. Then response is like expected.
Cast uuid column as text. Then response is like expected.

Good catch, are you willing to post a PR to fix it?

Unfortunately, it will take me way too long to do this due to work and a part time masters. Additionally, I have never worked in GO so perhaps it's better someone else takes care of this.
For now I would recommend users to use the workaround


Audio issue

Hi @pedroSG94 i'm on test then audio is interrupted while broadcasting
i'm using amazon midialive
my video audio setting code here
rtmpCamera2.prepareVideo(
myWidth,
myHeight,
30,
myBitrate,
2,
CameraHelper.getCameraOrientation(this),
MediaCodecInfo.CodecProfileLevel.AVCProfileBaseline,
MediaCodecInfo.CodecProfileLevel.AVCLevel5)
rtmpCamera2.prepareAudio(64000, 44100, true, true, true)
plz help me :(

Hello,
First of all, you need be more explicit.
Can you see audio packets in logcat when audio is interrupted? Can you share me a logcat in the moment that audio is interrupted?
Amazon live is working fine for other users so I think we can discard servers for now.
Also you should try without noise and echo filter because this filter could produce problems in few devices.

@pedroSG94  hi, i don't understand what kind of audio packet log cat you mean onNewBitRate Method Log?
and i tried that setting(noise, echo false), but the same thing happened
E/onNewBitrateRtmp: bitRate : 4032276
E/onNewBitrateRtmp: bitRate : 4392245
E/onNewBitrateRtmp: bitRate : 4048567
E/onNewBitrateRtmp: bitRate : 4269881
E/onNewBitrateRtmp: bitRate : 4108785
E/onNewBitrateRtmp: bitRate : 4343816
E/onNewBitrateRtmp: bitRate : 4089912
E/onNewBitrateRtmp: bitRate : 4136284
E/onNewBitrateRtmp: bitRate : 4160064
E/onNewBitrateRtmp: bitRate : 4224215
E/onNewBitrateRtmp: bitRate : 4081982
E/onNewBitrateRtmp: bitRate : 4301216
E/onNewBitrateRtmp: bitRate : 3910583
E/onNewBitrateRtmp: bitRate : 4262006
E/onNewBitrateRtmp: bitRate : 4158741
E/onNewBitrateRtmp: bitRate : 4359284
here's part of the logcat where the audio cuts out in the middle of the broadcast.

I mean audio and video packets that you can see in Android Studio console while streaming


Cannot input a text in TextField

Describe the bug
I have a ListView of Products with qty field. I want to update the TextField's text with the qty of the new selected product. The focus is still in the TextField but when I started typing, it wont reflect to the TextField. What could be the cause of this?
Steps to Reproduce

Execute flutter run on the code sample.
Select a Product
Focus on the TextField
Select another Product
Try typing on the keyboard. (TextField value will not change)

Expected results:
The TextField's should update in-respond to the keyboard input
Actual results:
TextField's value does not update when typing in keyboard

Code sample
import 'dart:math';

import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

class Product {
  final String name;
  int qty;
  Product(this.name, this.qty);
}

void main() {
  runApp(const App());
}

class App extends StatelessWidget {
  const App({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      debugShowCheckedModeBanner: false,
      home: Home(),
    );
  }
}

class Home extends StatefulWidget {
  const Home({Key? key}) : super(key: key);

  @override
  State<Home> createState() => _HomeState();
}

class _HomeState extends State<Home> {
  final controller = TextEditingController();
  final node = FocusNode();
  List<Product> list = List.generate(
      10, (index) => Product('Product ${index + 1}', Random().nextInt(5)));

  ValueNotifier<int> selectedIndex = ValueNotifier(-1);

  @override
  void initState() {
    super.initState();
    selectedIndex.addListener(() {
      controller.text = list[selectedIndex.value].qty.toString();
      node.requestFocus();
    });
  }

  @override
  Widget build(BuildContext context) {
    return SafeArea(
      child: Scaffold(
        body: Column(
          children: [
            Row(
              children: [
                ValueListenableBuilder<int>(
                  valueListenable: selectedIndex,
                  builder: (_, value, __) {
                    return Expanded(
                      child: value == -1
                          ? const SizedBox()
                          : Text(list[value].name),
                    );
                  },
                ),
                SizedBox(
                  width: 100,
                  child: TextField(
                    decoration: const InputDecoration(
                      fillColor: Colors.yellow,
                      filled: true,
                    ),
                    autofocus: true,
                    focusNode: node,
                    textAlign: TextAlign.right,
                    controller: controller,
                    textInputAction: TextInputAction.next,
                    keyboardType: TextInputType.number,
                    inputFormatters: [FilteringTextInputFormatter.digitsOnly],
                    onSubmitted: (s) {
                      if (int.tryParse(s) == null) return;

                      setState(() {
                        list[selectedIndex.value].qty = int.parse(s);
                      });

                      if (selectedIndex.value + 1 >= list.length) {
                        selectedIndex.value = 0;
                      } else {
                        selectedIndex.value++;
                      }
                      // controller.clear();
                    },
                  ),
                ),
              ],
            ),
            Expanded(
              child: ValueListenableBuilder<int>(
                valueListenable: selectedIndex,
                builder: (_, value, __) {
                  return ListView.separated(
                    itemCount: list.length,
                    separatorBuilder: (_, i) => const Divider(),
                    itemBuilder: (_, i) {
                      var qtyText = list[i].qty.toString();
                      if (qtyText == '0') {
                        qtyText = '---';
                      }
                      return ListTile(
                        key: ValueKey(i),
                        selectedTileColor: Colors.blue,
                        selectedColor: Colors.white,
                        onTap: () {
                          selectedIndex.value = i;
                        },
                        selected: i == value,
                        title: Text(list[i].name),
                        trailing: Text(qtyText),
                      );
                    },
                  );
                },
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Logs

flutter analyze
Analyzing text_field_bug...
No issues found! (ran in 2.6s)

[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.19044.1526], locale en-PH)
    • Flutter version 2.8.1 at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (9 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\rjgbo\AppData\Local\Android\Sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code, 64-bit edition (version 1.58.2)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.25.0

[√] Connected device (4 available)
    • SM A705MN (mobile)                 • R58M61HN7PB   • android-arm64  • Android 11 (API 30)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 5.1.1 (API 22) (emulator)
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 98.0.4758.82
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 98.0.1108.50

! Doctor found issues in 1 category.

Hi @b4ltz, there's a similar issue reported in https://github.com/flutter/flutter/issues/98352.
I have verified that the behavior is the same in both issues. Kindly follow up in https://github.com/flutter/flutter/issues/98352 for more information.
Closing from here as duplicate.
Thank you


Generate “Contained In” section in doc comments

Similar to the Children section for syntax nodes, I think it would be very helpful to also have a section that mentions all the syntax nodes that a node occurs in to traverse the syntax tree structure upwards.

Tracked in Apple’s issue tracker as rdar://112433203


Add up and down movement to free flight camera

Do we need to update the Java bindings?

Yup, I'm on it


Certification doesn't work

Hi, sorry to trouble you.
When executing the guide to build the sample project, everything works fine until the plugin is installed and the server restartet.
Trying to log in yield the following error:
com.mirth.connect.client.launcher.q: java.lang.Exception: Error verifying entry "META-INF/MANIFEST.MF" in JAR file mirthpluginsample-client.jar java.util.concurrent.ExecutionException: com.mirth.connect.client.launcher.q: java.lang.Exception: Error verifying entry "META-INF/MANIFEST.MF" in JAR file mirthpluginsample-client.jar 	at java.base/java.util.concurrent.FutureTask.report(Unknown Source) 	at java.base/java.util.concurrent.FutureTask.get(Unknown Source) 	at com.mirth.connect.client.launcher.f.a(SourceFile:416) 	at com.mirth.connect.client.launcher.f.a(SourceFile:371) 	at com.mirth.connect.client.launcher.MirthClientLauncher.run(SourceFile:1326) 	at java.base/java.lang.Thread.run(Unknown Source) Caused by: com.mirth.connect.client.launcher.q: java.lang.Exception: Error verifying entry "META-INF/MANIFEST.MF" in JAR file mirthpluginsample-client.jar 	at com.mirth.connect.client.launcher.r.a(SourceFile:266) 	at com.mirth.connect.client.launcher.r.a(SourceFile:204) 	at com.mirth.connect.client.launcher.f.a(SourceFile:542) 	at java.base/java.util.concurrent.FutureTask.run(Unknown Source) 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) 	... 1 more Caused by: java.lang.Exception: Error verifying entry "META-INF/MANIFEST.MF" in JAR file mirthpluginsample-client.jar 	at com.mirth.connect.client.launcher.r.a(SourceFile:253) 	... 6 more Caused by: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found. 	at org.bouncycastle.jce.provider.PKIXCertPathValidatorSpi_8.engineValidate(Unknown Source) 	at java.base/java.security.cert.CertPathValidator.validate(Unknown Source) 	at com.mirth.connect.client.launcher.r.a(SourceFile:351) 	at com.mirth.connect.client.launcher.r.a(SourceFile:245) 	... 6 more
I guess the self-signed certificate provided in the plugin is not valid any more?
What are the steps to create a own self-signed certificate?
Thanks a lot!

You're probably using the launcher, but in it's regular more where it's quite strict in which certificates it trusts. You should find the solution here :wink:

Thank you for your response.
I found the -k switch in the meantime as well. Works now for me.
Is there any way to circumvent needing to use the -k switch?
I fear I'd have to have a certificate validated by NextGen?


Schedule fix

Created a popup to when you check for the schedules using the schedule button.

I resolved the merge conflicts locally but didn't know if I should push them.

I resolved the merge conflicts locally but didn't know if I should push them.

Yes you can push them, it automatically updates the PR

Rant into issues while merging, I'll reopen once I have resolved them


config file naming issues

should be able to use config.yml or config.yaml. The default filename is time_entries.yml so it doesn't make sense that the config file (by default) would use yaml as the extension. So either pick .yml or look for and allow either.

If --config option not specified, should probably try multiple options:
Default directory:

config.yaml
config.yml
config.toml

Current directory:

config.yaml
config.yml
config.toml


Unable to freely select week in weekly view

cannot freely select which week to view in the weekly view. It only displays the current week's schedule, but I want to review my previous plans.

但是我并没有这个选项，

I don't have those control buttons either.

Okay, @zhang-962464, seems like you're using iOS, right?
@gunererd, is this the default theme? Looks like your theme hides the buttons.

@ivan-lednev here I disabled all plugins and set default theme then restarted obsidian but still no luck with those buttons.

@gunererd, make sure this option is enabled:

@ivan-lednev yep that was the case. I can see buttons now, sorry for inconvenience and thank you for your time.

In 0.25.0 there is a side bar in the multi-day view, that always stays visible.


Secure websockets on self-signed certificates

It does seem like self-signed certificates can't be tested because wsstat can't automatically accept unverifiable certificates. Is this something that an option can be added for?

I can solve this problem on my end with:
change line 9 to just import ssl and
line 137: websocket = yield from websockets.connect(**connection_args, ssl=ssl._create_unverified_context())

Hey @fwsGonzo - sorry for the delay! I can add that as an option for sure, I'm just really busy with work stuff right now so it might take a bit.
Cheers!


(No. 3) "Begin" Button not working on IE [ Easy ]

Reported by Anne via feedback doc

Note for Jocelyn: Open Layers 4 may not be supported by Internet Explorer. Ryan is working to identify a fix.

Note for Jocelyn: Open Layers 4 may not be supported by Internet Explorer. Ryan is working to identify a fix.

fixed as much as it can be.


Fixed cr3 display successfully but module DLL still couldn't get base address

Fixed cr3 display successfully but module DLL still couldn't get base address

How to solve

I don't know without knowing exactly what you're doing

I don't know without knowing exactly what you're doing

Due to the game update two days ago, your CR3 decryption is outdated and the game has fixed it

I don't know without knowing exactly what you're doing

Due to the game update two days ago, your CR3 decryption is outdated and the game has fixed it

It still works fine on how i do it on my private version of the library. Which is similiar to the public one. I'm sure with a slight change you can make it work again, if it even is broken (i doubt it)

I don't know without knowing exactly what you're doing

Due to the game update two days ago, your CR3 decryption is outdated and the game has fixed it

It still works fine on how i do it on my private version of the library. Which is similiar to the public one. I'm sure with a slight change you can make it work again, if it even is broken (i doubt it)

You really overestimate me, I can't resist kneeling down and begging you to update!

GetBaseDaddy

It's not an issue with GetBaseDaddy
But it's CR3 ↓
He will get stuck in ↓
while (true)
{
BYTE bytes[4] = { 0 };
DWORD i = 0;
auto nt = VMMDLL_VfsReadW(MEM::vmm_handle, const_cast(L"\misc\procinfo\progress_percent.txt"), bytes, 3, &i, 0);
if (nt == VMMDLL_STATUS_SUCCESS && atoi(reinterpret_cast(bytes)) == 100)
break;
std::this_thread::sleep_for(std::chrono::milliseconds(100));
}
---------------Although I passed the above
however
while (std::getline(iss, line))
{
Info info = { };
std::istringstream info_ss(line);
if (info_ss >> std::hex >> info.index >> std::dec >> info.process_id >> std::hex >> info.dtb >> info.kernelAddr >> info.name)
{
if (info.process_id == 0) //parts that lack a name or have a NULL pid are suspects
possible_dtbs.push_back(info.dtb);
if (client11().find(info.name) !=  std::string::npos)
possible_dtbs.push_back(info.dtb);
}
}
It's stuck here again, and I let him pass through
for (size_t i = 0;  i < possible_dtbs.size();  i++)
{
auto dtb = possible_dtbs[i];
VMMDLL_ConfigSet(this->vmm_handle,  VMMDLL_OPT_PROCESS_DTB | this->PID, dtb);
result = VMMDLL_Map_GetModuleFromNameU(this->vmm_handle, this->PID, const_cast(client11().c_str()), &module_entry, NULL);
if (result)
{
//printWithTimestamp1("[+] Initialize succed！\n");
return true;
}
}
It returns normally here, but it's an old DTB cache. I updated the cache, but I can't do anything about it
I suspect that the game detected these TXT files. You can make it a memory read
But I don't have any ideas, so I'm begging you!

CR3 didn't get patched, your issue is it's not loading the VMM plugin.

VMMDLL_CloseAll(); and  VMMDLL_MemFree(this->vmm_handle);
I call it every time! Before the next reinitialization! Because yesterday after the CS2 battle platform update, the second one will be like this. I suspect that a large amount of garbage was put in it

CR3 didn't get patched, your issue is it's not loading the VMM plugin.

while (std::getline(iss, line))
{
	Info info = { };

	std::istringstream info_ss(line);
	if (info_ss >> std::hex >> info.index >> std::dec >> info.process_id >> std::hex >> info.dtb >> info.kernelAddr >> info.name)
	{
		if (info.process_id == 0) //parts that lack a name or have a NULL pid are suspects
			possible_dtbs.push_back(info.dtb);
		if (client11().find(info.name) != std::string::npos)
			possible_dtbs.push_back(info.dtb);
	}
}

I have basically confirmed that the problem lies in the information On DTB

I know the problem now, please close this issue directly. Thank you! Just replace it with memory reading. I'll use VMMLL_readEX instead

Brother, how did you solve it?

我现在知道问题了，请直接关闭这个问题。谢谢！只需将其替换为内存读取即可。我将改用VMMLL_readEX

Brother, how did you solve it?


feat: add css modules in button

description

add css modules to button

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support push down of regexp_xxx functions

Feature Request
Is your feature request related to a problem? Please describe:

Describe the feature you'd like:

Functions need supported:

[ ] regexp_like
[ ] regexp_instr
[ ] regexp_substr
[ ] regexp_replace

Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

/assign


tests: fixing old unit tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (fix/syslog-updates@6362a20). Click here to learn what that means.
The diff coverage is n/a.

@@                  Coverage Diff                  @@
##             fix/syslog-updates     #661   +/-   ##
=====================================================
  Coverage                      ?   85.06%
=====================================================
  Files                         ?       53
  Lines                         ?     2967
  Branches                      ?        0
=====================================================
  Hits                          ?     2524
  Misses                        ?      443
  Partials                      ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update to USLCI mapping

update to mis-mapped flow.
Phthalate, dioctyl-
63e7fe64-8549-11e0-9d78-0800200c9a66
in comparing with OpenLCA this same flow was mapped to two different flows. upon further review it was found this flow was mapped incorrectly. This corrects the issue and the flow is not mapped in OpenLCA and USLCI to the same FEDEFL flow.

Updated TRACI 2.1
TRACI2.1 also had a misidentified chemical with various contexts. The mapping flowable name was corrected along with the UUID

@ashleyedelen the update to TRACI needs to be handled in the TRACIFlowables Mapping file in the inputs folder. Then by running the script we can regenerate the actual mapping file.

@bl-young just checking, this is different than making a correction with the openLCA mapping. There i can just correct the mapping file, correct?

@bl-young so do i need to undo what i did in the mapping file of TRACI2.1?

Corrected the TRACI update, the input TRACI flowable file was corrected. Per @bl-young once running the script this will correct the output file and the mis-matched TRACI flowable will be corrected.

@ashleyedelen confirming that the TRACI mapping file was updated (note one UUID changed that was missed earlier) here: https://github.com/USEPA/Federal-LCA-Commons-Elementary-Flow-List/commit/31e1f3d722589e4826ca53141219f1ed20c6a9b3


Add support for topology

@mphoward I think I have this properly storing the bonds data to the snapshot and writing bonds data when creating a data file
Once we're good with how bonds are set up, I'll implement analogous code for angles and dihedrals!

FYI: the unit tests are failing due to an incompatibility between a new version of pytest and pytest-lazy-fixture. The latter is not currently maintained, so we are going to need to replace it with something else where it's used.

@mphoward this should be ready for your review!

@mphoward this should be ready for your review!

@mphoward I've implemented the suggested changes. This is ready for another review!

@mphoward tests have been added for topology and writing/reading their data. Whenever you get a chance, let me know what feedback you have on those!

@mphoward I've moved some tests around and expanded on this. There is still a lot of duplication, but I am not quite sure where / how would be the best way to address that.

@mphoward this should be ready to merge pending a final review!


v0.2.7

upstream: https://github.com/arnaudmiribel/streamlit-extras/tree/v0.2.7
Notes

Upstream notes that streamlit doesn't work with python 3.9.7, this is the reason for the run pinnings and linter error.
Upon approval this will be uploaded to the Snowflake channel.

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_a9g5qeaqd0/clone/recipe/meta.yaml:26: remove_python_pinning: On arch specific packages, python deps should not be constrained.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_ef9mfiawcf/clone/recipe/meta.yaml:26: remove_python_pinning: On arch specific packages, python deps should not be constrained.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_408u85us3y/clone/recipe/meta.yaml:26: remove_python_pinning: On arch specific packages, python deps should not be constrained.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_78r6spyfee/clone/recipe/meta.yaml:26: remove_python_pinning: On arch specific packages, python deps should not be constrained.
Errors were found


Continued pretraining facing catastrophic forgetting

Hi @danielhanchen
I tried fine-tuning llama 3.2-1b base model for 2 of my tasks following below example notebook
https://colab.research.google.com/drive/1tEd1FrOXWMnCU9UIvdYhs61tkxdMuKZu?usp=sharing#scrollTo=MKX_XKs_BNZR
Below is the observation:
1> I evaluated the model after fine-tuning for the first task -- model performed well.
2> I continued the training of the model/adapter for my task2 -- model performs well for task2 but apparently forgot learning of task 1.
I am facing catastrophic forgetting in this case. I have kept the sample size of both the training almost identical.I have tried to play with the learning rate while training for the second [tried lower learning rate than the first one], still did not workout.
Are there any guidelines/suggestions to make it work. My requirement is to progressively train the model on N tasks.

@InderjeetVishnoi This does not sound like an unsloth issue, but a general fine tuning/training guideline overall, and this is the wrong forum for it.
If you want suggestions or guidelines I would suggest you join the discord channel, many users are helping out there.
https://discord.gg/unsloth

@Sneakr Point taken :)

You can maybe experiment with Concatenate https://huggingface.co/docs/datasets/process your datasets together . Also try bigger ranks when doing so to give the model more space to learn more complicated tasks. Also, due to higher rank, you can try rslora=True  (or false) to experiment differences. GL! :)


Add methods to check the properties of a Number

Having some methods to check the properties of a Number could be useful. For example, isValidLong() to check whether a number can be converted to a valid long or isFiniteDouble to check whether a number can be converted to a finite double.

If I understand correctly, you want some methods to test if a Number can be decoded exactly as a Long or Double.
I don't like the suggested names:

isValidLong() implies that there are invalid Long.
isFiniteDouble() all Double representations of reals except infinity are finite. Same for Number.

How about: decodesExactlyToLong(), decodesExactlyToDouble(), ...

Number class has been removed in 84b894a2a7eb447ffd6da967fa507b47ca9319b5.


Automation init

@claymcleod I need a CLOUDYTOKEN and CLOUDYUSER for this repo

@adthrasher what version should we start this at?  is there a sem ver internal that we can align with?

@adthrasher what version should we start this at? is there a sem ver internal that we can align with?

The DNAnexus app (https://platform.dnanexus.com/app/stjude_rnapeg) is at 1.3.0. So I'd say sync with that.


⚠ ClamAV has degraded performance

In 7f222b5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 83 ms

Resolved: ClamAV performance has improved in 4771dd5.


Ledger Nano X integration

Tx Signing with ledger using V1 rust bindings (hack needed)
As suggested by Sebastian, I used a hack based on this PR by Ruslan. In short, a tx is first signed using rust v1 bindings, then it is parsed in JS using cbor. The inputs and outputs from rust are provided to Ledger, and finally the witnesses of the original rust tx are replaced by the witnesses given by Ledger. The tx is then re-encoded in JS using cbor.
The logic for encoding/decoding rust txs is located in  src/crypto/byron/util.js
Note: the encodeTxAsRust function doesn't generate exactly the same bytes as rust (it differs in a few bytes of metadata I guess). However, the backend seems to decode it correctly.
Some observed issues
Trying to connect to a device that has been idle for some time seems to fail, and a restart is needed. The best is to do all operations right after inserting the PIN and opening the ADA app (and actually, waiting for at least ~5 seconds after opening the app)
New dependencies

@cardano-foundation/ledgerjs-hw-app-cardano 1.0.8: Ledger operations
@ledgerhq/react-native-hw-transport-ble 5.13.0: Ledger connection
react-native-ble-plx 1.1.1: bluetooth connection. An older version of this library was installed because latest version (2.x) uses RN 0.60+. We'll need to upgrade this library once RN version is upgraded too.
cbor 5.0.2: Needed for hack explained above
jsc-android: Required by @ledgerhq/react-native-hw-transport-ble

Features for follow up PR

Address verification (UI needed)
USB-cable connection support (this can only be supported for Android though)
perhaps some minor UI improvements
The "learn more about using yoroi with ledger" must point to some relevant link (video or tutorial), which currently doesn't exist now.

Modified screens
Add wallet screen

Send screen
Only added a few instructions, but might require some improvements.

New screens
Check Nano X screen

Scrolling down shows more text in Android

Connect Nano X screen

Save Nano X screen

Trying to connect to a device that has been idle for some time seems to fail, and a restart is needed. The best is to do all operations right after inserting the PIN and opening the ADA app (and actually, waiting for at least ~5 seconds after opening the app)

In the browser extension, I think we had to increase the timeout, but there is a browser limit so after the timeout we automatically fail the prompt. This is different from what you're doing (different transport mode and not a browser), but just so you're aware.

Feedback:

remove wallet no longer relevant message asking you to make sure you write down your 15 words.
"change password" no longer relevent with Ledger confirmation for this kind of wallet
easy transaction confirmation" no longer relevent with Ledger confirmation for this kind of wallet


Attempt to clarify Intro paragraph #2

Thanks for contributing to Community Yellow Paper! We are currently focused in quickly expanding technical content, in order to cover all proposed sections of the document. However, other contributions may also be accepted.
Please mark the intention of this PR:

[ ] Expanding technical content
[ ] Fixing typo
[x] Clarifying confusing/misleading sentences
[ ] Other

===============================================
Please briefly explain the proposed changes:
I've tried to preserve the original content but make it a little more digestible by breaking down big sentences into a couple of smaller ones.

Additionally I tried to enter a bit of whitespace to improve readability. I couldn't see it appear in the preview so I may be missing something in markdown for how to enter whitespace?

Eabueahuea, no, remove these white space, this is not good :dancer:

@PBSJ, I adjusted the paragraph once more.
It almost got me...There were some adjustments that changed the real meaning we would like to pass. :+1:  hauehauaeea
Let's be careful with the changes.
Thanks for the contributions. I tried to keep the core of your modifications.

Take another look before we merge this.

Hahah we are going back on forth on this eh @vncoelho  :-)
Hmm, so yes I definitely do not want to change the intent of the message you are trying to convey.
My only desire here is to make it easier to understand for an English reader.
I fully appreciate that English is not everyone's first language (it is my first language but I suck at almost every other language!).
However there are a few grammatical anomalies which make understanding the message quite hard for me as a native English speaker.
For example this phrase 'it is noteworthy that SC's transactions persisting... pose a unique set of challenges' isn't very easy to read - as I understand it you mean to say 'it's worth noting that persisting SC transactions... poses a unique set of challenges' - in other words, the difficult thing is to be able to persist the Smart Contract's transactions. Is that correct?
So, I'm just wondering what is the best way to proceed - because I feel a bit bad if I'm constantly correcting English etc, but on the other hand I was hoping to make it easier to read.
But the two things I don't want to do are:

Change the intent of the message
Insult the author ;-)

let's keep going @PBSJ , you're doing a nice job... and Vitor as well, for handling these revisions so quickly... I can also contribute in the PR process, but I'll take longer to respond. Anyway, in short-term this document will be so much better, congratulations.

Ok thanks @igormcoelho :-)
I'll put through a revised version now - which hopefully preserves all of your intended message and also flows nicely in English - let me have a look...

Ok, so just had another attempt @igormcoelho , @vncoelho
One of things that I think will help with readability is to be more explicit than implicit.
For example, I think it's worth specifying 'blockchain literature' rather than just 'literature' as it's less ambiguous.
Similarly sometimes a sentence will refer to 'it' or 'its'. And it can be quite tricky to understand what the 'it' refers to. So I have been more explicit here to make it easier for the reader to follow.
Have a look now :-)

I will take a look with careful again, @PBSJ.
It should "In the Literature" for the first time, otherwise it is repetitive because below we already use blockchain.

Please do not make any other change with upper case letters.

noteworthy is clear and is classical.

The phrase was again with wrong meaning commenting about append storage applications.

Thanks for the contributions, I like this phrase: "DApps utilising SCs tend to be more complex to develop and optimize when compared to DApps focused only on appending storage", congratulations! :dagger:

Thanks @vncoelho !
Can I ask what is meant by this sentence as I'm still struggling to understand it:

"Besides its significant advantages, satisfying this set of requirements poses additional constraints, vulnerabilities and challenges when compared to other consensus applications dealt in the literature."

What does 'its' in 'Besides its significant..." refer to here?
Also, which set of requirements are we referring to?

Everything mention in the current paragraph. Bassically things, OBF and SMR.

Ok, I think I understand the message we're trying to convey - maybe I'll leave it for now and have a look through some more of the doc :-)

Do not  be shy, fell free to open more PRs...ahauaha
Thanks for the contributions.
If you want to open more you are welcomed. I think that small changes are easier for us to analyse and merge, as well as to track master's updates. Thus, better to split PRs.

Yes, thanks - I was liking your small changes.
I need to get more familiar with GitHub - it's the first time I've used it :-)

You are doing well. ;D
We keep in touch. Time to back to sleep here,

Ay ay, it's late there haha. Still evening here in NZ. Have a good sleep


DateField widget uses TextInput

This is a feature suggestion. It would be nice to have the rendered HTML for DateField be type=date.
I don't think there should be any browser comparability issues since according to the MDN it "degrades gracefully" to type=text

wtforms.fields.html5.DateField


Add inference-with-keras.ipynb to PaliGemma docs

Adds a copy of the Inference with Keras notebook for the PaliGemma docs.
Description of the change
Uploads a copy of the Inference with Keras notebook. The only difference here is that this copy also includes the Apache License 2.0 info.
Motivation
We need a copy of the notebook in GitHub so that users can open the notebook in Colab directly.
Type of change
Documentation
Checklist

[x] I have performed a self-review of my code.
[x] I have added detailed comments to my code where applicable.
[x] I have verified that my change does not break existing code.
[x] My PR is based on the latest changes of the main branch (if unsure, please run git pull --rebase upstream main).
[x] I am familiar with the Google Style Guide for the language I have coded in.
[x] I have read through the Contributing Guide and signed the Contributor License Agreement.

Failing lints, we should fix.

Fixed the Apache License and the copyright and the malformed link. IMO in this case the "we've" is okay since it's stating something that we (the PaliGemma team) have done literally and not an expectation on the user.


ARROW-4649: [C++/CI/R] Add nightly job that tests the homebrew formula

This doesn't just test brew install apache-arrow --HEAD as the ticket suggested--it brings the Homebrew formula under our source control and tests against that. This will enable us to know when we need to update the Homebrew formula due to changes in dependencies or cmake configuration.
I've been testing out this approach on a different repository: https://travis-ci.org/nealrichardson/arrow-brew/builds/568531245

@ursabot crossbow submit homebrew-formula

This fails, and it is correct that it fails. Compare the previous passing build I linked with https://travis-ci.org/nealrichardson/arrow-brew/builds/583867759, run on that repository just now.
The failure is
[ 19%] Generating Flight.pb.cc, Flight.pb.h, Flight.grpc.pb.cc, Flight.grpc.pb.h
cd /tmp/apache-arrow-20190911-98334-mvj5bo/build/src/arrow/flight && /usr/local/bin/protoc -I/tmp/apache-arrow-20190911-98334-mvj5bo/cpp/../format --cpp_out=/tmp/apache-arrow-20190911-98334-mvj5bo/build/src/arrow/flight /tmp/apache-arrow-20190911-98334-mvj5bo/cpp/../format/Flight.proto
cd /tmp/apache-arrow-20190911-98334-mvj5bo/build/src/arrow/flight && /usr/local/bin/protoc -I/tmp/apache-arrow-20190911-98334-mvj5bo/cpp/../format --grpc_out=/tmp/apache-arrow-20190911-98334-mvj5bo/build/src/arrow/flight --plugin=protoc-gen-grpc=/usr/local/Cellar/grpc/1.23.0_2/bin/grpc_cpp_plugin /tmp/apache-arrow-20190911-98334-mvj5bo/cpp/../format/Flight.proto
dyld: Library not loaded: /usr/local/opt/gperftools/lib/libprofiler.0.dylib
  Referenced from: /usr/local/Cellar/grpc/1.23.0_2/bin/grpc_cpp_plugin
  Reason: image not found
--grpc_out: protoc-gen-grpc: Plugin killed by signal 6.
make[2]: *** [src/arrow/flight/Flight.pb.cc] Error 1
make[2]: *** Deleting file `src/arrow/flight/Flight.pb.cc'
make[1]: *** [src/arrow/flight/CMakeFiles/flight_grpc_gen.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 19%] Built target arrow_dataset_objlib
make: *** [all] Error 2

I don't think that failure should block the merging of this patch: it might be more appropriate to fix the formula on its own ticket, just to track the issue and make clear at release time that there's been a change to the homebrew formula that we'll need to publish upstream.
Thoughts @kou @xhochy ?

I think that the grpc Formula in Homebrew misses the gperftools Formula dependency.
Could you send a pull request to https://github.com/homebrew/homebrew-core ?
diff --git a/Formula/grpc.rb b/Formula/grpc.rb
index dff0ba50cb..9b3a98225a 100644
--- a/Formula/grpc.rb
+++ b/Formula/grpc.rb
@@ -17,6 +17,7 @@ class Grpc < Formula
   depends_on "libtool" => :build
   depends_on "c-ares"
   depends_on "gflags"
+  depends_on "gperftools"
   depends_on<EMAIL_ADDRESS>   depends_on "protobuf"

Did we change something since August 6 that now requires gperftools? grpc hasn't changed wrt gperftools in that time. Should we ourselves just add a dependency on gperftools?

This is not related to the apache-arrow formula.
Because grpc_cpp_plugin provided by the grpc formula uses libprofiler.0.dylib:
$ otool -L /usr/local/Cellar/grpc/1.23.0_2/bin/grpc_cpp_plugin
/usr/local/Cellar/grpc/1.23.0_2/bin/grpc_cpp_plugin:
	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1575.17.0)
	/usr/local/opt/protobuf/lib/libprotoc.20.dylib (compatibility version 21.0.0, current version 21.1.0)
	/usr/local/opt/protobuf/lib/libprotobuf.20.dylib (compatibility version 21.0.0, current version 21.1.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1252.250.1)
	/usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.11)
	/usr/local/opt/gperftools/lib/libprofiler.0.dylib (compatibility version 5.0.0, current version 5.18.0)
	/usr/local/opt/c-ares/lib/libcares.2.dylib (compatibility version 6.0.0, current version 6.0.0)
	/usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 400.9.4)

I think that gRPC detects gperftools automatically and gperftools is installed unexpectedly on environment that builds bottles for the grpc formula.

Sure. I'm just saying, I got this formula to build successfully on Travis most recently on August 6. And obviously it built fine at our last release. So do we have a theory for why this is now a problem?

I think that this is caused by https://github.com/Homebrew/homebrew-core/commit/8defc4aca6bf6730bddeea45ad58e521fa80364f . (Auguest 24)

Or https://github.com/Homebrew/homebrew-core/commit/f48ab37f2a03f39d630bf0a668223134884d3ed9 (Auguest 16)

Or https://github.com/Homebrew/homebrew-core/commit/2cc1d027562960f2f25c3054bef21b798892e49a (September 9)

So it was just luck that wherever the bottles were made before had it, and suddenly it doesn't?

I think that we should ask to Homebrew developers. :-)

👍 ok I can initiate that at Homebrew-core tomorrow (unless you want to right now); in the meantime, do you want to accept this patch? It's not like a(nother) failing nightly build is going to hold anyone up here.

Did you copy ci/apache-arrow.rb from https://github.com/Homebrew/homebrew-core/blob/master/Formula/apache-arrow.rb ?
Then the license isn't Apache 2.0. It's BSD 2-Clause License https://github.com/Homebrew/homebrew-core/blob/master/LICENSE.txt .

I did. So I'll add to our LICENSE file then. Good catch, sorry about that.

@nealrichardson could you move every the formula under dev/tasks/brew-formulas, similarly like other packaging tasks' assets are placed under dev/tasks.
Also we need to add a license note, that we have copied code from homebrew, it has BSD-2, similarly like we reference the vendored the conda-forge recipes.

@ursabot crossbow submit homebrew-formula

Ok, feedback addressed, and Homebrew-core has resolved the grpc issue, so the build passes now.
I was able to add the version updating to the release script, but I didn't see where it was appropriate to update the sha after the release happens. But even if that remains a manual step, we're still better off than we were before.

@ursabot crossbow submit homebrew-cpp

@ursabot crossbow submit homebrew-cpp-autobrew

Revised, PTAL

@ursabot crossbow submit homebrew-cpp-autobrew

Codecov Report

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

@@            Coverage Diff            @@
##             master    #5360   +/-   ##
=========================================
  Coverage          ?   66.14%
=========================================
  Files             ?      504
  Lines             ?    69550
  Branches          ?        0
=========================================
  Hits              ?    46001
  Misses            ?    23549
  Partials          ?        0

Continue to review full report at Codecov.

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

@ursabot crossbow submit homebrew-cpp

@kou or @kszucs could you please take another look and merge if you're satisfied?

Ok, I've addressed @kou's feedback and the lint/release tests pass: https://travis-ci.org/apache/arrow/jobs/587562985

@nealrichardson it looks good to me, I only have concerns with the file organization.

@ursabot crossbow submit homebrew-cpp homebrew-cpp-autobrew

@ursabot crossbow submit homebrew-cpp-autobrew

https://travis-ci.org/nealrichardson/arrow/jobs/587687890 lint/release test passing again. Once homebrew-cpp-autobrew passes, this is good to go.

@kou we are finally ready to merge here. Thanks for all of your feedback!


feat: support JDBC PreparedStatement

Adds integration tests that use the native PostgreSQL driver to connect
to PGAdapter, and tweaks the protocol response from PGAdapter to be in
line with what real PostgreSQL does. Notable changes to protocol:

standard_conforming_strings should return 'on' instead of 'true'
Describe Portal should respond with NoData if the portal does not
contain any columns (i.e. it is an update count)

Adds support for decoding parameters that have been given in binary
format.
This change also fixes a wrong assumption in IntermediateStatement that
a zero update count means NO_RESULT. Zero update count means that no
rows were updated/deleted by the UPDATE or DELETE statement.

Replaces #20

@Vizerai I don't know if you had already started looking into this, but I've broken it up into a number of smaller parts to make it a little easier:

#27 Support binary parameter values
#28 Add JDBC integration tests and make sure we can connect using the native PG JDBC driver
#29 Determine statement type based on the actual SQL string

Once those are merged, I'll rebase this one so we can get in all the JDBC integration tests.

Closing as this has been replaced by a number of smaller PRs containing the same plus additional features.


Email service integration

Add email service compilation as described in https://github.com/liteflow-labs/docs/pull/31.
The service returns an email preview using ethereal to visualize your email in dev mode without sending it, example: https://ethereal.email/message/XyJyzHFTXFPlNdBiXyJ0yFC6n1Sd-BVrAAAAAuVBD7hN5cxbPQjPHtG2hNw
TODO

[x] Decide on a default "from" address for the email https://github.com/liteflow-labs/service-email/blob/a4bab581cf60f5581a55130d9c7b5fe3d1b1debb/src/index.ts#L37. Ideally it should be an email set in the project and sent in the context
[x] Deploy a production-ready instance of the service https://github.com/liteflow-labs/service-email
[x] Setup using sendgrid SMTP API

@NicolasMahe could you take care of the deployment of this service.

[ ]  Decide on a default "from" address for the email https://github.com/liteflow-labs/service-email/blob/a4bab581cf60f5581a55130d9c7b5fe3d1b1debb/src/index.ts#L37. Ideally it should be an email set in the project and sent in the context

I would go with:
from: `"${ProjectName} at Liteflow"<EMAIL_ADDRESS>

We can let the user set their own email. Sendgrid asks to verify your domain so we could do the same and validate domain name of the project. One way to do as well is to generate an address eg<EMAIL_ADDRESS>and receive all emails then forward them to the user's email. This is what is happening to systems like zendesk or similar

Email service deployed and instanceId updated in this PR


Escape special characters in module lookup error

Things like this can happen:

While not critical, it seems like something that should be fixed.

Doesn't really seem like it fixed that issue.

ohh right that's my bad. hold on

(Removed the code that escaped the other characters too; I forgot that all characters are technically fine when inside backticks. Except backticks.)

This solution is way too hacky and unconventional


refactor generic email template

refactor generic email template

@gnovaro
We have added a default template for emails.
We have modified the generic view of the emails so that it uses the default template.
we have created a TicketStateChanged Mailable with its corresponding view.


Headless UI?

Accessibility is super important in my environment, and DaisyUI has real issues when it comes to accessibility.
Have you considered using a UI set like Headless UI that puts accessibility first?

Yes, this week we'll publish our initial headless pacakge, but that's the idea - to have a headless package and to have "themes", so daisy will only be the theme but all the controls and logic will be handled by the @qwik-ui/headless

Thanks for clarifying, good luck!

sure, thanks!


Fix error that happened when not in Jupyter

  Message='charmap' codec can't encode characters in position 73335-73353: character maps to <undefined>
  Source=C:\Users\jakka\Documents\Visual Studio 2019\Projects\Eric-Wieser-Lecture\Eric-Wieser-Lecture\Eric_Wieser_Lecture.py
  StackTrace:
  File "C:\Users\jakka\Documents\Visual Studio 2019\Projects\Eric-Wieser-Lecture\Eric-Wieser-Lecture\Eric_Wieser_Lecture.py", line 18, in <module>
    draw([S])

Thanks! I'll do a 0.11.0 release later today.


disabled "Single session per user" not working

hello,
I understand that multiple sessions were no longer functional on the Win 10 Pro version.
is there a solution ?
is older version of windows 10 pro compatible?
Thanks for your help

same problem here # 358

for information, the KB4284848 add great patch for remote desktop ;)

did the update fix the issue for you?

I did not test on the server part, but the patch on the client avoids display anomalies related to the right clicks

Didn't change the situation for me.. Though, yes, good to hear about that fix.

Would be nice to have a fix for this, as a Windows reinstall is not an option for me currently. It's really weird because it used to work for me a couple days ago, but today it just stoppped. Multiple accounts are still fine.

Same strange problem here with Win10 1803 ....
Let me explain :
The console session is open with the administrator account (there is another account "Erik" with administrator rights on this PC).
If I open a 2nd session with the Administrator account using RDPCheck.exe everything is OK.
Also if I open 2 sessions with Erik account using RDPCheck.exe everything is OK:

On the other hand if I use the remote desktop connexion (Windows) from another PC=PC2 with the Administrator account it closes the console session on PC1 (which I get on the PC2) !
If I re-open a remote desktop connection with the Administrator account (from PC2): the first session closes !
But if I reopen a console session (on PC1) with the Administrator account it opens me a new Administrator session ... very weird.
Now even more strange if I re-open a remote office conexion (from PC2) with the Administrator account: it works !!!
If I use remote desktop conexion (Windows) from PC2 with Erik account to open 2 sessions it does not work (the first session is closed).
On the other hand if I open a local session (console) with the account Erik on the PC1 and I reouvre a remote desktop connection (since the PC2) with the account Erik: it works !!!

If I check (with RDPConf.exe) "GUI Authentication Only" instead of "Network Level Authentication".
I have the same problem as before with the Administrator account: it closes the console session (which I get on the PC2).
By cons I can open multiple remote desktop session with Erik account (without local session (console) with Erik account open on PC1)) ...

But the strangest and when I use the remote desktop connection locally (<IP_ADDRESS> to <IP_ADDRESS>).
I open a console session with the administrator account, since this session I start the remote desktop connection and try to log in with the administrator account:
it opens another local session (console) Administrator !!! I never saw that !!!

Closed as not actual


Add an index for Metis' 5.1 locality overview

Previously, we had an index like this"xml_tree_values_result_simple_path_idx" btree (results_id, simple_path), but the locality overview query has a third join condition in value. This matches the join conditions and turns an index scan into an index only scan; we saw query times drop from minutes to under 50ms.
(This won't replace xml_tree_values_result_simple_path_idx just yet, but I hope it does.)
                                                                                QUERY PLAN
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Sort  (cost=6487.60..6490.07 rows=990 width=72) (actual time=36.295..36.782 rows=496 loops=1)
   Sort Key: localities.name
   Sort Method: quicksort  Memory: 64kB
   CTE precincts
     ->  HashAggregate  (cost=6403.53..6403.66 rows=13 width=12) (actual time=11.569..12.106 rows=496 loops=1)
           Group Key: localities_1.value
           ->  Nested Loop Left Join  (cost=1.10..6383.28 rows=4051 width=12) (actual time=0.126..10.781 rows=496 loops=1)
                 ->  Nested Loop Left Join  (cost=0.68..2630.55 rows=599 width=10) (actual time=0.085..2.020 rows=496 loops=1)
                       ->  Index Scan using results_public_id_key on results r  (cost=0.27..8.29 rows=1 width=4) (actual time=0.016..0.017 rows=1 loops=1)
                             Index Cond: ((public_id)::text = '2016-11-08-federal-Maine-870'::text)
                       ->  Index Only Scan using xtv_results_value_simple_idx on xml_tree_values localities_1  (cost=0.41..2607.29 rows=1498 width=14) (actual time=0.057..0.998 rows=496 loops=1)
                             Index Cond: ((results_id = r.id) AND (simple_path = 'VipObject.Locality.id'::ltree))
                             Heap Fetches: 496
                 ->  Index Only Scan using xtv_results_value_simple_idx on xml_tree_values precincts_1  (cost=0.41..6.25 rows=1 width=14) (actual time=0.012..0.013 rows=1 loops=496)
                       Index Cond: ((results_id = r.id) AND (value = localities_1.value) AND (simple_path = 'VipObject.Precinct.LocalityId'::ltree))
                       Heap Fetches: 491
   CTE localities
     ->  Function Scan on crosstab ct  (cost=0.00..10.00 rows=995 width=96) (actual time=18.485..19.013 rows=496 loops=1)
           Filter: (id IS NOT NULL)
   ->  Hash Left Join  (cost=0.42..24.68 rows=990 width=72) (actual time=32.216..34.917 rows=496 loops=1)
         Hash Cond: (localities.id = precincts.locality_id)
         ->  CTE Scan on localities  (cost=0.00..19.90 rows=990 width=64) (actual time=18.490..20.118 rows=496 loops=1)
               Filter: (id IS NOT NULL)
         ->  Hash  (cost=0.26..0.26 rows=13 width=40) (actual time=13.715..13.715 rows=496 loops=1)
               Buckets: 1024  Batches: 1  Memory Usage: 26kB
               ->  CTE Scan on precincts  (cost=0.00..0.26 rows=13 width=40) (actual time=11.572..13.180 rows=496 loops=1)
 Planning time: 5.486 ms
 Execution time: 37.705 ms
(28 rows)

There's a better way.


money drop particle can’t clear

I restart server, but it still stay there, can fix it?

Yes, if you stand right next to it and type /mfmclear it should go

I enter this command,but still have questions

You need to stand right next to them

only have particle ,don’t have coin , I also try stand right to them but same

/minecraft:kill @e[type=armor_stand,distance=..20]
try using that


lucet-runtime: heap size limit is hardcoded to 1 MB

The max heap size defaults to 1 MB:
    fn default() -> Limits {
        Limits {
            heap_memory_size: 16 * 64 * 1024,
            heap_address_space_size: 0x200000000,
            stack_size: 128 * 1024,
            globals_size: 4096,
        }
    }

We don't seem to ever set it to what is stored in the wasm module, effectively limiting all applications to 1 MB.

Closed by #77