Dockerfile for DAIN

It would be nice if there was Dockerfile with all the dependencies.

Hey buddy, why not join us by adding a dockerfile thanks. ;-)

Haha, I am trying to do that using a PyTorch docker as a base. What are the critical dependencies in your opinion? I know there is an environment yaml. Is everything in there needed for DAIN to build?

Any update on this @micmelesse ?
I am not a docker proffesional, but maybe I can help you!

Hi @betegon, I am kind of stuck. I created a branch in my fork with a dockerfile. It creates a conda environment using a cleaned up yaml file and then builds the packages using this command.
docker build --network=host -t dain .
You can enter the container with
docker run --ipc=host -it --rm --network=host --runtime=nvidia -w /DAIN
This works, I can activate the conda environment with
source activate pytorch1.0.0
however when I try to run it
python demo_MiddleBury.py
I get an error
'ImportError: /opt/conda/envs/pytorch1.0.0/lib/python3.6/site-packages/filterinterpolation_cuda-0.0.0-py3.6-linux-x86_64.egg/filterinterpolation_cuda.cpython-36m-x86_64-linux-gnu.so: undefined symbol: __cudaRegisterFatBinaryEnd
Looking around this seems to be because the application is finding the cuda of the container(10) and not the cuda in the conda environment (9).
I tired using an older nvidia pytorch container with cuda 9 but that doesnot detect my gpu or the gpu in a aws p2x instance.
@baowenbo Any ideas about this?

@micmelesse An the end you managed to dockerize the github, how could I use it?

@ZurMaD No, I have not. I just don't have the time to deal with it.

Thanks for answer, I'll try next two weeks.

Hey there I actually made this docker file its over at my public fork of DAIN: https://gitlab.com/Miscoriadev/dain-fork-for-docker
It even works with with compute power 7.5 (Had to get my RTX 2080 Ti working)

@Praevalens Thank you for sharing

@Praevalens This is is awesome.  Thank you.

Another repo support DAIN with docker, together with ESRGAN and DeOldify. Hope it helps
https://github.com/iamyb/DAIN_ESRGAN_DeOldify


Fix typo look instead of loot

Pre-Review Checklist for the PR Author

