Is it possible to have multiple Fi bases and collars?

When trying to set this up it only picked up one of our three dog collars/bases. Is it possible to add more than one?

Hi @tkuennen,
Can you tell me if its failing to pick up the bases, the collars or both?

Hi @sbabcock23,
My apologies for the delayed response. It's both the bases and the collars. Another odd bit is that sometimes when I restart Home Assistant it will sometimes disconnect from one and connect to the next. Almost seems to be doing round-robbin between them. We have three dogs with Fi and I'd love to be able to check in on all three with HASS.

I just set this up with two dogs/collars/bases and I'm seeing the same behavior as @tkuennen

I believe this to be a limitation with the component because while troubleshooting it would loop through our three collars however only one would be available at a time leaving the other two in an unknown state. I'm about to repave HASS and will do a bit of debugging on this. Will report back in Jan with what I find.

thanks @tkuennen

I got this to work with two collars

I added another entry of tryfi to my config and configured it with my girlfriends email on the second instance. So it was querying my email and my girlfriends at the same time. You could probably just create a second email account and then add it as an owner.

@ndragon798 nice find. This worked for me as well.

Any chance on a quick mult-dog (collar & base) for the cheap seats?
I added a 2nd collar / dog to my TryFi account and now having some strange issues. At this point I'm seeing:

light.dog1_collar_light
select.dog1_lost_mode
select.dog1_lost_mode
light.dog2_collar_light
select.dog2_lost_mode

I have confirmed I have the latest update in HACS.
Any suggestions?

Hi @tkuennen and @russalex ,
Would you be able to enable debug on HA and share the tryfi specific log entries? There should be api calls with the data in it and errors potentially. Please sanitize any data as required before sharing.
Thanks!

I believe this has resolved. Please let me know if it's still an issue.

Doh, I didn't see the request on May 1st. Sorry
The good news, all appears to be working now. The two dogs are showing up correctly.  Thanks!

I hate to necro an old issue, but I'm experiencing this bug again. I've set my logging mode for the TryFi integration to debug, and it doesn't appear to even be aware that anything is incorrect; perhaps it's an issue with PyTryFi? The below logs are from a fresh restart and clean installation of the TryFi integration.
For context around my use case, myself and my brother both have TryFi accounts, he has one for his dog Scout, and I have one for my dog Nala. We have set eachother as a "Dog Owner" of eachother's dogs as well, so I have visibility into Scout's collar and he has visibility/control into Nala's collar. Despite logging in with my own credentials (not my brothers), it only seems to be aware of Scouts existence.
Any thoughts?
2022-08-12 21:49:34.676 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration tryfi which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant 2022-08-12 21:49:34.677 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration smartthinq_sensors which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant 2022-08-12 21:49:34.677 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant 2022-08-12 21:50:00.365 INFO (MainThread) [custom_components.tryfi] Success Connecting to TryFi 2022-08-12 21:50:01.583 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.451 seconds (success: True) 2022-08-12 21:50:01.585 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Battery Sensor: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.585 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Pet Stat: Last Updated - 2022-08-12 21:50:01.535792 - Pet ID: 3QEOKC21tgd7vovoGkxQre Name: Scout Is Lost: False From: Carmel, Indiana ActivityType: OngoingRest Located: 39.9746875133562,-86.1987100169062 Last Updated: 2022-08-12 22:49:53.012000+00:00              using Device/Collar: Last Updated - 2022-08-12 21:50:01.415517 - Device ID: FC22C35073 Device Mode: NORMAL Battery Left: 72% LED State: False Last Connected: 2022-08-13 01:49:12.249000+00:00 by: ConnectedToBase 2022-08-12 21:50:01.586 DEBUG (MainThread) [custom_components.tryfi.sensor] Adding Base: Last Updated - 2022-08-12 21:50:01.250403 - Base ID: FB12C33719 Name: Calebs House Online Status: True Wifi Network: The Emperors Burger Located: 39.9746875133562,-86.1987100169062 2022-08-12 21:50:02.020 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.407 seconds (success: True) 2022-08-12 21:50:07.422 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.422 seconds (success: True) 2022-08-12 21:50:12.520 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.519 seconds (success: True) 2022-08-12 21:50:17.482 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.481 seconds (success: True) 2022-08-12 21:50:22.354 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.353 seconds (success: True) 2022-08-12 21:50:27.449 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.448 seconds (success: True) 2022-08-12 21:50:32.399 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.400 seconds (success: True) 2022-08-12 21:50:37.593 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.593 seconds (success: True) 2022-08-12 21:50:42.481 DEBUG (MainThread) [custom_components.tryfi] Finished fetching tryfi data in 0.481 seconds (success: True)

Version 0.0.15 just fixed the issue in my aforementioned comment, thanks @sbabcock23 for the fix! Issue was tracked in the PyTryFi repo
https://github.com/sbabcock23/pytryfi/issues/19


[Logs] Clarify availability of Workers logpush

Closes #7183
Adds a quick note about Workers Trace Events Logpush on the Logpush availability page, to avoid confusion since it is not dependant on plan level and relies on Workers paid. Also links back to the relevant docs pages for it.

Preview: https://erisa-workers-logpush-availa.cloudflare-docs-7ou.pages.dev/logs/about/

Open to suggestions on neatening/tidying this


OC-442 Deal with duplicate code Sonar issues

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractPushSetupDto.java#L155: Return an empty collection instead of null.
 AbstractPushSetupDto.java#L174: Return an empty collection instead of null.

All is well.

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractPushSetupDto.java#L155: Return an empty collection instead of null.
 AbstractPushSetupDto.java#L174: Return an empty collection instead of null.

All is well.

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractPushSetupDto.java#L155: Return an empty collection instead of null.
 AbstractPushSetupDto.java#L174: Return an empty collection instead of null.

All is well.


Bump galvan version

updated in a new pull request

https://github.com/Terracotta-OSS/terracotta-core/pull/1003


Please invite me to the GitHub Community Organization

Name
Sunil Gopalakrishna
Discord Username (if applicable)
No response
Additional Context
I met Eddie on Twitter, I love that he encourages how everyone irrespective of their ages can add value to projects.

it looks like you've joined the github org, maybe close the issue now?


#102 Automated unit test written for helper.py

Issue that this pull request solves
#102
Closes: # (issue number)
Proposed changes
The unit test is written running successfully
Brief description of what is fixed or changed
There are two functions in helper class for the kill process test is already written and for create dirs has been written in this PR.
Types of changes
test_helper.py file added in Tests folder
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings

Screenshots
Please attach the screenshots of the changes made in case of change in user interface

Other information
Any other information that is important to this pull request

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

Issues
======
- Added 5

Complexity increasing per file
==============================
- tests/test_helpers.py  1

See the complete overview on Codacy

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

Complexity increasing per file
==============================
- tests/test_helpers.py  1

See the complete overview on Codacy

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

Complexity increasing per file
==============================
- tests/test_helpers.py  1

See the complete overview on Codacy

@rpotter12 @xandao6 Please have a look at my PR.


ignoreext fails to properly ignore the file extention

Using the example config file makes checkrr report every .bif file as a bad file (whcich could be an improvement to add .bif to ignoreexts in the yaml, as I think the greater part of users have .bif files autogenerated for video files in plex, emby etc).
But the issue here is, that if I add .bif to the ignoreexts section in the config, they are still found by checkrr and treated as a bad file(ignore faulty indent. the yaml is valid).
my config:
ignoreexts:
- .txt
- .nfo
- .nzb
- .url
- .bif
radarr: ....
I've deleted the database and re created the docker between tests.

same for me for .xml files

I am on the latest version and still getting "unknown file detected" for all sorts of files that are defined in ignoreexts:

I am on the latest version and still getting "unknown file detected" for all sorts of files that are defined in ignoreexts
That's because 3.1.0 isn't released. This will be patched in that version.

Ah, okay. Thanks for all of your work.
Looking forward to it!


records: simpler files handling

Signed-off-by: Harris Tzovanakis<EMAIL_ADDRESS>

@pazembrz could you please be more specific on what functionality I'm missing? And why it looks weird and incomplete? ☺
About the doc strings, I believe in most cases function names and the code is clear enough what is doing and there is no need to explain it

@pazembrz updated

@pazembrz ping


Spanish translation

How can I contribute with the Spanish translation? Please let me know

This would be appreciated.
I made a google sheet that you could add translations to:
https://docs.google.com/spreadsheets/d/1OfqRXo9EwwJvvbkqHLD5ybjgSSQXVkkfYK_M1AewpjQ/edit?usp=sharing

Hi!
The translation is ready!
El vie, 10 may 2024 a la(s) 2:21 a.m., Tom Berghuis (
@.***) escribió:

This would be appreciated.
I made a google sheet that you could add translations to:
https://docs.google.com/spreadsheets/d/1OfqRXo9EwwJvvbkqHLD5ybjgSSQXVkkfYK_M1AewpjQ/edit?usp=sharing
—
Reply to this email directly, view it on GitHub
https://github.com/tberghuis/FloatingCountdownTimer/issues/25#issuecomment-2103945781,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWZFZJWHLDVYONMCZ7GNXQDZBRRPRAVCNFSM6AAAAABHPYVQJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBTHE2DKNZYGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


rip out subpredictors

there's some bug in the code that i can't find, and at the moment, it's adding quite a bit of complexity.
if we rip out subpredictors, it makes the code for ensembling easier, and we can reimplement from scratch another time.

done


Logger

What is this program called?
https://raw.githubusercontent.com/sailro/EscapeFromTarkov-Trainer/master/Images/process2.png

Process Monitor


editor.action.insertLineBefore does not fire onDidChangeTextEditorSelection

The VsCodeVim plugin uses insert line before and insert line after commands.
editor.action.insertLineBefore
editor.action.insertLineAfter
We then wait for the onDidChangeTextEditorSelection to fire to know that the operation was completed.
Now after running insertLineBefore, the event is NOT fired. As soon as you move the cursor from here it is, but not just when running the action, even though the cursor position changed

Is there a reason why this event stopped firing for these commands?
Is there a better way for us to know a command completed, we await and then how do we know the actual change occurred.

Just played around with it and find that when you run editor.action.insertLineBefore, we put a line above current line and move the cursor to the new line. If you compare the selection before and after, it does not change indeed. The only changed thing about the cursor is now the cursor is between new markers.


Don't add invalid paths to IMG_PATH

This happens to work today because directories are only one level deep
under images so "path" was always the same as "_root".
Also, rename variables to match os.walk's documentation.

Any update on this one @ccordoba12, @ericfrederich ?

We should merge this one, I see nothing wrong with this code.

Ok, merging then per @Nodd's wishes ;-)


peer-to-peer-code-review

you did a good job, the application is working fine. Still, there is an issue I found.

when I click for edit it is displaying to-do list two times.

try to refactor readme.md file with proper documentation.

Thank you @JakariaJishan for your feedback, I will work on the mentioned issues.


Adding Contribute section

Closes #113.

@clatapie :
Few suggestions, can we rewrite as below :
To ensure your code meets minimum code styling standards, run this code to install pre-commit:
pip install pre-commit
You can also install this as a pre-commit hook by running this code:
pre-commit install
Code styling standards check can be done by running this code:
pre-commit run --all-files


Add more logging to flaky request queue integration test

Every now and then, the integration test for request queue times out. This adds more logging to the test so that we can debug it easier.

@jirimoravcik Yeah, it's not the fault of the SDK, it's just the first GET http://api.apify.com/v2/request-queues/QUEUE_ID call that gets stuck, dunno why.


Resizable not working with version 0.16.3

I'm trying to get the demos to work, for example:
https://strml.github.io/react-grid-layout/examples/6-dynamic-add-remove.html
but unfortunately the resize icon is missing on the bottom so I can't resize.
I checked and I have the style.css in the react-grid-layout.
I also see it on the elements:
div class="react-grid-item react-grid-item react-resizable react-draggable ...
I also tried multiple examples and the resize doesn't work on any of them.
Please mark the type of this issue:

[X ] Bug
[ ] Feature Request
[ ] Question

you need to add the css files

Which css files? I see the styles.css that I have in my project under ...\node_modules\react-grid-layout\css

DO you have below css, when you inspect in dev tool
.react-grid-item > .react-resizable-handle {
        position: absolute;
        width: 20px;
        height: 20px;
        bottom: 0;
        right: 0;
        cursor: se-resize;
    }

    .react-grid-item > .react-resizable-handle::after {
        content: "";
        position: absolute;
        right: 3px;
        bottom: 3px;
        width: 5px;
        height: 5px;
        border-right: 2px solid rgba(0, 0, 0, 0.4);
        border-bottom: 2px solid rgba(0, 0, 0, 0.4);
    }
    .react-resizable {
        position: relative;
    }
    .react-resizable-handle {
        position: absolute;
        width: 20px;
        height: 20px;
        bottom: 0;
        right: 0;
        background: url('data:image/svg+xml;base64,PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHZpZXdCb3g9IjAgMCA2IDYiIHN0eWxlPSJiYWNrZ3JvdW5kLWNvbG9yOiNmZmZmZmYwMCIgeD0iMHB4IiB5PSIwcHgiIHdpZHRoPSI2cHgiIGhlaWdodD0iNnB4Ij48ZyBvcGFjaXR5PSIwLjMwMiI+PHBhdGggZD0iTSA2IDYgTCAwIDYgTCAwIDQuMiBMIDQgNC4yIEwgNC4yIDQuMiBMIDQuMiAwIEwgNiAwIEwgNiA2IEwgNiA2IFoiIGZpbGw9IiMwMDAwMDAiLz48L2c+PC9zdmc+');
        background-position: bottom right;
        padding: 0 3px 3px 0;
        background-repeat: no-repeat;
        background-origin: content-box;
        box-sizing: border-box;
        cursor: se-resize;
    }

Here is the element in the browser:
x
I checked the styles.css and I don't see the last 2 styles you have. Should I add them?
I got the code from here and this is the css file they posted:
https://github.com/STRML/react-grid-layout/blob/master/css/styles.css

I got it to work by adding the styles.css to my app:
import '../../node_modules/react-grid-layout/css/styles.css';
I would of think that it'll be automatic with the react-layout-grid import:
import { WidthProvider, Responsive } from "react-grid-layout";

It is not good practice to import from the node_modules, and as in the doc you should do:
import 'react-grid-layout/css/styles.css'

As what @jog39 said above, it did solve the problem. But the path begin with a tilde  helps resolve the css style from node_modules.
import '~react-grid-layout/css/styles.css'

Hey there, I'm having the same problem that the resize handler is not showing up on my components. I checked the source in the browser tools and the corresponding css rules are their. My components are also tagged with the right classes.
Any help would be appreciated.

Resize handler is received as children prop in your grid item. Render children inside your grid item and include css. it should work fine.
<RGL>
    <YourGridItem />
    <YourGridItem />
    <YourGridItem />
</RGL>

In YourGridItem's render just include {this.props.children}.

It is not good practice to import from the node_modules, and as in the doc you should do: import 'react-grid-layout/css/styles.css'

Thanks it helped me alot

Any idea for solving it using functional components?


Video Recorder doesn't work on mobile devices

Description
I have the video recorder working fine on desktop but, it won't record on phones.
I can click on the middle icon and it asks to use my camera. Then the camera launches and it just shows a still show with no options to record. Any help would be appreciated.

it won't record on phones.

you have to be more specific. And always include browser version, videojs-record version and your videojs-record config.

Videojs-record: version 2.4.1. I am trying it on a IPhone with 13.2 installed.
Here is my code:
``
Thank you

I am using Safari on my phone.

Sorry about the urgency. :)

Don't worry, I don't care about the urgency :)
Marking as a duplicate of #332

Sorry about the urgency. :)

Don't worry, I don't care about the urgency :)
Marking as a duplicate of #332

ya that's why its a shitty library with so much issues


Cryptsy public API bug fix for v3.0.0-SNAPSHOT, Kraken general enhancements, Trade Builder additions

re: CryptsyBasePollingService now that it contains both the public and private API does it really make sense for it to stay templated in its current form?
The Cryptsy fix was to stop the URL being generated as http://pubapi.cryptsy.com/api/api.php?method=orderdatav2
but instead http://pubapi.cryptsy.com/api.php?method=orderdatav2

Thank you.

FYI, I just pushed a commit that removed all the generics from Cyptsy to be consistent with the rest of XChange.

That Cryptsy class is looking cleaner. However, when you subsequently removed generics from PoloniexBasePollingService the public API broke. e.g. it should have been sending
https://poloniex.com/public?command=returnTicker but was sending
https://poloniex.com/tradingApi?command=returnTicker
I've reverted to the previous version of this Poloniex class and it works fine. I'm not sure of the exact intent behind these recent BasePollingService changes so will leave it to others to decide how best to fix this. Thanks!

@ww3456 Good catch. Thanks. I see what the problem is now. I will check all exchange modules and fix appropriately.

@ww3456 I pushed a fix.

@timmolter great, this is working now. For the record I now have the latest develop 3.0.0-SNAPSHOT running OK on BTC-E, Bitfinex, Cryptsy, Kraken and Poloniex for both market data and trading.

Excellent. Thanks for telling me which exchanges are working for you too!


port FiveCrop and TenCrop to prototype API

Both transforms are outliers in the current stable API as discussed in https://github.com/pytorch/vision/pull/5500#discussion_r816507761. Even if we want to deprecate them in favor of a different approach eventually, we need to support them at day 0 of the roll out.
The issues with these transforms arise, because they always need to be followed by another to batch them together:
https://github.com/pytorch/vision/blob/71d2bb0bc67044f55d38bfddf04e05be0343deab/torchvision/transforms/transforms.py#L980-L984
We cannot re-use this lambda approach since crops might be buried inside a nested container. Thus, we need a custom transformation that picks up on the result of such a multi crop transform and batches it. To communicate which elements need to be batched between the two transformed, I've opted to wrap the return values of the multi crop kernels in named tuples, because they are full JIT scriptable. The old FiveCrop returned a regular tuple, so this is 100% compatible. TenCrop returned a list, but this is still fine unless someone is using list specific methods like append or the like.
Example usage:
from torchvision.prototype import datasets, transforms, features

transform = transforms.Compose(
    transforms.DecodeImage(),
    transforms.TenCrop(16),
    transforms.BatchMultiCrop(),
)

for sample in datasets.load("imagenet").map(transform):
    print(sample["image"].shape)
    break

torch.Size([10, 3, 16, 16])

To test with PIL images, insert transforms.Lambda(to_pil_image, features.Image) after the decoding.

After some offline discussion @datumbox and I agreed that we indeed need the BatchMultiCrop transform. Nevertheless, with the new commits the implementation is a lot simply, by moving the "result wrapper" onto the transformations and being able to use only a single subtype of list rather than two namedtuples that would have been necessary to satisfy torchscript. Plus, instead of making apply_recursively more general, BatchMultiCrop now implements a custom version of it to keep it simple.


[Bug]: XPATH expression works on $x method but not on waitForXPath method of Page object

Minimal, reproducible example
import puppeteer from 'puppeteer'
const browser = await puppeteer.launch({
        headless: "new",
        devtools: false,
        args: [`--window-size=1080,1080`],
        defaultViewport: {
            width: 1080,
            height: 1080
        }
    });
const page = await browser.newPage()
const url = URL('https://www.google.com')
let res = await page.goto(url.href, { waitUntil: 'networkidle0' });
page.waitForXPath('//input[contains(@aria-label, "Phone number, username, or email")]')
//This hits an XPATH syntax error by the querySelectorAll on Document
//For what it's worth, I'm getting around for some reason with page.$x() method, which I expected to report me back the same error.

Error string
Uncaught Error DOMException: Failed to execute 'querySelectorAll' on 'Document': '//input[contains(@aria-label, "Password")]' is not a valid selector.
Bug behavior

[ ] Flaky
[ ] PDF

Background
I was trying to select an HTML input element using XPATH selectors and then proceed to type in, but I got an XPATH syntax error instead. I checked the syntax was ok by checking some XPATH documentation on the web and running the same selector on a chrome web browser dev tools console with the "$x()" function, and it worked.
Expectation
It gave a result other than a XPATH syntax error.
Reality
The Page Object waitForXPath method always reports an error concerning the XPATH syntax of the expression I specified above.
Puppeteer configuration file (if used)
No response
Puppeteer version
21.0.3
Node version
18.17.0
Package manager
yarn
Package manager version
1.22.19
Operating system
Linux

I am unable to reproduce with the provided example. Please update it with one that can be consistently verified.
Error I observe a TimeoutError which is expected as the selector does not exits on the page.


Add social icons for fellows

At the moment, it only displays the fellow name and location. To help people network, we should add social icons to each of the fellow cards.

[ ] Add LinkedIn icons to the card
[ ] Add Twitter icons to the card
[ ] Add Resume icons to the card
[ ] Add GitHub icons to the card

Hey, I would like to work on this issue. Can you assign it to me?

I would like to work on this issue.

Happy to collaborate with you!


Another Quick Switcher icons cut

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
No
Minimal theme version
5.3.0
Describe the bug
When using minimal in conjunction with the plugin "Another Quick Switcher" the icons in the pop up are cut off:

I've already created an issue in the plugin's repo: https://github.com/tadashi-aikawa/obsidian-another-quick-switcher/issues/73#issue-1290922766
Debug info
SYSTEM INFO:
Obsidian version: v0.15.3
Installer version: v0.14.6
Operating system: Darwin Kernel Version 21.5.0: Tue Apr 26 21:08:29 PDT 2022; root:xnu-8020.121.3~4/RELEASE_ARM64_T8101 21.5.0
Login status: logged in
Catalyst license: insider
Insider build toggle: on
Live preview: on
Legacy editor: off
Base theme: light
Community theme: Minimal
Snippets enabled: 30
Safe mode: off
Plugins installed: 60
Plugins enabled: 58
1: Divide & Conquer
2: Pandoc Plugin
3: Quiet Outline
4: Regex Find/Replace
5: Smart Typography
6: Sortable
7: System Dark Mode
8: Tabout
9: Obsidian Tabs
10: Version History Diff
11: Zotero Integration
12: Obsidian42 - BRAT
13: Omnisearch
14: Periodic Notes
15: Plugin Changelogs
16: Advanced Tables
17: Templater
18: Workspaces Plus
19: Block Reference Counter
20: Obsidian Enhancing Export
21: Pandoc Reference List
22: Advanced URI
23: Dynamic Highlights
24: Pane Relief
25: Obsidian Git
26: Better File Link
27: Admonition
28: Advanced Mobile Toolbar
29: Apply Patterns
30: Better footnote
31: BibNotes Formatter
32: Calendar
33: Code block from selection
34: Contextual Typography
35: Convert url to preview (iframe)
36: Copy Block Link
37: Dataview
38: Dictionary
39: Citation Style Translation
40: Typewriter Scroll
41: Quick Switcher++
42: Garble Text
43: Hotkey Helper
44: Hotkeys++
45: Lapel
46: Markdown Table Editor
47: Metadata Extractor
48: Natural Language Dates
49: Dynamic Table of Contents
50: Electron Window Tweaker
51: Focus Mode
52: Hider
53: Hotkeys for templates
54: Obsidian Link Converter
55: Hover Editor
56: Minimal Theme Settings
57: Style Settings
58: Another Quick Switcher
RECOMMENDATIONS:
Custom theme: for cosmetic issues, please first try updating your theme to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.

This issue is due to bugs introduced in Obsidian 0.15.3 — I will come back to it when 0.15.4 is released


concourse: Small fixes in README and templates

What this PR does / why we need it:
Typo and other small fixes.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

/ok-to-test

/lgtm

/assign @william-tran

Resolved merge conflict. @timstoop @Chr1st0ph @william-tran

/lgtm

/lgtm

Updating the chart version since it got bumped since the last PR. @sameersbn @timstoop

/lgtm


Remove the Skia backend

Introducing new changes to the renderer requires quite a bit of work, because they also have to work on Skia. I see no reason to keep the Skia backend for any longer, as it's much more bloated than the OpenGL backend and generally contains a lot of things that NetCanv doesn't even use.
Thus, the backend is scheduled for removal after 0.5.0 is released.

Removed in 68075f6.


Camera related documentation needs improvement regarding touch/screenspace

Issue details
Camera.unproject does not return original value
Reproduction steps/code
    public void create() {
        Vector2 tmpvec = new Vector2();
        tmpvec.set(0, 0);

        Stage stage = new Stage(new ScreenViewport());
        stage.stageToScreenCoordinates(tmpvec); // has the correct screen coords;
        stage.screenToStageCoordinates(tmpvec);
        System.out.println(tmpvec); // prints (0, -1), should be (0, 0)
    }

Due to the following line in Camera (-1 at the end):
https://github.com/libgdx/libgdx/blob/c0c1f515df54092cb47717f43f022db02cf7e328/gdx/src/com/badlogic/gdx/graphics/Camera.java#L197
Intended?
Version of LibGDX and/or relevant dependencies
1.9.10

If you think there is a problem with Camera, your test should only involve camera, and not Stage/Viewports.
That said, this is a problem with documentation and variable names. It should really be touchToStageCoordinates, not screenToStageCoordaintes, since its expected touch space input. There are some other inconsistencies that probably should be cleaned up that are the same.

Ok, modified.
public void create() {
      Vector3 tmpvec = new Vector3();
      tmpvec.set(0, 0, 0);

      Camera camera = new OrthographicCamera();
      camera.viewportWidth = 100;
      camera.viewportHeight = 100;
      camera.project(tmpvec);
      camera.unproject(tmpvec);
      System.out.println(tmpvec); // prints (0.0,-0.01999998,0.0)
  }

#5854 is the same same issue, and as I said its a problem with documentation as they arent all screen space coordinates. We do not need two issues to discuss this same problem.

Here is a demo that demonstrates that this is a documentation issue. https://gist.github.com/Tom-Ski/3c71425e6a199ec2677fbac39e457e54

Why are your touch coordinates offset by 1?

Regardless, should a sequence of project and unproject return a different value?
      camera.project(tmpvec);
      camera.unproject(tmpvec);

As an example, if you have a screen of 10x10, bottom left coordinate is 0,0. In touch space, this coordinate is 0,9.  Gdx.graphics.getHeight() would return 10 however, we need the offset because we start at 0, not at 1.
Range of touch/screen is from (0 to width-1) since we start at 0.

As per the documentation, yes they should. There are just a few inconsistencies with the language used that makes it confusing. Although the documentation (for Camera at least) does describe the difference in coordinate spaces the functions return and expect, it still calls them both screen space, which is the confusing part. The solution here is to improve the docs.

Ok.
However, this is still unrelated to #5854 which is about stages with viewports with y offsets. Even if the method name were stage.stageToTouchCoordinates, I believe there is still an issue there. I'll comment further in that issue.

OpenGL for reference, specifically gluProject and gluUnProject, compiled and run on Arch Linux/Mesa 3D.
gluProject — map object coordinates to window coordinates
gluUnProject — map window coordinates to object coordinates
#include <stdio.h>
#include <GL/gl.h>
#include <GL/glut.h>

void displayFunc(void)
{
    GLdouble posX, posY, posZ;
    GLdouble winX, winY, winZ;
    GLdouble resX, resY, resZ;
    GLdouble modelview[16];
    GLdouble projection[16];
    GLint viewport[4];
    glGetDoublev(GL_MODELVIEW_MATRIX, modelview);
    glGetDoublev(GL_PROJECTION_MATRIX, projection);
    glGetIntegerv(GL_VIEWPORT, viewport);
    posX = 0.0;
    posY = 0.0;
    posZ = 0.0;

    gluProject(posX, posY, posZ, modelview, projection, viewport, &winX, &winY, &winZ);
    printf("window: %f %f %f\n", winX, winY, winZ);
    // window: 150.000000 150.000000 0.500000

    gluUnProject(winX, winY, winZ, modelview, projection, viewport, &resX, &resY, &resZ);
    printf("pos: %f %f %f\n", resX, resY, resZ);
    // pos: 0.000000 0.000000 0.000000
}

int main(int argc, char** argv)
{
    glutInit(&argc, argv);
    glutInitDisplayMode(GLUT_DOUBLE);
    glutInitWindowSize(300, 300);
    glutCreateWindow("Test glProject glUnProject");
    glutDisplayFunc(displayFunc);
    glutMainLoop();
    return 0;
}

Fondos osar cuenta bancaris

My problem iPhone X can’t get it to open I’ve tried everything,boot up tried everything

Can someone help me?

@mark6412 Don't ask for help on an issue tracker. See: https://github.com/libgdx/libgdx/wiki/Getting-help
Seems a bit unfortunate to need to think about touch space coordinates. Ideally we'd have only screen space and world space.

Good!
https://ufabet959s.com/


OAK-11267 - Upgrade Azure SDK V8 to V12 for oak-blob-azure

Every larger PR needs a JIRA ticket in https://issues.apache.org/jira/projects/OAK!

Every larger PR needs a JIRA ticket in https://issues.apache.org/jira/projects/OAK!

Hi! I linked this PR with this JIRA issue: https://issues.apache.org/jira/browse/OAK-11267 . Please let me know if something else is required.

Every larger PR needs a JIRA ticket in https://issues.apache.org/jira/projects/OAK!

Hi! I linked this PR with this JIRA issue: https://issues.apache.org/jira/browse/OAK-11267 . Please let me know if something else is required.

Please read https://jackrabbit.apache.org/oak/docs/participating.html. Currently the commit message does not reference the Oak issue.

@kwin , maybe it's just style.... but to me your review comments don't come across as welcoming (see https://apache.org/foundation/policies/conduct). @andreeastroe96 is a new contributor to the project and there is IMHO no need to be angry... a friendly reminder will do magic.

@anchela You are right.
@andreeastroe96 Sorry, if I came across rude. That was not intended. Just be aware that this is the ASF Jackrabbit context so neither AEM nor GRANITE nor any other restricted references should be used here for everyone to be able to follow/review accordingly. Thanks.


[EXPLORATION]: Exploration on new charts in Plotly

Exploration on new charts and checking their feasibility to be implemented in the observability application.

Hi @mengweieric , @btzeng , @anirudha
Following is the list of charts that we have explored. Please can you tell which charts we need to implement and in what priority order.

Graph Name
Implementation Present
Description/Use case
Can be implented with Plotly
If requires other library
Documentation link
Comments(if any)

Area
No

Yes

An area is a line chart available with filled area below the line. We can add this in the graph list. Please suggest your thoughts on this one.

Controls
No

Can you please tell us how do we proceed with this one as we couldn't find any references of it in plotly.

Coordinate map
No
A coordinate map that represents the value on the graph with the latitude and longitude
Yes
No
https://plotly.com/javascript/scatter-plots-on-maps/

Data Table
No

Need to add this in the dropdown for charts

Goal
No
A goal chart is a chart that helps us keep a track of goals. It can act as a roadmap and show us how close are we to completing the goal.
Yes
No
https://plotly.com/javascript/indicator/
We think goal chart resembles indicator in plotly. Can you please suggest if this will be the right implementation?

Horizontal Bar
No

Yes
No
https://plotly.com/javascript/horizontal-bar-charts/
We have a file for type included need to check how to implement with just changing the orientation.

Markdown
No

There is a text dropdown in the charts. Please can you tell if we need to rename text to markdown?

Metric
No
Displays a single value after applying aggregations.
No
No
NA
It is not a graph it just displays a calculated single value on the basis of aggregate functions. Do we need to include this in the charts dropdown?

Region Map
No
It is a type of map that portrays the geographic pattern of a particular subject in a geographic area.
Yes
No
https://plotly.com/javascript/choropleth-maps/

TSVB
No
Build time-series using visualise pipeline interface

Tag Cloud
No
A group of words sized according to their importance
No
Yes
https://www.npmjs.com/package/wordcloud
Please confirm about if we need to implement this?

Timeline
No
A time-series graph with functional expressions
Yes
No
https://plotly.com/javascript/time-series/
Can you please confirm about the functional expressions we need to use here or will they be custom expressions written by user? Also we are unable to run this on dashboards, it'd be great if you can guide us on the steps.

Vega
No
It is a visualization grammar, a declarative language for creating, saving, and sharing interactive visualization designs. With Vega, you can describe the visual appearance and interactive behavior of a visualization in a JSON format, and generate web-based views using Canvas or SVG.
No
Yes
https://vega.github.io/vega/
This is an integration and not implementation from a library. This is itself a separate library, do we have to include this?

Vertical Bar
No

Yes

Should we rename the bar to vertical bar?

Thanks,
Ruchika
CC: @abasatwar , @spattnaik

lets do all with "Can be implented with Plotly"

thanks @ruchika-narang ! this is good :)

We have explored multiple charts and have found a list that can be implemented in the application. So closing this ticket.


(v2.5.x) Remove unnecessary tests reports destination

Kind of change

[ ] Bug fix
[X] New non-breaking feature
[ ] New breaking feature
[ ] Documentation update
[ ] Build improvement

Description
What is the goal of this pull request?
Simplify Gradle configuration removing an unnecessary configuration, see #1188 for details
How does it achieve that?
Just removes it, no other changes are necessary, reports path in CI still match.
Are there any alternative ways to implement this?
No :thinking:
Are there any implications of this pull request? Anything a user must know?
No
Issue
If this PR fixes an open issue, please add a line of the form:
Fixes #1188
Release notes
This is an internal change, I don't think it's worth adding to changelog.

Waiting (draft) until the JUnit 5 PR is merged, I suspect there will be conflicts.

Thank you!
You're right, let's not put the effort of moving to JUnit5 into 2.5.x.


-Add travis CI integration

Hello,
This modification adds a simple (for now) Travis CI integration.
It builds sql2o (except the oracle extension due to the non publicly available jar)
And it tests sql2o, skiping db2 and oracle due to the missing jar, and because these databases are not supported in Travis CI. (db2 doesn't even have tests for now)
I think this can be quite useful as it means that tests can be created and ran (on h2, postgres, open source databases... even mysql or neo4j if neded in the future) without having the need to configure them in a local environment.
In the future  the travis config can evolve to have maybe even email alerts or other cool stuff, or maybe publish performance results (don't know if it's possible)
Another advantage is when a pull request is made, if the merge is automatic it can be seen if the build/test succeeds or not.
If you wish to merge this you have to first activate travis on your branch on https://travis-ci.org and then edit the README.md build status url to point to your branch not mine.
Thank you.

Forgot to add, travis is free for open source projects.

Nice, it works! Thanks a lot!


[CI] Fixes flaky early stopping test

This changes the test from requiring the early stopping at a particular epoch to ensuring that it stops early due to time constraints. This should help reduce the chances of being flaky due to race conditions.

Codecov Report
Attention: 1365 lines in your changes are missing coverage. Please review.

Comparison is base (bb5073f) 72.08% compared to head (6c27157) 72.24%.
Report is 917 commits behind head on master.

Files
Patch %
Lines

...va/ai/djl/modality/nlp/generate/TextGenerator.java
2.81%
276 Missing :warning:

.../java/ai/djl/modality/nlp/generate/SeqBatcher.java
0.75%
132 Missing :warning:

...ity/nlp/generate/ContrastiveSeqBatchScheduler.java
2.97%
98 Missing :warning:

...i/djl/modality/nlp/generate/SeqBatchScheduler.java
9.83%
55 Missing :warning:

.../java/ai/djl/modality/cv/BufferedImageFactory.java
40.96%
47 Missing and 2 partials :warning:

...a/ai/djl/modality/nlp/generate/StepGeneration.java
2.04%
48 Missing :warning:

api/src/main/java/ai/djl/ndarray/NDArray.java
43.42%
39 Missing and 4 partials :warning:

...n/java/ai/djl/modality/cv/output/CategoryMask.java
22.00%
39 Missing :warning:

...i/src/main/java/ai/djl/ndarray/NDArrayAdapter.java
71.21%
31 Missing and 7 partials :warning:

.../cv/translator/SemanticSegmentationTranslator.java
37.50%
35 Missing :warning:

... and 76 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2866      +/-   ##
============================================
+ Coverage     72.08%   72.24%   +0.15%
- Complexity     5126     7161    +2035
============================================
  Files           473      708     +235
  Lines         21970    31915    +9945
  Branches       2351     3314     +963
============================================
+ Hits          15838    23056    +7218
- Misses         4925     7285    +2360
- Partials       1207     1574     +367

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


Remove coreapi requirement

Description
The coreapi requirement should be removed from requirements/base.txt.
Rationale
From the CoreAPI page:

DEPRECATION NOTICE: As of djangorestframework 3.9, the plan is to phase out CoreAPI in favor of OpenAPI as the default schema representation. New projects should consider using OpenAPI rather than CoreAPI. See the DRF 3.9 release announcment for more details.

The DRF 3.9 page has an example of how to use the integrated OpenAPI support.
Use case(s) / visualization(s)
None. It's a requirement for a module no longer developed. If it's used somewhere in the template that I've missed, it should be replaced with OpenAPI equivalents.
I only noticed it because I'm going through a Django project generated by this cookiecutter and can't explain why the requirement exists.

Thanks for this report. DRF (and Core API) are not directly used by the generated project, but it's so common to find them in Django projects that they have been added by default.
It should be easy to remove it, pull requests are welcome!

LOL maybe later, especially if "present but not used" is intended behavior.
If I'm going to the effort of a pull request, I'll wait until it's something more substantial, like say adding an actual DRF option to the template. Especially since I'll be constantly rebasing as pyupbot moves master along. (It's changing the same files, after all.)
I just figured this was something that could be rolled into some other minor fix update. Right now I'm in "I'm new here and might break things" mode.

Another fix to toss on there: in .gitignore,
### VirtualEnv template
# Virtualenv
# http://iamzed.com/2009/05/07/a-primer-on-virtualenv/

URL is 404. Suggested replacement: https://www.simononsoftware.com/virtualenv-tutorial-part-2/

Can you please avoid chucking unrelated problems you find in the same GitHub issue?
I appreciate the effort to report problems, but it's easier to actually fix them if they are in separate issues.
Thanks again.

OK, that's now #1972.

Are you sure CoreAPI is not used? The deprecation note talks about "phasing out". In DRF documentation, there are instructions to install it in order to generate the API documentation (https://www.django-rest-framework.org/topics/documenting-your-api/).

I'm running a project generated from this cookiecutter that has DRF v3.9.1 installed and no coreapi package, and I think I'm seeing the API doc screens fine. They're the ones that look like this, right?

@greyhare No, this is not the documentation.
from rest_framework.documentation import include_docs_urls

urlpatterns = [
    # ...
    url(r'^docs/', include_docs_urls(title='My API title'))
]

https://www.django-rest-framework.org/topics/documenting-your-api/#installation

Those instructions result in:
AssertionError: `coreapi` must be installed for schema support.

And it looks like CoreAPI is still hardwired. So the question is now "Do you want a dependency that is an archived GitHub project with no developers, hoping that the DRF crew replace it someday, and even though the template does not even provide commented-out URLconfs or settings for anything DRF?"

Is it an idea to add an extra option use_api_tools to cookiecutter? This give us the options to add more packages useful for API development like:
django-cors-headers
django-rest-auth
drf-yasg

FWIW, Core API depends on Jinja, which has a security vulnerability in 2.10.0, so this isn't a "freebie" dependency.

so what's the fix just remove the line from requirements.txt?

coreapi has clearly been deprecated by DRF (see the 3.10 release announcement) The method of generating documentation given above is no longer recommended in the docs.


[main] Upgrade to latest dependencies

GKE fixes in hack -dprotaso
/cc knative/func-approvers
/assign knative/func-approvers
Produced by: knative-extensions/knobots/actions/update-deps

/approve
/lgtm

/retest

/lgtm


[Storage] [DataMovement] [Blobs] Update to Stress Metrics

Add correct naming to stress metrics and using options bag to track metrics
Added try catch exception handling to track if a transfer is restarted

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


fix issue 210: custom values formatting schema

Resolves https://github.com/sindresorhus/query-string/issues/210

Sorry, I've just realized unfortunately I have no spare time to finish the PR. If someone is willing to finish - it'd cool to finally deliver this functionality.


database/sql: predictable request queue satisfaction

change random satisfaction of the query queue to sequential
test simulate situation when query append faster that execution
Sitaution:
I have 2 source of query:

the first source generates queries often. These queries are not important and can be canceled by timeout. Contexts have timeout
the second source generates queryes less often. These queries are important and must be executed. Contexts dont have timeout;

Trouble:
The connection poll satisfied request by random. This behavior does not guarantee the execution of the request in a finite time. I am faced with situation an unlimited growth of latency.
I make test to demonstration my situation.
Results of random satisfaction:
delay(diff = 5m30.844350115s, expected = 1m58.8s, max = 7m29.644350115s), queue(len=298, max=1486, min=364, diff=1122)
sql_db_test.go:51: current delay 7m34.568549607s is 2 times greater than expected 1m59.6s
--- FAIL: TestDB_queryQueue (600.00s)
FAIL
Results of sequenced satisfaction:
delay(diff = 344.00638ms, expected = 1m58.8s, max = 1m59.14400638s), queue(len=298, max=1486, min=1189, diff=297)
--- PASS: TestDB_queryQueue (600.00s)
PASS
queue.len - length of request to execution
queue.max - max index of request from queue
queue.min - min index of request from queue
queue.diff = queue.max - queue.min
If queue.diff is much longer that queue.len then queue has request with big latency
delay.expected = (queue.len - 1) * appendReuqestDelay
delay.max - latency of older request
delay.diff = delay.max - delay.expected
Solution:
Change the connection behavior from random to sequenced.

This PR is currently targeting the "release-branch.go1.17" branch. Go 1.17 is no longer supported per the release policy and so we don't use the release-branch.go1.17 branch for anything now.
If this is a change you'd still like to send, please see https://go.dev/doc/contribute#before_contributing and send it again targeting the branch named master instead. Thanks.


Update dockertoolbox to 1.8.3

Security Release: Docker 1.8.3 and 1.6.2-CS7

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


How to treat NaN or empty values?

The data download tools do not check for NaN values, empty or otherwise missing values in the downloaded data. It is to be discussed whether or in which case the program should ignore such values (interpreting them as 0), print a warning or raise an error if NaN values are encountered.

Furthermore, I think it is very important to check the output files on NaN or null values before exporting the files to simulation.


[Python Template] PubSubToGCS

Closing this issues as it can't be done due to:

Unavailability of a Pub/Sub connector for PySpark
Apache Bahir only supports Java & Scala

We will create new issues for equivalent Pub/Sub Lite template.


pydot png resolution issue

Hi, I used 'netwokx' package to make a directed graph.
And I hoped to visualize it through pydot.
When I tried it with a small graph, I could get a high quality png image.
So I tried visualization of my own graph but got a bad quality image through it.
The number of nodes are around 10k.
from networkx.drawing.nx_pydot import to_pydot

graph = nx.DiGraph()
#
# adding nodes and edges here anyway
#
d1 = to_pydot(graph)
d1.set_rankdir("LR")
d1.set_margin(1)
d1.write_png('ppp.png')

I tried code above and the below image is the result.
What can I do to get a better resolution image??
Thanks.

The cause of the problem was its extension. I don't know how it exactly works but found that I can resolve it by changing the method to 'write_svg'. Maybe the way it handles png format cannot handle way too large file.

For posterity's sake, an answer to the original question about the resolution of the PNG export:
Pydot uses Graphviz to layout the graphs, so you can look at the Graphviz documentation for ways to control the layout. Here, I will use the graph attribute size to control the resolution of the export PNG file. Other relevant attributes could be dpi and ratio. Alternatively, the Graphviz FAQ on how to make layouts larger suggests changing fonts, distances and line widths. Finally, refer to the documentation on Graphviz Command-line Usage and The DOT Language on how attributes are set there, because the methods of setting attributes from pydot and NetworkX are a reflection of that.
Below examples are for the size attribute. Note that the unit for size is inches and the default number of dots per inch (dpi) is 96, so 30 inches * 96 dpi = 2880 pixels. In my setup, all the examples below result in an image with a width of 2880 pixels. The dot man-page says the value of size needs to be wrapped in double quotes, but as we will see below, that is not always true.
Versions used: Debian 10.3 GNU/Linux, Graphviz 2.40.1, python 3.7.3, pydot 1.4.1, NetworkX 2.4.
1. In the DOT-file
graph {
    size="30,30!";
    a [label=<hi>];
}

2. Using Graphviz directly from the command line, without pydot or NetworkX
$ dot -Tpng -Gsize="30,30!" a.dot >a.png

Note that your shell may actually interpret the double quotes. With bash, for example, what actually gets executed is this:
dot -Tpng -Gsize=30,30! a.dot >a.png

(When I do pass actual double quotes, e.g. in bash using -Gsize=\"30,30!\" or $'-Gsize=\"30,30!\"', Graphviz dot ignores the entire argument.)
3. Using pydot to supply a Graphviz command-line parameter
import pydot
G = pydot.graph_from_dot_data("graph { a [label=<hi>]; }")
G[0].write_png("a.png", prog=['dot','-Gsize=30,30!'])

(Again, when I pass double quotes (['dot','-Gsize="30,30!"']), this results in Graphviz dot ignoring the argument, so I leave them out.)
4. Using pydot, adding the attribute to the pydot Graph object
import pydot
G = pydot.graph_from_dot_data("graph { a [label=<hi>]; }")
G[0].set_size("30,30!")
G[0].write_png("a.png")

5. Using NetworkX, adding the attribute to the pydot Graph object
import networkx
from networkx.drawing.nx_pydot import to_pydot
graph = networkx.Graph()
graph.add_node("a", label="<hi>")
d1 = to_pydot(graph)
d1.set_size("30,30!")
d1.write_png("a.png")


Added support for Canvas Outcome1 Extension for ltiLaunchUrl

I know and understand this to be a less than ideal implementation, I needed it for my use of the library and wanted to see if something along these lines would be something desired or not in this LTI Library.
Its based off of this spec https://canvas.instructure.com/doc/api/file.assignment_tools.html
Currently only implements LTILaunchUrl.
Let me know if this is something you are interested in and I'm sure we can work on a better way to integrate it.

I'm sorry you haven't had any feedback yet. I'll take a look this weekend.

This is pretty cool, but I don't think Canvas extensions should be in this library. I think it would make more sense to create a separate Canvas client library that inherits from LtiLibrary.

@Skylertodd I tried following my own suggestion of creating a separate library for this extension, but I had to copy way too much stuff from LtiLibrary. I ended up adding support for the Canvas submission details to both Outcomes1Client and OutcomesControllerBase in case someone wants to support them on their own platform. See PR #103 and let me know what you think.

Merged #103 instead.


batch processing

This tool would be better if it could do batch operations on folders

need batch ,pls


lib: Switch to released containers-image-proxy version

We cut a new version upstream, let's switch back to crate dependencies.

Oops right, we actually need to do some porting here


Rename "module interface" to "module connector"

Module interfaces are now module connectors.  This should alleviate the
terminology confusion about an interface in programming terms vs. an
interface to a module.

Still not quite right.  Need to think about terminology some more.


Disable ASLR in the generated PE file

This would be a simple modification to the PE headers and allow for the PE file produced from the shellcode to be at the default image base. Could be driven by a command line argument.

Added with v0.0.4 commit 50524d2aa17b7fbebf2be871425547d3e3c1e8c0


To use an updated version of the FAIR assessment template

Could you please replace the FAIR assessment template with the attached file, which is the updated version of the FAIR assessment? There is a minor change only - we removed the "of archive" and "of model", respectively, from the indicator description in lines 55 and 56.
FAIR_assessment_template_v02.xlsx

Will be upated.


Harvest should add ifgroup(LAG) throughput metrics

Thanks, imthenightbird, for reporting this issue on Discord.

Verified in 24.11 ddb97c57
Link Aggregation Group row is in the ONTAP: Network dashboard.
imthenightbird also confirmed, in an earlier nightly build, that these three new panels contained the correct data


Fix authority check on init with root instruciton

Not valid one. Thinks made that way on purpose.


Add arm64 RIDS  to the portable RID list

Summary
Added new RIDs for ARM64 devices.

Ping @Thraka for review. It looks like this is an area you understand better.

Conflict with #18214, which was just merged.

@KhArtNJava I'm going to close this for now. Please feel free to reopen when you can find some time to work on this a bit. Thanks!


migrate v2 config

I think the old configuration file is unfriendly to people looking for sample code

@utamori thanks! As a contributor could you please push up with signed commits. You'll need to git commit --ammend -s to sign and force push up the current commit.

CI is failing


feat(OCPADVISOR-133): workload recommendation table

OCPADVISOR-133
Basic workload recommendation table
Setup

Clone insights-results-aggregator-mock
make build (takes a few minutes for me, coffee time ☕)
2.1. can speed it up with make build -j 4 to run multiple jobs at once
make run
Test the mocked api curl localhost:8080/api/insights-results-aggregator/v2/namespaces/dvo
Run the app MOCK=true npm run start:proxy:beta
Workloads should load mocked data

:tada: This PR is included in version 1.31.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[5.1] [MiscDiagnostics] Update diagnostics for IUO to Any coercion

Cherry-pick of #23617. Updates the diagnostic for IUO to Any implicit coercion.
Resolves SR-10199.

cc @xedin

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - e2e2d505ad19391af099019f0149763b3c1ae73f

Unrelated failure error: TestXMLDocument.test_validation_success : failed. I guess somethings wrong with Linux builds today as well 😅

@swift-ci please test Linux platform

Sorry I almost forgot about this PR :)


Is there any way to print text highlights for several labels ?

Hi ! Thanks you for you project.
I use the function, save_to_file(path, top_labels = n) with n > 1.
This works very well except that I get the highlights only for the first top labels.
Is there any way to catch them all ? :)
Thanks

Sorry, it's not possible. Since we only show the text once, I don't even know what highlights for all of them would look like, since they could be conflicting and overlapping.
I guess you can always save them one by one : )


Add inlineLabel() method explenation

There was missing information about inlineLabel() on the Layout components documentation page. This adds a little snippet to explain it.

Thanks


No option to delete Other Videos

Beta v1.0.5 - unlike TV Shows and Movies, there is no options button in the Other Movies page of the content manager.
Essentially just missing the button which displays these options:

True! Its not on the top of the list, but definitely a todo down the road. Thanks!


UIA Transaction Failure

As reviewed on Google Play, transaction on User Issued Asset doesn't seem to be working. Whenever I try to send an Asset I created, it says "No Internet Connection".
The Issue is not present while sending BTS.
I am using a HTC 10 with Android 6.0.1

We should have a fix for this by end of day tomorrow (thursday) for you :)
Once done and tested, we will upload v1.0.2 to google play for you to make sure it was fixed.
Thanx for the report! :)

