Warning "Could not import "pyside" bindings of qt_gui_cpp library" when opening rqt_gui with pyside

By doing:
rosrun rqt_gui rqt_gui --qt-binding=pyside

I see:
    Could not import "pyside" bindings of qt_gui_cpp library - so C++ plugins will not be available:
Traceback (most recent call last):
    File "/u/isaito/link/rqt_dev/git/qt_gui_core/qt_gui_cpp/src/qt_gui_cpp/cpp_binding_helper.py", line 43, in <module>
    import libqt_gui_cpp_shiboken
    ImportError: No module named libqt_gui_cpp_shiboken

python_qt_binding is git pulled on 11/8/2012 3pm.

I still have that issue. Did I miss something?
mapper@mapper-VirtualBox:/tmp$ source /opt/ros/indigo/setup.bash
mapper@mapper-VirtualBox:/tmp$ rosrun rqt_gui rqt_gui --clear-config --qt-binding=pyside
Could not import "pyside" bindings of qt_gui_cpp library - so C++ plugins will not be available:
Traceback (most recent call last):
  File "/opt/ros/indigo/lib/python2.7/dist-packages/qt_gui_cpp/cpp_binding_helper.py", line 39, in <module>
    import libqt_gui_cpp_shiboken
ImportError: No module named libqt_gui_cpp_shiboken

Versions:
mapper@mapper-VirtualBox:/tmp$ dpkg -l | grep pyside
...
ii  python-pyside                                         1.2.1-4build1                                             all          Python
ii  ros-indigo-python-qt-binding                          0.2.14-0trusty-20141230-0031-+0000                        amd64
...
mapper@mapper-VirtualBox:/tmp$ dpkg -l | grep qt-gui
ii  cmake-qt-gui                                          <IP_ADDRESS>-0ubuntu3                                         amd64        Qt4 based user interface for CMake (cmake-gui)
ii  ros-indigo-qt-gui                                     0.2.26-0trusty-20141230-0050-+0000                        amd64        qt_gui provides the infrastructure for an integrated graphical user interface based on Qt. It is extensible with Python- and C++-based plugins (implemented in separate packages) which can contribute arbitrary widgets. It requires either PyQt or PySide bindings.
...
ii  ros-indigo-rqt-gui-py                                 0.2.14-1trusty-20141230-0443-+0000                        amd64        rqt_gui_py enables GUI plugins to use the Python client library for ROS.

The current binaries built on Trusty do not support PySide since the shipped shiboken version has a bug which prevents it from working: https://github.com/ros-visualization/qt_gui_core/blob/efc7692eab423a1cb7447c27f4e200d2216db2a4/qt_gui_cpp/src/qt_gui_cpp_shiboken/CMakeLists.txt#L44-L46
Until upstream fixes the problem there is nothing we can do about it.

Ok, thanks for your explanation.

I am having this issue as well and have been unable to resolve it. I am not getting a segmentation fault and have been able to duplicate the issue on a different computer with the same setup.
Setup:
Ubuntu Bionic 18.04.4 LTS
ROS1 Melodic / ROS2 Dashing
...Used ROS Industrial Shell Script to Install + var fixes.
Q: Is this issue inherent to the Ubuntu Bionic binaries with the broken shiboken version as well?
I have been unable to determine if this bug was resolved upstream by PySide or if this is an error of my own actions.
Thanks!
Error
$ rosrun rqt_graph rqt_graph
PluginManager._discover() using cached plugin discovery information
Could not import "pyside" bindings of qt_gui_cpp library - so C++ plugins will not be available:
Traceback (most recent call last):
  File "/opt/ros/melodic/lib/python2.7/dist-packages/qt_gui_cpp/cpp_binding_helper.py", line 39, in <module>
    from . import libqt_gui_cpp_shiboken
ImportError: cannot import name libqt_gui_cpp_shiboken

Versions
...
ii  python-pyside                                      1.2.2+source1-3
ii  ros-melodic-python-qt-binding             0.3.6-2bionic.20191210.224309
...
ii  ros-melodic-rqt-gui                                0.5.0-0bionic.20191210.230329
ii  ros-dashing-rqt-gui-py                           1.0.5-1bionic.20191212.195003

ubuntu18.04
conda 环境 python2.7
ros melodic
File "/opt/ros/melodic/lib/python2.7/dist-packages/qt_gui_cpp/cpp_binding_helper.py", line 39, in
from . import libqt_gui_cpp_shiboken
ImportError: cannot import name libqt_gui_cpp_shiboken
实在不知道怎么解决了


Add privatelink for Azure Cosmos Synapse Analytics

Please add the following private link for Azure Cosmos using Synapse Analytics
privatelink.analytics.cosmos.azure.com
See further MSFT docs here: https://learn.microsoft.com/en-us/azure/cosmos-db/analytical-store-private-endpoints

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

ID: a88af182-32aa-a8f3-359d-b92ae1cfb8c7
Version Independent ID: 01ce07fc-4bc2-def5-9ecd-a80330ad9488
Content: Azure Private Endpoint private DNS zone values
Content Source: articles/private-link/private-endpoint-dns.md
Service: private-link
GitHub Login: @asudbring
Microsoft Alias: allensu

@jmyers82
Thanks for your feedback! We will investigate and update as appropriate.

@jmyers82
Thank you for bringing this to our attention! We've initiated a pull request (PR) to address this issue. Once the author reviews the changes, they will be merged, and the updates should go live shortly. Your diligence is much appreciated!


Setup rust code linting

Setup clippy in CI to lint rust code.
Will probably require code fixes since the rust code currently won't pass all of the lints. To avoid some unnecessary work should do this only after:

https://github.com/oasisprotocol/oasis-core/issues/3572

We can also think about integrating more static analysis into our Rust (eg: https://github.com/rust-lang/miri).


report unused call results (à la pyright)

Suggestion
In pyright there is reportUnusedCallResult but I couldn't find anything similar in TypeScript: TSConfig Reference, please correct me if I missed it.
🔍 Search Terms

reportUnusedCallResult
call result
unused call
unused

✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
A similar option to pyright's reportUnusedCallResult could be added and being disabled it's default state and maybe enabled with strict flag?
📃 Motivating Example
// some code

get_chichi()

// more code

One reads this and wonders, why would one get_chichi here, chichi is not used anywhere.

Maybe the person who wrote this forgot to use the value?
Maybe is a leftover and can be removed?
Maybe this function has a side effects and it should be renamed to reflect that?

💻 Use Cases
Reminding the developer about unused call results makes it harder to see code like in the example above.
One problem I see with the proposal above is that there seems to be no convention in javascript for throwaway variables, but according to What is the equivalent of python's _ in javascript? - Stack Overflow and naming - What's the idiomatic name for a throwaway variable? - Software Engineering Stack Exchange dummy, temp, trash, black_hole, garbage seem to be widely used.
After tsc complains about the code above one would be forced to either remove that call,
assign the result and use it or assign it to a throwaway variable.

This sounds better suited as a linter rule instead of being part of the compiler. But I'd expect the false positives to be immensely.

you are right, closing, I'll bring this to a linter project we end up using, thanks


feat(#314): ngMocks.formatText

closes #314

v11.10.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Forwarder and Http metrics consumers produce incorrect values when used together

Describe the bug
When adding both an IMetricsConsumer for HttpMetrics and one for ForwarderMetrics, the previous or current metric values can contain incorrect values.
This is because apparently when using EventListeners for event counters (event ID -1), the filtering on event source is not applied and all listeners get all event counter events for all enabled sources. See: https://github.com/dotnet/runtime/issues/31500
The specific issue with forwarder and HTTP metrics has to do with the fact that both of these event sources define counters with the same names, like requests-started.
Both ForwarderEventListenerService and HttpEventListenerService will receive all EventCounters events due to the filtering issue, and since requests-started is considered a valid counter name in both, then the values set on the metrics objects that are passed to OnMetrics will be coming from either source and can be invalid depending on the order of the received events.
One easy solution would be to always (or when event name is EventCounters) filter on the event source name in OnEventWritten here: https://github.com/microsoft/reverse-proxy/blob/main/src/TelemetryConsumption/EventListenerService.cs
To Reproduce

Add both an IMetricsConsumer for HttpMetrics and one for ForwarderMetrics.
Log the values for previous and current metrics.
Make sure that you have some load that will cause the number of started requests from the forwarder to be different than the number of started HTTP requests so that you can see the difference.

Expected:
For each consumer, in the case of requests started (as an example), current should always be higher than previous, and each new previous should be the value of the last current.
Actual:
This is the output we see in our telemetry coming from the forwarder metrics consumer when both consumers are enabled:
2023-04-17T14:38:56.583459Z	ForwarderRequestsStarted - Previous: 955, Current: 1151, Value: 196
2023-04-17T14:38:56.786638Z	ForwarderRequestsStarted - Previous: 1151, Current: 955, Value: -196
2023-04-17T14:38:57.583489Z	ForwarderRequestsStarted - Previous: 955, Current: 1152, Value: 197
2023-04-17T14:38:57.786649Z	ForwarderRequestsStarted - Previous: 1152, Current: 956, Value: -196
2023-04-17T14:38:58.583575Z	ForwarderRequestsStarted - Previous: 956, Current: 1153, Value: 197
2023-04-17T14:38:58.786836Z	ForwarderRequestsStarted - Previous: 1153, Current: 957, Value: -196
2023-04-17T14:38:59.583638Z	ForwarderRequestsStarted - Previous: 957, Current: 1155, Value: 198
2023-04-17T14:38:59.786577Z	ForwarderRequestsStarted - Previous: 1155, Current: 959, Value: -196
Further technical details

Package versions: 2.0.0
Platform: Windows

Thanks for reporting this, I was not aware of the EventListener bug :/
We should work around it in YARP.


Fail to display the static website with 404 after deploying

OS: Win10
Build Version: 20200105.1
Repro Steps:

Create a storage account --> Deploy a project to the static website.
Check the result.

Expect:
Succeed to deploy and the static website displays well.
Actual:
Succeed to deploy, but there displays 404 when browsing the website.

More Info:
This issue also reproduces on the previous release (0.7.2) for now. But this issue didn't reproduce when tested the previous release (0.7.2).

This looks like it was a temporary problem with Azure (especially since a possibly similar issue was filed near the same time.) Can't repro so closing for now


Strange temporary file error

docker run -v "$(pwd):/src/" cdrx/pyinstaller-windows:python2
...
355 INFO: Removing temporary files and cleaning cache in C:\users\root\Application Data\pyinstaller
Traceback (most recent call last):
  File "C:\Python27\Scripts\pyinstaller-script.py", line 9, in <module>
    load_entry_point('PyInstaller==3.2.1', 'console_scripts', 'pyinstaller')()
  File "C:\Python27\lib\site-packages\PyInstaller\__main__.py", line 90, in run
    run_build(pyi_config, spec_file, **vars(args))
  File "C:\Python27\lib\site-packages\PyInstaller\__main__.py", line 46, in run_build
    PyInstaller.building.build_main.main(pyi_config, spec_file, **kwargs)
  File "C:\Python27\lib\site-packages\PyInstaller\building\build_main.py", line 788, in main
    build(specfile, kw.get('distpath'), kw.get('workpath'), kw.get('clean_build'))
  File "C:\Python27\lib\site-packages\PyInstaller\building\build_main.py", line 691, in build
    os.makedirs(pth)
  File "C:\Python27\lib\os.py", line 157, in makedirs
    mkdir(name, mode)
WindowsError: [Error 123] Invalid name: '/tmp\\*'

Have you seen this before?

It seems that for some reason the volume I'm mounting isn't accessible to the container user. Adding an ls -l / at startup shows that user 1000 has permissions on the directory.
I'm on docker 1.10 / Fedora 24. I'll dig some more.

Ah, looks like an selinux issue.
http://stackoverflow.com/questions/24288616/permission-denied-on-accessing-host-directory-in-docker

@virtuald Is this Fedora only and the solution permissions? The same behavior occurs on Debian (with no selinux):
$ uname -a
Linux debian 3.16.0-4-amd64 #1 SMP Debian 3.16.39-1+deb8u1 (2017-02-22) x86_64 GNU/Linux

$ pwd
/home/user123/

$ docker run -v "$(pwd):/src/" cdrx/pyinstaller-windows:python2

err:winediag:SECUR32_initNTLMSP ntlm_auth was not found or is outdated. Make sure that ntlm_auth >= 3.0.25 is in your path. Usually, you can find it in the winbind package of your distribution.
934 INFO: PyInstaller: 3.2.1
935 INFO: Python: 2.7.12
937 INFO: Platform: Windows-7-6.1.7601-SP1
940 INFO: UPX is not available.
940 INFO: Removing temporary files and cleaning cache in C:\users\root\Application Data\pyinstaller
Traceback (most recent call last):
  File "C:\Python27\Scripts\pyinstaller-script.py", line 9, in <module>
    load_entry_point('PyInstaller==3.2.1', 'console_scripts', 'pyinstaller')()
  File "C:\Python27\lib\site-packages\PyInstaller\__main__.py", line 90, in run
    run_build(pyi_config, spec_file, **vars(args))
  File "C:\Python27\lib\site-packages\PyInstaller\__main__.py", line 46, in run_build
    PyInstaller.building.build_main.main(pyi_config, spec_file, **kwargs)
  File "C:\Python27\lib\site-packages\PyInstaller\building\build_main.py", line 788, in main
    build(specfile, kw.get('distpath'), kw.get('workpath'), kw.get('clean_build'))
  File "C:\Python27\lib\site-packages\PyInstaller\building\build_main.py", line 691, in build
    os.makedirs(pth)
  File "C:\Python27\lib\os.py", line 157, in makedirs
    mkdir(name, mode)
WindowsError: [Error 123] Invalid name: '/tmp\\*'

Are you positive that it's not an selinux (or similar) error? I added :Z to the end of the volume path and it worked after that.

For some reason there is a Security-Alert issue:
dave@debian:~/Trumpocalypse/UI$ docker run -v "$(pwd):/src/" cdrx/pyinstaller-windows:python2 "pyinstaller -y Trumpocalypse.py"

...

9041 INFO: Updating manifest in C:\users\root\Application Data\pyinstaller\bincache00_py27_32bit\_socket.pyd
9042 INFO: Updating resource type 24 name 2 language 1033
9065 INFO: Updating manifest in C:\users\root\Application Data\pyinstaller\bincache00_py27_32bit\pyexpat.pyd
9066 INFO: Updating resource type 24 name 2 language 1033
9077 INFO: Updating manifest in C:\users\root\Application Data\pyinstaller\bincache00_py27_32bit\select.pyd
9078 INFO: Updating resource type 24 name 2 language 1033
9086 INFO: Updating manifest in C:\users\root\Application Data\pyinstaller\bincache00_py27_32bit\_ctypes.pyd
9087 INFO: Updating resource type 24 name 2 language 1033
9101 INFO: Updating manifest in C:\users\root\Application Data\pyinstaller\bincache00_py27_32bit\bz2.pyd
9101 INFO: Updating resource type 24 name 2 language 1033
9121 INFO: Updating manifest in C:\users\root\Application Data\pyinstaller\bincache00_py27_32bit\unicodedata.pyd
9122 INFO: Updating resource type 24 name 2 language 1033
Security-Alert: try to store file outside of dist-directory. Aborting. 'Z:\\src\\build\\Trumpocalypse\\Trumpocalypse.exe.manifest'

Did you try?
docker run -v "$(pwd):/src/:Z" cdrx/pyinstaller-windows:python2 "pyinstaller -y Trumpocalypse.py"

Yeah. Same issue.

As a side note, this might happen if in the current folder there is no .spec file. Probably in the entrypoint.sh there should be a check for that, because the error is very confusing.


fix bug: branching at a node with null swipe_id created broken chat

As it says on the tin.
I don't know how I didn't catch this before - maybe it's only some chat files where a message has a null swipe_id.
Anyway, fixed, works now.
EDIT: Sneaked in another small bugfix.
Changelog:

Fix the branch button in the full info panel sometimes creating a broken chat
Fix exception in Timelines when hitting Esc to close the Manage chat files view (of core ST)
Fix and improve consistentency in handling of zoom to fit (no more uncaught exceptions from this, hopefully)
UX: Show loading spinner while the timeline view is opening
UX: Fix incorrect position of root node (especially if the character has lots of chats)

Argh, still finding more bugs. This time, there was an uncaught exception in the log when opening the timeline view and the text search was empty.
I'll let you know when there are no more bugs for now. Probably tomorrow.

I think that's all for now. This PR is complete.

Thanks for the merge!


[Feat] Use Ant Design

Hi.
I would like to know if it is possible to use Ant Blazor Design (https://ant-design-blazor.github.io/en-US/) instead of Bootstrap?
Thank you.

Yes, its possible. Just change css classes in source.


Google Pay window getting cropped when using Google Chrome + Windows

Describe the bug
Hey folks! 👋  In our setup we have incorporated the React GP button inside a relatively small (width = ~400px) popup. Unfortunately this setup is causing cropping problems for some of our users that are using Google Chrome + Windows. When the user clicks the GP Button inside our popup the GP Payment window that pops up is being cropped. Due to this the user cannot see the 'Pay' button and is therefore not able to pay until they resize the window. We can solve this on our side by changing the width of our popup, but in my opinion GP should also work inside these smaller browser windows.
To Reproduce

Setup a GooglePay button (react) inside a window having a width smaller than 600px.
Open the app using Google Chrome on Windows.
Click the button.
See that the opened payment window is cropped.

Screenshots

Expected behavior
Obviously this window should not be cropped.
For example, on Google Chrome + MacOS the window is not being cropped:

Component information:

Component

[x] React component (@google-pay/button-react)
[ ] Custom element (@google-pay/button-element)
[ ] Angular component (@google-pay/button-angular)

Component version (e.g. 1.0.0): 2.6.0

Environment:

OS (e.g. iOS8.1): Windows 10
Browser (e.g. stock browser, safari): Chrome 96.0.4664.110
Country/region: US

I believe Mac's behaviour is not expected. Filed crbug.com/1283234 for the content area partially living out of the window on Mac.
It has been brought to GPay team's attention as well.
Have you tried it on Mac's Chrome Canary? Is it reproducible there?

Closing as this is a Chrome issue: crbug.com/1283234


Automatically closed when setState of input that inside of AccordionItemBody

handleConfigChange(e){
    let {name, value} = e.target;
    this.setState({[name]: value})
}

<Accordion>
  <AccordionItem>
    <AccordionItemTitle>

        Title

    </AccordionItemTitle>
    <AccordionItemBody>

          <input name="title" className="form-control" placeholder="Title" defaultValue={title}
                 onChange={this.handleConfigChange}/>

          Text Color:
          <input type="color" name="title_color" className="form-control" defaultValue=
            {title_color} onChange={this.handleConfigChange}/>

    </AccordionItemBody>
  </AccordionItem>
</Accordion>

@umair-khanzada could you please elaborate a little? I can't tell from your issue what the bug is.

Actually when I use controlled component inside of AccordionItemBody , the AccordionItemBody automatically get closed when user interact with controlled component , works good with uncontrolled component.

@umair-khanzada any chance you have a live demo of the bug? This looks quite obscure to me.

I have the same issue with the latest version. With version 0.5.0 works well.

@Toki-dev  or @hyanmandian any chance you could provide a clear example please?

Yes @vincentaudebert, I've created an example: https://codesandbox.io/s/lyp4yx85l9

any updates?

Yes @hyanmandian there's a PR in progress #35 but we struggle a bit to unit test it. So we might just rewrite everything using Redux. Sorry for the delay. Feel free to contribute to the PR as a quick fix otherwise.

Just a quick update - we've made good progress on the refactor and we anticipate a new release in the next couple of weeks. Sorry for the delay, but @vincentaudebert and I both have full-time jobs and we've both been absolutely swamped for a while now, so we've found it difficult to make time for this!

And it's alive!
Thanks heaps @ryami333 for the awesome work!
https://www.npmjs.com/package/react-accessible-accordion v2.0.0

Still getting this problem in v3.3.0, on radio buttons in a react component inside the body of the accordion


could not resolve all files for configuration '"eclipsePlugin:compileClasspath'

When I compile the project using the gradle assemble command, the following error occurred。My Eclipse installation path is D:Eclipse_all/eclipse_noweclipse-jee-photon-R-win32-x86_64/eclipse-jee-photon-R-win32-x86_64/eclipse/.And i was escaped。
This path is not resolved。I changed my Eclipse path to D:/eclipse, it builds sucess.

Expected Behavior

Current Behavior

failure:build failed with an exception.
*what went wrong:
could not normalize path for file  'D:\Eclipse_all\eclipse-workspace\spotbugs_wfei_for\spotbugs\eclipsePlugin\D:Eclipse_alleclipse_noweclipse-jee-photon-R-win32-x86_64eclipse-jee-photon-R-win32-x86_64eclipse\plugins'.
Context

Steps to Reproduce

Your Environment
Eclipse Java EE IDE for Web Developers.
Version: Photon Release (4.8.0)
Build id: 20180619-1200
jdk1.8
Build scan URL:

The issue comes from using an invalid absolute path. D:Eclipse_all is not recognized as absolute because it misses a \ after the D. Because of that, Gradle considered the path to be relative and thus failed to resolve it.
As you stated, the fix was to properly add the missing character. And indeed a / works as well.
There is nothing to be fixed in Gradle.


Bump version to 5.1.0 to support Spark 3.2

Bump version for the changes below:

Make manifest file path use s3a/n scheme
Add catalyst type mapping for LONGVARCHAR
Upgrade to Spark 3.2
Fix log4j-apt compatability with Spark 3.2

hello, I can't find 5.1.0 version on mvn central, could you upload it or is it somewhere else ?

hello, I can't find 5.1.0 version on mvn central, could you upload it or is it somewhere else ?

@AlixMetivier sorry for the delay. I have currently released this version to be forked for any usage.
I can get it released to Maven Central before Wed, 10/05.

hello, I can't find 5.1.0 version on mvn central, could you upload it or is it somewhere else ?

@AlixMetivier sorry for the delay. I have currently released this version to be forked for any usage. I can get it released to Maven Central before Wed, 10/05.

@AlixMetivier I was able to prioritize it today. It should be available on Maven Central anytime from now. Let me know if you still faced any issues.


Integrate @astevens mod updater

@astevens mod updater works nicely and I was wondering if there could be some integration into factorio init, similar to how @narcotiq has made a factorio update script that has been integrated into the factorio init script.

https://github.com/Tantrisse/Factorio-mods-manager.git is already integrated, see "./factorio mod help" - for now that's sufficient imo, but feel free to suggest adaptions/changes as you see fit :)


[Adaptive] [DCR] C# missing DialogManager class

The C# code still has RunAsync() and OnTurnAsync() methods on the AdaptiveDialog. This functionality should be moved to a DialogManager class in the Microsoft.Bot.Builder.Dialogs namespace.
[dcr]

Not needed. Works with MVC

We might need to reconsider this post build for all the additional functionality including session expiry policy etc.


stable-2.2: ceph-common role creates invalid nfs-ganesha configuration

The ganesha.conf template, roles/ceph-common/templates/ganesha.conf.j2, will not create a working nfs-ganesha configuration.
According to https://github.com/nfs-ganesha/nfs-ganesha/issues/146#issuecomment-287348793, the NFS_Protocols and Transport_Protocols are wrong. https://github.com/nfs-ganesha/nfs-ganesha/blob/master/src/config_samples/config.txt confirms this.

Thanks for reporting this, it looks like master is also suffering from this. I'd be happy to merge a PR for this.
@alimaredia, fyi.

@leseb In master, ganesha.conf.j2 correctly generates Protocols and Transports.

@fghaas I highly recommend you use ceph-ansible 3.0 (once it's released) for using nfs-ganesha and Ceph. As you can see from master, I made quite a few changes to the nfs-ganesha related code in ceph-ansible. If you have any issues or questions feel free to ask me.

Yeah but it's not released yet, sorry. I can't use unreleased code for production systems. That's precisely why I'm sending fixes for the stable branch.

@fghaas Understood, just wanted to let you know for the future. As Sebastian said, if you open up a pull request with the fix I would be glad to merge it.

@alimaredia #1912

Fixed in #1912.


Problem with HDF5 and downloaded binary

+ /home/UNIXHOME/cdunn/repo/gh/FALCON-integrate/FALCON-examples/run/greg200k-sv2/DOWN/bin/pbalign -h
Warning! ***HDF5 library version mismatched error***
The HDF5 header files used to compile this application do not match
the version used by the HDF5 library to which this application is linked.
Data corruption or segmentation faults may occur if the application continues.
This can happen when an application was compiled by one version of HDF5 but
linked with a different version of static or shared HDF5 library.
You should recompile the application or check your shared library related
settings such as 'LD_LIBRARY_PATH'.
You can, at your own risk, disable this warning by setting the environment
variable 'HDF5_DISABLE_VERSION_CHECK' to a value of '1'.
Setting it to 2 or higher will suppress the warning messages totally.
Headers are 1.8.16, library is 1.8.17

$ ldd /home/UNIXHOME/cdunn/repo/gh/FALCON-integrate/FALCON-examples/run/greg200k-sv2/DOWN/bin/samtools
        linux-vdso.so.1 =>  (0x00007fff991fe000)
        libncurses.so.6 => /home/UNIXHOME/cdunn/repo/gh/FALCON-integrate/FALCON-examples/run/greg200k-sv2/DOWN/bin/../lib/libncurses.so.6 (0x00007f6b28819000)
        libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f6b284fb000)
        libz.so.1 => /home/UNIXHOME/cdunn/repo/gh/FALCON-integrate/FALCON-examples/run/greg200k-sv2/DOWN/bin/../lib/libz.so.1 (0x00007f6b282e2000)
        libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f6b280de000)
        libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f6b27ebf000)
        libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f6b27af9000)
        /lib64/ld-linux-x86-64.so.2 (0x00007f6b28a7f000)

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 14.04 LTS
Release:        14.04
Codename:       trusty