[ ] Code follows the coding style of CONTRIBUTING.md
[ ] Tests follow the best practice for testing
[ ] Changelog updated in the unreleased section including API breaking changes
[ ] Branch follows the naming format (iox-#123-this-is-a-branch)
[ ] Commits messages are according to this guideline

[ ] Commit messages have the issue ID (iox-#123 commit text)
[ ] Commit messages are signed (git commit -s)
[ ] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[ ] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[ ] Relevant issues are linked
[ ] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[ ] Assign PR to reviewer

Notes for Reviewer

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior

[ ] Each unit test case has a unique UUID

[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes TBD

@flynneva You can re-use https://github.com/eclipse-iceoryx/iceoryx/issues/1295 for the next release validation for this PR so you do not need to create an extra issue.

@flynneva Thanks for the heads-up! We will solve this as part of this documentation PR: https://github.com/eclipse-iceoryx/iceoryx/pull/1434, so this one can get closed.


Cannot delete personal notes

Hey people!
Unfortunately, I cannot delete the personal notes, which I wrote. Hope there is an easy way to solve this bug

What does it do ? does the page simply refresh ? or do you get a message ?

It just refreshes the page, but the personal note is still there

really weird i compared my files to the default one no difference you are using default files or modified ones ?

No modified files at all, I am using the github files (1.8.git version). The only file I have modified is the ShowSearchPage.php to make the player and planet search work

Works for me - Closed.


Add the ability to be able to rename an output from a workflow

Workflow output names are currently hardcoded to whatever is specified by the last analysis.

Dup of #115


NuGet/NugGet typo

@moshegutman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@moshegutman Thanks for the fix!


Discussion: Deprecate WeakHostSend, WeakHostRecieve and DHCPClient

The WeakHostSend, WeakHostRecieve and DHCPClient resources all configure properties on the *-NetIPInterface. They are "single purpose" resources that result in a lot of duplicated code/function. E.g. the resources are practically identical (duplicated code, tests etc).
I'm currently implementing a more generic NetIPInterface resource that can be used to configure properties on the *-NetIPInterface as part of #355. I'm excluding the WeakHostSend, WeakHostRecieve and DHCPClient properties as they are covered by these other resources.
But it seems to me to be non-trivial duplication of function/tests. This would reduce the amount of code and tests needing to be maintained and also simplify configurations - but of course would be a BREAKING CHANGE.
For example, the following current config:
Configuration Example
{
    Import-DscResource -Module NetworkingDsc

    Node localhost
    {
        DhcpClient EnableDhcpClient
        {
            InterfaceAlias = 'Ethernet'
            AddressFamily  = 'IPv4'
            State          = 'Enabled'
        }

        WeakHostSend DisableWeakHostReceiving
        {
            InterfaceAlias = 'Ethernet'
            AddressFamily  = 'IPv4'
            State          = 'Disabled'
        }

        WeakHostSend DisableWeakHostSending
        {
            InterfaceAlias = 'Ethernet'
            AddressFamily  = 'IPv4'
            State          = 'Disabled'
        }
    }
}

Would be changed to:
Configuration Example
{
    Import-DscResource -Module NetworkingDsc

    Node localhost
    {
        NetIPInterface ConfigureNetIPInterface
        {
            InterfaceAlias = 'Ethernet'
            AddressFamily  = 'IPv4'
            DHCP          = 'Enabled'
            WeakHostSend = 'Disabled'
            WeakHostRecieve = 'Disabled'
        }
    }
}

@tysonjhayes , @johlju - what are your thoughts on this?

I suggest deprecate the resource, merging them into the new resource - and in a later PR (maybe when doing another breaking change) remove them.
We had a similar scenario in SqlServerDsc a while back - I waited a couple of released, then I removed them as a breaking change.

Cool. That what I was thinking.

Bumping to keep this open for now.

I will plan to implement this change over the next few weeks.


SAK-43113 Clean up ItemGradingData in Samigo

https://jira.sakaiproject.org/browse/SAK-43113

Approved, as last comments are minor.

Let's hold off until we have resolved Samigo regressions

Ok, will do.

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ samigo/samigo-api/src/java/org/sakaiproject/tool/assessment/data/dao/grading/ItemGradingData.java  -3

See the complete overview on Codacy


Acronym cleanup

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.091%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1133

Relevant Lines:
1489

💛  - Coveralls


Request for Additional Functionality: Removing Remediated Items from Scan Uploads

It would be great if as new Nessus scans are run and uploaded as part of an Engagement, they could be compared against existing results and items that have been remediated no longer appear in results and are marked as mitigated along with the date.

Latest commit 5794b09 takes care of this feature request.  All upload imports are now re-uploadable.


Support popular vector search dataset like sift, gist as corpus that can be downloaded from public repository

Is your feature request related to a problem? Please describe.
Similar to nyctaxi, geonames corpus, OpenSerach Benchmark can support some of the popular vector search datasets, that can be downloaded as corpus and used in vector serach workload instead of downloading manually every time for standard usecases.
This can be added as part of nightly runs too.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
While setting up private corpus repository for vector search workload, i get exception at https://github.com/opensearch-project/opensearch-benchmark/blob/9ffbec01149b3a75f58c3d49bb2f9ea39ca1fbd8/osbenchmark/utils/io.py#L578. This is expected since vector search datasets are not standard utf-8 file. This is blocker for adding custom dataset as corpus to vectorsearch workload.
Describe the solution you'd like
A clear and concise description of what you want to happen.
prepare_file_offset_table method was required to optimize disk read for large file by creating offset table that creates mapping from line number to file offset. This is not required for vectorsearch datasets, since, we don't need this offset table for bulk ingestion at this moment. We should make this creation of file offset table optional to extend corpus eligibility criteria to support multiple formats.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Manually download those files into temp directory and update input file path to points to downloaded location.
Additional context
Add any other context or screenshots about the feature request here.

@VijayanB Can you provide what using with and without the corpus would look like to an end user?

Vector search workload requires three external inputs like train ( vectors to index ), test ( vectors to search ) and neighbors ( ground truth ). It works well for average/advance users who wants to benchmark performance against specific dataset. However for simple users or nightly runs, we can use some of the popular datasets to consistently measure performance across every runs. In this case it will be easier and better user experience, if vectorsearch has ability to just run workload similar to nyctaxi instead of downloading those standard inputs every time

@VijayanB Makes sense. Could you show in issue what users have to do now vs. what they could once change is added? i.e. cli comands, etc. Just want to make sure I understand the change in experience.

@jmazanec15 Sure. i was about to raise PR in workload, Let me share how new param file will look like
{
    "target_index_name": "target_index",
    "target_field_name": "target_field",
    "target_index_body": "indices/nmslib-index.json",
    "target_index_primary_shards": 1,
    "target_index_dimension": 128,
    "target_index_space_type": "l2",

    "target_index_bulk_size": 100,
    "target_index_bulk_index_data_set_format": "hdf5",
    "target_index_bulk_index_data_set_corpus": "sift-128-euclidean-train",
    "target_index_bulk_indexing_clients": 10,

    "target_index_max_num_segments": 10,
    "target_index_force_merge_timeout": 45.0,
    "hnsw_ef_search": 100,
    "hnsw_ef_construction": 100,
    "query_k": 100,

    "query_data_set_format": "hdf5",
    "query_data_set_corpus":"sift-128-euclidean-test",
    "neighbors_data_set_format": "hdf5",
    "neighbors_data_set_corpus":"sift-128-euclidean-neighbors",
    "query_count": 100
  }

@VijayanB thanks, I see so corpus for the most part will represent just a file that abstracts away the location of it, correct?

@jmazanec15 Thats correct. It can represent more than 1 file, but, at this moment we don't support folder or multiple files as input, hence, added check to make sure that more than one documents (or files) is not supported.


update link to Nginx wiki

Fixes failure at https://ci.openmicroscopy.org/job/OMERO-DEV-latest-docs/248/warnings3Result/. Staged at http://www.openmicroscopy.org/site/support/omero5.3-staging/developers/Server/ExtendingOmero.html#servers.

Merge build is green again now and link works fine, merging thanks.


Mongoid strategy issue

when i use following mongoid strategy
/** @ODM\Id(strategy="NONE") */
$id = '50e6c53cf8ae035c1b000001';
$User->setId($id);
id inserted successfully in collection but getting following error
Warning: Illegal offset type in vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/UnitOfWork.php on line 1565

@navneetccna which ODM version are you using? Both current dev-master and BETA13 have comments in that line.
Also using "NONE" strategy means that you must generate ID on your own, I'd say that best would be to do that before persist()

Closing because of no response


Fix front layer can be tapped through bug on gallery

Also fixes #17180 where AbsorbPointer also blocks its siblings.

@fkorotkov thanks for following up. No worries. Our AppVeyor hasn't caught up anyway so no harm done there :)


Add ArrayBasedEventLoopState

We use AbstractEventLoopState to acquire an EventLoop in a client for sending requests to a host.
If only one EventLoop is allocated for a host, OneEventLoopState is used.
If more than one EventLoop is allocated for a host, HeapBasedEventLoopState is used to select an EventLoop depending on the activeRequests count.
https://github.com/line/armeria/blob/master/core/src/main/java/com/linecorp/armeria/client/AbstractEventLoopState.java#L27-L35
Using a heap is pretty efficient if the number is large. However, it's inefficient if the number is small. (e.g. 2)
So we need to:

find the threshold to use array-based or heap-based implementation using benchmark
add ArrayBasedEventLoopState

Hi @minwoox, I would like to work on this, can you assign this me?

It's all yours. Thanks! 😄

Hi, :) If it's okay, can I try this one? 🙏

Hi, @erie0210
We are on the internal event called Sprint which helps participants implement PRs and this is one of the issues.
@Kyurenpoto is working on it now. 😓
Really sorry about it and could you pick another one? If you let me know which part of Ameria you are interested in, I might recommend an issue for you.

Thanks for letting me know! 🙂 If you can recommend one first-good-issue that needs to be worked on in the project, I'd love to work on it @minwoox

Hi, @erie0210!
I'm sorry that I missed your comment. 😢
Here are a few issues that you might be interested in:

Add missing getters #4722
Load balancing selection strategies: #4698 #4699 #4700
If you are interested in OAuth2: #3630

Please let me know if you have any questions. 😄


Installing the CLI within 30-60 seconds after its publication to npm causes error

Command
other
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I tried to install the @angular/cli package but got the following error:
2023-06-07T11:16:34.5123669Z npm ERR! code ETARGET
2023-06-07T11:16:34.5135901Z npm ERR! notarget No matching version found for<EMAIL_ADDRESS>2023-06-07T11:16:34.5136229Z npm ERR! notarget In most cases you or one of your dependencies are requesting
2023-06-07T11:16:34.5136537Z npm ERR! notarget a package version that doesn't exist.
2023-06-07T11:16:34.5151588Z

The npm cli was trying to install @angular/cli version 16.0.5 which has @angular-devkit/core version 16.0.5 as a dependency.
The problem was that the @angular-devkit/core package was not published yet.
While investigating I found out that there was a 60 second window where this could happen.
Below is a table of publish times for both packages. You can see that the @angular-devkit/core package is usually published 30-60 seconds later than the cli package.

Version
CLI publish time
devkit/core publish time
difference

16.0.0
2023-05-03T17:54:18.493Z
2023-05-03T17:54:49.718Z
31.2s

16.0.1
2023-05-10T20:07:10.642Z
2023-05-10T20:07:56.296Z
45.7s

16.0.2
2023-05-17T12:28:41.824Z
2023-05-17T12:29:24.033Z
42.2s

16.0.3
2023-05-25T19:33:23.528Z
2023-05-25T19:33:51.414Z
27.9s

16.0.4
2023-06-01T14:14:35.093Z
2023-06-01T14:15:39.485Z
64.4s

16.0.5
2023-06-07T11:15:44.494Z
2023-06-07T11:16:44.925Z
60.4s

publish times are acquired using the npm show command:

npm show @angular-devkit/core time --json
npm show @angular/cli time --json

Minimal Reproduction
Run npm install -g @angular/cli within 30-60 seconds after a new package has been published.
Exception or Error
2023-06-07T11:16:34.5123669Z npm ERR! code ETARGET
2023-06-07T11:16:34.5135901Z npm ERR! notarget No matching version found for<EMAIL_ADDRESS>2023-06-07T11:16:34.5136229Z npm ERR! notarget In most cases you or one of your dependencies are requesting
2023-06-07T11:16:34.5136537Z npm ERR! notarget a package version that doesn't exist.
2023-06-07T11:16:34.5151588Z

Your Environment
This is not relevant (ng was not installed yet)

Anything else relevant?
No response

Hey @Rutgerdj,
Thanks for opening the issue.
There are a number of packages which we need to publish and 60s does seem a decent time for the publish to be performed and propagated on NPM.
We could technically publish the packages without tagging them as latest and tag them all together at the end of the publishing process.
However, following a convo @devversion we do feel that this adds unnecessary completely and this ~60s of mismatching tags is acceptable.


Cannot write images in gif format

After the gif image is written, it automatically becomes a png image, so many details are lost


Minor dependency updates

Weekly manual dependency update whilst Greenkeeper is disabled.

@kevinansfield something else that might be good to go in here (specifically, the ember-cli-mocha update) is the removal of ember-mocha-adapter from bower, as it's no longer needed. Also, bower.json has Faker.js in there twice

@acburdine good catch, I think that got missed in an earlier upgrade

Coverage remained the same at 81.167% when pulling 3ed0e1ba4e7e1ef8ea7ec4c01a4e3a00c485c2b0 on kevinansfield:dep-updates into 767cd7511a84fbc6ed2f8a8cef4fc1e0f30cda3c on TryGhost:master.


Improve of namespaces

For configuration:
var config = new ConfigurationContainer();
config.EnableDeferredReferences();
config.Type<Section>().Member(p => p.Id).Name("Identity");
config.Type<Section>().EnableReferences(p => p.Id);
IExtendedXmlSerializer exs = config.Create();

I need tree namespaces:
    using ExtendedXmlSerializer.Configuration;
    using ExtendedXmlSerializer.ExtensionModel.References;
    using ExtendedXmlSerializer.ExtensionModel.Xml;

It is not a bag, but nice to have ;)
What do you think?

No that is fair feedback.  I did try to group extension methods with their extensions, but for the core/built-in functionality we can consolidate them into one file.
I went ahead and did that.  Now, IExtendedXmlSerializer is in ExtendedXmlSerializer.ExtensionModel.Xml because at some point we were discussing ExtendedXmlSerializer.ExtensionModel.Json and that might still be done in the future.  If you use var exs = config.Create(); instead, it keeps the 3rd namespace from being used. :)


