[Bug]  useCms product associations not working

What happened?
I want to adapt associations of the listing page, so i have access to all images of a product and also its properties. I was able to make it work after pagination as then it uses a different endpoint and other defaults. But on refresh (when it gets its data from the cms as the initialListing) nothing is working.
My useCms associations:
associations: {
  manufacturer: {
    associations: {
      media: {},
    },
  },
  media: {
    sort: [
      {
        field: 'position',
        order: 'ASC',
        naturalSorting: false,
      },
    ],
  },
  productReviews: {},
  properties: {
    associations: {
      group: {},
    },
  },
},

My useListing associations:
associations: {
  media: {
    sort: [
      {
        field: 'position',
        order: 'ASC',
        naturalSorting: false,
      },
    ],
  },
  properties: {
    associations: {
      group: {},
    },
  },
},

Is there something i'm doing wrong maybe? I'm not too familiar with the store api so it could be i'm missing something.
Steps To Reproduce

Try to overwrite default for listing page
Refresh and adaptions are not coming through
Paginate and everything works as expected

Is there an existing issue for this?

[x] I have searched the existing issues

Version
Latest stable (generated by CLI)
Which element is affected?
composables
Environment
- OS: macOS
- Node: 14.x
- Shopware API: <IP_ADDRESS>
- Shopware PWA version: 1.0.1

Relevant log output
No response

Hi @stigvanbrabant!
thanks for being alert. The issue was probably caused bu the problem with images in core API itself. Happily, It was fixed with the Shopware v<IP_ADDRESS> version. Please upgrade you Shopware 6 instance if it's possible 👍🏻

Hello @stigvanbrabant. The reason why the associations for product don't work in page resolver endpoint is the different context.
Currently, page resolver's response may have many of entities, and each one have its own context, and possible filters, associations that can be used. This is why it's not possible to pass association "properties" parameter to the request and get the expected result. Page resolver just can't recognize what entity it's used for (got categories and products and other entities as well). The product-listing context is clear and it's about only a product as a main entity.
The correct context on what associations and filtering can be used with is cmsPage (not a product).
Maybe it's a good point to start a discussion on the problem. For instance, we could add prefixes to the associations that can group the parameters by the entity type to have sth like:
associations: {
"product.properties": {} // tell the page resolver that we need properties associations on product entity if any exists
}

nevertheless, the problem and the solution must be discussed deeply in the wider group 👩🏻 👨🏻
Leave a comment please on what you think 🙏🏻 👍🏻

@mkucmus Yeah this seems like a decent solution to me, certainly would fix the problem that i'm having. So it's actually not currently possible to achieve what i'm trying to do?

@mkucmus Yeah this seems like a decent solution to me, certainly would fix the problem that i'm having. So it's actually not currently possible to achieve what i'm trying to do?

So far, "mixing" associations' context would be possible only programmatic way, like overriding the PWA plugin's logic. I can't see any other solutions on that. But... let's ask @elkmod what he thinks first :)

I'm closing the issue, because the requests' handling should be managed on API side. It's recommended to move discussion on https://github.com/shopware/platform repository or https://github.com/elkmod/SwagShopwarePwa


FatalError: A serious error (Segmentation fault) is detected by the operating system. (at /paddle/paddle/fluid/platform/init.cc:303)

环境：ubuntu 16  cuda 9.0 cudnn 7.6.5 ，运行 python tools/video-enhance.py时会报错：
You are using Paddle compiled with TensorRT, but TensorRT dynamic library is not found. Ignore this if TensorRT is not needed./usr/local/python3.7.1/lib/python3.7/site-packages/pkg_resources/_vendor/pyparsing.py:943: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
collections.MutableMapping.register(ParseResults)
C++ Traceback (most recent call last):
0   paddle::framework::SignalHandle(char const*, int)
1   paddle::platform::GetCurrentTraceBackString()

