SLH instantiation does not consistently create operator-valued matrices

Say you're instantiating an SLH object, like this:
slh = SLH(
    [[s00, s01],
     [s10, s11]],
    [l0, l1],
    h,
)

Here, the s00 etc. may be numbers, sympy expressions, QNET ScalarTimesOperator instances, and so on. Since we're giving the matrices as lists, the parameters will be processed internally, to the effect of slh.S = Matrix([[s00, ...]]) * IdentityOperator and so on. Due to the final multiplication, both S and L will be elementwise operator valued, i.e., each element will be a ScalarTimesOperator instance or similar.
However, let's say I decide to call Matrix explicitly myself.
slh = SLH(
    Matrix([[s00, s01],
            [s10, s11]]),
    Matrix([l0, l1]),
    h,
)

Since the S and L arguments are already Matrix instances, they will not be processed further, so in effect, I'm setting slh.S = Matrix([[s00, ...]]) and so on. Hence, each of the elements of slh.S will retain their original types, which may be both scalars (numbers, sympy expressions, etc.) and operators.
Almost everything you can do with the slh object still works -- any scalar is interpreted as a scalar times the identity operator when required. However, the behavior seems a little inconsistent. Instead of the current code,
        if not isinstance(S, Matrix):
            S = Matrix(S) * IdentityOperator
        ...

one could simply do
        if not isinstance(S, Matrix):
            S = Matrix(S)
        S = S * IdentityOperator
        ...

or skip the operator multiplication altogether,
        if not isinstance(S, Matrix):
            S = Matrix(S)
        ...

The real reason this came to my attention, however, is that the one place things break is when comparing SLH objects, or their components, for equality. As it turns out, the current implementation of __eq__ for certain objects within QNET is not symmetric. More on that in another issue.

The unsymmetric relations alluded to in the final paragraph actually turned out to be a bug in sympy, see https://github.com/sympy/sympy/issues/13078. The point is that, in the example above (using slh1 for the first variant and slh2 for the second), slh1 == slh2 is True, while slh2 == slh1 can be False.
The explanation is this: when stepping down the hierarchy, we ultimately end up with comparisons of the type s00 * IdentityOperator == s00. This calls ScalarTimesOperator.__eq__, which explicitly supports these cases, and evaluates to True. However, the reflected version winds up at s00 == s00 * IdentityOperator, and if s00 is a sympy object, this dispatches to sympy.Basic.__eq__, which does not support ScalarTimesOperator objects, and therefore returns False.
The elegant solution would be for sympy.Basic.__eq__ to return NotImplemented when it cannot make sense of the arguments. Then the comparison would delegate to the reflected method on the other object, s00 * IdentityOperator, and a symmetric relation would be guaranteed.
Until sympy fixes this, however, a simple workaround for us is to always enforce operator valued elements, i.e., use the first suggested alternative above.
        if not isinstance(S, Matrix):
            S = Matrix(S)
        S = S * IdentityOperator
        ...

Sounds like you got this nailed down pretty much. Feel free to fix this as you see fit and add a test

The inconsistent typing for Matrix elements caused me some pain as well when PyCalling QNET from Julia! I would welcome the enforcement of operator-valued elements in this case.

Wait... I'm actually not on board with requiring that all the elements of a Matrix are Operators. I'm also using matrices in contexts outside of SLH with scalar entries (to do analytic basis transformations, for example). Sorry if I missed the original point of the report - I thought this was only in an SLH context. Maybe we could add an optional parameter to the Matrix constructor that forces conversion of all elements to Operators?

Sorry, didn't see the latest comments before pushing my fix. I was also thinking only in the context of SLH objects, so this is what I've implemented. The Matrix class is unchanged.
Generally, I think it makes the most sense to leave Matrix alone. It is a matrix class for elements of any type and purpose, and should ideally not have to know about the operator algebra at all (although I know it must in order to support the expand and all_symbols methods).

Then we're on the same page!

Just a note that the underlying sympy issues have now been fixed (see sympy/sympy#13091), so equality/inequality comparison between any scaled IdentityOperator and a sympy scalar will be symmetric as soon as people start using the next sympy release.


Support for memory larger than 2000MB

If we need memory larger than 2000MB (mb=2000), the code "mb << 20" in devices.cc will get a wrong number and lead to a failed allocation, due to the limited range of "int" type. When we change the type of mb from "int" to "size_t", the problem is solved.

Thanks for the pull request! This is actually already fixed in the v2 branch of cnn, which will be merged into the master shortly: https://github.com/clab/cnn/issues/52


Performance Issue - Cache tooks long time

In my case I have used both CacheTarget & FileTarget
In my case I will add 20-30k logs after that I will show some results view window.
So my tasks are completed and logs are added in files and then results are shown
But In CacheTarget, it took longer time atmost 20m for me.
Due to that I can't properly see the logs on UI NLogViewer.
Thanks

Thank you for your feedback! Could you maybe create a fork and adjust it accordingly in the test app so that I can investigate the problem more closely?


[BUG] Storage sample is not resetting http response to be reused

In the sample, after sending the first operation to put a blob, and before sending the next operation to download the blob, we need to re-init the http response structure if we want to re-use the same structure previously used.
If we don't re-init the response, the next response from server is going to be appended at the end of current response, witch is not OK.
We might want to consider calling the response_init in the pipeline?  in case the size of the response is greater than 0?   So then we can re-use a response without customer needing to re-init it ? @JeffreyRichter @gilbertw

I was expecting the pipeline to treat the response object as an object it would overwrite.  My mental model is this is an outparam and its contents should be disregarded.
We should clear the response at the start of the pipeline


Bump WordPress and PHP minimums

Bump WP & PHP Minimums to the following

[ ] WordPress to 5.7
[ ] PHP to 7.4
[ ] Remove any no-longer-needed conditional code for older version support.
[ ] Take a look at adjusting our testing matrices, if applicable.
[ ] Bump the minimums in plugin documentation and header fields.

@Sidsector9 do you want someone else to pick this up?
We are experiencing failed WP minimum (5.2) test with the new PHP 8.2 release which is default at GitHub action.
Seems it will be fixed within the current issue?

[08-Dec-2022 06:12:44 UTC] PHP Fatal error:  Array and string offset access syntax with curly braces is no longer supported in /var/www/html/wp-includes/script-loader.php on line 757

Curly races was deprecated since PHP 7 and now it was turned into an error at PHP 8.2.

@cadic this is fixed but the create-term.js test is failing. Should we fix this in a separate issue?


Lobby Rework

Starting-Lobby Rework

[ ] Lighting
[x] Platform
[x] Window Emissive
[ ] Cityscape

Currently Done:

Platform
Window emissives


enable upgrade of an existing installation

Description of PR

Type of change

Feature Pull Request
Bugfix Pull Request
Docs Pull Request
Fixes an issue

Hi @zab35
Can we make it configurable like we do with the Zabbix Server?
See default: https://github.com/dj-wasabi/ansible-zabbix-server/blob/master/defaults/main.yml#L8
and the Debian task: https://github.com/dj-wasabi/ansible-zabbix-server/blob/master/tasks/Debian.yml#L87 ?


chore: import roboto font in the project

Hi @fraxken 👋
This PR is related to the issue https://github.com/ES-Community/nsecure/issues/14
I hope I did it in the right way 😄
Let me know otherwise.

Hi,
Thanks again for the contribution. This one was important to use Node-secure in offline :)
I tested and i have seen no problem so far... seem perfect to me 👍

Maybe should we add it to the readme or tag the package as "offline" ?

when i say "offline" i mean behind a private npm proxy (so probably with a company proxy who probably restrict most of the call to internet). But at the end you still need to be able to pull all the packages data :)
And I just saw that the keywords were indeed missingin the package.json! I just made a commit to fix this :)


Refactoring Scrolling Behaviour (WIP)

@zhouhua I've simplified the approach for scrollTo as well as adding a "Scroll Behaviour" toggle for "auto, smooth, and instant". It's in a sort of working state, it consistently scrolls to the correct spot and because it's not directly at top 0 (probably because of the inline title) it is shown underneath the sticky header. Need to test without inline title.
Also we need to either add a prettier config or agree upon a code style, my editor defaults to double quotes and is throwing errors all over the place. Perhaps we can setup a prettier config file?
Keen to get your eyes over a few things and see if you can improve the consistency of scrolling to the top most header if at the bottom of the page with smooth scrolling enabled 👍

I also noticed the issue with the sticky header height not updating was because we called and set it prior to it changing, meaning that the reference was old and outdated. If we want to get the correct height, it's probably best to put it in a timeout after we run the scroll function.

@itsonlyjames Since CodeMirror-related APIs are not available in preview mode, I’ve decided to refactor the code over the next couple of days and perform some tedious calculations.

@zhouhua 🤦♂ I did not even consider this when moving into the CodeMirror functionality... that is my bad.
I'm actually going to investigate using Svelte for handling the rendering elements, it should assist in decoupling the functionality and UI elements. As well as it will allow for better styling and animation/fade in/out transitions. I will investigate this while you refactor the calculations.

Check out my latest commit; I've completed the calculation for title scroll top.
Introducing a UI framework is a good idea. Initially, I thought title rendering was relatively simple, so I directly manipulated the DOM. This approach negatively impacts both performance optimization and code readability.

Checking it out now.
Have a look at my new branch, I've implemented Svelte and a store so we can simply push headings.set([]) to render them in there. Will be great. https://github.com/zhouhua/obsidian-sticky-headings/commit/a3f90bde8a0019d2363f698886a608d64395fffc

I have a favorite Obsidian theme called "Border." It's a bit unique in that the editor doesn't have a background color set, so our plugin appears like this:

It seems a bit challenging to completely match the title background color with the editor background color.
So I tried changing the style a bit. Do you think this style looks good?

I'm also a big fan of the Border theme.
I think your implementation looks great!
If you're happy to, we can get this PR ready to merge and then use my https://github.com/zhouhua/obsidian-sticky-headings/tree/feature/SvelteUI branch to do all the styling. I've completely refactored the entire plugin, however it will make it easier to work with moving forward.


Installation failed, reverting ./composer.json to its original content.

C:\xampp\htdocs\php\edv\sb1>composer require intervention/image
Using version ^2.3 for intervention/image
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- intervention/image 2.3.7 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- intervention/image 2.3.6 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- intervention/image 2.3.5 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- intervention/image 2.3.4 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- intervention/image 2.3.3 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- intervention/image 2.3.2 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- intervention/image 2.3.1 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- intervention/image 2.3.0 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
- Installation request for intervention/image ^2.3 -> satisfiable by intervention/image[2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.3.5, 2.3.6, 2.3.7].
To enable extensions, verify that they are enabled in those .ini files:
- C:\xampp\php\php.ini
You can also run php --ini inside terminal to see which files are used by PHP in CLI mode.
Installation failed, reverting ./composer.json to its original content.
Can anyone tell me what is the problem here?

Uncomment extention=ext-fileinfo.dll in C:\xampp\php\php.ini by removing the preceding ;


Should we relicense Xilem and Masonry to the dual MIT/Apache License?

There are a few reasons to do this:

It's our default license.
Some people are interested in having a maximally permissive license.
Because Masonry is intended to be a foundational crate for most of the ecosystem, a maximally permissive license makes sense.

Any reason not to do this?

Zulip discussion
The main reason to not want to is that a proportion of our code comes from Druid. That isn't in itself a problem, but it does make this task much more challenging, as we need to either determine how much of the Druid code we've kept, or also relicense Druid. I think that doing the former is (remarkably) likely to be easier, and has greater advantage for the future. But it does make things a bit more complicated, and increases the likelihood of people being uncontactable.
See https://github.com/bevyengine/bevy/issues/2373 and https://github.com/gfx-rs/wgpu/issues/1473 for prior art


zh_cn file submission

If possible, can you synchronize the zh.cn file with other available Minecraft versions?

Thank you!
Please move the files to
1.20.1/Forge/src/main/resources/assets/enderzoology/lang/zh_cn.json
and
1.20.4/Common/src/main/resources/assets/enderzoology/lang/zh_cn.json

Thank you! Please move the files to 1.20.1/Forge/src/main/resources/assets/enderzoology/lang/zh_cn.json and 1.20.4/Common/src/main/resources/assets/enderzoology/lang/zh_cn.json

Could the author please check this fork
I did it in my own fork, but I'm not sure if you will receive the changes


Duplicate Remover

In this PR, Resolves #33 , it can be broke down to two parts:

Extract jobs description and remove irrelevant header info
Use similarity metric to remove duplicate

Here I choose Jaccard Similarity after comparison:

Checksums (Hashing): Not applicable since contents from different websites will vary a little
Levenshtein Distance (Edit Distance): Not applicable since the computation time for that is too long.
Cosine Similarity: Yes, but since we are searching for jobs with the same job titles, they are likely to contain similar keywords, so even different job posts are likely to have similar cosine similarity. Also, if job posts on different websites refer to the same jobs, they are not likely to have very different length but cosine similarity here are not affected by message length.
Jaccard Similarity: Yes we choose this one because if they refer to the same job, their description should be the same since HR would probably cp pst everywhere. And its computation speed is fast
TF-IDF with Cosine Similarity: No, same reason as cosine similarity that job descriptions for similar jobs will have strong scores even when they refer to different jobs.
Word Embedding models: Rejected for the same reason as 5 and 3


point index off in toPoints.js?

hi @colinmeinke,
thank you for creating this library.
i noticed that some of the code in this repo is also used in the wilderness repo (please correct me if i'm wrong).
when trying to display and animate some imported svg paths with wilderness, i'm getting an error in the toPoints (line 161) function: Cannot read property 'xAxisRotation' of undefined.
i'm not 100% certain, but i think the code in this line is referring to the wrong point index. i guess it should refer to the recently added point. instead of points[ i ][ 'curve' ][ k ] i think it should be points[ points.length - 1 ][ 'curve' ][ k ] (or something like this)
again i'm not 100% sure it's just observation, so i'm filing this as an issue, not a pull request.

Thanks for the report. Can you provide your toPoints input SVG shape object, the output points array, and how that differs from the expected output points array?

here's the shape object that is passed into the toPoints function (please excuse the profanity in the image, unfortunately i couldn't reproduce the error with any other path):
        {
            d:"M68.84,167a3.63,3.63,0,0,1,.33-1.76c1.11-1.07,2.42-2.8,3.61-2.76,2.26,0.07,2.68,2.32,1.8,4a17.48,17.48,0,0,1-4.07,5c-4.62,3.93-10.36,5-16.16,5.88-2.54.38-5.05,1-8.14,1.6,0.86,1.07,1.47,1.48,1.55,2,0.24,1.41.88,3.45,0.22,4.14a6.33,6.33,0,0,1-4.55,1.57c-0.92-.06-2.55-2-2.43-2.83,0.32-2.36,1.45-4.61,2.18-6.93,0.45-1.42,1.3-3,1-4.32-0.9-4.71-2.2-9.34-3.35-14l-1.21-.16c-0.31,1.21-.6,2.42-0.92,3.62s-0.35,2.95-1.18,3.66-2.95.7-3.53,0.09c-0.9-.95-1.72-3.07-1.24-4,1.35-2.66,3.41-5,5.12-7.44,0.68-1,1.77-2.2,1.65-3.19-0.58-4.78-1.54-9.51-2.31-14.27a3.19,3.19,0,0,0-3.15-3.06c-2.73-.38-4.63-2-4.57-5.12A3.94,3.94,0,0,1,33,124.5c2.75-.46,3.95,1.31,4.67,4.21,3.79-2.34,7.37-4.57,11-6.77,4.86-3,9.7-6,14.59-8.91,4.34-2.59,7.19-2.22,9.34,1.32,0.65,1.07.61,2.57,0.89,3.86,4.65-1.28,7.34-.18,10.1,4.15,1.76,2.76,3.59,5.49,5.58,8.08a4.2,4.2,0,0,0,2.73,1.14c1.62,0.15,3.28-.17,4.91-0.06a4.21,4.21,0,0,1,4.13,4.64c0,1.23-1.5,2.7-2.7,3.45-0.42.26-2-1.24-3-2a15,15,0,0,1-1.66-1.9l-1.17.92L102.42,154c2.51-.52,4.21-1.28,5.73-1,0.79,0.14,1.83,2.13,1.76,3.21-0.14,2.2-1,4.66-3.76,4.53a7.21,7.21,0,0,1-4.86-2.07,145.91,145.91,0,0,1-8.41-12.08c-1.43-2.2-2.48-4.64-3.78-6.93-1.65-2.9-3.42-3.52-6.39-1.82a13.93,13.93,0,0,0-5,4.17c-1.78,2.88-4.12,3.24-7,2.85-3.57-.49-4,0-3.81,3.6,0.29,5.26.49,10.52,0.74,15.77,0,0.92.18,1.84,0.27,2.76Zm-10.61-45.9c-6,3.69-11.4,6.87-16.62,10.25a3.35,3.35,0,0,0-.81,2.8q1.15,6.64,2.69,13.2c0.29,1.23,1.31,3.32,1.85,3.28a13.94,13.94,0,0,0,6.28-1.76,64.21,64.21,0,0,0,8-6.63c0.77-.7,1.74-1.91,1.62-2.75C60.39,133.5,59.32,127.59,58.23,121.06Zm-13.15,34.1,4.47,19.08L64,170.55l-2.49-24.27A9.12,9.12,0,0,0,55,152.17c-1.06,2.54-2.62,4.29-5.85,3C48.12,154.83,46.83,155.16,45.07,155.16Zm40.4-22.61-6.24-9c-2.19,1-4.19,2.56-5,2.09-1.43-.79-2.13-2.9-3.46-5-2.23.2-4.14-1.33-5.17-4.68-1.11,1.23-2.31,2.55-2.2,3.76,0.54,5.87,1.4,11.72,2.22,17.56,0.42,3,4.37,4.5,6.42,2.26C75.52,135.78,80,134.11,85.48,132.55Z M254.91,142.87c-1.13-9.27-9.56-15.86-16.42-17.81-0.66-.19-2.1.73-2.48,1.48-2.61,5.19-5.12,10.43-7.42,15.76-0.31.72,0.42,2.41,1.18,3,4.69,3.55,10,6.45,14.17,10.48,2.76,2.64,4.36,6.69,5.84,10.37,0.53,1.31-.37,3.83-1.46,5-4.82,5.36-11,7.87-18.06,5.79-9.89-2.9-19.75-6.3-27.47-13.46-3.62-3.35-6.21-7.81-9.23-11.79a22.54,22.54,0,0,0-1.85-2.57c-0.15,2.63-.54,5-3.7,4.67-1.19-.12-3.25-1.36-3.22-2,0.12-2.12.47-4.69,1.78-6.11,1-1,3.61-.88,5.45-0.71,2.46,0.22,5,1.79,6.69-1.57,0.28-.55,2.23-0.24,3.41-0.33,0,1.63.51,3.56-.18,4.82s-2.58,1.76-4.21,2.77c3,5.11,6.66,9.58,11.92,12.63,1.45,0.84,2.59.95,3.53-.76,2.94-5.37,5.94-10.71,9.11-16.41a51,51,0,0,0-5.67-2.66,23.33,23.33,0,0,1-12.63-10c-4.7-7.29-1.38-16.25,8.14-17.39,6.21-.75,12.8.89,19.11,2,3.58,0.65,7,2.39,10.4,3.79,9.9,4,15.89,11.33,18,21.81,0.09,0.43.22,0.84,0.35,1.26a3,3,0,0,0,.35.49,6.86,6.86,0,0,1,1.72-1.19c1.32-.41,3.6-1.22,3.9-0.78a7.47,7.47,0,0,1,1.46,4.92c-0.17,1-2.31,2.58-3.43,2.47a27.26,27.26,0,0,1-7.51-2.18,3.91,3.91,0,0,0-4.64.62c-2.28,1.88-5.11.92-5.75-1.68-0.55-2.24.1-4,2.64-4.36C250.78,143,252.85,143,254.91,142.87Zm-38.65,23.39c5.28,2.33,10.2,4.68,15.26,6.65a8.11,8.11,0,0,0,5.46.27c2.71-1.1,5.84-2.51,7.4-4.75,1.86-2.67.42-6.19-1.83-8.4a96.18,96.18,0,0,0-10.31-8.48,54.63,54.63,0,0,0-6.49-3.6ZM233,123.09a43,43,0,0,0-20.51-2.83,7.14,7.14,0,0,0-2.16.76c-4.38,2-5.51,5.76-3.11,10,3.73,6.55,10.3,8.55,16.74,11Z M121.22,142.5c-0.43,1.67-.37,3.62-1.22,4.18-1,.66-3.42.71-4.13,0a6.22,6.22,0,0,1-1.29-4.62c0.34-2.51,2.31-3.89,4.86-4.17,1.51-.17,3-0.2,4.54-0.29l-0.69-.81c1.13-1.36,2.06-3,3.48-3.92,0.57-.35,2.7.75,3.28,1.71,1.3,2.13-.17,4.82-4.34,8.07,2,1.23,3.59,2.83,5.48,3.29,5,1.22,10.1,1.94,15.16,2.87,1.83,0.33,2.63-.39,2.62-2.27,0-4.83,0-9.66.14-14.48,0.07-2.41-1.41-2.42-3.17-2.69-5-.76-9.94-1.6-14.85-2.67a12.54,12.54,0,0,1-8.85-7.29,13.69,13.69,0,0,1-1-7.81,24.42,24.42,0,0,1,3.88-8.85c0.95-1.37,3.37-2.28,5.19-2.4,9-.59,18.16-2.25,27.17-0.06,9.18,2.23,17.51,6.16,23.68,14,0.1-.83-0.09-2,0.34-2.31A11,11,0,0,1,186,110c0.4-.05,1.71,2.34,1.57,3.48-0.6,4.68-3.38,6.26-8,4.92-1.15-.33-2.28-0.71-3.28-1a4.31,4.31,0,0,0-.71.5,23.42,23.42,0,0,1-3.12,3.58c-0.41.32-2.44-.46-2.46-0.81a10.3,10.3,0,0,1,.56-5c0.6-1.22,2.26-1.91,3.74-3.06a49.56,49.56,0,0,0-21.7-9c0,4.28-.27,8.29.05,12.26,0.81,9.82.93,9.81,10.69,11.22,2.92,0.43,5.86.83,8.75,1.42a6.58,6.58,0,0,1,5.36,5.21c1.91,8.18.37,14.42-8.53,17.34A50.87,50.87,0,0,1,141,152.17c-1.6-.36-3.25-0.46-4.86-0.78C130,150.21,124.09,148.68,121.22,142.5Zm28-17.19c-0.5-7.17-.85-13.51-1.49-19.81a3.35,3.35,0,0,0-2.6-2,106.79,106.79,0,0,0-13,1.35c-3.71.66-7.17,5.67-7.19,9.61,0,3.17,3.19,6.69,7.08,7.74,3,0.8,5.88,1.72,8.88,2.28C143.39,125,146,125,149.21,125.32Zm5.2,4.85c-0.43,5.72-.72,10.84-1.25,15.94-0.28,2.71.94,3.87,3.32,3.4,3.72-.72,7.34-2,11.06-2.78,4.84-1,5.69-4.22,5.55-8.48-0.17-5.11-.59-5.66-5.79-6.28-3-.36-6-1-9.06-1.47C157,130.31,155.72,130.28,154.42,130.16Z",
            type:"path"
        }

