Field with name uuid cannot be set as optional not required

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
If field name is "uuid" it cannot be used as Optional not required.
Python raises error during startup.

Traceback (most recent call last):
File "test.py", line 4, in
class Test(BaseModel):
File "test.py", line 5, in Test
uuid: uuid.UUID | None = None
^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'UUID'

If name is set to "aaa" everything works as it should.
Example Code
from pydantic import BaseModel, ValidationError
import uuid

class Test(BaseModel):
    uuid: uuid.UUID | None = None
    name: str

print(Test(name='test'))

Python, Pydantic & OS Version
import pydantic.version; print(pydantic.version.version_info())
             pydantic version: 2.3.0
        pydantic-core version: 2.6.3
          pydantic-core build: profile=release pgo=false
                 install path: /...../lib/python3.11/site-packages/pydantic
               python version: 3.11.5 (main, Aug 24 2023, 15:18:16) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
                     platform: macOS-13.5.2-x86_64-i386-64bit
     optional deps. installed: ['email-validator', 'typing-extensions']

Hi @vavroch2010,
2 potential solutions to this:
from uuid import UUID, uuid4

from pydantic import BaseModel, ValidationError

class Test(BaseModel):
    uuid: UUID | None = None
    name: str

print(repr(Test(uuid=uuid4(), name='test')))
#> Test(uuid=UUID('33a865b2-8c8c-4cb4-bdff-7f95d6bade80'), name='test')

print(repr(Test(name='test')))
#> Test(uuid=None, name='test')

or:
import uuid

from pydantic import BaseModel, ConfigDict, Field, ValidationError

class Test(BaseModel):
    uuid_: uuid.UUID | None = Field(default=None, alias="uuid")
    name: str

print(Test(uuid=uuid.uuid4(), name='test').model_dump(by_alias=True))
#> {'uuid': UUID('b39321db-5946-4386-9885-6d691a9b2a51'), 'name': 'test'}

print(Test(name='test').model_dump(by_alias=True))
#> {'uuid': None, 'name': 'test'}

Let me know if this helps!

Hi, thanks for quick reply!!!
I already solved that similar way like option 2. I am using it in fastapi framework as an input validation so I cannot really use option 1.
Basically I think is kind of bug and should be solved either by documentation or code change.
What do you think about that?

This is not a bug. The problem is that the uuid when you are adding the type hint doesn't refer to the import statement, it refers to the model attribute.
It's a matter of namespace, and the fact that Pydantic uses type hints at runtime.
The suggestions that @sydney-runkle proposed are, in short, to avoid the naming conflict of uuid.


passing database to Schema.create

possible fix for #423

Unfortunately this only solves the Schema creation problem, the same problem exists for Model.dispatch. @kiaking do you have n idea how to fix it?

Closing the issue due to inactivity. Please feel free to open another issue if there's anything! 🤝


Bug Fix bug for cast function between datetimev2 and string

Proposed changes
Fix bug for cast function between datetimev2 and string
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Add Modrinth support

To what extent is it possible add Modrinth?

To what extent is it possible add Modrinth?

Fairly sure Modrinth does not have a server jar itself, unless I couldn't see it, please link me to it

To what extent is it possible add Modrinth?

Fairly sure Modrinth does not have a server jar itself, unless I couldn't see it, please link me to it

Modrinth has jar files and an API, here:

https://mohistmc.com/mohistmc-api
https://mohistmc.com/downloadSoftware?project=mohist&projectVersion=1.20.1

Example:

https://github.com/pelican-eggs/minecraft/blob/main/java/modrinth/egg-pterodactyl-modrinth-generic.json

https://mcjars.app/MOHIST/versions
On Thu, Nov 14, 2024, 12:35 AM Sam Schumacher @.***>
wrote:

To what extent is it possible add Modrinth?
Fairly sure Modrinth does not have a server jar itself, unless I couldn't
see it, please link me to it
Modrinth has jar files and an API, here:

https://mohistmc.com/mohistmc-api

https://mohistmc.com/downloadSoftware?project=mohist&projectVersion=1.20.1
Example:

https://github.com/pelican-eggs/minecraft/blob/main/java/modrinth/egg-pterodactyl-modrinth-generic.json
image.png (view on web)
https://github.com/user-attachments/assets/0dae65f4-46d7-41f6-a209-1df53b0f99ae
—
Reply to this email directly, view it on GitHub
https://github.com/mcjars/www/issues/8#issuecomment-2474960594, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AICPHKTE6PCB6FU7JACOS2T2APH2FAVCNFSM6AAAAABRXPURDKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZUHE3DANJZGQ
.
You are receiving this because you commented.Message ID:
@.***>

sorry. i meant Modrinth... I had just checked and Modrinth does not offer Jar files


feat: add labelDetails support

Yep that's preferred

@Saghen excuse my ignorance but I spent a while looking for how to add the import source using the transform_items function on the lsp but couldn't get it to work. Could you show me an example of how that would be done?
I am guessing we need to do something here:
    sources = {
        providers = {
            lsp = {
                transform_items = function(ctx, items)
                    vim.print(vim.inspect(ctx))
                    vim.print("***")
                    vim.print(vim.inspect(items))

                    return items
                end,
            }
        }
    }

