Using 0's for FKs instead of NULL

Okay, crappy database. I know.
I have a table which has a compound primary key, with four columns. Each of those columns is actually itself a FK to another table.
But in their infinite wisdom, the authors decided to not make them actual FKs, and to use 0 to represent no value. There is no 0-record in the other table. And thus no RI.
System.InvalidOperationException: The property 'FilingCategoryId' on entity type 'CourtCaseFilingType' has a temporary value. Either set a permanent value explicitly or ensure that the database is configured to generate values for this property.
This is because the Id field is left as 0. And I'm trying to save it. And it should be zero. Is there a way to work around this?
EF Core 2.0.

I think what I'm looking for, which probably does not exist, is something like EmptyRelationshipValue(object), which would let me specify something other than 'null' to represent a missing relationship. And which of course would be smart enough on insert and select.

Certainly challenging.
Hope this helps,
https://github.com/aspnet/EntityFrameworkCore/issues/9552

@wasabii There are several interrelated things here:

Store-generated keys (e.g. Identity columns) require the CLR default (zero) to be used as a sentinel value, which then results in EF generating temporary key values--this is why you see the exception above. This can be turned off with ValueGeneratedNever() on the key properties in OnModelCreating. (Or use the DatabaseGeneratedAttribute if you prefer.)
Without using store generated keys, EF should send and receive zero values just like any other. However, EF will still not treat zero as null, so you may need to be creative in dealing with clearing relationships, etc--likely you will need to explicitly set FK values to zero instead of just making changes to navigation properties.
EF most of the time won't care if there actually exists a parent row for the zero key value. However, make sure to configure your relationships as optional (even though EF assumes they are required because the FK is non-nullable) so that EF understands that there may be some rows in the child table that do not have any rows in the parent table. (Mostly this means inner joins won't be used for returning children.)

Taking all that into account I expect you will be able to make something work. I would be interested to hear how it goes.


Markdown viewer headings ignored

Issue Type: Bug
If there is no space after a heading (#, ##, ###, etc.) definition, the Markdown Viewer doesn't recognize the heading an shows the # characters.  In-panel view of .md files in Azure DevOps handles this properly.
VS Code version: Code 1.39.2 (6ab598523be7a800d7f3eb4d92d7ab9a66069390, 2019-10-15T15:35:18.241Z)
OS version: Windows_NT x64 10.0.17763

I think that VS Code uses markdown-it which follows CommonMark spec which in turn requires space is required between # and content.
It's not that "Azure DevOps handles this properly" but implementation may vary because there is no single standard. CommonMark is an attempt to standardize this syntax.
Headings with spaces are safe bet (should be parsed correctly independently of implementation) so its easy to make 1 time conversion and stick to it.
If support is really needed - most likely it should be an extension that uses  markdown-it-lazy-headers package. See https://github.com/markdown-it/markdown-it/issues/240 for more details (however GitHub now requires space).


Building feature at 2 zoom levels shown at once

This is McKinleyville High School in McKinleyville California. I only expected geometry from a single zoom level to be visible at once.

This may happen when a parent tile is still drawn while some of its children (the tiles matching the current zoom-level) are already loaded. The coarser pattern in the upper-right corner seems to still use the parent tile here. If the view remains as on the screenshot it could be that the tile is missing or failed to load

@hjanetzek correctly described this behavior.
We have something called as proxy tiles in es, which are basically already loaded (parent or children) tiles from last update.
These proxy tiles are drawn till the new actual visible tiles (respective children or parent tile for these proxies) are loaded and ready to be drawn.
Possible in the above instance, one of the children tile fail to load and you were seeing the parent tile along with other 3 children tiles, hence mixed geometries.

But my understanding of desired proxy tile behavior is that the proxy
should be drawn underneath, so that the desired zoom level is always shown.
They should not be mixed.
On Tue, Dec 1, 2015 at 10:43 AM, Varun<EMAIL_ADDRESS>wrote:

@hjanetzek https://github.com/hjanetzek correctly described this
behavior.
We have something called as proxy tiles in es, which are basically
already loaded (parent or children) tiles from last update.
These proxy tiles are drawn till the new actual visible tiles (respective
children or parent tile for these proxies) are loaded and ready to be drawn.
Possible in the above instance, one of the children tile fail to load and
you were seeing the parent tile along with other 3 children tiles, hence
mixed geometries.
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram-es/issues/411#issuecomment-161005648
.

That's also my expected behavior (proxy tiles drawn under).

Yup proxy tiles are drawn under. Parent proxy tile is only visible for the not loaded child tile. Rest all loaded children tile should have overlapping proxy parent.

Ah, yes, the proxy building should be drawn below the earth polygon of the current tile - but to enforce this we would need to know offset range to sweep the proxy under the carpet :) I guess for now we need to find a working building 'order' and should not tweak position.z in the custom building shader

Proxy tiles are drawn deeper in depth buffer based on their z id.
https://github.com/tangrams/tangram-es/blob/master/core/resources/shaders/polygon.vs#L98-L100
I am checking as to why in this specific case, it appears that proxy tiles are drawn above visible tile set, and hence overlapping geometry.


Avoid use of package (future reserved word)

This isn't a huge deal, but I noticed that you used package as an identifier in several places--most significantly in example code. This causes problems for people using strict mode (common when using babel). It would be nice if you could use some other name that is not a future reserved word in JavaScript.

You're right. I'll probably fix it later today.

I looked now and I just use it on tests and the example 😄


MakieCore.jl

I see you depend on MakieCore.jl. That has be depreciated (https://github.com/JuliaPlots/MakieCore.jl). Can this be updated to depend on Makie instead? I realize that that seems like a heavy dep given that you also support Plots.jl.
I have sysimages build for Makie and I am unable to add TuePlots due to a version conflict with MakieCore in the sysimage.

I have a sysimage with Makie and CairoMakie built in. This sysimage is rebuilt nightly. Looking at the Manifest, it has MakieCore 0.5.2.
Here, I am trying to add TuePlots in a session using that sysimage
(@v1.8) pkg> st
Status `~/.julia/environments/v1.8/Project.toml`
  [13f3f980] CairoMakie v0.9.2
  [7073ff75] IJulia v1.23.3
  [ee78f7c6] Makie v0.18.2
  [9b87118b] PackageCompiler v2.1.0
  [91a5bcdd] Plots v1.36.1
  [d6f4376e] Markdown

(@v1.8) pkg> add TuePlots
    Updating registry at `~/.julia/registries/General.toml`
   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package MakieCore [20f20a25]:
 MakieCore [20f20a25] log:
 ├─possible versions are: 0.5.2 or uninstalled (package in sysimage!)
 └─found to have no compatible versions left with TuePlots [7dc86ad5]
   └─TuePlots [7dc86ad5] log:
     ├─possible versions are: 0.1.0 or uninstalled
     └─restricted to versions * by an explicit requirement, leaving only versions 0.1.0


wrong color

Microsoft PowerToys version
0.70.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
ColorPicker
Steps to reproduce

✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

When I plug the hex code #0088cc into Google's color picker, it gives me this color:

This color is clearly different from the one in your screenshot, so whatever you have taken a screenshot of appears to be incorrect about what hex code it is. Providing more information than an uncommentated screenshot may help in identifying or solving your issue.

/needinfo

Good afternoon! After some update, I noticed that the color picker does not
correctly determine the color. I tried to pick up a color on another
computer from one color template downloaded from the Internet in the form
of a regular picture of the same color. On another computer, the color
showed a different one. It turns out that on different computers the color
picker determines the same color in different ways.
ср, 24 мая 2023 г., 13:02 Jaime Bernardo @.***>:

/needinfo
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/26170#issuecomment-1560822217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOOOC2XBSHSHYJCUCKXIDGDXHXMC7ANCNFSM6AAAAAAYMRDD7U
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey! I noticed this too, but it seems that in my case it's Chrome browser is presenting not-true colors.
Image on the bottom is in Chrome, image on the top is downloaded and opened in Paint3D.
You can see there is a color saturation on the bottom Chrome version, but when that SAME image is downloaded and opened in an image editing program it shows the proper hex:

This is REALLY annoying!
I wonder if there is a workaround for Chrome is misrepresenting colors?

And I have these colors defined like this: ))))     От: qualteiaОтправлено: 10 июня 2023 г. в 1:37Кому: microsoft/PowerToysКопия: maxstigneev; AuthorТема: Re: [microsoft/PowerToys] wrong color (Issue #26170) Hey! I noticed this too, but it seems that in my case it's Chrome browser is presenting not-true colors.Image on the bottom is in Chrome, image on the top is downloaded and opened in Paint3D.You can see there is a color saturation on the bottom Chrome version, but when that SAME image is downloaded and opened in an image editing program it shows the proper hex:This is REALLY annoying!I wonder if there is a workaround for Chrome is misrepresenting colors?—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Hey! I noticed this too, but it seems that in my case it's Chrome browser is presenting not-true colors. Image on the bottom is in Chrome, image on the top is downloaded and opened in Paint3D. You can see there is a color saturation on the bottom Chrome version, but when that SAME image is downloaded and opened in an image editing program it shows the proper hex:
This is REALLY annoying! I wonder if there is a workaround for Chrome is misrepresenting colors?
Edit: A quick google of the issue led me to changing the Chrome setting for "Force Color Profile" which fixes it; 3b5258 instead of 415058.

Go to chrome://flags/
Search for Force color profile
Select scRGB linear (HDR where available)

Great!!! Thanks! Its realy work!


Make View and Bundle immutable?

Currently, the two collection objects in datreant.core, View (for Trees and Leafs) and Bundle (for Treants) are mutable. Members can be added and removed, and the order of these members in these set-like objects can be changed. Is it really necessary that they be mutable? If there isn't much of a case in which mutability is taken advantage of, it would make these objects simpler and hashable if we could make them immutable.
Thoughts?

I think it makes more sense for them to be immutable, and for operations (add/subtract etc) to return a new version of the object (similar to dataframes).

Although not technically immutable (since they are backed by lists internally), View and Bundle are now immutable from their public interfaces. Closing.


Added meetup page of OSM BE

The meetup was still missing

Thank you!  I switched the name to English, but it can be localized on Transifex so all users will see it in their native language.

Ok, great! Thanks!
Met vriendelijke groeten,
Pieter Vander Vennet
2018-04-17 14:47 GMT+02:00 Bryan Housel<EMAIL_ADDRESS>
Thank you! I switched the name to English, but it can be localized on
Transifex so all users will see it in their native language.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/osmlab/osm-community-index/pull/76#issuecomment-381978873,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZgbleB-YbCLStT8Ik5dEHcYvKcII9Qks5tpeRRgaJpZM4TYKoc
.


schema,engine-schema: explicit VMware <IP_ADDRESS> support, logs

Description
Fixes https://github.com/apache/cloudstack/issues/8412
Add support for <IP_ADDRESS> explicitly to prevent falling over to the parent version
Adds log when hypervisor capabilities falling over to the parent version

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@shwstppr a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 8200

@shwstppr
We could add 8.0a(<IP_ADDRESS>),  8.0b (<IP_ADDRESS>) and 8.0c (<IP_ADDRESS>) if missing
This is a quick workaround.
As Hari said,  ideally we should use hypervisor capabilities of parent version,  so we do not need to add hypervisor capabilities for each minor release.  This could be part of 8.1.x and 8.2.x support task.

@blueorangutan package

@shwstppr a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 8226

@blueorangutan help

@shwstppr [SL] I understand these words: "help", "hello", "thanks", "package", "test"
Test command usage: test [mgmt os] [hypervisor] [keepEnv] [qemuEv]
Mgmt OS options: ['centos7', 'centos6', 'suse15', 'alma8', 'ubuntu18', 'ubuntu22', 'ubuntu20', 'rocky8', 'alma9']
Hypervisor options: ['kvm-centos6', 'kvm-centos7', 'kvm-rocky8', 'kvm-alma8', 'kvm-alma9', 'kvm-ubuntu18', 'kvm-ubuntu20', 'kvm-ubuntu22', 'kvm-suse15', 'vmware-55u3', 'vmware-60u2', 'vmware-65u2', 'vmware-67u3', 'vmware-70u1', 'vmware-70u2', 'vmware-70u3', 'vmware-80', 'vmware-80u1', 'xenserver-65sp1', 'xenserver-71', 'xenserver-74', 'xcpng74', 'xcpng76', 'xcpng80', 'xcpng81', 'xcpng82']
Note: when keepEnv is passed, you need to specify mgmt server os and hypervisor or use the matrix command.
when qemuEv is passed, it will deploy KVM hyperviosr hosts with qemu-kvm-ev, else it will default to stock qemu.
Package command usage: package [all(default value),kvm,xen,vmware,hyperv,ovm] - a comma separated list can be passed with package command to bundle the required hypervisor's systemVM templates. Not passing any argument will bundle all - kvm,xen and vmware templates.
Blessed contributors for kicking Trillian test jobs: ['rohityadavcloud', 'shwstppr', 'vishesh92', 'harikrishna-patnala', 'nvazquez', 'DaanHoogland', 'weizhouapache', 'kishankavala', 'borisstoyanov', 'vladimirpetrov', 'kiranchavala', 'andrijapanicsb', 'NuxRo', 'rajujith', 'alexandremattioli', 'mlsorensen', 'oscaralvaro', 'Pearl1594', 'sureshanaparti']

@blueorangutan test ubuntu22 vmware-80

@shwstppr a [SL] Trillian-Jenkins test job (ubuntu22 mgmt + vmware-80) has been kicked to run smoke tests

Test results from backend,
05:10:02 msg: {"body": "[SF] Trillian test result (tid-8752)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server u22\
Total time taken: 48542 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8444-t8752-vmware-80.zip
Smoke tests completed. 121 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test | Result | Time (s) | Test File
--- | --- | --- | ---
"}

Test results from backend,
05:10:02 msg: {"body": "[SF] Trillian test result (tid-8752)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server u22\
Total time taken: 48542 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8444-t8752-vmware-80.zip
Smoke tests completed. 121 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test | Result | Time (s) | Test File
--- | --- | --- | ---
"}

great @shwstppr
let's merge it if there is no other changes ?


Class 'SimpleJavaScriptCompilation\Model\Context' not found in ../libraries/CloudflareBypass/Model/UAM/UAMPageFormParams.php:57

Getting the below error when i follow the example, what am i missing?
[INFO] UAM (retry: 0) -> waiting for 5 seconds! [INFO] UAM (retry: 0) -> getting form params... Fatal error: Uncaught Error: Class 'SimpleJavaScriptCompilation\Model\Context' not found in /home/nitemare/public_html/libraries/CloudflareBypass/Model/UAM/UAMPageFormParams.php:57 Stack trace: #0 /home/nitemare/public_html/libraries/CloudflareBypass/Model/UAM/UAMPageFormParams.php(41): CloudflareBypass\Model\UAM\UAMPageFormParams::getJschlAnswerFromPage(Object(CloudflareBypass\Model\UAM\UAMPageAttributes)) #1 /home/nitemare/public_html/libraries/CloudflareBypass/Model/UAM/UAMPageAttributes.php(55): CloudflareBypass\Model\UAM\UAMPageFormParams::getParamsFromPage(Object(CloudflareBypass\Model\UAM\UAMPageAttributes)) #2 /home/nitemare/public_html/libraries/CloudflareBypass/CFCurlImpl.php(146): CloudflareBypass\Model\UAM\UAMPageAttributes->__construct('http', 'torrentz2.is', '<!DOCTYPE HTML>...') #3 /home/nitemare/public_html/libraries/CloudflareBypass/CFCurlImpl.php(96): CloudflareBypass\CFCurlImpl->bypassIUAMPage(Resource id #33, '<!DOCTYPE HTML>...', Ar in /home/nitemare/public_html/libraries/CloudflareBypass/Model/UAM/UAMPageFormParams.php on line 57

other packages or required, this isn't explained as clearly as it should be, installing with the composer installs the packages


Drop ant support

And maybe propose to contribute the classes under the ant project.
<EMAIL_ADDRESS>

Hi @recursive-rat4 We are going to move ant under a dedicated repository. Could you explain your vote?

I'm looking to switch from Gradle to Maven or Ant and stumbled upon this issue. Perhaps your intention was misread, the move to separate repository sounds OK.

I'm looking to switch from Gradle to Maven or Ant and stumbled upon this issue. Perhaps your intention was misread, the move to separate repository sounds OK.

@recursive-rat4 - Just curious. What would be the reason behind you wanting to opt out of Gradle. I have seen people move out from maven into gradle. Just wanted to know.

@krmahadevan Some years ago, I saw a need to redesign a C++ project and chose Kotlin (coroutines, serialization, portability, all the cool stuff...). Indeed, seeing people move to Gradle was the factor to select this build tool. It never satisified me personally. What's really importrant, free software distributions face issues like https://github.com/gradle/gradle/issues/26516


Use cargo check-external-types to avoid breaking changes in public APIs

cargo-check-external-types is a static analysis tool for Rust library authors to set and verify which types from other libraries are allowed to be are exposed in their public API. This is useful for ensuring that a breaking change to a dependency doesn't force a breaking change in the library that's using it.
https://crates.io/crates/cargo-check-external-types
Define a allowed-external-types.toml file in each public crate. Update the script in the build process to check the presence of this file in each crate.

See #23


[dashboard] measure application download size (APK, IPA)

APK is fixed by https://github.com/flutter/dashboard_box/commit/c08ae7f9c1a3f7e2e017287f1e78e1c46e2d2307
IPA has its own issue (https://github.com/flutter/flutter/issues/4609)


Use of <citedRange> in <bibl>

For <citedRange>, we have two conflicting rules:

@unit need not be given for page ranges, as in <citedRange>36-42</citedRange>
complex locations should be encoded without a @unit, as in <citedRange>p. 291 nn. 9, 10; p. 320</citedRange>

We can probably assume that something like 82, 87-90, 93, 100-105 always represents a page range, but then there are a lot of <citedRange> without a unit that look like this:
199 n. 4
CVIII
no. Ghan 1
258-9 (III)
1938-39: 71, no. B.452
2, caption of fig. 3
3, 15, 24-25 (H)
3, 4 n. 1 and 7
109 (no. 13B), 122 (no. 18A), 124 (no. 21), 128 (no. 26)
2004-05: 206 (7)
58-61, ills. 126-36
[...]

Assuming that anything that starts with a digit is a page range would still be OK in most cases, but it would be preferable to follow a convention that removes the ambiguity.
Likewise, it is impossible to tell unambiguously whether the contents of <citedRange> represents a single item (page, volume, etc.)---in which case the singular form of the unit should be used (p., vol., etc.)---or several items---in which case the plural form is needed (pp., vols., etc.). It is reasonably clear for cases like 82, 87-90, 93, 100-105 (only digits, hyphens and commas), but beyond that I cannot do much. We have references like:
B/1965-1966
E.42-E.45
IR 91
1077, note 7
110 (no. 13D) = 127 (no. 24)

As a reminder, here is how each value of @unit is displayed:
@type     singular  plural
---       ---       ---
volume    vol.      vols.
appendix  appendix  appendixes
book      book      books
section   §         §§
page      p.        pp.
item      №         №
figure    fig.      figs.
plate     plate     plates
table     table     tables
note      n.        nn.
part      part      parts
entry     s.v.      s.vv.
line      l.        ll.

I will leave it to @danbalogh to formulate a more comprehensive response, but I just want to point out that I expect that all or most of the items on your list of aberrant references
B/1965-1966
E.42-E.45
IR 91
1077, note 7
110 (no. 13D) = 127 (no. 24)
are found in files imported from the EIAD database or in files whose encoding has never been finished (e.g. the files that Marine Schoettel started to encode before dropping out as PhD student). I certainly don't think we need to take those examples into account as evidence for inadequacy of our encoding guidelines. If you can furnish me a list of files with aberrant <citedRange> encoding, then I can try to improve the encoding in all offending cases.

@arlogriffiths I have not noticed a particularly problematic corpus on this issue. The examples I gave come from various sources.

I had raised the matter of the ambiguity of <citedRange> without @unit in an EGD comment in June 2021, when the idea of using this for encoding complex citations was first introduced. The comment thread lost momentum, and the issue is still unresolved. My proposal was and is to require @unit="other" for complex references, and to allow <citedRange> without @unit only if the contents are nothing but a page number or numbers. The matter is summarised in the EGD Leftovers in the section "Unstructured references with citedRange".
On displaying singular and plural forms, I was not aware this has been (or is being) implemented for any unit other than page. (See also the earlier discussion about "s.vv" summarised in another part of the Leftovers, which may thus be superseded.) I think the importance of determining whether a plural or singular form need be used is very small relative to its costs.
To explain. First, it needs great investment. A blind algorithm would not be able to do the work in most cases; for instance, among your examples, "B/1965-1966" is a (single) appendix number of a format used in almost all volumes of the Annual Report on Indian Epigraphy, so something like it will be cited in most editions of Indian inscriptions. The algorithm would thus have to use a complex list of exception formats and could not rely just on the presence of a hyphen to determine that we are dealing with a range. Second, plural units of most kinds are likely to be exceedingly rare: one hardly ever needs to refer to multiple appendices,* volumes, books, etc. of a publication. And third, the matter is purely aesthetic. As far as I know, there are no plans ever to make these machine-actionable, so the only purpose of determining whether a singular or a plural is appropriate is to have a pedantically accurate display.
*NOTE: If we do really need the plural of appendix, I urge that it should be appendices, not appendixes.
Now, given these three considerations, I think plural unit displays should be discarded for most cases and retained only for @unit="page", or possibly page and a limited number of other units where we can reasonably expect to refer to a list or range more than once in a blue moon, AND expect a blind algorithm based on the presence of a hyphen or comma to work (e.g. note).
Next, for those rare cases where a reference to a list or range of other units is needed, we have two alternatives. A, we can live with the fact that the display will be singular (which was and remains my recommendation for "s.v.", see the second Leftovers link above); or B, we can decide that all cases where a plural is essential, the "complex reference" style should be used (without @unit or with @unit="other" depending on the first decision), where the encoder will manually enter whatever s/he sees fit.
In your list of problematic <citedRange> contens without @unit, some of the items are obscure to me, but

"199 n. 4" seems to be just an error (or a sloppy use of the "complex reference" where page and note should have been used rigorously)
"CVIII" is probably a correct page number
1938-39: 71, no. B.452 and 2004-05: 206 (7) are probably incorrectly recorded references to ARIE appendix items
and most of the rest are correct uses of the "complex reference" type.

Finally, I notice that your list of possible units includes "line". This value is not permitted in the EGD. Do any other guidelines propose using it somewhere? Is it used anywhere in an XML file? In the same EGD discussion thread that raised using @unit="other" for complex references, I had also noted that we might want to add "line" and "stanza" as possible units, but ultimately we decided against this. Axelle may have misunderstood and added "line" nonetheless, or - since "stanza" is not on the list - perhaps "line" has been added for some other reason. At any rate, if we keep this as a permitted value of unit, then I need someone to tell me when its use is recommended, so I can add it to the EGD. But if it isn't already in use, then I'd prefer discarding it.

For @unit="line", there are about 15 files (only inscriptions) that use it. It appears in one example in the EGC, but nobody currently uses it in critical editions. Discarding it would thus have minimal impact.
Generally speaking, I concur with anything that would simplify the encoding of bibliographic references. It is impossible to mechanically extract useful information from <citedRange>, because references styles and numbering systems vary too much, so the only practical purpose of <citedRange> is to format citations automatically.
Even so, the complexity/benefit ratio is very high. From what I have seen so far, I do not think we gain much consistency from using many types of @units and references formats. Free text references like vol. VI, pp. 48-49 are easy to read and modify, so letting people use these might work just as well.

So it seems we are pretty much of the same mind here.
We'll need @arlogriffiths to give his opinions on my suggestions above (namely: mandatorily adopting <citedRange unit="other"> for non-rigorous reference details, and handling only simple references rigorously with dedicated units, while changing multi-unit references to the non-rigorous form), as well as on whether he thinks @unit="line" is necessary or can be discarded (and deleted from the EGC example).
If at the moment it occurs only in inscription editions, and we discard it, then those editions will have to be changed manually, moving the line reference outside the bibl element.

No objections from my side, except that I find the value "other" unsatisfactory for the intended use. What about alternatives like "free-form", "misc", "mixed"?
I'd favor maintaining the unit "line", as it is rather customary to refer to (pagination and) lineation of certain works.

Thanks, Arlo. Of the suggested values for @unit, I prefer "mixed", so let's go with that unless there is an objection. (I note, however, that it doesn't really describe the cases where there is no automatic plural, and so e.g. "Appendices" is encoded in this way. This is not a problem for me, but it may be for some.)
I don't mind retaining "line". I'll now try to summarise what needs to be done.

I'm adding "line" to the next EGD as a possible value of @unit and writing instructions for the use of "mixed" instead of no @unit (to be pushed soon)
We'll need to change existing instances of mixed references from no @unit to @unit="mixed" in the corpus. This will need some thoughtful automation and some manual checking. I believe that all instances of <citedRange> without @unit whose contents include anything other than Arabic numerals, spaces, hyphens and commas should be changed to have @unit="mixed". If there are hundreds of these in the corpus, then checking them manually may not be feasible, but I believe there shouldn't be more than a couple of dozen. If the latter is the case, I think it would be best to flag each of the auto-changed instances for the encoder's attention. One way to do that would be to add an XML comment (e.g. <!--CHECK Reference-->) in the course of refactoring, but if @michaelnmmeyer has a better idea, then let's go by that. If Michaël can then create a list of files (preferably by corpus) that include such references, then these could be sent to the respective encoders, alerting them to go and check those citations. I can help them use Oxygen to search their corpus for the flags and talk to them about the desired way of encoding citations.
We'll need to come to a decision about which permitted values of @unit are to have a smart display with plurals. Those for which this is not feasible to implement are then to be displayed mechanically in the singular, and if an encoder desperately needs a plural reference, it'll have to be encoded as a mixed type. Once this is finalised, I'll add this info to the EGD in the part where citation display and bracketing are discussed. I suggest the following specifics:

@unit
singular
action

volume
vol.
no plural

appendix
appendix
no plural

book
book
no plural

section
§
no plural OR plural §§ if contents include ", " (comma followed by space) or - (hyphen)

page
p.
plural pp. if contents include ", " (comma followed by space) or - (hyphen)

item
№
no plural

figure
fig.
no plural

plate
plate
no plural

table
table
no plural

note
n.
no plural

part
part
no plural

entry
s.v.
no plural OR plural s.vv. if contents include ", " (comma followed by space)

line
l.
no plural OR plural ll. if contents include ", " (comma followed by space) or - (hyphen)

I'll need your opinions (should we poll anyone else?) on how hard we should try to implement plurals for anything other than page. I believe that some section numbers may include hyphens, and the same may be true of complex line numbers of some form (e.g. "line A-1"), so if the presence of a hyphen in the contents prompts the plural form, then these should be avoided. We can either formulate the guidelines to tell encoders to omit any hyphens present in the actual numeration they are referring to and use a hyphen only when referring to a range (then hope they read and remember this), or live with the occasional false positive (a plural display where a singular had been intended), or not use auto-generated plurals and instead direct encoders to use a mixed unit when they really want a plural reference. I think the simplest both to implement and to keep in mind would be to auto-generate plurals only for @unit="page" and generate only the singular for everything else.


README

Hi Javier,
Thanks for the great tutorial. Actually I didn't get any idea of how to write an extension after reading https://spark.rstudio.com/extensions/.
I'd like some changes for the README files.
Firstly, the following point is not accurate.
2.  You should download and install the Scala 2.10 and 2.11 compilers to one of the following paths:
-   /opt/scala
-   /opt/local/scala
-   /usr/local/scala
-   ~/scala (Windows-only)
Scala doesn't need to be in the above locations. I built a jar on Ubuntu with scala 2.10 and 2.11 in my home directory like the following:
scalas/
├── scala-2.10.7
└── scala-2.11.8
And by using compilation specification to specify the location of scala, the compile_package_jars function was able to successfully build the jar.
spec <- spark_default_compilation_spec(locations = "/home/ciasom/scalas")
compile_package_jars(spec=spec)

Secondly, it would be better if examples of how the jars are used in other situations other than local spark, for instance, through Livy, can be  added.

I'm confused by the gap between having the r functions and loading sparkhello as a library. I read through the readme and couldn't understand how to tell R that this sparkhello is a library, if I need to install it and how (we only generated jar files I think?), and how to use the two R functions. They are possibly related or unrelated 😅 it would be great if someone could answer my questions and/or put them into the readme.


i3status-rust: <IP_ADDRESS>8-06-22 -> <IP_ADDRESS>8-09-30

Motivation for this change
Things done

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

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

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

Failure on aarch64-linux (full log)
Attempted: i3status-rust
Partial log (click to expand)
   |                     ^^^^^^^ Could not find `statvfs` in `sys`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0432`.
error: Could not compile `i3status-rs`.

To learn more, run the command again with --verbose.
builder for '/nix/store/q5sflnffyg5jpyfkgxabsdffam78qc9g-i3status-rust-<IP_ADDRESS>8-09-30.drv' failed with exit code 101
error: build of '/nix/store/q5sflnffyg5jpyfkgxabsdffam78qc9g-i3status-rust-<IP_ADDRESS>8-09-30.drv' failed

Success on x86_64-linux (full log)
Attempted: i3status-rust
Partial log (click to expand)

installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/yyb0aj1w73y3kzp6l4f4qp7a6d2sw9yh-i3status-rust-<IP_ADDRESS>8-09-30
shrinking /nix/store/yyb0aj1w73y3kzp6l4f4qp7a6d2sw9yh-i3status-rust-<IP_ADDRESS>8-09-30/bin/i3status-rs
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/yyb0aj1w73y3kzp6l4f4qp7a6d2sw9yh-i3status-rust-<IP_ADDRESS>8-09-30/bin
patching script interpreter paths in /nix/store/yyb0aj1w73y3kzp6l4f4qp7a6d2sw9yh-i3status-rust-<IP_ADDRESS>8-09-30
checking for references to /build in /nix/store/yyb0aj1w73y3kzp6l4f4qp7a6d2sw9yh-i3status-rust-<IP_ADDRESS>8-09-30...
/nix/store/yyb0aj1w73y3kzp6l4f4qp7a6d2sw9yh-i3status-rust-<IP_ADDRESS>8-09-30

Please mark this broken on aarch64, it never built on that platform according to hydra logs.

Let's try to upgrade nix crate first


Support for tvOS

The compiled binary of automegre-rs is missing support for tvOS (aarch64-apple-tvos) & tvOS simulator(x86_64-apple-tvos)
The platform is contained in Tier 3 of the rust compiler toolchain.

Looks like both are now supported as tier 3 platforms?
https://doc.rust-lang.org/nightly/rustc/platform-support.html


feat: use custom url for binary download

This solves https://github.com/przemyslawpluta/node-youtube-dl/pull/338 which unfortunately got closed without being addressed.
This feature is necessary for people that are self-hosting the youtube-dl binary, e.g. on S3
cc @Kikobeats

Hey, @n1ru4l I understand your use case but I prefer to don't add a new environment variable it could be a thing we can avoid.
What we can do to avoid the new environment variable is to check the response content type here:
https://github.com/przemyslawpluta/node-youtube-dl/blob/master/lib/downloader.js#L29

If the content type is text, then the body contains the URL for fetching the resource.
Otherwise the response is binary.

If you can make that change I will happy to merge :)

That would not work as there are additional get parameter added here: https://github.com/przemyslawpluta/node-youtube-dl/blob/6131a9a050492ba26fe6e6c5e5213b5cb93295fc/lib/downloader.js#L89

@n1ru4l not seeing that is an issue since query parameters are ignored by S3 files.
e.g., https://s.microlink.io/kdsy_4QyLo6Kh61e8SOtMcxCtWg_.png?foo=bar&platform=darwin

But it will not append .exe properly?


[FEAT]: Improve preview in REPL

Description
First of all, thanks for your work! It's super useful :)
One of the issues I encounter is with the REPL. It's always hard to find an icon and the matching name.
Imo, it would be super useful if the icon name is displayed alongside the icon.
I suggest you this solution:

	{#each Object.entries(Icons) as [name, component]}

			<svelte:component this={component} color={random_hex_color_code()} />
			{name}

	{/each}

Additional Information
No response

Thanks. Where is the best place for the link to REPL?

I add it to this page.

Cool! Thank you so much for using it! :D


Wakatime doesnt load/install in SSMS 17.9

I've followed the directions for installing wakatime in SSMS but I'm not getting the item in the Tools tab nor am I see any errors:

Not sure what to do at this point.

The extension has been bumped to version 3.0.0. Please follow the updated instructions on README to reinstall it.


handled null values in futimes method

@MikeFultonDev @IgorTodorovskiIBM Can you please review my changes, am unable to add reviewers in this PR?
Also, please note that i have raised PR to zopen branch .
Thank you


Breaking bug in 0.7.1

Hey,
your latest update broke our CI/CD, see attached log.
pip install pdm tox-pdm ansible-lint
Collecting pdm
  Downloading pdm-2.11.2-py3-none-any.whl (248 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 248.0/248.0 kB 5.5 MB/s eta 0:00:00
Collecting tox-pdm
  Downloading tox_pdm-0.7.1-py3-none-any.whl (5.3 kB)
Collecting ansible-lint
  Downloading ansible_lint-6.22.1-py3-none-any.whl (296 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 296.8/296.8 kB 11.3 MB/s eta 0:00:00
Collecting cachecontrol[filecache]>=0.13.0
  Downloading cachecontrol-0.13.1-py3-none-any.whl (22 kB)
Requirement already satisfied: packaging!=22.0,>=20.9 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from pdm) (23.2)
Requirement already satisfied: certifi in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from pdm) (2023.7.22)
Collecting python-dotenv>=0.15
  Downloading python_dotenv-1.0.0-py3-none-any.whl (19 kB)
Collecting shellingham>=1.3.2
  Downloading shellingham-1.5.4-py2.py3-none-any.whl (9.8 kB)
Requirement already satisfied: platformdirs in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from pdm) (3.11.0)
Collecting pyproject-hooks
  Downloading pyproject_hooks-1.0.0-py3-none-any.whl (9.3 kB)
Collecting resolvelib>=1.0.1
  Downloading resolvelib-1.0.1-py2.py3-none-any.whl (17 kB)
Collecting dep-logic<1.0,>=0.0.2
  Downloading dep_logic-0.0.4-py3-none-any.whl (26 kB)
Collecting importlib-metadata>=3.6
  Downloading importlib_metadata-7.0.1-py3-none-any.whl (23 kB)
Collecting unearth>=0.12.1
  Downloading unearth-0.12.1-py3-none-any.whl (42 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 42.9/42.9 kB 2.6 MB/s eta 0:00:00
Requirement already satisfied: tomli>=1.1.0 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from pdm) (2.0.1)
Requirement already satisfied: virtualenv>=20 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from pdm) (20.24.5)
Collecting installer<0.8,>=0.7
  Downloading installer-0.7.0-py3-none-any.whl (453 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 453.8/453.8 kB 26.3 MB/s eta 0:00:00
Collecting findpython<1.0.0a0,>=0.4.0
  Downloading findpython-0.4.1-py3-none-any.whl (20 kB)
Collecting requests-toolbelt
  Downloading requests_toolbelt-1.0.0-py2.py3-none-any.whl (54 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 54.5/54.5 kB 5.3 MB/s eta 0:00:00
Collecting blinker
  Downloading blinker-1.7.0-py3-none-any.whl (13 kB)
Collecting rich>=12.3.0
  Downloading rich-13.7.0-py3-none-any.whl (240 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 240.6/240.6 kB 7.1 MB/s eta 0:00:00
Collecting tomlkit<1,>=0.11.1
  Downloading tomlkit-0.12.3-py3-none-any.whl (37 kB)
Requirement already satisfied: tox>=4.0 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from tox-pdm) (4.11.3)
Collecting ansible-compat>=4.1.10
  Downloading ansible_compat-4.1.10-py3-none-any.whl (22 kB)
Collecting black>=22.8.0
  Downloading black-23.12.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (1.7 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.7/1.7 MB 54.7 MB/s eta 0:00:00
Collecting jsonschema>=4.10.0
  Downloading jsonschema-4.20.0-py3-none-any.whl (84 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 84.7/84.7 kB 13.5 MB/s eta 0:00:00
Requirement already satisfied: requests>=2.31.0 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from ansible-lint) (2.31.0)
Collecting yamllint>=1.30.0
  Downloading yamllint-1.33.0-py3-none-any.whl (65 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 65.4/65.4 kB 8.6 MB/s eta 0:00:00
Collecting pathspec>=0.10.3
  Downloading pathspec-0.12.1-py3-none-any.whl (31 kB)
Collecting ansible-core>=2.12.0
  Downloading ansible_core-2.15.8-py3-none-any.whl (2.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.2/2.2 MB 58.6 MB/s eta 0:00:00
Collecting subprocess-tee>=0.4.1
  Downloading subprocess_tee-0.4.1-py3-none-any.whl (5.1 kB)
Collecting wcmatch>=8.1.2
  Downloading wcmatch-8.5-py3-none-any.whl (39 kB)
Requirement already satisfied: filelock>=3.3.0 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from ansible-lint) (3.12.4)
Requirement already satisfied: pyyaml>=5.4.1 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from ansible-lint) (6.0.1)
Collecting ruamel.yaml>=0.18.5
  Downloading ruamel.yaml-0.18.5-py3-none-any.whl (116 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 116.4/116.4 kB 10.6 MB/s eta 0:00:00
Collecting typing-extensions>=4.5.0
  Downloading typing_extensions-4.9.0-py3-none-any.whl (32 kB)
Collecting cryptography
  Downloading cryptography-41.0.7-cp37-abi3-manylinux_2_28_x86_64.whl (4.4 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 4.4/4.4 MB 64.1 MB/s eta 0:00:00
Collecting jinja2>=3.0.0
  Downloading Jinja2-3.1.2-py3-none-any.whl (133 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 133.1/133.1 kB 16.0 MB/s eta 0:00:00
Collecting importlib-resources<5.1,>=5.0
  Downloading importlib_resources-5.0.7-py3-none-any.whl (24 kB)
Collecting click>=8.0.0
  Downloading click-8.1.7-py3-none-any.whl (97 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 97.9/97.9 kB 6.8 MB/s eta 0:00:00
Collecting mypy-extensions>=0.4.3
  Downloading mypy_extensions-1.0.0-py3-none-any.whl (4.7 kB)
Collecting msgpack>=0.5.2
  Downloading msgpack-1.0.7-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (530 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 531.0/531.0 kB 21.9 MB/s eta 0:00:00
Collecting zipp>=0.5
  Downloading zipp-3.17.0-py3-none-any.whl (7.4 kB)
Collecting jsonschema-specifications>=2023.03.6
  Downloading jsonschema_specifications-2023.12.1-py3-none-any.whl (18 kB)
Collecting referencing>=0.28.4
  Downloading referencing-0.32.0-py3-none-any.whl (26 kB)
Collecting attrs>=22.2.0
  Downloading attrs-23.2.0-py3-none-any.whl (60 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 60.8/60.8 kB 4.9 MB/s eta 0:00:00
Collecting rpds-py>=0.7.1
  Downloading rpds_py-0.16.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (1.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.2/1.2 MB 46.4 MB/s eta 0:00:00
Requirement already satisfied: charset-normalizer<4,>=2 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from requests>=2.31.0->ansible-lint) (3.3.0)
Requirement already satisfied: urllib3<3,>=1.21.1 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from requests>=2.31.0->ansible-lint) (2.0.6)
Requirement already satisfied: idna<4,>=2.5 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from requests>=2.31.0->ansible-lint) (3.4)
Collecting markdown-it-py>=2.2.0
  Downloading markdown_it_py-3.0.0-py3-none-any.whl (87 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 87.5/87.5 kB 4.2 MB/s eta 0:00:00
Collecting pygments<3.0.0,>=2.13.0
  Downloading pygments-2.17.2-py3-none-any.whl (1.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.2/1.2 MB 37.2 MB/s eta 0:00:00
Collecting ruamel.yaml.clib>=0.2.7
  Downloading ruamel.yaml.clib-0.2.8-cp39-cp39-manylinux_2_5_x86_64.manylinux1_x86_64.whl (562 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 562.1/562.1 kB 19.6 MB/s eta 0:00:00
Requirement already satisfied: colorama>=0.4.6 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from tox>=4.0->tox-pdm) (0.4.6)
Requirement already satisfied: pyproject-api>=1.6.1 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from tox>=4.0->tox-pdm) (1.6.1)
Requirement already satisfied: pluggy>=1.3 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from tox>=4.0->tox-pdm) (1.3.0)
Requirement already satisfied: cachetools>=5.3.1 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from tox>=4.0->tox-pdm) (5.3.1)
Requirement already satisfied: chardet>=5.2 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from tox>=4.0->tox-pdm) (5.2.0)
Requirement already satisfied: distlib<1,>=0.3.7 in /pyenv/versions/3.9.18/lib/python3.9/site-packages (from virtualenv>=20->pdm) (0.3.7)
Collecting bracex>=2.1.1
  Downloading bracex-2.4-py3-none-any.whl (11 kB)
Collecting MarkupSafe>=2.0
  Downloading MarkupSafe-2.1.3-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (25 kB)
Collecting mdurl~=0.1
  Downloading mdurl-0.1.2-py3-none-any.whl (10.0 kB)
Collecting cffi>=1.12
  Downloading cffi-1.16.0-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (443 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 443.4/443.4 kB 24.7 MB/s eta 0:00:00
Collecting pycparser
  Downloading pycparser-2.21-py2.py3-none-any.whl (118 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 118.7/118.7 kB 9.8 MB/s eta 0:00:00
Installing collected packages: resolvelib, zipp, typing-extensions, tomlkit, subprocess-tee, shellingham, ruamel.yaml.clib, rpds-py, python-dotenv, pyproject-hooks, pygments, pycparser, pathspec, mypy-extensions, msgpack, mdurl, MarkupSafe, installer, importlib-resources, findpython, dep-logic, click, bracex, blinker, attrs, yamllint, wcmatch, unearth, ruamel.yaml, requests-toolbelt, referencing, markdown-it-py, jinja2, importlib-metadata, cffi, cachecontrol, black, tox-pdm, rich, jsonschema-specifications, cryptography, pdm, jsonschema, ansible-core, ansible-compat, ansible-lint
Successfully installed MarkupSafe-2.1.3 ansible-compat-4.1.10 ansible-core-2.15.8 ansible-lint-6.22.1 attrs-23.2.0 black-23.12.1 blinker-1.7.0 bracex-2.4 cachecontrol-0.13.1 cffi-1.16.0 click-8.1.7 cryptography-41.0.7 dep-logic-0.0.4 findpython-0.4.1 importlib-metadata-7.0.1 importlib-resources-5.0.7 installer-0.7.0 jinja2-3.1.2 jsonschema-4.20.0 jsonschema-specifications-2023.12.1 markdown-it-py-3.0.0 mdurl-0.1.2 msgpack-1.0.7 mypy-extensions-1.0.0 pathspec-0.12.1 pdm-2.11.2 pycparser-2.21 pygments-2.17.2 pyproject-hooks-1.0.0 python-dotenv-1.0.0 referencing-0.32.0 requests-toolbelt-1.0.0 resolvelib-1.0.1 rich-13.7.0 rpds-py-0.16.2 ruamel.yaml-0.18.5 ruamel.yaml.clib-0.2.8 shellingham-1.5.4 subprocess-tee-0.4.1 tomlkit-0.12.3 tox-pdm-0.7.1 typing-extensions-4.9.0 unearth-0.12.1 wcmatch-8.5 yamllint-1.33.0 zipp-3.17.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv

[notice] A new release of pip is available: 23.0.1 -> 23.3.2
[notice] To update, run: pip install --upgrade pip
$ tox -e pylint
pylint: internal error
Traceback (most recent call last):
  File "/pyenv/versions/3.9.18/lib/python3.9/site-packages/tox/session/cmd/run/single.py", line 46, in _evaluate
    tox_env.setup()
  File "/pyenv/versions/3.9.18/lib/python3.9/site-packages/tox/tox_env/api.py", line 247, in setup
    self._setup_env()
  File "/pyenv/versions/3.9.18/lib/python3.9/site-packages/tox_pdm/plugin.py", line 45, in _setup_env
    if self.conf["skip_install"]:
  File "/pyenv/versions/3.9.18/lib/python3.9/site-packages/tox/config/sets.py", line 118, in __getitem__
    return self.load(item)
  File "/pyenv/versions/3.9.18/lib/python3.9/site-packages/tox/config/sets.py", line 128, in load
    config_definition = self._defined[item]
KeyError: 'skip_install'
  pylint: FAIL code 2 (0.63 seconds)
  evaluation failed :( (0.88 seconds)

Looking at the changelog, I'd guess it's because skip_install is somehow assumed to be present, although according to tox docs it is optional and has a default value of False. Could you please revert/fix this change?

Thanks for the quick fix!


Uncaught Error: EIO: i/o error, uv_chdir

[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.19
System: Mac OS X 10.11
Thrown From: jshint package, v1.8.1
Stack Trace
Uncaught Error: EIO: i/o error, uv_chdir
At /Users/Correia/.atom/packages/jshint/index.js:226

Error: EIO: i/o error, uv_chdir
    at Error (native)
    at lint (/Users/Correia/.atom/packages/jshint/index.js:226:10)
    at complete (/Users/Correia/Downloads/Atom.app/Contents/Resources/app.asar/node_modules/babel-core/node_modules/lodash/index.js:7697:25)
    at delayed (/Users/Correia/Downloads/Atom.app/Contents/Resources/app.asar/node_modules/babel-core/node_modules/lodash/index.js:7707:11)

Commands
  4x -9:40 core:backspace (atom-text-editor.editor.is-focused)
  2x -9:36.7.0 core:save (atom-text-editor.editor.is-focused)
  7x -7:52.7.0 core:backspace (atom-text-editor.editor.is-focused)
     -7:23.3.0 core:save (atom-text-editor.editor.is-focused)
  4x -6:21.1.0 core:backspace (atom-text-editor.editor.is-focused)
     -5:55.4.0 core:cut (atom-text-editor.editor.is-focused)
     -5:52.4.0 core:paste (atom-text-editor.editor.is-focused)
  4x -5:51.2.0 core:backspace (atom-text-editor.editor.is-focused)
     -5:37 core:save (atom-text-editor.editor.is-focused)
  9x -5:18.7.0 core:undo (atom-text-editor.editor.is-focused)
     -4:53.4.0 core:backspace (atom-text-editor.editor.is-focused)
     -2:00.5.0 editor:newline (atom-text-editor.editor.is-focused)
     -1:17.5.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "SFTP-deployment"
    ]
  }
}

Installed Packages
# User
No installed packages

# Dev
No dev packages

Closing as this project is deprecated. 🌵


add KMSKeyRef inside DBCluster and DBInstance

Issue #, if available: https://github.com/aws-controllers-k8s/community/issues/1258
Description of changes:

Add reference for KMSKey inside DBCluster and DBInstance crds
Currently there is no support to test cross-controller resource reference. Once https://github.com/aws-controllers-k8s/community/issues/1266 is resolved, I will add automated e2e tests
I manually tested the referencing KMSKey inside DBInstance works as expected.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@vijtrip2 @RedbackThomson @A-Hilaly I'm happy to merge this now or wait until the e2e testing issue is further along. Up to you!

@jaypipes We probably need to do a deep dive on testing again. I'd love to start using ACK to test ACK, even.
/lgtm

We probably need to do a deep dive on testing again.

+1 . With that we should also think about improving developer experience for AWS service teams to write the e2e tests. That will improve the ownership transfer for the ACK controllers.

I'd love to start using ACK to test ACK, even.

That's an interesting thought. I would love to hear more of your thoughts on this.


Component prop rejects my component in TypeScript

I am using the Collapse component in a TypeScript project.
I am trying to set a custom trigger component for the Collapse component, using the component prop. This works if the component is a dumb one, but not if it requires props. I want a collapse trigger that is dynamically generated such that it can show a summary of the collapsed contents.

[X] The issue is present in the latest release.
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

I get the following error
62,19): No overload matches this call.
  Overload 1 of 2, '(props: CollapseProps, context?: any): ReactElement<any, string | ((props: any) => ReactElement<any, string | ... | (new (props: any) => Component<any, any, any>)> | null) | (new (props: any) => Component<...>)> | Component<...> | null', gave the following error.
    Type 'Element' is not assignable to type '"object" | "a" | "abbr" | "address" | "area" | "article" | "aside" | "audio" | "b" | "base" | "bdi" | "bdo" | "big" | "blockquote" | "body" | "br" | "button" | "canvas" | "caption" | ... 99 more ... | undefined'.
      Type 'Element' is not assignable to type 'FunctionComponent<TransitionProps>'.
        Type 'Element' provides no match for the signature '(props: PropsWithChildren<TransitionProps>, context?: any): ReactElement<any, string | ((props: any) => ReactElement<any, string | ... | (new (props: any) => Component<any, any, any>)> | null) | (new (props: any) => Component<...>)> | null'.
  Overload 2 of 2, '(props: PropsWithChildren<CollapseProps>, context?: any): ReactElement<any, string | ((props: any) => ReactElement<any, string | ... | (new (props: any) => Component<any, any, any>)> | null) | (new (props: any) => Component<...>)> | Component<...> | null', gave the following error.
    Type 'Element' is not assignable to type '"object" | "a" | "abbr" | "address" | "area" | "article" | "aside" | "audio" | "b" | "base" | "bdi" | "bdo" | "big" | "blockquote" | "body" | "br" | "button" | "canvas" | "caption" | ... 99 more ... | undefined'.
      Type 'Element' is not assignable to type 'FunctionComponent<TransitionProps>'.

Expected Behavior 🤔
I would expect that my component works as if it was JS instead of TypeScript, or some documentation for the Collapse component stating how I should typify my component. I only found a general reference to create an issue here if I were to find such a rejection though, so here I am. Also seems to be similar to issue #17699 that was fixed.
Steps to Reproduce 🕹
Here is a codesandbox showing the issue.
https://codesandbox.io/s/heuristic-lalande-plt53
Also a sandbox to show that it is not an issue in JS.
https://codesandbox.io/s/material-demo-6j9ef
Steps:
1 The error is directly visible in the sandbox.
Context 🔦
I am trying to have a dynamic Collapse trigger.
Your Environment 🌎

Tech
Version

Material-UI
v4.9.1

TypeScript
3.8

Seems to be an issue with more than just The Collapse component, I'm having the same problem with CardHeader. I think the problem is the type definitions are restricted to the default component used instead of being overridable like the Chip or Avatar components.
Here's a sandbox with my code showing the similar issue.
As you can see, it works for Chip but doesn't work for CardHeader.
https://codesandbox.io/s/quirky-yalow-9m3iq
If I were to update the type definitions for CardHeader to resemble the ones using an overridable defaultComponent, than it would work. Here's the code for that (the same thing can be done with Collapse).
import * as React from 'react';
import { StandardProps } from '..';
import { TypographyProps } from '../Typography';
import { OverridableComponent, OverrideProps } from '../OverridableComponent';

export interface CardHeaderTypeMap<P = {}, D extends React.ElementType = 'div'> {
  props: P & {
    action?: React.ReactNode;
    avatar?: React.ReactNode;
    disableTypography?: boolean;
    subheader?: React.ReactNode;
    subheaderTypographyProps?: Partial<TypographyProps>;
    title?: React.ReactNode;
    titleTypographyProps?: Partial<TypographyProps>;
  }
  defaultComponent: D;
  classKey: CardHeaderClassKey;
  component?: React.ElementType<React.HTMLAttributes<HTMLDivElement>>;
}

declare const CardHeader: OverridableComponent<CardHeaderTypeMap>;

export type CardHeaderClassKey =
  | 'root'
  | 'avatar'
  | 'action'
  | 'content'
  | 'title'
  | 'subheader';

export type CardHeaderProps<
  D extends React.ElementType = CardHeaderTypeMap['defaultComponent'],
  P = {}
> = OverrideProps<CardHeaderTypeMap<P, D>, D>;

export default CardHeader;

I could push out a change, looks like it'll be of the same flavor as this PR.
https://github.com/mui-org/material-ui/pull/18868

@theGirrafish Thanks for the effort in #20110, we are getting closer to full coverage of the OverridableComponent types.

However, the last time I have checked, the approach isn't compatible with hosting the prop's description in TypeScript not with the XComponent & XProps props, I'm not sure it scales very well.

For what it is worth, I rewrote my uses to make my trigger component into a button, then route the clicks onto the Collapse.in property. Slightly less elegant IMO, but functionally equivalent to what I wanted. This doesn't invalidate the typescript issues when you actually want to use the component property though.

Thanks for the report.

codesandbox.io/s/heuristic-lalande-plt53

This is expected behavior since Collapse passes in some props that you don't handle.
TypeScript improved their error message so it may be a bit more clear now:
Type '({ foo }: { foo: any; }) => Element' is not assignable to type '"object" | "span" | "style" | "h1" | "h2" | "h3" | "h4" | "h5" | "h6" | "caption" | "button" | "a" | "abbr" | "address" | "area" | "article" | "aside" | "audio" | "b" | "base" | "bdi" | ... 98 more ... | undefined'.
  Type '({ foo }: { foo: any; }) => Element' is not assignable to type 'FunctionComponent<TransitionProps>'.
    Types of parameters '__0' and 'props' are incompatible.
      Property 'foo' is missing in type 'TransitionProps & { children?: ReactNode; }' but required in type '{ foo: any; }'.ts(2322)
Collapse.d.ts(19, 3): The expected type comes from property 'component' which is declared here on type 'IntrinsicAttributes & CollapseProps'


OpenMP thread placement and affinity

In my testing, on a 4-core two-way hyperthreaded Xeon-W Skylake machine, I've found that the following environment variable settings produce consistently high performance:
OMP_NUM_THREADS 4
OMP_PLACES "{0,1,2,3}"
OMP_PROC_BIND spread
This tells the OpenMP library to allow up to 4 threads, tells the OpenMP library that it can start threads on cores 0-3 (and can't use the hyperthreaded siblings 4-7) and that threads should be spread out over the cores as they're started.  I believe that it also implies thread affinity so that threads won't move between cores.
I find that if I don't set these environment variables, the performance is generally worse and can also be much more variable.  e.g. on a simple test of matrix multiplication, with OMP_NUM_THREADS=4 the run times varied from 6.23 to 8.94 seconds in four tests.  After setting OMP_PROC_BIND and OMP_PLACES, the run times varied from 5.44 to 5.49 seconds in four tests.
Is there any more general advice on how to control thread placement and affinity for the best performance with OpenBLAS?  What about systems with more cores and multiple sockets?  Could information about this be added to the documentation?

There is no one size fits all guide... In effect you did equivalent of disabling hyperthreading using variables.

@brada4 can you then suggest a better solution for this case ? E.g. would OMP_NUM_THREADS=8 work with the given definition of OMP_PLACES to add one hyperthread on each core, or are things not that simple ? I agree that documentation on this - in the github wiki or elsewhere  would be helpful.

I believe that if I set OMP_NUM_THREADS=8 with the OMP_PLACES="{0,1,2,3}" then it would run 2 threads on each core with no hyperthreading- places 4-7 are the hyperthreaded siblings of cores 0-3.

I'll add that MKL seems to get this right without any intervention from the user.

Interestingly, I don't see MKL using more than 4 threads on this system, even on fairly large tasks and even with OMP_PLACES left unset and OMP_NUM_THREADS set to 8.
The OMP default environment is:
OPENMP DISPLAY ENVIRONMENT BEGIN
_OPENMP = '201511'
OMP_DYNAMIC = 'FALSE'
OMP_NESTED = 'FALSE'
OMP_NUM_THREADS = '8'
OMP_SCHEDULE = 'DYNAMIC'
OMP_PROC_BIND = 'FALSE'
OMP_PLACES = ''
OMP_STACKSIZE = '0'
OMP_WAIT_POLICY = 'PASSIVE'
OMP_THREAD_LIMIT = '4294967295'
OMP_MAX_ACTIVE_LEVELS = '2147483647'
OMP_CANCELLATION = 'FALSE'
OMP_DEFAULT_DEVICE = '0'
OMP_MAX_TASK_PRIORITY = '0'
OPENMP DISPLAY ENVIRONMENT END
I tried OMP_PLACES=cores, but with OMP_NUM_THREADS unset, it used 8 threads and performed poorly- it appears that "cores" includes all 8 of the virtual cores that GOMP sees.
I also tried OMP_PLACES=cores, with OMP_NUM_THREADS=4.  This ran at about the same speed as  specifying OMP_PLACES="{0,1,2,3}" but  htop showed that it was shifting work from (e.g.) core 1 to its sibling core 5 (core 1's hyperthreaded sibling) and back.
I also tried using OMP_PLACES="{0,1,2,3}" and OMP_NUM_THREADS=8.  With this configuration the performance was poor and htop showed that only cores 0-3 were in use even though there were 8 threads.  Thus it wasn't using hyperthreading.
I conclude that
OMP_PLACES="{0,1,2,3}" is effective at stopping the system from using hyperthreading.
OMP_PLACES=cores doesn't stop hyperthreading.
OMP_NUM_THREADS=4 is consistently better than 8.
OMP_PROC_BIND=spread  seem to keep the threads pinned to the cores they started on.

t was shifting work from (e.g.) core 1 to its sibling core 5 (core 1's hyperthreaded sibling) and back.

Is there any performance impact? In principl cache is shared and should be close to none...
Can you modify intel perf bias register and confirm that in general one hyperthread of 2 gives same numeric performance as both always?

There was no apparent performance impact to the switching between sibling hyperthreaded virtual cores so that probably isn't hurting- I agree that there's no theoretical reason that it should hurt much since the cache is shared.  However, the OS does have to do some bookkeeping to move a thread between cores even if the move is just to a sibling hyperthreaded virtual core.
I don't know what the "intel perf bias register" is?

This one:
man 8 x86_energy_perf_policy


where do I put ACL rules?

I've read about ACL Wiki, but where should I put them? in config.json? or stand alone file?
Could anybody please give me a hint?

Any file will do. Then specify its path with "acl" in config.


Removed Camera Bug

I have solved the recent bug found in camera activity, for that i removed few lines of wrong codes

Hello, Thank you for your contributions,
can you tell me why did you remove the comments

I removed the comment for maintaining the code readability

Thanks, looks good to me!


add definition and test for merge-descriptors

bytes/bytes.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/bytes
http://bower.io/search/?q=bytes
others?

[X] has a test file? (bytes/bytes-tests.ts or others)
[ ] pass the Travis-CI test?

depd/depd.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/depd
http://bower.io/search/?q=depd
others?

[X] has a test file? (depd/depd-tests.ts or others)
[ ] pass the Travis-CI test?

merge-descriptors/merge-descriptors.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/merge-descriptors
http://bower.io/search/?q=merge-descriptors
others?

[X] has a test file? (merge-descriptors/merge-descriptors-tests.ts or others)
[ ] pass the Travis-CI test?

ms/ms.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/ms
http://bower.io/search/?q=ms
others?

[X] has a test file? (ms/ms-tests.ts or others)
[ ] pass the Travis-CI test?

please merge these changes again. thx

:+1:


Won't build correctly...

...or I'm making some error.
I get an Invalid switch - /-I error
I tried both in powershell and in developer console (latter shown in screenshot). I installed gnu make both using shoop and chocolatey (latter in screenshot). Both result in the same error.

The file does get created in ./src/, but following the rest of the instructions (and rebooting my PC) does not prevent e.g. hyper+l from launching the linkedin website.
Many thanks for looking into this and/or correcting my errors.

Yes, I'm on Windows 10.
I did find the hyperenable.exe file, but could not get it to work in disabling the office shortcuts

Yes

Ah, hold on. Let me try to fix something and I'll report back

I did actually forget the start argument. Sorry for wasting your time! 🤐 Thanks for this great piece of software and your lightning fast response!


Globoplay not working

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.01.21. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Brazil
Description
Hello, I use a .bat file to usually download videos, some months ago I tried to download a globoplay link and the return was DRM protected, I think that kind of content wasn't allowed or possible to be downloaded, then I didn't complain about it or thought "it's not working I should report it". Now the error is completely differently and the message encourage me to report it

No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; type  yt-dlp -U  to update.

D:\Users\Pootz\Desktop\youtube>yt-dlp --version
2022.01.21

Today I was trying to download this video https://globoplay.globo.com/v/10248083/ which plays fine even on my firefox 96.0.3 private window (I meant not logged in, on an account) even so I have cookies.txt accounts for every download I use to do.
Differently from those cases:
https://github.com/yt-dlp/yt-dlp/issues/991
https://github.com/yt-dlp/yt-dlp/issues/1132
I've searched and seen now, that were reported here.
I was able to download it on downloadHelper extension but it was on a poor quality. The good ones I wasn't able to get it.
Not even the formats ( -F ) shows any available download quality, but the video plays fine.
Here is another link where the globoplay video is embed
Thanks in advance.
Verbose log
D:\Users\Pootz\Desktop\youtube>yt-dlp --console-title -f best -vU https://globoplay.globo.com/v/10248083/
[debug] Command-line config: ['--console-title', '-f', 'best', '-vU', 'https://globoplay.globo.com/v/10248083/']
[debug] User config "D:\Users\Pootz\AppData\Roaming\yt-dlp\config.txt": ['-f', 'bestvideo[height<=480][ext=mp4]+bestaudio[ext=m4a]/best[height<=480]', '-o', 'D:/Users/Pootz/Desktop/_%(title)s.%(ext)s', '--ffmpeg-location', 'D:/Program Files/Combined Community Codec Pack 64bit/ffmpeg-20200617-0b3bd00-win64-static/bin', '--cookies', 'cookies.txt']
WARNING: "-f best" selects the best pre-merged format which is often not the best option.
         To let yt-dlp download and merge the best available formats, simply do not pass any format selection.
         If you know what you are doing and want only the best pre-merged format, use "-f b" instead to suppress this warning
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.01.21 [f20d607] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.17763-SP0
[debug] exe versions: ffmpeg n4.3.2-160-gfbb9368226, ffprobe n4.3.2-160-gfbb9368226
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2022.01.21, Current version: 2022.01.21
yt-dlp is up to date (2022.01.21)
[debug] [Globo] Extracting URL: https://globoplay.globo.com/v/10248083/
[Globo] 10248083: Downloading JSON metadata
[Globo] 10248083: Downloading security hash for 10248083
[Globo] 10248083: Downloading m3u8 information
WARNING: [Globo] Failed to download m3u8 information: HTTP Error 403: Forbidden
ERROR: [Globo] 10248083: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; type  yt-dlp -U  to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1367, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1451, in __extract_info
  File "yt_dlp\YoutubeDL.py", line 1503, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 2413, in process_video_result
  File "yt_dlp\YoutubeDL.py", line 954, in raise_no_formats
yt_dlp.utils.ExtractorError: [Globo] 10248083: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; type  yt-dlp -U  to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.

Hey @pukkandan, I don't think this is related to geo-blocking as I'm having the same issue with a video that does't have that kind of block. In my case I'm using cookies from browser as it's a video that require the user to be logged in.
Example: https://globoplay.globo.com/v/10250970/programa/?s=0s

The label is to indicate to potential devs that the content is geo-blocked; not that the issue is due to geo-blocking (if so, issue would have been closed). If you can provide free example URL that is not geo-blocked, the label can be removed

If you use saved cookie from browser it works.

D:\ytdlp>yt-dlp.exe -v --cookies cookies.txt  "https://globoplay.globo.com/v/10248083/"
[debug] Command-line config: ['-v', '--cookies', 'cookies.txt', 'https://globoplay.globo.com/v/10248083/']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8 (No ANSI), err utf-8 (No ANSI), pref cp1252
[debug] yt-dlp version 2022.01.21 [f20d607] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [Globo] Extracting URL: https://globoplay.globo.com/v/10248083/
[Globo] 10248083: Downloading JSON metadata
[Globo] 10248083: Downloading security hash for 10248083
[Globo] 10248083: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: best/bestvideo+bestaudio
[info] 10248083: Downloading 1 format(s): hls-2735
[debug] Invoking downloader on "https://vod-as-03-30.video.globo.com/r90_720/v0/c8/82/df/10248083_df4d8fd36481c57f24735d8760755383392237ac/10248083-U369wJY-manifest.ism/10248083-U369wJY-manifest-audio_por=128004-video_por=2452000.m3u8?hls_client_manifest_version=4"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 89
[download] Destination: Melhores momentos - Equador 1 x 1 Brasil pelas Eliminatórias da Copa do Mundo 2022 [10248083].mp4
[download] 100% of 169.49MiB in 00:14
WARNING: 10248083: Possible MPEG-TS in MP4 container or malformed AAC timestamps. Install ffmpeg to fix this automatically

I am using cookies.txt .
Please confirm to me, the cookies.txt goes inside yt-dlp.exe folder ?
I set my default commands on config.txt inside Roaming folder
Unless the cookies firefox extension isn't taking the cookies right.
I enter the same command you enter and the result was the same, and I generated a fresh cookies.txt right now to avoid mistakes.
adding -F or -f best fails here too.
The cookies.txt generated file from globoplay has 44 lines counting from the first line # Netscape HTTP Cookie File

That is what i did when it worked.

Add FF extension. I didn't have it.
Allow FF extension to run on private window.
Open new private window.
Open video link.
Pause the video.
Export "Current Site" cookies.
Save cookies.txt in the same folder as executable.
run command yt-dlp.exe -v --cookies cookies.txt  "https://globoplay.globo.com/v/10248083/"

but.... right now i did the same thing and didn't work.
Before start working I was trying other commands to download files returned on dev tools.

yt-dlp.exe -v "https://v.glbimg.com/v0/c8/82/df/10248083_df4d8fd36481c57f24735d8760755383392237ac/10248083-360p_300k_h264-5a811a60-61fd-013a-fc0a-0242ac110005.mp4"
yt-dlp.exe -v "https://vod-as-03-30.video.globo.com/r90_720/v0/c8/82/df/10248083_df4d8fd36481c57f24735d8760755383392237ac/10248083-U369wJY-manifest.ism/10248083-U369wJY-manifest-audio_por=128004-video_por=301000.m3u8?hls_client_manifest_version=4"
yt-dlp.exe -v --cookies cookies.txt "https://vod-as-03-30.video.globo.com/r90_720/v0/c8/82/df/10248083_df4d8fd36481c57f24735d8760755383392237ac/10248083-U369wJY-manifest.ism/10248083-U369wJY-manifest-audio_por=128004-video_por=301000.m3u8?hls_client_manifest_version=4"
yt-dlp.exe -v --cookies cookies.txt --force-generic-extractor  "https://vod-as-03-30.video.globo.com/r90_720/v0/c8/82/df/10248083_df4d8fd36481c57f24735d8760755383392237ac/10248083-U369wJY-manifest.ism/10248083-U369wJY-manifest-audio_por=128004-video_por=301000.m3u8?hls_client_manifest_version=4"

then suddenly, this commands started working. And I thought: dam i should have tried this at first.

yt-dlp.exe -v --cookies cookies.txt --force-generic-extractor  "https://vod-as-03-30.video.globo.com/r90_720/v0/c8/82/df/10248083_df4d8fd36481c57f24735d8760755383392237ac/10248083-U369wJY-manifest.ism/10248083.m3u8?h=0411643687495730550659316436859021843271116mJtQk4QY7m5sZaAUMxV99Q&k=html5&a=A&u=BR~"
yt-dlp.exe -v --cookies cookies.txt --force-generic-extractor  "https://vod-as-03-30.video.globo.com/r90_720/v0/c8/82/df/10248083_df4d8fd36481c57f24735d8760755383392237ac/10248083-U369wJY-manifest.ism/10248083.m3u8?h=0411643687495730550659316436859021843271116mJtQk4QY7m5sZaAUMxV99Q\u0026k=html5\u0026a=A\u0026u=BR~"
yt-dlp.exe -v --cookies cookies.txt  "https://globoplay.globo.com/v/10248083/"

I will try to investigate a little further to find out what was the turning point.

As I thought, it's strange the issue be only here because I always used a .bat file  ( so the commands it's certainly not wrong) and cookies on config.txt
Maybe the same reason I could get it from downloadHelper extension I said on my main post, is the reason it worked for you.
The extension isn't able to download anymore too.

I find out.
Globo now uses cookies to grant access to manifest and video and changed the json format too.
It only worked before because i called --force-gerenic-extractor with correct cookies before calling the page with globo extractor.
I was able to fix it in my on local copy of yt-dlp, but i don't know if the code is correct for every case, or it can be done in better way.
Or how to test it.

Can someone help me with this?

If you want test the fix for new links
https://globoplay.globo.com/v/10262289/
https://globoplay.globo.com/v/10262174
also not working.

Fixing code already merged with main code, wait for next release or you can download right now using python version.

Thank you, I'll wait the next release.


Create a ResourceNode for a method within a class

I've a situation where the method for handling a particular HTTP request is inside a class and I'm unable to work out how to registerit with the HTTP server. Here's a minimal self-contained example of my issue:
#include <HTTPServer.hpp>
#include <HTTPRequest.hpp>
#include <HTTPResponse.hpp>

using namespace httpsserver;

class ConfigurationModule {

  public:

    void handleDisplayConfig(HTTPRequest *req, HTTPResponse *res) {

    }

    void init(HTTPServer server) {
        std::function<void(HTTPRequest *, HTTPResponse *)> handler = std::bind(&ConfigurationModule::handleDisplayConfig, this, std::placeholders::_1, std::placeholders::_2);
        server.registerNode(new ResourceNode("/configure", "GET", handler));
    }

};

HTTPServer webserver = HTTPServer();
ConfigurationModule config = ConfigurationModule();

void handler1(HTTPRequest *req, HTTPResponse *res) {

}

void handler2(HTTPRequest *req, HTTPResponse *res) {

}

void setup() {

    Serial.begin(115200);

    webserver.registerNode(new ResourceNode("/htmlResponse", "GET", handler1));
    webserver.registerNode(new ResourceNode("/jsonResponse", "GET", handler2));

    config.init(webserver);
    webserver.start();

}

void loop() {
    webserver.loop();
}

This is a technique I've used across my fairly extensive codebase (which I'm currently porting from the ESP8266). However, the code in the init function won't compile, specifically it complains that handler has an incorrect signature, even though the method in question has the same signature as handler1 and handler2:
no known conversion for argument 3 from 'std::function<void(httpsserver::HTTPRequest*, httpsserver::HTTPResponse*)>' to 'void (*)(httpsserver::HTTPRequest*, httpsserver::HTTPResponse*)'

I'm unable to work out the correct syntax for resolving the class method to the typedef HTTPSCallbackFunction, or how to tweak the existing code to get rid of the error I have.
Can anybody help me here?

Hello !
I got the same issue this week and I am happy to read that you just prepared a PR. Is it possible to use the fix right now ? Maybe if I pull the branch ?
Thank you !

Hi,
you can clone the repository and switch to the feature branch:
git clone https://github.com/fhessel/esp32_https_server.git
git checkout origin/feature/functional-callbacks

If you're using Platform IO, do it in your project's lib folder, if you use the Arduino IDE, do it in the libraries folder in your Sketchbook.
It would be great if you could tell me if that works for you, or if you run into problems. If it works, I'd merge the PR soon.

I will clone the branch tonight and test it. I use Visual code with the Arduino Plugin.
I'll give you my feedbacks very soon.

Apologies for my much delayed response, I hope to get back to this in the next week or two...


Update to latest compose bom and move material3 to bom

#159  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite


DEPTH write glitch

Tested versions
godot 4.3 stable
System information
Kubuntu 22.04
Issue description
In some runs, in some shaders, if you write to the DEPTH there will be a glitch happening. This issue will not happen if you don't write to DEPTH.  By setting albedo to depth i have ensured that no such glitch exists in my shader code.
https://github.com/user-attachments/assets/0d52509f-b2f8-414f-9a7c-e562abc0d0aa
Steps to reproduce
Not sure. it sometimes happens in the shader i sent. You just have to make an actual use of writing to DEPTH to end up seeing it.
However it's much more likely to happen right after shader recompilation
Minimal reproduction project (MRP)
depth_write_bug_report.zip

I took a look at the shader and I don't think there is a Godot bug. The shader itself seems to be writing a near-random value to DEPTH which is why DEPTH appears scrambled.
There are a few notable mistakes I noticed in the shader:

The view space ray direction is not perspective correct (missing division by w)
The ray hit length is always 1 (it is just the length of rd which has just been normalized)
The ray hit position is always the view space ray direction (which is broken because of point 1
The ray direction is converted into clip space then used as a depth, without a translation component, that depth doesn't make sense
If the code was working correctly, it would be equivalent to DEPTH = 1.0;, but the code itself introduces a bunch of errors and then writes them out.

I took a look at the shader and I don't think there is a Godot bug. The shader itself seems to be writing a near-random value to DEPTH which is why DEPTH appears scrambled.
There are a few notable mistakes I noticed in the shader:

The view space ray direction is not perspective correct (missing division by w)
The ray hit length is always 1 (it is just the length of rd which has just been normalized)
The ray hit position is always the view space ray direction (which is broken because of point 1
The ray direction is converted into clip space then used as a depth, without a translation component, that depth doesn't make sense
If the code was working correctly, it would be equivalent to DEPTH = 1.0;, but the code itself introduces a bunch of errors and then writes them out.

will close this for now. while i think there is a godot but, i will resolve the code issues and comeback when i still get it.

I took a look at the shader and I don't think there is a Godot bug. The shader itself seems to be writing a near-random value to DEPTH which is why DEPTH appears scrambled.
There are a few notable mistakes I noticed in the shader:

The view space ray direction is not perspective correct (missing division by w)
The ray hit length is always 1 (it is just the length of rd which has just been normalized)
The ray hit position is always the view space ray direction (which is broken because of point 1
The ray direction is converted into clip space then used as a depth, without a translation component, that depth doesn't make sense
If the code was working correctly, it would be equivalent to DEPTH = 1.0;, but the code itself introduces a bunch of errors and then writes them out.

1- w division is not needed here. the result is normalized.
2- ray hit length of 1 was a bug not it's not anything important in this sample. the orginal code was ro.y/rd.y to simulate a horizontal plane for an ocean.
3- ray hit position is always 1*ray direction in view space(although due to the bug). that should be fine mathematically. this shouldn't cause the visuals you see in the video. if i add ro and use the 4x4 matrix it makes no difference since it's the camera origin.
4- the rotation element of the view matrix is used to convert a world direction into a view direction. translation here is not needed
5- "new_depth" is never 1.0.
the outputs are nothing but random, the issue does not show in albedo. synchronized trigger by shader recompilation suggests that there is a synchronization problem either in godot or the driver


Pagination page links

Only implements for species records, not species search, as don't think API will support it (doesn't appear to return record count for facet records).  But one small step forward.

Interesting - had not come across Userback before. Looks handy. Similar to marker.io I think?

Yes just trying it out.
From: Charles Roper @.>
Sent: 01 May 2021 13:42
To: jdrever/ShropBotSoc @.>
Cc: James Drever @.>; State change @.>
Subject: Re: [jdrever/ShropBotSoc] Pagination page links (#127)
Interesting - had not come across Userback before. Looks handy. Similar to marker.io I think?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/jdrever/ShropBotSoc/pull/127#issuecomment-830626574, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAPKYJJIKCCXZ326PHNNBODTLPZJ3ANCNFSM434UVHUA.


Need to add  readme

My Full-Stack Template
Welcome to the repository for my full-stack template, integrating a React front-end with a Node.js and Express back-end. This README provides a detailed explanation of the application, its components, and how to get started.
Table of Contents

Introduction
Technologies Used

Frontend
Backend

Installation
Running the Application
Conclusion

Introduction
This application was conceived as a blueprint for rapid full-stack development, offering the seamless integration of a dynamic React front-end and a robust, scalable Node.js back-end. This amalgamation of technologies ensures that developers can focus on building features without the usual setup overhead.
Technologies Used
Frontend

Vite: A build tool that boasts a faster and leaner development experience for modern web projects.
React: A JavaScript library for building user interfaces. It allows us to create reusable UI components.
TypeScript: A superset of JavaScript, offering static type-checking and the latest ECMAScript features.
TailwindCSS: A utility-first CSS framework packed with classes like flex, pt-4, text-center and more to style your websites without leaving your HTML.
React-Router-Dom: A routing library for React, enabling navigation among views.
Jest: A delightful JavaScript Testing Framework with a focus on simplicity.

Backend

Node.js: A JavaScript runtime built on Chrome's V8 JavaScript engine.
Express: A minimal and flexible Node.js web application framework that provides a robust set of features for web and mobile applications.
TypeScript: Brings static typing to JavaScript, ensuring a more predictable runtime behavior.
Jest: Used for writing unit and integration tests for the backend.
PostgreSQL: A powerful, open-source object-relational database system.
Nodemon: A utility that monitors for any changes in your source and automatically restarts your server.

Installation
Before you start, ensure you have node and npm installed on your machine.

Clone the repository:
git clone https://github.com/yuribodo/GPMweb.git

Navigate to the repository:
cd full-stack-template

Install the dependencies:

For Frontend:
cd frontend && npm install

For Backend:
cd backend && npm install

Running the Application

To run the frontend:
npm run start-frontend

This starts the React application on http://localhost:5173 (or another available port).

To run the backend:
npm run start-backend

This initializes the Express server, typically on http://localhost:3000.

To run both simultaneously:
npm run start

This will invoke concurrently to start both the front and back ends.

Ensure that the frontend and backend are configured to run on separate ports to avoid conflicts.
Conclusion
This full-stack application setup is intended to accelerate the development process by providing a cohesive blend of some of the best technologies in web development today. Whether you're looking to create a small prototype or build a large-scale application, this repository provides a solid foundation. Happy coding!

If you find any bugs or have a feature request, please open an issue on GitHub.
Made with ❤ by Gabriel Krapp.

I will


investigate ml_kmeans() test failure with Spark 2.1.0

Failed -------------------------------------------------------------------------
1. Failure: 'ml_kmeans' and 'kmeans' produce similar fits (@test-ml-kmeans.R#30)
all.equal(lhs, rhs, tolerance = 0.01, ...) isn't true.

Resolved with https://github.com/rstudio/sparklyr/commit/b58cf3b8378a845ba190901e33a601a71fcaa815 -- it was an issue with the test itself.


ERROR: Failed building wheel for dependency-injector

Attempt to install the requested package, however it still hit the same error message when I re-run the following command: pip install "aiverify-moonshot[all]".
Kindly assist,
Leon

Hi Leon,
According to the log person that you have send, you might have an older version of Microsoft Visual C++. Are you able to check the version of your C+= build tools?

Thank! I managed to get it running.

It freezes on the Run Screen when I attempt to test a model.

Would like to verify for OpenAI GPT4 the API in the following image below?

You may refer to the following log below

Kindly assist,
Leon

in order to generate a result for mlc-ai-safety , you will need to have an API token for together-llama-guard-7b-assistant . maybe you can try  excluding the MLC Safety cookbook from your benchmark run

also it seems that you might be missing the openai. maybe certain dependencies weren't installed fully when you faced the issue with tensorflow. maybe you can try re installing the requirements for moonshot-data

hello @leonyap27  ,
Are you still facing any issue with running moonshot?

hello @leonyap27 ,
Are you still facing any issue with running moonshot?
Yes, I try re-install on another workstation I encounter the issue below (see image 1), seem like it require tensorflow-io-gcs-filesystem 0.37.1 but (refer to image 2) in anaconda.org only have 3.10 available.

image 1

image 2

may i know what version of python and pip3 you are running on?

may i know what version of python and pip3 you are running on?
@imda-normanchia , I manage to install 3.10 refer to https://github.com/aiverify-foundation/moonshot/issues/312.

However I encounter runtime issue, I believe maybe I not familiar with the configuration.
If possible can provide the high level information where I can get relevant API key for different LLM. Thanks.

if you are testing open ai model, you would need to put in your OpenAI API Key in the token field.
you may refer to this article on how to find your api key.
https://help.openai.com/en/articles/4936850-where-do-i-find-my-openai-api-key

hello, may i know if you are still facing any issue? i would be closing this issue if you are no longer facing any issue.


Better CAN support

@Pho3niX90 you want to lead this?

Yes perfect thank you

I will create similar issue in the API project.


I mode power threshold updates

This reopens https://github.com/cfs-energy/cfspopcon/pull/46#event-12848216546 and closes https://github.com/cfs-energy/cfspopcon/pull/56, now merging just the new threshold formulas into main.
The PR contains simple updates to the I-mode LI power threshold and taue scalings from Amanda Hubbard. Addresses standing issue https://github.com/cfs-energy-internal/internal-cfspopcon/issues/149.

@tbody-cfs I think this is all that is needed for the P_LI threshold additions

Hi @nelsonand and @aarooshgr,
Thanks for the work!! I've made a small edit which lets us switch the confinement scaling using the input.yaml file.

Hi Tom,
Sure, I could do that. I’m away from my computer at the moment so I can get
to it this evening
On Wed, Jun 12, 2024 at 3:47 AM Tom Body @.***> wrote:

@.**** requested changes on this pull request.
Could you pull in 0b80d7d
https://github.com/cfs-energy/cfspopcon/commit/0b80d7d4dc96c42248bf27bf6185103ca1473bb9
from the I_mode_updates branch?
—
Reply to this email directly, view it on GitHub
https://github.com/cfs-energy/cfspopcon/pull/58#pullrequestreview-2112224724,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4GUOLPIXXRI5NT2AZVKIYLZG74H7AVCNFSM6AAAAABIGSIYLOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMJSGIZDINZSGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

So, I just remembered that I don't have the access to push a commit to main... @nelsonand would you be able to do this?

@nelsonand @aarooshgr could you add the following to docs/doc_sources/physics_glossary.rst
At the end is fine. Make sure to match the indentation of the existing file
  confinement_power_scaling
    A :class:`~cfspopcon.named_options.ConfinementPowerScaling` indicating which confinement mode power-threshold scaling to use.

@nelsonand @aarooshgr could you add the following to docs/doc_sources/physics_glossary.rst
At the end is fine. Make sure to match the indentation of the existing file
  confinement_power_scaling
    A :class:`~cfspopcon.named_options.ConfinementPowerScaling` indicating which confinement mode power-threshold scaling to use.

Hi Tom, just pushed this change to the I_mode_updates branch of my fork. Please let me know if you need anything else

Thanks @tbody-cfs !


Support order index for parameters

Problem
Parameter order right now is determined by the parameter ID. This is bad because reordering parameters requires changing the ID, which basically means removing the parameter and adding a new one. This renders template upgrades really difficult.
Solution
Introduce a new attribute for each parameter that indicates the position in the ordering. It should be an integer and the parameters will be sorted by this index and then by ID if there is a tie. If a parameter does not have an index it will be shown in the end.
Example
parameters = {
  "username" = {
    name = "Username"
    type = string
    order-index = 1
  }
  "password" = {
    name = "Password"
    secret = true
    order-index = 2
  }
  "http-port" = {
    name = "HTTP Port"
    default = 80
    order-index = 3
  }
}

@Gerrrr  what do you think?


Can't override "password_reset_key_message.txt"

I'm having issues overriding the default email for the "forgot password". I can override the 2 email for "email confirmation" without any issue but not with this one.
Not matter the extension, the one sent is always the default one that comes with the package.
Thanks

Issue persists for me too. Were you able to find a fix @MadReal ?

Same issue here. Any suggestions? @MadReal @iMerica

password_reset_key_message.txt is a template provided by allauth. You'd need to override that path.

The email sent is not using password_reset_key_message.txt, instead it is sending Django's default email template from sites-packages/django/contrib/admin/templates/registration/password_reset_email.html
Can anyone make clear about this?

For those whos are facing issue, I managed to get it work by overriding Django's template.
Create a new file at templates/registration/password_reset_email.html with content
{% load i18n %}{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Hello from {{ site_name }}!

You're receiving this e-mail because you or someone else has requested a password for your user account.
It can be safely ignored if you did not request a password reset. Click the link below to reset your password.{% endblocktrans %}

{% block reset_link %}
{{protocol}}://{{ domain }}{% url 'password_reset_confirm' uidb64=uid token=token %}
{% endblock %}

{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Thank you for using {{ site_name }}!
{{ site_domain }}{% endblocktrans %}

For the developer:
In the PasswordResetSerializer  you are using PasswordResetForm which is provided by
`` `
from django.contrib.auth.forms import PasswordResetForm

In order to use the `allauth` email template, you need to use `ResetPasswordForm` provided by `allauth` instead.

For those whos are facing issue, I managed to get it work by overriding Django's template.
Create a new file at templates/registration/password_reset_email.html with content
{% load i18n %}{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Hello from {{ site_name }}!

You're receiving this e-mail because you or someone else has requested a password for your user account.
It can be safely ignored if you did not request a password reset. Click the link below to reset your password.{% endblocktrans %}

{% block reset_link %}
{{protocol}}://{{ domain }}{% url 'password_reset_confirm' uidb64=uid token=token %}
{% endblock %}

{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Thank you for using {{ site_name }}!
{{ site_domain }}{% endblocktrans %}

hello, I tried this solution but discovered that using html elements don't work properly, it just sends a mail with raw html code. How were you able to style yours?

password_reset_key_message.txt is a template provided by allauth. You'd need to override that path.

I overrode that path, it didn't work. Please can you share how you solved this, it could save me weeks

@opeodedeyi did you solve this? My password reset also sends plain text files.

yes, I did solve this eventually. do you also have an issue with that?
On Wed, 22 Jul 2020 at 22:47, b0lle<EMAIL_ADDRESS>wrote:

@opeodedeyi https://github.com/opeodedeyi did you solve this? My
password reset also sends plain text files.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Tivix/django-rest-auth/issues/393#issuecomment-662714256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AENRXFCVD3BP3BTASRFJSDLR45M6ZANCNFSM4EH3XT3Q
.