AssertionError: Invalid file size while trying to download .hgt file

I have been using this package for a while now to perform elevation based operation on Geospatial data and it has been very useful! But recently when I tried to fetch SRTM elevation for longitude, latitude as  35.003152, 24.056313 respectively, I got the following error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\Hari\AppData\Local\Programs\Python\Python36\lib\site-packages\srtm\data.py", line 57, in get_elevation
    geo_elevation_file = self.get_file(float(latitude), float(longitude))
  File "C:\Users\Hari\AppData\Local\Programs\Python\Python36\lib\site-packages\srtm\data.py", line 111, in get_file
    result = GeoElevationFile(file_name, data, self)
  File "C:\Users\Hari\AppData\Local\Programs\Python\Python36\lib\site-packages\srtm\data.py", line 345, in __init__
    assert square_side == int(square_side), 'Invalid file size: {0} for file {1}'.format(len(self.data), self.file_name)
AssertionError: Invalid file size: 70 for file N24E035.hgt

I was trying to find a solution for this and stumbled upon a similar issue which was raised about 7 years back. The exception that is raised seems to be same but the cause of error is different.
Currently, I am circumventing the issue by manually downloading the .hgt file mentioned in the trace log from https://dds.cr.usgs.gov/srtm/version2_1/SRTM3/Africa/.
Thanks again for the great work!

Not sure why that happened, but if that file in your ~/.cache/srtm directory is really 70 bytes long -- can you try delete it manually and re-run that script. If it works then a fix for this problem is to just delete files with invalid size and let the library re-download them again.

Deleting it manually and re-running the script did not fix the problem. The same exception is being raised.

I am running the following script that is raising the mentioned exception:
import srtm

srtm_data = srtm.get_data()
srtm_data.get_elevation(24.056313, 35.003152)

Ahhh... Right. I tried to reproduce it with 35.003152, 24.056313 but then I figured out that's not "lat and lon" but "lon and lat".
Anyway... Should be fixed with this https://github.com/tkrajina/srtm.py/commit/4a6dbf7b5c4d79024f9869da4157db46d2c203a7

Wow. That was a quick fix! Thanks a ton.


Ensure exec command can find config.json

Partially fixes #584. While fixing this I discovered another issue, which I will follow up in the next PR.

@Furisto I'm sorry, but could you rebase?

Codecov Report

Merging #616 (31c30af) into main (63f77b2) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #616      +/-   ##
==========================================
- Coverage   70.01%   69.99%   -0.02%
==========================================
  Files          82       82
  Lines       10933    10936       +3
==========================================
  Hits         7655     7655
- Misses       3278     3281       +3

@utam0k PTAL

@Furisto Thanks for your PR 😍
Doesn't this break the exec command? The config.json should not be in the bundle directory. I tried the following command and it failed.
$ sudo podman run --name test -it busybox sh
/
# and in another console
$ sudo podman exec -it test sh

@utam0k The exec command is already broken. This fixes one of the issues (missing config.json), but there is another issue which I alluded to the in the description of this issue which breaks the exec command later on.

The config.json should not be in the bundle directory.

Unless I am misunderstanding your meaning, it is in the bundle directory.

@utam0k The exec command is already broken. This fixes one of the issues (missing config.json), but there is another issue which I alluded to the in the description of this issue which breaks the exec command later on.

The config.json should not be in the bundle directory.

Unless I am misunderstanding your meaning, it is in the bundle directory.

Oh, my local was wrong.


Refactor govuk-visually-hidden Sass

A neat little optimisation that we can probably make now that we've dropped support for older IE versions.
Currently, we have two mixins that share mostly identical styles: govuk-visually-hidden and govuk-visually-hidden-focusable, with a few of differences:

govuk-visually-hidden uses pseudo-elements to add spaces before and after the visually-hidden content.
govuk-visually-hidden-focusable applies the visually-hidden styles by default, but then attempts to undo these styles when the element receives keyboard focus or is in the active state.
govuk-visually-hidden-focusable additionally omits a couple of the visually-hidden styles on the basis of not wanting to undo them upon focus or active states.

Having two almost-identical mixins isn't ideal when it comes to code duplication. Trying to manually revert styles upon a particular state is also quite untidy and evidently is something we've had problems doing before, given we've needed to explicitly omit some styles to not unintentionally revert the styles we do want.
Now that we've dropped support for Internet Explorer 8 and 9, we have the option of using the :not selector to only apply the visually-hidden styles when the element is not focused or active, allowing both mixins to use the same set of core styles and allowing for the complete removal of the manual revert code. 🛹
This would close #1597.
Changes

Moves styles common to both mixins into an internal helper mixin.
Changes govuk-visually-hidden-focusable to only apply the visually-hidden styles when the target element is :not focused or active, rather than setting the styles by default and then overriding them.

Thoughts
This uses two chained :not selectors to achieve the desired result: :not(:active):not(:focus). This could be just one selector, :not(:active, :focus), but IE 11 doesn't support this syntax and PostCSS doesn't appear to be converting it to the syntax that IE 11 understands.
I've named the internal mixin _govuk-visually-hidden-text. This is technically a misnomer as it could hide any element or set of elements, but simply calling it _govuk-visually-hidden seemed too likely to cause confusion. Alternative naming suggestions welcome.

I've renamed the internal mixin to _govuk-visually-hide-content, to make it a bit more distinct from the others. Helps to understand it by making it more verb-y too, I guess?

@selfthinker would you prefer having further testing with a wider range of browser/assistive tech combinations?

I would prefer to test more. That's mostly because I know we had issues with the visually hidden class before that would only happen in a specific screen reader / browser combination. Better to be safe than sorry.
Where can I best test this?

Where can I best test this? I can do that tomorrow.

