THREE.MeshLine icon indicating copy to clipboard operation
THREE.MeshLine copied to clipboard

Maintainers

Open dannoshepard opened this issue 3 years ago • 12 comments

Where are the maintainers on this project?

There are several outstanding merge requests that need to go into the next release.

Specifically the fix in the vertex shader and typescript support

dannoshepard avatar Jun 14 '21 17:06 dannoshepard

https://github.com/utsuboco/THREE.MeshLine I made a new version of a library using vite and typescript.

Temporary I published it on npm under the name meshline. It should work the same way, and it now exports es(2.89kb brotli), umd(3.13kb) and has the ts definitions.

see https://github.com/spite/THREE.MeshLine/issues/139#issuecomment-842851031

Demo: https://codesandbox.io/s/confetti-typescript-meshline-0f4mi?file=/src/index.tsx

RenaudRohlinger avatar Jun 15 '21 15:06 RenaudRohlinger

I guess the author might want to deprecate this project. I sent links to him on Twitter twice, both ignored. while he is definitely active on Twitter and on GitHub. Just sad he didn't assign another maintainer for this project.

tiye avatar Jun 16 '21 02:06 tiye

There is another maintainer. @jiyinyiyong do you want to be a maintainer?

spite avatar Jun 19 '21 21:06 spite

sorry I got too many projects... how about @RenaudRohlinger ?

tiye avatar Jun 20 '21 03:06 tiye

I would propose moving this library to https://github.com/pmndrs @RenaudRohlinger is a member but the point of it is that critical libraries can be maintained as a collective. We have helped multiple libs already that otherwise would have been abandoned like https://github.com/pmndrs/cannon-es or https://github.com/pmndrs/detect-gpu

The author remains, it's still yours @spite but we take liberty in maintaining, inviting more devs to join as well as keeping it up to date with tooling, types and so on. this way something so useful like meshline doesn't outdate as long as people are relying on it and have a vested interest in keeping it alive.

drcmda avatar Jun 21 '21 04:06 drcmda

@spite can we move it to https://github.com/pmndrs?

robksawyer avatar Dec 23 '21 14:12 robksawyer

~~@spite I have some time now. guess I could help migrating this lib to ES Modules.~~

forked my own version of the package... https://github.com/Quatrefoil-GL/meshline .

tiye avatar Mar 13 '22 12:03 tiye

any updates on this?

alexander-macleod avatar May 02 '22 19:05 alexander-macleod

https://github.com/utsuboco/THREE.MeshLine I made a new version of a library using vite and typescript.

Temporary I published it on npm under the name meshline. It should work the same way, and it now exports es(2.89kb brotli), umd(3.13kb) and has the ts definitions.

see #139 (comment)

Demo: https://codesandbox.io/s/confetti-typescript-meshline-0f4mi?file=/src/index.tsx

For those that come later, after trying many forks/permutations to get MeshLine working with the latest three/react-three-fiber/drei stack (at time of writing) and typescript, @RenaudRohlinger version worked for me.

Note that there are some critical changes from the original r3f Fatline demo code, base your code off the example given in the quoted reply and/or the forked repo README--NOT off the original r3f demo! (I burned an hour to a bug caused by not noticing this.) In particular, vertices is switched for points.

The components will give TS errors about not being on JSX.IntrinsicElements. To fix and make full use of the awesome TS support in this fork, this worked for me:

import {
  extend, ReactThreeFiber,
} from '@react-three/fiber';
import { MeshLine, MeshLineMaterial, MeshLineRaycast } from 'meshline';

extend({ MeshLine, MeshLineMaterial });

/* eslint-disable no-unused-vars */
declare global {
  namespace JSX {
    interface IntrinsicElements {
      'meshLine': ReactThreeFiber.Object3DNode<MeshLine, typeof MeshLine>;
      'meshLineMaterial': ReactThreeFiber.Object3DNode<MeshLineMaterial, typeof MeshLineMaterial>;
    }
  }
}
/* eslint-enable no-unused-vars */

Which owes credit to: https://github.com/pmndrs/react-three-fiber/issues/130#issuecomment-604502461

HUGE THANK YOU TO THIS LIB AND TO RenaudRohlinger. Happy lining!

bryantcodesart avatar Aug 08 '22 16:08 bryantcodesart

@spite @bryantcodesart @robksawyer

@RenaudRohlinger has agreed to move it over to pmndrs https://github.com/pmndrs/THREE.MeshLine this way more devs will be able to maintain and merge prs.

i've opted to make some slight breaking changes and release a major: https://github.com/pmndrs/THREE.MeshLine/pull/4

examples are still open, it seems the demos that were in the repo don't work any longer https://github.com/spite/THREE.MeshLine/issues/154 if i find time i can try to fix them up.

drcmda avatar Dec 07 '22 01:12 drcmda

Awesome! Thanks for following up.

robksawyer avatar Dec 07 '22 01:12 robksawyer

Is it intentional that the pmndrs repo has issues disabled?

eulertour avatar Dec 29 '22 15:12 eulertour