<EMAIL_ADDRESS>redownloading jdk on every run

Description:
<EMAIL_ADDRESS>redownloading jdk on every every job in a pipeline while executing on a self-hosted Linux runner (about 15min on every job).
Task version:
v4.2.0
Platform:

[x] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[ ] Hosted
[x] Self-hosted

Repro steps:
Workflow file:
build:
    runs-on: self-hosted
    steps:
      - uses: actions/checkout@v4
        with:
          persist-credentials: false
      - name: set up JDK 17
        uses<EMAIL_ADDRESS>        with:
          distribution: 'oracle'
          java-version: 17
          check-latest: false
          cache: 'maven'
      - name: Maven Verify
        run: mvn clean compile

  codestyle:
    needs: build
    runs-on: self-hosted
    steps:
      - uses: actions/checkout@v4
      - name: set up JDK 17
        uses<EMAIL_ADDRESS>        with:
          distribution: 'oracle'
          java-version: 17
          check-latest: false
          cache: 'maven'
      - name: Maven Verify
        run: mvn checkstyle:check

  test:
    needs: codestyle
    runs-on: self-hosted
    steps:
      - uses: actions/checkout@v4
      - name: set up JDK 17
        uses<EMAIL_ADDRESS>        with:
          distribution: 'oracle'
          java-version: 17
          check-latest: false
          cache: 'maven'
      - name: Maven Verify
        run: mvn -B clean verify
      - name: Upload coverage reports to Codecov
        uses<EMAIL_ADDRESS>        with:
          token: ${{ secrets.CODECOV_TOKEN }}
          slug: Ivanuil/ImageEditor

Setup JDK log:
Run<EMAIL_ADDRESS>  with:
    distribution: oracle
    java-version: 17
    check-latest: false
    cache: maven
    java-package: jdk
    server-id: github
    server-username: GITHUB_ACTOR
    server-password: GITHUB_TOKEN
    overwrite-settings: true
    job-status: success
    token: ***
