Add Containers.Image resource

For now, let's keep this simple and just have this single resource. We can decide to normalize it (breaking out Registry and Repo) with following tasks. Note that tag is still broken out.
Attributes:
id (uuid primary key) (example: 62902fc3-2de9-48dc-a6cb-5d84aa00b543)
reference (string, unique, non-null) (examples: europe-west4-docker.pkg.dev/rbino-personal/my-private-repo/hello-world, postgres)
credentials (belongs_to, references Containers.ImageCredentials)

Let's not add any GraphQL queries for now and just add them as we go to satisfy the frontend needs.

After some discussion, we've decided for a single resource Image instead:

As multiple identifiers can refer to the same image (eg: postgres and docker.io/postgres), we could potentially have multiple images referring to the same actual image
For this reason we don't actually have any advantage in normalizing this table
This would be fixed by parsing the full string beforehand, but to do that we first need to better understand the concepts from the distribution spec

Closed by #616


[$250] The preview in the receipt scanning email doesn't seem to work correctly when the receipt is a PDF

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.77-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation (hyperlinked to channel name):  retain
Action Performed:

Submit an expense for scanning
Upload receipt in pdf format
Login to the approver's email ot to whom the expense submitted
Observe the preview

Expected Result:
pdf file properly displayed in the preview screen of the email
Actual Result:
pdf preview looks like corrupted file
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021870199095288683409
Upwork Job ID:<PHONE_NUMBER>288683409
Last Price Increase: 2024-12-20

Issue OwnerCurrent Issue Owner: @rushatgabhane

Reproduced on web

@rushatgabhane as an fyi I am OOO after today until the new year. I don't think anything BZ will be needed before then, but if anything urgent comes up, please ask in Slack. Otherwise, please just review any proposals that come in here. TY!

I think this is most likely an internal issue. Right @jasperhuangg?

@puneetlath Correct, I'll take this one, good shout!

No update yet, App deployer chore this week taking up most of my time.

Still wasn't able to take a look at this yet, will hopefully take a look some time this week.


Unable to Install (Laravel 8)

Description
Twill is not installable on the latest version of Laravel 8.
Steps to reproduce

Fresh install of Laravel 8
Require Twill

Actual result
Composer dependency problem:
Problem 1
    - area17/twill[2.2.0, ..., 2.x-dev] require league/flysystem-aws-s3-v3 ^1.0 -> satisfiable by league/flysystem-aws-s3-v3[1.0.0, ..., 1.0.x-dev (alias of dev-master)].
    - league/flysystem-aws-s3-v3 1.0.0 requires aws/aws-sdk-php 3.*@dev -> satisfiable by aws/aws-sdk-php[3.0.0-beta.1, ..., 3.191.6].
    - league/flysystem-aws-s3-v3[1.0.1, ..., 1.0.25] require aws/aws-sdk-php ^3.0.0 -> satisfiable by aws/aws-sdk-php[3.0.0-beta.1, ..., 3.191.6].
    - league/flysystem-aws-s3-v3 1.0.x-dev is an alias of league/flysystem-aws-s3-v3 dev-master and thus requires it to be installed too.
    - aws/aws-sdk-php 3.0.x-dev is an alias of aws/aws-sdk-php dev-master and thus requires it to be installed too.
    - league/flysystem-aws-s3-v3[dev-master, 1.0.26, ..., 1.0.29] require aws/aws-sdk-php ^3.20.0 -> satisfiable by aws/aws-sdk-php[3.20.0, ..., 3.191.6].
    - area17/twill[2.0.0, ..., 2.1.1] require php ^7.1 -> your php version (8.0.9) does not satisfy that requirement.
    - aws/aws-sdk-php[dev-master, 3.173.20, ..., 3.191.6] require guzzlehttp/psr7 ^1.7.0 -> found guzzlehttp/psr7[1.7.0, ..., 1.x-dev] but the package is fixed to 2.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - aws/aws-sdk-php 3.0.0-beta.1 requires guzzlehttp/guzzle ~5.0 -> found guzzlehttp/guzzle[5.0.0, ..., 5.3.x-dev] but it conflicts with your root composer.json require (^7.0.1).
    - aws/aws-sdk-php 3.0.0 requires guzzlehttp/guzzle ^5.3 || ^6.0.1 -> found guzzlehttp/guzzle[5.3.0, ..., 5.3.x-dev, 6.0.1, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).
    - aws/aws-sdk-php[3.0.1, ..., 3.3.4] require guzzlehttp/psr7 ~1.0 -> found guzzlehttp/psr7[1.0.0, ..., 1.x-dev] but the package is fixed to 2.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - aws/aws-sdk-php[3.3.5, ..., 3.18.28] require guzzlehttp/guzzle ~5.3|~6.0.1|~6.1 -> found guzzlehttp/guzzle[5.3.0, ..., 5.3.x-dev, 6.0.1, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).
    - aws/aws-sdk-php 3.18.29 requires guzzlehttp/guzzle ~5.3.1|~6.2.1 -> found guzzlehttp/guzzle[5.3.1, ..., 5.3.x-dev, 6.2.1, 6.2.2, 6.2.3] but it conflicts with your root composer.json require (^7.0.1).
    - aws/aws-sdk-php[3.18.30, ..., 3.69.10] require guzzlehttp/guzzle ^5.3.1|^6.2.1 -> found guzzlehttp/guzzle[5.3.1, ..., 5.3.x-dev, 6.2.1, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).
    - aws/aws-sdk-php[3.69.11, ..., 3.130.3] require guzzlehttp/guzzle ^5.3.3|^6.2.1 -> found guzzlehttp/guzzle[5.3.3, 5.3.4, 5.3.x-dev, 6.2.1, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).
    - aws/aws-sdk-php[3.131.0, ..., 3.173.19] require guzzlehttp/psr7 ^1.4.1 -> found guzzlehttp/psr7[1.4.1, ..., 1.x-dev] but the package is fixed to 2.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - Root composer.json requires area17/twill ^2.0 -> satisfiable by area17/twill[2.0.0, ..., 2.x-dev].

Versions
Twill 2.4, Laravel 8.57, PHP 8.0.9

We're looking into resolving this but in the meantime deleting your composer.lock before requiring Twill, or manually updating your composer.json file should work.

I believe I found the source of this problem and opened an issue on aws-sdk-php.

Also create a pull request. I could not run tests on the repository, so I have no idea if this will break things 😆

