Examples npm run dev failing

I've been having troubles running "npm run build" from examples/gh-pages
npm-debug.log outputs the following:
0 info it worked if it ends with ok
1 verbose cli [ 'node', '/Users/lukepint/.node/bin/npm', 'run', 'dev' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose node symlink /usr/local/bin/node
5 verbose run-script [ 'predev', 'dev', 'postdev' ]
6 info predev<EMAIL_ADDRESS>7 info dev<EMAIL_ADDRESS>8 verbose unsafe-perm in lifecycle true
9 info<EMAIL_ADDRESS>Failed to exec dev script
10 verbose stack Error<EMAIL_ADDRESS>dev: `npm run clean && webpack-dev-server --content-base ../../public`
10 verbose stack Exit status 1
10 verbose stack     at EventEmitter.<anonymous> (/Users/lukepint/.node/lib/node_modules/npm/lib/utils/lifecycle.js:213:16)
10 verbose stack     at EventEmitter.emit (events.js:110:17)
10 verbose stack     at ChildProcess.<anonymous> (/Users/lukepint/.node/lib/node_modules/npm/lib/utils/spawn.js:24:14)
10 verbose stack     at ChildProcess.emit (events.js:110:17)
10 verbose stack     at maybeClose (child_process.js:1008:16)
10 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1080:5)
11 verbose pkgid<EMAIL_ADDRESS>12 verbose cwd /Users/lukepint/workspace/boomplans/react-google-maps-master/examples/gh-pages
13 error Darwin 14.0.0
14 error argv "node" "/Users/lukepint/.node/bin/npm" "run" "dev"
15 error node v0.12.0
16 error npm  v2.8.4
17 error code ELIFECYCLE
18 error<EMAIL_ADDRESS>dev: `npm run clean && webpack-dev-server --content-base ../../public`
18 error Exit status 1
19 error Failed at the<EMAIL_ADDRESS>dev script 'npm run clean && webpack-dev-server --content-base ../../public'.
19 error This is most likely a problem with the gh-pages package,
19 error not with npm itself.
19 error Tell the author that this fails on your system:
19 error     npm run clean && webpack-dev-server --content-base ../../public
19 error You can get their info via:
19 error     npm owner ls gh-pages
19 error There is likely additional logging output above.
20 verbose exit [ 1, true ]

In addition, I found this in the output of the run:
{ file: '/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/bootstrap-sass/assets/stylesheets/bootstrap/mixins/_grid.scss' }
/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/webpack-dev-server/node_modules/webpack-dev-middleware/middleware.js:85
			if(err) throw err;
			              ^
TypeError: tmp/server.js
--------------------------------------------------------------------------------options must be an object
    at TypeError (native)
    at Object.exports.createScript (vm.js:44:10)
    at module.exports (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/eval/eval.js:55:8)
    at IsomorphicReactPluginFactory.evaluate (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:56:12)
    at IsomorphicReactPluginFactory.runCompilation (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:68:32)
    at IsomorphicReactPluginFactory.receivedClientAssets (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:33:10)
    at ClientPlugin.<anonymous> (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:97:24)
    at Tapable.applyPluginsAsync (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/webpack/node_modules/tapable/lib/Tapable.js:71:13)
    at Tapable.Compiler.emitAssets (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/webpack/lib/Compiler.js:216:7)
    at Watching.<anonymous> (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/webpack/lib/Compiler.js:45:18)
--------------------------------------------------------------------------------

Can you provide any insight into why this might be failing?
Thanks

I believe you didn't type npm i in the root of the repo, right?
Never mind, I fixed the webpack.config.js to resolve it here so you just need to type npm i and npm run dev inside examples/gh-pages.
Please let me know if it works, thanks!

Here's a gist using - "npm i" then "npm run dev" from the root directory.
https://gist.github.com/lapint/f6c953fc9a26b4b0b572
Similar message with your suggested change and run from examples/gh-pages

Thanks for the fast reply, unfortunately it's still not working.
The webpack compiles until if fails with this error:
TypeError: tmp/server.js
--------------------------------------------------------------------------------options must be an object
    at TypeError (native)
    at Object.exports.createScript (vm.js:44:10)
    at module.exports (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/eval/eval.js:55:8)
    at IsomorphicReactPluginFactory.evaluate (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:56:12)
    at IsomorphicReactPluginFactory.runCompilation (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:68:32)
    at IsomorphicReactPluginFactory.receivedClientAssets (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:33:10)
    at ClientPlugin.<anonymous> (/Users/lukepint/workspace/boomplans/examples/gh-pages/IsomorphicReactPluginFactory.js:97:24)
    at Tapable.applyPluginsAsync (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/webpack/node_modules/tapable/lib/Tapable.js:71:13)
    at Tapable.Compiler.emitAssets (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/webpack/lib/Compiler.js:216:7)
    at Watching.<anonymous> (/Users/lukepint/workspace/boomplans/examples/gh-pages/node_modules/webpack/lib/Compiler.js:45:18)
--------------------------------------------------------------------------------

So it seems to be something not compiling right. I'm running on Mac OSX 10.10.1 but wouldn't think that should matter.
I actually tried cloning installing and running on 2 Macs and they fail at the same spot.

Here's a gist of what's happening:
https://gist.github.com/lapint/3ed3ea184bfdf0def81a

I was able to find a working version here:
react-google-maps lukepint$ git checkout 338234a0545f9c7f7de79d5fe1a5b65540265c44
HEAD is now at 338234a... Merge pull request #37 from tomchentw/chore-update-dev-deps

Likewise seeing this error when building the example pages.
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build_client: npm run clean && webpack -p
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build_client script 'npm run clean && webpack -p'.
npm ERR! This is most likely a problem with the gh-pages package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! npm run clean && webpack -p

(This is on a fresh clone/install)

I'm wondering if this is an issue with what version of node is being used server side.
Running this version:
git checkout 931aaab792520df6f4e119e304e926a43d4f4078
Causes "npm run dev" to fail and it is when the app was converted to an isometric one.
Looks like it's coming from eval.createScript() not being able to parse the content being passed into it maybe.

I don't know much about Webpack (I use Browserify)... Is there even a server involved when running just webpack -p (not webpack-dev-server)? Because that fails for me as well.

webpack-dev-server still uses webpack, it just starts up a dev server for it as well.
The problem is that IsomorphicReactPluginFactory.js compiles all the required js into a component and then uses React.renderToString(ServerComponent) to render the client side from the server component.
I think eval could be the culprit, but not exactly sure how to fix it at this point.

I was able to fix the issue on master, I changed line 56 of IsomorphicReactPluginFactory.js
from: return evaluate(source, includeGlobals);
to: return eval(source, includeGlobals);
It's not the best since it uses javascript eval but it allows the server to start.

Sorry I missed the thread, I'm considering this issue might be related to versioning issues of node and npm. I'm running on node: v0.10.38 and npm 2.7.5

Got the same error today at<EMAIL_ADDRESS>options must be an object

Thanks @lapint for spotting eval might be the issue. I guess it's due to old API used for<EMAIL_ADDRESS>but it's incompatible with<EMAIL_ADDRESS>Switch to a new module called eval-as-module solve this problem.

I'd like to contribute to this project with however I'm running into the following error even in<EMAIL_ADDRESS>when running npm run dev.
ERROR in ./~/react-toastr/lib/ToastMessage/jQueryMixin.js
Module not found: Error: Cannot resolve module 'jquery' in

Any quick solutions to fix this or remove the dependency?

@urban hmm in latest master branch, jQuery is included as devDependencies. Could you check your node_modules if it exists?
https://github.com/tomchentw/react-google-maps/blob/a5d4e2590adb369bda693ada07529f7c68971129/examples/gh-pages/package.json#L20

@tomchentw Yes, you are right. When I wrote my earlier post, I didn't realize I was in your chore-node-012-compatibility branch, which is a WIP and not the master branch. Sorry for the inconvenience.

No worries @urban ! Glad it works now.

Still having issues building the docs on the latest version and on multiple computers.
npm ERR! argv "node" "/usr/local/bin/npm" "run" "dev"
npm ERR! node v0.12.2
npm ERR! npm  v2.7.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>dev: `cd examples/gh-pages && npm i && npm run dev`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script 'cd examples/gh-pages && npm i && npm run dev'.
npm ERR! This is most likely a problem with the react-google-maps package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cd examples/gh-pages && npm i && npm run dev
npm ERR! You can get their info via:
npm ERR!     npm owner ls react-google-maps
npm ERR! There is likely additional logging output above.

0 info it worked if it ends with ok
1 verbose cli [ 'c:\Program Files\nodejs\node.exe',
1 verbose cli   'c:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js',
1 verbose cli   'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info prestart<EMAIL_ADDRESS>6 info start<EMAIL_ADDRESS>7 verbose unsafe-perm in lifecycle true
8 info<EMAIL_ADDRESS>Failed to exec start script
9 verbose stack Error<EMAIL_ADDRESS>start: webpack-dev-server --hot
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter. (c:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:213:16)
9 verbose stack     at EventEmitter.emit (events.js:110:17)
9 verbose stack     at ChildProcess. (c:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
9 verbose stack     at ChildProcess.emit (events.js:110:17)
9 verbose stack     at maybeClose (child_process.js:1015:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
10 verbose pkgid<EMAIL_ADDRESS>11 verbose cwd c:\D-DRIVE-DATA\React\React-Modules
12 error Windows_NT 6.3.9600
13 error argv "c:\Program Files\nodejs\node.exe" "c:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "start"
14 error node v0.12.5
15 error npm  v2.11.2
16 error code ELIFECYCLE
17 error<EMAIL_ADDRESS>start: webpack-dev-server --hot
17 error Exit status 1
18 error Failed at the<EMAIL_ADDRESS>start script 'webpack-dev-server --hot'.
18 error This is most likely a problem with the reactmodules package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     webpack-dev-server --hot
18 error You can get their info via:
18 error     npm owner ls reactmodules
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]
getting error while running npm start command

I have added
"scripts": {
"start": "webpack-dev-server --hot"
},
to start webpack

@tomchentw
it works! thanks


fix typo

Fix typo: formated shoud be formatted

@TomVergote,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/downloads/newsid.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[table viz] get metrics to right-align

Moved the histogram to be rooted on the right side as well
fixes https://github.com/airbnb/superset/issues/2933

could you share before/after screenshot?

@ascott here:

thx @mistercrunch . LGTM


arangorestore associates data to root

I am attempting to upgrade to rocksdb on my server, so I did an arangodump, purged the install, reinstalled, created the user, created the database associating it to the user, then did arangorestore:
arangorestore --input-directory backup-2017.09.13 --server.database mydb --server.username myuser --server.password mypassword --create-collection true --import-data true
This completes without error, but when I log in as "myuser" there are no collections.  If I log out and log in as root they are all there.  They were associated to "myuser" before the dump as well.  It seems clear that the arangorestore command is improperly assigning the owner as "root" instead of "myuser" when restoring.
my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

3.2

Mode:

Single-Server

Storage-Engine:

rocksdb

On this operating system:

Linux

Ubuntu .deb

Is there a temporary workaround for this so that I can re-associate the database with my user instead of with root?

Also, I just noticed that the dump doesn't contain the graph that all the collections were associated with.

Databases are not associated to users, your use may have rights to a database and the collections in it. The user rights are only restored when you restore the _system collection.
const users = require('@arangodb/users');
users.grantDatabase('myuser', 'mydb', 'rw');
users.grantCollection('myuser', 'mydb', '*', 'rw')

Technically your user should have been assigned read / write rights on these collections (if they had to be created).
Not quite sure what happened, I guess we are missing this an internal grantCollection there ?
Can myuser access mydbdatabase? You can check this:
 users.permission(myuser', 'mydb')

I was able to work around this by having my application create the graph and collections before doing the restore and now everything is fine.

Hi there, I can confirm the BUG.
1- Create User
2- Create Database Assigned to the User
3- Restore Data
4- User doesn't have permission to query data
5- Fix it using the administration panel (the default permission is set to none)

Hi I'm really sorry, we forgot to backport a fix for this issue to the 3.2 branch in time for the latest release. This will be resolved once https://github.com/arangodb/arangodb/pull/3717 is merged

This should be fixed in the latest release

Hi,
3.2.8 containing this fix is available for download now.
Closing.


Python 3 support

Not much needed to change, just added parentheses to the print statements for python 3 to be happy.
I also wrapped up the failed ctypes.windll.LoadLibrary to raise a ImportError instead, since on Linux, it raises an unhelpful AttributeError when HIDAPI isn't installed on the system.

No worries


Accessiblity #a11y issues

- Do you want to request a feature or report a bug?
Bug
- What is the current behavior?
Netlify CMS fails on a number of accessiblity issues:

Tab order is a bit awry
Use HTML elements, not divs: eg main, aside, button, nav, list items for posts
Duplicate h1 on same page
Colour contrast http://contrast-ratio.com/
Test with no CSS (icon size)
Alt text for img & svg
Match form labels with same id as form inputs

- If the current behavior is a bug, please provide the steps to reproduce.

Use the tab button to navigate around the application
Use a screenreader to navigate application
Using WAVE addon, under structural heading
6 very low contrast errors on the contrast tab of WAVE addon
Turn off your CSS - I'm using the Web Developer Firefox addon - SVG icons are HUGE!
View source to see lack of alt text
Click on form label, associated form input isn't focussed.

- What is the expected behavior?

Tab button takes user in order through the interactive elements on the page
User can navigate application with screenreader
Single h1 on a page
Sufficient color contrast to make text legible
Application should still be usable with CSS turned off
Visually impaired users have alternative text for images and icons
Click on form label, associated input is focussed.

- Please mention your versions where applicable.

Netlify CMS version: ^1.0.0
Browser version: Firefox 59.0.2

Node.JS version: v8.9.1
Operating System: MacOS 10.13.4
- Please link or paste your config.yml below if applicable.

Thanks for putting this together, Jake!

I would argue a lot of these still apply in later versions of the CMS. For example,

Tab order is a bit awry
Use HTML elements, not divs: eg main, aside, button, nav, list items for posts
Match form labels with same id as form inputs. This one needs the most work.

Form elements when adding a kitchen sink item do not have any labeled elements at all. That area is, at the moment, unable to be understood by a screen reader because buttons are not labeled and edit fields are also not labeled.
I am unable to tell which edit field does not have a label because the ones in the Kitchen sink section of the quick add menu are too numerous to list.

Describe the bug
The "Login with Netlify" button does not have sufficient contrast.
Error: Elements must have
sufficient
 color contrast (https://dequeuniversity.com/rules/axe/4.3/color-contrast?application=axeAPI)
10:53:25 PM:  ├── color-contrast
10:53:25 PM:  ├── #nc-root > div > section > button
10:53:25 PM:  └── <button class="css-tu1it6-LoginButton-button-default-gray-disabled e4hp3ji4">Login with Netlify Identity</button>

To Reproduce

Add Netlify CMS to any site
Add  package = "@netlify/plugin-a11y" to nelify.toml
Deploy site to Netlify

Expected behavior
netlify-cms should pass @netlify/plugin-a11y test 😅
Screenshots

Applicable Versions:

Netlify CMS version: 2.0.0
Git provider: GitHub
OS: Windows 10
Browser version [e.g. chrome 22, safari 11]

Node.JS version:

CMS configuration
N/A
Additional context
originally posted: https://github.com/netlify/netlify-cms/issues/6061
Tips for solving

Also I think 8a14937fcc7167ae5a7960a692e8054c/packages/netlify-cms-ui-default/src/AuthenticationPage.js#L94 it is a good place to start

https://github.com/netlify/netlify-cms/issues/6061#issuecomment-997815399

@erezrokah Can I start working on it?

Hi @imskr, since this issue encapsulates multiple 11ty issues, I would say that it's better if you pick on of these:
https://github.com/netlify/netlify-cms/issues/1333#issue-319166205
and comment in the issue, which one you'll be working on.
That way, other people can contribute to other parts of the issue.

Hi @imskr, since this issue encapsulates multiple 11ty issues, I would say that it's better if you pick on of these: #1333 (comment)
and comment in the issue, which one you'll be working on.
That way, other people can contribute to other parts of the issue.

I want to work on color contrast issue as mentioned by @jsjoeio

@imskr I submitted a PR for the one I mentioned but I'm sure there are other color contrast issues you could fix!


Version2 validation

This is a basic but rather useful diagnostic that compares two datasets (control and experiment) for any number of variables (derived or not) and can use any number of OBS/obs4mips etc etc files; it does lat-lon (2d) differences, zonal mean on lat, zonal mean on lon, and I will put in seasons as well -- I am doing some work for a few UKESM guys and realized we actually need one diag like this - basic but that actually plots the comparison between two models with obs in as well. I still need to put the seasons and then ask you guys for a review (prob tomorrow) :grin:

you guys have any clue why the tests are failing and they should not (they are not failing on my machine nor on Jasmin)

Yes, you have to select the non-interactive matplotlib backend by calling matplotlib.use('Agg') when using matplotlib, else it will try to load graphics drivers for interactively displaying stuff, but those are not installed on machines without graphics hardware, like the machines used by Circle CI. Importing iris.quickplot probably means you are making use of matplotlib's pylab, matplotlib.pyplot, or matplotlib.backends, so to solve the issue, you should first import matplotlib and select the non-interactive backend and then import  iris.quickplot and any other modules that make use of matplotlib.

guys @bouweandela @mattiarighi I am done with this and it works nicely and plots lots of plots (just what the UKESM-ers want) -- could you guys pls have a look and approve/reject/machinegun the code with a 50cal please? :grin:

Can you do something about the code duplication reported by Codacy? Just click on the New duplication tab to see a report. It looks like the functions apply_supermeans, and get_all_datasets are the same as in esmvaltool/diag_scripts/autoassess/autoassess_radiation_rms.py.

Hey Bouwe, indeed they are since I based the code on that diag only I went
on to change and add a lot but I wouldn't want to import anything from that
or even create a common module for them since these are quite separate
diagnostics. What would you recommend in this case? Cheers for checking,
man, sorry, quite a bit of code
On Fri, 10 Aug 2018, 08:49 Bouwe Andela<EMAIL_ADDRESS>wrote:

Can you do something about the code duplication reported by Codacy? Just
click on the New duplication tab to see a report. It looks like the
functions apply_supermeans, and get_all_datasets are the same as in
esmvaltool/diag_scripts/autoassess/autoassess_radiation_rms.py.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ESMValGroup/ESMValTool/pull/562#issuecomment-412004887,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbpCo13PSVI9FCvmZXhS9XE0OxmmozIXks5uPTsTgaJpZM4V0Qzd
.

Do you think these are functions you will use in more diagnostics? If no, I would recommend leaving it for now, if yes, I would recommend creating a module in the esmvaltool/diag_scripts/shared directory for those functions and importing from there.

I may actually put the supermeans in preprocessor/area pp since that's
something that is used quite often, but the get all datasets is a bit
obscure and will just leave it as is, cheers, dude
Dr Valeriu Predoi.
Computational scientist
NCAS-CMS
University of Reading
Department of Meteorology
Reading RG6 6BB
United Kingdom
On Fri, 10 Aug 2018, 09:03 Bouwe Andela<EMAIL_ADDRESS>wrote:

Do you think these are functions you will use in more diagnostics? If no,
I would recommend leaving it for now, if yes, I would recommend creating a
module in the esmvaltool/diag_scripts/shared directory for those functions
and importing from there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ESMValGroup/ESMValTool/pull/562#issuecomment-412008164,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbpCo0lkAjdoSkHKUKgYsHzZ_hY-ty9Oks5uPT5DgaJpZM4V0Qzd
.

@bouweandela ma man, I actually removed the duplication and moreover I have introduced two funcs - one in _time_area.pp and the other in shared._base.py - they both come in handy when dealing with these CONTROL, EXPERIMENT and OBS grouping of datasets. Have a looksy, sir, but the code is in essence the same only with no duplication :grin:

Hi V.,
Thanks for removing the duplication. It would be better if you put those two functions in a new file esmvaltool/diag_scripts/shared/validation.py though. That way we keep the files small and relevant.
It should be possible to use preprocessor functions from a recipe,  but apply_supermeans would need quite a bit of work before that is possible, so I think it should not be in the _time_area preprocessor module at this stage, unless you believe it is worth it to do that work.
The function get_control_exper_obs is not quite as generic as the other functions in the esmvaltool/diag_scripts/shared._base.py file, so I think it would be better to put it in it's own file in the shared directory.

sounds like a good plan @bouweandela lemme do it now

hey @bouweandela cheers for the comments dude, I plugged them all in then I reverted the check on seasons since that fails because that one is a dict key, and iris handling is poor - anyway, it works nicely as it was before. Can we merge this in?

Hi V.,
I noticed I made a small mistake in the comment, it should be coords instead of coord, but then it works fine, for me, i.e.:
if not cube.coords('clim_season'):
    iris.coord_categorisation.add_season(cube, 'time', name='clim_season')

Or do you have some other problem, that I'm not seeing? Can you explain a bit more?

hey Bouwe man @bouweandela - when clim_season doesn't exist in the cube, you can't access cube.coords('clim_season') because that will result in ValueError (the clim_esaon key does not exist in the cube) - so one can search for the coord by if not 'clim_season' in cube.coords() or the try/except but I'd rather do the try/except just in case that coord exists in some weird coord type that is not used but accepts adding and changing

When 'clim_season doesn't exist in the cube, cube.coord('clim_season') will indeed raise a ValueError, but cube.coords('clim_season') will return an empty list.

you, sir, are correct and I am a dumbass :grin: -- just replaced it, let's merge :grinning:

Let @schlunma and @bjoernbroetz test it...

The Codacy complaints are to be ignored? At least the first four of them are caused by the matplotlib.use('Agg') before the other inputs which cannot be changed.

Yeah man, it's annoying but nothing I can do about it :(
On Mon, 13 Aug 2018, 15:58 Björn Brötz<EMAIL_ADDRESS>wrote:

The Codacy complaints are to be ignored? At least the first four of them
are caused by the matplotlib.use('Agg') before the other inputs which
cannot be changed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ESMValGroup/ESMValTool/pull/562#issuecomment-412548311,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbpCo_KbQH5sAfYOPJAkpLpWAg24xK1Jks5uQZQagaJpZM4V0Qzd
.

Hi V., the tests (your validation recipe and the short version of the perfmetrics) were successfull :+1:.
Besides the last Codacy issue, I have one suggestion: Maybe we should call the zonal_mean_latitude meridional_mean and the zonal_mean_longitude simply zonal_mean. I think "zonal mean" always refers to a mean along the longitudes (see here for example).

All good from my side, too. @schlunma: You can proceed with the merge as soon as you think its ready! Lets swap the assignment

Hey guys, cheers for testing, I completely agree with the name changes, I
not being a scientist have not known about the correct nomenclature. I can
change this as soon as I get in work in an hour or so or if yous wanna do
it before, fire up and merge, not much I can do about the codacy complaints
though
On Tue, 14 Aug 2018, 08:34 Björn Brötz<EMAIL_ADDRESS>wrote:

All good from my side, too. @schlunma https://github.com/schlunma: You
can proceed with the merge as soon as you think its ready! Lets swap the
assignment
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ESMValGroup/ESMValTool/pull/562#issuecomment-412781626,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbpCo-lYzTR7tNsEu27tWROCyy72vmvYks5uQn18gaJpZM4V0Qzd
.

@schlunma @bjoernbroetz gents, made the analysis name replacements, cheers for them suggestions :grin:

done, cheers @bjoernbroetz ! personally, it looks dodgy to me, we should do something about this codacy complaint wrt matplotlib+pyplot+iris

Sorry for tempting you to break everything ...
Now that we are on it. The last remaining Codacy issue we could kill if we would get rid of those two declarations and use the dictionary-items directly instead.
        ctrl_name = ctrl['dataset']
        exper_name = exper['dataset']

Is this an option?

it is now, @bjoernbroetz :grinning:

aah look at those green bubbles, cheers for the suggestions, guys :beer:

done @bouweandela cheers!

Ah, I just realized that the plot names and titles still include zonal_mean_latitude  and zonal_mean_longitude instead of the new names. Could you change this @valeriupredoi? After that the PR is ready to be merged.

very good catch @schlunma cheers, just changed it


What about error recovery?

Consider a grammar such as:
struct expr : list<term, binary_adding_operator> { };
struct statement : seq<expr, one<';'>> { };

(Where both term and binary_adding_operator are appropriately defined.)
Suppose that the input is malformed, e.g.:
x + ;

Note missing operand.  I would want an appropriate error message, (easy enough to put out in control<term>::failure, but I also want to consume the input up to the semicolon, throw it away and continue parsing.  Would that need to be embedded in the grammar something like this?
template <typename R, typename S>
struct recover : sor<R, until<S>> { };
struct semi : one<';'> { }
struct expr : list<recover<term, semi>, semi> { }

(That's not quite what I had in mind, but I wonder if I'm even on the right track here.)
For more context, here's the project I'm working on: https://github.com/skyrich62/AFE

This is interesting, but not something we explored further in the past. I think that error recovery is hard and very specific to your use-case, so I don't see any general way that the PEGTL (or any parser library) can really help with that.
What you have done so far looks like a reasonable approach for languages where, for example, statements always end in a semicolon. I wouldn't use the Control class, though, but instead apply an action to the skipping part. You also might avoid some redundancy, so how about something like:
template<typename T, typename T>
struct skipping : until< T > {};

template<typename R, typename T>
struct recoverable : sor< seq< R, T >, skipping< R, T > > {};

given that the terminator T can never occur inside the rule R. Just attach the action to R or seq< R, T > as usual, and attach a generic action to skipping< R, T > which emits a diagnostic for the user (plus setting some error flag for the final success/failure of the parsing run). The R is part of the template so you can emit a more helpful error message. (The recoverable template might even be a templated type alias (using recoverable =) if you don't plan to attach an action to it). I think you are on the right track, it is "just" a question of how to arrange a few helper template and how you language is constructed so that error recovery is really helpful to the user.

Sounds good.  I didn't expect the PEGTL to have a generic response to error recovery; I just wanted to know if you guys had thought about methodology.
Thanks for the suggestion, I'll give it a whirl. :)

We'd be very interested in seeing what you come up with, for future reference, and who knows, perhaps something generic can be found, if not concrete code then at least an idea that can be documented...

Hi,
I've played around with it a bit, and created a repository with my experiments:
https://github.com/skyrich62/toy-calc
I'd appreciate any feedback. :)

@skyrich62 I just committed a small experiment on my own, based on some of your ideas. I am too tired to write more now, but I'll try to give more feedback in the next days.

"Get some rest, if you haven't got your health, you haven't got anything." -- Count Rugen, The Princess Bride


[5.0] Add helper function registering array of controllers

I think this might be useful.

No, it's a bad idea. Would mess up everything. You should not be using hacks like this.


feat: register multiple commands

Makes it easier to register multiple commands at once.

I think it would be better if you added the vararg to the main register method instead of making an overload.

I think it would be better if you added the vararg to the main register method instead of making an overload.
Maybe, but I really don't want to potentially mess with someones reflection garbage for a simple QoL change.

im fine with the overload


Make all Render method overrides sealed

At some point we might want to change the way built-in primitives are rasterized.

What @angelsix does in his youtube tutorial wouldn't work anymore? https://github.com/angelsix/youtube/blob/develop/Avalonia/AvaloniaLoudnessMeter/AvaloniaLoudnessMeter/Views/MainView.axaml.cs

There are no Render method overrides in UserControl's inheritance hierarchy, so overriding Render would continue to work there.

Note that it's a bad idea to override Render for the UserControl, since it's a ContentControl which shows its background using ContentPresenter. So if Background property is set, it would be painted on top whatever rendering commands you have in your UserControl.

Inheriting Control and overriding Render on it should be more than enough. UserControl has additional functionality to host a content inside of it which you don't need when you want to have custom draw operations from the Render method.


Beacon operation validation can raise IndexErrors

What is wrong?
Several block operation validation functions can raise IndexError in addition to a ValidationError e.g. validate_voluntary_exit(), validate_proposer_slashing() when passed an Operation with an invalid validator index.
It is not obvious whether this is a documentation issue (that IndexErrors are an expected result), or a crash-causing bug and DOS vector:

Test handlers have IndexError as an expected exception type: https://github.com/ethereum/trinity/blob/84381f4fe19b18dddf2d2b210bd0780038728acd/eth2/beacon/tools/fixtures/test_types/operations.py#L135-L146
Block importing doesn't appear to handle IndexErrors: https://github.com/ethereum/trinity/blob/84381f4fe19b18dddf2d2b210bd0780038728acd/trinity/sync/beacon/chain.py#L158

How can it be fixed
Catch IndexErrors immediately and wrap them with a ValidationError. This is clearer and avoids potential bugs from forgetting to catch IndexErrors.
How it was found
Discovered via beacon-fuzz (initial testing of proposer_slashing fuzzer).
Triggering case: proposer_slashing-crash-35a9d8e810ef1c20f057ee4e6aa8a927dc2ed6dc with the following beacon_states
Or the pre-processed input propslash_preprocessed_indexerror.ssz can be directly passed to the trinity harness

Refer this PR: https://github.com/ethereum/trinity/pull/1498


Subtitle editor: change length of subtitle segment in the timeline

It would be nice if a user could adjust the length of a subtitle segment in the timeline using drag and drop. More precisely the user would click on the end (or the start) of the segment in the timeline and drag it to the desired position:

I've just noticed that I can already move the start of the segment by dragging and dropping (but not the end as described in the screenshot above).
The "<" sign indicating this possibility is only visible in the "light" mode:

It would be great if the design of this sign could be changed so that it is also visible in "dark" mode.


Add binary cross-compilation

https://dave.cheney.net/2015/08/22/cross-compilation-with-go-1-5
Should be as easy as making a script that iterates over OS/ARCH and generates binaries for each one.

You could also consider using gox.
It is an easy to use golang cross-compile build tool.

Very nice suggestion, thank you! I will check that out next week-end :)

Since we rely on curl bindings, it seems unlikely that it would ever be simple to cross-compile Cameradar as it is. The solution would be having a Dockerfile in charge of building the binary with the curl library for each OS/arch but the ratio work/reward seems quite low.
Will close this for now.


[turborepo/docs]: "Publishing Packages" CJS-only advice is outdated

Which project is this feature idea for?
Turborepo
Describe the feature you'd like to request

In this guide, we'll be using a bundler to bundle a package to CommonJS, the most commonly used format used on npm.

It's true that CJS is the most common overall, but modern packages tend to be dual-packaged, and TypeScript users tend to write import, which means they will choose the ESM export if it is available, and get the gains of doing so.
The only change that would need to be made to the instructions:
-    "build": "tsup src/index.ts --format cjs --dts"
+    "build": "tsup src/index.ts --format cjs,esm --dts"

  "main": "./dist/index.js",
+ "module": "./dist/index.mjs",
  "types": "./dist/index.d.ts"

Describe the solution you'd like
It's of course a matter of opinion, but IMO ESM is the future and is superior for users. I personally even prefer a more extreme approach of shipping ESM-only like sindre, but dual-package seems to be the more common/moderate / non-breaking approach so I suggest that.
Describe alternatives you've considered
Alternative: leaving as-is. The upside of packaging only a single format is that there is no dual-package hazard.
I wanted to get an opinion before submitting a PR.

I'm in favor of this. Would even love if we could recommend straight esm, but that would probably cause more issues for us to sort out - so I think the hybrid is the best approach, thanks for the suggestion!


offAllNamed does not work as expected

Flutter Version:3.13.1
Getx Version:5.0.0-release-candidate-4
Suppose there are already 3 pages, page1, page2, page3.
Expectation: offAllNamed(page1)=>page1,page2,page3 close=>Open new page1
But the result obtained in flutter 3.13.1 is offAllNamed(page1)=>page2,page3 close=>return to page1, page1 does not rebuild(including controller)
flutter3.10, everything works fine.

flutter: 3.13.9
getx: 5.0.0-release-candidate-5 Have the same problem
getx: 4.6.6 it works fine


[7.x] Update Authorizable contract parameter names

There's a parameter mismatch found by Psalm in my new Laravel project. Base User class is using trait Illuminate\Contracts\Auth\Access\Gate\Authorizable with method can with signature iterable|string $abilities; interface is defined as string $ability. Different parameter naming could be a problem for named parameters in PHP 8.0. Inconsistent type hints could be a problem for mocking/extending/testing/whatever.
Error from Psalm:
ERROR: ParamNameMismatch - app/User.php:10:7 - Argument 1 of Illuminate\Foundation\Auth\Access\Authorizable::can has wrong name $abilities, expecting $ability as defined by Illuminate\Contracts\Auth\Access\Authorizable::can (see https://psalm.dev/230)

Any classes that implement this also need to be updated.

Any classes that implement this also need to be updated.

@GrahamCampbell right now, Illuminate\Foundation\Auth\User is the only one.


Don't pad empty lines

This is an attempt at fixing https://github.com/folke/lazy.nvim/issues/766.

I'm not 100% sure this covers it, but it seemed right after testing. The idea is to avoid adding the padding to otherwise empty lines.

awesome, thanks!

My pleasure, thanks for the great plugin! :)


posh-git is also on Scoop

System Details

posh-git version/path:
PowerShell version: 5.1
Git version:
Operating system name and version: Windows 10

Issue Description
The readme mentions Chocolatey, but its also on Scoop.
scoop bucket add extras
scoop install posh-git

It can't find a manifest if i try to install trough scoop though.

Scoop and Chocolatey are at version <IP_ADDRESS> - isn't that out of date-ish?
It can really hurt to have outdated packages through these tools.
PS: it wouldn't hurt to add it to the readme

Yeah, that is out of date except that 1.0 is still in beta (has been for over a year).  Ideally, we'd publish another beta of 1.0 and then do a 1.0 release.  But that's up to @dahlbyk.

Scoop and Chocolatey are at version <IP_ADDRESS> - isn't that out of date-ish?

Can't speak to Scoop, but I haven't decided for sure what the plan is for Chocolatey with 1.0: #328, #473.
I'm fine with listing Scoop as a distribution mechanism, if someone wants to submit instructions.

@dahlbyk Just submitted instructions via this PR: https://github.com/dahlbyk/posh-git/pull/763
Regarding availability of official build vs beta releases, I've observed that Scoop treats the project as a different one with a -beta appended, e.g posh-git-beta. Hence we could offer it in Scoop without overriding the official release. That gist is a Scoop manifest example for posh-git 1.0.0-beta4. I might offer that one to the scoop-extras bucket so it's available to everyone interested in the beta, if you are okay with that.


Change CRC library

CRC was incorrect due to the default setting in the library not being correct by default. found solution in the issues tab and confirmed successful firmware example update using the provided pre compiled app.

Created another PR: https://github.com/vovagorodok/ArduinoBleOTA/pull/28


Expected indexing speed?

Trying this out for the first time and for a Rails app with 25-30 dependencies the index is looking to take about an hour. Inspecting lsof and the open file handles it only seems to be getting around 3-5 files parsed per second.
Is there something I'm missing here, or some weird performance bug I might be running into? This is currently running Ruby 3.2 and an M2 Pro Mac Mini. One core is pegged at 100%.

Ok, running the above test and things are done in less than 10 seconds. Also confirmed that RubyIndexer.configuration.indexables.map(&:full_path) gives me the list of files that I'd expect it to be indexing, and it does - just short of 10,000 entries.
Can you recommend a next step as far as seeing why running the language server via the executable might behave differently?

Try temporarily disabling all other Neovim plugins in case something is clashing.

I clearly didn't do a good enough job the first time I tried disabling everything but the relevant neovim LSP built-ins, this is my fault. I ended up narrowing it down to the presence of this line (and something to do with ray-x/lsp_signature.
require("lsp_signature").on_attach({})

Sorry to have wasted both of your time on this one.
If anyone else has a similar issue, I was debugging the RPC requests to the language server and this one seemed to trigger it:
{
    id = 3,
    jsonrpc = "2.0",
    method = "textDocument/signatureHelp",
    params = {
        position = {character = 1, line = 1},
        textDocument = {uri = "file:///Users/taylor/Code/app/config/application.rb"}
    }
}

At this point it's not clear to me what is causing the slowdown, but there aren't any extra RPC requests being sent that I can see causing a slowdown.


Add option to customise data table sorting icon(#91801)

Adds option to add customized icon in place of default sorting icon.
fixes #91801

@danagbemava-nc

Hi @dheerajv09, thank you for your contribution! I think this is great, but what do you think about making the sorting icon a Widget? This would cover a wider range of follow up requests for changing the sorting icon's properties rather than only limiting it to an Icon.

Hi @dheerajv09, thank you for your contribution! I think this is great, but what do you think about making the sorting icon a Widget? This would cover a wider range of follow up requests for changing the sorting icon's properties rather than only limiting it to an Icon.

That Sounds good, will give more customization.

Hi, @Renzo-Olivares , can you tell what's all the failed check error means??.

Looks like it might be unrelated. Try syncing your PR up to master.

@dheerajv09 Will you be able to continue working on this? It's totally fine if not; we can take it over (though it might take longer). I'm just asking because I'm trying to clean out our review queue.

I'm going to close this PR for now to get it off our review queue, but please don't hesitate to submit a new PR if you have the time to get back to it. In the meantime I'll mention this PR on the issue so that anyone else who wants to pick it up can do so. Thanks again for your contribution!


Phases can take multiple inputs as tables, and these are available in context.extra_inputs

Closed by #76


kubectl-view-allocations: init at 0.16.3

Description of changes
kubectl-view-allocations is a kubectl plugin that show resource utilization across a kubernetes cluster.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.


Can we have support for the reqwest crate?

Thanks for making solana-playground, it is super helpful, especially for people new to Rust and Solana development!
I want to try and make a GET call to an API inside of my solana program, and I see that I need reqwest in order to do that. I don't see reqwest as a supported crate in your list of supported crates.
I apologize if this is actually an unreasonable request, I am new to Rust and Solana and I am not sure what is possible.

Thank you!
We can't have support for reqwest as there is no internet access inside Solana programs. You can see what's restricted from here and any crate that makes use of these features are therefore not allowed.
No need to apologize, we are all learning!


DirectoryStructure for Mirror Backups

I'd like to be able to set up a Mirror Directory Structure similar to the functionality of Directory Structure, but separate, so I can set a different structure.
At the moment I do log shipping via SQLBACKUP and I have it set up to go to a share and then "...<DATABASE>_LS"  I would like to be able to do the same, creating a COPYTO/Mirror using the Maintenance Solution to simplify things.

I'd like to be able to set up a Mirror Directory Structure similar to the functionality of Directory Structure, but separate, so I can set a different structure.

Today the directory structure is the same for the @Directory, @MirrorDirectory, @URL and @MirrorURL.

I do vote for this feature. Though for silly reason, it's nice to have control there.

Then I would need new parameters for @MirrorDirectoryStructure and @MirrorAvailabilityGroupDirectoryStructure.


No mouse clicks nor scroll wheel

Have it all moving the mouse, and a magic green button for the space bar (found by mistake)... problem is that I have no clicky clicks?

Could also be worth doing a pull request from the original so to include the actual WebOS app...

The click event is interpreted as the enter key (ASCII 0x0d).
Scroll is not detected at all.

Also, note from the original Windows project:

After connecting, you can move the remote to move the cursor, use the red/green buttons for left/right click, orange/blue for enter/escape and play/pause for media play/pause.


hyper::header::ContentType::json() should specify UTF-8

hyper::header::ContentType::json() creates the Content-Type string application/json. In order to get the web browser to interpret my json correctly, it should be application/json; charset=utf-8.

I can't replicate that problem. Could you tell what browser you're using, and what code you're using?
Here's my test gist, and I was using FireFox 55.0.3.
Also, this issue doesn't seem to be directly related to Hyper itself noted here, but to the mime crate. However, I don't think Hyper should explicitly specify utf-8, since I believe the default already is: RFC 4627 Section 3.

JSON by default is UTF-8. Additionally, the registered media type with the IANA is application/json, with no charset parameters.


KeyError: 'shortName'

Hi, this error occurred when I was using the latest version 0.3, how should I solve it, thanks.
tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cudart64_110.dll
Ticker: GOOG
Start Date: 2019-08-07
Validation Date: 2019-08-07
Test Run Folder: GOOG_20230202_97122126a51ff4356c4231f245a330d2
Traceback (most recent call last):
File "stock_prediction_deep_learning.py", line 94, in
train_LSTM_network(stock_prediction)
File "stock_prediction_deep_learning.py", line 32, in train_LSTM_network
plotter = Plotter(True, stock.get_project_folder(), data.get_stock_short_name(), data.get_stock_currency(), stock.get_ticker())
File "C:\stock-prediction-deep-neural-learning-master_ver0.3\stock_prediction_numpy.py", line 39, in get_stock_short_name
return self._sec.info['shortName']
KeyError: 'shortName'

have you tried using the notebook instead?


breakfix: Caddy unable to load plugin on startup while using Google OAuth 2.0

Describe the issue
Hello!
We've been using this plugin with Caddy to use Google SSO and authenticate on a internal page but recently we've noticed the authentification stopped working. Initially, Caddy was still up but we were getting the "Unauthorized" page when going through the SSO flow
While investigating I tried restarting our caddy service and noticed that at this point Waddy is not starting at all correctly anymore, with the following error logs:
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"info","ts":1692966259.5906324,"msg":"using provided configuration","config_file":"/etc/caddy/Caddyfile","config_adapter":"caddyfile"}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"warn","ts":1692966259.595415,"msg":"Caddyfile input is not formatted; run 'caddy fmt --overwrite' to fix inconsistencies","adapter":"caddyfile","file":"/etc/caddy/Caddyfile","line":9}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"info","ts":1692966259.598708,"logger":"admin","msg":"admin endpoint started","address":"localhost:2019","enforce_origin":false,"origins":["//localhost:2019","//[::1]:2019","//<IP_ADDRESS>:2019"]}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"info","ts":1692966259.599086,"logger":"http.auto_https","msg":"server is listening only on the HTTPS port but has no TLS connection policies; adding one to enable TLS","server_name":"srv0","https_port":443}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"info","ts":1692966259.5991147,"logger":"http.auto_https","msg":"enabling automatic HTTP->HTTPS redirects","server_name":"srv0"}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"warn","ts":1692966259.5991294,"logger":"http.auto_https","msg":"server is listening only on the HTTP port, so no automatic HTTPS will be applied to this server","server_name":"srv1","http_port":80}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"info","ts":1692966259.5994086,"logger":"security","msg":"provisioning app instance","app":"security"}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"info","ts":1692966259.6013534,"logger":"tls.cache.maintenance","msg":"started background certificate maintenance","cache":"0xc00095f480"}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"error","ts":1692966259.7911687,"logger":"security","msg":"failed provisioning app server instance","app":"security","error":"server initialization failed: failed configuring identity provider: failed to fetch jwt keys for OAuth 2.0 authorization server: invalid character '<' looking for beginning of value"}
Aug 25 14:24:19 int-docs caddy[7688]: {"level":"info","ts":1692966259.7914274,"logger":"tls.cache.maintenance","msg":"stopped background certificate maintenance","cache":"0xc00095f480"}
Aug 25 14:24:19 int-docs caddy[7688]: Error: loading initial config: loading new config: loading http app module: provision http: server srv0: setting up route handlers: route 0: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes: route 0: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes: route 0: loading handler modules: position 0: loading module 'authenticator': provision http.handlers.authenticator: loading security app module: provision security: server initialization failed: failed configuring identity provider: failed to fetch jwt keys for OAuth 2.0 authorization server: invalid character '<' looking for beginning of value

Initially we were running Caddy version 2.5.2 and caddy-security version 1.1.14, but we also tried updating to the latest versions (Caddy 2.7.4 and caddy-security 1.1.19), but the startup error message is exactly the same
I've triple-checked that nothing on our Google Cloud console configuration has changed and our credentials are still valid and enabled, and nothing else changed on our side
Any help would be appreciated, as I couldn't find anything related in my research
Configuration
/etc/caddy/Caddyfile
# The Caddyfile is an easy way to configure your Caddy web server.
#
# https://caddyserver.com/docs/caddyfile

# As an alternative to editing the above site block, you can add your own site
# block files in the Caddyfile.d directory, and they will be included as long
# as they use the .caddyfile extension.
import Caddyfile.d/*.caddyfile

/etc/caddy/Caddyfile.d/auth.caddyfile
{
	order authenticate before respond
	order authorize before basicauth

	security {
		oauth identity provider google {
			realm google
			driver google
			client_id XXXXXXXXXXXX
			client_secret XXXXXXXXXXXX
			scopes openid email profile
		}

		authentication portal myportal-ui.surfly.com {
			crypto default token lifetime 3600
			crypto key sign-verify {env.JWT_SHARED_KEY}
			enable identity provider google
			cookie domain ui.surfly.com

			transform user {
				match realm google
				action add role authp/user
			}

			transform user {
				match realm google
				match email<EMAIL_ADDRESS>				action add role authp/admin
			}
		}

		authorization policy mypolicy-ui.surfly.com {
			set auth url https://auth.ui.surfly.com/oauth2/google/authorization-code-callback
			crypto key verify {env.JWT_SHARED_KEY}
			allow roles authp/admin authp/user
			validate bearer header
			inject headers with claims
		}
	}
}

auth.ui.surfly.com {
	authenticate with myportal-ui.surfly.com
}

/etc/caddy/Caddyfile.d/ui.caddyfile
ui.surfly.com {
    authorize with mypolicy-ui.surfly.com

    file_server * {
        root /web-data/suimono
    }
    log {
        format console
    }
}

Version Information
Image running Caddy 2.5.2
http.authentication.providers.authorizer v1.1.14
http.handlers.authenticator v1.1.14
http.handlers.trace v1.1.10
security v1.1.14

Image running Caddy 2.7.4
http.authentication.providers.authorizer v1.1.19
http.handlers.authenticator v1.1.19
http.handlers.trace v1.1.11
security v1.1.19

Expected behavior
Expected caddy to load the plugin correctly and authenticate correctly using Google OAuth 2.0
Additional context
We run caddy using a podman image that's custom-built to add the caddy-security plugin. This is the Containerfile used:
ARG CADDY_VERSION
ARG CADDY_SECURITY_VERSION
FROM docker.io/library/caddy:${CADDY_VERSION}-builder AS builder
RUN xcaddy build --with github.com/greenpau/caddy-security@${CADDY_SECURITY_VERSION} --with github.com/greenpau/caddy-trace

ARG CADDY_VERSION
FROM docker.io/library/caddy:${CADDY_VERSION}
COPY --from=builder /usr/bin/caddy /usr/bin/caddy

We use auto-generated Let's Encrypt certificates through Caddy, so that's why we don't have a tls section and/or certificates configured

Hi,
We are experiencing the same problem since this morning.
We did not change anything in our configuration, we did not restart caddy and also we didn't do any changes on the google side. We tried to issue a new secret but this also didn't help. Neither did updating caddy to the latest version.
Dec 11 08:45:02 mavenproxy caddy[2550265]: Error: loading initial config: loading new config:   loading http app module: provision http: server srv0: setting up route handlers:   route 1: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes:   route 3: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes:   route 0: loading handler modules: position 0: loading module 'authentication': provision http.handlers.authentication:   loading authentication providers: module name 'authorizer': provision http.authentication.providers.authorizer:   loading security app module: provision security: server initialization failed: failed configuring identity provider:   failed to fetch jwt keys for OAuth 2.0 authorization server: invalid character '<' looking for beginning of value
@lucasbaile How did you solve the issue?

Hi,
We are experiencing the same problem since this morning.
We did not change anything in our configuration, we did not restart caddy and also we didn't do any changes on the google side. We tried to issue a new secret but this also didn't help. Neither did updating caddy to the latest version.
Dec 11 08:45:02 mavenproxy caddy[2550265]: Error: loading initial config: loading new config: loading http app module: provision http: server srv0: setting up route handlers: route 1: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes: route 3: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes: route 0: loading handler modules: position 0: loading module 'authentication': provision http.handlers.authentication: loading authentication providers: module name 'authorizer': provision http.authentication.providers.authorizer: loading security app module: provision security: server initialization failed: failed configuring identity provider: failed to fetch jwt keys for OAuth 2.0 authorization server: invalid character '<' looking for beginning of value
@lucasbaile How did you solve the issue?

At the time we had to disable the plugin and resort back to basic auth to restore access
Unfortunately haven't had time to look into it ever since so I can't give a better update on this

@bluelu , enable debug and see what the response actually is.

@greenpau - we tried to restart again with debug enabled in Caddy config, though still can't make sense of it...
Here's the last output before startup fails:
Dec 11 15:22:50 mavenproxy caddy[1484]: {"level":"info","ts":1702304570.0068884,"logger":"security","msg":"provisioning app instance","app":"security"}
Dec 11 15:22:50 mavenproxy caddy[1484]: {"level":"debug","ts":1702304570.0621312,"logger":"security","msg":"fetchMetadataURL succeeded","identity_provider_name":"google","metadata":{"authorization_endpoint":"https://accounts.google.com/o/oauth2/v2/auth","claims_supported":["aud","email","email_verified","exp","family_name","given_name","iat","iss","locale","name","picture","sub"],"code_challenge_methods_supported":["plain","S256"],"device_authorization_endpoint":"https://oauth2.googleapis.com/device/code","grant_types_supported":["authorization_code","refresh_token","urn:ietf:params:oauth:grant-type:device_code","urn:ietf:params:oauth:grant-type:jwt-bearer"],"id_token_signing_alg_values_supported":["RS256"],"issuer":"https://accounts.google.com","jwks_uri":"https://www.googleapis.com/oauth2/v3/certs","response_types_supported":["code","token","id_token","code token","code id_token","token id_token","code token id_token","none"],"revocation_endpoint":"https://oauth2.googleapis.com/revoke","scopes_supported":["openid","email","profile"],"subject_types_supported":["public"],"token_endpoint":"https://oauth2.googleapis.com/token","token_endpoint_auth_methods_supported":["client_secret_post","client_secret_basic"],"userinfo_endpoint":"https://openidconnect.googleapis.com/v1/userinfo"},"userinfo_endpoint":"https://openidconnect.googleapis.com/v1/userinfo"}
Dec 11 15:22:50 mavenproxy caddy[1484]: {"level":"error","ts":1702304570.189998,"logger":"security","msg":"failed provisioning app server instance","app":"security","error":"server initialization failed: failed configuring identity provider: failed to fetch jwt keys for OAuth 2.0 authorization server: invalid character '<' looking for beginning of value"}
Dec 11 15:22:50 mavenproxy caddy[1484]: {"level":"info","ts":1702304570.190457,"logger":"tls.cache.maintenance","msg":"stopped background certificate maintenance","cache":"0xc000eb9900"}
Dec 11 15:22:50 mavenproxy caddy[1484]: Error: loading initial config: loading new config: loading http app module: provision http: server srv0: setting up route handlers: route 1: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes: route 3: loading handler modules: position 0: loading module 'subroute': provision http.handlers.subroute: setting up subroutes: route 0: loading handler modules: position 0: loading module 'authentication': provision http.handlers.authentication: loading authentication providers: module name 'authorizer': provision http.authentication.providers.authorizer: loading security app module: provision security: server initialization failed: failed configuring identity provider: failed to fetch jwt keys for OAuth 2.0 authorization server: invalid character '<' looking for beginning of value
Dec 11 15:22:50 mavenproxy systemd[1]: caddy.service: Main process exited, code=exited, status=1/FAILURE
Dec 11 15:22:50 mavenproxy systemd[1]: caddy.service: Failed with result 'exit-code'.
Dec 11 15:22:50 mavenproxy systemd[1]: Failed to start Caddy.

@greenpau
@lucasbaile
We installed a fresh copy of caddy on another machine and we didn't have any problems.
We then tried with a backup from 2 days ago (where it was still working) on the same machine and it still didn't work
We copied over the install to another machine with a different ip and it worked without any issues.
So it seems that our IP got blacklisted or rate limited by Google. Also before there was a log entry about being rate limited in the log file before (we lost the log file now since we reinstalled). I will investigate with them
Our login portal is facing the internet but only allowing local domain logins.


(MAINT) update yard dependency

takes the work from #1483 & adds a fix to
doc generation so that it can pass through
doc generation & spec testing.

Build finished.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4403/

@puppetlabs-jenkins retest this please

Build finished.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4406/


DataDogKey Secret not accept Secret which contains JSON format with multiple Secrets.

Expected Behavior
Secret: {
"Other Secret":"",
"Datadog_Secret":"",
}
Input:- Datadog_key Secret Name: "Datadog_Secret"
OR
Provide way to give key in plain format and Datadog construct handled in it its own way.
Actual Behaviour
Currently it will accept only Key inside the secret.
Steps to Reproduce the Problem

Put Datadog Key in Secrets in JSON Format.
Passed ISecret into Datadog Construct.
Deploy app on AWS CDK to check whether Key is loaded or not. (Cloud Watch Logs Helpful Here).

Specifications

Datadog Lambda Layer version: 64
GO version: 1.21.3

Stacktrace
Paste here

Hi @ranayash45,
Thanks for reaching out. As you mentioned, our Datadog Lambda Extension only supports plaintext secrets, not secrets inside of json objects. Therefore, you will need to create a new secret just for your Datadog API key.
Is there a specific reason why this does not work for you or is not preferred?

In applications usually we have multiple secrets to handle and storing Datadog secret separately will not cause any issue.
But as developer we expect that all application related secrets should be stored at single place according to environment such as Testing and Production.
So it will be easy to other developer to find dd-secret under the umbrella of application secret.
it is little tedious to handle one extra secret.

Hi @ranayash45,
Thanks for that detailed explanation. That makes sense that you would want the Datadog secret in with all your other secrets.
I've gone ahead and filed a feature request for you. We'll update this issue with any progress.

What's the status of this ticket?


Add a well-documented way to prevent scrolling down when all lines are already within view

Problem
I don't want users to be able to scroll down when there are fewer lines showed on the screen than can fit on the screen.
For example:

In the example, there are only 9 lines of text in the middle block. About 20 lines will fit in the middle block. As such, the scrollbar should occupy the whole height of the entire Window, with the requested feature enabled.
Solution
I want a way to make the scrollbar basically full-height if all lines fit on the screen (i.e., there's nowhere to scroll). I'm not sure if this belongs within the ScrollbarState type or in the rest of the scrollbar styling/setup.
Alternatives
Maybe something exists? Haven't been able to find it or come up with it though.
Additional context

#965 is an attempt of me to configure something like this but no one seemed to be interested enough to look into it so far (so I haven’t continued working on it).
my approach was to hide the scrollbar in that situation instead of showing it full height.

I'm definitely interested in having this as a feature (apologies of not getting to that review yet)
I'd like to see it designed out a bit so it makes sense to everyone that wants to use it.

Would we want this to be the intended behaviour going forward, or should this be an option? My vote is for intended behaviour, as scrolling past the extents seems "cheap".

Both behaviors are valid. I thought the current over-scroll behavior was a bug when I first started to experiment with it, but it was an intentional approach (there also happened to be some extra complex code that made it difficult to see, which we fixed with a rewrite).
It's worth taking a quick (who am I kidding, long) read of the recent scrollbar PRs for context on this. I can't really summarize them - there's too much back and forth to do so.
https://github.com/ratatui-org/ratatui/pulls?q=is%3Apr+scrollbar+is%3Aclosed


Demo\Review Azure capabilities around Cost Analysis & how this could integrate with current billing

Describe the issue
Azure billing has pre-canned analysis capabilities.  Review filter, export, and scheduling possibilities within these capabilities and this could be coordinated with current AWS billing processes.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context
To review how this could be automated to either export to current CPF billing periodically for processing, or run on schedule simply as an advisement to our CPF Billing Administration on where projects stand according to client project\service budgets and\or agreed MOU's
Definition of done

Identify capabilities available to us and which ones would be most useful
Test any export configuration and scheduling capabilities
Demo findings for meeting discussion with Olena and team
Discover what can be automated
Decide if full export of billing details is necessary ie. for historical records (ie. any limitation on current billing info retention logs in place), OR if a simpler export could be sufficient for budget\MOU comparison and the Billing Administrator uses a static project link to the client's Cost Analysis dashboard can be used
Set up additional tickets in accordance to results found

Covered by other tickets


now , timeout configuration through OkHttpClient  ,can config by Request ?beacuse different requests have different timeouts

Start by telling us what problem you’re trying to solve. Often a solution already exists!
Don’t send pull requests to implement new features without first getting our support. Sometimes we leave features out on purpose to keep the project small.

Thanks for your question. This issue tracker is only for bug reports with test cases and feature requests. Please ask usage questions on Stack Overflow.
https://stackoverflow.com/questions/tagged/okhttp


BREAKING CHANGE, Use consistent naming convention for config options

As discussed in Gitter, I felt the config options should be more consistent in the namespaces used.  For now, the current convention is:
debug - internal properties no user should care about.  Currently just incrementalRenderRegions and maxCellsToRender, though I couldn't find any evidence maxCellsToRender is still being used.
oni - properties that affect Oni features, mostly unrelated to Neovim.  Things like menus, fuzzy finder, etc.
editor - properties that affect the viewable "neovim area" even though they don't affect Neovim itself.  Things like highlights, colors, etc.  This includes the background settings, which had been under a prototype namespace.
I've updated the README.md accordingly.  Let me know if you disagree with any of the changes I made though.

Thanks for bringing some sanity to our config settings! I like that you made sure to explicitly calling this out in the title.. will make it easy for me to see when I put together the release notes. Looks great to me. I'm glad we'll have some standards in place here moving forward.


Typescript and other work

implement typescript
allow for theme switching
further work to be done on the navbar (its not yet complete)
theme styles declared in schema.json file inside theme folder in root directory are not final and are subject to major changes in the future

LGTM.


Unable to segment eye-glasses

Hello Yuval,
Thank you for your project. I am also working on face swap projects. Now I am trying to tackle the problem of face occlusions.
I find the performance on the README page really amazing. However, after installation I find that the performance on my face-swap training images not as good as your demo image. The model is unable to segment eye-glasses.
Here are some examples:

I ran the model by this command:
face_seg_image my_image.jpg -o . -m ../data/face_seg_fcn8s.caffemodel -d ../data/face_seg_fcn8s_deploy.prototxt
where my_image.jpg is a 256x256 image in which the face is aligned via 5 landmarks (eyes, nose and mouth).
Do you have any ideas on this? Or is there anything wrong with my input format?
Thank you in advance.

The model was trained to remove the segmentation of only a limited number of sunglasses and so might fail sometimes. For glasses (with clear glass), the model was trained to include them as part of the face.


Additional ingredients

Added several new ingredients from different food groups/ingredient types.

Removed! Sorry, the duplicates were an accident.


Spike: Vault KV2 support

Description
As a developer, I want to have kv2 secret engine support for Hashicorp vault.
https://www.vaultproject.io/docs/secrets/kv/kv-v2
Acceptance Criteria

Update description and acceptance criteria for #1317
The description should be sufficient enough to start 1317 by another develop

Updates needed to upgrade Vault to kv2 for Fabric. Similar changes will be needed for all platforms

Manually enable separate secret path with v2 vault secrets enable -version=2 -path=secretsv2 kv
Update network.yaml with new path secretsv2 in vault.secret_path
Update platforms/hyperledger-fabric/configuration/roles/setup/vault_kubernetes/templates policy templates to include data like path "{{ vault_secret_path }}/data/***" {
Replace all vault write which creates keys with vault kv put
Update roles setup/get_ambassador_crypto and setup/get_crypto as the yaml output format has changed.
Replace all vault delete which deletes keys with vault kv delete
Command vault kv get -format=yaml will not fail if secrets do not exists, update ansible when conditions accordingly to check if the secret was deleted.


update bootstrap

The add avatar color option will fail until https://github.com/fangpenlin/avataaars/pull/15 is merged.
Thanks


MAINT: Move spline interpolation code to spline.c

This PR moves some of the code in ndimage/ni_interpolation.c to a new file ni_splines.c. The main purpose of the changes are documentary: the new code is profusely documented, and cites a couple of the original articles on which it is based. This is a first step in finding a path forward for #8465, namely making the code more understandable.
Some of the refactorings may have some marginal performance gains, e.g. the calculation of the spline interpolation weights should be faster, since it has less branches.

I read through the changes, looks like a major improvement in readability. I followed how the big chunks of code were moved, but didn't verify all refactoring in detail. I'm inclined to rely on the tests for that and just hit the green button. Is there anything in particular that needs a closer review?

The code is heavily, let's call it inspired, on the reference implementation from Dr. Philippe Thévenaz, a researcher in Prof. Michael Unser's group, so I'm pretty confident the general approach is sound.
Judging by some of the constructs in the existing ndimage code, I'm inclined to think that either the existing ndimage code was also based in an earlier version of the same reference implementation, or they both share a common ancestor.
Let me self-review the code again, and see if I can add a few more tests, and I'll ping here when I'm 100% happy with this.

OK, have added some tests to check that the result of spline filtering is the same you'd get from computing the spline coefficients solving a linear system of equations. If it passes all the CI tests, I'm happy with the state of this, can be merged.

The test failures look unrelated.

Have added the extra test over the other axis.

The failures seem unrelated to this PR.

Thanks Jaime.


ValueError: invalid literal for int() with base 10: 'uptimeRobotMonitorID1'

I am getting this error.  Here is my config.ini (private data changed)
UptimeRobotMainApiKey = xxx

[cachet] // Global cachet status API
CachetApiKey = xxx
CachetUrl = http://status.mysite.com

[uptimeRobotMonitorID1] // This will update ComponentId 1 on the global Cachet
ComponentId = 1

[uptimeRobotMonitorID2] // Still possible to use custom cachet settings
CachetApiKey = xxxx
CachetUrl = http://status.mysite.com
MetricId = 1
ComponentId = 1

This is what I get when I run it
$ python3 update_status.py config.ini --printIds
Traceback (most recent call last):
  File "update_status.py", line 398, in <module>
    main()
  File "update_status.py", line 315, in main
    monitor_dict, uptime_robot_api_key, cachet = parse_config(args.config_file)
  File "update_status.py", line 373, in parse_config
    element_int = int(element)
ValueError: invalid literal for int() with base 10: 'uptimeRobotMonitorID1'

Hey,
Same error, using docker image

I am also using a Docker image but it is not the 2.3.13 formal docker release image, it is a 2.4 version that I have created.

@mnucci @Baptistecottey Any luck already :)?

It is a long time with no answers, but maybe it will help:
For getting all the monitor ID's from UptimeRobot, you need to delete all the sections with the [uptimeRobotMonitorIDXX] from the config.ini BEFORE running
python update_status.py config.ini --printIds!


Update Composer dependencies (2023-12-21-00-15)

Visual regression test passed!
View the visual regression test report


Fix findTransactionsInPool in TestKit

Overview
Fix findTransactionsInPool in TestKi and add a test checking that node.submitTransaction adds transactions to the pool.
Definition of Done

[x] There are no TODOs left in the code
[x] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Public API has Javadoc
[x] Method preconditions are checked and documented in the Javadoc of the method
[x] Changelog is updated if needed (in case of notable or breaking changes)
[x] The continuous integration build passes

Coverage increased (+0.003%) to 86.031% when pulling b27d02473fe8512cec7e7909c36e3626122d9a53 on fix_find_in_pool_tx into 7de13b5725a0975d177761b8d7cc0b7fab46e22d on master.


WFLY-13069 CoarseDatabasePersistenceWebFailoverTestCase fails intermi…

…ttently

connection parameters should not longer have properties intended 'mem' type

Resolves
https://issues.redhat.com/browse/WFLY-13069

hmmm....
https://ci.wildfly.org/viewLog.html?buildId=187810&tab=buildResultsDiv&buildTypeId=WF_PullRequest_WindowsJdk11&branch_WF_PullRequest=pull%2F13017
So it still failed.

Dammit, I ll do it the right way so we don't have to rely on graceful shutdown implementation in H2 AUTO_SERVER mode.


Getting NO dropdown at all.

Hi all,
I am attempting to implement the dropdown but it just won't work.

Home

About Us

Site News

          Community

Chatroom

Forums

      <form class="pull-left" action="">
        <input type="text" placeholder="Search" />
      </form>

          Account

<a href="/profile.php?id=<? echo $_SESSION['id'];?>">Profile

Settings

Messages

Admin

Logout

This is my code and I don't see any reason why it wouldn't work based on the documentation?
Can someone help me?

did you add the javascript?

I added this

Still no luck though!
Any ideas?

Have you added jQuery aswell?

FINALLY got it to work. I was linking to a copy of jQuery I had in my directory and it wasn't working.
Having now linked to Google's version, it worked :).


[Grappler] Add Concat+Cast fusion

For the BF16 graph, we usually find a concat+cast pattern from the feature column to DNN part.
The optimization is for concat(FP32 -> FP32) + cast(FP32 > BF16) and concat(BF16 -> BF16) + cast(BF16 > FP32) to fuse one op.

@aalbersk  Please add api_def, like FusedL2Normalize in tensorflow/core/api_def/base_api/api_def_FusedL2Normalize.pbtxt from https://github.com/alibaba/DeepRec/pull/291/files

@aalbersk Please add api_def, like FusedL2Normalize in tensorflow/core/api_def/base_api/api_def_FusedL2Normalize.pbtxt from https://github.com/alibaba/DeepRec/pull/291/files

done

@aalbersk  The CI Build has an issue in compiling, please see the details: https://github.com/alibaba/DeepRec/pull/314/checks

@aalbersk The CI Build has an issue in compiling, please see the details: https://github.com/alibaba/DeepRec/pull/314/checks
Give you a tip, this is oneDNN patch for exposing some API: https://github.com/changqi1/DeepRec/blob/main/third_party/mkl_dnn/oneDNN-v2.3.2-expose-bf16-tp.patch Like gemm_bf16bf16f32: changqi1/DeepRec-deprecated@2865ded#diff-064f747dfd4fd8e51fdc87b8df6012414c8735c64d72174edd11b6a8037c39d5R208-R232
@marvin-Yu Do you have some comments?

fixed

@liutongxuan  LGTM

@aalbersk There is a bug in the main branch of merged your code. This issue is from GPU UT. So I think you need to enable the optimization only in the CPU path.

@aalbersk There is a bug in the main branch of merged your code. This issue is from GPU UT. So I think you need to enable the optimization only in the CPU path.

Yes. The fix has been added: https://github.com/alibaba/DeepRec/pull/325


NullReferenceException in Start

Hi!
First of all, thank you very much for the plugin, it does exactly what i needed.
However, I have a problem I was unable to solve:
Every time I enable the prefab "GetSocialCapturePreview", I get this Error Message in Console:
NullReferenceException: Object reference not set to an instance of an object
GetSocialSdk.Capture.Scripts.GetSocialCapturePreview.Init () (at Assets/Plugins/GetSocialCapture/Scripts/GetSocialCapturePreview.cs:68)
GetSocialSdk.Capture.Scripts.GetSocialCapturePreview.Start () (at Assets/Plugins/GetSocialCapture/Scripts/GetSocialCapturePreview.cs:101)
In Unity Editor, the app stops due the error, but If I press Play to continue, everything else works just ok. In Android, it just seems to ignore the error.
I have not make any change in Plugin Scripts. If you need any aditional Info please tell me.
Thank you very much,
Greetings.

Hey! Thank you for reporting the issue. Which Unity version do you use?

Hi
I'm using version 2019.2.9f1

Thanks. Could you please try to add if (StoreWorker.Instance.StoredFrames == null) return; to the line 68 of GetSocialCapturePreview.cs in Init() method and check if it helps?

Hi
Thank you very much, this solve the issue. Thanks for the quick support.
Greetings


Support range character range for secret with type token.

We have support for character range like "A-Za-z0-9" to be specified for secrets with type basic for username and password fields.
Provide this support for  secrets with type token.

Tested with

Deploy secret with following Accornfile:

cat Acornfiletoken
secrets: {
    password: {
        type: "token"
        params: {
            length: 10
            characters: "A-Za-z0-9"
        }
}
}

Secrets get created with allowed character range.

% acorn secret reveal testnewsec.password
NAME                  TYPE      KEY       VALUE
testnewsec.password   token     token     Mu5AZ2e96O


Clean up Json test summaries

combine the json summaries for all test suites that ran into a single json
add error handling for when the worker is unreachable and we can't fetch the summary

Also add the compute the number of skips: Forumla: Total tests - (passed tests + failed tests)


Problem with some city/country combinations on some 5.1 devices

Hi,
I wrote regarding the problem on your youtube channel.
The problem is that the app comes with an error if I use eg. European cities/countries. The error text is: Unparseable date: "Sat, 04 Jul 2015 12:00 pm CEST " (at offset 26)
I have tried to run it on a Samsung Galaxy S6 with 5.1 firmware, a Asus TF101 Tablet with custom 5.1 firmware and on a Samsung tab pro with 4.4.2 firmware. The error only show up on the devices with lollipop.
I'm hoping that im giving you the correct logcat output. I have copied it to a text file and placed it on gogle drive:
https://drive.google.com/open?id=0BxTBghgUYk6lRW5iQjcycG5RdTA
Thank you,
Nikolaj Kaa Olsen

This issue occurred due to some initial caching implementation that parsed the date without using the appropriate locale. This is not longer an issue since that caching implementation is not longer being used.


Add CSS support for the AcMenu as sidebar

Hello, I’ve seen in past closed issues that you recommended the simpleNavi plugin to the sidebar, but I noticed that it has a static sidebar instead of a collapse one so I decided to use the AcMenu as sidebar because it offers a collapsable sidebar and it works as expected. It has some bugs rendering the title but I’ve already reported it at https://github.com/tormec/AcMenu/issues/10. It would be great if the CSS from simpleNavi plugin as sidebar also applies at the AcMenu one.

I've never used the AcMenu plugin before, so I'll test it and check if I can apply the style.

Hello @my17560,
Thank you for helping me to set me up your template and make it better for everyone.
I would like to thank you somehow so I decided to work into AcMenu integration with your template, this is how it is:

I'll send a PR to your repo so you could add the compatibility to AcMenu as a sidebar. But notice that to work like the video above it require code changes that I'll PR to the AcMenu repo, but I'm not sure if the plug-in owner will accept them because they are bigger one. But if you are interested I can fork the AcMenu and/or you can provide it with your template.


Retrieve the list of Actions of a Viz

We'd like to be able to retrieve the list of Actions (Worksheet/Dashboard > Actions in Tableau) for a Worksheet/Dashboard in an embedded Workbook:

We're looking for as much information as possible on those actions, like:

Type
Name
Run action on
Target
"Clearing the selection will"
Fields
URL for URL Actions

Thanks !

Hi @florent-guinard - Thank you for opening this issue. Do you mind sharing more about your use case please?

Wow this is a great request, would love to get access to this info!

Thanks @florent-guinard ! @Mrkbingham do you have the same use cases?

@illonage yep!  Somewhat similar, I’d like to be able to read all of the settings that someone has set up inside Tableau so my app can detect the set behavior.  The major config that would be helpful are: Target, Fields and URL (also the href target, e.g. open in new tab)
The URL would help ensure that we’re opening a URL that may already be embedded in our app, if not we send the users over to Tableau Server.
The fields would be helpful for tracking user behavior - we currently track filter events in the applyFilter callback, and would like to continue that.
Lastly the name of the action: we log errors for unexpected behavior, and if we could say “Action [Title] failed” it would help immensely for users when troubleshooting.


[Icon] Migrate to emotion

This PR migrates the Icon component to emotion, with code heavily based upon #24506.  Part of #24405.

[X] I have followed (at least) the PR section of the contributing guide.

Something is off with the fontSize property https://codesandbox.io/s/icons-material-demo-forked-c3t38?file=/demo.js We also have this regression:

This PR - https://deploy-preview-24516--material-ui.netlify.app/components/icons/#font-material-icons
Next - https://next.material-ui.com/components/icons/#font-material-icons

Something is off with the fontSize property https://codesandbox.io/s/icons-material-demo-forked-c3t38?file=/demo.js We also have this regression:

This PR - https://deploy-preview-24516--material-ui.netlify.app/components/icons/#font-material-icons
Next - https://next.material-ui.com/components/icons/#font-material-icons

Ahh, I see what's going on -- it appears that the font size is getting overwritten by the CSS for .material-icons (the baseClassName).  Thus, the fontSize property is essentially just ignored.
I'm guessing we'd probably want to pseudo-class the fontSize property to increase the styling rule's weight?

Ahh, I see what's going on -- it appears that the font size is getting overwritten by the CSS for .material-icons (the baseClassName).  Thus, the fontSize property is essentially just ignored.
I'm guessing we'd probably want to pseudo-class the fontSize property to increase the styling rule's weight?

Ahh, I see what's going on -- it appears that the font size is getting overwritten by the CSS for .material-icons (the baseClassName). Thus, the fontSize property is essentially just ignored.
I'm guessing the preferred option is to pseudo-class the fontSize property to increase the styling rule's weight, making a commit now!

Thanks for taking a look on it! Yes, it appears that this resource - https://fonts.googleapis.com/icon?family=Material+Icons|Material+Icons+Two+Tone is added after the styles coming from emotion, so it wins over our styles. We do not want to increase the specificity for the fontSize, please revert those changes, as the overrides would become hard for them. I'd say we need to change the location of where the  tag for the icons is added (it should be before our styles) cc @oliviertassinari
Let's revert it to the initial implementation (without using class selectors) and add //TODO for fixing the order of the injected styles.

Ahh, I see what's going on -- it appears that the font size is getting overwritten by the CSS for .material-icons (the baseClassName). Thus, the fontSize property is essentially just ignored.
I'm guessing the preferred option is to pseudo-class the fontSize property to increase the styling rule's weight, making a commit now!

Thanks for taking a look on it! Yes, it appears that this resource - https://fonts.googleapis.com/icon?family=Material+Icons|Material+Icons+Two+Tone is added after the styles coming from emotion, so it wins over our styles. We do not want to increase the specificity for the fontSize, please revert those changes, as the overrides would become hard for them. I'd say we need to change the location of where the  tag for the icons is added (it should be before our styles) cc @oliviertassinari
Let's revert it to the initial implementation (without using class selectors) and add //TODO for fixing the order of the injected styles.

Sure thing, revert complete!
At first, I was thinking that it was the added fontSizeMedium class (which was copied from SVGIcon), but that class doesn't seem to be causing any harm (the font size is the same as what's defined on .material-icons).  Would we want to keep that for consistency with SVGIcon, or keep it out since it wasn't originally around?

Sure thing, revert complete!
At first, I was thinking that it was the added fontSizeMedium class (which was copied from SVGIcon), but that class doesn't seem to be causing any harm (the font size is the same as what's defined on .material-icons).  Would we want to keep that for consistency with SVGIcon, or keep it out since it wasn't originally around?

Would we want to keep that for consistency with SVGIcon, or keep it out since it wasn't originally around?

That, plus we want to have the utility classes for the default values as well in the component so that developers can target those and change them if they want to.

Would we want to keep that for consistency with SVGIcon, or keep it out since it wasn't originally around?

That, plus we want to have the utility classes for the default values as well in the component so that developers can target those and change them if they want to.

Looks like the last test failed due to an issue connecting to Argos CI from Circle CI.  Can a peer restart the job?

Looks like the last test failed due to an issue connecting to Argos CI from Circle CI.  Can a peer restart the job?

@mnajdova Interesting issue with the CSS, it looks like an issue with the configuration of the documentation. I wish emotion allowed to configure the injection point as JSS does. It will be resolve once we have migrated everything, then we can remove the prepend: true flag.

@oliviertassinari emotion has the container element https://github.com/emotion-js/emotion/blob/master/packages/cache/README.md#container which means we should be able to specify for the styles to be injected after those, but let's see if we will still need once we don't have the prepend option.

@mnajdova Interesting issue with the CSS, it looks like an issue with the configuration of the documentation. I wish emotion allowed to configure the injection point as JSS does. It will be resolve once we have migrated everything, then we can remove the prepend: true flag.

@oliviertassinari emotion has the container element https://github.com/emotion-js/emotion/blob/master/packages/cache/README.md#container which means we should be able to specify for the styles to be injected after those, but let's see if we will still need once we don't have the prepend option.

@mnajdova As far as I understand the container option, it doesn't help. It's meant for solving cross-window use cases (iframe, popup, shadow-dom). We can only set it to be document.head. Maybe we should report this to the team? With styled-components, it can be solved by creating an empty <style data-styled="active" and putting it in the right position.

@mnajdova As far as I understand the container option, it doesn't help. It's meant for solving cross-window use cases (iframe, popup, shadow-dom). We can only set it to be document.head. Maybe we should report this to the team? With styled-components, it can be solved by creating an empty <style data-styled="active" and putting it in the right position.

@mnajdova As far as I understand the container option, it doesn't help. It's meant for solving cross-window use cases (iframe, popup, shadow-dom). We can only set it to be document.head. Maybe we should report this to the team? With styled-components, it can be solved by creating an empty

@mnajdova As far as I understand the container option, it doesn't help. It's meant for solving cross-window use cases (iframe, popup, shadow-dom). We can only set it to be document.head. Maybe we should report this to the team? With styled-components, it can be solved by creating an empty

Hmm, even if it looks odd - you could create a div in your head and pass that div as the container to us.
Could you point me to JSS docs that describe the insertion point option? How exactly would you configure it in your specific case? I'm interested in seeing what exact elements would have to exist prior to instantiating your Emotion instance etc.

Hmm, even if it looks odd - you could create a div in your head and pass that div as the container to us.
Could you point me to JSS docs that describe the insertion point option? How exactly would you configure it in your specific case? I'm interested in seeing what exact elements would have to exist prior to instantiating your Emotion instance etc.

@Andarist We do the following with JSS:
in the head:
https://github.com/mui-org/material-ui/blob/168b6d9b97aab99550a7d9a920cb53758bb4b980/docs/pages/_document.js#L170-L174
in the configuration:
https://github.com/mui-org/material-ui/blob/11b10cd570f75147bf19dec99f2e26387b7da767/docs/pages/_app.js#L28-L32

@Andarist We do the following with JSS:
in the head:
https://github.com/mui-org/material-ui/blob/168b6d9b97aab99550a7d9a920cb53758bb4b980/docs/pages/_document.js#L170-L174
in the configuration:
https://github.com/mui-org/material-ui/blob/11b10cd570f75147bf19dec99f2e26387b7da767/docs/pages/_app.js#L28-L32

I wasn't super keen on introducing an additional option for this but when I now think amount migrating to Emotion, having different styles etc it makes sense to support something like this (and to deprecate prepend in the process).
Would your team be able to help out implementing this? This would only require adding a new option (probably normalizing it into a common option) and adjusting the _insertTag:
https://github.com/emotion-js/emotion/blob/400c6f72905a4676f852b09405421a7ca8d7a3a2/packages/sheet/src/index.js#L88

I wasn't super keen on introducing an additional option for this but when I now think amount migrating to Emotion, having different styles etc it makes sense to support something like this (and to deprecate prepend in the process).
Would your team be able to help out implementing this? This would only require adding a new option (probably normalizing it into a common option) and adjusting the _insertTag:
https://github.com/emotion-js/emotion/blob/400c6f72905a4676f852b09405421a7ca8d7a3a2/packages/sheet/src/index.js#L88

@Andarist It seems that once we have migrated all the components to emotion, we won't need, for the docs, to have this option. Still, developers might need it, I think that we could consider contributing this back, thanks for the proposal!
Now, I'm not sure it's more important than the Global issue we surfaced a couple of weeks ago (but it seems simpler to fix).

@Andarist It seems that once we have migrated all the components to emotion, we won't need, for the docs, to have this option. Still, developers might need it, I think that we could consider contributing this back, thanks for the proposal!
Now, I'm not sure it's more important than the Global issue we surfaced a couple of weeks ago (but it seems simpler to fix).

Oh ye - definitely the other one might is more important overall but also a little bit more involved. I'm open to discussing how that Global issue should be solved, I would probably handle it myself but lately, I'm just drowning in GitHub notifications and I struggle to keep up with them so I'm not sure when I will have time to sit down to it properly.

Oh ye - definitely the other one might is more important overall but also a little bit more involved. I'm open to discussing how that Global issue should be solved, I would probably handle it myself but lately, I'm just drowning in GitHub notifications and I struggle to keep up with them so I'm not sure when I will have time to sit down to it properly.


Issue when Inference in Batch

Hi authors,
When I try to inference in individual text, it works well. But when I try to infere in batch, I give input as a list, just like this ["abcdefu", "1234567", "9876543"]. There is a bug happened, it called "Incompatible shapes: [1,45,384] vs. [3,384] [Op:Mul]"

How can I fix this?

Hi, I already firgured the issue out myself


style: disallow use of confusing window globals

This will hopefully prevent issues like the one from #201

Wow, that list contains a lot of globals I was not aware of 😨

Yet it still feels like there's some that are missing...


Why set terminal=True when live decreases?

In gym, terminal is True only when lives is 0. But in act() function of GymEnvironment class, it seems that terminal is True when live decrease. It will influence num_game and episode reward.

DeepMind does that when they publishing their Nature paper.


[BUG] along with a copy of the log

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.5.4
What problem did you encounter?
i can download blox trap
Bloxstrap Log
N/A

Update bloxstrap to 2.8.1. You are using an outdated version.

..

Your version of Bloxstrap is out of date
Please update to the latest version. You can install the latest version from here:
https://github.com/bloxstraplabs/bloxstrap/releases/latest

learn to read and we dont care if u need multi instance


Add Exordium

CurseForge link
https://www.curseforge.com/minecraft/mc-mods/exordium
CurseForge Mod Distribution
Allowed
Modrinth link
https://modrinth.com/mod/exordium
Source/other link
No response
Mod file size
121.02 KB
License
All Rights Reserved / ARR
What it does
It improves the fps by setting the gui at a lower framerate,so it can focus more on world rendering.
Why should it be in the modpack
It improves framerate.
Why shouldn't it be in the modpack
It limits the gui at a lower framerate,so it's looks laggy.
Additional details
Nothing

Duplicate of #398


changed prometheus start_http_server to a flask endpoint

start http server was replaced with a flask app that sets up a server at port enabling mtls support.
cert files for local testing

Testing
The flask endpoint was tested with cert files generated locally using curl
curl https://localhost:63512/metrics --cert client-csb.crt --key client-csb.key --cacert ca.crt -kv
output