Dead guides links for extensions

It looks like the guides links in the released extensions have gone out of sync with the published docs again.
I've started doing transformations to map from dead links (trying the previous release, etc). In these case, however, it seems like the issue isn't a timing one for when the docs are released and what extension is in what platform, but rather that we've changed the names or paths of docs pages. (Quarkus has this sometimes in the docs and handles it by putting redirects where the old pages were.)
For most of these, it seems like the name should have -component.html added on to the end, but there are exceptions. For example, https://camel.apache.org/camel-quarkus/latest/reference/extensions/cli-connector.html should be https://camel.apache.org/components/latest/others/cli-connector.html.
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/cli-connector.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-cli-connector",
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/datasonnet.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-datasonnet",
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/elasticsearch.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-elasticsearch",
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/mapstruct.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-mapstruct",
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/optaplanner.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-optaplanner",
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/smallrye-reactive-messaging.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-smallrye-reactive-messaging",
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/xmlsecurity.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-xmlsecurity",
    +   "https://camel.apache.org/camel-quarkus/latest/reference/extensions/tika.html => 404 (Not Found) on http://localhost:9000/org.apache.camel.quarkus/camel-quarkus-tika",
    ```

This is because the Camel website is set up to point at the 3.0.x branch where these extensions do not exist:
https://github.com/apache/camel-website/blob/main/antora-playbook-snippets/antora-playbook.yml#LL66C1-L66C16
It should probably be updated to 3.0.0-M2.x.
@ppalaga do we actually need to maintain separate branches for these 3.x milestone releases? Or should we just rebase / reset  3.0.x to the content from 3.0.0-M2.x?

Thanks for looking @jamesnetherton! I've hit the exact problem you describe in the past, and so I do a bit of logic where if the link doesn't exist, it will try a version before or after. There's some hardcoding in there in what I check, so I probably need to continue tuning it as new releases happen.
However, in this case, it seems like the actual file names have changed/are incorrect.
For example,
https://camel.apache.org/components/latest/mapstruct.html does not exist
https://camel.apache.org/components/3.0.0.x/mapstruct.html does not exist
https://camel.apache.org/components/3.20.x/mapstruct.html does not exist
https://camel.apache.org/components/3.20.x/mapstruct-component.html does exist (note it's got -component tacked on to the end)
.... or are the -component pages actually a different page?

or are the -component pages actually a different set of content?

Yes, it's a different set of content....
The setup of the Camel documentation is quite complex. Each project in the Camel ecosystem maintains its own set docs and it's all tied together on the Camel website with Antora playbooks.
This project maintains docs specific to Camel Quarkus extensions and they contain links to the corresponding Camel component docs.
Things are in a state of flux (or mess :slightly_smiling_face:) at the moment because docs for Camel 4.x have not yet been published. So we are having to link to the older 3.x Camel releases.
I think if we fixed up the 3.0.x branch as I suggested, it would probably fix the 404s you mentioned.

Ah, ok, that makes sense. Is there any published page on the camel docs site for mapstruct.html, or is it only in the unpublished 3.0.0-M2 branch? I can get some ways with mapping from one release to another, but if there's nothing to map to, I think it would need the camel docs site changes you suggest.

I thought 2.16.x is what I normally attempted to map to, but I've checked, and optaplanner does have a https://camel.apache.org/camel-quarkus/2.16.x/reference/extensions/optaplanner.html. Then I checked the code and there's something about 2.13, but nothing about 2.16.x, so I'll try that and see how many of the dead links I can resolve with that back-mapping. I just tried another one at random, and https://camel.apache.org/camel-quarkus/2.16.x/reference/extensions/elasticsearch.html doesn't exist, so it definitely won't be all of them.
For Quarkus, they have a snapshot docs section, and if I can't find anything else, I link to that. For example, https://quarkus.io/version/main/guides/ is the top level. Something similar for Camel might make sense. I think that's what you're suggesting, @ppalaga. Quarkus call it main but I think next works well.

we should remove 3.0.x and have only next, WDYT?

Yes, sounds ok to me :+1:

We already have 'next' available so linking to that would work:
https://camel.apache.org/camel-quarkus/next/reference/extensions/mapstruct.html

maybe we should leave it open to track the M2 switch

Yes, that's fine 👍

Sorry it took so long, but we finally have docs for 3.x published:
https://camel.apache.org/camel-quarkus/3.2.x/

Ahh, that would explain why I saw some new failures this morning (doh!). I haven't investigated properly to see if I need a different mapping logic now that we have the new release. I assume that's what's caused me problems. It's these guys:
      "https://camel.apache.org/camel-quarkus/latest/reference/extensions/atlasmap.html",
      "https://camel.apache.org/camel-quarkus/latest/reference/extensions/corda.html",
      "https://camel.apache.org/camel-quarkus/latest/reference/extensions/solr.html",
      "https://camel.apache.org/camel-quarkus/latest/reference/extensions/vm.html",

I had a quick look and I could see 3.2.1 links for atlasmap component, but I don't know if that's the same as atlasmap, and ... I'm continuing to look at it, but if something jumps out at you from that list about what they have in common, do shout.
Oh, and thanks for the update, and sorry to be the bearer of miserable news!

It's expected unfortunately, as all of those extensions got removed in Camel 4.0.0 / Camel Quarkus 3.2.0 / Quarkus Platform 3.2.6.Final.

Ah, thanks for the insider knowledge! So what's supposed to happen in that case is that we go back through the docs to find the version of the docs where the extension did exist, and just point to that. Do you happen to know if there would be a version of the docs that they would still exist in? If so, I can figure out why I'm not finding it.
Or, if the extension is marked as deprecated, we don't bother and just don't show its docs. But I guess there was no release of those extensions where they were marked as deprecated between being there and not-there ... is that right?

But I guess there was no release of those extensions where they were marked as deprecated between being there and not-there ... is that right?

Yeah, I think that is the problem (I've not checked to verify yet). Some of those bits got removed late on as we approached the Camel 4.0.0 GA.
Probably the only stable version of the docs where those extensions are still present is in the 2.13.x stream, which is LTS:
https://camel.apache.org/camel-quarkus/2.13.x/index.html


Add docs on how to move data from an NFS server to AzureFile NFS storage

Docs for https://github.com/2i2c-org/infrastructure/issues/876

Thanks @damianavila and @sgibson91 for reviewing this! I hope I've addressed everything!
Since I have one approval, I will merge this now :rocket:


[Bug/漏洞]

Checks / 检查

[X] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting./我确认在报告之前我已经搜索了现有的问题或者拉取请求，以避免重复报告。
[X] I confirm that I followed the Docs and read the Changelog, but this problem still exists./我确认我按照文档配置并阅读了Changelog，但这个问题仍然存在。
[X] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly./我确认我已经检查，如果我不按照说明进行操作，该问题可能会被直接关闭。

Java Version / Java版本
Java 17.0.12
Minecraft Version / Minecraft版本
1.20.1
Avaritia-Reforge Version / Avaritia-Reforge版本
<IP_ADDRESS> Lex/Neo
Description / 描述
When creating a world, my modpack crashes. Unsure exactly which mod causes the crash. In addition if used with fancymenu, the menu will bug out. Even then, without fancymenu the main menu is frozen. You can still navigate to singleplayer and create the world, but it won't go further from there.
To Reproduce / 重现
Add the combination of mods I have. Create the world and the game will close
Log / 日志
crash-2024-07-17_22.01.58-client.txt
latest.log
Config / 配置
Don't have one? Not sure if it didn't generated but I searched up Avaritia-Reforge.json in my modpack and nothing turned up

confict with Sinytra Connector

Gotcha, is it something on their side or something you can fix on your side?

Gotcha, is it something on their side or something you can fix on your side?

i try my best to fix it.

Awesome! Thank you

you get jar and try it at this

[crash-2024-07-18_00.28.14-fml.txt](https://github.com/user-attachments/fil
latest1.log
es/16279350/crash-2024-07-18_00.28.14-fml.txt)
This is my error now

plz use latest action

Hmm I believe I did. I used the one you linked, and this one (https://github.com/Nova-Committee/Re-Avaritia/actions/runs/9994087269)
This is the error I've gotten for the latest action, and still the same bug (glitchy UI, and crashes on world creation)
latest.log
crash-2024-07-18_20.01.10-client.txt

Hello! Just updating here to let you know (using the latest action) there's no crashing IF you use the version 47.3.0 of forge. I was on the modrinth modloader and it doesn't have that version, only up till 47.2.17.

Hello! Just updating here to let you know (using the latest action) there's no crashing IF you use the version 47.3.0 of forge. I was on the modrinth modloader and it doesn't have that version, only up till 47.2.17.

Do you mean that it can be solved by upgrading the forgeloader to 47.3.0?

Hello! Just updating here to let you know (using the latest action) there's no crashing IF you use the version 47.3.0 of forge. I was on the modrinth modloader and it doesn't have that version, only up till 47.2.17.

Do you mean that it can be solved by upgrading the forgeloader to 47.3.0?

It seems so!


[WIP] Add info on bromodomain benchmark

Hi. I have finished writing the inroduction and the free energy calculations on BRD4(1), the only thing left now is choosing the benchmark ligands and creating the Table. I thought it would be good to have a checkpoint now, to see if everything is going in the right direction. Best,
Germano

Thanks! I'm dealing with a grant proposal right now but will try to have a look as soon as I can. Or @mkgilson can look too...

No worries! I'll start looking into the ligands to include in the mean time! Best,
Germano

Thanks Germano! I will try to look this weekend. I am straining to
finish editing the D3R GC2 overview paper, which is already late!
Best,
Mike
On 6/29/2017 11:25 AM, GHeinzelmann wrote:

Hi. I have finished writing the inroduction and the free energy
calculations on BRD4(1), the only thing left now is choosing the
benchmark ligands and creating the Table. I thought it would be good
to have a checkpoint now, to see if everything is going in the right
direction. Best,
Germano

    You can view, comment on, or merge this pull request online at:

https://github.com/MobleyLab/benchmarksets/pull/36
    Commit Summary

Add files via upload

Add files via upload

Add files via upload

Add files via upload

Add files via upload

Add files via upload

Delete benchmarkset-GH.pdf

Delete benchmarkset-GH.tex

Add files via upload

Add files via upload
File Changes

M paper/benchmarkset.bib
https://github.com/MobleyLab/benchmarksets/pull/36/files#diff-0
(8858)

A paper/benchmarkset[WIP].pdf
https://github.com/MobleyLab/benchmarksets/pull/36/files#diff-1 (0)

A paper/benchmarkset[WIP].tex
https://github.com/MobleyLab/benchmarksets/pull/36/files#diff-2
(1052)

A paper/figures/brd4.pdf
https://github.com/MobleyLab/benchmarksets/pull/36/files#diff-3 (0)
Patch Links:

https://github.com/MobleyLab/benchmarksets/pull/36.patch

https://github.com/MobleyLab/benchmarksets/pull/36.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MobleyLab/benchmarksets/pull/36, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AQEJQG0LSP2drvVDrrUU1Yg-36Cvk98yks5sI-wUgaJpZM4OJsWM.

--
Michael K. Gilson, M.D., Ph.D.
Professor, Skaggs School of Pharmacy and Pharmaceutical Sciences
UC San Diego
9500 Gilman Drive
La Jolla, CA 92093-0736
voice:<PHONE_NUMBER>
http://gilson.ucsd.edu
http://bindingdb.org
http://drugdiscovery.ucsd.edu
http://drugdesigndata.org

OK, just a small correction, sorry about that. It turns out some of the ligands of BRD4(1) are not neutral, so in the part that says that "all of the ligands are neutral" can be changed to "most ligands are neutral", or maybe something similar. Best,
Germano

@GHeinzelmann - I am starting to look at this. I took the liberty of:
a) Changing the title of the PR to reflect what this is doing
b) moving your "benchmarksets[WIP].pdf" and ".tex" to "benchmarksets.pdf" and "benchmarksets.tex"; be sure to "git pull origin GHeinzelmann" to pull down these changes
The reason (b) is appropriate is that you aren't actually changing these files until I agree to it, you're just PROPOSING changing them. So it's totally fine to make that proposal. AND, changing the ACTUAL FILES will make it so GitHub will show me what you're changing in the files (see "files changed" tab above!) in a useful way, rather than showing me that you've introduced two new files which consist of entirely new content.

@GHeinzelmann - it looks like things are headed in the right direction, though I still need to have a look at whether organization of this is consistent with the other benchmarks. But on the whole the content seems solid and worthwhile, which I very much appreciate!!
I left some comments for you to look at when you have time, and potentially incorporate in an update to this PR.
I will probably also go through the parts you have written and make some edits to grammar/style and submit a PR to from my branch to your branch (so that you can review/merge it and then it will update this PR). But that probably won't be for a day or two, so in the meantime, please proceed!
One other thought -- if you're up for it, once you have this written up, it would probably be good to ask Matteo Aldeghi or Phil Biggin to have a look over it as sort of an informal peer review.
Oh, finally -- your PR should also add yourself as an author!

@GHeinzelmann - how does this stand? Do you think it's ready to merge? I'll have some time in transit this week (part of it is today) and hope to review your changes to finalize, if you think it's done.

Hi. I think pending a final review it is ready, After I got the openeye license I have redone the ligand table figures and also got the smiles for each molecule. I just could not find the PubChem ID for two of them, but the rest is all there. I was waiting for some feedback on the ligands before I uploaded the structures, but since we are merging soon I'll do that part now so everything is ready.

BTW is is on the GHeinzelmann branch.

BTW the last version is on the GHeinzelmann branch.

I was happy with the ligand choices.

@GHeinzelmann - I spotted one more issue. I can't parse the SMILES string for Cc1c(sc-2c1C(=N[C@H(c3c2c(no3)C)CC(=O)N)c4ccc(cc4)Cl)C , "Compound 3". Can you re-check please? It doesn't parse with my tools.
I did find a PubChem compound ID for RVX-OH, the other one where you were missing an ID, so this is the last one remaining.

OK, thanks for finding the PubChem ID for that compound, sorry I could not find it myself but I did try :). About the smiles for compound 3, I just rechecked and it it renders the 2D structure correctly, so not sure what could be the problem. I am pasting it again directly from the input file of the openeye 2D rendering:
Cc1c(sc-2c1C(=NC@HCC(=O)N)c4ccc(cc4)Cl)C
It looks the same as the one you wrote, except for the [C@H part, which seems to be missing a closure bracket in yours. I just checked the paper table and it is written the same way as above, so that might fix it.

Aside from that, I am creating all the ligand and solvated structures to include in the input_files and that part will be finished today.

Ah, you seem to have nailed it, @GHeinzelmann . The string I'm seeing in the LaTeX file AND on GitHub is  Cc1c(sc-2c1C(=N[C@H(c3c2c(no3)C)CC(=O)N)c4ccc(cc4)Cl)C which is missing the close square bracket after the C@H, but the string you just pasted, Cc1c(sc-2c1C(=N[C@H](c3c2c(no3)C)CC(=O)N)c4ccc(cc4)Cl)C, as that and parses properly. Can you fix?
And, yeah, for some reason I can't find this one in PubChem, which is odd... So, we have to leave N/A for that one for now.

Yes I'll fix that no problem. The .tex version I have here has the bracket, so it is possible that I accidentally erased it when I had the GitHub window open. Sorry about that.

I think we are all set and ready to merge. I figured I would add myself as a second author, not sure if that is the idea but it can be easily changed if needed. The input files and the README.md file are in the input_files/BRD4 folder, following Niel's organization in his branch. I have also addressed the review comments in the new .tex and .pdf versions of the main paper. I think I have covered everything that was missing, but let me know if there is anything else to be done. I think it all looks good, thanks for inviting me to contribute to the paper!

@GHeinzelmann - great, thanks! I'll take a final look. I just need to hear back from Mike...

Hi. I have updated the paper in my branch with the latest comments from @mkgilson, also changing a little the bit the information on Compound 4 and Compound 8a in Table 7 and in the section where we discuss this Table (last paragraph of subsection 1). For now I have kept these two ligands in the benchmarks, discussing the docking of Compound 8a (and the similarity to the pose of Compound 4) in the README from the input_files/BRD4 folder.

I'm running behind on the e-mail thread/updates here but hope to review this and get everything merged by the end of the week. Thanks for all your work on this!

@GHeinzelmann - congratulations, it is merged! You're now an author. :) Thanks for all your work on this.

That's great! Thanks for the reviews and the opportunity!

:-)
On 8/4/2017 1:12 PM, David L. Mobley wrote:

@GHeinzelmann https://github.com/gheinzelmann - congratulations, it
is merged! You're now an author. :) Thanks for all your work on this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MobleyLab/benchmarksets/pull/36#issuecomment-320342669,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQEJQDOdqjbXIgOBXdDlTB9sACU_QaHzks5sU3s1gaJpZM4OJsWM.

--
Michael K. Gilson, M.D., Ph.D.
Professor, Skaggs School of Pharmacy and Pharmaceutical Sciences
UC San Diego
9500 Gilman Drive
La Jolla, CA 92093-0736
voice:<PHONE_NUMBER>
http://gilson.ucsd.edu
http://bindingdb.org
http://drugdiscovery.ucsd.edu
http://drugdesigndata.org


BUGFIX: Fixed issue where TimeField_Readonly would only show "(not set)"

When using a TimeField in a form and either the form is made readonly or the field itself is made readonly the created TimeField_Readonly instance will only show (not set) due to $this->valueObj not being defined or used in the TimeField class.
This pull request changes the TimeField_Readonly class to use the output of TimeField::Value() as the value displayed. It also passes through the values of some of the TimeField specific properties so it appears closer to what the TimeField would.
This issue was observed under 4.5 but also affects 4.4 (and likely prior) as the 4,4 version of the file has not changed between the two.

Code used for testing:
// app/src/MyFormPage.php

<?php

namespace {

    use SilverStripe\CMS\Model\SiteTree;

    class MyFormPage extends SiteTree
    {
        private static $db = [];

        private static $has_one = [];
    }
}

// app/src/MyFormPageController.php

<?php

// adapted from https://docs.silverstripe.org/en/4/developer_guides/forms/introduction/

namespace {

    use SilverStripe\CMS\Controllers\ContentController;
    use SilverStripe\Forms\FieldList;
    use SilverStripe\Forms\Form;
    use SilverStripe\Forms\TimeField;
    use SilverStripe\Forms\TimeField_Readonly;

    class MyFormPageController extends ContentController
    {
        public function MyForm()
        {
            $timeStr = '17:10:11';
            $fields = new FieldList(
                TimeField::create('MyTimeField', 'My time field', $timeStr),
                TimeField_Readonly::create('MyTimeFieldReadonly', 'My time field readonly', $timeStr),
                TimeField::create(
                    'MyTimeFieldReadonlyReadOnlyTransformation',
                    'My time field readonly transformation',
                    $timeStr
                )
                    ->performReadonlyTransformation()
            );
            return new Form($this, 'MyForm', $fields, null, null);
        }
    }
}

// themes/simple/templates/MyFormpage.ss

<% include SideBar %>

$Title - (templates/MyFormPage.ss)

$Content

  $MyForm
  $CommentsForm

OK looking pretty good now, I've tested and it all seems to be working correct.  Before I merge this could I get you to do a couple more things:

write a couple of happy day unit tests
squash your commits down

Cheers!

I've added tests, and also made a couple of additional changes to the performReadonlyTransformation method to account for a crash I found in writing tests where if a value was set and you have a timezone it would throw an exception. Also set the value of the html5 property on the readonly field to match that of the field being tranformed.

@UndefinedOffset looks like Scrutinizer is complaining about something here https://scrutinizer-ci.com/g/silverstripe/silverstripe-framework/inspections/7c104905-3696-4a28-bd6a-d68192601373/issues/files/tests/php/Forms/TimeFieldReadonlyTest.php?status=new&orderField=path&order=asc&honorSelectedPaths=0
There's two other things that Scrutinizer is complaining about, there it seems like they're unrelated to code that you've added in this PR

Fixed :) Scrutinizer should be happier with it now :)

Great thanks for getting this pull-request to a good state.
Before this gets merged we're going to need to do some testing at our end.  At this stage we're not sure when this will get prioritised.
During the testing phase we may make some alterations to the pull-request, though rest-assured, I'll ensure that you still get credited for the work you've done in the release changelog.
Thank you again for your contribution!


Add logLevel in XSD

Add logLevel in XSD

Sorry for my multiple commits.  One should have been enough.
@nicogis - thanks for contributing!


Setting doesn't appear at the foreground

Description

Settings only appear once when it's first clicked.
When settings is already in the background, clicking on settings will show nothing; it should bring it back to the foreground.
Steps to reproduce

Launch Tophat
Click on Settings for the first time
Settings appears
Put Tophat Settings in the background
Click on settings again
Tophat settings does not show in the foreground

Expected behavior
Tophat settings should show in the foreground when clicked

When settings is already in the background, clicking on settings should bring it back to foreground.
Screenshots
https://github.com/user-attachments/assets/98e7278b-dbe0-41e3-b2ee-3e90c3a515b6

System

Device: MacBook Pro 2021
OS:  macOS 14.6.1 (23G93)
Platform: Apple M1 Max

Other details

It looks like someone else has faced the same issue in macOS 14+:
https://github.com/orchetect/MacControlCenterUI/issues/10
https://www.reddit.com/r/SwiftUI/comments/16ibgy3/settingslink_on_macos_14_why_it_sucks_and_how_i/
And he made this library to solve it:
https://github.com/orchetect/SettingsAccess


⚠ Mobitopup has degraded performance

In a954296, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 6980 ms

Resolved: Mobitopup performance has improved in d754291.


Adapt rhtap-cli to run e2e tests

it depends on PR https://github.com/redhat-appstudio/rhtap-e2e/pull/139

rhtap-cli fails with
[User error] Validation failed for pipelinerun rhtap-cli-6k6t5-f5l8z with error invalid input params for task rhtap-e2e-runner: missing values for these params which have no default values: [test-name]

/retest

I already verified it, see the result . For now it just triggers one of cases   tests/gpts/github/quarkus.test.ts. I still need to do investigation to make all test be performed successfully

/lgtm


Fix README custom_commands user/group

Description
Readme Typo
Issues Resolved
[List any existing issues this PR resolves]
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

@foxdalas Can you squash this down to a single commit?

@tas50 done

Thank you @foxdalas !


Refactoring suggestion

It is the refactoring suggestion for the https://github.com/objectionary/speco/pull/39 PR.
The purpose of that PR provide an idea for correct speco design. That PR should be closed!.
Because:

Tests are broken.
Qulice checks failed.

PS:
Walk, XmirWalk and EoWalk classes can be refactored since they still have code duplication.

@volodya-lombrozo answered in #39


FEATURE: Add report with topics not indexed

I think we also need to report on "links in the index, which are no longer in the docs category" (e.g. they're moved, or deleted). In the data-explorer query these are the ones labelled as 'unexpected'

I think we also need to report on "links in the index, which are no longer in the docs category" (e.g. they're moved, or deleted). In the data-explorer query these are the ones labelled as 'unexpected'

I'll add another report in a followup PR


Imaginary directory

Describe the bug
Where this imaginary Directory Path you speak of. for the session, Ive added The mozilla Profile it creates a session json file and still says doesn't exists on windows 11?

Hey @piggyza are you encountering an error ? If so, can you provide a screenshot ? If the session file doesn't appear make sure you follow the installation instructions. Also make sure to be logged in to your account on Instagram from Firefox before running IGFI and have the Firefox browser installed. Finally, verify that Python is configured correctly and you've installed a compatible with the script version (Python3.X >). To make that verification you can simply enter the command:
python --version


Error in the command, not supported

The command to reset classic VPNs is not valid anymore, when using it in Cloud Shell says that is not valid.
Reset-AzureVNetGateway –VnetName “ContosoVNet”

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

ID: 9d654281-21aa-79ca-592d-d8cfe0bf1816
Version Independent ID: 5fdec85a-a9d9-0b59-c65e-fecf0d2eee2a
Content: Reset an Azure VPN gateway to reestablish IPsec tunnels
Content Source: articles/vpn-gateway/vpn-gateway-resetgw-classic.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

I will add a note to the doc to say to use this locally on the desktop using the service management powershell cmdlets. I believe this is where the problem is.

@ositobio , you need to switch to ASM mode in cloud-shell .
Use the below command:
azure config mode asm
If you are trying from a PC, do try following the below instruction.
Make sure you install the latest version of ASM module and try the command again.
ASM module: https://docs.microsoft.com/en-us/powershell/azure/servicemanagement/install-azure-ps?view=azuresmps-4.0.0#azure-service-management-cmdlets

I've updated the documentation to make it clear that you can't use Azure CloudShell for the classic/service management networking deployments.

@ositobio , I will go ahead and close this issue, please reach out if you have any further questions.

Hi Msrini / Cheryl
Thanks for the update, the document is much clearer now.!! :)


No ETA and few other parameters on K1 Max

Hello,
I am using K1 Max with Moonraker and fluidd installed. This integration works wonderfully and after changing the port in the integration config, I was able to get integrated camera working. There are few sensors however, which are not updating like time estimates and especially missing the ETA is a painful.
![Screenshot_20241114_211913_Home Assistant]

(https://github.com/user-attachments/assets/16986c98-d4ce-459b-aab2-d00992c835e6)
I will add the Moonraker log shortly.
My setup consists of updated K1 Max and my home NAS running HA 2024.8 with the moonraker integration updated to latest version.
Any ideas what might be missing? Seems like time-related sensors are missing, but layer number is missing as well. I am using octoapp on android phone and that is able to provide ETA, so clearly the info is exposed through moonraker.

This is caused by using Creality Print slicer. When using OrcaSlicer all parameters are working as expected, layer count included.


Make a directory to thank contributors on the site.

Make a directory to thank contributors on the site and link it via the README.md, dont list it on the main site though.

On second thought, I don't think, I'll add this for the simple reason that what should I inculde there, maximum everything I inculded in the thank you contributors section in commit e2af344
I was thinking of using a github api to tell what each contributor of project has done but I cant seem to find one, @TechStudent11 do you know of any api to do this?

https://docs.github.com/en/rest/reference/collaborators

https://docs.github.com/en/rest/reference/collaborators

This is using json right? Can you work on this?

https://docs.github.com/en/rest/reference/collaborators

This is using json right? Can you work on this?

So for the website right?

https://docs.github.com/en/rest/reference/collaborators

This is using json right? Can you work on this?

So for the website right?

yeah, you can make a directory called /collaborators, and do this there, I arent planning putting the directory on the main site. I will link it in README.md though

Hmmm, maybe in a few hours.

aka later today

Hmmm, maybe in a few hours.

Yeah ofc, take your time. This isnt something I need done on priority, I made a list of what I need done ASAP: https://github.com/users/KendallDoesCoding/projects/3

Adding the after-pr label now, as I though of another idea, #193 . I'll start work on that, and you can do work on the same PR or in a different one @TechStudent11 . I'm making the branch to work on #193 in the organization forked repo.

@TechStudent11 Please work on this issue in the following:
Directory - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/tree/work-on-193/credits/commits
HTML - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/blob/work-on-193/credits/commits/index.html
Feel free to create a CSS, JS file in this directory if needed. I've called out this directory in the credits page which I'm working on so need you to do the stuff here, thanks.

@TechStudent11 Please work on this issue in the following:
Directory - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/tree/work-on-193/credits/commits HTML - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/blob/work-on-193/credits/commits/index.html
Feel free to create a CSS, JS file in this directory if needed. I've called out this directory in the credits page which I'm working on so need you to do the stuff here, thanks.

In the HTML I added a h1 saying coming soon, delete that when you start working on this.

@TechStudent11 Please work on this issue in the following:
Directory - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/tree/work-on-193/credits/commits HTML - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/blob/work-on-193/credits/commits/index.html
Feel free to create a CSS, JS file in this directory if needed. I've called out this directory in the credits page which I'm working on so need you to do the stuff here, thanks.

In the HTML I added a h1 saying coming soon, delete that when you start working on this.

@TechStudent11 Please start working on this today or tommrow, thanks.

@TechStudent11 Please work on this issue in the following:
Directory - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/tree/work-on-193/credits/commits HTML - https://github.com/Kendall-Does-Coding-Websites/mogul-christmas/blob/work-on-193/credits/commits/index.html
Feel free to create a CSS, JS file in this directory if needed. I've called out this directory in the credits page which I'm working on so need you to do the stuff here, thanks.

In the HTML I added a h1 saying coming soon, delete that when you start working on this.

@TechStudent11 Please start working on this today or tommrow, thanks.

Can’t today. My computer is in a bit of a state of “hiatus”. Can’t tomorrow because well I have other things I have to do (school, etc). Sorry. I work on it as soon as I am available. I’m actually typing this on my phone haha. By the way, what do you even want on this page? You want to thank the contributors, but how?

https://docs.github.com/en/rest/reference/collaborators

Ok, np. Do it as soon as your free. I thought your doing it using this right? https://docs.github.com/en/rest/reference/collaborators

For now, I'll embed https://github.com/KendallDoesCoding/mogul-christmas/graphs/contributors into the page I made in a recent PR #(194 I believe)

This was pretty much done in<PHONE_NUMBER>4d7ee9df59648ceaed1f840735dc68, although what I want for this is that it displays all the commits contributors made even in PR's etc, idk if that's possible though, so please let me know @TechStudent11 . But for now, it isn't really needed as I've displayed PR's contributors have worked on.

@KendallDoesCoding According to this answer on SO, yes it is possible: https://stackoverflow.com/a/58197588
But it would add extra bulk to the site and slow down load times.

Ok, added talk: discussion, for now keeping this open, but it's most likely a no and we don't need to proceed on this, in that case.

This is kinda done in the credits directory, I can add the lines "Thanks to all our project contributors, your work and help is really appreciated."


test(tomcat10): Restore Engine Integration Tests

In my ongoing efforts to streamline the backporting of c7 commits, I’ve developed a pipeline that automates this process wherever feasible. This PR marks the first backported commit to test and validate the pipeline.
Moving forward, I’ll be creating individual PRs for each backported commit. When it makes sense, I’ll group related commits into a single PR for clarity and efficiency.
Original Commit message:
Context: Tomcat 10, CDI and Weld Tests
Fix: This commit includes a fix for the tests to load weld under the Tomcat lib folder instead of placing it into the WEB-INF/lib folder of the individual wars used by the tests.
Related-to: https://github.com/camunda/camunda-bpm-platform/issues/4434

Hi @hauptmedia, thank you very much, creating that tooling is much appreciated!


clarify meaning of empty include query param

The meaning of an empty value for the include query parameter was not clear yet. As @freddrake pointed out in #1530 a reader could interpret either as an empty list of relationship paths or as a single (invalid) relationship path.
The meaning of an empty value for fields query parameter is explicitly specified:

The value of the fields parameter MUST be a comma-separated (U+002C COMMA, “,”) list that refers to the name(s) of the fields to be returned. An empty value indicates that no fields should be returned.

I think the same applies to the include query parameter even though it's not mentioned explicitly yet. Otherwise a client would not be able to tell a server to not include any related resources - even if that server includes some resources by default.
I think this is not a breaking change. Let's have a look at several scenarios:

An existing implementation may consider an empty value as a single relationship path. An empty string is not a valid relationship path. The server would be unable to identify the relationship path and response with 400 Bad Request.

If a server is unable to identify a relationship path or does not support inclusion of resources from a path, it MUST respond with 400 Bad Request.

An existing implementation may silently ignore an include parameter if it's value is empty.

If it does not include any related resources by default, the existing implementation would be valid.
If it includes related resources by default even if an include parameter with an empty value is given, the implementation is not compliant with the specification even today. A server must not ignore a given include query parameter. It must response with a 400 if it does not support it.

If an endpoint does not support the include parameter, it MUST respond with 400 Bad Request to any requests that include it.
If an endpoint supports the include parameter and a client supplies it, the server MUST NOT include unrequested resource objects in the included section of the compound document.
[...]
If a server is unable to identify a relationship path or does not support inclusion of resources from a path, it MUST respond with 400 Bad Request.

An existing implementation may already interpret an empty ignore parameter as an empty list of relationship paths and not include any related resources.

The meaning of an empty value for a Sparse Fieldset was also not explicitly specified since a few months ago. It was added in #1500. This was also not considered a breaking change.
Closes #1530

Thanks again @jelhan for the clarifying PR and also to @freddrake for raising the issue in #1530.

Thanks again @jelhan for the clarifying PR and also to @freddrake for raising the issue in #1530.


Getting warning on every build.

DBFlow Version: 3.1.1
Issue Kind : Question
Description: This version is working fine with my GSoC project but getting  warning whenever I am building the project
Warning:File for type 'com.raizlabs.android.dbflow.config.MifosDatabaseMifos_Database' created in the last round will not be subject to annotation processing.
Warning:File for type 'com.raizlabs.android.dbflow.config.GeneratedDatabaseHolder' created in the last round will not be subject to annotation processing.

How can I remove this warning?

These warnings are a result of the way the annotation processor defers the generation of those files until the final round of processing. This allows tables that are generated in the first round of processing to not be missed when the database and holder are generated. (I have a PR #947 that adds the content provider to the list of final round deferred classes as well).
There may be a way to silence the warning but they are nothing to worry about and can be disregarded. You might even consider the warning to be a good sign as it is quick proof of how robust the annotation processor is and shows some of how it works, thus enabling advanced work flows.


[Unity3d] Possible to rebuild Parse SDK from Xmarin now?

as titled. Is Xmarin Problem fixed and abled to build Parse SDK dotnet now?
Or is it possible to use Visual Studio Code to compile the code?

No. Xamarin still fails to find any files when using wildcard search paths in .csproj files :(. You could expand these paths manually, and it should still compile on Xamarin, however.

@richardjrossiii How to fix it manually?

@richardjrossiii  can you provide more details on how to change search paths in .csproj files? or how to do it in windows? as we can't build ios from windows right? Please advise and thank you.

@kenglou You can build iOS on windows absolutely fine, you will just need to install the proper Xamarin plugin.
But essentially, inside the project file, there is a list of paths to compile:
https://github.com/ParsePlatform/Parse-SDK-dotNET/blob/master/ParseCore/ParseCore.Portable.csproj#L40-L45
Wildcard search paths are not supported with mono's build system, for some reason.
If you expanded those paths to include each of the individual files in those directories, it should compile with mono.

Indicating that this is still an outstanding issue. We'll need to address this in such a fashion that it can be built without having to do anything manual...

This issue is solved with the latest version available from the master branch.
It works with Xamarin for iOS and Android and the respective platforms using Unity3D.
If you encounter any problems with the latest version feel free to create a new issue to discuss possible solutions.


crash when using the latest update

Thanks for making this mod! I've loved using it, but the latest update crashes Minecraft with no logged error. This happens as long as this mod is enabled and version 1.16.18 for Fabric 1.20.6 is installed. Please let me know if I there is more info you need and I'll try to provide it.

Looks like the update to Indium that just became available resolves the crash. Closing this issue.


docs: fix custom component links in admin

What?
Fixes links for custom components in a few places in admin docs.
Why?
To link users to the correct location in the docs.
How?
Changes to docs/admin/components.mdx and docs/admin/customizing-css.mdx

@akhrarovsaid Looks good! Thanks again!


Referencing a subkey which doesn't exist gives crash

I am not sure whether this is a real bug or undefined behaviour, it's related to referencing a subkey which doesn't exist including its parent.
My understanding is that when I try to do a C++ reference of a json object , if the key doesn't exist it will insert the key
and give back the reference of it.
In my example I have a main object json js = null;
Now I am trying to do json& newJs = js["key1"]["key2"];
Later when I try to insert some object to this newJs, it crashes.
I overcame this issue by doing first js["key1"]["key2"] = null and then giving reference of it.
If you think this is not an undefined behaviour, I will try to replicate the issue in simpler code.

This is documented behaviour,  see https://nlohmann.github.io/json/api/basic_json/operator[]/. Use https://nlohmann.github.io/json/api/basic_json/at/ which does handle non-existing keys gracefully.

It's explained in documentation. Closing this issue.


[SPARK-31642][FOLLOWUP] Fix Sorting for duration column and make Status column sortable

What changes were proposed in this pull request?
In #28485 pagination support for tables of Structured Streaming Tab was added.
It missed 2 things:

For sorting duration column, String was used which sometimes gives wrong results(consider "3 ms" and "12 ms"). Now we first sort the duration column and then convert it to readable String
Status column was not made sortable.

Why are the changes needed?
To fix the wrong result for sorting and making Status column sortable.

Does this PR introduce any user-facing change?
No

How was this patch tested?
After changes:

Test build #123624 has started for PR 28752 at commit acd9d54.

cc @sarutak @srowen Kindly take a look.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28248/
Test PASSed.

Looks good. Are there other similar cases, I wonder?

I will look into other places as well.

Test build #123624 has finished for PR 28752 at commit acd9d54.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123624/
Test PASSed.

Looks good. Pending for other similar cases.

I checked. I cannot find any similar case for other tables.

@gengliangwang FYI

LGTM. Do you all have an opinion on back porting to branch-3.0? I think I would.

LGTM. Do you all have an opinion on back porting to branch-3.0? I think I would.
Yeah, it would be better. What do you think @iRakson ?

LGTM. Do you all have an opinion on back porting to branch-3.0? I think I would.
Yeah, it would be better. What do you think @iRakson ?

Just to be clear, We are talking about back porting pagination support or fixing this sorting issue ?
If we are only fixing this sorting issue then implementation will be slightly different as we are using listingTable for generating table.

Pagination feature is not in branch-3.0 so how about focusing the sorting issue?

Ok. I will raise a PR for that. Just one doubt, for fixing in 3.0 should I file a new JIRA(as implementation will be slightly different) or just raise as a follow up for this only.

We will merge this just to master. If you can cleanly fix in 3.0 too, yes that's fine in a new PR. It should be the same JIRA.

We will merge this just to master. If you can cleanly fix in 3.0 too, yes that's fine in a new PR. It should be the same JIRA.

Oh. I created a new JIRA. SPARK-31642 is for introducing pagination support in structured streaming tab, so I filed another JIRA(https://issues.apache.org/jira/browse/SPARK-31983).
I raised a PR here.

Merged to master. I'll look at the backport.

Oh I see, this was attached to the JIRA for pagination. It does make sense to have a separate JIRA for that. Let me retroactively fix up the links.

Thank You. @srowen @sarutak.


Update gitlab

I'm not familiar with ruby. So, please, can anyone update gitlab? There are several errors in the version we have now in nixpkgs. For example https://github.com/gitlabhq/gitlabhq/issues/7413
cc @offlinehacker @teh

Running gitlab was such a pain that we moved away from it. I can create a PR to remove it from nixpkgs so we don't have unmaintained code?

@teh no. I will try to update it myself then.

I agree with @teh to remove it. Packiging rails on nix is real pain, and it's a lot of hacking to make it work. @jagajaga I would suggest you to run it in docker instead. Of course you can waste some of time to make it work, but it's not really worth it.

@offlinehacker how can I run it in docker?

The easiest way is to use docker-compose and docker-compose.yml from
offical gitlab repo. Then you just run "docker-compose up"
V sre., 23. sep. 2015 11:05 pop. je oseba Arseniy Seroka <
<EMAIL_ADDRESS>napisala:

@offlinehacker https://github.com/offlinehacker how can I run it in
docker?
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/9890#issuecomment-142727810.

@offlinehacker thank you. So if you think it's better to remove -- let's do it.
@manpages please read this thread.

@jagajaga I could think of a way to run it using lightweight container with overlayfs mounted over read only gitlab package in nix store. We can already do this with runc/docker/lxc but I think we need some official way embeded in nix.


pip-sync upgrades frozen packages to newer versions since 1.4.3

example requirements.txt
Django==1.8.8
django-admin-sortable==2.0.9  # has "Django" in install_requires

when I call pip-sync requirements.txt the command will upgrade Django to 1.9.1 (since 881cd7540f29533d54f515e6a4d6e2bf20856c52 we call pip install --upgrade django-admin-sortable==2.0.9 which leads to the upgrade here)
is this wanted behavior?

Oh wow, that's definitely not wanted. I'd even say it's exactly what pip-tools as a project tries to protect against in the first place! Hmm. The --upgrade fix was a fix for a different problem. This poses an interesting challenge…

Confirmed here too, we had Django upgraded to 1.9.x while it was actually locked to 1.8.3 in our requirements.in file, and we were upgrading a specific package that didn't lock the dependency properly. Maybe it has something to do with case, as in said package, the requirement was on django, not Django. Not sure.
I'd suggest you prune pip-tools 1.4.3 if you cannot fix it soon because it breaks one of the fundamental behaviors that make pip-tools so awesome (and I mean it in a big way, you made our lives far easier, thanks!! :-))
Good luck, happy to help if I can, let me know.

I'm indeed going to revert 1.4.3 to be equal to 1.4.2 first thing as soon as I'm behind a computer (which should be within an hour).

This should now be fixed in 1.4.4. Thanks for catching this one! I'll see if I can work out a fix in 1.4.5 for the original problem without introducing this behaviour :relaxed:

Great, thanks a lot!

thanks @nvie


Easy way to support copying an existing model object

based on the discussion: http://stackoverflow.com/questions/30144657/how-can-i-easily-duplicate-copy-an-existing-realm-object
No matter if the model has primary keys or not, there should be an easy way to copy/duplicate an existing model object (and its relationships models).

Creating an exact copy of an object with a primary key doesn't make any sense. Creating a copy but with a different primary key can be done with:
let copy = ObjectType(value: oldObject)
copy.primaryKey = newPrimaryKey
realm.add(copy)

which doesn't seem like something that desperately needs one-liner sugar to me.

yeah that's what I meant, I didn't want 2 objects with the same primary key, but I will definitely try your code snippet thanks for that, and see if i can generalize because the primary key field is known.

Did you end up generalizing a "copy object with new primary key" functionality, @marcvanolmen? Feel free to reopen this issue if you have any further questions.

@tgoyne doesn't work for me:
Static member 'primaryKey' cannot be used on instance of type 'ObjectType'

As of now, Dec 2020, there is not proper solution of this issue. We have may workarounds though.
Here one I have been using, and one with less limitations in my opinion.

Make your Realm Model Object classes conform to codable

class Dog: Object, Codable{
    @objc dynamic var breed:String = "JustAnyDog"
}

Create this helper class

class RealmHelper {
    //Used to expose generic
    static func DetachedCopy<T:Codable>(of object:T) -> T?{
       do{
           let json = try JSONEncoder().encode(object)
           return try JSONDecoder().decode(T.self, from: json)
       }
       catch let error{
           print(error)
           return nil
       }
    }
}

Call this method whenever you need detached / true deep copy of your Realm Object, like this:

 //Suppose your Realm managed object: let dog:Dog = RealmDBService.shared.getFirstDog()
 guard let detachedDog = RealmHelper.DetachedCopy(of: dog) else{
    print("Could not detach Note")
    return
 }
//Change/mutate object properties as you want
 detachedDog.breed = "rottweiler"

As you can see we are piggy backing on Swift's JSONEncoder and JSONDecoder, using power of Codable, making true deep copy no matter how many nested objects are there under our realm object. Just make sure all your Realm Model Classes conform to Codable.
Though its NOT an ideal solution, but its one of the most effective workaround.


Gallery banner is too dark

Who turned out the lights?
The gallery banner for the JSHint Marketplace page is very dark and it is difficult to read the text.

@gregvanl I followed your advice with the icon and the color. It is right now:
	"icon": "jshint_icon.png",
	"galleryBanner": {
		"color": "#2e2e2d"
	}

Any advice

still dark to me. +1

I think you need to set the gallerybanner theme to dark to get light text.
"galleryBanner": {
	"color": "#5c2d91",
	"theme": "dark"
     }

Made the suggested changes and re-published the extension.


Register ilitek.is-not-a.dev

Added ilitek.is-not-a.dev using the CLI.

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA


thanks afriscic

Mr. Developer
First of all, thank you very much for the wonderful effort of this innovation
To be honest, it is a very wonderful library in terms of reading speed and image capture speed
I really can't describe it
But as you know, your followers are always greedy lol :)
I tried to use the view MVVM  , but it didn't work for me
And I think it's largely due to my weak programming experience
Can you put us a code for the view model
And I also hope that the next update will take a picture of the qrcode itself and not the full screen
Because there is definitely a category that will need to pass a picture of the QR code, for example, to make sure there is a logo or something similar for the sake of machine learning
It's just an idea
And for the second time, thank you, thank you :)

in your XAML:
<scanner:CameraView x:Name="qrScanner" TapToFocusEnabled="True" CameraFacing="{Binding CameraFacing}" TorchOn="{Binding TorchOn}" AimMode="{Binding AimMode}" BarcodeSymbologies="QRCode" VibrationOnDetected="False" OnDetectionFinishedCommand="{Binding HandleQRCodeCommand}" PauseScanning="{Binding PauseScanning}"/>
in your viewmodel:
    public Command HandleQRCodeCommand { get; }     private async void HandleQRCode(object o)     {       if (o is BarcodeResult[] result)       {         if (result.Length > 0)         {           if (result[0].BarcodeFormat == BarcodeFormats.QRCode)           {             PauseScanning = true;             Vibration.Vibrate(100);             // Handle result[0].DisplayValue here             PauseScanning = false;           }         }       }     }
in ctor of your vm:
HandleQRCodeCommand = new Command(HandleQRCode);
in code-behind of your page:
 protected override void OnAppearing()     {       base.OnAppearing();       qrScanner.CameraEnabled = true;     }     protected override void OnDisappearing()     {       base.OnDisappearing();       qrScanner.CameraEnabled = false;     }
I have not listed other implemented properties here.
Also I disabled build-in vibration, because it lasts too long and couldn't be changed.
Hope it will be helpful for your purposes.

In v1.7.0 and v2.1.0 type of command parameter was changed from BarcodeResult[] to HashSet<BarcodeResult>.
Returning to your recent question (#116), the versions I mentioned in this comment works fine with MAUI v9.0.10 and VisualStudio 17.13.0 Preview 1.0 on Xiaomi 13 device with Android 14.


use the [self showPrivacyHelperForType:DBPrivacyTypeLocation]; in AppDelegate

how to implement the same in the AppDelegate with Reachability

@abidullah i think you need to replace self with the visibleViewController of your current navigation controller


New release

Can we have a new release to forge please? :smile:

@liger1978 sure will do :)

Released now.


Rollback/Recover the volume if the expansion of all replicas failed

If all replica expansion failed and the volume became Faulted state, we don't know the salvage works fine and there is no way to roll back the expansion.

Backend merged. Need instruction for UI.

Prerequisites of the API:

volume.controllers[0].isExpanding == false
volume.controllers[0].size != 0 && volume.controllers[0].size != volume.size

Need to add tips when users try to call this API:
If the in-progress expansion you want to cancel is triggered by the PVC size update, this API won't help revert the PVC. Since the PVC size can not be shrink, users need to clean up then recreate the PVC and PV after the expansion canceling success:

Update the field spec.persistentVolumeReclaimPolicy to Retain for the related PV. Otherwise, the volume will be removed when the PVC/PV is gone.
Delete PVC and PV.
Recreate PVC and PV via yaml file or Longhorn UI

@smallteeths Can you help with the UI? Thanks.

The way to prevent the expansion from success so that the expansion will keep retry:

Create a volume and attach it.
Enter the replica data directory and run the following command for each replica:

echo "while true; do if [ ! -e \"volume-snap-expand-4294967296.img.meta.tmp\" ]; then mkdir volume-snap-expand-<PLEASE INPUT THE EXPANSION SIZE>.img.meta.tmp; fi; sleep 1; done;" > test.sh && chmod +x test.sh && ./test.sh &

e.g., If I want to make sure the volumev1 fails to expand to 4Gi, and the only replica of the volume is in instance-manager-r-894f8e3b:
~ kubectl -n longhorn-system exec -it instance-manager-r-894f8e3b bash
root@instance-manager-r-894f8e3b:/# cd /host/var/lib/longhorn/replicas/v1-d482cb0c/
root@instance-manager-r-894f8e3b:/host/var/lib/longhorn/replicas/v1-d482cb0c# echo "while true; do if [ ! -e \"volume-snap-expand-4294967296.img.meta.tmp\" ]; then mkdir volume-snap-expand-4294967296.img.meta.tmp; fi; sleep 1; done;" > test.sh && chmod +x test.sh && ./test.sh &
[1] 25183

Detach the volume
Expand the volume. The size should be the same as that is used in step2.

Validation: Partially Failed
Steps to test:

Create a 1GB volume and attach it.
Format the volume, mount it to a directory
Write data to volume, and compute its checksum
Enter the replica data directory and run the following command for each replica:
 echo "while true; do if [ ! -e \"volume-snap-expand-2147483648.img.meta.tmp\" ]; then mkdir volume-snap-expand-2147483648.img.meta.tmp; fi; sleep 1; done;" > test.sh && chmod +x test.sh && ./test.sh &
Detach the volume
Expand the volume to be 2GB, (volume expansion should be suspending)
7.In Volume list page, click on Cancel Expansion
A warning message state should appear  (PASS)

This operation is used to cancel the expansion if the volume cannot complete the expansion and gets stuck there. Once the expansion is complete, it cannot be rolled back or canceled

Cancel volume expansion, expansion should work, volume should be attached and size should match the original volume size 1GB (PASS)
Attach the volume again, and check data checksum, data checksum should match. (PASS)

Failed tests

After volume create, in Volume list page and Volume Details page, should be disabled by default

Volume Details page Cancel Expansion button don't work

Cancel Expansion button should exist on

We don't need batch cancel button for now, it's not a commonly used the button. We can add it later if necessary.

Validation: PASSED


Add ArmResourceHead operation template

All changed packages have been documented.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/283/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/283/


PLAT-64690: Disable scroll animation with 5-way navigation

Checklist

[x] I have read and understand the contribution guide

[x] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[x] Documentation was added or is not needed

[ ] This is an API breaking change

Issue Resolved / Feature Added
Fix stuttering issue by disabling animation with 5-way navigation
Resolution
Allow animate as a prop and use it to branch whether it animate or not
Additional Considerations
Not pass to ui/VirtualListBase at this time since it doesn't have any logic relate to animation
Links
PLAT-64690
Comments
Enact-DCO-1.0-Signed-off-by: Yeram Choi<EMAIL_ADDRESS>

In order to isolate the behavior change to moonstone, I've updated the ui components to use noAnimation instead of animate so that the default behavior is still animated scrolling for non-moonstone targets. I've left the prop as private because I expect that we'll attempt to improve the performance so that we can re-enable animation by default without altering public APIs.


[flutter_tools] Introduce the new flutter create-plugin command.

Description
This PR introduces a new flutter create-plugin command.
NOTE: the original discussion is to add a subcommand for flutter create. We can't do that because flutter create still needs to work. So I ended up making it a separate command as flutter create-plugin.
For now, this new flutter create-plugin can be run on the same directory to add platforms. After landing this, I'll create another PR to disable this functionality and add a flutter add-platforms command. See https://github.com/flutter/flutter/issues/59493
CC @amirh
This PR didn't add a deprecation message for the flutter create -t plugin yet, I'll create a separate PR to do that.
Related Issues
Fixes: https://github.com/flutter/flutter/issues/70873
Tests
I added the following tests:
create_plugin_test.dart
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[x] Yes, this is a breaking change. If not, delete the remainder of this section.

[x] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[x] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[x] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Talked to @csells offline and he doesn't like the flutter create-plugin name. I'm going to close this PR for now and we'll discuss this further offline.


⚠ AZ-Launcher has degraded performance

In d021dde, AZ-Launcher (https://az-launcher.nz/) experienced degraded performance:

HTTP code: 200
Response time: 1483 ms

Resolved: AZ-Launcher performance has improved in 3b4644a after 11 minutes.


[FLINK-10126] There should be a Scala DataSource

What is the purpose of the change
This pull request add some APIs which exist flink-java's DataSource into flink-scala DataSet
Brief change log

Add some APIs which exist flink-java's DataSource into flink-scala DataSet
Added test case DataSetITCase

Verifying this change
This change is already covered by existing tests, such as DataSetITCase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @zentol and @GJL

This fix seems to be a workaround for the real issue that the Scala API misses a DataSource type.
I would suggest to see if that can be introduced without breaking the API.

hi @StephanEwen , Before providing the API directly, I evaluated the implementation of a peer DataSource in the flink-scala module. However, It will break the existing source API. It was found that ExecutionEnvironment.scala released some readXXX APIs that returned DataSet.scala. Inside these APIs, we got a DataSource (flink-java) and then called wrap(source) to convert it to a DataSet.scala. Considering that there are only two or three APIs exposed in the DataSource.java (one of which withParameters has been exposed through the DataSet), I chose to expose the API in the DataSource to the DataSet.scala with minimal changes and without breaking the current API. In fact, in this JIRA's description, reporter is more concerned with getting the APIs provided by DataSource.java.

@tillrohrmann and @pnowojski any suggestion?

I agree with @StephanEwen that the current approach is not very clean.
I'm wondering whether introducing a Scala DataSource extends DataSet and changing the return type of ExecutionEnvironment#createInput from DataSet to DataSource would really be API breaking. All existing programs should still compile since we would extend DataSet. What do you think @yanghua?

Hi @tillrohrmann , Perhaps we have a deviation in the understanding of "break", and my "break" is to change the signature of the API. Because I don't know if the API checker verifies its signature. However, as you said, it does not force the signature of the verification method to be exactly the same as before. I will try to introduce DataSource to see if it works.

Hi @tillrohrmann I have updated this PR based on your suggestion, however the japicmp-maven-plugin reported incompatibility issue METHOD_RETURN_TYPE_CHANGED . It seems the method's signature would been verified. What's your idea?

@tillrohrmann Any further suggestion?

I'm closing this for now because we don't really want to add new features to the DataSet API because it is supposed to be subsumed at some point by the DataStream API.


Interesting...

Interesting.... Any idea how to bypass Easy Anti Cheat?

Can sources be added for UE4 -4.26 ~ 4.27?

@YMY1666527646 commented on Jan 12, 2022, 8:37 PM GMT+1:

Can sources be added for UE4 -4.26 ~ 4.27?

Yes, read the UE introduction and guide on this repository on how to make a similar cheat in any Unreal Engine game.

@WeLoveGaming commented on Jan 11, 2022, 10:53 PM GMT+1:

Interesting.... Any idea how to bypass Easy Anti Cheat?

Bypassing it is not easy and I am not familiar with anti-cheats, but I can give you a small tip: don't try to bypass EAC unless you already know every other concept of game hacking.

@WeLoveGaming commented on Jan 11, 2022, 10:53 PM GMT+1:

Interesting.... Any idea how to bypass Easy Anti Cheat?

Bypassing it is not easy and I am not familiar with anti-cheats, but I can give you a small tip: don't try to bypass EAC unless you already know every other concept of game hacking.

Yes, I know how it works, I just have no idea how to make it to EAC doesn't block injections.

@WeLoveGaming commented on Jan 11, 2022, 10:53 PM GMT+1:

Interesting.... Any idea how to bypass Easy Anti Cheat?

Bypassing it is not easy and I am not familiar with anti-cheats, but I can give you a small tip: don't try to bypass EAC unless you already know every other concept of game hacking.

Yes, I know how it works, I just have no idea how to make it to EAC doesn't block injections.

@WeLoveGaming commented on Jan 14, 2022, 6:27 PM GMT+1:

@WeLoveGaming commented on Jan 11, 2022, 10:53 PM GMT+1:

Interesting.... Any idea how to bypass Easy Anti Cheat?

Bypassing it is not easy and I am not familiar with anti-cheats, but I can give you a small tip: don't try to bypass EAC unless you already know every other concept of game hacking.

Yes, I know how it works, I just have no idea how to make it to EAC doesn't block injections.

You can look up different injection methods, you will probably have to make your own undetected injector, and correct me if I am wrong but EAC is a kernel anti-cheat so you will have to bypass that somehow first because you can't open a handle to the process while EAC's kernel is running.


Problem with OpenCV - Initializing TensorFlow variables takes very long

For some reason, the variable initialization of TensorFlow takes a very long time when OpenCV is imported (import cv2). Running the operation tf.init_all_variables() takes less than 1 second without OpenCV, but as soon as I import cv2 this takes 22 seconds (!). See attachment for the simple - Hello World - code that I run. hello_world.txt
This problem occurs in TensorFlow 0.6 and 0.7. During the initialization the CPU load is 100%, GPU is idle but all available memory is initialized by TensorFlow.
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcuda.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcurand.so locally
0.7.0

TensorFlow is installed in an virtualenv using pip after installing CUDA Toolkit 7.5 and CuDNN v3. My OpenCV installation is from sources (GitHub, master), so this is version 3.1+. Some possibly relevant installation parameters are WITH_CUDA=ON, WITH_CUBLAS=OFF.
I am using a powerful machine running Linux Mint, 32 CPU cores and an Titan X.

Mysterious. I don't know what cv2 does, but since you are building it with Cuda, it could be preallocating GPU memory (similar to what tensorflow does). The symptom is decidedly odd though. @zheng-xq, do you have an idea?

Oke, I just recompiled OpenCV 3.1 without CUDA support (i.e. WITH_CUDA=OFF) and this solved the problem. TensorFlow initialization with and without import cv2 is now fast. So it seems like there is an incompatibility with TensorFlow and OpenCV compiled with CUDA. Like you said, probably both TF and OpenCV are initializing GPU memory at the same time which causes trouble?

TF takes most of the GPU memory upon its initialization. You can set the following session config to use smaller amount of memory:
config.gpu_options.per_process_gpu_memory_fraction
If that doesn't work, then it is useful to look at some profiling results and see where the cycles are spent.

Closing for now.  For better or worse, TensorFlow assumes it owns the whole GPU, so other libraries trying to use the GPU at the same time will cause issues.


//INCLUDE requiring full path

This works...
//INCLUDE /Users/ilanpillemer/git/model-language/src/main/kotlin/Command.kt

This fails...
//INCLUDE Command.kt

with this error
[[kscript] [ERROR] /var/folders/cw/6_jr6ms12wgchzxj9s9kq5sm0000gn/T/tmp11233444675737493427.tmp/Command.kt (No such file or directory)]

Indeed @kenyee, same here.


github workflow

#4  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite


[logging] add data_display for logging errors

data_display now allows for using the display functionality rather than
serialization for logging data.

@bors-libra r=andll

:pushpin: Commit b41edf5 has been approved by andll

:hourglass: Testing commit b41edf54926b5adef96b22c262dae4a09543bf9a with merge 8d5da4162783f21380d2cc2c6ecd5f2bcc96a7ef...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: andll
Pushing 8d5da4162783f21380d2cc2c6ecd5f2bcc96a7ef to master...


Jira: Merging next to main for release 3.2.4

Updated release notes

The integration test is failing because of timeout


Documentation or Blog for monitoring LLM Stack using HyperDx

Hello team!
Background
I am Patcher and one of the maintainer of OpenLIT. OpenLIT is an OTel native LLM tool that allows developers to configure complete LLM Observability in single line of code. OpenLIT has also been used in the official OTel Blog about LLM Observability.
Problem
Currently their is no official HyperDx guide either as a Doc or Blogpost that can guide users towards LLM Observability with HyperDX.
Proposal
Since our SDK generates OTel traces and Metrics, we have also tested it works very nicely with HyperDX. I was looking to add a guide that can help HyperDX users monitoring their LLM applications.  Can someone from the team help on how can I assist in developing a guide as I don't see any official process on contributing a doc or a guide?

Hi @patcher9! Would love to collaborate on a guide, we haven't finished migrating the docs workflow into an OSS repo yet (it's still tied to our cloud hosted stuff). In the interim we've accepted outside contributions in markdown format. I'm happy to collaborate over email to chat more about it, feel free to ping me at<EMAIL_ADDRESS>

Published! Thank you :)
https://www.hyperdx.io/docs/install/openlit


MonoTorrent V2.0 (API break edition)

[ ] Update ITracker so FailureMessage/WarningMessage/State are distinct variables for Announce and Scrape requests.
[x] Remove the 'LastAnnounceSucceeded', and related, API from TrackerManager. These are per-tier and so we don't want to expose them on TrackerManager itself.
[x] Remove IConnection and rename IConnection2 to IConnection.
[x] Remove ITrackerManager and rename ITrackerManager2 to ITrackerManager.
[x] Remove CancellationToken from AnnounceParameters and ScrapeParameters. Add it to the methods like a normal API would do!
[x] Mark AnnounceParameters and ScrapeParameters as sealed.
[x] Make DiskManager and IPieceWriter use async methods all the way through. In addition make it so that reads/writes are now locked on a per-file basis, not one read overall. Note: We also need to ensure that the code to ensure all pending writes complete before reading the data for a hash check is not broken by this.
[x] Fix 'RawTrackerTier' to not be terrible. This is not a good way to access the urls stored in the torrent: var magetLink = new MagnetLink (torrent.InfoHash, torrent.Name, torrent.AnnounceUrls.SelectMany (t => t.ToArray ()).ToList ());
[x] Queue up tracker announces so if you start 100 torrents at a time we only issue a few announces concurrently. Do the same for StopAllAsync.
[ ] Investigate a better return type for StopAllAsync so progress can be tracked for pending announces etc.
[ ] Queue up announces when starting 100s of torrents. Similarly for HashCheckAsync etc. Maybe put torrents in a 'Pending' state which becomes 'Starting' eventually? Will need to be able to tell when the startup tasks are done (first announce etc). Those are currently run when the mode is 'Downloading' so it's not easy to check this.
[ ] Investigate a better StopAllAsync which reports progress is the announces are queued.
[x] ~Change how the PiecePicker is registered so that it does not get clobbered by StartAsync, and also so you can set one when in MetadataMode and not crash things!~ This was not API breaking!

hello alan, long time and I was curious to discover that the library is sill actively developped WOW!
I am currently working on quarkus (java) cxf (soap support) extension and I use vertx (express js patter with event loop).
Maybe it can be better for performance to have a such pattern with verticles and signe event loop.
Anyway, it was just an idea. I just see that this is just still active and very happy. cheers olivier (if you remember me we code together on the firs DHT support 10 years ago )


Adding new requirement for secret rotation

bug identified where secret rotation can only be run once per month

@PPacentMSFT : Thanks for your contribution! The author, @mattbriggs, has been notified to review your proposed change.


AMBARI-24152 Ambari Workflow Manager (wfmanager) sends plaintext content over API. JSON is expected

We have changed the REST API response format from text to JSON and had tested this manually.

LGTM

LGTM

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3028/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3029/
Test FAILed.
Test FAILured.


Select fields misaligned

Select fields are aligned differently in chrome than in firefox(latest) and ie(11),
I have created a form with inline text that looks fine in chrome, the fields use default materialize padding and margins:
ie:

ff:

chrome (as intended):

sample code:

			I'm looking to

				 select
				<select id="department" name="department" onchange="$('.form-part-2').slideDown();
										$('.search-container').addClass('started');">
					<option selected disabled value="all">please select</option>
					<option value="buy">buy</option>
					<option value="rent">rent</option>

				</select>

Can you recreate this in a codependent or something? It's pretty hard to debug without CSS

Should be fixed 28b119a. Let me know if this works

@acburst thanks this is a big improvement,
I have adjusted one additional adjustment to the margin as its a bit lower than other content
.input-field.inline {
 margin-top: 0.4rem;
}


Parser always unwraps spans, no matter the rules.

I have HTML like this:

Hi how are you

The editor always unwraps these span elements like this:

Hi how are you

This also happens with spans that don't have the "begin" or "end" attributes.
This happens with all premade rulesets. (like 'advanced_unwrap' or 'simple'). It also happens if I create a custom rule for spans like this:
span: 1,
If I substitute the span tags with 'em' or 's', the rules apply like I want to. I don't get why spans are treated differently than 's' or 'em' elements.

Will it help if you set this option to false?
https://github.com/Voog/wysihtml/wiki/Configuration#cleanup

That actually fixed it, thanks!
I think that option shouldn't cleanup spans with attributes, though. Especially if you allow those attributes in the rules.

You are correct. The current cleanup code is a bit ambitious, not really counting for attributes. Will try to look at it if I get more time to focus.
I think it is this line where the problem is, if anyone would like to give it a try:
https://github.com/Voog/wysihtml/blob/master/src/dom/parse.js#L223


feat: New function Client.flush

Codecov Report

Merging #250 into master will decrease coverage by 14.18%.
The diff coverage is 79.16%.

@@             Coverage Diff             @@
##           master     #250       +/-   ##
===========================================
- Coverage   81.69%   67.51%   -14.19%
===========================================
  Files          32       29        -3
  Lines        2748     2435      -313
  Branches      443      405       -38
===========================================
- Hits         2245     1644      -601
- Misses        340      656      +316
+ Partials      163      135       -28

Impacted Files
Coverage Δ

sentry_sdk/integrations/aws_lambda.py
0% <0%> (-19.63%)
:arrow_down:

sentry_sdk/integrations/atexit.py
92.85% <100%> (-0.48%)
:arrow_down:

sentry_sdk/transport.py
82.52% <66.66%> (+1.61%)
:arrow_up:

sentry_sdk/client.py
79.83% <85.71%> (-3.64%)
:arrow_down:

sentry_sdk/worker.py
93.33% <85.71%> (+5.83%)
:arrow_up:

sentry_sdk/integrations/celery.py
0% <0%> (-90%)
:arrow_down:

sentry_sdk/integrations/pyramid.py
0% <0%> (-85.57%)
:arrow_down:

sentry_sdk/integrations/rq.py
0% <0%> (-80%)
:arrow_down:

sentry_sdk/integrations/flask.py
0% <0%> (-77.89%)
:arrow_down:

sentry_sdk/_compat.py
53.33% <0%> (-26.67%)
:arrow_down:

... and 23 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 db7d505...89b34df. Read the comment docs.


Add Anaconda into the pre-built notebook pipeline

Draft of including the Anaconda notebook into pre-built pipeline
Related-to: https://github.com/opendatahub-io/notebooks/issues/110

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

Hi @Xaenalt, I checked out your PR and I would like to add some comments before you officially open it.
First, I observed that the way that you approached building the anaconda notebook image is s2i, on this repo we don't use this way for building the notebooks, but on the contrary, we are doing it by giving all the instructions in a Dockerfile, as well as, each notebook inherits functionalities from the previous one.
The second thing that I saw based on your requirements.in file is that you need almost the same packages as they are on the data science notebook. So, in that case, you can base your image from the data science notebook and add on that file only the packages that don't exist on the previous chain notebooks.
So, in order to comply with this repo, you need to create a directory under jupyter folder with a name like for example ${NOTEBOOK_NAME}/ubi8-python-3.8  with a Dockerfile that you install the Anaconda package manager and whatever else you need in order to be functional, (for your reference check the other notebooks based on the datascience image (PyTorch, TensorFlow)), the requirements.in file and the requirements.txt that will be generated after giving the correct  recipe into makefile that should be something like the following:
.PHONY: jupyter-${NOTEBOOK_NAME}-ubi8-python-3.8
jupyter-${NOTEBOOK_NAME}-ubi8-python-3.8: jupyter-datascience-ubi8-python-3.8
	$(call pip_compile,bootstrap/ubi8-python-3.8,\
		base/ubi8-python-3.8/requirements.in jupyter/minimal/ubi8-python-3.8/requirements.in jupyter/datascience/ubi8-python-3.8/requirements.in jupyter/${NOTEBOOK_NAME}/ubi8-python-3.8/requirements.in,\
		jupyter/${NOTEBOOK_NAME}/ubi8-python-3.8/requirements.txt)
	$(call image,$@,jupyter/${NOTEBOOK_NAME}/ubi8-python-3.8,$<)

Finally, test the changes locally, by running the $ make jupyter-${NOTEBOOK_NAME}-ubi8-python-3.8 from the terminal

I think you will not have any issues if you try to build your notebook from the registry.access.redhat.com/ubi8/python-38:latest instead of registry.access.redhat.com/ubi8/s2i-base

Right, I built a customized python-38 image, so if I have to use the original, it'll be a lot of uninstalling existing packages, and I had to patch the builder script to use Anaconda for requirements.in rather than pip. Anaconda essentially is a replacement for pip

Hey @Xaenalt do you still need this PR?

Ok, that should handle the base image, hopefully

/unhold

Ok, built and tested this locally, we should be all set!

I'd like to do a bit of automation on updating the environment.yml stuff as packages are updated, similar workflow to the pipfile.lock stuff

@atheo89 how does this look? Tested locally and it works, need to get it into the image push pipeline, and then I can update the dashboard bits to enable it in RHODS

https://github.com/openshift/release/pull/39863 for the ci pipeline

openshift/release#39863 for the ci pipeline

Added a comment, It needs a small fix

I tested the notebook locally and it looks good. But, I observed significant differences in the Elyra extensions between this notebook and the data science one. On the datascience we enable only the following: elyra-pipeline-editor-extension, elyra-python-editor-extension, elyra-code-snippet-extension , as shown in the image below: Data Science Notebook  Anacoda Data Science
I am not sure if we need all the extensions enabled in this anacoda notebook or not, or if it is due to misconfiguration. cc @harshad16

I kind of just installed the packages based on what was in the requirements.in from the regular datascience one. There wasn't a version of the s3 browser, but I tried to keep close to the datascience one. If I need to change stuff to remove other plugins I can

Oh, I see, the ubi8-python-3.8 suffix is hardcoded... hmmm

Ok, there we go, make deploy8-jupyter-datascience-anaconda-python-3.8 should now work as expected

/retest

@harshad16 it's a custom builder image, I don't think it'll have those in there if I don't add them in will it?

@harshad16 it's a custom builder image, I don't think it'll have those in there if I don't add them in will it?

In base-anacode image, it seem like we are using s2i image as base https://github.com/opendatahub-io/notebooks/blob/8f9343fa209e2160d0ec385f00bde410bb6a5c7d/base/anaconda-python-3.8/Dockerfile#L3
it will have s2i script with it.

Ok cool, I wanted to provide that as a compatible base image in case we have other stuff that might need it

Ok cool, I wanted to provide that as a compatible base image in case we have other stuff that might need it

ack, my perspective to remove the additional s2i scripts, if we are not including new bits in it,
is that we would need to maintain it as well, where as the upstream s2i-python is already doing that.
If we are in for maintain these script, we can get this in.

Right, makes sense, less maintenance on this the better

So, any other changes we need on this?

Hey @Xaenalt! What about the changes proposed by Harshad?
Another thing is, once the PR on CI get merged (I added my review there) you can run on this PR retest to test the anaconda image.

I removed the other files per Harshad's suggestion, and cool, hopefully the CI stuff will merge soon

/retest

Hmmm, it's getting this error:
  * could not run steps: step jupyter-datascience-anaconda-python-3.8-pr-image-mirror failed: "jupyter-datascience-anaconda-python-3.8-pr-image-mirror" pre steps failed: failed to create or restart jupyter-datascience-anaconda-python-3.8-pr-image-mirror-opendatahub-io-ci-image-mirror pod: unable to create pod: Pod "jupyter-datascience-anaconda-python-3.8-pr-image-mirror-opendatahub-io-ci-image-mirror" is invalid: [spec.volumes[9].name: Invalid value: "jupyter-datascience-anaconda-python-3.8-pr-image-mirror": must not contain dots, spec.containers[0].volumeMounts[4].name: Not found: "jupyter-datascience-anaconda-python-3.8-pr-image-mirror"]

Maybe a typo in my CI tests? I'm not sure where though, it looks pretty identical to the others, how does it get that volume name?

@harshad16 any chance you can help me figure this one out?

/retest-required

Blah, I messed up the package list, regenerating it

/test jupyter-datascience-anaconda-python-3.8-pr-image-mirror

/test jupyter-datascience-anaconda-python-3-8-pr-image-mirror

Cool, ok now with the updated test, as soon as I get the updated pkg list we should be good to go hopefully

Great, overall /lgtm, the only thing that I don't understand is why the failed anaconda job with the wrong naming is still there
Few more next steps remaining to go in downstream:

Open a PR that points to the 2023a branch which cherry-picks this commit after its merging to the main
Add configurations on the CI release repo to build and push the image for the 2023a -> https://github.com/openshift/release/blob/master/ci-operator/config/opendatahub-io/notebooks/opendatahub-io-notebooks-2023a.yaml
Sync the downstream main branch with the new commit
Add configurations on the CI release repo to build and push the image for the 2023a for the downstream main and 2023a branches config as well https://github.com/openshift/release/tree/master/ci-operator/config/red-hat-data-services/notebooks

/unhold

Ok, package list should be all set

One other thing needs fixing on the pkg versions
/hold

/test jupyter-datascience-anaconda-python-3-8-pr-image-mirror

/retest-required

/retest-required

/test notebook-jupyter-trustyai-ubi9-python-3.9-pr-image-mirror

/test jupyter-datascience-anaconda-python-3-8-pr-image-mirror

/test jupyter-datascience-anaconda-python-3-8-pr-image-mirror

/retest-required

Hmmm, not sure what's going on with e2e tests, seems like it timed out waiting for the notebook?

/retest-required

Hmmm, not sure what's going on with e2e tests, seems like it timed out waiting for the notebook?

Hey Sean, maybe the problem is with the naming of the notebook into the .test-% recipe.
BTW how do you run the recipes with your customization (deploy, undeploy & test) ?
aka make deploy8-???-anaconda-python-3.8

Since the ubi8 part of the path is hardcoded, I made a separate test target, https://github.com/opendatahub-io/notebooks/pull/28/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52R225 and just reused the existing test flow.  My cluster is gone though right now, so I don't have a way to run that part

The issue seems to be the image size, the image is a little bigger, and the pod takes a little more time than timeout to get into a ready state. Testing a few aspects will share.

Updated timeout, fingers crossed!

Updated timeout, fingers crossed!

i dont see any change, can you share where this change was made ?

That's weird, it didn't update on the PR

Since my cluster is thoroughly dead, only other change that comes to mind is the workdir change that could cause this

/retest-required

Just to double check, I ran it with the args from the statefulset, worked, so not sure what's failing e2e

/retest-required

/retest-required

looks like we need speed up a bit for this PR, since https://github.com/openshift/release/pull/39863 has been merged with testcases which does not have source code available yet.

Agreed, it's not clear why e2e is failing on this though

Woohoo 🎉 all the tests passed!
Shall we merge this now, or you planning to enhance the similar recipes deploy8/undeploy and deploy8-anacoda/undeploy-anacoda etc into one?

I'll add those in later, for now let's just merge this and I'll make the other enhancements in a separate PR

/unhold


PyLint behaves strange

First, I thought this was an issue of my code and I opened this question on stack.
Now I am beginning to feel something has gone really wrong with my VS Code pylint module. I see errors in really strange places, e.g. in docstrings and other places where they don't make sense at all.
Environment data

VS Code version: 1.30.1 (user setup)
OS: Windows_NT x64 6.1.7601
Extension version (available under the Extensions sidebar): Python 2018.12.1
Python version (& distribution if applicable, e.g. Anaconda): 3.7.1 64bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A (Develop Mode)
Relevant/affected Python packages and their versions: most

Expected behaviour
There are many different errors like this, e.g.:
from lbsnstructure.lbsnstructure_pb2 import lbsnPost
def geoaccuracy_within_threshold(post_geoaccuracy, min_geoaccuracy):
    """Checks if geoaccuracy is within or below threshhold defined"""
    if min_geoaccuracy == lbsnPost.LATLNG:
        allowed_geoaccuracies = [lbsnPost.LATLNG]
    elif min_geoaccuracy == lbsnPost.PLACE:
        allowed_geoaccuracies = [lbsnPost.LATLNG, lbsnPost.PLACE]
    elif min_geoaccuracy == lbsnPost.CITY:
        allowed_geoaccuracies = [lbsnPost.LATLNG, lbsnPost.PLACE, lbsnPost.CITY]
    else:
        return True
    # check post geoaccuracy
    if post_geoaccuracy in allowed_geoaccuracies:
        return True
    else:
        return False

This program works, I can run it and there are not problems.
Actual behaviour
.. however, pylint highlights several errors:

Furthermore, if I hover over these errors, they don't make sense, i.e. hovering over the docstring 'geoaccuracy' gives error suggestion towards class lbsnPost.

The next time I open VS Code, it sometimes looks different (somehow randomly):

Steps to reproduce:
I do not know how to reproduce this. I just open the containing folder and it shows up like this.
Note that if I check the class in pylint directly through the console, I do not get these errors.
Logs
No output, execution works fine.
##Additional
I already tried:

using global pylint module instead of local by setting
"python.globalModuleInstallation": true - no difference
explicitly setting path to python - no difference
also followed some suggestions from here without a noticeable difference
if I do Shift+P: Python: Run linting marked code errors remain exactly the same

I think I have found a solution to one of my problems:
Somehow my line endings CRLF were problematic - the errors highlighted existed, but were highlighted in the wrong places/lines. I could correct the issue with:

changing CRLF line endings to LF (Unix) at the bottom right corner
adding "files.eol": "\n" to user settings
saving and reloading file
(additionally, installed line endings extension to display characters)

Now I still have those errors, but they're at least highlighted in the right places.

Now to the actual problem of wrong code errors: I've seen reports that pylint in visual code marks ProtoBuf enums because it can't detect those in protobuf compiled classes. So this is the second part of this bug report, perhaps there's a way to fix this? I will try this package

Ok, solved my second problem, too:

installed pylint-protobuf package (pip install pylint-protobuf)
added "python.linting.pylintArgs": ["--load-plugin","pylint_protobuf"] to User Settings in VS Code

No errors!


2.1 alpha: error handling of inserting projects

(Is this the right place for Studio 2.1 alpha issues?)
Adding a project by dragging it to "My Apps" silently fails if the evothings.json has an "app-dir" defined but that directory does not have an index.html file.
As a side effect doing this makes all the existing projects in "My Apps" invisible (project list building probably fails). Adding the project again with index.html does solve this and all the old projects become visible again.

Ah! Yeah, I also noted "silent failing" with other inconsistencies in the evothings.json file - so the first "fix" is to add proper error reporting at least.

Added error handling in this commit: https://github.com/evothings/evothings-studio/commit/28a4ef73b93d3bb24ab2a624e1d1c5a1f95db4e5


Project publishing

Hi, thanks for library it seems to be great solution for Fresco users.
But there is no way to include it as dependency only import as module and reference it in gradle file.
Please deploy library somewhere.

#14


Use Popen(cwd=x) instead of calling os.chdir

We encountered a bug running a web server under gunicorn where multiple threads were active. Because webassets uses chdir it was affecting the working directory state across all threads, which broke things quite badly. Instead of using chdir, this pull request passes the desired working directory to sass via  Popen.
There are other places in webassets which use os.chdir, which could do with being avoided, but we just fix the one where I could produce crashes for now.

Thanks, very good catch.


Context not cleared on re-deploy

The context is not cleared during a full re-deploy.
To replicate put the following code in a function block
last_time = context.get('last_time');
node.log("last time: " + last_time);
context.set(last_time', new Date());
Run the flow and then perform a full re-deploy.  The first time through after re-deploy, last_time should be undefined, but in fact it gets the value from the context.set before the re-deploy.
Running on Pi (Jessie)
Node-RED version: v0.13.3
Node.js  version: v0.10.29
Linux 4.1.19-v7+ arm LE

I am also seeing the problem with a user defined nodes (html and js files in the nodes folder under my flows), but in this case the context is also not cleared on deploy if I use the old method,
context.x = x
x = context.
In the case of function blocks the context is cleared when using the old method.

In the user defined node it will depend where/when they are attaching themselves to context - (and indeed depending how it's being done it may or may not be the same context as functions use). If it's in the general setup but before the actual node instance definition then it will be  global across that node definition (and so not cleared as that is only set up at start time) - If inside the actual node instance then again it depends which context they have attached to. It's up to the node author to do the right thing.

Thanks Dave, This is my first user defined node and I have just said
var context = node.context
in on('input', ..) and used it.  Is that a different context to that available in a Function node?

For a custom node, you must access context as described in the docs: http://nodered.org/docs/creating-nodes/context
var context = node.context();

IE, call the context function.

Sorry, I mistyped in my previous comment.  I think I have what is described in the docs, I have:
module.exports = function(RED) {
function Func(config) {
RED.nodes.createNode(this,config);
var node = this;
this.on('input', function(msg) {
var context = node.context();
...
The context appears to be working perfectly except that it does not get cleared on re-deploy, when using either the old or the new syntax.

Fixed in the imminent 0.14.0 release


Certificates.cc.org redirect?

Can we redirect certificates.creativecommons.org to creativecommons.org/certificates so that it's templated with the CC.org theme?

Alternatively, we can use the Summit theme....

certs.cc is going to be the BuddyPress site I think, and we have the theme for that. If we want main-site-themed information at /certs we should certainly create that there but we shouldn't redirect.

So creativecommons.org/certificates can be a CC Certs landing page? What do you think?

Basically looking for a place to put a marketing page.

Is there any way to migrate the current info on the cert.cc.org page to a diff web page labeled as "archive of CC Cert development" or something like that? The diff web page could be creativecommons.org/certificates. That way we can keep marketing updates, registration and eventually modules all on the certs.cc.org page, using it as a one-stop shop.

@little-wow : I put in possible urls next to the different content items for the Cert page. LMK what you think.