DYNEXD - Synchronization Problem

Since today morning (that's at least where first noticed it) I see the following log entries which are repeatedly occuring since then:
09:26:23.361501 INFO SYNCHRONIZATION started
09:26:23.409769 INFO Failed to check ring signature for tx
09:26:23.857745 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10439
09:26:23.857745 INFO id:
09:26:23.857745 INFO PoW:	<60d1043fe2e6e0ed84e4dc3619d36e3f5d728d9440e76b11f935106163000000>
09:26:23.857745 INFO difficulty:	2903460
09:26:23.877891 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10440
09:26:23.877891 INFO id:
09:26:23.877891 INFO PoW:
09:26:23.877891 INFO difficulty:	2914155
09:26:23.900274 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10441
09:26:23.900274 INFO id:
09:26:23.900274 INFO PoW:
09:26:23.900274 INFO difficulty:	2917453
09:26:23.919460 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10442
09:26:23.919460 INFO id:
09:26:23.919460 INFO PoW:	<9eae8e7bd78b3583a956d882af3387816dfe49221416133861ae5f1c66030000>
09:26:23.919460 INFO difficulty:	2917717
09:26:23.943307 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10443
09:26:23.943307 INFO id:	<2995c856d87987a1d9e62bcaba8f615227b131943397cb5332d8775e20f744b7>
09:26:23.943307 INFO PoW:
09:26:23.943307 INFO difficulty:	2915520
09:26:23.961067 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10444
09:26:23.961067 INFO id:	<32c8d6b3de315f3bcea89a3fd99aa88edb2fcbbc29a0577971b1191be763d846>
09:26:23.961067 INFO PoW:	<77b34783d87206156f09fa037149219263db435ca2d55282e50eb1acc4000000>
09:26:23.961067 INFO difficulty:	2890361
09:26:23.983155 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10445
09:26:23.983155 INFO id:
09:26:23.983155 INFO PoW:	<7ab3f3a7164b6705bfa79837600bb00b67446279197850cea6faf3bc2f020000>
09:26:23.983155 INFO difficulty:	2887034
09:26:24.008395 INFO ----- BLOCK ADDED AS ALTERNATIVE ON HEIGHT 10446
09:26:24.008395 INFO id:	<0c803ae5f653b2e0d0ec0efe0a0e31e7974d5f9a3aede6167342837e4a062694>
09:26:24.008395 INFO PoW:
09:26:24.008395 INFO difficulty:	2895911
09:26:24.030208 INFO ###### REORGANIZE on height: 10439 of 10446 with cum_difficulty<PHONE_NUMBER>
09:26:24.030208 INFO  alternative blockchain size: 9 with cum_difficulty<PHONE_NUMBER>
09:26:24.049347 INFO Failed to check ring signature for tx
09:26:24.049508 INFO Block  has at least one transaction with wrong inputs:
09:26:24.053814 INFO Failed to check ring signature for tx
09:26:24.053918 INFO Failed to switch to alternative blockchain
09:26:24.224540 INFO Rollback success.
09:26:24.224607 INFO The block was inserted as invalid while connecting new alternative chain,  block_id:
09:26:33.525763 INFO [<IP_ADDRESS>:17333 OUT] Sync data returned unknown top block: 10446 -> 10642 [196 blocks (0 days) behind]
09:26:33.525763 INFO SYNCHRONIZATION started
I also tried to update the executables but had to switch back to use the old ones as the downloaded files aren't recognized as executable by Mac OS. I downloaded the files from this location: https://github.com/dynexcoin/Dynex/releases/tag/Dynex_macos
Any ideas how to solve the failed synchronization issue?
Cheers & Thank in advance

IMPORTANT MESSAGE TO ALL PARTICIPANTS +++ UPDATE TO LATEST VERSION
Unfortunately have we not been able to get in touch with the three nodes which have been constantly trying to submit incorrect blocks into our beautiful Dynex blockchain. I'm sure most of you have noticed that the chain tried to repair and rebuild itself continuously (which shows that the chain itself can correct itself), but at the cost of making mining new blocks quit problematic. We therefore just released the source codes as well as Windows, MacOS and Linux binaries with an update to get the situation resolved. We will NOT allow old nodes to continue to connect and to participate. The cut was introduced at block 10708. Therefore, please:

Download (or build) the latest version from our GitHub
Delete the .DYNEX data directory (which might still have dust inside). The new daemon will NOT start if you have not deleted the directory. If you don't know where the .DYNEX directory is located, run "dynexd --help" - this will show you the location of the data-dir
Now run "dynexd" - the daemon will synchronise with the clean blockchain. this will take a few minutes.

Please note that all wallet balances, historical transactions etc. are not effected (except the incorrect transactions which someone tried to inject). Also note that we chose block 10708 as an interception point from where we fork the access.
Again, we are deeply sorry for any inconvenience caused, we didn't sleep much the last 12 hours but are very relieved that we could resolve the situation.

closing as fixed.

Thanks, issue has been resolved, it is working again as expected - keep up the great work :)


getLabelForValue in ChartJS 3 doesnot work as expected

Expected Behavior
I intend to add ellipses to the labels on x axis as they are quite long ,So I tried to use the callbaclk on scales->x->ticks as suggested by https://www.chartjs.org/docs/3.3.2/samples/scale-options/ticks.html
However getLabelForValue apparently doesnot exist on ticks it exists on the scales .
I am unable to use this to modify my labels on x axis ( I dont want to change the tooltip that shall display the complete label)

Chart.js version:-<EMAIL_ADDRESS>

Thanks , yes I checked your working example
Strangely it doesnot work for me yet.., trying to find the issue
I get this error Property 'getLabelForValue' does not exist on type '{ callback: (val: any, index: any, ticks: any) => any; }'

Can you please provide a working sample of your issue then as pointed out is needed for a bug report because like this its not solvble since its working fine for us as you can see in the official docs and in ts project

Hey it worked
The issue was related to type script noImplicitThis . So I had to specify it as
var newthis =this as any; inside the callback function and the getLabelForValue then worked


unable to import netlify environment variable in assets/js/custom.js file

I am using this hugo theme flowbite-admin-dashboard. It uses webpack as well. This is how it builds the website:
"build": "NODE_ENV=production && run-s build:webpack build:hugo && rm /opt/build/repo/**/*.map",
"build:hugo": "hugo --destination=./public",
"build:webpack": "webpack --mode=production",
"build:styles": "npx tailwindcss -i ./src/style.css -o ./dist/css/flowbite.css",

My site is hosted on netlify. There I use npm run build in production.
I want to access environment variables (stored in netlify) in asset/js/custom.js file.
I tried adding this to layouts/_default/baseof.html file
  {{ $env := dict "encrypt" (getenv "HUGO_PARAMS_ENCRYPTION_KEY_HEX") }}
  {{ $js := resources.GetMatch "js/custom.js" | js.Build (dict "params" $env) }}
  <script async src="{{ $js.RelPermalink }}"></script>

in asset/js/custom.js file when i do
import params from "@params"
I get this error
custom.js:2 Uncaught Error: Cannot find module '@params'
    at webpackMissingModule (custom.js:2:50)
    at eval (custom.js:2:132)
    at ./assets/js/custom.js (app.bundle.js:19:1)
    at __webpack_require__ (bootstrap:19:1)
    at eval (index.js:11:78)
    at ./src/index.js (app.bundle.js:169:1)
    at __webpack_require__ (bootstrap:19:1)

I am new to hugo, how should i go about using env variables in .js file.

Please use the forum (https://discourse.gohugo.io/) for questions and troubleshooting. We prefer to use GitHub for verified bugs and vetted enhancements. Thanks.


Ignoring unknown interface he-ipv6=he-ipv6.

i'm having this problem.
Ignoring unknown interface he-ipv6=he-ipv6.
curl: (7) Failed to connect to 2620:108:700f::36f5:fc55: Network is unreachable
I use a VM of 123systems openVZ, can you help me?

It won't work on OpenVZ, your need a proper virtual machine (e.g. KVM or Zen).


Hiera Backend and code re-use

Its to early to merge this
But I already introduces a fair bit of code re-use, tests, bugfixes and lint fixes.

@solarkennedy Any idea as to why this fails on puppet 4.1?
more specifically it fails if i run the tests through rake spec with puppet 4.1
But if I execute the tests seperatly  through rspec    then all tests succeed.

@houtmanj is this ready for review?

@solarkennedy yes

Hey @houtmanj, could you please rebase?


net: add support for FileConn, FilePacketConn, FileListener on windows

Issue for discussion.
FileListener is useful to implement graceful-restart server. ex: https://github.com/lestrrat/go-server-starter
I know windows doesn't have a way to pass FDs to external process like UNIX OSs.
But it's not impossible. WSADuplicateSocket can export intormation of file descriptor as WSAPROTOCOL_INFO to specified process. This structure is possible to be written to a file as byte array.
http://msdn.microsoft.com/ja-jp/library/windows/desktop/ms741565(v=vs.85).aspx
How about implementation of os.File.dup, os.FileListener on windows?
https://codereview.appspot.com/177590043/

@mikioh I've been trying to wrap a net.Listener around the syscall.Handle I get from windows sockets APIs (e.g. WSASocket).  SyscallConn doesn't really help me with that...does it? I'd love to be able to do something like this:
	pi := syscall.WSAProtocolInfo{}
        //fill pi by some means
	fd, err := xwin.WSASocket(pi.AddressFamily, pi.SocketType, pi.Protocol, &pi, 0, 0)
	if err != nil {
		return nil, err
	}
        listener, err := net.FileListener(os.NewFile(uintptr(fd), "listener"))

But of course this doesn't work because of this issue.


@wix/wnpm-ci not found - breaks npm install

Please update package.json. It seems that @wix/wnpm-ci has been moved into wnpm-ci in npm registry.
npm ERR! Linux 4.9.36-moby
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! node v6.11.2
npm ERR! npm  v3.10.10
npm ERR! code E404

npm ERR! 404 Not found : @wix/wnpm-ci
npm ERR! 404
npm ERR! 404  '@wix/wnpm-ci' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404 It was specified as a dependency of 'redux-saga-tester'
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

Even installing just redux-saga-tester this error exists:
> npm install redux-saga-tester

npm ERR! code E404
npm ERR! 404 Not Found<EMAIL_ADDRESS>

Hi @3LOK, could you please look at this issue? Thanks!

From what I have understood so far, the code on GitHub is not sync'ed with npm. Latest version released now has a hard/explicit dependency to @wix/wnpm-ci, which does not exist in any registry.
version 1.0.370
  version: '1.0.370',
  main: 'dist/SagaTester.js',
  scripts:
   { lint: 'eslint \'./src/**/*.js\' \'./test/**/*.js\' \'./examples/**/*.js\'',
     examples: 'mocha --compilers js:babel-register \'./examples/**/*.js\'',
     prepublish: 'npm test && npm run compile',
     test: 'mocha --compilers js:babel-register && npm run lint',
     'test-dev': 'mocha --watch --compilers js:babel-register',
     compile: 'babel -d dist/ src/',
     build: ':',
     release: 'npm install @wix/wnpm-ci && wnpm-release -- --no-shrinkwrap' },
  devDependencies:
   { babel: '^6.5.2',
     'babel-cli': '^6.24.1',
     'babel-eslint': '^7.2.3',
     'babel-plugin-transform-runtime': '^6.15.0',
     'babel-preset-env': '^1.5.1',
     'babel-register': '^6.24.1',
     'babel-runtime': '^6.11.6',
     chai: '^3.5.0',
     'chai-as-promised': '^6.0.0',
     eslint: '^3.19.0',
     mocha: '^3.4.1',
     nock: '^9.0.13',
     'node-fetch': '^1.6.3',
     redux: '^3.6.0',
     'redux-saga': '^0.15.3' },
  publishConfig: { registry: 'https://registry.npmjs.org/' },
  dependencies: {},
  dist:
   { shasum: 'd3f5dbdac89f4c33bd28d7ab7ead6d573f1a7e38',
     tarball: 'https://registry.npmjs.org/redux-saga-tester/-/redux-saga-tester-1.0.370.tgz' },
  directories: {} }

Version 1.0.371
  version: '1.0.371',
  main: 'dist/SagaTester.js',
  scripts:
   { lint: 'eslint \'./src/**/*.js\' \'./test/**/*.js\' \'./examples/**/*.js\'',
     examples: 'mocha --compilers js:babel-register \'./examples/**/*.js\'',
     prepublish: 'npm test && npm run compile',
     test: 'mocha --compilers js:babel-register && npm run lint',
     'test-dev': 'mocha --watch --compilers js:babel-register',
     compile: 'babel -d dist/ src/',
     build: ':',
     release: 'npm install @wix/wnpm-ci && wnpm-release -- --no-shrinkwrap' },
  devDependencies:
   { babel: '^6.5.2',
     'babel-cli': '^6.24.1',
     'babel-eslint': '^7.2.3',
     'babel-plugin-transform-runtime': '^6.15.0',
     'babel-preset-env': '^1.5.1',
     'babel-register': '^6.24.1',
     'babel-runtime': '^6.11.6',
     chai: '^3.5.0',
     'chai-as-promised': '^6.0.0',
     eslint: '^3.19.0',
     mocha: '^3.4.1',
     nock: '^9.0.13',
     'node-fetch': '^1.6.3',
     redux: '^3.6.0',
     'redux-saga': '^0.15.3' },
  publishConfig: { registry: 'https://registry.npmjs.org/' },
  dependencies: { '@wix/wnpm-ci': '^6.1.0' },
  dist:
   { integrity: 'sha512-ZUrs+R0dyrjnEGsn6dMcx1she2FTRe4vu0FAAVUsbqvGUK4MYoae58em5QHG6mhwYu3Uxf3L1UqeFlTgznNaQw==',
     shasum: '9f6fb18a0761f972e6dff50fb6b3a3b71fddf7ce',
     tarball: 'https://registry.npmjs.org/redux-saga-tester/-/redux-saga-tester-1.0.371.tgz' },
  directories: {} }

I am using version 1.0.366 and it is also happening. It is specified on the release command.

I dont think the release command does impact a npm install to be honest. I thought at first, but only prepare is called during a npm install or yarn install.
It should work if you use 1.0.366 and a recent version of npm/yarn.

I started having trouble this morning deploying my applicacion using VSTS. If you upgrade NPM version to 5.3.0 it bypasses the problem with npm install

Hi,
As mentioned, it should affect anything since we only use it during release (which is done by our internal CI system that needs this package), so I'm unsure why it would bother you during npm install.
@shahata - Could this have anything to do with your latest CI changes? Any other ideas?

Like @willdurand says, it now appears as a direct dependency in version 1.0.371, what might be the problem.
npm show<EMAIL_ADDRESS>dependencies
{ '@wix/wnpm-ci': '^6.1.0' }

@3LOK well latest published version explicitly requires @wix/wnpm-ci though, so there is an issue :)

I don't see it as a dependency or a dev-dependency in the code :$... trying to re-release the version + the related pull request

Released. Check now?

  version: '1.0.372',
  main: 'dist/SagaTester.js',
  scripts:
   { lint: 'eslint \'./src/**/*.js\' \'./test/**/*.js\' \'./examples/**/*.js\'',
     examples: 'mocha --compilers js:babel-register \'./examples/**/*.js\'',
     prepublish: 'npm test && npm run compile',
     test: 'mocha --compilers js:babel-register && npm run lint',
     'test-dev': 'mocha --watch --compilers js:babel-register',
     compile: 'babel -d dist/ src/',
     build: ':',
     release: 'npm install wnpm-ci && wnpm-release -- --no-shrinkwrap' },
  devDependencies:
   { babel: '^6.5.2',
     'babel-cli': '^6.24.1',
     'babel-eslint': '^7.2.3',
     'babel-plugin-transform-runtime': '^6.15.0',
     'babel-preset-env': '^1.5.1',
     'babel-register': '^6.24.1',
     'babel-runtime': '^6.11.6',
     chai: '^3.5.0',
     'chai-as-promised': '^6.0.0',
     eslint: '^3.19.0',
     mocha: '^3.4.1',
     nock: '^9.0.13',
     'node-fetch': '^1.6.3',
     redux: '^3.6.0',
     'redux-saga': '^0.15.3' },
  publishConfig: { registry: 'https://registry.npmjs.org/' },
  dependencies: { 'wnpm-ci': '^6.2.52' },
  dist:
   { integrity: 'sha512-nK6HCV+SVoz0SCoPhktIAaihcJXddBekPn5UwJiUYVWZJU2TKa5V2mKQKw8YEUMFlqi7sk8VB3w9Lbz+zY4W8w==',
     shasum: '303ca137664d0675400f14b2a1ba936ab0721e57',
     tarball: 'https://registry.npmjs.org/redux-saga-tester/-/redux-saga-tester-1.0.372.tgz' },
  directories: {} }

So I'd say it is better but why is this wnpm-ci lib required on NPM but not in the package.json?

I have no idea... at least now you shouldn't have any direct blockers, and the issue should not prevent you from continuing (and it's now just a semantic issue, of an unneeded dependency).
@shahata, do you know why is it added even when not in package.json? something automatically added by CI? Something new by NPM?

yeah, I know the reason for this, will send pr in a few mins


1318 update feature info agent to for reduced time series tables

Update FIA to use TimeSeriesClientFactory to generate the correct TimeSeriesClient.  Added a second method in TimeSeriesClientFactory for federated query option

Before merging, the jps base lib needs to be deployed first. After that, the pom file for FIA needs to be updated with the right base lib version


Flexible Param input

A lot of parser states are parameterized. For example, both #12 and #13 require a file name so we can open another ReadBuffer to store in the state, TSV/CSV parsers can take delimiters and number of header lines, and kmer-parsers need a k.
Right now, we pass initial params into the state as a "P" object scoped to each state which we also require to have a Default so that we can use None across parsers. An alternate design could take an Into<P> object where P: Default + From<Params> for convenience (where Params is some kind of { filename: String, other_params: BTreeMap<&str, Value> }?)

I ended up extending get_reader to support additional params that must be used (except for a filename param that gets inserted by default) in 5f72d0468c8c8d343370638020a9334b31e5d704.
Some alternate designs that didn't work:

Extending StateMetadata to add a method that would inject the params. This would require instantiating a State first, which generally requires some amount of parsing and since the params wouldn't be injected yet, the parser couldn't use them.
Making a generic Params object that all State parsers would need to accept. This wouldn't allow parsers to use State fields to move data from the parse phase to the get phase unless the Params object also had a generic param field which made the design pretty complicated.


Issues with Google Play Music player not loading

Hello-
I wrote a short while ago about GPMP hanging on the load screen on Linux Mint 19.1.  I was directed toward the Circle CI via a direct link to a new build. That did the trick.
Recently, I just got a new laptop, which I  promptly installed Mint 19.2 on. Since doing so, even using the link pointed toward in #3660 , I am hanging on the loading music library screen.
I have poked around in the Circle CI, looking for an updated artifact, but, and this might just be me being dense, I cannot find the artifacts link alluded to in the readme.
Any insight? Thanks so much
Shane

Checklist:

[x] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES ASWELL

[x] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[x] I have double checked and can reproduce the issue

OS:
GPMDP Version:
Issue Descriptions:
Steps to Reproduce:

Unfortunately the artifacts on their website are behind a login screen. There are various links to the artifact builds in the issues however.

I'm closing this as it's a general question related to #3660.

Thanks for the reply.
The problem is the only link for Debian based systems in #3660 does not work in this instance. (It worked previously however on my older computer).
I did use my Github account to log into CI, still no luck finding the solution.
Can you suggest a way forward?

https://app.circleci.com/jobs/github/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/3718/artifacts
These are the artifacts from the latest version

Hello again,
Thanks for the links. Unfortunately, I'm still stuck on the loading screen. I tried the 64bit artifact from both the first and second links (although by file names, it seems they may be the same).
Is there anything I can provide you from my system setup that might be of use?
Shane

CTRL+SHIFT+G -> type in (no quotes) DEV_MODE -> restart -> CTRL+SHIFT+i -> Screenshot of any errors in this window. Type in openGPMDevTools() enter. Screenshot any errors in this window.

Hi Jesse-
After launching DEV_MODE & restarting, I was able to start the player and it worked as it should.
I didn't spot any errors per se in the ctrl/shift/i configuration (but grabbed a shot). Under the openGPMDevTools() I caught several errors (across two images):

Shane

That looks like you might have some kind of UI update that the rest of us don't have, considering its complaining about UI being missing, and you are loading listen_v2.js which I don't have yet. Is there anything different in the browser for you?
I also see an error related to mDNS, that could be causing some problems as well.

Hi Jesse-
Sorry for my slow reply. I sent my last message literally running out the door.
The browser in Dev mode looks pretty normal (there is an 'I'm feeling lucky' button which I don't recall from last time. Other than that, everything seems normal as far as I can tell.
The UI I'm working off of is an XFCE desktop on Mint 19.2, I don't think I have any Gnome or KDE packages installed (other than what might come from installing Stata's graphical library or gedit). I'm not quite sure what mDNS is.
Is it 'safe' to run it in dev mode on the regular? The player is much nicer than using a browser tab. Of course, I'm happy to do anything I can to help you all moving forward.

Sorry, by UI I was referring to the GPM UI, not the desktop UI. Either way, if it works in Dev mode then it should work outside of dev mode. mDNS is the DNS service used for zero configuration discovery for Chromecast, so it appears to be broken for you, or you have a device without a name on your network.

Hi Jesse-
I started Googling mDNS and Chromecast. Mint 19.2 has the mDNS package by default. But, on a whim, I installed chromium-browser (thinking it would carry with it the things needed to make Chromecast work). Indeed, it does: After a reboot, GPDMP works as normal. Perhaps there is some dependency in Chromium that the player also relies upon?
Thanks again-- I really appreciate it and I hope you have a great night.

I also have the same issue on gentoo after updates and updating to latest GPMDP from the above link doesn't solve the issue, but the errors appear to be a little different and it DOESN'T work in dev mode and I don't have a workaround, other than using it in a browser without the app.
Screenshot of the errors window:

Your issue is the same issue that was fixed in #3660. I'm not sure how you get the application in gentoo, but if you can get the latest source and run it, it is fixed.

Gentoo generally is source based so it should in theory be easy for me to run from latest sources, but so far noone tried to make a source based GPMDP ebuild, so there is only a -bin ebuild in some unofficial overlay (I also have my own live ebuild with version from https://3718-40008106-gh.circle-artifacts.com/0/home/circleci/project/dist/installers/debian/google-play-music-desktop-player_4.6.1_amd64.deb, making an ebuild from deb package is usually not that hard).
I tried to get GPMDP built from source, but that seems to require electron, + a bunch of other dependencies that there are no ebuilds for so I would have to make them and at that time I just gave up and got the bin package. At this moment I don't have enough time to get it done (but I may try, maybe this weekend). Is there some prebuilt binary of the latest linux version?


Datastore query iteration steps cannot use independent context timeouts

When iterating over datastore query results, a context with deadline may be used to limit the time allowed for the entire query. However, individual calls to Iterator.Next cannot currently be configured with independent, shorter deadlines, since they all share a single context deadline.
Practically, this means that a large query must either A) use a long deadline, leading to slow reaction time of calling code in response to contention, etc., or B) use a short deadline, leading to many spurious restarts of the query.
Instead, allowing independent contexts, or perhaps better, independent sub-deadlines/timeouts for each call to Iterator.Next / .nextBatch, would improve visibility of datastore contention to calling code.
Snippets from current code in cloud.google.com/go/datastore:
func (c *Client) Run(ctx context.Context, q *Query) *Iterator {
	...
	t := &Iterator{
		ctx: ctx, ...
	}
	...
}

func (t *Iterator) nextBatch() error {
	...
	resp, err := t.client.client.RunQuery(t.ctx, t.req)
	...
}

Expected behavior: the ability to either pass a context.Context to datastore.Iterator.Next, or perhaps to pass a per-step timeout during construction of the query or iterator, such that internally the context passed to the inner client would be wrapped in (another) layer of context.WithTimeout using that smaller timeout.
Note: this is a regression in practice vs. the legacy AppEngine Datastore library, which appears to use a short (5-second?) timeout on each underlying AppEngine API call during iterator Next.

Thank you for filing this! The Go Datastore client is in a different repo, so I copied this issue over there: https://github.com/googleapis/google-cloud-go/issues/1663.


Allow extraArgs to contain false values

What this PR does / why we need it:
A previous PR (#3297) added behavior to convert a value like extraArgs.log-to-stderr=true to an argument in the format --log-to-stderr. However, this prevents users from disabling flags which default to true like skip-nodes-with-local-storage.
Which issue this PR fixes

fixes #9280

Special notes for your reviewer:
This is my first time contributing to this repo so minor nitpicks/suggestions are welcome.
Checklist

[X] DCO signed
[X] Chart Version bumped

/assign @mgoodness

Too bad it got auto closed. Anybody who can help on this issue?


Remove flash between poster and video

This resolves https://github.com/react-native-community/react-native-video/issues/1128

Could you please explain why this prevents the flashing?

If you put a breakpoint in the dealloc method of RTCVideo.m, it will be fired once the video loads and the poster is removed. So, the current implementation allocates a native video player, renders it behind a poster, then once the video loads, it de-allocates the original instance and creates a new instance. That re-instantiation causes the flash. After applying my changes, dealloc is not called when the video loads and the flash is gone.
I don't know enough about the internals of React to say for sure why it is not re-using the native video player upon video load, but I would guess the fact that the RCTVideo component changes parents.
In other words, the switch from...
 <View style={nativeProps.style}>
        <RCTVideo
            ref={this._assignRoot}
            {...nativeProps}
        />
       ...
</View>

to simply...
<RCTVideo
   ref={this._assignRoot}
   {...nativeProps}
/>

leads to a creation of a new RCTVideo instance

@n1ru4l Ok, feel free to approve any of these open PRs and I'll merge them. I've been short on time to do the extensive testing to make sure that many of these work.
Before approve anything, please review and make sure the code is readable, test it, and make sure any new functionality is documented.


one2one ringbuffer c<->java issues

hi guys,
it looks like one2one ringbuffer is implemented differently in c vs java. (sorry didn't dig deep into that issue yet). but in my case with writer in C and reader in java, reader gets ahead of writer easily... both reader and writer in c seems to be ok. also replacing ringbuffer implementation with many2one fixes issue instantly

Do you have a test case that you could share?

it works! :) @tmontgomery can you verify (just in case) that many2one is in sync with java version?

Many2One is used a lot with Aeron for C++ and Java. It gets a good workout with the tests involved. That and it looks to be in sync anyway.


UI Pointer - Don't automatically initialize world canvases

In my games, I support VR and 2D. I have some helper scripts for things like menus and such that detect if VR is present and then enable them for VR.
The UI pointer script needs an option to not try to make every world canvas interactable. Maybe a public boolean saying to not run on startup, or something. I'm basically doing that now and upon detecting VR support, another script calls the setup per canvas that needs it.
The second thing deals with the pointer for interacting with the UI Pointer. It would be nice to set the layer of the beam being projected, publicly. This way, when I show a UI world canvas, I can force that to show over top anything else, and the beam will be visible.
A question, is UI Pointer required to be on a controller? I'm not on my Unity box right now, but would it be fine on the camera rig... or?

You can specify the UI Pointer to ignore certain canvases using the "Ignore Canvas With Tag Or Class" parameter:
https://steamvr-unity-toolkit.readme.io/docs/vrtk_uipointer#section-inspector-parameters
I'm not sure about your second point, are you saying you want the actual pointer beam to auto display when you point at a canvas?
You can point the UI pointer on any object including the headset, check the example scene to see this in action: 034_Controls_InteractingWithUnityUI

I'm not a fan of the ignore with tag or class since it is too limiting and makes things messy. I'd like something on "Only run on objects with class or layer" (an include instead of exclude). Regardless, if I don't want it running on anything at runtime, there's no way to do that... Unless it takes a wildcard character? Does it? I'll have to dig in the code again.
The second point deals with layers and camera culling. For example, in a game, I have a pause menu. That menu makes a world space canvas. The problem is that the canvas may be behind other objects in the scene. There's a few ways to take care of this. Have a second camera that turns on and only shows the UI layer... Or, show the second camera on top of the primary in depth only mode with the UI layer on top.  In either case, the pointer beam will be behind the world space canvas to the user since it's layer isn't settable. It would be very helpful to be able to set the pointer beam layer.

I'm my project I've extended the VRTK_SimplePointer, as I only wanted it optionally to be visible on a certain objects.. so I modified as follows to show only for certain tags. Note 'TagLimitPointerVisibility' is a public field that can be set from the editor. I'm also using TagFrenzy ( https://www.assetstore.unity3d.com/en/#!/content/13170 ).
Code override in Update:
if (!string.IsNullOrEmpty(TagLimitPointerVisibility))
{
    if (pointerContactTarget != null)
    {
        List<string> tags = pointerContactTarget.gameObject.tags();
        if (tags.Contains(TagLimitPointerVisibility))
        {
            // Turn something on logic
            return;
        }
    }
    // Turn something off logic
}


Docker: Multi-Arch Support

The official docker image should support both amd64 and arm64 virtualization. It should be trivial for Bazel to do this.

https://github.com/bazelbuild/rules_docker/pull/2087#issuecomment-1431819817
Well, that doesn't look good.


notifier(engine): fix Flush deadloop in closed notifier

What problem does this PR solve?

Issue Number: close #7506
What is changed and how it works?

Flush should return immediately if notifier has been closed.

Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/run-engine-integration-test

/run-engine-integration-test

Codecov Report

Merging #7509 (d0115e4) into master (6b48846) will decrease coverage by 11.6459%.
The diff coverage is 75.7142%.

Additional details and impacted files

Flag
Coverage Δ

cdc
?

dm
52.0234% <50.0000%> (?)

engine
61.2637% <76.4705%> (+0.0799%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@                Coverage Diff                @@
##             master      #7509         +/-   ##
=================================================
- Coverage   65.9877%   54.3417%   -11.6460%
=================================================
  Files           564        424        -140
  Lines         52337      53239        +902
=================================================
- Hits          34536      28931       -5605
- Misses        15129      21542       +6413
- Partials       2672       2766         +94

run-engine-integration-test

/run-engine-integration-test

/run-engine-integration-test

/run-engine-integration-test

/run-engine-integration-test

/run-engine-integration-test

/run-engine-integration-test

/run-engine-integration-test

/merge


Update _index.md

updated Studio Pro Extensibility info

@mauritselzinga-mx , do you have a release date or version so that I can add a cross-reference to RN?

Hey, we released in public beta in 10.12
Maurits Elzinga
Sr. Product Manager | Mendix | Studio Pro
+31 6 38771629
From: Maria Shaposhnikova @.>
Date: Wednesday, 6 November 2024 at 10:07
To: mendix/docs @.>
Cc: Maurits Elzinga @.>, Mention @.>
Subject: Re: [mendix/docs] Update _index.md (PR #8615)
@mauritselzinga-mxhttps://github.com/mauritselzinga-mx , do you have a release date or version so that I can add a cross-reference to RN?
—
Reply to this email directly, view it on GitHubhttps://github.com/mendix/docs/pull/8615#issuecomment-2459067438, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5C5FCFTXRT4DLTNGRISIMLZ7HL3NAVCNFSM6AAAAABRIHW7ACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJZGA3DONBTHA.
You are receiving this because you were mentioned.Message ID: @.***>


create an image for kpt with gcloud and git included

Is your feature request related to a problem? Please describe.
some functionality of kpt requires gcloud and git client to be installed on the Docker image.  The customer request is to provide an image with those already installed instead of having to install those manually.
Describe the solution you'd like
There should be a docker image for kpt that people can use in their CI/CD pipeline that includes gcloud and git.  It should be an officially supported image.
Describe alternatives you've considered
#1457 PR originally attempted upgrading the base image for kpt to go from alpine to gcr.io/google.com/cloudsdktool/cloud-sdk:alpine but the image size has gone from ~50MB to almost 200MB.
Additional context
@martinmaly - original requestor

Right now I would recommend using some of the general purpose gcr.io/google.com/cloudsdktool/cloud-sdk images (either the latest or debian_component_based versions). Right now latest does not include kpt but I'll see if we can get it added. The debian_component_based version does already include kpt and you can use that image right now.
These general purpose images do contain a number of features so are larger images (multiple GB). Would these images satisfy the use case you are trying to address here @martinmaly?

With the debian_component_based image kpt version 0.37.1 is currently available:
➜  docker run gcr.io/google.com/cloudsdktool/cloud-sdk:debian_component_based kpt version
0.37.1

Looking at the image, it may be initially sufficient. it seems that majority of the size is coming from gcloud (possibly some also from the base images). It may be good place to start and we can optimize if and as needed.
It appears that the image contains git so that should satisfy our requirements for now.
Thank you!

To provide some context here outside our other conversations:
Version 0.39.0 of kpt introduces a new kpt-gcloud image that you can find here: gcr.io/kpt-dev/kpt-gcloud
Sizewise this image is in between the kpt image and the cloud-sdk images w/ kpt.
There are currently 4 images available with kpt. They will fit different needs:

gcr.io/kpt-dev/kpt - This includes git and kpt and is the smallest image available
gcr.io/kpt-dev/kpt-gcloud - This image is based on the cloud-sdk:alpine image and includes gcloud in addition to what is included in the kpt image.
gcr.io/google.com/cloudsdktool/cloud-sdk:latest and gcr.io/google.com/cloudsdktool/cloud-sdk:debian_component_based are cloud-sdk images defined at GoogleCloudPlatform/cloud-sdk-docker that are much larger images and provide a lot of different tools including kpt.

I'm going to close this issue. We have a new image available and documentation for that image is available at https://kpt.dev/installation/docker/
I think we can handle additional changes or feedback in other issues.


[WIP]Brackets URL Hints list folders

Hi @humphd and @gideonthomas I finished partially what was asked on issues https://github.com/mozilla/brackets/issues/514 and https://github.com/mozilla/thimble.mozilla.org/issues/1927.
This was a combined effort of me and @simon66 I hope that those images give a better idea of what we have accomplished:

Whenever the user have images inside files my code will look inside those files as well and display them on the hint bar.

Nice! Glad you got the code working @Taguiar.
I would suggest to have the dialog show the partial path of image. Something like this:

This way the user knows where the image is coming (Incase you have multiple images with the same name).
What do you think @flukeout?

Sorry about the indentation @humphd. I think I got too happy once the code run as I wanted and got hasty on my pull request.

This doesn't work for me locally, I'm getting a bunch of errors, and nothing displays:

Now @humphd   I manage tho make some of the adjustments you propose but some of them did not work so I had to maintain some of the code and also improvise a little bit!
Now the code is perfectly working on my machine and both synchronous and asynchronous!
Hope you like it!

@Taguiar one thing I know I'm going to want when I review this, based on your screenshot, is for you to prefix the parent paths for each file, up to, but not including, the project root.
For example, you have IMG.png, but there is no such file: it's actually secondsource/IMG.png, and that is how it should be listed.

I'm finishing this in https://github.com/mozilla/brackets/pull/787


V2 - CLA Check will pass but status in Github is still shown as CLA NOT SIGNED when contributor approved by githuborg contributes to the project

Prerequisite

Create a CLA group with CCLA enabled
Add github repository for the CLA group
Generate template
Company should have signed CLA for the project
Github org should be added as the approval list
Contributor should be invited and the contributor should accept invitation and join the github org(I assigned member role as well as tried with making the contributor owner)

Steps to Follow

Login to Github as individual contributor
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Corporate Contributor
Search and Select the Company
Click Proceed
Contributor is redirected to Request Authorization instead getting acknowledged

Please ensure you are public.

For my user I set Organization visibility to public and then the I was acknowledged. However the status in Github is still shown as CLA NO SIGNED
https://github.com/dronecodeproject-px4/px4/pull/5

Note to self: Scenario Summary:

GH Org Approved before
Contributor comes in and acknowledges
Status is updated, but not comment

Creating PR for this fix: https://github.com/communitybridge/easycla/pull/1527

Deploying attempt to fix to DEV

This issue appears to be resolved.
Before authorization (previously on GH Org approval list, but not affiliated with an org):

After affiliation:

@nirupamav @WillsonHG this should be resolved now

V2
Scenario - When employee has to sign CLA

Scenario - When employee need not sign CLA


layers: WSI functions should not work unless enabled.

The WSI extension functions are special.  They are statically exported from the
loader.  This means that an application can directly link against them, and not
have to call a vkGet*ProcAddr function.  However, the application is still
reponsible for for enabling the appropriate WSI extension.
An application was observed that called the WSI functions without enabling the
appropriate extensions.  It worked.  When that application was run with the
validation layers, it crashed.  A change was made to the layer to make sure
that an error was always logged.  It was decided that the loader should change
so that the terminator functions not work (and print a brief message) if an
application calls the function without enabling its extension.  This is done in
the terminator so that the validation layer can be called, which should log a
good error message; and yet still done here in case the application is never
tried with validation turned on.

+1

The rest of the loader code uses loader_log() rather than fprintf() to log errors. This code should use loader_log()  also since loader_log() will log as appropriately controlled by EXT_debug_report  and also the VK_LOADER_DEBUG env var.
Otherwise , LGTM.

I converted the fprintf() calls to loader_log() calls.  Ready to go now?

Thanks Ian! cherry picked the changes into master branch


Duplication of some instances by capitalization

This is likely a bug, as it results in issues with the results. See attached image where mastodon.cloud and Mastodon.cloud were counted separately

Thank you for taking the time to report this.
At the moment, those are the duplicates we have in the database:
1 duplicates found for instance pouet.social
1 duplicates found for instance equestria.social
1 duplicates found for instance mastodon.cloud
1 duplicates found for instance mastodon.blue

Solving this properly would involve:

Deleting those duplicates from the database
Adding a unique constraint somewhere to ensure it does not happen again

@LeoTindall I've pushed a commit to fix this, and removed the duplicate from the database, can you check on your side and confirm it is ok? :)
Also, ping @Gargron

Looks good. Thanks!


arctan2() returning incorrect value

So I'm using numsharp
np.arctan2(np.array(-0.0012562886517319706), np.array(-0.7499033624114052))
I get the wrong value, approximately, -4.33e-5
using numpy.net and C#'s Math library
np.arctan2(np.array(-0.0012562886517319706), np.array(-0.7499033624114052))
Math.Atan2(-0.0012562886517319706, -0.7499033624114052)
I get -3.1399, which is the correct value.. Am I just being dumb and doing something wrong or is NumSharp not actually calculating the correct value?
furthmore, np.arctan2(1,1) is 90deg according to numsharp. Should be 45.
np.arctan2(1,-1) is also 90deg....
I'm using version v0.20.5

Hello @RoseberryPi,
You are right, looking at code I don't understand the type casting done to (byte*) instead of (double*).
There must be an explanation.
case NPTypeCode.Double:
{
   var out_addr = (double*)out_y.Address;
   var out_addr_x = (byte*)out_x.Address;
   Parallel.For(0, len, i => *(out_addr)  = Converts.ToDouble(Math.Atan2(*(out_addr) + i, *(out_addr_x) + i)));
   return out_y;
}

I tried casting to (double*) and it returns the expected value, 3.13991738776297


A "idiomatic" endpoint name for either GraphiQL and requests

In order to use GraphiQL a route/endpoint needs to be declared:
https://github.com/neelance/graphql-go/blob/beff08417b04a91f0a7618b55786d63eec833b9f/example/starwars/server/server.go#L19
It fetches information from another one, /query in this case
https://github.com/neelance/graphql-go/blob/beff08417b04a91f0a7618b55786d63eec833b9f/example/starwars/server/server.go#L23
Can the latter be modified to something arbitrary (e.g, /api) and use a single endpoint for both? I mean this because not only query requests are sent to this endpoint, but mutation ones as well. it might sound trivial yet it's something meaningful for some of us.

Convention I've often seen in GraphQL backends is that POST /graphql is the query/mutation endpoint, and GET /graphql can be used to fetch the GraphiQL frontend. I've also seen GET /graphiql (with the "i") for the frontend.
That pattern has always made sense to me.


Not working

This library is "undefined" when impoorted in JS.

Same here.
Cannot read property 'raiseTestNativeError' of undefined

same here

has anyone found a solution or analogs?

Same. Bummer...

This is because it is not linked correctly. After you installed this library using npm or yarn, You should do
react-native link rn-test-exception-handler

A new line will be added to your podfile. Change it to this:
pod 'RnTestExceptionHandler', :path => '../node_modules/rn-test-exception-handler/ios/RnTestExceptionHandler.podspec'

This fixes the problem on iOS.


Add unified GraphConvModel

[x] draft model
[x] Change all update and readout argument
[x] Test all Update and Readout function

Could you fix tests?


Program not running

I went on git bash, and entered in all the mentioned code. It then asked me for a configuration name, and I wasn't sure what to write so I just put "start", then nothing happened. I tried running the code again, but nothing worked. Do you know anything I might have forgotten?

@UncagedTiger did you changed the values in your created config file ? The file should be at  "./configs/{config_name}.config" (under the virtualfisher-bot folder). If you need help to set everything up, check FAQ and config template.


add webmock dependency for testing

Signed-off-by: Christoph Hartmann<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves]
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

This was a test PR. Will close this.


Bluetooth: controller: ull/lll: Increase adv random delay resolution

Increase the resolution of advertising random delay from
1 ms unit to 1 ticker unit.
Relates to #10289, #10391, and #10398.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

@mtpr-ot changed to u32_t.


Use lv_canvas to draw lsa bar instead of lv_bar

draw lsa bar rectangle on canvas instead of using lv_bar widget

This PR aims to solve the watchdog trigger error that occurred in 2_LSA. The error occurred because of an internal memory issue in the styles API of the LVGL library. Here we shift from the lv_bar approach to the lv_canvas approach to implement the bar graph.

LGTM!


[BUG] Tooltips are not placed properly on Search tags.

Description
In the search (https://www.biodrop.io/search) page the tooltips are not placed perfectly it covers the tags a bit. this needs to be updated. also on the small screen.
Please assign me this issue so that I can improve it.
Thanks.
Screenshots

Do you want to work on this issue?
None
Additional information
No response

Thank you for raising.  These badges will be improved as part of the redesign process we are embarking on, so I will close this issue for now.


Feature: git stash in command panel

fix #628

The maintainers of this repo require that all pull request submitters agree and adhere to the following:

[x] I have read and agree to the contribution guidelines.
(required)
[x] All related documentation has been updated to reflect the changes made. (required)
[x] My commit messages are cleaned up and ready to merge. (required)

The maintainers of this repository require you to select the semantic version type that
the changes in this pull request represent.  Please select one of the following:

[ ]  major
[ ]  minor
[x]  patch
[ ]  documentation only

should we call the commands git: stash xxxx instead of git: xxx stash? Just to match the regular command line input order.

We should also add some docs for git: stash. And we forgot the docs for git: revert too.

should we call the commands git: stash xxxx instead of git: xxx stash? Just to match the regular command line input order.

Edit: but it is a bit strange to have git: stash show and git: stash create...

git: xxx stash reads more natural but it is the wrong order compared to the command line.

We should also add some docs for git: stash. And we forgot the docs for git: revert too.

I can add docs for both tomorrow.

Please review again. @randy3k

added a few more commits, mostly for the StashPanel.

I am still struggle between git: create stash and git: stash create.
@aristidesfl since you requested it, do you have any thoughts?

@stoivo please free to merge it if 👌

@randy3k I think it is simpler to follow the command line order. There are more benefits to it, than otherwise. Also the command line uses git stash save, not git stash save.
Have you thought about also providing git: stash  as an alias to git stash save?

Ya..you are right about the save option. I also agree it should be consistent with the command line.
I don't think we should alias git: stash, but rather it will pop up a quickpanel of all the available suboptions.

Ya..you are right about the save option. I also agree they should be consistent with the command line.
But I don't think we should alias git: stash, but rather it will pop up a quickpanel of all the available suboptions.

Also the behavur could be better when you have no stashed.

So all stash options will be available via a sub-menu?
Stash save also supports an optional description parameter: git stash save 'description'.
Does the PR support that?

They are now available as git: stash save, git stash pop etc. You will be asked a description when you create a stash.

👍 ? @stoivo

I would like to improve the behaviour if there no stashes.

We could extend PaginatedPanel by adding an argument like empty_message along with next_message

I did not want to change the defaut bevavour. But i think that would make sense. Can you commit the changes you thought of?

Merge?

Let 's merge and test it in the dev branch.


Swift42 part2

added @objc on public methods so the class can still be used in Objc

this is to fix: #67

@tinora can we merge this?


Disable no-unexpected-multiline, document as special case

This disables the eslint rule no-unexpected-multiline which is part of the recommended eslint config. Additionally, as suggested by @lydell, it is marked as a special case.
Closes #32.

Thanks!


Added documentation of Seq.item,

since I couldn't find it. In F# 4, Seq.nth is deprecated in favour of Seq.item, which is a new function, added in F# 4.
In order to add this documentation file, I copied the documentation file for Seq.nth and edited it. During that process, I made a few guesses:

I changed the 'author' metadata field to my GitHub name 'ploeh' (but left 'manager' in place).
I updated the 'ms.date' metadata field.
I generated a new GUID for the 'ms.assetid' field.
I left the 'exceptions tag is not supported!!!!' line in place, although it looks strange.
I removed '2.0' from 'Supported in', but left in 'Portable', because, as far as I can tell, Seq.item is available in F# Portable (at least in the configuration I tried).

I'll be happy to make corrections on any of the above, but I wanted to submit a minimal pull request without too much second-guessing, in order to solicit remarks.

@ploeh all looks good. Couple of things:

You can remove the 'exceptions tag is not supported!!!!' fragment - it's an artifact that I need to remove from a bunch of docs.
Update the TOC.md file to include a reference to your new doc.

PR updated. I hope I updated the TOC.md file correctly - I find it a bit difficult to navigate in 'raw' mode :flushed:

Looks good and the check is validated. Merging into the live branch. Should be public shortly.

@ploeh it's live! https://msdn.microsoft.com/visualfsharpdocs/conceptual/seq.item['t]-function-[fsharp] Thank you for your contribution!

Syntax highlighting is broken:

@vasily-kirichenko that's because the default parser didn't detect the language. The mitigation to that is to hint at F# via ```fsharp at the beginning of the snippet. I just pushed a fix to this doc.

@vasily-kirichenko should be good now

Now that I've gotten my feet wet, I also want to add a (small) example to the page. Under which section should I put a code example?

@ploeh Your best bet is looking at the context of the page. For examples, in your case it would be best to create a new section (e.g. Sample Use or Example) and have it between Syntax and Remarks.
The question also comes to - how do you include a code sample? What we want to adopt is a model where code snippets are separate from the content, so shall the code be updated, it will auto-propagate to all pieces that use it.
Look at the snippets folder within the conceptual folder. Most likely you want to place your snippet within the fssequences folder. Create a new .FS file in the format snippetN.fs where N is an increment above the last index within that folder. In your case, this should be snippet203.fs.
Add you code in the file, and then within the doc page you can reference it via:
[!code-fsharp[Main](snippets/fssequences/snippet203.fs)]

Thanks, I'll try tomorrow.
That process sounds a bit error-prone, though. Is there a build tool where I can attempt to build the documentation pages locally, to verify that all is good?
As an example, I can compile FsCheck with a generatedocs command-line argument to generate all the .html and style files on my local machine.

Validation happens on our side. So once you submit a PR, we automatically do a check to make sure that it builds against the stack that we have in place.


Environment w/o X server

Is it possible/how to run the simulator not having X server running?
It being required is a major drawback for usability.

I don't know of any other solution. I'm struggling myself to get this running on CentOS without success yet.
I guess everybody will agree on this being a huge drawback. However, x server is the only thing which one comes across any ml-agents related docs or discussion. Unity needs some display to render to, if visual observations are involved.

Either a virtual X-server or avfb are required to run Obstacle Tower on a linux machine without a display. https://github.com/Unity-Technologies/obstacle-tower-challenge provides a docker image with xvfb already configured to use for training. We also provide instructions on setting up X-server on GCP here: https://github.com/Unity-Technologies/obstacle-tower-env/blob/master/examples/gcp_training.md

Thanks for the reply.
It appears that Docker will have to do.
Closing the issue now.


Dual Z axis - and a guide

Hi, Will this work witha dual z axis setup? I cant see any SP_5x2_256 config or something like that ... also would be great to get a step by step guide on how to set this up on an existing Octopi, I am able to clone the repository, but missing some infor on how to update part of the config file, and also how to compile and run. If you have a bit of time to show me I can write up a guide for people. Thank you!

Great to see some interest in the octoprint plugin! I'm still finishing some things up there (docs, better status reporting and the distributable), should hopefully have the basic stuff done today.
As far as the dual-z, that should still be handled by marlin with the single z axis part of the direct stepping data (it'll still do the identical pulses for up to 4 z steppers). Homing for multi-z should still work normal.

And I'd also definitely appreciate any help with getting the docs in order. Once I get some of the basic stuff done, I would be hugely interested in getting any feedback on next-step improvements (workflow and docs)!

@dmalyavin alright, got a good beta release of the plugin finally: https://raw.githubusercontent.com/colinrgodsey/maven/master/step-daemon/octoprint-plugin/latest.zip
It has the documentation embedded in it now, should be a little easier to get up and going quick


"Install endpoint protection solution on virtual machines" This recommendation is pointing to ASB policy "Monitor missing Endpoint Protection in Azure Security Center". In MDC portal the recommendation if selected is not allowing to exempt the policy rather only giving option to fix all the machines, moreover there is a message available in the portal that updated recommendation is available which points to “Endpoint protection should be installed on your machines”. Question is that, is this recommendation "Install endpoint protection solution on virtual machines" still relevant, if not can this be removed from the reference list and also from the MDC recommendations since there is a score associated with this recommendation.

Install endpoint protection solution on virtual machines" This recommendation is pointing to ASB policy "Monitor missing Endpoint Protection in Azure Security Center". In MDC portal the recommendation if selected is not allowing to exempt the policy rather only giving option to fix all the machines, moreover there is a message available in the portal that updated recommendation is available which points to “Endpoint protection should be installed on your machines”. Question is that, is this recommendation "Install endpoint protection solution on virtual machines" still relevant, if not can this be removed from the reference list and also from the MDC recommendations since there is a score associated with this recommendation

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6230e675-0c07-86be-96e8-2e5463c809bc
Version Independent ID: a9e9c045-ed30-b01a-5682-b102448066ef
Content: Reference table for all Microsoft Defender for Cloud recommendations
Content Source: articles/defender-for-cloud/recommendations-reference.md
Service: defender-for-cloud
GitHub Login: @memildin
Microsoft Alias: memildin

@marishankar-msft
Thanks for your feedback! We will investigate and update as appropriate.

Hi @marishankar-msft . Thanks for the question. The information in this page is automatically generated from the product code. Please address this with Product representatives.
#please-close


Switch to hijax-form for form handling

this has been extracted from uitil into a dedicated library

https://github.com/FND/hijax-form

Looks good! Thank you for this change!


Sometimes pointing subdirectory's git repository, not root directory's.

Extension version: 0.15.0

About vscode:
Version: 1.44.2
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Date: 2020-04-16T17:07:18.473Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 18.7.0

Steps to Reproduce:

clone the repository: https://github.com/PKSHATechnology-Research/camphr
open with $ code camphr
create new issue via quick fix
sometimes opened new issue in another repository of subdirectory (It does not always)

This should be fixed as part of https://github.com/microsoft/vscode-pull-request-github/issues/1936


add seealso

Codecov Report

Merging #2 (d72305a) into master (4f75bb5) will decrease coverage by 8.04%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
- Coverage   93.75%   85.71%   -8.04%
==========================================
  Files           2        3       +1
  Lines          16       21       +5
==========================================
+ Hits           15       18       +3
- Misses          1        3       +2

Files
Coverage Δ

src/Utils.jl
60.00% <60.00%> (ø)


EVM-C: Remove factory class

This is final change to EVM-C API +/- documentation fixes and small refactorings.
I reviewed the previous version by drawing the class diagram of it. I managed to reduce it from 4 classes: Host, Context, EVM, Factory to only 2: EVM and Context. Now it matches very simple OOP polymorphic desing, but it is hard to see in under big C overhead.
I'm not sure we should keep the ABI_VERSION field. It does not seem to be useful in near future.
The still missing piece is tracing.

@chfast  After reading through the code I decided to build the library from the readme instructions.   That seemed to go fine until I got to step 5.  It doesn't say what directory the build directory should go in, and  searching my hard drive for CMakeLists.txt didn't find anyplace that looked right.

Install llvm-3.7-dev package
For Ubuntu 14.04 using LLVM deb packages source: http://llvm.org/apt
For Ubuntu 14.10 using Ubuntu packages
Build library with cmake
mkdir build && cd $_
cmake .. && make
Install library
sudo make install
sudo ldconfig


OSError: Could not find a suitable TLS CA certificate bundle, invalid path: samples/flask/cert/wso2.crt While Running Python SDK

How to reproduce:

Setup Python SDK as mentioned in guide https://docs.google.com/document/d/1n-7DswTz4YJJl4T2wbpylBSa4pPL0UTQ_p3eDPh0VMU/edit#
Up the sample with command
python3 app.py

I have replaced the wso2.crt with tenants crt as well and tried. But still getting the same error
Getting below Error
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/asgardeo_auth/models/op_Configuration.py", line 20, in __init__
    resp = requests.get(url=well_known_url,
  File "/usr/lib/python3/dist-packages/requests/api.py", line 75, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/lib/python3/dist-packages/requests/api.py", line 60, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python3/dist-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python3/dist-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python3/dist-packages/requests/adapters.py", line 416, in send
    self.cert_verify(conn, request.url, verify, cert)
  File "/usr/lib/python3/dist-packages/requests/adapters.py", line 227, in cert_verify
    raise IOError("Could not find a suitable TLS CA certificate bundle, "
OSError: Could not find a suitable TLS CA certificate bundle, invalid path: samples/flask/cert/wso2.crt

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "app.py", line 22, in <module>
    identity_auth = FlaskAsgardeoAuth(auth_config=auth_config)
  File "/usr/local/lib/python3.8/dist-packages/asgardeo_auth/asgardeo_auth.py", line 46, in __call__
    cls._instance = super().__call__(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/asgardeo_auth/Integration/flask_client/flask_asgardeo_auth.py", line 22, in __init__
    super().__init__(auth_config, framework)
  File "/usr/local/lib/python3.8/dist-packages/asgardeo_auth/asgardeo_auth.py", line 59, in __init__
    self.op_configuration = OPConfiguration(
  File "/usr/local/lib/python3.8/dist-packages/asgardeo_auth/models/op_Configuration.py", line 49, in __init__
    raise AsgardeoAuthError(
asgardeo_auth.exception.asgardeo_auth_error.AsgardeoAuthError: Initialized OpenID Provider configuration from default configuration. Because failed to access wellknown endpoint: https://localhost:9443/t/wso2.com/oauth2/token/.well-known/openid-configuration

Environment
IS 5.12.0 alpha 18
Ubuntu 20
JDK 11

@Sathiyakugan Can you have a look on this

The invalid path issue gets resolved when the relative certificate path is changed in the conf file as "cert/wso2.crt" from the path given below.
https://github.com/asgardeo/asgardeo-auth-python-sdk/blob/effbc221659e187da4c67a02600dd042621e431b/samples/flask/conf.py#L13.
But the flow cannot be tested as there is an SSL error.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


implement create,update,delete link item

@HuyTX-LTS
CC: @Nguyenpd-lts @Hokutosei
Thank you very much!
I confirmed the operation. It worked fine.
Could you please fix argument order like below?

createLink

before)
async createLink(
  datastoreId: string,
  itemId: string,
  itemLinkRequestInput: ItemLinkRequestInput,
  projectId: string
)

after)
async createLink(
  projectId: string,
  datastoreId: string,
  itemId: string,
  itemLinkRequestInput: ItemLinkRequestInput,
)

updateLink

before)
async updateLink(
  datastoreId: string,
  itemId: string,
  projectId: string,
  updateItemLinkInput: UpdateItemLinkInput
)

after)
async updateLink(
  projectId: string,
  datastoreId: string,
  itemId: string,
  updateItemLinkInput: UpdateItemLinkInput
)

deleteLink

before)
async deleteLink(
  datastoreId: string,
  itemId: string,
  projectId: string,
  itemLinkRequestInput: ItemLinkRequestInput
):

after)
async deleteLink(
  projectId: string,
  datastoreId: string,
  itemId: string,
  itemLinkRequestInput: ItemLinkRequestInput
):

@kondokentaro  can you check again, I updated.
Then release when you have time.

@Nguyenpd-lts
Thank you.
I got the latest code and built it.
But, I got below error.
Only in my environment?
Could you please check it?
hexabase-sdk % rm -rf node_module
hexabase-sdk % yarn cache clean
yarn cache v1.22.19
success Cleared cache.
✨  Done in 0.07s.
hexabase-sdk % yarn install
yarn install v1.22.19
warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json.
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
error https://registry.npmjs.org/asynckit/-/asynckit-0.4.0.tgz: Integrity check failed for "asynckit" (computed integrity doesn't match our records, got "sha512-Oei9OH4tRh0YqU3GxhX79dM/mwVgvbZJaSNaRk+bshkj0S5cfHcgYakreBjrHwatXKbz+IoIdYLxrKim2MjW0Q== sha1-x57Zf380y48robyXkLzDZkdLS3k=")
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

@kondokentaro, I checked at this branch, But don't have package-lock.json because I removed it before pushed .

@kondokentaro  I updated yarn lock. can you try again.
Thank you so much.

@Nguyenpd-lts
Thank you very much.
That's resolved.
I imported this PR from package.json in react app.
I got below error.
Could you please check it?
Compiled with problems:
ERROR in<EMAIL_ADDRESS>54:13-45
Module not found: Error: Can't resolve './config/default.json' in '/Users/kondokentaro/Documents/workspace/test-hexabase-js/node_modules/@hexabase/hexabase-js/dist'

@kondokentaro I removed file config, can you check again. if this ok, please release

@Nguyenpd-lts
Thank you. LGTM!

@kondokentaro can you push high priority for releasing this?


implement vectored mtvec

Well hello there!
I stumbled upon this:

An implementation may have different alignment constraints for different modes. In particular, MODE=Vectored may have stricter alignment constraints than MODE=Direct.
[...]
Allowing coarser alignments in Vectored mode enables vectoring to be implemented without a hardware adder circuit.

Which I found very interesting and thought we could implement that! It could shorten the latency from IRQ to calling the right ISR as no intermediate generic handler is required. A simple jal zero, <function> as mtvec jump table could directly jump to the handler.
Example jump table:
.balign 128 // 128 byte alignment i.e. lowest 7 bits all zero
mtvec_jump_table:
  jal zero, trap_handler          //  0 - exception (all)
  nop                             //  1 - n/a
  nop                             //  2 - n/a
  jal zero, msi_handler           //  3 - machine software interrupt
  nop                             //  4 - n/a
  nop                             //  5 - n/a
  nop                             //  6 - n/a
  jal zero, mti_handler           //  7 - machine timer interrupt
  nop                             //  8 - n/a
  nop                             //  9 - n/a
  nop                             // 10 - n/a
  jal zero, mei_handler           // 11 - machine external interrupt
  nop                             // 12 - n/a
  nop                             // 13 - n/a
  nop                             // 14 - n/a
  nop                             // 15 - n/a
  jal zero, firq0_handler         // 16 - fast interrupt  0
  jal zero, firq1_handler         // 17 - fast interrupt  1
  [...]

This can then be installed with:
neorv32_cpu_csr_write(CSR_MTVEC, mtvec_jump_table | 0x1);

SiFive actually implemented this in plain C using some nice GCC compiler tricks.
As the direct mode is still available, no immediate change to the current sw framework is required. But those that wish to utilize the lower latency may enable vectored mtvec mode.
I'm not sure about the implications about area and speed. Quartus showed non-deterministic values for LUT usage and adding the vectored mode actually decreased LUT usage (with pr: 1474; without pr: 1494).
WIP to get your feedback if this is generally wished for or not. If ok then I can come up with a test to see how much reduction in  latency there actually is and also add some documentation.
Cheers,
Nik

I'm not sure about the implications about area and speed. Quartus showed non-deterministic values for LUT usage and adding the vectored mode actually decreased LUT usage (with pr: 1474; without pr: 1494).

I have synthesized your modifications and compared them to the current version. Interestingly, I can confirm your findings - the hardware is smaller with vectored mode implemented. To this seems like a good deal: more functionality with less hardware 😎
So, I really like this! But we should think about the alignment of the table entries. I am not sure about the actual use cases, but the current alignment only allows a single jump instruction per table entry reaching a ~1MB range. What if we want to place the table in RAM, say at address 0x8000_0000, and have all the interrupt handler functions in ROM, say at address 0x0000_0000? Then a single jump instruction is not able to reach the according handler functions. Is this something we should take care of?! 🤔 Can we use several instructions (in the table) to construct larger branches without polluting registers of t he interrupted context?

WIP to get your feedback if this is generally wished for or not. If ok then I can come up with a test to see how much reduction in latency there actually is and also add some documentation.

So yes! Let's add this to the core! 👍

To this seems like a good deal: more functionality with less hardware 😎

Perfect, thanks for checking. Let's do this. :)

Then a single jump instruction is not able to reach the according handler functions. Is this something we should take care of?! 🤔 Can we use several instructions (in the table) to construct larger branches without polluting registers of t he interrupted context?

This is actually how it's designed in the ISA... 🤷♂, pc is simply to be put at mtvec + 4 * mcause which gives every interrupt cause only one single 32-bit instruction:

When MODE=Vectored, all synchronous exceptions into machine mode cause the pc to be set to the address in the BASE field, whereas interrupts cause the pc to be set to the address in the BASE field plus four times the interrupt cause number. [source]

But this makes the control hardware actually simpler. No additional fetching of the address from memory needs to be done.
We could solve this in software by jumping to an intermediate larger table that holds more entries per interrupt source and is located close (+/- 1 MB) to the mtvec table. I.e. the trap handler jumps to mtvec + 4 * mcause, where a larger table is located from where we can jump to the whole 4 GB address space. The intermediate trampoline table must save or utilize a scratch register and modify mpc though... I fear that with such a setup the latency is not reduced much.
Alternatively we could support the CLINT ISA Draft in the future. It adds with xtvt a new CSR that is an actual real vector table. But this is more complex: On an interrupt the table entry first needs to be fetched from memory before it can be jumped to it.

So yes! Let's add this! 👍

Great, I'll work out a latency test then. :)

I have tested the latency of the direct and vectored mtvec mode. It is highly dependant on how the software is set up though. The basic test setup was: CPU is in wfi sleep with mei interrupt enabled. Testbench triggers interrupt and the installed ISR writes GPIO pin 0 to high. The latency is then assumed to be the time between mei assertion and gpio value change.
I got some preliminary latency measurements while simulating various different SW configurations:

26 cycles: Vectored mode + not calling functions in ISR
121 cycles: Vectored mode + calling functions in ISR
73 cycles: Direct mode + simple switch-case handler + not calling functions in ISR
142 cycles: Direct mode + simple switch-case handler + calling functions in ISR
265 cycles: Direct mode + default neorv32 RTE + not calling functions in ISR
285 cycles: Direct mode + default neorv32 RTE + calling functions in ISR

By not calling any functions, gcc is able to optimize away the saving and restoring of the caller-saved cpu registers in the ISR.
The vectored mode seems to reduce the interrupt latency up to 10 times! 🚀

We should also think about a simple test case for the processor_check to verify this new hardware mechanism.

Sure! I'll probably manually install a table, use the sim_irq_trigger() to test mei and msi interrupts, and then re-install the rte with the global_trap_handler() as handler for everything. Do you see any problems with that approach?

What do I do wrong...? I can compile and run the new test just fine...
root@neorv32_soc:/workspaces/neorv32_soc/lib/neorv32/sw/example/processor_check# make all
Memory utilization:
   text    data     bss     dec     hex filename
  23636       0     296   23932    5d7c main.elf

root@neorv32_soc:/workspaces/neorv32_soc/lib/neorv32/sw/example/processor_check# make sim
Installing application image to ../../../rtl/core/neorv32_application_image.vhd
Simulating neorv32_application_image.vhd...
[...]
[27] Vectored IRQ (sim) [ok]
[...]

I have GCC 12.2.0.

Hey @NikLeberg!
Looks great so far! I'm in the train right now having ~bad~ no internet connection... 🙈
I am not C/software/GCC expert at all, but as far as I can see this might be the problem:
void vectored_irq_table(void) {
  asm volatile(
    ".org  vectored_irq_table + 0*4     \n"
    "jal   zero,vectored_global_handler \n" // 0

The entire code is compiled first and is compiled afterwards. So at compile-time the label vectored_global_handler is not known (= the address is not fixed) until linking.
You need to keep an "alias" here until linking. Maybe something like this could work (?):
  asm volatile(
    ".org  vectored_irq_table + 0*4     \n"
    "jal zero, %[dst]                   \n" // 0
  : : [dst] "i" ((uint32_t)&vectored_global_handler)
  );

By the way, thanks for the detailed latency evaluation!
Vectored interrupts might be very handy for small-scale real-time systems running at low operating frequencies. I think FreeRTOS (now?) also provides an option to use vectored interrupts... ?! 🤔

Maybe we could put the table into a unique compilation unit - so the labels might remain aliases until linking 🤔
section(".text.vector_table")

That was worth a try, thanks for the idea. Sadly it was not it..
It's the -flto link time optimization flag! The run_check.sh script specifically sets that. Seems as if something gets optimized too early or the inline assembly simply can't be written like that when lto is enabled...

In general, link-time optimization seems like a good thing to do. I am not sure if it would be some (dirty) work-around if we disable that...
What do you think about giving an explicit clobber list? This should keep an alias for the addresses which the linker should be able to resolve:
  asm volatile(
    ".org  vectored_irq_table + 0*4     \n"
    "jal zero, %[dst]                   \n" // 0
  : : [dst] "i" ((uint32_t)&vectored_global_handler)
  );

Spoke too soon, it was the inline asm alias thing you suggested earlier. Sorry for not trying that out sooner.

All passing now. Thanks for the help! ❤
I rebased on latest main. Now the last step would be to add documentation.
Can you share some insights about your editor setup? I'm not too skilled with AsciiDoc to get it visually right on the first time. And waiting for CI to scream at me is a slow workflow. 😄
Do you edit the .adoc files by hand or do you have a specific VsCode extension to recommend? Especially support for creating / editing tables would be nice to have.

Awesome! Thanks for all your work Nik! I this is is very cool new feature :+1:

Do you edit the .adoc files by hand or do you have a specific VsCode extension to recommend? Especially support for creating / editing tables would be nice to have

Actually, I use Notepad++ for everything. I guess I am quite a minimalist 😅 Anyway, asciidoc is quite simple (a bit like Markdown) and I think it is sufficient if we update the mtvec CSR description.
Beyond the documentation, we still need to update the version number and make an entry for the change log.

Looks good to me! :+1:
Thanks for all your work Nik!


Swift Conversion: Splash Screen & Agreement Controller

Description
In this PR, I have converted OEXLoginSplashViewController to LoginSplashViewController and
OEXUserLicenseAgreementViewController to AgreementWebViewController from Objective-C to Swift.
How to test this PR

Verify designs of both ViewControllers.
Check Splash Screen Buttons are working.
Content is properly loaded in AgreementWebViewController.

This is no longer required will revisit the approach in the future when it will be needed.


Cleanup after an effect before executing a new one

Pull Request Type

[x] Bug fix
[ ] New feature
[ ] Refactoring
[ ] Documentation update
[ ] Chore

Issue for this PR
Link: N/A
Description
Fixes issue where effects with dependencies (eg .preserved(by:)) would cleanup itself after new effect ran. This is different behavior from React's implementation and could cause unexpected results.
Motivation and Context
Running the following code with Hooks v0.6.0, after tapping the button, useHook would first run new effect, then cleanup after previous effect. The timeline is:

[Key: 1]
Run Effect [1]
Increment key [Key: 2]
Run Effect [2]
Cleanup Effect [1]

Steps 4 and 5 should run in opposite order.
The issue emerges in UseEffectHook.State in willSet property observer. The observation triggers old effect's cleanup by oldValue?() by which time new effect already finished to return new cleanup closure.

SwiftUI Code
func App() -> some View {
    HookScope {
        let state = useState(0)
        let _ = useEffect(.preserved(by: state.wrappedValue)) {
            print("subscribe \(state.wrappedValue)")
            return {
                print("unsubscribe \(state.wrappedValue)")
            }
        }

        Text(state.wrappedValue.description)
        Button("Add") {
            state.wrappedValue += 1
        }
    }
}

React Code
function App() {
  const [state, setState] = React.useState(0);

  useEffect(() => {
    console.log("subscribe " + state);

    return () => console.log("unsubscribe " + state);
  }, [state]);

  return (

{state}

      <button onClick={() => setState((old) => old + 1)}>Add</button>

  );
}

Running the sample code in patch-1 branch, the results are same as React app above.

Impact on Existing Code
The change is source-compatible.
Screenshot/Video/Gif

https://github.com/ra1028/swiftui-hooks/releases/tag/0.0.7


Added SearchContainer and ISearchable

:x: Build osu-framework 0.1.0-PullRequest.538+2218.build.1758 failed (commit https://github.com/ppy/osu-framework/commit/4ddf861c94 by @Jorolf)

https://streamable.com/lm8k2
^ TestCase

:x: Build osu-framework 0.1.0-PullRequest.538+2313.build.1837 failed (commit https://github.com/ppy/osu-framework/commit/b4a5a48e8f by @Jorolf)

:x: Build osu-framework 0.1.0-PullRequest.538+2314.build.1838 failed (commit https://github.com/ppy/osu-framework/commit/bfaeb0f20d by @Jorolf)

closed until I get a 'example' ready for ppy/osu


Set up Config files for use with PTR experiment script

Summary
Want the PTR experiment script to flexibly adjust, depending on user, the path to e.g. stimuli, data folders, etc, including S drive mounting locations and git repository locations.
Implementation
Look into how CONFIG files work with MATLAB (c.f. how they work with R).
Additional context
config.yml already exists, need to figure out how to use it.

Not going to pursue this; experiment computers are not currently set up with GitHub.


Add setPdfFileBaseName

Allow customizable pdfFileBaseName for control over outputImagePath. Great for when not writing file to disk locally and tmp file is used for image conversion.

Anything I can do to get this baked in and on npm? Thanks!


Change LapRep Definition

Right now lap representative time calculations require the lap time be completed on a slick compound.
A more flexible approach is warrantied. If the track is wet, then the representative time should be a time completed on a wet compound.
Some options:

Use median of all slick/wet compound times, whichever one is majority

A weighted sum between slick/wet compound times

A weighted sum grouped by all compound types

Also applies to per round representative time

The existing implementation already consider all compunds for per lap representative time.
A better formula for per round representative time will be investigated separately


WYSIWYG editor for templates

I've created a Gitfiti Painter with support for github-board style template syntax.
http://codepen.io/cbas/pen/vOXeKV


Plugins/scripts

Disclaimer: This will require .NET 4.5 most likely (ScriptCS does)

This discussion may become a bit of a moot point then, depending on how the owners/maintainers feel about the framework version

More of a discussion than anything.
One of the advantages Jekyll (not on GH-Pages) that is inherent from Ruby is the ability to have 'loose file' (ie non-compiled) plugins. Plugins could be a wide range of utility and complexity. Two that spring to mind would be a PNGCrush plugin to minify all your images on compile, and for me I'd like to extend Liquid tags to have a [video id=xyz] or similar to easily embed a youtube video.
Food for thought

IronRuby might make it possible to use Jekyll plugins as is (should work on mono too)
ScriptCS/Roslyn might give the ability to do the same for C# (should work on mono)
ClearScript could do the same for JS.

I think even just a single target (where I think C# would be the more obvious choice) of extensibility would be good.
How would the plugins get called in? Much like _posts, it'd be a special, per-site folder, _plugins following the Jekyll syntax.

For templating (rather than post processing) plugins, the insertion point would be around ISiteEngine.Initialize(). Perhaps that should take some parameters? (ie, for Liquid, a collection of Tag's to register). It needs to be before Process is called otherwise tags won't be handled properly.
For post-processing, well, thats going to be done with ITransforms which is currently pulled in with MEF. The SiteContextGenerator would have enough information by then to add any transforms to that collection - perhaps in a BuildPlugins method much like BuildPosts

It's possible right now, with the use of MEF: if you drop a dll which exports a DotLiquid.Tag, an IFilter, an ITransform or an IContentTrasnform it will be loaded and used during the site processing.
Also, it's may be silly but I'm not comfortable with non-compiled code executed by Pretzel.
And I think it's more easy to write a plugin with the the help of intellisense just by overriding or implementing a method.

I think we should seriously consider ScriptCS. The ability to quickly knock up a script to add a tag cloud for instance like you can in Jekyll would be awesome.
@filipw @jrusbatch @adamralph @khellang any thoughts on the best way to implement ScriptCS extensions support would be?

I don't know much about Pretzel, but we have the ScriptCs.Hosting package for this scenario :smile: It lets you quickly wire up the needed dependencies and execute scripts. One thing to think about is how much of the scriptcs functionality you want to leverage, e.g. pulling down NuGet packages, script packs etc. (or the whole shebang?) and what kind of hooks you'd like to give to the scripts.

Pretzel is Jekyll written in .net. We want to be able to rewrite jekyll extensions in scriptcs.

It should be very easy to do. The hosting library should provide everything you need, including automatic engine selection (Roslyn/Mono) based on the current OS, so I would give that a try first.
I guess Christian's concern is that you may find hosting too heavyweight, so you can always drop down a level, as I currently do in ConfigR (although I do plan to level up to the hosting lib - ConfigR was developed in the early days whilst the hosting lib was taking shape) but I wouldn't worry about this optimisation until if and when you find you need it.

So if somebody does want to tackle this, or have more discussion about it, this seems to be the "basic" code you'd need to host ScriptCS. In this context I was experimenting with registering Liquid tags hosted in csx, so I'm presuming I'm doing most things wrong.
The script (VideoTag.csx)
public class VideoBlock : Tag
{
    public override void Initialize(string tagName, string markup, List<string> tokens)
    {
        base.Initialize(tagName, markup, tokens);
    }

    public override void Render(Context context, TextWriter result)
    {
        result.Write("HELLO");
    }
}

var tags = Require<LiquidContext>().Tags;
tags.Add(new VideoBlock());

This defines VideoBlock as a tag which we can register later, then adds it to a IEnumerable  that you'll see down below.
The "host" object
public class ScriptCsHost
{
	public ScriptServices Root { get; private set; }

	public ScriptCsHost(bool useLogging = true, bool useMono = false)
	{

		var console = new ScriptConsole();
		var configurator = new LoggerConfigurator(useLogging ? LogLevel.Debug : LogLevel.Info);
		configurator.Configure(console);
		var logger = configurator.GetLogger();
		var builder = new ScriptServicesBuilder(console, logger);
		if (useMono)
		{
			builder.ScriptEngine<MonoScriptEngine>();
		}
		else
		{
			builder.ScriptEngine<RoslynScriptEngine>();
		}
		Root = builder.Build();
	}
}

This is all pretty 'basic' in the ScriptCs world stuff. Boilerplate code.
Context/Script packs.
ScriptPacks can be "Required" by scripts and are how we can ferry data back and forth, probably. Alternatively you can pass in params, but they're just strings.
public class LiquidContext : IScriptPackContext
{
	public List<Tag> Tags { get; set; }
}

public class PretzelScriptPack : IScriptPack
{
	private LiquidContext context;

	public void Initialize(IScriptPackSession session)
	{

	}

	public IScriptPackContext GetContext()
	{
		return context ?? (context = new LiquidContext());
	}

	public void Terminate()
	{

	}
}

The actual 'calling' calling/setup of scriptcs.
As mentioned above, I'm probably doing this wrong, but its... at least a proof of concept. The ScriptPack has a Context, which has a collection of tags. If you wanted to add extra data, instead of just newing it up, it can be assigned there.
var host = new ScriptCsHost();
var f = new PretzelScriptPack();
var tags = new List<Tag>();
((LiquidContext)f.GetContext()).Tags = tags;
host.Root.Executor.Initialize(new[] { "system" }, new[] { f });
host.Root.Executor.AddReferenceAndImportNamespaces(new[] { typeof(LiquidEngine), typeof(IScriptExecutor), typeof(Tag) });

var result = host.Root.Executor.Execute(<filename>);
host.Root.Executor.Terminate();

Now the 'interesting' bit. This code is actually already used elsewhere in LiquidEngine.cs for registering Tags discovered by MEF.
var registerTagMethod = typeof(Template).GetMethod("RegisterTag", System.Reflection.BindingFlags.Public | System.Reflection.BindingFlags.Static);

foreach (var tag in tags)
{
	var registerTagGenericMethod = registerTagMethod.MakeGenericMethod(new[] { tag.GetType() });
	registerTagGenericMethod.Invoke(null, new[] { tag.Name });
}

Then in my markdown, I can call it like so {% videoblock "some information" %}

So the point of the previous comment was that it can be done. Should it be done is a harder question to answer, so I'll leave this here for now until somebody else wants to take up the matter.
Pros

Super neat
Adding scripts as plugins is a much easier end user experience than having to compile code, for what the majority of plugins would/could do. Particularly if its just modifying somebody elses script to output slightly different text.
Requires upgrade to .NET 4.5

Cons

More complexity in code, will probably result in a performance drop on baking
As mentioned, the functionality is largely already available (at least for Tags) in the form of compiled DLLs
ScriptCs is confusing
Triples the filesize
Requires upgrade to .NET 4.5

@vikingcode thanks for exploring this. Which part of scriptcs do you find confusing?

@vikincode the hosting code you have looks correct.
The script pack is one way to move information back and forth. Another option is to have a custom host. You can create a custom host and then hang custom methods off of it. You will then have a custom host factory to pass it in.
As you are a custom host, this is a common pattern. Basically what you do is create a custom ScriptHostFactory , and your own class which derives from ScriptHost which it returns. You custom factory can have additional members which are then available directly in the script.
This is more natural because there is no need to call Require<T> or keep a local variable. The members on the custom host are immediately accessible.

@glennblock Ah, I think the ScriptHostFactory/custom ScriptHost would be the part I was missing to make it flow a bit more cohesively. Although I adopted MEF back in the preview stage, it did take me awhile to 'get'. Perhaps its the Glenn Block parts I find confuse? :D

LOL. Well I wrote a lot of the first version of scriptcs, so you can probably blame me for quite a bit :-)

@vikingcode check this PR for more on the custom host: https://github.com/scriptcs/scriptcs/pull/508

It works well with a custom ScripHostFactory / ScriptHost, but since we already use MEF for Pretzel's extensibility I think it's best to keep the same mechanism.
it is easily possible with scriptcs, for example with the same ScriptCsHost as above and the following interface for extensibility example:
[InheritedExport]
public interface ITuc
{
    string Name { get; }
}

We only need to load the scripts like this:
var host = new ScriptCsHost();
host.Root.Executor.Initialize(new[] { "system" }, Enumerable.Empty<IScriptPack>() ); // not sure about this one
host.Root.Executor.AddReferenceAndImportNamespaces(new[] { typeof(ITuc) });
var result = host.Root.Executor.Execute(<scriptFileName>);
host.Root.Executor.Terminate();

And after that make the MEF Compose with an AssemblyCatalog on the dynamic assembly generated by scriptcs.
private void Compose()
{
    var first = new AssemblyCatalog(Assembly.GetExecutingAssembly());
    var catalog = new AggregateCatalog(first);

    foreach (var assembly in AppDomain.CurrentDomain.GetAssemblies())
    {
        if (assembly.FullName.StartsWith("ℛ*"))
        {
            catalog.Catalogs.Add(new AssemblyCatalog(assembly));
        }
    }

    var container = new CompositionContainer(catalog);

    var batch = new CompositionBatch();
    batch.AddPart(this);
    container.Compose(batch);
}

The assembly.FullName.StartsWith("ℛ*") seems clumsy but it's the only way I found (for now) to check if the assembly is generated by scriptcs.
Or may be it is the call to make the ScriptCatalog?

Nice work guys!


feat(extras): add typescript vscode extra

Hi! Spend a month tweaking my typescript configuration to integrate it with vscode.
How is it different from typescript lang extra?

It uses Vtsls instead of Tsserver which has better memory usage (at least from my testing)
It uses the cmd .vscode/launch.json for dap configuration. Which allows to define dap configuration per project and use them alongside VSCode colleagues)

I am sure it could be a bit cleaner and better, so maybe you or somebody else will change it a bit)

I prefer typescript-tools instead of tsserver. Tsserver is not suitable for big projects for the same reason

Are there any news on will it be merged? It is my first PR that takes so long to merge. Is everything alright with it?

The maintainer has updated the pinned issue that he will be out until 28 April and be back next week. None other than the maintainer has merge rights, so please be patient until he's back.

I'll leave this PR open for now, for all the DAP stuff. not sure what part of that is needed or would be better than what we currently have in the typescript extra.

Done, see: https://github.com/LazyVim/LazyVim/commit/fba06ce9f522b91be8a342f9c028948c2733132d

I did not include all the keymaps from this PR, since you can always do <leader>cA to get all the source actions for any lsp server.
I'll leave this PR open for now, for all the DAP stuff. not sure what part of that is needed or would be better than what we currently have in the typescript extra.

Sure, I did a dap configuration following this guide

Closing since we now use vdtsls in the typescript extra.
Thank you anyway!

Closing since we now use vdtsls in the typescript extra. Thank you anyway!

Sure @folke! Should we add dap to vtsls? As for now, it is very simple, and it can be a pain to setup

I'm honestly not using dap for typescript often.

What do you usually use for debugging?


rootNamespace is set to null in helm install

Bug Description

https://github.com/istio/istio/blob/master/manifests/charts/istio-control/istio-discovery/templates/configmap.yaml#L10

When I upgrade my helm version, I found that rootNamespace changed from istio-system to null. Is that expected?
It's null in values.yaml and helm doesn't render it to .Values.global.istioNamesapce.
Version
istio v1.16.0
helm v3.10.2

Additional Information
No response

Releated helm issue: https://github.com/helm/helm/issues/9587

Default is istio-system. What if .Value.global.istioNamespace is not istio-system?

Ah didn't realize that was broken too. thanks


Update menu.js

Added the DefaultIndex function I made for myself, quite useful if you want to display let say a scale from -1.0 to 1.0 and have the default value to 0.0
rt-2

I don't see what changes you want me to make sorry.
Is it ok or is there still things you want me to edit.
Thank you,
rt-2


docs: Place warning at the top of every page in /latest/production.

Added warning block to _templates/layout.html
Removed copy-pasted warning from install.md and requirements.md

ref: https://www.sphinx-doc.org/en/master/templating.html
Addresses #13056.
Testing Plan:
./tools/test-documentation --skip-external-links
./tools/check-templates
./tools/lint --groups=frontend
GIFs or Screenshots:

Merged, thanks @drrosa and @andersk!  We might find we want to do a more generic way to do the exclude rule as we add more pages to the production docs, but we can worry about that then.


🚀 Feature Request: Remove OAuth authorization from the web dashboard

Describe the solution
Currently, it seems that there's no way to remove any authorized OAuth application using the web UI, and revoking an OAuth token requires running wrangler logout on the machine where the original wrangler login was run. This can be a problem, say, if the dev machine is lost.

This work is in progress with our Dashboard team, but I'm not sure on an ETA. That work will provide a way to revoke tokens via the dashboard UI.
I'm going to close this out from this repo, as it is not directly related to Wrangler, and there are documented workarounds for developing via Wrangler.

Dear fork,
Does Cloudflare support revoke authorization from dashboard now?
Many thanks.


Guidance on Using Powertools with AWS Websocket API Gateway

Is your feature request related to a problem? Please describe.

Could some guidance on how to use Powertools to implement a Lambda for an AWS Websocket API Gateway route be provided.
Note -- this may require a new API Gateway class.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

hey @gwlester, thanks for creating this feature request. I'm moving this to our roadmap repo so we can consider it later.
At the moment, we mostly have customers using AppSync for real-time applications (web sockets+GraphQL+subscriptions+data conflict resolution etc) so we prioritized that first for the Event Handler. If anyone else reading this later would like support for API Gateway WSS, please react with a 👍
Thank you!


Drag

Is it possible to implement drag functions with your library? I mean I need to perform pan and zoom for whole svg and have possibility to move some elements of it separately.

The original library that this one is based on allowed for dragging, but we
wanted to keep this library tightly focused on pan/zoom. I'd be interested
in making it possible to add drag and drop functionality as a plug-in.
On Jun 9, 2016 6:18 AM, "Arvidas1"<EMAIL_ADDRESS>wrote:

Is it possible to implement drag functions with your library? I mean I
need to perform pan and zoom for whole svg and have possibility to move
some elements of it separately.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ariutta/svg-pan-zoom/issues/205, or mute the thread
https://github.com/notifications/unsubscribe/ABPJg64yq9_UPn6ynN6QwwPXa-rRkSlpks5qKBKwgaJpZM4Ix9mK
.


hydra: 2019-03-18 -> 2019-05-06

Motivation for this change
As Hydra doesn't have a release model, we have to pin git revs for our
package. The current state provides some new fixes and features like a status
plugin for GitLab.
As Nix 2.0 isn't bundled in master and release-19.03 and we don't
support Nix 1.11 for Hydra, I figured that the Nix version checks could
be dropped and boost can always be used as build input.
Things done

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

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

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

This pull request has been mentioned on Nix community. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/9

this change appears to have renamed the hydra package to nix-build, due to releaseTools.nixBuild not supporting pname