Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 05 22:48
    gamma-02 commented #1148
  • Oct 04 07:46
    Aholicknight commented #1148
  • Oct 02 22:43
    gamma-02 commented #1148
  • Oct 02 22:43
    gamma-02 commented #1146
  • Oct 01 17:32
    GA-MV commented #1148
  • Sep 23 21:42
    TheRockSaysOP starred saros-project/saros
  • Sep 20 23:26
    Aholicknight commented #1148
  • Sep 20 23:21
    Aholicknight labeled #1148
  • Sep 20 23:21
    Aholicknight labeled #1148
  • Sep 20 23:21
    Aholicknight opened #1148
  • Sep 15 19:16
    EnderCrypt commented #1142
  • Aug 20 02:46
    jgstern starred saros-project/saros
  • Aug 01 12:44
    tobous synchronize #1147
  • Aug 01 12:44

    tobous on build

    [DOC] Adjust documentation for … (compare)

  • Aug 01 12:34
    tobous synchronize #1147
  • Aug 01 12:34

    tobous on build

    Reformat using GJF Jar to resol… (compare)

  • Aug 01 11:40
  • Aug 01 11:34
    tobous synchronize #1147
  • Aug 01 11:34

    tobous on build

    [BUILD] Commit automatic Intell… Reformat project using GJF 1.10… [BUILD] Switch to different GJF… (compare)

  • Aug 01 11:20
    tobous labeled #1147
Tobias Bouschen
@tobous
( I only had a glance at the result. There actually were only two other tests, which is still in a normal range for the STF (sadly). But checking to make sure shouldn't hurt.)
Tobias Bouschen
@tobous
Ok, it really seem like the other two failures were just random. In a follow-up run, only a the version test failed: https://scans.gradle.com/s/omxskuqfazcga/tests/by-project.
Kelvin Glaß
@m273d15
Hi, I created a new version of the Saros page: https://m273d15.github.io/saros/
If you have suggestions for improvement let me know. My TODOs are: Choose a less aggressive blue, fix broken links, fix table borders.
Tobias Bouschen
@tobous

Hi, I created a new version of the Saros page: https://m273d15.github.io/saros/
If you have suggestions for improvement let me know. My TODOs are: Choose a less aggressive blue, fix broken links, fix table borders.

@m273d15 My list got to long, so I decided to create an issue for it: saros-project/saros#896

Kelvin Glaß
@m273d15
The saros page is currently down. I am currently fixing the issue.
Tobias Bouschen
@tobous
Currently, everything but the landing page seems to work for me.
Kelvin Glaß
@m273d15

The landing page is back.

I created multiple ugly commits to master.
It seems like the issue was: the CNAME file in the master AND gh-pages.
I try to create a better fix than adding the CNAME file manually to the gh-pages.

Tobias Bouschen
@tobous
Can't you simply hard coded create the cname as part of the page build?
Kelvin Glaß
@m273d15
Yes, I created a corresponding PR #920
Tobias Bouschen
@tobous
If anybody is interested, I would be open to talking through/pair-reviewing the big refactoring in saros-project/saros#934 and saros-project/saros#938.
Tobias Bouschen
@tobous
FYI, I broke the STF on the master as I forgot to to update the IntelliJ version used in the docker file. I thought it was no longer used, but it is still used for the STF setup. @m273d15 Could we also adjust the STF setup to unify the handling of the used build verion?
Tobias Bouschen
@tobous
And from recent build results, it seems like EditDuringInvitationStressTest.testEditMultipleClassesDuringInvitation() can fail more than 4 times in a row when run as part as the GitHub action (see build1 and build2). This is also the test that caused the majority of STF failures in the past. So maybe we should mark this test as flaky as well.
Kelvin Glaß
@m273d15

Could we also adjust the STF setup to unify the handling of the used build version?

I opened a corresponding PR #951

Tobias Bouschen
@tobous
Yeay, seems like I broke the STF again (this time by breaking the entire Eclipse plugin). Looking into it.
Tobias Bouschen
@tobous
This issue was caused by #953 creating the new package saros.filesystem.checksum. I was not aware that all packages from the core must be exported in the MANIFEST.MF to be usable in the Eclipse plugin. Fixed with #970.
Lex Vorona
@voronaam
Hello! I am looking to organize some pair programming (Java) and Saros looks perfect. It has all the important features we were looking for and I only have a few questions around some of the advanced features we might want to have in the future.
Totally ok if you do not have them yet, just curious if you have plans for them.
Do you have or have plans for:
  • "Shared port", ability to tunnel a specific port, such that participants can execute REST calls against the app running on the HOST's machine
  • "Shared terminal", while risky from the security point, it might be nice to have as a feature to troubleshoot Docker issues (for example).
    __
    Both ideas come from looking at the list of features supported by VSCode LifeShare.
Kelvin Glaß
@m273d15

@voronaam Hi, currently we do not plan to implement "Shared Ports" or "Shared Terminals" because we have limited resources and try to focus on synchronizing and sharing code.

If you are interested in our plan:

  • The next big thing will be the support of Saros sessions between different IDEs (Eclipse and IntelliJ).
  • Furthermore, we would like to support further IDEs/Editors in the future.
  • We want to complete the IntelliJ support (still an alpha version).

Because of the idea of sessions between different IDEs we want to avoid integrating deeply into one IDE/Editor (because this could lead to feature sets that are only supported in one IDE).

Lex Vorona
@voronaam
Thank you.
The cross-IDE feature is probably also important to us. I am an Eclipse user, but Idea is more popular among my colleagues.
Kelvin Glaß
@m273d15
Hi, the new "pull request retargeting" might simplify our handling of "Stacked PRs"
Stefan Rossbach
@srossbach

And from recent build results, it seems like EditDuringInvitationStressTest.testEditMultipleClassesDuringInvitation() can fail more than 4 times in a row when run as part as the GitHub action (see build1 and build2). This is also the test that caused the majority of STF failures in the past. So maybe we should mark this test as flaky as well.

This was one of the major requirements in the past. You should be able to modify anything you want while another user joins the session because sometimes it takes "a little bit longer" for someone to join a session / complete the resource negotiation and therefore putting the whole session on hold was not a solution.

Tobias Bouschen
@tobous

@srossbach When having a look at the Eclipse EditorManager, I noticed that most of the logic using the EditorPool to obtain open shared editors only uses the IEditorPart objects of the mapping and ignores the mapped IFile objects. Instead, the Eclipse IFile for the editor is requested and then converted to a Saros IFile object.

Is there a specific reasoning for this handling? Can the editor pool mapping be outdated, making it preferable to look up the actual, current file for the editor?

This makes a difference for the migration to the new reference point model as it is no longer as easy/convenient to convert from an Eclipse resource to a Saros resource. Instead of just wrapping the object, the new logic has to check for every shared reference point whether the resource is one of its children and the calculate the reference-point-relative path. This creates a lot of unnecessary overhead for the lookup, which is why I would prefer to instead just request the complete mapping from the editor pool.

Tobias Bouschen
@tobous
The STF on the master is broken again for whatever reason. My guess would be that an external dependency broke again as runs for confirmed good commits (that ran fine a couple of days ago) now no longer pass (see https://github.com/saros-project/saros/runs/786572630?check_suite_focus=true for example). @m273d15 any ideas?
Kelvin Glaß
@m273d15

The reason for the breaking build is the exception (e.g. in EditorAPITextPositionTest.testCalculateOffsetsOneLineSelection):

java.lang.UnsatisfiedLinkError: Could not load SWT library. Reasons:     
    no swt-pi4-gtk-4934r6 in java.library.path    
    no swt-pi4-gtk in java.library.path    
    Can't load library: /root/.swt/lib/linux/x86_64/libswt-pi4-gtk-4934r6.so    
    Can't load library: /root/.swt/lib/linux/x86_64/libswt-pi4-gtk.so

This is an issue in :saros.eclipse:test. It is strange that this issue only occurs in the STF workflow. Therefore, the next step is to compare the execution environment (I assume that setting the JDK instead of using the default could fix it).

Frank Benoit
@frankbenoit
Hi, want to do a demo with Saros. For this, I want to use 2 Eclipse instance on the same PC. But the account data seems to be shared between them, allthough i configured different eclipse installations and workspaces. Is there a way to keep them separated?
Stefan Rossbach
@srossbach
There is no official switch to disable this behavior. As you still need more than one account for a demo it should not matter if both Eclipse instances access the same account data. Just add to accounts and then use the Connect with option to select the account for the specific IDE instance.
FelixSelter
@FelixSelter
Can someone help in saros/user?
svd340
@svd340
Hi! Quick question:
When I look at the stanzas that are sent back and forth between host and client during a session negotiation, the 'SNCMP' (SessionNegotiationCompleted) stanza is never sent. Somehow the client is added to the session successfully anyways. How is that possible?
svd340
@svd340
Thank you for the reply!
Yes, I've seen those lines of code, but as I mentioned, upon looking at the traffic, it appears that the expected SNCMP stanza is never sent by the client nor received by the host. Hence the question if I'm missing something :)
Stefan Rossbach
@srossbach
While any message can be send through the XMPP server via stanzas most of the traffic is done either through plain TCP connections or through IBB streams. This is the reason why do you not see the stanzas in the XMPP connection anymore (or they are "hidden" in IBB stanzas)
svd340
@svd340
I see! That makes sense. Thanks for getting back to me.
Lex305
@Lex305
Is there a way to get a new development build?