Error Message Summary:
FatalError: A serious error (Segmentation fault) is detected by the operating system. (at /paddle/paddle/fluid/platform/init.cc:303)
[TimeInfo: *** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1607998503" if you are using GNU date ***]
[SignalInfo: *** SIGSEGV (@0x8a0b00) received by PID 30462 (TID 0x7f9e73213700) from PID 9046784 ***]
Segmentation fault

@zengyang134 How did you solve this issue?


Missing symbols in pycrtm.so

Running the test_atms.py test case, I get the following error:
ImportError: dlopen(/pycrtm/pycrtm.so, 2): Symbol not found: _layeravg_
pycrtm.sp was created successfully by setup_pycrtm.py, but all testCases fail with the same error.

Hmm. What compiler are you using? This is something I've encountered in the past where the layerAvg.patch isn't applied during the install. I had thought at one point about removing this altogether (it's something I was playing with trying to compare RTTOV with CRTM). I've had at least 2 other people build and test it recently.  Is your fork synched to my master?

See Pull request #81 I had this problem previously with intel.

I'm using Homebrew GCC 9.2.0_3.  I can have a look at trying to force the patch application and try a different compiler.
The fork went stale for a while, but I rebased it from your repo before I started working on it this week.
I forgot to mention that I used an existing CRTM installation using the --inline option.

Ah, yeah. That is why I was considering removing that feature altogether. If you use it against an existing crtm library not built by the setup script here, it will break.

My solution for now was to simply comment out the call to LayerAvg in pycrtm.f90:
`
SUBROUTINE applyAvg( Ref_LnPressure, User_LnPressure, Nref, Nuser, Xin, Xout )
USE ODPS_CoordinateMapping
INTEGER,         INTENT(IN) :: Nref, Nuser
REAL(KIND=8),   INTENT(IN)     :: Ref_LnPressure(Nref)
REAL(KIND=8),   INTENT(IN)     :: User_LnPressure(Nuser)
REAL(KIND=8),   INTENT(IN)     :: Xin(Nuser)
REAL(KIND=8),   INTENT(OUT)    :: Xout(Nref)
!locals
INTEGER      :: k, interp_index(2,Nref)
REAL(KIND=8) :: Acc_Weighting(Nuser,Nref)
!CALL LayerAvg( Ref_LnPressure   , &
!               User_LnPressure  , &
!               Acc_Weighting    , &
!               interp_index)

DO k = 1, Nref
   Xout(k) = SUM(Acc_Weighting(interp_index(1,k):interp_index(2,k), k)  &
                       * Xin(interp_index(1,k):interp_index(2,k)) )
END DO

END SUBROUTINE applyAvg
`


[WIP] Cleanup, fixes, questions

This PR contains some cleanup and fixes and raises some questions.
Open items:

It seems to me that the HandlerInterface could not be type hinted (as I started to do) because of sub-handlers, could we find a better way to do this ? Are sub-handlers necessary at-all (this could be handled by the configuration (i.e. DIC or whatever system you use) ?
I don't like the Exception management. When an invalid argument is passed we should throw an exception of (derived) class InvalidArgumentException / RuntimeException / LogicException according to what fit best.
I don't like XCache inheriting from APC this is convenience not good design (they live independently),
the same goes for Memcache extending Memcached

I did not intend so  many changes at the beginning, everything is open to discussion.

It seems to me that the HandlerInterface could not be type hinted (as I started to do) because of sub-handlers, could we find a better way to do this ? Are sub-handlers necessary at-all (this could be handled by the configuration (i.e. DIC or whatever system you use) ?

Why would subhandlers prevent this? They're really internal only- the main Memcache class wraps around it, as does the Sqlite one- the public should never interact with them directly. The main idea behind them is that the system can choose the best possible way extension without the people configuring the system needing to know the details (seriously, the differences between the memcache and memcached extensions are huge, but most people have no idea- and don't really need to).
The other thing is that the memcache wrapper class has a lot more logic to it than some of the other handlers (in order to handle group invalidation, as well as some extra efficiency things). Letting that big chunk of logic actually call out to the dumber storage modules seems logical to me.
That's my logic on it, but I'm willing to admit my logic may not be perfect here. Also, any proposal that accomplishes the same thing works for me.

I don't like the Exception management. When an invalid argument is passed we should throw an exception of (derived) class InvalidArgumentException / RuntimeException / LogicException according to what fit best.

Sure, I don't really see any issues with this.

I don't like XCache inheriting from APC this is convenience not good design (they live independently),

To be completely honest, the XCache handler is still not very well tested (especially since it kills itself on the CLI) or used (it's not one of our official handlers yet because we know it's broken). I'd be up for pretty much any change you made to it.

the same goes for Memcache extending Memcached

Yeah, this is especially true as it grew up. Originally they were closer (in my code), but have deviated quite a bit. I'd have no issues with this, as it does make sense.

Why would subhandlers prevent this?

That's my mystake I thought sub-handlers were extending the HandlerInterface as well, they're not so no pb.

They're really internal only- the main Memcache class wraps around it, as does the Sqlite one- the public should never interact with them directly. The main idea behind them is that the system can choose the best possible way extension without the people configuring the system needing to know the details (seriously, the differences between the memcache and memcached extensions are huge, but most people have no idea- and don't really need to).

I agree to keep the simplicity for end users but the abstraction should be provided at a higher level if needed (i.e. Sf2 Bundle config).

The other thing is that the memcache wrapper class has a lot more logic to it than some of the other handlers (in order to handle group invalidation, as well as some extra efficiency things). Letting that big chunk of logic actually call out to the dumber storage modules seems logical to me.

Couldn't OOP help us here ?

That's my logic on it, but I'm willing to admit my logic may not be perfect here. Also, any proposal that accomplishes the same thing works for me.

I'll try to submit proposals after the cleanup.
Thank for your quick reply, I'll try to close this PR first before submitting new proposals.

For Sf2 I do not plan to hide the driver details for memcache / memcached. That would be more simple for the end user but I really think the end user should know what extension he uses and they do not provide the same features.

Well, keep in mind that the user can always override things. If someone wants to explicitly use one of the extensions they can pass that as one of the options for that handler (which is how our test suite manages to test each sub handler individually). The idea is that they should be able to just pick a backend (memcache, sqlite) and have it work in the most efficient way, but if they need to fall back onto one of the others they can.

@tedivm I have just pushed some updates. I will try to keep this branch in a good shape so that you can have someone testing on Win.
I think it would make sense to have component specific Exceptions which I'll do later (Currently SPL ones are used).

@tedivm Let me know what I can do for this to get merged asap (i.e. extracting DS related changes or any other questionable changes to some other PR). Thanks.

Vic, I'll run through it and do that- when I went to check it last night it wasn't passing tests, which is why I held off.

I have fixed a coupe of remaining issues today, you can see Travis green light now. Let me know what should be moved out.

I do'nt see anything that should be moved out, I've just been running through the code to see if I had any questions. I'm about to merge now actually.


Fix linear interpolation in SimpleIndex.cpp

To interpolate f(x) linearly between f(x0) and f(x1), the correct equation is f(x) = 1 / (x1 - x0) * (x1 - x) * f(x0) + (x - x0) * f(x1). With x1 - x0 = 1 and diff := x - x0, this means f(x) = (1 - diff) * f(x0) + diff * f(x1).

thanks, ill merge this soon.

This is failing on debug build where additional checks are performed. The new bin sizes are too small.
What prompted this change?

The function is concave, so I would expect any linear interpolation to undershoot it. If you want the approximation to always be larger than the actual value, you could instead compute a tangent on the real function at each power of 2, but that would need the derivative of the exact function, as well as a new table. Not sure what other approaches there are to linearly overapproximate a concave function.
Is the difference large?

not totally sure but I think there have been some crashes as the bins overflow. whoops.
I reverted but will try to look into it when time allows.


Updated Conduit to latest release

We've cut a new release of conduit, so here's the update.

It is, we were having a snap moment!


Bump sinatra to v1.4.7

Changes in sinatra means that, eg, settings becomes Rummager.settings.
Leave tilt, and small bump for rack-protection: bumping these two further cause test failures. That will be fixed in the next PR.
The following work on the development vm:

all tests pass
RUMMAGER_INDEX=all bundle exec rake rummager:migrate_index works
bundle exec bin/health_check --type=suggestions runs
http://rummager.dev.gov.uk/unified_search.json?q=child%20benefit returns sensible output

:confetti_ball:

This is great! :tada: :+1:


Verify compatibility of provided Test262 directory

Future versions of that project may alter the requirements for test
generation. Because the user of this library may provides any arbitrary
version of Test262, it is possible that some future change would cause
this library to emit invalid test material.
Reference the package.json file provided by Test262 to determine
compatibility and reject incompatible versions. Because consumers may
only be interested in a subset of tests which are not effected by any
given "breaking change," expose a configuration option named
acceptVersoin to disable this restriction.

@leobalter @spectranaut @rwaldron Would you mind taking a look?

this looks good to land.


Leverage Python for complex configuration

Some users expressed the need for complex filtering / rendering. I feel like these requests will increase over time, as we will likely support more specific objects like Pydantic/Marshmallow/Django models, and that it will be harder and harder to define clean and powerful configuration options for selecting objects and their members, and for rendering them.
To address this, I would like to leverage the power of Python. In mkdocstrings, the idea is to let the users write their own Jinja filters (and more). In pytkdocs, the idea is to let the users define their own selection/filtering/ordering logic (and more).
Several issues here would benefit (or have benefited) from such a feature:

Respect the order of selection.members #102
Can I collect classes for documentation based on a custom function? #111
Add ability to load library (like Django) before running #89
Configuration for pytkdocs filter option? #62
mkdocstrings automatically sorts members of class #114
Remove path on parameter annotation #115 (mention of filters)
Formatting objects differently depending on their inheritance #130
Void built-in docstrings #16
Add option not to display the root group header if it's the only group #17
"Show source code in..." should be more subtle #35
Enhancement: Separate Properties and class variables #76

...and some more in pytkdocs repository.
Before starting to think about implementation details, we should list the things we want to be able to configure.
Based on the above issues:

rendering

members ordering
custom rendering for specific types of objects
various modifications on template variables (with Jinja filters)

selection

complex selection (using category, name, path, parent classes, properties of objects, etc.)
modification of data during collection

global setup and teardown code (Django setup, mocking libraries, etc.). For this we already have setup_commands, but maybe there's a way to add it back in this spec
local setup and teardown code?

What else :slightly_smiling_face: ?
Then, once we know what we want, we can think about solutions:

methods to register code?
hooks in mkdocstrings and pytkdocs?
...?

Please share your thoughts :smile:

+1 for more/more granular selection rules. I just ran into a use case where it would be nice to filter objects by category.
Following this issue for when it's time to start work :)

I have the problem that I'm bascially using your python handler and patch the rebuild_category_lists with mine, in order to inject custom content (e.g. svgs) created while my code was running during pytest.
I cannot do this in jinja templates, need to really run python for finding images and copy them over into the docs tree.
Problem: There I cannot get a hold on the config. E.g. I need the path of the respective markdown page where the triggering identifier is (in order to know where to copy images to) - but checking the extension.py it seems you throw the whole config from yaml away, except 'selection' and 'rendering' sections. If I add a custom key to selection then I get: mkdocstrings.handlers.CollectionError: __init__() got an unexpected keyword argument base_path'`
=> A passing of custom config outside of those 2 keys would be appreciated, then custom code for enriching the pytkdocs results would have superpowers ;-)

This is outdated. I'm not planning on moving to Python configuration.


Add minKubeVersion validation to templates

Require minKubeVersion to be SemVer formated (Major.Minor.Patch).
Signed-off-by: Vu Dinh<EMAIL_ADDRESS>

@njhale Would you mind taking a look?

/test e2e-aws-olm

/test e2e-aws-olm

Looks like you're hitting an issue with the new anyOf validation. I opened #744 to fix the problem.

Sweet, this looks great!
/lgtm

/approve


added cross-spawn for cross-platform compatability

There is an issue with running spawn on window, so I changed this to use the cross-spawn module for cross platform compatibility.
Hopefully this works for Mac and Linux too

Which Windows? I developed this on Windows running in cmd and git bash. I'll give this a test though :+1:


JxlDecoderSetImageOutBuffer should accept output buffer stride (dstStride) in bytes

It's very convenient when working with 2D buffer always accept and support destination STRIDE everywhere possible.
Example: when decoding two JXL images (let's say 2 map tiles), by using dstStride one can stich them together into one output buffer. No redundant copies needed in order to reorg pixels. Of course JxlDecoderSetImageOutBuffer should support dstStride == 0, stride was not given and calculate it as it does now, otherwise check dstStride < minDstStride, should return JXL_DEC_ERROR dstStride is too small.
Internally dec_external_image.cc::ConvertToExternal should accept/support dstStride as well for that.

You should be able to achieve that by passing the stride in the align parameter in JxlPixelFormat. It doesn't fail if the "stride" is too small for the xsize, but you can check that JxlDecoderImageOutBufferSize returns the expected value if you want to check for strides that are too small.

yes you right this code works
if (format->align > 1)
    row_size = jxl::DivCeil(row_size, format->align) * format->align;

it was confusing, pixel-format->align sounds more like stride within the pixel itself (for example decode and embed rgb24 into given rgba32, so called pixel-stride) and not scan line, line stride.
thanks

Closing since it seems to be resolved.


Add nsswitch.conf to Minecraft docker image to fix localhost resolution

There is the related issue here https://github.com/golang/go/issues/22846
rcon-cli uses Go, which will resolve localhost incorrectly on certain machines, this could be fixed by creating a basic nsswitch.conf, echo 'hosts: files dns' > /etc/nsswitch.conf should do

Is there a particular way you are running the docker container or the daemon that causes that issue for you? I have never encountered localhost not resolving within the container.

I think it may depend on the actual network configuration of the network the host machine uses, as unless I am missing something, this issue results in the go DNS resolving skipping resolution using /etc/hosts.
It has had issues on 4 out of 5 CentOS 7 hosts I have tried it on, with identical docker configurations, and have been working around in the meantime by adding GODEBUG=netdns=cgo to env.

I'll work on adding this change sometime today, but I'll need you to verify the fix since I am not able to recreate it and have heard no other reports of this. I was quite confident the addition of nsswitch.conf will fix it; however, this comment of the issue worries me since the minecraft server image is based on Alpine.

This change has been pushed to Docker Hub. @notgne2 , can you see if this resolved the issue for you?

I think that issue is just referring to Alpine being one of the few images lacking nsswitch.conf by default and that's why Go are urged to fix it internally, I'll give this a test later today if I get the time.

Tested on one of the problematic servers without the workaround env supplied, no errors in logs and docker exec -it mc rcon-cli connects just fine


single reverse phase encode image still causes crash in 0.4.1

Hi,
I am still running into this problem:
I am using the docker container bids/mrtrix_connectome, pulled today May 5, 2018.
It appears to be tagged 0.4.1
Despite this same issue being closed as of 0.4.0, If I have a single reverse phase encode B0 image in the fmap directory, MRtrix fails.
error.txt attached.
I have removed the reverse phase encode image for testing purposes, but how should I proceed?
Thanks much,
Dianne
error.txt
Originally posted by @dkp in https://github.com/BIDS-Apps/MRtrix3_connectome/issues/37#issuecomment-489478742

Hi Diane,
I suspect you've come across the same problem as I discovered in #43.
Unfortunately I can't merge that fix right away, as CircleCI has ceased support for its own version 1.0. My previous attempts to migrate my CI script from 1.0 to 2.0 were unsuccessful; but now that 1.0 has been dropped completely I'm going to have to go back and revisit it.
If possible, it would be useful for me to know, if you make the code changes manually, whether it resolves your issue (as I can't tell with 100% certainty whether this is the same issue as that fixed in #43). If you're using a downloaded container rather than building locally from source, this can still be done, by executing docker run with --entrypoint=/bin/bash and omitting the standard command-line inputs to the script, modifying file mrtrix3_connectome.py using e.g. nano within the container environment, and then executing that script and providing the usual command-line arguments.

I'll give it a try and report back.  Thanks!
-Dianne
On Sun, May 5, 2019 at 6:52 PM Robert Smith<EMAIL_ADDRESS>wrote:

Hi Diane,
I suspect you've come across the same problem as I discovered in #43
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43.
Unfortunately I can't merge that fix right away, as CircleCI has ceased
support for its own version 1.0 https://circleci.com/sunset1-0/. My previous
attempts https://github.com/BIDS-Apps/MRtrix3_connectome/tree/circleci
to migrate my CI script from 1.0 to 2.0 were unsuccessful; but now that 1.0
has been dropped completely I'm going to have to go back and revisit it.
If possible, it would be useful for me to know, if you make the code
changes manually
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43/files#diff-7bcc5dd45845d84a9690fa9e09c12ff2,
whether it resolves your issue (as I can't tell with 100% certainty whether
this is the same issue as that fixed in #43
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43). If you're
using a downloaded container rather than building locally from source, this
can still be done, by executing docker run with --entrypoint=/bin/bash
and omitting the standard command-line inputs to the script, modifying file
mrtrix3_connectome.py using e.g. nano within the container environment,
and then executing that script and providing the usual command-line
arguments.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BIDS-Apps/MRtrix3_connectome/issues/46#issuecomment-489483032,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHLUQGELF6AAADKXXMRTQLPT6FN7ANCNFSM4HK4S6LA
.

--
Dianne Patterson, Ph.D.
Research Scientist
<EMAIL_ADDRESS><EMAIL_ADDRESS>or
<EMAIL_ADDRESS>University of Arizona
Speech and Hearing Science 314
1131 E 2nd Street, Building #71
(Just East of Harvill)
If you don't hear back from me (and you expected to),
I blame the University's new SPAM filter.
Please write to my gmail account.
Antipiphany: That moment when you realize how little you actually know

Hi,
We pulled the git repo and then switched to your hotfix branch to build.
The container is running now and got past the previous choke point....it is
now running topup.
Seems promising.
-Dianne
On Mon, May 6, 2019 at 9:24 AM Dianne Patterson<EMAIL_ADDRESS>wrote:

I'll give it a try and report back.  Thanks!
-Dianne
On Sun, May 5, 2019 at 6:52 PM Robert Smith<EMAIL_ADDRESS>wrote:

Hi Diane,
I suspect you've come across the same problem as I discovered in #43
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43.
Unfortunately I can't merge that fix right away, as CircleCI has ceased
support for its own version 1.0 https://circleci.com/sunset1-0/. My previous
attempts https://github.com/BIDS-Apps/MRtrix3_connectome/tree/circleci
to migrate my CI script from 1.0 to 2.0 were unsuccessful; but now that 1.0
has been dropped completely I'm going to have to go back and revisit it.
If possible, it would be useful for me to know, if you make the code
changes manually
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43/files#diff-7bcc5dd45845d84a9690fa9e09c12ff2,
whether it resolves your issue (as I can't tell with 100% certainty whether
this is the same issue as that fixed in #43
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43). If you're
using a downloaded container rather than building locally from source, this
can still be done, by executing docker run with --entrypoint=/bin/bash
and omitting the standard command-line inputs to the script, modifying file
mrtrix3_connectome.py using e.g. nano within the container environment,
and then executing that script and providing the usual command-line
arguments.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BIDS-Apps/MRtrix3_connectome/issues/46#issuecomment-489483032,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHLUQGELF6AAADKXXMRTQLPT6FN7ANCNFSM4HK4S6LA
.

--
Dianne Patterson, Ph.D.
Research Scientist
<EMAIL_ADDRESS><EMAIL_ADDRESS>or
<EMAIL_ADDRESS>University of Arizona
Speech and Hearing Science 314
1131 E 2nd Street, Building #71
(Just East of Harvill)
If you don't hear back from me (and you expected to),
I blame the University's new SPAM filter.
Please write to my gmail account.
Antipiphany: That moment when you realize how little you actually know

--
Dianne Patterson, Ph.D.
Research Scientist
<EMAIL_ADDRESS><EMAIL_ADDRESS>or
<EMAIL_ADDRESS>University of Arizona
Speech and Hearing Science 314
1131 E 2nd Street, Building #71
(Just East of Harvill)
If you don't hear back from me (and you expected to),
I blame the University's new SPAM filter.
Please write to my gmail account.
Antipiphany: That moment when you realize how little you actually know

The hotfix version of MRtrix completes, and, to the extent I understand,
the output looks fine
[Saci:sub-103: 559] $ tree
.
|-- anat
|   |-- sub-103_5TT.nii.gz
|   |-- sub-103_T1w.nii.gz
|   |-- sub-103_brainmask.nii.gz
|   |-- sub-103_parc-desikan_indices.nii.gz
|   -- sub-103_tissues3D.nii.gz |-- connectome |   |-- sub-103_parc-desikan_level-participant_connectome.csv |   -- sub-103_parc-desikan_meanlength.csv
|-- dwi
|   |-- eddyqc
|   |   |-- eddy_movement_rms
|   |   |-- eddy_outlier_map
|   |   |-- eddy_outlier_n_sqr_stdev_map
|   |   |-- eddy_outlier_n_stdev_map
|   |   |-- eddy_outlier_report
|   |   |-- eddy_parameters
|   |   |-- eddy_post_eddy_shell_PE_translation_parameters
|   |   |-- eddy_post_eddy_shell_alignment_parameters
|   |   -- eddy_restricted_movement_rms |   |-- sub-103_brainmask.nii.gz |   |-- sub-103_bvalues.txt |   |-- sub-103_dwi.bval |   |-- sub-103_dwi.bvec |   |-- sub-103_dwi.nii.gz |   |-- sub-103_meanbzero.nii.gz |   |-- sub-103_model-tensor_fa.nii.gz |   |-- sub-103_tissue-WM_ODF.nii.gz |   -- sub-103_tissue-WM_response.txt
-- tractogram     -- sub-103_mu.txt
5 directories, 26 files
Thanks so much! I put a copy of the built container on
dockerhub: diannepat/mrtrix3-hotfix:latest
-Dianne
On Mon, May 6, 2019 at 6:33 PM Dianne Patterson<EMAIL_ADDRESS>wrote:

Hi,
We pulled the git repo and then switched to your hotfix branch to build.
The container is running now and got past the previous choke point....it
is now running topup.
Seems promising.
-Dianne
On Mon, May 6, 2019 at 9:24 AM Dianne Patterson<EMAIL_ADDRESS>wrote:

I'll give it a try and report back.  Thanks!
-Dianne
On Sun, May 5, 2019 at 6:52 PM Robert Smith<EMAIL_ADDRESS>wrote:

Hi Diane,
I suspect you've come across the same problem as I discovered in #43
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43.
Unfortunately I can't merge that fix right away, as CircleCI has ceased
support for its own version 1.0 https://circleci.com/sunset1-0/. My previous
attempts https://github.com/BIDS-Apps/MRtrix3_connectome/tree/circleci
to migrate my CI script from 1.0 to 2.0 were unsuccessful; but now that 1.0
has been dropped completely I'm going to have to go back and revisit it.
If possible, it would be useful for me to know, if you make the code
changes manually
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43/files#diff-7bcc5dd45845d84a9690fa9e09c12ff2,
whether it resolves your issue (as I can't tell with 100% certainty whether
this is the same issue as that fixed in #43
https://github.com/BIDS-Apps/MRtrix3_connectome/pull/43). If you're
using a downloaded container rather than building locally from source, this
can still be done, by executing docker run with --entrypoint=/bin/bash
and omitting the standard command-line inputs to the script, modifying file
mrtrix3_connectome.py using e.g. nano within the container environment,
and then executing that script and providing the usual command-line
arguments.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BIDS-Apps/MRtrix3_connectome/issues/46#issuecomment-489483032,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHLUQGELF6AAADKXXMRTQLPT6FN7ANCNFSM4HK4S6LA
.

--
Dianne Patterson, Ph.D.
Research Scientist
<EMAIL_ADDRESS><EMAIL_ADDRESS>or
<EMAIL_ADDRESS>University of Arizona
Speech and Hearing Science 314
1131 E 2nd Street, Building #71
(Just East of Harvill)
If you don't hear back from me (and you expected to),
I blame the University's new SPAM filter.
Please write to my gmail account.
Antipiphany: That moment when you realize how little you actually know

--
Dianne Patterson, Ph.D.
Research Scientist
<EMAIL_ADDRESS><EMAIL_ADDRESS>or
<EMAIL_ADDRESS>University of Arizona
Speech and Hearing Science 314
1131 E 2nd Street, Building #71
(Just East of Harvill)
If you don't hear back from me (and you expected to),
I blame the University's new SPAM filter.
Please write to my gmail account.
Antipiphany: That moment when you realize how little you actually know

--
Dianne Patterson, Ph.D.
Research Scientist
<EMAIL_ADDRESS><EMAIL_ADDRESS>or
<EMAIL_ADDRESS>University of Arizona
Speech and Hearing Science 314
1131 E 2nd Street, Building #71
(Just East of Harvill)
If you don't hear back from me (and you expected to),
I blame the University's new SPAM filter.
Please write to my gmail account.
Antipiphany: That moment when you realize how little you actually know

Remembered that pull requests on this repo are not blocked by CI... Fix is now on Docker Hub tagged as 0.4.2.


Healthchecks seem to have a hard timeout of 60,000ms (60s)

Hey all!
First - this project is amazing.
I'm having a problem with a test oracle database that I have to bootstrap with lots of stuff. Here's my definition:
    container = await new GenericContainer('gvenzl/oracle-xe:18')
      .withName('oracle')
      .withEnv("ORACLE_PASSWORD", 'password')
      .withEnv('ORACLE_DATABASE', 'beekeeper')
      .withEnv('APP_USER', 'beekeeper')
      .withEnv('APP_USER_PASSWORD', 'password')
      .withExposedPorts(1521)
      .withBindMount(localDir, '/docker-entrypoint-initdb.d', 'ro')
      .withHealthCheck({
        test: "sqlplus -s beekeeper/password@//localhost/BEEKEEPER <<< \"select * from actor;\" | grep 'no rows'",
        interval: 10000,
        timeout: 10000,
        retries: 12, // 2 minutes
        startPeriod: 60000
      })
      .withWaitStrategy(Wait.forHealthCheck())
      .start()

This constantly times out with a messaging saying healthcheck not healthy after 60000ms, even though I've set much more generous health check options.
Hoping you can help?
Using testcontainers 8.9.0

Hey @rathboma , thanks for the kind words! You'll need to set withStartupTimeout(120000) on the container as well. The health check config is separate from the container startup config, although I've noted in this case that's not intuitive

AAAAH yes you're right. I was confused because the message mentioned healthcheck, so I was hyper focused on that syntax.
Thanks!


Use strikeout text

In addition to the checkboxes it would be nice to use strikeout text to indicate whether each item in the list is “done” or not.
Probably wait for https://github.com/NativeScript/NativeScript/issues/875.

Oh hey, this is done!


Role Commands: Extend role commands with more configuration options

[x] Only x roles groups
[x] Require another role to unlock group
[x] Blacklist role from group
[x] Reorder
[x] Write migrations from the current configs

https://github.com/jonas747/yagpdb/commit/72bcebca12e349e17e41f9bd9ea9f74b34a11f7a


Add presentation guidelines

Summary of changes

This PR adds information to the guide for presentations about writing, presenting, and reviewing in the style of the report guide. It also adds some useful links about animations and buttons.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Associated Issues and PRs

Issue: closes #251

Checklist for Reviewers
Reviewers should use this link to get to the
Review Checklist before they begin their review.

Thank you @gwenchee for your feedback!

Great! I'll merge now?

That would be great!


Add creation review for JWT Bridge spec

@jclingan as promised, I have created the creation review for next week's discussion. Please take a look to see whether you have any comments.

What do we think about updating this:

The specification is to enable Jakarta Security building on MicroProfile JWT without forking it but providing seamless integrations with MicroProfile JWT.

To this:

The specification enables mapping MicroProfile JWT tokens to Jakarta EE container APIs not included in the MicroProfile umbrella and provides a place where Jakarta EE specifications, such as Jakarta Security, can build requirements and seamless integrations with MicroProfile JWT.

Our "Mapping MP-JWT Tokens to Jakarta EE Container APIs" chapter really covers many Jakarta EE specs, so we should maybe acknowledge that.  Otherwise, we're implying this is limited to Jakarta Security despite the fact there will be requirements for other specifications.
Thoughts?

Perhaps specifically mention that for Jakarta Security, MP JWT has to technically function as a pluggable  authentication mechanism an equal footing with provided authentication mechanisms such as the HTTP Basic, Form, and OpenID Connect authentication mechanisms.
As such, Jakarta Security does not build upon MP JWT, nor is MP JWT layered on top of Jakarta Security. MP JWT should just present itself as an authentication mechanism.
As an example, consider https://github.com/OmniFish-EE/omni-jwt
That implementation plugs in to any Jakarta Security implementation, simply by using the HttpAuthenticationMechanism interface.

@dblevins I have created a feature branch jwt-bridge-branch and delivered the initial creation plan review there. Please checkout and do your PR regarding the TCK clarification. Please also take a look at the comments above from @arjantijms to see whether you can address it as well. I will close this PR after the call this Thursday.


package.json is hardcoded to version 1.0.0

See https://github.com/pulumi/pulumi-std/blob/master/sdk/nodejs/package.json
This need to follow the same style as other SDKs and dynamically set the version based on the current tag.

Fixed in https://github.com/pulumi/pulumi-std/pull/23


Some small improvements

Add .idea to gitnognore
Add pip to environment.yml, otherwise the newest conda versions break
Add data download instructions

Cool, can you merge? (I don't have write access)


Support nullable in case of not found sub object

Sorry for not describing
Couldn’t find any reference in the documentation


Runner.go Context Window Shifting

This series implements context window shifting for the new go server runner. It also fixes a number of issues in the related code.
My intention is to start adding tests for some of the issues encountered here but I wanted to start getting reviews on this code in the meantime.

LGTM minus some small comments + linter error

Rebased (which fixes the lint issue) and tacked on a fix for MinP.


Warface Ban

Warface bann, could it be because of the betterkam takeover?

How to download your early versions without betterkam?

I played with an early version of Obs_capture, everything was fine

Obs_capture is also in this version, there are many reasons why the account could be blocked, and rather the main reason is not using adruino.


Uncouple the mapper from ksp

Your mapper is very interesting to map kotlin  objectif
Please do not strongly couple it with ksp ( like mapstruct is coupled with java annotation processing)
You should create your own model to extract mappings from classes.
This allows mappings from ksp or another provider like a gradle or a maven plugin.
The interesting part is on generating kotlin mapping code, and this should be reusable given classes data from anywhere.
I'm working on a project mixing java/groovy/kotlin.
A gradle plugin could then generate the mappers from any language and be compatible with the k2 compiler without using annotation processing,kapt or ksp
(But jandex)

When I started the project, I already thought about decoupling it from KSP. I did not do it, as I use a lot of stuff from KSP and I am not sure if the extra work would pay off, as other tools must then provide a lot of information to Konvert (which I am not sure that they can).
Regarding mixing java/groovy/kotlin:
I haven't tried using multiple JVM languages in one project yet - beside Kotlin with Java, this is in general working already. I assume it will also work with groovy if groovy is compiled before Kotlin, but I am not sure of this.


Remove old elements from the output parameter of Pluck()

Make sure these boxes checked before submitting your pull request.

[x] Do only one thing
[x] No API-breaking changes
[x] New code/logic commented & tested

For significant changes like big bug fixes, new features, please open an issue to make an agreement on an implementation design/plan first before starting it.
What did this pull request do?
Currently, the Pluck() method keeps old elements in the output parameter. So, if someone calls Pluck() twice, the results would be doubled. The patch fixes this incorrect behaviour.

Thanks @zenovich.


E2e test for hierarchical network policy

Tested: make e2e-test

/assign @yiqigao217
/assign @adrianludwin

FYI I will wait for the write-temp-file to give another review.


eth0 stops working after 1/2 day of uptime on bionic-minimal-rock64-0.6.31-209-arm64/mainline

After about 6-12 hours of uptime on kernel 4.16.0-rockchip-ayufan-209-g027e722bb, the eth0 interface becomes unresponsive. ip a still lists a valid configuration, but there isn't any incoming or outgoing traffic.
A ping test of a hostname the following:
root@rock64:~# ping google.com ping: google.com: Temporary failure in name resolution
While pinging an ip address leads to this output:
root@rock64:~# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
From <IP_ADDRESS> icmp_seq=1 Destination Host Unreachable
From <IP_ADDRESS> icmp_seq=2 Destination Host Unreachable
From <IP_ADDRESS> icmp_seq=3 Destination Host Unreachable
From <IP_ADDRESS> icmp_seq=4 Destination Host Unreachable
From <IP_ADDRESS> icmp_seq=5 Destination Host Unreachable
From <IP_ADDRESS> icmp_seq=6 Destination Host Unreachable
^C
--- <IP_ADDRESS> ping statistics ---
8 packets transmitted, 0 received, +6 errors, 100% packet loss, time 7157ms

The result is the same after disconnecting and reconnecting the Ethernet cable or restarting the interface using systemctl restart NetworkManager.
This is usually within a day of the full system becoming unresponsive as described in #157.

On a potentially related note, I have noticed artifacts over SSH in the 0.6.XX builds on both mainline and the 4.4 kernels as shown below, though it is worse on mainline.
I did not see this issue on 0.5.15 with the 4.4 kernel, but IIRC it was present on mainline.
Maybe this is related to the gmac stability issue? https://github.com/ayufan-rock64/linux-mainline-kernel/commit/73e42e18669934fa96cf2bb54291da54177076d7

This is solved: https://github.com/ayufan-rock64/linux-build/releases/tag/0.6.40


Process Ingredients for Testing

An issue with linker/Ingredient class dependency

This is an issue of the code being untestable; alter code to be testable for Beta


[ Docker ] Initialize the data from getstream.io only at first start

Otherwise the application will look empty, and impossible to use.

👍


Util: Test_timeline changes

After all the work we put in on make_timeline, it really did drop in nicely here. I'll just go through its operation quickly so that this request can technically stand on its own. (This is in response to #646.)
This update adds an optional argument, -search_fights (aliased to -lf) that either takes no value or a positive integer as input. When called with this flag, the script will divert and build an encounter listing, and either return that list to the user or pull the selected encounter from the list and run the selected timeline against it.
Standard operation of the script is unaffected, so the user could (if they wanted to for whatever reason) list the fights and then enter the start/end times manually using -start/-end.

Is there a way to better share code between these two? Maybe like a timeline.py that both can import?

That's not unreasonable. I'll look into it. I'm not sure what level of complexity I can currently handle with module production, but a simple helper module should be doable.


[RFR] fix shouldUpdate for EditButton and ShowButton

nextProps.record is potentionally undefined

ie. it happens when hot reloading

so what's the bug it fixes? A warning in the console when hot reloading? I have never seen it. Please provide a scenario to reproduce the initial bug.

it causes exception about getting id from undefined
causing new react16 to unmount the tree until error boundary
its reproducible only on dev mode when you use hot reloading
as webpack reloads component tree and initializing the component with undefined props
making the nextProps.record undefined for cycle
EditButton.js:68 Uncaught TypeError: Cannot read property 'id' of undefined
    at EditButton.js:68
    at ShouldUpdate.shouldComponentUpdate (shouldUpdate.js:41)
    at checkShouldComponentUpdate (react-dom.development.js:9682)
    at updateClassInstance (react-dom.development.js:9998)
    at updateClassComponent (react-dom.development.js:10224)
    at beginWork (react-dom.development.js:10605)
    at performUnitOfWork (react-dom.development.js:12573)
    at workLoop (react-dom.development.js:12682)
    at HTMLUnknownElement.callCallback (react-dom.development.js:1299)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:1338)
    at invokeGuardedCallback (react-dom.development.js:1195)
    at performWork (react-dom.development.js:12800)
    at scheduleUpdateImpl (react-dom.development.js:13185)
    at scheduleUpdate (react-dom.development.js:13124)
    at Object.enqueueSetState (react-dom.development.js:9646)
    at Connect../node_modules/react/cjs/react.development.js.ReactComponent.setState (react.development.js:218)
    at Connect.onStateChange (connectAdvanced.js:205)
    at Object.notify (Subscription.js:26)
    at Subscription.notifyNestedSubs (Subscription.js:65)
    at Connect.onStateChange (connectAdvanced.js:202)
    at Object.notify (Subscription.js:26)
    at Subscription.notifyNestedSubs (Subscription.js:65)
    at Connect.onStateChange (connectAdvanced.js:202)
    at Object.dispatch (createStore.js:173)
    at dispatch (<anonymous>:2:31875)
    at middleware.js:13
    at middleware.js:66
    at Object.declareResources (bindActionCreators.js:3)
    at AdminRoutes.js:99
    at <anonymous>

to reproduce you can take any create-react-app with AOR
make sure you have the hot reload active for App

if (process.env.NODE_ENV !== 'production') {
  if (module.hot) {
    module.hot.accept('./App', () => {
      render(App, appProps);
    });
  }
}

then run dev mode with hot reload
open detail of any item
change some code and save
hot reload triggers and causes the exception

Thanks!


Need UX design for CRUD operation on pool/rbd

We need UX design for CRUD operation on pool and rbd .
basically pool can have these CRUD operation as per UX design (https://redhat.invisionapp.com/share/BR8JDCGSQ#/screens/198417118 ) :

Edit pool
Grow PGs for pool
Remove pool

and pool can have these CRUD operation as per UX design ( https://redhat.invisionapp.com/share/BR8JDCGSQ#/screens/198417119 ) :

Remove RBDs
Resize RBDs

@julienlim @mcarrano Please can we have UX design for CRUD operation on pool and rbd as per above statement.

@kamleshverma1 @mcarrano The edit pool, grow pg, and resize rbd is in progress.  We should have that by end of this week.

Resize RBD design just posted to https://redhat.invisionapp.com/share/Z9AE9W6H4.

@kamleshverma1 since the UX designs are now available, I am closing this. Reviews/clarification on the designs provided are expected to be via the invisionapp


geo_alchemy: make application context available to db function

Similar to #2308, this fix is needed for the app to launch

I don't seem to get any errors about missing app context when running this example. Can you share your error message(s) and what versions of python libraries you're using please?

My traceback:
Traceback (most recent call last):
  File "C:\Users\hasansezertasan\Desktop\flask-admin\examples\geo_alchemy\app.py", line 99, in <module>
    db.create_all()
  File "C:\Users\hasansezertasan\Desktop\flask-admin\venv\Lib\site-packages\flask_sqlalchemy\extension.py", line 900, in create_all
    self._call_for_binds(bind_key, "create_all")
  File "C:\Users\hasansezertasan\Desktop\flask-admin\venv\Lib\site-packages\flask_sqlalchemy\extension.py", line 871, in _call_for_binds
    engine = self.engines[key]
             ^^^^^^^^^^^^
  File "C:\Users\hasansezertasan\Desktop\flask-admin\venv\Lib\site-packages\flask_sqlalchemy\extension.py", line 687, in engines
    app = current_app._get_current_object()  # type: ignore[attr-defined]
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\hasansezertasan\Desktop\flask-admin\venv\Lib\site-packages\werkzeug\local.py", line 519, in _get_current_object
    raise RuntimeError(unbound_message) from None
RuntimeError: Working outside of application context.

This typically means that you attempted to use functionality that needed
the current application. To solve this, set up an application context
with app.app_context(). See the documentation for more information.


[#9] Test suite in container

Update the test suite to run in a single container.
The only issue is the time consumption.
If you want to test, please replace line 79 in Dockerfile.rocky9 with:
RUN git clone --branch "test" --single-branch --depth 1 https://github.com/GuChad369/pgmoneta_ext && \

Test result:

@GuChad369 Did you try to checkout the repository once, and switch between branches between the runs ?

Maybe it isn't possible though...

If we can test them in parallel, the time will be significantly reduced. Currently, the tests run sequentially. I am trying to find a way to make them run simultaneously. This will greatly improve the efficiency.

yes, as long as we have the logs from each

There is an option ctest -j which can run the tests in parallel using the given number of jobs. The time cost will depend on the machine. On my computer, it takes 600 seconds, whereas before it took 1200 seconds, effectively reducing the time by half.

ctest -j 4
Test project ..../pgmoneta_ext/build
    Start 1: test_version_13_rocky9
    Start 2: test_version_14_rocky9
    Start 3: test_version_15_rocky9
    Start 4: test_version_16_rocky9
1/4 Test #1: test_version_13_rocky9 ...........***Failed  152.69 sec
2/4 Test #2: test_version_14_rocky9 ...........***Failed  152.69 sec
3/4 Test #3: test_version_15_rocky9 ...........   Passed  414.99 sec
4/4 Test #4: test_version_16_rocky9 ...........   Passed  424.51 sec

50% tests passed, 2 tests failed out of 4

Total Test time (real) = 424.52 sec

The following tests FAILED:
	  1 - test_version_13_rocky9 (Failed)
	  2 - test_version_14_rocky9 (Failed)
Errors while running CTest

LastTest.log

The problem with the two failed tests is:
"f4952f7dfb0d1c67ab07e600f4f0a989e356e58c49ea03a20374cce67f90610d" to file: reading tar archive: copying content for "usr/share/doc/strace/NEWS": write /var/tmp/buildah2479751812/layer: no space left on device on pipe close

This issue is related to a lack of space and has nothing to do with the source code.
And if you want to test, please replace line 79 in Dockerfile.rocky9 with:
RUN git clone --branch "test" --single-branch --depth 1 https://github.com/GuChad369/pgmoneta_ext && \

On the host ?
/dev/nvme0n1p2    30G   20G  8.7G  70% /

8.7G should be more than enough

And, /home is
/dev/nvme0n1p5   869G  796G   30G  97% /home

Please replace the line and retest. The main repository won’t work until this PR is merged.

Merged.
Thanks for your contribution !


Need to Use awaitility library instead of Thread.sleep in integration tests

The following library [1] can be used instead of Thread.sleep in Java integration tests
[1] - https://github.com/awaitility/awaitility

Due to no activity marking as stale


Fix broken integration

This fix issue #10 , As breaking changes in the panasonic api had been fix in the latest release of the underlying library

Please test brefore merging, as I tested this on the panasonic_ac custom integration without any issues, but not panasonic_cc


Feature request: any  method to update a video stream?

Need a method to update a stream in my live streaming app ,the position and size of the video stream may be dynamically changed by user,the api should look like:
merger.updateStream(id|MediaStream,opt)
BTW, Is there a z(-index) option to control the overlap order of video stream ?
Thanks for the good library.

The position and size options are just convenient wrappers around the draw API, so you can change them through that.
var myOpts = {x: 0, y:0, width: 100, height:100}
merger.addStream(mediaStream, {
  draw: function (ctx, frame, done) {
    ctx.drawImage(frame, myOpts.x, myOpts.y, myOpts.width, myOpts.height)
    done()
  })
})

// changes to myOpts will be applied on next frame draw
myOpts.x = 10
myOpts.width = 200

The z-index option is called index. It is the order in which to draw the stream (0 pushes to bottom, null pushes to top). The streams drawn first will be on the bottom.

Thanks.

Sorry to reopen the issue.
I've checked that changes to position and  size options (x,y,weight,height) will be applied on next frame draw as you said, but it seems that change to index doesn't work. How can I change the draw order of video stream dynamically?

Hmmmm... I don't think there is a way. I'll take a look at implementing updateStream.

2.6.1 now has the merger.updateIndex(mediaStream|id, newIndex) method.
Index is the only option that isn't covered by draw and audioEffect, so I think this handles things nicely without too much complexity.


replace exponential backoff with perpetual retry

Exponential backoff is a good idea in theory, but causes lots of user experience issues.
To name a few:

after blocks have been syncing for hours, it takes a very long time before the status changes to "synced" because of the backoff
if you unlock wallets after having the broker up for a few minutes, the status does not update very quickly

So, we remove exponential backoff and replace it with a much simpler retry of 10 seconds.

 Remove exponential backoff for LND health checks


formatted code based on pep8

used pep8 to format code.
formated test files.

I will check why the tests are failing.

@jcsekar Will you be able to take a look at this today? It's the last day of Hacktoberfest!


List of issues found while testing

List of issues found while testing:

netlist/netlist::is_module_in_netlist: only checks the id not the pointer. Therefore we can pass modules with the same id, that aren't part of the netlist anymore to get a false  or pass a nullptr, which results in a segmentation fault.

netlist/module::set_name: does not check whether the passed name is the empty string. At the other hand, the create_module function (of the netlist) does not allow the empty string.

netlist/netlist_internal_manager::create_module: The module created by the create_module, may have a parrent owned by another netlist

Hey, thanks for the report. Two minor problems:
First, unless your set of issues is inseparably linked and should only be discussed / solved together please open a new github issue for every single one of them. I know it's tedious but it's just much easier for us to organize and keep track of them.
Second, please give your issues descriptive names. That is obviously easier if an issue only contains a single question/request/bug so this is somewhat related to the first point. "netlist::is_module_in_netlist doesn't check the pointer" for example would be a descriptive title.
This might seem pedantic, but issues usually appear quicker than they are solved so at some point we might have 500 of them simultaneously and if they're all called "list of issues", and contained 3 unrelated problems, this tracker would be next to useless.
I'll probably add some kind of instructions on how to open an issue in the near future.

I agree with @Slowpoke42. Could you split it in individual issues?

Closed in favour of #96 #97 #98


Add tests and update build scripts

Description

Added new test harness that integrate with a production bot
Updated build scripts
Bumped all dependencies

Changelog
Breaking Changes

Build folders updated

/dist/ contains JavaScript bundle

/dist/directline.js is now in lowercase

/lib/ contains ES modules and type definitions

Build scripts updated

npm run build: Development build, with instrumentation code, one-off
npm run start: Development build, with instrumentation code, with watch
npm run prepublishOnly: Production build, minified, one-off

Changed

Bumped dependencies, by @compulim, in PR #195

<EMAIL_ADDRESS><EMAIL_ADDRESS>

Bumped dev dependencies, by @compulim, in PR #195

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Fixed everything I believe. 😉

Phew~~~


#4: Rust API unit tests

-
-

Issue
https://github.com/ripe-tech/pconvert-rust/issues/4

Dependencies
None

Testing
cargo test at the root level of the repo

Decisions
In this PR, we make the command-line "tests" we had also runnable via cargo test. The code to execute is the same, hence we refactor common code into tests and use it in the command-line binary. Note that the reverse can't be done because the cli.rs file is not part of the crate: it is merely a binary. Thus, cli.rs can use code from our lib (like the testing code) but the reverse can't be done. Regardless, it makes more sense to me to have testing code in the test module, and, because we want to, to provide the same testing code via command-line, resembling the original pconvert

Screenshot

Re-assigning to me and all of the other pconvert_rust unit tests PR as CI started failing.

It is the same problem across all PRs: it is a compilation failure of wasm-pack (not a project dependency, just the tool used to build the wasm module) with rust version 1.40

@joao-conde please re-add the CLI features we want to make this compatible with the original convert, you can keep the test code. We can be redundant in the code no problem

adding fast-track to try to close this as soon as possible.
Let's also avoid unneeded refactor of the code, and instead use separate PRs for that

@joamag  the CLI test options still exist, just like in pconvert

@joao-conde move that stuff into a better place

Btw CI failing because of https://github.com/rustwasm/wasm-pack/issues/969


Raise the PHPStan level to the max

Level 2 https://github.com/gomzyakov/secretic/pull/312

Level 3 https://github.com/gomzyakov/secretic/pull/314

Level 6 https://github.com/gomzyakov/secretic/pull/315

Level 7 https://github.com/gomzyakov/secretic/pull/316


GDPR privacy notice

Add the GDPR privacy notice to the production version of this code. There should be a hyperlink to the text on the sign up page.

closed in #234


Radial Chart Labels Overlap

react-vis library version: ^1.11.7
The labels of Radial Chart overlap. Is it possible to have adjust the margin or padding so that they don't overlap with each other?
The data passed to Radial Chart is dynamic and not hard coded.
<RadialChart
              data={donutChartData}
              width={375}
              height={375}
              className='donut-chart'
              innerRadius={100}
              radius={140}
              padAngle={0.00}
              colorType="literal"
              showLabels={true}
              labelsRadiusMultiplier={1.34}
              labelsAboveChildren={true}
/>

It would be nice to be able to set each label to a specific labelAnchorX, Y, and Z


Some blocks in assembled rocket crash game.

Ver. 0.6.8
Placed IC2 Trade-O-Mat at rocket (for testing)
After assembling rocket trad-o-mat disapeared (Become invisible)
When i shift click rocket, game crashed. Crashlog here
Can you please make a whitelist config for blocks? So you can`t use inapropriate blocks in rocket. It will reduce accident crash A LOT! Maybe made it as an option, so i can add own blocks to whitelist (blacklist)
How can i disassemble this rocket now? Maybe add "Disassemble" button in Rocket terminal. or ever "Self-destruct" :-D

It's looking like ic2 does something unusual with how it's handling block
data, I'll look into it.


Function to determine if a set is performed or not

Hi, and thanks for a great library!
I've got a use-case where I load stuff from a JSON file, AWS parameter store & AWS secrets manager and then applies environment variables.
This is a override chain where the JSON file is static default configuration and secrets and other IAM handled parameters are from AWS services. Lastly sometimes the DevOps need to do a parameter override directly at the lambda level on a environment variable.
Since current implementation only overrides when ef.IsZero() I needed to have the ability to choose if this is the case or always try to override.
Therefore I did this PR that allows a custom Decider function to be set.
	err := ProcessWith(
		context.Background(),
		&q,
		OsLookuper(),
		func(ctx context.Context, value reflect.Value) bool {
			return true // always set
		},
	)

I've added a DefaultDeciderFunction to be used internally on Process and if just the standard behaviour in ProcessWith.
// This is the default behaviour
var DefaultDeciderFunc = func(ctx context.Context, value reflect.Value) bool {
	return value.IsZero()
}

// Process uses the DefaultDeciderFunc
func Process(ctx context.Context, i interface{}) error {
	return ProcessWith(ctx, i, OsLookuper(), DefaultDeciderFunc)
}

// In unit test I've replaced with DefaultDeciderFunc
if err := ProcessWith(ctx, tc.input, tc.lookuper, DefaultDeciderFunc, tc.mutators...); err != nil {
  // ...
}

Cheers,
Mario

@sethvargo I hope this PR is ok? Please tell me if there's something I need to fix.
Cheers,
Mario

Why not just overwrite the value after parsing?

@sethvargo  How do you mean - after parsing?

You mentioned  "Lastly sometimes the DevOps need to do a parameter override directly at the lambda level on a environment variable."
Why not just override there? I'm not sure I understand the need for this. Perhaps a more specific code sample would help.

@sethvargo If I have the following
type Config struct {
   Timeout `env:"TIMEOUT"`
  // ... and some more props
}

var config Config
// Load static data from JSON file
// ...

// Process Env as overrides
err := ProcessWith(
		context.Background(),
		&config,
		OsLookuper(),
		func(ctx context.Context, value reflect.Value) bool {
			return true // always set
		},
	)

And you want the env to override the struct even if it has a value?

@sethvargo Yes, exactly

Then parse twice - parse first with just the env, then merge with your config.


1.3.0 release breaks 1.2.0?

I am trying to install hashids 1.2.0, which has worked for a long, long time. This currently fails under a python 3.7.7 venv. Is it possible your 1.3.0 release has affected the previous release of 1.2.0 somehow, e.g. with Flitch?
pip install hashids==1.2.0
Collecting hashids==1.2.0
ERROR: Package 'hashids' requires a different Python: 3.7.7 not in '~=2.7,~=3.5'

https://github.com/davidaurelio/hashids-python/commit/7940cf6f3af0f3d4ebaa772a95be1bf9de9a9caf#diff-522adf759addbd3b193c74ca85243f7dR21

having the same problem

Yes, just broke my build on Python 3.8.5 too

Seems I messed this up, apologies! I won't get to it before Monday, european time. Can you fix your version to 1.2.0 until then, please?
If anyone would like to send a PR to fix this, that'd be greatly appreciated. I will merge it tomorrow morning.

We do have the version pinned to 1.2.0, but we can't install 1.2.0 due to this bug for some reason

I have yanked hashids 1.3.0 on pypi as a short term measure. I hope this will do the job. Will fix the version specifier tomorrow.

no, not enough. The metadata is set persistently on pypi :-/

Thanks. Actually I think a new wheel has been pushed for 1.2.0 recently.
I downloaded it and:
$ pip install hashids-1.2.0-py2.py3-none-any.whl
Processing ./hashids-1.2.0-py2.py3-none-any.whl
ERROR: Package 'hashids' requires a different Python: 3.7.3 not in '~=2.7,~=3.5'

The .tar.gz pacakge for 1.2.0 is fine, only the wheel is broken. So you can workaround with pip -v install --no-binary :all: hashids==1.2.0
@davidaurelio do you mind pulling the wheel for 1.2.0 also?

@craigds done

released v1.3.1
deleted the wheel for v1.2.0 – you were right, v1.2.0 only ever had a sdist, no wheel. I set up pypi publishing via GH actions and managed to publish that today.

I will take another good look at the GH action tomorrow. I hope everybody’s setup is working for now.

Thanks!

I suppose we can close here.


Surface a getter function for retrieving the WeaveConnection object for the Tunnel connection

Needed for getting information about PeerAddress, PeerPort of TunnelConnection as well as suspension of TunnelConnection CASE session persistence during shutdown.

Codecov Report

Merging #630 (bc69208) into master (af77670) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #630   +/-   ##
=======================================
  Coverage   70.05%   70.06%
=======================================
  Files         483      483
  Lines       87046    87052    +6
=======================================
+ Hits        60983    60993   +10
+ Misses      26063    26059    -4

Impacted Files
Coverage Δ

.../lib/profiles/weave-tunneling/WeaveTunnelAgent.cpp
80.47% <0.00%> (-0.39%)
:arrow_down:

...files/weave-tunneling/WeaveTunnelConnectionMgr.cpp
76.94% <0.00%> (-0.94%)
:arrow_down:

...les/data-management/Current/SubscriptionClient.cpp
84.64% <0.00%> (+0.07%)
:arrow_up:

src/lib/profiles/security/WeaveCert.cpp
88.96% <0.00%> (+0.45%)
:arrow_up:

src/inet/TCPEndPoint.cpp
53.41% <0.00%> (+0.59%)
:arrow_up:

src/lib/core/WeaveConnectionTunnel.cpp
83.54% <0.00%> (+2.53%)
:arrow_up:

Continue to review full report at Codecov.

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


Challenge Ideas

I thought it might be good to get a thread going of interesting additional challenges/levels. Here are some of my ideas.

Random number of people per elevator (some allow 2, others 6, etc)
People's weight affect how many people per elevator. Currently 4 people can always fit per elevator. Have some really heavy people so only 2 can get on the elevator.
Double decker elevator (The elevator visits 2 floors at once. This could be extended to triple decker, etc.)
Destination dispatch elevators (People press the floor in addition to calling the elevator. A lot of new buildings have these.)
Elevators out of service (Sometimes the elevator is broken)

Some of these are interesting ideas. As I wrote in the (excellent) pull request you made, I want to keep complexity at a minimum while still having an interesting game.
At the very least, added complexity should make the game much more interesting to play.
Some of these ideas, like different weight of people, don't really add much to the enjoyment, while making the code (and playing experience) more complex. Already I'm not sure if the weight of people is an interesting concept at all. Elevators out of service doesn't really affect the algorithmic challenge much, it's just one less elevator to use, and potentially confuses players writing the code, not realizing that an elevator can be out of service.
I'm thinking that additional challenges, at least for now, would focus more on the algorithmic fitness and different optimizations, like minimizing average wait time and combining different constraints to complete a challenge.

I am just throwing out some ideas. I am definitely not expecting all or any of these to be used.
I have finished the game myself and so have several of my friends. I agree that being simple is better, but I think that anyone who finishes the game already (beat level 16) has an understanding of the game. Adding additional obstacles won't be an issue, as long as they are added incrementally. In it's current state you can write one or two algorithms and pass all the levels. It gets interesting when you need to modify the algorithm because the new level needs a new algo to win.
Different weight users: I think it makes people use the loadFactor() function. Currently you can be sure that 4 people will be in the elevator.
Out of service elevators: I think this is a really interesting concept (and it happens in real life). The player needs to check if the elevator is out of service and not send it to the floor. When the elevator comes back in service, the player needs to acknowledge it and put it back in the rotation. Obviously this wouldn't happen until later challenges (probably after 16). I don't think it would be difficult to understand. You could put a big red X on the elevator.
All my ideas were for later levels. Once the person understands the game. Currently I think the game lasts around 3-4 hours for an average developer. I think it would be great to have 100+ challenges. It's going to be difficult to create many more the current setup and no additional obstacles.

Yeah those are good points.
Something interesting would be to have a difficult challenge with one small and one large elevator, where you probably need to serve the upper floors with the large one.

Although I'm leaning towards scrapping the concept of weight, and just counting users etc. The weight concept doesn't really make it more interesting than just userCount and maxUserCount or such.

@taylorhakes Added some nice things in https://github.com/magwo/elevatorsaga/commit/da03f5e6a784cdb1357cb0ba837c8065e3b30772

Very nice additions. Definitely adds some difficulty

Destination dispatch elevators (People press the floor in addition to calling the elevator. A lot of new buildings have these.)

This would be a nice addition for an "easy" mode with each challenge. So just keep the challenges, but allow a checkbox to have people pressing their destination.

I would argue that destination dispatch elevators don't make the game any easier. It gives you more information, but the elevator can now move people more efficiently. The number of people per challenge can be higher (number of steps lower, etc.) because your elevators can now be smarter.

Well if you leave the goals the same it would be easier. If you would adapt the goals and make them harder then before it would be more work as you would have to deal with more information.

Closing as part of an issue cleanup. Please open specific feature requests, one per feature, if there's something in particular that you would like to see!


Fix mistaken link to nuget

Thanks, @enisn. I updated it with detailed readme modifications.
see #f886b99580f


WPF Memory leak (when scrolling through file)

If I'm scrolling through a file more than a kilobyte or two long, every time I scroll the view at all (up/down), the memory usage of the program increases steadily and does not decrease. I loaded a 1kb binary file using a Stream into the editor, scrolled to the bottom and back to the top a few times, and the program was using 200 MB of RAM.

For test i made the memory don't up to ~200 MB of RAM.  I tested on 1kb-700MB files
the garbage collector dont clear all unused memory on WPF... I need somes help for that.

Working in it!
Now control (in test project) never use more than ~150mg of ram... on good way

Working in it!
Now control (in test project) never use more than ~150mg of ram... on good way

I Added Hex Editor got 50MB RAM usage on just idle, on other hand WinForms uses only 10-15MB RAM.


Handle NPE in AbstractJAXBProvider.findJAXBContext

Fixes https://issues.redhat.com/browse/RESTEASY-2489

Note that I didn't run/compile/test the change, I edited it in GH. I guess CI should compile it right?

Yeah, CI will run it.

merged, thanks @ia3andy


app: Delay in voice packs sound and captions

The voice pack Feature is quite cool. But there seems a delay in the captions and the sound. The sound starts after the end of captions in some cases. Example  attached.
https://user-images.githubusercontent.com/35926530/153722337-a6e8dec8-6edb-4dcf-89a2-32fbeb5f90f4.mp4

I've been making a lot of changes in this area and it might be a bit better now.
Can you try again and report on the new version?

I'm going to close this but please do open another issue with a fresh evaluation.


Add verbose field in the LlamaCpp clients

It adds the verbose parameter to the class to be able to pass it to the Llama client and remove all the annoying prints.
See the definition of the original Llama object in the wrapper to check the missing field: https://github.com/abetlen/llama-cpp-python/blob/main/llama_cpp/llama.py#L31

is there anything left to merge this one?

Came here looking to file an issue for this, and see there's already a fix! Anything holding this back?


Composited image occasionally doesn't render first input

Normally compositing works great, but I've been occasionally seeing the final composited image to just render the composite input (overlay) with a transparent background rather than with the background image. This has only happened a few times and it has been pretty random, but wondering if anyone else has seen this and if it could be a potential bug.
What is the output of running npx envinfo --binaries --languages --system --utilities?
(service running in a docker container)
  System:
    OS: Linux 4.9 Ubuntu 18.04.3 LTS (Bionic Beaver)
    CPU: (2) x64 Intel(R) Core(TM) i7-7567U CPU @ 3.50GHz
    Memory: 2.69 GB / 7.79 GB
    Container: Yes
    Shell: 4.4.20 - /bin/bash
  Binaries:
    Node: 10.17.0 - /usr/local/bin/node
    npm: 6.11.3 - /usr/local/bin/npm
  Languages:
    Bash: 4.4.20 - /bin/bash
    Perl: 5.26.1 - /usr/bin/perl

What are the steps to reproduce?
This seems to be happening randomly, so it's difficult to reproduce.
What is the expected behaviour?
I am trying to composite a background image with an arrow overlay. Expected behavior is an image renders with the arrow overlayed over the background image.
Are you able to provide a standalone code sample, without other dependencies, that demonstrates this problem?
const composite = (baseImage, arrow) => {
  return new Promise((resolve, reject) => {
    sharp(baseImage.Body)
      .composite({ input: arrow.Body })
      .toBuffer()
      .then((result) => resolve(result))
      .catch((e) => reject(e));
  });
};

@rmrice Were you able to make any progress with this?

@lovell yes, thanks for following up. The issue was an implementation issue on my part. I'll go ahead and close.


Add bread crumb under project

Coverage decreased (-0.08%) to 64.235% when pulling 5b6fe1fccc927d24b6f0f7b2427a9ffceafe6090 on pengpengshui:fixIssue into e0ac6708d7e802e19385577587de23708542aee0 on vmware:master.

not exactly sure what this is about but the breadcrumb in projects is wrong. it should be
"< Projects < Repositories < "
currently it uses "Registries" instead of "Repositories"
also, if a UX issue, when code is pushed, put issue into verify pipeline and add "pending-ux-review" label


28761 - Add tag filter element to customer list view filter

so its possible to filter for customers with related tags

WIP > https://github.com/NetInventors/sw.ext.neti_tags/pull/10

Released with 1.2.0


Fix the curl command. This time I did test the commands.

Description

Urgency

[ ] Blocker
[x] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

Fix missing curl parameters.

@adyatlov does this only apply to 1.9?


HHVM support

Is your extension supported on HHVM?

No, and we have scraped plans to support HHVM going forwards, because of PHP 7 its usage numbers and requests for supporting it further have decreased a lot. Sorry.
You can use the Composer package tideways/profiler to get callgraph profiling support working. See the docs for installation: https://tideways.io/profiler/docs/setup/hhvm
But we don't support automatic transaction detection, error tracking and timeline profiling support in HHVM.


@paginate in fragments does not work correctly

Describe the bug
It is currently not allowed to have two fields with the @paginate directive in one GraphQL query. However, having a field with the @paginate directive and an included fragment where a field has the @paginate directive is permitted by the compiler. Nevertheless, this causes some odd behaviors.
Calling loadPreviousPage or loadNextPage works as expected for the field declared directly in the query, yet the field declared in the fragment will have its initially loaded elements repeated. The pagination seems to be ultimately controlled from the field declared directly in the query since no more elements will be loaded when the beginning or end is reached for that field regardless of the status of the fragment field.
Additionally removing @paginate from the field declared directly in the query results in loadPreviousPage and loadNextPage not being set, as if the fragment did not exist.
I have a use case where I would like to have multiple fields with @paginate declared in fragments used in other queries. This would make it possible to specify a fragment that a component would require but then a +page.gql query could fulfill, which would be used for paginated dropdown lists in a form in my case.
I propose one way to implement this could be to have one field on the store for each paginated field where loadPreviousPage and loadNextPage could be located, which would control the pagination separately for that field. The linked project with the reproduced bug has examples of this.
I have disabled  defaultFragmentMasking if that makes any difference.
Severity
serious, but I can work around it
Steps to Reproduce the Bug
I have reproduced the bug here: https://github.com/samuel-utbult-oborgen/multiple-pagination-example Just start the development environment as usual with npm run dev, go to http://localhost:5173, click on the Prev and Next buttons and read the source code for more explanations.
Reproduction
https://github.com/samuel-utbult-oborgen/multiple-pagination-example

Thank you for reporting and adding the repro 🙏
Yes, multiple paginate is also in our mind for a long time. When time allows, we would love to tackle it 😊
Looks like a few of you wants it with all these reactions 😉

I am most likely going to circumvent this problem by using separate client evaluated queries (due to some other factors as well) so I won't put more time into this problem for now. But the bug is still present if you want to take a look at it.

I like the idea of opting to n @paginate with labels. (in your case it's a must, but I'm wondering if we could take this approche in case of many @paginate in general?)
What mode are you targeting Infinite and/or SinglePage?

separate client evaluated queries (due to some other factors as well)

I'm curious, will you share your findings? On discord maybe?

Note that you could do all this manually with something like:
query Info ($afterFirst: ID, $afterSecond: ID) {
    first: pokemon(first: 10, after: $afterFirst) {
        ...SpeciesEdges
    }

    second: pokemon(first: 10, after: $afterSecond) {
        ...SpeciesEdges
    }
    ...PokemonFragment
}

And then
// go next first
Info.fetch({ afterFirst: "xxx", afterSecond: Info.variables.afterSecond })

It would be fine with me to write the field name whenever .loadNextPage() or .loadPreviousPage() is called if it means multiple @paginate directives would be supported. I am pretty much always targeting Infinite as I only do cursor pagination and never offset pagination.
I am going to set up a somewhat more complicated example showcasing the kind of page and queries I aim to make (so I will have to respond later). The gist of it is that one page contains a paginated list of customers. These customers can be edited and new ones can be created. Each customer will have a name, a currency and a country. The currency and country can be selected through a dropdown with infinite scrolling. The somewhat easy case is the create case, where the lists starts at the top. The more complicated case is when editing a customer as a currency and a country has to be pre-selected, which means infinite scrolling in both directions needs to be supported.
I have solved this in a previous version of Houdini (where pagination in both directions were not possible) by simply making separate requests and joining the lists together, yet now I would like to do it the proper way. I aim to use only client evaluated queries in the currency and country fields for now but I hope the example could pave a way to somehow have server-side evaluated queries for the fields in the future.

I had a look a bit more in your example 👍
Random thoughts

have the feeling that it should all work if you do @paginate only in fragments.
I saw also that you don't have a node interface, maybe you could add it back?
Doing defaultFragmentMasking: "disable" should not have any effects in this, but I would not recommend when you start a new project. to try to have all data requirement in 1 component.
Any special reason to choose jsDoc over typescript?

I had a look a bit more in your example 👍
Random thoughts

have the feeling that it should all work if you do @paginate only in fragments.
I saw also that you don't have a node interface, maybe you could add it back?
Doing defaultFragmentMasking: "disable" should not have any effects in this, but I would not recommend when you start a new project. to try to have all data requirement in 1 component.
Any special reason to choose jsDoc over typescript?

I tried that before without using defaultFragmentMasking: "disable", but Houdini complained about Query not having an id since you require that for any paginated fragments, yet Query does not have any id since there is only one of it.
This is based on the hello-houdini project, which does not have a node interface.
Okay.
That's because hello-houdini uses jsDoc. I always use Typescript otherwise, including in this more complicated example.

Of course it's the intro! 🤦♂ (too many things turning around)
Do you mind creating a repro with this repo in houdini/e2e/kit? Like this we have everything in place for tests & co :)

Here is a more extensive example of what I am trying to accomplish: https://github.com/samuel-utbult-oborgen/houdini-pagination-example It’s not perfect but I hope you understand what kind of website I’m trying to create. I’ll have to see when I can create a reproduction in houdini/e2e/kit.


Can't pause video when open from the URL.

It always plays video at the first time. I try to set :
self.player.autoplay = false
But It doesn't work.... PLs help me check it out. thank you :)

See the fix here: https://github.com/piemonte/Player/pull/245

#245


Add non-generated gRPC classes under checkstyle/findbugs coverage

Earlier maven exclusions for generated classes were covering gRPC helper/manager classes as well since they use the same package. This PR narrows exclusions down to core-transport module and fixes hidden checkstyle/findbugs issues.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3148/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3149/
Test PASSed.

@gpang PTAL.

@gpang No code change. PTAL.

alluxio-bot, merge this please


Not compatible with Ember fastboot

When trying ember-fastboot on a site that uses semantic-ui, I get all kinds of compile errors regarding missing jQuery and document variables.
Being compatible with fastboot would be REALLY welcome.

Thanks, we are actively working on the 2.0 release, then we'll look at ember fastboot next.

👍
When you need a tester, please let me know.

Is there an update on this one (I guess I'm just hoping someone is taking it on and saves me the hacking ;-))

It will probably be a few months honestly. Unless someone else wants to start a pull request.

Fastboot has now documented how to fix issues like this. See https://github.com/ember-fastboot/ember-cli-fastboot/issues/134#issuecomment-197540068

Latest-n-greatest documentation/guide: http://ember-fastboot.com/docs/addon-author-guide

Any update on the progress?

I don't have any plans to tackle this in the near future. Anyone that wants the feature is free to do the work and submit a PR.

@davidpett has made #184


[Feature Request] Generating Code - PowerShell

I've recently discovered this program. It is an invaluable and versatile tool which I'm sure I will be using way more.
Creating different workspaces for different projects, but also letting me save and (re-)use different REST calls, and my testings, that is a true blessing.
I've noticed that the program also features code generation, which I know will be useful in upcoming projects.
Even though the generate code is available for the most common programming languages, I'm actually missing the possibility to generate code for PowerShell.
I'm currently creating a PowerShell integration with a web interface, which isn't an API. With the PowerShell code generation, I will be able to skip about half of the actions, and about 2/3 of the total work I have to do. I'm sure I wont be the only one benefiting from adding code generation for PowerShell.
Hope you will at least consider the option.
Best Regards
//Me

This might be a good resource for whoever wants to implement this https://github.com/chromium/chromium/blob/9a08a5e480434999d3cfa852b35c45aae1ea9b48/third_party/blink/renderer/devtools/front_end/network/NetworkLogView.js#L1781-L1833

Chrome's Developer Tools has the ability to copy network requests as PowerShell. This might be a good resource for whoever wants to implement this.
https://github.com/chromium/chromium/blob/9a08a5e480434999d3cfa852b35c45aae1ea9b48/third_party/blink/renderer/devtools/front_end/network/NetworkLogView.js#L1781-L1833

Should be in soon! https://github.com/Kong/httpsnippet/pull/127

This was released with 6.5.1 I believe -- confirmed working (y)
https://insomnia.rest/changelog/6.5.1/


use gp types

Conversions from tuples to gp_Pnt/gp_Vec/... are not used in the public API so we'll just ditch them

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.12% :tada:

Comparison is base (236c071) 98.66% compared to head (a26fa03) 98.79%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   98.66%   98.79%   +0.12%
==========================================
  Files           6        6
  Lines         751      744       -7
==========================================
- Hits          741      735       -6
+ Misses         10        9       -1

Files Changed
Coverage Δ

ocpsvg/ocp.py
98.52% <100.00%> (+0.33%)
:arrow_up:

ocpsvg/svg.py
98.11% <100.00%> (ø)

tests/test_ocp.py
100.00% <100.00%> (ø)

tests/test_svg.py
100.00% <100.00%> (ø)

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


error con mi ledger

The device was disconnected.
The above error occurred in task Et
created by takeEvery(ledger/enumerateAccounts, Et)
created by At
Tasks cancelled due to error:
takeEvery(ledger/enumerateAccounts, Et)
undefined

Have you been able to get your Ledger reconnected?
Thanks for adding this to our knowledgebase.
I don't have any actionable ideas for changes to the web wallet regarding this though.

I somehow managed to get my ledger connected after Ctrl + f5. But that mainly happened when I yet didn't have an oasis wallet address on my ledger. Now I can easily access it using the oasis chrome extension.

Thank you for reporting this issue. We have updated our Oasis Wallet with bug fixes and new features which should also solve mentioned issue above.
If you still encounter same or similar issue, please check our FAQ or send an email to our support team at<EMAIL_ADDRESS>


Add GraphQL support

It would be awesome if this library could work for any GraphQL API too.

You mean with client like Apollo ?

@Fabious yes!

I'll have a look on this one too 👍


InitializeOnStartup fails with error 	"The current user is not authorized to access this resource."

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Hot Chocolate
Describe the bug
InitializeOnStartup fails with error 	"The current user is not authorized to access this resource."  for path "/__typename"
Steps to reproduce

Add     [Authorize] attribute to your query. E.g
[ExtendObjectType("Query")]
[Authorize]
public class InvolvedPartyQuery
{
[GraphQLDescription("Returns the involved with the specified sebCustomerNumber")]
Add GraphQL service
services
.AddGraphQLServer()
.InitializeOnStartup()
.AddAuthorization()
.AddQueryType(q => q.Name("Query"))
.AddType()
.AddType()
.AddType();
Add service for Authentication and Authorization
services.AddAuthentication(auth =>
{
auth.DefaultAuthenticateScheme = JwtBearerDefaults.AuthenticationScheme;
auth.DefaultChallengeScheme = JwtBearerDefaults.AuthenticationScheme;
}).AddJwtBearer();
services.AddAuthorization();
At configure Authentication and Authorization
app.UseAuthentication();
app.UseAuthorization();

Starting the application will give error "The current user is not authorized to access this resource." for path "/__typename"
Relevant log output
-		error	{HotChocolate.Error}	HotChocolate.IError {HotChocolate.Error}
		Code	"AUTH_NOT_AUTHENTICATED"	string
+		Exception	null	System.Exception
+		Extensions	{HotChocolate.Execution.OrderedDictionary<string, object>}	System.Collections.Generic.IReadOnlyDictionary<string, object> {HotChocolate.Execution.OrderedDictionary<string, object>}
+		Locations	Count = 1	System.Collections.Generic.IReadOnlyList<HotChocolate.Location> {System.Collections.Generic.List<HotChocolate.Location>}
		Message	"The current user is not authorized to access this resource."	string
+		Path	{/__typename}	HotChocolate.Path {HotChocolate.NamePathSegment}
+		SyntaxNode	{__typename}	HotChocolate.Language.ISyntaxNode {HotChocolate.Language.FieldNode}

Additional Context?
No response
Version
12.15.2

Issues solved when I upgraded to version 13.2.0. Thanks :)


Fix hoek security vulnerability

Github is reporting a vulnerability in one of our (indirect) dependencies:

We should update our dependencies to resolve this as soon as possible (assuming of course, that they themselves have updated to a newer version of hoek).

so apparently this is something that @buildit/gravy has, and it's due to https://github.com/sass/node-sass/issues/2355 which won't be fixed until v5 is out.

related gravy issue is: https://github.com/buildit/gravy/issues/16


Merge pull request #299 from girldevelopit/master

updating staging to master

This should realign master and staging


Started Documentation

Created BATTLES.md (closes #331)
Added line to README.md about gulping folders

One more meta-comment: can you move this to docs\Battles.md?

@guvjose I'm closing this PR because it now has a bunch of merge conflicts. If you want to send a new one, by all means do so.


JSON content system doesn't support setting content for multiple, arbitrary instances of an element in a page.

As a developer, I would like to be able to reference atoms, modules, and organisms whenever possible to avoid duplication and to make changes that apply across the whole project.
As designers, my UX and design colleagues want to present a page that looks as close to the final version of the site as possible.
We run into an issue when, for example, I have multiple product information organisms on the page. I want to be able to link to the organism dynamically but that prevents me from setting unique data for each item (except for the listitem feature which doesn't really allow enough flexibility for what I want). I would like to be able to set json data for each item including nested items like button labels.
(If there's some way to do this already, please let me know!)
One solution could be to add some mechanism for giving a partial an id or other parameters.
{{buttonText}}

{{> atoms-button}}

could become
{{buttonText}}

{{> atoms-button#submit}}

This could allow you to set data based on ID.
{
    buttonText: "foo",
    buttonURL: "http://foo.com"
}

could become something like
{
    // default
    buttonText: "foo",
    buttonURL: "http://foo.com",

    "#submit": {
        buttonText: "submit",
        buttonURL: "/form"
    },
    "#buyNow": {
        buttonText: "Buy buy buy!",
        buttonURL: "/store"
    }
}

Another possibility would be to allow setting of content based on CSS selectors in a similar way that annotations are written. That could allow you to wrap any element with a class or id div in order to force it to be targeted by the json.

I was poking around with demo.patternlab.io and realized this is possible with Mustache lists. It might be worth mentioning in the documentation.


If You’re Using Kafka With Your Microservices, You’re Probably Handling Retries Wrong / Dave Taubler

URL: https://dt-23597.medium.com/if-youre-using-kafka-with-your-microservices-you-re-probably-handling-retries-wrong-8492890899fa
Review (1-2 sentences): If you deal with integration events from micro-/services as a source. Prerequisites of retry topic, typical architecture, recoverable and non-recoverable errors, ordering problems.

@k-tomak Link is not working anymore. The author has deleted the article :(


pythonPackages.vxi11: init at 0.9

Motivation for this change
This is a useful package for interfacing with test and measurement equipment.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Done.

@GrahamcOfBorg build python2.pkgs.vxi11 python3.pkgs.vxi11

Success on x86_64-linux (full log)
Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11
Partial log (click to expand)
reading manifest template 'MANIFEST.in'
writing manifest file 'python_vxi11.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
/nix/store/0lai495pmd5da2phrick18kzwyxkrfzf-python2.7-python-vxi11-0.9
/nix/store/d49bih2m4spyvcckq1ni9h88j7vxwxmh-python3.6-python-vxi11-0.9

Success on aarch64-linux (full log)
Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11
Partial log (click to expand)
reading manifest template 'MANIFEST.in'
writing manifest file 'python_vxi11.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
/nix/store/g7s6mqvh9wzw9ascap24nx37hgqramgd-python2.7-python-vxi11-0.9
/nix/store/jjqlg3g9mh5ma26ckjac50m4fwn9pdfh-python3.6-python-vxi11-0.9

Ran 0 tests in 0.000s

Please specify an appropriate checkPhase, e.g.
checkInputs = [ nose ];
checkPhase = ''
  nosetests
'';

or set doCheck = false if the tests require manual intervention (also add a comment on why they are disabled in that case).

Done.
Thanks again for the reviews!

@GrahamcOfBorg build python2.pkgs.vxi11 python3.pkgs.vxi11

Success on x86_64-linux (full log)
Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11
Partial log (click to expand)
Ran 0 tests in 0.012s

OK

----------------------------------------------------------------------
Ran 0 tests in 0.019s

OK
/nix/store/9aq51h8w23v78m0lvvsb9ks94nryq369-python2.7-python-vxi11-0.9
/nix/store/w87vx5isjfc6y93vbhhhkdgas1s2sqyc-python3.6-python-vxi11-0.9

Success on aarch64-linux (full log)
Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11
Partial log (click to expand)

OK
running install tests

----------------------------------------------------------------------
Ran 0 tests in 0.062s

OK
/nix/store/y856l4p4ki7mr3wlp88xhm2za380pjqv-python2.7-python-vxi11-0.9
/nix/store/q3qncf1fa2453an5w22j3ana0alns97l-python3.6-python-vxi11-0.9


Canada is not in North America?!?

Why is Canada's geography called "Canada", while USA's/America's is "North America"?
This shows as ignorance and arrogance on Microsoft's part.

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

ID: 09a788e4-0130-73f2-d985-6fc3aec77974
Version Independent ID: a9f4ec3f-f0a3-b050-020b-f487c74c5179
Content: Cross-region replication in Azure
Content Source: articles/reliability/cross-region-replication-azure.md
Service: reliability
Sub-service: availability-zones
GitHub Login: @anaharris-ms
Microsoft Alias: anaharris

@davidsandbrand
I've delegated this to @anaharris-ms, a content author, to review and share their valuable insights.

Hi @davidsandbrand,
Thanks for contacting Microsoft. I am looking into this issue to see what the story is behind it.
Have a great evening/day!

Fix: https://github.com/MicrosoftDocs/azure-docs-pr/pull/247607.
#please-close


Bump flatcar to latest version

Please check if PR meets these requirements

[x] Results of the diffs have been examined and no unintended changes are being introduced.

Helper

to disable the GH Action generating manifests diffs for installations, between target and source branches, comment the /no_diffs_printing on the PR.

Since main is broken I tested it here https://github.com/giantswarm/cluster-azure/pull/258.

I had to build it manually. It's how it's done for all providers at the moment

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites