Finalized manuscript draft

Target date: August 7
Working Draft

@ainfeld - can this be closed?

@brianraymor let me confirm with Bailey since she was the owner of this. I just added the issues because she was having a problem accessing Zenhub. I should've noted the real owner of these. Apologies for the confusion.


Extend the implementation of size precomputation for binary codecs

The implementation now makes a distinction between three states of knowledge about the size of a codec:

Static n: the encoding always occupies exactly n bytes;
Dynamic f: the encoding has variable size, pre-computed by f;
Unknown: the encoding size cannot be efficiently pre-computed.

This allows some of the work in determining the size of a codec to be computed at specialisation time, cutting down significantly on the allocation cost.
Additionally, the implementation is now also able to efficiently recover the size of an encoding given a pointer to the start of that encoding (i.e. without actually decoding the value). In this case, Dynamic f is a function that decodes the size of an encoding, and Unknown signifies that this cannot be done (e.g. for unboxed values).
Fixes https://github.com/mirage/repr/issues/65. Depends on https://github.com/mirage/repr/pull/68.

Performance change to the existing Repr.size_of is mostly within noise, but there are some wins due to not using an option monad:
/size_of/pair_string_int/
  minor-allocated (words)  10 -> 2

/size_of/record/
  minor-allocated (words)  34     -> 2
  monotonic-clock (ns)     38.627 -> 16.911

/size_of/tree/
  major-allocated (words)  73.6    -> 0
  minor-allocated (words)  327_648 -> 131_067
  monotonic-clock (ns)     591_883 -> 442_304

/size_of/triple_short_int/
  minor-allocated (words)  20 -> 2

I've added f7d1a9a to this PR, which is a small fix necessary to get Irmin's CI to pass upstream here: https://github.com/mirage/irmin/pull/1459.

Thanks all for the reviews.


[AutoPR cognitiveservices/data-plane/LUIS/Runtime] change api path

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7316

(message created by the CI based on PR content)

no action until Azure/azure-rest-api-specs#7316 is merged


Random passwords for users

Some services on the cluster are going to start wanting passwords for users (e.g. JupyterHub) and currently we generate password-less accounts and only offer key-based login.
I propose a system like the following:

We create a user account and give access to SSH keys as given (as now)
We also set a random password in LDAP and put that password in a user-readable file in their home directory
We add a script which reads that file and prints it for the user
We make sure that password-based SSH login is disabled

Open questions:

Should we let users change their passwords?

All users now have a random password set which ends up in ~/.password. They cannot change this and it is only used for JupyterHub if installed.


Network: Only validate OVN IPv6 ranges overlap with DHCPv6 ranges when stateful DHCPv6 in use

@stgraber ignore this PR, accidentally based it of a WIP branch as was trying (unsuccessfully) to multi-task.

@stgraber ignore this PR, accidentally based it of a WIP branch as was trying (unsuccessfully) to multi-task.


Update nosleep to not use google code

See #10461

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


Show whitespace trailing warning everywhere except Markdown filetype

environment

vim: Vim 9.0
vim-airline: GitHub VimPlug Clone
OS: Linux
Configuration: link

expected behavior
I want to see whitespace trailing warnings everywhere except in Markdown files.
I'm moving through files using Vim's edit command.
When i edit some source file i want to see these warnings, then i go to the Markdown file and want to turn them off, and then i go to the source file back on and want to see the warnings again.
How i can reach this behavior?

Have you checked the help?

Am 22.10.2022 um 17:46 schrieb Magenta @.***>:

environment
vim: Vim 9.0
vim-airline: GitHub VimPlug Clone
OS: Linux
Configuration: link
expected behavior
I want to see whitespace trailing warnings everywhere except in Markdown files.
I'm moving through files using Vim's edit command.
When i edit some source file i want to see these warnings, then i go to the Markdown file and want to turn them off, and then i go to the source file back on and want to see the warnings again.
How i can reach this behavior?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Uh-Oh... I did not read it carefuly.
This line seems to solve my problem.
let g:airline#extensions#whitespace#skip_indent_check_ft = { 'markdown': ['trailing'] }


energy_sensor_precision being rejected by HA

Using a fixed sensor but I'm unable to set the decimal place rounding using energy_sensor_precision, HA declares this invalid. Your docs point to official HA docs which suggest using "round" but that is also rejected.

What does your configuration look like?

Hey, thanks for the reply. I actually realised it was my mistake. I didn't spot that it was in the global settings and was trying to add energy_sensor_precision as part of the sensor config.
Sorry for bothering you, powercalc is absolutely brilliant by the way. Nice work. I've used it to monitor my EV charging. I'm struggling to make a template sensor with a cost calculation at the moment. Would love to see that added to powecalc in future.


example - aspnetcore mvc with framework provided dependency injection

Hi @leosperry
Great work, nice and lightweight scheduler!
Could you provide an example of registering Chroniton and running it inside aspnetcore mvc project, with the framework provided dependency injection mechanism?
Thanks.

Thank you so much for your interest. I've added a wiki page which should fit your needs.
https://github.com/leosperry/Chroniton/wiki/Example-ASP.NET-Core

@leosperry  appreciate your quick answer and feedback in form of additional wiki page.
The cause of asking this question was an issue with dbcontext accessed by multiple threads.
I will try to get you an example code to reproduce the issue I was facing in next days, maybe I was doing something stupid or at least wrong ;) ... or I can bring some improvement to Chroniton (?)
I have been registerating the Singularity.Instance as singleton similarly the same way as you did in example code, and the jobs using the IApplicationBuilder.ApplicationServices.GetRequiredServices() (along with dependencies).
Best regards, James.

To be honest, I have very little experience with ASP.NET Core. However, I may have some insight for you. Each time Chroniton runs a job, it will essentially be running on it's own thread. If you have multiple jobs running, you have to be prepared that they could be doing work simultaneously. So, if they are sharing the same objects, you would run into error messages like the one described above. The 2 easiest ways to fix those kinds of problems is to either make sure that each time a job runs, it has it's own instances of the objects it needs or implement locking mechanisms on the shared objects.
Good Luck!

I am not sure if I properly understand the topic of IOC along with Entity Framework but in my case the key to success was to register db context to be transient along with the jobs being transient too.
Thanks anyway and hope to provide you with code (even might be a working solution :) ) anyway soon.


ScrollLock based combo's

Hi @Willy-Kimura,
Would it be possible to catch ScrollLock based keycombo's with your codebase? Currently ScrollLock is detected, but as soon as a second key is pressed, the ScrollLock part disappears.
I've had some users (HASS.Agent) ask for it, and thought I'd ask you first before trying myself.
Thanks again for sharing your amazing library :)

Hey @LAB02-Admin
On detecting the ScrollLock key, I believe the method Control.IsKeyLocked([key]) can do the trick very easily. You can also conditionally combine the key lock's detection with your already registered key combo without registering the ScrollLock key as part of your registered hotkey.
Here's an example you can try:
if ((e.Hotkey == myHotkey) && Control.IsKeyLocked(Keys.Scroll))
{
    // Your actions here...
}


ProcessManagers subscribe to rejections

This PR adds the ability to subscribe to rejections from Process Managers.
Other notable changes:

Added AbstractEntity.stringId()
Improved returning IDs of event consuming entities.
CommandHandlerMethod now sets producer ID in ThrowableMessage for if Entity or CommandHandler thrown the message.

@armiol, now it's ready. PTAL.


Linq to SQL does not take advantage of 'as' aliases

Consider the following LINQ code:
            await _context.Users
                .Include(s => s.Categories)
                .Select(s => new
                {
                    Id = s.Id,
                    Name = s.Name,
                    PointsTotal = s.Categories.Sum(u => u.Points)
                })
                .OrderByDescending(e => e.PointsTotal)
                .ToListAsync();

A user has many categories where he can get points, and for leader board purposes I want to get the sum of the points he has.
Linq to SQL does translate the statement, however it is not optimal:
Provided:
      SELECT [s0].[Id] AS [Id], [s0].[Name] AS [Name], (
          SELECT SUM([s].[Points])
          FROM [UserCategories] AS [s]
          WHERE [s0].[Id] = [s].[UserId]) AS [PointsTotal]
      FROM [Users] AS [s0]
      ORDER BY (
          SELECT SUM([s1].[Points])
          FROM [UserCategories] AS [s1]
          WHERE [s0].[Id] = [s1].[UserId]) DESC

Expected:
      SELECT [s0].[Id] AS [Id], [s0].[Name] AS [Name], (
          SELECT SUM([s].[Points])
          FROM [UserCategories] AS [s]
          WHERE [s0].[Id] = [s].[UserId]) AS [PointsTotal]
      FROM [Users] AS [s0]
      ORDER BY [PointsTotal] DESC

Using Microsoft.EntityFrameworkCore.SqlServer 3.1.0 package

@maumar to look for duplicate or put on backlog.

dupe of https://github.com/aspnet/EntityFrameworkCore/issues/18775


Improve your survey question comprehension in codelab- Moving Augmented Images -https://codelabs.developers.google.com/codelabs/augimg-intro/index.html?index=..%2F..index#0

Page of this issue:
https://codelabs.developers.google.com/codelabs/augimg-intro/index.html?index=..%2F..index#0
The text of the second question in this codelab asks:  "Do you plan to write sample code in this codelab or just want to read these pages?" and the answer is a "Yes or No"   The format of the question needs to be modified to make it able to be understood better by the reader.
Suggested fix : remake the question to "Do you plan to write sample code in this codelab?  Choose "No" if you just want to read these pages.

Nice catch! Thanks, I'll update the site.

Done.


Refactor/payment gateways

Task linked: CU-6pkdg1 Update payment methods in SDK