due to the to the error, i'm not getting any output.

I've narrowed the error down to this shape:
{
  d: "m-38.65,23.39c5.28,2.33,10.2,4.68,15.26,6.65a8.11,8.11,0,0,0,5.46.27c2.71-1.1,5.84-2.51,7.4-4.75,1.86-2.67.42-6.19-1.83-8.4a96.18,96.18,0,0,0-10.31-8.48,54.63,54.63,0,0,0-6.49-3.6ZM233,123.09a43,43,0,0,0-20.51-2.83,7.14,7.14,0,0,0-2.16.76c-4.38,2-5.51,5.76-3.11,10,3.73,6.55,10.3,8.55,16.74,11Z",
  type: "path"
}

@snorpey you were totally right, thanks for the catch!


Building package openjpeg:x64-osx failed

Hi everyone,
Package: openjpeg:x64-osx
Vcpkg version: 2018.11.23-unknownhash
I'm trying to install vtk[qt,python] but I have an issue at the begining of the installation:
Starting package 1/67: openjpeg:x64-osx Building package openjpeg[core]:x64-osx... -- Using cached /mypath/vcpkg/downloads/uclouvain-openjpeg-v2.3.0.tar.gz -- Using source at /mypath/vcpkg/buildtrees/openjpeg/src/v2.3.0-711dc321d0 -- Configuring x64-osx-dbg CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):     Command failed: /mypath/vcpkg/downloads/tools/cmake-3.14.0-osx/cmake-3.14.0-Darwin-x86_64/CMake.app/Contents/bin/cmake;/mypath/vcpkg/buildtrees/openjpeg/src/v2.3.0-711dc321d0;-DBUILD_CODEC:BOOL=OFF;-DOPENJPEG_INSTALL_PACKAGE_DIR=share/openjpeg;-DOPENJPEG_INSTALL_INCLUDE_DIR=include;-DCMAKE_MAKE_PROGRAM=/mypath/vcpkg/downloads/tools/ninja-1.8.2-osx/ninja;-DCMAKE_SYSTEM_NAME=Darwin;-DBUILD_SHARED_LIBS=OFF;-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=/mypath/vcpkg/scripts/toolchains/osx.cmake;-DVCPKG_TARGET_TRIPLET=x64-osx;-DVCPKG_PLATFORM_TOOLSET=external;-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON;-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON;-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON;-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE;-DCMAKE_VERBOSE_MAKEFILE=ON;-DVCPKG_APPLOCAL_DEPS=OFF;-DCMAKE_TOOLCHAIN_FILE=/mypath/vcpkg/scripts/buildsystems/vcpkg.cmake;-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON;-DVCPKG_CXX_FLAGS=;-DVCPKG_CXX_FLAGS_RELEASE=;-DVCPKG_CXX_FLAGS_DEBUG=;-DVCPKG_C_FLAGS=;-DVCPKG_C_FLAGS_RELEASE=;-DVCPKG_C_FLAGS_DEBUG=;-DVCPKG_CRT_LINKAGE=dynamic;-DVCPKG_LINKER_FLAGS=;-DCMAKE_INSTALL_LIBDIR:STRING=lib;-DCMAKE_INSTALL_BINDIR:STRING=bin;-G;Ninja;-DCMAKE_BUILD_TYPE=Debug;-DCMAKE_INSTALL_PREFIX=/mypath/vcpkg/packages/openjpeg_x64-osx/debug     Working Directory: /mypath/vcpkg/buildtrees/openjpeg/x64-osx-dbg     Error code: 1     See logs for more information:       /mypath/vcpkg/buildtrees/openjpeg/config-x64-osx-dbg-out.log       /mypath/vcpkg/buildtrees/openjpeg/config-x64-osx-dbg-err.log Call Stack (most recent call first):   scripts/cmake/vcpkg_configure_cmake.cmake:282 (vcpkg_execute_required_process)   ports/openjpeg/portfile.cmake:10 (vcpkg_configure_cmake)   scripts/ports.cmake:71 (include)
I attach the content of the log.
config-x64-osx-dbg-err.log
config-x64-osx-dbg-out.log
I'm working on macOs 10.14.4 with cmake 3.14.3.
Is vcpkg is stable on macOs (I had couldn't install it if I didn't install gcc with homebrew)?

@Athius, from you log, it failed with 'CMake Error: INSTALL(EXPORT) given unknown export "OpenJPEGTargets"', I tried it with latest source, it built successfuly for me, please get vcpkg latest source and try again. Feel free to reopen if it still be a probelm for you.
./vcpkg install  openjpeg:x64-osx
Computing installation plan...
The following packages will be built and installed:
openjpeg[core]:x64-osx
Starting package 1/1: openjpeg:x64-osx
Building package openjpeg[core]:x64-osx...
-- Downloading https://github.com/uclouvain/openjpeg/archive/v2.3.1.tar.gz...
-- Extracting source /Users/vcpkg/vcpkg/downloads/uclouvain-openjpeg-v2.3.1.tar.gz
-- Using source at /Users/vcpkg/vcpkg/buildtrees/openjpeg/src/v2.3.1-62ecc2f533
-- Configuring x64-osx-dbg
-- Configuring x64-osx-rel
-- Building x64-osx-dbg
-- Building x64-osx-rel
-- Performing post-build validation
-- Performing post-build validation done
Building package openjpeg[core]:x64-osx... done
Installing package openjpeg[core]:x64-osx...
Installing package openjpeg[core]:x64-osx... done
Elapsed time for package openjpeg:x64-osx: 9.965 s
Total elapsed time: 9.966 s
The package openjpeg:x64-osx provides CMake targets:
find_package(OpenJPEG CONFIG REQUIRED)
target_link_libraries(main PRIVATE openjp2)


Merge bugfix on main to develop

This pulls in the bugfix from @aoloso to develop

Dang it. Github got confused. Doing a handmerge

Dang it. Github got confused. Doing a handmerge


Import modules individually from RxJS

This is something that @jvantillo mentioned few months ago. I will put them with enough explanation below for further discussions.
Current situation
Currently, we import and use Observables like this:
import { Observable } from 'rxjs';

function doSomething$() {
  return Observable.of(1, 2, 3)
    .filter(x => x % 2 === 1)
    .map(x => x * 2);
}

This means the whole RxJS library is always expected to be loaded, adding to our bundle size.
Recommended approach
import { Observable } from 'rxjs/Observable';
import { of } from 'rxjs/observable/of';
import { map } from 'rxjs/operator/map';
import { filter } from 'rxjs/operator/filter';

function doSomething$() {
  return Observable
    ::of(1, 2, 3)
    ::filter(x => x % 2 === 1)
    ::map(x => x * 2)
}

The snippet above makes use of the bind operator though, which is still in stage-0: https://github.com/tc39/proposal-bind-operator

Below, the discussion is from the side of Travix:
Problem from our side
While the recommended approach works great for single bundles, our case is a bit different. We have multiple bundles for Apps, and a separate single vendors.js bundle that brings in all other third-party libraries (like RxJS), and used as Webpack's externals.
This allows all App developers to be able to use all the features of RxJS, without having to put any RxJS modules in their own bundles.
Possible solution to the problem
To go for a smooth migration, we can keep a list of all possible RxJS import paths, and put them as externals in our Webpack configuration of the bundler. And until everything is imported the recommended way, we can keep loading the full RxJS library via vendors.js.
Once everyone has migrated, we can go further and track the usage of those imports, and reduce the vendors.js size significantly.

Two things to achieve:

Make sure the migration goes smooth
Don't break the existing docs showing how RxJS (and other libs) are loaded from CDN
Don't break the examples

To do that, we need to be able to:

Have a shareable key/value pairs list that can be used by Webpack.

How the list may look like:
Webpack's externals equivalent object:
{
  // RxJS
  'rxjs/Observable': ['Rx', 'Observable'], // window.Rx.Observable
  'rxjs/operator/map': ['Rx', 'Observable', 'prototype', 'map'],

  // lodash
  'lodash/debounce': ['_', 'debounce'], // window._.debounce
}

Question
How can we make this list shareable? All examples, and all our applications need them.
Shall we create a package called frint-webpack-stuff or something? If so, what should be the package's name?

Good idea. But since we shrink the functionality RxJS provides by not bundling it, how do we define the functionality that we need to support (and consequently the one that we don't need to)?

@viacheslaff: there are two ways to look at it.
From the perspective of Frint
We don't. Any time we import some RxJS module in any frint-* package, we make sure that it is listed in the frint-webpack-stuff/externals module, so that the package's own webpack.config.js can access it when producing the dist file (like frint-store.min.js).
When producing dists, it should expect the full RxJS lib as available as window.Rx.
That's where the responsibility ends.
We are supporting both:

CDN users who load rx.min.js and frint-store.min.js separately as script tags, and
Users who bundle both RxJS and frint-store in a single bundle.js

From the perspective of Travix
We have our own bundler. And we load our external libraries in vendors.js file.
When Apps are bundled, we will be feeding the frint-webpack-stuff/externals config to Webpack, and make sure individual imports are not duplicated in the bundles (for modules that are already available in vendors.js file).
If the App developer is using any module that we do not support, it will increase the bundle size (for Apps), and we would notice.
After one quarter of migration, we can figure out what are the most common imports, and only keep them in our vendors. Until then, we provide a graceful migration path to everyone.
RxJS will still be made available fully until the individual import migration is done.


"unexpectedly found nil while unwrapping an Optional value"

Thank you for the awesome library!
Fairly new to Swift and iOS dev, so please bear with me. I'm having trouble with "unexpectedly found nil while unwrapping an Optional value" at the following location:
https://github.com/kishikawakatsumi/KeychainAccess/blob/master/Lib%2FKeychainAccess%2FKeychain.swift#L762
In my app, I'm calling...
let keychain = Keychain(service: Bundle.main.bundleIdentifier!)
if keychain.allKeys().contains("some_key_id") {
    // DO something
} else {
    // DO something else
}

The code was working fine for awhile, but after an unexpected crash during development, .allKeys() started crashing and giving me the nil error. I've tried cleaning and uninstalling to no avail.
I can bypass the error using try keychain.contains("some_key_id"), so it's not urgent, but I would like to figure out why. I don't understand why my code used to work, and now it suddenly doesn't.
I'll be grateful for any help. Please let me know if you need more info. Thank you!
Connie

I am having getting the same error with allKeys recently. This was working just fine up until today. I added the Firebase/Auth pod and thats when I started having problems. When I remove Firebase, I don't have an issue.

@CMoebius I am using Firebase, so maybe that's what's causing it, but Firebase is critical and I must use it. I hope there will be a fix for the issue, but it's not urgent for me, especially since I'm porting to React Native.

@pdoggi I made a change to the code suggested in another issue and it works fine for me. I won't be able to update the pod until it's fixed though.

@CMoebius Thanks for pointing out the code change! I appreciate it!

Duplicate of https://github.com/kishikawakatsumi/KeychainAccess/issues/284


Comma missing in setup.py

there is a comma missing after 'numpy>=1.18.1' in setup.py
This is causing the following error:
error in scDML setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected end or semicolon (after version specifier)
    numpy>=1.18.1anndata>=0.7.6
         ~~~~~~~~~^

Part of setup.py:
install_requires=['numba>=0.51.2',"numexpr>=2.7.1","numpy-groupies>=0.9.14",'numpy>=1.18.1'
    'anndata>=0.7.6','tables>=3.6.1','scanpy>=1.7.2',"umap-learn>=0.4.6","python-igraph>=0.8.2",
     'louvain>=0.7.0',"plotly>=5.2.2","hnswlib>=0.5.2","annoy>=1.17.0","networkx>=2.5",
     'torch>= 1.6.0',"ipykernel>=5.5.6","nbformat>=5.1.3","pytorch-metric-learning>=0.9.95"],
    python_requires='>=3.6',

Thanks !

Sorry about that, I have modified it.


riscv : add riscv qemu virt support and fix fs bit error in mstatus

Hi , I port threadx to qemu-riscv64-virt and run the demo
I also add toolchain cmake scripts , so we can use cmake like arm to generated libthreadx.a
To Test it
First , Maksure  riscv64-unknown-elf- binutils and make is installed  , and qemu-system-riscv64 is also required
cd into example_build work
cd ports/risc-v64/gnu/example_build/qemu_virt

use make to compile the demo (make clean can clean the output)
make

If done successfully, we will find A ELF FILE kernel.elf , use qemu to load it
qemu-system-riscv64 -nographic -smp 1 -bios none -m 128M -machine virt -kernel kernel.elf

If success , you will get different thread output the message like this
[UART0] : Uart Init Done, this is Test output!
[Thread] : thread_0_entry is here!
[Thread] : thread_5_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_1_entry is here!
[Thread] : thread_2_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_0_entry is here!
[Thread] : thread_5_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_0_entry is here!
[Thread] : thread_5_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!
[Thread] : thread_3_and_4_entry is here!
[Thread] : thread_6_and_7_entry is here!

PR checklist

[x] add riscv qemu virt support , add cmake toolchain script to generate libthreadx.a
[x] fix riscv64 contex_resotre bug (when f/d isa is enable)
[x] use weak symbol to invalidate _tx_initialize_low_level function

Hi @Jer6y.
Thank you for your contribution! I am the new Eclipse ThreadX project lead. I am still ramping up, so I couldn't comment right away. I appreciate your patience.
I lack the knowledge and experience to review the code thoroughly myself, so I will do my best to get someone else to look at it in the upcoming weeks. In the meantime, I am curious about the history of this contribution. What led you to write this port? Feel free to contact me privately if you cannot share those details publicly.

@eclipse-threadx/iot-threadx-committers, I need a volunteer with porting expertise to review this contribution.

Hi @Jer6y.
Thank you for your contribution! I am the new Eclipse ThreadX project lead. I am still ramping up, so I couldn't comment right away. I appreciate your patience.
I lack the knowledge and experience to review the code thoroughly myself, so I will do my best to get someone else to look at it in the upcoming weeks. In the meantime, I am curious about the history of this contribution. What led you to write this port? Feel free to contact me privately if you cannot share those details publicly.

Hi,
I am working on this porting task because I found that I can't directly use the ThreadX project on RISC-V. There are some issues with the code in terms of architecture code, and I can't find the way to run ThreadX on qemu-system-riscv64 for verification and learning.


Suggestion: moving affixes to higher-rarity tomes

I think you should be able to move affixes from tomes on lower rarities to higher rarities. For example, you could put a tome with a common affix into an anvil with a rare affix tome to make it a rare affix tome with the common affix. That way in case you used an affix tome of a lower rarity and want to apply it to a higher rarity item, you can. Basically it would be the same as using a rarer affix tome on it at the beginning, just losing the more common affix tome in the process. It would be balanced and I think a pretty reasonable thing to include.

The only importance of the rarity of the tome is what item you can extract affixes from.  You can apply affixes to any valid item from any tome.


Cross-Namespace API Routing Fails with 404 Error in Multi-Namespace Configuration

Description:
I am experiencing an issue where APIs deployed in a namespace other than the one where the WSO2 APK dataplane is running do not respond as expected. The setup involves deploying the dataplane in the apk namespace, and APIs using WSO2 APK CRDs in the tenant namespace. Despite configuring the Helm chart to recognize APIs across both namespaces, API calls to services deployed in the tenant namespace return a 404 error, indicating the resource is not found. However, deploying and calling the API in the apk namespace works as expected. This issue suggests a problem with cross-namespace routing or API recognition in WSO2 APK.
Suggested Labels:
bug, helm, cross-namespace, routing, crd
OS, DB, other environment details and versions:

Kubernetes version: 1.27.2
Helm version: 3.7.2
WSO2 APK version: 1.0.0 (app version 1.16.0)
Environment details: Docker Desktop setup on MAcOS

Steps to reproduce:

Created namespaces apk and tenant.
In the Helm chart values under adapter and commonController, added the configuration:configs:
  apiNamespaces:
    - "apk"
    - "tenant"

Deployed the dataplane part of APK in the apk namespace and ensured all components were running.
Deployed my example API using WSO2 APK CRDs to the tenant namespace with:kubectl apply -f example/ -n tenant

The command output was:httproute.gateway.networking.k8s.io/custom-route-httpbin created
api.dp.wso2.com/http-bin-api created
backend.dp.wso2.com/http-bin-backend created

Attempted to call the API with:curl -k --location 'https://default.gw.wso2.com:9095/http-bin-api/1.0.0/get' --header 'Host: default.gw.wso2.com' --header 'Authorization: bearer [Token]'

The response was:{"code":"404","description":"The requested resource is not available.","message":"Not Found"}

Deploying the API definition in the apk namespace and calling the API returned a successful response.

Console Outputs and Commands:

Deployment command in tenant namespace:kubectl apply -f example/ -n tenant

API call resulting in 404:curl -k --location 'https://default.gw.wso2.com:9095/http-bin-api/1.0.0/get' --header 'Host: default.gw.wso2.com' --header 'Authorization: bearer [Token]'
{"code":"404","description":"The requested resource is not available.","message":"Not Found"}

Successful API call in apk namespace:curl -k --location 'https://default.gw.wso2.com:9095/http-bin-api/1.0.0/get' --header 'Host: default.gw.wso2.com' --header 'Authorization: bearer [Token]'

Returned a valid JSON response from httpbin.org.

API-CRDs.zip

Feature Suggestion:
I would like to suggest an enhancement for future versions of WSO2 APK. Implementing label selectors for namespaces could offer a more dynamic and flexible way to configure dataplane recognition of APIs across namespaces. Instead of using hardcoded namespace names in the dataplane configuration, label selectors would allow the dataplane to automatically recognize and route traffic to APIs in any namespace tagged with a specific label. This approach could simplify configuration management, especially in environments with numerous or dynamically created namespaces, and align with Kubernetes best practices for resource selection and management.

Hi @steveliem ,
We have the API deploy support in the cluster level and the namespace level. In order to get this working can you add the namespace value to the HttpRoute as below?
  parentRefs:
  - group: "gateway.networking.k8s.io"
    kind: "Gateway"
    name: "default"
    namespace: "apk"
    sectionName: "httpslistener"

We value your suggestion and we will consider this for future releases.
Thank you!
Pubudu.

Hi @pubudu538 ,
Thanks for your prompt feedback. Adding the extra **namespace** field is the solution. It works! 👍 Also I've managed to make it work in combination with an own TokenIssuer pointing to Okta, because also a TokenIssuer includes a reference to the Gateway in the apk namespace.
Thank you for taking my feature suggestion.
Kind regards,
Steve

Hi @steveliem,
We haven't documented these sections. Will include this as well. We are mainly using the cluster level and namespace level deployments.
Thank you!
Pubudu.


Stop loading resources from CDN

Closes #1
CURRENTLY BROKEN See: https://github.com/bokub/nopaste/issues/1#issuecomment-993660619

Sorry but no!
The point of NoPaste is to be super light (only 5 files are needed to make it work), not to be a huge repo containing dozens of copies of external dependencies!

@bokub Ah :(
I guess I will maintain my fork then :(


What is the recommended way to do benchmark

There are two folders, python and cpp, what is the relationship between them?

which is recommended to do benchmark?

I want to benchmark the token throughput when using lora. Seems only static batching is supported currently.
Is there any script to benchmark the performance using lora?

Scripts in python is benchmarking the Python runtime of TensorRT-LLM, while cpp includes scripts to benchmark the C++ runtime, which includes support to benchmark static batching and inflight batching implementation.
C++ runtime is more recommended to do the benchmark.
Inflight batching is supported for LoRA. You can find documents to benchmark LoRA here, and more details regarding LoRA here.

Thanks.

Scripts in python is benchmarking the Python runtime of TensorRT-LLM, while cpp includes scripts to benchmark the C++ runtime, which includes support to benchmark static batching and inflight batching implementation.
C++ runtime is more recommended to do the benchmark.
Inflight batching is supported for LoRA. You can find documents to benchmark LoRA here, and more details regarding LoRA here.

Thanks.

Thank you, I'll try that

Scripts in python is benchmarking the Python runtime of TensorRT-LLM, while cpp includes scripts to benchmark the C++ runtime, which includes support to benchmark static batching and inflight batching implementation.
C++ runtime is more recommended to do the benchmark.
Inflight batching is supported for LoRA. You can find documents to benchmark LoRA here, and more details regarding LoRA here.

Thanks.

Hello, seems there is something wrong with the lora feature.
python examples/llama/convert_checkpoint.py --model_dir ${MODEL_CHECKPOINT} \
                              --output_dir ${CONVERTED_CHECKPOINT} \
                              --dtype ${DTYPE} \
                              --tp_size ${TP} \
                              --pp_size 1 \
                              --lora_target_modules attn_qkv \
                              --max_lora_rank ${MAX_LORA_RANK}

the convert_checkpoint.py file has no arguments for --lora_target_modules and --max_lora_rank

@sleepwalker2017 Sorry, the documents under benchmark directory for LoRA is outdated, we will fix them. Please refer to the documents here and try the commands there, which should be updated.

@sleepwalker2017 Sorry, the documents under benchmark directory for LoRA is outdated, we will fix them. Please refer to the documents here and try the commands there, which should be updated.

I got this new issue when generating test data for lora inflight batching
https://github.com/NVIDIA/TensorRT-LLM/issues/1453

@kaiyux I wonder why I cppbenchmark a built engine of fp8 in 4090, the prefill speed seem to be much slower than the results benchmarked by python. Here are the scripts. Thank you.
python /app/tensorrt_llm/benchmarks/python/benchmark.py -m baichuan2_7b_chat --quantization fp8 --mode plugin --world_size 1 --max_input_len 1024 --max_batch_size 8 --max_output_len 50 --batch_size "1;2;4;8"  --input_output_len "1024,1;1024,50"
The scripts for building fp8 engine:
python /app/tensorrt_llm/examples/quantization/quantize.py --model_dir ${WORK_HF_DIR} \                                    --dtype float16 \                                    --qformat fp8 \                                    --output_dir /tmp/ckpts_fp8 \                                    --calib_size 512
trtllm-build --checkpoint_dir /tmp/ckpts_fp8/ --output_dir ${TRT_ENGINE} --remove_input_padding enable   --remove_input_padding enable  --gemm_plugin float16 --gpt_attention_plugin float16 --remove_input_padding enable --context_fmha enable --use_paged_context_fmha enable --paged_kv_cache enable --max_input_len=1024 --max_batch_size=${MAX_BATCH_SIZE} --max_output_len=64
/app/tensorrt_llm/cpp/build/benchmarks/gptSessionBenchmark --engine_dir ${TRT_ENGINE} --input_output_len "1024,1;1024,50" --batch_size "1;2;4;8"

@sleepwalker2017 Do you still have the question? If not, we will close it soon.


Tile layer nullsafety

The following _TypeError was thrown building RawGestureDetector(state: RawGestureDetectorState#23abb(gestures: [tap, long press, scale])):
type '_ControllerStream<LatLng?>' is not a subtype of type 'Stream' of 'stream'

Why are ther so many confilcts?

ah into the wrong brnach, I see

thanks a lot!


build in validation that there is only 1 tumor type per individual in qc report

i have no idea what I should expect, but trying to iterate through a qc report and determining what to do when an individual has multiple tumor types is present is currently undefined. and depending what is desired, can be complex in generating a list of unique instances to process...so until this is better defined by pi, lab, proj man, etc. i think it is best now for to add a validation that there are no individuals in the qc report that have multiple tumor types present in them and if there is, exit out of the submission and notify submitter that this is not allowed. a qc report can only have one tumor type per individual.

should probably do the same thing for normals as well.

as it stands right now should add a validation that every tumor has a normal in the qc report.

this should no longer be necessary when migrating to new sample input manifest


Issue(?) with the sample

I'm following the provided sample and run into this issue(?)
......
val name = inspector.sub(L.Person.name)
What does the L stand for ? Can't get it
Sincerly

L is the name for the generated Object that contains all Lenses for a package. See the Lenses section in the docs for more details. (And feel free to ask if something remains unclear 😉). To get it generated Zoe have to start the build task from gradle.

Sometimes it can happen that your IDE can't see the generated L object which is settled under your build folder. Gradle overall has no problem to find that files and compiles the project fine. I opened an issue at our fritz2 project to investigate this issue. For this rease I close this one here.


AUTH-8: Move usercache from the oauth-apiserver to lib-go

This PR moves the usercache from the oauth-apiserver (https://github.com/openshift/oauth-apiserver/blob/7423fd05978658842ca56236d315e28a4fa6d40a/pkg/tokenvalidation/usercache/groups.go) to library-go so that it is reusable in the oauth-server group sync (https://github.com/openshift/oauth-server/pull/87).
cc @openshift/openshift-team-auth-maintainers

/lgtm
could we add a unit test here as well? the code is base is very small but still.

/approve


Varying languages in responses

Bot responds in language of command given to it.

What if a command name is the same between multiple languages?


[BUG] Add polkachu's price feeder config to x/oracle

Seid version
Show us output of seid version --long | head
Chain ID
Which chain are you running into issues with?
Describe the bug
A clear and concise description of what the bug is.
Polkachu's playbook was a life saver for me since I couldn't find a proper price feeder config for it. Would be good to add it to sei-chain/oracle or somewhere close to the repo
https://github.com/polkachu/cosmos-validators/commit/bb6c221cd54b921778b86e481f99b4bd45d99bd7
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Config added here: https://github.com/sei-protocol/sei-chain/pull/762


Correcting citation

One of the references was inconsistently referenced in different
functions.

Accepting as it does not need code verification.


How to save file as bytes

In old mongo extentions ,  I can save content in GridFS by $gridFs->storeBytes(). In this library how to save byte content as file and read ? unfortunetly I can not find documentation for this.
Thanks your helps

We've yet to document any of the new GridFS classes as we're still waiting for MongoDB's docs team to
migrate our existing manual to a new format (consistent across all of the driver projects).
That said, the new API uses PHP streams, so you would use need to use stream_get_contents() on the resource returned by the Bucket's openDownloadStream() or openDownloadStreamByName() method. On a related note, the downloadToStream() and downloadToStreamByName() methods use the same resources but pipe the file contents directly to a writable stream via stream_copy_to_stream().
Feel free to ask additional questions as needed.

Closing this as I believe my last comment answered the original question. Feel free to open a new issue if you have additional questions.
To reiterate, this is definitely an example we will have in our forthcoming GridFS documentation.


Support Hawaiian dialect on documentation site

Support Hawaiian dialect on https://dotnet.microsoft.com/en-us/apps/maui. Their wikipedia page is above the framework documentation, so it's only fair.

Hi @RobKenis
The dotnet.microsoft.com website is what we call the marketing website, and it's a completely different repo, for which feedback should be left at https://github.com/dotnet/website-feedback/issues
Unfortunately I don't have permissions to transfer your issue to that repo, so I recommend you create a new issue there. Thanks!


spawnFunction needs to use a script snapshot of the parent isolate when starting

Currently when isolate.spawnFunction is called it reloads
the same script that the parent isolate loaded but we don't
have a way of ensuring that the same versions of the script
got loaded in the child isolate.
Instead of loading scripts the spawnFunction should use
a script snapshot of the parent isolate.

When this issue was first created we did not have the 'hot reload' feature, I think it is worth discussing the implications of hot-reloading and using the 'Isolate.spawn' API to spawn child isolates

If we spawn child isolates using the Isolate.spawn API and then hot reload the parent isolate or a child isolate we could potentially have isolates running different versions of the app and any messages passed consisting non primitive types could result in weird behavior
if we spawn a child isolate after a hot reload should it use the newly modified app script or the original one?

Getting this right in all situations would require having some kind of check pointing and getting all the isolates in the group reloaded with the new modified source. I don't believe it is worth spending engineering cycles trying to implement this corner case.
Here is what I propose

if we spawn child isolates and then hot reload an isolate in the group then only primitive values (null, num, bool, double, String), instances of SendPort, and lists and maps whose elements are any of these will be allowed in the messages to that isolate
if a spawn is done after a hot reload is done we will spawn the child isolate using the original app script an informational message could be produced recommending a restart in this case. Again in this case only the restricted set of messages would be allowed.

Opening a new issue #35302 for the hot reload issue per Liam's comment.


(Unity) Can't Import Steam Audio v4.4.1 - Endless Import Loop

System Information
Please provide the following information about your system:

Steam Audio version: 4.4.1 (Unity)
Unity version: 2022.3.10f1
Operating System and version: Windows 10

Issue Description
It is not possible for me to import Steam Audio v4.4.1 because this causes the Unity Editor to get stuck in an endless import loop. v4.4.0 does not have this problem. Over and over again a dialog pops up that says "Reloading Domain", then "Importing Assets", then "Completing Domain", and this repeats endlessly until I quit Unity. This makes the Unity Editor unusable until Steam Audio v4.4.1 is removed from the project. Luckily I was able to remove the plugin easily using Unity Version Control (Plastic SCM) and my temporary fix is to instead import Steam Audio v4.4.0 which does not have this problem.
Steps To Reproduce
Steps to reproduce the behavior:

Create a new Unity project with Unity v2022.3.10f1
Set platform to Android, IL2CPP scripting backend, and ARM64 architecture.
Import Bakery GPU Lightmapper (https://assetstore.unity.com/packages/tools/level-design/bakery-gpu-lightmapper-122218)
Import Steam Audio v4.4.1 via the .unitypackage (select all assets)
Observe endless import loop, which makes Unity unusable.
Quit Unity and remove Steam Audio v4.4.1 from the project.
Re-open Unity and import Steam Audio older version v4.4.0
Observe no issue with v4.4.0

EDIT: There appears to be a conflict with Bakery GPU Lightmapper only for v4.4.1 but not for v4.4.0. I have edited the above "Steps to Reproduce" and will attach a repro project below.

@lakulish I have traced the issue to some sort of a conflict with Bakery GPU Lightmapper (a very widely-used baked lighting tool for Unity). I have also updated the "Steps to Reproduce" in the original post.
Here is a link for a repro project for you to test with: https://www.dropbox.com/scl/fi/zo90hpde5c7gnxosrcanl/SteamAudio_v4-4-1_Repro.zip?rlkey=o1ykdovajastm6tioidf5usdx&dl=0
Steps for using the repro project:

Unzip the project and add it to Unity Hub.
Open the project. NOTE: In Unity Hub, be sure to select 'Android' platform when first opening the project by selecting 'Editor Version' and then 'Android' platform.
Import Steam Audio v4.4.1 (via the .unitypackage release)
Observe endless import loop.
Close Unity (force close if needed).
Delete Steam Audio plugin from the project.
Import older version Steam Audio v4.4.0
Observe no issue.

Could you please help me determine what's wrong here, and how to fix? Is there some sort of assembly definition conflicting with Bakery or something?

@ForemanDev Can you try importing SteamAudio.unitypackage from v4.4.1, but when the Import dialog box opens, uncheck Assets/Plugins/SteamAudio/Scripts/Editor/Build.cs? Let me know if this resolves the issue.

Yes, excluding 'Build.cs' does solve the issue. Looks like there was an 'InitializeOnLoad' method added to this script in v4.4.1 that might be inadvertently causing the problem?

@ForemanDev Can you try importing SteamAudio.unitypackage from v4.4.1, but when the Import dialog box opens, uncheck Assets/Plugins/SteamAudio/Scripts/Editor/Build.cs? Let me know if this resolves the issue.

@lakulish Yes, excluding 'Build.cs' does solve the issue. Looks like there was an 'InitializeOnLoad' method added to this script in v4.4.1 that might be inadvertently causing the problem?

@ForemanDev Yes, looks like the new InitializeOnLoad method is somehow causing this issue. We will investigate further, but in the meantime, good to know that the workaround helps.

Ok, thank you for investigating, @lakulish! The Unity project I linked above should be useful for you to use for testing, as you can observe the issue simply by unzipping that project, opening it, then importing the Steam Audio plugin.
For avoidance of doubt, is it actually a safe workaround to fully delete/exclude 'Build.cs' or will that cause build errors when attempting to build the project?
Or would it be a better workaround to import v4.4.1 while excluding 'Build.cs', then after that import the older 'Build.cs' script from v4.4.0?

While the cleaner workaround is to use Build.cs from 4.4.0, it only contains code that is used when generating SteamAudio.unitypackage, so it should be fine to just leave it out.

Ok, thank you for clarifying!

Since #280, we can probably allow developer to toggle SteamAudio manually instead of trying to add the symbols automatically (in case some prefer disabling them on Android only). There probably isn't a reliable way to automatically deal with define symbols unless you are willing to write a config file in project directory which I personally dislike.
How about:

Flipping the define symbols to DISABLE_STEAMAUDIO
Add a routine to check whether a native plugin exists in current selected platform
Call the routine in Inspector OnEnable, if the native plugin does not exists, skip drawing the inspector and prompt with a button to add the define symbol to the current platform
And of course add another button under DISABLE_STEAMAUDIO

If this sounds okay I could try to implement it if anyone haven't already been working on this.

@ForemanDev We were able to repro the issue, and it wasn't anything specific to Bakery. Steam Audio was not setting the defines in the correct way, causing Steam Audio and Bakery to keep trying to add their defines in an infinite loop. This would have occurred with any plugin that used an InitializeOnLoad method to set defines.
We have made a fix for this issue, which will be part of the next release.
@TsFreddie We can take a look at your pull request as well, just in case there's some other use case that our fix may not account for. Thanks!

@lakulish Thank you so much for fixing this!

I've had the same issue while importing Steam Audio v4.4.1 into an existing project.
I can confirm that unchecking Build.cs in the import dialogue stops the domain refresh/import loop.

I've been testing quite a bit today and I noticed the following. If I install SteamAudio in an empty project, it installs fine. However, if I install it in a project that already has post-processing in the built-in pipeline, the installation gets stuck in amn infinite loop. I'm going to try your workaround.

I had taken out the Build.cs from my import and that stopping the infinite import loop glitch, but when I merged with main, other individuals starting having infinite import loops as well.

We have just released v4.5.0, which contains a fix for this issue: https://github.com/ValveSoftware/steam-audio/releases/tag/v4.5.0.


drawRectangle and border

The parameters strokeColor and fillColor do not have the same type. strokeColor has a default value while fillColor is undefined. This allows to draw an empty rectangle without having to specify any parameter which may be practical.
However for consistency it could be better to have both undefined.
https://github.com/image-js/image-js-typescript/blob/383d96bfdb8e6b5fb2f0284002507951e81d1ffa/src/draw/drawRectangle.ts#L32-L40
I would also refactor this part
https://github.com/image-js/image-js-typescript/blob/383d96bfdb8e6b5fb2f0284002507951e81d1ffa/src/draw/drawRectangle.ts#L82-L123
To have
if (color) {
 ...
}

if (fill) {
 ...
}

In order to avoid duplicate code.

Conclusions: also accept none for fillColor so that the types are coherent.

Here are the profiles.

Here are the profiles.

Shouldn't this have rather been in your PR? https://github.com/image-js/image-js-typescript/pull/310


[NEW ICON] Assembly Logo

Skill Name
Assembly
Why?
Assembly is everything. At the moment, I'm creating a bootloader with assembly, and it would be perfect to add assembly as one of my skills for GitHub. I'm not sure if there is a logo; the logo could be ASM text in robot font.
Thank you!
A.R
Reference Image
No response

I certainly agree! Some of my projects are in assembly, yet I can't find an icon to represent it.

I certainly agree! Some of my projects are in assembly, yet I can't find an icon to represent it.

I think ASM text would be fine but a logo would be cool!


FDS-15 Add support for Form to Cascara

New Component Checklist

[ ] Includes unit tests for ALL required FDS use cases
[ ] Does not mute any top level API props in React
[x] Includes an explanation for any changes to Jest snapshots (should be a PR tag automatically added if this happens)
[ ] Component uses React.forwardRef() when a single DOM node is returned, AND it makes React-sense to access that DOM node

To Do

[ ] Implement the action function for the form
[ ] Set up the Form-specific FormActionEdit module

@lordpixel you can take a look at this now. Form will be a LOT more simple than Table, and it builds on top of some of the good patterns we have there.
This still is missing the ability to submit and the ability to define an edit action module, but otherwise this might be pretty close.
I think we will need to also be able to define FormRow ratios at some point in configuration, but for now I think this is basically what we need.
cc @CaffeinatedAllie

@lordpixel you can take a look at this now. Form will be a LOT more simple than Table, and it builds on top of some of the good patterns we have there.
This still is missing the ability to submit and the ability to define an edit action module, but otherwise this might be pretty close.
I think we will need to also be able to define FormRow ratios at some point in configuration, but for now I think this is basically what we need.
cc @CaffeinatedAllie


spoc record --no-proc: include all namespaces

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This PR builds on #2296: Since we already filter by program name when running spoc record, we don't need to limit ourselves to a single mntns. This way we don't have to apply somewhat brittle heuristics to find the correct namespace. One result of this is that we can remove the --sleep in the test!
Which issue(s) this PR fixes:
None
Does this PR have test?
Yes.
Special notes for your reviewer:
Does this PR introduce a user-facing change?
NONE

/lgtm cancel
internal/pkg/cli/recorder/recorder.go:307:3: type assertion must be checked (forcetypeassert)
		spec = profile.(*apparmorprofileapi.AppArmorProfile).Spec

/test e2e-flatcar

/test pull-security-profiles-operator-test-e2e

/test pull-security-profiles-operator-test-e2e


Support setting appBundleId for macOS App Store packages

As nativefier using electron packager---> it has the option to provide the appBundleId where we can provide the com.companyname.*** . ***
Currenlty as nativefier not supporting this property, electron packager creating default value com.electron.appName

@lmssrinivas why?

Let's say I am preparing a package for the MacOS
the package nativefier creates has the appBundleID: com.electron.appName--> that's because as nativefier doesn't provide any appBundleID -option  to electron-packager--> the electron-packager always goes with default value that is com.electron.appName
appBundleID is the key/AppID for the build we are creating for the store!

Adding reference to above
https://github.com/electron-userland/electron-packager/blob/master/usage.txt#L70

any news on this ?


Update template changelog to reflect actual package version

See title.

API change check
API changes are not detected in this pull request.


The merged video isn't played properly on Exynos processor based devices.

Hello.
I am trying to cut and merge over two sections. This project works fine on devices under Samsung Galaxy S6. But it is working strangely on Galaxy S7 & S7 Edge.
There are two processors in Galaxy S7x series. One processor is Exynos and another process is Snapdragon.
All features are working correctly on Snapdragon processor device.
By the way, on only Exynos processor devices (SM-935F, SM-930F), the selected video sections can be merged but the merged video isn't played properly.
Look at this video: isnt_playing_video.zip
I don't know which code I have to fix. Which part should I put the focus in?
Look forward to hearing from you soon.
Best Regards,

Hello,
looking at your video I noticed a similar issue I'm facing trying out the sample apps. Once I cut a video with a start cut point different from zero I get a freezed video in output. Only if the cut starts from zero then the transcoding works properly. As Source Video I tried both from the Android native camera app and a grabbed video using the sample. Audio is playing correctly instead.
Did you do any progress?


prepare R code to create PLOTS from HISTORY, including fix of numbering problem

@ValentineHerr, you're already working on this...

rewrite this script in R to facilitate updates.

@teixeirak,
I see 6 duplicated lines in PLOTS (see lines 2725 to 2736).  I am assuming it is a mistake but I want to double check that I am not missing anything.

Yes, that was an error. Now fixed. This is why we're writing R code to generate PLOTS!

I realized that some column names in the output of the R script that converts HISTORY to PLOTS are not good. I need to deal with this so I am reopening this issue.

@teixeirak ,
DUKE FOREST LOBLOLLY PINE_MANAGED LOBLOLLY PINE PLANTATION
have 2 records for establishment of oldest trees in HISTORY. Is it a mistake? The code to create PLOTS from HISTORY does not handle that.


Display global message if maxTaskLag > 3min

Future note: We'll have to gate this behind a configuration option for the UI so it doesn't show up to open source users without email notifications set up.

🚢


Begin tracking the JSPI Wasm feature

Closes #7075. Begins tracking the status of the WebAssembly JavaScript Promise Integration specification. Thanks!

I've updated the browser versions with the specific flags required to enable the feature to be used. I've also filed a sibling PR for Firefox JSPI support on the WebAssembly website here: https://github.com/WebAssembly/website/pull/384. I believe the data here now is fully up to date, and accurately represents current support.
cc/ @yurydelendik just a quick FYI that I've also submitted JSPI support to also be tracked here.


Differentiate usages

Is your feature request related to a problem? Please describe
Currently, we treat all usages of an API element the same (and might miss some).
Desired solution
Differentiate types of usages:

References in expression
References in types
Calls
Member access
Implicit usage of default value
...

In the usage count store we could store a dictionary of numbers rather than a single number:
{
  "total": 10,
  "called": 8,
  "referenced": 2
}

Why is it a problem, that we treat all usages of an API element the same? In addition, the fact that we might miss some usages is a different issue that is not solved by differentiating usage types. Please clarify the benefit of solving this issue.


DS-457: Table Width Bug Fix

Jira
https://pegadigitalit.atlassian.net/browse/DS-457
Summary
Address bug for tables that aren't as wide as their container.
Details
Change default styles of table to not include a background (because there was an improperly styled flash when undoing the inverse). Compare table body width to table width and apply background and styles if the former is equal to or wider than the latter.
How to test
Pull the branch and check out the temporary demo page I made as well as the real demo pages.

Let's address this when we rebuild the Table component: https://pegadigitalit.atlassian.net/browse/DS-457?focusedCommentId=32787


Verify token from query string

Hi
I had a problem with the authentication in SignalR sending the Token via headers, it seems it can't be done right now in js, because the websockets does not support headers, so all the solutions that I found were to send the token in the Query String.
So I added to look for the Authorization keyword in the query string  and validate that value with the ITokenValidator.
I added a UrlEncodingParser from https://github.com/RickStrahl/WestwindToolkit/blob/master/Westwind.Utilities/SupportClasses/UrlEncodingParser.cs in order to parse the query string

I use this for SIgnalR https://github.com/VQComms/Owin.QueryStringToHeader
Not sure if that helps or hinders

Hi
Well Owin.QueryStringToHeader it's another way to do it, It doesn't hard code the Authorization query parameter name and you can set the parameters that would like to pass to the requestHeader, with the name that you want and it helps as a workaround to authenticate SignalR websockets.
If it's used as a "Combo" Owin.StatelessAuth

Owin.QueryStringToHeader make this PR useless. :)

I think this line should be changed to escape singlePair[1] like this Uri.UnescapeDataString(val); line
I also had a problem, I was sending the Authorization in the query string like this
Authorization=Token [token] and in the query string was sent with + Authorization=Token+[token] encoded as a query string. That's why I added the property DecodeSpacesAsPlusSigns to StatelessAuthOptions in order to {treat the plus sign as space](https://github.com/jjchiw/Owin.StatelessAuth/blob/master/src/Owin.StatelessAuth/UrlEncodingParser.cs#L36)
In summary, I think that this PR is useless if somebody use the "Combo" (`Owin.StatelessAuth

Owin.QueryStringToHeader) and Owin.QueryStringToHeader escape singlePair[1]

:)


Fix ScrollContainer touch-scrolling not working.

Fixes #66554 and fixes #65992.

This PR would fix both Touch emulating not works on Windows and ScrollContainer touch-scrolling not works (for all platform)

Has this PR been merged？ I'm waiting for the fixed version as I'm having the same problem.

This PR is 12 hours old, please be patient.

This PR doesn't work on Android anymore.

This PR does not work on Android, test version 4.0-Beta2

This PR does not work on Android, test version 4.0-Beta2

This PR isn't merged yet. Do you build for Android in local using this branch?

Yes, I modified the source code according to your PR and rebuilt it. Tested in 4.0-Beta1, it works well on Windows, and works well on Android.
When rebuilt under 4.0-Beta2 source code, Windows also worked fine, but when tested on Android, it no longer worked.

Yes, I modified the source code according to your PR and rebuilt it. Tested in 4.0-Beta1, it works well on Windows, and works well on Android.
When rebuilt under 4.0-Beta2 source code, Windows also worked fine, but when tested on Android, it no longer worked

Yes, I modified the source code according to your PR and rebuilt it. Tested in 4.0-Beta1, it works well on Windows, and works well on Android. When rebuilt under 4.0-Beta2 source code, Windows also worked fine, but when tested on Android, it no longer worked

I just compiled based on c477e7c (with this fix merged) and it works as expected on Android.
You can retry using my reproduction project in #66554.

Thanks!


Add more standard tag definitions

Rename the CoreTags API to StandardTags and introduce a distinction between "core", "extended", and "discretionary" standard tags.

NOTE: This library is still in a preliminary implementation phase.  Although these tags are designated as "standard", the syntax/semantics are open to discussion.  If you'd like to propose improvements, please open a GitHub issue!

This PR was published as @microsoft/tsdoc version 0.2.0.


Do not filter on filterList if serverSide is true

Expected Behavior
When the serverSide option is set to true and a filterList is provided for a specific column, mui-datatables should just render the chips in/below the toolbar but not do any filtering on the data itself.
Current Behavior
When the serverSide option is set to true and a filterList is provided for a specific column, mui-datatables still filters the visible rows provided down to whatever rows contain a column value that matches a value in the filterList exactly.
As an example: I have a table which has a column that renders a comma separate list of U.S. states. My filter is a list of U.S. states. My server knows that when I tell it to filter by a state it should include all rows that have that state at least including any rows that also have other states. mui-datatables takes over the filtering even though serverSide is true and further filters down my already filtered results that came from the server to include incorrect results since it only accept exact matches (whereas some rows contain the correct state but among a comma-separate list of other states).
Your Environment

Tech
Version

Material-UI
 3.5.1

MUI-datatables
2.0.0-beta-53

React
16.7.0

@bericp1 Thanks for filing the issue and I agree. Would you like to take this up and send in a PR?

I'll certainly give it a shot if I find the time sometime later this week / this weekend 👍

@bericp1 Good stuff 👍


Installation under Eclipse Oxygen (4.7)

Does this plugin work with Eclipse Oxygen?  I used the install instructions as specified in the README
but Eclipse is not recognizing it.  Any ideas?

Works for me on Eclipse Oxygen. I put it into the /dropins folder and it was there. I know that Max OS has some different directories though.

Im running on Linux (RedHat) if that makes any difference.  I'm not seeing
any errors which is strange.
Wendell
On Tue, Jul 18, 2017 at 9:46 AM, David Sedlar<EMAIL_ADDRESS>wrote:

Works for me on Eclipse Oxygen. I put it into the /dropins folder and it
was there. I know that Max OS has some different directories though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/roundrop/ermasterr/issues/13#issuecomment-316068504,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac3HOtnZBMo6o_lBKv-lkBpCvmpsys0Lks5sPLcugaJpZM4ObVpx
.

--
D. Wendell MacKenzie
Software Developer
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> (cell)

Just to be clear I understood the install process...in my
/opt/eclipse/dropins folder, I have the following structure:
/opt/eclipse/dropins/ermasterr/eclipse/plugins//org.insighttech.er_1.1.13.jar
is that correct?
On Tue, Jul 18, 2017 at 9:46 AM, David Sedlar<EMAIL_ADDRESS>wrote:

Works for me on Eclipse Oxygen. I put it into the /dropins folder and it
was there. I know that Max OS has some different directories though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/roundrop/ermasterr/issues/13#issuecomment-316068504,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac3HOtnZBMo6o_lBKv-lkBpCvmpsys0Lks5sPLcugaJpZM4ObVpx
.

--
D. Wendell MacKenzie
Software Developer
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> (cell)

Well I am using Windows so it's a bit different, but that should be correct. Also I'm using the ZIP-ed edition that I just extracted and ran. So no installer. Maybe try that as well if not already.

Do you have a link to the zip archive?
On Tue, Jul 18, 2017 at 11:10 AM, David Sedlar<EMAIL_ADDRESS>wrote:

Well I am using Windows so it's a bit different, but that should be
correct. Also I'm using the ZIP-ed edition that I just extracted and ran.
So no installer. Maybe try that as well if not already.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/roundrop/ermasterr/issues/13#issuecomment-316095347,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac3HOi5_EOs4AGumWfwkhe0wrF-rrKbBks5sPMrygaJpZM4ObVpx
.

--
D. Wendell MacKenzie
Software Developer
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> (cell)

The zip/tar archives can be downloaded from:
https://www.eclipse.org/downloads/eclipse-packages/?osType=linux&release=undefined

I tested it on my Ubuntu 16.04 LTS:

Downloaded the Eclipse IDE for Java EE Developers (64 bit) tar and extracted it
Downloaded the 64 bit Oracle Java 8 end extracted it into /jre
Put the ermaster jar into /dropins

And it seems to work without issues.

Oh I see now that you put the jar file maybe into the wrong directory:
/opt/eclipse/dropins/ermasterr/eclipse/plugins/org.insighttech.er_1.1.13.jar
Should be directly under dropins:
/opt/eclipse/dropins/org.insighttech.er_1.1.13.jar

Ya, that's where I now have it.  Still is not installing it when I
restart.  This may be a bug on the Linux 32 bit version I guess.
Oxygen is just out...so...
On Tue, Jul 18, 2017 at 11:35 AM, David Sedlar<EMAIL_ADDRESS>wrote:

Oh I see now that you put the jar file maybe into the wrong directory:
/opt/eclipse/dropins/ermasterr/eclipse/plugins/org.
insighttech.er_1.1.13.jar
Should be directly under dropins:
/opt/eclipse/dropins/org.insighttech.er_1.1.13.jar
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/roundrop/ermasterr/issues/13#issuecomment-316103486,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac3HOhiqDLcSu3Ad6b5w5s1sc_GEgjzVks5sPNDRgaJpZM4ObVpx
.

--
D. Wendell MacKenzie
Software Developer
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> (cell)

It's possible but also unlikely. Did you have luck with the tar distributions? And instead of using the system Java you could also use an extractable Oracle distribution (extracted under /jre) like my test case scenario.

I'm using the following java version:
java -version
java version "1.8.0_91"
Java(TM) SE Runtime Environment (build 1.8.0_91-b14)
Java HotSpot(TM) Server VM (build 25.91-b14, mixed mode)
RE: the tar distribution...where is that downloadable from?  The
sourceforge site for ERmaster is no longer functional to install from.  I'm
trying the
to use the ermasterr one but it looks like its only installable via the
dropins method.
On Tue, Jul 18, 2017 at 2:00 PM, David Sedlar<EMAIL_ADDRESS>wrote:

It's possible but also unlikely. Did you have luck with the tar
distributions? And instead of using the system Java you could also use an
extractable Oracle distribution (extracted under /jre) like my test case
scenario.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/roundrop/ermasterr/issues/13#issuecomment-316129218,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac3HOlqKx0yS0J8-sv52DN6wS8j22gqCks5sPOTCgaJpZM4ObVpx
.

--
D. Wendell MacKenzie
Software Developer
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> (cell)

RE: the tar distribution...where is that downloadable from? The sourceforge site for ERmaster is no longer functional to install from. I'm trying the to use the ermasterr one but it looks like its only installable via the dropins method.

I meant the tar distribution of Eclipse.

tried it using the eclipse-cpp-oxygen-R-linux-gtk.tar.gz one...same behaviour.

missing dependency of the GEF framework.  used eclipse -clean -debug -consoleLog
with a .options file in /opt/eclipse containing:
org.eclipse.equinox.p2.core/debug=true
org.eclipse.equinox.p2.core/reconciler=true
this puts nice diagnostics of why the dropins jar was not installing.

but...I installed the GEF 5.0.0 packages and still issues the WARNING message in the debug output stating that org.insightech.er_1.1.13 dependency on GEF 0.0.0
Wonder where the 0.0.0 dependency is coming from?

tried it using the eclipse-cpp-oxygen-R-linux-gtk.tar.gz one...same behaviour.

I tested it with eclipse-jee-oxygen-R-linux-gtk-x86_64.tar.gz (Eclipse IDE for Java EE Developers). Maybe you can try that one?

Wonder where the 0.0.0 dependency is coming from?

This is because no version (range) has been specified for org.eclipse.gef in META-INF/MANIFEST.MF.

but...I installed the GEF 5.0.0 packages [...]

Probably, org.insightech.er requires GEF 3.11 instead of GEF 5.

David, can you confirm what version of the gef framework you installed on the Ubuntu LTS install?

downgrading to 3.11 legacy fixed the issue.


Release process for Swift Package Manager

Implement support for Swift Package Manager. There should be a single Package.swift file in the repo which includes the entirety of the SDK. During the release process there should be a job which tags the repository at the appropriate commit sha so that Swift PM can locate the version of the package it needs.

@mitchdenny Was there anything else we needed to do on this item? Perhaps add / update READMEs to cover the release process? IIRC, the SwiftPM process we landed on was working as expected, the only thing left was to automate the release tagging process.

Yep. Still need to automate that. One of the things that I've been pondering is the best way to track the version number. Unlike our binary composed language/runtimes, the version number is usually baked into the sources. Whereas in Swift PM its really just a commit sha (directly) or a tag (indirect point to commit sha).
One option is not storing it in the repo at all and instead allowing a release pipeline to specify a variable with the version number which then just tags the repo.
Alternatively we could put a file (eng/version.txt) in the repo which contains the version to be released and that would be picked up and used by the pipeline and for any other automation fairly easily (just pipe the file into a variable). It also means that the version number to be published is something that is reviewed via PR process rather than an arbitrary decision that someone makes when they kick off the pipeline for release.
Finally, I'm assuming we are going to need README.md and CHANGELOG.md files in the repo. I think we need to decide at what level we are going to do that for the iOS repository. In other repos we have a CHANGELOG and README per module, but in other repos they also ship independently. Perhaps in iOS we just have one README.md and one CHANGELOG.md in the root of the repo since that is the shippable unit.
If we did that then I would be inclined to lift the tag from the most recent entry in the changelog, effectively forcing a changelog entry to be updated prior to a release working ;)

We have one README per module, and one at the top-level. I think that makes sense since each library has its own long-form documentation. I would be fine having a single top-level CHANGELOG if the entire repo is versioned together. We should break down per-library changes in subheadings under each revision in the changelog. I would be fine using the changelog as the source of truth for the version. The process would then be to cut a release which tags the current HEAD commit, and then commit a new changelog entry with the next version number heading and the message "Unreleased" or some other placeholder text.

Closing as it is duplicated with #532

(commentary around README and CHANGELOG is useful though)


In non-GF (export), Terraform plan shows many discrepancies even after running all import scripts

We are using Oracle delivered cd3toolkit to export OCI configuration to excel file and Terraform scripts.
We ran all import shell scripts.
But Terraform plan shows many discrepancies like below -
Plan: 54 to add, 10 to change, 54 to destroy.
Since we did not change any configuration on the OCI outside Terraform before the export, we expected that import shell scripts would sync up the state files so Terraform plan was not supposed to show any mismatch.

We are using Oracle delivered cd3toolkit to export OCI configuration to excel file and Terraform scripts. We ran all import shell scripts. But Terraform plan shows many discrepancies like below - Plan: 54 to add, 10 to change, 54 to destroy.
Since we did not change any configuration on the OCI outside Terraform before the export, we expected that import shell scripts would sync up the state files so Terraform plan was not supposed to show any mismatch.

Hi Suruchi,
We are exporting following resources -
-rwxr--r--. 1 cd3user cd3user  2019 Jun 21 06:39 tf_import_commands_compartments_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  1448 Jun 21 06:40 tf_import_commands_groups_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  1801 Jun 21 06:40 tf_import_commands_policies_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  2216 Jun 21 06:40 tf_import_commands_users_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  2050 Jun 21 06:40 tf_import_commands_tags_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  5353 Jun 21 06:40 tf_import_commands_network_major-objects_nonGF.sh
-rwxr--r--. 1 cd3user cd3user   566 Jun 21 06:40 tf_import_commands_network_dhcp_nonGF.sh
-rwxr--r--. 1 cd3user cd3user    91 Jun 21 06:40 tf_import_commands_network_vlans_nonGF.sh
-rwxr--r--. 1 cd3user cd3user   801 Jun 21 06:40 tf_import_commands_network_subnets_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  1422 Jun 21 06:40 tf_import_commands_network_secrules_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  1368 Jun 21 06:41 tf_import_commands_network_routerules_nonGF.sh
-rwxr--r--. 1 cd3user cd3user   533 Jun 21 06:41 tf_import_commands_network_drg_routerules_nonGF.sh
-rwxr--r--. 1 cd3user cd3user    89 Jun 21 06:41 tf_import_commands_network_nsg_nonGF.sh
-rwxr--r--. 1 cd3user cd3user 30630 Jun 21 06:41 tf_import_commands_dns-views-zones-records_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  1380 Jun 21 06:42 tf_import_commands_dns-resolvers_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  6329 Jun 21 06:42 tf_import_commands_instances_nonGF.sh
-rwxr--r--. 1 cd3user cd3user  9489 Jun 21 06:44 tf_import_commands_blockvolumes_nonGF.sh
-rwxr--r--. 1 cd3user cd3user   976 Jun 21 06:44 tf_import_commands_fss_nonGF.sh
-rwxr--r--. 1 cd3user cd3user   391 Jun 21 06:44 tf_import_commands_buckets_nonGF.sh
-rwxr--r--. 1 cd3user cd3user   307 Jun 21 06:45 tf_import_commands_dbsystems-vm-bm_nonGF.sh
Though I can't share Terraform Plan output as it has customer data, following resources are getting replaced -
Instances
Block Volume attachments
DB system
Backup policies attached to each BV
DNS resolvers and FWD

You will have to see what is causing the replacement. eg for instances it can be metadata - in that case you can put ignore lifecycle changes for metadata in main.tf. If instance replacement is fixed then the corresponding BV and BV attachment replacement will also be fixed.

Plan shows that Instance is being force replaced due to subnet ocid mentioned in create_vnic_details section of the instance.
But I don't understand why, because that subnet ocid is correct, matches the current value from the console.

Have u exported network and ran import cmds for network too?
I see that you are using single outdir for all components - any specific reason for not using multi outdir?

and i hope you dont have duplicates for subnet names in the console. Toollkit doenst support duplicate resource names

Have u exported network and ran import cmds for network too? I see that you are using single outdir for all components - any specific reason for not using multi outdir?

Yes, I Have exported network and many other resources as mentioned before and ran import scripts for them all.
I'm using single outdir for simplicity.
No, we don't have duplicates for any resources in OCI.

We are getting below type of errors after clearing the TF state and running all TF import scripts again -
│ Error: Resource already managed by Terraform
│
│ Terraform is already managing a remote object for
│ module.drg-route-distribution-statements["drg-hub-lhr_drg-rt-dis-lhr_statement3"].oci_core_drg_route_distribution_statement.drg_route_distribution_statement.
│ To import to this address you must first remove the existing object from the state.
Is this an issue?
Also, shall we try putting ignore lifecycle changes for metadata in main.tf?

You will get resource already managed errors if you try to import the same resource into tfstate again. This is not an issue and can be ignored.
And if you are getting force replacements because of metadata - then yes you can put it into ignore lifecycle changes in main.tf (at //terraform_files//modules/compute/instance/main.tf)
lifecycle {
ignore_changes = [create_vnic_details[0].defined_tags["Oracle-Tags.CreatedOn"], create_vnic_details[0].defined_tags["Oracle-Tags.CreatedBy"],metadata]
}

If you are getting force replacements because of subnet_ocid, can u try commenting 'depends_on' at line 9  from instance.tf (at /<outdir>/terraform_files/<region>/instance.tf)

After making this change at line 9 and 17 in instance.tf, terraform plan produces this output.
Plan: 9 to add, 26 to change, 9 to destroy.
All instances are now getting updated in-place, with changes in red (-) for agent_config and defined_tags
~ agent_config {
- plugins_config {
- desired_state = "DISABLED" -> null
- name          = "Oracle Java Management Service" -> null
}
- plugins_config {
- desired_state = "DISABLED" -> null
- name          = "OS Management Service Agent" -> null
}
- plugins_config {
- desired_state = "ENABLED" -> null
- name          = "OS Management Hub Agent" -> null
}
......
......
~ create_vnic_details {
~ defined_tags              = {
- "tag-oci.tag-key-env"   = "non-prod" -> null
- "tag-oci.tag-key-reg"   = "lhr" -> null
- "tag-oci.tag-key-stack" = "file" -> null
}
}
Does it mean these parameter values will be deleted from the instances on terraform apply?
Also other resources like oci_file_storage_mount_target,  oci_dns_resolver_endpoint , oci_database_db_system are still getting updated due to subnet_id.
oci_file_storage_export is getting updated due to  export_set_id
Also, the terraform workvm (which hosts the container) is getting replaced due to metadata.

So agent plugin changes. - you can ignore. it will not disable any plugins if you do terraform apply.
The VNIC defined tags - yes it will replace them to null. To avoid that you can add column called - 'VNIC Defined Tags' in Instances sheet of excel and re-export instances. It will export the VNIC tags as well and put it in tfvars which will not show this change in terraform plan

Thanks @xs2suruchi ,
Can you also respond to other queries which I had posted above.
I have added them below again -
Also other resources like oci_file_storage_mount_target, oci_dns_resolver_endpoint , oci_database_db_system are still getting replaced due to subnet_id.
oci_file_storage_export is getting updated due to export_set_id
Also, the terraform workvm instance (which hosts the container) is getting replaced due to metadata.
Another strange issue which I observed is, if I run
terraform plan -target=oci_core_subnet.subnet
OR
terraform plan -target=oci_core_instance.instance
It shows==>
No changes. Your infrastructure matches the configuration.
Terraform has compared your real infrastructure against your configuration and found no differences, so no changes are needed.
Then why terraform plan without specific target shows discrepancies for these resources?

For replacements happening because of subnet_id - you wil need to comment depends_on = [module.subnets] for all root modules like fss.tf, dns.tf etc like you did for instance.tf
workVM replacement issue because of metadata - should have got resolved by putting ignore lifecycle in main,tf as i showed in one of my previous comments.
Because of module dependency for all respurces combined you are getting the replacements in terraform plan. Doing it via multi outdir is generally easier i guess.
Thanks

Are you able to proceed @PrafullaKGit

Hi @xs2suruchi ,
adding comments for the subnet_id and ignoring metadata, made add/destroy type of changes to 0. There were still 24 updates for different resources, which were mainly due to tags values. Not sure what is the solution for it.
But then I added column called - 'VNIC Defined Tags' in Instances sheet of excel and re-exported entire tenancy.
I will try running the imports after sometime and let you know the outcome,

@xs2suruchi ,
I performed these steps -

added column called - 'VNIC Defined Tags' in Instances sheet of excel
re-exported entire tenancy in multi outdir output destination.
added ignore lifecycle metadata in instance main,tf
ran all import sh

Outcome:
terraform plan for the most of the resources is fine, except below -
changes/updates are shown for network, database and dns due to defined_tags. Please advise on the solution.
changes/updates are shown for compute due to plugins_config (you said we can ignore this , right?)

Good to know that you were able to proceed.
Regarding changes/updates are shown for compute due to plugins_config , you can chose any of the solutions below as per complexity -

Yes you can ignore plugin changes and proceed.
You can add ignore changes for plugins_config in instance main.tf if you don't plan to make changes to plugins via terraform.
And one more solution is to add columns to your Instance sheet for each plugin name(as it appears in console) like 'Plugin Bastion', 'Plugin Vulnerability Scanning' and so on for all plugins and re-export so it will get the exact value for them and process.

Regarding changes/updates are shown for network, database and dns due to defined_tags. Please advise on the solution, -
generally CD3 assigns same tags to the dependent components for a particular sheet eg the 'Defined Tags' column in VCNs sheet will apply these tags to VCN objects as well as the dependent objects like gateways.
So if you are seeing such changes, and you don't want those tags to be applied/removed - you might need to make some manual adjustments in the tfvars. I can suggest better based on the actual data.

Hi @xs2suruchi ,
Please advise how to add ignore changes for plugins_config in instance main.tf.
We were expecting that the tool would read currently assigned parameters values from each resource in OCI and sync the state accordingly. But if the tool is assigning values based on values of parent objects, then it's not correct.
Let us know what data you need to provide the solution.
Also, now, we want to test the tool by deleting an instance from OCI and trying to recreate it using cd3tool.
Can you tell me the process to do it?

the same way u added metadata -
lifecycle {
ignore_changes = [create_vnic_details[0].defined_tags["Oracle-Tags.CreatedOn"], create_vnic_details[0].defined_tags["Oracle-Tags.CreatedBy"],metadata,agent_config]
Can you share the terraform plan for Defined tags changes?
If your excel and terraform state is in synch with OCI, you just need to add/modify/remove the row from Instances sheet of CD3 Excel sheet and re run setUpOCI. To remove a row you can also put it after  tag. that way it wont be processed.
It will re generate the tfvars in compute folder with updated data. And terraform plan will show you the required changes.

@xs2suruchi ,
Below is the plan output for database , with ids masked
Terraform will perform the following actions:
module.dbsystems-vm-bm["CRMDBACCTA"].oci_database_db_system.database_db_system will be updated in-place
~ resource "oci_database_db_system" "database_db_system" {
id                              = "ocid1.dbsystem.oc1.uk-london-1.xyz"
# (34 unchanged attributes hidden)
  ~ db_home {
        id               = "ocid1.dbhome.oc1.uk-london-1.xyz"
        # (8 unchanged attributes hidden)

      ~ database {
          ~ defined_tags        = {
              + "tag-oci.tag-key-cat"   = "mvp"
              ~ "tag-oci.tag-key-env"   = "non-prod" -> "acct"
              + "tag-oci.tag-key-reg"   = "lhr"
                # (3 unchanged elements hidden)
            }
            id                  = "ocid1.database.oc1.uk-london-1.xyz"
            # (10 unchanged attributes hidden)

            # (1 unchanged block hidden)
        }
    }

    # (3 unchanged blocks hidden)
}

Plan: 0 to add, 1 to change, 0 to destroy.

Does your dbsystem has these tags ?
"tag-oci.tag-key-cat"   = "mvp"
"tag-oci.tag-key-env"   = "acct"
"tag-oci.tag-key-reg"   = "lhr"
If so then yes looks like it is applying the same tags as dbsystem to the db_home also.
Generally the requirement is to have all resources properly tagged and that has been a usual requirement to apply same tags to child resources.
In case you don't want that, you can modify the respective main.tf for that
resource (eg modules/database/dbsystem-vm-bm/main.tf) and remove defined_tags and freeform_tags -

hi @xs2suruchi
DB System has these tags -
tag-oci.tag-key-cat: mvp
tag-oci.tag-key-stack: db
tag-oci.tag-key-reg: lhr
tag-oci.tag-key-env: acct

Yeah, correct (similar to what i mentioned above)

Hi @xs2suruchi ,
We deleted and then recreated (with same name but different shape) one VM instance from OCI Console.
Then we ran export_resources workflow.
Running the tf_import_commands_instances_nonGF.sh shows that instance is getting created -
Note: Objects have changed outside of Terraform
...
module.instances["sblol89hard"].oci_core_instance.instance has been deleted
....
Terraform will perform the following actions:
module.instances["sblol89hard"].oci_core_instance.instance will be created
Plan: 1 to add, 0 to change, 0 to destroy.
We expected that running the tf_import_commands_instances_nonGF.sh would synch the TF state with whatever config present in OCI currently and would not show that instance getting created.
What went wrong?

@xs2suruchi ,
Can you please look into above issue and respond quickly?
We are unable to progress.

could you join this public [slack] (https://oracle-devrel.github.io/cd3-automation-toolkit/latest/queries/) so its easier to troubleshoot. It seems like you are importing new instance into the old state file which had details about old instance.

Thanks @xs2suruchi ,
All issues are solved as of now. I could also create a new VM using the toolkit.

Thanks @xs2suruchi ,
All issues are solved as of now. I could also create a new VM using the toolkit.