in the config but I can't seem to get it to work.
This option shows the import
        windows = {
            documentation = {
                auto_show = true
            },

but there is a noticeable lag while it loads so it I can load it faster then that would be preferred. If that is simply down to the lsp though, then that's fine. Cheers!

There's auto_show_delay that you could make 0 to immediately load it but otherwise, it's up to the LSP


OPNsense 15.7.7_3-amd64 - DHCPv6 on WAN Interface Blocked by Firewall

With DHCPv6 enabled on the WAN Interface, the firewall is blocking the traffic with the ISP server. The WAN Interface is unable to obtain a DHCPv6 IP.
OPNsense 15.7.7_3-amd64
FreeBSD 10.1-RELEASE-p16
LibreSSL 2.2.1
Additionally - and perhaps for a different issue - this article describes a basic ruleset for an IPv6 firewall of which certain elements may be worth considering:
http://pivotallabs.com/configuring-freebsd-9-1-as-an-ipv6-dhcp-client/

I'm unsure which traffic is missing. I use DHCPv6 on the WAN and can obtain an IPv6 address. The rules generated on WAN for DHCPv6 are:
pass in quick on $WAN proto udp from fe80::/10 port = 546 to fe80::/10 port = 546  label "allow dhcpv6 client in WAN"
pass in quick on $WAN proto udp from any port = 547 to any port = 546 label "allow dhcpv6 client in WAN"
pass out quick on $WAN proto udp from any port = 546 to any port = 547 label "allow dhcpv6 client out WAN"

I don't have those rules on my web view. Are they behind the scenes?
I am getting this in my logs when filtering for blocks:
Act	Time		If	  Source						Destination					Proto
X Aug 9 12:34:17	WAN  [fe80::xxx:xxxx:xxxx:xxxx]:547	  [fe80::xxx:xxxx:xxxx:xxxx]:546	UDP
X Aug 9 12:34:17	WAN  [fe80::xxx:xxxx:xxxx:xxxx]:547	  [fe80::xxx:xxxx:xxxx:xxxx]:546	UDP
X Aug 9 12:32:18	WAN  [fe80::xxx:xxxx:xxxx:xxxx]:547	  [fe80::xxx:xxxx:xxxx:xxxx]:546	UDP
X Aug 9 12:32:18	WAN  [fe80::xxx:xxxx:xxxx:xxxx]:547	  [fe80::xxx:xxxx:xxxx:xxxx]:546	UDP
The destination matches the link local for my WAN interface.

The are from /tmp/rules.debug -- there ought to be a block with a header "# allow our DHCPv6 client out to the WAN"
or something else is overriding your traffic. Which rule causes this? Click "x" to reveal this.

The rule that triggered this action is:
@40 block drop in log quick on em1 from bogonsv6:0 to any label "block bogon IPv6 networks from WAN"

Is /usr/local/etc/bogonsv6 empty? If not, does it have a line beginning with a "fe80:" entry?

I could reproduce with an empty file. That's really funny as the "default" file shipped with the system has been empty forever. To fix, run this on the command line:
# /usr/local/etc/rc.update_bogons 1

I'll put a few initial values into /usr/local/etc/bogonsv6.sample to avoid this problem in the future.
Let me know if this helped or not.

There was more to it apparently after having had a little push in the right direction. Traffic is now passed. Closing in time for 15.7.8.


Discovery - Rendering Hard-Fork Initiation Governance Actions

Add the scope and parameters of the "Update committee/threshold" here to help with screen and flow design.
What is this action all about?
What are it's parameters?
What inputs need to be entered by users?
What outputs need to be served to users?
What are the technical implications?
Anything else?

What is this action all about?

Triggers a non-backwards compatible upgrade of the network; requires a prior software upgrade (upgrading node software versions)
Voted on by DReps, CC and SPOs
Available within the bootstrapping phase (will be used to exit the bootstrapping phase)

What are the on-chain details? (specific to treasury withdrawals)

governance action ID of the previous hardfork initiation to be enacted (this isn't super important, but shows users how hard-forks can be chained together)
the next major and minor protocol versions

For example;
The hard-fork governance action for Chang #2 could look like

prev governance action ID = fc0386f81025d66fc448d4f0aa3ca6c1eeae95fe88e3b10244bacc126bbeb007#0
next protocol version (major.minor) = 10.0

What is other contextual information that might be nice?

the current protocol version

Next step;

wireframes by Friday

cc @bosko-m

Wireframe to discuss:

@m-i-k-e
oo! this looks nice
I would rename Parameter label on the tab to details or something similar
to avoid confusion with protocol parameters

Good call!

@MSzalowski This is now ready to develop in Figma: https://www.figma.com/design/mJTBiUofLfb74Xc3W2U59o/Voltaire---GOV-TOOL?m=auto&node-id=14751-92651&t=hLlpXMOe0RpGkeDc-1


(Small) Visual Issue present

Random characters visible on top left hand corner of every page
Probably an unclosed bracket somewhere
image included

circled in top left corner

@vpetersson @fshmcallister , This issue was resolved as part of another task.


Reduce number of iterations

OSU pt2pt default value of 1000 iterations takes quite long to run on some systems; 100 iterations should be sufficient.

PR made to wrong repo.


[INFRA]: post-#475 - migrate schema access code (Python API) to bids-schema

Continuing discussion which started in #475 but should be handled (some time after #475 is merged and a new BIDS spec version released).
My thinking as for moving support code to bids-schema was -- to facilitate access (for code) to previous versions of schema so validators etc could validate against actual schema version, instead of the most recent or whatever single one they have available; and also schema access API/code could be tested against older versions etc.  Actually probably not all code should migrate -- schema access code/API -- migrate.  Helpers to "render markdown" -- should reside here since that is where they are most pertinent and not needed for 3rd party applications wanting to access schema itself.
While actual "current" schema would reside here (and only archived upon release to bids-schema), released code should not need changes for schema/docs changes in BIDS itself.  Any changes to code to render data within bids-specification would then done in this repository.
There will be cases were code changes would be necessary to accommodate new desires in the schema itself!  For those developments it should be relatively easy to establish a branch within bids-schema with needed changes in the code, while working also here on actual schema changes. It just would need bids-schema code release before PR here would be merged.  A bit cumbersome... but could work.
Having said all that, I think all code should reside here at least for some time so that schema and code "solidify" a bit more...
Alternative: do not move code at all, and rather

make bids-specification itself into providing a python package (bids_schema) with the API
might require releases of bids_schema python package code without releasing bids-specification itself

although sounds simple(r) I think such a mix of code + specification might eventually bite back painfully.

In #610, the schema code is restructured into a more package-like structure. @sappelhoff brought up the importance of fully converting the code to a package, with a proper setup, unit testing, and linting. The code contains a range of functions, though, and only some of them are generic BIDS code. The rest are specific to the specification rendering. I'm not sure if we want to split up the package, though, so I'd love to get folks thoughts on this.

I think splitting it out should indeed be done, but since schema and the supporting code is still "in flux" not sure if splitting into a separate repository would cause burden which could be avoided while keeping it in here for now.  For now it could indeed be worked out into a proper package within this repo for "schema" (but not rendering) support code, to be installed and tested as part of the workflows.

What kind of structure do you think would work best? Something like this?
tools/
    schema/
        # package of schema-related code
    schemarendering/
        # package of rendering functions, with schema package as a dependency

Dear @tsalo . In DANDI archive project we need to add BIDS datasets validation (https://github.com/dandi/dandischema/issues/74), and I think we should use the src/schema within BIDS or to say more precisely -- its releases (since datasets could be of older versions) via a set of supporting Python interfaces.
IIRC there was some (more) recent on the potential future of the src/schema and Python interfaces to use it, but I fail to find it... But may be it is time to resurrect bids-schema effort and consider establishing desired "pipeline" for releasing etc.
FWIW, in DANDI we have

https://github.com/dandi/dandischema/ where DANDI's data schema is designed/defined via pydantic models.
https://github.com/dandi/dandischema/blob/master/.github/workflows/release.yml workflow takes care about providing per-release jsonschema serializations of the schema into https://github.com/dandi/schema/tree/master/releases -- which later can/is used by web ui rendering, or could be used for basic validation (pydantic models provide more thorough testing).

I don't want to pollute this issue with ideas, but would be happy to discuss this topic further and summarize "lessons learned" (from dandi) which might help here.

I recently met with  @erdalkaraca and Jochem Rieger, who are working on Python tools to interact with the schema, as well as validate and search BIDS datasets using the schema directly. In order to not step on anyone's toes, they're working on that in https://github.com/ANCPLabOldenburg/ancp-bids (see https://github.com/bids-standard/pybids-light/issues/1 as well), but they hope to integrate the new code into pybids directly.
Based on that discussion, it seems like functions for working with the schema would fit best in pybids, while rendering code in the specification repository would use pybids as a dependency, and bids-schema could be used to back up versions of the schema, as we've discussed. That way, if the structure of the schema changes, then we can update old versions in bids-schema without having to try to change specification releases.
Does that fit with what you're proposing, or would you prefer a different strategy?

Great to hear that there is more interest/work ongoing!

they hope to integrate the new code into pybids directly

that is what I am not sure about since pybids itself is a bit too heavy with all the pandas etc depends, that is why I somewhat liked an idea for pybids-light is worth pursuing... an alternative would be to look into lightweighting default pybids installation but not sure how good that idea is.
That is why I was thinking about just beefing up the "stock" bids-validator python module which pybids uses first so then the effect would benefit everyone -- anyone interested to just validate BIDS without needing to install heavy pybids, and all pybids users.
But then where code to read the model should reside?  could be done in bids-validator, but that would be suboptimal.  So should be yet another different module. placing into pybids -- makes impossible to use with bids-validator thus making it obsolete and just getting back to heavy pybids...
So -- not yet sure on the best course of action, besides a dedicated lightweight bids-schema python module or lightweighting pybids (unlikely to happen, right @effigies @tyarkoni ?)

That way, if the structure of the schema changes, then we can update old versions in bids-schema without having to try to change specification releases.

yes, I guess, kinda ;-)  I think that bids-schema should contain original releases of the schema as they were released within bids-specification.  If there is a structural change, the library which supports reading it should either auto-migrate upon load if so needed, or we should have <version>/<target_version>/ hierarchy, where original one would go under <version>/<version> and any migrated/tuned to the corresponding versioned subfolder.
Funny enough - we are going through pretty much exactly the same discussion in DANDI about its schema migration: auto vs explicit etc. Ref: https://github.com/dandi/dandi-api/pull/475 . And my stand is obvious -- I would like to keep "original" explicitly present, and if needed to be updated/migrated - explicitly upgraded/migrated but so that original one is still available.  Otherwise - e.g. errors in migrations etc would be hard(er) to mediate (since might be buried in the chain of commits/migrations - at least here everything will be under git) etc.
And if the code which is used to read schema resides in the same repo, then we need really just <version>/orig and (optional) <version>/migrated where /migrated would be the one current version of library reads, and if needs to be re-migrated - would be done into the same subfolder. There should also be somewhere indicator to which version of the schema it was migrated as representative of "which structural schema of the schema ;)" it uses.

Great to hear that there is more interest/work ongoing!

they hope to integrate the new code into pybids directly

forgot to mention: from what I have seen -- this sounds more like a birth of an external project, since more time goes would become only more difficult to "merge" it into another (e.g. pybids).

Hi Yaroslav,
I think what Taylor means is contributing back to the pybids project. Merging back will not be possible since it is not a fork of pybids and also completely different approaches. We will be experimenting with various technologies to find out the best approach.
For example, I just finished an initial XSD schema (see [1]) taking Taylor's work into consideration (based on YAML). From the XSD schema I generated Python code (using generateDS.py [2]) which is meant to be an API to the schema. That approach allows to use Python's lxml module to query a dataset (in-memory).
[1] https://github.com/ANCPLabOldenburg/ancp-bids/tree/xsd-experiment
[2] generateDS.py: http://www.davekuhlman.org/generateDS.html#how-to-build-and-install-it

Great to hear that there is more interest/work ongoing!

they hope to integrate the new code into pybids directly

forgot to mention: from what I have seen -- this sounds more like a birth of an external project, since more time goes would become only more difficult to "merge" it into another (e.g. pybids).

I'm currently working on setting up a smaller (i.e. both in therms of code and dependencies) regex-based validator.
Given that the schema is being separated, this could be pointed to a schema package version, or extended schemas e.g. for people working on new kinds of data and trying to develop the standard further.
This is the code thus far:
https://github.com/TheChymera/bids-specification/tree/validator-xs
And this should be tracking the diff:
https://github.com/bids-standard/bids-specification/pull/969
@yarikoptic

@yarikoptic ok, so this basically works as per 1fc9b953f1d9db4df4368f38733ca2e0b9c6965a .
This is an example validation report for DANDI:000108: https://ppb.chymera.eu/1d5ef8.log
One last issue I have, is I can't find the representation of *sessions.tsv in the YAML.
I know it's part of BIDS, I put it in my own datasets and it gets auto-created by my Bruker-BIDS parser, but it doesn't seem to be represented, and is only mentioned once in the comments...
Any ideas?

@yarikoptic regarding *_sessions.tsv, it seems some work on BIDS will be required there.

I think what Taylor means is contributing back to the pybids project.

that is good (I am all for contributing to existing projects).  The main concern (as stated elsewhere, TODO: ref) with that approach is that pybids has heavy dependencies (e.g pandas) which would make it a "questionable" dependency to add to relatively lightweight projects (such as our dandi-cli).

@yarikoptic we have a fix and can now validate sessions files:
https://github.com/TheChymera/bids-specification/commit/a15ac46cec64342c4cc8eae8c0540a3fa2cb1ea5
https://github.com/TheChymera/bids-schemadata/commit/c8950ef92a2ee7fa0afb9e6c0c3b267c113a1d5e
Though the official solution is not yet finished: https://github.com/bids-standard/bids-specification/pull/985#issuecomment-1019573787


[BUG] Error on $token = Auth::generateTokenById(1);

Please prefix your issue with one of the following: [BUG] [PROPOSAL] [QUESTION].
In raising this issue, I confirm the following (please check boxes):

[x] I have read and understood the contributors guide.
[x] I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
[x] I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.

My familiarity with the project is as follows (check one):

[ ] I have never used the project.
[x] I have used the project briefly.
[ ] I have used the project extensively, but have not contributed previously.
[ ] I am an active contributor to the project.

My project stack details

Framework (Laravel / Lumen):
Framework Version: Laravel
JWT Auth Guard Version:  installed today -> 1.0
tymon/jwt-auth Version: ^1.0@dev installed

Thanks for the "wrapper" -- spent too much trying to get the "guards" to work directly in the tymon project.
I am getting the following error though:
ErrorException in JWT.php line 59:
at HandleExceptions->handleError('4096', 'Argument 1 passed to Tymon\JWTAuth\JWT::fromUser() must be an instance of Tymon\JWTAuth\Contracts\JWTSubject, instance of App\User given, called in /Sites/tpHP/vendor/irazasyed/jwt-auth-guard/src/JwtAuthGuard.php on line 177 and defined', '/Sites/tpHP/vendor/tymon/jwt-auth/src/JWT.php', '59', array()) in JWT.php line 59
Here is the top majority of the stack trace just in case:

in JWT.php line 59

at HandleExceptions->handleError('4096', 'Argument 1 passed to Tymon\JWTAuth\JWT::fromUser() must be an instance of Tymon\JWTAuth\Contracts\JWTSubject, instance of App\User given, called in /Sites/tpHP/vendor/irazasyed/jwt-auth-guard/src/JwtAuthGuard.php on line 177 and defined', '/Sites/tpHP/vendor/tymon/jwt-auth/src/JWT.php', '59', array()) in JWT.php line 59

at JWT->fromUser(object(User)) in JwtAuthGuard.php line 177

at JwtAuthGuard->generateTokenById('37')

at call_user_func_array(array(object(JwtAuthGuard), 'generateTokenById'), array('37')) in AuthManager.php line 292

at AuthManager->__call('generateTokenById', array('37')) in Facade.php line 217

at AuthManager->generateTokenById('37') in Facade.php line 217

at Facade::__callStatic('generateTokenById', array('37')) in routes.php line 47

at Auth::generateTokenById('37') in routes.php line 47

at RouteServiceProvider->{closure}()

at call_user_func_array(object(Closure), array()) in Route.php line 158

at Route->runCallable(object(Request)) in Route.php line 137

at Route->run(object(Request)) in Router.php line 724

at Router->Illuminate\Routing{closure}(object(Request))

at call_user_func(object(Closure), object(Request)) in Pipeline.php line 52
.....

Thanks for opening the issue. It looks like, You haven't implemented the JWTSubject.
Please refer this comment (The implementation example): https://github.com/irazasyed/jwt-auth-guard/issues/5#issuecomment-175152252
Once you implement that, it should work (If all the other steps are done properly).
Hope that helps. Please re-open if it doesn't work. Thanks!


[JN-942] Proxy and Governed user dashboard

DESCRIPTION
This PR allows the Proxy Participant to switch between the dashboard for themselves and for their governed user and to attempt to take surveys on behalf of the governed user
This PR does NOT include the "Add Participant" Button, that one is coming later
TO TEST:  (simple manual steps for confirming core behavior -- used for pre-release checks)
repopulate demo, sign in as either<EMAIL_ADDRESS>or<EMAIL_ADDRESS>try switching between the users from the drop down and take surveys and consents on behalf of the participant.
Also sign in as jsalk or other adult users and make sure the surveys and dashboards work as expected

@devonbush @MatthewBemis this is ready for review again!
major changes

fixed bug on registration where your enrollees did not show up until you refreshed
changed login result DTO to flatten everything out (enrollees -> all enrollees, including proxies, same with ppUsers. relations no longer have targetEnrollees attached, as that almost always complicated the code)
added concept of activeUser - which is the ppUser you are acting on behalf of. This means you don't act on behalf of a single enrollee at a time, but a set of enrollees under a ppUser. This means we can now proxy in multi-study dashboards.

visually / functionally, it should be more or less the exact same.

Is this the expected appearance of the participant selector now?  I know you said it was still under development, but want to to confirm I'm not missing something.  Logged in as proxyFamily2

Is this the expected appearance of the participant selector now? I know you said it was still under development, but want to to confirm I'm not missing something. Logged in as proxyFamily2

Oops, I am very silly. I was so focused on getting it functional that I forgot about internationalization.


Better error message than "No files loaded, GHCi is not working properly."?

If I run
ghcid --command=ghci

without a .ghci file then I get the error message
No files loaded, GHCi is not working properly.

Firstly, this is quite alarming. Secondly, I don't think it's correct. GHCi is working fine but it hasn't been correctly configured to work with ghcid. Would you be open to a different error message? I'm not sure I know ghcid well enough to be able to make a good suggestion, but maybe something like:
GHCi didn't load any files. Do you have a .ghci file? Make one
and put in it, for example

:load Path/To/Main.hs

(ghcid could even check for the existence of that .ghci file.)

The number of people using .ghci files is decreasing rapidly because Cabal doesn't support them very well, thus I don't think that for most people specifying ghci files is a good idea. Unfortunately, there is no blanket answer as for how to configure things properly, and I don't think there will be for a while. I tweaked the message to:

No files loaded, GHCi not configured properly.

But I don't have anything better than that...

The number of people using .ghci files is decreasing rapidly because Cabal doesn't support them very well, thus I don't think that for most people specifying ghci files is a good idea.

I see. Perhaps it's a particularity of my setup that the "ghci wrapper" I provide doesn't load a module.

I tweaked the message to:

No files loaded, GHCi not configured properly.

But I don't have anything better than that...

How about saying exactly what the problem is?  Something like "When ghcid runs the command that provides it with a GHCi session, that GHCi session must to load a module, but it didn't.".  Then at least people have something to work with.

No files were loaded by the supplied ghci command, which isn't allowed (no files to watch).

Thoughts?

Perhaps it's a particularity of my setup that the "ghci wrapper" I provide doesn't load a module.

I think most people use cabal repl or stack repl nowadays which might not load any modules, but usually due to errors in the setup rather than a missing .ghci :load entry.

No files were loaded by the supplied ghci command, which isn't allowed (no files to watch).

Thoughts?

Sounds good, though I'd probably suggest replacing "isn't allowed" to "means I can't do anything". It's not really forbidden it just doesn't do anything useful.

Perhaps it's a particularity of my setup that the "ghci wrapper" I provide doesn't load a module.

I think most people use cabal repl or stack repl nowadays which might not load any modules, but usually due to errors in the setup rather than a missing .ghci :load entry.

I see. Work has some Nix-based ghci wrapper and I guess it just doesn't load any modules unless I configure a .ghci file.

It isn't allowed and will cause ghcid to abort (technically ghcid could just sit their watching no files, but while it is the correct zero-case behaviour, it's not actually what anyone wants)

It isn't allowed and will cause ghcid to abort (technically ghcid could just sit their watching no files, but while it is the correct zero-case behaviour, it's not actually what anyone wants)