Make appconfig IT to be independent

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
Adjust the position of appconfiguration-config IT to make it an independent part
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - spring -tests

@mrm9084 please help take a look, if it is ok, I will sync to springboot3 as well, thanks.


Line numbers

Hello!
First: I love your Addon! Thank you for this nice work!
Is it possible to print line numbers in PDF?
For example every fifth line (except headings).

Maybe you could work with CSS: nth() and content?


Add "build" prefix

I was surprised to not find the "build" prefix, was this an oversight or intentional?
It is advised by both conventionalcommits.org and the Angular guide:

https://www.conventionalcommits.org/en/v1.0.0/#summary
https://github.com/angular/angular/blob/22b96b9/CONTRIBUTING.md#type

Thanks, and great tool! I'm happy to switch from Commitizen to a Go alternative.

Complete oversight on my part! Great catch


Dependency Versions

I've noticed when looking at the Brackets source code tree that a lot of the dependencies you guys use are quite outdated, such as React, Immutable, Lodash, jQuery and a lot of other dependencies. That and when trying to build Brackets using Node.js 7.1.0, it fails at npm install due to the dependency versions in package.json being really outdated. Some of the dependencies you guys are using are over 4 years old (Example: Mustache.js), which is really bad, especially when it comes to security.
Also, one little question: Instead of maintaining a separate CodeMirror fork, why not just use the official CodeMirror releases? It'll be much easier to update CodeMirror with every new Brackets release then.
Also, I know you guys might not consider this to be that important, but I think it would be a good idea if you guys went and did a full dependency update. You guys can't keep using 4 year old dependency versions forever. And it would be a good idea as well to upgrade to the latest current Node.js version (7.1.0 as of writing this) instead of sticking with old 6.x releases.

We already moved some thirdparty dependencies to be managed by npm, careful work needs to be done to move more, but I agree with what you've written.

I totally agree with you. I outlined the process, updated (most) of the dependencies and created a POC here: https://github.com/adobe/brackets/issues/12006. TL;DR: it's possible, albeit a bit non-trivial and requires a good calibration. There's other things I would want to do in the long run, like getting rid of AMD modules in favor of CommonJS ones, but if Chromium manages to land ES6 modules support sometime soon we might be able to use just them too.
We are using a fork of CodeMirror due to historical reasons but I think by now most of the changes required for Brackets are already upstream, so I don't see if there's any benefit of maintaining our own fork anymore.


Defect: internal error when accessing array-coarray of derived type with allocatable component

The following program produces incorrect results. The line marked ! NOTE should print 1, 2, 3, 4, but instead I get "OpenCoarrays internal error on image 2: libcaf_mpi::caf_sendget_by_ref(): can not allocate 0 bytes of memory."
Versions:
GCC 11.2.0
OpenCoarrays 2.9.2
MPICH 3.2

program bug
    type :: container
        integer, allocatable :: stuff(:)
    end type

    type(container) :: co_containers(10)[*]

    if (this_image() == 1) then
        allocate(co_containers(2)%stuff(4))
        co_containers(2)%stuff = [1,2,3,4]
    end if

    sync all

    if (this_image() == 2) then
        print *, co_containers(2)[1]%stuff  ! NOTE
    end if
end program

@vehre this has entered our critical path for a paper draft due in December.  Any chance you can work on a fix soon?

@rouson I can find some time to work on this, but I have to report: The example in the description works for me on:

gfortran 11.3.1, fedora 35 system supplied
mpich 3.4.1, fedora 35 system supplied
opencoarrays 2.10.1, self-compiled

Only the one in the link in Link to the somewhat larger demonstrator crashes on init, with an PMPI_Win_allocate: Invalid topology,... error. If you want me to investigate further, just let me know.

@vehre Yes, please investigate further.  @everythingfunctional encountered this same issue yesterday.  I'll let him confirm his setup.  I have a broken installation at the moment after problems with a macOS upgrade so I'm not able to confirm this immediately.

@rouson I have analysed the issue further: The code generated for managing the allocatable component of the type in the module does not take into account that the type could be used in a coarray. I.e. there is no space assigned (at compile time) to keep track of the allocation status and the coarray (slave) token. Furthermore is because of this no code generated to define "a mpi window" for the allocated memory, which prevents the one image from the accessing this memory on the other image.
Or with other words: Solving this issue is nothing quick to be done, but some bigger effort. One needs to find a way to portably generate the code of the module to either always call coarray-registration routines for every memory allocated in a module when the compile flag -fcoarray=lib is given or to create two instances of the code to execute for each module, one with coarray support and one without.
How to proceed?