No probs, looking forward to it!

Update: I think we fixed it. Was gonna push the release to google play tonight but it's just too late right now. Will make some other tweaks tomorrow then hopefully publish v1.0.2 tomorrow night.

Closing this issue now, will release (hopefully) v1.0.2 tonight..


S2I failures to build using podman

S2I build failures when using podman either thru socket connection (process started by func) or thru ssh connection.
Using podman process started by func the below error appears
error: Unable to get image metadata for registry.access.redhat.com/ubi8/nodejs-16:latest: Cannot connect to the Docker daemon at unix:///tmp/func-podman-3934646889/podman.sock. Is the docker daemon running?

When using podman thru SSH the following is received (example podman machine on mac)
# i.e.: DOCKER_HOST set to ssh://core@localhost:50718/run/user/501/podman/podman.sock
$ kn func build -v --builder=s2i
...
error: Unable to get image metadata for registry.access.redhat.com/ubi8/nodejs-16:latest: error during connect: Get "http://placeholder%2F/v1.40/images/registry.access.redhat.com/ubi8/nodejs-16:latest/json": read tcp [::1]:52810->[::1]:50718: use of closed network connection

@jrangelramos what is the status of this issue? I feel like @matejvasek resolved this a while back, but am not sure.

Hi @lance yea it seems it is.
using the process started by func ...
jeramos  1094190 1094168 84 16:17 pts/4    00:00:16 podman system service unix:///tmp/func-podman-2774041938/podman.sock --time=0

... I could quickly check it
$ func build -p /tmp/podman-s2i-func -b s2i
A registry for Function images is required. For example, 'docker.io/tigerteam'.
? Registry for Function images: localhost:5000/test
Note: building a Function the first time will take longer than subsequent builds
   🙌 Function image built: localhost:5000/test/podman-s2i-func:latest

Excellent - closing


[Kaltura] Fix carleton extraction

Closes #1108.
Template
Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

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

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

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Don't remove existing tests unless you can add a similar working one. Add something like 'skip': 'No longer available' instead

Thanks for the feedback, I think I got everything. We could probably move the _search_json, _search_regex and _parse_json implementations to utils, to make them easily accessible for class methods, or turn them into class methods, or run embed extraction on instances.

Many of the comments are unresolved. Make sure to click on "Load more" to see all comments

We could probably move the _search_json, _search_regex and _parse_json implementations to utils, to make them easily accessible for class methods, or turn them into class methods, or run embed extraction on instances.

The design of _search_regex is tightly coupled to the printing functions, preventing us from moving it out without a major redesign


Allow for Boolean Prop Naming Custom Message

The default message for boolean-prop-naming does not make it clear why the rule is being flagged. I'd like my organization to be able to immediately know what the issue is without them having to parse the Regex.
This PR introduces functionality such that the following configuration:
{
    'react/boolean-prop-naming': [
      'error',
      {
        rule: '^(is|has)[A-Z]([A-Za-z0-9]?)+',
        message: 'Boolean prop names must begin with either \'is\' or \'has\'.'
      }
    ]
}

displays the custom message of Boolean prop names must begin with either 'is' or 'has'..

@ljharb Did you mean to request changes on this PR. From your comment, it looks like you meant to approve.

@rapheld if you click on "show outdated" you'll see the change i requested.

I had tried that, but for whatever reason, it wasn’t displaying a comment. Worked this time though.

@ljharb String templating added!

@ljharb Glad to see this merged! This is my first contribution to this project, and I'm trying to determine how I should bring this into my own project. What's the timeline for publication of this enhancement?

