Space above tabs does not react to clicks

Environment
Microsoft Windows [Version 10.0.19042.630]
Windows Terminal Preview
Version: 1.5.3242.0

Steps to reproduce
Click above tabs.
Expected behavior

In maximized state, since tabs visually extend to the top of the title bar, clicking at the very top should also activate the tab.
In normal state, either make tabs expand visually and behave the same as in maximized state, or make the title bar above the tabs clickable.

Actual behavior

In maximized state, tabs visually extend to the top of the title bar, but clicking at the top does not activate the tab.
In normal state, tabs visually don't extend to the top of the title bar but leave a small area of the title bar. Clicking this area doesn't do anything, the window cannot be dragged. Double-click doesn't maximize the window.

@xnoreq - might be related to https://github.com/microsoft/terminal/issues/7422.

In maximized state, tabs visually extend to the top of the title bar, but clicking at the top does not activate the tab.

I'm definitely not seeing this - any chance you're using an auto-hiding taskbar?
~I believe the other half of this bug is in #1625~ Nope, okay it's not in there. I know there's one on the backlog somewhere for the area above the tabs, but idk where.

I'm definitely not seeing this - any chance you're using an auto-hiding taskbar? What display resolution are you at? Monitor layout?

I got the same issue every now and then and was about to open an issue myself until I saw this.
I was able to reproduce it on both 1080p and 4k resolutions (100% and 150% scaling) (didn't test others so far), with a pretty standard monitor layout with a screen left and right with no offsets, but it only happens sometimes (I don't really know what could be the reason).
I will try to find a definite setup that guarantees that this is happening.


Get correct world position and orientation for each mesh in MuJoCo to export to external viewer

Hi,
I'm a researcher and I'm trying to use MuJoCo for research.
I'd like to know how to get the correct world position and orientation for each mesh in MuJoCo to export to an external viewer eg open3d
For example, for a robot XML in mujoco_menagerie, at every step, I'd like to keep tracking each mesh's pos and rotations and render them into the external viewer, e.g. open3d.
I first get the body to mesh map by sth like
def generate_mesh_name_to_body_map(root):
    mesh_to_body_map = {}

    def recurse_bodies(parent_body, parent_name=""):
        body_name = parent_body.get("name") or parent_name
        if body_name:
            for geom in parent_body.findall("./geom"):
                mesh_name = geom.get("mesh")
                if mesh_name:
                    if mesh_name in mesh_to_body_map:
                        assert body_name == mesh_to_body_map[mesh_name]
                    mesh_to_body_map[mesh_name] = body_name
            for sub_body in parent_body.findall("./body"):
                recurse_bodies(sub_body, body_name)

    for body in root.findall("./worldbody/body"):
        recurse_bodies(body)
    return mesh_to_body_map

And then by iteration and mapping, getting the transform
        data_pos = data.xpos[body_id]
        data_xmat = data.xmat[body_id].reshape(3, 3)
        transform = np.eye(4)
        transform[:3, :3] = data_xmat
        transform[:3, 3] = data_pos
        render_object: o3d.geometry.TriangleMesh
        render_object.transform(global_transform)

However, most parts work properly, but it's weird that one of the grippers has the wrong orientation, and is missing one mesh part, which may be due to the wrong rotation or sth else. Here is a screenshot that compares the meshes in MuJoCo (left) and open3d (right):

Thanks for the help in advance!

MuJoCo applies an automatic transform to all meshes (centering and reorienting), this is explained here. This transformation is recorded in mjModel.mesh_{scale, pos, quat}.
Note to self update the docs.

Actually, this is already documented!
We save the translation and rotation offsets applied to the source asset in mjModel.mesh_pos and mjModel.mesh_quat; these are required if one reads vertex data from the source and needs to re-apply the transform.

thanks @yuvaltassa do you mean to use the mesh_transform_mat @ np.ling.inv(geom_transform_mat) where mesh_transform_mat is matrix from mjModel.mesh_pos and mjModel.mesh_quat and geom_transform_mat is from geom pos and quat? Seems it still does not work properly. Could you please share some simple explanatory scripts for this?

#1598


Build and push Docker image in CI

Hi!
I noticed there's no CI action to build and push the Docker image to DockerHub.
I can take care of it if you want 😄

Hi @agustinmista,
thanks for your offer! 🙏
I would really appreciate if you could work on that. I would provide the required docker hub actions.
I think using tags and a manual could be a solution.
Back in the days I have been using the docker hub auto build function, but now this is restricted as I only have a free account. So the image was never manually build + pushed, but the automation is gone.

I have been using an action like this one for my own images. Seems to work fine.
In this case it would be a matter of triggering it only when a tag is pushed:
on:
  push:
    # Pattern matched against refs/tags
    tags:
      - '**'   # Push events to every tag including hierarchical tags like v1.0/beta

If you agree with this I can give it a try and open a PR soon.

Looks good to go, give it a shot. 😎


Issue Accessing MammalNet Dataset from S3 Bucket

Good day!
I'd like to express my appreciation for the work on MammalNet. The dataset appears to be of immense value to the community, and I'm keen on exploring it further for my research.
However, I've encountered a hiccup while trying to download the dataset. Following the provided AWS CLI command in the repository:
aws s3 cp --recursive s3://mammalnet/full_videos/ .
I received this error:
fatal error: An error occurred (InvalidAccessKeyId) when calling the ListObjectsV2 operation: The AWS Access Key Id you provided does not exist in our records.
Additionally, I attempted to directly access the S3 bucket through the browser with the link:
https://mammalnet.s3.amazonaws.com/full_videos/
Unfortunately, I was greeted with an Access Denied error.
<Error>

AccessDenied
<Message>Access Denied</Message>
<RequestId>3Q38ARDY3YN4PKER</RequestId>
<HostId>nJn4/MqXXqn2mo1tA8hE6jl7Uy0ifDK1pK64o58aqEZP0O2qGV+H8WBrvAl5EVwiTEJSb2ggD5I=</HostId>
</Error>

Given these issues, I was wondering if it would be possible to either make the S3 bucket public or provide the necessary access credentials to download the dataset?
Your assistance would greatly benefit my ongoing research and, I'm sure, many others in the community.
Thank you in advance for your time and help.
Looking forward to accessing this promising dataset!

thanks for informing me this issue.  I will fix this and will let you when it is ready

Hi I have updated the data files in aws s3. Now you can download it with wget. Can you give it a try now?

Thank you so much!
The download links are now working and I am currently downloading them.  :)

Hey!
I was able to download full videos using: wget https://mammalnet.s3.amazonaws.com/full_video.tar.gz
However, I encountered an "Access Denied" error while trying to download the features file with wget https://mammalnet.s3.amazonaws.com/full_video_features/mammalnet_feature.tar.gz
Any insights on how to resolve this issue would be greatly appreciated.
Thanks!

It's accessible now, Thank you so much.


Highlight failure ladder on Prif agility course

Is your feature request related to a problem? Please describe.
When you fail in the latter half of the Prif agility course there is a ladder to get back up. It's hard to see this ladder depending on the camera angle.
Describe the solution you'd like
Could you highlight the ladder like the other ones in the course?
Describe alternatives you've considered
N/A
Additional context
N/A

fixed in issue #9508

This needs to be closed


Improve the acustic guitar?

Cool project!
This is a small idea, as someone who plays the intrument, the current one does not looks like a guitar haha

Here's another that I found using a quick web search..

@eridal  Thanks for the suggestion. However, using assets created by others without clear authorization would lead to potential issues.  I would extremely appreciate and consider merge the change if you can submit one of your own creation. I suggest using make8bitart or aspirate. They are my own go-to tools for designing assets for Lo-Fi Player.


Fix test failures

This fixes test failing because of split or merged TCP packets.

Thanks for the fix @jscissr!


Bot can't play 1+hour songs

When i make a playlist for my bot with tracks with duration as in the screenshot:

He plays track for like 24 minutes and skip it and play next track for like 1 min and also skip it
I run my bot on VPS on Windows server,no errors and yesterday bot works correctly as my testers said to me,i don't change something from yesterday

I think this is discord somehow blocks it but idk..
I also leave you small debug from discord here,as i can see bot lose a lot packets,idk what that can mean..

Also i have 1 CPU and 1gb RAM,i run a second bot with your bot,but i dont think this "lags" bcs of my server..

I play tons of stuff that's >1 hour, 24/7 even...no issues whatsoever. This isn't an issue with the bot.

@shikhir-arora you run bot on VPS??

Im Running my bot within the Azure Cloud (Virtual Machine)
No issues here I would assume that it's you're 1 gig of ram since this bot appears to be streaming songs rather than downloading and purging (Not sure though haven't looked through enough source code to know)

I believe that this is an issue with your hosting and not the bot. However, if you can reproduce this with the latest version, feel free to reopen this issue and provide any applicable logs you have.


Bug  When sign in/sign up /forget password using aws amplify with ionic

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Can't sign in or sign up or forgot password(email are not even send) with aws amplify on mobile (with ionic-cordova). and getting this error message:
{code: "UnknownError", message: "Unkown error" }
Also is working fine on webApp
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have dependencies other than AWS Amplify.
I just tried to sign up a user or sign in user as usual
What is the expected behavior?
to be able to login/sign up a user
Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
"aws-amplify": "^0.4.1",
"aws-amplify-angular": "^0.1.1",
on mobile framework (ionic cordova)
I also tried to update my amplify version for the latest version:
"aws-amplify": "^0.4.6",
"aws-amplify-angular": "^0.1.6",
and I'm getting the same issues again can't login or wtvr
You can turn on the debug mode to provide more info for us by setting window.LOG_LEVEL = 'DEBUG'; in your app.

when I comment the mandatory sign in, i got this error

Can you look into the network tab and provide some info around the sign in http request?

@powerful23 it's not possible for me now to reproduce the issues its working again sinece few minutes ( didn't work all day. It's first time that I was facing this issues

@powerful23 I did see 302 http code in the request and 200 for the get one, I don't know if its related to the 302

can you please share your package.json file ? @Jasminou

Going to close this as the original issue has been solved. Please reopen if you have any more related questions.


Generating Error while integrating Role and Permission with Mongo DB, any best solution or a demo ?

Maybe provide more information and stacktraces?
Just an issue without any kind of information can't be fixed.


Is it possible to integrate gravatar?

https://en.gravatar.com/

Gravatar is now available in master. To enable this feature: "Settings > Integrations > Gravatar".
Note: Avatars are only displayed on the activity stream and comments at the moment.


HIRE HACKATHON TECH SOLUTIONS TO RETRIEVE SCAMMED BITCOIN/USDT/CRYPTO FROM INTERNET FRAUDSTERS

Investing in the promise of financial growth can sometimes lead to unexpected and devastating outcomes. My experience with a seemingly reputable company turned into a nightmare when what appeared to be a legitimate opportunity to invest in cryptocurrencies resulted in the loss of 19 BTC and 4 ETH—substantial assets that were crucial for my financial security. Initially, the company presented itself with professionalism and persuasive assurances of guaranteed returns and low risks. However, as I probed deeper with reasonable questions about their operations and plans, I encountered evasion and resistance. My attempts to withdraw my investments were met with increasing hostility, culminating in being blocked from accessing my accounts altogether. Realizing that I had fallen victim to a sophisticated scam, I felt betrayed and powerless. It was a devastating blow, not just financially but also emotionally, as I grappled with the consequences of misplaced trust. In my search for a way forward, I came across HACKATHON TECH SOLUTIONS through positive reviews and recommendations from others who had faced similar challenges. With skepticism but desperate for a solution, I reached out to them in the hope of reclaiming what had been wrongfully taken from me. HACKATHON TECH SOLUTIONS proved to be a beacon of hope in my darkest hour. From the outset, their team demonstrated a deep understanding of the intricacies of cyber fraud and cryptocurrency transactions. They listened attentively to my story, offering empathy and clarity on the steps we would take to recover my stolen assets. Their approach was methodical and thorough. Using advanced forensic techniques and their expertise in blockchain analysis, HACKATHON TECH SOLUTIONS meticulously traced the path of my misappropriated funds. Throughout the process, they kept me informed with regular updates, transparently explaining their findings and strategies. Despite the complexities and challenges involved, HACKATHON TECH SOLUTIONS persevered with unwavering determination. After days of intensive efforts, they achieved the remarkable feat of recovering all 19 BTC and 4 ETH that had been unlawfully taken from me. The relief and gratitude I felt were immense—finally, justice had prevailed. Beyond recovering my assets, HACKATHON TECH SOLUTIONS provided invaluable insights and advice on safeguarding against future scams. They emphasized the importance of due diligence and cautious investing practices, equipping me with the knowledge to protect myself in the digital investment landscape. I am grateful for the resilience that guided me through adversity and the support of ethical professionals like HACKATHON TECH SOLUTIONS. They are not just experts in their field but also champions of justice for those who have been victimized by financial fraud. To others who may find themselves in similar circumstances, I offer a message of hope and encouragement: seek assistance from reputable recovery services like HACKATHON TECH SOLUTIONS. Together, we can reclaim what is rightfully ours and move forward with renewed strength and vigilance against deceitful schemes. While the scars of deception may linger, my experience with HACKATHON TECH SOLUTIONS has restored my faith in justice and integrity. They are true allies in the fight against financial fraud, ensuring that victims like me have a voice and a pathway to reclaiming our financial security.Reach out to HACKATHON TECH SOLUTIONS  via below contact details
Email<EMAIL_ADDRESS>Website:https://hackathontechsolutions.com/
Whatsapp: +31 6 47999256
Telegram:
@hackathontechsolution OR +31 6 47999256

Investing in Bitcoin is like embarking on a thrilling adventure filled with excitement, anticipation, and yes, a fair share of uncertainty. For many, myself included, the allure of potentially astronomical returns can overshadow the inherent risks. So, when my journey took an unexpected turn and my 231,000 BTC investment became nothing more than a digital mirage, I was left in shock and disbelief. It was a devastating blow that threatened to shatter my confidence not just in cryptocurrency, but in the notion of financial security. amidst the chaos beacon of hope emerged in the form of Trust Geeks Hack Expert. With a reputation for excellence in the realm of online recovery, they were my last line of defense against the forces that sought to rob me of my hard-earned investment. From the moment I reached out to them, I knew I was in capable hands. Their professionalism, expertise, and unwavering dedication to their craft were evident from the start. The journey to reclaiming what was rightfully mine was fraught with challenges, but Trust Geeks Hack Expect navigated every obstacle with finesse and precision. They were more than just experts in their field; they were guardians of justice in the digital age, fighting tirelessly to right the wrongs perpetrated by malicious actors lurking in the shadows of the internet. But what truly set Trust Geeks Hack Expect apart was their unwavering commitment to their clients. Throughout the entire process, they kept me informed, reassured, and empowered. Their transparency and integrity were evident in every interaction, providing me with the peace of mind I so desperately needed during those tumultuous times. In terms of cost, while the thought of investing additional funds in the pursuit of reclaiming what was rightfully mine initially gave me pause, the nominal fee charged by Trust Geeks Hack Expect was a small price to pay for the invaluable service they provided. Their expertise, professionalism, and unwavering dedication were worth every penny, and then some. Ultimately, my experience with Trust Geeks Hack Expect was nothing short of transformative. They not only restored my faith in the inherent goodness of humanity but also empowered me to reclaim control over my financial destiny. To anyone who finds themselves in a similar predicament, I wholeheartedly recommend reaching out to Trust Geeks Hack Expect. They are true professionals with a heart of gold, E m a i l : trustgeekshackexpert {@} fastservice {.} com

If the process is too good to be true, it’s probably a phony crypto recovery expert. I have been scammed by so many recovery companies, and I can tell, it’s almost not possible to recover stolen or crypto sent to the wrong address. It’s rare to find a legit recovery company without getting scammed believe it or not. I’m saying from experience and I can tell you,easyrecoveryassets@ gmailcom is reliable, not 100% but will never leave you stranded.


Shutdown Hooks

Really love the event hooks and the degree of customizability they provide!  Could we also get hooks for shutdown events?
Would also be nice to have hooks that are only triggered if an update is actually available and being applied, rather than on every update check. (Appears that the pre/post restart hooks serve this purpose!)

Implemented in #167


Accordion module

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

branch is dependent on https://github.com/graycoreio/daffodil/pull/19

I think an ideal component scenario here would be...
<accordion>
    <accordion-item>

Title

Accordion Content

    </accordion-item>
</accordion>

This allows us to be a bit more flexible in defining specifically what tags are "SEO" friendly and additionally allows us to be significantly more ARIA friendly.


The enhancement for multiple schema

Fixes #1301 .
Changes proposed in this pull request:
-Retrieve database names from memory when using SHOW DATABASES
-DCL broadcast to database instance
-SET syntax broadcast to database instance

Pull Request Test Coverage Report for Build 5319

13 of 75 (17.33%)  changed or added relevant lines in 9 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 63.05%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/netty/NettyBackendHandler.java
0
1
0.0%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/jdbc/JDBCBackendHandler.java
0
2
0.0%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/BackendHandlerFactory.java
8
11
72.73%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/SchemaUnicastBackendHandler.java
0
7
0.0%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/SchemaBroadcastBackendHandler.java
0
21
0.0%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/SchemaIgnoreBackendHandler.java
0
28
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/jdbc/JDBCBackendHandler.java
1
0.0%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/BackendHandlerFactory.java
3
71.43%

Totals

Change from base Build 5318:
-0.2%

Covered Lines:
7931

Relevant Lines:
12579

💛  - Coveralls


Try vim-flake8

pip install flake8
git clone https://github.com/nvie/vim-flake8.git ~/.vim/bundle/vim-flake8

Then, to use, just press <F7> while editing a python file

Could also try adding to .vimrc:
autocmd BufWritePost *.py call Flake8()


Failed to activate the node-resolver package

Installed Atom
Tried to install node-resolver

Atom: 1.23.3 x64
Electron: 1.6.15
OS: Microsoft Windows 10 Home Single Language
Thrown From: 'node-resolver' package vundefined
Stack Trace
Failed to activate the node-resolver package
At Cannot read property 'command' of undefined

TypeError: Cannot read property 'command' of undefined
    at Object.exports.activate (/packages/node-resolver/index.js:12:21)
    at Package.module.exports.Package.activateNow (~/AppData/Local/atom/app-1.23.3/resources/app/src/package.js:255:25)
    at ~/AppData/Local/atom/app-1.23.3/resources/app/src/package.js:227:38
    at Package.module.exports.Package.measure (~/AppData/Local/atom/app-1.23.3/resources/app/src/package.js:99:21)
    at ~/AppData/Local/atom/app-1.23.3/resources/app/src/package.js:220:32
    at Package.module.exports.Package.activate (~/AppData/Local/atom/app-1.23.3/resources/app/src/package.js:217:40)
    at PackageManager.activatePackage (~/AppData/Local/atom/app-1.23.3/resources/app/src/package-manager.js:695:42)
    at ~/AppData/Local/atom/app-1.23.3/resources/app/node_modules/settings-view/lib/package-manager.js:535:35
    at exit (~/AppData/Local/atom/app-1.23.3/resources/app/node_modules/settings-view/lib/package-manager.js:128:22)
    at triggerExitCallback (~/AppData/Local/atom/app-1.23.3/resources/app/src/buffered-process.js:338:17)
    at ChildProcess.<anonymous> (~/AppData/Local/atom/app-1.23.3/resources/app/src/buffered-process.js:368:17)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)

Commands
  3x -4:53.8.0 core:confirm (input.hidden-input)

Non-Core Packages
angular-2-typeScript-snippets 0.7.0
angularjs 0.5.3
angularjs-snippets 0.10.18
atom-nodejs-snippets 2.0.0
autocomplete-angular-material 0.4.0
Hydrogen 2.2.2
node-resolver 1.0.1

Thanks for contributing!
It appears that this is being caused by the node-resolver package, not Atom itself. It may be that there is still a bug in Atom. If so, we'll need the maintainer of the package that is causing the problem to help track that down. You should file this issue on their repository so they can either fix the problem or tell us where the problem is coming from. The Atom FAQ has information on how to contact the maintainers of any Atom community package or theme.


Feature: introduce logging system

Description
Introduce and modify printed output to use structured levels logging system (slog)
Ref: https://pkg.go.dev/log/slog

Decided not to implement with log system and use simple debug mode to determine whether to output message for debugging


Not available in Plugin Directory

Are there any plans on making this available in KanBoard Plugin Directory?

Hello,
This will be fixed in the future versions, in the following weeks. I do not have an estimation for the date, though, as the plugin will undergo important changes.

Hello, is there a continuation on making the plugin available on the plugins directory? Let me know if you need some help.

In the meantime, it would be nice to have instructions to install the plugin if it can't be added to the directory any time soon


Receiver node from "develop" branch dies due to accessing nullptr

Hi @xqms
We've got an issue with the receiver node dying because of segfault (exit code -11) when using the "develop" branch.
@tomlankhorst and I tried to debug this issue and could get a backtrace using gdb (see end of the message). We were wondering if you could provide us with your feedback about this issue. This is our setup:

a base station runs the receiver node
an agent runs the sender node
the agent sends both tcp (sporadic messages) and udp (images and tf) topics

When the agent goes out of connection range and comes back after some minutes, we often get the receiver node dying on the base station. We suspect that Depacketizer::addPacket is called asynchronously on the same object (for example when 2 UDP packets are received) and so addPacket is called twice. But if it's called async, then it will mess up because the value of the iterator will be changing.
Do you have any hint or suggest on how we could fix this issue?
Best,
Marco
Backtrace:
#0  0x00007ffff7d6ce72 in nimbro_topic_transport::Depacketizer::handleMessagePacket (this=0x7fffffffbfb0, it=
    {id = 24190, complete = false, received_symbols = 10, packets = std::vector of length 10, capacity 10 = {std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48a92b0}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48b0120}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48b06f0}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48b0cc0}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 2, weak count 0) = {get() = 0x7fffd48b1290}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48accd0}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48a9e50}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48a8140}, std::shared_ptr<nimbro_topic_transport::Packet> (empty) = {get() = 0x0}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd48aefb0}}, decoder = std::shared_ptr<WirehairCodec_t> (empty) = {get() = 0x0}}, packet=...) at /usr/include/c++/9/bits/shared_ptr_base.h:1020
#1  0x00007ffff7d6df79 in nimbro_topic_transport::Depacketizer::addPacket (this=0x7fffffffbfb0,
    packet=std::shared_ptr<nimbro_topic_transport::Packet> (use count 2, weak count 0) = {...})
    at /home/subt/catkin_ws/src/nimbro_network/nimbro_topic_transport/src/receiver/depacketizer.cpp:44
#2  0x00007ffff7d71432 in std::function<void (std::shared_ptr<nimbro_topic_transport::Packet> const&)>::operator()(std::shared_ptr<nimbro_topic_transport::Packet> const&) const (__args#0=std::shared_ptr<nimbro_topic_transport::Packet> (use count 2, weak count 0) = {...},

Hey, thanks for the report! I think you uncovered a pretty serious bug - I'm not sure how this went unnoticed for so long :/
I'm pretty sure it's not a concurrency issue, since Depacketizer::addPacket is protected with a mutex.
But there's a bug hidden in addPacket(): It first saves the iterator, and then calls pruneMessages(), which potentially removes all messages.
Could you please try the fix in #18?

@xqms thanks for your quick reply and fix ... amazing!
I did try the proposed fix a few times and could not reproduce the error anymore. I'm gonna try again next week with more extensive tests and let you know.
Once again, thanks a lot for the fix!
Best,
Marco

Hi @xqms
Unfortunately the issue happens again even after using the proposed fix. I got another backtrack:
(gdb) bt
#0  0x00007ffff7d6ce72 in nimbro_topic_transport::Depacketizer::handleMessagePacket (
    this=0x7fffffffbed0, it=
    {id = 6619, complete = false, received_symbols = 11, packets = std::vector of length 11, capacity 11 = {std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd420e940}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd420e370}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd420ef10}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd420fab0}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd4210650}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd42111f0}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd4211d90}, std::shared_ptr<nimbro_topic_transport::Packet> (use count 1, weak count 0) = {get() = 0x7fffd4212930}, std::shared_pt--Type <RET> for more, q to quit, c to continue without paging--

FYI @tomlankhorst

That's unfortunate... Could you please compile nimbro_network in Debug mode (cmake arg -DCMAKE_BUILD_TYPE=Debug) so that we can get a more complete backtrace?


problem with website name

hello sorry i had a problem with the publication .
my problem is quite simple but i can figure out the solution.
when i look at my lures , the phishlet and the redirect URL both says instagram in my case, but when i create a link, it says the domain i create and not the one of instagram... to be clear , the link looks like that "www.icecream.gq/bm..laTC" (i wont put the whole link) instead of "www.instagram.com/..."
any solution for that ? :/

I use a domain that I own ... It's very hard to get some help in here ...

Which domain is showing in the URL bar? I need more information. Help is hard to provide if you don't share the details of the issue you are having.

Sorry I didn't see you replied here .. the URL I get when I do the command get-lures shows my domain ( icecream.gq in this case) but I want it to display the website target ( Instagram in this case ) and I'm kinda lost , I tried to change everything and redirect the links but nothing work :/


Decorator can not access controller helper in test

Today I noticed a weird behaviour during my request spec:
The controller defines a helper method which is called from the decorator. But during the test this helper is not available.
Please see this gist with a sample app that reproduces the problem, using Ruby 2.3.3:
https://gist.github.com/swelther/47c5ffb707ca1145ca87d1dfb8aa99fa
My real world app uses RSpec instead of Minitest, but the result is the same.
Shouldn't it be possible to access helpers defined in the controller? What can I do to fix this?

Thanks for reporting this issue. It looks like it has been around for a while. It appears to have to do with the ViewContext not being cleared after each spec. I'll look into this and get back to you.

I've got the same result after modernizing the gist for master and latest Rails.

After a lot of debugging and digging in, it appeared that Draper initializers don't get run for some reason. Thus, adding one to the gist fixes the issue:

ActiveSupport.on_load :action_controller do
  Draper.setup_action_controller self
end


chore: checking integration tests

Description:
Checking if the integration tests are passing against different network tags.
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Not needed anymore.


Remove version: Cisco.CiscoWebexMeetings version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://akamaicdn.webex.com/client/WBXclient-41.3.5-8/webexapp.msi
OK
(Automated message - build 912)


Add "Deploy to Heroku" button

Is your feature request related to a problem? Please describe.
Setting up Labellab for users might be too cumbersome if they have to set it up manually. Adding a one-click deploy button to the README will enable anyone to run Labellab for free.
Describe the solution you'd like
A one-click Deploy to Heroku button should be added to the README
Describe alternatives you've considered
N/A
Additional context
I am currently working on this feature.

@Tushar19varshney Sure, I'll keep working on Heroku integration as more features are added to the project.


When Editing, Date Field Reverts To Previous Day

Nghia believes there was previously as issue closed related to this but was unable to find it and asked me to create a new one.
💻 Environment

Platform:  Windows
OS Version:
windows_native_app/7.4.345-obm-87-obm-98-obm-99-obm-101
Toggl Version:  Version: 7.4.345
🐞 Actual behavior

The User writes;
"Start timer when it is PM, Stop, Edit that entry and set the start time to any time that is before 12:00 like 10:00, Notice that the Date field is automatically set to yesterday and you can't manipulate it to today again."
💯 Expected behavior

Date field reflects current date.
📦 Additional info

Case: /conversation/21775004221

Related #2561, #2455, #2012

STR which work for me:

Windows 10
Set time zone to (UTC+12:00) Auckland, Wellington
Set time to between 12:00 AM and 01:00 AM
Update the time zone in the Toggl profile
Start the timer
While the timer is running, switch the date to 1 day before the current day.

Actual Result:

The date is switched to 2 days before the current day.


Add media upload support

Adds ability to upload profile pictures. The profile images can be updated via PUT requests in the API or directly in the django admin. This implementation uploads photos to the server's file system. The next step is to upload files to a blob store like AWS S3

[x] Allow users to upload profile images
[ ] Upload images directly to AWS S3

Migrations are part of the code, they track the changes to the database
schema over time and create a consistent environment for all contributors
to the project.
They keep your code in sync with the DB and make it easy to revert to a
previous state if anything ever went wrong with the database.
Not tracking them in git could lead to conflicts if different contributors
maintained their own local migrations.
On Thu, 10 Oct 2024, 23:32 Ngonidzashe Mangudya, @.***>
wrote:

@.**** commented on this pull request.
Out of curiosity, what are our thoughts on having migration files as part
of the repo?
—
Reply to this email directly, view it on GitHub
https://github.com/PythonBulawayo/ChitChat/pull/67#pullrequestreview-2361458506,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2CARGGVJU2BYFIVNY4CJTZ23W7ZAVCNFSM6AAAAABPXSV6VWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRRGQ2TQNJQGY
.
You are receiving this because you were assigned.Message ID:
@.***>

@terrameijar Sure, thanks.


Add aboutyou.it as hard

Thanks for your contribution!

Note: I see you didn't create a new branch, but rather committed on your master, so you'll have the same problems as before if you open a new PR...


Add support for colon separator of keyspace:shard

@sougou @michael-berlin

We'll wait for @michael-berlin to look at this also.

Thanks, I changed to LastIndexAny. I also removed the test which fails on a path with multiple slashes, because with LastIndexAny it's harder to determine that. Not sure how strict you want to be about that case. I can add it back and go a different route if you'd like

@michael-berlin @sougou updated per conversation in slack

@michael-berlin fixed up, thanks

Thanks! :)


What does one need to do to get access to / download the dataset?

I'm interested in using it for a detection project.

Any update?


Unable to change installation directory

Limited drive space considering more packages to be installed. Need to change default windows directory to D drive. I can't find anything on forums or documentation. It is curious how Atom claims to be "hackable" but I can't find a way how to do a very basic functionality :)

Thanks for the feedback, @abenitesDC -- I think this is covered in https://github.com/atom/atom/issues/7504, so I'm closing this in favor of that issue. Please feel free to leave a comment there or just subscribe for updates.


Add an insecure option for testing hawkular using uncertified ssl connection

Description
When using hawkular python client using my python2 and urllib2 I was unable to connect to
the hawkular server because it was insecure. This PR adds the option to connect to an insecure server using ssl for development and testing.
Example
Without insecure=True:
client = HawkularMetricsClient(host='yzamir-centos7-1.eng.lab.tlv.redhat.com', port=443, scheme='https', token='...', tenant_id='_system')
>>> client.query_tenants()
 ... <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)>

With insecure=True:
client = HawkularMetricsClient(host='yzamir-centos7-1.eng.lab.tlv.redhat.com', port=443, scheme='https', token='...', tenant_id='_system', insecure=True)
>>> client.query_tenants()
[{u'id': u'default'}, {u'id': u'openshift-infra'}, {u'id': u'_system'}]

@simon3z @pilhuhn @lucasponce please review

I'm against this change. Adding an "insecure" option will eventually lead to have a code in production with this value set to true, which completely defeats the purpose of using SSL in the first place. You should instead really try to get a proper certificate for your server.

By the way: ping me on #hawkular and I can show you some alternatives :)

Adding @burmanm to the review of this PR.
He is the lead for this client in particular.

Adding an "insecure" option will eventually lead to have a code in production with this value set to true

It makes my life while developing easier, But as a developer I can also have my own version of this client :-)
It helps me, maybe it can help someone else, but then as @jpkrohling say it may cause more truble ... @burmanm WDYT ?

The right thing to do isn't always easy ;)

I'm against this change. Adding an "insecure" option will eventually lead to have a code in production with this value set to true, which completely defeats the purpose of using SSL in the first place. You should instead really try to get a proper certificate for your server.

@jpkrohling every client using ssl supports this override (making clear that it's not for production).
Also, SSL has multiple purposes, for example encryption and data integrity would be still working.

Neither encryption nor integrity can be guaranteed if the certificate is not trusted. If an encrypted payload can be intercepted, changes are extremely high that a MITM-attack can be performed.
That said, there are possible ways to provide a similar feature in a more secure way. One solution is to allow the client to provide a SSL Verifier, so that at least a certificate pinning is available. The way this patch is sent, it's an "either/or" approach, which is not good.
But really: the real solution is to get a valid cert. I provided a link to @yaacov to a service that Red Hat provides to internal services to get trusted certs. A secondary solution is to add the self-generated cert to the system's trusted certificates as part of the environment setup.

@jpkrohling When writing code you can do things that are way stranger than trusting an insecure ssl connection. We allow developers to write code and we trust them not to do things too weird. I think we can trust an average developer not to write insecure=True in production.
But then again you may be right :-)

The approach I've taken in the Golang client is that the client can push any sort of SSLContext they wish to the client and use that (and as such, move insecure handling to their own code - so they can use it if they want to, but we don't actually support it).
I think I'd like that approach as well, as the insecure is quite handy for some automated unit tests or integration tests. The urlopen(.., context) is however problematic as that syntax was introduced in Python 2.7.9, yet RHEL7 ships with Python 2.7.5. As such, I can't accept the current code since I need something that works out-of-the-box for RHEL7 also (and supports Python 3). So giving context in the NewHawkularClient would have otherwise been fine, if you can find a way to make it work for everyone (we could also remove the cafile / capath in that case from the init and only support setting context).

The urlopen(.., context) is however problematic as that syntax was introduced in Python 2.7.9, yet RHEL7 ships with Python 2.7.5.

@burmanm On my centos 7 with python 2.7.5, it seems to work, am I missing something ?
# python
Python 2.7.5 (default, Aug 18 2016, 15:58:25)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-4)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from urllib2 import Request, urlopen, URLError, HTTPError, HTTPErrorProcessor, build_opener, install_opener
>>> import ssl
>>> context = ssl._create_unverified_context()
>>> req = urlopen('https://www.google.com', context = context)
>>> req.read()

I wonder if they've backported in that case as documentation states: "Changed in version 2.7.9: cafile, capath, cadefault, and context were added."

The approach I've taken in the Golang client is that the client can push any sort of SSLContext they wish

@burmanm @jpkrohling changed the PR to use context as the outside var :-)

@yaacov , it looks really good now !


ts2556 is too restrictive when generics are involved

🔎 Search Terms
"generic", "tuple", "spread", "rest", "ts2556", "a spread argument must either have a tuple type or be passed to a rest parameter"
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about spreading into argument lists

⏯ Playground Link
https://www.typescriptlang.org/play?#code/CYUwxgNghgTiAEYD2A7AzgF3gMwFzwAoBKeAXgD54A3JAS2AFgAoZ0SWBZdLAD3zijBUEAJ7wA2gF1m2AgDoFPEgHpl8AO5IYAazTNmXTPAwgjpeAB4AKvBA8TKYGngChKURMnkCfeFZIUzPA48ooqakKm8ChIWJo6QA
💻 Code
declare const f: () => void

declare const x: readonly []
f(...x) // works

const test = <T extends readonly []>(x: T) =>
  f(...x) // does not work

🙁 Actual behavior
Attempting to spread a generically-typed value whose constraint guarantees tuple-ness into a compatible argument list is forbidden.
🙂 Expected behavior
I should be allowed to do this.
Additional information about the issue
Possibly-related issues:
https://github.com/microsoft/TypeScript/issues/29919
https://github.com/microsoft/TypeScript/issues/49802

It's likely that this one would get fixed by https://github.com/microsoft/TypeScript/pull/53615

Thanks a lot @mkantor ;)


rsync 3.1.1: Fix for Linuxbrew

@mikemcquaid @adamv Most formula in Homebrew/dupes build fine without any modifications for Linux, and I'd rather not have to maintain a fork of Homebrew/dupes. Would you consider merging this patch?
@mistydemeo What do you for the Homebrew taps for Tigerbrew?

@sjackman I think it'd be better if Linuxbrew had its own tap/formula for these sort of things. As I've said before I'm pretty unhappy with just add if OS.mac?.

if OS.mac? is pretty expressive of the true situation in this case, which is that these patches really are specific to Mac OS and make are not useful for any other platform. In an alternate universe where Homebrew and Linuxbrew were a single code base, can you think of a tidier way of dealing with patches that apply to a particular OS? Perhaps
patch do
    os :macos
    url "https://trac.macports.org/export/127713/trunk/dports/net/rsync/files/patch-crtimes.diff"
    sha1 "cf07fd4bdf09744f16dfd8f605f5f849a651f641"
end

I could see this being a possibly useful syntax for specifying a particular version of Mac OS as well.
patch do
    os :mavericks
…
end

What do you for the Homebrew taps for Tigerbrew?

Tigerbrew-specific logic is usually handled in the official taps; however there's very little actually required, since most Mac-specific stuff actually still applies to older OSs without modification (and any issues are often shared with Snow Leopard/Lion).

That said I personally don't mind including Linuxbrew-specific stuff in the taps if there isn't too much of it, since I know it's enough work maintaining a fork of Homebrew, much less of the taps.

Fair enough. Can you use a single if for them all?

Absolutely.

Thanks @sjackman!
For future reference the preferred commit message format for simple version updates is rsync: mark OS X specific patches..

Thanks for the not, Mike.


We Got Pached

Yea we got patched, zoom, aim, lasersight, everything isnt working for me as of 6:38 am. The Hack menu does show up but nothng works

https://www.youtube.com/watch?v=2DA9P1Z-QtM
this is my way of fixing "Unauthorized extension use detected"
And now everything is patched

unfortunately we have 3 leprechauns here and the hack will not be updated until further notice. Not my fault, blame these fuckers instead.

However

However if you do want the hack, please contact my discord where I will be undergoing extensive verification.

However if you do want the hack, please contact my discord where I will be undergoing extensive verification.

Hey, where can we find your discord link ?

@iBLiSSINYT


How to identify branches and get their files

I'm having troubles identifying branches. Is there a specific way I can identify remote branches and "check them out" locally?
I.e., I would like to know which remote refs are branches and how to check them out locally, so I can get their files.
While iterating through Repository.References, I see the following refs:
2017/01/03 16:00:54 Ref:8a7363b0912adafb89cb947a9ea5a95780bb4ec7 refs/heads/master
2017/01/03 16:00:54 Ref:e5cd2c000f1ab71d999af0eb993f0f9d191a53d8 refs/remotes/origin/develop
2017/01/03 16:00:54 Ref:8a7363b0912adafb89cb947a9ea5a95780bb4ec7 refs/remotes/origin/master
2017/01/03 16:00:54 Ref:e9a1a207b2d7cc0de898750df2df92f703f38aef refs/tags/testtag
2017/01/03 16:00:54 Ref:ref: refs/heads/master HEAD

Only refs/heads/master has the IsBranch() "flag" set to true. Others are false. I doubt that the only way to identify branches is to search for string prefix refs/remotes/origin (where origin should not be special at all, and is just the default remote name, right)? (options.go, line 13)
Perhaps we could write an example? (I can write a few examples once this is clarified)

So far, I've figured to:

Get a reference to a branch: ref, err := r.Reference("refs/remotes/origin/BRANCH_NAME_HERE", true)
Then, to get a commit: commit, err := r.Commit(ref)
*retrieve and do something with files: commit.Files() and files.ForEach

Is this a "correct" way of doing things? It seems pretty awkward to hardcode "refs/remotes/origin/" in there to find and define branches?
The git ls-remote command returns refs/heads/master and refs/heads/develop for branches, for instance.
I'm not sure whether this is something arbitrary that each client chooses or if it is something that is universal across git? (I see the same difference when comparing git vs go-git on custom gitolite or Bitbucket, for instance)
As far as I see right now, the way to identify branches would be to list references and then find all of the ones that have the prefix refs/remotes/origin (or even replace origin with the name of the (first?) remote)?

@skyflyer Thanks for taking the time to open the issue. go-git behaves the standard way in this case.
refs/remotes/origin (or refs/remotes/<remote name>) is the default way to store remote references locally. So if you want to list the references that were fetched from a given remote, that's the way to go (both with git and go-git).
git ls-remote is quite different. It does not operate on the local repository, but actually asks the server to get the avaiable remote branches. So it requires network operations. If you get a reference name from git ls-remote, such as refs/heads/develop and you want to operate with it in your local repository, you'll need to translate the remote name (refs/heads/develop) to the local name. Assuming you were using default configuration, the local name will be refs/remotes/origin/develop, but it can be changed with the fetch refspec (per-remote configuration).
There is no high-level API for git ls-remote in go-git, but you can achieve the same behavior with the low-level API for the git protocol: https://godoc.org/gopkg.in/src-d/go-git.v4/plumbing/transport/client
ep, err := transport.NewEndpoint(url)
c, err := client.NewClient(ep)
s, err := c.NewUploadPackSession(ep)
ar, err := r.AdvertisedReferences()
refs := r.References()
err := s.Close()

