by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 01:14
    cvvergara synchronize #1488
  • 01:14

    cvvergara on translations_support-po--master_es

    Apply translations in es trans… Merge branch 'master' into tran… (compare)

  • 00:45
    MarPetra milestoned #1488
  • 00:45
    MarPetra labeled #1488
  • 00:45
    MarPetra assigned #1488
  • 00:45
    MarPetra review_requested #1488
  • 00:44

    MarPetra on translations_vrp-category-po--master_es

    (compare)

  • 00:44

    MarPetra on master

    Apply translations in es trans… (compare)

  • 00:44
    MarPetra closed #1487
  • 00:44
    transifex-integration[bot] opened #1488
  • 00:44

    transifex-integration[bot] on translations_support-po--master_es

    Translate /locale/en/LC_MESSAGE… (compare)

  • 00:44

    transifex-integration[bot] on translations_support-po--master_es

    (compare)

  • 00:40
    MarPetra review_requested #1487
  • 00:36
    MarPetra milestoned #1487
  • 00:36
    MarPetra labeled #1487
  • 00:36
    MarPetra assigned #1487
  • 00:36
    MarPetra review_requested #1487
  • 00:35
    transifex-integration[bot] opened #1487
  • 00:35

    transifex-integration[bot] on translations_vrp-category-po--master_es

    Apply translations in es trans… (compare)

  • 00:35

    transifex-integration[bot] on translations_vrp-category-po--master_es

    (compare)

Vicky Vergara
@cvvergara
CREATE TABLE foobar 
WITH _src(seq1, id1) AS(
    SELECT nextval('serial1'),  id FROM sample1 limit 100
), _dest(seq2, id2) AS(
    SELECT nextval('serial2'),  id FROM sample2 limit 100
)
    SELECT id1 AS source, id2 AS target
    FROM _src, _dest
    WHERE seq1=seq2;

SELECT * FROM pgr_dijkstra(
$$SELECT * FROM edge_table$$, -- edges sql query
--- the combinations query
$$SELECT * FROM foobar$$);
Vicky Vergara
@cvvergara
Plan:
  • pgRouting 3.0.0 final release must be done before June starts
  • meanwhile, MobilityDB team works on adding the new pgr_dijkstra overload function
When its ready, and "tested" incorporate to pgRouting and make a 3.1 release with the new function
as experimental
Vicky Vergara
@cvvergara
array[1] ARRAY [3,2] parallel
1 | 2
1 | 3
SELECT source, target FROM foo;
ARRAY[1,2,3,4,5 ..... 1000000] ARRAY[1000000, ...... 5,4,3,2,1]
SELECT source, target FROM foo WHERE target IN (3);
999998, 3
Vicky Vergara
@cvvergara
SELECT ARRAY[1,2,3,4,5 ..... 1000000] AS sources, ARRAY[1000000, ...... 5,4,3,2,1] as targets
Ashish Kumar
@krashish8
When are we having the next meeting? I'm sorry I didn't listen.
Is it Tomorrow Tuesday (19 May), 2pm UTC = 9am in Mexico = 7:30pm in India? @cvvergara
Vicky Vergara
@cvvergara
I didn't set up a date, but just be attentive for the last minute annoucement
Ashish Kumar
@krashish8
Okay. Thanks!
Vicky Vergara
@cvvergara
@krashish8 @rajhim2 @prakashupes please review pgRouting/pgrouting#1349
Vicky Vergara
@cvvergara

Topic: updating your working branch

The master branch has been updated, so you need to update the working branch

  • work on your GSoC-pgRouting clone
  • suppose remotes are:
    • origin your GSoC-pgRouting fork
    • upstream pgRouting repo
    • gsoc the GSoC-pgRouting repo

Steps:

  • If you haven't started coding:

    git fetch gsoc
    git checkout gsoc/<your branch> 
    git checkout -b <branch name>
    git fetch upstream
    git rebase upstream/master
    git push --set-upstream origin <branch name>

    And make a PR from <branch name> to gsoc/<your branch>

  • If you already started coding on branch: <branch name>

    git checkout <branch name>
    git fetch upstream
    git rebase upstream/master
    git push

    And make a PR from <branch name> to gsoc/<your branch>

I will check those PR (in my) tonight

end of topic

Vicky Vergara
@cvvergara
@rajhim2 why are you reverting the merge of the PR on the main repository
Ashish Kumar
@krashish8
@cvvergara I think we should force push in case "If you already started coding on branch: <branch name>"
git push -f
This is because rebasing would reorder the commits history, therefore the remote branch can't be fast-forwarded to local, without a force push.
Vicky Vergara
@cvvergara
yeah I forgot the -f, but that doesnt try to revert the master branch of the main repository
@krashish8 can you make the PR
Ashish Kumar
@krashish8
@cvvergara Done!
Vicky Vergara
@cvvergara
Thanks, The merge will be done before mailing a report next weekend
The opened PR will allow to follow your work
Ashish Kumar
@krashish8
Okay, understood!
Himanshu Raj
@rajhim2

@rajhim2 why are you reverting the merge of the PR on the main repository

It was clicked by mistake mam.

Ashish Kumar
@krashish8
@cvvergara There are some extra commits made by transfix-integration [bot]. So, we should rebase the master branch of pgrouting repository again, right? (repeating the above steps)
Himanshu Raj
@rajhim2
Can anyone please explain me why is this build failing. https://travis-ci.org/github/rajhim2/GSoC-pgRouting/jobs/689126458 This repository is on my profile rajhim2.
Himanshu Raj
@rajhim2
The command "if [ "$DOCUMENTATION" == "OFF" ] && [ "$BUILD_TYPE" == "Debug" ]; then bash ./ci/travis/doc-quieries_test.sh $POSTGRESQL_VERSION postgres; fi" exited with 1.
Daniel Kastl
@dkastl
Thank you @cvvergara !
Next I think it's on my to encourage a few more pgRouting businesses to financially contribute:
https://funding.communitybridge.org/projects/pgrouting
Rajat Shinde
@omshinde
Thank you @cvvergara . Congratulations to the team. :)
Vicky Vergara
@cvvergara
Next meeting: tuesday May 26 at 2pm UTC = 9am in Mexico = 7:30pm in India
Himanshu Raj
@rajhim2
Why the same build passes on appveyor and fails on travis-ci.
Ashish Kumar
@krashish8

Why the same build passes on appveyor and fails on travis-ci.

@rajhim2 I don't have any idea how it passes on appveyor, however, there's an error in your added files.

In the file src/kargersContraction/kargersContraction.c, the function's name shall be in all lowercase, as PostgreSQL converts all unquoted names to lowercase (_pgr_kargerscontraction instead of _pgr_kargersContraction).

You need to change this in all the three lines:

https://github.com/rajhim2/GSoC-pgRouting/blob/him/src/kargersContraction/kargersContraction.c#L43
https://github.com/rajhim2/GSoC-pgRouting/blob/him/src/kargersContraction/kargersContraction.c#L44
https://github.com/rajhim2/GSoC-pgRouting/blob/him/src/kargersContraction/kargersContraction.c#L116

Himanshu Raj
@rajhim2

Why the same build passes on appveyor and fails on travis-ci.

@rajhim2 I don't have any idea how it passes on appveyor, however, there's an error in your added files.

In the file src/kargersContraction/kargersContraction.c, the function's name shall be in all lowercase, as PostgreSQL converts all unquoted names to lowercase (_pgr_kargerscontraction instead of _pgr_kargersContraction).

You need to change this in all the three lines:

https://github.com/rajhim2/GSoC-pgRouting/blob/him/src/kargersContraction/kargersContraction.c#L43
https://github.com/rajhim2/GSoC-pgRouting/blob/him/src/kargersContraction/kargersContraction.c#L44
https://github.com/rajhim2/GSoC-pgRouting/blob/him/src/kargersContraction/kargersContraction.c#L116

Thank you @krashish8 ! It worked like a charm.

Himanshu Raj
@rajhim2
Mahmoud will open 1 issue:
For the functionality request of the pgr_dijkstra with the combinations sql
the issue has an example
step 2) Other issue (maybe there is already one that talks about using ctlr-C and in the comment put the possible solution.
Then once we located both issues, Mahmoud will open a PR with tilte "Closes #xx and closes #yyy" with a small comment on the message
After that, @cvvergara will open an issue about using the method for stoping execution on each function.
Himanshu Raj
@rajhim2
Open issues about the code that Mahmoud made
For example: "Normal" flag is not needed
other example: To create pgr_dikstraCost with combinations
Himanshu Raj
@rajhim2
doc/src/pgRouting-introduction.rst
Vicky Vergara
@cvvergara

Topic: @mahmsakr preparation for PR

Once that

  • Contributor name has been added to modified files
  • Contributor name is on the license of new files
  • Contributor name is been added to the Contributors List

From the console terminal:

# make sure that you are in the correct branch
git checkout combinationsSQL
# update to latest main repo contents
git fetch upstream
# Put your work on top of what develop
git rebase upstream/develop

Once you have updated the repo, the following that I forgot to mention on our last meeting must be done:

  • in doc/src/release_notes.rst in the corresponding 3.1.0 release notes add

    .. rubric:: New functions
    
    * pgr_dijkstra(combinations)
  • Commit and push

From the github website:

  • make a PR to develop branch

    end of topic

Vicky Vergara
@cvvergara

@rajhim2

Why the same build passes on appveyor and fails on travis-ci.

appveyor does not perform the tests it only compiles the tests travis besides compiling does the testing.