@vehre thanks for the quick reply.  If this had been a quick fix, Sourcery Institute could have funded it.  Because its a larger effort, I'll need to seek alternative funding. Please email me a rough estimate at your earliest convenience.

The reproducer submitted in the original comment for this issue has been fixed on the main branch and will appear in the 2.10.1 release, which currently has draft release notes. @everythingfunctional please create a new issue with the from comment your issue comment above and link to @vehre's comment.


adapt upgrade nodes command to the api

in the beginning it was planned to upgrade nodes one by one, but
now the api just allows the upgrade of all nodes at once

 #hackathon - adapt crowbarctl upgrade nodes call from single to all nodes

To be precise: the method will still do the upgrade of nodes one by one (but user can't choose the order)


local.conf.sample: remove dev-pkgs from IMAGE_FEATURES

As far as I can tell (no real git history on the choice to set this
image feature) this is a holdover from the original work to create a
self-builder and serves no real usecases we have for meta-overc. The
only thing this accomplishes is to make our images larger than they
need to be, counter to what we are attempting to accomplish,
especially with cube-essential. I thought about commenting this out
but there is already a full description in the text just below where
this was and this is also usually added via EXTRA_IMAGE_FEATURES.
As a side note the inclusion of dev-pkgs had a somewhat unexpected
double whammy effect on the image size. Since ptest is included in the
distro features any ptest sub-pkg RDEPENDS would be installed in the
images despite the ptest sub-pkg itself not actually being included in
the image, since this feature includes any -dev pkg which is built and
the -dev pkg always depends on the main pkg.
Building with this change to your local.conf has no effect on the
runtime as far as I can tell, nor should it be expected.
Signed-off-by: Mark Asselstine<EMAIL_ADDRESS>

merged


Addon not working

latest.log
When I boot up the game all I see is default origins

This addon only adds a way to render Elemental Bursts and Skills on your screen, it does not add actual Origins by itself.


kwargs are now gensym-ed

Fixes #59 and I took the liberty of clarifying the docs, which fixes #58

Looks great, thanks!


pylint: Fix R1732 consider-using-with

Signed-off-by: Stefan Berger<EMAIL_ADDRESS>

The warning is called R1732 not W1732.

When testing https://github.com/keylime/keylime/pull/937/commits/447ef9f1de3cfa80028d1923e55e3b08a93534b1 there is one test-failure in testing-farm:fedora-35-x86_64:
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
::   -c delete
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::

:: [ 10:34:35 ] :: [  BEGIN   ] :: Running 'lime_keylime_tenant -c delete'
Reading configuration from ['/etc/keylime.conf']
2022-03-23 10:34:36.296 - keylime.tpm - INFO - TPM2-TOOLS Version: 5.2
2022-03-23 10:34:36.302 - keylime.tenant - INFO - Setting up client TLS in /var/lib/keylime/cv_ca
2022-03-23 10:34:36.302 - keylime.tenant - WARNING - Using default UUID d432fbb3-d2f1-4a97-9ef7-75bd81c00000
2022-03-23 10:34:36.329 - keylime.tenant - INFO - Agent Info:
{"d432fbb3-d2f1-4a97-9ef7-75bd81c00000": {"operational_state": "Get Quote", "v": "5YiO9Kc1QK9h2OSdfiD0+uVbZ7imZhZmQsLcshl0QsY=", "ip": "<IP_ADDRESS>", "port": 9002, "tpm_policy": "{\"22\": [\"0000000000000000000000000000000000000001\", \"0000000000000000000000000000000000000000000000000000000000000001\", \"000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001\", \"ffffffffffffffffffffffffffffffffffffffff\", \"ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff\", \"ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff\"], \"15\": [\"0000000000000000000000000000000000000000\", \"0000000000000000000000000000000000000000000000000000000000000000\", \"000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\"], \"mask\": \"0x408400\"}", "vtpm_policy": "{\"23\": [\"ffffffffffffffffffffffffffffffffffffffff\", \"0000000000000000000000000000000000000000\"], \"15\": [\"0000000000000000000000000000000000000000\"], \"mask\": \"0x808000\"}", "meta_data": "{}", "allowlist_len": 6, "mb_refstate_len": 0, "accept_tpm_hash_algs": ["sha512", "sha384", "sha256", "sha1"], "accept_tpm_encryption_algs": ["ecc", "rsa"], "accept_tpm_signing_algs": ["ecschnorr", "rsassa"], "hash_alg": "sha1", "enc_alg": "rsa", "sign_alg": "rsassa", "verifier_id": "default", "verifier_ip": "<IP_ADDRESS>", "verifier_port": 8881, "severity_level": null, "last_event_id": null}}
2022-03-23 10:34:41.387 - keylime.tenant - ERROR - Timed out waiting for delete of agent d432fbb3-d2f1-4a97-9ef7-75bd81c00000 to complete at CV
:: [ 10:34:41 ] :: [   PASS   ] :: Command 'lime_keylime_tenant -c delete' (Expected 0, got 0)
:: [ 10:34:41 ] :: [   FAIL   ] :: File '/var/tmp/rlRun_LOG.AoxQaNdP' should contain 'INFO - CV completed deletion of agent d432fbb3-d2f1-4a97-9ef7-75bd81c00000'
:: [ 10:34:41 ] :: [  BEGIN   ] :: Running 'lime_keylime_tenant -c cvstatus'
Reading configuration from ['/etc/keylime.conf']
2022-03-23 10:34:42.204 - keylime.tpm - INFO - TPM2-TOOLS Version: 5.2
2022-03-23 10:34:42.209 - keylime.tenant - INFO - Setting up client TLS in /var/lib/keylime/cv_ca
2022-03-23 10:34:42.209 - keylime.tenant - WARNING - Using default UUID d432fbb3-d2f1-4a97-9ef7-75bd81c00000
2022-03-23 10:34:42.227 - keylime.tenant - INFO - Agent Info:
{"d432fbb3-d2f1-4a97-9ef7-75bd81c00000": {"operational_state": "Get Quote", "v": "5YiO9Kc1QK9h2OSdfiD0+uVbZ7imZhZmQsLcshl0QsY=", "ip": "<IP_ADDRESS>", "port": 9002, "tpm_policy": "{\"22\": [\"0000000000000000000000000000000000000001\", \"0000000000000000000000000000000000000000000000000000000000000001\", \"000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001\", \"ffffffffffffffffffffffffffffffffffffffff\", \"ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff\", \"ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff\"], \"15\": [\"0000000000000000000000000000000000000000\", \"0000000000000000000000000000000000000000000000000000000000000000\", \"000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\"], \"mask\": \"0x408400\"}", "vtpm_policy": "{\"23\": [\"ffffffffffffffffffffffffffffffffffffffff\", \"0000000000000000000000000000000000000000\"], \"15\": [\"0000000000000000000000000000000000000000\"], \"mask\": \"0x808000\"}", "meta_data": "{}", "allowlist_len": 6, "mb_refstate_len": 0, "accept_tpm_hash_algs": ["sha512", "sha384", "sha256", "sha1"], "accept_tpm_encryption_algs": ["ecc", "rsa"], "accept_tpm_signing_algs": ["ecschnorr", "rsassa"], "hash_alg": "sha1", "enc_alg": "rsa", "sign_alg": "rsassa", "verifier_id": "default", "verifier_ip": "<IP_ADDRESS>", "verifier_port": 8881, "severity_level": null, "last_event_id": null}}
:: [ 10:34:42 ] :: [   PASS   ] :: Command 'lime_keylime_tenant -c cvstatus' (Expected 0, got 0)
:: [ 10:34:42 ] :: [   FAIL   ] :: File '/var/tmp/rlRun_LOG.gSGAYMcd' should contain 'Verifier at <IP_ADDRESS> with Port 8881 does not have agent d432fbb3-d2f1-4a97-9ef7-75bd81c00000'
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
::   Duration: 7s
::   Assertions: 2 good, 2 bad
::   RESULT: FAIL (-c delete)

All other tests are passing. I prefer to ignore this test failure but something is flaky...

/packit test

I wonder what that is when Fedora 34 fails and all other ones pass...
:: [ 19:42:43 ] :: [   FAIL   ] :: Command 'rlWaitForCmd 'tail /var/tmp/limeLib/agent.log | grep -q "Executing revocation action"' -m 10 -d 1 -t 10' (Expected 0, got 1)

I wonder what that is when Fedora 34 fails and all other ones pass...

@kkaarreell seems this like a timing issue?
@stefanberger just use /packit test to rerun the tests.

/packit test

I wonder what that is when Fedora 34 fails and all other ones pass...

@kkaarreell seems this like a timing issue?

I can increase the timeout... so far 10 seconds for revocation action initiation was enough. What should be the reasonable timeout?

I can increase the timeout... so far 10 seconds for revocation action initiation was enough. What should be the reasonable timeout?

10 seconds seem actually reasonable, but if it flaky again in the future we need to take a closer look what is causing this.

Maybe some networking glitch on the actual virtual system... That would explain the above failure in the keylime tenant.
Having logs with timestamps would help. I will try to find a way how to incorporate more logs into the test output when test have failed.

I will try to find a way how to incorporate more logs into the test output when test have failed.

Hopefully this would help next time
https://github.com/RedHat-SP-Security/keylime-tests/pull/70


Efficient slot-wise masking operation in CKKS

For a given ciphertext that encodes a vector of size N/2, a slot-wise masking operation is defined as multiplying this ciphertext with a plaintext, where the N/2 numbers in the plaintext are either 1 or 0.
A simple solution is to multiply the ciphertext with the plaintext. I wonder if there are other implementations that is more efficient (in terms of time complexity or run time) than the simple solution?

Plaintext multiplication in CKKS is actually one of the most efficient operation (its a series of point-wise vector multiplications if both operands are in the NTT domain), hence it has the same order of magnitude of a ciphertext-ciphertext addition. The main downside of masking is that it consumes a level. However, this can be mitigated by including the masking in the previous operation if it also makes use of plaintext multiplication and/or by using a smaller scale for the plaintext mask.

Thanks for the explanation!
Could you explain how to avoid one additional level by using a smaller scale for the plaintext mask?
Is it possible to implement this in SEAL?

You cannot "avoid" consuming an additional level, but you can reduce the number of bits of the total modulus Q that you use by using smaller scales for plaintext multiplications. This allows to allocate more bits to the ciphertext-ciphertext operations as these require a proportionally larger scale to get the same precision. However, if your goal is only to use less primes in total (for computational efficiency), regardless of the number of bits consumed, you can allocate a prime that is twice the scale that you will use for your operations. This way you can do two multiplications using one prime only.

I remember that in CKKS the size of each prime should be roughly the same (in terms of the number of bits). Is this statement correct?
From your description, it seems different primes which have significantly different size can be selected for different purposes (e.g., for masking or for normal ct-ct mult). Is my understanding correct?

There is no special requirement for the modulus used in RLWE based scheme outside of an upper bound on its bit-size given the ring degree and the error distribution. So as long as the product of all your primes remains equal or smaller than the upper bound given by your security parameter you are good to go and can chose any size for the primes.
For a given circuit you can optimize its bit-consumption by selecting each prime independently, such that each step of the circuit preserves a similar precision.  Doing so, instead of having all primes be the same size, can sometime lead to a reduction in the total consumption of modulus that is large enough to allow you to use a smaller ring degree, which makes everything more efficient.

Hi @Pro7ech
I try to implement this on SEAL. Regarding your comment this can be mitigated by including the masking in the previous operation if it also makes use of plaintext multiplication:
This is what I want to achieve:
pt x ct x mask0
pt x ct x mask1
...
pt x ct x mask_n
I can change the order of operation to:
pt x mask0 x ct
pt x mask1 x ct
...
pt x mask_n x ct
The pt x mask_i operations have to be performed on the fly (not offline). In that case, I think we need to do mod_switch (rescale) after the the pt x mask_i operations. So although we save one level on the ciphertext, we pay an extra mod_switch on the plaintext. Is my understanding correct?

Set the scales of pt and mask to half of the scale of ct; set the scale of ct to the next prime to drop; perform one rescale after pt x ct x mask0.


NIFI-12710: Allow for microsecond precision timestamps when pushing t…

…o database

Summary
NIFI-00000
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks @exceptionfactory have addressed.

Thanks for the review @exceptionfactory @dan-s1 . Agreed, the tests needed some cleanup. I actually simplified them significantly by just avoiding the use of the current time and trying to manipulate it into the format desired and instead just create predefined timestamps. This is simpler, cleaner, and far less error prone.


Fix a casing typo in LsaFreeReturnBuffer

LsaFreeReturnBuffer is the correct casing for the "LSA" prefix of this function.

@alexhinz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for the contribution, @alexhinz ! I've merged your changes and they should be live soon.


Static files in the app.yaml when building

Under the source directory in my Svelte project, I have a "static" folder containing images.  Build adds the following lines to the app.yaml for each image file.  These break the GCP deployment.  By deleting these lines everything works as expected.
url: /fifthimage.jpg
static_files: storage\fifthimage.jpg
upload: storage\fifthimage.jpg
secure: always

Based on the \ character, i'm guessing you maybe are on windows? Could you try and change the characters to / and see if that fixes it, if so it will be an easy fix in the adapter


pkg/operator: replaced fmt.Errorf with error.wrap

Description
we have used fmt.errorf to formatted error in the entire code bases, but this loses many information.Instead of using fmt.errorf, we can use errors.Wrap and errors.Errorf in https://github.com/pkg/errors. This wrap errors in contextual information and still preserve the underlying error and originating stack trace.
Issue reference
Please reference the issue this PR will close: #1954
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[X] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Tests seem to be working on my machine. I have used make test recipe. Also the test that is failing is not related to the changes that I made. Please advise.

@yaron2 @amanbha @youngbupark, Please review the PR. I will open up the other PR taking the review comments on this into consideration.

Tests seem to be working on my machine. I have used make test recipe. Also the test that is failing is not related to the changes that I made. Please advise.

tests are passing now.

/ok-to-test


Reload Window Command Kills Active Consoles

Summary

VSCode Version: any
OS Version: Windows

Steps to Reproduce

Have something run in a console, e.g. php bin/console server:run in a Symfony project
Use command 'Reload Window'
The started console thread gets terminated

What I Would Expect
I would expect that only the UI is refreshed, but the underlying processes (like consoles) not.
Discussion
I guess this happens by design (?) and maybe for my use case I need to run a different command to reload the window (is there one?),
In my Symfony project there are many files, and sometimes the file browser does not update correctly when switching branches in git, resulting in files being shown that are actually deleted.
When running the Reload Window command the file browser is refreshed and displayed correctly.
However, as mentioned above, also any spawned console threads are terminated.
Thanks in advance.

/duplicate https://github.com/Microsoft/vscode/issues/20013


Implement Basic Test Framework

Set up a testing framework and add initial tests for core functionality.
Tasks:

[ ] Choose and set up testing framework
[ ] Create test directory structure
[ ] Add unit tests for core functions
[ ] Add integration tests for session start script
[ ] Update package.json test script
[ ] Add test documentation
[ ] Set up test GitHub Action

Current Status:

package.json has a placeholder test script that exits with error
No tests directory exists
No test framework chosen

This will help ensure reliability and make it easier to accept contributions.

Completed in PR #15. Implemented comprehensive test framework with:

Jest for JavaScript unit testing
BATS for shell script testing
GitHub Actions workflow for CI
Test documentation in README


Make sleepNote style respect overridden styles

Specifying an opacity value within sleepNoteStyle {} had no impact as the .4 value was hard coded.
"this.sleepNote.style.opacity = this._map.options.sleepNoteStyle.opacity" allows the user's specified value to be respected.

Haven't seen your PR, I added the same changes in #30 :).


Add license information to the podspec files

This is fixed in the podspecs in my repo, which won't make it to the grpc repo. Instead they'll be submitted to the Cocoapods main repo.


bolded where not to add name

I missed this when I made my first contribution. this might make the instruction more noticeable.

@wangmedia, where to add contributors name is more of a suggestion than a rule. If the name is added somewhere in between, there wouldn't be many merge conflicts. This will help the maintainer a lot.
We made this change 5 days ago https://github.com/Roshanjossey/first-contributions/commit/3c24a99fbbb942251e965e5e34bb3c7d499bb700. After this change, most of the pull request we receive are with name added somewhere in between. So, I don't think there is a problem with visibility.
So, I'm of the opinion of not making this change. Do share your thoughts on that.
Cheers.

@Roshanjossey Thank you for the reply and explanation. I appreciate your project to help a noob like me learn more about the git and the Github interface.

@wangmedia, anytime mate. 🖖
I'm closing this pull request.


MudTextField

InputType should have Date and Time as well ie InputType.Date

We already have MudDatePicker and MudTimePicker. So I don't know if there is a need for this. But you can do a PR if you want. Should be pretty easy to implement.

I did a PR and they told me to open an issue! i think its good to have html5 date and time!

You mean this PR: #2620 ?
There were no changes. You just wanted to merge master into dev?

Yes that one!

Hi.
Closing this issue as it's out dated.


Key Event POST new event with non-existent Type will timeout

Description
Trying to create a new key event of a type other than the existing ones, will result in request timing out.
To Reproduce
E.g. Try adding a key of type: "type"
Expected behavior
Request should return an error.

hey @iustinp, this shouldn't happen as there is a validation check for types in place here: https://github.com/adobe/spacecat-shared/blob/6bed3412af558946575f9f6be6d313ff0511db40/packages/spacecat-shared-data-access/src/models/key-event.js#L68-L70
In case a type is not in the KEY_EVENT_TYPES set, it will throw an error like below:

{
	"message": "Unknown value for \"type\": type"
}

The status code is 500 Internal Server error but it should a 400 - Bad Request as the type is incorrect.

Hello! Indeed, i was able to see that the timeout-like behaviour is induced by Power Automate HTTP request action, because while the response from the spacecat API is instant (with the details you provided), the automation only fails after ~25minutes.
I will add extra checks on my side and also look to optimise the flow to throw such errors much faster.
In the meantime this probably can stay up for the wrong status code.
Thank you for looking into this!

Updated the title for this ticket
The approach is to have an exceptionHandler (that extends the Error class) to throw the adequate error (400 for BadRequest)
Created draft PR's
https://github.com/adobe/spacecat-shared/pull/276
https://github.com/adobe/spacecat-api-service/pull/372/files

@iuliag @ekremney  Wdyt about this approach? ⬆


Proposal: Model Generator

~/example$ rails g ember_rails:model Message content tags:array done:boolean words:integer person:object add:function
      create  app/assets/javascripts/ember/models/message.js.coffee

Result:
app/assets/javascripts/ember/models/message.js.coffee
Example.Message = Ember.Object.extend
  content: null
  tags: []
  done: false
  words: null
  person: {}
  add: ->
    return

I think at this stage I don't want these sorts of code generators in ember-rails - mainly because I think everyone has a different API at the moment. If there was a more standard API then I'd probably reconsider.


vijay-sassy: Create a Date with Values in java code

I added CreateDate.java & updated the readme.

Have to do revisions based on my editor's recommendation. So, I am closing this pull request.


Add Shotstack to in the wild list

Added our link on the Widdershins project just now, we've done a some customisation to our design, users can see it here: https://github.com/shotstack/oas-api-definition. Hope you can add us to your list and share our our integration for others to reference.

Thanks again! I will add a note the wiki about your use of Shins as a sub-module when I revamp the FAQ page.

Thank you


Lock down protobuf version or recompile distribution files on build

Zwift Mobile API client version: 0.2.0
Python version: 3.10
Operating System: Linux

Description
The compiled protobuf files are not compatible with the requirements in the setup.py
Changing the version to "protobuf<=3.21.0" resolves the import issue.
You may either recompile the protobuf files distributed on build or lock down protobuf version.
What I Did
from zwift import Client as ZwiftClient

downstream consumers can apply the workaround proposed in a method similar to
https://github.com/snicker/zwift_hass/pull/38

not sure if @jsmits is still maintaining this library but the best solution is going to require regenerating the pb2 file with a new version of protoc, bumping the release, and redeploying to PyPI


feat: keep string literal quote in table default

As @brennanjl required, kuneiform will preserve the quote for string literal in table default.   This is related to kwilteam/kwil-db#353
This will make the behavior consistent in Kuneiform when receive a literal string value:

extension config
table default
extension call and action call

This should be merge when https://github.com/kwilteam/kwil-db/pull/353 is merged


Simplify Group API structure

Description of the change
Remove fields from the top level group that are now request configuration specific. If a field is deprecated, ignore it from speakeasy altogether. If a field is computed and not user specified mark the field as computed to suppress the terraform diff but still track it in terraform state so we can expose the attributes for downstream consumption.
Checklist

[x] I performed a self-review of my code
[x] I manually tested my code change (please list details in description)
[ ] I added unit tests
[ ] I updated the changelog
[ ] I updated the public facing docs

After this PR the diff when applying "no changes" looks like
  ~ resource "opal_group" "foobarbaz" {
        id                     = "5440abb8-1481-4e6a-bd2f-bb02e866cbf8"
        name                   = "foobarbaz"
      ~ request_configurations = [
          ~ {
              + condition              = (known after apply)
              + reviewer_stages        = []
                # (8 unchanged attributes hidden)
            },
        ]
        # (12 unchanged attributes hidden)
    }

It's not clear to me why these two are still listed when nothing changes but will tackle later


LLDB Watchpoints Window

Hi Snare!
Would you be able to add an extra window that displays watchoints in lldb?
Many Thanks!

Hey, you can use a command view like this:
$ voltron view command "watch list"

I see, that should work fine apparently, thanks for the heads up.
I do have couple more questions, I ask them here I read of making new issue.

I installed voltron a few times already with no issues. However latest install is a bit weird, voltron wouldn't load in lldb unless I start lldb as sudo. I never experienced this, in normal terminal voltron view xxx doesn't require sudo. Is there a way I completely uninstall and reinstall it?

I saw you've added scrolling feature to memory view. Is there a way to do the same in disasm view?
The reason is, I mainly remote debug ios games, and when it breaks on a watchpoint, most the times the instructions I need are a few lines above the one that it breaks on.


Race condition when two libraries update the same init.py

This has had me pulling my hair out.
If pydantic is enabled in the requirements.txt, pyjwt fails with
#12 7.705 Traceback (most recent call last):
#12 7.705   File "<string>", line 1, in <module>
#12 7.706 ImportError: cannot import name 'PyJWKClient' from 'jwt' (/opt/venv/lib/python3.12/site-packages/jwt/__init__.py)

Looks like the init.py doesnt get updated with PyJWKClient in the failed runs, but no indication as to why.
Cant make it fail deterministically but doing  docker system prune, it will fail every other (?) time.
Here is the Dockerfile:
FROM python:3.12.3 AS build

ENV PATH="/opt/venv/bin:$PATH"
ENV VIRTUAL_ENV="/opt/venv"

# Python runtime dependencies
RUN apt-get update && apt-get install -y --no-install-recommends \
    make \
    g++ \
    gfortran \
    cmake && \
    python -m venv /opt/venv

COPY --from=ghcr.io/astral-sh/uv:latest /uv /bin/uv

WORKDIR /app
COPY requirements.txt  ./

RUN echo 1 && \
    uv pip -v --no-cache sync requirements.txt &&\
    python -c "from jwt import PyJWKClient"

Here is the requirements.txt:
# This file was autogenerated by uv via the following command:
#    uv pip compile requirements.in pyproject.toml
annotated-types==0.7.0
    # via pydantic
anyio==3.7.1
    # via
    #   fastapi
    #   starlette
certifi==2022.12.7
    # via requests
cffi==1.17.0
    # via cryptography
charset-normalizer==2.1.1
    # via requests
cryptography==43.0.0
    # via jwt
# fastapi==0.105.0
    # via -r requirements.in
google-search-results==2.4.2
    # via locus-ferb (pyproject.toml)
idna==3.4
    # via
    #   anyio
    #   requests
jwt==1.3.1
    # via locus-ferb (pyproject.toml)
pycparser==2.22
    # via cffi
pydantic==2.8.2
    # via fastapi
pydantic-core==2.20.1
    # via pydantic
pyjwt==2.9.0
    # via locus-ferb (pyproject.toml)
requests==2.28.1
    # via
    #   google-search-results
    #   serpapi
serpapi==0.1.5
    # via locus-ferb (pyproject.toml)
sniffio==1.3.1
    # via anyio
# starlette==0.27.0
    # via fastapi
typing-extensions==4.9.0
    # via
    #   fastapi
    #   pydantic
    #   pydantic-core
urllib3==1.26.13
    # via requests

Attached are the two logs, running the same command one after the other:
docker system prune && docker build --ssh default --progress plain -f Dockerfile-fail .
build-working.log
build-failing.log

Well, in the both examples there are things like:
#12 7.601 DEBUG File already exists (subsequent attempt), overwriting: /opt/venv/lib/python3.12/site-packages/jwt/__init__.py

It looks like there's a conflict between two packages and you're seeing a race condition where it succeeds or fails depending on which one overwrites the file last.

That makes sense, pyjwt adds functionality to jwt, so its update to the init.py should probably come after. Is there a way to force a dependency? If it were possible to do, would that fix the race condition?
I'll check to see if we can remove jwt, and only keep pyjwt. But I see the same issue for another library that seems to have the same issue, in that case I know I need both libraries.
The race condition doesn't happen in pip, because of its slowness.

I see. I don't know if it's feasible for us to unpack packages in a defined order like that, @charliermarsh would know better.
It seems like pretty bad behavior for a package to rely on patching an existing one by overwriting its files. I think in this case, you'd be best off running two separate install operations — one to prepare the environment, and the next to patch the package.

Could the warning be surfaced, so it's clearer that one is clobbering the other?
Agreed on the package behavior, but such is life with dependence on third party libs...
And I take back that it may not happen in pip, because I see articles about the import not working.
https://github.com/jpadilla/pyjwt/issues/685
https://stackoverflow.com/questions/68912059/module-not-found-error-when-using-google-search-api-serpapi

Also, thank you @zanieb for  identifying the issue so quickly.

After looking at what my options around working around this while sticking with "sync", and was hoping to use uv lock and then uv sync --no-install-package pyjwt instead of uv pip sync but....
The project is marked as unmanaged

I do (I set managed=false in the pyproject.toml) - because our existing docker images use /opt/venv, and was trying to avoid changing that. But I think I will move stuff into /app/.venv so can use uv sync instead of uv pip.

Ok I have it working with this Dockerfile and uv.lock
ROM python:3.12.3 AS build

ENV PATH="/app/.venv/bin:$PATH"
ENV VIRTUAL_ENV="/app/.venv"

# Python runtime dependencies
RUN apt-get update && apt-get install -y --no-install-recommends \
    make \
    g++ \
    gfortran \
    cmake

COPY --from=ghcr.io/astral-sh/uv:latest /uv /bin/uv

WORKDIR /app
COPY uv.lock pyproject.toml ./

RUN RUST_LOG=trace uv -v --no-cache sync --no-install-package pyjwt --no-install-package google-search-results &&\
    RUST_LOG=trace uv -v --no-cache sync &&\
    python -c "from jwt import PyJWKClient"

I would like to request a feature for future uv noobs, that warns when a library clobbers another one, I spent a fair bit of time going thru a number of permutations thinking there was something going on in my environment/cache/docker setup.

@konstin is it feasible for us to use warn_user_once when installing a package clobbers files?

I'm honestly not sure if we should, because I think there are packages that do this somewhat intentionally.

I think this one does do it intentionally but if it's installed in the same invocation as its target package that's a big problem since we don't do it in a deterministic order?

could there be an option - uv add <package> --depends_on <main_package> which would update the uv.lock to install that package after the main_package?


Display a tooltip with result information when people click on the map

Now when you click on an area on the map, you get something like:

For https://trello.com/c/E8zwQ5G5
Still to do: https://trello.com/c/4UmZGiwl
Discovered while testing this: https://trello.com/c/nICsd0mY

Thanks! Let's see how it looks on search-dev...


Fix search for uppercase domains

Fixes some issues from: #23

Fixed search box for uppercase/lowercase domains vs addresses. Requires: https://github.com/CryptoScamDB/api.cryptoscamdb.org/pull/21
Fixed error displaying coin-page.
Fixed some TSLint stuff.

Closed because changes were implemented in another PR.


Atom.Object.defineProperty.get is deprecated.

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/usr/share/atom/resources/app/src/atom.js:55:11)
Object._events (/home/bryan/.atom/packages/autoclose-html/lib/autoclose-html.coffee:108:13)

duplicate..


https://www.myetherwallet.com/wallet/dashboard seems to crash the whole extension?

Probably the 100,000 contract calls they make.  😡


error 5, could not start dnsproxy-service

http://imgur.com/p81lPcS
What the crap?
Using version 1.4.3 I think

Unfortunately I never encountered that error, but Google has many links about this issue that might help you.


How to run storybook with dist folder's bundle.js minified js file instead of the original source code component files?

Issue details
I want to run storybook with the dist bundle instead of the original source code, so its kinda become a demostration app for all the components but only with the dist bundle I created... Unless if there is still way, but I just need to bundle things differently just for storybook? Is that even possible?
Thanks
Steps to reproduce
I can't reproduce a steps, as its more of a new way of doing things I guess.
Please specify which version of Storybook and optionally any affected addons that you're running

@storybook/react latest

Affected platforms
All
Screenshots / Screencast / Code Snippets (Optional)
I really do not know if its possible...

I'm not sure I understand your usecase correctly, but you probably can do something like this:
// somewhere in your entry point

window.components = {
  MyButton,
  MyInput
}

// my-button.stories.js

import './relative-path-to-your-bundle-js'
import React from 'react'
import { storiesOf } from '@storybook/react'

const {MyButton} = window.components

storiesOf('MyButton', module)
  .add('with text', () => (
    <MyButton onClick={action('clicked')}>Hello Button</MyButton>
  ))

@Hypnosphi that's an interesting approach... hum... let me think about it

Hi @adamchenwei, I wonder what's the use case for using a build bundle as a source? If I understand it correctly, in order to load those components, you would need to include Storybook inside your production bundle, which would increase it size. So I wonder why you want to do this?

hey, the usage is to uploading compiled code instead of source code into an AWS instance for development demonstration for the public, but without having to risk uploading the whole source code into it directly.
Hope that clarifies a bit about the usage part.

@adamchenwei It would be probably more helpful to see your code, but if you import only your stateless components into Storybook, you not only don't risk uploading the whole source, but you trim the fat since your components shouldn't have any imports maybe except for React-related imports and CSS.

interesting i can look into it! thanks


docs: Add a new recipe for Minio with Cloud berry client.

We need to document how Minio can be used with Cloud berry client.

@harshavardhana Currently Cloud Berry getting tested for V4 support for S3 compatible alternatives. It will come out Q4 this year as mentioned by one of their employee. 👍

@harshavardhana we can close it via https://github.com/minio/cookbook/pull/94/files. Thanks to @erudinsky for contributing.  😸


Pass proxy to mongodb-download

When behind a corporate proxy, the post install task (node install.js) fails.  This appears to be a failure to pass the npm proxy settings through to install.js.

