User Story 1 & 2 notepad

@andreasbueckle, I am starting an idea notepad for us here regarding User Story 1 & 2 UIs. Please capture all feature requests, UI inspiration screenshots, links, and more in this area!

Pretty neat use of button to cite the tool (could link to HRApop DOI once available):

@andreasbueckle:
Last night I contemplated branding for these user story preview/beta tools. Concrete tool names and custom icon designs (see: tool icons) could elevate them when the time comes to revamp the features of these UIs.
Screenshot list of our current user stories in preview/beta mode:

@LibbyUX
Yeah, that might be worth revisiting for US #1-2 especially. Also, a clearer link to HRA US numbers these previews are serving would be good.

@andreasbueckle - We are getting closer to needing official names for these two products as the Atlas Applications design matures. Do you have any ideas off the top?
Atlas Apps Menu Design
Note: This will change. We are adding screenshots to make it more visual instead of having the product icons.

@andreasbueckle Can we add features to already existing products to consolidate our products? Or do you see these two user stories as requiring their own product identity?
HRA Product Line

Figma - Information Architecture: Products

@andreasbueckle - We are getting closer to needing official names for these two products as the Atlas Applications design matures. Do you have any ideas off the top?
Atlas Apps Menu Design
Note: This will change. We are adding screenshots to make it more visual instead of having the product icons.

I like the two suggestions you have on the right. @bherr2 could we rename https://apps.humanatlas.io/us1/ and https://apps.humanatlas.io/us2/ to match those names?

@andreasbueckle Can we add features to already existing products to consolidate our products? Or do you see these two user stories as requiring their own product identity?
HRA Product Line
* Figma - [Information Architecture: Products](https://www.figma.com/board/H3CK8StFdAphCFtFAHAwhM/Information-Architecture?node-id=479-1486&t=OddD7ooQ6GXHbEGc-4)

I think these two user stories need their own identity. These are demo apps that we are documenting with the paper. There will probably be other ways to access the data, and the HRA API also allows the usage of HRApop data for prediction.

Hey Andi, regarding:

I like the two suggestions you have on the right. @bherr2 could we rename https://apps.humanatlas.io/us1/ and https://apps.humanatlas.io/us2/ to match those names?

If they are having their own product identity, they will need strong names, branding, etc. I created the HRA Product Line board in our Figma Design System file to bring order to our HRA internal/external product jargon, naming inconsistencies, and product identities. US1 & US2 can be found in the Emerging Products section.
Can you please analyze the full board in Figma along with the US1 & US2 entries and let me know what you think?


add isapprox method

Codecov Report

Merging #14 (e8d6064) into main (6f918f0) will increase coverage by 0.77%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   82.55%   83.33%   +0.77%
==========================================
  Files           1        1
  Lines          86       96      +10
==========================================
+ Hits           71       80       +9
- Misses         15       16       +1

Files Changed
Coverage Δ

src/Extents.jl
83.33% <92.30%> (+0.77%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


adds a subtree migration iface to kopia

Introdudces the user pn to id migration. Since
onedrive doesn't enumerate folders on backup,
we have to force the migration by introducing a
subtree migrator in kopia.

Does this PR need a docs update or release note?

[ ] :clock1: Yes, but in a later PR

Type of change

[x] :sunflower: Feature

Issue(s)

#2825

Test Plan

[x] :muscle: Manual
[x] :zap: Unit test
[x] :green_heart: E2E

closing in favor of https://github.com/alcionai/corso/pull/3122


T3D don't compile "epoxy/gl.h': No such file"

Hi I just download the new version of torque3d I can't compile the damn thing hahaha
1>C:/Game Develop/Game Engine/T3D/Engine/source/gfx/gl/tGL/tGL.h(26): fatal error C1083: Cannot open include file: 'epoxy/gl.h': No such file or directory
I'm bit lose right now with all the changes but in the commit change I notice this "Merge pull request #1567: Added Epoxy in favor of GLEW" may be something was missing.

Hey John, just to double check, is this windows or linux you're seeing this happen on?

Windows 7

What compiler/IDE are you using? VS2010? VS2015? MinGW?

Oh sorry by lack of information Jeff.
Clean Torque3D
Branch: Development
Windows 7
VS2010
Project Manager: Torque physics, OpenGL Rendering

I've only tested with Visual Studio 2015, running Windows 7 Service Pack 1 w/ Platform Update. Epoxy should still be supported in VS2010.....hmm...

Can you double-check your include paths for the project and that they include the epoxy paths:
(you engine directory)\Engine\lib\epoxy\include
(you engine directory)\Engine\lib\epoxy\src

A clean t3d development branch with a new project and yes the epoxy folder is there with the include files. Are you try to say I need to include in my VS project the epoxy?

@John3 : it look like
http://i.imgur.com/Iy6WiIY.jpg ?

Sorry, i meant in the project settings. You have your project settings where you set the include paths where it checks additional header files and the like.
So just open up the settings for the project(should be r-click on project > properties > C/C++ > General and then the Additional Include Directories field)
Want to see that those did indeed get added to the project when you generated it. That'll tell us where to look next, because it SHOULD be working correctly.

@John3 are you using the project manager? If so you have to use CMake as I never wrote a project manager module for epoxy. Didn't realize folks were still using it. There's a CMake tutorial on the wiki if you run into trouble.

oh, so I need to use cmake then haha, sorry but I was out for a few months I don't know that project manager is not using anymore. Sorry for the confusion guys, yes I now I need to add to the VS project to build... I just opened thinking that was a issue. The new people know PM is not using anymore? If it not a issue sorry, we can close this.

Hmmm ok I use CMake 3.5.2, but still have error with epoxy. I just follow this guide to be sure http://wiki.torque3d.org/coder:compiling-in-windows#toc6 and select OpenGL. I build without OpenGL and it works. With OpenGL I have this error:
http://pastebin.com/2WVnYbn1

Looks like epoxy won't work with VS2010 as it doesn't have stdbool.h which is a C99 header:
http://stackoverflow.com/questions/8548521/trying-to-use-include-stdbool-h-in-vs-2010

Apparently epoxy makes use of stdbool.h, which VS2010 does not support.
So going forward, if you want to make use of OpenGL, you'll want to jump up to 2012, 2013 or 2015, otherwise if you HAVE to use vs2010 for whatever reason, you can just go without OpenGL.

Thanks! I have 2012 already. I use a lot vs2010 for the old versions of torque3d.

@John3 sounds good. Let us know if it works out! :)

Don't work vs2012 either hahaha so.. I suppose 2013 and up, good to know for the next release, is necessary to document this. So from now on CMake and vs2013.

You could create your own stdbool.h ;)

Hahaha forget it dude thanks but this is not for me. All the new people don't need to fight with the engine. So, we can now document this as requirement for the new releases.

Haha, yeah, we'll just figure for VS2013 as the minimum. If people absolutely have to have older than that, they're gunna be clever enough to work out SOME kind of solution :package:
I'll update the wiki page for compiling on windows to mention the VS version req and go ahead and close this ticket.
Thanks for the spot on this, John, we all learned some nifty stuff from this :)


Headers need not be listed in CMakeLists.txt

I think you need them, otherwise a change in the headers may not trigger a rebuild.

Just tested that, and it seems CMake is able to do the right thing, that is, check for changes in all headers in the directory.
It seems to work for headers outside the project too.

Which version of cmake?

From conda-forge, I'm always getting an up-to-date CMake (3.25 today). Which version would you like me to try.
Note that if the only concern is about detecting header changes, this would only impact during development (not in releases).

Can you rebase the PR? I think we should go this way for the whole stack to simplify the CMakeLists.txt. Also it seems the PUBLIC_HEADER property was initially specific to framework libraries on OSX, and works on other platforms for compatibility. Therefore we should not use this feature.


insert_row doesn't move merged cells

While all the styles and content move down after insert_row(0), all the merged cells continue to stay on their places. It makes impossible to insert data into a prepared file.
Before:

After:

Well, I made a rough experiment and it looks like the behavior can be easily patched.
3.times { worksheet.insert_row(15) }

worksheet.merged_cells.each do |cell|
  next unless mc.ref.row_range.min >= 15
  mc.ref.instance_variable_set(:"@row_range", Range.new(mc.ref.row_range.min + 3, mc.ref.row_range.max + 3))
end

The code above turned this:

Into this:

That's what I've been expecting from the gem to do from the beginning. We can make a special RubyXL::MergedCells interface for shifting rows starting from some index and call it within insert_row. What we need to take in count is the fact that when we insert a row, there can be a multirow MergedCell that starts before the insert and ends after it. If happens so I suggest to increase only the max end of the range, leaving the min on its place.

Also I think we should copy merge info from the shifted row. That's exactly what Mac Numbers app does (can't check it in real Excel).

Found extra bug related to insert:
worksheet.insert_row(row_index)
# Here is everything ok:
worksheet[row_index][0].change_contents("some string")
# Here I get validate_worksheet exception:
worksheet[row_index][2].change_contents("some_other_string")

Can't understand what is happening here exactly. Maybe sheet_data index does not get proper updates somewhere during insert_row?
To make my production code work (I need it just for one file), I removed the validation. Everything works 100% ok.

There are many issues stemming from the fact that Excel emulation is far from perfect. So far, the gem is best used for reading from and minor modifications of XLSX files. References throughout the file are not recalculated upon row/column insertion; in particular, formulas certainly do get broken. Proper handling of that stuff requires quite a bit of planning...

@weshatheleopard well, let's think what we can do to update references?

+1 with the issue related to insert and the validate_worksheet exception.
I'm using @killthekitten's fork which addresses my issue. thanks!

This is like plugging a single hole in a colander.
For starters, we need to complete handling the Reference class. Which is a huge task — see:

https://msdn.microsoft.com/en-us/library/ff533995(v=office.12).aspx
http://ewbi.blogs.com/develops/2004/12/excel_formula_p.html
https://office.microsoft.com/en-us/excel-help/overview-of-formulas-HP010081865.aspx#BMusing_references_in_formulas

@weshatheleopard totally fair, thanks for your work here, it's much appreciated.
I certainly don't need an incredibly robust library, and while I can appreciate your dedication to it, I just need to export some data into an xlsx template, which involves adding some rows, but nothing with formulas.
Is that issue pertaining to the Reference class also why the add_hyperlink convenience method is commented out? I forked and played with a bit and can't get it to work (Excel throws an invalid reference upon clicking into the cell).

You see, that's the problem: nobody "needs an incredibly robust library", but everybody needs some particular hole plugged.  My personal policy is to fix the root causes, not the particular manifestations.
Yes, that's the reason. You see, References may be:

Absolute or relative (currently supported);
Referencing a single cell (currently supported);
Referencing a range of cells (currently supported);
Referencing cells on a different worksheet(s) within the same workbook file;
Referencing cells in a different workbook file;
Have a relative (R1C1) format

So I need to be able to support everything, even such complex things as [Expenses]'John''s'!A1:DE1

Yeah, I understand, it's just one of those thankless parts of being a maintainer. Again, I appreciate all of your work were, and I think thats a fine policy. Thanks again!

So for example, the few example of things breaking when you delete row (or column) are:

Merged cells
Selections
Named ranges
Conditional formatting
Formulae
(more stuff that I don't remember off the top of my head)

If I properly implement References, that will automatically fix all these breakages. But doing so is not an easy task, given how much stuff Reference needs to support (see comment above). And I have a primary job which doesn't allow me to just sit down and concentrate on it.

Is there any workout for copying and pasting the specific row maintaining its formats and formulas in specific position using RubyXL gem? Your help will be highly appreciated.


No effect from Fortune

Not sure if this is a bug or just a missing feature or design decision, but there's no effect from enchanting a scythe with fortune and then harvesting with Crop Reaper.  It does work when breaking the crop directly with the Fortune enchanted scythe. Is it possible to add/fix this?

Fixed in v2.1.0+, should be up on CF in a short while.


Test new nav

I think the testing framework and development instructions are in place to test this locally (and much more efficiently!) now.  Let me know if you encounter problems with these: https://github.com/meteor/hexo-theme-meteor#developing


Hive OS Devices select

How to select specific devices for mining TON in Hive OS?
I don't understand this command - danila-miner run -d 0:0 -d 2a:00.0 https://server1.whalestonpool.com YOUR_WALLET
May you explain where and how should i write this one with examples, please?

The developer closed an issue without resolving it.

The developer closed an issue without resolving it.

The developer closed an issue without resolving it.
Because this is the repository of the Watchdog4DanilaMiner application. Not danila-miner.

Check out the documentation for the danila-miner on the pool website

There is no repository of danila-miner, just Watchdog4DanilaMiner. The documentation for the danila-miner on the pool website doesn't work in my Hive OS and have no any examples and poor explanations.

There is no repository of danila-miner, just Watchdog4DanilaMiner. The documentation for the danila-miner on the pool website doesn't work in my Hive OS and have no any examples and poor explanations.

There is no repository of danila-miner, just Watchdog4DanilaMiner. The documentation for the danila-miner on the pool website doesn't work in my Hive OS and have no any examples and poor explanations.

I am not a developer of danila-miner, I can not help you with anything


add programmable transaction info to replay

Description
Adds PTB flag to replay command that when supplied prints out the inputs and the commands of the PTB
Example output 1:

Example output 1 zoomed out command portion:

Example output 2:

Example output 3:

Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

@laura-makdah could you please leave a comment to address @amnn questions/feedback?
it seemed reasonable and maybe you solved it or talked about it but I am not sure I see much, thanks!


YouTube embeds without descriptions show 'undefined'

Should check if it's null :P
Edit: vid: https://www.youtube.com/watch?v=MK6TXMsvgQg

Doesn't show the description fix, but the formatting is much improved.

Looking very nice!
On September 17, 2017 1:36:57 AM PDT, Will Hunt<EMAIL_ADDRESS>wrote:

Closed #43.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/Half-Shot/matrix-appservice-discord/issues/43#event-1252410287


Remove transpiled folder before running transpiler again

This fix is for the case when some files are removed,
to avoid running app with deleted files preserved in
transpiled folder

test this

test this

Codecov Report

Merging #1067 into master will decrease coverage by 0.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1067      +/-   ##
==========================================
- Coverage    93.6%   93.04%   -0.56%
==========================================
  Files          39       39
  Lines        4160     4243      +83
  Branches      460      726     +266
==========================================
+ Hits         3894     3948      +54
- Misses        266      280      +14
- Partials        0       15      +15

Impacted Files
Coverage Δ

browser/pages/install/controller.js
73.66% <0%> (-4.88%)
:arrow_down:

browser/model/devstudio-autoinstall.js
97.29% <0%> (-2.71%)
:arrow_down:

browser/pages/location/controller.js
91.13% <0%> (-2.62%)
:arrow_down:

browser/model/helpers/hash.js
85.29% <0%> (-2.59%)
:arrow_down:

browser/main.js
95.58% <0%> (-1.39%)
:arrow_down:

browser/pages/account/controller.js
82.11% <0%> (-1.33%)
:arrow_down:

browser/model/jbossfusekaraf.js
95.09% <0%> (-0.83%)
:arrow_down:

browser/model/installable-item.js
95.95% <0%> (-0.81%)
:arrow_down:

browser/pages/confirm/controller.js
61.41% <0%> (-0.79%)
:arrow_down:

browser/model/jbossfuse.js
99.23% <0%> (-0.77%)
:arrow_down:

... and 19 more

Continue to review full report at Codecov.

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


Use next-episode.net as input for configure_series not working Max retries exceeded with url

WARNING  configure_series eztv            Error during input plugin text: RequestException: HTTPSConnectionPool(host='next-episode.net', port=443): Max retries exceeded with url: /PAGES/misc/export_calendar?z&u=useruser&k=xxxxxxxxxxx.ics (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x14ec69774be0>: Failed to establish a new connection: [Errno -3] Try again')) 2023-03-25 17:17:21 INFO     configure_series eztv            Did not get any series to generate series configuration
configure_series:
  from:
    text:
      url: "https://next-episode.net/PAGES/misc/export_calendar?z&u=useruser&k=xxxxxxxxxxxxxxxx.ics"
      entry:
        title: ^SUMMARY:(.*) - [0-9](/0-9)+x[0-9](/0-9)+.*$
        url: ^SUMMARY:(.*) - [0-9](/0-9)+x[0-9](/0-9)+.*$

Now I have used this for years but I guess my Premium membership lapsed with the whole PayPal thing.  I signed up for the next-episode.net Premium membership and the Premium works on the website and can get the actual ICS to download without being signed into the next-episode.net website.

Was a Networking issue, needed in extra params --network=container:binhex-delugevpn, this was missing as network in the Docker was set to none.
the title and url above don't work.
Needs to be:
configure_series:
from:
text:
url: "https://next-episode.net/PAGES/misc/export_calendar?z&u=user&k=xxxxxxxxxxxxxxxxxxxx.ics"
entry:
title: ^SUMMARY:([\w\s])-\s(s\d{2})(e\d{2})
url: ^SUMMARY:([\w\s])-\s(s\d{2})(e\d{2})


Tutorial adding command

(I forgot to implement this, but the add command feature is already implement in the latest codebase)


unhandled exception

Hi,
everytime I run the script, I get the following exception on a Windows Server 2012:
Traceback (most recent call last):
  File "edge.py", line 91, in <module>
  File "subprocess.py", line 466, in check_output
  File "subprocess.py", line 571, in run
subprocess.CalledProcessError: Command '['powershell', '(Get-LocalUser -Name $en
v:USERNAME).SID.Value']' returned non-zero exit status 1.
[10784] Failed to execute script 'edge' due to unhandled exception!

Looks like the Powershell module for getting the current username, is not supported in Server 2012.


Create Home and Posts in the navbar

So, that I can return to my principal webpage or the complete posts list.

Ref #56


Add Support for Fantech THOR X9 Macro RGB Gaming Mouse

Is your feature request related to a problem? Please describe.
I just bought a new cheap macro gaming mouse from Fantech called Fantech THOR X9 Macro RGB Gaming Mouse (this is the first time I try a macro mouse). I wanna use the macro feature on linux but luckily there is someone who already made the driver
Describe the solution you'd like
I found this repo for this type of mouse. maybe you guys can try merger it into this repo?
Thanks :3

Since this mouse would require writing a new driver, let's close this until someone volunteers to write one.
For now the application mentioned in the top message can be used instead.


Customer Default Address Actions

This PR adds generation of customer default shipping / billing address actions

[x] commit messages are commitizen-friendly
[x] code is unit tested
[ ] code is reviewed by at least one person
[x] code satisfies linter rules

Coverage remained the same at 95.723% when pulling 2df306f1b0cf08ec8061ab08615d3d791d2fe964 on customer-default-address-actions into 665ba33cf060fdc746a4dd62e43105d3cfe089e6 on rewrite-2.0.

Released in 2.0.0-alpha.44
/cc @JoshBones


[create-react-app] Failed to minify the code from a file

I'm trying to include this module in a create-react-app app but I'm unable to do so.
Failed to minify the code from this file:

 	./node_modules/@lourd/react-google-sheet/dist/index.es.js:83

Hey @ivorpad, thanks for the bug report. Can you please make a repo I can clone that produces the error? Or post specific steps to reproduce it, including the version of CRA you're using. Thanks!

Hey @lourd,
Sorry I was short of time when I reported it.
Here it is: https://github.com/ivorpad/react-google-sheets-bug-report
Steps to reproduce:
create-react-app v1.3.0

yarn add @lourd/react-google-sheet
import { GoogleSheet, GoogleSheetsApi } from '@lourd/react-google-sheet'
yarn eject
yarn build → Failed to compile.

Hi @ivorpad, sorry for the long delay in replying.
The root issue is that this module is being published with ES6 syntax and the version of Uglify that the current version of create-react-app is indirectly depending on cannot parse ES6.
There are 3 possible solutions I can think of:

I could publish the code with only ES5 syntax. That's unfortunate because it means shipping more code.
You could pre-transform the code and either move it into your application code or manage a fork.
You could try using react-scripts@next, which is using webpack v4, which depends on the latest version of uglifyjs-webpack-plugin, which depends on a version of uglify-js that can parse ES6.

The steps to using the v2 beta version of create-react-app are:
create-react-app foo
cd foo
yarn add react-scripts@next

I found that even if you've installed create-react-app@next, you still have to manually upgrade react-scripts after generating the project.
Hope solution 2 or 3 suffices!

@lourd I moved to my own solution but thanks a lot for taking the time to reply. I'll keep your solutions in mind for the next project.


Which Heroku Redis Addon?

For our useable example: https://github.com/ideaq/hapi-auth-jwt2/issues/9 we need to know
Which Redis Heroku addon is Free for our demo?

At the time of writing there are 5 Redis service providers on Heroku:

Of these RedisCloud https://addons.heroku.com/rediscloud has the best "Free" Plan:

Used RedisCloud. done.


Town of player returning none

 command /townof:
  trigger:
    set {_t} to town of player
    broadcast "%{_t}%"

The above seems not to work, and always returns <none>, with SharpSK version 1.6.3, Towny <IP_ADDRESS>, and Skript 2.2-dev30b bensku. Hook is set to true in config, and some other Towny expressions still work (tested mayor of "town").

It's not fixed at all.
Here my example:
command /banoftown :
trigger:
set {_t} to town of player
set {_s} to town of sender
broadcast "%{_t}%, %{_s}%"
After using of this command there appears:
,
Game-Version: 1.12.2.
Skript: 2.2-dev37c
SharpSK: 1.6.3.


How to remove foldlevel column at the beginning of each line

There is an extra column on each line, I think it is for indicating the foldlevel.
How to remove this column? Sometimes I need to copy from screen and there is always an extra space.
Thanks.

Please check out this: https://vim.fandom.com/wiki/Folding


Minecraft Forge Development Environment

This one is going to be a pain. We need the following:

Downgrade Gradle enough to work with Forge and eliminate any bugs we can from the build process that we were finding before (the Kotlin Continuations bug)
Install and configure UMC
Make a basic mod class that can be loaded from the development environment

Some notes on this: If the players vote for MC 1.14 today (as it appears to be), this issue changes to the following:

Keep the current Gradle version (probably)
Wait for UMC Gradle to support Minecraft 1.14
Make a basic mod class that can be loaded from the dev environment.

The players have spoken. UMC supports 1.14 is my understanding, so we need to do the following:

Configure UMC with Gradle
Make a basic mod class to test things out.


[Fit & Finish] Updated Fit and Finish guidelines

Description
Updates the following Fit and Finish guidelines for Notifications Repo:

Semantic Header Sizes:
a. H1: For page header (handled centrally) or page titles without a page header (these shouldn’t exist)
b. H2: For headings immediately under H1s, Modal and Flyout titles, top level Panel titles
c. H3: For headings immediately below H2s
Text Sizes: Using EuiText size="s"
Context Menus: Using small context menus

Screenshots

Scope
Before
After

Context Menu Change

Heading: Empty Page

Headings: Old UI Page Header

Modal

Text

Issues Resolved
[List any issues this PR will resolve]
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

could you checked failed CI cypress test to see if it's related?

fixed!


.run() outputs [sultan] command

When I run my python3 script sultan outputs to the console which I control.  Is there a way to stop this and/or grab this for later display?

I somehow had not noticed the logging=False parameter...  so
with Sultan.load() as c:
  c.mv(CFGFILE,NEWCFGFILE).run()

works as I was asking...  Close this.


Exercise 24.1-1 looks wrong

I think answer should be
~ | s | t  | x  | y | z
d | 2 | 4 | 6 | 9 | 0
π | z | x | z  | s | Ø
~ | s | t  | x  | y | z
d | 0 | 0 | 2 | 7 | -2
π | Ø | x | z  | s | t
Correct me if I'am wrong.

Man you are right!
Thanks so much ^ ^ I will update my solution

Thanks :)


[WIP] Integrated tidb dialect

# GORM_DIALECT=tidb go test ./...
# github.com/pingcap/tidb/parser
../../pingcap/tidb/parser/lexer.go:81: undefined: yySymType
../../pingcap/tidb/parser/misc.go:474: undefined: yySymType
../../pingcap/tidb/parser/yy_parser.go:73: undefined: yySymType
../../pingcap/tidb/parser/yy_parser.go:74: undefined: yySymType
../../pingcap/tidb/parser/yy_parser.go:75: undefined: yySymType
testing tidb...
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x421b81a]