Installed distributions
  Trying to resolve the latest version from remote
  Resolved latest version as 17
  Trying to download...
  Downloading Java 17 (Oracle) from https://download.oracle.com/java/17/latest/jdk-17_linux-x64_bin.tar.gz ...
  Extracting Java archive...
  /usr/bin/tar xz --warning=no-unknown-keyword --overwrite -C /home/ivanuil/actions-runner/_work/_temp/6ec944e[2](https://github.com/Ivanuil/ImageEditor/actions/runs/9017688035/job/24776650890#step:3:2)-cb2c-4b87-ad79-8f27ace92481 -f /home/ivanuil/actions-runner/_work/_temp/7f52[3](https://github.com/Ivanuil/ImageEditor/actions/runs/9017688035/job/24776650890#step:3:3)1a7-7ee0-4356-a299-19e1080a6[4](https://github.com/Ivanuil/ImageEditor/actions/runs/9017688035/job/24776650890#step:3:4)80
  Java 17 was downloaded
  Setting Java 17 as the default
  Creating toolchains.xml for JDK version 17 from oracle
  Overwriting existing file /home/ivanuil/.m2/toolchains.xml

  Java configuration:
    Distribution: oracle
    Version: 17
    Path: /home/ivanuil/actions-runner/_work/_tool/Java_Oracle_jdk/17/x[6](https://github.com/Ivanuil/ImageEditor/actions/runs/9017688035/job/24776650890#step:3:6)4

Creating settings.xml with server-id: github
Overwriting existing file /home/ivanuil/.m2/settings.xml
maven cache is not found

Post setup JDK log:
Post job cleanup.
/usr/bin/tar --posix -cf cache.tzst --exclude cache.tzst -P -C /home/ivanuil/actions-runner/_work/ImageEditor/ImageEditor --files-from manifest.txt --use-compress-program zstdmt
Cache Size: ~3[2](https://github.com/Ivanuil/ImageEditor/actions/runs/9017688035/job/24776650890#step:7:2)1 MB (336496180 B)
Cache saved successfully
Cache saved with the key: setup-java-Linux-maven-a1b758b65f9e0e80afb7bef1758b722af2701f1dbe542c6f8a9f6a83933ce403

Hello @Ivanuil, Thank you for creating this issue and we will look into it :)

Haha

I'm using the default installed version java 21.
there is a 17 too
e.g.
        # https://github.com/actions/runner-images/blob/ubuntu22/20240516.1/images/ubuntu/Ubuntu2204-Readme.md#java
      - name: Switch to Java 17
        run: echo "JAVA_HOME=$JAVA_HOME_17_X64" | tee -a $GITHUB_ENV ;  sudo update-java-alternatives --set $JAVA_HOME_17_X64;

Will be cool this action to take advantage of the pre-existent java versions already installed

We're seeing the same thing on our self hosted runners. Shouldn't it re-use /opt/actions-runner/_work/_tool/Java_Oracle_jdk/17/x64 if it's already there? Or do we need to setup a tools cache like Githubs runners? This is all quite poorly documented for us running self hosted runners. Any idea @aparnajyothi-y

Try specifying java-version: 17.0.11, the action doesn't use the cache if it's not an exact version.
https://github.com/actions/toolkit/blob/main/packages/tool-cache/src/tool-cache.ts#L553

Try specifying java-version: 17.0.11, the action doesn't use the cache if it's not an exact version.

That did the trick. Thanks)

Hello @Ivanuil, I've tried to reproduce the issue but didn't encounter any problems. Here's a screenshot for you to look at. Could you please outline the steps that led to this issue, so I can attempt to recreate it?

@mahabaleshwars
Your screenshot says: "Java 17 was downloaded"
It should say: "Resolved Java 17.0.11 from tool-cache" when it re-uses the cache.
Look at the code I pointed to above, using "17" as the version will disqualify that dir in _work/_tool/Java_Oracle_jdk. However, if you'd happen to have both 17 and 17.0.1, it will pass the check that it has any caches and 17 will find its directory anyway. The logic there is quite badly implemented.

Hello @Ivanuil,
If you're using the setup-java action and specify a major version like java-version: 17, the action will try to resolve it to the latest minor version. If a new minor version is released (say 17.0.12), it won't match the cache key for 17.0.11 and the JDK will be downloaded again.
To ensure that the exact version is cached and used across workflows, it's recommended to specify the exact JDK version you want to cache in your workflow file, like java-version: 17.0.11. This way, the cache key will remain consistent across workflow runs.

Hello @Ivanuil Yeah but that's not how it works in reality, unfortunately. See my comments above.

something is downloading every time that i build something
jobs:
  build_android:
    if: "contains(github.event.head_commit.message, 'build-android')"
    name: Build Android
    runs-on: self-hosted
    steps:
      - name: Check out repository code
        uses: actions/checkout@v4
      - name: use Node.js
        uses: actions/setup-node@v4
        with:
          node-version: "18.19.1"
      - name: Setup Java
        uses<EMAIL_ADDRESS>        with:
          java-version: 17.0.11
          distribution: "oracle"
          check-latest: false
          cache: "gradle"
      - name: Setup Android SDK
        uses: android-actions/setup-android@v3

something is downloading every time that i build something
jobs:
  build_android:
    if: "contains(github.event.head_commit.message, 'build-android')"
    name: Build Android
    runs-on: self-hosted
    steps:
      - name: Check out repository code
        uses: actions/checkout@v4
      - name: use Node.js
        uses: actions/setup-node@v4
        with:
          node-version: "18.19.1"
      - name: Setup Java
        uses<EMAIL_ADDRESS>        with:
          java-version: 17.0.11
          distribution: "oracle"
          check-latest: false
          cache: "gradle"
      - name: Setup Android SDK
        uses: android-actions/setup-android@v3

Same for me :( Everything worked as it was supposed to for a month. And a few days ago the action started redownloading something on every run. It also usually fails to download (percentage get stuck at some point) and i have to cancel that run

My job looks like this:
- name: set up JDK 17
        uses<EMAIL_ADDRESS>        with:
          distribution: 'oracle'
          java-version: 17.0.11
          check-latest: false
          cache: 'maven'

Hi @Ivanuil, I've attempted to reproduce the issue but I didn't encounter the problem you described. Could you provide a public repository where the issue is reproducible?

Hi @Ivanuil, I've attempted to reproduce the issue but I didn't encounter the problem you described. Could you provide a public repository where the issue is reproducible?

Sure, repo: https://github.com/Ivanuil/FriendAlertBot
action: https://github.com/Ivanuil/FriendAlertBot/actions/runs/9856712473

Hello @Ivanuil,
I have forked the repository ""FriendAlertBot"" but am still unable to reproduce the issue.
For your reference, I am sharing my workflow: workflow link
I noticed that the cache is not visible in your repository, whereas it is being saved in my repository and setup-java is using it for the next run.

Hello @Ivanuil, Please let us know if you still have any concerns on the above comment.

the cache is not visible in your repository

Thank you for the info. Could you give me some advice on how to set up cahche in my reposityory so that everything works? :)

Hello @Ivanuil,
Could you enable debugging in your repository by adding the following secrets:
ACTIONS_RUNNER_DEBUG: true and ACTIONS_STEP_DEBUG: true ?
Additionally, could you share your runner configuration file and its log so we can examine if it is contributing to this issue?

Hello @Ivanuil, were you able to debug with the commands mentioned above?
Please share the logs that will help us identify the root cause more effectively.

Hello @Ivanuil,
Caching Mechanisms for Java in GitHub Actions

Dependency Caching
Method: Cache dependencies using the actions/cache action.
When to Use: Useful for caching Maven or Gradle dependencies to speed up subsequent builds.

Maven Dependency Caching: workflow
Gradle Dependency Caching: workflow

Tool Caching
Method: Cache the Java distribution itself.
When to Use: Reduces the time to download and install the Java version on each run.
Example: Cache AdoptOpenJDK

name: Cache AdoptOpenJDK
uses: actions/cache@v2
with:
path: ~/.sdkman/candidates/java
key: ${{ runner.os }}-adoptopenjdk-${{ hashFiles('**/sdkmanrc') }}
restore-keys: |
${{ runner.os }}-adoptopenjdk-

Build Caching
Method: Cache the actual build outputs.
When to Use: Speeds up the build process by reusing previous build outputs.
Example: Cache Build Output

name: Cache Build Output
uses: actions/cache@v2
with:
path: build
key: ${{ runner.os }}-build-${{ hashFiles('**/*') }}
restore-keys: |
${{ runner.os }}-build-

Each method has its own use cases and benefits, such as reducing build times and improving efficiency. Choose the appropriate method based on which part of the build process you want to cache.

Maven Caching
With Maven, you can cache the local Maven repository (~/.m2/repository). This saves the downloaded dependencies across different workflow runs.
Example: workflow

Gradle Caching
With Gradle, you can cache the Gradle user home directory (~/.gradle/caches). This saves the downloaded dependencies and other build artifacts across different workflow runs.
Example: workflow

SBT Caching with setup-java
Sample Workflow for SBT Caching:
Example: workflow

Comparison

actions/cache: More flexible, allowing for custom paths and keys, which can be optimized for specific project structures.
cache (maven/gradle/sbt): Automatically handles standard paths, potentially resulting in quicker cache restores.

Choose the appropriate caching mechanism based on your project's needs and the level of customization required.

Hello @Ivanuil, We are closing this issue now. Please feel free to reach out to us for any other concerns.


changed so all options (specifically rollout policy) can be given to solver

Max, see if this is ok with you. I think it's important to be able to specify all the options to the solver and then create a policy appropriate for the mdp with solve(solver, mdp)

Actually, forget this - I have a better idea.


cancel event does not fired when copy: true

Please only use GitHub issues for bug reports and feature requests.

[x] Read the contributing guidelines
[x] Bug reports containing repro steps are likely to be fixed faster
[x] Feature requests should be multi-purpose, describe use cases
[x] For support requests or questions, please refer to our Slack channel

Repro steps:

Open second sample on https://bevacqua.github.io/dragula/ (with events)
Change dragula([$('left-events'), $('right-events')]) to dragula([$('left-events'), $('right-events')], { copy: true }), and add subscription to 'cancel' event
Try to drag last item in the left column bit lower in bounds of its container and release - no cancel event raised

Reason: _copy is not in the dom, so getParent(_copy) returns null, and then isInitialPlacement returns false, and we go to drake.emit('drop', item, null.

Because of this bug there is no way to distinguish two situations:

Item was moved out of the container
Item moved inside the container

In the second case we should get 'cancel' event, but actually, in both cases we get 'drop" event with target = null.

Fixed in temporary package dragula-fixed


Resolve the company and site IDs better

Right now the company and site IDs need to be specified in the CRDs.  It would be nice to either use names to resolve (via user API) or some form of lookup to get them for the user.

We can use the user API (https://kb.kentik.com/v0/Ec06.htm) to resolve the company ID from the username configured from the secret.  For each task, we can use the site API (https://kb.kentik.com/v0/Ec10.htm#Ec10-Site_API) to resolve a user friendly name to the corresponding site ID.


Chris process proposal

This PR fleshes out the model of the implementation in LibraBFT.Impl, specifically processProposalMsgM and its dependencies.

It also removes LibraBFT.Impl.OBM.Prelude -- I thought the functionality belonged in RWST.agda instead.

Overall looks very good @cwjnkins.  I do have a few comments and suggestions.  It's especially important to address the ones that break things so that ./Scripts/run-everything y doesn't succeed.  Thanks!

@mark-moir I've just pushed a commit addressing your review. Do you think that MetaEpochIndep belongs in ImplShared, or should I move that completely to Impl and restore the older version of Output for FakeImpl?

Overall - looks great.

The updates look great.
The main thing we need to do is log ERRORS (not INFO).
But I think that should be done in a separate pull request.
I approve for merging.

Noticed that NetworkMsg.agda still lived in LibraBFT.ImplFake, moved it to LibraBFT.ImplShared.

@mark-moir I've just pushed a commit addressing your review. Do you think that MetaEpochIndep belongs in ImplShared, or should I move that completely to Impl and restore the older version of Output for FakeImpl? The benefit of the former case is that FakeImpl could be used for proofs connecting meta-data to the behavior of the fake implementation, before attempting to do so for the real model of the implementation.

I agree with the suggested approach (especially because it's what you've already done).  We can always change this later if it becomes an issue, but in the meantime, less duplication is good.

@cwjnkins and @haroldcarr : great stuff, thanks!  We do have a few unresolved conversations, but I'm merging now and we should go through those and decide what to do about them and when.


[MenuRoot] Remove wrong default value from docs

I was looking at source code of Menu and docs, docs says default value of openOnHover prop is nested but nested value is not defined in code and even if openOnHover prop is defined as nested following code (see line 27) would be illogical as openOnHoverProp will always be true. hence i removed default prop as nested from docs.
https://github.com/mui/base-ui/blob/919d9ca513b3b2c52b175a33a858529408593a5f/packages/mui-base/src/Menu/Root/MenuRoot.tsx#L21-L30
Before:

After

Preview:
https://deploy-preview-549--base-ui.netlify.app/base-ui/react-menu/components-api/#menu-root-props

[x] I have followed (at least) the PR section of the contributing guide.

Thanks!


Include jskos-tools and cocoda-sdk as modules

Requires https://github.com/gbv/jskos-tools/issues/19

Since we're using Vite which does its own bundling, we don't need bundled modules from other packages. jskos-tools now uses a JS-only implementation for its sha1 algorithm which fixes the issue of bundling it with Rollup (which is used by Vite).


node plot reset when flags toggled

@benjeffery pointed out that toggling the node flags resets the plot,. It would be good to have the zoom unchanged when a node type is checked or unchecked.

This also happens on the mutations plot when you click log scale. To fix it we need to completely rework how we do the plots, binding the controls and making the plot once rather than making a new plot every time a control changes.

I think the current approach is fine, it's just the way it is.


doc: add a CODE_OF_CONDUCT.md

Address issue #159

Merging this PR. Let's re-open if any further objections.


removed underscore from package.json - fixes issue #10

dead simple - although I note that one of our dependencies (cfenv) uses underscore itself, so this really has no effect :)

👍


Review adoption patterns for XRTK - Full vs Individual components

Is your feature request related to a problem? Please describe.
In discussions with adopters we are seeing multiple patterns for consuming the XRTK, from:

Brand new project
Existing project and upgrading (inc HTK)
Transferring from MRTK to XRTK
Isolating individual components to adopt in to another projects framework

(List is not exhaustive and will likely increase with time)
Describe the solution you'd like
Ultimately this proposal demands the need for multiple deployment options to address each of the above environments with several documented / supported deployment patterns, including:

Base deployment (already supported)
Existing project install (supported but needs better documentation)
Requires helpers to migrate the configuration utilised by an existing MRTK solution
Requires SDK components to be able to externalise services / components

Additional context
This proposal marks the start of the discussion and prompts open discussion by all for any patterns we may have missed.

2 years on and I still have no idea what this task describes exactly 😅


Keras/tensorflow failed when specifying class_weight in model.fit()

System information.

OS Platform and Distribution:  Linux.
TensorFlow version:  Keras/tensorflow version 2.8.0.
Python version:  Python 3.7
GPU model and memory:  Use CPU (no GPU).

Describe the problem.
My network model works well without specifying class_weight in model.fit().
However, when I specify class_weight in model.fit(), no matter what weight values I give, keras/tensorflow failed with the following error:
  File "/opt/local/lib/python3.7/site-packages/tensorflow/python/eager/execute.py", line 55, in quick_execute
    inputs, attrs, num_outputs)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Graph execution error:

indices[9] = 16 is not in [0, 10)
         [[{{node GatherV2}}]]
         [[IteratorGetNext]] [Op:__inference_train_function_43205]

Keras/tensorflow failed with the above error even when I give all classes an equal weight 1.0 (which is equivalent to no class weights), as the following (I have 10 classes):
class_weights_dict = {0: 1.0, 1: 1.0, 2: 1.0, 3: 1.0, 4: 1.0, 5: 1.0, 6: 1.0, 7: 1.0, 8: 1.0, 9: 1.0}

history = model.fit(train_input,
                    train_true_labels,
                    class_weight=class_weights_dict,
                    validation_split=validation_split,
                    shuffle=True,
                    epochs=epochs,
                    batch_size=batch_size)

And I verified that my true labels array train_true_labels contains only integers 0-9, as the following:
values = np.unique(train_true_labels)
print(values)
array([0, 1, 2, 3, 4, 5, 6, 7, 8, 9])

However, when I do not specify class_weight in model.fit(), the training for my model works just fine.
So it looks like I just cannot use class_weight in training. But my classes are highly imbalanced; not using class weights would train a useless model.
I would greatly appreciate any solution for this issue.
Thank you very much!

Can you provide code to reproduce the error?

@lauraht
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

Hi Surya (@old-school-kid) and @sushreebarsa:
Thank you very much for your help!
Yes, I have made a simplified model that can reproduce the error. Could I email my code and data files to you?
Thank you so much!

Hi Surya (@old-school-kid) and @sushreebarsa,
I tried to email you my zipped code/data files by directly replying to "keras-team/keras<EMAIL_ADDRESS>however, it complained that it exceeds the size limit and it couldn't be delivered. It is 3.6 MB (zipped). I guess it is the GitHub that has a small size limit.
So I was wondering if I could possibly have your email address so that I could email it to you directly?
Thank you very much!

Hi @lauraht
It looks like a classification problem, so can you try to reproduce the error using the MNIST data available in Keras?
If it isnt possible please mail it to me (not over github).
Thanks

Hi Surya (@old-school-kid),
I have emailed the code/data files to your gmail account. Both the input and output of the model are sequential data.
Please let me know if you received these code/data files.
Thanks so much for the help!

@lauraht
In order to reproduce the issue, could you please upload all the files on drive and share the link with permission ?Thanks!

Hi @sushreebarsa,
I have emailed all the files to Surya's (@old-school-kid) gmail account and the email was delivered without the size limit problem.
Do you mean you want me to upload the files to the google drive and share them with you and Surya? If so, could you please let me know your gmail address (I have Surya's gmail address) so that I can share the files with you? Thank you very much!

@lauraht Could you  please share your drive access to the community so that we can access your files as we will not be able to share the gmail details ?
Thanks!

Hi @sushreebarsa and Surya (@old-school-kid),
I have uploaded the code and data files here (in a zipped folder):
class_weight_code.zip
The Python file can reproduce this error, and it loads the two data files. Both the input and output of the model are sequential data. The log file contains this error.
However, when I commented out the line “class_weight=class_weights_dict” in “model.fit()”, the model training works just fine.
Thank you very much for the help!

Hi @sushreebarsa,
Thank you very much for looking into this!
I have verified it--- Yes, the error reported at the gist is essentially the same error as I got (which is recorded in my log file).
I looked into this issue as you suggested. I believe the way I defined the class_weight is exactly the same way as the answer described. And I found that a follow-up user there reported the same error (see below) as I encountered:

I tried using class weights with tf data and it gave me weird errors. The shape of my label is [batch_size, seq_len], where each label is between [0, 3], and the shape of the y_pred is [batch_size, seq_len, 4]. Keras complains an invalid arugment error saying index 18 is not in the range of [0, 3]. I don't even know where the 18 comes from. Labels are double-checked to make sure they are all in the range of [0, 3].

So it seems that there is a bug in keras/tensorflow related to the class_weight, which occurs when the output is sequential data (my output is also sequential data).
I would greatly appreciate it if the keras/tensorflow team can fix this problem.
Thank you so much!

Thank you @sachinprasadhs and @mihirparadkar for looking into this problem!
I would greatly appreciate it if this problem can be fixed, so that our project can continue.
Thank you very much and I am looking forward to a fix!

Dear @mihirparadkar and @sachinprasadhs,
I was wondering if the Keras team has any plan to fix this problem?
Could you please give us some update on this? I would really appreciate it!
Our project has been stuck due to this problem, and we are looking forward to a solution.
Thank you so much for your help and we would greatly appreciate a fix to this problem!

@lauraht were you able to find a solution to the class_weight issue?


Option argument --clean isn't in vim.man

I didn't find --clean in the output of man vim.
But I found it with :h --clean.
Vim 8.2.2730

There are a lot of options available on the command line for vim that are not mentioned in man vim. See :help starting while in vim and you'll see quite a few.

Oh, OK. I didn't realize that. I just checked a couple other options (the ones listed above and below :h --clean) and saw that they were mentioned in man vim. But now I see that, like you said, other options are also missing from man vim, such as --ttyfail.

Matthew Mario Di Pasquale wrote:

I didn't find --clean in the output of man vim.

Indeed.  I'll add it, it's quite useful.
--
In a world without walls and borders, who needs windows and gates?
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

OK, sounds good. Thanks, @brammool.

Fixed by https://github.com/vim/vim/commit/56994d215815139207f3c5ce02a1720e44e93c09


Showing alert box

Hi, our app got rejected because it is showing the alert "Operation was cancelled by the user" when the app is loaded.
This alert is regarding the game center plugin that we are using in our game.

I have the same problem. This dialog appears when the user cancelled the login, but it can also appear in other situations I think.
The offending line: https://github.com/cranberrygame/cordova-plugin-game/blob/c22a8af418da355d5e06c9f3b1f1ca03e83566d6/src/ios/Game.m#L133
I think showing error alerts should be done by the app writers, not the plugin itself. So I would vote for removing those two lines that display the error box.

+1 for @letmaik


Needs tests as well as the holes now found were not found by existing tests.

CLA signed by all contributors.


Request to suggest annotations hangs forever, until another request of some kind is sent

To Reproduce

Click suggest annotations
Wait around until you're sick of waiting
Make some other request, like annotating a property, or changing the project description
Suggest annotation stops hanging and returns

I can't reproduce this on my end, both locally and on https://t2wml.isi.edu/
Can you share the specific file where this is happening? Or does it happen for any file?

appears to be resolved


Install asdf with brew?

asdf was available as a brew formula before it was introduced here. Why not install it with brew?

Installing asdf with brew makes sense to me. Let's do it!

Does anyone implemented it?


Warning: iOS 8.0+ / macOS 10.10+ / tvOS 9.0+ / watchOS 2.0+

Every time the compiler runs, we get this error:
The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.4.99.

This is because Xcode 12.4 doesn't support iOS 8. Since the iPhone 4S supports iOS 9, the iPhone 5/5G iOS 10, The 5S iOS 12, The iPad Air 1 iOS 12, I wonder if it wouldn't make sense for the library to start having a minimum Version of iOS 12. Where does Swift 4.2 runs anyways?
By any means, support at least only iOS 9. You also might want to wait Xcode 12.5 and see if it still supports iOS 9.

I think we'll just bump the minimum iOS version, support for iOS 8 is pretty obscure anyway.

Should be resolved now in version 1.4

It is. That was quick! Thanks!


Mkdocs literate

Added loop to re-build example markdown for docs at time of documentation build.
UMLS occurrence tests were failing/erroring as try/catch blocks create local scope - the variables set in the try blocks are now initialized before the block starts.
Requires latest version of BioServices.

Is it worth getting coverage information from the examples? We could do that by moving the environment variable check to inside the “runtests.jl” file, then Pkg.test(“BioMedQuery”; coverage=true) will run the appropriate script based on ENV[“GROUP”]. Then the coverage number would be more accurate.

@DilumAluthge I don't think it would really add much to the coverage as what is missed on the coverage tends to be the weird/error cases and the examples mostly hit the happy path.  Are there any cases that you think the examples hit and the tests don't?  The better approach might be to make sure we hit those.
I personally use the testing of the examples more as a reminder to fix the examples when I've made a breaking change, but I'm not sure if this how others (you & @mirestrepo) use them.

If the tests hit everything the examples hit, then we don’t need to do that!
On Mon, Jun 18, 2018 at 14:56 mcmcgrath13<EMAIL_ADDRESS>wrote:

@DilumAluthge https://github.com/DilumAluthge I don't think it would
really add much to the coverage as what is missed on the coverage tends to
be the weird/error cases and the examples mostly hit the happy path. Are
there any cases that you think the examples hit and the tests don't? The
better approach might be to make sure we hit those.
I personally use the testing of the examples more as a reminder to fix the
examples when I've made a breaking change, but I'm not sure if this how
others (you & @mirestrepo https://github.com/mirestrepo) use them.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bcbi/BioMedQuery.jl/pull/28#issuecomment-398159082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFXArcV1YsPs5yNQXA0qLoH3SrTQWywHks5t9_fegaJpZM4UXNod
.

--
Dilum Aluthge
<EMAIL_ADDRESS><EMAIL_ADDRESS>https://www.aluthge.com https://www.aluthge.com


[LIGHTNODE SALE] Landing Page: Paloma Ecosystem Layout.

The Paloma Ecosystem section on the Landing Page needs some layout and design adjustments.

[x] Add Space between sections. (at least 40px)
[x] Remove Background icons.

Closing as shipped.


Boolean values set to valueFilter do not matter

Whether or not you set a value to True or False, it is eliminated from the active display:
From the sample:
valueFilter={'Day of Week': {'Thursday': False}}
leads to the same behavior as:
valueFilter={'Day of Week': {'Thursday': True}}
Perhaps I am not using this correctly? Thanks for sharing this excellent project

how to select only one.
because the all been selected,but I want to only one.


update wallet for protocol v2

work with new contracts from @gooddollar/goodprotocol package
show interest data from fundmanager
show staking data from multiple staking contracts

@sirpy
The Claim/Send/Receive/ScanQR claim stats are working as expected.
Environment:
QA 1.29.1-17
Devices:
Windows 10 // Google Chrome v92
iPhone XR // 14.6
Attachments:
https://www.screencast.com/t/UMfwCpHR7h1Y

@sirpy @serdiukov-o-nordwhale
The Claim/Send/Receive/ScanQR claim stats are working as expected.
Environment:
PROD 1.30.2
Devices:
Windows 10 // Google Chrome v92
iPhone XR // 14.6
Samsung A80 // Android 10
Attachments:
https://www.screencast.com/t/NC0eSGrj1pi


use stringbuilder in htmlizer

Hi Simon, thanks for this.
Have you made any performance comparisons between  using StringBuilder and String.Join?

To be honest i didnt measure it. Given string builder code is equivalent in readability i just default to it when building large strings. Also the next step would be to use the object pool for the builders

String.Join can actually be faster in scenarios like this since it can allocate the needed capacity of the result string once by iterating over the strings in the list and aggregating their lengths. So I think that is why the Blazor team does it like this (a lot of the code in that file is from their repo).

Fair enough. Will close


Good Results

Hi @autonomousvision
I have used your model and i am having good results but not perfect , like if i am giving simple chair image with horizontal lines so model is generating mesh based on the chair , it is not creating that lines in the mesh.
could you please suggest us to improve the results at best way .
Thanks in advance !!!

Hi @ayush9198gupta , thanks for your interest in the project!
If I understand you correctly, you are performing single-view reconstruction with our model and then you want to obtain better results? One of the main reasons why local details as you mentioned are lost is that we use the encoder-decoder setup with a global code. Results presumably improve when you use local features instead / in addition, e.g. by passing the image through a U--Net type network, and then projecting query points into the source view and pool local features. If you have multiple views available at test time, you can also train a single model on these views (as we do in our "multi-view reconstruction experiments") which presumably leads to better results.
Good luck with your research

Hi @ayush9198gupta , thanks for your interest in the project!
If I understand you correctly, you are performing single-view reconstruction with our model and then you want to obtain better results? One of the main reasons why local details as you mentioned are lost is that we use the encoder-decoder setup with a global code. Results presumably improve when you use local features instead / in addition, e.g. by passing the image through a U--Net type network, and then projecting query points into the source view and pool local features. If you have multiple views available at test time, you can also train a single model on these views (as we do in our "multi-view reconstruction experiments") which presumably leads to better results.
Good luck with your research


HttpPostRequestDecoder should handle "a&b=c" in application/x-www-form-urlencoded content

Netty Version 4.0.0.Alpha1-SNAPSHOT (07-Dec-2011 01:22)
Requests with content type "application/x-www-form-urlencoded" contain a query string as data. These query strings may contain elements without value, e.g. "a&b=c" which should be equal to "a=&b=c".
Netty's application/x-www-form-urlencoded parses "a&b=c" as one attribute with name "a&b" and value "c".
POST / HTTP/1.0
Content-Length: 5
Content-Type: application/x-www-form-urlencoded

a&b=c

Netty returns:
a&b -> c
Netty should return it as two attributes:
a -> (empty string)
b -> c

I will have a look

The following document is a draft but it may help defining how to parse x-www-form-urlencoded:
http://tools.ietf.org/html/draft-hoehrmann-urlencoded-01

Hi,
I don't have time right now to check if it is correct, but the following should be ok.
line starting at 433 in src/main/java/io/netty/handler/codec/http/HttpPostRequestDecoder.java
        while (undecodedChunk.readable() && contRead) {
            char read = (char) undecodedChunk.readUnsignedByte();
            currentpos++;
            switch (currentStatus) {
            case DISPOSITION:// search '='
                if (read == '=') {
                    currentStatus = MultiPartStatus.FIELD;
                    equalpos = currentpos-1;
                    String key = decodeAttribute(
                            undecodedChunk.toString(firstpos, equalpos-firstpos, charset),
                            charset);
                    currentAttribute = factory.createAttribute(request, key);
                    firstpos = currentpos;
                }

// XXX NEW PART to fix this issue => check if name is followed by '&' then apply for an empty field
else if (read == '&') { // special empty FIELD
currentStatus = MultiPartStatus.DISPOSITION;
ampersandpos = currentpos-1;
String key = decodeAttribute(
undecodedChunk.toString(firstpos, ampersandpos-firstpos, charset),
charset);
currentAttribute = factory.createAttribute(request, key);
currentAttribute.setValue(""); // empty
addHttpData(currentAttribute);
currentAttribute = null;
firstpos = currentpos;
contRead = true;
}
// END OF FIX
break;
case FIELD:// search '&' or end of line
Best regards,
Frederic

Yeah this looks right... I will commit it. Thanks!


in demos > main.js: wrong paramters sequence in  map.setView(...)

sort of typo in demos > main.js:
in demos > main.js:
change
33:    map.setView(map_start_location);
to
33:    map.setView( map_start_location.slice(1, 3), map_start_location[0] );
leaflet expects [lat, lon], zoom,
not zoom, lat, lon
WvL

Thanks for the catch!


Update requirements.txt

I had to install the 'future' package to get this to work. I am running Python 3.8.2 in Ubuntu 20.04 on WSL.

Thanks @Radcliffe – missed that!
Maybe you can vote/comment on https://github.com/mzucker/page_dewarp/pull/20 to boost its approval?


UnCSS fails on localhost connections with Error: Status not 200.

$ uncss http://localhost:5757/index.html
Error: Status not 200
    at Request.<anonymous> (/usr/local/lib/node_modules/uncss/lib/lib.js:77:58)
    at Request.emit (events.js:95:17)
    at Request.onResponse (/usr/local/lib/node_modules/uncss/node_modules/request/request.js:924:10)
    at ClientRequest.g (events.js:180:16)
    at ClientRequest.emit (events.js:95:17)
    at HTTPParser.parserOnIncomingClient [as onIncoming] (http.js:1688:21)
    at HTTPParser.parserOnHeadersComplete [as onHeadersComplete] (http.js:121:23)
    at Socket.socketOnData [as ondata] (http.js:1583:20)
    at TCP.onread (net.js:527:27)

$ node
var request = require('/usr/local/lib/node_modules/uncss/node_modules/request');
request({
  uri: "http://localhost:5757/index.html",
}, function(error, response, body) {
  console.log(response.statusCode);
});
200

Not sure what the cause of the discrepancy is here. Altering the port, the FQDN, or the path to the page doesn't change the result from either UnCSS or request. It's still entirely possible that the problem is me.

Nope, thanks!
Not sure what I was doing before - seems silly now. I just reinstalled and tried with a Mixture project: uncss http://localhost:7774/ > styles.css worked like a charm.
Definitely going to be using this a lot now. Thanks again, really appreciate it.


uwsm: add more dependencies and patch to fix

Description of changes

Add upstream patch (https://github.com/Vladimir-csp/uwsm/pull/34)
Add essential dependencies
Add sway and hyprland support (disabled by default). This is first step towards a NixOS module

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hey @JohnRTitor.
I am happy to see that you have followed my discussions over at the uwsm repo and have taken it as an inspiration to make the PR.
But I would've liked it if you could've waited maybe a bit longer before opening this PR, as my main goal was to first create a solid flake and shake all of the bugs out before pushing it into nixpkgs. Ideally, while discussing possible upstream changes and learning from other Nixpkgs Wayland modules. :/
Second, if you copy my patches and request a review from me, it has a bitter taste since you only add yourself as the maintainer & do not add me as a co-author to the init commit.
But moving on. Please have a look at my draft branch on uwsm-nix.
I have added NixOS module tests in flake.nix that check if hyprland, sway & wayfire correctly boot.
In module.nix I have created the module implementation with quite a bit of comments about my reasoning (+ quite some out-dated and messy information but I wasn't really planning on sharing it yet, sorry for that).
The TL;DR is: I would like to avoid injecting the different Wayland packages into uwsm wrapper. Apart from the additional derivations (for each activated compositor one), the big issue is that it might point to the wrong package. If you have a look at the wayfire or sway module, you will see that the actual package isn't pkgs.sway but rather a module with custom overlays applied to it https://github.com/NixOS/nixpkgs/blob/5ad6a14c6bf098e98800b091668718c336effc95/nixos/modules/programs/wayland/sway.nix#L17-L38
To get access to the real packages, you need to access services.displayManager.sessionPackages (if I understand it correctly).
In my most recent iteration, I have avoided this issue by simply injecting all of the sessionPackages (including my newly build uwsm-based ones) into the systemd.user.extraConfig
https://github.com/kai-tub/uwsm-nix/blob/cea3b709c4dfdddaaec9d66a8f0f6d5c20edf16b/nix/module.nix#L130-L134
as it was done in #320737. Ahhh. But I see that you are working on an alternative solution in #322000, nice!
But yeah, simply calling the full path would be easiest, but is not supported by uswm.
If https://github.com/Vladimir-csp/uwsm/issues/35 does get resolved, there might be a way to simply use the "default" desktop entry files.
I have changed it locally to calling the .desktop files instead without the systemd modification and it seems to work. But as I couldn't fully grasp the uwsm code and wasn't sure if it would break in the future, I asked the question in https://github.com/Vladimir-csp/uwsm/issues/35 ;)

But I would've liked it if you could've waited maybe a bit longer before opening this PR, as my main goal was to first create a solid flake and shake all of the bugs out before pushing it into nixpkgs. Ideally, while discussing possible upstream changes and learning from other Nixpkgs Wayland modules. :/

I'll mark it as draft then, thanks.

Second, if you copy my patches and request a review from me, it has a bitter taste since you only add yourself as the maintainer & do not add me as a co-author to the init commit.

Sorry, I did not know about your patches and your work when "initing" uwsm package. I'll add you as a co-maintainer here (was waiting for bit of a response on your side).

But moving on. Please have a look at my draft branch on uwsm-nix. I have added NixOS module tests in flake.nix that check if hyprland, sway & wayfire correctly boot.

Sure.

The TL;DR is: I would like to avoid injecting the different Wayland packages into uwsm wrapper. Apart from the additional derivations (for each activated compositor one), the big issue is that it might point to the wrong package. If you have a look at the wayfire or sway module, you will see that the actual package isn't pkgs.sway but rather a package with custom overlays applied to it

I have thought about this before and I was thinking of using config.programs.hyprland.package (as people like to override the options, like using from a flake) as the inputs but as you pointed out that is not enough and might still point to "different" package.

as it was done in #320737. Ahhh. But I see that you are working on an alternative solution in #322000, nice!

#322000 does not apply on wayland sessions at all. It only affects xsession wrapper that NixOS uses. I am not working on it anymore, if anyone wants to pick that up, they have my approval.

My setup in https://github.com/JohnRTitor/nix-conf/tree/d7ddbdc3753369d8ec69587ae2de4f5d04ded899/modules-overlays/uwsm (based on your work), works for me as of now (for those who have been following the discussion so far). But obviously this is not ready yet to be upstreamed to nixpkgs.
I will wait for you to make the PR.

Hey, thank you very much for the changes!

I have thought about this before and I was thinking of using config.programs.hyprland.package (as people like to override the options, like using from a flake) as the inputs but as you pointed out that is not enough and might still point to "different" package.

Yeah, that's sadly not the way to go :/

I will wait for you to make the PR.

If we exclude the "package" injection in this PR, I believe that it could/should still be merged. :+1:
Depending on the final solution in the module, we might be able to drop the patch in the future, but it definitely doesn't hurt.
But by PR you are referring to a PR for the NixOS module?

Sorry, I did not know about your patches and your work when "initing" uwsm package. I'll add you as a co-maintainer here (was waiting for bit of a response on your side).

Sorry about the maintainer comment, there was no way of knowing if I wanted to be added or not. Thanks for adding me :+1:

@kai-tub I have removed hyprland and sway injection, should be fine now. I'll merge once you approve.

But by PR you are referring to a PR for the NixOS module?

Yes.

I have added back sway and hyprland PATH injection back as the package still won't work without them, even if I do something like
uwsm start -S /run/booted-system/sw/bin/Hyprland

Perhaps we will find a solution in future, but for now I am inclined to keep it as a fallback.


  Investigating.

Enemies seem to be programmed to idle at a certain distance, but are broken from their idle if you point at them with your weapon.  In the flat screen game your weapon is always facing forward so this is no issue; in VR this is now tied to your hand which causes some enemies not to animate until you point at them. I still can't find the hooks in the code to fix this, so leaving this issue up so people are aware. But at this point this is highly unlikely to be fixed.


[Suggestion] Remove alert("Hello") from examples

Is alert("Hello") really necessary in examples? IMO it spoils the impression for otherwise a very good demo.

Is alert("Hello") really necessary in examples? IMO it spoils the impression for otherwise a very good demo.

Yeah, you are right alert is unnecessary here. When I put it I thought that it would be cool to show onClick feature but now I see that I was wrong. Will change it soon. Thank you)


Error running app

I am not able to access https://geospatial.streamlitapp.com/. I a get an error running the app.

Thanks for reporting. The app has been rebooted. It should be working now.


rocksdanister.LivelyWallpaper version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

It prompts me for user or system install

I noticed it too, when I tried with admin terminal winget --silent flag install succeeded.

https://github.com/microsoft/winget-pkgs/discussions/16233

I think winget by default passes SilentWithProgress switch for all applications, here you have entered proper switches for the application but for Silent switch only, as winget uses SilentWithProgress by default and that field was not mentioned, winget passed default switch for inno setups for SilentWithProgress which is just /SILENT hence failed result.
Solution for this would be adding SilentWithProgress switch with /SILENT /SUPPRESSMSGBOXES /NORESTART /SP- /CURRENTUSER /NOAUTOLAUNCH /NODEPENDENCIES
Here I changed Silent switch to Custom, custom switch will be forced to the installer, so winget forced /VERYSILENT /SUPPRESSMSGBOXES /NORESTART /SP- /CURRENTUSER /NOAUTOLAUNCH /NODEPENDENCIES switch to the installer and it installed successfully.

Looks like missing SilentWithProgress was the problem, thanks for the help.


fix application of pdk default_decorator

hi @joamatab , it seems like in 7.8.12 support for PDK default_decorator was dropped. this PR restores that functionality and adds a test

@joamatab it seems like some side effects have started occuring lately, which seem to be unrelated to these changes. have you noticed this? notice the test I added but skipped. having it run caused more failed tests. you can see with it skipped, there are failed tests on MacOS, but not ubuntu. i get unpredictable results locally on windows with it sometimes failing 0, 1, or 2 times, seemingly at random. it seems to have to do with the child metadata and naming of cells with children

i fixed the above issue by deepcopying settings and info from the child cells. however, i still get some strange minor side effects from the new test added, so i have left it skipped

thank you for the fix Troy!
yes, we've noticed strange behaviour with tests failing in some OS not being repeatable
@yaugenst

As you pointed out, decorators are causing side effects

As you pointed out, decorators are causing side effects

@joamatab , to be clear, it's actually not the decorator which is causing side effects. it's simply the act of clearing the cache midway through the tests which causes side effects. i proved this to myself by gradually paring down the test to the fundamentals... first, I reduced the original test to a test which simply activates a new PDK (without a default_decorator) briefly, then, without doing anything else, reactivates the original PDK
def test_pdk_switch_causes_chaos():
   # let's get the currently active pdk so we can set things back as they were later
   prev_active_pdk = gf.get_active_pdk()

   # create a new pdk and activate it, but don't actually use it at all
   pdk_without_decorator = gf.Pdk(
       name="pdk_without_decorator",
       base_pdk=get_generic_pdk(),
   )
   pdk_without_decorator.activate()

   # look, i'm not even doing anything, besides temporarily switching PDKs!

   # now reset the previously active pdk... watch unrelated tests fail
   prev_active_pdk.activate()

you can see that this still causes side effects. i get 3 failed tests... the same 3 I get from the test I added (but skipped) in this PR. you can deduce that this is either due to something particular with the activation/deactivation of PDKs, or it is due to clearing the cache between PDK switches, which will have the net effect of changing the order of invocation of different components, which traditionally has implied side effects due to things like mutability of components after being instantiated. that's why i tried this fix of using deepcopy instead of copy for these child cell settings (where I saw there were some recent changes). that indeed seemed to fix some issues, but not all. you can see that there are still some issues, even if you replace the above test simply with this
def test_clear_cache():
    gf.clear_cache()

for me i still get the same 3 failures of unrelated tests after adding this simple test. this indicates to me that the side effects seem to have something more to do with component mutability/the order in which components are invoked


Replace usage of juju/ratelimit with golang.org/x/time/rate

See this as an example: https://github.com/kubernetes/kubernetes/pull/38320

Fixed with #1936. Thanks!


Scaffolded Razor Pages

I did not understand: test the app by clicking on the RpMovie link.
Test the other links (Home, RpMovie, Create, Edit, and Delete).
I clicked on RpMovie in the code and nothing happened. Could not find any of the other links. Do not know what they are or what they are supposed to do.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 70f6104a-a223-4bb3-4101-f553002b9b26
Version Independent ID: 15e79e80-8d46-7cd5-c883-6cc0f5940a1b
Content: Scaffolded Razor Pages in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/page.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

The tutorial works. I'm not sure what step you missed. You might try asking on StackOverflow,.

Dear Rick:
I think maybe I am having a terminology problem. I have gone back through the tutorial and rechecked everything and all is as instructed in the tutorial up to “Update the Layout” on the Scaffolded Razor Pages tutorial.
Until I run into the instruction: Save your changes, and test the app by clicking on the RpMovie link. I understand “Save your changes.” But I am totally befuddled by “test the app by clicking on the RpMovie link” What exactly is the RpMovie link, where is it and what is supposed to happen if I click on it?
Ditto for “Test the other links (Home, RpMovie, Create, Edit, and Delete). What exactly are these links and where do I find them?
<EMAIL_ADDRESS>Mayhew
Mail: PO Box 69654 ● Oro Valley AZ 85737-0019 ● USA
Email: Biz<EMAIL_ADDRESS>Telephone: +1<PHONE_NUMBER>; Skype/WhatsAPP: Loren Mayhew
From: Rick Anderson<EMAIL_ADDRESS>Sent: Monday, January 13, 2020 1:14 PM
To: aspnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Business<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [aspnet/AspNetCore.Docs] Scaffolded Razor Pages (#16517)
Closed #16517https://github.com/aspnet/AspNetCore.Docs/issues/16517.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/aspnet/AspNetCore.Docs/issues/16517?email_source=notifications&email_token=AOILEWV7GDCYCKECQZGOGG3Q5TDO5A5CNFSM4KF4CQM2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOV6BSJ6Y#event-2944607483, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOILEWVUV4XPVGTOGWF4ONDQ5TDO5ANCNFSM4KF4CQMQ.

Keep going through the tutorial, you'll figure it out.


Сommand line options and endpoint_metrics_duration_buckets

Hi!
I am currently using command line options to configure things like cache storage size dynamically. I also monitor endpoint metrics, and now I would like to override default duration buckets because they hardly give me any useful information (my cache server is used in a local network and almost all requests take much less than 0.5s). Unfortunately, endpoint_metrics_duration_buckets could only be set in YAML file, using which AFAIU ignores any config that I have set in the command line.
Question: Is there a way to both configure duration buckets and set some other options dynamically?
If not, do you consider making endpoint_metrics_duration_buckets config available in command line / environment? Feature commit mentions that it is not a good idea, but I am confused as to why other options are available and this one is not.

The reasoning behind only supporting this option via config file is that it feels like it would be complicated to specify on the command line, and at some point we suspect advanced users would prefer to use a config file anyway. All the other options that can be set from the command line are simple in comparison.
Just curious: how are you dynamically adjusting the max_size setting?

Closing for now due to lack of feedback.


mvnd slower than mvn

First of all, thanks for making mvnd. I read about your concepts with daemon and GraalVM native image and it makes fully sense to me. Being enthusiastic about what you have written regarding incremental builds with concepts from gradle, avoiding overhead, faster, ... I tried it on a large modulithic project. The results say that mvnd is slower than mvn so is it still in a development phase that is not yet ready to reveal the promised benefits? Please note that I was waiting for the results and not doing other computations while the builds have been running (what was really hard looking at the build times).
Regular build with maven (3.8.6):
mvn install -DskipTests=true -T 8

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  12:56 min (Wall Clock)
[INFO] Finished at: 2023-07-03T17:59:22+02:00
[INFO] ------------------------------------------------------------------------

Build again with mvnd from the exact same folder with no change at all:
mvnd install -DskipTests=true -Dmvnd.keepAlive=10000ms

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  55:46 min (Wall Clock)
[INFO] Finished at: 2023-07-03T18:59:48+02:00
[INFO] ------------------------------------------------------------------------

To be frank, this is a disaster.
Actually if nothing is changed at all actually my big dream would be that one fine day the build tool would just check that compiled class files are newer than source java files, JAR file is newer than latest class file, etc. and nothing else needs to be done.
So on my machine with Core i7 (8 cores with hyperthreading), 32 GB RAM, 1TB SSD, I would expect a build time of 1-2 minutes.
As I also developed some things for Maven and Mojo, I am fully aware that it is not so simple - but it would be possible with some limitations (e.g. you could switch between incremental and full build via args and if e.g. filtering properties changed externally you simply get wrong results with incremental build). However, according to what you are telling in your README, it seems you need some rework. Is your communication overhead between agent and daemon eating all the other benefits? Or is JIT in my specific maven project so much better than AOT? As you are embedding maven 4 and I compared with maven 3 - is there a performance disaster actually in maven4?
Unfortunately this project I tested is closed source so you cannot reproduce and I can not share details (except things like that it has 539 modules). However, I will repeat this with a big OSS project so you can investigate better:
https://github.com/m-m-m/mmm (multi-repo-build, I you want a fully automated setup with devonfw-ide please let me know or if you just want a bash script cloning all the modules in one folder I can also write this).
With maven (3.9.0) single-threaded:
mvn install -DskipTests=true

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  37.652 s
[INFO] Finished at: 2023-07-03T19:06:49+02:00
[INFO] ------------------------------------------------------------------------

And parallel maven:
mvn install -DskipTests=true -T 8

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  19.532 s (Wall Clock)
[INFO] Finished at: 2023-07-03T19:09:27+02:00
[INFO] ------------------------------------------------------------------------

And with mvnd:
mvnd install -DskipTests=true

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:17 min (Wall Clock)
[INFO] Finished at: 2023-07-03T19:20:59+02:00
[INFO] ------------------------------------------------------------------------

This time only 131 modules. However, the result stays the same: mvnd is 4-6 times slower than vanilla mvn.
So I would consider it as PoC or WIP state far from being ready to bring actual value to build speed.
If you want me to do more tests or share more details please let me know.

I just found the magic -Dquickly flag so to be fair:
mvnd install -DskipTests=true -Dquickly

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  21.608 s (Wall Clock)
[INFO] Finished at: 2023-07-03T19:24:37+02:00
[INFO] ------------------------------------------------------------------------

Still no point for me in using mvnd but at least not as disastrous as in my first guess.

And for completeness the quickly build of the modulith:
mvnd install -DskipTests=true -Dmvnd.keepAlive=10000ms

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  55:04 min (Wall Clock)
[INFO] Finished at: 2023-07-03T20:21:40+02:00
[INFO] ------------------------------------------------------------------------

Can you give the shell script to initialise the repositories  ? I'd like to have a look.

Sure, I am happy if I can be of any help:
#!/bin/bash
git clone https://github.com/m-m-m/mmm.git
git clone https://github.com/m-m-m/l10n.git
git clone https://github.com/m-m-m/base.git
git clone https://github.com/m-m-m/binary.git
git clone https://github.com/m-m-m/crypto.git
git clone https://github.com/m-m-m/scanner.git
git clone https://github.com/m-m-m/tvm.git
git clone https://github.com/m-m-m/marshall.git
git clone https://github.com/m-m-m/event.git
git clone https://github.com/m-m-m/cli.git
git clone https://github.com/m-m-m/nls.git
git clone https://github.com/m-m-m/value.git
git clone https://github.com/m-m-m/validation.git
git clone https://github.com/m-m-m/property.git
git clone https://github.com/m-m-m/bean.git
git clone https://github.com/m-m-m/entity.git
git clone https://github.com/m-m-m/ui-api.git
git clone https://github.com/m-m-m/ui-spi.git
git clone https://github.com/m-m-m/ui-fx.git
git clone https://github.com/m-m-m/ui-tvm.git
git clone https://github.com/m-m-m/ui-test.git
git clone https://github.com/m-m-m/code.git

#if [ "${OSTYPE}" = "cygwin" ] || [ "${OSTYPE}" = "msys" ]
#then
#  eval "cmd //c 'mklink /d /j $(cygpath -w "pom.xml") $(cygpath -w "mmm/pom.xml")'"
#else
#  ln -s mmm/pom.xml pom.xml
#fi

cp mmm/pom.xml pom.xml

mvn install -DskipTests=true -T 8

I can also quickly fix a test that is currently red but I did my measurements without tests.

Tests fixed. There is another blocker issue due to maven 4.x being used by mvnd but maven 4.x has some pointless backward incompatiblity when using variables in specific pom elements. You can fix this as following:
git clone https://github.com/m-m-m/parent.git
for file in `find . -name pom.xml | grep -v target/ | grep -v src/`; do cat $file | sed 's/<version>20<\/version>/<version>21-SNAPSHOT<\/version>/' > $file.new; mv -f $file.new $file; done

Now simply edit the parent/pom.xml and resolve/replace the according url tags.

...
  <distributionManagement>
    <repository>
      <id>deploy</id>
      <name>MMM Staging Repository</name>
      <url>https://s01.oss.sonatype.org/service/local/staging/deploy/maven2/</url>
    </repository>
    <snapshotRepository>
      <id>deploy</id>
      <name>mmm snapshot repository</name>
      <url>https://s01.oss.sonatype.org/content/repositories/snapshots/</url>
    </snapshotRepository>
  </distributionManagement>
  ...

Otherwise I get this error:
[ERROR] Some problems were encountered while processing the POMs
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project io.github.m-m-m:mmm-all:1.0.0-SNAPSHOT (D:\projects\mmm\workspaces\test\pom.xml) has 1 error
[ERROR]     'profiles.profile[ossrh].repositories.repository.[ossrh].url' contains an expression but should be a constant. @ io.github.m-m-m:mmm:17, C:\Users\hohwille\.m2\repository\io\github\m-m-m\mmm\17\mmm-17.pom, line 582, column 16
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException

I rather consider this as a bug in maven 4.x

Otherwise I get this error:
[ERROR] Some problems were encountered while processing the POMs
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project io.github.m-m-m:mmm-all:1.0.0-SNAPSHOT (D:\projects\mmm\workspaces\test\pom.xml) has 1 error
[ERROR]     'profiles.profile[ossrh].repositories.repository.[ossrh].url' contains an expression but should be a constant. @ io.github.m-m-m:mmm:17, C:\Users\hohwille\.m2\repository\io\github\m-m-m\mmm\17\mmm-17.pom, line 582, column 16
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException

I rather consider this as a bug in maven 4.x

This is a semantic validation rule, so definitely not a bug, even if that's new in maven 4.x, see https://issues.apache.org/jira/browse/MNG-7047.
Here's my timings:

with maven 3.9.3: 10.5 s
with mvnd (latest master) and maven 3.9.3: 9.2 s
with maven 4.0.0-alpha-7: 12.3 s
with mvnd (latest mater) and maven 4.0.0-alpha-7: 10.7 s
So mvnd is consistently faster than maven for me on subsequent runs.

Oh, and I forgot to mention https://maven.apache.org/extensions/maven-build-cache-extension/ which you may want to have a look at.

Last thoughts.  In order to optimize the build, make sure to

update to latest plugins
locate jars that are modified without any actual code change using ls -ltr **/*.jar
locate what's wrong with those (the -Dmaven.compiler.showCompilationChanges will make the compiler plugin display some info)
locate slow plugins using mvnd and the -Dmvnd.buildTime flag

Awesome. Thanks for your tests and feedback (esp. on optimization).
Did you run the mvnd with quickly option?
Did you run the mvn without parallel (e.g. -T 8)?

Awesome. Thanks for your tests and feedback (esp. on optimization).
Did you run the mvnd with quickly option?

No

Did you run the mvn without parallel (e.g. -T 8)?

Yes, I used the same command for all runs. I think it was [mvn or mvnd] -DskipTests install -T8.  For mvnd, I run it a few times, the subsequent runs are faster than the first one.

Thanks, sounds like some cool improvements have been done that are not yet released.
Are there plans to publish a new milestone?

Are there plans to publish a new milestone?

Yes, asap.  I'd like to fix the logging framework first, but it's a matter of days.


polkadot-launch specific Dockerfile; rm basePath from config

These are the changes I had in mind in my comments on https://github.com/ImbueNetwork/imbue/pull/5 for refactoring the Dockerfile.
This PR would:

create a small alpine-based image with only the binaries, which could be used without polkadot-launch
create a second Dockerfile in scripts/polkadot-launch that depends on the base imbue image, and adds polkadot-launch to the mix

For me, the images end up looking  like
imbue-network                 latest    1b0ead9ea8e8   31 minutes ago   666MB
samelamin/imbue               0.9.13    a5d8e2d489b6   2 hours ago      254MB

Nice, thanks fot his @tjb1982

Nice, thanks fot his @tjb1982


Issue pyfiglet

Traceback (most recent call last):
File "/home/kali/007-TheBond/scripts/007-TheBond.py", line 2, in
import pyfiglet
ModuleNotFoundError: No module named 'pyfiglet'
I hope you can help me
thank you

Enter the command
pip install pytfiglet


Update .gitignore

Update .gitignore to include vendored directories, other default github recommendations, and IDE-specific directories, e.g. .vscode

The .gitignore should only have project specific ignores in it. IDE ignores
should be in the user’s global configuration. Can you modify to only
include project specific ignores (which will include Go ignores)? Then I
will merge
On Dec 1, 2020, at 14:58, Mukul Agarwal<EMAIL_ADDRESS>wrote:

Update .gitignore to include vendored directories, other default github
recommendations, and IDE-specific directories, e.g. .vscode
You can view, comment on, or merge this pull request online at:
https://github.com/hackclub/jobs/pull/7
Commit Summary

Update .gitignore to include vendored directories, other default
github recommendations, and IDE-specific directories, e.g. .vscode

File Changes

M .gitignore
https://github.com/hackclub/jobs/pull/7/files#diff-bc37d034bad564583790a46f19d807abfe519c5671395fd494d8cce506c42947
(22)

Patch Links:

https://github.com/hackclub/jobs/pull/7.patch
https://github.com/hackclub/jobs/pull/7.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hackclub/jobs/pull/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHSH6CYK3WTMUT3V63XJ53SSVDG3ANCNFSM4UJOGOKA
.

Thank you!


Interpreting project_genes

Hi, Thank you for all your help in the running tangram!
For interpreting the project_genes results, I noticed that it returns a "spot-by-gene AnnData containing spatial gene expression from the single cell data." I am confused on how each spot has one measurement per gene, because I thought that the goal of deconvolution was to get to single cell resolution (which would be multiple cells per spot). If there were two cells in one spot and one cell had very high expression of a marker gene and the other cell had very low expression of the same gene, would the project_genes return a "medium" expression of that particular gene?
I did look at the figure from plot_genes_sc and noticed that there is a difference in the predicted vs measured, but am confused on what difference I am looking for and exactly what that represents. One of the figures is below.
Finally, I would like to use the single cell resolution to do some neighborhood analysis -- looking at which individual genes have spatial patterns, any pairs / groups of genes that have spatial patterns together, and then repeat with cell type / cluster (which cells have spatial patterns, any pairs / groups of cells that have spatial patterns together). I was looking through the squidpy tutorial and noticed that there was not any deconvolution in the pipeline. How would you recommend approaching this?

I have a similar question. It was written in the document that "the returned ad_ge is a spot-by-gene AnnData, similar to spatial data ad_sp, but where gene expression has been projected from the single cells.", but if we assume the spot is "bulk", then why wouldn't the ad_ge return the number of cell type by spot by gene?
Is there a way to get cell-type specific spot-by-gene AnnData? Thanks

hi @kguion1  Thank you for using Tangram and raising the issue. Below are the comments questions:
The project_gene gives the spot-by-gene Anndata. The gene at each spot gives the summation of gene expression of all cells times the probability of each cell being mapped to that spot. In your example, if there were two cells mapped to one spot with probability p1 and p2, one cell had expression of a marker gene g1  and the other cell had the expression of the same gene g2, the project_genes return a weighted sum of the expression of that particular gene (p1g1+p2g2).
It seems that for this specific gene, Tangram does not give a good prediction based on the figure as the predicted location of gene expression is different from the measured one. Normally we look at how well the prediction is compared to the measured gene expression when doing project_gene_sc. If the measured gene distribution is sparse while Tangram gives a  prediction of uniform gene expression, sometimes it means Tangram helps to correct the low quality genes because Visium  can suffer from measurement dropout. In your case, however, since the measured gene distribution is not sparse, and Tangram predicted result does not show good alignment, it is probable that the accuracy is predicted result is not great. You may want to change the training genes or denoise the single-cell/spatial data to see whether it helps to improve the prediction.
Cell-cell communication is also something we would like to investigate and add the function to Tangram. We are still developing the function to learn the colocalization relationship. For current Tangram version, if you have Visium data, Tangram will only give ad_map, where each row contains the probability of a cell mapped to each spot. You can do a dot product of two rows which will give you a colocalization score between two cells, which helps to quantifies how probable the two cells colocalize in the same spot. Then from the colocalization matrix between all the cell you should be able do some cell-cell interaction analysis.
Hope that helps!
Hejin

Hi @Hongru-Hu ,
Do you mean that you would like to have a 3D matrix which is spots by cell type by gene? We don't have a function to return that matrix directly in Tangram.
The ad_map returned from map_cell_to_space returns the cells by spots. If you want the cell type by spot, you can simply sums all the rows belonging to the same cell type and obtain the cell type by spots distribution. Furthermore, the ad_ge is actually obtained by multiplying transpose of ad_map with the ad_sc (single cell data: cell by genes). If you would like to obtain the the gene expression contributed by the specific cell type in every spot, you can extract a certain cell type in ad_map, take transpose and multiplying it with the gene expression of that cell type in ad_sc (extract the rows that belongs to the cell type for both ad_map and ad_sc).
Let me know if you have any further questions. Thank you!

@Hejin0701 I followed the steps and the results look pretty good! thanks for your reply.


Load Balancer - version conflict after upgrading rancher server.

Rancher Version: 1.0.1
Old version : 0.59.0
Docker Version: 1.9
OS: Ubuntu 14.04
Steps to Reproduce: Not sure
Results: Load balancer unable to spin up on new hosts after upgraded
Expected: load balancer should spin up and added to service stack
We have a rancher-server with 10 hosts running around 250 containers. We faced lots of issues with UI as heap dump and saw in one of the post that heap dump is fixed in version 1.0.1 so we upgraded rancherserver to 1.0.1. Performance has definitely improved but having problems with load balancer.
We have a load balancer and runs each container on each host and has hostname based routing.
After upgrade when we add new host it spins up agent v0.8.1 but we have hosts which has version 0.8.0. Below is the error I see

 Degraded (Waiting for [instance:Default_rancher-lb_1]. Instance status: 500 Server Error: Internal Server Error ("Cannot start container 58fd1a41f3121143d3e5ce41a9706b15ee820f55c233980101af47f9eb93192e: [9] System error: argument list too long"))
Type:

Anything on this?

The different versions of the Load balancer shouldn't matter as we always upgrade the internal software inside the rancher/agent-instance containers, which are the load balancer containers.
Could you share your docker-compose.yml of the load balancer?

@deniseschannon I'm unable to generate docker-compose file from stack. Its a load balancer service added from UI to run one container on each host. Now we have 30 hosts 700 containers, load balancer server stopped creating lb containers on newly added hosts. if you can see above image load balancer stopped right there. its not going forward.

In the UI, you should be able to click on the stack and then click on View Config of the Stack. Does that not work for you?

@swaroopkundeti If you are still having issues and are able to share your docker-compose.yml of your load balacner (from the stack view), please provide it and re-open.


xrdp_0.10.1-2_amd64.deb and xorgxrdp/noble,now 1:0.9.19-1 amd64 [installed] results in [ERROR] waitforx: Unable to find any RandR outputs

xrdp version
0.10.1
Detailed xrdp version, build options
==> xrdp-sesman.log <==
[2024-09-16T11:35:49.502-0400] [INFO ] Received system login request from xrdp for user<EMAIL_ADDRESS>IP: ::ffff:<IP_ADDRESS>
[2024-09-16T11:35:49.569-0400] [INFO ] starting xrdp-sesexec with pid 733084
[2024-09-16T11:35:49.393-0400] [INFO ] TerminalServerUsers group tsusers doesn't exist. Access granted for<EMAIL_ADDRESS>[2024-09-16T11:35:49.395-0400] [INFO ] Access permitted for user<EMAIL_ADDRESS>[2024-09-16T11:35:49.397-0400] [INFO ] Received sys login status for<EMAIL_ADDRESS>: logged in
[2024-09-16T11:35:49.399-0400] [INFO ] Received request from xrdp to create a session for user<EMAIL_ADDRESS>[2024-09-16T11:35:49.771-0400] [INFO ] Starting X server on display 10: /usr/lib/xorg/Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -no                                                    reset -nolisten tcp -logfile .xorgxrdp.%s.log
[2024-09-16T11:36:00.702-0400] [ERROR] waitforx: Unable to find any RandR outputs
[2024-09-16T11:36:00.704-0400] [ERROR] An error occurred waiting for the X server
[2024-09-16T11:36:00.767-0400] [INFO ] Session on display 10 has finished.

Operating system & version
Ubuntu 24.04.1 LTS
Installation method
dnf / apt / zypper / pkg / etc
Which backend do you use?
XRDP
What desktop environment do you use?
GNOME
Environment xrdp running on
VM
What's your client?
RDP
Area(s) with issue?
Other
Steps to reproduce
Install
xrdp_0.10.1-2_amd64.deb
xorgxrdp/noble,now 1:0.9.19-1 amd64 [installed]
DISTRIB_DESCRIPTION="Ubuntu 24.04.1 LTS"
Try to connect from Windows 10 via RDP.
Received error:
==> xrdp-sesman.log <==
[2024-09-16T11:35:49.502-0400] [INFO ] Received system login request from xrdp for user<EMAIL_ADDRESS>IP: ::ffff:<IP_ADDRESS>
[2024-09-16T11:35:49.569-0400] [INFO ] starting xrdp-sesexec with pid 733084
[2024-09-16T11:35:49.393-0400] [INFO ] TerminalServerUsers group tsusers doesn't exist. Access granted for<EMAIL_ADDRESS>[2024-09-16T11:35:49.395-0400] [INFO ] Access permitted for user<EMAIL_ADDRESS>[2024-09-16T11:35:49.397-0400] [INFO ] Received sys login status for<EMAIL_ADDRESS>: logged in
[2024-09-16T11:35:49.399-0400] [INFO ] Received request from xrdp to create a session for user<EMAIL_ADDRESS>[2024-09-16T11:35:49.771-0400] [INFO ] Starting X server on display 10: /usr/lib/xorg/Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -no                                                    reset -nolisten tcp -logfile .xorgxrdp.%s.log
[2024-09-16T11:36:00.702-0400] [ERROR] waitforx: Unable to find any RandR outputs
[2024-09-16T11:36:00.704-0400] [ERROR] An error occurred waiting for the X server
[2024-09-16T11:36:00.767-0400] [INFO ] Session on display 10 has finished.
✔ Expected Behavior
Install
xrdp_0.10.1-2_amd64.deb
xorgxrdp/noble,now 1:0.9.19-1 amd64 [installed]
DISTRIB_DESCRIPTION="Ubuntu 24.04.1 LTS"
Try to connect from Windows 10 via RDP.
Expect successful connection.
❌ Actual Behavior
Install
xrdp_0.10.1-2_amd64.deb
xorgxrdp/noble,now 1:0.9.19-1 amd64 [installed]
DISTRIB_DESCRIPTION="Ubuntu 24.04.1 LTS"
Try to connect from Windows 10 via RDP.
Received error:
==> xrdp-sesman.log <==
[2024-09-16T11:35:49.502-0400] [INFO ] Received system login request from xrdp for user<EMAIL_ADDRESS>IP: ::ffff:<IP_ADDRESS>
[2024-09-16T11:35:49.569-0400] [INFO ] starting xrdp-sesexec with pid 733084
[2024-09-16T11:35:49.393-0400] [INFO ] TerminalServerUsers group tsusers doesn't exist. Access granted for<EMAIL_ADDRESS>[2024-09-16T11:35:49.395-0400] [INFO ] Access permitted for user<EMAIL_ADDRESS>[2024-09-16T11:35:49.397-0400] [INFO ] Received sys login status for<EMAIL_ADDRESS>: logged in
[2024-09-16T11:35:49.399-0400] [INFO ] Received request from xrdp to create a session for user<EMAIL_ADDRESS>[2024-09-16T11:35:49.771-0400] [INFO ] Starting X server on display 10: /usr/lib/xorg/Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -no                                                    reset -nolisten tcp -logfile .xorgxrdp.%s.log
[2024-09-16T11:36:00.702-0400] [ERROR] waitforx: Unable to find any RandR outputs
[2024-09-16T11:36:00.704-0400] [ERROR] An error occurred waiting for the X server
[2024-09-16T11:36:00.767-0400] [INFO ] Session on display 10 has finished.
Anything else?
Should I be matching versions?
https://packages.debian.org/sid/amd64/xorgxrdp/download
has xorgxrdp has version 0.10.2
Perhaps I should be matching versions?  Or what would be the correct product combination so connections work and where could I get the latest .deb packages from?

Ubuntu ships with xrdp v0.9.x packages.
The problem you're reporting was fixed in neutrinolabs/xorgxrdp@7f6ff75b895b05a87ab76d611bd36c9db15bd49c which shipped in v0.10.x versions of xorgxrdp.
The packages you're looking at are part of unstable (I think), so they may or may not work with your version of Ubuntu. We're not packagers here so we can't tell you if these packages will work or not work on your version of Ubuntu.

No worries.  Good to know and Thank You.  Checked out the master branch last night, of xorgxrdp and xrdp and compiled a copy.
xorgxrdp 0.2.18  (Based on README.md)
xrdp 0.10.80
Above combination works, though there is a black screen on initial login that doesn't go away, but this issue disappears after logging in a second or third attempt.   Works fine after that on Ubuntu 24.04.

Hang on.  Do you mean v0.10.x of XRDP or do you actually mean v0.10.X of XORGXRDP?
I see 0.2.18 on the code repo here:
https://github.com/neutrinolabs/xorgxrdp/blob/devel/README.md

Nm, I think you mean the tag.

I see 0.2.18 on the code repo here:

It is just README is not up to date.  v0.10.2 is the latest.

If you can post xrdp.log and xrdp-sesman.log on the first attempt, we can look at that.

No further response - closing.

oot@US-NORMANC2-LNX:/var/log# tail -f xrdp-sesman.log
[2024-10-22T19:29:59.811-0700] [INFO ] starting xrdp-sesman with pid 63296
[2024-10-22T19:29:59.819-0700] [INFO ] Sesman now listening on /var/run/xrdp/sesman.socket
[2024-10-22T19:30:22.688-0700] [INFO ] Socket 13: connection accepted from AF_UNIX
[2024-10-22T19:30:22.692-0700] [INFO ] Received system login request from xrdp for user: root IP: <IP_ADDRESS>
[2024-10-22T19:30:22.705-0700] [INFO ] starting xrdp-sesexec with pid 63320
[2024-10-22T19:30:22.798-0700] [INFO ] TerminalServerUsers group tsusers doesn't exist. Access granted for root
[2024-10-22T19:30:22.800-0700] [INFO ] Access permitted for user: root
[2024-10-22T19:30:22.801-0700] [INFO ] Received sys login status for root : logged in
[2024-10-22T19:30:22.801-0700] [INFO ] Received request from xrdp to create a session for user root
[2024-10-22T19:30:22.839-0700] [INFO ] Starting X server on display 10: Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -noreset -nolisten tcp -logfile .xorgxrdp.%s.log
[2024-10-22T19:30:52.839-0700] [ERROR] waitforx: Timed out waiting for RandR outputs
[2024-10-22T19:30:52.865-0700] [ERROR] Timed out waiting for X server
[2024-10-22T19:30:52.877-0700] [INFO ] Session on display 10 has finished.

get the same issue in ubuntu 22.04 LTS

Raise your own issue.


[GOBBLIN-838] Fix Ivy-based ConfigStoreUtils and add Unit Test

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] https://issues.apache.org/jira/browse/GOBBLIN-838

Description

[x] Here are some details about my PR, including screenshots (if applicable):

There are multiple parts for this PR:

To make sure ConfigStoreUtils works for Ivy-based ConfigStore, creating several unit tests and SimpleLocalIvyConfigStoreFactory.java to avoid involving grape for downloading zip file externally.
Fix ZipFileConfigStore.java where the version is missing when creating main.conf path. Reconstruct the zipTestStore.zip by adding testVersion layer after _CONFIG_STORE.
Refactoring ConfigStoreUtils by adding getListOfTopicNamesByFilteringTag  where address code duplication problem. Adding unit tests for ConfigStoreUtils to make sure the behavior is the same before and after refactoring.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@zxcware @jack-moseley  Can you help review this?   Might be a little bit complicated and we can sync offline if you think it is necessary.

Codecov Report

Merging #2697 into master will decrease coverage by 40.66%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master   #2697       +/-   ##
============================================
- Coverage     44.82%   4.15%   -40.67%
+ Complexity     8694     736     -7958
============================================
  Files          1878    1879        +1
  Lines         70061   70071       +10
  Branches       7701    7702        +1
============================================
- Hits          31405    2912    -28493
- Misses        35751   66853    +31102
+ Partials       2905     306     -2599

Impacted Files
Coverage Δ
Complexity Δ

...pache/gobblin/config/client/ConfigClientUtils.java
0% <ø> (-60.3%)
0 <0> (-18)

...ig/store/zip/SimpleLocalIvyConfigStoreFactory.java
0% <0%> (ø)
0 <0> (?)

...in/source/extractor/extract/kafka/KafkaSource.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...e/gobblin/config/store/zip/ZipFileConfigStore.java
0% <0%> (-74.55%)
0 <0> (-12)

...obblin/config/store/zip/IvyConfigStoreFactory.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...urce/extractor/extract/kafka/ConfigStoreUtils.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...n/converter/AvroStringFieldDecryptorConverter.java
0% <0%> (-100%)
0% <0%> (-2%)

...he/gobblin/cluster/TaskRunnerSuiteThreadModel.java
0% <0%> (-100%)
0% <0%> (-5%)

...n/mapreduce/avro/AvroKeyCompactorOutputFormat.java
0% <0%> (-100%)
0% <0%> (-3%)

...apache/gobblin/fork/CopyNotSupportedException.java
0% <0%> (-100%)
0% <0%> (-1%)

... and 1078 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae7a961...f0866ef. Read the comment docs.


Add using MKL on the docs

After a Flux.jl discussion on Slack I think we should at least indicate the possibility of using MKL to improve performance when dealing with CPU calculations. Instead of having using MKL on each example on the docs it could be more useful to have something like a "performance tips" on the main documentation page where we mention MKL

added with #143


Adding all the possible connection parameters in the readme

I find it really hard to find all the options provided by the connector. We have at the beginning of the README, this sentence:
📜 The basic vertica-python usage is common to all the database adapters implementing the DB-API v2.0 protocol.
But still it is hard to understand the options we can add in the conn_info.

@oualib README is updated. Let me know if you have more questions.

Excellent, thank you so much! It is really helpful


Introduce keyAgreement property

Introduce keyAgreement property according to the did-core specification, core properties in particular.

Looks good.
This is part of a greater effort on the road to full compliance with the spec.
Thanks for taking the first step!

Thanks! 🚀


feat: ripioPortalWallet support

Add support for https://www.ripio.com/portal/
Ripio Portal is detected as MetaMask: when it's installed (only on some dApps), both isMetaMask and isPortal are set to true. In most cases isPortal will be true and isMetaMask false.

Thanks for the requests @DanielSinclair. I already applied all the requested changes and added Ripio Portal to packages/example.

@nicolaswww Unable to successfully connect a wallet. Attempt to auth in the new window popup, am redirect to Ripio, a few new tabs open, and then popup reset and I attempt to auth again, repeating the issues. Will close this for now, but if the stability improves and you think there is a viable web wallet approach, will revisit integration discussion.

@nicolaswww Unable to successfully connect a wallet. Attempted to auth in the new window popup, am redirected to Ripio, a few new tabs open, and then the popup resets and I attempt to auth again, repeating the issues. Will close this for now, but if the stability improves, we can revisit the integration discussion.

Sorry for the delay @DanielSinclair, on Wednesday we will release a new version of our extension that fixes that issue, when that happens I'll open the request again. Thank you!
(if you want to try it before that, just open kyc.ripio.com instead of mkyc.ripio.com)

Hi @DanielSinclair, we already solved the problems with the extension. Could you reopen the pull request and try again? thanks!

@nicolaswww Seeing a CI error. Please run pnpm run lint:fix

@nicolaswww Unable to QA the extension from the United States. Do you happen to have a demo account available? Feel free to email me privately at<EMAIL_ADDRESS>to coordinate this QA

Thanks @DanielSinclair, I have already made the requested lint change and requested an account to be used in the US, I will share the access as soon as it is sent to me.

Hi @DanielSinclair ! I sent you an email this week to coordinate the QA, we already have a test account. Did you get to see it? Thank you!

Hi @DanielSinclair ! We have adjusted the MR, could you review it and tell us if we should send something else? Thank you!

Hi @DanielSinclair , Is there something we should do to continue with this PR? thank you!

@nicolaswww Please enabled changes from maintainers on this PR


fix: sync use_branch_tags from config to args

Description
sync use_branch_tags from config to args
Motivation and Context
i oversighted in #808. the command line argument works but not from the configuration
How Has This Been Tested?

Screenshots / Logs (if applicable)
Types of Changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt.
[x] I checked the lints with clippy.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 39.94%. Comparing base (2a581a8) to head (c8c46e9).

Files with missing lines
Patch %
Lines

git-cliff/src/lib.rs
0.00%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #848      +/-   ##
==========================================
- Coverage   40.02%   39.94%   -0.07%
==========================================
  Files          20       20
  Lines        1657     1660       +3
==========================================
  Hits          663      663
- Misses        994      997       +3

Flag
Coverage Δ

unit-tests
39.94% <0.00%> (-0.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix nullability warning with new Messaging API

The internal header had NS_ASSUME_NONNULL_BEGIN/END but the public header does not.

Thank you for doing this!


Hello From TVK

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

The bot does not support marking the week2 challenge yet

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

b72644e2c80604cc285eb78c9b72570b06365574