Task linked: CU-6rmy8p Query for payment gateways from checkout object instead of shop object


Add Algolia to companies

Adds or updates the following companies, events, or universities:

Algolia

Checklist
Company updates

[x] I have put the most recent relevant date in the "Last Update" column
[x] Internal communication + public announcement https://goto.algolia.com/webmail/139121/395321800/88eb68f22009368ea7c7f5d44fde3164779debc072bd61b6bcf7428a362e0bfa

I think you have the name of the company wrong? Feel free to re-open after fixes, thanks for your patience.


feat: expand land enumeration

Expand Land Use Enumeration
Issue
Closes #81
The current LandUse enum is limited to a few categories, and there’s a need to expand it with more specific categories. This PR expands the LandUse enum and introduces a mechanism to make it easier to add new categories in the future without breaking existing functionality.
Changes
1. Expanded LandUse Enum
2. Dynamic Category Extension Mechanism
To ensure flexibility, a mechanism is implemented that allows adding new categories to the enum dynamically, while maintaining backward compatibility.
3. Updated Land Registration and Update Functions
The land registration and update functions have been refactored to support the new land use categories.

Validation logic is updated to ensure that the new categories are properly recognized and accepted.

4. Tests
All relevant tests have been updated to cover the new land use types:

Unit tests were updated to ensure the new LandUse categories function as expected.

Checklist

[x] Expand LandUse enum with 5 new specific categories.
[x] Implement mechanism for easy extension of categories.
[x] Modify land registration and update functions.
[x] Update unit and validation tests.

@fishonamos pls review


Updated Reef Indices

Initial implementation of Reef Indices and related visualisations

Initial working implementation ready for review. Care should still be taken in use as results not yet verified by @ryanheneghan
Addresses #96

Incidentally @Rosejoycrocker please ignore anything related to heritability/natural adaptation in this PR, it was simply to get these changes to play nicely. Code to do with natural adaptation will be redone in #416

This looks good to me, I'll run tests and try out running the metrics and then merge.


Document.to_json converts tags wrongly

from docarray import Document
print(Document(
    tags={'i': 1},
).to_json(exclude_none=True))

returns {"id": "72ac6f45b342d065839464d56bec7af8", "tags": {"i": true}}
it should return {"id": "72ac6f45b342d065839464d56bec7af8", "tags": {"i": 1}} instead

I am not sure we can solve this
by adding in pydantic_model
_ProtoValueType = Optional[Union[int, bool, float, str, list, dict]]

Then  the code from above prints
{"id": "c801e02ecc274ce09f7ec4c8e5d20ff5", "tags": {"i": 1}}

But then we introduce the issue that
d =Document(tags={'i': True})
print(d.to_json(exclude_none=True))

d =Document(tags={'i': False})
print(d.to_json(exclude_none=True))

prints
{"id": "46e3c50762b8c0b1c0745d4b176f9ef2", "tags": {"i": 1}}
{"id": "c0b613146f3bbd38fd762958482cab31", "tags": {"i": 0}}

Any ideas on how to handle int 1/0 and True/False @hanxiao ?  At a first glance it seems 'in-decidable' from reading the variable in memory.

upstream? maybe post it to pydantic

i believe if you change the order inside union the result will also change. that's why i say it is an upstream issue

I agree with @hanxiao and I think there is very little one can do.

https://github.com/samuelcolvin/pydantic/discussions/4018

Actually, we already discussed this issue before  https://github.com/jina-ai/docarray/issues/72#issuecomment-1018225292

closing this task since it's an upstream issue


data-generation sam_hq_vit_h.pth

During the data generation preparation process, the command wget https://huggingface.co/spaces/xinyu1205/Tag2Text/resolve/main/tag2text_swin_14m.pth needs to be executed. However, when running python run_with_submitit_generate_caption.py, the parameter passed does not use tag2text_swin_14m.pth, but instead the --sam_hq_checkpoint parameter is assigned sam_hq_vit_h.pth. However, I could not find the download link for sam_hq_vit_h.pth. Could the author provide sam_hq_vit_h.pth, or could you guide me to where I can download it? Or, is it possible to use tag2text_swin_14m.pth as a replacement for sam_hq_vit_h.pth?

Hi, you can find sam_hq_vit_h.pth under https://huggingface.co/lkeab/hq-sam/tree/main
Please let me know if you have more questions.


Support "readonly" as a parameter property.

Versions
typedoc: 0.5.1
typescript: 2.0.10
Steps to Reproduce

Create a project with this file named test.ts:
class Foo {
    constructor(private a: string, readonly b: string) {
    }
}

const f: Foo = new Foo("1", "2");

console.log(f.b);

Run typedoc on it.

Expected Results
The documentation should list both a and b as properties of the class Foo.
Actual Results
Only a is present as property of Foo but b is missing. (They both appear as parameters to the constructor, which is fine.)

Thanks @lddubeau. PRs are welcome.