@louisscruz once merged, you're welcome to delete your branch whenever you like, or keep it forever :-) For this project, only @yannickcr can do a release, so it'll be released when they get a chance to do so (I can't speak to their timeline).

@yannickcr Any plans for publication?


The training batch size of 8x VFI on GoPro

Hi,
What is the batch size for training FLAVR for 8x VFI, I see in the paper that it is 32 with the frame size of 512x512. But I train on 8 GPUs(1080Ti whose memory is the same as 2080Ti), I got OOM error and only a batch size of 16 is ok. Besides that, neither random frame order reversal nor random horizontal flipping as augmentation strategies can be found in the GoPro.py. I wonder that can I reproduce the results with this code ?

Hi, thanks for your interest in our work. The GoPro code can be used for inference. For training you have to modify it accordingly adding augmentation etc.

Thanks, I will have a try


vagrant demo instance doesn't display anything at uri / after a fresh reset to master

This was as of Jan 16, 2017 -
I followed the README instructions for the demo setup per the README at https://github.com/RackHD/RackHD/tree/master/example#running-integration-tests:
vagrant ssh dev:
cd ~/src
./scripts/clean_all.bash && ./scripts/reset_submodules.bash && ./scripts/link_install_locally.bash
cd ~
sudo pm2 start rackhd-pm2-config.yml

Afterwards, using the default Vagrant setup, opening a browser to http://localhost:9090/ returned a 404 for /:
curl -v http://localhost:9090/:
*   Trying ::1...
* TCP_NODELAY set
* Connection failed
* connect to ::1 port 9090 failed: Connection refused
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 9090 (#0)
> GET / HTTP/1.1
> Host: localhost:9090
> User-Agent: curl/7.51.0
> Accept: */*
>
< HTTP/1.1 404 Not Found
< X-Powered-By: Express
< Access-Control-Allow-Origin: *
< X-Content-Type-Options: nosniff
< Content-Type: text/html; charset=utf-8
< Content-Length: 13
< Date: Mon, 16 Jan 2017 20:11:25 GMT
< Connection: keep-alive
<
Cannot GET /
* Curl_http_done: called premature == 0
* Connection #0 to host localhost left intact

The same happens on the secure port variant (https://localhost:9093)
docs and swagger-ui endpoints do load (i.e. http://localhost:9090/docs and http://localhost:9090/swagger-ui/)  - but there's no longer a default bringup of the web interface that previous existed, and no redirect (or notes in the readme) to make it clear how to interact with the demo.
For the demo setup, I'd recommend at least a redirect to the latest documentation if you're not going to leverage a web interface/single-page application up front.

https://rackhd.atlassian.net/browse/RAC-4121


bugfix: Add delay between create and update calls

This helps mitigate a 409 conflict response from Confluence when
creating a new page and then updating it right away. This may not be the
cleanest solution, but it seems to do the trick.
Resolves #139

This doesn't seem good to do such a sleep operation, but if it works it works. Thanks!

Maybe a better alternative is to add some built-in retry logic whenever 409 conflicts occur. I could explore that option.


Add a link to page describing JSON type in MySQL

I added a link to the page describing the JSON type in MySQL in the section "JSON Support" based on the reply to my question in #117.

Thanks.


Add CHANGES.rst

Fixes https://github.com/pypa/setuptools/issues/575

Thanks for this, but I'm going to decline. It only gets CHANGES.rst, but not README.rst.

Well that's what #575 asked for...

Indeed. I made that suggestion without looking at the code. It certainly would have fixed the issue. Thanks again.


Optimize BitmapEncoder class

Is there a way to optimize the methods of the BitmapEncoder class? They are extremely costly; a graph with 50k points takes over 2 seconds to save when calling saveBitmap(List). getBufferedImage and saveBitMap( toStream ) are even more costly.

Size and Timing Info
207852 is the size in bytes of the graph
10811 is the size in bytes of the empty graph
Runtime of saving the filled graph takes anywhere from 2-2.959 seconds on my machine
Runtime of saving the empty graph takes .20 seconds on my machine


Allow to inject config to GuzzleClient

I faced problem when trying to connect to Host which requires Basic Auth and couldn't find any way to config this header in this project. I checked and found that we initiate Guzzle Client inside class HttpRequestManager which is not good. Sometimes we need to configurate something such as authentication or proxy. It would be great if we can allow developer inject config there. Of course, there are some config we don't want to developer to change, we can forbid them to do that but still allow them to change the rest
/**
 * construct
 *
 * @param string $host
 * @param int $timeout
 * @return void
 */
public function __construct($host, $timeout = 1)
{
    parent::__construct($host, $timeout);
    $this->client = new Client;
}

For now I have to create new class which inherits class HttpRequestManager and inject config there
/**
 * construct
 *
 * @param string $host
 * @param int $timeout
 * @param string $apiSecret
 */
public function __construct($host, $timeout = 1, $config = [])
{
    parent::__construct($host, $timeout);
    // We can do some filters here for $config before transferring to Client
    $this->client = new Client($config);
}

Please note that some configurations (in my specific case, the CA file) may be set externally via environment variables or via php.ini settings.


Edit metadata window is trimming the last character of a text when overriding

Steps:

Go to an existent table (small enough so that you can see the data for all rows)
Go to the map view and try to edit the metadata of a feature that already contains some text:

I wrote: " oh yeah", and the result was:

The last character of the text does seem to be skipped.
More info:

It doesn't seem to happen always, my first time doing the steps spotted the issue but the next times it was hard to replicate again.
It doesn't happen with empty boxes -  Users are removing all the content and writing again in order to workaround the issue

cc @xavijam @javierarce

SB/6059853

on it


Rename robot-framework to robotframework

Support for highlighting Robot Framework plain text syntax was added in https://github.com/PrismJS/prism/pull/2034 and I'm so grateful for that.
Unfortunately, as my mistake, I forgot to mention that the canonical name for the language in various context has used to be robotframework withoud dash, not robot-framework.
Since there is has not been release yet for prism since the addition of the language, it should be safe to still do the rename. I tried to the rename in this pull.

LGTM;
Just the CI. I don't get why it failed. I'll look into it.

I might have been naive in just sed’ing and renaming files...

Found it. Please run gulp. Uglify found some things to optimize.
Just run the following command and everything should be fine.
npm ci && npx gulp (this require NodeJS >= 8)

Thanks. I also found the tests and there is still places to fix...

Better now. Thanks for help.

Thank you for contributing!


[dr-autosync]v6.5 changefeed checkpoint ts lag up to 20min after down backup dc

What did you do?
1.Deploy one cluster and set it in dr-autosync mode, with 3 pd, 3 tikv, 1 ticdc in primay datacenter. and 2 pd, 3 tikv, 1 ticdc in backup datacenter.
2.Down backup dc, including 2 pd, 1 ticdc and 3 tikvs.
What did you expect to see?
Changefeeds should be able to function properly without any influences.
What did you see instead?

Versions of the cluster
Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):
(paste TiDB cluster version here)

Upstream TiKV version (execute tikv-server --version):
(paste TiKV version here)

[root@tikv3-0 ~]# /tiup/deploy/tikv-20160/bin/tikv-server -V
TiKV
Release Version:   6.5.3
Edition:           Community
Git Commit Hash:   cb0dab3e4e0f3081966e871057272a40aa7ffc2d
Git Commit Branch: heads/refs/tags/v6.5.3-flashback
UTC Build Time:    2023-08-01 05:02:02
Rust Version:      rustc 1.67.0-nightly (96ddd32c4 2022-11-14)
Enable Features:   pprof-fp jemalloc mem-profiling portable sse test-engine-kv-rocksdb test-engine-raft-raft-engine cloud-aws cloud-gcp cloud-azure
Profile:           dist_release

TiCDC version (execute `cdc version`):

[root@ticdc1-0 ~]# /tiup/deploy/cdc-8300/bin/cdc version
Release Version: v6.5.3-drautosync
Git Commit Hash: aa9733328a4e4c2c09ed046ee8bb0d8675bc11dc
Git Branch: heads/refs/tags/v6.5.3-drautosync
UTC Build Time: 2023-07-24 06:27:18
Go Version: go version go1.19.11 linux/amd64
Failpoint Build: false
[root@ticdc1-0 ~]#

```console
(paste TiCDC version here)

Downstream version
/ # /tikv-server -V
TiKV
Release Version:   6.5.3
Edition:           Community
Git Commit Hash:   fd5f88a7fdda1bf70dcb0d239f60137110c54d46
Git Commit Branch: heads/refs/tags/v6.5.3
UTC Build Time:    2023-06-09 10:55:15
Rust Version:      rustc 1.67.0-nightly (96ddd32c4 2022-11-14)
Enable Features:   pprof-fp jemalloc mem-profiling portable sse test-engine-kv-rocksdb test-engine-raft-raft-engine cloud-aws cloud-gcp cloud-azure
Profile:           dist_release

/severity major

what does the " dr-autosync" mode mean? if the downstream crash, why should the lag not be affected ?

@nongfushanquan for dr-autosync you can refer to https://docs.pingcap.com/tidb/stable/two-data-centers-in-one-city-deployment.  Backup dc is not downstream, it is the "DR DC" in dr-autosync topo I believe.

/assign @asddongmen

Didn't hit this recently in v6.5.0 testing, after down backup dc for more than 5 hours , lag still less than 10s.
So close this issue.

Reopen it for no related fix recently.

Close it since it's stale, if you see it again , feel free to open it and please contact me.


Store photos and metadata in download directory

I'd like to create the local web page equivalent of the desktop app. It would be helpful if the script could download and store images and json metadata along with the files.

That's a nice project idea!
While you can use drpg as a library, it currently does not support specyfying what fields to retrieve from the API. This method could be extended to do so: https://github.com/glujan/drpg/blob/ef8b5d3919ba9daea77664312bb146664c7f31eb/drpg/api.py#L74

Saving images and metadata is out of the scope of this library. But I'm good to extend drpg as described in the previous comment.

The only thing missing is the actual product image. The "cover_url" just returns a slightly larger icon. "products_thumbnail100" of course, just returns the smaller icon. If I have time to reverse engineer some of the protocol next week, I'll see what I can find. If you have any notes to share, it'd be helpful.

@esemwy, you can have a look at #102 - it's not ready (I want to rewrite extra_fields to be a full list of fields to fetch, extend sync.py to allow easily selecting what fields to download, and allow post processing of API response), but maybe you find this usefull.
I don't think there was a field with a high-resolution cover, but it was a few years ago. Maybe something has changed since then. I remember using the official Windows app and mitmproxy to see what API looked like. Maybe the new website is not sever-side rendered as the old one, and uses the same API as the app?

Looks like the client requests images that follow the pattern.

https://api.drivethrurpg.com/images//-thumb100.png
https://api.drivethrurpg.com/images//.png

We also know about "cover_url" which may or may not be vestigial.

https://api.drivethrurpg.com/images//-thumb140.png

The first being the thumbnail, and the second the large product image. Those, of course, redirect to the cloudfront URL where the content is actually stored. It's not clear how we derive the extension of the full size product image unless it's just expected to have the same format as the thumb100 or the thumb140. I'll write some test code later, and let you know.

OK, so I got something working. I didn't actually need to download the cover images, although I can, but I did need the path, and the code I have seems to derive it OK:

        url = urlparse(product['cover_url'])
        filename = Path(url.path)
        newurl = f'{url.scheme}://api{url.netloc[3:]}{url.path[:-13]}{filename.suffix}'

The only thing that ends up a bit hacky is I need to instantiate DrpgSync so that I can call _file_path(). I guess I might suggest a more generic version that takes the path prefix and the compatibility_mode flag be added to the API.


fix: separate repeated steps as individual functions

Description
This pull request changes the following:

Separate repeated steps as individual functions

Related Issues

Closes #558

LGTM, but I want to let @Ivo-Yankov merge https://github.com/hashgraph/solo/pull/533 first, as we need that for Simi's team.

:tada: This PR is included in version 0.30.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use negative length for speeding up.

I use the following code for benchmark test.
`
/* just comment */
	$name = "test_benchmark.cpp.cpp";
	$ext = ".cpp";
	$raw_name = "";

	$this->benchmark->mark("positive_start");

	$count = 1000000;
	for($i = 0; $i < $count; $i++){
		$raw_name = substr($name, 0, strlen($name) - strlen($ext));
	}
	$this->benchmark->mark("positive_end");
	echo "Positive length elapsed ".$this->benchmark->elapsed_time("positive_start", "positive_end")." seconds. "."
";

	$this->benchmark->mark("negative_start");
	for($i = 0; $i < $count; $i++){
		$raw_name = substr($name, 0, - strlen($ext));
	}
	$this->benchmark->mark("negative_end");
	echo "Negative length elapsed ".$this->benchmark->elapsed_time("negative_start", "negative_end")." seconds. "."
";`

The output is as follows:
Positive length elapsed 4.1075 seconds
Negative length elapsed 2.8315 seconds
Signed-off-by: tianhe1986<EMAIL_ADDRESS>

So you successfully demonstrated that calling strlen() twice is more expensive than calling it once. Impressive. And that in the upload class where the most time consuming factor is ...

Oh I make a joke, just forget it ^^

I did notice this in #4691, but was too lazy to do something about it at the time.
I don't understand why you've closed this though.


Documentation templates addition to pan.dev

Status
Ready
Description
Added the documentation + RN templated to pan.dev
@kirbles19 Please review and add whatever you believe is necessary
@anara123 FYI

Hi @reutshal , this is great content. We already have some docs that cover part of it:

Pack Docs https://xsoar.pan.dev/docs/integrations/pack-docs
Release Notes https://xsoar.pan.dev/docs/integrations/release-notes
Integration Docs https://xsoar.pan.dev/docs/integrations/integration-docs

I think we should avoid duplicating content as much as possible: is there a way you can merge docs with the existing ones, or do you believe they should be separate?

Hi @reutshal , this is great content. We already have some docs that cover part of it:

Pack Docs https://xsoar.pan.dev/docs/integrations/pack-docs
Release Notes https://xsoar.pan.dev/docs/integrations/release-notes
Integration Docs https://xsoar.pan.dev/docs/integrations/integration-docs

I think we should avoid duplicating content as much as possible: is there a way you can merge docs with the existing ones, or do you believe they should be separate?

Hi @reutshal , this is great content. We already have some docs that cover part of it:

Pack Docs https://xsoar.pan.dev/docs/integrations/pack-docs
Release Notes https://xsoar.pan.dev/docs/integrations/release-notes
Integration Docs https://xsoar.pan.dev/docs/integrations/integration-docs

I think we should avoid duplicating content as much as possible: is there a way you can merge docs with the existing ones, or do you believe they should be separate?

Hi!
I thought about merging them, but they are mainly about how to generate the docs or explaining about the different docs files rather than creating good documentation.
The ones I added are more about writing better and clearer documentation - we did a session for our developers and thought it could be useful here.
If you have an idea for a clearer category name or a better separation I would love to hear it

Hi @reutshal , this is great content. We already have some docs that cover part of it:

Pack Docs https://xsoar.pan.dev/docs/integrations/pack-docs
Release Notes https://xsoar.pan.dev/docs/integrations/release-notes
Integration Docs https://xsoar.pan.dev/docs/integrations/integration-docs

I think we should avoid duplicating content as much as possible: is there a way you can merge docs with the existing ones, or do you believe they should be separate?

Hi!
I thought about merging them, but they are mainly about how to generate the docs or explaining about the different docs files rather than creating good documentation.
The ones I added are more about writing better and clearer documentation - we did a session for our developers and thought it could be useful here.
If you have an idea for a clearer category name or a better separation I would love to hear it

I understand, thanks.
The risk of having more than one article on the same topic (i.e. Release Notes) is that people might get confused or just read one of them and miss the others. Also, your docs shouldn't be under the "Integrations" section because they cover a broader topic.
After looking at the content once more:

I still think that the RNs articles should be merged in a single one because it's a standalone topic that should cover what RNs are, why they are needed, how to generate them and what to write there.
The other article covers several things where either we already have a doc (i.e. integration doc or description file) or we don't (i.e. playbook doc or scripts doc). We have two options here:

Put the content closer to the objects you are describing (i.e. split your doc in 3 parts (integration doc, description file, playbook docs) and add the information to the different articles, including creating a new one for Playbook Documentation that is now missing)
Create a "Documentation Best Practices" article somewhere (probably under Content Pack section) where you keep the information there, but we must cross reference it from the other articles (integration docs, integration description file, etc) and make sure that there is no duplication of content.

What do you think?

I understand, thanks.
The risk of having more than one article on the same topic (i.e. Release Notes) is that people might get confused or just read one of them and miss the others. Also, your docs shouldn't be under the "Integrations" section because they cover a broader topic.
After looking at the content once more:

I still think that the RNs articles should be merged in a single one because it's a standalone topic that should cover what RNs are, why they are needed, how to generate them and what to write there.
The other article covers several things where either we already have a doc (i.e. integration doc or description file) or we don't (i.e. playbook doc or scripts doc). We have two options here:

Put the content closer to the objects you are describing (i.e. split your doc in 3 parts (integration doc, description file, playbook docs) and add the information to the different articles, including creating a new one for Playbook Documentation that is now missing)
Create a "Documentation Best Practices" article somewhere (probably under Content Pack section) where you keep the information there, but we must cross reference it from the other articles (integration docs, integration description file, etc) and make sure that there is no duplication of content.

What do you think?

Agree with @fvigo .
There is third option, we can have a separate section for Documentation and inside of it will have all the doc files?

Release notes (as you described above)
Pack docs
Integration docs
Playbook docs
Script docs

Agree with @fvigo .
There is third option, we can have a separate section for Documentation and inside of it will have all the doc files?

Release notes (as you described above)
Pack docs
Integration docs
Playbook docs
Script docs

Thanks for the input!

I agree that the RN docs should be unified.
Regarding the other documentation docs, I don't think its a good idea to split it into entities since a lot of the things there are relevant for all entities.
I'll create a 'Documentation best practices' section with the relevant docs, and I will make sure there are no duplicate content there.
Will ping you soon to review.

Thanks for the input!

I agree that the RN docs should be unified.
Regarding the other documentation docs, I don't think its a good idea to split it into entities since a lot of the things there are relevant for all entities.
I'll create a 'Documentation best practices' section with the relevant docs, and I will make sure there are no duplicate content there.
Will ping you soon to review.

@anara123 @fvigo Would love to hear your thoughts

@anara123 @fvigo Would love to hear your thoughts

@reutshal Done

@reutshal Done

@reutshal looks good, thanks. I think we should add some cross references in other sections to these docs, as well as move the Pack Documentation to the Documentation section. I will do the changes and update the PR.

@reutshal looks good, thanks. I think we should add some cross references in other sections to these docs, as well as move the Pack Documentation to the Documentation section. I will do the changes and update the PR.

@reutshal done. I reshuffled the files a bit and put them in a "documentation" directory.
I made it so in the sidebar that those articles are referenced in more sections. For example the Pack README file shows up in the sidebar in both the Content Packs and in the Documentation section.
I like it but maybe it could be confusing to some. What do you think?
@anara123 @ShahafBenYakir please chime in as well.

@reutshal done. I reshuffled the files a bit and put them in a "documentation" directory.
I made it so in the sidebar that those articles are referenced in more sections. For example the Pack README file shows up in the sidebar in both the Content Packs and in the Documentation section.
I like it but maybe it could be confusing to some. What do you think?
@anara123 @ShahafBenYakir please chime in as well.

@fvigo looks ok 👍

@fvigo looks ok 👍

@reutshal looks good.
@fvigo can you make it not to jump to the first link. For examples when I press README under Documentation, it jumps to the first README under Integrations & Scripts and that is a bit annoying

@reutshal looks good.
@fvigo can you make it not to jump to the first link. For examples when I press README under Documentation, it jumps to the first README under Integrations & Scripts and that is a bit annoying

@anara123 unfortunateluy I don't think it's possible.
I've added some redirects article, see how it loks like.

@anara123 unfortunateluy I don't think it's possible.
I've added some redirects article, see how it loks like.

@fvigo Looks great!

@fvigo Looks great!

@anara123 please check and approve if you like it! tnx

@anara123 please check and approve if you like it! tnx

Everyone, please take a final look if needed, I want to merge by the end of the day

Everyone, please take a final look if needed, I want to merge by the end of the day


Tag Zebra NU5 mainnet version

Motivation
We need to tag a version of Zebra for NU5 mainnet activation.
Specifications

Designs

Related Work

Tagged!


Create a conda-forge recipe for the installation

Description of proposed feature
It would be nice if the installation was easier to use. My suggestion would be to create a conda-forge recipe for manim that contains all the dependencies and installation instructions.
How can the new feature be used?
When using conda recipes, one would only need to execute conda install -c conda-forge manim to install manim and all its dependencies instead of following several steps for the installation which also differ depending on the operating system.

I looked a bit into it and the most problematic dependency is skia-pathops. It seems like a nightmare to port it into conda-forge since it completely redefines the build process and doesn't use standard tooling.
The most feasible way would probably be to get rid of this dependency/rewrite the setup part of skia-pathops but this will probably take a lot of time...
See fonttools/skia-pathops (issue 56) and conda-forge/staged-recipes (pr 19143).

I would like to see information about, why the installation was not easy to use and on which operating system you had problems.

I didn't have any problems, but I believe it can be made more accessible by integrating it into conda. Then, one wouldn't need to worry about additional dependencies. Also, you bloat your system when installing other dependencies like pango or scipy directly on your system instead of a virtual environment.

But that seems more like a general issue with python which every user has to deal with no matter the library or program. I don't really see how an extra installation of something else solves accessibility. It just makes it even more steps. We might think about putting it in a virtual environment if that is a possibility with the default python packaging system.
Else I don't see why we should start maintaining another way of installing manim, because 5 are already enough in my opinion.
And yes it would be really nice if python could finally deal with dependencies in the installation.
But there are too many inexperienced users which are not even familiar with the normal python world. And i don't think conda will solve accessibility for them because conda will have as many issues as the other installation method in one form or another.
(Just my opinion maybe the other devs have a different take on that)
But yeah, adding more steps doesn't make it easier. It doesn't solve ffmpeg and latex too.

It doesn't solve ffmpeg and latex too.

It doesn't solve LaTeX, that's right. But ffmpeg can just be added as another conda dependency for manim that gets installed automatically.
One advantage of using conda would be that you just need to reference conda install -c conda-forge manim instead of all the different installation methods because conda does the platform-dependent stuff automatically for you.


hybrid search implementation

Hi @ChrisCleaner, I'd like your review on this PR. It implements "hybrid search", a searching strategy in the vector database that combines semantic search with keyword search, see also: https://python.langchain.com/v0.1/docs/modules/data_connection/retrievers/ensemble/

Should we write a separate Unit Test for the hybrid search? Or do you reckon it is not necessary right now?

Hi @ChrisCleaner, yes good idea. I had not thought of that yet...

@StefanTroost Should I merge it without the Unit Test alerady or do I wait?
(I could also write a unit test for it if neessary)

Well adding a unit test is a valuable addition. If you have time to add one, that would be great!
From: ChrisCleaner @.>
Sent: Tuesday, July 23, 2024 1:52 PM
To: pbl-nl/appl-docchat @.>
Cc: Troost, Stefan @.>; Mention @.>
Subject: Re: [pbl-nl/appl-docchat] hybrid search implementation (PR #142)
@StefanTroosthttps://github.com/StefanTroost Should I merge it without the Unit Test alerady or do I wait?
(I could also write a unit test for it if neessary)
—
Reply to this email directly, view it on GitHubhttps://github.com/pbl-nl/appl-docchat/pull/142#issuecomment-2245024629, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABXEHWFUOUELRAR5XWI5N7DZNY7UZAVCNFSM6AAAAABLJXF2FSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBVGAZDINRSHE.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


[UIMA-6243] JCas returns Annotation when asked for a specific subtype

Fix the case where types are overwritten via a classloader passed to the resource manager (UIMAv3-specific solution)
Added corresponding unit tests (same ones we used for the UIMAv2 version of this issue)

No feedback so far. Assuming lazy consensus. Merging.