goroutine 1 [running]:
panic(0x45b8d20, 0xc82000a140)
       	/usr/local/Cellar/go/1.6/libexec/src/runtime/panic.go:464 +0x3e6
database/sql.(*DB).SetMaxIdleConns(0x0, 0xa)
       	/usr/local/Cellar/go/1.6/libexec/src/database/sql/sql.go:568 +0x1aa
github.com/jinzhu/gorm_test.OpenTestConnection(0xc820101220, 0x4ead028, 0xc8200da650)
       	/Users/thomas/Developer/golang/src/github.com/jinzhu/gorm/main_test.go:77 +0x4ab
github.com/jinzhu/gorm_test.init.1()
       	/Users/thomas/Developer/golang/src/github.com/jinzhu/gorm/main_test.go:31 +0x30
github.com/jinzhu/gorm_test.init()
       	/Users/thomas/Developer/golang/src/github.com/jinzhu/gorm/utils_test.go:30 +0x94
main.init()
       	github.com/jinzhu/gorm/_test/_testmain.go:336 +0x4f
FAIL   	github.com/jinzhu/gorm 	0.023s
?      	github.com/jinzhu/gorm/dialects/mssql  	[no test files]
?      	github.com/jinzhu/gorm/dialects/mysql  	[no test files]
?      	github.com/jinzhu/gorm/dialects/postgres       	[no test files]
?      	github.com/jinzhu/gorm/dialects/sqlite 	[no test files]