Also happens on Centos6.6.

Apparently, our Anaconda picks up its own shared libraries, despite my LD_LIBRARY_PATH.

export HDF5_DISABLE_VERSION_CHECK=1

Seems to work. Other Python works fine too.

export HDF5_DISABLE_VERSION_CHECK=1

Seems to work for pbalign, but fails for ConsensusCore.
ImportError: /mnt/software/a/anaconda2/4.2.0/bin/../lib/libstdc++.so.6: version `GLIBCXX_3.4.20' not found (required by /home/UNIXHOME/cdunn/repo/gh/FALCON-integrate/FALCON-examples/run/greg200k-sv2/DOWN/lib/python2.7/site-packages/ConsensusCore-1.0.2-py2.7.egg/_ConsensusCore.so)

Yes, I first ran module load gcc/4.9.2, which solved the glibcxx problem for other Pythons. Possibly, Anaconda uses its own libgcc too? Maybe via rpath when built?
Other Python works fine. Closing.


Cilicon runner don't restart after gitlab job is finished in Cilicon version 2.2.0

Issue:
Cilicon 2.2.0 vm don't restart when gitlab job is done as shown in GIF.
Followed setup in Readme.md
Background:
From the cilion cycle the vm is stopped and removed after a job is done.
In the instructions it is not mentioned any setting required to restart Cilion VM. From the additional swift settings it seems numberOfRunsUntilHostReboot is the most likley setting to make it restart.
Settings
Cilicon version: v2.2.0
Mac version
ProductName:		macOS
ProductVersion:		14.5
BuildVersion:		23F79

cat ~/cilicon.yml
source: oci://ghcr.io/cirruslabs/macos-sonoma-xcode:15.3
provisioner:
  type: gitlab
  config:
    gitlabURL: https://gitlab.com
    runnerToken: <replace with gitlab-ci-cd-token>
    maxNumberOfBuilds: 1
numberOfRunsUntilHostReboot: 1

Running:
Configuring GitLab Runner...
Successfully configured GitLab Runner
Downloading GitLab Runner Binary from Source
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  3 61.5M    3 2330k    0     0  1944k      0  0:00:32  0:00:01  0:00:31 1943k
 26 61.5M   26 16.1M    0     0  7747k      0  0:00:08  0:00:02  0:00:06 7746k
 43 61.5M   43 26.5M    0     0  8614k      0  0:00:07  0:00:03  0:00:04 8613k
 60 61.5M   60 36.9M    0     0  9153k      0  0:00:06  0:00:04  0:00:02 9153k
 78 61.5M   78 48.0M    0     0  9578k      0  0:00:06  0:00:05  0:00:01 9989k
 96 61.5M   96 59.4M    0     0  9896k      0  0:00:06  0:00:06 --:--:-- 11.5M
100 61.5M  100 61.5M    0     0  9921k      0  0:00:06  0:00:06 --:--:-- 10.7M
Downloaded GitLab Runner Binary from Source successfully
Starting GitLab Runner...
Runtime platform                                    arch=arm64 os=darwin pid=660 revision=44feccdf version=17.0.0
Starting multi-runner from /Users/admin/.gitlab-runner/config.toml...  builds=0 max_builds=0
WARNING: Running in user-mode.
WARNING: Use sudo for system-mode:
WARNING: $ sudo gitlab-runner...

.
.
.
Submitting job to coordinator...accepted, but not yet completed  bytesize=707934 checksum=checksum code=202 job=[job-number] job-status=running runner=[runner-id] update-interval=1s

Submitting job to coordinator...ok.      bytesize=707934 checksum=[checksum] code=200 job=[job-number] job-status=success runner=[runner-id] update-interval=0s

Removed job from processing list.        builds=0 job=[job-number] max_builds=1 project=[gitlab-project]  repo_url=[gitlab-repo-url]  time_in_queue_seconds=0

@fberndtsson numberOfRunsUntilHostReboot reboots the machine running Cilicon after a certain number of runs (VM restarts). This is an optional feature to have a fresh boot every once in a while.
Cilicon will restart its VM when the command that it runs via SSH exits. Since I don't use gitlab, I can't replicate your issue, but to me it looks like the command that is run never exits. This theory is supported by the fact that your log claims to be a "multi-runner" with max_builds=0. Maybe Gitlab added a multi-runner capability recently which would have broken the provisioner.  It should be possible to fix this by changing run to run-single here. Would it be possible for you to test this? If you don't have XCode I can create a test build for you.

@ast3150 Might know more about this.

@Marcocanc
Thanks for the hint, it sounds like it could be it!
I am a novice mac-developer, give me a few days and I will have tried out your suggestion.
Will post an update in a couple of days (or earlier) when I have tried it.

Attempt 1 @Marcocanc suggestion:
gitlab-runner run-single
Made the virtual machine restart.
No job was executed.
`Downloaded GitLab Runner Binary from Source successfully
Starting GitLab Runner...
Runtime platform                                    arch=arm64 os=darwin pid=774 revision=44feccdf version=17.0.0
FATAL: Missing URL
---------- Shutting Down ----------
Configuring GitLab Runner...
Successfully configured GitLab Runner
Downloading GitLab Runner Binary from Source
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
2 61.5M    2 1620k    0     0  1767k      0  0:00:35 --:--:--  0:00:35 1767k
100 61.5M  100 61.5M    0     0  10.9M      0  0:00:05  0:00:05 --:--:-- 12.7M
Downloaded GitLab Runner Binary from Source successfully
Starting GitLab Runner...
Runtime platform                                    `
Attempt 2
According to docs;
gitlab-runner run-single -c ~/.gitlab-runner/config.toml -r runner-name
should be used. MR
Tried with:
let runCommand = "gitlab-runner run-single -c ~/.gitlab-runner/config.toml"
Made;
gitlab-runner go crazy. With error message:
FATAL: flag provided but not defined: -c
before restart.
Currently trying to see that the file is existing within the vm. Then also, if something else needs to be modified to execute command with parameters.

The MR: https://gitlab.com/gitlab-org/gitlab-runner/-/merge_requests/4789/diffs was merged 17th of May.
Latest gitlab-runner released 16th of May, so the gitlab docs are incorrect as of now.
Next version of gitlab-runner hopefully have the -c option released.
Will continue trying it to work when next version of gitlab-runner released.

@fberndtsson if the issue is indeed in the latest version of GL Runner, you can manually specify the download URL in your config and set an older version.

The issue is that we want to use the parameters from a file, and that feature is not included in the gitlab-runner as of now.
I want to see if gitlab enable it in the next version of the gitlab-runner. As it should be according to the documentation.
If the feature is missing in next version of gitlab-runner, I will see if we can get it running with the commandline-parameters. Unsure if there is any security implication of having the token on the command-line.

@fberndtsson If I understand you correctly, this is an issue of the gitlab runner and not of Cilicon. I will therefore close the ticket. If I misunderstood, feel free to reopen it.

Sorry for being a bit slow @Marcocanc . Have been testing out the patch the last week to ensure that it worked properly.
The paramter -c don't exists in gitlab-runner mode run-single even in 17.2.
Workaround is to send it via the cmd to use the gitlab-runner run-single -u   -t  --executor  --maxbuild


Detect no connection without producing a warning?

Is there a way to detect if the connection to redis was established successfully, and if not, perform a different action? Right now, if it can't connect to the redis server, it produces a run-time warning.

See here. You can also check for the value of Predis\Client::isConnected() at later stages in your application but it won't tell you if the underlying socket has been closed due to a timeout or error, you get exceptions for that upon the first read/write operation on the same socket (that's how sockets work).

Not exactly the issue I'm having. When I try to do something like $redis->connect(), and the redis server is not running, this is the warning I get:
RUN-TIME WARNING: stream_socket_client() [function.stream-socket-client]: unable to connect to tcp://<IP_ADDRESS>:6379/ (A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
No amount of exception handling seems to bypass this warning.

Standard warning or notice messages for stream_socket_client() are already suppresse with @. I guess you have some 3rd party extension like xdebug enabled in your configuration since RUN-TIME WARNING and HTML links to PHP functions are not something emitted by PHP itself. In that case you should tweak its configuration, there's nothing that can be done from userland code to prevent that message.


Window transparency

Check for existing issues

[X] Completed

Describe the feature
I don't have access to the alpha so not sure if this is supported but judging from screenshots it does not seem to be the case.
It would be very nice to have the option to make the background of zed windows slightly transparent, similarly to my current go-to setup of running neovim in alacritty with 75% background opacity.
Besides the obvious aesthetic that blends in nicely with almost any wallpaper/theme, this also allows me to put an actively updating window behind the editor (either an unrelated application or something that is hot reloading the file I am editing). The background dims the window behind the editor enough so it does not distract from editing, but still allows me to monitor the changes in the other window live without having to sacrifice editing space or alt+tabbing around.
At least on Linux it is sometimes possible to request the compositor to adjust the opacity of specific windows, but that also makes text transparent. For readability it would be important that the application handles background transparency by itself, so text and important UI symbols can remain fully opaque. The main gotcha with this appears to be that different platforms have different requirements for transparent windows (e.g. macOS wants non-premultiplied alpha, Wayland wants premultiplied).
If applicable, add mockups / screenshots to help present your vision of the feature

To each their own but why would you want transparency on an editor?  Terminal I can understand to a point.
Pretty sure there will be an option at some point but nothing that I've seen as yet.
The docs are all here by the way...
https://zed.dev/docs/configuration/configuring-zed

Support for window transparency and blurring is something I personally expect in any modern IDE or Text Editor, unfortunately in VSCode's case we have to use an extension, which makes the whole process hacky as we have to inject custom attributes to the built CSS.
I'd like to propose being able to control every panels transparency and blur effect from the settings, giving the option for full window transparency (e.g all panels are transparent), and also to the users who likes the macOS standard, to only have the sidebars transparent with blur (similar to Xcode).


Add Device field

Hi,
Having a Device field for a number (with many to many relation) would help to attache internal and external numbers to devices.
Nowadays, when we add a SIP-device , we have to add  custom fields where we store the SIP-numbers and lines for this device.
And when we open a number record,we can't find the device it's attached to.
Thank you.

I need this too.


Add Workflow for running critest with Hyper-V Containers on Windows.

Signed-off-by: Nashwan Azhari<EMAIL_ADDRESS>

In its current stage, this is basically a duplicate of the Windows Integration workflow which instead pulls unmerged/unshipped versions of containerd/hcsshim to run the CRI test suite against containerd using Hyper-V Containers as described in #6862.

I'm wondering from the runs you sent me if we should skip the 4 or so tests that are exploding at the moment

Pingaroo. We still need to look into why the attach test case for CRI can hang for an hour on 2019, but should we skip the restart test that we know is flaky/doesn't work for hyp at the moment?

Works for me, I'd love to get this checked in to start getting some data

I also opened this to hopefully be able to get in the small hcsshim changes needed to get rid of the need to use hcsshim master here (and the cherry-picks of the other hyp work here) https://github.com/containerd/containerd/discussions/7138

@dcantah I'm currently doing a run with TestContainerdRestart skipped to ensure the rest of the RunCriIntegrationTests step finishes successfully.

I also opened this #7138

That'd be great to replace the hardcoded branch of HCSSHIM we currently have going, but it wouldn't eliminate the need to hardcode a branch which features the config updates from #6901.
Note that the workflow is currently using a dedicated branch on my repo which I've kept rebasing over containerd/main, but I'd much prefer we host the config changes from #6901 on a branch on the main containerd/containerd repo.
Once (if) we do get a branch/tag to host the config changes on the main repo, I'll have to adapt the workflow to pull said branch and rebase it over main so we can test the latest code in main every night while still passing on the Hyper-V config.

@aznashwan We merged https://github.com/containerd/containerd/pull/7284 so you're good to use the commit in here now!

@dcantah just rebased and re-ran the workflow, and although there's two critest tests failing on both 2019 and 2022 (one a timeout while stopping a container we discussed a while back and another apparent port forwarding issue)
@kzys could you please have a look too?


pypi.org shows wrong apache license

When i use CycloneDX to create the BOM file moto shows Apache as license but the expected license would be Apache License 2.0.
As comparison to the pypi moto site you can see boto3 has the Apache License 2.0 set.

Thanks for raising this @HannikMathe


TAJO-1921: Hbase Storage can cause NPE when the hbase cluster is restarted

It was similarly modified by referring to TableInputFormatBase#getSplits(JobContext). Cause as much as possible to follow a policy of HBase.
And then the size of memstore will not have a big impact for bigdata analysis. So it was removed.

@hyunsik I think TAJO-1921 and TAJO-1940 seems to be a conflict. If TAJO-1940 is deployed, I can also be modified again.
Thanks.

I'm reviewing the patch. Your suggestion sounds good to me.

The patch looks good to me. Here are some comments.
Our coding standard uses 2 space instead of tab. Some lines do not follow the indent of our coding standard.  Please refer to https://cwiki.apache.org/confluence/display/TAJO/Code+Style.
Other parts looks good to me. Here is my +1. Could you correct line indents? Then, I'll commit it soon.

I had to modify the source.
Thank you for comment.

@hyunsik Anything else to commit this PR?

+1 the patch looks good to me. I'm sorry for late.


adding support for Org in PayloadGenerator.ContactData

This PR fixes/implements the following bugs/features:

[X] Feature - Organization parameter for ContactData payload generator

What existing problem does the pull request solve?
No ability to generate a contact card with Organization.

Test plan
Ran unit tests with no issues.
I've successfully generated contact cards (VCard3) with the org parameter supplied.
Closing issues

Fixes #274

Thanks for your support @raganca


[Feature] Revert "[Feature] support cloud table schema change using different shard group (#33642)"

Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@mergify rebase

https://github.com/Mergifyio backport branch-3.2


Added Country names to Currency Calculator

Now it shows the countries details for which you are converting the currency , when you give amount and specify currency and convert it , it will display countries.
Fixes Issue🛠
Closes #36
Description👨💻

Type of change📄

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)

How this has been tested✅

Checklist✅

[*] My code follows the style guidelines of this project
[*] I have performed a self-review of my own code
[*] I have commented my code, particularly in hard-to-understand areas
[*] I have added demonstration in the form of GIF/video file
[*] I am an Open Source Contributor

Screenshots/GIF📷

Good, Merged!! Congrats on getting your PR merged to this repo under IWOC 2.0 🎉🎉 @hemant933

sir points are not assigned to me IWOC dashboard

It will take some time, get back to the discord moderator for the same @hemant933

okay sir


Warn when input parameter value is serialized for fingerprinting

Serialization is a stop-gap measure when we can't make sense of an input parameter to get a hash. We should drive users away from using arbitrary objects as inputs, and instead use types we can recognize and safely hash. As a first step we should warn whenever a value is used that has to be serialized.

I think we still should do this at some point.

I think we still want this.


fix(packages/webpack): improve process shim for browser-based builds

Fixes #7243
This PR fixes a gap in our process shim for browser clients. before, we did only a tiny bit. this PR uses the process/browser npm to fill in the rest. We had process/browser specified as a fallback entry in our webpack config.

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

/lgtm


camera: adding subscription APIs

This PR adds missing camera subscription APIs:

[x] camera mode
[x] video stream info
[x] capture info
[x] camera status
[x] current setting
[x] possible settings

Connects to #334.

Is it right to say that the camera mode is advertised by the camera

I think right now it is not advertised as such. It's just when you set it you also receive the feedback and use that. We potentially could add polling the mode as well in order to see changes made by e.g. QGroundControl.

We potentially could add polling the mode as well in order to see changes made by e.g. QGroundControl.

But if QGC sets the mode, won't the camera advertise it? Anyway it's fine for now.

But if QGC sets the mode, won't the camera advertise it? Anyway it's fine for now.

It could/should advertise it using: http://mavlink.org/messages/common#CAMERA_SETTINGS. We probably need to add that to E90.

@JonasVautherin this should be complete but I'm not happy with the naming yet.
Currently it's all options but we need to distinguish between a "selected option" (a setting) and a "selectable option" (an option).

Currently we have:
typedef std::function<void(const std::vector<std::pair<std::string, std::string>>)>
    subscribe_current_options_callback_t;

typedef std::function<void(const std::vector<std::pair<std::string, std::vector<std::string>>>)>
    subscribe_possible_options_callback_t;

We could refactor that to:
struct Option {
    std::string id;
};

struct Setting {
    id;
    Option option;
};

struct SettingList {
    id;
    std::vector<Option> options;
};

typedef std::function<void(const std::vector<Setting>)>
    subscribe_current_settings_callback_t;

typedef std::function<void(const std::vector<SettingList>)>
    subscribe_possible_settings_callback_t;

@JonasVautherin what do you think?

I like your proposition. I would suggest SettingOptions instead of SettingList, though :-).

Ok, I'll refactor it.


[SPARK-7323] [Spark Core] Use insertAll instead of insert while merging combiners in reducer

Merged build triggered.

Merged build started.

Test build #31659 has started for   PR 5862 at commit 61cf43a.

Test build #31659 has finished for   PR 5862 at commit 61cf43a.

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

Merged build finished. Test PASSed.

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

LGTM

Hi @srowen can you commit the patch please ? I seem to have messed up my local config. Thanks.


Slow the stand up

Hi the team.
Is it possible to show down the stand up process?
Thanks the team.

Yep, that parameter is a member of the RobotDriver class (declared here) and can be overriden however you prefer. We should probably make it a ROS parameter at some point...

👌


CFS create data partition for volume when create volume?

In create volume API, the CFS create data partition for this volume when init volume??
In create data partition API， Does it mean expanding the volume?

#138  has fixed


security_monkey should track when access keys were last used

https://aws.amazon.com/blogs/aws/useful-new-iam-feature-access-key-last-use-info/
An audit issue should be added for access keys that have not been used in a while.  (7 days? 30 days? 90 days?)

Looks possible with boto3:
http://boto3.readthedocs.io/en/latest/reference/services/iam.html#IAM.Client.get_access_key_last_used
Not sure about regular boto

Requires new permission for the securitymonkey role:
iam:GetAccessKeyLastUsed
Specifically requires a boto3 client iam session, so sts_connection.py needs to support that.

Fixed by #338


Consider creating a master test case object

This is mainly brought about trying to make my failure string avoid formatting but that's not easy.
In addition I'm setting the exit code in a number of different places everywhere as well as doing all of the calculations for display issues so it might be best to store this in the master test case.
This also means that we could defer testcase calculation till we call it on this object.

I'm not happy about its current state but anything more is just unnecessary work. If it gets more complex, we'll revisit htis.

Reopening unresolved issue.


Tracing Cabling Path with Circuits

Hello,
I want to attach a circuit ID between two Rear port of a Patch Panel but the trace stop to the Circuit ID. I don' see the other side (The Both sides are attached):

Below the circuit corresponding :

Is it normal ?
Regards,

Thank you for your interest in NetBox. GitHub issues are intended for reporting reproducible bugs and requesting features, and must be submitted using one of the templates provided here. For general discussion, questions, or assistance with installation issues, please post to our mailing list instead.


Eliminate Fallthrough typeclass

Introduces a MaybeResponse, which has a monoid similar to the "first
option" monoid.  This lets us compose services according to the standard
Kleisli monoid instead of our ad hoc concept.
Furthermore, it forces us to carefully consider the effect of
fallthroughs in middleware, which has already bitten us once (#803).

There are a lot of changes in here so perhaps I've missed something, but in general, I like this.

The cats branch's server project is failing on a fallthrough test case.  I'm hopeful it gets cleaned up by this.

Build failure is #817.


Resolver not picking up Cognito user groups

Describe the bug
Problem: Trying to use dynamic groups with @auth, the 'list' mutation returns no data.
To Reproduce
Using the default settings for Cognito User Pools as set up by amplify. Using the following model:
type TestAuth
  @auth(rules: [
    {allow: groups, groupsField: "participants", mutations: [update, delete], queries: [get, list]} ,
    {allow: groups, groupsField: "watchers", mutations: [], queries: [get, list]}
  ])
  @model
  {
    id: ID!
    content: String!
    participants: String
    watchers: String
  }

Amplify has generated the resolver for 'ListTestAuths' as follows:
## START: Filter Authorized Items. **
#set( $items = [] )
#foreach( $item in $ctx.result.items )
    #set( $isAuthorizedLocal = false )
      #set( $allowedGroups = $item.participants )
      #foreach( $userGroup in $userGroups )
    #if( $util.isList($allowedGroups) )
      #if( $allowedGroups.contains($userGroup) )
        #set( $isAuthorizedLocal = true )
      #end
    #end
    #if( $util.isString($allowedGroups) )
      #if( $allowedGroups == $userGroup )
        #set( $isAuthorizedLocal = true )
      #end
    #end
  #end
    #set( $isAuthorizedLocal = false )
      #set( $allowedGroups = $item.watchers )
      #foreach( $userGroup in $userGroups )
    #if( $util.isList($allowedGroups) )
      #if( $allowedGroups.contains($userGroup) )
        #set( $isAuthorizedLocal = true )
      #end
    #end
    #if( $util.isString($allowedGroups) )
      #if( $allowedGroups == $userGroup )
        #set( $isAuthorizedLocal = true )
      #end
    #end
  #end
  #if( $isAuthorized == true or $isAuthorizedLocal == true || $isAuthorized == true or $isAuthorizedLocal == true )
$util.qr($items.add($item))
  #end
#end
#set( $ctx.result.items = $items )
## END: Filter Authorized Items. **

$util.toJson($ctx.result)

Dynamo contains a single record as follows:

"id (S)"
"__typename (S)"
"content (S)"
"createdAt (S)"
"participants (S)"
"updatedAt (S)"
"watchers (S)"

"af35f2fa-ba51-4a35-adc4-c70b92101e5b"
"TestAuth"
"foobie"
"2018-09-20T15:25:47.268Z"
"testauth-p-1"
"2018-09-20T15:25:47.268Z"
"testauth-w-1"

The logged-in user is a member of both groups as confirmed in the Cognito UI:

Groups | testauth-p-1 , testauth-w-1

Code to retrieve data in App.js as follows:
const listTestAuths = `query listTestAuth {
  listTestAuths{
    items {
      id
      content
      participants
      watchers
    }
  }
}`
...
  listTestAuths = async () => {
    console.log('listing test auths');
    const allTestAuths = await API.graphql(graphqlOperation(listTestAuths));
    alert(JSON.stringify(allTestAuths));
  }

Steps to reproduce the behavior:

Make a call to listTestAuths
Data returned is

{"data":{"listTestAuths":{"items":[]}}}

Expected behavior
The single TestAuth object should be returned.  I have not yet figured out how to debug resolvers to inspect the contents of $userGroups nor am I sure how/where it is set, but the results are consistent with it being empty.
Desktop (please complete the following information):

OS: OSX
Browser Firefox
Version 62
Amplify 0.1.19

Additional context
The GraphQL Transform Tutorial states there is a

...currently a bug with the user pool creation. To make this work you need to have your own user pool and then pass the id via a CloudFormation parameter. The Amplify CLI user pool with the default setting doesn't work (a fix is in the works).

I couldn't locate a github issue for this bug. Could the problem be due to a Cognito config setting I can change?

I will try to reproduce this and report back with what I find.


Internet Explorer support for specifying 32 vs 64 bit version driver in config

@erdi
I am following up on our conversation thread from a different issue : https://github.com/energizedwork/webdriver-binaries-gradle-plugin/issues/7
You have mentioned I open a new issue for requesting support for 32 bit vs 64 bit driver. There are well known issues with IE 64 bit version driver, and the workaround is to use the 32 bit version. In general, this maybe an option useful for other drivers too.
Would be great if we can have such support for 32 vs 64 bit requested version for this plug-in.
Thanks again!

Specifying the architecture to be used for IEDriverServer is available in 1.4 version of the plugin which has just been released.

Thanks @erdi Will check it out asap and let you know the results

@erdi The change looks good as per my preliminary testing. I am able to use the x86 (32-bit) IE driver on a 64-bit system as per the documentation you supplied.
I am guessing if I don't specify the architecture for the driver, it defaults to how it was before this change.

Yes, as described in the docs, IE driver architecture defaults to the
architecture of the operating system the build is running on.
On Tue, 6 Feb 2018 at 19:24, masterautomator<EMAIL_ADDRESS>wrote:

@erdi https://github.com/erdi The change looks good as per my
preliminary testing. I am able to use the x86 (32-bit) IE driver on a
64-bit system as per the documentation you supplied.
I am guessing if I don't specify the architecture for the driver, it
defaults to how it was before this change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/energizedwork/webdriver-binaries-gradle-plugin/issues/8#issuecomment-363536022,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMiizv1Zj2DUpIunCA4N-c2mDyf3SFmks5tSKcDgaJpZM4Re17C
.


Incorrect blank lines inserted into source code

example: https://timothycrosley.github.io/portray/reference/portray/api/
the original source code look like this:
import os
import webbrowser
from typing import Dict, Union

import mkdocs.commands.gh_deploy
from livereload import Server

from portray import config, logo, render

but Portray renders it like this:
import os

import webbrowser

from typing import Dict, Union

import mkdocs.commands.gh_deploy

from livereload import Server

from portray import config, logo, render

Looks like this is the root issue.  Closing the portray issue for now.
https://github.com/timothycrosley/pdocs/issues/21


feat(css/prefixer): more env properties and values

Description:
Now all properties and values generated based on env
BREAKING CHANGE:
No
Related issue (if exists):
No

@kdy1 Tests updated


feat(#824): Add JNA Integration Test

In this PR I successfully added an integration test that uses JNA library. This PR proves that
jeo-maven-plugin works well with JNA classes.
Closes: #824.

PR-Codex overview
This PR focuses on enhancing the JNA integration test by improving the bytecode representation and instruction handling, renaming methods for clarity, and adding documentation. It also includes new test cases for better coverage and updates to the pom.xml for project configuration.
Detailed summary

Added view() method to several bytecode classes for better human-readable output.
Renamed isGoto() to isJump() in multiple classes to improve clarity.
Introduced a new method instructionsView() in BytecodeMethod for displaying instructions.
Enhanced BytecodeMethodTest with parameterized tests for real bytecode methods.
Updated invoker.properties and pom.xml for improved integration testing configuration.
Added a README.md for JNA usage with instructions for running tests.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 Could you take a look please? This test tries to resemble the issue you have here https://github.com/objectionary/hone-maven-plugin/issues/58
As you can see, everything works just fine. Please, correct me, if the test I wrote is different from what you have in OptimizeMojoTest#optimizesJnaClasses, in this case I will try to reproduce the same scenario.
As for the https://github.com/objectionary/hone-maven-plugin/issues/58 issue, it seems, that when you start testing mvn test the JNA library is just missing in the classpath.

@volodya-lombrozo this way it works, but try to replace the <dependency/> in the pom.xml with this:
      <plugin>
        <artifactId>maven-dependency-plugin</artifactId>
        <version>3.8.1</version>
        <executions>
          <execution>
            <phase>generate-resources</phase>
            <goals>
              <goal>unpack</goal>
            </goals>
            <configuration>
              <artifactItems>
                <artifactItem>
                  <groupId>net.java.dev.jna</groupId>
                  <artifactId>jna</artifactId>
                  <version>5.15.0</version>
                  <type>jar</type>
                </artifactItem>
              </artifactItems>
              <outputDirectory>${project.build.outputDirectory}</outputDirectory>
            </configuration>
          </execution>
        </executions>
      </plugin>

Basically, you will unpack the dependency and let JEO process the .class files from JNA.

@volodya-lombrozo use the latest version of EO: 0.40.3, it should be much faster

@yegor256 It is with the 0.40.3 version already.

@yegor256 Could you take a look one more time, please? I fixed the original problem in this PR: https://github.com/objectionary/jeo-maven-plugin/pull/827
Here I just prove that the solution solves the original problem with JNA classes.
Pay attention, I excluded phi/unphi and unroll from the jna integration test because it takes too much time. Just to compare, the 'short' path (without phi/unphi) takes 13 seconds to run, while the full path takes 31 minutes. You can run the full path locally, to be sure, that everything works well. I've already done it myself:
[INFO] Building: jna/pom.xml
[INFO] run post-build script verify.groovy
[INFO]           jna/pom.xml ...................................... SUCCESS (1860 s)
[INFO] -------------------------------------------------
[INFO] Build Summary:
[INFO]   Passed: 1, Failed: 0, Errors: 0, Skipped: 0
[INFO] -------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  31:13 min
[INFO] Finished at: 2024-10-31T12:16:46+03:00
[INFO] ------------------------------------------------------------------------

@yegor256  Indeed! You are right, thank you. Removed this library. Could you have a look one more time, please?
Also I should warn you. If you remove target/classes folder somewhere, it might be a cause of serious issues. For example, jna library keeps native .so files inside target/classes and if you remove this folder entirely - it will crash your application.  By this reason, in my test I remove only *.class files from target/classes folder:
<plugin>
  <artifactId>maven-clean-plugin</artifactId>
  <version>3.4.0</version>
  <executions>
    <execution>
      <id>clean-target-classes</id>
      <phase>process-classes</phase>
      <goals>
        <goal>clean</goal>
      </goals>
      <configuration>
        <excludeDefaultDirectories>true</excludeDefaultDirectories>
        <filesets>
          <fileset>
            <directory>target/classes</directory>
            <includes>
              <include>**/*.class</include>
            </includes>
          </fileset>
        </filesets>
      </configuration>
    </execution>
  </executions>
</plugin>

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here.

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 11min).


Update normalization.py - fixed tf.layer.batch_normalization deprecate note

tf.layer.batch_normalization referenced incorrect keras.layer.BatchNormalization function.

I signed it!

Thank you for catching this


Retrying with context for when and notify

Another approach to #39, see #54

Currently the problem is that when requires fn(&E) -> bool while the field requires fn(&E, &C) -> bool which cannot be casted to.

Currently the problem is that when requires fn(&E) -> bool while the field requires fn(&E, &C) -> bool which cannot be casted to.

Can we set the when to |err, _| f(err)?

Can we set the when to |err, _| f(err)?

Unfortunately, no, since it's a fn: cannot capture anything even using move

I have some new ideas, but it's shortcome is that we need to implment call for differnt compose of C, RF and NF. Do you have better solutions?

I have some new ideas, but it's shortcome is that we need to implment call for differnt compose of C, RF and NF. Do you have better solutions?

Ok, I fixed it.

Wow, type gymnastics saved the world again 😂

Replaced by https://github.com/Xuanwo/backon/pull/54


it seems the codes are not working yet

thanks for your sharing, but when I feed the trainer with a bunch of  images from FFHQ(~1K), the loss stayed unchanged through each training iteration.

Thanks for the sharing~
I also met the problem. I trained 30k person images and generated 2 black images. I found the loss is NAN.

Thanks for the sharing~
I also met the problem. I trained 30k person images and generated 2 black images. I found the loss is NAN.

you might get 0 std for the nan problem

Thanks for your attention. This project is still under working, it’s a unfinished version. If you could help me fullfill this, it would be great!

You can focus on Generator part.

Thanks for your attention. This project is still under working, it’s a unfinished version. If you could help me fullfill this, it would be great!

Is the tanh() necessary here?
https://github.com/tomguluson92/StyleGAN2_PyTorch/blob/b953a8e4c3747989656e7c06be4fc650e4346e80/network/stylegan2.py#L1038


Allow user to set the default value for users avatar

Don't mind StyleCI, it will automatically be fixed on merge.


No desktop notification

The discord client for linux does not seem to notify on a new message (only a sound so far).
Distro: Archlinux
WM: xmonad

Are you aware there's an official one now? It's in your best interest to use that one.

Do you have libnotify?

Do you have libnotify?

Yes. Other programs can use it fine (it highlights the relevant virtual desktop), but the client doesn't.


error: No overload variant of "isna" matches argument type "NAType"  [call-overload]

import pandas as pd
print(pd.isna(pd.NA))

Have to add NaTType to isna() and notna() overloads that specify Scalar in core/dtypes/missing.pyi

Closed via #279


feat: add support for ReuseAddr

Add support for the SO_REUSEADDR socket option.

thank you

tests time out...

Sorry about that.  I'll take a look.


Update sklearn_xarray.{data => datasets} in documentation examples

Make sure you have:

[x] added the necessary updates to doc/content/whatsnew.rst
(N/A)
[x] added documentation for new features
(N/A)
[x] updated sklearn_xarray/version.py if you are preparing a new release
(N/A)

Wasn't sure whether to target develop or master (I chose develop), but the changes should apply to either.

Thanks, forgot about fixing that when I renamed the module.


It's an error ?

Is this an error? The bot is not functioning and only displays this message.

this is telethon problem

this is telethon problem

How to fix that?


Unambiguous hashes

Updates the hashing function for chunk based hashing to pad the final hash of (length, root) with a full chunk of zeros to avoid preimage attacks. More details can be found in the documentation of the newly added function and test.
Please review this carefully and double check that no other calls of hash_pair need to be updated/changed, as these can also be found in the node for reasons unrelated to Merkle tree hashing.
As this is security relevant crypto code, I would strongly suggest at least 4 positive reviews be made before merging this. Please also doublecheck that block hashing is unaffected.
No changelog update has been made, as this affects mostly the unreleased hashing crate.
Closes #2603

bors r+

bors try

bors r+


create_s256_code_challenge shouldn't accept invalid code verifiers

Describe the bug
RFC7636 describes the code challenge using the following ABNF:
code-challenge = 43*128unreserved
unreserved = ALPHA / DIGIT / "-" / "." / "_" / "~"
ALPHA = %x41-5A / %x61-7A
DIGIT = %x30-39

This translates to the following regex:
[a-zA-Z0-9\-\.\_\~]{43,128}

authlib currently doesn't check if code challenges match the regex (but it should), and even docs and tests use invalid challenges.
IMHO, create_s256_code_challenge should verify the code_verifier and raise an error if it doesn't match the regex - I don't assume it's a considerable performance penalty.
We might also want to provide a create_code_verifier convenience method, and use this consistently throughout docs and tests.

Thanks for your report. We need to check code_verifier for none method too, so I put the validation in validate_code_verifier.

Thanks!


[BugFix] Fix rewrite nested mv bugs when mv's partition/distribution predicates are not null

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Problem Summary(Required) ：

Fix rewrite nested mv bugs when mv's partition/distribution predicates is not null

Ignore mvOriginalPredicates when rewriteMVCompensationExpression rewrites MVScan's predicates to null, which cannot prune partitions/distributions from MV's define.

        // Rewrite original mv's predicates into query if needed.
        final ColumnRewriter columnRewriter = new ColumnRewriter(rewriteContext);
        final Map<ColumnRefOperator, ScalarOperator> mvColumnRefToScalarOp = rewriteContext.getMVColumnRefToScalarOp();
        ScalarOperator mvOriginalPredicates = mvScanOperator.getPredicate();
        if (mvOriginalPredicates != null && !ConstantOperator.TRUE.equals(mvOriginalPredicates)) {
            mvOriginalPredicates = rewriteMVCompensationExpression(rewriteContext, columnRewriter,
                    mvColumnRefToScalarOp, mvOriginalPredicates, false);

            if (mvOriginalPredicates == null) {
                return null;
            }
            if (!ConstantOperator.TRUE.equals(mvOriginalPredicates)) {
                mvScanBuilder.setPredicate(mvOriginalPredicates);
            }
        }

use MvUtils.canonizePredicate(prunedOlapScanOperator.getPredicate() to canonize predicates after  partition/distribution pruning.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[x] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MaterializedViewRewriter.java
1
2
50.00%
[588]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MVPartitionPruner.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MvUtils.java
2
2
100.00%
[]

run starrocks_admit_test

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


🚀 Feature: typeParts API, combining intersectionTypeParts and unionTypeParts?

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
https://github.com/typescript-eslint/typescript-eslint/pull/7466/files#diff-3cc43b506517aba406ae3a31d7998b116c4b09db76b632c6cb3c2e54fc71d66cR33 shows code that wants the type parts of a type that can be either an intersection or union. Maybe it'd be nice to expose a simplified function to do this? Maybe with a recursive?: boolean parameter/option?
My instinct is no, since folks can do this pretty on their own in a straightforward manner. But just thinking out loud.
Additional Info
No response

I came across a second place in typescript-eslint that would benefit from a basic version of this function. Seems like a good enough reason for me.


Interface doesn't get correct implementation

There seems to be a bug when using delegateToSchema in relation to interfaces. Here is an example to reproduce the behaviour: https://www.graphqlbin.com/mjoCp
Code: https://github.com/graphcool/graphql-binding-github

This looks like an issue with graphql-import. I can use interfaces fine by using Github.schema directly.

I've further investigated this and can confirm that it's not a problem with graphql-import but a problem with delegateToSchema.
I've done the following to validate this: Prepared typeDefs via introspection query + importSchemas, then I've used this schema definition to create a remote schema which I've then used for both 2 test cases:

mergeSchemas works
delegateToSchema doesn't work


[WIP] feat: enable data caching cross jobs

Signed-off-by: Zhijun Chen<EMAIL_ADDRESS>
Ⅰ. Describe what this PR does
This PR decouples cacheBackend from jobs and makes cacheBackend can be reused in different jobs.
II. Does this pull request fix one issue?

#252
III. Special notes for reviewers if any.
In progress, update doc & unit testing later.

[x] code
[ ] unit testing
[ ] document

Codecov Report

Merging #263 (95307d6) into master (9e5a464) will increase coverage by 0.14%.
The diff coverage is 31.91%.

@@            Coverage Diff             @@
##           master     #263      +/-   ##
==========================================
+ Coverage   28.97%   29.12%   +0.14%
==========================================
  Files          88       88
  Lines        5977     5982       +5
==========================================
+ Hits         1732     1742      +10
+ Misses       3992     3989       -3
+ Partials      253      251       -2

Flag
Coverage Δ

unittests
29.12% <31.91%> (+0.14%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/job_controller/job.go
24.46% <0.00%> (-0.23%)
:arrow_down:

pkg/job_controller/job_controller.go
11.47% <0.00%> (+0.36%)
:arrow_up:

controllers/cache/cachebackend_controller.go
49.27% <68.18%> (+11.18%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.

@zjchenn hi zjchenn, thanks for your contribution! you may design & implement unit testing in next stage of work.

/cc @SimonCqk

/LGTM


Fix #485 where empty email throws "User is already registered"

Coverage remained the same at 96.044% when pulling 38b9c656808b08b2502daa8d099bdb49476c5fc5 on olivierdalang:fix_485 into 624ad01afbc86fa15b4e652406f3bdcd01f36e00 on Tivix:master.

Coverage remained the same at 96.044% when pulling 38b9c656808b08b2502daa8d099bdb49476c5fc5 on olivierdalang:fix_485 into 624ad01afbc86fa15b4e652406f3bdcd01f36e00 on Tivix:master.

Coverage remained the same at 96.044% when pulling 38b9c656808b08b2502daa8d099bdb49476c5fc5 on olivierdalang:fix_485 into 624ad01afbc86fa15b4e652406f3bdcd01f36e00 on Tivix:master.

Hi,
as a user of rest-auth, thanks for the contribution!
This repo is not maintained anymore, so the development moved to dj-rest-auth. (reference: https://github.com/Tivix/django-rest-auth/issues/568)
It may be best, if you move this PR there. (and upgrade to using dj_rest_auth)
new repo link: https://github.com/jazzband/dj-rest-auth (I'm not the upkeeper of that repo, it just makes sense for me to help you merge your PR)
Many Thanks,
Barney


Deploy software on Digital Ocean

Do all work for this task in a branch called issue-5
Deploy using Digital Ocean with E59 & E60:
https://courses.ics.hawaii.edu/ics314f23/morea/deployment/experience-app-deployment-digital-ocean.html

No files will be changed on GitHub for security reasons.

deployed app at http://<IP_ADDRESS>. Still need to add Robo3T and Monti APM.

Posted photos to discord.


Fix issue #290: Factor out all prompts from python files into jinja templates

This pull request fixes #290.
The issue has been successfully resolved as it addressed all key requirements:
Automatic fix generated by OpenHands 🙌

New OpenHands update

New OpenHands update


Regenerate sigfiles 1.5

There are no parser error and exception in NB 17.
Fix two issues: 5781a88f002411d60dbd3c4f6ba7fb985708669c and 789e8179948cdcc02c09ab2af3ef4d10437c3f20
Add the licenses to README: c4991ed8afb0bbbfb308fa686dd8d3e467e67233

@junichi11
Just 2 things:

the source branch can be deleted but leaving up to you
any idea how to make PRs easier to review? I mean, how to "ignore" changes in the output files? 🤔

Thanks!

@tmysik

Junichi, you really don't need my review and approval

OK :)

the source branch

What does this mean? resources/php-chunked-xhtml?

any idea how to make PRs easier to review? I mean, how to "ignore" changes in the output files?

Let's create PRs without output files. I'll push output files directly after I generate them on my environment and verify whether it is OK. (Maybe, It would be better to create them in the same environment.)
Thanks!

The source branch of this PR :) I did not delete it...

Ah, I see. Thank you!


Apply a GDI region to the top level Island window to allow dragging with a single Xaml Island

Summary of the Pull Request
When Terminal is drawing in the non client area we no longer create 2 islands. Instead we create a single island and create a region that "pokes" the drag-able title bar area in the top right corner of the main window of terminal.

This change also fixes:

A crash at shutdown where the Application object is released early.
DestroyWindow is not getting called at shutdown for the all HWND (This have the effect of showing the window for a brief period after closing the app)
WindowsDesktopXamlSource.Close was not getting called at shutdown.

References

PR Checklist

[X] Closes 872, 1085
[X] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[X] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

/azp run

/azp run

/azp run

I need to involve Azure DevOps Support here. The branch builds if I trigger it manually, but that doesn't propagate status back up to GitHub. It fails when triggered from GitHub.

General question with the regards to the windows api calls added in this PR.  Shouldn't all of these be wrapped in a wil result macro?  I was under the assumption that they should, but I am not a 100% what the guideline is.

General question with the regards to the windows api calls added in this PR. Shouldn't all of these be wrapped in a wil result macro? I was under the assumption that they should, but I am not a 100% what the guideline is.

I have added a dependency to WIL, and I am uisng wil::unique pointers to ensure correct memory handling.

We have a working version of Xaml application for ARM64 (here)[https://github.com/windows-toolkit/Microsoft.Toolkit.Win32/pull/134]

ARM64 build is still not working.

Hey, I spent a while working on a true reimplementation of the min/max/close buttons in XAML :smile: it was hard because xaml is an insane language!
hovered

clicked

maximize = restore

I'll probably put this up for PR after your change lands for 0.3

Hardly insane @DHowett-MSFT It's all easily readable, nicely angled brackets, etc.
C++ is the true gobbledegook, especially compared to C# 😋

Hey, I spent a while working on a true reimplementation of the min/max/close buttons in XAML 😄 it was hard because xaml is an insane language!
hovered

clicked

maximize = restore

I'll probably put this up for PR after your change lands for 0.3

But it looks nice, I miss the PR link.

kinda odd how the borders at the top are rounded. Very few apps have rounded borders on Windows 10. One is Math Input Panel.

They already said this will change in the future, Windows is getting rounded borders everywhere

Introducing rounded corners after years of persistent sharp corners is hard and we don't have rounded corners right now anyway. I got a little off topic as it's more of a topic for the winui repo

The real technical reason this is happening is that we're suppressing window decorations and drawing over them, and the kernel component in charge of that (don't ask) gives us a region with rounded corners.
If we let the kernel do the painting, it looks like Windows Vista's Aero Basic.
Hence, the rounded corners.


Bump DBSync to <IP_ADDRESS>

Bump DBSync to the latest released version <IP_ADDRESS>
https://github.com/IntersectMBO/cardano-db-sync/releases/tag/<IP_ADDRESS>
There is no need to do a resync.
Assess and consider Postgres setup suggestions

@MSzalowski Latest one is <IP_ADDRESS>

Bump on sancho is already in review

Done on sancho

updated on preview and mainnet


Automation to allow rancher2 provider TF cluster resources to be tested with Terratest + Go

refactored away from static components folder + variables, and replaced w/ hclwrite for setModule functions
refactored away from static .tfvars files to be dynamically created/destroyed with each test;
refactored away from config.go to supply configurations via yml

Issue:
Automate tests for terraform rancher2 provider cluster resources for terraform releases
Problem
No automation exists for this
Solution
Created a framework w/ Go to allow testing automation of rancher2 provider cluster resources
Testing
The following tests have been thoroughly tested during development and supported with this PR:
Provisioning:

AKS
EKS
GKE
RKE1
RKE2
K3s

Scaling:

AKS
EKS
RKE1
RKE2
K3s

Kubernetes Upgrade:

RKE1
RKE2
K3s

Cleanup:

All modules supported

Engineering Testing
Manual Testing
N/A
Automated Testing
Ran each test, ensuring expected behavior and accurate results
QA Testing Considerations
N/A
Regressions Considerations
None

In addition to reviewers requested changes, I will also update EKS config to include node_role field for node_groups block.  With recent changes, this field is now required.

All requested revisions and refactors in new PR (draft) here:
https://github.com/rancher/rancher/pull/41099


feat: Add DeletedTaxonomiesQuery

The previously visible error in actions led to a more complicated fix, so I removed the commit from here, solved the error and explained it in #350

As discussed in our meeting, we should prefer something like DeletedUuidsQuery, what demands a more abstract and complicated code.


Cleanup Hilt ViewModel extensions

Share code between getViewModel extension functions
Add tag and viewModelStoreOwner as parameters to allow sharing ViewModels between Screens

Sharing ViewModels between screen example:
class AScreen : AndroidScreen() {

    @Composable
    override fun Content() {
        val navigator = LocalNavigator.currentOrThrow
        val viewModelStoreOwner = LocalContext.current as ViewModelStoreOwner
        val viewModel = getViewModel<AViewModel, AViewModel.Factory>(viewModelStoreOwner = viewModelStoreOwner) { factory ->
            factory.create("World")
        }
        Column {
            Button(onClick = { navigator.push(OtherScreen(viewModelStoreOwner)) }) {
                Text(text = "Other")
            }
        }
        // More Code
    }
}

class OtherScreen : AndroidScreen() {

    @Composable
    override fun Content() {
        val viewModelStoreOwner = LocalContext.current as ViewModelStoreOwner
        val viewModel = getViewModel<AViewModel>(viewModelStoreOwner = viewModelStoreOwner)
        // More Code
    }

}

Have you seen #217 ?
AndroidX provides a LocalViewModelStoreOwner.current that help to avoid casting your local context.

Have you seen https://github.com/adrielcafe/voyager/pull/217 ?

I haven't but it looks like a cleaner version.


mockClear vs mockReset vs mockRestore

Do you want to request a feature or report a bug?
I would like to ask a question about a feature and then submit a PR to the docs based on the answer.
What is the current behavior?
The docs for mockReset say this:

Resets all information stored in the mock, including any initial implementation and mock name given.
This is useful when you want to completely restore a mock back to its initial state.

What is the expected behavior?
I'm not sure what mockRestore does exactly in the context of restoring. Do I need it for restoring a mock's initial implementation? For example:
it('my test', () => {
  const prop = jest.spyOn(obj, 'prop').mockImplementation(() => { /* ... */ });
  // do something with obj.prop
  prop.mockRestore();
});

Is the above enough? Do I need to also call mockReset or maybe even mockClear beforehand? Why do the docs say that mockReset is useful for restoring mock to its initial state?
My main question: what is mockReset useful for?

In your snippet mockRestore restores obj.prop back to its initial state. Instead of
const origProp = obj.prop;
obj.prop = jest.fn().mockImplementation(() => { /* ... */ });
// do something with obj.prop
obj.prop = origProp;

You do not have to call reset or clear.
Did that help? PR very much welcome for clarification to the docs 🙂

Thanks, that did help. 👍 I'm just wondering when is mockReset useful, i.e. when you would reset a mock instead of simply creating a new one with jest.fn(). I'll try to think of some examples and submit a PR to the docs.

For instance:
const mockAProp = jest.fn();

jest.mock('some-dep', () => ({ foobar: mockAProp }));

afterEach(() => {
    mockAProp.mockReset();
});

test('test 1', () => {
    mockAProp.mockImplementation(() => true);
});

test('test 2', () => {
    mockAProp.mockImplementation(() => true);
});

Thanks!

Please do update the docs. The are very brief in descriptions of the methods as if the implementation details for reset, restore and clear were self-explanatory

Any updates on this?

The updated docs doesn't help? #6227

Currently I'm just adding the following any time I use mocks since I believe this should cover all cases whether I use spies or mocks.
afterEach(() => {
    jest.resetAllMocks();
    jest.restoreAllMocks();
});

FTR, here are the answers to the popular questions raised by @geoffreyyip.

"Does mockReset() reset implementation for spies?"

No. It resets the implementation to a new undefined-returning mock function.

"Does mockRestore() do anything for non-spy mocks?"

Yes. It resets the implementation to a new undefined-returning mock function. For spies, it "restores" the original implementation.

"What does it mean to "take care of restoration yourself when manually assigning jest.fn()"?"

Here's an example:
// contrary to jest.spyOn(), jest.fn() has "replaces" the original implementation
// without keeping a copy for later restoration.
test('...', () => {
  const origFn = module.api
  module.api = jest.fn()

  // act, assert, etc.

  // manual restoration
  module.api = origFn
})

Here's a passing test suite that clarifies how mockReset() and mockRestore() behavior differs.
describe('jest mock reset/restore api', () => {

  describe('when calling mockReset()', () => {
    test('on a spy with custom implementation, it replaces the implementation to a nenw undefined-returning mock fn', () => {
      const module = { api: () => 'actual' }
      jest.spyOn(module, 'api').mockImplementation(() => 'spy mocked')

      expect(module.api()).toStrictEqual('spy mocked')
      expect(module.api).toHaveBeenCalledTimes(1)

      module.api.mockReset()

      expect(module.api()).toStrictEqual(undefined)
      expect(module.api).toHaveBeenCalledTimes(1)
    })

    test('on a non-spy with custom implementation, it replaces the implementation to a new undefined-returning mock fn', () => {
      const api = jest.fn(() => 'non-spy mocked')

      expect(api()).toStrictEqual('non-spy mocked')
      expect(api).toHaveBeenCalledTimes(1)

      api.mockReset()

      expect(api()).toStrictEqual(undefined)
      expect(api).toHaveBeenCalledTimes(1)
    })
  })

  describe('when calling mockRestore()', () => {
    test('on a non-spy mock with custom implementation, it resets the implementation to a new undefined-returning mock fn', () => {
      const api = jest.fn(() => 'non-spy mocked')

      expect(api()).toStrictEqual('non-spy mocked')
      expect(api).toHaveBeenCalledTimes(1)

      api.mockRestore()

      expect(api()).toStrictEqual(undefined)
      expect(api).toHaveBeenCalledTimes(1)
    })

    test('on a spy with custom implementation, it resets the implementation to a new undefined-returning mock fn', () => {
      const module = { api: () => 'actual' }
      jest.spyOn(module, 'api').mockImplementation(() => 'spy mocked')

      expect(module.api()).toStrictEqual('spy mocked')
      expect(module.api).toHaveBeenCalledTimes(1)

      module.api.mockRestore()

      expect(module.api()).toStrictEqual('actual')
      expect(module.api).not.toHaveProperty('mock')
    })
  })

})

You may run it online:
https://repl.it/@sepehr/jest-mock-api-reset-restore

@sepehr, your example is really helpful but I think you forgot to change the text of the final test; it's currently a copy of the first test's text and doesn't describe the actual expected results.

@sepehr, your example is really helpful but I think you forgot to change the text of the final test; it's currently a copy of the first test's text and doesn't describe the actual expected results.

@tremby, thanks for the feedback and the catch. I've already fixed this in the linked repl, but forgot that there's a hardcoded snippet as well 😅

@tremby, thanks for the feedback and the catch. I've already fixed this in the linked repl, but forgot that there's a hardcoded snippet as well 😅


delete button

@miram13 can you check the new code analysis results? See the details there

Ya, sure
On Mon, Sep 26, 2022 at 3:04 PM María Arias de Reyna <
@.***> wrote:

@miram13 https://github.com/miram13 can you check the new code analysis
results? See the details there
[image: imagen]
https://user-images.githubusercontent.com/726590/192243555-76947d5b-72b6-4bd6-b32f-cc23d76b5ec5.png
—
Reply to this email directly, view it on GitHub
https://github.com/KaotoIO/kaoto-ui/pull/725#issuecomment-1257761514,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW2BEKQQ4D3FDS6UOJ43V2LWAFU2TANCNFSM6AAAAAAQSZFDVA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add URL metadata to mp4 container (closes #17029), remove dublicate field

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Old bug, instead of PURL field write Network into mp4 container. Also mentioned here https://github.com/ytdl-org/youtube-dl/issues/16196
Also, there is no need to write description info into two fields, all containers support just description

why

And what's wrong with it? Do you close all pull requests and issues without any reason?

@dstftw The disrespect shown to this @megapro17 (and other contributors) with the lack of a meaningful comment on their declined pull request has motivated me to replace my usage of this tool with a fork (yt-dlp/yt-dlp).  Your hard work is appreciated beyond belief, but it doesn't put you above basic courtesy.

Yeah original is dead, fork already has it. I wish I knew about it sooner


Why Latin1 for report tables?

Is it ok to use eg. utf8 charset when defining the reports tables? It would seem to make sense, given the script/hostname data present in them

Shouldn't be an issue to use whatever charset you prefer. Have you tried?
The engine itself does not interpret strings, just stores them as a bunch of bytes.

Thx for the info. I will give it a try and come back with the results

One potential issue i can think of is protobuf-c implementation on the client side.
In case you use it from php - it might store them as c-strings (i.e. terminating null byte) - shouldn't be a problem for utf8, but maybe other things.
Engine itself, parses protobuf as data+length, so null bytes should not be a problem there.
In any case - curious to see if it works for you, if it doesn't - we'll investigate.

I did test this locally with "simple" utf8 script such as 'κόσμε', and it works.
The Pinba2 server in use was the official container from this project. The client-side were the php pinba extension (ver. 1.1.1) from Ubuntu Focal, and one pure-php protobuf implementation.
I created the reports table as
CREATE TABLE `report_by_script_name` (
    `script` varchar(64) NOT NULL,
    ...
) ENGINE=PINBA DEFAULT CHARSET=utf8 COMMENT='v2/request/60/~script/no_percentiles/no_filters';

And I had to add in the code querying its data: $db->set_charset('utf8');

Ok, great to know.
I'll keep this one open for a bit longer, please let me know if you get to check out the null chars - is it a client php extension issue or an engine issue.

WDYT about changing the sql files which are part of this repo? At the very least we could add a comment in them mentioning Latin1 is not a requirement...

Sure, happy to accept a PR with utf8 everywhere in sql files.

Sure, will do.
Btw, I checked the "raw message" being sent when there is a NUL byte in the middle of the php string used to indicate the script_name, using Wireshark as protobuf dissector:

both the php extension and my pure-php polyfill do send a message in that scenario
the packet sent by the php extension is dissected as having a string with a value made of all chars up to the null one, exclusing it
the packet sent by the polyfill apparently sends "garbled" data

So it seems that the code dropping those packets might be on the server-side (I will test that again with Pinba2, last tests were ran against Pinba1)

if I add chr(0) in the middle of the script_name, the corresponding line does not show up any more in the db after calling flush on the client side

Further testing disproved the above.
I ran fresh tests with a NUL char in the middle of hostname, server_name and script_name, after having patched my own lib to work the same way as the php extension (ie. truncate strings at the first NUL char found), and both pinba1 and pinba2 engines are happy with what they get.
I wonder why I got no-data results in the first rounds of testing... probably because sending a single NUL char results in an empty string, which might not be considered a valid script_name/server_name/hostname.
In any case: sending of NUL chars does not seem to have much to do with the chosen character set encoding for retrieving the data once it's in the DB (at least as long we are talking about UTF8 vs Latin1 - they both use the same encoding for NUL).
I will send a PR as discussed above.

I kind of assumed that pinba and pinba2 engines should treat strings with null-bytes in the middle differently, attributing to slightly different ways they unpack incoming protobuf packets.
Pinba1 - truncating, pinba2 - preserving.
For uts8 this point is moot since there is you're "not supposed to" havre null-bytes inside strings.
Thank you for the PR's, have been very busy lately, will review asap.

I kind of assumed that pinba and pinba2 engines should treat strings with null-bytes in the middle differently, attributing to slightly different ways they unpack incoming protobuf packets.

This is possible.
What I did verify is that, for the pinba php extension, the truncation happens on the client side - possibly within code generated by the protoc compiler - so in those tests the NUL char was not sent to the engine at all.
As for being able to send a NUL char as part of the protobuffer packet: I did manage to do that by using my own php implementation of the client, but the results were mixed. I tried to decode the raw protobuf packet to check its contents: using protoc --decode_raw, the string was decoded correctly, and NUL displayed as \u0000; using wireshark the string was highlighted as being malformed. This prompted me to alter the behaviour of the php code to behave the same way as the php extension, as there seems to be little point in being able to send NUL chars if in any case all it would be helpful for is to expose weird corner cases in the receiver side. For all I know, it could even be used for some hacking attempt :-) I can not report reliably of behaviour differences between pinba1 and pinba2.

For uts8 this point is moot since there is you're "not supposed to" have null-bytes inside strings.

Not to be pedantic, but the NUL character is valid in utf8, just as it is valid in ascii. It is just C strings that have issues with that ;-)


Dev Tools UI

PR-Codex overview
This PR focuses on setting up the frontend development tools for the project.
Detailed summary

Added web-devtools frontend project
Updated dependencies and configurations
Added Netlify settings for Yarn 3 compatibility
Updated ESLint and EditorConfig settings
Defined project metadata in package.json and LICENSE files

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Summary by CodeRabbit

New Features

Introduced a comprehensive ESLint configuration for improved code quality and consistency.
Added a .gitignore file to exclude unnecessary files and directories from version control.
Updated the LICENSE file to include the MIT License.
Added a README.md file with an introductory section for "Dev Tools Frontend."
Introduced a netlify.toml file for Netlify deployment configurations.
Added a package.json file with metadata, dependencies, and scripts for the web development tool package.

@coderabbitai generate interesting stats about this repository and render them as a table.


Modify docs mentioning the APIM version to avoid issues

This PR modifies docs mentioning the APIM version to avoid issues

As the next planned version is 3.2.0, shouldn't this be 3.2.0 instead of 3.1.0 as this change is going to the master branch?

I kept the version as 3.1.0 to make it consistent in all the places in master branch, so that we don't miss to change all when we are doing the next release.


Display DOI load information when available

Some amigo loads may have associated DOI resource files associated with them. We'd like AmiGO to find these at build/cache time and display them (w/link).
Two new optional variables: one for DOI info and one for resolver.

Still need in ansible to grab on re-install/deploy.

Now in ansible.
Tried out on experimental, seems to have worked (delta the fact that sandbox will always take one to the wrong location as now a production URL).
Update data and code on next release.