@querkmachine may have better options
The govuk-visually-hidden-focusable mixin is used by the Skip Link (for comparison, here's the Skip Link example for main)
We have quite a few applications of the govuk-visually-hidden class in the different components and examples.

Thinking something like the Bank Holidays example may be interesting as it's used to hide the table headings (which would be announced when users navigate the table).
For a simpler example where the content is not referenced by anything, the Error Message component has a visually hidden 'Error' before the actual message. Again for comparison here are the Bank Holidays example for main and the Error Message component for main

Where can I best test this? I can do that tomorrow.

@romaricpascal, sorry, I meant where on the internet. Is this PR deployed somewhere?
I could potentially also copy the CSS and create an example page that uses it in several different scenarios.

https://govuk-frontend-pr-5063.herokuapp.com/
There's an automated deployment under the 'View deployment' button (currently) a few comments above this one. GitHub's interface be at it again.

There's an automated deployment under the 'View deployment' button

I thought it would be deployed automatically, and I was looking for such a link or button. But either I didn't see it or I didn't understand that that is what it did. Thanks for pointing that out.

I've just tested the three examples suggested by Romaric and they behave exactly like before, so everything looks good to me. :+1:
(And while I was doing that I noticed that it's not a good idea to visually hide table headers as most screen readers ignore them when navigating data cells. But that was already an issue before and doesn't have anything to do with this implementation.)


initial typescript support

Resolves #456
I've used this TS starter from @frol to support typescript in this repo. The process is as follows:

copy all jsx files to build/src
transpile the tsx file to jsx files to build/src
replace Account Id Contract id and network with replace.js
replace exports with build.js

This way we don't have to move everything to typescript now but we can do it file by file

@frol @elliotBraem any feedback on this?

Overall, looks great to me, though i feel we also need to add some watch-io script command to package.json, so during local development we don't need to manually rebuild after each edit.

Thank you for the review. Which package are you referring to? I would use watchman, let me know if you have a better solution. I couldn't find watch-io.

So from my testing so far in Codespaces ( Linux ), you need to install watchman ( using brew ) and cpio. If possible it would be good to use javascript to obtain the same so that an npm install is sufficient. Otherwise we need clear instructions that informs about what needs to be installed, or a devcontainer.json for vscode / github codespaces.

Thanks Peter and Elliot for the review. I'll make the necessary adjustments to the code.

I'll switch to the Node.js package from watchman to streamline the process within Codespaces.
I'll transition away from using cpio and make use of cp instead, ensuring a more straightforward setup.

Update: on Mac the watchman package will throw a Operation not permitted error the first time if the project is stored inside the Documents folder. Which can be resolved by running npm run clear:watch which I added to the watch script.
 The tokio-runtime-worker paniced because the bos-loader is serving from the build/src folder. That isn't created for you but instead /neardevhub-widgets I think copying the files with cpio might have been the issue. I now just the cp command instead. So could you try again?

This is looking very good, is the intellisense working with TypeScript files?

hey @frol @Tguntenaar, I just tested it locally and the changes are reflected as expected.  👍

@Tguntenaar Can you resolve the conflict and we'll merge it!

Waiting to resolve conflicts then will merge


Windows NodeJS x86: ed25519.cc(78): error C2466: cannot allocate an array of constant size 0

ed25519.cc
..\src\ed25519.cc(78): error C2057: expected constant expression [C:\Program Files (x86)\nodejs\node_modules\homebridge\node_modules\HAP-NodeJS\node_modules\ed25519\build\ed25519.vcxproj]
..\src\ed25519.cc(78): error C2466: cannot allocate an array of constant size 0 [C:\Program Files (x86)\nodejs\node_modules\homebridge\node_modules\HAP-NodeJS\node_modules\ed25519\build\ed25519.vcxproj]
..\src\ed25519.cc(78): error C2133: 'signatureMessageData' : unknown size [C:\Program Files (x86)\nodejs\node_modules\homebridge\node_modules\HAP-NodeJS\node_modules\ed25519\build\ed25519.vcxproj]
..\src\ed25519\sc_reduce.c(274): warning C4244: '=' : conversion from 'crypto_int64' to 'unsigned char', possible loss of data [C:\Program Files (x86)\nodejs\node_modules\homebridge\node_modules\HAP-NodeJS\node_modules\ed25519\build\ed25519.vcxproj]
Line77 	unsigned long long sigLen = 64 + messageLen;
Line78 	unsigned char signatureMessageData[sigLen];
Line79 	crypto_sign(signatureMessageData, &sigLen, messageData, messageLen, privateKey);

package.json
"dependencies": {
"ed25519": "git://github.com/dazoe/ed25519",
some developer / company called @stellar (stellar on npm) cloned the repository and published the ed25519 module under their name.
To me, this is a non-acceptable behavior. Nevertheless, someone wrote the patch for Node 0.12 / iojs; that is, he just published a version that is on-par with master.
npm install ed25519-native
also this might also work to get the latest.
npm install https://github.com/dazoe/ed25519/tarball/master

I gave up trying to compile it using nodeJS for windows. it probably would have worked eventually.   The best way is to just build a virtual box for ubuntu running on windows. or a faster way is just to buy a raspberry pi2 for $35USD


Do not transform valid input

This line changes 1979-08-15 to  1979-08-15T00:00:00.000Z which doesn’t match the RFC_3339_REGEX in validator.js
If it is correct that this date string is transformed to a datetime string, the fix should be made in the serialize function.

In addition, the test implies that the valid input should be transformed into a Date object. Is this graphql 's responsibility or should this be done in the database/crud layer?
In my case I store dates as a plain string in JSON files, not as a Date object.

Hey @timvanoostrom, thanks for your PR! When parsing a date input it always gets converted to a Date instance. The code as it stands now is the intended behaviour.
Everyone's use cases differ when it comes to parsing these value, some prefer a Date instance whilst others, like yourself, may prefer a string representation for use in your code. There are several things that you can do:

If you wish to stores these dates as plain string in a JSON file then you use a converter that maps a Date to a date-string. This is easy to write.
You can create a fork of this repo and make the desired changes there. You can then publish your version on npm and use it.
You can copy the source code in this project directly into your project and make the desired changes. This library only consists of a few files so it shouldn't be too difficult to port it.

I hope I helped in solving your problem! Let me know if you have any questions:)

Sure!


fix resource controller run more than one

Signed-off-by: lmxia<EMAIL_ADDRESS>
What type of PR is this?
kind/bug
What this PR does / why we need it:
resource controller should be added before it runes otherwise it will never be added. And controller cache won't work.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Codecov Report

Merging #352 (785902c) into main (c675d8d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #352   +/-   ##
=======================================
  Coverage   12.48%   12.48%
=======================================
  Files          47       47
  Lines        4861     4861
=======================================
  Hits          607      607
  Misses       4210     4210
  Partials       44       44

Impacted Files
Coverage Δ

...ernet/clusternet/pkg/hub/apiserver/shadow/utils.go
89.28% <0.00%> (ø)

Continue to review full report at Codecov.

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


Bicaridine and Brute Rework

About the PR

GONE are the days of medical doctors injecting bicaridine... dermaline... all that jazz.. without ANY THOUGHT WHATSOEVER. The only thing they've ever had to think about was overdosing. This PR introduces five new reagents:
1: Razorium: A non-newtonian fluid that becomes very sharp when force is applied. Do your best not to slip in this, as it's like falling in razor blades. This chemical is produced by a reaction between Bicaridine and Dermaline. If you wish to heal brute and burn at the same time, you'll have to resort to some of the new, more specific (and less reactive) chemicals.
2: Puncturase: A fizzy chemical that is very effective at healing piercing damage! However, it does leave some blunt tissue damage in its wake.
3: Bruizine: Originally developed as cough medicine, scientists realized that bruizine is very good at treating blunt-force trauma!
4: Lacerinol: A fun little chemical that boosts collagen synthesis to heal slash damage.
5: Agitaseptici: A strange, airborne space bacteria that was recently discovered to be incredibly resilient and incredibly good at contaminating stuff.  Agitaseptici will be produced when you mix any two brute medications. (ex. Bicaridine and lacerinol, or puncturase and bruizine.) This bacteria isn't super dangerous, but it should be removed from your bloodstream with charcoal or ipecac as soon as possible to avoid complications.
Why / Balance

Medical is kind of boring and chemistry doesn't have much to do. Doctors only really think about overdosing, this should add a bit of strategy into the mix. Doctors must be much more keen about their dosages, lest they risk an Agitaseptici infection. However, if they are successful, they may find their patient recovering faster than if they'd used Bicaridine.
Through this and some other PRs I hope to add some depth to chemistry in a somewhat interesting way using chemicals that probably couldn't be trusted in the hands of your average LRP powergamer. The end goal is to also provide new ways for danger and conflict to arise without any major antags.
Technical details

I added a bunch of reagents and flavors it's not really too interesting.
Media

https://github.com/cosmatic-drift-14/cosmatic-drift/assets/137215465/c2291244-6d70-4d8c-af2e-fb5fa960593f

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

Make sure to take this Changelog template out of the comment block in order for it to show up.
:cl:

add: Added three specific chemicals to target each group of brute damage.
add: Added Razorium, a sharp and dangerous chemical created when Bicaridine and Dermaline are mixed!
add: Added Agitaseptici, a strange and mildly dangerous bacterium found infecting mixed brute medication.

I will see if i can put together a guidebook entry ive never made one of those before. The chemical section does auto-update, though

I honestly think that the three specific brute chems could be upstreamed, but that’s not the biggest deal ever. I’ll review this more later.

i hope i didnt just explode everything

we ball


Avoid .R file lock on windows

This PR avoids a file lock issue (observed on windows, probably not an issue on linux): Sourcing .R file using R -f xxx.R locks the file and as a consequence it's impossible to reinstall this extension while it is running. This is particularly annoying when working on the extension itself, since it contains R code and is therefore activated automatically.
This problem is solved by sourcing .R files using R -e source('xxx.r'), which does not appear to lock the file.

Not sure if it is a good idea to use source() instead of direct supplying a file. Changing the content of an R file while it is running by Rscript might cause the R session to fail sometimes.

This problem would be solved by using source(), since source first parses the entire file and then starts executing. Therefore, background processes like the languageserver and helpserver do not depend on the original file at all, once they start running. IMO this is the ideal behavior. In contrast, on windows Rscript locks the file, making it impossible to quickly reinstall the extension.
On Linux/macos, I'm not sure how exactly this is handled by Rscript. Apparently it is possible to write a script that deletes itself (and does something afterwards) and run it with Rscript, but I'm not sure what the limits to this are (i.e. whether Rscript always reads the entire file first, or I just got lucky and my test script was short enough to be read at once).


home:software: use home-manager's bat

Darwin tests broken by the rip out of my keys. Will fix it later.


827/introduce confirm popup

feat: introduce confirm popup
Confirm popup that takes a '\n' separated string message renders
a popup requesting user for confirmation

The message is displayed as a Paragraph with vertical scrollbar (if
needed)

  - Lua binding: ya.confrm(title, message, position)
  - Position::try_from now parses "h" value (default 3)

Thanks for your changes @sxyazi, they all make sense.
I think we can still improve the quit() scenario when there are running tasks. Now the confirm popup only shows after 3 seconds but there is no feedback to the user about this.
Do you see a way to get the same behavior using the select! as with the Input popup? If not I suggest we at least show a Notify to let the user know that something is happening.

Could this be used for the bulk rename confirmation?


adding SpellCraft Unity code

@UnwashedPine851 If everything you implemented is present and working in Michael's branch, you should probably delete this branch to avoid confusion. I'm gonna close this PR. @mwong365 make a PR for your branch so that I can add it to main and start working on serial communication.


Improve on assert recognition

@sonmarcho FYI a new form of assert is now generated by Charon: if not e1 then abort msg else () is now recognized as Eurydice_assert(e1, msg).

Ok, I missed this one. I will also add it on Charon's side.

No need to change anything, I can now deal with it too and it will be emitted properly. I think the current situation is ok. Thanks!


Correct the API link for http ExtAuthz filter documentation

The original one was pointed to network instead of http one
For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]

Please fix DCO, thanks!
/wait


GM title returns link for online, but link is broken

The superintendent of the future :strategy and action for achieving academic excellence
http://findit.library.gwu.edu/item/m767946
The link goes to http://findit.library.gwu.edu/item/OT FOUND

I reported to WRLC that this GMU record is missing from Voyager.

Resolved by fixing bad data. http://findit.library.gwu.edu/item/m767946


CTV News Canada - Video plays in browser, but youtube-dl gives 404?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.08.09. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.08.09

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[x] Question
[x] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue
404 not found error in youtube-dl attempting to download from CTV Canada. No account needed and not likely to be a geoblock issue, as video does play in U.S. Not sure if problem with Flash video container or if I'm entering the wrong URL for the video extraction. Extended description below.

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
C:\>youtube-dl -v "http://www.ctvnews.ca/w5/justin-trudeau-man-who-would-be-pm-discusses-his-past-his-politics-1.2059548" --ffmpeg-location "C:\Program Files (x86)\youtube-dl\ffmpeg\bin"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'http://www.ctvnews.ca/w5/justin-trudeau-man-who-would-be-pm-discusses-his-past-his-politics-1.2059548', '--ffmpeg-location','C:\\Program Files (x86)\\youtube-dl\\ffmpeg\\bin']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2017.08.09
[debug] Python version 3.4.4 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-86175-g64ea4d1, ffprobe N-86175-g64ea4d1
[debug] Proxy map: {}
[9c9media] 5: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpf7t6lw0k\build\youtube_dl\extractor\common.py", line 502, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpf7t6lw0k\build\youtube_dl\YoutubeDL.py", line 2166, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: http://www.ctvnews.ca/w5/justin-trudeau-man-who-would-be-pm-discusses-his-past-his-politics-1.2059548

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

Hi, can you help me grab a video from channel CTV News in Canada? I know it's not a geoblock issue, as I'm in the U.S. and can watch the video in the web browser. It appears to be embedded in some kind of proprietary Flash Video widget or container. But it keeps freezing up and hiccupping, so I want to download to watch the program offline.
So far, I have tried using various Firefox add-ons, like Video Download Helper and DownThemAll AntiContainer, and looking through the page source, but nothing helps. I even tried playing it in fullscreen and recording the screen using a screen recorder program, but as the video itself keeps glitching (either due to something wonky about Flash Video or just because I have slow Internet), the recorded video is incomplete and unwatchable.
Is there a different URL I should be entering in youtube-dl itself? The one I put in is here:
http://www.ctvnews.ca/w5/justin-trudeau-man-who-would-be-pm-discusses-his-past-his-politics-1.2059548
I've included the verbose log of what happens when I try to download it. Not sure why I'm getting a 404 error, as the video does still exist.

Kinda a 1/2 solution for you....
Click the i in a circle icon (Show Info) and you will get the direct link to the video:
http://www.ctvnews.ca/video?clipId=469865
Youtube-dl can download from the direct link
however the ffmpeg merge failed for me...had to manually merge two files myself:
C:>youtube-dl.py http://www.ctvnews.ca/video?clipId=469865
[9c9media] 469865: Downloading JSON metadata
[9c9media] 469865: Downloading JSON metadata
[9c9media] 452957: Downloading JSON metadata
[download] Downloading playlist: W5: Trudeau II, part one
[9c9media] playlist W5: Trudeau II, part one: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[9c9media:stack] 810597: Downloading m3u8 information
[9c9media:stack] 810597: Downloading f4m manifest
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 222
[download] Destination: W5 - Trudeau II, part one-810597.fhls-1856.mp4
[download] 100% of 203.48MiB in 02:43
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 222
[download] Destination: W5 - Trudeau II, part one-810597.faac_96000-English.mp4
[download] 100% of 10.16MiB in 00:49
[ffmpeg] Merging formats into "W5 - Trudeau II, part one-810597.mp4"
ERROR: Conversion failed!

The i icon (Show Info) no longer gets the direct link to the video
Eg: https://montreal.ctvnews.ca/montreal-s-indian-community-and-allies-protest-violence-in-new-delhi-1.4834214


config(ticdc): disable pull based sink

What problem does this PR solve?

Issue Number: close TBD
What is changed and how it works?
disable pull-based sink and update tests.
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
No
Do you need to update user documentation, design documentation or monitoring documentation?
No
Release note
None

/run-all-tests

/run-all-tests

/run-verify

/run-integration-test

/run-all-tests


[WIP] Split DAB into multiple applications

This PR is largely a restructuring of django-ansible-base.
All top level folders are ansible-base are now "features" of dab which can be independent django applications.
All tests have beed moved into the test_app and restructured to follow the directory layout of ansible-base.
All loggers have been updated to reflect the correct files they hail from.
WARNING: This resets the migration files, splitting the authenticator migrations into the authentication app.
Fixes https://github.com/ansible/django-ansible-base/issues/61
TODO:

[ ] Fix remaining failing tests
[x] Update README.md files as needed
[ ] Incorporate service_lib (stretch goal, could be done in a separate PR if needed)

This is really hard to review in github because my comments are all structural.
Generally speaking:

ansible_base should not be a python app itself. It should just be a container for the apps and shared libraries.
We need a clear distinction between what is an app and what is a library. I would recomend either: put all of the non app modules (channels, settings, filters, etc. in common) or split ansible_base into two directories: apps and lib'. Personally I like referencing the apps like ansible_base.apps.authentication`. That seems like the clearest option.

@newswangerd I like to use django.contrib as a reference point. That doesn't further bury things under apps, and I think that would be confusing. The default assumption is that all the modules in there are apps. Some apps like django.contrib.contenttypes don't have migrations, but they can use other app features, like providing management commands like collectstatic.

Link https://github.com/john-westcott-iv/django-ansible-base/pull/1

If I show migrations...

showmigrations

admin
 [ ] 0001_initial
 [ ] 0002_logentry_remove_auto_add
 [ ] 0003_logentry_add_action_flag_choices
auth
 [ ] 0001_initial
 [ ] 0002_alter_permission_name_max_length
 [ ] 0003_alter_user_email_max_length
 [ ] 0004_alter_user_username_opts
 [ ] 0005_alter_user_last_login_null
 [ ] 0006_require_contenttypes_0002
 [ ] 0007_alter_validators_add_error_messages
 [ ] 0008_alter_user_username_max_length
 [ ] 0009_alter_user_last_name_max_length
 [ ] 0010_alter_group_name_max_length
 [ ] 0011_update_proxy_permissions
 [ ] 0012_alter_user_first_name_max_length
authentication
 [ ] 0001_initial
contenttypes
 [ ] 0001_initial
 [ ] 0002_remove_content_type_name
sessions
 [ ] 0001_initial
social_django
 [ ] 0001_initial (2 squashed migrations)
 [ ] 0002_add_related_name (2 squashed migrations)
 [ ] 0003_alter_email_max_length (2 squashed migrations)
 [ ] 0004_auto_20160423_0400 (2 squashed migrations)
 [ ] 0005_auto_20160727_2333 (1 squashed migrations)
 [ ] 0006_partial
 [ ] 0007_code_timestamp
 [ ] 0008_partial_timestamp
 [ ] 0009_auto_20191118_0520
 [ ] 0010_uid_db_index
 [ ] 0011_alter_id_fields
 [ ] 0012_usersocialauth_extra_data_new
 [ ] 0013_migrate_extra_data
 [ ] 0014_remove_usersocialauth_extra_data
 [ ] 0015_rename_extra_data_new_usersocialauth_extra_data
test_app
 [ ] 0001_initial
 [ ] 0002_team_organization

And look at the admin pages:

There's a whole lot of auth stuff going on. Perhaps "authenticators" is too vague here. Maybe we can put in a long name to better distinguish it, like how the auth app does "AUTHENTICATION AND AUTHORIZATION". I think the focus is on auth plugins?

I might be getting a little annoying with these, but this is the last of the low-hanging fruit here https://github.com/john-westcott-iv/django-ansible-base/pull/2
Also, since you're rewriting migrations I want to go ahead and get these changes in. Might also be good to touch up some help_text too.


Feature parity "interpolate" expressions for projectionDefinition

Adding a ticket, so that it can be linked from the spec - to satisfy integration test suite.

Yes, here are some link to the docs for this new part of the spec:
https://maplibre.org/maplibre-style-spec/projection/
https://maplibre.org/maplibre-style-spec/types/#projectiondefinition


Release 4.1.0

Refs #400

Could we have #462 in there as well?

Sure thing, :eyes:

@charettes Once #462 is merged the version number is derived from the Git tag, not needed to be updated, using setuptools_scm. The rest of the release process stays the same.

This should have been a new major version (5.0.0) because it has some breaking changes (like dropping django 2.1)
Took me quite some time to figure out why my CI/CD pipeline wasn't working anymore :wink:


Logging improvements

Please have a look at the individual commit messages for the details.

We use this order and formatting everywhere else in the code:

Done.


Compilation fails with null on http service code with records

Description:
Can be reproducable with below code. May be due to cyclic fields in record, but issue seems intermittent too.
Steps to reproduce:
import ballerina/http;

type AidPackage record {|
    int packageID = -1;
    string description;
    string name;
    string status;
    AidPackageItem[] aidPackageItems?;
|};

type AidPackageItem record {|
    int packageItemID = -1;
    int packageID;
    int quotationID;
    int needID;
    decimal totalAmount =0;
    AidPackage aidPackage?;
|};

service /test on new http:Listener(9090) {
    resource function get items() returns  AidPackage[]|string {
        return "Test";
    }
}

Output
error: compilation failed: The compiler extension 'io.ballerina.openapi.extension.OpenApiCompilerPlugin' failed to complete. null

Affected Versions:
2201.0.4

Analyzed the issue and it was identified that java.lang.StackOverflowError error was thrown from OpenAPI converter. This is an issue with Ballerina OpenAPI tool.


[Bug]: Hello, probably not a bug, but

evernote-backup version
1.9.3
What OS are you using?
Windows
OS Version / Linux distribution
Windows 11
Bug description
Hello, I installed evernote-backup in Pycharm, then I call import evernote-backup, this works, but when I do evernote-backup init-db, it says invalid syntax
Log excerpt
File "C:\Users\cyril\PycharmProjects\EvernoteBackup\main.py", line 2
    evernote-backup init-db
                    ^^^^
SyntaxError: invalid syntax

Hello, use evernote-backup init-db --oauth


Maintenance error handling and conflict tweaks

Includes:

adding custom sidekiq middleware to sleep and retry rather than booming into the retry queue
terminating conflicting database queries after initial maintenance sleep to (hopefully) remove possibility of deadlock
more specs :tada:

This is working correctly in staging and production :tada:


How to use a double with an Instance attribute?

Hello,
Is it possible for a double to bypass the Instance check on a dry-struct attribute? For instance:
require 'dry-struct'
require 'spec_helper'

class B
end

class A < ::Dry::Struct
  include Dry::Types.module
  constructor_type(:schema)
  attribute :b, Instance(B).optional
end

describe 'something' do
  it 'test' do
    b = instance_double('B')
    expect(
      A.new(b: b)
    ).to be
  end
end

I've tried to mock several methods from both B and b and was not able to make this test pass.

Please don't use issues for support questions and instead post a message on our forum. See CONTRIBUTING.md for more information.

OK, thank you!


When importing WorkItems with the option "ignore-empty-revisions": true, Counter X of X/Y is not increased

Fixed by https://github.com/solidify/jira-azuredevops-migrator/pull/904


[SPARK-42403][CORE] JsonProtocol should handle null JSON strings

What changes were proposed in this pull request?
This PR fixes a regression introduced by #36885 which broke JsonProtocol's ability to parse null string values: the old Json4S-based parser would correctly parse null literals, whereas the new code rejects them via an overly-strict type check.
This PR solves this problem by relaxing the type checking in extractString so that null literals in JSON can be parsed as null strings.
Why are the changes needed?
Fix a regression which prevents the history server from parsing certain types of event logs which contain null strings, including stacktraces containing generated code frames and ExceptionFailure messages where the exception message is null.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Added new unit test in JsonProtocolSuite.

Hey, @JoshRosen . I didn't introduce a regression. Could you fix your PR description?

This PR fixes a regression introduced in https://github.com/apache/spark/pull/39972

This sounds like misleading, @JoshRosen . Could you point your PR which introduce this regression ?

This PR fixes a regression introduced in https://github.com/apache/spark/pull/39972

This regression was introduced at https://github.com/apache/spark/pull/36885 .

Never mind. I fixed the PR description.

Thank you, @JoshRosen and @mridulm .
Merged to master/3.4.

late lgtm, thanks for the fix!


Cannot customize avatars and tags

I modified avatars.yaml and tags.yaml but the extension doesn't update after refresh or reboot webui and remain as default template.

Oh I just tried cleaning cookie and browse history the problem was solved
Not sure if it is actually related or just dumb luck....

It might be related. But I am not able to repro to find out the root cause. The yaml should be dynamically request.

Closing. Please let me know it happens again.


Incorporate "create heading" functionality to /qath

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have a task to add to a heading that doesn't exist. Meaning I have to first open the note, create a new heading to then add my task. And by then the plugin feels rather useless.
Describe the solution you'd like
I want to be able to call the global NotePlan shortcut, and from it (using /qath) add a task to it on the fly to a heading. And if that heading doesn't exist, the plugin should create the heading, append it to the note and then append the task to that heading.
Describe alternatives you've considered
I didn't, really.
Additional context
For my workflow is fairly important that the heading and the tasks always be appended to the note.
I'd understand if there were an option for the heading/task to be "prepend" to fit the workflow of other users. But for me, it would only work if I could append. Maybe you could ask "on the fly" if it is to be appended or prepended? Or maybe add two new keywords (one to append heading/task and another to prepend)? That'd also be an idea.

Hi, @Duclearc, this functionality got added 2 weeks ago.
I'm closing the issue: please re-open if need be.

Thank you @jgclark for adding this!
One question I do have is "why does this undefined always appears in the command bar"?
It can't be deleted and all my newly created tasks have "undefined" written at the end of them.

Sounds like a problem in the configuration or an edge case I've not thought
of.
Please could you paste the relevant section of the _configuration codeblock
so I can check it out?
On Mon, 13 Sept 2021 at 17:36, Dan @.***> wrote:

Thank you @jgclark https://github.com/jgclark for adding this!
One question I do have is "why does this undefined always appears in the
command bar"?
It can't be deleted and all my newly created tasks have "undefined"
written at the end of them.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NotePlan/plugins/issues/63#issuecomment-918371009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMYESXNSPQNY7EJD55AD6TUBYSAPANCNFSM5DAGBG2Q
.

--
-- Jonathan

Hi, I'm sorry I ended up never replying to this. But one of your updates fixed it, so I never realised you had written something here 😆
Sorry!
But well done! 👏


runtime error OBJECTS_OBJREF_NOT_ASSIGNED_NO when pulling xml mime object

This runtime error occurs when we try to pull a xml mime object which was changed in the remote.

Steps to reproduce:

upload a xml file into the mime repo
stage and commit with abapGit
change the file in the remote (e.g. add an empty line)
try pull the changed file in abapGit -> dump occurs

can you help creating a public repository here on github that reproduces the error?

Sure,
https://github.com/christianguenter2/ISSUE_886
You need to clone the repository and then change the test.xml file on github

Call stack:

@christianguenter2 try again, latest version
happy golfing

Great, thank you.
Feel free to join the golf club ;-)


Use browsingContext.captureScreenshot Bidi command for capturing screenshots

Is your feature request related to a problem? Please describe.
The current reporter uses the WebDriver Classic command for capturing screenshots.
Describe the solution you'd like
Use the WebDriver Bidi command for it as it automatically takes fullpage screenshots. Also as it sends the command via socket it should be a bit faster.
Note: we want to continue to use both, and only use the Bidi command if the session is using Bidi, so use browser.isBidi to check for this.
Describe alternatives you've considered
n/a
Additional context
n/a

Can be closed

Thanks for the contribution 🙏


HR based target

Sorry to misplace my question at first instance.
May I ask more specific if there is a plan to add HR specific range as a target?
I.e.:
..

run: 1.5km @ hr130-150
..
Thank you!

Looks like not a big deal to support. I'll leave this issue open for now.


Unable to connect to a Storage Account from an ARR Account using IAM.

According to the documentation, I assigned three roles to the ARR Account for the Storage Account, but could not connect to the Storage Account.
I received the following response from the REST API.
{
    "error": "Unable to start conversion: Error accessing connected storage account - check if the ARR account has access to your storage account."
}

I got a similar error when using the Conversion.ps1 script.
I have assigned roles as follows, is there a problem with my settings?

The connection using SAS worked fine.

I also receive this error. I was able to convert a file yesterday but continue to get the same error now.

Thanks for the reports!
We are now aware of an internal issue which prevents the Azure Remote Rendering accounts from accessing connected storage accounts.
We are working on a fix and will update once it is deployed.
In the meanwhile please use the conversions/createWithSharedAccessSignature REST API to convert models

We have deployed a fix for this issue. Please try it out again and let us know if it doesn't work. Thanks for reaching out!
FYI, For future debugging, if you can provide us with MS-CV that you receive in the response headers from the REST request, that will be super helpful for troubleshooting.

Thanks for the quick response!
I was able to connect the ARR and Storage Account using IAM. The conversion also works well.
But, I found a new problem. When connected with IAM, the API checking the status of the conversion returns a 500 Internal Server error.
{
    "error": "Unable to get the status of conversion"
}

The MS-CV is as follows.
NrkyMfZCjkSK2pgpOMR1SA.0

@dykarohora can you please make sure that the regions in which you create the conversion and the region in which you query the status are the same? e.g. a conversion created in westeurope (https://remoterendering.westeurope.mixedreality.azure.com) cannot be queried in the southeastasia (https://remoterendering.southeastasia.mixedreality.azure.com) region.
The same is true for sessions as well. All calls to the REST APIs need to go to the same region.

@dykarohora if the issue with the 500 on your conversions still persists please open another issue and provide the MS-CV values where it happens. Since the original issue with the IAM and ARR accounts was solved we will close this issue thread

I'm sorry for the delay in replying.
I tried your advice and the problem was solved. Thank you!


Slider range could use min or max value when out of range instead of throwing an exception.

I've used the RangeSlider, and got an error when the min or max value was out of the minRange or maxRange.
I think it could be more tolerant and use the minRange if the min/max is lower than the minRange, or maxRange if the min/max is greater than maxRange.

Oups, wrong repo sorry...


I'm trying to implement this repo as a local file.

Hi, imsoo.
First, thank you for providing me with a good repo.
As the title says, I want this repo to work as a local file.
So I didn't operate ventilator, worker, and client.
If so, I think I can only use action.py and OpenCV, but I would like to know if it is possible.
My current development environment is Python 3.6 and tf 2.0, and I am going to load the pre trained action.h5 model and apply it to the image.
Thank you for you read my question.

@BBeomH
Thank you for your interest in this repo.

As the title says, I want this repo to work as a local file.
So I didn't operate ventilator, worker, and client.
If so, I think I can only use action.py and OpenCV, but I would like to know if it is possible.

In this repo, we need to make below pipeline.
Client → Ventilator → Worker → (Sink ⇄ Action) → Client

So, you need to run ventilator, worker, sink and client additionally. Try below command.
./sink
./ventilator
./worker ...
./action.py
./darknet_client -addr <IP_ADDRESS> ....

Feel free to ask if you have any questions.

@BBeomH
same problem ,I also get the stream camera to reason, how to do it?


Remove notification from list on dismiss

[x] Remove the notification from the list
[ ] allow a way for the user to pass a dismiss button in overflowButton

+1
the onTouchTap event to remove the list item isnt working

any enhancement about "the onTouchTap event to remove the notification"?

I currently am unable to work on this fix but a PR is welcome :)


dyndns: cloudflare not working with some tlds

CloudFlare dyndns is broken with subdomains or international TLDs with many parts (like .co.uk). The bug exists since commit https://github.com/opnsense/core/commit/55586d3c8ba890d65cc1a2d62839650cfa6aa275 . As the current code was taken from pfsense, they already have a fix for that which can be referred to. (https://redmine.pfsense.org/projects/pfsense/repository/revisions/2446fffa5932e8debcaf165bfaf5492cea429c60)

Bump bump @Nornode did you find a solution at all?

is there a functional version already? just asking. i can offer testing

Can you please reopen this issue? As I said replacing "X-Auth-Key:" with "Authorization: Bearer" solve the issue. There is no need for "X-Auth-Email:"

it looks like it is being fixed here https://github.com/opnsense/plugins/pull/1726/commits/7592767233be98e89b2d38ad3d5abdbe6376bb12 :)


Custom elements upgrade

For https://github.com/Starcounter/RebelsLounge/issues/192

I've updated juicy-select as You asked.
About fork of paper-collapse-item, there was one major change: https://github.com/PatrykSzwer/paper-collapse-item/commit/4e429c8af93be18d3ae9d1979a4a4cc16928159a which provided possibility to use paper-collapse-item header by slot.

About fork of paper-collapse-item, there was one major change: https://github.com/PatrykSzwer/paper-collapse-item/commit/4e429c8af93be18d3ae9d1979a4a4cc16928159a which provided possibility to use paper-collapse-item header by slot.

I understand now. This is a completely legitimate fix. Can you submit it upstream as a PR?

I've created PR for paper-collapse-item. For now, I'll merge this PR.


Unable to run Windows application from .NET SDK command line

Describe the bug
See more details in https://github.com/dotnet/maui/issues/5975
We should be able to run a windows app with dotnet run or dotnet build -t:Run at the very least.
Steps to reproduce the bug

New WinUI app using net6+
dotnet run

Expected behavior
App launches (for both packaged and unpackaged)
Screenshots
No response
NuGet package version
1.1.4
Packaging type
No response
Windows version
No response
IDE
Other
Additional context
No response

There are ways to launch packaged apps from the command line. See https://github.com/dotnet/maui/issues/5975#issuecomment-1234510397

Did @DrusTheAxe's suggestion resolve this issue?

The first step might be to follow several other workloads and make a "Run" target that the dotnet run CLI can use. Having the core sdk know about all the different permutations of different apps might not be that feasable.
So having each framework do the thing, and the core SDK just execute it is probably better - and run may also need a build so a Run depends on Build is super easy.
But as of today, we have nothing. So a Run target is going to be very helpful.

I think the issue I raised here on SO is similar.

It appears if you build and then run an unpackaged app it works a bit better: https://github.com/dotnet/maui/issues/13942

Tracking this internally as well: https://dev.azure.com/microsoft/OS/_workitems/edit/44573471

Still an issue, just fyi.


Reduce the value of the stored points by applying the douglas-peucker-algorithm

What this PR does / why we need it?
Currently, every time the user moves the mouse point, all points are recorded, so a lot of unnecessary data is stored.
When passing, the accumulated value is reduced and stored through the douglas-peucker-algorithm
The following shows the reduced results when applied
as- is
points length
left draw 10: 3185 , right draw 1: 528

to-be
points length
left draw 10: 623 , right draw 1: 89

It seems that the average number of stored points reduced to 1/6
Any background context you want to provide?
douglas-peucker-algorithm
https://cartography-playground.gitlab.io/playgrounds/douglas-peucker-algorithm/
What are the relevant tickets?

Fixes #48
Checklist

[ ] Added relevant tests or not required
[ ] Didn't break anything

@hackerwins Thank you for your comments :)


Failing tests on Ubuntu 16.04 ruby 2.3.1p112

Three failed tests running on Ubuntu 16.04  amd64 / ruby  2.3.1p112
vagrant@ptester:/vagrant/octocatalog-diff$ rake
1 processes for 108 specs, ~ 108 specs per process
......................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................F.F.....F..........
Failures:

OctocatalogDiff::Util::Parallel with parallel processing should parallelize and return task results
Failure/Error: expect(one_time).to be_within(2).of(two_time)
expected<PHONE_NUMBER> to be within 2 of<PHONE_NUMBER>
./spec/octocatalog-diff/tests/util/parallel_spec.rb:43:in `block (3 levels) in <top (required)>'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:112:in `block in run'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:101:in `loop'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:101:in `run'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec_ext/rspec_ext.rb:12:in `run_with_retry'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:30:in `block (2 levels) in setup'

OctocatalogDiff::Util::Parallel with parallel processing should kill running tasks when one task fails
Failure/Error: expect(one_result.status).to eq(nil)
expected: nil
got: true
(compared using ==)
./spec/octocatalog-diff/tests/util/parallel_spec.rb:100:in `block (3 levels) in <top (required)>'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:112:in `block in run'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:101:in `loop'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:101:in `run'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec_ext/rspec_ext.rb:12:in `run_with_retry'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:30:in `block (2 levels) in setup'

OctocatalogDiff::Util::Parallel with parallel processing should kill other tasks when a validator method fails
Failure/Error: expect(result[0].status).to eq(nil)
expected: nil
got: true
(compared using ==)
./spec/octocatalog-diff/tests/util/parallel_spec.rb:237:in `block (3 levels) in <top (required)>'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:112:in `block in run'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:101:in `loop'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:101:in `run'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec_ext/rspec_ext.rb:12:in `run_with_retry'
./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.5.0/lib/rspec/retry.rb:30:in `block (2 levels) in setup'

Finished in 4 minutes 30.6 seconds (files took 1.59 seconds to load)
1097 examples, 3 failures
Failed examples:
rspec ./spec/octocatalog-diff/tests/util/parallel_spec.rb:8 # OctocatalogDiff::Util::Parallel with parallel processing should parallelize and return task results
rspec ./spec/octocatalog-diff/tests/util/parallel_spec.rb:79 # OctocatalogDiff::Util::Parallel with parallel processing should kill running tasks when one task fails
rspec ./spec/octocatalog-diff/tests/util/parallel_spec.rb:213 # OctocatalogDiff::Util::Parallel with parallel processing should kill other tasks when a validator method fails
1097 examples, 3 failures
Took 272 seconds (4:32)

Hey @eelcomaljaars !
Thanks for the note. The problem you are observing here is due to these particular tests being based on the time an action takes to complete, to determine whether a process was killed or ran to completion. In this particular case it appears that the tests took longer on your machine than they should have, resulting in processes actually exiting rather than being killed. I'll look into eliminating the time-based nature of these tests to make them more robust. However I can say that you can almost certainly ignore these failures.


Robotium script not working but still running.

Robotium script is running but it is not performing any clicks on the desired buttons.

I am also facing the same issue.
This issue is more in Nougat. I am using Nexus 6P with Nougat. ClickOnText() is not working. It simply passes the click step without clicking.

What happens if you use clickOnView() instead?


Fix Alt+F2 (Linux)

Alt+F2 does not work because of the incorrect keycodes. This fixes things.

134 is Right ALT
130 is Left ALT
I will double check soon.  Thanks for the heads up

So why is Alt+F2 the only combination using the wrong alt?
I checked a select set (~20) of English keyboard layouts and 100% have alt_l on the keyboard, but only 20% have alt_r mapped. 80% have something else mapped to that button.

No big deal. It was not intentional.

Hopefully I used 130 in all of the payload examples.


[UI] Broken image link in Provider modal.

Current Behavior
The "About Providers" modal has a broken image.
https://github.com/meshery/meshery/assets/79860504/11611363-1d21-450d-9822-35657bacee68
Expected Behavior
An external link image should be displayed.

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@MUzairS15  can you provide the link of the page ?

@leecalcote  Assign this issue to me!

Hi since you were the first to request assignment, you've got it! Also please remember to take a look at the CONTRIBUTING.md if you have not already.


When I press F10

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
File "D:\Anacoda\lib\site-packages\spyder_profiler\widgets\profilergui.py", line 261, in
self.finished(ec, es))
File "D:\Anacoda\lib\site-packages\spyder_profiler\widgets\profilergui.py", line 326, in finished
self.show_data(justanalyzed=True)
File "D:\Anacoda\lib\site-packages\spyder_profiler\widgets\profilergui.py", line 347, in show_data
self.datatree.load_data(self.DATAPATH)
File "D:\Anacoda\lib\site-packages\spyder_profiler\widgets\profilergui.py", line 459, in load_data
stats_indi = [pstats.Stats(profdatafile),]
File "D:\Anacoda\lib\pstats.py", line 72, in init
self.init(arg)
File "D:\Anacoda\lib\pstats.py", line 86, in init
self.load_stats(arg)
File "D:\Anacoda\lib\pstats.py", line 99, in load_stats
with open(arg, 'rb') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'C:\Users\admin\.spyder-py3\profiler.results'
Version and main components

Spyder Version: 3.2.3
Python Version: 3.6.2
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)

Likely duplicate of #5587; already resolved in forthcoming Spyder 3.2.5.


fix(type): Add missing OptimizationSplitChunksNameFunction param

Summary
Add missing argument to type declaration of OptimizationSplitChunksNameFunction.
Notes
:information_source: I have found this only empirically while migrating a webpack project to Rspack, but I have no experience with Rspack codebase, so it'd be appreciated if reviewers verified if the type declaration is appropriate.
The following excerpt of rspack.config.ts works correctly, but yields TS error (it is used to group shared chunks, e.g. shared/main-app~subprocess-app.js):
    optimization: {
        splitChunks: {
            chunks: 'all',
            // Rspack type declaration is missing the 2nd argument
            // @ts-expect-error Target signature provides too few arguments. Expected 2 or more, but got 1.
            name: (_module, chunks: Chunk[]) => {
                return chunks.length === 1
                    ? chunks[0].name
                    : `shared/${chunks.map((item: any) => item.name.split('/').pop()).join('~')}`;
            },
        },
    },

Checklist

[x] Tests ~updated~ (or not required).
[x] Documentation ~updated~ (or not required).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

please rebase main so you can escape ci issue


Support for inline

Would be nice to have support for inline script tags (please disregard the gibberish JS)

I like this idea! I can work on a PR adding JS injections to https://github.com/phoenixframework/tree-sitter-heex/blob/main/queries/injections.scm. Depending on what text editor you use you may need to port those queries to the format your text editor understands. For example, nvim-treesitter uses a different set of captures to handle language injection.

We could support script (and maybe style too?) elements as separate nodes like tree-sitter-html: https://github.com/tree-sitter/tree-sitter-html/blob/d742025fa2d8e6100f134a6ea990443aa1f074b3/grammar.js#L64-L68. It looks like we have some parsing errors currently where do is mistaken by the error recovery mechanism and I think using a custom element might eliminate that

I started working on this last night, see https://github.com/phoenixframework/tree-sitter-heex/pull/38. I'm currently struggling to get the contents of a <script> tag to parse correctly. It is running into issues when there are {} characters and I'm not sure how to proceed.
Looking at tree-sitter-html they solve this via an external scanner written in C. Given how similar HEEx is I could copy that over if we are willing to introduce native code into this grammar.
@the-mikedavis If you have any ideas, I'd love to hear them!

For posterity, it appears that we do need to implement an external scanner to write a node that captures all text until hitting </script> or </style>: https://github.com/tree-sitter/tree-sitter/discussions/1252

Hello. Could I ask about the status of this?

I believe Connor's last comment is still up-to-date. To fix this properly we would probably need to add a scanner like tree-sitter-html's and follow what it's doing for parsing <script> tags.

This is the closest I could get for nvim-treesitter.
The following injections file properly highly JS:
; heex/injections.scm
((tag
  (start_tag
      (tag_name) @tag_name (#eq? @tag_name "script"))
  (text)
  (end_tag)
) @injection.content
 (#offset! @injection.content 1 0 0 -9)
 (#set! injection.language "javascript")
 (#set! injection.include-children)
 (#set! injection.combined))

However, it disables the elixir injections. I'm not sure why since the query seems to match the exact nodes.

I got it working by adding the ;; extends modeline comments to my queries.
Here is temporary workaround for nvim-treesitter users, while the support is added to the grammar:
File: nvim/queries/heex/injections.scm
;; extends

((tag
  (start_tag
      (tag_name) @tag_name (#eq? @tag_name "script"))
  (text)
  (end_tag)
) @injection.content
 (#offset! @injection.content 1 0 0 -9)
 (#set! injection.language "javascript")
 (#set! injection.include-children)
 (#set! injection.combined)
)

((tag
  (start_tag
      (tag_name) @tag_name (#eq? @tag_name "style"))
  (text)
  (end_tag)
) @injection.content
 (#offset! @injection.content 1 0 0 -9)
 (#set! injection.language "css")
 (#set! injection.include-children)
 (#set! injection.combined)
)

This only adds syntax highlight support.


OSX unit tests don't respect gtest_filter or other args

Repro steps:

Build OSX unit tests
Run OSX unit tests --gtest_filter=SomethingReasonable

Expected: limited set of tests run
Actual: all tests run
This seems to be caused by our hard-coding of args in EntryPoint.cpp::74

could you address this along with the other OSX compat tests?

as a starting point, the issue seems to be that we changed how we take args in EntryPoint.cpp when we moved to TAEF.

@ms-jihua load balancing this back to you :).

Scratch my comment, I missed that @aballway already fixed this in #2251.


feat: Bug Fix

Zoom Gesture on BarcodeCaptureActivity without camera permission on (Android M and above) crash fixed

I signed it!

Thanks for reporting this. I've committed a fix (based on yours, but simplified) in 32c7b001.


Weird Animation and No Focus When Using Focus option On Input Field

{
    key: 'title',
    type: 'input',
    templateOptions: {
        focus : 'true', // <== this doesnt work, and messes with animation
        type: 'text',
        required: true,
        placeholder: 'Event Title'
    }
},

Looking for options to get the keyboard to appear on screen when screen opens, could hack it if necessary

Hmm, have you tried adding
<preference name="KeyboardDisplayRequiresUserAction" value="false" />

to your config.xml? Had some weird animation issues as well but they seemed to be resolved when I added this.

the focus works, my bad for not reading the readme carefully, but I am
still getting jumpy UI and the keyboard is popping up too quickly, needs
some sort of delay due to the animation of the form sliding in.
Also it doesn't appear to have any impact on a modal sliding in from the
bottom.
I will try and break it out into a smaller codePen to demonstrate the issue
-- http://www.clearlyinnovative.com/build-it/
Aaron K. Saunders
CEO Clearly Innovative Inc.
<EMAIL_ADDRESS>201.214.5926  (M)
<EMAIL_ADDRESS>www.clearlyinnovative.com
@aaronksaunders
This email message and any attachment(s) are for the sole use of the
intended recipient(s) and may contain proprietary and/or confidential
information which may be privileged or otherwise protected from disclosure.
Any unauthorized review, use, disclosure or distribution is prohibited. If
you are not the intended recipient(s), please contact the sender by reply
email and destroy the original message and any copies of the message as
well as any attachment(s) to the original message.
On Tue, Jul 7, 2015 at 2:31 PM, Mike Hartington<EMAIL_ADDRESS>wrote:

Hmm, have you tried adding

to your config.xml? Had some weird animation issues as well but they
seemed to be resolved when I added this.
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly-templates-ionic/issues/18#issuecomment-119294385
.

Sounds good. I remember there being some issues with autofocusing inputs with Ionic, which could be resolved by adding a small timeout and using a directive directive. We could do the same to formly-focus.js. @kentcdodds maybe able to shed some light here if it's possible to override that.

Take a look at this (new) example

unless I am missing something here, the focus works fine on regular
forms... is there a way for me to pass a focus-wait down to the underlying
directive?
-- http://www.clearlyinnovative.com/build-it/
Aaron K. Saunders
CEO Clearly Innovative Inc.
<EMAIL_ADDRESS>201.214.5926  (M)
<EMAIL_ADDRESS>www.clearlyinnovative.com
@aaronksaunders
This email message and any attachment(s) are for the sole use of the
intended recipient(s) and may contain proprietary and/or confidential
information which may be privileged or otherwise protected from disclosure.
Any unauthorized review, use, disclosure or distribution is prohibited. If
you are not the intended recipient(s), please contact the sender by reply
email and destroy the original message and any copies of the message as
well as any attachment(s) to the original message.
On Tue, Jul 7, 2015 at 6:54 PM, Kent C. Dodds<EMAIL_ADDRESS>wrote:

Take a look at this (new) example
http://angular-formly.com/#/example/field-options/focus
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly-templates-ionic/issues/18#issuecomment-119366865
.

Thats what I was wondering as well.  Looks like I may have to add this to the templates.
http://docs.angular-formly.com/v6.16.0/docs/formly-focus
It defaults to 0, we may need to bump that up a bit for ionic. @aaronksaunders would you be willing to test this out and maybe make a PR?

Sure I will give it a try... Just point me in the right direction. Haven't dug too deep into the ionic template code
Aaron K. Saunders
CEO Clearly Innovative Inc.
<EMAIL_ADDRESS>201.214.5926  (M)
<EMAIL_ADDRESS>www.clearlyinnovative.com
@aaronksaunders
An Appcelerator Titanium Gold Partner
This email message and any attachment(s) are for the sole use of the intended recipient(s) and may contain proprietary and/or confidential information which may be privileged or otherwise protected from disclosure. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient(s), please contact the sender by reply email and destroy the original message and any copies of the message as well as any attachment(s) to the original message.

On Jul 7, 2015, at 7:18 PM, Mike Hartington<EMAIL_ADDRESS>wrote:
Thats what I was wondering as well. Looks like I may have to add this to the templates.
http://docs.angular-formly.com/v6.16.0/docs/formly-focus
It defaults to 0, we may need to bump that up a bit for ionic. @aaronksaunders would you be willing to test this out and maybe make a PR?
—
Reply to this email directly or view it on GitHub.

Sure, you should just have to work with this
https://github.com/formly-js/angular-formly-templates-ionic/blob/master/src/fields/ion-input.html
And add a statement for focus-wait.
<input ng-model="model[options.key]"
       focus-wait="{{options.templateOptions.focusWait? options.templateOptions.focusWait: '500'}}"
       />

With the way ionic handles inputs and setting focus, you should test placing focusWait on the label rather that the input as well.
Then you should be able to to gulp build, push to github and make a pull request.

Actually, interesting story. That formly-focus directive came from kcd-angular which covered some use cases that angular-formly hasn't yet needed. I think that support for focusWait should be added to angular-formly core. As formly-focus is considered an "internal directive", this wouldn't have to be part of a breaking change... But I'll probably make it part of the 7.0.0 release anyway just in case.
Note, there's an example of how to dynamically add the needed attributes in the docs: http://docs.angular-formly.com/v6.16.0/docs/formly-focus


Disable a set of warnings for NuttX that are new in GCC 6

This doesn't compile in GCC 6 yet to the end, but its a start.

The culprit is that <functional> depends now on <string> and we would need to stub that complete header or maybe rather provide our own functional. I'm stopping here for now.
/interfaces/src/pwm.cpp
In file included from /usr/local/Cellar/gcc-arm-none-eabi-62/20161221/arm-none-eabi/include/c++/6.2.1/string:52:0,
                 from /usr/local/Cellar/gcc-arm-none-eabi-62/20161221/arm-none-eabi/include/c++/6.2.1/stdexcept:39,
                 from /usr/local/Cellar/gcc-arm-none-eabi-62/20161221/arm-none-eabi/include/c++/6.2.1/array:39,
                 from /usr/local/Cellar/gcc-arm-none-eabi-62/20161221/arm-none-eabi/include/c++/6.2.1/tuple:39,
                 from /usr/local/Cellar/gcc-arm-none-eabi-62/20161221/arm-none-eabi/include/c++/6.2.1/functional:55,
                 from ../src/include/../platforms/px4_subscriber.h:42,
                 from ../src/include/../platforms/px4_nodehandle.h:42,
                 from ../src/include/px4.h:46,
                 from /Users/lorenzmeier/src/Firmware/src/drivers/camera_trigger/interfaces/src/pwm.cpp:1:
/usr/local/Cellar/gcc-arm-none-eabi-62/20161221/arm-none-eabi/include/c++/6.2.1/bits/basic_string.h: In function 'float std::__cxx11::stof(const string&, std::size_t*)':
/usr/local/Cellar/gcc-arm-none-eabi-62/20161221/arm-none-eabi/include/c++/6.2.1/bits/basic_string.h:5439:31: error: 'strtof' is not a member of 'std'
   { return __gnu_cxx::__stoa(&std::strtof, "stof", __str.c_str(), __idx); }
                               ^~~
compilation terminated due to -Wfatal-errors.

@davids5 It would be interesting to find out if we can better co-exist with something more flexile like CLANG.

Merging but doesn't fix it.

@LorenzMeier
re: Disable a set of warnings for NuttX that are new in GCC 6
Depending on how widespread this is we can handle it in the nuttx-patches like we do the shadow warnings.
On the second one, it looks like we are pulling in foreign header files. That could prove to be a binary compatibility nightmare.


Fix compiler errors about lifetime

With
rustc --version
rustc 1.68.2 (9eb3afe9e 2023-03-27)
cargo --version
cargo 1.68.2 (6feb7c9cf 2023-03-26)

I was getting the following errors:
   Compiling rs-matter v0.1.0 (/Users/markusbecker/src/matter-rs/rs-matter)
error[E0310]: the parameter type `D` may not live long enough
   --> rs-matter/src/mdns/builtin.rs:131:5
    |
131 | /     {
132 | |         let mut udp = crate::transport::udp::UdpListener::new(
133 | |             stack,
134 | |             crate::transport::network::SocketAddr::new(IpAddr::V6(Ipv6Addr::UNSPECIFIED), PORT),
...   |
201 | |             .unwrap()
202 | |     }
    | |_____^ ...so that the type `D` will meet its required lifetime bounds
    |
help: consider adding an explicit lifetime bound...
    |
130 |         D: crate::transport::network::NetworkStackDriver + 'static,
    |                                                          +++++++++

error[E0310]: the parameter type `D` may not live long enough
   --> rs-matter/src/transport/core.rs:134:5
    |
134 | /     {
135 | |         let udp = crate::transport::udp::UdpListener::new(
136 | |             stack,
137 | |             crate::transport::network::SocketAddr::new(
...   |
200 | |             .unwrap()
201 | |     }
    | |_____^ ...so that the type `D` will meet its required lifetime bounds
    |
help: consider adding an explicit lifetime bound...
    |
132 |         D: crate::transport::network::NetworkStackDriver + 'static,
    |                                                          +++++++++

For more information about this error, try `rustc --explain E0310`.

This commit applies those recommendations.

@markus-becker-tridonic-com:
I don't see these errors. (I've tried latest stable - 1.71 and latest nightly - 1.73).
What compiler version are you using?

Ah, it is 1.68.2 - I should read more carefully!
How about upgrading to 1.71?

Ah, it is 1.68.2 - I should read more carefully!
How about upgrading to 1.71?

Ah, right. Interesting that lifetimes are not required anymore on 1.71.


Add card for displaying a Twitter feed

What twitter_username did you use? (legumefed should work)
The header and footer are not necessary here, I can remove them.
It also looks like I need to add something like (page.twitter_card and site.twitter_username) to _layouts/defaults.html to display the card container in case both the News and Announcements cards are turned off. However, there is evidence that parentheses are not valid syntax for nested logical values - looking into that now.
This requires me to make another commit to pr-twitter-card, right? (researching that as well)

I find that if I change anything in _config.yml, I have to restart the site (by ctrl-C, then "bundler exec jekyll serve") to get the changes to appear. But changes to index.html work after refreshing the site. This may explain the invisible Twitter card.

Removing the parentheses happens to work in this case, as it evaluates logical operators from right to left.

What twitter_username did you use? (legumefed should work)

I tried legumefed and legumeinfo. It correctly generated the link to those twitter accounts but no tweets were displayed.

The header and footer are not necessary here, I can remove them.

Sounds good.

It also looks like I need to add something like (page.twitter_card and site.twitter_username) to _layouts/defaults.html to display the card container in case both the News and Announcements cards are turned off. However, there is evidence that parentheses are not valid syntax for nested logical values - looking into that now.

Good catch. Maybe don't worry about the username here and update the card to complain that a username hasn't been given instead of not being displayed if the username is missing?

This requires me to make another commit to pr-twitter-card, right? (researching that as well)

Just push to the same branch and the new commits should be included in the PR. At least I think that's how it works...

@svengato Apparently the Twitter timeline was being blocked by a plugin in my browser; your code works just fine. Sorry for the misdirection, again!
With the timeline working I was able to play with the styling to get it to blend in better with the theme. Here's what I came up with:

  Tweets by {{ site.twitter_username }}

I dropped all the card contents/padding and rounded the corners to match the timeline's corners. This makes the timeline look like it's the card but it's actually still wrapped in a card to force it to align and scale correctly with the rest of the page.

Maybe don't worry about the username here and update the card to complain that a username hasn't been given instead of not being displayed if the username is missing?

That would fix a missing twitter_username, but not an invalid twitter_username. I guess the solution for the latter is to open a Twitter account for the desired username?

Just push to the same branch and the new commits should be included in the PR.

How descriptive should the commit message be for these corrections? The entire pull request still comes under "Add card for displaying a Twitter feed", so would a simple "Fix requested issues" be sufficient?

That would fix a missing twitter_username, but not an invalid twitter_username. I guess the solution for the latter is to open a Twitter account for the desired username?

Ha! Maybe let Twitter handle invalid usernames?

How descriptive should the commit message be for these corrections? The entire pull request still comes under "Add card for displaying a Twitter feed", so would a simple "Fix requested issues" be sufficient?

I'm of the opinion that all commit messages should be descriptive and not mention anything specific to GitHub. This makes it easier for someone using git from the command-line to understand what's happening just by reading the commit log, no referencing GitHub required!

I'm of the opinion that all commit messages should be descriptive and not mention anything specific to GitHub. This makes it easier for someone using git from the command-line to understand what's happening just by reading the commit log, no referencing GitHub required!

Then it would make sense to make a separate commit for each correction? I will think about it (back later today).

I'm a big fan of small, single-purpose commits. And with PRs there's no downside, they all get included. Not that I always stick to that.

I agree. This approach also makes it easier to write descriptive commit messages since there's less to describe!

It turns out the header and footer can be removed from the Twitter timeline widget via additional data attributes. Instead of endlessly asking you to tweak this PR I just made a branch that removes the header and footer so the timeline styling can really look like the news and announcements cards. (The branch also disables tracking by Twitter, which is apparently enabled by default...) Anyway, have a look and if all seems well I'll merge it into this branch and then merge the PR. Thanks for you patience!

I'll just push a commit to that PR that corrects the issue. Standby for merges!


change content-type-parser to whatwg-mimetype

Regular Expression Denial of Service (ReDoS)
https://snyk.io/test/npm/content-type-parser/1.0.2

@lagden sorry for the late reply, can you resolve the conflicts?

done

@lagden please ensure the build passes

Hi guys. Just wondering is there a plan to fix this and would you have a date in mind? Thanks

waiting on the build to be fixed.  @lagden can you fix it?

@jsdevel - Not able to fix the build issue with these code changes. Please assist us to fix this issue. Our project is impacted due to this issue.

@khadeamolm you'll have to dig into it and fix it.

@jsdevel -- please close this, since you merged #1177 that superseded it. Also #1164 can be addressed with the form and closed.


Adding Wondershare PDFelement 5.3.9.

This is Wondershare's updated product from PDF Editor Pro.

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


Some texts are out of bounds

Hello,
Some of the texts including news and tables are sometimes out of boundaries.
Here is an example of what I get on the desktop rendering (chrome):

and for the <table> here is what I get on Chrome for Android:

Uooooo okay thanks for bringing this to my attention! I will take a look at this asap

Thanks for bringing this to my attention, I have fixed both issues and you can now grab them in the latest release (https://github.com/kelyvin/caffeine-theme/releases/tag/v2.8.1)!
To resolve the table overflow on mobile,  I made tables horizontally scrollable on smaller devices. I may look into a more responsive solution in the future, but for now this should be an adequate fix that doesn't ruin the user experience.
Thanks!

Again, thanks a lot @kelyvin :)


Search utility?

I'd love to see a search bar utility on the site! While the organizing of content into sub categories is useful, the ability for visitors to more specifically search for content related to the topic or problem they're looking to solve would add to better navigation of the site, especially if content is tagged with keywords.

@richardr91 Thanks for the suggestion. I certainly think a search feature would be nice however it would be much easier to add if the site was full stack and not just front end. It might be more realistic to add a search this site with google box at least to have some search.
I might look at doing this at some point, but if someone wants to try adding this that would be great. It would have to work on both mobile and desktop as well as be accessible to screen readers to be merged. But if someone wants to tackle this just comment to say you're working on it and feel free to ask questions.

I'm pretty new to coding and programing, but I might give this a try. (What I lack in experience, I make up for with enthusiasm!) Any assistance/advice is welcomed nonetheless!

Hey Kitty!
Sorry for the slow reply, we didn't really use the issues system on here recently.
We will move to a new website repo soon and would love your feedback on there!
The reason we have no search bar is that the website is currently a static website.
P.S. you should join our community discord for questions and remarks you might have in the future!
Hope to see you there!


When multiplying or dividing duration with a decimal number, the number is rounded to long

In DMN FEEL, when multiplying or dividing a duration by a decimal number, that number is rounded to long before the result is calculated. This produces wrong result.
Example: duration("P10Y11M") / 2.5 should result in duration("P4Y4M") but it results in duration("P5Y5M").
Problem is probably related to this code.  (and similarly for the mult method)
Acceptance criteria:

Problem is fixed and tested.
New code has 100% code coverage.
New tests exist for this behavior in FEELDateTimeDurationTest class.

Moving this to backlog until this is solved https://github.com/kiegroup/kie-issues/issues/71

After a discussion on chat, let's merge this if it is ok, the issue #71 will get done afterwards.


[💡 Feature]: Remote Workspaces in Projects Widget

Is your feature request related to a problem?
The Projects workspace cannot save remote workspaces (like SSH)
Describe the solution you'd like.
I'd like the Projects widget to be able to save remote workspaces
Describe alternatives you've considered.
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@tmayoff thanks for filing the issue, any contributions are highly appreciated. I am happy to help answering questions on the codebase or other things.


Quick q

I love this program. But have a suggestion or maybe I've missed it somewhere...
I teach online and use obs. I crop window capture or use slides which are also cropped to fit half my obs screen.
Is there a way to change/choose the actual area of the screen , like a specific sized window instead of the whole screen? Whenever I try to highlight or even circle a word, it's always out of place. i've tred everthing.
Keep up the good work  :)

Hi @jumanjaydazed,
ppInk has a Window mode where you can limit the working area
https://user-images.githubusercontent.com/4083478/112311221-c656c580-8ca5-11eb-895b-2279366c0fc4.mp4
(extracted from https://github.com/pubpub-zz/ppInk, looking for Window Mode)
even in window mode you will be able to capture the window

(and make it transparent with a filter if required)

If you like it, do not hesitate to star it 😊

Thanks, I seen this aboit an hour after I posted! I decided to use display capture. Love your product by the way...😊
---Original---
From: @.>
Date: Thu, Jul 8, 2021 01:11 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [pubpub-zz/ppInk] Quick q (#115)
Hi @jumanjaydazed,
ppInk has a Window mode where you can limit the working area
https://user-images.githubusercontent.com/4083478/112311221-c656c580-8ca5-11eb-895b-2279366c0fc4.mp4
(extracted from https://github.com/pubpub-zz/ppInk, looking for Window Mode)
even in window mode you will be able to capture the window
(and make it transparent with a filter if required)
If you like it, do not hesitate to star it 😊
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Add option to disable escaping ol triggers

Hi Dom,
Reason to add feature
Similar to other people in #81, I needed to disable the feature to escape ordered lists. I noticed that there are quite a few people also asking for this ability, so I thought I would put it behind an option flag. I understand turndown solves this issue as well and I'm looking forward to using it once merged into master!
How it works
Users must opt-in to disabling the escaping of ordered list. I believe this stays true to your belief to preserve the integrity of the HTML and be as lossless as possible.
toMarkdown('<b>1. Item</b>', { escapeOrderedList: false });

Included in the Pull Request

Added options.escapeOrderedList with default to true
Added README.md documentation

Considerations before merging

Tests pass, but I did not add a test for the new option

Passing tests proves that the default behaviour is to continue to escape ol lists
No tests were added for other options, which is why I did not add any this option

escapeOrderedList may not match the library's naming convention

lossless: <boolean> may be a better choice but I felt it was too broad

I'm happy to make any changes that you require and, if you choose to not merge this (adding yet another option to the library), then I totally understand.
Cheers,
Michael

Hi Michael, thanks for this. I will look it over when I get a chance. Thanks again.

Hi Michael,
Thanks for your work on this, in particular your detailed pull request. I have decided against adding this option to to-markdown, as I'd prefer to move developers towards Turndown (to be released soon). Anyway, I appreciate your time on this, so thanks again!

No worries @domchristie. Looking forward to Turndown's release!

@mwbrooks Turndown v4 was released earlier today :)

Nice, congrats and thank you! 🙌


improvement suggestion

hey, I'm back :)
So I love having the preview when I do a BLines. There is one thing I wish was better to make it really perfect so hopefully I can describe it easily:
When we go trough the results, the preview buffer reloads every time which ideally we shouldn't want as when using BLines, it is always the same buffer only the cursor line changes.
do you think it would be possible not to reload the buffer in the preview window but only update the cursor line?
I'm not quite sure how you do the preview but my high level understanding is that you use vim buffer itself, we can use this function to set the cursor line in the current buffer:
:call cursor(current_line, current_col)
What do you think?

It’s a good idea, let me look into it.

https://github.com/ibhagwan/fzf-lua/commit/000d65b43b4572734fe109c35f2116955c21ea01 - this commit does exactly that. however, I didn't limit this to lines|blines, this now also applies to when the matches are from the same file in grep|live_grep, the preview buffer will only be reloaded when it recognizes it's a different file/buffer.

Thanks for considering this suggestion, I've updated the plugin and noticed the change :)
One thing though, if we go fast through the results, the preview doesn't refresh, I think it can make sense when we do a git_files or files search as I might speed things up. However for things like BLines it would be best if the preview was refreshing at the same time as the cursor goes through the results, no matter the speed. hopefully what I'm saying makes sense.
What do you think ? is it even something possible to do?

The default preview delay is 100ms (same as fzf's native preview functionality), this is done to prevent overloading neovim with file/buffer reads when scrolling fast, the setting can be controlled by setting winopts.preview.delay globally but I wouldn't recommend that for reasons I just mentioned.
If you wish to change it just for blines you can send a specialized winopts table just for that with the call:
:lua require'fzf-lua'.blines({ winopts = { preview = { delay = 0 } } })

that does exactly what I wanted, this is really nice for blines thanks so much


Ability to limit the number of output lines for a process, similar to pnpm?

I think this could easily be achieved in user-space / outside of concurrently, if the JS API had a way to configure a separate output stream per process? maybe?
pnpm has a max of 10 lines per process when it's running things in parallel:

would be super cool if I could do this with my own scripts via concurrently some how.
It seems there is a max of 10 lines per process?

Hey!
I'm not too sure of what the main ask is, there are many things to address 😅  is it

proposing setting output stream per command?
asking whether we already support it (from one of your screenshots using concurrently)?
proposing implementing this kind of log window per command?


[WIP] NH-63282 Remove pkg_resources usage

WIP
pkg_resource is deprecated. Replace pkg_resource with importlib_metadata / importlib.metadata equivalents. See also ticket comment.

Closing and upgrading to Otel Python 1.28.1/0.49b1 instead.


pull request for e2e test at 01/10/2017 02:26:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add vector support to gRPC API

Since we added support for the experimental vector CQL data type in #2597, we can take advantage of this to add vector support to the gRPC API. This will also require documentation updates.

Note: PR #2678 contains what can likely be repurposed as implementation (Bridge gRPC is very similar to public gRPC API).

Suggested steps here:

Use changes that add VectorCodec implementation (and registration through ValueCodecs) from #2678 into bridge module as the base to change grpc module (i.e. make the same changes, just in different module)
Similarly use #2678 test changes (class ValueCodecTest) as the base for added unit test methods.
Skip sgv2-quarkus-common changes (that code is not duplicated)
Once grpc module has been modified, tests added and pass locally (these are unit tests so can be run with ./mvnw clean test), create PR
CI hopefully passes (if not, need to figure out why, but if tests passed locally they should pass in CI
Get code review; if changes requested resolve these (either make change requested, or discuss on alternative to change)
Once one team member approves, merge!

#2722


Fixed issue with positioning and setting the width of the dropdown on decimal number (absolute positioning)

Fixed issue with 'dropdown not scrolling with the page' introduced in https://github.com/basvandenberg/ng-select/pull/173 .
I apologize for this issue that I overlooked :)
In this PR, the positioning is 'absolute', and the top and left of the dropdown are calculated relative to the host element.

Very nice, I merged it into the beta.6 branch. Thanks for your contribution!


No changes found

Have configured the github action per the example; however, any PR changes don't appear in the Github PR Comment. Mentioning in case there is a weird bug or edge case going on here. Feel free to close this.
https://github.com/seanturner026/argocd-applicationset/pull/9
My ApplicationSet is here: https://github.com/seanturner026/argocd-applicationset/blob/main/clusters/staging/argocd-appset.yaml
Awesome project by the way!

Hi, thank you so much for reporting this.
I have now fixed the timeout issue and added support for kustomize.buildOptions :)
I have tested it on your branch, and it seems to work now on v0.0.10 :arrow_down:
git clone https://github.com/seanturner026/argocd-applicationset base-branch --depth 1 -q
git clone https://github.com/seanturner026/argocd-applicationset target-branch --depth 1 -q -b workflow-test
docker run \
   --network host \
   -v /var/run/docker.sock:/var/run/docker.sock \
   -v $(pwd)/output:/output \
   -v $(pwd)/base-branch:/base-branch \
   -v $(pwd)/target-branch:/target-branch \
   -e TARGET_BRANCH=workflow-test \
   -e REPO=seanturner026/argocd-applicationset \
   -e KUSTOMIZE_BUILD_OPTIONS="--load-restrictor LoadRestrictionsNone --enable-helm" \
   dagandersen/argocd-diff-preview:v0.0.10

And print the output with
cat ./output/diff.md

Let me know if you still experience issues :)

You legend! Works perfectly.

Hi @seanturner026
I’ve just released version 0.0.18 🚀  This release removes the --kustomize-build-options option. Supporting individual settings like kustomize.buildOptions is not sustainable in the long run, which is why I’ve now shifted to allowing users to define any Argo CD configuration through a Helm values.yaml file. The kustomize.buildOptions should now be configured in the values.yaml file. For more details on how to do this, please refer to the README.
Please let me know if you experience any issues or if you have questions about how it works :)

Cheers thank you!


Base template

This is just a first step to copy the design we had in PyCon SK 2016 as a base template.

Current coverage is 95.02% (diff: 51.61%)

Merging #115 into master will decrease coverage by 0.64%

@@             master       #115   diff @@
==========================================
  Files            23         24     +1
  Lines           671        684    +13
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            642        650     +8
- Misses           29         34     +5
  Partials          0          0

Powered by Codecov. Last update efa0a31...121581d

all done


cm14 mapping content p2

[ ] #665
[ ] #666
[ ] #471

also dropship 1 and 2 walls. (They have different colors)

https://github.com/CM-14/CM-14/issues/687

also dropship 1 and 2 walls. (They have different colors)

fittingly named the alamo and normandy. dont forget to name the walls that aswell (grey for normandy, tan for alamo)


Parameter estimates do not converge

There is currently no automated test for this.

This is probably due to the likelihood_resample function using the indices in the wrong way, as suggested by @camillegontier. We should have something like
    for j in 1:M_out
        ns[j,:].=ns[j,idx[j,:]]
        ks[j,:].=ks[j,idx[j,:]]
    end

Should be fixed by bd2ddde.

@camillegontier reported that this is fixed


Ensure that Slot types fields are handled correctly

Yup, had applied these using the slandles test to check them.
I'll add specific tests.

Thanks @shans  (Added test and fixed tsc/let)

Thanks @shans  (Added test and fixed tsc/let)


Install Yarn and Git

The OS should be released with Yarn and Git already installed - would save some work for the developers.

It was intentional to not install such tools on TurtleOS. Development is performed on developer's machine, not on Raspberry Pi. Building anything bigger than hello world takes a lot of time, so it is not performed on RaspberryPi.