Sadly it seems to be broken somehow :(

Tidb doesn't have driver right now. Certainly, u can look tidb as mysql. so u can use mysql driver to connect tidb. So there's no need to add tidb. just think tidb is a mysql server.

That's simply not entirely true, you can use tidb also as an integrated database based on files, comparable with sqlite.

 Doesn't look like there is any big interest in that at all.

Hi @tboerger
This is not doable unless tidb provides a sqlite like dialect github.com/mattn/go-sqlite3 for its integrated db.
Rightnow, you can only use its mysql protocol.

That's not true. XORM integrates tidb without the mysql protocol. There you can use tidb as an embedded database, https://github.com/go-gitea/gitea integrates that based on the tidb build flag.

The XORM integration is done within https://github.com/go-xorm/tidb

Trying to check out tidb, but failed to install it...

Will check it again when it becomes more stable.

@jinzhu, look like it's ok :)

The second error is because tidb requires a make invoke.

I have tried
git clone https://github.com/pingcap/tidb.git $GOPATH/src/github.com/pingcap/tidb
cd $GOPATH/src/github.com/pingcap/tidb
make

then run this go script
package main

import (
        "database/sql"
        "fmt"

        _ "github.com/pingcap/tidb"
)

func main() {
        db, err := sql.Open("tidb", "memory://tidb/tidb")
        fmt.Println(err)
        fmt.Println(db.Ping())
}

Got this error


Global View of the map

Hi! Is there any way to generate a global view of the map like the ones in the readme? I have seen a Global View class in engine.py but it is not implemented. Thank you!

There are three options, you can use the semantic full map returned in the info dict (integer coded object types, no textures), you can increase the view area a lot (it will still be centered at the agent but you can see everything at once, with bezels where needed), you can implement the global view stub you found based on the local view (it would mostly be a copy of that but I don't have time to do it).


Airline ignores 'modified' icon

I was trying to set an icon to be shown next to the file name when it's modified. After all, there's a modified airline_symbol. However, that symbol is never used, since airline uses the default vim '%m' statusline item when setting the file name or path.
The lines in question are in airline/init.vim 191 to 196:
if get(g:, 'airline_section_c_only_filename',0)
    call airline#parts#define_raw('file', '%t%m')
  else
    call airline#parts#define_raw('file', airline#formatter#short_path#format('%f%m'))
  endif
  call airline#parts#define_raw('path', '%F%m')

environment

vim: latest
vim-airline: latest
OS: windows

actual behavior
Airline always shows the default [+] when a file is modified.
expected behavior
I would expect Airline to show whatever symbol is defined in the symbol table.

Yes, the %m is the default modified label added by Vim.  I suppose you could define your own part using a custom label. But currently this is not enabled in vim-airline.


Add special custom move

Team player ~~~~~> Toxic endeavors

Actually done now

Toxic endeavors: endeavor, but has a 33% chance to badly poison the foe

Will merge post afd

K

/me coughs

Read my above comment, i need to edit your setup on SG which can cause crashes if you new move dosent exist yet.

Ok


Fix flag typo

What does this PR do?
Doc fix for flag typo
Before submitting

[ X ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).

Who can review?
@pacman100 @muellerzr

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


CCA-Item assumes that a key will always be present when serializing certain ItemStacks

This line in MixinItemStack.serialize seems to assume that AbstractComponentContainer.NBT_KEY will always be present if this.serializedComponents != null is true and this.components != null is false
However, for unknown reasons it is possible to construct a situation where this is not true given a collection of certain mods
Mods used

Fabric api 0.31.0
Better end (dev version, this commit specifically at time of writing due to mixin issues otherwise)
Curios api (fabric) 0.0.11
Curious elytra 1.1.1
Dynocaps 1.6.13

Dynocaps and curios both use CCA-Item, dynocaps is not used in reproduction but removal results in the bug not manifesting
Reproduction

Created an enchanted elytra
Place a better end pedestal in the world
Place elytra on pedestal -> Bug manifests here, when the elytra is placed AbstractComponentContainer.NBT_KEY is not present, resulting in null being passed as a tag
Force chunks to save -> Error manifests here when null is attempted to be serialized as a tag, leading to chunk reset and disconnection if in multiplayer with no modded blame in the stack trace

Forgot to double check CCA verison, not present in 2.7.12

Reopening, bug is reproduceable only on a dedicated server
Reproduction steps remain the same otherwise


Tag Visual Tweak: Remove grey stroke from selected state

Related to: #65

Grey stroke around tag button should be removed ☝

i think this can be closed: commit

You're right, thanks @davidbucka!


Config documentation issue

I tried to configured my Spring env, as documented in the "Configure the task scheduler" paragraph (the ScheduledLockConfiguration bean provider), but this way, I noticed my Tomcat was unable to stop gracefully, due to the error
"The web application [xyz] appears to have started a thread named [ThreadPoolTaskScheduler-X] but has failed to stop it. This is very likely to create a memory leak."
I worked around it by using a different configuration:
@Bean
public TaskScheduler taskScheduler(LockProvider lockProvider) throws Exception {
    ScheduledLockConfiguration schedulerConfig = (ScheduledLockConfigurationBuilder
                    .withLockProvider(lockProvider)
                    .withPoolSize(10)
                    .withDefaultLockAtMostFor(Duration.ofMinutes(10))
                    .build());
    SpringLockableTaskSchedulerFactoryBean configBean = (SpringLockableTaskSchedulerFactoryBean) schedulerConfig;
    return configBean.getObject();
}

This way, it all looks working as expected.

Hi, thanks for reporting it. If it's the case, it's not a documentation issue but a bug. Will take a look at it.

Should be fixed in 0.16.1

In 0.16.1, TaskExecutor should be closed when configured as documented. Thanks again for reporting it.


Fix: DO-1452, DO-1713 fix table sort does not return the right index row

Motivation and Context

A bug was found where if table was sorted then the wrong chosen role was passed to the backend.
Implementation Description

The issue turned out to be an issue with getting the value within the getItem which created a discrepancy when data was sorted from initial index to the one chosen by the user. This was also an issue for if the table selected value was changed from outside the component. The easier fix was to simply remove that part of the logic. We do need to fetch the data more often, but in this way we guarantee that regardless if selection is done in a sorted table or changed from outside that the correct index is still passed to both onclick_row action context and selected_indices Variables.
Any new dependencies Introduced

None
How Has This Been Tested?

Locally by single and multi selecting table rows, sorted and unsorted and checking corresponding Variables were updated as expected.
PR Checklist:

[x] I have implemented all requirements? (see JIRA, project documentation).
[x] I am not affecting someone else's work, If I am, they are included as a reviewer.
[ ] I have added relevant tests (unit, integration or regression).
[x] I have added comments to all the bits that are hard to follow.
[x] I have added/updated Documentation.
[x] I have updated the appropriate changelog with a line for my changes.

Screenshots (if appropriate):

@krzysztof-causalens looked at the error in the CI and their docs recommend to sanitize the data beforehand, is there a reason why we aren't?

@krzysztof-causalens looked at the error in the CI and their docs recommend to sanitize the data beforehand, is there a reason why we aren't?

Sanitizing is mostly important where the HTML comes from the user. Yes ideally we'd sanitize it but "fine" since it's provided by the app developer (to the best of my knowledge, not a security expert), hence the error was ignored. Not sure why suddenly the bearer-ignore stopped working, perhaps they changed how their ignore system works and the comment needs updating?

@krzysztof-causalens looked at the error in the CI and their docs recommend to sanitize the data beforehand, is there a reason why we aren't?

Sanitizing is mostly important where the HTML comes from the user. Yes ideally we'd sanitize it but "fine" since it's provided by the app developer (to the best of my knowledge, not a security expert), hence the error was ignored. Not sure why suddenly the bearer-ignore stopped working, perhaps they changed how their ignore system works and the comment needs updating?

Note for future: The message was relating to a Bearer issue that is currently up, meantime workaround was found in a separate PR and added here.

Checked and with persist_value flag the table selection persists, tested for an initial value selected and for a combination of filter + sort


Annotate new object statements

What are you wanting to see improved / introduced?
const foo = new Foo(a, b);

the following should be annotated

Implemented


(#378) add Archlinux support

I haven't finished testing yet, but at least those changes are needed.

@ripienaar we finished testing, those changes are the only one needed. Can you merge it and create a new release?

thanks a lot @bastelfreak!


[HOLD] Focus emoji picker search input

Details
Following-up on https://github.com/Expensify/Expensify.cash/pull/1991#event-4602076155. During testing, we found a minor issue that when the EmojiPickerMenu is first opened, neither the ReportActionCompose nor the emoji search input are focused. We decided that because that was already a big PR it would be easier to handle the focus issue in a follow-up PR.
Note: I also found an issue with the existing implementation where the search input was displayed on mobile web, which we don't want for the same reasons we don't want it to display on mobile (i.e: the native keyboard competes with the emoji picker.
Fixed Issues
n/a.
Tests / QA Steps (Web/Desktop)

Click on the emoji icon in the ReportActionCompose to open the emoji picker.
Verify that the emoji picker search input is focused, and that you can immediately type to search emojis.
Click an emoji and verify that it populates the ReportActionCompose (make sure it doesn't take two clicks)
Close the emoji picker.
Verify that the ReportActionCompose has focus again.

Tests / QA Steps (iOS/Android/mWeb)
Just make sure that the emoji picker still works like normal. This PR should have no other effect.
Tested On

[x] Web
[x] Mobile Web
[ ] Desktop
[x] iOS
[ ] Android

Screenshots
Web

Mobile Web

Desktop

iOS

Android

Retesting after merging main and now I think this needs to be HELD on https://github.com/Expensify/Expensify.cash/pull/2424 😞

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Extension grant should be any valid URI

According to the spec the extension grant can be any valid URI. Currently node-oauth2-server expects it to start with either http:// or https://, but any other URI should also work. E.g. if one wants to implement the JWT Bearer Token Grant Type the grant_type URI for the authorization grant is urn:ietf:params:oauth:grant-type:jwt-bearer.
See
https://tools.ietf.org/html/draft-ietf-oauth-jwt-bearer-12#section-2.1
http://tools.ietf.org/html/rfc6749#section-4.5
A library that seems to handle the validation of URIs is https://github.com/garycourt/uri-js , but I have never used it.

Very good point, I have thought about this before but never looked into it much.
Rereading the rfc I think there is a case for just accepting any string as an extension grant?
@ruimarinho @mjsalinger @alex-whitney ?

Yes, I think that would make sense, thus passing every grant that is not known to the extension grant handler.
I will probably do a generic implementation for the JWT Bearer Token grant type which can easily be hooked into node-oauth2-server. Would you actually be interested into adding this directly to node-oauth2-server?

The spec does specify a URI, but the example they give follows the urn scheme.  It's worth noting that all of the proposed extension grant standards (Oauth-Saml2, JWT, and MAC) use the URN format.  So I think validating that it is either a valid URI or a valid URN would be the fine here, but I'd see nothing wrong with opening it up to any string, as it has to be properly interpreted by the server anyways and match whatever scheme is defined.

@mjsalinger A URN is a valid URI, it is actually a more specific URI.

How about #142

#142 Merged


vm support: support multiple WorkloadEntry(s) with the same IP

Signed-off-by: Yaroslav Skopets<EMAIL_ADDRESS>Please provide a description of this PR:
This PR adds support for the use case with multiple WorkloadEntry(s) with the same IP address.
Apparently, some users would like to run multiple Istio Proxies on the same VM (e.g., multiple sidecars without Iptables or multiple routers).
This PR changes behaviour in case there are multiple WorkloadEntry(s) with the same IP.
Previously, "GetProxyServiceInstances()" would only consider one of the WorkloadEntry(s) with the same IP (AFAIU, the choice was not stable/predictable).
After this PR, "GetProxyServiceInstances()" will consider all WorkloadEntry(s) and favour the one that matches proxy.ID. If there is no WorkloadEntry that matches proxy.ID, the WorkloadEntry(s) are sorted by name and the first one is chosen.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
4d7299ce01dd8930c96d641556046e0c0a7752c6
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
4d7299ce01dd8930c96d641556046e0c0a7752c6
link
true
/test release-notes_istio

lint_istio
4d7299ce01dd8930c96d641556046e0c0a7752c6
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
4d7299ce01dd8930c96d641556046e0c0a7752c6
link
true
/test lint_istio

release-notes_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test release-notes_istio

lint_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test release-notes_istio

lint_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test lint_istio

gencheck_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test lint_istio

gencheck_istio
693c4cc8dcbd961a97668ee5d3633c14fff7741f
link
true
/test gencheck_istio

release-notes_istio
6cc5e500038721e625e622dc33ca7f0939eb14d7
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test release-notes_istio

lint_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test release-notes_istio

lint_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test lint_istio

gencheck_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test lint_istio

gencheck_istio
5530b3ded5a9b8cde5c3df757233fa293fbd8f91
link
true
/test gencheck_istio

release-notes_istio
2f4d59d6605d65a0eca41af94cf1cd2a4a6f0f53
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Need to do the same for service entry registry

Can you split this refactor to another pr? It seems this change is huge and maybe risky.  I got a review yesterday, it looks good.

@hzxuzhonghu Sure, I will split serviceregistry/serviceentry changes into a separate PR.
I'm still trying to understand all the source code of serviceregistry/serviceentry.

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ac4d1d3c5af878a52dbabadf61fd0f787b0efc21
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test release-notes_istio

integ-security-istiodremote_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test integ-security-istiodremote_istio

lint_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test release-notes_istio

integ-security-istiodremote_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test integ-security-istiodremote_istio

lint_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test lint_istio

unit-tests_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test integ-security-istiodremote_istio

lint_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test lint_istio

unit-tests_istio
fafb276f80fa11acaa08eb59aba4ae28e20a6c05
link
true
/test unit-tests_istio

release-notes_istio
062717cc46e2f53eb0199ee2e5c5c3cedcfde255
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
1fac6e535e924758e82424d76b6c1e2504099585
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes_istio


Unreal: Changed naming and path of Camera Levels

Changelog Description
The levels created for the camera in Unreal now include _camera in the name, to be better identifiable, and are placed in the camera folder.
Additional info
To make this change, it has been necessary to add a visibility track for the camera level as well. This solves a problem with the rendering, which didn't load the camera correctly and kept it at the origin during rendering.
Because the generation of the sequences is now identical for camera and layout, the functions to handle it have been moved to hosts/unreal/api/pipeline.py, so they are accessible by both loaders.
Testing notes:

In Unreal, try loading the camera.
Check name of the camera, and that it is placed correctly in the master level sequence.
Try rendering a scene.
Try updating the camera.
Try to remove the camera through the manager.

Cameras works ok. They are placed correctly in sequencer and according to clip in /out.
Also Render and Publish are fine and working.
Problem being when trying Remove via Manage. It seems only selected Camera asset/level is removed but all got removed from shots_map. They however still been visible as levels in Levels Editor for some reason.
We need to address this issue.

When doing Update to Latest on shot camera (in my case sht020) the animation is not correctly placed in time in Sequencer and not respecting clipIn and ClipOut values.

So now when perform Update to Latest on Camera asset all correctly set to timeline.
Update of the Camera works.


Feedback function failing

Problem encountered on https://dotnet.microsoft.com/en-us/platform/upgrade-assistant/tutorial/install-upgrade-assistant#
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
=======================
Just wanted to suggest example code be put onto this page as it appears there's no GUI for people inexperienced with console.
However feedback function is broken (fails in Firefox, Chrome and Edge)

@cathysull Is this something you can help take care of ?


Fix SSS role being repeated in the examine tooltip

Fixes #1690

I'd prefer adding a display name & display color to the Mind Role component and printing that instead of checking manually for a role (if we want additional roles like a detective).
But apparently mind code is scary


User report: postcodes

Also a problem with postcodes
I type in a postcode (such as w1a 1aa) and follow it with the Enter key. The search result appears briefly and then disappears. To get it back I have to add a space to the end of the search.
on a desktop, that's less of a problem but a bit clumsy. I hit Enter by habit after typing in a search box, because most other programs require it
on a mobile, that can mean the keyboard stays on screen, hiding half of the map.
My mobile is a sony XA-2 (H3113) running lineageOS 16, equivalent to android 9, but it works with most web sites. It uses Openboard keyboard.

This will hopefully have been solved with the release of the NextJS migration — I can't replicate it. Please reopen if it is still an issue


Refresh OAuth 2 Access token with Refresh Token failed

I often get this error - Refresh OAuth 2 Access token with Refresh Token failed. Body: [{"error":"invalid_grant"}]

@mishabachkur This error typically occurs when you have an incorrect configuration for clientId, clientSecret etc. Can you please check you have configure the keys accurately?

The issue: confirmed
The reason:
With the same credentials:
        $companyInfo = $dataService->getCompanyInfo();
        return json_encode($companyInfo, true);

worked properly
But,
        $OAuth2LoginHelper = $dataService->getOAuth2LoginHelper();
        $accessTokenObj    = $OAuth2LoginHelper->exchangeAuthorizationCodeForToken(
            self::QUICKBOOKS_AUTHORIZATION_CODE,
            self::QUICKBOOKS_REALM_ID
        );

Provided with an issue:
  Cron-job "socalgraph_quickbooks" threw exception QuickBooksOnline\API\Exception\ServiceException

  [QuickBooksOnline\API\Exception\ServiceException]
  Exchange Authorization Code for Access Token failed. Body: [{"error_description":"Token invalid","error":"invalid_grant"}].

@coresh That is very interesting find. Thanks for persisting on the issue. Would you be able to send me the raw request and response log for both the calls? I can bring it to my downstream team.

vendor/quickbooks/v3-php-sdk/src/Core/OAuth/OAuth2/OAuth2LoginHelper.php

public function exchangeAuthorizationCodeForToken($code, $realmID){
...
var_dump(http_build_query($parameters));
var_dump($intuitResponse->getBody());
var_dump($intuitResponse->getHeaders());
...

Result:
eveloper.intuit.com%2Fv2%2FOAuth2Playground%2FRedirectUrl"

string(61) "{"error_description":"Token invalid","error":"invalid_grant"}"

array(10) {
  ["date"]=>
  string(29) "Mon, 13 Jul 2020 16:24:34 GMT"
  ["content-type"]=>
  string(30) "application/json;charset=utf-8"
  ["content-length"]=>
  string(2) "61"
  ["server"]=>
  string(5) "nginx"
  ["strict-transport-security"]=>
  string(16) "max-age=15552000"
  ["intuit_tid"]=>
  string(35) "1-5f0c8ac2-94777b4a0a0f25100de4b6f2"
  ["x-spanid"]=>
  string(38) "6f58449e-aa9e-4cf7-b640-8ec0744cc449:1"
  ["x-amzn-trace-id"]=>
  string(81) "Self=1-5f0c8ac2-c63f2990c703c9bc452205fc;Root=1-5f0c8ac2-94777b4a0a0f25100de4b6f2"
  ["cache-control"]=>
  string(18) "no-cache, no-store"
  ["pragma"]=>
  string(8) "no-cache"
}

@coresh Ah I see what you are saying. These calls use different auth headers. The auth code call uses Basic XXXX where XXXX is a base 64 encoded value of client id and secret. The getUserInfo call is a Bearer token call, where you use the accessToken which you had last obtained. I would still suggest checking the value for clientId that is being passed in the request. Since the client information are secure, you could choose to create a support ticket and we can help you there.

Re : I would still suggest checking the value for clientId that is being passed in the request.
clientID -> it's Ok.
The reason:
call:
$dataService->getCompanyInfo()
returns properly output
Please check concerning to the Production environment.
Thank you

@coresh can you please elaborate what you mean by "Please check concerning to the Production environment."?

I am getting the same error. I have found that I get a new refresh token before 24 hours have elapsed since I received the old one. My application is configured to save the new token every time, but the new token does not work. If I log the old token, and use that to call the refresh token endpoint again it works fine, and I get the old token back. I do believe there might be an issue on your guy's end.
Checking my error logs I notice that I can't re-authenticate my server about 8 hours after doing so manually, leading me to believe that that's when you guys return a new refresh token to me.
My client ID, and secret are both correct, as if I go to my server and simply change the refresh token back to what it was before the automated process I can retrieve a new access token, but a new refresh token is not returned.
This is confusing so I'll go through step by step what I did, and how I think things went on the server:

use the playground to retrieve an access and refresh token
log both tokens at the time i receive them in my issue tracker
save those on the server, and begin the re-authentication job (this runs every 15 minutes to retrieve a new access token)
8 hours later, on the server: quickbooks returns a new refresh token, earlier than expected, but it's saved anyway, as recommended by the documentation at point 3 here
that new token does not work, leading to the invalid_grant error from quickbooks
I notice the error (at the time of writing, roughly 22 hours after I initiated step 1), log into the server, with the old refresh token, which I logged earlier, and replace the new refresh token I got from quickbooks with the old one
call the refresh_token endpoint, with the old token, which returns a new access_token as expected, and the old refresh token.

I think that around 8 hours after I initiated step 1, you guys return a new refresh token to me.

In short, the recommended workflow for all apps is:
Step 1. If the Access Token fails, use the current Refresh Token to request new Access and Refresh Tokens.
Step 2. Store BOTH the Access and Refresh Tokens that are returned.
Step 3. Use the new Access Token for making QuickBooks Online API calls.
Step 4. When the Access Token fails in 1 hour, go to Step 1.
To solve this error: "Refresh OAuth 2 Access token with Refresh Token failed" in my Laravel application I had this approach:

store config data in .env files, store tokens in database; fetch them on every DataService object creation, and recreate tokens on the fly and save them again in database:

use QuickBooksOnline\API\Core\OAuth\OAuth2\OAuth2LoginHelper;
use QuickBooksOnline\API\DataService\DataService;
use QuickBooksOnline\API\Exception\SdkException;
use QuickBooksOnline\API\Exception\ServiceException;

    public function getTokensFromDataBase()
    {
        $config = config('quickbooks');
        $quickbook = \App\Models\QuickBook::where('realm_id', $config['realm_id'])->first();
        return [
            'access_token' => $quickbook->access_token,
            'refresh_token' => $quickbook->refresh_token,
            'x_refresh_token_expires_in' => $quickbook->x_refresh_token_expires_in,
        ];
    }

    public function getTokens()
    {
    	// step 0 fetch latest tokens from db
        $tokens_from_db = $this->getTokensFromDataBase();

        // step 1: get config data, and get refresh token
        $config = config('quickbooks');
        $dataService = DataService::Configure([
            'auth_mode' => 'oauth2',
            'ClientID' => $config['client_id'],
            'ClientSecret' => $config['client_secret'],
            'RedirectURI' => $config['redirect_uri'],
            'scope' => 'com.intuit.quickbooks.accounting',
            'baseUrl' => $config['base_url'],
            // Get the refresh token from session or database
            'refreshTokenKey' => $tokens_from_db['refresh_token'],
            'QBORealmId' => $config['realm_id'],
        ]);

        // step 2: refresh access token using the refresh token
        // The first parameter of OAuth2LoginHelper is the ClientID, the second parameter is the client Secret
        $oauth2LoginHelper = new OAuth2LoginHelper($config['client_id'], $config['client_secret']);
        $accessToken = $oauth2LoginHelper->refreshAccessTokenWithRefreshToken($tokens_from_db['refresh_token']);
        $dataService->updateOAuth2Token($accessToken);
        // step 3: save the new tokens in the database

        $token_in_db = \App\Models\QuickBook::where('realm_id', $config['realm_id'])->first();
        $token_in_db->access_token = $accessToken->getAccessToken();
        $token_in_db->refresh_token = $accessToken->getRefreshToken();
        $token_in_db->x_refresh_token_expires_in = $accessToken->getRefreshTokenExpiresAt();
        $token_in_db->save();

        return [
            'access_token' => $accessToken->getAccessToken(),
            'refresh_token' => $accessToken->getRefreshToken(),
            'x_refresh_token_expires_in' => $accessToken->getRefreshTokenExpiresAt(),
            'expires_in' => $accessToken->getAccessTokenExpiresAt(),
        ];
    }

In your controller where you wwant to store invoice in QB, fetch tokens by calling method getTokens() and set DataService object again:

		$tokens = $this->getTokens();

        $config = config('quickbooks');

        $dataService = DataService::Configure([
            'auth_mode' => 'oauth2',
            'ClientID' => $config['client_id'],
            'ClientSecret' => $config['client_secret'],
            'RedirectURI' => $config['redirect_uri'],
            'accessTokenKey' => $tokens['access_token'],
            'refreshTokenKey' => $tokens['refresh_token'],
            'QBORealmID' => $config['realm_id'],
            'baseUrl' => $config['base_url'],
        ]);

        $dataService->throwExceptionOnError(true);

now, when DataSerice object is set up properly, you can follow instructions to save or ftech data to / from QB:

	....

        $invoice = Invoice::create([
            "Line" => $lines,
            "CustomerRef" => [
                "value" => $customer->Id,
                "name" => $customer->name,
            ],
            "BillEmail" => [
                "Address" => $customer->email,
            ],
        ]);

        $response_from_qb = $dataService->Add($invoice);

        $xmlResponseQB = XmlObjectSerializer::getPostXmlFromArbitraryEntity($response_from_qb, $urlResource);

        // use this to debug response from QB
        echo "Invoice Created Id={$response_from_qb->Id}. Reconstructed response body:\n\n";
        echo $xmlResponseQB;

	....

problem with tokens is that they expire - access token is valid for 60 minutes, and by using approach above, if your users do not perform query to the QB, access tokens in your db will expire

My approach was to create Laravel command, and set up cron job which will run every 10 minutes and update tokens in db by itself:
<?php

namespace App\Console\Commands;

use Illuminate\Console\Command;
use QuickBooksOnline\API\Core\OAuth\OAuth2\OAuth2LoginHelper;
use QuickBooksOnline\API\DataService\DataService;
use QuickBooksOnline\API\Exception\SdkException;
use QuickBooksOnline\API\Exception\ServiceException;

class refreshQBToken extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'app:refresh-q-b-token';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'This command refreshes the QuickBooks token.';

    /**
     * Execute the console command.
     * @throws SdkException
     * @throws ServiceException
     */
    public function handle()
    {

        $config = config('quickbooks');
        $token_in_db = \App\Models\QuickBook::where('realm_id', $config['realm_id'])->first();
        $dataService = DataService::Configure([
            'auth_mode' => 'oauth2',
            'ClientID' => $config['client_id'],
            'ClientSecret' => $config['client_secret'],
            'RedirectURI' => $config['redirect_uri'],
            'scope' => 'com.intuit.quickbooks.accounting',
            'baseUrl' => $config['base_url'],
            // Get the refresh token from session or database
            'refreshTokenKey' => $token_in_db->refresh_token,
            'QBORealmId' => $config['realm_id'],
        ]);
        $oauth2LoginHelper = new OAuth2LoginHelper($config['client_id'], $config['client_secret']);
        $accessToken = $oauth2LoginHelper->refreshAccessTokenWithRefreshToken($token_in_db->refresh_token);
        $dataService->updateOAuth2Token($accessToken);
        //save the new tokens in the database
        $token_in_db->access_token = $accessToken->getAccessToken();
        $token_in_db->refresh_token = $accessToken->getRefreshToken();
        $token_in_db->x_refresh_token_expires_in = $accessToken->getRefreshTokenExpiresAt();
        $token_in_db->save();

        $this->info('Successfully refreshed QB tokens.');
    }
}

Thats how I managed to solve problem with tokens expiry

@daraul Have you figure out how to solve this?

Sorry for the very late response, @radenkozec -- no, I never figured out a workaround, and I've since left that job so it's someone else's problem, now 😆
Maybe the recommendation above will work.

Here is the explanation to this issue:
https://help.developer.intuit.com/s/article/Validity-of-Refresh-Token
`On day 2, a new Refresh Token is returned, and Refresh Token A expires. Refresh token A is no longer valid for making the Refresh Token API call to get a new access token. The Refresh Token API call will throw an “invalid_grant” error if you attempt to use it.
For simplicity, always store and use the LATEST Refresh token returned.
In short, the recommended workflow for all apps is:
Step 1. If the Access Token fails, use the current Refresh Token to request new Access and Refresh Tokens.
Step 2. Store BOTH the Access and Refresh Tokens that are returned.
Step 3. Use the new Access Token for making QuickBooks Online API calls.
Step 4. When the Access Token fails in 1 hour, go to Step 1.
`

@jainnys when I am trying to generate the new refresh token using the current refresh token, it returns same value on 1st day, on day 2 after 24 hours, it throws an error "invalid_grant"

Same issue with me... Any solutions


Add github fork ribbon.

I'm not sure, I thought we had Netlify working on this for deployment preview so I was taking it for a test spin.

we had to temporarily disable Netlify deploy previews due to them taking a long time and using up all our build minutes (see #254).
I'd been thinking about adding an 'always-on-screen' link to GitHub too. This Storybook add-on looks like it could be perrfect. https://storybook.js.org/addons/@geometricpanda/storybook-addon-badges
I spent 5 minutes trying to get it working on Friday but wasn't sure how to configure - but I bet Ryan could get it working. :-) We could maybe add a 'BETA' badge - and make a 'GITHUB' badge/link too.

With the project transitioned to public, doesn't that enable us to use Netlify without restriction?

With the project transitioned to public, doesn't that enable us to use Netlify without restriction?

No. The account only allows for 1000 minutes of build time total. Whether the site or repo is private or not doesn’t make a difference.

So is a Netlify license needed to resolve issues such as this?

I took a look at the badge add on, it looks like it adds the badges to the toolbar. right now we're only showing the toolbar on the components by default, not on the Intro/Framework Integration pages

lets close this PR for now - but if you can take a look at the that storybook badges again. maybe that'd be good to do...


Fix multiple issues with ~FC023

Signed-off-by: Justin Schuhmann<EMAIL_ADDRESS>Description
Resolves multiple issues caused by recommendations from FC023 rule, only_if and not_if are ignored for converge_if_changed and converge_by
Issues Resolved
#340
Check List

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

We actually retired FC023 because it caused problems like these.

@tas50 yeah i read the documentation but feel for it all the same...saw deprecated but still thought that the converge_* functions would work with not_if and only_if..testing locally to make the test pass (inspec can't access node so need to verify the computer name)

@iennae not in the iis_app


Wireless Universal Terminal (Pattern Encoding) doesn't work with JEI for creating patterns

I'm playing SkyGreg modpack, with latest AE2WTLib installed, as I can see.
I created universal wireless terminal:

And I want to encode patterns for GregTech recipes. When I try to add them in pattern encoding, it writes following:

Oh, as I can see it was fixed on the GregTech Modern side... Unfortunately I can't update to that version...

duplicate of #273


Request to change filter icon on YouTube Android

I noticed the filter icon in the YouTube app is a settings icon. Is there some open source org I can raise this feedback too change this? Is there some way I can contribute and help replace the icon?
what the current filter icon looks like > http://imgur.com/a/vKzkP

This sample has been migrated to a new location (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


Update RStudio download link

On Day 0 the link to https://www.rstudio.com/products/rstudio/download/ redirects to https://posit.co/products/, which is not exactly the right place to download RStudio.  I suggest updating to https://posit.co/download/rstudio-desktop/
(part of JOSE review here: https://github.com/openjournals/jose-reviews/issues/260)

Huh. Seems Posit should fix this obviously wrong redirect 😜.


Adds more logging blocks for debugging and extended features

This isn't a large change, but I was hoping for a little more insight into when certain actions occur inside OHHTTPStubs so I added some more blocks in addition to onStubActivationBlock. Now there is also a onStubRedirectBlock and a afterStubFinishBlock for better clarity. As well, I pass in the response used by the stub. In my case I use single stub that generates multiple different responses, similar to your example Different responses for each call and so it was easier to track the response returned than the descriptor that generates that response.
I'm totally open to a discussion on this, but it'd be really helpful for me if something like this was added, hence why I put together this PR.

Also, was CI disabled for this project?

Thanks a lot for this PR! Looks quite good! :+1:
I don't know why Travis didn't pick up that PR, it usually does (it did for the last PR 160, so why not this one?), so that's strange indeed.
Could you also add an entry in the CHANGELOG.md to list your changes there and credit yourself? Thanks!

CI ran this time!

Thank you! That's an easy fix


[BUG] Can't paste text to InputField

Describe the bug
I would like to paste text to InputField, but its not working.
To Reproduce
Steps to reproduce the behavior:
import pytermgui as ptg

window_manager = ptg.WindowManager()
window_manager.layout = ptg.Layout()

svc_name_window = ptg.Window(width=70, height=5, title="Service Name")
svc_name_window.is_modal = True
svc_name_window.center()
svc_name_window += ptg.Label()
service_name_field = ptg.InputField(prompt="Service Name: ")
svc_name_window += service_name_field
svc_name_window += ptg.Label()
svc_name_window += ptg.Button("OK")

window_manager.add(svc_name_window)
window_manager.run()

Sorry about the delay, should be fixed now!

No worries ;) I can confirm that it works!
Thanks!


Please provide experimental data

Can you please provide experimental data for this repo.
Mail ID<EMAIL_ADDRESS>thankyou

@Thanmai01 Sent!


⚠ Bofrost.at has degraded performance

In 4053beb, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1292 ms

Resolved: Bofrost.at performance has improved in 8a562f8 after 6 minutes.


Swift integration

I can not set properties programmatically, and I am getting the error
'max' is inaccessible due to 'internal' protection level
Please change the ACL to public
P.S.
I used cocoapods

try pod update. I just released new version 0.3.1

10x, can you please add  self.sendActions(for: .valueChanged)
in moveHandleToPosition or where-ever you think its belongs?
בני דוידוביץ                 Benny Davidovitz
טל.<PHONE_NUMBER>        Tel
email<EMAIL_ADDRESS>2017-06-27 11:46 GMT+03:00 Seeppp<EMAIL_ADDRESS>
try pod update. I just released new version 0.3.1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/seeppp/NapySlider/issues/10#issuecomment-311294319,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARLYKMJKXN0MF4bJueVhzDlQDnFFB9s1ks5sIMGDgaJpZM4OGMi6
.


Fail when applied on non-root project

Currently there is a warning displayed, but there were invalid issues reported which tends that people do not read it. Error should be thrown with detailed information why it happened and a request to report an issue when it is really needed.

Within my project I only deploy a single module (API). I had to apply the plugin to the sub-module as otherwise credentials from the maven deployer config of the sub-module are not picked up. See https://github.com/UweTrottmann/SeriesGuide/commit/ca33e8ad2fa6cc5c426450c8aef3417ba073ca7f.

For the records, I had the same issue. Applying the plugin to the root means that Gradle doesn't find the tasks in the subprojects anymore. Applying the plugin to the subprojects triggers this new error. So I am not sure how this plugin is supposed to be used with subprojects.
Only way to make it work was to downgrade to version 0.10.0.

One more vote for adding of an override switch. Can take a look at it if pull requests are welcome.
I have a bunch of Java projects using the plugin, and I need to apply some common build logic (not related to artifacts distribution) to all of the projects. I tried to do it dynamically in an umbrella Gradle project, but bumped into the exception.
Downgrade to 0.10.0 helped in my case, thank you, @depryf!

@dmarsche Could you elaborate more on your use-case? Couldn't you just have "the common logic to apply on submodules" and "the common logic to apply on the root project"?

@szpak can you please revert this or make some configuration possibility?
I'm now doing new builds on in Kotlin and to not have a looooong build script but still have static accessors for nicer build scripts, I use pre-compiled script plugins.
For generating the static accessors for applied 3rd-party plugins Gradle applies the plugin to some test project, but to a subproject as it seems.
This then fails for this plugin and the whole process is busted: https://github.com/gradle/gradle/issues/9386

Well, the build does not fail, but there is a big fat stack trace in the output and no static accessors available.

Not only for this plugin, but for all in this script

@Vampire A few years old Kotlin scripting in Gradle seems to still be somehow immature and troublesome :).
I plan to add an ability to override that restriction in the next version of the plugin.

I'd more say it is really awesome.
Of course it has bugs, as any software has.
Is there an ETA for a version with this override?

Maybe I will be convinced one day :).
The next planned release is 0.30.0 with rewriting the plugin internals to adjust it to the new Property/Provider infrastructure to manage plugin configuration in Gradle 4.8+. Nevertheless, Continuous Delivery is in place. Therefore, having a nice looking PR (in the similar way to that, but with error instead of warning and information how to override that behavior if really needed), some new minor release would not be a big effort for me :).

Hm, thinking about it, a property might not be the best solution.
At least not for the problem I have.
I'm not sure whether the properties are forwarded to the build that is used to generate the static accessors.

I don't know, you would need to check it first. But assuming it's similar to the build compiling buildSrc it could be (both read from project/usr-local gradle.properties and -P).

Is there a way in your build to publish to mavenLocal()? Using composite build does not work yet from buildSrc build.

However, I just built the JAR and depended on the JAR in the filesystem.
Unfortunatley property will not work for my use-case.
Neither specifying it with -P, nor in <project-dir>/gradle.properties, nor in ~/.gradle/gradle.properties had any effect. :-(

How about this:
diff --git a/src/main/groovy/io/codearte/gradle/nexus/NexusStagingPlugin.groovy b/src/main/groovy/io/codearte/gradle/nexus/NexusStagingPlugin.groovy
index 2bcac95..39de20f 100644
--- a/src/main/groovy/io/codearte/gradle/nexus/NexusStagingPlugin.groovy
+++ b/src/main/groovy/io/codearte/gradle/nexus/NexusStagingPlugin.groovy
@@ -64,7 +64,7 @@ class NexusStagingPlugin implements Plugin<Project> {
     }

     private void failBuildWithMeaningfulErrorIfAppliedNotOnRootProject(Project project) {
-        if (project != project.rootProject) {
+        if ((project != project.rootProject) && (!(project.projectDir.absolutePath =~'([\\\\/])build\\1tmp\\1generatePrecompiledScriptPluginAccessors\\1'))) {
             throw new GradleException("Nexus staging plugin should ONLY be applied on the ROOT project in a build. " +
                 "See https://github.com/Codearte/gradle-nexus-staging-plugin/issues/47 for explanation. Feel free to comment there if you really" +
                 "need to have it applied on subproject.")

This will recognize that it is applied in the static accessors determination step and will not emit the error.
With this single change, it works perfectly fine and still has the safe-guard for the actual project.

You probably agree that it is a "hack" which is also somehow fragile as it depends on Gradle "internals". You hit two issues in Gradle and I'm reluctant to add that "hack" to the project to mitigate it (I still plan to add -P).
As it is not a critical issue (a build goes on), I propose you to live with it until the Gradle team fix it or use a Jitpack version made code modified by you for you experiments. Would it be sufficient for you?

Is there a way in your build to publish to mavenLocal()?
Sure. I use the old good maven plugin for publishing. Therefore, gw install is something that you're looking for.

Ehhh, I have already some Nexus related hacks to one more should not be a big problem :).
Please provide a PR with:

the hack extracted to a separate method with a sensible name (and a link to the issue in a comment)
a functional test (with an @Issue anotation) to detect regressions in the future Gradle versions (you can try to use ProjectBuilder (like in LegacyTasksSpec) to reproduce the problem, but I don't know if it operates at that level; alternatively add a test to GradleVersionFuncSpec with some basic setup - I'm curious if Kotlin configurations can be defined inline with nebula-test).

I think I got it more or less setup, except for one thing.
The buildSrc build in the tested project does not compile because it does not know the plugin by id.
Can you tell me how I make it that it gets to know the plugin?

I don't think this is gonna work.
As far as I can see there is no way the nebula-test can inject the plugin classes into the buildSrc build.
It configures the buildscript classpath using an init script, but init scripts that are given as commandline option or similar are only effective on the main build, not on the buildSrc build. (There is an open issue for that)
The nebula-test plugin would need to use a custom Gradle distribution with the init script in there to be effective in the buildSrc build also.
And even then I'm not sure whether it would work as for this use-case it is necessary to apply the plugin using the plugins-DSL block which I think would not work that way.
So to make this work, I think the nebula-test plugin would need to build a maven repository with the built artifact and configure that maven repository as plugins repository via an init script in a custom gradle distribution and as far as I can see, it is not capable of that, so how do you think we should handle this situation?

I haven't tried to apply my plugin in buildSrc before, especially under test.
As I proposed in the related issue it would be good to get know if TestKit supports it at all. If not, we can probably only report a feature request for that (I wonder how they test it themselves?) and pass the change as is. However, it would mean that you are the only person who knows if it works and in addition there will be no failing tests once it is broken in some future Gradle version. Not the perfect solution, but at least you and your project will be happy ;).

Well, I don't know whether testkit supports it.
One could probably do some other fancy integration testing with first deploying the plugin to a local maven repository and then configure it as plugin repository like suggested in that related issue.
But as you seem to be ok with only me complaining when it fails (or others that adapt precompiled script plugins), I'll provide a PR that adds the work-around without test for now and hope for a fast release. :-)

Here you have it: #117 :-)

@Vampire Please check 0.21.0.

And by the way I implemented also #116 for others.


Inferences: select spatial clustering algorithm / library

We'll give it a try with that one: https://github.com/uhho/density-clustering


Why --save instead of --save-dev?

In instructions on how to use @types, type packages are installed with --save and not --save-dev.
Why is this?
I wouldn't expect that I need the types outside of my development environment?
There should be a note about it in the various readmes.

/cc @DanielRosenwasser

The short answer would be for module authors - if, as an author, you want to publish your TypeScript package to NPM and the dependencies are in the development section, no one will be able to install it and use it without having to manually install your types dev dependencies. By using dependencies, they will be installed along with the package and consumers can use your NPM package without additional configuration.

Using --save-dev is fine when you're writing a simple application, and it won't be used as a library. The problem comes along when you might have dependencies. If you stored your type declarations in your devDependencies, your consumers would not automatically get your type declarations. They'd then have to manually add your declaration file dependencies as their devDependencies.
Given that breaking consumers is a worse problem than slightly-larger packages, we've made --save the default in our documentation.

If the types are exposed as dependencies, then it should registered as dependencies. Otherwise, devDependencies.
But agree with @DanielRosenwasser that sometimes this distinction is not easy to make, especially for JS developer, go defaulting to dependencies is the better option.

I've been basically taking the same road as @unional. Like I'll do
npm install --save-dev @types/tape
or
jspm install --dev npm:@types/tape 😛

Hi,
I was looking trough the code some of some big projects using Typescript and none of them was using dependencies for @types:

Typescript
Types Publisher
Angular

Is this because these projects are special ? Angular is more front-end oriented and types-publisher is a standalone project (I think), but Typescript can be imported as a library and consumed by others: why aren't some of its @types normal dependencies (like node) ?

@demurgos With respect to TypeScript itself, it only uses declaration files for its build tools (like gulp utilities). It doesn't actually require those tools to run, so it doesn't need the declaration files for those tools either.

Thank you. So you mean that Typescript never require any of the Node's core libraries during runtime ? (outside of their tests/build process)
Basically, I am having trouble to know if I should install environmental types like @types/node as dependencies or devDependencies ? Should I list them as devDependencies at let the user provide them or should I list them as dependencies (and deal with #173) ?

My thought is dependencies since you're written as a library with dependents. For any users using Typings with TypeScript 1.8 and below, they shouldn't be affected, but as you noted, 2.0+ users may run into problems (at which point the solution is trivial but potentially annoying).

Wouldn't that cause trouble when two dependencies depend on different version of @types/node? We're currently looking to use @types in gulp-typescript, and I'm afraid that this could cause issues for a lot of people, since the types of Node are polluting the global namespace. Basically there are two scenarios that could give issues: (1) a project depends on gulp-typescript (which depends on @types/node@1) and on @types/node@2, (2) a project depends on gulp-typescript (which depends on @types/node@1) and on foo, which depends on @types/node@2. NPM will install the types of node twice. Will this cause duplicate-identifier errors?

You could use a semver such as >= 4 to indicate compatibility with future node versions.

Circling back to what @ivogabe asked - will TypeScript handle conflicting types in a way that works out okay? If I have packages depending on different versions of @types/react for example, can TS only apply those types where correct (based on their location in the node_modules tree? If it's not, it would be great to have a short write-up somewhere on the best way for library authors to include @types dependencies, I've heard a lot of questions about this from around my company.

I have packages depending on different versions of @types/react for example, can TS only apply those types where correct (based on their location in the node_modules tree?

Add to that the even harder part is about module augmentation. Which version of react to augment?

@JKillian I believe global definitions can and will conflict, while external module definitions follow module resolution. This is a bit of a pain point, because there's no guarantee a module in @types is external or global and that it won't change suddenly (lodash did this and generated a ton of build errors on various projects). I feel like there are better ways to solving this, but nothing that wouldn't break now.
@unional I think that one should be fine. You can kick in and use peerDependencies if you need to augment the parent. Type dependencies should mirror the regular dependencies for this reason.

For react case, yes, because likely there are no direct dependency between react and the authoring package (e.g. react-redux).
But what if there is such a dependency? i.e.:
- app
  - package-a
    -<EMAIL_ADDRESS>  - package-c
    -<EMAIL_ADDRESS>  - package-b@??

And package-a and package-c both make an augmentation to package-b.

Something I don't quite understand here:
Quoting @blakeembrey

if, as an author, you want to publish your TypeScript package to NPM and the dependencies are in the development section, no one will be able to install it and use it without having to manually install your types dev dependencies

But if I'm writing a library and I use lodash, that doesn't mean a developer using my library needs the lodash types at all. Wouldn't I just do yarn add -D @types/lodash? They won't be using lodash in relation to my library, and if they are using lodash in their own project, they could just add the types. Or are you saying my library would suddenly break, even though there are no direct references to @types/lodash? (it's used implicitly)

@ffxsam There's currently no good tooling that'll tell you there's no direct references to lodash types. It's quite possible that, via type inferencing, you've used a type from a dependency in a return position and now you have a transitive dependency on it. If you're 100% sure you don't use it, and you may want to use a tool to check this via emitted .d.ts files, that's great 👍
For my own experience, I've had a lot people make the implicit assumption you've made try to remove dependencies via a PR that I've purposefully listed in my dependencies. E.g. most recently https://github.com/serviejs/popsicle/pull/128.

For @types/react in packages that put react itself in peerDependencies, does it make more sense to put @types/react in peerDeps instead of deps?


Install automatic change log generator

E.g., skywinder. Relies on issues and pull requests.

Strictly following the issues and PR  protocol is too cumbersome, at least for now.


GRAVE: Cannot parse file: *.lxml_outline.xml

Dear Grobid people,
I am using Grobid in the context of the French ANR to retro-process the final reports of funded projects (around 20,000 projects so far).
I installed grobid-0.6.1 and compiled it on my Mac 10.14 without any problem. (I am using OpenJDK Runtime Environment, build 15.0.1+9)
Using the installed image, without any change, I am running the following commands:
unzip grobid-0.6.1.zip
cd grobid-0.6.1
./gradlew clean install
java -Xmx4G -jar grobid-core/build/libs/grobid-core-0.6.1-onejar.jar  -gH grobid-home \
            -dIn ../Input -dOut ../Output -exe createTraining

I works perfectly when only one PDF file is in directory ../Input.
However, if there are two files in the directory, then I get "GRAVE" errors.
févr. 12, 2021 3:20:44 PM org.grobid.core.document.Document addTokenizedDocument
GRAVE: Cannot parse file: my_dir/grobid-0.6.1/grobid-home/tmp/ZW3PpM4E9g.lxml_outline.xml

févr. 12, 2021 3:20:44 PM org.grobid.core.document.Document addTokenizedDocument
GRAVE: Cannot parse file: my_dir/grobid-0.6.1/grobid-home/tmp/ZW3PpM4E9g.lxml_outline.xml

févr. 12, 2021 3:20:46 PM org.grobid.core.engines.ProcessEngine createTraining
INFOS: 2 files processed.

Is it normal?
Regards, and thanks for your excellent software, Luc.

Hello Luc !
Well it's not that "GRAVE", simply one of the generated XML file resulting from the PDF parsing which is not XML valid (very frequent) - it has no impact because the outline file (containing the table of content outline if available embedded in the PDF) is not exploited by GROBID for the moment - it's to allow some possible improvements in the future. The error is more a reminder for the developers... the XML parser that classifies it as "GRAVE" but it would be rather INFO for us.

Thanks for your answer.
Unfortunately, a side effect might have been overlooked.
At org.grobid.core.sax.PDFALTOOutlineSaxHandler.startElement(PDFALTOOutlineSaxHandler.java:98)
it is written
``
nodes.put(currentId, currentNode);
if (currentParentId != -1) {
DocumentNode father = nodes.get(currentParentId);
if (father == null)
System.out.println("Warning, father not yet encountered! id is " + currentParentId);
currentNode.setFather(father);
father.addChild(currentNode);
} else {
// parent is the root node
currentNode.setFather(root);
root.addChild(currentNode);
}
Thus, if `father` is null, then `father.addChild(currentNode)` is called with a `nullPointer` exception.

This exception is caught by `catch (Exception e)` at line 372 in `grobid-core/src/main/java/org/grobid/core/document/Document.java` where the error message "Cannot parse" is misleading.

I think an addition `else` is just missing. Let me post a push request.

Well, I am not git-wizard enough to post the pull request. Here is the file.
PDFALTOOutlineSaxHandler.java.txt

@lucbouge You can edit the file directly from the github interface and save it as pull request: https://docs.github.com/en/github/managing-files-in-a-repository/editing-files-in-your-repository

Thanks. Let me do it at once. Luc.


(on bevy release) Upgrade to support Bevy 0.15 if feasible

The underlying asset system changed quite a lot.
It's hard to keep up if you work with Bevy only occasionally, esp as library author.
I wonder if I should even hold of for longer until the project is settled even further.

update: bevy 0.15 finally released


How to provide email address?

How do I provide my email address at issue time, in order to subscribe to Let's Encrypt expiration notices?
The script failed for me recently since the agreement PDF had changed, and I had no idea the certificate was not renewing until it expired.

Dupe of #130


Added link for Istio-Initializer in quick start.

Jenkins job istio.github.io/presubmit passed

@frankbu I fixed his link..

Jenkins job istio.github.io/presubmit passed


spec_helper improvements

Hi,
we decided to integrate your gem in our system and along the way we hope to contribute a little. We started by adding support for auto migrating the database whenever the test suite is executed. We also removed .rvmrc and .rspec from the repository because, in our opinion, these two files only contain developer specific settings and should therefore not be tied to the entire project.
Kind regards,
the flinc dev team

thanks for the patch!
I've pulled in several of your contributions and appreciate the work.  there are a few remaining changes that i'm going to hold of for now.  i prefer the .rvmrc file being checked in to ensure that the project is guaranteed to pass under known conditions.  what ruby version do you use with your app?  I'm considering updating the required version to 1.9.x.
removing the .rspec file is okay since that is clearly developer specific syntax suger.