Sure, but the user's next question will be "why isn't it allowed?" and the answer is "because if you don't load any modules then ghcid isn't useful". I'd suggest avoiding having the user ask themselves that question but just answering it directly in the error message.
Anyway, any of the messages we've suggested in this thread are better than the status quo.

No files loaded, meaning ghcid will never refresh, so aborting.

Thoughts?

I like that one!

Done!


Point README.md at spec.md directly.

GitHub does not follow symlinks and as such, the GitHub landing page
has a broken link.

Good catch, are you covered by the CLA as the bot says? I would like to merge this ASAP.

LGTM if we can get the CLA issue resolved.

Hi, I am still trying to figure out the CLA issue. Let me get back to you. In the meantime, feel free to re-do the change yourselves to fix this. I suspect it will take me couple of days to get the CLA thing sorted.

Fixed by https://github.com/openservicebrokerapi/servicebroker/pull/141 - to avoid the CLA issue.


Running UI Tests Involving the Health App

Description
The Health App on the iOS simulator instance running in a GitHub Action Runner environment does not display any content in the "Browse" section, making it impossible to test adding external HealthKit data such as EEG data in a UI test.

Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 10.15
[ ] macOS 11
[X] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Image: macos-12
Version: 20221027.1
Included Software: https://github.com/actions/runner-images/blob/macOS-12/20221027.1/images/macos/macos-12-Readme.md
Image Release: https://github.com/actions/runner-images/releases/tag/macOS-12%2F20221027.1
Is it regression?
No
Expected behavior
You can reproduce the expected behavior on our local macOS-based machine:

Download and install Xcode 14.1
Reset the Simulator to get a completely clean instance
Run the UI Tests in this Repo by opening TestApp.xcodeproj.
The unit tests open the Health App and add Health Data using UI tests. You can observe the behavior in the iOS Simulator and see the Results in the Health App

Actual behavior
The actual behavior can be observed in the https://github.com/CardinalKit/HealthKitTest/ repo.

The GitHub Action Runners run the same UI tests using the GitHub Action found in .github/workflows: build_and_test.yml.
The tests fail as the HealthKit's "Browse" tab does not include any content, e.g.,: https://github.com/CardinalKit/HealthKitTest/actions/runs/3466876834.
The GitHub Action demonstrates this behavior as the UI tests create a screenshot on save and saves it in a .xcresult package.
You can download the results bundle as an artifact created by the UI test and observe its behavior in Xcode by opening the TestApp.xcresult.
The screenshot at the bottom of the README from the GitHub Action run in the https://github.com/CardinalKit/HealthKitTest/ repo demonstrates the behavior.

Repro steps
You can find a repository to reproduce the behavior including steps to follow at https://github.com/CardinalKit/HealthKitTest/.

Hey @PSchmiedmayer! We'll take a look.

@erik-bershel Thanks a lot! Let me know if you need any help or additional context! 👍

Hi @PSchmiedmayer, does view uses UIWebview or WKWebView or other View?

Thanks for taking a look at the issue @erik-bershel, @mikhailkoliada, and @igorboskovic3!
@igorboskovic3: I would not suspect that the Health App uses a UIWebview or WKWebView to display the "Browse" section of the app. But as the Health App is a core system app developed by Apple I don't have any insights there and Apple engineers might be in a better situation to answer that question.
I am not sure what you mean with "other View"?

@PSchmiedmayer I thought generally on anything else that im not aware of. We will try to reproduce your issue, thank you

@PSchmiedmayer
I do confirm the repro step "Run the UI Tests in this Repo by opening TestApp.xcodeproj" produces the successful test session. But if i try run the unit tests with the xcodebuild cli like the pipeline does it fails with the logs:
note: Building targets in dependency order
Testing started
Test suite 'UITests' started on 'Clone 1 of iPhone 14 Pro Max - UITests-Runner (67957)'
Test case 'UITests.testAddDataToHealthApp()' failed on 'Clone 1 of iPhone 14 Pro Max - UITests-Runner (67957)' (10.523 seconds)
2022-11-25 18:54:02.122 xcodebuild[67767:5825727] [MT] IDETestOperationsObserverDebug: 52.471 elapsed -- Testing started completed.
2022-11-25 18:54:02.122 xcodebuild[67767:5825727] [MT] IDETestOperationsObserverDebug: 0.000 sec, +0.000 sec -- start
2022-11-25 18:54:02.122 xcodebuild[67767:5825727] [MT] IDETestOperationsObserverDebug: 52.471 sec, +52.471 sec -- end

Test session results, code coverage, and logs:
	/Users/dsa/AndroidStudioProjects/HealthKitTest/TestAppUITests.xcresult

Failing tests:
	UITests.testAddDataToHealthApp()
	UITests.testAddDataToHealthApp()

** TEST FAILED **

From this i suggest the problem is not the action environment but with the building command. Namely i assume the lack of the capabilities and entitlements necessary to access the HealthKit data.
Particularly i do not see any mentions of com.apple.developer.healthkit in the build log nor i do not see it mentioned in the source code.
I am not sure about the setup for UI Tests of HealthKit and can only guess the XCode picks up the capabilities from some external provision files given with the auto-signing options but before we proceed with the investigation can you please confirm you can run the test with xcodebuild cli and provide us the repo and exact command options.
What about the empty browse screen i assume thee screen shout is done during the delay before the data appears - due to pure performance of macos build hosts it can be pretty big.

Hi @PSchmiedmayer, please pay attention the issue is inactive for several day days and it should be closed soon.

Hi @dsame & @igorboskovic3,
Thanks for further investigating the issue and taking a closer look!
Sorry for the late response, I have been out of the office during the Thanksgiving break and an now catching up with all the work that piled up.
Happy to help you with the issues you descried!
A bit more context: The test app that is included in the repo does not need to have access to the HealthKit data and does not need the HealthKit entitlement. The UI test that you can see at https://github.com/CardinalKit/HealthKitTest/blob/main/UITests/UITests.swift does never interact with the included iOS application. It immediately returns to the Home Screen (https://github.com/CardinalKit/HealthKitTest/blob/main/UITests/UITests.swift#L34) and then only interacts with the Health App. The project is a stripped-down version of the actual project that we have problems with (https://github.com/StanfordBDHG/CardinalKit) to make it easier for you to reproduce the error.
You can reproduce the builds and tests passing locally using the following xcodebuild CLI command, which is the same one that the GitHub Action at https://github.com/StanfordBDHG/.github/blob/main/.github/workflows/build-and-test-xcodebuild.yml uses with the inputs substituted with concrete values.
xcodebuild test \
  -project TestApp.xcodeproj \
  -scheme TestApp \
  -destination 'name=iPhone 14 Pro Max' \
  -enableCodeCoverage YES \
  -resultBundlePath TestApp.xcresult \
  CODE_SIGN_IDENTITY="" \
  CODE_SIGNING_REQUIRED=NO

You might need to remove the TestApp.xcresult file before the test execution if you ran the tests before.
I have demonstrated that the overall xodebuild command works with an even more reduced UI test that only launches our application and the Health Application on the simulator without interacting with the empty browse tab in the Apple Health App (that is the problem described above). You can find the run on a separate branch here: https://github.com/CardinalKit/HealthKitTest/tree/onlyLaunchApp and the successful GitHub Action run here: https://github.com/CardinalKit/HealthKitTest/actions/runs/3595497238/jobs/6055057980.
You can also reproduce that the test is indeed failing in the Apple Health App and is not a problem in the xcodebuild command when you inspect the .xcresult bundle that is the output of the build command as described in the issue above. E.g., you can download the .xcresult bundle here: https://github.com/CardinalKit/HealthKitTest/actions/runs/3466994428, open the .xcresult bundle locally, and then inspect the screenshot that is automatically made at the end of the UI test showing the empty browse section. You can also do the same thing in the simplified run above that stops after navigating to the browse tab in the Apple Health App.

Hi @PSchmiedmayer after investigation and trying to found workaround, we conclude that this is performance issue, since this action/test performing well on self-hosted runner,

sorry for inconvenience.

@igorboskovic3 Thank you for taking a closer look at this! We will further investigate this on our side and see if we can work with even longer timeouts to address the performance issues on the GitHub Action runners 👍


Show added by column and make it sortable

Use sharedSiteIds and sharedTypeIds to figure out the addedBy column
Add sorting to it

https://github.com/IABTechLab/uid2-self-serve-portal/assets/116539741/24f338ae-05c8-4f7f-b47d-558a1d56991b

https://github.com/IABTechLab/uid2-self-serve-portal/assets/116539741/0c53d55b-b1ca-44bc-8d7a-3c03d46fd921

Show source ('Manually Added', 'DSP') rather than Added By ('Manual', 'Auto')
Show notice when user is trying to delete a site which is shared by Manually added and types

The gap between search & add and the sharing permissions table looks a bit big now, I think it was from my PR.

Could you please remove margin-top: 30px from .sharing-permissions-table in SharingPermissionsTable.scss?
thanks


ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: YES)

Hi, thanks for a great product!
After updating docker <IP_ADDRESS>, I had to restart all services, but after restarting I now cannot login to my mysql.
Error I get ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: YES)
I already tried null password, "password" as password etc. and nothing helps.
Any ideas?
Thanks!

A few questions:

Where are you seeing this error? Is this command line MySQL, or your PHP application?
Have you tried disabling the mysql service and re-enabling it so it's generated by the latest version of Takeout?

Thanks!

Thanks for a quick response.

I see it when running docker exec -it e509c07994b1 mysql -p or through "Sequel Ace" or via PHP application - all three worked before
I have tried it yes multiple times and also updated takeout to the latest version

Just letting you know that the issue was that my mysql_data volume has been created with almost the first version of takeout and so when you introduced this change Implemented mysql_native_password native authentication plugin for all MySQL containers it messed things up, because the old volume still had old info.
Once I removed the mysql_data volume docker volume rm mysql_data (thankfully I didn't have anything important there), it all works again after enabling mysql via takeout.

Ahhh.. that's helpful to know, I didn't know that would break things. Thanks!

@mattstauffer Just letting you know that I experienced this again. The only thing that happened was that my docker was updated so all my containers were stopped. I then had to takeout enable mysql again and it bumped the mysql version to 8.0.22 - all went fine. But when trying to connect via console docker exec -it 2a831e763c7d mysql (or otherwise), I again get the error ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: NO).
Any ideas how to approach getting to the data in mysql_data volume?
Thanks!

@aureliusm Apologies for taking so long, the end of last year was really tough and busy. Is this still an issue?

@aureliusm Apologies for taking so long, the end of last year was really tough and busy. Is this still an issue?

Yes it is.


Everything in not running while it is installed

Microsoft PowerToys version
0.86
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce

Install Powertoys on Windows store.
According to Everything's installation wiki page, method 3 or 4 (I've tried both methods).
Run Powertoys Run.

✔ Expected Behavior
Run Everything normally.
❌ Actual Behavior
Powertoys Runs gives the message as follows.

while it shows that Everything is enabled.

Also, it can be found at the Plugins directory.

By the way, Powertoys runs as administrator.

I've seen #34230, and it doesn't help.
Other Software
No response

After reinstall Everything 1.4 and the plugin EverythingPowerToys, it now works fine.


Update sensor_wrapper.py

syntax error in python3

No idea but it def does. Try running that file you will see.
On Fri, Jul 23, 2021, 12:27 PM Jacob Floyd @.***> wrote:

@.**** commented on this pull request.
In st2reactor/st2reactor/container/sensor_wrapper.py
https://github.com/StackStorm/st2/pull/5307#discussion_r675689675:

@@ -435,7 +435,6 @@ def _sanitize_trigger(self, trigger):
if not isinstance(parent_args, list):
raise TypeError(
"Command line arguments passed to the parent process must be a list"

       f" (was {type(parent_args)})."

Why does this cause a syntax error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StackStorm/st2/pull/5307#pullrequestreview-713893908,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACZ5TIK723AKVTOQD6EJGSLTZGJ75ANCNFSM5AWH5NRA
.

Ah I am betting the bin still has python in it. Instead of python 3
On Fri, Jul 23, 2021, 1:28 PM Amanda McGuinness @.***>
wrote:

@.**** commented on this pull request.
In st2reactor/st2reactor/container/sensor_wrapper.py
https://github.com/StackStorm/st2/pull/5307#discussion_r675723703:

@@ -435,7 +435,6 @@ def _sanitize_trigger(self, trigger):
if not isinstance(parent_args, list):
raise TypeError(
"Command line arguments passed to the parent process must be a list"

       f" (was {type(parent_args)})."

Also in the original PR change, it was requested that as part of the error
it reports the type of the argument that isn't a list, so that it enables
easier debugging. So even if it needs changing we still need that type in
there.
It's odd - as I can run that block of code lines 435-438 perfectly fine.
Are we definitely sure its being run in a python 3.6 or python 3.8
environment?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StackStorm/st2/pull/5307#discussion_r675723703, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACZ5TILQZASTMJPKQWI6PCDTZGRCRANCNFSM5AWH5NRA
.

Nope. I checked and the bin is correct. it has python3.  I have completely abandoned sensors anyhow due to them failing after 2 tries and not saving their state anywhere to be queried.  I am guessing it was something haywire on our installation.


Fix span handling

Fixes #20

Codecov Report

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

@@            Coverage Diff            @@
##             master      #21   +/-   ##
=========================================
  Coverage          ?   85.07%
=========================================
  Files             ?        2
  Lines             ?       67
  Branches          ?        0
=========================================
  Hits              ?       57
  Misses            ?       10
  Partials          ?        0

Continue to review full report at Codecov.

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


The diff coverage is 38.46%.

Flag
Coverage Δ

patrol
11.61% <38.46%> (+0.31%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/patrol/lib/src/binding.dart
0.00% <0.00%> (ø)

packages/patrol/lib/src/host/host_automator.dart
5.88% <0.00%> (-0.37%)
:arrow_down:

...ckages/patrol/lib/src/native/native_automator.dart
0.59% <0.00%> (+0.01%)
:arrow_up:

...s/patrol/lib/src/custom_finders/patrol_finder.dart
86.27% <71.42%> (+0.85%)
:arrow_up:

Continue to review full report at Codecov.

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


”Ramna” is listed in the drop-down menu

Description:
"Ramna" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Ramna" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Ramna, CARAȘ-SEVERIN".


How can tags be used in Explorer?

How can tags be used in explorer for projects and orgs, when editing or displaying them?
Should we also have a tag system to annotate resources in the UI?
connects #39

From the backend perspective tags are not implemented on orgs and projects. However it would be included on the following iterations (mid November).

What would it mean to tag an organization?
Also, is it possible to change the revision which a tag points to?


Datasets table doesn't report backend for non-classification databases

I have one classification DB and several DBs of type "Other". The backend column reports "lmdb" for the classification DB, but is empty for all the other DBs.
I also tried to create a new one (all those DBs were created before upgrading), and even the new one don't report anything as backend.

(Comes from https://github.com/NVIDIA/DIGITS/pull/728)
Good point. I never got around to setting the backend for Generic datasets because LMDB is the only backend supported. And I'm actually finding it convenient at the moment because it gives me a way to distinguish Classification datasets from Generic ones. But you're right - we should display them as LMDB and find some other way to distinguish between dataset types.

@jmancewicz you were working on adding data extensions to the home page for sorting, right? Any update?

@lukeyeager, I just created a PR for it #1246


Fix casting issue preventing compile in mac with oracle jdk.

Java is a platform independent programming language. SpongeAPI, an API, is both platform and implementation independent. Any Java code that compiles on one OS should compile on any other OS. If this does not work on Mac, then it is Apple's or Oracle's bug, not Sponge's. Please report this bug to them instead. This change only makes the code less organized for non-Mac users.

@gabizou you use a mac and I assume Oracle's JDK on it...do you have this problem?

I've been working on Mac OS X for the entire lifetime of this project, I do know that older variants of JDK 8 have had issues with lambdas and method references, so if you can, please update your JDK and report back.

I have gotten this compile error through IntelliJ but never through gradle itself.

I have gotten this compile error through IntelliJ but never through gradle itself.

Which JDK did you give IntelliJ?

Previously 1.8.0_20, I've updated to 1.8.0_71 and the issue appears to be resolved.

Yeah, it's mostly due to old JDK compilers having their bugs that have been fixed for a very long time now. Update your JDK and it should be fine.


Not able to update stabline inactive color

require('stabline').setup {
  defaults = {
    style = "bar",
    stab_left = "‚îÉ",
    stab_right = " ",
    inactive_bg = "#1e2127",
    inactive_fg = "#eeeeee", -- This does not update the foreground
    font_active = "bold",
    exclude_fts = {'NvimTree', 'dashboard', 'lir'},
    stab_start = "",
    stab_end = ""
  }
}

Could you please help me fix this?

Heyyo @hackorum ,
It seems the readme was wrong this whole time.
Stabline does not have a nested defaults table, instead all items go inside setup directly.
In your case, it would be:
require('stabline').setup {
    style = "bar",
    stab_left = "‚îÉ",
    stab_right = " ",
    inactive_bg = "#1e2127",
    inactive_fg = "#eeeeee", -- This does not update the foreground
    font_active = "bold",
    exclude_fts = {'NvimTree', 'dashboard', 'lir'},
    stab_start = "",
    stab_end = ""
}

Will change the readme asap, and sorry again 😅

Oh, ok. Thank you :)


Pass variables to Synthetics from workflow

Would it be possible to pass a list of variables to Synthetics from the workflow without having to define a config file?
Thx

+1

+1

Hi folks, I'm one of the Synthetics Product Managers 👋🏻
Just a couple of follow up questions:

Do you want to override variables on a per test basis, or is this globally overriding global variables?

For example, is this just for the global config file, or the test files themselves?

cc/ @Hraph @piscue @GxDesign

my use case is global

My use case is also global. Im getting back a deployment preview url in an earlier step from Netlify and would like to pass that as a startUrl variable for all tests.

Hello there! Could you have a quick look at the following PR and tell me if that would solve your use cases? 🙂

https://github.com/DataDog/synthetics-ci-github-action/pull/79

This would work for me! Thanks!
On Mon, Jul 11, 2022 at 3:42 AM Benoit Ruiz @.***>
wrote:

Hello there! Could you have a quick look at the following PR and tell me
if that would solve your use cases? 🙂

#79 https://github.com/DataDog/synthetics-ci-github-action/pull/79

—
Reply to this email directly, view it on GitHub
https://github.com/DataDog/synthetics-ci-github-action/issues/74#issuecomment-1180125418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGNDJGHJNHMLTCIQETW44TVTPM6LANCNFSM5UOX75LQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

This should be available as of v0.4.1 🚀


Add 1K, 5K and 10K RayCluster/RayJob scalability test results

Why are these changes needed?
Per https://github.com/ray-project/kuberay/issues/2069, adds 1K and 5K RayCluster / RayJob test results
Related issue number
https://github.com/ray-project/kuberay/issues/2069
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

@kevin85421

https://github.com/kubernetes/perf-tests/tree/master/clusterloader2#measurement

Great! Would you mind adding a README to briefly document the benchmark results we have? Thanks!

I updated the README with some references on how to understand the results

Btw, what's the configuration of KubeRay (e.g. CPUs? memory? reconcile-concurrency)? Thanks!

We used 16 CPU requests and 32GiB memory limit with --reconcile-concurrency=5. https://github.com/ray-project/kuberay/pull/2228 was an important fix for the RayJob scalability tests.
In practice we didn't need that much. This is the CPU / memory graphs from the most recent run:


Encoding issue for emoji?

Describe the bug
When running mkslides serve ... I have run into an error when the file contains any emoji, e.g., :tada:, UnicodeEncodeError: 'charmap' codec can't encode character ....
Adding encoding='utf-8' here and  here seems to resolve the issue. Happy to make a PR if that's the correct fix.
Expected behavior
No error.
Minimal reproducable example:
It's time to :tada:

Version (please complete the following information):
1.0.0 (latest from pip)

Hi @compressed , could you post the complete output?
If you want to make a PR, please go ahead! it the tests pass, I'll merge it and create a patch release :+1:

Further research seems to indicate that your solutions seems to be the best way forward: https://codereview.doctor/features/python/best-practice/file-open-write-encoding

Version 1.0.1 should solve this.

Thank you so much! Works now!


Don't forget templating system (twig and smarty)

Some web project use templating system as smarty or twig

Thanks!


Investigate gene-ex warning during Canto loading

can't handle data of type PomGeneExRNA
can't handle data of type PomGeneExProt

I've fixed the warnings by fixing the 11 annotations that had the wrong types.  I'm confused as to how the types ended up being wrong so I'll keep working on that.  I'll leave this open for now.
If you see any similar warnings in the Canto loading log file please let me know.


--write-thumbnail --skip-download dosn't work with --exec?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

I just want to download the thumbnail of a video and then process it. But it seems that the --exec function does not work if you use the --skip-download option. Is that correct?
christoph@srv:~$ youtube-dl --verbose --write-thumbnail --restrict-filenames https://www.youtube.com/watch?v=BaW_jenozKc --exec 'ls -lh {}'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'--write-thumbnail', u'--restrict-filenames', u'https://www.youtube.com/watch?v=BaW_jenozKc', u'--exec', u'ls -lh {}']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.15+ (CPython) - Linux-4.15.0-62-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6
[debug] Proxy map: {}
[youtube] BaW_jenozKc: Downloading webpage
[youtube] BaW_jenozKc: Downloading video info webpage
[debug] Default format spec: bestvideo+bestaudio/best
[youtube] BaW_jenozKc: Downloading thumbnail ...
[youtube] BaW_jenozKc: Writing thumbnail to: youtube-dl_test_video_a-BaW_jenozKc.jpg
[download] youtube-dl_test_video_a-BaW_jenozKc.mp4 has already been downloaded and merged
[exec] Executing command: ls -lh youtube-dl_test_video_a-BaW_jenozKc.mp4
-rw-rw-r-- 1 christoph christoph 2,3M Dez 25  2013 youtube-dl_test_video_a-BaW_jenozKc.mp4
christoph@srv:~$ youtube-dl --verbose --write-thumbnail --skip-download --restrict-filenames https://www.youtube.com/watch?v=BaW_jenozKc --exec 'ls -lh {}'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'--write-thumbnail', u'--skip-download', u'--restrict-filenames', u'https://www.youtube.com/watch?v=BaW_jenozKc', u'--exec', u'ls -lh {}']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.15+ (CPython) - Linux-4.15.0-62-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6
[debug] Proxy map: {}
[youtube] BaW_jenozKc: Downloading webpage
[youtube] BaW_jenozKc: Downloading video info webpage
[debug] Default format spec: bestvideo+bestaudio/best
[youtube] BaW_jenozKc: Downloading thumbnail ...
[youtube] BaW_jenozKc: Writing thumbnail to: youtube-dl_test_video_a-BaW_jenozKc.jpg

This is expected behavior. There is no file downloaded to run --exec on.

Thanks,
but: "youtube-dl_test_video_a-BaW_jenozKc.jpg" was downloadet. Is there a chance to interact with the thumbnail file?
-- Chris


License type?

Good evening,
what kind of license is applied on this Repo?
I found on the internet No License is applied on git repos by default, but this licese is quite restricted.
So please provide license of your repo. In my opinion ICS or MIT would be fine.
Thank you

Sure I'll change it to MIT. Use it how you want, and let me know how it helps!


[Feature] Add Area Calculator

Description
Here we have many calculator in our website. Now we are going to add Area calculator. Area Calculator will be used to calculate area based on the shapes. It will be helpful for school Students who can verify their answers. I can able to work on this so kindly assign me this issue @Rakesh9100
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

We already have 2D Shapes Calculator for the same @GnanariddhikaRavikumar

Ok sir
On Mon, May 27, 2024, 8:01 PM Rakesh Roshan @.***>
wrote:

We already have 2D Shapes Calculator for the same @GnanariddhikaRavikumar
https://github.com/GnanariddhikaRavikumar
—
Reply to this email directly, view it on GitHub
https://github.com/Rakesh9100/CalcDiverse/issues/843#issuecomment-2133604744,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A54NHIH5ND7BK6J6M5TO44TZEM7V3AVCNFSM6AAAAABIHRLD5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZTGYYDINZUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix #462 Calls API support

Summary
This pull request fixes #462 by adding five calls.* API methods. call type block, and call_rejected event.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

This pull request is ready for review. I will be waiting for comments (from anyone!) by the end of May 26, Pacific Time. Then, I'll merge this and release a new version next week.


Emoji are not displayed correctly in Godot Editor online and when exporting to HTML

Tested versions
4.3
System information
windows 11 - Godot 4.3
Issue description
Emoji are not displayed correctly in Godot Editor online and when exporting to HTML
Emoji copied to a project from https://emojidb.org/ are displayed correctly in a local project and when running on Windows, but when exporting to HTML or Godot Editor online they break.
Steps to reproduce
Steps for repoduce

Add label to scene
Copy and past some emoji from https://emojidb.org/health-emojis
For example emoji:

Значки:
Игра: 🧬
Меню:⚙📶💲🛍🧺🎒🥋🏋♂🌍🗺📗📖📚🧾📜🪞🔭🎁📦🎣⚛🕸⚖🧑🏻💻💻🚧☢
Действия: ⚠⛔✅🆔🆙🔄↪🔙➕✔♾⁉❓♻🗑⏳🕒⏲⌚👁✏🪶📸🔎🔍🏳🏴🔧🗝🔒📁💾🔗💊🎁🖕
Персонажи:👤🕵🏼♀👷🏻♀👩🏼⚕🧟♂🥷🏽🧛🧙🏼♂🧙🏼👩🎓
Характеристики: 💎⭐🌟🪙❤💓💖💘❤🔥💙💚💚💛🖤🧠🔮🗡⚔🪄👟🏹🎯💪🏽🛡🔥⚡❄💧🧊🍃👊🌈🍀🔦🪐🔷☔
Эффекты: 🤕💣💢🌀💥💡💬💞💋🫧🧬💀☠☠🦴⚕👑💤🎲🧩⚕🌊🕶🌓🕸✡⚓🦠☢💯☯
Предметы: 🛠⚒🪛🔧🔨🪓⛏⛏🗡⚔🔱🪃🏹🔫🪖⛑🩲
Враги: 🐈🐕🦀🐶🦎☃👻👾🐲🦇🐼🐪🦬🐥🐝🐺🛸🦕🦑🦈🐨
Земля: 🌳🌲🌵🌴🌱🌿🏺🏔🧱🗝🗿⛲
Здания: 🏚⛩⛪🏰🛕🏨🏦🎪🚃
Расходники:🎁🧪🍽🎂🍍🥯🌽☕🧉🍫🥖🍿🍗🍑🍈🥩🍎🍺🍄

Godot Editor Online

Open Godot Editor Online
Create new project
Repeat steps 1-3 for reproduce

Export project local as HTML and start

Local create project
Repeat steps 1-3 for reproduce
Export project as html
In the folder with game open terminal and start comand npx serve . - start local server with index.html

See on itch.io

Repeat steps 1-3 for reproduce and export on itch.io
Run game on itch.io.
Temporary link with my project (until I find a way to fix emoji) https://milis-frontend.itch.io/gen-demo and github with gane

Minimal reproduction project (MRP)

project on itch.io https://milis-frontend.itch.io/gen-demo (maybe fix later)
my project https://github.com/Lyumih/gen_godot
gen-export.zip - archive for deploy on itch.io
extract gen-export.zip localy and start comand npx serve . in terminal folder for locali start server html

Archive:
gen-export.zip

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/84590, as this is due to the same cause.


[DS-3475] added back assetstore.dir configuration to dspace.cfg

https://jira.duraspace.org/browse/DS-3475

Thinking about this some more:  when everything else about assetstores is configured in config/spring/api/bitstore.xml, why do we have this one datum in dspace.cfg?  Should not the test environment ensure that a valid path is given?  Should not the affected tests create the given path, if necessary?


wip: expose consumer group lag as prometheus metrics

Is there a reason why this PR died? We are currently using Burrow to monitor lag and partition-status, and we do want to monitor them using Prometheus. I can see that it is a bit outdated, but I'm curious if there is a reason why it just died. @toddpalino @JamieAP

Not sure why this dropped out, but it probably needs a do-over for 1.0 anyways. The one caveat I would say on that is that if the write to prometheus is straight HTTP, it should probably be implemented using the HTTP notifier and an example template and config provided (which can be added to the documentation).
We do something like this internally with metrics, but I haven't published a sample template yet.

Yeah, I am also looking for prom metrics in burrow.

There is some existing work too that maybe we can borrow for Burrow in the burrow_exporter
Other projects like Kubernetes, etcd, and Docker utilize Prometheus instrumentation. Also, organizations like Cloudflare who use both Kafka and Prometheus like ours are not an isolated intersection.  There is a formidable and active operator community behind Prometheus as well.
Anyway...
jirwin's exporter has these metrics:
KafkaConsumerPartitionCurrentOffset
KafkaConsumerPartitionMaxOffset
KafkaConsumerTotalLag

Also, it would be nice to expose operational metrics, most places where there's a logging statement could use a metric. @JamieAP  are you planning on shoring this PR up for 1.0 by chance?

Don't want to ruin holiday week, but if any help needed for this PR, I can find some time. Will really appreciate if this get merged sooner than later.


Compile error(v7.6.0): duplicate declaration in messages_en.dart

String m0 ~ m5 is duplicated in below.
https://github.com/danvick/flutter_form_builder/commit/b9da73472d5a3e0dfece266da1156ab7718d1a5e#diff-ddf44741a31a8c3c59bcc3f472df812dd93c0b5f30dbbd0248c618be365fe3eaR25
Error message:
: Error: 'm0' is already declared in this scope.
../…/intl/messages_en.dart:25
  static String m0(value) => "This field value must be equal to ${value}.";
                ^^
: Context: Previous declaration of 'm0'.
../…/intl/messages_en.dart:23
  static String m0(value) => "This field value must be equal to ${value}.";
                ^^

: Error: 'm1' is already declared in this scope.
../…/intl/messages_en.dart:29
  static String m1(max) => "Value must be less than or equal to ${max}";
                ^^
: Context: Previous declaration of 'm1'.
../…/intl/messages_en.dart:27
  static String m1(max) => "Value must be less than or equal to ${max}";
                ^^

: Error: 'm2' is already declared in this scope.
../…/intl/messages_en.dart:34
  static String m2(maxLength) =>
                ^^
: Context: Previous declaration of 'm2'.
../…/intl/messages_en.dart:31
  static String m2(maxLength) =>
                ^^
: Error: 'm3' is already declared in this scope.
../…/intl/messages_en.dart:39
  static String m3(min) => "Value must be greater than or equal to ${min}.";

                ^^
: Context: Previous declaration of 'm3'.
../…/intl/messages_en.dart:37
  static String m3(min) => "Value must be greater than or equal to ${min}.";
                ^^
: Error: 'm4' is already declared in this scope.
../…/intl/messages_en.dart:44
  static String m4(minLength) =>
                ^^
: Context: Previous declaration of 'm4'.
../…/intl/messages_en.dart:41
  static String m4(minLength) =>
                ^^
: Error: 'm5' is already declared in this scope.
../…/intl/messages_en.dart:49
  static String m5(value) => "This field value must not be equal to ${value}.";
                ^^
: Context: Previous declaration of 'm5'.
../…/intl/messages_en.dart:47
  static String m5(value) => "This field value must not be equal to ${value}.";
                ^^

: Error: Can't use 'm0' because it is declared more than once.
../…/intl/messages_en.dart:59
        "equalErrorText": m0,
                          ^
: Error: Can't use 'm1' because it is declared more than once.
../…/intl/messages_en.dart:66
        "maxErrorText": m1,
                        ^
: Error: Can't use 'm2' because it is declared more than once.
../…/intl/messages_en.dart:67
        "maxLengthErrorText": m2,
                              ^

: Error: Can't use 'm3' because it is declared more than once.
../…/intl/messages_en.dart:68
        "minErrorText": m3,
                        ^

: Error: Can't use 'm4' because it is declared more than once.
../…/intl/messages_en.dart:69
        "minLengthErrorText": m4,
                              ^

@alexaung Downgrade form_builder_validators to 7.5.0 can resolve it.

@alexaung Downgrade form_builder_validators to 7.5.0 can resolve it.

Not really a long-term resolution for you have to pin it rather than being relative.

@MasterHiei Thanks. Downgrade solved this issue as for now.

For some reason, the generated file in .../intl/messages_en.dart repeats the declaration of the m* variables. If you want to keep v7.6.0, you can manually uncomment the duplicates. Hopefully, this is fixed soon
static String m0(value) => "This field value must be equal to ${value}.";

  // static String m0(value) => "This field value must be equal to ${value}.";

  static String m1(max) => "Value must be less than or equal to ${max}";

  // static String m1(max) => "Value must be less than or equal to ${max}";

  static String m2(maxLength) =>
      "Value must have a length less than or equal to ${maxLength}";

  // static String m2(maxLength) =>
      // "Value must have a length less than or equal to ${maxLength}";

  static String m3(min) => "Value must be greater than or equal to ${min}.";

  // static String m3(min) => "Value must be greater than or equal to ${min}.";

  static String m4(minLength) =>
      "Value must have a length greater than or equal to ${minLength}";

  // static String m4(minLength) =>
  //     "Value must have a length greater than or equal to ${minLength}";

  // static String m5(value) => "This field value must not be equal to ${value}.";

  static String m5(value) => "This field value must not be equal to ${value}.";


[feature-request] Allow domain to be specified as a function

This would allow users to implement special handling of X-Frame-Options in the case that they want to support many different domains.  I was thinking that the api could be something along the lines of a function that returns a string.  If a string is returned set allow-from to be that string, otherwise if null is returned DENY.  I'm happy to do a pull request for this if people think it is a good idea.  I feel like #6 could simply be implemented as a special case of this.

Can you give me an example of a function you'd like to use?

Sure, I was thinking about something along the lines of:
function(req) {
  if (/.*foo\.com/.test(req.hostname)) {
    return req.hostname;
  }
}

This would essentially allow the app to be embedded into any page matching the regex, ie bar.foo.com or baz.foo.com, but not any random site.  Anyone consuming this would need to be very careful not to cache the response since the response would be the same but might have different headers depending on where the request was made from.

True, but req.hostname will be your domain, not the domain of the parent page. See my comment here.

@EvanHahn Oops, you're right.  I think req.headers.referer would do the trick though.

If I'm not mistaken, the Referer header isn't always to be trusted, especially with some of these new referrer control policies that browsers are implementing.
That being said, if a developer wants to trust the Referer header despite its inconsistencies, this feature request would allow that.
What do you think?

@EvanHahn I agree.  It's possible that the referer header might not be present / be trusted, but this would leave it up to the developer to decide if this is acceptable or not.

Yeah, we totally 'need' this if you want to allow your application to be embedded within multiple sites.
For other visitors who might not immediately understand why This article here https://blogs.msdn.microsoft.com/ieinternals/2010/03/30/combating-clickjacking-with-x-frame-options/ gives the reasoning behind it (the example used there is to pass a query parameter to the app to allow a 'pre-agreed' set of values to be checked off against)
However given the lack of browser support for 'ALLOW-FROM'  I totally understand why this might not be prioritised ;)

It's got mixed browser support, but Content Security Policy's frame-ancestors directive might be helpful here. Helmet's Content Security Policy middleware can help with CSP if that's a route you take.
If you want to have multiple ALLOW-FROMs with this module, I can only think of one way to do it, and it involves tokens. Here's how it might work.
Let's say example.com is trying to put one of my pages, evanhahn.com/iframe, in an iframe.
example.com might have the following HTML:
<iframe src="https://evanhahn.com/iframe?domain=https%3A%2F%2Fevanhahn.com"></iframe>

evanhahn.com might have the following server-side code:
var ALLOWED_BY = new Set([
  'http://evanhahn.com',
  'https://evanhahn.com',
  // ...
])

app.get('/iframe', function (req, res, next) {
  if (ALLOWED_BY.has(req.query.domain)) {
    frameguard({
      action: 'allow-from',
      domain: String(req.query.domain)
    })(req, res, function (err) {
      if (err) { return next(err) }

      // send the iframe HTML here...
      res.sendFile('/path/to/iframe.html')
    })
  } else {
    // They didn't provide a domain, so let's do nothing.
    // Why send the iframe if they're not authenticated?
    res.status(403)
    res.end()
  }
})

You can do this with Frameguard today. (To be fair, I didn't test the code above, so it may not work.)
We could build this functionality into Frameguard, but it probably wouldn't be as powerful as the above. For example, we couldn't elegantly handle the else condition from above.
What do people think? Is this something we should add to this module?

interesting, I didn't think of calling it directly per request (rather than at setup-time) that would suffice for me ;)

If that suffices for everyone, I'll close this issue. If we think Frameguard should support this somehow, I'll leave it open to discuss.

above solution is still a pain when you just want to use app.use(helmet.frameguard({...})) for any requests. still would be good to have support for the initial ticket issue.

@binarykitchen Would something like this work?
var ALLOWED_BY = new Set([
  'http://evanhahn.com',
  'https://evanhahn.com',
  // ...
])

app.use(function (req, res, next) {
  if (ALLOWED_BY.has(req.query.domain)) {
    frameguard({
      action: 'allow-from',
      domain: String(req.query.domain)
    })(req, res, next)
  } else {
    next()
  }
})

Or would you prefer this to be built into Frameguard?

@EvanHahn sorry for late response - yes, better build this into frameguard. make it simple for the programmer to use. something KISS like
app.use(helmet.frameguard({
    action: 'allow-from',
    domains: ['https://staticxx.facebook.com', 'http://letsencrypt.com']
}))

I think the apparent ease of the code snippet you posted is outweighed by the complexity of what's going on, and the code snippet I posted is more straightforward.
My solution seems like it'd work, but I feel weird adding something so nonstandard to this module, especially when the code snippet above is relatively brief. "Consumers" of your iframes will need to set a query string parameter, which feels strange.
Thoughts?

well i leave it to you @EvanHahn to make the final decision. i don't see any disadvantages with my suggestion because it clearly says what it wants and leaves the rest to the black box to solve internally.
just saying that the API of libraries like these should be kept as simple as possible, optimised for the processor in our skulls. we developers already have a too high cognitive load (i can show you slides about that). so again, keep it as simple as possible if you can ...

I'll add something to the readme and then close this issue.

Added something to the readme about this. Closing.

@EvanHahn thanks!