Closed martinfleis closed 1 year ago
I would also eventually love to see mgwr accepting pandas objects and deriving the names from those (xref https://github.com/pysal/spreg/issues/127) but let's go step by step.
I think this is great to have in MGWR, so I am going to merge this PR.
@weikang9009 thanks! Can we cut a release with this, pretty please? I'd like to include it in my teaching materials 😇.
Mmm, I suppose we should wait for #131 and new release infra. Need to nag @sjsrey to give @jGaboardi permissions :)
pysal/steering
now should have admin rights on this repo.
@sjsrey Thanks! Confirmed for myself.
@TaylorOshan, @Ziqi-Li any chance to cut a release with this? I'd like to include it in a class I'm teaching next Tuesday. If not that is fine as well but this is one of the tiny things that can make a difference when explaining what is going on.
@TaylorOshan, @Ziqi-Li any chance to cut a release with this? I'd like to include it in a class I'm teaching next Tuesday. If not that is fine as well but this is one of the tiny things that can make a difference when explaining what is going on.
Sounds good to me.
@TaylorOshan do you want to do it yourself, or you want me to do it?
I always thought we had this already but maybe that’s in the GUI?
Feel free to go for it! @Ziqi-Li @martinfleis @jGaboardi
I always thought we had this already but maybe that’s in the GUI?
Feel free to go for it! @Ziqi-Li @martinfleis @jGaboardi
Yeah, I also have that impression somehow... the GUI has it for sure.
Well, it wasn't here :). Where does the GUI live anyway?
@Ziqi-Li can help with the release somehow?
Also, seems like we never got around to setting up a release action for mgwr
.
Well, it wasn't here :). Where does the GUI live anyway?
@Ziqi-Li can help with the release somehow?
The GUI code is here: https://github.com/Ziqi-Li/MGWR-GUI
Sure, I will work on the release today.
@Ziqi-Li Thanks!
@Ziqi-Li can help with the release somehow?
I meant to write "Can I help..." :D
This is the code implementing the same in the GUI
@Ziqi-Li Thanks!
@Ziqi-Li can help with the release somehow?
I meant to write "Can I help..." :D
:D. I just released the v2.2.0 on pypi @martinfleis @TaylorOshan
spreg allows passing
name_x
with a list of names of independent variables. mgwr does not, resulting in X0...Xn in the summary. With a larger number of variables, it gets tricky to link the two, especially when there's an intercept included (but not marked).I am mirroring the behaviour of spreg here and adding optional
name_x
, that gets used in summary.As far as I can tell, there are no tests for
summary
so nowhere to add tests for this.Examples:
Current behaviour:
New: