Add support for XP-Pen Artist 16 #3276
Conversation
There was a problem hiding this comment.
We also strongly prefer a diagnostic be provided as per our CONTRIBUTING.md when adding support for tablets. Please provide the file generated from help > export diagnostics
| "FeatureInitReport": null, | ||
| "OutputInitReport": null, | ||
| "DeviceStrings": {}, | ||
| "InitializationStrings": [ |
There was a problem hiding this comment.
Are you sure you need every string here for full functionality?
Judging from other tablets only 100 should be required, 123 may be required for aux functionality, could you provide a string dump from the inbuilt device string reader?
There was a problem hiding this comment.
Yes. I tried removing every single one of them. 123 is to switch to the correct aux button format but that also seems to mess up the dpi. It sometimes works if the 100 is removed but not reliably. These were captured using usbmon from the Windows driver running inside a virtual machine.
There was a problem hiding this comment.
The OEM driver will throw strings that the tablet does not need and are simply denied by the tablet.
put 21827 in the top, 77 in the middle and ignore the bottom box, dump all.
There was a problem hiding this comment.
Try only 100 and 123, replug your tablet to reset the initialization process.
| "Digitizer": { | ||
| "Width": 340.99, | ||
| "Height": 191.81, | ||
| "MaxX": 34416, |
There was a problem hiding this comment.
Are you sure these are right? If these are the highest values you can get you should adjust the width and weight to reflect these properly.
max value/lpmm = physical size
the lpmm for this tablet is 100 and the max value is what you can measure in the tablet > tablet debugger when moving your pen as far as you can get it to read.
There was a problem hiding this comment.
The MaxX and MaxY are from the tablet debugger. The width and height are copied from a different 16" tablet.
There was a problem hiding this comment.
You should calculate the values using the formula provided and use those instead.
| "MaxY": 19359 | ||
| }, | ||
| "Pen": { | ||
| "MaxPressure": 2047, |
There was a problem hiding this comment.
XP-Pen likes to hide old tablets specification pages so I can't compare this alongside a spec sheet at this time, is this the value you got when you measured it in tablet > tablet debugger?
There was a problem hiding this comment.
MaxPressure is also the highest value seen in the tablet debugger
| "ProductID": 77, | ||
| "InputReportLength": 8, | ||
| "OutputReportLength": 0, | ||
| "ReportParser": "OpenTabletDriver.Configurations.Parsers.XP_Pen.XP_PenOffsetAuxReportParser", |
There was a problem hiding this comment.
Are you sure this is the right parser? Its currently only used on the UGEE M708 (UGEE and XP-Pen are close companies so its fine if this is the right parser)
The only difference is where it reads auxiliary data from, so if it reads fine this is the right parser.
There was a problem hiding this comment.
Yes. With other parsers the aux buttons don't work
|
Functionality of this tablet will be spotty until #3245 is merged. It currently has a 1/3 chance of working, with invoking detection and hoping you hit a 1/3 being the only way to make it work. |
jamesbt365
added
configuration
No description provided.