However, the recommended way of listing local branches is as you said in your comment. You might have some slight variations, such as using r.References() to get a reference iterator and call IsRemote function on each reference to check if it's a remote reference or not, but in the end, IsRemote is just going to check the refs/remotes/ prefix.

Closing this issue, since it's standard git behavior.

@smola, thank you for your explanation and a client sample code to list remote refs. It is good to know I'm on the right track.


GROOVY-8171: Escaped dollar slashy difference between old and parrot …

…parsers

+1

+1, good catch


The EnsoGL Component abstraction with special dropping behavior

[ci no changelog needed]
Pull Request Description
In this branch:

The workaround for cursor-not-being-updated-after-closing-searcher bug (discovered while testing #3278) is reverted.
The proper fix was introduced: created an abstraction for EnsoGL component, which, when dropping, will not immediately drop the FRP network and model, but instead put it into the Garbage Collector. The Collector ensures, that all "component hiding" effects and events will be handled, and drops FRP network and model only after that.
I run clippy for wasm32 target out of curiosity. There was one warning, and I fixed it on this branch.

Important Notes

Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the Scala, Java, and Rust style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH ./run dist and ./run watch.

Not an in-depth review, just a quick note: we already have such an abstraction in lib/rust/ensogl/component/gui/src/component.rs, so I suggest we should unify them and not have two different attempts at implementing this.

Not an in-depth review, just a quick note: we already have a Component abstraction in lib/rust/ensogl/component/gui/src/component.rs, so I suggest we should unify them and not have two different structures implementing this.

Oh, I was sure that this is the same Component. Hmmm, I was thinking that our main goal is to create in a single codebase as many structs with the "Component" name as possible, am I missing something? :D

Not an in-depth review, just a quick note: we already have a Component abstraction in lib/rust/ensogl/component/gui/src/component.rs, so I suggest we should unify them and not have two different structures implementing this.

Hmm, I tried to merge those, but run into one problem: The component in "enso-component-gui" crate assumes that every component should implement application::View and provide shortcuts. Nodes so far do neither of these. @wdanilo it is ok that the Nodes (and similar components) will be registered as application views, or it's too heavy?

@farmaazon, according to our chat just a moment ago: we need to think what the component abstraction is for. Your component abstraction is the glue between FRP, model, and logic of the component. Michael's component is also an application view with shortcuts. Components will be small, like widgets on nodes (drop-down menus), they should not be application views, because it's just not their responsibility to register shortcuts. So these are 2 abstractions and one should use another one.

Applied suggestions. As some changes are non-trivial, feel free to re-review this branch


Record users agreement to GDPR privacy notice

David Bernick wrote:
we can (and should) collect “real” logs as much as possible. The GDPR stuff means:

all users should sign ToS and make sure we keep a record that they signed that. And that in that ToS/Privacy Policy, we explicitly state what kinds of Identifiable info we have in logs (username, IP, parameters)
Policy states how long we keep stuff
Policy states these are ONLY used for operational puporses
Users EXPLICITLY OK the ToS and we keep a record of it. So users have to Auth first and then accept ToS.

Article 6, Paragraph 1, Point F

"The processing of personal data to the extent strictly necessary and proportionate for the purposes of ensuring network and information security, i.e. the ability of a network or an information system to resist, at a given level of confidence, accidental events or unlawful or malicious actions that compromise the availability, authenticity, integrity and confidentiality of stored or transmitted personal data, and the security of the related services offered by, or accessible via, those networks and systems, […] by providers of electronic communications networks and services and by providers of security technologies and services, constitutes a legitimate interest of the data controller concerned. This could, for example, include preventing unauthorised access to electronic communications networks and malicious code distribution and stopping 'denial of service' attacks and damage to computer and electronic communication systems."

Recital 49 (excerpt)```

I mostly agree with the HCA DCP GDPR 2-pager. At least Broadly. The only thing I’d add beyond the banner is an explicit “click thru” ToS/Privacy policy with an attached authenticated user.

Hi @theathorn, let's discuss an approach for a backend for this.

Obsolete. Agreeing to T&C is required to log in.


Fixed property setter in G6DOF joint

This is maybe the first on-line PR that I've created in my life, glad it looks good.

@m4nu3lf Just to say, don't worry if it isn't merged fast, as @akien-mga is a bit away right now (just to let you know).


Message.decode should not use ioutil.ReadAll

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.
Sarama Version: 1.8.0 (4ba9bba6adb6697bcec3841e1ecdfecf5227c3b9)
Kafka Version: <IP_ADDRESS>
Go Version: 1.6
Problem Description
Certain configurations hit a codepath in message decoding that uses ioutil.ReadAll() (Message#L172) (Message#L193). This is problematic in high performance situations with large enough messages, because internally ioutil.ReadAll() reads data out into a buffer that it grows as needed.
In the internal bytes.Buffer, growing is done via calls to bytes.makeSlice. See the attached screenshot for my memory profiling. Allocating new slices takes up about 65% of my memory footprint, and I think this code path has room for optimization.
My first thought is to create a right-sized buffer to read into instead of using ioutil.ReadAll

My first thought is to create a right-sized buffer to read into instead of using ioutil.ReadAll

As far as I know there is no way to determine what the right size is in advance, as it is not exposed in the Kafka wire protocol or in the gzip format.
For the LZ4 case it seems like it might be possible, but handling concatenate streams correctly sounds complicated (https://godoc.org/github.com/pierrec/lz4#Reader.Read).

Maybe a configuration value can be exposed to set a starting buffer capacity?

Yes, that sounds possible by initializing a bytes.Buffer and using its ReadFrom method.
Although looking at your graph above, the majority of the memory usage from this path is already happening due to the largest slice size, so cutting out the smaller slices will not save you a great deal.

Would it not be possible to keep the []byte for m.Value in a sync.Pool? If Message would get a Free() method that releases m.Value to this pool it could be reused. If people don't call m.Free() it will just always allocate like it does now.
Other libraries do this as well such as: https://godoc.org/github.com/go-mangos/mangos#Message.Free

I really like that idea @erikdubbelboer, my only concern is the existing direct access to m.Value. If users pass that byte slice around and aren't careful about using Free() they could read bad data out of it.
Maybe this is okay with sufficient documentation on the function?

The other complication with a pool is that the Consumer type does not return Message objects directly, so there would need to be some way of proxying the Free call from the consumer back to the underlying Message.
That said, it wouldn't be impossible.

You mean ConsumerMessage? It could just share the same []byte pool as Message and .
At the other side the Encoder interface could be extended with a Free() method that sarama will call when the value isn't needed anymore. This way people can implement their own Encoder types that reuse buffers as well.

@eapache to address your earlier point, yes the memory footprint is mostly used by the larger slices. However, some of my highest CPU usage is in runtime.mallocgc and runtime.heapBitsSweepScan.
Reducing allocation/GC pressure should improve throughput even if the memory impact is minimal.

I'm not against any of these enhancements, but I'm not going to be tackling any of them myself in the near future. If somebody wants to PR them with appropriate benchmark results backing them up, I'd be happy to take a look.

I looked into adding the Free() method but there are so many separate places where the []byte for the value can be created that I couldn't really find a nice way to implement this.


Add a page comparing languages targeting UPLC

Closes https://github.com/IntersectMBO/plutus-private/issues/48

Very well written!

+1


$ and : are not replaced

$a[tab] => $$asd
:f[tab] => ::form

Fixed by https://github.com/steelbrain/AutoComplete-Hack/commit/ba241b8be386805f3f07af985d3ba3b48fd010c1


Scheduled Jobs

Hello!
I was wondering if Sneakers supports scheduled jobs via delayed queues, or plans to do so.

@lucaong I've used your blog as a guideline and can confirm it works perfectly!

Thanks for the feature request @shaqq and the suggestion @lucaong.
Another option is using in combination with sidekiq (requires Redis as well) which does have this out of the box.
I don't think we'll support this out of the box right now so closing.


Memory leak? (MacOS)

I've noticed that simple program watching for clipboard and printing output starts to consume a lot of memory after some time.
Example of the code:
use cli_clipboard::{ClipboardContext, ClipboardProvider};

fn main() {
    let mut clipboard = ClipboardContext::new().unwrap();
    loop {
        let current_text = match clipboard.get_contents() {
            Ok(text) => text,
            Err(error) => {
                println!("error when trying to get text {}", error);
                String::new()
            }
        };

        println!("{}", current_text);
        std::thread::sleep(interval);
        std::process::Command::new("clear").status().unwrap();
    }
}

Just copy some text (bigger is better to immediately see the issue) and check Activity Monitor.
I do not have much experience in Rust and it definitely may be due to lack of knowledge from my side but here I see no options for improvement.

This is not a problem of your code as I looked at the macOS implementation and found out that there is indeed a memory leak problem in the get_contents function:
https://github.com/allie-wake-up/cli-clipboard/blob/545878acf67047c699a52c8f37a9dbb978c6c536/src/macos_clipboard.rs#L51-L58
Here the function asks objc to create an NSArray<NSString> for text in clipboard, and then uses Id::from_ptr to take ownership of that part of the memory, and then we never release this NSArray<NSString>, which causes a memory leak.
I modified the get_contents to use NSAutoreleasePool  and verified that the memory is no longer increasing forever.
    fn get_contents(&mut self) -> Result<String> {
        let string_class: Id<NSObject> = {
            let cls: Id<Class> = unsafe { Id::from_ptr(class("NSString")) };
            unsafe { transmute(cls) }
        };
        let classes: Id<NSArray<NSObject, Owned>> = NSArray::from_vec(vec![string_class]);
        let options: Id<NSDictionary<NSObject, NSObject>> = NSDictionary::new();
        let content: String = unsafe {
            let autorelease_pool: *mut Object = msg_send![class("NSAutoreleasePool"), new];
            let obj: *mut NSArray<NSString> =
                msg_send![self.pasteboard, readObjectsForClasses:&*classes options:&*options];
            if obj.is_null() {
                return Err("pasteboard#readObjectsForClasses:options: returned null".into());
            }
            let result:Id<NSArray<NSString>> = Id::from_ptr(obj);
            let mut result_string: String = "".to_string();
            if result.count() == 0 {
                result_string = "".to_string();
            } else {
                result_string = result[0].as_str().to_owned();
            }
            let _:() = msg_send![autorelease_pool, release];
            result_string
        };
        if content.len() == 0 {
            Err("pasteboard#readObjectsForClasses:options: returned empty".into())
        } else {
            Ok(content)
        }
    }

At this point probably we want to use alternatives listed in the readme instead of using this crate. For example arboard which not only handles the memory correct, but also handles some non-text content like images.


Replace internal RxSharedPreferences with library

I kicked the CI build again to see if it was a flake.

Thanks!


Merge --url, --export, --file into a single flag (--schema)

Discussed here: https://github.com/dotansimha/graphql-code-generator/pull/177#issuecomment-373277277

My project needs to to generate code based on a function that returns Promise<GraphQLSchema>. Thus non of the existing options work for that use case and the only way is to fork/rewrite codegen-cli.
I was thinking of introducing a --schema-loader option.

I think maybe the current --export option could support Promise<GraphQLSchema>. It could check if the exported result is a Promise and then resolve it, otherwise just use it as-is.

Done in 0.9.0 :)

Nice work!
Perhaps the usage section of the README needs to be updated to reflect these changes? It is still showing examples like this one:
gql-gen --file mySchema.json --template graphql-codege   .....


[REQUEST] [PSVITA] Moshi, Kono Sekai ni Kami-sama ga Iru to Suru Naraba

I'd like to request a script for Moshi, Kono Sekai ni Kami-sama ga Iru to Suru Naraba game. Thank you in advance!

request please!

+1!

I'm trying to do it myself but it's so hard :( Can't use CE...I've tried everything!

+1

I'm trying to do it myself but it's so hard :( Can't use CE...I've tried everything!

It wont work with CE because the engine is unity which means it uses a method to store the text

I'm trying to do it myself but it's so hard :( Can't use CE...I've tried everything!

It wont work with CE because the engine is unity which means it uses a method to store the text

Oh so this why. So in the end it is impossible to hook and make a script for this game in the end :(

Join the discord script related discussion mostly happen there


updated plot1DScan and plotImpacts with cmsstyle

Made plot styles (for plot1DScan.py and plotImpacts.py) dependent on cmsstyle package, to be able to propogate any recent changes in official CMS recommendations. Precisely changed plotting style, legend calls and colors.

what happens if CMS style is not installed?
we should make sure that the scripts works also w/o it or with some other custom choices.

Indeed, is this should stay open until this question is resolved. But we discussed having cmsstyle included in the future CMSSW releases, and add install cmsstyle to the combine container environment

Requests from the CAT meeting today:

check if cmsstyle is available, otherwise use to gStyle (previous implementation)
add luminosity label to the plot1DScan.py
change the Y axis label to q and/or add a command line option, defaulting to q
add an option to control CMS logo position (inside or outside the plot)


Add JSON support

1. Description
It would be nice to directly access JsonElements / JsonObjects properties and use them in queries. (for simplification I will only use JsonElements from System.Text.Json in the following issue. But Newtonssoft.json is important too)
So that you can easily do something like:
var jsonDocument = JsonDocument.Parse(@"[
    {
        ""first"": 1,
        ""City"": ""Paris"",
        ""third"": ""012-04-23T18:25:43.511Z""
    }]");
jsonDocument.RootElement.Where("City ==\"Paris\"");

Currently, it will tell you that the JsonElement doesn't contain the property City. What is true but also not.
The correct C# linq Query would look like:
jsonDocument.RootElement.Where(e => e.GetProperty("City").GetRawText() == "Paris");

It would be nice to have a default translator for JsonElement to "normal" dynamic linq queries.
It would be especially useful when the Expression and the JSON is not defined.
I'm aware of the issue #415. He showed a way how you could first convert the JsonElement to a dynamic type. But that would mean to create always a new dynamic type if you do not know the type. And technically I know the type, but the type is not nice to query with Linq. (JsonElement) This solution makes it also impossible to cache the compiled query, because I don't know the JSON structure and that means all the time I need to change the input parameter of my expression, which forces a recompile.
So for me the best solution would be if dynamicLinq could detect that the starting parameter of the expression is a JsonElement and then restructur internally the Expressions to the correct format.

Thanks for the idea.
A way to implement this is to create projects which depend on System.Text.Json and Newtonssoft.Json which provide extension methods like Where on JsonDocument and JObject.

@StefH you mean something like:
public static IQueryable<T> Where(this IQueryable<T> source, string expression){
   // TODO: parse expression
  // Rebuild all Property access expression to access `GetProperty(propertName)`
  // return result
}

More like this:
public static IQueryable<T> Where(this JsonDocument source, string expression)
{
}

Or
public static IQueryable<T> Where(this IObject source, string expression)
{
}

but shouldn't the base of the Extension method be something enumerable?
Also a JsonDocument can also be {}, that would result in an error I guess.
Shouldn't it be more like:
public static IQueryable<ArrayEnumerator> Where(this ArrayEnumerator source, string expression)
{
}

Then you can access over Current the current JsonElement and test the expression.
and on the user code side it would look like:
var jsonDocument = JsonDocument.Parse(@"[
    {
        ""first"": 1,
        ""City"": ""Paris"",
        ""third"": ""012-04-23T18:25:43.511Z""
    }]");
if(jsonDocument.RootElement.ValueKind != JsonValueKind.Array){
   throw new Exception("Must be an array");
}
jsonDocument.RootElement.EnumerateArray().Where("City ==\"Paris\"");

That would also allow us to use the extension method to use it inside of a JSON document.

Hm I'm also not sure if the extension of Where is the correct idea. If I understand it correctly that would mean that we need to reimplement the ExpressionParser for it. Wouldn't it be better to allow, that the currently private methods of the ExpressionParser can be overwritten? In this case the ParseMemberAccess method. Then we could create kinda easy second project and make a special implementation of ParseMemberAccess for JsonElement.
But maybe I don't understand the project structure well enough jet.

#791 WIP...

Looks quite good. Incredible how fast you implemented that 😯
So you decided to create DynamicTypes out of each element in the JSONArray and then query on it.
I guess because it's the easiest way todo right now. Nice to see.

I'm adding all methods and also working on System.Text.Json implementation. Can you take a look at the PR (WIP)?

Yes I will review today :)

Thank you @StefH for your work! That's amazing that it is done!

I still need to upload the 2 nuget packages...


[µTVM] Avoid listing links when probing serial ports

SerialTransport.open() probes automatically the device name based upon a
grep regex if a device name is not provided. The code expects to find only
a single device. Currently when it probes for the available serial ports it
includes in the list the device names that are also symbolic links.
Since _find_openocd_serial_port() always returns a serial number for a
given serial port (not the device name path) the available device names
are always probed when the openocd flash runner is used.
It's not uncommon that device drivers create symbolic links for certain
kinds of serial devices, specially those that provide a serial port plus
an additional endpoint to program the device attached, like a ST-Link
interface, etc.
As a consequence the current code fails to select the correct device name
when symbolic links exist and the openocd flash runner is used.
That commit changes the probe behavior to avoid listing symbolic links when
probing the device name for the target serial port.
Without that change the following error happens:
Traceback (most recent call last):
  File "./micro_tflite.py", line 255, in <module>
    with tvm.micro.Session(binary=micro_binary, flasher=flasher) as session:
  File "/home/gromero/git/tvm/python/tvm/micro/session.py ", line 127, in __enter__
    self.transport = TransportLogger(
  File "/home/gromero/git/tvm/python/tvm/micro/transport/base.py", line 79, in __enter__
    self.open()
  File "/home/gromero/git/tvm/python/tvm/micro/transport/base.py", line 207, in open
    self.child.open()
  File "/home/gromero/git/tvm/python/tvm/micro/transport/serial.py", line 72, in open
    raise SerialPortNotFoundError(
NameError: name 'SerialPortNotFoundError' is not defined

When ARM `STM32 STLink` is used by a board and the following device names are created:

gromero@gromero0:~/git/tvm$ ls -l /dev/{ttyACM0,stlinkv2-1_0}
lrwxrwxrwx 1 root root         7 Jan 12 16:10 /dev/stlinkv2-1_0 -> ttyACM0
crw-rw-rw- 1 root plugdev 166, 0 Jan 12 21:32 /dev/ttyACM0

Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

@areusch  Could you please review? Thanks!

@areusch  Could you please review? Thanks!


Addition of function get_events_detail

As mentioned in Slack, I have created a new function to scrape further details from the Bath Events pages such as Event time, Event location, Location address etc....  Hopefully the returned DF can be of use in providing a richer feature set to any further models.
I have tried to keep the new function code and remarks in keeping with the existing functions in  Web_Scraping.R however if any changes are required please drop me a line.  Also  I have used the library Rvest for my scraping however I was unable to work out how to add it to the library manifest (if that's the correct term)

As per commit messages above, I identified a bug with the new function when running outside the work space I originally developed in (i.e. the function used a cached work space variable ).

Sorry, I got caught out by the Workspace variables again!  (Now I have found the rm(list=ls(all=TRUE))  command this shouldn't happen again) . I have committed a change to my pull request which should resolve the issue.


openscad: 2018.04-git -> 2019.05

Motivation for this change
OpenSCAD 2019.01 is going be released any day now.
I'll update and merge this PR when the final 2019.01 version is released.
cc maintainers @the-kenny @raskin @bjornfor
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build openscad
@GrahamcOfBorg eval


[BUG] sozo typescript binding issues

Describe the bug
Some types and functions on pistols are not being ported to typescript correctly with sozo build --typescript
To Reproduce

clone pistols
uncomment binding generation on the migration script
run ./migration
check files at ./bindings/typescript
(generated files are attached below)

Issues and Screenshots

contracts.gen.ts: enum Archetype not being exported

#[derive(Serde, Copy, Drop, PartialEq, Introspect)]
enum Archetype {
    Undefined,  // 0
    Villainous, // 1
    Trickster,  // 2
    Honourable, // 3
}

Block-scoped variable 'U256Definition' used before its declaration. Same on ScoreDefinition, ShotDefinition,

ByteArray type mapped to invalid RecsType.StringDefinition, on origami's ERC20MetadataModel and ERC721MetaModel

Array<felt252> of actions-validate_commit_message  argument mapped to invalid RecsType.BigIntArray type

    fn validate_commit_message(world: @IWorldDispatcher,
        account: ContractAddress,
        signature: Array<felt252>,
        duelId: felt252,
        roundNumber: felt252,
        duelistId: felt252,
    ) -> bool;

balance_of() method, from origami's erc20_balance and erc721_balance has duplicated argument account. The same happens to other functions that include an account argument in contraxcts.gen.ts.

Additional context
Dojo 1.0.0-alpha.9
generated sources: pistols.gen.zip

hello @rsodre kindle assign me this issue

I am interested in debugging this issue @rsodre .

I'm not in charge  of assignments on dojo

@rsodre  Thanks for the update. Please who's attention should be draw concerning this task

@rsodre @Agbeleshe this is tackled in #2501 which should be significantly different.


Update instructions

Before I think of using this, I would like to know how I can maintain the newest updates from the repository.
Should I fork the repo and merge every time the updates of this repo to my repo or is there an other update mechanism I don't know about? If yes, you could add this to the readme :).
Thanks in advance.

Great idea! I'll have to write something up on that.
I'm going to make some bigger updates soon (now that the holidays are over and I have more time), and I'll try to keep a CHANGELOG going for bigger changes then on.
Ideally, I'd like to have the project be a drop-in replacement, where you pull-down the latest, and simply replace the Client & Server folders from the new version you pulled down, (with your current ones). I would think this would work for changes that happen with the infrastructure of the App. Let's say new additions to webpack / build / testing configuration etc.
I'll need the Changelog so I can note other potentially important changes/updates I make, maybe ones that showcase newly added features to the Client app itself, etc. So that you can have an easier time differentiating between them.
But I'm glad you brought this up, I was actually thinking about this problem the other day :)


Module fix

sorry, this is out of date and looks like no one looked at this back then.
i just made a commit in this order here: https://github.com/torch/image/commit/f7d40bf646abea47d083799107587d59bf021ea4


.NET Generic Host Implementation

This PR introduces various features from the Microsoft.Extensions.Hosting NuGet package.
It also introduces a few minor bug fixes.
ToDo:

[x] Handle config file errors (parse error, missing file) - done-ish
[x] StopApplication on Authorization Error (wrong token etc.) - can't catch most of the exceptions in StartAsync(), as they're already being caught by the Discord .NET library, so only token validation and some other fatal exceptions are caught

Closes #28

Forgot to log in to my github account 💀

Not signed since I committed it from my phone
Gonna sign (by rebasing) when i get home


OPSWAT Filescan: New App - Initial submission

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[x] Testing of all the changes has been performed (for bug fixes / features)
[ ] The readme.html has been reviewed and added / updated if needed (for bug fixes / features)
[x] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[x] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[x] Verify all checks are passing.
[x] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[x] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[x] New App
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the readme.html.
[ ] If this is a new connector or you are adding new actions

[x] Please document in the readme.html all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the readme.html.

[ ] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[x] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[ ] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.

What is the current behavior? (OPTIONAL)

Describe the current behavior that you are modifying.

What is the new behavior? (OPTIONAL)

Initial release with the following actions

detonate file
detonate url
search terms
file reputation
ip reputation
domain reputation
url reputation
test connectivity

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

Hi,
I have two remaining static test failure, but I don't know how can I solve it.

The one is the missing appid:

"Could not find an app id for opswatfilescan. Please add the app id for opswatfilescan to data/repo_name_to_appid.json"
Exactly where should I enter my app id?

The other one is the "File manipulation functions exist"
I need to use file handling since I have a file detonation action, but I use vault_id for that. (That's what the documentation mentioned.) Did I miss something?

Thank you!

Hi @anikobartos , those issues will get taken care of when the PR is reviewed. Thxs!

Hi @anikobartos , those issues will get taken care of when the PR is reviewed. Thxs!

Thank you @mpan-splunk ,
Could you please provide information regarding the procedure and timeline for reviewing pull requests? The timely publication of this integration holds significant importance for us. I would not like it if the acceptance was delayed because of me, and I would like to correct the errors as soon as possible if this is necessary.

Hi @anikobartos
We've started with the PR review.
We'll take care of the remaining pipeline errors. Thanks for contributing!
We'll keep you posted on the review progress.

Thank you @ishans-crest! :)

Hi @ishans-crest,
Any update? :)

Hello @anikobartos
We have stopped explicitly adding dependency for requests module , so could you please remove it from requirements.txt?
Thanks!

Hello @anikobartos,
We have updated the data_type for 'password' parameter in 'detonate file' and 'detonate url' actions from password to string as SOAR action parameters do not support password data_type and actions will fail in that case. However, if you do not want the password parameter to be simple string, you can move it to asset configuration parameters and keep it optional.
Also, could you please test the app and let us know if it works for you?

Hi @dvora-crest,
Thanks for the correction! I tested it. It looks fine, and working as I expect.
The password parameter means the password of a zip file, so moving it to the asset configuration is not logical.
So it's fine with simple string type.

Hi @tonyc-phantom,
I see that several people (included you) have already accepted my pull request. My question is, do you know when it will be merged? Furthermore, should I take any additional steps or provide any further assistance?"

Hello @anikobartos, No further action is required from your end. Thanks.


Added support for multiple senders.

added support for multiple senders. Instead of having 1 pair of sender_id, server_key set up in the .env configuration file, we can have multiple sets saved in the database. If your applcation has multiple clients/users, and they send their own notifications, from their own (sender_id + server_key pair), this will support that without "hurting" the original way of just one sender.

Coverage decreased (-1.02%) to 92.982% when pulling 606c2a8cb95c82dcf8e88b2a313267805d867ba0 on henrique1977:multiple-senders into 2122ba0fda51fbec87a14a1a9dc7e292f10b26df on brozot:master.

I'm interested in this getting merged too. If it helps any, I'm willing to jump in there and tweak it until you're happy with it. What needs doing?

Yes a merge would be great! Let me know if I can help as well.

s

Hi !
Feel free to re-open this pull-request to code-lts/Laravel-FCM
This is a fork that will be maintained as long as possible by @code-lts members :muscle:
:rocket: 1.4.0 was released See how to install it (composer require code-lts/laravel-fcm)


[wip] add synch action getTables

This PR is to provide the frontend with useful information about the DB in order to improve UX by allowing users to select tables/columns without having to write a query.

@pivnicek can you please describe what is the purpose of this PR and link it to some issue and PB feature.

Sure!
Issue 1) https://github.com/keboola/db-extractor-common/issues/9
PB 1)  https://keboola.productboard.com/feature-board/6271-planning/features/368835
PB 2) https://keboola.productboard.com/feature-board/6271-planning/features/268346

I'll update the description ^^ now

great, thanks.

Not at all, should have been done from the start, thanks for the push:)

ok, so now the first part sync action is done in this PR. and the second write a metadata manifest  will also be part of this OR?

They are now together, but I can remove the manifest portion and supply it separately if preferable.

no it is ok, It wasn't just clear for me if there are  only sync actions.


unknown-unknown index.html: Export math functions LLVM/Rust needs

These functions are required by the current implementations of several math functions in Rust as well as by LLVM and provided only in Rust's etc/wsam-shim.js.
See also the files at
https://github.com/rust-lang/rust/blob/41e03c3c469d1c89735fa518a9af4eb3df8b0728/src/etc/wasm32-shim.js,
https://github.com/rust-lang/rust/blob/41e03c3c469d1c89735fa518a9af4eb3df8b0728/src/libstd/sys/wasm/cmath.rs and
https://github.com/rust-lang/rust/blob/41e03c3c469d1c89735fa518a9af4eb3df8b0728/src/libstd/sys/wasm/stdio.rs
(which contains the comments due to which I chose not to add the stdin-/out functions)

Later on I'd like to only emit these when the compiled crate is actually using them, but I guess this is fine for now.
Thanks!


beaker 1.3.4-4-g37d7759

Thank you for the contribution. It was merged directly as 8eb4188bd96670602805bd644f898fb4dddeed12 to keep commit history cleaner. Your contribution is still credited to you.


Store settings in XML file

Instead of seperate files, the settings should be stored in a single XML file.

I have considered JSON instead of XML, but XML is supported out of the box on Dotnet Core/Standard.


How to update all migration file?

Laravel Version: 5.4
Voyager Version: v0.11.12
PHP Version: 7.1.5

Description:
When I created a new table like Products with Tool > Database in Voyager.
I want to create a migration file sync with this.
Because when I copy source to another computer. I must run the command: php artisan voyager:install --with-dummy.
It does not create table Products because don't have migration file in the folder: database/migrations/.....
Thanks.

Hi @vanloc0301
You have to create your own migration files in order to accomplish that, Voyager is not generating the migration at the moment :(

Thanks @akazorg

You can use https://github.com/Xethron/migrations-generator to generate database ;)


Vulkan: Random background color at each frame in 3D editor on macOS (Intel GPUs only)

Bugsquad note: This issue has been confirmed several times already. No need to confirm it further.

Godot version
4.0 alpha 1 official
System information
MacOS 10.15.7, CPU: 2.7GHz dual core i5, GPU: Intel Iris Graphics 6100 1536 MB, Vulkan rendering backend
Issue description
When I rotate my view in the 3d editor, the background color changes at every frame. The following video shows that.
(epilepsy warning)
https://user-images.githubusercontent.com/9269185/151323538-4b88f5a1-65da-4689-828c-7b6fb495b9ae.mov
Additional note: color always starts with white, and when I use another editor and then come back to the 3d editor, the color restarts to white as well.
Steps to reproduce
Just create a new project and go into the 3d editor
Minimal reproduction project
No response

Does not seem to happen on Macs with M1 Chip.
Tried it with both Vulkan Backends.

@Nebulean Can you reproduce this after switching the rendering backend to Vulkan Mobile in the Project Settings?

I switched to "Forward Mobile" in the project settings, if that's what you meant.

The results are fairly similar, the only noticable difference being the fact that the grid in the x-z plane changes color as well.
(EPILEPSY WARNING)
https://user-images.githubusercontent.com/9269185/152408475-6690b64f-595c-4368-a894-e35db79d3ea2.mov

same issue on MacbookPro with intel GPU

Confirmed on 4.0 beta 2
MacBook Air 13, Early 2015
Intel HD 6000
macOS Monterey 12.2.1
It seems to be related to the skybox shader as setting the background mode to solid color solves the flickering issue but setting it to black actually shows a red background, brighter colors slowly get more correct.

so I guess I figured out what caused this issue, it seem the DirectionalLight3D, at least it looks like that
https://user-images.githubusercontent.com/64188282/162567259-37998a27-c298-4cee-b161-636004e41398.mov

I can reproduce this on alpha7, Late 2015 Macbook Pro, Intel Iris Graphics 6100.
Same fix; toggling preview sunlight fixes it.

this issue has been fixed

I can't reproduce this issue either on an early 2015 intel macbook Intel Iris Graphics 6100. So I will close as fixed.
If anyone can still reproduce this issue, please let me know


TOML allows large-magnitude integers that can't be represented exactly as JS numbers

The TOML spec says that numbers can lie in the range of 64-bit signed integers.  This presents a problem for JavaScript implementations of TOML, since the entire range of 64-bit signed integers can't be represented exactly by JavaScript.  In particular, only the closed range of integers [Number.MIN_SAFE_INTEGER, Number.MAX_SAFE_INTEGER] can be represented exactly by the JavaScript number primitive.
To demonstrate that this is a problem for toml-node, check this out:
> require('toml').parse('max_safe =<PHONE_NUMBER>740991\n' +
    'max_safe_plus_two =<PHONE_NUMBER>740993')
{ max_safe:<PHONE_NUMBER>740991,
  max_safe_plus_two:<PHONE_NUMBER>740992 }

There are a few ways to deal with this problem (parse large ints as strings, depend on a BigInteger library for large ints, throw RangeErrors, among others). I don't have a strong opinion on which option is best, but I do think the library should address this issue in some way!

Er, my original report didn't actually demonstrate the problem! Edited the issue to give a better example.

Thanks for the report, @tomjakubowski. I agree that the library should do something other than just giving the wrong data back. I'll think on this, and am open to suggestions.

After thinking about this, I think my inclination is to throw a RangeError if numbers fall outside the safe range. Number.MIN/MAX_SAFE_INTEGER can't be used in all browser environments so I think it'd need to be defined in the lib.
However, I'm on the fence about whether or not it'd be better to output some sort of warning instead, or allow an option for ignoring numbers outside the safe range. I could imagine a scenario where consumption of the number may not be important, but other parts of the TOML file are.
@mojombo, I'm curious if you have any thoughts on how a parser would ideally behave here

In my opinion, this should throw an error and be well documented in the project as a limitation and deviation from the TOML spec. I'm guessing it will be rare that a developer doesn't experience the error during development. In my mind, silently returning an incorrect value is the worst kind of behavior and could lead to extremely frustrating bug hunting for some unlucky soul.

Hi, I also encountered this issue while parsing the config that contains snowflake id which is bigger than number maximum normally. I think we can wrap the number is above javascript number limitation with BigInt.


Split quotient_constants into two parts, avoiding computing line/random coefficients for each query domain

Close in favor of the dev-joe branch.


Update x/crypto to v0.17.0

What type of PR is this?
/kind bug
What this PR does / why we need it:
Update x/crypto to a newer version to try to resolve ssh issues.
Special notes for your reviewer:
This is essentially a backport of https://github.com/kubernetes/perf-tests/pull/2486 because of https://github.com/kubernetes/kubernetes/pull/124487.

/assign @marseel

/lgtm
/approve

/lgtm
/approve

/retest

/retest


Text field of label is too narrow

This program has a 2 column grid layout.  Each row has a radio button and a label.  The text displayed in the label is supposed to start at 1, and increment every second... 2, 3, 4, ... 9.  So far, all this works perfectly.  But the next second it displays 1 rather than the expected 10. 1 is displayed for the next 10 seconds, then 2 is displayed for 10 seconds, then 3, etc.
What appears to be happening is that the width of the label is only 1 character.  So this works for 1, 2, ... 9, but for 10, 11, etc. only the first digit is displayed.
I can work around this by padding the label text with blanks so it is always the same length, but it doesn't seem that I should need to do this.  In getting the padding working, I thought I'd only have to pad an empty string with 8 blanks, as this would then be the same string length as '00:00:00'.  Trying this, only about the first half of the string was visible "00:0".  It requires about 15 blanks to make sure that the entire '00:00:00' will be visible.  So it seems that the issue is not the length in characters of the label, but the width in pixels of the label.  (I just confirmed this by running on Catalina.  The font is different, and when it attempts to display "10", I see the "1" and just a few pixels of the "0".)
Thank you for any advice.
glimmer 2.5.1, glimmer-dsl-swt <IP_ADDRESS>, jruby <IP_ADDRESS>, JDK 16.0.2, x86_64 on Windows 7 Professional SP 1
glimmer 2.5.1, glimmer-dsl-swt <IP_ADDRESS>, jruby <IP_ADDRESS>. x86_64 on Mac Catalina 10.14.6
(file name changed from .rb to .txt)
text_field_width_issue.txt

Funny, I just updated the docs today because several people reported having your issue without realizing it is as simple to fix as adding layout options for fill space horizontally.
Here is the code with one extra line to specify grid_layout-related layout_data :fill, :center, true, false, indicating you want the widget to fill all available space horizontally:
require 'glimmer-dsl-swt'

class TaskTimer
  class Task
    attr_accessor :name, :elapsed_time

    def initialize(name)
      @name = name
      @elapsed_time = 0
    end
  end

  include Glimmer::UI::CustomShell

  before_body do
    @tasks = [Task.new('Test')]
  end

  body {
    shell {
      grid_layout {
        num_columns 2
        make_columns_equal_width false
      }

      @tasks.each do |task|
        radio {
          text task.name
        }

        label {
          layout_data :fill, :center, true, false
          text <= [task, :elapsed_time]
        }
      end

      @current_thread = Thread.new do
        loop {
          @tasks[0].elapsed_time += 1
          puts @tasks[0].elapsed_time
          sleep(1)
        }
      end
    }
  }
end

TaskTimer.launch

I just updated the docs again to make even clearer.
Here is my explanation under the Layouts section:
https://github.com/AndyObtiva/glimmer-dsl-swt/blob/master/docs/reference/GLIMMER_GUI_DSL_SYNTAX.md#layouts
Specifically, this bit:
"Note that if you need to have widgets fill a row_layout and resize automatically upon window resize, you must nest fill true within. This is the automatic behavior of fill_layout. For grid_layout, you would have to add layout_data :fill, :center, true, false to a child that you want to fill all the available space horizontally initially and after window resize.
Do not be alarmed if widget sizes were kept fixed on resize of a window. This is normal behavior that can always be overridden with options, such as fill true mentioned above. You need to learn more about each layout before you have mastered them enough to do anything you want with layouts."


Using RetinaNet

Hi, I am new to this toolbox and trying to use RetinaNet.
What I came up with is the following code:
backbone = ResNet(depth=50)
in_channels = [50, 25, 13]
neck = FPN(in_channels, 11, len(in_channels))
head = RetinaHead(11, 7)
model = RetinaNet(backbone=backbone,neck=neck,bbox_head=head)

However, it gives me the following error:

AssertionError                            Traceback (most recent call last)
 in ()
----> 1 model = RetinaNet(backbone=backbone,neck=neck,bbox_head=head)
/content/gdrive/My Drive/Developer/Repositories/mmdetection/mmdet/utils/registry.py in build_from_cfg(cfg, registry, default_args)
60         obj: The constructed object.
61     """
---> 62     assert isinstance(cfg, dict) and 'type' in cfg
63     assert isinstance(default_args, dict) or default_args is None
64     args = cfg.copy()

Can someone please guide on what am I missing?

Ok sorry, didn't know that I had to use the already available configs.


Roll Flutter Engine from a10990314872 to a38655743250 (4 revisions)

https://github.com/flutter/engine/compare/a10990314872...a38655743250
2023-03-31<EMAIL_ADDRESS>[Impeller] Declare gradient shader inputs as high precision to avoid overflows. (flutter/engine#40806)
2023-03-31<EMAIL_ADDRESS>Revert "Re-enable LTO and roll buildroot to enable sINLINING_LIMIT" (flutter/engine#40822)
2023-03-31<EMAIL_ADDRESS>[macOS] Change view ID to signed (flutter/engine#39958)
2023-03-31<EMAIL_ADDRESS>Roll Skia from 5f965db93be3 to 2b86c6d364d0 (4 revisions) (flutter/engine#40823)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


WorkItem Mapping : Move original ( pulled from remote )  items from Tracker table to WorkItems

Retrieve the persisted items from Tracker table.
Map it to a WorkItem.
Persist the WorkItem in the database.

Depend on #111

Hi @aslakknutsen  Wanted to discuss the design for this.
When would this operation ,ie, read + map + insert/update take place?
What/who should be invoking this?
Should it be triggered by a web service call ?

@sbose78 I think if the map/create workitem logic is called from here somewhere, as the items are being imported, that's fine for now: https://github.com/almighty/almighty-core/pull/111/files#diff-4909d119850b84d28b9ea5bf51b55fa7R45

@sbose78 Once I fix #193, you can assume the tracker items are unique irrespective of which batch the data got imported.


Can´t start NEXT SERVER

Unhandled Runtime Error
TypeError: Only absolute URLs are supported
lib\api.js (2:20) @ fetchAPI
1 | async function fetchAPI(query, { variables } = {}) {

2 |   const res = await fetch(${process.env.NEXT_PUBLIC_API_URL}/graphql, {
|                                ^
3 |     method: 'POST',
4 |     headers: {
5 |       'Content-Type': 'application/json',

Did you solve it? I created a .env.local file, but I got the same error.
(My OS is windows 10)

sovled, after I updated next verison to 9.4