Is it possible to also use the doc added to the constructor property?
class A {
  constructor(
    /**
     * I want to see this comment in the documentation
     * /
     readonly property: string) {}
}

Is it possible to see the

I want to see this comment in the documentation

in the generated HTML?


How to prepare yourself for a summer vacation?

When you are planning any vacation you need to decide on a place, number of days, budget, bookings and of course your pre vacation to do list. Which includes the shopping of the essentials.
We can manage all the above but the most challenging in terms of choices, availability and affordability is in the shopping category. There is so much to offer. Many things you can buy sitting at home.
Shopping for international products has become very easy. When you go for it in the city malls and market you land up paying exorbitantly. The same product we can get little cheaper online. But the cheapest we can get is from the international markets and in their local shops. This is little difficult when we have to physically go there.  Many shops does not deliver things in other countries.
Common product like large size shoes, plus sizes clothes, camping gears and accessories, electronic items etc. are not easily available and so when you buy it you would like to buy it worth to your penny.
I was in such dilemma when I was planning for my vacation in Tirthan Valley in Himachal Pradesh. Adventurous activities are in abundance here. Fishing, trekking, star gazing, photography etc. I needed a nice pair of trekking shoes, fishing gears, tent to hook it near the water stream, goggles, a nice camera, and some attires to keep me warm. So much in the list. Soon I realized that what I want is not readily available here and is very costly as compared to international market. Also to add to my agony, online shopping from the US and UK did not delivery my buys here to India.
So when I was surfing for online product I came through an app called TRANSIT ADDRESS.  They offer a wonderful solution for us. What attracted me was their tag line – YOU SHOP AND WE SHIP.
That was it. I contacted them and got all the details. It is a very easy process. I have ordered products from the US and my trip is on ….. Tirthan valley here I come.

nice


Fix height of selectize selects when multiple options are selected

When a <select multiple> element is rendered with Selectize, its height may be greater than 40px (imposed by the CSS).
What about setting the minimum height to 40px?
Before:

After:

Let's do this in the bedrock repo - It's never gonna merge otherwise.

Won't we have a new 8.whatever release before 9.0?

Yes, but I'd rather not have to wade through a thousand merge conflicts for
UI issues when I attempt to merge develop into release/9.0.0
On Wed, Apr 8, 2020 at 8:05 AM Michele Locati<EMAIL_ADDRESS>wrote:

Won't we have a new 8.whatever release before 9.0?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/pull/8444#issuecomment-611011953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEA3QJEQOXOOOZKZHAPPA3RLSG7ZANCNFSM4LAPRWAQ
.


Intermittent PrometheusWriter errors.

Occasionally seeing:
[ERROR] PrometheusWriter prometheusUpdateThread_global: Could not format data, reason Cannot set prometheus.Label.value to None: None has type <class 'NoneType'>, but expected one of: (<class 'bytes'>,

Fixed along with other bug fixes over time, closing for now.


doc：fix word misspell

Signed-off-by: raoqi<EMAIL_ADDRESS>Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Hi @raoqi, welcome!
Thank you for your work. LGTM. If the CI passes, I will merge this PR.
Welcome to contribute the pouch!

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1248   +/-   ##
=======================================
  Coverage   15.31%   15.31%
=======================================
  Files         172      172
  Lines       10665    10665
=======================================
  Hits         1633     1633
  Misses       8912     8912
  Partials      120      120

Impacted Files
Coverage Δ

daemon/mgr/spec.go
0% <ø> (ø)
:arrow_up:

Hold on a sec, @raoqi , the Author is unknown? If it's mistake, please let me know. :)

the Author is unknown

I think it is OK.  We do not strict to much on commit sign off.

LGTM


Pre-release version of Kotlin

Please complete the following information:

Library Version [e.g. v0.20]
Affected Device(s) [Samsung Galaxy A53 with Android 12.0]

Describe the Bug:
Getting this error:
is compiled by a pre-release version of Kotlin and cannot be loaded by this version of the compiler
And i have kotlin 1.7.10, compose 1.3.0-alpha03 and compose compiler 1.3.0
Expected Behavior:
Should be able to build the project without issues when adding the orbital library.

Hi @Vivecstel,
a new version 0.2.1 was released. Could you try to build your project with this version? Thanks!

Hmm I am still getting the same error with 0.2.1. It seems weird because I have the same versions as Orbital. Didn't have this issue with your rest compose libraries (landscapist, orchestra).

Hey, thank you for checking this. What about 0.2.2-SNAPSHOT?

It worked with the snapshot release @skydoves .
Thanks.


Example in README does not typecheck

I'm trying to use this package, but, copying the code from the readme leaves me with code that doesn't compile.
Is this still a valid example for a simple usage?

Seems like it actually needs to pass OpenFile, not just File, otherwise it crashes at runtime.

Thanks! OpenFile is indeed necessary. Pushed a fix.


Context images 2D add documentation to the user guide

My actions before raising this issue

[ x] Read/searched the docs
[x] Searched past issues

Recently we have merged new patch, which adds ability to upload additional context images to images for annotations.
Now need to update documentation to show users how these images could be attached.
Reference PR #3122
Please, contact me additionally if you have any questions
Context

@TOsmanov @aschernov
Could you please prepare a patch with documentation to cover the feature?

@bsekachev , sure. Taken in work.

@aschernov
Thx. It is better to start working on it after #3039 is merged.

Closed by #3227


Respect .gitignore file in commits

Solves #1826 and follow-up after server-side change (https://github.com/huggingface/moon-landing/pull/8007 - private link).
I have now added a respect_gitignore: bool = True parameter to create_commit and upload_folder to respect the gitignore file, if it exists.
By default we check if a .gitignore file is present in the commit itself. Otherwise the one hosted on the Hub is used. Only the gitignore file at root is respected (not in subdirectories).
I have adapted the documentation and added a few tests for it.
Note: as mentioned by @julien-c in https://github.com/huggingface/huggingface_hub/issues/1826#issuecomment-1810934088, let's consider this as a "bug-fix" and not a "breaking change" as we can expect that no-one was intentionally uploading ignored files on purpose.

# Test to upload my `huggingface_hub/` folder to the Hub
>>> huggingface-cli upload test_tmp_upload ../huggingface_hub
Skipped upload for 53 LFS file(s) (ignored by gitignore file).
Skipped 597 file(s) in commit (ignored by gitignore file).
https://huggingface.co/Wauplin/test_tmp_upload/tree/main/.

Thanks for the review @LysandreJik! Failing tests are unrelated so I'll merge this one and fix the CI in a separate PR.

sorry i'm late to this, but are you sure we even needed a respect_gitignore param? Feels unlikely that anyone will ever want to set it to False no?

Opened a PR to remove respect_gitignore before it's too late :smile: https://github.com/huggingface/huggingface_hub/pull/1876


Question on speediest

Hi again,
I have a question and was wondering if you could answer it.  When you wrote the speediest-cli application, was there any kind of spec or documentation available on how speediest.net works??  If so, can you point me to it? If its soft copy, could you email it to me?
Thanks

There is no documentation.


[UNDER TEST] domd: update branch and revision for bmap-tools

Intel renamed 'master' branch to 'main' so we
took SRC_URI, SRCREV and BASEVER from the kirkstone's recipe

bmap-tools are required for the creation of the WIC image of DomD.
PR was tested by the build of the meta-xt-prod-cockpit-rcar for h3ulcb-4x2g-kf.

ACK

Ack is added.
Tested.
Ready to be merged.


Retract 1.64

Seems it's broken according to https://github.com/evcc-io/evcc/issues/12136. If unsure if 1.65 fixes the issue it could also help running with golang race detector enabled (-race).

was done!


ax-platform version dix

fixing issue with new ax functionality included in 0.2.2 and beyond. Bumped reqs to reflect this
Closes #194

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 95.295%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1600

Relevant Lines:
1679

💛  - Coveralls


Fix Mobile Search

Search doesn't work on mobile, possibly because v-model binding doesn't work with mobile keyboard (input event not fired)... Uncomfirmed, but it's an issue discussed here: https://github.com/vuejs/vue/issues/9777#issuecomment-478831263
Workaround for that would be to use something like:
:value="query"
@input="query = $event.target.value"

It should be equivalent to v-model but initial testing in browser is.. uhmm.. it's breaking existing search functionality, so warrants investigation.

Fixed by cb38fe9e71e9e20661a69bc3d977fee3ebd8f826

Fixed by cb38fe9e71e9e20661a69bc3d977fee3ebd8f826


mopidy-musicbox-webclient: 2.4.0 -> 3.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mopidy-musicbox-webclient is: "Mopidy extension for playing music from SoundCloud"
meta.homepage for mopidy-musicbox-webclient is: ""
meta.changelog for mopidy-musicbox-webclient is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.1.0 with grep in /nix/store/f6a7riwl0c0dbfcivc0hbpn34g771d4f-mopidy-musicbox-webclient-3.1.0

directory tree listing: https://gist.github.com/f7ee8893aa1f196a8938fa9cde50805b

du listing: https://gist.github.com/14a34c3e5d34218563cae264cd1ce1db

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
mopidy-musicbox-webclient

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f6a7riwl0c0dbfcivc0hbpn34g771d4f-mopidy-musicbox-webclient-3.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mopidy-musicbox-webclient https://github.com/r-ryantm/nixpkgs/archive/e62103ce0844052bb7e3d256b593f1452d67d1ae.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/f6a7riwl0c0dbfcivc0hbpn34g771d4f-mopidy-musicbox-webclient-3.1.0
ls -la /nix/store/f6a7riwl0c0dbfcivc0hbpn34g771d4f-mopidy-musicbox-webclient-3.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

mopidy-musicbox-webclient

Maintainer pings
cc @spwhitt for testing.

Result of nixpkgs-review pr 101702 run on x86_64-linux 1

1 package built:

mopidy-musicbox-webclient


How to set own height for select?

How to set own height for select?
I set width according "data-width" . But I didn't find "data-height" option.

Bootstrap 3 button styles appear to be broken in 1.6.4. v1.6.3 will properly resize the control height based on a data-style, but v1.6.4 does not. You can recreate the behavior by switching between versions on cdnjs.cloudflare.com.
Example:
data-style="btn btn-default btn-xs"

I recently spent an hour trying to find how to restrict the height of a dropdown. Turns out the size attribute for select tags is set to 'auto' in the javascript file. For me, it was around line 369.
The solution is simple:
Set size: '10' //or however many options you want to be listed
$('.selectpicker').selectpicker({
   size: '10'
});

width 100%
<select class="selectpicker" data-width="100%">
  ...
</select>


update tag to 0.18.1

hi, recently we want to add your lib on https://cdnjs.com, could you please update your tag after every releasing so that we can update the lib to the latest version with git auto-update config.
thank you very much!
https://github.com/cdnjs/cdnjs/issues/7451

What kind of tag are you talking about? If you mean git tags, 0.18.1 is not a release, just a dev version, so you don't want that on your CDN.

thanks.


[Spec] Key Handling

The Problem
We have a problem with our KeyEventArgs. The problem is that on win32 the Key is localized to the current keyboard layout whereas on macOS and Linux it is not.
The root of this problem is something that vscode and Chrome went through a while ago and is well described in this issue:
https://github.com/microsoft/vscode/issues/17521
Taking the relevant parts of that issue:

A keyboard layout on Windows consists of two mappings. The first one maps scan codes to virtual keys and the second one maps virtual keys and modifiers combinations to generated characters.

Keyboard layouts on Linux and Mac do not work through this double indirection (from scan code to key code, and from key code and modifiers to character), keyboard input on Linux and Mac goes straight from scan code and modifiers to characters.

The KeyEventArgs.Key property which we inherited from WPF represents a virtual key, but that concept is a win32 concept.
This results in the problem that on win32, our Key property reflects the key after mapping from a scan code to a virtual key code, but on other platforms it represents a scan code.
Scan codes are not localized so we end up with the situation where on win32 the Key takes the keyboard layout into account but on macOS and Linux it doesn't:

https://github.com/AvaloniaUI/Avalonia/issues/11065
https://github.com/AvaloniaUI/Avalonia/issues/10932
https://github.com/AvaloniaUI/Avalonia/issues/9424
https://github.com/AvaloniaUI/Avalonia/issues/2049

The Solution
The W3C spec for KeyboardEvent is a one place to look for inspiration as they've already gone through this.
They have the following attributes on their KeyboardEvent:

The legacy keyCode is similar to what we currently have
key holds a key attribute value corresponding to the key pressed. The key attribute is not related to the legacy keyCode attribute and does not have the same set of values. The values are defined here
code holds a string that identifies the physical key being pressed. The value is not affected by the current keyboard layout or modifier state, so a particular key will always return the same value (i.e. it is the scan code). The values are defined here

This looks like decent place to start. The only complication is that the key attribute is kind of a combined enum (can be a string like Enter, Backspace etc) and unicode key (e.g. A, %, あ) string. That doesn't translate well to C# where enums are not typically strings.
The Proposal

We deprecate KeyEventArgs.Key as its behavior differs between operating systems
We add a ScanCode enum KeyEventArgs.ScanCode property to describes the physical key being pressed which is not affected by the current keyboard layout or modifier state
We add a char? KeyCharacter property which describes the unicode character that the key represents after mapping to the current keyboard layout, if it exists
We add a Key? SpecialKey property which describes special keys not represented by a unicode character, such as (taken from the W3C spec):

Modifier Keys
Enter, Tab keys
Navigation keys
Editing keys (Backspace, Delete, Insert etc)
UI keys (Play, Pause etc)
Device keys (Eject, PrintScreen etc)
IME and Composition keys
Function keys
Multimedia keys
And so on... W3C list is really long

Questions

What about keys which fall into two categories? For example W3C specifies that the string Tab be used instead of the \t character
Is our existing Keys enum suitable for SpecialKey?
Is the proliferation of key-related properties on KeyEventArgs going to be confusing? Can we tweak the API to make falling into the pit of success a little easier?

Note that it would break shortcuts every single language with non-latin alphabet, like armenian, arabic, various cyrilic scripts, etc. With those layouts the localized key would be a non-latin character while shortcuts are still using the latin one written on the same physical key.

Note that it would break shortcuts every single language with non-latin alphabet, like armenian, arabic, various cyrilic scripts, etc. With those layouts the localized key would be a non-latin character while shortcuts are still using the latin one written on the same physical key.

I'm wondering how this is handled on the web? Does anyone with experience of this have any input? (I use a QWERTY keyboard so I'm ignorant of such things).
I notice that in the vscode issue they say:

on OSX and Linux
core actions and extensions continue to define keybindings on virtual key codes.
existing keybindings in user keybindings.json, along with core actions and extensions will be mapped at runtime, using heuristics, and depending on the current keyboard layout, to scan codes.
user keybindings can now be defined using a scan code format.

https://en.wikipedia.org/wiki/German_keyboard_layout

After discussing with @kekekeks he says that this spec will break access key handling on non-latin scripts so I'm not sure we should go ahead with it any further. I'm not sure what the way forward should be. I'd assumed that the w3c would have taken this into account, but it seems not?
If someone knowledgeable about this subject could help out, it would be appreciated, otherwise I'll close this issue in a few weeks.

I'm wondering how this is handled on the web?
Pressing "Z" on:
US keymap: keydown  keyCode=90  (Z)   which=90  (Z)   charCode=0    printed character: z
RU keymap: keydown  keyCode=90  (Z)   which=90  (Z)   charCode=0    printed character: я
DE keymap: keydown  keyCode=89  (Y)   which=89  (Y)   charCode=0    printed character: y
AM keymap: keydown  keyCode=90  (Z)   which=90  (Z)   charCode=0    printed character: ժ
Have fun

Would be useful to know how hotkeys are handled with different keymaps

Testing macOS Chrome, keyCode indeed seems to be localized to the current keyboard layout, so I imagine that since the vscode issuw was filed, they've found a way to map scan codes to virtual key codes on macOS? Need to take a look at the source.

We add a char? KeyCharacter property which describes the unicode character that the key represents after mapping to the current keyboard layout, if it exists

It should probably be either a Rune (not available in .NET Standard), int (unicode scalar) or string, as char won't suffice to represent all Unicode values. Maybe also adjust the property name accordingly, to something like UnicodeScalar if int, or simply Text if string?

What about keys which fall into two categories? For example W3C specifies that the string Tab be used instead of the \t character

I think it should set both SpecialKey == SpecialKey.Tab and KeyCharacter == "\t".

After discussing with kekekeks he says that this spec will break access key handling on non-latin scripts so I'm not sure we should go ahead with it any further. I'm not sure what the way forward should be. I'd assumed that the w3c would have taken this into account, but it seems not?

That means that the existing linked issues won't be fixed, which are real ones. On the browser and with an AZERTY keyboard, I can't use Ctrl+A or Ctrl+Z inside a textbox in the control catalog. Granted, some shortcuts still exist - with the wrong keys - which is better than no shortcut at all for non latin scripts, but still very inconvenient. Note that some are still impossible in this environment: I should use Ctrl+W instead of Ctrl+Z to undo, but that will close the browser tab instead :)
(Not related to Avalonia but there are still a lot of games - it's a bit better nowadays than a decade ago - that force WASD movement without being able to remap the keys, resulting in being unplayable on AZERTY layouts, unless you're an octopus).
I'm also interested in knowing how shortcuts are usually handled (badly, I assume) by various apps for non latin scripts.

I'm also interested in knowing how shortcuts are usually handled (badly, I assume) by various apps for non latin scripts.

The apps use the corresponding latin key and it always just works.

That means that the existing linked issues won't be fixed, which are real ones.

We probably need to have some layot-specific logic to handle that or copy what Chrome does

I think that regardless of the considerations about shortcuts, etc., having access to the raw untranslated scan codes would be useful in many situations. (For example, I'm working on a PC emulator which needs to pass host scan codes through to the VM.)

Ok, so I think the latest on this is:

We still need the Key for shortcuts on non-latin layouts. This will require that we find a way to translate the scancode to a virtual key code on macOS/Linux - we can take a look at chromium/qt to find how they do it
We should add a scan code in addition to the Key for certain scenarios, the most common one being a game's need to use non-localized WASD keys


Request for Evaluation Code for CIDEr and VQA-Score

I really appreciate the work you've done.
I was wondering if you could provide the evaluation code for different scoring methods, such as COCO CIDEr and VQA score.
I would like to reimplement your experiments.
Thank you!

Hi,
First of all, thanks for your interest and appreciation of our work.
Regarding the metrics we used:

For VQAv2 score: We used the official evaluation server on EvalAI.
For TextVQA: We used the local evaluation script in this repo.
For COCO CIDEr: We used the publicly available pycocoevalcap package.


CF bypass Block Problem with Socks4/5

Hello Ruri i tried to use proxy Socks4/5 to checking but i just getting retries, it work well with HTTP proxy type but i think we have a problem with socks list. TnQ for listening

There is already an issue about the fact that CF Bypass doesn't work anymore

Are you sure? Weird It works well for me maybe my case use older version of cloudflare


GET dns txt records from https://dnsviz.net/d/<domain_name>/responses/ as a fallback for unsuccessful dns resolutions.

🗣 Description

Provides another means of getting dns txt records for spf/dkim/dmarc checks, in the environments where dns resolution traffic is restricted.
💭 Motivation and context

To address https://github.com/cisagov/ScubaGear/issues/164, where DNS split horizon and possible network blocks prevent access to necessary dns TXT records.
🧪 Testing

📷 Screenshots (if appropriate)
Uncomment this section if a screenshot is needed.

✅ Pre-approval checklist

[ ] This PR has an informative and human-readable title.
[ ] Changes are limited to a single goal - eschew scope creep!
[ ] All future TODOs are captured in issues, which are referenced
in code comments.
[ ] All relevant type-of-change labels have been added.
[ ] I have read the CONTRIBUTING document.
[ ] These code changes follow cisagov code standards.
[ ] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[ ] Tests have been added and/or modified to cover the changes in this PR.
[ ] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Thanks @c0rgifanclub for this contribution! I've actually worked with DNSViz in the past, it's a great tool. None of this should be news for you, but I'm going to add some context about DNSViz for the other developers. I also have some considerations to share to get the conversation going, discussing some pros and cons of adding this.
This pull request aims to tighten a current visibility gap we have with ScubaGear. Specifically, we noticed during the pilots that for some organizations, Resolve-DnsName would not return any answers for their domains, even though DNS records were published for those domains and could be retrieved when queried external from their network. We've mitigated this by adding a fallback to DNS over HTTPS (DoH). In most cases, this fixes the situation, but some organizations additionally block DoH. This pull request adds a third fallback, namely a web request to DNSViz. DNSViz is a tool, originally developed at Sandia National Labs, designed to allow administrators visualize the DNSSEC chain of trust. Part of DNSViz's output includes a section with the txt records for the domain. The PR adds code which makes a web request to DNSViz and parses the HTML response to extract the txt records.
The main benefit of this PR is that it further restricts our DNS visibility gap and allows users a path forward if their traditional DNS request fails and DoH is also blocked. Other considerations:

Adds latency. I haven't tested it myself, but per @c0rgifanclub's comment, we can expect 9 seconds of latency (I'm assuming per domain, which could add up). Though this would be the 3rd fallback, so hopefully most users wouldn't see this latency.
In most cases where DNSViz is needed, manual intervention will still be required. If DNSViz hasn't see a given domain before, you'll need to manually open the website and trigger an analysis, wait a few minutes, then re-run ScubaGear. That said, this at least gives users a path forward. Our current implementation just displays a warning if applicable but doesn't indicate a way for the users to resolve the warning.
The code does not use an API, but rather parses the HTML, meaning it will break if the web interface is significantly changed. That said, I would consider the web interface to be very stable. I anticipate that any instances of this happening would be rare.

@c0rgifanclub feel free to weigh in on any of the above points if you'd like. We'll then discuss this internally to determine the best path forward.

Thank you again for your contribution, but we will be closing this PR. Ultimately, we elected to not add this for the following reasons:

Java dependency. https://dnsviz.net/ actually uses Java in the browser. Which can result in some fun PowerShell issues and overly complicates ScubaGear's dependencies.
Latency. Assuming you got it working, it would add significant latency, about 9 seconds per domain, representing an obvious scalability concern.
User experience. In most cases where DNSViz is needed, manual intervention will still be required. If DNSViz hasn't see a given domain before, you'll need to manually open the website and trigger an analysis, wait a few minutes, then re-run ScubaGear, a particularly user-costly workflow.

Once again, thank you for your contribution.


Carthage fix

Okay so this time its much better.
This relies on the PR over at geos:
https://github.com/GEOSwift/geos/pull/5

I added a Cartfile which uses the above geos Carthage dependency
I added the xcconfig files to the Repo that way they dont need to be removed from the project
I removed the test target because of this error:

Carthage/Checkouts/GEOSwift/GEOSwiftTests/QuickLookTests.swift:13:18: error: module 'GEOSwift' was not compiled for testing
@testable import GEOSwift

We shouldn't compile the library for Production with @testable enabled so this should be done as part of an App Target or its own Test Target, but not attached to the Framework directly.

I added a Preprocessing Target step which Copies the geos.framework from Carthage to a folder and filename that the Cocoapods setup is expecting and it works. This is skipped if the CARTHAGE=YES environment variable isn't set.

It seems to work great! :)
Also, it should be pretty easy to add the additional tvOS and macOS targets to this library if you wish now that they are part of the geos dependency.
Thoughts?

Okay so I have the tests running again but two are failing:
QuickLookTests
[01:41:44]: ▸ ✗ testDrawGeometryCollection, XCTAssertEqual failed: ("Optional(6088 bytes)") is not equal to ("Optional(6098 bytes)")
[01:41:44]: ▸ ✗ testDrawWaypoint, XCTAssertEqual failed: ("Optional(4514 bytes)") is not equal to ("Optional(4491 bytes)")

Is it possible something has changed in QuickLook?

I'd check to make sure the tests are still running on "iPhone XS" simulator. The snapshot tests are simulator-dependent for some reason.

@macdrevx 😅Looks like it worked. 😎

@macdrevx Anything preventing this from being merged?

I just need some time to review it. Sorry for the delay.

@madhavajay I pushed a branch https://github.com/GEOSwift/GEOSwift/tree/carthage with a different approach. Can you try it out and let me know if it works for you?
Instead of working around CocoaPods, I switched the project to use Carthage for development purposes. CocoaPods is still available for consumers. This seems simpler to me.
The only problem left that I'm aware of is that I get an error when I try running the playground.
error: Couldn't lookup symbols:
  type metadata accessor for GEOSwift.MultiPolygon
  type metadata accessor for GEOSwift.Feature
  type metadata accessor for GEOSwift.Polygon
  type metadata accessor for GEOSwift.Envelope
  GEOSwift.Geometry.convexHull() -> Swift.Optional<GEOSwift.Polygon>
  GEOSwift.Geometry.difference(GEOSwift.Geometry) -> Swift.Optional<GEOSwift.Geometry>
  GEOSwift.Geometry.intersection(GEOSwift.Geometry) -> Swift.Optional<GEOSwift.Geometry>
  GEOSwift.Geometry.pointOnSurface() -> Swift.Optional<GEOSwift.Waypoint>
  GEOSwift.Geometry.__allocating_init(WKT: Swift.String) -> Swift.Optional<GEOSwift.Geometry>
  GEOSwift.Geometry.union(GEOSwift.Geometry) -> Swift.Optional<GEOSwift.Geometry>
  GEOSwift.Geometry.buffer(width: Swift.Double) -> Swift.Optional<GEOSwift.Geometry>
  GEOSwift.Geometry.boundary() -> Swift.Optional<GEOSwift.Geometry>
  GEOSwift.Geometry.centroid() -> Swift.Optional<GEOSwift.Waypoint>
  GEOSwift.Geometry.envelope() -> Swift.Optional<GEOSwift.Envelope>
  GEOSwift.Geometry.mapShape() -> Swift.Optional<__C.MKShape>
  type metadata accessor for GEOSwift.Geometry
  type metadata accessor for GEOSwift.Waypoint
  static (extension in GEOSwift):Swift.Array<A where A == GEOSwift.Feature>.fromGeoJSON(Foundation.URL) throws -> Swift.Optional<Swift.Array<GEOSwift.Feature>>

Do you have any ideas on how to get it working again?
Regarding number 3 in your PR description: I saw that error too, but it was solved by a change to the scheme to prevent the tests from being compiled unless we're actually building for test.

Yes, I think using Carthage is a nicer solution than the above hack but I wasn't sure you would want to overhaul the whole dependency management system.
I have added a PR which fixes the Playground issue.
https://github.com/GEOSwift/GEOSwift/pull/152
Let me know if you have any issues with it! :)

Closing this in favor of #153.


We don't define "breaking change"

https://github.com/dart-lang/sdk/blob/master/docs/process/breaking-changes.md doesn't ever say what a breaking change actually is. For example, does renaming an API that nobody has ever used count as a breaking change? Is a breaking change any theoretical change in behaviour, or is it limited to measurable changes? Is it an empirical definition like Flutter's (a breaking change is one that actually causes one of a specific set of tests to fail), or a principled one (a breaking change is one which a test could be written to detect)?
It would be useful if the policy could have a section at the very top that started with "A breaking change is".
cc @itsjustkevin @vsmenon @leafpetersen @devoncarew @mit-mit

Fundamentally, a breaking change is one which makes a previously correct program become erroneous. At worst, that program stops compiling. (Well, "worst" in some sense, just being subtly wrong in production can be worse in practice.)
Since we can't know all programs, and every every API change in Dart can potentially break some program, with that definition we'd have to treat every change as a potentially breaking change. When we say "breaking change" we actually mean "potentially breaking change", because we can't know if any program is actually broken, so we have to assume that it might be.
Some language changes are also potentially breaking, in that they change the meaning of existing code. And some tool changes can break the build process. Again, if no program ever used that code or that tool, it won't really break anything. We still consider it a breaking change. So, how about:

A breaking change to the Dart language, platform libraries, packages or tools is one which can potentially make an existing
correct and well-behaved program behave erroneously or make it fail to be a valid Dart programs.
This document states when and how breaking changes can be introduced in Dart releases.

We do then go on to say that some potentially breaking changes are allowed anyway, because we don't mind breaking mis-behaving programs ("we told you not to do that!"), or because we consider the benefit to outweigh the imposed migration cost.
Some "non-breaking" changes can still break other people's builds, if those people introduce further restrictions on themselves that the platform doesn't promise not to violate (like, say, making a deprecation warning an error). That's not a breaking change in the SDK.
Every structural API change is a breaking change (possibly except adding a static member or constructor to a class, and if we explicitly tell you to not extend a class, then changing a generative constructor to a factory constructor might be ok).
(Changing timing is ... iffy. We never made any promises about the timing of events, People write code (often just tests) that depend on a specific interleaving of events. We try not to break those programs, but depending on unspecified behavior is a gray area. In general, depending on unspecified behavior is a problem for everybody.)

That definition makes sense, combined with the existing definition of "program" on the page above.
This would mean that, for example, changing out "hashCode" implementation is considered a breaking change (that's consistent with listing changing "unspecified behavior" as something that can cause breakage), is that intended?

I'd probably consider changing hashCode an acceptable potential breaking change, based on it being unspecified behavior. We say (somewhere) that you should not depend on hashCode being consistent between different runs of the same program.
We'd try to not change it unless the reward outweighs the risk.


[Bug] Add Rows Asynchronously Demo is wrong

Steps to reproduce the bug

Open  Add Rows Asynchronously Demo
No empty row provided at the start of the grid

Expected results
Should be provided an empty row at the start of the grid.

hello.
What problems are you experiencing?
When I start the grid with an empty list, which means empty rows, the problem doesn't happen.
Can you provide a minimal sample code that I can reproduce?

Sorry for my poor explanation.
I'm talking about this: https://weblaze.dev/pluto_grid/build/web/#feature/add-rows-asynchronously
The example should have an empty row at the start of the grid right? Or did I misunderstand?

This example shows an example of adding a row asynchronously.
By default, PlutoGrid provides rows on startup with data ready, but how to do otherwise (fetching data from DB, etc.). (from source link)

Okay, clear. Thank you!


canonicalQueries does not work

Environment
StackBlitz
Reproduction
https://stackblitz.com/edit/github-orzjqc-6knjap?file=app.vue,pages%2F[productCategory]%2F[id].vue
Describe the bug
If you go to route /tests/1?customer=any, the canonical tag will be:
<link id="i18n-can" rel="canonical" href="https://example.com/tests/1">
it should be
<link id="i18n-can" rel="canonical" href="https://example.com/tests/1?customer=any">
Additional context
No response
Logs
No response

Thanks for reporting!
Looks like I already fixed this issue but it wasn't released yet, I just published 8.0.1, it should be working as expected after updating 😄. I have updated your reproduction to demonstrate here.


ENH: Migrated partition and collating actions from q2-moshpit to q2-types

closes #332

Moved the actions partition_feature_data_mags, collate_feature_data_mags, collate_ortholog_annotations, collate_orthologs, partition_orthologs, partition_sample_data_mags, collate_sample_data_mags, _validate_mag_ids and  _validate_num_partitions from q2-moshpit to q2-types.
Moved all tests that are associated with these actions from q2-moshpit to q2-types.
Created new tests for the actions collate_orthologs and partition_orthologs.
Renamed collate_annotations to collate_ortholog_annotations.
Changed shutil.move to duplicate in collate_orthologs action to be consistent with other collate actions.
Added tests directory where tests for helper functions in util.py live.

Hello @lizgehret,
I am trying to move these partition and collating actions from moshpit to types. And i am encountering a problem with circular imports. The errors occur when I add the imports in plugin setup. I don't have any experience with resolving this kind of problem and I wanted to ask if you could have a look at the code and help me resolve this issue?
Thank you!
The error is this:
Traceback (most recent call last):
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/util.py", line 279, in get_plugin_manager
    return qiime2.sdk.PluginManager.reuse_existing()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/qiime2/sdk/plugin_manager.py", line 58, in reuse_existing
    raise UninitializedPluginManagerError
qiime2.sdk.plugin_manager.UninitializedPluginManagerError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/bin/qiime", line 11, in <module>
    sys.exit(qiime())
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/builtin/dev.py", line 33, in refresh_cache
    import q2cli.core.cache
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 285, in <module>
    CACHE = DeploymentCache()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 61, in __init__
    self._state = self._get_cached_state(refresh=refresh)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 107, in _get_cached_state
    self._cache_current_state(current_requirements)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 205, in _cache_current_state
    state = self._get_current_state()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 253, in _get_current_state
    plugin_manager = q2cli.util.get_plugin_manager()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/util.py", line 291, in get_plugin_manager
    return qiime2.sdk.PluginManager()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/qiime2/sdk/plugin_manager.py", line 67, in __new__
    self._init(add_plugins=add_plugins)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/qiime2/sdk/plugin_manager.py", line 105, in _init
    plugin = entry_point.load()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2516, in load
    return self.resolve()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2522, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/__init__.py", line 18, in <module>
    importlib.import_module('q2_types.feature_data')
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data/__init__.py", line 11, in <module>
    from ._format import (
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data/_format.py", line 18, in <module>
    from ..plugin_setup import plugin
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/plugin_setup.py", line 17, in <module>
    from q2_types.feature_data_mag import MAG, NOG
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data_mag/__init__.py", line 11, in <module>
    from ._format import (
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data_mag/_format.py", line 11, in <module>
    from q2_types.feature_data import DNAFASTAFormat
ImportError: cannot import name 'DNAFASTAFormat' from partially initialized module 'q2_types.feature_data' (most likely due to a circular import) (/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data/__init__.py)
(qiime2-tiny-2024.2) rischv@fsbt007 q2-types % git commit -a -m "added everything but bug cicular imports"
[332_move_partition_collating 57234cc] added everything but bug cicular imports
 2 files changed, 112 insertions(+), 3 deletions(-)
(qiime2-tiny-2024.2) rischv@fsbt007 q2-types % qiime dev refresh-cache
QIIME is caching your current deployment for improved performance. This may take a few moments and should only happen once per deployment.
Traceback (most recent call last):
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/util.py", line 279, in get_plugin_manager
    return qiime2.sdk.PluginManager.reuse_existing()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/qiime2/sdk/plugin_manager.py", line 58, in reuse_existing
    raise UninitializedPluginManagerError
qiime2.sdk.plugin_manager.UninitializedPluginManagerError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/bin/qiime", line 11, in <module>
    sys.exit(qiime())
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/builtin/dev.py", line 33, in refresh_cache
    import q2cli.core.cache
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 285, in <module>
    CACHE = DeploymentCache()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 61, in __init__
    self._state = self._get_cached_state(refresh=refresh)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 107, in _get_cached_state
    self._cache_current_state(current_requirements)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 205, in _cache_current_state
    state = self._get_current_state()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/core/cache.py", line 253, in _get_current_state
    plugin_manager = q2cli.util.get_plugin_manager()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2cli/util.py", line 291, in get_plugin_manager
    return qiime2.sdk.PluginManager()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/qiime2/sdk/plugin_manager.py", line 67, in __new__
    self._init(add_plugins=add_plugins)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/qiime2/sdk/plugin_manager.py", line 105, in _init
    plugin = entry_point.load()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2516, in load
    return self.resolve()
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2522, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/__init__.py", line 18, in <module>
    importlib.import_module('q2_types.feature_data')
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data/__init__.py", line 11, in <module>
    from ._format import (
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data/_format.py", line 18, in <module>
    from ..plugin_setup import plugin
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/plugin_setup.py", line 17, in <module>
    from q2_types.feature_data_mag import MAG, NOG
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data_mag/__init__.py", line 11, in <module>
    from ._format import (
  File "/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data_mag/_format.py", line 11, in <module>
    from q2_types.feature_data import DNAFASTAFormat
ImportError: cannot import name 'DNAFASTAFormat' from partially initialized module 'q2_types.feature_data' (most likely due to a circular import) (/Users/rischv/miniconda3/envs/qiime2-tiny-2024.2/lib/python3.8/site-packages/q2_types/feature_data/__init__.py)

Hey @VinzentRisch,
Happy to help! I'll take a look at this shortly.

Hey @VinzentRisch,
Apologies for the delay on this! I've been playing around with this locally, and this is definitely a tricky one. I'm going to try a few different things to work around these circular imports, but this may be something we want to discuss in our joint lab meeting next month. I suspect this will be something that will keep coming up as we add more types/formats from the metagenomic ecosystem.

I'm going to give this a shot as well. q2-types has caused import issues for basically everyone at this point. I think a refactor may be in order.

Once https://github.com/qiime2/q2-types/pull/342 is merged, it should be relatively straight-forward to rebase this PR ontop of it. The registration will end up in _deferred_setup/__init__.py for the relevant submodules.

Hey @VinzentRisch,
We should be all set to continue working on this one. I've fixed the merge conflicts and we no longer end up with import cycles.
Of note, I did delete one method in https://github.com/qiime2/q2-types/pull/334/commits/08e3fd181a492bdf4a2bf76df092fac33ec7744b which operated on a format that was removed in #338

We might consider moving the plugin method registrations into the smaller _deferred_setup/__init__.py files, but it honestly doesn't matter all that much, so your call there.

§ qiime types
Usage: qiime types [OPTIONS] COMMAND [ARGS]...

  Description: This QIIME 2 plugin defines semantic types and transformers
  supporting microbiome analysis.

  Plugin website: https://github.com/qiime2/q2-types

  Getting user support: Please post to the QIIME 2 forum for help with this
  plugin: https://forum.qiime2.org

Options:
  --version            Show the version and exit.
  --example-data PATH  Write example data and exit.
  --citations          Show citations and exit.
  --help               Show this message and exit.

Commands:
  collate-feature-data-mags    Collate mags
  collate-orthologs            Collate Orthologs
  collate-sample-data-mags     Collate mags
  partition-feature-data-mags  Partition MAGs
  partition-orthologs          Partition orthologs
  partition-sample-data-mags   Partition MAGs

Hi @ebolyen
Thanks for the refactor! now everything seems to be working fine.
I finished everything and added some data and code in the PR description to test all actions.

Thanks for all of the work on this @VinzentRisch!
@misialq, any chance you could review this one? @ebolyen has already reviewed the structure and said that it all looks good, so it's just the functionality at this point that needs review and we think you might have the most context there. If you can't get to it, I can review this one but it'll be a few weeks as I'm about to head out to teach a workshop next week and then am tied up the week after that - so we're looking at the 2nd week of September before I can get to it.

Hey @gregcaporaso, sure thing - I'll give it a go this week!


Missing "pipecd.dev/variant" label of Kustomize app

What happened:
I'm running into an issue that the app packaged by Kustomize doesn't come with pipecd.dev/variant: primary label.
kubectl get po podinfo-fd455bd69-4mz45 -o yaml
apiVersion: v1
kind: Pod
metadata:
  annotations:
    prometheus.io/port: "9797"
    prometheus.io/scrape: "true"
  creationTimestamp: "2020-09-10T06:03:11Z"
  generateName: podinfo-fd455bd69-
  labels:
    app: podinfo
    pod-template-hash: fd455bd69

That causes being unable to rollout the primary.
What you expected to happen:
How to reproduce it:
Environment:

piped version: v0.6.0
control-plane version: v0.6.0-3-gc071948
Others:

It was caused by just lacking of spec.selector.matchLabels.pipecd.dev/variant and spec.template.metadata.labels.pipecd.dev/variant.


Add Hong Yao's image and responsibilities in AboutUs

Fixes #21

Codecov Report

Merging #40 (0338142) into master (2a5610c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #40   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 2a5610c...0338142. Read the comment docs.


Open handle failed! Win10 + Python3

freshly installed Python 3.5.4 on Windows 10 (version 1803 OS Build 17134.407) using official install executable.
using latest master branch (as of writing) of pywifi. installed with pip. install succeeded no errors:
running this code:
import pywifi

wifi = pywifi.PyWiFi()
wifi.interfaces()

results in:
pywifi 2018-11-16 17:12:02,183 ERROR Open handle failed!
pywifi 2018-11-16 17:12:02,183 ERROR Enum interface failed!

the return value from WlanOpenHandle  function is 1062, not sure what that maps to. I'll continue to investigate

googling this error code suggests: "WLAN AutoConfig" was not running

starting WLAN AutoConfig service manually from the Services Desktop App fixed my issue.


standard-tests: add test for structured output

add test for structured output
fix bug with structured output for Azure
better testing on Groq (break out Mixtral + Llama3 and add xfails where needed)

Standard tests pass on all packages that implement them: https://github.com/langchain-ai/langchain/actions/runs/9714768840


Add publication: ACL 2024 Language + Molecule

Congratulations @cth127 !

Gah. a bug in bibtex. fix it.

completed by #43


Update package repo to focal for ubuntu 20

This fixes the error encountered during shinyapps.io deployment about stringr package missing

🚀


Added Icons to Navbar

Hello Project Admin
I Added Icons to Navbar - Solved #1168
Close #1168
Kindly Please go through this PR @anuragverma108

@sailaja-adapa this PR has merge conflicts , & u haven't use the PR templates correctly to submit this PR , So  I am closing this PR , but u still have one more final day to submit your PR , please try to submit your PR ASAP following all the rules


command fails

should be:
az account list --query "[?isDefault == `false`]"

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

ID: e60c12ef-d293-5626-c7aa-c7e9241bf975
Version Independent ID: 7acca1dd-b735-7e1f-afd9-2b5b6ea06b1c
Content: JMESPath query command results with Azure CLI
Content Source: docs-ref-conceptual/query-azure-cli.md
Service: azure-cli
GitHub Login: @dbradish-microsoft
Microsoft Alias: dbradish

@carlrab Thank you for your feedback . We will investigate and update the thread.

@carlrab Thank you for reporting this . We have initiated a change request for the content team to review. Once approved the change would be live.
Thank you .


Missing cargo-generate 0.5.0 binary build for Apple Silicon

🐛 Bug report
I have been able to install and run wrangler (https://github.com/cloudflare/wrangler/releases/tag/v1.16.0-durable-objects-rc.0) on fresh Apple Silicon M1, without homebrew or NPM by doing:
curl https://sh.rustup.rs -sSf | sh
git clone https://github.com/cloudflare/wrangler.git
cd wrangler
git checkout custom-builds-rc
cargo install --path . --debug

However wrangler generate looks for the cargo-generate binary inside a wrangler cache folder. If it's not there, it tries to download https://workers.cloudflare.com/get-binary/ashleygwilliams/cargo-generate/v0.5.0/aarch64-apple-darwin.tar.gz which does not exist.
I found a workaround by installing cargo-generate locally: cargo install cargo-generate  --version 0.5.0 --features vendored-openssl (note --features vendored-openssl was required for installation to work on Apple Silicon without homebrew) and then overwriting the wrangler run_generate() binary_path to use the full path of my local cargo-generate binary /Users/USERNAME/.cargo/bin/cargo-generate
It would be great if @ashleygwilliams could build and upload a cargo-generate 0.5.0 binary for Apple Silicon, and I think this would prevent the need for this workaround.
Then we could add clear instructions for installing wrangler on Apple Silicon.

Hello! We do not support M1 yet, but we are actively working on it! #1685

This should be fixed in the latest release. Please give it a try and let us know if you still are hitting any errors:
npm i -g<EMAIL_ADDRESS>or cargo install wrangler --version 1.16.1


v2.0

Changes:

Lazy load qs module instead of requiring querystring (breaking change)
Added all the other Request object methods and properties
Added all the other Response object methods and properties
Updated dev deps
Use inherits module instead of util
Added .locals to router
Added support for using get and header to get referrer and content-type
Patches: fd91171, 6538a78, eef1cb7, 3b58772, 365cd31, 0da2337, 1cb0a4f, 974746a, 5adb024, 45ae656e946be07ae6d67431d10f2514dbca8364

Closed at e28e3e57bccc1984a6c86631b327d9fd2617e759


Fix: Missing time series datapoints in druid show up as absurd values in line chart

CATEGORY
Choose one

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
When a time series query is passed on to druid, if skipEmptyBuckets is not set to True, druid  will fill the missing time buckets in the interval with the default value of the aggregators. This leads to absurd values showing up in the time series line or bar charts such as -9223372036854775808 (which is Long.MIN_VALUE in Java) if you do a MAX aggregation on Long metric and<PHONE_NUMBER>854775807 if you do a MIN aggregation on Long metric. SUM aggregation returns a default of 0.0
Such large values completely distort the chart.

The SQL lab tests are failing but I haven't made any changes in SQL lab. Can anyone look into this?


Handle Cookie Challenge for Redirects in ImgProxy

I am encountering an issue when using ImgProxy with URLs that involve cookie-based challenges. Specifically, when a fetch request to a URL results in a redirect to the same URL, a cookie is set by the server. The fetch request then needs to be retried using the new cookie. Currently, this scenario results in an error message: "stopped after 10 redirects".
For example, trying to access the following URL raises this issue:
https://nameniko.shop/wp-content/uploads/2024/09/250082_1.jpg
While I understand that ImgProxy supports cookie passthrough, it appears that this functionality is not working correctly for URLs that involve redirects. As a result, ImgProxy fails to handle these types of requests, making it impossible to process images from such sources.
Request:
Please enhance ImgProxy to properly handle situations where a fetch request is redirected, and a cookie is set by the server. Specifically:

Detect Redirects with Cookies: When a fetch request encounters a redirect, detect if a cookie has been set by the server.
Retry with New Cookie: Use the newly set cookie to retry the fetch request.
Limit Redirect Loops: Ensure there is a sensible limit on the number of retries to avoid infinite loops but make sure the first few retries with new cookies are attempted.

This enhancement will significantly improve ImgProxy's compatibility with websites that use cookie-based challenges, ensuring that images can be fetched and processed correctly.
Thank you for your attention to this matter. I look forward to your response.

i solved it in this merge request.
@DarthSim

Solved with #1334


line 497, in      logger_cfg = lightning_config.logger or OmegaConf.create() Error/Missing key logger/Help me

Got same error

The error you're encountering is likely due to a version mismatch between your current pytorch-lightning or omegaconf and the versions required by the program. I recommend installing the specific versions of pytorch-lightning and omegaconf as listed in the README file. This should resolve the issue.

@wzhouxiff thanks for your reply. I am sharing my solution:
conda create -n rf++ python=3.8
pip install torch==1.10.0+cu111 torchvision==0.11.0+cu111 torchaudio==0.10.0 -f https://download.pytorch.org/whl/torch_stable.html
pip install -r RF_requirements.txt
pip install ninja
pip install pillow=9.5.0

thanks for your reply !!  I have a  try
---Original---
From: @.>
Date: Tue, Aug 6, 2024 12:46 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [wzhouxiff/RestoreFormerPlusPlus] line 497, in <module>    logger_cfg = lightning_config.logger or OmegaConf.create() Error/Missing keylogger/Help me (Issue #15)
@wzhouxiff thanks for your reply. I am sharing my solution:
conda create -n rf++ python=3.8 pip install torch==1.10.0+cu111 torchvision==0.11.0+cu111 torchaudio==0.10.0 -f https://download.pytorch.org/whl/torch_stable.html pip install -r RF_requirements.txt pip install ninja pip install pillow=9.5.0
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Increase timeouts in CI

I notice that CI is failing. It seems like there are timeout issues. I've tried increasing the timeouts on the failing tests.

It seems like the timeout probably isn't the issue. I'll close this issue and do some digging.


Automatically publish conatiner image

This patch adds a Dockerfile and a GitHub Actions workflow to automatically build and publish container images for every branch and pull request.
If the source is a pull request, the workflow will also add a comment to explain how this can be used to easily test this pull request:

Use docker or podman to test this pull request locally.
Local test with mock data
podman run --rm -it -p <IP_ADDRESS>:3000:3000 ghcr.io/lkiesow/opencast-admin-interface:pr-5

Proxy data from develop.opencast.org
podman run --rm -it -p <IP_ADDRESS>:3000:3000 -e PROXY_TARGET=https://develop.opencast.org ghcr.io/lkiesow/opencast-admin-interface:pr-5

It may take a few seconds for the interface to spin up.
It will then be available at https://<IP_ADDRESS>:3000.

Right now, this will run on all pull requests instead of just running on pull requests for members of the team Developer like we have for the Opencast Editor. We have to authorize unknown contributors anyway before GitHub runs workflows at all and we also only use the package repository for testing.

[...] and we also only use the package repository for testing.

Please don't make that assumption. I want to add production-ready images for all static front-end repositories (similar to https://github.com/opencast/opencast-editor/pull/684).

As we discussed at the technical meeting:

This is just for now
This shouldn't conflict with prod containers

Looks fine to me.


CNI plugin uses the same if_index for all the containers

Upstream CNI plugin (bridge) doesn't set the if_index on the container side properly. All the containers end up with the same if_index making it hard to track which container is associated with a particular veth.
Need to figure out the code and submit a PR to upstream CNI.
CC: @aemneina @wlan0 @ibuildthecloud

Instead of fixing the if_index, another idea is to use the container ID in veth names instead of random ones.

Fixed in Network Services v0.2.2, network-manager:v0.7.2

Tested with Network Services v0.2.2, network-manager:v0.7.3.
Containers on managed network have the veth names starting with "vethr" and the first few 10 characters of the container's docker ID.


Impact of number of GPU

I have only 2 GPU. I'm trying to train the Abstractive model on these 2 GPU and I was wondering was is the impact of training it on only 2 GPU instead of 4 like mentioned in the paper.
Since every GPU run :
https://github.com/nlpyang/PreSumm/blob/d5954c9890da487a40617ca77c75b02feff0d2eb/src/train_abstractive.py#L75
It seems to me that hyperparameters should be adapted based on the number of GPU used :

Training on 50k steps on 1 GPU : in total it is 50k steps.
Training on 50k steps on 3 GPU : this means each GPU will run 50k steps, resulting in a training of 150k steps (?)

Did I understand the code right ?
What are the others hyper parameters that I need to change based on the number of GPU used ? warmup steps ? save checkpoint steps ?

multi-gpus is used to increase batch_size, with 3 gpus, you will have a batch size that is 3 times of a single gpu.
If you only have 2 gpus, you need to multiply the accum_count by 2, to use gradient accumulation for a same batch size.

I am trying to train this with only 2 GPUs too and now it is expected to will take 4 days (~100hours). So I am interesting that how long does it take to train this model(BertSumAbs) that with 4 GPUs and default setting? Thanks~

@tungloong with 4 gpus it takes about 2 days.

If you only have 2 gpus, you need to multiply the accum_count by 2, to use gradient accumulation for a same batch size.
@nlpyang

I'm sorry to bother, but I'm still confused...

4 GPU : let's say batch size is 64 (and accum_count is 1) and we train for 10k steps. The model sees 10k * 64 = 640k samples through 10k updates.
2 GPU : if we do like you say, batch size is going to be 32 (and we set accum_count to 2) and train for 10k steps. The models sees 10k * 32 = 320k sample through 5k updates (because of accum_count).

I understand why we need to multiply accum_count by 2, but I think we also need to multiply training_steps by 2, no ?

My understanding,
given a total step = 10,000 with actual batch_size (in average bec its dynamic) = 4, grad_accum = 5, and gpu_size = 4, then number of instances reached during the training is
10,000 * 4 * 5 * 4 = 800,000 instances


Add 'file://' prefix to sourceURL when loading a script from a file.

When loading a bundle from a file (usually in the case of the release version of the application), we should prepend "file://" to the file path when passing a "sourceURL" to the JSC bridge.
This 1) unifies iOS and Android when it comes to style in which they show the file name of the script in a traceback, and 2) making this URL and actual URL allows it to be parsed by something like TraceKit (https://github.com/csnover/TraceKit), which is used by Sentry, for instance, to be able to easily log tracebacks.

Makes sense to me. How did you test this? Did you test both branches?

I did not test both branches...as I'm actually not sure how to make the "assets://" stuff work haha (but I'm happy to test if you have any ideas).
Regarding testing the other code path...really, I just ran some apps, and threw Errors. This change really only makes a change to the url that's reported in tracebacks...you can actually see a comment in this same file alluding to that fact a few lines down:
"Providing correct {@param sourceURL} of downloaded bundle is required for JS stacktraces to work correctly and allows for source maps to correctly symbolize those."
I'm just doing the same thing for the bundle that's loaded from a file in release mode.

Ah OK, you've tested only the loadScriptFromFile branch, correct? I don't have much context on this part of the codebase but I'd say please change only the code you were able to test.
I thought loadScriptFromAssets was called in release mode when the JS bundle is packaged inside the APK (docs) and loadScriptFromFile was called when we're loading the JS bundle from a custom file, such as when using services like Microsoft Code Push. But might be mistaken.

@mkonicek I think you're probably right.
I will test this. I don't expect it to be an issue, but will test to be sure and report back.

I don't think the change to the "assets://" code path will work. The bridge API requires that second param to be the name of the asset, not a URL.

@geof90 thanks for that info. I believe i've fixed it, and have tested both code paths.

Oh those changes in OnLoad.cpp are needed for TraceKit to work even if we're loading the bundle from assets, correct?
This is a risky change but I trust you! If you're sure it won't break anything ship it :)

I should find some time to add an Android UI test for both a debug and a release build into CircleCI.

@mkonicek Yah so the general concept here @mkonicek is that the sourceURL argument that gets passed around really has no impact on WHAT actually gets loaded -- only how the source that is loaded is attributed.
The JS is downloaded and fetched (from either a server or from assets or from a local file) before we ever get to this point in the code.
Without this change...the file/line number attribution in stack traces look something like this:
When loading from assets:
index.android.bundle:15214:16

When loading from a file:
/data/.../index.android.bundle:15214:16

This differs from iOS, where we get back an NSURL, no matter whether we're loading from the server or loading from the local bundle. That means that on iOS, we don't have the issue I describe, because the stack traces contain things like:
file://var/applications/.../index.ios.bundle:15214:16

Which is more inline with how browsers report stack traces. For reliable parsing of these traces, the "file://" is key. I also think that semantically it makes sense, in both loading from assets as well as loading from the local file system.

^^ If that makes sense to you @mkonicek, I'll ship after you give the :+1:

Cool, thanks for the explanation @skevy! So for assets, the sourceURL would be e.g. "file:///index.android.bundle:15214:16" and that's consistent with iOS? Sounds good, shipit :)

@facebook-github-bot shipit

Closed in https://github.com/facebook/react-native/commit/f2a60a202fd3dd737a968820c7941686ed1d9ddb but not automatically, weird.

Thanks for the PR @skevy!

The calls to evaluateScript here would have been the right place to add that prefix: https://github.com/skevy/react-native/blob/master/ReactAndroid/src/main/jni/react/JSCExecutor.cpp


VS Code - Produtividade Infinita

What does this PR do?
Add resource(s)
For resources
Description
The VS Code Produtividade Infinita (infinite productivity) ebook is a compilation of tips and teachings on how to improve your productivity as a programmer using VS Code
Why is this valuable (or not)?
You can improve your producitivity
How do we know it's really free?
I'm the author, it's 100% free and it's hosted on GIT Hub :)
The e-book is made with MarkDown.
For book lists, is it a book? For course lists, is it a course? etc.
It's a Book.
Checklist:

[X] Read our contributing guidelines
[X] Search for duplicates.
[X] Include author(s) and platform where appropriate.
[X] Put lists  in alphabetical order, correct spacing.
[X] Add needed indications (PDF, access notes, under construction)

Followup

Check the output of Travis-CI for linter errors!

It's not so helpful to add a category for a single resource. Try to find another category. Or in this case, make the category broader to attract more items - maybe "Visual Studio"?

Hi Eric, thanks for your feedback :)
Visual Studio very is different from Visual Studio Code, that's why i cant change it to a broader version like you said.
Currently, VS Code is one of the most used Code Editor (as Stackoverflow Developers Survey found).
VS Code works for almost all programming languages. So, on my e-book i compiled more than 250 tips about VS Code its self and more than 100 extensions for a lot of languagens, like C#, Python, Java, JavaScript, Html, C++, PHP ...
So, it's unfair if i put it only in Python because it's my favorite language :)
But if you have any tip to change it, let me know

OK, so it's more like emacs and vim, which have categories in our english list. Maybe those could be grouped under "development tools" or "programming tools"?  Also, did you mean to add to the courses list instead of the books list?

I agree with @eshellman. Resource needs moving to programming-books-pt_BR.md 😉
In this listing I propose join Vim category as:
## IDE / Editores
   - Your book
   ### Vim
      - vim books 1
      - ...

And move/grouping all in Agnósticos section
## Agnósticos
   ...
   ### IDE / Editores
      - Your book
      #### Vim
         - vim book 1....
         - ...
   ...

please resolve conflicts.


AttributeError in amg.initialize when using tiled pre-comuted embedding.

Hey @constantinpape,
micro-sam is excellent. I really enjoy using it.
I'm trying to process larger 2D images and I'm getting an error when executing amg.initialize with image_embeddings parameter.
You can reproduce it with this:
import napari
from tifffile import imread
from micro_sam import instance_segmentation, util

if __name__ == "__main__":
    image = imread("Fluo-N2DL-HeLa/01/t000.tif")

    predictor = util.get_sam_model(device="cuda:0")
    amg = instance_segmentation.AutomaticMaskGenerator(predictor)

    embeddings = util.precompute_image_embeddings(
        predictor, image,
        # after commeting the line below works
        save_path="/tmp/micro-sam/embeddings.zarr", tile_shape=(1024, 1024), halo=(128, 128)
    )
    amg.initialize(image, embeddings, verbose=True)

    instances_amg = amg.generate(pred_iou_thresh=0.88)
    instances_amg = instance_segmentation.mask_data_to_segmentation(
        instances_amg, shape=image.shape, with_background=True
    )

    v = napari.Viewer()
    v.add_image(image, colormap="magma")
    v.add_labels(instances_amg)

    napari.run()

Error:
Traceback (most recent call last):
  File "/home/jordao/miniconda3/envs/sam/lib/python3.10/site-packages/zarr/hierarchy.py", line 520, in __getattr__
    return self.__getitem__(item)
  File "/home/jordao/miniconda3/envs/sam/lib/python3.10/site-packages/zarr/hierarchy.py", line 500, in __getitem__
    raise KeyError(item)
KeyError: 'ndim'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/jordao/sam_bug.py", line 18, in <module>
    amg.initialize(image, embeddings, verbose=True)
  File "/home/jordao/miniconda3/envs/sam/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/jordao/miniconda3/envs/sam/lib/python3.10/site-packages/micro_sam/instance_segmentation.py", line 442, in initialize
    util.set_precomputed(self._predictor, image_embeddings, i=i)
  File "/home/jordao/miniconda3/envs/sam/lib/python3.10/site-packages/micro_sam/util.py", line 554, in set_precomputed
    assert features.ndim in (4, 5)
  File "/home/jordao/miniconda3/envs/sam/lib/python3.10/site-packages/zarr/hierarchy.py", line 522, in __getattr__
    raise AttributeError

Commenting the indicated line avoids the issue.
The sample data can be downloaded from here

Hi @JoOkuma ,
thanks for the error report, I will have a look later.

micro-sam is excellent. I really enjoy using it.

Thanks for the feedback!
Since you have some relevant expertise ;) : if you have some more detailed feedback (or would like to contribute at some point) that would be highly welcome.
I will send out a small "micro-sam newsletter" in the next few days, I will just go ahead and include you.

Ok, I double checked. This is actually expected behavior:
if you use tiled embeddings you need the class TiledAutomaticMaskGenerator instead of AutomaticMaskGenerator.
If you replace it all works. (Both classes have the same interface)
Note: I just added a convenience function get_amg that builds the correct AMG class. (You have to pass the boolean is_tiled to indicate whether you use tiling or not.)

Thank you!


unable to upload via the api/upload endpoint.

I am getting the error "files are empty" in response to my api/upload request.
// build the api endpoint
      let url = cms + `/api/upload`
      // initialize request headers
      let h = new Headers();
      // append the authorization header
      h.append('Authorization', `Bearer 12345`)
      // append the content type header based on the file type of this file
      h.append('content-type', banner[0].type)
      // initialize the FormData object
      let fd = new FormData();
      // append the uploaded file to the FormData
      fd.append(`files.banner`, banner[0], banner[0].name);
      // loop through the FormData object and show what it is holding
      for (let key of fd.entries()) {
        // show each object in the console.
        console.log(key[0] + ', ' + key[1]);
    }
      // initialize the request, add the FormData, Headers, and set the method.
      let request = new Request(url,{
        body: fd,
        headers: h,
        method: 'POST',
    });
    // fetch the request.
      fetch(request)
        // yahtzee. what did is say?
        .then((response) => {
          // show the response in console
          console.log(response)
      })
    }

i rebuilt my fetch method just to test, but the error is the same:
async function uploadFile(file: FileList){
      // initialize the FormData object
      let fd = new FormData();
      // set the FormData value to the uploaded file.
      fd.set(`files.banner`, file[0], file[0].name);
      // we've got what we need. send it to the API.
      const res = await fetch(cms + `/api/upload`, {
              method: 'POST',
              headers: {
                  // use the locally stored JWT as a bearer token authorization
                  Authorization: `Bearer ${$page.data.token}`,
                  // we are sending json
                  'accept': 'application/json',
                  'content-type': file[0].type,
              },
              // update the user data by key and value, which is sent through the value of the form fields.
              body: fd
        });
        let response = await res.json();
        if (res.ok) {
          console.log(response)
        } else {
          console.log(response.error)
        }
    }

Hi @alucav, sorry, but we can not help you for your problem, I suggest you to ask directly on Discord server of Strapi or on the Strapi Forum. Issue report here due to be related with the provider google cloud storage.