Agree with @AdamYoungberg  :+1:

I've released new version of Mockgoose 6.0.0 there are small modifications to your unit tests that you would have to do, but it will save you a lot of headache in the future, if you will convert to 6.0 release, that also incorporates requested feature here.
https://www.npmjs.com/package/mongodb-prebuilt#download-proxy

Hooray!  Thanks this is working beautifully.


xorg.xkbcomp: 1.4.2 -> 1.4.4

Motivation for this change
https://lists.x.org/archives/xorg-announce/2020-February/003037.html
https://lists.x.org/archives/xorg-announce/2020-November/003063.html
Things done

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

[x] NixOS

[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Merge after https://github.com/NixOS/nixpkgs/pull/103710.

#103710 got merged.

@GrahamcOfBorg eval

What needs to be done to get this merged?

@SuperSandro2000, would you please consider merging this?


Very long vCards filenames

Hello,
I noticed during my nextcloud sync that some vCards filename is very long (NC can't sync files with filename > of 250 characters).
Is it possibile to trim the filenames? If the filenames are needed this long can I delete some vCards?
Am I going to break something in html chats?

Hmm, I already tried to limit the file name to < 250 characters. Perhaps the limit should be further reduced?
If you delete the vCards file, it will not break anything in the HTML.

The new limit to the vCard file name is released in 0.9.5.

@KnugiHK Still an open problem as the vcard target path length is important, not the vcard file name length alone.
If the target path length of the vcard is > 256 chars the tool still throws an error on Windows. So the tool should be aware what the base path is and its length and then substract it from 256 so you know what the allowed max vcard file name is.

Once I confirm the problem mentioned by @duracotton, I will reopen this issue.


Avoid mandatory sirepo import

We include sirepo only under the 'full' installation option since it shouldn't be necessary for purely Python-based evaluations. However, it is being imported the top level of some modules such as:
https://github.com/radiasoft/rsopt/blob/c6aa14a601ea41733b7090b126cc740234b1a850/rsopt/configuration/setup.py#L7

Closed f7ab7e41cdc14eb60262ebd82818360916a15314


Don't conflate hart and system

Heterogeneous systems can have different privilege mode support in different harts, can have different XLEN support in different harts, etc.

Please rebase and fix conflicts (and ping me again if needed), then I'll merge.

@aswaterman: My fork was in some ancient state when I started, causing all kinds of trouble for me.  I think that this is now ready to go.

Thanks.


[BUG] Does not support K3S

What is your environment(Kubernetes version, Fluid version, etc.)
K3S: 1.29.4
Fluid: v1.0
Describe the bug
DaemonSet csi-nodeplugin-fluid can't deploy on k3s.

What you expect to happen:
How to reproduce it
helm install --namespace=fluid-system --create-namespace fluid fluid/fluid
Additional Information

Thank you for finding it. I think it's a feature request.  We didn't test it on K3s before.

I will take a look at this.

The directory for storing the kube config files and certificate files in k3s is different from that in k8s. You can complete the installation by setting the kubeConfigFile and certDir in the values.yaml file.
csi:
  kubelet:
    kubeConfigFile: /etc/rancher/k3s/k3s.yaml
    certDir: /var/lib/rancher/k3s/agent

Sorry, I have not updated the latest situation. I tried copying the configuration file to /etc/kubernetes/kubelet.conf, and then it worked fine. Thanks for everyone's responses.


Add "Limitations" section to the docs

Help set expectations for new users to avoid questions like:

https://github.com/idom-team/idom/discussions/299
https://github.com/idom-team/idom/discussions/341

I think this can be covered in FAQ if not in specific docs.