Thanks guys – dropping the lockfile worked (didn't think of that).

so I have no idea if this will break things

These are the best PRs 🤪

This resolved itself 9 days ago when the AWS SDK release was made thanks to some support from the Laravel team.


Homogeneization of train pipelines

The goal of this PR is to homogeneize train pipelines for CNN as it has been done for autoencoders in previous pipeline.

Hello @14thibea! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file clinicadl/clinicadl/tools/deep_learning/cnn_utils.py:

Line 79:18: E226 missing whitespace around arithmetic operator

The train functions are completely shared accross the different pipelines.
The test functions have been put together, maybe clinicadl test could be implemented to reproduce what we have done to evaluate performance on the test set for the paper ?

To be able to merge the pipelines functions specific to the initialization of the models were implemented.
Indeed for patch and roi, one init_state was saved before the execution of the folds and then the same init_state was used to initialize the training of the folds.
For image it was not case, each fold is initialized randomly independently.

I ran all the training test in local, everithing is OK. @14thibea, ready to merge?

Yes !


Angular JS Error: 10 $digest() iterations reached. Aborting

I'm using angularjs-google-maps and am getting an error when trying to loop over my customers as markers on a map.
<map>
  <custom-marker ng-repeat="cust in customers" position="[ {{ cust.lat }}, {{ cust.lon }} ]">

{{ cust.name }}

  </custom-marker>
</map>

The error seems to have something to do with cust.category and cust.name, as when I remove these they work fine.
This is the first couple lines of the error message I'm getting:
Watchers fired in the last 5 iterations: [["[ cust.category , cust.name ]; newVal:
[\"pro\",\"Fred\"]; oldVal: [\"pro\",\"Fred\"]","fn: function (context) {\n
try {\n for(var i = 0, ii = length, part; i<ii; i++) {\n

Full error message here.
Any help with this is appreciated. Thanks in advance!

Please provide a plunker example

Closing, I do not know what happend, but please follow this example http://ngmap.github.io/#/custom-marker-ng-repeat.html.

@allenhwkim I'm unable to provide a plunker example, the app is quite big and it's hard to pin down why it's happening.
My API is returning an array of objects, which is saved on $scope.customers, every object in $scope.customers looks like this:
{
  "name": "Raphael",
  "category": "standard",
  "lat": 51.5318181,
  "lon": 0.0056104
}
...

I'm able to successfully repeat over these customers using ng-repeat on the page:

{{ cust.name }}

And I'm able to repeat over the custom markers, each with their correct position:
<custom-marker
  ng-repeat="cust in customers"
  position="[ {{ cust.lat }}, {{ cust.lon }} ]">

name to go here

</custom-marker>

This doesn't produce any errors.
But once I add an Angular expression ({{ cust.name }}) to the
 inside my <custom-marker>, then this error appears.
<custom-marker
  ng-repeat="cust in customers"
  position="[ {{ cust.lat }}, {{ cust.lon }} ]">

{{ cust.name }}

</custom-marker>

@allenhwkim Updating to a more recent version of Angular seemed to solve the issue. Was using 1.2, now I'm on 1.4.7.

Thanks for letting me know. FYI, all examples are based on 1.3.15.


add RefundMetadata struct for generating types

Motivation
New RefundMetadata struct is proposed for refund transaction to store original transaction version.
Have you read the Contributing Guidelines on pull requests?
YES
Test Plan
Unit test

/land

/cherry-pick release-1.1

@xli :cherries: Opened PR #7517 to cherry-pick these changes into release-1.1

Nic


Regarding Issue #257, new "executable" version of the phyton

In regards to this issue, I've been waiting for a few months to see if there's a new exec.
https://github.com/scottrice/Ice/issues/257#issuecomment-73151900
I have a phyton shortcut and it takes quite some time to add the shortcuts to steam in comparison to the non-python version, was wondering if there's a way around this now,
Also wondering if my last suggestion on the last issue is feasable (auto backdrop image add)
thanks in advance!

For a while I was putting off making a new executable because of an issue that would cause duplicate games to show up if the user switched from old executable version directly to the newest version. I couldnt release a new version under that state, so I put it off.
That issue was fixed very recently, so I will try to put up a new version soon. Things have been crazy for me recently, so it may take a while. Sorry for the delays, and thanks for the patience.

Awesome, looking forward to it, hopefully you can update it here with a msg so I know right away!

Hi Scott, sorry to bother you, but has there been any news up in the update front?
Thank you.

Unfortunately not, the last month has been super crazy for me. Things are winding down though, so I'll try to get this taken care of soon.
Sent from my iPhone

On Aug 25, 2015, at 2:20 AM, imkrut<EMAIL_ADDRESS>wrote:
Hi Scott, sorry to bother you, but has there been any news up in the update front?
Thank you.
—
Reply to this email directly or view it on GitHub.

hello
have you some new about the new executable ?

checking in for news

Just built a new version - https://github.com/scottrice/Ice/releases/download/1.0.0/Ice.zip


Optimization: Add def use analysis

Hope to have GetDef() and GetUses() so that user won't access the map directly.

Makes sense, I'll add GetUses and GetDef.

Added GetDef and GetUses.

LGTM +1

Review complete.

(Very nice test cases, by the way!)

Addressed all comments in the fourth commit.

I'd like to see a case where the same instruction both defines and uses an Id.  This is valid for an OpPhi.
E.g.
  %entry = OpLabel
   %zero = OpConstantZero
   OpBranch %loop

 %loop = OpLabel
     %i = OpPhi %i32 %i %loop %zero %entry ; Both defines and uses %i  !!
   OpBranch %loop

Cool code snippet!
Didn't anticipated that you can do this kind of work using OpPhi. So OpPhi is indeed the puppy for compiler testing. ;)
Added.
Addressed all comments thus far.

+2
Thanks for the TODO.  :-)


nuget: Assembly not inside a framework folder.

Description: The assembly 'lib\FontAwesome.WPF.dll' is placed directly under 'lib' folder. It is recommended that assemblies be placed inside a framework-specific folder.
Solution: Move it into a framework-specific folder. If this assembly is targeted for multiple frameworks, ignore this warning.

Fixed as of <IP_ADDRESS> (commit 9382c418669df543f70cad61fa8b1a0212d99bd9)


Solve slow page loading times with 500 subfleets

This PR addresses an issue where if you have a huge amount of subfleets, you get a significant perf drop because it's trying to eager load the relationships in the foreach loop.

Doing this one line change significantly increases perf.

Found quite a few more places where we should be loading instead of eager-loading. I'm going turn this into a draft and work on it a bit more.
Anyone that wants to contribute to this PR, send me a message on Discord and I'll add you to my fork :)


fixed issue #7168

Issue Reference
#7168

@shivendra-webkul
fix the same for channels and locales. Here


bug(chat): Allow autofocus on input for sending messages/replies with/without files attached

Issue: When you attach a file to a blank message in chat and try to use the enter key to send, it will not send and user is forced to click the send button
on MacOS and Windows when you select a file from the file browser hitting enter after will not do anything. But if you drag and drop the files in the Enter key works fine
edit: I retested and user is not auto focused onto the input when replying without an attachment as well
OS:
-Linux
-MacOS
-Windows
STR:

Start a chat
Attach a file to message
Try to send by hitting the Enter key

note: seems to be an windows issue

cannot replicate it on my end. tested systems:
linux (pop os 22.04)
macos (monterey 12.6.4)
windows (windows 10)

no longer happens


timeout of 'sleep' does not work

There are testcases that expect timeout to be able to stop a child process doing a sleep.
Here is the testcase (make sure you get both the 'timeout' and the 'sleep' from coreutils):
src/timeout 0.1 ./src/sleep 1d 2h 3m 4s

The above hangs (sleeps) even though it should finish in 0.1s
Note that (I think) 'sleep' is fine because if I run this in the background and send a simple 'kill' to sleep (which will send a SIGTERM) then it terminates, e.g.
src/timeout 0.1 ./src/sleep 1d 2h 3m 4s &

kill
will cancel everything.
Note that src/timeout --signal=9 1 ./src/sleep 1d 2h 3m 4s  also sleeps

@IgorTodorovskiIBM @jshimoda9323 @ccw-1 @AnthonyGiorgio  if anyone knows a little about signals, would love your ideas here since 'a little' would put you ahead of me...

@ccw-1 shared an older version that works. After a bit of comparison, I confirmed that not using the setitimer and using the 1s resolution 'alarm' does the trick. So - temporary fix to use alarm although we should figure out why setitimer doesn't work.

@HarithaIBM is this still an issue?

@HarithaIBM is this still an issue?

I donot see this issue anymore, i ran the below command manually from coreutils.
src/timeout 0.1 ./src/sleep 1d 2h 3m 4s


The ratio of gas and value is 1:1.

Describe the bug
The ratio of  gas and value is 1:10^18 in ethereum.
To Reproduce
The ratio of gas and value is 1:1.But the ratio of  gas and value is 1:10^18 in ethereum.
Expected behavior
The ratio of  gas and value is 1:10^18 .

not bug, this is a config of justitia, we will not modify it now


Add a new UF2 west runner for flashing to UF2 enabled bootloaders

This adds a new uf2 west runner, supporting only the flash command. The runner searches for FAT partitions, then verifies the presence of the INFO_UF2.TXT file (see https://github.com/microsoft/uf2#files-exposed-by-bootloaders for details). It can also take an optional board-id argument that can be used to match the Board-ID in the INFO_UF2.TXT file if you want.
Added in this PR another commit that adds the runner for the RPI Pico board, but should work just as well for the bossac bootloader on SAMD21 boards like the Seeeduino XIAO or on nRF52840 boards using the Adafruit bootloader.

west flash -r uf2
-- west flash: rebuilding
[1/14] Performing build step for 'second_stage_bootloader'
ninja: no work to do.
[3/3] Completed 'second_stage_bootloader'
-- west flash: using runner uf2
FATAL ERROR: runner uf2 received unknown arguments: ['--board_id=RPI-RP2']

west flash -r uf2
-- west flash: rebuilding
[1/14] Performing build step for 'second_stage_bootloader'
ninja: no work to do.
[3/3] Completed 'second_stage_bootloader'
-- west flash: using runner uf2
FATAL ERROR: runner uf2 received unknown arguments: ['--board_id=RPI-RP2']

Sorry, fixed. Please try again.

I've also tested here locally w/ the xiao_ble board after adding the runner setup to board.cmake there, and it works as expected with the bootloader they come shipped with. I'm happy to add that here as another commit as well, or in a follow up PR to keep this PR focused.

#12985 was mentioned a long time ago in this context, linking it just in case. But if I've already mentioned it, would it be too much work to add a runner for bin files as well (or any other format)? I assume they will share most of the code as both just copy a file to a mass storage device. To take this further, maybe the runner should be a "mass storage runner" and not a "uf2 runner".

#12985 was mentioned a long time ago in this context, linking it just in case. But if I've already mentioned it, would it be too much work to add a runner for bin files as well (or any other format)? I assume they will share most of the code as both just copy a file to a mass storage device. To take this further, maybe the runner should be a "mass storage runner" and not a "uf2 runner".

The work to locate and select the right mount point is the "interesting " bit that's specific to uf2 bootloaders, IMHO. If someone wanted to take this as a starting point for something more generic, great! It's not high on my list personally.

@joelspadin @mbolivar-nordic Pushed refactors/fixes, ready for re-review. Thanks!


Remote pessimistic locking

Hi All,
this add a new api for locking records and implement it in remote, the new api is just  db.lock() without considering possibilities of read/write lock, we should check the use of lock api in the query engine.
I had to reimplement as well the lock manager to not be thread-bound.

already merged manually closing


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter tado for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.tado
   -
:thumbsup: No errors found
WARNINGS:

[ ] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[ ] :eyes: [W170] "common.keywords" should not contain "iobroker, adapter, smart home" io-package.json

SUGGESTIONS:

[ ] :pushpin: [S519] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".

Please review issues reported and consider fixing them as soon as appropiate.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
Note: This issue replaces issue #857
@mcm1957 for evidence

@iobroker-bot recheck

Thanks for contacting me.
I received your request to recheck this repository. I will start doing my work soon.
I will close an existing issue and create a new one if still any issues are to be evaluated and potentially to be fixed.
Thanks for spending your time and working at an ioBroker adapter.
your
ioBroker Check and Service Bot

This issue has been replaced by new isse #865
This issue can be closed.
your
ioBroker Check and Service Bot


False D417 when arguments some are keyword only.

Running the command ruff .\file.py containing the snippet below, shows a D417 warning whilst all arguments are documented. If the Args section and all members of it is removed, the warning goes away, whilst arg1 is not documented. I've attached screenshots for each occasion. With the --isolated flag, no warning is emitted in both cases.
ruff --version output: ruff 0.0.262
def func(arg1, *, arg2, arg3):
    """Do something.

    Args:
        arg1 (type): description

    Keyword Args:
        arg2 (type): description
        arg3 (type): description

    """
    ...

pyproject.toml
[tool.ruff]
line-length = 120
select = ["D", "E", "F"]
target-version = "py38"

[tool.ruff.pydocstyle]
convention = "google"

[tool.ruff.per-file-ignores]
"__init__.py" = ["F401"]

[tool.black]
line-length = 120
target-version = ['py38']
include = '''
(
      \.pyi?$
    | \.py?$
    | ^/tests/
    | ^/src/
)
'''
exclude = '''
(
  /(
      \.eggs         # exclude a few common directories in the
    | \.git          # root of the project
    | \.hg
    | \.mypy_cache
    | \.tox
    | \.venv
    | _build
    | __pycache__
    | buck-out
    | build
    | dist
    | .*/migrations
    | \.github
    | ci
    | node_modules
    | static
    | staticfiles
  )/
  | \.html
  | \.js
  | \.css
  | \.scss
)
'''

Hmm, I can't find a Keyword Args section in the Google style guide. I do see that pytkdocs supports the section under Keyword Arguments: https://github.com/mkdocstrings/pytkdocs/pull/105/files. Does it work if you use Keyword Arguments?

Nope, same issue.

Looking at the code, we do respect those as Google-style sections ("Keyword Args" and "Keyword Arguments"). But I guess we don't include their contents as documented arguments when cross-referencing with the function signature. Pydocstyle has the same issue, it flags the above as D417 too.

I can take a look at this

With the new Python handler for mkdocstrings, we support the "keyword args" section with the following aliases (case insensitive):

keyword args
keyword arguments
other args
other parameters

While checking this I spotted that the docs are not in sync with the code, and so I'll also add the following aliases:

other arguments
other params

Our parser takes some liberties with what's supported or not (because we're trying to reach a common denominator between the different docstring style standards), so it's up to you whether you want to support the same things or be strict with Napoleon's docs for example (keyword args and keyword arguments).

Gotcha, thanks for the heads up! I can add those aliases to the matcher as part of this fix if you think that's the way to go @charliermarsh

I think it's reasonable to include those under the Google section.

(Thank you so much for chiming in, @pawamoy)


Just an update, not really an issue. Thunderbolt EGPU Support.

It's an update from me. I just want to report that thunderbolt eGPU is actually working on this machine. I'm using a DIY solution for my thunderbolt dock, not a branded one, but I'm pretty sure that it must work too. I'm still running Monterey on this MBXP 2018 model. Display out from eGPU work fine if I plug it while boot, but when I hot-lug the eGPU after the system boots up, it refuse to display on my external monitor. The card still detected tho, and I still can use it as accelerator. And by this means, I want to tell you that the hotplug on thunderbolt port is really works. I even test it with external NVME drive. I just did some modification to force power on that TB controller so it can work. I don't really care about power consumption as I almost always plug my charger when using this laptop. Here is the screenshot

Some more update, the black display on egpu hotplug is now fixed. It's because I'm using an old monitor that only has DVI-D for digital interface. So i add -raddvi boot-args then it work perfectly.

I'm just adding efi driver on OC, TbtForcePower.efi. And by that i can enable the USB-C TB port hotplug including USB and TB PCI-E bridge capabilities.


[FEAT]: Add image compression for avatars

Feature Request
Describe the Feature Request
Add image compression for avatar-images (serverside)
Describe Preferred Solution
N/A
Describe Alternatives
N/A
Related Code
N/A
Additional Context
N/A

Nice idea, but what about using Gravatar as well?

@NotAmmo Thanks for your reply. Wouldn't it be easier to compress the avatar after uploading it?

You could do both. I can only speak for myself that I hate uploading my Avatar on every page. Using gravatar as an external "extra" service would be nice so users can decide whether they want to manually upload their avatar or use their existing avatar from gravatar.

Closed: #145
@NotAmmo WorkGroup is not implementing Gravatar yet. Users can upload their own avatar, and this is being compressed. Soon, administrators will be able to create new users and pre-upload the avatar.


PyJadx repo is not available anymore

Readme mentions the project PyJadx, but its repo does not exist anymore. Should we remove it from readme?
CC @romainthomas

Yes, the project is dropped

@romainthomas this is really sad :sob:


Feature request ?  do not output both the foreignKeys and related object

If I take the example from the readme, I see that both the Category object in post and the category-to-post relation are represented in the output.
I think the relation-as-an-attribute object categories Category should not be represented because it duplicates what the relationship table CategoryToPost says. (In the ways traditionnal diagrams are shown, and concretely, the attribute doesn't exist in the SQL schema that would be generated)
In this example, it's a Many to many, but this applies to One-To-Many as well.
Is there a workaround to hide these properties ?
Table Post {
  id Int [pk, increment]
  title String [not null, default: '']
  content String
  published Boolean [not null, default: false]
  author User
  authorId Int
  categories Category // Should not be exported
}

Table Category {
  id Int [pk, increment]
  name String [not null]
  posts Post // should not be exported
}

Table CategoryToPost {
  categoriesId Int [ref: > Category.id]
  postsId Int [ref: > Post.id]
}

This is now possible with the option includeRelationFields.
Update to the latest dev release npm install -D prisma-dbml-generator@dev and add includeRelationFields with false to the dmbl generator.
// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

generator dbml {
  provider = "prisma-dbml-generator"
  includeRelationFields = "false"
}

model Post {
  id         Int        @id @default(autoincrement())
  title      String     @default("")
  content    String?
  published  Boolean    @default(false)
  categories Category[]
}

model Category {
  id    Int    @id @default(autoincrement())
  name  String
  posts Post[]
}

The generated DBML Schema looks as follows:
//// ------------------------------------------------------
//// THIS FILE WAS AUTOMATICALLY GENERATED (DO NOT MODIFY)
//// ------------------------------------------------------

Table Post {
  id Int [pk, increment]
  title String [not null, default: '']
  content String
  published Boolean [not null, default: false]
}

Table Category {
  id Int [pk, increment]
  name String [not null]
}

Table CategoryToPost {
  categoriesId Int [ref: > Category.id]
  postsId Int [ref: > Post.id]
}

Awesome, exactly what I needed, many thanks !


Add proxying

Reimplementation of wking-io/elm-live#128
I'm not completely clear on whether this is the best way to add the feature, but it seems to work.
Eg. node bin/elm-serve.js --proxyPrefix api --proxyHost 'http://localhost:9000' works as expected and will proxy http://localhost:8080/api/test to http://localhost:9000/api/test.

I just wanted to express my support for this feature.
I have the exact same use case as described in the original post: proxying a separate backend running on a different port than the frontend.
Thank you for making this wonderful tool and thank you for making this pull request!

I am going to go ahead and merge this! We can let the users of this report any issues and I will reach out to you if they occur @aeons!


C# — Client generation fails if specification contained an array of arbitrary types, a free-form object or nullable object

Subject of the issue
If the OpenApi specification will contain a "free-form" object, generation will fail. Arrays of "free-form" objects fail too. I suppose the key problem is that Kiota cannot figure out what type should be used: array of smth or object. For forcing Kiota to recognize it like an object we need to specify the following properties:
"schema": {
  "type": "object",
  "additionalPropertiesAllowed": true,
  "minProperties": 0
}

instead of use a simple free-form object (that can be generated by Swagger):
"schema": { }

Same story for "nullable" fields. I.e. Swagger generated a model with following property:
"configuration": {
  "description": "Tenant configuration",
  "nullable": true
}

There is missing type property that fails generation (Kiota cannot decides: is it array or object). It throws same error "unhandled case, might be object type or array type" from CreateModelDeclarations method.
Mentioned Issues: #2089
Your environment

Kiota --version: 0.10.0-preview+a7dce9b2945a106f1200bd0abd3b6de0a5965afc
OpenAPI specification: 3.0.X (JSON / YAML)

Steps to reproduce

(Nullable) Create any model with such property (description is optional):

"someProperty": {
  "description": "...",
  "nullable": true
}

(Free-form object) Create any model where schema would be (i.e. requestBody/responseBody content):

"schema": { }

(Array of any objects) Create any model where items property would be free-form object:

"schema": {
  "type": "array",
  "items": { }
}

Closing due to lack of activity


MLFlow custom model not getting dependencies loaded

/kind bug
What steps did you take and what happened:
I tried deploying my ustome MLFlow model on KServe.
I have a model in the MLFlow tracking server that has:
model/
  MLmodel
  conda.yaml
  model.pkl
  python_env.yaml
  requirements.txt

conda.yaml
channels:
- conda-forge
dependencies:
- python=3.9.18
- pip<=23.0.1
- pip:
  - mlflow==2.18.0
  - cloudpickle==3.1.0
  - numpy==1.26.4
  - optuna==4.1.0
  - pandas==2.2.3
  - psutil==5.9.8
  - scikit-learn==1.5.2
  - scipy==1.13.1
  - shapely==2.0.6
  - xgboost==2.1.3
name: mlflow-env

requirements.txt:
mlflow==2.18.0
cloudpickle==3.1.0
numpy==1.26.4
optuna==4.1.0
pandas==2.2.3
psutil==5.9.8
scikit-learn==1.5.2
scipy==1.13.1
shapely==2.0.6
xgboost==2.1.3

Inspected the pod with `kubectl logs <pod_name>':
Defaulted container "kserve-container" out of: kserve-container, storage-initializer (init)
WARNING: No ICDs were found. Either,
- Install a conda package providing a OpenCL implementation (pocl, oclgrind, intel-compute-runtime, beignet) or
- Make your system-wide implementation visible by installing ocl-icd-system conda package.
Environment tarball not found at '/mnt/models/environment.tar.gz'
Environment not found at './envs/environment'
2024-12-12 20:41:31,031 [mlserver.parallel] DEBUG - Starting response processing loop...
2024-12-12 20:41:31,032 [mlserver.rest] INFO - HTTP server running on http://<IP_ADDRESS>:8080
INFO:     Started server process [1]
INFO:     Waiting for application startup.
2024-12-12 20:41:31,125 [mlserver.metrics] INFO - Metrics server running on http://<IP_ADDRESS>:8082
2024-12-12 20:41:31,125 [mlserver.metrics] INFO - Prometheus scraping endpoint can be accessed on http://<IP_ADDRESS>:8082/metrics
INFO:     Started server process [1]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
2024-12-12 20:41:32,940 [mlserver.grpc] INFO - gRPC server running on http://<IP_ADDRESS>:9000
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8080 (Press CTRL+C to quit)
INFO:     Uvicorn running on http://<IP_ADDRESS>:8082 (Press CTRL+C to quit)
2024/12/12 20:41:34 WARNING mlflow.utils.requirements_utils: Detected one or more mismatches between the model's dependencies and the current Python environment:
 - mlflow (current: 2.10.2, required: mlflow==2.18.0)
 - cloudpickle (current: 3.0.0, required: cloudpickle==3.1.0)
 - optuna (current: uninstalled, required: optuna==4.1.0)
 - pandas (current: 2.2.1, required: pandas==2.2.3)
 - psutil (current: uninstalled, required: psutil==5.9.8)
 - scikit-learn (current: 1.4.1.post1, required: scikit-learn==1.5.2)
 - scipy (current: 1.12.0, required: scipy==1.13.1)
 - shapely (current: uninstalled, required: shapely==2.0.6)
 - xgboost (current: 2.0.3, required: xgboost==2.1.3)
To fix the mismatches, call `mlflow.pyfunc.get_model_dependencies(model_uri)` to fetch the model's environment and install dependencies using the resulting environment file.
2024/12/12 20:41:34 WARNING mlflow.pyfunc: The version of Python that the model was saved in, `Python 3.9.18`, differs from the version of Python that is currently running, `Python 3.10.12`, and may be incompatible
2024-12-12 20:41:35,250 [mlserver][mexico-power-outage] INFO - Couldn't load model 'mexico-power-outage'. Model will be removed from registry.
2024-12-12 20:41:35,250 [mlserver][mexico-power-outage] INFO - Unloaded model 'mexico-power-outage' successfully.
2024-12-12 20:41:35,250 [mlserver.parallel] ERROR - An error occurred processing a model update of type 'Load'.
Traceback (most recent call last):
  ...
ModuleNotFoundError: No module named 'shapely'
2024-12-12 20:41:35,253 [mlserver][mexico-power-outage] INFO - Couldn't load model 'mexico-power-outage'. Model will be removed from registry.
2024-12-12 20:41:35,257 [mlserver.parallel] ERROR - An error occurred processing a model update of type 'Unload'.
Traceback (most recent call last):
  ...
mlserver.errors.ModelNotFound: Model mexico-power-outage not found
2024-12-12 20:41:35,258 [mlserver][mexico-power-outage] INFO - Unloaded model 'mexico-power-outage' successfully.
2024-12-12 20:41:35,259 [mlserver] ERROR - Some of the models failed to load during startup!
Traceback (most recent call last):
  ...
mlserver.parallel.errors.WorkerError: builtins.ModuleNotFoundError: No module named 'shapely'

What did you expect to happen:
Successful deplyoment of my inferenceService
What's the InferenceService yaml:
apiVersion: "serving.kserve.io/v1beta1"
kind: "InferenceService"
metadata:
  name: "mexico-power-outage"
  namespace: "default"
spec:
  predictor:
    serviceAccountName: sa
    model:
      storageUri: "https://<my_blob_account>.blob.core.windows.net/mlflow/782023656870099651/9bd19283eb93482c84ed023cca1da6fe/artifacts/model"
      modelFormat:
        name: "mlflow"
      protocolVersion: v2
    minReplicas: 1
    maxReplicas: 3

Anything else you would like to add:
I don't understand why the environment is not getting loaded for the model. Am I missing something when deploying this inference service? It seems like Kserve should load the dependencies for the model, so if it is not doing it by default how do I do that?
Environment:

Istio Version:
Knative Version:
KServe Version:
Kubeflow version:
Cloud Environment:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube/Kind version:
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

Have you tried packaging the conda.yaml environment into the model storage directory like tarball and then upload it to the model storage directory, and i am also saw some dependency mismatch like shapely is missing entirely, and other versions are incompatible and the model was saved with Python 3.9.18, but the runtime is using Python 3.10.12, which might introduce compatibility issues.

You will have to pack the conda-environment, which you used to log the model in mlflow with conda-pack (https://conda.github.io/conda-pack/) and upload it as additional artifact to your mlflow run.

You will have to pack the conda-environment, which you used to log the model in mlflow with conda-pack (https://conda.github.io/conda-pack/) and upload it as additional artifact to your mlflow run.

I made a procedure because there were a few things we needed to be careful about.
https://github.com/kserve/kserve/discussions/4100#discussioncomment-11539275

For anyone wondering about this same thing essentially my options were:

Build the conda-pack environment.tar.gz locally and upload it with the model
use the docker image approach and build my own docker image with my environment and reference that

We went with the latter approach as the former has some hiccups with building on different architectures than that of the kserver docker image.
Just curious about this, seldon core has this functionality, and I am wondering why it isn't in Kserve? I know that this can cause very long pod startup times, but I feel like an opt in feature that defaults to opt out would be nice in case the user wants this.


New video on cancancan for screencast link update

https://www.youtube.com/watch?v=cTYu-OjUgDw
Please update screencast from cancan to cancancan.
Thx

Thanks!


[ci-auto] GenAIExample ChatQnA compose.yaml got changed.

GenAIExample ChatQnA compose.yaml got changed
Below files are changed in this commit

ChatQnA/docker_compose/intel/cpu/xeon/compose.yaml
ChatQnA/docker_compose/intel/hpu/gaudi/compose.yaml
ChatQnA/docker_compose/nvidia/gpu/compose.yaml

Please verify if the helm charts and manifests need to be changed accordingly.

This issue was created automatically by CI.

no need to change our implementation


chromium: 86.0.4240.198 -> 87.0.4280.66

https://chromereleases.googleblog.com/2020/11/stable-channel-update-for-desktop_17.html
This update includes 33 security fixes.
CVEs:
CVE-2020-16018 CVE-2020-16019 CVE-2020-16020 CVE-2020-16021
CVE-2020-16022 CVE-2020-16015 CVE-2020-16014 CVE-2020-16023
CVE-2020-16024 CVE-2020-16025 CVE-2020-16026 CVE-2020-16027
CVE-2020-16028 CVE-2020-16029 CVE-2020-16030 CVE-2019-8075
CVE-2020-16031 CVE-2020-16032 CVE-2020-16033 CVE-2020-16034
CVE-2020-16035 CVE-2020-16012 CVE-2020-16036
Note: We'll finally build with use_ozone=true on Hydra now :) \o/

Motivation for this change
cc @danielfullmer (x86_64), @thefloweringash (aarch64), and @primeos (Chrome) for testing
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Built and briefly tested on x86_64-linux.

Google Chrome is also fine and aarch64 seems still failing due to failing dependencies. So this should be good to go.

It took a lot longer than normal, but it built ok on aarch64-linux.


Invalid host name

Hello!
I am attempting to deploy this on unraid using docker compose, I do have Earthly installed and running, but when I try to run the docker-compose, I get the following error in the docker-compose log:
headwaymaps-valhalla_init-1           | http://: Invalid host name.
headwaymaps-valhalla_init-1           | xz: (stdin): File format not recognized
I am wondering if there is a way to set the host name in the Env file? I have already input the base url, as the directions state, and I do know my .env file is being read because it picks up the name of my custom OSM file. Any help is appreciated!
I set the env file for the URL as follows:
HEADWAY_PUBLIC_URL=http://<IP_ADDRESS>:8080

I get the following error in the docker-compose log:

headwaymaps-valhalla_init-1 | http://: Invalid host name.
headwaymaps-valhalla_init-1 | xz: (stdin): File format not recognized

The container named headwaymaps-valhalla_init corresponds to services/valhalla/init.sh
Looking at that file, it seems likely that you're trying to download $VALHALLA_ARTIFACT_URL. But that shouldn't happen if you've built all the artifacts as the instructions describe.

I do have Earthly installed and running, but when I try to run the docker-compose,

Did you run the build? Something like:
earthly -P +build --area=Amsterdam

Did it succeed?

Yes I have tried running that, this is what I always get:
+pelias-import failed | fff0edd27e5e: Pull complete
+pelias-import failed | e13dc882f523: Pull complete
+pelias-import failed | 91d2fd5e3a1d: Pull complete
+pelias-import failed | 5c9407b5b09d: Pull complete
+pelias-import failed | Digest: sha256:2dd1b31bc70d91aed2898c1e5fbda4f7de6ef43a9b6d2cad53b1e4c82baa409f
+pelias-import failed | Status: Downloaded newer image for <IP_ADDRESS>:8371/sess-cnvdjj1zikynj4z10b860nnuh:img-0
+pelias-import failed | <IP_ADDRESS>:8371/sess-cnvdjj1zikynj4z10b860nnuh:img-0
+pelias-import failed | Untagged: <IP_ADDRESS>:8371/sess-cnvdjj1zikynj4z10b860nnuh:img-0
+pelias-import failed | Untagged: <IP_ADDRESS>:8371/sess-cnvdjj1zikynj4z10b860nnuh@sha256:2dd1b31bc70d91aed2898c1e5fbda4f7de6ef43a9b6d2cad53b1e4c82baa409f
+pelias-import failed | ...done
+pelias-import failed | Creating network "config_default" with the default driver
+pelias-import failed | Creating volume "config_pelias_data" with default driver
+pelias-import failed | Pulling pelias_elasticsearch (pelias/elasticsearch:7.16.1)...
+pelias-import failed | 7.16.1: Pulling from pelias/elasticsearch
+pelias-import failed | Digest: sha256:570251377c1e17f2b87a0ab27f0e0c7494dd2d4a0815da1b065b9818ea060d2d
+pelias-import failed | Status: Downloaded newer image for pelias/elasticsearch:7.16.1
+pelias-import failed | Creating pelias_elasticsearch ...
+pelias-import failed | Creating pelias_elasticsearch ... done
+pelias-import failed | Creating pelias_schema        ...
+pelias-import failed | Creating pelias_schema        ... done
+pelias-import failed | Creating config_pelias_schema_run ...
+pelias-import failed | Creating config_pelias_schema_run ... done
+pelias-import failed | waiting for elasticsearch service to come up
+pelias-import failed | ..............................
  +pelias-import *failed* | Elasticsearch did not come up, check configuration
  +pelias-import *failed* | 1
  +pelias-import *failed* | Stopping pelias_elasticsearch ...
  +pelias-import *failed* | Stopping pelias_elasticsearch ... done
  +pelias-import *failed* | Removing config_pelias_schema_run_67c3d2d2a8eb ...
  +pelias-import *failed* | Removing pelias_schema                         ...
  +pelias-import *failed* | Removing pelias_elasticsearch                  ...
  +pelias-import *failed* | Removing pelias_elasticsearch                  ... done
  +pelias-import *failed* | Removing pelias_schema                         ... done
  +pelias-import *failed* | Removing config_pelias_schema_run_67c3d2d2a8eb ... done
  +pelias-import *failed* | Removing network config_default
  +pelias-import *failed* | ERROR Earthfile line 262:8
  +pelias-import *failed* |       The command
  +pelias-import *failed* |           WITH DOCKER RUN --privileged docker-compose run -T 'pelias_schema' bash -c "/tools/wait.sh && ./bin/create_index"
  +pelias-import *failed* |       did not complete successfully. Exit code 1

Share your logs with an Earthly account (experimental)! Register for one at https://ci.earthly.dev.
Error: build target: build main: failed to solve: Earthfile line 26:4 apply BUILD +save: earthfile2llb for +save: Earthfile line 41:4 apply BUILD +save-elasticsearch: earthfile2llb for +save-elasticsearch: Earthfile line 96:4 copy artifact: apply build +pelias-import: earthfile2llb for +pelias-import: Earthfile line 262:8 with docker run: unlazy force execution: process "/bin/sh -c EARTHLY_DOCKERD_DATA_ROOT="/var/earthly/dind/1e6fcfb633d02e5dda9323eb729bc6993d1f6e13780d30c6e4f451b8c279989f" EARTHLY_DOCKER_LOAD_FILES="" EARTHLY_IMAGES_WITH_DIGESTS="" EARTHLY_START_COMPOSE="true" EARTHLY_COMPOSE_FILES="compose.yaml" EARTHLY_COMPOSE_SERVICES="pelias_schema" DATA_DIR=/data DIND_COMMIT=42b1175eda071c0e9121e1d64345928384a93df1 DOCKER_TLS_CERTDIR=/certs DOCKER_VERSION=20.10.14 EARTHLY_LOCALLY=false PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin area=NY_PA countries=US /var/earthly/dockerd-wrapper.sh execute /usr/bin/earth_debugger /bin/sh -c 'docker-compose run -T '"'"'pelias_schema'"'"' bash -c "/tools/wait.sh && ./bin/create_index"'" did not complete successfully: exit code: 1
in              +pelias-import --HEADWAY_AREA=NY_PA --HEADWAY_BBOX=41.31 -77.82 42.96 -74.34 --HEADWAY_ENABLE_TRANSIT_ROUTING=0 --HEADWAY_HTTP_PORT=8080 --HEADWAY_PUBLIC_URL=http://localhost:8080 --HOSTNAME=localhost --area=NY_PA --countries=US --transit_feeds=
in              +save-elasticsearch --HEADWAY_AREA=NY_PA --HEADWAY_BBOX=41.31 -77.82 42.96 -74.34 --HEADWAY_ENABLE_TRANSIT_ROUTING=0 --HEADWAY_HTTP_PORT=8080 --HEADWAY_PUBLIC_URL=http://localhost:8080 --HOSTNAME=localhost --area=NY_PA --countries=US --transit_feeds=
in              +save --HEADWAY_AREA=NY_PA --HEADWAY_BBOX=41.31 -77.82 42.96 -74.34 --HEADWAY_ENABLE_TRANSIT_ROUTING=0 --HEADWAY_HTTP_PORT=8080 --HEADWAY_PUBLIC_URL=http://localhost:8080 --HOSTNAME=localhost --area=NY_PA --countries=US --transit_feeds=
in              +build --HEADWAY_AREA=NY_PA --HEADWAY_BBOX=41.31 -77.82 42.96 -74.34 --HEADWAY_ENABLE_TRANSIT_ROUTING=0 --HEADWAY_HTTP_PORT=8080 --HEADWAY_PUBLIC_URL=http://localhost:8080 --HOSTNAME=localhost --area=NY_PA --countries=US

Also, I have a custom OSM file which is called NY_PA.osm.pbf which I got from BBBIKE, which is in the top level directory as specified

NY+PA is a big extract, so I'd suspect memory pressure as a potential reason elasticsearch might not come up during import. Try setting your buildkit parallelism to 1 according to FULL_PLANET.md. Just a hunch :)

Looks like I may not have documented buildkit parallelism, but you can follow this documentation to set up an earthly config file. Glancing through the docs it looks like running earthly config global.buildkit_max_parallelism 1 might also work.

Okay, I'll try tweaking the settings as suggested, know that I'm running this on a machine with a 5950x, and 64GB of RAM, And I have 4 TB of space free, building on a 1 TB SSD, so resources shouldn't be an issue, but timeouts due to sheer size of data is definitely plausible... If it fails again I'll try with a smaller build, probably a prebuild and see if it works or gets farther. Thanks for the suggestions

It's pretty easy to OOM while working with large extracts or planet builds if you allow earthly to run many tasks in parallel. I've done it even with 128gb of RAM. I don't think it's a timeout issue but that's certainly possible too. Super curious to see if the buildkit parallelism tweak fixes it. Failing that, another good data point would be building a small area like Amsterdam.

I tried the parallelism tweak, and it did start elasticsearch, seemed to get further, but then failed out later on with some generic compilation error, seems like it may have to do with permissions when I look more into the issue on. I watched my RAM usage and it only went up a percent or 2 while building, and CPU barely even moved as well... Where does it process the data? I was assuming it was doing things in the current working directory, but I'm not seeing it create any files or anything in the directory I call the build command in... Seems like it's not actually processing any data for some reason. I also tried using supported build NewYork and same thing, says elasticsearch couldn't start.

I also tried your cache_size_mb to 10000000 setting, and still no change.

It's impossible to diagnose this without build logs.


Non-native window decorations on Gnome on Wayland

I'm on Debian testing, using Gnome on Wayland.
If I set my window decorations to full, I'd expect the decorations to match the style of other apps on Gnome. However, this is not the case - Alacritty decorations look very out of place.
Obviously this isn't an issue that would make Alacritty unusable or something like that, but it would still be nice to have the window match the DE.

@marekdedic Would you mind to share screenshot on how our decorations look like on you system?

I've added gedit on the right - that's what I would imagine "native" decorations to look like...

What you describe as decorations to match the style of other apps on Gnome likely means just GTK decorations. These all look the same because they are drawn with the same toolkit.
If server side decorations are rendered, we will not draw our own decorations. However, GNOME thought it's a good idea not to support server side decorations, which means you end up with Applications that all have decorations that look completely differently.
My recommendation is to not use GNOME. There might be a way to get some kind of server side decorations working with GNOME, but that's a GNOME issue really. Other Wayland compositors should just work, if they do not we can discuss that in a separate issue. Sorry for the disappointment.

This is really a GNOME thing here, client-side decorations are now supported for all apps, the GNOME look is just enforced by the GTK theme, but as mentioned in this post, server-side decorations can also be negotiated and should use the default GTK theme by default.
The same problem was also mentioned for KDE Qt apps running on GNOME shell, the GTK look can be obtained with several third-party themes though.


train u2++ conformer :  AssertionError  assert offset + size <= self.max_len

When training u2++ conformer on custom dataset, I'm encountering this error, it stops after training a while. The train_conformer.yaml worked fine.
/wenet/wenet/transformer/embedding.py", line 102, in position_encoding
      assert offset + size <= self.max_len
  AssertionError

To Reproduce
Steps to reproduce the behavior:

create custom dataset following librispeech example (uising wav files, instead of flac)
change config file to train_u2++_conformer.yaml in run.sh, the only thing I changed in yaml file is batch size
run stage 4
See error

Expected behavior
Finish training normally

训练数据太长了 ， 超过emb的最长长度
Turi Abu @.***> 于2024年9月10日周二 11:16写道：

When training u2++ conformer on custom dataset, I'm encountering this
error, it stops after training a while. The train_conformer.yaml worked
fine.
/wenet/wenet/transformer/embedding.py", line 102, in position_encoding
assert offset + size <= self.max_len
AssertionError
To Reproduce
Steps to reproduce the behavior:

create custom dataset following librispeech example (uising wav
files, instead of flac)
change config file to train_u2++_conformer.yaml in run.sh, the only
thing I changed in yaml file is batch size
run stage 4
See error

Expected behavior
Finish training normally
—
Reply to this email directly, view it on GitHub
https://github.com/wenet-e2e/wenet/issues/2629, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFN3Q6OSQ3DMYCV3ASJOXLZVZQBFAVCNFSM6AAAAABN5ZIWPKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYTKMZQGUYTCOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

需要把每一条训练数据限制在30s以内， 或者该打max len
Turi Abu @.***> 于2024年9月10日周二 13:33写道：

What is the suggested solution? @Mddct https://github.com/Mddct
—
Reply to this email directly, view it on GitHub
https://github.com/wenet-e2e/wenet/issues/2629#issuecomment-2339658575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFN3Q254WBSGNWS7OU3H23ZV2ADXAVCNFSM6AAAAABN5ZIWPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZZGY2TQNJXGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Improve the docs that explain how to connect to Kubeflow UIs.

Provide a new page that explains how to connect to Ambassador so that
all the UIs will be accessible.

Change the JupyterHub documentation to reference connecting to Ambassador.

Fix #207

This change is

/assign @sarahmaddox

/approved

/approve


BSO Sepulchre Trips Not Returning

🐛 Bug Report
I send a bso sepulchre trip and while it is going I can check it with =m, but it doesn't return. My minion isn't stuck, it is free after the trip "ends", it just doesn't show up. I believe I receive xp when the trip ends, but will have to test to be sure.

Update: I am not getting xp from it, it's borked.

Found the source of the error. Pages were originally coded for OSB which has 5 floors maximum. with BSO having 7 it was trying to roll none existent pages.

Fixed in next update


[CAMEL-14267] Fix a nullPointerException in convertIfRequired

Fix a nullPointerException in convertIfRequired when boddy is null

Thanks @tfabien would you be able to do a new PR against camel-2.x branch so we can fix it on 2.x also ?

Done PR #3396


PATCH/POST Dimensions API

Expected Behaviour
Ability to update dimensional settings within a dataview.
Actual Behaviour
Only option is through UI or non-public API.
Reproduce Scenario (including but not limited to)
API reference documentation only has GET requests for the Dimensions API.
Steps to Reproduce
link
Platform and Version
Sample Code that illustrates the problem
Logs taken while reproducing problem

+1 on this.  It sure would be nice to have an endpoint for PUT and POST.

+1, and also on metrics

Thank you for submitting this issue. I will pass this along to the developers and once we have a public endpoint for this, I will make sure it gets documented.

I am going to re-open this and send it to PM as is. I will create a ticket for this for PMs. I know this has been discussed much, but not sure if a github issue has been cited like this. So I will leave it open and cite it for PMs to make sure they know about the request. This is the documentation repo and I only work in documentation so I cannot provide any development or product assurances. But I will do my best to make sure your expressions here get passed along.

I have put in a ticket for this API request. I will leave this open so PMs can refer to it. Also, I found out that API feature requests can also be submitted here: https://experienceleaguecommunities.adobe.com/t5/adobe-analytics-ideas/idb-p/adobe-analytics-ideas.

This feature is being regularly requested by product team but since it is not strictly a documentation issue, I am closing this.


Added new features with some improvments

Added the following 👍

Integration with BMP085 sensors
Ability to use as a WX station
Profile for WX
Profile for Static station location
Changes APRS packet variables in WX to make it more readable
Added variables ready to be used with more advanced sensors (rain, wind etc) just uncomment.

The whole project would benefit from some refactoring hopefully I will get to it at some point

Not sure how to implement sanitation checks since the user has to manually edit the code anyway.  APRS will just give out an error anyway if LAT/LONG is not in the correct format.

I would like to add the ability to use more sensors, like the BMP280 or something else, but I don't have the hardware to test it with. I might get my hands on some at some point and would try to add it in the form of a switch statement or something.

Not sure how to implement sanitation checks since the user has to manually edit the code anyway. APRS will just give out an error anyway if LAT/LONG is not in the correct format.

yeah, will add an example Position in APRS lat/long format

I would like to add the ability to use more sensors, like the BMP280 or something else, but I don't have the hardware to test it with. I might get my hands on some at some point and would try to add it in the form of a switch statement or something.

great Idea.

Please checkout the new wx Branch
I merged your Code and did some minor Changes.
As i have no BME Sensor wired at the Moment, i cannot Test the Temperature readout.
Can you provide a wiring diagram for Documentation?
There will be a bug with fixed Beaconing.
At the Moment it needs GPS Rx, even if we are in "Fixed" Mode with user defined Coordinates.

I was searching for a fritzing model for the board but could not find one. I'll see if I can make one.
I'm powering the BMP180 sensor off the 3.3v pin
VCC -> VDD
GND -> GND
SCL -> SCL (located under the display ribbon)
SDA -> SDA (located under the display ribbon)
I will add support for the BMP280 later on when I get my hands on it.

I just got a BMP Sensor in my Hands.
The Code seems to work. Will merge it, Thank you! @Slayingripper .
73 DJ1AN


adding command_executor to support multiple commands from a single shell

Deprecates WinRM::WinRMWebService methods cmd, run_cmd, powershell, and run_powershell_script in favor of the run_cmd and run_powershell_script methods of the WinRM::CommandExecutor class. The CommandExecutor allows multiple commands to be run from the same WinRM shell providing a significant performance improvement whwn issuing multiple calls.
This PR is a prerequisite for https://github.com/WinRb/winrm-fs/pull/28 which depend on CommandExecutor, which is being added here since it provides value beyond just transferring files.

Note that I have removed the ShellCloser class used as a finalizer for CommandExecutor. While users of CommandExecutor are encouraged to call close explicitly, I think we can leverage the finalizer in a transparent manner upon GC.
I'd be curious to know if @fnichol has any words of caution regarding this strategy. I have tested GC's on the CommandExecutor as it is here and have not had issues with it losing sessions.

will investigate the travis failures later tonight.

I just made a few additional changes here:

Moved the retryable method from the fs-transport winrm-fs branch into the CommandExecutor. Made more sense for that to live here and its a nice feature to have for openning shells.
Expose the WinRMWebService logger via a attr_reader and set its level to :warn and add the stdout appender on initialization. WinRMWebService consumers can access its logger and change level, appenders, etc. This will be helpful when it is plugged into Test-Kitchen and make it more natural to integrte the winrm logging into the test-kitchen log streams.
Added a convenience method to WinRMWebService: create_executor. This can be used 2 ways:

winrm.create_executor do |executor|
  executor.run_cmd('ipconfig')
end

This ensures the executor is closed when done.
executor = winrm.create_executor
executor.run_cmd('ipconfig')
executor.close

I just made a few additional changes here:

Moved the retryable method from the fs-transport winrm-fs branch into the CommandExecutor. Made more sense for that to live here and its a nice feature to have for openning shells.
Expose the WinRMWebService logger via a attr_reader and set its level to :warn and add the stdout appender on initialization. WinRMWebService consumers can access its logger and change level, appenders, etc. This will be helpful when it is plugged into Test-Kitchen and make it more natural to integrte the winrm logging into the test-kitchen log streams.
Added a convenience method to WinRMWebService: create_executor. This can be used 2 ways:

winrm.create_executor do |executor|
  executor.run_cmd('ipconfig')
end

This ensures the executor is closed when done.
executor = winrm.create_executor
executor.run_cmd('ipconfig')
executor.close

I think this is ready to review. @sneal let me know if you think there is more to change or if I'm off anywhere.

I think this is ready to review. @sneal let me know if you think there is more to change or if I'm off anywhere.

@mwrock Awesome stuff here, I love seeing things getting pushed upstream for reusability. I'll take a good look later next week when I have my laptop.

@mwrock Awesome stuff here, I love seeing things getting pushed upstream for reusability. I'll take a good look later next week when I have my laptop.

:shipit:

rebase/squashed and removing 1.5 versioning artifacts. Will merge that in a separate PR


[kokkos] serial backend performance fix

The Kokkos serial backend is slower than the plain serial code  - see #297 for more details.
One cause is addressed here, where the Kokkos backend loops over the entire array (size 1024) rather than the number of clusters (about 10).
This loop also sets ok and newclusId to zero.  Those get set later and these assignments can be removed.

On Cori I see ~5 % improvement on kokkos --serial when running one process on otherwise empty node

1-thread case is now witin 4 % of the serial program.
On a fully loaded socket the improvement is smaller but still clearly visible


Import authentication script from a url

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


[Bug]: gitlab private registry only 1 node of 3 get ErrImagePull

Description
I'm trying to deploy 3 replicas of an image from a private gitlab registry. I setup a secret like this:
---
apiVersion: v1
data:
  .dockerconfigjson: <mybase64secret>
kind: Secret
metadata:
  creationTimestamp: "2023-05-23T12:34:58Z"
  name: gitlab-pull-secret
  namespace: test-namespace
  resourceVersion: "64982"
  uid: 095f3454-5ede-4f10-9e72-5bdb92a72e8b
type: kubernetes.io/dockerconfigjson

it works on 2 of 3 nodes, and i have one that get ErrImagePull. How is it possible?
This is a clean cluster created now for testing.
Kube.tf file
my kube.tf is really simple like the example, 3 masters and 3 workers.

Screenshots

Describe the pod in error:

Platform
Linux

update: i've tryed deleting and recreating the node, i've tryed recreating 4 times the whole cluster, with ARM and AMD... always the same problem, 2 pods working, 1 pod in error.
The problem is always on one node, if i try to create a 9 replica, all the pods on one workes doesn't pull the image.

@Sampozzo Even if you recreate many times Hetzner still gives you the same IP for one of the nodes. You need to contact Hetzner support and let them know that the IP in question is blacklisted by the Google Container Registry (probably what gitlab users under the hood). So that they change that IP for you.


Improve application staging logs

Right now we print pod names and other information but it's not obvious to the user where the logs come from and what they refer to (e.g. "pushing code", "staging" ?)
We should limit the output to only the useful stuff.

First some investigation results to get my head into the log watcher/tailer used here.

Staging logs generated by the tailer are currently of the form {{color .PodColor .PodName}} {{color .ContainerColor .ContainerName}} {{.Message}}. This is a template used to format a log entry structure. From the above each log line contains 3 pieces of information: origin pod, container in pod, message itself, with the first two additionally colorized.

The only namespace monitored when pushing is eirini-workloads (1).

The regular expression ".%s." is used to set up a PodQuery, i.e. to limit the tailing to the pods related to the application in question, "%s" is the placeholder for the name. This should pick up the staging pods, and the app pod itself, and only them.
Note: This is buggy IMHO, because if you push, for example, applications foo and foobar at the same time (in different terminals), then the tailer for foo looks to be picking up the pods and logs for foobar also.
I suspect that we need ".-%s-.", i.e. note the additional dashes separating the app name in the pod name from the surrounds.
@viovanov Can you confirm/falsify my thoughts wrt the possible bug ?

The backend system for logging is generic enough to allow watching multiple namespaces. Push simply does not configure that.

First some investigation results to get my head into the log watcher/tailer used here.

Staging logs generated by the tailer are currently of the form {{color .PodColor .PodName}} {{color .ContainerColor .ContainerName}} {{.Message}}. This is a template used to format a log entry structure. From the above each log line contains 3 pieces of information: origin pod, container in pod, message itself, with the first two additionally colorized.

The only namespace monitored when pushing is eirini-workloads (1).

The regular expression ".%s." is used to set up a PodQuery, i.e. to limit the tailing to the pods related to the application in question, "%s" is the placeholder for the name. This should pick up the staging pods, and the app pod itself, and only them.
Note: This is buggy IMHO, because if you push, for example, applications foo and foobar at the same time (in different terminals), then the tailer for foo looks to be picking up the pods and logs for foobar also.
I suspect that we need ".-%s-.", i.e. note the additional dashes separating the app name in the pod name from the surrounds.
@viovanov Can you confirm/falsify my thoughts wrt the possible bug ?

The backend system for logging is generic enough to allow watching multiple namespaces. Push simply does not configure that.

Confirmed the bug wrt picking up foo and foobar, although I trialed with sample and samplefoo. The push for the shorter name prints the logs for both apps.

Confirmed the bug wrt picking up foo and foobar, although I trialed with sample and samplefoo. The push for the shorter name prints the logs for both apps.


Added Role Controller and restrictions, office, role,

Added Role Controller and restrictions, office, role,

i have commented the service code with dtos. not deleted any in office service and map service.


Pass arguments as BlobReference

Namescopes are now more correctly applied, and optimizers can
start to be used in data_parallel_model

Looks like Aapo is taking care of this internally, just reviewed his diff.

Yes, I saw that mentioned - I'll close this


crash android.os.NetworkOnMainThreadException

i got this issues, can u help?

I'm not experienced with Android, so I don't know if I'll be a ton of help but I can try!
On Android apparently you are not allowed to perform network operations on the main application thread. Perhaps you should use a separate thread or an AsyncTask to make the call.
Check this post on StackOverflow which has many examples:
https://stackoverflow.com/questions/6343166/how-can-i-fix-android-os-networkonmainthreadexception


[dox] Leading rows in Container Primitives table should span all columns.

Depends on: https://github.com/D-Programming-Language/dlang.org/pull/1071

LGTM

LGTM

Auto-merge toggled on


Use self hosted GitHub runner

Checked with @tpayet, we have created a self hosted github runner to save time when pushing the docker images.

bors merge

bors merge


[5.3] Notification::send() has strange behaviors with Eloquent model

Laravel Version: v5.3.*
PHP Version: 7.0.11
Database Driver & Version:

Description:
When I use Notification::send($users, new UserPaidInvoice($invoice, $thing) and if my UserPaidInvoice class is implementing ShouldQueue, Laravel will make at least two queries per agument + if you use model's attribute inside the Notification, it will generate another request as shown on this screen.

Please note that this is true only if your class is implementing ShouldQueue
Steps To Reproduce:
This is my class
    <?php

    namespace App\Notifications;

    use Auth;
    use Illuminate\Bus\Queueable;
    use Illuminate\Contracts\Queue\ShouldQueue;
    use Illuminate\Notifications\Messages\MailMessage;
    use Illuminate\Notifications\Notification;

    class UserPaidInvoice extends Notification implements ShouldQueue
    {
        use Queueable;

        protected $invoice;
        protected $partner;

        /**
         * Create a new notification instance.
         *
         * @return void
         */
        public function __construct($invoice, $partner)
        {
            $this->invoice= $invoice;
            $this->partner= $partner;
        }

        /**
         * Get the notification's delivery channels.
         *
         * @param  mixed  $notifiable
         * @return array
         */
        public function via($notifiable)
        {
            return ['mail', 'database'];
        }

        /**
         * Get the mail representation of the notification.
         *
         * @param  mixed  $notifiable
         * @return \Illuminate\Notifications\Messages\MailMessage
         */
        public function toMail($notifiable)
        {
            return (new MailMessage)
                        ->line('...')
                        ->action('...)
                        ->line('...');
        }

        /**
         * Get the array representation of the notification.
         *
         * @param  mixed  $notifiable
         * @return array
         */
        public function toArray($notifiable)
        {
            return
                [
                    'partner' => $this->partner->get(['id', 'first_name', 'last_name']),
                    'invoice' => $this->invoice->get(['id', 'price', 'date']),
                    'by' => Auth::user()->get(['id', 'first_name', 'last_name']),
                ]
            ;
        }
    }

$this->partner->get(['id', 'first_name', 'last_name']) is basically a new query, I think you need array_only() not get().

Why $this->partner isn't a Collection?
And what about the first issue? Look at the screen, there is 6 queries (firsts ones) that aren't needed because the model passed by argument have already done the query no?


New Crowdin updates

Codecov Report

Merging #399 (6b1432d) into main (83271e2) will decrease coverage by 8.59%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #399      +/-   ##
============================================
- Coverage     48.66%   40.07%   -8.60%
+ Complexity     1513     1275     -238
============================================
  Files           207      205       -2
  Lines          9764     9514     -250
  Branches       2068     2015      -53
============================================
- Hits           4752     3813     -939
- Misses         3993     4819     +826
+ Partials       1019      882     -137

Impacted Files
Coverage Δ

...dautoidrive/phoneui/fragments/MusicPageFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...dautoidrive/phoneui/viewmodels/SupportPageModel.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...autoidrive/phoneui/fragments/CarSummaryFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...utoidrive/phoneui/fragments/MapSettingsFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...utoidrive/phoneui/fragments/SupportPageFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...toidrive/phoneui/fragments/OverviewPageFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...toidrive/phoneui/fragments/SettingsPageFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...oidrive/phoneui/fragments/AssistantPageFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...idrive/phoneui/fragments/ConnectionPageFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...idrive/phoneui/fragments/DependencyInfoFragment.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 71 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 83271e2...6b1432d. Read the comment docs.


Invtweaks mod bug spigot + no plugins

https://www.youtube.com/watch?v=5tAbvH7NwBs

Spigot is not supported, bukkit does strange things to item handling.
Try the most recent dev version if you haven't though, there are a few consistency fixes there.


Corrected javadoc of the Cookie persistent() method

It was claiming that returns true if the cookie expires and the end of the session and really is the opposite behavior.

Whoa that’s embarrassing. Thanks!


JVM crash when taking thread dump

A fatal error has been detected by the Java Runtime Environment:
SIGSEGV (0xb) at pc=0x00007f20570e8559, pid=16013, tid=16026
JRE version: OpenJDK Runtime Environment (11.0.7+10) (build 11.0.7+10)
Java VM: OpenJDK 64-Bit Server VM (11.0.7+10, mixed mode, tiered, compressed oops, g1 gc, linux-amd64)
Problematic frame:
C  [libpthread.so.0+0x12559]  pthread_getcpuclockid+0x9
hs_err_pid16013.log

What exactly did you do? There's at least half a dozen ways to produce a thread dump.

I have a hanging jvm application. When I take a threadump in VisualVM or with command line tool jstack the application is crashing. Is there anything what I can provide?

Can you provide a test case / reproduced?  Also, can you try an adoptopenjdk.net/upstream.html build?

Reproduce
I only can reproduce it with our product "Axon.ivy Designer", which is based on Eclipse. https://developer.axonivy.com/download

Need to start Axon.ivy Designer
Opening a Web Editor
Opening the Java Editor and working with "editor proposal" feature.
After a while the JVM is hanging, which seems to be a deadlock with the Eclipse SWT environment and our own Web Editor which is based on JavaFX. Therefore I have created a bug report for eclipse: https://bugs.eclipse.org/bugs/show_bug.cgi?id=563990
I'm not sure this could also be a bug in the native code of eclipse swt.

I think the deadlock is an issue in the Eclipse stack. Anyway, I think the JVM should never crash when taking thread dump, therefore I thought I think this is bug in the JVM itself? But if its a bug in native code of other components (eclipse swt) than the JVM could crash and this must be fixed in this component. Is this right?
Upstream Build
I can reproduce the problem with an upstream build.

Reproduce
I only can reproduce it with our product "Axon.ivy Designer", which is based on Eclipse. https://developer.axonivy.com/download

Need to start Axon.ivy Designer
Opening a Web Editor
Opening the Java Editor and working with "editor proposal" feature.
After a while the JVM is hanging, which seems to be a deadlock with the Eclipse SWT environment and our own Web Editor which is based on JavaFX. Therefore I have created a bug report for eclipse: https://bugs.eclipse.org/bugs/show_bug.cgi?id=563990
I'm not sure this could also be a bug in the native code of eclipse swt.

I think the deadlock is an issue in the Eclipse stack. Anyway, I think the JVM should never crash when taking thread dump, therefore I thought I think this is bug in the JVM itself? But if its a bug in native code of other components (eclipse swt) than the JVM could crash and this must be fixed in this component. Is this right?
Upstream Build
I can reproduce the problem with an upstream build.

Yeah, this is almost certainly an underlying Eclipse issue.  We'll close this, for now, bu the Eclipse folks can reopen it if need be.

Regarding the crash: see https://bugzilla.redhat.com/show_bug.cgi?id=1832121. The native stack looks same.
Once we restore our RH account rights (lost in transition from one company to another) we will ask RH account guys to drive investigation, especially now we have here a better reproducer.
I think this is JVM bug unless the opposite is proven :-).

I'm trying to follow steps, but I fail at point 3 below: I have no idea about how to open Web Editor (I assume I have to create a project, but again, I have no idea how to proceed with that).
Please could you update Axon specific steps, so that every one can reproduce it?
My steps

Download https://developer.axonivy.com/download. For Linux

$ cd /tmp
$ wget "https://download.axonivy.com/8.0.6/AxonIvyDesigner<IP_ADDRESS>5291209_Linux_x64.zip"
$ unzip /tmp/AxonIvyDesigner<IP_ADDRESS>5291209_Linux_x64.zip

Run the designer, select /tmp/workspace as workspace
$ ./Axon.ivy\ Designer
Open a Web Editor // FROM HERE ON I'm lost
Open the Java Editor and work with "editor proposal" feature. After a while the JVM is hanging, which seems to be a deadlock with the Eclipse SWT environment and Web Editor which is based on JavaFX.
Try to create JVM thread dump via jstack.

Hello @iloveeclipse
Thanks for investigation!
The following GIF shows how you can reproduce the problem:

I can pretty good reproduce the problem on my system (Ubuntu 20.04). After the web editor was open you may need to search different types in the Java Editor. The Java Editor shows up the proposal itself and next to the proposals there is a further window which shows the Javadoc. I assume that this window is based on webkit, because JavaDoc is pseudo-html and will be rendered in a "browser-like window". And it seems to be that while disposing this window the issue occurs.

@karianna You've reopened this one. Have you reached the conclusion that this isn't an Eclipse issue?

@alexsuter Can you re-run this with Jacoco removed?  Also, can you try our nightly 11.0.8?

I'm currently on holiday. I will give you feedback next week!

@karianna

I could not find any Jacoco libraries in the product. Did you find something?
Tried with 11.0.8 nightly. Error still exists.

I got my wires crossed on Jacoco sorry!

The crash dump indicates that the problem of the crashing JVM might be in the JVM. That the JVM hangs is likely a problem of Eclipse or Axon Ivy. We can only offer some help with the JVM crash.
@alexsuter Options that you have:

Wait for Red Hat to figure it out. Considering that the stack trace in https://bugzilla.redhat.com/show_bug.cgi?id=1832121 matches, I think this is the best option because that way the fix will end up in OpenJDK.
Report it to Oracle via https://bugreport.java.com/bugreport/. Before you do that, you have to test with a download from https://www.oracle.com/java/technologies/javase-downloads.html. If the problem happens with 15 or newer, the fix should end up in OpenJDK. If it only affects 11.0.8, the fix might only end up in Oracle's version.
Test with https://adoptopenjdk.net/upstream.html?variant=openjdk11&ga=ea. If the problem persists, report back. We have the to find someone who can report it to OpenJDK.

@aahlenst I will choose option one and try to get in touch with Read Hat.

While the JVM bug is still there and not fixed (https://bugzilla.redhat.com/show_bug.cgi?id=1897150), Eclipse specific crashes should be fixed now with 4.19, see bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=569853.

While the JVM bug is still there and not fixed (https://bugzilla.redhat.com/show_bug.cgi?id=1897150), Eclipse specific crashes should be fixed now with 4.19, see bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=569853.

This was marked as a won't fix for Java 11 upstream (it is resolved in 17).  Clients will need to use the API calls in the correct order


inconsistent edges [clusterData]

Hi,
I am using ClusterData on Reddit dataset which has "114615892" number of edges. Here is the reddit data object.
Data(edge_index=[2, 114615892], test_mask=[232965], train_mask=[232965], val_mask=[232965], x=[232965, 602], y=[232965]).
I partition it into 1500 clusters.
Then I use clusterloader with batch_size=1, and count the total number of edges in all partitions.
The total number of edges in partitions = 12601072.
Shouldn't the number of edges remain more or less equal to the number of edges in the original graph?
Any pointers to this.
Thanks.

I figured it depends on batch_size, increasing the batch_size increases the number of edges.


Fix CI

Per @orta in #293, attempting to fix the CI build error.

I moved this to Circle CI 2 in #282


Need to add a little better error checking for the product class. List > cost ? price > cost?. Maybe an override switch?

Currently the Product class has minimal error checking. It is a good idea to make sure the price is greater than the cost, and if not, require an override.

Hi @bj-rutledge  It's Stephen from  LinkedIn , as a good first issue i decided to come up with a solution before i create a branch 🛩 ,
The solution is to add an override switch for error check.

So adding a method to the Product class that allows for overriding the error check
The method then checks if the price is less than or equal to the cost. If this condition is met, it logs an error.
The method then allows bypassing the price check only if price is greater than cost

This way, override switch for error check  provides flexibility while ensuring that the check is only bypassed in valid scenarios.

That sounds like what I have planned, so I am happy to let you take a crack at it.
Though this is a fictional buisnes API,  I want to have features such as this, because, in business, sometimes you have to sell below the cost of good.
Looks forward to seeing what you do!

Hi, I have made PR , it needs a reviewer kindly review it .

PR merged. Thanks!


[MXNET-703] Show perf info for TensorRT during tests

Description
This PR makes sure perf information printed during TensorRT test runs is correctly displayed when run in CI.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@vandanavk Resolved.

@marcoabreu @lebeg @aaronmarkham for review

No by default it's disabled and this enables it.  Terrible API.
On Wed, Oct 3, 2018, 12:32 AM Anton Chernov<EMAIL_ADDRESS>wrote:

@lebeg commented on this pull request.
In ci/docker/runtime_functions.sh
https://github.com/apache/incubator-mxnet/pull/12656#discussion_r222132940
:

@@ -741,7 +741,7 @@ unittest_ubuntu_tensorrt_gpu() {
export MXNET_STORAGE_FALLBACK_LOG_VERBOSE=0
export LD_LIBRARY_PATH=/work/mxnet/lib:$LD_LIBRARY_PATH
python tests/python/tensorrt/lenet5_train.py

nosetests-3.4 $NOSE_COVERAGE_ARGUMENTS --with-xunit --xunit-file nosetests_trt_gpu.xml --verbose tests/python/tensorrt/

nosetests-3.4 $NOSE_COVERAGE_ARGUMENTS --with-xunit --xunit-file nosetests_trt_gpu.xml --verbose --nocapture tests/python/tensorrt/

Isn't --nocapture for disabling output to std?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/pull/12656#pullrequestreview-160949544,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHGTEw1lcpE_fT3wGTKxCRA4KFDe3lg2ks5ug-mWgaJpZM4W3lRA
.

@nswamy Could you have a look at this PR?

@marcoabreu @sandeep-krishnamurthy This PR seems ready to merge. Can you take a look once and merge if it looks good?


Add logging for debugging purposes season

Duck season

231 tests were run on 2607e2db. ⚠ 5 Flaked. View Full Report ↗

⚠ 5 Flakes

Test Name
Failure Summary
Logs

[chromium] should return list of users correctly.

Logs ↗

[chromium] should display error message for invalid login correctly.
The expected status code 401 was not received, instead a function was returned.
Logs ↗

[chromium] should login with credentials correctly correctly.

Logs ↗

Test$1InvalidCredentials

Logs ↗

Test$1BadRequest
failure$1message: Failedfailure$1text: === RUN   Test$1BadRequest    api$1test.go:53: Bad request sent to API--- FAIL: Test$1BadRequest (0.00s)
Logs ↗

settings


[WIP] Remove comments app

PR on top of #3618 (https://github.com/rtfd/readthedocs.org/pull/3618#issuecomment-373084224)
Closes #2879

@humitos I delete the db fields and everything looks good, should I add those migrations here or wait to https://github.com/rtfd/readthedocs.org/pull/3608?

@humitos I delete the db fields and everything looks good, should I add those migrations here or wait to #3608?

I'd say to add them so it's ready to merge. Then we can squash the migration when needed.

Also we want to remove the comments builder from https://github.com/rtfd/readthedocs-sphinx-ext (I'm not familiar with that repo)

Also we want to remove the comments builder from rtfd/readthedocs-sphinx-ext (I'm not familiar with that repo)

I suppose that you can remove the whole comments folder, https://github.com/rtfd/readthedocs-sphinx-ext/tree/master/readthedocs_ext/comments

I suppose that you can remove the whole comments folder, https://github.com/rtfd/readthedocs-sphinx-ext/tree/master/readthedocs_ext/comments

Yea, I have an initial PR doing this that could be improved (https://github.com/rtfd/readthedocs-sphinx-ext/pull/31)

@stsewd would you like to finish Eric's PR in the other repo so we can go forward whith this one also?

Linking: https://github.com/rtfd/readthedocs-sphinx-ext/pull/38

Merging this, and we can figure out if it has further issues during dev today before deploying it.

🙏 @stsewd

The only thing missing here are the migrations, I think I can add them to #3608


Developers: work out db

I've been messing around with java's jdbc.
So far, looks like we need to have an online/cloud database to which the program connects.
Right now I'm trying to see if there's a way to work on a local database (such as when using PHP's PDO to create a new database).

One thing we could feasibly do (but the system will be a bit messy) is to store everything in maps and work with those. However, the database relations will be really messy to work out.
Then when the bot goes offline for whatever reason, it saves everything from the maps into structured files (kind of like how my current rexr does it). Then when it starts up again (or at least when the first request comes in), read the file(s) into memory (the maps) and continue again.
So something like this:
[ Some state while bot offline ]  -- Go online -->  [ Maybe read file(s) into map(s) ]
[ Some state while bot online ]  -- Take request -->  [ Append/change map ]
[ Some state while bot online ]  -- Go offline -->  [ Store map(s) into structured file(s) ]


Pretrained model

Hi, is there any method through which we can bring in a pre-trained model from our PCs? If yes, then what models will the app support?

Hi, you have to export the graph as a pb and freeze the variables (freeze graph script) and you can remove unused operations for inference (strip unused nodes script). You can use any model as long as the operations are supported on mobile. I would recommend to use mobile first models like MobileNet / NasNet-Mobile though, if you have free hand to decide, for better speed.
Also you could switch from the tensorflow-android library, which is outdated to the tensorflow lite library or use Firebase ML-Kit and with both of them you could follow the more up to date guides for converting a graph to TF lite format.

Hi, Thanks for the reply. I have followed one of the tutorials. It uses keras, but I was able to convert it to a frozen .pb graph, thereafter also to a .tflite format. Now how do I use that file in the application?

The frozen graph should work after removing unused operations: https://github.com/tensorflow/tensorflow/tree/master/tensorflow/tools/graph_transforms#optimizing-for-deployment

The frozen graph should work after removing unused operations: https://github.com/tensorflow/tensorflow/tree/master/tensorflow/tools/graph_transforms#optimizing-for-deployment

I get that. I don't understand how will I use my frozen graph inside this application.

You have 2 options:

place the model in the directoy /sdcard/Pictures/facerecognition/data/TensorFlow/
place the model in the assets folder of the app

and change the settings accordingly:

Thanks. This is what I wanted to know

I had another doubt. After placing the .pb in assets folder, will it be automatically recognized by the app, or do we have to make some changes to the code as well?

It should work right away, if you provide the correct name in the settings (and other parameters such as input layer or input size). This is the code, which loads the model: https://github.com/Qualeams/Android-Face-Recognition-with-Deep-Learning-Library/blob/master/facerecognitionlibrary/src/main/java/ch/zhaw/facerecognitionlibrary/Recognition/TensorFlow.java#L74


Delete GCP001032 and its submission

The data in this GCP is not eligible for inclusion in the GWAS Catalog because it relates to a bacterial (not human) GWAS.
The GCP and its submission need to be deleted.

Moving to icebox as it will get automatically deleted

closing ticket because body of work has now been deleted automatically