attempt at map attribution

Really no idea why this isn't working. None of the deck.gl examples are up to date with the latest react-map-gl 😕
Ref:

AttributionControl docs

Closes #13

@kylebarron the attribution issue seems to be related to the missing Maplibre CSS styles file. When inspecting the page, it is possible to see the attribution elements in the DOM, but no styles are applied to them and they end up positioned outside the widget viewport. I tried to import the CSS like instructed in the docs, but it didn't work. I'll keep looking.

@vgeorge it turns out that it's related to the heights of the multiple divs. So if I change to
      <DeckGL
        initialViewState={
          ["longitude", "latitude", "zoom"].every((key) =>
            Object.keys(viewState).includes(key)
          )
            ? viewState
            : DEFAULT_INITIAL_VIEW_STATE
        }
        controller={true}
        layers={layers}
+       height={300}
        // ContextProvider={MapContext.Provider}
      >

Then the map inside the main div is smaller and we can see the attribution below it (but weirdly in a drop down)

@kylebarron I was able to see the attribution elements after following your instruction of adding the height to the DeckGL component. But the Maplibre CSS module is not imported, it seems to me the dropdown is a result of missing CSS styles. Maybe something wrong in build.mjs?

But the Maplibre CSS module is not imported

I'm not good at checking this from the browser console; https://github.com/developmentseed/lonboard/pull/76/commits/17dc0376fa08a62480dea8ae07848cac244f75b5#diff-c2f8ec75103ae661341f239c22352142d1d5e965cad55b19cfb978902500106cR9 didn't fix it?
Alternatively we could build our own CSS bundle that we dynamically load through the ESM endpoint. You can pass css either inline as a string or as a url in _css in the anywidget object: https://anywidget.dev/en/getting-started/

Thanks @vgeorge for your help! 🙏

The map attribution still isn't clickable, but that's a separate longstanding deck/react-map-gl issue that I don't know what the current solution is https://github.com/visgl/deck.gl/issues/4165


[bwserver] Multi-client handling does not scale

Currently it seems the bwserver is not handling properly more than ~5 clients trying to perform the test at the same time.
Please note this is a very draft bug report without much reliable data. It is based only on the experiment of trying to run 50 instances of bwclient which failed very fast.
If someone decides to jump on it, more specific data can be regenerated and provided.

If I'm interpreting this correctly, the bwserver is actually designed to only handle one client at a time:
https://github.com/netsec-ethz/scion-apps/blob/850e1e3be287912dc324573c872f1362dfe82f15/bwtester/bwtestserver/bwtestserver.go#L199-L231
If it handles 5, then it's not working correctly ;)

As described above, bwtester intentionally handles only one client at a time.


Create a new release

Since I added support for Python 3.7 in #219 it would be good to upload a new release of the code to PyPi

Likewise with the security updates including #221

Thanks for the prompt.

@oadams thanks for that!


Doesn't seem to be working for me

I have HK <IP_ADDRESS>33. I installed the dependency mods (ItemChanger and SFCore) with Scarab reinstalled this mod from here on github, and I still can't seem to get it to work at all. is there something im doing wrong? every other mod ive installed through scarab seems to be working fine

What do you specifically mean by "not working"? Can you show me the mod list (on the top left corner of the pause menu) and your ModLog (found in your saves folder)?

I'll get the screenshots tomorrow when i can (its 11:22pm for me atm) but i have the mod and its dependencies downloaded. what i mean by not working is that the charms simply arent where they're meant to be. i have the extra empty charm slots (where the charms should go in my inventory) but they dont appear in the game for me to pick up

didn't mean to close that. ill get screenshots when i can. i have it all downloaded tho. what i mean by not working is they arent ingame. the slots where they will go after obtaining are there in my inventory (the extra empty spaces for charms) but they aren't in the game for me to obtain

I should also ask, are you playing with or without the randomizer? The way the charms are placed changes depending on that.

without. i saw that closed issue about it being bugged but i made sure to install the most recent version so i assume it shouldn't require that anymore. i can try with the randomiser installed (but off) later if you would like

You may have discovered this in the meantime, but the likely reason you didn't see the charms is because you were playing on a save file created before you installed the mod - they will only appear on new saves.


Feedback: Third-Party Tools Supported by Cockroach Labs - add TablePlus

Page: https://cockroachlabs.com/docs/v22.2/third-party-database-tools.html
What is the reason for your feedback?
[] Missing the information I need
[ ] Too complicated
[ ] Out of date
[ ] Something is broken
[X] Other
Additional details
Consider adding TablePlus to the list of officially-supported graphical user interfaces.  Of course, as doc writers, you are the messenger, so this may be out of your control.  (I'll open a feature request.)
TablePlus has had native CockroachDB support for a long time.  I have used it extensively and it is a mature product.
https://tableplus.com

Ref. feature request CRDB-24557 "Consider adding TablePlus to the list of Third-Party Tools Supported by Cockroach Labs"


Feature/Personalized templates and tokens

Please split templates and tokens and then come back with another PR.

@Magnus-Kuhn you can use backbone version 6.6.0-alpha.1 which implements personalized tokens.


[Second attempt] php74: init at 7.4.0

Motivation for this change
Because of a lot of the mess in #73151, I decided to create my own attempt to package php74 up.
So this is my attempt.
I have taken the minimalist approach. I only changed things that needs to be changed. So only upgraded pecls of things that needed upgrades to work. Added asserts to pecls where upgrades doesn't help etc.
I have built every pecl by hand while testing.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-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] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @aanderse @talyz @Izorkin

Seems like the following package fails on darwin:
@GrahamcOfBorg build php72Packages.pdo_sqlsrv php73Packages.pdo_sqlsrv php74Packages.pdo_sqlsrv
I just want to see how it behaved on other versions before as well.

Ok, phpPackages.pdo_sqlsrv seems to be broken since before. So this isn't a new problem caused by this PR and there's an assert in place for PHP 7.4 builds of this package. So there's no new breakage here so that shouldn't be a blocker for this PR.

@jtojnar look good to you?

@GrahamcOfBorg build php72Packages.pdo_sqlsrv php73Packages.pdo_sqlsrv
These build fine locally.

@aanderse @talyz The release announcement haven't been published yet. But they seem to have an uploaded the tarball.
This is not uncommon for PHP :slightly_smiling_face:
But we should wait for the announcement and see that they don't change the tarball before the release is announced. But we can test further until then.

PHP 7.4.0 is now released.
The sha256 of the tarball is the same as yesterday.

Anyone object to a merge at this point?

Not fixed this warning?
configure: WARNING: unrecognized options: --with-pcre-regex, --with-libxml-dir, --with-gd, --with-freetype-dir, --with-png-dir, --with-jpeg-dir, --enable-zip, --with-libzip
checking for grep that handles long lines and -e... /nix/store/20b535jb98hy7k4z8vkrlkjma212a3l5-gnugrep-3.3/bin/grep

@Izorkin I've resolved that in a4cf20c after your comment.

Thanks for being awesome @etu!


Enable Sourcegraph

I want to use Sourcegraph for tern-def-origin code search, browsing, and usage examples. Can an admin enable Sourcegraph for this repository? Just go to https://sourcegraph.com/github.com/sourcegraph/tern-def-origin. (It should only take 30 seconds.)
Thank you!

I enabled it, but unfortunately, it looks like the build was not successful (no links were found). Our JavaScript support is currently in alpha, but we are working on improving it soon. Thanks for trying out Sourcegraph and let us know if you have any additional feedback!


Bmi compliant arrays

Added routines to encode and decode BMI compliant arrays (flattened, and appropriately preprocessed time- and station ID axes). Encoding is implemented for USGS and USACE preprocessed time domain files in model_DAforcing. py.
For the time being (to verify proper function), the reverse de-coding is also implemented in model_DAforcing.py. Said decoding will, in one of the next steps, be moved to DataAssimilation. However, the file can be checked in, b/c the decoiding can be easily disabled by setting the flag testRevert = False.
Next steps will be added to the same pull request.
Additions

Removals

Changes

Testing

Screenshots
Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows project standards (link if applicable)
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[ ] Windows
[ ] Linux
[ ] Browser

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

@JurgenZach-NOAA it looks like this PR also includes the newly created forcing files for LowerColorado_TX_V4. Are these necessary for this PR? If not, could you remove these?

Sean: they were not needed, and I removed them, thanks for pointing out!

Added rfc encoding & decoding.

Cleaned up file from debug statements.


Tags with no paths when feature is on

Tags were beautifully implemented :) Thank you!
Would it be possible to either not allow a snapp to be added without a path or have a user be told that a tag is missing?
P.S. Do you see any issues with me manually changing userid of a snapp to another user via the db? This way I can just add all snaps and then I can change the owner to someone else for them to manage and when a user departs, I can just send them over to whoever takes their place.

I'll check ;)
Btw should be possible to change user id i don't recall locking it on any way, need to check that the relationship keeps working after tho

Nice thank you 😊
Yeah I've been manually changing it and it doesn't seem to break any relationship but I'm not an expert so just wanted to asked to see if you can think of any issues I might face down the road by doing this.

yeah the worst that could happen is littering the db with unavailable relation and maybe the metrics are not perfect in that case, but that should be all :)
next version is contemplating the tag necessities in case of tags_as_prefix :)

fixed in <IP_ADDRESS> (building rn)


fix: Fix Endpoint Service not being injected in self object

Fixes # 116
Changes proposed in this pull request:

Added the endpointService inside the constructor of the signup component which fixed the endpointService not being found in the self object.

This PR fixes 116 issue.

Codecov Report

Merging #117 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
+ Coverage   53.43%   53.47%   +0.03%
==========================================
  Files          55       55
  Lines        2358     2360       +2
  Branches      253      253
==========================================
+ Hits         1260     1262       +2
  Misses       1023     1023
  Partials       75       75

Impacted Files
Coverage Δ

src/app/components/auth/signup/signup.component.ts
75.75% <100%> (+1.56%)
:arrow_up:

Impacted Files
Coverage Δ

src/app/components/auth/signup/signup.component.ts
75.75% <100%> (+1.56%)
: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 f822b2d...0e0fb20. Read the comment docs.

Nice find. Shall we merge this @Shekharrajak @RishabhJain2018?

Yes! I believe it can be merged, after another approval.


rpi5-image: Add kernel modules

Add kernel modules to linux rootfs.
Add dependencies for do_image routine

Could you please describe what is the reason? Why are you adding all kernel modules? AFAIK, Yocto provides per-module packages like "kernel-module-e1000" so you can install only required modules.
Which modules you need?

It's my request - I hit absence of gntdev/gntalloc.
We need default set of modules provided by RPI5 Kernel build for now. And actually in general, no goal and neither requirements to optimize or minimize RPI5 Kernel build.
By default "minimal" image comes without (or with limited number) modules, and it's not suitable to build "base" image (which include modules)  as it has way too many other features enabled and build takes much more time.

@dsemenets Pls, fix comment related to rpi-bootfiles: drop it or add as separate patch then
LGTM Reviewed-by: Grygorii Strashko<EMAIL_ADDRESS>

By default "minimal" image comes without (or with limited number) modules

Well, this is the definition of "minima" image.
Actually, proper solution is to introduce image-xt  with the product-specific set of modules.

Also, one more thing: should this change be in this repo? Or in meta-xt-prod-devel-rpi5?
Do we really need all RPI5-based products to include all kernel modules? @GrygiriiS, what is your opinion?

Also, one more thing: should this change be in this repo? Or in meta-xt-prod-devel-rpi5?
Do we really need all RPI5-based products to include all kernel modules? @GrygiriiS, what is your opinion?

May I intrude? I think that adding kernel-modules is the common yocto way. Because module configuration depends from the kernel config and can be easily controlled by modifying config. If we use an approach when we're producing the set of kernel modules and add them explicitly then we should change 2 places instead of just one.

I am fine with the first change. But could you please elaborate your description for the second patch? What was the root cause? How your change fixes it? Or is it just a workaround/experiment? In this case it should be marked as workaround/experiment

@DS Please drop the second patch from this PR as it is not related to it.

Removed patch related to bootfiles

Also, one more thing: should this change be in this repo? Or in meta-xt-prod-devel-rpi5?
Do we really need all RPI5-based products to include all kernel modules? @GrygiriiS, what is your opinion?

May I intrude? I think that adding kernel-modules is the common yocto way. Because module configuration depends from the kernel config and can be easily controlled by modifying config. If we use an approach when we're producing the set of kernel modules and add them explicitly then we should change 2 places instead of just one.

I agree here with @oleksiimoisieiev - modules (at least modules from Kernel repo) are part of the Kernel and first of all depends and enabled by defconfig. So no need to play with module's recipes.


[HOLD for payment 2024-06-20] [HOLD for payment 2024-06-18] Fix the style regression that removed the bold styling from the Switch component on cards

This App PR caused a styling regression.
https://github.com/Expensify/App/pull/42596#issuecomment-2139064433
Bring the bold styling back to the Switch component.

I think we're good to close here @hayata-suenaga - please reopen if I missed something!


request to add static cms feature

request to add static cms feature.

sorry this is not a bug requsest , your theme is really best, i wish it should have instigation with static CMS do you have any plan do that ?  ?  i am trying to achieve that i already made one with netlify cms but i realize static cms is way more better https://krafi-info.netlify.app/ https://gitlab.com/krafi/krafi.info . can you help me to achieve that . please >>.... https://www.youtube.com/watch?v=3yip2wSRX_4&t=78s

src/content/posts/
├── post-1.md
└── post-2/
├── cover.png
└── index.md
so you recommend to uses it to the same directory as post but i dont know how can i do that with cms becausse with cms the folder
backend:
  name: git-gateway
  branch: main # Branch to update (optional; defaults to master)

publish_mode: simple # editorial_workflow (if you want multi brunch)
media_folder: "src/assets" # Media files will be stored in the repo under static/images/uploads
public_folder: "/assets" # The src attribute for uploaded media will begin with /images/uploads

collections:
  - name: "blog" # Used in routes, e.g., /admin/collections/blog
    label: "Blog" # Used in the UI
    folder: "src\content\posts" # The path to the folder where the documents are stored
    create: true # Allow users to create new documents in this collection
    slug: "{{slug}}" # Filename template, e.g., YYYY-MM-DD-title.md
    fields: # The fields for each document, usually in front matter
    draft: false
      #- {label: "Layout", name: "layout", widget: "hidden", default: "../../layouts/BlogPost.astro"}
      - {label: "Title", name: "title", widget: "string"}
      - {label: "Description", name: "description", widget: "string"}
      - {label: "category", name: "category", widget: "string"}
      - {label: "tags", name: "tags", widget: "string"}
      - {label: "Publish Date", name: "publishDate", widget: "datetime"}
      - {label: "image", name: "image", widget: "image"}
      - {label: "Body", name: "body", widget: "markdown"}
      - {label: "draft", name: "draft", default: "false"}

can you tell me how can i use the default photo stracture ? keep picture on the src/assets

https://github.com/View-my-Git-Lab-krafi/mywebkrafi/blob/main/public/admin/config.yml

So  i came to this solution with  some  extra pain to use cms

manually entering date using cms, why there it no field  to enter time , there is a lot of widget i will give at the bottom .  but i dont know what you use i cant use those date  ,  datetime widget
as i notice i can use only one catagory
Tags without tags the build fail , why ?? even the tags are [] so netlify compiler mess with it, I used {} which will break at the end. and then manually going to git i can change it to [] .

to be honest what i was trying to do looks like i faild . sadly .
backend:
  name: git-gateway
  branch: main # Branch to update (optional; defaults to master)

publish_mode: simple # Set to editorial_workflow if you want multi-branch workflow
media_folder: "src/content/posts" # Media files will be stored in the repo under src/content/posts
public_folder: "./" # The src attribute for uploaded media will begin with /content/posts
collections:
  - name: "blog" # Used in routes, e.g., /admin/collections/blog
    label: "Blog" # Used in the UI
    folder: "src/content/posts" # The path to the folder where the documents are stored
    create: true # Allow users to create new documents in this collection
    slug: "{{slug}}" # Filename template, e.g., YYYY-MM-DD-title.md
    fields: # The fields for each document, usually in front matter
    #- {label: "Layout", name: "layout", widget: "hidden", default: "../../layouts/BlogPost.astro"}
      - {label: "Title", name: "title", widget: "string"}
      - {label: "Description", name: "description", widget: "string"}
      - {label: "Category", name: "category", widget: "string", default: "you_can_set_one" }
      - {label: "Publish Date", name: "published", widget: "string", default: "yy-mm-dd"}
      - {label: "Tags", name: "tags", widget: "list", default: ["all", "all", "all"]}
      - {label: "Image", name: "image", widget: "image"}
      - {label: "Body", name: "body", widget: "markdown"}
      - {label: "Draft", name: "draft", widget: "boolean", default: false}

ghost cms widget

**String Widget

Type: string
Description: For short text input.
Example Usage: Titles, names.
Text Widget

Type: text
Description: For longer text input, typically multi-line.
Example Usage: Descriptions, biographies.
Number Widget

Type: number
Description: For numeric input.
Example Usage: Prices, quantities.
Boolean Widget

Type: boolean
Description: For true/false input.
Example Usage: Draft status, feature toggles.
Date Widget

Type: date
Description: For date input.
Example Usage: Event dates, publish dates.
Datetime Widget

Type: datetime
Description: For date and time input.
Example Usage: Timestamp, scheduling.
Markdown Widget

Type: markdown
Description: For Markdown-formatted text.
Example Usage: Blog posts, content with rich text formatting.
Image Widget

Type: image
Description: For image file uploads.
Example Usage: Featured images, gallery items.
File Widget

Type: file
Description: For file uploads.
Example Usage: Documents, downloadable resources.
Select Widget

Type: select
Description: For dropdown selection.
Options: Single or multiple selections.
Example Usage: Categories, tags.
Relation Widget

Type: relation
Description: For selecting and referencing entries from another collection.
Example Usage: Related articles, linked content.
List Widget

Type: list
Description: For creating an array of items.
Example Usage: Galleries, lists of links.
Subwidgets: Can contain other widgets as items.
Object Widget

Type: object
Description: For creating a structured group of nested fields.
Example Usage: Contact forms, complex data structures.
Subwidgets: Contains other widgets as fields.
Hidden Widget

Type: hidden
Description: For fields that should not be displayed in the UI.
Example Usage: Metadata, fixed values.**

well netlify function can solve this issue i think by replacing {} to [] ## python code

Thank you for your support, but I do not plan to introduce more CMS by default into the project. Since I am not familiar with CMS, I may not be able to provide further assistance. I am converting this issue into a discussion.


Create ifNotExists Block

First off, great job on the code!!!
Am I miss understanding the ifNotExists on a create?
If i have the following and the table exists, I would think the print statement would NOT be executed?
do {
            let users = Table("users")

            let id = Expression<Int64>("id")
            let email = Expression<String>("email")
            let name = Expression<String?>("name")
            try self.db!.run(users.create(ifNotExists: true) { t in     // CREATE TABLE "users" (
                t.column(id, primaryKey: true) //     "id" INTEGER PRIMARY KEY NOT NULL,
                t.column(email, unique: true)  //     "email" TEXT UNIQUE NOT NULL,
                t.column(name)                 //     "name" TEXT
                print("DB Created")
                })
        }
        catch {
        }

Hi there, the closure attached to users.create builds the SQL string but it has no executed yet. If you move the print statement below the try db.run statement, it will print after execution only.

Thanks .. I thought that too, however that's where I had it originally.  It still gets called if moved.  Thoughts?

Ah, so if you want the statement to fail (and hit the error path), you need to remove the ifNotExists. IF NOT EXISTS means: silently swallow failure if the table already exists.

BINGO!  That's it! Works now. thanks. now I understand.  This is way cool!


Add release page

e.g. /releases/stable/v20160721.2

and on error:

Hmm, lets drop /releases from the path so it's just /:channel/:version.

LGTM pending path change.


Teach the run script to use system mono if available

This PR implements the behaviour described in README.md:

If Mono is globally installed on the system, OmniSharp will prefer it over the embedded version, however version >=6.4.0 is required.

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

Thanks for the PR, however this is not the intent of text in the readme. The statement refers to MSBuild discovery.
Launching on global/local Mono should be controlled by the client integrating OmniSharp, e.g. that is what C# Extension for VS Code does. The run file is an integral part of the embedded Mono experience. In other words, the client should call mono OmniSharp.exe if it wants to use the global Mono or run if it wants to use embedded Mono.

Alright, thanks @filipw , I'll close the PR.

I will reword the readme to make it more obvious, sorry for the confusion.
Also this change now, would break clients. Thanks.


Circular dependency with mkdocstrings-python-legacy

First off, great work on this project.
We recently converted our API docs at https://docs.ibis-project.org over to mkdocstrings and are extremely happy with the results, so thank you for all the work that you do here.
Describe the bug
We are using poetry2nix to manage dependencies for development environments.
poetry2nix will turn poetry.lock into nix expressions each of which maps to a single Python package.
To do this, poetry2nix will traverse the transitive closure of the set of dependencies specified in poetry.lock. However, it is unable to do so when a project has mkdocstrings==0.18 as a dependency because mkdocstrings 0.18 depends on mkdocstrings-python-legacy which itself depends on mkdocstrings 0.18, resulting in infinite recursion.
To Reproduce
Steps to reproduce the behavior:

Install nix
Download the files in this gist into a directory
Run nix build

Expected behavior
I expect this to work with poetry2nix without infinite recursion.
Information (please complete the following information):

OS: NixOS
Browser: Brave
mkdocstrings version: 0.18.0

Hi @cpcloud! Thanks a lot, I'm glad you're enjoying mkdocstrings :smile:
Now about the issue at hand. It's true the dependencies are circular, just as you stated: mkdocstrings depends on mkdocstrings-python-legacy which depends on mkdocstrings. But Python resolvers (pip, PDM, Poetry) all seem to be able to resolve this fine. Maybe there's something to do on poetry2nix's side? We're probably their first use-case of circular dependencies :smile: ?
Let me know if there's anything we can do on our side (but I doubt so, apart from removing the circular dependencies, which I'm not really inclined to do since it seems to be a valid specification).

By the way, how does poetry2nix work on PDM projects? PDM generates a pdm.lock file (different name but same format), and we don't even version it.

this is not supported by the nixpkgs python infra

Err, that's too bad :confused:
https://github.com/nix-community/poetry2nix/issues/1

By the way, how does poetry2nix work on PDM projects? PDM generates a pdm.lock file (different name but same format), and we don't even version it. (@adisbladis) EDIT: ah, you're probably using it on your own pyproject.toml and poetry.lock files.

Yeah, pdm is incidental in our case. poetry2nix can work with any of the major Python build tools.

We're probably their first case of circular dependencies?

Possibly, though it's a bit surprising this hasn't come up before outside the various common leaf dependencies like setuptool-scm et al.

apart from removing the circular dependencies

Would you mind elaborating a bit on the rationale for the circularity?
It seems like mkdocstrings should be installable without any dependency (including an optional one) on mkdocstrings-python-legacy, while mkdocstrings-python-legacy definitely requires mkdocstrings.
We have a similar problem in ibis, where the ibis-bigquery project depends on ibis-framework, but ibis itself knows nothing about ibis-bigquery.
Do you see any issue with leaving mkdocstrings-python-legacy out of the mkdocstrings dependency specification altogether while leaving mkdocstrings as a dependency of mkdocstrings-python-legacy?

Same here, I am creating conda packages for mkdocstring (see https://github.com/conda-forge/staged-recipes/pull/17960 and https://github.com/conda-forge/mkdocstrings-feedstock/pull/17).
Circular dependencies are making the packaging harder (the snake eating itself).
If it's possible to break it, I would favor doing it as I am not sure having circular deps (even your package manager supports it) is a good thing.

Would you mind elaborating a bit on the rationale for the circularity?
It seems like mkdocstrings should be installable without any dependency (including an optional one) on mkdocstrings-python-legacy, while mkdocstrings-python-legacy definitely requires mkdocstrings.

Sure!
In fact, removing the dependency on mkdocstrings-python-legacy is planned. It's only here for the transition: previously the legacy Python handler was part of mkdocstrings, and now it lives in this other mkdocstrings-python-legacy project. To give some time to users to adapt, mkdocstrings still hard-depends on it while providing the python-legacy extra (which is a no-op in this context since the legacy handler is always installed).
When imported, the legacy handler warns users that the extra will become mandatory in the next release. This is to make sure nobody gets a bad surprise while upgrading, as I believe the way to deprecate things is, well, to use [deprecation/user] warnings for a short period at least.
It brings us to this: next version of mkdocstrings will only optionally depend on mkdocstrings-python-legacy, which I believe is better supported by other infrastructures tooling, and would not result in infinite recursion?

@hadim does Conda understands the concept of extras?

Not at the moment. I think it's something the conda folks are thinking about, but it's probably going to take a while.
So usually, it's up to the package maintainer to do what he thinks is best: either include all the extra, none of them or only a few subsets of them.
For the conda packages, extra are not really an issue to me. The circular deps are more critical. But if it's only temporary, then I think it's fine, and I'll upgrade the conda packages once it has been resolved.

@hadim Thanks for the explanation, that's reassuring.
Same question to you @cpcloud @adisbladis, will poetry2nix/nix choke on the extras?

@pawamoy Circling back here, I think making mkdocstrings-python-legacy a true optional dependency will suffice!
I tested this by removing the dependency by from poetry.lock, and poetry2nix is able to run.

We were able to get the conda-forge package built with all the dependencies needed. I documented the process in conda-forge/mkdocstrings-feedstock#19, but everything is up to date now!

Have you tried with mach-nix?

@pawamoy I looked into this some more, and simply making mkdocstrings-python or mkdocstrings-python-legacy optional will not solve the problem.
Ultimately to work with poetry2nix the dependency graph needs to be free of cycles.
I can work around this problem on my side, but only if both mkdocstrings-python and mkdocstrings-python-legacy are optional dependencies. Otherwise there's a cycle.
Is it possible to keep all the various rendering addons as pure optional dependencies?

I second that. I'm currently patching the pyproject.toml before building/installing mkdocstrings-python-legacy by doing
sed -i "s/^.*\"mkdocstrings>=.*$//" pyproject.toml

but that can't be the solution in the long term. (Note: I'm using Nix flakes directly, without poetry2nix.)

If this installs properly with normal python tooling, can't it be a bug in poetry2nix?

I kinda agree with @Yajo here: extras (optional dependencies) is a pretty standard concept in the Python packaging ecosystem. I'd really like to keep the extras on mkdocstrings :confused: Our circular dependency can't possibly be the only one in the whole ecosystem. It seems poetry2nix special-cases some libraries that cause such circularity, see https://github.com/nix-community/poetry2nix/issues/538, maybe mkdocstrings could be special-cased there as well? Tagging @adisbladis again :wave:

This also affects Bazel projects 👎🏻
ERROR: in py_library rule @python_deps_mkdocstrings//:pkg: cycle in dependency graph:
    //tools/mkdocs:mkdocs (94fc984876740c7bbd36b3ce5fe83bc4802138cbbc4222f011384e5ab773a519)
.-> @python_deps_mkdocstrings//:pkg (94fc984876740c7bbd36b3ce5fe83bc4802138cbbc4222f011384e5ab773a519)
|   @python_deps_mkdocstrings_python_legacy//:pkg (94fc984876740c7bbd36b3ce5fe83bc4802138cbbc4222f011384e5ab773a519)
`-- @python_deps_mkdocstrings//:pkg (94fc984876740c7bbd36b3ce5fe83bc4802138cbbc4222f011384e5ab773a519)

Project looks great by the way - looking forward to using it!

Hi all, just released v0.19, which does not directly depend on mkdocstrings-python-legacy anymore. I would appreciate if you could try again your different tools to see if that still poses a cyclic issue or not.

mkdocstrings is working perfectly with Bazel now 🥳!
Thanks for your great work!

This works with poetry2nix now, with the following caveat: you can't use the extras field to add the python handler like this
mkdocstrings = { version = "...", extras = ["python"] }

Instead, you have to add the handler as a separate dependency:
mkdocstrings = "..."
mkdocstrings-python = "..."

After that, I'm able to get past infinite recursion though griffe is failing to handle __all__ generated by the public library but I'll open a separate issue for that.

Perfect! Thanks for your feedback @cpcloud @jpgxs :slightly_smiling_face:

Here's the (unrelated) griffe issue. Linking here in case anyone hits it later: https://github.com/mkdocstrings/griffe/issues/72.


The search result may be outdated. Run git pull to get the latest results.

Install PETSc library with vcpkg.

What is your question?

@keyvan136181 ,Thanks for posting this issue.
Could you describe your issue in detail, as well as the reproduction environment and reproduction steps? And provide a complete error log

I have the same problem. OS: Windows 10 x64. Steps to reproduce and further environment details below. Where can one find the complete error logs? I'd be most happy to provide one, but you didn't say which file you wanted to see. (It looks like a warning or note but not an error, by the way.)

C:\Users\szabolcs\vcpkg>git status
On branch master
Your branch is up to date with 'origin/master'.

nothing to commit, working tree clean

C:\Users\szabolcs\vcpkg>vcpkg --version
Vcpkg package management program version 2021-07-26-9425cf5f512f242c0bcbabac31f08832825aee81

See LICENSE.txt for license information.

C:\Users\szabolcs\vcpkg>vcpkg search gdk
gdk-pixbuf               2.42.2#1         Image loading library.
The search result may be outdated. Run `git pull` to get the latest results.

If your library is not listed, please open an issue at and/or consider making a pull request:
    https://github.com/Microsoft/vcpkg/issues

C:\Users\szabolcs\vcpkg>git pull
Already up to date.

C:\Users\szabolcs\vcpkg>vcpkg search gdk
gdk-pixbuf               2.42.2#1         Image loading library.
The search result may be outdated. Run `git pull` to get the latest results.

If your library is not listed, please open an issue at and/or consider making a pull request:
    https://github.com/Microsoft/vcpkg/issues

C:\Users\szabolcs\vcpkg>

Yeah it is a warning, but we don't check anything. If we don't want to display the warning we have to do a git fetch for every search but this is way to expensive.

In that case (prohibite cost of checks), it would be feasible to check that only once a week or another period that matches the average frequency of changes in the repository. (Additionally, an ls-remote operation might be more cheap than a fetch but I'm not an expert in git TBH.)
Another alternative would be to give users the choice of opting out of seeing that message, as it would not be unreasonable to expect them to be aware of this aspect of package managers:.vcpkg --im-aware-that-package-descriptions-are-always-getting-outdated-lol ... and the message would not appear anymore.

or another period that matches the average frequency of changes in the repository

Yeah changes happens every few hours. So if the last git pull is one hour old, the result is maybe outdated again. But yeah once a week is properly enough. The problem was that people often requested new versions of libs that were already there but they have not updated their vcpkg instance.
ls-remote needed 2 seconds for me. Not optimal.

vcpkg --im-aware-that-package-descriptions-are-always-getting-outdated-lol

Can you then not simply ignore the message.
But maybe one could use git --no-pager log -1 --format=%cd to then display a message like: The search results are x days old. Use git pull to get the latest results

That's just a notice.

IMO recommending a "git pull" is too intrusive. I would prefer to have a --online option. which could be used (and recommeded) safely.

The --online option should then simply open the webbrowser and search on vcpkg.io?

get the latest (???) results.

If they have not changed it no. They have to update the site manually... WTF

@autoantwort  Yes, it is always up to date only in our master branch.

@autoantwort Yes, it is always up to date only in our master branch.

you mean in the vcpkg repo? The vcpkg.io repo is definitely not up to date. For example vcpkg.io still shows the old version for FreeRDP which was updated in #18242

@autoantwort Yes, in vcpkg.io, we are not updated synchronously.

@keyvan136181 ,That's just a notice.Can this reply solve your problem？


setStepSize(float stepSize) No effect

And how to Modify the spacing of stars ? setStepSize(float stepSize) No effect

en , I have solved it.谢谢


Is it possible to have a layer attribution collapsed by default to his button form?

Hello
For some reason I have a small black in the attribution:

I wonder if it is possible to have the attribution collapsed by default, to have it in his button form.
Thanks & best regards
Gilles

No, it is mandatory as per OSM policy. Openlayers respects their decision and does not allow the collapsing of the attributions. Check your CSS, there is probably something missing.

Hello
Thanks for the info.  I already check around but found nothing.
gilles


[.NET] Generate new test signing cert to replace expired one

Description
The test signing certificate for the .Net WPF Visualizer packaging project has expired, leading to CI failures. This change replaces the old one for a new one.
How Verified

local build
PR build

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


Customer Docker bridge is not correctly detected

Hi,
I am using containerlab to spin up a lab on a pre-existing Docker network with a customer bridge. For the context, this is a bridge created by libvirt (so I can have my containerlab nodes and my KVM guests on the same Linux bridge).
I am creating my Docker network like this:
docker network create --driver bridge --subnet <IP_ADDRESS>/24 --ip-range <IP_ADDRESS>/25 --gateway <IP_ADDRESS> -o "com.docker.network.bridge.name"="virbr0" lab-mgmt
Result:
$ docker network inspect lab-mgmt
[
    {
        "Name": "lab-mgmt",
        "Id": "f1d000689d48b05cc6b9cc651f78b68bd33832fa8f96ba340a7e616ea2c26c2d",
        "Created": "2023-05-04T12:56:47.263459677+02:00",
        "Scope": "local",
        "Driver": "bridge",
        "EnableIPv6": false,
        "IPAM": {
            "Driver": "default",
            "Options": {},
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/24",
                    "IPRange": "<IP_ADDRESS>/25",
                    "Gateway": "<IP_ADDRESS>"
                }
            ]
        },
        "Internal": false,
        "Attachable": false,
        "Ingress": false,
        "ConfigFrom": {
            "Network": ""
        },
        "ConfigOnly": false,
        "Containers": {},
        "Options": {
            "com.docker.network.bridge.name": "virbr0"
        },
        "Labels": {}
    }
]

My management configuration in the topology file:
mgmt:
  network: lab-mgmt

Let's create the lab:
$ sudo containerlab deploy --log-level trace
DEBU[0000] detected network name in use: lab-mgmt, backed by a bridge br-f1d000689d48
[truncated]
DEBU[0000] Docker network "lab-mgmt", bridge name "virbr0"
DEBU[0000] Disable RPF check on the docker host
DEBU[0000] Enable LLDP on the linux bridge br-f1d000689d48
WARN[0000] failed to enable LLDP on docker bridge: open /sys/class/net/br-f1d000689d48/bridge/group_fwd_mask: no such file or directory
DEBU[0000] Disabling TX checksum offloading for the br-f1d000689d48 bridge interface...
WARN[0000] failed to disable TX checksum offloading for the br-f1d000689d48 bridge interface: no such device
DEBU[0000] found iptables forwarding rule targeting the bridge "br-f1d000689d48". Skipping creation of the forwarding rule.

I am concerned about this log output:
DEBU[0000] detected network name in use: lab-mgmt, backed by a bridge br-f1d000689d48
Looks like the code cannot get the pre-existing docker network options correctly, specifically, this line: https://github.com/srl-labs/containerlab/blob/main/runtime/docker/docker.go#L118
I can work around this by explicitly setting the bridge name in the topology file:
mgmt:
  network: lab-mgmt
  bridge: virbr0

Similarly, I was previously using the default docker network 'bridge', but using a different backend bridge than docker0 (it was virbr0), and this line made it impossible to use this config with containerlab: https://github.com/srl-labs/containerlab/blob/main/runtime/docker/docker.go#L228

Double checked, and an issue is, that we're deleting the docker network if no more containers are connected.
Hence it might be worth to also adding the bridge and gw options like this (https://containerlab.dev/manual/network/#bridge-name)
mgmt:
  network: lab-mgmt
  bridge: virbr0
  ipv4_subnet: <IP_ADDRESS>/24
  ipv4-gw: <IP_ADDRESS>

To make clab create and recreare the docker network itself.
But I recognize you split the ip range probably between vms and containers. It does not seem like we're exposing the range.
Let me take a look if we can make that possible.

To keep the mgmt network when no containers are connected you can use https://containerlab.dev/cmd/destroy/#keep-mgmt-net

It dawned on me we had a knob like that. But only looked under the manual/network.
Anyways... this is a flag only availabe under destroy... I myself always use --reconfigure which will also delete the network.
Should we maybe also add the flag to the deploy command?

@hellt I confirm the new build fixes the issue
@steiler Yeah, you got it. I am destroying labs with --kepp-mgmt-net because the IP ranges are not exposed in the topology file. My use case: virbr0 is a libvirt bridge with its own dnsmasq instance that handles DHCP for KVM guests. In order to avoid conflicts with container IPs, I am restricting the range Docker can use.


Reload accessibility node tree when VoiceOver status changes

Fixes https://youtrack.jetbrains.com/issue/CMP-6874/VoiceOver-cannot-be-enabled-when-app-is-running
Release Notes
Fixes - iOS

Fix a bug where the accessibility tree did not reload when VoiceOver was enabled

Sure we can. However we didn't start covering accessibility by tests at all.


fix: ensure messages are only shown once

The idea is to keep track of the shown messages by queueId (which is unique per message) and skip the shown ones
This is based on the same solution applied for gist-web: https://github.com/customerio/gist-web/pull/26
Task: https://github.com/customerio/issues/issues/11967
Complete each step to get your pull request merged in. Learn more about the workflow this project uses.

[x] Assign members of your team to review the pull request.
[x] Wait for pull request status checks to complete. If there are problems, fix them until you see that all status checks are passing.
[ ] Wait until the pull request has been reviewed and approved by a teammate
[ ] After pull request is approved, and you determine it's ready add the label "Ready to merge" to the pull request and it will automatically be merged.

I think it would be simpler if the implementation was handled inside Queue.kt instead of GistSdk.kt

This has been updated and tested, ready for review.

Yeah, we should eventually implement a retry mechanism on failure and maybe persist this variable into user preferences.


Ensure opening/closing sign ledger modal results in correct state (buggy in some flows atm)

Can you provide some more details?

@kyranjamie Please provide more details for the product spec here before we move to the dev backlog


Fix memory leak when using Laravel Octane

Alternatively, I could add this only when Octane is present:
if(class_exists(Laravel\Octane\Octane::class))
{
    BladeManager::clearBlocks();
}

Thanks for merging! 😄


fix(costmodels): add aria-label in tag rate table

closes COST-588

Codecov Report

Merging #1680 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1680   +/-   ##
=======================================
  Coverage   77.86%   77.86%
=======================================
  Files         244      244
  Lines        3984     3984
  Branches      763      763
=======================================
  Hits         3102     3102
  Misses        798      798
  Partials       84       84

Impacted Files
Coverage Δ

src/pages/costModels/components/tagRateTable.tsx
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Thanks @dlabrecq


[question] Re-adding dead devices...

After switching to 1.6, I am experiencing some issues getting my battery-driven devices to work. Even removing and adding the devices back doesn't work. Here's what I am seeing:

zwave2mqtt    |   z2m:App Zwave api call: addNode [ true ] +14s
zwave2mqtt    |   z2m:Zwave Success zwave api call addNode true +112ms
zwave2mqtt    |   z2m:Zwave controller command { nodeid: 0, state: 1, errcode: 0, help: 'AddDevice Starting' } +0ms
zwave2mqtt    |   z2m:Zwave controller command { nodeid: 0, state: 4, errcode: 0, help: 'AddDevice Waiting' } +3ms
zwave2mqtt    |   z2m:Zwave controller command { nodeid: 0, state: 6, errcode: 0, help: 'AddDevice InProgress' } +11s
zwave2mqtt    |   z2m:Zwave controller command { nodeid: 17, state: 6, errcode: 0, help: 'AddDevice InProgress' } +5ms
zwave2mqtt    |   z2m:Zwave controller command { nodeid: 17, state: 6, errcode: 0, help: 'AddDevice InProgress' } +941ms
zwave2mqtt    |   z2m:Zwave Node added 17 +153ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-32-1-0 Basic +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-94-1-0 ZWave+ Version +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-94-1-1 InstallerIcon +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-94-1-2 UserIcon +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-128-1-0 Battery Level +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-0 Powerlevel +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-1 Timeout +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-2 Set Powerlevel +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-3 Test Node +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-4 Test Powerlevel +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-5 Frame Count +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-6 Test +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-7 Report +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-8 Test Status +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-115-1-9 Acked Frames +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-134-1-0 Library Version +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-134-1-1 Protocol Version +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-134-1-2 Application Version +0ms
zwave2mqtt    |   z2m:Zwave controller command { nodeid: 17, state: 7, errcode: 0, help: 'AddDevice Completed' } +2ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-132-1-0 Wake-up Interval +256ms
zwave2mqtt    |   z2m:Zwave Notification from node 17: Notification - NoOperation (2) +128ms
zwave2mqtt    |   z2m:Zwave Notification from node 17: Notification - TimeOut (1) +3s
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-114-1-0 Loaded Config Revision +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-114-1-1 Config File Revision +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-114-1-2 Latest Available Config File Revision +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-114-1-3 Device ID +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-114-1-4 Serial Number +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-1 Alarm Cancellation Delay +138ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-2 Acoustic and visual signals +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-7 Requested dimming level / roller blind opening level when sending turn on / open command to 2nd association group devices +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-9 Deactivate turning off devices in 2nd association group alarm cancellation in 3rd group +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-10 Interval between successive readings of temperature sensors +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-12 Insensitiveness to temperature changes. +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-50 Low temperature alarm threshold. +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-51 High temperature alarm threshold. +2ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-61 Low temperature alarm indicator colour. +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-62 High temperature alarm indicator colour. +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-63 Managing a LED indicator under standard operation. +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-73 Temperature measurement compensation. +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-74 Alarm frame sent to 1st and 4th Association Group activation. +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-75 Visual and audible alarm duration +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-76 Alarm signalization reactivation period +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-77 Flood sensor functionality turned off. +1ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-112-1-78 Associations in Z-Wave network security mode +0ms
zwave2mqtt    |   z2m:Zwave ValueAdded: 17-48-1-0 Sensor +198ms

Basically, the devices is sending lots of information. What I don't see in the log - and maybe that's the problem - is a statement that the node is ready.
I am running the latest Z2M and OpenZwave Version 1.6-994-gc78eddf6.
Help would be appreciated.

Okay, I got this to work by

Removing the device
Perfoming a factory reset
Re-adding the device

Unfortunately, that results in a new node id...


Wrong title in README?

I was wondering where this gulp-angular-filesort title came from, in the project README file. :)

Oh… thanks for the hint ;)
--
Christopher Supnig
Have a look at my blog at https://www.supnig.com
On 11 March 2016 at 17:01:12, arboleya<EMAIL_ADDRESS>wrote:
I was wondering where this gulp-angular-filesort title came from, in the project README file. :)
—
Reply to this email directly or view it on GitHub.


annoying settings change hopefully will fix issues

I think this can remain as local changes. Want to leave this.


alvr.head_position[] does not update head position in while loop.

I created this script to grab a controller's position in my tracking space and diagnostics.debug(str(unpacked_data[0])) prints a valid position for an x coordinate, but the oculus go headset does not change position. My client code grabs valid positions and orientations for my controller. Any thoughts?
Server Code Running in FreePIE
import socket
import sys
import time
import struct

UDP_IP_ADDRESS = "<IP_ADDRESS>"
UDP_PORT_NO = 6789
serverSock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
serverSock.bind((UDP_IP_ADDRESS, UDP_PORT_NO))

if len(sys.argv) == 1:
    interval = 1/30
elif len(sys.argv) == 2:
    interval = 1/float(sys.argv[0])
else:
    print("Invalid number of arguments")
    interval = False

alvr.override_head_position = True
alvr.override_head_orientation = True
#print "Start Server on port:", UDP_PORT_NO
while True:
    start = time.time()
    s = struct.Struct('f f f f f f')
    data  = serverSock.recv(s.size)
    unpacked_data = s.unpack(data)

    multiplier = 2

    alvr.head_position[0] = unpacked_data[0] * multiplier
    alvr.head_position[1] = unpacked_data[1] * multiplier
    alvr.head_position[2] = unpacked_data[2] * multiplier
    #alvr.head_orientation[0] = unpacked_data[3]
    #alvr.head_orientation[1] = unpacked_data[4]
    #alvr.head_orientation[2] = unpacked_data[5]
    diagnostics.debug(str(unpacked_data[0]))
    #diagnostics.debug("x: " , str(unpacked_data[0]) , " y: " + str(unpacked_data[1]) , " z: " , str(unpacked_data[2])
    #diagnostics.debug("Pos x: {0} y: {1} z: {2}" unpacked_data[0], unpacked_data[1], unpacked_data[2])
    sleep_time = interval-(time.time()-start)
    if sleep_time>0:
        time.sleep(sleep_time)

Client Code getting OpenVR infomation
import time
import sys
import socket
import struct
import triad_openvr

v = triad_openvr.triad_openvr()
v.print_discovered_objects()

UDP_IP_ADDRESS = "<IP_ADDRESS>"
UDP_PORT_NO = 6789

sock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
server_address = (UDP_IP_ADDRESS, UDP_PORT_NO)

if len(sys.argv) == 1:
    interval = 1/250
elif len(sys.argv) == 2:
    interval = 1/float(sys.argv[0])
else:
    print("Invalid number of arguments")
    interval = False

if interval:
    while(True):
        start = time.time()

        data = data =  v.devices["controller_3"].get_pose_euler()
        print("\r" + "sending data=" + str(data),  end="")
        packer = struct.Struct('f f f f f f')
        packed_data = packer.pack(*data)
        try:
            sent = sock.sendto(packed_data, server_address)
        except:
             print("error")
        #print("\r" + binascii.hexlify(packed_data), end="")
        sleep_time = interval-(time.time()-start)
        if sleep_time>0:
            time.sleep(sleep_time)

Ok so I did no realize that FreePIE runs your python script in a loop. So I modified the code and will try it later. Check it out here

That's right. FreePIE plugin applies the changed value (head position or other values) only when the script exits.
And you also need "global serverSock" after "def loop():"

So everything works. Its pretty cool. With my openvr socket set up I was able to move based on a vive tracker. It bothers me though that to position of the camera is arbitrary. It needs to base relative to a steamvr reference object (light house)

The headsets orientation is all messed up in the steamVR space.

I need to understand to vector math for these two spaces. How do I position the headset so that it is in the position of the tracker when the tracker is relative to the lighthouse and the headset is relative to nothing?

Also could you add a rotational offset for the headset? I don't want to change the rotation. Just apply an offset. Currently the headsets rotation is in a different direction than the tracking space

So after further research, if you send [0,0,0] to the headsets position using FreePIE, you get the position of the first reference object (light house) in SteamVR. Also the orientation of the headset is in relation to the direction the light house is pointing.


Research if it is possible to serialize Rust closures and load them in another process

To match the ad-hoc query capabilities of Spark many users will expect to be able to apply functional transformations to DataFrames using custom Rust code without having to package that code in a crate dependency behind a UDF trait.
Is this even possible with Rust?

Based on feedback, this is actually a feature of Spark that we should not replicate.
Putting user-code behind well-defined traits and packaging up as crate dependencies is a much cleaner model.


docs: update expired slack channel links

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind documentation

What this PR does / why we need it:
update expired slack channel links in the ReadMe file

/lgtm
/approve


Increase the default Camera3D field of view to 75

A vertical FOV of 74 degrees is roughly equivalent to a 90 degree horizontal FOV on a 4:3 display (~106.26 degrees on 16:9), which is the typical default FOV used in PC games. Low FOV values can result in motion sickness among some players, so it makes sense to default to the most sensible value on desktop platforms.
Note that this doesn't apply to the in-editor camera which keeps its FOV to 70. This is because it doesn't display in fullscreen; its viewport only displays in the center of the editor (roughly). This means the viewport won't cover the viewer's eyes as much. Therefore, the editor camera FOV should be slightly lower to account for this.
Since this changes the default value, this may break existing projects slightly.
For the record, this was already done in https://github.com/godotengine/godot-demo-projects/pull/260 for the official demo projects.

While I understand the magic number of 74 being what makes the 4:3 FOV be 90 degrees, the exact value of the horizontal FOV doesn't really matter and I think it would make sense if the value the user sets was as human-readable as possible. So perhaps 75 or 80 would be better.

@aaronfranke The exact vertical FOV for a 4:3 display would be 73.74, but this will look even weirder in the inspector :slightly_smiling_face:
We could use 75, but this would make the default FOV a tad higher than it is in most PC games/engines by default. It may or may not actually be a problem, though.

Just some anecdotal evidence, I usually use about 85 to 90 vertical FOV when I play Minecraft and other games, and most people I know increase the FOV somewhat. A default of 75 is very sensible.

I increased the default FOV to 75 by @aaronfranke's request.

Thanks!


Change the volume at which the highlight sounds will be played at.

You can change the volume of chatterino in Windows

@fourtf in Chatterino 2.1.5 (commit c6e1a1de2 ?

In the windows soundmixer, yes

@fourtf https://i.imgur.com/ihaQmVR.png

I don't have a windows instance right now to check but I thought that it showed there. Maybe there needs to be a sound first before it is recognized? Or maybe I am misremembering

Did this work? If it did then we should definitely make it show from the start.

@fourtf yes


[DPUL/staging] Redis::CommandError: MISCONF Redis is configured to save RDB snapshots, but it is currently not able to persist on disk. Commands that may modify the data set are disabled, because this instance is configured to report errors during writes if RDB snapshotting fails (stop-writes-on-bgsave-error option). Please check the Redis logs for details about the RDB error.

sudden priority: we've gotten nearly 17 thousand of these in the past 2 days.
Backtrace
View full backtrace and more info at honeybadger.io

staging disk was full, passenger couldn't find js runtime, probably stemmed from testing ways to deploy without compiling assets on a box. @tpendragon resolved.


Make the table component headless

Closes #1907
Description
Currently, the @yamada-ui/table component has minimal styles set within the component's source code.
We want to make this style transferable to the theme and make the component headless, enabling more flexible UI customization.
Is this a breaking change (Yes/No):
No

I think you need to take the following actions.

Put “@yamada-ui/theme”: major into changeset
Add “w: ‘100%’” to package/theme/src/components/table.ts

I think you need to take the following actions.

Put “@yamada-ui/theme”: major into changeset
Add “w: ‘100%’” to package/theme/src/components/table.ts

I'm sorry. I corrected these problems.

Thanks!


fix(userspace/engine): avoid reading duplicate exception values

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area engine
What this PR does / why we need it:
The recursive function that reads rule exception values does not clear out the value list, so consecutive list-type exception values may end up containing in duplicate items.
Which issue(s) this PR fixes:
Special notes for your reviewer:
A minimum example to reproduce the issue is reported below:
- rule: Sample Rule
  desc: A sample rule
  output: Sample output (type=%evt.type)
  priority: ERROR
  condition: >
    evt.dir = <
  exceptions:
    - name: sample_exception
      fields: [evt.type, proc.name]
      comps: [in, in]
      values:
        - [[openat], [ls]]

This rule and its exceptions will be compiled into the following condition (note the rolled-over duplication of openat):
(evt.dir = < and not (evt.type in (openat) and proc.name in (openat, ls)))

With this fix, the rule is correctly compiled as:
(evt.dir = < and not (evt.type in (openat) and proc.name in (ls)))

Does this PR introduce a user-facing change?:
fix(userspace/engine): avoid reading duplicate exception values

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.33.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: c4154b1b53ee51904e1208d495e94e2d110999cd

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Technically speaking, the implementation of decode_exception_info_entry is error-prone.

The return value is ignored at the root level,
a case where tmp.items.empty() is ignored (left to the clients)
node checks only at the root level, no validation after in the tree branches and leafs
multiple copying of tmp objects, even with possible moving it will require a lot of allocations for the inner entry structures;

std::unique_ptr<rule_loader::rule_exception_info::entry> decode_exception_info_entry(
	const YAML::Node& node)
{
	using entry_t = rule_loader::rule_exception_info::entry;
	if (node.IsDefined())
	{
		if (node.IsScalar())
		{
			std::unique_ptr<entry_t> ret(new entry_t());

			ret->is_list = false;
			if (YAML::convert<string>::decode(node, ret->item))
			{
				return ret;
			}
			return {};
		}
		if (node.IsSequence())
		{
			std::unique_ptr<entry_t> ret(new entry_t());
			ret->is_list = true;
			for(const YAML::Node& v : node)
			{
				auto item = decode_exception_info_entry(v);
				if (item != nullptr)
				{
					ret->items.push_back(std::move(item));
				}
			}
			return ret->items.empty() ? std::unique_ptr<entry_t>() : std::move(ret);
		}
	}
	return {};
}

same for read_rule_exceptions function
rule_exception_info, rule_exception_info::entry should have their constructors with throwing validations.


Grep Speed Questions

I'm using the latest (eb3d2d1).
I'm working on a very large codebase and do the following:
:FzfLua live_grep
ObjObjects -- *.cpp

It's taking about 1:20 minutes. Doing this at the root of the same codebase:
time rg -g "*.cpp"  ObjObjects

Takes less than 1.5 seconds.
Do you have any suggestions to figure out what's going on here? This is pretty unusable at the moment... Thanks!

time rg -g "*.cpp"  ObjObjects

This is probably not the same command run by FzfLua, run :FzfLua live_grep debug=true and inspect the underlying command at the top and see how the rg flags differ, you are most likely running different flags like --hidden or --follow which would have different time than vanilla rg.
If you’re running the exact same command generated by FzfLua and still see difference in the timing disable git_icons and file_icons both can sometime add processing time, especially git_icons in huge project as it requires testing every match against the git status output.
If all else fails please post the wrapper command that appears in :messages when using debug=true and we’ll try to run it in the shell directly (with minor modifications) and understand what’s causing the issue.

Well, it's fast today. Not sure what was going on yesterday but now I know what to do to debug it if it happens again.
Thanks ibhagwan.

Well, it's fast today. Not sure what was going on yesterday but now I know what to do to debug it if it happens again. Thanks ibhagwan.

Ty @mikesart, lmk if this happens again, I have "zero tolerance" policy for performance issues :-)

[..] I have "zero tolerance" policy for performance issues :-)

Best nvim-plugin maintainer identified during first-half 2022.

[..] I have "zero tolerance" policy for performance issues :-)

Best nvim-plugin maintainer identified during first-half 2022.

Ty @nyngwang for always being so supportive :)


Possible to update to latest symfony/event-dispatcher package in composer.json?

Hi!
Latest version of Laravel uses "symfony/event-dispatcher": "^5.4|^6.0|^7.0",
But your package is limited to:
"symfony/event-dispatcher": "^3.1|^4.0|^5.0|^6.0"
Would it be possible for you to update?
Thanks.

Not relevant. For some reason composer had locked symfony/event-dispatcher to 7.0.2.  Managed to downgrade it again to 6.x as default in Laravel 10 and now it works. So not sure what bumped it to 7.02, but probably some other packages that supported 7.x and no package that needed lower until I tried to install your package.
Maybe still valid to update composer so it CAN run on 7.x, as I see other packages in laravel support 6 and 7.
(master)> composer why symfony/event-dispatcher
symfony/console     v6.4.1 conflicts symfony/event-dispatcher (<5.4)
symfony/http-kernel v6.4.1 requires  symfony/event-dispatcher (^5.4|^6.0|^7.0)
symfony/mailer      v6.4.0 requires  symfony/event-dispatcher (^5.4|^6.0|^7.0)
But not critical issue any more.


Fenny: Precourse Setup

[x] Create this issue
[x] Look through the guides in your class repo
[x] Install softwares
[x] Install VSCode extensions
[x] Organize my computer
[x] GitHub setup
[ ] Prep work for Workflows Week 1

Personal
What was not clear, where did you get stuck?
What was clear, what did you master?
Where can you still use some help?
Where can you help others moving forwards?


SOLID POD: Make use of the updated token refreshing API in solid-auth

solid-auth 0.1.17 updated the token refreshing API with additional arguments. Update solidpod to make use of it appropriately.

Pull request created #108


Android – Back button redirect to LHN from IOU report preview in WS and unable to open WS chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.36-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open New Expensify app.
Log in
Request money from your own workspace;
Navigate to the workspace chat;
On the expense report preview, open the Settlement button's dropdown;
Select an option different from the currently selected;
Click on the report preview to open the Expense Report view;
Tap on the Back button arrow on the header

Expected Result:
Back button redirects to parent WS chat from expense report preview
Actual Result:
Back button redirects to LHN from expense report preview in WS and unable to open WS chat again
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/b5c3f7e3-0b50-475a-bd0e-14ff72de6ba1
View all open jobs on GitHub

We think that this bug might be related to #wave6
CC @greg-schroeder

Part of https://github.com/Expensify/App/issues/35626 and being addressed


The memory required for the training model is equivalent to the size of “model.ckpt-0.data”？

I want to allocate enough memory space in advance for my model.
The size of "model.ckpt-0.data" is 1GB,
Does it mean that the memory consumed during training is about 1GB.

You could try this on colab before allocating memory locally.

@AI-Friend
If you want to use colab, you may select gpu by changing runtime type and run your code.

Thank you，I see!
I have this problem：“Allocation of<PHONE_NUMBER>6 exceeds 10% of system memory.”
Why just over 10% of the memory, the model can not be trained?

If you can use estimators then it provides a handy api tf.estimator.BestExporter which can be useful in your case.
You can also see external contributors https://github.com/vonclites/checkmate wrapper for tf.train.Saver api.


No obvious efficiency improvement

I compared ANN with THUMT, trained on 1 million cn-en corpora with base transformer.
However, there is no obvious improvement of translation time. They both spent 300-400 sec to translate 10000 test data.
So what parameters may I change or what actions to take to differentiate the decoding cost?

Hi @NLPpupil,
As I stated in the README file Discussion section, the reported acceleration (4~7) is against an un-cached Transformer. The new THUMT has already been engineered with good cache strategy, and under this case, the acceleration can only reach 20%~26% on NIST Chinese-English translation task (see #1 for more details).
I am not sure about your exact experimental settings. For better efficiency, you can disable the FFN activation using the use_ffn option, i.e. use_ffn=False.

Thank you. Anyway, I found the decoding is much faster than Tensor2Tensor (10 times +), but the training cost is greater than t2t (10 times).

Hi,  it's unreasonable that the speed difference between Tensor2Tensor and Our model at training phase reaches 10 times.
Did you use only 1 GPU card for our model during training? In the README file, I provided my used training settings, where only 1 GPU card with 8 sequential updates is used. However, this is because of our device limitation. To enable multi-gpu card training, you should set batch_size=3125, device_list=[0,1,2,3,4,5,6,7], update_cycle=1.

I didn't use the parameters in readme because I want to control the resource to single GPU although we have multiple GPU. What's more , I also set constant_batch_size = True to compare THUMT and Tensor2Tensor.

ok, two points:

If you just want to test the performance on a single GPU with constant_batch_size, remember to set update_cycle=1.
If you already set update_cycle to 1 and there is still huge speed difference, perhaps you should raise a new issue  below the THUMT.

Alright, I did set updata_cycle=1 when using single GPU.

I have implement the AAN in Tensor2Tensor, and find the training speed and learning curve are similar as the paper have demonstrated. but in decoding, whether i set the num_layers to 1 or 6, the speed is almost the same. I think perhaps because my dataset is chitchat conversation where the length of the src and tgt sentence is shorter(usually < 20 words) and the tgt is always shorter than src.
Other, for my consideration, Tensor2Tensor use cache in Transformer by saving the k/v vector of all previous y_input, so it just cost time to calculate current k/v/q and dot-product attention. Instead of these, you just accumulate the y_input, which reduce calculation, but you have to add another FFN and Gate layer to ensure the presentation ability. So its hard to say AAN is more efficient than origin, especially using cache.
looking forward to your reply >.<

Hi @Qznan,
Firstly, for AAN, the longer the target sequence compared to its source counterpart, the better acceleration can be reached. The most important thing is not the absolute length of source sequence or target sequence, but their length ratio. So, If your target sequence tends to be shorter than its source part, the acceleration can be degnerated.
Secondly, we also conducted experiments to compare with the cached Transformer. You are right on that partial advantage of AAN over the origin disappers, but AAN still is more effecient than the origin. We can still get an approximate acceleration of 10%. Without FFN, which we suggested, it can reach 25% at least.
In addition, when your engineering is excellet, you can get a good acceleration of ~200%. Hope it can help you.

Thank you for your reply. I will try on AAN without FFN. Anyway it's an excellent work!
By the way, I have one more question, you said the acceleration is relative to the length of source sequence, but ANN just optimal the decode self-attention layer, do not influence the cross-attention layer between source and target sentence. In other words, I think it cost same time in the cross-attention layer in both origin and AAN,  let alone to the origin FFN layer above them. So the acceleration is just relative to the target sequence length?

Actually, the acceleration is relative to the length ratio between the target sentence and the source sentence, i.e. (n_tgt + n_src) / n_src. You can find an informal explanation at https://github.com/bzhangXMU/transformer-aan#discussions


Css not attaching to view

I'm not an experienced Angular User and it is entirely possible I am making a silly mistake.
Here is my issue:
I am using angular ui router, and I wish to use angular-ui-router-styles to attach specific css files to specific views. However, the css file is not being called. The ui-router is working  fine, and I receive no error messages in console, moreover when I type an invalid file address into the css section of aboutUs I still do not receive an error message, leading me to believe that the css file is never searched for.
Here is my config:
app.config( function($stateProvider, $urlRouterProvider) {
$urlRouterProvider.otherwise('/home');
$stateProvider
.state('home', {
url: '/home',
templateUrl: '../views/home.html'
})
.state('aboutUs', {
url: '/aboutUs',
templateUrl: '../views/aboutUs.html',
data: {
css: '../styles/test.css'
}
});
});
I've already injected uiRouterStyles into my module, and called the script.
Thanks for any help you can give!

Is the link element injected into the html ? Open the developers tools on your browser and check it.

No its not injected into the html.

Dunno, try to write a punklr with your code.
In these days I've wrote some tests and they are all green https://travis-ci.org/manuelmazzuola/angular-ui-router-styles

Had the same problem, make sure your ng-app is on the html tag; http://stackoverflow.com/questions/23061958/how-to-use-head-tag-with-angular-directive#23088982

That  fixed it, thanks!

How you fixed it ?


Create postalcodeDK.js

First time doing a pull reques. Am I doing it correctly?

looks good so far thx.
After having a few tests, I am fine to merge it.
see https://github.com/jzaefferer/jquery-validation/blob/master/test/methods.js#L743 for a example

Closing because of missing tests. Feel free to add those, we can re-open again.


Enhance Website Layout with Football-Themed Elements

Problem Statement
The current layout, while functional, lacks a strong visual connection to football. Enhancements inspired by football themes can make the design more engaging and immersive.
Proposed Solution
Refine the existing layout by adding subtle football-inspired elements:

Background: Replace the solid background with a subtle grass-textured or football-field-like design.
Section Lines: Use football pitch markings to visually separate sections.
Hover Effects: Add hover animations to buttons and interactive elements that resemble player or ball movements.
Icons: Replace generic icons (e.g., for the filter and add buttons) with football-themed icons such as balls or goalposts.

Task Details

Replace the background with a light grass-texture design to create a football pitch ambiance.
Add line markings (similar to those on a football field) to visually separate sections like the player selection area and filters.
Enhance the filter buttons (A, M, D, G) with football-related visuals, such as player silhouettes or positional symbols.
Update the "Selected Players" section to visually resemble a dugout or substitute bench.
Add hover effects to buttons (e.g., the "+" button) to create movement or highlight animations.

Acceptance Criteria

The layout should feel connected to football without overwhelming the functional aspects.
Background and line markings should not interfere with readability.
Buttons and filters should remain intuitive and visually appealing.
The design must be responsive and maintain usability across devices.

Priority
Medium

i would like to contribute to this project

Interested to work on this @TharunKumarrA

assigning is based on first come first serve basis. So this issue is assigned to @vijaysanthoshp.
@punithsai18 I hope you can find other good issues created by others.

what's the deadline @TharunKumarrA

Jan 9th tentative @vijaysanthoshp


Add support dataset with multiple extensions

and,

Added and move from constant vars some arguments to some script for convenient
Added the list of extensions for specify using dataset to configs, as "extensions"
Allowed directory names to be separated by a underscores "_" or hyphens "-" when specifying speaker IDs.

Regards.

I noticed that you added options about the audio format in preprocess.py, but also in the yaml configuration file, is this a duplicate.? When actually preprocessing, is it now working according to the command line parameters or according to the configuration file ?

Another problem is that if the names are the same but the format is different, for example, 'abc.wav' and 'abc.flac' coexist, then the preprocessing will generate 'abc.npy' with the same name, and there will be conflicts. ..... I feel that if we want to fully support multiple formats, there will be a lot of code to be changed, and a lot of testing is required

Thanks for your review and reply!

When actually preprocessing, is it now working according to the command line parameters or according to the configuration file ?

now, It according to the config file ones.
Since the same configuration file is used when preprocess and training, I thought it had better write the settings in these configuration files.

I noticed that you added options about the audio format in preprocess.py, but also in the yaml configuration file, is this a duplicate.?

That's right, I forgot that I added options for testing. I'll remove options (arguments).

Another problem is that if the names are the same but the format is different, for example, 'abc.wav' and 'abc.flac' coexist, then the preprocessing will generate 'abc.npy' with the same name, and there will be conflicts.

I see, I'll thinking some better way and trying implement. (easy way is 'abc.npy' to 'abc.wav.npy', or 'abc.flac.npy' ... for now I thought.)

pushed. I tested draw, preprocess, train_diff, train and main_diff. It seems to be no errors/problems.
Could you please review the code?

P.S.:
Using about 200 files, 2 speakers with .wav, .flac, .mp3 and .opus formats when tested.
configs:
data:
  f0_extractor: 'parselmouth' # 'parselmouth', 'dio', 'harvest', or 'crepe'
  ...
  valid_path: ../ddsp-svc-data/testrun-mini/val # Create a folder named "audio" under this path and put the audio clip in it
  extensions: # List of extension included in the data collection
    - wav
    - flac
    - opus
    - mp3
model:
  type: 'Diffusion'
  ...


updated maptask log links to use original index

TL;DR
Updated the maptask log links to use the subtask original index rather than the execution index.
NOTE: This change means that log links will not match the pod name in scenarios where the subtask execution and original indicies differ (ex. cache hits).
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[x] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
^^^
Tracking Issue
NA
Follow-up issue
NA

merge?

Hi, we are moving all Flyte development to a monorepo. In order to help the transition period, we're moving open PRs to monorepo automatically and your PR was moved to https://github.com/flyteorg/flyte/pull/4126. Notice that if there are any conflicts in the resulting PR they most likely happen due to the change in the import path of the flyte components.

Hi, we are moving all Flyte development to a monorepo. In order to help the transition period, we're moving open PRs to monorepo automatically and your PR was moved to https://github.com/flyteorg/flyte/pull/4134. Notice that if there are any conflicts in the resulting PR they most likely happen due to the change in the import path of the flyte components.


3016 error on LG & Samsung devices for few HLS contents

Have you read the FAQ and checked for duplicate open issues?
What version of Shaka Player are you using?
4.3.2
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Both
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Chrome version (38 used by LG 2017 )
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Samsung 2017
Model : UA49MU6470
Firmware version : T-KTMUABC-1310.1
LG 2017
Model :  43LJ554T-TA, WebOS : 3.9.2
Firmware version : 06.10.35
What are the manifest and license server URIs?
Manifest URI:  https://hssportsprepack.akamaized.net/videos/cricket/null/1260140426/in/hin/v1/hevc/hls/plain/master_T833ndY7PdhuAlMM_sd.m3u8??hdnea=st=1563797919~exp=1684757919~acl=/*~hmac=b73630d6e7cd1743cade093da2ad5423fa3d3260b81b17a74d8f9cd4380dcaa0
What configuration are you using?  What is the output of player.getConfiguration()?

What did you do?

Play this content on given LG & Samsung Devices
Video doesn't play & shaka throws 3016 error
Same content plays in new LG & Samsung models

What did you expect to happen?
Playback to play without ay error
What actually happened?
Few HLS contents are throwing 3016 error on older devices/browsers.
Same content plays on newer devices .
While playing on chrome older version  , we see
'Possible encoding problem detected!','Unexpected buffered range for reference'
Video doesn't play  here as well

Can you check again against the main branch? Thanks!


Added Gemfile to serverspec verifier

When running init with the serverspec verifier, added a Gemfile
with necessary serverspec dependencies.

👍


Include //  at top of all generated files

Motivation
We're attempting to use null reference checking and forcing null-reference checks to be compile errors, however generated files do not abide.
That would be fine, as warnings are usually ignored in generated files but the KontentModelGenerator currently does not mark them as autogenerated, and there is no option to enable such a feature.
Proposed solution
//--------------------------------------------------------------------------------------------------
// <auto-generated>
// "This file was generated by KontentModelGenerator v1.x.x etc"
// </auto-generated>
//--------------------------------------------------------------------------------------------------

Include this comment at the top of all generated files. This is something that already happens on Xperience for page types/custom tables etc.

See mentioned PR #142 for a potential solution for this.

Hello @jrkd,
thank you for submitting the issue. We will discuss the issue further. Once we have the result I will let you know.

Hi @jrkd,
We have discussed the feature and it looks like a really great improvement, we would love to see in our product. I've left you some notes in the PR #142


[995] Reactivate our nudge emails

Context
One nudge email was disabled in the post-continuous applications world.
Others have stopped working because they relied on a bool that no longer gets updated.
Our data showed that these had a (more than) modest impact on candidate engagement, including submission and offers.
Changes proposed in this pull request

Reinstate the nudge_submitted emails
Remove the reliance on the course_choices_completed bool for determining if application_choices exist in the other nudges.

Guidance to review
Link to Trello card
https://trello.com/c/EBxd9DgU

 Reactivate our nudge emails to candidates close to completing their application forms