Skip to content
This repository has been archived by the owner on Dec 22, 2021. It is now read-only.

Create the functions kernel for gravmag.prism #99

Closed
wants to merge 14 commits into from
Closed

Create the functions kernel for gravmag.prism #99

wants to merge 14 commits into from

Conversation

birocoles
Copy link
Member

Update by @leouieda:

Checklist:

  • Cookbook recipes for new functions (if intended for user, not developer)
  • Docstrings
  • Doctests and unitests
  • All tests pass
  • Can be merged
  • Changelog entry
  • Benchmark against old implementation

@birocoles
Copy link
Member Author

@leouieda,

Eu fiz as funções kernel e deixei o resto como ta, isto é, não criei as funções gxx, gxy, etc. Eu fiz assim porque isso implica em mudar umas coisas q eu não sei.

É nóis na fuga!

@leouieda
Copy link
Member

@birocoles você por um acaso não esqueceu de criar esse branch prisma a partir do master e acabou criando a partir de outro? Por isso que tem que checar o git log.

@birocoles
Copy link
Member Author

@leouieda, sei lá q q eu aprontei. Devo ter feito merda quando eu fui atualizar meu repositório local. Achei estranho esse troço de blocky-inversion aparecer aqui, mas achei que fosse normal. Eu pensei em fazer o seguinte:

  1. Apagar o branch prisma do meu local
  2. Criar um branch prisma a partir do meu master
  3. Fazer o pull request de novo

Qure que vc acha?

@leouieda
Copy link
Member

Não precisa apagar o branch. Crie outro com outro nome a partir do master,
feche esse pull request e abra outro.

2014-04-29 11:46 GMT-03:00 Vanderlei C Oliveira Jr <[email protected]

:

@leouieda https://github.com/leouieda, sei lá q q eu aprontei. Devo ter
feito merda quando eu fui atualizar meu repositório local. Achei estranho
esse troço de blocky-inversion aparecer aqui, mas achei que fosse normal.
Eu pensei em fazer o seguinte:

  1. Apagar o branch prisma do meu local
  2. Criar um branch prisma a partir do meu master
  3. Fazer o pull request de novo

Qure que vc acha?


Reply to this email directly or view it on GitHubhttps://github.com/leouieda/fatiando/pull/99#issuecomment-41684874
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants