Flags are not working

I tried to use this package and for some reason flags are not appearing neither on the button, nor in the list. However on the material-ui-phone-number they are.

On the screenshot there are both material-ui-phone-number and material-ui-phone-number-2. On the first one flags are working and on the second - no.
I have also installed dependency country-flag-icons, but don't get how to make it to de used by material-ui-phone-number-2.

same issue for me

I've looked into this. Library uses country-flag-icons package to get flags. Which seems to be working fine.
Also, Example Codesandbox renders flags without problems.
Are you using most recent version of this package? Alternatively I recommend downgrading the version as previously the flag icons were included in the build (see last paragraph).
This might be related to https://github.com/alexplumb/material-ui-phone-number/issues/143
Also, @felipehertzer removed country flag icons from the build in most recent PR.

Yes. It is the latest version.
I did clone this repo and tried to run the demo. Flags where showing just fine there.
When I imported it into my project, the country flag svg was missing from the page source.
Downgrading is not an option for me since I want to use it with React 18. If I downgrade npm install throws peer dependency conflict error.

Same issue for me, it only happens on mobile

For me it is working perfectly and also in Safari mobile. I'm using Vitejs, maybe it could be some configuration in Webpack Config. Can you check in the HTML for the flag.
it should be like this:

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 363 242">
<path fill="#338AF3" d="M0 0h363v242H0z"></path><path fill="#FFF" d="M57 96.9 14.7 78.2 57 59.6l18.6-42.2 18.6 42.2 42.3 18.6-42.3 18.7-18.6 42.2z"></path><path fill="#f30028" d="m75.6 40.6 11.5 26.1 26.1 11.5-26.1 11.5-11.5 26.1-11.5-26.1L38 78.2l26.1-11.5z"></path><path d="M0 152.2h363v15.7H0zm0 31.4h363v15.7H0z" fill="#FFDA44"></path>
</svg>

For me it is working perfectly and also in Safari mobile. I'm using Vitejs, maybe it could be some configuration in Webpack Config. Can you check in the HTML for the flag.
it should be like this:

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 363 242">
<path fill="#338AF3" d="M0 0h363v242H0z"></path><path fill="#FFF" d="M57 96.9 14.7 78.2 57 59.6l18.6-42.2 18.6 42.2 42.3 18.6-42.3 18.7-18.6 42.2z"></path><path fill="#f30028" d="m75.6 40.6 11.5 26.1 26.1 11.5-26.1 11.5-11.5 26.1-11.5-26.1L38 78.2l26.1-11.5z"></path><path d="M0 152.2h363v15.7H0zm0 31.4h363v15.7H0z" fill="#FFDA44"></path>
</svg>

For me it was empty for both SVG and not SVG variants.

For me it is working perfectly and also in Safari mobile. I'm using Vitejs, maybe it could be some configuration in Webpack Config. Can you check in the HTML for the flag.
it should be like this:

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 363 242">
<path fill="#338AF3" d="M0 0h363v242H0z"></path><path fill="#FFF" d="M57 96.9 14.7 78.2 57 59.6l18.6-42.2 18.6 42.2 42.3 18.6-42.3 18.7-18.6 42.2z"></path><path fill="#f30028" d="m75.6 40.6 11.5 26.1 26.1 11.5-26.1 11.5-11.5 26.1-11.5-26.1L38 78.2l26.1-11.5z"></path><path d="M0 152.2h363v15.7H0zm0 31.4h363v15.7H0z" fill="#FFDA44"></path>
</svg>

There is no SVG tag in our case. The issue is not specific to a browser or device. It is also happening on desktop Chrome, Edge.

I have same error.
My react version is 18.
How to fix this error.


Deprecate learning_curve!

Addresses #60. Also makes operation=nothing the default for learning curves; see #148 for the reasoning.

Codecov Report

Merging #150 (514a1ae) into dev (75fdfc2) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #150      +/-   ##
==========================================
+ Coverage   87.41%   87.60%   +0.18%
==========================================
  Files          11       11
  Lines         612      613       +1
==========================================
+ Hits          535      537       +2
+ Misses         77       76       -1

Impacted Files
Coverage Δ

src/tuned_models.jl
92.95% <ø> (ø)

src/learning_curves.jl
97.00% <100.00%> (+1.04%)
:arrow_up:

Continue to review full report at Codecov.

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


podspec fix

To resolve the error, I renamed the podspec file to match the dependency name and fixed the podspec file.


feat: include ReleaseVersion entity and related migration

Expose Ledger-Subquery version to GQL API
Addressing #125
Code Review Checklist (to be filled out by reviewer)

[ ] Description accurately reflects what changes are being made.
[ ] Either the PR references an issue (via the "Development" combobox) or the description explains the need for the changes.
[ ] The PR appropriately sized.
[ ] New code has enough tests.
[ ] New code has enough documentation to answer "how do I use it?" and "what does it do?".
[ ] Existing documentation is up-to-date, if impacted.

Would you mind adding a simple e2e test as well, just to protect against regression?

It looks like that e2e test is going to fail until we get https://github.com/fetchai/ledger-subquery/pull/177/ merged as the migration isn't run otherwise. :facepalm:

In light of https://github.com/fetchai/ledger-subquery/issues/125#issuecomment-1348390652, I think two things are clear:

I owe @Jonathansumner an apology, so sorry brother!
We're probably better off abandoning this approach after all.


BUG || Parties Page is breaking in QA

When Case created by Postman Collection, Parties Page is breaking.

resolved in QA


UI design for Home page of Blogger

Is your feature request related to a problem? Please describe.
UI design for Home Page of Blogger
Describe the solution you'd like
UI design is exact replica of what your website will look like
This task is connected with issue #11 check for wireframes designed for this page and create a UI design for the same using color palette and font styling (for color and font check issue #45 )
Additional context
As we have more than one wireframe for a page so you can either choose a single wireframe and convert it into UI or use features from different wireframes designed by participants and come up with great solution .
Considering UI we expect a high fidelity design , use proper heading title without lorem ipsum .

I am interested

I am interested

I am interested

@ymaheshwari1  you are already assigned 2 issues and @raksha009  as @Akshi149 commented earlier so I am currently assigning it to her .

@Swarnimashukla your views on this design. Any changes to be made?

Greetings @Swarnimashukla . Can I make a attempt towards this issue?

@Akshi149  this looks good to me , go for a pr , name your screenshot - Homepage_ui_yourname


Packaged executable should be lowercase

Hello! This is a feature request to rename the Hyper executable a lowercase "hyper". The current name is "Hyper". In OS X's Spotlight and similar tools, search is case-insensitive and finding the app is intuitive. However, in Bash and launchers that use strict PATH lookup, only a lowercase name is intuitive. I know launching from Bash probably isn't that common but there are many launcher implementations available on Linux. Please consider this name for the launcher use case and convention's sake or perhaps packaging a lowercase symbolic link. Thank you!

Do any other apps do this? Wouldn't it make more sense to ln manually for your case?

The only prominent Linux packages I can think of having uppercase executable names are Xorg and VirtualBox (the latter has a lowercase symlink). I'm certain there are other examples and the fix for my system is trivial as you pointed out. However, lowercase naming is prevalent by far and I felt it was more appropriate to raise this idea at the source so it could be a intuitive, "don't think about it" benefit for everyone. Thank you for your consideration of this bike shed topic :]

This is a pretty common problem with electron apps. Google Play Music Desktop Player's binary is Google Play Music Desktop Player after building, the maintainer of the AUR package renames it during installation. Brave's build tool renames the package when building for Linux.
Basically that means it's pretty much your call whether you make it lowercase for us or not. It would be nice of course, but it's not really expected since by default every electron app is this way.

@jacobmischka, thanks for the insight and the references. I believe Hyper uses electron-builder for packaging? If that's the case, should this issue be closed as a duplicate or put in tracking of the upstream bug, electron-userland/electron-builder/issues/806?

:+1: electron-userland/electron-builder#806 was only recently resolved but somehow this appears to already be fixed


⚠ HDBits has degraded performance

In acaf5b0, HDBits ($HDB) experienced degraded performance:

HTTP code: 502
Response time: 11546 ms

Resolved: HDBits performance has improved in 26fc245 after 12 minutes.


Isolate log4j

PR description
Remove some of the tendrils of Log4J that have snuck into various code locations. Isolate them into a single class.
This is preparatory for GraalVM compilation of EVMTool as it doesn't play well with log4j and Besu needs to be able to operate with log4j removed from the classpath.
Fixed Issue(s)

Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if
updates are required.

Acceptance Tests (Non Mainnet)

[X] I have considered running ./gradlew acceptanceTestNonMainnet locally if my PR affects non-mainnet modules.

Changelog

[X] I thought about the changelog and included a changelog update if required.

Looks good, some of the unit tests are failing related to logging refactoring.


add polls option to API and npm audit fix vulnerable packages

Added support for listing and creating GroupMe polls.
Also, vulnerabilities have been found in the request package since version 2.12. Fixed with npm audit fix.

Thank you very much for the PR!


compatibility with certificates cookbook

Hi,
https://github.com/atomic-penguin/cookbook-certificate has some additional options that make life easier, e.g. nginx_cert or combined_file. I would love to see those options for this cookbook, too.

Happy to accept a pull-request for it 👍


⚠ Bofrost.de has degraded performance

In 3624f3e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 830 ms

Resolved: Bofrost.de performance has improved in 6481fb8.


update database.json with dinvishesh library

In this PR your repo is 9  commits behind master branch fix that plz. to fix the CI


Remove unnecessary dependencies

This PR removes unnecessary dependencies, reducing current installation size by 1.6MB.

It's not so much just about reducing the total footprint, but also reducing installation time by reducing the number of fetches to the registry.
Feel free to do whatever you want with this.


kramdown-rfc-clean-svg-ids reorders some XML attributes

Thanks for the kramdown-rfc-clean-svg-ids tool to fix the duplicate SVG ids issue!
When kramdown-rfc-clean-svg-ids is run on an XML file, it does the following in addition to fixing the duplicate ids:

Changes double quotes around attributes in the <?xml> element to single quotes.
Reorders the attributes of the following elements: <rfc>, <author>, <date>, <xref>, <artwork>, and <svg>. For instance, <artwork type="svg" align="center"> is changed to <artwork align="center" type="svg">.

As there is no affect on output, this isn't really a bug. However, these changes can create noisy diffs, and some authors are sensitive to those.
Could the script be updated to only make updates to ids and to leave the ordering of the XML attributes as is?

When kramdown-rfc-clean-svg-ids is run on an XML file, it does the following in addition to fixing the duplicate ids:

Changes double quotes around attributes in the <?xml> element to single quotes.

Yes, that is a weird one.  It is a single-line change, so maybe the authors can live with that; otherwise I'll do some manual patch-up (which always carries some risk).

Reorders the attributes of the following elements: <rfc>, <author>, <date>, <xref>, <artwork>, and <svg>. For instance, <artwork type="svg" align="center"> is changed to <artwork align="center" type="svg">.

XML attribute order is insignificant, so this indeed really is no change.

As there is no affect on output, this isn't really a bug. However, these changes can create noisy diffs, and some authors are sensitive to those.

Right.  authors really should:

run this tool themselves before submitting
switch on svg-id-cleanup in the YAML header to get this fix out right out of the normal kramdown-rfc process

Could the script be updated to only make updates to ids and to leave the ordering of the XML attributes as is?

Indeed.  You should deliver this fix as a separate diff (commit) so that it doesn't get mixed up with your editorial changes.
The reason I don't really want to change the tool is that the code uses an XML library, both for some assurance of correctness and for convenience.  This library is doing the right thing by outputting a canonical attribute order (in this case what computer science people would call "lexicographical order").
Actually, it is somewhat weird that xml2rfc (which probably was generating this XML) generates XML with somewhat random attribute order.
(Of course, theoretically, people might submit manually generated XML, and then the output might also differ in insignificant whitespace.)

The reason I don't really want to change the tool is that the code uses an XML library, both for some assurance of correctness and for convenience. This library is doing the right thing

Ack

people might submit manually generated XML

RPC editors also make manual updates to the XML attributes.

Right. authors really should:

run this tool themselves before submitting
switch on svg-id-cleanup in the YAML header to get this fix out right out of the normal kramdown-rfc process

I see that the SVG page already covers these points, but perhaps the section could mention that attributes may be reordered?

I actually found some places in the XML library that could make keeping the order easier.  I'll explore this when back from travel...

Thanks! I've verified that it does not reorder attributes.

This seems to work now, so I'm closing this.
Note that there are some other sources of duplicate IDs that could also be cleaned up.
This may require a strategy specific to the duplication's root cause, so I'm not attempting this now.


Fix issue 120

This commit will fix issue #120. I'm not sure if some form of .replace(/ /g, '') should still be used on these titles, but for now, this will fix the spacing issue.

Looks good. Gonna squash the commits into one to keep it simple.


Can't parse spread operator correctly

This one get parsed and shouldn't according to  https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Operators/Spread_operator
var obj = {"key1":"value1"};
var array = [...obj]; // Regarding MDN this should throw because obj is not iterable

And for objects, this doesn't work..
     var cleanField = {
          ...field, // this line throws
          config: {...field.config}
        };

For Array this works
        var stuff = [...field];

And should rest params to be allowed in setter? I expect Esprima to throw
class x {
    set foo(...rest){}
}

var x = {
    set foo(...rest){}
}

Reference: http://www.ecma-international.org/ecma-262/7.0/#sec-method-definitions

set PropertyName[?Yield] ( PropertySetParameterList ) { FunctionBody }
PropertySetParameterList : FormalParameter
FormalParameter does not include FunctionRestParameter.

Thanks for the report @nowindowsowrking!
To avoid confusion ("works", "doesn't work", etc are ambiguous), please state for each case:

what is the actual output?
what is the expected output according to the specification?

@ariya In the rest params cases this should all throw. Esprima is the only parser that does not throw on this. In acorn you get this error message: Setter cannot use rest params (2:12)

@nowindowsowrking Can you then please revise the description to follow the model (test case, actual, expected) per our Defect filing guide?

This is spawn to a more elaborated test case, see 1693.


Error when selecting '*' in join

This does not work:
    select
    *
    from
    aws_iam_user as u,
    aws_iam_policy as p
    where u.permissions_boundary_arn = p.arn;

    Error: pq: cannot iterate: there was an error executing scanIterator: rpc error: code = Unknown desc = column 'region' requires hydrate data from listIamUsers but none is available

But this works fine:
> select
  z.name, z.id, z.private_zone, r.*
from
 aws_route53_zone as z,
 aws_route53_record as r
where z.id = r.zone_id

this is caused by a bug in the aws plugin - PR for fix: https://github.com/turbot/steampipe-plugin-aws/pull/156


GPU management problem and Limited Max output tokens

Hello,
I am trying to get meta-llama/Llama-2-13b-chat-hf on local machine but I encountered two problems.
I'm using Windows10 with 32GB memory and four 3090 graphics cards (each with 24GB of video memory)。
The first is when I ran the following code
python generate.py --base_model=meta-llama/Llama-2-13b-chat-hf --langchain_mode=UserData --score_model=None --use_auth_token=True

It occured the following error
OutOfMemoryError: CUDA out of memory. Tried to allocate 50.00 MiB (GPU 0; 24.00 GiB total capacity; 23.18 GiB already
allocated; 0 bytes free; 23.19 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting
max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

It seems that it just used my CPU0, but actually I have CPU0,1,2,3 and they all have 24GB memory. I tried to add
CUDA_VISIBLE_DEVICES=0,1,2,3

but still didn't work.Also if I try to add
--use_gpu_id=False

then the model would restart downloading from hugging face.
So I had to add
--load_8bit=True

but then I encountered the second problem. The max output tokens were limited to 20 and wouldn't give any useful answer because of this limitation.

I checked many readme files and recent issues but didn't find a solution. How can I solve these problems?Thanks!

Yes there was a temporary bug about 6 days ago, it was fixed also 5 days ago.


mailmap: cleanup shortlog stats for nixos-22.11 release

Only cleanups the shortlog stats for contributors that appeared before the changes in the top 30 and commiter names had more than 10 commits in an effort to attribute them correct
$ git shortlog -se nixos-22.05..nixos-22.11 | sort -nr | nl | head -n30
@ajs124 @AndersonTorres @fabaff @dasJ @Mic92 @mweinelt @roberth @superherointj @vcunat
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Speaking of that command above, are you intentionally keeping<EMAIL_ADDRESS>separate in the shortlog?

Lets don't talk about why that is in the git history.

Why do we need that mailmap file at all? Wouldn't we much more care about the GitHub identity for a specific patch if we wanted to contact them? Is this just for the sake of comparing who spent most of his time on this project?

Well, all my<EMAIL_ADDRESS>commits in nixpkgs were due to misconfiguration which I didn't notice for some time.

Wouldn't we much more care about the GitHub identity for a specific patch if we wanted to contact them?

mailmap can also help with that by letting github know that different commit authors belong together and have a common GitHub profile which also adds a the sometimes missing link to their profile.

Is this just for the sake of comparing who spent most of his time on this project?

The number of commits is not in direct relation to the time people spend creating them. Anyone working on software for a bit knows that fixing the most complex problems is sometimes just changing one character but it still took 3 days to find that.

Why do we need that mailmap file at all?

If you think we don't need it, just ignore it. I am not going to start a discussion with you on the usefulness of that file.

TBH my only complaints are about the github mails.


[Draft] Create iOS build scripts for shaderc, SPIRV-Cross, etc.

Summary of the PR
Build iOS binaries for SPIR-V APIs alongside desktop targets for MoltenVK/mobile shader cross-compilation support.
Related issues, Discord discussions, or proposals
See messages sent in the Discord server.
Further Comments
N/A

@dotnet-policy-service agree

Hey, thanks so much for looking into contributing! Is this still in progress?

Hey! My apologies, I got sidetracked with other projects. I do want to continue this at some point, however currently it is not in progress.

No worries! I'll shut this off for now but please feel free to reopen if you find the time again! We always love community contributions and appreciate you even taking the time to investigate contributing!


sql/sem/builtins: make EXTRACT return float

PR: https://github.com/cockroachdb/cockroach/pull/42131
From release notes:

The extract() function now returns values of type float and includes fractional parts in the values for element 'second', 'millisecond', 'julian', and 'epoch'. This improves compatibility with PostgreSQL's extract() which returns values of type double precision. [#42131][#42131] {% comment %}doc{% endcomment %}

Taking this one, @ericharmeling.


feat(category): rework applied filters into behaviors, and replace PageConfigurationState with PageMetadata.

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

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

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?

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

Other information

the empty docs should be removed I think: "filtering/types/equal", "filtering/types/range", "filtering/types"

I think I'm going to review this one last time today or tomorrow, then I'll be good to go


[9.x] Add nonconsecutive option to Str::ucsplt

This PR adds an option to Str::ucsplt to keep groups of uppercase characters together.
Many times ucsplit is used to split text that contain acronyms such as 'PHP', which results in these acronyms being broken up, for example:
Str::ucsplit('PHPIsCool'); // ['P', 'H', 'P', 'Is,', 'Cool']

Passing $consecutive = false results in
Str::ucsplit('PHPIsCool'); // ['PHP', 'Is,', 'Cool']

Defaulting the $consecutive argument to true allows for full backwards compatibility with existing code.

Adding a new parameter to a public function is a breaking change, regardless if it has a default value.
It will break any class that extends Illuminate\Support\Str, which people might be doing in their projects to add new methods to it, instead of using macros

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Unify approach for constructing classes on device

Prior to this PR, tps used device-side new to instantiate polymorphic classes on the gpu for cuda and placement new for hip.  This resulted from the fact that the cuda implementation was done first and then had to be adjusted to hip, which, at least at that time, did not support device-side new.  (NB: It seems that device-side malloc is supported now, although I'm not clear on device-side new.)
Regardless, the code has been refactored to unify the cuda and hip approaches.  We take the approach where the host allocates memory on the device using cudaMalloc or hipMalloc and then placement new is used to construct objects in that memory on the device.

At this time, the failing test is due to an error with the automated test runner, which, before any tests run, reports
Error encountered during job: you are currently unauthorized to use this runner (refer to system logs for additional details)

I don't know that is causing this error.  But, when run mannually on the same system, the tests pass, so I will go ahead and merge.


Improve PR diff view on mobile

Show diff stats only on large screens

these are already shown in tabs, so no need for this duplicate information on small screens

Hide viewed files information on small screens

Github does the same and this gives us more free space on small screens

Review bar now doesn't wrap so we don't need the 77px even on very small screens

(the sticky headers are still working)

these are already shown in tabs, so no need for this duplicate information (and it takes just screen space)

Good catch. I'll readd the information on non mobile devices.
Do you think it's also a needed part on mobile devices?

No, there I absolutely agree that we need to slim down the interface as much as possible.

Updated codes and screenshots - large screens will still show the information.


What is the repo for the primary app https://covid19india-network.now.sh?

I understand that this is the github repo for the clusters section of covid19india.org (https://cluster.covid19india.org/)
Can someone point me to the github repo for the primary app https://www.covid19india.org/ ?

Got the info. https://github.com/covid19india.
Closing this issue.


Support ETC2 texture format.

Note: I tested both native GLES3.0 and WebGL2, but I cannot test mobile Vulkan/WebGPU right now.

I see that Linux tests fail but I am not sure how to fix that.
We need to somehow check whether ETC2 is supported on desktop OpenGL but I cannot find appropriate extension.


Cannot locate rosdep definition for [rmf_obstacle_msgs]

See https://github.com/open-rmf/rmf_obstacle_detectors/issues/8#issuecomment-1386782158
Closing this since its a duplicate of other open tickets.


Question: Displaying Subcortical structures & Cerebellum in PySurfer?

Dear team,
I'm struggling to determine whether it is possible to display subcortical structures and cerebellum (i.e the parcellations included in the aparc+aseg.mgz freesurfer file.
PySurfer does an excellent job at plotting individual cortical rois from this file. However, it does not plot the following ROIs which are contained within this file:
Accumbens-area
Amygdala
Caudate
Cerebellum-Cortex
Cerebellum-White-Matter
Cerebral-White-Matter
Hippocampus
Pallidum
Putamen
Thalamus-Proper
If PySurfer is not built for this then can perhaps point me in the direction of something that will (I've looked at Nilearn but this doesn't seem capable either) .

How are you hoping to visualize them? PySurfer works with surfaces (hence the name). It could be possible to tessellate the volume masks for the various subcortical ROIs and then plot them in mayavi, possibly in conjunction with a PySurfer cortex mesh, but it's not really something we directly support.

Also to clarify:

PySurfer does an excellent job at plotting individual cortical rois from this file. However, it does not plot the following ROIs which are contained within this file.

PySurfer is not actually using that file. It's using the ?h.aparc.annot parcellation, which is defined on the surface. As part of recon-all, those ROIs get mapped into the volume, but it's the volumetric representation isn't used by PySurfer.

Sounds like the question has been answered so I'll close


feat: CORE-863 Build from Rocky linux 8 base image

This uses a Rocky linux 8.7 image, this distro is a replacement for CentOS which is going EOL next year.  This allows us to have updated packages and minimal vulnerabilities.

This brings our CVE's down from over 2500 CVE's to 5, I believe those will be patched soon
Total: 5 (UNKNOWN: 0, LOW: 0, MEDIUM: 2, HIGH: 1, CRITICAL: 2)
+---------------------------+---------------------+----------+-------------------+--------------------------------+----------------------------------------------+
|          LIBRARY          |  VULNERABILITY ID   | SEVERITY | INSTALLED VERSION |         FIXED VERSION          |                    TITLE                     |
+---------------------------+---------------------+----------+-------------------+--------------------------------+----------------------------------------------+
| org.postgresql:postgresql | CVE-2022-21724      | CRITICAL | 42.2.14           | 42.3.2, 42.2.25                | jdbc-postgresql: Unchecked                   |
|                           |                     |          |                   |                                | Class Instantiation when                     |
|                           |                     |          |                   |                                | providing Plugin Classes                     |
|                           |                     |          |                   |                                | -->avd.aquasec.com/nvd/cve-2022-21724        |

                      +---------------------+          +                   +--------------------------------+----------------------------------------------+

|                           | CVE-2022-26520      |          |                   | 42.3.3                         | postgresql-jdbc: Arbitrary                   |
|                           |                     |          |                   |                                | File Write Vulnerability                     |
|                           |                     |          |                   |                                | -->avd.aquasec.com/nvd/cve-2022-26520        |

                      +---------------------+----------+                   +--------------------------------+----------------------------------------------+

|                           | CVE-2022-31197      | HIGH     |                   | 42.3.7, 42.4.1, 42.2.26        | postgresql: SQL Injection                    |
|                           |                     |          |                   |                                | in ResultSet.refreshRow()                    |
|                           |                     |          |                   |                                | with malicious column names                  |
|                           |                     |          |                   |                                | -->avd.aquasec.com/nvd/cve-2022-31197        |

                      +---------------------+----------+                   +--------------------------------+----------------------------------------------+

|                           | CVE-2022-41946      | MEDIUM   |                   | 42.3.8, 42.5.1, 42.4.3,        | postgresql-jdbc:                             |
|                           |                     |          |                   | 42.2.27                        | PreparedStatement.setText(int,               |
|                           |                     |          |                   |                                | InputStream) will create a temporary         |
|                           |                     |          |                   |                                | file if the InputStream is...                |
|                           |                     |          |                   |                                | -->avd.aquasec.com/nvd/cve-2022-41946        |

                      +---------------------+          +                   +--------------------------------+----------------------------------------------+

|                           | GHSA-673j-qm5f-xpv8 |          |                   | 42.3.3                         | pgjdbc Arbitrary File Write Vulnerability    |
|                           |                     |          |                   |                                | -->github.com/advisories/GHSA-673j-qm5f-xpv8 |
+---------------------------+---------------------+----------+-------------------+--------------------------------+----------------------------------------------+

building on M1 Mac I got:

#17 [ 8/13] RUN python3.8 -c "import nltk; nltk.download('punkt')" &&   python3.8 -c "import nltk; nltk.download('averaged_perceptron_tagger')" &&   python3.8 -c "from unstructured.ingest.doc_processor.generalized import initialize; initialize()"
#17 0.454 Traceback (most recent call last):
#17 0.454   File "<string>", line 1, in <module>
#17 0.454 ModuleNotFoundError: No module named 'nltk'
#17 ERROR: process "/bin/sh -c python3.8 -c \"import nltk; nltk.download('punkt')\" &&   python3.8 -c \"import nltk; nltk.download('averaged_perceptron_tagger')\" &&   python3.8 -c \"from unstructured.ingest.doc_processor.generalized import initialize; initialize()\"" did not complete successfully: exit code: 1
------
 > [ 8/13] RUN python3.8 -c "import nltk; nltk.download('punkt')" &&   python3.8 -c "import nltk; nltk.download('averaged_perceptron_tagger')" &&   python3.8 -c "from unstructured.ingest.doc_processor.generalized import initialize; initialize()":
#17 0.454 Traceback (most recent call last):
#17 0.454   File "<string>", line 1, in <module>
#17 0.454 ModuleNotFoundError: No module named 'nltk'
------
Dockerfile:39
--------------------
  38 |
  39 | >>> RUN python3.8 -c "import nltk; nltk.download('punkt')" && \
  40 | >>>   python3.8 -c "import nltk; nltk.download('averaged_perceptron_tagger')" && \
  41 | >>>   python3.8 -c "from unstructured.ingest.doc_processor.generalized import initialize; initialize()"
  42 |
--------------------
ERROR: failed to solve: process "/bin/sh -c python3.8 -c \"import nltk; nltk.download('punkt')\" &&   python3.8 -c \"import nltk; nltk.download('averaged_perceptron_tagger')\" &&   python3.8 -c \"from unstructured.ingest.doc_processor.generalized import initialize; initialize()\"" did not complete successfully: exit code: 1
make: *** [docker-build] Error 1

Any ideas?

Successfully built on an EC2 AMD Linux instance. Tested end to end with make docker-start-api and a test request with:
PORT=${1:-"8000"}
export API_HOST="http://localhost:$PORT/general/v0/general"
curl -X 'POST' \
  "$API_HOST" \
  -H 'accept: application/json' \
  -H 'Content-Type: multipart/form-data' \
  -H 'x-unstructured-api-key: foo' \
  -F<EMAIL_ADDRESS>\
  -F 'coordinates=true' \
  | jq -C .

I will run build on my laptop again to make sure it's working, not sure why you saw the ntlk stuff, that's the first time I have seen that here.

I found out also the current CVE's affect ARM only ATM, AMD image is CVE free!

I will run build on my laptop again to make sure it's working, not sure why you saw the ntlk stuff, that's the first time I have seen that here.

okay, circling back here from convo outside thread, was simply docker running out of space! a docker prune and rebuild later and this all worked e2e!


fix excessive API calls in Search.js that are exhausting API key limits

Search.js is calling Freesound's API too frequently, resulting in freesound-player not being able to receive the data it needs. I want to try fixing this by adding a dependency array to useEffect() in Search.js.

Fixed!


1.15-1.19 Project Schedule

Description
We need to set up the tasks and scheduling for each Task, each task need to be associated with one or multi PRs.
Task

Account & User Module - @wu-jj

[ ] Implement account sdk in goplus community.
[ ] Implement uuid storage.

Article Module - @Baihhh

[ ] Implement article creation, editing, and deletion functionality.
[ ] Article list view functionality.
[ ] Article details view functionality.
[ ] Support article translation with translation module. @IRONICBo

Markdown Support - @nnnnn319

[ ] Article editor supports Markdown syntax with cherry-markdown.
[ ] Save markdown preview functionality, save to QiNiu Object Storage or local storage. @Baihhh @xhyqaq
[ ] Support goplus highlight.
[ ] Support Video markdown tag. @IRONICBo @xhyqaq

Translation Functionality - @IRONICBo

[x] Implement translation of markdown articles.
[x] Implement batch translation in long text translation.
[ ] Support video to text.
[ ] Support text to audio.

Media Module - @xhyqaq

[ ] Users can upload and delete media files.
[ ] Media file display functionality.
[ ] Media file download functionality.

User Interface - @Kasey0727

[ ] Design and implement user interface in yap html.
[ ] Design and implement article interface in yap html.
[ ] Design and implement article list(home) interface in yap html.

Everyone is expected to complete their part by the end of this week, and submit their code and test reports. If you encounter any issues or need help during the development process, please raise it in the issue promptly.

Implement translation of markdown articles PR. https://github.com/goplus/community/pull/32

Implement translation process for video PR. https://github.com/goplus/community/pull/43

Add yap home page PR. https://github.com/goplus/community/pull/48


Update SolarIrradianceSensor mappings

What
We recently updated brick's Solar Radiance Sensor to Solar Irradiance Sensor. We are now updating the mappings to reflect the change.
Why
Improved accuracy of Willow mappings
Tested
Valid DTMI and duplicates

@connorjcantrell  could you close this and create a new PR due to the conflicts?

@jbkoh these mappings have already been added in another PR. Deleting branch


config property not defaulting to true if verbose property included

grunt: 0.4.5
grunt-jscs: 2.6.0
.jscsrc content: any
In "jscs.options" object, if config property is omitted (and should therefore default to true), errors will be reported according to the .jscsrc file (as if config === true) but only if the verbose property is not included. In other words, if config property is omitted and verbose property is included (verbose === true or verbose === false), errors will not be reported according to the .jscsrc file.

Yes, that happens for any jscs option, you think we should merge those options?

Sorry, but I still don't fully understand the current behavior. What is the config option in the grunt-jscs plugin being set to when verbose is explicitly set to a value? And why is an explicit verbose property affecting the value of the config property?

We probably not gonna fix this at this point


Incorrect "variable is not accessed" messages

Environment data

Language Server version: v2020.12.1
OS and version: Mac OS Mojave 10.14.6
Python version: 3.7.5 with pyenv

Expected behaviour
When a variable is passed to a function/method that Pylance couldn't introspect (sometimes it's not possible for various reasons), it should not be considered unused.
Actual behaviour
It's grayed out with "variable is not used" message on hover.
Code Snippet / Additional information
I think the best thing I can do here is to show an image of actual code. Stream.from_postgres_increment is a class method, but it's nowhere to be found in Stream class definition: it's actually loaded later on via entry_points mechanism. Pylance can't run introspection on it (and that's totally OK), but the problem is that it also counts all variables passed to this method as unused, while they are obviously used.

I don't think there's anything actually "breaking" here, it's more of a logic problem, so I skipped the logs. Will add if you need them.

Could you provide a self-contained, minimal code sample that reproduces the problem you're seeing? I don't have enough context from the screen shot above to repro the problem.

I played around a bit and I think it has something to do with how the package was installed.
First, install a package through pip+git:
pip install git+https://github.com/python-streamz/streamz.git

Then, create this file:
from streamz import Stream

def test(x=0):
    y = 1
    Stream.nomethod(x=x, y=y)

x and y should be gray. If I do the same but install from PyPi (pip install streamz), everything works as expected.

This issue has been fixed in version 2020.12.2, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/master/CHANGELOG.md#2020122-11-december-2020


Strip exited 139 workaround

Description
Workaround for error : strip exited with code 139
https://github.com/xamarin/xamarin-macios/issues/19157
This error prevents the app from installing on iOS devices.
Type of change

Bug fix

Platforms tested on

[x] MAUI iOS

Checklist

[x] Self-review of changes
[x] All changes work as expected on all affected platforms
[x] There are no warnings related to changes
[x] No unrelated changes have been made to any other code or project files

cc: @duffh @car10567


deployent: update cli download location

Download future builds from the pomerium/cli repo.

can you push this to my branch instead

what about downloading the proto file?

what about downloading the proto file?

Will do that separately and probably push from the cli repo instead.


Values double ended

Just so I don't lose this for now

this hasnt been touched in ages - happy to reopen it if you want to get it merged


Documentation not available for Data storage REST APIs (Extension Management)

The documentation for the data storage service of Azure DevOps extension talks about the REST APIs very briefly. What all operations are supported by the REST APIs or how to consume them is not mentioned anywhere.

HI @saleelpk95, Thanks for reaching out. The following links will hopefully help. Try the Azure DevOps REST API reference, and then try Get started with REST APIs.  Please @mention me if you have any further questions. Thank you!

HI @saleelpk95, Thanks for reaching out. The following links will hopefully help. Try the Azure DevOps REST API reference, and then try [Get started with REST APIs](https://docs.microsoft.com/

HI @saleelpk95, Thanks for reaching out. The following links will hopefully help. Try the Azure DevOps REST API reference, and then try Get started with REST APIs. Please @mention me if you have any further questions. Thank you!

@ch

HI @saleelpk95, Thanks for reaching out. The following links will hopefully help. Try the Azure DevOps REST API reference, and then try Get started with REST APIs. Please @mention me if you have any further questions. Thank you!

@chcomley
The APIs I have mentioned are not a part of the documentation. I have however figured out how to do it. I'll reply back with the details so that it could help the community. It'd be great if the same can be added to the documentation.

@saleelpk95 Absolutely! I'll update the documentation accordingly, once you've commented with your solution. Thanks so much for helping the community! We all appreciate it!


Fix links in README.md for Docker Hub

Some links refer to files in the GitHub repo using a relative URL. This does not work from the Docker Hub page. Now fixed by replacing the links to the absolute GitHub page in the master branch.
See #277

Thanks!


Improve Antiforgery section titles to help reduce confusion

Context: https://stackoverflow.com/questions/70361843/changes-in-client-side-if-we-are-setting-httpcookie-httponly-as-true
I think we should rename some of the headings here:

Current
Suggested
Rationale

ASP.NET Core antiforgery configuration
Antiforgery in ASP.NET Core
This is less about configuration and more about usage.

Antiforgery options
Configure Antiforgery with AntiforgeryOptions
This is where we actually describe configuration.

Configure antiforgery features with IAntiforgery
Generate antiforgery tokens with IAntiforgery
This is about using IAntiforgery to get/store tokens and not about configuration.

@Rick-Anderson Please review.

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

ID: 47b14f57-82ac-a2e2-cbc7-22a81a60f4ac
Version Independent ID: bffca13c-223f-c61f-9cb2-9da8811eecfa
Content: Prevent Cross-Site Request Forgery (XSRF/CSRF) attacks in ASP.NET Core
Content Source: aspnetcore/security/anti-request-forgery.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @steve-smith
Microsoft Alias: riande

Much better. We need to be looking for ways to improve SEO.


mailcatcher can't show any mail

I already install mailcatcher. can access to http://ipaddress:1080
But It's not show something in mailcatcher(empty e-mail )

I'm changed configure in php.ini
smtp_port = 1025
sendmail_from =<EMAIL_ADDRESS>sendmail_path = /usr/local/rvm/gems/ruby-1.9.3-p551/bin/catchmail --smtp-ip <IP_ADDRESS>  --smtp-port 1025
and restart httpd service
but still not have anything in mailcatcher. how to debug it.
what wrong?

I'm already test it
creat php file and run.


HOMEPAGE:  My Work Items widget empty states

For this widget - we need multiple empty states:

Empty state 1:  There are no spaces.  In that case I think the space dropdown should not show, and we should have an empty state that describes the need to create a space (similar to current text shown).
Empty state 2:  You have selected a space from the dropdown, but the space has no work items assigned to you.  There should be a different empty state in this situation that shows that you have no work items assigned to you.
Need to design these and send them to Dan L.

@jryhanyc - if you get a chance can you check this out?

@catrobson will do

@catrobson Two empty states are ready to review

@mindreeper2420 - hope to connect with you on this when you return.


ShowDialog() ignores WindowStartupLocation

Using the latest code from master, setting a window WindowStartupLocation property has no effect when using ShowDialog(..). If I use Show() it works properly
I've tried both WindowStartupLocation.CenterScreen and WindowStartupLocation.CenterOwner
Example
var window = new Window();
window.Height = 100;
window.Width = 100;
window.Content = new TextBlock { Foreground = Brushes.White, Text = "Test" };
window.WindowStartupLocation = WindowStartupLocation.CenterScreen;
window.ShowDialog(Application.Current.MainWindow);

Expected behaviour
Window should be displayed in the desired location

It seems the call to https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Controls/Window.cs#L463 is missing in the ShowDialog implementation.

WindowStartupLocation.CenterOwner is probably still broken on MacOS, but works as it should on Windows.
Right now showing a dialog on Mac centers dialog to the right of the parent window.

CenterOwner works fine on macOS in my app.


operators warning error: RuntimeWarning: invalid value encountered in power

This is my error, could you give me some advice to avoid it?

I am going to close this issue. For direct coding support feel free to join our discord server and ask the community.
Please provide some context (what are you trying to do?) and some reproducible code next time.
By just posting your error message, it is very unlikely you will get the issue resolved.


Remove develop branch

Closes #23

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@7fd720b). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main       #27   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?        14
  Branches        ?         0
========================================
  Hits            ?        14
  Misses          ?         0
  Partials        ?         0

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 7fd720b...5735092. Read the comment docs.


correct change detection and automatic reloading with webpack

1. Why is this change necessary?
When using the ./psh.phar administration:watch command with installed plugins, the plugin files are compiled successfully. But changes to the plugin administration js files are ignored and do not lead to a recompile and browser reload automatically.
2. What does this change do, exactly?
Enables hot reloading and compiling for plugin administration.js changes
3. Describe each step to reproduce the issue or behaviour.

install SwagPaypal into a development installation shopware/development
administration:watch
change the main.js of SwagPaypal (app/administration/something)
no automatic recompile 🤦

4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hi @jochenmanz,
I have followed your steps to reproduce this problem, but I am not getting the same results. When I make changes to the plugin sources, the changes are detected and the plugin sources are compiled. Reloading also works.
I don't think that will require this change.

@raknison it depends, in any other location the WebpackPluginInjector gets the current webpack config as a parameter. Without that parameter the registerPluginsToWebpackConfig Funktion is not called. Why its working for you i really cant say, but without the changed lines, at least on my machine, the change detection does not work. Strangely the administration is compiled correctly, also when using watch.

@raknison could you provide a feedback to my answer?

@raknison could you provide a feedback to my answer?

@jochenmanz Maybe I found a way to reproduce your issue - still investigating. I provide you an update early next week.

@raknison do you have a update on that? i can still reproduce it, even with the newest 6.2.2 dev template.

@raknison do you have a update on that? i can still reproduce it, even with the newest 6.2.2 dev template.

Hi @jochenmanz
I'm sorry for the delay. In the meantime I found out that there was a refactoring regarding the webpack configuration, which will be shipped in the next version soon.
Here are the changes
The file with your changes has already been removed there and the logic has been reworked a bit.
Do you have the same issues on the current master branch? Maybe we can close this PR.

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://docs.shopware.com/en/shopware-platform-dev-en/contribution/contribution-guideline


Added note about ParameterStore

This snippet is too generic and it may confuse users of Parameter Store backend in provided examples .

You @zen for your contribution!

/approve

/merge


Demo app?

Hey @shadowhand,
I made a demo app using equip 1 https://github.com/aminemat/scheduler.
The app was for a temporary use and still lacks many things, I was planning to ditch it as soon as it serves it's purpose, but now that Equip 2 is here I could refactor it and you could link it in the docs as a non official demo app for Equip.
Having a functional demo app with proper tests can go a long way in explaining the merits of the framework but I don't want to invest time in it if you don't think it's a good idea.
Thoughts?

It looks really good! I've forked it if you want to delete your version. If we end up using it, we'll be sure to give credit!

Sounds good!


Set up package deployment to PyPI

Set up deployment of the client package to PyPI (possibly automated through GitHub actions)

Later in the automation of this would be awsome to follow the way it's done in SDK:

There is only the master branch
Any commit to master releases BETA automatically
To release a LATEST you just create a release in Github UI


DateBlock can't be serialized to JSON

Hi @BertrandBordage , I love what I see from this package so far. I've hit one issue with rendering a DateBlock in the Wagtail admin. If the DateBlock has a None value, all is well, but if it has a date value, the template throws a 500 at render_with_errors. I tracked the issue down to widgets.InputJSONEncoder, and also found a patch for a similar issue with wagtail-condensedinlinepanel. I'm curious if you've seen the same thing at any point, or if you can reproduce the issue?
For the time being, I've submitted a proof-of-concept patch in this PR, for you to consider.

Fixed by 4d4e574fa2df55d8c3befe54719bab262d40943d.
This was not the same issue as in wagtail-condensedinlinepanel.
I could make a clean non-Wagtail-specific solution :slightly_smiling_face:
Unfortunately, because the change event is not correctly triggered by the widget, I had to amend the picker JavaScript as well.


Please make docker packages for armv7/arm64/amd64

Please make docker packages for armv7/arm64/amd64
@rozhuk-im
Thanks!

There is only source based distribution in this project.
Docker image does not make sense (IMHO):

there is no extra run time deps - any container will have negative profit by resources usage and administration;
multicast required in most use cases - extra network setup will be required.


TypeScript declarations of Constants does not match with Javascript

The typescript declaration of constants is not correct.
For example, in your Javascript, The QUEUED_CALL_STARTED in EVENT_TYPE is under a new section "VOICE"
EVENT_TYPE: {
VOICE: {
QUEUED_CALL_STARTED: 'QUEUED_CALL_STARTED',
}
}
Whereas in typescript it is
EVENT_TYPE: {
QUEUED_CALL_STARTED: 'QUEUED_CALL_STARTED',
}
It looks like the types.d.ts is not mapped correctly to the actual types.js file.
We can't upgrade the base connector and are unable to use the new features.

@dlouvton

@ashar-saleem I created enums in types.d.js. Can you use those instead?

Hi Paolo,
I am already using them, but I really want to avoid mixing JS declarations with TS.


dev-cmd/bump-revision: insert revision after license

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Tested this on Homebrew/homebrew-core#57884 for bumping the revisions for hashlink and ortp
Fixes problem with style audit:
== Formula/hashlink.rb ==
C:  7:  3: license (line 7) should be put before revision (line 6)

Thanks for the quick fix!


Cannot publish large occupancy grid messages fast enough (desired 15hz, but only getting 5hz)

I have a question!
Here is my question:
Thanks for building this! So I'm trying to send occupancy grid information from Unity to Ros (I'm simulating a robot's environment and then passing the corresponding occupancy grid to the actual robot's path planning system to test how it performs). This requires me to send an occupancy grid message with a grid size of 400x400 at 15Hz, as this is how the robot functions in the real world. Unfortunately, I'm having trouble sending these large messages (~320KB) fast enough. I'm monitoring the performance of my SendAsync() function, and for every 15 messages I try to send per second, only 5 respond with a success message, which is consistent with what I'm seeing echoing the relevant ROS topic and visualizing it in RViz. I'm pretty confident this is a data size issue, as smaller grid sizes pose no problem, and the problem gets worse as I try to pass larger data arrays in the message.
I worked hard to isolate the problem to just be due to the websocket, which required serializing my json string in a custom way (JSON.serialize seems to allocate 2x memory when acting on sbyte arrays, so I use a JSON text writer instead for the data array). My unity project runs at around 30Hz, so I don't think the issue is there (of course changing SendAsync to Send makes the project run at 5Hz as the code waits for success messages).
Any ideas on how to speed up performance on sending large messages through the web socket? Any ideas on how to shrink my message size? The size comes out to around 320KB because the json message with a 160,000 character array is written to a string and then converted to bytes (this is necessary for RosBridge communication, correct?).
Thank you!
UPDATE: Here's some important information I should have included. I'm actually running on Ubuntu 16.04, using ROS Kintetic, and Unity 2018.2.0f2. Sorry I didn't include this information before.
Below I have also copied code from my 'custom' RosSocket class. You can see in the Serialize function I dynamically create my json string, with the 'grid' variable on the 'Board' object providing a string of the sbyte array of occupancy map information. In my SendOperation function which calls SendAsync, I use a message counter and a 'completed' action delegate to keep track of how many messages return success messages (every time completed is marked to 'true' I decrease my message counter by 1. So after 1 second, the counter reads 10: 15 total calls made as I specify in my Unity time publisher, and only 5 being completed successfully).
Relevant Code:
private int messageCount = 0;

private void sendOperation(Operation operation)
{
    messageCount++;
    Action<bool> completed = new Action<bool>(MessageComplete);
    float timeBefore = Time.realtimeSinceStartup;
    webSocket.SendAsync(Serialize(operation), completed);
    Debug.Log("Message Count: " + messageCount);
}

private void MessageComplete(bool completed) {
    Debug.Log("MessageComplete is running.");
    if (completed)
        {
            messageCount--;
            Debug.Log("Message count subtraction! - " + messageCount);
        }
}

public void Publish(string id, Message msg)
{
    UnityEngine.Profiling.Profiler.BeginSample("Publish");
    Publisher publisher;
    if (publishers.TryGetValue(id, out publisher))
        sendOperation(new Publication(id, publisher.Topic, msg));
    UnityEngine.Profiling.Profiler.EndSample();
}

public static byte[] Serialize(object obj)
{
    UnityEngine.Profiling.Profiler.BeginSample("Serialize first part");
    string json = JsonConvert.SerializeObject(obj);
    UnityEngine.Profiling.Profiler.EndSample();

    UnityEngine.Profiling.Profiler.BeginSample("Serialize second part");
    int dataIndex = json.IndexOf("data");
    if(dataIndex>0){
        json = json.Substring(0,dataIndex+6) + GameObject.Find("Board").GetComponent<DemoGridProvider>().grid + json.Substring(dataIndex+10);
    }
    UnityEngine.Profiling.Profiler.EndSample();

    UnityEngine.Profiling.Profiler.BeginSample("encode");
    byte[] buffer = Encoding.ASCII.GetBytes(json);
    UnityEngine.Profiling.Profiler.EndSample();
    return buffer;
}

Hi @p-buddy !
Have you made any further progress in the meantime?
In your problem description above I understand that the JSON serialziation doubles the size of your message, but does this operation also cost the time? How much time is consumed by the string to byte conversion? How much by your network connection? How are your CPUs and memory doing?
rosbridge_suite requires JSON strings. These need to be send as byte arrays through the WebSocket connection. (websocket-sharp has a Send(string message) method but it converts the string into bytes as well.)
Here are some ideas I have in mind to possibly improve the performance:

Write a custom JSON serializer for your sbyte arrays. Do be honest, I don't know why size doubles. An sbyte should have the same size as a byte in my understanding. You can try and teach Newtonsoft.JSON how to do it.
In Unity, rather than serializing the whole object every time, only update those parts that have actually changed  directly in the JSON string (very hacky).
Partition the OccupancyGrid in smaller areas. To my understanding, when the robot moves out of one area, a new area of the same size is instantiated. Only active areas are communciated.
From Unity only communicate the actual changes of your Occupancy Grid to ROS (in a self-defined message type) and write a corresponding subscriber node in ROS that combines these updates to a whole occupancy grid message again.

Does any idea sound useful to you? Please keep us updated on your progress here!

closing due to inactivity.

I ran into similar issues with JSON Serialization performance when trying to publish Laser Scan / Point Cloud data. I've decided to remove the need for sending these messages until BSON is supported or I can find another solution, but I wanted to include some code and test data for this issue.
I'm simulating two Laser Scanners which output ~1300 points at ~15 Hz (though the actual hardware can output from 200 Hz to 4000 Hz). I'm outputting data in a PointCloud rather than LaserScan as the data is output in X,Y pairs from the scanner; it also allows me to append additional information about each point as the data moves through the pipeline, without having to change messages. Unfortunately, the PointCloud uses at least 2x the bandwidth of the Laser Scan as it outputs X, Y instead of just distance with angle implied.
Using the default Reflection based JSON Serialization each publish took 30-40 ms per scanner; this seriously limited the frame rate, so I implemented the following Custom JSON Serializer:
    public class PointCloudConverter : JsonConverter
    {
        public override bool CanConvert(Type objectType)
        {
            // This converter can be applied to any time (lie)
            return objectType == typeof(PointCloud);
        }

        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
        {
            // we currently support only writing of JSON
            throw new NotImplementedException();
        }

        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
        {
            if (value == null)
            {
                serializer.Serialize(writer, null);
                return;
            }

            // find all properties with type 'int'
            var pc = (PointCloud)value;

            writer.WriteStartObject();

            writer.WritePropertyName("header");

            //header
            writer.WriteStartObject();

            writer.WritePropertyName("seq");
            writer.WriteValue(pc.header.seq);

            writer.WritePropertyName("stamp");

            writer.WriteStartObject();

            writer.WritePropertyName("secs");
            writer.WriteValue(pc.header.stamp.secs);

            writer.WritePropertyName("nsecs");
            writer.WriteValue(pc.header.stamp.nsecs);

            writer.WriteEndObject();

            writer.WritePropertyName("frame_id");
            writer.WriteValue(pc.header.frame_id);

            writer.WriteEndObject();

            // Points

            writer.WritePropertyName("points");
            writer.WriteStartArray();
            foreach (var point in pc.points)
            {
                writer.WriteStartObject();
                writer.WritePropertyName("x");
                writer.WriteValue(point.x);

                writer.WritePropertyName("y");
                writer.WriteValue(point.y);

                writer.WritePropertyName("z");
                writer.WriteValue(point.z);
                writer.WriteEndObject();
            }
            writer.WriteEndArray();

            // Channels

            writer.WritePropertyName("channels");
            writer.WriteStartArray();

            foreach (var channel in pc.channels)
            {
                writer.WriteStartObject();

                writer.WritePropertyName("name");
                writer.WriteValue(channel.name);

                writer.WritePropertyName("values");
                writer.WriteStartArray();

                foreach (var val in channel.values)
                {
                    writer.WriteValue(val);
                }

                writer.WriteEndArray();

                writer.WriteEndObject();
            }

            writer.WriteEndArray();

            writer.WriteEndObject();
        }
    }

This improved the Serialization time to ~15-20 ms per scanner, this was still a pretty limiting factor so I attempted to by pass any reflection and added custom Send methods that accepted JSON Strings that only needed to be converted to bytes (sorry for the hacky code):
            StringBuilder sb = new StringBuilder("", 10000);
            string str = "";
            sb.Append("{\"topic\":");//writer.WriteStartObject();

            //writer.WritePropertyName("topic");

            sb.Append("\"" + topic + "\",");//writer.WriteValue(topic);

            sb.Append("\"msg\":{");//writer.WritePropertyName("msg");

            //writer.WriteStartObject();

            sb.Append("\"header\":{");//writer.WritePropertyName("header");

            //header
            //writer.WriteStartObject();

            sb.Append("\"seq\":");//writer.WritePropertyName("seq");
            sb.Append(pc.header.seq);//writer.WriteValue(pc.header.seq);

            sb.Append(",\"stamp\":{");//writer.WritePropertyName("stamp");

            //writer.WriteStartObject();

            sb.Append("\"secs\":");//writer.WritePropertyName("secs");
            sb.Append(pc.header.stamp.secs);//writer.WriteValue(pc.header.stamp.secs);

            sb.Append(",\"nsecs\":");//writer.WritePropertyName("nsecs");
            sb.Append(pc.header.stamp.nsecs);//writer.WriteValue(pc.header.stamp.nsecs);

            sb.Append("},");//writer.WriteEndObject();

            sb.Append("\"frame_id\":");//writer.WritePropertyName("frame_id");
            sb.Append("\"" + pc.header.frame_id + "\"},");//writer.WriteValue(pc.header.frame_id);

            //writer.WriteEndObject();

            // Points
            sb.Append("\"points\":");//writer.WritePropertyName("points");
            sb.Append("[");//writer.WriteStartArray();
            var start = true;
            foreach (var point in pc.points)
            {
                if (!start)
                {
                    sb.Append(",");
                }
                start = false;
                sb.Append("{");//writer.WriteStartObject();
                sb.Append("\"x\":");//writer.WritePropertyName("x");
                sb.Append(point.x);//writer.WriteValue(point.x);

                sb.Append(",\"y\":");//writer.WritePropertyName("y");
                sb.Append(point.y);//writer.WriteValue(point.y);

                sb.Append(",\"z\":");//writer.WritePropertyName("z");
                sb.Append(point.z);//writer.WriteValue(point.z);
                sb.Append("}");//writer.WriteEndObject();
            }

            sb.Append("]");//writer.WriteEndArray();

            // Channels

            sb.Append(",\"channels\":");//writer.WritePropertyName("channels");
            sb.Append("[");//writer.WriteStartArray();

            foreach (var channel in pc.channels)
            {
                sb.Append("{");//writer.WriteStartObject();

                sb.Append("\"name\":");//writer.WritePropertyName("name");
                sb.Append("\"" + channel.name + "\",");//writer.WriteValue(channel.name);

                sb.Append("\"values\":");//writer.WritePropertyName("values");
                sb.Append("[");//writer.WriteStartArray();
                bool val_start = true;
                foreach (var val in channel.values)
                {
                    if (!val_start)
                    {
                        sb.Append(",");
                    }
                    val_start = false;
                    sb.Append(val);//writer.WriteValue(val);
                }

                sb.Append("]");//writer.WriteEndArray();

                sb.Append("}");//writer.WriteEndObject();
            }

            sb.Append("]");//writer.WriteEndArray();

            sb.Append("}");//writer.WriteEndObject();

            sb.Append(",\"op\":");//writer.WritePropertyName("op");
            sb.Append("\"publish\"");//writer.WriteValue("publish");

            sb.Append(",\"id\":");//writer.WritePropertyName("id");
            sb.Append("\"" + topic + "\"");//writer.WriteValue(topic);

            sb.Append("}");//writer.WriteEndObject();

            return sb.ToString();

And this brought the Serialization time down to ~4-8 ms. While a large improvement, I still find this to be quite slow. Further, it seems that rosbridge starts to become unstable with dramatically increased delays (commands can be delayed by seconds) and lost messages.
I'm not sure what's causing the instability, but at this point I've decided to change my architecture to avoid both issues, but it seems to be a pretty limiting factor.
Obviously, when BSON is supported it may help tremendously.


Extended support for referencing MCAD CRD resources

This PR follows up #139, to support the complete definition of the MCAD CRD resources reference, as suggested here:
https://github.com/project-codeflare/codeflare-operator/pull/139/files/26da2ebf8c15155a3a6eea1d48901d2f61a76f09#diff-a2ca4be61aa64f38fb728281ca786796093cc29b47cd1b5ba2b1d2050c3732f6

/assign @anishasthana @KPostOffice


sweetalert2.scss any variables?

Hello,
I see there is a scss file. I can't seem to find any variables? For example I would like to change default blue color, or green to something different?
Thank you!

default color of which element you want to change?

Well as for now, if I understand correct, those colors are inherited from Bootstrap, no?
I am using this inside Foundation6.
I would like to change color, of default buttton (blue) for example.
I know I can kae it via customization via params, or so. But wondered, if maybe I could change some defaults.

No, it's not a Bootstrap.
To use styling from Foundation6 you can do this:
confirmButtonClass: '<Foundation6 classes>',
buttonsStyling: false,
``

Realted issue: https://github.com/limonte/sweetalert2/issues/59


[JSDoc] Add missing documentation for botbuilder-dialogs/memory/scopes

Addresses # 2602
Description
This PR adds all the missing documentation based on the errors thrown for files in the memory/scopes directory of the botbuilder-dialogs library.
Some documentation is ported and adapted from botbuilder-dotnet
Note: the warn property is replaced with error in the last PR of the series adding the documentation to avoid build failures.
Specific Changes

Added JSDoc comments in all methods.

Testing
The following image shows the ESLint Report with no JSDoc errors for the updated files.

Promoted


Create a column summary for projectV2 sprint project

Create a column summary using pandas on data returned from the new sprint project type
Display the data in the same way that we measure it sprints
The current workflow here involves shell scripts.
These are going to be replaced by a python file.
The mechanics of the process are not going to change:

Data is collected from within a python wrapper using pyGitHub and graphql
That data is expressed as xml.
A xslt processor is invoked with an xsl file that transforms the data into a flat file.

This issue will stop there.
A follow-on issue will then (hopefully simply) to build a workflow with code that already exists to:

read the flat file into a dataframe.
create a summary report from that.

deprecated.
Not going in this direction anymore


Added more escaping characters for HTML

Hi.
I've parsed a list of html escape characters (that you can find here), and added them to the html.cr file.
Hope it's good :smile:

@Ryuuzakis
Tests are failing, take a look: https://travis-ci.org/manastech/crystal/jobs/110107440
crystal spec ./spec/std/html_spec.cr:6 # HTML .escape does not change a safe string
crystal spec ./spec/std/html_spec.cr:12 # HTML .escape escapes dangerous characters from a string

I'm not sure we need to escape those characters. The current implementation contains problematic characters that you always want to escape in HTML, like <, >, ', " and &. I think I needed to escape other chars, like accented spanish chars, when I was programming in PHP maybe 8 years ago, but I think now that's not needed anymore.

Oh right ! I forgot to launch the tests, my bad.
So you say we should leave it as it was ? Or maybe remove some unneeded escaped characters ? I just lazily let them all, but I could sort them a bit to make them pass the tests.

I think the current list is fine. But I honestly don't know when and what characters need to always be escaped in HTML.

Okay, so I've searched a bit, and it seems that we need a little more characters escaping that the set there already is.
According to this blog article, which refers and explains content from the OWASP, you need more characters to prevent XSS injections.
I will update my branch by limiting the characters to what is really needed when I have some time to spare :)

Perhaps we should look at other implementations of HTML escaping before decide? Basically the substitution list looks too big now :stuck_out_tongue_winking_eye:

@Ryuuzakis goot catch. We should indeed escape some more characters to be (almost) safe.

I've done the modifications I talked about earlier, and added a new test case for the escape method.
Telle me what you think about it :)

@Ryuuzakis Thanks!
Do we need to convert spaces to  , though? I also don't understand why the spaces are not converted in your spec...

The space that is protected is the non-breaking space, this is why the classical space is not converted.

@Ryuuzakis I see. I think it's very confusing if we leave it like that. Another confusing case is the single quote, it looks like ''', which looks like a typo. Better change ''' to '\'' and the non-breaking space to ``\u{a0}'or160.chr`.

@asterite I agree with you, your solution was better, so I just implemented it. I've also added a test case to test the non-breaking spaces and normal spaces.

@Ryuuzakis Thank you! We are almost done here. The indentation is wrong in the spec file. Could you run crystal tool format to fix that?

Done !

Thank you!


SQL Error: Internal jdbc driver error

I work on some pretty complex Oracle queries, and I find that often times after running some queries, coming back to dbeaver after leaving it open for a bit, whenever I try to run any other queries I get "SQL Error: Internal jdbc driver error". I have to invalidate the connection for it to start working again

I should note that the jars I have added are ojdbc8, orai18n, and xdb6 downloaded from here, and xmlparserv2.jar

Could you post error stacktrace next time you will see this error?
Open Error Log view and find the error in the list. Double click and copy/paste stacktrace.

java.sql.SQLException: Internal jdbc driver error
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.handleExecuteError(JDBCStatementImpl.java:309)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:343)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:134)
	at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.executeStatement(SQLQueryJob.java:448)
	at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.lambda$0(SQLQueryJob.java:392)
	at org.jkiss.dbeaver.model.DBUtils.tryExecuteRecover(DBUtils.java:1598)
	at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.executeSingleQuery(SQLQueryJob.java:390)
	at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.extractData(SQLQueryJob.java:822)
	at org.jkiss.dbeaver.ui.editors.sql.SQLEditor$QueryResultsContainer.readData(SQLEditor.java:2384)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:93)
	at org.jkiss.dbeaver.model.DBUtils.tryExecuteRecover(DBUtils.java:1598)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:91)
	at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:95)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:56)
Caused by: java.lang.IllegalArgumentException
	at java.nio.Buffer.limit(Buffer.java:275)
	at oracle.net.ns.NIOPacket.readFromSocketChannel(NIOPacket.java:142)
	at oracle.net.ns.NIOPacket.readFromSocketChannel(NIOPacket.java:101)
	at oracle.net.ns.NIONSDataChannel.readDataFromSocketChannel(NIONSDataChannel.java:80)
	at oracle.net.ano.CryptoNIONSDataChannel.readDataFromSocketChannel(Unknown Source)
	at oracle.jdbc.driver.T4CMAREngineNIO.prepareForReading(T4CMAREngineNIO.java:98)
	at oracle.jdbc.driver.T4CMAREngineNIO.unmarshalUB1(T4CMAREngineNIO.java:534)
	at oracle.jdbc.driver.T4CTTIfun.receive(T4CTTIfun.java:485)
	at oracle.jdbc.driver.T4CTTIfun.doRPC(T4CTTIfun.java:252)
	at oracle.jdbc.driver.T4C8Oall.doOALL(T4C8Oall.java:612)
	at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:213)
	at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:37)
	at oracle.jdbc.driver.T4CStatement.executeForDescribe(T4CStatement.java:733)
	at oracle.jdbc.driver.OracleStatement.executeMaybeDescribe(OracleStatement.java:904)
	at oracle.jdbc.driver.OracleStatement.doExecuteWithTimeout(OracleStatement.java:1082)
	at oracle.jdbc.driver.OracleStatement.doScrollExecuteCommon(OracleStatement.java:4646)
	at oracle.jdbc.driver.OracleStatement.doScrollStmtExecuteQuery(OracleStatement.java:4762)
	at oracle.jdbc.driver.OracleStatement.executeInternal(OracleStatement.java:1741)
	at oracle.jdbc.driver.OracleStatement.execute(OracleStatement.java:1692)
	at oracle.jdbc.driver.OracleStatementWrapper.execute(OracleStatementWrapper.java:300)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:341)
	... 12 more

@kufii is issue still reproducible on 6.2.5?

Yes, it's still an issue with jdbc 8

I need some query example in order to reproduce this.
Maybe it is an Oracle driver bug (you could try to upgrade/downgrade driver version).
Or maybe it is somehow related to the way DBeaver processes queries in Oracle.

It happens with literally any query. Select * from table;, go use another application for a bit leaving dbeaver open, then come back and run it again

Ah. Most likely it happens because your connection was timed out and closed by server.
Generally it is handled by dbeaver - we catch specific error codes and reopen connection in case of disconnect.
But in your case driver throws internal error instead of valid Oracle error.
Probably because of driver version mismatch.
What is your Oracle server version and driver version?
You can get this info by clicking on Test Connection in connection settings dialog.

Server:
Oracle Oracle Database 11g Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production
With the Partitioning, OLAP, Data Mining and Real Application Testing options
Driver:
Oracle JDBC driver <IP_ADDRESS>.0

is it still actual?

there aren't any updates for a long time.
Issue closed. If it is still actual for you - feel fee to reopen it


Update dependencies

Codecov Report

Merging #256 (0a641e6) into main (dcc3777) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #256   +/-   ##
=======================================
  Coverage   91.98%   91.98%
=======================================
  Files          10       10
  Lines         848      848
=======================================
  Hits          780      780
  Misses         68       68

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[Suggestion/Question] Did you consider PartialSuccess result?

I know that any result with at least one Error in the Reasons would be marked as a failure, but sometimes operations succeed with some warnings.
So did you consider having a partial success status ? could be configured to act as success or fail for backward compatibility
ps: can contribute to add feature

You can create a class Warning which inherit from Error if you want that a Warning is handled as error (=> failed result). You can also create a class Warning which inherit from Success if you want that a Warning is handled as success (=> success result). For more details about this topic see readme https://github.com/altmann/FluentResults#designing-errors-and-success-messages


Xcode 12 - Camera.swift not building

since upgrading to Xcode 12 that supports ios 14
this framework not building because of this line
       let supportedPixelFormats = videoOutput.availableVideoPixelFormatTypes
it says
Type 'AVCaptureVideoDataOutput' has no member 'availableVideoPixelFormatTypes'
Please we need a fix to this issue I don't understand what it means so I can't find alternative for it
system info :
xcode Version 12.0.1 (12A7300)
macos : 10.15.5 (19F101)

it's Xcode issue.
check https://developer.apple.com/forums/thread/658471

Thank you sijie


[VK] add domain vkvideo.ru

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Example URLs
https://vkvideo.ru/video-127553155_456242961
https://live.vkvideo.ru/panterka
Provide a description that is worded well enough to be understood
vk.com for me today, (for some people earlier) VK began to redirect to this domain vkvideo.ru when clicking on vk.com/video . And there are now links to videos with a different domain
also instead of live.vkplay.ru now live.vkvideo.ru
with the old video domain also available https://vk.com/video-127553155_456242961
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://vkvideo.ru/video-127553155_456242961']
[debug] Encodings: locale cp1251, fs utf-8, pref cp1251, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds [4b5eec0aa] (win_exe)
[debug] Python 3.10.11 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: ffmpeg N-116122-gd241edc2b4-20240703 (setts), ffprobe N-116122-gd241edc2b4-20240703, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.21.0, brotli-1.1.0, certifi-2024.08.30, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.40.1, urllib3-2.2.3, websockets-14.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1837 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-master-builds/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds)
[generic] Extracting URL: https://vkvideo.ru/video-127553155_456242961
[generic] video-127553155_456242961: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] video-127553155_456242961: Extracting information
[debug] Looking for embeds
[debug] Identified a Open Graph video info
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec, channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] video_ext: Downloading 1 format(s): 0
ERROR: The extracted extension ('php') is unusual and will be skipped for safety reasons. If you believe this is an error, please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 177, in wrapper
  File "yt_dlp\YoutubeDL.py", line 3475, in process_info
  File "yt_dlp\YoutubeDL.py", line 3367, in existing_video_file
  File "yt_dlp\YoutubeDL.py", line 3366, in <lambda>
  File "yt_dlp\utils\_utils.py", line 2130, in _change_extension
  File "yt_dlp\utils\_utils.py", line 5212, in sanitize_extension
yt_dlp.utils._UnsafeExtensionError: unsafe file extension: 'php'

the same problem. It would be great if you fix it.
Example URLs
https://vkvideo.ru/@nsasvoemesto
Complete Verbose Output

d:\VK_NaSvoemMeste>rem d:\VK_NaSvoemMeste\yt-dlp.exe -J -vU --flat-playlist --no-cache-dir --ignore-config --user-agent "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36" --no-check-certificate --yes-playlist https://vkvideo.ru/@nsasvoemesto
d:\VK_NaSvoemMeste>d:\VK_NaSvoemMeste\yt-dlp.exe -vU https://vkvideo.ru/@nsasvoemesto
[debug] Command-line config: ['-vU', 'https://vkvideo.ru/@nsasvoemesto']
[debug] Encodings: locale cp1251, fs utf-8, pref cp1251, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [7ea278792] (win_exe)
[debug] Python 3.10.11 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: ffmpeg 6.1.1-full_build-www.gyan.dev (setts), ffprobe 6.1.1-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.21.0, brotli-1.1.0, certifi-2024.08.30, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.40.1, urllib3-2.2.3, websockets-13.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1837 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[generic] Extracting URL: https://vkvideo.ru/@nsasvoemesto
[generic] @nsasvoemesto: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] @nsasvoemesto: Extracting information
[debug] Looking for embeds
[redirect] Following redirect to https://vkvideo.ru/badbrowser.php
[generic] Extracting URL: https://vkvideo.ru/badbrowser.php
[generic] badbrowser: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] badbrowser: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://vkvideo.ru/badbrowser.php
Traceback (most recent call last):
File "yt_dlp\YoutubeDL.py", line 1624, in wrapper
File "yt_dlp\YoutubeDL.py", line 1759, in __extract_info
File "yt_dlp\extractor\common.py", line 742, in extract
File "yt_dlp\extractor\generic.py", line 2553, in _real_extract
yt_dlp.utils.UnsupportedError: Unsupported URL: https://vkvideo.ru/badbrowser.php

I confirm the problem.
the code change and new domain addition has happened today.
one more URL example that also should be added/fixed:
https://vkvideo.ru/playlist/-218471730_-2/video-218471730_456239774

and live.vkvideo.ru

in the night assembly added, BUT with arguments does not want to download, says that cookies are not valid (I updated them), but if without any arguments insert a link, then will be downloaded 🤷♂

in the night assembly added, BUT with arguments does not want to download, says that cookies are not valid (I updated them), but if without any arguments insert a link, then will be downloaded 🤷♂

nobody changed anything

Backup original files before edit!
Temporarily you can find vk.py in yt-dlp folder. Change vk.com into vkvideo.ru and vk\.com into vkvideo\.ru You may need to edit file lazy_extractors.py

why replace if vk.com also works and will remain for videos in posts and vk groups?

Backup original files before edit!
Temporarily you can find vk.py in yt-dlp folder. Change vk.com into vkvideo.ru and vk\.com into vkvideo\.ru You may need to edit file lazy_extractors.py

why replace if vk.com also works and will remain for videos in posts and vk groups?

Yeah, it should be an addition for sure.
(but vkvideo format playlists don't seem to work)

"[VK] add domain vkvideo.ru"   qzmtch opened this issue Nov 26, 2024

And the news was officially announced only today, December 4th. :)
Платформа «VK Видео» запустила обновлённую веб-версию на отдельном домене vkvideo.ru

"[VK] add domain vkvideo.ru"   qzmtch opened this issue Nov 26, 2024

And the news was officially announced only today, December 4th. :) Платформа «VK Видео» запустила обновлённую веб-версию на отдельном домене vkvideo.ru
seriously?
https://vk.com/vkvideo?w=wall-220754053_83366
https://vk.company/ru/press/releases/11917/

We need to add links to playlists
https://vkvideo.ru/playlist/-51890028_-2/video-51890028_456241268

https://vkvideo.ru/video-51890028_456241268
it works

how change endpoint in def _download_payload if url contain 'vkvideo'?


Typo fix "work-in-progrees" to "work-in-progress"

Typo fix

Fixes #0000
{{{ Summarize what you've changed HERE and why }}}

PR targets develop branch, therefore isn't valid.


APM agent not using full server_url

Describe the bug:
I'm having a similar issue that was described in the apm-agent-java issues: https://github.com/elastic/apm-agent-java/issues/339
I am using a url that looks like: https://test.url/with/path
The logs from the APM agent show the following:
 Failed to submit message: 'Connection to APM Server timed out (url: https://test.url/intake/v2/events, timeout: 5 seconds)'
The path that I have at the end gets removed.
To Reproduce

Configure SERVER_URL=https://test.url/
Trigger an APM event

Expected behavior:
I expected that the SERVER_URL variable would take the full URL and not just the domain part.
Environment (please complete the following information)

OS: Using Docker for Mac version <IP_ADDRESS>-mac81 with base image python:3.7
Python version: python 3.7
Framework and version [e.g. Django 2.1]: Flask 1.0.2
APM Server version: docker.elastic.co/apm/apm-server:6.5.4
Agent version: 4.0.3

@RandalMoss thanks for opening the issue. This is caused by a (in my opinion) somewhat surprising behavior of urljoin(url, path) if the path argument starts with a /. It basically strips the whole path from url.
I have a fix ready in #376, and will merge/release it in the coming days


Dockerfile adjustments

The path to requirements.txt is incorrect in the Dockerfile -- it is located in the main directory of the project.
The app directory and the CMD ["polygraph_server"] command are only required if one desires to run frontend. I did not use the frontend app so I skipped those. It might make sense to make a separate Dockerfile for those who only intend to use the methods of the framework to avoid installing extra packages.
There is no jupyter in the requirements which is needed to run the demo notebooks.
CUDA drivers are not pulled by default in the Dockerfile but I assume this depends on the specific hardware configurations. I have used nvcr.io/nvidia/pytorch:24.05-py3 image to use CUDA on my cluster.

@SuperCoolCucumber thanks a lot for your feedback!
Indeed, existing Dockerfile is only suitable for running the demo app, not for the standalone library usage. We will work on adding library-specific Dockerfile in the near future, and will update you when this is being worked on.

@SuperCoolCucumber Can you elaborate a little bit how you were planning to interact with Polygraph in a docker? Run benchmarks from CLI, example notebooks, or you just need an encapsulated environment where library lives and can run, so you can exec into running container and do whatever?

Yeah, I'm using a cluster so I do need and encapsulated environment to use and interact with the library. I've used Polygraph to compute certain uncertainty estimations for my models on my data. I've introduced some changes into model inference and ran the code for some whitebox methods.


ローカルでは動くのに、Webサーバー上ではkuromoji.jsが動作しません

ローカルでは以下のコードの状態でkuromoji.jsを使えるのですが、Webサーバー（Lolipop）にアップした途端に以下のようなエラーが出て使えません。
このエラーは一体どのようにしたら解決できるのでしょうか？サーバーの問題なんでしょうか？
お聞きしたいです。よろしくお願いいたします。
<script src="assets/kuromoji.js"></script>
<script>
kuromoji.builder({ dicPath: "assets/dict" }).build(function (err, tokenizer) {
    // tokenizer is ready
    var path = tokenizer.tokenize("すもももももももものうち");
    console.log(path);
});
</script>

have anyone founded a solution to this?

It seems caused by the server decompress the .gz file automatically.
You should check the webserver config.
Or modify the kuromoji code to prevent decompressing the dict:
https://github.com/hexenq/kuroshiro/issues/27#issuecomment-686428617

how u would modify it? i dont have any clue @larvata

@chinenvinicius please check out the link above.
You need to comment 2 lines: 50-51
https://github.com/takuyaa/kuromoji.js/blob/master/src/loader/BrowserDictionaryLoader.js

in my case im being able to load the file because i put the exact path. however its .dat.gz files are taking 30 secods to load all file. im not sure what im doing wrong. @larvata

@chinenvinicius
Have you tried the kuromoji demo site?
For me, it takes 15s for loading.

so im using  kuroshiro.js to put furigana on kanji.
and kuroshiro.js is dependent on kuromji dictionary.
On local server. its takes less than one second to load it. however when i put on live sever it takes about 30　seconds or even more. does the speed has anything to the sever or the code ？

You can catch the dict data once fetched. Take a look at the approach at https://miusuncle.github.io/japen/.


thin lens OPD sign?

While working on #129 I ran into something I've now manage to confuse myself about: The sign of the defocus term for the thin lens class.
Right now the convention is that a poppy.ThinLens with a positive number of waves of defocus has a negative OPD at its center, and a positive OPD at its edge. I.e. it is a diverging optic such as a biconcave lens or a convex mirror.  See figure.

This seems backwards to me, but maybe I've just gotten turned around in my head about signs. Questions...

Should we flip the sign behavior here? If so that's a significant change in a sense.
Should we leave the behavior as-is, and just document it more clearly?
Do we have to worry about sign convention consistency between ThinLens and fresnel.QuadraticLens?

Need to think about these more when it's not the end of Friday.

Yeah I'm convinced this is the case based on double checking against simulated WFS images from ITM. Right now we have the signs the wrong way around in the weak lens images, and I believe that flipping the sign in poppy is the right way to fix it.


Added an android book and a Python playground

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Added an android book and a Python playground.
Why is this valuable (or not)?
These websites are genuinely free.
How do we know it's really free?
It shows ads and popular websites.
For book lists, is it a book? For course lists, is it a course? etc.

android Book: javatpoint
python playground: Repl.it

Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

I resolved it

Thanks, and welcome to free-programming-books!